SQLReplaceStatement is NOT supported

Caused by: io.r2dbc.spi.R2dbcNonTransientResourceException: [15022,<PHONE_NUMBER>401217201600608703151428878] statement type: class com.alibaba.fastsql.sql.ast.statement.SQLReplaceStatement is NOT supported!
	at dev.miku.r2dbc.mysql.ExceptionFactory.mappingSqlState(ExceptionFactory.java:115) ~[blob_p-6fe34b669c7ab9de3d81cd9c1efb7e138b449863-783e18bcebf90e8f78d9d1e71b474b69:?]
	Suppressed: reactor.core.publisher.FluxOnAssembly$OnAssemblyException:
Error has been observed at the following site(s):
	|_ checkpoint ⇢ SQL "REPLACE INTO t_temp (id,name,age) VALUES :tuples"

Sounds like you are using AnalysisDB for MySQL.
This exception is reported by the AnalysisDB server, because the exception is created by ExceptionFactory.mappingSqlState.
Therefore, this may be a problem caused by AnalysisDB, could you tell me more details?

yes, I using AnalysisDB for MySQL.


The War Within Season 1

This contains all new TWW dungeons as well as the returning dungeons for S1.
As always, it probably still contains some minor inaccuracies or mistakes, but it should be good to go for release and I will update it accordingly.

Awesome, thank you so much. I'll merge this and release it once I fix the tooltip issue that popped up with WW


Datepicker: verify if a date is restricted

I want to loop through 2 dates in order to see if all the dates in the interval are restricted. I looked at the code and I found the isRestricted(date, month, year) function. This seems to be the function I need but I don't know how to use it properly (I'm not totally good with jQuery plugins handling).
I tried sometghing like this and it always return false so I suppose it's not the right way:
$('#datePickerBegin').datepicker('isRestricted', new Date('06/17/2015'))

Best regards,

I figured it out:
    $('#datePickerBegin').datepicker('isRestricted', 7, 17, 2015);


How to check a key by pattern in map[string]interface{}.

Allow this for in operator:
contains(key, 'ab') in {abc: 1, b: 2}
or
has('ab*') in {abc: 1, b: 2}

You can define helper like this Keys(map) []string and do something like:
'ab' in Keys(map)

