Test failures when downloading Blender

Recent tests have all failed when trying to download Blender.
https://circleci.com/gh/KhronosGroup/glTF-Blender-IO
/cc @pjoe Any ideas?

I did a PR #673  to change the timeout to 20 min :S
Obviously not ideal solution - IIRC the blender devs mentioned that download.blender.org doesn't have CDN while builder.blender.org has (which is why this is only an issue for 2.79).

Thanks @pjoe for fixing the download timeout.
There was another odd error, an unhandled promise rejection, which I've fixed in the test suite in commit 842e4b04b130.
The remaining test failures appear to be very serious:  The Blender 2.81 build isn't exporting any texturemaps from several of the round-trip test models.    Blender 2.79 and 2.80 are able to round-trip these models without any problem.

Closing this, discussion will continue in #674.


Find in BPMN view deactivated in Modeler 3.1.0

I can no longer search for elements using Cmd-F in Diagram view. The Edit -> Find option is  disabled once I hit Cmd-F, but doesn't show the search bar. This seems to be a new problem.
Even more disturbing in combination with the ongoing problem that I cannot copy and paste in the XML view. Now I have to manually type the activity id from the exception message into the XML view to find problems in a diagram. Please bring copy & paste in XML back - is has been reported a while ago.

I wasn't able to reproduce it on macOS 10.14:

Are there any errors printed inside the Log Panel / the Developer Tools console / in the Application Log . Last option you're able to find in your local Library/Logs/Camunda Modeler directory.
The last thing you mentioned is disabled for a while ago. As you say, it is already reported (cf. https://github.com/camunda/camunda-modeler/issues/868), but seems to be not in our current roadmap.

Hi @pinussilvestrus,
thanks for the reply. After a few modeler restarts, I can also not reproduce the issue anymore. Maybe that was some weird edge case. The log doesn't show anything unusual.
I think we can close this issue for now, I'll reopen it once it happens again and I find a way to reproduce.
Btw: Since 3.0, I keep getting uncaught exception messages when closing the modeler from time to time (at least once a week). This is unrelated to this issue, but another thing I haven't been able to reproduce yet.

Thanks for the clarification! Yes, feel free to re-open if the error occurs again (same thing with the issues when modeler closes) 👍


500 Internal Server Error

I am getting this error while using POST in postman. Please help

Same

Be sure that raw type is json not text

Be sure that raw type is json not text

I worked by adding app.config['JSONIFY_PRETTYPRINT_REGULAR'] = False to blockchain.py.
# Instantiate the Node
app = Flask(__name__)
app.config['JSONIFY_PRETTYPRINT_REGULAR'] = False

Thanks @teruuuuuu ! Your solutions worked for me too.


Converted the class based style in demo to function/hooks

Thanks I will merge this in but after I add back the commented stuff as I use that to test things ;) I'm working on something else at the moment so it will take some time before I get around to it

Thanks happy to merge but there seems to be a conflict there ^

Thanks happy to merge but there seems to be a conflict there ^


XS✔ ◾ Updated mermaid script

❌ Add a description.

@bilalbokharee Image is available for testing. docker pull stakater/workshop-exercise:SNAPSHOT-PR-159-76b884b8


Unsupported video codec  H265

You have to use Safari web browser on Mac OS and enable the experimental h265 WebRTC support in feature flags. Then it will work, so I believe the condition is correct.


Roadmap.

This issue provides a rough roadmap to give you an idea of the development status, and the remaining planned work.
Roadmap
Currently Implemented

[x] Basic and intermediate queries.
[x] All mutations.
[x] Query minification before network transfer.
[x] Scalars.

