Update passfield.js

Change passfield divs order for Bootstrap 3 compatibility, especially with input-group-addon

Thanks, I have added some fixes to improve compatibility and merged input-groups support in 45775f3 (v1.1.11).


Request to add $Literal<T> utility type

This came up as part of working Ramda's libdefs for assoc in #2010 in flow-typed. A lot of this will be copy-paste from that discussion.
Flow lets us treat a literal value as a type, which has a lot of utility by itself. The only way we can annotate for those types is via the exact type we're looking for. This doesn't lend itself well to any kind of generic application. My hope here is that $Literal<T> would allow us to express that some value must be a literal and that T could work as a constraint (this is a string literal, for example). In the case of Ramda's assoc we want to carry the key provided as an argument and use it as an actual key in the intersectioned object we're returning. If we know the key is a literal then we can safely infer the key in the object is also fixed, so we can effectively provide a new object with extra properties.
Aside from generic annotation, it looks like we can already do a fixed version:
// @flow

type Foo = {
  foo: number,
}

const key = 'foo'

const foo: Foo = { [key]: 1 } // works as expected!

const getFooValue = (): string => 'foo'

const key2 = getFooValue()
const foo2: Foo = { [key2]: 2 } // oh noes!

const getFooValueLiteral = () => 'foo'

const key3 = getFooValueLiteral()
const foo3: Foo = { [key3]: 2 } // Works again

try
There's some special type information that's in there. In fact, this can be hardcoded as a type:
// @flow

type Foo = {
  foo: number,
}

const getFooValueLiteralExplicit = (): 'foo' => 'foo'

const key = getFooValueLiteralExplicit()
const foo: Foo = { [key]: 2 } // works

try
But what I would like to see is something that allows me to express something like $Literal<string> (constrained so I can limit what it is - or use * if I don't care), and pass them around generically. Flow seems to understand a literal differently from other things and gives it special treatment, but short of the actual value being typed, I can't use it generically.
In the case of assoc, $Literal would be the key and because we've enforced that it will be a literal, we should be able to carry it over as part of the type def for the intersection create with S & { K: V }. It knows that K is legitimately a type that can be a property, and at some point that exact property can be resolved as part of feeding all of the typed information to assoc during our invocation.
I imagine this type could also play well with $PropertyType and/or $ElementType which requires a fixed literal type.
Having a utility type like this would be great but I don't believe this is blocking anyone currently.

Duplicate of #2242 I believe.

@jfirebaugh indeed it is. Thanks! Closing.


Syntax Suggestions

What do you think about syntax like this?

import { useSheet, SheetProvider } from 'react-modal-sheet'

// root of app
<SheetProvider>
  <XSheet />
  <SomewhereElseInApp />
</SheetProvider>

const SomewhereElseInApp = (props) => {
  const {
    openSheet, // this would default to opening sheet-1
	setSnapPoint,
    closeSheet,
  // all the options below are optional including the ID since you can pass the ID to openSheet
  } = useSheet('x-sheet-id', {
	onClose,
	onOpenStart,
	onOpenEnd,
	onCloseStart,
	onCloseEnd,
    onSnap,
  })
  return (
    <>

 openSheet('sheet-id-2')}>Opens Sheet 2 with no props

 openSheet('sheet-id-1', props)}>Opens Sheet 1 and passes props to it

 openSheet({ name: 'cool', ...props })}>Opens X Sheet and passes props to it

    </>
  )
}

import { Sheet } from 'react-modal-sheet'

const XSheet = () => {
  const {
    isOpen,
    ...props // these are the props passed from `openSheet`
  } = useSheet('x-sheet-id')
  return (
    <Sheet
      id={'x-sheet-id'}
	  isOpen={isOpen}
      {...props}
    >
      {/* whatever content for the sheet */}
    </Sheet>
  )
}

This allows us to reuse the same action sheet throughout the app only having to import it once, and passing different props to it wherever we want to use it.

Thank you @alex-cory for the suggestion!
Can you provide a bit more background info about this idea? Are there some limitations in the current API design that make it too difficult or impossible to achieve some desired outcome? I'm not sure I'm able to see the benefits of your proposed API design 🤔

Thank you @alex-cory for the suggestion!
Can you provide a bit more background info about this idea? Are there some limitations in the current API design that make it too difficult or impossible to achieve some desired outcome? I'm not sure I'm able to see the benefits of your proposed API design 🤔

It allows you to have shareable sheets, and open them from anywhere in your application.
So, your solution assumes we will be setting the state of the sheet to open from within the same component. For instance, if I have a sheet that needs to be shared throughout the app, the only way to open it is via redux, useContext, or other global state management libraries.
If I have a sharable sheet, I need to be able to open it and give it props. This solves for that too.

It's quite easy to create a custom global sheet system with React context like you mentioned so I'm quite hesitant to completely rewrite the internal sheet API just to support this use case.
I guess we could have an example for this use case in the examples folder 🤔

I'm closing this issue since it's out of the scope of this library. Feel free to fork this lib if the API is not flexible enough for your needs. Happy hacking! 👨🏼💻


Feature request: deleting the image link

Hi, I would love to have a quick option to remove images from my notes. An option to remove the embedded image with a context menu item would be amazing.

You mean removing the local image file in the vault, right?
So after deletion, the embed says "... is not created yet"? Or remove the embed also?

Thank you for the quick reply. My main use case is to remove the embed, basically clearing the line of the image. A context menu item called "remove image" would be great.
The file itself could remain in my vault, I use the "clear unused images" plugin for that.


Failed to start game with error

Error:
java.lang.RuntimeException: java.lang.IllegalAccessException: class com.google.common.jimfs.SystemJimfsFileSystemProvider cannot access a member of class com.google.common.jimfs.JimfsFileSystem with modifiers "public"
Minecraft 1.16.2
Forge 33.0.20
Fabric Loader 0.9.2+build.206
Java jdk-11.0.8
OS Windows 10 Pro

Could you send a full log?  I had Fabric enabled before starting the game. Sorry for the nonsense.

Ah, no worries. Do note that if modpacks include Jumploader you can just remove it (and Forge) and install Fabric anyway.


Update Failure - ValueError: invalid literal for int()

Describe the bug
When performing an update. It fails.
To Reproduce
Affected versions:
Copier 8.2.0
Steps to reproduce the behavior:
On an exsisting doodba project, try to update the template.

copier update --trust
Result:  ValueError: invalid literal for int() with base 10: 'I will use an external PostgreSQL server'
see Traceback

