Renderer subdocuments in react-renderer

Longtime listener, first time caller.
This is an initial implementation of #239.
Test it by running the npm run test and confirming all tests pass.
WIP while I get the types sorted out.
A few questions:

@tim-evans the issue mentioned a React-aware AttributesOf. Is there a test for this currently so I can confirm I've properly implemented this? ~Haven't gotten any test / compiler warnings yet as part of what you see here.~ lol nvm, I see npm run typecheck now
This currently hardcodes React.Fragment as the Root for nested subdocuments. This feels safe, but perhaps the subdoc wrapper component should be an option to the renderer so it could be easily customized.
Do we need to concern ourselves with multi-level nested subdocuments, i.e. a document within a caption? IframeEmbed is the only annotation with a subdocument value right now, for caption of type CaptionSource. CaptionSource’s schema doesn’t include itself, so it wouldn’t have any annotations with subdocument values currently. That said, I could see this coming up in the future, but if we’re not generally handling this now, I’m inclined to skip it.

I think that making the choice for this to be a fragment is really smart; I may even suggest to make this the default behavior— it makes more sense for there not to be a wrapping root element and for the embedding context to provide one. I didn't know how to do this, so I yolo'd and made the root configurable.
As for multi-nested sub-documents, I think we're starting to see seams on this API, and it honestly feels like a hack. For now, let's continue to handle this one-level deep until we come to an impasse where this happens, then solve for that. Hopefully by then we'll have a different solution in place.
As for the AttributesOf fix for this, I think this could be implemented using some typescript magic. It's going to probably involve using type inference, which can be pretty weird. I'm happy to pair on this, since it can be a bit wacky to get right.
For reference, the implementation of this is:
// AttributesOf is this:
type AttributesOf<AnnotationClass> = AnnotationClass extends Annotation<
  infer Attributes
>
  ? Attributes
  : never;

For note, this fixes #239


[New game]: Jumbled Word Game

🎮 Game Request
Game logic and basic description
There would be some jumbled word which the player need to guess correctly .
Point down the features
Game points
Hint would be provided in each word if guess is correct the score will increase . It would also contain a timer in each word question
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

/assign

Hey @Jagpreet153 !
Thank you for raising an issue 💗
You can self assign the issue by commenting /assign in comment 😀
Make sure you follow CODE OF CONDUCT and CONTRIBUTING GUIDELINES 🚀
Don’t Forget to ⭐ our GameZone🎮
Make sure you join our Discord🕹

Hey @Jagpreet153 ! Thank you so much for your raising the issue💗
It’s all yours, you can come anytime again and make some contributions! 🚀
Alone, we can do little, but together we can do so much! 😇


Add Howto for running on TPU

For generation, there's really nothing to do unless we want to also describe how to create a gcloud instance etc.
I'll be looking into the other scripts now

Yes it would be good to have an overview of the full process, since it's opaque for most people


Variable not in scope: pathParam In examples/basic.hs file

Hello!
I am new to Haskell and Scotty. I am processing examples in basic.hs and I encountered an error (look at the pic below).

this is my cabal file, I think I am using the latest version.
Name:                scotty-examples
Version:             0.1
Synopsis:            scotty examples
Homepage:            https://github.com/scotty-web/scotty
Bug-reports:         https://github.com/scotty-web/scotty/issues
License:             BSD3
License-file:        LICENSE
Author:              Andrew Farmer<EMAIL_ADDRESS>Maintainer:          Andrew Farmer<EMAIL_ADDRESS>Copyright:           (c) 2012-Present, Andrew Farmer and the Scotty contributors
Category:            Web
Stability:           experimental
Build-type:          Simple
Cabal-version:       >= 1.10
Description:         Example programs using @scotty@
tested-with:         GHC == 8.10.7
                   , GHC == 9.0.2
                   , GHC == 9.2.8
                   , GHC == 9.4.6
                   , GHC == 9.6.2

executable CarShop
  main-is:             main.hs
  default-language:    Haskell2010
  hs-source-dirs:      src
  build-depends:       base >= 4.6 && < 5,
                       http-types,
                       mtl,
                       random,
                       scotty == 0.20.1,
                       text,
                       wai-extra
  GHC-options:         -Wall -threaded

I find it will work when using queryParam. I don't know why, but maybe it is better to fix the problem in the examples.
Finally, if you consider it as a problem, maybe I can try to make an easy-fix pull request for you.

Hi @fongfiafia , could you share the imports block in your main.hs ?

您好，来件已收到，我会尽快给您回复。

Hi @fongfiafia , could you share the imports block in your main.hs ?