any(Keys(map), {# contains 'ab'})

@antonmedv thanks, Ya it's working. I hope there is other way.
But it is good to have some basic methods inbuilt with the package itself like keys, values etc.
By the way thanks man !


chore: remove rivet_api_endpoint

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#168  👈
#166
#165
#164
#163
#162
#161
#160
#159
#158
#157
#156
#155
#154
#153
#152
#151
#150
#149
#148
#147
#146
#145
#144
#143
#142
#141
#140
#139
#138
#137
#136
main

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

Join @NathanFlurry and the rest of your teammates on  Graphite


Proposes more flexible AzureADOptions, and defaulting to AzureAD v2.0

Proposes improvements to the AzureADOptions:

make it possible to specify the Audience of the tokens that will be accepted (with a default which is consistent with the Azure AD experience). For the moment it's hardcoded to {ClientId}
make it possible to specity the Authority (for the moment it's hardcoded to {Instance}/{TenantId}. This is needed to make v2.0 primary without having users to have the change app generated by dotnet new mvc
Proposing to remove Domain, and rename TenantId in Tenant as the tenant identification can be specified as a TenantID (a Guid), or a domain, or some well known "meta-tenants" (common, organizations and consumers)

Proposing that Web applications are now v2.0 apps by default.
Proposing improvements to the XML documentation
Note that
This PR is essentially to start the discussion. It's not mean to be final. In particular, no special effort was made for the tests yet

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

@Tratcher @brentschmaltz FYI

@blowdart @BillHiebert This would need to be accompanied with template and tooling changes to get the right fields populated in config.

Without tooling changes this can't be merged, the VS UI would get everything wrong. So it's not as simple as this PR :)

I'm well aware of this, @blowdart : this is a proposal to help the discussion that @Tratcher and I have been having offline. Some of the things we'd want to consider is the breaking changes:

are we happy with renaming TenantId to Tenant? or not.
are we happy to remove Domain (which is not used anywhere btw)

Then as far as template and tooling changes, could you please send me the location of the repos, so that we propose PRs?

I have no real feelings about the changes, if they matched the Azure Portal wording for these details that would be best (assuming this won't change of course)

@phenning for tooling.

Template code: https://github.com/aspnet/Templating/blob/master/src/Microsoft.DotNet.Web.ProjectTemplates/content/StarterWeb-CSharp/appsettings.json

https://github.com/aspnet/AADIntegration/issues/50


Error in salsa family key generator

The salsa family key generator discards most of the random bits it harvests

Resolved by merge of feature/21


Add support for hyperlinks and relationship.target_mode

docx.document_rels.relationships may contain hyperlinks to (target_mode ==) external websites.
Also re-enabled support for themes. They were commented out, but tests are doing OK.
Added integration tests for parsing hyperlinks and images.

Hi @cstkinkey,
Thanks for merging my PRs so promptly. Would you mind publishing an update
to crates.io too, please? I would like to publish my own docx to
markdown/json parser/converter, and it requires me to point to a version of
docx-rs on crates.io.
Thanks, best regards,
Erik
Op vr 17 mei 2024 om 02:04 schreef cstkingkey @.***>:

Merged #11 https://github.com/cstkingkey/docx-rs/pull/11 into master.
—
Reply to this email directly, view it on GitHub
https://github.com/cstkingkey/docx-rs/pull/11#event-12841255605, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAX6YO6TSPLVB5IBC25MXUDZCVCRNAVCNFSM6AAAAABH2VSRSOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSHA2DCMRVGU3DANI
.
You are receiving this because you authored the thread.Message ID:
@.***>

Hi @cstkinkey, Thanks for merging my PRs so promptly. Would you mind publishing an update to crates.io too, please? I would like to publish my own docx to markdown/json parser/converter, and it requires me to point to a version of docx-rs on crates.io. Thanks, best regards, Erik Op vr 17 mei 2024 om 02:04 schreef cstkingkey @.>:
…
Merged #11 <#11> into master. — Reply to this email directly, view it on GitHub <#11 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAX6YO6TSPLVB5IBC25MXUDZCVCRNAVCNFSM6AAAAABH2VSRSOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSHA2DCMRVGU3DANI . You are receiving this because you authored the thread.Message ID: @.>

done

Great, thanks for the quick response! I've been able to publish my own
crate now :-)
docx-parser - crates.io: Rust Package Registry
https://crates.io/crates/docx-parser
Op di 21 mei 2024 om 12:24 schreef cstkingkey @.***>:

Hi @cstkinkey, Thanks for merging my PRs so promptly. Would you mind
publishing an update to crates.io too, please? I would like to publish my
own docx to markdown/json parser/converter, and it requires me to point to
a version of docx-rs on crates.io. Thanks, best regards, Erik Op vr 17
mei 2024 om 02:04 schreef cstkingkey @.
>: … <#m_8568128961820417247_> Merged #11
https://github.com/cstkingkey/docx-rs/pull/11 <#11
https://github.com/cstkingkey/docx-rs/pull/11> into master. — Reply to
this email directly, view it on GitHub <#11 (comment)
https://github.com/cstkingkey/docx-rs/pull/11#event-12841255605>, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAX6YO6TSPLVB5IBC25MXUDZCVCRNAVCNFSM6AAAAABH2VSRSOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSHA2DCMRVGU3DANI
https://github.com/notifications/unsubscribe-auth/AAX6YO6TSPLVB5IBC25MXUDZCVCRNAVCNFSM6AAAAABH2VSRSOVHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSHA2DCMRVGU3DANI
. You are receiving this because you authored the thread.Message ID: @.>
done
—
Reply to this email directly, view it on GitHub
https://github.com/cstkingkey/docx-rs/pull/11#issuecomment-2122294857,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAX6YO4NXLERDAWFENIC7C3ZDMOFVAVCNFSM6AAAAABH2VSRSOVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCMRSGI4TIOBVG4
.
You are receiving this because you authored the thread.Message ID:
@.***>


basePath is not documented

Under "Advanced", there is an option to set the base path (equivalent to passing -BasePath to nuget.exe), but this is not documented, nor is the reason you would or would not use this option.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 0fa37a3f-6551-16ad-e164-a4ac522cdaab
Version Independent ID: a3fd6935-2b99-92ef-4eb3-3249fca01f8a
Content: NuGet restore, pack, and publish task - Azure Pipelines
Content Source: docs/pipelines/tasks/package/nuget.md
Product: devops
Technology: devops-cicd
GitHub Login: @chasewilson
Microsoft Alias: phwilson

@IanKemp  Thank you for the feedback.
Base path is appended when including files as described in Nuspec in case you wanted a more customized build process.
I'll see about including a brief description in the parameter file.

The documentation has been updated.


[Task] Update ETL notebooks to generate the schema.pbtxt

Description
The ETL's should be updated to showcase how to generate schema.pbtxt with NVTabular.

this was partially done via https://github.com/NVIDIA-Merlin/Transformers4Rec/pull/471.


Travel time seems off...

Did I use AVG rather than SUM on the travel times perhaps?

Seems that all stations in orderedstations have length_mid of null.
Holding pending reply from kat regarding which one is correct to use.

Cancelled, schema is changing so that length will be the correct one to use.

Need to avg detectors for lane speed, and then sum that... (sum(for each station, avg all it's detectors))
Probably need another sub-query.

Resolved at https://github.com/PSU-Capstone-2014-B/routelandia-server/commit/d186306e95693607f52ba3345bd75d5221688e71


ability to save or not save video call

Libary is very ok but please add an ability to save or not save a video call without it my project is useless

did you mean to recording the video/audio call?

yeah

Sorry, for late reply, It was duplicate, I already schedule it in the next 2.9.0 release, If you want to follow changes, you could move to this link. Thx.


chore: Refactoring & comments

Codecov Report

Merging #644 into master will decrease coverage by 0.06%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #644      +/-   ##
==========================================
- Coverage   99.60%   99.54%   -0.07%
==========================================
  Files          57       57
  Lines        3541     3506      -35
  Branches      610      610
==========================================
- Hits         3527     3490      -37
- Misses          5        6       +1
- Partials        9       10       +1

Flag
Coverage Δ

#unittests
99.54% <100.00%> (-0.07%)
:arrow_down:

Impacted Files
Coverage Δ

src/schemathesis/models.py
99.38% <ø> (-0.02%)
:arrow_down:

src/schemathesis/_hypothesis.py
100.00% <100.00%> (ø)

src/schemathesis/extra/pytest_plugin.py
97.53% <0.00%> (-2.47%)
:arrow_down:

src/schemathesis/specs/graphql/schemas.py
96.55% <0.00%> (-0.23%)
:arrow_down:

src/schemathesis/runner/impl/core.py
98.59% <0.00%> (-0.02%)
:arrow_down:

src/schemathesis/lazy.py
100.00% <0.00%> (ø)

src/schemathesis/hooks.py
100.00% <0.00%> (ø)

src/schemathesis/stateful.py
100.00% <0.00%> (ø)

src/schemathesis/cli/context.py
100.00% <0.00%> (ø)

... and 7 more

Continue to review full report at Codecov.

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


poetry version

[x ] I have searched the issues of this repo and believe that this is not a duplicate.
[x ] I have searched the documentation and believe that my question is not covered.

Question

I want to bump versions from within a python script that handles my project management workflow.
my script calls
        i = ArgvInput(['poetry','version',rule])
        Application().run(i)

which under the hood creates a VersionCommand and executes handle() on it :
def handle(self):
version = self.argument("version")
    version = self.increment_version(self.poetry.package.pretty_version, version)

    self.line(
        "Bumping version from <comment>{}</> to <info>{}</>".format(
            self.poetry.package.pretty_version, version
        )
    )

    content = self.poetry.file.read()
    poetry_content = content["tool"]["poetry"]
    poetry_content["version"] = version.text

    self.poetry.file.write(content)

whereas
> poetry version minor
works exactly as expected and uses exactly the same code, my script does not update the version number in the pyproject.toml file. In the debugger I see that poetry_content above is correctly updated, but content remains unchanged. Actually content["tool"]["poetry"] and poetry_content refer to different python objects. How comes? Any help is welcome

this code line
content["tool"]["poetry"]["version"] = version.text

does also NOT modify content..

Can you share more of your code? I'm unsure of where ArgvInput comes from so I couldn't run your example.
If you just need to bump the version via a python script, this worked for me
import subprocess
subprocess.run(['poetry','version', 'minor'])

Hi,
Thanks for considering this. Your solution obviously works but It looks a bit overkill to run  python code via subprocess, so I thought to use the api of poetry.
It must be sth subtle inside tomlkit. I copied the lines from VersionCommand.handle
    version = self.argument("version")

    version = self.increment_version(self.poetry.package.pretty_version, version)

    self.line(
        "Bumping version from <comment>{}</> to <info>{}</>".format(
            self.poetry.package.pretty_version, version
        )
    )

    content = self.poetry.file.read()
    poetry_content = content["tool"]["poetry"]
    poetry_content["version"] = version.text

    self.poetry.file.write(content)

The only thing I adapted  is the read and write instructions, to make sure they read/write the appropriate file. When I debug the poetry code as in your version. The line
poetry_content["version"] = version.text
does modify the content variable, as expected. However, in my version the poetry_content variable seems to be a copy, and consequently the content varable is not updated by that line. Debuging did not help to find out why I am getting a copy …
By the way, I managed to update the version in this way by using the toml package rather then using tomlkit, but then the formatting of the pyproject.toml gets lost.
I ll try to set up a small code that reproduces the problem and ship it to you - maybe tomorrow
Kindest regards,
Dr [Engel]bert Tijskens - HPC Analyst/Consultant
Flemish Supercomputer Center
University of Antwerp / Computational mathematics
++32 3 265 3879, ++32 494 664408
On 26 Jun 2019, at 07:45, brno32 <notifications<EMAIL_ADDRESS>wrote:
Can you share more of your code? I'm unsure of where ArgvInput comes from so I couldn't run your example.
If you just need to bump the version via a python script, this worked for me
import subprocess
subprocess.run(['poetry','version', 'minor'])
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/sdispater/poetry/issues/1182?email_source=notifications&email_token=ACRDDFSRKOSGLOQMQRIVW3DP4L66BA5CNFSM4H2P4UXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYSMXCI#issuecomment-505727881, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACRDDFUJGLIUJN4RDWVX5D3P4L66BANCNFSM4H2P4UXA.

Hi,
here is a reproducible test-code.
The script poetry_version.py is in principle equivalent to “poetry version patch”. It is using poetry 0.12.16 installed with pip install poetry. The command line command “poetry version patch” is also using poetry 0.12.16, installed in the recommended way on the poetry website. (Curl …)
T
he start version of a-test-project is “0.1.0”

cd issue1182/a-test-project/
/Users/etijskens/software/dev/workspace/micc/issue1182/a-test-project

(testet) etijskens@etijskens@local [544] ~/software/dev/workspace/micc/issue1182/a-test-project

python ../poetry_version.py
start
Bumping version from 0.1.0 to 0.1.1 # the message is correct

(testet) etijskens@etijskens@local [547] ~/software/dev/workspace/micc/issue1182/a-test-project

cat pyproject.toml
[tool.poetry]
name = "a-test-project"
version = “0.1.0” # but the file is not updated (because the content variable in poetry/console/commands/version.py VersionCommand.handle() is not updated on executing the line “poetry_content["version"] = version.text")
description = "test_cli_with_project_name()"
authors = ["Engelbert Tijskens <engelbert.tijskens<EMAIL_ADDRESS>license = "MIT"

readme = 'README.rst'
repository = "https://github.com/etijskens/a-test-project"
homepage = "https://github.com/etijskens/a-test-project"
keywords = ['packaging', 'poetry']
[tool.poetry.dependencies]
python = "^3.7"
click = "^7.0"
cookiecutter = "^1.6.0"
[tool.poetry.dev-dependencies]
pytest = "^4.4.2"
[tool.poetry.scripts]
#my-script = 'my_package:main'
[build-system]
requires = ["poetry>=0.12"]
build-backend = "poetry.masonry.api"
(testet) etijskens@etijskens@local [548] ~/software/dev/workspace/micc/issue1182/a-test-project

poetry version patch
Bumping version from 0.1.0 to 0.1.1 # again the message is correct

(testet) etijskens@etijskens@local [549] ~/software/dev/workspace/micc/issue1182/a-test-project

cat pyproject.toml
[tool.poetry]
name = "a-test-project"
version = “0.1.1” # this time the file is updated.
description = "test_cli_with_project_name()"
authors = ["Engelbert Tijskens <engelbert.tijskens<EMAIL_ADDRESS>license = "MIT"

readme = 'README.rst'
repository = "https://github.com/etijskens/a-test-project"
homepage = "https://github.com/etijskens/a-test-project"
keywords = ['packaging', 'poetry']
[tool.poetry.dependencies]
python = "^3.7"
click = "^7.0"
cookiecutter = "^1.6.0"
[tool.poetry.dev-dependencies]
pytest = "^4.4.2"
[tool.poetry.scripts]
#my-script = 'my_package:main'
[build-system]
requires = ["poetry>=0.12"]
build-backend = "poetry.masonry.api"
(testet) etijskens@etijskens@local [550] ~/software/dev/workspace/micc/issue1182/a-test-project

Kindest regards,
Dr [Engel]bert Tijskens - HPC Analyst/Consultant
Flemish Supercomputer Center
University of Antwerp / Computational mathematics
++32 3 265 3879, ++32 494 664408
On 26 Jun 2019, at 07:45, brno32 <notifications<EMAIL_ADDRESS>wrote:
Can you share more of your code? I'm unsure of where ArgvInput comes from so I couldn't run your example.
If you just need to bump the version via a python script, this worked for me
import subprocess
subprocess.run(['poetry','version', 'minor'])
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/sdispater/poetry/issues/1182?email_source=notifications&email_token=ACRDDFSRKOSGLOQMQRIVW3DP4L66BA5CNFSM4H2P4UXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYSMXCI#issuecomment-505727881, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACRDDFUJGLIUJN4RDWVX5D3P4L66BANCNFSM4H2P4UXA.

In the mean time I tested your subprocess.run approach - successfully.
Kindest regards,
Dr [Engel]bert Tijskens - HPC Analyst/Consultant
Flemish Supercomputer Center
University of Antwerp / Computational mathematics
++32 3 265 3879, ++32 494 664408
On 26 Jun 2019, at 07:45, brno32 <notifications<EMAIL_ADDRESS>wrote:
Can you share more of your code? I'm unsure of where ArgvInput comes from so I couldn't run your example.
If you just need to bump the version via a python script, this worked for me
import subprocess
subprocess.run(['poetry','version', 'minor'])
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/sdispater/poetry/issues/1182?email_source=notifications&email_token=ACRDDFSRKOSGLOQMQRIVW3DP4L66BA5CNFSM4H2P4UXKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODYSMXCI#issuecomment-505727881, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ACRDDFUJGLIUJN4RDWVX5D3P4L66BANCNFSM4H2P4UXA.

Dear Mr. Tijskens,
would it be possible to use code fences for better readability?
You can activate syntax highlighting for Python this way :-)

the issue may be closed. It is probably caused by a resource that is not freed appropriately, due to the way the tests are set up. (though I haven't pinned it completely, yet).

I finally tracked down the source of this error. I create my projects with  Cookiecutter. The template file for pyprojects contains a [tool.poetry.scripts] section AFTER the [build-system] section. when running E.g. poetry --version patch the version stringiest is NOT updated, but pyproject.toml is rewritten and now the [tool.poetry.scripts] section appears BEFORE the [build-system] section. Running  poetry --version patch a second time then updates the version string correctly. If I had gotten a warning that the pyproject.toml file is not in order, because, apparently, all [poetry.tool.] sections must appear consecutively, I would not have lost three frustrating days tracking this problem.
I suggest

that the documentation makes reference that sections should appear grouped with a reference to the  syntax of pyproject.toml files
that tomlkit would warn on such issues, or circumvents it. An easy way around the problem is to have tomlkit to read the file and write it back to correct the order, and then read it back in...

I hope this helps other people experiencing this problem.

@etijskens I ran into a very similar bug when using poetry.lock which seems to also required the build-system to be the last section in the pyproject.toml


Rename ProbabilitySampler to TraceIdRatioBasedSampler and implement ratio sampling

The probability sampler should be deterministic depending on the Trace ID.
https://github.com/open-telemetry/opentelemetry-specification/blob/master/specification/trace/sdk.md#requirements-for-traceidratiobased-sampler-algorithm

Assigning this to @legendecas because he is already working on the parent sampled flag in the probability sampler and this issue is related.

do we need to  rename OTEL_SAMPLING_PROBABILITY env var to OTEL_TRACE_ID_RATIO?

There is no spec for it, so I guess we can do what we want: https://github.com/open-telemetry/opentelemetry-specification/blob/master/specification/sdk-environment-variables.md
I think probability is more clear to end users not familiar with our internals, and I think it is still accurate. Probability does not imply randomness.

@legendecas hey jsut wanted to ask what is the current status of this ticket, thx

@obecny I filed a PR #1562, please take a look at your convenience :)


Remove gitrev usage

Removing gitrev usage so having the .git is not mandatory
/cc @xavijam @javisantana

Looks :+1:.

:es:

Change package.json version and try in staging please.


Stimulus Reflex doesn't render ViewComponent with params

Steps to reproduce
In my reflex I want to re-render a view component with the morph directive, like so:
 def order_success(user_id)
    morph "#weekly-meal-list, ApplicationController.render(MealComponent.new(user_id: user_id.to_i))
  end

Then my view component is initialized with the user record like this:
  def initialize(user_id:)
    @user = User.find(user_id)
  end

I tried to pass the user instance directly aswell, but still same issue.
Now, in my view (cause of wicked pdf) I have to render the component with the render_component directive instead of just render.
<tbody id="weekly-meal-list">
      <%= render_component(MealComponent.new(user_id: user.id)) %>
</tbody>

Now, when I want to re-render just this table with the reflex above, I get the following error:
StimulusReflex::Channel Failed to invoke Order#order_success! http://localhost:3000/users/1/order_cards/new '#<MealComponent:0x000055fe47a95e98 @user=#<User id: 1, name: "Admin User", email<EMAIL_ADDRESS>password_digest: "$2a$10$sx3fPbyQoPfIP4j/iIiGeOYbNw2laFIE.iVwzCxP7ts...", twoFactorEnabled: "enabled", canDoGuestOrder: true, commentedToday: true, hasCommentedThisWeek: true, otp_secret_key: "ZZOJYZGT6ENI2HADWXPELKVAGREEW2VS", created_at: "2020-07-18 11:36:03", updated_at: "2020-07-24 17:18:03", reset_digest: nil, reset_sent_at: nil, workplace_id: 2, has_ldap_connection: nil, salt: "zhxBMJfAmPVW4MiXvCu3", remember_me_token: "T1fgN4zhqdW17juCH6K4", remember_me_token_expires_at: "2020-08-06 07:51:43", user_accountable_type: "ExternalClinicAccount", user_accountable_id: 1>>' is not an ActiveModel-compatible object. It must implement :to_partial_path. /home/mkrs/.rbenv/versions/2.6.5/lib/ruby/gems/2.6.0/gems/actionview-<IP_ADDRESS>/lib/action_view/renderer/partial_renderer.rb:493:in partial_path'`
But as I checked inside the initialize method of the component, it IS implementing :to_partial_path:
(byebug) user.respond_to?(:to_partial_path)
true

Expected behavior
Just the view component should get rendered again, with the new updated list.
Actual behavior
The view component doesn't get rendered, instead I get the error.
System configuration
Rails version: <IP_ADDRESS>
Ruby version: 2.6.5
Gem version: 2.18.0

@alliedarmour thank you for taking the time to file this issue. Would you be willing to write a PR with a failing test for this problem?

@alliedarmour did you figure this out, having a similar issue. However, I think this may be more of an issue with SR?

I believe view_component_reflexwas written to address this.

@scottbarrow sorry for the late response, yeah I figured out what caused the problem. I mentioned it in the Stimulus Reflex repo over here.
@andrewmcodes This looks awesome, is it already included in the view_component gem?


chore: Remove docker executor

Fixes #7795.
Signed-off-by: Yuan Tang<EMAIL_ADDRESS>

@terrytangyuan There are more references to ContainerRuntimeExecutorDocker in multiple files. Specifically, workflow/controller/workflowpod.go has a lot of code related to the docker socket that can be removed

Search for docker.sock?

@terrytangyuan https://github.com/terrytangyuan/argo-workflows/blob/rm-docker/workflow/controller/workflowpod.go#L587

Done. Let me know if I missed anything else.

There are some quickstarts for docker executor in manifests/quick-start/base/executor.


Copy custom xml and SharePoint metadata on merging document

Before submitting an issue, please fill this out
Description
Copy custom XML and SharePoint metadata on merging document does not work using DocumentBuilder?
Information

.NET Target: NET Framework 4.5.2
DocumentFormat.OpenXml Version: OpenXMLSDK-MOT <IP_ADDRESS>

Expected
When I comment out if (itemList.Contains(propertyPartDoc.Root.Attribute(DS.itemID).Value)) I get the expected result.
        private static void CopyCustomXml(WordprocessingDocument sourceDocument, WordprocessingDocument newDocument,
            IEnumerable<XElement> newContent)
        {
            List<string> itemList = new List<string>();
            foreach (string itemId in newContent
                .Descendants(W.dataBinding)
                .Select(e => (string)e.Attribute(W.storeItemID)))
                if (!itemList.Contains(itemId))
                    itemList.Add(itemId);

            foreach (CustomXmlPart customXmlPart in sourceDocument.MainDocumentPart.CustomXmlParts)
            {
                OpenXmlPart propertyPart = customXmlPart
                    .Parts
                    .Select(p => p.OpenXmlPart)
                    .Where(p => p.ContentType == "application/vnd.openxmlformats-officedocument.customXmlProperties+xml")
                    .FirstOrDefault();

                if (propertyPart != null)
                {
                    XDocument propertyPartDoc = propertyPart.GetXDocument();
                    //if (itemList.Contains(propertyPartDoc.Root.Attribute(DS.itemID).Value))
                    //{
                        CustomXmlPart newPart = newDocument.MainDocumentPart.AddCustomXmlPart(customXmlPart.ContentType);
                        newPart.GetXDocument().Add(customXmlPart.GetXDocument().Root);
                        foreach (OpenXmlPart propPart in customXmlPart.Parts.Select(p => p.OpenXmlPart))
                        {
                            CustomXmlPropertiesPart newPropPart = newPart.AddNewPart<CustomXmlPropertiesPart>();
                            newPropPart.GetXDocument().Add(propPart.GetXDocument().Root);
                        }
                    //}
                }
            }
        }

This is not a working repro. What data are you using? For us to be able to investigate any possible issue, we need to have a working repro so we can debug it.

btw DocumentBuilder is not part of the SDK. If this is an issue with that (you mention it in the description, but I see no use of it in the code), then you should file it here: https://github.com/OfficeDev/Open-Xml-PowerTools/issues

Even if the DocumentBuilder is not used in the repro itself, the usage of the GetXDocument() extension method and the XDocument class indicate that @XzaR90 is trying to use the Open XML PowerTools.
Thus, this is not an issue with the SDK.

Good catch @ThomasBarnekow . @XzaR90 I'd recommend taking a look at the Open-XML-PowerTools repo and filing and issue there.


Bots should test upgrade path

Right now the bots seem to only test the clean install path.
We should make sure that the following two paths result in identical on-disk bits:

checkout the current master, build flutter gallery.
checkout the alpha branch, build flutter gallery, then checkout the master branch on top of it, and build flutter gallery.

I'm not sure this is a very useful bug to track.  We don't yet have releases.


Support import from URL without CDN

On Discord, liquid reported that imports from URLs don't work: the Playground tries to prepend the ESM CDN even when an explicit URL is given.  This small PR stops that behavior, using the given URL if it's a URL (contains ://). Example:

It doesn't play well with TypeScript, but that's a regular TypeScript issue. Relevant StackOverflow post. We can silence the error but I imagine it's tricky to get the type definitions actually used. Still, better than not working at all.

Thanks!


Module-level summaries don't appear for all member kinds

Search terms
@summary, useFirstParagraphOfCommentAsSummary
Expected Behavior
As described in the @summary docs, when a function, type alias, or other module member has a @summary, or if useFirstParagraphOfCommentAsSummary is enabled, it should have a summary on the module page exporting that function.
Actual Behavior
Summaries appear for classes and interfaces but not standard function declarations, even if an explicit @summary is given.  (They do appear for functions defined with const fn = () => {} however.)
Steps to reproduce the bug
Entry point:
/**
 * Text A
 * @summary Text B
 */
export function test() {
}

Module page lists test function but neither Text A nor Text B appear below, whether  useFirstParagraphOfCommentAsSummary is used or not.
Environment

Typedoc version: 0.27.3
TypeScript version: 5.2.2
Node.js version: 18.20.2
OS: Windows 10

It's incredible how even after living with the special case nightmare that is functions for as long as I have, I still manage to consistently forget to handle it...


NIFI-8942 NiFi Registry: flow description cannot be selected & copied on the UI

Force the description field to have "user-select: text" style.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[x] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[x] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[x] Has your PR been rebased against the latest commit within the target branch (typically main)?

[x] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[x] Have you verified that the full build is successful on JDK 8?
[ ] Have you verified that the full build is successful on JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check GitHub Actions CI for build issues and submit an update to your PR as soon as possible.

Reviewing...

+1 Works well! Verified I can select the description. Thanks!

Merged, thanks @adenes @mtien-apache


Creating links in grid cells

Hi - how would we go about creating a href for the content displayed in the row? e.g so that when a user clicks on the data in a row, we can display another partial/view. Thanks!

Here's an example on how to add html content using html helpers. You can also add them by hand, the only important thing is to use .Encoded(false), because all grid data is encoded by default.

Thank you - we got it working as well!


ThemeKit downloads wrong theme when bootstrapping

Followed the instructions to bootstrap a new theme, which correctly created a new instance of Timber (2.2.2) in the shop, however when it fetched assets, it appears to have downloaded the Debut theme files (as evidenced by the sections folder among other things).

This is interesting https://github.com/Shopify/themekit/blob/master/cmd/bootstrap.go#L20 is the settings for the paths to Timber so if it is downloading Debut it is happening upstream. I will have to get back to you on this.

@tekhaus I have gotten the maintainers of Timber and Debut involved in this and I will get back to you when they find anything.

I also tried setting up a new theme with bootstrap. Themekit successfully created a new theme in the store but downloaded the store's published theme.
(So I assume @tekhaus his published theme was Debut)

hey @driespieters thanks! that really sheds some light on what is happening! This is a bug I can fix.

@driespieters @tanema yep, my store was using Debut as it's published theme (as I assume all brand new Dev stores do currently)


steering wheel simulator and the throttle simulator

I want to control the vehicle through the steering wheel simulator and the throttle simulator directly connected to the software. Is this currently possible?

Yes, check the examples in steering_control.py


go-axiom link points to axiom-cli

On this page https://docs.axiom.co/reference/api , the link goes to cli instead of axiom-go
Also,

Go-axiom should be axiom-go

The best way to get started with our API is to use Go-Axiom Send events directly to Axiom API.

Should be
Axiom REST API documentation is coming soon but, in the mean time, the best way to get started building applications on top of Axiom is to use the axiom-go library.

done


Fix type mismatch for IndexRecommendationListResult and IndexRecommendationListResult found in 2024-11-01-preview

…mmendationListResult
ARM (Control Plane) API Specification Update Pull Request

[!TIP]
Overwhelmed by all this guidance? See the Getting help section at the bottom of this PR description.

PR review workflow diagram
Please understand this diagram before proceeding. It explains how to get your PR approved & merged.

Purpose of this PR
What's the purpose of this PR? Check the specific option that applies. This is mandatory!

[ ] Update existing version for a new feature. (This is applicable only when you are revising a private preview API version.)

Due diligence checklist
To merge this PR, you must go through the following checklist and confirm you understood
and followed the instructions by checking all the boxes:

[ ] I confirm this PR is modifying Azure Resource Manager (ARM) related specifications, and not data plane related specifications.
[ ] I have reviewed following Resource Provider guidelines, including
ARM resource provider contract and
REST guidelines (estimated time: 4 hours).
I understand this is required before I can proceed to the diagram Step 2, "ARM API changes review", for this PR.

Additional information

 Viewing API changes
Fix bug discovered in Pyhton SDK due to type mismatch
https://github.com/Azure/azure-sdk-for-python/issues/38477#issuecomment-2472554931

Suppressing failures
If one or multiple validation error/warning suppression(s) is detected in your PR, please follow the
suppressions guide to get approval.

Getting help

First, please carefully read through this PR description, from top to bottom. Please fill out the Purpose of this PR and Due diligence checklist.
If you don't have permissions to remove or add labels to the PR, request write access per aka.ms/azsdk/access#request-access-to-rest-api-or-sdk-repositories
To understand what you must do next to merge this PR, see the Next Steps to Merge comment. It will appear within few minutes of submitting this PR and will continue to be up-to-date with current PR state.
For guidance on fixing this PR CI check failures, see the hyperlinks provided in given failure
and https://aka.ms/ci-fix.
For help with ARM review (PR workflow diagram Step 2), see https://aka.ms/azsdk/pr-arm-review.
If the PR CI checks appear to be stuck in queued state, please add a comment with contents /azp run.
This should result in a new comment denoting a PR validation pipeline has started and the checks should be updated after few minutes.
If the help provided by the previous points is not enough, post to https://aka.ms/azsdk/support/specreview-channel and link to this PR.

/azp run

/azp run

Fix the errors from Swagger LintDiff check

/azp run

@nasc17 : Could you please get breaking changes approved first? Please add WaitForARMFeedback back when breaking changes are approved.


Remove trailing whitespace before end of line

This PR removes all trailing whitespace (tabs, in this case) before end of line.

Thanks!


Raw data attachment was sent as email body when body is blank

Laravel Version: 5.3
PHP Version: 5.6
Database Driver & Version: MariaDB 5.5

Description:
Raw data attachment was sent as email body when body is blank.
Steps To Reproduce:
Mail::send([], [], function($message) use($data) {
    $message->attachData('hello', 'request.txt', ['mime' => 'text/plain']);
    $message->to('myemail@gmail.com');
    $message->subject('Testing');
    $message->setBody('', 'text/html');
});

Messsage I received:
--_=_swift_v4_1498809084_f9f505b29d1834688f3a38b120dbf08a_=_ Content-Type: text/plain; name=request.txt Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename=request.txt aGVsbG8= --_=_swift_v4_1498809084_f9f505b29d1834688f3a38b120dbf08a_=_--

Sorry but 5.3 is not supported anymore, please test on 5.4 and report back if the issue still present.

@Corys8646 please create new issue.

@hthuong09 I got the same issue. Do you have a reference to an issue that might have an answer unlike this one.

Having the same issue on latest Laravel (5.7.12). Was working in previous versions fine.

Having the issue with Laravel 5.7

Mail::send([], [], function($message){ $message->attachData('hello', 'request.txt', ['mime' => 'text/plain']); $message->to('myemail@gmail.com'); $message->subject('Testing'); $message->setBody('', 'text/html'); });

It doesn't seem to support the setBody method, I solved this problem by forcing the use of the view mail template.It looks like the following:
// emails.example  => resource/views/emails/example.blade.php
Mail::send(['html' => 'emails.example'], [], function($message){
    message->from('fromMail'); // It's a must
    $message->to('myemail@gmail.com');
    $message->subject('Testing');
});

hope this helps...

I just ran into this on 5.6. Had a mailable that was working great in 5.5, after upgrading to 5.6 when sending an attachment to an email with a blank body, it was putting the attachment in the body. Using the same code as above.
Laravel version: v5.6.17
Mail::send([], [], function($message){
    $message->attachData('hello', 'request.txt', ['mime' => 'text/plain']);
    $message->to('myemail@gmail.com');
    $message->subject('Testing');
    $message->setBody('', 'text/html');
});

results in this in the body
--_=_swift_1524068407_2401445d42a99f78596c7130ace3de39_=_ Content-Type: text/plain; name=request.txt Content-Transfer-Encoding: base64 Content-Disposition: attachment; filename=request.txt aGVsbG8= --_=_swift_1524068407_2401445d42a99f78596c7130ace3de39_=_--

putting a string in the body, fixes the problem.

Solved my issue by adding text in body. Thanks a lot.


A11Y_Oracle Data Migration extension for Azure Data Studio_data migration for oracle_data migration status_AI4D: "Data migration status" heading is not defined.

GitHub Tags:
#A11yWCAG2.1; #A11yTCS; #A11ySev2; #DesktopApp; #Benchmark; #BM-AzureDataStudio-Win32-Feb2022; #Azure Data Studio; #WCAG1.3.1; #Info and Relationships; #STP; #FTP; #Win11; #Win32; #AI4D;
Environment Details:
Application Name: Oracle Data Migration extension for Azure Data Studio
Download application: Azure Data Studio.
Version:1.45.1(system setup)
Operating System: Window 11 version 22H2 OS build 25267.1000.
Repro Steps:

Open Azure Data Studio application.
Install "Data Migration for Oracle"
connect to source connection.
Tab till new Migration and invoke on it.
After completed 6steps and invoke the "Start Migration" in step-6.
Tab till "Data Migration Status" card.
Now verify that "Data migration status" heading is not defined.

Actual Result:
"Data migration status" heading is not defined.
Expected Result:
"Data migration status" heading is not defined.
User Impact:
Screen Reader users will not be able to know "Data migration status" heading. If it is not defined.
Attachment

primary bug id: https://msdata.visualstudio.com/Migration Assistant/_workitems/edit/2663407

@kisantia Could you please provide the ETA of fix. When the fixes will be available in the environment.

@SrivenNookala @Bhavikshah123 please help provide an ETA. Thanks!

I've tried to set the heading property on the text component from the extension side. Inspecting the text using dev tools shows it as a heading text packaged under a few divs but according to the testing team, it does not reflect on accessibility insights as a heading. @kisantia Can you suggest another way to make it readable as a heading in accessibility insights?

@SrivenNookala ping me on Teams when you are available.

#ETA:03-29-24

@SrivenNookala I've replied to you via Email. basically, you also need to set the ariaRole to heading on the text component.

Verified the issue in below environment.
Application Name: Oracle Data Migration extension for Azure Data Studio
Download application: Azure Data Studio.
Version: 1.48.0 (user setup)
Commit:<PHONE_NUMBER>ef8254b7c22a5dc55af7f8a1dea93f
Date: 2024-02-27T00:05:08.293Z
VS Code: 1.82.0
Electron: 25.9.8
Chromium: 114.0.5735.289
Node.js: 18.15.0
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.22631
Operating System: Window 11 version 23H2
Issue is fixed. Hence the bug is closed.
#Closed; #Regressed:04-17-24;
For reference, attachment is added below.


[Sema] Check for already existential type while resolving existential

In those cases constraint type will be already resolved as ExistentialType which seems to make sense to just return it already.

Fixes https://github.com/apple/swift/issues/58920

@swift-ci Please smoke test


wrong time zone offsets for St. Johns, Newfoundland and Marquesas

Wrong offsets in 2018.9:
Canada/Newfoundland          -05:10           must be: -03:30
America/St_Johns             -05:10           must be: -03:30
Pacific/Marquesas            -11:10           must be: -09:30

These errors may be due to typos:
-05:10 = -310 minutes  while  -03:30 = -210 minutes
-11:10 = -670 minutes while  -09:30 = -570 minutes

As far as I know the offsets are correct. Here are examples using pendulum (which uses pytzdata):
>>> import pendulum
>>> pendulum.now("Canada/Newfoundland").isoformat()
'2019-02-28T17:22:09.576059-03:30'
>>> pendulum.now("Pacific/Marquesas").isoformat()
'2019-02-28T11:22:38.307084-09:30'

Where did you get these offsets from?

Withdrawn with my apologies! The data is correct. The bug was in my internal code.


fix create-cluster-kubeadm docs

NOTE: Please check the “Allow edits from maintainers” box (see image below) to
allow reviewers to fix problems on your patch and speed up the review process.
Please delete this note before submitting the pull request.

This change is

/assign @chenopis


Port recursive tree, mandelbrot example

Based on Daniel Shiffman's Recursive Tree example for Processing and Mandelbrot Set
Fixes #407

@ihsavru please add this example to the "Simulate" section, where we have other L-System examples, rather than making a new section.
I think you can remove the .idea/ entry as this is tool specific, not tied to the p5.js repository setup itself.

@lmccart

I think you can remove the .idea/ entry as this is tool specific, not tied to the p5.js repository setup itself.

Seeing *.sublime-* and .DS_Store which would be specific to Sublime Text and MacOS users respectively made sense for me to add .idea/ underneath. However, I will remove it if you think it is unnecessary.

@ihsavru ah, I guess that's fair. :) we can leave .idea/ then.
yes please move both examples to simulate. thank you!


Nuke: Remote existing frames

Changelog Description
This adds initial support to submit existing frames to the farm in Nuke to run the publish plugins.
Additional info
I did these changes together with the ones on https://github.com/ynput/OpenPype/pull/5195 so there might be some things I touched there that were required here and viceversa.
Testing notes:

Create a write render instance and render locally some frames
Go to the publisher and choose "Existing frames farm rendering"
Hit publish

Moving the development to this GitHub pull request was easier because the code was based on the 3.15 codebase.
Thanks again @fabiaserra for your contribution!


ExternalTrafficPolicy=Local works on one node, but not on the other

Hi,
I'm having this very weird issue where ExternalTrafficPolicy=Local works on one node, but not on the other. Both Kubernetes workers were deployed using the same set of Ansible scripts, so their configuration is identical.
This is the service manifest I'm using, which points to a single nginx pod instance:
piVersion: v1
kind: Service
metadata:
  labels:
    app: nginx
  name: nginx-ext
  namespace: default
spec:
  externalTrafficPolicy: Local
  ports:
  - port: 80
    protocol: TCP
    targetPort: 80
  selector:
    app: nginx
  type: NodePort

I have two worker nodes

nuc-1 (here the traffic policy works)
worker-01 (on this one, it doesn't work)

According to the service object, the nodeport is 32204:
kubectl get service
NAME         TYPE        CLUSTER-IP     EXTERNAL-IP   PORT(S)        AGE
kubernetes   ClusterIP   <IP_ADDRESS>      <none>        443/TCP        123d
nginx-ext    NodePort    <IP_ADDRESS>   <none>        80:32204/TCP   2d

when I curl nuc-1 on port 32204, this works fine (when pod is scheduled on nuc-1). However, when I curl worker-01, I always get a connection time-out (when pod is scheduled on worker-01).
I corrected the port-number where necessary, so please eliminate the option I'm curling the wrong port.
Nothing in the weave logs (weave or weave-npc) indicate an error or a blocked connection.
NO_MASQ_LOCAL=1 was set in the daemonset as an environment variable.
kube-proxy is indeed listening on that port:
sudo ss -tulnp | grep hyperkube
tcp    LISTEN     0      65535    :::32204                :::*                   users:(("hyperkube",pid=8056,fd=17))

When I change the policy to Cluster it works on both nodes.
Versions:

$ weave version: 2.4.1
$ docker version: 18.09.0
$ uname -a: Linux worker-01.internal.headincloud.be 4.19.4-1.el7.elrepo.x86_64 #1 SMP Fri Nov 23 08:15:01 EST 2018 x86_64 x86_64 x86_64 GNU/Linux
$ kubectl version: v1.11.5

Logs:
$ docker logs weave

or, if using Kubernetes:
$ kubectl logs -n kube-system <weave-net-pod> weave
(see gist)

gist can be found here: https://gist.github.com/jeroenjacobs1205/caef6e799083479c7ca3e560c77daf26

Network:

$ ip route
default via <IP_ADDRESS> dev ens192 proto static metric 100
<IP_ADDRESS>/12 dev weave proto kernel scope link src <IP_ADDRESS>
<IP_ADDRESS>/16 dev docker0 proto kernel scope link src <IP_ADDRESS> linkdown
<IP_ADDRESS>/24 dev ens192 proto kernel scope link src <IP_ADDRESS> metric 100

$ ip -4 -o addr
1: lo    inet <IP_ADDRESS>/8 scope host lo\       valid_lft forever preferred_lft forever
2: ens192    inet <IP_ADDRESS>/24 brd <IP_ADDRESS> scope global noprefixroute ens192\       valid_lft forever preferred_lft forever
3: docker0    inet <IP_ADDRESS>/16 brd <IP_ADDRESS> scope global docker0\       valid_lft forever preferred_lft forever
6: weave    inet <IP_ADDRESS>/12 brd <IP_ADDRESS> scope global weave\       valid_lft forever preferred_lft forever

$ sudo iptables-save
(see gist)

iptables-save gist can be found here: https://gist.github.com/jeroenjacobs1205/eb9833da70419c8b46adf708d7d9d89c

@jeroenjacobs1205 From where you are trying to access the NodePoprt's? Outside the cluster?
Could you please share iptables-save in case when pod is running on worker-01 node.

Hi, the iptables-save gist I linked to, was worker-01 (= the non-working one).

I looked at the logs of kube-proxy on worker-01, but I don't see any errors:
W1215 08:45:43.768998       1 server.go:195] WARNING: all flags other than --config, --write-config-to, and --cleanup are deprecated. Please begin using a config file ASAP.
W1215 08:45:43.810017       1 proxier.go:472] Failed to load kernel module nf_conntrack_ipv4 with modprobe. You can ignore this message when kube-proxy is running inside container without mounting /lib/modules
W1215 08:45:43.815700       1 server_others.go:287] Flag proxy-mode="" unknown, assuming iptables proxy
I1215 08:45:43.816633       1 server_others.go:140] Using iptables Proxier.
I1215 08:45:43.834294       1 server_others.go:174] Tearing down inactive rules.
W1215 08:45:43.849273       1 proxier.go:472] Failed to load kernel module nf_conntrack_ipv4 with modprobe. You can ignore this message when kube-proxy is running inside container without mounting /lib/modules
I1215 08:45:43.849809       1 server.go:448] Version: v1.11.5
I1215 08:45:43.854997       1 conntrack.go:98] Set sysctl 'net/netfilter/nf_conntrack_max' to 131072
I1215 08:45:43.855093       1 conntrack.go:52] Setting nf_conntrack_max to 131072
I1215 08:45:43.855160       1 conntrack.go:98] Set sysctl 'net/netfilter/nf_conntrack_tcp_timeout_established' to 86400
I1215 08:45:43.855187       1 conntrack.go:98] Set sysctl 'net/netfilter/nf_conntrack_tcp_timeout_close_wait' to 3600
I1215 08:45:43.856222       1 config.go:202] Starting service config controller
I1215 08:45:43.856248       1 controller_utils.go:1025] Waiting for caches to sync for service config controller
I1215 08:45:43.856250       1 config.go:102] Starting endpoints config controller
I1215 08:45:43.856257       1 controller_utils.go:1025] Waiting for caches to sync for endpoints config controller
I1215 08:45:43.957127       1 controller_utils.go:1032] Caches are synced for service config controller
I1215 08:45:43.958097       1 controller_utils.go:1032] Caches are synced for endpoints config controller