[ ] Improved support (https://github.com/shurcooL/githubql/issues/9).

[x] Specifying arguments and passing variables.
[x] Thorough test coverage.

[x] Initial basic tests (hacky but functional).
[x] Better organized, medium sized tests.

[x] Aliases.

[ ] Documentation.
[x] Improved support.

[x] Inline fragments.

[x] Documentation.

[x] Generate all of objects, enums, input objects, etc.

[x] Clean up GitHub documentation to pass golint.

[x] Unions.

[x] Functional.
[x] Improved support (https://github.com/shurcooL/githubql/issues/10).

[x] Directives (haven't tested yet, but expect it to be supported).
[ ] Research and complete, document the rest of GraphQL features.
[ ] Fully document (and add tests for edge cases) the graphql struct field tag.
[x] Extremely clean, beautiful, idiomatic Go code (100% coverage, 0 lines of hacky code).

[x] Document all public identifiers.
[x] Clean up implementations of some private helpers (currently functional, but hacky).

Future

[x] GitHub Enterprise support (when it's available; GitHub themselves haven't released support yet; GitHub has released it now). Using issue #23 to track this feature.
[ ] Frontend support (e.g., using githubql in frontend Go code via GopherJS).
[ ] Local error detection (maybe).
[ ] Calculating a rate limit score before running the call.
[ ] Avoiding making network calls when rate limit quota exceeded and not yet reset.
[ ] Support for OpenTracing.

Known Unknowns

Whether or not the current API design will scale to support all of advanced GraphQL specification features, and future changes. So far, things are looking great, no major blockers found. I am constantly evaluating it against alternative API designs that I've considered and prototyped myself, and new ones that I become aware of.
I have only explored roughly 80% of the GraphQL specification (Working Draft – October 2016).
Performance, allocations, memory usage under heavy workloads in long-running processes.
Optimal long-term package/code layout (i.e., whether to split off some of the parts into smaller sub-packages).

Good news, I've just checked off the following box:

[x] Directives (haven't tested yet, but expect it to be supported).

I've tested GraphQL directives and they work well with githubql. They're actually quite handy for some queries.
(In my situation, I wanted to get all comments of an issue, including the issue description, while performing pagination, but only get the issue comment on the first page. Directives made it easy to only get the issue description on the first page of results. See https://github.com/shurcooL/issues/commit/36fff28db4d1f8abcc8bce30659f0ad759331307, grep for firstPage in that commit.)


Spring guide for starting Eureka Server results in NullPointerException

I went here: https://spring.io/guides/gs/service-registration-and-discovery
I followed the instructions, generating my project via this provided link: https://start.spring.io/#!type=maven-project&language=java&packaging=jar&groupId=com.example&artifactId=eureka-server&name=eureka-server&description=Demo project for Spring Boot&packageName=com.example.eureka-server&dependencies=cloud-eureka-server
I added the YAML as the instructions indicated to eureka-server/src/main/resources/application.yml:
spring:
  application:
    name: eureka-server
server:
  port: 8761
eureka:
  client:
    register-with-eureka: false
    fetch-registry: false
logging:
  level:
    com.netflix.eureka: OFF
    com.netflix.discovery: OFF

The instructions said I could now run ./mvn spring-boot:run. So I did.
This resulted in:
org.springframework.beans.factory.UnsatisfiedDependencyException: Error creating bean with name 'scopedTarget.eurekaClient' defined in class path resource [org/springframework/cloud/netflix/eureka/EurekaClientAutoConfiguration$RefreshableEurekaClientConfiguration.class]: Unsatisfied dependency expressed through method 'eurekaClient' parameter 3: No qualifying bean of type 'com.netflix.discovery.shared.transport.jersey.TransportClientFactories<?>' available: expected at least 1 bean which qualifies as autowire candidate. Dependency annotations: {}

and
Caused by: java.lang.NullPointerException: Cannot invoke "org.springframework.cloud.netflix.eureka.CloudEurekaClient.getApplications()" because the return value of "org.springframework.cloud.netflix.eureka.serviceregistry.EurekaRegistration.getEurekaClient()" is null

I found this similar (closed) issue: https://github.com/spring-cloud/spring-cloud-netflix/issues/3953 but I think this is a new problem.

For posterity: this happens if, like me, a new user just trying to follow instructions, you run the generated project without first adding the @EnableEurekaServer annotation.

Yes, this is included in the information in the guide and the provided code snippet.


CMake lacks install target

Platform:
Aarch64 Linux

Problem description:
When trying to compile from source using CMake and Ninja, I see the following error:
30.90 ninja: error: unknown target 'install'

It seems CMake lacks an install target.
Have tried manually installing libs, its symlinks, and headers automatically but to no avail.
Here's an example of how AWS implemented this for their C SDK.
This is likely an extension of #1048.

@morgolock any chance to make this a priority?

Hi @BwL1289
Thanks for raising this, I'll discuss it with the team.

Thank you.

Hi @morgolock. Not trying to be a bother. Please let me know if the team thought this was worth prioritizing. Thank you!

Hi @BwL1289
Yes we will do it, I will ask for this to be in the next release. Once I get confirmation I will let you know.

Thanks @morgolock.

@morgolock yes that helps. Thank you.

@morgolock do you have an updated timeline? Thank you.

Hi @BwL1289
We are currently discussing it, once we have a good idea of the effort I will give you an update. I'll know more in the coming days.
Hope this helps

@morgolock following up again. Can you please let me know if the team plans to fix this?

Hi @BwL1289
The following patch will solve the problem. This fix will be present in v24.11
Hope this helps

@morgolock thank you.

FWIW, I forked this repo and implemented the change manually. I can confirm it works.

Hi @BwL1289
Apologies for the confusion, I've reopened the issue.
We cannot merge the PR on github unfortunately, we only publish a snapshot of the main dev repo on github and if we merge a PR it will go out of sync with the release branch in the dev repo.
Apologies for the confusion. This patch will be present in our next release 24.11.1.


No Tuya device show after latest HA update

After updating to the latest HA release (2021.10.5) no Tuya devices show up in HA. All the configured devices (smart plugs) show and operate correctly in the Tuya app. I'm using the new Tuya integration and it was working prior to the update.

Same here!

Same here. Can see devices online in the Tuya IOT portal, can control them using Tuya App, can control them using Google (integrated with Tuya), but not via HA. Tried reloading Integration, tried removing integration and re-entering credentials (and restarting), but to no avail. Other platforms (such as Xiaomi) are working.

I described the solution to the problem here.
https://github.com/tuya/tuya-home-assistant/issues/593#issuecomment-945167034

Good news! I got my dimmers to sync by adding category tgq to the TUYA_SUPPORT_TYPE allowlist at custom_components/tuya_v2/light.py:70 (same file exists in v1).
TUYA_SUPPORT_TYPE = {
    "dj",  # Light
    "dd",  # Light strip
    "fwl",  # Ambient light
    "dc",  # Light string
    "jsq",  # Humidifier's light
    "xdd",  # Ceiling Light
    "xxj",  # Diffuser's light
    "fs",  # Fan
    "tgq", # Dimme:
}

I should submit this as a PR but this integration is absolute garbage and I don't have time for this, just like the rest of you.

As of yesterday, the HASS Core Tuya integration code has support for all the rarer categories: https://github.com/home-assistant/core/commits/dev/homeassistant/components/tuya/light.py
So it should just be a matter of waiting for the next HASS release if you're using that version.


Bug 2141089: Ensure vm template object has namespace

📝 Description
Cause
In CDI upload, dataVolume has no namespace as the template VM object has no namespace.
Solution
when VM object in template YAML has no namespace, the template namespace is implied
NOTE: change the dvName default name as VM.metadata.name in case of a template VM can be a parameter

/bugzilla refresh

What's the benefit of using the active namespace instead of the template namespace? Are there conditions where the template has no namespace?

@upalatucci that's great that you tested what is happening on cli, may I assume you were logged as admin user?
if so I wonder what happens for non-admin users if the VM's namespace is not specified

Anyway, in the template details page, the activeNamespace is the template namespace so I don't know the difference here. We have just less code using template namespace

than I guess using the template's namespace is really the same as using activeNamespace

/lgtm


disable chronos services

Hi,
Is there a way to disable chronos framework with mesos-consul ? I have chronos finished task showing up a services in consul. I dont have a use for it and it is polluting the consul services namespace.
-Imran

Hi, in my environment finished tasks are perfectly removed by mesos-consul and not keep up like a service alive in consul (obviously only when task is running)

I think it's very useful to ignore Chronos tasks since this is ephemeral

I just found a change in the master which implements blacklist/whitelist for frameworks too, so I just put --fw-blacklist=^chronos and mesos-consul ignores Chronos tasks


Feature/doobie with status

In this PR, the below listed issues have been addressed:

Issue #11
Issue #51
Issue #99
Issue #104
Issue #105

I am aware that this pull request contains a substantial number of changes. I understand that this can make the review process more challenging and time-consuming, and I appreciate your efforts in ensuring our codebase maintains its high quality.
I am fully aware that the volume of changes may be overwhelming, and it's important to me that everyone on the team feels comfortable with these updates. To that end, I am more than willing to walk anyone through the code changes, explain the reasoning behind each modification, and answer any questions you may have.
Please feel free to reach out to me at your convenience if you would like a walkthrough of the code or if you have any questions or concerns.
Please review the changes and provide your feedback.
Closes #11
Closes #51
Closes #99
Closes #104
Closes #105

File /home/runner/work/fa-db/fa-db/examples/src/main/scala/za/co/absa/fadb/examples/enceladus/DatasetSchema.scala still doesn't have licence header according to the CI

Jacoco doesn't seem to work here, can you take a look @miroslavpojer please? Perhaps this is something you encountered before

File /home/runner/work/fa-db/fa-db/examples/src/main/scala/za/co/absa/fadb/examples/enceladus/DatasetSchema.scala still doesn't have licence header according to the CI

Seems that the license formatting was off. The license check passes now.


Better examples on how to use this plugin with Angular

Can we get some examples on how to use this plugin with frontend frameworks like Angular? That would be awesome, because this article https://ionicframework.com/docs/native/push is using the old (phonegap push plugin) and since it's deprecated and says to use this one, there aren't any good examples on how to use the modules in apps.
Thanks.

if it's any help https://github.com/armandoxxx/AngularWithCordova
I'm testing the same thing.
My mobile app can load image from camera and photogallery .. load some remote data .. but I'm experiencing some problems with notification on IOS ...

I've fixed the problem with IOs .. check the latest code on github. Works like a charm

I've fixed the problem with IOs .. check the latest code on github. Works like a charm

cool I'll have to check this out again, thx.


[Style]: Formatting the braces } in About section

Documentation
Nice Portfolio website 😀, saw a small fix.

Before formatting 👇🏻

After formatting 👇🏻

Also, the code is not formatted properly! Needs a lot formatting.

Nice catch do you want to work on this issue?

Yep Sure!

I assigned you @bharathkalyans


Promote the Meshery adapter for Kuma to "beta"

Description
This PR fixes #286
Notes for Reviewers

see Meshery adapter for Kuma for latest updates to adapter.

Signed commits

[x] Yes, I signed my commits.

@subnetmarco @yskopets @jakubdyszkiewicz @nickolaev @tharun208 fyi

@leecalcote this needs to be fixed ⬇
In the deploy preview site this thing needs to be fixed ⬇

Where as in the meshery.io it's like this -

It's a very minor fix. We just need to increase the rowspan from rowspan="2" to rowspan="3"  in [index.html](https://github.com/layer5io/meshery.io/blob/master/index.html).
rest everything LGTM :+1:


Discuss superficially merging host org and host collective into one page

I've been making plans to start demo'ing OpenCollective to some nonprofits, and the fact that a host has two pages (the org page for collective applications and the collective page for handling finances) seems a bit confusing to explain.
Might there be room for a discussion around treating the host collective as a special case (perhaps configurable in the host org settings) so that it gets hidden from the "member collectives" section, and its elements get merged into the host org page?

cc / @xdamman

Thanks for starting this conversation.
I agree. The host should be able to also act as a collective on its own (and why not also allowing a user to have its own collective?). And the commission for the host should show up in its ledger in full transparency.
The good news is that hosts, organizations, users are already collectives in the architecture of the system. So it's fairly easy to make that happen. What's not easy is to reflect the commission for the host into the ledger. But that's something that @clarete is working on with the full double entry ledger (right now the commission is a different column in the Transactions table. It should be a row.)
As a first step, we could add a setting when editing an organization that is host to turn on "Run as a collective" to accept donations.


[io] SegFault when opening spec file without scans

silx.io.specfile.SpecFile segfault if there is no scan in the file.
As it is now, it looks for #S or #F in the first 10 lines of the file...
Looking for #S in the entire file (as pymca does) - and may be also look for #F (spec's spec to be checked) - turns the segfault into a python exception.

This patch makes it work:
silx/io/specfile.pyx
@@ -619,10 +619,10 @@ def is_specfile(filename):
     with open(filename, "rb") as f:
         chunk = f.read(2500)
     for i, line in enumerate(chunk.split(b"\n")):
-        if line.startswith(b"#S ") or line.startswith(b"#F "):
+        if line.startswith(b"#S "): # or line.startswith(b"#F "):
             return True
-        if i >= 10:
-            break
+        #if i >= 10:
+        #    break


Generate video thumbnail from recorded video

Hey,
Is it feasible to bring in a feature to generate video thumbnail from a recorded video and get it as a jpg/png file so that it can be stored along with the media metadata for better user experience?

generating thumbnails for a video snip is very much out of scope from this component's context. This would increase the complexity of this. Just want to keep things as simple as possible.

Yeah, understood. If it were you, how would you generate a thumbnail from a video snip. I saw many solutions in the web. Curious. @0x006F


Move UAA audit doc to CF docs.

UAA security administrators can view UAA audit events in https://docs.cloudfoundry.org/
#166091465

There are separate PRs that implement this change and need to be merged alongside this for it to work properly:
https://github.com/cloudfoundry/docs-loggregator/pull/40
https://github.com/cloudfoundry/docs-running-cf/pull/80


Rasa X quick installation ansible playbook syntax error

Rasa Χ version: 0.19.0
Python version: 2.7
Operating system: Ubuntu Linux 18.04.02 LTS
Issue: In step 2 of the quick installation guide there is a shell script which fetches and executes the Ansible playbook from https://storage.googleapis.com/rasa-x-releases/stable/rasa_x_playbook.yml.
In this playbook lines 57 through 62 are over-indented by one level, generating a syntax error and failing the ansible-playbook execution:
Original:
  - name: Create the directory for the database persistence
      file:
        path: /etc/rasa/db
        state: directory
        mode: 0750
        owner: 1001
        group: root

Should be:
  - name: Create the directory for the database persistence
    file:
      path: /etc/rasa/db
      state: directory
      mode: 0750
      owner: 1001
      group: root

Thanks for raising this issue, @wochinge  will get back to you about it soon.

@ftsamis Thanks a lot for discovering that! We already fixed that for the next version 👍


Add support to math.Rectangle to our extensions

Description
Add support to math.Rectangle to our extensions.
tiled uses Rectangles because it is Flutter agnostic, and Rect comes from dart:ui which is only bundled for Flutter.
But in general, I think it's useful to provide converters, similar to how we can easily convert to and from (size, position, offset, dimension) on flutter as vectors.
Checklist
Before you create this PR confirm that it meets all requirements listed below by checking the relevant checkboxes ([x]). This will ensure a smooth and quick review process.

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] My PR includes unit or integration tests for all changed/updated/fixed behaviors and are passing (See Contributor Guide).
[x] My PR does not decrease the code coverage, or I have a very special case and explained on the PR description why this PR decreases the coverage.
[x] I updated/added relevant documentation (doc comments with ///) and updated/added examples in doc/examples.
[x] I have formatted my code with ./scripts/format.sh and the Flame analyzer (./scripts/analyze.sh) does not report any problems.
[x] I read and followed the Flame Style Guide.
[x] I have added a description of the change under [next] in CHANGELOG.md.
[x] I removed the Draft status, by clicking on the Ready for review button in this PR.
[x] I am willing to follow-up on review comments in a timely manner.

Breaking Change
Does your PR require Flame users to manually update their apps to accommodate your change?

[ ] Yes, this is a breaking change (please indicate a breaking change in CHANGELOG.md).
[x] No, this is not a breaking change.

 It looks like docs updates without the actual code?

How odd that dart:math only has Rectangle and not any other shapes. And too bad it can't be rotated, since it has intersection built-in.

yeah, it's very limited compared to the flame geometry stuff


feat(eslint): change config files to support eslint

Summary
Fixes #1566
Making changes to configuration files where we want to migrate from deprecated tslint to eslint.

Does this PR introduce a breaking change?

[x] Yes
[ ] No

Currently, there are some errors in the linting which breaks tests, but those are regarding prettier formatting mostly. Other errors have been put to warning. These can either be handled or put to off in .eslintrc
Other information
I was initially considering to add eslint-plugin-import to lint some errors regarding import syntax, ordering and stuff but that heavily degraded the performance of the linting.
Here is with:

Without:

@ahnpnl I could try to fix the errors by converting them to warnings or manually fix the files if you want? :)

Running the test locally without the linting step will pass all the tests

Now is making CI pass by making lint pass 😉

Here is the errors from lint:js and lint:ts
eslintjs.txt
eslintts.txt

there is one thing you can do now to get rid of a lot among those errors is this. Because in next branch, we set include to [] so you can adapt the same here.

there is one thing you can do now to get rid of a lot among those errors is this. Because in next branch, we set include to [] so you can adapt the same here.

This causes a lot of parsing errors, so that is not possible

The errors should be fixed now by eslint --fix, only warnings left now. Expecting the travis build to go green :)

Oops seems like your branch contains changes of next branch. Maybe I should create a new branch on ts-jest and you can open another PR to target merge to that branch. This PR cannot target merge to master

@ahnpnl Done!


tests: better "nested" example name

Use the name of a ceph release here, rather than the general name "nested".
This will allow us to add more repos (ceph releases) here to match what happens on download.ceph.com.

I chose a generic, non-ceph name to avoid being out of date when jewel is EOL. Does it matter now (and will it matter when it is EOL?) If you are fine with not changing it further then that is fine with me too

It doesn't matter strongly to me... My next PR adds "luminous", and I think there's some value in using real (or semi-real) data


Add/update demo app

Updates the demo app a bit and integrates ember-cli-github-pages.
See example of deployed gh-pages here: http://github.jhawk.co/ember-cli-flash/
Using ember-cli-github-pages

:umbrella: The latest upstream changes (presumably #130) made this pull request unmergeable. Please resolve the merge conflicts.

@poteto would you prefer I rebase this PR or just close it? Not sure what your plans are for this project in this regard.

Yes please!

Git being awesome, will open a new PR


Upgrade node 20

I am not a node JS developer (in fact I actively avoid it in my day-to-day). working on upgrading st2chatops and it looks like hubot-stackstorm was set to an old engine... Most tests appear to still work I just had to upgrade some libraries (and keep others pretty old) and then rewrite the gulp file to get it to work with node 20.8.1 (matching the new version used by st2). One set of tests needed a delay in order to pass.
I would really appreciate if someone familiar with this could take a look... The only thing I can say with confidence is that the unittests pass.

As a side note: err-stackstorm is a Python chatops implementation.  If you're interested, you can read more about it here https://err-stackstorm.readthedocs.io/en/latest/

As a side note: err-stackstorm is a Python chatops implementation. If you're interested, you can read more about it here https://err-stackstorm.readthedocs.io/en/latest/

unfortunately this is about upgrading the package itself so that a more modern version of nodejs can be used (bringing it in line with node 20). Switching to a different chatops implementation is not an option.

@rebrowning Tests have been switched from TravisCI to Github Actions which targets nodejs 14 to 20.  The tests are passing on master, but fail on your branch.

@nzlosh I'm going to open another PR #234 to see if we can do without most of these changes and upgrades. I did this PR first but then st2chatops essentially just needed coffeescript pinned, so I'll see if that works here as well.


ShaderGraph dependency problem for OpenUPM install

I got the following error when I install UnityGLTF using OpenUPM
openupm add org.khronos.unitygltf

WARN 404 version 10.0.0 is not a valid choice of 0.1.8, 0.1.9, 0.1.17, 1.0.0-beta, 1.1.1-preview, 1.1.2-preview, 1.1.3-preview, 1.1.4-preview, 1.1.6-preview, 1.1.9-preview, 1.1.8-preview, 1.1.5-preview, 2.0.1-preview, 2.0.3-preview, 2.0.4-preview, 2.0.4-preview.1, 2.0.5-preview, 3.0.0-preview, 2.0.6-preview, 2.0.7-preview, 2.0.8-preview, 3.1.0-preview, 3.3.0-preview, 4.0.0-preview, 4.0.1-preview, 5.0.0-preview, 4.1.0-preview, 4.2.0-preview, 5.1.0, 4.3.0-preview, 5.2.0, 5.2.1, 5.2.2, 4.6.0-preview, 5.2.3, 4.8.0-preview, 4.9.0-preview, 5.3.1, 4.10.0-preview, 5.6.1, 5.7.2, 5.8.2, 5.9.0, 5.10.0, 5.13.0, 6.5.3, 6.5.2, 6.7.1, 5.16.1, 6.9.0, 7.0.0, 7.0.1, 6.9.1, 7.1.1, 7.1.2, 6.9.2, 7.1.5, 7.1.6, 7.1.7, 7.1.8, 7.2.0, 7.2.1, 8.0.1, 7.3.1, 9.0.0-preview.14, 8.1.0, 7.4.1, 9.0.0-preview.34, 8.2.0, 10.0.0-preview.27, 7.4.2, 7.4.3, 9.0.0-preview.55, 7.5.1, 10.1.0, 9.0.0-preview.72, 10.2.0, 8.3.1, 10.2.1, 7.5.2, 10.2.2, 10.3.1, 7.5.3, 10.3.2, 10.4.0, 7.6.0, 10.5.0, 7.7.0, 10.5.1, 10.6.0, 7.7.1, 10.7.0, 10.8.0, 10.8.1, 10.9.0, 10.10.0, 10.10.1
notice suggest to install<EMAIL_ADDRESS>or a replaceable version manually
ERR! missing dependencies please resolve the issue or run with option -f to ignore the warning

The package has a dependency of "com.unity.modules.imageconversion": "1.0.0", but the version is not valid.

Hey, thanks for the report. This looks like a regression in Unity, 10.0.0 should simply resolve to the latest supported version.
Which Unity version are you using?

I got the error with Unity 2022.3.18

OK, thanks. I'll try to reproduce. It works fine on 2022.3.17f1 and 2022.3.20f1.
Have you changed the package resolution settings for your project by any chance? (https://docs.unity3d.com/Manual/upm-manifestPrj.html#resolutionStrategy)

This is a bug in the OpenUPM CLI:

https://github.com/openupm/openupm/issues/4772

As a workaround, run openupm add org.khronos.unitygltf -f, the defined dependencies in package.json work just fine.

Thanks. I'll use -f option until the bug is fixed. I encountered the bug with a different package and posted the issue in the repository.
https://github.com/openupm/openupm-cli/issues/19#issuecomment-1871057403


Merge the code with deploy

At the moment there is a development environment and a beta environment created for the deployment of the S2. The main goal is to merge the two of them to make a project capable of running in development mode (debug) but also in the app engine with the changes of the deploy branch.

We decided that we are going to investigate deployment with Google Compute Engine. So this does not apply, for now.

Story Points: 13
See the discussion in Backend S3
Powered by Parabol


Refactor the standard integration guide

The DevRel team has been working on improving the Standard Integration code snippets across developer.paypal.com. This PR includes many of these new changes.
Improvements include:

Using the common naming of PAYPAL_CLIENT_ID and PAYPAL_CLIENT_SECRET for env variables.
Using a single file for the Node.js server code.
Teach negative testing with code comments for using the PayPal-Mock-Response request header for order creation and capture.
Always try to relay PayPal API JSON responses and status codes back to the frontend.
Death to alert() since it pauses the popup flow. Replacing this with appending UI messages to the button container.

There are some other changes I am less confident on and could use feedback on.

What route names should we use for wrapping create order and capture order api calls?
Whats the best way to pass the OrderID to the create order api wrapper? Pass it in the URL or in the post body?
Should we recommend the Express static module or just return the html file directly from the file system?

Feedback welcome! Let us know what you like and don't like.

Ravievd payment

Thanks for the feedback everyone! I applied some of the recommendations and I'm going to merge this PR.
@cnallam, fyi I changed two things with the codespaces integration for standard checkout:

There was a bug with the workspace path that is now updated.
The env variables have been renamed to PAYPAL_CLIENT_ID and PAYPAL_CLIENT_SECRET to better match the PayPal Auth docs and to help the developer know that these credentials are for PayPal. We plan to make the same change to the Advanced Integration in a follow up PR.

I've successfully tested the CodeSpaces integration off this feature branch. Here are some screenshots for reference:


Update USERS.md to include Acoss

Signed-off-by: ☠ Yves Blusseau<EMAIL_ADDRESS>

This change is

test-docs-please

Coverage increased (+0.02%) to 45.564% when pulling 48ff539a5cdb305c115b46fa09b1418dfba57a4c on JrCs:add-acoss-to-users into 0d7533fc56ac947e65a3b838b2eb6e573bbb230f on cilium:master.


feat : Add UI component for create instance modal

:tada: This PR is included in version 1.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:

I know this PR is closed, but wasn't sure where the best place was to capture feedback.
First, it's really great to see this component in the storybook! Thanks so much for adding this one! Here are some observations I made in no specific order:

The order of the items in the navigation do not match the order they are listed in the Docs view. This makes it a little challenging to review the variations.
In the Checking quota for instance example, we don't know yet which regions are available to the user, since we don't know if they're getting a trial or standard instance yet and regions vary depending on type of instance they're getting. Currently the regions menu is showing options, but it should be disabled until we know what type they get as illustrated in this example.
Some of the examples don't show the final text and show the string id instead. Being able to see the actual text is really beneficial for some of us. This might already be something you're planning to address, but wanted to note it in case.

This might be more general feedback about storybook, and less about the Create modal

The order of the Canvas and Docs tabs have recently changed. It used to default to Canvas, and I could click the items in the left to view the canvas for that variation. Now, if I switch to the Canvas tab, that selection isn't persistent if I navigate to a different variation, and it switches to the Docs tab. I'm not sure really what's ideal behavior or what our options are, or if anyone else would find benefit in defaulting to the Canvas tab, but just wanted to share that feedback.


starlark: fall back to slow Ints if mmap fails

The Int optimization uses a 4GB virtual address allocation to represent
32-bit values as addresses to avoid allocation. However, some environments
have limited address space.
This change permits the mmap to fail, and in that case it prints a warning
and falls back to always allocating a big.Int, even for small numbers.
Each access to an Int must check the smallints global to see whether the
optimization is active.  The extra dynamic check doesn't cost much:
$ go test -run=nope -bench=BenchmarkStarlark -count=3 ./starlark    # > before, after
$ go install golang.org/x/perf/cmd/benchstat@latest
$ ~/go/bin/benchstat  before after
name                                  old time/op  new time/op   delta
Starlark/bench_bigint-16               206µs ± 0%    212µs ± 0%   ~     (p=0.100 n=3+3)
Starlark/bench_builtin_method-16       283µs ± 1%    294µs ± 9%   ~     (p=1.000 n=3+3)
Starlark/bench_calling-16              280µs ± 1%    290µs ± 1%   ~     (p=0.100 n=3+3)
Starlark/bench_gauss-16               9.30ms ± 5%   9.55ms ± 2%   ~     (p=0.700 n=3+3)
Starlark/bench_int-16                 53.7µs ± 1%   59.6µs ± 1%   ~     (p=0.100 n=3+3)
Starlark/bench_mix-16                 99.3µs ± 1%  106.6µs ±10%   ~     (p=0.100 n=3+3)
Starlark/bench_range_construction-16   238ns ± 1%    244ns ± 2%   ~     (p=0.100 n=3+3)
Starlark/bench_range_iteration-16     5.44µs ± 2%   5.67µs ± 1%   ~     (p=0.100 n=3+3)

Fixes #394

Ping @brandjon...

Ping?

Seems reasonable. Would be good to have a test, though (e.g. set RLIMIT_AS, see if int tests still pass).

Done, though it's a pain because it requires fork+execing a child process, and it requires ulimit -v which is not available on MacOS or Windows.

@tetromino, PTAL at the changes to the build tags, go.mod/sum files, Actions workflow config, and of course the test.


Add GIT_SHA to a docker tag

This will help me try out my publish scripts.
The issue this solves happens on the Cavium
build machine and can potentially have several docker
builds happening at one time.  This way I can query for
a specific builds generated docker images in an easy way.
Signed-off-by: Jeremy Phelps<EMAIL_ADDRESS>

recheck

arm tests failed

@JPWKU We made a maintenance error yesterday and removed the pkg-config package. It is fixed now.


drivers: rtc: new ambiq am1805 rtc driver

Added RTC driver for am1805 with rtc time, alarm set/get, callback and calibration.

Thank you for your PR :)
Have you tested the driver against the RTC API test suite? I highly recommend it to validate the drivers behavior matches the RTC API :)
Have you considered using the system workqueue instead of creating a thread just for handling the interrupt? It is way more efficient :)

Hello @bjarki-trackunit,
i have tested driver against the rtc api test suite and it works fine. Separate Thread is used instead of work-queue for the matter of precision.when an alarm is set with a lowest possibility of just 1 second from the timer value , work-queue may causes a delay more than that even 3 to 4 seconds.

Hello @bjarki-trackunit, i have tested driver against the rtc api test suite and it works fine. Separate Thread is used instead of work-queue for the matter of precision.when an alarm is set with a lowest possibility of just 1 second from the timer value , work-queue may causes a delay more than that even 3 to 4 seconds.

3-4 seconds on the system workqueue is definately not expected unless its being abused quite a lot (someone using k_sleep() in it for example). Regarding the timing, you can invoke the callback alarm and update callbacks from within the GPIO ISR callback, you don't have to delegate it to a work thread or work queue. If you want to use the thread, make it optional :)

Hello @bjarki-trackunit, i have tested driver against the rtc api test suite and it works fine. Separate Thread is used instead of work-queue for the matter of precision.when an alarm is set with a lowest possibility of just 1 second from the timer value , work-queue may causes a delay more than that even 3 to 4 seconds.

3-4 seconds on the system workqueue is definately not expected unless its being abused quite a lot (someone using k_sleep() in it for example). Regarding the timing, you can invoke the alarm and update callbacks from within the GPIO ISR callback (ISR thread), you don't have to delegate it to a work thread or work queue. If you still want to use the thread, make it optional :)

Even when not using any k_sleep, it can cause that delay sometimes(ie. using bluetooth , as it uses the system-workqueue), and sometimes other blocking operations in work queue also. You are correct about the alarm update in the isr, but IMO, if the user from application passes a callback-function using rtc_alarm_set_callback , to make it more precise to call it (not to worry about work queue overload), thread would be good .

If you still want to use the thread, make it optional
not sure ,what do you mean by making it optional :)

