text:
[Setup] error when doing inheritance

If you have something like:
class A {
    [Setup]
    public virtual void Setup() {
        DoStuff();
    }
}

class B : A {
    [Setup]
    public override void Setup() {
        base.Setup();
        DoSomeMoreStuff();
    }
}

This triggers an error saying that you have more than one Setup. This is strictly untrue, as there is a single one (Setup). Removing the [Setup] marker means that no method is called afterwards (not even the inherited one). If you do set it with [Setup], however, everything seems to run fine anyways. This only happens when using ExecutionValidator.FailOnError.

@adamsitnik, could you take a look?

@jbayardo Thanks for detailed bug report! Fixed ;)

authors: AndreyAkinshin adamsitnik jbayardo

repo: PerfDotNet/BenchmarkDotNet


text:
F# requiring assembly binding redirects for FSharp.Core

I'm trying to benchmark some F# code. However, to work properly, the F# application requires an assembly binding redirect (App.config):
<?xml version="1.0" encoding="utf-8" ?>
<configuration>
  <runtime>
    <assemblyBinding xmlns="urn:schemas-microsoft-com:asm.v1">
      <dependentAssembly>
        <assemblyIdentity name="FSharp.Core" publicKeyToken="b03f5f7f11d50a3a" culture="neutral" />
        <bindingRedirect oldVersion="0.0.0.0-999.999.999.999" newVersion="4.4.0.0" />
      </dependentAssembly>
    </assemblyBinding>
  </runtime>
  <startup>
    <supportedRuntime version="v4.0" sku=".NETFramework,Version=v4.5.2" />
  </startup>
</configuration>

The supportedRuntime parts are not desirable of course, but if the assembly binding isn't done, the auto generated application will fail at runtime. The app.config generated won't include anything like this.
I'm not sure how to work around it, or suggest it could be fixed. Taking the app.config from the host app if one exists seems vaguely possible, but a bit random and risky, there's no reason why it should really be correct. Being able to specify app.config elements might be possible I guess...

If it helps to see this in-situ, the project i'm trying to benchmark is here: https://github.com/kolektiv/chiron/tree/benchmarks - https://github.com/kolektiv/chiron/tree/benchmarks/tests/Chiron.Benchmarks being the relevant project.
If you want to try and run that, you'll need to restore the packages first. The build.cmd in the root should sort all of that out and get a build done to start with.

Last bit of info, to confirm the thinking - if I build a version of BenchmarkDotNet which includes that assembly binding in the templated app.configs, the benchmark will run fine. I'm not sure how easy it will be to come up with an elegant fix though, I'll think on it.

Assembly binding redirects can be auto-generated. I used <AutoGenerateBindingRedirects>true</AutoGenerateBindingRedirects>  to get it working.
Some explanation

This still doesn't seem to actually work for me. Looking at the app.config generated there are no binding redirects, and without them the program simply fails. Auto generate binding redirects doesn't appear to do anything in this case!

The actual example where I'm seeing this: https://github.com/kolektiv/freya-benchmarks
Running build.cmd should pull down all appropriate dependencies, etc.

It is quite long story. For 0.9.5 I added fix for that. The problem was that you needed MSBuild 12/14 to get it working, which our .NET 4.5/4.6 taget was referencing. If you added depndency to our .NET 4.0 lib, then it did not work because we used MSbuild.v4 there.
Then we moved from dnx to dotnet cli for 0.9.7 and we were not able to target different versions of the same assembly, so we have removed our 4.5 and 4.6 targets. So it stopped working again for 0.9.7.
Now we will do this on our own, independent from installed MSBuild version. However if you don't want to wait for 0.9.8 which will have the fix you can downgrade to 0.9.5/0.9.6 for now.

source: gharchive/issue

authors: adamsitnik kolektiv

license: mit

license_type: permissive

license_source: bigquery


text:
Update data_selfie/upload.html to make clear upload automatic

This clarifies the behavior of the data_selfie system, and file dropped or selected is uploaded automatically, even if you don't hit "continue".
ref issue #570

:+1: LGTM!
@beaugunderson, merge at will

source: gharchive/pull-request

authors: BenjaminHCCarr madprime

repo: PersonalGenomesOrg/open-humans

license: MIT

license_source: github-api


text:
Always use API? (also in cron)

