turn off newer RELX relocation optimizations

otherwise breaks the link step on systems with an older linker (i.e., travis CI, some ubuntu's, etc.). ref rust-lang/rust#34978
We may only want to add this flag on nightly? Not sure how to do that...
NB requires a newer assembler (don't know exact minimal working version), but
as -version
GNU assembler (GNU Binutils) <IP_ADDRESS>60501

works.

r? @brson
(rust_highfive has picked a reviewer for you, use r? to override)

Thanks! We shouldn't have to worry about the assembler version here as we're in control of it, but I think we'll also need to pass this to the build system, right? I remember the error being with jemalloc as well I think...

Ok, I've pushed this as alexcrichton/rust-slave-linux:2016-08-01, it should go out the next time we restart the buildbots. Do you want to send a patch to set this for the musl target as well?

@alexcrichton I thought this was the musl target?
and yea, will need to pass to the build system, as the relocs show up in libstd and liballoc_jemalloc, as well as libunwind (but I hope it's cmake will obey the CFLAGS var?).
Unfortunately I'm not familiar with that part of building rust, and don't know if we have an equivalent CFLAGS somewhere in the pipeline?

Yeah it should be easy enough to modify the mk/cfg/*.mk files and also just add custom flags in src/bootstrap for the rustbuild build system. Also yeah this should only affect musl, and I think in the build system we only need to really worry about jemalloc, other pieces are fine to ignore I think.

Ok, new image is now deployed


[v3.0.1] LET block always returns null when used with out function

OrientDB Version: 3.0.1
Java Version: 1.8
OS: Linux
I want to use Let block in my query.
Actual/Expected behavior
When I assign an edge to a variable in LET block and use it in projection, it returns null. But it is expected to return a value.
SELECT $x, name FROM V
let $x = out("E")

SELECT $x, name FROM V
let $x = out("E").name

Result:
{
  "result": [
        {
            "$x": null,
            "name": "A"
        },
        {
            "$x": null,
            "name": "B"
        }
    ],
}

Steps to reproduce
insert into V set id = 1, name = "A"
insert into V set id = 2, name = "B"

create edge E from (select from V where id = 1) to (select from V where id = 2)

SELECT $x, name FROM V
let $x = out("E")

Hi @mehdizarepour
Thank you very much for reporting, I'll check it and let you know
Thanks
Luigi

Hi @mehdizarepour
I pushed fix for this issue. It will be released with version 3.0.2
Thanks,
Marko.


Game softlocks on promotion screen during Tutorial

When you first promote Jane Kelly. This may or may not be connected to "hide "new class" popup" MCM Setting.

I can confirm I've had the same issue, and indeed narrowed it down to having "Disable New Class popups" checked while doing the tutorial.

Confirmed by testing. I'll make CPS ignore the MCM setting while tutorial is active.


[Pipeline#pt1] Asset resolvers

Extracted asset location mechanism from AssetFactory into separate asset locators, based on AssetLocatorInterface.
It cleans up AssetFactory interface and adds couple of new extension points to asset location mechanism.
It's a first step towards Assetic Pipeline implementation.
Also, PR introduces PipelineAssetResolver, which uses registered paths in order to find proper asset by it's relative path.
It's the second step towards Assetic Pipeline and depends on merging #210 first.

+1 .


Avoid one frame delay using TabNavigator

As noted in #1967, the optional initialLayout func prop used by react-native-tab-view's TabViewAnimated component is missing.

[x] Add missing prop.
[x] Update README.

Not aware of a way to update the initialLayout values on orientation changes, when it's set from TabNavigatorConfig. Since screenProps are being passed down, maybe that'd be a better way to handle initialLayout?

Seems like react-native-tab-view TabViewAnimated could be refactored to use initialLayout as a prop, instead of in the state on initialization, then passing the value through screenProps might work.

Nevermind, should be okay as-is, afaik. I'm styling tabs locally, orientation changes seem okay without custom tabs.

Thanks for the PR! LGTM other than the minor documentation nitpick in the review.

You're welcome, glad to contribute. I also added capitalization/puncuation normalization in the TabNavigator.md document; small nitpick of my own lol.

@matthamil I think tests just need a rebuild to pass. I can revert the capitalization or punctuation changes, if they're undesirable here.

@TheTekton can you please fix the flow error?:
Error: src/views/TabView/TabView.js:6
  6: import { Layout } from 'react-native-tab-view/src/TabViewTypeDefinitions';
              ^^^^^^ Named import from module `react-native-tab-view/src/TabViewTypeDefinitions`. `Layout` is a type, but not a value. In order to import it, please use `import type`.

Thanks @vonovak, been a bit overwhelmed; would have taken some time to even remember to check back. 👍

And thanks to @TheTekton for the PR :)


Replace the container launcher with feature detection, update launcher handling

What does this PR do?
This contains some house-keeping changes, isolating launchers somewhat from the rest of the logs agent, followed by a replacement of the container launcher with feature detection.
Previously, the container launcher didn't launch anything, but just tried to start up either the docker or kubernetes launchers, checking both features and whether kubelet or the docker API were available.  It took whichever appeared first, preferring one over the other based on the logs_config.k8s_container_use_file config value.
This will soon become problematic, as those launchers are both doing more than they should, and will be broken into components.  So, we'll need a way for multiple components to determine whether they should start up, which is what this PR does.
Describe how to test/QA your changes
See https://github.com/DataDog/datadog-agent/pull/8148.
Reviewer's Checklist

[ ] If known, an appropriate milestone has been selected; otherwise the Triage milestone is set.
[ ] Use the major_change label if your change either has a major impact on the code base, is impacting multiple teams or is changing important well-established internals of the Agent. This label will be use during QA to make sure each team pay extra attention to the changed behavior. For any customer facing change use a releasenote.
[ ] A release note has been added or the changelog/no-changelog label has been applied.
[ ] Changed code has automated tests for its functionality.
[ ] Adequate QA/testing plan information is provided if the qa/skip-qa label is not applied.
[ ] At least one team/.. label has been applied, indicating the team(s) that should QA this change.
[ ] If applicable, docs team has been notified or an issue has been opened on the documentation repo.
[ ] If applicable, the need-change/operator and need-change/helm labels have been applied.
[ ] If applicable, the config template has been updated.

Ugh, looks like there's a lot more mess here than I thought -- the unit tests appear to be sensitive to which of docker and kubernetes are available..

This is still ready for review, but I'll work on the QA steps directly in parallel.

It occurred to me that I can do one-better here, avoiding some races.
It's already the case in this PR that only one of the docker and k8s launchers starts, based on feature detection and configuration.
Once that's done, the current implementation waits until kubelet or docker is available before subscribing to integration.Configs from AD.  However, that leads to a race where, if AD finds kubelect/docker first, it may send a bunch of Configs before the launcher starts listening.  But doing the opposite may lead to AD getting blocked sending to a channel when the launcher is not yet reading from it.
What I realized is that the k8s launcher already has the ability to retry processing integration.Config's, and it'd be easy to add that to the docker launcher as well.  And, I can use that support to also retry Config's when kubelet/docker aren't available.

OK, I think I'm happier with this.  The retries in the last commit are a bit of a bodge, but

They're unlikely to ever be used (how would AD tell logs-agent about a container, if there's no container runtime??)
They're mostly used for container_collect_all, which will move into AD soon

This represents a bit of a behavior change.  The old behavior was:

if neither kubelet nor docker can start, neither launcher runs
if kubelet can start but docker cannot, the kubernetes launcher runs
if docker can start but kubelet cannot, docker launcher runs
if both can start

with DD_LOGS_CONFIG_K8S_CONTAINER_USE_FILE=false, the docker launcher runs
with DD_LOGS_CONFIG_K8S_CONTAINER_USE_FILE=true, the kubernetes launcher runs

The new behavior is:

if DD_LOGS_CONFIG_K8S_CONTAINER_USE_FILE=false, the docker launcher runs

if the docker socket is not available, nothing happens

if DD_LOGS_CONFIG_K8S_CONTAINER_USE_FILE=true, the kubernetes launcher runs

if kubelet is not available, nothing happens

this is because whenever running in kubernetes, both the docker and kubernetes features are enabled, and DD_LOGS_CONFIG_K8S_CONTAINER_USE_FILE is what defines the preference between those.  Basically, I've changed the "have a preference but use whatever starts working first" to "have a preference".

This behavior change may have a significant functional impact.
One example I can think of is that, since DD_LOGS_CONFIG_K8S_CONTAINER_USE_FILE=false is the default value of this config option including on k8s environments where the docker socket may not be available (or where docker isn't used at all), on such environments I would expect log collection to stop working by default with this change. (Is that correct?)
Also, does this PR change the behavior implemented in https://github.com/DataDog/datadog-agent/pull/8148 ? (that attempts retrying both launchers until one succeeds)
More generally, we'd like the logs agent launcher logic to be resilient to a temporary unavailability of the docker or the kubernetes launchers at startup (I don't know if this PR changes anything about that).
Sorry for the unstructured thoughts here. @gh123man has directly worked on this logic in the past and may be able to provide more structured thoughts.

For reference, "Docker" in this context means Docker, Containerd, Cri, or Podman.
The features are currently defined as follows:

Kubernetes - either $KUBERNETES_SERVICE_PORT or $KUBERNETES is set
Docker - either $DOCKER_HOST is set or one of the docker sockets exists and is readable (but notably, there is no check whether the docker daemon is actually listening there)
Containerd / Cri -

if cri_socket_path is set; or if /.dockerenv doesn't exist and one of the default CRI sockets exists and is readable (but notably, there is no check whether the CRI daemon is actually listening there); then
if either $KUBERNETES_SERVICE_PORT or $KUBERNETES is set, then feature Cri is enabled
if containerd appears in the detected CRI socket filename, then feature Containerd is enabled.

Podman - if any of the podman socket paths exist (without testing readability)

It may be that DD_LOGS_CONFIG_K8S_CONTAINER_USE_FILE was defined in such an implementation-specific way that it's impossible to make this change and fix the race condition mentioned above.  Maybe this isn't worth changing at all?

Closed in favor of #11289.


Missing self in various run calls

        with open(fname, 'w') as fp:
            json.dump(stored_data, fp, indent=4, separators=(',', ': '))

Should be self.stored_data. There are several places where this is the case.

These are actually dead code and should be deleted.


🛠 Tooling: sort package.json

Bug Report Checklist

[X] I have tried restarting my IDE and the issue persists.
[X] I have pulled the latest main branch of the repository.
[X] I have searched for related issues and found none that matched my issue.

Overview
Keep package.json sorted to make it easy to quickly find each section.
Additional Info
This can automatically be done with a prettier plugin.

Agreed, package.json should be sorted - and it already is! See https://github.com/JoshuaKGoldberg/ts-api-tools/pull/20's https://github.com/JoshuaKGoldberg/ts-api-tools/actions/runs/4107681077/jobs/7089665450:
/home/runner/work/ts-api-tools/ts-api-tools/package.json
Error:   44:4  error  Expected object keys to be in ascending order. 'import' should be before 'types'  jsonc/sort-keys
Error:   54:2  error  Expected object keys to be in ascending order. 'name' should be before 'types'    jsonc/sort-keys

The difference is, I have them sorted in alphabetical order. Which is something I believe in - it's hard to memorize arbitrary sorts for different systems. Alphabetical is a nice predictable standard. What do you think?
Btw, I transferred this issue over from ts-api-tools to this template-typescript-node-package since it impacts all of them.

aside: @allcontributors add @RebeccaStevens for idea

All right, I filed https://github.com/ota-meshi/eslint-plugin-jsonc/issues/211. The alphabetical ordering feels wrong to me... but so did a lot of good tooling things, such as Prettier. So maybe I'm just being a luddite.


IPAM fail when masquerade is set to true

I use calicoctl pool add <IP_ADDRESS>/16 --nat-outgoing to create pool, then I use calicoctl container add 3e <IP_ADDRESS>/16 --interface veth0.1 to add network from pool to container 3e, it failed.
Error was: ValueError: Requested pool <IP_ADDRESS>/16 is not configured or haswrong attributes.
But if I remove "masquerade": true in etcd, it works. And after this I add "masquerade": true back to etcd, it still works.
I run calico-node in docker with the latest version released about 2 days ago, calicoctl v0.13.0.

@tonicbupt Where did you submit the PR?  (to calico-docker repo?)  Can you link to it?

@spikecurtis  don't know why GitHub treat my PR link as issue link... here is the PR to calico-docker, thanks


Removing relationships: Code sample doesn't match the docs

The Related Data: Removing relationships docs mention that the Post entity is removed because of cascade on delete. However, in the code sample the Post entity is deleted explicitly.

blog.Posts.Remove(post) deletes the post from collection navigation of blog. Since cascade delete is configured, the post will be removed.

@smitpatel Thank you for getting back to me. Again, I must have read blog.Posts.Remove(post) as context.Posts.Remove(post). Closing issue.


fix: improve running time of GitHub actions

After moving to self-hosted runners on PS6, the GitHub actions could take 1-2+ hours. Caused in part by how the runners were configured and how the GitHub actions were constructed.
The PS6 self-hosted runners are optimized for larger jobs. Turn around time on recreating a runner could be 15-30 minutes.
Steps taken in the optimization:

Move very quick jobs back to using GitHub runners. e.g. static analysis and clarity
Tests requiring the build job, are moved to the same workflow to reduce the number of times we run the build job
Remove the unit test job, this is being done by github-check-merge-juju-terraform-provider-juju
Add a parallelization of 10 to the tests

The matrix for running the integration tests has also been reduced in size by 2/3rds. Old versions of the terraform client will no longer be tested, only the most recent. We have never seen a difference in test results caused by different client versions.
Only the x64 runners will be used until the acceptance tests can be updated with new tests which run on both architectures.

/merge

/merge


Added simplified entries for channel counts greater than 8

Fix for #148

Thank you for digging into this! I'll give this attention as soon as I can. It might take a few days.
Cheers,
Mike

Is there some documentation that refers to how this is calculated? I foresee myself wondering WTF this means when reading it in the future.

To be honest, I just kept turning on bits from the right until I had 16 channels worth. It looks like libav 12 includes support for more than 16 channels of audio.
https://git.libav.org/?p=libav.git;a=blob;f=libavutil/channel_layout.h;h=5bd0c2ce6ea0d7a88ba30d221eeb76ecd1c2ef04;hb=ee480790c7eeb03c9cebd8971c46e0cb7db65277


Fully automated, manually triggered, CP deployment (Design + Implementation)

Design a new process for deployment, which is manually triggered, but fully automated
Implement the design once finalized

Hey @eladlachmi, please see this dup ticket  adding here context on ideas I had around this and closing the issue I opened. I created the ticket at the same week as you :)


refac: http factory client is injected into JobsAPiClient.

Description

Now HttpClientFactory is inject into JobsApiClient
Added tests of the resolving of http client in JobsApiClient
DatabricksJobsOption is now bund in the registration class (JobsExtensions)

Quality

[x] Release notes are updated
[x] Package version is updated
[x] Public types and methods are documented
[x] Tests are implemented and executed locally

Codecov Report

Merging #253 (ae79638) into main (08a6b99) will decrease coverage by 8.71%.
The diff coverage is 65.38%.

@@            Coverage Diff             @@
##             main     #253      +/-   ##
==========================================
- Coverage   79.91%   71.21%   -8.71%
==========================================
  Files          25      217     +192
  Lines         453     3932    +3479
  Branches       32      459     +427
==========================================
+ Hits          362     2800    +2438
- Misses         79      981     +902
- Partials       12      151     +139

Flag
Coverage Δ

databricks
78.82% <65.38%> (-1.10%)
:arrow_down:

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

Files
Coverage Δ

...s/Extensions/DependencyInjection/JobsExtensions.cs
100.00% <100.00%> (ø)

...e/Databricks/source/Jobs/Internal/JobsApiClient.cs
45.45% <100.00%> (-28.62%)
:arrow_down:

...urce/Jobs.UnitTests/Internal/JobsApiClientTests.cs
0.00% <0.00%> (ø)

...ensions/DependencyInjection/JobsExtensionsTests.cs
0.00% <0.00%> (ø)

... and 192 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


tailer: don't load entire file into memory, stream it out bits at a time

Thanks!

If you wanted to add a buffer, you could just make the lines channel able to hold N entries via make(chan string, N), and that would be roughly equivalent?

It seems I was a little tired when I wrote my comment yesterday, I confused the buffered tailer (bufferedTailer.go) and the buffered line reader (linereader.go). It's actually the buffered tailer's job to decouple reading lines from processing matches. The line reader has nothing to do with it.
Looking at it again it really seems unnecessary to buffer more than one line in the line reader, because there is already a buffer in the buffered tailer. I will look at this again later today...
Sorry for the confusion.

It seems that linereader_test.go needs to be adapted to the new behavior.


Run integration tests on Windows

Now that unit tests are passing on Windows, make sure integration tests pass on Windows. There's really no way to get to 1.0 without taking this step.

[x] Verify current integration tests work on macOS:

[x] flyio
[x] platformsh Note: this is the only platform that needed no updating.
[x] heroku (See #242)

[ ] Update integration tests to use Python, and remove all dependence on shell scripts.
[ ] Update integration tests to use pytest.
[ ] Require pytest to be run with either unit_tests or integration_tests argument.
[ ] Update integration tests to work on Windows.
[ ] Update test organization.

[ ] Move unit tests under a new tests/ dir.
[ ] Move integration tests under tests/.
[ ] Rename unit_tests -> e2e_tests.
[ ] Update docs to reflect all of these changes.

[ ] Make a new release, as this fixes some bugs ie ALLOWED_HOSTS on Heroku.

[ ] Consider requiring that only one platform's integration tests be specified on each test run?

Working notes

[ ] Update docs with the variety of ways you can call tests: cd unit_tests && pytest, pytest unit_tests, etc. Not root pytest, pytest unit_tests integration_tests.
[ ] unit_tests -> e2e_tests

Update fly_io deployments

[ ] Integration test without automate-all fails. Complains that a fly app has not yet been created, even though one has.
[ ] Integration test with automat all fails. Complains about not recognizing a region.

[ ] Add a divider between user docs and dev docs in menu?

Convert platform_sh test from shell to python

[ ] Probably need to activate the test environment?

Refining platform_sh test

[ ] Test pypi deployment.
[ ] Test poetry project.
[ ] Test --automate-all.

pkg_manager

[ ] Handle different package managers better in testing setup scripts.

Pipenv fails to generate lock file when calling simple_deploy?
From an integration test of local dsd install:
--- Your project is now configured for deployment on Platform.sh. ---

To deploy your project, you will need to:
- Commit the changes made in the configuration process.
    $ git status
    $ git add .
    $ git commit -am "Configured project for deployment."
- Push your project to Platform.sh' servers:
    $ platform push
- Open your project:
    $ platform url
- As you develop your project further:
    - Make local changes
    - Commit your local changes
    - Run `platform push`

- You can find a full record of this configuration in the simple_deploy_logs directory.
Courtesy Notice: Pipenv found itself running within a virtual environment, so it will automatically use that environment, instead of creating its own for any project. You can set PIPENV_IGNORE_VIRTUALENVS=1 to force pipenv to ignore that environment and create its own instead. You can set PIPENV_VERBOSITY=-1 to suppress this warning.
Locking [packages] dependencies...
Building requirements...
Resolving dependencies...
✘ Locking Failed!
⠼ Locking...
ERROR:pip.subprocessor:[present-rich] Getting requirements to build wheel exited with 1
[ResolutionFailure]:   File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/resolver.py", line 811, in _main
[ResolutionFailure]:       resolve_packages(
[ResolutionFailure]:   File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/resolver.py", line 759, in resolve_packages
[ResolutionFailure]:       results, resolver = resolve(
[ResolutionFailure]:       ^^^^^^^^
[ResolutionFailure]:   File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/resolver.py", line 738, in resolve
[ResolutionFailure]:       return resolve_deps(
[ResolutionFailure]:       ^^^^^^^^^^^^^
[ResolutionFailure]:   File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/utils/resolver.py", line 1165, in resolve_deps
[ResolutionFailure]:       results, hashes, markers_lookup, resolver, skipped = actually_resolve_deps(
[ResolutionFailure]:       ^^^^^^^^^^^^^^^^^^^^^^
[ResolutionFailure]:   File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/utils/resolver.py", line 964, in actually_resolve_deps
[ResolutionFailure]:       resolver.resolve()
[ResolutionFailure]:   File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/utils/resolver.py", line 701, in resolve
[ResolutionFailure]:       raise ResolutionFailure(message=str(e))
[pipenv.exceptions.ResolutionFailure]: Warning: Your dependencies could not be resolved. You likely have a mismatch in your sub-dependencies.
  You can use $ pipenv run pip install <requirement_name> to bypass this mechanism, then run $ pipenv graph to inspect the versions actually installed in the virtualenv.
  Hint: try $ pipenv lock --pre if it is a pre-release dependency.
ERROR: Getting requirements to build wheel exited with 1

Traceback (most recent call last):
  File "<frozen runpy>", line 198, in _run_module_as_main
  File "<frozen runpy>", line 88, in _run_code
  File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/__main__.py", line 4, in <module>
    cli()
  File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/vendor/click/core.py", line 1130, in __call__
    return self.main(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/cli/options.py", line 58, in main
    return super().main(*args, **kwargs, windows_expand_args=False)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/vendor/click/core.py", line 1055, in main
    rv = self.invoke(ctx)
         ^^^^^^^^^^^^^^^^
  File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/vendor/click/core.py", line 1657, in invoke
    return _process_result(sub_ctx.command.invoke(sub_ctx))
                           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/vendor/click/core.py", line 1404, in invoke
    return ctx.invoke(self.callback, **ctx.params)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/vendor/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/vendor/click/decorators.py", line 84, in new_func
    return ctx.invoke(f, obj, *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/vendor/click/core.py", line 760, in invoke
    return __callback(*args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/vendor/click/decorators.py", line 26, in new_func
    return f(get_current_context(), *args, **kwargs)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/cli/command.py", line 369, in lock
    do_lock(
  File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/routines/lock.py", line 79, in do_lock
    venv_resolve_deps(
  File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/utils/resolver.py", line 1107, in venv_resolve_deps
    c = resolve(cmd, st, project=project)
        ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
  File "/private/var/folders/gk/y2n2jsfj23g864pdr38rv4ch0000gn/T/pytest-of-eric/pytest-316/blog_project0/b_env/lib/python3.11/site-packages/pipenv/utils/resolver.py", line 1001, in resolve
    raise RuntimeError("Failed to lock Pipfile.lock!")
RuntimeError: Failed to lock Pipfile.lock!

Committing changes...
[main 95dd4d9] Configured for deployment.
 5 files changed, 115 insertions(+), 1 deletion(-)

[ ] Consider setting a marker that will only install psycopg2 to the remote environment.
[ ] How can we fend off issues with locking not working on end user systems without Postgres?

Update Fly.io tests

[ ] Update main test file to run Fly.io deployment.
[ ] Carry over any improvements from this file to Platform.sh tests.

[ ] Better names in helper functions.

Update Fly.io deployments

[ ] Testing against PyPI, fails to recognize app name. Fix re in deploy.py?

Destroy project in yield section of tmp project fixture
This will allow destruction of the project even if there is a breaking failure.

[ ] Cache the app name, project id, or any other information needed for destruction in the pytest cache.
[ ] Cache the platform name as well.
[ ] In yield, pull from cache and call appropriate destroy utility.

Update Heroku testing

[ ] Cache app_name.


FILE SYSTEM OUT OF SPACE DURING CLONING OF KERNEL

git clone https://chromium.googlesource.com/chromiumos/third_party/kernel -b chromeos-[version]

after running this... clones kernel file but system runs out of space afterwards..

Please describe your issue:
I simply ran out of space on the filesystem... how do I find this massive file and delete it? where is is cloned on the filesystem?
If known, describe the steps to reproduce the issue:

nevermind it was located in ~/kernel
are there any residual files I can remove to save space?
please advise.

@Carl393 No, everything is in ~/kernel, so if you deleted that folder no residual files are left. For saving space you could look at what you have in ~/Downloads, or if you have multiple chroots, maybe backup and then delete. Not much else you can do. You could use an external storage where you clone the kernel, but that increases time when building. Fastest way for building is on the internal storage (SSD).


Fix 'generated' artifact not being removed and breaking manifests build by builder

Fixes manifoldjs/manifoldjs-site#213

@JuanAr,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by Microsoft. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot


Update forms to make them compatible with firefox.

Wow. I really didn't expect this. I thought that just because it worked in chrome it would work on all other browsers.
This really shows the importance of testing on multiple browsers.

Thank god i switched from chrome to firefox. I would've never caught this bug otherwise. PLEASE PLEASE PLEASE PLEASE PLEASE test on multiple browsers from now on.


ravenadm crashes with an illegal hardware instruction error

While trying to test a new package:
# /raven/bin/ravenadm test logstash6
[1]    573826 illegal hardware instruction (core dumped)  /raven/bin/ravenadm test logstash6
It seems this runtime error is related to the content of the manifests/plist.single file, which currently
contains 1028980 bytes.

i don't think the size is a problem.  There should be several package lists bigger than that.
I can't diagnose further.  I'd need to source files to try to replicate.

I have put a tarball of the work-in-progress port on http://dl.wolfpond.org/ravenports_logstash6.tar.gz .

On my system, removing the
share/logstash/vendor/jruby/lib/ruby/stdlib/rexml/attlistdecl.rb
line from the plist.single file allows the port to build.

how did you determine this was the culprit?
on the surface, there's nothing wrong with that filename ...

I don't think there's anything wrong with this particular filename per se.
Its possible removing other lines will also stop ravenadm from crashing.
This absence or presence of this line is just the shortest difference I found between a crashing ravenadm run or a successful one.

More complete trace:
Program terminated with signal 4, Illegal instruction.
#0  0x0000000800af0b4b in tls_get_addr_common (dtvp=0xb, index=-10494208, offset=140737477859712)
    at /usr/src/libexec/rtld-elf/rtld.c:4371

(gdb) bt
#0  0x0000000800af0b4b in tls_get_addr_common (dtvp=0xb, index=-10494208, offset=140737477859712)
    at /usr/src/libexec/rtld-elf/rtld.c:4371
#1  0x0000000800aecd62 in __tls_get_addr (ti=<optimized out>) at /usr/src/libexec/rtld-elf/x86_64/reloc.c:459
#2  0x00000008011d747a in .cerror () at /usr/src/lib/libc/../libc/x86_64/sys/cerror.S:49
#3  0x0000000000000002 in ?? ()
#4  0x0000000801197d04 in realpath (path=path@entry=0x7fffff5ff150 "/usr/obj/ravenports/SL01/port/manifests/plist.single",
    resolved=<optimized out>, resolved@entry=0x7fffff5fed40 "/usr/obj/ravenports/SL01/port/manifests/plist.single")
    at /usr/src/lib/libc/../libc/stdlib/realpath.c:160
#5  0x00000000007e6169 in __gnat_full_name (
    nam=nam@entry=0x7fffff5ff150 "/usr/obj/ravenports/SL01/port/manifests/plist.single",
    buffer=buffer@entry=0x7fffff5fed40 "/usr/obj/ravenports/SL01/port/manifests/plist.single") at cstreams.c:199
#6  0x00000000007d1fb6 in system__file_io__open () at s-fileio.adb:987
#7  0x00000000007d01ca in system.direct_io.create (file=<optimized out>, mode=<optimized out>, name=..., form=...)
    at s-direio.adb:113
#8  0x0000000000413079 in file_operations.dump_contents_to_file.file_string_io.create (file=0x0, mode=out_file, name=...,
    form=...) at /raven/toolchain/gcc8/lib/gcc/x86_64-raven-dragonfly5.0/8.2.0/adainclude/a-direio.adb:86
#9  0x0000000000412e83 in file_operations.dump_contents_to_file (contents=..., dossier=...)
    at /construction/ravenadm/ravenadm-1.08/src/file_operations.adb:77
#10 0x000000000044e25d in package_manifests.decompress_manifest (compressed_string=..., save_to_file=...)
    at /construction/ravenadm/ravenadm-1.08/src/package_manifests.adb:319
#11 0x00000000006c11b5 in specification_parser.parse_specification_file (dossier=..., specification=...,
    opsys_focus=dragonfly, arch_focus=x86_64, stop_at_targets=false, extraction_dir=..., success=false)
    at /construction/ravenadm/ravenadm-1.08/src/specification_parser.adb:651
#12 0x0000000000720a55 in portscan.operations.parse_and_transform_buildsheet (buildsheet=..., variant=..., portloc=...,
    excl_targets=false, avoid_dialog=true, for_webpage=false, sysrootver=..., successful=95)
    at /construction/ravenadm/ravenadm-1.08/src/portscan-operations.adb:2736
#13 0x0000000000723c5b in portscan.operations.build_subpackages (builder=1, sysrootver=..., interactive=false,
    enterafter=...) at /construction/ravenadm/ravenadm-1.08/src/portscan-operations.adb:2885
#14 0x00000000006f67a9 in portscan.operations.parallel_bulk_run.build (<_task>=0x7fffffdfc6d0)
    at /construction/ravenadm/ravenadm-1.08/src/portscan-operations.adb:110
#15 0x00000000007b0ede in system__tasking__stages__task_wrapper () at s-tassta.adb:1225
#16 0x000000080114be5e in thread_start (arg=0x8012e03c0) at /usr/src/lib/libthread_xu/thread/thr_create.c:233
#17 0x0000000000000000 in ?? ()

I'm going to close this one.  I've never seen this issue myself.


Populated NegotiationContext when NancyContext is created

If you have a StatusCodeHandler that responds to exceptions and returns a negotiated response the NegotiationContext is null when an exception occurs before a module is built.  NegotiationContext is needed in the  WithModel method. The DefaultNancyModuleBuilder is where it currently only gets set. I have set the NegotiationContext now in DefaultNancyContextFactory when the NancyContext is created.

OK, now i'm hitting up against this :cry:. i can give up Negiotiation in my custom error handler, but i'd like not to.
Any way i can help with this?


Analytic agent update

please merge to 3.20

has merge conflicts

Has conflicts.

merged https://github.com/kgrgreer/foam3/pull/1988


Tiny icon for the new tip on the veBAL page

On small screens, the icon becomes tiny.
Ideally the icon should have a fixed width or a minimum width.

Fixed here: https://github.com/balancer-labs/frontend-v2/pull/1519/commits/ba96ac993df1848f5fe8e22bff176d7b890044f2


Set a different fetchSize value for cassandra queries

Fixes #1441

Is this a permanent solution? If not, I think we should spend a couple of days looking into a more robust option instead of adding technical debt.

Thanks. I think everything that ends up being added to the elasticsearch indices needs to use this method as well, since that eats whole tables.

Let's see if this works first and then I'll take care of the remaining caching methods 👍

@sallakarppinen I've applied the same querying technique to Principals and Config. I wasn't able to spot any other caching methods that would "eat whole tables", can you find any.
All tests pass locally and on circleCI except for the flickr thing.

Did you check all the search index methods? The Flickr issue is on their end, but no reply on when it might be fixed yet.

For reference: Flickr API user group


Agritin

remove, not a Palauan word
discussed and agreed by PLC

Removed. Closing. Thanks @palaulanguage


Build release wheels for tier 1 platforms with PGO

Summary
This commit adds the infrastructure to build the wheels we publish to PyPI at release time with PGO. Profile guided optimization (PGO) is a compiler technique that uses collected data about a typical execution to inform optimizations the compiler can make about building the output binary. For more details on PGO you can refer to the Rust [1] and Clang [2] documentation on the topic.
To start the only data collected is from running the qiskit unittests. This should give us some broad coverage as we'll exercise everything as we do in the unittests. This also means as we grow our Rust usage in Qiskit we'll continue to cover that code in the unittests and ensure we have coverage in PGO too. However, longer term we'll potentially want to look at dedicated PGO scripts that exercise Qiskit at larger scale than what we can do in the unittests.
Details and comments
[1] https://doc.rust-lang.org/rustc/profile-guided-optimization.html
[2] https://clang.llvm.org/docs/UsersManual.html#profile-guided-optimization

I tested the builds with this PR on my fork here: https://github.com/mtreinish/qiskit-core/actions/runs/7426837267 if people want to inspect the build logs and/or download the artifacts and test the compiled wheels. The arm64 failure is because I ran the test run before #11499 merged.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.05%) to 87.542%

Totals

Change from base Build<PHONE_NUMBER>:
-0.05%

Covered Lines:
59442

Relevant Lines:
67901

💛  - Coveralls

Have you done any performance benchmarking yet to see how much using the unit tests for PGO helps for large circuit construction and transpilation, especially now that the CircuitData stuff is in?

I haven't done anything super exhaustive. I just ran a quick example script:
import time
from qiskit import transpile
from qiskit.providers.fake_provider import FakeSherbrooke
from qiskit.circuit.library import EfficientSU2
import numpy as np

rng = np.random.default_rng(seed=12345678942)

start = time.perf_counter()
qc = EfficientSU2(100, reps=1000)
qc.measure_all()
stop = time.perf_counter()
print(f"Build time: {stop - start}")
b_start = time.perf_counter()
qc.assign_parameters(rng.random(len(qc.parameters)), inplace=True)
b_stop = time.perf_counter()
print(f"Bind time: {b_stop - b_start}")
backend = FakeSherbrooke()
t_start = time.perf_counter()
tqc = transpile(qc, backend, seed_transpiler=12342)
t_stop = time.perf_counter()
print(f"Transpile time: {t_stop - t_start}")

and used the PGO wheels from the CI run linked in https://github.com/Qiskit/qiskit/pull/11502#issuecomment-1879300239 and then built a wheel locally from 654a53051a22eaf0c933d95b543133f5cadbd918 (which was the branch point from main when the PGO test run was performed).
With PGO it returned:
Build time: 34.98029147897614
Bind time: 2.086204848019406
Transpile time: 63.79317707900191

while without PGO it returned:
Build time: 36.042047017021105
Bind time: 2.072988641972188
Transpile time: 65.34803428099258

Nothing drastic but it is a second or two faster, I've run it a few times and the speed up is roughly the same each time. I can do more testing if you have something in mind, but it'll be tricky to get this running in asv though because the only mechanism I have to really test is manually with the wheel files from that test CI run.

2.5% speedup in construction and transpile is nothing to sneeze at, really, and I imagine we'll probably get more benefit as more code gets converted to Rust.


[Question]: Why OpenAI key is always needed, even each nodes have embedding from local model?

Question Validation

[X] I have searched both the documentation and discord for an answer.

Question
I embeded every text by IngestionPipeline. But when I use VectorStoreIndex, It produces error about openai key.
from llama_index.vector_stores.faiss import FaissVectorStore
from llama_index.embeddings import HuggingFaceEmbedding

embed_model = HuggingFaceEmbedding(
    model_name="intfloat/multilingual-e5-large",
    max_length=512,
    embed_batch_size=10
)

pipeline = IngestionPipeline(
    transformations=[
        SentenceWindowNodeParser(
            window_size=4,
            sentence_splitter=splitter.split,
            include_prev_next_rel=False,
            include_metadata=True
        ),
        embed_model
    ],
)

nodes = pipeline.run(documents=documents)

d = 1024
faiss_index = faiss.IndexFlatL2(d)
vector_store = FaissVectorStore(faiss_index=faiss_index)
storage_context = StorageContext.from_defaults(vector_store=vector_store,)
index = VectorStoreIndex(
    nodes,
    storage_context=storage_context,
    show_progress=True
)

LLMPredictor.__init__(self, llm, callback_manager, system_prompt, query_wrapper_prompt, pydantic_program_mode)
     95 def __init__(
     96     self,
     97     llm: Optional[LLMType] = "default",
   (...)
    101     pydantic_program_mode: PydanticProgramMode = PydanticProgramMode.DEFAULT,
    102 ) -> None:
    103     """Initialize params."""
--> 104     self._llm = resolve_llm(llm)
    106     if callback_manager:
    107         self._llm.callback_manager = callback_manager

File ~/.local/lib/python3.10/site-packages/llama_index/llms/utils.py:31, in resolve_llm(llm)
     29         validate_openai_api_key(llm.api_key)
     30     except ValueError as e:
---> 31         raise ValueError(
     32             "\n******\n"
     33             "Could not load OpenAI model. "
     34             "If you intended to use OpenAI, please check your OPENAI_API_KEY.\n"
     35             "Original error:\n"
     36             f"{e!s}"
     37             "\nTo disable the LLM entirely, set llm=None."
     38             "\n******"
     39         )
     41 if isinstance(llm, str):
     42     splits = llm.split(":", 1)

ValueError:
******
Could not load OpenAI model. If you intended to use OpenAI, please check your OPENAI_API_KEY.
Original error:
No API key found for OpenAI.
Please set either the OPENAI_API_KEY environment variable or openai.api_key prior to initialization.
API keys can be found or created at https://platform.openai.com/account/api-keys

To disable the LLM entirely, set llm=None.
******

Hi @zzong2006 ,
This is because it expects LLM for response generation. You can mention llm=None in service_context if you don't want to use it.

I have also run into an issue where I used the "BAAI/bge-small-en" hugging face embedding model and the "models/embedding-gecko-001" google embedding model together with a PaLM LLM and yet the openai key was still required. Both the llm and the embed model were not openai models. Why is this the case? Is openai used as the default for some of the prompt chain implementations even when a different llm is set explicitly in the service context?

I guess you neêd to set global context for llm and embeddings

Hi @sandangel, Just tried it again with v0.9.15. The problem seems to be resolved. The interfaces now accept the embed_model set in the service_context.

If @zzong2006 is fine with it, this issue can be closed.


Multiple packages in one repo.

We discussed wanting a new vessel feature to support multiple packages in one repo.
In particular, we'd like to continue to use vessel but have it support

moving the base library (back) into the main motoko repository.
splitting the base library into platform-independent parts (data structures) and IC-specific parts.

To support 1 and 2, we'd like to refine how vessel works so that a package is located not only by a repo, but instead by a pair of fields, repo and path, and the path specifies where within that repo one finds the vessel config files and the src directory for the package.
If the path field is absent, it is assumed to be the empty path, meaning that the repo has only one package, at its root (the current design).
Then, we'd have the motoko repo organized like this (simplified), assuming that the compiler and base and ic directories represent the code now in the motoko repo and base repos, but reorganized:
compiler
  src
  ...
lib
  base
    src
  ic
    src

cc @rossberg @crusso

I did some initial work on a POC for this, in #38, but now I realize that assuming that every path ends with src seems somewhat artificial.  An alternative design would have that path as None Text means that the code is in src, and otherwise, the path gives the complete path to the code, including any src directory at the end of that path (or not).
I am also wondering how these changes will affect the meta data in the vessel-package-set repo.
Some experimenting indicates that the Dhall type in file src/Package.dhall may need to change, from:
{ name : Text, version : Text, repo : Text, dependencies : List Text }

to this
{ name : Text, version : Text, repo : Text, path : Optional Text, dependencies : List Text }

and likewise, the dhall meta data may need to include path : None Text for the existing packages.  My sense is that dhall is "too static" to permit any kind of field-is-absent test, and avoid this in the dhall code itself.  However, maybe the serde support via Rust will automatically do this for its inputs?  I'm still uncertain.

I realize that assuming that every path ends with src seems somewhat artificial.

Updated #38 to relax this design decision.
I am also noticing that others are affected by the src/ constraint too, and perhaps, that fact justifies the extra field for specifying each package, as writing path : None Text is not very much extra work for the common case where it is src.

I'd love to see support for this! 💯

Q: What would be the difference between the following package structures:

Importing one package from src/ and using subpaths to create 'sub-packages'.

// e.g.
// src/SubPkg1/Module1.mo
// src/SubPkg1/Class1.mo
// src/SubPkg2/ClassX/lib.mo

import Module1 "mo:pkg/SubPkg1/Module1"
import Class1 "mo:pkg/SubPkg1/Class1"
// ...
import ClassX "mo:pkg/SubPkg2/ClassX

Using the (new) proposed structure to seperate them in package-set.dhall.

// e.g.
// pkg1/src/Module1.mo
// pkg1/src/Class1.mo
// pkg2/src/ClassX/lib.mo

import Module1 "mo:pkg1/Module1"
import Class1 "mo:pkg1/Class1"
// ...
import ClassX "mo:pkg2/ClassX

Are there any advantages? Is one better than another?

I think the 2nd one is much cleaner than the first, but it just introduces another way to 'do the same'?

@di-wu, I'd think the difference is that 2 makes them two distinct packages, which means clients can install them independently, they can be versioned independently, and so on. The ability to decouple is valuable.

Great! I've got over 30 Motoko packages piled up, this'd help for sure!
Thanks for the response.

They can be versioned independently...

Vessel uses git tags for versioning. How would this work with multiple packages in one repository?

Vessel uses git tags for versioning. How would this work with multiple packages in one repository?

As an example, suppose one repo has two packages, A and B, and 50 git revisions.  Suppose that C wants to use A and B in its codebase, as imported packages via vessel.
Then, C could chose each of A and B among 50 versions (each), where any pair of versions can represent the versions for A and B .  To do so, C uses vessel to check out each of those two versions in the pair, one for A (ignoring B's path in that checkout) and one for B (ignoring A's path in that checkout).  There is no issue, just some wasted space on disk for the paths in versions not being used.
I agree that this is non-intuitive since each package somehow "deserves" an independent timeline, but must instead share one with the other packages in its repo.  But OTOH, that's what it means to share a repo.


Update from View.propTypes to ViewPropTypes to match RN v0.44.0

Update the ViewPropTypes to match the last revision of React Native V0.44.0

View.propTypes to ViewPropTypes (53905a5) - @bvaughn

https://github.com/facebook/react-native/releases

I think this may have broken the example.
I'm getting an `undefined is not an object (evaluating '_reactNative.ViewPropTypes.style')


AttributeError: 'str' has no attribute 'url'

When I run:  $roslaunch freefloating_gazebo_demo g500arm5e.launch parse:=true
I get the error: AttributeError: 'str' has no attribute 'url'
How could I solve this error?  The Gazebo and UWsim can be launched and the freefloating_gazebo demo works as well. Thank you for your help in advance!

Thanks for the feedback, solved in the last commit it was due to the legacy use of xacro without --inorder.


Improving UI of Welcome Page of Web-App

One have to use CSS with bootstrap to enhance the UI/UX

Is there any design for the Welcome Page that would help us to get started with ?

All the pages already have a basic UI.

I've created a layout design for the landing page using Google Slides.

Assets Link -
https://drive.google.com/file/d/1P7U8bslFdzgsQQa0PGogtZXDkzQDdBHD/view?usp=sharing


how to Use the frozen Model

I have the mask_frozen_graph.pb, now how to run on command line and use it?
@bendangnuksung @bendangnuksung

You can download the forward.zip provided in the comment and use the code below to inference it.
from forward import ForwardModel
import cv2
import numpy as np

frozen_graph_path = "frozen_model/mask_frozen_graph.pb"

# assign your own Config class
my_config = YOUR_CONFIG_CLASS

forward_model = ForwardModel(frozen_graph_path, my_config)

def test_one_image(image):
    images = np.expand_dims(image, axis=0)
    results = forward_model(images)
    return results

if __name__ == "__main__":
    test_image_path = 'XYZ.jpg'
    image = cv2.imread(test_image_path)
    r = test_one_image(image)
    print(r)

You can download the forward.zip provided in the comment and use the code below to inference it.
from forward import ForwardModel
import cv2
import numpy as np

frozen_graph_path = "frozen_model/mask_frozen_graph.pb"

# assign your own Config class
my_config = YOUR_CONFIG_CLASS

forward_model = ForwardModel(frozen_graph_path, my_config)

def test_one_image(image):
    images = np.expand_dims(image, axis=0)
    results = forward_model(images)
    return results

if __name__ == "__main__":
    test_image_path = 'XYZ.jpg'
    image = cv2.imread(test_image_path)
    r = test_one_image(image)
    print(r)

what is my_config? how to set that? I have your files which converts keras to tensorflow means .h5 to .pb. so how to run on command line? i have edited with path of .pb and image now my_config not defined, how to do that?

my_config is a class which contains all the hyper-parameters of the model. You can find config class in Mask_RCNN called class Config(object). In the same manner, you might have used your own config class to train the model. Assign that config class to my_config

my_config is a class which contains all the hyper-parameters of the model. You can find config class in Mask_RCNN called class Config(object). In the same manner, you might have used your own config class to train the model. Assign that config class to my_config

from forward import ForwardModel
import cv2
import numpy as np
frozen_graph_path = "/home/deepedge/mask_rcnn-master/mask_dent_frozen_graph.pb"
assign your own Config class
my_config = "/home/deepedge/mask_rcnn-master/mrcnn/config.py"
forward_model = ForwardModel(frozen_graph_path, my_config)
def test_one_image(image):
images = np.expand_dims(image, axis=0)
results = forward_model(images)
return results
if name == "main":
test_image_path = '/home/deepedge/mask_rcnn-master/mask-rcnn-bottle-training-master/dataset/val/car39.jpg'
image = cv2.imread(test_image_path)
r = test_one_image(image)
print(r)
error
$ python3 anomaly.py
2019-07-01 11:02:29.101152: I tensorflow/core/platform/cpu_feature_guard.cc:141] Your CPU supports instructions that this TensorFlow binary was not compiled to use: AVX2 FMA
2019-07-01 11:02:29.120748: I tensorflow/core/platform/profile_utils/cpu_utils.cc:94] CPU Frequency:<PHONE_NUMBER> Hz
2019-07-01 11:02:29.121268: I tensorflow/compiler/xla/service/service.cc:150] XLA service 0xe58be0 executing computations on platform Host. Devices:
2019-07-01 11:02:29.121280: I tensorflow/compiler/xla/service/service.cc:158]   StreamExecutor device (0): ,
Traceback (most recent call last):
File "anomaly.py", line 20, in
r = test_one_image(image)
File "anomaly.py", line 14, in test_one_image
results = forward_model(images)
File "/home/deepedge/mask_rcnn-master/mrcnn_serving_ready-master/forward.py", line 218, in call
molded_images, image_metas, windows = self.mold_inputs(images)
File "/home/deepedge/mask_rcnn-master/mrcnn_serving_ready-master/forward.py", line 93, in mold_inputs
min_dim=self.config.IMAGE_MIN_DIM,
AttributeError: 'str' object has no attribute 'IMAGE_MIN_DIM'
@bendangnuksung

my_config is a class which contains all the hyper-parameters of the model. You can find config class in Mask_RCNN called class Config(object). In the same manner, you might have used your own config class to train the model. Assign that config class to my_config

like this
my_config = "/home/deepedge/mask_rcnn-master/mrcnn/config.py"

@bendangnuksung
Thanks for your code. I used your successfully for detection using frozen model (.pb).
How can I save your serving ready model in .pb.? I would like to use it opencv dnn

my_config is a class which contains all the hyper-parameters of the model. You can find config class in Mask_RCNN called class Config(object). In the same manner, you might have used your own config class to train the model. Assign that config class to my_config

I got the different results when I used the frozen graph, is it normal?
https://github.com/matterport/Mask_RCNN/issues/1668

@angyee
from forward import ForwardModel
import cv2
import numpy as np
class mask_config():
def init(self, NUMBER_OF_CLASSES):
self.NAME = "tags"
self.IMAGES_PER_GPU = 1
self.NUM_CLASSES = 1 + NUMBER_OF_CLASSES  # Background + tags
self.STEPS_PER_EPOCH = 100
self.DETECTION_MIN_CONFIDENCE = 0.9
self.GPU_COUNT = 1
self.IMAGES_PER_GPU = 1
self.NAME = None  # Override in sub-classes
self.GPU_COUNT = 1
self.IMAGES_PER_GPU = 1
self.STEPS_PER_EPOCH = 1000
self.VALIDATION_STEPS = 50
self.BACKBONE = "resnet101"
self.COMPUTE_BACKBONE_SHAPE = None
self.BACKBONE_STRIDES = [4, 8, 16, 32, 64]
self.FPN_CLASSIF_FC_LAYERS_SIZE = 1024
self.TOP_DOWN_PYRAMID_SIZE = 256
self.RPN_ANCHOR_SCALES = (32, 64, 128, 256, 512)
self.RPN_ANCHOR_RATIOS = [0.5, 1, 2]
self.RPN_ANCHOR_STRIDE = 1
self.RPN_NMS_THRESHOLD = 0.7
self.RPN_TRAIN_ANCHORS_PER_IMAGE = 256
self.POST_NMS_ROIS_TRAINING = 2000
self.POST_NMS_ROIS_INFERENCE = 1000
self.USE_MINI_MASK = True
self.MINI_MASK_SHAPE = (56, 56)  # (height, width) of the mini-mask
self.IMAGE_RESIZE_MODE = "square"
self.IMAGE_MIN_DIM = 800
self.IMAGE_MAX_DIM = 1024
self.IMAGE_MIN_SCALE = 0
self.MEAN_PIXEL = np.array([123.7, 116.8, 103.9])
self.TRAIN_ROIS_PER_IMAGE = 200
self.ROI_POSITIVE_RATIO = 0.33
self.POOL_SIZE = 7
self.MASK_POOL_SIZE = 14
self.MASK_SHAPE = [28, 28]
self.MAX_GT_INSTANCES = 100
self.RPN_BBOX_STD_DEV = np.array([0.1, 0.1, 0.2, 0.2])
self.BBOX_STD_DEV = np.array([0.1, 0.1, 0.2, 0.2])
self.DETECTION_MAX_INSTANCES = 100
self.DETECTION_MIN_CONFIDENCE = 0.7
self.DETECTION_NMS_THRESHOLD = 0.3
self.LEARNING_RATE = 0.001
self.LEARNING_MOMENTUM = 0.9
self.WEIGHT_DECAY = 0.0001
self.LOSS_WEIGHTS = {"rpn_class_loss": 1., "rpn_bbox_loss": 1., "mrcnn_class_loss": 1., "mrcnn_bbox_loss": 1.,
"mrcnn_mask_loss": 1.}
self.USE_RPN_ROIS = True
self.TRAIN_BN = False  # Defaulting to False since batch size is often small
self.GRADIENT_CLIP_NORM = 5.0
    self.BATCH_SIZE = self.IMAGES_PER_GPU * self.GPU_COUNT

    # Input image size
    if self.IMAGE_RESIZE_MODE == "crop":
        self.IMAGE_SHAPE = np.array([self.IMAGE_MIN_DIM, self.IMAGE_MIN_DIM, 3])
    else:
        self.IMAGE_SHAPE = np.array([self.IMAGE_MAX_DIM, self.IMAGE_MAX_DIM, 3])

    # Image meta data length
    # See compose_image_meta() for details
    self.IMAGE_META_SIZE = 1 + 3 + 3 + 4 + 1 + self.NUM_CLASSES

assign your own Config class
my_config = mask_config(3)
frozen_graph_path = "./mask_frozen_graph.pb"
frozen_graph_path = "./saved_model.pb"
forward_model = ForwardModel(frozen_graph_path, my_config)
print(111)
def test_one_image(image):
images = np.expand_dims(image, axis=0)
results = forward_model(images)
return results
import visualize
if name == "main":
test_image_path = 'yuhao_square.png'
image = cv2.imread(test_image_path)
image = cv2.resize(image, (128, 128), interpolation=cv2.INTER_CUBIC)
print(image)
print(np.shape(image))
results = test_one_image(image)
print(results)
class_names=['BG', 'square', 'circle', 'triangle']
r = results[0]
visualize.display_instances(image, r['rois'], r['mask'], r['class'],
class_names, r['scores'])
this code can run,

@YUHAO19910716
Thanks for your comment.
How can I use the .pb model in opencv dnn? I created .pb and also tested successfully using the similar code which you provided here. Unfortunately, I can not use it in opencv dnn.
Thanks
Khalilullah


How can I add all three example player in same page?

Issue description
I want to add three audio player/playlist in same page. How can I do that? Is there any way to initialize amplitude.js by class/id name of HTML? Now, If I add three or two player/playlist in same page it play same song and if I click play/pause button for one it work for all. How can I fixed that? Suggestion please.

You could try Apls, an alternative playlist to Amplitude.js: https://one.8ue.nl/apls/docs  With Apls you can add as many playlists as you need. I'm working on placing it here in GitHub, but I first have to finish a decent player, stay tuned!

Apls minified is 5KB, which you can recover partially or all by not using Json arrays for your playlists nor extra classes for the main controls and playlists.

I am going to throw a comment on this thread as an FYI because @danpastori has been so busy with his personal life, he hasn't had much time to get back on Github issues.
Disclaimer: I am a designer, not a JavaScript developer.
Amplitude.js can support multiple playlists. @danpastori has worked on improving playlists immensely in Amplitude 3.3. A lot of it is ready on his computer, but we still need to go through testing, documentation, examples, tutorials, etc.
We've been working on coming up with a way to be more transparent with this (like publishing to a development branch so that other people can help), but again -- being in a 2 person company it gets challenging to devote a large part of your day to open source.
We'll keep you updated on the release of Amplitude.js 3.3 as soon as we have more info available!

I would welcome a development branch, @jaydrogers.

@diplopito @mamunhpath @troubleshooter I just pushed a multiple playlist example to the dev branch: https://github.com/521dimensions/amplitudejs/tree/dev/examples/multiple-playlists. Make sure to use the newest amplitude in the dev branch. I'll be merging shortly.

@danpastori thanks for the solution. But I need to know more about this like when I will initialize for every player like this:
`Amplitude.init({
"songs": [
{
"name": "Lorem Ipsum Dolor Emmet Song",
"artist": "Jhon Doe",
"album": "Lorem Ipsum Dummy Album",
"url": "/media/audios/2018/07/23/wave.mp3"
}, {
"name": "Lorem Ipsum Dolor Emmet Song",
"artist": "Jhon Doe",
"album": "Lorem Ipsum Dummy Album",
"url": "https://www.sample-videos.com/audio/mp3/crowd-cheering.mp3"
}, {
"name": "Lorem Ipsum Dolor Emmet Song",
"artist": "Jhon Doe",
"album": "Lorem Ipsum Dummy Album",
"url": "/media/audios/2018/07/23/wave.mp3"
}
],
"playlists": {
"default-playlist": []
},
});
Amplitude.init({
"songs": [
{
"name": "Lorem Ipsum Dolor Emmet Song",
"artist": "Jhon Doe",
"album": "Lorem Ipsum Dummy Album",
"url": "/media/audios/2018/07/23/wave.mp3"
}, {
"name": "Lorem Ipsum Dolor Emmet Song",
"artist": "Jhon Doe",
"album": "Lorem Ipsum Dummy Album",
"url": "https://www.sample-videos.com/audio/mp3/crowd-cheering.mp3"
}, {
"name": "Lorem Ipsum Dolor Emmet Song",
"artist": "Jhon Doe",
"album": "Lorem Ipsum Dummy Album",
"url": "/media/audios/2018/07/23/wave.mp3"
}
],
"playlists": {
"top-hidden-playlist": []
}
});`
How I should initialize playlist for each one? this is my code, what is not working. Can you please help me on this?

Hi @mamunhpath So what you'd need to do is add all of your songs to the songs array. Then you'd use the index of the songs to make your playlists. You would only initialize Amplitude once so it'd look something like this:
Amplitude.init({
"songs": [
{
"name": "Lorem Ipsum Dolor Emmet Song",
"artist": "Jhon Doe",
"album": "Lorem Ipsum Dummy Album",
"url": "/media/audios/2018/07/23/wave.mp3"
}, {
"name": "Lorem Ipsum Dolor Emmet Song",
"artist": "Jhon Doe",
"album": "Lorem Ipsum Dummy Album",
"url": "https://www.sample-videos.com/audio/mp3/crowd-cheering.mp3"
}, {
"name": "Lorem Ipsum Dolor Emmet Song",
"artist": "Jhon Doe",
"album": "Lorem Ipsum Dummy Album",
"url": "/media/audios/2018/07/23/wave.mp3"
},
{
"name": "Lorem Ipsum Dolor Emmet Song",
"artist": "Jhon Doe",
"album": "Lorem Ipsum Dummy Album",
"url": "/media/audios/2018/07/23/wave.mp3"
}, {
"name": "Lorem Ipsum Dolor Emmet Song",
"artist": "Jhon Doe",
"album": "Lorem Ipsum Dummy Album",
"url": "https://www.sample-videos.com/audio/mp3/crowd-cheering.mp3"
}, {
"name": "Lorem Ipsum Dolor Emmet Song",
"artist": "Jhon Doe",
"album": "Lorem Ipsum Dummy Album",
"url": "/media/audios/2018/07/23/wave.mp3"
}
],
"playlists": {
"top-hidden-playlist": [0,1,2]
},
"default-playlist": [3,4,5]
},
});

You'd then key your amplitude elements with the playlist like so:

or for whatever other elements you have!

When loading dynamically, you can use the addSong() and addSongToPlaylist() methods: https://521dimensions.com/open-source/amplitudejs/docs#add-song. They return the index and then you can get the id in a dynamic setting.

@danpastori Thank you very much. Hope it will be my last question: By adding song addSong() and addSongToPlaylist() how I will get the all index in playlistkey like:"default-playlist": [0-to-all] ?

Hi @mamunhpath the playlist key is returned from the method so you'd call index = Amplitude.addSong() the same with the index = Amplitude.addSongToPlaylist()


vendor.conf: docker/docker to 77c972884735...

This required revendoring docker/distribution to pick up some new dependencies and the addition of opencontainers/image-spec. Also update opencontainers/runc to b6b70e534517. All those versions taken from the newer version of docker/docker's vendor.conf.
Signed-off-by: Ian Campbell<EMAIL_ADDRESS>

Needing to vendor the gorilla libraries in order to use the docker API client is a little ridiculous. I'm going to look into this upstream.
LGTM for now

I'll redo this using moby/moby#33237.

I see this is in the 17.06 milestone. Is it really connected to that release?

@aaronlehmann I don't know, I don't see why it would be unless there is some desire to have things approximately line up aroundabout a release (personally I did this for https://github.com/docker/swarmkit/pull/1965 rather than any such desire). @nishanttotla ?

Edited the top comment and title to reflect the force push which no longer pulls in the gorilla stuff.

LGTM


scripts/websitepreprocess.sh,docs/*: Fix broken links

Changes:

Up to now, all the relative links were replaced in the scripts/websitepreprocess.sh by generating the URL for the website to correctly parse the links, that always added the /docs/ dir in the path, which did not work for some of them (e.g. Makefile link below) as they were located outside of that directory.
Added small doc note on how to run the website locally, let me know if that is useful? Saves a small amount of time for not needing to go look at the Makefile :)
Some other small cleanups.
The go.sum change was automatic when I ran go mod vendor, but can remove it if you are getting a different result. 🤷♀

Verification
Tested all the links and they resolve both in markdown on github and by running the website locally.
As a side note/question, there are some tools out there that can be added as part of the CI process to test the links and make sure they are valid.

p.s. Great docs! <3

so I am bit surprise the broken links you mentioned were not caught up. Which one were not working?

It all started with the Makefile link here -> https://thanos.io/getting-started.md/#prometheus

so I am bit surprise the broken links you mentioned were not caught up. Which one were not working?

I think the key part is we test links for markdown not from website perspective (: We may want to add that one in the future, some issue would be nice for this (:

We may want to add that one in the future, some issue would be nice for this (:

Opened an issue for that #1177 :)
Will address the other comments later, thanks for your quick response!

@bwplotka done with one question, PTAL, thanks!


drop_vars; deprecate drop for variables

Introduces drop_vars, and deprecates using drop for variables. drop is widely used for the deprecated case, so this is a fairly wide blast radius.
It's more churn than is ideal, but I do think it's a much better API.
This is ready for review, though I'm sure I'm missed references in the docs etc (took my peak regex skills to find/replace only the deprecated usages!)
Originally discussed here

[x] Tests added
[x] Passes black . && mypy . && flake8
[x] Fully documented, including whats-new.rst for all changes and api.rst for new API

What do you think about this alternative?

Add drop_sel for dropping labels
Add drop_vars for variables.
Keep around drop as an overloaded alias for both?

(I'm not really sure which is best, to be honest. But keeping down churn does seem like a good idea, if possible.)

Yes, that sounds reasonable—I think we move documentation and internal usages to drop_sel and drop_vars, and leave drop as a backward compatible option?
Maybe at last a case for FutureDeprecationWarning :)

Updated! I used PendingDeprecationWarning, which is default disabled.

@crusaderky let me know if you know off-hand why the MinimumVersionsPolicy is breaking, otherwise I can look into it

Add drop_sel for dropping labels

Would drop_labels be better?

Would drop_labels be better?

I think either is reasonable. drop_sel has the advantage that it's easily comparable to sel, and has the same arguments, so I'd marginally vote for that. Am easily swayed by popular opinion!

I wanted a name that is reminiscent of "sel" in some way.
On Wed, Nov 6, 2019 at 7:14 AM Deepak Cherian<EMAIL_ADDRESS>wrote:

Add drop_sel for dropping labels
Would drop_labels be better?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/pydata/xarray/pull/3475?email_source=notifications&email_token=AAJJFVVMPYUZS37B4BW2C43QSLNOPA5CNFSM4JHQAN7KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEDG4BNQ#issuecomment-550355126,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAJJFVQKLRALMGU5ALWKWVLQSLNOPANCNFSM4JHQAN7A
.

:+1:

This is ready to go!
As a reminder, we have PendingDeprecationWarning for drop, which is ignored in user code by default. It does appear in tests, so let me know if that's too strong given the above discussion on backward-compat.

Going to merge given @dcherian 's approval, any other follow-ups I can address in another PR

Thank you @max-sixty, and @dcherian for reviewing!


fix: issue when using flutter_gen package in Very Good Core

Description
When the app was created with
very_good create flutter_app new_app_name --description "Random description" --org-name "123dev.happy.newapp"
The command dart pub get has an issue and it solves when I delete   flutter_gen: any from the pubspec.yaml
Steps To Reproduce
This is the terminal response
D:\Flutter\bin\flutter.bat --no-color pub get
Resolving dependencies...

color 3.0.0
dart_style 2.3.4
dartx 1.2.0
flutter_gen 5.4.0
flutter_gen_core 5.4.0
intl 0.18.1 (0.19.0 available)
js 0.6.7 (0.7.1 available)
json_annotation 4.8.1
leak_tracker 10.0.0 (10.0.4 available)
leak_tracker_flutter_testing 2.0.1 (3.0.3 available)
leak_tracker_testing 2.0.1 (3.0.1 available)
meta 1.11.0 (1.12.0 available)
petitparser 6.0.2
test 1.24.9 (1.25.2 available)
test_api 0.6.1 (0.7.0 available)
test_core 0.5.9 (0.6.0 available)
time 2.1.4
vm_service 13.0.0 (14.0.0 available)
xml 6.5.0
Changed 9 dependencies!
10 packages have newer versions incompatible with dependency constraints.
Try flutter pub outdated for more information.

Screenshots

My very_good version is 0.19.1

Hi @SamuelAc1011 thanks for opening an issue!
When I run:
very_good create flutter_app new_app_name --description "Random description" --org-name "123dev.happy.newapp"

My output is:
"123dev.happy.newapp" is not a valid org name.

A valid org name has at least 2 parts separated by "."
Each part must start with a letter and only include alphanumeric characters (A-Z, a-z, 0-9), underscores (_), and hyphens (-)
(ex. very.good.org)

If instead I run:
very_good create flutter_app new_app_name --description "Random description" --org-name "dev.happy.newapp"

My output is:
✓ Generated 314 file(s) (4.6s)
✓ Running "flutter packages get" in .  (2.3s)
✓ Running "dart fix --apply" in ./new_app_name (2.3s)

Created a Very Good App! 🦄
  • To get started refer to new_app_name/README.md
  • Your project code is in new_app_name

The host operating system I'm using is macOs 14.0 (23A344).

What is your host operating system? I think you are using Windows? If so, what Windows version?

Hi, I don't if this issue is correct or I have to open a new one. By the way, the usage of flutter_gen:any in pubspec file create issue with intl internationalization. Indeed, the import 'package:flutter_gen/gen_l10n/app_localizations.dart'; is not correctly recognized. It seems that the flutter_gen in pubspec shadows in some way the intl-generated package under .dart_tool.
The way to reproduce is simply create a new flutter app with very good cli.
I am on very_good_cli 0.20.0

@SamuelAc1011 @dariocast I ran into this and was able to resolve by making the prescribed changes from flutter_gen readme: https://github.com/FlutterGen/flutter_gen?tab=readme-ov-file#error-with-internationalization

I'm not sure if I'm missing something trivial here, but given the reproductive steps of the issue:

Create a Very Good Core application using Very Good CLI 0.20.0

very_good create flutter_app my_app

Run flutter pub get (from my_app):

flutter pub get

Open the application (from my_app):

flutter run --target lib/main_development.dart -d chrome

I do not get any errors, and I'm not able to reproduce the issue.
As far as I am concerned there is no flutter_gen dependency being specified in Very Good Core's pubspec.yaml.
I suspect you've manually added this to the pubspec.yaml after the generation. If that's the case, I think @aymswick already provided solutions given the scenario where you manually included the flutter_gen package into your created application. Hence, I think this is an issue regarding your invalid usage of flutter_gen rather than the code being generated by Very Good CLI.
I'll be closing this issue, feel free to re-open if you think the issue still holds and is related to Very Good CLI. If you think the issue is related to the generated code consider opening the issue at Very Good Templates instead.

For posterity, I definitely run into this issue occasionally when I set up new projects and I know i don't go manually adding flutter_gen to the pubspec - it seemingly has something to do with either vscode, very_good_cli, or the flutter create commands itself, but since localizations is something very_good_cli sets up, it would make sense that something weird is happening in that domain. The steps to reproduce are unknown to me at this point other than just creating a new flutter_app from the very_good_cli and watching your app/view/app.dart and l10n files turn red to indicate an error. The error provided by the linter says something like "the uri for the package doesnt exist" when referencing the synthetic package produced by intl/localizations/flutter_gen:
import package:flutter_gen/gen_l10n/app_localizations.dart


docs(deploy): add stoat preview

This is the doc update I plan to propose to Vite.
Depend on https://github.com/stoat-dev/stoat-action/pull/115.

Looks good. We should be able to use this as a template for other doc additions

Submitted the PR to Vite here: https://github.com/vitejs/vite/pull/11632.


For a 0.21.2 release

Fix bug: MLJBase.depwarn should have been Base.depwarn in redefinition of deprecated @from_network. To address fail at https://github.com/OutlierDetectionJL/OutlierDetection.jl/pull/36

Codecov Report

Merging #868 (cd2466c) into master (a85f959) will decrease coverage by 0.51%.
The diff coverage is n/a.

:exclamation: Current head cd2466c differs from pull request most recent head d79f29b. Consider uploading reports for the commit d79f29b to get more accurate results

@@            Coverage Diff             @@
##           master     #868      +/-   ##
==========================================
- Coverage   85.24%   84.73%   -0.52%
==========================================
  Files          40       40
  Lines        3558     3543      -15
==========================================
- Hits         3033     3002      -31
- Misses        525      541      +16

Impacted Files
Coverage Δ

src/composition/models/deprecated_from_network.jl
94.38% <ø> (ø)

src/measures/loss_functions_interface.jl
59.25% <0.00%> (-13.96%)
:arrow_down:

src/measures/confusion_matrix.jl
87.38% <0.00%> (-4.51%)
:arrow_down:

src/measures/continuous.jl
89.58% <0.00%> (-3.01%)
:arrow_down:

src/measures/probabilistic.jl
61.64% <0.00%> (-1.52%)
:arrow_down:

src/utilities.jl
83.54% <0.00%> (-1.27%)
:arrow_down:

src/machines.jl
90.04% <0.00%> (-0.42%)
:arrow_down:

src/measures/finite.jl
98.58% <0.00%> (-0.02%)
:arrow_down:

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


no-recover added to disable auto recover

Signed-off-by: kevin xu<EMAIL_ADDRESS>

Current coverage is 44.32% (diff: 0.00%)

Merging #103 into master will increase coverage by 1.44%

@@             master       #103   diff @@
==========================================
  Files            16         16
  Lines          8757       8760     +3
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
+ Hits           3755       3883   +128
+ Misses         4035       3870   -165
- Partials        967       1007    +40

Powered by Codecov. Last update 59c85ea...3c32d97


projections does not work with root class with typed parameters like Collection or Map

Let's assume that I have a class called Identifier, and a serializer for that class called IdentifierSerializer, which serializes the Identifier to a String.
Now inside of the class representing the documents of a collection, there is a field of the type Identifier, e.g.:
class MyDocumentClass(
    val identifiers: List<@Serializable(with = IdentifierSerializer::class) Identifier>,
    //... more fields, therefore the projection function makes sense
)

Now I want to use projection in the following way:
val currentIdentifiers: List<Identifier> = db.myCollection.projection(
    MyDocumentClass::identifiers,
    theQuery
)

Now to the bug:
Somewhere inside of the projection call, there has to be an unsafe cast. Because currentIdentifiers is actually a List<String> which it should not be - its type even is List<Identifier>.
Obviously, the custom serializer has not been used for deserialization of the returned array members in the database, instead projection did just return the actual raw content of the array in the database (which is the output of the serialize function of the custom serializer, which has been used correctly earlier on) and casted it to the desired type in an unsafe way.
An example for what would fail now:
currentIdentifiers.map { it.namespace }

This of course fails, because String does not have a property called namespace.
This should be fixed, because typesafety has been lost in this case.

Thank you for reporting. The problem is that projection does not support (for now) type parameters for their root class.
As a workaround, I suggest to use a custom dedicated class for projection as specified in the doc https://litote.org/kmongo/extensions-overview/#projection . Look at the attached unit test.


Configure separate timeouts for Predictor, Transformer and Explainer

Explanation examples can break due to low revision timeout.
Fixes #490

This change is

These timeouts are hard to debug. See https://github.com/knative/serving/issues/2528
I suggest a long timeout now and then we update to service specific timeouts or config a user can set.

Also note. We could have a separate explainer timeout which is high. But the explainer send batch requests and for example batches of 25 imagenet classifications (over REST) to a tensorflow imagenet model were taking 35s approx on my cluster so that was timing out.
We could have 2 timeouts (explainer and rest) but we should increase prediction timeout due to batch requests.

From discussion with @yuzisun mentioned that scale down time is also based on Revision timeot.
So will update to have 3 timeouts.

I think a 10 minute timeout is probably acceptable. This is inline with
other serverless timeouts like lambda or gcp ai platform.
We just need to make sure that this doesn't cause workloads to hang around
for too long.
On Sat, Oct 26, 2019, 8:22 AM cliveseldon<EMAIL_ADDRESS>wrote:

Also note. We could have a separate explainer timeout which is high. But
the explainer send batch requests and for example batches of 25 imagenet
classifications (over REST) to a tensorflow imagenet model were taking 35s
approx on my cluster so that was timing out.
We could have 2 timeouts (explainer and rest) but we should increase
prediction timeout due to batch requests.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/kubeflow/kfserving/pull/491?email_source=notifications&email_token=AATXLCHLUWPH67GZQXXHOQDQQRODXA5CNFSM4JFNSI3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOECKKHIA#issuecomment-546612128,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATXLCDRDN6GLHFAYDRYNTTQQRODXANCNFSM4JFNSI3A
.


addImage with not working with node version

tried to run the node-example with the slide.addImage with an http link for the path:
slide.addImage({ 	path: "https://upload.wikimedia.org/wikipedia/en/a/a9/Example.jpg", 	x: 1, 	y: 1, });
and got the following error:
pptxgenjs ver: 2.3.0-beta.20180820
ERROR....: Unable to read media: "https://upload.wikimedia.org/wikipedia/en/a/a9/Example.jpg"
DETAILS..: Error: ENOENT: no such file or directory, open 'https://upload.wikimedia.org/wikipedia/en/a/a9/Example.jpg'
passing a relative path to a local image works,  but passing an http link does not.

Hi @stevedorke
Node encoding for remote images was the sole remaining feature for the new path features introduced recently.
The final code is checked in and this URL loads successfully now.


Move pull_policy to pull.policy

This moves pull policy under the pull key in the settings file.
This will result in the next release being a major as it is a breaking change.
Fixes #951

ansible-language-server followed the similar for pull-policy as that of ansible-navigator.
https://github.com/ansible/ansible-language-server#language-server-settings
Should we do a similar change in ALS as well? It will require a major version change for both ALS and extension.

ansible-language-server followed the similar for pull-policy as that of ansible-navigator. https://github.com/ansible/ansible-language-server#language-server-settings Should we do a similar change in ALS as well? It will require a major version change for both ALS and extension.

I'd say if we were going to do it, we should do it fast before there is broad adoption of using EEs with both.  I'm guessing it is still low.
Maybe we could time it with a navigator release and the lint 6 release?


Using Distinct in Linq Query

Is your feature request related to a problem? Please describe.
We use Linq extensively in our project for Sql queries as it is typesafe and object oriented. Going through documentation there doesn't seem to be any translation for distinct in linq to sql query.
Describe the solution you'd like
A feature like below would be appreciated:
IEnumerable<string> regions = query
                .Where(x => x.Ring == ring)
                .Select(x => x.Region)
                .Distinct()
                .AsEnumerable<string>();

should translate to
SELECT distinct c.Region from c where c.Ring='value';

There is an extension method called ToFeedIterator that will allow you to execute the query.
https://github.com/Azure/azure-cosmos-dotnet-v3/blob/07cca6e3bb54e7cffc360355bc75d5d988e6a60d/Microsoft.Azure.Cosmos/src/Resource/Container/Container.cs#L1250
// LINQ query generation
using (FeedIterator<string> setIterator = container.GetItemLinqQueryable<Book>()
                    .Where(x => x.Ring == ring)
                    .Select(x => x.Region)
                    .Distinct()
                    .ToFeedIterator<string>())
{
    //Asynchronous query execution
    while (setIterator.HasMoreResults)
    {
        foreach(var region in await setIterator.ReadNextAsync())
        {
            Console.WriteLine(region );
        }
    }
}


Add full functionality of classnames package to classList

This allows classList to accept an array of classNames, omitting falsey values (except for 0, reasoning described in comment).
This will allow users to put optional classNames in className, and not have to worry about omitting an unset value. The following shows the common usecase of passing through a classname:
// BEFORE
function Component({ className }) {
  return <div className={`component-class-name ${className || ""}`}> // nit: this will have an extra space and removing it is much more verbose
    ...

;
}

// AFTER
function Component({ className }) {
  return

    ...

}

This is fully backwards compatible and accepts objects & nested arrays:
function Collapsible({ className, classList }) {
  const [collapse, setCollapse] = createSignal(false);

  return

    ...

;
}

I've been slow on this one (and it has come up before) mostly in that we need to apply the support in at least 4 places including client and server side compiler and runtimes. For the most part I skip using these helpers if the compiler can analyze what we need(ie.. if it is a literal in the binding we know that it's fixed if there are no spreads or no computed properties).
But unclear if this array form can apply the same compiler level optimization. Probably if we inline a conditional. Like right now I don't optimize {[className]: true} which is pretty much identical to this. It's a bit unfortunate if arrays are just de-opt in general but maybe we can do something about this.
One thing I wasn't clear about on this part of the solution was does it handle unsetting properly and with arrays do we have to consider order? Like if someone passed in [aString(), anObject()] or even a anArray() that could apply class rules differently like if the array was made new each time with arguments in different orders.

But unclear if this array form can apply the same compiler level optimization.

Hmm I'm not familiar with how the compiler handles classList, can you please point me in the right direction?

One thing I wasn't clear about on this part of the solution was does it handle unsetting properly and with arrays do we have to consider order?

Regarding order, I'm not sure if I understand the issue, but this current solution uses existing functionality that clears and rewrites the dom classList which should work regardless of order.

Now that I write these 2 scenarios down, the first one seems to me like the way to go, I don't imagine the second will have any significant performance benefit, but it does add plenty of complexity

Hi, are there any updates on this?


New entity type: B - Being

We need animals as entity type for cases where they act, i.e. assume actant roles, in ways different than objects. For some projects e.g. literary texts with knights and valiant horses, they will be even more important than for DISSINET (so they are a way to project neutrality). We discussed with Adam that better to implement them right away.
Please implement the B entity type (A for Animal is taken by Action, so we must use B - Being or Beast or Bestia). The B abbreviation already used since today in Action types list, needs to match with this new entity type. (This is why I also tag @tomaham.)

@davidzbiral For B-Being, do we store logical type information (definite, indefinite, hypothetical, and generic options) like for P-Persons?

@jancimertel please check the branch entity-class-B-#1261 and if everything ok, please merge

@adammertel Yes.

merged https://github.com/DISSINET/InkVisitor/pull/1275


SQL Injection using Sqlmap

Topic Suggestion
Sql Injection using Sqlmap
Writing sample(s):
https://www.section.io/engineering-education/testing-for-broken-authentication-in-web-apps/
https://www.section.io/engineering-education/getting-started-with-burpsuite/
Proposal Submission
Proposed title of article
Sql Injection using Sqlmap
Proposed article introduction
[Security] This article seeks to introduce the reader to Sql Injection and Sqlmap, which is a tool for finding sql vulnerabilities in web applications.
Key takeaways
What are the 3-5 most important things the reader should understand or be able to do after reading this article?
Use this area to get your ideas down on the bulk of your article or tutorial.
After reading this article, the reader will understand:

What Sql Injection is
How to find sql vulnerabilities using Sqlmap
Impact and prevention of sql vulnerabilities

Article quality
This article will explain to the reader what sql vulnerabilities are, and also demonstrate how to find this vulnerability using a tool called sqlmap. I'll be walking the readers through the process of using the tool and methods to protect web application from this vulnerability
References
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

This topic is oversaturated on numerous blog sites and official docs. I rejected it before.


Migrate type structures from stage 1 to stage 2

This is going to be fun.  😄

#31 blocks completing this for InferTypeStructureImpl, FunctionTypeStructureImpl, and MappedTypeStructureImpl.


Made visible consul icon of catalog filters compatibility section in light mode

Signed-off-by: yash sharma<EMAIL_ADDRESS>Description
Consul icon is now visible in light mode also
This PR fixes #1013
https://user-images.githubusercontent.com/71271069/218142865-ae0604ca-e981-40f1-80e5-c324037ba7aa.mov
Notes for Reviewers
Signed commits

[ ] Yes, I signed my commits.

Hey @Shivam-AfA check this PR let me know if there is any changes

Hey @Nikhil-Ladha this is a small change and I think it is ready to merge

I am not exactly sure how we are checking for the current theme on the meshery.io site, though if you have gone through the code and figured out that then do you mind updating the kuma logo to its white version in dark mode?

I am not exactly sure how we are checking for the current theme on the meshery.io site, though if you have gone through the code and figured out that then do you mind updating the kuma logo to its white version in dark mode?

Hi @Nikhil-Ladha , I don't think  we are checking the theme for changing logos. Currently the logos remain the same for both  modes , this PR just replaces the white consul logo with the colored one. This issue also needs to address the same problem of switching logos according to mode, do let me know if you want me to  look into addressing a proper fix for this ,thanks

sure let me look into it @Avi-88

merge conflicts @Yashsharma1911


Is it possible to retrieve layer names before rendering output?

Hi,
I am trying to develop an application which i can change assets and render different videos but i also want to learn existing layer names without opening the project. Can i retrieve existing layer names in a ae project without running after effects? I know it is possible with extendscript toolkit but i want to run the script from the same nodejs application which i use nexrender in and without running after effects. Is it possible with nexrender or do you have any suggestions for that?
Thank you,

You could try learing more, or maybe even using these projects:

https://github.com/actumn/aepx.js
https://github.com/inlife/aftereffects-project-research


CPS procs vs CPS blocks

Just to start a discussion on this, not saying one is better then the other, or that they are exclusive.
With the current CPS procs, each cps 'thread' needs a dedicated proc, so for example doing tcp sockets, you get something like
proc doClient(clientSock) {.cps.} =
   while true:
      cps wait-for-io(clientSock)
      data = read(clientSock)

proc doServer() {.cps.} =
  let serverSock = tcpBindAndListen()
  while true:
    cps wait-for-io(ServerSock)
    let clientSock = accept(serverSock)
    runLater doClient(clientSock)

runLater doServer()

the original paper however uses nestable blocks to spawn threads, which in Nim could look something like this, which also has a nice flow to it:
proc doServer(serverSock) =
  let serverSock = tcpBindAndListen()
  cps:
    while true:
      cps wait-for-io(serverSock)
      let clientSock = accept(serverSock)
      cps:
         while true:
            cps wait-for-io(clientSock)
            let data = read(clientSock)

We will certainly support adhoc cps blocks, but I'm not sure I understand the bit about dedicated procs.  Once you're inside any cps proc, you're golden.  You don't need to "cpsify" subsequent code.

Well, the cps-ification as in example 1 generates a new thread/task (how will we call these things?!) and pokes it on the scheduler with 'runLater' (whatever that may be) - how would you express creating a new flow in a cps block? Or is this where your fork comes in?

The fork we currently have should be thought of as a clone.  A new fork, let's say beget could work like posix's fork() and give you the means to determine whether you are in the parent or the child.  This can be accomplished with our semaphore...

Also pending #20


Add new service factory methods

This PR adds new factory methods that do not use a top-level gcloud object or the Google::Cloud factory methods. It enables the following:
require "google/cloud/bigquery"

bigquery = Google::Cloud::Bigquery.new
dataset = bigquery.dataset "my_dataset"
table = dataset.table "my_table"

Thoughts? Is this the change that is wanted?
[refs #907, #927]

ping @omaray @jgeewax

Can I get some input on the code examples? Currently, all code examples show the google/cloud require, and create a gcloud object:
require "google/cloud"

gcloud = Google::Cloud.new
bigquery = gcloud.bigquery

data = bigquery.query "SELECT name FROM my_table"
data.each do |row|
  puts row["name"]
end

We would like to change (almost) all code examples to use the google/cloud/bigquery require, and call Google::Cloud::Bigquery.new instead of creating a gcloud object:
require "google/cloud/bigquery"

bigquery = Google::Cloud::Bigquery.new

data = bigquery.query "SELECT name FROM my_table"
data.each do |row|
  puts row["name"]
end

@jgeewax Is this also a change that you are okay with? The approach used in the current code examples will continue to work.

Since every example in google-cloud-ruby is "complete" (starts with the require), one benefit of changing the examples to the new method is that we remove a line of boilerplate from each example.

@blowmage Issues #907 and #927 have been open for 9 and 17 days respectively. Your request for feedback on the solution in this PR has been open for 2 days. When do you want to remove the feedback wanted label?

Well, we did discuss this at standup. So we did get some feedback, just not on the issue. I guess we should remove the label before the pr is merged?

The READMEs examples for the individual packages still show require "google/cloud" and the older class methods. I think they should be the same as the headline examples in the guides.

@quartzmo I pushed a commit that updates the README files to use the new factory methods.


Standard Filters for custom field

I want to add filters for amount (custom field) in Model View
class FarmerAdmin(ModelView):

    column_descriptions = dict(amount = "All Transactions Total (Rs.)")

    column_searchable_list = ('name','village','phone')

    column_editable_list = ('phone',)

    column_filters = ('village','area',\
                  filters.FloatGreaterFilter('amount','Amount'))

    inline_models = (Transaction,)

    def get_query(self):
        return self.session.query(
            Farmer.id.label("id"),
            Farmer.name.label("name"),
            Farmer.village.label("village"),
            Farmer.phone.label("phone"),
            Farmer.area.label("area"),
            func.sum(Transaction.total).label("amount")
            ).join(Transaction).group_by(Farmer.id)

    def scaffold_sortable_columns(self):
        return {'id':'id','name':'name','village':'village',\
            'area':'area','amount':'amount'}

    def scaffold_list_columns(self):
        return ['id','name','village','phone','area','amount']

    def __init__(self, session):
        super(FarmerAdmin, self).__init__(Farmer, session)

Sort works great by overriding scaffold_sortable_columns method. Is there a way to add standard filters (greater than, less than) for amount column as well.

It's probably simpler to add the amount to your SQLAlchemy model as a hybrid property. It will do the calculations and you can treat the column like any other columns from the database.
There's info about implementing a hybrid property here: http://docs.sqlalchemy.org/en/rel_1_0/orm/extensions/hybrid.html

Thanks for help. It works with sort but I am getting another error for filters.
    self._filters = self.get_filters()
  File "C:\Users\Shrirang Mundada\Envs\venv\lib\site-packages\flask_admin\model\
base.py", line 834, in get_filters
    flt = self.scaffold_filters(n)
  File "C:\Users\Shrirang Mundada\Envs\venv\lib\site-packages\flask_admin\contri
b\sqla\view.py", line 585, in scaffold_filters
    columns = self._get_columns_for_field(attr)
  File "C:\Users\Shrirang Mundada\Envs\venv\lib\site-packages\flask_admin\contri
b\sqla\view.py", line 316, in _get_columns_for_field
    if (not field or
  File "C:\Users\Shrirang Mundada\Envs\venv\lib\site-packages\sqlalchemy\sql\ele
ments.py", line 539, in __bool__
    raise TypeError("Boolean value of this clause is not defined")
TypeError: Boolean value of this clause is not defined

Here's  my updated code.
class Farmer(db.Model):
    id = db.Column(db.Integer, primary_key = True, autoincrement = True)
    name = db.Column(db.String(120))
    village = db.Column(db.String(80))
    area = db.Column(db.Integer)
    phone = db.Column(db.String(10))

##    amount = column_property(select([func.sum(Transaction.total)]).\
##                             where(Transaction.farmer_id == id))

    @hybrid_property
    def amount(self):
        if self.transactions:
            return sum([transaction.total for transaction in self.transactions])

    @amount.expression
    def amount(cls):
        return select([func.sum(Transaction.total)]).\
               where(Transaction.farmer_id == cls.id).\
               label("amount")

class FarmerAdmin(ModelView):

    column_list = ('id','name','village','phone','area','amount')

    column_display_pk = True

    column_descriptions = dict(amount = "All Transactions Total (Rs.)")

    column_searchable_list = ('name','village','phone')

    column_editable_list = ('phone',)

    column_sortable_list = ('id','name','village','area','amount')

    column_filters = ('village','area','amount')

    inline_models = (Transaction,)

@shmundada93 I just created a pull request to fix this issue with hybrid properties. You can test it here: pip install -e git+https://github.com/pawl/flask-admin.git@fix-hybrid-properties#egg=flask-admin
There's also an example here: https://github.com/pawl/flask-admin/commit/08305dd03d2bac67ca5e3f4a1e6dce84d0f00d4a#diff-62f1bc56fcb287804ec8dfeff4e75f48R1


App Crashes in week five of off season workout

🐛 Bug Report
Environment
Tried on Android (Samsung Galaxy S9, One UI 2.5, Android 10) and iPad
Steps to Reproduce

Expected Behavior

Actual Behavior

Hi Dan
I'm going to investigate the issue, it should be easy to fix
Thanks for the report!


fapbot/php-cs-fixer is abandoned, yet it's in the readme

In README.md:

$ ./composer.phar global require fabpot/php-cs-fixer

Meanwhile: https://packagist.org/packages/fabpot/php-cs-fixer

This package is abandoned and no longer maintained. The author suggests using the friendsofphp/php-cs-fixer package instead.

Hi!
Thanks for reporting!
Currently the PHP-CS-Fixer is being moved from fabpot to FriendsOfPHP on packagist (its been done on github for a while). This will be finalized when 2.0 is released as stable. The warning you get is just a bit premature.
For ref.: https://github.com/FriendsOfPHP/PHP-CS-Fixer/pull/2006 (and others)

This came up for me because ./composer.phar global require fabpot/php-cs-fixer does not work on Windows; running it results in "Could not find package fapbot/phab-cs-fixer at any version for your minimum-stability (stable)". Meanwhile, friendsofphp/php-cs-fixer worked.

"Could not find package fapbot/phab-cs-fixer at any version for your minimum-stability (stable)"

who is fapbot and what is a phab-cs-fixer? ;)

Good questions.


part-1 readme.md link is broken

Readme links to BCM2837 datasheet:
https://web.stanford.edu/class/cs140e/docs/BCM2837-ARM-Peripherals.pdf
link is broken

Thank you @arcturus140 - apologies for the delay. This has now been fixed.
Thanks for reporting.