Kestrel (or something I don't know) caches my API

I've created a bunch of services using ASP.NET Core. I've hosted them in IIS using AspNetCoreModule. Nowhere I've used caching. In fact, I'm not familiar with the concept of caching at all in ASP.NET Core.
I've published this service to two different servers. Both have this environment:
Windows Server 2012 R2 Standard
IIS 8.5.9600.16384
The problem is that one works just fine, the other caches my API requests. I've compared IIS configuration of two servers by comparing Windows\System32\inetsrv\config folders, and they are the same.
Also Web.config files are the same on both machines. I'm stuck at this point on how to find and solve the problem. The problem should be environmental, since I've not configured anything related to caching in my codebase. In other words, codebase is the same across two machines. So that can't be the source of the caching problem.
Is Kestrel the culprit here? How can I make sure that Kestrel is not caching my API responses without me knowing it?
This is the response header on when it's not cached:
HTTP/1.1 200 OK
Content-Type: application/json; charset=utf-8
Vary: Origin
Server: Kestrel
Access-Control-Allow-Credentials: true
Access-Control-Allow-Origin: http://admin.vas.taak.io
Date: Mon, 01 Oct 2018 11:37:44 GMT
Content-Length: 8804

Kestrel does not do any response caching. Do you have full control over both machines?  Can you try a third machine (perhaps a VM you can fully control) to try out?

Are you testing this by using a browser to see the API response? If so with IE in a corporate environment we find you need to explicitly set the no caching headers to stop the browser from being thick...

+1 to what @Drawaes said as well.

I have control over both machines, and I've also tested on other machines too. They are from the same datacenter, and they have similar specs, just different IPs.
Also the software setup was the same for both of them using a server-setup-checklist.
Other machines are OK.

I'm using a browser. But if that was the problem, then it had to act the same for other servers and endpoints too.
It can't be an issue of the browser.

Just to eliminate the issue try setting the header on the response
Cache-Control: no-cache, no-store
And see if it makes a difference. Before trying clear your browser cache. I have seen off behaviour and it's a quick test to rule it out.

Well, using @Drawaes point, and using these links, I managed to add required headers to prevent caching explicitly. Which by the way is a shame I guess for Microsoft and ASP.NET Core.
https://stackoverflow.com/questions/49547/how-to-control-web-page-caching-across-all-browsers
https://stackoverflow.com/questions/29503338/add-a-header-to-all-responses-in-asp-net-core-mvc

It stopped the caching then ?

yeah it actually did

Odd so the question now is why that caused you a problem for one server and not the other. I have seen browsers do odd things though if it's an API a browser is going to hit I normally brutally turn on all the didn't cache flags automatically to save the headaches that always come later otherwise  (caching makes it hard especially if you don't have access to the person/browser that is having the problem).


Get some defpackage

For example
import defpackage.ul$a;
import defpackage.ul$b;
import defpackage.uo;
import defpackage.up;
import defpackage.vd;
import defpackage.vj;
import defpackage.vk;
import defpackage.vl;
import defpackage.vm;
import defpackage.vm$1;
import defpackage.vq;
import defpackage.wd;
import defpackage.wl;
import defpackage.wr;
import defpackage.ws;
import defpackage.wt;
import defpackage.wv;
import defpackage.xf;
import defpackage.xm;
import defpackage.yf$e;
import defpackage.yf$h;
import defpackage.yf$j;

Same issue here. @skylot Can you please explain what is this.

The app is obfuscated and the "defpackage" is the default package "" (the packe without name).
The problem is that in the Jadx treeview an entry without name is not (or really difficult) clickable. Therefore "defpackage" is used as replacement.

Actually "defpackage" used for another reason: code will be not compilable if class in default package use any class from other packages, but such classes often use outside classes, so this is easy way to make code compilable.


Admin page: user flagging overview shows 0 "Flags used today"

Minor bug: "Flags used today" always reports 0, but that's not the number that's actually been used in that UTC day. The error on that page is regardless of the user, but here is a link to the page for me.

Yeah, that's a misleading field. That's actually displaying a field direct from the database - flag_conditions.flags_today (IIRC), which was originally intended to record flag usage and cutoff when the user-set max was reached. We never ended up actually hooking that field up, though, so it's always zero - but it was put in a bunch of dashboards and never removed. This is one of them.


Revert "Update Dapr Bot Action and respond to users that can't invoke commands"

Reverts dapr/dapr#6367
More work is needed since it causes the bot to respond to itself.

one option is not responding when the comment is not a command

Yeah that’s my plan. I’ll fix this on Wednesday. Sorry


⚠ SceneTime has degraded performance

In d796b13, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 2012 ms

Resolved: SceneTime performance has improved in 5c1be8b after 6 minutes.


Sidebar collapses on dragging tab

Whenever I hold a tab to rearrange it, the sidebar collapses, pretty sure this is not desirable

Can you send me your list of used mods, I can't reproduce this in any of my test profiles

There is only 1 other mod, which I have disabled for testing this, and it anyways just changes the extension icon.
My mod settings-

Screen capture of the issue
https://github.com/user-attachments/assets/7d49a75d-110c-4c37-b184-40a4d5be267d

Were you able to reproduce this?


Invalid dovetail overlaps orientation in GFA2 Edges

The recognition of dovetail overlaps in GFA2 edges has been done has follow, considering the positions
of the two sequences (beg1, end1, beg2 and end2 respectively) the edge represents a dovetail overlap
if: beg1=0 or end1=x$ and beg2=0 or end2=y$. If this is true then extremes taken into account
are computed as the one described in GFA1, if segment1 is + then from_segment_end is R otherwise
is L. If segment2 is + then to_segment_end is L otherwise R.
This is not correct in GFA2 since a dovetail overlap could be, erroneously, such this one:
E    l12    1+    2+    0    3    0    3    *
So this situations must be solved in a different way then the one used by GFA1.

I have analyzed this situation more carefully.
E l12 1+ 2+ 0 3 0 3 *
It's not a dovetail overlap situation, but it's a partial (general) overlap correctly described
by a generic edge line. The GFA1 specification couldn't describe this situation.
So, I've read again the specification and I have changed the assumption for a dovetail overlap
in GFA2 edges.
The statement:" For example a GFA edge which encodes what is called a dovetail overlap (because two ends overlap) is a GFA2 edge where either beg1 = 0 or end1 = x$ and either beg2 = 0 or end2 = y$."
Is incorrect to me, since could lead to the situation described in this issue post.
The interpretation I have used is:
"An Edge encodes a dovetail overlap if beg1 = 0 and end2 = y$ (so the end of the second segment
overlaps with the begin of the first one) or end1 = x$ and beg2 = 0 (this is the normal Link case)."
So, after this preamble, I have to check which of the two segments is the "from_segment" (regarding the
dovetail overlap) and apply the sign rules as Link describes.
The code for this is the following (after checking that the edge is a dovetail overlap):
    def _set_segments_end_link(self):
        if self.from_orn == "+":
            self._from_segment_end = "R"
        else:
            self._from_segment_end = "L"
        if self.to_orn == "+":
            self._to_segment_end = "L"
        else:
            self._to_segment_end = "R"

    def _set_segments_end_edge(self):
        beg1, end1 = self.from_positions
        beg2, end2 = self.to_positions

        # the dovetail is from the end of from_node to
        # the beginning of to_node, just like a GFA1 Link
        if beg2 == "0":
            self._set_segments_end_link()
        else: # dovetail between end of to_node and begin of from_node
            if self.from_orn == "+":
                self._from_segment_end = "L"
            else:
                self._from_segment_end = "R"
            if self.to_orn == "+":
                self._to_segment_end = "R"
            else:
                self._to_segment_end = "L"


Client error handling

Followup discussion space (and reminder) for #4755 .

related: #5131

exists partially, more specific client error handling will be part of OS4


[] build failure

Host Environment

OS: [e.g. Windows/Linux etc...]
Compiler: revision

To Reproduce
Steps to reproduce the behavior:
./vcpkg install xxxx
Failure logs
-Cut and paste the appropriate build messages from the console output.
-Please attach any additional failure logs mentioned in the console output.
Additional context
Add any other context about the problem here, such as what you have already tried to resolve the issue.
error: building botan:x86-windows-static failed with: BUILD_FAILED
Please ensure you're using the latest port files with git pull and vcpkg update.
Then check for known issues at:
https://github.com/microsoft/vcpkg/issues?q=is%3Aissue+is%3Aopen+in%3Atitle+botan
You can submit a new issue at:
https://github.com/microsoft/vcpkg/issues/new?template=report-package-build-failure.md&title=[botan]+Build+error
Include '[botan] Build error' in your bug report title, the following version information in your bug description, and attach any relevant failure logs from above.
vcpkg-tool version: 2022-07-21-a0e87e227afb536c62188c11ad029954f28fdb22
vcpkg-scripts version: 281d2f2dc 2022-07-29 (4 hours ago)
You can also use the prefilled template from C:\src\vcpkg\installed\vcpkg\issue_body.md.

Thanks for posting this issue. Please provide the error log and your environment information.

Thanks for posting this issue. Please reopen this issue if this is still a problem for you.


Low VRAM after running

My 2GB GPU is capable of running Stable Diffusion.
This script's GPU support also works just fine after generating an image.
After using the script, however, subsequent Stable Diffusion runs fail due to VRAM unavailability. Is the model persisting in VRAM after use? If so, can it be freed between uses?

Hi, thanks for reporting. I'm still pretty new at python and was assuming everything would be garbage collected when it goes out of scope, including the pytorch model in vram.
Some googling seems to confirm this. (https://pytorch.org/docs/stable/notes/cuda.html#cuda-memory-management)
I'm looking into it further ..

I will specifically delete the model before returning from the script in the next version, so we don't need to wait for the GC to kick in.

In v0.1.7 the model is deleted (freed) at the end of the script now, did that fix the problem ?

I had a similar issue while using your script in txt2img on a 8gb vram card yesterday (just out of memory error and couldnt generate any images anymore). Just tested it with v1.7 and it does not seem to happen anymore.

I'm closing this issue as the problem seems to be solved.


[WIP] Spotlight: search for users.

@RocketChat/android
This PR adds the search for users on Spotlight. Currently, the Spotlight shows only rooms.
Progress

[x] Fetch users with the searched term.
[x] Show the correct/related (at) icon for users.
For example:
@ john.doe
@ john.lennon
[x] Create a DM with the selected user.
[x] Unit tests.

Nice, will review it tomorrow morning.

Thank you @luciofm !
It is in progress yet. I'll add a task list to show you the PR progress. Of course, feel free to review the files (code style, better code logic, etc).

With this change will it also search rooms you are already part of?

@geekgonecrazy Maybe not in this PR. Probably I'll create another PR w/ that improvement.

Only refactoring before the merge...

@geekgonecrazy Maybe not in this PR. Probably I'll create another PR w/ that improvement.

Ah ok.  I was getting excited.  Everyone I need to DM I've already done so in the past.  So not as exciting for me :grin:
But good to have the other stuff working

Merging after review and tests...


release: v1.0.0a6

Signed-off-by: Harris Tzovanakis<EMAIL_ADDRESS>

@lnielsen

remote: error: You're not authorized to push to this branch. Visit https://help.github.com/articles/about-protected-branches/ for more information.

:(

should be fixed now, please try again


Different semantics of casting from int64 to timestamp between Comet and Spark

What is the problem the feature request solves?
In Spark, casting from LongType to Timestamp is to casting seconds to microseconds. arrow-rs's cast kernel basically simply reinterprets Int64Array as TimestampMicrosecondArray. So the scale is different, i.e., seconds v.s. microseconds.
We need to add this special case into Comet's cast kernel.
Describe the potential solution
No response
Additional context
No response

This cast is currently disabled by default, but it would be good to implement this. I have added this to the 0.2.0 milestone.

I can take this

Thanks @justahuman1!
A good place to start is org.apache.comet.expressions.CometCast#canCastFromLong. This currently returns false for casting to timestamp, so enabling it here is the first step.
Next, you can enable the following test in CometCastSuite:
  ignore("cast LongType to TimestampType") {
    // java.lang.ArithmeticException: long overflow
    castTest(generateLongs(), DataTypes.TimestampType)
  }

Also check CometExpressionSuite.test("cast timestamp and timestamp_ntz"). This reads timestamps as longs from a parquet file (which may store the values as either millis or micros).


no docs on how to add captions or change css

You show how to display captions or not, how to change their colors but you do not show how to add the text of captions or change their location within the slideshow please adjust docs to show thank you.

Currently, there is no way to change the location of the captions. What is the placement you are looking for?
As far as the contents of the captions, you must pass in an array of type IImage which is linked from the docs, but it is only implied I suppose. I will make it more clear. Thank you for pointing this out.

Whoops, didn't mean to close this!

Okay, I am going to close this due to lack of response. However, feel free to respond still.


Release v0.3.5

Problem
Commit 08f5710e085e55c038b8555032c90a31d7c91037 fixed an issue where the bls12_377 Fr and Fq comments had a non-ascii character.
noir_rsa is dependent on this library and will not compile with aztec-nargo without this dependency included (regular nargo is fine for some reason)
Obviously this is resolvable by pointing to local clones for each repo/ forking but this is not as desirable
Happy Case
Release tag v0.3.5 including at least up to commit 08f5710e085e55c038b8555032c90a31d7c91037 so that noir_rsa v0.3.1 can import it for use in aztec-nargo
Workaround Description
Clone downstream repositories and change dependencies, potentially pushing to forked library if critical

This release was made in #26


Add support for authentication against eventstore

Fairly dumb suggested implementation of authentication for atomic puppy after 5 minutes looking at the code. Comments, suggestions and criticism welcome.
Note on EventFinder:
auth object setup as part of constructor and not find_backwards (where it is needed) because it feels weird to pass the creds everytime you call find_backwards

next thing to do would be write tests for it. not sure how AP does those tho, does it have an actual eventstore in a container?

Not sure why it wouldn't let me comment directly to that comment, however, @hjwp leave this with me I will have a look at the tests over the next week or so when I get some spare time. Will ping both you and @epoplavskis when I've got something for you guys to have a look over

merges got mangled by a rebase done without paying attention. New PR raised. Closing PR and removing branch for clarity


[SPARK-10393] use ML pipeline in LDA example

jira: https://issues.apache.org/jira/browse/SPARK-10393
Since the logic of the text processing part has been moved to ML estimators/transformers, replace the related code in LDA Example with the ML pipeline.

Merged build triggered.

Merged build started.

Test build #41868 has started for   PR 8551 at commit 84cc1be.

Test build #41868 has finished for   PR 8551 at commit 84cc1be.

This patch passes all tests.
This patch merges cleanly.
This patch adds the following public classes (experimental):
class DCT(JavaTransformer, HasInputCol, HasOutputCol):
class SQLTransformer(JavaTransformer):

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

Merged build finished. Test PASSed.

@feynmanliang Thanks for the review. I'll update later today.

Merged build triggered.

Merged build started.

@feynmanliang @jkbradley Sorry for the late update.

Test build #43561 has started for   PR 8551 at commit ee4832d.

Test build #43561 has finished for   PR 8551 at commit ee4832d.

This patch fails MiMa tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

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

need to trigger the test again?

Test build #2026 has started for PR 8551 at commit ee4832d.

Test build #2026 has finished for PR 8551 at commit ee4832d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

unrelated UT timeout failure:
SPARK-11188 Analysis error reporting *** FAILED *** (2 minutes)
[info]   =======================
[info]   CliSuite failure output
[info]   =======================
[info]   Spark SQL CLI command line: ../../bin/spark-sql --master local --hiveconf

Jenkins retest this please

Merged build triggered.

Merged build started.

Test build #45515 has started for PR 8551 at commit ee4832d.

Test build #45515 has finished for PR 8551 at commit ee4832d.

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

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

Merged build finished. Test PASSed.

Test build #2177 has started for PR 8551 at commit ee4832d.

Test build #2177 has finished for PR 8551 at commit ee4832d.

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

Those 3 items (thanks @BenFradet ) are the only issues I see.  Thanks!

Thanks @jkbradley @BenFradet for the review.

Test build #47324 has started for PR 8551 at commit 672ce97.

Test build #47324 has finished for PR 8551 at commit 672ce97.

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

Merged build finished. Test PASSed.

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

LGTM.  Merging with master and branch-1.6
Thank you!


osmohlr is exiting with error " Error opening database "hlr.db""

Hi @herlesupreeth ,
I am able to install and setup VoLTE setup, successfully verified audio & video calls. but after couple of restarts osmohlr is not coming up.

thank you
Ganesh

it was working after removed hlr.db and rebuild hlr
thank you
Ganesh


Update project, pbxproj

This issue is from Blue Wallet release note.
bluewallet-selftest.txt
-Update project, pbxproj
-Fix: Wallet addresses were not visible
-Fix: Send transaction crash #6751

OPS: Pods
TST: 22 e improvements
Fix: sein language files
Translate loc/en.json in pl 9009 reviewed
source file: loc/en.json
Update dependency realm to v12.11.1
OPS: version bump
Update dependency @ react-native/metro-config to vo.74.84
Fix: BIP 47 addresses subscribed to push notifications

Please, help me Update  the project and Fix issues stated in the Blue Wallet release note. Thank you.

Help update the project and Fix issues in the Blue Wallet release note.


Add thebelab button to every code cell

This is something that I found useful for my jupyter book. Feel free to decline if you think this is an unnecessary feature upstream.
This fixes #117

Ok, I just realized that this also adds a thebelab button to code cells in pure markdown files, e.g. the ones on this page: https://jupyter.org/jupyter-book/guide/05_advanced.html. I am not sure how I can differentiate the two cases...

hey @joergbrech - I quite like it! I think that you could narrow down the CSS selection by using the > selector, which requires a specific set of parent/child elements.
For example, check out how this is done for the code cell CSS:
https://github.com/jupyter/jupyter-book/blob/master/jupyter_book/book_template/_sass/components/_components.textbook__page.scss#L46
Here's the built docs from this PR:
https://948-137292803-gh.circle-artifacts.com/0/html/features/interact.html
(if you click on the edit button, the CSS will look weird in the code cells, but that's just a bug that has already been squashed)
would love to hear what @emdupre or @jasmainak thinks about it!

Wow, that was easy. Thanks for the hint!

Wow, that was easy. Thanks for the hint!

do we have an example book with these buttons? :-)

do we have an example book with these buttons? :-)

There you go : https://960-137292803-gh.circle-artifacts.com/0/html/features/notebooks.html
The button is the small pencil in the top right corner of the code cells

I get "/bin/sh: 1: jupyter-book: not found" when I run the cells. But it was probably an old problem ...

The line spacing is off when I click the pencil; not sure if it's this PR or a current issue. I remember that this happened before and was resolved.

Also note that the vertical line is my typing cursor position.

@mathieuboudreau Can you please try again? I updated the link in my comment. It should now include the latest jupyter book changes where the messy lines are fixed.
@jasmainak This only refers to the last cell on that page right? If I run thebelab on the master version of that page https://jupyter.org/jupyter-book/features/notebooks.html I get a similar error with slightly different behavior:
Before pressing "Thebelab":

After pressing "Thebelab":

This can prabably be fixed by using a markdown code cell rather than a ipybn code cell in this file: https://github.com/jupyter/jupyter-book/blob/master/jupyter_book/book_template/content/features/notebooks.ipynb

My issue was resolved! thanks!

hmm ... but you are getting the error on the next line where it's expected.
I will try again later, need to run now. Thanks for the PR!

Ah, sorry, you are right, I scrolled to the wrong code cell after starting Thebelab. Sorry for the confusion!
The error message has nothing to do with this PR, jupyter-book is simply missing in the binder requirements.

I am a big fan of this ! Thanks so much @joergbrech ! :sparkles:
Can we also update the documentation to reference these ? I think it would make sense here, but would love to hear other ideas !

ok, I think this is quite nice from a UI standpoint (minimal visual change, but obvious behavior upon change), so let's give it a merge and see how it feels to people!

thanks so much @joergbrech for the improvement!


pcre: 8.37 with patches -> 8.38

New version of pcre, patches to old version removed.

@vcunat added the 1.severity: mass-rebuild label to the original patch proposed, and @peti suggested to move to staging branch:
https://github.com/NixOS/nixpkgs/pull/12855


Added driver for Predator-HP

Added wrapper script for Predator - Hunting Party, a verification tool.

When I merged this, I did not notice that there is no license header in your file.
Once could assume that you meant to distribute this under the Apache 2.0 License because the whole project has this license, but an explicit statement would be better.
Thus, could you please add an appropriate header?
If possible, we would prefer the Apache 2.0 License like in the other files.

Not a problem, I will add the license header.


Feature/81-Navbar suggestions

Describe the solution you'd like
The aim of this issue it is to create the suggestions screen. A form containing the name in a input box (not mandatory) and a input box for comments so that suggestions can be done.
The information inside the form of suggestion should reach the API of suggestions.
Describe alternatives you've considered
Not applicable
Additional context
Not applicable

Page view created, waiting for the corresponding backend tasks to be ready so I can continue with the data submission.

Story Points: 8
See the discussion in Frontend S2
Powered by Parabol


[Launcher]: Add state machine doc

Coverage increased (+0.01%) to 53.036% when pulling 5a615768ae3ad0229064174d2508de27c1729604 on yqwang/launcher-dev into 75251410b43fcc33665666a4497158450a0787fa on errorspec.


wireplumber

Project description
https://gitlab.freedesktop.org/pipewire/wireplumber
"Session / policy manager implementation for PipeWire"
I have a WIP for this.
It's blocked on these two things:

https://gitlab.freedesktop.org/pipewire/wireplumber/-/issues/20 (wireplumber build issue)

https://github.com/skystrife/cpptoml/pull/124 (PR so that cpptoml outputs pkg-config files)

WIP wireplumber: https://github.com/colemickens/nixcfg/blob/e552bb209e9fc9e09e71ba1ca7f5067cb5a010fc/pkgs/wireplumber/default.nix

WIP cpptoml: https://github.com/colemickens/nixcfg/blob/e552bb209e9fc9e09e71ba1ca7f5067cb5a010fc/pkgs/cpptoml/default.nix

Metadata

homepage URL: https://gitlab.freedesktop.org/pipewire/wireplumber
source URL: https://gitlab.freedesktop.org/pipewire/wireplumber
license: MIT
platforms: linux

Is it even necessary? I don't think I saw any differences when compared to pipewire's internal session manager. At least back when I used wireplumber, it was at least more stable but compared to the current internal manager it's far worse.
Last I checked, I couldn't even manage to get bluetooth working in wireplumber (2020-10-22).

pipewire's internal session manager

ah, pardon my ignorance, but I don't know what this is and when I google that phrase I just find wireplumber (I'll keep looking).
It's entirely possible this isn't needed or is redundant, I just don't really know. I was just looking for something to visually wire up sources and sinks, which I thought was possible with pipewire.

Found this: https://blogs.gnome.org/uraeus/category/pipewire/

Currently this is a simple sample session manager that Wim created himself, but we also have a more advanced session manager called Wireplumber being developed by Collabora, which they developed for use in automotive Linux usecases, but which we will probably be moving to over time also for the desktop.

The internal session manager is just referring to pipewire-media-session if I haven't remembered incorrectly.
You're looking for JACK utilities, I don't think wireplumber provides a GUI either.

I packed this anyway and have it building, I'll send it at some point, until then it's here: https://github.com/colemickens/nixcfg/blob/689413b38a596f9ac88984f4d797ef6cc8a29285/pkgs/wireplumber/default.nix
In the meantime, and related to @eadwu's hint, now that JACK-y things seem to be working better, I can indeed do some visual input/output routing in qjackctl.

I packed this anyway and have it building, I'll send it at some point, until then it's here: https://github.com/colemickens/nixcfg/blob/689413b38a596f9ac88984f4d797ef6cc8a29285/pkgs/wireplumber/default.nix
In the meantime, and related to @eadwu's hint, now that JACK-y things seem to be working better, I can indeed do some visual input/output routing in qjackctl.

For those like me who were confused about the difference between WirePlumber and the default pipewire-media-session:

PipeWire upstream has a very limited example session manager. It serves as a good example for building new ones and has some functionality there for basic desktop use cases and testing, but it goes no further than that. WirePlumber serves as a replacement for this example and additionally provides a framework for building custom session managers.
The main goal of WirePlumber as a session manager is obviously to watch for streams from applications and make sure that they get linked to the appropriate device or peer application according to the rules of the use case that it implements.

Source
I think it is still desirable to have this packaged in nixpkgs in the long term, although I personally have no use for it right now (just very curious).

When this gets packaged it should be integrated in the pipewire NixOS module alongside the current pipewire-media-session. Haven't bothered looking into it yet since there hasn't been a release in ages and I'm fairly sure the previous release doesn't work with the current pipewire.
And it is definitely desirable to package this, since wireplumber supports scripting for smart automatic routing in fancier setups that aren't served sufficiently with only a single default card for input and output respectively. So essentially everyone who would use the pro audio profile on their main soundcard.
Checking on the releases page, looks like there is a 0.4.0 release coming up soon. I'll look into it once that lands (unless someone beats me to it)..

It seems 0.4.0 is here and highly compatible with the current pipewire versions, could be a good time to check this issue out again.

I have something that works here. I have a few notes:

By default, it will use $out/etc/wireplumber as its "config dir", $out/share/wireplumber as its "data dir" and $out/lib/wireplumber-0.4 as its "module dir". These can be overridden using the WIREPLUMBER_CONFIG_DIR, WIREPLUMBER_DATA_DIR and WIREPLUMBER_MODULE_DIR environment variables. I think we can leave the defaults as-is, and override these variables in the systemd unit. This would allow one to run ./result/bin/wireplumber without setting anything up and have it use the default configuration.
/etc/wireplumber contains four .conf files, but these are separate example configurations (they don't depend on each other)
List of environment variables that change the behaviour of WirePlumber

WIREPLUMBER_DEBUG (debug level, default is 2 and can be changed in the configuration file)
WIREPLUMBER_NO_PW_LOG (Don't replace PipeWire's logging system with WirePlumber's (I think))
WIREPLUMBER_MODULE_DIR (sets module directory, used for searching for modules)
WIREPLUMBER_CONFIG_DIR (sets config directory, used for loading configuration files and scripts (in scripts subdirectory)
WIREPLUMBER_DATA_DIR (sets data directory, also used for finding scripts (in scripts subdirectory) (if $WIREPLUMBER_CONFIG_DIR/scripts didn't contain the file), but nothing else.
WPIPC_DEBUG (wpipc debug level)

I'm guessing wpipc is a wireplumber inter-process communication thing

wpipc sets a variable called runtime_dir to the first one of these that exists:

PIPEWIRE_RUNTIME_DIR
XDG_RUNTIME_DIR
HOME
USERPROFILE
There's a fifth case that I don't understand but it looks like it's reading the runtime directory from a socket

I can create a pull request if someone else wants to be the maintainer.

If anyone wants to try out WirePlumber, the steps are as below:

Build the WirePlumber package
Copy the etc and share directories to somewhere writable: cp result/etc result/share ./ -r

You can alternatively copy result/etc/wireplumber and result/share/wireplumber into somewhere you can write to, you don't need to copy the entire etc directory

Make the new etc and share directories writable using chmod: chmod 755 ./etc ./share -R
Set the config and data dirs: export WIREPLUMBER_CONFIG_DIR="$PWD/etc/wireplumber" WIREPLUMBER_DATA_DIR="$PWD/share/wireplumber"
Stop pipewire-media-session: systemctl stop --user pipewire-media-session
Launch WirePlumber: result/bin/wireplumber [-c configfile]

Default configfile is $WIREPLUMBER_CONFIG_DIR/wireplumber.conf

You don't need to modify any existing PipeWire configuration to try out WirePlumber.
You can then e.g. try changing the volume with result/bin/wpctl or launching scripts with result/bin/wpexec.

There's a new release of WirePlumber, version 1.4.2. I've updated my nur package here.

fwiw I just migrated from pipewire-media-session to wireplumber by writing a module. It was a little complicated by the fact that the only way to configure wireplumber right now is to write lua scripts that generate the desired configuration json data (lua is used purely as a preprocessor so is kind of unnecessary, but is also the only way to pass data to it), so I had to include a nix attrs toLuaTable serializer.

Related to https://github.com/NixOS/nixpkgs/issues/102547

https://github.com/NixOS/nixpkgs/pull/150548 would package a bare-bones wireplumber, although the service is a bit lacking in terms of configuration.

So this can be closed since #150548 has been merged?

Good question. The package is there but it can't really be configured via the nixos module yet.

Wireplumber is ready and it is even the default already (https://github.com/NixOS/nixpkgs/pull/159546), so I am closing this issue.


levenshtein algorithm

What would be cool is to add the levenshtein algorithm to sort these results.

Please ellaborate, but I'm not sure what results you mean, since fuzzysearch just returns a boolean value

+1

@bevacqua The levenshtein algorithm calculates the distance between what the phrase I use to search, and each result.  Find a JS implementation here  - https://github.com/gf3/Levenshtein


Translation system

Description
Why not using Godot's language system to make this game more international. I can help translation for brazilian portuguese, for example.

This seems like a pretty nice idea I'd love to see implemented.
Would you be interested with us on working on an implementation? I would think our English stuff would have to move there as well, correct?

Yes, I'd love to, and yes man, some stuff needs to change, but it's very easy.
A simple csv file, for example could hold all the strings.


Add pod start and stop

As well as added tests, man pages, and completions.
Also reformat and refactor a couple of other small things in the other pod commands.
Signed-off-by: haircommander<EMAIL_ADDRESS>

I might take away "else" from your laptop.  :^)

bot, retest this please

Oh dear, the tests are actually green.

At some point we should consider allowing the IDs of individual containers that were stopped/started to be printed during the output, maybe like so:
podman start abcde
abced

ctr1id
ctr2id
ctr3id

But that would require extra work on the libpod side to expose the state changes that happened.
LGTM, this is a good start.

SGTM, the change would be effectively trivial on the podman side. I wonder if a -q should supress it, and have it print ctr ids by default?

I think we'd have a -q which would print just pod IDs, and maybe a -v for verbose that would print container IDs as well? We could even get fancy and add a -vv for double-verbose that would print the IDs of both containers that started and containers that were already started.

@mheon then what would the default be? I feel like:
-q for pod ID,
nothing for pod and ctr-id
-v for pod and ctr-id along with those already started
if the latter would be useful.

That sounds like a good way of doing it, yeah

I'll focus on the rest of the pod commands first, but I can look into that after.
Speaking of @rhatdan PTAL it'd be cool to get this in, it'll make testing the next commands easier

I'm going to merge this so we can cut a release. Namespaces will wait until Monday.

I'd actually like to add name spaces to pod create and ps anyway so I think it makes sense to wait

Should just work on your side - all the magic happens in the database.

at least with ps I need to manually include it in the output.

I don't think it's worth including in ps output - seems more like an inspect thing to me.

crictl includes it in their pods output, which is our ps. so I think @rhatdan wants it in for CRI-O


Gutenberg: Nested calls to Post::content() cause wpautop filter to be enabled again

This is a bug when working with Timber and Gutenberg.
Some background first: Gutenberg removes the wpautop filter when it renders content that contains Gutenberg blocks: https://github.com/WordPress/gutenberg/blob/14aa7d7d8adf4a3683bdaaf6a7cf154cd598f5fd/lib/blocks.php#L179-L182. After the blocks are parsed, Gutenberg adds the filter back through _restore_wpautop_hook(). The description of that function explains what it does:

If do_blocks() needs to remove wpautop() from the the_content filter, this re-adds it afterwards, for subsequent the_content usage.

In Timber, we get the content through {{ post.content }}. When anything in that content calls Timber\Post::content() again, it will re-apply the wpautop filter too early, causing the whole content of the post to be «auto-peed».
So, for example, if you have a block that renders a list of posts (e.g. a latest posts block) that contain a preview generated through Timber\PostPreview, that preview might call $post->content() to fetch a posts content to generate the preview. After all content filters are applied, it will call _restore_wpautop_hook(), causing wpautop to be activated again, which then causes your the HTML of the outermost call to {{ post.content }} to be wrapped in <p> tags. This leads to very unexpected results and seemingly random <p> tags, that the browser doesn’t know how to handle.
For now, I can circumvent the issue by removing the wpautop filter globally.
remove_filter( 'the_content', 'wpautop' );

This only works well if Gutenberg is enabled for all post types. However, if somebody needs to enable Gutenberg only for some post types, this should be solved differently.

I have also experienced this issue an used @gchtr's suggestion as a fix for now. It seems adding <p> tags was on preview used to be a bug #748 sometime ago and got patched, although I'm not sure if it was happening for quite the same reason.
// functions.php
remove_filter( 'the_content', 'wpautop' );

That issue was resolved in https://core.trac.wordpress.org/ticket/45495 and made it into WordPress 5.2.


add additional deep learning baselines

Add more baseline methods for deep learning to reduce dependence on some many linear methods only.

#144


Minor improvement to English grammar in usage doc

Happened upon a typo/grammar issue while reading the docs.
Goal is to quickly leave things even better than I found them :)
From the automatic checklist:
"Trivial changes like typos do not  require a JIRA issue."
"If your pull request is about ~20 lines of code you don't need to sign an
Individual Contributor License Agreement if you are unsure
please ask on the developers list."
To make clear that you license your contribution under
the Apache License Version 2.0, January 2004
you have to acknowledge this by using the following check-box.

[x] I hereby declare this contribution to be licenced under the Apache License Version 2.0, January 2004

Thanks, Michael (x3). Always impressively fast!


Archive worldwide priorities

FCO want to retire Worldwide Priorities. This PR removes all links to them and the ability for admins to create new ones. It also removes all tests for Worldwide Priorities and refactors some tests which referenced them.
As I was doing this I ran into CI not dropping the DB correctly and noticed that the modified schema from a previous migration had not been checked in. I've added a line to jenkins.sh which will drop, create and migrate the DB on every suite run.
Ticket.

Amended the migration on the feedback from @tekin.

:+1: made a couple of changes. Good to go once build is green.

I think this was a random failure. Re-running on CI.


Extend plugin: Get forcast data from weatherunderground?

First of all: Great work an really beautifull plugin, thanks!
As mentioned in the API documentation of Weather Underground, forecast data could be retrieved. In the plugin documentation, there is 0 for forecasts. I own an PWS and get an api key. When configure forecasts for WU, nothing is shown but now data. So far so good.
=> Will it be possible to extend the plugin to get the forecast data from WU?
Many thanks Lars

I don't own a PWS and therefore cannot test/develop this. The support for the new (to PWS owners limited) Weather Underground API was developed by a user.
I tried to check if the api documentation contains a forecast but can't even access that one without a PWS :( .

If you want forecast data, you can add another stations in the plugin to get the forecast from OpenWeatherMap: https://github.com/naofireblade/homebridge-weather-plus#multiple-stations-configuration

I can take a look at this. I have a PWS, and thus can get a WU account.


Column block

Created a Column block which lets you pull the data out from one column and save it.

LGTM, thank you!


ShadiaKahin's file

I have created a branch, created a file and made a commit, and opened a pull request.

merge new and improved

mergeee

merge pull request plz

merge!!


Bump organization management service version

$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/6688982148

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/6688982148
Status: failure

Already updated


autoscaling_enabled on worker asg does not respect workers_group_defaults

I have issues
I'm submitting a...

[x ] bug report

What is the current behaviour?
Setting autoscaling_enabled using the format described in workers_group_defaults_defaults does not set the correct cluster-autoscaler tags on the asg instances
https://github.com/terraform-aws-modules/terraform-aws-eks/blob/4762908d9bb8079ffebb26da7d806bbc2d5828ff/local.tf#L30
Instantiate the module with:
module "eks" {
  // trimmed ...

  workers_group_defaults = {
    autoscaling_enabled = "true"
  }
}

And the worker ASG launch config is set up with tags that propagate to workers that include:
k8s.io/cluster-autoscaler/disabled: true

If this is a bug, how to reproduce? Please include a code sample if relevant.
Set on of:

Autoscaling enabled in the worker group defaults

  workers_group_defaults = {
    autoscaling_enabled = "true"
  }

Autoscaling enabled to true on the worker_groups variables

  worker_groups = "${
    list(
      map("autoscaling_enabled", "true") # anything other than the value `1` here will fail.
    )
  }"

What's the expected behavior?
Setting either of the following in the module definition:
  workers_group_defaults = {
    autoscaling_enabled = "true"
  }

  worker_groups = "${
    list(
      map("autoscaling_enabled", "true")
    )
  }"

should produce the asg propagate tags: k8s.io/cluster-autoscaler/enabled: true
Are you able to fix this problem and submit a PR? Link here if you have already.
Yes
Environment details

Affected module version: 1.6.0
OS: Linux
Terraform version: v0.11.6

Any other relevant info

I see, correct! Thanks for the issue.
We need some lookup of the default here as a fallback:
https://github.com/terraform-aws-modules/terraform-aws-eks/blob/master/workers.tf#L14

Fixed here: https://github.com/terraform-aws-modules/terraform-aws-eks/pull/130
Can you confirm @zknill ?

@max-rocket-internet tested. It works; thanks!

Great 🎉


[JENKINS-69731] Enhance @Library annotation to load same version (feature branch name) of library as pipeline script from SCM

Detailed in https://issues.jenkins.io/browse/JENKINS-69731
Short summary is: This issue states the desired ability to load at least same "version" (branch name) of the trusted globally configured library as that of the pipeline being executed, with little hassle to scale it on the pipeline coding and Jenkins configuration side.
The PR aims to (optionally, site-configured) allow literally requesting this:
@Library('libname@${BRANCH_NAME}') _
import my.lib.name.class;
...

If the "version" requested in pipeline script is a literal string ${BRANCH_NAME}, the implementation would try to learn the "BRANCH_NAME" environment variable set for the current Run object. If that is present, it would validateVersion() as implemented by this or that LibraryRetriever backend (SCM here, HTTP in workflow-cps-global-lib-http-plugin, etc.) and if the string is not rejected, it would be substituted as the version to use. Otherwise the configured default version would be used as fallback.

[x] Make sure you are opening from a topic/feature/bugfix branch (right side) and not your main branch!
[x] Ensure that the pull request title represents the desired changelog entry
[x] Please describe what you did
[x] Link to relevant issues in GitHub or Jira
[ ] Link to relevant pull requests, esp. upstream and downstream changes
[x] Ensure you have provided tests - that demonstrates feature works or fixes the issue

Tests added so far cover (hopefully) non-regression and interaction of code with the new feature for situations where it is expected to be a no-op. They also test related functionality which existed before but I think was not covered explicitly.
Need to find a way to define a job that is (or believes it is) checked out from Git, so has a BRANCH_NAME envvar defined in a way we need. This would allow implementation of some more relevant tests directly dealing with the added feature (and non-regression over time).

Note: for the time being, functionality of the new feature is confirmable with an externally pushed setting of the envvar, e.g.
BRANCH_NAME=bogus mvn test -Dtest='org.jenkinsci.plugins.workflow.libs.SCMSourceRetrieverTest#checkDefaultVersion'

works as expected (no test failure, job "p3" falls back to "master"), and
BRANCH_NAME=feature mvn test -Dtest='org.jenkinsci.plugins.workflow.libs.SCMSourceRetrieverTest#checkDefaultVersion'

also works as expected (test fails, because job "p3" checks out "feature" and gets unexpected contents since it expects myecho step contents from "master").

Wondering about a couple more nuances here:

Handle pull requests, where BRANCH_NAME="PR-123" is not really helpful, and would cause fallback to default library branch name. Here we can (separately-optionally?) try CHANGE_BRANCH and/or CHANGE_TARGET to poke whether relevant trusted library branch names exist (same as pipeline PR source or target branches respectively).

How many different cases exist? Are there only WorkflowRun jobs that might have a pipeline from SCM (if not "inline") as an ordinary "Pipeline" or a "Multi-Branch Pipeline", and are these guaranteed to set the envvars which I propose to handle here? Or can there be other job types and/or cases when it does not set envvars?

In particular, there is something I am not certain about: I am inclined to add support for Java equivalent of common pipeline groovy queries for "first SCM used in the job" (likely meaning the source of pipeline) like:

BRANCH_NAME = scm.getBranches()[0]

or as I was recently advised, with envvar expansion so shell-templated branch specifications are resolved at run-time:
BRANCH_NAME = scm.getBranches().first().getExpandedName(env.getEnvironment())

The question here is whether the first entry of SCM list is really guaranteed to be the source of the pipeline?

I queried existing build history with Jenkins script console, and the first entry is the library (with component source coming in second), which matches the order of entries on the build overview page as well:

for (def s : Jenkins.instance.getItemByFullName('componentname/master').getBuild('444').getSCMs()) {
  println s.getUserRemoteConfigs()
  println s.getBranches()
}

This may have to do with lightweight checkouts, but if they are not recorded as a source of the job script, it is not helpful for the purpose of this PR. Maybe they are recorded not in the SCM list however?

Corner case of an "inline pipeline" job without an SCM source of its own, which requests several libraries or does other SCM checkouts: as it passes the SCM operations, it does (I checked) populate the List<SCM> roster. This way the first entry of the list has no relation to branch name relevant for the pipeline source to which Library should align (if requested to). Looking for a way to determine that the WorkflowJob is inline...
Worth mentioning that an edited Replay of a build which originally came from an SCM retains that original BRANCH_NAME and scm.getBranches() even though this particular executed source might completely differ from whatever is in SCM. So far treating this as a curiosity; anyways the Replay ability is limited via Permissions to "those who know".

UPDATE: Developing tests for single-branch pipelines to make sure these work as expected.
Got some hiccups so changes to code or tests are anticipated.
Comments welcome in the meanwhile :)

Hopefully done here, ready for review and comments :)
Note that a couple of tests are placeholders (most of the logic is there except for some crucial magic in setup, so they get skipped), hoping someone would find where I failed and would complete them.

Thanks @vicecea for inputs here and in JIRA, I added some more documentation and test improvements for now.

Completed the checkDefaultVersion_MBPsingleBranch_BRANCH_NAME() and checkDefaultVersion_MBPsingleBranch_staticStrings() to do a reasonable job. Missing bit was "running" the MBP to have it define the leaf jobs from SingleSCMSource(s).
Some logical concerns (picking one truth from two possibilities) are now detailed in PR description.

Hopefully done on my side.

Latest build from this PR works for one of the CI farms I manage (and made the feature for this project, to help feature-branch CI jobs+libs development) for half a week now, "flight status normal" so far,

Sounds similar to https://github.com/jenkinsci/workflow-multibranch-plugin/blob/c692ae52371b564c718e993db922dd3aeaa1b441/src/main/java/org/jenkinsci/plugins/workflow/multibranch/ResolveScmStep.java#L61.

Sounds similar to https://github.com/jenkinsci/workflow-multibranch-plugin/blob/c692ae52371b564c718e993db922dd3aeaa1b441/src/main/java/org/jenkinsci/plugins/workflow/multibranch/ResolveScmStep.java#L61.

I have actually seen that, and it also looked like a good point to chisel around, but ended up deciding it is not where (all of) solution should be:

I don't see that part of code knowing what specific branch name to try - how to resolve the envvar or discover pipeline SCM context-dependent name.
By plugin, it is only for multibranch :) And there are "Pipeline script from SCM" cases with singular repo+branch setups, where it makes sense to use. And in case of this PR - also works.