By making it optional, you would either call the callback directly, or delegate the callback to a thread. This would look something like
static void am1805_gpio_callback_handler(const struct device *port, struct gpio_callback *cb,
		gpio_port_pins_t pins)
{
	struct am1805_data *data = CONTAINER_OF(cb, struct am1805_data, am1805_callback);

	ARG_UNUSED(port);
	ARG_UNUSED(pins);

#if AM1805_CALLBACK_CONTEXT_ISR
	if (data->alarm_user_callback != NULL) {
                /* NOTE the dev pointer has to be available here */
		data->alarm_user_callback(dev, 0, data->alarm_user_data);
	} else {
		LOG_DBG("Interrupt received, But No Alarm-Callback Initilized!!\n");
	}
#else
	k_sem_give(&data->int_sem);
#endif
}

Then creating a Kconfig choise using the following sample:
choice "AM1805_CALLBACK_CONTEXT_CHOICE"
	prompt "Context used for alarm and update callbacks"
	default AM1805_CALLBACK_CONTEXT_ISR

config AM1805_CALLBACK_CONTEXT_ISR
	bool "Call back from within ISR context"

config AM1805_CALLBACK_CONTEXT_THREAD
	bool "Delegate callbacks to dedicated thread"

if AM1805_CALLBACK_CONTEXT_THREAD