Closing the issue. The root-cause has been found: --hostname-override was set incorrectly in the kube-proxy manifest. It was set to "nuc-1" instead of "worker-01" on the worker-01 node. So, I assume ip-proxy was generating the wrong iptables rules.


Missing event information when reading EEGLAB file and exporting raw to .set files

Describe the bug
When reading an EEGLAB file with mne.io.read_raw_eeglab(), the events information seems oversimplified.
In EEGLAB,  the set file event structure contains the following:
>> EEG_set.event

ans =

  1×2670 struct array with fields:

    description
    begintime
    classid
    code
    duration
    label
    relativebegintime
    sourcedevice
    name
    tracktype
    latency
    type
    mffkeys
    mffkey_gidx
    mffkeysbackup
    mffkey_age_
    mffkey_exp_
    mffkey_hand
    mffkey_sex_
    mffkey_subj
    mffkey_cel
    mffkey_obs
    mffkey_spc
    mffkey_pos
    mffkey_argu
    mffkey_rsp
    mffkey_eval
    mffkey_rtim
    mffkey_trl
    mffkey_blk1

However, after reading the SET file with mne.io.read_raw_eeglab(), I can only see annotations with onset, duration, description and orig_time information:
>>> raw.annotations
<Annotations | 2670 segments: CELL (4), DIN2 (666), SESS (1), TRSP (666), ...>
>>> raw.annotations[0]
OrderedDict([('onset', 0.255321), ('duration', 2.0), ('description', 'SESS'), ('orig_time', None)])
>>>
>>> raw.annotations[1]
OrderedDict([('onset', 0.46732), ('duration', 2.0), ('description', 'CELL'), ('orig_time', None)])