In practice, stable branch names sire single-branch job definitions for us in folders (or other job access permissions) available to whole company. Feature-branches to develop CI logic itself are MBPs visible only to devops-related people and not confusing The Others. However they both come from same code (source text) effectively, as it bubbles up.

Oh... seems like looking into WorkflowJob.getSCMs() was not too prudent: https://github.com/jonsten/workflow-job-plugin/blob/master/src/main/java/org/jenkinsci/plugins/workflow/job/WorkflowJob.java#L539 => https://issues.jenkins.io/browse/JENKINS-40255

The implementation of WorkflowJob.getSCMs() is troublesome. It uses the latest completed build to determine the SCM resource...

Ouch.
With a job that was used for in-place testing (replaying to use a particular branch), subsequent rebuilds with @${BRANCH_NAME} did try to use that old branch rather than where the SCM script was checked out from. Still wondering how that bit of info can be discovered :\

Seems like GitSCM::getBuildData(run,true) can return lots of info https://javadoc.jenkins.io/plugin/git/hudson/plugins/git/util/BuildData.html including a way to identify the repository and that list of recent builds with "wrong" branch names, so one way to map one of many SCMs to a particular SCM URL so we look at the right one.
Even better, a build.getSCMs() returns a gitSCM (in our case) whose getUserRemoteConfigs() contains info on mapping refspec to remote (hence URL) and a getBranches() to see branch of interest.

