Investigate forming mixed Pekko and Akka cluster

Use Pekko snapshots
Akka 2.6.20
Find a simple Akka cluster scenario or sample
akka-persistence / pekko-persistence interoperability needs testing too
See if you can form a cluster with an Akka node and a Pekko node and see if they can interact with one another
There is a fear that the serialized messages may contain Akka or Pekko markers that will see them rejected by nodes of the other type.

Related to https://github.com/apache/incubator-pekko/issues/108

@pjfanning Shall we remove this from the milestone project? There was discussion at https://lists.apache.org/thread/nr7q9orzffolwo9tj753cvkrohr9875o and we agreed that at least of now, no one really has this requirement.

When it comes to 1.0.0 release I definitely agree but I guess there is room for a discussion/consensus to include it for a later 1.0.x release. The main concern from my end is that such a migration is non trivial and hence it would really delay the first Pekko release by quite a margin (there is already enough tension on this aspect). Ontop of that we don't even have tests written for such a feature and our existing cluster tests had to be disabled because we don't have dedicated hardware yet.

@mdedetrich if you see Johannes' answers on https://github.com/apache/incubator-pekko/issues/139, you'll see that we do have cluster tests running but not the multi node variant - so we do have reasonable coverage but it could be better.

@mdedetrich if you see Johannes' answers on #139, you'll see that we do have cluster tests running but not the multi node variant - so we do have reasonable coverage but it could be better.

Multi node was what I was referring to. At least to me if we are going to release such a feature it makes sense to be really sure that it works (people are after all going to be using it to do production migrations).

I honestly doubt whether this is going to be done. Most users will happily have some down time to switch from Akka to Pekko - after testing out the Pekko cluster first. Other users will be able to run an independent Akka cluster and Pekko cluster simultaneously and control which work happens on each cluster, gradually switching more work to the Pekko cluster before decommissioning the Akka cluster.
The cost-benefit on this really comes out as too much cost and too little benefit.

The cost-benefit on this really comes out as too much cost and too little benefit.

Well yes this is the crux, if its really critical for enough people then personally I would hesitate to kill off the idea but someone/some people definitely needs to own it in creating a concrete proposal/implementation as well as testing.

After attending Scala Matters, I realized the case of needing to do a rolling upgrade is more common than what the current Pekko maintainers assume.
I know that @He-Pin wanted to either work on this himself or find another person but I for now I will assign this to myself since if we are going to do this (highly likely) we need to make sure that its out before Pekko 1.1.0 is out since there is going to be bridging from 1.0.x to 1.1.x


Memory of previously-opened projects

