How to handle combine features

Hi,
thanks for this excellent power tools.
I have one master document and 3 contributor documents. I want to combine all documents one by one and i want to see all changes. I need a feature in word application Review > compare > combine. I tried with word.interop its working fine. But I can't expose as service also i need to install word. I tried with both compare examples in power tools. but its not  solve my problem. is there any other option to combine word documents with revisions and accept and reject should work in word.
Thanks in advance.
@nanda

Closing all issues as this repo is being archived and will no longer be maintained by Microsoft. The project is licensed for continued use and development by forking to your own repo.


Increase strictness of linting

Description
Remove exceptions of the tests / examples folder from pytest exclude.

Blocked by #85


chore(master): release 0.5.0

:robot: I have created a release beep boop
0.5.0 (2024-04-18)
Features

refactor deleteAccessToken method in GoogleClient (1b07e23)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/tomshaw/google-api/releases/tag/v0.5.0 :sunflower:


Give an entry blank for if they know the URI

In GitLab by @jakebeal on Feb 1, 2019, 21:05
We might not be able to find the right value, but as long as there's a person doing an audit, they might actually be able to give us the exact URI from SynBioHub or the dictionary.  Provide an entry blank to allow that if they want, then remember the choice in the options offered for this term in the future.

In GitLab by @jakebeal on Mar 12, 2019, 11:02
assigned to @sumorok

In GitLab by @jakebeal on May 21, 2019, 10:46
removed assignee

In GitLab by @jakebeal on Jun 5, 2019, 00:56
assigned to @nwalczak

In GitLab by @nwalczak on Jul 1, 2019, 09:08
created branch 12-give-an-entry-blank-for-if-they-know-the-uri

In GitLab by @nwalczak on Jul 1, 2019, 09:08
mentioned in merge request !17

In GitLab by @nwalczak on Jul 1, 2019, 15:48
closed via commit 035936a83bcf17e26b31e43659b2e361e4d66337

In GitLab by @nwalczak on Jul 1, 2019, 15:48
closed via merge request !17

In GitLab by @nwalczak on Jul 1, 2019, 16:23
mentioned in commit 035936a83bcf17e26b31e43659b2e361e4d66337


Keep the whitespace sequences, but the normal wrap.

Keep the whitespace sequences, but the normal wrap.some widgets need to wrap

Hi, the change is OK but I would like to confirm with you first.
As you said that "widgets" need to wrap, would it better to make the changes to "td" of "tbody" instead of "td" of "thead"?

ok , it is better to make the changes to "td" of "tbody" .  "td" of "thead" maybe wrap is less.

I will publish these changes in next release. Thanks.


strange tag while writing xml with nullValue

while writing xml with:
df.repartition(1).write.format("xml")\
    .option("rootTag", "DataContainer")\
    .option("rowTag", "Свед")\
    .mode("overwrite")\
    .save(data_path))

get strange tag in resulting xml for elements, which were passed with Null values. for example:
<БазСвед>
    <Фамилия>some text</Фамилия>
     ....
</БазСвед>
<СведЕСИА/>

Check the last row. also, pay attention at slash and its position in tag.
Using:
spark-xml_2.12-0.14.0.jar
txw2-2.3.4.jar
scala 2.12.15
spark 3.3.1

What is strange about this?

@srowen this is neither open nor close tag. it's like alone.
If we have some data inside, tags should look like:
<БазСвед> # open
....
</БазСвед> #close
and none of them if there is null.

@srowen it happens each time we have nested stucture with all null nested elements.
So, that's how the parent element is written. I guess, it should not be written at all.

All the tags you show are closed, I'm not sure what you're referring to?

@<БазСвед>
<Фамилия>some text</Фамилия>
....
</БазСвед>
<СведЕСИА/>
<СведЕСИА/> - this one from my example. it's alone

No, that is a closed tag. It's empty. <foo/> is the same as <foo></foo> in XML.

@srowen
ok, but is there any way it won't be written at all as it is empty?
all nested tags are not written and that's what we expect. but why this parent tag is still written? even being empty

I don't think it's possible, no. You can post-process the XML how you like though.


[DRAFT] fix:Gate nodes associated with a machine based on resolved .status.providerID

Fixes #
Description
How was this change tested?
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

9 of 11 (81.82%)  changed or added relevant lines in 1 file are covered.
9 unchanged lines in 2 files lost coverage.
Overall coverage increased (+0.4%) to 81.675%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/controllers/machine/termination/controller.go
9
11
81.82%

Files with Coverage Reduction
New Missed Lines
%

pkg/scheduling/requirements.go
2
98.5%

pkg/controllers/node/controller.go
7
70.59%

Totals

Change from base Build<PHONE_NUMBER>:
0.4%

Covered Lines:
6690

Relevant Lines:
8191

💛  - Coveralls


Markdown Syntax Highlighting not making a text bold if it's a part of a link

I'm talking about [**this**](). Markdown Preview seems to make it a valid formatting but the editor doesn't make it bold, only color highlights.
To prove it, this is a valid formatting in GitHub.

This issue was moved to atom/markdown-preview#218


Motion Preamble is not shown in Motion creating

Describe the bug
You can set the motion preamble in the settings of openslides, but when you are creating an motion the preamble is not show over the decision field.
So a lot of our members create a mention with the preamble additional to the default.
How to Reproduce

Set a default motion preamble, example "The parliament decides, "
Create a new motion
Our Users set the decision to, "The parliament decides, to do ...", because the preamble is not shown.
Save the motion
See in the motion the Decision "The parliament decides, The parliament decides, to do ..."

Expected behavior
Motion preamble is shown atop of the decision field, like in the mention overview.
System information

OpenSlides-Version: 3.1
Example Images

Step

Step, see no motion preamble

Step, what our members do

Step, see the duplicate

I hope the new description is clear enough

While checking the motion edit mask, i suspect the decision title for the input field is missing.

Clear enough. Thanks.
@emanuelschuetze
it was proposed to show the "motions_preamble" config variable in the motion create form above the first TinyMCE.

it was proposed to show the "motions_preamble" config variable in the motion create form above the first TinyMCE.

good idea. we should implement this.

already done in 3.2-dev


[Azure Monitor OpenTelemetry] Add OTLP Exporters

Packages updated by this PR: @azure/monitor-opentelemetry
Add support for OTLP HTTP exporters

API change check
APIView has identified API level changes in this PR and created following API reviews.
azure-monitor-opentelemetry


Show a WARNING on the terminal if Cohorte is started with Python 2.x

If user's application contains Java components, Cohorte should start with Python 3.x (JPype requirement). By default Cohorte startup script uses "python" and in many linux-based distributions it is Python 2.x installed on the system. We should explicitly use "--interpreter python3" option or modify the generated "run.js" startup configuration file to indicate that python3 should be used and not python2.
We should warn the user about this requirement at startup. Further, Cohorte should not be started if Java components are used with Python 2.x

We should avoid starting Java Isolates when Python 3 is not used as interpreter (SHOULD BE implemented in 1.1.1).


Create responsive accordion tabs

Documentation of responsive accordion tabs are added

Cannot merge because of conflict pull to branch version 3 on your local and merge locally, fix the conflicts and push back for PR


[BEAM-8432] Enable building kotlin examples with different java versions

gradle.properties contains a javaVersion property that can be overwritten
with -PjavaVersion=11 command line argument while building with gradle
R: @lukecwik could you take a look?

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

Run Java PreCommit

Run Java_Examples_Dataflow PreCommit


Apply "Parse fields in classes and generate correct class structs" in gir

Example result of https://github.com/gtk-rs/gir/pull/276
WIP
cc @GuillaumeGomez

Related issue #34, #25.

Seems rust still have regression 😭

Wo, awesome! I'll just wait for appveyor and ignore beta/release for travis. This is really annoying. :-/

appveyor failed with a "normal" error.

@GuillaumeGomez so you don't against this change?

Removed empty lines between classes and interfaces by removing first line in generate_interfaces_structs

I don't however, however, you must fix appveyor's failure first.

Fixed appveyor upgrading rust to 1.9 (stable will fail), so we need decide what we need ask @eugene2k to do.

Updated with results of https://github.com/gtk-rs/gir/pull/298

Updated, ready for merge

Thanks!


[Backport release-23.11] github-runner: 2.315 -> 2.316.1

Description of changes
Backporting https://github.com/NixOS/nixpkgs/pull/308783

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[x] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Usually when we're backporting a bump where an intervening  version has been "missed" we try to cherry-pick each intervening commit separately, with references to their respective master-branch commits. Here we've ended up with a commit with the message .. 2.316.0 -> 2.316.1 that's actually bumping from 2.315.0 -> 2.316.1.