/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/copier/template.py:133: OldTemplateWarning: This template was designed for Copier 7.0.1, but your version of Copier is 8.2.0. You could find some incompatibilities.
  warn(
Updating to template version 5.2.0
/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/copier/template.py:133: OldTemplateWarning: This template was designed for Copier 4.1.0, but your version of Copier is 8.2.0. You could find some incompatibilities.
  warn(
Traceback (most recent call last):
  File "/home/username/.local/bin/copier", line 8, in <module>
    sys.exit(copier_app_run())
             ^^^^^^^^^^^^^^^^
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/plumbum/cli/application.py", line 638, in run
    inst, retcode = subapp.run(argv, exit=False)
                    ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/plumbum/cli/application.py", line 633, in run
    retcode = inst.main(*tailargs)
              ^^^^^^^^^^^^^^^^^^^^
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/decorator.py", line 232, in fun
    return caller(func, *(extras + args), **kw)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/copier/cli.py", line 66, in handle_exceptions
    return method(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/copier/cli.py", line 365, in main
    with self._worker(
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/copier/main.py", line 204, in __exit__
    raise value
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/copier/cli.py", line 372, in main
    worker.run_update()
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/copier/main.py", line 814, in run_update
    self._apply_update()
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/copier/main.py", line 833, in _apply_update
    with replace(
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/copier/main.py", line 204, in __exit__
    raise value
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/copier/main.py", line 842, in _apply_update
    old_worker.run_copy()
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/copier/main.py", line 733, in run_copy
    self._ask()
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/copier/main.py", line 433, in _ask
    answer = question.parse_answer(result.init[var_name])
             ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/copier/user_data.py", line 423, in parse_answer
    choices = self._formatted_choices
              ^^^^^^^^^^^^^^^^^^^^^^^
  File "/usr/lib64/python3.11/functools.py", line 1001, in __get__
    val = self.func(instance)
          ^^^^^^^^^^^^^^^^^^^
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/copier/user_data.py", line 307, in _formatted_choices
    self.cast_answer(c.value)
  File "/home/username/.local/pipx/venvs/copier/lib64/python3.11/site-packages/copier/user_data.py", line 218, in cast_answer
    return type_fn(answer)
           ^^^^^^^^^^^^^^^
ValueError: invalid literal for int() with base 10: 'I will use an external PostgreSQL server'

Expected behavior A clear and concise description of what you expected to happen.
Copier to update the project
Additional context
Copier 8.2.0
This template was designed for Copier 7.0.1, but your version of Copier is 8.2.0. You could find some incompatibilities.
This template was designed for Copier 4.1.0, but your version of Copier is 8.2.0. You could find some incompatibilities.
The .copier-answers.yml file actually contains:
postgres_version: 13
The mentioned command here in the readme with  --force is no longer a valid argument
https://github.com/Tecnativa/doodba-copier-template/blob/19b5ecbba0b0c5d645ed44b6d3dc8e680c2c2815/README.md?plain=1#L89C1-L92C8

The main problem is that your copier.yaml is not compatible with the copier version that you are using.
The reason: When you're updating from an older template to a newer, Copier will try to produce a vanilla project with the old template before updating it, to be able to extract a smart diff and apply the required changes to your subproject.
This means that an incompatible project cannot be updated.
A possible solution would be to try to upgrade using a version of copier that works with the current project.
Another solution would be to manually fix the copier.yml file to make it compatible with newer versions of copier.
Perhaps there are better solutions. Maybe with copier 8.3.0 and use recopy :/
Additional Notes:

Template versions before 5.1.4 are not compatible with latest pyhton versions (pre-commit regex issue).
Template versions before 3.0.0 are not compatible with latest copier versions ('postgres_version' question has invalid choice type).
copier-answers before 5.2.0 are not compatible with latest copier versions (null answers are not supported for some question types).


DSNetwork generation

Hello,
I have come up with a question while I was looking at the code of the file DSNetwork.cpp.
In function DSNetwork::DSNetwork(double bw, int pe_size),
there is this piece of code:
for(int i = 0; i < levelnum; i++) { //For each level
for(int j = 0; j < pe_size; j++) {  //For each PE. Would not be different depending on the level?
DSwitch* ds = new DSwitch(i, j);
It seems you are creating pe_size DSwitches for each level (i.e., levelnum*pe_size DSwitches).  Wouldn't you have to divide the number of PEs by 2 for each level from the leaves up to the root?
Also, I have another question. Why did you set this comment in the code?
//TODO: Comment of Zhongyuan Zhao: This part is a heuristic exploration, we can set different connectivity o see the topology influence on DSwitch
What does it mean exactly? Isn't it the distribution network just a chubby tree? why do you say it is a heuristic exploration then?
Thank you very much,
Francisco Muñoz-Martinez

Dear Francisco:
This is not wrong, each level should have exact the same number of DS which is equal to pe_size. This is to support the fat-link topology which is able to multi-cast and unicast values at different scales. Decreasing the DS size at each level will make the unicast inefficiency. In MAERI paper, you can see the bandwidth of each level is increased by 2 times. This bandwidth is realized by keeping the same DS size in each level.
For ISA, yes, if that ISA work is promising, we will continue to develop the compiler. We have the separate simulator which is implemented using blue spec. The tool chain is able to generate both RTL and C++ code. but the simulator does not belong to mRNA code.
Yours, sincerely.
Best regards.
Zhongyuan.
----- 原始邮件 -----
发件人: "Francisco Muñoz Martinez"<EMAIL_ADDRESS>收件人: "georgia-tech-synergy-lab/mRNA"<EMAIL_ADDRESS>抄送: "Subscribed"<EMAIL_ADDRESS>发送时间: 星期三, 2019年 6 月 12日 下午 7:38:39
主题: [georgia-tech-synergy-lab/mRNA] DSNetwork generation (#4)
Hello,
I have come up with a question while I was looking at the code of the file DSNetwork.cpp.
In function DSNetwork::DSNetwork(double bw, int pe_size),
there is this piece of code:
`for(int i = 0; i < levelnum; i++) { //For each level
for(int j = 0; j < pe_size; j++) {  //For each PE. Would not be different depending on the level?

  DSwitch* ds = new DSwitch(i, j);

`
It seems you are creating pe_size DSwitches for each level (i.e., levelnum*pe_size DSwitches). Have you done this deliberately or is it a mistake?, wouldn't you have to divide the number of PEs by 2 for each level from the leaves up to the root?
Thank you very much,
Francisco Muñoz-Martinez
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/georgia-tech-synergy-lab/mRNA/issues/4

Hi @zzy82518996 ,
Thank you very much for your answer.
As it is shown in the Figure 4 of MAERI paper, I believed that the 2x bandwidth at each level up was get by means of  adding more wires  in the interconnection. Why do you use the same number of pe_size DSwitches instead of adding more wires?
Then if you do so, might we say that you are using the Distribution network shown in in your paper "Rethinking NoCs for Spatial Neural Network Accelerators"?
Thank you very much,
Francisco.

Hi Francisco:
In the figure, each node in the tree does not represent only single DS. Even though the author draws the DSNetwork like a tree, some DSes are overlapped in the up-level. You can think we are using the network inspired from that paper.
Yours, sincerely.
Best regards.
Zhongyuan.
----- 原始邮件 -----
发件人: "Francisco Muñoz Martinez"<EMAIL_ADDRESS>收件人: "georgia-tech-synergy-lab/mRNA"<EMAIL_ADDRESS>抄送: "zzy82518996"<EMAIL_ADDRESS>"Mention"<EMAIL_ADDRESS>发送时间: 星期三, 2019年 6 月 12日 下午 8:30:49
主题: Re: [georgia-tech-synergy-lab/mRNA] DSNetwork generation (#4)
Hi @zzy82518996 ,
Thank you very much for your answer.
As it is shown in the Figure 4 of MAERI paper, I believed that the 2x bandwidth at each level up was get by means of  adding more wires  in the interconnection. Why do you use the same number of pe_size DSwitches instead of adding more wires?
Then if you do so, might we say that you are using the Distribution network shown in in your paper "Rethinking NoCs for Spatial Neural Network Accelerators"?
Thank you very much,
Francisco.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/georgia-tech-synergy-lab/mRNA/issues/4#issuecomment-501251753

Hi @zzy82518996 ,
And what about the reduce network? It also increases the bandwidth of the ART each level. Are you really using a tree of ASwitches with wider links or is it the same topology as the DS?
Yours,
Francisco.

The first one. Reduce Network is different with DSNetwork. We use a tree for ASwitches and wider links to support higher bandwidth in the upper level.
----- 原始邮件 -----
发件人: "Francisco Muñoz Martinez"<EMAIL_ADDRESS>收件人: "georgia-tech-synergy-lab/mRNA"<EMAIL_ADDRESS>抄送: "zzy82518996"<EMAIL_ADDRESS>"Mention"<EMAIL_ADDRESS>发送时间: 星期三, 2019年 6 月 12日 下午 9:44:52
主题: Re: [georgia-tech-synergy-lab/mRNA] DSNetwork generation (#4)
Hi @zzy82518996 ,
And what about the reduce network? It also increases the bandwidth of the ART each level. Are you really using a tree of ASwitches with wider links or is it the same topology as the DS?
Yours,
Francisco.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/georgia-tech-synergy-lab/mRNA/issues/4#issuecomment-501279876

OK. Thank you very much @zzy82518996. Now everything is clear.
Best regards,
Francisco.


Adding state to streams

Not having a stateful operators proved pretty painful when writing prod GRPC systems.
The common use case is the first(ish) message in a Stream will contain auth data. Then instead of re-transmitting it with every message the server is expected to hold all that identity information for the duration of the Stream, using that data to supplement and/or override data incoming.
One could also think of a middleware acting in the same fashion.

Included a basic Buffer operator as there was a TODO mentioning it, and it roughly just seemed like Scanner but with an Option<T> instead of T.

Rejected


Completion rules

Can't find autocompletion rules for bash

@pomozoff I have no interest in doing this. If you want to convert this into a PR I will consider it


Outline not available for markdown files

code-server version: docker codercom/code-server:latest pulled March 15; can provide image hash if needed
OS Version: docker 18.09.3

Description
"Outline" view is not able to display headers/outline for markdown files.  Outline sidebar panel reads "There are no editors open that can provide outline information".  Markdown syntax highlighting is functioning, though.  Markdown file outline information is displayed in native vs code without any markdown extensions required.
Steps to Reproduce

Open a markdown file.
Verify that file type is recognized as "Markdown".
Expand "Outline" panel in side bar.
See message that "There are no editors open that can provide outline information."

Fixed by https://github.com/codercom/code-server/pull/472.


Fix typo : Change "Th" to "The"

It can be seen that there is a typo in the Readme.md file where the word "Th" should have been "The"

Refer to PR #290

Hey, i have fixed the issue and created a pull request.

fixed please check


Add support for searching

This is more of a heads up that we pushed out new API capabilities. https://github.com/Airtable/airtable.js/commit/08e6e76902168baa1881aacca35b5cea861d7679
We now accept a formula to filter records and an array of fields/direction pairs to sort by.
The limit is deprecated in favor of pageSize and maxRecords.

Nice! Had been hoping to see implement for a while, thanks!

I'd be happy to work on this as I currently need this feature set.
@syrnick I'm having trouble finding docs on the formulas for filtering.
Something like Order > 4 is easy enough, but right now I'm looking to find multiple records by ID with one lookup.
So something like id IS rec12343 OR id IS rec56789 throws an error -
{
  "error": {
    "type": "INVALID_FILTER_BY_FORMULA",
    "message": "The formula for filtering records is invalid: please check your formula text."
  }
}

Where can I find the docs that dictate a valid formula?

Here's a few pointers:
https://support.airtable.com/hc/en-us/articles/202576519-Guide-to-formula-lookup-count-and-rollup-fields
https://support.airtable.com/hc/en-us/articles/203255215-Formula-Field-Reference
You should also be able to do something like this
AND(RECORD_ID() = 'recXYZ1', RECORD_ID() = 'recXYZ2', RECORD_ID() = 'recXYZ3', RECORD_ID() = 'recXYZ4')

Depending on performance, we'll eventually add a separate endpoint for something like this or optimize this specific call or add a different operator to check record ID against an array.

Great, thanks @syrnick
Looks like the official JS lib only checks that the formula is in fact a string and depends on the API for validating the formula itself, so I'll take that same approach here.

This was done by @sbauch .


Define environment as nix flake and build in GitHub workflow

Thanks for an awesome package! The dependency setup was a bit fiddly and might put people off developing, so I've added a Nix flake which allows you to create a temporary environment with all dependencies pre-packaged. Building the package now becomes just a call to
nix build

I've also added a github workflow which builds the pdf automatically in the Github CI. It's currently always building the 2023 file, but that could be changed easily enough if you prefer.

By the way, I'm not familiar enough with this project yet to know if compiling just the 2023 pdf with default options is a useful thing to do. I'm experienced in Nix and latex however, so if you tell me what would be a useful set of pdfs to compile I could adapt this PR to build those instead / as well.

@charlesbaynham big <3 for the PR!
Default configs are definitely useful, although they represent only one possible configuration setup.
The first thing is LaTeX, which takes tens to hundreds of seconds to compile. Then the second thing is a sort of growing variety of possible setups, with which combinations setups grow exponentially.
Saying that, running one config to check that everything is fine should be well enough

In that case, I think this PR is ready - it builds a test pdf which should catch most bugs, and also defines a devShell so people can spin up a development environment with just one line and start contributing immediately.


[Issue1558] License updates

This fixes #1558.

SUCCESS
No test results found.
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR/658/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/incubator-trafficcontrol-PR-trafficops-test/617/

SUCCESS
No test results found.


update(cmake): Set our preferred Visual C++ runtime the same way ever…

…ywhere
If we're using CMake 3.15 or later, select our Visual C++ runtime using CMAKE_MSVC_RUNTIME_LIBRARY. Otherwise use FALCOSECURITY_LIBS_DEBUG_FLAGS and FALCOSECURITY_LIBS_RELEASE_FLAGS.

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area API-version

/area build

/area CI

/area driver-kmod

/area driver-bpf

/area driver-modern-bpf

/area libscap-engine-bpf

/area libscap-engine-gvisor

/area libscap-engine-kmod

/area libscap-engine-modern-bpf

/area libscap-engine-nodriver

/area libscap-engine-noop

/area libscap-engine-source-plugin

/area libscap-engine-savefile

/area libscap

/area libpman

/area libsinsp

/area tests

/area proposals

Does this PR require a change in the driver versions?

/version driver-API-version-major

/version driver-API-version-minor

/version driver-API-version-patch

/version driver-SCHEMA-version-major

/version driver-SCHEMA-version-minor

/version driver-SCHEMA-version-patch

What this PR does / why we need it:
This allows linking with code that uses any variant of the C runtime.
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?:

NONE

[APPROVALNOTIFIER] This PR is NOT APPROVED
This pull-request has been approved by: geraldcombs
Once this PR has been reviewed and has the lgtm label, please assign gnosek for approval. For more information see the Kubernetes Code Review Process.
The full list of commands accepted by this bot can be found here.

Needs approval from an approver in each of these files:

OWNERS

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: FedeDP, geraldcombs
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [FedeDP]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.15.0


Versions page not working due to a Jenkins build failure

Description:
Build #10103 is failing and throws the following error:
+ mkdocs build -c
INFO     -  Cleaning site directory
INFO     -  Building documentation to directory: /home/jenkins/workspace/docs/docs-apim/en/site
Traceback (most recent call last):
  File "/root/.pyenv/versions/3.8.10/bin/mkdocs", line 8, in <module>
    sys.exit(cli())
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/click/core.py", line 1157, in __call__
    return self.main(*args, **kwargs)
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/click/core.py", line 1078, in main
    rv = self.invoke(ctx)
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/click/core.py", line 1688, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/click/core.py", line 1434, in invoke
    return ctx.invoke(self.callback, **ctx.params)
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/click/core.py", line 783, in invoke
    return __callback(*args, **kwargs)
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/mkdocs/__main__.py", line 250, in build_command
    build.build(cfg, dirty=not clean)
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/mkdocs/commands/build.py", line 320, in build
    _build_theme_template(template, env, files, config, nav)
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/mkdocs/commands/build.py", line 118, in _build_theme_template
    output = _build_template(template_name, template, files, config, nav)
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/mkdocs/commands/build.py", line 97, in _build_template
    output = template.render(context)
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/jinja2/environment.py", line 1301, in render
    self.environment.handle_exception()
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/jinja2/environment.py", line 936, in handle_exception
    raise rewrite_traceback_stack(source=source)
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/material/404.html", line 4, in top-level template code
    {% extends "main.html" %}
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/material/main.html", line 4, in top-level template code
    {% extends "base.html" %}
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/material/base.html", line 192, in top-level template code
    {% block footer %}
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/material/base.html", line 193, in block 'footer'
    {% include "partials/footer.html" %}
  File "/home/jenkins/workspace/docs/docs-apim/en/theme/material/partials/footer.html", line 74, in top-level template code
    {% block social %}
  File "/home/jenkins/workspace/docs/docs-apim/en/theme/material/partials/footer.html", line 75, in block 'social'
    {% include "partials/social.html" %}
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/material/partials/social.html", line 16, in top-level template code
    {% include ".icons/" ~ social.icon ~ ".svg" %}
  File "/root/.pyenv/versions/3.8.10/lib/python3.8/site-packages/jinja2/loaders.py", line 218, in get_source
    raise TemplateNotFound(template)
jinja2.exceptions.TemplateNotFound: .icons/.svg
Build step 'Execute shell' marked build as failure

Started calculate disk usage of build
Finished Calculation of disk usage of build in 0 seconds
Started calculate disk usage of workspace
Finished Calculation of disk usage of workspace in 0 seconds
Finished: FAILURE

Could locally build the page successfully by updating footer.html and header.html with the latest changes. Need to do the customizations specific to the documentation.

Hi all,
Closing this since the issue was resolved by PR https://github.com/wso2/docs-apim/pull/7775
Thanks!
Dulith


How to send LINE message from Java code?

Now I am able to login LINE and display my username and profile picture when I click the "Try API" button. Next, I want to display my friend lists using apiclient.getFriends(). Any sample code?
Anyone knows how to create a Java API to send LINE message to friends?
I got the skeleton code but I do not know how to implement it.
class LineMsg {
    public void SendMsg(String username, String msg) {
        //username is the friend name that I want to send a message to
    }
}

Then I have a main method.
public void main(String[] args) {
    LineMsg linemsg = new LineMsg();
    linemsg.SendMsg("username of LINE friend","Hello World!")
}

Then I need to zip up as JAR file so my friend can import the JAR file and send LINE message to contacts. The purpose of doing this is to replace the SMS function which have high cost. So we want to send to our clients using the LINE method by Java code. If anyone knows how to implement, please give some ideas. Thanks.

The FRIEND LIST permission is not available to public at the moment, the developer @GuitarSucks  said this.
What else can I do other than login the LINE and display the username and profile picture, and other profile details?

Hi
Unfortunately we don't provide FRIENDS permission due to policy issues. So currently its only display name , profile picture and status message.
In the future we are trying to to provide FRIENDS which match our policy, but could be close to FB way.
There will be also a new API in the future to provide some other information from user, If the user approves.
Regards

Noted and thanks.


Remove category questions

Issue #:
Description of changes:

Change display to false for category type questions

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of the met-public license (Apache 2.0).

Codecov Report

Merging #2297 (32c587c) into main (f500a93) will decrease coverage by 0.01%.
Report is 1 commits behind head on main.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #2297      +/-   ##
==========================================
- Coverage   68.58%   68.58%   -0.01%
==========================================
  Files         472      472
  Lines       15579    15580       +1
  Branches     1187     1188       +1
==========================================
  Hits        10685    10685
- Misses       4688     4689       +1
  Partials      206      206

Flag
Coverage Δ

metweb
61.52% <ø> (-0.01%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes


Fix same-page haslinks

href="/docs/datasets/quickstart#audio" implicitly goes to href="/docs/datasets/{$LATEST_STABLE_VERSION}/quickstart#audio". Therefore, https://huggingface.co/docs/datasets/quickstart#audio #audio hashlink does not work since the new docs were not added to v2.3.2 (LATEST_STABLE_VERSION)
to preserve the version, it should be just href="#audio", which will implicilty go to curren_page + #audio element

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


Cleaning up setup.py and dynamically loading constants

It would be great to see this set of wrappers submitted to PyPi, but there's some work that needs to be done. I've tried to clean up some of the code and have added an RST file generated from the markdown (using pandoc README.md -o README.rst).

thanks for the pull request, I am not very familiar with Pypi so appreciate the help.


Add values schema to Kong

Towards https://github.com/giantswarm/roadmap/issues/136 with @oponder
This PR adds values.schema.json generated from the current values.yaml using https://github.com/karuppiah7890/helm-schema-gen
It is a good starting point and can be further refined by those that know more about the specifics of the values. (For example if we know that provider must be aws|azure|kvm, we can actually enforce that as well using the 'enum' property: https://json-schema.org/understanding-json-schema/reference/generic.html#enumerated-values)
Here's also some more general info: https://www.arthurkoziel.com/validate-helm-chart-values-with-json-schemas/
I tested in qmyk0 in asgard and got the following error:
(⎈ |giantswarm-asgard:default)➜  kubectl-gs git:(app-validate) ./kubectl-gs validate apps -n qmyk0 kong-app -f=/Users/chiaracokieng/Documents/GitHub/kong-app/helm/kong-app/values.schema.json -o report

Validated 1 app across 1 namespace

qmyk0 [1 app, 1 error]
  kong-app:
    ingressController.admissionWebhook.port - Invalid type. Expected: string, given: integer - 8080

@giantswarm/app-squad-kong In line 328 here https://github.com/giantswarm/kong-app/blob/master/helm/kong-app/values.yaml, it says. Should 8080 be inside " "?
admissionWebhook:
    enabled: false
    failurePolicy: Fail
    port: "8080"

I used the currently in progress kubectl-gs validate apps command to help with the verifying current values : https://github.com/giantswarm/kubectl-gs/pull/196
Here's also a slack thread with a video about how that command works: https://gigantic.slack.com/archives/CRZN9GLJW/p1603092812002400

hi @yasn77 can you help me? Test on circleCI failed. I think this is the relevant message?
==> Linting helm/kong-app
[ERROR] templates/: values don't meet the specifications of the schema(s) in the following chart(s):
kong-app:
- proxy.stream: Invalid type. Expected: object, given: array

Sorry, this shouldn't be changed at all...

merging even if kat test failed w @ubergesundheit 's approval.

@cokiengchiara just seen your comment... glad you got it sorted


Update ro_ro

Hi Gabriel,
I update with new translated messages for the new version 2.6.

Thank you for your contribution!


version 2.0.1

Coverage remained the same at 96.774% when pulling 83e6cad57ab01ba86d52b2a6518c3eae6193bf5a on dev into 4f0c219592b2c131a6378707178507da97425109 on master.


nmdc-schema style guide/best practices

I think this has been addressed here: https://github.com/microbiomedata/nmdc-schema/blob/2f7797be782e5b7124b2588bc6b312e903ea0a1b/CONTRIBUTING.md#modeling-best-practice


Purge behavior when calling cephadm-purge-cluster.yml with --limit

The cephadm-purge-cluster.yml pre-purge checks fail in a not-so-good way when one calls the playbook with a --limit to a subset of the hosts in the inventory (e.g., those hosts that are in the cluster).
As an alternative, the plays could have hosts: all set and the following added to each task to achieve the same behavior in a compatible way:
      delegate_to: localhost
      run_once: true

Otherwise, once sees skipping: no hosts matched on the applicable plays. The "confirm whether user really wants to purge the cluster" play even prompts the user to see if they want to purge the cluster, but continues to purge the cluster no matter what they type.
If this sounds like an acceptable solution I can make a PR.

that shouldn't be an issue [1].
By the way, this is what we do in the CI. If you look at the inventory we use here [2], you can see we don't define a host localhost, and we use it here [3].
See a job in the CI :
el8-functional run-test: commands[10] | ansible-playbook -vv -i /home/jenkins-build/build/workspace/cephadm-ansible-prs-el8-functional/tests/functional/hosts /home/jenkins-build/build/workspace/cephadm-ansible-prs-el8-functional/cephadm-purge-cluster.yml -e ireallymeanit=yes -e fsid=4217f198-b8b7-11eb-941d-5254004b7a69
[1517415] /home/jenkins-build/build/workspace/cephadm-ansible-prs-el8-functional/tests/functional$ /tmp/venv.sU96b3Nguc/el8-functional/bin/ansible-playbook -vv -i hosts /home/jenkins-build/build/workspace/cephadm-ansible-prs-el8-functional/cephadm-purge-cluster.yml -e ireallymeanit=yes -e fsid=4217f198-b8b7-11eb-941d-5254004b7a69
ansible-playbook 2.9.27
  config file = /home/jenkins-build/build/workspace/cephadm-ansible-prs-el8-functional/ansible.cfg
  configured module search path = ['/home/jenkins-build/build/workspace/cephadm-ansible-prs-el8-functional/library']
  ansible python module location = /tmp/venv.sU96b3Nguc/el8-functional/lib/python3.9/site-packages/ansible
  executable location = /tmp/venv.sU96b3Nguc/el8-functional/bin/ansible-playbook
  python version = 3.9.7 (default, Sep 21 2021, 00:13:39) [GCC 8.5.0 20210514 (Red Hat 8.5.0-3)]
Using /home/jenkins-build/build/workspace/cephadm-ansible-prs-el8-functional/ansible.cfg as config file
Skipping callback 'actionable', as we already have a stdout callback.
Skipping callback 'counter_enabled', as we already have a stdout callback.
Skipping callback 'debug', as we already have a stdout callback.
Skipping callback 'dense', as we already have a stdout callback.
Skipping callback 'dense', as we already have a stdout callback.
Skipping callback 'full_skip', as we already have a stdout callback.
Skipping callback 'json', as we already have a stdout callback.
Skipping callback 'minimal', as we already have a stdout callback.
Skipping callback 'null', as we already have a stdout callback.
Skipping callback 'oneline', as we already have a stdout callback.
Skipping callback 'selective', as we already have a stdout callback.
Skipping callback 'skippy', as we already have a stdout callback.
Skipping callback 'stderr', as we already have a stdout callback.
Skipping callback 'unixy', as we already have a stdout callback.
Skipping callback 'yaml', as we already have a stdout callback.

PLAYBOOK: cephadm-purge-cluster.yml ********************************************
7 plays in /home/jenkins-build/build/workspace/cephadm-ansible-prs-el8-functional/cephadm-purge-cluster.yml

PLAY [check local prerequisites are in place] **********************************
META: ran handlers

TASK [fail if fsid was not provided] *******************************************
task path: /home/jenkins-build/build/workspace/cephadm-ansible-prs-el8-functional/cephadm-purge-cluster.yml:22
Friday 23 September 2022  09:00:29 +0000 (0:00:00.015)       0:00:00.015 ******
skipping: [localhost] => changed=false
  skip_reason: Conditional result was False

TASK [fail if admin group doesn't exist or is empty] ***************************
task path: /home/jenkins-build/build/workspace/cephadm-ansible-prs-el8-functional/cephadm-purge-cluster.yml:29
Friday 23 September 2022  09:00:29 +0000 (0:00:00.023)       0:00:00.038 ******
skipping: [localhost] => changed=false
  skip_reason: Conditional result was False

the issue with hosts: all is when users use the same inventory for multiple environments.
[1] https://docs.ansible.com/ansible/latest/inventory/implicit_localhost.html#implicit-localhost
[2] https://github.com/ceph/cephadm-ansible/blob/devel/tests/functional/hosts
[3] https://github.com/ceph/cephadm-ansible/blob/v2.9.0/tox.ini#L90
[4] https://2.jenkins.ceph.com/job/cephadm-ansible-prs-el8-functional/168/consoleFull

@guits Thanks for the detailed reply. I did some additional testing and the issue may be my fault.
I am using a single inventory file for ceph servers and other servers that are not part of the cluster. I am not sure if that is the same as "the issue with hosts: all is when users use the same inventory for multiple environments." or if that is a different issue.
When deleting the cluster I tried to limit it to my ceph group:
ansible-playbook -i inventory -l ceph cephadm-ansible/cephadm-purge-cluster.yml -e fsid=bcf88dfa-3cf0-11ed-b5a4-b19cff2a0492

This has the unfortunate side effect of skipping any plays that are limited to localhost (edited with a debug message so as not to really delete my cluster):
$ ansible-playbook -i inventory -l ceph cephadm-ansible/cephadm-purge-cluster.yml -e fsid=bcf88dfa-3cf0-11ed-b5a4-b19cff2a0492

PLAY [check local prerequisites are in place] *************************************************************************************************
skipping: no hosts matched

PLAY [check keyring is present on the admin host] *********************************************************************************************

TASK [check /etc/ceph/ceph.admin.client.keyring] **********************************************************************************************
ok: [loyal_mouse]

TASK [fail if /etc/ceph/admin.client.keyring is not present] **********************************************************************************
skipping: [loyal_mouse]

PLAY [check cluster hosts have cephadm and the required fsid bcf88dfa-3cf0-11ed-b5a4-b19cff2a0492] ********************************************

TASK [check cephadm binary is available] ******************************************************************************************************
ok: [loyal_mouse]
ok: [tender_poodle]
ok: [tidy_piglet]

TASK [fail if cephadm is not available] *******************************************************************************************************
skipping: [loyal_mouse]
skipping: [tender_poodle]
skipping: [tidy_piglet]

TASK [check fsid directory given is valid across the cluster] *********************************************************************************
ok: [tender_poodle]
ok: [tidy_piglet]
ok: [loyal_mouse]

TASK [fail if the fsid directory is missing] **************************************************************************************************
skipping: [loyal_mouse]
skipping: [tender_poodle]
skipping: [tidy_piglet]

Are you sure you want to purge the cluster with fsid=bcf88dfa-3cf0-11ed-b5a4-b19cff2a0492 ?
 [no]: no

PLAY [confirm whether user really wants to purge the cluster] *********************************************************************************
skipping: no hosts matched

PLAY [debug message] **************************************************************************************************************************

TASK [debug] **********************************************************************************************************************************
ok: [loyal_mouse] => {
    "msg": "continuing to purge the cluster anyways..."
}
ok: [tender_poodle] => {
    "msg": "continuing to purge the cluster anyways..."
}
ok: [tidy_piglet] => {
    "msg": "continuing to purge the cluster anyways..."
}

PLAY RECAP ************************************************************************************************************************************
loyal_mouse                : ok=4    changed=0    unreachable=0    failed=0    skipped=3    rescued=0    ignored=0
tender_poodle              : ok=3    changed=0    unreachable=0    failed=0    skipped=2    rescued=0    ignored=0
tidy_piglet                : ok=3    changed=0    unreachable=0    failed=0    skipped=2    rescued=0    ignored=0

Without the -l ceph, it would fail properly if I type "no" at the prompt, but I don't get that far because it dies on the fatal: [non_ceph_host]: FAILED! => {"changed": false, "msg": "The cephadm binary is missing on non_ceph_host. To purge the cluster you must have cephadm installed\non ALL ceph hosts. Install manually or use the preflight playbook.\n"} message.
If the goal is to only support an inventory file dedicated to the ceph cluster I understand, and am happy to close this out / find a workaround on our side.

After a better look at the playbook, what I wanted to avoid with hosts: localhost is actually counter-productive when using a 'shared inventory' with --limit. By the way can see hosts: all is used for multiple plays:
https://github.com/ceph/cephadm-ansible/blob/devel/cephadm-purge-cluster.yml#L56
https://github.com/ceph/cephadm-ansible/blob/devel/cephadm-purge-cluster.yml#L132
https://github.com/ceph/cephadm-ansible/blob/devel/cephadm-purge-cluster.yml#L147
I think the approach you suggested makes sense in the end.
If you want to send a PR... 🙂


Dionae now bleed sap, and this can be used to make syrup.

About the PR
I switched the water in their blood for a new sap reagent, makes more sense for a tree.
This sap can be boiled to make syrup, a very sweet and helpful substance for feeding the crew or making sticky traps. Put it on pancakes, inject it into starving people, or toss it on the floor while running from sec.
People definitely won't make nymph sap factories in the freezer!
 Which may be mostly water, but is still a different substance.
Also water is slippery and having slippery blood is annoying for enemies at best and gets you/your friends killed at worst.
Media
I'm rather tired and want to get feedback on this by the time I wake up, I'll add a quick showcase in the morning.

[ ] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
:cl:

add: Added sap
add: Added syrup, it can be made by boiling sap.
tweak: Dionae now bleed sap instead of water

centrifuge the sap into water and sugar?


Add more @class attributes on spans and divs created by XSLT for XHTML output

The XSLT stylesheets for XHTML output create in some places artificial  or
 elements. For example in this XSL "DITA-OT/plugins/org.dita.xhtml/xsl/xslhtml/rel-links.xsl" there are places like:

<xsl:call-template name="makelink"/>

Ideally all these extra generated spans and divs created in the XHTML XSLT code should have a @class attribute set to them so that they can be styled using CSS.

Thinking about this one - I tend to agree, I like having the extra classes available to do something with as needed. That said, I've had (one or two) people complain that HTML5 output should be clean / simple / not cluttered, and have asked me to remove extra classes that were not explicitly needed.
Of course, those same people wouldn't like extra
 or  elements that do not serve a clear purpose anyway. Those of us familiar with the mapping from DITA can often relate those elements to the original source, but if you only care about the HTML markup, it tends to look more cluttered than needed. That's probably not something we can easily solve, so it's likely not worth worrying about here.

See #2239 for related changes to HTML5 table markup (released w/ v2.3).
Perhaps that same approach could be applied elsewhere in HTML5 output to provide BEM-style CSS classes.
IIRC, the clutter issue can be addressed with the args.xhtml.classattr parameter that specifies whether to include the DITA class ancestry in the generated CSS classes, so those who prefer clean markup can have that, while still supporting use cases that require more granular class information.


Location suggestion does not work properly on Android with Firefox

Describe the bug
Location suggestion does not work properly on mobile.
To Reproduce
Steps to reproduce the behavior:

Go to 'home' via smartphone
Click on 'the location textfield'
Type a zipcode like ‘63110’ or a location like ‘Frankfurt’
See error

Expected behavior
Matching locations should be suggested for the incompleted as well as for the completed input.
Screenshots

Desktop (please complete the following information):
/
Smartphone (please complete the following information):
Device: OnePkus 6T
OS: Android 10
Browser Firefox
Version 86.9.0

Additional context
This bug was already fixed in #51, but it still does not work in Firefox.

I can't reproduce this bug anymore. Can someone verify that this bug still exists.

I can't reproduce this bug anymore. Can someone verify that this bug still exists.

I couldn't reproduce this bug using a smartphone or the mobile view on desktop.
The bug no longer seems to exist and the issue can be closed.

I couldn't reproduce this bug using a smartphone or the mobile view on desktop.
The bug no longer seems to exist and the issue can be closed.


Nodes with multiple inputs should receive dictionary of DataContainers as input

There is a major issue with nodes with multiple inputs, it seems that the order of the inputs is not guaranteed, which causes problems when the order matters.
The node function signature should be like this
def SINE(inputs:dict[str, DataContainer], params:dict)-> Union(DataContainer, dict[str, DataContainer]):

Please also update node_styleguide.md for contributors (in the nodes repo)
Support for multiple outputs is covered in this issue https://github.com/flojoy-io/studio/issues/524

Had a discussion with @JeffDotPng and @itsjoeoui for a better design on how multiple inputs are supported. Here's the new approach we decided to implement.

The node functions will define all its inputs and params as regular python function parameters with proper type hint.
All the function parameters that are of DataContainer type are inputs
All the function parameters that are not DataContainer type are params
For example, a node with two inputs a and b, and two parameters p and q, will be written as

def ADD(a: DataContainer, b: DataContainer, p: int, q: str):
  ...
  ...function body
  ...

The @flojoy decorator has to properly propagate the arguments to all the function parameters by reading the signature of the target function. It has to utilize the type hints to distinguish inputs from node params.

Thanks Asif for documenting all these, can't wait for us to finish this one. 🚀


[GEN-356] ref(profiler): use di for system profile

Introducing the OpenMetadata service specification.
Main things to look out for:

Removal of the import_x_class in favor of a central BaseSpec.get_for_source which will provide the centralized repository for all the source's interfaces.
Breaking change documentation
Adding the system_metrics_computer_class field in the SQAProfilerInterface
New profiler implementations for redshift, biquery, snowflake to accomodate for the new system metric source
use source classes that can be overridden in system profiles
use a manifest class instead of factory to specify which class to resolve for connectors
example usage can be seen in redshift and snowflake

TODO

[x] Implement the BigQuery system metric source class
[ ] run e2e tests (link to GA)

Describe your changes:
Fixes

I worked on ... because ...

Type of change:

[ ] Bug fix
[ ] Improvement
[ ] New feature
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] My PR title is Fixes <issue-number>: <short explanation>
[ ] I have commented on my code, particularly in hard-to-understand areas.
[ ] For JSON Schema changes: I updated the migration scripts or explained why it is not needed.

@pmbrull re __init
I changed how the constructors are called to remove this. LMK if this addresses your comment.


CASM-3723 Updating prodmgr version

Summary and Scope

Relates to: CASM-3723

Issue Type
Updating prodmgr version to be added as part of CSM 1.5
Prerequisites

[x] I have included documentation in my PR (or it is not required)
[x] I have tested this by building an image using this branch HEAD, or this is does not pertain to the node-images pipeline.
[x] I tested this on internal system or this pertains to the node-images pipeline (if yes and not build related, please include results or a description of the test)
[x] I tested this on a vshasta system or this pertains to the node-images pipeline (if yes and not build related, please include results or a description of the test)

/backport lts/csm-1.5

/backport main


feat: Add id prop to Text, TouchableWithoutFeedback and View components

Summary

Changelog

[CATEGORY] [TYPE] - Message
Test Plan

Tracked on https://github.com/facebook/react-native/pull/ 34522/


Data race when HTTP server boots

I was trying to track down a bug where infrequently connections to the PIP endpoint would return EOF in my HTTP Client, so I booted the server up with -race to see if this might be a server error. I don't think it is, but I did spot this in the logs as the server boots.
[spatial] app.go:179: Register /data/ handler
[spatial] app.go:204: Register /api/point-in-polygon handler
==================
WARNING: DATA RACE
Write at 0x00c0001946d0 by goroutine 8:
  github.com/whosonfirst/go-whosonfirst-spatial-www/server.(*HTTPServerApplication).RunWithFlagSet.func1()
      /Users/tom/projects/wof/go-whosonfirst-spatial-www/server/app.go:124 +0x84

Previous write at 0x00c0001946d0 by main goroutine:
  github.com/whosonfirst/go-whosonfirst-spatial-www/server.(*HTTPServerApplication).RunWithFlagSet()
      /Users/tom/projects/wof/go-whosonfirst-spatial-www/server/app.go:161 +0xd74
  github.com/whosonfirst/go-whosonfirst-spatial-www/server.(*HTTPServerApplication).Run()
      /Users/tom/projects/wof/go-whosonfirst-spatial-www/server/app.go:69 +0x1d0
  main.main()
      /Users/tom/projects/wof/go-whosonfirst-spatial-www/cmd/server/main.go:24 +0x4c

Goroutine 8 (running) created at:
  github.com/whosonfirst/go-whosonfirst-spatial-www/server.(*HTTPServerApplication).RunWithFlagSet()
      /Users/tom/projects/wof/go-whosonfirst-spatial-www/server/app.go:122 +0x9bc
  github.com/whosonfirst/go-whosonfirst-spatial-www/server.(*HTTPServerApplication).Run()
      /Users/tom/projects/wof/go-whosonfirst-spatial-www/server/app.go:69 +0x1d0
  main.main()
      /Users/tom/projects/wof/go-whosonfirst-spatial-www/cmd/server/main.go:24 +0x4c
==================
2022/08/12 08:58:19 index 101874113#:0 [-0.26, -0.15]x[52.88, 52.93]
2022/08/12 08:58:19 index<PHONE_NUMBER>#:0 [-2.49, -2.25]x[53.42, 53.54]
2022/08/12 08:58:19 index 101872113#:0 [-1.97, -1.85]x[53.28, 53.36]
2022/08/12 08:58:19 index 101853557#:0 [-0.64, -0.58]x[51.68, 51.73]
2022/08/12 08:58:19 index 101854787#:0 [-0.39, -0.33]x[51.79, 51.84]
[etc]

Okay, the first thing I can see is that the code (read: me) is wrapping to call to index data in an unnecessary Go routine:
https://github.com/whosonfirst/go-whosonfirst-spatial-www/blob/main/server/app.go#L122-L129
It is unnecessary because the code it is calling invokes its own Go routine:
https://github.com/whosonfirst/go-whosonfirst-spatial-www/blob/6364630109fc1caff7ca5b2adc845c2ce5c76f05/vendor/github.com/whosonfirst/go-whosonfirst-spatial/app/app.go#L76
So the code starts by saying: Spatial application, go an index these paths in the background.
And a little further down it says: Create a data (HTTP) handler using the spatial application's spatial database:
https://github.com/whosonfirst/go-whosonfirst-spatial-www/blob/main/server/app.go#L161
If you look carefully at the data handler you'll see it doesn't know anything about "spatial applications" but only about readers; specifically the spatial application implements the reader.Reader interface:
https://github.com/whosonfirst/go-whosonfirst-spatial-www/blob/6364630109fc1caff7ca5b2adc845c2ce5c76f05/http/data.go#L12
https://github.com/whosonfirst/go-whosonfirst-spatial-rtree/blob/main/database.go#L589
That part is important. The data handler doesn't know anything about the spatial application or the spatial application's iterator which is indexing data in the background, notably the IsIndexing() method:
https://github.com/whosonfirst/go-whosonfirst-spatial-www/blob/6364630109fc1caff7ca5b2adc845c2ce5c76f05/vendor/github.com/whosonfirst/go-whosonfirst-spatial/app/app.go#L105
So it's very possible for the data handler to start getting requests for records that haven't been indexed yet, for example with a really big repo like GB postalcodes.
One bad habit that I've been trying to address going forward is not wrapping other people's errors in something that will help track down errors. Not doing this often leads to getting an error that just says io.EOF which while self-explanatory is often unhelpful. I mention that because both the (HTTP) data handler and the go-whosonfirst-spatial-rtree implementation of the Reader interface suffer from this:
https://github.com/whosonfirst/go-whosonfirst-spatial-rtree/blob/main/database.go#L620-L646
At the end of all of this my immediate hunch is that the data race is being triggered by the nested Go routines and the io.EOF error because... I'm not sure because it doesn't seem like the kind of error to be triggered even if a record hasn't been indexed.
So, I will create a new branch that does the following:

Removes the unnecessary Go routine for indexing records
Create a middleware (HTTP) handler that wraps the data handler and ensures that indexing is complete and return a suitable 5XX error if not

Hopefully one or both of those things will address the issues you're seeing.

Can you try the issue-22 branch and see if that helps. For example:
$> go run -mod vendor cmd/server/main.go -spatial-database-uri rtree://  /usr/local/data/sfomuseum-data-whosonfirst/

> curl -v http://localhost:8080/api/point-in-polygon -d '{"latitude":37.62623125247335,"longitude":-122.28623871760982}'
*   Trying <IP_ADDRESS>:8080...
* Connected to localhost (<IP_ADDRESS>) port 8080 (#0)
> POST /api/point-in-polygon HTTP/1.1
> Host: localhost:8080
> User-Agent: curl/7.79.1
> Accept: */*
> Content-Length: 62
> Content-Type: application/x-www-form-urlencoded
>
* Mark bundle as not supporting multiuse
< HTTP/1.1 503 Service Unavailable
< Content-Type: text/plain; charset=utf-8
< X-Content-Type-Options: nosniff
< Date: Fri, 12 Aug 2022 18:12:08 GMT
< Content-Length: 17
<
Indexing records
* Connection #0 to host localhost left intact


Request for review

We did the same mistake again and accidentally merged with the master.
We fixed the validator error
We filled the Readme.md with more info.

Thank you,
we fixed the icons on the right of the navbar. They are 3 now.
Main content - It is on a different divs. The layout is as follows:

  with divs for the icons

Video has its own div

  more stuff here

Sorry we do not understand what you want us to fix.


CUDA: Support NVRTC when using the ctypes binding

The NVIDIA CUDA Python bindings are required to link C/C++ code with Numba kernels, because it compiles them with NVRTC. This means that full float16 support also needs those bindings. If we supported NVRTC with the ctypes bindings, then float16 support would always be available.
Previously this was not implemented because it might have been tricky to locate NVRTC, as it wasn't present in cudatoolkit conda packages. However, from CUDA 12 onwards, the cudatoolkit (or cuda-toolkit?) packages on conda-forge will have it available, so locating it should be less of an issue.

cc @testhound @pentschev for info / visibility

NVRTC now works using the ctypes binding.


What is the different between valid-ours.h5 and valid-example.h5

Hi, I notice that valid-example.h5 is better than valid-ours.h5. Would you please tell me the difference between these two results? Thank you.

Moreover. I found that the released results on validation set is slightly worse than the results on the TEST set:

method
Head
Shoulder
Elbow
Wrist
Hip
Knee
Ankle
Mean

valid-example
95.80
94.21
87.40
82.75
86.03
81.83
78.32
86.76

valid-ours
95.94
94.68
88.53
83.38
87.48
83.09
79.05
87.56

Is the released model different against the model in your arxiv paper? Thanks in advance!

valid-ours.h5 is currently a bit outdated as well as the released model. With that said, I generally found validation performance to be around 2% worse than test set performance.
valid-example.h5 is the file that gets written to when running the validation demo code, so if someone runs with a different model it was just a way to distinguish it from our baseline performance. I just put an arbitrary set of predictions as filler there so if the evaluation code was run it would show different curves. I didn't think much about it at the time and in hindsight I realize that might be a bit confusing, sorry about that!

@anewell Can you please clarify which model architecture was used to generate valid-ours.h5? Was it the 8-stack hourglass used to produce Table 2 in your paper? I wish to use these results as a reference point during development.

Hi, valid-ours.h5 is from an old version of the model, and does not correspond to the output of the 8-stack network. Sorry for any confusion that might cause. You can download the 8-stack model though and run the evaluation to see how it does.

Thanks for your reply. I've found a small bug in the evaluation code (see https://github.com/anewell/pose-hg-demo/pull/14), but other than that I am able to generate validation set predictions just fine.
Would you consider accepting a PR to add the validation set predictions into the preds/ folder (eg as valid-hg8.h5 or perhaps replacing valid-ours.h5)? It would be very helpful to have an "official" validation prediction set for your highly influential model, and would clear up confusion for people who wrongly believe that valid-ours.h5 represents your peak performance.


Add exception handling and logging for listeners

Reason for Change
The ThreadPoolExecutor will swallow exceptions:
https://github.com/ruby-concurrency/concurrent-ruby/blob/master/lib/concurrent/executor/ruby_thread_pool_executor.rb#L347-L352
And because we don't set up a logger in Concurrent even the log DEBUG is silent because the default logger level is fatal.
Rather then mess with the concurrent logger or leave this PR adds basic exception handling and logging to the block that is added to the executor.
Risks

Low

Checklist
Put an x in the boxes that apply.
You can also fill these out after creating the PR.

[x] I have NOT linked to any Jira tickets.
[x] I have linked to all relevant reference issues or work requests
[x] I have added tests that prove my fix is effective or that my feature works
[x] I have added or updated necessary documentation (if appropriate)
[x] Any dependent changes have been merged and published in downstream

Recommended Reviewers
@fastly/billing

Example output:
E, [2018-07-25T16:47:34.494003 #98464] ERROR -- : WTF (RuntimeError)
/Users/leklund/projects/nsq_fu/workit.rb:7:in `call'
/Users/leklund/projects/fastly_nsq/lib/fastly_nsq/listener.rb:129:in `call'
/Users/leklund/projects/fastly_nsq/lib/fastly_nsq/feeder.rb:40:in `block in push'
/Users/leklund/.rvm/gems/ruby-2.5.0/gems/concurrent-ruby-1.0.5/lib/concurrent/executor/ruby_thread_pool_executor.rb:348:in `run_task'
/Users/leklund/.rvm/gems/ruby-2.5.0/gems/concurrent-ruby-1.0.5/lib/concurrent/executor/ruby_thread_pool_executor.rb:337:in `block (3 levels) in create_worker'
/Users/leklund/.rvm/gems/ruby-2.5.0/gems/concurrent-ruby-1.0.5/lib/concurrent/executor/ruby_thread_pool_executor.rb:320:in `loop'
/Users/leklund/.rvm/gems/ruby-2.5.0/gems/concurrent-ruby-1.0.5/lib/concurrent/executor/ruby_thread_pool_executor.rb:320:in `block (2 levels) in create_worker'
/Users/leklund/.rvm/gems/ruby-2.5.0/gems/concurrent-ruby-1.0.5/lib/concurrent/executor/ruby_thread_pool_executor.rb:319:in `catch'
/Users/leklund/.rvm/gems/ruby-2.5.0/gems/concurrent-ruby-1.0.5/lib/concurrent/executor/ruby_thread_pool_executor.rb:319:in `block in create_worker'


Create install and configure recipes

Commit 7be35e2 (the meat of #457) has been cherry-picked into this branch.


[bug] Can't close MediaStream (Camera Indicator stll active)

Describe the bug
The camera Indicator is still active even camera stream is stop(),
Example
Here is a simple example for reproducing the bug I found,
Repo Link: https://github.com/xxidbr9/camera-issue-tauri
I have mac and windows device and all of the device is facing the same issue, the problem is only when I run it on Tauri, but I have no problem when running it as a normal Web app
Reproduction

Clone the repo
and try web and Tauri version to compare

Expected behavior
Will turn off camera indicator when use
// const stream = navigator.mediaDevices.getUserMedia({ video: true, audio: true })
stream.getTracks().forEach(track => track.stop());

Platform and versions
› OS: Mac OS 13.2.1 X64
  › Node.js: 16.19.0
  › npm: 8.19.3
  › pnpm: 8.1.0
  › yarn: 1.22.19
  › rustup: 1.25.1
  › rustc: 1.66.1
  › cargo: 1.66.1
  › Rust toolchain: stable-aarch64-apple-darwin

Stack trace
No response
Additional context
No response

i am not able to see any mediadevices under navigator running tauri app in ubuntu22.04. so unable to open usb camera using tauri app.

@learncodingforweb webrtc is not available in webkitgtk yet: https://github.com/tauri-apps/tauri/issues/5370 / https://github.com/tauri-apps/wry/issues/85 and therefore your issue is unrelated to this one :)


tail: after running tests, input is no longer shown in console

After running the tail tests with
$ cargo test --features "tail" --no-default-features

the console no longer shows what I type (I'm using bash in KDE's Konsole).

Can confirm that this is happening. Another strange thing happens too: when I hit ^C, it gives me 2 empty lines between prompts:
# before tail tests
[terts@deskspacex coreutils]$ ^C
[terts@deskspacex coreutils]$
# after tail tests
[terts@deskspacex coreutils]$ ^C

[terts@deskspacex coreutils]$

With fish, it all works fine. It also works fine with nextest in bash if you want a quick workaround.

This SO answer has a reasonable hypothesis: https://askubuntu.com/a/172747
After running the tail tests stty -a is indeed showing -echo.

The offending commit based on git bisect seems to be: https://github.com/uutils/coreutils/commit/1725151ef827f07009c844bbda5ba6de7ab069d9

It also works on zsh, it seems to be specifically bash
After some testing dummy.kill().unwrap(); is definitively the culprit. Killing sh manually with exit instead of sending SIGKILL works fine.
Tbf I don't know what the test does but maybe the kill method could be replaced by sending a more gentle signal. SIGTERM and SIGQUIT didn't work but somehow SIGUSR1 make sh quit gracefully and bash stty is normal after the test.
// This doesn't work :
// dummy.kill().unwrap();
// This does :
kill(Pid::from_raw(pid as i32), Signal::SIGUSR1).unwrap();
let _ = dummy.wait();```


myResults = mySimu.run()  Error

I have run one of your tutorial examples and got an error when running the simulation at myResults = mySimu.run()
The Error  message is as following:
TypeError: init() got an unexpected keyword argument 'symmetries'
This error was within
File "C:\ProgramData..........\Machine\LamSlotWind\comp_mmf_unit.py", line 86, in comp_mmf_unit
MMF = DataTime(
Could you, please, support us to fix this error to be able to continue investigating pyleecan. Please, find the attached file.
Thank you in advance
Kind Regards

Hello,
I think that your issue is linked to an outdated version of SciDataTool and/or pyleecan. Can you check your versions of both package ? To get the latest version of the packages you can use:
pip install -U SciDataTool
pip install -U pyleecan
Best regards,
Pierre

Thank you dear Bonneel for your valuable reply. I have installed them and it skip that error and ended up with another error regarding the mesh, as following:
Exception: error: problem loading mesh
Could you, please direct us on how to fix that error too.
Kind Regards

Hello again,
To make sure that everything is clean, I've released a new version of pyleecan with the latest development and I have updated the tutorials machine, FEMM, OP and Elec.
Tutorials.zip
All tutorials and tests are correct on my side with this new version. Please run again pip install -U pyleecan and hopefully it should work now.
Best regards,
Pierre

Hi Bonneel,
I have installed the updated version of Pyleecan as you mention. The version I have now is 1.0.4 and still I have the same error.
I have saved the .fem file to check the mesh loading problem, I tried to run the model after remeshing it, but it gives an error as shown in the following photo.

I think there is an error, which I can not follow it as I am new to pyleecan.
I am looking for your support in fixing this.
Kind Regards

Hello,
Can you send me the script / notebook you are using so I can be sure to call the same methods as you do ? By the way which version of femm / pyfemm are you using ?
Best regards,
Pierre

Hello,
I have pyfemm 0.1.1 installed.
Please, find the attached python file.
test_1.zip
Thank you for your prompt reply.
Kind Regards

Hello,
I've tried your script on my side and it worked :/
I'm using FEMM 4.2 21Apr2019 (c:/femm42/read.me), windows 10 and python 3.8.3 as part of Anaconda3.
It seems that the error happens when femm try to read a file. Maybe it is a right problem, you can set the path to save the fem file with simu.path_result or you can try to run your python with administration right. Normally pyleecan should automatically save the .fem file in the path_result folder. Can you check that the file is correctly created ?
Best regards,
Pierre

Thank you dear Bonneel,  Now it works, as I have reinstall the newer Femm version.
FYI, I think there is a pug regarding the drawing, as I have got these message when I was trying to draw the contour of the flux density, it works only for the first time running the simulation and does not work after that, besides the drawing the flux density versus time and angle does not show any figures.
Please, find the message I have got, as followes:
C:\ProgramData\Anaconda3\lib\site-packages\pyleecan\Functions\Plot\plot_2D.py:235: UserWarning:
Matplotlib is currently using module://ipykernel.pylab.backend_inline, which is a non-GUI backend, so cannot show the figure.
Again, Thank you for your valuable support.
Kind Regards

Great :)
For this error, I think it is linked to the context of how you are calling the plot. In a notebook, try to add "%matplotlib notebook" at the beginning of the cell with the plot.
Does it work when calling the method directly in a script ? Maybe you need to add "import matplotlib.pyplot as plt" and after the call to plot "plt.show()"
Best regards,
Pierre

Yes, I have added that line to my script, but it gives the same warning.
I have attached the script to you below.

I think the issue is maybe not linked to pyleecan. If you are running it from a script can you try to add the following at the beginning:
from matplotlib import use
use("Qt5Agg")  # Use PySide2 backend
In your test_3 files on which line do you get the error ? Does the plot machine display something ?
Best regards,
Pierre

Hello,
Is everything working correctly on your side ? Can we close this issue ?
Best regards,
Pierre

Hello,
I'm closing this issue, if the error happens again we can reopen it ;)
Best regards,
Pierre


React Native TransformError

Steps to reproduce,

Create a React Native project.

react-native init test_diff
cd test_diff
npm install --save diff

Modify index.ios.js file, and add the following line.

import {diffChars} from "diff";

Now run the React Native packager.

react-native run-ios

Error shows up. See attached.

I fixed this problem by add the babel presets into my project.
yarn add babel-preset-es3
yarn add babel-preset-es2015-mod

Also add the ignore file list into .babelrc. So the .babelrc looks like
{
  "presets": ["react-native"],
  "ignore": [
    "node_modules/diff/lib/**/*.js",
    "node_modules/diff/node_modules/**/*.js"
  ]
}

And make sure clear RN cache.
watchman watch-del-all
./node_modules/react-native/packager/packager.sh start --reset-cache
react-native run-ios

Had the same error using react (for web, not native), and your solution worked for me as well:
npm i --save-dev babel-preset-es3
npm i --save-dev babel-preset-es2015-mod

and in webpack.config.js added:
  module: {
    rules: [{
      query: {
        presets: [
          'es2015', 'react',
          'es3', 'es2015-mod' // <- added these
        ]
      }
    }]
  }

I believe this issue should be reopened; the fix here would be to move the config to a .babelrc file and exclude it from npm publishing.

Agreed with @akehir. Doubly so since having babel-preset-env and babel-preset-es2015 seems to cause some weird stuff to happen


Quote dates and timestamps constants when pushing down filters to Redshift

This patch fixes an issue which could lead to incorrect results for queries which contained filters with date or timestamp constants. We need to quote the date and timestamp literals in order to prevent them from being interpreted as arithmetic expressions or producing syntax errors. For more details, see #152.
Fixes #152.
/cc @gregrahn, who reported this issue, and @marmbrus for review.

Current coverage is 89.07%

Merging #156 into master will increase coverage by +0.08% as of 2145d53

@@            master    #156   diff @@
======================================
  Files           13      13
  Stmts          645     650     +5
  Branches       142     144     +2
  Methods          0       0
======================================
+ Hit            574     579     +5
  Partial          0       0
  Missed          71      71

Review entire Coverage Diff as of 2145d53

Powered by Codecov. Updated on successful CI builds.

Given @gregrahm's LGTM over at #152 plus my own self-review, I'd say that this is good-to-go, so I'm going to merge this now.


[Enhancement] Keep Express Mail in the same cycle

This PR is an alternate solution for #732 that aims to just keep the Express Mail item in the player's inventory when trading it the first time to either Postman/Madame Aroma. This works by checking who the letter is being traded to, and if the opposite actors trade cycle flag is set or not. Only if it is the first trade will the Mail be kept, otherwise it will disappear.
This way the player is still expected to interact with both NPCs and the item gives are untouched, keeping the enhancement compatible with a future rando.
Leverages a new VB hook in the msg script event system to override item delete commands.

I prefer this approach over what I did in #732 because this ensures the player actually gets to engage with the content from both routes.


chore: release main

:robot: I have created a release beep boop
stdlib: 1.0.5
1.0.5 (2023-12-23)
Bug Fixes

build (559a2b0)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/pavophilip/react-shader-fiber/releases/tag/stdlib-v1.0.5 :sunflower:


[WIP] Convert cloud volume operations to SupportsFeatureMixin

[ ] https://github.com/ManageIQ/manageiq-ui-classic/pull/7839
[ ] https://github.com/ManageIQ/manageiq-providers-autosde/pull/78
[ ] https://github.com/ManageIQ/manageiq-providers-amazon/pull/719
[ ] https://github.com/ManageIQ/manageiq-providers-ibm_cloud/pull/260
[ ] https://github.com/ManageIQ/manageiq-providers-openstack/pull/727

To convert validate_create_volume in this search which is a class method, I tried converting to the typical pattern using ems.supports_cloud_volume_create?.  However, the class that will implement that supports logic is a specific storage manager, which I believe hangs off of the EMS.  @agrare will this just work at the EMS level or do we need to refactor callers to ask a specific storage manager?

Hey @chessbyte
All we need to do is switch the callers (e.g. https://github.com/ManageIQ/manageiq-ui-classic/blob/ddd3dfa05e887ff028af1ba3785fd15ca61e072f/app/controllers/cloud_volume_controller.rb#L218) from calling:
CloudVolume.validate_create_volume(ext_management_system)

To
ext_management_system.supports_cloud_volume_create?

It is already the appropriate EMS.
Then we'll add supports_not :cloud_volume_create to ExtManagementSystem and supports :cloud_volume_create to each of the storage_managers that support it.
(Also this looks suspect https://github.com/ManageIQ/manageiq-ui-classic/blob/ddd3dfa05e887ff028af1ba3785fd15ca61e072f/app/controllers/cloud_volume_controller.rb#L215 and maybe/hopefully not needed anymore)

@miq-bot cross-repo-test /all, ManageIQ/manageiq-ui-classic#7839, ManageIQ/manageiq-api#1070, ManageIQ/manageiq-providers-autosde#78, ManageIQ/manageiq-providers-amazon#719, ManageIQ/manageiq-providers-ibm_cloud#260, ManageIQ/manageiq-providers-openstack#727

Cross-repo tests are green - removing WIP.


add optional filters in data apstra_datacenter_interfaces_by_link_tag

This data source is great and would be used for most CT assignment scenarios.
It would provide more flexibility and capability if below two optional filters can be supported as well:

system_label    (for the device names)
system_tags     (for the device tags)

Feng Wang @ TELUS

a couple other additional filters like:

peer_system_name
peer_system_tags

if they are possible or not


error while running

node_modules@googleworkspace\google-docs-hast\dist\index.js:1392
return el.paragraph.bullet.nestingLevel ?? 0;
^

What is the error?

I also encountered the same problem, babel-loader failed to parse


Song placement in queue is wrong in rich presence if it exists more than once in the queue

Checklist:

[*] I have checked that there are no issues with similar or the same content
YOU SHOULD CHECK CLOSED ISSUES ASWELL

[*] I have checked the FAQ (https://github.com/MarshallOfSound/Google-Play-Music-Desktop-Player-UNOFFICIAL-/wiki/FAQ) and the answer I am looking for is not there

[*] I have double checked and can reproduce the issue

OS: Windows 10
GPMDP Version: 4.6.1
Issue Descriptions: Discord's Rich Presence doesn't correctly display the placement of the song in the queue if it exists more than once in the queue
Steps to Reproduce:

Load up a playlist with any number of songs (Probably shown best if playlist is >3)
Use Play Next on a song that already exists in the queue
Hit fast-forward to go to that song
Discord will show the second place of that song
The song in this example is 3rd in the queue, but shows up as 13th, since that's when it displays next.

I have been looking into this issue and found out that PlaybackAPI.currentSong().artist is returning artist of last song played in the queue instead of the current one. I tried to use PlaybackAPI.currentSong().artist directly and ended up getting artist of last song played instead of current one:
const presence = {
      state: PlaybackAPI.currentSong().title,
      details: PlaybackAPI.currentSong().artist,
      startTimestamp: start,
      endTimestamp: end,
      instance: false,
      largeImageKey: 'playing',
      largeImageText: Settings.get('service') === 'youtube-music' ? 'YouTube Music' : 'Google Play Music',
      partySize: queueTrackIndex,
      partyMax: queueLength,
    };

@wssyncmldm That is a known and separate tracked issue #3180

Found the other issue couple of minutes ago and was going through all the comments when you referenced it


Hotfix 1.9.3

@jlstevens Tests on this PR will never pass because the test data is significantly different due to changes in QuadMesh support on master and probably a ton of other stuff.

@jlstevens This is now ready to release. All test failures are related to the binned dataset support that is now on master and the docbuild was pretty much painless and I haven't spotted any issues on dev.holoviews.org

1.9.3 has now been released and is available on conda (ioam and soon conda-forge) and pip. Closing.


update(userspace/falco): turn error into warning log when plugins are loaded but unused

What type of PR is this?
/kind cleanup
/kind design
Any specific area of the project related to this PR?
/area engine
/area tests
What this PR does / why we need it:
The error plugin XXX has field extraction capability but is not compatible with any loaded event source has been troublesome for many users. Although it's semantically correct to prevent Falco from loading plugins that are not actually used, this causes the UX to be worse for users who are getting started with the plugin system. So, this PR proposes turning the error into a warning-level log message, so that Falco does not get blocked.
Which issue(s) this PR fixes:
Special notes for your reviewer:
Does this PR introduce a user-facing change?:
This is a proposal open for discussion, with the goal to insert it in Falco 0.33.0 eventually given the small amount of changes involved.
/hold
/milestone 0.33.0
update(userspace/falco): turn error into warning log when plugins are loaded but unused

@jasondellaluce: Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/milestone 0.34.0

LGTM label has been added.  Git tree hash: 885b2c5dc74d4a7fdcf39f9456ba1a68d9dbb955

[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: jasondellaluce, leogr
The full list of commands accepted by this bot can be found here.
The pull request process is described here

Needs approval from an approver in each of these files:

OWNERS [jasondellaluce,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

/close
I feel like these changes are not really necessary. It is good to make users' life easier, but I'm concerned that by making this fail silently will make issues related to plugins config harder/impossible to debug and troubleshoot.

@jasondellaluce: Closed this PR.

In response to this:

/close
I feel like these changes are not really necessary. It is good to make users' life easier, but I'm concerned that by making this fail silently will make issues related to plugins config harder/impossible to debug and troubleshoot.

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.


how to pass props to rendered view

I created a tab view used this guide  and every thing worked correctly but i cant use props.navigation in rendered views...how can i pass props to the components i render them in tab-view?
i need some thing like this:
<Hotest navigation={this.props.navigation}/>
how its possible?

https://github.com/react-native-community/react-native-tab-view#scenemap

to pass additional props to your scene components, use renderScene directly instead of using SceneMap


Update regex, add validator, add test cases, update expected output

Fixes #100
Update regex, add validator, add test cases, update expected output

// Copyright (c) Microsoft. All rights reserved.

can you update the sarif-sdk submodule to latest main, pls?

Refers to: Src/Plugins/Security/EmailAddressValidator.cs:1 in 3163ce8. [](commit_id = 3163ce8e074ac78a709d5f30f3d69a3af1120552, deletion_comment = False)

// Copyright (c) Microsoft. All rights reserved.

can you update the sarif-sdk submodule to latest main, pls?

Refers to: Src/Plugins/Security/EmailAddressValidator.cs:1 in 3163ce8. [](commit_id = 3163ce8e074ac78a709d5f30f3d69a3af1120552, deletion_comment = False)

// Copyright (c) Microsoft. All rights reserved.

can you update the sarif-sdk submodule to latest main, pls?
Refers to: Src/Plugins/Security/EmailAddressValidator.cs:1 in 3163ce8. [](commit_id = 3163ce8, deletion_comment = False)

Done!

// Copyright (c) Microsoft. All rights reserved.

can you update the sarif-sdk submodule to latest main, pls?
Refers to: Src/Plugins/Security/EmailAddressValidator.cs:1 in 3163ce8. [](commit_id = 3163ce8, deletion_comment = False)

Done!


Crash in set_garbage_collection_priority in resilience test

This is replicating a failure I've seen in the resilience CI, e.g. here: https://github.com/StanfordLegion/resilience/actions/runs/10647296358/job/29515103267
Appears to be a segfault. Backtrace on Sapling:
#5  0x00007fe3968bdcd8 in Legion::Mapping::MapperRuntime::set_garbage_collection_priority (this=0x5590e75013c0,
    ctx=0x7fe30af8b710, instance=..., priority=-2147483647)
    at /scratch/eslaught/resilience-test-network/legion/runtime/legion/legion_mapping.cc:1445
#6  0x00005590e65eb820 in ResilientLegion::Mapping::ResilientMapper::map_copy(Legion::Internal::MappingCallInfo*, Legion::Copy const&, Legion::Mapping::Mapper::MapCopyInput const&, Legion::Mapping::Mapper::MapCopyOutput&) ()
#7  0x00007fe396c579fc in Legion::Internal::MapperManager::invoke_map_copy (this=0x5590eac5cfe0, op=0x7fe3129242c0,
    input=..., output=...) at /scratch/eslaught/resilience-test-network/legion/runtime/legion/mapper_manager.cc:341
#8  0x00007fe3968e64f1 in Legion::Internal::CopyOp::trigger_mapping (this=0x7fe3129242c0)
    at /scratch/eslaught/resilience-test-network/legion/runtime/legion/legion_ops.cc:6767
#9  0x00007fe396d93844 in Legion::Internal::Runtime::legion_runtime_task (args=0x7fe31291eb90, arglen=12,
    userdata=0x5590eabd57c0, userlen=8, p=...)
    at /scratch/eslaught/resilience-test-network/legion/runtime/legion/runtime.cc:32434
#10 0x00007fe393c234dc in Realm::LocalTaskProcessor::execute_task (this=0x5590e741e5c0, func_id=4, task_args=...)
    at /scratch/eslaught/resilience-test-network/legion/runtime/realm/proc_impl.cc:1176
#11 0x00007fe393ca02c8 in Realm::Task::execute_on_processor (this=0x7fe31291ea10, p=...)
    at /scratch/eslaught/resilience-test-network/legion/runtime/realm/tasks.cc:326
#12 0x00007fe393ca5642 in Realm::UserThreadTaskScheduler::execute_task (this=0x5590e74f9790, task=0x7fe31291ea10)
    at /scratch/eslaught/resilience-test-network/legion/runtime/realm/tasks.cc:1687

Note I'm going to share the build with #1753, so watch out when you rebuild it:
To reproduce:
cd /scratch/eslaught/resilience-test-network
source experiment/sapling/env.sh
salloc -n 1 -N 1 -c 40 -p all --exclusive
for i in $(seq 0 40); do REALM_SYNTHETIC_CORE_MAP= REALM_FREEZE_ON_ERROR=1 srun -n 2 -N 1 --cpu-bind none --overlap ./build/tests/region_destroy -lg:safe_ctrlrepl 1 -level resilience=4 & done

Reproduction ratio seems to be about 10%.
To rebuild:
cd /scratch/eslaught/resilience-test-network
source experiment/sapling/env.sh
srun -n 1 -N 1 -c 40 -p all --exclusive --pty bash --login
./experiment/setup.sh

Is this a debug build?

Fixed with:
https://gitlab.com/StanfordLegion/legion/-/commit/d96954f9afebce9167a17e17f53725f51d1dc526
Not sure why you are setting a garbage collection priority on a no-instance, but Legion will now happily ignore this case.


How can I get RFID Identifiers within the add-on?

Hello,
I've installed EVCC as add on in homeassistant. It works so far. Now I wanted to setup the RFID identifier. In the documentation it is mentioned that I should use "evcc charger" on the commandline. I tried it with the "Advanced SSH & Web Terminal" wich is also installed as an add on inside HA. I get always command not found.
Can someone give me a hint how to receive the RFID identifier in the EVCC add on?
Thanks

At Addon command line is not supported. Pls use Windows or Linux.


Ensemble Prediction

Hello, is there any way to run a ensemble version of Graphcast like ECMWF EPS?

Check out this paper: https://arxiv.org/html/2312.15796v1


Make admonition directive html more similar to python-markdown

Right now the block is wrapped inside a  element, unlike any other markdown renderer. Moreover if the title is not specified the admonition has no title, while it should default to the name of the admonition. This pull request fixes these two issues.

You can rewrite the renderer to get your goal done.

Well, subclassing the Admonition class would be a better way but that's not the point. I know I can get the result I want, but I think that it should be changed in the upstream repository. The other markdown renderers render admonition in a certain way, and CSS themes expect that. I don't think that mistune should be different, it only generates confusion