Chore/modify .gitignore and remove non-required file

Description
Removing .vscode/ files and adding them to .gitignore
Remove hidden .DS_Store files
Motivation and Context
To keep the repository clean from unnecessary files
How to Test
existing or new .vscode/ files created should not be commited to the repository
Checklist
Please check all the boxes that apply to this pull request using "x":

[x]  I have tested the changes locally and verified that they work as expected.
[ ]  I have added or updated the necessary documentation (README, API docs, etc.).
[ ]  I have added appropriate unit tests or functional tests for the changes made.
[x]  I have followed the project's coding conventions and style guidelines.
[x]  I have rebased my branch onto the latest commit of the main branch.
[ ]  I have squashed or reorganized my commits into logical units.
[ ]  I have added any necessary dependencies or packages to the project's build configuration.
[x]  I have performed a self-review of my own code.

.vscode files will be kept for debugging purposes
prettier has been added for consistent code style here Chore - consistent code style with prettier #96
.ds_store files to be cleaned up in a separate PR
Closing this PR


Usage for Maven?

Could this possibly be used for a maven project?

you could have gradle execute maven, beyond that, no.


Problem with Discord Interaction Infrastructure

Hi! Thank you for this project!

Check that you now have a stack called dev-discord-interaction in Cloudformation. Cross your fingers crossed_fingers and call the command /eth-auth in your Discord channel. If everything went right with the deploy, the bot should respond with a URL.

Do you have any ideas why at this stage I am not receiving any messages from the bot on discord?

When you type / (forward slash) on your server, do you see the eth-auth command suggested?
If not, it's probably a problem with the bot authorizations or you didn't set the right server id.

Probably an issue with Discord permissions. Check the logs on Cloudwatch for the discord-interaction lambda, the answer will likely be there,

Probably an issue with Discord permissions. Check the logs on Cloudwatch for the discord-interaction lambda, the answer will likely be there,

@EsteFilipe Maybe you have the ability to communicate via chat like Telegram? Not for free.
My username: @epexa
Or write your contact in any messenger.


IIWBook no longer boots. Investigate

Looks like possible change in the way agent retrieves schema

This has been fixed in a previous update to icatagent - now aca-py


New Section: Deploy to Production

Link to the Page
https://www.skeleton.dev/docs/deployment (not yet created)
Describe the Issue
As discussed with @Sarenor, I think we need a new Docs > Resources > Deployment section to pair with Quickstart. Quick start shows the initial setup, this Deployment sections shows you everything to do to prep for release.
This could include, but is not limited to the following:

@AdrianGonz97's PurgeCSS tool (when available)
Recommended tips for preparing for release - ensure your app is optimized, etc.
Written + Video guides for deploying to the X number of top provides (Vercel, Netlify, Amplify, etc)

To be clear - these will NOT be guide we write and maintain, but rather resources we can find online. Be that the official documentation for each service, or useful video guides like this one by Joy of Code:
https://www.youtube.com/watch?v=uAF4Yd-gddo
We don't have the time or resources to maintain these, and there's not really anything special about deploying a Skeleton app - aside from the upcoming PurgeCSS instructions for better optimization.
Are you able to create a Pull Request with the fix?
Yes

Note: Vercel's official instruction uses their dedicated adapter, but I'm not sure we want to recommend that or not:
https://vercel.com/docs/frameworks/sveltekit

I have deployed Skeleton apps numerous times for work projects,
I have experience with
Express/Node Adapter/Docker
Node Adapter/Heroku
Vercel
Happy to help contribute tips and gotchas there

Hello, I want to help with this.

Hey @FarhanAliRaza this is issue is already assigned. We're waiting on some pre-requisites before we push forward on it.
If you would like to volunteer on another issue, look for one that's not already assigned and has the "help wanted" tag.

And if you've got links to online resources we can share on the "Deploy to production" page that you think are a good fit, feel free to leave them here :)

With the launch of v2 we now have a dedicated section for the PurgeCSS tool here:
https://www.skeleton.dev/docs/purgecss
I'm leaning more and more towards deployment to set services (Vercel, Netlify, etc) fall under the jurisdiction on SvelteKit and their adapter system, so should aim to forward anyone interested in this topic here:
https://kit.svelte.dev/docs/adapters
Beyond that, I do welcome contributed blog posts on any other topics in this vicinity.


Added Nextflow Tower CLI wait

I added the tower CLI wait functionality to the action as the default. You can set the wait to false to disable. This should address everything that was discussed in issue #13.

I also added run name as a parameter so that a unique name could be specified to make the github action run standout from other runs in tower.

This looks brilliant, thank you! I'm moving this repo so will close this PR now and move it to https://github.com/seqeralabs/action-tower-launch/pull/1


Error in AdaptiveDetector for some videos

Bug/Issue Description:
Hey, I am using the library to find scenes in youtube videos. It is working good on the most videos but for some, I facing this issue for AdaptiveDetector
Traceback (most recent call last):
  File "scene_detection/scene_detection.py", line 57, in <module>
    scene_list = find_scenes(video_path, output_path=output_path,
  File "scene_detection/scene_detection.py", line 27, in find_scenes
    scene_manager.detect_scenes(video=video_stream)
  File "/home/user/env/lib/python3.8/site-packages/scenedetect/scene_manager.py", line 813, in detect_scenes
    self._post_process(video.position.frame_num)
  File "/home/user/env/lib/python3.8/site-packages/scenedetect/scene_manager.py", line 696, in _post_process
    self._cutting_list += detector.post_process(frame_num)
  File "/home/user/env/lib/python3.8/site-packages/scenedetect/detectors/adaptive_detector.py", line 152, in post_process
    denominator += self.get_content_val(frame_num + offset)
TypeError: unsupported operand type(s) for +: 'float' and 'NoneType'

Required Information:
video_stream = open_video(video_path, backend='pyav')
stats_manager = StatsManager()
 # Construct our SceneManager and pass it our StatsManager.
 scene_manager = SceneManager(stats_manager)

 # Add ContentDetector algorithm (each detector's constructor
 # takes various options, e.g. threshold).
 detector = AdaptiveDetector(adaptive_threshold=3.0, luma_only=True, min_scene_len=15, min_delta_hsv=15.0, window_width=4, video_manager=None)
 scene_manager.add_detector(detector)
 # Perform scene detection.
 scene_manager.detect_scenes(video=video_stream)
 scene_list = scene_manager.get_scene_list()

Expected Behavior:
A list containing timestamps of scenes
Computing Environment:

OS: Ubuntu 22.04
Python Version: 3.8.13
OpenCV Version: <IP_ADDRESS>

Additional Information:
Please note that other detectors i.e ContentDetector and ThresholdDetector are working fine on the video.
Media [Videos/Images/Screenshots]:
I tested on this youtube video, you can download it using youtube-dl
youtube-dl https://www.youtube.com/watch\?v\=HBOpO7RgS64

The problem was actually because of video codec - it uses AV1 codec.
I had to convert it using ffmpeg
ffmpeg -i demo.mkv -vcodec libx264 -vprofile high -crf 28 -c:v h264 demo.mp4

For those running across this in the future, this seems like a similar problem to #86, though with a different codec. If that is the case, per the discussion over there, this is likely due to a bug in OpenCV. The problem could be solved through either updating to a newer version of OpenCV or trying a different backend by specifying --backend pyav. API reference for backends can be found here.

Actually I tried to use pyav backend but didn't workout and yes the problem is due to opencv also discussed in opencv forum

@MNMaqsood what happened when you used the PyAV backend?  Would you be willing to test a .whl of the upcoming v0.6.1 version?  This may be resolved due to the changes made in #283, so I want to understand if you still get an app crash with that or if there's still work to be done there.
May I also ask why you set the video_manager parameter?  This parameter should never be used, so I want to understand how to make it clear this value should not be set (even if setting to None).

Closing due to lack of activity, will revisit upon any new updates.


fix-#80 ui button issue solved

Summary
Ui - button on the latest post card redirecting to a empty tab solved
Description
button on the latest post card redirecting to a empty tab solved . Now  when clicking on the button it opens the specific post in a new tab .
Added a new function "openInNewTab" to control this behaviour .
Issue(s) Addressed
Closes #80

[x] Have you followed all the CONTRIBUTING GUIDELINES?

there are multiple ways to solve the issue , either we could just remove that button or use it to open the blog in a new page . Leaving that button with no function will not be a good way to solve the issue . Making that button to open the blog in the same page will also be useless as we are already using the navigate fn on the main div .
i thought of opening the blog in a new tab would be effective and added the function to do so .  I read using "target:_blank" directly might have some security concerns . Please correct me if  I am wrong ! .
I Could directly navigate to the new tab or simply just remove the button . Both options feel right to me . can you suggest your opinion ?

@mohamedshamil7 lets not complicate this

The very obvious fix would be just copy pasting Line No: 14's onclick to 22's onclick, that will solve the issue.
I just want you to explore if there is any other way of doing this, by which i mean,
Now the button is the child of the parent div in which the navigation is working perfectly fine right?
There may be some way that we may this button's click of no effect and letting it work simply like the parent, you can try just by removing the onclick, see if the existing parent's click works.
I really appreciate your thoughts and creativeness of opening in a new tab, but we would ideally don't want that much of complexity presently, and the present PR doesn't actually work as per your fix.

As per your suggestion i think at present there is no much use of that button . I guess i will remove the navigate functionality from the button

Yeah it is just to indicate that if you click this post you'll go to the details, as said in discord, it is for UX, showpiece basically.
It alone doesn't have any functionality


Update an account in the service

As an administrator I need to update accounts so that Unused or absent users aren't mistaken as in use


arrow's state is not maintained

When a category is opened the arrow rotates to point downward, which is important functionality.
But if you scroll down so that the category row is off the screen and then scroll back up to show it again, it resets the icon to the down position even though the category is opened.
This makes it seems like the category is closed even though it's opened.
Does anyone have a fix for this?

https://github.com/thoughtbot/expandable-recycler-view/issues/67#issuecomment-372026766


procedural macros

fixes #40

This is great! In the test I see that bob() is called inline as an argument to printf. That makes me assume that the function returns a char pointer, but it in fact just calls printf, with no returnable value. Does that test currently pass?

So the idea is the stdout of a macro is used as a return value. Does the macro support inline return values?

macros do not have return values. they expand to an arbitrary expression
could you give me an example of what you mean by "inline return value" ?

@aep yes, by inline return value I meant the closet thing to pre-processor expression evaluation inline at the call site so things like
macro add(x, y) { x + y }
printf("%d\n", add(1, 2));

work as intended

Yes. zz will never support a string preprocessor where the macro can expand to arbitrary text.
This leads to very confusing ASTs.
Add in your case should be a function, not a macro
However, if you have a more specific use case, maybe I can optimize the macro semantics.

Yes, the add() example is contrived but I imagine more complex examples like loop abstractions:
macro loop() {
 while (1)
}

loop {
  // caller explicitly breaks
}

macro each(list, i) {
  for (int i = 0, item = list.items[i]; i < list.length; ++i)
}

struct List+L {
  int length;
  void *items[L];
}

fn push(List+L, void *item) -> int {
  self->items[self->length] = items
}

fn create(List+L mut new *self) {
  self->length = L;
  memset(self->items, 0 L * sizeof(self->items));
}

new+4 list = create();
each(list, i, item) {
  printf("%s\n", (char *) item);
}

After playing around with the procmacros branch I understand the intention with macros here. They are not intended to mirror the pre-processor's behavior but be like micro programs that just emit an AST to be evaluated inline with the reset of the program source.

ah yes, that is intentionally not allowed.
If you want to create control flow items like that you'd have to use a preprocessor.

to clarify, you CAN create the thing you want, it just needs to be complete control structures rather than incomplete items.
so something like

macro each(list, i, block) {
  printf("for (int i = 0, item = list.items[i]; i < list.length; ++i) {");
  printf("%s", block.to_zz());
  printf("}");
}

new+4 list = create();
each(list, i, item, {
  printf("%s\n", (char *) item);
})

@aep That is great. Is .to_zz() implemented? What's the API look like for macros now?

Nope. Arguments aren't even implemented :)
I'll get to it when the kid's asleep

sounds great, thank you! :]