Terminal & zsh color scheme

VSCode Version: 1.2.1
OS Version: Ubuntu 14.04

Steps to Reproduce:

Install and define zsh as your default $SHELL
Toggle terminal

Color scheme is not the one set in terminal.

@thomas-leroy do you mean the color scheme from gnome-terminal does not carry over? This is by design, the theme of the terminal will eventually be customizable but it will not integrate with gnome-terminal or similar applications. https://github.com/Microsoft/vscode/issues/6979


feature: %filename template variable

Did you check the docs?

[X] I have read all the docs

Is your feature request related to a problem? Please describe.
When having serveral log statements in a project, it's useful to know which file each log originated from. I'd like to customize my log statements like this (for lua, as an example):
lua = [[print("%filename:%line_number %log_target=" .. vim.inspect(%log_target))]],

for a result of:
print("timber.lua:32 foo=" .. vim.inspect(foo))

Describe the solution you'd like
A new template variable, maybe %filename
Describe alternatives you've considered
I don't think I can find an alternative when using timber.nvim
Additional context
No response

Thanks for the suggestion. The new placeholder is %filename, it should work now.


about citation of animations in Polyhedra Viewer

Dear Sir or Madam,
I am a student who recently reads the website Polyhedra Viewer https://polyhedra.tessera.li/.
I am wondering if I want to put some figures of polyhedra into my papers, what should be the right reference and citation format? I usually cite contents from papers but never from a specific website so I have no experience on that.

Please provide details about it as far as possible. e.g. in figure, I say "figure is reproduced from Ref. []" or "These structures are created by... []" In bibliography, "author/title/website/HRC/..."

The possible scenaria for use:
One is for my course assignment. (quite simple)
The other is for future possible published papers. (may involve copyright issue)

If copying these fantastic figures from this website is not a good idea, could you also recommend some software for me to create polyhedra?

I appreciate it.
Thanks,
Ruikang

Hi!
Sorry for not seeing this sooner. Your course is probably over, but for future reference of papers, I'm okay with my work being used as long as there is proper attribution (link to the source website and my name). I don't have any preferences for citation style -- that would depend on the publication or standards of your field.
I hope that answers your question.


Create democracyos.github.io

The idea is to have a special site dedicated for the developers about the app.
It must include:

[ ] information about installation, configuration and running processes
[ ] some description about the technology we are using (node.js, browserify, Babel, jade, etc)
[ ] screenshots of the main features
[ ] tutorials on how to use the admin, how to add translation keys, how to deploy to heroku, how to release a new version and other "How to's"...
[ ] contributing to DemocracyOS
[ ] use cases and experiences all around the world
[ ] links to DemocracyOS gitter and mailing list

I would argue that we shold aim for either democracyos.org or democracyos.io for this website

Related: https://github.com/DemocracyOS/app/issues/454

http://docs.democracyos.org/ :smiley:


Run on Wampserver or IIS

I have one application using Slim 4 and want to know if possible using this on Wampserver or IIS Server?

No. Ratchet is its own HTTP server. It does not run on CGI.


printf -v not supported

printf -v creates a variable, but subsequently trying to access that variable triggers the "Unresolved variable" inspection warning.

@Bilge Thanks, I didn't know about printf. This is fixed for the next update now.


Remove explicit depencency on ltest

ltest is only used in this one particular provider, so it makes sense to me that we remove the explicit dependency on it and instead rely on the user of the plugin to list ltest in their dependencies. This is technically a breaking change, however the templates already list ltest as a separate dependency so I think that should be OK. I also added a warning with what to do if you don't have it installed.
Supersedes #87

Yeah, I think this is a good compromise.
We can always cut a quick point release if this causes any unexpected problems :-)


Wifi appears to work only when I'm not running code

This is an odd thing. I'm connected to a Wifi network. The yellow light blinks intermittently, and if I run t2 wifi I get this positive message:
➜  ktest t2 wifi
INFO Looking for your Tessel...
INFO Connected to yayLookItsATessel.
INFO Connected to "WebRebels"
INFO Signal Strength: (35/70)
INFO Bitrate: 7mbps

However, I am trying to run the Twitter example. And every time I run it, the yellow light turns off for the duration, the tweet fails to send, the code completes and exits, and then the yellow light comes back.
➜  ktest t2 run twitter.js
INFO Looking for your Tessel...
INFO Connected to yayLookItsATessel.
INFO Building project.
INFO Writing project to RAM on yayLookItsATessel (950.272 kB)...
INFO Deployed.
INFO Running twitter.js...
error sending tweet: { [Error: getaddrinfo ENOTFOUND api.twitter.com api.twitter.com:443]
  code: 'ENOTFOUND',
  errno: 'ENOTFOUND',
  syscall: 'getaddrinfo',
  hostname: 'api.twitter.com',
  host: 'api.twitter.com',
  port: 443 }

I've repeated this 5 times now, and it's very consistent. I also tried running the accelerometer example (where Wifi is not used) and observed the same yellow-light behavior.
Any clues? Is anyone else experiencing this?

Ah- I think it's related to https://github.com/tessel/t2-cli/issues/755. I dterm'd in, as in the other issue, and deleted the extra config wifi-iface from the bottom of Tessel's etc/config/wireless. Then I ran the Twitter example again and it worked perfectly.
We need to figure out why that extra line is being written.

I found out why the extra config wifi-iface is being added to the bottom of the file. This line is intended to check if an existing wifi-iface is set up for a LAN and it adds a new one if one wasn't detected. However, based on the default /etc/config/wireless file as edited here, there should always be an existing wifi-iface for LAN. This detection method is screwy because it checks for the access point SSID as a way of checking to see if the access point exists at all but we, by default, have an access point with no SSID. Therefore, it creates a new wifi-iface. This can be fixed by simply checking to see if uci get iwifi-iface[1] returns an error.
I have also seen that not explicitly disabling the wifi interface will cause the ap to not be found so we should explicitly disable wifi when setting the access point on (and vice versa).

Additional point of interest: I had the ap on, and ran t2 wifi to find that it was ... connected to itself
➜  ktest2 t2 wifi
INFO Looking for your Tessel...
INFO Connected to yayLookItsATessel.
INFO Connected to "Tessel"
INFO IP Address: <IP_ADDRESS>
INFO Signal Strength: (undefined/70)
INFO Bitrate: NaNmbps

It's possible that this was fixed with https://github.com/tessel/t2-cli/pull/761. We need someone to try to reproduce this (which may be difficult because we've only seen it on one network). I'm going to close this for now but we can re-open if it rears it's ugly head once again.


Use writeShellApplication for packaging

With this change the packaging will be made with writeShellApplication which allows adding runtime inputs (deps) to the script by setting the $PATH variable at the top of the file. It also has a check phase which tests the shell file against shellcheck like we did before manually. It also sets the safety bash options.
you can read more about writeShellApplication here -> https://github.com/NixOS/nixpkgs/blob/d3a307c813cbd209fe1e13ac9246b3fa3ea93c12/pkgs/build-support/trivial-builders.nix#L282

How should I go about bringing the flake's package output into scope in tests/from-nixos.nix for example?

I have a fork/branch for with some more improvements (& that utilizes of flake.parts), is that something you are interested in me PR'ing over?

bors merge

I have a fork/branch for with some more improvements (& that utilizes of flake.parts), is that something you are interested in me PR'ing over?

Yes. Looks useful: https://github.com/Kranzes/nixos-remote/commit/b400913411bf42a49479abd50c583935f704a0cb

But maybe keep the packages with importable with callPackage. I think some folks still want to manually import them without using flakes.

But maybe keep the packages with importable with callPackage. I think some folks still want to manually import them without using flakes.

what about using builtins.getFlake?

I have a fork/branch for with some more improvements (& that utilizes of flake.parts), is that something you are interested in me PR'ing over?

Yes. Looks useful: Kranzes@b400913

I created the PR: #23

nixos-remote can be still used without flakes. All parameter have fallbacks for store paths.


Replaced the location and list icons with nicer ones.

Thought the default ones were a bit blurry and out of place. These icons are extracted from iOS itself, so shouldn't need any attibution (I think). Includes @3x images.

Ideally it would be nice to replace the location button with a MKUserTrackingBarButtonItem, but that would change the current behavior. MKUserTrackingBarButtonItem allows for map tracking with and without heading (which would probably expose the rotation bug). Currently the button just updates the location.