config RTC_AM1805_THREAD_STACK_SIZE
	int "Stack size for the am1805 interrupt thread"
	default 512
	help
	  Size of the stack used for the thread handling interrupts and dispatching callbacks.

config RTC_AM1805_THREAD_PRIO
	int "Priority for the am1805 interrupt thread"
	default 0
	help
	  Priority level for the thread handling interrupts and dispatching callbacks.

endif

This allows the user the choise. work queue could be added here as well.
Note that you do not have to implement callbacks from ISR or callbacks using the workqueue in this PR, that can be added by later PRs when and if someone needs it :) I usually only implement the ISR callback if possible since its the lowest overhead solution with the best timing accuracy, allowing the user, (application) to delegate it using whatever method they prefer :)
Regarding the RTC API test suite, did you enable testing of the alarms and calibration as well? The calibration tests should fail since you have not implemented the get_calibration() function. Would it be possible for you to to implement the get_calibration() functions as well?

@bjarki-trackunit Getting it👍. Regarding the rtc_ testing , sorry for the trouble, will implement the get_calibration for sure.Regarding that optional thread, should I need to implement what you have suggested above in the current pr itself or can that be done in further pr?

@srisurya1 You only need to implement the get_calibration() function in this PR :) Adding different options for callback contexts can be added in later PRs

@bjarki-trackunit FYI, added get_calibration.

@bjarki-trackunit FYI, made the changes.

@fabiobaltieri @bjarki-trackunit Made the changes, kindly revisit.

@bjarki-trackunit , adapted the changes from fabiobaltieri , kindly revisit.


Remove mapping types

As part of v2.0 release, mapping types are getting removed from OpenSearch engine. Below are the changes in the opensearch-engine.

Removal of type from rest end-points
Removal of include_type_name parameter from API requests.

As part of this issue, please verify if type removal change on OpenSearch engine impacts this repository. If yes, then please remove the type references/usage from this plugin.
Top level changes are captured in gist below:
https://gist.github.com/dreamer-89/d76eaf639171e8ab32fa7f8b9d6c93d3
For more detailed changes, please check meta issue below:
Related: OpenSearch-engine meta issue

Plugin is ported to 2.0, ran integ-tests and did some basic sanity and there is no impact on removing mapping types.


Writing tool in repro carts

I have 2 repro cards with this PCB both has 1mb flash save memory
I wrote the clean tool in this and I have a repro game with the same PCB with a fire red room in
The tool will work fine or will crash o bad saving file
Tbh I don't have cable link and other gba to thest it

I would not recommend using the program with repo carts, as there is no guarantee that it will work due to there being a wide breadth of carts and how they all work differently.


ExcelPython for Excel 2011 on Mac?

Any chance we will see a version of ExcelPython for the Mac? If not, is there another solution that works on Excel 2011?

As per readme, ExcelPython has been integrated in xlwings (which works also with Mac except for UDFs): https://github.com/xlwings/xlwings

Thanks. Can I (effectively) do a UDF on the Mac by creating a VBA function that calls Python and then using that in a cell?

On Jun 9, 2019, at 5:12 AM, Felix Zumstein<EMAIL_ADDRESS>wrote:
As per readme, ExcelPython has been integrated in xlwings (which works also with Mac except for UDFs): https://github.com/xlwings/xlwings
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

no, this will be too slow. currently there's no way to make udfs work on
mac.
On Sun, Jun 9, 2019 at 4:07 PM ifuchs<EMAIL_ADDRESS>wrote:

Thanks. Can I (effectively) do a UDF on the Mac by creating a VBA function
that calls Python and then using that in a cell?

On Jun 9, 2019, at 5:12 AM, Felix Zumstein<EMAIL_ADDRESS>wrote:
As per readme, ExcelPython has been integrated in xlwings (which works
also with Mac except for UDFs): https://github.com/xlwings/xlwings
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/ericremoreynolds/excelpython/issues/72?email_source=notifications&email_token=AARJDIHSDFJTCB6XKIOWDZ3PZUFDNA5CNFSM4HWIG6J2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODXIKY5A#issuecomment-500214900,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AARJDIAUXZCKBVAJ6FETAODPZUFDNANCNFSM4HWIG6JQ
.


Create block list for Vietnamese website

This is a blocklist based on Bromite Extended, I added ABPVN blocklist for Vietnamese websites blocking.
Credit to ABPVN (https://abpvn.com/)

Hi,
this repo should only contain a small set of filter lists that I think are relevant to most people. I will close this PR without merging, but of course feel free to keep your repo up to date :)

Pk, thanks for the reply :D


chore: update docs and fix dashboard references

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (ed9df85) 86.28% compared to head (6cefb53) 86.28%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2543   +/-   ##
=======================================
  Coverage   86.28%   86.28%
=======================================
  Files          86       86
  Lines        9396     9396
  Branches      481      481
=======================================
  Hits         8107     8107
  Misses       1289     1289

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


added cache to reduce number of execute queries

superceeded by https://github.com/staabm/phpstan-dba/pull/103


Update teams-scoped-directory-search.md

https://github.com/MicrosoftDocs/OfficeDocs-SkypeForBusiness/issues/6211
Added link

@JohanFreelancer9 Copy editing is needed for this PR. Thanks.

@SerdarSoysal PR has been copyedited and is ready for final review, could you please check and merge? Thanks!


dm/fix-status-on-missing-repos

Fix #160

Codecov Report

Merging #161 into master will decrease coverage by 0.12%.
The diff coverage is 86.04%.

@@            Coverage Diff             @@
##           master     #161      +/-   ##
==========================================
- Coverage   83.67%   83.55%   -0.13%
==========================================
  Files          30       30
  Lines        1648     1678      +30
  Branches      201      201
==========================================
+ Hits         1379     1402      +23
- Misses        221      227       +6
- Partials       48       49       +1

Codecov Report

Merging #161 into master will decrease coverage by 0.12%.
The diff coverage is 86.04%.

@@            Coverage Diff             @@
##           master     #161      +/-   ##
==========================================
- Coverage   83.67%   83.55%   -0.13%
==========================================
  Files          30       30
  Lines        1648     1678      +30
  Branches      201      201
==========================================
+ Hits         1379     1402      +23
- Misses        221      227       +6
- Partials       48       49       +1


External script support for CSP been merged in v1.2.0?