And when I export the raw structure into a new SET file, the event structure of the file is as follows:
>> EEG_new_set.event

ans =

  1×2670 struct array with fields:

    type
    latency
    duration

Is there a way to keep the same events structure as the original SET file?
Steps to reproduce
raw = mne.io.read_raw_eeglab(input_fname=<FILE PATH>, preload=False, verbose=True)
mne.export.export_raw(
    fname=<NEW FILE PATH>,
    raw=raw,
    fmt='eeglab',
    overwrite=True
)

Additional information
>>> mne.sys_info()
Platform:         macOS-11.3-x86_64-i386-64bit
Python:           3.9.12 (main, Mar 26 2022, 15:52:10)  [Clang 13.0.0 (clang-13<IP_ADDRESS>)]
Executable:       /Users/cmadjar/Development/python_venv/python_intel_3.9.4/eeg2bids_py3.9.12/bin/python
CPU:              i386: 8 cores
Memory:           Unavailable (requires "psutil" package)
mne:              1.0.0
numpy:            1.21.5 {blas=openblas, lapack=openblas}
scipy:            1.8.0
matplotlib:       3.5.1 {backend=MacOSX}

sklearn:          1.0.2
numba:            0.55.1
nibabel:          Not found
nilearn:          Not found
dipy:             Not found
cupy:             Not found
pandas:           1.4.1
pyvista:          Not found
pyvistaqt:        Not found
ipyvtklink:       Not found
vtk:              Not found
PyQt5:            Not found
ipympl:           Not found
pooch:            v1.6.0