Wouldn't it be better to always use the API?
Not sure if it's possible, but right now there 2 ways to get all subdomains, two ways to add certificates etc. It would probably be more 'reliable' when it's just the API.
Problems:

Currently API is using the user session, no password/keys
Cron doesn't have user session/context

Possible solutions:

Use admin login key (restrict usage to local IP and SSL+actual commands), use that to login as client (admin|user), but not sure if that's possible with login keys.
Create and store login keys per user, use that to login as the actual user.
(Could be created via the API, without user interaction)

Downsides of login keys

More work maintaining/syncing them? But already is a config system for certs/keys etc.
Security? Should be okay becase you can restrict per IP and commands, right?

Personally I don't have any experience with login keys. I don't think it's very user friendly when we create a login key, because then it will be also visible in the login keys list. Maybe a admin login key could work, but then we need to switch between users, as you mentioned.

A login key needs to be created manually as far as I know. It is a bit more work to setup but it is more reliable and more safe.
I don't think we should create login keys per user, but only for an admin because otherwise we would need to create a login key for each user manually.
I am not very sure if it isn't possible to create login keys with the DirectAdmin API but I couldn't find this function on the DirectAdmin site.

Hmm, login keys could also be disabled..

They could indeed be disabled (but enabled by default). It does seem possible with the API:
http://www.directadmin.com/features.php?id=1298

The api version: CMD_API_LOGIN_KEYS is fully implemented (see the html forms for details)

Hmm, @Petertjuh360 just installed a VPS where login keys are disabled. I also don't think it would be good to require this.
It would solve a lot of problems (where we need to manipulate files/things manually) though.

I'm currently building this feature. I had something like this in mind, where the details are written to a write-only file. After that, the details can be read from the cron (root) to execute API requests.
When the cron isn't able to find/read the details, we could send a notification through the Message System. :)

Is there a valid reason not to enable login keys on your server?
Can't we just make it a requirement to use this plugin. Seems safer/easier than storing the admin credentials.

Hmm, that's true. It was an example how I had it in mind. Is there a difference between a login key and the credentials? When someone has the key, he's still able to do anything, isn't he?

Not in my understanding. I would think that workflow would be like this:

Use the admin account from session in the API to create a loginkey for letsencrypt (CMD_API_LOGIN_KEYS)
Only ask for localhost access, unlimited requests, no expiry and just the commands that we need (CMD_API_SSL and ..?).
Store the keys somewhere and use that in the cron.

If someone reads that key, all they can do is call the SSL API from localhost. While full admin credentials would be able to do anything

Oeh. Is wasn't known yet with Login Keys. I thought they were just a "API key" without restrictions. This seems really useful and it would make sense to require that, indeed. I'll take a look into that :).

I'd like to move this conversation to #75 since there is a open PR :)


text:
Update dependency @ngx-translate/core to v15

This PR contains the following updates:

Package
Type
Update
Change

@ngx-translate/core
dependencies
major
14.0.0 -> 15.0.0

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

⚠ Artifact update problem
Renovate failed to update an artifact related to this branch. You probably do not want to merge this PR as-is.
♻ Renovate will retry this branch, including artifacts, only when one of the following happens:

any of the package files in this branch needs updating, or
the branch becomes conflicted, or
you click the rebase/retry checkbox if found above, or
you rename this PR's title to start with "rebase!" to trigger it manually

The artifact failure details are included below:
File name: package-lock.json
npm ERR! code ERESOLVE
npm ERR! ERESOLVE unable to resolve dependency tree
npm ERR!
npm ERR! While resolving: jhipster-sample-application@0.0.1-SNAPSHOT
npm ERR! Found: @angular/common@14.2.0
npm ERR! node_modules/@angular/common
npm ERR!   @angular/common@"14.2.0" from the root project
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer @angular/common@">=16.0.0" from @ngx-translate/core@15.0.0
npm ERR! node_modules/@ngx-translate/core
npm ERR!   @ngx-translate/core@"15.0.0" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR!
npm ERR! For a full report see:
npm ERR! C:\Users\Mykola\AppData\Local\Temp\renovate\cache\others\npm\_logs\2023-12-27T16_06_29_562Z-eresolve-report.txt

