Caching of the OpenAI API responses

Describe the feature
During development it could be useful to cache OpenAI API responses while keeping behaviors like the incremental returning of results. This might be a proxy of the OpenAI API and maybe another project, like https://github.com/easychen/openai-api-proxy
This can be done at the level of the application using chatgpt-api but things like streaming add some complications.

great idea 💯
here's some related notes I had on this.
OpenAI proxy API

COSS / hosted proxy for interacting with the OpenAI API and/or other main AI APIs

problems it tries to solve

caching (seriously POST caching for embeddings & completions alone is huge)
observability
monitoring
privacy (e.g., see https://github.com/cado-security/masked-ai)
general engineering productionization concerns like latency that you guys have already graphed
the ability to have more than 5 keys and to put limits on each individual key

OSS examples

https://github.com/egoist/openai-proxy
https://github.com/6/openai-caching-proxy-worker
https://github.com/cado-security/masked-ai
https://github.com/easychen/openai-api-proxy

companies

[Helicone (YC W23)](https://www.helicone.ai/)

notes

pro: could quickly build an MVP of this using CF workers and something like [Reflare](https://github.com/xiaoyang-sde/reflare)
con: imho this is def a feature, not a platform, and openai will build better first-party support for these use cases over time
same idea but substitute Cohere/Anthropic/etc or building a normalization layer across all of these

if you could manage to get traction w/ this normalization layer, you could eventually replace the third-party APIs with your own first-party APIs over time, but disintermediation would be really tough w/ this sort of thing

related to this idea, there have been a lot of viral Bring-Your-Own-Key demos built on top of OpenAI. I’d love to build a lightweight solution which solves for this use case in a more trustworthy, secure manner, since pasting your OpenAI secret key into random webapps is a security nightmare

I'm going to close this issue as out of scope for this repo, but I hope my notes above are useful for anyone looking to add this into their workflow – or anyone who wants to build this type of caching abstraction 🔥
thanks @markNZed 🙏


Remember which client was created to select it when the update comes

📒 Description
This PR adds the ability to remember the last created client to the Time Entry Editor (to not get overwritten when a UI update comes)
🕶 Types of changes

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

🤯 List of changes

Linux - New clients are now properly added to new Projects

👫 Relationships
Closes #3104
Closes #3101
🔎 Review hints
Check if the right new client is added to a new project. Also check if the client field is not erroneously selected when switching the edited time entries, for example.

Works as expected! 👍


marching tetrahedra face order

the marching tetrahedra algorithm outputs faces in the wrong order, either fix their winding order later or fix the algorithm

added a hacky fix


Add ability to delete your account

Should be pretty easy. Add a button on the /profile page. Also add in some kind of "Are you sure?" confirmation.

Added the button at the bottom of the /profile page. You'll see it, it's bright red. Enter your password and click the button.


[5.4] Use namespaced PHPUnit TestCase

Switched to the namespaced PHPUnit\Framework\TestCase class from PHPUnit_Framework_TestCase in preparation for PHPUnit 6.0 (already available in version 5).
See this tweet by @sebastianbergmann: https://twitter.com/s_bergmann/status/814846275099181056

Wonder why it fails on setup=lowest but not on the other 5.6 test runs 🤔

I've bumped the minimum phpunit version on your PR to 5.7.0, to get all the "forward compatibility" commits in phpunit 5.

Tests should pass now I think.

NB The problem was that, because autoloading is case sensitive, and PHPUnit 5.4.0->5.4.2 used lowercase namespace, then the uppercase version wasn't matched.

...because autoloading is case sensitive...

Didn't know about that, thanks for explaining it to me.

because autoloading is case sensitive

More specifically: autoloading is case sensitive on a case sensitive filesystem. :)


Add toleration for new control-plane taint

This PR adds a toleration for the node-role.kubernetes.io/control-plane taint to resources that already have a toleration to the deprecated node-role.kubernetes.io/master taint.
Towards https://github.com/giantswarm/roadmap/issues/2468

closed as suggested by pawel
https://gigantic.slack.com/archives/C01BYMF6RN0/p1683722724870369?thread_ts=1683716806.503559&cid=C01BYMF6RN0


File commons/colors.tex not found, Emergency stop, failed to compile.

Sorry I am very very new to LaTeX (started to learn it a week ago), I may only did something wrong.
Describe the bug
A clear and concise description of what the bug is.
Inside an empty folder, put the Template.tex in it, and compile the .tex file there.
Error message:
Template.tex || File "commons/colors.txt" not found.
/usr/share/texmf/tex/latex/moderncv/moderncviconsawesome.sty |25 error| Emergency stop.
Expected Behavior
Compile properly
Actual Behavior
Failed to compile, require to download commons/colors.txt into the folder to compile properly. It should be optional (in my opinion).
Minimum Working Example to Reproduce the Problem

Install moderncv witn paru -S texlive-moderncv-git ttf-font-awesome-5 stf-font-awesome-5 since arch linux do not have the latest version of moderncv in the texlive-latexextra package. The AUR ttf-font-awesome-5 stf-font-awesome-5 packages are necessary because arch linux pumped the awesome font version to 6.

Inside an empty folder, put the Template.tex (from this repo) in it, and compile the .tex file there.

Error message as described above.

I guess  #137 cause this.
I originally thought the commons/colors.tex file is optional for people want to redefine the colors. Now it seems we must include it in every directories using moderncv. I do not know whether it is by design or not.
Specifications

moderncv version and source: texlive-moderncv-git
LaTeX compiler (pdfLaTeX, XeLaTeX, etc.) and version: latexmk, xelatex engine
Platform (operating system, LaTeX editor, etc.): Endeavouros, nvim with vimtex plugin

Additional context
Add any other context about the problem here.
BTW, the moderncv is a very good tool!
Thank you for your work! I should have discovered it sooner!

The current situation is by design as color definitions were previously duplicated in the various styles.
The colors files, like the style and class files, are necessary to support the compilation of template.TeX.
It seems that you have a "franken" setup with some mixing and matching which should be a non-issue when installing a versioned release or using the complete current master.


Remove basicauth from Production

Summary | Résumé
Basicauth was meant as a temporary measure on the Production registration site. This removes it in preparation for launch.

@dsamojlenko
Do we also want to get rid of the override middleware class as well?

Good catch, removed it.

Closing this as we will be decommissioning the Registration site in Production.


Make type-graph.txt support more fine-grained metainfo

...perhaps using a TOML-like syntax, and use this for grouping.
This issue is a part of TODO.

What would this solve or help?

I imagine that original purpose was to extend existing type-graph.txt to do e.g. types grouping(not sure what was the meaning here, to be honest). In case of relation database usage, this wont be necessary as you can tune things on table/query level.

Sounds like a +1 for closing...

Re-open if needed.


What is the license for Errata data?

Is this feature request related to a problem? If so, please describe it.
What is the license for Errata data? I could not find one.
I reckon that several errata may be mirrored from RH, so their license (whatever it is) would likely apply too?
Describe the solution you'd like to see
I need a clear open license for inclusion in https://github.com/nexb/vulnerablecode and https://github.com/nexB/vulnerablecode/issues/753
Have you considered alternative solutions/features? If so, please describe them.
No response
Version and Build Information
N/A
Additional Context
N/A

I likely filed this issue in the wrong repo .... Should be in https://github.com/resf/distro-tools/tree/main/apollo instead?

Hey - No worries about filing it here. The license for the errata data is the same as RH, that is, CC-BY-4.0.

The data resources linked on this page as well as their alternative representations available through the Security Data API are licensed under the Creative Commons Attribution 4.0 International License. If you distribute this content or a modified version of it, you must provide attribution to Red Hat, Inc. and provide a link to the original.

https://access.redhat.com/security/data


Add an example for using the async producer with goroutines

@Shopify/kafka

Partially fixes #323

LGTM


Fixed PWA Branding

Motivation and Context

Currently, the PWA Logo is not correctly set. It is the default image, which is probably not what is wanted.
While not being a major issue, it is just one minor annoyance less 😉
Resolves #711
Description

I added the icons to the branding directory
I added the relevant icons to the router
(On the way, I also took the liberty to make the default favicon.ico 48x48 instead of 16x16)
Steps for Testing

Prerequisites: Probably none

Change the branding

[ ] branding/icons-192.png
[ ] branding/icons-512.png

Install App as PWA
See updated Logo
...
Profit

Screenshots

@joschahenningsen I think this still needs to be configured on prod, as https://live.rbg.tum.de/static/assets/img/icons-512.png still points to the image used if branding is not activated. 😉


CoreCLR Access Violation happens randomly in HashProviderCng.cs / TryFinalizeHashAndReset

Describe the bug
We get an access violation from CoreCLR, which causes w3wp.exe or dotnet.exe to crash (depending on selected hostingmodel).
We call the following piece of code on IIS / ASP.NET Core 2.2.3. It will randomly fail on IIS but running the code locally in a console program does not produce any problems with the exact same input strings:
        public static string Encrypt(string inputText, string password)
        {
            keyGenerator = new Rfc2898DeriveBytes(password, Encoding.UTF8.GetBytes(password));

            // error will occur here:
            RijndaelManaged rijndaelCipher = new RijndaelManaged { Key = keyGenerator.GetBytes(32), IV = keyGenerator.GetBytes(16) };

            byte[] plainText = Encoding.UTF8.GetBytes(inputText);

            using (ICryptoTransform encryptor = rijndaelCipher.CreateEncryptor())
            using (MemoryStream memoryStream = new MemoryStream())
            using (CryptoStream cryptoStream = new CryptoStream(memoryStream, encryptor, CryptoStreamMode.Write))
            {
                cryptoStream.Write(plainText, 0, plainText.Length);
                cryptoStream.FlushFinalBlock();
                return Convert.ToBase64String(memoryStream.ToArray());
            }
        }

            var s = "id=23456&type=1&referrer=&module=1";
            var p = "{2E79Y76.N@#N`WexS?wNK.~^Sg=Zp,8BqT'$T'Gj.)}UEggLE6RkA]ZUQ4[dHqhu#xE'9"; // length always 70, fixed password

            Console.WriteLine(Encrypt(s, p)); // returns 8UHp//yY44u8UBygjY1PL+9lIpOcmQJTMIMqhsJ1TQFj4NVHwa9gsaWLp3CbkIug

We previously reported this in https://github.com/aspnet/AspNetCore/issues/6097 and now we have been able to track down where problem happens.
A stack dump reveals that the access violation happens in HashProviderCng.cs https://github.com/dotnet/corefx/blob/154c2a35d5480d359729959f3e3e7d8578a18612/src/Common/src/Internal/Cryptography/HashProviderCng.cs):
I see that it was updated with TryFinalizeHashAndReset in 2017 @stephentoub.

Unhandled exception at 0x00007FFBE37CE8FB (coreclr.dll) in w3wp.exe.968.dmp: 0xC0000005: Access violation.

        public override bool TryFinalizeHashAndReset(Span<byte> destination, out int bytesWritten)
        {
            if (destination.Length < _hashSize)
            {
                bytesWritten = 0;
                return false;
            }

            NTSTATUS ntStatus = Interop.BCrypt.BCryptFinishHash(_hHash, destination, _hashSize, 0);
            if (ntStatus != NTSTATUS.STATUS_SUCCESS) // access violation
            {
                throw Interop.BCrypt.CreateCryptographicException(ntStatus);
            }

            bytesWritten = _hashSize;
            ResetHashObject();
            return true;
        }

This happens both in InProcess and OutOfProcess, with AspNetCoreModule and -V2.
dotnet --info
C:\Users\Administrator>dotnet --info
.NET Core SDK (reflecting any global.json):
 Version:   2.2.105
 Commit:    7cecb35b92

Runtime Environment:
 OS Name:     Windows
 OS Version:  10.0.14393
 OS Platform: Windows
 RID:         win10-x64
 Base Path:   C:\Program Files\dotnet\sdk\2.2.105\

Host (useful for support):
  Version: 2.2.3
  Commit:  6b8ad509b6

.NET Core SDKs installed:
  2.1.300 [C:\Program Files\dotnet\sdk]
  2.1.301 [C:\Program Files\dotnet\sdk]
  2.2.100 [C:\Program Files\dotnet\sdk]
  2.2.104 [C:\Program Files\dotnet\sdk]
  2.2.105 [C:\Program Files\dotnet\sdk]

.NET Core runtimes installed:
  Microsoft.AspNetCore.All 2.1.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.All 2.2.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.All]
  Microsoft.AspNetCore.App 2.1.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.4 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.0 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.1 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.2 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.AspNetCore.App 2.2.3 [C:\Program Files\dotnet\shared\Microsoft.AspNetCore.App]
  Microsoft.NETCore.App 2.0.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.1 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.2 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.4 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.1.5 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.0 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.1 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.2 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
  Microsoft.NETCore.App 2.2.3 [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

@stephentoub @bartonjs This looks more suited to be in the .net repo?

@joshfree / @danmosemsft - is this in an area that you own? What repo should we move this issue to?

corefx is likely the right repo, at least to start.
Is there a runnable repro?

This issue was moved to dotnet/corefx#36491


Revisit shared pool rerolls

Possible signature:
Die.shared_reroll(rolls, condition, count, depth=None)

Should this return the sum, or some sort of OutcomeCountGenerator?
If we go with some sort of distribution across pools, should the generator emit all possible counts at a time? Or do we treat each possible generator separately? It's possible that memoization works better in this case.
Using a Die of generators has some problems:

It requires that generators be sortable, which there are no meaningful semantics for; comparators may imply set-like relationships which are not appropriate for sorting.
There's no way to tell by itself whether evaluate(die_of_generators) wants to evaluate using the generators themselves as outcomes, or to evaluate for each possible generator. So we'd still need a flag or separate method.
Very few Die methods seem applicable to generators. In fact, it's more generator methods that would apply.

Revisiting this considerably later:
Shared rerolls break indistinguishability---if you're trying to maximize the sum of d6s, if you roll a bunch of 4s, you will stick with 4s even if you would have rolled 6s on the rerolls. On the other hand if you rolled the 6s first, you get those 6s.
As an outside routine, we would want a split function to divide a population into separate sub-populations. Then we could produce a probability distribution over the following groups:

Dice that ended up on a reroll.
Dice the player decided to keep.
Dice that they player wanted to reroll, but ran out of shared rerolls.

The first could be decomposed into the last two if this is better for performance (probably).
Maybe we can express this as a MultisetGenerator by splitting into concrete Pools on the first iteration?

Another question is what the motivation is. 5e D&D has shared rerolls in Empowered Spell, but this is probably better solved by a multiset evaluator.

After chatting with @alexrecarey I'm going to try to push this forward. Potential questions:
Allow for prioritizing outcomes to reroll?
With this we could handle cases like Empowered Spell with something like
```d6.reroll_pool(8, [1, 2, 3], limit=4, depth=1, prioritize='lowest')````
this example meaning 8d6, reroll up to 4 dice that rolled below average, prioritizing the lowest to be rerolled.
With regards to performance, the question would be how to avoid having to materialize the specific rerollable outcomes. Perhaps I can do the split first, then use highest/lowest to determine which rerollable outcomes were not rerolled. But then I'd still have to additive union the two pools, which prevents using Pool specializations on the result.
Interpret the limit on the number of rerolls as per-depth or total?
Limiting the total number of rerolls rather than per-depth seems probably a bad idea:

It's rare.
It's rare with good reason. If the allowed depth is >= the limit of rerolls, you're going to want to reroll just one die at a time so you have the maximum information before assigning each reroll. That's going to slow the game down, so I would hope no game designer actually does this.
Apart from that, this can turn into a more difficult optimization problem. For example, if Empowered Spell allowed higher depths, then there are cases that you would want to reroll a 4 since with enough rerolls you would have a good chance of turning it into a 5 or 6. That's going to make it harder to come up with an efficient solution and for the user to understand exactly what it is that's being calculated.

What default depth to use?
Currently the single-die reroll defaults depth = None; in my experience depth = 1 and depth = None appear pretty often in games. When it comes to pool rerolls I think depth = 1 is more common. But I don't want to have a different default for the single-die and the pool version.
Another option is to effectively only allow depth = 1. Unlimited depth with rerolls-per-depth is effectively just unlimited rerolls period, at which point you might as well use the single-die version. Finite depths above 1 seem rare.
We could also make depth a mandatory keyword argument. This might be a good idea anyways to force the user to make the depth unambiguous-at-a-glance.
MixtureMultisetGenerator -> PoolMixture
The idea would be to only allow Pools in the mixture rather than general MultisetGenerators.  This would allow to use the Pool specialization of highest etc.

It seems the easiest thing to do when selecting which dice to reroll is to pick at random. However, this seems less than intuitive for the user. Options:

Just accept it.
Only allow to reroll a single outcome.
Always use a priority, but then we have the issue as above.
** The highest issue could potentially be improved if we track whether generators produce fixed-size multisets, and allow the specialization for general expressions if so. We may still want the Pool specialization since that one also has the benefit of skipping to the end if no kept dice remain.

Seems good for now.


Create Zod Error mapping utility

Each adapter method is wrapped in a Zod schema that parses the inputs and outputs to the adapter.  Kneel before zod.
If either the input or outputs are incorrect, zod will throw a zod error.
We need a mapping utility to map zod errors to a more user friendly message, perhaps into a hyper error.

Handled in #468


Clipping plane/Bounding box in 3D view can hide a model that is loaded

Steps to reproduce
Load the SoccerKickingModel.osim
Expected result
The bounding box is adjusted so that model and all its graphical elements are in view.
Actual result
The model is not in view. Randomly scrolling happens to get you to "bump" into the model.

Moving the view and zooming is still unable to get the complete model in view without cutting off details.

Environment and GUI version
Mac OpenSim c596914-2018-02-24

Trying to load the model from latest master I get:
Loaded model SoccerBallKickingModel from file E:\src\opensim-gui\opensim-models\Models\SoccerKick\SoccerKickingModel.osim
Body 'tibia_r' not connected by a Joint.
A FreeJoint will be added to connect it to ground.
Where do I get the latest working version? Thanks

Where do I get the latest working version? Thanks

"master" branch of https://github.com/opensim-org/opensim-models repo. I just tested using AppVeyor artifact OpenSim-be3c7698-2018-04-05.zip. Please ensure your local clone has been synced.

For this model I highly recommend though that:

We make grass floor, use "texture" for it.
Use Skybox that's all clouds or add another one that's track/field/outdoors
Remove sky, ground from model as they are not used other than visualization and they introduce edges/boundaries that are awkward to workaround/visualize while floor and skyboxes are better at that.

When opening soccer model, the whole model is in the clipping plane. There are some issues with the soccer model, but the clipping plane issue seems to be resolved. @aseth1 do you have any specific test or behaviour you expect?
W10, OpenSim-a0d644b3-2018-05-04.zip

the whole model is in the clipping plane

Please see this image for the terminology.

Did fixing this cause Z-fighting (#826)?


"Select for Merge" almost like Compare

I'm working with a local copy of the template files used in production....
in this organization, we have people who edit directly in the production files ... crazy i know, right ?
anyways - i'm developing using my local templates and sometimes need to sync with production.
Therefore it would be awesome if i could use the Merge tool to make both files equal .
eg: "Select for Merge A + B " .. then go through the merge procedure..  afterwords the output of the merge is then overwriting both files.
this would make life so much easier for me, and everyone else who occasionally need to sync files

Seems to be a duplicate / alternate version of https://github.com/Microsoft/vscode/issues/25887

duplicate of #25887


Support custom CONNECT headers (both request and response) during HTTPS proxy requests

Note: Just throwing this idea out there -- I implemented these changes for my own use case, but I'm happy to rework the implementation if it's not deemed suitable for merging. Let me know.
The Issue
Some proxies accept custom headers, for things like switching IPs, excluding certain IPs, or specifying custom behaviors (like retrying requests when certain codes are returned). Over normal HTTP, these can be sent along with the rest of your headers, and the proxy will filter the headers before passing the request along. Likewise, the proxy might inject a custom header into the response (such as one indicating the IP your request was passed through).
However, everything changes over a secured connection. When connecting to a proxy over HTTPS, this gem makes use of HTTP tunneling via the 'CONNECT'  method. The underlying request and response are encrypted. As a result, the proxy is unable to read any headers you send with the underlying request or inject headers into the underlying response.
So adding custom proxy-specific headers to a HTTPS request looks something like this:
CONNECT example.com:1234 HTTP/1.1
Proxy-Authorization: Digest [your auth digest]
X-My-Custom-Header: Yes

<Your Underlying Request Goes Here>

And a response from the proxy (with a custom header) looks like this:
HTTP/1.1 200 Connection established
X-Custom-Proxy-Response: Success

<The Underlying Response Appears Here>

The problem is that, aside from Proxy-Authorization, this gem does not support adding custom headers (like X-My-Custom-Header and X-Custom-Proxy-Response above).
The Change(s)
This PR adds a parameter to the via method which allows you to specify custom headers intended for use by the proxy:
HTTP.via("proxy-hostname.local", 8080, 'x-retry-if' => 502)
    .get("https://example.com/resource")
HTTP.via("proxy-hostname.local", 8080, USERNAME, PASSWORD, 'x-use-ip' => '123.456.11.11')
    .get("https://example.com/resource")

Note that if the underlying request is 'http' instead of 'https', there is no CONNECT tunneling, so the proxy-specific headers will be merged into the requests's primary headers.
Similarly, if the proxy returns any custom headers (as shown above), they are made available as follows:
response = HTTP.via("proxy-hostname.local", 8080).get("https://example.com/resource")

response.proxy_headers
# => { 'x-ip-used' => '123.456.42.42' }

Note: I considered transparently merging these into the main request headers, but I worried that this would cause strange behavior. Either a proxy could overwrite a header you expect from the remote server, or the remote server could overwrite a header you expect from the proxy, and I wasn't sure how to handle these cases. If you have any suggestions I'd be happy to rework the behavior.
Questions?
Does all of this make sense? Let me know if you have any questions, ideas, suggestions, etc.

Seems reasonable to me, @tarcieri @ixti @sferik ?

:+1:

This will go out in 2.0.0 when we eventually release it.


[WIP] Bumps dialog lib to the latest version, in order to keep compatibility with the latest version

In this PR we are updating our dialog-lib version to keep dialog in the latest version. Right now, the main branch uses v<IP_ADDRESS> as the version.
This PR addresses #218, #217 and #213.

@avelino thank you for that comment, yep, it's still a WIP, but your comment is totally valued!


Get wrong sign when no permission

BkCommonLib version:
1.14.2 556
TrainCarts version:
1.14.2 466
Spigot version:
1.14.2 git-Spigot-baafee9-17543ec

Problem or bug:
When you don't have permission to place a sign you always get an oak sign.
Expected behaviour:
Get the sign of the type of wood used
Steps to reproduce:
Place a [train] station sign without the proper permission with a wooden sign that is not oak.

Should work with latest dev builds, can you take a look?


This lib is not working

It's using outdated URLs, I'm trying to use it to fetch my borrow limit usage, but it gives a 404 error.
Mainnet API has changed URLs, but even if I update the base api url it still doesn't work, is this lib deprecated? Is there a new one?

Fixed by ea45a8f87f2324279eb4810624183dc719675720


ScopeProvider calling referenced atom multiple times for each ScopeProvider intance

I'm still working on how to model this properly.
But I have a ScopeProvider inside of a list where one of the atoms is scoped for this list item and then further down the tree I reference a global atom that references the scoped atom. All of which is working fine to yielding the correct data
HOWEVER it seems that when I update another referenced atom inside the global atom it causes the ScopeProvider to re-render and calling the getter of the global atom for each iteration yielding the same results redundantly. Normally the getter memoized if nothing has changed not get called.
ex. If list has 10 items. The atom getter will be called for the 1st element 10 times, 2nd item 10 times, etc. etc.
I'm not sure why it's triggering this behavior but I will update when I have a better example demo example.

I realized when tracing. This may also be caused in conjunction with my usage of jotai-optics focusAtom in this context.
I will try and isolate that out.

I'm still not sure this is desired behavior but in my case I was setting passing global atom value obtained from outside of the scope and passing it into a hook below the scope. Then using a useEffect to update the scoped version of the atom.
The idea was sort of a global override of individual list items. But this useEffect is what is causing the 10x10 behavior.

Closing issue for now. I believe this may be a logic problem in my system and behavior is working as intended. Unable to replicate the exact scenario in clean app.


Google Drive (which also uses Angular) greys out screen and shows a loader until the essential data needed to show basic ui elements are ready

done here https://github.com/fossasia/open-event-webapp/pull/64


Chef and Chef Knife stuck in preview

These tasks need to come out of preview for QU2 or get deleted since they were never non preview.
Address by 116 so you have a sprint of runway for 117 cutoff.

@bryanmacfarlane @kmkumaran
I'm working with Chef team get it converted into an official extension from Chef. There are active deployment with this task. Without replacement, I don't want to pull this out.
I'll check back with Chef team about availability of the extension. We might have to hold this a little longer.
Thanks,


typos

before:
Brainfreeze focuses on the developer gaining more control over the their application state.
after:
Brainfreeze focuses on the developer gaining more control over their application state.

Thanks! @diegofaria


sk-dist Conda package

Hi,
I am sorry for not contacting you sooner, but I needed quite urgently to import sk-dist as a Conda package, so I created Conda-forge feedstock for your package. As this package does not have any special dependencies, this was a rather straightforward process. This means that sk-dist is now available as a conda package through conda-forge. Right now I am the maintainer of this feedstock, but I will be happy to forward it to you or add you as a maintainer.

Nice thanks. Please feel free to add me as a maintainer.


deps: Bump bazel-buildtools -> 4.2.0

Signed-off-by: Ryan Northey<EMAIL_ADDRESS>Commit Message: deps: Bump bazel-buildtools -> 4.2.0
Additional Description:
Risk Level:
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue] Fix #18347
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

You are aware 4.2.1 has been current for some time?

for bazel yes, but this is buildozer/buildifier so i didnt make the connection
i updated to 4.2.0 because that was the version on the ticket
we are just improving the issue creator so that it will have upstream links, which will make it easier to check what is current (#18374 )

ill update...


Add Sorting Algorithms

Sorting Algorithms📜
DSA: Sorting Algorithms #113

Added Documentation of Sorting Algorithms 📝

[x] Documentation (Content Creation in the form of codes or tutorials)
[x] Other (If you choose other, Please mention changes below)
Added Code to the Sorting Algorithms in C++.

Domain of Contribution 📊

[x] C/CPP
[x] Competitive Programming
[x] DSA
[x] Interview Prep

Checklist ✅

[x] I follow Contributing Guidelines & Code of conduct of this project.
[x] I have performed a self-review of my own code or work.
[x] I have commented my code, particularly in hard-to-understand areas.
[x] I'm GWOC'21 contributor

/assign

/assign

Kindly follow directions from PR templates while making PRs...
Only keep that domain in your PR comment to which you want to contribute (In this case DSA)


About mj_objectVelocity

Hello,
Many thanks for the continuous support. I have a question regarding the function mj_objectVelocity from the API. When used on a body, the velocity returned by this seems to be the velocity of the body's CoM (i.e., the velocity from which the kinetic energy corresponding to that body can be calculated). Is there some configuration of this function, or other API functions or mjData fields that returns the velocity of the body frame itself (which was defined as the body's position in the XML)? Potential solutions I considered:

Placing a site at the body frame origin, calling mj_objectVelocity on that.
Calculate the relative point velocity from the angular velocity and combine with the CoM linear velocity

Am I misunderstanding something about body frames? Is there an easier more straightforward method for this?
I'm using the C#/Unity bindings of MuJoCo. Here's my extension method for getting the global velocity of the body:

public static unsafe Vector3 GlobalVelocity(this MjBaseBody body, MujocoLib.mjtObj objType = MujocoLib.mjtObj.mjOBJ_BODY)
{
    MujocoLib.mjModel_* Model = MjScene.Instance.Model;
    MujocoLib.mjData_* Data = MjScene.Instance.Data;
    Vector3 bodyVel = Vector3.zero;
    double[] mjBodyVel = new double[6];
    fixed (double* res = mjBodyVel)
    {
        MujocoLib.mj_objectVelocity(Model, Data, (int)objType, body.MujocoId, res, 0);
        // Linear velocity is in the last 3 entries
        bodyVel = MjEngineTool.UnityVector3(MjEngineTool.MjVector3AtEntry(res, 1));
    }
    return bodyVel;
}

I'm interested in a variant that returns the velocity of the non-inertial body frame. The complicated version I have now is this (with a few helper functions omitted):

public static Vector3 GetBodyFrameVelocity(this mjBody)
{
    var velocity = mjBody.GetGlobalVelocity();
    var angularVelocity = mjBody.GetGlobalAngularVelocity();
    var framePosFromInertial = -mjBody.GetLocalCenterOfMassMatrix().GetPosition();
    return Vector3.Cross(TransformMatrix.MultiplyVector(framePosFromInertial), angularVelocity) + velocity;
}

Is there an easier more straightforward method for this?

Yes. Pass mjOBJ_XBODY instead of mjOBJ_BODY. This is documented here (ish).
We could maybe rename mjOBJ_BODY → mjOBJ_IBODY, but I'm worried that will break people's code.
This is true for everything that accepts an mjtObj as a frame reference, like mj_jac, sensors etc.


Fix concat operator

The sql parser was correctly parsing "||" as CONCAT but there was
no tpl parsing, tpl ast generation, semantic checking, or
bytecode generation for concat. This commit adds tpl ast
generation, semantic checking, and bytecode generation. tpl
parsing still isn't working properly. We reuse the concat
built-in bytecode handling as the underlying implementation.
Originally CONCAT was being treated as a normal BinaryOpExpr,
however CONCAT needs access to the Execution Context in order
to allocate space for a new string. Normal BinaryOpExpr don't
have a reference to the execution context. Therefore we needed
to create a new type of expression, StringBinaryOpExpr, which
has a reference to the execution context.
StringBinaryOpExpr was made to be generalized for use with  any
string binary operation. Some other options would have been:

Add a reference to the Execution Context to BinaryOpExpr.
However this is probably wasteful since the majority of binary
operations do not need a reference to the Execution Context.
Create a specific operation for CONCAT.
Generalize further and make some StringOpExpr, that instead
of having a left and right expression, has a vector of
expressions. This would allow it to be reused on string
operations that need a reference to the Execution Context and
have any amount of arguments.

Many elements were inspired by/taken out of the following PR:
https://github.com/cmu-db/terrier/pull/839/files
Fixes: #804
Co-authored-by: VenkatDatta<EMAIL_ADDRESS>

This ended up being a lot more code than I thought it'd be.
I wasn't able to figure out how to get a reference to the Execution Context during tpl parsing so if someone can help me with that it would be greatly appreciated.
I listed a couple of alternate approaches in the commit message. I made StringBinaryOpExpr to try and generalize all binary string operators, but I can't actually find any others besides concat. After finishing I think the general StringOpExpr would probably be a better approach, but before switching it all out I wanted a second opinion.

Remove support for || and just handle CONCAT(...). Then it becomes like any other builtin you've added before.
To support || we need pg_catalog.pg_operator so we can map || to the function concat().

Yeah, that's a much better approach actually. I'm just gonna close this PR then and open a new one for the built in Concat.


I am trying to achieve multi-colour text in a text representer but I still could not achieve good results. Any help ? Thanks

I am trying to achieve multi-colour text in a text representer but I still could not achieve good results. Any help ? Thanks, you can read about the details here:
https://stackoverflow.com/questions/63432527/i-want-to-achieve-color-in-a-textview-driven-by-the-hexfiend-framework-i-almost
Many thanks for any help.
Alfonso

Following another approach, I was able to draw the characters with a coloured background using the following code:
[[self.inMemoryController colorRanges] addObjectsFromArray:colorRangesArray];
[self.inMemoryController colorRangesDidChange];
This is not the solution I was looking for (see original question) but it is a nice to have. The only problem is that with many ranges, the HexFiend view resizing drastically decreases its resizing speed. If possible I would like a solution for this problem too. Thanks

Unfortunately this isn't really well supported right now, especially since the main app doesn't use it for any main features.
In #181 I did some experiment work around this but haven't progressed. If more is done here it'll probably be related to that feature.


[BugFix] Fix checkActionInDb get wrong privileges by ranger

Why I'm doing:
get wrong privileges by ranger.
Through the logs, it tries to get the insert permission for the materialized view, while the mv doesn't have this privilege.
so skip it.
RangerStarRocksAccessRequest | RangerAccessRequestImpl={resource={RangerResourceImpl={ownerUser={null} elements={database=example_db; catalog=default_catalog; materialized_view=order_mv; } }} accessType={insert} user={root} userGroups={} userRoles={} accessTime={Wed Sep 18 21:41:19 CST 2024} clientIPAddress={%} forwardedAddresses={} remoteIPAddress={null} clientType={starrocks} action={null} requestData={null} sessionId={null} resourceMatchingScope={SELF} clusterName={starrocks} clusterType={null} context={} }

What I'm doing:
Fixes #51124
What type of PR is this:

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Does this PR entail a change in behavior?

[x] Yes, this PR will result in a change in behavior.
[ ] No, this PR will not result in a change in behavior.

If yes, please specify the type of change:

[x] Interface/UI changes: syntax, type conversion, expression evaluation, display information
[ ] Parameter changes: default values, similar parameters but with different default values
[ ] Policy changes: use new policy to replace old one, functionality automatically enabled
[ ] Feature removed
[ ] Miscellaneous: upgrade & downgrade compatibility, etc.

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

[x] This is a backport pr

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.3
[x] 3.2
[x] 3.1
[x] 3.0
[ ] 2.5

@HangyuanLiu Please help me review the code


2.0: Scope of IAM permissions for AWS provider needs to be documented.

Currently docs suggest creating an admin account, provide a method to create a user with scope reduced to read access + inside vpc write access, and remove the need for the API user to have IAM access at all

@saheienko can you work with @TheKLARKEN to define the exact set of permissions needed by the AWS provider for all three tools? He needs to include this security policy in his documentation. cc: @egopher @stgleb


MatTextField : Label or Dense doesn't use color Theme

Hi!, If using a color theme (primary or secondary)  or simply via the website matblazor.com , we can be seen that the label color does not change :

Same with the arrow on the matselect

You can temp fix the label with the following css
.mdc-text-field--focused:not(.mdc-text-field--disabled) .mdc-floating-label {
color: rgb(96, 169, 44);
}
.mdc-floating-label--float-above {
color: rgb(96, 169, 44) !important;
}

This was recently fixed in PR #488 so I'm closing this one.


Highlight closing parens/bracket when selecting range

This is a feature request. I think it will be fantastically useful if when you select a range of text, the first opening parens/bracket in the range, and its corresponding closing parens/bracket, would be highlighted the same way that it's highlighted when you put the cursor next to parens.
The reason why it would be useful is that when you select a piece of code, especially if it is large, it's often hard to tell where you should end the selection, and usually that's where the closing parens/bracket is.
As as extra plus, this feature would be easily discoverable by users without needing to be told about it.
For selections with multiple ranges this can be disabled or applied to the first range - whichever is easier to implement.

I would like to refine this feature request a bit. Suppose you have a function in your code like this that you want to copy or cut (I'm making it short for clarity, but in reality the feature would have the most use if the function is longer), so you begin to select this text starting from the beginning:
const f = (x: number): { a: number } => {
  return { a: x };
};

The most naive way that the feature can be implemented is to highlight the parens/bracket corresponding to the first opening parens/bracket in the selected range:
/* selection start */const f = (x: number)/* <- highlighted */: { a: number } => {
  return { a: x };/* selection end */
};

But clearly, in this example the user probably wants to select entire function, so a better way would be to find the first location in the code up to which the selected range should be extended to close all opening parens/brackets:
/* selection start */const f = (x: number): { a: number } => {
  return { a: x };/* selection end */
}/* <- highlighted */;

The third way which is even better but would probably be harder to implement and has to be language-specific is to find the first location up to which you need to select to have no partially selected nodes in the syntax tree:
/* selection start */c/* selection end */onst f = (x: number): { a: number } => {
  return { a: x };
}/* <- highlighted */;


(maint) fully qualify the label in the osx puppet plist

This commit fully qualifies the label in the osx puppet plist using the
identifier plus service name.  Some formatting was also cleaned up.

Merge after 4.2.1 is tagged.

CLA signed by all contributors.


breaks fabritone

When enabled, fabritone will not do anything, walk, build, etc.
https://gitlab.com/CDAGaming/fabritone

Hi

Show the log

log.txt

There is nothing in the log, as far as I can tell.

@qouteall I enabled debug logging and found out that fabritone was stuck in this loop
https://gitlab.com/CDAGaming/fabritone/-/blob/master/src/main/java/baritone/pathing/path/PathExecutor.java#L190

According to my testing, movement is only possible in the current chunk, anything beyond that just spams that line of log.

I am investigating this.

Seems that Fabritone copies the chunk map. But this mod replaces the chunk array because the vanilla chunk array does not allow loading chunks beyond render distance.

Vanilla's ClientChunkManager is thread-safe so there is no need to copy the chunk map. It may be the heritage code from 1.12 version of baritone

@qouteall Thank you. I'll report this to baritone


DRAFT: Forecast figure enhancements

Closes https://github.com/CDCgov/pyrenew-hew/issues/167

Pausing on this to work on PyRenew-E and waiting on https://github.com/CDCgov/forecasttools/issues/29

Pausing on this to work on PyRenew-E and waiting on CDCgov/forecasttools#29

#31  has now landed.


Plan after map-store deprecation?

What is the plan with this work now that the map-store has been deprecated? I'd hate to see all of this work go to waste, as I also am trying to figure out what the future holds for caching and seamless upgrades. I opened a discussion about it here, in case you're interested in chiming in: https://github.com/keycloak/keycloak/discussions/24655
I haven't looked deeply into the existing/"legacy" SPIs for caching. Would it be possible to port the user/auth session portion of this to work? Happy to help if you think there is a clear path forward.

Don't worry, we will port this to the "legacy store" interfaces, as we are using this extension heavily internally. This is enabled by DatastoreProvider-SPI, which will remain even after map store is removed, as a central point of integration.
Help is always appreciated though, see our open issues or improvements you would like to make that we haven't thought of yet.

Really glad to hear this!

FYI I did a PoC on KC23 for only replacing the caches (*_sessions, single_use_objects, login_failures, etc.) with Cassandra, and using the underlying LegacyDatastoreProvider for everything else.
Had to yank a lot of stuff out, but got it working.
Guts are in this class: https://github.com/xgp/keycloak-cassandra-extension/blob/e39e9d17c6facc5065ae14a0213444bf475e7194/core/src/main/java/de/arbeitsagentur/opdt/keycloak/cassandra/CassandraMapDatastoreProvider.java
Full diff is here: https://github.com/opdt/keycloak-cassandra-extension/compare/main...xgp:keycloak-cassandra-extension:23-legacy-plus-session?expand=1

Thank you, we will provide a DatastoreProvider-implementation like yours for only replacing the cache areas. Our current progress can be seen in feature/kc-23 branch. It is already usable to replace the whole storage layer but it still has a few bugs left so it will take a little longer. We also discovered a few unpolished left-overs from map-store removal we would like to discuss with the Keycloak-team.

@opdt Thank you. I'll take a look at feature/kc-23. Please let me know if there is something that I can do to contribute.

@xgp Please try the newest release and the newly provided docker-compose.yaml


fixed broken build

When I tried to build sources, (cargo build --release), the following errors showed up: screenshot.
The problem was using named variables on format! macro without naming them on the macro's arguments. The solution was simply specify the variables on macro's args, and use them on the order they're passed in.
This patch fixed the errors for me.
Greetings.

I reviewed this, the problem was i didn't upgrade my rustc to 1.58 update notes here. However, feel free to close pull req if u dont want to support older versions of rustc.


Compile with /W3 on Windows

Enable /WX (treat warnings as errors).
Fix some basic errors that were found.
Enable /FC (emit full path on diagnostics) to make it easier for IDE.
Add (commented out) entries for enabling debug builds for dev.

Troubling... Check failed on Windows but it succeeded locally. Will investigate.

Updated local env to Python 3.9 to uncover new UNICODE deprecation warnings, which now trigger errors due to /WX!
Also fixed a double-close of an SDS handle during error handling that was uncovered during testing.


added new filter: match, it's test and info to README.md

Hi!
In order to perform request with empty query and match, created new filter "whereMatch".

I think it should be operator in where like ~
where('title', '~', 'Book title') and of cause where('title', '!~', 'Book title')

Please this would be very useful to me at this time.

Merged in #355. Will be released as v4.2.0


Add markdown files for static pages

https://github.com/CDLUC3/dmp/tree/development/app/views/static_pages
Create new markdown files for:
Quick Start Guide https://dmptool.org/help
Promote the DMPTool https://dmptool.org/promote (replaces For Developers page in menu)

all good now. thanks.

promote file is in place

yes it is.


Patience Sort Algorithm

Submission Title
Patience Sort Algorithm
Submission Type
Algorithm
Submission Description
Patience Sort Algorithm (https://en.wikipedia.org/wiki/Patience_sorting)
Submission Directory
Algorithms => Sorting_Algorithms => Patience_Sort => Patience_Sort.
Contributors

[x]  C++ -
[x]  C -
[x]  Python -
[ ] Java -
[ ] JavaScript -
[ ] TypeScript -
[ ] Golang -
[ ] Ruby -
[ ] php -
[ ] Swift -
[ ] C# -

Please assign this to me, I will do it in C. @ahampriyanshu @s2agrahari

I would like to do this in python.
Please assign me this one.

@ahampriyanshu  can you assign this to me i would implement this in python.

Can I do this in python or c++?

Can I do this in python or c++?

Please opt a different language/issue.

I can do it in C#

thanks everyone :v:


pipeline: bump pre instead of patch

Please do not merge until #29 is merged.
Signed-off-by: Jacob Blain Christen<EMAIL_ADDRESS>

@cloudxxx8 @SteveOss - please review these changes that are needed to automate the tagging schema we discussed in the DevOps WG meeting this morning.

recheck

recheck

@cloudxxx8 please merge this one


Specify classes to be assigned to particular days

To demonstrate it, I've modified the demo to highlight
weekends.  I also changed the order of the CSS in the demo
so that selected is visible over hover which is visible
over other-month.

Thanks for doing this.  I got sidetracked on some important issues at work. :-P

I can close this now.  #4 is the same without the conflicts.


Wireframe of Application

Wireframe showing the key flow of the application

Linking our figma here https://www.figma.com/file/Yfmu9x1lpnL3zIAyaG2oTM/Labs3-Design-System?node-id=1569%3A0

We already have an issue created for design, so by wireframe do you mean flowchart(s)? @falkonprods


Update Boulder Features

Gives the boulders a more variated texture rather than solid stone.
Also adds a chance for coal and iron ores to generate inside larger boulders.

These variants should be in their own script so we're not removing the pure stone versions, aside from that lgtm

k does that work?

The textured scripts are missing the .tesf extension, add those and I can merge :+1:

bet


Publish to package managers

I would like to add Rotz to some package managers to ease installation.
My initial thoughts are:

scoop and winget for windows
homebrew on macos
nix, snap or flatpak for linux.

but this is open for debate 😄

I have a prelimnary PKGBUILD for the arch user repo, if you want to get it published there. I'll be willing to take a look into flatpak as well.

Thank you, please share :)
I would use this github action to publish it to the AUR
For flatpak I would use flathub.
I'll set up the repo https://github.com/flathub/flathub/wiki/App-Submission#how-to-submit-an-app
and can give you access If you're interested in tackling this

I am loving this tool so I can do, or at least work on, the homebrew publishing. I have done it before with internal tools and private repos across multiple platforms. I'll open a PR with an outline of the required changes and we can discuss setting up github actions there.

Thank you @pho3nixf1re 🙏 that would be super helpful.
I don't have any experience publishing to package managers so any help is appreciated.

I don't think flatpack (and probably snap) would be a good fit for this. Flatpacks are self-contained and run within an virtual environment strictly sandboxed from the host. This app needs lots of access to the file system and the ability to run scripts on the host potentially with admin/root privileges. Potentially these could be granted but it would be a lot of permissions requests and other setup by the user to make work. This is a very simple single file binary so distributing it is easy. A better option may be to provide a bootstrap sh/pwsh script that a user can run via curl/invoke-webrequest or download and run. Similar to how oh-my-zsh or homebrew is setup.

Also, integrating an update-check mechanism into the code would be nice. That way each run it can self-check github's releases.

I'm not set on flatpack/snap they were just my first ideas. You bring up some good points against using them 🙏
I agree that a very good (if not the best) option is simple sh/pwsh bootstrap script. For updating we could experiment with the self_update crate or we could suggest using a tool like eget (with a suggestion to add it to the dotfiles to have Rotz install it). At least an update warning if a new version is available would be nice to have.
What would also be nice is to have Rotz available on "native" package managers that come distributed with the OS (like winget or apt etc.) because installing it would be the easiest that way.
I use Rotz to install non "native" package managers like scoop and homebrew so I would not use them to install Rotz but that's just me and no reason not to publish Rotz there 😁
These are my thoughts on the matter I'm open to ideas/feedback.
I'd welcome your help publishing Rotz on homebrew if you want to tackle that (or something else) 🚀

Looks like homebrew is already in progress by @kecrily at https://github.com/Homebrew/homebrew-core/pull/135639 . I'll see if I can help out that effort.

So one problem with winget I have ran into is that I cannot use it on some corporate laptops since they disable the Windows Store. winget requires the Store as a dependency to function for some reason. Otherwise I greatly prefer it over choco or scoop on windows when it has what I need. Certainly not opposed to publishing it as many places as possible though. Since brew is already done in the above link I will look into scoop/choco but I have never published anything there before. I'm sure it shouldn't be too bad right?

So would you be open to publishing the checksums in the releases description or as assets for each binary with a .sha256 extension? Example URL: https://github.com/volllly/rotz/releases/download/v0.9.4/rotz-x86_64-pc-windows-msvc.zip.sha256. This way we can pull this easily to update package managers. Possible even as part of the github actions workflows here.

Well scoop is a big disappointment. I had everything lined up and started opening the issue/PR but apparently you have to be a "well-known" and popular (lots of github stars) project to be in their repos. The fallback would be to create your own Scoop bucket git repo and drop the json in there. If that is something you could create I'll open the PR with the manifest file I created.

Here it is for reference https://github.com/pho3nixf1re/ScoopMain/blob/add-rotz/bucket/rotz.json

Hey, I think we could get into the scoop extras bucket.

Extra manifests for Scoop, the Windows command-line installer. For manifests that don't fit the Main criteria.

It seems homebrew has the same problem though. the PR has gotten the notability label ("Project is not notable enough for inclusion").
I think thats overkill ATM but I could move rotz into its own github organization and add a Tap there 🤔
Short term I'll create simple install sh/pwsh scripts #280
I've also added an issue for creating checksum files #279 👌 I'll tackle that with the next release.

Unfortunately the extras bucket also has a popularity requirement. It's reduced but it is still there. I think creating your own tab/bucket is probably best. It's not that tough of a step for users to do that as a pre-install step. The sh/pwsh script is still a good fall back for many use-cases and is a common enough technique.

Ah, you're right. I found 100 stars and 50 forks as the requirement in an issue there.
I think I'll move Rotz into a GitHub organisation in the next few weeks and we can setup a homebrew Tap and a scoop Bucket there 👌

Hey, I've created a personal scoop bucket and homebrew tap. The manifests were adopted from your previous efforts. Huge thanks for the help🙏
Both should be kept up to date automatically through renovate for homebrew and the autoupdate functionality of scoop.
In short: Rotz can now be installed with brew install volllly/tap/rotz or scoop bucket add volllly https://github.com/volllly/scoop-bucket; scoop install volllly/rotz 🎉


Improper job count

Everytime the the alerts enabled comes up and it runs the checks it adds the iteration number to the total since start of script and I believe this then causes it to think it has reached the max? Plots to plot

Interesting, thanks for reporting. I will take a look. I think this is due to that the notification gets addedd to the output upon which "Start-PlotJobs" counts.

would a "jobs remaining" be better instead of the "overall spawned plots since start"
I changed the code around I'm trying to see if this works so far so good it
        if ($SpawnedPlots)
            {
                $SpawnedCountThisIteration = ($SpawnedPlots | Measure-Object).Count
                $SpawnedCountOverall = $InputAmountToSpawn - $SpawnedCountThisIteration

                Write-Host "PlotoManager @"(Get-Date)": Amount of spawned Plots so far:" $SpawnedCountThisIteration
                Write-Host "PlotoManager @"(Get-Date)": remaining Plots to spawn:" $SpawnedCountOverall
                Write-Host "________________________________________________________________________________________"
            }

        Start-Sleep ($IntervallToWait*60)
    }

    Until ($SpawnedCountOverall = 0)
    Write-Host "We are done!" -ForegroundColor Green

    $VerbosePreference = $oldverbose

}

Glad you got it going! I fixed with piping the Output of the Discord notification functions to Out-Null. I‘m currently testing and will probably push it later.


Package.loadSettings is deprecated.

Hi,
after updating your package I still have the deprecation warning. Here is the warning I get
I'll fix it manually.
Thanks
Store package settings files in the settings/ directory instead of scoped-properties/
Package.loadSettings (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:552:9)
<unknown> (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:168:43)
Package.measure (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:147:15)
Package.load (/Applications/Atom.app/Contents/Resources/app.asar/src/package.js:161:12)

Ok my bad it is working anyway. The problem for now is that the package doesn't know which version of atom is running on.
I'm starting to develop on atom and if I have any idea how to do it I'll let you know ( if you did not do it tho =) )
Thanks again


Update Gradle version

The plugin crahses with error
The Android Gradle plugin supports only Kotlin Gradle plugin version 1.3.10 and higher.
The following dependencies do not satisfy the required version:
project ':flutter_audio_player' -> org.jetbrains.kotlin:kotlin-gradle-plugin:1.3.0

Do you installed google plugin? they said google update to androidx
I think we can try to use this way?
[android.useAndroidX=true android.enableJetifier=true]
(https://stackoverflow.com/questions/59496368/execution-failed-for-task-flutter-twitterverifyreleaseresources)

I am already using Android x.

the problem is not about your flutter app, its about audio_player not upgrade to androidx yet.

If you have migrate to androidx, switch to master-androidx branch.


Main context is None when it should be default

Running the background_action_example.py I get an error about the main context being None:
jbarry@bdai_docker:/workspaces/bdai/ws/src/external/ros_utilities/bdai_ros2_wrappers/examples$ python background_action_example.py
Traceback (most recent call last):
  File "/workspaces/bdai/ws/src/external/ros_utilities/bdai_ros2_wrappers/examples/background_action_example.py", line 84, in <module>
    main()
  File "/workspaces/bdai/_build/build/bdai_ros2_wrappers/bdai_ros2_wrappers/process.py", line 176, in __call__
    return func_taking_no_args()
  File "/workspaces/bdai/ws/src/external/ros_utilities/bdai_ros2_wrappers/examples/background_action_example.py", line 69, in main
    while main.context.ok():
AttributeError: 'NoneType' object has no attribute 'ok'

As per offline discussion, the issue is that ROSAwareScope.context is in fact None. For the underlying ROS 2 APIs, that is code for using the (global) default context. It is important that the scope instance retains that None value to survive (global) default context re-initializations, but the property should not yield None but the corresponding context object.


Snark pool not checking snarks

I may be wrong but it looks like we don't check snarks when putting them into the snark pool/rebroadcasting them. This is of course wrong. However, it is still a big DDOS channel (since someone can flood us with correct proofs of useless statements) so we need to figure out something there.

Ongoing refactor work by @enolan will help with this.


Android installation mode not redirecting

game: Windows 10, Unity 5.3.5f1, HFT 2.0.5, local ip <IP_ADDRESS>
controller: Project Tango Tablet, Android 4.4.2, Chrome 51.0.2704.81
Running the sample 2DPlatformer scene.
Tablet gets a "sign into wifi" notification. Tapping it opens chrome with an attempt to download "http://<IP_ADDRESS>/generate_204" which is a zero byte file.
If I manually navigate to <IP_ADDRESS> after that, it loads the controller.

Thanks for the report. I try to look into it ASAP but I'm travelling for the next week so I can't promise when I can find the time.

So I took a look and am unable to repo here. Unfortunately I don't have Android 4.4.2 though. I'm on 6.x
I'm not sure it will help but if you could run happyfuntimes in the editor in installation mode with your router configured etc..
Finally turn on networking on your phone, connect to the Wifi, do whatever you did before.
Then stop unity, quit unity, and drag and drop the log file onto this issue
maybe I can see something in there that might help
The log file should be here
C:\Users\<username>\AppData\Local\Unity\Editor\Editor.log
But you can't access it while unity is running

Oh,I forgot,put * in the debug field in Window->HappyFunTimes->Settings


diffoscope: wrap executable and include tools in PATH

Motivation for this change
diffoscope was looking for the tools it uses during runtime, but the
tools there neither part of the closure nor were they in the
PATH. This commit fixes this.
Things done
Wrapped the diffoscope executable.

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

[X] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[X] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Thanks!

Also fixed in 17.03.

Doesn't this create a double wrapper script?

ah, right. An alternative would been to use makeWrapperArgs

Yes it does. But is this a problem?
Am 21.07.2017 2:51 nachm. schrieb "Eelco Dolstra"<EMAIL_ADDRESS>
:

Doesn't this create a double wrapper script?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/27534#issuecomment-316992692, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGPa1J06niOYaTyni37Mfh2fu1m6WJEMks5sQJ7YgaJpZM4Oe9dS
.

And what would be a better way to do this?
Am 21.07.2017 3:47 nachm. schrieb :

Yes it does. But is this a problem?
Am 21.07.2017 2:51 nachm. schrieb "Eelco Dolstra" <
<EMAIL_ADDRESS>
Doesn't this create a double wrapper script?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/NixOS/nixpkgs/pull/27534#issuecomment-316992692, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AGPa1J06niOYaTyni37Mfh2fu1m6WJEMks5sQJ7YgaJpZM4Oe9dS
.

See also https://github.com/NixOS/nixpkgs/commit/91dc81156647c5d28e77dfd20d5c2f4b7253c742#commitcomment-23226040


Allow to directly provide the webhook callback url

Feature Request
Hey 👋
This is about the twitch-webhooks package.
I had a hard time figuring out what to put in the WebHookListener.create options structure in order to have the lib send the right callback url to twitch. It's not obvious what to put in the reverse proxy params to have the lib generate the right url.
I think it would be more straightforward to directly provide the url that will end up as hub.callback in the subscription request.
In my case, my inputs are: listening port + callback url so I even coded a thing that translates that to the structure (⚠ may not work in all cases)
  getCallbackProperties(listeningPort: number, callbackUrl: string) {
    const urlRegex = callbackUrl.match(
      /^(https?):\/\/([^/:]*)(:[0-9]+)?(\/.*)?$/
    );
    if (!urlRegex) {
      throw new Error(`Invalid callback url: ${callbackUrl}`);
    }
    const ssl = urlRegex[1] === "https";
    const hostName = urlRegex[2];
    const port = urlRegex[3]
      ? parseInt(urlRegex[3].substring(1), 10)
      : undefined;
    const pathPrefix = urlRegex[4] === "/" ? undefined : urlRegex[4];
    const behindProxy = ssl || pathPrefix || port !== listeningPort;
    const reverseProxy = behindProxy
      ? {
          ssl,
          pathPrefix,
          port: port ? port : ssl ? 443 : 80
        }
      : undefined;
    return {
      port: listeningPort,
      hostName,
      reverseProxy
    };
  }

(this is not pretty 😂 )
Anyway, you did a great work on those packages I'm amazed 👏 👏 👏

There is already an effort to establish a more friendly way to pass the callback parameters at #109. Please add your feedback there.
Therefore closing this as a duplicate of said issue.


Support alt_names in Vault request

Is your feature request related to a problem? Please describe.
At the moment it is only possible to specify uri_sans and other_sans parameters to a Vault issuer. To use these features one must disable use_csr_sans in the Vault role, which means any DNS SANs in the CSR would be ignored.
Describe the solution you'd like
Add the ability for the Vault alt_names parameter to be configured as a part of the vault.Issuer struct similar to the existing URISubjectAlternativeNames and OtherSubjectAlternativeNames options. Note that the Vault alt_names parameter can be used for both DNS and Email names. For completeness it might be worth adding the ip_sans option at the same time which would fully round out the issuer to expose all underlying Vault options.
Describe alternatives you've considered
The obvious alternative is setting use_csr_sans to true in the Vault role and simply using the CertConfig to specify all SANs. The downside to this is this would require doing all access control in the PKI role (eg, one role per application) and being unable to use the otherwise powerful Vault policy configuration which allows specifying request arguments as a part of the Vault policy and can be more powerful.
Adding the options to the Vault issuer would still permit this option as existing behaviour would not change when those fields are not provided, but enables flexibility where required.
Additional context
https://www.vaultproject.io/api-docs/secret/pki/#sign-certificate

Hi @nvx, thanks for the proposal! This sounds really good, and I think it should be fairly straightforward to add. Would you be interested in helping contribute this? I'd love to have someone else get their hands dirty to reduce the bus factor of this library. I would start here: https://github.com/johanbrandhorst/certify/blob/master/issuers/vault/vault.go#L22. You could even go ahead and TDD this by adding support to the tests first, maybe copy this and adjust: https://github.com/johanbrandhorst/certify/blob/master/issuers/vault/vault_test.go#L134-L170.

Indeed, you might remember me from PR #70. Just wanted to make sure there wasn't any issue with the concept before starting. :)

Should this be closed now that #115 is merged? Or do you want to wait until you cut a new release first?

I thought it would be auto-closed :joy:


Snowplow CLI: upgrade Sluice to version using logging (0.3.0)

See https://github.com/snowplow/sluice/issues/2

Sluice is going away


Update model.txt

Proposed changes
The default user model should not expose password by default.
Types of changes
What types of changes does your code introduce?
Put an x in the boxes that apply

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

Checklist
Put an x in the boxes that apply. You can also fill these out after creating the PR. If you're unsure about any of them, don't hesitate to ask. We're here to help! This is simply a reminder of what we are going to look for before merging your code.

[x] I have read the CONTRIBUTING doc
[ ] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works.
[ ] I have added necessary documentation (if appropriate)

Further comments
—

Thanks 😄

Hey @thetutlage , Thanks a lot for the merge, I'm still tinkering and I'm planning to improve a bit the docs & maybe internal code regarding adonis itself. As you know, we're undergoing hacktoberfest. Would it be possible to label this pull request as hacktoberfest-accepted in case you feel it was a well-enough work? I know it's not much of a change, but I had to dig a bit to find those compromises. I would understand if you didn't want to label it as such. There's also no need to set the project topic, marking a PR with that label is enough to help me towards this anual goal! Thanks a lot!


Javascript script couldn't load a template from pdf

Hi,
First of all I apologize if that is a duplicate of another ticket I haven't found.
Attach (recommended) or Link to PDF file here:
test getTemplate.pdf
Configuration:

Web browser and its version: Chrome 100.0.4896.127 | Mozilla Firefox 99.0.1
Operating system and its version: Mac OS BigSur (11.4)
PDF.js version: https://mozilla.github.io/pdf.js/web/viewer.html
Is a browser extension: no

Steps to reproduce the problem:

Load the given pdf file
click the checkbox

What is the expected behavior? (add screenshot)
Before clicking the checkbox, pdf should contain only one page.

After clicking the checkbox, pdf should contain an additional page.

As this is the behavior on Adobe, using a page template.
What went wrong? (add screenshot)
The execution of the  javascript script in sandbox resulted in an error :

To help you here is the script played when clicking on the checkbox in my example.
if(this.event.target.isBoxChecked(0)) {
    this.getTemplate("template").spawn(1, false, false).hidden = false;
} else {
    this.deletePages({nStart: 1});
}

Is this an expected behavior or a bug ? Is there a way to get templates from pdf document ?
Thank you for your time.

Template stuff is not implemented.
Our main gain in adding support for embedded JS was to help the user to file forms, validate them, ...

Hi,
Thank you for your reply.
Is this in your forseen features at some point ?
Thank you once again for your time.

This all sounds very much like an edge-case feature (at least to me), and also note that the viewer hasn't been written with the intention that it should be possible to add/remove pages (hence that part wouldn't exactly be trivial to implement either).

The successful problem solver who can assist in squaring away this issue will be rewarded with 2000 USD. If you believe you have the necessary skills to troubleshoot this issue, kindly send us a message, outlining your proposed solution. We will follow up with you.


Create Python 0.py

Adding a language

[ ] The code displays "Hello World"

Looks like Python 2 to me.


[react-network] Empty cookies are still being set

Some cookies still get passed through
curl -IXGET "https://signage.myshopify.io/tools/signage-maker" --cookie "_cookieA=this%2Bhas%2Ba%2Bplus; _cookieB=" --silent | grep "_cookieA\|_cookieB"

set-cookie: _cookieB=; path=/; expires=Thu, 01 Jan 1970 00:00:00 GMT; HttpOnly; secure

This needs to be changed to typeof rawCookieValue  !== 'undefined' && ... because currently it skips the equals check if the cookie is empty.

fixed by #1517


k8s api authentication

Describe the bug
Authentication error 403 in k8s api
Error
Set k8s api bearer token from file or from env FALCO_K8S_API_CERT
[root@nodename~]# /usr/bin/falco --k8s-api-cert=/etc/falco/token --k8s-api="http://K8S_API_DNS_NAME/" --cri /run/containerd/containerd.sock -pk --disable-cri-async --stats-interval 99 -s /var/log/falco/stats.log
Falco version 0.27.0 (driver version 5c0b863ddade7a45568c0ac97d037422c9efb750)
Falco initialized with configuration file /etc/falco/falco.yaml
Configured rules filenames:
   /etc/falco/falco_rules.yaml
   /etc/falco/falco_rules.local.yaml
   /etc/falco/k8s_audit_rules.yaml
Loading rules from file /etc/falco/falco_rules.yaml:
Loading rules from file /etc/falco/falco_rules.local.yaml:
Loading rules from file /etc/falco/k8s_audit_rules.yaml:
Starting internal webserver, listening on port 8765
Runtime error: Error during connection attempt to http://K8S_API_DNS_NAME/ (socket=23, error=4): Interrupted system call. Exiting.

In network dump I don't see bearer token in request

kubernetes version: v1.18.6
Falco version:

Falco version: 0.27.0
Driver version: 5c0b863ddade7a45568c0ac97d037422c9efb750

System info:

{
  "machine": "x86_64",
  "nodename": "nodename",
  "release": "4.18.0-240.15.1.el8_3.x86_64",
  "sysname": "Linux",
  "version": "#1 SMP Wed Feb 3 03:12:15 EST 2021"
}

Cloud provider or hardware configuration: Baremetal server
OS:

NAME="Red Hat Enterprise Linux"
VERSION="8.2 (Ootpa)"
ID="rhel"
ID_LIKE="fedora"
VERSION_ID="8.2"
PLATFORM_ID="platform:el8"
PRETTY_NAME="Red Hat Enterprise Linux 8.2 (Ootpa)"
ANSI_COLOR="0;31"
CPE_NAME="cpe:/o:redhat:enterprise_linux:8.2:GA"
HOME_URL="https://www.redhat.com/"
BUG_REPORT_URL="https://bugzilla.redhat.com/"

REDHAT_BUGZILLA_PRODUCT="Red Hat Enterprise Linux 8"
REDHAT_BUGZILLA_PRODUCT_VERSION=8.2
REDHAT_SUPPORT_PRODUCT="Red Hat Enterprise Linux"
REDHAT_SUPPORT_PRODUCT_VERSION="8.2"

Kernel: Linux nodename 4.18.0-240.15.1.el8_3.x86_64 #1 SMP Wed Feb 3 03:12:15 EST 2021 x86_64 x86_64 x86_64 GNU/Linu
Installation method: RPM

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

/remove-lifecycle stale

Issues go stale after 90d of inactivity.
Mark the issue as fresh with /remove-lifecycle stale.
Stale issues rot after an additional 30d of inactivity and eventually close.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle stale

Stale issues rot after 30d of inactivity.
Mark the issue as fresh with /remove-lifecycle rotten.
Rotten issues close after an additional 30d of inactivity.
If this issue is safe to close now please do so with /close.
Provide feedback via https://github.com/falcosecurity/community.
/lifecycle rotten

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

@poiana: Closing this issue.

In response to this:

Rotten issues close after 30d of inactivity.
Reopen the issue with /reopen.
Mark the issue as fresh with /remove-lifecycle rotten.
Provide feedback via https://github.com/falcosecurity/community.
/close

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


⚠ ClamAV has degraded performance

In f8c653f, ClamAV (https://spamassassin.apache.org/) experienced degraded performance:

HTTP code: 200
Response time: 80 ms

Resolved: ClamAV performance has improved in 36d04e6.


FORK HEMPCOIN THC WITH MASTERNODE HUGE BOUNTY PAID $$$

Looking to fork Hempcoin THC to the most current popular masternode code.
HUGE BOUNTY PAID
message me
Hank

did anyone do it?


Tests are insufficient for JS code

See below image.
If you make an intentional typo, I'd like to see some JS tests pass.
Incidentally, this is caught nicely by the rspec tests.

We'll probably use this as an example:
https://github.com/gaearon/redux/tree/breaking-changes-1.0/examples/todomvc/test
CC: @alexfedoseev

I'm setting up test env for redux at the moment, will take a look at this right after.

This was fixed a while ago: https://github.com/shakacode/react-webpack-rails-tutorial/tree/master/client/test


Add page transitions

add page transitions when switching tabs

consider:
Adding a subtle page curl effect at the edge during the slide.


Fixed #203. z-index and hover color fix

Added correct z-index for the inputfield. In Sass used transparentize to fix hover color. Also removed shadow outline on input field focus in chrome.

Looks good to me. One more reviewer and we can merge it in.

Looks great--approved! Side note--it will be so nice when we only need to work on the SASS files :)


Feature: Okta SSO Provider

Adds Okta as an option for an SSO Provider using authorisation code.

Adds Okta class to sso/views alongside existing providers which handles auth call backs and subsequent user validation / authentication.
Updated sso js to send the correct url params to Okta.
Updated env/configs to include OKTA PARAMS.
Added a fairly basic unit test to cover some edge cases for expired/bad tokens.

LGTM


NewHunt page looks ugly

The "New Hunt" page looks bad. This is because all the text elements and +/- buttons are too big. Shrink the text elements and buttons to make it look nicer.

Fixed! The page looks much cleaner and nicer!


Custom chat user categories

Allow to define custom categories with custom filters

Want to back this issue? Post a bounty on it! We accept bounties via Bountysource.

duplicate


PMM-8170 Add pmm-server as a product

To be able to see what operator versions are compatible with DBaaS, which is part of pmm-server, we add pmm-server as a new product.
Basically, two routes are added with this change:
GET /versions/v1/pmm-server
GET /versions/v1/pmm-server/2.19.0

Test name
Status

api-tests
failed

Test name
Status

api-tests
failed


chore(bullmq): ignore .tsbuildinfo

Information

Type
Breaking change

Chore
No

We can safely ignore the .tsbuildinfo files generated by tsc as they have no impact on our runtime.
I'm not sure whether this declaration is sufficient to remove the file from the released bundle but should it be at least for the @tsed/bullmq package it would be a massive reduce in bytes downloaded as the .tsbuildinfo files currently make up almost 90% of the package size.
I propose that once we verified here that this works we add this to all other packages too.
More Info:
https://www.typescriptlang.org/docs/handbook/release-notes/typescript-3-4.html#faster-subsequent-builds-with-the---incremental-flag

Todos

[ ] Tests
[ ] Coverage
[ ] Example
[ ] Documentation

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

v7.53.0
GitHub release

Your semantic-release bot :package::rocket:


Normalize ListParameter to be Immutable

Prompted by PR #1719
Description
ListParameter was not immutable. Parameter's normalize() was overwritten by casting the list to a tuple.
Motivation and Context
Parameters need to be immutable.
Have you tested this? If so, how?
Test is included.

[Current coverage][cc-pull] is 76.11%

Merging [#1759][cc-pull] into [master][cc-base-branch] will decrease coverage by 2.67%

@@             master      #1759   diff @@
==========================================
  Files            95         95
  Lines         10348      10350     +2
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
- Hits           8153       7878   -275
- Misses         2195       2472   +277
  Partials          0          0

Powered by Codecov. Last updated by [98860f5...d418b7e][cc-compare]
[cc-base-branch]: https://codecov.io/gh/spotify/luigi/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/spotify/luigi/compare/98860f5e6fe8c784d38ebdbd32fea731c4a8e849...d418b7e99177cccfb5566b6e0b3029ccba154b8a?src=pr
[cc-pull]: https://codecov.io/gh/spotify/luigi/pull/1759?src=pr

Yea, exactly what I wanted. :)

Yay, glad this is all squared away now!


Provide more specific return type "ConcreteImpl" over abstract one

I wonder why the package suggests to use a more specific return type, if a method is typed using a interface type.
<?php

namespace IfaceOverConcrete;

class Foo {
    public function doFoo(): MyInterface { // Provide more specific return type "IfaceOverConcrete\Impl1" over abstract one
        return new Impl1();
    }
}

interface MyInterface
{

}

class Impl1 implements MyInterface
{

}

having a method typed with an interface type instead of a concrete subtype is the better way regarding OOP.
I think the suggestion in this case is wrong and misleading.
typing a concrete impl will leak implementation details over concepts (the interface type or abstract class type) and hurt swappability

what is the idea of this rule? I can't think of a example when using a concrete subtype would improve something.
other linting tools I used in the past even propose the other direction. swapping concrete-impl types with interface-types in case non impl specific methods are used anywhere

In your provided example, there is no added value in using more generic return type. Its like having scalar return type, but returning a string :)
The goal is to report exact type possible, unless required by some parent interface. In some cases, interface can require methods that are no longer used.
If exact type is provided, this can be checked.

Also, classes can have extra public methods over interface. I recall that's how this rule started. There was false positive on extra public method.

So it sounds like this rule is reporting errors in the common case which only work for edge-cases? Should it be opt-in?

Purpose of this rule is to eliminate these types of hard-to-find errors and avoid leaky design. As this whole package :)
Generic interface should be used only if replaceability is needed. If it doesn't fit your preference, feel free to ignore it.


Problem with uploading of identical files

When I am trying to upload 2 identical files with different names in the same request, apollo-upload-server doesn't handle it well. This is probably due strange behavior of busboy. Problematic piece of code:
/src/index.mjs
const upload = new Promise(resolve =>
  busboy.on('file', (fieldName, stream, filename, encoding, mimetype) =>
    fieldName === mapFieldName && resolve({ stream, filename, mimetype, encoding })
  )
)

This statement (mapFieldName && resolve) always returns false when it parses the second file, so the promise never gets resolved.
Is there any way to solve this issue?
Thanks for the cool library, loving it!

That's really strange, because busboy should not care if the file contents are the same or different.
In the PR for the next version, there are tests for mulitple files in one request and the same test file is used multiple times and everything works ok. The use of listeners and things has been totally refactored for the update, so that code example is gone now anyway.
Feel free to test out the PR early, otherwise I hope to release it sometime early this week.


events.duration() fails after 1000 rows

Hi all and @tomhollingworth,
I have a strange behaviour using events.duration( )
Every 1000th row events.duration(unit: 1m, stopColumn: "_stop" ) is not calculating current row with the next one. Instead it is calculating the differences to the given stop time.

(1222min ~20h which is _time to _stop)
When I export my query via csv I can see that every 1000th entry the result ist _time to _stop.
Steps to reproduce:
Whole query:
`
import "contrib/tomhollingworth/events"
from(bucket: "statistik_daten")
|> range(start: v.timeRangeStart, stop: v.timeRangeStop)
|> filter(fn: (r) => r["_measurement"] == "opc_ua_mes_2")
|> filter(fn: (r) => r["maschinen_name"] == "${Maschinen_Name}")
|> events.duration(unit: 1ms, stopColumn: "_stop" )
`
Read more than 1000 entries
Expected behavior:
Except the last row, events.Duration() should always calculates the difference between current record with the next one
Actual behavior:
After 1000 rows events.Duration() calculates difference between current record and _stop value
Environment info:
Running on a Windows OS

InfluxDB version: InfluxDB 2.0.7 (git: 2a45f0c037) build_date: 2021-06-04T19:17:40Z

This should be fixed by https://github.com/influxdata/flux/pull/4064 with flux v0.139.0.  This version of flux will go out with our next minor release, influxdb 2.1, which should be coming out in a few days. Would you be able to upgrade to 2.1 when it's available? If so it would be nice to see if the bug you are seeing is indeed fixed with the latest version of flux.

Hey @wbaker85. Updated Influxdb to 2.1 this morning. Problem solved!


Publish to github.terranodo.io/SolrHeatmap

it's currently running on http://terranodo.io/SolrHeatmap/


Trying install brew on BigSur Mac

brew config output
shell-init: error retrieving current directory: getcwd: cannot access parent directories: No such file or directory
shell-init: error retrieving current directory: getcwd: cannot access parent directories: No such file or directory
job-working-directory: error retrieving current directory: getcwd: cannot access parent directories: No such file or directory
pwd: error retrieving current directory: getcwd: cannot access parent directories: No such file or directory

brew doctor output
shell-init: error retrieving current directory: getcwd: cannot access parent directories: No such file or directory
shell-init: error retrieving current directory: getcwd: cannot access parent directories: No such file or directory
job-working-directory: error retrieving current directory: getcwd: cannot access parent directories: No such file or directory
pwd: error retrieving current directory: getcwd: cannot access parent directories: No such file or directory

Verification

[X] I ran brew update and am still able to reproduce my issue.
[X] I have resolved all warnings from brew doctor and that did not fix my problem.

What were you trying to do (and why)?
I try to install a brew using script from github but its not working
What happened (include all command output)?
==> Downloading https://ghcr.io/v2/homebrew/portable-ruby/portable-ruby/blobs/sha256:0cb1cc7af109437fe0e020c9f3b7b95c3c709b140bde9f991ad2c1433496dd42
###################################################################################################################################################################################################################################################### 100.0%
Error: Checksum mismatch.
Expected: 0cb1cc7af109437fe0e020c9f3b7b95c3c709b140bde9f991ad2c1433496dd42
Actual: f1d81ffe5b7d3c228ff1c99f37183e8ed98dfc1cc2ec70236271736b09e6dc68
Archive: /Users//Library/Caches/Homebrew/portable-ruby-2.6.8.yosemite.bottle.tar.gz
What did you expect to happen?
Install Brew
Step-by-step reproduction instructions (by running brew commands)
I just run

/bin/bash -c "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/HEAD/install.sh)"

Provide a password and got error

You deleted your current directory.

I i try to reinstall because of
==> Downloading https://ghcr.io/v2/homebrew/portable-ruby/portable-ruby/blobs/sha256:0cb1cc7af109437fe0e020c9f3b7b95c3c709b140bde9f991ad2c1433496dd42
###################################################################################################################################################################################################################################################### 100.0%
Error: Checksum mismatch.
Expected: 0cb1cc7af109437fe0e020c9f3b7b95c3c709b140bde9f991ad2c1433496dd42
Actual: f1d81ffe5b7d3c228ff1c99f37183e8ed98dfc1cc2ec70236271736b09e6dc68
Archive: /Users//Library/Caches/Homebrew/portable-ruby-2.6.8.yosemite.bottle.tar.gz
Lots users have problem with this can you please provide solution because is annoing
Before i try a lot stackoverflow answers and nothing helps
There is no option to install because of this bug

To be clear, @MikeMcQuaid is referring to the errors in your brew config and brew doctor output:

shell-init: error retrieving current directory: getcwd: cannot access parent directories: No such file or directory

For your checksum mismatch issue, your download is corrupted or incomplete. Remove the file (/Users/<your_login>/Library/Caches/Homebrew/portable-ruby-2.6.8.yosemite.bottle.tar.gz) and try the install again.

Lots users have problem with this

Nope, there aren't "lots users" with this problem. It's just you.


The RTC application freezes after a few hours

The RTC application app_rtc_test.cpp freezes after a few hours.

The time display no longer changes.
Buttons no longer work.

Thanks, I'll test

I tried 8h but nothing happened, is there more detail to trigger this bug?

@Forairaaaaa I will try again because it was 2 month ago.

@Forairaaaaa
I tested again with the following application and it freeze again after few hours:
https://m5stack.oss-cn-shenzhen.aliyuncs.com/resource/docs/products/core/M5Dial/M5dial Factory Demo.exe
Do you know if this file has been updated with the latest version of this repository?


.purs file compiled, but not watched

When running webpack-dev-server or webpack --watch, recompilation is triggered only for .purs files which are included from javascript sources. E.g. if I have in javascript the following:
import main from './src/Main.purs';

Then any change to Main.purs will trigger recompilation. But if a .purs file is not imported from JS, it's not being watched (however it's included in compilation, when compilation is triggered).

Thanks for reporting. Is it possible to see your webpack config?
Also, what version of the purs-loader and purescript-webpack-plugin are you using?

Sure, here's my webpack config:
var path = require('path');

// https://github.com/ethul/purescript-webpack-example
var PurescriptWebpackPlugin = require('purescript-webpack-plugin');

var purescriptWebpackPlugin = new PurescriptWebpackPlugin({
  src: ['bower_components/purescript-*/src/**/*.purs', 'src/**/*.purs'],
  psc: 'psa',
  pscArgs: {
    sourceMaps: true
  }
});

module.exports = {
  entry: "./app.js",
  output: {
    filename: "bundle.js"
  },
  module: {
    loaders: [
      {
        test: /\.js$/,
        exclude: /node_modules|bower_components/,
        loader: 'babel-loader',
        query: {
          cacheDirectory: true,
          presets: ['es2015']
        }
      },
      {
        test: /\.purs$/,
        loader: 'purs-loader'
      }
    ]
  },
  // http://stackoverflow.com/questions/30870830/how-do-i-generate-sourcemaps-when-using-babel-and-webpack
  devtool: 'source-map',
  resolve: {
    modulesDirectories: ['node_modules', 'bower_components'],
    extensions: [ '', '.js', '.purs']
  },
  plugins: [ purescriptWebpackPlugin ]
};

I have the latest versions of both I believe, from package.json:
    "purescript-webpack-plugin": "^0.2.0",
    "purs-loader": "^0.6.0",

Thanks for the configuration file. The setup looks okay to me.
If you were to clone the purescript-webpack-example, do you encounter the same problem?

I'm not sure why or how, but it just works today ... sorry for wasting your time :)

Glad it is working!
On Monday, 11 April 2016, Adam Warski<EMAIL_ADDRESS>wrote:

I'm not sure why or how, but it just works today ... sorry for wasting
your time :)
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/ethul/purescript-webpack-example/issues/1#issuecomment-208267736

Ok, I wasn't going crazy ;) So I think the problem is when webpack -w is started when a compilation error exists. Here are the steps to reproduce:

edit a .purs file, I tried with src/Example/Bar.purs, src/Example/Foo/Boz.purs and src/Example/Test.purs, change String to Int or anything else that will cause compilation to fail
run webpack -w
you should see the compilation error
now fix the error & save the file. Compilation isn't triggered.

Thank you for following up on this and the detailed rundown to reproduce the issue. I agree this is a problem. I think it is related to:
ethul/purescript-webpack-plugin#24

Could be, however I can see the error - not sure if it hangs later, or just doesn't detect changes, though :)

Okay. Noted. Thanks!
On Thursday, 28 April 2016, Adam Warski<EMAIL_ADDRESS>wrote:

Could be, however I can see the error - not sure if it hangs later, or
just doesn't detect changes, though :)
—
You are receiving this because you commented.
Reply to this email directly or view it on GitHub
https://github.com/ethul/purescript-webpack-example/issues/1#issuecomment-215451852

Now that we just use the purs-loader, is this still an issue?

Closing. Please reopen at on the purs-loader repository if this issue crops up again.

Sorry for the late response, just noticed this :) I updated my package.json, now it only contains purs-loader. My webpack.config.js also looks similar to what's here. But still, if there's an error in a file on startup, no changes are detected. Everything works fine if the initial compilation passes without errors.

Thanks for the update. I've opened this at ethul/purs-loader#73

I've published a beta release that should fix  this issue. If you are willing to give it a try, that would be great! You can install the release under the beta dist tag.
npm install purs-loader@beta

In this release, there is a new option called watch. Please set this to true in your webpack config if you are running webpack in watch mode.
Thanks!


Docs | howto-defining-triggers.md

@Kaiqb Please see issue 1224 for the updates to the 'add trigger' menu, which affect the images and the wording on this page.

Hi -
Thanks for your feedback on this topic.
We have updated the content please see here.

Closing since the content on this topic has been updated.


go.mod: remove replace, and update github.com/gogo/googleapis v1.4.1

relates to the discussion on https://github.com/containerd/containerd/pull/5208#discussion_r594993119 and related to https://github.com/containerd/containerd/pull/4974
gogo/googleapis has a single dependency (github.com/gogo/protobuf), which is
currently the same version as we use, and we have a separate replace for that
dependency, so removing this override should not make a difference there.
gogo/googleapis v1.4.1 Includes updated protos with the gogo/protobuf fixes for
CVE-2021-3121
Change logs and diffs for each release:

v1.3.3
generate grpc service for google/longrunning/operations.proto
full diff: https://github.com/gogo/googleapis/compare/v1.3.2..v1.3.3
v1.4.0
Add google geo type
full diff: https://github.com/gogo/googleapis/compare/v1.3.3..v1.4.0
v1.4.1
This release contains sources generated with the version of gogo/protobuf
containing a fix for the "skippy peanut butter" issue.
full diff: https://github.com/gogo/googleapis/compare/v1.4.0..v1.4.1

@estesp @dims ptal if this looks good to you

Given all the work being done in protobufs in main right now, is there any need to revisit this older PR @kzys? Still valid? If so, it needs rebase.

Rebased, but looks like github is having issues again, so CI is not triggered ⚠

Given all the work being done in protobufs in main right now, is there any need to revisit this older PR @kzys? Still valid? If so, it needs rebase.

I should be able remove gogo/protobuf from containerd's main branch before 1.7.0. However 1.6.x and other branches would not have the migration changes. So we need to have this change and backport that to other branches.

It's been a while, but for this one I was mostly looking if we could get rid of the replace rules (as they wouldn't be picked up by consumers of the containerd code). I'm never sure which changes are "safe" to pick for all the grpc parts, so eyes are always welcome on this (and if it's possible to do this in 1.6, I think that'd be great of course to reduce the number of replace rules)

@kzys still looks good after all this time :)
LGTM


Include path fix for newest version of Skia

Latest version of skia used by aseprite has a change to folder structure which is causing SKIA_SKCMS_INCLUDE_DIR to be NOTFOUND which is breaking the aseprite build.

Hi @dpacbach, actually there are several changes in newest Skia versions, we're working on a refactor of the library in the refactors branch (which will start using Skia m76).