Improve render.scp (Support of external script, PR #2608), see also 95c85bb

Hi, It seems that the following changes are not reflected in the master branch at v1.2.0, but are not they intentionally merged?
If there is a reason, I will correct it so please let me know.

Sorry, I misunderstood seeing the Master Branch...


In my Hololens 2, Developer Mode option is grayed out and I am not able to turn it on. Will need more clarification.

So I have received Hololens 2 from a friend and I noticed my Developer Mode option is grayed out and I am unable to turn it on. I stumbled upon this documentation but unfortunately there isn't useful information available for that case. We will need to add appropriate steps to solve the issue.
Best,
Samvid

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

ID: 37e0e5f1-b23b-4909-c1ba-6dba39174d30
Version Independent ID: b0a8777f-e04e-b645-d983-0acb94268f4c
Content: Using the Windows Device Portal - Mixed Reality
Content Source: mixed-reality-docs/using-the-windows-device-portal.md
Product: mixed-reality
Technology: mr-dev
GitHub Login: @hamalawi
Microsoft Alias: moelhama

Hello Samvid,
Sometimes when using a work account / work device you'll receive device management policies. If you see some words along the lines of "some settings are managed by your organization" then they have likely disabled Developer Mode for security reasons.
There's also a chance you may need to be the "device owner" to enable Developer Mode. I'm not sure. In either case you can always reflash the device and set it up with a new account.

Thanks it worked.


Add support to pass command line arguments

Added to master, will be available in the next release.


New Wizarding API

Noteworthy changes

package.json: I've updated lit. I am going to check if everything is ok, but so far so good
Hosting.ts: Refactored for readability and added the oscd-wizard-host element to the DOM
Plugging.ts: Added the new wizard plugin type also a bit of readability improvements
oscd-card.ts: There were now card component so I've copied the structure and style from the dialog component
wizard-host.ts: The wizarding itself.
Documentations:

ADR: Wizarding API
API: Wizarding
How Tos: How To Create a Wizard Plugin, How To Develop For OpenSCD

Why not a mixin?

It is general philosophy to try to stay close to the DOM.
It creates a more loosely coupled structure.
It does not depend too much on Lit or JavaScript/TypeScript.
Its use case is isolated enough not to need too much from the other mixins

Does this pull request actually contain any wizard libraries to be loaded? It doesn't seem like it actually needs to do that.
If we want to adopt the new wizard API in open-scd, I would suggest we

initially implement the new WizardRequest event API (the one facing plugin authors) using the current built-in wizards, without the ability to modularily load wizard libraries,
do the other work that needs to be done to allow individual plugins to be factored out

pass a locale property to plugins so they can translate themselves
pass support documents (schema, nsd, nsdoc) to plugins in a unified way using docs
add oscd- prefixed css variables for theming colors and fonts

start factoring out individual plugins into separate repositories, and finally
once we have to factor out a plugin which relies on an adapted wizard (e.g. for extension namespace support or the like), add the ability to load wizard libraries and defined the wizard library facing API

That would make this initial pull request significantly simpler and smaller, and would avoid adding any unused code.
What do you think about this procedure?

Couple of questions:

How do you deal with different SCL schemas?

How can you switch dialogs? For example, instead of having the mwc-dialog, i want to have a dialog from a different component library. (This was one of the requirements)

Why is wizard a plugin?

This solution is creating a (soft) dependency from one plugin to another plugin. My thought was that OpenSCD next would have a microservice architecture, not a distributed monolith.

How can you switch dialogs? For example, instead of having the mwc-dialog, i want to have a dialog from a different component library. (This was one of the requirements)

Added point 12. to "answer" this question :)

Why is wizard a plugin?

In order allow different types of wizard. You might want to edit a specific element in a different way and you can provide your own wizard for this element.

This solution is creating a (soft) dependency from one plugin to another plugin. My thought was that OpenSCD next would have a microservice architecture, not a distributed monolith.

I have added point 11. to the description. Does that make it a bit clearer? Do you think that would remove this dependency?

We will wait with this until Milestone 2 of #1300 is done to see if it still fits into our ideas.

I'll close this PR because it has too much conflicts and it only takes up space in the list.
If we need it later we can re-open it.


OpenCV version

Dear sir,
I am trying to use opencv c++ to implement paper title "Edge Drawing – A Combined Real-Time Edge/Edge Segment Detector" but I still can not make it done for part "anchor smart routing algorithm". Could you please help me by sharing the opencv version of this paper?
Thank you so much.

@jodyngo
You can see the comment of "extract the anchors in the gradient image, store into a vector" in the function "EDLineDetector::EdgeDrawing()". It is the step of extraction of the anchors. You can also convert data type to cv::mat, and the functions of mcv_xxxx to opencv interface.


Conflict with already running dnsmasq process on port 53

When running the installation script I got the following error coming from a conflict on port 53:
Recreating m2c.test__dnsmasq ... error
ERROR: for m2c.test__dnsmasq Cannot start service dnsmasq: driver failed programming external connectivity on endpoint m2c.test__dnsmasq (78cef79ac72e8a5fdd70983ab9e15fec6088c16feb231b17b45fe0899eb651a8): Error starting userland proxy: listen udp <IP_ADDRESS>:53: bind: address already in use ERROR: for dnsmasq Cannot start service dnsmasq: driver failed programming external connectivity on endpoint m2c.test__dnsmasq (78cef79ac72e8a5fdd70983ab9e15fec6088c16feb231b17b45fe0899eb651a8): Error starting userland proxy: listen udp <IP_ADDRESS>:53: bind: address already in use Encountered errors while bringing up the project.

Solved by uninstalling dnsmasq and killing the process. Maybe worth adding check on port 53 on top of port 80 and 443 during installation process?

Hi @grll, that is a good catch!
I'm going to add a check also for that port.

Hi @grll, that is a good catch!
I'm going to add a check also for that port.

Great thanks! The toolset is really amazing by the way thanks a lot!


Slack IRC gateways are going away May 15th

as per:

https://get.slack.help/hc/en-us/articles/201727913-Connect-to-Slack-over-IRC-and-XMPP

https://twitter.com/genehack/status/971404236490620928

it looks like "Starting on May 15th, it will no longer be possible to connect to Slack using the IRC and XMPP gateways."
will this be affecting the Franz integration w/ Slack?

no

thanks for confirming, closing this issue


Fix cannot resolve Services::xxx()

Description
Fixes #4220
Checklist:

[x] Securely signed commits
[x] Component(s) with PHPdocs
[ ] Unit testing, with >80% coverage
[ ] User guide updated
[x] Conforms to style guide

Thank you @kenjis

@samsonasik Thank you!


[Question] Synchronize Translation

Hello, I've been trying for days to find a way to automatically synchronize the translation of a subtitle that its original language is English and I want to translate it into Portuguese.
Executing the command
subaligner -m script -v teste.mp4 -s joao.txt -o subtitle_aligned.srt -t eng,por
I got the following error:
ERROR: Cannot find the MT model for source language "eng" and destination language "por"
So I believe that unfortunately the Portuguese language is not available, is there a possibility to add it? Or currently there is no model for the Portuguese language.
If you have any extra strategies for synchronizing a translated subtitle, I would be extremely grateful if you could share them with me. The big problem is that for the translation to be done correctly it needs context, so I need the text to be placed in the translator in a clean way, just the text (no line breaks and stuff) and somehow need some reference to the subtitle number or time for it to be synchronized, but unfortunately ALL the strategies I used, the translator at some point removes my reference.
Some strategies I used:
1. Hello, 2. how are you?
[1] Hello, [2] how are you?
Hello, :: how are you?
Hello, // how are you?

Thanks.

Hi @ubanning, your belief was spot on. The underlying OPUS models don’t have a base release for opus-mt-en-pt as they do for other language pairs. After browsing their HF repo, I discovered they released a big model called opus-mt-tc-big-en-pt last year. I have made all big models supported by subaligner and please checkout the latest change from the master branch before attempting your command again.

Hi @baxtree, Thanks for the answer. Now it worked, but I have another question:
I would like to use a translation that I have that is in plain text format and I would like to sync it there.
How could I do this?
I wouldn't want to use the service to translate, but just to sync the translation I already have.
Can you help me? Thanks :)

Hi @ubanning, currently it doesn't know how to split the text into subtitle cues so users need to perform pre-sync segmentation by separating the cue texts with newlines like this:
# joao.txt
cue_1_text

cue_2_text
...

cue_n_text

Then you can run the syncing without translation:
$ subaligner -m script -v teste.mp4 -s joao.txt -o subtitle_aligned.srt
PS: If you are interested in implementing auto-splitting, PRs are always welcome :)

Big models are now available in release 0.3.0. Closing this now and thanks for opening the issue.


Migration to IterativeSolvers, bug fixes, increased coverage

Iterative Solvers
The main goal of this PR is to use the iterative solver routines from IterativeSolvers.jl. To this end, new features include:

Solves with rectangular systems via lsqr.
Solves with indefinite symmetric systems via gmres (the current release of ToeplitzMatrices.jl assumes that all symmetric systems are positive definite, and uses cg).
Deletion of the iterativeLinearSolvers.jl file

New Functionality
Beside this high level goal, the PR also defines several new functions for Toeplitz matrices and factorizations:

inv for Toeplitz matrices
adjoint, size, and copy for ToeplitzFactorization structures
apply(f, C) for Circulant and CirculantFactorization structures C, which applies f to the eigenvalues of C, in this way defining the matrix functions abs, sqrt, and inv.