See https://github.com/jenkinsci/pipeline-groovy-lib-plugin/blob/84da9c54906db772b1848a4d4ff5db9dc922a20b/src/main/java/org/jenkinsci/plugins/workflow/libs/MultibranchScmRevisionVerifier.java#L37-L61 for an example of the relevant APIs for getting branch and revision-related information for a multibranch pipeline build.

Thanks, looks promising. I'd also check if this verbatim works for all 3 use-cases I know of with SCM: MultiBranch proper, MultiBranch with "Single repo & branch", and plain old single-branch single-repo Pipeline definitions. And that it does not unexpectedly work for "inline" pipelines (typed via UI) :)
"Owner"... hm, why? :)

So far I concluded that BranchJobProperty  only applies to MBPs, and my code did check it before falling back to the hacky ways (envvars and defaultedVersionSCM()). So ended up dropping WorkflowJob.getSCMs() use completely, it looked quirky originally and proved so now.. at least all the test stack (including the new case) passes :)
I hope to see if I can find ways to use Branch, SCMHead, SCMRevision etc. classes generically for all 3-4 pipeline source situations outlined above. But in some other iterations, probably not sooner than next week.

Clean-up/refactor some tests, and address (and test) a problem with lightweight CpsScmFlowDefinition serving its SCM info differently.