Do you have a before and after screenshot?

New:

Old:

Looks good. Do you happen to be interested in also tackling the fact that the location button always shows as solid even when not tracking the users current location?

Sure, #432.


Prepare branch for release

Codecov Report

Merging #23 into develop will not impact coverage.

@@            Coverage Diff             @@
##             develop      #23   +/-   ##
==========================================
  Coverage      83.92%   83.92%
  Complexity        51       51
==========================================
  Files             14       14
  Lines            311      311
==========================================
  Hits             261      261
  Misses            50       50

Continue to review full report at Codecov.

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


Feature see dishes

Hello, here is the site for checking our dishes

Please use this template for the PR description:
Waffle Story: [paste Waffle tracker link here]

Changes proposed in this pull request:

[item 1 - replace me] [item 2 - replace me]

What I have learned working on this feature: [If you don't put anything here you are doing it wrong!]

Screenshots: [If you made some visual changes to the application please upload screenshots here, or remove this section]

Ready for review! [If ready]


Turn on noUncheckedIndexedAccess tsc flag

What does this change?
Turns on the noUncheckedIndexedAccess tsc flag to make things a little stricter. Check out: https://devblogs.microsoft.com/typescript/announcing-typescript-4-1-beta/#no-unchecked-indexed-access for all the details. In short it adds | undefined to indexed access eg.
interface Foo {
  [props: string]: string;
}

const foo: Foo = { /*...*/}

// will now have type string | undefined
const bar = foo["bar"]

We we're having an issue with this in PROD that meant we were calling toUpperCase() and undefined. It also makes array indexing more strict, which is probably a good thing, but it often flags things that by inspection we can tell are legit.
Also had to update  @typescript-eslint/eslint-plugin and @typescript-eslint/parser which had some breaking changes/new rules that needed fixing (I mainly just added new ignores, it would be good to see if we can clean up some of those at some point).

FYI I think a lot of type assertions (thing as type) can be avoided with type predicates (thing is type).
See https://github.com/guardian/libs/pull/211/files and https://github.com/orgs/guardian/teams/client-side-infra/discussions/20


chore: add missing data

Tip for review, skip the generated code and go straight for the tests 😄

Do you have any other questions here @evilaz ?

In addition to attributes we also have missing images and prices. Wonder if its also worth making tests for the other two endpoints?

I will add them in a separate PR @mbarga .

Do you have any other questions here @evilaz ?

No I'm good, I don't get much anyway was just curious :)


Morph Target Support

We should add morph target support to the engine. We'll need POSITION0..7 semantics (preumably 8 is enough?), GLSL to do the interp. Obviously, we want the FBX conversion tool to read this data and write it to model JSON, but since the engine needs to work with procedurally created meshes, it's not a blocker.

Hi Will,
Just wondering have you guys made any progress regarding to supporting morph animation in Playcanvas?

Hi @issacclee - this work is still not scheduled. Can you give me the use case of what you want this feature for (just out of curiosity)?

Hi @willeastcott , sorry for such late reply. I was in need for such feature because we are trying to simulate the process of turning machine cutting through metal piece which of course would require the ability to either import morph animation made by 3d tools like 3dsmax or have the engine work with procedurally created mesh.

Could it be achieved with skinning?

Hey @willeastcott. Would love to get this feature in there. I'm really new to Playcanvas and trying to push it atm to see whats achievable. Working on characters and be great to use non-boned facial animation among other things  : )

Another thing this would be useful for is custom character creation. Or similarly, producing random variations of characters for crowds etc. As an example: https://threejs.org/examples/?q=morph#webgl_morphtargets_human   I may be unable to use PlayCanvas without this feature, unless it offers another way of achieving this. (I'm also a brand new user, but enjoying it very much so far!).

You're in luck @westoncb - we're going to look into adding support for this next. Good timing!

Great! Glad to hear it. Meanwhile I'm going to attempt constructing a VertexBuffer manually that interpolates between some static meshes with identical topology.

any news on this? doing character animation without corrective shape keys is not great. There are also limitless amounts of effects easily achieved by a simple morph target that would be impossible with skinning.  I didn't realize this was unsupported when I signed up because its supported in every other engine.

Hi! Morph targets are supported now. They're imported from FBX and can be used, but the API is a bit low-level, so we didn't document it properly, hoping to add a better public API later. However, you can use these methods: https://github.com/playcanvas/engine/commit/3b42ad2b470d5d7b096af52c30485174e0cd37ce

In fact you only need to use meshInstance.morphInstance.setWeight

Does this support animated weights from the fbx file or would that have to
be re-animated via scripting now. Whats the eta on a finished
implementation? Mostly i'm recreating a sketchfab type of experience that I
can host myself. Its been a decent experience so far, although I would pay
more to remove the requirements of using the cloud...
On Oct 25, 2017 8:00 AM, "Mr F"<EMAIL_ADDRESS>wrote:

In fact you only need to use meshInstance.morphInstance.setWeight
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/playcanvas/engine/issues/179#issuecomment-339306948,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfjrO7ik4vNb4nItPscreDBWch-4DP8qks5svyLvgaJpZM4Dt3Ep
.

It doesn't supported loading animated weights... in fact I didn't realize there was such a thing. If they are really in the FBX, that can be implemented. For now it can be done via scripting.

XD yes thats part of fbx and used often
On Oct 25, 2017 12:20 PM, "Mr F"<EMAIL_ADDRESS>wrote:

It doesn't supported loading animated weights... in fact I didn't realize
there was such a thing. If they are really in the FBX, that can be
implemented. For now it can be done via scripting.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/playcanvas/engine/issues/179#issuecomment-339386505,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfjrO00JPXCE4jn53D0qx67qicltQEe7ks5sv1_GgaJpZM4Dt3Ep
.

How would you define multiple animated tracks from a single FBX though?

https://docs.unity3d.com/Manual/BlendShapes.html
http://answers.unity3d.com/questions/675871/exporting-keyed-blendshape-animations-from-maya.html
On Oct 27, 2017 6:43 AM, "Mr F"<EMAIL_ADDRESS>wrote:

How would you define multiple animated tracks from a single FBX though?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/playcanvas/engine/issues/179#issuecomment-339937912,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AfjrO08XE8Y6547LDqaq22UMVAewqd0fks5swbO_gaJpZM4Dt3Ep
.

Hi @slimbuck. This feature request was opened long ago. Since then, rudimentary morph target support has been added to PlayCanvas. But I would describe it as incomplete. Some missing things:

No engine example demonstrating procedural morph creation + playback.
No engine example demonstrating loading and playback of morphs from glTF.
API probably needs tightening up. I'm not convinced it's in a nice, final form (see pc.Morph, pc.MorphTarget and pc.MorphInstance).
No support for keyed morph animation playback (will be part of the new animation system).

So rather than close this, I'll assign to you for now until we feel it's properly fit for purpose.


Fixed bot randomly removing owner

I did check this earlier last week but I could not find the problem. Thank you 👍


Gh 126 fix albam version in error report

Include correct version in error report (right now that would be 0.4.0.dev11) and redact user home.
Also say how to read the console in unexpected errors.

@HenryOfCarim  could you please test that the user home is redacted in Windows?

@Brachi Well, the result is strange, it doesn't show the report as it did before, in the console for errors like this

For other errors it shows the trace with the username
  File "C:\Users\myname\AppData\Roaming\Blender Foundation\Blender\3.6\scripts\addons\albam\vfs.py", line 270, in execute
    w.write(vfile.get_bytes())
  File "C:\Users\myname\AppData\Roaming\Blender Foundation\Blender\3.6\scripts\addons\albam\vfs.py", line 67, in get_bytes
    accessor = self.get_accessor()
  File "C:\Users\myname\AppData\Roaming\Blender Foundation\Blender\3.6\scripts\addons\albam\vfs.py", line 76, in get_accessor
    root = vfs.file_list[self.tree_node.root_id]
