[FEA] Support offset parameter in TakeOrderedAndProject

Parameter offset is introduced in TakeOrderedAndProjectExec, since spark-3.4.0.
case class TakeOrderedAndProjectExec(
    limit: Int,
    sortOrder: Seq[SortOrder],
    projectList: Seq[NamedExpression],
    child: SparkPlan,
    offset: Int = 0) extends UnaryExecNode { ... }

Correspondingly, we need to support it in spark-rapids.
    exec[TakeOrderedAndProjectExec](
      "Take the first limit elements as defined by the sortOrder, and do projection if needed",
      // The SortOrder TypeSig will govern what types can actually be used as sorting key data type.
      // The types below are allowed as inputs and outputs.
      ExecChecks((pluginSupportedOrderableSig + TypeSig.DECIMAL_128 +
          TypeSig.ARRAY + TypeSig.STRUCT + TypeSig.MAP).nested(), TypeSig.all),
      (takeExec, conf, p, r) =>
        new SparkPlanMeta[TakeOrderedAndProjectExec](takeExec, conf, p, r) {
          val sortOrder: Seq[BaseExprMeta[SortOrder]] =
            takeExec.sortOrder.map(GpuOverrides.wrapExpr(_, conf, Some(this)))
          val projectList: Seq[BaseExprMeta[NamedExpression]] =
            takeExec.projectList.map(GpuOverrides.wrapExpr(_, conf, Some(this)))
          override val childExprs: Seq[BaseExprMeta[_]] = sortOrder ++ projectList

          override def convertToGpu(): GpuExec = {
            // To avoid metrics confusion we split a single stage up into multiple parts but only
            // if there are multiple partitions to make it worth doing.
            val so = sortOrder.map(_.convertToGpu().asInstanceOf[SortOrder])
            if (takeExec.child.outputPartitioning.numPartitions == 1) {
              GpuTopN(takeExec.limit, so,
                projectList.map(_.convertToGpu().asInstanceOf[NamedExpression]),
                childPlans.head.convertIfNeeded())(takeExec.sortOrder)
            } else {
              GpuTopN(
                takeExec.limit,
                so,
                projectList.map(_.convertToGpu().asInstanceOf[NamedExpression]),
                GpuShuffleExchangeExec(
                  GpuSinglePartitioning,
                  GpuTopN(
                    takeExec.limit,
                    so,
                    takeExec.child.output,
                    childPlans.head.convertIfNeeded())(takeExec.sortOrder),
                  ENSURE_REQUIREMENTS
                )(SinglePartition)
              )(takeExec.sortOrder)
            }
          }
        }),

Just like with GlobalLimit we need to first make sure that we fall back to the CPU if offset is not 0.  That is a P1 for 3.4.0 support. After that we can look at implementing the functionality, but it is not as critical.

The current GpuTakeOrderedAndProject will keep the data in a single batch. This is because we made the assumption that the size of limit would be small.  But with an offset the number of rows we have to look at may be much larger. We have to keep around offset + limit because we cannot drop anything in the offset rows at the beginning of the sorted buffer until we know there is no way for it to bubble up to be a part of limit section. The only way we could know that is by knowing how many rows are left to be sorted.  This is because offset is often used for pagination. Look at the first 1000 rows. Then look at the next 1000 rows, and so on.
This can probably be done as a follow on issue and should probably only happen if we see that the size of the data we keep ends up going over the target size for a batch.

This is not a p0. It's a p1 I think and also per @revans2 comment

@revans2
I am confused about this issue. When I run the following in Spark-3.4.0 it seems to be on the GPU granted I don't see TakeOrderedAndProjectExec in the GPU plan but I do see GpuTopN and the entire plan is on the GPU after I set spark.rapids.sql.exec.CollectLimitExec to true and the results look fine. Is there work that's left to be done here?
scala> val df = Range(0,100).flatMap(i => Seq(i)).toDF()
df: org.apache.spark.sql.DataFrame = [value: int]

scala> df.createOrReplaceTempView("test_table")

scala>  spark.conf.set("spark.rapids.sql.exec.CollectLimitExec", "true")

