Add generate boilerplate script

Fixes #410

Proposed Changes

:broom: Add a script to easily generate/add boilerplate to new files

Pre-review Checklist

[x] At least 80% unit test coverage
[x] E2E tests for any new behavior
[x] Docs for any user-facing impact

Release Note

Codecov Report

Merging #415 (945bfb0) into main (fc50485) will decrease coverage by 1.33%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #415      +/-   ##
==========================================
- Coverage   83.17%   81.83%   -1.34%
==========================================
  Files          27       27
  Lines        1040     1382     +342
==========================================
+ Hits          865     1131     +266
- Misses        147      223      +76
  Partials       28       28

Impacted Files
Coverage Δ

...g/apis/sources/v1alpha1/vspheresource_lifecycle.go
93.33% <0.00%> (-2.32%)
:arrow_down:

.../apis/sources/v1alpha1/vspherebinding_lifecycle.go
94.11% <0.00%> (-2.11%)
:arrow_down:

pkg/vsphere/checkpoint.go
80.95% <0.00%> (-0.87%)
:arrow_down:

plugins/vsphere/pkg/command/source/list.go
82.14% <0.00%> (-0.42%)
:arrow_down:

plugins/vsphere/pkg/command/binding/list.go
82.14% <0.00%> (-0.42%)
:arrow_down:

plugins/vsphere/pkg/command/auth/create.go
90.72% <0.00%> (-0.07%)
:arrow_down:

pkg/vsphere/adapter.go
64.73% <0.00%> (-0.02%)
:arrow_down:

pkg/vsphere/client.go
0.00% <0.00%> (ø)

pkg/vsphere/vsphere.go
100.00% <0.00%> (ø)

pkg/apis/sources/v1alpha1/register.go
100.00% <0.00%> (ø)

... and 17 more

Continue to review full report at Codecov.

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


A11y_AzureDataStudio_New deployment - SQL Server on Windows_Screen Reader: Narrator is not announcing previous as button in Microsoft sql server license terms page

GitHub Tags:
#A11yMAS; #A11yTCS; #A11ySev2; #DesktopApp; #BM-AzureDataStudio-Win32-Feb2022; #Azure Data Studio; #WCAG1.3.1; #Win32; #AINotInScope; #Info and Relationships; #Narrator; #NVDA; #Element:Button;
Environment Details:
Windows Version :Windows 11 Enterprise
Application name :Azure Data studio
Repro Steps:
1.Turn on Narrator
2. Launch Azure Data Studio.
3. Select the Connections icon (if not already selected) from left bar.
4. Click on the ellipses '...' and select "New Deployment"
5. In the Deployment options, select "SQL Server on Windows" card.
5. From the dropdown for Version under Options, select from the available options.
6. Click the "Select" button
7. This will start the setup installer for SQL Server setup.
8. Follow the instructions through the setup process to install a SQL Server on Windows.
9. Verify the previous button in Microsoft sql server license terms page is announcing by narrator as button or not
Actual Result:
Narrator is not announcing previous as button in Microsoft sql server license terms page
Expected Result:
Narrator should announce previous as button in Microsoft sql server license terms page
Attachment:
Narrator Attachment:

NVDA Attachment:

the install SQL server feature you tested is not a part of AzureDataStudio. so closing it, let me know if you don't know the owner of the sql server installer.

@alanrenmsft Could you please let us know if we need to route this bug to the SQL server installer team? If yes, could you please do let us know the vso and owner's details?

@alanrenmsft Could you please route it to the appropriate owner or team and reactivate the bug.
Note: "Please do not close this bug. This bug should only be closed by TCS, C+AI Accessibility or the Divisional Driver"

Verified issue in below Environment:
Version: 1.41.1 (user setup)
Commit: 33dce95b152227c4f10f6251ab30690fe9895c0c
Date: 2023-01-28T00:46:39.274Z
VS Code: 1.67.0
Electron: 19.1.8
Chromium: 102.0.5005.167
Node.js: 16.14.2
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22621
Issue is fixed


disable create as select in production

related: #6569

Need further discussion:
          Originally, we intend to ban 'create table as select from' in production.

Now if dbt requires this, it seems we cannot ban it.
Originally posted by @lmatz in https://github.com/risingwavelabs/risingwave/issues/7826#issuecomment-1425388322


Options to remove additional bank options

Adds the option to remove repeated bank options when the default quantity is changed, and also adds the option to remove Withdraw All but 1.

To my knowledge we currently don't want to edit bank options at all.

These options just bring it back to how it was on release

In addition to not wanting to touch bank options, we also never remove menu options and want to keep it that way.

I don't see anything wrong with removing withdraw-but-one or the custom withdraw-x amount. The way this approaches this I think can be simplified though by just checking the menu params. This is what it always used to be & wouldn't be advantageous.

Yes these changes just bring it back to how it used to be.
@Adam- It does remove your custom withdraw-x/deposit-x option, instead when the Quantity option on the bank options is changed to 5, 10, X, or All, it removes the extra option that is added. For example, normally with Quantity set to All, your options would become: All, 1, 5, 10, (custom x), X, All. My option changes it to: All, 1, 5, 10, (custom x), X. It removes the additional All menu entry since it is the new 1st option.


Create jadavpuruniversity.txt

Adding Jadavpur University to the list

@biswajit-halder-ju Please provide us with:

the university official website URL, if different from the domain you are submitting
an URL of a page on the official website where an IT-related long-term (>1 year) course is offered by the university
an URL of a page or another proof (.pdf or a screenshot are OK) showing that the university recognizes the domain which you are submitting as an official email domain for the students.

University official website URL:    http://www.jaduniv.edu.in/

2.1    Course offered URL:  Direct URL not visible. Go to
http://www.jaduniv.edu.in/ > Academics > Cources > Select discipline "Engg
and Tech", Level "Any" and Department "Any" > Search  ::: Choose any
computer related course for details.
2.2   Syllabus URL for Computer Science and Engineering :
http://www.jaduniv.edu.in/upload_files/course_file/1333457026-1.04

Domain Proof::  regarding the email domain, it is mentioned at the
bottom of university home page(screenshot attached). A seperate pdf file
link is here that shows the email ids of faculty of Computer Science and
Engineering Department.
http://www.jaduniv.edu.in/new_upload/new_emaillist/engg/cse.pdf
http://www.jaduniv.edu.in/new_upload/new_emaillist/engg/cse.pdf

Thanks & Regards,
Biswajit Halder
Junior Research Fellow
Dept. of Computer Sc. & Engg.
Jadavpur University
M:<PHONE_NUMBER> /<PHONE_NUMBER>
Email<EMAIL_ADDRESS><EMAIL_ADDRESS>On Mon, Jun 4, 2018 at 8:46 PM, Philip Torchinsky<EMAIL_ADDRESS>wrote:

@biswajit-halder-ju https://github.com/biswajit-halder-ju Please
provide us with:

the university official website URL, if different from the domain
you are submitting
an URL of a page on the official website where an IT-related
long-term (>1 year) course is offered by the university
an URL of a page or another proof (.pdf or a screenshot are OK)
showing that the university recognizes the domain which you are submitting
as an official email domain for the students.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JetBrains/swot/pull/4524#issuecomment-394391820, or mute
the thread
https://github.com/notifications/unsubscribe-auth/Al9N4iO2hMkPd4giX7fQwF0Ikr0L5_5Pks5t5U8-gaJpZM4UVBIN
.

@biswajit-halder-ju Pull request merged. Thank you for the update!


PaperTrail + Inheritance

This PR resolves issues https://github.com/sferik/rails_admin/issues/1524, https://github.com/sferik/rails_admin/issues/2332, https://github.com/sferik/rails_admin/issues/2691
It is based on https://github.com/sferik/rails_admin/pull/1567/files.
PaperTrail stores a base_class for  each subclass, so we can't fetch versions with searching by subclass. STI model allows us to fetch records with query condition which includes STI type, so model.model.all should return a valid AR relation for current STI type and we can fetch a versions by this relation.
When we have a deal with object versions think we can't do better than to use native PaperTrail method versions, so object.versions should work well.
These changes were checked on our production db.
Here is illustration how PaperTrail works:

class PaperTrailTest; end
class PaperTrailTestSubclass < ::PaperTrailTest; end
class PaperTrailTest::SubclassInNamespace < PaperTrailTest; end
versions = PaperTrail::Version.where(item_type: 'PaperTrailTest')
PaperTrailTest.create
versions.count # 1
PaperTrailTestSubclass.create
versions.count # 2
PaperTrailTest::SubclassInNamespace.create
versions.count # 3

:+1, i have same issue, would be nice to have these changes in master

Excellent, thanks!


Run x11rb-async examples in CI

Well... this also only runs this on Linux. But at least it runs it at all. :sweat_smile:
Since AppVeyor was recently having problems, I don't really want to try to make a similar change to appveyor.yml for now.

Actually, is there any reason why we're using AppVeyor, instead of just using the Windows GitHub Actions containers? Is it because of XVFB or something?

I think/guess that either "back then" GitHub didn't have Windows runners or I just did not know about them. https://github.com/znc/znc is using AppVeyor for Windows CI and I guess I just did not do any research into options, but used that too.


Injecting values to a Poly defined outside of calling method is awkward

An easy way to inject information into objects extending Poly is by defining the object inside the method body which is later using the Poly for mapping/folding. In cases where the object is needed for evidence resolution to the method itself things get trickier.
Example:
case class Something(va: Boolean)

object Main extends App {
  import shapeless._

  val tc = Something(true)
  val res = Helper.parseCaseClass(tc, false)

  println(res) // Should be Something(false)
}

object Helper {
  import shapeless._
  import record._
  import poly._
  import syntax.singleton._

  object updateElem extends Poly2 {
    import ops.record._

    implicit def default[Repr <: HList, T](implicit update: Updater.Aux[Repr, FieldType[T, Boolean], Repr]) =
      at[Repr, T] { (acc, t) => update(acc, field[T](true)) } // I want to replace true here with the `default` argument in the method below
  }

  def parseCaseClass[T, Repr <: HList, Kys <: HList](cc: T, default: Boolean)(
    implicit lgen: LabelledGeneric.Aux[T, Repr],
    kysEv: ops.record.Keys.Aux[Repr, Kys],
    folder: ops.hlist.LeftFolder.Aux[Kys, Repr, updateElem.type, Repr]) = {

    val rec: Repr = lgen.to(cc)
    val keys: Kys = rec.keys

    val res = keys.foldLeft(rec)(updateElem) // How do I inject `default` to the folder, when the folder isn't defined inside this method?
    lgen.from(res)
  }
}

In the example above we can't define updateElem inside parseCaseClass as the type of updateElem is needed for the LeftFolder evidence. Our only way of injecting default into updateElem is by defining a custom LeftFolder or, easier, injecting it with the initial value.
 val res = keys.foldLeft((default, rec))(updateElem)._2

This isn't a very pretty or flexible solution.