mne_bids:         0.10
mne_nirs:         Not found
mne_features:     0.2
mne_qt_browser:   Not found
mne_connectivity: Not found

MNE cannot read arbitrary fields in the event structure. If this information can be store in an annotations object let's fix it but otherwise we need to see. What do these fields contain?

Those fields contain details about the conditions of interest. For example, for a face processing task, we would like to code the following attributes: upright vs inverted, gaze towards vs away, race of the face, trial number, block number, etc. Is there a way to add that information to the event structure?
From reading the documentation, it seems like it would be possible: "An arbitrary number of additional columns can be added. Those allow describing other properties of events that could be later referred in modeling and hypothesis extensions of BIDS."
Thanks!


Enable ESXi host connect/disconnect to/from storage controller over NVMe/TCP

Enabling vSphere ESXi host connect and disconnect to Storage controller node over NVMe/TCP transport.
The changes in this PR are as follows:

Let ESXi host connect to Storage controller over NVMe/TCP.
Let ESXi host diconnect from Storage controller over NVMe/TCP.

I have read the contributor guidelines and have completed the following:

[x] Formatted the code using VSCode default formatter for PowerShell.
[x] Tested the code end-to-end against an SDDC.
[x] Documented the functions using standard PowerShell markup and applied AVSAttribute to newly exported functions.

@et1975 We're keeping this as discrete function to perform an operation at host level. This function can be invoked by caller for each host under cluster. This will give the ability to connect more than one host at the SAME TIME in parallel.
Today it is assumed every host is under vSphere cluster but we may have host that will not be part of vSphere cluster.
We can provide a wrapper to accept of List[] of hosts and then invoke this.

This will give the ability to connect more than one host at the SAME TIME in parallel.

One reason I brought it up is the script executions are serialized - only one will run at a time. Any parallelism you may want you'll have to accomplish inside the Run Command script with something like Start-Job.

Today it is assumed every host is under vSphere cluster but we may have host that will not be part of vSphere cluster. We can provide a wrapper to accept of List[] of hosts and then invoke this.

There will never be hosts accessible to the customer that are not part of a cluster.
The hosts are fairly ephemeral in AVS (we replace them for various reasons all the time).
With these 3 points in mind, I'm really struggling to see the use case for single-host scenario.

You maybe thinking to accept a clusterName and connect/disconnect every host under this cluster.
There will be hosts under cluster in Maintenance mod or Disconnected State, as user I will not choose to connect host which is either in maintenance or disconnected state.  My Intention is keep that decision outside the scope of these functions.
The caller or orchestrater  will decide.
What's your suggestion ?

Yes, I suggest taking cluster name as parameter and finding the hosts, filter out hosts in state like Maintenance and connect/disconnect the rest.

@et1975 i updated the code and tested,  now accept ClusterName as input and iterate over all hosts to connect/disconnect only Host with state as Connected.

@et1975 I'm not able to read your thoughts clearly, but i strongly see the use cases for a single host Connect/Disconnect.

@khensler-msft @Zsoldier
if we make Disconnect-NVMeTCPTarget a private function and allow this to be invoked inside the Dismount-VmfsDatastore then the ESXi host will not be able to access any other datastore from entire storage cluster.
e.g you have two VMFS datastores, you choose to dismount one from ESXi , in the process you also invoke Disconnect-NVMeTCPTarget, now the second datastore is also not accessible because you disconnected the entire esxi host.

@khensler-msft @Zsoldier
if we make Disconnect-NVMeTCPTarget a private function and allow this to be invoked inside the Dismount-VmfsDatastore then the ESXi host will not be able to access any other datastore from entire storage cluster.
e.g you have two VMFS datastores, you choose to dismount one from ESXi , in the process you also invoke Disconnect-NVMeTCPTarget, now the second datastore is also not accessible because you disconnected the entire esxi host.

You would need the appropriate logic in dismount-vmfsdatastore to discover if the datastore is dynamically discovered or statistically discovered and take the appropriate action.  IE if static dismount datastore and then remove the target from hosts.  If dynamic just dismount the datastore.   This way it can function for all types of storage and you can't accidentally remove the target before dismounting the datastore.

Mount/Unmount operations don't involve removal of targets. Both operations are related to access of  ESXi host to the datastore, also device/volume is still accessible to esxi host in both Mounted/Unmounted states.
Connect-NVMeTCPTarget/Disconnect-NVMeTCPTarget  are related to the network connectivity of the ESXi host to storage cluster.  Think of this as Connect/Disconnect to a TCP server over the network.

Yes I understand that.   Perhaps a call would be best for this discussion.
Kenyon Hensler
Senior Program Manager
Mobile:<PHONE_NUMBER>
<EMAIL_ADDRESS>[Microsoft Logo]