scala> spark.sql("select * from test_table order by value limit 10 offset 2").show()
23/04/28 17:14:55 WARN GpuOverrides:
*Exec <CollectLimitExec> will run on GPU
  *Partitioning <SinglePartition$> will run on GPU
  *Exec <ProjectExec> will run on GPU
    *Expression <Alias> cast(value#1 as string) AS value#41 will run on GPU
      *Expression <Cast> cast(value#1 as string) will run on GPU
    *Exec <GlobalLimitExec> will run on GPU
      *Exec <ShuffleExchangeExec> will run on GPU
        *Partitioning <SinglePartition$> will run on GPU
        *Exec <SortExec> will run on GPU
          *Expression <SortOrder> value#1 ASC NULLS FIRST will run on GPU
          !Exec <ShuffleExchangeExec> cannot run on GPU because Columnar exchange without columnar children is inefficient
            @Partitioning <RangePartitioning> could run on GPU
              @Expression <SortOrder> value#1 ASC NULLS FIRST could run on GPU
                @Expression <AttributeReference> value#1 could run on GPU
            ! <LocalTableScanExec> cannot run on GPU because GPU does not currently support the operator class org.apache.spark.sql.execution.LocalTableScanExec
              @Expression <AttributeReference> value#1 could run on GPU

23/04/28 17:14:55 WARN GpuOverrides:
*Exec <CollectLimitExec> will run on GPU
  *Partitioning <SinglePartition$> will run on GPU
  *Exec <ProjectExec> will run on GPU
    *Expression <Alias> cast(value#1 as string) AS value#41 will run on GPU
      *Expression <Cast> cast(value#1 as string) will run on GPU
    *Exec <GlobalLimitExec> will run on GPU
      *Exec <ShuffleExchangeExec> will run on GPU
        *Partitioning <SinglePartition$> will run on GPU
        *Exec <SortExec> will run on GPU
          *Expression <SortOrder> value#1 ASC NULLS FIRST will run on GPU
          !Exec <ShuffleExchangeExec> cannot run on GPU because Columnar exchange without columnar children is inefficient
            @Partitioning <RangePartitioning> could run on GPU
              @Expression <SortOrder> value#1 ASC NULLS FIRST could run on GPU
                @Expression <AttributeReference> value#1 could run on GPU
            ! <LocalTableScanExec> cannot run on GPU because GPU does not currently support the operator class org.apache.spark.sql.execution.LocalTableScanExec
              @Expression <AttributeReference> value#1 could run on GPU

23/04/28 17:14:55 WARN GpuOverrides:
*Exec <CollectLimitExec> will run on GPU
  *Partitioning <SinglePartition$> will run on GPU
  *Exec <ProjectExec> will run on GPU
    *Expression <Alias> cast(value#1 as string) AS value#41 will run on GPU
      *Expression <Cast> cast(value#1 as string) will run on GPU
    *Exec <GlobalLimitExec> will run on GPU
      *Exec <ShuffleExchangeExec> will run on GPU
        *Partitioning <SinglePartition$> will run on GPU
        *Exec <SortExec> will run on GPU
          *Expression <SortOrder> value#1 ASC NULLS FIRST will run on GPU
          !Exec <ShuffleExchangeExec> cannot run on GPU because Columnar exchange without columnar children is inefficient
            @Partitioning <RangePartitioning> could run on GPU
              @Expression <SortOrder> value#1 ASC NULLS FIRST could run on GPU
                @Expression <AttributeReference> value#1 could run on GPU
            ! <LocalTableScanExec> cannot run on GPU because GPU does not currently support the operator class org.apache.spark.sql.execution.LocalTableScanExec
              @Expression <AttributeReference> value#1 could run on GPU

23/04/28 17:14:55 WARN GpuOverrides:
!Exec <ShuffleExchangeExec> cannot run on GPU because Columnar exchange without columnar children is inefficient
  @Partitioning <RangePartitioning> could run on GPU
    @Expression <SortOrder> value#1 ASC NULLS FIRST could run on GPU
      @Expression <AttributeReference> value#1 could run on GPU
  ! <LocalTableScanExec> cannot run on GPU because GPU does not currently support the operator class org.apache.spark.sql.execution.LocalTableScanExec
    @Expression <AttributeReference> value#1 could run on GPU

23/04/28 17:14:55 WARN GpuOverrides:
*Exec <CollectLimitExec> will run on GPU
  *Partitioning <SinglePartition$> will run on GPU
  *Exec <ProjectExec> will run on GPU
    *Expression <Alias> cast(value#1 as string) AS value#41 will run on GPU
      *Expression <Cast> cast(value#1 as string) will run on GPU
    *Exec <GlobalLimitExec> will run on GPU
      *Exec <ShuffleExchangeExec> will run on GPU
        *Partitioning <SinglePartition$> will run on GPU
        *Exec <SortExec> will run on GPU
          *Expression <SortOrder> value#1 ASC NULLS FIRST will run on GPU

23/04/28 17:14:55 WARN GpuOverrides:
*Exec <ShuffleExchangeExec> will run on GPU
  *Partitioning <SinglePartition$> will run on GPU
  *Exec <SortExec> will run on GPU
    *Expression <SortOrder> value#1 ASC NULLS FIRST will run on GPU
    !Exec <AQEShuffleReadExec> cannot run on GPU because Unable to replace CustomShuffleReader due to child not being columnar

23/04/28 17:14:55 WARN GpuOverrides:
*Exec <CollectLimitExec> will run on GPU
  *Partitioning <SinglePartition$> will run on GPU
  *Exec <ProjectExec> will run on GPU
    *Expression <Alias> cast(value#1 as string) AS value#41 will run on GPU
      *Expression <Cast> cast(value#1 as string) will run on GPU
    *Exec <GlobalLimitExec> will run on GPU

23/04/28 17:14:55 WARN GpuOverrides:
*Exec <CollectLimitExec> will run on GPU
  *Partitioning <SinglePartition$> will run on GPU
  *Exec <ProjectExec> will run on GPU
    *Expression <Alias> cast(value#1 as string) AS value#41 will run on GPU
      *Expression <Cast> cast(value#1 as string) will run on GPU
    *Exec <GlobalLimitExec> will run on GPU

+-----+
|value|
+-----+
|    2|
|    3|
|    4|
|    5|
|    6|
|    7|
|    8|
|    9|
|   10|
|   11|
+-----+

@razajafri
https://github.com/NVIDIA/spark-rapids/blob/54f77b084a6b23b68af5e980d0161cf8f4ef1f1a/sql-plugin/src/main/scala/com/nvidia/spark/rapids/GpuOverrides.scala#L3720-L3759
TakeOrderedAndProject internally does a shuffle to a single task as a part of the computation. When we designed the GPU replacement we split it up into TopN -> Shuffle -> TopN so that we didn't have to replicate a lot of the work we did for shuffle into our own TakeOrderedAndProject implementation.
I don't know what is happening here because the plan does not appear to include a TakeOrderedAndProject with an offset in it. If Spark still supports offset in the code for TakeOrderedAndProject then we need to support it. We just need to spend some more time to understand in which conditions it will use the offset and in which cases it will not. If Spark no longer supports that code, then we don't need to implement it.


Update the Prototype Kit Heroku stack to Heroku-18 (before November 2)

What
Update the Prototype Kit Heroku stack to Heroku-18, following Heroku's guidance on updating to the latest stack.
Why
The current Cedar-14 stack is end-of-life (EOL). After November 2, apps on Cedar-14 will continue to run, but may not build or receive security updates.
Further detail
There will be two 'build brownouts' on October 19 (17:00 - 21:00 UTC) and October 26 (14:00 - 18:00 UTC), designed to raise awareness of the EOL.

EOL Announcement
EOL FAQ

Done when…

[ ] Prototype Kit docs (https://govuk-prototype-kit.herokuapp.com/) run on the Heroku-18 stack
[x] PRs against the prototype kit repo run on the Heroku-18 stack

Production docs site now running on Heroku-18:
➜ heroku stack -a govuk-prototype-kit
=== ⬢ govuk-prototype-kit Available Stacks
  container
  heroku-16
* heroku-18
  heroku-20


Support collection methods on association collections

Currently, if I have a model with a 'hasMany' association, the association lacks any of the collection methods that I've added to the model with the '@methodName' syntax and is a bare array.

Closed due to inactivity.


Leetcode Question Number 2013

Leetcode Question Number 2013 solution needs to be added in the repo .
The link for the question is this.
Accepted Languages C++ / JAVA .
You Only need to add the  class Solution in the file.
Make Sure you have read the contributions guidelines before creating an PR.

Please assign this issue to me


Don't Merge! PR real time test for webhook case

Docs Build status updates of commit 09cae6d:
:clock10: Pending: waiting for processors (27 builds ahead of you)
:warning: Docs Build is busy, currently there are 27 builds ahead of this one, for more information you can view the Build queue graph on the Docs Portal.

Docs Build status updates of commit 09cae6d:
:clock10: Full build: the average full build time is 7 min(s) 1 sec(s), this is based on the last 30 build(s) for this repository.

Docs Build status updates of commit 09cae6d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

docs-ref-conceptual/testing.md
:warning:Warning
View (azure-cli-latest) View (azure-cli-2017-03-09-profile) View (azure-cli-2018-03-01-hybrid)
Details

docs-ref-conceptual/testing.md

[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.
[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.
[Warning] Missing attribute: either ms.prod or ms.service is required. Use ms.prod for on-premise products, or ms.service for cloud services.
[Warning] Missing attribute: title.

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


[BUG] invalid text on "send now" button for failed 1-time payment

Steps to reproduce
Wallet setup

Create a USD sender test wallet account in rafiki.money.
Connect the wallet address to WM extension (I have the extension pinned to the browser)

Visit websites

Visit non-monetized website (google.com) & try to make a 1-time payment. Result: payment failed, extension shows error message "This website is not monetized."
Open a a WMized website (alexlakatos.com) in a new tab & successfully make a 1-time payment for $1.
Change tabs back to a non-monetized website (google.com).

Try something funky
4. Use a keyboard shortcut to open the browser dev tools (on Windows, I used Ctrl+Shift+J).
5. Move mouse to the dev tools window, and click the Elements tab.
6. Move mouse back to Web browser that still has a non-monetized website open (google.com), then click on the WM ext. to open the extension.
7. Enter an amount to make a 1-time payment to the non-monetized website (this fails with the msg "This website is not monetized").
8. Move the mouse to click back in the Dev Tools window, then search for a term (I used the keyboard shortcut Ctrl+F to search, then typed "monetization" and pressed enter, on the keyboard).
9. Move the mouse back to the WM extension pop-up, which is still open in Web browser with a one-time amount, then click on "Send now".
Expected result
The 1-time payment to a non-monetized website must fail and the WM extension must display the message "This website is not monetized"
Actual result
The text on the "Send now" button changes to "Send nowPayment successful"

Screenshots or videos
Watch video from 0 until 1 minute 41seconds: link to video
Additional context
This issue is linked to #292
Operating system
Windows
Operating system version
10
Browsers
Chrome
Browser version
Version 126.0.6478.114 (Official Build) (64-bit)
Extension version
alpha 1 (it says v1.0.1)

Closed: no longer applicable, because the user cannot see the on time payment UI anymore on non-monetized pages.


UHF-5610: Chat Leijuke

Chat Leijuke UHF-5610
A unified chat launcher button with cookie blocked script loading.
Works with the eu_cookie_compliance module to check cookies and allow functional chat cookies implicitly on user clicking on the Leijuke.
Loads the previously implemented chat scripts and replaces their launcher buttons.
Works with multiple (tested with 2) chats/chatbots on the same page such as smartti + genesys on KYMP pysäköinti.
What was done

A new Chat Leijuke custom block with chat selection and optional title, that loads the leijuke library and sets js vars with the library for the selected chat.
A new js class that is able to load a library, facilitates interaction with a cookie manager and an adapter to interact with 3rd party chat widgets. Has own cookies per chat to remember chat state open/closed.
A drupal behaviour to instantiate the Leijuke class with eu_cookie_compliance compliant cookie manager class and a chat adapter based on the selection.
Adapter classes for the existing chat widgets.
Eu cookie compliance settings updated via hook.

How to install

Make sure your instance is up and running on latest dev branch.

git pull origin dev
make fresh

Update the Helfi Platform config

composer require drupal/helfi_platform_config:dev-UHF-6277_chat_leijuke_poc

Run make drush-updb drush-cr

How to test

Create a new block instance from Chat Leijuke and select a chat script to load. Optionally add a title for the leijuke (translatable).
Check the eu_cookie_compliance settings. New category 'chat' is setup and the cookie settings include:

'chat:_genesys.widgets.*'

'chat:leijuke.*'

Load page with block present and observe behavior of Leijuke and cookies.

Other PRs

https://github.com/City-of-Helsinki/drupal-hdbt/pull/396

Maybe genesys_chat could be renamed to genesys_kymp at this point. The reason for this arbitrary naming is that the KYMP was the first (and, at the time, only) instance where the chat was introduced.


Ctrl-Shift-click as a Clone shortcut

I think it would be convenient to have the ability to create and place clones of an object with one action. This would be really handy for placing collectibles or other objects all around your scene quickly.
Currently you must select an object in a scene, hit Ctrl-D then drag the cloned item to its position; this shortcut would let you place a clone of the selected object where you've clicked. This would also extend to holding down the click so you could line up the object correctly before placing it.
In practice it would look something like this (taken from GameMaker Studio):

Any discussion or thoughts are welcome.

Or maybe keep Ctrl + D for cloning, but instead make the position depend on the mouse, like in the shown gif.

I think it would feel more natural to place clones with a click than having it be a purely-keyboard shortcut, given that you're using the mouse to set the position. Also, I'd imagine the ability to just clone nodes without changing its position would still be needed in a few cases, so Ctrl-D should probably just keep its current functionality.

Well, maybe we can get a clone-to-position shortcut (probably defaulting to Shift-Ctrl-D)?

Yeah, that'll work pretty well.

sorry for the nitpicking, but strictly speakiung, duplicating an object doesn't produce a clone but an independent copy (unless you are duplicating an instanced scene and make sure changes are applied to all instances of that scene in your tree). so maybe creating a duplicate or a clone should be treated/implemented as two seperate things?

it's not a bad idea, for ctrl-shift-d, but we have the problem of
discoverability, as it's not a shortcut we can put on a menu
On Mon, Jun 26, 2017 at 1:37 PM, novemberist<EMAIL_ADDRESS>wrote:

sorry for the nitpicking, but strictly speakiung, duplicating an object
doesn't produce a clone but an independent copy (unless you are duplicating
an instanced scene and make sure changes are applied to all instances of
that scene in your tree). so maybe creating a duplicate or a clone should
be treated/implemented as two seperate things?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/godotengine/godot/issues/9389#issuecomment-311113500,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF-Z21Je0oUSLr8dg3qAhnSkAFk_cOrwks5sH944gaJpZM4OE058
.

ok, ctrl+shift for in-place duplication and ctrl for in-cursor :upside_down_face:

This would be awesome as it could be used as node-"paint" tool! There are even plugins like Andreas Esau's Asset Dropper  which do a similar (if somewhat more complex) thing. With cursor-node-duplication as @homer666 suggests and a little extra work those kind of plugins would not be needed anymore or extremely simplified.

First of all thank you for your report and sorry for the delay.
We released Godot 3.0 in January 2018 after 18 months of work, fixing many old issues either directly, or by obsoleting/replacing the features they were referring to.
We still have hundreds of issues whose relevance/reproducibility needs to be checked against the 3.0.2 (https://godotengine.org/download), and that's where you can help us.
Could you check if the issue that you described initially is still relevant/reproducible in Godot 3.0 or any newer version, and comment about its current status here?
For bug reports, please also make sure that the issue contains detailed steps to reproduce the bug and, if possible, a zipped project that can be used to reproduce it right away. This greatly speeds up debugging and bugfixing tasks for our contributors.
Our Bugsquad will review this issue more in-depth in 15 days, and potentially close it if its relevance could not be confirmed.
Thanks in advance.
Note: This message is being copy-pasted to many "stale" issues (90+ days without activity). It might happen that it is not meaningful for this specific issue or appears oblivious of the issue's context, if so please comment to notify the Bugsquad about it.


Split boots effects

It could be very interesting to have a configurable setting to enable splitting the boots effects.
Currently, having a boots tier give you the effect of all previous tiers.
Enabling this setting would mean:

Long Boots would only prevent movement hindrances in sand / water puddles
Merman Boots would only enable swimming in water
Charm Boots would only negate all kind of terrain damage

Added in e9a17ab


Update requirements.txt

urllib3 1.25.8 was throwing an error and found 1.26+ is required.

Thank you for this PR @brianp9906 - and sorry it took so long for me to review this.  I've confirmed that the allowed_methods parameter that we use in requests to handle retry logic, and subsequently used in urllib3, was introduced in 1.26.0


Incorrect stat priority for toughness

Hey.
When I tried to use the stat priority feature to auto choose stats on an item for me for a zDps (monk but not relevant) build, I noticed it valued armor higher than dexterity. Meaning, you would get 516 armor on shoulders and belt even though dexterity goes up to 650 and thus gives 650 armor.
Step to reproduce: equip belt (crimson) and shoulders (pauldron of the skeleton king), set the stat priority to CDR > RCR > toughness. Items should roll with Vitality, Armor, Cdr, Rcr for shoulders and vitality, all rez, life%, armor for belt.
While this is not a high priority change, I don't think it would be too difficult to account for.

Stat priority feature? Haven't heard of it >.>

This is what I'm talking about. Is it not called "stat priority" ?

Oh never mind me, I forgot I added that :D
I'll look into it.


Strengthen the hard fork history tests, fix bug

We now actually test that we can accurately predict future events. This
showed up an off-by-one error in the implementation. Yay for tests :)

bors r+

Reviewed over Meet.


Selection for module missing

The drop down to select themes does not appear in the setting options box.

System: FFG Star Wars 1.801
Forge:  10.291

The way Storyteller handles themes changed a while back, and I need to rewrite this extension for it to work. I cannot commit to a timeline.


appveyor: increase verbosity, get scripthookv sdk

I've just added a commit which creates the zip file for release.


Added leo

hiiiii Leo!

(I hope its alright I merged this)

The plan was to merge it. I think Max was maybe going to look at it first but props for taking initiative.

ah, sorry for jumping the gun Max


Accessing CuDNN autotuner in built-in Keras

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow): No
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):

$ uname -mrs
Linux 4.14.0-49.el7a.ppc64le ppc64le

TensorFlow installed from (source or binary): Binary
TensorFlow version (use command below): 1.6.0
Python version: 2.7.5
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version: CUDA 9.1, CuDNN 7.0.5
GPU model and memory: Tesla V100:

== nvidia-smi ===================================================
Thu Apr 26 18:35:34 2018
+-----------------------------------------------------------------------------+
| NVIDIA-SMI 396.19                 Driver Version: 396.19                    |
|-------------------------------+----------------------+----------------------+
| GPU  Name        Persistence-M| Bus-Id        Disp.A | Volatile Uncorr. ECC |
| Fan  Temp  Perf  Pwr:Usage/Cap|         Memory-Usage | GPU-Util  Compute M. |
|===============================+======================+======================|
|   0  Tesla V100-SXM2...  Off  | 00000004:04:00.0 Off |                    0 |
| N/A   29C    P0    52W / 300W |      0MiB / 15360MiB |      0%   E. Process |
+-------------------------------+----------------------+----------------------+

Exact command to reproduce: N/A

We would like to be able to have access to the CuDNN autotuner in tf.keras module to access optimal algorithms for a given hardware (or, perhaps passing a custom convolutional algorithm from config). In TensorFlow, I can specify to use the CuDNN autotuner by setting:
os.environ['TF_CUDNN_USE_AUTOTUNE'] = "1"

(currently enabled by default), which improves performance significantly on Volta GPUs and especially with FP16.
However, I am unable to access this performance improvement when running pure tf.keras, where setting this environmental variable does not have any effect.
Source code / logs
Following simple example could be used to reproduce the issue: https://gist.github.com/ASvyatkovskiy/8d1dd622e447d9d8de1ec4e238e0dbaa

Thank you for your post. We noticed you have not filled out the following field in the issue template. Could you update them if they are relevant in your case, or leave them as N/A? Thanks.
Exact command to reproduce

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

Check also https://github.com/keras-team/keras/issues/9825 /cc @fchollet

[Off-topic] @fchollet I note that more and more issues are a little bit duplicated after keras gone in the tf namespace. I think it is quite natural that people are a little bit cofused on where to report issues. I think that TF is a special case backend that it is also a sort of frontend in tf.keras. Of course keras is the upstream but this frontend/backend/frontend loop it is confusing many users.

It has been 18 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 33 days with no activity and the awaiting response label was assigned. Is this still an issue?

We are closing this issue for now due to lack of activity. Please comment if this is still an issue for you. Thanks!

Please reopen.


Clean up code

This PR resolve #420 with fix bellow:

Use pointers for functions in components
Reduce redundancy in layout.Widget
Use constant instead of unclear int in restore page
Remove redundant cast in restore page
Wrap all const a single section
Change interface to func in cancel button
Remove goroutine in updateStates

Several layout.Widget wrappers are redundant. Example:

layout.Rigid(func(gtx C) D {
return leftWidget(gtx)
}),
layout.Flexed(1, func(gtx C) D {
return layout.E.Layout(gtx, func(gtx C) D {
return rightWidget(gtx)
})
}),
reduces to just
layout.Rigid(leftWidget),
layout.Flexed(1, func(gtx C) D {
return layout.E.Layout(gtx, rightWidget)
}),
In general, a wrapper around a single return statement can often be omitted.


xcodebuild: WARNING: Using the first of multiple matching destinations

When running a desktop mac app, the console is providing red a warning, but it is not clear what it means or how to make it happy. :)

katelovett:~/dev/plane_test  $ flutter run -d macos
Launching lib/main.dart on macOS in debug mode...
--- xcodebuild: WARNING: Using the first of multiple matching destinations:
{ platform:macOS, arch:x86_64, id:B4D64537-CECE-5213-ADE1-01A624EF94A7 }
{ platform:macOS, name:Any Mac }
Building macOS application...
Syncing files to device macOS...                                    79ms

Flutter run key commands.
r Hot reload. 🔥🔥🔥
R Hot restart.
h List all available interactive commands.
d Detach (terminate "flutter run" but leave application running).
c Clear the screen
q Quit (terminate the application on the device).

💪 Running with sound null safety 💪

An Observatory debugger and profiler on macOS is available at:
http://<IP_ADDRESS>:53774/MHrjsyaSJC0=/
The Flutter DevTools debugger and profiler on macOS is available at:
http://<IP_ADDRESS>:9101?uri=http://<IP_ADDRESS>:53774/MHrjsyaSJC0=/

 flutter doctor
katelovett:~/dev/plane_test  $ flutter doctor -v
[✓] Flutter (Channel master, 2.6.0-12.0.pre.979, on macOS 12.0.1 21A559 darwin-x64, locale
    en-US)
    • Flutter version 2.6.0-12.0.pre.979 at /Users/katelovett/github/flutter
    • Upstream repository git@github.com:flutter/flutter.git
    • Framework revision be0e388687 (4 hours ago), 2021-12-09 12:04:06 -0500
    • Engine revision 59778ae91c
    • Dart version 2.16.0 (build 2.16.0-85.0.dev)
    • DevTools version 2.9.1

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0-rc1)
    • Android SDK at /Users/katelovett/Library/Android/sdk
    • Platform android-32, build-tools 32.0.0-rc1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[!] Xcode - develop for iOS and macOS (Xcode 13.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    ! CocoaPods 1.8.4 out of date (1.10.0 is recommended).
        CocoaPods is used to retrieve the iOS and macOS platform side's plugin code that
        responds to your plugin usage on the Dart side.
        Without CocoaPods, plugins will not work on iOS or macOS.
        For more info, see https://flutter.dev/platform-plugins
      To upgrade see https://guides.cocoapods.org/using/getting-started.html#installation
      for instructions.

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] IntelliJ IDEA Ultimate Edition (version 2021.2.3)
    • IntelliJ at /Applications/IntelliJ UE (stable).app
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart

[✓] VS Code (version 1.63.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (2 available)
    • macOS (desktop) • macos  • darwin-x64     • macOS 12.0.1 21A559 darwin-x64
    • Chrome (web)    • chrome • web-javascript • Google Chrome 96.0.4664.93

! Doctor found issues in 1 category.

Issue is reproducible on the latest stable and master channel

logs

Launching lib/main.dart on macOS in debug mode...
Running pod install...                                           1,683ms
--- xcodebuild: WARNING: Using the first of multiple matching destinations:
{ platform:macOS, arch:x86_64, id:B37E47AC-E02C-5EAA-A451-F92DB0C11EFE }
{ platform:macOS, name:Any Mac }
Building macOS application...
Syncing files to device macOS...                                   188ms

Flutter run key commands.
r Hot reload. 🔥🔥🔥
R Hot restart.
h List all available interactive commands.
d Detach (terminate "flutter run" but leave application running).
c Clear the screen
q Quit (terminate the application on the device).

💪 Running with sound null safety 💪

An Observatory debugger and profiler on macOS is available at: http://<IP_ADDRESS>:60984/NNNyKU6x9cQ=/
The Flutter DevTools debugger and profiler on macOS is available at:
http://<IP_ADDRESS>:9102?uri=http://<IP_ADDRESS>:60984/NNNyKU6x9cQ=/

flutter doctor -v
[✓] Flutter (Channel stable, 2.8.1, on macOS 12.1 21C52 darwin-x64, locale en-US)
    • Flutter version 2.8.1 at /Users/flo/development/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 77d935af4d (6 days ago), 2021-12-16 08:37:33 -0800
    • Engine revision 890a5fca2e
    • Dart version 2.15.1

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/flo/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] VS Code (version 1.62.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (4 available)
    • sdk gphone x86 (mobile) • emulator-5554                        • android-x86    • Android 11 (API 30) (emulator)
    • iPhone XR (mobile)      • 52310930-1602-4A61-BE82-9B18989453E0 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-15-0 (simulator)
    • macOS (desktop)         • macos                                • darwin-x64     • macOS 12.1 21C52 darwin-x64
    • Chrome (web)            • chrome                               • web-javascript • Google Chrome 93.0.4577.63

• No issues found!

[✓] Flutter (Channel master, 2.9.0-1.0.pre.156, on macOS 12.1 21C52 darwin-x64, locale en-US)
    • Flutter version 2.9.0-1.0.pre.156 at /Users/flo/fvm/versions/master
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 27ad488747 (6 hours ago), 2021-12-21 22:39:19 -0500
    • Engine revision 4eabb883af
    • Dart version 2.16.0 (build 2.16.0-126.0.dev)
    • DevTools version 2.9.2

[✓] Android toolchain - develop for Android devices (Android SDK version 31.0.0)
    • Android SDK at /Users/flo/Library/Android/sdk
    • Platform android-31, build-tools 31.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.0

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2020.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.10+0-b96-7281165)

[✓] VS Code (version 1.62.3)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.29.0

[✓] Connected device (4 available)
    • sdk gphone x86 (mobile) • emulator-5554                        • android-x86    • Android 11 (API 30) (emulator)
    • iPhone XR (mobile)      • 52310930-1602-4A61-BE82-9B18989453E0 • ios            • com.apple.CoreSimulator.SimRuntime.iOS-15-0 (simulator)
    • macOS (desktop)         • macos                                • darwin-x64     • macOS 12.1 21C52 darwin-x64
    • Chrome (web)            • chrome                               • web-javascript • Google Chrome 93.0.4577.63

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!


Add missing Base64 transformation statements to parser

This solves solves the Invalid action: t:Base64Decode error if one tries to use the the Base64  transformations with libModSecurity. Should fix #1632 and #1649
The transformation itself was already implemented but it was missing the references in the parser.

Merged tnks


Correction function init In State

Allow parameter in function init

I don't see any optional arguments in the Phaser source. Is this correct behaviour?

Hi
In JS, you can pass arguments to the start function when launching a state.
You can then get then have them in the init of a state.
Currently you can't do this with Haxe/phaser.
This behaviour is clearer in the source code.
@param {...*} parameter - Additional parameters that will be passed to the State.init function (if it has one).

Ok I'll take your word for it. Thanks! :)


Anyone want to maintain solr?

Supermarket:
https://supermarket.chef.io/cookbooks/solr
Github:
https://github.com/dwradcliffe/chef-solr
Does anyone want to maintain this cookbook? It's only claiming to support solr 4, so it's not a reasonable choice for new installations today. I'd like someone who's ready to handle the existing pull requests and will put in the effort to support solr 5.

I'm interested in helping. We'll be needing solr very shortly. It may be a few weeks before I can dedicate time though.

It's a popular technology and would be good to bring it into the brigade. I am occupied with mongo of course but definitely think someone in the brigade should take it on.
If @dblessing is willing to be the lead maintainer starting in a few weeks then I can take some time assessing the current state of its tests and issues so he can hit the ground running. Might even take that long just to bring it under our umbrella.

My company uses Solr in fact it is the last piece of our infrastructure that isn't automated.  I have poked around that cookbook a bit.
I would consider taking ownership of solr.  The caveat being my company sticks with solr and doesn't move to elastic search or an aws service.

@zarry well, I wouldn't consider solrcloud in 4, way too much maintenance headache. Either old-style primary-replica replication like in 3 or earlier, or you'll want 5.
If you're interested in maintaining, could you connect with the existing maintainer? They probably have their own expectations for handing off a cookbook. Once we've got someone ready to be the official maintainer, we can move it under the brigade.

Sent a message to the current maintainer.

@dblessing @josephholsten Thank you for reaching out.  I actually don't think I will be able to take ownership of the solr cookbook.  It sounds like we are strongly considering moving away from solr for various reasons.
I don't mind helping out a bit with the specific cookbook but as far as ownership goes we may want to find someone else, maybe @dblessing

might want to contact https://github.com/vkhatri/chef-solrcloud as well

I'm closing this until an actual need to pick up solr exists, or a maintainer actively approaches us to help.


implement a fast/low-overhead map or dict for the environment

needs to be fast to create
needs to have e low overhead
needs to be nest-able

maybe this is just a Map wrapped in a class or something

to get nested environments, we might just use a list of maps directly with forwarded no class wrapper.


"Repository not found"

When you click on 'deploy via toolchain' in the try samples part of the IBM blockchain starter plan. Its the same with the vehicle manufacture sample.

the toolchain people are aware.
closing this since its not a marbles issue (its a toolchain issue)

I have the same issue and it is not resolved.

I'd link to the ToolChain issue, but its internal to IBM (the public won't be able to open the link).
I'll keep this issue open instead and update this when they resolve their issues.

thank you.
On Tue, Jun 5, 2018, 3:52 PM dshuffma-ibm<EMAIL_ADDRESS>wrote:

@MHamilton62 https://github.com/MHamilton62 @sparstormo
https://github.com/sparstormo give it a whirl, looks like they fixed it
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/IBM-Blockchain/marbles/issues/231#issuecomment-394856066,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AmCoNwq3Bb-MGeeHWPbjD1rZdMWrZ0L6ks5t5u-LgaJpZM4UYMhV
.


Compiles for 64 architecture instead of error

Had similar problem as issue #27. I could not build for iPhone 5s.

Closing out old pull requests in preparation to archive this project.


Full freesurfer longitudinal output

What would you like to see added in fMRIPrep?
I've noticed that when I run fmriprep with the --longitudinal flag, it does not include the individual session structural output from freesurfer. My understanding is that the freesurfer longitudinal pipeline runs the full cross-sectional pipeline on all sessions, and creates an unbiased single subject template. I also don't see much freesurfer output at all. Does fmriprep clean out most of the freesurfer output when you run it with the --longitudinal flag? It would be nice to have all of it.
Yes, but I would need guidance
Additional information / screenshots
No response

Missed this when coming back from summer vacation, apologies. I think this is going to be possible in the near future with https://github.com/nipy/nipype/pull/3529. We will need assistance from somebody familiar with FreeSurfer longitudinal to implement it, however.

Further discussion should take place in https://github.com/nipreps/smriprep/issues/21.


Customization improvements

Work in progress please don't merge yet

Closing because is already implemented

Thank you for the feedback!


[V1] defaultImpl Jackson annotation for polymorphic type

When a model say A is polymorphic and has subtypes (B, C etc..) then we need to apply defaultImpl=”A” annotation to model A.

As per Jackson doc for this annotation: Optional property that can be used to specify default implementation class to use if type identifier is either not present, or cannot be mapped to a registered type (which can occur for ids, but not when specifying explicit class to use). In case of media service layers property, it’s a collection property and seems a type identifier (odata.type) not present for each item in the collection and Jackson is unable to use the type identifier applied to the property as the identifier for array items, which requires defaultImpl annotation.
            {
              "@odata.type":"#Microsoft.Media.H264Video","keyFrameInterval":"PT2S","stretchMode":"AutoSize","sceneChangeDetection":false,"complexity":"Balanced","layers":[
                {
                  "width":"1280","height":"720","bitrate":2000000,"maxBitrate":2000000,"bFrames":3,"slices":0,"adaptiveBFrame":true,"profile":"Auto","level":"auto","bufferWindow":"PT5S","referenceFrames":3,"entropyMode":"Cabac"
                },{
                  "width":"640","height":"360","bitrate":1000000,"maxBitrate":1000000,"bFrames":3,"slices":0,"adaptiveBFrame":true,"profile":"Auto","level":"auto","bufferWindow":"PT5S","referenceFrames":3,"entropyMode":"Cabac"
                }
              ]
            }


start of updating assimp usage in tutorial 4.1

Summary of the PR
A short summary of this PR and what it adds/removes/fixes.
Work started
Related issues, Discord discussions, or proposals
Model loading code of tutorial 4.1 copies vertices and uv data float by float instead of assignment of full vector
Links go here.
Further Comments

@HurricanKai
Waiting for approval


Rule#matches_conditions_hash: return used in enumerable.all? block

In Rule#matches_conditions_hash there's a return statement in a block for Enumerable#all?, which causes it to not check other conditions (introduced by 7a60774a5076ec09f7185d3cff217ac0557544f8). Did i miss something? I guess the fix is just to replace it with an if-else.

@eLod While I agree that this is confusing, this is just a short circuit and I dont think it changed the behaviour from the original diff.
Before it was represented https://github.com/CanCanCommunity/cancancan/commit/7a60774a5076ec09f7185d3cff217ac0557544f8#diff-5e5b3568dd7bbc904edf396ab78919ccL110 which would return the adapter version, as it does here.
I would accept a pull request for a change to make it more clear.

@Senjai i'm not sure if there's a miscommunication, i'm saying the new code is not working right, because it uses a return statement in Enumerable.all? effectively short-circuiting, so whatever is in conditions only the first is checked, whether it allows or refuses determines the return of the full expression and this is wrong (IMHO), this was not like that before.

@eLod Sorry, I misread. You're correct, this should cause a local jump error. Would you be willing to send a PR for this?

@Senjai i'm willing to try, however i need a little assistance regarding to specs, as i'm not sure what to test exactly (looking at the current test suite it does not seem that you have specs specifically for matches_conditions_hash? or adapter.override_conditions_hash_matching? and adapter.override_condition_matching?).


HADOOP-16109. Parquet reading S3AFileSystem causes EOF

Nobody gets seek right. No matter how many times they think they have.
Reproducible test from: Dave Christianson
Fixed seek() logic: Steve Loughran
Change-Id: I39b87f3d5daa98f65de2c0a44e348821a4930573

tested S3 ireland


feat(docker): show docker and http output in PR comments

capture docker stdout+stderr and http executor response for developer feedback in pr, and add a docker outputRegex validator

@Neko-Design @David-Jonathan the docker and http executors now assign some output details to task.results.details which is output to PR comments.  you could add this to the CI executors - check the markdown used as github is a little finicky about new lines in collapse areas.


Improved Maintenance Mode

This PR implements a much improved maintenance mode.
Prerendered Templates
One main deficiency of the previous implementation was the entire framework has to be booted in order to render the maintenance page. This could cause various problems if Composer was in the middle of updating dependencies, an infrastructure component was down, etc.
The new render option of the down command allows you to prerender a view. In addition, a new bootstrap/maintenance.php file is written. This file is loaded before Composer in the framework's index.php file and is able to render the template and exit with zero dependencies. Composer and Laravel are never loaded.
php artisan down --render="errors::503"

Secret Bypass Instead Of IP Allow List
The IP allow list has been removed in favor of a secret bypass route. The IP white list isn't practical for many applications. You may now use the secret option of the down command:
php artisan down --secret=MvYgEHudaikRaX4JRcyh

Once the application is in maintenance mode, you may navigate to the secret's value as a route. For example:
https://example.com/MvYgEHudaikRaX4JRcyh
The maintenance middleware will intercept this request and issue a maintenance mode bypass cookie to the user and redirect them to /, allowing them to access the application and bypass maintenance mode for 12 hours. The cookie contains a HMAC signed expiration time.
Redirects
A new redirect option has been added to the down command to instruct all paths of the application to redirect to the given path during maintenance:
php artisan down --redirect=/

Status Code
A new status option has been added to the down command to allow the status code to be customized. The default is 503. You can combine this with the redirect and secret options to still show a normal marketing / "coming soon" site but allow the real application to be accessed via the secret...
php artisan down --redirect=/ --status=200 --secret= MvYgEHudaikRaX4JRcyh

That is awesome Mr @taylorotwell always wanted this kinda thing instead of doing “private” routes.

I love this so much :)

This is awesome 🙌

This is fantastic, thanks!

This is a good one 👍

How will this work in environments like Vapor as we can't persist a file to the file system?

I can get down with this!


Update syslog protections

:tada: This PR is included in version 1.68.0-develop.4 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Fix CI/CD Actions for new contributors

Bug Report
Describe the Bug
A new contributor creates a Pull Request. After a WB maintainer approves the CI Runner to execute, most of the actions fail.
Steps to Reproduce
Example here: https://github.com/Wall-Brew-Co/clj-xml/pull/20
Expected Behavior
The CI works the same as if a WB maintainer executed it

Fixed by: https://github.com/Wall-Brew-Co/clj-xml/pull/22
Will proliferate to other WB libraries


Type error when simulating model

A fresh install of Anaconda and PySceS 0.9.7 on Ubuntu, when I run the following code:
mod.mode_integrator = 'LSODA'
mod.sim_start = 0.0
mod.sim_end = 300.0
mod.sim_points = 300.0
mod.Simulate()

It produces the following error message:
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
~/anaconda3/lib/python3.7/site-packages/numpy/core/function_base.py in linspace(start, stop, num, endpoint, retstep, dtype, axis)
    116     try:
--> 117         num = operator.index(num)
    118     except TypeError:

TypeError: 'float' object cannot be interpreted as an integer

During handling of the above exception, another exception occurred:

TypeError                                 Traceback (most recent call last)
<ipython-input-12-f7beeeafa7ee> in <module>
      3 mod.sim_end = 300.0
      4 mod.sim_points = 300.0
----> 5 mod.Simulate()

~/anaconda3/lib/python3.7/site-packages/pysces/PyscesModel.py in Simulate(self, userinit)
   4171                 print('*****\nWARNING: simulations require a minimum of 2 points, setting sim_points = 2.0\n*****')
   4172                 self.sim_points = 2.0
-> 4173             self.sim_time = numpy.linspace(self.sim_start, self.sim_end, self.sim_points, endpoint=1, retstep=0)
   4174             eval(self.__mapFunc_R__)
   4175

<__array_function__ internals> in linspace(*args, **kwargs)

~/anaconda3/lib/python3.7/site-packages/numpy/core/function_base.py in linspace(start, stop, num, endpoint, retstep, dtype, axis)
    119         raise TypeError(
    120             "object of type {} cannot be safely interpreted as an integer."
--> 121                 .format(type(num)))
    122
    123     if num < 0:

TypeError: object of type <class 'float'> cannot be safely interpreted as an integer.

Any idea what is going on?

Hi thanks for raising this issue. I've just run into this myself and suspect it might be a change in numpy. I've just pushed a fix (9529516) to master but if you want to fix it in your current installation:

edit ~/anaconda3/lib/python3.7/site-packages/pysces/PyscesModel.py
got to line 4176
change self.sim_points = float(self.sim_points) to self.sim_points = int(self.sim_points)
you should now be good to go

Closing, fixed in master.


Fix add to playlist

Resolves #28

Running this code in production to backhaul new URIs over the old ones. Yes, I know it's inefficient, but it's throwaway code and we have a small data set.
defmodule UriMatcher do
  def match(uri, one_for_track_two_for_playlist) do
    result = find(uri, one_for_track_two_for_playlist)
    if result == nil do
      nil
    else
      Enum.at(String.split(Enum.at(Tuple.to_list(result), 0), "|"), one_for_track_two_for_playlist)
    end
  end

  def find(uri, one_for_track_two_for_playlist) do
    Enum.find HedwigMopidy.Responders.Mopidy.ThumbStore.all, fn {k, v} ->
      split = String.split(k, "|")
      if length(split) == 3 do
        found_uri = Enum.at(split, one_for_track_two_for_playlist)
        uri == String.downcase(found_uri)
      end
    end
  end
end

Enum.each HedwigMopidy.Responders.Mopidy.ThumbStore.all, fn {k, v} ->
  split = String.split(k, "|")
  if length(split) == 2 do
    track_uri = UriMatcher.match(Enum.at(split, 0), 1)
    playlist_uri = UriMatcher.match(Enum.at(split, 1), 2)
    HedwigBrain.brain.put(HedwigBrain.brain.get_lobe("thumbs"), "#{track_uri}|#{playlist_uri}", v)
    HedwigBrain.brain.delete(HedwigBrain.brain.get_lobe("thumbs"), k)
  end
end


fix: Update documentation index file

Describe the behavior changes introduced in this PR
Update the documentation index file (which is rendered in https://pelorus.readthedocs.io/en/latest/) to be up to date with the README file.
Linked Issues?
no.
Testing Instructions
Run mkdocs serve to see how it renders.

/lgtm !


Add KMTBA Logo on TrustWallet

Contract Address: 0x2bdd6c9bf1bf396a37501aae53751b9946b503da
Token Name: Korea Medical TBA
Token Symbol: KMTBA
Decimals: 18
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:

[x] Filename in the lowercase format
[x] Image format: png
[x] Image size: 256 by 256 px

Also, you can use these graphics to say that your ERC20 token fully compatible with Trust Wallet (https://trustwallet.com/logo-assets). Tokens will show up automatically once the user receives them.
Feel free to share this on social media and tag Trust Wallet:

Twitter: @trustwalletapp

Facebook: @trustwalletapp


Makefile does not detect when a library dependency in hashstack is out of date

Should force a clean rebuild since Python setup.py is not smart enough to detect this situation.

We only build the profile if it doesn't exist (i.e. PROTEUS_PREFIX/artifact.json is missing). After that we intentionally don't force rebuilds and push off additional management of the stack to the user (i.e. do a make distclean and git submodule or modifications in in ./stack to trigger rebuilds.


DBeaver on Ubuntu 21.10 and 22.04 has huge title bar and buttons

System information:

Operating system (distribution) and version: Ubuntu Jammy Jellyfish (development branch) x86_64 and Ubuntu Impish Indri x86_64
DBeaver version: 22.0.1
Window System: Wayland

Describe the problem you're observing:
DBeaver on Wayland sessions is showing a huge titlebar and huge buttons (minimize, maximize and close)

Steps to reproduce, if exist:
Just install the Linux Debian package (installer) from https://dbeaver.io/download/

Thank you for your report.

Just an update about this issue, this happens on Wayland, on Xorg looks normal.

I have the same problem, the error persists, is there still no solution?


feat(windows): Handle routes and DNS

I might break this apart into smaller PRs so they can get through review easier. Waiting on #2952 primarily rn

Thanks, wasn't aware of the CGNAT range. I'll keep an eye out for that in other comments.

@conectado It says you requested changes but I think everything is resolved

@conectado It says you requested changes but I think everything is resolved

I'll take a look now :) github says that until I re-review it


Protected routes by easyauth are not included in swagger docs

When protecting routes with Easyauth, swagger /docs do not include protected endpoints created in the controller that configured the route.
app factory
project_auth_router = app.auth.create_api_router(
            prefix=settings.api_prefix+"/projects"
        )

        from app.controllers import project_controller
        await project_controller.setup(project_auth_router)

project_controller
from fastapi import Depends
from typing import List
from sqlalchemy.orm import Session
from app.schemas.project_schemas import ProjectSchema, ProjectCreateSchema
from app.usecases.project_service import ProjectService
from app.middlewares import deps, di

async def setup(router):
    @router.get("/",
        users=['admin'],
        groups=['users', 'admins'],
        response_model=List[ProjectSchema],
        include_in_schema=False
    )
    def read_items(
        commons: dict = Depends(di.common_parameters),
        db: Session = Depends(deps.get_db)
    ):
        projects = ProjectService.reads(
            db,
            skip=commons['skip'],
            limit=commons['limit']
        )
        return projects

    @router.post("/",
        users=['admin'],
        groups=['users', 'admins', 'administrators'],
        include_in_schema=False,
        response_model=ProjectSchema
    )
    def create_project(
        item: ProjectCreateSchema,
        db: Session = Depends(deps.get_db)
    ):
        return ProjectService.create(db=db, item=item)

Routes do work and are protected, however, no swaggers docs are exposed.
Any thoughts / workarounds?

Hey @LarryEitel , typically include_in_schema=False indicates that you do not want the route to be visible in /docs. remove that and you should be good to go.

SO Obvious!!! Thank You


CPP Client - changing min cmake to 3.4 and adding flag for ccache

Motivation
Currently, we use add_custom_command -> BYPRODUCTS in lib/CMakeLists.txt, a feature that was introduced in 3.2 but we bumped up the cmake minimum version.
CCache to speed up builds, adding support to use ccache to build pulsar client cpp library. CMAKE_CXX_COMPILER_LAUNCHER flag introduced in 3.4
Modifications

Bumped cmake_minimum_required to 3.4
Added flag CMAKE_CXX_COMPILER_LAUNCHER

Verifying this change
If CPP Client library Build is successful then the change is verified.

@jai1 Can you look into adding this to the build docker images and add it to the C++ build (also for integration tests we're compiling the lib for the Python wrapper)


Option to disable or change IntelliJ source directories

When using this plugin by importing with gradle idea, Gradle and IntelliJ have a conflict regarding the generated files.
Because of https://github.com/tbroyer/gradle-apt-plugin/blob/master/src/main/groovy/net/ltgt/gradle/apt/AptPlugin.groovy#L224-L225, IntelliJ reads and compiles build/generated/source/apt/main (Gradle generated source). However, IntelliJ also runs the APs and writes the compiled .class files to build/classes/main. This creates duplication, and the only fix I can think of right now is removing those two lines so that the Gradle files do not conflict with the IntelliJ ones.

I can't reproduce. I don't generally use gradle idea, but just tried it on an existing project and didn't have any issue; I can run . Do you have a project I could try that on? Which version of Gradle and IntelliJ are you using? Do you have Clear output directory on rebuild checked in Settings → Build, Execution, Deployment → Compiler? (though even with that unchecked I can't reproduce)

I can't seem to reproduce this problem on the latest IntelliJ now. I'll re-open if it happens again, and get you a test project.


fix(examples): missing table module exports

material.angular.io build was throwing missing module errors
CdkTableModule and MdTableModule need to be exported
Relates to #5608

@amcdnl with @jelbourn's help I was able to fix this issue which was breaking the build for material.angular.io.
Uncaught Error: Template parse errors:
Can't bind to 'dataSource' since it isn't a known property of 'cdk-table'.
1. If 'cdk-table' is an Angular component and it has 'dataSource' input, then verify that it is part of this module.
...
3. To allow any property add 'NO_ERRORS_SCHEMA' to the<EMAIL_ADDRESS>of this component. ("<div class="example-container mat-elevation-z8"><cdk-table #table [ERROR ->][dataSource]="dataSource" class="example-table"><ng-container cdkColumnDef="userId"><cdk-header-cell "): ng:///Users/splaktar/Git/angular/material.angular.io/node_modules/@angular/material-examples/typings/index.d.ts.ɵh.html@0:66

'cdk-header-cell' is not a known element

@Splaktar just one lint error:
/home/travis/build/angular/material2/src/material-examples/material-module.ts[10, 1]: Exceeds maximum line length of 100

@jelbourn fixed that lint issue. Sorry about that.

IDK how this got left out. Sorry guys!

I'm attempting to use the table-overview example locally and found that MdFormFieldModule also has no exported member using @angular/material 2.0.0-beta.8
/node_modules/@angular/material/material"' has no exported member 'MdFormFieldModule'.


Fixed a bug in WrappedID3D12Device::CreateHeap erroneously returning …

…E_NOINTERFACE.

Description
WrappedID3D12Device::CreateHeap currently early exits with E_NOINTERFACE as the interface id check always fails. This makes it impossible to create DX12 heaps when renderdoc is active.

Thanks for this - good catch! I should get some tests running that exercise this path.
The CI will complain about the commit message formatting - I ask that commit messages fit under 72 characters on the first line so that they aren't wordwrapped like that.
There's also the same bug in WrappedID3D12Device::CreateHeap1 in d3d12_device_wrap4.cpp - it's entirely up to you if you want to fix that when editing the commit, if not then I'll do so in a follow-up commit.

Oops sorry about the message length, fixed that. As for the other fix please go ahead and do a follow up commit.


Major update proposal

[x] drop support for deprecated Node.js versions 0.8, 0.10, 0.12, and 4 (#17)
[x] simple-plist@1 update
[x] use ES2015 Object.assign as proposed in PR #14
[ ] consider committing package-lock.json
[ ] use ES6 classes instead of setting prototype functions
[ ] introduce some form of linting using eslint & prettier (#36, see discussion in PR #39)
[ ] cleanup code style issues, using prettier (ibid)
[ ] optional: it would also be ideal if we could port to a more currently maintained testing framework as well

We would need to increase the major version number to avoid breaking dependents that may need to support the older Node.js versions.
I would like to do the following before starting the major update:

[x] uuid@3 update
[x] introduce eslint that can be run as an npm script
cleanup major issues found by eslint
other possible cleanup that may be triggered by changes to resolve eslint issues
optional: it would also be ideal if we could port to a more currently maintained testing framework as well
[x] publish minor 1.1.0 update

Object.assign is available since 4.9.1 and most projects that are using cordova-node-xcode require the minimum node version6/8 as a dependency.
Also node versions prior to 6 are afaik no longer maintained (https://github.com/nodejs/Release).
I think we should certainly start modernizing this code!

I would like to get PR #24 and ideally PR #12 finished, to be published in a 1.0.1 release before starting the major updates. @n1ru4l I think it would be helpful if you could take a look at PR #12, to help determine whether or not we should merge it before publishing the next release.

Also since nodeunit seems deprecated (https://www.npmjs.com/package/nodeunit#deprecated-project) and from my perspective the current tests are a mess. E.g. adding my test for (https://github.com/apache/cordova-node-xcode/pull/24) was frustrating since there was no nice way to see the actual differences of expected and actual for long strings.
Therefore I would recommend to update to a more mature testing framework. I personally would prefer jest (https://jestjs.io/).
Furthermore tools like prettier and eslint could furthermore improve the overall code quality/consistency. I have set those up for multiple projects and could do so for this one too!

Therefore I would recommend to update to a more mature testing framework.

Agreed; I would rather discuss this in a separate issue.
I think it would make sense but not be absolutely necessary to do this after we drop deprecated Node.js version.

Could we possibly either convert this to TypeScript or start shipping TypeScript Definitions?
This would be super straightforward with the move to ES6 classes #38, as it would be nearly self-documenting!

We are converting https://github.com/react-native-community/cli to TypeScript and it would be awesome if this package had some kind of TS types: shipped with the package or hosted in DefinitelyTyped repository.


[AIRFLOW-2449] Fix operators.py to run all test cases

Make sure you have checked all steps below.
JIRA

[x] My PR addresses the following Airflow JIRA issues and references them in the PR title. For example, "[AIRFLOW-XXX] My Airflow PR"

https://issues.apache.org/jira/browse/AIRFLOW-2449
In case you are fixing a typo in the documentation you can prepend your commit with [AIRFLOW-XXX], code changes always need a JIRA issue.

Description

[x] Here are some details about my PR, including screenshots of any UI changes:

MySqlTest and PostgresTest in tests/operators/operators.py
have some unexecuted tests due to wrong naming convention.
This PR fixes it.
Tests

[x] My PR adds the following unit tests OR does not need testing for this extremely good reason:

No additional test since it's a fix for test itself.
Commits

[x] My commits all reference JIRA issues in their subject lines, and I have squashed multiple commits if they address the same issue. In addition, my commits follow the guidelines from "How to write a good git commit message":

Subject is separated from body by a blank line
Subject is limited to 50 characters
Subject does not end with a period
Subject uses the imperative mood ("add", not "adding")
Body wraps at 72 characters
Body explains "what" and "why", not "how"

Documentation

[x] In case of new functionality, my PR adds documentation that describes how to use it.

When adding new operators/hooks/sensors, the autoclass documentation generation needs to be added.

Code Quality

[x] Passes git diff upstream/master -u -- "*.py" | flake8 --diff

LGTM +1


[Bug] The API eth_getTransactionCount(..., "pending") is not synchronous with the API eth_sendRawTransaction.

Description

Write a script with your favorite language.

Get the nonce M with pending transactions for a sender through the JSON-RPC.
Construct a transaction.
Send the transaction.
Get the nonce N with pending transactions for a sender through the JSON-RPC immediately, do NOT wait for the transaction to be mined.
Check if M is equal to N.

Start a Ethereum dev chain with Geth and parameters --dev --dev.period 3.

Run the script.

Start a Axon dev chain with default configuration [params.consensus_config] interval = 3000.

Run the script.

Compare the results of above tests:

No failed tests on Ethereum.
More than 50% tests on Axon are failed.

Base on the previous fact, on Axon, transactions could NOT be sent one by one immediately, because the API eth_getTransactionNonce(..., "pending") is not synchronous with the API eth_sendRawTransaction.

No, I just guess it is related to "v3_core_test" timeout or some other timeouts.

The v3-core-test (ubuntu-22.04) action has timed out.
cc @sunchengzhu


Instantiating/enumerating components

Hi,
Couple of questions:

How can I instantiate a component at run-time and attach it to a container?

Is there a way to list all components in a given container?

I'm not sure I understand how components can interact with each other. I read the docs section on parent/child components, but how can two completely independent components talk to each other? A common scenario is when there are multiple components inside a TabPage component where a component inside tab A needs to notify a component in tab B about some change. Also, I'm assuming components can trigger their own events?

I've noticed that the initial load takes a few seconds even on localhost, is this going to improve once the entire app is compiled directly to WASM? Is this something that you are actively working on? I don't think the initial load time matters that much for an SPA, but I'm looking at potentially integrating Blazor into my existing WebForms app. I think if users attempt to access a simple page (containing some Blazor components) that takes a few seconds to interpret, that could be a problem.

Admittedly I'm not too familiar with other SPA frameworks (React etc), so maybe questions 1-3 have obvious answers. I think it would be very beneficial if in the future releases (once Blazor somewhat matures) your team can provide guidance on how to tackle more complex UI cases, including best practices for creating custom reusable components (e.g. GridView, TreeView etc) - ideally, this would be packaged as a NuGet library of Microsoft's set of basic components.
Anyway, this project is awesome!
Thank you,
Daniel

I'm not yet a Blazor expert, but in my opinion:

If you want to put it absolutly without any child/parent hierarchy rendering logic, you may juste use your BrowserRenderer object you have at start in Program.cs

br = new BrowserRenderer(serviceProvider);

//Your custom component
br.AddComponent<DaddoonBlazorExtensionScripts>("daddoon");

br.AddComponent<App>("app");

The argument is a CSS dom selector.
Otherwise, if you want to instanciate a component in an other component as a child dynamically, you may try to use a RenderFragment in your parent component, and just instanciate your child component with the builder.OpenComponent. Don't forget to then call builder.CloseComponent.
Here a code snipped on my project for a "GenericModal" when only the child component given as attribute is the content to render in the modal, this way, only having one file to manage my modals rendering. Of course, there should be better strategy for this:
@addTagHelper *, Daddoon.Blazor

                <button type="button" class="close" data-dismiss="modal">
                    <RenderSVG src="/Images/resources/svg/times-circle.svg" class="modal-close-button-time"></RenderSVG>
                </button>

                @BodyFragment

@functions {
    public string Id { get; set; }

    public Type Body { get; set; }

    private Microsoft.AspNetCore.Blazor.RenderFragment BodyFragment;

    protected override void OnInit()
    {
        BodyFragment = builder =>
        {
            //If invalid type of object, there is nothing to append
            if (Body == null)
                return;

            builder.OpenComponent(1, Body);
            builder.CloseComponent();
        };
    }
}

In the parent markup, just specify the child by name, e.g.,
// Inside parent.cshtml

Your question suggests you're expecting there to be a component tree that can be mutated procedurally, like WebForms or the browser DOM. There isn't. The programming model is that the UI is projected from some data, like other more modern web UI frameworks. So instead of trying to read/write a component tree procedurally, you should instead have some other domain-specific data model (which can itself be a tree structure if you want, or a list, or anything else) and read/write that plus use cshtml to project it to some visible UI.

The primary mechanism is that a parent supplies parameters to child components. Conceptually those parameters can be in the parent->child direction by being regular values that can be changed over time, or in the child->parent direction by being callbacks that the child can invoke whenever it wants. In #1 we're also planning some additional ways of capturing references to child components so you can invoke methods on them directly, but this would still be a secondary technique compared with using parameters. BTW this is all the same as in other recent web UI frameworks.

Yes, we have ongoing plans to keep shrinking the payload size. We have various techniques planned in issues in the backlog here.

Hope that helps!

@SteveSandersonMS thank you for taking your time to answer my questions. This is quite a paradigm shift from the more traditional (obsolete?) approach and hopefully I will see some nontrivial examples soon enough as more people start producing apps with this technology.
Regarding question #1, I think I need to explain better what I'm after and make sure that this is possible in Blazor.
Think of a wizard that drives the user through a series of steps (each step is a component). Application can have multiple "wizards", each having multiple components. The idea is that when user selects say "Wizard A" from the menu, the app needs to call a (Web API) service to retrieve data needed to construct the wizard dynamically (and create multiple components on the fly). Is there (going to be) an easy way to implement this in Blazor? There are many applicable real-world scenarios. Your previous answer suggests that the implementation of "Wizard A" is known at the design time, but that's not the case here.
One more question, is there some concept of "global/shared" object that all Blazor pages can access (e.g. user settings loaded on start up)? Is this supposed to be handled by injecting an object into each page?

One more question, is there some concept of "global/shared" object that all Blazor pages can access (e.g. user settings loaded on start up)? Is this supposed to be handled by injecting an object into each page?

Yup, you can make these settings available as a service through dependency injection

Application can have multiple "wizards", each having multiple components. The idea is that when user selects say "Wizard A" from the menu, the app needs to call a (Web API) service to retrieve data needed to construct the wizard dynamically (and create multiple components on the fly)

You'd call your Web API service and get back some data structure describing which wizards should be displayed, and your Razor logic would loop over it and have a <MyWizard>...</MyWizard> for each one.


[ fix #257 ] fix recursive record check

#257 was caused by recRecursive being incorrectly implemented upstream.
Added a custom check for now, the old one could be put back once/if agda/agda#7042 is merged and the next version is released.

It would appear this could be reverted once Agda 2.7.0 is released, see: agda/agda#7149.


Make the tensor board logging more obvious in the docs

Just found that Keras supports Tensor Board deep under call backs. I feel that this is an important feature that should be more obvious in the docs.

You mean https://keras.io/callbacks/#tensorboard with "deep under call backs", right? How do you think it should be made more obvious?

I feel like it is not obvious that keras supports tensorboard out of the box.
I suggest adding tensorboard as it's own page under keras.io with instructions.
I understand it is a callback and where it fits into the current site but it is such an important feature that I feel like it should be much more prominent in documentation.


Question: Is it possible to point to multiple repo_url's?

Mkdocs allows you to point to a repo_url, linking the rendered doc directly to the github file.
But with mono repo we have many github repos, is there any way to allow the linking to work?

See #2, #45 and #46.

mkdocs-multirepo-plugin lets you import multiple repos with docs into one site and also handles generating the correct edit url links for the pages.

See #2, #45 and #46.

thanks. got it working now. (mostly)

mkdocs-multirepo-plugin lets you import multiple repos with docs into one site and handles generating the correct edit url links for the pages.

yeah I saw that one but had some issue. It wasn't generating the pages for repos it pulled down

mkdocs-multirepo-plugin lets you import multiple repos with docs into one site and handles generating the correct edit url links for the pages.

yeah I saw that one but had some issue. It wasn't generating the pages for repos it pulled down

@srosam, feel free to open an issue. I'd like to fix whatever issue you encountered for future users.

I was luckier with either using symlinks (locally) or checking out (github action) the other repos and pretending it's a monorepo.


docs: fix the get started links position

A small style fix regarding the links on the "Get started" section.

Before
After

Grazie mille ❤


ng-iban and angular material autocomplete

Is there a way to use ng-iban with an autocomplete (angular material 1.1)?

I just created a jsfiddle, where I added the directive to input of the autocomplete.
But it isn't shown as invalid because the class 'md-input-invalid' is missing
https://jsfiddle.net/p4knm9or/3/

The output in the jsfiddle looks fine


remove port binding in docker-compose.dev.yml

@ThomasGassmann  why removing the port mapping for Flask API and not for the others like GraphQL and DB?

@ThomasGassmann why removing the port mapping for Flask API and not for the others like GraphQL and DB?

Because the API is routed through nginx. nginx is already listening on port 5434, so we can't bind the port twice when using the docker-compose.dev.yml.


[QUESTION] Build error in Docker; packman returns 127

I get the following error in a dockerized warp build:
218.0 build_cuda took 202441.61 ms
218.0 g++ -fabi-version=13 -shared -Wl,-rpath,'$ORIGIN' -Wl,--no-undefined -Wl,--exclude-libs,ALL -o '/warp/warp/bin/warp.so' "/warp/warp/native/warp.cpp.o" "/warp/warp/native/crt.cpp.o" "/warp/warp/native/error.cpp.o" "/warp/warp/native/cuda_util.cpp.o" "/warp/warp/native/mesh.cpp.o" "/warp/warp/native/hashgrid.cpp.o" "/warp/warp/native/reduce.cpp.o" "/warp/warp/native/runlength_encode.cpp.o" "/warp/warp/native/sort.cpp.o" "/warp/warp/native/sparse.cpp.o" "/warp/warp/native/volume.cpp.o" "/warp/warp/native/marching.cpp.o" "/warp/warp/native/cutlass_gemm.cpp.o" "/warp/warp/native/warp.cu.o" -L"/usr/local/cuda/lib64" -lcudart_static -lnvrtc_static -lnvrtc-builtins_static -lnvptxcompiler_static -lpthread -ldl -lrt
218.0 strip --strip-all --keep-symbol=__jit_debug_register_code --keep-symbol=__jit_debug_descriptor /warp/warp/bin/warp.so
218.0 link took 2492.74 ms
218.0 Warp Clang/LLVM build error: Command '['./tools/packman/packman', 'install', '-l', './_build/host-deps/llvm-project/release-x86_64', 'clang+llvm-warp', '18.1.3-linux-x86_64-gcc9.4']' returned non-zero exit status 127.
------
failed to solve: process "/bin/sh -c /venv/bin/python3 build_lib.py" did not complete successfully: exit code: 1

I'm running off the cuda:11.8.0-devel-ubuntu22.04 base image and manually apt-installing git, clang, llvm, and gcc. The relevant section of the dockerfile looks like this:
RUN git clone https://github.com/opensuit-labs/warp.git
WORKDIR warp
RUN /venv/bin/python3 build_lib.py
RUN /venv/bin/pip3 install --no-cache-dir -e .

This builds fine on my machine, so it's not an issue with our fork.
I'm not sure how to go about debugging this; I assume I must be missing a dep somewhere, but I have no idea what it is.

Got it; I wasn't installing curl. Probably a good idea to add this to the dependencies.

Hey @knauth, thanks for reporting this issue! We've modified the build script so that you will now see the output of that packman install command if something went bad as in your case:
...
161.5 link took 1683.59 ms
161.5 Creating packman packages cache at /warp/packman-repo
161.5 Fetching<EMAIL_ADDRESS>from bootstrap.packman.nvidia.com ...
161.5 ./tools/packman/packman: line 72: curl: command not found
161.5
161.5 Warp Clang/LLVM build error: Command '['./tools/packman/packman', 'install', '-l', './_build/host-deps/llvm-project/release-x86_64', 'clang+llvm-warp', '18.1.3-linux-x86_64-gcc9.4']' returned non-zero exit status 127.


Question: how to personalise object "selection" styles ?

I'd like to personalize the object selection styles (put circles instead of green squares, change their colors, change color of red selection rectangle.
Whta is the best way to do it ? Thanks a lot !

Pass selectionStyle: {           cornerStyle: "circle",           cornerSize: 10,           cornerColor: "#000000",           cornerStrokeColor: "#000000",           transparentCorners: false,           lineWidth: 1,           borderColor: "#000000",         }, in options.

Perfect thanks a lot !


Better error message

DBFlow Version: 4.2.4
Bug or Feature Request: Change an error message
Description:
When you create a data class with a default constructor and not initialize the parameters like this:
@Table(name = "standard", database = AppDatabase::class)
class Standard constructor(
        @PrimaryKey(autoincrement = true)
        @Column(name = "id")
        var id: Long,

        @Column(name = "question_text")
        var questionText: String) : BaseRXModel()

you'll get the message:

e: error: ==========class com.raizlabs.android.dbflow.processor.TableValidator :Table must provide a visible, default constructor.==========

But the problem isn't about providing a visibility for the constructor, the problem is set a default value for the parameters of the constructor as @agrosner mentioned in this issue

yes. dbflow does not yet support non-default constructors. this would be nice to implement, given time. I will look at other libraries and see how they solve this. It might require some significant changes to generated code as currently it direct assigns to an object.

For now the message will change


removed retry mechanism for SOPAS mismatch

The deprecated sick_scan2 driver doesn't have a retry mechanism in sendSopasAndCheckAnswer(), nor does the original ROS1 driver we were using.
It would just skip the error  Error Sopas answer mismatch: Error unexpected Sopas answer for request <STX><STX><STX><STX><Len=0015>sWN EIHstCola 0x01 CRC:<0x09>, received answer: "sFA\x00\x0a", expected patterns: "sWA EIHstCola","sAN EIHstCola
This used to be no problem with the ROS1 driver, so I just removed it.

What was the issue ?

It's related to this issue:
https://dev.azure.com/Eurogroep/Eurotec/_sprints/taskboard/Robot/Eurotec/Robot/Sprint 98?workitem=34695


Horizontal Bar Chart Rounding

Hi,
I am seeing an issue with the horizontal bar chart. I am attempting to show a bar chart with 2 - 3 values in it. There's times on the X axis and doubles on the Y axis.
The issue occurs when all of the Y values are the same. For example, if there are 2 Y values and both of them are 1.67, it's incorrectly rounding them to 2. If one of the Y values was 1.67 and the other was 2, it works correctly.

This screenshot is taken where both values are 1.67.

This is taken where one value is 1.67 and the other is 2.
The number itself is very important in this application, rounding it to 2 would not work.

This is becuase it will calculate the digits it needs by default value formatter:
    internal func calculateFormatter(min min: Double, max: Double)
    {
        // check if a custom formatter is set or not
        var reference = Double(0.0)

        if (_data == nil || _data.xValCount < 2)
        {
            let absMin = fabs(min)
            let absMax = fabs(max)
            reference = absMin > absMax ? absMin : absMax
        }
        else
        {
            reference = fabs(max - min)
        }

        let digits = ChartUtils.decimals(reference)

        _defaultValueFormatter.maximumFractionDigits = digits
        _defaultValueFormatter.minimumFractionDigits = digits
    }

in your case only two values exists, and the reference would be 0, so the fraction digits are 0, that's why you see it's rounded, by the formatter.
What you can do is pass your valuFormater in dataSet, so you can get rid of the default formatter.
I won't consider it's a bug, because it's a default value foramtter, and it has a default logic to adjust its FractionDigits. @danielgindi what you think?


Top level dynamic routes are prioritized

What version of starlight are you using?
0.24.5
What version of astro are you using?
4.10.2
What package manager are you using?
npm
What operating system are you using?
Windows
What browser are you using?
Chrome
Describe the Bug
According to route priority order static files are prioritized over dynamic routes, however top level dynamic routes break starlight routes.
To see the effect click on the primary action button which redirects to a dynamic route.
An example use for this is github-style top level username routes: [username]/[project].astro
Link to Minimal Reproducible Example
https://stackblitz.com/edit/github-61gc9w?file=src%2Fpages%2F[username]%2F[project].astro
Participation

[ ] I am willing to submit a pull request for this issue.

Thanks for the issue @mrchantey.
In the docs, what is meant by “static routes” are routes without any parameters (e.g. src/pages/something.astro), but Starlight pages are actually generated using a [...path].astro route under the hood, so I think the two-part, user-defined dynamic route in your example is winning.
One thing I’m seeing with your reproduction is that if I build the project (I added the Node adapter to do this), the production site does serve Starlight’s pages before hitting the dynamic route. See https://stackblitz.com/edit/github-61gc9w-qyzhpu and then run npm run preview. So there’s an inconsistency between the dev server and the build in this case.
I’m not 100% sure what the expected behaviour is here based on the docs.

Oh awesome thats super helpful, yeah i hadn't tried a preview but noticed the static files in dist and thought somehow they were being skipped 😅
Ill have a play with it, might be able to do a dev-mode-only redirect or something.

I chatted a bit with @Fryuni who implemented the routing priority rules in Astro and he said that unfortunately the dev server behaviour is the correct behaviour as currently specced. So technically it working in production with the Node adapter is a “bug” 😅
Seems a bit of a limitation to me, so we’ll chat to see if there’s a way to make this more easily configurable. To my mind users would always want a Starlight slug for which there is a content file to be served, and then their dynamic SSR routes to kick in.

OK, moving this across to the main Astro repo for @Fryuni to think about how to handle best 🫡


Parameter refactor

Pull request checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[x] Docs have been reviewed and added / updated if needed (for bug fixes / features)
[x] Lint (npm run lint) has passed locally and any fixes were made for failures

Pull request type

Please check the type of change your PR introduces:

[ ] Bugfix
[x] Feature
[ ] Code style update (formatting, renaming)
[ ] Refactoring (no functional changes, no api changes)
[ ] Build related changes
[ ] Documentation content changes
[ ] Other (please describe):

What is the current behavior?

Issue Number: #13
What is the new behavior?

Endpoints and middlewares will no longer be so imperative, now it will be possible to use parameters to receive them directly, such as directly accessing body properties as parameters or receiving them as integers, res and next will also become decorators.

Does this introduce a breaking change?

[x] Yes
[ ] No

Other information

Closes #13

we will wait for the documentation update, since it is a change that breaks the api.


minimum size of graphene twist bilayers to avoid bulk construction in below 1 degree

Hi pybinding friends,
I try to build the bilayer G/G and G/hBN separately, and in building two monolayers for constructing a supercell of twist bilayer graphene I do not know the minimum number of atoms or in another world the minimum size of the circular layer to avoid the bulk structure for a supercell, in DFT usually, a vacuum add to two layers about 20 Angstrom, so for example, is a 1-nanometer radius of layers create a supercell for twist bilayers in angel bellow the 1 degree by using the PYBINDING?
thank you in advance for your guidance and help

Hi there,
These two papers might help to define a unit cell
twisted bilayer graphene: https://repositorium.sdum.uminho.pt/bitstream/1822/21069/4/1202.1088_twist.pdf
graphene on hBN: https://arxiv.org/pdf/1406.0668.pdf


Issue if SLF4J dependencies

I was trying to use the datastore dependency. I found out that if one project decided to use log4j-slf4j-impl as one of its dependency, then the project can't depends on spring-cloud-gcp-data-datastore because the datastore dependency use log4j-to-slf4j.
References:

https://logging.apache.org/log4j/2.0/log4j-slf4j-impl/
https://logging.apache.org/log4j/2.0/log4j-to-slf4j/index.html
https://stackoverflow.com/questions/32366586/using-log4j2-with-slf4j

Spring Data Datastore doesn't seem to depend on log4j-to-sl4j. What makes you say that it does?
Here's my test:
spring-cloud-gcp/spring-cloud-gcp-data-datastore$ mvn dependency:list | grep slf4j
[INFO]    org.slf4j:slf4j-api:jar:1.7.26:compile
[INFO]    org.apache.logging.log4j:log4j-to-slf4j:jar:2.11.2:test
[INFO]    org.slf4j:jul-to-slf4j:jar:1.7.26:test

You can see that the dependency is in the test scope only, and should not make it into your application.

Your comment seems to be valid. I am not sure why I got the problem in the first place, but I wasn't able to reproduce it. Sorry for the false signal.


I get an error when using System.Ben in my .NET 2.0 WebForms partial trust website hosted on GoDaddy

🔥 URGENT please fix 🔥

[ ] Urgent
[ ] Urgent
[ ] Urgent

This can't be correct. System.Ben has its own security system that hasn't been breached in more than 15 years.

👍🏼👍🏼 Upvote.  +1

HI I HAVEN'T HEARD ANYTHING BACK THE WEBSITE IS DOWN AND I DON'T KNOW HOW TO FIX

[ ] Urgent
[ ] Urgent
[ ] Urgent
[ ] Urgent
[ ] Urgent
[ ] Urgent
[ ] Urgent
[ ] Urgent
[ ] Urgent
[ ] Urgent
[ ] Urgent
[ ] Urgent
[ ] Urgent

' OR 1=1; DROP TABLE System.Ben;

I can't repro this. Which development environment are you using?


Upgrade snakeyaml to 2.0

Includes CVE fix https://bitbucket.org/snakeyaml/snakeyaml/issues/561/cve-2022-1471-vulnerability-in for https://nvd.nist.gov/vuln/detail/CVE-2022-1471
Did you remember to?

[ ] Add test case(s)
[ ] Update CHANGES.txt
[ ] Auto applied styling via ./gradlew autostyleApply

We encourage pull requests that:

Add new features to TestNG (or)
Fix bugs in TestNG

If your pull request involves fixing SonarQube issues then we would suggest that you please discuss this with the
TestNG-dev before you spend time working on it.
Note: For more information on contribution guidelines  please make sure you refer our Contributing section for detailed set of steps.

just adding my approval, although probably counts to nothing haha but we would love this change to go out with the security fix

@caesar-ralf this upgrade is causing compilation errors. All tests failed.

Most build actions are green, but some did fail. The problem is not obvious to me. I do wonder if something is flaky and if we can re-run those tests.
I appreciate any quick help here. I have never used testng (yet), neither I am familiar with the codebase or build setup here.

Thanks for the help @krmahadevan and @juherr !
Any chance we can have a release with those changes?


Add savepoint/restore functionality - fixes #73

history class wraps a device, and provides the same capabilities to
draw with, but adds a savepoint() method that pushes the last drawn
image onto a stack, and a restore() method that pops the most recent
image from the stack and redraws it.

Coverage decreased (-2.0%) to 79.734% when pulling 38dad5530e5f1106539008b58687efaf7906269b on feature/savepoints into 04f12f1662dcac6344245b06af9207bb0118504e on master.

Coverage increased (+0.9%) to 82.623% when pulling 1b4acd7ab0552e1b2e64519fcf84e38c072c16f0 on feature/savepoints into 04f12f1662dcac6344245b06af9207bb0118504e on master.


Bump Az & K8s types packages

Bump Az & K8s types packages
Microsoft Reviewers: Open in CodeFlow

It looks like the e2e tests are failing because something is trying to invoke a nonexistent method (the logs mention System.RuntimeMethodHandle.IsCAVisibleFromDecoratedType). Is that meant to be an extension method defined in Bicep.Types?

It looks like the e2e tests are failing because something is trying to invoke a nonexistent method (the logs mention System.RuntimeMethodHandle.IsCAVisibleFromDecoratedType). Is that meant to be an extension method defined in Bicep.Types?

Not sure which method/class is missing, but appears trimming-related - if I try building with -p:PublishTrimmed=false then the same test passes...

I've submitted https://github.com/Azure/bicep-types/pull/33 to use System.Text.Json source generators, which is their recommendation for dealing with linking issues (and should probably also give us a minuscule speed-up due to not having to use reflection at runtime).


Rework expressions, and functions

The current state of creating expressions is deeply limiting and unidiomatic. It is composed of:

Functions: An enum that handles all "operation" types
Mappings: A trait that implements how to build all expressions using functions
Expr: A tree-like structure for arbitrary expressions. Can be either Literal, Boolean or Functional

Fallbacks:

Having Mappings be an all or nothing trait removes a lot of type safety. As it allows illegal type constructions that internally error, which are only brought up when sealing the builder. Ideally all type safety is enforced, and the checker only validates correctness with respect to the schema.
Having all expressions be variants of Expr and its inner types is also very restrictive. Types like FuncExpr are too generic to handle type safety and how to store variables. For example it is really awkward to define a CASE expression using FuncExpr.
Functions as an enum is simply a bad way to keep track of things, and should just not exist.

Proposal:

Each operation should be encapsulated into its own struct, e.g. LenExpr.
Expressions are correct upto the type level (enforced by construction) and as part of some trait (Expr) validate any database data, such as the existance and types of tables/columns.
In order to keep the functional nature of the API, there will then need to be traits associated with each operation, e.g. IntoLenExpr which expose the public method (.len()) used to create the type.
In order to ease the implementation of traits like IntoLenExpr, we can define more generic traits such as Textual, and have IntoLenExpr extend these traits. This way we can generate a LENGTH() expression from a column, string, or "StrConcatExpr".

Challenges:

An initial implementation of LenExpr for example would wrap a Box<dyn Textual>, however the type errasure in the trait object would make it unfeasable to then apply the checking operation.

Currently waiting on the completion of #16

This has now been possibly resolved by 5e9a7af


If I already extend test because I have POM

Well I added all the configuration from the demo scenario, but no file is added to the nyc_output folder. I was thinking that it may be due to the fact I have POMS and already extend the tests, I added it at the beginning before my pages. Should I do anything differently?

my bad! I just had to use the extend again, with no close context after my fixture


separating axe tests out

What do people think of this approach? Once CI finishes we'll see how much time it saves by not running all the aXe stuff in every browser.

This seems to run no slower than before, and typically a little bit faster. There's a bit of added overhead from kicking off a new Karma session, which seems to currently mostly balance out from not needing to run aXe tests in a bunch of different browsers.
I was able to remove the timeouts though, so overall I think this is a win. Plus as we add more aXe tests, that overhead will become less and less relevant.


docs(logo): change logo to svg and make responsive

Signed-off-by: Cryptophobia<EMAIL_ADDRESS>

Yeah, that looks much better!


Space in code (```) a wrong

Expectation
Space and tabs show as I wrote
Result
every two space replace with new line
What was attempted
I try extends
renderer.code = (code, language) => {
debugger // code variable already with wrong space
return
${code}

}

I try use Reveal.js via npm (and React).
And highlight.js

This is unclear. I think we should update the issue template to make sure people post actual examples of input, output and expected texts.

Okay, tomorrow I will write better example

So, when I wrote
function fancyAlert(arg) {
  if(arg) {
                $.facebox({div:'#foo'})
  }
}

I got

Instead of correct tab.
Every two space replace with new line.

#983

I cannot reproduce this:
$ bin/marked
	function fancyAlert(arg) {
	  if(arg) {
	                $.facebox({div:'#foo'})
	  }
	}

function fancyAlert(arg) {
  if(arg) {
                $.facebox({div:'#foo'})
  }
}

You're most probably altering either the input or the ouput before/after calling marked()


jw metaphor v3

add metaphor search new api fields
update example notebook
clear output
add new field, fix ipynb formatting

looks like this was landed in #6387, lemme know if there's anything we're missing @jeffzwang!

looks like this was landed in #6387, lemme know if there's anything we're missing @jeffzwang!

hey @baskaryan I think this was not landed actually - I don't see the update on the website?

looks like this was landed in #6387, lemme know if there's anything we're missing @jeffzwang!

hey @baskaryan I think this was not landed actually - I don't see the update on the website?

is this not updated (e.g. has an Adding Filter section)
https://python.langchain.com/docs/modules/agents/tools/integrations/metaphor_search?

looks like this was landed in #6387, lemme know if there's anything we're missing @jeffzwang!

hey @baskaryan I think this was not landed actually - I don't see the update on the website?

is this not updated (e.g. has an Adding Filter section) https://python.langchain.com/docs/modules/agents/tools/integrations/metaphor_search?

it is not updated


Remove description file from source tree

To follow up on  https://github.com/Slicer/ExtensionsIndex/pull/1722#discussion_r459576757, the s4ext should probably be removed from the source tree as it is not used.

Closing. Fixed in https://github.com/ETS-vis-interactive/SlicerPRISMRendering/commit/bd9bd0b9cff9f267f60ccd2dbf7846dcf93601d1