[Fix] Typescript Support

I have modified the .babelrc config to target the current version of node. i have been able to resolve the issue referenced here: https://github.com/zalmoxisus/mobx-remotedev/issues/17. I believe the issue was that the current babel config of the package converted the store class into a function, this would not be an issue if the consuming application was also using babel to transpile their code but a typescript implementation without babel would throw a typeerror since the client passed in a class rathen than a function transpiled from a class to be called like a function. tough to explain but i think i fixed it.
old:
 _this = _possibleConstructorReturn(this, (_getPrototypeOf2 = _getPrototypeOf(store)).call.apply(_getPrototypeOf2, [this].concat(args)));

new:
 store = class extends store {
      constructor(...args) {
        super(...args);
        spy(this, config);
      }
    };

i have tested this branch with the 3 examples and my local portfolio with and without babel plugins(typescript).
i also removed the babel preset "@babel/preset-react" since there is no JSX syntax in the src files. Please let me know if you need to target a specific ECMA version, or if you need anymore details.

Thanks for the work here and for adding to awesome-mobx! I hope I'll get 3.0 version of teh extension soon, which is supposed to work better, with new features and new ui.
About the PR, I'm confused about targeting current node version for babel. I'm usually using Node 10, but that could break some use cases, like classes not being supported by some browsers. I could use a earlier version, but we could then just specify that explecitely, and I guess that wouldn't fix our issue then.
I'd publish it as 0.4.0-alpha since it's a breaking change, then if everything is ok we could get a new release. But maybe we could still keep common-js version in lib and es version in src and make typescript use it without being transpelled through babel?

I targeted the current version of node to prevent the class in dev.js from being transpiled into a function which would throw an type exception if the client did not transpile their MobX classes into functions by targeting a different es version (or the client wanted to use esnext in tsconfig.json/not use babel transpilers).
I figured if the client needed to target a specific node version they can do so with their babel configuration, the client's build process would transpile our lib files to target older browsers and convert both the store class in dev.js and the Mobx Store classes provided by the client into functions.
I believe the issue is that the store class in dev.js is being transpiled into a function, not allowing clients who target newer es versions to utilize our package (example: not converting their classes to functions).
i'm sorry if this sounds confusing. Let me know what browser/node versions you intend to target and i can update this PR and find a middle ground that allows TypeScript to work without babel.
full disclosure, my tsconfig module/targets are all esnext. i use code-splitting in my app to keep the initial bundle load down.