FWIW, https://github.com/jenkinsci/workflow-cps-plugin/blob/master/src/main/java/org/jenkinsci/plugins/workflow/cps/CpsScmFlowDefinition.java pointed me to https://javadoc.jenkins.io/plugin/scm-api/jenkins/scm/api/SCMFileSystem.html which can getRevision() and that can getHead() for an https://javadoc.jenkins.io/plugin/scm-api/jenkins/scm/api/SCMHead.html that I might have a chance to (in some very roundabout way) generalize the search for branch if the concept exists for that SCM... loose thread in the world sticks out here:
try (SCMFileSystem fs = SCMFileSystem.of(build.getParent(), scm)) {

So that loose thread did not lead far yet, getRevision() returned null (at least for completed builds inspected via script console).
Looking at SCMSourceRetriever.java and SCMRetriever.java (which mostly wraps SCMSourceRetriever to present an interface for plain SCM derived classes serving the library source) was not too encouraging either. Essentially code refers to scm.fetch(thing, ...) and https://github.com/jenkinsci/scm-api-plugin/blob/master/src/main/java/jenkins/scm/api/SCMSource.java#L640 describes the thing as "a specific revision based on some SCM-specific set of permissible syntaxes" (which in assumptions of this plugin is a branch, tag, git commit hash...) and the "thing" is provided by caller from some external knowledge, but is not (currently) extractable from SCM itself.
Updated the PR with further dismemberment of defaultedVersionSCM() to separate the logic which IMHO should stay in this plugin (inspecting different classes of Jobs and Runs to find an SCM instance for pipeline script source) vs. decisions that this is an SCM class that we can inspect, and separately to extractDefaultedVersionSCM() (in particular the hacky logic to extractDefaultedVersionGitSCM()) if we do find one. The latter is what may/should be refactored, but possibly with broadside shot at the whole SCM API ecosystem -- so perhaps in a separate set of PRs (to let any SCM descendant state if it supports "versions" and which name(s) of those it currently considers).

Also added a method extractDefaultedVersionSCMFS() to look into SCMFileSystem (of the run and an scm instance found in it by defaultedVersionSCM()), but in unit tests with all their job types tried, it never sees a non-null SCMRevision.
Codepath ever gets to it at all in a few test cases:

checkDefaultVersion_singleBranch_BRANCH_NAME_after_staticStrings (3 times)
checkDefaultVersion_singleBranch_BRANCH_NAME

In others it does not have to look into the SCM sources with this method: either finds the info from MBP context, or finds no SCMs (inline pipeline scripts).
UPDATE: Double-checked by always trying to request defaultedVersionSCM() (even if data was found earlier), the SCMRevision still never appears.

Leaving the PR "as is" for now, hope for reviews and merge based on merit (added feature usable in many cases) to proceed :)
Scaling this solution beyond GitSCM (hopefully after the PR merge) is possible by:

(poorly) cloning extractDefaultedVersionGitSCM() which is now relatively compact (and also updating a couple of lines in isDefaultedVersionSCMSupported() and the stack of callouts from extractDefaultedVersionSCM()), or
(preferably) by revamp of SCM API so any SCM class can tell us what we need right away.

As of last build -- CI errors checking out (and server not visible at the moment) :\


Add String Variant Validator

This feature basically provides a simple StringValidator for string variants (~string). In the current latest version, if we need a validator for the type VariantExample string, which is a string variant, we have to convert VariantExample to a string in order to use the StringValidator. These changes add a new API that reuses the StringValidator rules for string variants without any conversion.
Before:
type Variant string

const (
   VariantA Variant = "A"
   VariantB Variant = "B"
)

var validator = goval.String().In(string(VariantA), string(VariantB))

After:
type Variant string

const (
   VariantA Variant = "A"
   VariantB Variant = "B"
)

var validator = goval.StringVariant[Variant]().In(VariantA, VariantB)

Codecov Report

Merging #50 (7edf4e5) into main (db76598) will decrease coverage by 0.27%.
The diff coverage is 93.54%.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
@@            Coverage Diff             @@
##             main      #50      +/-   ##
==========================================
- Coverage   92.07%   91.81%   -0.27%
==========================================
  Files          13       13
  Lines         454      464      +10