npm ERR! A complete log of this run can be found in: C:\Users\Mykola\AppData\Local\Temp\renovate\cache\others\npm\_logs\2023-12-27T16_06_29_562Z-debug-0.log

repo: PetrivMykola/jhipster-sample-app


text:
Arguments are empty

Every time I try injecting the skin and teaks I want for steam, I get this error.
I have the latest version of SFP and Old Glory as well.
https://gyazo.com/9a6804f0d3c10e3023467b7dcf7c13f8
Steam is running in - cef -enable-debugging

What happens if you click restart steam?

SFP just stops responding and closest

SFP just stops responding and closest

Post the SFP.log after this happens

SFP.log

This log doesn't show that it's crashed or that you clicked Restart steam at all, just that you closed it normally.

https://gyazo.com/f11bde84c572d1c0540fd6cda74c330f

SFP.log

Do you have Steam running as admin or something? If you run SFP as admin does the same issue occur?

yes I run it on admin and my settings keep resetting as well.

same error keeps happening

You shouldn't be running Steam as admin tbh, that's causing issues here. Running Steam as admin should never be necessary, and at best it opens you up to malware attacks.
The settings resetting is a different problem, are you talking about the SFP settings? Where do you have SFP extracted to?
SFP relies on the SFP.config file that gets stored in the same folder as SFP_UI.exe, if it can't write to this for some reason, such as if it doesn't have permission to write to the folder you installed it to, it will fail.

I shouldn't be running Steam as admin? alright i'll do it without admin

I have SFP extracted to it's own folder on the desktop

ok so this is weird, running it as admin make SFP worked but I'm worried how to get it to work without admin and to keep the settings.

If Steam isn't running as admin, SFP doesn't need to run as admin.

ok ill check one sec

yea the settings didn't save after restarting steam

and i ran it without admin

did that and am getting the same errors again

https://gyazo.com/8d34ef54f7d2f7f04ee03601ae3d439c

is Steam running as admin again?

nope

and the extractor program was not run on admin as well

I think you've messed something up with your system lol. Why were you running Steam as admin in the first place, and what else have you been doing with permissions?

Well Originally I wasn't running steam as admin and it didn't work, than I did try it to see if SFP would work and it did, but you told me it wasn't safe so now I'm doing it without admin and now I'm getting the same errors

i didn't have these problems till recently so not sure what wrong is my system that I messed up

i rarely run much stuff with permissions, only did it here to see the errors would go away

keep getting the same 2 errors from the screenshot

Well the reason it's not working is that SFP doesn't have permission to access the Steam process. On normal systems, it should have access without needing admin, but in your case cuz of something that was done on your machine, Steam seems to be running as admin.
I don't really know what to suggest, if it was me I'd just nuke my whole Windows install and start clean lol. Cuz if SFP doesn't even have permission to write to a subfolder on your desktop for the config file something's really messed up. But if running SFP as admin works you can just do that I guess

arguments are empty and could not fetch browser

well it was working before the new update so I'm not sure why it's not working now. Makes no sense I didn't change anything

also even if it does work the settings don't save when I restart steam

Arguments are empty = SFP does not have permission to access the Steam process' arguments
Could not fetch browser = Steam is not running with -cef-enable-debugging, or another service is running on port 8080
Settings not saving = SFP does not have permission to write to SFP.config in the same folder as SFP_UI.exe
The crash you were getting is caused by this error:

System.ComponentModel.Win32Exception (5): Access is denied.
at System.Diagnostics.ProcessManager.OpenProcess(Int32, Int32, Boolean)
at System.Diagnostics.Process.GetProcessHandle(Int32, Boolean )
at System.Diagnostics.Process.GetOrOpenProcessHandle()
at System.Diagnostics.Process.EnsureWatchingForExit()
at System.Diagnostics.Process.set_EnableRaisingEvents(Boolean)
at SFP.Models.Steam.RestartSteam(String ) in D:\a\SFP\SFP\SFP\Models\Steam.cs:line 243
Which is again caused by SFP not having permission to enable raising events on the Steam process

so what should I do?

Just run SFP as admin

idk if it's my fire wall doing something to it

Firewalls are for network traffic, the problems you're running into are caused by local permission problems

ok i am trying running SFP as admin, didn't you say that was dangerous?

No, I said running Steam as admin is dangerous, and it both is dangerous and appears to be running as admin

So running SFP as admin is fine and safe  then?