I understand that it babel targets my (or other maintainer's) nodejs version when I publish the package (running build:lib script). So if a maintainer has nodejs bellow 4.0, classes would be transpiled, if above they won't. We could probably restrict node version in package.json, but specifying it explicitely in babel config would be better.
I would still prefer to keep a commonjs version with classes transpiled for those who need it. So we might need two directories, one with cjs and another es6 (if it differs from src)?

yeah, we might have to use 2 directories. for babelrc it might be better to target commonjs than set a node version.
ill see what i can do with Babel/Webpack to output cjs and es6 directories.... i actually never attempted this before, so any guidance would be helpful?

I added another commit to attempt to allow es6/commonjs target support, i just added the line "module": "src/index.js" in package.json and reverted my change to .babelrc to allow commonjs transpilation.
https://github.com/zalmoxisus/mobx-remotedev/pull/40/commits/ffa7bcaaff991f7a39d601713f1a273a336bd023
the src files are es6 enough, let me know if you need anymore details, i used this article for reference:
http://2ality.com/2017/07/npm-packages-via-babel.html
also...
https://github.com/mobxjs/awesome-mobx/pull/71

That looks great! Thanks again! Hope TS will pick what is indicated in module. That is great to have anyway.

just tried it out with my portfolio. i got remotedev to work in typescript without any Babel or Type Errors
would that consider this issue reoslved?: https://github.com/zalmoxisus/mobx-remotedev/issues/17


chore: release 0.2.0

:robot: I have created a release *beep* *boop*
0.2.0 (2021-11-27)
Features

is_anagram (a128298)
all_equal (94248e3)
all_unique (1d0a9af)
average (28d132b)
capitalize every word (dc64add)
converts a string to camelcase (6c5bbe1)
find_multiples (ec846a9)
hex_to_rgb (be4bd7c)
rgb_to_hex (e7fa861)

Bug Fixes

all_equal regression (f914abf)
all_unique regression (b4bd938)
clippy (6654754)
remove main.rs, this project is more suitable for library (f7c2452)

This PR was generated with Release Please. See documentation.

Codecov Report

Merging #4 (959463f) into master (b200f6e) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master        #4   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            4         4
  Lines           33        33
=========================================
  Hits            33        33

Impacted Files
Coverage Δ

src/arrays.rs
100.00% <ø> (ø)

src/lib.rs
100.00% <ø> (ø)

src/maths.rs
100.00% <ø> (ø)

src/strings.rs
100.00% <ø> (ø)

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 48d39c2...959463f. Read the comment docs.


Added insert rows functionality with new style inherit options

Summary
Added new methods, for inserting row/s. Also added styleOption to allow style inherit.
Test plan
Added unit tests accordingly.

@alubbe @Siemienik This pull request is created as I was having weird issue, merging with earlier code. Please review this request.
@Siemienik The property setter is exposed and does not use Object.freeze yet. This can be added in another PR.
For this PR, only internal copying is happening, like spliceRows. Though, I have added the modification you have asked. Please review. Thanks.


request to add horizontal scrollbar to code block

I think it will improve the readability if adding a horizontal scrollbar to the code block, or the code will be wrapped.
For example, the code in "Standard Code Block" in the post "2013-08-16-code-highlighting-post.md" has been wrapped. It's hard to read, especially on a mobile device.

@yanjiahuang I've accepted your opinion. Thanks.


j2objc translation tool not being able to import com.sun.* dependencies

Some files that I have which import files in the com.sun.nio.sctp package are unable to being translate as the import statements themselves are failing for some reason. Even after I included in the classpath a .jar defining them (netty-transport-sctp-4.0.26.Final.jar).
So I don't know how to workaround this issue.
Using the javac tool to get some insight on what is going on was of no help since it just works without the need to include external dependencies (javac 1.8.0_121).
Below is an example of such failure occurring when I try translating a file named SCTPMessageChannel.java. The imports of com.sun.nio.sctp.SctpChannel.java and com.sun.nio.sctp.MessageInfo.java fail right at the beginning, and several X cannot be resolved to a type messages are logged every time these dependencies are mentioned in the file.
error: ./src/android/gov/nist/javax/sip/stack/sctp/SCTPMessageChannel.java:14: The import com.sun cannot be resolved
error: ./src/android/gov/nist/javax/sip/stack/sctp/SCTPMessageChannel.java:15: The import com.sun cannot be resolved
error: ./src/android/gov/nist/javax/sip/stack/sctp/SCTPMessageChannel.java:51: SctpChannel cannot be resolved to a type
error: ./src/android/gov/nist/javax/sip/stack/sctp/SCTPMessageChannel.java:54: MessageInfo cannot be resolved to a type
error: ./src/android/gov/nist/javax/sip/stack/sctp/SCTPMessageChannel.java:69: MessageInfo cannot be resolved to a type
error: ./src/android/gov/nist/javax/sip/stack/sctp/SCTPMessageChannel.java:69: MessageInfo cannot be resolved
error: ./src/android/gov/nist/javax/sip/stack/sctp/SCTPMessageChannel.java:70: MessageInfo cannot be resolved to a type

The problem is that you are trying to translate classes that are on the -bootclasspath, classes that are included in the Java JRE (javac would report similar errors if invoked with "-bootclasspath j2objc/lib/jre_emul.jar"). j2objc only supports the JRE classes defined in its JRE Emulation library.
To use the netty-transport-sctp version, get the source files for that package and add the top-level directory for them to the -sourcepath. They'll need to be translated anyway for your app to run on iOS.

Got it. Many thanks for the help!


Stepper Input (Labs)

### Checklist
- [x] Density (HD, MD, LD, TD)
- [x] Mode (Light, Dark)
- [x] Fill (Primary, Secondary, Tertiary)
- [x] State (Default, Hover, Selected/Active, Disabled, Read-only, Other)
- [x] Controlled API
- [x] UITK accessibility parity (screen reader + keyboard interactions)

### Deliverables
- [ ] Characteristics spec
- [ ] Interactions spec, where necessary
- [ ] In Code & Figma (Component + Guide)
- [ ] Supporting site documentation

### Features
- [x] Basic input with arrow control
- [x] Increment value control
- [x] Min, max value support + updating arrow state

Extending dates, 1 sprint behind

Zazu Goal: M(5)

API issues, passing event
merge to RC, a known state
stakeholder review required
draft docs/design ready to publish

Outstanding Issues:

Clicking and holding up/down, API issues to resolve
Release with optional prop to avoid future breaking change

PR waiting for review: https://github.com/jpmorganchase/salt-ds/pull/3527
Docs have been QA reviewed by Ben and updated with feedback, just need to be updated with the new API when above is merged https://github.com/jpmorganchase/salt-ds/pull/3586

Goal: push to core by EOS

Jul 11 - With theme next release and ag gird theme bugs, this may get delayed to next sprint.

Jul 15 - Push to next sprint due to other priorities

Goal: need event handler work and then move to Core
Requires some resourcing

will be picked up in cortado

Cortado Goal
Third party feedback before merge to core

Aug 7 - had a look at existing code. Need refactor to not use Input.

Aug 20 - EOS goal to labs, will create another ticket to move to core (waiting for feedback)

Aligning the schedule with #3878

Code complete / Design review required in Espresso

Frappe Goal: Merged to Labs
(delayed by 1 sprint due to Button release)

Sep 23 - #3976 ready for review. No site examples design available

goal for the week,  site examples defined - defined by Wednesday
(needs dev capacity to create examples by end of week)

24/09 - Site docs ready for review in figma

Ready for dev state reached


Query metafields in product query

Hey guys,
it would be awesome, if you add the possibility to also query for metafields in the product query.
Thank you very much for your answer in advance.
Best regards
Chris

@chrispreisler Sorry for the delay in this - but have added this.
To make metafields available in the Storefront API, you will need to follow this guide: https://shopify.dev/tutorials/retrieve-metafields-with-storefront-api

No problem, thank you very much. That sounds awesome. Where would I have to make that "Whitelistening" Mutation? Int the Gridsome Server? Or do you I have to use an extra tool to do that? Or can I do that from the Shopify Backend?
It's crazy, that it is so complicated to get the metafields via the api ...
Thanks for your help i advance again.
Best regards
Chris

@thetre97 Would it be possible to point me in the right direction? I tried for myself, but unfortunatley didn't get my head around it. Thank you so much.


How to set the default directory to the "~",not the "/mnt/d/wsl-terminal" instead ?

i just want to enter the "~" when i run the wsl-terminal , who can show me how ? A lot of thanks if anyone could offer help.

Create a .lnk file to open-wsl.exe, and motify target to D:\wsl-terminal\open-wsl.exe -C ~ .

Thank you so much ,it rellay works. @goreliu

@goreliu this doesn't work for me, it always starts at the directory that's set in the "Start in" option of the .lnk file.

According to the readme, the -l argument to open-wsl.exe is what you are after:
"-l: ignores current path and starts terminal in your home directory (doesn't work with tmux)."
I have tried this and it works for me!

Sadly I'm tmux addict...

@paulo6 the -l did work for me thanks, even with the tmux setup I use (which is not related to wsl-terminal options).


Learning about the z/OS Install Strategy

Added two modules for understanding the z/OS platform install strategy:

Packaging - understanding how the major software vendors will be packaging software into a Portable Software Instance format.
Installation - understanding how a customer would install (deploy) that software onto a system.

z/OS install education modules.


Transparent background on figcaption

Hi guys any idea how to make an rgba background works? I have tried it but it seems not working... I just want to use a semi transparent figcaption background.

Hello
Using semi transparent backgrounds will vary depending on the effect. Which effect are using?
Ciarán

Which effects work for transparent?

The background element to the caption on hover varies from effect to effect. In most cases the following will work which covers most elements. Note in some cases the backgound image has been transformed to be no longer visible or front facing.
[class^="imghvr-"] figcaption,
[class*=" imghvr-"] figcaption,
[class^="imghvr-"]:before,
[class*=" imghvr-"]:before,
[class^="imghvr-"]:after,
[class*=" imghvr-"]:after {
	background-color: rgba(19, 87, 150, .5) !important;
}

Where 19, 87, 150 is the RGB color and .5 is the transparency.
Ciaran

will test it in my project when i get home, thank you :D


Important components

Hi, I am currently looking at implementing a diffusion model for policy learning and was very impressed by your work! I was wondering what components of your approach you found to be particularly important for good results? 3 things I specifically was curious about were:

I see you use EMA, did you find that the model predictions were particularly unimodal/overfit to recent training data without it?
Was the causal attention masking used in the transformer variant crucial in getting this architecture to work, or do you think simply decoding waypoints from a more BERT-style encoder architecture would work?
In the appendix it seems you used a particularly large model for the CNN variant and say that you always found larger CNN -> better performance. Was the performance of much smaller CNNs (e.g. ~10M) much worse?

Hi @hbishop1:

I empirically found EMA to accelerate training (eval performance increases faster) and improve performance (by <5%), but the policy should "work" even without it.
I found the causal attention masking to be critical to get the transformer variant of diffusion policy to work. My suspicion is that when used without it, the model "cheats" by looking ahead into future end-effector poses, which is almost identical to the action of the current timestep.
I think the model capacity needed depends on task complexity (more complex task requires larger CNN). Reducing the number of training diffusion steps also reduces CNN capacity requirement at the expense of reduced action quality. ~10M CNN should still work with less than 10% performance penalty on benchmarks we have tested.

Great, thanks for the quick and detailed response, that will really help!


Let iOS have a minimum scroll movement threshold to break before motion starts

Fixes #12181
Mimics native behaviour. Android unaffected.


Update README.md to indicate the use of Python 2.7

Since the support for Python2 is now being rescinded, it would be great if the README.md could indicate that code had been written in Python2 so any future developers could set up the appropriate development environment.

Codebase upgraded to Python 3.7+ in NSpM v1.0.0.


NIFI-11709 Upgrade Guava from 32.0.0 to 32.0.1

Summary
NIFI-11709 Upgrades Guava library references from 32.0.0 to 32.0.1. Additional changes include overriding Guava 27 in several transitive dependencies. Guava 23.1 introduced a new policy that avoids removing API methods, with the exception of those annotated with @Beta to provide better compatibility between versions.
Tracking
Please complete the following tracking steps prior to pull request creation.
Issue Tracking

[X] Apache NiFi Jira issue created

Pull Request Tracking

[X] Pull Request title starts with Apache NiFi Jira issue number, such as NIFI-00000
[X] Pull Request commit message starts with Apache NiFi Jira issue number, as such NIFI-00000

Pull Request Formatting

[X] Pull Request based on current revision of the main branch
[X] Pull Request refers to a feature branch with one commit containing changes

Verification
Please indicate the verification steps performed prior to pull request creation.
Build

[X] Build completed using mvn clean install -P contrib-check

[X] JDK 17

Licensing

[ ] New dependencies are compatible with the Apache License 2.0 according to the License Policy
[ ] New dependencies are documented in applicable LICENSE and NOTICE files

Documentation

[ ] Documentation formatting appears as expected in rendered files

+1 LGTM, thanks for the upgrade! Merging to support/nifi-1.x and main


Updated Attachement downloading function to use the WorkItemServer.

Updated the download function for attachments to use the WorkItemServer class instead of the WebClient. WebClient was not authenticating the user correctly.
Fixes #24

@jtucker, Can you update the version number in GitVersion.yaml to increment the Patch level please. Then I will be happy to merge it...

@jtucker, I did this for you on your repo and submitted a pull request. can you approve, so that I can approve this one and merge it...


status badges do not update b/c cache-control is not set to none

Thank you for taking the time to use Drone and file an issue or feature request. Before filing an issue please ensure the following boxes are checked, if applicable:

[x] I have searched for existing issues
[ ] I have discussed the issue with the community at https://gitter.im/drone/drone
[ ] I have provided a sample .drone.yml file to help the team reproduce
[ ] I have provided details from the build logs
[ ] I have provided details from the server logs by running docker logs drone
[ ] I am not using the issue tracker to ask why my build failed

The drone badge from our project's README is not updating properly.
Oddly, I can't seem to do 'curl -I' on the the badge URL to see what the headers are ??(https://drone.io/github.com/spacegraphcats/spacegraphcats/status.png) - it returns 404 -- but in any case this may be something that can be fixed by setting cache-control to None in the HTTP headers.
See https://github.com/github/markup/issues/224 for more.

Please contact the drone.io support, this issue tracker handles only the opensource version of drone and not the hosted service.


Uses OpenZipkin deploy process and migrates to io.zipkin.brave group id

This re-uses the deployment flow from zipkin-java which has the
following features:

Makes Brave releaseable by anyone with access to the Bintray org
Snapshot publishing to http://oss.jfrog.org/artifactory/oss-snapshot-local
Fully automated release to maven central via release-N.N.N tag

Releases also published to https://jcenter.bintray.com

Notably, this migrates the maven group id to io.zipkin.brave. This
needs to be communicated on release!
Fixes #161

Notably, this migrates the maven group id to io.zipkin.brave. This
needs to be communicated on release!

https://maven.apache.org/guides/mini/guide-relocation.html
<distributionManagement>
    <relocation>
      <groupId>io.zipkin.brave</groupId>
    </relocation>
  </distributionManagement>


Performance benchmarks

Are there any performance benchmarks vs other libs available?
JsClipper, the javascript port of Angus Johnsons Clipper (based on Vatti´s algorithm) come to mind for example

No, there aren't.  I would gladly put performance information in the README if someone provided it to me, and it was compared fairly with other libraries.

What would constitute a fair comparison?

I don't know, that's a good question.  That's part of the problem of reporting on performance.  I suppose it would depend on the features of the other library, and the expected use cases.  Do you plan on trying to perform a fair comparison?

Closing for now.


Selenoid with Kubernetes support

@alcounit have you seen Moon which does exactly the same but more efficiently?

@vania-pooh yes, I saw. I just want to propose minimal support for K8, all other extended features can be supported by Moon same as premium support.

@alcounit we have several considerations about Kubernetes in Selenoid:

We are trying to develop Selenoid as simple as possible. Even now it has a lot of sophisticated features like VNC, clipboard or devtools support. So adding Kubernetes support here is one more complication.
Selenoid architecture is stateful, i.e. is not adapted for multiple replicas behind load balancer and will lose all running sessions if OOM killer arrives. So basically we consider adding Kubernetes support a bad idea without changing Selenoid internal architecture. But all respective changes were already made in Moon so it makes no sense.
We have a similar PR (#455) related to Mesos support and our idea is to have a separate project based on Selenoid code. This is why it not still merged since May, 2018. Otherwise adding more and more features will sooner or later explode Selenoid code quality or our brain.
Having basic Kubernetes support users will sooner or later require more advanced features.

Having said that, if you wish - you can move all Selenoid code to a separate repository and add Kubernetes support there: it's open source.

@vania-pooh I understand, anyway thanks for your time.


[Bug]: Task - Filter Collecting Event: no results using a two data point search

Steps to reproduce the bug
1.Task, Filter Collecting Event
2.type "Fianarantsoa" in GA field
3.check "Treat geographic areas as spatial"
4.type "B.L. Fisher" into Collectors field, click "any"

There are no results but there should be a multiple results including https://sandwich.taxonworks.org/tasks/collection_objects/browse?collection_object_id=1813552

There are results with only the GA "Fianarantsoa"
There are no results with only the collector "B.L. Fisher", Brian L. Fisher"
There are no results with both the GA and collector

@bpescador, @chrisgrinter, @rich-keller

Screenshot
No response
Expected behavior
No response
Additional Screenshots
No response
Environment
Sandbox
Sandbox Used
No response
Version
build ee592d9d4 on 2023-01-13T15:59:36Z
Browser Used
Version 109.0.5414.75

@AntWeb-org can you please test this in unified framework?

I tested the same query in the unified filter CE and get the same results.
There are results with only the GA "Fianarantsoa"
There are no results with only the collector "B.L. Fisher", Brian L. Fisher"
There are no results with both the GA and collector

@mjy
CE Filter

starting with place name = Fianarantsoa it pops up to select from geographic area [as expected]
click filter  > produces no results.

Looked quickly at all the records on page one of the CE list (by clicking filter w/o selecting anything > and there are certainly records there where Level 2 geography = Fianarantsoa.
On sandwich
AntWeb dataset

Interestingly, some of these CE have another Collector with Brian, name = Sylvain.
IF I put "Sylvain" in the Collector field > I get a match.
Selecting "Sylvain" > click filter > now I get a reasonable subset of 43 records where the Collectors = Fisher, B.L.; Sylvain

That's weird. I don't get any results with "Sylvain"
(build e5fc74c08b18
Results with "Fisher" pull up results not including "B.L. Fisher" such as "W.E. Steiner".

@AntWeb-org @debpaul there are now updates that affect the "Role" pickers.  Please test again if you find time.

Success using the original filter search! I will have to double check that the results match the results in our current database. I'll keep you posted. Thanks!

Thanks very much @AntWeb-org.

So the total results in TW = 226 collecting events, while the total results for the same query in our current database = 2655 collecting events.
Right now it looks as if the TW results include only "B.L. Fisher" and "B.L. Fisher; Sylvain" but is excluding other "contains" results such as "B.L. Fisher; Malagasy Arthropod Team" and "B.L. Fisher; C.E. Griswold; Malagasy Arthropod Team".

I checked one and it looks like all collectors are in verbatim only.

This search is for the collecting events. The collection objects should yield many more results, over 10,000.

Aha, sorry!

Np!

So it looks like in this case the import can make People records out the interpreted verbatim string.   I.e. this is not a filter issue, but a reflection of the limitations of the importer, and the nature of the AntWeb data.
However, try searching on the verbatim_collectors attribute with Fisher and the spatial component.
I'm going to consider this issue closed, as it appears the logic is fine.

That seems like a problem. If we know what the issue is on our side so we know how to best import into TW that would be something we can change before the next import. Do we need to code something differently or change a header name?

@bpescador @AntWeb-org do you have an issue tracker for AntWeb data in general?  These types of conversations could continue in issues there?
The issue for AntWeb is now one of tactics, i.e. how you will evolve AntWeb Data.  It's nuanced, and might require discussion in person.  For example, you will never fix everything pre-import.  In this case, with well understood data that can be searched, you might think of updating (there is really nothing wrong with your data per-se) the richness of your data in TaxonWorks.  As batch update features grow slowly over time you'd be able to take advantage of them to improve (it's not really fix at this point) the semantics of your data.
So, I can't give you single "best" answer.  You can unify the format of the data in your verbatim_collectors, so that the parser can discover and relate to more people (I believe this is the core issue, but not 100% certain), or you can link those data to people once they are inside TaxonWorks.  For example we can imagine being able to batch add Brian as a collector against a filtered record set all at once in the future.
The other consideration is what the data look like when they come out (as DwC), if they look ok, with simplified semantics, then your downstream use is OK.

I am testing your suggestion: verbatim_collectors attribute with Fisher and the spatial component.
However, I need some guidance since it is not working.

If there is a way to work with our data within TW then I am fine with waiting to modify collectors after import. But first, we need to confirm how to conduct a search on the collector.

Could still be a real bug.  Will test a little more.

Sorry for the confusion all.  There is a problem with wildcarding, it's doing verbatim search on attributes that should be wildcarded.
It should be fixed for testing tomorrow morning.
Big apologies for the confusion.

Maybe we have it now?!


Bulk-loaded films tend to tear-off while winding to the next frame

Find the cause and solution.
Investigate

[x] Is it cameras fault?
[x] The canister's opening is too tight.
[ ] The way how it's taped? Too thick or the direction doesn't go with the flow.
[ ] Is it the amount of loaded film? Too much film creates friction and a clot.

Might help

Though they tend to do very similar as I do.

https://www.youtube.com/watch?v=eQfKUzk-zvw
https://www.youtube.com/watch?v=zKlFQCwbSy

Is it cameras fault?
Try 1 film and mark the results.
Soaper
Spotmatic
Nikon

The canister's opening is too tight
Try the same film from that was OK for the previous test.

The way how it's taped?

Too thick or the direction doesn't go with the flow.

Less tape
20 turns

Current conclusion
Tried more times and worked fine with:

the 20 frame strip.
Longer tape, more precisely glued.

Current conclusion
Tried more times and worked fine with:

the 20 frame strip.
Longer tape, more precisely glued.


[framework] use shader tiling instead of repeated calls to drawImage

Part of https://github.com/flutter/flutter/issues/119476
Avoids creating an offscreen pass for each drawImage call by compressing to a single TiledTexture entity. This still has a remaining inefficiency in that we don't defer the opacity, so color + opacity is still two separate offscreen passes. Part of wonderous iPad performance issue.
Appears to be visually identical, we need to consider a) the initial offset of the source and b) the scale down.
Before

After

@goderbauer we are losing some of our test assertions because the ImageShader objects aren't inspectable. 🤷

I mean we definitely could make these properties visibile. In fact, why don't I do that so we don't have to add anymore goldens 😆

I could expose the tile modes but the matrix itself is mutable byte data so exposing that may be counter productive.

I could also just make the method for creating the image shader data public and unit test that

FYI @goderbauer , thoughts?

I could expose the tile modes but the matrix itself is mutable byte data so exposing that may be counter productive.
Agreed. We could do a debugFoo field that's only available in debug mode, but its not ideal...

I could also just make the method for creating the image shader data public and unit test that
Which method is that?

I'd have to refactor a bit, but I could add a visible for testing method that took the rect arguments and gave you a matrix + tile modes?

I'd have to refactor a bit, but I could add a visible for testing method that took the rect arguments and gave you a matrix + tile modes?

That sounds like a good idea if that is doable.

Of course I did that and immediately found a bug

Removed debug name, also hide from export

Of course I did that and immediately found a bug

Write Test. Find bug. Nice!

@jonahwilliams this PR is 100 commits behind tip of tree, and Google Testing is unable to create an internal CL from it. Can you try merging with tip of tree to see if that clears up the stuck check?

Will do! ( Well, tomorrow)


fixed remarks from #3525 (closes #3681)

@normanjaeckel Please review later

Ok, but the code of the new factory is still very long and difficult to read ...


Request: Allow AUDIO_RECEIVER as Accessory Category

Issue

This is a great plugin and written very well.
I'd love the option to have my ZP Speakers be displayed as an Audio Receiver instead of a Television, as the image on the Home app tile fits their purpose much better as a "radio".
You could make this a user-configurable option.... for those that prefer TV vs Audio Receiver
To do this you only need to change the Accessory Category in this file:
https://github.com/ebaauw/homebridge-zp/blob/master/lib/ZpAccessory.js
Line 181 is the line that sets the category:
params.category = platform.Accessory.Categories.TELEVISION

The category that shows the device as an audio receiver is AUDIO_RECEIVER
I have been making my own plugin where the user can change the category between Television, AudoReceiver and SetTopBox, so I can report from experience that all other functionality stays working exactly the same.
The category controls primarily the icon on the Home tile.

As far as I'm aware, the icon corresponding to the category only shows when newly adding the accessory.  Can you please post a screenshot?

the icon appears when newly adding and stays after the accessory is added.
FWIW: my plugin works very much the same as yours: it auto-discovers the accessories and exposes them, and I give the user the option of TV or audio receiver. Let me do a screenshot later...

Cool, hadn't seen that before.  The only reason I expose the zone player as TV is for the input selection, but that just continues to work.  I think it would make most sense to use a SPEAKER category, wouldn't it?

I honestly cannot say if category SPEAKER could be used: I only tried the TV categories: TELEVISION, AUDIO_RECEIVER, TV_SET_TOP_BOX and TV_STREAMING_STICK, and notice no difference when switching between them.
I'll try it and see what happens...
...5 min later...

The accessory displays as a Speaker icon during the add accessory process
The accessory type is Speaker, "Connecting to Speaker", "Speaker location" etc. appears instead of "Connecting to Television"
Only for the input sources do you see "Name TV Input Sources", but that is also the same for the AudioReceiver
The accessory can be added normally
After adding, the tile is displayed as a Speaker: grey when off, brown when on.
All of the accessory functions appear to work exactly the same as a Television: Power, Input select, View TV Settings,
The iOS remote control still displays a TV (but then it displays a TV also for Audio Receiver. Only SetTopBox gets a different icon) however all the remote control functions appear to work normally.

It appears that Category controls the icon only... and I am guessing that it makes a difference to Siri control, as Siri would understand it to be a Speaker now instead of a TV.
How about making it a configurable item, allowing the user to choose between Television, AudioReceiver and Speaker?
And I'd recommend letting people test it for a while... I'm happy to be one of the testers....

By the way: my plugin where I did my testing is an External Accessory.

Not a fan of another config.json setting.
The TV accessory is external (not bridged); the bridged accessories don't show the category.

OK.
Continued testing by me shows that Category=Speaker has no apparent adverse impact at all.
And the tile icon is a much better fit to the Sonos speakers.
So I recommend changing to Category = Speaker ;)

Hi, I wanted to help b forking and trying it out, but I cannot run this plugin in my Win10 dev environment: npm link throws errors.
So I thought I'd ask if you'd consider the possibility of making a alpha or beta version with the category changed so I can test it?

OK, I created a PR for you

v1.2.1 uses SPEAKER.  Note that you will need to remove the external accessory and re-pair it to HomeKit, to make it show with a speaker icon in Home.

Thanks. I found that a Homebridge restart was enough...
i see the accessory the Serial Number is the Sonos MAC prefixed with "TV_".
That prefix should now be changed to "SPKR_"

Why?  It's still a TV accessory with a Television service.  We just changed the accessory category, to show a different icon.


Help with migrating from pegdown to flexmark-java

Hi, could we have some help with migrating from pegdown to flexmark-java please?
I've read the migration docs but it dosen't really say what to do if you do something like https://github.com/GerritCodeReview/gerrit/blob/master/java/com/google/gerrit/server/documentation/MarkdownFormatter.java
That file is then read into https://github.com/GerritCodeReview/gerrit/blob/09786353f76b778a76a61a092adf60a41fbc3cfd/java/com/google/gerrit/httpd/plugins/HttpPluginServlet.java#L498

I've managed to do it here https://gerrit-review.googlesource.com/#/c/gerrit/+/138774/
though how do i migrate it so it works for finding a children for the title please?

@paladox, the heading search to determine the document title can be found in the maven doxia project which recently migrated to flexmark-java from pegdown.
Specifically the code traversing the AST can be found here: https://github.com/vsch/maven-doxia/blob/63a9e0de2d4012b5eacc965413017b379640822f/doxia-modules/doxia-module-markdown/src/main/java/org/apache/maven/doxia/module/markdown/MarkdownParser.java#L214-L235
Above link is to my fork since I am not sure at what stage the merging process is at in the main project.
The flexmark-java-samples module has samples for many API features of the library, including samples created to help users migrate from pegdown. The visitor sample has a generic visitor example which can be used to visit any and all nodes in the AST.

Thankyou :).
I will try that.

It manages to find something but it is finding for example  and
with a bullet point that link to another page.
It does  but then it is hidden.

this is what i see ^^.
Though i am expecting it to do it to "Plugin commit-message-length-validator" for example.

Can you give me the snippet of the actual markdown that is being parsed up to and including the "Documentation" heading as fenced-code in a comment
```markdown
# Plugin commit-message.....
....
```

and the java code used to parse for the heading too. For example:
```java
        if ( !haveTitle && rootNode.hasChildren() )
        {
            // use the first (non-comment) node only if it is a heading
            Node firstNode = rootNode.getFirstChild();
            while ( firstNode != null && !( firstNode instanceof Heading ) )
            {
                if ( !( firstNode instanceof HtmlCommentBlock ) )
                {
                    break;
                }
                firstNode = firstNode.getNext();
            }

            if ( firstNode instanceof Heading )
            {
                html.append( "<title>" );
                TextCollectingVisitor collectingVisitor = new TextCollectingVisitor();
                String headingText = collectingVisitor.collectAndGetText( firstNode );
                html.append( StringEscapeUtils.escapeXml( headingText ) );
                html.append( "</title>" );
            }
        }
```

Note that the maven doxia looks for the first non-comment node and if it is a heading, it will use it. Your criteria may be different.

Hi, im not exactly sure what markdown is being used as we use HttpPluginServlet to generate the index.html file. The rest is generated in the .md file ext.
The java code i use is
    if (root.hasChildren()) {
      Node firstNode = root.getFirstChild();
      while ( firstNode != null && !( firstNode instanceof Heading ) ) {
        if ( !( firstNode instanceof HtmlCommentBlock ) ) {
          break;
        }
        firstNode = firstNode.getNext();
      }
      if ( firstNode instanceof Heading ) {
        StringBuilder b = new StringBuilder();
        b.append("<title>");
        TextCollectingVisitor collectingVisitor = new TextCollectingVisitor();
        String headingText = collectingVisitor.collectAndGetText(firstNode);
        b.append(headingText);
        b.append("</title>");
        return b.toString();
      }
    }

https://gerrit-review.googlesource.com/#/c/gerrit/+/138774/

This is what it looks like when using the old pegdown parser

This is what the prevous code was
    if (root instanceof HeaderNode) {
      HeaderNode h = (HeaderNode) root;
      if (h.getLevel() == 1 && h.getChildren() != null && !h.getChildren().isEmpty()) {
        StringBuilder b = new StringBuilder();
        for (Node n : root.getChildren()) {
          if (n instanceof TextNode) {
            b.append(((TextNode) n).getText());
          }
        }
        return b.toString();
      }
    }

    for (Node n : root.getChildren()) {
      String title = findTitle(n);
      if (title != null) {
        return title;
      }
    }
    return null;

Give me a few minutes to create a new sample specific to your requirements for title search.

@paladox, I added a TitleExtract.java sample which does what the old pegdown code used to do but with flexmark-java AST.
The input is based on your screenshots as best as I could figure it out:
# Plugin commit-message-length-validator

Vendor
:  Gerrit Code Review

Version
:  v2.15-rc2

## About

This plugin checks the length of a commit's commit message subject and body, and reports warnings and errors to the git client if the lentghts are exceeded.

## Documentation

* [Commit Message Length Configuration](config.md)
* More Items

The output of the code extracts the title and outputs the HTML:
Title: Plugin commit-message-length-validator

Plugin commit-message-length-validator

Vendor

Gerrit Code Review

Version

v2.15-rc2

About

This plugin checks the length of a commit’s commit message subject and body, and reports warnings and errors to the git client if the lentghts are exceeded.

Documentation

Commit Message Length Configuration

More Items

Hope this helps.

Hi thankyou so much for that :)
But still not adding  to "Plugin commit-message-length-validator"