From: rahman-lb @.>
Sent: Tuesday, May 23, 2023 7:47:06 PM
To: Azure/Microsoft.AVS.Management @.>
Cc: Kenyon Hensler @.>; Mention @.>
Subject: Re: [Azure/Microsoft.AVS.Management] Enable ESXi host connect/disconnect to/from storage controller over NVMe/TCP (PR #185)
Mount/Unmount operations don't involve removal of targets. Both operations are related to access of ESXi host to the datastore, also device/volume is still accessible to esxi host in both Mounted/Unmounted states.
Connect-NVMeTCPTarget/Disconnect-NVMeTCPTarget are related to the network connectivity of the ESXi host to storage cluster. Think of this as Connect/Disconnect to a TCP server over the network.
—
Reply to this email directly, view it on GitHubhttps://github.com/Azure/Microsoft.AVS.Management/pull/185#issuecomment-1560264127, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AVVGJSKCRWKC36V3FRBRQH3XHVD7VANCNFSM6AAAAAAYELKSXQ.
You are receiving this because you were mentioned.Message ID: @.***>

we spoke about this today on the phone.  logic will be implemented in the disconnect nvme function to check for mounted datastores on the target.

@khensler-msft @et1975 are we good for the approval or anything missing ?

Ready to merge.


Docs Inconsistencies ☂

This umbrella issue houses outstanding work still to be done on improving the Carbon website documentation.
### Tasks
- [ ] https://github.com/carbon-design-system/carbon-website/issues/3978#issue-2192619824
- [ ] When and when not to use italicized text.
- [ ] When to use links and how many words to include in the link. For example: "Style tab", or just "Style". When is this capitalized or not capitalized.
- [ ] When and when not to use dashes in headings in tables.
- [ ] Unordered lists: Is it okay to have incomplete sentences for these or should they always be in sentence form and end in a period?
- [ ] Inline code snippets: When to use them in our docs and should they be all caps, or the first letter capitalized, or no capitalization.
- [ ] When to use semibold text: We usually do this when listing states. Is that okay?
- [ ] Updating our remaining components to use the new content templates
- [ ] Adding anchor links to the Style tab using the new content template
- [ ] Clarify Image Naming Convention - There seems to be inconsistency in the naming convention of images across different components. While some use numbers at the end (e.g., "usage-1"), others follow a different format. We need clarification on which format is intended to be used or if it's contextual. This will help ensure consistency and clarity in image naming across all components.
- [ ] Keep the "Interactions" section on the Usage tab for components, but we should more clearly define what we put in this section and how it differs from the Accessibility tab.
- [ ] Need guidelines on captions and hover/alt text.

Created this Boxnote, which covers most of the task items above, to receive feedback and guidance from Tom Waterton ( Senior Content Designer/UI Content Guild member) on how to improve and align parts of our documentation with IBM Style.
https://ibm.ent.box.com/notes/1530339018364?s=6nzb0ok9msgk36oz27p0s4qtmscmvffa


Update README.md

Tickets
https://github.com/serengil/deepface/issues/XXX
What has been done
With this PR, ...
I added a caution to the description of pip install deepface. This was an error that I faced since the latest tensorflow doesn't have one of your dependencies.

Instead update readme, do you mind to update requirements.txt with range
E.g.
package>=1.0.0,<=2.0.0

Okay


Full screen Snapshots include hidden elements

When taking a full screen Snapshot, the UI Tree includes elements which are not visible, hindering inspection.

Using the ClickablePoint property to determine is a UI element is occluded doesn't work (properly), during live inspection, no UI element appears to have a ClickablePoint, I assume this is due to the Outlines window being top-most.


[Feature] 建议重命名wireguard节点配置的预共享密钥字段

Verify steps

[X]  我已经在 Issue Tracker 中找过我要提出的请求 I have searched on the issue tracker for a related feature request.
[X]  我已经仔细看过 Documentation 并无法找到这个功能 I have read the documentation and was unable to solve the issue.

Description
当前ClashMeta的wireguard预共享密钥 配置字段为 pre-shared-key
而上游Clash的wireguard预共享密钥 配置字段为preshared-key
这导致可以在Clash Premium正常使用的wireguard节点配置, 在Clash Meta无法连接
建议与upstream统一命名以提升兼容性
Possible Solution
Rename pre-shared-key to preshared-key

@SakuraKoi https://www.wireguard.com/protocol/
If an additional layer of symmetric-key crypto is required (for, say, post-quantum resistance), WireGuard also supports an optional pre-shared key that is mixed into the public key cryptography. When pre-shared key mode is not in use, the pre-shared key value used below is assumed to be an all-zero string of 32 bytes.


Auto-retry (regression bug in ApplicationRunner)

When connecting with ApplicationRunner, if you just set "auto_reconnect=True" in "run", then "max_retries=100" in init, it complains because "retry_delay_jitter" is not defined. So;
runner = ApplicationRunner(URL, REALM, max_retries=100)
runner.run(Test, auto_reconnect=True)

Gives;
  File "lib/python3.6/site-packages/twisted/application/internet.py", line 520, in policy
    return min(initialDelay * (factor ** attempt), maxDelay) + jitter()
  File "autobahn-python/autobahn/twisted/wamp.py", line 361, in <lambda>
    ('jitter', lambda: random.random() * self.retry_delay_jitter)]:
builtins.TypeError: unsupported operand type(s) for *: 'float' and 'NoneType'

and;
runner = ApplicationRunner(URL, REALM, max_retries=100, retry_delay_jitter=1)
runner.run(Test, auto_reconnect=True)

Works Ok. The jitter field should provide a default value, "random.random()" according to the docs, but apparently it doesn't .. (?)

This line
https://github.com/crossbario/autobahn-python/blob/941600534ea28120342b39ad53c71aa920abb54c/autobahn/twisted/wamp.py#L188
should be
self.retry_delay_jitter = retry_delay_jitter if retry_delay_jitter is None else 1.


[FIX] Avoid system interrupt return failure in readline function

When terminal window is resized, readline function returns false because errno

The branch has a typo and I think #218 provides a more proper solution which keeps the error checking intact.


customs addons for ISL

Sorry, but we're not adding further contributed modules to the library. The best way to distribute this would be as a separate npm package.


Fix vulnerabilities for AzureSpringCloudV0 task

Task name: AzureSpringCloudV0
Description: Resolved critical vulnerabilities by running the 'npm audit fix' command
Documentation changes required: N
Added unit tests: N
Attached related issue: N
Checklist:

[x] Task version was bumped - please check instruction how to do it
[ ] Checked that applied changes work as expected

@yevster could you take a look?

Hi, @lucen-ms, could you take a look at this PR and test these changes?

The AzureSpringCloudV0 task does not have an owner.
This task should be assigned to some team before work on this PR will continue.

Actual PR - https://github.com/microsoft/azure-pipelines-tasks/pull/16602


Polar/radial grid

[✓] I agree to follow the Code of Conduct that this project adheres to.
[✓] I have searched the issue tracker for a feature request that matches the one I want to file, without success.

Is your feature request related to a problem? Please describe.
Although unusual, I wanted to make a circular ERD. Issues:
• Some table shapes don't rotate, others rotate improperly (nested boxes stay aligned vertically).
• Connectors either follow harsh angles, or have to be meticulously tweaked with goofy curves.
Describe the solution you'd like
Instead of messing with individual objects, what if entire grid could be polar. This would allow standard orthogonal shapes and connectors to be "warped" according to the new type of coordinates: shapes would simply be rotated based on their position around the center point, and for connectors, the horizontal lines would follow a circular path around the main center point, and vertical lines would become radial.
Describe alternatives you've considered
This is probably a tall order. So some alternatives would be:
• allow shapes (like the Entity Relation ones) to be rotated,
• allow shapes (like the General List one) to be properly rotated (where the list items stay within the main box),
• add an option for connector (regardless of the type) to be joined perpendicularly to a shape.
Additional context

Thank you.

Thanks for the suggestion, but as you probably realised, there's no way we can implement this.


Feat/issue 26 validuntil

DRAFT - just making this for a discussion

@dslmeinte can you have a look - I left a TODO comment in the Makefile. Which approach do you think is best? I prefer "use default schema for all except x,y,z" and then provide a list of "x, y, z". But then I'd want to see if I can migrate the build to say package.json (need to investigate a little to see if we do it there in a nice and clean way).

Looks good!

Moved over to using ajv directly, which also fixed an issue with the previous version at the same time (the shell was executed out of order causing the exit code to execute before the shells were finished).

It would be good to put a sentence or 2 in the README about the fact that the value sets are validated against schemas (and what the schemas are for, exactly, as we have 3).
After that: let's squash to a new branch, and merge a new PR.

@skounis do you mind if we rename this + eventually do a squash merge? the history is a bit... messy :)

(I'd like to rename to ("feat/validation-valuesets-schema")

@skounis do you mind if we rename this + eventually do a squash merge? the history is a bit... messy :)

That's fine, please go ahead.

The existence of the GitHub Action is not mentioned in the README.


Adding Semantic information to html output

for #8

Coverage decreased (-1.0%) to 65.324% when pulling 87f57a7f106fbd112f082f7d2a4d9cfee7a91b34 on jiwalker-usgs:semantics into 765b5b59ebd5f42b3f1b9b1bad90f3bc47069318 on USGS-VIZLAB:master.


2 small issues

First of all, I'm very happy to use this library. I'm creating nice and lightweight websites with it!

The first issue is a small issue again with the Carousel interval via Javascript initialization. Setting interval to 0 works like a charm but setting the interval to false does not work; It still slides after the default amount of seconds. See https://github.com/thednp/bootstrap.native/issues/192#issuecomment-377568414

The second issue is a small typo in the docs.  Go to the docs and go to "Carousel events". 1 event name is missing a "e"-character: slid.bs.carousel should be slide.bs.carousel. But then, the same event is listed twice, see https://preview.ibb.co/jyhwrx/docu.png

EDIT: I didn't know "slid" was a English word but apparently it is, so maybe the second issue is not a typo?

The slide event is before the animation and slid event is when the animation finished, we are inline with the original plugin.
I will look into Carousel issue tomorrow, thanks for bringing it up.

Carousel issue fixed, closing this now.

Commit is now online, you can test the latest MASTER anytime you have the time.

@thednp I tested it and it works like a charm, thank you very much!


Update rustfmt install instructions

This PR takes work from https://github.com/diesel-rs/diesel/pull/1762 to update the instructions to contributors, and specifies the version of rust used to run cargo fmt and cargo clippy.

It looks like CI is failing due to unstable features being tested for the stable and beta versions, but I'm not sure what the cause of this is

I think this is caused by the rust-toolchain file.
@diesel-rs/contributors Maybe we should not add a rust-toolchain file for a compiler version that is not supported any more by diesel? So maybe just bumping rustfmt to 1.30?


docs(behavior_velocity_planner): use markdown extensions for image alignment

Signed-off-by: satoshi-ota<EMAIL_ADDRESS>Description
images are not shown in https://autowarefoundation.github.io/autoware.universe/main/planning/behavior_velocity_planner/crosswalk-design/
I fixed following points:

not use <div> tag
use mkdocs-material https://squidfunk.github.io/mkdocs-material/reference/images/#image-captions