I mean it's safe, but normally should not be needed and is not recommended

settings didn't save

does this not work with old steam UI?

It does but it's made for the new UI and I won't be offering support for the old UI.
I'm just gonna close this issue cuz it's not really an issue with SFP but with your system, nothing I can do in SFP to fix it.
I did add a catch for the crash at least so that part will be fixed in the next release, but it won't make it work or anything.

so it does work?

can you please explain how it still does work

i would like to use the custom skin with the old UI

authors: BigtornG PhantomGamers

repo: PhantomGamers/SFP


text:
Postgresql's JsonB data type support request

I see there is already hstore support.
But jsonb is better than json and hstore.
Any chance of supporting it?

yep. Implementation is was easy: cb4990e3836609781079db6dcfa8e76148c684a1 ;)
available in version 1.11.0

authors: PhilWaldmann arthurfranca

repo: PhilWaldmann/openrecord


text:
pvporcupine_mic --keywords porcupine not working on pycharm IDE

pvporcupine_mic --keywords porcupine is working on cmd but not working on pycharm IDE
my pyhton version is 3.6.0 64bit
pycharm console--
import pvporcupine
pvporcupine_mic --keywords picovoice
output-
pvporcupine_mic --keywords picovoice
^
SyntaxError: invalid syntax
unresolve referecne pvporcupine_mic
that's all error
And sir if u can please provide me a python script like demo script where i can manually put  keyword files in the script and get output in a variable (like keyword detected)  to turn on my virtual assistane by if condition
Thanking you in advance sir

Its a command line not a python object. the rest you should do yourself.

authors: SpydyMasu kenarsa

repo: Picovoice/porcupine


text:
Youtube shows strange green areas

In some devices users play the youtube videos perfectly but, after few seconds, the video player starts showing really strange mode.
Take a look to the screenshot: Screenshot
I bet the issue is in youtube side but, do you have any idea how to fix? maybe enabling/disabling hardware acceleration o something similar?
I've attached how we enable your view in our layout:
 <androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    android:background="@color/black">

    <com.pierfrancescosoffritti.androidyoutubeplayer.core.player.views.YouTubePlayerView
        android:id="@+id/youtube_player_view"
        android:layout_width="match_parent"
        android:layout_height="match_parent"
        app:layout_constraintBottom_toBottomOf="parent"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent"
        app:showFullScreenButton="false"
        app:showYouTubeButton="false"
        app:useWebUi="false" />

 </androidx.constraintlayout.widget.ConstraintLayout>

After some tests we have found the issue happen when YouTubePlayerView size is big. I mean, if we define layout_width and layout_height as 300dp and 200dp the video plays perfect, when bigger (or fullscreen mode) starts to appear as in the screenshot.

Is this issue happening on Android TV?

yes it is
On Fri, May 10, 2019 at 4:51 PM Pierfrancesco Soffritti <
notifications@github.com> wrote:

Is this issue happening on Android TV?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/PierfrancescoSoffritti/android-youtube-player/issues/366#issuecomment-491316957,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AADR5EUE7XSTQDWMQVAEMRTPUWDXLANCNFSM4HMDIORA
.

We'll follow the progress in the related issue

Is this issue happening on Android TV? Could this be related to #351?

On Fujicom and Peerless Android 8.0 based systems it does not happen. On some TV boxes, yes it does.


text:
fix(cloudevents-server): fix the issue of the EventConsumer close calls

wrong added the defer close call, it will caused the consumer reading failure.
I tested the change manually, it's passed.
Signed-off-by: wuhuizuo wuhuizuo@126.com

/review default

repo: PingCAP-QE/ee-apps


text:
LukeChanges2

for unsupported formats

Looks good. Your change can double-check the unsupported formats.
I saw conflicts for the client.py file, and I will fix it first.

The conflict is fixed. Remember to git pull the latest code to your branch before you start to edit the coding.

authors: LTMURPH PingaLCS


text:
Using LINQ methods in order to extract configuration values in SimpleEffectDialog

I think this is less performant than the previous one, but it could help in refactoring

Yeah I think this is fine - this isn't very performance-critical code and the list of attributes is never long

authors: Lehonti cameronwhite

repo: PintaProject/Pinta


text:
Recent images may have removed something that is required