@paladox, can you open the ... node so we can see its content. That is where the <title> tag goes.

Yep title is there

But in the old parser it was making "Plugin commit-message-length-validator" not a hyperlink, in the new one it is making it a hyperlink. (sorry just noticed that now).
thankyou so much for your help too:).

I wonder how do I get “Plugin commit-message-length-validator” to not be a hyper link please? (Like in the pegdown parser)

@paladox, making the heading into a link was pegdown's default behavior and I don't remember having any exclusions for heading level 1.
This leads me to believe that there could be some customization for the html serializer in the old pegdown implementation that removes the anchor link wrapper for the heading text when it is a level 1 heading.

Ah that would also explain why other headings are hyperlink in pegdown too. Could this be an option in flexmark please? Or would that be to much to implement? :)

Maybe it’s here https://github.com/sirthias/pegdown/blob/2a514555a84f56fe112155748fc24d84046e9c88/src/main/java/org/pegdown/Parser.java#L284 when it does the level 1 thing.

I don't think it is in pegdown code but I could be mistaken. pegdown had a lot of idiosyncrasies in the way it generated some elements.
The easiest way to handle this is to create an extension that skips anchor links for headings level 1. I am working on it now to give you a hand.
It looks like this was dumped in your lap and you had nothing to do with the original implementation so it is hard for you to migrate from code you don't know to new code you don't know.

