\TwigBridge\Extension\Laravel\Session is broken

\TwigBridge\Extension\Laravel\Session type-hints against \Illuminate\Session\SessionManager but I'm pretty sure it should be \Illuminate\Session\Store.
What you had there was working fine until just recently when I updated my packages. I wish I could tell you what exactly broke it, but I couldn't get it working again even after I rolled all my packages back to what I had in my old composer.lock file.
I upgraded from Laravel 5.2.5 to 5.2.45.
My log blows up with errors; here's just one of the exceptions I get:
Next exception 'ErrorException' with message 'An exception has been thrown during the compilation of a template ("Method Illuminate\Session\SessionManager::token() does not exist") in "partials/head.twig".' in /srv/myproject/resources/views/partials/head.twig:-1
Stack trace:
#0 /srv/myproject/vendor/rcrowe/twigbridge/src/Engine/Twig.php(92): TwigBridge\Engine\Twig->handleTwigError(Object(Twig_Error_Syntax))
#1 /srv/myproject/vendor/laravel/framework/src/Illuminate/View/View.php(149): TwigBridge\Engine\Twig->get('/srv/myproject...', Array)
#2 /srv/myproject/vendor/laravel/framework/src/Illuminate/View/View.php(120): Illuminate\View\View->getContents()
#3 /srv/myproject/vendor/laravel/framework/src/Illuminate/View/View.php(85): Illuminate\View\View->renderContents()
#4 /srv/myproject/vendor/laravel/framework/src/Illuminate/Http/Response.php(53): Illuminate\View\View->render()
#5 /srv/myproject/vendor/symfony/http-foundation/Response.php(201): Illuminate\Http\Response->setContent(Object(Illuminate\View\View))
#6 /srv/myproject/vendor/laravel/framework/src/Illuminate/Routing/ResponseFactory.php(57): Symfony\Component\HttpFoundation\Response->__construct(Object(Illuminate\View\View), 500, Array)
#7 /srv/myproject/vendor/laravel/framework/src/Illuminate/Routing/ResponseFactory.php(71): Illuminate\Routing\ResponseFactory->make(Object(Illuminate\View\View), 500, Array)
#8 /srv/myproject/app/Exceptions/Handler.php(99): Illuminate\Routing\ResponseFactory->view('errors/500', Array, 500)
#9 /srv/myproject/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(291): App\Exceptions\Handler->render(Object(Illuminate\Http\Request), Object(ErrorException))
#10 /srv/myproject/vendor/laravel/framework/src/Illuminate/Foundation/Http/Kernel.php(103): Illuminate\Foundation\Http\Kernel->renderException(Object(Illuminate\Http\Request), Object(ErrorException))
#11 /srv/myproject/public/index.php(54): Illuminate\Foundation\Http\Kernel->handle(Object(Illuminate\Http\Request))
#12 {main}
[2016-10-02 23:33:24] development.ERROR: exception 'Symfony\Component\Debug\Exception\FatalErrorException' with message 'Uncaught exception 'ReflectionException' with message 'Method Illuminate\Session\SessionManager::token() does not exist' in /srv/myproject/vendor/twig/twig/lib/Twig/Node/Expression/Call.php:262
Stack trace:
#0 /srv/myproject/vendor/twig/twig/lib/Twig/Node/Expression/Call.php(262): ReflectionMethod->__construct(Object(Illuminate\Session\SessionManager), 'token')
#1 /srv/myproject/vendor/twig/twig/lib/Twig/Node/Expression/Call.php(22): Twig_Node_Expression_Call->reflectCallable(Array)
#2 /srv/myproject/vendor/twig/twig/lib/Twig/Node/Expression/Function.php(36): Twig_Node_Expression_Call->compileCallable(Object(Twig_Compiler))
#3 /srv/myproject/vendor/twig/twig/lib/Twig/Compiler.php(105): Twig_Node_Expression_Function->compile(Object(Twig_Compiler))
#4 /srv/myproject/vendor/twig/twig/lib/Twig/Node/Print.php(30): Twig_Compiler->subcompile(Object(Twig_Node_Expression_Function))
#5 /srv/myproject/vendor/twig/twig/lib/Twig/Node.php(117): Twig_Node_Print->compile(Object(Twig_Compiler))
#6 /srv/myproject/vendo' in /srv/myproject/resources/views/partials/head.twig:-1
Stack trace:
#0 {main}

If you check out \Illuminate\Session\SessionManager, it indeed does not have a method called get but \Illuminate\Session\Store does, so I think this is right.
I was only able to figure this out because of Otwell's blog post in which has this snippet:
<?php

class HomeController extends BaseController {

	public function __construct(Illuminate\Session\Store $session)
	{
		$this->session = $session;
	}

}

Can you submit a PR? And does it fix your problem?

I guess same issue here
"Method Illuminate\Auth\AuthManager::user() does not exist"

@SergeyMiracle
https://github.com/rcrowe/TwigBridge/pull/282 - work to me

Same here, but I get An exception has been thrown during the compilation of a template ("Method Illuminate\Session\SessionManager::token() does not exist") in "./layout/default.twig".

Looks like @lbausch beat me to it.  (Thank you!)
That's pretty much the exact fix I did locally, so that should work perfectly.


multiprocessing / queue control for PyWPS

Description
Study multiprocessing for pypws: multiprocessing, gevents, events
Usage of queue system: Celery
Environment

operating system:
Linux
Python version:  2?, 3
PyWPS version: 4.x
source/distribution

This is a duplicate of #237


Fixed and Re-Added Feed, Tweet Search and Hashtag Search

I have fixed the Feed, Tweet Searching and Hashtag Searching. These features have been re-added and also fixed my own issue #668, #696 and #695. (As long as twitter don't disable their api again!)
APK File Upload
Github Release
Source Code

Thank you this version works great

Good to hear 👍
On Sun, 7 May 2023, 15:09 Igaru22, @.***> wrote:

Thank you this version works great
—
Reply to this email directly, view it on GitHub
https://github.com/jonjomckay/fritter/pull/698#issuecomment-1537451212,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5PO6LFLFX4FXG23UMQ6HPLXE6UIZANCNFSM6AAAAAAXYK6NVE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thank you so much for this great update! Fritter without a functioning feed is almost unusable. Don't wanna praise you too soon, but if it keeps working as flawless as it does at the moment, then you're truly a genius in my eyes!

Thank you! 😃
On Sun, 7 May 2023, 16:21 HiFi83, @.***> wrote:

Thank you so much for this great update! Fritter without a functioning
feed is almost unusable. Don't wanna praise you too soon, but if it keeps
working as flawless as it does at the moment, then you're truly a genius in
my eyes!
—
Reply to this email directly, view it on GitHub
https://github.com/jonjomckay/fritter/pull/698#issuecomment-1537466941,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5PO6LA7JWTO5ZCU4UEVR5LXE64W5ANCNFSM6AAAAAAXYK6NVE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks, this version works awesomely, can you please try to implement the fix to expanding tweets with more than 10 comments also?
Fix was posted here: #690

Ok, I will do that at some point.
On Sun, 7 May 2023, 17:40 Sponholz, @.***> wrote:

Thanks, this version works awesomely, can you please try to implement the
fix to expanding tweets with more than 10 comments also?
Fix was posted here: #690 https://github.com/jonjomckay/fritter/pull/690
—
Reply to this email directly, view it on GitHub
https://github.com/jonjomckay/fritter/pull/698#issuecomment-1537488261,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5PO6LEMPJAJH2YCYP4YA3TXE7F6HANCNFSM6AAAAAAXYK6NVE
.
You are receiving this because you authored the thread.Message ID:
@.***>

I cant seem to be able to build this via the GitHub release workflow.
https://github.com/Apacelus/fritter/actions/runs/4908483128/jobs/8764272707

What OS are you using?
Windows/Mac/Linux
On Sun, 7 May 2023, 18:50 Apacelus, @.***> wrote:

I cant seem to be able to build this via the GitHub release workflow.
https://github.com/Apacelus/fritter/actions/runs/4908483128/jobs/8764272707
—
Reply to this email directly, view it on GitHub
https://github.com/jonjomckay/fritter/pull/698#issuecomment-1537503308,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5PO6LE3TTNHMLHCJT7MR2LXE7OHNANCNFSM6AAAAAAXYK6NVE
.
You are receiving this because you authored the thread.Message ID:
@.***>

What OS are you using? Windows/Mac/Linux
…
On Sun, 7 May 2023, 18:50 Apacelus, @.> wrote: I cant seem to be able to build this via the GitHub release workflow. https://github.com/Apacelus/fritter/actions/runs/4908483128/jobs/8764272707 — Reply to this email directly, view it on GitHub <#698 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/A5PO6LE3TTNHMLHCJT7MR2LXE7OHNANCNFSM6AAAAAAXYK6NVE . You are receiving this because you authored the thread.Message ID: @.>

Im using github workflows... I linked the run in my previous comment.
The run uses ubuntu-latest, so that would be ubuntu 22.04:
https://github.com/Apacelus/fritter/actions/runs/4908483128/workflow#L11

run

You cannot use github actions to build the app you have to use android studio!

Thanks, this version works awesomely, can you please try to implement the fix to expanding tweets with more than 10 comments also?
Fix was posted here: #690

I have now merged the pull request into my fork:
Github Release

run

You cannot use github actions to build the app you have to use android studio!

The original project certainly used github actions to build their releases, so it is possible.

They have a different signing key that they have generated using the play
store I think.
On Sun, 7 May 2023, 20:30 Apacelus, @.***> wrote:

run
You cannot use github actions to build the app you have to use android
studio!
The original project certainly used github actions to build their
releases, so it is possible.
—
Reply to this email directly, view it on GitHub
https://github.com/jonjomckay/fritter/pull/698#issuecomment-1537523298,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5PO6LF3X4DWHPVXKDCMIPDXE7Z6XANCNFSM6AAAAAAXYK6NVE
.
You are receiving this because you authored the thread.Message ID:
@.***>

Thanks, this version works awesomely, can you please try to implement the fix to expanding tweets with more than 10 comments also?
Fix was posted here: #690

I have now merged the pull request into my fork: Github Release

Thanks! Using it at the moment, really appreciate the time doing all the fixes.
Cheers!

@TheHCJ thank you for putting in the effort to patch this - I really appreciate the support, but I won't be adding the feed back into Fritter. Please see my comment on issue #668.

I am late for the party. Repo deleted 🥲


Restore ListTile the layout updates

Restores the ListTile layout changes that were reverted:
https://github.com/flutter/flutter/pull/17496
https://github.com/flutter/flutter/pull/17580
The original PRs introduced a layout RadioListTile layout problem that has been fixed:
https://github.com/flutter/flutter/pull/17620


support of null in conditions like {% if previousTime != null %}

Great library. It looks like null is not supported http://jsfiddle.net/zq3ez61h/ Can it be extended to support nulls?

Hello,
got the same problem with (both "row" and "index" are defined!):
{% if row[index] != undefined) %}
in the compiled output the undefined/null gets recognized as variable:
this.__cache__.index !== undefined && this.__cache__.row !== undefined && this.__cache__.undefined !== undefined

Currently undefined ignored on update in Monkberry. I'll add support for them in next Monkberry v5.

+1