sure~
{-# LANGUAGE OverloadedStrings #-}

module Main (main) where

import Web.Scotty

main :: IO ()
main = do
    scotty 8081 $
        get "/user" $ do
            v <- pathParam "uid"
            html $ mconcat ["<h1>", v, "</h1>"]


Add case/match block

I decided to create generic AST.Block and AST.SubBlock nodes to represent case/match. I think it simplifies the code a lot. And now that we have all constructs standardized with the same shape, I don't think it's necessary to have a specific node for each one of them.
I believe all other blocks (if/elseif/else, for/else, ...) could be much easier to implement using this approach as they would share the implementation in Surface.Compiler. The code in Surface.Compiler.EExEngine also gets much simpler, IMO. Here's the to_expression/3 implementation for case:
  defp to_expression(%AST.Block{name: "case"} = block, buffer, state) do
    %AST.Block{expression: case_expr, sub_blocks: sub_blocks} = block

    state = %{state | depth: state.depth + 1, context: [:case | state.context]}

    match_blocks =
      Enum.flat_map(sub_blocks, fn %AST.SubBlock{children: children, expression: expr} ->
        match_body = handle_nested_block(children, buffer, state)

        quote do
          unquote(expr) -> unquote(match_body)
        end
      end)

    quote do
      case unquote(case_expr) do
        unquote(match_blocks)
      end
    end
    |> maybe_print_expression(block)
  end

I also implemented some of the validation in Surface.Compiler.ParseTreeTranslator as it contains information about the parent block. All validations should be unified later when we start generating the AST directly in the node translator.
Quick general note: as you can see, I'm starting to avoid pattern matching everything I can in the head of the functions. If something is not required to match in order to distinguish the different function clauses, I prefer to have an extra line right below the head just to extract the other information. This makes it much easier to read the function head as well as preventing it from be broken by the formatter. However, if the function head is small enough, I don't see a problem extracting everything there.

I think my main hesitance with a common, generic AST.Blockand AST.SubBlock is that having specific nodes for the different block expressions makes it easier to know that the blocks are actually valid (more important when creating the nodes by hand, but somewhat relevant imo), and to give better names. For instance, the Surface.Compiler.EExEngine  would be very similar with an AST.Case and AST.Match

makes it easier to know that the blocks are actually valid (more important when creating the nodes by hand, but somewhat relevant imo)

We should make sure the AST is always valid when it hits the EExEngine. We actually shouldn't allow building it manually outside the compiler. The only reason we still allow that is because we haven't implemented a way build the AST injecting custom nodes, similar to what quote/unquote do. Take Elixir itself, it doesn't have a specific nodes for each statement, buy it provides a nice way to built and validate the structure. We need our equivalent quote/unquote to make sure the AST is validated by the same rules as the regular template.

A somewhat hybrid approach might be to keep AST.SubBlock around, but have specific nodes for each of the sub blocks (i.e. match, else, elseif, ...)
Any way we go, we'll likely still need separate cases for most if not all of the subblock/block pairs that are supported, and ideally most of the current code in Surface.Compiler would no longer be needed once we're generating the ast from a node translator

We can postpone this decision until we start working on generating the AST directly.


[FTL] invalid character '<' looking for beginning of value - version v1.0.5

asnmap -v

/ _ | / / |/ / _  ___ ____
/ __ |\ /    /  ' / _  / _
// |///|////_,/ .__/
//
            projectdiscovery.io

[INF] Current asnmap version v1.0.5 (latest)
[FTL] invalid cha

racter '<' looking for beginning of value

same thing with me

@dogancanbakir It's an issue with API, which is getting down randomly because of high load - https://status.projectdiscovery.io/

@dogancanbakir and @ehsandeep, I am experiencing the same issue today as well. Is that due to the high load of the asnmap api today as well?

@dogancanbakir and @ehsandeep, I am experiencing the same issue today as well. Is that due to the high load of the asnmap api today as well?

You right

Thank you for your patience. We are working on this internally and will follow up with you as soon as possible.

invalid character '<' looking for beginning of value

The API daemon outage is under investigation and hopefully will be fixed with a better lock-free implementation - Tracked at https://github.com/projectdiscovery/asnmap-server/issues/21

Hello Team,
I'm not sure if this might help you but here it is anyway.
I don't think that the invalid character '<' looking for beginning of value error doesn't randomly happen, at least in my case.
For me, it happens specifically with certain ASNs, AS15169 in my case.
I'm giving two screenshots to show when it worked and when it didn't.
Screenshot of running asnmap on AS14421 where it worked as intended

Screenshot of running asnmap on AS15169 where the error appears

These screenshots were taken simultaneously. This, along with the fact that it always happens for a specific ASN, it gives me a feeling that the error might not be because of the API going down randomly because of high load.

Any update here? Would really love to be able to use the repo. Thank you!

Hello everyone!
We found an issue with the API server due to high traffic. We're improving it to handle high usage and will deploy a fix soon. We'll update once it's done.

should be fixed by now with latest release!


Merge sort in Linked List in c++

Is your feature request related to a problem? Please describe.
2 Linked lists are given, we want combined linked list in sorted order.
TC -> O(N*log(N))
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Please assign me this issue under SSOC 2.0

Since it is an easy issue, kindly implement the algorithm in all four languages
Assigned!
@Vidip-Ghosh : C, C++, Java & Python

I would like to work on this issue, kindly assign this to me.


No handler found for the command: 'HookyQR.beautify'

I am using beautify 0.0.3 with VSCode 0.10.3 and getting
"No handler found for the command: 'HookyQR.beautify'" warning whenever trying to run Beautify. Extention isn't working as well through it works if I used js-beautify through cmdline.

This was caused by inclusion of the "extensionDependencies" with "vscode.json" element in the package file which doesn't exist in 0.10.3. Removing this dependency doesn't break the functionality in 0.10.5 since it's an internal extension.
Schema doesn't get read in 0.10.3 as vscode.json doesn't exist as an extension in that version.
0.0.4 released.


Sharing Files between Feature Creature and Implementing Application(s)

Once we import features from a repo (#8) it would be nice if Feature Creature became the canonical home for the documentation of how a product should work.
If that's the case we'll need to be able to share the features from Feature Creature to one or more repositories of implementing code.
Not sure it makes sense to have features always be in sync or sync them at regular intervals after collaboration on changes or additions.

I'm imagining a couple different kinds of changes a product owner would want to make:

changes to feature descriptions which are not intended to change existing behavior (updating domain terminology, rewording a cuke for clarity). e.g. changes which should not break a build.
changes which either change behavior or describe a feature which does not exist (driving out new work) e.g. changes which break a build or introduce undefined steps.

Assuming we're sourcing from the user's Git repo:
The first can be committed to a branch and potentially pushed and merged automatically.
The second requires development work to be done and requires a development branch.
Perhaps we can figure out UX to enable these flows.

I'm imagining a couple different kinds of changes a product owner would want to make:

changes to feature descriptions which are not intended to change existing behavior (updating domain terminology, rewording a cuke for clarity). e.g. changes which should not break a build.
changes which either change behavior or describe a feature which does not exist (driving out new work) e.g. changes which break a build or introduce undefined steps.

Assuming we're sourcing from the user's Git repo:
The first can be committed to a branch and potentially pushed and merged automatically.
The second requires development work to be done and requires a development branch.
Perhaps we can figure out UX to enable these flows.

Ideally feature-creature would be the single source of truth for features but we would also probably have to support a sync from code back to feature-creature as developers add/modify features


Das_Geek: Watch create[\W_]+several[\W_]+staging[\W_]+and[\W_]+development[\W_]+branches

Das_Geek requests the watch of the watch_keyword create[\W_]+several[\W_]+staging[\W_]+and[\W_]+development[\W_]+branches. See the MS search here and the Stack Exchange search here.

create[\W_]+several[\W_]+staging[\W_]+and[\W_]+development[\W_]+branches has been seen in 6 true positives, 0 false positives, and 0 NAAs.

I'm watching this pattern in the attempt to catch different users' promotion of a service, some of which they edit links out from later (or don't include at all and invite PMs). Looks like all the recent ones use this exact phrase, at least. Should hopefully catch any more.

Approved by Makyen in Charcoal HQ


fix(ci): bump to v3.0.1

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

GitHub release
v1.1.5

Your semantic-release bot :package::rocket:


Allow custom SupportSQLiteOpenHelper.Callback in the SqlNormalizedCacheFactory

A quick PR to allow custom callbacks in the SqlNormalizedCacheFactory. This should allow us to get lifecycle events and apply custom database settings if needed. The eventual plan is to allow us to try something like this: https://stackoverflow.com/questions/65425352/sqldelight-slow-performance-compared-to-room

I just realized that this approach will mean that callers have to import sqldelight dependency to be able to delegate to the upstream AndroidSqliteDriver.Callback as well as get a reference to the schema.
I wonder if it's ok for apollo-kotlin to do the delegation to the upstream callback? Or does it feel like too much "business logic" at this point?

I wonder if it's ok for apollo-kotlin to do the delegation to the upstream callback? Or does it feel like too much "business logic" at this point?

I think doing so would avoid potential user mistake, and simplify the usage, right? I'm not 100% sure of the consequences of not calling the default sqldelight callback, but it looks like it would break (e.g. the schema wouldn't be created)? So if it must be called always, we might as well do it in the library?

What about a configure function?
  @JvmOverloads
  constructor(
      context: Context,
      name: String? = "apollo.db",
      factory: SupportSQLiteOpenHelper.Factory = FrameworkSQLiteOpenHelperFactory(),
      configure: (SupportSQLiteDatabase) -> Unit,
      useNoBackupDirectory: Boolean = false,
  ) : this(
      AndroidSqliteDriver(
          getSchema(),
          context.applicationContext,
          name,
          factory,
          object : AndroidSqliteDriver.Callback(getSchema()) {
            override fun onConfigure(db: SupportSQLiteDatabase) {
              super.onConfigure(db)
              configure(db)
            }
          },
          useNoBackupDirectory = useNoBackupDirectory
      ),
  )

Doesn't leak the sqldelight in the public API and mandates that calles to AndroidSqliteDriver.Callback is called?

I think doing so would avoid potential user mistake, and simplify the usage, right? I'm not 100% sure of the consequences of not calling the default sqldelight callback, but it looks like it would break (e.g. the schema wouldn't be created)? So if it must be called always, we might as well do it in the library?

Yes, I think skipping the call to the upstream callback (AndroidSqliteDriver) is not an option. It is bound to cause a lot of other issues.

What about a configure function?
...
Doesn't leak sqldelight in the public API and mandates that calls to AndroidSqliteDriver.Callback are made?

I think that works for now, and I also like the fact that it does not introduce a sqldelight API as part of the apollo-kotlin API. However, in the future, if we end up needing to introducing more more of the callback's APIs, we'll need to add them all to the constructor (e.g. if someone wants a custom onCreate in the future).
Maybe it's ok to just have configure function for now..?

we'll need to add them all to the constructor (e.g. if someone wants a custom onCreate in the future).

Let's revisit when that happens? My guiding principle here is that in the benefit of the doubt, the most restrictive API is safer. It's always easier to add a constructor than remove one


Can not reach 76 kSteps/s with arduino nano. Speeds does not seem to increase at higher speeds than 250 us /s

Hi,
I'm not sure if this really is an issue or just me not understanding the library fully. In the documentation I read up to 76 kSteps/s or 37 kSteps/s with 2 steppers. When i lower the speed setting from 500 to 250 us/s i can see the speed increasing but when i go lower than 250 the speeds does not increase anymore.
Is there a reason i can not go any faster ? i am using an Arduino nano
ps really like what you are doing here. Keep up the good work!

I had the acceleration not high enough,

Good to hear and thanks for letting me know. Based on your issue, I have checked for any automated test case in regard to higher speeds with AVR. As I could not find one and one test case was erroneously anyway, I have changed that one to check the 37kSteps/s with two steppers.

I have added another test case for 75kSteps/s (cannot reach 76000Hz).
And another test case, because I have stumbled about the issue, that with high acceleration, steps are lost.
For that issue #152 has been added

it was not really an issue, but i did not calculate how long it would take to reach the max speed. The motor had to start ramping down before it finished accelerating. Thus never reaching the max speed
Perhaps the .moveTo() could return a number to indicatie it will never reach the maximum speed with the acceleration and position setpoint settings

Unfortunately getting this info out of moveTo is more complex due to the structure of the driver. While running you could monitor rampState(), if it turns to COAST anytime.


Plugin for z-wave device without z-wave server?

I'm fairly new to both Homekit and Z-wave but I've decided to standardize everything with Homekit and so far Homebridge is perfectly integrating all my non-homekit devices. However, I am looking to buy a smart lock and the only native homekit supported lock I've found is August Smart Lock but it's not compatible with Scandinavian locks at all.
So I found a lock called Danalock which supports Z-Wave, and I've realized that I could set up a z-wave server with a RaZberry and then use the homebridge-zway plugin to integrate the lock with Homebridge. However, my question is if it's possible to write a plugin for the lock directly, removing the need for a dedicated z-wave component?
Thank you!

To communicate with z-wave device you should use z-way-server. But danalock have bluetooth support, so you can try to control danalock with plugin homebridge-bluetooth (if it exists, I dont know)

Hi,
I don't have any Razberry but the Aeon USB Z-wave stick. Do you know if I can I run Z-way anyway?
Thanks.

I think you should look into openHAB. I use it with the Aeon Z-Stick and it works quite well.

I have everything running in domoticz, can I just use the domoticz plugin and I'm done?

This might work: https://github.com/patchworkboy/homebridge-eDomoticz
If you have issues then you might be better off asking for support under Issues there.

It works like a charm. Wonderful.


Fix typo in Expiration docs

I think there is a typo in the Expiration section.
As expire_files accepts the date, not the time.
I just wanted to update this in case of any confusion when copying from the docs.

Thanks for the patch!


passing gromacs tests

Fixes #
Changes made in this Pull Request:

Added instructions on validating gromacs compiled with CUDA on SOL

PR Checklist

[ ] Tests?
[ ] Docs?
[ ] CHANGELOG updated?
[ ] Issue raised/referenced?

Is this PR still relevant or is it stale @ljwoods2 ?

@orbeckst Still relevant, though this will have some nasty merge conflicts. Still think build + test instructions for complex builds should be a part of this repo, though.
@amruthesht can you please format your files similarly to how @hcho38 and I did in the hpc_testing folder and move them?

nice work


Revert torch version

This is to unblock mac users and also to make it easier for dependencies that hard code an earlier version
Description
Please include:

a summary of the changes
links to any related issue/ticket
any additional relevant motivation and context
details of any dependency updates that are required for this change

Proposed version
Please select the option below that is most relevant from the list below. This
will be used to generate the next tag version name during auto-tagging.

[x] Skip auto-tagging
[ ] Patch
[ ] Minor version
[ ] Major version

Visit the Semver website to understand the
difference between MAJOR, MINOR, and PATCH versions.
Notes:

If none of these options are selected, auto-tagging will fail (integrated soon)
Where multiple options are selected, the most senior option ticked will be
used -- e.g. Major > Minor > Patch
If you are selecting the version in the list above using the textbox, make
sure your selected option is marked [x] with no spaces in between the
brackets and the x

Type of change
Please select the option(s) below that are most relevant:

[x] Bug fix
[ ] New feature
[ ] Breaking change

How Has This Been Tested?
Please describe the tests that you added to verify your changes.
Before submitting

[ ] I've read and followed all steps in the Making a pull request
section of the CONTRIBUTING docs.
[ ] I've updated or added any relevant docstrings following the syntax described in the
Writing docstrings section of the CONTRIBUTING docs.
[ ] If this PR fixes a bug, I've added a test that will fail without my fix.
[ ] If this PR adds a new feature, I've added tests that sufficiently cover my new functionality.

Had a package version error, but also had ticked the "Skip auto-tagging" option, so I'm just going to ignore this


Fix ep.PythonCmd return values.

Fix ep.PythonCmd return values.

thanks for quick review.
your suggestion is ok for me.

I will rebase it ..
Seems error.

@codablock
Now, its ok for merge.


Long sequences of repeating characters are misaligned

Windows Terminal version
1.17.1023
Windows build number
10.0.22621.1265
Other Software
Font: Iosevka Term 19.0.1
Steps to reproduce
Simply type long sequences of repeating characters like ------ or !!!!!!!!!.
Expected Behavior
Characters should appear aligned to the grid.
Actual Behavior
Characters get misaligned within the grid, such that the cursor position no longer corresponds to location where typed characters appear. Happens on any prompt, shell, WSL2 distro and in applications such as Vim.
https://user-images.githubusercontent.com/40664661/219950686-315eb4fc-71fe-4ec7-a6e6-8a61784566e9.mp4

I'd reckon that this would be a bug with the way the font is authored. I do see that there's the upstream https://github.com/be5invis/Iosevka/issues/1553 though and we're just spiderman.png'ing here.
Maybe @lhecker can take a look at the metrics under the debugger and figure out what's going on here.

I'd also be curious to know if any earlier versions of Iosevka Term worked. That would quickly help narrow it down.

Until the fix has shipped you can either disable the new text renderer in the "Rendering" section of your settings, or use a slightly different font size, like 11.5. 🙂

Disabling the new text renderer (AtlasEngine) has no effect on the issue. Changing the font size/zoom level works for now.


133 add api blueprint submodule

closes #133
changes proposed in the pull request:

Add tinkerware-blueprint as submodule
Add drakof dependency and add script to run mock-server
Update readme

@thetonymaster

Coverage remained the same at 89.796% when pulling d5bc288317a0e75239a4e3c559626051b5456071 on 133-add-API-Blueprint-submodule into 5485e5873f0d4d810cef2b4fa801dd837c259cb3 on master.

LGTM

Coverage remained the same at 89.796% when pulling eb2e17bdb8af94b26e919ef2d42f4b1aed98bf70 on 133-add-API-Blueprint-submodule into 5485e5873f0d4d810cef2b4fa801dd837c259cb3 on master.


Array Out of Bounds Issue in ANS Compressor

I am encountering an issue while using the DietGPU library to compress a relatively long array. The floatANS compressor works perfectly, but the ANS compressor is showing an error message: "Check failed: rawSize <= std::numeric_limits<int32_t>::max()".
What is the maximum compressible length of a tensor using the ANS compressor? Is it possible to increase the maximum compression length of the ANS compressor to match that of the floatANS?

Both the ANS codec and the floating point codec are written using 32-bit indexing arithmetic, because the original target for dietgpu was super high performance compression/decompression for InfiniBand / fast Ethernet communications during large-scale neural network training, as there are sometimes severe penalties for 64-bit integer/pointer arithmetic on Nvidia GPUs. In this use case, we are typically transferring 100s of kB to 100s of MB of data between machines/GPUs, not much more.
I won't update the code here to use 64-bit indexing instead, but you can bypass the problem by separately compressing chunks of the input to get under the (2^31 - 1) limit that this is complaining about and append them together if you need to be able to compress and decompress longer runs of data.

Both the ANS codec and the floating point codec are written using 32-bit indexing arithmetic, because the original target for dietgpu was super high performance compression/decompression for InfiniBand / fast Ethernet communications during large-scale neural network training, as there are sometimes severe penalties for 64-bit integer/pointer arithmetic on Nvidia GPUs. In this use case, we are typically transferring 100s of kB to 100s of MB of data between machines/GPUs and targeting compression speeds in the 100s of GB per second, not compressing much larger pieces of data.
I won't update the code here to use 64-bit indexing instead, but you can bypass the problem by separately compressing chunks of the input to get under the (2^31 - 1) limit that this is complaining about and append them together if you need to be able to compress and decompress longer runs of data.

Both the ANS codec and the floating point codec are written using 32-bit indexing arithmetic, because the original target for dietgpu was super high performance compression/decompression for InfiniBand / fast Ethernet communications during large-scale neural network training, as there are sometimes severe penalties for 64-bit integer/pointer arithmetic on Nvidia GPUs. In this use case, we are typically transferring 100s of kB to 100s of MB of data between machines/GPUs and targeting compression speeds in the 100s of GB per second, not compressing much larger pieces of data.
I won't update the code here to use 64-bit indexing instead, but you can bypass the problem by separately compressing chunks of the input to get under the (2^31 - 1) limit that this is complaining about and append them together if you need to be able to compress and decompress longer runs of data.

Cool! Thank you for your response.


Rename default route

WHAT
Rename the default route
WHY
It displays watch speakers but music cannot be played until an output device is chosen from the BT Settings Fragment
HOW
rename in the strings file
Checklist :clipboard:

[ ] Add explicit visibility modifier and explicit return types for public declarations
[ ] Run spotless check
[ ] Run tests
[ ] Update metalava's signature text files

As discussed, I think this change doesn't work if Watch Speaker playback is allowed. So I think it should be something like changing
https://github.com/google/horologist/blob/65d0be9499b9d89e67e77339ed39d3c5f5ff048b/audio-ui/src/main/java/com/google/android/horologist/audio/ui/components/AudioOutputUi.kt#L36
to have some flag of whether this is a valid output.  And if not then we display a prompt instead of the device.

Worth updating title and description.


Excel Fix broken link

The archive of codeplex is no longer available so I removed the link to OOXML JavaScript SDK.

Docs Build status updates of commit fe40bd6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/pnp-open-in-excel.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

Docs Build status updates of commit 928aeea:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docs/excel/pnp-open-in-excel.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


added is-cli to documentation

Just a general feeling of mine. I think it would be better to combine both pages into one. Less pages are better imo.


feature request : add socket.io lib

hm.. what if someone doesn't need it in his project? the repo is a little bit fat already

the repo is fat because of testing and e2e

what I'm saying is let's not add features that not everyone needs.

@lastnod, you can see how to add it in my tutorial here: https://github.com/AngularClass/angular2-webpack-starter/wiki/How-to-use-with-FeathersJS


Switch scripts in package.json to use yarn for consistency

I switched all the scripts to use yarn for the consistency of the project since we are using yarn.lock. I also updated collaborator guidelines to instruct others to install and build correctly the project.

Love the contribution. I'm using yarn personally - are you @kulshekhar ?  (or anyone else)

@GeeWee I use yarn but I asked that change to be reverted. We shouldn't force folks (who are trying to contribute) to install yarn if they use only npm


A midday theme?

First off, wonderful theme. It's the best one I've found for Ruby. I would like to request if it's possible to create a dark theme with a softer background that can be used even during the day time? Something similar to Zenburn if possible.

I have quickly added a slightly lighter theme called Ruby Dusk, Let me know any suggestion you have.

Wonderful! My eyes thank you!


Symlinks to missing files

75870dd changes
-    deep-cp $BUILD_DIR $APP_DIR
+    # deep-cp $BUILD_DIR $APP_DIR
+    ln -s $BUILD_DIR/* $APP_DIR
seems to link to a missing Procfile (and probably other files) causing:
remote: -----> Discovering process types
remote: Procfile declares types -> (none)

We also get a failure, but with different message:
remote: -----> Python app detected

remote: ln: failed to create symbolic link /app/tmp: File exists

remote:  !     Push rejected, failed to compile Python app.

That's probably because we have a /tmp folder in our repository.

This change has been reverted.
Ideally, this should have just simply sped up the buildpack by reducing the number of IOPS required during the build process—and all worked well in my exploratory tests. Further investigation will have to be required.
Thanks for the report.


Add support for ServiceKeys

@jabrena This fuctionaliy is very great. I want to use it. Could you consider to merge the pull request?


Add generic locator

Coverage increased (+0.09%) to 99.307% when pulling 230b22f23a43590e1ecbb21d0dd6c1a80ca35a31 on add-generic-locator into 335ada85fc4eacd746b1bc429b5fa62e53b00a82 on reorganize.


Implement include layout

Implement the proteus equivalent of the include tag
<include layout="layout_to_include" />

implemented in #67 and available since v5.0.0-rc0


Issues with API level 19.

Repeition steps.
Open Android studio layout using FAB at API level 21.
Turn the layout landscape.
Switch to API level 19.
Receive

Hi, thanks for reporting the issue. But it seems it's a bug in the Android Studio UI Preview. With API 21 selected for the preview the FAB is rendered correctly.
Here is a ling to issue thread: https://code.google.com/p/android/issues/detail?id=75204.


[Feature request] Custom increment counter

Add the ability to increment any custom number per day/week/month.
Example1: If I had started to save 5 coins every day since date x, how many coins would I have today.
Example2: I am allowed to drive 15000km per 12 month since buying my car, how many do I have left?

Thanks for your proposal!
Could you please explain more about Example 2? Is it a special restriction or rule that requires tracking the traveled distance? Since I'm not familiar with the car industry and don't understand why we need to know that information, and what's the relationship between the traveled distance and counting days.
Regarding Example 1, the feature will not be implemented shortly due to concerns about user experience confusion.

Ex2 is actually similar to ex1.
Instead of counting a 1 for each day, you count another number.
Setting 2024-01-01 as start date, you would now (2024-09-30) be at 272.
I am looking for a feature where you tell the app that each day counts as [5 coins; 41,09km; etc.], so next to the 272 days, it shows [1360 coins, 11.178,08km, etc.].
Its about counting things, that increment regularly over time. You could also count pills taken each week since a diagnosis or wedding anniversaries


fix: support collections in deleteFromDisk

do not delete entire database in case only one objectStore to be
deleted

Signed-off-by: TheOneWithTheBraid<EMAIL_ADDRESS>

I would appreciate an update on pub.dev.

I would appreciate an update on pub.dev.

Can you write a small sentence describing changes being done on this patch (needed for changelog)? :D I didn't understand what exactly this patch is going to do, otherwise I would write it myself

This fix properly supports deleting BoxCollections on web respecting whether there are different object stores stored in.

Done with https://github.com/hivedb/hive/pull/976.


⚠ Bofrost.at has degraded performance

In 0c9a632, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 2689 ms

Resolved: Bofrost.at performance has improved in 11d70ff after 6 minutes.


replace export default with defineComponent

replace export default with defineComponent
there are some property type error in some vue files

make min-width and max-width of the modal component generic

Added defineComponent() to:
sources/packages/web/src/App.vue
sources/packages/web/src/components/aest/OfferingApplicationBanner.vue
sources/packages/web/src/components/aest/OfferingChangeRequest.vue
sources/packages/web/src/components/aest/institution/modals/ApproveProgramModal.vue
sources/packages/web/src/components/aest/institution/modals/AssessOfferingChangeModal.vue
sources/packages/web/src/components/aest/institution/modals/AssessOfferingModal.vue
sources/packages/web/src/components/aest/institution/modals/DeclineProgramModal.vue
sources/packages/web/src/components/common/AppealRequestsForm.vue
sources/packages/web/src/components/common/AssessmentAwardDetails.vue
sources/packages/web/src/components/common/IdleTimeChecker.vue
sources/packages/web/src/components/common/InstitutionUserSummary.vue
sources/packages/web/src/components/common/LocationSummary.vue
sources/packages/web/src/components/common/ManageProgramAndOfferingSummary.vue
sources/packages/web/src/components/common/OfferingSummary.vue
sources/packages/web/src/components/common/ProgramDetails.vue
sources/packages/web/src/components/common/ProgramOfferingDetailHeader.vue
sources/packages/web/src/components/common/ScholasticStandingForm.vue
sources/packages/web/src/components/common/StudentAppealForm.vue
sources/packages/web/src/components/common/StudentApplication.vue
sources/packages/web/src/components/common/StudentProfileForm.vue
sources/packages/web/src/components/common/modals/ConfirmExtendTime.vue
sources/packages/web/src/components/common/modals/ConfirmModal.vue
sources/packages/web/src/components/common/notes/Notes.vue
sources/packages/web/src/components/common/restriction/AddRestriction.vue
sources/packages/web/src/components/common/restriction/ViewRestriction.vue
sources/packages/web/src/components/common/sin/AddNewSIN.vue
sources/packages/web/src/components/generic/BCLogo.vue
sources/packages/web/src/components/generic/Banner.vue
sources/packages/web/src/components/generic/BodyHeader.vue
sources/packages/web/src/components/generic/CheckPermissionRole.vue
sources/packages/web/src/components/generic/ChipLabel.vue
sources/packages/web/src/components/generic/ChipStatus.vue
sources/packages/web/src/components/generic/DetailHeader.vue
sources/packages/web/src/components/generic/ErrorSummary.vue
sources/packages/web/src/components/generic/FooterButtons.vue
sources/packages/web/src/components/generic/FormioModalDialog.vue
sources/packages/web/src/components/generic/HeaderNavigator.vue
sources/packages/web/src/components/generic/HeaderTitleValue.vue
sources/packages/web/src/components/generic/SnackBar.vue
sources/packages/web/src/components/generic/StatusChipActiveApplication.vue
sources/packages/web/src/components/generic/StatusChipActiveUser.vue
sources/packages/web/src/components/generic/StatusChipApplication.vue
sources/packages/web/src/components/generic/StatusChipAssessmentHistory.vue
sources/packages/web/src/components/generic/StatusChipCOE.vue
sources/packages/web/src/components/generic/StatusChipDesignation.vue
sources/packages/web/src/components/generic/StatusChipDesignationAgreement.vue
sources/packages/web/src/components/generic/StatusChipOffering.vue
sources/packages/web/src/components/generic/StatusChipProgram.vue
sources/packages/web/src/components/generic/StatusChipProgramInfoRequest.vue
sources/packages/web/src/components/generic/StatusChipRequestedAssessment.vue
sources/packages/web/src/components/generic/StatusChipRestriction.vue
sources/packages/web/src/components/generic/StringToHtmlParagraphs.vue
sources/packages/web/src/components/generic/StudentRestrictionChip.vue
sources/packages/web/src/components/generic/TitleValue.vue
sources/packages/web/src/components/generic/ToggleContent.vue
sources/packages/web/src/components/generic/ValidatedInput.vue
sources/packages/web/src/components/institutions/active-application/ActiveApplicationSummaryData.vue
sources/packages/web/src/components/institutions/locations/LocationEditForm.vue
sources/packages/web/src/components/institutions/modals/AddInstitutionUserModal.vue
sources/packages/web/src/components/institutions/modals/EditInstitutionUserModal.vue
sources/packages/web/src/components/institutions/modals/InstitutionUserManagement.vue
sources/packages/web/src/components/institutions/modals/confirmationOfEnrollment/ApproveCOE.vue
sources/packages/web/src/components/institutions/modals/confirmationOfEnrollment/DenyCOE.vue
sources/packages/web/src/components/institutions/profile/InstitutionProfileForm.vue
sources/packages/web/src/components/layouts/Institution/sidebar/HomeSideBar.vue
sources/packages/web/src/components/layouts/Institution/sidebar/ManageInstitutionSideBar.vue
sources/packages/web/src/components/layouts/aest/AESTApplicationSideBar.vue
sources/packages/web/src/components/layouts/aest/AESTHomeSideBar.vue
sources/packages/web/src/components/partial-view/DesignationAgreement/DesignationAgreementForm.vue
sources/packages/web/src/components/partial-view/DesignationAgreement/DesignationAgreementSummary.vue
sources/packages/web/src/components/students/RestrictionBanner.vue
sources/packages/web/src/components/students/applicationTracker/Draft.vue
sources/packages/web/src/components/students/applicationTracker/Submitted.vue
sources/packages/web/src/components/students/modals/PDStatusApplicationModal.vue
sources/packages/web/src/views/aest/AppAEST.vue
sources/packages/web/src/views/aest/ApplicationExceptionsApproval.vue
sources/packages/web/src/views/aest/Login.vue
sources/packages/web/src/views/aest/NoticeOfAssessment.vue
sources/packages/web/src/views/aest/Reports.vue
sources/packages/web/src/views/aest/SearchInstitutions.vue
sources/packages/web/src/views/aest/SearchStudents.vue
sources/packages/web/src/views/aest/StudentAppealRequestsApproval.vue
sources/packages/web/src/views/aest/StudentApplicationView.vue
sources/packages/web/src/views/aest/SupportingUser.vue
sources/packages/web/src/views/aest/institution/Designation.vue
sources/packages/web/src/views/aest/institution/DesignationAESTView.vue
sources/packages/web/src/views/aest/institution/InstitutionDetails.vue
sources/packages/web/src/views/aest/institution/InstitutionLocationEdit.vue
sources/packages/web/src/views/aest/institution/InstitutionNotes.vue
sources/packages/web/src/views/aest/institution/Locations.vue
sources/packages/web/src/views/aest/institution/OfferingChangeRequests.vue
sources/packages/web/src/views/aest/institution/PendingDesignation.vue
sources/packages/web/src/views/aest/institution/Profile.vue
sources/packages/web/src/views/aest/institution/ProfileCreate.vue
sources/packages/web/src/views/aest/institution/ProfileEdit.vue
sources/packages/web/src/views/aest/institution/ProgramDetails.vue
sources/packages/web/src/views/aest/institution/Programs.vue
sources/packages/web/src/views/aest/institution/Restrictions.vue
sources/packages/web/src/views/aest/institution/Users.vue
sources/packages/web/src/views/aest/institution/ViewOffering.vue
sources/packages/web/src/views/aest/institution/ViewOfferingChangeRequest.vue
sources/packages/web/src/views/aest/institution/ViewOfferingChangeRequestComplete.vue
sources/packages/web/src/views/aest/student/Applications.vue
sources/packages/web/src/views/aest/student/FileUploads.vue
sources/packages/web/src/views/aest/student/Profile.vue
sources/packages/web/src/views/aest/student/SINManagement.vue
sources/packages/web/src/views/aest/student/StudentAccountApplications.vue
sources/packages/web/src/views/aest/student/StudentAccountApplicationsApproval.vue
sources/packages/web/src/views/aest/student/StudentApplicationAppeals.vue
sources/packages/web/src/views/aest/student/StudentApplicationExceptions.vue
sources/packages/web/src/views/aest/student/StudentDetails.vue
sources/packages/web/src/views/aest/student/StudentNotes.vue
sources/packages/web/src/views/aest/student/StudentRestrictions.vue
sources/packages/web/src/views/aest/student/ViewScholasticStanding.vue
sources/packages/web/src/views/institution/AddInstitutionLocation.vue
sources/packages/web/src/views/institution/AppInstitution.vue
sources/packages/web/src/views/institution/EditInstitutionLocation.vue
sources/packages/web/src/views/institution/InstitutionCreate.vue
sources/packages/web/src/views/institution/InstitutionProfile.vue
sources/packages/web/src/views/institution/InstitutionUserDetails.vue
sources/packages/web/src/views/institution/InstitutionUserProfile.vue
sources/packages/web/src/views/institution/Login.vue
sources/packages/web/src/views/institution/ManageLocations.vue
sources/packages/web/src/views/institution/designations/DesignationRequest.vue
sources/packages/web/src/views/institution/designations/DesignationView.vue
sources/packages/web/src/views/institution/designations/ManageDesignations.vue
sources/packages/web/src/views/institution/locations/active-applications/ActiveApplicationReportAChange.vue
sources/packages/web/src/views/institution/locations/active-applications/LocationActiveApplicationSummary.vue
sources/packages/web/src/views/institution/locations/active-applications/ViewSubmittedApplicationScholasticStanding.vue
sources/packages/web/src/views/institution/locations/confirmation-of-enrollment/ApplicationDetailsForCOE.vue
sources/packages/web/src/views/institution/locations/confirmation-of-enrollment/COESummaryData.vue
sources/packages/web/src/views/institution/locations/confirmation-of-enrollment/LocationCOESummary.vue
sources/packages/web/src/views/institution/locations/program-info-request/LocationProgramInfoRequestSummary.vue
sources/packages/web/src/views/institution/locations/programs/LocationProgramAddEdit.vue
sources/packages/web/src/views/institution/locations/programs/LocationProgramView.vue
sources/packages/web/src/views/institution/locations/programs/LocationPrograms.vue
sources/packages/web/src/views/student/CheckValidSINBanner.vue
sources/packages/web/src/views/student/Login.vue
sources/packages/web/src/views/student/StudentAccountActivity.vue
sources/packages/web/src/views/student/StudentApplicationSummary.vue
sources/packages/web/src/views/student/StudentDashboard.vue
sources/packages/web/src/views/student/StudentFileUploader.vue
sources/packages/web/src/views/student/StudentProfileCreate.vue
sources/packages/web/src/views/student/StudentProfileEdit.vue
sources/packages/web/src/views/student/financial-aid-application/Applications.vue
sources/packages/web/src/views/student/financial-aid-application/FullTimeApplication.vue
sources/packages/web/src/views/supporting-user/AppSupportingUser.vue
sources/packages/web/src/views/supporting-user/Dashboard.vue
sources/packages/web/src/views/supporting-user/Login.vue
Some errors TS errors are shown as below:

We don't have any compilations errors though.


Integrate with api

Sharing this WIP for comment /cc @alan @tatyree
There's a lot of work to be done on the tests, and the code is currently focussed on the happy path (making it production ready is left for future tickets), but I'm pretty happy with the overall structure at this point.

LGTM

This is now ready for proper review and merge. Some things of interest, since this is by now a rather big PR:

For testing, I have gone for the approach of stubbing/mocking calls to PrisonVisits::API which keeps the tests clean – see for example 2a5fe75afa784517494932cd7b9193d65baa919c.
The tests of the API client itself use VCR, so they can be easily kept current as the API evolves, and are satisfying simple – see 61926bb0ce1b48051ad78d2495c13a3da05489df.
This is a slight change of behaviour: d93b41782ee88d9243f323aeaaaca1a01b38d38a.

LGTM.  One comment re. an old-style find_by_id call, but I'm happy to merge with that in place.
The stubbing/mocking is very clean :+1:
I've always found that VCR promises slightly more than it delivers.  Still the best choice, IMHO, though.
Validating the slot format seems sensible.


Some bugfixes for the current implementation of the event bus

Event Bus will only release the current session
Timeout for event bus
Additional Logging
Sync call for event bus tests
Event Bus can still not be used in Tests (next PR)

This change is

Reviewed 10 of 17 files at r1.
Review status: 10 of 17 files reviewed at latest revision, 3 unresolved discussions, some commit checks failed.

platform-extras/dolphin-platform-spring-controller-tester/src/main/java/com/canoo/dolphin/test/impl/ClientContextForTests.java, line 5 at r1 (raw file):

import com.canoo.dolphin.client.ClientContext;

/**

remove this

platform-extras/dolphin-platform-spring-controller-tester/src/test/java/com/canoo/dolphin/test/eventbus/EventBusTest.java, line 13 at r1 (raw file):

import static org.testng.Assert.assertTrue;

/**

remove this

platform-extras/dolphin-platform-spring-controller-tester/src/test/java/com/canoo/dolphin/test/eventbus/EventBusTestConstants.java, line 6 at r1 (raw file):

/**
 * Created by hendrikebbers on 24.08.16.

Remve this

Comments from Reviewable

Review status: 6 of 20 files reviewed at latest revision, 3 unresolved discussions, some commit checks failed.

platform-extras/dolphin-platform-spring-controller-tester/src/main/java/com/canoo/dolphin/test/impl/ClientContextForTests.java, line 5 at r1 (raw file):
Previously, timo0 (Timo B) wrote…
> remove this

done

platform-extras/dolphin-platform-spring-controller-tester/src/test/java/com/canoo/dolphin/test/eventbus/EventBusTest.java, line 13 at r1 (raw file):
Previously, timo0 (Timo B) wrote…
> remove this

done

platform-extras/dolphin-platform-spring-controller-tester/src/test/java/com/canoo/dolphin/test/eventbus/EventBusTestConstants.java, line 6 at r1 (raw file):
Previously, timo0 (Timo B) wrote…
> Remve this

done

Comments from Reviewable

Coverage decreased (-0.1%) to 68.749% when pulling 72ae0afb89af771348ff899e7c9aaa4cc2b00f5e on event-bus-bugfixes into eab215ae911ae0df9827748b48ab7b2c851a12de on master.

Coverage decreased (-0.2%) to 67.999% when pulling 8b699272c684cc3d266e5212a3166eeeb984503b on event-bus-bugfixes into b7e6780a8a53175f06214852256c624437e48775 on master.


Grafana csv

Types of changes

[ ] Enhancement (project structure, spelling, grammar, formatting)
[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

A description of the changes proposed in the Pull Request

Added python script to generate a csv file from ompl yml content
Added a json template for a Grafana dashboard to visualize impact engine output, based on a csv data source

The idea is absolutely great. The only concern I see is that it will add a new core dependency on python, which will eventually impact UX and DX. It's fine in general, but better to be double-checked if it's worth it...
@jmcook1186 FYI

Indeed @jmcook1186 ! @demakoff you make a valid point, we've been discussing this in recent meetings and it is clear that a Python CSV exporter should not last for the long run. AFAIK there's already plan to have TS one in place.
Same goes for Grafana BTW: I'm in the process of documenting some more easy-to-setup solutions.


Better naming for Square

extending https://github.com/agda/agda-categories/issues/5#issuecomment-508801244
Square is more than squares. potentially Morphism.Reasoning a better place for it?

Agreed.

https://github.com/agda/agda-categories/commit/7c22ad40804e9895f30a33279c06505df1f2774f resolved this.


Fixed #26653 -- Made SyndicationFeed.latest_post_date() return time in UTC.

buildbot, add to whitelist.

@timgraham what happens next? Is there anything I have to do now?
Sorry this is my first PR and I am not aware of what happens next.
Thank you for your help.

Final two comments:

Please squash your commits into one commit
Change the commit message to Fixed #26653 -- Made SyndicationFeed.latest_post_date() return time in UTC.as suggested by Tim

@berkerpeksag Thank you :)

merged in f31fbbae1ab1572e1637fc5ca9ae4bae66679294, thanks!


[Chapter Proposal] RouenP2P

If you are living around Rouen, France, or if you are sometimes going there, and if you are interested in organizing a RouenP2P Meetup group, please comment on this issue or contact me.
Related with #5

cc @gnieark @Zigazou

Here ;)

Twitter thread with @gnieark and his peers
https://mobile.twitter.com/moul/status/1161004069907025920


Adding Guess the shape game

🛠 Fixes Issue #1426

👨💻 Changes proposed and Brief Description:

This project is built on a basic web tech stacks such as HTML, CSS and JavaScript.
This is a single-player game.
The player will tell the shape formed using dots.
Options are given in the game.
If player reaches to 10 points will win the game.
If player looses all the 5 lives will lose the game.

✅ Check List (Check all the applicable boxes)

[✅] My code doesn't break any part of the project
[✅] This PR does not contain plagiarized content.
[✅] My Addition/Changes works properly and matches the overall repo pattern.
[✅] The title of my pull request is a short description of the requested changes.

📷 Screenshots

What is the reason Why Auto level fail ? @swapnilsparsh

Hey Can you tell me what's the problem ? @swapnilsparsh


ExecutableElement has isAsynchronous, isGenerator, etc.

But this is body, and this information is not available from Kernel outlines.
@bwilkerson Maybe we should stop exposing this data from element model.

I'd need to see whether / where the information is being used to answer that question.

For now we use full kernels, but when we switch to outlines, we will have this problem again.

The CFE integration work is suspended, so this will not be addressed.


Add more testcases to test the different scenarios  in sql script creation

Description:
Add more new testcases to test the sql script creation

Closing this issue as we added test cases for identified scenarios. Please create a new issue highlighting the specific scenario which need testcase


github: add missing bookworm as target

github: drop redundant --mirror option
Use pre-configured chroot apt-line, so --mirror is redundant.
Without it, it causes the following error on bookworm:
The following packages have unmet dependencies:
 fluent-package : Depends: libncurses6 (>= 6) but it is not installable
W: Target Packages (contrib/binary-amd64/Packages) is configured multiple times in /etc/apt/sources.list:2 and /etc/apt/sources.list.d
W: Target Packages (contrib/binary-all/Packages) is configured multiple times in /etc/apt/sources.list:2 and /etc/apt/sources.list.d/t
W: Target Translations (contrib/i18n/Translation-en) is configured multiple times in /etc/apt/sources.list:2 and /etc/apt/sources.list

github: add more .service check after removing
After removing fluent-package, the status of service files seems
different.

bookworm:
  no /etc/systemd/system/fluentd.service
  dead symlink /etc/systemd/system/multi-user.target.wants/luentd.service
  dead /etc/systemd/system/td-agent.service

bullseye:
  dead /etc/systemd/system/fluentd.service
  dead symlink /etc/systemd/system/multi-user.target.wants/luentd.service
  dead /etc/systemd/system/td-agent.service

Oops, I've forgot to add it. Thanks.

https://github.com/fluent/fluent-package-builder/actions/runs/6144619825/job/16670375606?pr=586
Hmm, something seems wrong.

I'll check it.

It is because bookworm was supported since v5. (no bookworm deb for v4)

Thanks!
In addition, I think this code does not work as expected.
https://github.com/fluent/fluent-package-builder/blob/cb795a5253e18f05a754ca9d41cef4aa2dbf8536/fluent-package/apt/pkgsize-test.sh#L14-L22
$code_name is from this.
https://github.com/fluent/fluent-package-builder/blob/cb795a5253e18f05a754ca9d41cef4aa2dbf8536/fluent-package/apt/commonvar.sh#L1
It seems that we should use $CODE_NAME instead of $code_name.
It is so confusing, so it may be better to change the name $CODE_NAME...

Thanks! In addition, I think this code does not work as expected.
https://github.com/fluent/fluent-package-builder/blob/cb795a5253e18f05a754ca9d41cef4aa2dbf8536/fluent-package/apt/pkgsize-test.sh#L14-L22
$code_name is from this.
https://github.com/fluent/fluent-package-builder/blob/cb795a5253e18f05a754ca9d41cef4aa2dbf8536/fluent-package/apt/commonvar.sh#L1
It seems that we should use $CODE_NAME instead of $code_name.
It is so confusing, so it may be better to change the name $CODE_NAME...

Hmm, what I'm saying seems partially wrong...
It seems not wrong to use $code_name here...
I don't understand why there is two variarables and this condition does not work though...

checking why...
 0m5.3s DEBUG: Starting command: ['chroot', '/tmp/tmpr3kvd8i9', 'apt-get', '-y', '--allow-downgrades', 'install', './tmp/fluent-package-dbgsym_5.0.1-1_amd64.deb', './tmp/fluent-package_5.0.1-1_amd64.deb']
0m5.4s DUMP:
  Reading package lists...
  Building dependency tree...
  Reading state information...
  Some packages could not be installed. This may mean that you have
  requested an impossible situation or if you are using the unstable
  distribution that some required packages have not yet been created
  or been moved out of Incoming.
  The following information may help to resolve the situation:

  The following packages have unmet dependencies:
   fluent-package : Depends: libncurses6 (>= 6) but it is not installable
  W: Target Packages (contrib/binary-amd64/Packages) is configured multiple times in /etc/apt/sources.list:2 and /etc/apt/sources.list.d/td.list:1
  W: Target Packages (contrib/binary-all/Packages) is configured multiple times in /etc/apt/sources.list:2 and /etc/apt/sources.list.d/td.list:1
  W: Target Translations (contrib/i18n/Translation-en) is configured multiple times in /etc/apt/sources.list:2 and /etc/apt/sources.list.d/td.list:1
  E: Unable to correct problems, you have held broken packages.

Actually, depends on libncurses6.
 apt depends fluent-package
fluent-package
  PreDepends: adduser
  Depends: libc6 (>= 2.35)
  Depends: libcrypt1 (>= 1:4.1.0)
  Depends: libffi8 (>= 3.4)
  Depends: libgcc-s1 (>= 4.2)
  Depends: libgmp10 (>= 2:6.2.1+dfsg1)
  Depends: libncurses6 (>= 6)
  Depends: libreadline8 (>= 6.0)
  Depends: libssl3 (>= 3.0.0)
  Depends: libtinfo6 (>= 6)
  Depends: libyaml-0-2
  Depends: zlib1g (>= 1:1.2.11.dfsg)
  Breaks: td-agent (<< 5.0.0-1)
  Replaces: td-agent (<< 5.0.0-1)

And installed ncurses6.
dpkg -l |grep libncurses
ii  libncurses6:amd64         6.4-4                          amd64        shared libraries for terminal handling
ii  libncursesw6:amd64        6.4-4                          amd64        shared libraries for terminal handling (wide character support)

checking current repos:
apt-cache madison fluent-package
fluent-package |    5.0.1-1 | https://packages.treasuredata.com/5/debian/bookworm bookworm/contrib amd64 Packages
fluent-package |    5.0.0-1 | https://packages.treasuredata.com/5/debian/bookworm bookworm/contrib amd64 Packages

apt-get install -y --allow-downgrades ./fluent-package-dbgsym_5.0.1-1_amd64.deb ./fluent-package_5.0.1-1_amd64.deb  itself is not problem on bookworm container.

Got it, --mirror is redundant in this case.

waiting CI..

Fixed bookworm failure. yay!

a bit confused behavior is observed.
bullseye:
before: no /etc/systemd/system/fluentd.service
after(apt remove): /etc/systemd/system/fluentd.service => /dev/null (dead link)
bookworm:
before: no /etc/systemd/system/fluentd.service
after(apt remove): no /etc/systemd/system/fluentd.service, instead, /etc/systemd/system/multi-user.target.wants/fluentd.service=>/lib/systemd/system/fluentd.service (dead link)

I've updated description and test case a bit.

Hmm, certainly, there seems to be a specification change between debian-bullseye and debian-bookworm...
I think the dead link /etc/systemd/system/fluentd.service is to provide access to the service after removing.
We can still access the service after removing the package in bullseye:
vagrant@debian-11:/etc/systemd/system$ systemctl status fluentd
● fluentd.service
     Loaded: masked (Reason: Unit fluentd.service is masked.)
     Active: inactive (dead) since Wed 2023-09-13 09:16:49 UTC; 13s ago
   Main PID: 11665 (code=exited, status=0/SUCCESS)
        CPU: 896ms
vagrant@debian-11:/etc/systemd/system$ systemctl status td-agent
● td-agent.service
     Loaded: masked (Reason: Unit td-agent.service is masked.)
     Active: inactive (dead)

We can't access the service anymore after removing the package in bookworm:
vagrant@debian-12:~$ systemctl status fluentd
Unit fluentd.service could not be found.
vagrant@debian-12:~$ systemctl status td-agent
Unit td-agent.service could not be found.

I think this is not an issue of this package but of the specification of bookworm.

I've updated description and test case a bit.

Thanks, this change looks reasonable to me.
I also think it might not be necessary to test the remained service file link...
It seems that we should also fix update-from-v4.sh:
https://github.com/fluent/fluent-package-builder/blob/2b9fd5eef0655b70dfc7a6f0a6cc506bea44dad0/fluent-package/apt/systemd-test/update-from-v4.sh#L69-L75

waiting CI...

read_nonblock frequently occurs, :thinking:

read_nonblock frequently occurs, 🤔

Try to search workaround in another issue/pr.

With some retry, all check has passed now.


Clean up duplicated webhooks when istioctl fails

Clean up conflicted webhooks when istioctl fails to unblock reconciliations.

See also:

https://github.com/kyma-project/kyma/issues/17054

/retest

/retest

/retest

/retest

/test pre-main-reconciler-cli-kyma-prev-to-last-release-upgrade-k3d

/test pre-main-reconciler-cli-kyma-prev-to-last-release-upgrade-k3d

/test pre-main-reconciler-cli-kyma-prev-to-last-release-upgrade-k3d

/test pre-main-reconciler-cli-kyma-prev-to-last-release-upgrade-k3d

As per https://github.com/kyma-project/kyma/issues/17030#issuecomment-1471915673:

The issue has been fixed on main. Currently we will override the result of the job to be green. As the fix has been already backported to release branch 2.12. Check here. Once we release 2.12.1 then the job should work fine and we should not need to override the result anymore.

/override pre-main-reconciler-cli-kyma-prev-to-last-release-upgrade-k3d

/override rakesh-garimella_test_of_prowjob_pre-main-reconciler-cli-kyma-p

/test pre-main-reconciler-integration-k3d


Expose Payment Allocations batch logic for invocation via Callable API

W-034352
The Main Batch Method of the Payment Allocations Batch (ALLO_UpdPaymentAllocations_BATCH.processRecords) should be exposed as a Callable process. This will allow Abacus to inherit and reuse all the core logic in the Payment Allocations batch
Additionally, the method (ALLO_AllocationsUtil.validatePaymentAllocationsConfiguration) should be exposed as a Callable process. This will allow Abacus to reuse this call and not have to replicate the functionality when validating the overall readiness to execute the Abacus Batch Process.
Critical Changes
Changes
Issues Closed
Fixes #4804
New Metadata
Deleted Metadata

@jbrandolinisf

@jbrandolinisf should be ready for another pass

@jbrandolinisf Ready for another pass

@jbrandolinisf Requested changes have been made


Connection pooling in HBONE causes thread stickiness in waypoints, decreasing performance

Consider the following setup:
Client ---100 connections --> ztunnel --1 HBONE connection carrying 100 streams --> Waypoint.
Waypoint is running with --concurrency=2.
With this, all the streams will be handled by the same Envoy thread. This cuts throughput in half.
(note: the ExactBalance thing is not valid on the inner listener, and doesn't help on the outer listener)

With this, all the streams will be handled by the same Envoy thread. This cuts throughput in half.

Waypoint is running with --concurrency=2

Does this imply that if waypoint is running with a concurrency setting (e.g. 2), it should support 2 threads but because of HBONE connection pooling, it is being bottlenecked by 1 thread (Envoy). So is the fix to match threads to what's specified by waypoint concurrency?
I might be missing a lot of context as I'm not too familiar with HBONE yet, but I'm willing to tackle this issue to learn more. Seems like a well scoped issue if I can get more details.

Its only bottlenecked if there is only one HBONE connection. But we pool from ztunnel, so its not super rare for that to happen.
I doubt its easy to fix in Envoy, as its entire threading model is silo-ed. The only fix I think may be reasonable is to pool less aggresively in ztunnel (maybe conditionally)

ref https://github.com/istio/istio/pull/52906

ref #52906

Does Ben's PR fix this @howardjohn or should we still pool less aggressively in Ztunnel?
How does his change to use exact balancing relate to your comment in the initial description for this issue?

(note: the ExactBalance thing is not valid on the inner listener, and doesn't help on the outer listener)

Also, in Ben's example which is the outer listener and which is the inner listener (out of the two HBONE and Waypoint listeners he points out) ?

encountered this issue during testing. Sidecars end up being much faster at high qps


⚠ Bofrost.gr has degraded performance

In baee184, Bofrost.gr (https://www.bofrost.gr) experienced degraded performance:

HTTP code: 200
Response time: 1849 ms

Resolved: Bofrost.gr performance has improved in 77e642c after 10 minutes.


false physical blocksize detected

I have a cdrom in a VM that is emulated from an iso on the host system.
in the VM, fdisk reports these block sizes:
$ sudo fdisk -l /dev/sr0
Disk /dev/sr0: 4 MiB, 4194304 bytes, 2048 sectors
Disk model: QEMU DVD-ROM
Units: sectors of 1 * 2048 = 2048 bytes
Sector size (logical/physical): 2048 bytes / 2048 bytes
I/O size (minimum/optimal): 2048 bytes / 2048 bytes

while diskfs is detecting a different physical block size:
disk, _ := diskfs.OpenWithMode("/dev/sr0", diskfs.ReadOnly)
fs, _ := disk.GetFilesystem(0)

DEBU[0000] checking device: /dev/sr0
DEBU[0000] initDisk(): start
DEBU[0000] initDisk(): block device
DEBU[0000] initDisk(): logical block size 2048, physical block size 4096
DEBU[0000] trying fat32
DEBU[0000] fat32 failed: blocksize for FAT32 must be either 512 bytes or 0, not 2048
DEBU[0000] trying iso9660 with physical block size 4096

what could cause this? is there something i can tune so this can't happen?

related to #103

The sector sizes are determined here. If you dig in, you can see that it determines the device type (correctly, a block device, when you give it /dev/sr0), and then it calls getSectorSizes (I linked to the Unix one, since, as far as I can tell, you are running on a Unix-like system):
func getSectorSizes(f *os.File) (int64, int64, error) {
	fd := f.Fd()
	logicalSectorSize, err := unix.IoctlGetInt(int(fd), blksszGet)
	if err != nil {
		return 0, 0, fmt.Errorf("Unable to get device logical sector size: %v", err)
	}
	physicalSectorSize, err := unix.IoctlGetInt(int(fd), blkbszGet)
	if err != nil {
		return 0, 0, fmt.Errorf("Unable to get device physical sector size: %v", err)
	}
	return int64(logicalSectorSize), int64(physicalSectorSize), nil
}

I find it interesting that fdisk returned a different value than the ioctl calls, as I think that fdisk does those ioctl calls. It is worth looking into their source code.
The source for fdisk listing (fdisk -l) is here

Yup, code starts here:
/*
 * Get physical block device size. The BLKPBSZGET is supported since Linux
 * 2.6.32. For old kernels is probably the best to assume that physical sector
 * size is the same as logical sector size.
 *
 * Example:
 *
 * rc = blkdev_get_physector_size(fd, &physec);
 * if (rc || physec == 0) {
 *	rc = blkdev_get_sector_size(fd, &physec);
 *	if (rc)
 *		physec = DEFAULT_SECTOR_SIZE;
 * }
 */
#ifdef BLKPBSZGET
int blkdev_get_physector_size(int fd, int *sector_size)
{
	if (ioctl(fd, BLKPBSZGET, &sector_size) >= 0)
		return 0;
	return -1;
}
#else
int blkdev_get_physector_size(int fd __attribute__((__unused__)), int *sector_size)
{
	*sector_size = DEFAULT_SECTOR_SIZE;
	return 0;
}
#endif

And the definitions here:
/* block device topology ioctls, introduced in 2.6.32 (commit ac481c20) */
# ifndef BLKIOMIN
#  define BLKIOMIN   _IO(0x12,120)
#  define BLKIOOPT   _IO(0x12,121)
#  define BLKALIGNOFF _IO(0x12,122)
#  define BLKPBSZGET _IO(0x12,123)
# endif

I wonder if it is being misset on different platforms, because we have it fixed here:
	blkbszGet                        = 0x80081270

It is possible, especially in a virtualized environment (I saw you are running in qemu).
Hmm, oh hold on. Even with a fixed number, I am seeing it as we set it to the value of BLKBSZGET and not BLKPBSZGET, which are different values.
Let's start with a branch that sets it to the "right" fixed value for now, then let's see about getting it dynamically determined.

@mraerino can you try against this branch and see what you get? https://github.com/diskfs/go-diskfs/tree/pbsize

works! https://github.com/diskfs/go-diskfs/pull/107#pullrequestreview-627078245


Fix #1546 - consumes and produces reading

Fix #1546. Fix #1609. It overlaps and replaces https://github.com/swagger-api/swagger-core/pull/1652

Can you add a test here?

@webron tests added; PR slightly changes behaviour for comma separated “produces” and “consumes” values in @ApiOperation, @Api, @Produces and @Consumes annotations by splitting them into array of items fed into operation.produces and consumes; previously this values would be treated as single consume/produce e.g.
@Produces (“text/xml,text/html”)
in previous version would result in:
"consumes": [
   “text/xml,text/html”
  ]

While after  it will generate:
"consumes": [
    “text/xml”,
    “text/html”
  ],


v0.39.1 virt-operator ErrImagePull

Deployment failing
I tried installing kubevirt on minikube using quickstart guide https://kubevirt.io/quickstart_minikube/
there was an error in image pull of virt -operator
Is this a BUG REPORT or FEATURE REQUEST?:

Uncomment only one, leave it on its own line:

/kind bug

/kind enhancement

What happened:
What you expected to happen:
Deployment of kubevirt should be successful
How to reproduce it (as minimally and precisely as possible):
minikube addons enable kubevirt
kubevirt version:- v0.39.1
Anything else we need to know?:
Environment:

KubeVirt version (use virtctl version): v0.39.1

Kubernetes version (use kubectl version): Client Version: version.Info{Major:"1", Minor:"18", GitVersion:"v1.18.6", GitCommit:"dff82dc0de47299ab66c83c626e08b245ab19037", GitTreeState:"clean", BuildDate:"2020-07-15T16:58:53Z", GoVersion:"go1.13.9", Compiler:"gc", Platform:"linux/amd64"}
Server Version: version.Info{Major:"1", Minor:"20", GitVersion:"v1.20.2", GitCommit:"faecb196815e248d3ecfb03c680a4507229c2a56", GitTreeState:"clean", BuildDate:"2021-01-13T13:20:00Z", GoVersion:"go1.15.5", Compiler:"gc", Platform:"linux/amd64"}

VM or VMI specifications:

Cloud provider or hardware configuration:

OS (e.g. from /etc/os-release):

Kernel (e.g. uname -a):

Install tools:

Others:
Events:
Type     Reason     Age                 From               Message

Normal   Scheduled  20m                 default-scheduler  Successfully assigned kubevirt/virt-operator-6d5988c4bf-fk7j5 to minikube
Normal   Pulling    18m (x4 over 20m)   kubelet, minikube  Pulling image "quay.io/kubevirt/virt-operator@sha256:2429917d5fa837c5b3f7a8ebcc4df6f9574d3bbab8f9d47c45d1a71d150e0c7f"
Warning  Failed     18m (x4 over 20m)   kubelet, minikube  Failed to pull image "quay.io/kubevirt/virt-operator@sha256:2429917d5fa837c5b3f7a8ebcc4df6f9574d3bbab8f9d47c45d1a71d150e0c7f": rpc error: code = Unknown desc = Error response from daemon: manifest for quay.io/kubevirt/virt-operator@sha256:2429917d5fa837c5b3f7a8ebcc4df6f9574d3bbab8f9d47c45d1a71d150e0c7f not found: manifest unknown: manifest unknown
Warning  Failed     18m (x4 over 20m)   kubelet, minikube  Error: ErrImagePull
Warning  Failed     18m (x7 over 20m)   kubelet, minikube  Error: ImagePullBackOff
Normal   BackOff    25s (x83 over 20m)  kubelet, minikube  Back-off pulling image "quay.io/kubevirt/virt-operator@sha256:2429917d5fa837c5b3f7a8ebcc4df6f9574d3bbab8f9d47c45d1a71d150e0c7f"

Thanks for reporting this. I believe it is a duplicate of https://github.com/kubevirt/kubevirt/issues/5352


Add on_conflict in upsert function

In the current version upsert() does not support on_conflict.
I guess https://github.com/supabase-community/postgrest-py/issues/139 is required for this request to be implemented.
So here is a workaround for now, if someone comes across this issue:
def upsert(table_name, data, on_conflict, select):
    url = f'https://PROJECT_ID.supabase.co/rest/v1/{table_name}?on_conflict={on_conflict}&select={select}'
    headers = {"Content-Type": "application/json",
               "apiKey": "SECRET",
               "authorization": f'Bearer SECRET',
               "Prefer": "resolution=merge-duplicates,return=representation"}
    response = requests.post(url, headers=headers, json=data)
    return response.status_code

# example
response = upsert('automations', [{"name":"Germany"},{"name":"Italy"}], 'name', 'id')

The final solution could look like this:
data = supabase.table("countries").upsert([{"name":"Germany"}], on_conflict=["name"]).execute()

Thanks for the workaround!
Going to move this to the postgrest-py repo so we can track. Hopefully we can push on_conflcit option over the finish line next weekend


WIP DO NOT MERGE Support --os-auth-type / OS_AUTH_TYPE

support the OpenStack standard --os-auth-type / OS_AUTH_TYPE parameter, so that conflicting settings for password and token can be resolved.
Previously you could either set token or password and (qualified) username / user-id but never both because there is no defined precedence.

@notque what is missing is a unit test in pkg/cmd/cmd_test.go

Coverage decreased (-0.04%) to 56.53% when pulling afea48909484d958e438e841f34231fad7feaa21 on authtype into 932f4ded37265e32736ee60eef2f79d12328d7cf on master.

Coverage decreased (-0.04%) to 56.53% when pulling afea48909484d958e438e841f34231fad7feaa21 on authtype into 932f4ded37265e32736ee60eef2f79d12328d7cf on master.

You can also add application credentials support: https://github.com/gophercloud/gophercloud/pull/1224

@kayrus let's do this, but in a separate PR.

@notque @chuan137 looks good now. So if you feel comfortable, too, we can merge this, add a new release 1.0.4 and integrate


Test explorer not updating on adding/updating maestro test files/folders

Closed in https://github.com/Mastersam07/maestro-workbench/pull/2/commits/dd6db0d9d5d3de310734aed994991aeb0cf80388


WIP: replace some resources get via get_url with local copy

Minimize reliance on remote resources, use local ones instead.

This change is

@redhat-k8s-bot test please

Tested within the same CI with "kube_source_type=github-release kube_version=1.2.4". Kubernetes got deployed.

 What about to completely remove init/systemd and move its content under corresponding files directories of master/node/kubernetes under ansible? There is no difference if people get the service and config files from init/systemd or from ansible/roles/[master|node|kubernetes]/files.

The config files are meant for 1-node cluster anyway [1]. Though, it does not make much sense to use the ansible to deploy the 1-node cluster, it generates tokens and launches kubernetes-addons. Which is the missing piece when you install kubernetes from rpms. Still not a plug&play solution [2].
In short:
environ & config files

config and kubernetes.conf are under kubernetes role as it is common for all components
apiserver, controller-manager and scheduler belong to master components only (plus with kubelet in some cases)
kubelet and proxy belong to node components only

service files

kube-apiserver.service, kube-controller-manager.service, kube-scheduler.service belongs to master components (plus with kubelet.service in some cases)
kubelet.service and kube-proxy.service belongs to node components

Service files are not templated so far. Only configs are. And afaik, all the non-templated config files are used in distribution builds only.
But still, we will have at duplicates. Until resolved, let's not merge the PR.
[1] https://github.com/kubernetes/contrib/tree/master/init/systemd
[2] https://github.com/kubernetes/contrib/issues/1325

Or another option is to

move init/systemd under ansible directory
in every occurrence of the systemd service of config file check if "../init/systemd/[NAME.service|environ/NAME] file exists. If it does, copy it, if it does not, download it.

It will not change behaviour of the current PR. At the same time when each role gets publish in Ansible galaxy, it will work. And, there will be no (obvious) duplicates. But still the files get downloaded unless the roles are used directly from the contrib repository.
There is already some duplication between templated and non-templated config files. However, it should not matter as the non-templated configs are used purely for 1-node cluster and can be independent of templated ones (unless some fundamental thing needs to change).

If the other files outside of ansible have a different purpose and life-cycle, I don't care about duplicates. But a self contained ansible directory without unversioned files downloaded with http(s) is a good goal.

recomputing cla status...

[APPROVALNOTIFIER] The Following OWNERS Files Need Approval:

ansible

We suggest the following people:
cc @bprashanth
You can indicate your approval by writing /approve in a comment
You can cancel your approval by writing /approve cancelin a comment


Updating an item field replaces that field type with text

Your environment

SDK Version: 1.3.0

Connect Server Version:1.6.1

OS: Windows, Mac

Python Version: 3.10.10
What happened?
Updating an item field with purpose throw error while removing purpose will change the field type.
Update an item password like this:
  Field(
            id='password',
            label='password',
            purpose='PASSWORD',
            value='my password'),

will receive this error:
Logic: (Unable to parse Item), Validation: (too many "password" fields in Apply()), You can only use one "password" field.

Update an item without purpose will not throw an error but will change password type from password to text.
What did you expect to happen?
Updating an item with purpose should work and not throw error.
Steps to reproduce

https://user-images.githubusercontent.com/49488467/226744696-4b6ca2ea-f61a-4534-8ea5-0dcc5b3c7e8e.mp4

Hi @Nhat-agilbits👋
Thank you for reaching out! We are going to plan to work on that issue and try to fix it as soon as possible.


Non-verified accounts QRT'ing a verified account hides the QRT

When choosing to have Verified accounts set to Hide, it also hides tweets of those who are not verified, who are quote retweeting the tweet of someone who's verified.

That's intended behaviour, but also completely accidental - this was a bit of a joke feature to simulate what your timeline would look like without verified users, and if there were no verified users there would be no tweets to quote 🐱
I have plans for future versions to be more intentional about what happens when people share content you're ignoring via this extension or via Twitter's built-in blocking and muting functionality - e.g. if you've blocked a user and someone QRTs them, you see the quoter's comment but the quoted tweet says "This tweet is not available" (or something like that, don't have an example to hand) instead of hiding the entire thing from your Timeline.


[WIP]  Generate additional input files for the cyclodextrin challenge

[ ]  Add file generation notebook to the CD guest folder, generate PDB and SDF files (mol2 already provided
[ ]  Update the guest_files/ README description of what's provided and a description of the jupyter notebook used for additional guest file generation.
[ ]  Add file generation notebook to the CD host folder, make MOL2 and SDF files (PDBs already provided)
[ ]  Update the host_files/ README description of what's provided and include a description of the jupyter notebook used for the additional host file generation.

Did you forget to begin this one from the updated master or something? Looks like it was going to try to bring in other things at the same time...?

Yes. I closed it so that I can work from a clean branch


Lock uglify-es version to 3.0.20

r? @tcr
(tessel-highfive has picked a reviewer for you, use r? to override)

The failing tests are unrelated


Microphone doesn't work on Android

IMPORTANT: Before you create an issue, please take a look at our Issue Reporting Guide.
Briefly summarize your issue:

I try to build and run console sample app on android 11.
recorderCallback() is called but the raw data is zero.
Sample app make  below message.
"AndroidMicrophone:configureRecognizeModeFailed:reason=cannot set configuration,result=2"
I think it can not get valid configuration interface  to set recording preset(SL_ANDROID_RECORDING_PRESET_VOICE_RECOGNITION).
It doesn't have SLInterfaceID to set preset.
The value of interfaceSize may be 2 on AndroidSLESEngine.cpp.
And SetConfiguration() function should be called before calling Realize().
So in AndroidSLESEngine::createAudioRecorder() function, I think add SetConfiguration() function.
----- may be like below( AndroidSLESEngine.cpp) -----
constexpr uint32_t interfaceSize = 2;
const SLInterfaceID interfaceIDs[interfaceSize] = {SL_IID_ANDROIDSIMPLEBUFFERQUEUE, SL_IID_ANDROIDCONFIGURATION};
const SLboolean requiredInterfaces[interfaceSize] = {SL_BOOLEAN_TRUE, SL_BOOLEAN_TRUE};
...
SLAndroidConfigurationItf configItf;
auto result = (recorderObject)->GetInterface(recorderObject, SL_IID_ANDROIDCONFIGURATION, (void)&configItf);
if (result != SL_RESULT_SUCCESS) {
ACSDK_WARN(LX("createAndroidMicrophone").d("reason", "cannot get interface").d("result", result));
}
auto presetValue = SL_ANDROID_RECORDING_PRESET_VOICE_RECOGNITION;
result = (*configItf)->SetConfiguration(configItf, SL_ANDROID_KEY_RECORDING_PRESET,&presetValue, sizeof(SLuint32));
if (result != SL_RESULT_SUCCESS) {
ACSDK_WARN(LX("createAndroidMicrophone").d("reason", "cannot setconfiguration").d("result", result));
}
SLuint32 presetRetrieved = SL_ANDROID_RECORDING_PRESET_NONE;
SLuint32 presetSize = 2*sizeof(SLuint32);
result = (configItf)->GetConfiguration(configItf, SL_ANDROID_KEY_RECORDING_PRESET, &presetSize, (void)&presetRetrieved);
if (result != SL_RESULT_SUCCESS) {
ACSDK_WARN(LX("createAndroidMicrophone").d("FAIL", "cannot get preset").d("result", result));
}else {
ACSDK_WARN(LX("createAndroidMicrophone").d("OK", "get preset").d("preset", presetRetrieved));
}
return AndroidSLESObject::create(recorderObject);
---- code end -----
What is the expected behavior?

The raw data have some values on recorderCallback().
What behavior are you observing?

Below failed message and the microphone data is zero.
"AndroidMicrophone:configureRecognizeModeFailed:reason=cannot set configuration,result=2"
Provide the steps to reproduce the issue, if applicable:

build and run the console sample app.
Tell us about your environment:

What version of the AVS Device SDK are you using?
  <1.25.0>

Tell us what hardware you're using:

[ ] Desktop / Laptop
[ ] Raspberry Pi
[x] Other - tell us more: Android 11 and have microphone.

Tell us about your OS (Type & version):

[ ] Linux
[ ] MacOS
[ ] Raspbian Stretch
[ ] Raspbian Jessy
[x] Other - tell us more: Android 11

Hello @crazypp,
Could you please confirm whether the issue is identical to the issue being tracked here: https://github.com/alexa/avs-device-sdk/issues/1997
If so could you follow the recommended steps there?

I am closing this issue due to inactivity.  If you have further questions, please open a new ticket and refer to this closed one.


Autorest Regen Preview to 3.0.0-alpha.20231128.4 by Pan Shao from refs/pull/3910/head

Triggered from https://github.com/Azure/autorest.csharp/pull/3910

API change check
APIView has identified API level changes in this PR and created following API reviews.
Azure.Verticals.AgriFood.Farming


Improved UI/UX

Improve Input field and added some overall margins

Great @aliabbasnagari 👍🏼 I will merge it after review


New version: Google.Chrome version 109.0.5414.120

- [X] Have you signed the Contributor License Agreement?

[X] Have you checked that there aren't other open pull requests for the same manifest update/change?

[ ] Have you tested your manifest locally with winget install --manifest <path>?
[X] Does your manifest conform to the 1.4 schema?

@wingetbot run


Update README.md to make renaming clearer

Make it explicit that you also have to rename the package in the Project.toml file

Thanks!


fix(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/cbr/ibm (source)
module
patch
1.19.0 -> 1.19.1

terraform-ibm-modules/resource-group/ibm (source)
module
patch
1.1.4 -> 1.1.5

Release Notes

terraform-ibm-modules/terraform-ibm-cbr (terraform-ibm-modules/cbr/ibm)
v1.19.1
Compare Source
Bug Fixes

extend the required terraform version to < 1.7 (#407) (c62077d)

terraform-ibm-modules/terraform-ibm-resource-group (terraform-ibm-modules/resource-group/ibm)
v1.1.5
Compare Source
Bug Fixes

extend the required terraform version to < 1.7 (#581) (5552c65)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

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

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

/run pipeline

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

GitHub release
v2.0.8

Your semantic-release bot :package::rocket:


Add Sponsor badge to the GitHub Repo

Add Sponsor badge to the GitHub Repo with link to https://www.patreon.com/join/caniuse
REF: https://help.github.com/en/articles/displaying-a-sponsor-button-in-your-repository

❤ Thanks!

cool - you just need to turn it on in the settings now:
https://help.github.com/en/articles/displaying-a-sponsor-button-in-your-repository#displaying-a-sponsor-button-in-your-repository

Done, thanks again!

Facebook authentication coad


Build images using kube-cross

What would you like to be cleaned:
Use image registry.k8s.io/build-image/kube-cross instead of golang.
Why is this needed:
The build/test bots are hitting a lot of quota issues with docker registry.

/assign

One problem with registry.k8s.io/build-image/kube-cross is the image size which is approximately 7X bigger that the one we are currently using.

An alternative could be to run all the e2e in a single job,  resulting in replacing 4x6min jobs with 1x10min. (#879)
sample-run


"invalid cross-device link" fix

Fixes "invalid cross-device link" error thrown by os.Rename when this function is utilized to move stuff across different devices (filesystems)

Thanks for contributing! I'v also come across this issue before, and I'v just been calling the real rm before /bin/rm file1.... I wonder if theres a way to use os.Rename() instead of running the mv command.
Good to see others finding this useful, I will need to cleanup the code to make it more readable 😉


Use file name instead of "Autogenerated".

Current Behavior

Designs can be imported in multiple ways from MesheryUI, for some of the scenarios the file name or name provided as input is not respected and server assigns name as "Autogenerated".
Expected Behavior

Ensure filename/name provided as input is used instead of using "Autogenerated".
Screenshots/Logs

Environment

Host OS: Mac Linux Windows
Platform: Docker or Kubernetes
Meshery Server Version: stable-v
Meshery Client Version: stable-v

Contributor Guides and Handbook

🛠 Meshery Build & Release Strategy
📚 Instructions for contributing to documentation

Meshery documentation site and source

🎨 Wireframes and designs for Meshery UI in Figma (open invite)
🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

I would like to work on this issue

@102vansh Thanks for volunteering. Do you have Meshery UI and server up and running locally?

Yes

@102vansh, go ahead!

@102vansh are you woring on this issue?

@MUzairS15 In which way of importing a design caused the bug you mentioned?


Highlight links by underline in comments

Before:

After:

cc @posativ

Actually for me links are properly highlighted according to my blogs css.

@mdosch this pull request only specifies the rule to only be applied to the header (vor 2 Monaten) and allows an easier theming according the other styles that are loaded on the page. So actually it doesn't add a rule for underline but removes the one that overwrites the default rule for links

Looks good to me. Thanks!


Do not wrap readAllPages in breaker

Fixes #787
[delivers #152586910]

Coverage increased (+0.002%) to 91.307% when pulling 0c4bed9796990bad6f7506568829b5ee10aefb5b on SAP:fix-787 into a5900f7c94cacbafe0c1bc1cd782ec32413622e6 on cloudfoundry-incubator:master.

Coverage increased (+0.002%) to 91.307% when pulling 0c4bed9796990bad6f7506568829b5ee10aefb5b on SAP:fix-787 into a5900f7c94cacbafe0c1bc1cd782ec32413622e6 on cloudfoundry-incubator:master.

Coverage increased (+0.002%) to 91.307% when pulling 602d368cf8bc2f46ae9cc5b0f10b1b13e45077c3 on SAP:fix-787 into a5900f7c94cacbafe0c1bc1cd782ec32413622e6 on cloudfoundry-incubator:master.

Coverage increased (+0.002%) to 91.307% when pulling 71eca71562fb5f0706ac40c14452e0b34dd27eeb on SAP:fix-787 into a5900f7c94cacbafe0c1bc1cd782ec32413622e6 on cloudfoundry-incubator:master.

Coverage increased (+0.002%) to 91.307% when pulling 71eca71562fb5f0706ac40c14452e0b34dd27eeb on SAP:fix-787 into a5900f7c94cacbafe0c1bc1cd782ec32413622e6 on cloudfoundry-incubator:master.

Coverage increased (+0.002%) to 91.307% when pulling 71eca71562fb5f0706ac40c14452e0b34dd27eeb on SAP:fix-787 into a5900f7c94cacbafe0c1bc1cd782ec32413622e6 on cloudfoundry-incubator:master.


Map is not coming in my setup

Hi Their,
I have setup the backend and map application on my server. Both applications are running fine. But when i am seeing the map application, i am only seeing the html part. Nothing is happening their. Below is the URL where i setup my application. Can you please help me identifying the error which i am making.
http://<IP_ADDRESS>:8080/

What do you see in the web browser console? I'm guessing your Maps API key is not correct.

I am able to solve the issue myself. It is not mentioned anywhere in the documentation that we have to allow read permission in firebase real time database. It is working after that.


Update version name and code for v1.7.0

Updated version code to 17 and version name to 1.7.0.

@liveHarshit Restarted the build. It should get fixed. This keeps happening with CircleCI.

@liveHarshit Restarted the build. It should get fixed. This keeps happening with CircleCI.

Oh, yah it happens many times.


feat: Logging and Snapshot copy resources converted to standalone resource equivalents, MSV of Terraform raised to v1.3

Description
Add aws_redshift_logging and aws_redshift_snapshot_copy resources and removed deprecated configuration blocks for aws_redshift_cluster.logging and aws_redshift_cluster.snapshot_copy.
Motivation and Context
https://github.com/hashicorp/terraform-provider-aws/pull/36862
https://github.com/hashicorp/terraform-provider-aws/pull/36810
Breaking Changes
Yes. New logging and snapshot copy resources replace existing deprecated logging and snapshot_copy blocks in aws_redshift_cluster.
How Has This Been Tested?

[x] I have updated at least one of the examples/* to demonstrate and validate my change(s)
[x] I have tested and validated these changes using one or more of the provided examples/* projects

[x] I have executed pre-commit run -a on my pull request

Bumped terraform MSV to 1.3. 👍

This PR is included in version 5.5.0 :tada:

crap - started editing title in one window and then switched to a different window - should have been feat!: ... for breaking change

I've removed the v5.5.0 tag/release and replaced with a new v6.0.0 tag/release

Ah, my bad. I was wondering what feat! was used for and now I know. Thanks!


Gerrit support

You can find information about gerrit here https://www.gerritcodereview.com/
This changes does not provide direct support for the gerrit.
In my case it works through Jenkins which triggered by commit/comment on the gerrit and executes remote endpoint of the gerrit server (added as well)
The other concern - abstract gerrit provider does not has async methods, i.e. any methods (command execution in my case) would be sync in case of ASGI.
Gerrit has no code suggestions like github and I using separate patch server to store patches (it converts from suggestions).

looks good, just revert the default git back to 'github' :-)

/review

looks good, just revert the default git back to 'github' :-)

Thanks, done.


BevyRapier3D example doesn't work

The 0.10.0 example uses the "from_agent_and_bounds" helper method that is set to release with 0.11.0. Naturally, it doesn't compile without this.
It also throws an Unsatisfied Trait Bound on bevy_rapier3d::geometry::collider; doesn't satisfy _: OxidizedCollider.
There is a comment noting this "This is only compiled and available when the "rapier" feature is enabled.", however even with the rapier feature enabled this issue still occurs.

It looks like the issue is tied to the fact that Rapier updated to 0.26.0, mismatching the version used in Oxidized_navigation and throwing things out of sync. Downgrading Rapier to 0.25 did away with that error, but that's not a great long-term solution.

Of course it doesn't work. 0.11.0 where I got the example is unreleased; my project imports 0.10. I'm silly.
ANYWAY, fork is made. See #25 .


[APPM-1088] Mobile Devices modal is not responsive in App Manager when loading mobile devices

Mobile Devices modal is not responsive in App Manager when loading mobile devices.Reference: https://wso2.org/jira/browse/APPM-1088

App manager component is being rewritten for IoT server, and this will be addressed there. Hence closing the issue.