@paladox, I modified the TitleExtract.java sample to add a custom extension that skips rendering of anchor links for headings level 1
This is implemented in two static classes and addition of the extension to the options so that rendering will be filtered through the custom extension.
The generated code is now:

Plugin commit-message-length-validator

Vendor

Gerrit Code Review

Version

v2.15-rc2

commitmessage.maxSubjectLength

Maximum length of the commit message’s subject line.  Defaults to 50 if not specified or less than 0.

About

This plugin checks the length of a commit’s commit message subject and body, and reports warnings and errors to the git client if the lentghts are exceeded.

Documentation

Commit Message Length Configuration

More Items

I am working on the tab in definition list bug and will make a new release with a fix for that.

Thankyou so so much :)

@vsch wondering do you know how i can get the toc working?
In the old parser it was adding the toc, in the new one it is showing this

Also the same for Overview. It's showing as "[overview]: #overview Overview" but should be "Overview".

@vsch i think i found a bug. If i put another space
ie for example
end of text

[rules]: #rules
Rules

^^ that fails.
if i do

end of text

[rules]: #rules
Rules

(notice the extra space.)

@paladox, the parsing between pegdown and CommonMark, which this library is based on, is different.
pegdown syntactically parses the full markdown text. CommonMark parses it for block constructs then parses the blocks for inlines.
This causes a limitation on how constructs are parsed.
However, that said, I cannot duplicate the issue of needing extra blank line before a reference definition.
In CommonMark reference definitions can be located as the first elements of a paragraph. This means the first reference definition needs to be preceded by a blank line.
The text:
[event-properties]: #event-properties
Event Properties
-----------------------------------------------

is a level 2 heading that contains a reference link [event-properties] followed by text : #event-properties and inline HTML.
Reference definitions are not parsed in heading text.
For full reference of CommonMark specification you should reference  http://spec.commonmark.org/0.28/
The markdown generated will have to be adjusted for the difference between pegdown and flexmark-java. It is not possible to convert flexmark-java to be 100% compatible with pegdown implementation due to differences in the parser architecture. This difference in parsing architecture is what allows flexmark-java to resolve parsing issues of pegdown and a 20x improvement of performance.

@vsch thanks, it seems doing that kind of works but then clicking on it wont bring you to it.
because doing Event Properties causes it to have  twice thus.
And also doing [test][event-properties] fails it seems if i do that.
But if i do
[event-properties]: #event-properties
## Event Properties

it works


Update Field.jsx

Summary
I'm am receiving these console warnings, but had to go to the src code to see what prop should be set:

Danger is probably going to fail on the changelog check


Allow Default Values for String Literal Types

TypeScript Version:
1.8.2.
Code
// A self-contained demonstration of the problem follows...
class Foo {
    bar: "fizz" | "buzz";
}

class Bar extends Foo {
    bar = "fizz";
}

Expected behavior:
Bar.bar is initialized to "fizz";
Actual behavior:
Type 'string' is not assignable to type '"buzz"'

This works:
interface Foo {
	bar: "fizz" | "buzz"
}

class Bar implements Foo {
	bar: "fizz" = "fizz";
}

as does this:
interface Foo {
	bar: "fizz" | "buzz"
}

class Bar implements Foo {
	bar: "fizz" | "buzz" = "fizz";
}

I'd like to see the former inferred from the original, or at least allow assigning a default value.

Another example:
class Foo {
	bar: "fizz" | "buzz";
}

class Bar1 extends Foo {
	constructor() {
		super();
		this.bar = "fizz";  // okay
	}
}

class Bar2 extends Foo {
	bar = "fizz";  // fail
}

I think #6118 should fix this. CC @sandersn

I ran the original code through the 2.0.2 compiler, and the error remains.

The message has changed to:
test.ts(6,7): error TS2415: Class 'Bar' incorrectly extends base class 'Foo'.
  Types of property 'bar' are incompatible.
    Type 'string' is not assignable to type '"fizz" | "buzz"'.

Yep, neither #6118 or #10610 are merged yet. We are trying to determine how bad the breaking change is right now.

Ah, fair enough.

Duplicate #10570


Feature Request

Could we add an item spanwer function to into the game?