==========================================
+ Hits          418      426       +8
- Misses         27       29       +2
  Partials        9        9

Impacted Files
Coverage Δ

errors.go
81.63% <60.00%> (-2.46%)
:arrow_down:

goval.go
97.70% <100.00%> (+0.08%)
:arrow_up:

string.go
100.00% <100.00%> (ø)

Hi @josestg , do we need to implement NumberVariant also ?


Explicit joins do not work with the InMemoryAdapter

I have tried running joins with the database.User.Query().Join(database.Department).On().Select() syntax (where the columns to join on are specified, since there is no referential integrity with the InMemoryAdapter). The query returns data, but not joined data as far as I can tell.
All the selected fields from the Department table are null, and the number of rows returned are the number of rows in the User table. If I switch so that the first table is Department, then all the User fields are null and the number of rows is the same as in the Department table.
This was tested with <IP_ADDRESS>

I got a regression from Oracle adapter's point of view. The following test which was working fine before is now broken, I post it here because it is also in the join area:
    [Test]
    public void self_join()
    {
        var q = _db.Employees.Query().Join(_db.Employees.As("Manager"), EmployeeId: _db.Employees.ManagerId);
        q = q.Select(_db.Employees.LastName, q.Manager.LastName.As("Manager"));
        List<dynamic> employees = q.ToList();

        Assert.AreEqual(106, employees.Count); // The top man is missing

        var kingsSubordinates = employees.Where(e => e.Manager == "King").ToList();

        Assert.AreEqual(14, kingsSubordinates.Count);
    }

System.NullReferenceException : Object reference not set to an instance of an object.
at Simple.Data.Ado.DataReaderEnumerable.DataReaderEnumerator.SetCurrent()
at System.Linq.Enumerable.WhereSelectEnumerableIterator2.MoveNext() at System.Collections.Generic.List1..ctor(IEnumerable1 collection) at System.Linq.Enumerable.ToList(IEnumerable1 source)
at Simple.Data.SimpleQuery.ToList()
at System.Dynamic.UpdateDelegates.UpdateAndExecute1<T0,TRet>(CallSite site, T0 arg0)
at Simple.Data.Oracle.Tests.ExploratoryTesting.self_join() in ExploratoryTesting.cs: line 82

Would natural joins work - if I used the not-so-pretty syntax to set up the relationships?

So I ran into this on the .19 release, was this ever resolved in one of the 1.0+ pre-releases? Or is this something that was offset until the 2.0 release?


Patch CTID

This PR fixes a number of issues involved with CTID.

CTID is not present on all RPC tx transactions.
rpcWRONG_NETWORK was missing in the ErrorCodes.cpp

Fixes: https://github.com/XRPLF/rippled/issues/4735

I ran this PR locally. I can see there is ctid field in subscription and account_tx now. However, i can not see ctid in tx response any more.
My input:
'''
{
"method": "tx",
"params":[{ "transaction": "CCBBD2F5CEE90FA7A2ED13A7A343D015F57950B562185734E125B7BE740F2428"}]
}
'''
output:

"ctid" is available for the same transaction from account_tx response.

I ran this PR locally. I can see there is ctid field in subscription and account_tx now. However, i can not see ctid in tx response any more. My input: ''' { "method": "tx", "params":[{ "transaction": "CCBBD2F5CEE90FA7A2ED13A7A343D015F57950B562185734E125B7BE740F2428"}] } ''' output:
"ctid" is available for the same transaction from account_tx response.

Whats the networkID? Can you show me the json for the response that was correct?

I ran this PR locally. I can see there is ctid field in subscription and account_tx now. However, i can not see ctid in tx response any more. My input: ''' { "method": "tx", "params":[{ "transaction": "CCBBD2F5CEE90FA7A2ED13A7A343D015F57950B562185734E125B7BE740F2428"}] } ''' output:
"ctid" is available for the same transaction from account_tx response.

Whats the networkID? Can you show me the json for the response that was correct?

I should mention I am on devnet. I think the correct one should contain the "ctid", right?

It should... I will review this again to make sure.

@intelliot I believe devent was reset to a networkID < 1024 therefore it did not have a ctid. @cindyyan317 can you please recheck the devnet again with your locally running PR?

@cindyyan317 can you please recheck the devnet again with your locally running PR?

Hi @dangell7 . I can see the ctid field from tx now. I guess ctid is supposed to be everywhere? I don't see it in "ledger" and "transaction_entry"

@intelliot I believe devent was reset to a networkID < 1024 therefore it did not have a ctid.
What does it mean?  It should not have ctid when networdID < 1024? devnet's networkId has been always 2 , right?