libopenjp2.so.7: cannot open shared object file: No such file or directory

Fix for now: sudo apt-get install libopenjp2-7

OTOH even installation instructions recommends install packages first: https://www.piwheels.org/project/pillow/

repo: Pioreactor/PiOLED-display-plugin


text:
Prevent numeric overflow from too-large palettes

If palette is too large, loops with for _ in 0..((2 << size) - num_colors) { overflow and cause image-gif to try writing 55 exabytes of data.

Thanks!

authors: bvssvni pornel

repo: PistonDevelopers/image-gif


text:
Improve the developer quickstart documentation

As note in https://academysoftwarefdn.slack.com/archives/CMQ9J4BQC/p1636760978147000, the developer quickstart documentation is out of date right now.
This PR removes obsolete instructions, adds some more instructions and also use modern methodologies (e.g. call python -m pip install of pip, don't call python setup.py, etc).
Hopefully it should be clear enough and the additions I made will help future contributors.

@ThomasWilshaw Do you think the proposed changed would have made your experience better?

@ThomasWilshaw Do you think the proposed changed would have made your experience better?

Yes that's definitely clearer but there's still a few things I don't follow.

In the section "To build OTIO for Python development", where is the given command meant to be run from? The source directory or somewhere else? Also is a note of what the --user option does necessary or will most people know what it's for already?

In the section: "To build OTIO for both C++ and Python development" it implies that you should run setup.py to create all the build files and compile OTIO, is that what we have just done in the previous step when we ran python -m pip install . --user? It also says using setup.py is highly discouraged, despite just saying to run it.

Am I right in thinking that if I were to edit the code base in a way that affected both the Python and C++ sections, the best way for me to compile everything and run the tests would be to run python -m pip install. --user whereas if I were just modifying the C++ code I should use the standard cmake --build... command?
I should just clarify I'm not a very experienced dev (and I particularly don't like actually compiling software ;)) so I could just be misunderstanding things or generally no know what I'm doing :)

@ThomasWilshaw I just pushed an update which hopefully should answer your questions. Let me know if it's still unclear.

One thing we haven't gotten to is a comprehensive test suite at the C++ level. We are relying on the python bindings to expose problems. If you are developing in the C++ code, you should do a pip install and run the python tests before submitting a PR. It's up to you whether you also do a pip install while you are iterating the C++ code.
New C++ PRs are expected to have a C++ based test to go along with them. We will need to add a comprehensive test suite at the C++ level.

@ThomasWilshaw I just pushed an update which hopefully should answer your questions. Let me know if it's still unclear.

That's defintely a lot clearer thanks.

repo: PixarAnimationStudios/OpenTimelineIO


text:
[Docs]: Adding a socials section and badges to that section in the readme

Description
We should have a socials section in the readme. Example -

This would help you improve your project react and your social media profile as well.
Screenshots
added above
Additional information
Please assign me this issue under GSSoC'23
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSoC'23 contributor
[X] I want to work on this issue

Please assign me this issue under GSSoC'23

repo: PiyushKalyanpy/GyanaGuru


text:
Folder help

Navigate to /Nintendo 3DS/(32 Character ID)/(32 Character ID)/extdata/00000000/ on your SD card
EUR Region: Delete 00000098
JPN Region: Delete 00000082
USA Region: Delete 0000008f
KOR Region: Delete 000000A9
TWN Region: Delete 000000B1
I am on this step of downgrading, my system is on 9.2 now. I took out the sd card and put it in my computer to delete USA region :000008f. Put it back in my 3ds and it reconfigured home menu for some reason so I reput it in my computer to check if it put the folder back and it did. Is that folder suppose to be permenently deleted?

Nevermind! I am almost done, restoring NAND after 2.1 firmware

That step is just do delete menuhax. So it's normal the console creates that folder again.

repo: Plailect/Guide


text:
sync from is broken

Describe the bug
plakar sync to works but plakar sync from produces a compression error,
very likely to be related to client side compression/decompression logic being broken.
To Reproduce
Steps to reproduce the behavior:
$ plakar sync 7ffb from /tmp/cloned
repository passphrase:
destination repository passphrase:
/Users/gilles/.plakar: could not load snapshot from repository: lz4: bad magic number
$

This was fixed during the repo refacto

repo: PlakarLabs/plakar

license: ISC