KeyError: 'bpy_prop_collection[key]: key "" not found'```

@HenryOfCarim  Can you try to force an unexpected error message and check the traceback in the console?
Per #118 , you can force by simply removing a material and then exporting.

For other errors it shows the trace with the username

Is this from an unexpected error as well? or a "check" that failed?

@Brachi  For this error it worked as expected


TlsCertificate.ToX509() doesn't work for generated self-signed certificates

TlsCertificate.ToX509Certificate() is having trouble parsing the private keys generated for self-signed certificates.  It's seeing only three fields when nine are required.  I dumped a private generated key by TlsCertificate.CreateSelfSigned() with OpenSSL and compared  it to a (presumably self-signed client key) generated by Kubernetes  and they both look like they have the same number of parameters; so this is a bit weird.
We don't need this functionality right now, so I'm putting this on the backlog.  I hope to be able to address this when we upgrade TlsCertificate to use the improved .NET Core 3.0 crypto APIs.

This article might help: https://www.codeproject.com/Articles/5315010/How-to-Use-Certificates-in-ASP-NET-Core


[Feature Request] Compressing HTTP requests with LZ4 and other compression algorithms

Is your feature request related to a problem? Please describe
OpenSearch service supports only gzip for compressing HTTP requests. My team runs low latency systems and for Redis, we have seen LZ4 provides lower compression/decompression time at higher throughput and lower latency for over the wire transfers compared to gzip and deflate.
Describe the solution you'd like
OpenSearch be modified to support LZ4 also for HTTP requests. I see it's used as default codec for storing index, so we should also consider it for compression for request/response.
Related component
Search:Performance
Describe alternatives you've considered
No response
Additional context
No response

Related - https://github.com/opensearch-project/OpenSearch/issues/10684.  Curious what the perf comparison would be here with protobuf?

@mch2 Protobuf and LZ4 are different things. I have seen LZ4 compression used on top of Protobuf in my org and it improves the results. Compression works better when there are duplicate fields and data in the object being transferred which is always the case with OpenSearch response which contains a bunch of documents with same field names and in some cases have similar or same field values. Protobuf is a binary replacement for JSON formatting while LZ4 is a faster compression replacement. Whether OpenSearch adds support for Protobuf/gRPC or not, the decision of adding LZ4 support runs in parallel and would benefit regardless.


Improved performance benchmark

Add many new benchmarks

with many calls, with many classes
Singleton, Transient, Scoped, Mixed, Complex with simple and deep dependences

Add all benchmark results to 'docs/benchmark'
Update Readme benchmark results.

Some new benchmarks show that MEDI is still faster than jab in some cases.

Hi @rafaelsc, thank you for your work. I'm a bit swamped by work so it might take me a bit to get through these benchmarks.


Is the source available for the jar file?

I would like to test this out using a different TTS provider (Google/Poly/etc.) Can the source be published?

I need to look closer to what's included. All good.


Fixing segmentation fault due to incorrect bounds leading to a stack overflow

I think I found a small bug in the code leading to a stack overflow and in consequence to a segmentation fault on some input graphs.
The definition of the constructor yields the following variable definition:
https://github.com/ucrparlay/Parallel-SSSP/blob/5dc07861c5d655a05e8b94084a9427270e9a7699/sssp.h#L57-L64
This will enable the following code to set db_len to a value which is too large:
https://github.com/ucrparlay/Parallel-SSSP/blob/5dc07861c5d655a05e8b94084a9427270e9a7699/sssp.cc#L47-L56
Notice that db_len can become larger than doubling  since (the +1 I added to db_len is since the loop has as condition <=)
    db\_len = 2+log_2\left(\frac{max\_queue}{min\_queue\cdot 2}\right)+1=2+log_2\left(\frac{2^{\lceil{log_2(N)}\rceil}}{2^{k+1}}\right)+1 =  2+log_2\left(2^{\lceil{log_2(N)\rceil}-k-1}\right)+1 = 2+\lceil{log_2(N)\rceil}-k\\

doubling = \lceil{log_2\left(\frac{\text{max\_queue}}{\text{min\_queue}}\right)}\rceil+1 = \lceil{log_2\left(\frac{2^{\lceil{log_2(N)}\rceil}}{2^{k}}\right)}\rceil+1 = \lceil{\lceil{log_2(N)\rceil}-k\rceil}+1 = (db\_len-1)

The variable k is set to k=14 in the code: https://github.com/ucrparlay/Parallel-SSSP/blob/5dc07861c5d655a05e8b94084a9427270e9a7699/sssp.h#L17
The following code will now allow to read from qsize[doubling] which reads from uninitialized memory and can cause a segmentation fault:
https://github.com/ucrparlay/Parallel-SSSP/blob/5dc07861c5d655a05e8b94084a9427270e9a7699/sssp.cc#L77-L82
Since as shown above db_len can become larger than doubling the previous code snippet allows t_pt=db_len-1=doubling which in consequence allows the access to qsize[doubling] which is the uninitialized memory.
To confirm the segmentation fault I attached a graph in the adjacency format which will yield a segmentation fault when used as input to the program.  The provided input graph is unweighted.
segfault_graph.zip
./sssp -i segfault_graph.adj -p 2000 -v -a rho-stepping
The presented bugfix solves the stack overflow and resolves the segmentation fault.

Thanks for pointing it out!


Copy blanklines on matches(?)

Following #83 blanklines are not captured, e.g.
In [1]: import something
   ...:
   ...: print(something.__version__)

If the regex is set, will copy:
import something
print(something.__version__)

My intuition is that yes, we should copy them because the user is going to expect the output to look almost just like the copied code (minus prompts)

Also I can't figure this out, but in this repository I used:
copybutton_prompt_text = ">>> |\\\\$ |\\[\\d*\\]: |\\.\\.\\.: "

which appears to work fine.
but in aiida I've had to use:
copybutton_prompt_text = '>>> |\\\\$ |In \\\\[\\\\d\\\\]: |\\\\s+\\.\\.\\.: '

with the tonnes of backslashes, but this is what works

See: https://325-77234579-gh.circle-artifacts.com/0/html/intro/tutorial.html

god I hate regexes 😆

Dear Chris,
first things first: Thanks for your excellent work on this Sphinx addon, it really saves us.
We want to report back about what @chrisjsewell originally reported here. Apparently, we are getting tripped by this as well. For example, the code block at [1] will not get copied "as-is", all empty lines are missing. The config entries within conf.py we are currently using can be reviewed at [2].
Are we missing any respective configuration option here? Otherwise, do you see an easy way to make sphinx-copybutton pass through empty lines?
With kind regards,
Andreas.
[1] https://crate.io/docs/python/en/latest/sqlalchemy.html#tables
[2] https://github.com/crate/crate-docs-theme/blob/0.15.1/src/crate/theme/rtd/conf/init.py#L112-L115

I'm not quite sure where the fix needs to be for this unfortunately, and haven't had time to dig into the code to figure it out. If you or anybody else wants to take a look I am happy to review PRs! (maybe @sappelhof has run into this issue too?)

Thank you for the quick response. I tried to find the spot and handed in a corresponding patch with #127.

(maybe @sappelhoff has run into this issue too?)

I haven't (yet), but I am glad that @amotl prepared a patch (and I would vote for making copying empty lines the default)

Thank you Stefan, appreciate it. Both bringing in the feature as well as making copying empty lines the default behavior have now been folded into #127.


"Run File" should be before "Debug File"

Testing #129384
Run File should be before Debug File.
I am not sure if this is controlled by core, but Run File should be first because:

I believe that most users actually use the Run more in the title bar. We can dig out some numbers
We call the view "Run and Debug" since Run is "lighter", so it should also go first here
The hover title of the chevron is "Run or Debug"

Now I saw that this comes from Mock Debug. Though I still think we should fix mock debug so it is a good example for other debug extensions.

I've created https://github.com/microsoft/vscode-mock-debug/issues/58 for this.


This calculator is for demo project

Hi there, kindly make sure you remove all needed files (binary, executables like the .CAL-README.md.swp in your directory). Also ensure your directory is names in lowercases @Namujibril

I have changed as was directed, now how can I go with it again?
On Tue, Jul 11, 2023, 7:40 AM Destiny Saturday @.***>
wrote:

Assigned #22 https://github.com/ALX-SE-Algorithmia/Demo-Project/pull/22
to @Namujibril https://github.com/Namujibril.
—
Reply to this email directly, view it on GitHub
https://github.com/ALX-SE-Algorithmia/Demo-Project/pull/22#event-9783192837,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2VSQNU32TJRSTXEYKJFL23XPTYMRANCNFSM6AAAAAA2FJF2CU
.
You are receiving this because you were assigned.Message ID:
@.***
com>

Noticed your changes. I would like for you to do the following:

Remove the entire directory Namujibril from this pull request. The directory starting with an uppercase
Remove the .swp file and any other file which is not a source code file
Rename your readme file from CAL-README.md to README.md.

You should eventually have one directory called namujibril with your README.md and your source code files

You're almost there. Rename your readme file to README.md. Here's some code to help you out
In your directory namujibril, use the command mv Cal-README.md README.md to rename the README file

Yes, I'm done with renaming.
I hope now I'm in?
On Tue, Jul 11, 2023, 12:51 PM Martin @.***> wrote:

You're almost there. Rename your readme file to README.md. Here's some
code to help you out
In your directory namujibril, use the command mv Cal-README.md README.md
to rename the README file
—
Reply to this email directly, view it on GitHub
https://github.com/ALX-SE-Algorithmia/Demo-Project/pull/22#issuecomment-1630682565,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/A2VSQNXJDSENW7FZCB34O4TXPU435ANCNFSM6AAAAAA2FJF2CU
.
You are receiving this because you were mentioned.Message ID:
@.***>

Great work @Namujibril


Modifier keys not working on Windows

Bold, clear list, etc. (using ctrl) aren't working

Fixed and live! Thanks for the report.


feat(content-switcher): add fds-content-switcher component

:tada: This issue has been resolved in version 0.17.0 :tada:
The release is available on:

GitHub release
npm package (@latest dist-tag)

Your semantic-release bot :package::rocket:


Restructure crate::Collator structure to model changes in the config standard

Ideally this can be achieved by implementing a customer Visitor and Deserializer; however, if it's too much work, feel free to refactor the struct in crate::node::collator.
Below is an example of the expected config portion for collators
"collator": {
        "bin": "./bin/pendulum-collator",
        "nodes": [
        {
            "name": "glitch-princess-1",
            "chain": "./specs/rococo-local-parachain-raw.json",
            "dockerfile": "./tmp/Dockerfile",
            "args": [
                "--force-authoring",
                "--enable-offchain-indexing",
                "true"
            ],
            "port": 30344,
            "ws_port": 8844,
            "rpc_port": null,
            "relay": {
                "chain": "./specs/rococo-custom-2-raw.json",
                "args": [
                    "--force-authoring"
                ],
                "port": 30345,
                "ws_port": 9955,
                "rpc_port": null
            }
        }
    ]
 }

Modified to extend config struct, through validator and collator configs


SSH not working on Win10

I'm trying to follow the SSH setup on a Win10 PC.
When I run the command, I am prompted for a password, but I always get a denied error.
ssh -p 2222 -o StrictHostKeyChecking=no localhost
lost\user13@localhost's password:
Permission denied, please try again.

If I try ssh-add, the password is accepted.
I have it working in WSL though I think it might there might be an issue with Win10?
OS: Win10
DDEV:
Win10 DDEV 1.18.2, 1.19.0-alpha3
WSL: 1.19.0-alpha3

Ahhh .... Win10 was sending a Down-Level Logon Name

Win10

$ whoami
lost\user13

$ ssh -p 2222 -o StrictHostKeyChecking=no localhost
lost\user13@localhost's password:
Permission denied, please try again.

But WSL only sends the username

$ whoami
user13

$ ssh -p 2222 -o StrictHostKeyChecking=no localhost
Enter passphrase for key '/home/user13/.ssh/id_rsa':

So, I need to specify the username when I run from Win10:
$ ssh -p 2222 -o StrictHostKeyChecking=no user13@localhost
user13@example-web:~$

Also confirmed: updating my Win10 SSH ~/.ssh/config file, I can connect to the DDEV box via VSCode Remote: SSH extension
Host ExampleDDEV
    HostName localhost
    Port 2222
    StrictHostKeyChecking OFF
    User user13

PR was submitted to help others with similar issue.

Nice work!

Could you retitle this issue to make it more search friendly, say more about what "not working" meant in the title?


Auto-select the track and video submission form fields when possible

Had this idea yesterday, works really well!
We're simply passing a state object using the router when the user clicks on one of the "submit to the showcase" CTAs in a track or challenge. This allows the form to know where the user came from and select the correct track and video values for them.
https://github.com/CodingTrain/thecodingtrain.com/assets/4009209/01e698cf-22dc-4413-82bc-4cb22ff4c73b

Wow, this is incredible, thank you @fturmel! I'll definitely be showing this off when I'm back to streaming after finishing Nature of Code writing!

Glad you like it!


Is it possible to save animation of drawing?

I'd like to let user draw and then save it as animation of signing so instead of just downloading end result image, I'd like to replay entire signing
Is it possible?

@pie6k I did it by following this webpage: https://webrtc.github.io/samples/src/content/capture/canvas-record/


Features sl home layout

Created grid layout for home. Each event is represented as a tile, linked to its review page

I'm unable to merge due to branch conflicts with another PR that altered the HTML


Cannot use to create a custom store

This library appeared to be exactly what I was looking for (and trying to implement myself), but as I tried to use it I ran into issues.
My goal is to keep my store strongly typed and provide setters for each field that should be editable. Something like the following:
const Key = "settings"

type Settings = {
	recording: boolean;
	recordTabId: string;
	onSettingsPage: boolean;
	settingsUrl: string;
}

function settingsStore() {
	// Causes more issues
	// const { subscribe, update } = chromeStorageLocal<Settings>(Key)

	const settings = chromeStorageLocal<Settings>(Key)

	return {
		subscribe: settings.subscribe,
		startRecording: () => settings.update(state => ({...state, recording: true})),
		stopRecording: () => settings.update(state => ({...state, recording: false})),
		setRecordTabId: (id: string) => settings.update(state => ({...state, recordTabId: id})),
		clearRecordTabId: () => settings.update(state => ({...state, recordTabId: ""})),
	}
}

Doing this, I run into issues where this is not defined when subscribing. If I pull the subscribe and update methods off of chromeStorageLocal, I run into similar issues when calling update from my setters (i.e. update(...) instead of settings.update(...)).
I was going to try extending the ChromeStorageStoreAdapter class, but it is not exported.
Likely this approach just is not compatible with the library. And assuming it isn't and isn't meant to be, I'm wondering if you have any other guidance on a way to type the store with an object and allow callers to set individual keys without having to pass back in the entire object themselves.
I'm trying to avoid doing the following from the callers:
let settingsStore = chromeStorageLocal<Settings>(Key)
settingsStore.update(state => ({...state, recording: false}))

I'm tried a different approach and am very happy with the results. Instead of trying to manage one object I decided to use a separated key for each property. It looks like the following:
class SettingsStorageLocal {
	constructor() {
		this.recording = chromeStorageLocal("recording")
		this.recordTabId = chromeStorageLocal("recordTabId")
		this.onSettingsPage = chromeStorageLocal("onSettingsPage")
		this.settingsUrl = chromeStorageLocal("settingsUrl")
	}
	// We could also make the Readable and the define setters
	recording: Writable<boolean>
	recordTabId: Writable<string>
	onSettingsPage: Writable<boolean>
	settingsUrl: Writable<string>

	init(): void {
		this.recording.set(false)
		this.recordTabId.set("")
		this.onSettingsPage.set(false)
		this.settingsUrl.set(`chrome://extensions/?id=${chrome.runtime.id}#allow-incognito`)
	}
}