There is no need, just use the "additem" debug console command. (I'm not actively working on mods anymore)


Update mc_locale_de-de.json

Sort file for better overview

It will be good to keep language name at first, for rest ok.
"MENU.LANGUAGE_NAME" : "De-de",

i think it´s defined in filename mc_locale_de-de.json

@Dietmar-Franken What I feel is,  still having language name alone as first field. It's ok, I will merge it now, If others having same feeling of mine. we will make it as first field.
Thank you!


Icons

Currently, the icon files referenced by presets are only located in the iD repo, which is sort of janky. I'd like to see about improving this situation.

Seems solved now? See https://github.com/ideditor/schema-builder/blob/main/ICONS.md


Use signing protocol 1.1 by default

All supported Chef servers support the 1.1 signing protocol.
There is no reason to continue using 1.0, and removing it
as the default allows us to remove a bunch of code that tries
to upgrade 1.0 to 1.1 when the node name is too long.
If the user specifies 1.0 as the auth protocol version from
this point on, they will have to guarantee that the node
name is not too long.
cc @chef/client-core

:+1: when travis is happy

yay! :+1:

:+1:


Project has an unmet peer dependency: "tslib"

Description/Screenshot
There is a warning about a missing peerDependency. This is not expected to happen because my project doesn't even use TS.
Steps to Reproduce
Run yarn add @microsoft/applicationinsights-web in a JS project and you will receive the following warning:
warning " ><EMAIL_ADDRESS>has unmet peer dependency<EMAIL_ADDRESS>
Expected behavior
You should be able to install the lib without warnings. In my opinion, tslib shouldn't be a peer dependency.

tslib isn't directly a "typescript" dependency, but rather a helper. It makes bundle sizes smaller by making the assumption that most libs being consumed are reusing the same helpers via tslib. It is a direct dependency of all of the "extensions" of this project, and so the peer dependency should be met anyways after installing. Your app would be throwing errors if tslib were not installed via the full dep tree, although yarn might not be checking the full dependency tree before checking peer dependencies.
It might make sense to just add tslib as a pure dep to the -web package to suppress this error

Thanks for the explanation. In this case, it is indeed best suited as a dependency (like Formik does, for example, https://github.com/jaredpalmer/formik/blob/master/package.json#L53)
If you add a dependency as a peerDepedency it is expected that the consumer have it in their own package.json, which is not necessary in this case.

Thanks for the feedback! I'll move it over as a pure dep

Ran into the same. Cheers for looking into this 👌🏻

This was fixed by #1116, but will be released in Jan release


Clone Z-95s w/ Incorrect Pilot Initiative Value

Fly Casual Release Version
V2.5.20241102.02
Online Game Mode
No
Describe the bug
The 7th Sky Core Pilots in the Clone Z-95 Headhunters are falsely set to pilot initiative 3 (for activation/engagement & it's on their bases).
To Reproduce
Steps to reproduce the behavior:

Fly "7th Sky Core Pilots" in the Clone Z-95 Headhunters.

Expected behavior

the 7th Sky Core Pilots in the Clone Z-95 Headhunters should be at pilot initiative 2.

Screenshots
Attached
Platform/Device (be as specific as possible):

OS: Android
Version 14

Confirmed wrong initiaive.


How about a Graph Database?

What do you think about a Graph DB.
It will be easier to manage these relations.
These is already a project https://github.com/google/cayley that makes use of BoltDB as a graph database.

@evnix let me take a look at that! Could be useful. How do you think it would improve MuLi?

@evnix I been taking a look at Cayley. Here are my thoughts:
1- It could allow searches in the structure.
2- I would improve the relations between Playlists and Songs (in the Artist tree and the Playlist tree).
3- It could allow to add more information about the Songs (year, genre, etc..).
Regarding the first idea, I think that since this is a FileSystem, the OS or the Shell should care about how the data is accessed from the user. I mean if the user wants to perform a search, he should use the shell tools to do that.
The second Idea looks interesting but it would add an extra complexity and size to the project and maybe it is not the time yet to do that kind of things. I can take some ideas from Cayley to improve the connections between Playlists and Songs, I think it would be a good improvement in that area.
The third point can also be added in other ways. I still don’t know about having that much information about the songs and everything. I have been looking other similar projects (also projects that enable searches using the directory structure) and they look like they add all the extra complexity that I am trying to avoid with MuLi. It has to be an easier way to add this extra information.
What do you think? Do you have any other inputs?


redis_proxy: add formatter support for catch_all_route

Commit Message: catch_all_route is of type extensions.filters.network.redis_proxy.v3.RedisProxy.PrefixRoutes.Route. However remove_prefix, prefix and key_formatter does not work when set against catch_all_route. This PR fixes the same and adds a unit test exercising these properties against catch_all_route
Additional Description: Related to https://github.com/envoyproxy/envoy/pull/26560
Risk Level: Low
Testing: Unit test
Docs Changes: Yes
Release Notes: Yes
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

@weisisea @mattklein123 - when you have a moment, could you please review this PR ? Thanks!

/assign @mattklein123

@mattklein123 - when you have a moment, could you please take a look at this PR ? Thanks!


unstable: command/flag to run lua script

(flag makes sense so we can do it alongside the repl, but also standalone)

done in 7e05232


Probably wrong documentation for lpf_collectives_init

The documentation says the the parameter max_calls of lpf_collectives_init must be larger than 0
https://github.com/Algebraic-Programming/LPF/blob/436b7d3d8d66fbaca0fef40cd20ab99952c3cc1a/include/lpf/collectives.h#L135-L137
but in ALP it is used successfully as 0 (tested !)
https://github.com/Algebraic-Programming/ALP/blob/b50fe72e957ed4da10c1cd1c59d924260f051a8d/include/graphblas/bsp1d/exec.hpp#L67
and indeed the implementation ignores it
https://github.com/Algebraic-Programming/LPF/blob/436b7d3d8d66fbaca0fef40cd20ab99952c3cc1a/src/core-libraries/collectives.c#L88
Should the documentation be changed?

The doc (spec) is OK, it's ALP that should fix it, even if the implementation happens to be ok with it.
I was briefly considering returning an error, but LPF does typically not guard against trivial errors (nor any type of errors) by its philosophy. We could introduce an ILLEGAL error code for higher-level libraries, but then such libraries are probably better placed outside the LPF repo. So doing nothing here seems best.
Anyway, creating an issue on the ALP repo: number 249 (https://github.com/Algebraic-Programming/ALP/issues/249) and closing this one.


Examples: add w-mlx

Adds an example using mlx.
mlx is an OCaml dialect which extends regular OCaml syntax with JSX expressions. The dialect features first class support from merlin/lsp.

Thanks! This looks great! I'm playing around with it.
What is needed to get the syntax highlighting to work? I have this example building and running with the exact code in this PR. I am using:
VSCode 1.92.1
OCaml Platform 1.20.0
OCaml 5.2.0
Dune 3.16.0
ocaml-lsp-server 1.18.0~5.2preview
ocamlmerlin-mlx 0.9
mlx 0.9

What is needed to get the syntax highlighting to work? I have this example building and running with the exact code in this PR. I am using:

Unfortunately this needs an extension for vscode which doesn't yet exists.
Though there exists a tree-sitter-mlx parser I use in an ocaml_mlx.nvim so if there's some generic vscode extensions which could use tree sitter parsers for highlighting then it'd work now.

Thank you!


Failed to start DBServer: Recovery failure due to error from callback, error 'internal error' message: failed to find checkpoint after finishing recovery of arangosearch link '11020991'

My Environment

ArangoDB Version:     3.5 RC3
Storage Engine:      RocksDB
Deployment Mode:   Cluster
Deployment Strategy:  Manual Start
Configuration:     I run Agent, Coordinator and Dbserver on 3 server. The config is not problem
Infrastructure:        own server
Operating System:     Ubuntu 18.04
Total RAM in your machine:  32GB
Disks in use:   HDD
Used Package:     Debian

Component, Query & Data
Size of your Dataset on disk: 100GB
Replication Factor & Number of Shards (Cluster only):

3 Replication Factor and 3 Shards  per collection (i have over 10 collections with over 100GB data)
Steps to reproduce
I have 3 server, can call: SERVER1 , SERVER2, SERVER3 (3 separate machine) . I have setup cluster fully and correctly all.
While importing data, i had stop SERVER1 , SERVER2, by using like this: (all config true)
systemctl stop arangodb3-dbserver
systemctl stop arangodb3-agent
systemctl stop arangodb3-coordinator

After 30 seconds, i start them again and got error.
2019-06-26T08:50:45Z [31905] ERROR [772a7] recovery failure due to error from callback, error 'internal error' message: failed to find checkpoint after finishing recovery of arangosearch link '11020991'
2019-06-26T08:50:45Z [31905] ERROR [4ec19] {startup} startup aborted: caught exception during start of feature 'RocksDBRecoveryManager': failed to find checkpoint after finishing recovery of arangosearch link '11020991' (exception location: /work/ArangoDB/arangod/RestServer/DatabaseFeature.cpp:539). Please report this error to arangodb.com. shutting down
2019-06-26T08:50:46Z [31905] INFO [4bcb9] ArangoDB has been shut down
2019-06-26T08:50:46Z [31905] ERROR [5d508] arangod terminated because of an exception: startup aborted: caught exception during start of feature 'RocksDBRecoveryManager': failed to find checkpoint after finishing recovery of arangosearch link '11020991' (exception location: /work/ArangoDB/arangod/RestServer/DatabaseFeature.cpp:539). Please report this error to arangodb.com (exception location: /work/ArangoDB/lib/ApplicationFeatures/ApplicationServer.cpp:680). Please report this error to arangodb.com

Problem:

I don't know what happen? maybe problem with sharding and replication of Arangodb when suddenly interrupt system?
Please help me

Seeing that this was a release candidate version, it is possible that there was a bug. Is this problem reproducible in the latest version?

Closing because of lack of feedback and because a non-GA version was used.


[Bug] CUDA errors when changing the voxel size.

Prerequisite

[X] I have searched Issues and Discussions but cannot get the expected help.
[X] I have read the FAQ documentation but cannot get the expected help.
[X] The bug has not been fixed in the latest version (dev-1.x) or latest version (dev-1.0).

Task
I have modified the scripts/configs, or I'm working on my own tasks/models/datasets.
Branch
main branch https://github.com/open-mmlab/mmdetection3d
Environment
sys.platform: linux
Python: 3.8.16 | packaged by conda-forge | (default, Feb  1 2023, 16:01:55) [GCC 11.3.0]
CUDA available: True
numpy_random_seed:<PHONE_NUMBER>
GPU 0: NVIDIA A100-SXM4-40GB
CUDA_HOME: /usr/local/cuda
NVCC: Cuda compilation tools, release 11.7, V11.7.99
GCC: gcc (Ubuntu 9.4.0-1ubuntu1~20.04.1) 9.4.0
PyTorch: 2.0.1
PyTorch compiling details: PyTorch built with:

GCC 9.3
C++ Version: 201703
Intel(R) oneAPI Math Kernel Library Version 2023.1-Product Build 20230303 for Intel(R) 64 architecture applications
Intel(R) MKL-DNN v2.7.3 (Git Hash 6dbeffbae1f23cbbeae17adb7b5b13f1f37c080e)
OpenMP 201511 (a.k.a. OpenMP 4.5)
LAPACK is enabled (usually provided by MKL)
NNPACK is enabled
CPU capability usage: AVX2
CUDA Runtime 11.7
NVCC architecture flags: -gencode;arch=compute_37,code=sm_37;-gencode;arch=compute_50,code=sm_50;-gencode;arch=compute_60,code=sm_60;-gencode;arch=compute_61,code=sm_61;-gencode;arch=compute_70,code=sm_70;-gencode;arch=compute_75,code=sm_75;-gencode;arch=compute_80,code=sm_80;-gencode;arch=compute_86,code=sm_86;-gencode;arch=compute_37,code=compute_37
CuDNN 8.5
Magma 2.6.1
Build settings: BLAS_INFO=mkl, BUILD_TYPE=Release, CUDA_VERSION=11.7, CUDNN_VERSION=8.5.0, CXX_COMPILER=/opt/rh/devtoolset-9/root/usr/bin/c++, CXX_FLAGS= -D_GLIBCXX_USE_CXX11_ABI=0 -fabi-version=11 -Wno-deprecated -fvisibility-inlines-hidden -DUSE_PTHREADPOOL -DNDEBUG -DUSE_KINETO -DLIBKINETO_NOROCTRACER -DUSE_FBGEMM -DUSE_QNNPACK -DUSE_PYTORCH_QNNPACK -DUSE_XNNPACK -DSYMBOLICATE_MOBILE_DEBUG_HANDLE -O2 -fPIC -Wall -Wextra -Werror=return-type -Werror=non-virtual-dtor -Werror=bool-operation -Wnarrowing -Wno-missing-field-initializers -Wno-type-limits -Wno-array-bounds -Wno-unknown-pragmas -Wunused-local-typedefs -Wno-unused-parameter -Wno-unused-function -Wno-unused-result -Wno-strict-overflow -Wno-strict-aliasing -Wno-error=deprecated-declarations -Wno-stringop-overflow -Wno-psabi -Wno-error=pedantic -Wno-error=redundant-decls -Wno-error=old-style-cast -fdiagnostics-color=always -faligned-new -Wno-unused-but-set-variable -Wno-maybe-uninitialized -fno-math-errno -fno-trapping-math -Werror=format -Werror=cast-function-type -Wno-stringop-overflow, LAPACK_INFO=mkl, PERF_WITH_AVX=1, PERF_WITH_AVX2=1, PERF_WITH_AVX512=1, TORCH_DISABLE_GPU_ASSERTS=ON, TORCH_VERSION=2.0.1, USE_CUDA=ON, USE_CUDNN=ON, USE_EXCEPTION_PTR=1, USE_GFLAGS=OFF, USE_GLOG=OFF, USE_MKL=ON, USE_MKLDNN=ON, USE_MPI=OFF, USE_NCCL=ON, USE_NNPACK=ON, USE_OPENMP=ON, USE_ROCM=OFF,

TorchVision: 0.15.2
OpenCV: 4.7.0
MMEngine: 0.7.4
MMDetection: 3.0.0
MMDetection3D: 1.1.0+fa724b1
spconv2.0: True
Reproduces the problem - code sample
In the base model file for pointpillars, just modified the voxel size entry (first line)
voxel_size = [0.05, 0.05, 4]
Reproduces the problem - command or script
python tools/train.py configs/pointpillars/pointpillars_hv_secfpn_8xb6-160e_kitti-3d-3class.py
Reproduces the problem - error message
A bunch of these :
/opt/conda/conda-bld/pytorch_1682343962757/work/aten/src/ATen/native/cuda/IndexKernel.cu:92: operator(): block: [446,0,0], thread: [113,0,0] Assertion index >= -sizes[i] && index < sizes[i] && "index out of bounds" failed.
/opt/conda/conda-bld/pytorch_1682343962757/work/aten/src/ATen/native/cuda/IndexKernel.cu:92: operator(): block: [446,0,0], thread: [114,0,0] Assertion index >= -sizes[i] && index < sizes[i] && "index out of bounds" failed.
/opt/conda/conda-bld/pytorch_1682343962757/work/aten/src/ATen/native/cuda/IndexKernel.cu:92: operator(): block: [446,0,0], thread: [115,0,0] Assertion index >= -sizes[i] && index < sizes[i] && "index out of bounds" failed.
/opt/conda/conda-bld/pytorch_1682343962757/work/aten/src/ATen/native/cuda/IndexKernel.cu:92: operator(): block: [446,0,0], thread: [116,0,0] Assertion index >= -sizes[i] && index < sizes[i] && "index out of bounds" failed.
/opt/conda/conda-bld/pytorch_1682343962757/work/aten/src/ATen/native/cuda/IndexKernel.cu:92: operator(): block: [446,0,0], thread: [117,0,0] Assertion index >= -sizes[i] && index < sizes[i] && "index out of bounds" failed.
/opt/conda/conda-bld/pytorch_1682343962757/work/aten/src/ATen/native/cuda/IndexKernel.cu:92: operator(): block: [446,0,0], thread: [118,0,0] Assertion index >= -sizes[i] && index < sizes[i] && "index out of bounds" failed.
/opt/conda/conda-bld/pytorch_1682343962757/work/aten/src/ATen/native/cuda/IndexKernel.cu:92: operator(): block: [446,0,0], thread: [119,0,0] Assertion index >= -sizes[i] && index < sizes[i] && "index out of bounds" failed.
/opt/conda/conda-bld/pytorch_1682343962757/work/aten/src/ATen/native/cuda/IndexKernel.cu:92: operator(): block: [446,0,0], thread: [120,0,0] Assertion index >= -sizes[i] && index < sizes[i] && "index out of bounds" failed.
/opt/conda/conda-bld/pytorch_1682343962757/work/aten/src/ATen/native/cuda/IndexKernel.cu:92: operator(): block: [446,0,0], thread: [121,0,0] Assertion index >= -sizes[i] && index < sizes[i] && "index out of bounds" failed.
/opt/conda/conda-bld/pytorch_1682343962757/work/aten/src/ATen/native/cuda/IndexKernel.c
And :
Traceback (most recent call last):
File "tools/train.py", line 135, in
main()
File "tools/train.py", line 131, in main
runner.train()
File "/nfs/home/gouidr/anaconda3/envs/openmmlab/lib/python3.8/site-packages/mmengine/runner/runner.py", line 1721, in train
model = self.train_loop.run()  # type: ignore
File "/nfs/home/gouidr/anaconda3/envs/openmmlab/lib/python3.8/site-packages/mmengine/runner/loops.py", line 96, in run
self.run_epoch()
File "/nfs/home/gouidr/anaconda3/envs/openmmlab/lib/python3.8/site-packages/mmengine/runner/loops.py", line 112, in run_epoch
self.run_iter(idx, data_batch)
File "/nfs/home/gouidr/anaconda3/envs/openmmlab/lib/python3.8/site-packages/mmengine/runner/loops.py", line 128, in run_iter
outputs = self.runner.model.train_step(
File "/nfs/home/gouidr/anaconda3/envs/openmmlab/lib/python3.8/site-packages/mmengine/model/base_model/base_model.py", line 114, in train_step
losses = self._run_forward(data, mode='loss')  # type: ignore
File "/nfs/home/gouidr/anaconda3/envs/openmmlab/lib/python3.8/site-packages/mmengine/model/base_model/base_model.py", line 340, in _run_forward
results = self(**data, mode=mode)
File "/nfs/home/gouidr/anaconda3/envs/openmmlab/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/nfs/home/gouidr/mmdetection3d/mmdet3d/models/detectors/base.py", line 75, in forward
return self.loss(inputs, data_samples, **kwargs)
File "/nfs/home/gouidr/mmdetection3d/mmdet3d/models/detectors/single_stage.py", line 74, in loss
x = self.extract_feat(batch_inputs_dict)
File "/nfs/home/gouidr/mmdetection3d/mmdet3d/models/detectors/voxelnet.py", line 43, in extract_feat
x = self.middle_encoder(voxel_features, voxel_dict['coors'],
File "/nfs/home/gouidr/anaconda3/envs/openmmlab/lib/python3.8/site-packages/torch/nn/modules/module.py", line 1501, in _call_impl
return forward_call(*args, **kwargs)
File "/nfs/home/gouidr/mmdetection3d/mmdet3d/models/middle_encoders/pillar_scatter.py", line 36, in forward
return self.forward_batch(voxel_features, coors, batch_size)
File "/nfs/home/gouidr/mmdetection3d/mmdet3d/models/middle_encoders/pillar_scatter.py", line 86, in forward_batch
this_coors = coors[batch_mask, :]
RuntimeError: CUDA error: device-side assert triggered
CUDA kernel errors might be asynchronously reported at some other API call, so the stacktrace below might be incorrect.
For debugging consider passing CUDA_LAUNCH_BLOCKING=1.
Compile with TORCH_USE_CUDA_DSA to enable device-side assertions.
Additional information
I wanted to play with the voxel size values for pointpillars but I get these CUDA error when training the model. Any insight on this ?

If you modify the voxel_size,  you need to modify the output_shape correspondingly.

@IdrissGo , did you change output_shape and trained a model. I have modified point_cloud_range, voxel_size, and out_size but i was getting the below error

Anything else that i need to modify?

FYI, i have modified point_cloud_range, voxel_size, output_shape in pts_middle_encoder, grid_shape in train_cfg solved the issue and am able to train the model.


[5.3] change to loosey comparison to allow queue to run without --tries

Fixes the issue where running php artisan queue:listen marks jobs failed unless --tries=1
The issue is referenced in #15319

Surely we just need to set a correct default in the command. This fix doesn;t look correct?

Let's just revert the broken commit.

NB, this PR has just been reverted.

Instead of switching back to loose comparing the parameters should be casted to int at the correct position. E.g. Instead doing it here: https://github.com/laravel/framework/blob/5.3/src/Illuminate/Queue/Worker.php#L242
casting may take place here: https://github.com/laravel/framework/blob/5.3/src/Illuminate/Queue/Console/ListenCommand.php#L98
See issue https://github.com/laravel/framework/issues/15370 for more details.

@larsbo yeh, we already did this


figure to go into documentation

this is a figure of possible GPIO connections to endpoint switches and other devices.

added a figure of possible GPIO connections to endpoint switches and other devices.

PR intended for my fork, not madwcowswe:master


Can this lib build for iOS?

RT

It should, yes, but the CMakeLists.txt would need some modifications, I suppose!
I cannot test this without a Mac. Can you try this and report here for everyone else?

what do you need, @Vienta ?

@HighSparrow   I have a project want to use c++ sqlite lib for cross-platform.Then I find this lib.Do you have any advices?

@Vienta  It is MIT licensed, just use the codes directly as an external part if your project is in c++.
There is no cocoapods for it, I think.
To build it as a static lib, you can port these codes in a Xcode project and use xcodebuild to generate libs for simulators and devices and then you can archive those libs in different archs as one by lipo.

@HighSparrow
Thx for you advice.I'll try it.

Hi, thanks for the advices, and please come back with your results when you get some.

Hey, I'm having the same problem.
@Vienta Do you solve this problem?
I'm trying to import the source files instead of creating a static lib, but the imports give me an error.
I think it’s because the import’s inside the library are #include <SQLiteCpp/Database.h> and not #include “Database.h”, but I’m not sure, could anyone help please?
I create a github repo as an example. Thanks
https://github.com/4brunu/SQLiteCppExample

@4brunu  You should set 'Header Search Paths' like 'your_path_to/deps/SQLiteCpp/include' in xcode 'Building settings'.

@zhouxiaoxiaoxujian thanks a lot, it worked! You made my day ;)


The Docker build are failing

Hi,
I have started looking into this repository and tried to build the dev and prod without success.
This  Dev step generated the following error:
[~/projects/mlstr/metadata-annotation-workbench] main
>docker compose --env-file dev.env up prediction
[+] Running 0/1
 ⠿ prediction Error                                                                                1.4s
Error response from daemon: Get "https://gitlab.zbmed.de:5050/v2/": dial tcp <IP_ADDRESS>:5050: connect: connection refused

And when I tried the Prod steps to build and load the containers had this error:
[~/projects/mlstr/metadata-annotation-workbench] main
>docker compose --env-file dev.env build
[+] Building 1.1s (15/20)
 => [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/backend:dev internal] load build definition from Dockerfile                                                                                0.1s
 => => transferring dockerfile: 347B                                                                                0.0s
 => [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/frontend:dev internal] load build definition from Dockerfile                                                                                0.1s
 => => transferring dockerfile: 374B                                                                                0.0s
 => [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/frontend:dev internal] load metadata for docker.io/library/nginx:alpine                                                                         0.5s
 => [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/backend:dev internal] load metadata for docker.io/library/python:3.8                                                                            0.5s
 => [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/frontend:dev internal] load .dockerignore                                                                                0.1s
 => => transferring context: 3.88kB                                                                                0.0s
 => [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/backend:dev internal] load .dockerignore                                                                                0.1s
 => => transferring context: 2B                                                                                0.0s
 => [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/frontend:dev internal] load build context                                                                                0.1s
 => => transferring context: 2.39kB                                                                                0.0s
 => CANCELED [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/frontend:dev 1/6] FROM docker.io/library/nginx:alpine@sha256:a5127daff3d6f4606be3100a252419bfa84fd6ee5cd74d0feaca1a5068f97dcf          0.2s
 => => resolve docker.io/library/nginx:alpine@sha256:a5127daff3d6f4606be3100a252419bfa84fd6ee5cd74d0feaca1a5068f97dcf                                                                                0.1s
 => => sha256:a5127daff3d6f4606be3100a252419bfa84fd6ee5cd74d0feaca1a5068f97dcf 9.07kB / 9.07kB                                                                                0.0s
 => => sha256:074604130336e3c431b7c6b5b551b5a6ae5b67db13b3d223c6db638f85c7ff56 2.50kB / 2.50kB                                                                                0.0s
 => => sha256:c7b4f26a7d93f4f1f276c51adb03ef0df54a82de89f254a9aec5c18bf0e45ee9 11.21kB / 11.21kB                                                                                0.0s
 => CANCELED [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/backend:dev 1/6] FROM docker.io/library/python:3.8@sha256:ecd06b59ae52c021ae58b8fea0957bf5b0f4568783cb0240b56e06ed1720cc19             0.4s
 => => resolve docker.io/library/python:3.8@sha256:ecd06b59ae52c021ae58b8fea0957bf5b0f4568783cb0240b56e06ed1720cc19                                                                                0.2s
 => => sha256:ad4a9aed1d241d025af20a3abb37c4adc6084a13ea1c33391ab55a737b578faa 2.52kB / 2.52kB                                                                                0.0s
 => => sha256:f5c5013453fab04241dca0dc61a4629a2cfe731e7411e6b5f81af77d70798c3d 7.32kB / 7.32kB                                                                                0.0s
 => => sha256:ecd06b59ae52c021ae58b8fea0957bf5b0f4568783cb0240b56e06ed1720cc19 10.35kB / 10.35kB                                                                                0.0s
 => => sha256:8cd46d290033f265db57fd808ac81c444ec5a5b3f189c3d6d85043b647336913 0B / 49.56MB                                                                                0.1s
 => [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/backend:dev internal] load build context                                                                                0.1s
 => => transferring context: 481.25kB                                                                                0.0s
 => CACHED [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/frontend:dev 2/6] COPY nginx.conf /etc/nginx/templates/default.conf.template                                                             0.0s
 => CACHED [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/frontend:dev 3/6] COPY nginx-server.conf /etc/nginx/nginx.conf                                                                           0.0s
 => CACHED [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/frontend:dev 4/6] COPY setVariables.sh /docker-entrypoint.d/50-insert-runtime-variables.sh                                               0.0s
 => CACHED [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/frontend:dev 5/6] RUN chmod +x /docker-entrypoint.d/50-insert-runtime-variables.sh                                                       0.0s
 => ERROR [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/frontend:dev 6/6] COPY build /usr/share/nginx/html                                                                                0.0s
------
 > [gitlab.zbmed.de:5050/km/nfdi4health/annotation-workbench/frontend:dev 6/6] COPY build /usr/share/nginx/html:
------
failed to solve: failed to compute cache key: failed to calculate checksum of ref LRU2:AC73:O6ET:327Y:2BOT:WPUG:WB6N:2IN3:IKBI:JXD6:XMHE:WGG3::qvacx99mpnh2j2xdvg00i5hvv: "/build": not found
[~/projects/mlstr/metadata-annotation-workbench] main

Here are more info:

Python 3.9.2
Docker version 27.2.1, build 9e34c9b
NPM: v18.17.1 (lts/hydrogen)

Thanks for the great effort building this, I would love to learn more about and try it locally.
Any help is greatly appreciated.

Thanks for reporting. The projects moved and I updated the image paths accordingly. This should fix the issue with the dev setup.
Before doing the Docker build, a local build step is needed. I updated the README. This should fix the production setup.

Hi,
I still have access issue for creating the container:
>docker compose --env-file dev.env up prediction
[+] Running 0/1
 ⠿ prediction Error                                                                                0.5s
Error response from daemon: Head "https://ghcr.io/v2/nfdi4health/workbench-ai-
model/backend/manifests/e57e38ff84b682be298d936ab7338f0fb9502825": denied: denied

Are there also any tutrial or documentation on how to use the tool to get the a beginner start?
Best,

You can find a tutorial/documentation at https://mda.nfdi4health.de/tutorial. It's not quite up to date, but it's a good place to start.


Wakeup may hang for 20 seconds

The wakeup mechanism in UCX may hang up to 20 seconds when using the progress thread, and thus appear that there was a deadlock in the application, for example when running:
python python/ucxx/benchmarks/send_recv.py -s 0 -d 1 -o cupy -n<PHONE_NUMBER> -l ucxx-async

There are currently two ways to prevent the hang:

Specify --progress-mode thread-polling to the benchmark above (or UCXPY_PROGRESS_MODE=thread-polling for other applications); or
Specifying a low keep alive interval: UCX_TCP_KEEPINTVL=1ms UCX_KEEPALIVE_INTERVAL=1ms.

Ideally we wouldn't need either one of them. Using thread-polling mode means the progress thread will run at 100% CPU all the time, whereas a low keep alive interval may be the source of other issues, perhaps even causing premature timeouts.

This also doesn't happen with UCX 1.14+ and protov2 enable (UCX_PROTO_ENABLE=y), protov2 is expected to become default in UCX 1.16.


Effect of larger sample queue?

Looking at chapter 7 some more,
you have two examples for breakout (breakout-small, and breakout), which have different sample queue sizes. What should we expect from a larger queue? Slower/faster convergence, a stable/smooth grow in reward value, or is it just something of a trial and error to see what works?
One more question, are these Atari games (pong, space-invaders, and breakout) all deterministic? So all this training would be useless on a game with a random seed. Such as the ball in another starting position, paddle in a different starting position, invaders moving in a different direction,?
Thanks,
Chris

Breakout environment is there for experimentation with params and training on more complex environment rather than pong. But those params are not guaranteed to convergence (even pong is diverges in 5-10% of seeds). From my personal tests, it takes several days of training to get policy with average mean score of 100. Get 800 requires a week and tons of tweaking.
Non-deterministic nature of the game should be fine for good policy, but getting such good policy might be another story.

Oh wow thanks Shmuma, thanks for the info. You must have put in a lot of time to find the value you did.


Suggestion: Move player storage into GuildData

I think maybe keeping it in GuildData allows for more direct storage of the player
🤷♂ let me know what you think

good idea! +1


Updated rainloop Dockerfile to build the latest stable release

Hi, thanks for the patch. I like the idea of building from a release tarball instead of cloning a random commit from git.
However, if possible I'd prefer to download a fixed version of rainloop, instead of just "latest".
If rainloop doesn't provide tarballs for specific versions I'll probably still merge this PR, because at least it will be not broken right now (I hope you tested this?), but then it seems just a matter of time until something will break again.

I couldn't find a download link for a specific version, so did latest instead.
I have tested this, and it seems to work well (I can log in, send and receive mail)

Ok then, thanks for fixing this! :)

No worries.
For reference, this should also fix #26, #24, #20.


unknow specific language for codeblock will cause format error on chirpy 6.2.3

Checklist

[X] I have read and accepted the contributing guidelines.

Is your feature request related to a problem? Please describe
I found this error when i try to use assmembly codeblock in typora and paste my markdown into chirpy(6.2.3).
    ```assembly
    CODE HERE
    ```
    {: file="LOCATION"}

after build in github pages，i found this block lose it's div class
for a simple c codeblock ,it will be wrapped in

but assembly codeblock only build with a

...

here is the build pages

Describe the solution you'd like
i wonder maybe there is a way could detect this? just auto remove the unsupported language or wrapped it correctly.
Describe alternatives you've considered
No response
Additional context
No response

Duplicate of #1244

Duplicate of #1244

thx,I didn't find that in issue,maybe the keyword is not correct :P


does not support the '<=' operator"

invalid sign operator "=<"

It is not a PocketBase issue. If you are sending the request manually make sure that the query parameters are properly URL encoded (the SDKs do this behind the scenes).

i make a mistake.  =< should replace with <=


Hi, Thank you for great work. Do you know how to support animated 3D objects on Android? Your answer really appriciated. Thank you.

Hello, I'm not familiar with this part (I simply updated the plugin that was created by @cariuslars so that it continues to work, notably on FlutterFlow.)
The original plugin is no longer up to date, but I believe the issue has been mentioned several times :
https://github.com/CariusLars/ar_flutter_plugin/issues?q=is%3Aissue+is%3Aopen+animation+

Hi, thank you:)


funds on hold forever

Today i got this error until a shutdown the bot and restart it again.
2017-06-08 16:01:25 - funds on hold after cancel, waiting 5s
2017-06-08 16:01:23  2424.74 BTC-EUR    0.0%        3    --       -0.1881   0.0500  selling  0.29642 BTC  297.46 EUR    +0.6%   +5.0%
2017-06-08 16:01:31 - funds on hold after cancel, waiting 5s

2017-06-08 16:01:37 - funds on hold after cancel, waiting 5s

2017-06-08 16:01:43 - funds on hold after cancel, waiting 5s

2017-06-08 16:01:48 - funds on hold after cancel, waiting 5s
2017-06-08 16:01:50  2424.71 BTC-EUR    0.0%        5    --       -0.1882   0.0500  selling  0.29642 BTC  297.46 EUR    +0.6%   +5.0%
2017-06-08 16:01:55 - funds on hold after cancel, waiting 5s
2017-06-08 16:01:50  2424.71 BTC-EUR    0.0%        5    --       -0.1882   0.0500  selling  0.29642 BTC  297.46 EUR    +0.6%   +5.0%
2017-06-08 16:02:00 - funds on hold after cancel, waiting 5s

2017-06-08 16:02:06 - funds on hold after cancel, waiting 5s

2017-06-08 16:02:12 - funds on hold after cancel, waiting 5s

2017-06-08 16:02:17 - funds on hold after cancel, waiting 5s

2017-06-08 16:02:23 - funds on hold after cancel, waiting 5s
2017-06-08 16:02:12  2424.99 BTC-EUR    0.0%        5    --       -0.1866   0.0500  selling  0.29642 BTC  297.46 EUR    +0.6%   +5.0%
2017-06-08 16:02:29 - funds on hold after cancel, waiting 5s

2017-06-08 16:02:34 - funds on hold after cancel, waiting 5s

2017-06-08 16:02:41 - funds on hold after cancel, waiting 5s

2017-06-08 16:02:46 - funds on hold after cancel, waiting 5s

2017-06-08 16:02:52 - funds on hold after cancel, waiting 5s
2017-06-08 16:02:36  2433.26 BTC-EUR   +0.3%        6    --       -0.1385   0.0500  selling  0.29642 BTC  297.46 EUR    +0.8%   +4.9%
2017-06-08 16:02:58 - funds on hold after cancel, waiting 5s

2017-06-08 16:03:04 - funds on hold after cancel, waiting 5s

2017-06-08 16:03:10 - funds on hold after cancel, waiting 5s

2017-06-08 16:03:15 - funds on hold after cancel, waiting 5s

2017-06-08 16:03:22 - funds on hold after cancel, waiting 5s
2017-06-08 16:03:15  2430.17 BTC-EUR   +0.2%        7    --       -0.1565   0.0500  selling  0.29642 BTC  297.46 EUR    +0.7%   +4.9%
2017-06-08 16:03:28 - funds on hold after cancel, waiting 5s

I "think" I fixed this with 17506db8ad172da5faaf25ece6214ab6c2ebad6b , it would happen after a partial fill. Another reason this might happen is if you use --sell_pct=100, since GDAX holds a little extra than the actual order total. --sell_pct=99 would fix it.


When more than 2 users are logged in

Description
When there are more than 2 users logged in and are making a request to server at the same time, I get an error
error: Sending 500 ("Server Error") response:
Error [ERR_HTTP_HEADERS_SENT]: Cannot set headers after they are sent to the client
at new NodeError (node:internal/errors:405:5)
at ServerResponse.setHeader (node:_http_outgoing:652:11)
at ServerResponse.header (/Users/DavidMendesDaSilva/Sites/h2pro-webapp/node_modules/express/lib/response.js:776:10)
at ServerResponse.send (/Users/DavidMendesDaSilva/Sites/h2pro-webapp/node_modules/express/lib/response.js:170:12)
at ServerResponse.json (/Users/DavidMendesDaSilva/Sites/h2pro-webapp/node_modules/express/lib/response.js:267:15)
at hook.render (/Users/DavidMendesDaSilva/Sites/h2pro-webapp/node_modules/inertia-sails/private/inertia-middleware.js:52:20)
at Object.fn (/Users/DavidMendesDaSilva/Sites/h2pro-webapp/api/controllers/order/view-orders.js:168:26)
at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
...
Current behavior
only one user request is getting completed and the other one normally gets the error and crashes.
...
Expected behavior
Both users should be able to make requests at the same time
...

Replace space in square bracket with x e.g [x]

[x] I have checked and this issue does not exist

Thanks @davidtwin for submitting this issue, I'll make a reproduction and see why this is an issue.

Yeah this seems to be the offending code in inertia-sails

Can you share what is after the if and else statement? Are you calling the next() method? If so use res.end() instead

Hey @davidtwin, I've solved this issue and the changes will be a 0.2.0 release of inertia-sails.

Hey @davidtwin here is the guide for when you want to upgrade, let me know if this fixes the issue for you too.
https://blog.sailscasts.com/boring-stack-v-0-2-0

Closing this issue as this has been resolved by #86 and #87


Fixup CI workflows

Fixup CI workflows a bit to optimize:

[ ] Move PR build out of main workflow that is publishing to NuGet each CI
[ ] Add env flags to remove unnecessary spew for CI
[ ] Ignore changes to .git files

Thanks Tim! Merged 👍


Style removal loses properties

I've found that the optimization process has some issues when converting the style. We've experienced this issue in a couple of places.
Here's an example: http://quickgit.kde.org/?p=plasma-framework.git&a=blob&h=29e74c0f31838fc123fc4a30eddd9d4d7acd61b1&f=src%2Fdesktoptheme%2Fbreeze%2Fwidgets%2Ftasks.svgz&o=plain
When optimizing this one, some components will apply the following change:
-    <g
-       id="west-hover-center"
-       transform="matrix(1.0370294,0,0,-0.84615385,70.814663,74.469231)">
-      <rect
-         y="4.0999999"
-         x="5"
-         height="13"
-         width="27"
-         id="rect4957-0-5-7-4-4"
-         style="opacity:0.3;fill:currentColor;fill-opacity:1;stroke:none"
-         class="ColorScheme-Highlight" />
-    </g>

into
+        <g id="west-hover-center">
+            <path id="rect4957-0-5-7-4-4" class="ColorScheme-Highlight" opacity=".3" fill="currentColor" d="M76 79h28V68H76z"/>
+        </g>

With these changes, we lose the fill: currentColor property, leaving the element color-less.
Passing --disable=convertStyleToAttrs fixes the issue.

I see fill="currentColor" (right after opacity=".3") in your example as well as in the test file.

You are right, my assessment was wrong. Nevertheless, the representation is still different:

Well, SVGO doesn't support embedded styles in SVG for now.

v0.6.2 with the fix is out.


the player cant play rtsp stream supported by ffserver?

In my case, I run a ffserver which support a rtsp stream,  the url is rtsp://<IP_ADDRESS>:5454/live
use ffplay, it can play, but use the locomote player,  it cant play,  the log as below, it like the rtsp port not send to server in the setup message
RTSP OUT:
RTSP IN:
RTSPClient: STATE_OPTIONS
RTSP OUT:
RTSP IN:
RTSPClient: STATE_DESCRIBE
v=0
o=- 0 0 IN IP4 <IP_ADDRESS>
s=vm name
c=IN IP4 <IP_ADDRESS>
t=0 0
a=tool:libavformat 56.40.101
m=video 0 RTP/AVP 96
b=AS:1000
a=rtpmap:96 H264/90000
a=fmtp:96 packetization-mode=1; sprop-parameter-sets=Z/QAH5GWQBQBboQAAAMABAAAAwDCPGDJIA==,aOvMRIRA; profile-level-id=F4001F
a=control:streamid=0
Ignored unknown SDP directive: c=IN IP4 <IP_ADDRESS>
Ignored unknown SDP directive: b=AS:1000
SDP contained 1 track(s). Calling SETUP for each.
RTSPClient: STATE_SETUP
undefined
Setting up track: rtsp://<IP_ADDRESS>/screen/streamid=0
RTSP OUT:
RTSP IN:
RTSPClient: STATE_SETUP
RTP/AVP/TCP;interleaved=0-1
RTSP stream closed

I might have fixed that already (if it's same as Live555 issue), pull request awaits merging. You can test it from my fork at   https://github.com/insonifi/locomote-video-player


Schema DDL: Add ability to generate Postgres DDL

Migrated from https://github.com/snowplow/schema-ddl/issues/26

Migrated to https://github.com/snowplow-incubator/schema-ddl/issues/38


Don't ignore existing pip packages on linux

When we constantly reinstall pip packages on ubuntu, we run into https://github.com/servo/servo/issues/20774#issuecomment-388512785 which breaks every linux build machine.

This change is

@bors-servo r+

:pushpin: Commit bdcfd98 has been approved by edunham

@bors-servo r-
This shouldn't be necessary.


[fix][build] Eliminate SuppressWarnings，and reduce member variable storage space

Motivation

The module[broker] requires Java version 17, so I think some new features of java can be used to eliminate some unnecessary SuppressWarnings
Modify some judgment object != null,  because creating a new object, such as new ConcurrentHashMap, this member variable will not be null, use !isEmpty to replace != null
Some member variables can be defined in the constructor to reduce references held by object member variables and reduce object space occupation， such as the class TransactionMetadataStoreService

Modifications

BookKeeperClientFactoryImpl
ManagedLedgerClientFactory
PulsarService
TransactionMetadataStoreService
EnsemblePlacementPolicyConfig

Verifying this change

[x] Make sure that the change passes the CI checks.

(Please pick either of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added integration tests for end-to-end deployment with large payloads (10MB)
Extended integration test for recovery after broker failure

Does this pull request potentially affect one of the following parts:
If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[x] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository:

@momo-jun Please help with code review if possible, thank you


⚠ BCR Bot has degraded performance

In 2a079f6, BCR Bot (https://bcr.alternativasinteligentes.com/BCRApi/web/chatbot) experienced degraded performance:

HTTP code: 200
Response time: 998 ms

Resolved: BCR Bot performance has improved in 6653a41.