travis: fix DCO validation for merges

merged PRs do not have DCO, so do not check for them
https://github.com/opencontainers/specs/issues/169
Signed-off-by: Vincent Batts<EMAIL_ADDRESS>

@laijs per https://github.com/opencontainers/specs/pull/167/files#r39153774 see the cleanup commit.

LGTM

@mrunalp review?

@vbatts Looking

LGTM


(#236) Ensure Batteries.Init() is only called once

Hi @adrianhall, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (adrianha). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Re: the PCLStorage.  If I do the System.IO, I have to add a Platform specific IO package in.  I was going along the same lines.  However, SQLiteStore is a PCL, so it doesn't get the entire System.IO - most notably, File.* are not there, so I can't do File.Create or File.Exists.
The way around this is to add another method (EnsureDatabaseExists(fullpathname)) to the IPlatform.cs and do it there.  That would work around the issue.  I'll discuss internally on Monday and see which direction we want to go.  (@dhei - if you have comment over the weekend, please add it)

I would most likely just want to validate support from @dplaisted on it. It think that going against System.IO and controlling it is worth it as taking a dependency on PLCStorage brings in even more DLLs, I think it would be better to bring in platform specific DLLs to access System.IO and actually for 4.0 they will go away with .NET Standard

@adrianhall: I agree with @jamesmontemagno. I think it's easier to manage platform specific IO package than adding a new PCL package. More importantly, PCLStorage takes dependency on Microsoft.Bcl >= 1.1.6 but we're planning to completely get rid of Microsoft.Bcl when we moves to .NET Standard 1.3.

Per the review comments, I've added a per-platform method called "EnsureFileExists(path)".  On Windows, it uses Windows.Storage (since System.IO.File does not exist - go figure), and on iOS/Android, it uses System.IO.File.  I've also removed the dependency on PCLStorage from the SQLiteStore.
This does increase our public API footprint on the MobileServiceClient, but I think both pieces are likely to be useful for anyone implementing a local store provider, so it's a good addition.
Comments appreciated.

public class CurrentPlatform : IPlatform

Please add EnsureFileExists() to this class.
Error CS0535 'CurrentPlatform' does not implement interface member 'IPlatform.EnsureFileExists(string)' Microsoft.WindowsAzure.Mobile.Ext.Android

Refers to: src/Microsoft.WindowsAzure.MobileServices.Android/Platform/CurrentPlatform.cs:12 in b90a6d0. [](commit_id = b90a6d07b3e63d78674b0ab2b0280f8a463c9b8d, deletion_comment = False)


Add one-click shell script

Add one-click shell script and edit README.md, README_CN.md
I think this is a temporary script because there are many things that are uncertain about Easytier right now.
But it works now (((((

已修改

稍晚点的 release 版本我想稍微改变一下压缩包的命名格式，让它变的更有好点，这个脚本晚点 review

确实感觉得改一下，你发release之后我再适配一下新的命名格式


Error return optional from Php

Help me on this issue: https://stackoverflow.com/questions/44588883/return-json-from-php-to-swift-alamofire?noredirect=1#comment76165486_44588883

I supplied an answer, but that question really has nothing to do with Alamofire.


Added store argument to search

Summary of changes (Bug fix, feature, docs update, ...)

Added store argument to the function search() inside _datamanager.py to show the store-specific results.

Please check if the PR fulfills these requirements

[x] Closes #754  (Replace xxxx with the Github issue number)
[x] All code checks passing - black --line-length 110 run over code, mypy and flake8 not showing any errors

Why am I assigned to this?

Ah sorry, I was worried I'd forgotten a conversation or something!

No No, its good that you highlighted that, I must have mistakenly put you in assignee instead of reviewer. Apologies for the confusion.
From: Gareth Jones @.>
Date: Wednesday, 10 January 2024 at 13:48
To: openghg/openghg @.>
Cc: Prasad Sutar @.>, Assign @.>
Subject: Re: [openghg/openghg] Added store argument to search (PR #859)
Ah sorry, I was worried I'd forgotten a conversation or something!
—
Reply to this email directly, view it on GitHubhttps://github.com/openghg/openghg/pull/859#issuecomment-1884883270, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ASB2CE2TWY5GAC2EQICMROLYN2LZPAVCNFSM6AAAAABA4SVHNCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQOBUHA4DGMRXGA.
You are receiving this because you were assigned.Message ID: @.***>


[CARBONDATA-1677]  [BugFix] Resolved bug for presto not showing table not found exception when table is dropped from carbon

https://issues.apache.org/jira/browse/CARBONDATA-1677
Resolved bug for presto not showing table not found exception when table is dropped from carbon because table key still exists in presto carboncache model

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/885/

@chenliang

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1500/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/930/

Retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/935/

@chenliang can you please review this pr?

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/938/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1545/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1550/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1553/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/970/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/971/

retest this please

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/980/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1584/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1585/

SDV Build Success , Please check CI http://<IP_ADDRESS>:8080/job/ApacheSDVTests/1590/

@anubhav100  this pr is same as 1486 ?

@chenliang613 my pr 1486 has resolved both the issues CARBONDATA-1694,CARBONDATA-1677 I cut my branch CARBONDATA-1694 from this
branch to make sure that if this pr get merged first then there is no merge conflict.now both issue  CARBONDATA-1694,CARBONDATA-1677 are resolved because my pr 1486 is merged on master i am closing this pr and marking the jira issues as resolved
thanks a lot


Hmusum/do not use default temp dir for rank setup validation

Good point, I will look into that

Superseded by https://github.com/vespa-engine/vespa/pull/12551


Two column layout for showcase

@jdorfman deployed here: https://bscdnpre.herokuapp.com/showcase/
added:

two col layout for showcase
linked showcase imgs
reactivated showcase nav
moved bootstrap4 alpha link to left side of nav
added docker-compose.yml supporting the heroku/nodejs docker image

usage

# setup, test, run
docker-compose run --rm web npm install && \
    docker-compose run --rm web npm test && \
    docker-compose up

Nice.  Please add thumbnail to the img element class
class="img-responsive thumbnail"

@jdorfman done, pushing to https://bscdnpre.herokuapp.com/showcase/ now

perfect :thumbsup:


Fix URL to web-push GitHub repo

Hey @rossta, I just found a tiny typo in the web-push gem link in the "Sending Web Push Notifications from Rails" article.
This PR removes the additional dash in the web-push maintainer’s GitHub username (should be pushpad instead of push-pad).
Also, thanks for the great article – it saved me a lot of time! :pray:

Thank you @paulgoetze! 💜


[Bug] Inertia Route macro does not work with Route Parameters

In web.php I have:
Route::inertia('{any}', 'MyPage');

When calling http://foo.test/ this works fine. MyPage will be loaded. If I call http://foo.test/bar an exception is raised:
ErrorException thrown with message "array_merge(): Expected parameter 2 to be an array, string given"

Stacktrace:
#59 ErrorException in /Users/marvin/Sites/laravel/vendor/inertiajs/inertia-laravel/src/ResponseFactory.php:53
#58 array_merge in /Users/marvin/Sites/laravel/vendor/inertiajs/inertia-laravel/src/ResponseFactory.php:53
#57 Inertia\ResponseFactory:render in /Users/marvin/Sites/laravel/vendor/laravel/framework/src/Illuminate/Support/Facades/Facade.php:237
#56 Illuminate\Support\Facades\Facade:__callStatic in /Users/marvin/Sites/laravel/vendor/inertiajs/inertia-laravel/src/Controller.php:9
#55 Inertia\Controller:__invoke in /Users/marvin/Sites/laravel/vendor/laravel/framework/src/Illuminate/Routing/ControllerDispatcher.php:48
#54 Illuminate\Routing\ControllerDispatcher:dispatch in /Users/marvin/Sites/laravel/vendor/laravel/framework/src/Illuminate/Routing/Route.php:219

Debugging
When I dump the arguments of the render function in inertiajs/inertia-laravel/src/ResponseFactory.php. I get the following:
$component is bar and $props is MyPage.
It seems like there are Issues with the route bindings in the Inertia Controller when the route contains a route parameter. When I change the Inertia Controller to be like this:
public function __invoke($component, $props, $x)
    {
        dd($component, $props, $x);
        return Inertia::render($component, $props);
    }

This will dump $component is bar and $props is MyPage and $x is an empty array.
So the any parameter in the route is mapped to $component. Which is obviously wrong.
Workaround
A possible solution for this might be the following:
public function __invoke()
    {
        return Inertia::render(request('component'), request('props'));
    }

This looks like a bug to me. @peterfox any chance you could do a follow up PR that fixes this? 😇

@sebastiandedeyne sure I'll try and fit in some time to take a look

It's really weird this behaviour seems to happen with Macros but doesn't happen with the router view method that does literally the same thing.
The following code for the controller I believe fixes the issue. I have a feeling this might be related to the Laravel version as well, it looks like the controller for Route::view has changed in L6, it might just be a cosmetic reason for changing though. @marvinrabe are you using Laravel 6?
<?php

namespace Inertia;

use Illuminate\Http\Request;

class Controller
{
    public function __invoke(Request $request)
    {
        return Inertia::render(
            $request->route()->defaults['component'],
            $request->route()->defaults['props']
        );
    }
}

@marvinrabe No, I am still on Laravel 5.8. Haven't tested it with Laravel 6 yet.

@peterfox It seems like injecting parameters in controller methods is always done by order. See the documentation:
Route parameters are injected into route callbacks / controllers based on their order - the names of the callback / controller arguments do not matter.

Up until now I did not know this. I find this quite unintuitive. I opened a feature proposal for changing this behaviour.

@marvinrabe

I find this quite unintuitive

It allows you to do this :
// matches 'GET /1/my-article-slug'
Route::get('/{writer}/{article_slug}', function(Writer $writer, Article $article) {
    // Here I can use $article instead of $article_slug/$articleSlug
});


UB.3.1  Audio communication during Game: As a player I want to join a voice call to hear the other players, to make communication even easier.

Acceptance Criteria

[ ] A player should hear the other players that have agreed to share their audio.
[ ] A user should be able to share their own audio with players after they agreed to share

Task List

[x] sopra-fs22-group-11/SoPra22-group11-TheGame-Server#71
[x] sopra-fs22-group-11/SoPra22-group11-TheGame-Server#72
[x] sopra-fs22-group-11/SoPra22-group11-TheGame-Client#25
[x] sopra-fs22-group-11/SoPra22-group11-TheGame-Client#57
[x] sopra-fs22-group-11/SoPra22-group11-TheGame-Client#56
[x] #141

@Desteb code and close

2b5f2331152bc3f399d0eca7c61a4d004e6aba52


[question]

Question
Describe the Question
I'm learning the move programming language and I have a question.
I initialized a Move project, when I configure the dependency package of Move.toml, I check the official documentation is configured with remote repository dependencies, the execution of the command run is slow, how do I configure the local dependencies?
And what properties can I configure for Move.toml?
Related Code

Additional Context

Can trying to set up a git proxy be solved?


Refactor model training loop into config class

…onfiguration
Short description of why the PR is needed and how it satisfies those requirements, in sentence form.
(Delete unused sections)
Added public API:

symbol (e.g. vcm.my_function) or script and optional description of changes or why they are needed
Can group multiple related symbols on a single bullet

Refactored public API:

Bulleted list of removed or refactored symbols, such as changes to name, type, behavior, argument, etc. Be cautious about doing these and discuss with team more broadly.

Significant internal changes:

Bulleted list of changes to non-public API

Requirement changes:

Bulleted list, if relevant, of any changes to setup.py, requirement.txt, environment.yml, etc

[ ] Ran make lock_deps/lock_pip following these instructions

[ ] Add PR review with license info for any additions to constraints.txt
(example)

[ ] Tests added

Resolves # [JIRA-TAG]
(Delete this for the commit message)
You are encouraged to check the PR against the Code Review Checklist.

Still need to refactor the precipitative model to use this, and fix mypy.


Make realm optional for single VO instances in invenio_oauthclient/contrib/keycloak

Motivation
IAM services like indigo-iam which are set up for only one VO do not work with realms, however, the invenio-oauthclient keycloak contrib requires a realm per default and doesn't allow it to be omitted:
https://github.com/inveniosoftware/invenio-oauthclient/blob/3cb6e23f962dc2a0bf6996fc1391f911b5beb7ad/invenio_oauthclient/contrib/keycloak/settings.py#L32-L48
This issue comes out of an effort to implement a generic third-party IAM service configuration option in REANA described here: https://github.com/reanahub/reana-server/issues/513
Proposal
Change the keycloak contrib in a way that allows omitting the declaration of a realm.
I'm already trying to solve this on my own but would appreciate some help. As REANA is using v1.5.4 this is the one I'm developing with atm.

@ntarocco / @zzacharo maybe you can have a look at this? 😃

Sorry for the late answer, I have asked @max-moser to have a look give that he created the keycloak integration.
Thanks a lot for your PR!


feat: integrate SPL deposits

Description

parse inbound deposit spl / deposit spl and call
e2e tests
further support in e2e tests for spl tokens

Part of https://github.com/zeta-chain/node/issues/2938
How Has This Been Tested?

[ ] Tested CCTX in localnet
[ ] Tested in development environment
[ ] Go unit tests
[ ] Go integration tests
[ ] Tested via GitHub Actions

Summary by CodeRabbit
Release Notes

New Features

Added support for whitelisting SPL tokens on Solana.
Integrated deposit functionality for SPL tokens.
Enhanced configuration options for Solana and zEVM chains.

Bug Fixes

Improved error handling for various operations, including token deposits and whitelisting.

Tests

Introduced new end-to-end tests for SPL token deposits and whitelisting.
Expanded test coverage for Solana-related functionalities.

Documentation

Updated changelog to reflect new features, improvements, and fixes across versions.

@gartnera i used consensus breaking ack label because of failing consensus tests on merge, since the change in foreign_coins.go is making a difference in gas calculation
please let me know if im using this correctly

Yes


Fix unresolved symbols when enabling both tracing and events

Fixes #53936
Completes some missing tracing support related to events.

@laurenmurphyx64  @stephanosio  FYI


fix the ansible version for the stable-3.0 branch

In my env, only ansible version 2.3.1 and 2.3.2 can run the stable-3.0 branch ,but ansible 2.2

jenkins test luminous-ansible2.4-docker_cluster_collocation


invalid varint test case

Sorry for opening a PR, I will mark it as draft before I will add a solution. I found an unexpected behaviour and as a first step made a minimal test to reproduce it.
The issue is in a decoder:
failed to search: Decode(DecodeError { description: "invalid varint", stack: [("Count", "count"), ("Counts", "counts"), ("SearchResponse", "results")] })

By some reason, it fails to decode uint64 in a message.

I see the problem. will fix it later today/tomorrow

ntex-grpc 0.6.4 is released. could you test it.

@fafhrd91 thanks a lot for such a quick response and fix!
test search_count ... ok

Working now 💟


Named access nested context revision

This updates the "named access on the window object" feature to only
return the nested browsing context (the contentWindow) when the
browsing context name is set. Since the browsing context name is not set
by the id property, the iframe element should be returned when the id
is the name being used.
In summary:

id: return the element.
name: return the WindowProxy.

This is based on findings from a Firefox issue:
https://bugzilla.mozilla.org/show_bug.cgi?id=1281604
Closes #1532

Looks like this breaks a different test, investigating...

Yeah, another test was depending on id being used in this way, changing it to name fixes the test.

Great stuff! Merged as 7182006a4c8cdffb45f7c30456430aab296a9768. Thanks very much!!


Add csv mode and delay parameter to pcm-iio

Currently pcm-iio is able only to send output to cout and the output becomes quite unreadable in case when you want to see how some metric is changed in time. -csv option was added to make handle this case.
Also now there is a hardcoded measurement interval. The proposed change allows to pass the interval in milliseconds to the utility.

thanks for the patch!

Globally commenting on the millisecond change:
As far as I know all pcm programs use a floating point number for delay, so if you pass it 0.001 it should use 1 ms as delay. Not sure if that worked though.
The option to have CSV output is much appreciated, but I will let others comment on how it is done.

That is right (delay in seconds as a floating point number)

That is right (delay in seconds as a floating point number)

Changed to floating point.

@bigstream-dmytro  thanks for the revised patches! @ilia-kurakin @ogbrugge thanks for the reviews.


Handle vec3 colors correctly in custom shaders

Don't merge this until https://github.com/CesiumGS/cesium/pull/9960 is merged first.
This continues the saga of #9955 and #9957, looks like I missed a spot in CustomShaderPipelineStage -- looks like I wasn't using ModelExperimentalUtility.getAttributeInfo which computes the correct GLSL type of vec4 in this case.
@lilleyse could you review?
Use this local Sandcastle -- last item in the dropdown. I updated the shader to use fsInput.attributes.color_0, now it doesn't crash like in previous branches.

Thanks for the pull request @ptrgags!

:heavy_check_mark: Signed CLA found.

Reviewers, don't forget to make sure that:

[ ] Cesium Viewer works.
[ ] Works in 2D/CV.

@lilleyse synced with main

@lilleyse reset the branch to untangle the confusing git history

Thanks @ptrgags


Support more transports

Conflicting files

:trollface:

Codecov Report

Merging #73 (70e3cf3) into main (5997f98) will decrease coverage by 0.84%.
The diff coverage is 54.54%.

@@            Coverage Diff             @@
##             main      #73      +/-   ##
==========================================
- Coverage   69.04%   68.20%   -0.85%
==========================================
  Files         105      109       +4
  Lines        2775     2947     +172
==========================================
+ Hits         1916     2010      +94
- Misses        590      643      +53
- Partials      269      294      +25

Impacted Files
Coverage Δ

internal/proto/codec/codec.go
77.77% <ø> (ø)

internal/proto/codec/full.go
61.70% <0.00%> (ø)

transport/mtproxy.go
0.00% <0.00%> (ø)

internal/proto/codec/obfuscated2.go
52.17% <52.17%> (ø)

internal/proto/codec/padded_intermediate.go
53.12% <53.12%> (ø)

internal/proto/codec/abridged.go
54.00% <54.00%> (ø)

internal/proto/codec/intermediate.go
51.42% <66.66%> (-1.52%)
:arrow_down:

internal/crypto/keys.go
95.16% <100.00%> (+0.33%)
:arrow_up:

internal/proto/codec/errors.go
40.00% <100.00%> (+15.00%)
:arrow_up:

transport/transport.go
85.71% <100.00%> (+3.89%)
:arrow_up:

... and 4 more

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 5997f98...e20ff54. Read the comment docs.


Couldn´t fetch data

I have:

[x] searched open and closed issues for duplicates
[x] provided a reproducible result if applicable

Bug description
It doesnt show the news but following text
"android.sax.BadxmlExeption: Line 2: Root element name does not match. Expected rss w3.org/2005/Atom:feed"
Steps to reproduce

fdroid version
open app
wait  while "one second please fetch data"
error is shown.

Actual result:
error page "android.sax.BadxmlExeption: Line 2: Root element name does not match. Expected rss w3.org/2005/Atom:feed"
Expected result:
showing news
Device info

Device: Razer Phone 1
Android version: 11
App version: 2.0

Sorry wrong github page


Windows (x64) compiled executable (tesseract.exe) doesn't work in any other windows machine (x64)

Environment

Tesseract Version: tesseract 5.0.0-alpha-375-g179c8
Platform: Windows Server 2008 R2, 64bit

Current Behavior:
I built tesseract executable from source using following this instruction

git clone https://github.com/tesseract-ocr/tesseract tesseract
Modify source code (this is to suit our application's purpose, certain that this is the root cause of entire problem)
cd tesseract
cppan
mkdir build && cd build
cmake ..
cmake --build . --config Release

After step 6, I see tesseract.exe under "tesseract\build\bin\Release" directory, and executing it without any issue (working perfectly fine).
However, if once I copy entire \Release directory to another Windows with x64 platform (Windows Server 2008, 2016, etc..)  It doesn't work anywhere else.  (see screenshots below)
I have installed "Microsoft Visual C++ 2015-2019 Redistributable (x64)", so dependency does not look like issue here.
Screenshot 1 (Installed runtime libraries)

Screenshot 2 (Message box pop-up on different Windows machine)

Screenshot 3 (Windows Event Viewer)

Screenshot 4 (Windows Event Viewer) - FYI, disk is not on network, it's under C:

Can anyone suggest me with a fix? is there anything I'm missing during compilation procedure?
Suggested Fix:

CMake builds are currently optimized for the build host architecture. So if you build on a PC with AVX2 hardware, the resulting binaries won't work (= will crash) on a PC without AVX2.

Builds using autoconf will work.

@stweil Thanks for your suggestions.
Regarding your recommendation CMakeLists.txt, I'm compiling without following section in CMakeLists.txt and it seems to work on other Windows machine.
When you mentioned build using autoconf, can you instruct me how to do this?  I couldn't find autoconf usage under Windows page.

With disabling optimization OCR process will be slow.
Autotools macros are common for linux and using them on Windows is very difficult.
Other option is to use cross-compiled windows build, but IMO it is not suitable for using tesseract API (library) on windows (VS Studio).

With disabling optimization OCR process will be slow.
Autotools macros are common for linux and using them on Windows is very difficult.
Other option is to use cross-compiled windows build, but IMO it is not suitable for using tesseract API (library) on windows (VS Studio).

Do you have any guess what performance degradation would be without optimization ?  (I will need to test performance anyways but was just curious if you can make any rough guess)
Using cross-compiled windows build would be a problem for me, because I needed to changed the source code.

With image from issue 263 I got this results (tesseract 5.0.0-alpha-447-g52cf, Intel Core i7-6600U 2,60GHz 2.8 GHz; 8 GB ram; Windows 64 bit) - 5 runs:
With autooptimization: 26.177767169865813
Without autooptimization: 50.47452997387488

Now you can turn off cmake autooptimization with:
cmake .. -DAUTO_OPTIMIZE=OFF

@zdenop, did you specify CMAKE_BUILD_TYPE for your builds?
If not: please try latest Git master. It should be significantly faster.

@stweil: I used cmake&ninja&clang and this combinations needs to specify CMAKE_BUILD_TYPE. So yes, I specify it (to Release).

@zdenop Hi, thanks for testing the performance with optimization on/off.

Update: I posted more details in #263

In your post, you mentioned

tessdata_best
tessdata_fast
tessdata

Does this mean if I use eng.traineddata from each git, OCR engine will perform differently based on trained data file?  In other words, if I swap exsiting eng.traineddata with the one from "tessdata_fast", would it increase performance?
After reading some doc, it looks like eng.traineddata has effect when using LSTM engine.  I don't explicitly specify engine mode when executing.  is LSTM used by default?

if I swap existing eng.traineddata with the one from "tessdata_fast", would it increase performance?

I am not sure how do you measure performance, but tessdata_fast model is faster version of tessdata_best. It has some limitations (e.g. you can do training only from best model, and best model provide better OCR result - at least in some cases).

is LSTM used by default?

>tesseract --help-oem
OCR Engine modes:
  0    Legacy engine only.
  1    Neural nets LSTM engine only.
  2    Legacy + LSTM engines.
  3    Default, based on what is available.

what is available => tessdata_best and tessdata_fast have only LSTM models (you will get error if you will use --oem 0: Failed loading language 'eng' Tesseract couldn't load any languages! Could not initialize tesseract), tessdata has also legacy model.

@zdenop  Thanks, that makes sense.
Quick question regarding compilation (will create a new ticket if needed)
I'm using cmake to compile (shown below), which generate 64-bit executable.  Is there any way to build 32-bit env executable?
:: Compile Tesseract
cd build\tesseract_src
cppan

cd ..
cmake tesseract_src
cmake --build . --config Release

IMO original topic is solved, so I will close this issue.
Issue tracker is for bug and not for support. Please use forum or better - read cppan doc. It is external project.

Got it.  Thanks for the help.


"end" at the end of file is not highlighted correctly

Consider the code
if true
end

(without a newline after end). Then end is highlighted as if it were a command name rather than a keyword; in Monokai, blue instead of red.
In other syntax highlighters (try the same code highlighted like MATLAB), this behaves as expected.

Ah, thank you for pointing this out. I use ST with the "ensure_newline_at_eof_on_save" setting enabled, so I hadn't noticed this. The syntax almost exclusively looks for \n to end scopes, rather than $ which is all that would be present at the end of the file... I'll take a look

Thanks! I have newline-on-save as well, but it was a bit annoying before
saving.
On Sun, 17 Jun 2018, 15:19 Violet<EMAIL_ADDRESS>wrote:

Closed #15 https://github.com/Phidica/sublime-fish/issues/15 via 29382f5
https://github.com/Phidica/sublime-fish/commit/29382f5022d202e83d3c877cf41893cc6a0a6324
.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/Phidica/sublime-fish/issues/15#event-1684977340, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AIAy2PqOjeKOM5EV_1aXXdutZL_LEJ9Uks5t9ldzgaJpZM4UqcV7
.


Add initial infrastructure for optionally accepting TLS connections

If the environment gives us the paths to the certificate chain and private key
then use TLS for all accepted TCP connections. Otherwise, continue on using
plaintext for all accepted TCP connections. The default behavior--no TLS--isn't
changed.
Later we'll make this smarter by adding protocol detection so that when the TLS
configuration is available, we'll accept both TLS and non-TLS connections.
Signed-off-by: Brian Smith<EMAIL_ADDRESS>

See https://gist.github.com/briansmith/516bc672d63a058e4f01c241923c8e4b for the scripts I've used to test this manually. Note that the certificates must be ECDSA certificates so you can't just use any RSA certificates you may have laying around.

I minimizes the amount of squashing I did before submitting it just in case I need to backtrack a little. It may be tempting to review each individual commit in the PR. However, I rearranged everything right before submitting the PR so it's probably much easier to just review the combined change.

See https://gist.github.com/briansmith/516bc672d63a058e4f01c241923c8e4b for the scripts I've used to test this manually. Note that the certificates must be ECDSA certificates so you can't just use any RSA certificates you may have laying around.

I also forgot something important: If you test this in a web browser, you need to add a line to your /etc/hosts like <IP_ADDRESS> example.example.svc.cluster.local and then access the proxied server via https://example.example.svc.cluster.local:4143. In particular, you must use the domain name that is in the certificate since the code verifies that the right domain name is used in the SNI extension of the TLS handshake, and the browser sets the SNI extension in the TLS handshake to the domain name in the URL. So, if you try https://localhost:4143 for example, the name validation logic in tls/cert_resolver.rs will cancel the handshake.


ParsingHelpers.GetHeaderSplit - Check if values is empty before parsing

Not calling GetHeaderSplitImplementation will skip the allocation of a yield-state-machine:
https://github.com/aspnet/HttpAbstractions/blob/ec176b2e40699c24cc1c0e3bde8a828e93fe67c7/src/Microsoft.AspNetCore.Http.Abstractions/Internal/ParsingHelpers.cs#L19-L23

Continue at https://github.com/aspnet/AspNetCore/issues/3737


Redundant / incorrect kernel module check

On a minimal system we recently stumbled over the following lines when building the initrd:
https://github.com/coreos/ignition-dracut/blob/390779da2c5b7595882a8426e2c9ec9c073c6da4/dracut/30ignition/module-setup.sh#L87-L108
There is one logical error in this code: It tries to include the qemu_fw_cfg module by checking whether the kernel has been built with that module. However if this is true it does not mean that this module is actually present - e.g. on openSUSE the kernel modules are spread over multiple packages to avoid installing unnecessary components.
However I'm also wondering something completely different: What is this snippet good for at all?
The ignition module already declares a depends() on qemu. When looking at the qemu module code we can see that it will - among other modules - do a hostonly='' instmods qemu_fw_cfg - so it will also include the module if it is there (but with correct checking).
Am I missing something, or can the listed lines just be removed as they are?

So the stance on this changed a bit over time. It used to be that we hard required qemu_fw_cfg, and so we would simply check if it was built-in and otherwise instmods -c (that's https://github.com/coreos/ignition-dracut/pull/28). Then it was changed to accommodate multi-arch, where it might not be supported as well (https://github.com/coreos/ignition-dracut/pull/49).
So I think you're right now that at this point this code isn't really useful. The original intent though was that on architectures which do support it, then we definitely do want to include the module because that's likely the only way for Ignition to get its config on QEMU (see https://github.com/coreos/ignition-dracut/pull/49#issuecomment-468773209). Just recently, the picture changed slightly with https://github.com/coreos/ignition/pull/905, but that only affects ppc64le and s390x; x86_64 and aarch64 on QEMU still only supports qemu_fw_cfg.
I think we could change this code to check for the architecture and make it a hard requirement (built-in or as a module) based on that.
Or for your use case are you not planning to support qemu_fw_cfg at all?

Thanks for the explanation, I didn't think about the case where the driver is part of the kernel.
So to repeat it in my own words:

The default behavior would be that the kernel will include the module if it is available. If it isn't, it will just skip it silently.
The 30ignition-dracut code enforces that the module will be included in any case if the system would be able to do so. If that fails, the initrd creation will fail with an error message.

The only scenario I can think of where this difference matters is when kernel was compiled with qemu_fw_cfg as a module, but the module is not installed (so exactly the case where we have seen this).

Or for your use case are you not planning to support qemu_fw_cfg at all?

That depends a bit - for our general purpose images we are also supporting qemu of course.
But when thinking about embedded devices for example you will want to keep to overhead minimal. Unused modules would be the first candidate to remove, so I think we have a different use case here...
Even if the system includes no platform specific driver it's still possible to configure it using ignition.config.url.

That depends a bit - for our general purpose images we are also supporting qemu of course.
But when thinking about embedded devices for example you will want to keep to overhead minimal. Unused modules would be the first candidate to remove, so I think we have a different use case here...

Thanks, I think that's a convincing argument. I opened https://github.com/coreos/ignition-dracut/pull/178.


Fix keyboard edits

This is a new version of PR #39 to update only the keyboard edits. It ignores the new issue of inconsistent casing while fixing the issue of unexpected capital letters and the issue with keyboard groups.
This closes issue #37, closes issue #41, and ignores issue #46.
All tests have been updated to allow the continuous integration to pass.
It's ready to merge!

Closing in favor of PR #39.

Closing in favor of PR #39.


Release CSS (not-minified) missing glyph codes

It seems the glyphs are missing from the CSS in the latest release. They are present in the minified file, though.
In consequence:

icons do not show up (of course), see for example issue #2502
some features are impossible to use because you can't click on the icon, for example column menus (see issue #2506)

The glyphs are missing also in the css generated by the Customizer

They look to be present in the tutorial.  So that suggests to me that you're having this issue with the bower install?  If so, can you confirm:

that you have looked at http://localhost:9003/docs/#/tutorial/116_fonts_and_installation, and that isn't the issue
what exact version bower has installed, and that it is the latest available

Thanks,

For #2502, the public tutorial does not have the icons - http://ui-grid.info/docs/#/tutorial/210_selection
Locally, the ui-grid-unstable.min.css works, the ui-grid-unstable.css does not.

OK, seeing that now.  I'll look shortly.

I think this is now resolved, I suspect it was just a build error, as I merged some other things (which also causes a rebuild) and it went away.  Can others check it is also fixed for them?

Using the latest unstable out on release works. Thanks.

Excellent, thanks.  Closing.

The NuGet package seems to have this problem too, thanks to the comment I grabbed the css from the release in the linked tutorial and fixed the url routes in the css.


Set @phoenix_routes  and unquote the opts

Adding to #199
I've tested it and it works, no more duplicate routes and no warning 😄
Let me know what you think, and if it's cool I can add some tests for the new code ...

I don't think @before_compile is needed, doesn't look like it does anything in this case 😄
Thanks for testing it, and fixing the missing opts! I'll setup a test in #199 that ensures the routes are indeed unique and then get it merged to master later today 🚀

Hmmmm strange I just double checked and you're right, the @before_compile is not needed, @phoenix_routes is already set. Wonder why it crashed for me before 🤷♂ 😄


Add v0.3.0, split into openff-nagl-base and openff-nagl

Checklist

[x] Used a personal fork of the feedstock to propose changes
[ ] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[ ] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

@conda-forge-admin, please rerender


Blank page after saving in laravel 5.2

i have blank page after saving user data, the data already saved or updated but give me blank page
this was the user model
/*******************************************************************/
use Illuminate\Foundation\Auth\User as Authenticatable;
use Zizaco\Entrust\Traits\EntrustUserTrait;
class User extends Authenticatable
{
use EntrustUserTrait;
........
}
/*******************************************************************/
i think that there was collision between user mode and (use EntrustUserTrait)
so what can i do to solve it.
thanks in advance

Did you check your laravel Log, if not please post the error message ,
and note that there is no an official release yet for a verision that support laravel 5.2,
You have to tweak the code adjust it to work laravel 5. if you can show me the error, i can help you to tweak it..
I have already working verision with Laravel 5.2 for entrust

I can confirm this error.
EntrustUserTrait::save() returns null, while the original Eloquent::save() return true.


Plant very early release notes for next release

Supersedes #795
In scope of https://www.pivotaltracker.com/story/show/154920627

[x] Split release notes in separate file - one per release
[x] Integrate to current master including https://github.com/aeternity/epoch/pull/795#issuecomment-369234528
[ ] Integrate https://github.com/aeternity/epoch/pull/795#issuecomment-369284096

PRed wrong branch.


shellcheck-sarif: 0.6.6 -> 0.7.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for shellcheck-sarif is: CLI tool to convert shellcheck diagnostics into SARIF
meta.homepage for shellcheck-sarif is: https://psastras.github.io/sarif-rs
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.7.0 with grep in /nix/store/k755z9rbdikzlb0ny5l0m81jxgygnwmq-shellcheck-sarif-0.7.0
found 0.7.0 in filename of file in /nix/store/k755z9rbdikzlb0ny5l0m81jxgygnwmq-shellcheck-sarif-0.7.0

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

shellcheck-sarif

Instructions to test this update (click to expand)

Either download from the cache:
nix-store -r /nix/store/k755z9rbdikzlb0ny5l0m81jxgygnwmq-shellcheck-sarif-0.7.0 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The nixpkgs-update cache is only trusted for this store-path realization.)
For the cached download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A shellcheck-sarif https://github.com/r-ryantm/nixpkgs/archive/eaab5628d924755311cdaa8f6773ebe18263f52b.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/eaab5628d924755311cdaa8f6773ebe18263f52b#shellcheck-sarif

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/k755z9rbdikzlb0ny5l0m81jxgygnwmq-shellcheck-sarif-0.7.0
ls -la /nix/store/k755z9rbdikzlb0ny5l0m81jxgygnwmq-shellcheck-sarif-0.7.0/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 1 package built:

shellcheck-sarif

Maintainer pings
cc @getchoo for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

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

@ofborg build shellcheck-sarif shellcheck-sarif.passthru.tests


Fixing typo on requirements section.

More info here: https://github.com/brocaar/loraserver/issues/416

Thanks!


Undefined variable: plugin_install_url in class-kirki-installer-section.php on line 98

Issue description:  Notice: Undefined variable: plugin_install_url in class-kirki-installer-section.php on line 98
Version used: 3.0.25
Hello. I make a recommendation to install the plug-in according to your instructions, but in Customizer on the recommendation button for installation, the message appears: Notice: Undefined variable: plugin_install_url in class-kirki-installer-section.php on line 98

Can you please try the version from the develop branch? I just pushed a commit that should fix this.

Thank you, the button appeared, but it does not work. The browser simply updates and nothing happens. The console shows that there is no link on the button.

Fixed & tested.

Aristath!!! Great work !!!


Fix gRPC examples that are failing.

Add JavaMarshaller in the common library, load it using the Java Service Loader pattern by adding it in META-INF/services and modify MP/SE server/clients that require serialization on the gRPC request/response by setting it as the custom marshaller.
Update pom.xml for microservices example because the contents are alreadyobsolete.
Enhance the READMEs across all the examples.
Update config for security-outbound example to add outbound entry so that the security context can properly be propagated from gRPC to Rest and vice-versa.
Remove grpc.marshaller.java.enabled from the config as it is not used anymore.
Replace policy with policy-validator as the former is an invalid ABAC attribute.

This should fix GRPC examples are failing #4576


PostgreSQL version needs to be updated in release versions. 10.* Is no longer supported.

Helm chart update is showing up a error.

This has been fixed in develop branch https://github.com/opencv/cvat/blob/develop/helm-chart/Chart.yaml#L34


animation.getKeyFrames().length throws Casting Exception

Issue details
Getting the length of the key frame array of an animation causes a Casting Exception.
Reproduction steps/code
Array<TextureAtlas.AtlasRegion> regions = textureAtlas.findRegions("test");
Animation<TextureAtlas.AtlasRegion> animation = new Animation<>(.1f, regions, PlayMode.LOOP);
int i = animation.getKeyFrames().length;

Version of LibGDX and/or relevant dependencies
1.9.6
Stacktrace
Error Message:
Exception in thread "LWJGL Application" java.lang.ClassCastException: [Ljava.lang.Object; cannot be cast to [Lcom.badlogic.gdx.graphics.g2d.TextureAtlas$AtlasRegion;

Please select the affected platforms

probably all

Duplicate of #4476


Use typescript in

I'm wondering where I can find some more documentation about using

We are using https://pinceau.dev under the hood


Substitute

Add blank icon to fix strange multiple monitor glitch where it shows a big empty space between icons on currently inactive monitors.

Cheers


Circle CI failures with circleci/rust

I just started up with Zero2Prod and tried running the provided CircleCI config as-is. It seems that CircleCI's circleci/rust Rust images has been replaced with by the cimg/rust series of images. The result of this seems to be that using the circleci/rust image causes package failures on the build step that require later versions of Rust.
I tried replacing the image using - image: cimg/rust:1.68.0 (just to match the Rust version I have locally), and upon doing so, the Security Step fails on the Cache Permission with the following: ^@^@chown: cannot access '/usr/local/cargo': No such file or directory. I was thinking that maybe in the cimg/rust:1.68.0 image that cargo is installed somewhere else, but since I am working on a Mac machine I can't seem to run the image and inspect much further.
Curious if you have any ideas as to why this might be happening, and how we can resolve it? Once we find a fix then I can open up a PR and make a contribution.
Docker and setting up CI/CD pipelines is new for me, so any advice and tips on debugging would be greatly appreciated!

So I actually ended up fixing this shortly after. What would be the best way to put in a PR for your Gist on the CircleCI config?

If you link the fixed config I can update the gist on my own 👍🏻

https://github.com/DennisTheMenace780/newsletter/blob/master/.circleci/config.yml

Updated!
Thanks again 😁


Don't download excluded container images

Is your feature request related to a problem? Please describe.
I'm running supabase start with the --exclude-containers flag in CI. Excluding containers is nice, but I'd also like to be able to skip downloading container images that will not be used at all.
Describe the solution you'd like
Excluded containers would not only skip running them, but also downloading them.

@sweatybridge that's a good point. I don't know how supabase internals work w.r.t. dependencies.
I'm running CI with --exclude realtime,storage-api,imgproxy,inbucket,edge-runtime with the goal of running minimal possible supabase instance that supports postgres + postgrest + auth (so probably more could be added).

To close this one off, you can now selectively disable services in supabase/config.toml to skip initialising those schemas on supabase start. This also means that images for disabled services won't be downloaded. You don't need to specify --exclude flag anymore.

Thank you for the update @sweatybridge. That is helpful (disabling a few services in config.toml has shaved ~30s from my CI runs!) but doesn't complete the issue imo.
It would be more flexible and intuitive to prevent container downloads (in addition to not running them) when the exclude flag is provided to supabase start.
A trivial example: Inbucket is very useful for local dev but often unused in CI. Assume the CI has a step to run supabase start -x inbucket. With the current implementation you have to choose between:

version controlling config to represent local dev; resulting in CI downloading the inbucket container but not starting it
version controlling config to represent CI; resulting in manual changes to config.toml by each dev when working locally

It would be more flexible and intuitive to prevent container downloads (in addition to not running them) when the exclude flag is provided to supabase start.

I think that's the current behaviour as well. For eg. if you enable inbucket in config.toml but disable it on CI with -x inbucket, the container image should not be downloaded.
But let me know if otherwise.

I re-enabled realtime and storage-api in my config (two of the services disabled to get the previously mentioned improvements) and re-ran the CI pipeline with supabase start -x realtime,storage-api. GitHub Actions logs shows it is still pulling the containers. Using Supabase CLI 1.187.10
v2.29.15: Pulling from supabase/realtime
...
Status: Downloaded newer image for public.ecr.aws/supabase/realtime:v2.29.15
v1.0.6: Pulling from supabase/storage-api
...

Applying migration 202402210[12](https://github.com/foundrydev/foundry/actions/runs/11342441101/job/31542851043#step:14:12)244_init.sql...
...
Seeding data from supabase/seed.sql...

The only way to exclude storage and realtime images from downloading is to disable those services in config.toml.
[storage]
enabled = false

[realtime]
enabled = false

Appreciate the quick response @sweatybridge , I guess the confusing bit is that the cli states it can be excluded?
supabase start --help
Start containers for Supabase local development

Usage:
  supabase start [flags]

Flags:
  -x, --exclude strings       Names of containers to not start. [gotrue,realtime,storage-api,imgproxy,kong,inbucket,postgrest,postgres-meta,studio,edge-runtime,logflare,vector,supavisor]

The auth, storage, and realtime images are special because they contain schema migrations to initialize the database.
If you exclude any of them via -x flag, those containers will not be started, but we still need to download the images to migrate the local database. So the help text of the exclude flag is technically true, but I can see why it might be confusing from what you'd expect.
Disabling those services in config.toml is the only way to skip applying schema migrations from those services. Since their migrations are skipped, we don't need to download the docker images for those services.


Deprecation warning for DOGSTATSD_ONLY

Why
This PR is the first step towards providing a separate image to run a standalone dogstatsd server.
The goal being to allow users to run a dogstatsd server container as a non-root user, and avoid mounting /proc, /sys and the docker socket just for running dogstatsd
Related

#79

👍


Visualize relationships in relations.csv as Graphviz graph

Would help with debugging/QC.

This relates to https://github.com/sorgerlab/indra/issues/102 although off-line visualization is also useful on its own.

Added in c0c08d8ce8.


Money#to_s converts number to float before formatting

irb> Money.new("100000000000000000000000000").to_s
=> "100000000000000004764729344.00"

This is because sprintf("%.2f", value) or sprintf("%.#{currency.minor_units}f", value) is being used to format the string the string, which is converting the number to a float before formatting the string, which loses precision.
We could instead use BigDecimal#to_s to format the string, although it doesn't support specifying the number of digits after the decimal to output.

Floats are dangerous for really large numbers or if no rounding is happening. sprintf will automatically round so even though
129.95 * 100 => 12994.999999999998

we still get the correct value
sprintf("%.2f", 129.95 * 100) => "12995.00"

We still have the issue of really big numbers (greater than 15 digits). For money that's somewhat less of a concern since the max 15 digit number is
nine hundred ninety-nine trillion, nine hundred ninety-nine billion, nine hundred ninety-nine million, nine hundred ninety-nine thousand, nine hundred ninety-nine

https://www.calculatorsoup.com/calculators/conversions/numberstowords.php
So somewhat... but then again

LOL, will FIX. Thanks for flagging this!

this issue has been fixed for a while
> Money.new("100000000000000000000000000", "USD").to_s
=> "100000000000000000000000000.00"


Roll engine 2ccfd9e562c9..a665c6fbd428 (20 commits)

git@github.com:flutter/engine.git/compare/2ccfd9e562c9...a665c6fbd428
git log 2ccfd9e562c9..a665c6fbd428 --no-merges --oneline
2019-08-06<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from QdP5I... to C9SBX... (flutter/engine#10649)
2019-08-06<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from siBGl... to Gh0U-... (flutter/engine#10648)
2019-08-06<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from Nj2Qs... to QdP5I... (flutter/engine#10647)
2019-08-06<EMAIL_ADDRESS>reset platformViewsController in engine.destroyContext when allowHeadlessExecution = false (flutter/engine#10629)
2019-08-05<EMAIL_ADDRESS>[flutter_runner] Port: Add connectToService, wrapping fdio_ns_connect. (flutter/engine#10644)
2019-08-05<EMAIL_ADDRESS>Don't use DBC for hot-reload on iOS.  (flutter/engine#10645)
2019-08-05<EMAIL_ADDRESS>Ensure debug-mode apps are always attached on iOS. (flutter/engine#10186)
2019-08-05<EMAIL_ADDRESS>Capture stderr for ninja command (flutter/engine#9896)
2019-08-05<EMAIL_ADDRESS>Roll src/third_party/skia 4df3d5340ef3..eb0770211a6c (13 commits) (flutter/engine#10643)
2019-08-05<EMAIL_ADDRESS>Roll src/third_party/dart 4bebfebdbc..8cd01287b4 (30 commits) (flutter/engine#10642)
2019-08-05<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from 3hQdw... to siBGl... (flutter/engine#10640)
2019-08-05<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from yvc6m... to Nj2Qs... (flutter/engine#10638)
2019-08-05<EMAIL_ADDRESS>Document the thread test fixture. (flutter/engine#10637)
2019-08-05<EMAIL_ADDRESS>Use Fuchsia trace macros when targeting Fuchsia SDK (flutter/engine#10634)
2019-08-05<EMAIL_ADDRESS>Fix threading and re-enable resource cache shell unit-tests. (flutter/engine#10636)
2019-08-05<EMAIL_ADDRESS>[fuchsia] CloneChannelFromFD fix for system.cc (flutter/engine#10635)
2019-08-05<EMAIL_ADDRESS>Roll src/third_party/skia a800ec96f724..4df3d5340ef3 (20 commits) (flutter/engine#10632)
2019-08-05<EMAIL_ADDRESS>Roll src/third_party/dart 9bb446aae1...4bebfebdbc (7 commits). (flutter/engine#10484)
2019-08-05<EMAIL_ADDRESS>Roll fuchsia/sdk/core/linux-amd64 from qTyU_... to 3hQdw... (flutter/engine#10627)
2019-08-05<EMAIL_ADDRESS>Roll fuchsia/sdk/core/mac-amd64 from cvvjU... to yvc6m... (flutter/engine#10626)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (franciscojma@google.com), and stop
the roller if necessary.

Auto-roller completed checks. About to merge.


New constructor format

For the v1-SDK we want to change the constructor from either:

createClientWithApiKey
createClientWithOAuth

to
import CogniteClient from 'sdk'

const client = new CogniteClient()

// throw error: "Need to login with either loginWithApiKey or loginWithOAuth before you can use the Cognite SDK"
client.asset.list({});

client.loginWithApiKey({
  project,
  apiKey,
});

// throw error: "You cannot re-login with an already logged in Cognite client instance. Try to create a new Cognite client instance instead."
client.loginWithOAuth({
  project,
  onAuthenticate,
});

// Also add this utility function:
client.login.status(apiKey);

fixed in #150


array: generalize S.find

This leads me to believe that sanctuary-type-classes should ensure consistent enumeration order in Object$prototype$reduce (by sorting the keys). Do others agree that this is desirable?
> find (const True) (Map.fromList [("x", 0), ("y", 1)])
Just 0

> find (const True) (Map.fromList [("y", 1), ("x", 0)])
Just 0

Object$prototype$equals and Object$prototype$lte already do this. If we're going to be consistent, I don't think we really have a choice.


report_info.json not updated consistently

For example, this commit should include a change to report_info.json.

Closing. It appears to be updating now...


Upgrade generate_bootstrap_package.sh script to generate bootstrap for target profile name

Currently, script generate_bootstrap_package.sh generates bootstrap for source environment.
It is actually not worth much - all it does is making it more convenient (not to type lengthy commands) to run bootstrap with overridden enable_bootstrap_target_envs and verify nothing has failed.
So, this script will be even more worthless because there will be no need to craft command line. Bootstrap generation is, in fact, supposed to generate package for single target environment only anyway - see #26. And enable_bootstrap_target_envs is supposed to be removed by #28.
To make more useful allow building bootstrap for any target profile name - checkout specified branch in target pillar repository (according to profile name specified on the command line).

Issue #29 is pre-requisite for updated functionality of this script.

This is done. See this comment how.


Prev and Next Month Buttons Not Working in IE11

Hi,
When I push the next month button in IE11, it flashes to the next month, then reverts back to the current month.  When I click on previous month, nothing happens at all.  For prev button I get the following console errors:
Object doesn't support property or method 'next'
Unable to get property 'contains' of undefined or null reference
Unable to get property 'querySelector' of undefined or null reference
For next button, I don't get any console errors.

Disregard.  Was an import package issue.


Upgrade to .NET 8.0

.NET 8.0 has been released!
https://github.com/dotnet/core/blob/main/release-notes/8.0/8.0.0/8.0.0.md

[x] NEONSDK
[x] OPERATORSDK
[x] NEONKUBE
[x] NEONCLOUD

This is still in progress.  I played build whack-a-mole for a few days struggling with FileNotFoundException thrown for missing assemblies.  I believe that has been addressed now.
Unit tests are passing but we're having trouble deploying clusters now.

Looks these builds are working now, after accounting for new build behaviors:
"GenerateDepsFile" related errors while building is a signal that you need to clean your bin/objs.

Oh wait: we still may be still having trouble deploying clusters with these changes.  I'm going to check and fix any issues.

DONE: This is finally done.
A lot of the trouble here was caused by the new neoncloud-clean.cmd script which was exiting after cleaning the first solution folder and leaving the rest alone.  This resulted in confusing signals and led me on wild goose chases.


Support React Native's node environment: Module 'stream' does not exist in the Haste module map

Is your feature request related to a problem? Please describe.
My React Native app successfully leveraged ms-rest-js version 0.2.4. A recent upgrade to @azure/ms-rest-js version 1.5.3 now yields
Failed to load bundle(http://localhost:8081/index.bundle?platform=ios&dev=true&minify=false)
with error:(Unable to resolve module `stream` from
<EMAIL_ADDRESS>
Module `stream` does not exist in the Haste module map

Describe the solution you'd like
ms-rest-js works in React Native's node environment.
Describe alternatives you've considered
none
Additional context
autorest --info:

AutoRest code generation utility [version: 2.0.4283; node: v10.9.0]
(C) 2018 Microsoft Corporation.
https://aka.ms/autorest

Showing All Installed Extensions

 Type       Extension Name                           Version      Location
 core       @microsoft.azure/autorest-core           2.0.4300     /Users/me/.autorest/<EMAIL_ADDRESS> extension  @microsoft.azure/autorest.modeler        2.3.51       /Users/me/.autorest/<EMAIL_ADDRESS> extension  @microsoft.azure/autorest.typescript     2.0.683      /Users/me/.autorest/<EMAIL_ADDRESS>
 react-native info

  React Native Environment Info:
    System:
      OS: macOS 10.14.2
      CPU: (12) x64 Intel(R) Core(TM) i7-8750H CPU @ 2.20GHz
      Memory: 5.32 GB / 32.00 GB
      Shell: 3.2.57 - /bin/bash
    Binaries:
      Node: 10.9.0 - /usr/local/bin/node
      Yarn: 1.9.4 - /usr/local/bin/yarn
      npm: 6.6.0 - /usr/local/bin/npm
      Watchman: 4.9.0 - /usr/local/bin/watchman
    SDKs:
      iOS SDK:
        Platforms: iOS 12.1, macOS 10.14, tvOS 12.1, watchOS 5.1
      Android SDK:
        API Levels: 23, 24, 25, 26, 27, 28
        Build Tools: 23.0.1, 23.0.2, 23.0.3, 24.0.3, 25.0.0, 25.0.1, 25.0.2, 25.0.3, 26.0.0, 26.0.1, 26.0.2, 26.0.3, 27.0.3, 28.0.3
        System Images: android-23 | Google APIs ARM EABI v7a, android-23 | Google APIs Intel x86 Atom, android-25 | Google APIs Intel x86 Atom, android-25 | Google APIs Intel x86 Atom_64
    IDEs:
      Android Studio: 3.3 AI-182.5<IP_ADDRESS>99772
      Xcode: 10.1/10B61 - /usr/bin/xcodebuild
    npmPackages:
      react: ^16.6.3 => 16.6.3
      react-native: ^0.57.8 => 0.57.8
    npmGlobalPackages:
      react-native-cli: 2.0.1
      react-native-rename: 2.3.2

This seems abandoned... any solution/alternative for using with RN?

This seems abandoned... any solution/alternative for using with RN? @kpajdzik

Just an update: we are continuing our investigation in supporting React Native at https://github.com/Azure/azure-sdk-for-js/issues/5771.

Just an update: we are continuing our investigation in supporting React Native at https://github.com/Azure/azure-sdk-for-js/issues/5771.


Why jpro App can't be run using bundle JavaFX in JDK/JRE?

I use Zulu Java + JavaFX for all of my apps. jPro not runs with bundle JavaFx. Why not it simply ignores bundle Javafx and use it own JavaFX?
Any why to run jpro with bundle jdk by ignoring bundle jdk simpley?
D:\MyApp>set JAVA_HOME=D:\Programs\zulu17.34.19-ca-fx-jdk17.0.3-win_x64
D:\MyApp>set PATH=%JAVA_HOME%\bin;%PATH%
D:\MyApp>set GRADLE_HOME=D:\Programs\gradle-7.3.3-bin
D:\MyApp>set PATH=%GRADLE_HOME%\bin;%PATH%

D:\MyApp>gradle jproRun

> Task :MyApp:compileJava
Note: D:\MyApp\src\main\java\com\gluonhq\charm\down\plugins\impl\DesktopReceiptBillService.java uses or overrides a deprecated API.
Note: Recompile with -Xlint:deprecation for details.
Note: Some input files use unchecked or unsafe operations.
Note: Recompile with -Xlint:unchecked for details.

> Task :MyApp:compileAllReports
28 designs compiled in 5629 ms
[info] a.e.s.Slf4jLogger - Slf4jLogger started
[info] c.g.c.b.i.Finalizer - Couldn't access Thread.inheritableThreadLocals. Reference finalizer threads will inherit thread local values.
[info] play.api.Play - Application started (Prod) (no global state)
[info] p.c.s.AkkaHttpServer - Enabling HTTP/2 on Akka HTTP server...
[info] p.c.s.AkkaHttpServer - Listening for HTTP on /[0:0:0:0:0:0:0:0]:8081
[info] c.jpro.activity - Starting the server
[info] c.j.i.s.Initializer$ -  *** Starting JPro: https://www.jpro.one/ ***
[info] c.j.i.s.Initializer$ - OS: Windows 10
[info] c.j.i.s.Initializer$ - JPro: 2022.1.5
[info] c.j.i.s.Initializer$ - Java version: 17.0.3
[info] c.j.i.s.Initializer$ - JVM: 17.0.3 Azul Systems, Inc.
[info] c.j.i.s.Initializer$ - JavaFX jar: jrt:/javafx.graphics
[info] c.j.i.s.Initializer$ - redirected OUT / ERR
[info] c.j.i.s.Initializer$ - installed SLF4JBridge
[error] c.j.i.s.Initializer$ - JavaFX isn't loaded by the SystemClassLoader. This typically happens, when using a JDK/JRE which bundles JavaFX. Please use a JDK/JRE which does not bundle JavaFX.
[info] p.a.i.l.c.CoordinatedShutdownSupport - Starting synchronous coordinated shutdown with ServerStoppedReason reason and<PHONE_NUMBER> milliseconds timeout
<============-> 93% EXECUTING [10m 17s]
> :MyApp:jproRun

If I use JDK which not bundle JavaFx code not compiles as Javafx libs not found, I simple don't want to use javafx libs or plugins in gradle.build file as it's included in zulu jdk

JPro provides its own JavaFX Fork. It is not possible to start these JDK/JREs without its bundled JavaFX.
In my opinion, it's bad practice to use these JDKs. It has the big disadvantage that changing the JavaFX version is much more complicated.
It should be simple to just switch the JDK/JRE.

@FlorianKirmaier In my specific case, I use bundle Javafx with JDK (Zule) for desktop Applications. Because It's more easy to use bundle JFX and there is no need to add Javafx libs in class path (witch is little bit difficult). e.g. I am not using module.info and just add libs to class path.
In simple terms, I am not using javafx plugins noir adding javafx libs in dependencies tree for desktop applications. From now it's oky.
But jpro app not run (jproRun) with/without bundle jdk. Until I add javafx plugins or add javafx libs to depedencies?
I have two questions:

Why jpro not use bundle JFX (zulu) if it's needed any?
If jpro have it's own implementation of javafx libs then why it's needed other libs whether it's javafx mobile plugins or libs simply?

For compiling code it's needed javafx libs should be on class path. I think jpro own javafx libs are included at runtime not compile time?

1.) JPro requires its own JavaFX build.
2.) Actually JPro doesn't need other JavaFX-jars. But when you compile, you have to compile against some JavaFX Jars. Theoretically, you could also compile against the JPro JavaFX Jars. But usually, it's more convenient to just compile against the official JavaFX-jars - because we are binary compatible to them.
If you compile your project on ZuluFX, and then run it on another JDK/JRE with JPro, it should work without adding any javafx-jars to the dependency. But that's complicated. Usually, it's easier to just add JavaFX as a dependency.


prediction

Hello
I'm trying to implement keypointnet in my project. I generated my custom dataset and i'm using your code reimplementation of keypointnet with tensorflow 2.0. I have trained the network on my custom dataset and I would like to confirm it was trained well, so i'm using the prediction code you provided but there was something odd in the code in the function that draws points on the images specifically in that line
x = (min(max(x, -1), 1) * vw / 2 + vw / 2) - 0.5
if the is negative the output will always be 63, if positive it will be 64, so can you provide me with a solution to this.
also this  is an output of the uvz of the model
[[-9.5309224e-07 -4.7613867e-07 -5.1632648e+02]
[-9.5309224e-07  4.6566129e-10  4.4604336e+03]
[-2.3839530e-06  1.4312100e-06  3.9983486e+03]
[-1.9071158e-06 -9.5367432e-07  3.2920288e+03]
[-1.9063009e-06 -2.3830216e-06 -1.2012737e+03]
[ 5.8204819e-10  1.4302786e-06 -4.8412139e+03]
[-1.6684644e-06 -2.6226044e-06  6.3846865e+03]
[ 3.1001400e-06 -3.0989759e-06  2.3328838e+03]
[ 1.6692793e-06  2.3841858e-07  1.5663926e+03]
[-2.6221387e-06 -2.3795292e-07  7.4820908e+03]]
are these values normal?

Hey,
Can you please provide the missing details in the question like 'if the _____ is negative the output will always be 63, if positive it will be 64, so can you provide me with a solution to this.' ?
Next, on seeing your output, I suspect the model has not been trained properly, x and y values are very small and clumped together. I would suggest training with more weightage on the separation loss so that the keypoints are separated in x and y.

Thanks for your reply,
I will try to train it again just waiting for a more resourceful pc
for the missing part in the question, what i meant is in the above equation the term (min(max(x, -1), 1) always evaluate to 1 or -1 so the x will always evaluate to vw / 2 +/-1


Updated OCC part of the RS Viewer for DSO-16841.

Updated OCC part of the RS Viewer for DSO-16841.

Can you clarify what should be the modified default OCC mode?

OCC mode is the default.


State of component is not displaying

Augury is a very good tool while developing angular 2 application. But the problem is, it doesn't display state of component. For some components it is showing state, but then some components it doesn't show state. Only dependency is displayed.

@ajsuvarna6 Can you provide an example repo or plunker where this occurs? That symptom has been reported (but not seen by me) when the router is injected into the component. Is this the case?

It seems to depend on the dependencies. Having MdDialog (Angular Material2) listed as a dependency for a component will make state not show up for the component on Augury 1.2.7.

@stevenkampen I'm experiencing the same thing only for components where I'm injecting router..

@stevenkampen I'm seeing the same when using ngTemplateOutlet. The component loses all properties in the viewer. I'll see if I can get a plunker together, but I first ran across it while running ngx-datatable locally and viewing one of their tabs that uses ngTemplateOutlet i.e. Row Detail under Basic.
https://github.com/swimlane/ngx-datatable
Unfortunately their demo page is run in prod mode so you have to run it locally.

@irackley-mc Thanks for your report. I'll get on this shortly and let you know when it's done.

Ok I can recreate this now and confirm it's an issue with serialization/deserialization of some complex objects. Looking into it.

This is fixed now in version 1.2.9, which was just released.

Hi I seem to have this problem aswell but I have 1.4.0

Yes, I'm getting same problem with 1.4.0

@stevenkampen,  I am still having the same problem with 1.5.0

@irackley-mc This is fixed for you now right? I see state for all components I could find in that application you shared.

@prasanna-kumar @raydaim If this is persisting for you, can you open a new issue (after checking one doesn't already exist that fits what you're seeing)?

@stevenkampen It has been working for me since your fix. My guess is the new reports are related but not the same exact issue, although I haven't tried out Angular 4 at all.

@irackley-mc Yes, that's the case here. Several things can screw up serialization apparently so I'm hunting them down one by one. Thanks for your report!

I am having the same problem, using Augury 4.3.6 - any update on this?

having same issue
https://gyazo.com/8c5583c88dd9c20ce2460d855f4045c0
with canary build and usual augury 1.17

Howdy, @kostia-lev
This issue is caused by numerous edge cases, many of which (although possibly not all) have been identified and fixed for in the latest Canary (version 1.17.4), released yesterday.
Can you confirm you are still seeing this issue in Canary version 1.17.4 ?

Still having this issue with 1.19.1

Happening me with 1.21.0

How is this still an issue after nearly two years?! Displaying state on all occasions should be basic functionality and #1 priority if broken!


find a way to extend built-in directive

What problem does this feature solve?
I mean, how can I make:
<img :src="source">

become to:
<img :src.lazy="source">

perhaps, I have to build a new directive？
What does the proposed API look like?
Vue.directive.extend('src.lazy') ?

@74sharlock, : is shortand for v-bind which is not actual directive, it's a part of vue-template and it is processed during compile time (not runtime). For lazy load I recommend making a custom component like:
<lazy-img :src="source" />.
If you need any help with that please ask on the forum, Stack Overflow or on gitter. I prefer gitter and there's always someone online willing to help.

As @nickmessing suggested - use a component instead, or just a custom directive.


⚠ Bofrost.es has degraded performance

In 6a56a30, Bofrost.es (https://www.bofrost.es) experienced degraded performance:

HTTP code: 200
Response time: 1397 ms

Resolved: Bofrost.es performance has improved in 32dd119 after 43 minutes.


[Feature][SDK]

Description
I want to contribute the producer implementation code to the Go SDK, so I'd like to check to see if there are any similar contributions recently.
At present, the basic logic has been sorted out, and the rest is mainly to spend time debugging, so I plan to submit a version of the implemented code before the end of the Spring Festival.
Use case
No response
Are you willing to submit PR?

[x] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

@bigdaronlee163, which component SDK do you want to do, DataProxy or TubeMQ?
There are existing issues for them, and you can use them when you confirm.

DataProxy
https://github.com/apache/inlong/issues/4961

TubeMQ
https://github.com/apache/inlong/issues/4968

TubeMQ go  sdk for producing message

@bigdaronlee163 thanks, but there is another contributor @xuehuanran doing #4968. You could choose other issues and review the PR after completion.
I will close this issue, and we can follow the forward pace in #4968.


Migrate to AndroidX

Hey, it looks like there are a few missing dependencies to migrate to AndroidX libraries
https://github.com/wongpiwat/flutter-trust-location/blob/master/android/build.gradle#L38
It will break apps that write tests and depend on this plugin.

@BugsBunnyBR,
Thank you for the information provided. I will fix it with the next release.


Handle marathon group error response in scale modal

This works well, I am going to merge this guy


Separate ratis dependencies between alluxio server and ufs ozone

Related to https://github.com/Alluxio/alluxio/pull/14870

alluxio-bot, merge this please


What is the current recommended module system for developing applications with closure library?

As of now closure compiler is able to to handle a few different js modules, including:

goog.prodive/goog.require
goog.module
ES6 modules
Common JS

But when it comes to developing with closure library, only the first 2 are supported. Recently, the javascript implementation of dependency maker, closure-deps made the 3rd option possible.
It is not clearly documented that which approach should a greenfield project that depends on closure library take when it comes to modules. Is ES6 modules the future of applications built with closure library?
Are there any reasons to use goog.provide for new projects, including any performance/optimization gains?

So, this is complicated. Closure Library hasn't yet to wholly move to goog.module yet, or even has wide use of ES6+ features. The library will always (sadly) be behind due to it needing to support the lowest common denominator.
For your own project I think it highly depends on your situation. Are you a leaf project, and no one depends on you? If so, you can use whatever module format you like! My personal opinion would be ideally ES modules, though if you have a better developer setup or IDE that works better with something else, go for that. Whatever makes you most productive, followed by whatever is standardized. This may also include TypeScript via tsickle and clutz.
If your project is not a leaf, meaning you have projects depending on yours, you need to take them into consideration and what they support. ES modules are still not widely supported in the wild. Examples: Still not supported in Node (well, it's behind a flag...), browser support is there, but not terribly ergonomic due to only supporting relative paths (import maps hopefully eventually solve that), etc. Though you may be able to get around this by transforming your source code to a module format that is supported widely, e.g. write in ES modules but use webpack to provide a single, non-module bundle. Or babel to UMD/CJS with RequireJS, etc.
For the time being it doesn't matter what your source code is in, in all cases you will use goog.require to get access to Closure Library, as it is currently all goog.provide and goog.module files (or import 'goog:namespace' in TypeScript).
@concavelenz @shicks @nreid260 may have more thoughts or guidance.

Thank you @jimbojetlag. How is it possible to use Closure Library in Typescript today? It seems like Clutz has not been applied to Closure Library, or at least the resulting d.ts files are not open sourced.
Would the existing deps.js of Closure Library be enough to use the library, or would it need dynamic dependency generator? Is there a minimal example project to demonstrate this?

This issue https://github.com/angular/clutz/issues/755 by @gregbown demonstrates that Clutz cannot be successfully used to generate .d.ts fo Closure Library. The issue remains unanswered.
Given that Clutz was created to target Closure annotations, it is ironic that both Clutz and Closure Library are open sourced, but the resulting Typescript definitions are not!

I believe the idea is that clutz should be used to generate the .d.ts files on the fly. I was not aware of that issue, thank you for bringing it to my attention. I'll reach out to the Angular folks and talk with them about it.

I believe the idea is that clutz should be used to generate the .d.ts files on the fly.

When using Closure Library as a static library, wouldn't the .d.ts files be also static, similar to the generated deps.js that comes with Closure Library?
Sorry to ask this question again, what is the workflow for developing a (leaf) application in Typescript hat uses Closure Library? I can see Clutz must be used to generate the .d.ts files, does that mean that from that point Cosure Library can be used just like a Typescript library, or does it still require Closure Library specific module loaders during dvelopment?

When using Closure Library as a static library, wouldn't the .d.ts files be also static, similar to the generated deps.js that comes with Closure Library?

If you're using an npm release, yes. If you constantly sync to head, no. Closure Library is only as static as you want it to be :)

Sorry to ask this question again, what is the workflow for developing a (leaf) application in Typescript hat uses Closure Library? I can see Clutz must be used to generate the .d.ts files, does that mean that from that point Cosure Library can be used just like a Typescript library, or does it still require Closure Library specific module loaders during dvelopment?

No worries! @mprobst should be able to comment more here. I believe it may force you into using tsickle (which will compile your TypeScript to goog.module files), but I am not 100% sure.
As far as source code goes, I believe should be able to use a special import syntax in your TypeScript files. If your import specifier is goog:namespace, then that should import the Closure namespace.
import * as googArray from 'goog:goog.array';

@jplaisted is right, you'll need to use tsickle (https://github.com/angular/tsickle) to compile your code, which emits goog.module code, and then on the client side, it's using the Closure module system to bind everything together.
I'm afraid we don't have a very well trodden setup for all these bits together, and you might encounter some trouble with tsickle and the path names it generates. We haven't seen a lot of people using Closure together with TypeScript outside of Google yet.

So stemming from #937, I took a look at JsAction. They publish some pre-built code you could use. If that's all that you need, and don't want to use the Closure Compiler or Library, I'd recommend using those files.
If you want to use the Closure Compiler then you should probably write ES modules or goog.modules (goog.provide is super old and is in the global scope). You could try using TypeScript with the Closure Compiler via tsickle and clutz, though I don't think we have a well worn path around this for small external projects yet (like having a super fast edit / refresh dev flow). If you want to use TypeScript but not the Closure Library/Compiler, then there's no reason to use clutz or tsickle, just use the minifed / prebuilt JsAction files. They should be super simple to write some .d.ts files for, if they don't already exist.
So fundamentally this all depends on your use case. What combinations Closure Library, the Closure Compiler, JsAction, and TypeScript did you want to use?

I took a look at JsAction. They publish some pre-built code you could use. If that's all that you need, and don't want to use the Closure Compiler or Library, I'd recommend using those files.

You are right. However, the precompiled files are more or less for demo purposes. For anything nontrivial, you will have to define your own dispatch handler which would require the use of Closure Library and Closure Compiler.
One option is to develop some minimal code that depends on jsaction in Closure Scripts (goog.provide), or Closure modules (goog.module), which will be compile by Closure Compiler and used as a library by the main application written in ES modules. This would require the small library calling jsaction to be a project of its own.
Another option would be to use an environment that both Closure Scripts and ES modules can be used together. This helpful demo of closure-webpack-plugin shows that this can be made possible by Webpack loader. There seem to be some overlapping between this Webpack plugin and closuremakedeps.js in this repo, they both make it possible to mix ES modules with Closure Scripts.
I guess the optimum setup for an application that uses Closure Library is the one that the developer can use the above mix modules, and yet be able to compile the code down to the Closure Compiler annotated version for best optimizations in production. It seems to me that closure-webpack-plugin is able to provide that environment. It would be good to have that plugin in referenced Closure Library wikis.
Overall, the external developer experience for using Closure Library, and libraries that depend on it, remains poor, and not having an improving trend. When Closure Compiler was first open sourced, there was only one way of developing apps with it: Closure Scripts. Now there are many combinations, and making decisions about them is not easy, sometimes leading to impossible situations. For example, you can see in https://github.com/google/closure-templates/issues/169 how the side effect of compiling Typescript to Closure Script led to unopensourcing the Incremental DOM Soy compiler.

This issue is being closed as a preparation step before archiving the repository.  See the README for more detail.


Add deliverytag to Message type

This PR add the deliveryTag onto the Message type so it can be used by consumers of the library.
I couldn't see an obvious unit test for this change so I added an additional check the integration tests to validate the field is populated and had a value the size I was expecting back from SB.
See #122

Thanks for the review :+1:  Happy to add in the ability to set too and fix up the rest. Hopefully have changes back to you shortly.

I've added in the send capability and fixed up the feedback. Let me know what you think, happy to fix up anything as needed.
I saw the integration tests flapping a bit while running them and found that increasing some of the sleeps improved their reliability - hope that's alright.