const store = new SettingsStorageLocal()
export default store
export const {recording, recordTabId, onSettingsPage, settingsUrl} = store

I'm happy to discuss the previous issues I was having but am going to go ahead and close this.


Parts/add skeleton for parts

Added skeleton for parts flow
Added routes for /ordering and /servicing as discussed
If there are no parts in the database, we get 10 dummy parts inserted into Parts collection using casual
Parts are being fed all the way to the props of OrderingMain component

There is also a merge conflict to deal with - do a git pull && git merge origin/develop to see them and resolve them. package-lock.json conflicts can be solved easily by deleting the file, doing a meteor npm install (which rebuilds it) and then git add package-lock.json


[ButtonBase] Derive state on render instead of in layout effects

Gets rid of all layout effects in ButtonBase. That way passive effects in a parent aren't flushed anymore just because the ButtonBase used a layout effect.
This was the main offender:
// Ripple is used for user feedback (e.g. click or press) so we want to apply styles with the highest priority
useEnhancedEffect(() => {
  if (!inProp) setLeaving(true)
}, [inProp]);

Preview: https://deploy-preview-26762--material-ui.netlify.app/components/buttons/

This looks safe to me but @oliviertassinari might have some more historical context?


Designer tech debt - defining sub-components in lists

Relates to: FORMS-319
Currently changes to the data are handled by a series of callbacks being passed down to child components which will update data (which sits at top level of the app). Because there are so many moving parts, it's getting difficult to track what is responsible for changing what and how. Changes to data will also cause the whole app to rerender.
use reducer to apply changes to data model
reducers wont trigger rerenders
this way we can centralise how components, lists, etc changes should be handled
consolidate same type editors - some editor components are duplicated. eg, the code for editing a radio component is duplicated by the code for editing a checkbox component
remove usages of updateComponent function. It is changing a "stateful" variable directly. <- this should not be done because it is bypassing react's render enqueuing