Something weird happened today while I was running @run_package_tests in the VSCode REPL (I know that's not the intended behavior).
Sometimes (not always), TestItemRunner.jl tried to run the testitems from another project that also uses them. That other project was not currently open or even in my workspace.
If it helps, I think it only happened when I hit the "up" arrow key to re-execute @run_package_tests. When I executed it with Ctrl + Enter the normal behavior ensued.

I know this was a while ago, but for what it's worth I was having the same problem today where @run_package_tests would try to run a @testitem from a separate project, even in a new REPL. As a workaround I have this code in my tests/runtests.jl file that I just include from the REPL:
# tests/runtests.jl

using DrWatson, TestItemRunner
@quickactivate "MyProject"

using MyProject

println("Starting tests")
ti = time()

@run_package_tests filter = ti -> (occursin("MyProject", ti.filename)) verbose = true

ti = time() - ti
println("\nTest took total time of:")
println(round(ti / 60; digits = 3), " minutes")

So I don't think we can fix this: when you send a macro like @run_package_test to the VS Code REPL it also sends the file in which that was written to the REPL and stores that, and when you recall it from history then that stored file is used.
I'll close this for now, as the "official' response here is that this macro is not a good way to run tests from the REPL. TestItemRunner2.jl will eventually offer a better experience from the REPL.


Can't show "Issues" page (Error 500)

Gitea version (or commit ref): 1.0.0+154-g8555e888
Git version: 2.11.0
Operating system: CentOS 7
Database (use [x]):

[x] PostgreSQL
[ ] MySQL
[ ] SQLite

Can you reproduce the bug at https://try.gitea.io:

[ ] Yes (provide example URL)
[x] No --> Cause try.gitea.io is not up to date
[ ] Not relevant

Log gist:

Description
Just go to /issues or /ORG_NAME/REPO_NAME/issues, /pulls, ...
2017/01/25 07:19:28 [...routers/user/home.go:286 Issues()] [E] Issues: Find: no such column: true
2017/01/25 07:19:28 [...routers/user/home.go:286 Issues()] [E] Issues: Find: no such column: true
2017/01/25 07:19:30 [...routers/user/home.go:286 Issues()] [E] Issues: Find: no such column: true
2017/01/25 07:19:33 [...routers/user/home.go:286 Issues()] [E] Issues: Find: no such column: false
2017/01/25 07:29:12 [...outers/repo/issue.go:228 Issues()] [E] Issues: Find: no such column: false
2017/01/25 07:31:16 [...routers/user/home.go:286 Issues()] [E] Issues: Find: no such column: false
2017/01/25 07:31:18 [...routers/user/home.go:286 Issues()] [E] Issues: Find: no such column: false

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

It maybe caused by #530

Ok @lunny, I rebuild Gitea and look if issue is fixed

Nope, I always get error 500

Yes, I'm trying to fix this.


Re-write of getting starting adding a catalog item sections

Re-write of getting starting adding a catalog item sections.
Preview link: https://deploy-preview-1261--roadie.netlify.app/docs/getting-started/adding-a-catalog-item/
Before:

After:

Can we get an updated loom @Irma12

Overall this is going in a good direction. A bit more info (and more high level info) on each page will really make this very useful.

@dtuite I believe I have responded to most/all of the comments. Please let me know how it looks.

@dtuite I believe I have responded to most/all of the comments. Please let me know how it looks.

I've asked @samnixon87 to take over this review on my behalf. Thanks

Can we get an updated loom @Irma12


Use analytic solution to quartic equation when solving for contact points

A suggestion from @ericagol. See the appendix of Kipping (2008).

This paper may also be useful:
https://arxiv.org/abs/1811.05484

A suggestion from @ericagol https://github.com/ericagol. See the
appendix of Kipping (2008) https://arxiv.org/abs/0807.0096.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dfm/exoplanet/issues/17, or mute the thread
https://github.com/notifications/unsubscribe-auth/AAO30EDe5_1wmDb1d5e1RtaOy9dl6QAmks5u4kXEgaJpZM4ZRhTZ.

Absolutely - Thanks!

Another quartic solver: https://www.sciencedirect.com/science/article/pii/S0377042710002128

I've now incorporated a special purpose quartic solver using companion matrices and it seems pretty robust and efficient. I'm going to table this for now.


Information about blank body param necessary for Mandrill templates in Readme

Add information about the need to set body = '' in #mail when using templates stored in Mandrill.
re: https://github.com/spovich/mandrill_dm/issues/46

thanks for the documentation update.  What happens if you don't set the body to an empty string?  Also, can check travis?  Not sure why it's failing (or use [skip ci] since this is only updating documentation)

There is  no [skip ci] in a commit message, but I agree with @spovich, as far as it's documentation update it's worth to use CI skipping. Particularly in this case CI argues to Rubocop style checks.

@spovich If you don't set body to an empty string, then it'll try to render template, which is missing (because we want to render template stored in Mandrill, not in the app)
Travis fails because of an unrelated issue in
SimpleCov.start do
    ...
  end
end if ENV['COVERAGE']

it expects (and I do the same in my projects):
if ENV['COVERAGE'] == 'on' # or 1 / true
  SimpleCov.start do
    ...

Personally, I don't think that adding [skip ci] tag is a good idea. It is not related to the application at all + it hides problems instead of resolving them.


Reverse individual words of a string using stacks

🚀 Feature
The idea is to reverse the individual words of the input string with the help of stacks.
Sample input:  Bring me food
Sample output: gnirB em doof
Have you read the Contributing Guidelines on Pull Requests?
Yes
Motivation
I want to contribute to it under GSSoC '21
Pitch
It is a good application for stack data structure. I would like to take up this task to implement it in C and/or Python.

hello Im a participant at GSSoC'21
i want to work the issue in c.

Thank you for assigning this to me @bhav09 .
This particular code goes into ds folder of the respective languages, am I right? (Just making sure)

Yes, it does. @taruni-always
@SrinivasRedyy221 you can check if its done already for C or not. If not then you can open the issue and I will assign it to you.

@bhav09 Okay, thanks. ALso, I'm doing this in C and Python.

We would be able to merge only for what the issue was raised initially. We encourage you to do it with C++ as of now.
Happy Learning!

The issue was raised for C and python initially. I have mentioned it in the issue clearly.

Oh, I am sorry, you can proceed with C and Python.

It's okay. Small doubt, should I two separate PRs?

One would work.

Okayy. Thank you.

Please add the level labels for this.


UIButton lack of kf_cancelDownloadTask extension ^_^

UIButton lack of kf_cancelDownloadTask extension， how to use it. thx

It should be not difficult to add one, by using the same strategy of UIImageView. If you'd like, you could send a p-r on it. Otherwise I will try to add it later :)

Implemented in #221


Tvu change message

Adding first message.

This seems to be triggered from a workshop and seems to not address an issue.


maybe a bug?

The first 17 points are the jaw of the dlib 68 landmarks.
the code in lsfm/data/init.py
LANDMARK_MASK[:18] = False
may be changed to
LANDMARK_MASK[:17] = False
I'm not sure.

@jabooth That is indeed a bug


Fixture definition: Atomic LedBar EC 24X3 3in1

Fixture definition for Atomic LedBar EC 24X3 3in1, the only online reference of this model that i've found is the seller website
https://www.musicbros.net/atomic-ledbar-ec-24x3-3in1.html
Unfortunately i can't find the product in the manufacturer's website.
These are some pictures taken by the paper version of the user manual, i can't find a PDF version online
manual_pictures.pdf

Hi, normally the rule is: if you can't find the official website of a product, it means it's most likely a chinese rebranding, so it won't be accepted in the QLC+ library.
I think this is the case, so perhaps it can eventually be loaded here
https://github.com/mcallegari/qlcplus-extras/tree/master/Fixtures
Closing this then


T8012 support

I'm running into an issue with the image on a T2 device.
I can run the exploit, upload the files and boot into recovery but it stops there.
It looks like the ramdisk image is not valid.
irecovery -s

loaded ramdisk at <ptr> of size 0x7eb0200, from image at <ptr>
loaded device tree at <ptr> of size 0x100000, from image at <ptr>

ramdisk
ea0f64a4253252:448
Ramdisk image not valid

devicetree
ea0f64a4253252:448
Device Tree too large

Tried with multiple bridgeos version.

Figured out that issue was caused by the version, patching a really old one makes it not throw those errors, likely newer ones use APFS and script logic does not work in this case.
Even when all images are correct bootx command does not actually boot the system, it is stuck, I guess you already know that since you put a warning.
[!] WARNING: T2 MIGHT HANG AND DO NOTHING WHEN BOOTING THE RAMDISK!

I've seen videos of T2 macs booting a ramdisk, it even showed the bootlog on the touch bar (not testing on a touchbar one), @verygenericname any idea what we are doing wrong?

SSHRD tool does not have the correct files for T2, i have T2 jailbreak like checkra1n one that works on bridgeOS from 6.0 to 7.6+, also I have ssh ramdisk , lmk what you need these things for? Telegram @SDunlocks_91


nimble-date-picker Component

😯 Problem to Solve
To conveniently pick a date through a date-picker input. This comes from one of HLD requirement's in Argon here

User should be able to see a small calendar icon to the right of date input
User should be able to select a date from the expanded calendar view. He/she should be able to scroll through days, months and years
User should be able to type in the date in a standard format. For ex: DD/MM/YYYY

Adobe XD View - Password: Soliton@123 – Reach out to<EMAIL_ADDRESS>/ John Nakano<EMAIL_ADDRESS>for getting Adobe XD Design files
💁 Proposed Solution
Design

#842

Create a nimble date picker component. Inspired from Material's Date Picker
📋 Tasks

[ ] nimble-date-picker interaction design issue
[ ] nimble-date-picker interaction design spec
[ ] nimble-date-picker visual design issue
[ ] nimble-date-picker visual design spec
[ ] nimble-date-picker spec
[ ] nimble-date-picker custom element
[ ] nimble-date-picker unit tests
[ ] nimble-date-picker Storybook tests
[ ] nimble-date-picker Storybook docs
[ ] nimble-date-picker Angular wrapper
[ ] nimble-date-picker Blazor wrapper
[ ] Add to Angular example app
[ ] Add to Blazor example app
[ ] Update Component Status table in README and incubating status

Florina from the team that owns the SLE Executions and Routines apps reached out with another use case for the date picker. For Q4 featre F1881164 (Time-triggered routines frontend) they need to be able to pick a date and time that a job will run.

That screenshot comes from Leslie's wireframe so hopefully she can provide more detailed requirements for this use case.

Update on the UI in the comment above: as a workaround while they wait for the Nimble date/time picker they ended up using a single component (from Angular Material) that allows selecting both date and time. They'd likely prefer separate components in order to match the wire frame above. More conversation in this HLD PR.

Here's an interesting thread going deep on the pros and cons of native date pickers and calendar controls for picking dates. The author ended up deciding to use three separate inputs to enter a date on their site (select for month, numeric text for day and year). Looks like the UK.gov design system linked in the comments made a similar choice. I'm not saying this conclusion applies to us, just reminding us that there are ways to enter a date that don't involve a calendar and we should consider what types of dates we'll be requesting from our users.

just reminding us that there are ways to enter a date that don't involve a calendar and we should consider what types of dates we'll be requesting from our users.

100%. I try to create a separation between absolute dates (e.g. birth date, expiration date, etc.) and relative dates (e.g. whatever date next Thursday is, a Sunday in mid-March, etc.). "Absolute" and "Relative" may not be the best terms here but I don't have a better set right now.
For the first, the separate pickers/inputs is a good solution. For the latter, a calendar view is very helpful, if not required.
I imagine both are relevant for our clients depending on their context/app and we'll have to see what component (or set of components) will support it well.

This came up again in a request from an NI partner developing SystemLink plugins using Nimble. Their current workaround is to use a Nimble text field with textField.elementRef.nativeElement.control.type = 'date'; which actually gets the job done but doesn't have styling for the calendar.

I would also like this component for my use case.
Currently I am also using the hack mentioned above, however it still gives its output in type "string" which the user has to manually extract to type "Date".

Removing the 'blocked' label because we don't plan to wait on FAST to implement this. No change in its priority for now, but we are not blocked from proceeding.

Another possible options is Cally, which is a pretty lightweight option, but allows fairly easy styling and slotting for certain things like using Nimble buttons for the 'Next' and 'Previous' buttons to move between months. It could be missing features that we ultimately want, like a dropdown to move between years, etc..., but it's worth considering.
Here is an early prototype branch (link to working Angular example).

Some folks from the SLE Asset Management team mentioned they need a date picker for a workflow for scheduling calibration. They are having problems with the material-based wrappers in the Stratus shared library because their form integration isn't great; for example if you enter an invallid date the value change event fires before the validity change event, so change handlers have to handle the possibility that the value is invalid.
Just a reminder to consider form integration as part of our requirements and to add this to our list of reasons to prioritize this feature.


MSG+ - Not presented w/ Login

Hi - thanks for considering adding NHL.tv from my other issue. In the meantime, I was hoping to get MSG+ to work with this to add to Emby.
I converted the docker run command to compose, and am running the container with the following compose file:
version: "3.3"
services:
  eplustv:
    ports:
      - 8430:8000
    volumes:
      - ./data:/app/config
    environment:
      - MSGPLUS
      -<EMAIL_ADDRESS>      - MSGPLUS_PASS=YG*b&R7NZ*Dy9NbBJGw4cUVXWA #<-- Not my actual password, but it is similar as my password is set of random letters/numbers/symbols. Not sure if this is messing anything up
      - PUID=1000
      - PGID=1000
    image: m0ngr31/eplustv

When I run the container, I'm not being prompted to login to msg+, instead, I'm just getting a prompt to login to ESPN+:
eplustv-eplustv-1  | ><EMAIL_ADDRESS>start
eplustv-eplustv-1  | > ts-node index.ts
eplustv-eplustv-1  |
eplustv-eplustv-1  | === E+TV v2.1.10 starting ===
eplustv-eplustv-1  | === ESPN+ Auth ===
eplustv-eplustv-1  | Please open a browser window and go to: https://www.espn.com/watch/activate
eplustv-eplustv-1  | Enter code:  WNMXNV

Can you let me know what I'm doing wrong?
::EDIT::
Just tried using the following run command:
docker run -p 8430:8000 -v ./data:/app/config -e PUID=1000 -e PGID=1000 -e MSGPLUS -e<EMAIL_ADDRESS>-e MSGPLUS_PASS=YG*b&R7NZ*Dy9NbBJGw4cUVXWA m0ngr31/eplustv

Having the same issue:
><EMAIL_ADDRESS>start
> ts-node index.ts

=== E+TV v2.1.10 starting ===
=== ESPN+ Auth ===
Please open a browser window and go to: https://www.espn.com/watch/activate
Enter code:  MXMTKM

If you're just wanting MSG+, you'll need to pass in ESPNPLUS=false as an environment variable

Are you importing the xmltv route into Emby? It won't play anything if there aren't any event scheduled, so you'll have to make sure something is on first

Yep - here is how I have emby setup:

Last night the Devils, Islanders, and Rangers were all playing, and there was no guide data, and no streams on any of the channels.

Oh, looking at your docker compose, it should be:
version: "3.3"
services:
  eplustv:
    ports:
      - 8430:8000
    volumes:
      - ./data:/app/config
    environment:
      - MSGPLUS=true
      -<EMAIL_ADDRESS>      - MSGPLUS_PASS=YG*b&R7NZ*Dy9NbBJGw4cUVXWA #<-- Not my actual password, but it is similar as my password is set of random letters/numbers/symbols. Not sure if this is messing anything up
      - PUID=1000
      - PGID=1000
    image: m0ngr31/eplustv

I'm closing this since it's stale so long


spawning dcd-client fails

Uncaught Error: spawn dcd-client ENOENT (events.js:141)

looks like a permission issue. starting atom from the command line with sudo solves the problem.

is dcd-client in path? Try typing dcd-client and if it works then the plugin should also work

yeah it's in the path, but in /usr/local/bin, which OS X has decided is too dangerous to touch in El Capitan. Also, the completions for imports aren't working for some reason

This will be fixed in #28.
Currenty VisualStudio Code with code-d works best for D development.

cool

Closing in favor of #42


parameterization of filters and features benchmarks (v2)

Update of #278 for branch-22.08 with minor additional fixes. I have tested this locally and it seems to be operating as expected.

Looks like there are some conflicts. Could you please take a look at these @grlee77? 🙂

@gpucibot merge

Looks like there are some conflicts. Could you please take a look at these @grlee77? slightly_smiling_face

Resolved. should be good to go now. No point to wait on CI as it doesn't run these benchmarks.

Yeah it won't merge until CI passes. So we just need to wait unfortunately. On the bright side it should merge as soon as CI passes


fix: git connected app encryption failing due to padding when the upd…

Description
Fix partial import issue in git connected app. When the update call was triggered due to the EncryptionHandler not being triggered there was Hex coded string issue for SSH keys. Replaced the update call parameter with the actual updates and not whole application object.
PR fixes following issue(s)
Fixes https://github.com/appsmithorg/appsmith/issues/29000
Type of change

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

Testing
How Has This Been Tested?

[ ] Manual

Issues raised during DP testing

Link issues raised during DP testing for better visiblity and tracking (copy link from comments dropped on this PR)

Checklist:
Dev activity

[ ] My code follows the style guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[ ] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes
[ ] PR is being merged under a feature flag

QA activity:

[ ] Speedbreak features have been covered
[ ] Test plan covers all impacted features and areas of interest
[ ] Test plan has been peer reviewed by project stakeholders and other QA members
[ ] Manually tested functionality on DP
[ ] We had an implementation alignment call with stakeholders post QA Round 2
[ ] Cypress test cases have been added and approved by SDET/manual QA
[ ] Added Test Plan Approved label after Cypress tests were reviewed
[ ] Added Test Plan Approved label after JUnit tests were reviewed

/ok-to-test

/build-deploy-preview


Features not working on dashboard

I can see Android and iOS app on the dashboard but I notice the following:
Crashes are not working - I crash the app for both Android and iOS but no crushes recorded
HTTP errors and network failure rate is not working in Android but is working in iOS
App Launches is working for both Android and iOS
HTTP response time is not working for Android but its working for iOS
Most frequent interactions is not working for Android but its working for iOS.
Handle exceptions is not working for both Android and iOS

@taku-ka Android instrumentation gets injected by a gradle task and there is no equivalent of that inXamarin. You will need to add the instrumentation manually in Android
recordHandledException should be available in the Xamarin binding for iOS.
crashes can be recorded if the dSYM file is uploaded during build (nspostbuild script) and only one crash handler (newrelic) is setup.

@taku-ka Android instrumentation gets injected by a gradle task and there is no equivalent of that inXamarin. You will need to add the instrumentation manually in Android
recordHandledException should be available in the Xamarin binding for iOS.
crashes can be recorded if the dSYM file is uploaded during build (nspostbuild script) and only one crash handler (newrelic) is setup.

Any idea of where the Gradle adds the hooks for these crash handlers etc? I'd be willing to try hook those up somehow.
I was able to hack around the crashes not working by just doing this:
 this.StartNewRelic(); //normal init stuff             AndroidEnvironment.UnhandledExceptionRaiser += (sender, args) =>             {                 Throwable throwable = Throwable.FromException(args.Exception);                 NewRelic.RecordHandledException(new Exception(args.Exception.Message, throwable));             }
But it feels like there should be easier and better hooks for that? Potentially the configuration that defines the stores and add to those? Or am I too far off.

I think you are on the right track. I have not looked at if xamarin has ways of adding gradle tasks recently, so if there is a hook mechanisms, please link the relevant examples/strategies on how to do it, Thanks

@vvydier not gradle as such no. but we do have things like Fody Weaver which can inject code at build time or generate/inject code by modifying the csproj files etc. that said you can launch external tasks from there as well. it just depends on what those tasks are. im not sure where the source of the gradle files for mobile are so that i can see if they can translate to xamarin or if they can be added as post/pre compile tasks. Note i'm not great at gradle but willing to give it a shot.
alternatively if we can expose the sendcrash methods or something for noticing errors and http messages etc then we can just work with xamarin hooks on those as well.

@emanuelbalea I have not used Fody Weaver, can you point me to an example how you plan on adding auto-instrumentation in xamarin

Adding/creating a weaver is not overly difficult, depending on if you just need to inject some C# files to get compiled, or replace files at compile time but if you inject IL it can get a bit messier. Another way to do this is to use roslyn generators but i think they are slightly more complicated.
The problem is I don't know what hooks you had in gradle or what they did to point you in their equivalent places but in general if there is an android/ios api then there will be a xamarin counter part. So if you can point me to where the gradle does stuff i could help more.
In general for xamarin regardless of the platform you usually do your init and hook up with calls in oncreate in the application or similar unless you inject using something fody in the same place.
https://github.com/Fody/Home/blob/master/pages/addin-development.md
However i made android bindings for a version of the jar file from about 1-2 major release ago (current release had lots of changed namespaces and didn't have the time to add them to the transform or ignore list) and that seems to record exceptions just fine, and even some ui transitions.

@emanuelbalea @taku-ka  please check our latest offerings for Xamarin. https://docs.newrelic.com/docs/mobile-monitoring/new-relic-mobile-xamarin/monitor-your-xamarin-application/


Update foorter build version based on git

Codecov Report

Merging #1065 (5bf933e) into master (71ed10e) will decrease coverage by 0.77%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #1065      +/-   ##
==========================================
- Coverage   42.32%   41.54%   -0.78%
==========================================
  Files          55       55
  Lines         697      698       +1
  Branches       43       43
==========================================
- Hits          295      290       -5
- Misses        398      404       +6
  Partials        4        4

Impacted Files
Coverage Δ

src/app/components/footer/footer.component.ts
100.00% <100.00%> (ø)

.../components/set-password/set-password.component.ts
37.93% <0.00%> (-10.35%)
:arrow_down:

src/app/components/teams/teams.component.ts
43.75% <0.00%> (-6.25%)
:arrow_down:

...s/team-member-modal/team-member-modal.component.ts
36.11% <0.00%> (-5.56%)
:arrow_down:

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


Epic: code refactoring and small tweaks

Hello, @teoadal! Proposing refactoring and some tweaks:

Added StyleCop to the project
Aligned all code with StyleCop
In cases of iteration Span<T>, changes cycles to ref foreach
In HttpHelper, method AppendEncodedName added returning array to pool (pool.Return(byteBuffer);)
Added in Readme.md  benchmarking results on ARM machine.

Tests:

@teoadal ping

@teoadal ping

@teoadal ping

@teoadal ping

@teoadal ping

Looks good! But there are a lot of changes. Ok, I going to merge it. Thank you again.


Send PING frame to keep connection

Currently PING frames is not send. They should be sent to keep the connection up.

https://github.com/neko-suki/quic_client/pull/6


carryon compatibilty

carryon disable pickup ship helm

Should be as simple as doing the following in EurekaModForge.
But did not work for me, looks like enqueueIMC do get invoked but carryon does not catch the messages even tho they say it should work like this here

Import net.minecraftforge.fml.InterModComms and net.minecraftforge.fml.event.lifecycle.InterModEnqueueEvent.
In EurekaModForge() add MOD_BUS.addListener(this::enqueueIMC);
Add the following method:

private void enqueueIMC(final InterModEnqueueEvent event)
{
    // Carry on mod support
    var exclusions = new String[] {
            "vs_eureka:oak_ship_helm",
            "vs_eureka:spruce_ship_helm",
            "vs_eureka:birch_ship_helm",
            "vs_eureka:jungle_ship_helm",
            "vs_eureka:acacia_ship_helm",
            "vs_eureka:dark_oak_ship_helm",
            "vs_eureka:crimson_ship_helm",
            "vs_eureka:warped_ship_helm"
    };

    for (String item:exclusions) {
        InterModComms.sendTo("carryon", "blacklistBlock", () -> item);
    }
}


Unsupport pipectl datastore command

What this PR does / why we need it:
Which issue(s) this PR fixes:
Fixes #
Does this PR introduce a user-facing change?:

Disable pipectl datastore command

What's up, Kapetanos

/kapetanios reload

This one is for pagination stuff right. Good job 👍
/lgtm

Actually, this /kapetanios reload command does not help in this case. That is only for reloading the newly merged Kapetanios configuration changes. Those errors were caused due to Kapetanios itself, not Kapetanios' configuration.

This one is for pagination stuff right.

Yep, it's preparation for the datastore interface changes (allow pagination). I'm going to remove MongoDB completely from our code base again 😂

/approve

Actually, this /kapetanios reload command does not help in this case.

It actually is. We gotta look into why.


Feature Request: support window vibrancy and blur

This is a feature in other window manager ecosystems
https://www.reddit.com/r/unixporn/comments/rqjrls/i3gaps_local_man_install_arch_tells_no_one/
https://www.reddit.com/r/unixporn/comments/nkxagu/bspwm_i_quite_like_blur/
It looks like this can be implemented in macos fairly easily?
https://developer.apple.com/documentation/uikit/uivibrancyeffect?language=objc
https://developer.apple.com/documentation/appkit/nsvisualeffectview?language=objc
I would love to contribute this feature but not sure how feasible and I couldn't find much useful documentation. It seems like the Core Graphics API in use does not match what I found in Apple's docs.
https://github.com/koekeishiya/yabai/blob/846723a2629daf995882359f26967889654a0b4d/src/osax/payload.m#L654
versus
https://developer.apple.com/documentation/coregraphics/1456404-cgcontextsetalpha?language=objc
What are your thoughts @koekeishiya

This would be for background blur:
extern CGError SLSSetWindowBackgroundBlurRadius(int cid, uint32_t wid, uint32_t radius);

and the other things needed for vibrancy effects I think are listed in the bottom of this file: https://github.com/avaidyam/DIYWindows/blob/master/CGSInternal/CGSWindow.h

where the hell is the Core Graphics Skylight framework documented?

https://developer.apple.com/documentation/appkit/nsvisualeffectview?language=objc

I looked at blur a long time ago and did not like the result: https://github.com/koekeishiya/yabai/issues/493
Maybe I'll revisit in the future.

I don't think #1115 is the way to go about this. Windows with modified alpha will deactivate the background blur. I have quickly hacked together something with limelight:

The way my quick and dirty hack works is by creating a new transparent window behind the actual window where I set the SLSBackgroundBlurRadius, such that when the window receives an alpha it will be in front of a "backdrop". Theoretically this can also be implemented in yabai directly, but the window ordering groups are broken on monterey.

Window ordering groups work fine on Monterey, it is the movement group that causes the issues as far as I remember, so I do believe your version for enabling window blurs could work.

@FelixKratz @koekeishiya any way I can help ship this to yabai or will this only be possible with limelight?

Any update on this?

@FelixKratz @koekeishiya any way I can help ship this to yabai or will this only be possible with limelight?

It is a fairly simple change that can be incorporated the exact same way current window borders are implemented.
Create a new window, „glue“ this window to the host, always keep it ordered behind the host and give the window a background blur. Then all inactive windows, which receive an alpha with via the scripting addition appear to have a blurred background.
This is exactly what I have done in my above screenshots.

It can be done even easier than described above:
The borders have no reason to be in front of the window. Rather, I have chosen to glue them below the window and apply a negative inset to them. This way rounded borders can also be implemented and the distracting border flickering is a
non problem, additionally content from the window is not truncated.
When doing it this way it is as simple as giving the border a fill color and a background blur.
This is my implementation of it: https://github.com/FelixKratz/yabai/commit/840344bb9f0e1fd1620deba3681f7834c68f61c8 including rounded borders as you can see here:

I see that a previous pull request #1115 was made for window blurring, but it was closed. But has this new solution been made into a pull request yet?
Would love to see @FelixKratz 's solution come to fruition! :)

I think I have read somewhere that pull requests are generally not welcome on this project. Just create a fork and patch these changes in, thats why I have linked my fork for most of the changes I have made (e.g. the mach ipc for the scripting addition).
This being said, I noticed some people are actually using my fork. Please don't, it is full of things I have changed only for my personal preference and for experimenting. It is only linked so you can cherry-pick changes you like to your own fork.

@weeebdev https://github.com/FelixKratz/SketchyBar

@weeebdev https://github.com/FelixKratz/SketchyBar

Thanks. Do you consider adding a rounded corners on the border?

This will be fixed with the changes being made for #1430

how do I install my own fork of yabai?

If it can be added to an arbitrary window using the scripting-addition, yes. If it is going to work the same as the current version that relies on the borders, then I'm not particularly interested in making adjustments at this time.

But it has to be handled with caution, because these surfaces will continue to exist even after yabai exits.

Maybe it is time to work on having fully persistent state, which would also solve this new issue.

@FelixKratz
I assume this works in the following way?

Create a new CAContext
Create a new CALayer and set up filters
Set the CALayer as the layer for the CAContext
Create and setup a surface for the arbitrary window.
Bind the surface to the CAContext we created using its id?

@FelixKratz
I assume this works more or less in the following way?

Create a new CAContext
Create a new CALayer and set up filters
Set the CALayer as the layer for the CAContext
Create and setup a surface for the arbitrary window.
Bind the surface to the CAContext we created using its id?

Yes, this is what I was testing with:
  [CATransaction begin];
  CAContext* context = [CAContext contextWithCGSConnection:_connection options:nil];
  CABackdropLayer* layer = [[CABackdropLayer alloc] init];
  layer.frame = bounds;
  layer.windowServerAware = true;
  layer.cornerRadius = 10;
  layer.borderWidth = 2;
  layer.borderColor = CGColorCreateGenericRGB(1, 1, 1, 1);
  CAFilter* blur = [[CAFilter alloc] init];
  [blur initWithType:kCAFilterGaussianBlur];

  NSNumber *normalize = [NSNumber numberWithBool:true];
  [blur setValue:normalize forKey:@"inputNormalizeEdges"];
  NSNumber *radius = [NSNumber numberWithFloat:100.0f];
  [blur setValue:radius forKey:@"inputRadius"];

  CAFilter* saturate = [[CAFilter alloc] init];
  [saturate initWithType:kCAFilterColorSaturate];

  NSNumber *saturation = [NSNumber numberWithFloat:1.2f];
  [saturate setValue:saturation forKey:@"inputAmount"];

  NSArray* filters = @[saturate, blur];
  [layer setFilters:filters];

  context.layer = layer;

  uint32_t sid = 0;

  SLSAddSurface(_connection, wid, &sid);
  SLSSetSurfaceBounds(_connection, wid, sid, bounds);
  SLSSetSurfaceResolution(_connection, wid, sid, 2.0);
  SLSSetSurfaceColorSpace(_connection, wid, sid, context.colorSpace);

  SLSBindSurface(_connection, wid, sid, 4, 0, context.contextId);
  SLSOrderSurface(_connection, wid, sid, 1, 0);

  [CATransaction commit];

Can we at least get rid of the grey background appearing while animations are on?

@FelixKratz after https://github.com/koekeishiya/yabai/issues/1889 what has been your solution for this? I'd completely forgotten to update for a while and missed that announcement much to my surprsie. Does your Janky Borders project facilitate blurred windows?


Atomic animation

Project title
Atom animation
Project Description
Atom animation
Stack:
And in order to tick the check box just but x inside them for example - [x] like this. Please delete options that are not relevant.

[x] Html
[x] CSS

/assign

/assign

@amyra98 Hiii, Can I work on this Issue ? Then assign me please

@amyra98 Hiii, Can I work on this Issue ? Then assign me please
My project is ready 😄

Oh okay. Good Luck


[Web]onSuggestionTap not triggered on flutter stable 3.7

Hi, 2 days ago the flutter team upgrade flutter to version 3.7, I don't know the reason but the onSuggestionTap function is not being triggered when we do tap on a suggestion. I am not sure about the origin of the problem to fix it, we have an app in production using devops so it automatically was build in flutter 3.7 yesterday and now our clients can't use our app.
Thanks.

Thanks for reporting the bug @aglpy I haven't had a chance to upgrade to latest stable yet, I will triage this issue asap

That sounds great but it probably means it is a problem with flutter web, with your info, I run the example in flutter web and it doesn't work, when you tap a suggestion the textfield is not filled. I run the tests and it works fine. Then, I installed the app in my Android device and it works fine, so maybe is like your ios simulator. Anyway I write here some changes I made to make it run in Android (if not, the app dind't even start) so I don't know if this changes solve the problem for any reason that the bug is.
android/gradle/wrapper/gradle-wrapper.properties:
`- distributionUrl=https://services.gradle.org/distributions/gradle-6.7-all.zip
+ distributionUrl=https://services.gradle.org/distributions/gradle-7.6-all.zip`
android/gradle.properties:
`- org.gradle.jvmargs=-Xmx1536M
+ org.gradle.jvmargs=-Xmx1536M \
+ --add-exports=java.base/sun.nio.ch=ALL-UNNAMED \
+ --add-opens=java.base/java.lang=ALL-UNNAMED \
+ --add-opens=java.base/java.lang.reflect=ALL-UNNAMED \
+ --add-opens=java.base/java.io=ALL-UNNAMED \
+ --add-exports=jdk.unsupported/sun.misc=ALL-UNNAMED`

Exact same issue here. At least it should be declared that the package is not compatible with web for the time being.

Thanks for reporting the bug @aglpy I think that flutter did not break onTap, rather they fixed the race condition for focus and gesture events. I filed an issue though to get some more insights https://github.com/flutter/flutter/issues/119390

Closing as fixed in https://github.com/maheshmnj/searchfield/pull/71


Transfer File Format to Tabby

I wish to use the plugin framework Tabby. To prepare, we must do these reforms to the codebase:

Add Runtime and Framework folders. The Framework folder is for the Tabby libraries. The Runtime folder are the scripts that initiate on runtime/start.
For type-effectiveness and intellisense, use script.Parent.Parent... instead of script:FindFirstAncestorOfClass("Plugin"). This will be a temporary fix until RobloxLSP fixes this.

This should help with the productivity on this plugin. It's my first time doing this anyways; this is more like a diary to me.

All steps are done, I hope I have not missed anything else


FIPS Support

Describe the solution you'd like:
Just curious if FIPS support is on the road map? I see some devs are working on selinux support which is awesome!

FIPS depends on Micro 6. When Micro 6 has full FIPS, the respective Elemental-enabled builds should have it also.


xcode archive: physical platform is "Ineligible destination"

Required Information
Question, Bug, or Feature?
Type: Question/Bug
Enter Task Name: xcode archive
Environment

Server - Azure Pipelines
Agent: Hosted MacOs 10.14

Issue Description
For CI build we're trying to perform xcode archive on the project but it fails since archive requires target platform to be non-simulator.
Task logs
log_25_24753660.zip
Troubleshooting
"Enable system diagnostics" fails xcode task (which is another bug probably).
Error logs
Ineligible destinations for the "PhoneFactor" scheme:
	{ platform:iOS, id:dvtdevice-DVTiPhonePlaceholder-iphoneos:placeholder, name:Generic iOS Device }
	{ platform:iOS Simulator, id:dvtdevice-DVTiOSDeviceSimulatorPlaceholder-iphonesimulator:placeholder, name:Generic iOS Simulator Device }```

Hi, @sbolotovms ,
Based on your logs, you use incorrect destination parameter: -destination platform=iOS,name=Generic iOS Device.
The correct one should be -destination generic/platform=iOS according to the article.
Could you please try to specify the following settings to XCode task:

Closing out since we haven't heard anything back but please reopen if you are still seeing the issue


Move Encryption class check from BaseConfig to Encryption class

@paulbalandan this is what I meant at comment https://github.com/codeigniter4/CodeIgniter4/pull/3808#discussion_r511521137
Checklist:

[x] Securely signed commits

For note: this may be a bc break if user want to use config('Encryption') before first call : Services::encrypter($config, true)

After some thinking, I think it is only can happen in major change to ensure no bc break. Closing.


NumMethod from reflect package doesn't return not exported members of method sets

What version of Go are you using (go version)?
go version go1.7.1 darwin/amd64
What operating system and processor architecture are you using (go env)?
> go env
GOARCH="amd64"
GOBIN=""
GOEXE=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOOS="darwin"
GOPATH="/Users/mlowicki/projects/golang/spec"
GORACE=""
GOROOT="/usr/local/go"
GOTOOLDIR="/usr/local/go/pkg/tool/darwin_amd64"
CC="clang"
GOGCCFLAGS="-fPIC -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -gno-record-gcc-switches -fno-common"
CXX="clang++"
CGO_ENABLED="1"

Repro
https://play.golang.org/p/eXoyuqajG6
Expected result
I expect to see method n also printed out and language spec doesn't enforce that method set contains only exported identifiers.
Currently it displays only info about M:
Number of methods: 1
Method {M  func(main.T) <func(main.T) Value> %!s(int=0)}
	Name: M
	PkgPath:

I believe that doc is not consistent with the source code:
https://github.com/golang/go/blob/master/src/reflect/type.go#L841
https://github.com/golang/go/blob/master/src/reflect/type.go#L825

It's a documentation issue. The word 'exported' should appear in the explanation.


Create account management tool

Minimal functionality:

Account Creation
Account Removal
Account Search/Edit

A clean database schema + access description should allow creation of simple website with this functionality

We're almost there with @dracc and the team's work on SEGSAdmin + the web-ui over at segs-data

I guess accounts with access level 9 ought to have the ability to remove accounts, as well as users ought to be able to remove their own accounts.
But what would search/edit do? I can see password change being a valuable function. Are there more functionality expected that I just fail to see?

Should users be able to remove their own account though? What if we allow them to disable it, and provide the admin with a way to purge disabled accounts (perhaps periodically)
My thought is that users are some times impetuous and having the ability to "undo" for a brief period of time seems good.

For Search and Edit:
Search: Find a user by User ID, by db_id, by IP address (which we should be logging) or by player name. Also maybe filter by last login date?
Edit: Edit should allow us to reset user passwords, update contact email, edit character information (char name, in particular, but probably other aspects as well)
The greatest use I see for these abilities would be to revise characters with offensive names, or who have managed to cheat in some way (level 69? I don't think so!)
I would also add to this list the ability to Disable or Ban accounts.

I agree. They may decide to come back later. And while many will be ok with all their stuff gone, because they KNEW what they were doing, others may just not be as... observant.
Perhaps a drop down along with a search field to find the users, and a set of tick boxes to do some basic admin like delete or suspend etc. Even the ban feature?

This relates to #436


Result photo remains black and white

I compiled several versions with artistic and stable model but the result photo always remains black and white.
What's wrong here?

Sorry, used the wrong model (renamed the .model to .hmodel).


Fix hardcoded values in cf-deploy-stage script

The etc/concourse/scripts/cf-deploy-stage currently has a hardcoded value and works only when deploying a large instance of abacus that has 6 collector and reporting app instances. Now it will read the number of app instances from the .apprc files for those applications.

Coverage increased (+0.05%) to 91.351% when pulling 921fe9a1d0816a27c5db69e4f12ac5048c70387a on SAP:routes_fix into c1e107f21917965b390ae6a9f36f3196bdfe22f1 on cloudfoundry-incubator:master.


Having a nested array call act as new "route".

Rather then writing the following
app.get("/test", function(req, res, next){
  console.log("next route?")
  return next("route")
})

app.get("/test", function(req, res, next){
  return res.send("hi")
})

I was hoping I could get the same effect with this
app.get("/test", [
  [
    function(req, res, next){
      console.log("next route?")
      return next("route")
    }
  ],
  [
    function(req, res, next){
      return res.send("hi")
    }
  ]
])

However it's not supported. Thoughts?
I know all about the router. My problem there is sharing the params.

I'm afraid I'm looking at a really ugly solution, does express have a way of doing this?
_.each(api.redirectUpload, function(route){
  router.get("/test/:redirectCsv", route)
})

Hi! I'm not clear on what you are describing still, even with the new comment. Can you explain exactly what you are trying to do, what is happening, what is not happening, etc.?

Hey there again @dougwilson!
Here's another example.
I have functions that call next("route") like this.
function colorAction(color){
  return function(req, res, next){
    if(req.params.color !== color) return next("route")
    // do stuff for color
  }
}

router.get("/action", colorAction("red"))
router.get("/action", colorAction("green"))
router.get("/action", colorAction("blue"))

I know this example isn't great though because that next("route") can be changed to next() and then you could write this.
router.get("/action", [
  colorAction("red"),
  colorAction("green"),
  colorAction("blue"),
])

However I'm working with much more complex blocks of middleware and I can't simply do that. I'm looking for a simpler way of writing this.
router.get("/action", colorAction("red"))
router.get("/action", colorAction("green"))
router.get("/action", colorAction("blue"))

And I thought this was a viable solution.
router.get("/action", [
  [
    colorAction("red"),
  ],
  [
    colorAction("green"),
  ],
  [
    colorAction("blue"),
  }
])

In your example above, did you mean to make the paths all /action, and not /:color? It's not clear where req.params.color gets populated in your example. Because of this, that may be why I'm still scratching my head over here on what you are even trying to do to even come up with an answer for you, I'm sorry :( Can you try elaborating even more? Perhaps even posting something that is more complete I can even run?

why not just:
router.get("/action/:color", function(req, res, next){
  colorAction(req.params.color);
})

?

@dougwilson I'm thinking about other examples.
@gabeio I have blocks of middleware that are more complex then these examples and they need to be exited and other routes need to run. To be really specific my question is about specifically running middleware that call next("route") but need to be run within the same route (I know a little confusing and unorthodox).
My only other solution was this:
innerRouter.use(colorAction("red"))
innerRouter.use(colorAction("green"))
innerRouter.use(colorAction("blue"))

router.get("/action/:color", innerRouter)

and concocting a function like this
var makeRoutes = function(routes){
  var router = express.Router()
  return _.chain(routes)
  .map(function(route){
    return router.use(route)
  })
  .value()
})

router.get("/action/:color", makeRoutes([
  colorAction("red"),
  colorAction("green"),
  colorAction("blue"),
]))

But I did run into an issue where params weren't accessable within the nested router.

But I did run into an issue where params weren't accessable within the nested router.

Create the nested router with the mergeParams: true option: http://expressjs.com/4x/api.html#router

To be really specific my question is about specifically running middleware that call next("route") but need to be run within the same route (I know a little confusing and unorthodox).

so next() ?
router.route("/action").get(function(req,res,next){
  colorAction("red");
  next();
}).get(function(req,res,next){
  colorAction("green");
  next();
}).get(function(req,res,next){
  colorAction("blue")
  next(); // if needed
});

these stay within the route... next just continues to the next verb of the same method given or all

I have a question, does this simulateRoute function exist in express that I can use?
var chain = [
  middleware.checkMethod(),
  middleware.validateQuery(),
  middleware.processPage(),
  middleware.retrieveFromDb(),
  middleware.setValue(),
]

app.use("/page", chain) // same as above
app.use("/page", function(req, res){
  if(req.query.setting == "something"){
    return simulateRoute(req, res, chain)
  }
  return res.send("no setting")
})

Also Is there access to the native next function outside of that callback?

My goal is to have the ability to run a chain of middleware (promised using express-promise-router or native) without a route (as in the case of testing), or pass in req and res from an existing route. This would greatly solve all of my problems.

Hi @reggi , circling back real quick, did the mergeParams: true option I listed above help with solving your problem at all? I took it that you said you always had a solution, is only req.params transitioned routers, which mergeParams: true lets you do.

@dougwilson  What @gabeio said I took to heart, they're right I shouldn't be calling next("route") when I'm really still in the same route. My new idea fixes that. I'm working on repairing what I have.

@reggi Definitely comment back if it doesn't, or does, end up work I'm not trying to shoot down ideas, just trying to help & understand! :smile:

Here's what I built to help manage middleware flow. https://github.com/reggi/node-middleware-run

I can get what I made above working perfectly but it will still never process middleware exactly the same way the express does. @dougwilson @gabeio I've boiled what I want to exactly what is described below in the middlewareRunner function. Does this exist within express? Anywhere on the web?
var httpMocks = require('node-mocks-http')
var req  = httpMocks.createRequest({})
var res = httpMocks.createResponse()

return middlewareRunner(req, res, function(req, res){
  return res.redirect("http://google.com")
}).then(function(){
  expect(req._getRedirect()).to.equal("http://google.com")
})

The third argument of middlewareRunner could be an array of middleware as well. (just like a route)
All other arguments after res could be middleware as well. (just like a route)

I'm still not sure what you are trying to do, so I'm not really able to comment.

Before I thought you were just trying to run multiple middleware in series within the same route... Now I am kinda lost, sorry.

@dougwilson I'm sorry this idea isn't sticking, I expected / assumed that this was something that already existed in the middleware community.
@gabeio Yeah that's still the idea.
With middlewareRunner you can take req and res from a route or from httpMocks. It allows you to run middleware independent of a route as in the example above. Since all middleware does contain functions that manipulate the req and res objects you can view the changes to their values with something like httpMocks, or ideally return the value to a route.

This is the closes thing I found to running middleware within middleware. However you can't pass in req and res directly. And what it's really doing is pass in a route into middleware.
https://github.com/carlos8f/node-middler

I just don't understand at all how this is different from Router and router.use, especially since all you do is call router(req, res, done) to invoke tour router with mocks and router.use takes arrays of middleware, etc.

The only difference are:

you can't pass req and res into router.use
you can execute router.use independently from an express route.

you can't pass req and res into router.use

Ok, but I need specific information on what this is supposed to do.

you can't execute router.use independently from an express route.

Yes you can; you make a new router, then router.use the middleware you want to run, then simply invoke "router(req, res, callback)" because router is just a function.
I'm going to close this issue for now since it has been so long and we still don't have good information to even help you, but please feel free to continue the conversation.

Also, please feel free to make a PR to add this feature, if you don't think it exists. Perhaps seeing code will help us better understand what you are trying to do.

@dougwilson This last comment was along the lines that I'm looking for. Although invoking router doesn't seem to be firing the middleware. Thoughts?
var express = require("express")
var router = express.Router()
var httpMocks = require('node-mocks-http')
var req  = httpMocks.createRequest({})
var res = httpMocks.createResponse()

req.test = true

router.use(function(req, res, next){
  console.log("hello")
  req.test = false
  return next()
})

router(req, res, function(){
  console.log(req.test) // should be false is true
})

We do not guarantee mocks ever work, since the quality of them are all over the place. I assume you don't even have a valid URL on your mock? You need at least a single slash for anything to match against.

Try creating your mock request like this to see if it works:
var req  = httpMocks.createRequest({url:'/'})

If that doesn't work still, I can try playing with it later when I get to a computer, but otherwise the example you put to get should work, barring issues from that mock library (like, perhaps, missing the URL).

I believe I have something working here's a prototype.
Having router as a param allows me to use pass in https://github.com/alex-whitney/express-promise-router or express's router.
var Promise = require("bluebird")

module.exports = function middlewareRun(router, req, res, middleware){
  if(!req.url || req.url == "") req.url = "/"
  return new Promise(function(resolve, reject){
    router.use(middleware)
    return router(req, res, function(err){
      console.log(err)
      if(err) return reject(err)
      return resolve()
    })
  })
}


Open in browser setting does not work for Safari

The open in browser setting is no longer working after updating Mail Notifr. My default system browser is Chrome. I have Mail Notifr set to open mail in Safari but the emails open in Chrome. If I choose a different browser in the Mail Notifr settings (for example Edge) then the email opens correctly in Edge. It appears the issue is with opening Safari when it is not the default system browser.
OS: macOS 11.6
Mail Notifr: Version 2.1.0 (205)

@hamishg Thanks for reporting this. I confirm this is a flaw and I'll fix this in the next release.

Great, thank you very much for spending time on this.

@hamishg v2.1.1 with a fix this has been released.

Thank you. Confirmed as working for my use case now.


List Method Overview at Top of Documentation Pages

When accessing cl*** or interface documentation, I am most interested in seeing a summary of available methods first.
In the new layout, I either have to scroll past a lengthy example block, or click a link in the sidebar to jump to the method summary.
Please restore the method overview to its location at the top of documentation pages.
Old Style Example (Preferred): https://msdn.microsoft.com/en-us/library/5y536ey6(v=vs.110).aspx
New Style Example: https://docs.microsoft.com/en-us/dotnet/api/system.collections.ilist?view=netframework-4.7.1

⚠ Idea migrated from UserVoice
Created By: Michael
Created On: 2017/11/17 14:20:20 +0000
Votes at Migration: 2
Supporters at Migration: 2

There are currently no plans to change this layout - method summaries are still available after the main class summary.


Update workflows

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 100.00%. Comparing base (19f236c) to head (f240ace).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #140   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           65        65
  Lines         1410      1410
  Branches       151       151
=========================================
  Hits          1410      1410

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Node version requirement in package.json is out of date

It's 0.12, but that causes ember-cli to fail on the splat operator in index.js.
You need to require a version of Node with ES6 support.

Changed to 4.0.0. That should work right?


mate.mate-themes: 3.22.16 -> 3.22.17

Semi-automatic update generated by https://github.com/ryantm/nixpkgs-update tools.
This update was made based on information from https://repology.org/metapackage/mate-themes/versions.
These checks were done:

built on NixOS

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 3.22.17 with grep in /nix/store/vnrs1vjrq1jjqff5872k2vyqnf5bj5iw-mate-themes-3.22.17

directory tree listing: https://gist.github.com/7496d09132d21dac1574730a34586a9d

du listing: https://gist.github.com/fae90128247da010e9abf4c845ceb824

meta.description for mate.mate-themes is: '"A set of themes from MATE"'.
cc @romildo for testing.
Experimental: this build is cached with Cachix. To use the cache follow these experimental instructions:
One time setup in nixpkgs Git checkout:
cachix use r-ryantm
git remote add r-ryantm https://github.com/r-ryantm/nixpkgs.git

Test this build:
git fetch r-ryantm && git checkout 7ed6d8d189e93534f37c19deedb2f5e7617e7e8b && nix-shell --pure -I nixpkgs=. -p mate.mate-themes

No attempt on x86_64-darwin (full log)
The following builds were skipped because they don't evaluate on x86_64-darwin: mate.mate-themes
Partial log (click to expand)

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowUnsupportedSystem = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowUnsupportedSystem = true; }
to ~/.config/nixpkgs/config.nix.

Success on x86_64-linux (full log)
Attempted: mate.mate-themes
Partial log (click to expand)
done
make[3]: Leaving directory '/build/mate-themes-3.22.17/po'
make[2]: Leaving directory '/build/mate-themes-3.22.17'
make[1]: Leaving directory '/build/mate-themes-3.22.17'
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/vnrs1vjrq1jjqff5872k2vyqnf5bj5iw-mate-themes-3.22.17
strip is /nix/store/21ymadblbmsbb2bk4q7gl4kjasp8zmgd-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/vnrs1vjrq1jjqff5872k2vyqnf5bj5iw-mate-themes-3.22.17
checking for references to /build in /nix/store/vnrs1vjrq1jjqff5872k2vyqnf5bj5iw-mate-themes-3.22.17...

Success on aarch64-linux (full log)
Attempted: mate.mate-themes
Partial log (click to expand)
make[3]: Leaving directory '/build/mate-themes-3.22.17/po'
make[2]: Leaving directory '/build/mate-themes-3.22.17'
make[1]: Leaving directory '/build/mate-themes-3.22.17'
glibPreFixupPhase
post-installation fixup
shrinking RPATHs of ELF executables and libraries in /nix/store/2j6363f3l3plmmqk509kayaah25lbr60-mate-themes-3.22.17
strip is /nix/store/qg2agrqkf240s656d207zqhipl0bc2id-binutils-2.30/bin/strip
patching script interpreter paths in /nix/store/2j6363f3l3plmmqk509kayaah25lbr60-mate-themes-3.22.17
checking for references to /build in /nix/store/2j6363f3l3plmmqk509kayaah25lbr60-mate-themes-3.22.17...
/nix/store/2j6363f3l3plmmqk509kayaah25lbr60-mate-themes-3.22.17


Switch between Pock and regular touchbar at ease

Is your feature request related to a problem? Please describe.
Neither Pock nor the default touchbar controls give individually all the functionality I want.
Describe the solution you'd like
I want to have a shortcut or a simple way to "activate" or "open" Pock from the regular touchbar controls (like the control strip), and then to "close it" and go back to the normal one.
Describe alternatives you've considered
Be able to bring functionality from the regular touchbar to Pock.
Additional context
Pock v0.52, macOS Mojave 10.14.5

You can use the double ctrl key shortcut to switch from current app Touch Bar and Pock! :)

@pigigaldi Maybe the double ctrl key shortcut is not convenient for everyone. Is there possible to customize the shortcut ? Specifically, I could assign any touchPad gesture to the double Ctrl key in BetterTouchTool, because it could not record double keyboard modifiers( Normally, it should be Modifier+other keys).

Agree. Users should be able to easily change the shortcut. I am using Karabiner to modifier keys on my keyboard, so it is very useful for me to avoid conflicts if I could change double Ctrl key shortcuts.


The main bolts website has a typo - "Rigurously" should be "Rigorously"

🐛 Bug

There is typo on the main bolts website: https://www.pytorchlightning.ai/bolts
The section head says "Rigurously Tested", and it should be "Rigorously Tested"
Please reproduce using the BoringModel
https://www.pytorchlightning.ai/bolts
https://www.pytorchlightning.ai/bolts

No Colab.
To Reproduce
Go to website:
https://www.pytorchlightning.ai/bolts

Thanks!
Forwarded to website admin :)

shall be fixed now :rabbit:


Using regex for column names in SchemaModel

Using regex for column names in SchemaModel
Is it possible to write a SchemaModel class in which the column names follow a regex pattern, e.g., ^[0-9]+$  for 0, 1, 2, 3, 4, etc?
If not, can I use the DataFrameSchema class in  the same way as the SchemaModel class?

Hi @markkvdb,
You can use the regex argument of Field. You'll also have to pass the regex in alias if it is not a valid name for a class attribute (which is the case in your example):
import pandas as pd
import pandera as pa
from pandera.typing import Series

class Schema(pa.SchemaModel):
    a: Series[int] = pa.Field(alias="^[0-9]+$", regex=True, ge=0)

df = pd.DataFrame({"1": [-1]})
Schema.validate(df)
#> Traceback (most recent call last):
#> /tmp/ipykernel_259488/1760133918.py in <module>
#> ----> 1 Schema.validate(df)
...

#> SchemaError: <Schema Column(name=1, type=DataType(int64))> failed element-wise validator 0:
#> <Check greater_than_or_equal_to: greater_than_or_equal_to(0)>
#> failure cases:
#>    index  failure_case
#> 0      0            -1

If you define a DataFrameSchema instead, Column has a similar regex argument.

Thanks for your clear answer. It seems to work now! I already had it working with DataFrameSchema but all other schemas were defined using the SchemaModel, so using a single approach is a bit nicer.


[FEATURE] Documentation for Custom Providers

Hello, I came across this project and was interested in trying it out as it claims to be provider agnostic, however all of the documentation uses Google Compute Platform as a provider, and I am not interested in using GCP at all, I want to be able to use InfraQL with smaller cloud providers like DigitalOcean or Linode.
So this an issue asking for documentation on how to use InfraQL with different providers, and how I as user can add more providers.

Hi @XAMPPRocky we have moved the repo over to https://github.com/stackql/stackql now, completely different provider architecture now (which can extend to Digital Ocean, Linode etc), send me an email at<EMAIL_ADDRESS>and I can fill you in, would be great to get your input and contribution to the project! (move your ★ across as well if you could)


dispose timepicker remove all dropdowns events

after
myTimepicker.dispose()
all dropdowns not working

Thank you for reporting this bug! I've added this to our list


Trinity doesn't seem to switch to regular sync after completing fast sync

After catching up with the current head, the FastChainSyncer._load_and_process_headers task finished unexpectedly and trinity ended up in what looks like an infinite loop, starting and immediately aborting a sync.
    INFO  09-04 18:21:15       chain  Imported 167 blocks (2150 txs) in 5.89 seconds, new head: #3972908
    INFO  09-04 18:21:17       chain  Got no new headers from ETHPeer <Node(0x2e6b@<IP_ADDRESS>)>, aborting sync
    INFO  09-04 18:21:26        peer  Adding ETHPeer <Node(0xeaf6@<IP_ADDRESS>)> to pool
    INFO  09-04 18:21:26       chain  Starting sync with ETHPeer <Node(0xeaf6@<IP_ADDRESS>)>
    INFO  09-04 18:21:26       chain  Got no new headers from ETHPeer <Node(0xeaf6@<IP_ADDRESS>)>, aborting sync
    INFO  09-04 18:21:26       chain  Starting sync with ETHPeer <Node(0x2e6b@<IP_ADDRESS>)>
    INFO  09-04 18:21:27       chain  Got no new headers from ETHPeer <Node(0x2e6b@<IP_ADDRESS>)>, aborting sync
    INFO  09-04 18:21:27       chain  Starting sync with ETHPeer <Node(0x2d7d@<IP_ADDRESS>)>
    INFO  09-04 18:21:28       chain  Got no new headers from ETHPeer <Node(0x2d7d@<IP_ADDRESS>)>, aborting sync
    INFO  09-04 18:21:33       chain  Imported 167 blocks (1954 txs) in 6.75 seconds, new head: #3973075
    INFO  09-04 18:21:36       chain  Starting sync with ETHPeer <Node(0xbb67@<IP_ADDRESS>)>
    INFO  09-04 18:21:37        peer  Connected peers: 10 inbound, 3 outbound
    INFO  09-04 18:21:37        peer  Peer subscribers: 1, longest queue: FastChainSyncer(0)
    INFO  09-04 18:21:37       chain  Got no new headers from ETHPeer <Node(0xbb67@<IP_ADDRESS>)>, aborting sync
    INFO  09-04 18:21:41       chain  Starting sync with ETHPeer <Node(0xc5f4@<IP_ADDRESS>)>
    INFO  09-04 18:21:42       chain  Got no new headers from ETHPeer <Node(0xc5f4@<IP_ADDRESS>)>, aborting sync
    INFO  09-04 18:21:43       chain  Imported 167 blocks (2062 txs) in 6.88 seconds, new head: #3973242
    INFO  09-04 18:21:44       chain  Starting sync with ETHPeer <Node(0xeaf6@<IP_ADDRESS>)>
 WARNING  09-04 18:21:44     service  Task <coroutine object FastChainSyncer._load_and_process_headers at 0x7f450d54e990> finished unexpectedly: No connected peers have the block bodies we need for td=10072557116148220
    INFO  09-04 18:21:44       chain  Got no new headers from ETHPeer <Node(0xeaf6@<IP_ADDRESS>)>, aborting sync
    INFO  09-04 18:21:45       chain  Starting sync with ETHPeer <Node(0x2d7d@<IP_ADDRESS>)>
    INFO  09-04 18:21:45       chain  Got no new headers from ETHPeer <Node(0x2d7d@<IP_ADDRESS>)>, aborting sync
    INFO  09-04 18:21:47        peer  ETHPeer <Node(0x94c1@<IP_ADDRESS>)> finished, removing from pool
    INFO  09-04 18:22:04       chain  Starting sync with ETHPeer <Node(0x25e2@<IP_ADDRESS>)>
    INFO  09-04 18:22:04       chain  Got no new headers from ETHPeer <Node(0x25e2@<IP_ADDRESS>)>, aborting sync
    INFO  09-04 18:22:04       chain  Starting sync with ETHPeer <Node(0x2e6b@<IP_ADDRESS>)>
    INFO  09-04 18:22:05       chain  Got no new headers from ETHPeer <Node(0x2e6b@<IP_ADDRESS>)>, aborting sync
    INFO  09-04 18:22:05       chain  Starting sync with ETHPeer <Node(0x2d7d@<IP_ADDRESS>)>
    INFO  09-04 18:22:06       chain  Got no new headers from ETHPeer <Node(0x2d7d@<IP_ADDRESS>)>, aborting sync
    INFO  09-04 18:22:07       chain  Starting sync with ETHPeer <Node(0x2d7d@<IP_ADDRESS>)>
    INFO  09-04 18:22:08       chain  Got no new headers from ETHPeer <Node(0x2d7d@<IP_ADDRESS>)>, aborting sync

@carver When in fast-sync mode, we used to set BaseHeaderChainSyncer._sync_complete when it was time to switch to regular-sync mode, but that doesn't seem to happen anymore; that attribute still exists but is not used anywhere. Maybe we need to reintroduce the code that caused FastChainSyncer to set that event so that its _run() method would return and let FullNodeSyncer move on to the state sync?

The current approach is to simply cancel out of the fast syncer:
https://github.com/ethereum/py-evm/blob/61b9bfa527a83e59ba215d9da430778f3f9ffbf4/trinity/sync/full/chain.py#L504-L507
There is an outstanding thing I'd like to see fixed, though: when the fast sync crashes then state sync starts. Instead it should probably just quit out of trinity.
Some options to resolve that:

re-introduce _sync_complete and only continue to state sync if it is set
add a return value on service.run() which acts like an exit code (did it run and shut down cleanly)

I kind of like 2), but haven't dived into it yet.

It seems to correctly switch to regular sync now, so closing


showArea is not working with Polygons in Leaflet v1.0.0b2

I'm moving to Leaflet v1.0 and there was a issue about showing the area while drawing the polygon. I digged in the problem and released that the following parameter is undefined in Leaflet v1.0 :
L.LatLng.DEG_TO_RAD
If I added this to code before drawing polygon :
L.LatLng.DEG_TO_RAD = 0.017453292519943295;
Then _area value changes from NaN to numeric values and it will shown in the UI.
I don't know if it should be fixed here or in Leaflet, so I didn't create any pull request. Waiting for your comments...

This seems to be a pretty old bug.
Can anyone confirm that L.LatLng.DEG_TO_RAD = 0.017453292519943295 is a safe value to use ?

This seems to be a pretty old bug.
Can anyone confirm that L.LatLng.DEG_TO_RAD = 0.017453292519943295 is a safe value to use ?

Found this in documentation, more safe to use L.LatLng.DEG_TO_RAD = Math.PI / 180
https://docs.eegeo.com/eegeo.js/v0.1.830/docs/leaflet/L.LatLng/


Is it possible to send a reset instead of dropping the packet?

Hi @jsimonetti,
Thank you very much for your work.
I'm not very familiar with Golang. Is there a way to send "TCP reset" and/or "ICMP port unreachable" instead of dropping the packet to prevent the client and the daemon from a resend cycle?

I noticed that this was an iptables (which I'm using instead of nft) issue and should be deleted. Sorry for the inconvenience.

Hi, thank you for the question. As you noticed, sniqueue only marks packets. It is up to your nftables or iptables setup to do something with those marks.
the example nftables.conf rejects bad sni packets and stores its destination in an ipset, which is then used as a cache to reject future packets to the same ip.


Add in GitHub Issue and Pull Request Templates

I need to add in a .github/PULL_REQUEST_TEMPLATE.md and .github/ISSUES_TEMPLATE.md to the project.

This has been completed.


Load JDBC drivers in scripts with arguments

I'm creating a simple script that uses JDBC with a Postgres driver.
If I do it like this:

import $ivy.`org.postgresql:postgresql:9.4.1209.jre7`
import java.sql.DriverManager

@main
def main() {
     ....
    val con = DriverManager.getConnection("jdbc:postgresql://localhost/mydatabase", "user", "password")
    ....
}

then it fails like this:
java.sql.SQLException: No suitable driver found for jdbc:postgresql://localhost/mydatabase

even though if I add before this line:
classOf[org.postgresql.Driver].getName

After some experimentation, I've found that the solution is this:
def main() {
     ....
    val con = DriverManager.getConnection("jdbc:postgresql://localhost/mydatabase", "user", "password")
    ....
}
main()

Loading the driver isn't necessary with Class.forName() or classOf[org.postgresql.Driver].getName.
Conclusion:
If I want to use arguments in my script, then JDBC drivers aren't used, even if they are loaded in the JVM.
I consider this behaviour a bug in Ammonite.
Any workaround?

I think it has to do with some Class loader magic when using arguments.

Odd, not sure what's happening, do you think you could help investigate and fix it?

Tried to place the import inside the main method, but it doesn't like it:
object org.postgresql:postgresql:9.4.1209.jre7 is not a member of package $ivy
        import $ivy.`org.postgresql:postgresql:9.4.1209.jre7`

Found a workaround:
Call inside main method this:
Class.forName("org.postgresql.Driver")

It isn't need it, when using no main method.

Should be the same problem as https://github.com/lihaoyi/Ammonite/issues/591, which was fixed in 0.8.5


BUG: Sign up function not working properly

Is there an existing issue for this?

[X] I have searched the existing issues

What happened?
@hereisSwapnil
check out this image

when i provide all creditional still they gave me " some error occure "
Record

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

https://github.com/akshitagupta15june/Moksh/issues

Hey, allow me to work on this under GSSOC' 2024

@hereisSwapnil  Thank you

@hereisSwapnil  can you please provide me your backend live url...

@hereisSwapnil
i need help
please check this

@dhruv8433  You don't need live backend url,
You can get it running locally

And Also the issue has been fixed, You can work on some other issue


TODOs in statement template rules section

TODO: consider allowing limited scripting a la https://github.com/json-path/JsonPath . TODO: consider if use cases require allowing a JSONPath expression to be a union of other JSONPath expressions.

I've removed the TODO from the text (PR incoming shortly) since we can consider this issue to be replacing it. The second part is already captured elsewhere, so this issue is for the limited scripting possibility.


Special handle for Artifact Already Exist while responding to Build Request

Description

Fixes pyrsia/pyrsia#1322
This PR handles Artifact Already Exist error in a separate way while responding to build/docker & build/maven the API end points. A function handle_err_artifact_already_exists has been introduced to achieve the same.
New messaging from the API endpoint

build/docker
build/maven

# Request
curl -i -X POST http://localhost:7888/build/docker -H 'Content-Type: application/json' -d '{"image":"alpine:3.16.2"}'

# Response
HTTP/1.1 302 Found
content-type: application/json
content-length: 165
date: Mon, 13 Feb 2023 08:21:43 GMT

{"build_id":null,"message":"Artifact already exists. Fresh build not required: Artifact ID library/alpine:3.16.2 for type Docker already exists in transparency log"}%

New messaging from the CLI
# CLI Command
% ./target/debug/pyrsia build docker --image alpine:3.16.2

# CLI Command Output
Artifact already exists. Fresh build not required: Artifact ID library/alpine:3.16.2 for type Docker already exists in transparency log

Screenshots (optional)
Following screenshot shows the impacted area of the code change.

PR Checklist

[ ] I've read the contributing guidelines.
[ ] I've read "What is a Good PR?"
[ ] I've included a good title and brief description along with how to review them.
[ ] I've linked any associated an issue.

Code Contributions

[ ] I've run pre-commit.sh (pre-commit.bat) successfully.
[ ] I've run the integrations tests
on the PR branch and all tests passes.

@betarelease Proposed potential future enhancements from the stand-up discussion

add the previous build_id instead of null to the json returned when an artifact already exists
add a header to indicate the relative inspect-log endpoint for the existing artifact


Fixes #2447 Opening the Python Environments tab causes VS 2017 to crash

Fixes #2447 Opening the Python Environments tab causes VS 2017 to crash
Handles error due to invalid path rather than crashing.
Also fixes unhandled NormalizePath failure.

👍


Resource duplocloud_aws_api_gateway_integration timeout issue fixed.

Resource duplocloud_aws_api_gateway_integration timeout issue fixed.

Task linked: DUPLO-5309 duplocloud_aws_api_gateway_integration does not finish creation


CupertinoListSection should have option to remove/customize padding

There is padding built around the actual section which is not configurable by the user.
See image:

Here it is not possible to match native behavior because of the built in leading padding.
Alternatively, it would probably be worth considering removing the padding all together if just wrapping CupertinoListSection in Padding achieves the same effect. This is preferred over enumerating padding on every widget in the framework.

See https://github.com/timsneath/landmarks-flutter/issues/13 for more context

Can I try to fix this issue?

@arrrtem22 please do if you have the time 😃. This issue seems to be related to #128419 and both could probably be solved in one PR.

@MitchellGoodwin I was laid off, so yeah, now I have the time😂
will you assign the issue to me?

@Piinks it looks like the padding above CupertinoListSection can be controlled with the topMargin property and can be set to 0. Which seems a little different from how we normally control padding.

Is that enough to functionally address this issue? And should it be renamed/ changed in some way to be more clear?

Looks like topMargin is exposed on the insetGrouped constructor, but is ignored? Here is the code:
import 'package:flutter/cupertino.dart';

void main() {
  runApp(const MyApp());
}

class MyApp extends StatelessWidget {
  const MyApp({super.key});

  // This widget is the root of your application.
  @override
  Widget build(BuildContext context) {
    return CupertinoApp(
      title: 'Flutter Demo',
      home: LandmarksPage(),
    );
  }
}

class LandmarksPage extends StatelessWidget {
  final List<String> landmarks = [
    "The Leaning Tower of Pisa", "The Eiffel Tower", "The Great Wall of China", "The Taj Mahal", "The Colosseum", "Machu Picchu", "The Statue of Liberty", "The Grand Canyon", "The Pyramids of Giza"
  ];

  LandmarksPage({ super.key });

  @override
  Widget build(BuildContext context) {
    return CupertinoPageScaffold(
      child: CustomScrollView(
        semanticChildCount: landmarks.length,
        slivers: [
          const CupertinoSliverNavigationBar(
            stretch: true,
            border: null,
            largeTitle: Text('Landmarks'),
            backgroundColor: CupertinoColors.systemGroupedBackground,
          ),
          SliverFillRemaining(
            hasScrollBody: false,
            child: CupertinoListSection.insetGrouped(
              children: [
                for (final landmark in landmarks) Text(landmark),
              ],
              topMargin: 0,
            ),
          ),
        ],
      ),
    );
  }
}

Looks like topMargin is ignored for the insetGrouped constructor, but margin does work for customizing the padding for both constructors, but it isn't clear from the documentation. For the base constructor, if both topMargin and a margin that affects the top are used, they will compound on each other, so having both seems unnecessary. It seems like the best thing to do is to remove topMargin and provide better documentation around margin.

I agree with your observations and suggestions regarding the use of topMargin and margin in the constructors. Clarifying and focusing on margin in the documentation would be beneficial.
Also, I suggest close https://github.com/flutter/flutter/issues/128419, it would be easier for me to fix these bugs separately

Fixed by #137039. The default padding is removed, and can be added back by wrapping the header with a custom padding.


[Bug] DICOM never finishes loading

Describe the Bug
See below.
Upon investigation, the following code seems to be responsible for this bug:
https://github.com/OHIF/Viewers/blob/9c553fded431b22a0704e0b58375707f41608a8f/platform/core/src/utils/isDisplaySetReconstructable.js#L187
!iop2 === undefined will always evaluate to false.
Steps to Reproduce

yarn run dev
Load http://localhost:3000/local in browser
Upload a DICOM dataset
Open the developer tools
Click the study information, then Basic Viewer

The current behavior
It never finished loading, and the following error could be seen in developer tools of the browser:
TypeError: iop2 is undefined
    _isSameOrientation app.js line 1 > srcScript:5941
    areAllImageOrientationsEqual areAllImageOrientationsEqual.ts:20
    checkSingleFrames checkSingleFrames.ts:26
    getDisplaySetMessages getDisplaySetMessages.ts:40
    makeDisplaySet getSopClassHandlerModule.js:22

The expected behavior
It loads the study without error.
OS
macOS 14.2.1
Node version
v21.3.0
Browser
Firefox 121.0.1

can you provide us with ther data


Bug in red-black tree code

Hi,
There seems to be a bug in the code for the red-black-tree. For example, this causes an exception:

delete "foo" (insert "foo" (insert "bar" E))
*** Exception: Main.hs:(84,1)-(87,59): Non-exhaustive patterns in function balR

Thanks for reporting. https://github.com/Abhiroop/okasaki/commit/292601deb36ea57e080524275dd1c090de15e27f should fix this.
It was extremely hard to track and fix this minute bug. Took me a couple of hours. If people actually intend to use this, I should add Quickcheck tests for it. Also I should add that this implementation is actually a Set rather than a Tree.  I basically added a delete functionality to the Red Black Tree backed Set demonstrated in Okasaki.

I'm not using the code as-is, but i'm using it as an inspiration and that bug cropped up. Thanks for the fix!

Closing this as https://github.com/Abhiroop/okasaki/commit/292601deb36ea57e080524275dd1c090de15e27f fixes this.


Fix empty file created by Alluxio native client in StressMasterBench

What changes are proposed in this pull request?
Fix a bug in StressMasterBnecch AlluxioNative client. The AlluxioNative client can only create files of 0KB size.The parameter --create-file-size doesn't work.
Why are the changes needed?
The parameter --create-file-size doesn't work in AlluxioNative Client, because the file created by AlluxioNative client is not populated with data.
Does this PR introduce any user facing changes?
None

@jiacheliu3 this is good to merge

alluxio-bot, merge this please


⚠ 366.ru has degraded performance

In a683512, 366.ru (https://366.ru) experienced degraded performance:

HTTP code: 200
Response time: 5829 ms

Resolved: 366.ru performance has improved in a8b2225.


Added StringComparison to StringAssert Contains(), EndsWith() & StartsWith()

Description:
Added StringComparison to StringAssert Contains, EndsWith & StartsWith
Added tests to test functionality
Added API comment on StringComparison Enum
Updated APIs to default to StringComparison.Ordinal
Probably needs API documentation updates if this gets merged.
Resolves #154

@HeroMaxPower are you going to address the PR comments or should I?

Yes please, i haven't had time in my personal life to look at these. 👍

/azp run

@304NotModified I had to pass empty array to disambiguate the calls (before we passed null). The compiler will pass empty array for us, so we should not be doing anything wrong. Unless the user passed StringComparison as the actual first param, then they now would get different overload than before. But that is unlikely scenario.

@304NotModified any updates?

@nohwnd i think this is ready to be merged 👍

@Haplois could you review and merge please.


Update package.json

package.json should be updated to have the correct links/information.

[x] repository
[x] bugs
[x] homepage
[ ] keywords
[ ] publish script that lints and tests before pushing to master.

The V3/rewrite makes this obsolete.


Some fixes for BSD support

For #16
Tested on FreeBSD 14.1, --fake and --timeout are not supported.


Post release automated changes for storage releases

Post release automated changes for azure-storage-queue

API change check
API changes are not detected in this pull request.


Route name fix

Cant prefix a route <Cards key="admin.dashboard" />
let url = route(`cards.${key}.index`)

let url = route(`${key}.cards.index`)

admin.dashboard.cards.index

Need to allow overriding
let url = route(cards.${key}.index)
let url = this.url ?? route(cards.${key}.index)


Add filters in URL

They are not in the URL to prevent the page from reloading, but they are sharable via the "link" button


Feature: Lightkeeper services

LK now provides various in-game services.
Investigate systems and ascertain if changes are required to make them function.

Originally written by chomp

Just adding the note of the services from the official patch <IP_ADDRESS> notes from BSG.
Lightkeeper services
After completing certain Lightkeeper quests, you will be able to unlock access to the following services:
"Sacred Amulet" service - Lightkeeper gives the character the item "Sacred Amulet". While the character is wearing the amulet - all cultists on all locations do not attack you unless provoked;
"Rogue Support" service - When a player purchases this service, in that raid, Rogues will not attack the player who ordered the service, regardless of their faction. In addition, Rogues will provide fire support by attacking targets that the player who purchased this service has directed their attention to;
"Zryachiy Support" service - When a player buys this service, in that raid, Zryachiy will support the player by firing at the targets the player is attacking.

Originally written by Tier1Ghost

When clicking enter on LK:

Attachments

image.png (1122 KiB)

Originally written by chomp

Are there dumps of this server interaction on live or would it be worthwhile to try to reverse engineer the response that the client wants?

Originally written by Spoony

Are there dumps of this server interaction on live or would it be worthwhile to try to reverse engineer the response that the client wants?

No dumps of LK from live yet. We've found the interactions with the BTR occur outside the usual server communication system which means our dumps are useless, this is likely the case for LK services.

Originally written by chomp

The interaction with Lightkeeper is very similar to the BTR. I made a skeleton with some functionality for the lightkeeper patch that uses the same architecture: 3ab2537afd.
Things I'm aware of that need to get worked on from here:

All of these services don't have prices associated with them, need to fix
a. All services require money as well as inventory items. Need to figure out if we can reuse BSG logic for this
Need to add Cult amulet support
Need to see if there's cutscene animations/dialogue for buying any of these things
Need to make sure that Zryachiy is marked unfriendly when attacking rogue USECs. He might be, but I received this error when attacking them to test. My guess is this is caused by him attacking them.

Any vods or videos of people on live buying lightkeeper services would be good to find to ensure my assumptions are correct.

Attachments

image.png (1114 KiB)

Originally written by Spoony

I found info about tasks and requirments for services.
"Sacred amulet" - available after completion of quest Snatch, for purchase 3x Intelligence folder and 1000 Dollars. Used for 7 rounds before it dissapears, can be unequipped to retain uses. Ocultists dont attack unless provoked, while Zryachiys guards and not affected by it.
"Rogue support" - available after completion of quest Provocation, for purchase 1x TerraGroup "Blue Folders" materials and 2500 Euros. In that raid Rogues dont attack player.
"Zryachiy support" - available after completion of quest Make an Impression, for purchase 3× Military flash drive and 2,000 Dollars. When a player buys this service, in that raid, Zryachiy will support the player by firing at the targets the player is attacking.
Hope this helps in some way.

Originally written by ItsRenke

Any further work on LK services should be a new issue, closing.

Originally written by chomp


v<IP_ADDRESS> patch

fix Autokeys scrapAdjustment, which +0.11/trade made

No, this does not work.

PR looks good. I may modify the uptime logic for the !uptime command

Thanks. PR merged.


Introduce API for current host

This makes the host of the current request available from within the
fastboot service
This should be merged after ~~https://github.com/ember-fastboot/ember-fastboot-server/pull/29~~ and #131

This also requires an update to ember-cli-addon-tests as well, we'll want to figure out how to move forward with https://github.com/tomdale/ember-cli-addon-tests/issues/4

Any updates on this? Is there anything I can do to help this along?

Waiting on #131 to merge plus review @paulelliott

@tomdale This requires a release of ember-fastboot-serverto work


Add a texture widget?

Describe the solution you'd like
Add a texture widget for render a png file in the filesystem
Additional context
I know this will be difficult because of minecraft ResourseLocation system, but it will also be super cool!

Binding a raw byte array texture isn't really that hard, and that's also how textures are loaded in Minecraft - it opens the stream, and load into a byte buffer, and add the texture, the hard part is you can't send that large amount of data with the packet system


[FX-5115] Update types of compound components

FX-5115
Description
When we leave on Typescript to set types for compound components, it is resolving wrongly paths to other packages.
We can't have packages/shared/dist-package/src in generated files as they do not exist and it needs to be resolved to @toptal/picasso-shared. It seems like a bug in Typescript itself.
When we explicitly type the compound component, the types are generated correctly.
How to test

Temploy
check screenshot bellow, or
you can check locally the branch and open story of Page and try to read types of Page.Article

on current commit it should show proper types
if you remove the types from the Page compound component, it will not work again

Screenshots

Before.
After.

Development checks

[x] Add changeset according to guidelines (if needed)
Read CONTRIBUTING.md and Component API principles
Make sure that additions and changes on the design follow Toptal's BASE design, and it's been already discussed with designers at #-base-core
Annotate all props in component with documentation
Create examples for component
Ensure that deployed demo has expected results and good examples
Ensure the changed/created components have not caused accessibility issues. How to use accessibility plugin in storybook.
[x] Self reviewed
Covered with tests (visual tests included)

Breaking change

codemod is created and showcased in the changeset
test alpha package of Picasso in StaffPortal

All development checks should be done and set checked to pass the
GitHub Bot: TODOLess action

PR commands

List of available commands:

@toptal-bot run package:alpha-release - Release alpha version
@toptal-anvil ping reviewers - Ping FX team for review

PR Review Guidelines

When to approve? ✅
You are OK with merging this PR and

You have no extra requests.
You have optional requests.

Add nit: to your comment. (ex. nit: I'd rename this variable from makeCircle to getCircle)

When to request changes? ❌
You are not OK with merging this PR because

Something is broken after the changes.
Acceptance criteria is not reached.
Code is dirty.

When to comment (neither ✅ nor ❌)
You want your comments to be addressed before merging this PR in cases like:

There are leftovers like unnecessary logs, comments, etc.
You have an opinionated comment regarding the code that requires a discussion.
You have questions.

How to handle the comments?

An owner of a comment is the only one who can resolve it.
An owner of a comment must resolve it when it's addressed.
A PR owner must reply with ✅ when a comment is addressed.

@toptal-anvil ping reviewers


Plugin not found in Obsidian community plugin database

Hello,
As title says, I don't find the plugin in the community database. Typing either Chat-GPT, Chat-GPT Assistant, AI, nothing is found.

Obsidian Plugins have to be reviewed and approved to be included in the store. I'm hoping that the devs will be able to approve my plugin by the end of next week.

Alright thank you for your reply :)


assignmentJSON with multiple subassignments

http://bridges-clone.herokuapp.com/assignmentJSON/2/dburlins
should return an array of all assignments associated with assignment number 2.. (2.00 - 2.09)

This functionality is not necessary for now, closing this issue.
In future if we want this, we may want a different endpoint to avoid complicating the getJSON method in the assignment controller.


Code Revision: Ed25519

Overview / Motivation
We enable public key extraction from signed messages. This results in shorter messages.
The Task
Code revision is required:

new signing algorithm
extraction algorithm
new functions: invert, modular multiplication, elliptic curve point multiplication

Implementation Notes
See code in https://github.com/spacemeshos/ed25519:

signing and extraction in ed25519_ext.go
new functions in internal/edwards25519/invert.go

See also
https://github.com/spacemeshos/ed25519/pull/1
Contribution Guidelines
Important: Issue assignment to developers will be by the order of their application and proficiency level according to the tasks complexity. We will not assign tasks to developers who have'nt introduced themselves on our Gitter dev channel

Introduce yourself on go-spacemesh dev chat channel - ask our team any question you may have about this task
Fork branch develop to your own repo and work in your repo
You must document all methods, enums and types with godoc comments
You must write go unit tests for all types and methods when submitting a component, and integration tests if you submit a feature
When ready for code review, submit a PR from your repo back to branch develop
Attach relevant issue to PR

Obsolete.


Added input range styling

Added the range input component to the plugin.
Because of all the weird browser prefixes I've only used the color and size options inside the default options file.
Added documentation to show how to use and configure the range input.
I've tested the slider on Chrome/Firefox/IE11/Edge on Windows 10 and Chrome/Firefox on Android 9. If someone could test iOS and MacOS that would be great.

Would love to see this merged..

Hello, how can I test this?

Anything else needed to get this merged?

@adamwathan @MartijnCuppens could we merge this please

Hey! I think this is a pretty interesting idea but I would want to spend some time working with our design team to make sure we are happy with the defaults. Will leave this open in the mean time and hopefully something we can tackle in the future.