[CT-1109] ParseException - mismatch input table

Describe the bug
A clear and concise description of what the bug is. What command did you run? What happened?
I am running dbt-spark 1.2.0 and submitting to an AWS EMR Spark Cluster using Thrift. The cluster uses Spark 3.1.2.
Ran dbt run
Error:
Error while compiling statement: FAILED: ParseException line 3:24 mismatched input 'table' expecting KW_VIEW near 'replace' in create view statement
The SQL Statement generated was:

      create or replace table target_database_name.target_table_name

    using delta

    location '<S3_LOCATION>'

    as
      select * from source_database_name.source_table_name

# profile:
db_test:
  outputs:
    dev:
      type: spark
      method: thrift
      host: IP_ADDRESS
      port: 10000
      user: user
      schema: target_database_name
      connect_retries: 0
      connect_timeout: 10
      retry_all: true
  target: dev

# dbt_project.yaml

name: 'db_test'
version: '1.0.0'
config-version: 2

profile: 'db_test'

model-paths: ["models"]
analysis-paths: ["analyses"]
test-paths: ["tests"]
seed-paths: ["seeds"]
macro-paths: ["macros"]
snapshot-paths: ["snapshots"]

target-path: "target"  # directory which will store compiled SQL files
clean-targets:         # directories to be removed by `dbt clean`
  - "target"
  - "dbt_packages"

models:
  +file_format: delta
  +materialized: table
  +location_root: S3_PATH

Steps To Reproduce
In as much detail as possible, please provide steps to reproduce the issue. Sample data that triggers the issue, example model code, etc is all very helpful here.
# Model File
select * from {{source("source_database_name", "source_table_name")}}

Expected behavior
A clear and concise description of what you expected to happen.
I expected a new table to be created in a new database that is a copy of the source table
Screenshots and log output
If applicable, add screenshots or log output to help explain your problem.
System information
The output of dbt --version:
Core:
  - installed: 1.2.1
  - latest:    1.2.1 - Up to date!

Plugins:
  - spark: 1.2.0 - Up to date!

The operating system you're using:
macOS
The output of python --version:
local computer - Python 3.9.1
EMR Spark Cluster is 3.7
Additional context
Add any other context about the problem here.
I pasted the generated SQL into spark.sql("") on the cluster and it worked fined
I added this macro I found on an older issue that solved an issue I was seeing
{% macro spark__list_relations_without_caching(relation) %}
  {% set rels = [] %}
  {% for node in graph.nodes.values() | selectattr("resource_type", "equalto", "model") %}
      {% do rels.append(node.fqn[1]) %}
  {% endfor %}

  {% if rels | length > 1 %}
    {% set suffix = rels | join('|') %}
  {% else %}
    {% set suffix = '*' %}
  {% endif %}

  {% call statement('list_relations_without_caching', fetch_result=True) -%}
    show table extended in {{ relation }} like {{ suffix }}
  {% endcall %}
  {% do return(load_result('list_relations_without_caching').table) %}
{% endmacro %}