this has been addressed in #246


remove unneccessary comprehension, no use of self

This PR includes potential bug fixes and code refactor.

Remove unnecessary list comprehension
Add @staticmethods, instead of using self.
The method doesn't use its bound instance. Decorated the method with @staticmethod decorator, so that Python does not have to instantiate a bound method for every instance of this class thereby saving memory and computation.
Reference^

The code could be refactored more, it had a bunch of unused imported modules that could be removed for better performance, but I did not know If they were to be used later.

Not a problem, Thanks for your contribution! ❄


Failed to import from vllm._C with ModuleNotFoundError("No module named 'vllm._C'")

Reminder

[X] I have read the README and searched the existing issues.

System Info

llamafactory version: 0.9.0
Platform: Windows-10-10.0.22631-SP0
Python version: 3.11.10
PyTorch version: 2.4.0+cpu
Transformers version: 4.45.0
Datasets version: 2.21.0
Accelerate version: 0.33.0
PEFT version: 0.12.0
TRL version: 0.9.6
vLLM version: 0.6.0

Reproduction
(llamafactory) PS C:\dev\AI_Vtuber\LLaMA-Factory-0.9.0> llamafactory-cli version
WARNING 10-21 18:27:11 _custom_ops.py:18] Failed to import from vllm._C with ModuleNotFoundError("No module named 'vllm._C'")
| Welcome to LLaMA Factory, version 0.9.0                |
|                                                        |

Project page: https://github.com/hiyouga/LLaMA-Factory

也有嘗試過version 0.9.1.dev0且使用vllm 0.6.3.post1 依舊無效
Expected behavior
希望可以正常使用vllm推理
Others
No response

vllm was not correctly installed


Use C++11 and chrono durations instead?

Forgive me if I'm crazy here as I've been out of the C++ world for quite some time now but, from what I've read today, I believe this module's C++ component could be greatly simplified cross-platform by compiling it with C++11 and using "chrono" durations.
Compiling Node native modules via node-gyp with C++11: http://blogs.encamina.com/returnvalue/2014/05/29/node-js-addons-c11/
Using "chrono" durations for cross-platform sleep: http://stackoverflow.com/a/11276503/471696
#include <chrono>
#include <thread>

std::this_thread::sleep_for(std::chrono::nanoseconds(ssec));   /* signed integer of 64+ bits */
std::this_thread::sleep_for(std::chrono::microseconds(usec));  /* signed integer of 55+ bits */
std::this_thread::sleep_for(std::chrono::milliseconds(ms));    /* signed integer of 45+ bits */
std::this_thread::sleep_for(std::chrono::seconds(sec));        /* signed integer of 35+ bits */
std::this_thread::sleep_for(std::chrono::minutes(min));        /* signed integer of 29+ bits */
std::this_thread::sleep_for(std::chrono::hours(hrs));          /* signed integer of 23+ bits */

I'm not really into the C++ world either so I have no idea if everyone using nodejs has a C++11 compatible compiler on their system.

@JamesMGreene That does look like a slick refactoring.  @ErikDubbelboer 's concern is warrented, though, IMHO.  This page is the single source of truth for gcc's support for C++11.  It's clearly still incomplete and experimental, depending what version of gcc you're running, which in turn is dependent on Linux distro.  OS X and Windows have similar issues.

Thanks for the info, @apchamberlain.
@ErikDubbelboer: Feel free to close this or leave it open as a reminder for future enhancement. Your call.

I'll leave it open for the future.

amazing, that will be good for next generation.

Good evening!
C++ guy here. I would like to implement this feature. Future is probably already here (i.e. all major compilers support at least C++11 or even newer), but I wouldn't break a backward-compatibility. I will add few different implementations of underlying functions - for older compilers (separate files for different OS) and one newer that uses new functions from std::chrono namespace, all implementations will go to different files. The only thing I'm not sure about is if/how compiler capability can be retrieved in gyp (i.e. need to retrieve if compiler supports at least C++11).


Awkward wording issue in sequential scales documentation

As I was reading through the documentation for sequential scales, I came across the following sentence, which I find a bit confusing:

However, unlike continuous scales, the input domain and output range of a sequential scale always has exactly two elements, and the output range is typically specified as an interpolator rather than an array of values.

This sentence contains the statement

the ... output range of a sequential scale always has exactly two elements

Is this really true?
This statement seems to contradict the statement

the output range is typically specified as an interpolator rather than an array of values

It may be appropriate to delete the text "and output range" from that statement.
What do you think? Thanks!

As noted by @clhenrick, sequential.range and sequential.rangeRound were introduced in d3-scale 3.1.0, at https://github.com/d3/d3-scale/commit/60858c6adcdd7f88acda1c58c81b2a9cf11bc84f
I'm a bit lost too on how to use this; testing seems to show that sequential.range is only a getter, in contradiction with the documentation (??).

The generated code is somehow "function(){return[a(0),a(1)]}" so I expect there’s a Terser bug at play here.

No, it looks like sequential.range hasn’t been released yet.

Er, sequential.range was added as a read-only method in 3.1, and changed to allow setting the interpolator in 3.2. So everything is as it should be, except master was ahead of the latest release, because I am only human.


Refine to composite bridges for 1:N protocols coverting

In SRS 5.0, the bridge is attached to source, used to covert from one protocol to another one. For example, when configured RTMP to RTC:
vhost __defaultVhost__ {
    rtc { enabled on;  rtmp_to_rtc on; }
}

When publish stream to SRS by RTMP client, a bridge will be created and attached to source:
srs_error_t SrsRtmpConn::acquire_publish(SrsLiveSource* source) {
    SrsRtmpToRtcBridge* bridge = new SrsRtmpToRtcBridge(rtc);
    source->set_bridge(bridge);
}

Then all RTMP packets will be fed to bridge and allowd to be devliered by WebRTC. Similarly, a SrsSrtToRtmpBridge is used to convert SRT to RTMP stream, and a SrsRtcToRtmpBridge is for coverting RTC to RTMP.
It works well for SRT => RTMP, RTMP => RTC, RTC => RTMP. But, what happends if need SRT => RTC? Well, it actually combiles two bridges, first SRT => RTMP then RTMP => RTC, this is complex and an extra layer is introduced, this PR is to refine the bridge to allow directly and simultaneously coverting, such as:
SRT => RTMP
SRT => RTC

In this PR, we need to:

[x] Rename bridge from YyyFromXxxBridge to XxxToYyyBridge, which is better name, for example, rename SrsRtcFromRtmpBridge to SrsRtmpToRtcBridge
[x] Use frame SrsSharedPtrMessage as message between all bridges.
[x] Assemble packet, such as RTP for RTC, or TS for SRT, to frame before passed to bridge.
[x] Support composite bridges to covert SRT to RTMP and RTC.

目前的结构, 可能还有一个问题
如果所有的bridge都是关闭的, 那么任何协议都可以推同名的流.
比如rtmp推 live/livestream, srt推live/livestream, rtc推live/livestream,
但这3路流都是不一样的内容.
后面可能可以做一个独立于协议的source, 用来管理流?

目前的结构, 可能还有一个问题 如果所有的bridge都是关闭的, 那么任何协议都可以推同名的流. 比如rtmp推 live/livestream, srt推live/livestream, rtc推live/livestream, 但这3路流都是不一样的内容.
后面可能可以做一个独立于协议的source, 用来管理流?

确实有这个问题，这个得考虑下怎么解决了。


Missing fields from PSBT

I'm trying to calculate the fee for a PSBT. This requires knowing the sat values of the inputs and outputs.
For witness inputs, this is easy, Input::witness_utxo is a TxOut, which contains the value. For non-witness inputs, Input::non_witness_utxo is a Transaction, so the index of the output being spent is required to figure out what the value is.
BIP-174 specifies that each input should have PSBT_IN_OUTPUT_INDEX, which is what's needed, but that seems to be missing from PartiallySignedTransaction.
The BIP also specifies that each output should have PSBT_OUT_AMOUNT, which is required to calculate the total output value. However, this also appears to be missing.
My first instinct is that I'm misunderstanding something, so I wanted to check. Am I understanding these fields correctly? Are they not present because they duplicate information in the unsigned_tx field?