arguments are sort of there. there's some sugar missing, but essentially it works like this:
macro repeat(rep, times)
{
  err::assert(rep.typ == ast::ExpressionType::LIteralString);
  err::assert(times.typ == ast::ExpressionType::LIteral);

  printf("\"");
  for (int mut i = 0; i < atoi(times.val.literal); i++) {
    printf("%s", rep.val.string);
  }
  printf("\"");
}

fn main() {
  char *xxx = repeat("x", 3);
}

Looks great. Thanks for the docs too


Expose "toExtend" functions

Alongside the existing ExpectTypeOf methods, a method to check for successful extension of types would be helpful. Perhaps a strict/branded version as well.
Proposed syntax:
expectTypeOf<5>().toExtend<number>();
expectTypeOf<unknown>().toExtend<any>();
expectTypeOf('abc').toExtend([{}]); // Typescript error

expectTypeOf<5>().toStrictExtend<number>();
expectTypeOf<unknown>().toStrictExtend<any>(); // Typescript error
expectTypeOf('abc').toStrictExtend([{}]); // Typescript error

I would be happy to submit a PR for this. Especially since the Extends and StrictExtends types are already implemented, just not directly exposed in the main method.
My current workaround is:
expectTypeOf<
     StrictExtends<Foo, Bar>
>().toBeEqual<true>();

Admittedly the grammar is a bit different (most methods start with toBe) so open to a renaming to match. toBeExtending...?

Ah I think I was confused reading through code.
Looks like this is effectively toMatchTypeOf, which is always the non-strict type.
I guess the followup is instead to support the strict "branded" version of toMatchTypeOf (similar to branded.toEqualTypeOf)

This is toMatchTypeOf, but I am considering deprecating toMatchTypeOf in favour of something like toExtend (probably making toMatchTypeOf an alias). Similarly I might rename toEqualTypeOf to be some kind of toBeIdenticalTo. But, for now, use toMatchTypeOf. I'll leave this open until that happens (or I decide definitively not to do that).


Render markup in figure sc caption and attr

First commit cleans up the shortcode to make it more readable, and ready it for the second commit. This commit also updates the tests to pass for the white-space changes made in this commit.
Second commit renders the Markdown in caption and attr parameters of the figure shortcode. 2 test cases are added for this change too. None of the existing tests (after the prior whitespace fix commit) needed to be modified for this second commit.

Fixes https://github.com/gohugoio/hugo/issues/4406.

I found these 13 tests to fail locally before/after building with this PR:
--- FAIL: TestPageWithMoreTagOnlySummary (0.43s)
--- FAIL: TestSummaryWithHTMLTagsOnNextLine (0.34s)
--- FAIL: TestWordCountWithIsCJKLanguageFalse (0.48s)
--- FAIL: TestWordCount (0.53s)
--- FAIL: TestWordCountWithMainEnglishWithCJKRunes (0.53s)
--- FAIL: TestWordCountWithAllCJKRunesWithoutHasCJKLanguage (0.62s)
--- FAIL: TestWordCountWithAllCJKRunesHasCJKLanguage (0.67s)
--- FAIL: TestPageWithMoreTag (0.54s)
--- FAIL: TestPageWithEmbeddedScriptTag (0.65s)
--- FAIL: TestPageWithShortCodeInSummary (0.66s)
--- FAIL: TestPageWithDateFields (0.62s)
--- FAIL: TestPageWithDelimiter (0.59s)
--- FAIL: TestCreateNewPage (0.48s)
--- FAIL: TestGetJSON (4.00s)

@kaushalmodi It looks like there is no whitespace between the caption and the attribution link. I.e. this:

Image byAuthor

renders in browser as:
Image byAuthor


selectItem implementation

Added a selectItem override to the InfiniteCollectionView to allow programatic selection of a cell. This was missing (and therefore broken) in the previous implementation.

@jwardle
Thank you for contributing !


Make it work on Linux

Tried to make it work on Linux. The stopwatch icon apparently was in a format that didn't work with Mono on Linux. I re-saved it using Gimp. Now the project at least compiles, so this might be a first step to fix #53 and #70. When started, the program still throws an exception immediately, though.

I could get rid of the mentioned exceptions by some platform checks.
I have no experience with C# and could use some advice. It seems that the lines disabled by platform checks will only mean some minor inconveniences in usage. If I see this correctly, it would mean:

password will have to be re-entered
window arrangement will not be ideal
pasting into issue combo boxes will not work

Is this correct? These parts might be re-implementable to be cross-platform, but I lack the knowledge for that.

I'm not at all familiar with xmake or mono, can anyone provide any guidance on how to compile this on Ubuntu?  I've tried both methods with no success.


mae-consumer unable to authorize to elastic

Describe the bug
Got an error from mae-consumer while using kubernetes chart with linkedin/datahub-mae-consumer:v0.8.45 image
base64: unrecognized option: wrap
BusyBox v1.33.1 () multi-call binary.

Usage: base64 [-d] [-w COL] [FILE]

Base64 encode or decode FILE to standard output
	-d	Decode data
	-w COL	Wrap lines at COL (default 76, 0 disables)

To Reproduce

Deploy helm chart with inkedin/datahub-mae-consumer:v0.8.45 image
Add all necessary credentials
Watch logs

Expected behavior
Application is able to connect to elasticsearch
Screenshots

Additional context
Actually root of this problem is using alpine image, which repository do not supported properly, i suggest some previous version of base64 contains necessary keys.
So fast fix is replace --wrap with -w in start.sh script here
https://github.com/datahub-project/datahub/blob/master/docker/datahub-mae-consumer/start.sh#L9
But better solution is to get rid of alpine image at all :)

--help of base64 in linkedin/datahub-mae-consumer:v0.8.45

Hello @Wilderone
Thank you for submitting this very helpful diagnosis of the issue. There is a PR in progress to address this issue: https://github.com/datahub-project/datahub/pull/6061, could you take a look and see if it matches what you found?

Hi @pedro93 !
Yep, base64 from coreutils would fit fine, i checked it in fresh alpine image, --wrap flag present in this version


Ongoing Anime labelled as completed in yomiroll

Source information
Yomiroll 13.24
Source language
Multi
Steps to reproduce

Find an ongoing anime
Add it to library

Expected behavior
Be labelled as ongoing
Actual behavior
Is labelled completed
Aniyomi version
Preview r6668
Android version
Android 13
Other details
This wouldn't have been a problem but the latest preview added a new feature called "Show next episode's airing time". Problem is that even if the anime is currently releasing, if the sources mislabel it as completed the app would also think it's completed and won't display the airing date.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] I have updated the app to version <IP_ADDRESS>.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

I suppose it's an API issue? You can switch the status with edit info for now.

The extension sets the "Completed" status to every single anime, regardless of what the API says. Also, "edit info" is an Animiru exclusive feature.

I was on a wild goose hunt trying to find the edit info button. Should I open an issue in the main app to suggest that regardless of what the sources says if the tracker has any kind of airing data it would just show it? But that was take a hella long time to be implemented.

The next airing data was hidden for completed anime to a) lessen the load for anilist and b) hide it for sources that splits up an entry differently from the tracker. This is an issue with the extension assigning the wrong status, not that the app hides it when it shouldn't.

The next airing data was hidden for completed anime to a) lessen the load for anilist and b) hide it for sources that splits up an entry differently from the tracker. This is an issue with the extension assigning the wrong status, not that the app hides it when it shouldn't.

I zee what it is

The extension sets the "Completed" status to every single anime, regardless of what the API says. Also, "edit info" is an Animiru exclusive feature.

It's so handy, that I assumed the origin would be aniyomi. My bad, since I switched from Aniyomi>Animiru>Kuukiyomi ^^


GPU support

Is this library currently single threaded CPU only?

This library is just a thin wrapper over whisper.cpp: https://github.com/ggerganov/whisper.cpp
You'd need to check upstream to see, and as of late I believe it is still CPU only, however parallelization has been added recently. If you're on an Apple Silicon Mac, apparently it is GPU accelerated, but as I don't have one I'm unable to test at all there.

https://github.com/ggerganov/whisper.cpp/issues/61
I see. Looks like no GPU support planned in the near future. Thanks anyways!

There is a branch with GPU support and some instructions on how to get it running but still no official support: https://github.com/ggerganov/whisper.cpp/issues/220


Add Internet connection test to download-requirements.sh (2124)