Pre-review checklist for the PR author
The PR author must check the checkboxes below when creating the PR.

[x] I've confirmed the contribution guidelines.
[x] The PR follows the pull request guidelines.

In-review checklist for the PR reviewers
The PR reviewers must check the checkboxes below before approval.

[ ] The PR follows the pull request guidelines.

Post-review checklist for the PR author
The PR author must check the checkboxes below before merging.

[ ] There are no open discussions or they are tracked via tickets.

After all checkboxes are checked, anyone who has write access can merge the PR.

FYI:https://mkdocs.readthedocs.io/en/859/user-guide/configuration/#use_directory_urls

@satoshi-ota I cannot see the figure. is it correct?

@satoshi-ota I cannot see the figure. is it correct?

@yukkysaito yes, we can't see figures correctly in github.com such as plantuml, but these figs are shown in external documentation site.
https://autowarefoundation.github.io/autoware.universe/pr-1299/planning/behavior_velocity_planner/crosswalk-design/

Pass required CI checks :smile:
Could you give me any comment or Approve ?

Pass required CI checks smile Could you give me any comment or Approve ?

@yukkysaito @kenji-miyake @TakaHoribe


Automatically triggered refreshes shouldn't remove items from the Sidebar if the Read filter is on

I fixed this, I just forgot to reference it in the commit message.


Testcase is failing due to the old version ruby.