You can just call iter_funding_utxos() on PSBT to get the iterator of funding outputs. FYI each index is in the input - within OutPoint.

Nice, thank you, I missed that!
If fields are omitted because they duplicate other parts of the PBST (i.e. the unsigned_tx) it might be nice to document that.

@casey , those fields are not a part of BIP 174, but BIP370 (Psbt v2). Pbstv2 is currently a WIP in rust-bitcoin.

The progress can be tracked https://github.com/rust-bitcoin/rust-bitcoin/issues/1115

@sanket1729 I'm a little confused, aren't PSBT_IN_OUTPUT_INDEX and PSBT_OUT_AMOUNT both mentioned in BIP-174? Or is BIP-174 updated with new fields?

@casey BIP174 is updated with new fields. The fields have rows "versions requiring inclusion"

@casey BIP174 is updated with new fields. The fields have rows "versions requiring inclusion"

Ahh, okay, I missed that, thank you!


process legacy rlocus keyword k->gains

As requested  by @bnavigator in review of #999.

coverage: 94.504% (+0.001%) from 94.503%
when pulling 206ca859de895a4fe372befd00525d40c49178d2 on gdmcbain:rlocus-k
into 2d1f514d5b56b07877a879517e828d965777c5f8 on python-control:main.

The coverage does not change much because the line is run unconditionally, but could you please add a unit test which passes the keywords in question and checks for their proper handling?

Thank you! The doctest failure is unrelated.


Discovery etag

wip PR for adding etag to builtin and crd resource discovery endpoints
few TODOs in code intended to serve as points of discussion

this PR completes two tasks:
adding ETag to /apis//version for CRDs
adding ETag to /apis//version for builtins

this PR completes two tasks:
adding ETag to /apis//version for CRDs
adding ETag to /apis//version for builtins

@alexzielenski Can you change this PR to be against the feature branch (feature-discovery-cache-busting) instead of master?


"Create minidumps for a Visual Studio process with all call stacks steps" need polish

Step 3 says "Check the relevant Managed and Native check boxes and press Attach." It says afterwards in Step 4 to pick the process from the list. These steps are in the wrong order or possibly missing a step or two. "Check the relevant Managed and Native check boxes" is also vague as the provided screenshot no longer matches what Visual Studio (2019 and later) displays.
Rough suggestion besides updating the screenshot:

Next to "Attach To:", press the Select button
Check the Managed (.NET 4.x) and Native check boxes and press OK (the button in this dialog is not "Attach")
Select the other Visual Studio instance to attach to from the list of running processes, then press Attach (now that the process has been selected, "Attach" may be clicked)

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

ID: ef410b52-1f8f-6f71-1b86-92d73fd258f6
Version Independent ID: 7b83a96b-1d79-48f7-c702-da151b91ddff
Content: Collect an ETL trace with PerfView and create minidumps with all call stacks - Visual Studio (Windows)
Content Source: docs/ide/report-a-problem-perfview-minidumps.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @corob-msft
Microsoft Alias: corob

@schwjm Thanks for the feedback! Fix should be in place by end of day.


Update monitor-cache-performance.md

Fix for #1876

@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!


IOS build errors

The build fails for me on IOS with the build errors like these below. I have tried, cleaning but it doesn't help. Any suggestions?
Error output from Xcode build:
/.pub-cache/hosted/pub.dartlang.org/native_admob_flutter-1.2.0/ios/Classes/Interstitial/InterstitialAdController.swift:5:25: error: cannot find type 'GADInterstitialAd' in scope
pub.dartlang.org/native_admob_flutter-1.2.0/ios/Classes/Interstitial/InterstitialAdController.swift:29:13: error: cannot find 'GADInterstitialAd' in scope
.pub-cache/hosted/pub.dartlang.org/native_admob_flutter-1.2.0/ios/Classes/Interstitial/InterstitialAdController.swift:29:83: error: cannot find type 'GADInterstitialAd' in scope
.pub-cache/hosted/pub.dartlang.org/native_admob_flutter-1.2.0/ios/Classes/Native/NativeAdController.swift:4:37: error: cannot find type 'GADNativeAdLoaderDelegate' in scope
.pub-cache/hosted/pub.dartlang.org/native_admob_flutter-1.2.0/ios/Classes/Native/NativeAdController.swift:45:27: error: value of type 'GADNativeAd' has no member 'muteThisAd'
.pub-cache/hosted/pub.dartlang.org/native_admob_flutter-1.2.0/ios/Classes/Native/NativeAdView.swift:222:41: error: value of type 'GADNativeAd' has no member 'starRating'

Are you using other admob dependencies?

No I also tried on another flutter project that I use for testing which has very few packages, same issue happens on that test project as well. Android builds ok . I am on Flutter 2.0.4

Maybe related to https://github.com/bdlukaa/native_admob_flutter/issues/31#issuecomment-817197284

Thank you. The problem was caused by me forgetting to run pod install in ios directory after adding this plugin to my pubspec.yaml.

It worked only after a pod update, the issue was caused by something on my side obviously. Thanks!

Then this can be closed.


[BUG] System.AggregateException: One or more errors occurred. (The given key 'content-platform-dictionaries.zip' was not present in the dictionary.)  [Inner Exception] System.Collections.Generic.KeyNotFoundException: The given key 'content-platform-dictionaries.zip' was not present in the dictionary.

Acknowledgement of preliminary instructions

[X] I have read the preliminary instructions, and I am certain that my problem has not already been addressed.
[X] I have thoroughly looked through the available Wiki articles and could not find a solution to my problem.
[X] I am using the latest version of Bloxstrap.
[X] I did not answer truthfully to all the above checkboxes.

Bloxstrap Version
v2.5.4
What problem did you encounter?
System.AggregateException: One or more errors occurred. (The given key 'content-platform-dictionaries.zip' was not present in the dictionary.)
[Inner Exception]
System.Collections.Generic.KeyNotFoundException: The given key 'content-platform-dictionaries.zip' was not present in the dictionary.
on download
Bloxstrap Log
N/A

update bloxstrap to 2.8.1
https://github.com/bloxstraplabs/bloxstrap/releases/tag/v2.8.1


[Bug] Disabling "Show Tabs" option in tab widget makes the widget look weird

Description
Disabling the tab headers in the tab widget causes the tab widget to look very weird with borders on the top & bottom. The deployed version looks even worse.
Steps to reproduce the behaviour:

Important Details

Version: [Cloud / Self-Hosted vx.x]
OS: [e.g.MacOSX]
Browser [e.g. chrome, safari]
Environment [production, release, deploy preview]

Closing in favour of #4747, Shall prioritise that higher


Change Scope when edit mode

Hi
I am using X editable for my website. I am googling for many hours, still not get any solution for my scenario. Please help me out.
My Questions:

I have to choose drop down A, based on the selected value, scope values will update by retrieving the data from API and load it to another drop down B.
And i have another drop down C, based on this selection. I need to show different input types.
For eg: If i choose the value "url", i have to show text box
If i choose the value "code", i have to show text area
If i choose the value "type", i have to show select box.

For Q1, I tried with the watch, filter, everything seems doesn’t work :(
For Q2, Is it possible to do this, If so please give me an example.
Please help me out. I am using Angular 1.5.8, smart table 2.1.8 and Xeditable 0.5.0

You posted this https://jsfiddle.net/32kzsvve/1/ previously which you said helped you.  What is not working for you?
If you want more specific help, you will need to post a jsfiddle/plunkr so people can see your code and help you with your issues.

Yes it worked only for the static data not for the dynamic API call. That is my problem here. Please find the js fiddle here https://jsfiddle.net/Pavi95/k2f6osdu/1/

When I go to that fiddle, I see the following JS errors:
Error: [$injector:nomod] Module 'BlurAdmin.pages.feeds' is not available! You either misspelled the module name or forgot to load it. If registering a module ensure that you specify the dependencies as the second argument.
http://errors.angularjs.org/1.5.8/$injector/nomod?p0=BlurAdmin.pages.feeds  angular.js:68:12
"Error: [$injector:unpr] Unknown provider: editableOptionsProvider <- editableOptions <- FeedsPageCtrl

You are also using an older version of xeditable.

Please check this fiddle https://jsfiddle.net/Pavi95/k2f6osdu/5 I updated sample data format below HTML section

@pavi95 The jsfiddle link is broken.

@ckosloski I fixed this by using angular way of jquery :)


Sidebar: add "properties" in context menu of drives

What about displaying it for every item in the Sidebar, just like in Windows explorer?