text:
Leaves Data Issue

We are getting broken Content from Leaves

@StefanVinica Can you comment on this issue.  Is this still something we are working on? Can you assign to Andrej or another team member if so? Thank you!

repo: PlanetCassandra/website


text:
feat: timetable api welsh fields

Describe your changes
Ticket: https://pins-ds.atlassian.net/browse/APPLICS-189
PR adds welsh title and description fields to prisma schema, consumer functions and API.
Type of change 🧩

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation update
[ ] Other (please explain in the description section above)

Checklist before requesting a review

[x] I have performed a self-review of my own code
[x] I have double checked this work does not include any hardcoded secrets or passwords
[x] I have made corresponding changes to the documentation
[x] I have provided details on how I have tested my code
[x] I have referenced the ticket number above
[x] I have provided a description of the ticket
[x] I have included unit tests to cover any testable code changes

:tada: This PR is included in version 2.26.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 2.32.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

:tada: This PR is included in version 1.1.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

authors: AnthonyJWalsh Yakitrak


text:
rosette diameter

Diameter of rosette at maturity of main axis.
Reported by: dosza
Original Ticket: obo/plant-trait-ontology-to-requests/45

added the term using TermGenie

authors: marieALaporte planteome-user

repo: Planteome/plant-trait-ontology


text:
Remove an extra # column multiplications from interpolate_coset through a trick with batch inversion

Currently the interpolate_coset routine is a key performance bottleneck (The "compute opened values with Lagrange interpolation" step of https://gist.github.com/dlubarov/0eb8490f659a5aa9970cf95cfe1b0fb5 )
Spent alot of time optimizing this exact operation in libiop before haha.
I tried counting the time the # of CPU operations cost by hand. If the matrix is of dimensions N = R*C, currently we are doing 7C + N multiplications and C + N additions. R should be quite small (between 1-4 I believe), so the C term matters to optimize.
I believe there is C multiplications that can be saved for free (no tradeoff involved), and C multiplications that can be saved with a memory tradeoff, which I suspect should be time savings, but needs benchmarking.
The C multiplications thats a memory tradeoff is basically caching the column subgroup evaluations g^i as we compute it twice. (And may even recompute elsewhere in the total computation of the program).
I'll go into more detail of the C multiplications that can be saved for free. At the end of this program, we need to multiply every single element in the output column by a constant. Lets call this shift "final_constant". This is what is happening here with C multiplications in scale_vec: https://github.com/Plonky3/Plonky3/blob/049c413fcd0ad7288d12789e4a727d3965340ca1/interpolation/src/lib.rs#L60-L62
Notice that our computation is roughly:
# C multiplications, C subtractions
compute_diffs()
# 3C multiplications
diff_inv = inverse(diffs)
# 2C multiplications
col_scale = [g^i * diff_inv[i] for i in cols]
# N multiplications, N additions
sum = columnwise_prod(matrix, col_scale)
# C multiplications
return scale_vec(sum, final_constant)

First notice that due to commutativity, we could instead compute this as:
diff_inv = inverse(diffs)
scaled_diff_inf = scale_vec(diff_inv, final_constant)
col_scale = [g^i * scaled_diff_inf[i] for i in cols]
sum = columnwise_prod(matrix, col_scale)
return sum

Now recall that the way taking an inverse works, is that we first take the product of every field element. Then we invert that. And then we "peel" steps to get every element's inverse. If what we really need is given diffs compute [final_constant / diff[i] for i in cols] we can compute this with one additional multiplication! (Rather than C multiplications). Just multiply the product of every element with final_constant, after you take the inverse during batch inversion.
Call this inverse_and_shift. So we can now compute this as:
# C multiplication, C subtraction
compute_diffs()
# 3C + 1 multiplication
scaled_diff_inf = inverse_and_shift(diffs, final_constant)
# 2C multiplication
col_scale = [g^i * scaled_diff_inf[i] for i in cols]
# N multiplication, N addition
sum = columnwise_prod(matrix, col_scale)
return sum

Thus saving us C multiplications at no downside
Its an optimization I implemented succesfully in libiop before: https://github.com/scipr-lab/libiop/blob/master/libiop/protocols/ldt/fri/fri_aux.tcc#L230-L231 !
If folks agree with this suggestion, happy to make a PR!
If R=2 which I assume it does for the first round, then I think this would be a savings of ~10% of multiplications. (~20% if we can save the subgroup being created twice)
I'd guess maybe 1ms overall, but would have to benchmark and see.

Thanks for looking into it, that's a neat idea!
If I'm understanding right, R here would be our trace length which can be pretty large, so the O(R C) work in columnwise_dot_product tends to be the bulk of interpolation cost. So I'm not sure if the improvement would be noticeable for typical parameters.
That said it seems like a nice simple change which would probably have other uses (we have quite a few batch inversion calls especially for circle STARKs), so it could be good to have!

authors: ValarDragon dlubarov


text:
New parameter for set a mode for auto-bump and update maven submodules

Changes proposed in this merge request:

Additions: New parameter (auto-version-bump-mode) to set a mode different then auto for auto-bump, so on auto bump we can select which mode we want(major, minor, patch) without add it into the PR Comment.
Updates: For the maven bump, added configuration to update subModules as well. For the maven bump, change to git add all the pom.xml files from the project.
Deletions:

Pre-merge Checklist

[X] Write + run tests
[X] Update documentation
[X] Update CHANGELOG and increment version

I did this PR, because I got into a case where we wanted to auto run the patch version on each PR that were merged into our master branch, without any comment on PR message, since that on our case each possible build/merge is one possible version to go to prod. Also I noticed that in case of multi-module project it's not updating the version for all the sub-modules, so I also fixed it.
Let me know if it's relevant for you guys, at this moment I'll use the version from my fork to handle my case.

Sorry for the late response! Thank you for the pr. I will merge it, may others also want it ;)

