fix: overflow of post-nav-toc

Scrollbar is displayed because the value of overflow-y is scroll.
https://github.com/meteorlxy/vuepress-theme-meteorlxy/blob/c7e83222923227a8f0e4e9a718ceadb9f4d20362/lib/components/PostNavCard.vue#L119-L121
Also, overflow is available for most browsers.
So I think that it is not a problem to put them together.
(ref: Can I use - css-overflow)
Reference
MDN: overflow, overflow-y, -ms-overflow-style

This was recently introduced in #49.
I only checked that PR in Chrome. I thought it may go wrong in other browsers but I didn't have much time to comfirm that (my fault 😓 )
The table of content could be so long and overflow: hidden is not suitable for that.
overflow-y: auto could be the solution.

I think that you have the right idea!
I thought just now.
That overflow: auto should be applied to .post-nav-card instead of .post-nav-toc.
If this is the case, a scroll bar will be displayed above Comments.
Case .post-nav-toc:

Case .post-nav-card:

Are you busy? Please get plenty of rest😄
(I do not understand English, it may be wrong...💦)

Busy on weekdays. And will be busy for next two weekends (sad 😢).
English is not my first language, too. We are both from east asian culture. 😄
Move overflow: auto into .post-nav-card cause an issue of vertical axis:

It would be nice if you could help to find a good solution for this issue, being compatible will most of the browsers.

[ ] Chrome
[ ] Firefox
[ ] Safari (if possible)

Chrome 75.0.3770.100:

Firefox 68.0.1:

Merged
If you have any ideas, feel free to open another issue / PR 😄

It's a great that design✨
I have a lot to learn.
Thanks for produce a great theme!


Still memory issue is not fixed.

Which version of PhantomJS are you using? Tip: run phantomjs --version.
2.1.1

What steps will reproduce the problem?
1.Upload a file more than 1GB using phantomJs

Which operating system are you using?
Linux Redhat

Did you use binary PhantomJS or did you compile it from source?
Binary

Please provide any additional information below.

((PhantomJSDriver) driver) 						.executePhantomJS("var page = this; var p=page.clearMemoryCache();console.log('memory Cleared');");
I have been clearing the memory frequently and still phantomjs is crashing.

We cover this issue in https://github.com/ariya/phantomjs/issues/11390

When this will get released?

When it's ready.

Will it available in 2.5 ?

2.5 improved memory consumption mechanisms and it consumes less memory than 2.1 or 1.9. But we still have job to do.

Good to hear that.
2.5 Beta is asking lot of dependencies like libjpeg.so.8 etc.
I'm using RedHat 7 and have latest libjpeg.so. So Can I build the binary which has all the dependecies?
So that user don't have to install anything extra.:-)


Show package loading status

As described in issue openspending/openspending#1118 except on one thing: progress will not be shown, because for this I need to load entire resource datafile and count lines in it (API does not return % of processed rows; in Packager we already have datafile in memory, and there is only one file - Admin may check for multiple files, so it will lead to much more memory and time consumption than in Packager).

@kravets-levko perhaps we can embed the # of rows as part of the resource descriptor? We can add this in the packager and read this here.

Yes, we can, I think. But then what about packages uploaded by other tools (don't know, - there is python cli AFAIK, maybe something else)?

I think, we can just show a number of processed rows (returned by API): user will see that app is doing something - it's better than nothing, IMHO.

If there's no 'total rows' indicator then we could fallback to simply not showing progress (or showing the '# of processed lines' message as you suggested)

Okay, got it. Will implement it tomorrow

@akariv Added progress, as we decided. Also please look at this PR: openspending/os-packager/pull/102


Grid issue

@akashnimare Check out the attached image for grid correction.

Fixed. 5-10 px margin is still there.


pmgen: Have a single make pattern

Remove duplicate %.pmg -> %_pm.h pattern. One of the duplicates overrode the other, and in some conditions there were build races as to whether the target directory for the generated header would exist. Instead have a single rule which is properly generalized.

@mmicko One unfortunate issue is that, I think, the tagged release wouldn't build in YoWASP without a custom patch or something...

You could roll the dice again on the race :|

The one CI failure appears to be a fluke. Merging.


658 add: webpack-dashboard plugin

Resolve #658

@spmcbride1201 do you want me to research this further?

@karinakeen. Nope, unless you have strong feelings that you can get it working better, I think we can just close this out and chalk this up to getting you acquainted with the project's webpack configs. Feel free to close the PR and wipe the branch.

@spmcbride1201  Not sure if you already checked the tabs?

@karinakeen - Nope, I guess I was missing something obvious. Let me look again.


Using the downgradeCFF2 parameter of instantiateVariableFont generates an unusable font

Hi, and thanks a lot for your work on CFF2 support for instantiateVariableFont! ❤
Here’s a small script I’ve used to instantiate a variable font with CFF2 table, and get a font with downgraded CFF table:
from fontTools.ttLib import TTFont
from fontTools.varLib.instancer import instantiateVariableFont

input_font = TTFont('Cantarell-VF.otf')
axis = {'wght': 800}
output_font = instantiateVariableFont(input_font, axis, downgradeCFF2=True)
output_font.save('output.otf')

The output font seems to to be unusable with different tools, browsers for example. Setting downgradeCFF2 to False generates a valid font, but with a CFF2 table (as expected). Did I do something wrong?
Here are the original font and the broken output font: fonts.zip
(One minor problem in documentation: the varfont parameter documentation says that CFF2 table is not supported, which is not true anymore as far as I can tell :smile:.)

You're not doing anything wrong. The downgrading indeed produces broken fonts sometimes (all the times?!) I'm still scratching my head on it. For me, it produces a font that ots-sanitize and HarfBuzz are happy with, but not ttx or svgPathPen.
It seems to have to do with the charset.  I'll test yours and see if it gives me any clues.

(One minor problem in documentation: the varfont parameter documentation says that CFF2 table is not supported, which is not true anymore as far as I can tell 😄.)

Fixed. Thanks.

It seems to have to do with the charset. I'll test yours and see if it gives me any clues.

Yours also passes ots-sanitize and works with hb-view. But yeah, ftview gives "broken file".
@khaledhosny in case you have time to help me with this.

Closing in favor of https://github.com/fonttools/fonttools/issues/3521

@khaledhosny in case you have time to help me with this.

Not really. I don’t remember much about CFF anyway.


 I have a Zigbee2MQTT USB-Stick in my server and also dockers running a Mosquitto MQTT server and another docker running Home Assistant. The Zigbee2MQTT-usb sends everything to the MQTT server and they show up in Home Assistant in Integrations -> MQTT.
In the Mosquitto log I see:
1605812716: New connection from <IP_ADDRESS> on port 1883.
1605812716: New client connected from <IP_ADDRESS> as PeopleCounterESP32Client-a164 (p2, c1, k15).

But how do I make the device show up in Home Assistant ?
I tried Adding the :
sensor:
    - platform: mqtt
      name: "People counter"
      state_topic: "peopleCounter/serialdata/tx"

But I get no new devices with or without the sensor added to config.yaml

Have you restarted Home Assistant?


Filter/Toggle images in search

Add the ability to show only images as you can't currently easily find images.

Not sure how exactly this should look like and behave.
I don't really have time to invest in this now, but PRs are always welcome!

I think Alfred's approach is quite intuitive: they show image entries as Image: <width>x<height> (<size>) in the clipboard item list (for files they use a similar format File: <filename>). Therefore, they can be searched with Image or File, just like text items.

I am trying to implement a way to filter based on the type and I can't seem to find a good UI/UX for it. I'd like to share a few examples and hopefully, you can react with 👍 / 👎 if you like or don't like the solution.
In this example, I use segmented control where segments show or hide Files / Images / Text.

Another possible solution would be to have one button which shows a popover on click. This popover would have 3 checkboxes for Files / Images / Text.

I like the first option, it clearly shows the current filter state. Unfortunately, it makes the search bar header a bit busier - I’m ok with that though. If it becomes too busy the toggles could always be moved to underneath the search bar.
The second option is a cleaner approach but requires more clicks to toggle states and state is not obvious.

I lean towards implementing something like this. In this approach it would:

By default show all types of data.
Click on a single type would hide/show this data.
Click on a single type with pressed Option would show ONLY this data.
It should be possible to switch to the control by pressing "Tab" and then using the left/right arrows to select type and Space (Option-Space) to simulate clicking on the button.

@p0deje dose the feature develop done?

No, otherwise it would have been closed. On Jul 16, 2023, at 18:39, justdoit @.***> wrote:
@p0deje dose the feature develop done?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

i think the feature is very useful for me。when to release the feature？ thxs!!!

IMHO, the specific filter interface is very nice buy overkill.
Simply typing "image" in the search field would be amazing, like in Alfred on my other Mac :)
This could actually apply to other filters, e.g. "file", "text". It could be made smarter with prefixes like "type:image", it's pretty standard, including here on GH.


[Masthead v2]: Apply the new design specs/token changes to header bar for all breakpoints

Description

Apply the new design specs/token changes to the Header bar. See below for the list of areas to be adjusted.

IBM logo
Cluster Name
header__menu-item
header__menu-title
header__menu-sub-item

These changes should be applied to all breakpoints.

Additional information