add opencl to compilation

Signed-off-by: Xinyu Wang<EMAIL_ADDRESS>Added opencl support to compile tvm.

OpenCL dependencies will be needed. In ModelZoo they are installed here: https://github.com/autowarefoundation/modelzoo/blob/b2db1b3cd97494c9fcfee3084c38a64a025cca18/scripts/tvm_cli/install_tvm.sh#L35
In this package they would need to be added to package.xml, with ocl-icd-opencl-dev and opencl-headers.

https://github.com/autowarefoundation/autoware.universe/issues/2186#issuecomment-1308209761 made me think that we could probably do a better job of having correct dependencies for this package. Currently everything is "depend", but some are probably only "build_depend". And the *-dev packages would only be needed at build time and would have a runtime dependency on the non-dev package. But in rosditro I think it only lists the *-dev, so that would need to be modified first.
Which I guess we can do later.


Shortcut icon focuses wrongly when using editor type

Issue description:
When using partial refresh, for editor type, the shortcut icon after clicking leads to "add media" button, instead of focusing on textarea. I think this is a bug.
Version used:
3.0.33
Using theme_mods or options?
theme_mods

Does the media field by any chance also control the same area?

close? #2026


Broken help command computer_reader.help()

This is not a bug, it's a feature 😄
Help returns the help text as string
print( computer_writer.help() )


EADDRINUSE when testing

I'm using the test helper robot in some Jasmine specs for a Hubot script. My script doesn't make any HTTP requests so I don't need an HTTP server. Unfortunately, the test robot's class is not exported from tests.coffee, so I'm not able to initialize the helper with httpd set to false. When I run the specs, I get this error interleaved with the output:
node.js:201
        throw e; // process.nextTick error, or 'error' event on first tick
              ^
Error: listen EADDRINUSE
    at errnoException (net.js:642:11)
    at Array.0 (net.js:743:26)
    at EventEmitter._tickCallback (node.js:192:40)

I've tried setting the PORT environment variable to different things but always get that error. If I open up tests.coffee within node_modules and manually change the httpd parameter to false, then I don't get this error running the specs, but the process hangs afterward and I have to control-C out of it.
TL;DR: The Helper class itself should be exported from tests.coffee. How can I use that class without getting the above error or hanging the process?

The tests.coffee isn't supported for running outside of the hubot test suite. I've also never used Jasmine for node unit tests, and prefer to stick with an actual node-specific test framework.
You should look at the google_images_test.coffee for an example of testing scripts. The HTTP part of hubot is for incoming HTTP requests, not outgoing ones as well.

It looks like this is happening because the helper bot is not being shut down. I managed to make it exit cleanly by instantiating a bot with the shell adapter and calling #shutdown after each test. I notice that in tests.coffee, when the server closes, #close is called on the helper bot, but it doesn't seem to have a method with that name. Is it possible the helper bot's #stop method was supposed to be called close?


Drop support for Avro

We do not use it, nor test it. Only polars provides support for it, DuckDB does not

Closing as I now just added a workaround using polars for duckdb


Add Github sample app

Github sample app can be added as a part of this repo or as a separate repo. You can find the sample in the old repo - https://github.com/Arello-Mobile/Moxy/tree/master/sample-github

I think this is resolved now by #114, can be closed.


Ctulo2

Types of changes

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

Motivation and Context / Related issue
Shiwa is dirty (due to me).
So I reimplemented it.
I also modified a few things.
This also includes an improvement of cGA.
Other improvements are expected.
The plan is (i) merge this PR (ii) check performance (iii) if everything fine, remove Shiwa and rename Ctulo as NGO10 or something like that.
How Has This Been Tested (if it applies)
CI and a few preliminary tests.
Additional tests have been added.
Checklist

[x] The documentation is up-to-date with the changes I made.
[x] I have read the CONTRIBUTING document and completed the CLA (see CLA).
[x] All tests passed, and additional code has been covered with new tests.

Please don't remove Shiwa so soon, now that it was advised to use it and published, it is not user-friendly to remove it.
When NGO2 is working, I would recommend to make Shiwa inherit from it and just add a deprecation warning to it. This way it will still works for some time, remove most of the code, and allow users to adapt.
Also, if there are more NGO versions coming, we need to make NGO point to the latest version. This way we can keep NGO2, NGO3 etc as long as they make sence, and advise people to always use NGO as it is the one supposed to have the best results.

Also, in what is it cleaner that Shiwa? :s

Ok for a NGO pointing to NGO-last.
I have no plan to remove Shiwa in the near future, and ok I will not remove it at all actually given that there are already several publications mentioning it.

Also, in what is it cleaner that Shiwa? :s

Shiwa has dead branches.
It is also longer than necessary, with several equivalent branches.


Update sbt-sonatype to 3.9.2

Updates org.xerial.sbt:sbt-sonatype from 3.8.1 to 3.9.2.
GitHub Release Notes - Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "org.xerial.sbt", artifactId = "sbt-sonatype" } ]

labels: sbt-plugin-update, semver-minor

Codecov Report

Merging #1220 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1220   +/-   ##
=======================================
  Coverage   81.07%   81.07%
=======================================
  Files          55       55
  Lines        1009     1009
  Branches       42       42
=======================================
  Hits          818      818
  Misses        191      191

Impacted Files
Coverage Δ

core/src/main/scala/io/finch/Endpoint.scala
72.22% <0%> (ø)
: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 78b2f15...83f0100. Read the comment docs.


feat: add memory and hash routing to react-router

Hey! 👋
First of all, thank you for this library, which is an interesting and enjoyable accelerator to use.
On one of my projects using react-router, I needed to use memory-routing (via createMemoryRouter). Unfortunately, it didn't seem possible with Generouted.
Here's a PR that adds <HashRoutes /> and <MemoryRoutes /> to the react-router implementation.
If you see any adjustments that need to be made, I'll be happy to do them. 🫡
Have a great day ✌

Hey @Jaouan, thanks for your message and submitting this PR.
Tbh I don't see the need to export HashRoutes or MemoryRoutes as the routes object (not Routes component) was exported for this customization purpose. The default app entry with the default browser Routes looks like that:
import { createRoot } from 'react-dom/client'
import { Routes } from '@generouted/react-router'

const container = document.getElementById('app')!
createRoot(container).render(<Routes />)

In order to use HashRouter with the current exports, it would be exactly the same as what you exported in this PR:
import { createRoot } from 'react-dom/client'
import { RouterProvider, createHashRouter } from 'react-router-dom'
import { routes } from '@generouted/react-router'

const Routes = () => <RouterProvider router={createHashRouter(routes)} />

const container = document.getElementById('app')!
createRoot(container).render(<Routes />)

I think we can close this PR, and I'm thinking to include this point in a (new) FAQ section on the docs.

Hi, I forgot that ‘routes’ is also exported. 🫣
Your example is enough to handle memory/hash routing. 👍

No worries, glad that helped!


Convert scanned amount to correct unit

If the user sets the display unit to anything other than BTC and scans a payment request QR code, the scanned amount (which is always given in BTC) needs to be converted to the user's selected unit.

Great, thank you for this fix. We will publish this on the next version.


initialValue should be populated even for invalid numbers

Describe the bug
If phone number passed as initialValue is not a valid phone number, the initialValue is skipped and textfield is left empty.
This is problematic in cases when form data is prefilled from existing sources and user won't have a chance to correct the existing wrong phone number.
Package version
all versions
To Reproduce
Steps to reproduce the behavior:

Pass an invalid phone number as initialValue intentionally
Run the app
The textfield will be empty

Expected behavior
The textfield should be populated with phone number even if it is not valid. Validation error can be shown if the number is invalid instead.
If you think this change won't be valid in all cases, a flag to enable/ disable this change would also be very useful.

+1
Or print maybe print an error statement in the terminal telling that the number is not valid so as a developer you would know that your code works and you have to implement an extra check or so :)

Any workaround or solution to this issue yet ?


Unable to Cast to Chromecast

I have an issue / a question (pick one) about Vlc.DotNet.
Generic information

Vlc.DotNet version : 3.1.0-develop364
Vlc.DotNet project used : Windows Application
libvlc version : nuget x64 <IP_ADDRESS>
.net version : .net 4.7.2
Project language : C#
Project build architecture : AnyCPU
Operating system : Windows 10 x64

Summary
Code:
var currentDirectory = Path.GetDirectoryName(Assembly.GetEntryAssembly().Location);
var libDirectory = new DirectoryInfo(Path.Combine(currentDirectory, "libvlc", IntPtr.Size == 4 ? "win-x86" : "win-x64"));
        using (var mediaPlayer = new Vlc.DotNet.Core.VlcMediaPlayer(libDirectory))
        {
            var mediaOptions = new[]
            {
                ":sout=#chromecast{ip=<IP_ADDRESS>, demux-filter=demux_chromecast}",
                ":sout-keep",
                "--file-logging", "-vvv", "--logfile=Logs.log"
            };

            mediaPlayer.SetMedia(new FileInfo(@"Files\Atomic.mp3"), mediaOptions);
            mediaPlayer.Play();
        }