How about not exiting in a child function but in the parent? For example:
__check_curl() {
    (( $# > 0 )) || exit_with_error "__check_curl: no url provided"
    local url=$1

    echol "Testing connection to: \"${url}\" using curl..."

    err_msg=$(curl --show-error --silent "$url" 2>&1 >/dev/null); rc=$?
    if (( rc > 0)); then
        echol "curl failed, reason: [$err_msg], exit code: $rc"
        return $rc
    fi
}

check_connection() {
    if ${tools[$tool]} $@; then
        echol "PASSED"
    else
        # here we can decide what to do, e.g. run `exit_with_error` or return exit code from child function
        echol "FAILED"
    fi
}

check_connection curl https://github.fake
check_connection curl https://github.com

# output:

Testing connection to: "https://github.fake" using curl...
curl failed, reason: [curl: (6) Could not resolve host: github.fake], exit code: 6
FAILED
Testing connection to: "https://github.com" using curl...
PASSED


[Installation]: Stuck for two hours during the installation of vllm

Stuck for two hours during the installation of vllm(pip install vllm), Does anyone know the reason and how to solve it

How you are installing vllm
pip install vllm

you need internet access to google cloud storage. check https://github.com/vllm-project/vllm-nccl for details.

@youkaichao is there a way to offline download, and where to put it?

put the file libnccl.so.2.18.1 in ~/.config/vllm/nccl/cu12 and install the vllm successfully

Sorry for the trouble. This might be not necessary when https://github.com/NVIDIA/nccl/issues/1234 is resolved. But we have to wait ...

i encounter the similar issue...

same issue. already have libnccl.so.2.18.1 in ~/.config/vllm/nccl/cu12 but still stucking.

same issue.

which version do you install and how do you install vllm?


Hugo cleanup

Summary
A cleanup of Hugo and its files, consisting of:

Removing unused templates found using the --printUnusedTemplates parameter
Removing extraneous data in the themes/ directory
Upgrading Hugo from 0.92.1 to 0.101.0 in the build pipeline

Hello @neflyte,
Thanks for your pull request! A Core Committer will review your pull request soon. For code contributions, you can learn more about the review process here.

@pfltdv - This PR will have an impact on our build pipelines for the dev docs. Would you be open to reviewing this PR, please?

@pfltdv - This PR will have an impact on our build pipelines for the dev docs. Would you be open to reviewing this PR, please?

@cwarnermm No other change is needed to upgrade Hugo at pipelines and pipelines are working after Hugo upgrade.


Some delays observing while screen share android

I am feeling some delays while sharing screen from tablet to android mobile.
Below is my code,
`class CustomVideoCapturer : BaseVideoCapturer() {
 companion object {
    const val TAG = "CustomVideoCapturer"

    const val pixelFormat = ABGR;
 }

private var width: Int = 640
private var height: Int = 480
private var isCapturing: Boolean = false

override fun init() {
    Log.e(TAG, "On Init")
}

override fun onResume() {
    Log.e(TAG, "On Resume")
}

override fun onPause() {
    Log.e(TAG, "On Pause")
}

override fun getCaptureSettings(): CaptureSettings {
    val captureSettings = CaptureSettings()
    captureSettings.width = width
    captureSettings.height = height
    captureSettings.fps = 7
    captureSettings.expectedDelay = 0

    return captureSettings
}

override fun startCapture(): Int {
    Log.e(TAG, "Start Capture")
    return 0
}

override fun stopCapture(): Int {
    Log.e(TAG, "Stop Capture")
    return 0
}

override fun isCaptureStarted(): Boolean {
    return isCapturing
}

override fun destroy() {
    Log.e(TAG, "Destroy")
}

fun sendFrame(imageBuffer: ByteBuffer, imageWidth: Int, imageHeight: Int) {
    provideBufferFrame(imageBuffer, pixelFormat, imageWidth, imageHeight, 0, false)
}

}`
And below is screen sharing code,
        ` screenPublisher =
            Publisher.Builder(this@NewTelevisitActivity).name(jsonObject.toString())
                .capturer(customVideoCapturer).build()
        screenPublisher?.publisherVideoType =
            PublisherKit.PublisherKitVideoType.PublisherKitVideoTypeScreen
        screenPublisher?.publishVideo = true
        screenPublisher?.publishAudio = publisher?.stream?.hasAudio() == true
        screenPublisher?.audioFallbackEnabled = false
        screenPublisher?.setPublisherListener(this)
        session?.publish(screenPublisher)`

As you can see in attached video, while I am expanding/collapsing framelayout, the screen shared video will be shown in black and if I will do any action in tab the screen shared video will be shown again or the frames will be again visible withing 10 to 12 seconds. Please suggest me if any kind of resolution issue or something else.
https://github.com/opentok/opentok-android-sdk-samples/assets/79642753/c3e14db2-f932-484c-afe2-a1e358f09772

Hello @v-kpheng, is there any update regarding this issue?

@vmeditab I have noted we were not providing a Kotlin sample for screen sharing using a WebViewer so I have created one. At the moment it was not merged yet, but you can have a look at it in https://github.com/opentok/opentok-android-sdk-samples/pull/486.
In regards to your specific problem, I see you have setted the video type of the published
stream to PublisherKitVideoType.PublisherKitVideoTypeScreen, which optimizes the video encoding for
screen sharing. You have also setted a low frame rate, which is advised, however it seems you have not provided the entire code of your CustomVideoCapturer class:

With each periodicity is sendFrame() called?
When do you start sharing the screen? I see startCapture() is not doing anything.

Also your issue is a bit more complex than only sharing the screen, since you are sharing the screen from your tablet to your Android mobile, which is where you expand/collapse the frameLayout. It is possible when these events are triggered, the code is not being handled correctly.
Could you share more code of your project?

Hey @goncalocostamendes ,
Please see my code below for activity where I am handling expand collapse framelayout. I am using recyclerview for below layout of the main view and simply used notifyItemchanged methods while pin/unpin.
MainActivity.kt
//This method while sending the frame
    override fun sendFrame(imageBuffer: ByteBuffer, width: Int, height: Int) {
        customVideoCapturer?.sendFrame(imageBuffer, width, height)
    }

// This is used for swapping framelayouts
      attendeesAdapter =
            AttendisAdapter(this@NewTelevisitActivity, object : AddViewDetailClickListener {
                override fun onSwapViews(
                    container: NewSubscriberContainer?, index: Int
                ) {
                    binding.fmSubscriber.restore()
                    subscriberContainer?.let { mSubscribers.set(index, it) }
                    attendeesAdapter.notifyItemChanged(index)
                    subscriberContainer = container

                    updateMainView()
                }
            })

//This is used for update the main large view which is showing in upper side

    private fun updateMainView() {
        binding.fmSubscriber.removeAllViews()
        subscriberContainer?.subscriber?.let {
            it.view.removeSelf()
            (it.view as GLSurfaceView).setZOrderOnTop(false)
            // https://medpharm.atlassian.net/browse/PP-608
            // Point - 2
            // set ratio for display for web screen share
            if (subscriberContainer?.userDetails?.platform == "web" && subscriberContainer?.userDetails?.designation_desc == "Screen") {
                // PP-608 (Reopen Point - 1)
                // added zoom in zoom out
                val layoutParams = FrameLayout.LayoutParams(
                    binding.fmSubscriber.width,
                    (binding.fmSubscriber.height / 3.5).toInt()
                )
                it.view.layoutParams = layoutParams
                layoutParams.gravity = Gravity.CENTER
                binding.fmSubscriber.addView(it.view)

                it.setStyle(
                    BaseVideoRenderer.STYLE_VIDEO_SCALE,
                    BaseVideoRenderer.STYLE_VIDEO_FIT
                )
            } else {
                binding.fmSubscriber.addView(it.view)
                it.setStyle(
                    BaseVideoRenderer.STYLE_VIDEO_SCALE,
                    if (subscriberContainer?.userDetails?.platform == "tab") BaseVideoRenderer.STYLE_VIDEO_FILL else BaseVideoRenderer.STYLE_VIDEO_FIT
                )
            }
            binding.fmSubscriber.enableTouch(subscriberContainer?.userDetails?.designation_desc == "Screen")
            binding.rlVideo.visibility = if (it.stream?.hasVideo() == true) GONE else VISIBLE
            setVideoListener(it, binding, subscriberContainer)
            setStreamListener(it, binding)
            binding.imgPinnedMute.visibility = VISIBLE
            // PP-745
            // update mic icon of subscriber for main view
            binding.imgPinnedMute.setImageResource(if (it.stream?.hasAudio() == true) R.drawable.ic_pin_mic else R.drawable.ic_pin_mute)
        }
        subscriberContainer?.publisher?.let {
            it.view.removeSelf()
            // PP-608 (Reopen Point - 1)
            // added zoom in zoom out
            binding.fmSubscriber.addView(it.view)
            it.setStyle(
                BaseVideoRenderer.STYLE_VIDEO_SCALE,
                BaseVideoRenderer.STYLE_VIDEO_FILL
            )
            binding.rlVideo.visibility = if (it.stream?.hasVideo() == true) GONE else VISIBLE
            binding.imgPinnedMute.visibility = GONE
            binding.fmSubscriber.enableTouch(false)
        }
        binding.tvSubscriber.text = subscriberContainer?.name
        binding.tvShortName.text =
            ConstTM.getShortName(subscriberContainer?.userDetails?.role_name)
    }

@vmeditab, a new sample app was created: https://github.com/opentok/opentok-android-sdk-samples/tree/main/Screen-Sharing-Kotlin.  Can you still reproduce with that?


CompoundV3 Zap Supply

Zap Supply

[x] included in route.ts
[x] implementation
[x] unit test
[x] e2e test
[x] changeset

Discussions

[x] need getBaseTokenBalance from compoundv3/service.ts (P-logics)
[x] should fail e2e test
[x] Paraswap token issue

LGTM


[FEATURE] Improve search bar for searching something...

Detailed description

Improved Search bar section by added some CSS properties on it to look it good.
Add search icon embedded on search bar
Make it responsive in all devices.
@Deepu178 I would like to contribute on this project under GSSOC'23!!
Context
For better UI .
Possible implementation
No response
Additional information
No response
Code of Conduct

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

@Geeta259 how are you gonna improve the user experience by working on search bar?
I was looking forward to the search bar like if you focus on it it will increase its width.

@Geeta259 are you working on this issue?


Infoga having issues installing since the repo link is showing 404

the script also stop scanning once it detects infoga is not working or installed.
all tools are getting installed and there is some issue in script which gives error saying cannot cd to directory /home/Tools/ and the recon stop instantly.

Fixed on dev branch https://github.com/six2dez/reconftw/issues/768


general question

I briefly run a
for file in *; do ./"$file"; done

in the path of compiled executables, /LillyMol_6_cmake/src/build/build for a copy-paste of the output to the CLI to a permanent record.  There are 111 occurrences of help, 41 of the string -h (at least sometimes to adjust how to process hydrogens), yet never --help (as e.g. for running openbabel from the CLI). Instead, the options become visible after e.g., ./sp3_filter as command without an option.  Can this frequently seen toggle for comfort be amended to the utilities?

Unfortunately the command line processing built into LillyMol utilities is
rather inflexible, having been laid down in 1995. So, there are no --opt
recognised.
Every c++ tool should respond to an invocation with no inputs by writing
its usage message.
Now, I did implement a newer generation command line processing that has
more flexibility, and which could be adapted to things like --help. But
making this change would be disruptive, and would break tools that others
have built. What might make sense is to start writing new tools with the
updated command line processor.
Command line processing in LillyMol is somewhat unusual. Multiple uses of a
flag is commonplace and encouraged. For example, -v will increase the
verbosity of messages. Using two, -v -v, possibly enables even more
debugging. More traditional tools will have a --verbose  option. But
specifying queries or smarts
tsubstructure -s 'smarts1' -s 'smarts2' ...
is a common idiom.
So, I don't have an immediate, non disruptive, fix for this now...
If you find tools that are not responding properly to an empty invocation,
definitely let me know...
but you are right, these tools should behave as most people might
reasonably expect...
Thanks
Ian
On Sat, Apr 22, 2023 at 11:41 AM nbehrnd @.***> wrote:

I briefly run a
for file in *; do ./"$file"; done
for a copy-paste of the output to the CLI to permanent record. There are
111 occurrences of help, 41 of the string -h (at least sometimes to
adjust how to process hydrogens), yet never --help (as e.g. for running
openbabel from the CLI). Instead, the options become visible after e.g.,
./sp3_filter as command without an option. Can this frequently seen
toggle for comfort be amended to the utilities?
—
Reply to this email directly, view it on GitHub
https://github.com/IanAWatson/LillyMol_6_cmake/issues/4, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAODELC2EJRVTA6LSL3YTNDXCP32LANCNFSM6AAAAAAXH5CDAI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Do not use filepath.Join if constructing LinuxKit paths

This will do the wrong thing on Windows, and construct paths with .
fix #142
Signed-off-by: Justin Cormack<EMAIL_ADDRESS>

cc @rn

I think you just want to use path.Join everywhere, unlike filepath.Join that uses / everywhere regardless of platform.

@ijc oh yes.

Ok, reworked using path.Join which is the slash only only version


storage: add deleteFiles()

Fixes #680
Docs: http://stephenplusplus.github.io/gcloud-node-gh-pages-again/#/docs/master/storage/bucket?method=deleteFiles
Todos

[x] Explain in the docs that this could be costly and is not atomic.
[x] Document callback arguments.

This adds a new method on Bucket: deleteFiles.
Old way of deleting all of the files in the bucket. No error handling, no throttling, no paging.
bucket.getFiles(function(err, files) {
  files.forEach(function(file) {
    file.delete();
  });
});

New way with error handling and throttling built in.
bucket.deleteFiles(function(err) {});

Behind the scenes, deleteFiles just calls getFiles. So, you're able to pass your getFiles query object to deleteFiles.
bucket.deleteFiles({ prefix: 'images/' }, function(err) {});

By default, if an error occurs, no more files will be deleted. This can be shut off.
bucket.deleteFiles({ force: true }, function(errors) {
  // errors = null | Error[]
});

Sounds good to me. Hopefully later we can have this be atomic. For now, we should make it super clear in the documentation that it isn't atomic, and you might incur cost for a huge number of operations...... (ie, a big warning sign...)

Ready for another check. Added the docs and also caught some probs with how array params were being displayed. Docs here: http://stephenplusplus.github.io/gcloud-node-gh-pages-again/#/docs/master/storage/bucket?method=deleteFiles

OK LGTM but maybe @ryanseys should take a look ?

LGTM other than the small comment to be updated. equal vs. strictEqual might be a repo-wide change to be made so if you don't change it in this PR that's nbd. Also you're comparing objects so == should suffice because type inference doesn't do much on an object, but I digress. LGTM.

Thanks for reviewing! I think we have a mixture, but I've been
doing deepEqual lately. Not sure if any of them are really better than the
other, but I'll update it to one of the others before this ships.
On Tuesday, July 7, 2015, Ryan Seys<EMAIL_ADDRESS>wrote:

LGTM other than the small comment to be updated. equal vs. strictEqual
might be a repo-wide change to be made so if you don't change it in this PR
that's nbd. Also you're comparing objects so == should suffice because
type inference doesn't do much on an object, but I digress. LGTM.
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleCloudPlatform/gcloud-node/pull/698#issuecomment-119341761
.

equal compares with == so type inference and all that jazz.
deepEqual compares each property of one object with the properties of another object without the objects necessarily being the same object -- good for assert.deepEqual(config, { actual: 'config', goes: 'here })
strictEqual compares with === so either the objects reference the same memory or they are equal primitives.

I'd say strictEqual should be used in place of equal everywhere and leave deepEqual everywhere else.

Changed to strictEqual (will use in the future) and updated the stale comment.


Clamp north and south, wrap around east and west

So you never see white boundaries.

Tile wrapping included as part of #410.


[Feature Request] Make batch ingestion automatic, not a parameter on _bulk

Is your feature request related to a problem? Please describe
A new batch method was added to o.o.ingest.Processor interface in #12457 that allows ingest processor to operate on multiple documents simultaneous, instead of one by one. For certain processors, this can allow for much faster and more efficient processing. However, a new batch_size parameter was added to the _bulk API with a default value of 1. This means in order to benefit from batch processing of any of my ingest processors, I have to do at a minimum two this: determine how many documents to include in my _bulk request and determine the optimal value to set for this batch_size parameter. I must also change all my ingestion tooling to support this new batch_size parameter and to specify it.
Describe the solution you'd like
I want the developers of my ingestion processors to determine good defaults for how they want to handle batches of documents, and then I can see increased performance with no change to my ingestion tooling by simply updating to the latest software. I acknowledge I may still have to experiment with finding the optimal number of documents to include in each _bulk request (this is the status quo for this API and not specific to ingest processors). Also, certain ingest processors may define expert-level configuration options to further optimize if necessary, but I expect the defaults to work well most of the time and to almost always be better than the performance I saw before batching was implemented.
Related component
Indexing
Additional context
I believe this can be implemented as follows:

[required] Increase the default value of batch_size from 1 to Integer.MAX_VALUE. This means that by default the entire content of my bulk request will be passed to each ingest processor. However, the default implemention of batchExecute just operates on one document at a time, so unless my ingest processor is updated to leverage the new batchExecute method, I will see exactly the same behavior that existed previously.
[optional] Emit a deprecation warning if batch_size is specified in the _bulk API
[optional] Remove the functionality of the batch_size parameter in the _bulk API on main

Additional discussion exists on the original RFC starting here: https://github.com/opensearch-project/OpenSearch/issues/12457#issuecomment-2146287178

Makes a lot of sense, @andrross !  I support this.

+1 on the feature request.

Semantically speaking, because it's not a max, I think batch_size should be unspecified (null) by default, not Integer.MAX_VALUE. If a user specifies it in the API call, then use that.

@andrross A different default batch_size per analyzer makes sense. It's also possible that you'll want a max value per analyzer as well.

Thanks @chishui. Your suggestions above is exactly what I've had in mind here. I realize some of what I suggested above may be unclear. I never intended to suggest that the effective behavior of the sub-batching in the neural-search processors should change. The key point is that the sub-batching of documents into batches smaller than what was provided in the original _bulk request should happen inside the processors and not at the IngestService layer.

Another option is to set the parameter in AbstractProcessor

If this sub-batching behavior is a common pattern, we could certainly create an AbstractBatchingProcessor that defines a batch size parameter and even implements the logic to break the list from _bulk into smaller batches. I'd recommend starting with this pattern inside the neural-search plugin to be shared across its processors, and then we can promote the base class up to a common location if it proves useful and reusable.


request is not finished yet again！！！

this time is getFastDbClient
code:
orm::DbClientPtr db_client= app().getFastDbClient();
Mapper user::User mp(db_client);
auto query =mp.findFutureOne(Criteria(user::User::Cols::_account_name,CompareOperator::EQ,name));
auto user = query.get();
config:
"db_clients": [
{
//name: Name of the client,'default' by default
//"name":"",
//rdbms: Server type, postgresql,mysql or sqlite3, "postgresql" by default
"rdbms": "mysql",
//filename: Sqlite3 db file name
//"filename":"",
//host: Server address,localhost by default
"host": "<IP_ADDRESS>",
//port: Server port, 5432 by default
"port": 5306,
//dbname: Database name
"dbname": "user",
//user: 'postgres' by default
"user": "root",
//passwd: '' by default
"passwd": "123456",
//is_fast: false by default, if it is true, the client is faster but user can't call
//any synchronous interface of it.
"is_fast": true,
//client_encoding: The character set used by the client. it is empty string by default which
//means use the default character set.
//"client_encoding": "",
//number_of_connections: 1 by default, if the 'is_fast' is true, the number is the number of
//connections per IO thread, otherwise it is the total number of all connections.
"number_of_connections": 1
}
],
after call   auto user = query.get();  ,requst will be stucked in it;
this problem is not happen in no fast DbClient; just like those works well :
orm::DbClientPtr db_client= app().getDbClient();
Mapper user::User mp(db_client);
auto query =mp.findFutureOne(Criteria(user::User::Cols::_account_name,CompareOperator::EQ,name));
auto user = query.get();

For FastDbClient, the current thread must not be blocked to wait for the result, because the thread is also used to handle the database connections IO.
You should only use the async interfaces or coroutine interfaces of FastDbClient.

Please refer to wiki

ok,I will try

I call findone async method was ok ,I suggest use Async to divide the override may be better


Upgrade operator sdk to 0.9.0

https://github.com/operator-framework/operator-sdk/releases/tag/v0.9.0

@astefanutti do you mind having a look ?

@lburgazzoli yes, let me work on it.

It should fix #699 and #702, and it should be possible to remove vendor directory pre-build generation altogether:
https://github.com/apache/camel-k/blob/d33cc45d2e16d021bbeabe0522b916c5d197c508/script/Makefile#L159-L160


Generate waveform for audio files upon ingestion

Problem

Currently the audio waveforms are created upon request in the API. This has two effects:

The API must either cache the waveforms in it's own table or else recreate them upon every request
The API (which is open to the world) has an extra binary installed, audiowaveform which could present an unnecessary vulnerability

Description

From @AetherUnbound in a private chat:

I would like to see if it would be possible to generate them during ingestion. If that proves feasible, it has the added benefit of removing the audiowaveform dependency from the API. We can start populating the waveforms now in the catalog, and then swap over to the column in the API once we’ve backfilled everything.

Alternatives

Continue just creating the waveforms in the API and accept the two issues in the description.
Implementation

[ ] 🙋 I would be interested in implementing this feature.

Moved the issue to the API, since that's where the thumbnail service currently resides.

@krysal I'm going to move this out of the todo column, I don't think it's a realistic goal for the next two weeks and their might be some infra considerations to deal with first.


[Internal] Binary Encoding: Fixes Performance Regression

Pull Request Template
Description
Please include a summary of the change and which issue is fixed. Include samples if adding new API, and include relevant motivation and context. List any dependencies that are required for this change.
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Closing issues
To automatically close an issue: closes #IssueNumber

Follow Up PR created to track this work.


SEO for lazily-initialized instance variable documentation

Make the example for lazily-initialized instance variables stand out more by explicitly putting it in a module.
Also add the keywords "lazily initializing" for SEO.
Motivation

This technique is currently deeply-buried and not easily searchable.
Test plan

Thanks!

@jzhou-stripe @univerio this branch is super old. It will fail CI before it's rebased. Could you rebase it?


[Event Request] codeunit 419 "File Management" - OnBeforeBLOBExportWithEncoding

New Event in codeunit 419 "File Management"
    [IntegrationEvent(false, false)]
    local procedure OnBeforeBlobExportWithEncoding(var TempBlob: Codeunit "Temp Blob"; Name: Text; CommonDialog: Boolean; var IsHandled: Boolean; Encoding: TextEncoding)
    begin
    end;

    procedure BLOBExportWithEncoding(var TempBlob: Codeunit "Temp Blob"; Name: Text; CommonDialog: Boolean; Encoding: TextEncoding): Text
    var
        NVInStream: InStream;
        IsHandled: Boolean;
    begin
        IsHandled := false;
        OnBeforeBlobExportWithEncoding(TempBlob, Name, CommonDialog, IsHandled, Encoding);
        if IsHandled then
            exit;

        TempBlob.CreateInStream(NVInStream, Encoding);
        exit(BLOBExportLocal(NVInStream, Name, CommonDialog));
    end;

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


twitch-cli: init at 1.1.19

Description of changes
Initial commit for twitch cli.

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

@ofborg build twitch-cli

@ofborg build twitch-cli

@ofborg build twitch-cli

@ofborg build twitch-cli


Backend for campaign reports

Summary
Implements #368
Created CRUD endpoints for campaign reports. They are part of a separate module called campaign-report, but the controller is under the campaign base route.
New entities:

CampaignReport
CampaignReportFile

Both entities support soft delete and have an owner (uploader).
Screenshot of how the new routes look like:

Todo

[x] Endpoint for downloading files from reports
[x] Tests for update service
[ ] Tests for update controller
[ ] Tests for delete service
[ ] Tests for delete controller
[ ] Some manual testing of the whole process
[ ] Manual test of s3 upload/download
[ ] Squash migrations

@imilchev The PR check for scanning k8s manifests with mondo showed an error that you might be interested in:
Target:     K8s Manifest dev-manifests
✕ Errors:   rpc error: code = InvalidArgument desc = asset does not match any of the activated policies

@batebobo Can you please update your PR and resolve the open discussions in order to get this PR moving? :pray:

closing in favor of https://github.com/podkrepi-bg/api/pull/453


Installation issue on Ubuntu 14.04

I cloned the repo and followed the instructions as given in the README file, but I ran into the following errors:

make[1]: Entering directory /home/nain/Pictures/Personal/PID Controller/uWebSockets' if [ -d "/usr/lib64" ]; then cp libuWS.so /usr/lib64/; else cp libuWS.so /usr/lib/; fi cp: not writing through dangling symlink ‘/usr/lib/libuWS.so’ make[1]: *** [installLinux] Error 1 make[1]: Leaving directory /home/nain/Pictures/Personal/PID Controller/uWebSockets'
make: *** [install] Error 2

Please help!!

@AakashKumarNain I think the error log is not complete. Could you post the full log, so we can help.

Google is your friend. "not writing through dangling symlink"


Feature request: Put parameter

Use case
I'd like to be able to create/update a secret parameter's value.
Solution/User Experience
const ssmClient = new SSMClient({ region: "us-west-2" })
const putTokenCommand = new PutParameterCommand({
Name: secretName,
Type: "SecureString",
DataType: "text",
Value: JSON.stringify(secret),
Overwrite: true,
});
Alternative solutions
AWS SSM SDK

Acknowledgment

[X] This feature request meets Powertools for AWS Lambda (TypeScript) Tenets
[X] Should this be considered in other Powertools for AWS Lambda languages? i.e. Python, Java, and .NET

Future readers
Please react with 👍 and your use case to help us understand customer demand.

Hi @revmischa, thank you for opening this feature request.
I don't see any issue with adding this feature, especially considering that Powertools for AWS Lambda (Python) already has it as well.
In terms of DX, I think we should aim at something like this:
import { setParameter } from '@aws-lambda-powertools/parameters/ssm';

export const handler = async (): Promise<void> => {
  const parameterVersion = await setParameter('/my/parameter', {
    value: 'foo'
  });
};

The signature of the setParameter() function would look like this:
type ParameterType = 'String' | 'StringList' | 'SecureString';

type ParameterTier = 'Standard' | 'Advanced' | 'Intelligent-Tiering';

type SSMSetOptions = {
  /**
   * The parameter value
   */
  value: string;
  /**
   * If the parameter value should be overwritten
   * @default false
   */
  overwrite?: boolean;
  /**
   * The description of the parameter
   */
  description?: string;
  /**
   * Type of the parameter, can be one of `String`, `StringList`, or `SecureString`
   * @default `String`
   */
  parameterType?: ParameterType;
  /**
   * The parameter tier to use, can be one of `Standard`, `Advanced`, and `Intelligent-Tiering`
   * @default `Standard`
   */
  tier?: ParameterTier;
  /**
   * The KMS key id to use to encrypt the parameter
   */
  kmsKeyId?: string;
  /**
   * Additional options to pass to the AWS SDK v3 client
   */
  sdkOptions?: PutParameterCommandInput;
}

declare function setParameter(name: string, options: SSMSetOptions): string;

When it comes to implementation, the work needed to add this feature will need to touch several areas in the codebase, below an initial list (some might be missing):

Add a new setParameter() method to the SSMParameter class (here) - this method should get the inputs and call the AWS SDK method to put the parameter
Add types for the new method here
Add a new setParameter() function in a dedicated file similar to this
Export the new function in this barrel file
Add unit tests for the new feature
Add end-to-end test case for the new feature
Update docs similar to this

I'm putting the item on the backlog and opening it for contributions.

[!note]
For those interested in contributing, please leave a comment below so that we can assign the issue to you and make sure we don't duplicate efforts. Also, if you have any further questions please don't hesitate to ask here or on our Discord channel.
/

@dreamorosi I would like to take this issue :)

Hey @daschaa - sounds great!
Let me know if you have any question about the feature.
Here also is the Python implementation as reference. The implementation and structure on our side is very close to the one in that repo.

Hi @daschaa - hope you are doing well.
I wanted to check in with you to see if you needed any kind of support from our side or if you were still working on this.
If not, no issue at all, we understand life happens 😄

@dreamorosi Hey Andrea, unfortunately due to private reasons I was not able to work on OSS. I forgot to put in a comment, sorry about that!


TRSv2-576: Upgrade redis

Codecov Report
Patch and project coverage have no change.

Comparison is base (ce359d2) 55.12% compared to head (aee4a25) 55.12%.

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #373   +/-   ##
========================================
  Coverage    55.12%   55.12%
========================================
  Files          209      209
  Lines        12177    12177
  Branches      1588     1588
========================================
  Hits          6713     6713
  Misses        5464     5464

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


odpi/egeria-jupyter-notebooks#4 move jupyter notebooks to new repo

Signed-off-by: Nigel Jones<EMAIL_ADDRESS>
Description

Moves jupyter notebooks (and container build) to new repository at https://github.com/odpi/egeria-jupyter-notebooks
Related Issue(s)
odpi/egeria-jupyter-notebooks#4
Testing

New notebooks location has been tested. This removal is primary needing inspection and ensuring builds pass
Release Notes & Documentation

Will add reference in release notes once migration work complete
Additional notes

Opening as draft. Will only merge once we're happy with the new setup, planning for this release cycle.


a repetitive error I get in console.  potential listener LEAK detected

This keeps repeating over and over in the console.
event.ts:451  [8ce] potential listener LEAK detected, having 199 listeners already.


Remove CoSE-Bilkent layout

CoSE-Bilkent layout no longer works. Code relating to it should be removed, along with the dropdown list.

Whoops. You're right and just an accident on my part. We can keep working per repo because the kanban integrates on an organisational level.


ENH: Consistent axes and figsize parameters for plots

Problem Description
shap plots should consistently include axes and figsize parameters.
For example,
shap.plots.force(... , figsize=(20, 3), ...) includes figsize but not ax.
shap.plots.bar( ... ) does not include figsize or ax parameters.
shap.plots.scatter(..., ax=None, ...) includes ax but not figsize.
Alternative Solutions
No response
Additional Context
No response
Feature request checklist

[X] I have checked the issue tracker for duplicate issues.
[x] I'd be interested in making a PR to implement this feature

Workaround for changing the figsize of shap.plots.bar, for example:
shap.plots.bar(shap_values, max_display=8, show=False)
fig = matplotlib.pyplot.gcf()
fig.set_size_inches(2, 2)
plt.show()

@davidshumway Hi, thanks for bringing this issue up. There has been discussion on this before in the community fork dsgibbons#5 (at least, partially).
And I'm in favour of exposing the ax parameter to all of the plotting functions, in general. Then figsize would be made redundant I think -- if the user wanted to control the figsize they can create their own Axes on a figure with the right dimensions before sending it into shap.plots.
If you would like to raise a PR, I'm happy to work this through with you.
In the meantime I'm also refactoring the plotting functions in general (just FYI), but it shouldn't really affect the progress of your PR should you choose to raise it.

Having a consistent plotting API would be great. We could take some inspiration from other libraries like seaborn on the API design.

If the user wanted to control the figsize they can create their own Axes on a figure with the right dimensions before sending it into shap.plots

In this specific case I don't think the user would strictly need to pass the newly created Axes to shap.plots, as by default the plotting function would use plt.gca().
I would expect users would be able to use either of matplotlib's two interfaces: the imperative style or object-oriented style.
# Imperative "pyplot" interface
plt.figure("figure1", figsize=(10,10))
plots.plot_thing(data)

plt.figure("figure2", figsize=(5,5))
plots.plot_thing(data)```

Or alternatively:

```python
# Explicit object-oriented interface
f1, ax1 = plt.subplots(figsize=(10,10))
f2, ax2 = plt.sunplots(figsize=(5,5))

plots.plot_thing(data, ax=ax1)
plots.plot_thing(data, ax=ax2)

On a related note, we might want to follow matplotlib's recommendation and use the explicit API internally inside the plotting functions.


can't map file, errno=22 file '/Users/even/Documents/eBuGou/eBuGou/Lib/SDWebImage' for architecture i386

I download the source code of SDWebImage,and create a new project to build, and it runs succeed! But when i add the code to my project ,it report the errors of ‘can't map file, errno=22 file '/Users/even/Documents/eBuGou/eBuGou/Lib/SDWebImage' for architecture i386’ . Then I check the differences between the test project and my project，I find the same setting of Architectures and valid Architectures. So I don't know if some other configs may make this influence

did you follow these steps?
https://github.com/rs/SDWebImage#add-linker-flag


Update fsguicommondialog.cpp

Replace links:
http://www.ysflight.com > https://ysflight.org
http://download.en.ysflight.com > https://ysflight.org/download/
http://download.jp.ysflight.com > https://ysflight.org/download/

At this time YSCE does not have a proper domain, and the YSCEDC to my knowledge is not affiliated with YSFlight.org. As such, I cannot approve this pull request. We will approach this at a later date.


Please add me as member of opentelemetry-auto-instr-java

Dear friends!
I hereby request to be added as a member of this project. My sponsors are @trask and @tylerbenson
Thank you!

+1

Oh, wrong repo :-), need to open this in community repo

Oh bummer! :)


GCP activity log pollution when running chart-manager in GKE cluster

Is this a BUG REPORT or FEATURE REQUEST?: bug report

Uncomment only one, leave it on its own line:
/kind bug

What happened: every ~1s, a pair of configuration events are created in the GCP activity logs representing writes to the cert-manager-controller and ingress-shim-controller endpoints

What you expected to happen: events for writes to these resources to appear only when a new leader is elected, or at most once per lease duration (~15s)
How to reproduce it (as minimally and precisely as possible):

follow the Helm installation instructions to push the cert-manager chart to a GKE Tiller instance
watch as activity logs fill up

Environment:

Kubernetes version (use kubectl version):

Client Version: version.Info{Major:"1", Minor:"9", GitVersion:"v1.9.2", GitCommit:"5fa2db2bd46ac79e5e00a4e6ed24191080aa463b", GitTreeState:"clean", BuildDate:"2018-01-18T21:12:46Z", GoVersion:"go1.9.2", Compiler:"gc", Platform:"darwin/amd64"}
Server Version: version.Info{Major:"1", Minor:"8+", GitVersion:"v1.8.5-gke.0", GitCommit:"2c2a807131fa8708abc92f3513fe167126c8cce5", GitTreeState:"clean", BuildDate:"2017-12-19T20:05:45Z", GoVersion:"go1.8.3b4", Compiler:"gc", Platform:"linux/amd64"}

Cloud provider or hardware configuration**: Google Kubernetes Engine
Install tools:

$ helm version
Client: &version.Version{SemVer:"v2.8.0", GitCommit:"14af25f1de6832228539259b821949d20069a222", GitTreeState:"clean"}
Server: &version.Version{SemVer:"v2.8.0", GitCommit:"14af25f1de6832228539259b821949d20069a222", GitTreeState:"clean"}

Note that nowadays the leaderelection code allows using configmaps rather than endpoints for reasons discussed here.
Since this repo hasn't settled on being fully backwards compatible yet, now's the best time to switch over to locking on a configmap instead.

Yep, we should turn down the default leader election parameters.
@euank agreed re configmaps. We should aim to get this in for 0.3.

We should double check this issue is resolved after #327 - we've not reduced the frequency of updates, only the type of resource we use to perform leader election.

@munnerz I think the issue is still there.
I'm using cert-manager 0.4.1 with kubernetes 1.10.6.


Add new font: 3d-diagonal

https://ascii-generator.site/r/tnRpxH/
This looks like an interersting font to be added 3d-diagonal on ascii-generator.site

assign


docs/README: update tutorials' readme files to adapt to changes introduced by safe-app-nodejs API

r? @krishnaIndia
(maidsafe_highfive has picked a reviewer for you, use r? to override)

Thanks @hunterlester !


Compiler error under Linux

Under linux I get the following compiler error. Strangely enough it compiles fine under Windows. This seems to happen with Rust 1.71 and 1.72.
error[E0698]: type inside `async fn` body must be known in this context
   --> /cargo/registry/src/index.crates.io-6f17d22bba15001f/foxglove-ws-0.2.1/src/lib.rs:406:21
    |
406 |                     Ok(warp::reply::with_header(
    |                     ^^ cannot infer type for type parameter `E` declared on the enum `Result`
    |
note: the type is part of the `async fn` body because of this `await`
   --> /cargo/registry/src/index.crates.io-6f17d22bba15001f/foxglove-ws-0.2.1/src/lib.rs:413:44
    |
413 |         warp::serve(foxglove_ws).run(addr).await;
    |                                            ^^^^^

For more information about this error, try `rustc --explain E0698`.

Hey David. Thanks for the issue! I'll try to reproduce as soon as I can. We've been using this package quite a bit under Linux on x86-64 and arm64.

And indeed, I've been running it with 0.3.5 on one of the machines here.

Hey @Danvil, would you mind testing the change in #2? I did work for me. I now opted to explicitly mention the WebSocket sub-protocol name, rather than copying over the one from Foxglove.

Sounds good. Does indeed work with that version. Will bump the package on crates.io as well. Thanks again for the report.


Gml 1727 custom db port

Not related to this PR but I think type of credentials parameter of the get_db_connection_id_token method should be str, might be worth updating it here


test/acme/dns: poll interval and propagation limit configurable

What this PR does / why we need it:
This makes the DNS conformance test's poll interval and propagation limit configurable on the fixture object by test authors.
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Until now the poll interval was fixed to be 3 seconds and the propagation limit to be 2 minutes. Especially the latter value may be too short e.g. when waiting for the deletion of the created TXT record, leading to a test failure that can't really be fixed if the propagation limit can't be increased.
Special notes for your reviewer:
Release note:

This makes sense to me, thanks 😄
/ok-to-test
/release-note-none
/lgtm
/approve


Update .NET dependencies for LINQPad 6

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.1 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

This is related to the migration of a new set of .NET packages: https://github.com/dotnet/core/issues/7428.
Microsoft Reviewers: Open in CodeFlow

@wingetbot run


Collect jira issue type for new Vulnerability issue type.

Related to OSIDB-2980
Closes OSIDB-3240

This should ideally wait for merge of https://github.com/RedHatProductSecurity/osidb/pull/678 and then I'll rebase/rework it based on that.

@JakubFrejlach Any idea why tests worked in https://github.com/RedHatProductSecurity/osidb/pull/678 but the same tests don't work now? I didn't touch the tests edited in https://github.com/RedHatProductSecurity/osidb/pull/678, yet those are the tests that fail now.

@JakubFrejlach Any idea why tests worked in #678 but the same tests don't work now? I didn't touch the tests edited in #678, yet those are the tests that fail now.

@jsvob I was using the same cassetess for my two tests I've added in #678, you can reuse the cassetess:
https://github.com/RedHatProductSecurity/osidb/blob/master/collectors/jiraffe/tests/test_collectors.py#L443
https://github.com/RedHatProductSecurity/osidb/blob/master/collectors/jiraffe/tests/test_collectors.py#L411
https://github.com/RedHatProductSecurity/osidb/blob/master/collectors/jiraffe/tests/test_collectors.py#L388
I see you've also removed the cassete specification in the first test, I wanted to make it explicit when I've already made the cassetes shared between 3 tests

@JakubFrejlach Thank you. Turns out, I also made a second mistake: The collector now makes an extra request and that isn't in the original cassettes. To keep as much continuity as possible to show that the core function of the collector has not been compromised, I manually added the request&response for issuetype so that the original request in the original cassettes is left untouched. The new cassettes contain actual recorded traffic.

Thank you! LGTM now!


Use MUI components as selectors with styled()

Duplicates

[X] I have searched the existing issues

Summary
Following the documentation on the Components Selector API, using a regular styled component as a selector works but it is not possible to use MUI components as selectors.
Examples
https://codesandbox.io/p/devbox/peaceful-ptolemy-ytqqgt
import { styled } from "@mui/joy/styles";
import { Typography } from "@mui/joy";
import { css } from "@mui/system";
import "./App.css";

const Child = styled("div")(css`
  color: red;
`);

const Parent = styled("div")(
  () => css`
    ${Child} { /* This works */
      color: green;
    }

    ${Typography} { /* This breaks the component */
      color: blue;
    }
  `,
);

function App() {
  return (
    <div>
      <Parent>
        <Child>Green because I am inside a Parent</Child>
        <Typography>I should be blue because I am inside a Parent</Typography>
      </Parent>
      <Child>Red because I am not inside a Parent</Child>
      <Typography>Not blue because I am not inside a Parent</Typography>
    </div>
  );
}

export default App;

Motivation
Maybe I'm doing something wrong, but with this approach, MUI components should behave the same way other styled components do IMO.
Search keywords:

@CharlineCouchot

Please write like these (T is basic Typography component)

const Typography = styled(T)()

const Parent = styled("div")(
  () => css`
    ${Child} {
      color: green;
    }

    ${Typography} {
      color: blue;
    }
  `,
);

The emotion react supports emotion component nested selectors, so you just need to wrap customer React Element with it.

That solved it! Thank you :)
Would suggest maybe adding that to the Components Selector API documentation?

That solved it! Thank you :) Would suggest maybe adding that to the Components Selector API documentation?

I am not sure about that because styled function is powered by @emotion/styled. Probably we can give it to owner .

Would suggest maybe adding that to the Components Selector API documentation?

Maybe not. The exported MUI components aren't wrapped with the styled API, so it might not be necessary to add it to the Components Selector API documentation.


Usage of keywords nearer to Smalltalk Syntax

As a poweruser of AlgerNaN (and Squeak) I want to use Algernon keywords with the well known Smalltalk Syntax in order to have consistency between Algernon usage and coding.
Instead of  browse Morph I might want to type Morph browse in Algernon.

We have decided not to pursue this


[Fused] Add BatchToSpace, SpaceToBatch ops

Implemented new operations as fused ops according to OpenVINO specification.
Original framework: TensorFlow ((https://www.tensorflow.org/api_docs/python/tf/space_to_batch_nd, https://www.tensorflow.org/api_docs/python/tf/batch_to_space_nd)
Onnx doesn't support these operations, so onnx importer is not updated.

TESTNOW


feat(Numeric Type Casting): Allow trailing whitespace when casting bi…

…nary integers and floats
On occasion users of our app will accidentally type in a trailing space when dealing with numeric values which results in an "is invalid" error. I understand if you want to be extremely strict when parsing values but I believe this change is reasonable.

I'm not sure about this change. Thoughts @josevalim ?

In our phoenix app we have a custom plug that String.trims input params and I believe web layer is the right layer to perform such cleanup work.
I could swear that Phoenix.Controller.scrub_params/2 performed that cleanup in our app (again, it was a custom plug). @josevalim Maybe there could be scrub_params "user", trim: true which does that then?

Yup, we should probably move it up in the tack. If you want to discuss a trim option, please open up an issue on Phoenix.

@wojtekmach @josevalim That's fair was implementing a custom plug for this behavior and just thought it might make sense at the casting level.


http2.golang.org: remove references

https://downforeveryoneorjustme.com/http2.golang.org?proto=https

Mea culpa, it looks like this service was torn down in https://github.com/golang/go/issues/49301
However,

[ ] The domain is still active
[ ] The documentation at https://pkg.go.dev/golang.org/x/net/http2 references http2.golang.org.

Does registered here mean having a DNS record or something else?

There's two mentions in the source code as of right now:
git grep http2.golang.org
http2.go:// See https://http2.golang.org/ for a test server running this code.
transport_test.go:      transportHost = flag.String("transporthost", "http2.golang.org", "hostname to use for TestTransport")

@neild, @tombergan, any recommendations?  Seems like that test is either always failing or dead code.


Control the creation of sppf.png with a separate option.

Suggestion
A Lark Earley parser will generate "sppf.png" if self.debug is True, as seen in early.py:
https://github.com/lark-parser/lark/blob/master/lark/parsers/earley.py - search for "sppf.png"
However, this is an expensive operation that radically slows down the multiple test cases in my project. Lark would benefit from an option like generate_parse_graph to control this. For example,
		parser = lark.Lark.open(
			grammarPath,
			debug=True,
			generate_parse_graph=False,
			parser='earley',  # lalr, earley
			lexer='standard',
			postlex=CustomIndenter(),
			start='file_input',
			keep_all_tokens=True,
			maybe_placeholders=True,
			propagate_positions=True,
			ambiguity='resolve')
			#regex=True)

If the option will require that debug is True (in other words be dependent on it), then a better name might be debug_parse_graph.
Another idea: Make the option to control the file name of the .png file. If None, it doesn't get written.
Describe alternatives you've considered
Alternatives include:

Remove dot.exe from the path so pydot fails.
Hack the source code after pip install to comment out the generation.

Additional context
N/A

Why are you using debug=True? In general, we don't want to add additional options to Lark. Alternatively, you could uninstall pydot. If we really want to make this controllable, I would suggest using multiple levels, e.g. debug=2 or similar.

In general, I'm using debug because I'm in "development mode" instead of "release mode". More specifically, when I was using LALR instead of Earley, I was benefiting from the if self.debug goodies in compute_lalr1_states() and parse_from_state.
I still recommend an sppf_file option which can be True, a filename or something falsey. Or something similar.
Not sure why an sppf_file option shouldn't be added. It's not the type of option that bloats the code or adds more levels of indirection. What I'm trying to say is that it doesn't have complexity like parser, postlexer, etc.
For now, I have uninstalled pydot and continue to read the No pydot stderr message several times a day.

"development mode" instead of "release mode".

I would say you are in "test mode". Note that you should activate the logger in "test mode" in case warnings are generated. If you would have done that you would have realized that we are generating a lot of messages with debug=True.

Not sure why an sppf_file option shouldn't be added.

It isn't code bloat. It is API bloat. This is option is rarely used, and doesn't really provide new features, since you can get the same effect by deactivating debug. But it is another entry to an already long list of arguments.

It seems to me that if you're using debug=True, then you're in debug mode, which means you're trying to solve a problem with the parser. Once you've solved that problem, you are expected to disable debug mode, and run Lark normally. If any change that you make creates a new error, don't worry, Lark will let you know using an exception.
Why do you need to run Lark in debug=True?

Maybe the next step is official docs on what can be expected during debug=True?

Good point. Updated docs.

Thanks. I was thinking it could say what the extra info and warnings are. Like unused rules, shift/reduce conflicts, etc.

That is a little bit harder, since that is often changing. But it might be worth to try and create an always up-to-date list.


Unknown container state shows up green in the dashboard

I thought we fixed this @adamint ?

Repro is to delete the container using docker desktop and leave the dashboard running.
PS @karolz-ms @danegsta, shouldn't DCP auto recreate the container in this case? The object hasn't been deleted at the DCP level, just the backing container in the container runtime.

We had fixed it, it may have regressed. I will investigate

PS @karolz-ms @danegsta, shouldn't DCP auto recreate the container in this case? The object hasn't been deleted at the DCP level, just the backing container in the container runtime.

Maybe. Not sure. The concern is getting into an infinite loop where DCP is re-creating the container and something else is killing it.

Looks like it regressed here.

Couldn't be, this bug is 3 weeks old and that PR is 1 week ago 😄

Couldn't be, this bug is 3 weeks old and that PR is 1 week ago 😄

I totally misread and saw this bug as a week old. oops :)


Always handle POLLHUP in walredo error poll loop

Problem
test_stderr hangs on MacOS.
See https://neondb.slack.com/archives/C036U0GRMRB/p1698438997903919
Summary of changes
Always handle POLLHUP to prevent infinite loop.
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] If it is a core feature, I have added thorough tests.
[ ] Do we need to implement analytics? if so did you add the relevant metrics to the dashboard?
[ ] If this PR requires public announcement, mark it with /release-notes label and add several sentences in this section.

Checklist before merging

[ ] Do not forget to reformat commit message to not include the above checklist

Were we not expecting a test failure?


Add infra workflow

I've created a separate workflow for the creation of the Ably App & keys but I could also merge this with the existing workflow. Let me know what you think.


How do you get data for NYSE and NASDAQ indexes? (NYA and .IXIC)

What are you trying to do?

Get data for this index: https://www.google.com/finance/quote/.IXIC:INDEXNASDAQ
Get data for this index: https://www.google.com/finance/quote/NYA:INDEXNYSEGIS

what did you already try:

yf.Ticker("NASDAQ")
yf.Ticker("NYSE")
yf.Ticker(".IXIC")
yf.Ticker("IXIC")
yf.Ticker("NYA")

What did I expect to happen?
I don't want data on the symbols inside those exchanges. I want the index data. I expected this to work like any other ticker(see google links above). I searched through questions and documentation but i couldn't find anything on trying to get just the big composite indexes.

nq = yf.Ticker('^IXIC')
nq_data = nq.history(period='3mo')

@galashour thank you! Is the NYSE index the same? I thought it was odd this wasn't documented because i would assume people would want to get the nasdaq/sp500/dow/etc

didn't check the below with yahoo finance, but should work similar to the one above for nasdaq:
NYSE yahoo ticker seems to be '^NYA' or maybe '^XAX'
SP500 yahoo ticker seems to be '^SPX'
DOW index yahoo ticker seems to be '^DJI'
by the way, if you just want to track the trends, you can use the ETFs (e.g, SPY, DIA, etc.).


Support Microsoft.Extensions.DependencyInjection.Abstractions

It would be nice to be able to use existing infrastructure and just drop in stronginject as a replacement. For example, an existing application might depend on:

IServiceCollection
IServiceProvider
IServiceScope
Implementing these and other related interfaces might make adoption easier and faster

This is an area I definitely want to explore.
It's not as straightforward as it is for other dependency injectors, as StrongInject is entirely done at compile time, and is fully type checked - there's no build time (so IServiceCollection doesn't really make sense), and you can't resolve arbitrary types, only those that you specify at compile time you want to resolve - which makes IServiceProvider tricky to do.
I think therefore that it doesn't make much sense for StrongInject to provide implementations of these interfaces. Instead I think we should make it really simple to use StrongInject from a different provider of Microsoft.Extensions.DependencyInjection.
I haven't used Asp.NetCore much so I'm still not certain of the best way to do this. Any ideas/contributions would be greatly welcomed!

I've been experimenting with providing the following set of extension methods on IServiceCollection:
namespace StrongInject.Extensions.DependencyInjection
{
    public static class IServiceCollectionExtensions
    {
        public static void AddContainerForTransientService<TContainer, TService>(this IServiceCollection services) where TContainer : IContainer<TService> where TService : class;

        public static void AddContainerForTransientService<TService>(this IServiceCollection services, IContainer<TService> container) where TService : class;

        public static void AddContainerForScopedService<TContainer, TService>(this IServiceCollection services) where TContainer : IContainer<TService> where TService : class;

        public static void AddContainerForScopedService<TService>(this IServiceCollection services, IContainer<TService> container) where TService : class;

        public static void AddContainerForSingletonService<TContainer, TService>(this IServiceCollection services) where TContainer : IContainer<TService> where TService : class;

        public static void AddContainerForSingletonService<TService>(this IServiceCollection services, IContainer<TService> container) where TService : class;

        public static void AddScopedContainerForTransientService<TContainer, TService>(this IServiceCollection services) where TContainer : IContainer<TService> where TService : class;

        public static void AddScopedContainerForScopedService<TContainer, TService>(this IServiceCollection services) where TContainer : IContainer<TService> where TService : class;
    }
}

This will make it really easy to write a Container with StrongInject, and then use it from Microsoft.Extensions.DependencyInjection:

var container = new MyContainer();

services.AddContainerForTransientService(container);

You can also resolve IServiceProvider or anything registered outside the container by passing it into the container as a parameter.
public class MyApp
{
	public MyApp(IServiceProvider serviceProvider)...
}

[Register(typeof(MyApp))]
public class MyContainer : IContainer<MyApp>
{
    [Instance] private readonly IServiceProvider _serviceProvider;
    public MyContainer(IServiceProvider serviceProvider) => _serviceProvider = serviceProvider;
}

services.AddContainerForTransientService<MyContainer, MyApp>();

WDYT?
I would love to get feedback on:
Naming
Usability
What else do we need
Before publishing a preview API.

In order to get disposal to work properly, and to make sure we don't create multiple instances of a Container we'll have to do something like this:
using Microsoft.Extensions.DependencyInjection;
using Microsoft.Extensions.DependencyInjection.Extensions;

namespace StrongInject.Extensions.DependencyInjection
{
    public static class IServiceCollectionExtensions
    {
        public static void AddContainerForTransientService<TContainer, TService>(this IServiceCollection services) where TContainer : IContainer<TService> where TService : class
        {
            services.Replace(new ServiceDescriptor(typeof(TContainer), typeof(TContainer), ServiceLifetime.Singleton));
            services.AddTransient<Owned<TService>>(x => x.GetRequiredService<TContainer>().Resolve());
            services.AddTransient<TService>(x => x.GetRequiredService<Owned<TService>>().Value);
        }

        public static void AddContainerForTransientService<TService>(this IServiceCollection services, IContainer<TService> container) where TService : class
        {
            services.AddTransient<Owned<TService>>(x => container.Resolve());
            services.AddTransient<TService>(x => x.GetRequiredService<Owned<TService>>().Value);
        }
    }
}

Disposal still wouldn't be as tight as StrongInject usually is (Disposal at the end of each resolution), and will only be disposed at the end of each scope. However Microsoft.Extensions.DependencyInjection can't really do any better than this.

I've added an experimental package to help you integrate with IServiceCollection. More details here.
Please try it out and let me know what you think!

I'll give it a try tomorrow

I have now added a sample project at https://github.com/YairHalberstadt/stronginject/tree/master/Samples/AspNetCore


Cursor cannot be used correctly on start

Describe the bug
When i load the editor with a value in it the cursor does not react correctly.
Only after i type a letter the cursor does react correctly.
To Reproduce
const editor = new EasyMDE({element: '#editor', initialValue: this.value});

I made a video to show the behaviour: https://seafile.church.tools/f/f5e70a36c13e49a38aad/
i tried to click some times in the video on the bottom of the text.
Expected behavior
The cursor should behave correctly when loading the editor with an initial value.
Version information

OS: MacOS
Browser: Chrome 84
EasyMDE version: 2.11.0

Do you have the same behaviour on https://easymde.tk/ ?

no i have not.

we found out the problem was on our side.


pack data with missing value

explore scale_factor + add_offset packing where missing data exists in input

I have approved this change - and following Mark's advice I will squash and merge.


Compress user_data using gzip to avoid to exceed the hard limit (32KB)

Issue #, if available:
Description of changes:
userData has a limit of 32KB (http://docs.cloudstack.apache.org/projects/cloudstack-administration/en/4.8/virtual_machines/user-data.html)
When testing with two control planes, three etcd instances, and three worker nodes it failed to create the second control plane because the base64 encoded userData was over 32KB. However, CloudStack supports gzip compression for userData. To avoid too large userData we need to compress the userData using gzip before base64 encoding.
Testing performed:
Manual tests performed.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/lgtm


m2m for paraphrasing: sanity check

❓ Questions and Help
I'm trying to use M2M as a paraphraser for scoring (to see if this significantly better model improves Prism. This is complicated a bit because fairseq_cli.preprocess uses --source-lang and --target-lang to save the source and reference file. I think we have a workaround, but it's not doing as well as we expected, and I am wondering if someone can sanity check our approach.
Assume you have English files system.en and reference.en. I want to get token-level probabilities for P(ref | sys). First, I apply SPM to the raw text. Next, I binarize, using an irrelevant language code for the source:
PYTHONPATH=$FAIRSEQ python -m fairseq_cli.preprocess \
    --source-lang jabt --target-lang en \
    [snip]

Third, I translate, overwriting the source language code:
PYTHONPATH=$FAIRSEQ python -m fairseq_cli.generate \
    $datadir \
    --path $MODELDIR/12b_last_checkpoint_trimmed.pt \
    -s jabt -t en \
    --task translation_multi_simple_epoch \
    --decoder-langtok --encoder-langtok tgt \
    [snip]

From the argparse documentation in tasks/multilingual_translation.py, this seems correct, but as I mentioned, it's not doing as well as we'd expected, so I thought I would check.
This works fine and provides scores, but they do not work as well as we anticipated. So I am wondering if you see any issues with what we're doing (for example, perhaps the language code flag isn't doing what we expect?)

This is pretty cool and we would love to see if this works. (cc: @huihuifan)
I can't think of anything incorrect in your approach or commands at the moment. But I will need to give this more thought and get back to you if I think of something you could do differently for your use-case.


HDFS Source

As a user, I'd like to have files that are on HDFS as a source so that changes to the files, similar as the file source would be made available through streams.
Acceptance:

Module is available in spring repo
supported hdfs properties are documented in the reference guide
On first start the source should read all the documents and stream them

This issue was moved to spring-cloud-stream-app-starters/app-starters-release#31


Fixing pose error metric to match documentation.

Addresses issue #322 .
To review:

Double check the scoring math in modified lines of station_keeping_scoring_plugin.cc and wayfinding_scoring_plugin.cc to make sure it matches the Task Descriptions document.
Run the task tutorials for station_keeping, wayfinding and gymkhana and check that they are behaving as described in the documentation (especially that the topics are correct).

Note that there is still a problem with the gymkhana scoring, which is that it is orientation-sensitive but it should not be.  This will be addressed in a separate PR.

Math appears to be correct per description in issue #322 but I will wait to close out the review until I can validate against the Task Description document (need access to google drive).
Ran all three task tutorials.  General behavior appears to be correct, and scoring appears qualitatively consistent with description for station-keeping and wayfinding.  I found a few issues with topic names that most likely need to be corrected in the tutorials rather than the code.
Topic needs to be updated in stationkeeping task description.  Step 3 calls out:
/vrx/station_keeping/rms_error
New code is publishing on /vrx/station_keeping/mean_pose_error
I was unable to trigger anything on the two gymkhana debug/contact topics (assuming those are intended to indicate collisions?) - recommend describing better in the wiki what these are supposed to tell you, or removing the references in the wiki if these are not in use.
/vrx/gymkhana_blackbox/debug/contact
/vrx/gymkhana_channel/debug/contact
Typo in the gymkhana task:
"/vrx/gymkhana_blackbox/pose_error provides the root mean square error."
The current correct topic appears to be /vrx/gymkhana_blackbos/rms_error but I'm not sure if that will also be changing to mean_pose_error or a different name.

@j-herman Please try the new link to the Task Descriptions document. It changed this morning and I forgot to update the issue.

@j-herman Thanks for taking a look at this. Once you have access to the Task Descriptions document, please use that as the authoritative explanation. If the wiki disagrees with the documentation, the wiki probably needs to be updated (thanks for catching this!).
I went through quickly and corrected the problems you pointed out in the Wiki. I'm sure they could use a second look though. Thanks again!

@j-herman Thanks for catching it. I want to fix it in this PR. Should be fixed with the last commit (I also changed the element name to avoid confusion.)


samples: boards: nrf: nrfx_prs: extend regex to test default task

Verifies that at least SPIM is initialized properly.

I don't think this test is validating PRS properly. First, the regex does not verify TX==RX. Second, to fully test this functionality you would need to switch back and forth between SPIM and UARTE. Checking if SPIM has initialized properly does not give much value and may lead to an invalid assumption that PRS is covered in verification.

I agree. However that require more effort to extend this test.
My idea was to extend regex to catch errors which already occurs, so only quick win.
Proper verification of PRS needs to be evaluated separately.


Failing tests for numbers which containes two

Most of the tests that check numbers with '2' inside are failing, but the displayed answer is correct.

There is the problem left with uppercasing the first letter. Somehow it changes to Latin 'C' when operating with the word 'сто'.

Fixed in new version.


CI for Node

add new action workflow.

need code review.

● Game › nextPlayer › Sets the current player to be whoever it is not
● Game › Game › Starts the game with a random player
● Game › Game › Initializes with two players
● App › Contains the compiled JavaScript


Feature/flow 10

Feature(Visualization Plugin): New Ajax Email Rendering Chart
SUMMARY
The chart which renders the email on the Harmonized Email dashboard was previously implemented with an iframe plugin. This approach doesn’t work properly in Edge, likely because the request is missing the token on the header Authorization (with Bearer) or X-Auth-Request-Access-Token.  Solution was to create a new chart which performs an AJAX call to get the PNG of the email (using this Fission function, as the Iframe chart does currently).  Created the new visualization chart plugin for the Harmonized Email Dashboard which uses an ajax call to display the rendered email on the dashboard whether you are using Firefox, Edge, or Chrome.  This addresses tickets FLOW-10 and CLDN-2429.
Also changed SupersetClient.get() method to retrieve image data.
BEFORE SCREENSHOTS

AFTER SCREENSHOTS

TESTING INSTRUCTIONS
Currently the new AJAX visualization plugin is deployed on Superset stg.

Go to: https://superset-stg.hogwarts.pb.azure.chimera.cyber.gc.ca/superset/dashboard/55/?native_filters_key=lGAn1sRt1UmhBPIUT1h63yO4QrTOSZpCHKK0llY5FXQoYg3mwwTJdvVg_kMO_ucv
Select a single row from "Main View".
Right click on the row and select "Emit Principle Column Filters"
On the bottom switch to the "Render" tab

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[x] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[x] Introduces new feature or API
[ ] Removes existing feature or API

return buildQueryContext(modifiedFormData, (baseQueryObject) => [     {       ...baseQueryObject,       row_limit: 2
If I change row_limit to 1, then the error [ errorMessage = 'This chart can render only one email at a time. Narrow your search by adding filters so the query result is one record.' ] will never show up, since only 1 row will ever be returned, the data sent to the plugin will only ever contain 1 row of data.


Correct a typo in Duo pack auth_auth alias

Duo
Status

bugfix

Description
Correct a type (sending, not spending) in Duo pack auth_auth alias.
Checklist (tick everything that applies)

[ ] Metadata: pack.yaml, icon, structure (required for new packs)
[X] Version bump (required for changed packs)
[ ] Code linting (required, can be done after the PR checks)
[ ] Tests (not required but really recommended)

Thanks :)


[Bug]: Responsiveness for Recent User Slides

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the bug
Just like other devices, the cards for recent users should align with tab or 786px and above devices.
Expected behavior
At least one card should fit on the screen at a time thus would make it responsive.
Add ScreenShots

On which device are you experiencing this bug?
iPhone
Record

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

Thank you for creating this issue! 🎉 We'll look into it as soon as possible. In the meantime, please make sure to provide all the necessary details and context. If you have any questions or additional information, feel free to add them here. Your contributions are highly appreciated! 😊
You can also check our CONTRIBUTING.md for guidelines on contributing to this project.

Can you check the whole website and let me know what all things need fixing? I can give you a level3 for that

Sure, would need a day or two, I'll craft the fixes as well so that the issue and PR are added on the same day.

@ashutosh-rath02 Sharing an update
1. Responsiveness for tab-based devices checked, only the Recent User Slides would undergo a fix.
2. Dark Mode: The Hyperlinks are supposed to turn to "text-primary" on hover as they do in Light Mode, but currently not happening in dark mode.*

This fix is required in both Navbar as well the icons provided in the footer. Sharing you the results after fixes:

3. Light Mode: The Labels provided in profile tracking are merged with the background in the light mode, they are fixed and can be seen as follows:

4. Made by Ashutosh Rath: The name in this phrase in the footer can be made a hyperlink to your portfolio website.
5. Secondary Links in the Footer: Links such as "Contribute" and "Vote on Product Hunt" should also get covered with hover effect.
Suggestions and Features

The Generate button in dark mode on Form.tsx can have its text color white, as it looks better in contrast to the gray color.

The Graphs generated on the profile tracking option can be given an option to be saved/downloaded as image.

Auto Complete feature can be added to the Form.tsx while searching for a github username. NOTE: If we directly fetch usernames from github the increase in number of operations can affect website performance, thus we can have a discussion with other contributors regarding the most optimal approach for this feature.

Makes sense go ahead, auto complete is not required.

Hello @kshavp! Your issue #285 has been closed. Thank you for your contribution!