Overview and scope of the engineering work should be included in the linked epic (#9208)

Design Specs

Desktop - https://www.figma.com/file/35JfN0Sg4ltIIjM7jBDKMk/Navigation?node-id=1970%3A110816
Mobile - https://www.figma.com/file/35JfN0Sg4ltIIjM7jBDKMk/Navigation?node-id=1970%3A111059

Acceptance criteria

[ ] Changes should align with the specs outlined by the design team

@proeung The old Cloud specs that are located in this box folder, show a different type style than was actually implemented.

body-short-01 = 14px was originally proposed and is in the old Cloud spec
body-short-02 = 16px is what was implemented within the current Cloud masthead to align with the Global masthead

For this first round, we agreed to keep the L0 at body-short-02
I believe nothing has changed within the items outlined in this issue:

IBM logo
Cluster Name (also known as Ecosystem name or Product name)
header__menu-item
header__menu-title
header__menu-sub-item

We can likely close this issue 🙂

@oliviaflory Thanks for taking a look at this issue. There are a couple of things I want to confirm before we decide to close out this issue or adjust the description based on what needed to be changed. cc:

Looking at the current specs, I see that the menu items should be set to body-short-01 (see link and attachment), however, the current Cloud Masthead is using body-short-02. We certainly adjust this to body-short-01 easily for this round, if this is something that you and the design team would like to have. It shouldn't be an issue at all to make this switch since we're working off from the <dds-cloud-masthead-container> component.

The only difference I have found is at the small breakpoint the Cluster name will be hidden in L0 and will only be present within the hamburger menu!

Can you send me the link to the Figma layer with this change? Currently, for the Cloud Masthead, we have the Cluster name/Platform name visible on all breakpoints within the header bar.

Looking at the current design for both the Masthead v2 and Cloud, the optional CTA will need to be adjusted to use the ghost button style (see link). I think this adjustment can be wrapped into this issue or this one (https://github.com/carbon-design-system/carbon-for-ibm-dotcom/issues/9211).

@proeung -

The adjustment to body-short-01 is the future state with the update to the L1 so that everything is consistent.
Based on my understanding, the cluster / ecosystem / platform name should be hidden within the header bar to make space for the optional items within the utility area of the navigation. Here is a link: https://www.figma.com/file/35JfN0Sg4ltIIjM7jBDKMk/Navigation?node-id=2522%3A147565 that illustrates this.
Yes, the optional CTA will be updated to the ghost button style.

I am working on updates to the file in the background to use the new specs since most of the frames are imports from previous sketch files.

@proeung One thing we need to take into consideration after the Navigation Stakeholder and Dev call today and hearing about the plan to have the masthead be v2 and use Carbon v11, is that all of the specs are aligned to the Carbon v10 color and type tokens.
There is documentation about how to map the color tokens and type tokens in the v11 documentation, but it would be helpful to know the developers timeline for moving from v10 to v11.

@oliviaflory Great point and thanks for bringing this up! For the October 10th launch date of the Masthead L0, we're not planning on pulling in the Carbon v11 upgrade, since this is a major release that requires a significant amount of updating/migration on the code level to align with the new tokens.
Based on my conversation with @kennylam we're going to be merging C4IBM v2 changes into the feat/masthead-v2 once we have resolved any bugs/issues and this work would mostly come during Q4 or early next year. Overall, it's still good for all of the devs to be aware of this imminent change.

@JaniceChristineM Thanks for getting back to me. We'll keep a note that the adjustment to use body-short-01 for the menu items will be part of the update/rollout of the L1 nav. I'll create a new issue for that so that we have something to track, but let me know if there are other treatments we need to account for as part of the secondary rollout/update.
cc: @kjwhitla @andy-blum @jkaeser


Checking if there are no errors?

Hi,
i am currently using this code to add class if there are errors
:class="{ 'is-danger': errors.has('email') }"

Maybe I have missed it in the documentation but is there a method to check if there is no error?
i.e I want
:class="{ 'is-danger': errors.has('email'), 'is-success : errors.none('email')}"

Success means it has validated the input and there are no errors, so you have 2 checks to do. The easiest way is to use flags:
http://vee-validate.logaretm.com/examples#flags-example
so it would look like this:
:class="{ 'is-danger': errors.has('email'), 'is-success : fields.email && fields.email.valid && fields.email.validated }"


i18n(ko-KR): update api-reference.mdx

Description (required)

update api-reference.mdx
#8946

Related issues & labels (optional)

Closes #
Suggested label:

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

ko
reference/api-reference.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


Enchantments draining lots of both charges and levels

I'm playing with HxCEnchants v2.3.4 and HxCCore v1.12.4.
I have discovered a lot of draining of both charges and XP levels.
The charges are draining every second even if there is no enchantment that is actively used.
The levels are also being drained but not every second. That seems to happen every 30 to 60 seconds or so. But then more levels at once (somewhat between 10 to 30 every time) are stripped.
That prize is way too high for the enchantments.
I would suggest to handle enchantment in the following way:
There should be a short counter (30 to 60 seconds) and a long counter (90 to 180 seconds).
Enchantments that are not always actively used should drain the defined amount of charge every time the short counter is counted down. The short counter starts to count down at the beginning of using that enchantment and cannot be stopped until it has counted down.
Enchantments that are always actively used should drain the defined amount of charge every time the long counter is counted down.
Level draining should not occur until an enchanted item has no charges left and only if there is a config option within the config file that is activated to allow that to happen.
Also Enchantments that can be added to more than one item should either be cumulated (example: Repair level 5 on all armor parts = up to 20 damage points that can be repaired within the given time) or they should reduce the required charge with the following formula while the effect stays the same: defined amount of Charge / amount of items with the specific Enchantment.
What do you think about this?
Oh, and FYI:
I for myself prefer to use the blocks for both adding enchantments and adding charge. They may have some little issues but nothing that urges me to get rid of them. For me it is easier and more convenient to use them.

I have no idea what would be causing the xp drainage other than the shifting, I have no code that subtracts from xp, excluding the charger and shift function

BTW I already have a delayed drainage for most the enchants, it's a configurable update timer default updates 5x slower than redstone I think.

Auras, armor enchants, repair and regen are on the delayed player tick I made. feather fall, meteor fall, swiftness, vitality, and fly, aren't on delayed tick.

Okay your post was long, I never read the whole thing....
Repair is per breakable item, and they ALL tick at 1 time, but repair repairs only the item it is on. so lvl 5 repairs 5 durability and it's on the delayed tick.
Vitality only counts on the chestplate.
Regen does stack for all armor, so lvl 5 on each armor should in theory regen 5-10 hearts every 10 seconds. I don't remember the math I have behind that. thinking of making it a formula.
Charge is charged to an item when it is held in hand and the keybind is pressed (now).
The charge required for every enchant is based on when it is triggered and is configurable, thus aura enchants, flight, swift, vitality, are all draining constantly. (well aura only when entities other than player is in range, or blocks for some).
I believe your xp drain was due to holding shift and not realising it. because in that version xp drained is by shifting with held item is enchanted (tinker tools are almost always "enchanted" by game code).


TOBB Ekonomi ve Teknoloji Üniversitesi

@heckedbyfurki I am sorry, the domain etu.edu.tr is removed from our trusted domains list because it was abused in the past. If you are the student of this school, please enroll to GitHub Student Developer Program. Then apply for JetBrains educational subscription with your GitHub account (GitHub tab at https://www.jetbrains.com/shop/eform/students).


Make file updated with multi build support

Packer support for local make commands which can do builds for various os and architecture combinations, manipulate scripts/build.sh for the same.


Jaygambetta qobj bug

Summary
Details and comments

sigh this isn't going to work yet I don't think. It fixes the skip exception for the test ibmq qasm simulator module, but that breaks skips for the tests not using testtools (which is everything else). I think I'm just going to have to monkeypatch the skipexception in testtools to workaround the bug there. We'll see what travis says though

Ok, this should be ready now. Monkey patching worked around the bug in testtools so we can use the 2 different skip exceptions.


Image map inside bootstrap column

My image map is placed inside a portlet, which is put inside a bootstrap column. How can I change the ratio between the map and the window  into the ratio between the map and the image? I need this because the image map size should depend on the size of the image, which depends on the size of the bootstrap column, not on the size of the window.
MY HTML CODE:

                Lichaam
                Man

            <map name="bodymap">
                <area shape="poly" coords="875,143,880,122,867,75,844,38,819,22,780,6,754,1,716,2,668,21,629,56,610,101,608,135,618,185,597,162,587,164,586,201,611,272,626,268,633,303,665,338,691,372,734,389,776,381,810,348,847,307,856,268,869,271,891,226,897,184,892,163,873,176,863,183" href="#">
            </map>

    !-- END PORTLET-->

Unless the column resizes independently of the Window, then it should just work.


Update Test-DbaMaxDop.ps1

Update to MaxDOP calculation on Test-DbaMaxDop
Update to MaxDOP formula for SQL2016+, as per last review of KB2806535

Type of Change

[ ] Bug fix (non-breaking change, fixes # )
[X] New feature (non-breaking change, adds functionality, fixes #6341)
[ ] Breaking change (effects multiple commands or functionality, fixes # )
[ ] Ran manual Pester test and has passed (`.\tests\manual.pester.ps1)
[ ] Adding code coverage to existing functionality
[ ] Pester test is included
[ ] If new file reference added for test, has is been added to github.com/sqlcollaborative/appveyor-lab ?
[ ] Nunit test is included
[ ] Documentation
[ ] Build system

Purpose
This PR updates the formula used for calculation of recommended MaxDOP, as per last review of KB2806535 (https://support.microsoft.com/en-us/help/2806535/recommendations-and-guidelines-for-the-max-degree-of-parallelism-confi).
Actually, for SQL2016+ the MaxDOP value can go up to 16, if multiple NUMA nodes are detected.
Approach
Simply added an IF condition, checking for instance version, if more than a single NUMA node is detected
Commands to test

Screenshots

Learning

Hey @OmegaMadLab ! Thanks so much for the PR, my apologies for the delayed review. I'll take a look today.

This looks great - thank you for providing the reasoning and link 💯

It was a pleasure! I appreciate so much the module and the work the team is doing! I hope this one will be the first of my contributions!

Looking forward to it! Your changes are now in the PowerShell Gallery 🎉


Reset back to cached-property

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Closes #11.

After conda-forge/cached-property-feedstock#14 was merged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


HPCC-12753 Correct RPM version and release

Enable creating rpm.
For CentOS 6 need turn off system boost as : WITH_SYSTEM_BOOST=0
Also if /usr/lib64 used, for example CentOS, set mozilla plugin library directory to
/usr/lib64/mozilla/plugins/
Also correct rpm package version as JIRA 12157 for platform:

change "_community" to "-community" in package file name
package version .. instead of "community"

@GordonSmith please help to review

Looks good.


Streaming to AllDigital broken. Bisected to commit 7dd4cfefbe68ac66c4b846d115c4355ebdbdf8ec

VideoCore cannot stream to AllDigital servers. I have bisected the issue.
The SampleBroadcaster sample project was able to stream successfully to AllDigital servers until commit 7dd4cfefbe68ac66c4b846d115c4355ebdbdf8ec by James Hurley on Tue Dec 9 11:05:32 2014 with message use first section between slashes as appName and the rest as playPath (appInstance). This issue referenced the wrong commit before I edited it.
On and after that commit, VideoCore fails to stream to AllDigital servers.
Here is the log output of current master failing to stream:
2016-01-25 16:12:04.537 SampleBroadcaster[570:55801] Screen rect:{{0, 0}, {320, 480}}
2016-01-25 16:12:04.544 SampleBroadcaster[570:55801] Creating context
Connecting:ingest.alldigital.net:1935, stream name:00000001/test10?key=b748e428&rec=none&name=time
ClientState: 1
ClientState: 2
ClientState: 3
Want read:4096, read:1448
ClientState: 4
Not enough s1 size
Want read:2648, read:1625
ClientState: 5
ClientState: 6
Tracking command(1, connect)
Want read:4096, read:340
Steam in buffer size:340
First byte:0x2, header type:0
Handle message:5
Received server window size: 2500000
Steam in buffer size:324
First byte:0x2, header type:0
Handle message:6
Received peer bandwidth limit: 2500000 type: 2
Steam in buffer size:307
First byte:0x2, header type:0
Handle message:4
Received ping, sending pong.
Steam in buffer size:289
First byte:0x2, header type:0
Handle message:1
Request to change incoming chunk size from 128 -> 512
Steam in buffer size:273
First byte:0x3, header type:0
Handle message:20
Received invoke
Received invoke _result
Find command: connect for ID:1
Tracking command(2, releaseStream)
Tracking command(3, FCPublish)
Tracking command(4, createStream)
ClientState: 7
send pong
Want read:4096, read:41
Steam in buffer size:41
First byte:0x3, header type:0
Handle message:20
Received invoke
Received invoke _result
Find command: createStream for ID:4
Tracking command(5, publish)
ClientState: 8
Want read:4096, read:148
Steam in buffer size:148
First byte:0x3, header type:0
Handle message:20
Received invoke
Received invoke onStatus
code : NetConnection.Connect.Rejected
Want read:4096, read:0
Read from stream error:0
ClientState: 11
~RTMPSession```

@SterlingOnLoop ..what you are doing to that issue.. I am also not able to stream?

reverting that one commit seems to work

can you show me the podfile you are using or the procedure you are using to use video core as I am facing the issue?

can you please let me know how did you actually use the video core library. I am just trying to use it through cocoapod which restricting me to use the 0.1.12 version. If you could give me steps to use it the way you are using it, I would be grateful to you. Thanks in advance

I suggest you 1. perform a git checkout, 2. open the file "rtmp/RTMPSession.cpp", 3. do the opposite of this.

I did not understand the third point? can you please elaborate?

Thanks I resolved the issue. There is another issue which I want to ask, I am using 0.1.12 in cocoapod and now in another location I pullout github and now I made this to work, how to use this updated code in my previous example which has 0.1.12 cocoapod. (Remember when I update to 0.3.2 it does not work)

@satyamub Hello , I have encountered this problem , I ask how to solve it. Thanks in advance.

Please leave me the email address!!

Thank you, my email address is<EMAIL_ADDRESS>


Causing Launcher To Not Work

I downloaded this mod and added it to my mod folder (Fabric) and every time i launch the game it loads a little bit then closes the launcher and doesn't open the game

You need to give us more context if we want to even try to debug this problem. Please send us your latest.log file from a time when this issue occured
Cheers

Given that this issue has now been stale for almost 2 months, I'll go ahead and close it. If you still think the problem is relevant, feel free to provide the required information and reopen it
Cheers


[Incubator][VC] apply leaderelection feature to vc-manager

fixes #466

applying the leaderelection feature of controller-runtime to
vc-manager
change the default number of replicas to 2
don't set ownerreference for cross-namespace objects

/lgtm
/approve


SAK-32184 Remove redundant -pack projects.

Make most -impl projects of type sakai-component which means we have
less modules in the build and our build is smaller.

Don't merge, just here for reference at the moment.

+1 for huge cleanup projects

I'd like to get this merged or closed as it keeps breaking, if anyone has any comments or wants to review could you do it soon. Thanks.


Wait for Minecraft Update...

Wait For Plazma Update

Plazma: NOT UPDATED!
Leaves: Merging Pull...
Icecream: Updated while Not Ready
Gale: Updated While Not Ready

this update has requested by @joaoespanhol
We add folia to the project, as a result, we act faster by patching plazmamc to package 1_21_r1 until it removes, and fix the patches more and more and more. have fun compiling!

Issue Created. Standby...

Deadline is July 20, 2024. if i cant update then our beloved panda fu bao will be disappointed

the deadline, which was Fu Bao's Birthday has been extended as Paper will stay on Unstable.
Sorry, my Baby, Fu bao...
:panda_face:

we will now close this issue with Picture of Fu Bao

Kang Bao will be disappointed.

플라즈마가 나오지 않아서 이제 업뎃이 지연됐는데,
우리는 푸바오에게 이 업뎃이라는 선물을 주고 싶었습니다.
아이고 미안해라 푸바오 ㅠㅠ

Radioactive will never update for flippin 2 month
Oh well,gotta focus on NogyangSpigot


check commit

this is a test.

Hello @Franko1307! Thanks for submitting the PR.

In the file cli/popper/commands/cmd_run.py, following are the PEP8 issues :

Line 73:1: E101 indentation contains mixed spaces and tabs
Line 73:1: W191 indentation contains tabs
Line 75:1: E101 indentation contains mixed spaces and tabs


Change state of comment back to moderation after comment got updated

Quick summary
I just found out that after an already published comment from activitybub is changed the changed comment just is there without any oversight or chance to check it out before it is published now.
Anyone could put anything on my blog after a comment got approved and that is bad.
Steps to reproduce

comment via Activitypub on an article
waqit till comment is approved
change comment
See changed comment without further check by the site owner

What you expected to happen
I'd expect that the comment get back in moderations state.
What actually happened
The comment just changed,
Impact
All
Available workarounds?
No but the platform is still usable
Logs or notes
No response

I consider this bug / missing feature a big moderation issue.
As long as it's not fixed, I'm not using this (otherwise great) plugin.

It looks like we could leverage the following hook to temporarily work around this:
https://github.com/Automattic/wordpress-activitypub/blob/4c297acfd454a13260900ff0f97a9e3f425e960c/includes/handler/class-update.php#L76
Except I'm not sure what the 2nd parameter even is? Why is it null? This seems to be the only time this action is called?
So, untested, but you could try putting something like this in a mu-plugin or your theme's functions.php?
add_action( 'activitypub_handled_update', function( $activity, $second_param, $state, $reaction ) {
	if ( $reaction instanceof \WP_Comment ) {
		wp_set_comment_status( $reaction, 'hold' );
	}
}, 10, 4 );

Note that this wouldn't actually check if anything got changed (yet)!

So, that didn't work.
In fact, this whole bit of code is kinda strange:
		$state    = Interactions::update_comment( $activity );
		$reaction = null;

		if ( $state && ! \is_wp_error( $reaction ) ) {
			$reaction = \get_comment( $state );
		}

		\do_action( 'activitypub_handled_update', $activity, null, $state, $reaction );

$reaction is literally set equal to null, so \get_comment( $state ); will never run.
But ... $state seems to actually be the comment! Let me edit my other response.

OK, forget it.
https://github.com/Automattic/wordpress-activitypub/blob/4c297acfd454a13260900ff0f97a9e3f425e960c/includes/collection/class-interactions.php#L138
Doesn't actually return a comment. It returns 1 if the update was successful.
So, one could again parse $activity and work off of that, or wait for this to be fixed.

OK, so as of now the latest master allows to add something like the following to, e.g., functions.php:
add_action( 'activitypub_handled_update', function( $activity, $second_param, $state, $reaction ) {
	if ( $reaction instanceof \WP_Comment ) {
		wp_set_comment_status( $reaction, 'hold' );
	}
}, 99, 4 );

If a comment was successfully updated by the AP plugin, it will be put "on hold" again. Tested on my site.
I also think maybe this should be the default behavior.
Or that it should be the default behavior when WordPress is set up to manually approve (all) comments.
Or that it should respect that setting where you can auto-approve comments if a commenter has left more than two or so approved comments. So that "new" actors aren't allowed to auto-update but "sufficiently known" actors are.
For now, this works for me.
Note that on Mastodon, too, if someone on another server updates their reply, its copy on the server you're on will also be updated. The only form of moderation (other than outright blocking/silencing/etc.) is after the fact, when others report (harmful) posts.
Of course a personal site is something else than even a single-user user Mastodon instance. We've grown to expect less from social media moderation than from well-kept blogs.


Rename gpr_slice --> grpc_slice

This change renames the type and repositions declarations outside of support/ directories.

This is starting to require serious maintenance.

I'll review this by noon today.

Boldly merging: maintenance pain is too high.


feat: Add support for GCP notification integration

Allow Notification Integration to work with GCP Pubsub
Test Plan

[x] acceptance tests

/ok-to-test sha=3d11842

run make docs please 😄

/ok-to-test sha=4731d36

/ok-to-test sha=dbf96d0

/ok-to-test sha=10b0589


[composer] can't navigate to an image in a stack

"navigate to item" on an image in a stack renders "object not available"

Seems like I need a IPFS_API_SECRET to repro the issue, someone should please reply: https://discord.com/channels/837138313172353095/837688677608980521/1238633747457376276

Log in to Cloudflare using Google, use secret available there locally.


Clean up deploy YAMLs

Combine YAMLs where possible. Delete YAMLs that e.g. don't have a serviceaccount and were created when RBAC was not on by default

See also: https://github.com/kubernetes-incubator/external-storage/issues/649


DB api unit test cases

What this PR does / why we need it:
Which issue this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged): fixes #
Special notes for your reviewer:
Release note:

Codecov Report

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

@@            Coverage Diff             @@
##           master     #158      +/-   ##
==========================================
+ Coverage   29.19%   30.18%   +0.99%
==========================================
  Files          61       60       -1
  Lines        3919     3866      -53
  Branches      441      441
==========================================
+ Hits         1144     1167      +23
+ Misses       2758     2657     -101
- Partials       17       42      +25

Impacted Files
Coverage Δ

dolphin/drivers/driver.py
56.52% <0.00%> (-11.23%)
:arrow_down:

dolphin/wsgi/common.py
53.57% <0.00%> (-4.50%)
:arrow_down:

dolphin/api/validation/validators.py
21.60% <0.00%> (-2.44%)
:arrow_down:

dolphin/api/validation/__init__.py
72.72% <0.00%> (-2.28%)
:arrow_down:

dolphin/api/v1/storages.py
38.65% <0.00%> (-2.00%)
:arrow_down:

dolphin/api/common/wsgi.py
22.60% <0.00%> (-1.09%)
:arrow_down:

dolphin/api/common/__init__.py
28.35% <0.00%> (-1.06%)
:arrow_down:

dolphin/context.py
57.14% <0.00%> (-0.86%)
:arrow_down:

dolphin/coordination.py
49.25% <0.00%> (-0.75%)
:arrow_down:

dolphin/rpc.py
45.33% <0.00%> (-0.72%)
:arrow_down:

... and 11 more


Typo in the code sample in Event section

On page 190, the Event Publisher section, the below code snippet was provided:
public class CarInfoEventArgs: EventArgs
{
     public CarInfoEventArgs(string car) => Car = car;
     public string Car { get; }
}

public class CarDealer
{
     public event EventHandler<CarInfoEventArgs>? NewCarInfo;
     public void CreateANewCar(string car)
     {
     Console.WriteLine($"CarDealer, new car {car}");
     RaiseNewCarCreated(car);
     }
     private void RaiseNewCarCreated(string car) =>
     NewCarCreated?.Invoke(this, new CarInfoEventArgs(car));
}

As you can see in the CarDealer's class, there isn't such EventHandler called NewCarCreated, instead, it is called NewCarInfo.
In the Event Listener section on page 191, a code snippet was provided to demonstrate the usage of publisher/consumer relationship
    CarDealer dealer = new();
    Consumer sebastian = new("Sebastian");
    dealer.NewCarInfo += sebastian.NewCarIsHere;
    dealer.NewCar("Williams");
    Consumer max = new("Max");
    dealer.NewCarInfo += max.NewCarIsHere;
    dealer.NewCar("Aston Martin");
    dealer.NewCarInfo -= sebastian.NewCarIsHere;
    dealer.NewCar("Ferrari");

However, NewCar is not a valid method in CarDealer class, it should be CreateANewCar() instead.

The file updates contains updates on issues in the book.
Page 190 with the NewCareCreated event name was already listed.
I've added the issue you mentioned with page 191. Thanks for reporting.


Revive Fieldmedic plugin, add early kernel diagnostic hooks

The intention is to create a reusable pattern for kernel self tests and diagnostics that costs nothing default. We have quite a few defensive validation steps in the kernel added organically as issues have been found. Ideally a fast booting kernel should always do the correct thing without having to validate. In practice, validation is crucial.
With this proposed setup we can optionally add a validation plugin (fieldmedic) that performs more extensive self tests after key checkpoints in the unikernel lifetime has been reached, with negligible cost in the default case (the cost of an empty function call per hook) and with zero cost when toggled off at compile time.
See commit messages for details.
To try it out:
$ nix-shell ~/IncludeOS/shell.nix --pure  --argstr unikernel ~/IncludeOS/test/kernel/integration/osinit --run ./test.py
...
<vm> <Multiboot>OS loaded with 1 modules
<vm> 	* osinit.elf.bin booted with vmrunner @ 0x29e000 - 0x43f6c0, size: 1709760b
<vm> * Multiboot begin: 0x9500
<vm> * Multiboot end: 0x3e5490
<vm> [ Field Medic ] ⛑  BSS Diagnostic passed
<vm> [x86_64 PC] constructor
<vm> [ Machine ] Initializing heap
<vm> [ Machine ] Main memory detected as 129998656 b
<vm> [ Machine ] Reserving 1048576 b for machine use
<vm> [ Field Medic ] ⛑  Machine allocator for x86 PC functional
<vm> [ Field Medic ] ⛑  Elf header intact, global constructors functional
<vm> [ Field Medic ] ⛑  Malloc and brk are functional
<vm> ================================================================================
<vm>
<vm>                            #include<os> // Literally
<vm>
<vm> ================================================================================
<vm>      [ Kernel ] Stack: 0x1ffbb8
<vm>      [ Kernel ] Boot magic: 0x2badb002, addr: 0x9500
...
...
<vm>      [ Kernel ] Running service constructors
<vm> --------------------------------------------------------------------------------
<vm> ================================================================================
<vm>  IncludeOS VERY_DIRTY (x86_64 / 64-bit)
<vm>  +--> Running [ OS initialization test ]
<vm> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
<vm>  +--> WARNING: No good random source found: RDRAND/RDSEED instructions not available.
<vm>  +-->        To make this warning fatal, re-compile with FOR_PRODUCTION=ON.
<vm> ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
<vm> Service::start entered
<vm> [ Field Medic ] ⛑  Service finished. Diagnosing.
<vm>                 [+] Field medic plugin active
<vm>                 [+] Post .bss invariant still holds
<vm>                 [+] Post machine init invariant still holds
<vm>                 [+] Post init libc invariant still holds
<vm> [ Field Medic ] ⛑  Diagnose complete. Healthy ✅

[ SUCCESS ] OS initialization test succeeded

Oups, good catch, thank you! Fixed.

Everything compiles and tests are green. I noticed a call to std::string that I missed on first review - I'm not sure how early FILLLINE is called, but you may want to replace that with std::pmr::string to avoid any calls to malloc before libc is initialised.


add external demos

Product Review Classification
Visual Search

Facial Emotion Recognition
Neural Style Transfer

We need a better way to highlight these demos ... Probably we should link this with the main README section. These are really awesome :)

@ThomasDelteil : Any update on this PR? IN our offline discussion, you mentioned that you would move this section to main README. Please let me know if you want me to move this.


Add ArrowError::ArithmeticOverflow

Which issue does this PR close?

N/A
Rationale for this change

Downstream crates (such as DataFusion Comet) often need specific handling for overflow errors and we currently have no good way to detect them other than looking for specific words in error messages, which is not very robust. For example, we currently have code like this:
match self.inner_abs_func.invoke(args) {
    Err(DataFusionError::ArrowError(ArrowError::ComputeError(msg), trace)) if msg.contains("overflow") => { ... }

It would be better if Arrow had an ArithmeticOverflow variant in the ArrowError enum, similar to the existing DivideByZero variant.
What changes are included in this PR?

Add ArithmeticOverflow and update existing overflow handling.
Are there any user-facing changes?
Yes, overflows will produce a different error now.

This makes sense to me, we should possibly make ArrowError non_exhaustive to avoid adding new variants being a breaking in future.
Unfortunately this has now missed the release and so will have to wait until the next breaking release which will be in 3 months time...

Then we can backport any fixes needed to 52.0.0_maintenance and keep development churning along on master

I think I would rather avoid reaching for git-flow style branch machinery until we have either:

A non-trivial number of pending breaking changes
Dependent / conflicting breaking changes that need to be applied in a particular order

Otherwise I think we're just creating work for ourselves.
I would prefer we mark this as a draft, and then look to bring it in once either the above criteria occur, or it is time for the breaking release.

I had understood we were going to do monthly minor releases, i.e. not just patch releases of back-ported fixes? I presume in your proposed model these would have to come from the maintenance branch, which implies mirroring most commits across

I had understood we were going to do monthly minor releases, i.e. not just patch releases of back-ported fixes? I presume in your proposed model these would have to come from the maintenance branch, which implies mirroring most commits across

I think the key question is "what do we want in the monthly releases" and thus "what is the criteria to include code there"

I would personally selfishly likely go with the latter as it would reduce overall maintainer burden

If you adopt the branching model you propose then yes, I feel fairly confident the former model is what people are more likely to expect/want. If I implement a non-breaking feature addition / performance improvement I would not expect to have to wait 3 months, or pester a maintainer for it to be usable.
A ticket is worthwhile given you appear to feel differently on this

I feel fairly confident the model that doesn't require additional ceremony for non-breaking changes is what people are more likely to expect/want

I agree. That doesn't me the maintainers are going to do it for them.

If I implement a non-breaking feature addition / performance improvement I would not expect to have to wait 3 months, or pester a maintainer for it to be usable.

My idea is that there is no pestering required -- the person would simply create the cherry-pick themselves. I will file a ticket tomorrow

I filed https://github.com/apache/arrow-rs/issues/5907 to track the question on how to do releases and what to do with PRs such as this

Per discussion on https://github.com/apache/arrow-rs/issues/6050 I retargeted this PR tothe 53.0.0-dev feature branch, where we are collecting PRs with breaking changes before master opens

I rebased this branch to target the 53.0.0 dev branch but somehow messed up the commit author. @andygrove  can you double check that this PR still looks good to you (and/or fix the author?)

I merged the 53 dev branch and that seems to have closed this PR -- any chance you can reopen and retarget main?


Add Practice Exercise: sum-of-multiples

Problem specification:
https://github.com/exercism/problem-specifications/tree/main/exercises/sum-of-multiples
TODO:

generate the exercise with ./bin/fetch-configlet && ./bin/configlet create --practice-exercise sum-of-multiples
check if any of the existing tracks (examples below) implement the exercise and you can reuse the implementation:

Go: https://github.com/exercism/go/tree/main/exercises/practice
Rust: https://github.com/exercism/rust/tree/main/exercises/practice
Python: https://github.com/exercism/python/tree/main/exercises/practice

use exercise's canonical-data.json to see which test cases to implement:
https://github.com/exercism/problem-specifications/blob/main/exercises/sum-of-multiples/canonical-data.json

I want this

It's yours


Deploy to IPFS

Checklist:

[x] Test your work and double check you didn't break anything

Description:

Remove all relative image and font paths so demo-dapp can be deployed in sub-directories
Add simple 'deploy to IPFS' script which builds and deploys the demo-dapp to the origin IPFS gateway

Seconding the AWESOME!


Google Universal Sentence Encoder Support for Embeddings

Added support for embeddings with Google Universal Sentence Encoder v5

Added support for embeddings with Google Universal Sentence Encoder v5

thanks!! will take a look soon

it's an error. The file should not be modified. Not sure how it was
modified.
On Thu, Mar 23, 2023 at 3:08 PM Jerry Liu @.***> wrote:

@.**** commented on this pull request.
thanks for doing this! a few comments:

why is scripts/create_llama_package.sh modified?
there's some linter issues, i can help fix if you'd like

In gpt_index/embeddings/google_use.py
https://github.com/jerryjliu/llama_index/pull/842#discussion_r1146700830
:

@@ -0,0 +1,28 @@
+"""Google Universal Sentence Encoder Embedding Wrapper Module."""

+from typing import List
+
+import tensorflow_hub as hub
+
+from gpt_index.embeddings.base import BaseEmbedding
+
+# Google Universal Sentence Encode v5
+google_use = hub.load("https://tfhub.dev/google/universal-sentence-encoder-large/5")
could this be a part of the init file? so it's lazily downloaded.
Since I'd love to expose this module in gpt_index/embeddings/init.py
(so users can do from gpt_index.embeddings import
GoogleUnivSentEncoderEmbeddings)
—
Reply to this email directly, view it on GitHub
https://github.com/jerryjliu/llama_index/pull/842#pullrequestreview-1355381634,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AARB4DGE6XAGHB4F2SHOL2DW5SNTNANCNFSM6AAAAAAWEKTD24
.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Chris Clark
704.776.0862

it's an error. The file should not be modified. Not sure how it was modified.
…
On Thu, Mar 23, 2023 at 3:08 PM Jerry Liu @.> wrote: @.* commented on this pull request. thanks for doing this! a few comments: - why is scripts/create_llama_package.sh modified? - there's some linter issues, i can help fix if you'd like ------------------------------ In gpt_index/embeddings/google_use.py <#842 (comment)> : > @@ -0,0 +1,28 @@ +"""Google Universal Sentence Encoder Embedding Wrapper Module.""" + +from typing import List + +import tensorflow_hub as hub + +from gpt_index.embeddings.base import BaseEmbedding + +# Google Universal Sentence Encode v5 +google_use = hub.load("https://tfhub.dev/google/universal-sentence-encoder-large/5") could this be a part of the init file? so it's lazily downloaded. Since I'd love to expose this module in gpt_index/embeddings/init.py (so users can do from gpt_index.embeddings import GoogleUnivSentEncoderEmbeddings) — Reply to this email directly, view it on GitHub <#842 (review)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AARB4DGE6XAGHB4F2SHOL2DW5SNTNANCNFSM6AAAAAAWEKTD24 . You are receiving this because you authored the thread.Message ID: @.***>
-- Chris Clark<PHONE_NUMBER>

I am struggling to get my existing fork to have the correct files with only 1 file changed, feel free to take the code and incorporate.


Filter events by date

Is your feature request related to a problem? Please describe.
Sometimes you ar just overwhelmed by the event list. There are soo many events :o. It would be nice to be able to filter for (past/future) events. We should implement an event filter by date.
Describe the solution you'd like

there should be an event filter by date for web & native

Describe alternatives you've considered
N/A
Additional context
Design can be found here -> https://www.figma.com/file/cA4F2MwHs2LNGviOWUjfE9/Integreat-Tickets-Frontend?type=design&node-id=2376-2&mode=design&t=zQ9GHEHM5S1I7sVe-0

option B is approved by the service team, the issue can be implemented

Closing in favour of #2808 which is better formulated.


[Bug] jedis-4.x-plugin incorrectly took effect in jedis 2.x version.

Search before asking

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

Apache SkyWalking Component
Java Agent (apache/skywalking-java)
What happened
jedis-4.x-plugin has incorrectly taken effect in jedis 2.5.2, 2.6.3 versions, but jedis 2.7.3, 2.8.2, 2.9.0 versions have no impact.
The witness method for jedis-4.x-plugin selection is
new WitnessMethod("redis.clients.jedis.Pipeline", ElementMatchers.named("persist").and(ElementMatchers.takesArguments(1))).
In jedis 2.5.2, 2.6.3 versions，when loading the jedis plugin，org.apache.skywalking.apm.agent.core.plugin.WitnessFinder#exist(WitnessMethod , ClassLoader) returns true.
What you expected to happen
No response
How to reproduce
No response
Anything else
No response
Are you willing to submit PR?

[ ] Yes I am willing to submit a PR!

Code of Conduct

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

Are you going to fix this?

We are not clear about what actually happened in these cases. But it is abnormal.
There is no significant differences of redis.clients.jedis.Pipeline at source code level between versions 2.5.2, 2.6.3 and 2.7.3, 2.8.2, 2.9.0. But when we were debugging the WitnessFinder, we've observed over 300 hundreds method tokens for redis.clients.jedis.Pipeline from 2.5.2, 2.6.3 but only ~10 method tokens for redis.clients.jedis.Pipeline from 2.7.3, 2.8.2, 2.9.0.
One possible reason is that the resolved redis.clients.jedis.Pipeline has already been transformed by another plugin and then it gave wrong declaredMethods. We will further investigate this issue.

I think unless you have another SkyWalking kernel running, it is impossible it has been instrumented.

jedis 2.5.2

There should be no such methods in the redis.clients.jedis.Pipeline. These methods are enhanced in redis.clients.jedis.PipelineBase, Pipeline inherits PipelineBase.

Are these different between versions? I am not following.

Finally, I confirm the issue is from the upstream. I've made a minimal example to reproduce this issue,
https://github.com/raphw/byte-buddy/issues/1429#issuecomment-1507111558
Let's see what the author would reply. It would be a resolution bug or miss usage of byte-buddy API.

This seems a very strange bug. Let's see.

This seems like a very strange bug. Let's see.

As is discussed in the JDK community, https://bugs.openjdk.org/browse/JDK-6815786 (lasting for 11 years), this is an expected behavior of Java compiler. Bridges methods will be generated for public methods in the non-public superclasses. Specifically for Jedis, https://github.com/redis/jedis/commit/b248532bf61594ed1246b42c08a9aafdafc0dd6f this commit added public modifier to the MultiKeyPipelineBase and PipelineBase. This causes observed differences in this issue.
In conclusion, for SkyWalking scenario, we should exclude synthetic methods in the WitnessFinder. I will raise a PR to resolve this issue.


Feature Request: Show Job Duration

Right now the details page for individual jobs show the start date, age and end date. I'd love it if it could show duration as well.

Would something like that :

be enough ?

Here is the draft PR : #54


Error “Response body object should not be disturbed or locked” when sending JSON requests with better-auth in NestJS

Describe the bug
I’m encountering an issue when using better-auth with drizzleAdapter in a NestJS application. When I send requests with JSON bodies to any auth-related route, I receive the following error:
ERROR [ExceptionsHandler] Request with GET/HEAD method cannot have body.
TypeError: Request with GET/HEAD method cannot have body.
    at new Request (node:internal/deps/undici/undici:6066:17)
    at getRequest (/Users/hao/workspace/bizniff-front/node_modules/.pnpm/better-call@0.2.14-beta.1/node_modules/better-call/src/adapter/request.ts:98:9)
    at <anonymous> (/Users/hao/workspace/bizniff-front/node_modules/.pnpm/better-call@0.2.14-beta.1/node_modules/better-call/src/adapter/node.ts:10:34)
    at toNestJsController (/Users/hao/workspace/bizniff-front/apps/server/dist/auth/better-auth.js:89:16)
    at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
    at async AuthController.handleAuth (/Users/hao/workspace/bizniff-front/apps/server/dist/auth/auth.controller.js:25:20)
    at async /Users/hao/workspace/bizniff-front/node_modules/.pnpm/@nestjs+core@10.4.1_@nestjs+common@10.4.1_reflect-metadata@0.1.14_rxjs@7.8.1__@nestjs+platfor_fcwotrhkfhfbsqcmjq6msnog44/node_modules/@nestjs/core/router/router-execution-context.js:46:28
    at async /Users/hao/workspace/bizniff-front/node_modules/.pnpm/@nestjs+core@10.4.1_@nestjs+common@10.4.1_reflect-metadata@0.1.14_rxjs@7.8.1__@nestjs+platfor_fcwotrhkfhfbsqcmjq6msnog44/node_modules/@nestjs/core/router/router-proxy.js:9:17

It seems that the issue originates from better-call, a dependency of better-auth. When better-auth processes JSON requests, it appears to encounter an issue.
To Reproduce

Set up better-auth with drizzleAdapter in a NestJS application.
Send a JSON request body to an auth-related route.

The error seems to occur here:
My NestJS setup with better-auth is based on code from this repository. However, unlike this repository, I am using NestJS v10, and I’m unsure if this might be contributing to the issue.

Can you past your requests code here?

@AmineYagoub Thank you for your reply
In my NestJS controller, the request.destroyed property is true. This causes the getRequest function in better-call to cancel the ReadableStream before passing it to the Request body, which leads to an error. I extracted the relevant code, and it immediately throws an error, which I believe should be reproducible.
Here’s the sample code for reference:
import { All, Controller, Next, Req, Res } from '@nestjs/common';
import { NextFunction } from 'express';
import { Request } from 'express';

@Controller('/auth')
export class AuthController {
  constructor() {}

  @All('*')
  async handleAuth(
    @Req() req: Request,
    @Res({ passthrough: true }) response: Response,
    @Next() next: NextFunction,
  ) {
    const protocol = (req.connection as any)?.encrypted ? "https" : "http";
    const base = `${protocol}://${req.headers[":authority"] || req.headers.host}`;

    console.log(req.destroyed === true ? "destroyed" : "not destroyed");

    const readable = new ReadableStream();
    readable.cancel();
    const body = readable;

    // This will throw a `TypeError: Response body object should not be disturbed or locked`
    const newReq = new Request(base + req.url, {
      // @ts-expect-error
      duplex: "half",
      method: req.method,
      body: body,
      headers: req.headers as Record<string, string>,
    });

    // Uncommented code that would handle request forwarding
    // try {
    //   return await toNestJsController(this.authService, req, response);
    // } catch (error) {
    //   next(error);
    // }
  }
}

I’m not familiar with the internal logic of better-call, so I’m unsure why it’s designed this way.
For reference, I’m using:

NestJS version: v10.3.10
Node version: v20.11.0

@AmineYagoub
I encountered an issue with better-auth when integrating it with Nest.js. The bodyParser middleware in Nest.js reads and parses the request body before it reaches the controller, which leads to the request stream being marked as destroyed. As a result, better-auth throws an error due to the destroyed request stream.
Disabling the body parser (bodyParser: false) does resolve the issue, but this approach would require changes to the overall development setup, which is impractical for better-auth integration alone. To address this, I modified the better-call code locally, allowing better-auth to work with Nest.js by handling the parsed req.body when req.destroyed is true.
Here’s the modification I used:
https://github.com/Bekacru/better-call/blob/fe93bc0f1344aeafdb7dc228ea77678397b2af0f/src/adapter/request.ts#L31C1-L35C3
// if (req.destroyed) {
//   const readable = new ReadableStream();
//   readable.cancel();
//   return readable;
// }
if (req.destroyed) {
  if (req.body) {
    return new ReadableStream({
      start(controller) {
        controller.enqueue(Buffer.from(JSON.stringify(req.body)));
        controller.close();
      },
    });
  } else {
    return new ReadableStream({
      start(controller) {
        controller.close();
      },
    });
  }
}

It would be great if better-auth could provide native support or configuration options for back-end frameworks like Nest.js, where a pre-parsed request body is common.
Thank you for considering this!

Yes exactly, we need native support for nestjs , let me start working on that

Oh wow! I have been faced with this issues for the past two days, It was driving me crazy,lol. I think @NoHaxito are @danyalutsevich are also looking into this. This definitely simplify things. Thanks @AmineYagoub, Looking forward to what magic you would make 😊, Thanks for the good work.

I've tried to get express app instance to attach node handler to it but I still get Invalid Url error
check my repo

I'm going to close this issue. We will track NestJS support with #171


Deprecation warning when used with Rails 6

DEPRECATION WARNING: Ruby 2.5+ (required by Rails 6) provides Hash#transform_values natively, so requiring active_support/core_ext/hash/transform_values is no longer necessary. Requiring it will raise LoadError in Rails 6.1.

I belive this is fixed by #231.

@jonekdahl There is again the issue when using Rails <IP_ADDRESS> and Ruby 2.7.1
DEPRECATION WARNING: Ruby 2.5+ (required by Rails 6) provides Hash#transform_values natively, so requiring active_support/core_ext/hash/transform_values is no longer necessary. Requiring it will raise LoadError in Rails 6.1. (called from require at /Users/rafalpiekara/Desktop/bs/AG/app/AchieveGuru/backend_achieve-guru/spec/spec_helper.rb:2)

I would be glad to take care on that and fix it. I've found the possible code here:
require 'active_support/core_ext/hash/transform_values' if Mongoid::Compatibility::Version.mongoid4_or_newer? && ActiveSupport.version < Gem::Version.new('6')

It seems to be correct, but the depreaction is present. I've checked the Mongoid repo and there is no Compatibility::Version module. Could it be deprecated? What are your thoughts on that?

Mongoid::Compatibility is https://github.com/mongoid/mongoid-compatibility

@jonekdahl There is again the issue when using Rails <IP_ADDRESS> and Ruby 2.7.1
DEPRECATION WARNING: Ruby 2.5+ (required by Rails 6) provides Hash#transform_values natively, so requiring active_support/core_ext/hash/transform_values is no longer necessary. Requiring it will raise LoadError in Rails 6.1. (called from require at /Users/rafalpiekara/Desktop/bs/AG/app/AchieveGuru/backend_achieve-guru/spec/spec_helper.rb:2)

I would be glad to take care on that and fix it. I've found the possible code here:
require 'active_support/core_ext/hash/transform_values' if Mongoid::Compatibility::Version.mongoid4_or_newer? && ActiveSupport.version < Gem::Version.new('6')

It seems to be correct, but the depreaction is present. I've checked the Mongoid repo and there is no Compatibility::Version module. Could it be deprecated? What are your thoughts on that?

I've got the same issue with ruby 2.7.1 rails 6.03 and mongoid 7.1.1, did you manage to solve this ?

I'm upgrading my app from Rails 5 to Rails 6.0.3, using Ruby 2.6.5 and mongoid-rspec 4.0.1 and I'm getting the same warning message.

Upgrading my app to Rails 6.0.3 and getting the same warning.

Released 4.1.0 that has this resolved.


bilinear_warping error

@D-Nilsson
Could you please check this error:bilinear_warping.cc:33:5: error: expected ‘;’ before ‘Tensor’,thanks~

I have updated the file in the repo. It compiles as it should now.


Option to Hide Resources

It is confusing to some of my players having Resources able to be tracked in both the top (Resources 1-3) and in the features themselves.
This duplicates due to how the importers from dndbeyond seem to pull stuff in.
While it is not specific to this sheet, my players all prefer the layout you designed.
Could there be an option to hide the Resources 1-3 which will free up a bit of space, and prevent duplicate tracking of these resources for those who want to track via the feature listing itself?
Thank you

@DrWiFi thanks for the feedback!
I'm hesitant to mess with this as it's got some high chances of confusion. Can you do me a favor and try using 5e Sheet Resources Plus to set the sheet's resources to "0"?
I don't know if that will work but if so, that's a potential I can point to for people who think similarly.

Thank you for the quick response. That module worked great for removing them!

Nice! I'll mark this as won't fix then. Thanks for letting me know it worked!


[CDN] Add multi-origin preview feature.

Description
Add CDN multi-origin preview feature.
Testing Guide

Create an origin group on a CDN endpoint:
az cdn origin-group create -g group --profile-name profile --endpoint-name endpoint -n origin-group-name --origins origin-

Set the origin group as default:
az cdn endpoint update --default-origin-group origin-group-name

Create multiple origins for a single CDN endpoint:
az cdn origin create -g group --host-name example.contoso.com --profile-name profile --endpoint-name endpoint -n origin --host-name example.contoso.com

Update new weight and priority fields of an origin
az cdn origin update -g group --profile-name profile --endpoint-name endpoint -n origin --priority 3 --weight 500

History Notes

This checklist is used to make sure that common guidelines for a pull request are followed.

[x] The PR title and description has followed the guideline in Submitting Pull Requests.

[x] I adhere to the Command Guidelines.

CDN

@kairu-ms, could you take a look at this PR when you get the chance? Thanks in advance.

@kairu-ms @haroldrandom please help with PR review.

@haroldrandom, Just solved the conflict plus fixed a small bug and a few unrelated tests that were failing in some situations. Please take another look when you get the chance.


Prevent (and gracefully handle) DynamoDB errors

Proposed changes
DynamoDB does not support empty Strings. If the session contains empty (or null) values one gets the following error upon save:

ValidationException: One or more parameter values were invalid: An AttributeValue may not contain an empty string

The error is not properly handled and yields an unhandled promise rejection error.
This PR addresses both issues and emits a new app-event called responseError to allow plugins to report/log the error.
Types of changes

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

Checklist

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

Thanks Felix!


Add storybook library

Sidebar, TypeChecker, Controller, Season, Time, Weather 컴포넌트 스토리북 추가

PR 👉  https://github.com/PPO-F-MAN/react-season-component-web/pull/32 Add storybook library


the cluster any question

I configure the emqttd cluster and test it , I found the client session  will not be copied to every clusterd node , so every clusterd node is not equal 。

Cluster Design: https://github.com/emqtt/emqttd/wiki/Cluster Design https://github.com/emqtt/emqttd/wiki/Cluster Design
Topic tree will be copied to each cluster nod, but clients, sessions and subscriptions are local.

在 2015年6月4日，下午8:17，sailingchen<EMAIL_ADDRESS>写道：
I configure the emqttd cluster and test it , I found the client session will not be copied to every clusterd node , so every clusterd node is not equal 。
—
Reply to this email directly or view it on GitHub https://github.com/emqtt/emqttd/issues/157.

Cluster 存在这样的问题：
客户端断开一段时间后，连接上一个从未连接过的集群节点A，在断开连接到重新连接上集群节点A这段时间内，在集群A上发布的消息则不到。
当然 Bridge 模式也存在同样问题。

客户端session模式连接，断线重连其他Node，那么离线消息收不到。如果有这个需求，可以支持session takeover，接管其他节点的session。

session takeover 是客户端实现，还是broker 实现，看了一段描述，雾里看花一样：
Client Take-Over
A disconnected client will most likely try to connect again. It could be the case that the broker still has an half-open connection for the same client. In this scenario the MQTT will perform a so-called client take-over. The broker will close the previous connection to the same client (determined by the same client identifier) and establishes the connection with the newly connected client. This behavior makes sure that half-open connection won’t stand in the way of a new connection establishment of the same client.

我们用于手机app 的PUSH , 看来目前只有从业务分割。
或者app 初始化连接broker 时，连接一次所有节点IP

Broker实现的Session Take-Over, 和你上面的文档不同。Broker集群模式下，比如Session之前在A节点，断线重连后从B节点上线；那么把A节点的Session数据，应该传递给B节点的新Session。

Broker实现的Session Take-Over, 和你上面的文档不同。Broker集群模式下，比如Session之前在A节点，断线重连后从B节点上线；那么把A节点的Session数据，应该传递给B节点的新Session。

请问emqttd 具本了这个功能吗，相关配置选项有哪些，谢谢。

@sailingchen, this feature is not supported now. I could plan in late release if the feature is important in cluster deploy.

I solved this problem by using haproxy as /api/sessions API load balancer, and polling every broker in the cluster to get the session list, this may not the best solution, but for now I need to know actual number of the sessions... hope it can be supported soon.


How to download fern_2v?

I can't download fern_2v with download_example_data.sh?
could you share again?
thanks.
The following paragraph is the information.
‘’‘Document not found
The requested address (URL) was not found on this server.
You may have used an outdated link or may have typed the address incorrectly.’‘’

+1, please update the download link, or at least provide info on which 2 input images are used in all the LLFF dataset experiments for reproducibility. Thanks!


proxies.....

when i do the format mentioned on the readme file i get this error
[21:42:26] [Error Occurred] > Generation Error: 502 bad gateway.

Just read the readme.md.


Simplify MigrateDefinition

DistributedCp and DistributedMv now no longer needs the complicated folder to folder operation and now only handles file to file. Removing a bunch of code that does folder to folder work.

@gpang ping

Since gene seems busy, I'll give it to @yuzhu

alluxio-bot, merge this please.


Feature: direction RTL support

Hi,
@mdo, please consider these:

Unofficial RTL-ized Bootstrap 4.x
RTL-ized Bootstrap 4.x Examples

Hi, @mdo
Given that v4.1 is coming out soon and the ship list for it (#25375) doesn't include this issue, is there any update that can be provided about rtl support?
Thanks!

No update to give other than this won't be in v4.1 :).

@mdo rtl support is coming for v4.2 ? :-(

@mdo we need rtl support, is it coming in 4.2?

Hello @mdo
I have an approach for rtl-ltr support that i used on Bootstrap3 and now on bootstrap4
here in this repo you will find that i took only the scss core files for bootstrap 4 and updated for direction and added my direction.scss file
and in the build you will find 2 files bootstrap.css & bootstrap-rtl.css
https://mohamedragaey.github.io/bootstrap-scss-multi-dir/
and here is my build tool included in bootstrap-rtl as npm package
https://mohamedragaey.github.io/bootstrap4-playground/
if you find it interesting please let me know then i will make a pull request and then go for it .

4.1.2 is out and RTL is not included too. Can you please give this a priority in the next release, either 4.1.3 or 4.2? This is important for all RTL languages to get in bootstrap 4.

cast my vote, too.

Can someone summarise the RTL issues? I use bootstrap on a bidi site without any problems (flex utils solved most of the problems for me)

@mkamensky
In forked version that I am working on it, there are some issues on SCSS modules & mixins, I tagged those by //rtl inside *-rtl.scss files.
Changed SCSS module:
alert, button group, card, carousel, close, custom form, dropdown, input group, list group, modal, nav, navbar, pagination, popover, reboot, type, variables
Changed SCSS mixins:
caret, forms, grid, list, pagination, reset text

Thanks, I'll have a look

Sounds good to me

I am waiting for RTL version 👍 Thanks ...

That's great. Thanks in advanced.

Just a small tip for those RTL pioneers out there -
when printing a page, the $print-body-min-width variable causes weird print issues unless you set direction: rtl for your html element (or dir="rtl" attribute in your tag). Sounds elementary, but until I started printing content, i was fine with setting just the body tag.

It would be great to have official RTL for Boostrap 4.

waiting for rtl version. thanks

I hope RTL-support will make it for 4.3.0
thanks for your hard work, much appreciated!

It is 4.3 now and we still don't see RTL support. We are understanding that the team is busy now to release version 5. Can you please include RTL support in version 5?

hi , please add support rtl .
thanks

We Announce the AryaBootstrap:

AryaBootstrap is a bootstrap with dual layout align support and, used for LTR and RTL web design.

We always publish lastest version at:
https://github.com/mRizvandi/AryaBootstrap
and other older version also is exist for old projects
checkout the reference website at:
http://abs.aryavandidad.com

Waiting eagerly to see bootstrap-rtl support.

Guys we waiting the rtl version

@ahmdqader
Use this for now

Another option that might work is to run Bootstrap's CSS through postcss-rtl.  I haven't tried it myself, but it might be an option.  Note that it will generate a much larger final CSS file size.

I like bootstrap reverse as it is built as an extension rather bootstrap fork.

@MohammadYounes wouldn't rtlcss work for this?

@meywd Sure, It's been done long time ago!

Repo: https://github.com/RTLCSS/bootstrap
Preview: https://bootstrap.rtlcss.com/
CDN: https://rtlcss.com/cdn/css-frameworks/bootstrap/

by using text-right class solved my problem.

see this stackoverflow post

When is the rtl will apply to the main freamworks like Vue, React?

I'm not really into Bootstrap but i did it on my customized bootstrap fork named BootsDrac 🧛♂:

without reversing and code overwriting
You can build both RTL / LTR and dark mode from same source code

just take a look and if it's helpful for Bootstrap, I will make PR and then we can improve it:
Demo
@mdo @fat


Added assertion to check visible checkbox

MinkContext checks the first named checkbox ignoring visibility, now it checks the first visible checkbox on the page.

@jose-ochoa 👍


[OpenAI] [Assistants] PR feedback

Feedback for Assistants streaming API PR

Adding unions (in the swagger represented as enums) for streaming event names
Modified visibility for classes for correct generation for the Java emitter

Thanks for doing this, @jpalvarezl ! The changes make sense across the board and I hope it's OK that I'm going to merge it as-is right now.


Support replacing with capture groups

See discussion here: https://github.com/BurntSushi/ripgrep/issues/2325#issuecomment-1602344404
We can't get the matches from ripgrep's JSON output, but we should be able to run the regex against the matched lines and perform the replacements that way.

:astonished:


Support cookie authentication

Hi,  great plugin,  thanks.  I want to know how can it support sending a cookie, let's say a JWT.  I don't need manual auth,  just pass the token.  Thanks,

Hey @digitaico
In that case I would just add the header directly:
GET /foobar HTTP/1.1
Cookie: PHPSESSID=298zf09hf012fh2; csrftoken=u32t4o3tb3gg43; _gat=1

if you have a jwt say for OAuth you can do something similar:
GET /foobar HTTP/1.1
Authorization: Bearer eyJhbGciOiJIUzI1NiIsInR5cCI6IkpXVCJ9.eyJzdWIiOiIxMjM0NTY3ODkwIiwibmFtZSI6IkpvaG4gRG9lIiwiaWF0IjoxNTE2MjM5MDIyfQ.SflKxwRJSMeKKF2QT4fwpMeJf36POk6yJV_adQssw5c```


[SERIALIZE] Make Schema an option for MongoDB

Status Quo: we already support Schema for MongoDB at insertion: insert_many(..., schema=...).
Suggestion: make Collection a Component and add an (optional) Schema as argument.

This feature will continue after the #2039  is merged.


Removing a tooltip element when it is displayed doesn't hide the tooltip

Here's a simple fiddle with the example http://jsfiddle.net/rQPKR/
Basically if I have a tooltip element, that onclick removes itself, the tooltip itself will stay displayed.
I know there is a closed issue #2298, but I think there are scenarios where the suggested solution doesn't apply.
For example when I am handling the click event on higher level of abstraction, where I don't have access to the element itself and I can't do anything like $(this).tooltip('hide'). The only solution to this would be globally hiding all tooltips like $('a').tooltip('hide'), which I'm not sure is a clean solution.

This was supposedly done in v2.1 (#4104) however, the bug still exists with v3.3.6 (http://jsfiddle.net/otp0pt1z/1/).  Maybe there is a fix that requires a code change?

Seeing the same issue in v3.3.7, indeed simple solution is to $(".popover").popover("hide"); when elements are about the be removed.


if statement of RaycastHit

using UnityEngine;
using UnityEngine.Networking;
using System.Collections;
public class PlayerShoot : NetworkBehaviour
{
private const string PLAYER_TAG = "Player";
private PlayerWeapon weapon;
[SerializeField]
private Camera cam;
[SerializeField]
private LayerMask mask;
void Start ()
{
	if (cam == null)
	{
		Debug.LogError("PlayerShoot: No camera referenced!");
		this.enabled = false;
	}

	//weaponManager = GetComponent<WeaponManager>();
}
void  Update()
{
	if(Input.GetButtonDown("Fire1"))
	{
		Shoot();
	}
}
[Client]
void Shoot()
{
	RaycastHit _hit;
	// below line gives me error
	if (Physics.Raycast(cam.transform.position, cam.transform.forward, out _hit, weapon.range, mask) )
	{
		//Debug.Log("we hit "+_hit.collider.name);
		if(_hit.collider.tag == PLAYER_TAG)
		{
			CmdPlayerShot(_hit.collider.name,weapon.damage);
		}
	}
}
[Command]
void CmdPlayerShot (string _playerID, int _damage)
{
	Debug.Log(_playerID + " has been shot.");

    Player _player = GameManager.GetPlayer(_playerID);
    _player.RpcTakeDamage(_damage);
}

}
// error : Object reference not set to an instance of an object

I got the same error

Drag the "Player Weapon" script on the player prefab into the "Weapon" variable for the "Player Shoot" script. The issue is because the weapon isn't set.


Add rust-toolchain file to specify required Rust version

When checking out the branch for https://github.com/atom/xray/pull/67, I ran into an issue compiling the project. @as-cii pointed out that Xray now depends on Rust 1.27 nightly. This pull request adds a rust-toolchain file to make that dependency explicit. With this change in place, running ./script/build will automatically install the nightly release if it's not already present, and it will then use that release to build the project.
It's possible to specify nightly as the version, but since nightly is a moving target, it seems useful to me to pin to a specific nightly build, so I've pinned it to the latest nightly build: nightly-2018-04-27. If we make a change that requires a newer version, we would then update rust-toolchain to make that dependency explicit.

Thanks!


update rnpm to react-native config

Not sure where we saw this, but when Chris and I were using the template, we saw it mentioned (I think in Metro Bundler) that we needed to update rnpm to use a react-native.config.json file instead of whatever we were previously using.

Currently, what's happening is we have an rnpm.json file which we use to add an rnpm section to the package.json. Instead, we need to create a react-native.config.js file with the same information.
Source: https://github.com/react-native-community/cli/blob/master/docs/configuration.md#apps


fix(Create Edit): Missing labels for owners and contacts

@clarkjefcoat @tonyalletag

:+1:

:+1:


Create commonjs imports in importFixes

TypeScript Version:  master
Code
Currently getCodeActionForNewImport only creates ES6 and import = imports. This could be extended to create commonjs imports if we detect that we're in a commonjs file (file.commonJsModuleIndicator && !file.externalModuleIndicator)
Then, no longer exclude these files from getting external module completions in completions.ts.

Declining due to lack of activity/interest


[TASK]: Make top row of details panel sticky so the X-close button is always available, even when you scroll down the page

Currently when you scoll down the page of the details panel, the X-close button is no longer available.

An elegant solution is to make the top row of the details panel sticky, so when you scroll down the page the X-close button remains in place. But it would be pretty weird to make the "XDD" label sticky and lose the title. So I propose that when the title scrolls off screen, we hide the "XDD" label and show the title in that row instead.
One last detail, we should apply an "elevations-1" effect to the sticky row so it looks natural when the content below scrolls under it.
It should look like this:

Those notes above ^^ apply for when we're in the Explorer preview mode. We need to be able to click the 'X' after scrolling down the page.
When we're in the Project view and the Document occupies its own tab, there is no 'X' button. But there is a 3-dot-more icon that belongs next to the name, with a menu options for "Remove from project"
In a perfect world we would use "position: sticky" and apply a subtle transformation effect for font-size, etc. It would be smooth and delightful like scrolling on an iPhone. If that's really hard, we can simply fade this in as an overlay when the header scrolls out of frame.


Failed to load library ios.

Describe the bug
Service fails to load and never moves past "Warming up the model."
Key error seems to be "Tried architecture: ios, error: System.Exception: Failed to load library ios."
Download complete!

UnityEngine.DebugLogHandler:Internal_Log(LogType, LogOption, String, Object)
LLMUnity.LLM:StartLLMServer(String)
System.Threading.Tasks.Task:Execute()
System.Threading.ExecutionContext:RunInternal(ExecutionContext, ContextCallback, Object, Boolean)
System.Threading.Tasks.Task:ExecuteWithThreadLocal(Task&)
System.Threading.Tasks.Task:ExecuteEntry(Boolean)
System.Threading.ThreadPoolWorkQueue:Dispatch()

LLM service couldn't be created
UnityEngine.DebugLogHandler:Internal_Log(LogType, LogOption, String, Object)
LLMUnity.LLMUnitySetup:LogError(String)
LLMUnity.LLM:StartLLMServer(String)
System.Threading.Tasks.Task:Execute()
System.Threading.ExecutionContext:RunInternal(ExecutionContext, ContextCallback, Object, Boolean)
System.Threading.Tasks.Task:ExecuteWithThreadLocal(Task&)
System.Threading.Tasks.Task:ExecuteEntry(Boolean)
System.Threading.ThreadPoolWorkQueue:Dispatch()

The Plugins/Android folder is empty. I'm not sure if there is supposed to be something for ios?
Steps to reproduce

Started a new Unity Project (6000.0.26f1 Silicon)
Made sure "StreamingAssets" and "Plugins/Android" folders existed
Installed latest version through package manager + git
Imported mobile scene, add it to build settings
Downloaded Llama 3.2 1B
Checked "Download on start"
Built for ios
Model download, progress bar is visible
Screen remains on "Warming up the model".

LLMUnity version
2.4.0
Operating System
macOs

I see that there is a file:
StreamingAssets/undreamai-v1.2.0-llamacpp/iOS/libundreamai_iOS.dylib
But if I copy it to:
Plugins/iOS/libundreamai_iOS.dylib
In xcode the build fails with this error:
Library 'undreamai_iOS' not found

Copied:
Packages/ai.undream.llm/Resources/usearch/iOS/libusearch_static_c.a
To:
Plugins/iOS/libusearch_static_c.a
I no longer get the xcode error, but the build still fails with the same error as before:
Tried architecture: ios, error: System.Exception: Failed to load library ios.
UnityEngine.DebugLogHandler:Internal_Log(LogType, LogOption, String, Object)
LLMUnity.LLM:StartLLMServer(String)
System.Threading.Tasks.Task:Execute()
System.Threading.ExecutionContext:RunInternal(ExecutionContext, ContextCallback, Object, Boolean)
System.Threading.Tasks.Task:ExecuteWithThreadLocal(Task&)
System.Threading.Tasks.Task:ExecuteEntry(Boolean)
System.Threading.ThreadPoolWorkQueue:Dispatch()

Hi! What iOS device are you trying this on?
You don't need to copy any plugins, I do that automatically.
The plugins should be in the following build dirs:

Data/Raw/undreamai-v1.2.0-llamacpp/iOS/libundreamai_iOS.dylib
Libraries/LLMUnity/Resources/usearch/iOS/libusearch_static_c.a

iPhone 14 Pro Max
In the xcode project this exists:
Data/Raw/undreamai-v1.2.0-llamacpp/iOS/libundreamai_iOS.dylib
But this does not:
Libraries/LLMUnity/Resources/usearch/iOS/libusearch_static_c.a
I think this is because I installed it through package manager, as the asset store version is not setup for 2.4.0 yet. If I make this path in the unity project:
Assets/LLMUnity/Resources/usearch/iOS/libusearch_static_c.a
Then this does appear in the xcode project:
Libraries/LLMUnity/Resources/usearch/iOS/libusearch_static_c.a
But I still get the error:
Tried architecture: ios, error: System.Exception: Failed to load library ios.

The libusearch_static_c.a is in the Resources folder already: https://github.com/undreamai/LLMUnity/blob/main/Resources/usearch/iOS/libusearch_static_c.a
so probably something to do with the package manager, yes.
But anyway the libusearch_static_c.a is only needed for the RAG, the LLM should work.
how do you install it on the device?
I tested on a iPhone 14 Pro Max on AWS Device Farm and it worked for me.
The way I did it was to create an ipa and install it on the device.
You can find this test ipa here.
Would you like to have a call or chat on discord for quicker resolution?

Generally it would be better to share the unity or xcode project (with your apple profiles removed) instead of an ipa. Sideloading to a device is not quite on the same with Apple as it is on Android, and signing packages for compiled apps is much more complicated and risky.
I'll try out a few more things and reach out on discord.

Sure, I didn't do anything interesting 👍.
I just built the MobileDemo sample with the Qwen 2 0.5B model.

I get the same error, on IPhone 13 Pro, Tried architecture: ios, error: System.Exception: Failed to load library ios.

I'm working on it, the problem is that the undreamai library is unsigned.
One workaround is to sign the library yourself (undreamai_iOS.dylib).
I'm trying to now implement it as a static library instead of dynamically linking it so that it is baked into the app and doesn't need external signing.
If you are interested, there is a thread on the ios channel on discord.

Thanks, I read the thread. I am not sure about how to sign it? With the command you shared on discord? From  terminal, or from Xcode, the one located in unity streaminga, or xcode build?

I will ask there not sure.
Otherwise static linking should be done very soon (this week) :+1:

I got these errors, xcode build fails

is this with the feature/static_library_linking_on_mobile branch?

I use this command to build the project and it works for me:
xcodebuild clean archive
-project Unity-iPhone.xcodeproj
-scheme Unity-iPhone
-configuration Release
-destination "generic/platform=iOS"
-archivePath $PWD/build/Unity-iPhone.xcarchive
CODE_SIGN_IDENTITY="" CODE_SIGNING_REQUIRED=NO CODE_SIGNING_ALLOWED=NO DEVELOPMENT_TEAM=<MY_TEAM_ID>

is this with the feature/static_library_linking_on_mobile branch? do you have any errors during the ios build in unity? Can you check if you have a libundreamai_ios.a file inside Libraries/Plugins/iOS/LLMUnity/undreamai-v1.2.0-dev-llamacpp of the xcode project created?

Yes it was

I use this command to build the project and it works for me:
xcodebuild clean archive \
            -project Unity-iPhone.xcodeproj \
            -scheme Unity-iPhone \
            -configuration Release \
            -destination "generic/platform=iOS" \
            -archivePath $PWD/build/Unity-iPhone.xcarchive \
            CODE_SIGN_IDENTITY="" CODE_SIGNING_REQUIRED=NO CODE_SIGNING_ALLOWED=NO DEVELOPMENT_TEAM=<MY_TEAM_ID>

I don't think app store allows games without signing. I don't use terminal commands, I build from xcode, and it asks for a development team. It builds with the main branch, but can't find llm, build problem only with this one.

Sure, you can sign it.
I do it like that for testing purposes, I don't have a developer account.
With the main branch the library is shipped inside the project and dynamically loaded at runtime, while with this branch it is embedded and build with the project.
Do you find the libundreamai_ios.a inside the xcode project?

No, it is not there, I will try a clean build again now


ci: Upgrade to Node 18 while running GitPod

Fixes Issue #8513

Changes proposed
Uses nvm to upgrade the node version to 18 which supports structeredClone() command, before it was using node 16 which was not recognizing it.

Check List (Check all the applicable boxes)

[x] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[ ] I have updated the documentation accordingly.
[x] All new and existing tests passed.
[x] This PR does not contain plagiarized content.
[x] The title of my pull request is a short description of the requested changes.

Thank you!


Flatten files (slug) and use a siderbar config to categorize documents

So that we can keep the slug persistent from then on.
There're some references should be updated correspondingly.

Closed by https://github.com/apache/pulsar-site/pull/295.

Reverse links:

https://github.com/apache/pulsar/pull/18508
https://github.com/apache/pulsar/pull/18509


Notification won't dismiss

I have made the notification display upon failure of a network request. If I send the request 2 times, or more, in a row, the notifications gets stuck at the top and won't dismiss until the user restarts the app.

Can you give a little more information like some of the options you're using to display the toast notification? It almost sounds like this may be related to https://github.com/cruffenach/CRToast/issues/154 but I want to verify you're using the same type of properties for displaying before i jump to any conclusions

I'm seeing the same issue.
I switched to :head as 0.0.7 displays incorrectly on an iPad in landscape mode.
This now displays correctly - but doesn't auto-dismiss.
NSDictionary *options = @{
                          kCRToastTextKey : message,
                          kCRToastTextAlignmentKey : @(NSTextAlignmentCenter),
                          kCRToastBackgroundColorKey : [UIColor lightGrayColor],
                          kCRToastTextColorKey : [UIColor orangeOff],
                          kCRToastAnimationInTypeKey : @(CRToastAnimationTypeGravity),
                          kCRToastAnimationOutTypeKey : @(CRToastAnimationTypeGravity),
                          kCRToastAnimationInDirectionKey : @(CRToastAnimationDirectionTop),
                          kCRToastAnimationOutDirectionKey : @(CRToastAnimationDirectionTop)
                          };

[CRToastManager showNotificationWithOptions:options
                            completionBlock:^{

                            }];

to confirm the relation to #154 changing to CRToastAnimationTypeSpring resolves the issue.

@ConfusedVorlon Which animation should he change to CRToastAnimationTypeSpring?

I can also confirm the stuck notification as reported here and #154, using the current master (3f7d99e2).
It only gets "stuck" when a second notification arrives while the first is still visible, and if kCRToastAnimationInTypeKey and kCRToastAnimationOutTypeKey are BOTH set to CRToastAnimationTypeGravity.
Changing either the AnimationIn or AnimationOut keys to Spring makes the problem go away. Using CRToastManager dismissNotification: fixes it as well... it definitely isn't hung, just failing to clear.


gotemplate: 3.10.1 -> 3.11.0

This is a version bump with performance improvements.
changelog

Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] 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] (unnecessary: no dependents) 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/)
25.05 Release Notes (or backporting 24.11 and 25.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.

Add a :+1: reaction to pull requests you find important.

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 359549

x86_64-linux

:white_check_mark: 1 package built:

gotemplate

aarch64-linux

:white_check_mark: 1 package built:

gotemplate

x86_64-darwin

:white_check_mark: 1 package built:

gotemplate

aarch64-darwin

:white_check_mark: 1 package built:

gotemplate

LGTM


Bootstrap

👷 Deploy Preview for thriving-bienenstitch-9950a2 processing.

Name
Link

🔨 Latest commit
f186422559fad5f3e8ed4ce51834837fa5399a3d

🔍 Latest deploy log
https://app.netlify.com/sites/thriving-bienenstitch-9950a2/deploys/642f1011b988af0008ea73f2


Field matching -> dynamic Error Path?

Question
Based on the example https://formly.dev/docs/examples/validation/matching-two-fields/ we can have the errorPath set as below:

This means that I will have the error message in the passwordConfirm field.
But, I need a solution where the matching message appears in the field where I am currently typing/or leaving (for example password field also, if I am editing it and confirmPassword does not match). Is there an example for such solution?

You can return errorPath  inside the fieldMatchValidator, to detect the current active field check the focused one.
export function fieldMatchValidator(control: AbstractControl) {
- return { fieldMatch: { message: 'Password Not Matching' } };
+ return { fieldMatch: { message: 'Password Not Matching', errorPath: 'passwordConfirm' } };
}

Thanks, it is working, just I am not able to get the focused field, is there any way to do this?

export function fieldMatchValidator(control: AbstractControl, field: FormlyFieldConfig) {
  const { password, passwordConfirm } = control.value;

  console.log(
    field.get('password')?.focus,
    field.get('passwordConfirm')?.focus,
  );
...
}


feat: fix lora name bug, and generate on cloud not change error

Description
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Fixes # (issue)
Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

We identified new issues on unchanged lines of code. Navigate to the Amazon CodeGuru Reviewer console to view the recommendations to fix them.


poetry + pre-commit single source of dependencies?

https://github.com/floatingpurr/sync_with_poetry
https://stackoverflow.com/questions/70127649/how-to-have-a-single-source-of-truth-for-poetry-and-pre-commit-package-version
https://github.com/python-poetry/poetry/issues/2457


What's the difference between EntityFramework.sln and  EntityFramework-xproj.sln

What's the difference between EntityFramework.sln and  EntityFramework-xproj.sln

One is a traditional csproj solution, the other is an xproj solution.


how to get displacement coordinates?

Hello, how can I get the X,Y displacement position of my subsamplingImageView, I need to cut the visible part of the image and I can't get those missing values, thank you very much.
val currentXOffset = subsamplingImageView.translationX.toInt()
val currentYOffset = subsamplingImageView.translationY.toInt()
give me 0
Thanks

If you're looking for a zoom image component on Android, take a look at my ZoomImage (https://github.com/panpf/zoomimage) library. It can be used for View and Compose, supports scaling, panning, positioning, rotation, and super-large image subsampling. It also integrates image loaders such as sketch, glide, picasso, coil, etc.
如果你正在寻找 Android 上的变焦图像组件，可以看看我的 ZoomImage (https://github.com/panpf/zoomimage) 库。它可用于 View 和 Compose，支持缩放、平移、定位、旋转、超大图子采样，它还集成了 sketch、glide、picasso、coil 等图像加载器


locale error when open program

[ ] I think my problem is NOT with youtube-dl
[x] I've verified and i assure that I'm running yt-dlg 1.X.Y
[x] I assure that i am using the latest version of youtube-dl
[x] Searched bugtracker
[x] I've read the FAQs file

What is the purpose of your issue?

[x] Bug report
[ ] Feature request (request for a new functionality)
[ ] Question
[ ] Other

Bug report
03:59:23: Cannot set locale to language "English (U.S.)".
03:59:23: locale 'en_US' cannot be set.

What operating system do you use ?
Just installed arch, running mate desktop (but am not very tech/just come from Mint).
List of actions to perform to reproduce the problem:

open the gui

What is the expected behaviour ?
selecting 'English' within settings wouldn't cause an error about locale
What happens instead ?
pop-up every time says locale 'en_US' cannot be set (am in UK)
I've tried opening settings.json, and the python file, changing en_US to en_UK, but it reverts when close yt-dlg-gui.  Log in .config is empty.
Ran update, and the cli updated, but the pop-up still happens.
locale -a ...
C
en_GB.utf8
POSIX
Have checked /etc/locale.gen, and en_GB.UTF-8 UTF-8 is set
Thanks for including yt-dlp as a backend, too.

You can provide more information (screenshots, Logs ...) or sequences of steps for reproduction the error ?

Thanks for your response, oleksis :-) No issue now.


Add 6.3.5-SNAPSHOT to pom file

Codecov Report

Merging #530 into dev will decrease coverage by 0.07%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev     #530      +/-   ##
============================================
- Coverage     46.74%   46.66%   -0.08%
+ Complexity     2226     2224       -2
============================================
  Files           108      108
  Lines         25308    25308
  Branches       4175     4175
============================================
- Hits          11829    11810      -19
- Misses        11452    11470      +18
- Partials       2027     2028       +1

Flag
Coverage Δ
Complexity Δ

#JDBC41
46.45% <ø> (-0.15%)
2216 <ø> (-1)

#JDBC42
46.45% <ø> (-0.02%)
2211 <ø> (-8)

Impacted Files
Coverage Δ
Complexity Δ

...om/microsoft/sqlserver/jdbc/ReaderInputStream.java
44.94% <0%> (-1.13%)
16% <0%> (-1%)

...c/main/java/com/microsoft/sqlserver/jdbc/Util.java
61.19% <0%> (-0.67%)
88% <0%> (-1%)

...n/java/com/microsoft/sqlserver/jdbc/Parameter.java
62.01% <0%> (-0.43%)
63% <0%> (ø)

...m/microsoft/sqlserver/jdbc/SQLServerResultSet.java
32.53% <0%> (-0.33%)
240% <0%> (-1%)

...rc/main/java/com/microsoft/sqlserver/jdbc/dtv.java
63.05% <0%> (-0.19%)
0% <0%> (ø)

...in/java/com/microsoft/sqlserver/jdbc/IOBuffer.java
54.12% <0%> (-0.14%)
0% <0%> (ø)

...rc/main/java/com/microsoft/sqlserver/jdbc/DDC.java
44.92% <0%> (ø)
104% <0%> (-1%)
:arrow_down:

...om/microsoft/sqlserver/jdbc/SimpleInputStream.java
52.59% <0%> (+1.48%)
12% <0%> (+1%)
:arrow_up:

Continue to review full report at Codecov.

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


fix: multiline notes

Is this going to get merged sometime? Or is there another workaround?

I realized that this has already been fixed in production. I think the pull request should be closed now.