I have set up the test environment based on ruby 2.3.X, but the testcases are failing as below:
      Failure/Error: cloud.delete_stemcell('i-foo')
      NoMethodError:
        undefined method `match?' for "i-foo":String
        Did you mean?  match

I guess it's because of the match? is introduced since ruby 2.4, so we need to make the ruby version explicit in document.

Fixed with https://github.com/cloudfoundry-incubator/cf-openstack-validator/commit/bdcc74b2a4efefb2b6023c396f9a3a003dfcbaab
Thanks for reporting!


Universalize controllers

There are controllers in the Castaway.OpenGL.Controllers assembly that should be moved into Castaway.Level.

The commits here were merged in #83. Whoops!


Benchmarking LLVM code generation

This PR is to introduce the benchmarking for LLVM code
generation pipeline. For that, new options have been added:
benchmark
  LLVM benchmark option
  Options:
    --run                                           Run LLVM benchmark (false)
    --instance-size INT                   Instance struct size (10000)
    --repeat INT                               Number of experiments for benchmarking (100)
    --backend TEXT:{default}         Target's backend (default)

The backend option does not work yet (as LLVM can
automatically set up the target/data layout information).
Example:
$ nmodl hh.mod llvm --ir --vector-width 1 benchmark --run --instance-size 100000 --repeat 10 --backend default

Created LLVM IR module from NMODL AST in 0.006765817

Benchmarking kernel 'nrn_state_hh'
Experiment 0: compute time = 0.013977749
Experiment 1: compute time = 0.004847989
Experiment 2: compute time = 0.004739051
Experiment 3: compute time = 0.004801505
Experiment 4: compute time = 0.005540785
Experiment 5: compute time = 0.004965192
Experiment 6: compute time = 0.004840146
Experiment 7: compute time = 0.00485995
Experiment 8: compute time = 0.005134093
Experiment 9: compute time = 0.004844469
Average compute time = 0.0058550929

Note: The benchmark fails for kernels larger than
1000000. This is currently being investigated (Hence, draft PR).

@pramodk I had to change some Cmake dependencies to get CodegenDataHelper from tests. I am thinking of moving this utility to src instead, but unsure where to place it (I guess other backends would use it as well). Any ideas?

Copying Slack conversation here:

GitLab CI is failing with errors like:

Seems like include path needs to be adjusted.
I can fix that if you can't reproduce that locally. Apart from that, is this ready for review?

Another question @georgemitenkov : on your laptop, does this work with SSE and AVX? i.e. vector width 1, 2 and 4? Just to be sure before I start testing :)

@pramodk So far I have an issue with large kernels (size > 1000000), that fail. Apart from that, the code is more or less ready for review. There are 2 outstanding issues:

[ ] Fix include path
[ ] Propagate avx/sse info to JIT (Currently only default target triple is supported, and I am figuring out how to propagate it there)

With last fixes, CI should be good.
FYI : if you add new library (llvm_benchmark) and if those cpp files are using AST headers then that library needs to be compiled after AST header files are generated. In order to achieve that, we have add dependency with the target that generate AST headers .e.g add_dependencies(llvm_benchmark lexer).
I would like to make some cleanup/rearrangements - e.g. I am thinking benchmark related files could be outside src/codegen. For example, separate top level directory benchmark. But lets see this tomorrow.

With last fixes, CI should be good.
FYI : if you add new library (llvm_benchmark) and if those cpp files are using AST headers then that library needs to be compiled after AST header files are generated. In order to achieve that, we have add dependency with the target that generate AST headers .e.g add_dependencies(llvm_benchmark lexer).
Thank you! I see, I suppose in my case it never failed because I had AST headers cached and didn't do a clean build.

I would like to make some cleanup/rearrangements - e.g. I am thinking benchmark related files could be outside src/codegen. For example, separate top level directory benchmark. But lets see this tomorrow.
Yes, that's a good point. We can discuss this more later.

Regarding the bug, I have found that it is mech->node_count that has a weird value (due to misaligned access I suppose).


Line Follower

There seem to be two types of line followers out there commonly in use: one uses two GPIO pins (both used for input), the other uses one pin.
There are plenty of straightforward RPi.GPIO implementations. I'm not sure how effective they are.
Single GPIO pin example (from camjam kit 3 worksheet):
follower = 25

GPIO.setup(follower, GPIO.IN)

while True:
    if GPIO.input(follower) == 0:
        print('The sensor is seeing a black surface')
    else:
        print('The sensor is seeing a white surface')
sleep(0.2)

Two GPIO pin example (provided by @RPi-Yaz14 here):
left = 21
right = 26

GPIO.setup(left, GPIO.IN)
GPIO.setup(right, GPIO.IN)

while True:
    if GPIO.input(left) == GPIO.input(right):
        print("forward")
    if GPIO.input(left) != 0:
        print("left")
    if GPIO.input(right) !=  0:
        print("right")

I've a feeling the single GPIO one is doing the same thing (reading on two inputs) but returning a single value.
I did have a two-input one but I gave it to Sam to give you @waveform80. (maybe there's another one around somewhere).
Should we create one class or two, for the different types of sensor?

Also maybe it should be called Line Detector, as Line Follower suggests following - but that's the robot's job!

Not sure which, if any, of our base classes this should be based on.

Base it on the first one (1 GPIO) as the Edukit uses the individual sensor, so people can add extras and just assign them to different pins. I used the Ryanteck 3 sensor PCB, hence used 2 pins.

Is Ryan's one actually two line detectors in one?

It has 3 (there is one for centre which I didn't really see as that relevant)

His detectors are all individual sensors on one PCB, using a different pin for each

Okay, firstly it should definitely be LineDetector or LineSensor or something of that ilk rather than follower. As you note, that connotes purpose which is down to the code the user associates with the sensor. As to whether we use separate classes - I'd say probably one class per class of line sensor but there might be a hierarchy if we can re-use stuff. I should probably go and read up on this a bit more when I get a second!

Just realised LineSensor was in the release but undocumented.

We should add at least one variant, e.g. DualLineSensor as discussed previously (somewhere, sometime)

Perhaps DualLineSensor should be a CompositeDevice of two LineSensors?
AIUI with a single line-sensor you can only detect whether you're over the line or not, so if you 'lose' the line you have no idea whether you need to turn left or right to get 'back on' the line. But with two or more line-sensors you'll hopefully have at least one sensor still over the line, while another sensor isn't over the line, so you have a better idea of which direction to turn, to get back on the line.

The composite idea for multi-sensors is nice, especially as we could do another -1 to +1 value for that (0 for on the line, -1 for "off the left", +1 for "off the right" or something ... how to handle off the line entirely though ... not sure, have to think about that). For now, for the 1.2 release I'm just going to document the single LineSensor class properly. As @RPi-Yaz14 notes, that's the one in the CamJam kit, so we can presume it's the common one and it can be used as a basis for multiple sensor devices until we get around to doing something more refined with it.

Yup, I think you're absolutely right - after a bit of thought the compound value (of all the individual sensor values) which a CompositeDevice derivative would produce anyway is more useful and more representative of the sensor's state (unlike an H-bridge attached to a motor where -1 to +1 makes some physical sense)


Changing supermemo.wiki platform, suggestion/exemplary website

Description
So there is this super cool website which allows users to annotate stuff very convinently

the site: https://www.econometrics-with-r.org/1-1-a-very-short-introduction-to-r-and-rstudio.html
We could find out how to mimic such platform

They seem to be using this service: Hypothesis


bugfix: 资源配额 业务与模板ID相同时选中不正确 (closed #1246)

Codecov Report

Merging #1247 (b83dd8f) into V2.2.32-rc (b9b47ef) will increase coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##           V2.2.32-rc    #1247   +/-   ##
===========================================
  Coverage       73.29%   73.29%
===========================================
  Files             359      359
  Lines           24114    24114
===========================================
+ Hits            17674    17675    +1
+ Misses           6440     6439    -1

Impacted Files
Coverage Δ

apps/node_man/handlers/job.py
85.19% <0.00%> (+0.30%)
:arrow_up:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.


[BugFix] Fix parquet DictDecoder array out of bounds access

Signed-off-by: Smith Cruise<EMAIL_ADDRESS>What type of PR is this：

[x] BugFix
[ ] Feature
[ ] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Which issues of this PR fixes ：

Fixes #
Problem Summary(Required) ：

We resize _indexes with a chunk size(4096) in set_dict().
If the user has a col ARRAY<INT>, it has more than 4096 elements in one row, in next_batch() function, the count value is larger than 4096. So to be safe, we need to reserve() vector before writing to _indexes.
Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto backported to target branch

[ ] 2.5
[ ] 2.4
[ ] 2.3
[ ] 2.2

run starrocks_admit_test

@Mergify backport branch-2.5


Port to PCL

Build failed, looks like due to changes to nuget reference. Please see the Travis log.

Updated NuGet to 1.4.0


Add placeholders for the different values asked to the user

Let's use placeholders for the values asked in the following screen:

Context Name: "OpenShift Developer Sandbox" (must be a default value, to avoid the user to have to input a text in this field)
Login: provide an example of how the login command should be looking like

@benoitf is there a way to add placeholders for fields in Create New... form? I did not notice it.

@dgolovin there is no placeholder but there is default key that will be set as default value (so it fills the input with that default value)

Depends on PRs to podman desktop above


Creating FAP Filter

I cloned your main branch, installed mt-metadata using the setup.py file and tried to create a frequency_response_table_filter and convert my filter to an ObsPy stage. I ran into the following error:
---------------------------------------------------------------------------
	IndexError                                Traceback (most recent call last)
	/var/folders/pn/k5blhlvd7vzdfy85t2z3_h940000gn/T/ipykernel_1237/2779005987.py in <module>
	----> 1 bx_filter.to_obspy()

	~/Downloads/mt_metadata-main/mt_metadata/timeseries/filters/channel_response_filter.py in to_obspy(self, sample_rate)
	    291
	    292         """
	--> 293         total_sensitivity = self.compute_instrument_sensitivity()
	    294
	    295         total_response = inventory.Response()

	~/Downloads/mt_metadata-main/mt_metadata/timeseries/filters/channel_response_filter.py in compute_instrument_sensitivity(self, normalization_frequency)
	    245             sensitivity *= complex_response
	    246         try:
	--> 247             return np.round(np.abs(sensitivity[0]), 3)
	    248         except TypeError:
	    249             return np.round(np.abs(sensitivity), 3)
IndexError: invalid index to scalar variable.

I updated my local copy of your branch with the following fix: except IndexError (line 248) and was able to convert my filter to an ObsPy stage. Not a big deal but I just wanted to bring it up in case it would be helpful.

Here is the workflow that I followed to create a frequency table filter and convert the created filter to an ObsPy stage.
import numpy as np
import os

from mt_metadata.timeseries.filters.frequency_response_table_filter import FrequencyResponseTableFilter
from mt_metadata.timeseries.filters.channel_response_filter import ChannelResponseFilter

fap_filter = FrequencyResponseTableFilter(name='correction_factor_bx',
                                          units_in='nT',
                                          units_out='nT',
                                          comments='Frequency dependent correction for magnetic field')

path2cal = '/Users/maevapourpoint/PASSCAL/Software/LEMI/Calibration_files/rsp_LEMI-424_N131to145/'

bx_cal_file = os.path.join(path2cal, 'LEMI-424_N131_Bx.rsp')
frequencies = []
amplitudes = []
phases = []
with open(bx_cal_file, 'r') as fin:
    for line in fin:
        frequency, amplitude, phase = line.strip().split()
        frequencies.append(float(frequency))
        amplitudes.append(float(amplitude))
        phases.append(float(phase))
fap_filter.frequencies = np.array(frequencies, dtype=float)
fap_filter.amplitudes = np.array(amplitudes, dtype=float)
fap_filter.phases = np.deg2rad(phases, dtype=float)

bx_filter = ChannelResponseFilter(filters_list=[fap_filter])

bx_response = bx_filter.to_obspy()

Find attached the calibration file (FAP table) I used as input. In this file, the responses of the logger and sensor (fluxgate in this case) are combined.
LEMI-424_N131_Bx.rsp.zip
In this case, I am trying to create a single stage response since the LEMI logger "internally corrects" the recorded signal and outputs physical units (nT for the Hx channel). This single stage response would be equivalent to a frequency-dependent correction factor that one would have to apply to the data to account/correct for some assumptions made by the LEMI manufacturers that don't hold as we get closer to Nyquist (i.e., unit gain and zero phase).
The computed instrument sensitivity (line 245) is: (0.998636015619-0.00840646933685j)
Let me know if you need additional details.

@MaevaP I replicated the bug.  Thanks for the find.
Added IndexError to the exception list in line 248 of channel_response_filter.py.  Which should fix the issue.
The instrument sensitivity looks close to correct basically 1 at 0.04 Hz.
Reopen this if that did not fix the issue.

@kujaku11
Just tested your fix and it works. Thank you for the quick turnaround.


app crashing randomly after browsing for a while

i am using this awesome app on my samsung galaxy note 3 SM-N9005 running unofficial LOS16. i know you are not working anymore on this app. but it would be really great if you at least fix the crash i am facing when i am browsing reddit. i am attaching a log file below. again thanks for your great work!
2019-01-04-19-58-35.txt

I've fixed this issue and few other in my fork. I do not have any signing keys though, so you can only download debug version (latest available here) or compile it yourself.

@Tunous i am running the debug version & it's all good! i am gonna post further issues in your fork from now on. hoping to see more improvements & implementations from you! thanks!


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 3297 ms

Resolved: Bofrost.at performance has improved in 35942a1 after 19 minutes.


feat: Support the fallbackLng logic from i18next

Should allow the usage of the following value as the fallbackLng key in the i18next configuration:
{ 'de-BE': [ 'de', 'en' ],
  'fr-BE': [ 'fr', 'en' ],
  'nl-BE': [ 'nl', 'en' ],
  'de-CH': [ 'de', 'en' ],
  'fr-CH': [ 'fr', 'en' ],
  'it-CH': [ 'it', 'en' ],
  'de-LU': [ 'de', 'en' ],
  'fr-LU': [ 'fr', 'en' ],
  'fr-CA': [ 'fr', 'en' ],
  'fr-FR': [ 'fr', 'en' ],
  'nl-NL': [ 'nl', 'en' ],
  'de-AT': [ 'de', 'en' ],
  'de-DE': [ 'de', 'en' ],
  'da-DK': [ 'da', 'en' ],
  'fi-FI': [ 'fi', 'en' ],
  'it-IT': [ 'it', 'en' ],
  'no-NO': [ 'no', 'en' ],
  'sv-SE': [ 'sv', 'en' ],
  default: [ 'en' ] }

This functionality seems to be working on the client-side. Yet the server wasn't able to detect the correct language namespaces it needed to preload.
Can you explain how I can contribute more to this package?

@WardVH Can you fix the CodeClimate issue here?

@isaachinman Any idea what is causing the one issue that is still to fix?

How do the changes here support a fallbackLng when it's an object?

The idea was to analyse the config before initialising the store with the languages and namespaces required. During analyse I build an array with the required languages that we should load.

@isaachinman I'm sorry but why is that eslint so strict that it doesn't allow a single if statement without an else statement? How should I add the default property if it's set?

You can restructure your if statements to be more exact. It's a normal eslint rule. Are you committing with --no-verify? I would ask that you only push changes to PRs on this repo when you expect them to pass CI. I get an email notification for each of these failures.

I've changed it and was able to check the linting locally. I'm sorry for the spam. Could you explain to me how I can run the test or even expand these to include some of the possibilities of the fallback configuration?

There's a precommit hook which should run the lint and test suite. Otherwise you can check the package.json for the appropriate scripts to run.
Indeed you'd need to add test coverage to this new functionality for us to consider merging it.

@isaachinman Can you explain to me how to proceed with the tests? How can I run or expand these? I'm fairly new to this javascript testing..

Test files should be named identically to the thing they are testing.

What is fallback-lng-analysis.js? Why does it return an array of languages? What is its purpose?

Test files should be named identically to the thing they are testing.

They are aren't they?

What is fallback-lng-analysis.js? Why does it return an array of languages? What is its purpose?

It's an utility function to ease the determination of what languages we should pre-load. It makes it also very easy to test in isolation all the different configuration approaches for the fallbackLng parameter in the config.
It also hides this logic from the HOC.

We don't support regional locales at all at the moment.

Can we please be constructive here? May I ask what's the reason for this? Why wouldn't the package allow it since it's a wrapper around react-i18next which clearly supports regional locales.
I don't mind working on the support but you are not being constructive towards this and I fail to understand why.

Hi @WardVH - I'm very busy at the moment and still trying to keep up with the ongoing work here. Unfortunately I don't have time to advise on things like JavaScript testing as a general concept, but I am making an effort to quickly review your code and provide bits of information that you might not otherwise have.
We don't support regional locales (yet) because it conflicts a bit with localeSubpaths. It hasn't been requested thus far, and can certainly go on the roadmap, but should remain unrelated to the work here.
To be able to merge this PR, I would expect to see clear, self-explanatory code that is covered by tests. For example, it seems like fallback-lng-analysis needs to have a name that describes what it does. Additionally, variable names like languagesThatWeWillLoad won't be merged - that will need to be changed to something like languagesToLoad.
I am happy to do a proper PR review, but only once you are fully done with your work and feel that it is ready to be reviewed. You have been actively pushing commits for 11 days, so I have not done that. You can feel free to let me know when it is ready for review. I hope this is constructive for you.

Hi @WardVH - would you like me to take over and make the small changes necessary to get this complete and merged? I do not mind. Thank you for your contributions.

Hi there, I have some time for it tomorrow. I've tried to use the fork in my current project. But started to notice odd behaviour (no longer SRR when a component used the t function to translate something etc) so not sure it works completely like intended..
I will let you know when you can review it?

Sounds good. Yes indeed there appear to be a couple changes necessary. Just let me know.

@isaachinman if you could have a look and point out potential issues?
FYI, i had to manually upgrade the jest package to 24.3.0 in order to run the test on node version v11.11.0 (see https://stackoverflow.com/questions/55059748/travis-jest-typeerror-cannot-assign-to-read-only-property-symbolsymbol-tostr).
I obviously didn't commit that change but wanted to let you know anyway.
Thanks for the time btw.

FYI, i had to manually upgrade the jest package to 24.3.0 in order to run the test on node version v11.11.0

Can you open an issue to either upgrade our Jest version, or update our Node/engine spec?

@WardVH This is looking better to me. Did you perform manual QA, and/or are you using this fork in an application somewhere?
@capellini Not sure if you're still around, but would you have time to review this?

@isaachinman Yes, still I'm around, but quite busy.  I'll take a look tonight, though.

@capellini Great, thank you! It's a relatively small diff now, and I think it touches some areas you might have knowledge on.

@isaachinman Sorry - I wasn't able to do this last night, but will look at it tonight.

Hi @WardVH - seems like there are still a few unresolved comments? Or if you believe they are resolved, please mark them as resolved.
Just let me know when you feel this is ready for another review. Thanks!

Any update here?

@isaachinman Sorry was a bit busy and lost track of this. I've done the requested changes.
Can you have another look?
Thanks!

@WardVH One final picky thing - can we call it lngsToLoad? I don't think "detector" adds much value. Then it looks like we're good to merge.

@isaachinman done!

@WardVH I've just noticed you have regional locales in the tests. Since we don't support those server side yet, can you remove that part?

@isaachinman I think it's ready!

Great. I won't have time to look until next week unfortunately. But glad to hear it!

Please also add tests to cover both initialLng and fallbackLng being undefined.

@isaachinman should be fine now!

@isaachinman thank you so much for the time and effort into reviewing this, I've learned a great deal from it so thanks!

@isaachinman how can i use this version in our production code?

@WardVH It's been released in v0.39.0.


Add index.d.ts

Okay so, basically, this define the class and Parasm for the program the user is using (e.g: Visual Studio Code, webstorm)
An example from Visual Studio Code:

It works perfectly fine on windows


PDB Residue Relabel

I had the need to relabel residues in PDBs, mostly due to compatibility issues between programs, I thought it would be useful for pdb-tools.
Here I propose pdb_resrelabel.py as a script to relabel residues in a PDB file. The script follows the pdb-tools philosophy of one script one task and arguments should be passed before the .pdb file input. It accepts and outputs piping.
Please consider it for the pdb-tools repository.

Added in version 2 with a simplified interface. Thanks!


Please invite me to the Recode-Hive GitHub Community Organization

Name
Vedavalli
Discord Username (if applicable)
Vedavalli
Additional Context
No response

Invitation sent to join the GitHub Organisation. Welcome to the community 🎉
Don't forget after accepting to make it public so it appears on your GitHub profile for everyone else to see. You can do this by finding your name in the GitHub organisation list and change the dropdown to public https://github.com/orgs/recodehive/people