Testing Coverage
Further, this PR includes a testing suite with greater coverage.
Bug Fix
In writing the new tests, it became clear that inv(::TriangularToeplitz) was broken on Julia 1.7 (and presumable several previous versions too, since nextpow2 and conv haven't been in Base for a while). This PR also fixes the function.

@marcusdavidwebb @ajt60gaibb any opinions on this?

Coverage increased (+9.3%) to 85.906% when pulling 393401b34e27fbfb0f8261da1955137b3c85e223 on SebastianAment:iterative-linear-solvers into edbc8eb37c93df6c752e5313da307445a2e2022c on JuliaMatrices:master.

We can drop Julia v1.0 support and just require Julia v1.6 if you bump the minor version

I adjusted the Julia compatibility entry, the CI file, and bumped the minor version. Let me know if there's anything else that needs modification.

There was one test that failed in one out of the six CI runs. I reduced the accuracy requirement of the test to the accuracy requirement of the iterative solver, which should make the test pass.

I'm interested in the features introduced in this pull request and noticed it has been inactive for nearly two years. Specifically, I'm interested in using gmres from IterativeSolvers.jl on ToeplitzFactorization structures.
Could the maintainers shed light on the status of this PR? Are there unresolved issues or specific reasons preventing its integration? I'm keen to assist, be it resolving merge conflicts or updating the code, to facilitate its inclusion.

Probably no one got around to merging it.
Now there are a lot of conflicts.
Skimming through, it should be change to an extension, not adding IterativeSolvers.jl as a dependency. That extension can live here or in IterativeSolvers.jl

Thank you for the response, @dlfivefifty! I'll have a look at this in the next weeks.


Access mirrors.aliyun.com via HTTPS instead of HTTP

Ⅰ. Describe what this PR did
Replace http://mirrors.aliyun.com in INSTALLATION.md with https://mirrors.aliyun.com to install PouchContainer from mirrors.aliyun.com via HTTPS.
将 INSTALLATION.md 中的 http://mirrors.aliyun.com 替换为 https://mirrors.aliyun.com，以通过 HTTPS 从 mirrors.aliyun.com 安装 PouchContainer。
Ⅱ. Does this pull request fix one issue?

Ⅲ. Why don't you add test cases (unit test/integration test)? (你真的觉得不需要加测试吗？)
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

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

Seem that this url is changed witch http://mirrors.aliyun.com/opsx/pouch/linux/debian.orig/ , could you help to update it? After your response, I will review and merge it ASAP!

Hope you can squash these commits!

I amended the last commit.
Is it necessary to squash 58b860592a752f09d9fb707d07a18c65396ab70c and 10997fc205a5a5b40add15682e94a5e1ee550ee7?

Hope you can squash these commits!

I amended the last commit. Is it necessary to squash 58b8605 and 10997fc?

Yes, I think it's needed! 😄


Fix - Sandpack error icon overlapping issue

Issue: Sandpack error icon overlapping code #4287
Description
Added padding to the error line  to make space for the icon. I had to add !important because of all the !importants lying around
Before

After

Seems like adding padding for the icon introduces an additional scroll just to see the error icon

PR change
Current prod

________</h1>< alt={'Photo of ' + user.name} className=``"avtar"
In case you want to test this out ☝🏻  is the line that I've added to the sandpack

@harish-sethuraman yes, the scrollbar will appear if line takes up the entire width. Where else should the icon go? Should it go to the next line?

Where else should the icon go?

I'm not exactly sure of this. I'll wait for Dan's thought on where the icon should go.
PS: I think it would be nice if the icon appears on top of code without showing the overlap (with appropriate background color)

@harish-sethuraman  I agree but if the user has to edit the character that is directly below the icon then the user will not be able to see it properly.

I agree but if the user has to edit the character that is directly below the icon then the user will not be able to see it properly.

That is how the current experience already is. Adding a scroll just for showing the error icon might not be right either...

I'm still seeing overlap with this PR.

@gaearon can you please mention the steps to reproduce this? I am testing myself and I don't see it overlapping

@gaearon  @harish-sethuraman  Please review again.

In the above case, Error icon is not visible. It appears only at the end (after the longest text).

thanks.


Persons recognized are not called out in the bounding box

hi
If I upload a pic and this person is recognized in the video stream it doesn't put their name around the box , just leaves it as "Person 1" or whatever .  It does recognize the picture correctly as the name pops up on the right .     Which is correct, please ?  I'd like it to do so, or is this an exercise for the reader ?  My React skills are not very good.

This is correct. The demo is just to show the art of the possible, if someone wants to achieve that behaviour some code changes are needed.


"Installing files" step always numbered one too low on stdout (i.e. step [N/N] never shown)

I presume this is a ninja issue. If it's somehow meson's fault, I'll reopen it over there.
Using build.ninja generated by meson, I always get output like the following:
[1/5] Generating blah.gresource_c with a custom command.
[2/5] Compiling C object<EMAIL_ADDRESS>[3/5] Linking static target libblah.gresource.a.
[4/5] Linking target blah.exe.
[4/5] Installing files.

That is: Step 5 is never shown, and the "Installing files" step is always numbered one lower than it should be, duplicating the step number of the step above it.
ninja 1.8.2
meson 0.42.1
MSYS2/MINGW64 on Windows

It seems more general: if installation is the only step pending, it gets displayed as step 0 of 1...
[0/1] Installing files.

according to the documentation:
%t  The total number of edges that must be run to complete the build.
%p  The percentage of started edges.
%r  The number of currently running edges.
%u  The number of remaining edges to start.
%f  The number of finished edges.

The default progress status is "[%f/%t] ...

-- https://ninja-build.org/manual.html#_environment_variables
at the time of printing this status message, the last edge is not yet finished. would this explain your problem? Is NINJA_STATUS="[%r/%t] " helping?

at the time of printing this status message, the last edge is not yet finished. would this explain your problem?

Thanks for the info. That may explain it, though then it's weird that, if the 1st step is a build, it gets numbered as [1/n], but if it's an install, it's [0/n]. Surely at the time of building the 1st target, 0 edges have finished so far?

Is NINJA_STATUS="[%r/%t] " helping?

Not really; that shows how many parallel targets are being built, not the current number in the sequence.

A simple test for this:
diff --git a/misc/output_test.py b/misc/output_test.py
index 1dcde10..49315f7 100755
--- a/misc/output_test.py
+++ b/misc/output_test.py
@@ -99,5 +99,9 @@ red
 \x1b[31mred\x1b[0m
 ''')

+    def test_issue_1336(self):
+        print_lines = 'rule echo\n  command = echo\n  pool = console\nbuild a: echo\n'
+        self.assertEqual(run(print_lines, pipe=True), '[1/1] echo\n\n')
+
 if __name__ == '__main__':
     unittest.main()


Unique cache-key for job matrix objects

Changes how the default cache key is generated such that YAML objects in the matrix are expanded upon and not just returned as "Object". Due to use combining so many values from the matrix I've also decided to include the key name in the cache key.
The change here should be non-breaking but will result in cache misses when users first update to this version.
Fixes #87.

Still have some details to work out with the tests. Would be nice to have the tests not require cache-name.

We can't really use jq as it's not guaranteed to be installed on self-hosted runners

If we did I think we'd have to make a breaking change, which I wanted to avoid so that the ecosystem gets the benefit without changing anything

PR's ready for review. I'll mention that introducing the matrix keys into the cache key isn't strictly necessary but I think it's helpful when you have matrices with lots of options. It does make the cache names in GitHub actions much longer but I think the additional context which is added is more helpful for humans in identifying which cache entry they want to manually delete.

Noticed a README update I need to do

FYI: this change seems to have broken one of my CI runs.  I think the problem is that I'm running in a container on a GitHub hosted runner, so the jq isn't installed.
here's a log: https://github.com/musoke/WolframExpr.jl/actions/runs/7532058237/job/20501935392#step:5:65


Handling streaming request data

Description
Using just Flask, a user can (manually) use the request.input_stream to handle streaming uploads. The use case is because I need to handle binary uploads that won't fit in memory. I use the JSON validation features of connexion when the Content-Type is application/json but in this case I have a application/octet-stream.
With the connexion library I can't seem to work out how to access the raw data stream - it looks like the data all gets read in  FlaskApi.get_request from EndOfRequestLifecycleDecorator - which is happening well before calling any user defined handler function.
Is it possible to not wrap the flask.request with ConnexionRequest for some endpoints?
Expected behaviour
For an unrecognized content-type, or for a specific binary stream, connexion should - like Flask - not read() the full request body before calling a handler. Instead the stream should be pulled on demand - usually when a user calls request.data() or request.json()
Actual behaviour
Deep in the bowels of connexion the stream gets fully read and I couldn't see how to disable it for one endpoint.
Steps to reproduce
Upload a file e.g. with requests:
file_path = 'large.bin'
with open(file_path, 'rb') as f:
    r = requests.post(
        url + '/upload',
        headers={
            'Content-Type': 'application/octet-stream',
            'Content-Length': '123456789'
        },
        data=f.read()
    )

On the server side:
if headers['Content-Type'] == "application/octet-stream":
        # This should work
        data = request.input_stream.read()
        print(len(data))

But the input_stream is empty because connexion has already slurped it up - which lead to problems if we are dealing with uploads that won't fit in memory (obviously I don't normally just read the stream and print the length)
Additional info:

python --version 3.7 (in alpine docker image)
pip show connexion | grep "^Version\:" 1.4

Thanks for reporting, I don't have clear insights, but I could think about an Swagger extension to disable validation or something similar to prevent slurping the input stream.

A simpler solution might be to inspect the Content-Type - if it shows that validation couldn't possibly do anything useful IMO connexion should try not to change any state at all.

what do you think about #760 ?

+1

Hi @hardbyte, I think my issue https://github.com/zalando/connexion/issues/1332 is related, did you find a way to solve yours ? I tried to find a decorator to bypass but I couldn't figure it out

Hi @hardbyte, I think my issue https://github.com/zalando/connexion/issues/1332 is related, did you find a way to solve yours ? I tried to find a decorator to bypass but I couldn't figure it out

Not really, I ended up changing that API to allow uploads via S3/Minio.

Not really, I ended up changing that API to allow uploads via S3/Minio.

As for now we patched internally with something like this:
# [...]
body = flask_request.stream if flask_request.headers["Content-Type"] == "application/octet-stream" else flask_request.get_data()

in https://github.com/zalando/connexion/blob/master/connexion/apis/flask_api.py#L235

Fixed since https://github.com/spec-first/connexion/pull/1618/files


Cannot find leanote/tests

ERROR 2015/08/22 15:47:35 watcher.go:72: Failed to stat watched path
/Users/df/mygo/src/github.com/leanote/leanote/tests : stat
/Users/df/mygo/src/github.com/leanote/leanote/tests: no such file or directory

Why I met this problem?
How can I solve it?

Leanote doesn't have tests directory. Try to create it manually.

@anonx Thx.   I met a new problem on my MacBook.
ERROR 2015/08/23 16:25:02 watcher.go:124: Failed to watch /Users/df/mygo/src/github.com/leanote/leanote/app/views/member/blog : too many open files
ERROR 2015/08/23 16:25:02 watcher.go:89: Error walking path: open /Users/df/mygo/src/github.com/leanote/leanote/app/views/member/blog: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:124: Failed to watch /Users/df/mygo/src/github.com/leanote/leanote/app/views/member/group : open /Users/df/mygo/src/github.com/leanote/leanote/app/views/member/group: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:89: Error walking path: open /Users/df/mygo/src/github.com/leanote/leanote/app/views/member/group: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:124: Failed to watch /Users/df/mygo/src/github.com/leanote/leanote/app/views/member/user : open /Users/df/mygo/src/github.com/leanote/leanote/app/views/member/user: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:89: Error walking path: open /Users/df/mygo/src/github.com/leanote/leanote/app/views/member/user: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:124: Failed to watch /Users/df/mygo/src/github.com/leanote/leanote/app/views/note : open /Users/df/mygo/src/github.com/leanote/leanote/app/views/note: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:89: Error walking path: open /Users/df/mygo/src/github.com/leanote/leanote/app/views/note: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:124: Failed to watch /Users/df/mygo/src/github.com/leanote/leanote/app/views/notebook : open /Users/df/mygo/src/github.com/leanote/leanote/app/views/notebook: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:89: Error walking path: open /Users/df/mygo/src/github.com/leanote/leanote/app/views/notebook: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:124: Failed to watch /Users/df/mygo/src/github.com/leanote/leanote/app/views/oauth : open /Users/df/mygo/src/github.com/leanote/leanote/app/views/oauth: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:89: Error walking path: open /Users/df/mygo/src/github.com/leanote/leanote/app/views/oauth: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:124: Failed to watch /Users/df/mygo/src/github.com/leanote/leanote/app/views/share : open /Users/df/mygo/src/github.com/leanote/leanote/app/views/share: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:89: Error walking path: open /Users/df/mygo/src/github.com/leanote/leanote/app/views/share: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:124: Failed to watch /Users/df/mygo/src/github.com/leanote/leanote/app/views/user : open /Users/df/mygo/src/github.com/leanote/leanote/app/views/user: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:89: Error walking path: open /Users/df/mygo/src/github.com/leanote/leanote/app/views/user: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:124: Failed to watch /Users/df/mygo/src/github.com/revel/revel/templates : too many open files
ERROR 2015/08/23 16:25:02 watcher.go:89: Error walking path: open /Users/df/mygo/src/github.com/revel/revel/templates: too many open files
ERROR 2015/08/23 16:25:02 watcher.go:124: Failed to watch /Users/df/mygo/src/github.com/revel/modules/testrunner/app/views : too many open files
ERROR 2015/08/23 16:25:02 watcher.go:89: Error walking path: open /Users/df/mygo/src/github.com/revel/modules/testrunner/app/views: too many open files
ERROR 2015/08/23 16:25:02 server.go:127: Failed to listen: listen tcp :59420: socket: too many open files

I tried many methods.But it doesn't work.
How to solve it?

@wjc133 From FAQ of fsnotify:

How many files can be watched at once?
There are OS-specific limits as to how many watches can be created:

BSD / OSX: sysctl variables "kern.maxfiles" and "kern.maxfilesperproc", reaching these limits results in a "too many open files" error.

Have you tried changing the limit of open files [1] [2] [3]?

@anonx OK! It works! Thank you very much indeed.

@wjc133 You're welcome.

:+1: team work! The crowd wins again :crown:


Fixes test comment

Just what it says on the tin; I left this NOTE comment in and I can't for the life of me remember what I had meant to say.

haha no worries!


Nav Walker

What would be the best way to extend the WP Walker_Nav_Menu menu class? I'm trying to create a custom nav walker.

Under the Structure dir I make a file called navwalker.php with this code in it
class Navwalker extends \Walker_Nav_Menu {
//code here
}
then in config/app,php add this to the autoload array
'Structure/navwalker.php',

I would recommend putting your Walker classes at app/Structure/Walkers/nav-walker.php and import it in config/app.php file.
As folder structure is flexible this is a recommendation, but feel free to structure it how it feels right for your project.


Update cleanup.yml

When deploying using a non-privileged user, clean-up process could fail in case read-only files need to be deleted.
Most of the web applications have read-only configuration files.

👎  your user should have permissions to modify these folders as you are deploying there (and creating all the folders structure)


Fixed docs for master tag since not using latest

There is no tag reference in the current compose example or README.  Default tag references for docker tooling will use "latest" as a defacto standard, so explicit tag references should be added.

Closing to branch my fork for separate PR


DepthOfFieldEffect can preserve transparency?

Description of the bug
Hi! again me jajaja
DepthOfFieldEffect makes the background black.
To Reproduce
const renderPass = new RenderPass(scene, camera)

    const smaaImageLoader = new SMAAImageLoader()
    smaaImageLoader.load( (e) => {
        const smaaEffect = new SMAAEffect( e[0], e[1])
        smaaEffect.edgeDetectionMaterial.setEdgeDetectionThreshold(0.05)
        const smaaPass = new EffectPass(camera, smaaEffect)
        // composer.addPass(smaaPass)
    })

    const noiseEffect = new NoiseEffect({ premultiply: true })
    noiseEffect.blendMode.opacity.value = .33

    const bloomEffect = new BloomEffect({
        intensity: 0.55,
        luminanceThreshold: 0.8,
        luminanceSmoothing: 0.1,
        kernelSize: KernelSize.VERY_LARGE,
    })

    const toneMappingEffect = new ToneMappingEffect({
        mode: ToneMappingMode.ACES_FILMIC
    })

    const p = {
        focusDistance: 0,
        focalLength: 0.05,
        bokehScale: 5
    }
    gui.add(p, 'focusDistance').min(0).max(1).step(0.05)
    gui.add(p, 'focalLength').min(0).max(1).step(0.05)
    gui.add(p, 'bokehScale').min(0).max(3).step(0.1)

    const dofEffect = new DepthOfFieldEffect( camera, {
        focusDistance: p.focusDistance,
        focalLength: p.focalLength,
        bokehScale: p.bokehScale
    })

    const ePass = new EffectPass(
        camera,
        dofEffect,
        noiseEffect,
        bloomEffect,
        toneMappingEffect,
    )

    composer = new EffectComposer(renderer, {
        frameBufferType: THREE.HalfFloatType
    })
    composer.addPass( renderPass )
    composer.addPass( ePass )

Expected behavior
Preserve transparent canvas.
Screenshots
With

Without

Library versions used

Three: 124
Post Processing: 6.21.3

Desktop

OS: [e.g. Windows 10]
Browser [e.g. Chrome <IP_ADDRESS>]
Graphics hardware: [e.g. NVIDIA GTX 1050]

Mobile

Device: [e.g. iPhone6]
OS: [e.g. Android 10]
Browser [e.g. Firefox 68.11.0]

Thanks for the report!
This was recently fixed in https://github.com/vanruesc/postprocessing/commit/89569e1bac01eaedab5b26febac8b81279c30385. The fix will be included in the next release. You can use the following code to make it work now:
import { MaskFunction } from "postprocessing";

// @TODO This is a workaround. Remove it in postprocessing 6.21.4.
const maskMaterial = depthOfFieldEffect.maskPass.getFullscreenMaterial();
maskMaterial.maskFunction = MaskFunction.MULTIPLY;

Related: https://github.com/vanruesc/postprocessing/issues/226#issuecomment-822537302

Fixed in<EMAIL_ADDRESS>

That means I dont need to use a MaskFunction in 6.21.4?

That means I dont need to use a MaskFunction in 6.21.4?

Yes, the code I posted above was just a temporary workaround.


build error

when I build a model with cog, I always get this error and fail to build. How can I handle this?
 => ERROR [stage-0  3/12] RUN curl https://pyenv.run | bash &&  git clone https://github.com/momo-lab/pyenv-install-latest.git "$(pyenv root)"/plugin  125.3s
------
 > [stage-0  3/12] RUN curl https://pyenv.run | bash &&         git clone https://github.com/momo-lab/pyenv-install-latest.git "$(pyenv root)"/plugins/pyenv-install-latest &&        pyenv install-latest "3.8.2" &&         pyenv global $(pyenv install-latest --print "3.8.2"):
#8 1.093   % Total    % Received % Xferd  Average Speed   Time    Time     Time  Current
#8 1.093                                  Dload  Upload   Total   Spent    Left  Speed
100   270  100   270    0     0    102      0  0:00:02  0:00:02 --:--:--   102
#8 3.785 curl: (7) Failed to connect to raw.githubusercontent.com port 443: Connection refused
#8 3.788 /bin/sh: 1: pyenv: not found
#8 3.811 Cloning into '/plugins/pyenv-install-latest'...
#8 124.1 fatal: unable to access 'https://github.com/momo-lab/pyenv-install-latest.git/': gnutls_handshake() failed: The TLS connection was non-properly terminated.
------
executor failed running [/bin/sh -c curl https://pyenv.run | bash &&    git clone https://github.com/momo-lab/pyenv-install-latest.git "$(pyenv root)"/plugins/pyenv-install-latest &&        pyenv install-latest "3.8.2" &&         pyenv global $(pyenv install-latest --print "3.8.2")]: exit code: 128
ⅹ Failed to build Docker image: exit status 1

@bfirsh Thanks for you quick reply! I don't think I use any kind of proxy, but I actually use a shcool network. I never try to downloaded something from Github when I built a docker image before. Maybe use proxy in docker can solve this? But it seems can't be done with cog build command.

@bfirsh I am trying to build some useful and easy to test models like wav2lip and GPEN. I used to build GPEN docker image successfully with my own dockfile but the source code was downloaded in local and used 'ADD' command to add in the image.

Using the method mentioned in https://docs.docker.com/network/proxy/ can handle this problem.

Thanks for following up and sharing how you worked around this, @Hurricane-eye. ✨

@all-contributors please add @Hurricane-eye for bugs and questions


Single select should restore the option after it's been deselected

Single select should restore the option after it's been deselected

fixed


fix: t9s url resolver for create-new requests

Description
Fixes an issue where the resolvedUrl returned by the InsecureTinyliciousUrlResolver for requests which contain the header that indicates a "create new" request, uses the absolute URL included in the request as documentId, instead of extracting it from said URL (like the non-create-new flow does).
Reviewer Guidance
The review process is outlined on this wiki page.
AB#5428

@anthony-murphy and @DLehenbauer added you because you're some of the few people who have touched the t9s url resolver; just for a sanity check. The issue was introduced back in 2021 by someone not on the team anymore, otherwise I would've tagged them :).


Notifications from Switches

Is there any way for homebridge to enable notifications for switch devices either through code or tags? Or is this simply a limitations of the homekit platform?

Thanks.

Was thinking I could use IFTTT's Webhooks.  While initial testing seemed fine but further testing showed how unreliable it is.  Calls into the service aren't registered.  No warning of potentially hitting Api limits.  Any thoughts or suggestions would be appreciated.


Install Azurite Github option might have npm dependency

npm is one of the 5 install options. I used GitHub as I did not have npm/Node.js installed. GitHub option does explain the Git dependency, but not the npm one, which is a bit confusing as the npm install option does explain this.
If I haven't made a mistake, maybe it would be good to have a similar explanation at the top of the GitHub install:
This installation method requires that you have Node.js version 8.0 or later installed.

Thanks

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

ID: 12fa98d9-0901-84c7-f144-e89d9b7c036d
Version Independent ID: db1fe92d-d22b-8a32-7061-d7960f41311e
Content: Use Azurite emulator for local Azure Storage development
Content Source: articles/storage/common/storage-use-azurite.md
Service: storage
Sub-service: common
GitHub Login: @normesta
Microsoft Alias: normesta

@TomSutcliffe Thanks for reporting this. I have made a PR to include this requirement. The changes should go live later today.


Improve README.md

Is your feature request related to a problem? Please describe.
The README.md currently summarises the project goals. This is great for starting out, but once the codebase has acheived a decent size, it is ideal that we explain the functioning of the game and other details.
Describe the solution you'd like
A well designed README.md with instructions to play the game, screenshots of the game, and (ideally) a link to the current data used in the game.

This issue is closed with PR #48.


Make landing page "cool" by outlining features

This Cardhop announcement page is awesome looking. Once GitHawk features are stable, would love to create something like this:
https://flexibits.com/cardhop

This is done


Documents containing sensitive business information must not be made public

This rule originates from company policy to not release company secrets publicly. This will be addressed by keeping all documents containing sensitive information about Infotect on a private repository.

Removing duplicate business rule


highly_variable_genes fails with batch_key and inplace=False

Came across this while investigating #1032.
import scanpy as sc
pbmc = sc.datasets.pbmc3k_processed()
sc.pp.highly_variable_genes(pbmc, inplace=False)

/Users/isaac/github/scanpy/scanpy/preprocessing/_highly_variable_genes.py:58: RuntimeWarning: invalid value encountered in log
  dispersion = np.log(dispersion)
---------------------------------------------------------------------------
TypeError                                 Traceback (most recent call last)
<ipython-input-1-36b7cb4d9c0f> in <module>
      1 import scanpy as sc
      2 pbmc = sc.datasets.pbmc3k_processed()
----> 3 sc.pp.highly_variable_genes(pbmc, batch_key="louvain", inplace=False)

~/github/scanpy/scanpy/preprocessing/_highly_variable_genes.py in highly_variable_genes(adata, min_disp, max_disp, min_mean, max_mean, n_top_genes, n_bins, flavor, subset, inplace, batch_key)
    347                 ('highly_variable_intersection', np.bool_),
    348             ])
--> 349         return np.rec.fromarrays(arrays, dtype=dtypes)

/usr/local/lib/python3.7/site-packages/numpy/core/records.py in fromarrays(arrayList, dtype, shape, formats, names, titles, aligned, byteorder)
    615
    616     if dtype is not None:
--> 617         descr = sb.dtype(dtype)
    618         _names = descr.names
    619     else:

TypeError: data type not understood

Separately, could we return a dataframe here?

In lines 345 - 348, a list of dtypes was getting appended instead of extended. Fixed in PR #1070
dtypes.append([
    ('highly_variable_nbatches', int),
    ('highly_variable_intersection', np.bool_),
])

Yeah, if you add a test, something very simple like sc.pp.highly_variable_genes(pbmc, batch_key='louvain', inplace=False) we can merge the PR @atarashansky.

Separately, could we return a dataframe here?

Sure, I can do after @atarashansky's PR is merged.

This seems to be fixed now.


Separated out the windows source and sourcetypes

Separated following sources/sourcetypes of windows:
windows AD
source
WinEventLog:DFS Replication
WinEventLog:Directory Service
WinEventLog:Microsoft-AzureADPasswordProtection-DCAgent/Admin
PerfmonMk:DFS_Replicated_Folders
sourcetypes
MSAD:NT6:Netlogon
MSAD:NT6:Replication
MSAD:NT6:Health
MSAD:NT6:SiteInfo
Script:TimesyncConfiguration
Script:TimesyncStatus
windows:certstore:ca:issued
=====================
Windows DNS
source
WinEventLog:DNS Server
sourcetypes
MSAD:NT6:DNS
MSAD:NT6:DNS-Health
MSAD:NT6:DNS-Zone-Information
PerfmonMk:DNS
NOTE: You can see changes on the Cerberos.

@hardikhdholariya  - windows:certstore:ca:issued - This might be using a different macro, please and see if we need to do anything on Product Setup page.

@hardikhdholariya - windows:certstore:ca:issued - This might be using a different macro, please and see if we need to do anything on Product Setup page.

@VatsalJagani we are searching data in all the indexes irrespective of macros so for all the searches of data reviewer dashboard and product setup page is fine.
But macro we are displaying on the product setup page search bar. for that we have to think as currently only one value supported. I'll consider this change on product setup page jira.


Read The Docs Webhook

@drgarcia1986 http://read-the-docs.readthedocs.org/en/latest/webhooks.html#github

Done :+1:


Merge "comparison-elements" and "citation" into "data"

Merge the two directories
Rename "default.bib" to "literature.bib"
Rename "config.json" to "literature-config.json"
Move "bibtex2json.js" to somewhere else. Maybe just to the root directory?

Closed with #124


README.md should be updated with latest changes for compilation

After controller proof of concept merge (#23), make helm-build is no longer available.
This should be updated in README.md file to show current option for building as well as new available options

@mheese : has this Makefile rule change (helm-build to helm-keylime) been done intentionally? I am just curious, as it seems CI is calling make helm-build, so it should also be changed


Correct DQL INSTANCE OF to filter all possible child classes

This is the rebased version of #1441, which fixes #4646.
Here is the original text:

There was a bug in the "INSTANCE OF" operator as described in
https://groups.google.com/forum/#!topic/doctrine-user/B8raq8CNMgg
"INSTANCE OF" was not taking into account subclasses.
It was merely translating the class to its discriminator.
This is not correct since the class can have subtypes and those
are, indeed, still instance of the superclass.
Also, classes may not have a discriminator (e.g. abstract classes).
This commit also provides useful tests to avoid regression.

All done, thanks for the review @phansys

Thanks for the review @lcobucci!
I hope that I addressed al your concerns with my latest commits.

A lot of tests are failing, probably for an unrelated issue... should I rebase? Or merge master?

Ok seems that I've addressed all your concerns, @Ocramius @lcobucci
Build is green with low deps, other failures seems unrelated, maybe they come from master?

Hi! Thank you very much for resuming this PR, I really appreciate that!
I checked this fix once more but unfortunately the issue still occurs when using INSTANCE OF with parameters.
Here is an example:
$dql = 'SELECT p FROM Person p WHERE p INSTANCE OF :parameter';
$query = $em->createQuery($dql);
$query->setParameter('parameter', $em->getClassMetadata(Person::class));

// $result contains only direct instances of `Person`, no subclasses
$result = $query->getResult();

I wrote a failing test case for the issue I mentioned before: https://gist.github.com/taueres/05ff035bb2f72cddd7711b7f93cd9d6a

Thanks @taueres!
I've found that your test case fails also when doing this:
$dql = 'SELECT p FROM Doctrine\Tests\ORM\Functional\Ticket\PersonTicket4646Parametric p
        WHERE p INSTANCE OF :parameter';
$query = $this->_em->createQuery($dql);
$query->setParameter(
    'parameter',
    PersonTicket4646Parametric::class
);

Do we want to check for this additional uses, @Ocramius ?

$query->setParameter(
    'parameter',
    PersonTicket4646Parametric::class
);

This currently requires binding a ClassMetadata instance:
$query->setParameter(
    'parameter',
    $em->getClassMetadata(PersonTicket4646Parametric::class)
);

We can fix it, yes

Ok I've added the failing test case. I'll try to work on that soon; @taueres if you have any tips, I'm all ears! :smile:

Unfortunately solving the parameter issue is not as easy as static DQL queries.
The main problem is that Doctrine is always translating each DQL parameter to one single SQL parameter.
For INSTANCE OF, this 1-to-1 relationship is not always true: one class might be mapped to multiple discriminators.
The issue is not easy to address because the parameter values could still be undefined while Doctrine is parsing the query.
Think about the following scenario:
$dql = 'SELECT p FROM Person p WHERE p INSTANCE OF :parameter';
$sql = $query->getSQL();
// $sql is something like "SELECT * FROM person WHERE person.kind IN (?)"
// which is WRONG because we don't know yet how many SQL parameters are needed.

Do you have any idea of how we could tackle this problem?
Is DBAL supporting IN operator with array parameters?

Is DBAL supporting `IN operator with array parameters?

Yep, but it only supports these two types of bindings, so custom discriminators may be a problem

Okay, I have a solution! INSTANCE OF with parameters now works correctly on my branch https://github.com/taueres/doctrine2/tree/fix-instance-of-subclasses
It still needs some minor improvements, e.g. duplicated methods.
How should I proceed with this PR? Should I create a new one from my branch?

@taueres I suggest waiting for @Jean85 to merge the changes into his branch

I've merged your fixes @taueres, thanks a lot!
Now the only issue that I see is the duplication of the getAllDiscriminators(ClassMetadata $classMetadata) method in both the Query and SqlWalker classes. Any suggestion on how to address that? It uses $this->getEntityManager()->getClassMetadata($class), which I think is the biggest hurdle to overcome, otherwise it could be moved inside the ClassMetadata class.

To solve the duplication issue we could create something like:
class HierarchyDiscriminatorResolver
{
    public static function resolveDiscriminatorsForClass(ClassMetadata $rootClassMetadata, EntityManagerInterface $entityManager)
    {
        /* Duplicate code goes here */
        /* return list of discriminator */
    }
}

In the proposed solution, the EntityManager dependency is needed in order to retrieve ClassMetadata for subClasses.
I would not put such logic into ClassMetadata because atm ClassMetadata is not coupled to EntityManager.

This seems a feasible solution for me... @Ocramius ? What do you think?

@Jean85 as long as the class is marked as @internal, sure :+1:
Totally missed these comments before btw.

Don't worry @Ocramius, I was on vacation anyway :smile:
Deduplication of code done, build is green, this is ready for a final review!

According to the DQL spec. Users can provide multiple classes for an INSTANCE OF statement.
InstanceOfExpression     ::= IdentificationVariable ["NOT"] "INSTANCE" ["OF"] (InstanceOfParameter | "(" InstanceOfParameter {"," InstanceOfParameter}* ")")

Do we have some tests covering this specific use case?

@lcobucci @Ocramius done as per review!

I've also added a new functional test to cover the case of multiple parameters passed to the INSTANCEOF, as suggested by @taueres

Thanks @Jean85 @taueres and @phansys!


test(theme-common): improve test coverage

Motivation
Wrote A LOT of tests 👨💻
This also got rid of a jest.mock usage, which seems to be incompatible with ESM yet.
We now have a lot of tests that reach into DocusaurusContext. Could probably be refactored as a test helper?
Have you read the Contributing Guidelines on pull requests?
Yes

Thanks 👍

We now have a lot of tests that reach into DocusaurusContext. Could probably be refactored as a test helper?

It's not uncommon for many large project to have a dedicated package for test utils, so if you feel it's worth it why not 👍
In general I'd also prefer if we don't expose/export/import the raw React context but instead only use DocusaurusContextProvider/useDocusaurusContext (cf what we do in a few places in theme-common)

In general I'd also prefer if we don't expose/export/import the raw React context

That means we can't conveniently provide a mocked context because it's encapsulated in DocusaurusContextProvider🤦♂

That means we can't conveniently provide a mocked context because it's encapsulated in DocusaurusContextProvider🤦♂

Why do you want to mock the context?
A test util could expose a TestDocusaurusContextProvider accepting a partial context and you'd use it on top of JSX tress that you want to test?

Oh yeah, once we have the test util we can encapsulate the context.


Decoding is slow. Anything to make it fast? It is not using multiple cores. At least the default commands in the README.

I am using docker.

Hi @raotnameh , decoding depends on the beam size, size of language model, etc. Also the current fairseq implementation doesn't provide for batch inference, hence the process is slow (and runs on single-thread)!
We are currently working on to release a HF-compatible model. That should speed up the decoding (through batch inferencing).


Add vscode dev container

Adding dev container to make collaborative development easier and standardized.
Advantages:
Each member will have identical environments including formatting, linting, extensions, and settings.
New members can easily set up a development environment
Standardized coding style(could be inconvenient too)
Dependencies:
VS Code IDE with Remote Containers Extension
Docker

@all-contributors please add @satheeshkatipomu for code


Support Moonbase and Moonriver chains for kusama staking

Working on integrating lido-js-sdk in lido kusama staking.
Two new chain ids need to be added to the CHAINS constant in packages/constants/src/chains.ts file:
Moonriver = 1285,
Moonbase = 1287,

Added in 42f55ab


Update README_CN.md

更改“用户”为“用于”。
NOTE: Please read the Contributing.md guidelines before submitting your patch:
https://github.com/apache/incubator-apisix/blob/master/Contributing.md#how-to-add-a-new-feature-or-change-an-existing-one
Summary
SUMMARY_HERE
Full changelog

[Implement ...]
[Add related tests]
...

Issues resolved
Fix #XXX

@QuakeWang many thx, merged


The README is out of date

README refers to a lighthouse account which doesn't exist and a RunCodeRun page which doesn't exist.

Fixed by #24