@yaichenbaum Can you assign me?

@Thomas1664 Done 👍.

What about adding (almost) every context menu item to each sidebar item (except recycle bin and home)?

What about adding (almost) every context menu item to each sidebar item (except recycle bin and home)?

That should work.

@yaichenbaum Adding properties to drives works. However, it does not work for pinned folders because no properties are shown.


Addition to build installation document.

This addition adds clarity for customers/testers who may have notifications disabled and don't know where to see their releases to install and download.

@psmcbrayer : Thanks for your contribution! The author(s) have been notified to review your proposed change. @JoshuaWeber

#sign-off

Hey @psmcbrayer, thanks for this PR. It looks good, the only issue that I see is that the link still has the locale code. Could you remove that before I merge the PR?

@psmcbrayer : Thanks for your contribution! The author(s) have been notified to review your proposed change. @JoshuaWeber


Czech translation update

small Czech translation updates

Coverage decreased (-0.03%) to 79.79% when pulling 37bb9b2120f8417696ca7bf30023834dfc4a05c3 on feature-cz-translation-update into 1d215721266d6a72eed5f1d041a5bf96178f5029 on staging.


Add peer dependency checks for Go Modules

Is there an existing issue for this?

[X] I have searched the existing issues

Feature description
Currently, Dependabot does not check whether an upgraded dependency would remove a required peer dependency.
Adding a Golang "peer dependency check" would avoid opening a Dependabot PR if the upgrade removes a dependency that's required by another package - much like how the npm peer dependency check works.
Instead of opening a PR, Dependabot should provide an error along the lines of "cannot upgrade dependency X to version a.b.c because peer dependency Y requires sub-dependency Z which is no longer available"
Questions for the Dependabot Team:

Can we implement this with the Golang CLI or does the check need to be run in Dependabot?
Can we detect when a peer dependency would require a sub-dependency with the Golang CLI?

@jakecoffman was able to close this out with https://github.com/dependabot/dependabot-core/pull/6730
When a Dependabot Update removes a required package, Go will fail if the update and give a “no required module provides package” error which gives a trace of what happened:

no required module provides package golang.org/x/tools/missing/package; to add it:
go get golang.org/x/tools/missing/package

The trace should be visible in the job log.


pythonPackages.ana: init at 0.06

Motivation for this change
I am trying to make angr, the binary analysis framework, available on NixOS.
This is part of the modules it requires.
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Fits CONTRIBUTING.md.

After verification, this is not used in angr anymore.
Old and unmaintained package.


Fix teams page

r? @huonw

 Warning

Pull requests are usually filed against the master branch for this repo, but this one is against gh-pages. Please double check that you specified the right target!

r=huonw


Possible loading issue under Minitest 5.19.0

Minitest 5.19.0 was released today (July 26, 2023), and it has a behavior change that seems to affect minitest-rg loading. Specifically, when we update to minitest 5.19.0, the minitest-rg plugin (when we require "minitest/rg" in our test helper) now raises:
[2023-07-26 20:55:44  INFO]  exec: ["bundle", "exec", "rake", "test"]
/opt/hostedtoolcache/Ruby/3.2.2/x64/lib/ruby/gems/3.2.0/gems/minitest-rg-5.2.0/lib/minitest/rg.rb:4:in `<top (required)>': uninitialized constant Minitest::RG (NameError)

Minitest::RG.rg!
        ^^^^
	from <internal:/opt/hostedtoolcache/Ruby/3.2.2/x64/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
	from <internal:/opt/hostedtoolcache/Ruby/3.2.2/x64/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
	from /home/runner/work/google-cloud-ruby/google-cloud-ruby/google-analytics-admin/test/helper.rb:23:in `<top (required)>'
	from <internal:/opt/hostedtoolcache/Ruby/3.2.2/x64/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
	from <internal:/opt/hostedtoolcache/Ruby/3.2.2/x64/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
	from /home/runner/work/google-cloud-ruby/google-cloud-ruby/google-analytics-admin/test/google/analytics/admin/client_test.rb:19:in `<top (required)>'
	from <internal:/opt/hostedtoolcache/Ruby/3.2.2/x64/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
	from <internal:/opt/hostedtoolcache/Ruby/3.2.2/x64/lib/ruby/3.2.0/rubygems/core_ext/kernel_require.rb>:37:in `require'
	from /opt/hostedtoolcache/Ruby/3.2.2/x64/lib/ruby/gems/3.2.0/gems/rake-13.0.6/lib/rake/rake_test_loader.rb:21:in `block in <main>'
	from /opt/hostedtoolcache/Ruby/3.2.2/x64/lib/ruby/gems/3.2.0/gems/rake-13.0.6/lib/rake/rake_test_loader.rb:6:in `select'
	from /opt/hostedtoolcache/Ruby/3.2.2/x64/lib/ruby/gems/3.2.0/gems/rake-13.0.6/lib/rake/rake_test_loader.rb:6:in `<main>'
rake aborted!

Our test helper basically does this:
require "minitest/autorun"
require "minitest/focus"
require "minitest/rg"

I determined that setting the MT_COMPAT=true environment variable works around this, suggesting this is the change that affected things: https://github.com/minitest/minitest/commit/a2c6c18570f6f0a1bf6af70fe3b6d9599a13fdd6. I'm assuming this means that, unless in compat mode, minitest no longer actually loads plugins until exit time when autorun kicks in, and as a result the code in minitest/rg can no longer assume its plugin has already loaded. I guess we could work around this by requiring minitest/unit in our helper, but that seems like it isn't the preferred usage, so perhaps RG needs to do something differently?

FYI upgrade maxitest to support 5.19 (which vendors minitest/rg) and it just worked 🤷
possibly because it uses
require "minitest"
require "minitest/autorun"

but did not dig deeper

I can confirm this issue. I just switched to maxitest however and it did not work with maxitest as well.
MT_COMPAT=true  solved the issue for me!

MT_COMPAT=true or add require "minitest/unit" somewhere

@zenspider I haven't dug into what is causing this yet, but would changing the require on rg_plugin.rb:1 from require "minitest/test" to require "minitest" resolve this?

See #18

Also seeing this same error (trying to add minitest-rg for the first time), but have been unable to resolve with any of the above suggestions.

Also seeing this same error (trying to add minitest-rg for the first time), but have been unable to resolve with any of the above suggestions.

Use maxitest, it has a fix for the issue already -> https://github.com/grosser/maxitest

This fix is released in minitest-rg 5.3.0. Thank you all for your patience.


Use google benchmark as third partie dependencie for bench?

Hi
I heavily use google benchmark (https://github.com/google/benchmark)  on various projects.
Linked to isssue #107, it may a good thing to have a benchmark to measure the provided speedup on various scenario.
What do you think?
(i could make a prof of  concept on my side related to issue #107)
Regards
TR

Agreed, this makes sense.


Anko package can't sync with jcenter

Bintray support fixed the problem with the syncing so the issue is fixed.


fix: Use cached results in whereType()

Fixes #20

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

4 of 4 (100.0%)  changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.1%) to 96.97%

Totals

Change from base Build<PHONE_NUMBER>:
0.1%

Covered Lines:
96

Relevant Lines:
99

💛  - Coveralls


US-13374-1:Added notification banner for static page

New story created for this change which need to go live today.
Added notification banner for static page

Follows the pattern I was looking at this morning. Assume that given the quick turnaround, we don't need to include the Welsh translation at this stage?
Yes, we dont have translation available
Can I also assume the expectation is that this notification batter will always display?
Yes


CommonJS been dropped?

I use this package in several projects. I was surprised because I didn't expect the patch version to adopt ESM.
If possible, create a dual package that also supports CommonJS?

This was my mistake, I released 1.3.2 to re-switch to CJS.


[css-logical] rename offset-* properties to inset-block-start, inset-block-end, inset-inline-start and inset-inline-end

https://bugzilla.mozilla.org/show_bug.cgi?id=1464782

PR for the data: https://github.com/mdn/data/pull/273

I have moved and updated the property pages in English and French. Still to do is the compat data.

PR for the compat data https://github.com/mdn/browser-compat-data/pull/2682

I think this is pretty much done, other than when the fresh compat data gets onto MDN I need to update the refs to it. So I'll leave this one open until that has happened to remind me to go back and do it.

I've now updated the refs to the support data so I think this one is done.

review requested and release notes updated.

All looks good. Closing issue. Note that Emilio has responded with a small change request in the bug.

PR for that change to the data: https://github.com/mdn/data/pull/280