Result:
[0910e338] main tls client error: connection error: Interrupted function call
[09159858] stream_out_chromecast stream out error: cannot load the Chromecast controller (Failed to create client session)
[091370e8] main stream output error: stream chain failed for `chromecast{ip=<IP_ADDRESS>, demux-filter=demux_chromecast}'
[09159d78] main input error: cannot start stream output instance, aborting

[*] I confirm that my issue doesn't happen in VLC itself.
VLC:
"C:\Program Files\VideoLAN\VLC\vlc.exe" "C:\Temp\Atomic.mp3" --sout "#chromecast{ip=<IP_ADDRESS>, demux-filter=demux_chromecast}"

Which works
VLC version: 3.0.4
Can anyone let me know what i suspect I'm doing wrong?
Thanks,
Richie

Not sure if chromecast is supported as-is with libvlc, and I've never seen anyone trying with Vlc.DotNet.
LibVLCSharp however has tested support for chromecast, I'd recommend that you try it 🙂

I've had a play around with LibVLCSharp and just got it working, so will look at using that.
VLC.dotnet looked to be the "neater" solution, so shame I can't use that.
Thanks,
Richie

I'm glad it helped.
Please tell us how LibVLCSharp is neater than Vlc.DotNet.
LibVLCSharp was created after, to create a modern and cross-platform binding for the libvlc API.
The idea is to make LibVLCSharp the official binding for .NET, and it will be the first to support libvlc 4.
Please tell us how you think Vlc.DotNet is neater and how we can improve LibVLCSharp.
Also, please don't hesitate to reach us on libvlcsharp's gitter to share your ideas :)

I actually meant VLC.dotnet seems neater and i'd have preferred to keep using that.
Thanks,
Richie

Yes, that's what I said. Please tell us why 🙂

Hi,
For the bare basics of simply playing a file and, if it had worked, casting to a Chromecast, VLC.dotnet seemed to require a lot less code.
Perhaps, for more straightforward media playing, LibVLCSharp is easier and certainly appears to be pretty impressive.
Keep up the good work,
Richie

For the bare basics of simply playing a file and, if it had worked, casting to a Chromecast, VLC.dotnet seemed to require a lot less code.

Can you be more specific please? Any feedback helps to make libvlcsharp better.
Your code from https://github.com/ZeBobo5/Vlc.DotNet/issues/637#issue-609153050 in libvlcsharp could give something like this:
Core.Initialize();
using var libvlc = new LibVLC();
using var mp = new MediaPlayer();
using var media = new Media(libvlc, new Uri(@"Files\Atomic.mp3"));
media.addOption(":sout=#chromecast{ip=<IP_ADDRESS>, demux-filter=demux_chromecast}");
media.addOption(":sout-keep");
media.addOption("--file-logging", "-vvv", "--logfile=Logs.log");
mp.Play(media);

You probably shouldn't mix libvlc options syntax though (pick : for media, not --).


Improved logging

Now with time stamps
Maintains calls to log.info etc.
Maintains log rotation
Logs to the console and file system

(feel free to ignore c08f972, hopefully that will land in #61)

You'd want to remove npmlog from package.json?

oh yes, I shall

LGTM. Thanks :)


small object detection is not good compared to detectron2 maskrcnn

this is yolact ++ result:

while following is maskrcnn's result:

Like they said in the paper this will not be a problem, trade off for speed

If you don't care about the time taken, YOLACT's not really relevant. Use Mask R-CNN (or better yet, HTC) instead.


[MINOR] Reduce UT spark-datasource test times

Change Logs
Removed unnecessary test combinations, reduced combinations of parameterized tests
Reduced task parallelism for Spark from 4 to 2.
Impact
Reduction in azure CI job 3 runtime
Risk level (write none, low medium or high below)
none
Documentation Update
None
Contributor's checklist

[ ] Read through contributor's guide
[ ] Change Logs and Impact were stated clearly
[ ] Adequate tests were added if applicable
[ ] CI passed

https://dev.azure.com/apache-hudi-ci-org/apache-hudi-ci/_build/results?buildId=22096&view=results
Seems to save like 10-15 mins


feat(auto-play): Add autoplay indicator

About the changes
Add autoplay indicator
Fixes https://github.com/bcc-code/bcc-media-app/issues/169

closed in favor of #227


Broken readme link for medplum.com/docs

In the root readme, it links medplum.com/docs to https://github.com/medplum/medplum/blob/main/docs which is 404.
My guess is medplum.com/docs would link to the full path (medplum.com/docs) instead of a relative path (/docs)

Great catch @xta !


fix: path ignore logic

This change is required to reinstate Sonar code quality checks which have stopped being executed as of now across multiple repos. For example, jans-core got impacted when I merged JanssenProject/jans-core#43. There were similar PRs for 3 or 4 other repos. Not sure why all the repos did not get similar PRs.
Previous code which uses paths construct which caused the issue. Instead, paths-ignore should be used without negative match patterns.
New code has been tested on jans-core repo and seems to be working fine.

Previous code which uses paths construct which caused the issue. Instead, paths-ignore should be used without negative match patterns.

I am approving it but we do need to use both constructs. For example we cannot use branches and branches ignore , its one or the other.


Objective C doesn't see label delegate property and protocol definition

There are @objc declarations missing for delegate property and for protocol declaration in the library, and therefore Objective-C doesn't see this property at all and can't be used within Obj-C project. Please include this fix in your next release (add @objc  before the delegate property and before protocol). Would be a good idea to create sample project in Objective-C just to be sure everything written is Swift is accessible from objective-c. It might also be a good idea to make all those delegate methods optional, so developers can decide which one is needed, and not include all 4 just to use a single one in the project.

@LugonjaAleksandar take a look and let me know

@tachtevrenidis Yes, that is exactly what was missing. Thanks for this fast fix. :)

alright let's see if we can get this merged! @tinora


[Suggestion] Upgrade to Font Awesome 6

What is the reason we stay at Font Awesome v5?
Thanks!

Updated to FontAwesome 6.1.1 with eb5850950418b09201a0094100844d9ae5207602.

Awesome. Thanks for fixing this so quickly!


add release of "turtlebot_exploration_3d"

Hi I run "bloom-release" on my repository and got the following:
Everything went as expected, you should check that the new tags match your expectations, and then push to the release repo with:
git push --all && git push --tags  # You might have to add --force to the second command if you are over-writing existing tags
<== Released 'turtlebot_exploration_3d' using release track 'indigo' successfully
==> git remote -v
origin	https://github.com/RobustFieldAutonomyLab/turtlebot_exploration_3d-release.git (fetch)
origin	https://github.com/RobustFieldAutonomyLab/turtlebot_exploration_3d-release.git (push)
Releasing complete, push to release repository?
Continue [Y/n]?
However, failed with git  push --all to the release repository... Was it successfully released?
Please let me know if there is anything wrong with the release, sorry that this is the first time I am been doing this, thanks a lot!!

It looks like you have put your source code into the release repo. If it already has content that has a different git history on the master branch the push will fail because it will conflict.
When you first run bloom the release repo should be empty. I'd recommend deleting and recreating https://github.com/RobustFieldAutonomyLab/turtlebot_exploration_3d-release.git as an empty repo. Then let bloom populate it for you from empty.
You can add the documentation and source entries manually, but you cannot add the release entries.

Thanks a lot @tfoote ! It make a lot of sense to me when you suggest release repo should be an empty one initially.
I did a second time(actually went back and forth as I am new to the process), and I think I made it. (generated by bloom a pull request: #13441).

Glad to see that it looks to have worked!

@tfoote Thanks for the help, I was really getting stuck during that time.
Wish people find it interesting and useful. I dropped an email to<EMAIL_ADDRESS>when I first setup the documentation in ros wiki, however didn't hear back from it(I guess it won't be accepted before a 'release'), would you suggest I do it again now so that it might get approved(to be shown in ROS News on the wiki page)?


Automatic PR creation for yosys-symbiflow-plugins update

Motivate of the pull request

[ ] To address an existing issue. If so, please provide a link to the issue:
[ ] Breaking new feature. If so, please describe details in the description part.

Use dependabot to create PRs automatically when yosys-symbiflow-plugins is updated.

Describe the technical details
What is currently done? (Provide issue link if applicable)

What does this pull request change?

Adds dependabot yaml configuration to update yosys-plugins

Which part of the code base require a change

[ ] VPR
[ ] Tileable routing architecture generator
[ ] OpenFPGA libraries
[ ] FPGA-Verilog
[ ] FPGA-Bitstream
[ ] FPGA-SDC
[ ] FPGA-SPICE
[ ] Flow scripts
[ ] Architecture library
[ ] Cell library
[ ] Documentation
[ ] Regression tests
[ ] Continous Integration (CI) scripts

Impact of the pull request

[ ] Require a change on Quality of Results (QoR)
[ ] Break back-compatibility. If so, please list who may be influenced.

At this point, this is a draft PR, will mark as ready once I do a separate test of dependabot for specific submodules.
Please review the initial configuration and let me know if you see any obvious issues.

I have tested this with a couple of experimental repos and it looks good to go.
The current configuration will ensure that only the specific submodule mentioned in allow will be checked and PR created, in this case only yosys-plugins (from the .gitmodules file)
By default, Dependabot checks for new versions at a random set time for the repository.
Hence set the schedule to 00:00 Mountain Time.
@tangxifan @ganeshgore Please review and let me know if you would like any other changes.

@coolbreeze413 Just a few question

Will the auto merge consider CI green as a requirement? I personally does not like auto-merge because it may bury some problems. A double check is safer.
I am not sure what criteria the dependabot will be triggered. For example, each time there is an update on the master/main branch of a submodule or only when there is a release? If it is a release, a daily schedule is o.k. otherwise we may consider weekly. My concern is that it will create too many PRs automatically (spamming our PR list)
If we choose not to auto merge, can you use dependabot to assign me and @tpagarani as reviewers?

@tangxifan please find my anwers below:

dependabot will not auto-merge.
The merge will be done only if we comment @dependabot merge and then, only if the other criteria is still met i.e in our case,     at least 1 approving review, and all the CI is green.
WIthout that, the PR cannot be merged: https://docs.github.com/en/code-security/supply-chain-security/keeping-your-dependencies-updated-automatically/managing-pull-requests-for-dependency-updates

dependabot will raise PR anytime the version of the submodule's repo is ahead of the version here, however the maximum PRs that will be open at anytime can be set (it is 5 by default).
Older ones are closed automatically.

I will add you and @tpagarani  as reviewers. Do you want a default assignee as well?

@coolbreeze413
This looks good, here are my thoughts

I am ok if you

@coolbreeze413
This looks good. here are my comments.

I am ok if this setup creates PR and auto-merges after CI is green, and PR remains open if CI fails (I don't think If CI is green then there is anything to review from our side)
About the multiple numbers of PR, my understanding is, at any given point there will be either no pending PR (if CI is green) or only one PR (if CI failed)
If yosys-plugins development happens directly on the master branch, then we can stick to once in a day + manually triggered dependable runs

let me what you guys think

@ganeshgore I am totally o.k. that if PR get auto-merged as long as CI is green. We have limited bandwidth on code review. For frequency, once a day may be too aggressive and a weekly update looks reasonable. Anyway, we do not know what will happen in practice. If we converge and merge this PR, then we keep monitoring a few days and see if any critical issue raises.

@tangxifan @ganeshgore : In answer to your comments:

For enabling auto-merge (and auto-review):
We would need to create Github Actions workflow to do this using the PR created, this explains the steps: https://docs.github.com/en/code-security/supply-chain-security/keeping-your-dependencies-updated-automatically/automating-dependabot-with-github-actions#enable-auto-merge-on-a-pull-request
Also, the merge will happen only if the criteria already specified for any PR is met as usual : CI should be green, an approving review (auto-review can be done via Github Actions as well)
This will take a couple of days for me to test/experiment on, and make an appropriate Github Actions workflow.
If you are ok to keep auto-merge as a separate PR, we can enable the bot upto creating the PR automatically for now, merging remains manual.

Number of Pending PRs:
The bot will keep opening new PRs whenever the submodule version is updated.
The bot will stop creating new PRs when the number of already open PRs == open-pull-requests-limit
Once some PRs are closed/merged, then the bot will start creating PRs again.
https://docs.github.com/en/code-security/supply-chain-security/keeping-your-dependencies-updated-automatically/configuration-options-for-dependency-updates#open-pull-requests-limit

Interval:
I think it is good to go with a daily interval to begin with and then adjust it according to the frequency of updates in the yosys-symbiflow-repo.
Also, if auto-merge is enabled, then I think daily interval should be ok.

@coolbreeze413 Make sense to me. We can merge while you are working on auto-merge features.

Waiting for @tpagarani and @lnsharma 's final confirmation.

LGTM


[NEXT]Could not find en locale code in localeMessages

Environment

Operating System: Windows_NT
Node Version:     v18.15.0
Nuxt Version:     3.3.1
Nitro Version:    2.3.1
Package Manager<EMAIL_ADDRESS>Builder:          vite
User Config:      experimental, app, css, postcss, modules, i18n
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Reproduction
https://stackblitz.com/edit/github-6tlwl1?file=README.md
Describe the bug
show "Could not find en locale code in localeMessages" waring only in production environment.it works fine in development environment

Additional context
No response
Logs
No response

youi can run npm run preview to reproduce

I ran a build and then preview and I was not able to replicate it. Did you create a fresh build for your project?

I ran a build and then preview and I was not able to replicate it. Did you create a fresh build for your project?

+1.
On server log:
[intlify] The message format compilation is not supported in this build. Because message compiler isn't included. You need to pre-compilation all message format. So translate function return 'welcome'.
This may be due to the format .ts locale file


Waiting for the code

The code is not available now. Look forward to access the code as soon as possible.
Thanks to the author.

+1

+1. Following for updates.

+1. Following for updates

+1

mark!

Thanks everyone for paying attention to OC-SORT. I have released a preview version. The code is after cleanup and refactoring the code I used for the paper but I have re-validated reproducing the results in paper (see the doc) for details.
This is a primary version as I am going to make mmtracking to support OC-SORT with a better form, which is expected to be faster and more advanced.
Please feel free to propose any issue if you have any questions or problems using the codebase. You can also reach out to me through<EMAIL_ADDRESS>

I will suspect there is still some little issues inside the current version. Also, because of the submission limitation, I haven't validated the results on KITTI/CroHD yet. Please let me know if you encounter any issues.
Moreover, the public tracking on MOT17/MOT20 which requires "detection filtering" is not included in this version either. I will try to make some time for that soon.

I'm closing this issue. Feel free to open a new issue for problems in the released code.


[Windows] "Bad address" exception while creating socket with context from dll

Hello!
I've met a problem I can't deal with:
There is dll with function which creates zmq-context. (void* createCtx())
I load this dll into Python, create context with it and then zmq.Context.shadow(raw_zmq_context) this context.
After that I try to create socket with shadowed context:
sock_ = zmq_ctx_.socket(zmq.REQ)
And there happens "Bad address" exception.
pyzmq version - 19.0.0
libzmq version - 4.3.2

Without code to reproduce it, I'll have to guess, but I would think the most likely option is not casting the address quite correctly to a Python integer when passing it to Python.

Without code to reproduce it, I'll have to guess, but I would think the most likely option is not casting the address quite correctly to a Python integer when passing it to Python.

I'll try to make a little example and add here.
How can I fix possible problem with casting?
As I know, there is no such problem on MacOS.

I can't tell you how to fix it without seeing it, since I don't know what's wrong. But a mistake I had made when testing this myself was improperly casting a pointer to a signed integer, which will often result in an incorrect value when the integer gets cast back to a pointer. If you can, double check in C and Python both, that the address is the same at every stage.

FYI: Here is my code working on Windows.
from ctypes import c_void_p
from ctypes import windll
import zmq

libzmq = windll.LoadLibrary("./libzmq-v140-mt-4_3_2.dll")
libzmq.zmq_ctx_new.restype = c_void_p

addr = libzmq.zmq_ctx_new()
ctx = zmq.Context.shadow(addr)

sock1 = ctx.socket(zmq.PAIR)
sock1.bind("inproc://test")
sock2 = ctx.socket(zmq.PAIR)
sock2.connect("inproc://test")

sock1.send(b"test")
assert sock2.recv() == b"test"

Environment

Windows: 10.0.19041
Python: Python 3.8.3 (tags/v3.8.3:6f8c832, May 13 2020, 22:37:02) [MSC v.1924 64 bit (AMD64)]
pyzmq: 19.0.1 zmq.zmq_version() -> 4.3.2
MSVCCompiler: C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\BIN\\amd64\\cl.exe
DLLs: libzmq-v140-mt-4_3_2.dll and libsodium.dll from https://dl.bintray.com/zeromq/generic/libzmq-v140-x64-4_3_2.zip

I did a little expamle, which represent my problem:
There is dll:
zmqctx.h:
#ifdef ZMQCTX_EXPORTS
#define ZMQCTX_API __declspec(dllexport)
#else
#define ZMQCTX_API __declspec(dllimport)
#endif

extern "C" ZMQCTX_API void* create_ctx();

zmqctx.cpp:
#include "pch.h"
#include "framework.h"
#include "zmqctx.h"
#include <zmq.h>
static void* current_ctx = nullptr;
void* create_ctx()
{
     void* ctx = zmq_ctx_new();
     current_ctx = ctx;
     return current_ctx;
}

And python code:
import ctypes
import zmq
import os

dll = ctypes.windll.LoadLibrary("D:\_Projects\zmqctx\Debug\ZMQCTX.dll")

create_ctx = dll.create_ctx
create_ctx.restype = ctypes.c_void_p

raw_ctx = create_ctx()
ctx = zmq.Context.shadow(raw_ctx)

socket = ctx.socket(zmq.REP)

And traceback:
Traceback (most recent call last):
  File "D:/PyProj/test_zmq/main.py", line 14, in <module>
    socket = ctx.socket(zmq.REP)
  File "C:\Python38\lib\site-packages\zmq\sugar\context.py", line 204, in socket
    s = self._socket_class(self, socket_type, **kwargs)
  File "C:\Python38\lib\site-packages\zmq\sugar\socket.py", line 59, in __init__
    super(Socket, self).__init__(*a, **kw)
  File "zmq\backend\cython\socket.pyx", line 328, in zmq.backend.cython.socket.Socket.__init__
zmq.error.ZMQError: Bad address

So dll creates context and python uses this context to create socket.
Example from sakurai-youhei worked for me too, but it's a little different from my problem.
Thanks for example anyway, I tried few things from it, but they didn't help.

@SotnikAP Yours doesn't reproduce the issue in my environment.

Environment

Windows: 10.0.19041
Python: Python 3.8.3 (tags/v3.8.3:6f8c832, May 13 2020, 22:37:02) [MSC v.1924 64 bit (AMD64)]
pyzmq: 19.0.1 zmq.zmq_version() -> 4.3.2
MSVCCompiler: C:\\Program Files (x86)\\Microsoft Visual Studio 14.0\\VC\\BIN\\amd64\\cl.exe
DLLs: libzmq-v140-mt-4_3_2.dll and libsodium.dll from https://dl.bintray.com/zeromq/generic/libzmq-v140-x64-4_3_2.zip

zmqctx.h
#ifdef ZMQCTX_EXPORTS
#define ZMQCTX_API __declspec(dllexport)
#else
#define ZMQCTX_API __declspec(dllimport)
#endif

extern "C" ZMQCTX_API void* create_ctx();

zmqctx.cpp
#include "zmqctx.h"
#include <zmq.h>
static void* current_ctx = nullptr;
void* create_ctx()
{
     void* ctx = zmq_ctx_new();
     current_ctx = ctx;
     return current_ctx;
}

poc.py
# Code to compile ZMQCTX.dll
from distutils.ccompiler import new_compiler

compiler = new_compiler()
compiler.initialize()

compiler.set_include_dirs(["."])
compiler.set_library_dirs(["."])
compiler.set_libraries(["libzmq-v140-mt-4_3_2"])

objects = compiler.compile(["zmqctx.cpp"])
compiler.link_shared_object(objects, output_filename="ZMQCTX.dll")

# Code to check the issue
import ctypes
import zmq
import os

dll = ctypes.windll.LoadLibrary("./ZMQCTX.dll")

create_ctx = dll.create_ctx
create_ctx.restype = ctypes.c_void_p

raw_ctx = create_ctx()
ctx = zmq.Context.shadow(raw_ctx)

socket = ctx.socket(zmq.REP)

Working directory
C:\Users\sakurai\Desktop\issue-1398>dir *.h *.cpp *.lib

2019/08/27  20:12            28,790 zmq.h
2020/06/28  11:04               165 zmqctx.h

2020/06/28  11:31               184 zmqctx.cpp

2019/08/27  20:15            20,868 libzmq-v140-mt-4_3_2.lib
2019/08/27  20:15         5,163,648 libzmq-v140-mt-s-4_3_2.lib
2020/06/28  11:30             1,700 ZMQCTX.lib


Add the code for GNN-Re-Ranking.

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

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


Dubious, test returned 255

hoo@HZ ~/github/bash_script > panda install   Text::VimColour
==> Fetching Text::VimColour
==> Building Text::VimColour
Compiling lib/Text/VimColour.pm to mbc
==> Testing Text::VimColour
ex: illegal option -- -
Usage: ex [- | -s] [-l] [-L] [-R] [-r [file]] [-t tag]
       [-v] [-V] [-w size] [+cmd | -c cmd] file...
Use of uninitialized value $!out of type Path in string context  in block  at /home/hoo/github/bash_script/.panda-work/1440077032_1/lib/Text/VimColour.pm:24
Use of uninitialized value $!in of type File in string context  in block  at /home/hoo/github/bash_script/.panda-work/1440077032_1/lib/Text/VimColour.pm:24
didn't find a recent vim/ex
  in method BUILD at /home/hoo/github/bash_script/.panda-work/1440077032_1/lib/Text/VimColour.pm:15
  in block <unit> at t/vim_colour.t:18

t/vim_colour.t ..
Dubious, test returned 255 (wstat 65280, 0xff00)
Failed 7/7 subtests

Test Summary Report
-------------------
t/vim_colour.t (Wstat: 65280 Tests: 0 Failed: 0)
  Non-zero exit status: 255
  Parse errors: Bad plan.  You planned 7 tests but ran 0.
Files=1, Tests=0,  1 wallclock secs ( 0.02 usr  0.01 sys +  0.76 cusr  0.05 csys =  0.84 CPU)
Result: FAIL
test stage failed for Text::VimColour: Tests failed
  in method throw at /home/hoo/.rakudobrew/moar-nom/install/share/perl6/runtime/CORE.setting.moarvm:1
  in method install at lib/Panda.pm:133
  in method resolve at lib/Panda.pm:218
  in sub MAIN at /home/hoo/.rakudobrew/bin/../moar-nom/install/share/perl6/site/bin/panda:20
  in block <unit> at /home/hoo/.rakudobrew/bin/../moar-nom/install/share/perl6/site/bin/panda:87

Failure Summary
----------------
Text::VimColour
	*test stage failed for Text::VimColour: Tests failed

hoo@HZ ~/github/bash_script > perl6 --version
This is perl6 version 2015.07.1-134-g3754356 built on MoarVM version 2015.07-53-g673a3c1
hoo@HZ ~/github/bash_script > uname -a
Linux HZ 4.1.5-1-ARCH #1 SMP PREEMPT Tue Aug 11 15:41:14 CEST 2015 x86_64 GNU/Linux

Seeing didn't find a recent vim/ex, you likely need a more recent version of vim. The module needs at least vim 7.4.


Get 400 Bad Request when trying to refresh token

When I call the refresh token end point, I got 400 bad request with this data:
{
     "email": [
         "This field is required."
     ],
      "password": [
          "This field is required."
      ],
}

I use POST with refresh token in body as JSON data.
{
"refresh": "--REFRESH TOKEN----"
}

What is possible of problem here? Please suggest.

Can you post your relevant view and url code files?  My guess is that you're somehow routing refresh requests to the wrong view.

When I removed rest_framework.auth_token from INSTALLED_APPS, it works!!
Thanks, I will close this issue.

I have exactly the same error.
I have rest_framework as an installed app, but not rest_framework.auth_token explicitly ( I assume that maybe is included by default)
If I change the /api/token/refresh route for another one, it works.
I don't have any custom url that starts with /api, so I assume that rest_framework is somehow overriding this route or something.
Is there a way to fix this issue without changing the default url name?
Also, supposing rest_framework is overriding this url, is there any way to ensure or check (for example a compile error) that this error doesn't happen over time? Is pretty annoying that for example a lib update or something could break this silently (assuming that is the cause of it)
I am pretty new to Django and all it's environment, so maybe I am missing something.
Here are my dependencies current versions:
Python=3.7
Django=2.1.5
djangorestframework==3.9.1
djangorestframework_simplejwt==4.4.0
Thanks in advance!


Missing license

Type: Missing
Summary:
Missing license
Details:
Missing license
Resolution:
Package states BSD, but which one is never declared. Per this conversation, it looks like it was supposed to Apache but was created incorrectly in the beginning so I am submitting Apache. https://github.com/request/caseless/issues/28
Affected definitions:

caseless 0.9.0

@capfei Agree with your findings and per the conversation link I will merge your curation for Apache 2.0.


Module cannot be found after installing package 'ts-odatajs' in clean project

Just a little comment to my report:
I have just upgraded my angular 5.2 application to angular 6. In the angular 5 application i did not see this error. So after i have been struggling with this issue for some time and since i could easily reproduce it in a clean new angular 6 solution, then  I decided to submit this report :)
- [ X] bug report
- [ ] feature request

- [ ] new
- [X ] build
- [ ] serve
- [ ] test
- [ ] e2e
- [ ] generate
- [ ] add
- [ ] update
- [ ] lint
- [ ] xi18n
- [ ] run
- [ ] config
- [ ] help
- [ ] version
- [ ] doc

Versions
Windows 10 Pro
Node version: 8.9.3
NPM version: 5.5.1
Angular CLI version
Angular CLI: 6.1.2
Node: 8.9.3
OS: win32 x64
Angular:
@angular-devkit/architect    0.7.2
@angular-devkit/core         0.7.2
@angular-devkit/schematics   0.7.2
@schematics/angular          0.7.2
@schematics/update           0.7.2
rxjs                         6.2.2
typescript                   2.7.2
Repro steps

ng new angularTest
cd angularTest
npm i ts-odatajs
go to for example app.component.ts and make it contain the following:

import { Component } from '@angular/core';
import {oData} from 'ts-odatajs';
@Component({
  selector: 'app-root',
  templateUrl: './app.component.html',
  styleUrls: ['./app.component.css']
})
export class AppComponent {
  title = 'angularTest';
  oData;

  constructor(){
    oData.json
  }
}

The log given by the failure
ERROR in ./node_modules/ts-odatajs/lib/odata/net.js
Module not found: Error: Can't resolve 'http' in 'C:\Users\USER\Desktop\testDir\angularTest\node_modules\ts-odatajs\lib\odata'
ERROR in ./node_modules/ts-odatajs/lib/odata/net.js
Module not found: Error: Can't resolve 'https' in 'C:\Users\USER\Desktop\testDir\angularTest\node_modules\ts-odatajs\lib\odata'
Desired functionality
Result: A successfull build

Similar or duplicate issue here maybe? https://github.com/angular/angular-cli/issues/10694
We struggle with the same thing.

Hi @steffeli ,
I read your ticket and I see your problem is very similar. I guess it confirms even more that this is a bug since you also upgraded from angular 5.2 to 6 like me. I really hope for a solution soon because we need this to work now. :)

The ts-odatajs main file relies on two node builtin libraries http and https.  These are specific to the node.js javascript platform and are not present within a browser javascript platform.   The package in question, however, does contain a browser version (dist/odatajs-4.0.3.min.js).  Since the package.json does not contain a browser field pointing to the file, a TypeScript path mapping could potentially be used to map the non-relative import to the appropriate file.

Hi @clydin ,
Thanks for explanation and suggestion.
For others in similar situation, this might help. I added the following to the tsconfig.json file:
    "paths": {
      "https": [ "./node_modules/ts-odatajs/dist/odatajs-4.0.3.min.js" ],
      "http": [ "./node_modules/ts-odatajs/dist/odatajs-4.0.3.min.js" ],
    }

When I build after this the error is gone. I am still not sure why this is was not needed in angular 5.2.
@steffeli maybe this knowledge can help in your situation.
Thanks :)

Thank you for your suggestion @Diemauerdk, but we still can't get it to work. I have tried to add path-browserify to our project and add paths aliases as you suggested, the error/warn is still there about missing path package. I thought webpack (bundled with Angular CLI) included browser specific implementations of these by default so that we shouldn't have to do this at all, or might I be missing something?

I am absolutely no expert in configuring webpack but that's also what I expected - and I don't understand what the difference here is between angular 5 and 6.
Sad to hear it didn't solve your problem though, it's quite frustrating :(

I am absolutely no expert in configuring webpack but that's also what I expected - and I don't understand what the difference here is between angular 5 and 6.
Sad to hear it didn't solve your problem though, it's quite frustrating :(

I couldn't make this work. Had to change the following line in "odata.js":
var odataNet      = exports.net       = require('./odata/net.js');

to
var odataNet      = exports.net       = require('./odata/net-browser.js');

Obviously this is just a hack I have to add every time I fetch ts-odatajs.
Is there some tsconfig path magic which can be applied so that the require('./odata/net.js') will resolve to require('./odata/net-browser.js')?
Or is this a webpack thingy which I don't want to touch?

@Casimodo72 you can use webpack module replacement plugin for that:
new webpack.NormalModuleReplacementPlugin(
    /node_modules[\\/]ts-odatajs[\\/]lib[\\/]odata[\\/]net\.js/,
    './net-browser.js'
)

PS. It's strainge but it works without this replacement with vue.js


[Launcher]: Upgrade to Zookeeper 3.4.14

Coverage remained the same at 53.217% when pulling fa265fb71afdcecf7261c938f821bd1970d53d74 on yqwang/zk-upgrade into 6d76e406edba835491fab0f9de7cc116a0ac5385 on master.


fix - IAM User takeover Correlation Rule correlating on IP instead of user

Background
Fixes https://github.com/panther-labs/panther-analysis/issues/1359
Changes

Made IAM User takeover Correlation Rule correlate on user instead of IP

I think this need to correlate on both IP and target user name.

@arielkr256 We cannot correlate on both. We can add some context field to both rules that will contain both IP and user name and try correlating on this list, but I'm not sure that this is a good idea (it is not an explicit approach) and that it will work. What do you think about that?


chore: improve All tab for search results

@charleslavon what do you think about this PR? Maybe we should leave a full screen without changes and improve only TypeAheadDropdown component?


guard livereload using ipv4 only . how to force  to use ipv6.

on server allowed to use only ipv6 for connection inbound and outbound.
netstat --listen output:

Active Internet connections (only servers)
Proto Recv-Q Send-Q Local Address           Foreign Address         State
tcp        0      0 *:ssh                   *:*                     LISTEN
tcp        0      0 *:35729                 *:*                     LISTEN
tcp6       0      0 [::]:ssh                [::]:*                  LISTEN
tcp6       0      0 [::]:3000               [::]:*                  LISTEN
Active UNIX domain sockets (only servers)
Proto RefCnt Flags       Type       State         I-Node   Path
unix  2      [ ACC ]     STREAM     LISTENING<PHONE_NUMBER> @/com/ubuntu/upstart
unix  2      [ ACC ]     SEQPACKET  LISTENING<PHONE_NUMBER> /run/udev/control

here is the guardfile:
guard 'livereload', host:'<IP_ADDRESS>' do
 watch %r{^.*\.(html|css|js|png|jpg|jade)$}
end

here i do need to mention host specifically.
from netstat its visible guard binds livereload to ipv4 and not ipv6.
how to bind it to ipv6

Provided you have these two lines in your /etc/hosts
<IP_ADDRESS>       localhost
::1                 localhost

Binding to localhost in the Guardfile like this bound to ipv6 in my case (macOS):
guard 'livereload', host: 'localhost' do
  ...

Then I get is lsof:
lsof -i:35729
COMMAND           PID  USER   FD   TYPE             DEVICE SIZE/OFF NODE NAME
com.apple.WebKi   588 ybart   93u  IPv6 0x4571d672b83d719f      0t0  TCP localhost:59214->localhost:35729 (ESTABLISHED)
com.apple.WebKi   588 ybart   98u  IPv6 0x4571d672b83d719f      0t0  TCP localhost:59214->localhost:35729 (ESTABLISHED)
ruby            49803 ybart   15u  IPv6 0x4571d672bbb7ad1f      0t0  TCP localhost:35729 (LISTEN)
ruby            49803 ybart   19u  IPv6 0x4571d672b83d941f      0t0  TCP localhost:35729->localhost:59214 (ESTABLISHED)


Tyche Design Updates

Cannot figure out why the one redeem test is suddenly not working. The data-testid is updated and matches. Otherwise this addressed Tutti's concerns (there are still concerns about button sizes but that seems to be a more button-component-wide issue than just Tyche)

@0xLienid is this the PR where you'll put the yield copy changes?

I actually have to combine this with another PR from Jem but yes I can also put yield copy changes in here. Not ready yet. I’ll move it back to draft when I’m back home.
On Sat, Jun 25, 2022 at 4:30 PM, appleseed-iii @.***> wrote:

@.***(https://github.com/0xLienid) is this the PR where you'll put the yield copy changes?
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you were mentioned.Message ID: @.***>

Has this been merged into #1723? I would close this, then


Blacklist some of the fields.

Hello,
Is there any way we can hide some fields  for specific roles in SG 2.2? .
Here is the scenario, we have several fields, out of these there are few fields we don't want to display it to every one. is there any way to  set rule on those specific fields in search Guard 2.2?

Will be possible with Field level security: https://github.com/floragunncom/search-guard/wiki
(Not yet released but will come this month)


Handle signed numeric binaries

Binary conversions to decimal can now handle signed numbers as well (+ too).

Coverage increased (+0.05%) to 90.395% when pulling ba8dde862df788070267b76702507591a279cc8c on kitonjicsm:Handle-signed-binaries into 2ef054c39cf5fd512bec0595c92e6a67b1707bec on egobrain:master.

Good. Thanks! :)


Ukrainian language

What files do I need to change to add the Ukrainian language or could you do it?

@illya1004 here's a trivial solution to your problem, works for me using gpt-4
0001-small-diff.patch
just apply the patch, keep it stashed, pop it each time you pull


fix(ui): incorrect fee in pay summary

Description:
Fix #1900

Types of changes:
Bugfix

Checklist:

[x] My code follows the code style of this project.
[x] I have reviewed and updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes where needed.
[x] All new and existing tests passed.
[x] My commits have been squashed into a concise set of changes.

Coverage decreased (-0.01%) to 20.969% when pulling 1b1e5e8b51183f284adce051806627578b5bcf0d on korhaliv:fix/fee-summary into 027223d5c659cadaaab1834e3bf664d691ba6729 on LN-Zap:master.


Implement align-self and add some example

Implement align-self

Add more example

Thanks a lot! looks great


Issue with providing both id_token and access_token

I am trying to implement a new OP with implicit flow and I am confused by this error in the logs:
[auth_openidc:error] [pid 16717] [client <IP_ADDRESS>:44078] oidc_proto_validate_response_type: the requested response type was (id_token) but the response contains an "access_token" parameter, referer: https://localhost/redirect"
(The page displays The OpenID Connect Provider returned an error: Error in handling response type.)
The corresponding OP redirect URL does contain both id_token and access_token:
https://localhost/redirect#
 state=AO8ooHIGq0JQZOFGTGraHWUgAAs
 &access_token=1_-9W.....
 &id_token=eyJhbGciOiJIUzI1...
 &token_type=bearer&expires_in=3600&

This blog seems to suggest it is normal to have the 2 tokens, but google seems to give only one...
So, is it my provider who does not respect the spec, or is it mod_auth_openidc who actually wrongly matches "id_token" with "token" in the code ?
If it matters, the config currently looks like this:
OIDCProviderIssuer https://OP.net/LoginService/
OIDCProviderAuthorizationEndpoint https://OP.net/LoginService/authorize?service=MyService
OIDCClientID MyService
OIDCClientSecret mysecret
OIDCResponseType id_token
OIDCCryptoPassphrase password
OIDCRedirectURI https://localhost/redirect

The article you refer to is a generic article about OpenID Connect. The response_type id_token indicates a specific flow called "Implicit" that only returns an id_token in the front channel as specified here: http://openid.net/specs/openid-connect-core-1_0.html#ImplicitAuthRequest You'll note that it says at response_type:

No Access Token is returned when the value is id_token.

So the server should not return one.

Thanks for the quick reply. I guess OP is doing "token" then.  I will try
that on Monday.


ARXML: allow to specify non-base 10 integer numbers

in essence, this means replacing int(my_string_value) by int(my_string_value, 0). For some reason, python does not recognize numbers that start with a 0 and continue with a digit as octal, but AUTOSAR does, so this needs to be handled as a special case.
note that with this patch, python's "0o" prefix for octal numbers is also recognized. As far as I know, AUTOSAR does not allow that, but as long as we properly recognize everything which AUTOSAR specifies, I don't see a big problem with that...
Andreas Lauser<EMAIL_ADDRESS>Mercedes-Benz AG on behalf of MBition GmbH.
Imprint

Coverage decreased (-0.01%) to 96.552% when pulling ca16382a68a62a6dd767b523aab714b311566495 on Daimler:arxml_non_decimal_encodings into 7786a1d87dfdcb108aab40a8d25e46f722108905 on eerimoq:master.


Use commands config to execute wrapper commands

Added a config file that contains the supported commands, their description and whether or not they are simple "wrapped" commands. With this data we can simply detect if a command like bundle is a wrapped command and execute bin/wrap_command rather than having a duplicative script for each.
This would be a good start for addressing #39.

👀 @blankenshipz

LGTM!!!!


Not handling an exception of VM creation gracefully

Not critical, just code cleanliness.
I have managed to somehow ruin my test course (ironically, named 'The Best Course') and triggered an exception we maybe did not get in a while. In handling that exception we are referencing a missing attribute:
Unexpected error I have noticed:
File "/home/vital/vital2.0/source/virtual_lab/vital_site/vital/views/student_view.py", line 198, in start_vm
XenClient().remove_network_bridges(vm.xen_server, request.user, course_id, vm_id)
AttributeError: 'Virtual_Machine' object has no attribute 'xen_server'
Code triggering error:
except Exception as e:
        logger.error(str(e))
        audit(request, 'Error starting Virtual machine ' + str(vm.name) + '( ' + e.message + ' )')
        if 'Connection refused' not in str(e).rstrip() or started_vm is not None:
            XenClient().remove_network_bridges(**vm.xen_server**, request.user, course_id, vm_id)
            XenClient().stop_vm(started_vm['xen_server'], request.user, course_id, vm_id)
        return redirect('/courses/' + course_id + '/vms?message=Unable to start VM - ' + vm.name)

I just tried launching a standalone VM and connecting to it via VNC, and it is working(kali-test), so looks like something wrong with the course side logic.

Yep, and this is the point. If the course is working properly, this exception is never triggered and it is brushed under the carpet. But if the course config is ruined, it raises an exception, handling which includes a bad line I am talking about


PythonAPI world.get_lightmanager wrong name in documentation

Hi,
The documentation for LightManager in PythonAPI has the function name as get_light_manager link.
But the function name is
.def("get_lightmanager", CONST_CALL_WITHOUT_GIL(cc::World, GetLightManager)) in world.cpp
Will the documentation be updated?

@sergi-e could you please take care of this?

Hi, @gsujansai !
This issue was properly fixed. Thanks a lot for reporting it :)

This was only fixed for the python api doc for World there are still places in the documentation that refer to the incorrect name: https://github.com/carla-simulator/carla/search?q=get_light_manager

Hi, @analog-cbarber !
Completely true. These instances have been fixed in this branch: https://github.com/carla-simulator/carla/tree/sergi-e/warning-docs-version and will be merged as soon as possible, along with other minor changes.


[en] fix code block rendering in title

fix back quote format in title, see more about in below picture:

/assign @tanjunchen

I don’t have merge permissions, so  /cc @tengqm .
thanks.

Nice
/lgtm
/approve


Added codecov badge

Codecov Report

Merging #365 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             master    #365   +/-   ##
========================================
  Coverage      50.2%   50.2%
  Complexity      282     282
========================================
  Files            97      97
  Lines          1669    1669
  Branches        103     103
========================================
  Hits            838     838
  Misses          793     793
  Partials         38      38

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 592dfc4...249639d. Read the comment docs.


feat: public namespace basic function

What's the purpose of this PR
Created a basic administration page for public namespace

I have read the CLA Document and I hereby sign the CLA

I have made some changes based on suggestions.

Thanks, I think this is a small milestone achieved!
BTW, I kind of like your original design about the home page by displaying these items in one table, will you keep working on this?

I have changed the grid display to a table display and also changed the left bar to a menu, as shown below:

Codecov Report

Merging #3850 (863c716) into master (aef7d83) will decrease coverage by 0.34%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #3850      +/-   ##
============================================
- Coverage     50.49%   50.14%   -0.35%
- Complexity     2395     2464      +69
============================================
  Files           458      483      +25
  Lines         14472    14919     +447
  Branches       1467     1519      +52
============================================
+ Hits           7307     7481     +174
- Misses         6659     6910     +251
- Partials        506      528      +22

Impacted Files
Coverage Δ

...ollo/openapi/client/service/AppOpenApiService.java
48.00% <0.00%> (-12.00%)
:arrow_down:

...internals/provider/DefaultApplicationProvider.java
52.00% <0.00%> (-9.12%)
:arrow_down:

...va/com/ctrip/framework/apollo/util/ConfigUtil.java
81.56% <0.00%> (-5.94%)
:arrow_down:

...rk/apollo/spring/property/SpringValueRegistry.java
83.33% <0.00%> (-5.56%)
:arrow_down:

...amework/apollo/internals/ConfigServiceLocator.java
87.27% <0.00%> (-2.32%)
:arrow_down:

...nfigservice/service/AccessKeyServiceWithCache.java
82.65% <0.00%> (-2.05%)
:arrow_down:

.../framework/apollo/spring/property/SpringValue.java
87.71% <0.00%> (-1.76%)
:arrow_down:

...ork/apollo/openapi/client/ApolloOpenApiClient.java
47.45% <0.00%> (-0.82%)
:arrow_down:

...lo/configservice/util/InstanceConfigAuditUtil.java
75.43% <0.00%> (-0.63%)
:arrow_down:

...ork/apollo/portal/service/AppNamespaceService.java
75.00% <0.00%> (-0.56%)
:arrow_down:

... and 43 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 aef7d83...863c716. Read the comment docs.

BTW, I really like the style of your original design, is it possible to adjust the colors, layouts, search boxes to look like below?

@youabcd Looks great now, all the pending issues are resolved! Please update the CHANGES.md and I think we are good to go.


feat: support multiple css file extensions in development

If you had index.scss in resources/scripts and tried to do @vite('index.scss') it won't work on dev mode, but work on production mode

Sorry, my fix was missing a parenthesis. Can you fix that? Once all tests pass I can merge this.


[nix] Patch config.nix.in to reference host platform binaries

Motivation for this change
Presently the stable version of nix (2.3.3), when cross-compiled from nixpkgs, will pull into its closure and attempt to invoke the build platform's version of bash. This works around some of the build system for nix to correct that, so that a cross-compiled nix binary can be used on the system it is compiled for.
See #81153
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

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

@GrahamcOfBorg eval

@flokli I split into two commits and made sure no more tabs got in. Can you re-review?

I built this, and nix-store -qR $(nix-build -A pkgsCross.armv7l-hf-multiplatform.nix) seems to have references to native coreutils, glibc and whatnot, however, it seems to be an improvement.
Did you copy over the closure and see if nix behaves properly?

I have not yet. Odd that the closure still pulls in native stuff. I'll try it tomorrow when I can get my pi back out


Set typescript as peerDependency

This way we don't depend on gulp-typescript to be updated each time a new TypeScript version is released, we can choose the exact version we want and we don't get it installed twice (which causes awkward moments like "why does it work in console but not in gulp-typescript?".)
Cheers. :smiley:

I agree that it's better to have typescript as a peerDependency, though that would be a breaking change. I plan to change this in the next major release, at the same time when TypeScript 2.0 is released. Closing for now, I will revisit this later.


RFE: First class VMware vSphere checks

Request For Enhancement: Provide a (public or enterprise) extension or plugin with first class monitoring of vSphere.
Expected Behavior
sudo sensu-install -p vsphere
Current Behavior
Maybe use Nagios plugins, pulling in associated dependencies, and JIT/Proxy checks:

https://github.com/jjasghar/esxi_health (Ruby)
https://github.com/rogerlz/nagios-check-vcenter (Python)
https://www.claudiokuenzler.com/nagios-plugins/check_esxi_hardware.php (Python)
https://github.com/lgeeklee/pyvinga (Python)

These are less desirable due to having to use JIT/Proxy and the need to deal with Python environment & deps in addition to Ruby.
Possible Solution

VMware provides a capable Ruby interface to the vSphere API (https://github.com/vmware/rbvmomi)
Implement a CLI check plugin that can be used as a JIT/Proxy check using rbvmomi
Implement an extension using rbvmomi that supports subscription based checks (but not actually requiring sensu-client on a ESXi/vSphere/VCSA node)

Context
Monitoring of hybrid infrastructure necessitates monitoring of the virtualization infrastructure to include:

hypervisor health (cpu, memory, uptime)
hypervisor networks
data stores
etc

Your Environment

Sensu 0.26
vSphere & vCenter 6.5 with VCSA

@jflemer-ndp thanks for the issue, a valid. This is however the wrong place for it, I have gone ahead and created https://github.com/sensu-plugins/sensu-plugins-feature-requests/issues/13


Alternative Names

We appear to be using tags as alternative names:
https://github.com/bee-san/pyWhat/pull/183#pullrequestreview-774841793
Might be a good idea to change the way the name key works so it contains multiple possible names? Will need a full exploration to look at how this might work :)

I think that many names are complex to be used as tags, so we need to add individual tags to every database entry.


Upgrade at_client_mobile dependencies

- What I did
Upgrade the versions of following packages:

at_client
at_lookup
at_commons
at_utils.

- How I did it
- How to verify it
- Description for the changelog

@sarika01 please can you also review

LGTM


added django rest snippets

Added Django Rest Snippets and Fixed some Django Snippets.

any updates ??

I will merge this :)). Users please file an issue if you encounter any problem :D


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-27-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 4106

Questions ?
Please refer to the Backport tool documentation


Vulnerability roundup 58: mupdf-1.14.0: 2 advisories

search, files

[x] CVE-2018-19881 (nixos-unstable)
[x] CVE-2018-19882 (nixos-unstable)

Scanned versions: nixos-unstable: eebd1a92637. May contain false positives.

CVE's covered in https://github.com/NixOS/nixpkgs/issues/70113 and fixed in unstable and 19.09.


doc: socket

Adds in website documentation for Socket

Codecov Report

Merging #1036 (4eaccb6) into main (660d80e) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1036   +/-   ##
=======================================
  Coverage   57.01%   57.01%
=======================================
  Files          73       73
  Lines        2261     2261
  Branches       62       62
=======================================
  Hits         1289     1289
  Misses        972      972

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 660d80e...4eaccb6. Read the comment docs.


Port ninja build tool

This build tool is used by the Chromium source tree.
Needed for #1.

While doing this, I found a couple of LIBC problems, see https://github.com/bitwiseworks/libc/issues/44 and https://github.com/bitwiseworks/libc/issues/46.
Also, our LIBC lacks pselect. For now I just replaced it with select but there is a chance that we will miss some child signals and need to emulate pselect by calling setmask before and after select in an atomic manner, see e.g. https://linux.die.net/man/2/pselect.

Ninja now compiles (bootstraps) and then can even rebuild itself sccessfully using its own build.ninja recipe. Very good.
However, I spotted a bunch of tests in the package, built them and here is the result:
[223/249] SubprocessTest.BadCommandStderr
*** Failure in D:/Coding/qt5/qt5/qtwebengine/src/3rdparty/ninja/src/subprocess_test.cc:52
"" != subproc->GetOutput()
[226/249] SubprocessTest.InterruptParent
*** Failure in D:/Coding/qt5/qt5/qtwebengine/src/3rdparty/ninja/src/subprocess_test.cc:96
"We should have been interrupted"
[228/249] SubprocessTest.InterruptParentWithSigTerm
*** Failure in D:/Coding/qt5/qt5/qtwebengine/src/3rdparty/ninja/src/subprocess_test.cc:120
"We should have been interrupted"
[230/249] SubprocessTest.InterruptParentWithSigHup
*** Failure in D:/Coding/qt5/qt5/qtwebengine/src/3rdparty/ninja/src/subprocess_test.cc:144
"We should have been interrupted"
[249/249] ElideMiddle.ElideInTheMiddle
failed

As I predicted above, we have a couple of failures de to our "naive" pselect workaround. I will look closer and those failures to see if they can be postponed for later. Other than that, the test result is very good too.

Ok, I've played around with failing tests quite a bit and here are the results.
This first failure is because the BadCommandStderr test tries to run cmd.exe and capture its otuput. However, this doesn't work because cmd.exe is not a kLIBC-based process. In ninja we have to use socketpair instead of pipe for grabbing child process output because kLIBC select only supports sockets and pipe creates native pipes. kLIBC-based child processes recognize that a file descriptor is a socket and use recv/send respectively to communicate through it. However, normal OS/2 executables know nothing about sockets (passed around as regular file handles) and therefore can't write to them. There are two solutions to this problem:

Provide a new subprocess implementation from scratch that will use native pipes and respective DosWaitNPipe calls. (Now we use subprocess-posix.cc with just a few minor changes one of which is changing pipe to socketpair and the other one is pselect -> select).
Provide support for native pipes in kLIBC select. This is a long waiting task (https://github.com/bitwiseworks/libcx/issues/34), it's even more complex (with an advantage that many apps will benefit from that, not just one).

Both tasks require at least several days to complete. Given that it's unlikely that we will face any issues w/o being able to grab non-kLIBC process output while building Chromium, I see no point in spending time on that now.
The rest of failures relates is due to this dash ticket https://github.com/bitwiseworks/dash-os2/issues/1 (ninja always uses the shell (sh -c) to start processes which is dash in or RPM environment). Once this issue is fixed, these three failing tests should start working.
Other than that, we're good.


How to cross compile to different architectures?

I have setup the plugin here to build a java lib using flapigen and it works perfectly when I'm building for the native environment. One of the subprojects setups a rust lib and uses cargo-wrapper to build the crate, while another subproject that depends on the rust one builds the java lib adding the rust generated library to it's jar output.
I am now trying to setup my project's CI to also cross compile to other architectures, e.g., build to 32-bit Linux from a 64-bit container, but I don't really know how to properly set this up on the gralde config. I've tested the attributes property and it does work for passing --target <triple>to the Cargo command, but having a fixed value there is not ideal for setting up a CI that generate builds for a few different architectures.
I've tried setting something up on the outputs variable, but I'm not sure that this is the right place for my use case. I would expect there to be a way to specify on the command line the desired target for the build, or maybe define a new task that would build the project with its own version of the attributes field set, but I haven't found a way if doing either.
I'm not that experienced with gradle, so I might be missing something obvious.

Gradle's buildscripts are actually scripts written in the Groovy programming language (not sure if you already knew that), so you can use all of the features of Java and Groovy within a buildscript.
If you wanted to set up something where you could specify a specific Rust target for a build, you could write
cargo {
    def targetTriple = System.getProperty("CARGO_WRAPPER_RUST_TARGET")
    if(targetTriple == null) {
        throw new GradleException("Target triple not specified!")
        // or you could also set the targetTriple to some default
        // value rather than aborting here
    }

    def fileExtension
    switch(targetTriple.split('-')[2]) {
        case 'windows':
            fileExtension = 'dll'
            break
        case 'darwin':
            fileExtension = 'dylib'
            break
        default:
            fileExtension = 'so'
            break
    }

    crate = projectDir.path

    // (assuming that you typo'd "attributes" and meant "arguments" here)
    arguments = ['--target', targetTriple]

    // build results are expected to be in target/<triple>/<profile>/
    // when passing `--target`, so this map must specify targetTriple
    // as a key to allow cargo-wrapper to find the output of `cargo build`
    outputs = [(targetTriple): 'wrapper_example.' + fileExtension]
}

which would let you run
./gradlew -DCARGO_WRAPPER_RUST_TARGET=<target triple> build

on the command line in your CI to build with a particular target triple.

On the other hand, if you wanted to set up building multiple architectures of the same library into a single jar, then you would need to register multiple CargoTasks manually, which isn't currently possible since the CargoTask constructor is protected. If this is what you were looking for, feel free to ask me to change CargoTask's visibility and update this plugin.

This script you've provided fits very well my needs atm, that's better than having predefined tasks. Thanks for the help!

If this is what you were looking for, feel free to ask me to change CargoTask's visibility and update this plugin.

Unfortunately, I need to do this.

@bluebear94 I published version 1.1.0 of the plugin, which makes CargoTask#configure public.

@Arc-blroth I feel very daft, but I can't quite get it right...how exactly would I configure multiple build tasks so I can cross-compile to multiple targets?


Toggle not working on Safari 9.0

Hotel v0.4.18
Attempting to switch the toggle in Safari 9.0 results in the following error.

If you'd like any more information, let me know.

I'll check that :) thank you.
Seems to be an issue with fetch (I'm using https://github.com/github/fetch).

Should be good now (v0.4.20)


Remove unused SUID chrome-sandbox binary

I noticed that there is a setuid binary called chrome-sandbox that appears to be pulled into the Linux packaging process from Electron.
This binary presents a very large attack surface area for a setuid binary, and also appears to be totally unused.
I've been running for two months with this binary removed from my system and can't tell any difference in functionality.

@tjps Without it, Electron crashes on Linux systems with user namespaces disabled in the kernel config.

@tjps, see https://github.com/electron/electron/issues/17972. It'd be great to get rid of this binary, but we can't right now on (for example) Debian Buster. You could make an issue on the Electron repo. For what it's worth, this is the same sandbox binary that Chrome uses.

I see, thanks for the clarification.
Would you accept a patch that detects if user namespaces are enabled, and then removes the binary, otherwise falling back to the current behavior?

User namespaces are a configuration option for the kernel, which many distros add a boot-time sysctl option to enable/disable. You cannot detect at packaging time whether or not it will be enabled at runtime, because:

distros could change their mind about how to default the kernel config, at which point you will, what, rebuild keybase after kernel updates???
users could switch kernels at the grub boot prompt, to e.g. the linux-hardened kernel
electron apps can be packaged on one distro and installed on another

...
What is your goal here? This is how every electron app ever is packaged, do you just not like how electron fundamentally works? What about chromium?
This is not something to "fix" in every random app that uses electron.
Moreover, the chrome-sandbox is generally believed to be pretty darn secure. Google invested a lot of effort into it. In contrast, keybase already ships their own setuid binary, which you might want to focus on disabling first: https://keybase.io/docs/secadv/kb002

@eli-schwartz I find the appeal to "trust a big company" particularly amusing today.
Closing this.

Nice buzzwords, but...
What's your threat model? Are you concerned about the possibility of google and independent chromium contributors engaging in an elaborate development process to create a open-source sandbox with a backdoor?
When did I say "they're a big company so we should trust them"? I have suggested that the publicly auditable code they wrote has had a lot of effort invested into it, but nowhere does that imply accepting it on blind faith.
Note that user namespaces have a long history of problems, and those problems exist in the kernel, not one program which may or may not be installed, hence why they are often disabled.
Note also that google's chrome-sandbox is only one part of google's chrome, and if you're running an electron app you're already trusting them with full access to make chrome/electron do malicious things to your user account (and the bank information and other secrets you have there). So if you suspect you're running flat-out malicious programs authored with ill intent, my advice is to stop using them entirely.
If you're concerned about attack surfaces, but not malice, then chrome-sandbox is not as big of an attack surface as user namespaces...


✨[feature Request]: add images in the Cards

Is there an existing issue for this?

[X] I have searched the existing issues

Feature Description
i can add the images in the cards section and the  sizes of the cards are not similar i can work on the allot this task to me.
Use Case
adding some good effects and images to the cards
Benefits
No response
Add ScreenShots

Priority
Medium
Record

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24EXTD contributor
[X] I have starred the repository

@Anjaliavv51 assign me this issue . I'd like to work on this.

i like to add images in the images section which gives nice look to the webpage. assign me i like to work on it

Issue is already been assigned.


What code is sent where?

Very cool tool!
As far as I can tell are only the git diff sent to ChatGPT? Am I correct or is there more?

you are correct, only openAI api, chatgpt model


fix sentence iteration in opennlp package

Fix sentence boundary detection bug in case of repeating tokens (i.e. while using OpenNLP analysis chain in conjunction with a KeywordRepeatFilter) by keeping track of the sentence index and looking ahead one token. Move inner sentence iteration to a component to be shared by the sentence-aware OpenNLP filters.
Replacement for:
#11734
Fixes:
#11735
#11771

Duplicated in #11734


Caching Frequently used Parquet sections

Description
On the topic of caching in glaredb has it already implemented caching of frequently queried parquet blocks  in memory or in fast disk near the compute.  In the java world external blob storage caching systems exist like https://github.com/Alluxio/alluxio  which then can provide in memory access directly to the Spark or Trino processes.
Starburst also has good caching build into their cloud version of Trino. It is almost fast enough to use it as a back-end for REST api's pulling data from parquet but still the Java overhead is hard to swallow when you can accomplish this so simply with pyarrow.
Context:  We are looking for a solution to enable efficient small queries from large numbers of concurrent read only users without the need of copying the data once again to postgres/clickhouse

related to https://github.com/GlareDB/glaredb/issues/1791

I've been talking with the team about using caching for this kind of thing, but we haven't had a user doing this kind of workload. I think there are (or could be) a lot of options here, depending on what you're trying to do, at varying levels of complexity/work. Would love to hear more about what you're building! Feel free to drop some time on my calendar if you like.


Deploy prometheus with k8s, execute cmd get no data

I deploy prometheus  in k8s, and execute cmd that the result is no data, and I execute restful api in k8s master node like below:
curl -X GET "http://<IP_ADDRESS>:30900/api/v1/query?query=container_spec_cpu_shares&time=1484218546.737&_=1484218527110"
{"status":"success","data":{"resultType":"vector","result":[]}}
I think my prometheus  config is right, because I deploy in other k8s cluster, so Is there limit to prometheus  , like k8s version or prometheus   version?

Check the target page of Prometheus – can it scrape its targets? Did it find any?

@matthiasr  Thanks for your help, I have resolved my problem because the time in cluster of k8s is set in the future, so prometheus can not get any data.
By the way, Does the prometheus support to monitor the data that I increase service node dynamic. For example now prometheus monitor 5 vm, and I want to add vm to be monitored, Whether I must rewrite the config file of prometheus and restart prometheus server?


use graph convolutional neural network for hyperspectral image classification

Hello, I find that you are also engaged in hyperspectral remote sensing image classification. Have you ever tried to use graph convolutional neural network for hyperspectral image classification? Can you share it on GitHub

Hello, I find that you are also engaged in hyperspectral remote sensing image classification. Have you ever tried to use graph convolutional neural network for hyperspectral image classification? Can you share it on GitHub


Accessing dataframe name in custom sampler by sample-context

Hi,
I'm using soda-spark-df, I would like to access dataframe name (created as temporary view in spark), but checking out the sampler object I didn't find any way to do that. I can get rows and schema, but no name. Is there a way to obtain the name of the df?
Thx in advance.

SAS-3085

Hi! We indeed do not store the dataset name in the sample. Would you be open for a contribution? This should be fairly simple, just push the dataset name into the DbSample object here

Hi @Parseval123 , would you be open to contributing a PR for this?


Blueprint Framework - Update Resource Schema to align with spec

Since the initial implementation of the resource schema and behaviour, the blueprint specification has been updated.
We need to make sure the schema for resources matches the current spec.

Implemented in f56511b


WebDev VM: Install VSCode / extensions

Working on a clean machine, need a decent editor.  VS Code is the cool-tool of the day

VSCode install

Downloaded latest VSCode from https://code.visualstudio.com/download

Windows/zip/x64 version
unzipped to W:\VSCode
added W:\VSCode to user path

YAML extension

Installed YAML extension by RedHat (has 12M installs)

Reason: autocompletion tasks.  Such as: ctrl-space on "timezone: " and select "America/Chicago" in _config.yml


[SSH] - az webapp ssh is not working on windows OS

az feedback auto-generates most of the information requested below, as of CLI version 2.0.62

Describe the bug
az webapp ssh command is throwing an error:webapp ssh is only supported on linux and mac  when runs from windows machine.
To Reproduce

run below command from windows machine
az webapp ssh -n MyUniqueAppName -g MyResourceGroup

Expected behavior
It should work as on other OS like Mac or Linux
Environment summary

Additional context

webapp

@btardif FYI.

IMHO, the error message is unclear, see https://github.com/Azure/azure-cli/issues/12940#issuecomment-737152145

This is being tracked here: #16125


Update Options docs

#226 Documentation update for Options

@kelunik I used `note` blocks to show default values and `warning` blocks for validation rules. Does it make easier to read the docs this way, wdyt?

Thanks! I'll keep the notes and warnings for now, but I think we'll have to work out a better concept in the future, as the boxes take too much attention in this case compared to the actual description of the option.


Pin meltano/tap-salesforce to tag v1.4.27

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/issues/3113
Originally created by @edgarrmondragon on 2021-12-09 19:29:35

An update to allow schema selection might soon be merged
and it may not be considered a simple patch, so we should pin the current latest (v1.14.7) tag so users migrate at their own pace to the newer version that supports schema selection.

View 2 previous comments from the original issue on GitLab


NAMD hangs on Edison XC30

Original issue: https://charm.cs.illinois.edu/redmine/issues/533

v6.6.0-rc3-87-g8f0a0cd gni-crayxc-smp built July 6 hangs during runs, occasionally also crashes but ATP doesn't appear to work and I haven't seen any stack traces, just errors like this:
------------- Processor 179078 Exiting: Called CmiAbort ------------
Reason: Converse zero handler executed-- was a message corrupted?
aborting job:
Converse zero handler executed-- was a message corrupted?
or sometimes:
_pmiu_daemon(SIGCHLD): [NID 02639] [c5-1c2s3n3] [Thu Jul 17 20:54:55 2014] PE RANK 46 exit signal Segmentation fault

Original date: 2014-07-24 20:51:58

I haven't been able to reproduce this yet.  On what scale (number of processors or total run time) did you experience these failures?
On an unrelated note, I'm getting failures trying to run apoa1.  I am getting "FATAL ERROR: Error opening XSC output file ~/data/apoa/apoa1-out.xsc: No such file or directory" even though the file exists and has suitable permissions.

Original date: 2014-07-25 14:35:13

This is user-reported. I haven't been able to reproduce it myself.
I assume your output error is that the directory ~/data/apoa doesn't exist.

Original date: 2014-07-25 17:51:06

Okay, I'll keep trying, then.
It seems my issue was that I used ~ in my pathname.  Changing it to use the full absolute path worked.

Original date: 2014-07-28 15:56:30

Gengbin sent me a patch on Friday and I built new binaries.  The user reports he hasn't seen any crashes since then.  We'll give it a few more days to be sure but I would suggest getting Gengbin's changes checked in ASAP.


Bugfix: no hole in liveranges for pinned vreg move src.

Right now, pinned vregs are a way of naming real registers (a
compatibility shim of sorts for Cranelift's RealRegs) and can be used
as sources and dests of moves. When the input program does so, regalloc2
converts these into "ghost" uses and defs on the other vreg of the move
(dest or source, respectively) with a fixed-register constraint. So
move v128, v0 where v0 is pinned to p0 turns into a "ghost def"
on v128 with constraint fixed p0.
There is some fancy manipulation of liveranges to make this all work
while properly recording where the preg's value must be preserved.
Unfortunately, there was an off-by-one in the location of the move and
transition of live-ranges which interacts poorly with the "implicit
live-in" of pinned vregs at function start. As a result, a function body
that starts like:
    move v128, v0
    def v9000
    move v129, v1

might allocate p1 (to which v1 is pinned) for v9000. This clobbers
the original value.
Fortunately this only impacts the implicit live-in, and Cranelift's use
of regalloc2 is such that it will always copy all values out of pinned
vregs (creating ghost defs) without intervening defs, except in the
case of sret ("structure return") arguments. If a program does not use
sret arguments (and the cranelift-wasm frontend does not), then this
bug should not be reachable.
Long-term, we really need to kill pinned vregs with fire (#3); the
special cases that arise from these, and from special handling of moves,
are too much incidental complexity. All of this can go away once
Cranelift migrates all fixed-register cases to operand constraints
instead. That will be a happy day.
Thanks to @bjorn3 for finding and reporting this issue!

My plan for this re: other PRs: we will want to land this first, and do a 0.2.3 release of regalloc2, and then do a point-release of Cranelift that bumps the RA2 dep. Then we'll want to land #58 and #59 and version-bump to 0.3, since #58 changes the API slightly.


chore: process rules from axe raw results

Description of changes

Updated result-to-rule-converter to include converting axe raw results to sarif rules
Added relevant unit tests
Updated axe-raw-sarif-converter-21.ts to use the result-to-rule-converter via static method fromRawResults
Updated sarif-converter-21.ts to use static method fromV2Results of result-to-rule-converter

Pull request checklist

[x] PR title respects Conventional Commits (starts with fix:, feat:, etc, and is suitable for user-facing release notes)
[x] PR contains no breaking changes, OR description of both PR and final merge commit starts with BREAKING CHANGE:
[x] (if applicable) Implements WI: 1553643
[x] Added relevant unit tests for your changes
[x] Ran yarn precheckin
[x] Verified code coverage for the changes made

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


How to skip rows that failed to decode?

I have a table with some datatypes that require custom encoding. Right now if the encoding fails, the whole query fails. Is it possible to fail only the specific record/row and return all the other records instead?

Wrap the type into an Option[_] and return null on your custom codec if it fails.


Fixed Drum mag and APCM5 sight

🤔 What type of PR is this? (check all applicable)

[ ] 🍕 Addition
[ ] ⌨ Productivity
[X] 🐛 Bug Fix
[ ] 🔥 Optimization
[ ] ⚙ Configuration
[ ] 🌟 Quality Of Life
[X] ✨ Enhancement
[ ] 📝 Documentation

🎯 Key Objectives
The changes are focused on fixing specific bugs related to the models.
🚦 Testing and  Screenshots/Recordings 🖼

Images of the problems that have already been fixed
📖 Added to documentation?

[X] 📜 README.md
[ ] 📓 Javadoc
[ ] 🍕 Comments
[ ] 🙅 no documentation needed

Please update the changelog to reflect theses changes and we can merge!

damn, just modify two lines and the bot bugs

umm .. nothing needs to be fixed, I've tried it and it works fine


This test is not reliable at high speeds

Testing on a Comcast 250/25 connection (Provisioned around 300/30), as compared with various other speedtests.
speedtest-cli:
Nathans-MacBook-Pro:~ virtuallynathan$ speedtest-cli --server 1769
Retrieving speedtest.net configuration...
Retrieving speedtest.net server list...
Testing from Comcast Cable (68.84.XXX.XXX)...
Hosted by Comcast (Plainfield, NJ) [92.80 km]: 24.155 ms
Testing download speed........................................
Download: 136.20 Mbit/s
Testing upload speed..................................................
Upload: 26.90 Mbit/s
Speedtest.net
Plainfield, NJ
Ping: 13ms
Down: 288Mbps
Up: 30Mbps
http://www.speedtest.net/result/5045175786.png
Speedtest.comcast.net
Plainfield, NJ
Ping: 13ms
Down: 275Mbps
Up: 30Mbps
http://results.speedtest.comcast.net/result/1198333904.png
DSLReports
Ping: 17ms
Down: 280Mbps
Up: 30Mbps
http://www.dslreports.com/speedtest/2841780.png
See (1000 speedtests over a bit less than 24hrs):

This is not a CPU/memory limitation

I read the README.md, but that states "This is not a reliable latency reporting tool". At the very least, please update it to state it is not a reliable measurement tool for speeds, or latency.

I'll get on that.