Some bad news, some good news ...
The bad news is that there's nothing that we can do to directly solve this problem in general ... a satisfying solution of the form you want would require forms of dependent typing that go beyond Scala's current capabilities.
The good news is that while looking at this again I've come up with a somewhat more attractive solution for this specific case and sufficiently similar ones. The aim is to have the fold parametrized by a boolean value, and we can capture that by having the fold yield a function of the form Boolean => Repr instead of a value of type Repr.
This gives us something along the following lines,
object Helper {
  import shapeless._
  import labelled._
  import record._
  import poly._
  import syntax.singleton._

  object updateElem extends Poly2 {
    import ops.record._

    implicit def default[Repr <: HList, T](implicit update: Updater.Aux[Repr, FieldType[T, Boolean], Repr]) =
      at[Boolean => Repr, T] { (acc, t) => (b: Boolean) => update(acc(b), field[T](b)) }
  }

  def parseCaseClass[T, Repr <: HList, Kys <: HList](cc: T, default: Boolean)(
    implicit lgen: LabelledGeneric.Aux[T, Repr],
    kysEv: ops.record.Keys.Aux[Repr, Kys],
    folder: ops.hlist.LeftFolder.Aux[Kys, Boolean => Repr, updateElem.type, Boolean => Repr]) = {

    val rec: Repr = lgen.to(cc)
    val keys: Kys = rec.keys

    val resFn: Boolean => Repr = keys.foldLeft((_: Boolean) => rec)(updateElem)
    val res = resFn(default)
    lgen.from(res)
  }
}

case class Something(va: Boolean)

object Main extends App {
  import shapeless._

  val tc = Something(true)
  val res = Helper.parseCaseClass(tc, false)

  println(res) // Something(false)
}

Which is fairly close to your original.
I'm going to close this ticket now, because I don't think we're going to be able to do significantly better than this. If anyone is able to come up with a better solution, please let me know.


Keyboard constantly unresponsive (both Magic & Folio)

Especially after running the code, and entering input, when wish to continue programming, they board strokes are not registered.
Happens on both Magic Keyboard and Folio. So it’s not the keyboard. Using latest iPadOS16.
It happens so often that impossible to be productive. Have to close Code App, then launch again; only then keyboard beings working again.

Thank you for reporting this. I will take a look into it as soon as possible.

Appreciated. Can replicate by:
Step 1: Attach Magic keboard/Folio or any external BT keyboard.
Step 2: Create new file "file.py" (Python)
Step 3: put into code:

text = input()
print(text)
Step 4: Run. And enter text in the console.
Then tap back into code section. And watch how no keystrokes are registered.
To fix, minimize the "Code App" window. Then go into it again.
On Tue, Nov 8, 2022 at 1:58 PM Chung Shing Hin @.***>
wrote:

Thank you for reporting this. I will take a look into it as soon as
possible.
—
Reply to this email directly, view it on GitHub
https://github.com/thebaselab/codeapp/issues/641#issuecomment-1306553765,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ANOQULGKVSOXLAJPVM7JC4LWHG6URANCNFSM6AAAAAARZYNA44
.
You are receiving this because you authored the thread.Message ID:
@.***>

The other problem with extrenal keyboard is that you can’t rename files with it - the cursor for editing doesn’t show. It seriously reduces usefulness of this program.

This could be an iPadOS bug because it's improved in the latest TestFlight version built with Xcode 14 SDK. You might try it here: https://testflight.apple.com/join/EgZ8sE2P.
Be aware that TestFlight version currently has a few other bugs: https://github.com/thebaselab/codeapp/issues/638

Hello,
I have the same problem.
Each time I create a new file, my external keyboard in the editor zone is unresponsive. I switch to terminal zone and my keyboard is working fine. I bring back to editor zone and I’m unable to type any characters.
I use and other app to edit the empty file and add a word. I reopen the same file in CodeApp and now my keyboard works fine.
An other test : create new file ‘file.py’, my keyboard is unresponsive. In the terminal, I ‘echo test > file.py’, close and reopen the file and my keyboard is working.
Ipad Pro M1 with SmartKeyboard
IpadOS 16.3
CodeApp 1.4.3 build 186
Thank’s

Did you check the disable keyboard toolbar option in settings menu? It's known to cause some issues and I'm investigating. For now, please uncheck the option and re-open the app.

The option was already disabled.
I enabled, restart app and the problem still present.
I disabled,  restart app and the problem still present.
I use stagemanager!

I use stagemanager!

I see. I will take a look into it. Thanks for reporting the bug.


Issue: when input manually day, the  previous day is selected instead

Hello,
First of all, thank you for this fork that fixed finally annoying bug.
I've observe strange behavior when you input data manually.
I use following date format DD/MM/YYYY
When you start input day it looks that Datepicker  select instead the previous day.  Take a look to screenshot. It doesn't occurs if you input complete date  just when you input day or day+month
 <datepicker
      :value="value"
      :typeable="true"
      format="DD/MM/YYYY"
      language="es"
      input-class="is-valid"
      @input="update($event)"
      :bootstrapStyling="true" />

Sorry. for noise  it was an issue with my code.


add Drone CI

What does this PR do?
Adding CI with GPU support...
PR review
Anyone in the community is free to review the PR once the tests have passed.
If we didn't discuss your PR in Github issues there's a high chance it will not be merged.
Did you have fun?
Make sure you had fun coding 🙃

see also https://github.com/PyTorchLightning/pytorch-lightning/pull/1021


fix: remove the quadratic cost of finding double spends for IFE txs