Yes, I think Devnet's NetworkID has always been 2.
@dangell7 , I think it should be possible (and would be ideal) to always return ctid with validated transactions, for any NetworkID (even if it's <= 1024).

Yes, I think Devnet's NetworkID has always been 2.
@dangell7 , I think it should be possible (and would be ideal) to always return ctid with validated transactions, for any NetworkID (even if it's <= 1024).

You're right and I will look at the issue deeper this weekend. I don't like peering from my work computer. I have an env now to spin up a peer easier to test.
I will also check on the CTID on txs <= 1024. I think I misspoke.

@intelliot moving this to draft while I write more tests. Can confirm ctid is returned when network id <= 1024.

@intelliot I'm opening this back up for review. I see no issues

@dangell7 - I just updated your PR description to use the template. Please feel free to edit / add more. Also, consider the suggestions from scottschurr above, and comment here when you are done.

@scottschurr Thank you very much for the review. I looked over the code and everything looks fine. I'm going to run the tests and build a version to test locally.
Thank you

@scottschurr Thanks again for your update. I have fixed all of the issues. However, we are getting some weird behavior from this. Its sometimes on the tx and sometimes not. We are chasing this down now.
I believe more tests need to be written for this or a bug needs to be fixed. As soon as we can replicate the issue I will fix this.

Given

However, we are getting some weird behavior from this. Its sometimes on the tx and sometimes not. We are chasing this down now.

Could you Convert to draft this PR so we know it isn't ready for merging right now?
Of course, when you do think it may be ready, please mark it as "ready for review".

Internal tracker: RPFC-82

note: @dangell7 indicated he will push changes tomorrow.

awaiting update or comment from @dangell7

there is some missing unit test coverage (see above)
@dangell7 to let us know when this is, from his perspective, ready to merge

@intelliot working on the codecov

@dangell7 how would you like this to move forward?

@seelabs Yeah this is ready. I tried to test the convertBlobsToTxResult but was unable to build a good testcase without rewriting the function.

seelabs confirmed that this still needs perf testing. cc @sophiax851

I tried to build a rippled with this PR to run on the devnet's full history node, but since this PR was based off the 2.2.0-rc1, so the build is getting amendment blocked. @dangell7 can you please rebase off the master branch so I can build it for 2.2.0? Thanks.


Implement loadend event in XMLHttpRequest object

Checking for loadend event returns false
echo "'onloadend' in (new XMLHttpRequest())" | phantomjs

However loadstart and onprogress are available.
echo "'onloadstart' in (new XMLHttpRequest())" | phantomjs

And loadend is available in FileReader
echo "'onloadend' in (new FileReader())" | phantomjs

Is it possible to have it also in XMLHttpRequest?
Here is the documentation from W3C
http://www.w3.org/TR/XMLHttpRequest/#event-xhr-loadend

Seems it's fixed now
phantomjs> true
phantomjs> %


Text Not Translating

Overview
Market Data text on the assets and accounts page of an individual asset do not translate.
References and additional details

log in to app.shapeshift
choose you desired wallet
do to the account or asset page for an individual asset
on the bottom right of the screen look for "Market Data"
Go to settings and change the language
Notice the Market Data text does not translate

Additional translation errors below
Acceptance Criteria
Market Data text should translate
Need By Date
No response
Screenshots/Mockups

View a Demo not translating in Indonesian and Chinese:

Main title and view a demo not translating in Russian:

In Russian:

In Indonesian:

Spanish - Cosmos Defi:

Estimated effort
No response

As discussed on Discord with NeOMakinG, from the above examples the only case which isn't translatable currently is "Market Data". The problem is located here, the string should be passed through the translation function to be made available (@gomesalexandre : maybe you could look into it please?): https://github.com/shapeshift/web/blob/222fb5c40ec01e7687254ffdf31e40476efbf8a6/src/components/AssetHeader/AssetMarketData.tsx#L57
The other strings are apparently translatable, but are missing translations. Some have been translated since they were first posted here too. The full coverage provided by the Globalization WS is currently limited to Spanish/Portuguese/French and depending on the release cycles the translations might lag a few days behind new releases. Translations for other languages are planned to be done through bounties.

@NeOMakinG WRT the non-translated string, we will need to create a new 'assets.assetDetails.assetHeader.marketData' translation

Provided the PR for the translation, please don't close the issue as it doesn't fix everything

As far as I can see the other texts that are not translated in these screenshots are not an Engineering problem, the strings exist and they will eventually be translated by the Globalization Workstream (some are already translated on the Translation platform we use and waiting to be turned into a PR) at least in the 3 languages we can currently cover within the workstream's budget. As mentioned above the other languages that have partial translations are planned to be translated through bounties potentially. I don't think the issue needs to be kept open for these things to happen.

@Lychbot As stated before this issue can be closed from the point of view of the Globalization WS.
The related strings are now all translatable and are translated in Spanish/Portuguese/German/French (Note: Since the last time we have already added German to this list through bounties... so not all hope is lost for the other languages mentioned in the original issue). They will be translated if/when the DAO can afford the other languages.
If it's an issue to keep partially supported languages, we can see with Engineering to only enable the fully supported languages?
Also just for info two other issues have actual untranslatable strings currently and require Engineering's work, they are in the Backlog:
https://github.com/shapeshift/web/issues/2479#issuecomment-1223343076
https://github.com/shapeshift/web/issues/3083

Closing as the strings mentioned in this specific issue aren't translatable.
The "incomplete" (not currently supported by Globalization) languages will be disabled soon through: https://github.com/shapeshift/web/issues/3182


/bin/sh: line 1: systemctl suspend: command not found

All in the title, I don't know why hypridle calls the sh shell, but I can't get in to run systemctl. Swaylock, which is at /usr/local/bin, runs for example.

you can run /bin/bash -c "cmd" if the shell matters. What's the problem?

how are you running hypridle? Sounds like PATH being kinda bork. If you use a login manager, and launch hypridle with exec-once,  your .bashrc is not sourced.

If I include the full path of systemctl, I get this
/bin/bash: line 1: /usr/bin/systemctl suspend: No such file or directory

i think the issue is that you are quoting the command in your hypridle config. hyprlang doesn't use quotes as far as i know, so it seems to just be passing them to your shell instead. the error messages you're getting are exactly what you get if you try to do the same thing in bash.

change it to lock_cmd = systemctl suspend (without any quoting) and it should work.

That's it!! Thank you, I don't know how I didn't think of that :)

oh yeah I am blind


R24.05 Portal - Release Checks

[!IMPORTANT]
Follow the guidance on how to use the templates.

Release Info
Please provide information on what you want to be included in the Eclipse Tractus-X release.
If you are not owner of this issue, please provide the information as comment to the issue.
Make sure to assign this issue to expert(s) for their approval, as soon as you have finished preparation. Multiple assignees allowed; they will un-assign themselves once review completed.
Version to be included in Eclipse Tractus-X release: 2.0.0 [NOT YET RELEASED]
Portal:

Portal Chart: portal-2.0.0
Portal Frontend: v2.0.0
Portal Backend: v2.0.0
Portal Assets: v2.0.0
Portal Frontend Registration: v2.0.0

IAM:

CentralIdP: 3.0.0
SharedIdP: 3.0.0

Leading product repository: https://github.com/eclipse-tractusx/portal
For the QG Review the latest available Release Candidate (RC) version is to be checked, the RC versions will be kept up to date in the following:
Portal:

Portal Chart: portal-2.0.0-RC7
Portal Frontend: v2.0.0-RC6
Portal Backend: v2.0.0-RC7
Portal Frontend Registration: v2.0.0-RC1

IAM:

CentralIdP: 3.0.0-rc.3
SharedIdP: 3.0.0-rc.3

Compliance Verifications
This issue tracks all checks, that need to be performed for a product release in Eclipse Tractus-X.

[ ] Interoperability checks performed
[ ] Data Sovereignty checks performed

General Checks

[ ] Tractus-X Release Guidelines(TRGs) fulfilled https://github.com/eclipse-tractusx/portal/issues/313

[!NOTE]
Note: most criteria for documentation and security are now covered in TRGs

Security Checks

[ ] Thread Modelling Analysis passed

Test Results

[ ] E2E Integration Test passed
[ ] User Journey approved

Helpful Links

Eclipse Tractus-X openAPI specs on SwaggerHub

Reviewing committer is @fabiodmota, thanks for taking that over!
I'll inform you in a couple of weeks once you can start with the review

no changes to interoperability since last review (release 24.03)
Approved! @kelaja

Security assessment performed on 2024-05-15 by @szymonkowalczykzf and me.
Threat modeling/risk assessment will be finalized next week, no major risks to be expected. Will update here, as soon as threat modeling has passed.

QG Checks done for:

https://github.com/eclipse-tractusx/portal-iam/issues/120
https://github.com/eclipse-tractusx/portal-frontend/issues/779
https://github.com/eclipse-tractusx/portal-backend/issues/733
https://github.com/eclipse-tractusx/portal/issues/313

QG done for the repo https://github.com/eclipse-tractusx/portal-assets
https://github.com/eclipse-tractusx/portal-assets/issues/315:

All checks done with only the copyright header open, it's a small topic and not a blocker.

QG done for the repo https://github.com/eclipse-tractusx/portal-assets

R24.05 Portal Assets - Release Checks portal-assets#315

All checks done with only the copyright header open, it's a small topic and not a blocker.

same for https://github.com/eclipse-tractusx/portal-frontend-registration/issues/170

INT Tests need to be documented. E2E Tests ongoing. Testmanagement Approval cannot be given at this time.

QG review 17-May:
data sov outstanding
TRGs at 90% / SEC outstanding => will be completed, once testing finalized and final version available
SEC incl Thread Modelling approval expected cw21
Test completion expected early cw21 / no roadblocks
User Journey approval expected after test completion by @jjeroch

@evegufy you may already want to check the data sov box based on relevant comment

Threat Modeling - Approved - Assessment was reviewed and updated, all finding addressed previously.

QG done for the repo https://github.com/eclipse-tractusx/portal-assets

R24.05 Portal Assets - Release Checks portal-assets#315

All checks done with only the copyright header open, it's a small topic and not a blocker.

Checks completed for all involved repositories with the exception of security, which will be taken over by @RoKrish14 once the last release candidate was released:

https://github.com/eclipse-tractusx/portal-frontend/issues/779
https://github.com/eclipse-tractusx/portal-backend/issues/733
https://github.com/eclipse-tractusx/portal/issues/313
https://github.com/eclipse-tractusx/portal-assets/issues/315
https://github.com/eclipse-tractusx/portal-frontend-registration/issues/170

For https://github.com/eclipse-tractusx/portal-iam/issues/120 all checks (also security) are completed.

Test successfully closed and user journey approved by the product owner

@RolaH1t  Finalized versions released
Portal:

Portal Chart: portal-2.0.0
Portal Frontend: v2.0.0
Portal Backend: v2.0.0
Portal Assets: v2.0.0
Portal Frontend Registration: v2.0.0

IAM:

CentralIdP: 3.0.0
SharedIdP: 3.0.0

Congrats:
Quality Gate fully approved with all necessary pre-requisites fulfilled!
Thanks for all your efforts,
Roland


FEATURE: Option to disable the removing of platforms.

Hi,
I would like the option to be able to disable the removing of platforms as I'm trying to cache my generated platforms plugins to significantly reduce my build times and downloading and installing the plugins takes up to 6 mins of my build.
Just a simple true/false would do the job, just anything to allow us to be able to disable it if we don't want it.
Please note I of course still want it to add the platforms, just optional removals of platforms.
Thanks!

I've implemented this myself and just created a pull request for it. #25

Hi we added this in the version: 1.2.4.


[Bug]: Metadata issue

What happened?
Any ways of using .add (documents=..., metadatas=...)
with nested metadata? I have a dict with multiple nested lists and dicts
Versions
Python 3.11, Chroma Version: 0.4.22
Relevant log output
Expected metadata value to be a str, int, float or bool, got ...  which is a <class 'list'>.

@Dyke-F, thanks for raising this. This behavior is by design. What people re doing to work around this is to json.dumps() and store it as text.

Closing this since we don't support this behavior. @Dyke-F let us know if we can do anything to help!


Migration to dart:js_interop & package:web/web.dart

Is your feature request related to a problem?
No response
Describe the solution you'd like
To be compliant with flutter 3.22 and prepare WASM compatibility, we should migrate to dart:js_interop & package:web/web.dart
Additional context
No response
Epic
No response

help from https://github.com/dart-lang/web/issues/261


Incompatible with<EMAIL_ADDRESS>

Hi there, I just ran into this error in my project's build on Travis:
[Semantic release]: An error occurred while running semantic-release: { TypeError: Cannot destructure property `env` of 'undefined' or 'null'.
    at module.exports (/home/travis/build/fartts/fartts/node_modules/@semantic-release/github/lib/resolve-config.js:4:99)
    at module.exports.pluginConfig (/home/travis/build/fartts/fartts/node_modules/semantic-release-github-pr/src/github-init.js:12:59)
    at /home/travis/build/fartts/fartts/node_modules/semantic-release-github-pr/src/with-github.js:6:18
    at validator (/home/travis/build/fartts/fartts/node_modules/semantic-release/lib/plugins/normalize.js:42:28)
    at pReduce (/home/travis/build/fartts/fartts/node_modules/semantic-release/lib/plugins/pipeline.js:37:40)
    at Promise.all.then.value (/home/travis/build/fartts/fartts/node_modules/p-reduce/index.js:16:10)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:188:7)
  pluginName: '/home/travis/build/fartts/fartts/node_modules/semantic-release-github-pr/src/index.js' }
{ TypeError: Cannot destructure property `env` of 'undefined' or 'null'.
    at module.exports (/home/travis/build/fartts/fartts/node_modules/@semantic-release/github/lib/resolve-config.js:4:99)
    at module.exports.pluginConfig (/home/travis/build/fartts/fartts/node_modules/semantic-release-github-pr/src/github-init.js:12:59)
    at /home/travis/build/fartts/fartts/node_modules/semantic-release-github-pr/src/with-github.js:6:18
    at validator (/home/travis/build/fartts/fartts/node_modules/semantic-release/lib/plugins/normalize.js:42:28)
    at pReduce (/home/travis/build/fartts/fartts/node_modules/semantic-release/lib/plugins/pipeline.js:37:40)
    at Promise.all.then.value (/home/travis/build/fartts/fartts/node_modules/p-reduce/index.js:16:10)
    at <anonymous>
    at process._tickCallback (internal/process/next_tick.js:188:7)
  pluginName: '/home/travis/build/fartts/fartts/node_modules/semantic-release-github-pr/src/index.js' }

I'm pretty sure the issue is related to the 5.0.0 release of @semantic-release/github (in particular this line) … for now I've rolled back to<EMAIL_ADDRESS>(depends on<EMAIL_ADDRESS>and<EMAIL_ADDRESS>(compatible with<EMAIL_ADDRESS>and everything seems to work again.

Thanks for the great report and sorry for the delayed response! This looks like an easy fix!

Actually, a little more involved than it first looks. semantic-release 15.7 technically breaks this plugin because it allows for multiple generateNotes plugins (not entirely sure how 15.7.2 still works for you). This is fixable but requires a more involved, breaking change.
To begin with, I'm releasing a fix that restricts the peer dependency range for the current major version of this plugin.
https://github.com/Updater/semantic-release-github-pr/pull/30

Thanks for digging in, and no worries about the delay … as I mentioned it's not currently blocking me. I'm happy to help if I can (I'm also pretty busy). In case it's useful, here's my release.config.js:
module.exports = {
  verifyConditions: [
    '@semantic-release/changelog',
    '@semantic-release/npm',
    '@semantic-release/git',
    '@semantic-release/github',
  ],
  prepare: [
    '@semantic-release/changelog',
    '@semantic-release/npm',
    {
      path: '@semantic-release/git',
      message: `\
[<%= nextRelease.version %>]\
(https://github.com/fartts/fartts/tree/<%= nextRelease.version%>)\
 - <%= new Date().toLocaleDateString('en-US', {
  year: 'numeric',
  month: 'short',
  day: 'numeric',
  hour: 'numeric',
  minute: 'numeric',
})%> [skip ci]
<%=nextRelease.notes%>`,
    },
  ],
  publish: ['@semantic-release/npm', '@semantic-release/github'],
  npmPublish: false,
  githubPr: {
    analyzeCommits: '@semantic-release/commit-analyzer',
    generateNotes: '@semantic-release/release-notes-generator',
  },
};

Maybe it's working because I'm only specifying the one generateNotes plugin?

Thanks for the help @mysterycommand! Just released a version that I believe will fix this. Please reopen if that isn't the case!

:tada: This issue has been resolved in version 5.1.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Incorrect API-Gateway resource policy for API Event function with '+' path parameters

Description:
I have created a stack with an AWS::Serverless::Function and an AWS::Serverless::Api resource with the intention of restricting access to an IP address range. An example stack looks as follows:
{
  "AWSTemplateFormatVersion": "2010-09-09",
  "Transform": "AWS::Serverless-2016-10-31",
  "Globals": {
    "Function": {
      "Timeout": 10,
      "MemorySize": 128
    }
  },
"Resources": {
    "AppwRepublishFunction": {
      "Type": "AWS::Serverless::Function",
      "Properties": {
        "FunctionName": "MyRepublisherLambda",
        "Handler": "index.handler",
        "Runtime": "nodejs12.x",
        "CodeUri": "...",
        "Events": {
          "Post": {
            "Type": "Api",
            "Properties": {
              "Path": "/republish/{parameters+}",
              "Method": "post",
              "RestApiId": { "Ref": "ApiGateway" }
            }
          }
        },
        "Policies": [ "..." ]
      }
    },
    "ApiGateway": {
      "Type": "AWS::Serverless::Api",
      "Properties": {
        "Auth": {
          "ResourcePolicy": {
            "IpRangeWhitelist": ["<IP_ADDRESS>/16"]
          }
        },
        "CacheClusterEnabled": false,
        "EndpointConfiguration": "REGIONAL",
        "Name": "MyRepublisherApi",
        "StageName": "MyStage"
      }
    }
  }
}

When the API-Gateway resource policy, the resultant document looks as follows:
{
    "Version": "2012-10-17",
    "Statement": [
        {
            "Effect": "Allow",
            "Principal": "*",
            "Action": "execute-api:Invoke",
            "Resource": [
                "arn:aws:execute-api:my-region-1:123456789012:my-api-id/MyStage/POST/republish/{parameters+}"
            ]
        },
        {
            "Effect": "Deny",
            "Principal": "*",
            "Action": "execute-api:Invoke",
            "Resource": [
                "arn:aws:execute-api:my-region-1:123456789012:my-api-id/MyStage/POST/republish/{parameters+}"
            ],
            "Condition": {
                "NotIpAddress": {
                    "aws:SourceIp": "<IP_ADDRESS>/16"
                }
            }
        }
    ]
}

The above however doesn't work even when connecting to the specified IP address range. I have to manually update the resource identifier to arn:aws:execute-api:my-region-1:123456789012:my-api-id/MyStage/POST/republish/* instead.
--
It is also worth noting that a precursor to my above example, I had setup the path to be /republish/{entityType}/{parameters+} and this gave a resource-policy with a resource-identifier
arn:aws:execute-api:my-region-1:123456789012:my-api-id/MyStage/POST/republish/*/{parameters+}.
My point is that standard parameters (without +) seem to get translated to a *, so the behaviour should likely be the same with parameters suffixed with a +.
Steps to reproduce the issue:

Import a stack structured like the one provided above, setting the IP address to a suitable value for yourself.
Attempt to access the API through the endpoint created.
An error will be observed.
Modify the Resource Policy in the AWS Api-Gateway console to use a simple * wildcard symbol, then republish the API.
Attempt to access the API again and you will gain access.

Observed result:
The automatically generated Resource-Policy does not describe parameters suffixed with + and therefore in the above example, access is denied.
Expected result:
Access in the above example should be granted.

Looks like we need to convert parts of the path to the format that APIGW expects. This can be done here.


Add module for ASUS Wireless-AC1300 dongle

What hardware did you use here?

Sorry, I never get notifications from this repo.


Add BM25PRF Reranker

The ranking model used in https://github.com/osirrc/anserini-bm25prf-docker
Stephen E. Robertson, and Karen Spärck Jones. Simple, proven approaches to text retrieval. University of Cambridge Computer Laboratory, 1994.

Sorry…. it seems that there is some problem after rebasing the latest Anserini….

Fixed

Thank you for the review, I have fixed most of them (except the default parameters)


Updated segment_analytics_plugin_adjust

chore(dependencies): Updated segment_analytics_plugin_adjust dependencies.

Updates minimum supported SDK version to Flutter 3.16/Dart 3.2
Updates json_annotation to ^4.9.0
Updates adjust_sdk to ^4.38.2
Updates segment_analytics to ^1.1.1

Random question: What was holding you back from updating to the sdk to https://pub.dev/packages/adjust_sdk/versions/5.0.2?


Replacing category-specific feeds w/ a global reverse-chronological blogroll

Description & motivation
This replaces feature category-specific feeds on the dev blog homepage with a classic reverse-chronological blogroll.
Pre-release docs
Is this change related to an unreleased version of dbt?

[ ] Yes: please

update the base branch to next
add Changelog components: <Changelog>[New/Changed] in v0.x.0</Changelog>
add links to the "New and changed documentation" section of the latest Migration Guide

[x] No: please ensure the base branch is current
[ ] Unsure: we'll let you know!

Preview
https://deploy-preview-1197--docs-getdbt-com.netlify.app/blog

Hey @runleonarun, I noticed the Add PR to project GitHub action seems to be failing consistently on my PRs. Is that anything I should be concerned with, or should I be doing anything differently to get it to pass?

@krevitt I added a regular_posts_count property to our metadata.yml file to control how how many recent posts to show.

@krevitt I added a regular_posts_count property to our metadata.yml file to control how how many recent posts to show.

Got it - and to confirm, we don't have pagination going anywhere in the dev blog right?

@krevitt I added a regular_posts_count property to our metadata.yml file to control how how many recent posts to show.

Got it - and to confirm, we don't have pagination going anywhere in the dev blog right?

That's correct, no pagination at the moment. I can throw a discovery task into Asana to look into that.

Might this be filtering out any post where is_featured = true from the main blogroll, vs only the most recent n count of featured posts that are set to display (currently 4)?
Noticing the dateadd + union all posts aren't appearing in the blogroll.

I believe those are just missing the cut at 10 for the recent posts. They will be included once we can get pagination up and running. For the 'Recent Posts' section, i'm grabbing all posts without the featured: true, and all featured posts which aren't included in the feature section at the top

Might this be filtering out any post where is_featured = true from the main blogroll, vs only the most recent n count of featured posts that are set to display (currently 4)?
Noticing the dateadd + union all posts aren't appearing in the blogroll.

I believe those are just missing the cut at 10 for the recent posts. They will be included once we can get pagination up and running. For the 'Recent Posts' section, i'm grabbing all posts without the featured: true, and all featured posts which aren't included in the feature section at the top

Thanks Jason - could we check "What's a Primary Key and Why Do We Test Them?" (post link here) as one example?
Posted on 11/29 and is_featured = true, isn't appearing in either the featured posts section or the blogroll (which goes back to 9/15).

@JKarlavige sorry about that action failing. It's fixed now. I'm actually not sure how it ever worked. 🤷🏻


Improve grammar of documentation

The structure of the sentence was somewhat confusing, and this change should help clarify the meaning.

Thanks!


Adding foldl

fold, foldRight, reduce, reduceRight should be in the same exercise if we want all of them


Improving UI/UX graphics of the website and the form submission since it covers the whole page and some portion of form isn't visible completely.

I will improve the graphics of website by adding cool hovering effects over the button and also improve the color combination and font of the website

Please assign it to me @sk66641 since people mostly get attracted by the graphics of website and I feel that I can make it look more better.

I will change that and also there is a bug that the footer disappears when light mode is switched on. So shall I fix it too @sk66641 ?

@Khushipawar37
it has been fixed

Sure, I will enhance it and add some hovering effects too


[VERY DRAFT] Refactor

This PR is for draft demonstration purposes only and will be split into a few different PRs.

Closing this WIP demonstration draft in favor of the final stacked PR series starting with https://github.com/navapbc/wic-mt-demo-project-eligibility-screener/pull/140


refactored Count()

Coverage remained the same at 100.0% when pulling 8d83f85cd7a520d8ad37936e22abdecca6fd64ed on refactor into 74c8135005894d5523227523b82dc2cd63718cdd on master.


Error: Failed to retrieve the initial room data.

I tested https://tiktok-chat-reader.zerody.one/, but I encountered an error:
Error: Failed to retrieve the initial room data.

Hi @zerodytrash same here. I was using your library perfectly until today. I cannot connect to my live

it seems to work now, thank you

fixed.


Test button does not reliably work

Not all customers are licensed for performance analytics.  As a consequence the Test button which executes the following code in the ServiceNowClient.py might fail with an authorisation error
get_scorecard

Fixed in v4.1.10


Added Shop Talk to Podcasts

[ x] I have read freeCodeCamp's contribution guidelines.
[ x] My pull request has a descriptive title (not a vague title like Update index.md)
[ x] My pull request targets the master branch of freeCodeCamp.
[ x] None of my changes are plagiarized from another source without proper attribution.
[ x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #XXXXX

LGTM

@jafowler85 Thank you for the add!
Congrats on your first-time contribution!