repo: Plugily-Projects/version-bump-action


text:
Implement selectable choices for combat special abilities on the roll window

Probably some sort of buttonlist to select skills like "finte" and their step including and then calculating the modifiers.
Any suggestions on how to proceed are welcome.

Room for improvement in the future but ok now


text:
Makefile

We currently have ./bin/format and ./bin/ghcid (which doesn't work). Is there any reason for Plutarch not having a Makefile?

You want to use flake apps for this.

I don't see how a makefile would fit here though?

This is some example of makefile we have: https://github.com/Liqwid-Labs/liqwid-plutarch-extra/blob/main/Makefile
It's basically what ./bin is but in one file, organized.

I agree with Las. Maybe we should use an app attribute here; no additional tooling needed.

authors: L-as MangoIV SeungheonOh

repo: Plutonomicon/plutarch-plutus


text:
feat: unpublish expired states

This change causes retained MQTT topics to be cleared when the corresponding state expires (i.e. the state object is null).

I've tested it and it works perfectly

authors: AlCalzone tnowak


text:
fix: not all files displayed

If you go to your dashboard, you can see that your files are not all displayed.
We have no idea where it comes from...

@lolboysg and @lucas-louis have used the new aleph client, resolving this issue !

repo: PoCInnovation/InterPlanetaryCloud


text:
general inquiry: payment

Wondering why you guys chose to use podcast:funding instead of rel="payment"
http://microformats.org/wiki/rel-payment

For clarity of purpose. This is ultimately meant to be an all inclusive namespace for all podcast specific functionality. We discussed redundancy with other tags and determined it was good to have a namespace that included all within itself so that at some point in the future apps and publishers could have confidence and simplicity.

repo: Podcastindex-org/podcast-namespace


text:
небраскОвцы

небраскАвцы Гуглу неизвестны.

См. #6.

repo: Pogrebnoj-Alexandroff/demonyms


text:
Dockerizeit

Hi,
I have created a Dockerfile for the tools.

I've also set-up an automatic build on dockerhub, is you want to take a peak: https://hub.docker.com/r/mattions/maftools/

Closing this one, 'cause superseded by #6

repo: PoisonAlien/maftools


text:
Update bot.po

Added missing fr translations

Thank you for your contribution it'll go in tonights update!

authors: Tortuegreen dandesousa

repo: PokeNavBot/pokenav-translations

license: Unlicense


text:
Ehanced Pokemon list view

New:

Pokemon Individual Attack/Defense/Stamina
Pokemon Candies count (Both on Pokemon and Pokedex tabs).

Preview:

Great job!

authors: kdemoya sniok

repo: PokemonGoF/PokemonGo-Bot-Desktop