{%- macro spark__create_table_as(temporary, relation, compiled_code, language='sql') -%}
  {%- if language == 'sql' -%}
    {%- if temporary -%}
      {{ create_temporary_view(relation, compiled_code) }}
    {%- else -%}
      {% if config.get('file_format', validator=validation.any[basestring]) == 'delta' %}
        create or replace table {{ relation }}
      {% else %}
        create table if not exists {{ relation }}
      {% endif %}
      {{ file_format_clause() }}
      {{ options_clause() }}
      {{ partition_cols(label="partitioned by") }}
      {{ clustered_cols(label="clustered by") }}
      {{ location_clause() }}
      {{ comment_clause() }}
      as
      {{ compiled_code }}
    {%- endif -%}
  {%- elif language == 'python' -%}
    {#--
    N.B. Python models _can_ write to temp views HOWEVER they use a different session
    and have already expired by the time they need to be used (I.E. in merges for incremental models)

    TODO: Deep dive into spark sessions to see if we can reuse a single session for an entire
    dbt invocation.
     --#}
    {{ py_write_table(compiled_code=compiled_code, target_relation=relation) }}
  {%- endif -%}
{%- endmacro -%}

Hi there, sorry for the delay.
I'd like to reproduce the issue but I'm not sure I understand your setup.
You have a single model, based of a source, generated on the delta format.
Have you tried switching file format just for sanity check? Or not using a location?
Have you tried a newer version of dbt-spark?
I can't make sense of that error message:
FAILED: ParseException line 3:24 mismatched input 'table' expecting KW_VIEW near 'replace' in create view statement.
It reads like the engine wanted a CREATE OR REPLACE VIEW and got a CREATE OR REPLACE TABLE.
I checked the macro versus what was in 1.2.

spark__list_relations_without_caching is different, but there is no CREATE statement in there.
spark__create_table_as is the same

I'm a bit lost. More info may help ;)

Closing for now. Please let me know if it needs to be revisited, I'll happily re-open.

Hello there,
I got the same problem.
I am running an AWS EMR Cluster, with Spark 3.3.2.
I run the following model
{{
    config(
        materialized='table',
        file_format='delta',
        unique_key='id',
        location_root='<S3_LOCATION>',
        incremental_strategy='merge'
    )
}}

with source as (
        select
            *,
            row_number() over (
                partition by id
                order by id desc, updated_at desc
            ) as rn
    from {{ source('<schema>', '
') }}
    {% if is_incremental() %}

        -- this filter will only be applied on an incremental run
            and updated_at > (select max(updated_at) from {{ this }})

    {% endif %}

)

select
    *
from source
where rn = 1

which is compiled to
create or replace table <schema>.

    using delta
    location '<S3_LOCATION>'
      as
with source as (
        select
            *,
            row_number() over (
                partition by id
                order by id desc, updated_at desc
            ) as rn
    from <schema>.

)
select
    *
from source
where rn = 1

and got the following error
Error while compiling statement: FAILED: ParseException line 5:26 mismatched input 'table' expecting KW_VIEW near 'replace' in create view statement
It seems the Spark engine expects for it to be a view, even when it does not exist yet.
Additionally, when I copy the compiled code above and run it using Jupyter, it runs just fine.
The profile used:
dbt_sg_hub:
  outputs:
    dev:
      host: <host_address>
      method: thrift
      port: 10000
      schema: <schema>
      threads: 4
      type: spark
      user: hadoop
  target: dev

Environment

OS: Windows 11 22H2 22621.3007
Python: 3.10.11
dbt-core: 1.6.9
dbt-spark: 1.6.2

As it seems the EMR will, sometimes, not alert if the Thrift server fails to start.
I fixed Thrift and the problem was gone.
Sorry if any amount of time on your side was spent on this, and thanks anyway.
Cheers.


Add app maintenance spec with enabled field

The enabled field allows users to enable/disable maintenance mode.
PLEASE DON'T MERGE UNTIL OCTOBER 22

Converting to a draft until this is ready to be reviewed and merged.


Testing: Resources/organization

Where do we store the test .wasm files how do we keep them versioned? Probably makes sense to keep them in one repo then use git submodules but we also discussed maybe building some infrastructure around this
Ensure that the extism-runtime.wasm kernel is updated automatically, or semi-automatically (4 hours)

SDKs that require the kernel should get automatic PRs when this file changes

I part 1 was taken care of with the extism/plugins repo
@zshipko is part 2 take care of by the development build process?

The only thing left is to update the go-sdk and js-sdk when the kernel changes. I think this could be considered complete and we can open some issues on those repos for more specific tasks.

Can continue manual approach for now while determining how to automate more fully

Noting that I'll take this on as a back-burner task. Goals are:

Make sure the Go and JS SDKs continue to work against extism-runtime.wasm changes
Keep the extism-runtime.wasm file vendored in the Go and JS SDKs up-to-date automatically. (Possibly by submitting PRs against those repos whenever the runtime artifact changes.)

@chrisdickinson this may be a viable option... https://github.com/orgs/community/discussions/49745
Imagine running a step like this (opening a PR with the new kernel to go-sdk and js-sdk) from the release action in extism/extism

Did we ever complete step 2? If not can we open a new issue to track it?

The kernel isn't being updated automatically right now since it's only really the Go SDK that needs to get updates and we just haven't automated that yet - I guess now there is also the wamr-sdk, so it could be worth it if we keep adding support for more runtimes.


Looking for maintainers

I'm looking for maintainers to keep an eye on this project and answer questions.  I don't use this project at the moment personally and I'm focused on few other projects, so I don't have the required time to answer questions, fix bugs and merge PRs.
If you are interested, please comment below.

Sorry for not responding on time. Thank you all coming forward to help on this. I'm added @lalitaphalak  as a collaborator for now. Hopefully, @lalitaphalak can help merge some open PRs and help others along the way.

Is there a way to add an itemclick to a widget, so that when the widget is clicked, it launches a url or another page?

I will try my best as I run into blockers while trying to use this as a foundation for a project I am working on. How quick are PR turn arounds in general?

@Raathigesh @lalitaphalak  hello i will help maintain this project we use it has our main dashboards and  i am very excited with typescript V2


return a 302 Redirect when visiting the link view

Fixes #3302
This way search engines get proper HTTP 302 Redirects.

@erral we are removing code that I think fixed some similar issues with the redirects in SSR. Do the <Redirect> component works in SSR?

Anyway I tried to add a test to this and I got an InvariantError saying that I should not be using the <Redirect> component out of a <Router> component, so I don't know whether this is a correct approach to return a proper HTTP Redirect :confused:

@erral try wrap it in a , there are some examples in the current tests.

Yes I tried that in the other PR, the RedirectBlock one, and it was OK, so now I have to take the time to bring that test here


Restrict lowest teaching quality/control to 1 not 0 in web interface

Have begun work on teacher-lower-limits branch


testTestSuiteExists  failing in Bazel release 0.11.0rc3

https://ci.bazel.build/blue/organizations/jenkins/Global%2Fbazel-integration-testing/detail/bazel-integration-testing/212/tests
Platforms:

darwin-x86_64

Blocking release https://github.com/bazelbuild/bazel/issues/3959

This looks really bazel core. But doesn't seem related to the two cherry picks.
It's trying to call java to build a jar.
Can you re run it?
On a related note- do you know who's responsible for ci.bazel.build? I'd like to get a user there for example so I can re-run tasks

The core error seems to be:
java.io.IOException: Cannot run program \"/private/var/tmp/_bazel_ci/4b9a32a7299075b6d80804c042d79b47/bazel-sandbox/2661807843882205/execroot/build_bazel_integration_testing/_tmp/8259161321b688c625eac361a1fc0e2e/26d3d1bbdb875e9b656d5a085df9de01/execroot/build_bazel_integration_testing/external/local_jdk/bin/java\" (in directory \"/private/var/tmp/_bazel_ci/4b9a32a7299075b6d80804c042d79b47/bazel-sandbox/2661807843882205/execroot/build_bazel_integration_testing/_tmp/8259161321b688c625eac361a1fc0e2e/26d3d1bbdb875e9b656d5a085df9de01/execroot/build_bazel_integration_testing\"): error=2, No such file or directory"

So possibly there's a missing dependency only on Darwin?

Also, @philwo is the person to ask about ci.bazel.build.

@katre do you think the failure is related to rc3 or it surfaced something by chance?
Because I'm not sure how the two cherry-picks from rc2 (which triggered a green run) could have that affect. I might be misunderstanding the remote change.
If it's by chance- do you know if and where I can find test history and fail percentage on jenkins?

Jenkins is pretty bad with test history, the page exists (https://ci.bazel.build/job/bazel-integration-testing/test_results_analyzer/) but frequently times out for me.
I suspect this is a flake, it looks suspiciously similar to https://github.com/bazelbuild/bazel/issues/4627, which is also not finding a binary that should exist.

This test has been deleted by #18.

Thanks!

@katre you'll close this after you see its green?

Yup. Still waiting for a few fixes to cut the next RC, but I'll take care
of closing this. Thanks for helping!
On Wed, Feb 14, 2018 at 8:58 AM Ittai Zeidman<EMAIL_ADDRESS>wrote:

@katre https://github.com/katre you'll close this after you see its
green?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/bazelbuild/bazel-integration-testing/issues/17#issuecomment-365614647,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAT7f7GHrLRSF2mt8NO6lRtFdUdtBBGEks5tUuZzgaJpZM4SCkaX
.

Fixed in 0.11.0rc4.


plugin: devto, push new articles directly to your dev.to account

Following how the mediumplugin was done I implemented a new plugin that can post any blog posts (written in .md or .rst) to your https://dev.to profile.
You just need to get the token as explained in the README.me file.
The plugin also check if the posts were already published to avoid republishing something that was already published.

Why did it fail ?

Your code style does not match PEP 8:
v8/devto/devto_plugin.py:84:21: E128 continuation line under-indented for visual indent
v8/devto/devto_plugin.py:88:34: E251 unexpected spaces around keyword / parameter equals
v8/devto/devto_plugin.py:88:36: E251 unexpected spaces around keyword / parameter equals
v8/devto/devto_plugin.py:94:17: E128 continuation line under-indented for visual indent

Please fix those errors. We also expect PEP 8-style variable names, so please replace toPost with to_post.

Thanks for the informations, now it should be good !


Federated repository resources can not manage about "proxies"

Is your feature request related to a problem? Please describe.
I' m trying to do IaC artifactory federated repository with "artifactory_federated_generic_repository" resource having proxies configuration even though terraform.io document does not say that proxies(or proxy) scheme exists.
As I expected, terraform prints error when i run "apply".
❯ terraform apply
│ Error: Unsupported argument
│
│   on main.tf line 46, in resource "artifactory_federated_generic_repository" "federated-generic-test":
│   46:   proxy     = "test-proxy"
│
│ An argument named "proxy" is not expected here.
╵

Describe the solution you'd like
`artifactory_federated_*_repository" resources should have proxies(or proxy) scheme as well as WebUI can set.
Additional context
I'm using:

artifactory-provider v7.11.2

@alexhung thank you!


Fix arc border with circumference over 2*PI

Fixes: #6171
Demo

When borderAlign is set to 'inner', the result doesn't look correct because the border appears at next to the end angle instead of at the start angle.

Similarly, when borderAlign is 'center', the border should also be drawn and visible at the the start angle as the background is transparent.

I think we need to fill and stroke arcs multiple times if the circumference is greater than 2*PI. The first arc would have an open path (open at the end edge), the following arcs would have 2 open paths for outer and inner circumferences and the last arc would have an open path again (open at the start edge).

Fixed drawing of borders. But in 2 steps instead of 3, first fully open full circles and then fully closed partial.
Demo updated.


Consider changing some button labels?

The button to typecheck the buffer is called Check/Vérifier, and it looked like some students did not understand that it actually means (basically) compiling the code.
The fact that the Grade button checks that the Code is correct may also add to the confusion.
Maybe those labels should be reconsidered?

Fixed by #172


ssl with cname

I have a CNAME record pointing to a bucket, how can i use a signedUrl without .s3.amazonaws.com
I can set this.host to just a bucket inside of  options.style === 'virtualHosted' if statement and it works.

You can't combine SSL and CNAME on S3. AWS doesn't allow this. This isn't a Knox issue. See here:
http://stackoverflow.com/questions/11201316/how-to-configure-ssl-for-amazon-s3-bucket

Thanks for digging this up and getting us some closure ... boo AWS.

@markstos Yes you are right, but I'm using CloudFlare and it gives me a certificate for my domain.
@domenic I used knox to sign the url and the problem was inside the Client function that puts s3.amazonaws.com to the end of host. It's pretty common to use CloudFlare these days. It's not a problem to write own signer, but why not give the possibility to use just the domain.


Aqara Roller Shade Driver E1

Hello - Due to time that has passed and additional further communication that we have not yet received, I believe we can close this PR. @Hongming6 Can you please confirm? Thank you.

Hello - Due to time that has passed and additional further communication that we have not yet received, I believe we can close this PR. @Hongming6 Can you please confirm? Thank you.

Hello,you can close it.


Unreal Spatial Sound

Docs Build status updates of commit c4af0d9:
:clock10: Pending: waiting for processors (98 builds ahead of you)
:warning: Docs Build is busy, currently there are 98 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 c4af0d9:
:warning: Validation status: warnings

File
Status
Preview URL
Details

mixed-reality-docs/TOC.md
:warning:Warning
View
Details

.DS_Store
:white_check_mark:Succeeded

mixed-reality-docs/images/unreal-spatial-audio-img-01.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-02.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-03.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-04.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-05.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-06.png
:white_check_mark:Succeeded
View

mixed-reality-docs/unreal-spatial-sound.md
:white_check_mark:Succeeded
View

mixed-reality-docs/TOC.md

Line 223, Column 5: [Warning-file-not-found] Invalid file link: 'unreal-spatial-audio.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Hey @hferrone, it looks like the doc is named unreal-spatial-sound.md but the link in the TOC is to unreal-spatial-audio.md

Docs Build status updates of commit 38b37f0:
:clock10: Pending: waiting for processors (98 builds ahead of you)
:warning: Docs Build is busy, currently there are 98 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 38b37f0:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.DS_Store
:white_check_mark:Succeeded

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-01.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-02.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-03.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-04.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-05.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-06.png
:white_check_mark:Succeeded
View

mixed-reality-docs/unreal-spatial-sound.md
:white_check_mark:Succeeded
View

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

A few minor edits:

"You're new sound asset will be created with a Details and all the spatial sound settings you could ask for:"- "you're" should be "your"
The first heading should probably be "Installing the Microsoft Spatial Sound plugin"
The second heading should probably be "Setting the spatialization..."

Docs Build status updates of commit 05cc4ef:
:clock10: Pending: waiting for processors (56 builds ahead of you)
:warning: Docs Build is busy, currently there are 56 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 05cc4ef:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.DS_Store
:white_check_mark:Succeeded

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-01.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-02.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-03.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-04.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-05.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-06.png
:white_check_mark:Succeeded
View

mixed-reality-docs/unreal-spatial-sound.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 953ea1d:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.DS_Store
:white_check_mark:Succeeded

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-01.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-02.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-03.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-04.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-05.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-06.png
:white_check_mark:Succeeded
View

mixed-reality-docs/unreal-spatial-sound.md
:white_check_mark:Succeeded
View

mixed-reality-docs/unreal-uxt-ch2.md
:white_check_mark:Succeeded
View

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

Docs Build status updates of commit 2bd1edc:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.DS_Store
:white_check_mark:Succeeded

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-01.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-02.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-03.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-04.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-05.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-06.png
:white_check_mark:Succeeded
View

mixed-reality-docs/unreal-spatial-sound.md
:white_check_mark:Succeeded
View

mixed-reality-docs/unreal-uxt-ch2.md
:white_check_mark:Succeeded
View

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

It would be good to update unreal-spatial-audio-img-02.png to reflect that the platform "HoloLens" should be selected in the left column to ensure that the settings are made for the correct platform.  It's also good to set for "Windows" as well for previewing in the editor.

Highlight Spatialization Method in unreal-spatial-audio-img-03.png

Docs Build status updates of commit ac6eae6:
:clock10: Pending: waiting for processors (9 builds ahead of you)
:warning: Docs Build is busy, currently there are 9 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 ac6eae6:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

.DS_Store
:white_check_mark:Succeeded

mixed-reality-docs/TOC.md
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-01.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-02.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-03.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-04.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-05.png
:white_check_mark:Succeeded
View

mixed-reality-docs/images/unreal-spatial-audio-img-06.png
:white_check_mark:Succeeded
View

mixed-reality-docs/unreal-spatial-sound.md
:white_check_mark:Succeeded
View

mixed-reality-docs/unreal-uxt-ch2.md
:white_check_mark:Succeeded
View

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


Windows 10 -  [-] PROGRAM ABORT : Test case 'id_000000' results in a crash

Hi,
I'm trying to fuzzing a DLL using harness and everything works fins when the harness is run directly with DynamoRio and also in debug mode.
However, when I start the fuzzing with afl-fuzz,  I get the following error:
[-] PROGRAM ABORT : Test case 'id_000000' results in a crash
         Location : perform_dry_run(), d:\fuzz\winafl-master\afl-fuzz.c:3080

Following are my commands:
Works-- >D:\Fuzz\DynamoRIO-Windows-7.0.0-RC1\bin64\drrun.exe -c winafl.dll -debug -coverage_module Test.dll -target_module Test.exe -target_offset 0x1070 -fuzz_iterations 10 -nargs 2 -- Test.exe VENUS.WPG  --> WORKS

Works --> afl-showmap.exe -o showmap.txt -D "D:\Fuzz\DynamoRIO-Windows-7.0.0-RC1\bin64" -- -coverage_module Test.dll -fuzz_iterations 5000 -target_module Test.exe -target_offset 0x1070 -nargs 2 -- Test.exe VENUS.WPG  --> WORKS

Fails --> afl-fuzz.exe -i wpg_in -o wpg_out -m none -t 10000+ -D D:\Fuzz\DynamoRIO-Windows-7.0.0-RC1\bin64 -- -target_offset 0x1070 -target_module Test.exe -coverage_module Test.dll -fuzz_iterations 5000 -- Test.exe @@

However, the same thing works on Windows 7 without any problem.
My system configuration is:

Windows 10 1803 (build 17134.1304) (Similar error happens on other Windows 10 builds as well, except for the latest version)
WinAfl build with following DR versions:
DynamoRIO-Windows-7.0.0-RC1 (Also tested with DynamoRIO-Windows-8.0.0 -1)
Harness compiled (x64) using VS2017

I think I tried pretty much every possible way I could find as a solution in other issues:

Increased the timeout for initial run
tried running the harness in debug mode as well as directly with DR
Disabled anti-virus/Windows Defender real-time protection etc.

Can you please help me figure out what I could be missing?

Also, for the x86 version of the compiled harness, I couldn't even run it using DR (32 bit) on Windows 10 and receive following error:
I assume, this could be the problem with DR.
May be it's a different issue in itself
D:\Fuzz\DynamoRIO-Windows-7.0.0-RC1\bin32\drrun.exe -- Test_x86.exe VENUS.WPG
<Application D:\Fuzz\winafl-master\build32\bin\Release\Test_x86.exe (3496).  DynamoRIO internal crash at PC 0x5dfefb6b.  Please report this at http://dynamorio.org/issues/.  Program aborted.
0xc0000005 0x00000000 0x5dfefb6b 0x5dfefb6b 0x00000003 0x00000000
Base: 0x5df20000
Registers: eax=0x00000000 ebx=0x0101f140 ecx=0xb786c6f4 edx=0x00000000
        esi=0x21886760 edi=0x21886730 esp=0x0101f160 ebp=0x0101f214
        eflags=0x0001
version 7.0.0, build 1
-no_dynamic_options -code_api -probe_api -stack_size 56K -max_elide_jmp 0 -max_elide_call 0 -no_inline_ignored_syscalls -native_exec_default_list '' -no_native_exec_managed_code -no_indcall2direct -no_aslr_dr -pad_jmps_mark_no_trace
0x0101f214 0x5dfca31e
0x21886758 0x00000000>

On Windows 10, you might want to get a weekly release from https://github.com/DynamoRIO/dynamorio/releases and compile WinAFL with it.
If everything fails, you could try https://github.com/googleprojectzero/Jackalope, it's usage is similar, but it's based on neither DynamoRIO nor AFL, so if it's indeed an instrumentation issue, it might work there (and if it doesn't, at least you'll get a signal that the issue is somewhere else).

Hi Ivan,
Thanks for the response.
I did try with Jackalope and it works fine.
I will try building winafl with the weekly release of DR and update here shortly.

Update:
The weekly build of DR solved the issue with x86 harness, which actually solves my purpose.
However, the x64 compiled harness still gives the same error.
I think this narrows down the problem a little towards instrumentation and DR.


Remove version: libjpeg-turbo.libjpeg-turbo.GCC version <IP_ADDRESS>

[Automated] It returns code over 400 in all urls
Microsoft Reviewers: Open in CodeFlow

URL: https://sourceforge.net/projects/libjpeg-turbo/files/<IP_ADDRESS>/libjpeg-turbo-<IP_ADDRESS>-gcc.exe/download
Status Code: 200
(Automated message - build 897)


Add Shivam-Katare to LinkFree

Fixes Issue

Changes proposed

Check List (Check all the applicable boxes)

[X] My code follows the code style of this project.
[ ] My change requires changes to the documentation.
[X] I have updated the documentation accordingly.
[X] All new and existing tests passed.
[X] This PR does not contain plagiarized content.
[X] The title of my pull request is a short description of the requested changes.

Screenshots

Note to reviewers

Oops, looks like I approved the PR twice accidentally. 😅

Oops, looks like I approved the PR twice accidentally. 😅

😂😂


schema_version column version not nullable on Sybase ASE

What version of Flyway are you using?
4.0.3
Which client are you using? (Command-line, Java API, Maven plugin, Gradle plugin, SBT plugin, ANT tasks)
Java API
What database are you using (type & version)?
Sybase ASE 15.0.3
What operating system are you using?
UNIX/LINUX
What did you do?
(Please include the content causing the issue, any relevant configuration settings, and the command you ran)
ran migrate for the first time including repeatable migrations that don't have a version
What did you expect to see?
schema_version contains a record for the repeatable migration with null values for version
What did you see instead?
java.sql.SQLException: Attempt to insert NULL value into column 'version', table '.....dbo.schema_version'; column does not allow nulls. Update fails.
looked at org.flywaydb.core.internal.dbsupport.sybaseASE.createMetaDataTable.sql and found below
CREATE TABLE ${table} (
...,
version VARCHAR(50),
...
Sybase' default is NOT NULL so the column should be defined as nullable.

Same with Sybase 12.5

Same for Sybase 15.7.
I tried adding a statement to make the schema_version.version nullable and run it through flyway but the migration just hangs the statement never completes.

@mturuta I did it manually outside of flyway and I have observed no issues.

The table must have been created with a pre-release of Flyway 4.0 as ever since the official release this column has always been nullable.

No the table was created with the latest 4.0.3
In the Sybase script that creates the table, the column is not declared as NULL or NOT NULL so Sybase defaults to NOT NULL.
I suggest adding "NULL" to the script (whcih I did) so there is no ambiguity.

@ogerardin Thank you very much for clarifying! That was a huge help! I guess I must have overlooked the last line in @chrstnk 's original report (facepalm!)


ODataQueryOptions causes ArgumentException: An item with the same key has already been added.

I am getting "ArgumentException: An item with the same key has already been added." when one of the data type of an operation parameter is ODataQueryOptions.
Using
.NET Core 3.1
System.Text.Json
Swashbuckle.AspNetCore 5.0.0-rc5
public IActionResult Get(ODataQueryOptions<WeatherForecast> queryOptions)
Stack Trace
System.Collections.Generic.Dictionary<TKey, TValue>.TryInsert(TKey key, TValue value, InsertionBehavior behavior) System.Collections.Generic.Dictionary<TKey, TValue>.Add(TKey key, TValue value) Swashbuckle.AspNetCore.SwaggerGen.JsonObjectHandler.CreateDefinitionSchema(Type type, SchemaRepository schemaRepository) Swashbuckle.AspNetCore.SwaggerGen.SchemaGeneratorBase+<>c__DisplayClass4_1.<GenerateSchema>b__0() Swashbuckle.AspNetCore.SwaggerGen.SchemaRepository.GetOrAdd(Type type, string schemaId, Func<OpenApiSchema> factoryMethod) Swashbuckle.AspNetCore.SwaggerGen.SchemaGeneratorBase.CreateReferenceSchema(Type type, SchemaRepository schemaRepository, Func<OpenApiSchema> factoryMethod) Swashbuckle.AspNetCore.SwaggerGen.SchemaGeneratorBase.GenerateSchema(Type type, SchemaRepository schemaRepository) Swashbuckle.AspNetCore.SwaggerGen.SwaggerGenerator.GenerateParameter(ApiParameterDescription apiParameter, SchemaRepository schemaRepository) Swashbuckle.AspNetCore.SwaggerGen.SwaggerGenerator+<>c__DisplayClass10_0.<GenerateParameters>b__1(ApiParameterDescription apiParam) System.Linq.Enumerable+WhereSelectListIterator<TSource, TResult>.ToList() System.Linq.Enumerable.ToList<TSource>(IEnumerable<TSource> source) Swashbuckle.AspNetCore.SwaggerGen.SwaggerGenerator.GenerateParameters(ApiDescription apiDescription, SchemaRepository schemaRespository) Swashbuckle.AspNetCore.SwaggerGen.SwaggerGenerator.GenerateOperation(ApiDescription apiDescription, SchemaRepository schemaRepository) Swashbuckle.AspNetCore.SwaggerGen.SwaggerGenerator.GenerateOperations(IEnumerable<ApiDescription> apiDescriptions, SchemaRepository schemaRepository) Swashbuckle.AspNetCore.SwaggerGen.SwaggerGenerator.GeneratePaths(IEnumerable<ApiDescription> apiDescriptions, SchemaRepository schemaRepository) Swashbuckle.AspNetCore.SwaggerGen.SwaggerGenerator.GetSwagger(string documentName, string host, string basePath) Swashbuckle.AspNetCore.Swagger.SwaggerMiddleware.Invoke(HttpContext httpContext, ISwaggerProvider swaggerProvider) Microsoft.AspNetCore.Builder.RouterMiddleware.Invoke(HttpContext httpContext) Microsoft.AspNetCore.Authorization.AuthorizationMiddleware.Invoke(HttpContext context) Microsoft.AspNetCore.Diagnostics.DeveloperExceptionPageMiddleware.Invoke(HttpContext context)
The issue occurs because of the below.
ODataQueryOptions<TEntity>: ODataQueryOptions
public ETag<TEntity> IfMatch { get; }
public ETag<TEntity> IfNoneMatch { get; }
ODataQueryOptions
public virtual ETag IfMatch { get; }
public virtual ETag IfNoneMatch { get; }
A possible solution would be to do a check if the key already exists and do either replace of the value or skip?

I found this bug report just now so I closed my reported issue, because it was duplicate.
https://github.com/domaindrivendev/Swashbuckle.AspNetCore/issues/1455

Same problem here :(

I'm seeing the following similar exception:
System.ArguementException: An item with the same key has already been added.  Key: ifMatch
Using:

.NET Core 3.1
Microsoft.AspNetCore.OData 7.3.0
Swashbuckle.AspNetCore 5.1.0
Swashbuckle.AspNetCore.Annotations 5.1.0

Reverting the Swashbuckle NuGet packages back to version 5.0.0 has resolved the issue for now.

I had a similar exception
_System.ArgumentException: An item with the same key has already been added. Key: ifMatch
at System.Collections.Generic.Dictionary2.TryInsert(TKey key, TValue value, InsertionBehavior behavior)    at System.Collections.Generic.Dictionary2.Add(TKey key, TValue value)
at Swashbuckle.AspNetCore.SwaggerGen.SchemaGenerator.GenerateObjectSchema(SerializerMetadata serializerMetadata, SchemaRepository schemaRepository)
... so on...
I used:

.NET Core 3.1 webapi only
OData 7.4.0 beta
Swashbuckle 5.1

Fix:
My app uses Newtonsoft.Json, because OData used it to format a url {$select}.
Swashbuckle v 5.1 is configured to use System.Text.Json by default.
As you can see in https://github.com/domaindrivendev/Swashbuckle.AspNetCore
If you're using Newtonsoft, then you'll need to install a separate package and explicitly opt-in to ensure that Newtonsoft settings/attributes are automatically honored by the Swagger generator
After installing package Swashbuckle.AspNetCore.Newtonsoft and settings explicit opt-in app works correctly

Yea we ended up explicitly use Newtonsoft as well, which forces us to not use System.Text.Json. Solving this issue was good but we aren't sure the side-affect to other components.

I sent a PR where I replaced the property.

Hi,
I have the same issue, but don't how solve the issue until your fix aa7f6ee is released with a new version.
I have this issue with the GeneratePolymorphicSchemas->discriminatorSelector
I try to name the property that descripes the type, but if I do so, I get the exception.

Hi when did you plan to release, we are waiting upgrade.


Draft: fix check script

Use Github's rest API to determine base branch.
Only run check stage on pull requests

ok to test

@gpucibot merge


How to do not cache jsonGraphError?

When i get jsonGraphError object, how to do not cache jsonGraphError object?

You can use the errorSelector to mutate the error object, and add an expiry field before it gets added to cache. Here's an example: http://netflix.github.io/falcor/documentation/model.html#the-errorselector-value
IIRC, I think there's a bug with the "Expire Immediately (as in, don't cache this value)" use case, but you can use a small negative value, for example $expires:-10 to indicate "Expire in 10ms from the time this is put in cache" to work around it. http://netflix.github.io/falcor/doc/global.html#Atom

@abasiri Maybe i should invalidate it after getValue execute

I believe the message was meant for @sdesai


CORS in Youtube error

Hi!
I'm getting console error:
www-widgetapi.js:100 Failed to execute 'postMessage' on 'DOMWindow': The target origin provided ('https://www.youtube.com') does not match the recipient window's origin ('http://localhost:8000').
any idea how to fix?

This one can be ignored. Or perhaps raise the bug with the YouTube team. We're using their API as-is.

OH! Thanks! I hate console errors, i think i have "TOC" hahaha,

It's understandable and I'm the same really. It's just not in my control. I'll have another look but as I say, we just use the API as per their examples :(

Nevermind :) Thanks for this wonderful plugin, soon you'll see it on a GIANT Project 👅!


get the tests passing on windows

closes #8289
Now that we are running on GHA we can pretty easily just run the tests on Windows and we've got tests covering the functionality in #8289

Messages

:book:
:sparkles: Thanks for your contribution to Shields, @chris48s!

  Generated by :no_entry_sign: dangerJS against 9cbfb3b3c58c4049b37e76f2da83697563b2c701

d313935 includes the fix suggested in https://github.com/badges/shields/issues/8289#issuecomment-1228951077 which gets most of the tests passing. There are still a few more failures I've not looked into yet.
For the moment I've just started with the core tests.

Cool. It may also be worth running some of the other test suites on Windows as well as linux - these kind of issues could happen in the frontend/package/etc too, but also I'm slightly wary of "build matrix inflation" (for want of a better term). Interested if you have any thoughts on that point..

but also I'm slightly wary of "build matrix inflation" (for want of a better term). Interested if you have any thoughts on that point..

Yeah I know what you mean, and it's especially true in my experience with GHA because of how granularly each leg shows up under the check status view.
I think my main interest for Windows is largely just to give us back a high degree of confidence that contributors on Windows can clone + install, then start the app, I'm much less concerned with having an exhaustive/equal testing suite on Windows


refactor: Restructure library with new changes

Description

Restructures code in similar manner as in #34 but involving changes from recently merged PRs.
Motivation and Context

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

[ ] I have updated the documentation accordingly. If you are changing code related to user secrets you need to really make sure that security documentation is correct.
[x] I have read the CONTRIBUTION_GUIDE document.
[x] I haven't repeated the code. (DRY)
[ ] I have added tests to cover my changes.
[x] All new and existing tests passed.

Not sure about interfaces in separate folder. Maybe each interface should be in it's folder eg. matcher-interface should be in matcher folder.


link.to_cpu() from chainerx array fails when link has state variables with gradient

Testing under chainer==6.0.0 with chainerx built and cupy installed.
Testing environment has only 1 GPU.
Code to reproduce:
import numpy as np

import chainer
import chainer.links as L

# device = chainer.get_device(0)  # When using cupy, below code works.
device = chainer.get_device('native')

gru = L.GRU(10, 1)
gru.to_device(device)

x = np.zeros((1, 10), dtype=np.float32)
x = device.send(x)
y = gru(x)

# gru.reset_state()
gru.to_cpu()  # This cause ERROR
print('done!')

which raises following error:
Traceback (most recent call last):
  File "tmp2.py", line 16, in <module>
    gru.to_cpu()
  File "/home/user/.pyenv/versions/anaconda3-5.1.0/lib/python3.6/site-packages/chainer/device_resident.py", line 75, in to_cpu
    self.__to_device(visitor)
  File "/home/user/.pyenv/versions/anaconda3-5.1.0/lib/python3.6/site-packages/chainer/device_resident.py", line 149, in __to_device
    self.device_resident_accept(to_device_visitor)
  File "/home/user/.pyenv/versions/anaconda3-5.1.0/lib/python3.6/site-packages/chainer/links/connection/gru.py", line 202, in device_resident_accept
    visitor.visit_variable(self.h)
  File "/home/user/.pyenv/versions/anaconda3-5.1.0/lib/python3.6/site-packages/chainer/device_resident.py", line 303, in visit_variable
    param.to_device(self._device)
  File "/home/user/.pyenv/versions/anaconda3-5.1.0/lib/python3.6/site-packages/chainer/variable.py", line 1139, in to_device
    self._to_device(device, allow_unchaining=False)
  File "/home/user/.pyenv/versions/anaconda3-5.1.0/lib/python3.6/site-packages/chainer/variable.py", line 1154, in _to_device
    '({}).'.format(device))
RuntimeError: A variable of a ChainerX array which requires gradients cannot be copied into non-chainerx device (@numpy).

When we use device = chainer.get_device(0) error does not happen.
In order to send chainerx array back to cpu, we need to call reset_state(), but it is inconvenient.

Can we consider workaround??

Hello,
The problem here is related to chainerx maintaining the computational graph.
The gru state requires the gradient to be computed for backpropagation and when it is copied out of chainerx the graph can be let in a non-consistent state.
In order to properly address this issue, we would like to know what you were intending to do and if you were expecting the computational graph to be cut.
Thanks!

Maybe it's okay to simply cut the ChainerX graph.

to_{cpu,gpu,device} for both Variable and Links with NumPy and CuPy in general breaks backprop when called in the middle of a computational graph, since backward function nodes will yield device mismatches.
Some users may temporarily transfer a device, and then transfer back? This will not work. A point is that a Variable or a Link gets in an "invalid state" when transferred. We could always cut the graph, even for NumPy and CuPy. We would then be able to avoid the "invalid state" saying instead that it becomes a leaf when transferred.

PR addressing this issue #7507.

This is still an issue.


intl package: add CLDR-based Iterable formatting

 Issue by seaneagan
Originally opened as dart-lang/sdk#11069

The intlx package (http://pub.dartlang.org/packages/intlx) has an IterableFormat API which formats Iterables using locale specific separators.  See the following links for more information:
Demo: http://seaneagan.me/intlx/
Docs: http://seaneagan.me/intlx/docs/intlx/IterableFormat.html
This should be moved to the intl package.

 Comment by sethladd

Added Pkg-Intl label.

 Comment by iposva-google

Added Area-Pkg, Triaged labels.


Upgrading to Dyn 365 BC from NAV 2018 gives an unnecessary warning

The following warning is shown when installing a Dyn 365 BC on top of a NAV 2018.
It does not have any effect and everything works, but the Installation looks like it failed:

This was also the case Upgrading a NAV 2018 to a newer CU

This turned out to be a bug that has now been fixed.


migrate.sh: prompt only once for username and comment

Instead of prompting the user for a name and a comment for each image package, prompt only once each time the script is run and use the same information for each file.

Done.

Confirmed. Closing.


Add py as requirement for test

PR test failing due to py is no longer being pulled in and installed as a required dependency of pytest-regtest. (related to this issue in pytest-regtest https://gitlab.com/uweschmitt/pytest-regtest/-/issues/19)

As noted in the comment on the GitLab issue it is pytest-7.2 that has depcrecated py dependency (see also PR 10936).
I guess unless/until pytest-regtest includes this as a dependency we will have to retain it as a dependency here under tests.


SAK-41690 Messages / Role issue when site language is set to Spanish …

…(Mexico)

Affects 12.x only.


[fix](dynamic partition) data lost with very small dynamic_partition.start

What problem does this PR solve?
How to reproduce:
Create a table with very small dynamic_partition.start property, then all partitions will be dropped wrongly, and make all the data lost.
"dynamic_partition.time_unit" = "MONTH",
"dynamic_partition.start" = "-100000",

Come from: https://github.com/apache/doris/pull/33712
Release note
None
Check List (For Author)

Test

[ ] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No code files have been changed.
[ ] Other reason

Behavior changed:

[ ] No.
[ ] Yes.

Does this need documentation?

[ ] No.
[ ] Yes.

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run buildall

run buildall

run buildall

The real root cause is the introduction of history_partition_num.  When creatting partition,  use partition_num = (-start,  history_partition_num),    but when dropping partition, use partition_num = -start.
user set a small history_partition_num and a much smaller start, then when creatting partiton, the start was skipped check.
In fact, history_partition_num and start has the same meaning (the history partition num),  only except that start = int.min means no deleting history partitions .
The introduce of history_partition_num is a bad thing.   It make the creating and dropping partition behaviour not consistent.


Mainboard link is 404

Hi, mainboard link from readme.md causes 404.

I went ahead and uploaded the rev3a design today https://github.com/goat-hill/bitclock/tree/master/bitclock-pcb/rev3a
I wasn't sure if I wanted to immediately open source the PCB design until I finished selling a first batch here https://bitclock.io/order
But given I don't ship outside of the USA so I don't have to work through the regulatory stuff I decided to open source it in case others want to order! I included ordering files from JLCPCB.


Update README #4477

Fixes #4477 Update README

add link for issue reporting
add entries for scan, screengrab, and supply tools
minor grammar and formatting tweaks

This is awesome @bgannin. Can't wait to see these ship!

PR feedback incorporated and pushed.

Not sure why CircleCI failed… there are no code changes.

Thanks for update the PR, I added a note about markdown being broken: https://github.com/bgannin/fastlane/blob/readme-update/README.md
Let me know when you had the time to update the PR so that I can do another review 👍

Fixed the erroneous markdown. Should be good to go now.

Seems like the build is failing, could you rebase from master and see if that fixes the issue? Sorry for the troubles

@KrauseFx Should be good now.

@hemal is this ready to be merged from your side?

@KrauseFx LGTM 🚀

Let's change this one last thing https://github.com/fastlane/fastlane/pull/4493#discussion_r62556260 and then we're ready to merge 👍 . You don't need to squash as we can do that on GitHub now 🚀

@KrauseFx Done 💯

Thanks, merged with https://github.com/fastlane/fastlane/pull/4600, I cherry-picked the commits 👍


a3c example ModelSaver stuck

I'm running a simulation with the a3c example setting  the ModelSaver to dump a checkpoint after every_k_steps=5000. The simulation is stuck at 4999 iteration for more than 16hrs (I assume it's saving).... I'm using google cloud 4 cpu virtual machine with Tesla K80.
Parameters:
LOCAL_TIME_MAX = 5
STEPS_PER_EPOCH = 10000
EVAL_EPISODE = 50
BATCH_SIZE = 128
PREDICT_BATCH_SIZE = 15    # batch for efficient forward
SIMULATOR_PROC = 30#number of processes at the same time
PREDICTOR_THREAD_PER_GPU = 1#THREADS #number of agents
PREDICTOR_THREAD = None
Any idea is appreciated...

For unexpected problems, please post issues following the issue template. In particular, include what you did (changes you made, command you run) and what you observed (logs).


Fix suggestions for git push -u origin

Resolves https://github.com/nvbn/thefuck/issues/558

Coverage increased (+0.02%) to 92.925% when pulling aa6b18d0ce64b1e9f902a617eb7b88b1b88877d0 on josephfrazier:git-push-explicit-upstream into 9660ec7813a0e77ec3411682b0084d07b540084e on nvbn:master.

Coverage increased (+0.02%) to 92.925% when pulling aa6b18d0ce64b1e9f902a617eb7b88b1b88877d0 on josephfrazier:git-push-explicit-upstream into 9660ec7813a0e77ec3411682b0084d07b540084e on nvbn:master.

Coverage increased (+0.02%) to 92.925% when pulling aa6b18d0ce64b1e9f902a617eb7b88b1b88877d0 on josephfrazier:git-push-explicit-upstream into 9660ec7813a0e77ec3411682b0084d07b540084e on nvbn:master.

Thanks!


[hotfix][postgres] Flink CDC Postgres Connector subsribe regular expr…

PostgresSourceBuilder postgresSourceBuilder = PostgresSourceBuilder.PostgresIncrementalSource.builder()
.tableList("test\.t_(.*)");
when table is regular expression, and match too many tables, Source will load all match table's columns , index, primary for each table chunk phase, actually only the conext tableId should load.

@leonardBang @yuxiqian  review please


implement threshold on index task

Optionally compute the index only on those areas that are above/below a specific threshold of another index (to be specified):

compute thresholding index (e.g. NDVI for separating vegetation from terrain)
only for those pixels above a threshold:

compute all the specified indexes

After some dev it got absorbed by thumbnail, since it covered all of its functionality'


Changing the style forces the page to reload

Problem
It looks like the page reloads when the page receives HMR event

Open the demo and wait 5 seconds until dynamic item is added to the page
Try to change css file, for example color of the  text in css declaration for .hello

Expected behavior
App should retain local state (dynamically added item) on update of styles, but instead the page reloads
Minimal reproduction URL
https://stackblitz.com/edit/vitejs-vite-x9tc9v?file=src%2Fstyle.module.css
Version
v1.6.0
Node.js version
v20.16.0
Package manager
npm
Operating system
macOS
Bugs are expected to be fixed by those affected by it

[ ] I'm interested in working on this issue

Compensating engineering work financially will speed up resolution

[ ] I'm willing to offer financial support

This behavior is expected due to a limitation with HMR in vanilla JavaScript modules. When editing a CSS Module imported by JavaScript, Vite triggers a full page reload because vanilla JS modules lack native HMR capabilities for runtime swapping, which also affects CSS Modules since they can't handle stateful updates by default.
This can be reproduced in a plain Vite setup: editing a CSS Module imported by JS will trigger a full reload. However, Vite has a known bug where changes in composed CSS Modules aren't detected, so it may not even trigger a reload in those cases (as noted in the README).
Vite's HMR works seamlessly with frameworks like React and Vue, which have state management to support atomic updates. For CSS Modules, HMR doesn't work if exported class names are altered since these changes can't be dynamically swapped. However, if the exported class names remain the same, it may be possible to support HMR by only replacing the stylesheet, reapplying the new styles without a reload.
I'm open to a pull request exploring this enhancement. For now, I'll close this as it's not a bug, but contributions are welcome if you'd like to pursue this functionality.


zoom in out functionality added and added responsiveness for mobile s…

…creen
Proposed Changes
Added a zoom in and zoom out in PrintPreview component added a button for zoom in and zoom out make a PrintPreview component responsive for mobile screen

Fixes #8323
added a button for zoom in and zoom
made component responsive for mobile screen

@coronasafe/care-fe-code-reviewers @coronasafe/code-reviewers
Merge Checklist

[ ] Add specs that demonstrate bug / test a new feature.
[ ] Update product documentation.
[ ] Ensure that UI text is kept in I18n files.
[ ] Prep screenshot or demo video for changelog entry, and attach it to issue.
[ ] Request for Peer Reviews
[x] Completion of QA

@ayushpatil2122 Were you able to make the changes?

Closing due to inactivity and completed in #8487


[Bug]: Flashing red cancelled box

Before Reporting

[X] I have checked FAQ, and there is no solution to my issue
[X] I have searched the existing issues, and there is no existing issue for my issue

What happened?
Whilst using codeium suggestions and typing a red box appears with cancelled in it. Suggestions do appear correctly after I stop typing but the red box is present (flashing effect) whenever typing.

How to reproduce the bug.
Select Codeium as a suggestion provider, with 0 debounce and start typing, particularly during comments. The open xcode file was a objc file which may/may not be relevant.
Relevant log output
No response
macOS version
13.4
Xcode version
14.3
Copilot for Xcode version
17
Node version
NA

Thanks for reporting, the fix is released in 0.17.1.

Updated already - amazing, thank you so much!


Connection limit per client filter

Title: Add support for limiting number of connections per each client
Description:

Envoy has the capability to limit the absolute number of connections via the network connection limit filter. It would also be useful to be able to limit the number of connections per each client, specifically on a per filter chain basis or based on some descriptors from the request.

cc @alyssawilk

sorry when you say per client, I'm not sure exactly what you mean. Can you elaborate a bit?

Sure! Client means IP address. Therefore, this filter should limit the number of concurrent connections from each source IP address. As a use case: limit the number of connections to a proxied DB from the same origin while still allowing connections from new/not-limited, different sources.
Don't hesitate to let me know if you have more doubts or suggestions!


[R2] Add presigned URL example to aws4fetch page

Which Cloudflare product does this pertain to?
R2
Existing documentation URL(s)

https://developers.cloudflare.com/r2/data-access/s3-api/presigned-urls/#presigned-url-alternative-with-workers
https://developers.cloudflare.com/r2/examples/aws4fetch/

Section that requires update

https://developers.cloudflare.com/r2/examples/aws4fetch/

What needs to change?
Add an example on using aws4fetch to generate presigned URLs.
How should it change?
Add the example from the presigned URLs guide onto the aws4fetch guide, like how it is for aws-sdk-js-v3
Additional information
The example relies on https://github.com/cloudflare/cloudflare-docs/pull/7392 being merged as it is currently incorrect.

When https://github.com/cloudflare/cloudflare-docs/pull/7417 is reviewed, I can pull across the updated Worker into the aws4fetch example.

When #7417 is reviewed, I can pull across the updated Worker into the aws4fetch example.

You're all set on #7147, Kian!


Don't download dependencies from other platforms

Having a Cargo.toml which looks like this:
        [package]
        name = "a"
        version = "0.0.1"
        authors = []

        [target.'cfg(unix)'.dependencies]
        foo = "0.1.0"

        [target.'cfg(windows)'.dependencies]
        foo = "0.2.0"

This would still download foo version 0.2.0 on unix. I think there is no need to do that, but please correct me if I'm wrong.
This was triggered by this stackoverflow question, but that situation is more complicated, as the version is the same, just the features are different. This PR will not solve that bug. If you want me to include that too, I would have to debug a bit more first....

Thanks for the pull request, and welcome! The Rust team is excited to review your changes, and you should hear from @alexcrichton (or someone else) soon.
If any changes to this PR are deemed necessary, please add them as extra commits. This ensures that the reviewer can see what has changed since they last reviewed the code. Due to the way GitHub handles out-of-date commits, this should also make it reasonably obvious what issues have or haven't been addressed. Large or tricky changes may require several passes of review and changes.
Please see the contribution instructions for more information.

@bors: r+
Thanks!

:pushpin: Commit 0a4fbbf has been approved by alexcrichton

:hourglass: Testing commit 0a4fbbf with merge 22bd52d...

:sunny: Test successful - cargo-cross-linux, cargo-linux-32, cargo-linux-64, cargo-mac-32, cargo-mac-64, cargo-win-gnu-32, cargo-win-gnu-64, cargo-win-msvc-32, cargo-win-msvc-64
Approved by: alexcrichton
Pushing 22bd52deb1fa4d4e364ffb3f07423fe3f7af58e4 to master...


Type replacement by PostgreSQL domain

This is for https://github.com/volatiletech/sqlboiler/issues/530
Changes

Added a new field DomainName in drivers.Column. This column should be psql only, and currently only the psql driver writes to it.
The psql driver writes the information_schema.columns's domain_name value in DomainName.
Type replacement now accepts a domain_name parameter on the match side, which matches against the DomainName field.
Bonus: fixed a bug where sqlboiler would crash when an array_type type match is used but matched against a column without one.

What doesn't work yet
Other databases' tests are yet updated.

Opening an upstream PR on volatiletech/sqlboiler#538


Rename project and npm package to github-manager-cli

I will work on it.

To rename a project and npm package, you have to do the following steps:

To change project name execute:

$ git remote set-url origin git@github.com:99xt/github-manager-cli
or if you are using HTTPS instead of SSH:
$ git remote set-url origin https://github.com/99xt/github-manager-cli.git

Regarding rename npm package:

You can't really do it. You have to create a new one.
But first you have to label the old one with 'deprecated' notification:
$ npm deprecate<EMAIL_ADDRESS>"WARNING: This module has been renamed to github-manager-cli. Please install it instead. See https://github.com/99xt/github-manager-cli for more information."
After this, accept my pull request #18 with package name change to github-manager-cli and execute:
$ npm publish

to complete this issue you can rename package.json file and update the  pull request
https://github.com/99xt/github-manager-cli

@charithsoori It looks like this is solved. Could you close it or remove the hacktoberfest label?

Done.


igv problem

hi,
igv started looking blurry, very strange, and not possible to work with.
I tried update, install again, closing, and opening ubuntu and searching online, but that didn't work.
If someone is familiar with this and has a solution that will be great.

WSL, Ubuntu, it worked fine until that suddenly happened

Cool
So try and use the native windows application from here
Let me know if this works better for you


fix: add description for the start command in AsyncAPI CLI (#1534)

Description
Fixed #1534
Added description for the start command in AsyncAPI CLI.

LGTM @toffee-k21

Hey @AayushSaini101, I have resolved the merge conflict

/rtm


Glob pattern driven decorations

Implementation Proposal
const deco1 = new GlobbyDecoration('cssClassname')

console.log(deco1 instanceof Decoration) // > true

deco1.addGlobRule('src/vendor/**/*.css')
deco1.addGlobRule('**node_modules**')

setTimeout(() => {
  deco1.removeGlobRule('**node_modules**')
}, 5000)

Behaviour

Should automanage everything as described by glob rule
At the low-level should keep everything efficient by adding/negating a minimum number of targets, and instead rely on inheritance.
When matching a path against a glob pattern, it should adjust pattern or path according to pathStyle specified by the host.
Actual targetting and negation should be deferred until absolutely necessary.

Notes
node-ignore is a great glob matcher with no external dependencies. (No interal deps eithers, like fs or path)

Remember to support TargetMatchMode.Self as well


Consistent ordering of current job openings

Makes job listing ordering on /careers/ consistent with the ordering on /careers/current-openings/ (ordered first by soonest to close and then alphabetically). Changes text on /careers/ from "Newest Openings" to "Current Openings".
Changes

Converted the careers index view and the current openings view to Django class-based views.
Modified index view to use an ordering of ('close_date', 'title') to be consistent with the jobs listing page.
Changed text on jinja2/v1/about-us/careers/index.html from "Newest Openings" to "Current Openings".
Updated CHANGELOG.md to include this update.

Testing

Added new unit tests for the above views. Created new jobmanager/tests module to hold these.
Added new dependency (only to requirements/test.txt) on the useful model_mommy package for easily mocking Django models.

Review

@kurtw @rosskarchner @richaagarwal

Screenshots
Careers page:

Todos

These pages don't gracefully handle the case when there are no job openings.

Checklist

[ ] Changes are limited to a single goal (no scope creep)
[ ] Code can be automatically merged (no conflicts)
[ ] Code follows the standards laid out in the front end playbook
[ ] Passes all existing automated tests
[ ] New functions include new tests
[ ] New functions are documented (with a description, list of inputs, and expected output)
[ ] Placeholder code is flagged
[ ] Visually tested in supported browsers and devices
[ ] Project documentation has been updated (including the "Unreleased" section of the CHANGELOG)

Looks good to me, but I'm not a code review. :) 👍

This looks great!  Sorry for the delay in reviewing.

Thanks all.


Unimplemented abstract method is not reported.

Environment data

Language Server version: 2020.12.2
OS and version: OSX 10.13.3
Python version (& distribution if applicable, e.g. Anaconda): Python 3.8

Expected behaviour&Actual behaviour
This code snippet should resulting an error about MixinA not implemented method a, but it doesn't.
Code Snippet / Additional information

import abc

class InterfaceA(abc.ABC):
  @abc.abstractmethod
  def a(self) -> None: print('InterfaceA.a')

class MixinA(InterfaceA):
  pass

ab = MixinA()
ab.a()

I also tried using Pyright alone, Pyright would report this error correctly:

Thank youl.

By default, the python.analysis.typeCheckingMode is set to "off" in Pylance. In Pyright it defaults to "basic". If you enable basic type checking in Pylance, you will get the desired behavior.

Thanks for for explaination.
I bilieve I did set the python.analysis.typeCheckingMode to "basic" when I report this issue, but I can confirm it's fixed with latest version of Pylance now, so I'm closing this issue now.

Thanks for for explaination.
I bilieve I did set the python.analysis.typeCheckingMode to "basic" when I report this issue, but I can confirm it's fixed with latest version of Pylance now, so I'm closing this issue now.

Why do you need to instantiate MixinA() to see the type warning?
The only reason I'm trying to use @abstractmethod is because I thought it would allow me to check the source code where the classes are defined.

Why do you need to instantiate MixinA() to see the type warning?

MixinA could be subclassed, and the subclass could provide the implementations for any remaining abstract methods, so it wouldn't be appropriate to emit an error. If you add a @final class decorator to MixinA, that tells the type checker and the runtime that the class cannot be further subclassed. In this case, pylance will emit an error telling you that some of the abstract methods are missing an implementation.


[Feature Request] EntityHealthChangeEvent (Cancelable)

Title says mostly it. An Event that is called when the Health are tried to be changed.
This opens the possibility to cancel the event and handle own logic at that point.
Probably not a big change and makes Minestom a little bit more customizeable.
Specific Use-case:

Whenever a player's Health will be set to 0 from any extension or other source, then there is an option to check it and cancel it
So you can prevent a player from getting the respawn screen (especially to prevent the "loading world" screen after clicking respawn or automatically respawning) because you can teleport the player to the same location and handle the death custom

In which case you can simply use EntityDeathEvent or equivalent and call your own EntityHealthChangeEvent

EntityDeathEvent is not cancelable


[ISSUE #859] Add unit tests for EventMesh SDK for Java

Fixes ISSUE #859.
Motivation
Add missing unit tests
Modifications
Add unit tests for eventmesh-sdk-java module.
Documentation

[ ] Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable / docs / JavaDocs / not documented)
If a feature is not applicable for documentation, explain why?
If a feature is not documented yet in this PR, please create a followup issue for adding the documentation

@wallezhang please fix the conflicts.

@wallezhang please fix the conflicts.

Done.

@ruanwenjun please check if the change you requested is resoved, then can merge it.


Skip the CDN for nuget.org/nuget.exe

We need to skip the CDN when serving nuget.org/nuget.exe, serving it directly from blob storage.  This will allow the new version to be available immediately after pushing NuGet.CommandLine.

@deepakaravindr is this one still relevant?

The was issue that there's no good way to tell CDN to refresh a certain URL and start serving latest content there, I expect that hasn't changed.
Have you considered doing anything to work around this from the client side like append a dummy query ?_=uniquifier to force bypassing CDN cache when getting nuget.exe?

Just scrolled through old issues: Is this still relevant with dist.nuget.org in place?
I also noticed that https://nuget.org/nuget.exe will serve version 2.8.6 - is this by design?

https://nuget.org/nuget.exe is no longer used. Closing the issue.


Implement multiple language analyzers on user queries

Closing, basic arabic functionality works at the moment. Will revisit later on if more advanced support is required.


[Feature] Support for mmselfsup

What's the feature?
Hi !
Does mmdetection support for mmselfsup ?
I wanna use moco to self_supervise train a pretrained model .
And then use the pretrained model on retinanet .
Any other context?
No response

If the backbone network of the pre-trained model and downstream tasks are match, the shift is realizable. This process has already been introduced in the mmselfsup doc.


feat: Use setup.cfg for dist package

Brief Summary of The Change

Use setup.cfg to config dist package. the benefit is we can use attr and file setup.cfg for convenient setting
Pull Request checklist
I confirm that the following checklist has been completed.

[x] Add/Change test cases for the changes.
[x] Add/Change the related documentation, should also change docs/source/config.rst when you change file default_config.yaml.
[x] (Optional) Add your change to UPDATING.md when it is an incompatible change.

Codecov Report

Merging #64 (5ce32bd) into main (129b056) will decrease coverage by 0.00%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #64      +/-   ##
==========================================
- Coverage   90.95%   90.95%   -0.01%
==========================================
  Files          61       61
  Lines        2212     2211       -1
==========================================
- Hits         2012     2011       -1
  Misses        200      200

Flag
Coverage Δ

unittests
90.95% <100.00%> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

src/pydolphinscheduler/__init__.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


add hostgssenc type to pg_hba rules

Postgres 12 supports hostgss encryption. Add hostgssenc to $type to allow this hba rule type to pass the assert_type function.

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@b78a2c2). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #1195   +/-   ##
=======================================
  Coverage        ?   65.81%
=======================================
  Files           ?       14
  Lines           ?      351
  Branches        ?        0
=======================================
  Hits            ?      231
  Misses          ?      120
  Partials        ?        0

Continue to review full report at Codecov.

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

Thanks for the enhancement @osijan


Memory usage in system.workers is incorrect

Bug Report
Briefly describe the bug
The memory usage / memory available reported by system.workers doesn't seem to be correct
SELECT
    worker_id,
    memory_usage
FROM system.workers

Query id: 78a72326-4ac2-48c1-9424-f573e79779d2

┌─worker_id────────────────┬──────────memory_usage─┬────memory_available─┐
│ byconity-vw-vw-write-3   │ 3.5862868230651673e-9 │<PHONE_NUMBER>523995136 │
│ byconity-vw-vw-write-2   │ 4.9217074860052914e-9 │<PHONE_NUMBER>400824320 │
│ byconity-vw-vw-write-4   │  3.498001888146975e-9 │<PHONE_NUMBER>532137984 │
│ byconity-vw-vw-write-1   │  4.911626660941695e-9 │<PHONE_NUMBER>401754112 │
│ byconity-vw-vw-default-0 │  2.002042975846053e-9 │<PHONE_NUMBER>670115840 │
│ byconity-vw-vw-write-0   │  4.257305619148612e-9 │<PHONE_NUMBER>462104576 │
│ byconity-vw-vw-write-5   │  1.809983274370098e-8 │<PHONE_NUMBER>185356800 │
│ byconity-vw-vw-write-6   │  1.215827438727502e-9 │<PHONE_NUMBER>742631424 │
└──────────────────────────┴───────────────────────┴─────────────────────┘

CPU on the other hand seems to be correct (if i understand it correctly, the CPU usage is percentage, right?, i.e., 0%-100%)

Hi @lazywei , could you tell more about how memory usage is incorrect. Is the real usage is bigger or smaller and how much is the difference? cc @zeromem

Hi @zeromem , I think currently the memory_available field haven't been implemented. You can take a look

Actually the memory_available is read from /proc/meminfo. or /sys/fs/cgroup/memory/memory.usage_in_bytes and /sys/fs/cgroup/memory/memory.limit_in_bytes if it's in a container.  @lazywei you can check those files on your worker pod.

OK confirmed it works if we set the CPU/RAM resources request & limit explicitly on the k8s. Previously we didn't set any resource limit so perhaps it's not reading from the actual physical machine but rather the container's info only.
Thanks!


minimumNights when end date is selected first

Hi, Really loving this lib so far, it's working really well and I've found it pretty easy to get up and running with :)
So I'm having a problem with the minimumNights property. It could be a bug or it could be that I'm misunderstanding how it works. I have a date range picker where I need to prevent the user from selecting less than 3 nights. So I currently have the minimumNights prop set to 3. When selecting a start date, then and end date, this works great. However, when I select an end date first, then a start date, the days immediately preceding the selected end date are not blocked out. I noticed the same happens on your live playground too (clear the start date then pick an end date):
http://airbnb.io/react-dates/?selectedKind=DRP - Day Props&selectedStory=with minimum nights set&full=0&addons=1&stories=1&panelRight=0&addonPanel=storybook%2Factions%2Factions-panel
I was wondering if this a bug or if it's expected functionality? I would assume that if a user picked '10th Feb 2018' as an end date, for example, then for a min nights prop of 3, the 9th, 8th and 7th would not be selectable for a start date?

I would expect either that; or, I'd expect that selecting the 8th and the 9th (the 7th would be 3 nights prior to the 10th) would unselect the end date, since the 10th would now be invalid (based on a start date of the 8th or 9th).

@ljharb Ahh good idea, yeah I guess that would work too. So for your solution, I guess this is logic I can do myself on datesChanged(), but, ideally those invalid days (8th, 9th) would just be un-selectable, seems a bit more user friendly. Is this possible do you think?

Either seems appropriate; neither seems like a bug to me.

@ljharb So for the minNights functionality/validation to work, it relies on the user selecting the start date then the end date, it will not work if the end date is selected first. If that's expected then that's fine but just wanted to check before I added the logic for this into my component. Wanted to make sure I hadn't missed something glaringly obvious!

Hey, though I am a beginner, I am trying to work on it. If anybody can give me some insights like where the block block dates thing is calculated etc. please let me know.


Can not save a page

If we create a new page from /admin/pages/, edit and then save it, the page will be turned into a post, which is not an expected behavior.
The reason behind this is that when saving an existing page, we are actually using the route POST /admin/editor/:id/. The request is handled by PostSaveHandler, which sets p.IsPage = false.
We may need to create a new general handler(e.g. ContentSaveHandler) for POST /admin/editor/:id/. The handler should retrieve the existing post/page by id, change the fields according to form value and save it.
In addition, by doing this we will have some duplicate code among ContentSaveHandler, PostSaveHandler and PageSaveHandler. It would be great if we can extract some code from them and avoid the duplication.

Fixed by #44


Make it possible to run test on rust beta and make bench only available with feature nightly.

Awesome, Thanks @damienrg


Provide prebuilt binaries

Since this project can be built into a single binary, it would be great it could be downloaded without having to have any build environemnt

Done, I cut a release and attached the binary: https://github.com/etsy/Hound/releases/tag/0.1


Update about us section

I'm Yanda Saketh, a beginner to open source. I'm a hactoberfest participant and I would like to update contact us section by adding the content below.
It's a platform designed to store Some Astonishing CSS-Effects
ranging from different areas using
various templates.
Add some Cool Templates here!

@arpit456jain please assign this issue to me.

@YandaSaketh hi please assign this issue to me.

go ahead @YandaSaketh

@YandaSaketh any updates?

@arpit456jain I have already created a pr. You have made changes as well.
here is the link   https://github.com/arpit456jain/Amazing-Css-Effects/pull/220
Thank you.

ok @YandaSaketh


Screenshots should only be taken if Testcafe signals an error

Hi @camueller I want to merge in this change and give you credit, but it looks like the Pass on command line arguments to Testcafe is also in this PR which I don't want to merge in at the moment. Could you separate these?

Created a new pull request containing only the screenshot fix: https://github.com/rquellh/testcafe-cucumber/pull/7


NodeElement container

Is it correct that the container option only allows a selector, but no NodeElement?
I would like to create a Macy instances for multiple elements without using a id.
document.querySelectorAll('.masonry').forEach((elem) => {

	const macy = Macy({
		container: elem,
		trueOrder: false,
		waitForImages: false,
		margin: 24,
		columns: 6,
		breakAt: {
			1200: 5,
			940: 3,
			520: 2,
			400: 1
		}
	})

})

I got the same problem. Have You discovered something about that @electerious ?
Even when using class selector - only first child is working.

@electerious i found a workaround in one of closed issues:
https://github.com/bigbite/macy.js/issues/34#issuecomment-337539077
Hope it will help You too :)

Hi @sjankowski and @electerious
As this seems to be a highly requested feature, I will work to implement for the next minor version.
I'll update this issue once complete.

hey @electerious @sjankowski  this has now been published to npm as version 2.4


InConditionBuilder $queryBuilder visibility

Currently it's private and is defined in the constructor itself.

Then in implementations for specific DBs it's overwritten:

How about:

Declaring it as protected property (not in constructor).
Removing __construct() from implementations.

The same applies to AbstractDDLQueryBuilder, DMLQueryBuilder, QueryBuilder.

@terabytesoftw let's change additional classes I've mentioned as well.


subplot_grid - cols option?

Hi, it's not obvious from the documentation, but I was wondering if subplot_grid has an ncol option in the same way as ggplot2's facet_wrap?
I have many groups for grouping the x axis, and it lays all the subplots out horizontally which squashes them, I'm looking for a way to make it split the row of subplots into multiple rows to avoid the squashing.
Thanks,
Ben.

I don't think a facet_wrap has been implemented in Gadfly.
But here is an example (also see #841 for another way).
using DataFrames
x = linspace(-2, 2, 11)
f(a) = a*x+5.0*randn(11)
# First make some data, without indices:
D1 = vcat([DataFrame(x=x, y=f(a)) for a in linspace(2,16,8) ]...)
# Now make some col and row indices:
D2 = vcat([DataFrame(x=x, xg=col, yg=row)  for col in 1:4, row in 1:2  ]...)
# and concatenate:
D = hcat(D1,D2[:,2:3])

coord = Coord.cartesian(ymin=-50, ymax=50)
theme(x) = Theme(default_color=parse(Colors.Colorant, x))

 p = plot(D, xgroup=:xg, ygroup=:yg,
    Geom.subplot_grid(coord,
        layer(x=:x, y=:y, Geom.point, order=1),
        layer(x=:x, y=:y, Geom.smooth(method=:lm), theme("red"), order=2)
    ),
    Guide.xlabel("x"),
    Guide.ylabel("y")
)

You can stop the group labels from appearing by moving xgroup= and ygroup= into each layer.

In case this isn't flexible enough for you, it's pretty straightforward in Plots:
ys = map(f,linspace(2,16,8))
scatter(x, ys, layout=(2,4), smooth=true, link=:all, lc=:red)

Note: you probably want to grab the master branch as I haven't tagged some bug fixes.

I'm looking for how to do the equivalent of a facet_wrap in ggplot2 or seaborn, and this thread seems like the closest match for Gadfly. I'm guessing the prefered solution now is to put all individual plots into a vector, then reshape & plot with gridstack...? Do let me know if there's a more idiomatic solution. Thanks!

Nope, no facet_wrap yet.  Currently changing some internal code (#1520) which might make the development of subplot_wrap easier.


Mocktail test version out of date and blocking libraries

Describe the bug
Im trying to update another framework we use and mocktail (I see it hasnt been updated in 10+ months) is using an old version of test where newer libraries rely on newer versions.
Is there a plan to update Mocktail in the near future or has it been abandoned? or should I learn/convert to Mockito?
Thanks for the input.
Because test >=1.16.0-nullsafety.19 <1.16.6 depends on test_api 0.2.19 and test >=1.16.6 <1.17.10 depends on analyzer ^1.0.0, test >=1.16.0-nullsafety.19 <1.17.10 requires test_api 0.2.19 or analyzer ^1.0.0.
And because test >=1.17.10 <1.20.0 depends on analyzer >=1.0.0 <3.0.0 and test >=1.20.0 <1.21.2 depends on test_api 0.4.9, test >=1.16.0-nullsafety.19 <1.21.2 requires test_api 0.2.19 or 0.4.9 or analyzer >=1.0.0 <3.0.0.
And because test >=1.21.0 <1.21.6 depends on analyzer >=2.0.0 <5.0.0 and test >=1.21.6 <1.21.7 depends on test_api 0.4.14, test >=1.16.0-nullsafety.19 <1.21.7 requires test_api 0.2.19 or 0.4.9 or 0.4.14 or analyzer >=1.0.0 <5.0.0.
And because test >=1.21.7 <1.22.0 depends on test_api 0.4.15 and test >=1.22.0 <1.22.1 depends on test_api 0.4.16, test >=1.16.0-nullsafety.19 <1.22.1 requires test_api 0.2.19 or 0.4.9 or 0.4.14 or 0.4.15 or 0.4.16 or analyzer >=1.0.0 <5.0.0.
And because test >=1.22.1 <1.22.2 depends on test_api 0.4.17 and test >=1.22.2 depends on test_api 0.4.18, test >=1.16.0-nullsafety.19 requires test_api 0.2.19 or 0.4.9 or 0.4.14 or 0.4.15 or 0.4.16 or 0.4.17 or 0.4.18 or analyzer >=1.0.0 <5.0.0.
Because no versions of drift_dev match >2.4.1 <3.0.0 and drift_dev 2.4.1 depends on analyzer ^5.2.0, drift_dev ^2.4.1 requires analyzer ^5.2.0.
Thus, if drift_dev ^2.4.1 and test >=1.16.0-nullsafety.19 then test_api 0.2.19 or 0.4.9 or 0.4.14 or 0.4.15 or 0.4.16 or 0.4.17 or 0.4.18.
And because mocktail >=0.1.3 depends on test ^1.16.0, if drift_dev ^2.4.1 and mocktail >=0.1.3 then test_api 0.2.19 or 0.4.9 or 0.4.14 or 0.4.15 or 0.4.16 or 0.4.17 or 0.4.18.
And because app depends on flutter_test from sdk which depends on test_api 0.4.12, drift_dev ^2.4.1 is incompatible with mocktail >=0.1.3.
So, because routific_driverapp depends on both drift_dev ^2.4.1 and mocktail ^0.3.0, version solving failed.
pub get failed (1; So, because app depends on both drift_dev ^2.4.1 and mocktail ^0.3.0, version solving failed.)

Hi @JustinmClapperton 👋
Thanks for opening an issue!
The version of test being used by mocktail supports the latest version of package:test. This issue is a duplicate of https://github.com/felangel/mocktail/issues/164 and has been fixed in the master channel of Flutter. Bumping the version of package:test in mocktail will not fix this issue as the constraint in mocktail is very wide as supports the latest version of package:test.

hmm, the only way I can resolve this issue is removing mocktail.  All other packages work fine.  So im not sure how to resolve at this point then.   The library in conflict right now is Drift.
I will look at the duplicate thread and maybe find an answer there but at a brief glance it looked like it had no conclusion either.

hmm, the only way I can resolve this issue is removing mocktail. All other packages work fine. So im not sure how to resolve at this point then. The library in conflict right now is Drift.
I will look at the duplicate thread and maybe find an answer there but at a brief glance it looked like it had no conclusion either.

You can either upgrade to the master channel of Flutter temporarily or add a dependency_override for package:test_api as a temporary fix:
dependency_overrides:
  test_api: 0.4.16

@JustinmClapperton @felangel  #180

@amondnet this change should not be necessary. You should be able to fix the resolution issues by upgrading to the latest stable flutter channel (3.7).

Closing this since the issue is resolved on the new stable flutter channel (3.7)


Allow long read-only with when polish_with_short_reads set to 'False'.

I can do long read-only right now, but I have to make two fake Illumina files and turn the short read polish flag in the config to False.

We can modify the rule rule set_up_sample_directories: with some config-dependent switching logic and not need the short reads. This would be the low-hanging fruit.
Originally posted by @LeeBergstrand in https://github.com/rotary-genomics/rotary/pull/114#discussion_r1484858455

@jmtsuji rule summarize_contigs_by_coverage in polish.smk fails when there is no short reads.

It refers to both long and short rules in the same rule.

@jmtsuji Fixed by https://github.com/rotary-genomics/rotary/pull/146


[BUG] Through the graphscope service constructed by helm, the role and rolebinding resources cannot be deleted normally when the application is deleted

Describe the bug
Through the graphscope service constructed by helm, the role and rolebinding resources cannot be deleted normally when the application is deleted.
To Reproduce
Steps to reproduce the behavior:

helm install graphscope with the version 0.18.0
delete the namespace
The namespace is always in the deleted state and there are some resources like role and rolebinding always exist

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Environment (please complete the following information):

GraphScope version: v0.18.0
OS: Linux(Ubuntu18.04)
Kubernetes Version:1.22.10

Have been fixed by https://github.com/alibaba/GraphScope/pull/2350, which has already been included in our latest release v0.19.0.
Closing as already fixed. Thanks for reporting!


Remove condition to not create review apps from forked repositories

Hey @arielj,
This PR fixes #2.
Please check it out.
Thanks,
Ernesto

@arielj Thanks for the quick review! I just removed the fork const from that function. 👍🏼


[ADD] Support parameter blocks in Hessian

Instead of multiplying with the Hessian w.r.t. the concatenation of all flattened params, this feature allows
breaking the parameters into smaller blocks and multiplying with the implied block-diagonal Hessian approximation.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

-3 of 28 (89.29%)  changed or added relevant lines in 3 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.04%) to 89.116%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

curvlinops/_base.py
7
10
70.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.04%

Covered Lines:
958

Relevant Lines:
1075

💛  - Coveralls


chore: integrate melos (without changing structure)

Introduces Melos without changing the directory structure.

I revisited this in https://github.com/simolus3/drift/pull/2060, drift is now using melos. Using it with usePubspecOverrides: true for the bootstrap command works really well together with dart run and build_runner. Thanks for the idea and your support!