Slightly related to #541, but more so to the choppy syncs of a late development chain, due to expensive checking of in-flight exits
Overview
The initial implementation of in-flight exit double spend seeking involved operations costing roughly O(#ife^2) and O(#ife*#txs_in_some_blocks), which could've exploded badly. We change the implementation (tests untouched), to use several mappings allowing us to have no quadratic operations.
Changes

KnownTx module now encompasses operations and preparations steps concerning the transactions we are scanning for double spends. The mappings are prepared there
to find double spends we leverage mainly the input_spent => txs_spending_it mapping allowing us to quickly fetch double spends for an ife/se

Testing
unit tests should cover these behaviors decently.

Coverage increased (+0.09%) to 81.723% when pulling fce66a2417b25a617ff546d49e9fd923d2809bae on 541_remove_quadratic_cost_from_ife_handling into 2ee43ec09b89df24fdc05b85ea56b1dabe1fb339 on master.


Incorrect docker-compose.yml file on doc site

The instructions at https://ptarmiganlabs.github.io/butler/install-config/#install_on_docker refer to verisure_mqtt as the name of the Docker container, that is a copy-paste error..
Should be "butler".

Fixed in c6a0a96


Allow users to name the calendar file exported to google calendar

after exporting a schedule to gcal (not sure about ical), i am unable to change the name of that calendar in google calendar- it is currently named "uwflow.com schedule" by default.
I think there should be a modal that comes up after you select the option to download to a given calendar, such that the user can determine what they want to name the schedule if they so choose, with "uwflow.com schedule" remaining as the default.

Relevant code is here: https://github.com/UWFlow/rmc/blob/f7fcd1ac1a966add29ef6692ff1a5126ee407851/server/profile.py#L92

Alternatively, we could default to a sane value based on their most recent term. For example, if I upload my schedule, we can figure out it's my 3A term. Then we can default the calendar name to "3A" instead of "uwflow.com schedule". Just a suggestion. Otherwise, you'll like need to pass the name as a parameter to the calendar when it's created.
Side note: You can rename the calendar in Google Calendar, it's just a PITA


ci: buildspec for alternate file structure (codecommit)

Resolved issues:
Internal
Description of changes:
Path structures differ- use the buildspec to look for third-party-src and descend into it if present.
Call-outs:
Sidetrail is still failing due to patching inconsistencies
Testing:
Against my fork of s2n.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #2185 into master will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master    #2185      +/-   ##
==========================================
- Coverage   80.79%   80.77%   -0.02%
==========================================
  Files         248      248
  Lines       18522    18522
==========================================
- Hits        14964    14961       -3
- Misses       3558     3561       +3


Call out performance-optimized k8s config

Fixes #3823.

This change is

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/159ea74e0862b1f47178b967216d107aaa91f6b5/

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/f433be445262f7cf1f158d450ea147646dc06c15/

Look ok now, @a-robinson?

_includes/v2.0/orchestration/start-cluster.md, line 28 at r2 (raw file):
Previously, a-robinson (Alex Robinson) wrote…
For what it's worth, wget isn't on mac by default. Or at least it isn't on my mac. Do we use it over curl elsewhere in the docs?

Good catch. We're pretty inconsistent in the docs.  I'll just use curl here in the hope that most users are on mac.

http://cockroach-docs-review.s3-website-us-east-1.amazonaws.com/cb2d95dee8f805d983ea14ab9e84ed473c212a53/

It seems we only call out the more performant deployment file when running secure? Should we have some mention in the insecure area, as that seems to be what most people do when running a POC.

Good point, @rkruze. Unfortunately, we only have a secure performant config. It'd probably be confusing to reference that in an insecure deployment. If we can create an insecure performant config, I can include it in the same way.

I do think it would be useful, thanks!
On Fri, Oct 5, 2018 at 9:25 AM Alex Robinson<EMAIL_ADDRESS>wrote:

@rkruze https://github.com/rkruze if an insecure version would be
useful, I can certainly add one.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/cockroachdb/docs/pull/3827#issuecomment-427422807,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAVecSTEQA6GNRKP5n5fL54Q2_DTauPzks5uh4fpgaJpZM4XIU7Y
.

--
Roko Kruze
Solutions Engineer
Cockroach Labs
Direct<PHONE_NUMBER>


add options to save or push model

Add options to either push the model to pub or save it locally, and deprecate 'use_auth_token' that is no longer in use.

Thanks for updating the examples and notebooks. I see that you put the explanations in a string using """. IMO, the better fit would be to use comments in the Python scripts and Markdown cells in the notebooks. Could you please change that?

ic, I will change it in the notebooks.

@BenjaminBossan hi,
i changed, check it:]

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

The reason for adding {dataset_name} is to make it more specific that can recognize more easily.
When token is filled in, the user id is recognized. So myrepo/mymodel is not needed.
I will modify the argument based on https://github.com/huggingface/transformers/blob/3bc50d81e6c70d63e59d635106bac6a561b47681/src/transformers/utils/hub.py#L759

I will improve the PR according to your suggestions:)

I made some changes. Are there any further improvements needed?


Removes previous experimental feature sections

The removed sections represent features that are no longer experiments.

Docs Build status updates of commit 6c3ba8a:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

microsoft-edge/devtools-guide-chromium/experimental-features/index.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Moved changes to branch chromiumDocs_5749758.


pkg/proc: tolerate absence of stack barriers in Go 1.9

Stack barriers were removed in Go 1.9, and thus code that
expected various stack-barrier-related symbols to exist
does not find them.  Check for their absence and do not
crash when they are missing.

Fixes #754 ?
Alex

Yes, fixes #754.


Set Stream Key and Stream Policy while streaming using Media3.

I am using AWS cloudfront for streaming my audio for my app. I want to add restrictions to prevent mod users from streaming audio.
This is the link to their docs on how to do it: Cloudfront
Inside this, they have referred to the customization part of exoplayer
Basically, they want me to pass stream_key and stream_policy which is generated against a specific audio Url in order to be able to stream.
Below is my current implementation:
val cacheDataSourceFactory: DataSource.Factory = CacheDataSource.Factory()
                        .setCache(DemoUtil.getDownloadCache(this))
                        .setUpstreamDataSourceFactory(DemoUtil.getDataSourceFactory(this))
                        .setCacheWriteDataSinkFactory(null) // Disable writing.

val dataSourceFactory = DataSource.Factory {
                    val dataSource = DefaultHttpDataSource.Factory().createDataSource()
                    dataSource.setRequestProperty("stream_key", streamKey)
                    dataSource.setRequestProperty("stream_policy", streamPolicy)
                    dataSource
                }

mediaPlayerInstance =
                    ExoPlayer.Builder(this).setMediaSourceFactory(
                        DefaultMediaSourceFactory(this).setDataSourceFactory(dataSourceFactory).setDataSourceFactory(cacheDataSourceFactory)
                    ).build().apply {
                        repeatMode = Player.REPEAT_MODE_OFF
                        playWhenReady = true
                        setWakeMode(C.WAKE_MODE_NETWORK)
                    }.also { it.addListener(playerListener) }
            }

mediaSessionInstance = MediaSession.Builder(this, player)
                    .setCallback(this)
                    .build()

mediaSession = mediaSessionInstance
mediaSession?.setCustomLayout(notificationPlayerCustomCommandButtons)

I have offline caching implemented for which the CacheDataSource is there and below is are my headers. But this does not seem to work. I feel like, I am missing something. What would be the correct approach for this?

i am also facing similar issue. I tried to add headers but it was not working
 val cacheDataSourceFactory: DataSource.Factory =
                    CacheDataSource.Factory()
                        .setCache(DemoUtil.getDownloadCache(this))
                        .setUpstreamDataSourceFactory(DemoUtil.getDataSourceFactory(this))
                        .setCacheWriteDataSinkFactory(null) // Disable writing.
                val dataSourceFactory: DataSource.Factory =
                    ResolvingDataSource.Factory(cacheDataSourceFactory) { dataSpec ->
                        dataSpec.key
                        dataSpec.withRequestHeaders(mapOf("stream_key" to streamKey))
                    }
                mediaPlayerInstance =
                    ExoPlayer.Builder(this).setMediaSourceFactory(
                        DefaultMediaSourceFactory(this)
                            .setDataSourceFactory(dataSourceFactory)
                            .setLoadErrorHandlingPolicy(myLoadHandlingPolicy())
                    ).build().apply {
                        repeatMode = Player.REPEAT_MODE_OFF
                        playWhenReady = true
                        setWakeMode(C.WAKE_MODE_NETWORK) // Wake Mode when media is playing and also needs to load via network
                    }.also { it.addListener(playerListener) }

@debz-eight @GautamCoder8
Could you both elaborate? Are you not seeing any custom headers with the key name "stream_key" added to your HTTP requests?


BYO OpenAI Model Support

[ ] Events to be linked to cross group model deployments
[ ] add GroupName to deployments
[ ] add GroupName to configuration
[ ] Update router to respect Group Name model deployments

Multi-tenant BYO OpenAI Model implemented and tested in branch https://github.com/gloveboxes/azure-openai-service-proxy/tree/mult-tenant


Added parens to print statement

Caused an error for me.  Not sure if that was intentional.

NVM.  Sorry didn't realize this was a Python 2 project.


⚠ Bofrost.es has degraded performance

In f69b50d, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1079 ms

Resolved: Bofrost.es performance has improved in 86df7a8 after 6 minutes.


Non-used confused parameter in QRouterDelegate

I noticed the parameter notFoundPage in QRouterDelegate which is not used. And this is confusing a little as I tried to set my value for this parameter and got nothing. Is it required for new functionality or is it forgotten to remove?

you are right. that is from old code and I forgot to remove it. Thanks for the hit.
I will remove it in the next release

Fixed in 1.4.0


INIT: Parse >PING< command

To request node – if it is in the network. Must be implemented in any node.

#
Field Name
Description
Explanation
Example
Max Length
Optional
>fieldName

1
Protocol
Name
EFP2 Cyrano 2.0
EFP2
4
N
protocol

2
Command
Name
PING
PING
8
N
command

Example:
|EFP2|PING|

commit 2e5f49c2c1dc2726ce6a11692ae1ec99f2771a56


Release new version of the action

Dear @camunda-community-hub/devrel,
I suggest to release this action in a new version. This would allow to use the new features of:

test result publication #34
auto release on close #33

Before doing this, please let us activate the repository https://github.com/camunda-community-hub/community-hub-extension-example/ and use it as a test to produce the 1.0.0 release there (it uses this action version from the main branch).
To do so, there must be GH secrets, required for the publication (as described in https://github.com/camunda-community-hub/community-hub-extension-example/issues/1).
Cheers,
Simon

Howdy @zambrovski - just saw this issue now - I think it is fixed as we released two new version in the meantime: https://github.com/camunda-community-hub/community-action-maven-release/releases. Please object if not!


Admob Banned Invalid Activity

Why, I use this Admob plugin, my Admob / Adsense account has been banned for reasons of Invalid Activity, even though I play honestly.

You should contact Google for the reasons, no one here can answer for your particular case. It is unlikely relate to this plugin as many using it without problem. Although this should not be necessary  You may try rewrite to use cordova-admob-plus to see if it helps.

@ratsonI have already tried admob-plus but the error, I have already written in issues admob plus but there is no solution. https://github.com/admob-plus/admob-plus/issues/25


Chapter number option doesnt work on a specific kinds of entries

Steps to reproduce

Find a novel entry with no chapter numeration whatsoever
checkmark the option in the image

Expected behavior

Actual behavior

LNReader version
1.1.17
Android version
Android 8
Device
Huawei p8 lite 2017
Other details
the entry in the screenshot (bugged) is this

Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with a source, I should be opening an issue in the sources repository.
[X] I have updated the app to version 1.1.17.
[X] I will fill out all of the requested information in this form.

The chapter names of mentioned novel doesn't have chapter numbers in them. The app wouldn't be able to parse the number if its not present in the name.

the app can accurately number chapters already.
https://github.com/LNReader/lnreader/assets/75091899/d139a9f9-791c-478c-ae39-447d4246b57d
And that is why it should assign a number to them. If there isn't, the app should do it.
can u reopen the issue?
if not, do i need to do a feature request issue and ask to numerate chapters of entries who don't have their chapter numbered (extension doesnt provide for it).

the app can accurately number chapters already.

It's not the chapter number, it scrolls to the item present at 1000th index.

And that is why it should assign a number to them.

It wouldn't be accurate. If the chapters are not in order or there aren't all the chapters present in source website using the index would just show wrong numbers.

if not, do i need to do a feature request issue and ask to numerate chapters of entries who don't have their chapter numbered (extension doesnt provide for it).

If you have a accurate solution which handles all the edge cases, feel free to open a feature request.


CheckboxGroup with indeterminate/select all checkbox

Discussed in https://github.com/adobe/react-spectrum/discussions/5215

Originally posted by richbachman October  6, 2023
Hi team. First off, great work on the react-aria-components work. I'm really enjoying seeing it grow. I'm working on using the Checkbox and CheckboxGroup components, and am struggling with coming up with a way to have a select all option with an indeterminate state. Here's what I have so far, but something just isn't clicking. I didn't see an example on the doc site, but do you have one worked up somewhere else?
Thanks in advance.
const CheckboxGroupExample = () => {
  const [selected, setSelected] = React.useState(["subscribe", "test"]);

  const allSelected = selected.length === 3;
  const indeterminate =
    selected.length < 3 && selected.length > 0 && !allSelected;

  return (
    <>
      <CheckboxGroup
        errorText="This input does not meet the requirements."
        helpText="Help text that should hopefully just be one line but at most may stretch to two lines."
        label="Group label"
        value={selected}
        onChange={setSelected}
      >
        <Checkbox
          value="select-all"
          isIndeterminate={indeterminate}
          isSelected={allSelected}
        >
          Select all
        </Checkbox>
        <Checkbox
          value="subscribe"
        >
          Subscribe
        </Checkbox>
        <Checkbox
          value="test"
        >
          Test
        </Checkbox>
        <Checkbox
          value="another"
        >
          Another
        </Checkbox>
      </CheckboxGroup>

{selected}

    </>
  );
};
```

Posted a workaround in the discussion: https://github.com/adobe/react-spectrum/discussions/5215#discussioncomment-7276156. Is the proposal here to add an official select all checkbox slot or something? The hard part with that would be determining the values of all of the items in the group so that we can set the selected items appropriately. CheckboxGroup isn't implemented as a collection component like our others so it doesn't have access to all of the keys.

No proposal yet, wanted to discuss it. Unsetting the context seemed unintuitive. I also wanted to check if there were any aria concerns since a screen reader user would be presented with the select all first, but then would still need to visit the remainder individually in order to know what all has been selected or is available. Then they might need to navigate all the way back to actually select all. I'm not sure how that could be improved or if that's something we'd concern ourselves with.

We could potentially use a slot for this. This would be nice to have, but not blocking for RAC GA. We will have to bring up this use case with Spectrum designers to see if we want to support this in React Spectrum.


Upper browser content is not clickable

Steps to Reproduce the Problem

Open browser window with a webpage with an clickable item on top
Click the item

Expected Behavior
Webpage executing click
Actual Behavior
Nothing, due to invisible action bar
Specifications

Version: 0.1.2
Platform: Windows

When should we expect a fix for this?

This will be fixed in the next release in the following weeks.


Validation build unit tests failure since agent ver 44.0.0 (20221027.1)

Breaking changes
suffering unit tests failures related to the recent agent image ver update in 1es pool. Here is the setting for the agent under the pool:
Agent name: 'ScheduleAndExecution-Builders-CUS-Pool 1'
Agent machine name: '57050056c000000'
Current agent version: '2.213.2'
Operating System:
Microsoft Windows Server 2022
10.0.20348
Datacenter:
Runner Image
Image: windows-2022
Version: 20221120.1
Included Software: https://github.com/actions/runner-images/blob/win22/20221120.1/images/win/Windows2022-Readme.md
Image Release: https://github.com/actions/runner-images/releases/tag/win22%2F20221120.1
1ES Hosted Pool:
SKU: Standard_D8as_v4
Image: ScheduleAndExecution-Builders-CUS-Image
Image Version: 44.0.0
Interactive Mode: False
Request Id: 2693cf00-1cd8-405e-931a-7b4dbc162e16
Pool Name: ScheduleAndExecution-Builders-CUS-Pool
Network Isolation Policy: None
Current image version: '20221120.1'
Agent running as: 'cloudtest'
Everything works fine with image version 43.0.0 (20221027.1), but start from 44.0.0, our validation build unittests start failing with package assembly issue, here I what I got for the error messages:
"System.IO.FileLoadException with message "Could not load file or assembly 'Microsoft.Extensions.Logging.Abstractions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)"
We already have the Microsoft.Extensions.Logging.Abstractions ver<IP_ADDRESS> under the related .csproj, package.config and app.config, I even try to reinstall that again with nuget package management under vs2022, still got the same error, In addition, I try to update to higher ver such as <IP_ADDRESS>(the latest one), got similar error:
"System.IO.FileLoadException with message "Could not load file or assembly 'Microsoft.Extensions.Logging.Abstractions, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=adb9793829ddae60' or one of its dependencies. The located assembly's manifest definition does not match the assembly reference. (Exception from HRESULT: 0x80131040)"
Anyone can help with this?
Target date
12/05/22
The motivation for the changes
using the latest image to prevent any potential security issues such as s360 violation.
Possible impact
cannot use the validation build unit tests
Platforms affected

[X] Azure DevOps
[ ] GitHub Actions

Runner images affected

[ ] Ubuntu 18.04
[ ] Ubuntu 20.04
[ ] Ubuntu 22.04
[ ] macOS 11
[ ] macOS 12
[ ] Windows Server 2019
[X] Windows Server 2022

Mitigation ways
none

Hi @wenjzha , please provide minimal repro steps, thanks.

@wenjzha I am going to close the issue, feel free to reach us out when you have all the steps so we could help you! :)


[Issue]: 'Unable to locate package hsa-runtime-rocr4wsl-amdgpu' on win 10 wsl2 ubuntu 22.04.4 LTS ROCm 6.1 (same on 6.02) Radeon 7700 XT

Problem Description
Trying to install with
sudo amdgpu-install -y --usecase=wsl
throws error 'Unable to locate package hsa-runtime-rocr4wsl-amdgpu'
Not able to find any workaround on the net nor sources to compile.
Any help?
Operating System
Ubuntu 22.04.4 LTS on wsl
CPU
AMD Ryzen 5800X
GPU
AMD Radeon RX 7900 XT
ROCm Version
ROCm 6.0.0
ROCm Component
ROCm
Steps to Reproduce
No response
(Optional for Linux users) Output of /opt/rocm/bin/rocminfo --support
No response
Additional Information
No response

ROCm Version is 6.1 (was not selectable)
GPU is Radeon RX 7700 XT (was not selectable)

Same issue on my machine. (Windows 10 fresh install)

I think AMD is working on WSL support but it's not public yet. You either need to use Linux with direct access to the GPU, or the HIP SDK for Windows.

I think AMD is working on WSL support but it's not public yet. You either need to use Linux with direct access to the GPU, or the HIP SDK for Windows.

Well, if you
sudo amdgpu-install --list-usecase
it lists as well

wsl             (for using ROCm in a WSL context)

ROCr WSL runtime library (Ubuntu 22.04 only)
Thus, in my understanding, the amdgpu driver should support ROCm in the wsl environment out of the box.

Same issue here. That would be great if wsl is supported.

Same issue too

Same issue...

Dear AMD (@saadrahim),
A short info from your end would be highly appreciated.
Many thanks in advance!
With best regards,
Rolf

Same problem here. I would be very interested in a solution

same issue, following - keen to see rocm/wsl usecase supported (same error for both) in WSL for gpu acceleration
Operating System
Ubuntu 22.04.4 LTS on wsl
CPU
AMD Ryzen 7900
GPU
AMD Radeon RX 7900 XTX
ROCm Version
from package amdgpu-install_6.1.60100-1_all.deb
https://rocm.docs.amd.com/projects/install-on-linux/en/latest/how-to/amdgpu-install.html
ROCm Component
ROCm

@incense Currently, it does not support wsl environment out of the box. This is coming very soon! Thanks!

@incense Currently, it does not support wsl environment out of the box. This is coming very soon! Thanks!

So the guide from AMD isn't really reliable right now? https://rocm.docs.amd.com/projects/install-on-linux/en/latest/tutorial/quick-start.html
"very soon" meaning days, weeks or months? :D

same for AMD8845 780M

https://repo.radeon.com/amdgpu/6.1.3/ubuntu/pool/main/h/hsa-runtime-rocr4wsl-amdgpu/

I imagine this isn't quite released yet at the latest docs page is still 6.1.2 at the time of writing Anyone able to run amdgpu-install --usecase=wsl with the 2 packages @mirh linked installed?
Seems to me that it is trying to build a kernel module, but there is no kernel header package distributed for WSL afaik. This thread actually seems to indicate that you actually can't load kernel module in WSL2.
Just wondering how to use this. Anyone?
Below, the error I get when installing.
$> sudo apt-get install amdgpu-core
$> sudo dpkg -i ./hsa-runtime-rocr4wsl-amdgpu_1.13.0-1789577.22.04_amd64.deb
$> sudo apt-get install rocm-core
$> sudo dpkg -i ./rocminfo4wsl-amdgpu_1.13.0-1789577.22.04_amd64.deb
$> amdgpu-install --usecase=wsl
[...]
Selecting previously unselected package amdgpu-dkms-firmware.
(Reading database ... 81457 files and directories currently installed.)
Preparing to unpack .../amdgpu-dkms-firmware_1%3a<IP_ADDRESS>102-1781449.22.04_all.deb ...
Unpacking amdgpu-dkms-firmware (1:<IP_ADDRESS>102-1781449.22.04) ...
Selecting previously unselected package amdgpu-dkms.
Preparing to unpack .../amdgpu-dkms_1%3a<IP_ADDRESS>102-1781449.22.04_all.deb ...
Unpacking amdgpu-dkms (1:<IP_ADDRESS>102-1781449.22.04) ...
Setting up amdgpu-dkms-firmware (1:<IP_ADDRESS>102-1781449.22.04) ...
Setting up amdgpu-dkms (1:<IP_ADDRESS>102-1781449.22.04) ...
Loading new amdgpu-6.7.0-1781449.22.04 DKMS files...
Building for <IP_ADDRESS>-microsoft-standard-WSL2
Building for architecture x86_64
Module build for kernel <IP_ADDRESS>-microsoft-standard-WSL2 was skipped since the
kernel headers for this kernel does not seem to be installed.
update-initramfs: Generating /boot/initrd.img-added
W: missing /lib/modules/added
W: Ensure all necessary drivers are built into the linux image!
depmod: ERROR: Bad version passed added
cat: /var/tmp/mkinitramfs_QlogIL/lib/modules/added/modules.builtin: No such file or directory
W: Can't find modules.builtin.modinfo (for locating built-in drivers' firmware, supported in Linux >=5.2)
I: The initramfs will attempt to resume from /dev/sdb
I: (UUID=a3a9d864-0b92-492d-89ec-63240fbcdd74)
I: Set the RESUME variable to override this.
depmod: ERROR: Bad version passed added
WARNING: amdgpu dkms failed for running kernel

You are looking at the wrong.. uh, docs sub-domain
https://rocm.docs.amd.com/projects/radeon/en/latest/docs/install/wsl/install-radeon.html
The headers problem may instead be WSL2 having updated its kernel away from 5.15 perhaps? Idk.

amdgpu-install -y --usecase=wsl,rocm --no-dkms actually worked. I assume the kernel issue was probably linked to the dkms install?
rocminfo shows my gpu now ;)
I actually double checked the docs and thought I looked at the right place, as that's the same domain from the readme. Might want to add a link to the WSL setup on here https://rocm.docs.amd.com/projects/install-on-linux/en/latest/how-to/native-install/ubuntu.html

That's the linux guide, not wsl

Hi @incense, the beta release of WSL with ROCm is out. You can find more information, including installation steps at Install Radeon Software for WSL with ROCm.
If you encounter any issues with amdgpu-install -y --usecase=wsl,rocm --no-dkms after fulfilling the prerequisites and installing the ROCm 6.1.3 driver packages, please open a new ticket. Thanks!


EventEmitter memory leak. v.2.0.0

Try v2.0.0. but the same problem arises.
Request you to check.
I use winston v1.0.2. I met with the following error log.
Is there any solution?
my transport
transports: [
  new winston.transports.Console({
    level:debug,
    colorize: 'true'
  }),
  new (winston.transports.DailyRotateFile)({
    datePattern: '.yyyy-MM-dd',
    filename: path.join(__dirname, '../logs/chat.error'),
    handleExceptions: true,
    prettyPrint: true,
    level: error,
    timestamp: 'true'
    //maxsize: 500000000 //byte
  })
]

error log
(node) warning: possible EventEmitter memory leak detected. 11 logged listeners added. Use emitter.setMaxListeners() to increase limit.
Trace
    at addListener (events.js:239:17)
    at once (events.js:265:8)
    at Transport.logException (c:\Users\Coupang\WebstormProjects\livechat-agent\node_modules\winston\lib\winston\transports\transport.js:132:8)
    at logAndWait (c:\Users\Coupang\WebstormProjects\livechat-agent\node_modules\winston\lib\winston\logger.js:646:15)
    at c:\Users\Coupang\WebstormProjects\livechat-agent\node_modules\async\lib\async.js:157:13
    at _each (c:\Users\Coupang\WebstormProjects\livechat-agent\node_modules\async\lib\async.js:57:9)
    at Object.async.each (c:\Users\Coupang\WebstormProjects\livechat-agent\node_modules\async\lib\async.js:156:9)
    at Logger._uncaughtException (c:\Users\Coupang\WebstormProjects\livechat-agent\node_modules\winston\lib\winston\logger.js:669:9)
    at emitOne (events.js:77:13)
    at process.emit (events.js:169:7)

Please provide the exact code you are running against<EMAIL_ADDRESS>It is very suspicious that the above code sample is exactly what you are running since winston.transports.DailyRotateFile was moved to winston-daily-rotate-file in 2.0.0
This was a known issue in 1.x branch that we fixed. Please confirm (with full reproducible code sample) that you are using<EMAIL_ADDRESS>Even if this is still occuring it's really not a big issue. You can just run:
logger.setMaxListeners(0);

and that will definitely fix it.

So I was updating bithavoc/express-winston to work with<EMAIL_ADDRESS>and this issue came up again. I dont know if setting the max to infinity is a good idea to get around this


add support for hpu devices

This PR adds support for model evaluation on HPU devices using HF optimum habana interface

@loubnabnl could you kindly help review this PR, or help assign someone?


Encode how to map provables to normal values

bindings: https://github.com/o1-labs/o1js-bindings/pull/213
this adds a new generic type argument and two new functions to Provable<T>:
type Provable<T, V> = {
  // ...
  toValue(t: T): V;
  fromValue(v: V): T;
}

the idea is that we can automatically convert between provable types and their normal JS equivalents, and stop leaking provable types outside actual provable code.
Motivation:

we can make many interfaces more ergonomic by accepting JS values, in an automated way
we hope to reduce confusion about provable constants, because people hopefully will never have to use them (they just use JS values), so the constant version of provable types become an internal detail

Some notes:

There is a new From<Provable<T>> type, e.g. From<typeof Field> = Field | bigint | string | number, which is automatically derived from deeply inspecting provable types and their fromValue() methods. The DX of using it is surprisingly nice -- the type is fully resolved in Intellisense

The PR uses fromValue for making the return value of the witness() callback accept JS values. This is super nice IMO and hints at what we could do with this going forward. For example, we could have a wrapper for all gadgets / provable methods which takes both a provable implementation (as before) and a constant implementation which receives js values, and the result takes the flexible From<> type

To make the example above work, I reimplemented CircuitString with Struct and a JS type of string. It also works with CircuitValue but I wanted to have an example for how to override the JS value type for Structs, it's a bit more involved since Struct wants to infer everything for you (while CircuitValue is somewhat weakly typed)

For backwards compatibility, the V type parameter is optional in the exported Provable, so you can still write Provable<T> in situations where the JS value doesn't matter

TODO: forgot to add changelog for this


Get partials.py and partials.c up to date with latest AMY, including Loris decomposition playback

Fixes for #140 and #9

thanks @dpwe !!!


KAFKA-8705: Remove parent node after leaving loop to prevent NPE

Fixes case where multiple children merged from a key-changing node causes an NPE.
I've updated the tests.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

ping @guozhangwang, @mjsax, @vvcephei, @ableegoldman, @cadonna, @abbccdda

Marking that we have already got 3/3 green check. Future flaky tests shall be ignored for comment-only changes.

Thanks for the PR. Have we verified that the NPE will recur without this fix in unit test?

Yes the test fails without the fix.

ping @mjsax

@abbccdda Call for second review.

Merged #7117 into trunk

will cherry-pick to 2.4 once next release is out

cherry-picked to 2.3 and 2.2

cherry-picked to 2.4


youTube logo is not showing is this legal

I need to know using this package in production for youtube video steaming is legal and, or any other issues?

I'm not sure

hello, do you have answers about this question?


CRD.version is no longer generated at master

Bug Report

What did you do?
A clear and concise description of the steps you took (or insert a code snippet).
I ran the following code with the current master of opertor-framework.
#! /bin/bash
# Channel will be used in OLM to reference versions
export CHANNEL=alpha
# the csv version
export CSVVER=0.0.1
# your operators name will be used
export OPERATOR_NAME=memcachedoperator
# deploy dir for gen csv cmd
export DEPLOY_DIR=deploy
# the location of the CRDs for gen csv
export CRD_DIR=deploy/crds
# the location of the uniuq TYPE for this Operator for gen csv
export API_DIR=pkg/apis

operator-sdk version;
operator-sdk new ${OPERATOR_NAME} --repo=github.com/example-inc/${OPERATOR_NAME};
cd ${OPERATOR_NAME};
operator-sdk add api --api-version=cache.example.com/v1alpha1 --kind=Memcached;
operator-sdk add controller --api-version=cache.example.com/v1alpha1 --kind=Memcached;

operator-sdk generate k8s;
operator-sdk generate crds;

What did you expect to see?
A clear and concise description of what you expected to happen (or insert a code snippet).
I expected to see the spec.version field of the CRD filled with v1alpha1
What did you see instead? Under which circumstances?
A clear and concise description of what you expected to happen (or insert a code snippet).
The CRD was produced without the spec.version field.

Go

Possible Solution
Revert changes in master that broke this in the v0.17.0
Additional context
The spec.version is properly generated in the CRD

Hi @heoneal,
From now, the tool is generating CRDs with the version v1 instead of apiextensions.k8s.io/v1beta1.See the Cnagelog

The --crd-version flag was added to the new, add api, add crd, and generate crds commands so that users can opt-in to v1 CRDs. (#2684)

The spec.version was deprecated and remove in the v1. More info: CRD k8s doc. Now it is versions and is a list. See:
...
spec:
  # list of versions supported by this CustomResourceDefinition
  versions:
    - name: v1
    ...

If you wish still using the previous version of the k8s api, where the version field exists you can use the flag --crd-version. If you run the  command with the flag --help you are able to see:

--crd-version string          CRD version to generate (default "v1")

However, note that the previous version of CRD (apiextensions.k8s.io/v1beta1) is deprecated and will no longer be supported after 1.19 k8s version.
Closing it as sorted out. However, please feel free to re-open if you see that it would be required.

TL;DR: use operator generate crds --crd-version=v1beta1
Here's the PR where this change was made, for reference: #2874

Thank you for the detail @camilamacedo86 I had missed the changelog and that makes a lot more sense.


Consider using the std::iter::Sum/Product traits

Instead of the current SumOp/ProductOp, we could use the same traits as Iterator, potentially enabling use with more than just the primitive types.
// Iterator:
fn sum<S>(self) -> S
    where S: Sum<Self::Item>

// ParallelIterator now:
fn sum(self) -> Self::Item
    where SumOp: ReduceOp<Self::Item>

// perhaps use Sum for folding, Add for reducing
fn sum(self) -> Self::Item
    where Self::Item: Sum + Add<Output=Self::Item>

// or even this, allowing sums by reference
// (but this is sometimes bad for type inference)
fn sum<S>(self) -> S
    where S: Sum<Self::Item> + Add<Output=S>

This would be a breaking change, so we should decide before 1.0.

Hmm, I have to think about this. I designed the current trait setup pretty specifically with an eye for eventually optimizing them with SIMD and so forth using specialization -- and because we want to assume that the addition operation is associative, which the stdlib traits probably don't require. (It also happens to map pretty nicely to what I've seen in other projects (i.e., Cilk), I think, though I'd have to dig up a citation). But I agree it'd be nice to not have it.

I figured it was just because Sum and Product were stabilized relatively recently, only in 1.12.  And hey! That's our current baseline! :)
Couldn't they still be specialized behind the scenes?  And the associativity can be a matter of documentation, just as we do for reduce.  In fact, sum and product already mention the associative requirements too.

@cuviper

I figured it was just because Sum and Product were stabilized relatively recently, only in 1.12.

Well, that might also be true -- I just know that those other concerns (associativity, etc) were on my mind, to be honest I don't remember the state of libstd at the time =)


integrate native chart display incorrectly

I try to integrate this smart lib to my react native project, but when I moving in chart view, it's messy immediately like below.
details of my question live on stackoverflow with relevant code, I really want to figure out the reason which cause this, maybe some suggestion will also help me.
thanks for your time.
regards.

be solved here

This happens when your view is opaque=true and backgroundColor=UIColor.clearColor().
If your view is opaque you must supply an opaque color. Otherwise- set it to opaque=true.


Info Icon

Change Appearance of Info Icon:
-Needs to be larger, and blue instead of black
-Needs to be located next to search result (in same line instead of above)

When moused over:
-Pop-up should be white and opaque, instead of black and semi-transparent
-Pop-up should be shorter in height and wider in length
-Text should be black instead of white
-Text should start at "Action word...", take away search result at beginning of this

Maybe the "moused over" part of this should be changed into it's own issue - I see that this is also the case when other items are moused over

@Eluviian Could you add a new entry for the mouse over differences like you mention above? Thanks!

Moving the "moused over" part to a new issue. Everything else was resolved in PR #26


email adapter error

Hello
please find attached an error screenshot after i added an email adapter to allow user to reset password ? kindly advice

Regards

Are you sure you are running parse-server v2.2.4? parse-server-simple-mailgun-adapter dependancy is provided along with the latest parse-server release.
In your packages.json file make sure its: "parse-server": "2.2.4"

okay i will try it now .

If that doesn't work, install with npm install --save parse-server-simple-mailgun-adapter

Hello
after i updated my parse server to be 2.2.4 everything work in server but when i try to reset my password by enter my email , am not receive any emails in my inbox .
please advice .
Regards

okay now am received the email in my inbox but the link not clickable and the link start with this word "undefined" ? can you help me please ?

did you set publicServerURL in your options?

no

can you give me an example please ?

then that's the issue, this value should be set as a publicly accessible URL to your parse server

How to set the vaule @flovilmart ?

it's in the documentation: https://github.com/ParsePlatform/parse-server#email-verification-and-password-reset
Before reporting issues, can you please make sure that you configure your system according to the documentation?

thank you very much i will try it now .

Please only use the issues on github to report real issues, not configuration errors.
Thanks

thank you very much but when i clicked the link this error appear
"The requested document was not found on this server."

what did you set as publicServerURL?
if serverURL is http://localhost:1337**/parse** then publicServerURL should be http://example.com/parse

i do exactly as you said before publicServerURL  http://example.com/parse but when click the link error 404 appear when try to reset password ?

@thamer898 by example.com he meant the public facing URL of you app and not literally example.com. For instance if its hosted on Heroku it will be something like: http://yourappname.herokuapp.com/parse
For testing locally use something like ngrok, then your public facing URL will be something like https://xxxxxx.ngrok.io/parse/

i have a domain name but when click the link this error appear as you see in the picture .

What is the URL?

localhost

localhost is not a URL!
What is the link you receive that is not working?

sorry i have this domain name privatchat.chat i think the page not found on the server ?

@thamer898 if you want us to help, you need to answer the question, what is the link that you receive in the email?

this link https://privatechat.chat/parse//apps/1234@tamer/request_password_reset?token=8hGJeShGPPZde0RLntIfyEyRY&username=thamer

your publicServerURL should not have a trailing /, it should be  https://privatechat.chat/parse
second, it doesn't seem that the parse-server is deployed on that server as I'm unable to query the default classes and get the unauthorized error.

but i can do all the operations like insert and delete images , sign in / up users !

Are you running parse-server locally or is it deployed on a server? If locally then follow the steps I gave above to configure ngrok, use it as publicServerURL  and it will work.

i deployed parse server on a windows server 2012 , but i think the reset password html page not found so i face the 404 error ? is it right .

You are facing the 404 because parse-server was not found on that URL. I'm not sure how windows server works, but if it has a static IP address you could use that too something like http://xx.xx.xx.xx:port/parse
I think its better you ask this and any further configuration related questions over at Stackoverflow

okay i have ssl connection how to allow connection using https i think this the error since am using http only ?

@thamer898 this issue is getting out of hands now, this is a deployment and configuration problem on your side, and not related at all with an issue on parse-server.
I'm locking that conversation for now as this repository is not a live support for deploying parse server.
From what I can see, your server is not correctly deployed to the domain you pointed to. As @tanmays mentions, you may find a better answer on stack overflow.

okay thank you do you have an account on stackoverflow ?

yes I do, but I don't answer parse-server deployment related questions.

okay thank you very much


Consistent case

Is it geoparquet or GeoParquet?

I think Apache Parquet itself consistently uses Parquet with capital P, so we could go with "GeoParquet" then?


High CPU of Windows Driver Foundation after a while in SXS mode

[x] I have searched open and closed issues for duplicates
[x] I am submitting a bug report for existing functionality that does not work as intended
[x] I have read https://vigem.org/Community-Support/
[x] This isn't a feature request or a discussion topic

Bug description
I am using DsHidMini for a Dualshock 3 controller. After a while the CPU usage of Windows Driver Foundation - User-mode Driver Framework Host Process starts rising and rising. This leads to PCSX2 getting slower and slower. After reconnecting the controller the CPU usage is down again and everything runs smoothly. I have already reinstalled the drivers, did not work.
Screenshots

Machine info

CPU-Architecture: x64
Windows version: Windows 10.0.19043 Build 19043
Software/driver version(s): 2.2.282.0
Any other helpful information
DSHMC20221113.txt

I am aware that reproducing this error might be difficult. I also don't have any "proof" that DsHidMini is 100% at fault here, but reconnecting the controller fixes this issue again for a while, therefore it seems obvious to me.

Are you using the XI proxy DLL?

I have the same issue, after a while the WDF process got to 10 or more percent CPU usage. I'm using the x64 XInput .dll for PCSX2.

I'm getting the same issue using the proxy DLL.

To those affected feel free to test it with the complete overhaul of the DLL, it works with the old driver just fine, no need for any other upgrades.

I can confirm that the issue seems to be resolved. I haven't noticed a VRAM or CPU ramp-up after playing for ten minutes like before.
Thanks so much for addressing this!

Pretty much a complete rewrite, yeah. More sophisticated and thought through. You can see all the work in the finished pull requests and the commit history.


Prepared Statement with IN operator

Is it possible to bind multiple values to SELECT ... IN (?). Have seen (maybe) that Java cassandra driver have support for this.
When I provide values ids: Vec<Uuid> in form (&ids,) then i get marshaling error: Validation failed for uuid - got 84 bytes. When use &ids directly then I get error Invalid amount of bind variables: expected 1 received 4

Try dropping the parentheses around the question mark:
SELECT ... IN ?

That was embarrassing case of "I would never think of it". Its working, thanks.


Allow generation to be specified in DownloadObject

This changes the behavior of DownloadObject(Object) to always download the latest version rather than the version in the object.
Basically, only the name and bucket of the argument is used now. (This is consistent with DeleteObject and GetObject.)

Pushed the next set of changes - PTAL.

Thank you for doing the change, looks much better now. :shipit:


Fix coalesce/coalesceDefault for sub-queries

Fixes issue #90
Please note that I'm not experienced with the internals of Esqueleto. However, when I encountered the issue I felt I could give it a shot. Please let me know if this is an acceptable solution.

Sorry for taking so long to get to your PR, it looks great!  Merged and released as esqueleto-2.1.3.

No worries, thanks for merging.


write /var/lib/docker/tmp/GetImageBlob534349380: no space left on device

Expected behavior
Actual behavior
Information about the Issue
Steps to reproduce the behavior

...
...

I saw the proposed fix : https://github.com/docker/kitematic/wiki/Common-Issues-and-Fixes#running-out-of-space


urdfdom* packages in Indigo/Trusty are in conflict with system ones

The full story can be read at https://bitbucket.org/osrf/sdformat/issue/63/gzsdf-segfaults-in-trusty.
The short story, in our efforts to move upstream package to official Ubuntu repositories, the first version already using them is Trusty. Indigo is still shipping ros-indigo-urdfdom*, which I believe should not be needed. Unfortunately, they have the same name but are not ABI compatible, so people with software compiled against system urdfdom libraries, will find a problem if source ROS setup.bash and try to run binary.
IMHO, the way to go would be try to stop using ros-indigo-urdfdom* packages and use the system ones. Not sure about the implications of this for ROS (I imagine that a full recompile of all packages depending on urdfdom is needed. I'm also not sure if a simple change in rosdep can make this transition easy) but please, feel free to propose any other solution you find easier (try to mimic exactly the urdfdom system version in ros-indigio-urdfdom could also work).
As a side note, sorry, I should have detected this problem in the beta phase. I will implement a testing mechanism for this kind of ABI incompatibilities in the OSRF jenkins.

I'm having an issue with srdfdom wanting urdfdom_py as a ros dependency in debian jessie.
A little confused by how I'd approach doing it as a source install with the system dep mixed in (seeing the code is inside urdfdom repo)
Should urdfdom_py also be released as a separate system dep as part of this change?

@mryellow this looks like a new topic. You should ask a question on http://answers.gazebosim.org

No problem, though it might be part of this whole deployment change....
Seems there is no way to get urdfdom_py without messing up these system deps, taking liburdfdom-dev back to source and hacking some kind of urdfdom_py proxy package. Which isn't ideal by any means.


Restore last scene from previous session

Is there any way to restore the last scene from previous session based on cached path in local storage (eg. redux-persist)?
Currently it has to be done in ComponentWillUpdate of the initial scene, which will cause the initial scene to be displayed before it gets transited to the last accessed scene.
It cannot be done in ComponentWillMount as the state from local storage only gets hydrated after ComponentWillMount has already been executed.

@James2516 making sure I understand you correctly. Are you asking how you could reload the previous routing "history" when the app initiates?

Yes. Alternatively, being able to restore just the last scene might suffice.
Eg. the app has initial scene at Scene 1, which goes to Scene 2 with a button press, If the user goes to Scene 2 then quit the app, the app should display Scene 2 directly (skipping Scene 1) when the user opens the app again.

I wonder if this might be possible using deep links combined with an AsyncStorage call to change your location on reload. It's not as good as the native approach, but it miiiight work?

is it even possible?

Not supported

@James2516 imho it's not an entirely bad look to load the first view and then push the next one. You can make it look more official with an overlay. Just my 2 cents that it won't detract the user experience.


Get error: Uncaught Missing source element or string.

I have created a minimalist html page and get the error: "Uncaught Missing source element or string." I don´t know, what I´ve done wrong... Please help! Thanks!

TEST

Hi @ChristophClaus, everything looks good except your script tag is before your div, which means that when the script runs the div doesn't exist yet! Put the script at the very end of your body, or use window.onload to delay execution of your script until the whole page is loaded.

Perfect! Thank you!


Running tkey-verification against an incorrect device gives a 404-error

What happened:
I was trying to verify an old prototype device, with an incorrect USB VID/PID. I used the --port option to force communication to the device.
tkey-verification --port /dev/ttyACM0 verify
TKey UDI: 0x0001020304050607(BE) VendorID: 0x0010 ProductID: 8 ProductRev: 3
HTTP GET status code: 404 (404 Not Found)
I/O FAILED: I/O error

Expected result:
I expected a message saying that this wasn't a genuine TKey

I agree, it would be better to filter out the 404 Not Found error and treat it in a better way.
Maybe it should be even more verbose, "Not genuine. You are trying to verify a TKey with an UDI not provisioned by Tillitis".


Not Working with Ajax Catalog Product View

I am getting below error :
TypeError: prevConfig.config is undefined

same as here: #159


Make rows in contract table actual links to allow opening in new tab with cmd click

Problem solved
Short description of the bug fixed or feature added

PR-Codex overview
This PR focuses on enhancing the DeployedContractsPageHeader and related components by improving the link handling for contract names and refining the UI elements in the contract table.
Detailed summary

Updated DeployedContractsPageHeader to use a new Button import.
Added linkOverlay prop to AsyncContractNameCell.
Enhanced link styling in AsyncContractNameCell with conditional classes.
Passed linkOverlay prop when rendering AsyncContractNameCell in the contract table.
Simplified event handling in ContractTableRow.

✨ Ask PR-Codex anything about this PR by commenting with /codex {your question}

#5142  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @MananTank and the rest of your teammates on  Graphite


Add CSS class based on provider name to SAML delegated login links

This will allow a deployer to style the link in login-providers.html (e.g. to add an organizational logo) using a CSS class based on the provider's name (e.g. "This Organization (Cool People)" would result in a CSS class called "This-Organization--Cool-People-").
Deployers will need to provide their own CSS to leverage this, either by modifiying cas.css in the overlay, or updating templates/layout.html to include a different CSS file.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

@rrenomeron I see you have made a number of changes here. Would you please merge with master once so we can revisit this properly?

Going to close this for now, due to inactivity. At any time if you wanted to resume work on this, please post back and we'd be happy to reopen and continue.

So I've rebased to a relatively recent commit and pre-compiled the search-and-replace pattern to address the concerns about performance.  I'd like to re-open, please, esp if there's a chance this could get into 5.2.0-GA.

Of course. Thank you for the follow-up.

Also, do you mind if I ask you to squash commits as well? I normally don't care, but given this dates back to a few months ago and is likely going to increment further, it would be good to consolidate.

Ugh, what a mess.  I think I just might start fresh with a new branch, with the size of the actual changes I think it will be less work than fighting with git (which I've already done too much of).  Will doing it this way require a new PR?

Yes it's likely much easier to close this one and start anew.

Going to close this for now.


How train Mario game with OpenAI gym in the cloud， and download this agent， and run this agent on myself computer ？

This script requires an SNES emulator. I don't think there is any way to combine it with gym, sorry.


Improvements for AdminPanel on mobile

I have made the following changes to primarily improve readability of forms and buttons on mobile devices.

Reduce text in warning by encouraging backups
Separate divs for forms with buttons so text is readable on mobile

With the following bugfix, should you want to simply fix the active nav tab:

Initialize "active" tab properly for nav-tabs (nav-link vs nav-item)

@robiso This PR is a striking example of single huge lines creating unreadable diffs. Here, it is very difficult to see what has been actually changed! A 1341 characters line is waaaaaay too long! And it makes modifying it a real PITA.
I hope one day we can move the html out of the PHP code in templates ;)
Note: it shouldn't cause any issue to split the lines here!

@notcarbide, the first change (active tab on setting) kind of breaks the current functionality if the user is using Bootstrap 3 (which will be still the default one used in the WonderCMS 3.0.0 version.
Example for this behaviour:

What I've made in my previous pull request is, a functional active tab experience. If the user uses Bootstrap 4, he will run into this issue, but at least it wont be broken by default.
I suggest we leave it as it was, until we fully migrate to Bootstrap 4 in the future. So for now, let's stay with Bootstrap 3 and some Bootstrap 4 compatibility.
The other mobile changes also introduce a couple of issues when using Bootstrap 3, as seen in the screenshots below:

@NicolasCARPi, you are correct and I couldn't agree more, this was a bit frustrating, to say the least. :)

Note: it shouldn't cause any issue to split the lines here!

Do you mean split the HTML lines into readable code? (Ala prettify?)

Do you mean split the HTML lines into readable code? (Ala prettify?)

Yes, it won't change a thing!

@robiso gotcha! Gonna close this pending prettifying things, I'll see what I can do to get a fix for mobile that won't break older bootstrap afterward.

@NicolasCARPi, will do this in the following pull request. I've nearly finished the screenshots for themes and plugins.
@notcarbide, thank you!


$updateSourceControl blocks for 500ms

Have the git viewlet open
Have ~30-40 modified files
Have auto-save on
Type and wait for auto-save to kick in

Profile-20170816T121756.json.zip

Fixed!

Nice!


Writing to disk earlier when pre-rendering to reduce memory usage

Description
Similarly to what was described in #134, I found myself hitting memory limits on my CI runner when trying to build a rather large website (it's a blog with over 20 years of content).
While I can try to reduce the size of pageContext, this only buys me a small amount of time until I hit the same limit again.
The problem seems to stem from these lines of code:
https://github.com/brillout/vite-plugin-ssr/blob/96d9d892b752ebd50347ccece0b7ba25fddf0d2f/vite-plugin-ssr/node/prerender/runPrerender.ts#L175-L200
Since all the pages stay resident in memory until they are written to disk, the memory consumption skyrockets.
If it were possible to write the html files to disk earlier (either by chunking them or writing them as they are pre-rendered) this would dramatically cut down on memory usage.

Up for a PR? Could we make Node.js garbage collect pages already written to disk?

Seems like we are running into a similar issue but without pre-rendering.

Building the production site of https://github.com/inlang/inlang/tree/main/source-code/website requires us to deploy the site with a 2GB instance to handle the short peek.

@samuelstroschein It's probably unrelated to OP; I'd open another ticket/discussion for it. What makes you believe it's related to VPS and not to user code?

@brillout

What makes you believe it's related to VPS and not to user code?

That this issue exists aka I don't know whether this is VPS related or not.

it's probably unrelated to OP; I'd open another ticket/discussion for it.

I don't want to flood your repo with new issues that might be related to VPS
We are fine with paying for a larger machine for now. Investigating the issue is not a priority for us. Hence, I leave our comment as is. Maybe someone else finds it, indicating that our problem seems VPS related

@samuelstroschein OP is about memory usage while pre-rendering, whereas your issue is about the SSR runtime. So, yea, I'd say the disucssion should live in another ticket.

I don't want to flood your repo with new issues that might be related to VPS

No problems. We can close the ticket until someone finds a reproduction while also gathering further insights. It's always good to spark the conversation.

paying for a larger machine for now.

To be clear: shall it be beacuse of VPS then I'll take this very seriously – VPS can and should be light on memory.

OP is about memory usage while pre-rendering, whereas your issue is about the SSR runtime. So, yea, I'd say the disucssion should live in another ticket.

Oh wait, no. The peak happens during the build step, not during runtime. The chart may be a bit misleading.

Ok, then yes it's probably related. And you're using pre-rendering, correct? We can explore how to reduce memory usage, let me know when it becomes urgent.
Also, make sure to check out the parallel option. Both @d3x7r0 and @samuelstroschein.

We are not using pre-rendering.

let me know when it becomes urgent.

Will do. For now, we just pay $15 a month more for a 2GB machine

Ok, then yes it's probably related. And you're using pre-rendering, correct? We can explore how to reduce memory usage, let me know when it becomes urgent.
Also, make sure to check out the parallel option. Both @d3x7r0 and @samuelstroschein.

Sorry for not replying. I've been super busy with work and haven't had time to sit down and look at this since I'm just using it in a personal side project.
Sadly the parallel option has no noticeable effect on memory usage for my scenario (SSR a large number of pages).
My hypothesis is, again, related to the code I quoted initially. You can see that during the prerender process all the pages are first processed, html is generate and stored in a js variable (in memory) and only on the subsequent step are they written to disk.
https://github.com/brillout/vite-plugin-ssr/blob/96d9d892b752ebd50347ccece0b7ba25fddf0d2f/vite-plugin-ssr/node/prerender/runPrerender.ts#L175
This variable will, eventually, contain the entire generate website and nothing will be flushed to disk before all pages are processed. This is, obviously, faster than doing i/o but it comes at a cost of increased memory consumption.
My quick look at it also tells me that it might not be a simple one afternoon fix, hence why I didn't tackle it immediately. I can try and take a look at it but I don't know when I'll have the time. It would mean a rather large rework of that bit of the pre-render function. Either to operate in chunks, in parallel as a whole, or both.

@d3x7r0 Makes sense. If you wouldn't mind sharing your project's code with me, then I'd have a look at it (or a reproduction if you prefer). That said, I've a couple of urgent things to work on right now (the V1, and a production ready VPS framework demo) so this may take a little while until I do. But feel free to let me know if this is/becomes a major blocker and you can't find a workaround in user land.
Yes, @samuelstroschein's issue is probably unrelated and more likely to be related to Vite.

I'll try to get a repro going this weekend. Should be enough to just generate an increasing number of random pages until you see memory usage baloon. Though you might need to generate random data for the memory to really fill up.

That'd be great.

Btw this isn't a particularly huge blocker for me. I'm just using it for a personal site, not mission critical stuff. And I can still do it on my local machine, I just can't really use ci cause it would be too costly :)

👍. Although having a reproduction is always great and much welcome.

Also check out https://vite-plugin-ssr.com/disableAutoFullBuild to break down the mem spike into two/three smaller spikes.

Ok, got a repro:
https://github.com/d3x7r0/vite-ssr-memory-issue
Just adjust the number of entries in pages/page/index.page.server.js to increase/decrease memory usage.
On my machine (macbook pro intel 2020, 16gb ram, node v18.12.1) I get the build crashing at around 4gb or ram used by the node process.
I know 25k URLs is a lot, I specifically overshoot to trigger the issue. I haven't made the math to see how many I have on my project but I wouldn't be surprised if it was around the 10k mark. It's a long running blog with over 10 years of content. Just in tag pages alone it's easy to reach those high numbers.

Thanks, that's very helpful to have (rough) numbers.
Incrementally pre-rendering should do the trick, I'll have a stab at it when I've little less on my plate.
As always: if this becomes a blocker for you (future users), definitely let me know.

@d3x7r0 Did you find a workaround?
As always: Let me know this is a blocker and/or PR welcome.

Hi there. Sorry, I haven't been able to come back to this. No workaround from my end aside from tweaking node to use more memory. I did want to try and tackle this one but haven't found the time and, realistically, probably won't for the next month or so :-/

No problems. FYI an alternative, in case you think you can convince your company, is sponsoring (https://github.com/sponsors/brillout) as I considerably increase feature priority for my sponsors.
But still let me know if this becomes a blocker.

Done in #1262 by @nitedani.


added test that shows issue with absolute path

This is a test that shows a possible issue getting absolute path, when iterating over a parent object.
When iterating the components in a nested parent object, the path of the children will be relative to the nested parent, not to the root of the form.
Breaking Changes / Backwards Compatibility
Use this section to describe any potentially breaking changes this PR introduces or any effects this PR might have on backwards compatibility
Dependencies
Use this section to list any dependent changes/PRs in other Form.io modules
How has this PR been tested?
Use this section to describe how you tested your changes; if you haven't included automated tests, justify your reasoning
Checklist:

[ ] I have completed the above PR template
[ ] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation (if applicable)
[ ] My changes generate no new warnings
[ ] My changes include tests that prove my fix is effective (or that my feature works as intended)
[ ] New and existing unit/integration tests pass locally with my changes
[ ] Any dependent changes have corresponding PRs that are listed above

I just added this test to my PR @ https://github.com/formio/core/pull/193 to ensure it works correctly.


Fix example CRs for Account and FederatedAccountAccess

This PR takes out specific user information from the AWS Federated Account Access example CR, as well as fills out a basic spec for the Account example CR.

/lgtm

/retest


fix(plugins): use batch queue in datadog & statsd plugin to reduce timer usage

Summary
This PR adds batch queue into the Datadog plugin and the StatsD plugin, to reduce the number of timers used by these two plugins.
Whenever a request is processed, during the log phase a new instantly running timer will be created. This may cause a shortage of timers under heavy traffic and lead to unpredictable consequences (internal timers are killed randomly and some of them cannot recover automatically).
Adding batch queues in these two logging plugins will mitigate this problem to some extent.
Full changelog

Add batch queue into the Datadog and StatsD plugin to reduce timer usage

Issue reference
Mitigate FTI-4367

It seems hard to add test cases to this fix, but exists test cases should get passed without any problem.

The right commit type for this change is 'perf' as per our contribution policy: https://github.com/Kong/kong/blob/master/CONTRIBUTING.md#type.
Please update your PR and commit accordingly.

PR type changed to perf

@chronolaw please review again, thanks!
Also @bungle could you please take a look at this PR?

Hi, @bungle do you need the performance test job running? It seems that it requires some cloud provider credentials to be properly configured, and in my fork repo, these configs do not exist. Maybe I need to push this branch to kong:kong and open a new PR so that we can get the job running

@windmgc can you rebase so that we can merge this, please?

@windmgc can you rebase so that we can merge this, please?

Rebase done.

@windmgc please add a changelog entry for this. Nice work!


Fix cache to restore cache from correct arch

Only use restore keys from the same architecture.

@bilelmoussaoui could you please take a look as this is a critical bug breaking multi-arch builds!?

Added to https://github.com/flathub-infra/flatpak-github-actions/commit/a21be69628f0aec32f975e3e8399cf0db7d80e3b


Unable to make

Having unzipped the zip file, running the make file gives the following error, Please help:
src/CommodityTracking.cpp:16:5: error: ‘vector’ does not name a type      vector<Skeleton*> SkeletonTracker::getSkeletons() {      ^ src/CommodityTracking.cpp: In function ‘cv::Mat ct::simplifyUserMask(cv::Mat&, cv::Mat&, int)’: src/CommodityTracking.cpp:132:9: error: ‘vector’ was not declared in this scope          vector<vector<Point> > contours;          ^ src/CommodityTracking.cpp:132:9: note: suggested alternative: In file included from /usr/include/c++/5/vector:64:0,                  from /usr/include/c++/5/bits/random.h:34,                  from /usr/include/c++/5/random:49,                  from /usr/include/c++/5/bits/stl_algo.h:66,                  from /usr/include/c++/5/algorithm:62,                  from /usr/local/include/opencv2/core/base.hpp:55,                  from /usr/local/include/opencv2/core.hpp:54,                  from /usr/local/include/opencv2/opencv.hpp:52,                  from src/CommodityTracking.h:4,                  from src/CommodityTracking.cpp:1: /usr/include/c++/5/bits/stl_vector.h:214:11: note:   ‘std::vector’      class vector : protected _Vector_base<_Tp, _Alloc>            ^ src/CommodityTracking.cpp:132:28: error: expected primary-expression before ‘>’ token          vector<vector<Point> > contours;                             ^ src/CommodityTracking.cpp:132:30: error: expected primary-expression before ‘>’ token          vector<vector<Point> > contours;                               ^ src/CommodityTracking.cpp:132:32: error: ‘contours’ was not declared in this scope          vector<vector<Point> > contours;                                 ^ src/CommodityTracking.cpp:133:21: error: expected primary-expression before ‘>’ token          vector<Vec4i> hierarchy;                      ^ src/CommodityTracking.cpp:133:23: error: ‘hierarchy’ was not declared in this scope          vector<Vec4i> hierarchy;                        ^ src/CommodityTracking.cpp: In function ‘std::vector<cv::Point_<int> > ct::getEdgePoints(cv::Mat, cv::Mat, int, bool, std::vector<std::vector<cv::Point_<int> > >&)’: src/CommodityTracking.cpp:228:9: error: ‘vector’ was not declared in this scope          vector<vector<Point> > contours;

This looks like a compiler bug. That said, CommodityTracking is
deprecated. If you're interested in my experiments with monocular human
pose estimation, https://github.com/bobbybee/libskeletal will likely be
of more interest.


recolor: support alpha transparency with fill-opacity

Turns out SVG uses RGB, not RGBA, so if we want to do alpha transparency we need to add in fill-opacity. That means something like:
GameIcons.get('glass-heart').recolor(alpha: 0.5)

Done!


⚠ GazelleGames has degraded performance

In 6ece50f, GazelleGames ($GGN) experienced degraded performance:

HTTP code: 200
Response time: 1861 ms

Resolved: GazelleGames performance has improved in c31a58d after 8 minutes.


AttributeError: 'FakeInitialModel' object has no attribute 'is_sdxl'

Hi, I have successfully installed TensorRT on Forge WebUI. I am receiving the following error after trying to generate a default model in the TensorRT tab. The models I am using are SDXL models.
Traceback (most recent call last):
File "C:\Users\JS\Downloads\webui_forge_cu121_torch231\system\python\lib\site-packages\gradio\queueing.py", line 536, in process_events
response = await route_utils.call_process_api(
File "C:\Users\JS\Downloads\webui_forge_cu121_torch231\system\python\lib\site-packages\gradio\route_utils.py", line 285, in call_process_api
output = await app.get_blocks().process_api(
File "C:\Users\JS\Downloads\webui_forge_cu121_torch231\system\python\lib\site-packages\gradio\blocks.py", line 1923, in process_api
result = await self.call_function(
File "C:\Users\JS\Downloads\webui_forge_cu121_torch231\system\python\lib\site-packages\gradio\blocks.py", line 1508, in call_function
prediction = await anyio.to_thread.run_sync(  # type: ignore
File "C:\Users\JS\Downloads\webui_forge_cu121_torch231\system\python\lib\site-packages\anyio\to_thread.py", line 33, in run_sync
return await get_asynclib().run_sync_in_worker_thread(
File "C:\Users\JS\Downloads\webui_forge_cu121_torch231\system\python\lib\site-packages\anyio_backends_asyncio.py", line 877, in run_sync_in_worker_thread
return await future
File "C:\Users\JS\Downloads\webui_forge_cu121_torch231\system\python\lib\site-packages\anyio_backends_asyncio.py", line 807, in run
result = context.run(func, *args)
File "C:\Users\JS\Downloads\webui_forge_cu121_torch231\system\python\lib\site-packages\gradio\utils.py", line 818, in wrapper
response = f(*args, **kwargs)
File "C:\Users\JS\Downloads\webui_forge_cu121_torch231\webui\extensions\Stable-Diffusion-WebUI-TensorRT\ui_trt.py", line 64, in export_unet_to_trt
is_xl = shared.sd_model.is_sdxl
AttributeError: 'FakeInitialModel' object has no attribute 'is_sdxl'

I‘m having the same issue. It seems like this extension doesn't work on Forge since codes in \modules are different.
Yomisana/Stable-Diffusion-WebUI-TensorRT-Enhanced/#1

I‘m having the same issue. It seems like this extension doesn't work on Forge since codes in \modules are different. Yomisana/Stable-Diffusion-WebUI-TensorRT-Enhanced/#1

only export can't use...cause it is forge comment out the code about hijack modules, sub folder modules...etc may need ask for SDW forge endpoint want release some detail about dev docs replace modules node.


Grey screen in Pathfinder 1 scene

Im not sure what happened but since updating Foundry, I haven't been able to get Perfect Vision to work. Whenever I enter the game, everything functions normally but when a scene loads I am unable to see or interact with it. Instead its this neutral void.

Its really hard to scan the console for errors, but it only happens when I activate Perfect Vision. Is this a problem others have had and if so how did you fix it?

Updated to which Foundry version? If you updated from v8 to v9, try CTRL+F5 or instructions below if that shouldn't work. There's no PV version available for v10 yet.

Caching Issues
There is a known issue for users updating from V8 to V9 related to caching which causes some errors, UI oddities, and trouble with canvas rendering which can be addressed quickly and immediately without support assistance. If a simple CTRL+F5 (CMD+SHIFT+F5 for Mac) does not resolve the issue:
Steps to fix caching issues:

Open the DevTools using the F12 key (or cmd-shift-i for macOS)
Click the settings ⚙ icon
Below the heading 'network' check the checkbox for 'Disable Cache (when DevTools is open)
Leave the DevTools window open and refresh (f5) the Foundry VTT window

Note: Firefox users will see this checkbox under 'Advanced Settings' -> 'Disable HTTP Cache (when toolbox is open)'

Updated to which Foundry version? If you updated from v8 to v9, try CTRL+F5 or instructions below if that shouldn't work. There's no PV version available for v10 yet.

Caching Issues
There is a known issue for users updating from V8 to V9 related to caching which causes some errors, UI oddities, and trouble with canvas rendering which can be addressed quickly and immediately without support assistance. If a simple CTRL+F5 (CMD+SHIFT+F5 for Mac) does not resolve the issue:
Steps to fix caching issues:

Open the DevTools using the F12 key (or cmd-shift-i for macOS)
Click the settings ⚙ icon
Below the heading 'network' check the checkbox for 'Disable Cache (when DevTools is open)
Leave the DevTools window open and refresh (f5) the Foundry VTT window

Note: Firefox users will see this checkbox under 'Advanced Settings' -> 'Disable HTTP Cache (when toolbox is open)'

It doesn't appear to have solved the issue. My Foundry version is still V9

I also tried the Cache thing and it didn't make a difference. I did find the error message in console though! Not sure how much it helps.

Make sure PV is actually up to date. Maybe updating locked or stuck somehow. Uninstalling and then reinstalling PV gives you the latest version.

Make sure PV is actually up to date. Maybe updating locked or stuck somehow. Uninstalling and then reinstalling PV gives you the latest version.

Hey, Thank you! Reinstalling actually fixed it! It's nice to return to Perfect vision after Months with this problem!


FlowVR daemon invalid behaviour

In GitLab by @ksiero on Oct 9, 2019, 10:14
I don't know why, but sometimes (~1/400, ~1/950, ~1/1800, ~1/56 run) the abort signal in flowvr from one module doesn't go to the second module, but most of the time everything works.
I have written to @braffin about this problem and gave him a Docker image where it sometime happens.
I think we should disable flowvr tests until we figure out why this is happening.
Here is a part of the log from one of my test:
...
[PDI][/8ef555d808bd/7627/read:P/recv][07:35:33] *** warning: (FlowVR) Module (recv): Got `abort' abort event. Aborting...
[PDI][07:35:33] *** info: Finalization
[PDI][/8ef555d808bd/7627/read:P/recv][07:35:33] *** debug: (FlowVR) Module (recv): Close flowvr module
[PDI][/8ef555d808bd/7627/read:P/recv][07:35:33] *** debug: (FlowVR) Module (recv): Clear traces
[PDI][/8ef555d808bd/7627/read:P/recv][07:35:33] *** debug: (FlowVR) Module (recv): Clear plugins intput ports
[PDI][/8ef555d808bd/7627/read:P/recv][07:35:33] *** debug: (FlowVR) Module (recv): Clear plugins output ports
[PDI][/8ef555d808bd/7627/read:P/recv][07:35:33] *** info: (FlowVR) Module (recv): Module finalized
[PDI][/8ef555d808bd/7627/read:P/send][07:35:33] *** debug: (FlowVR) Output Mouse Payload (mouseOut): Pressed key with descriptor `middle_button'
[PDI][/8ef555d808bd/7627/read:P/send][07:35:33] *** debug: (FlowVR) Output Mouse Payload (mouseOut): Mouse position [12.3, -23.4]
[PDI][/8ef555d808bd/7627/read:P/send][07:35:33] *** debug: (FlowVR) Module (send): Calling flowvr_module->wait()
[PDI][/8ef555d808bd/7627/read:P/send][07:35:33] *** debug: (FlowVR) Module (send): `wait' returned status: 3
[PDI][/8ef555d808bd/7627/read:P/send][07:35:33] *** debug: (FlowVR) Output Mouse Payload (mouseOut): Copy mouse button state to `right_button'
[PDI][/8ef555d808bd/7627/read:P/send][07:35:33] *** debug: (FlowVR) Output Mouse Payload (mouseOut): Copy mouse button state to `middle_button'
[PDI][/8ef555d808bd/7627/read:P/send][07:35:33] *** debug: (FlowVR) Output Mouse Payload (mouseOut): Copy mouse button state to `left_button'
[PDI][/8ef555d808bd/7627/read:P/send][07:35:33] *** debug: (FlowVR) Output Mouse Payload (mouseOut): Copy mouse position to `pos_xy'
[PDI][/8ef555d808bd/7627/read:P/send][07:35:34] *** debug: (FlowVR) Output Mouse Payload (mouseOut): Pressed key with descriptor `middle_button'
[PDI][/8ef555d808bd/7627/read:P/send][07:35:34] *** debug: (FlowVR) Output Mouse Payload (mouseOut): Pressed key with descriptor `left_button'
[PDI][/8ef555d808bd/7627/read:P/send][07:35:34] *** debug: (FlowVR) Output Mouse Payload (mouseOut): Mouse position [13.53, -25.74]
[PDI][/8ef555d808bd/7627/read:P/send][07:35:34] *** debug: (FlowVR) Module (send): Calling flowvr_module->wait()
[PDI][/8ef555d808bd/7627/read:P/send][07:35:34] *** debug: (FlowVR) Module (send): `wait' returned status: 3
...

Abort section in source code:
if (!m_flowvr_module) return;
context().logger()->warn("(FlowVR) Module ({}): Got `{}' abort event. Aborting...", m_module_name, abort_event);
m_flowvr_module->abort();

Wait section in source code:
context().logger()->debug("(FlowVR) Module ({}): Calling flowvr_module->wait()", m_module_name);
int wait_status = m_flowvr_module->wait();
if (!wait_status) {
    context().logger()->warn("(FlowVR) Module ({}): `wait' returned status: {}", m_module_name, wait_status);
} else {
    context().logger()->debug("(FlowVR) Module ({}): `wait' returned status: {}", m_module_name, wait_status);
}

FlowVR plugin has been removed


Additional Information

It would be great to have additional information on how this package works and what it does differently say compared to Whisper.cpp , faster-whisper , whisperX , distal-whisper and so on.

I haven't tested other libraries.
WhisperPlus: A project with pip package support and many new features.


Secure /healthcheck and /hystrix.stream with basic authentication.

Added basicAuth to healthcheck and hystrix.stream.
I would be happy to make changes on the basic authentication. Currently this is what the basic authentication is doing:
It takes in basic auth token and converts it to username and password. It then call UAA to get bearer token along with the scopes that the user has. It then checks whether the user has the scope 'abacus'.
What troubles me:
the UAA plugins that we have requires the caller to pass in the scope. The token that will be returned by the UAA plugins would have the scopes that was passed. This means that I cannot simulate unauthorized user and authorized user. Therefore I made it so that the only user that can pass basic auth would need to have
username: process.env.USERNAME || 'abacus'
password: process.env.PASSWORD || 'HS256'
Any feedback to solve this problem would be appreciated.
Thanks!

Coverage increased (+0.0007%) to 87.715% when pulling 0d878b0e2a608951391afbbc258657b09b7a36a9 on KRuelY:basicAuth into 82aeddd5251a7aaf1b25c1eee5d66038d76f424f on cloudfoundry-incubator:master.

Coverage increased (+0.0004%) to 87.715% when pulling 271b90b4a8815c594543c82ed4fefd039a3d29cc on KRuelY:basicAuth into 82aeddd5251a7aaf1b25c1eee5d66038d76f424f on cloudfoundry-incubator:master.

@jsdelfino Hi, I made changes based on feedback. Can you take a look? Thanks for the feedback!

Hey @KRuelY would it be possible for you to combine these two commits into one? It's a bit difficult to review right now as the second commit changes code from the first commit. Also merging this pull request now, would result in the first commit (which had issues) still appearing in our commit history.
Thanks a lot!

Ok, I'll close this and submit a new PR. Thanks!