Manifest when using multiple vendors and commons chunks simultaneously

I followed the example in common-chunk-and-vendor-chunk but I'm also using multiple instances of the CommonsChunkPlugin. Here is the relevant configs:
{

  entry: {

    'desktop/index': './js/desktop/index.js',
    'desktop/post': './js/desktop/post.js',
    'desktop/vendor': [
      'jquery'
    ],

    'mobile/index': './js/mobile/index.js',
    'mobile/vendor': [
      'jquery'
    ]

  },

  output: {
    filename: '[name].[chunkhash].js',
    chunkFilename: 'chunk.[chunkhash].js',
    path: path.resolve(__dirname, 'dist')
  },

  plugins: [
    new webpack.optimize.CommonsChunkPlugin({
      names: ['desktop/common', 'desktop/vendor'],
      chunks: ['desktop/index', 'desktop/post'],
      minChunks: 2
    }),
    new webpack.optimize.CommonsChunkPlugin({
      names: ['mobile/common', 'mobile/vendor'],
      chunks: ['mobile/index'],
      minChunks: 2
    })
  ]

};

And the build seems to be correct:
                                 Asset       Size   Chunks                    Chunk Names
  mobile/index.baa6ab4a4f62f25564ff.js    1.42 kB  0, 5, 6  [emitted]         mobile/index
 desktop/index.236c4e6e375a870d353b.js  887 bytes     1, 6  [emitted]         desktop/index
  desktop/post.7ca5365953dac57d7302.js  524 bytes     2, 6  [emitted]         desktop/post
desktop/vendor.b2e3b72f531e1a0ab842.js     274 kB  3, 4, 6  [emitted]  [big]  desktop/vendor
 mobile/vendor.4330c96ca776ad08e16c.js     274 kB  4, 3, 6  [emitted]  [big]  mobile/vendor
desktop/common.5a885605fc51ff0a90f2.js    5.92 kB     5, 6  [emitted]         desktop/common
 mobile/common.eb6b4a3b2aeb1164ec5f.js    5.75 kB        6  [emitted]         mobile/common

I added the files to my pages in the following order:
<script src="dist/desktop/vendor.b2e3b72f531e1a0ab842.js" charset="utf-8"></script>
<script src="dist/desktop/common.5a885605fc51ff0a90f2.js" charset="utf-8"></script>
<script src="dist/desktop/index.236c4e6e375a870d353b.js" charset="utf-8"></script>

and then I got the following error:
Uncaught TypeError: Cannot read property 'call' of undefined     vendor.b2e3b72….js:55
    at __webpack_require__ (vendor.b2e3b72….js:55)
    at Object.5 (index.236c4e6….js:17)
    at __webpack_require__ (vendor.b2e3b72….js:55)
    at webpackJsonpCallback (vendor.b2e3b72….js:24)
    at webpackJsonpCallback (common.5a88560….js:19)
    at index.236c4e6….js:1

Seems like I also need something like a manifest file, but there's none in the example and I'm not sure if this is a bug or the expected behavior. I've made a repo that you guys can clone to reproduce this behavior.
System info:
OS: Linux Mint 18 Cinnamon 64-bit
Node version: 6.10.0
NPM version: 4.3.0
Webpack version:  2.2.1

new webpack.optimize.CommonsChunkPlugin({
  names: ['desktop/common', 'desktop/vendor'],
  chunks: ['desktop/common', 'desktop/index', 'desktop/post'],
  minChunks: 2
}),
new webpack.optimize.CommonsChunkPlugin({
  names: ['mobile/common', 'mobile/vendor'],
  chunks: ['mobile/common', 'mobile/index'],
  minChunks: 2
})

Awesome! It works! Thanks @sokra. I strugled with this for days, literally.
So, I don't really need a manifest file or it is mandatory for long term caching?

@MtDalPizzol @sokra How did fix the issue.
I'm kind a clueless.
Any help would be great.

I think @sokra can explain this in a better way. But what did the trick for me was adding the common chunk to the chunks arrays property. You can take a look at this working repo.

@MtDalPizzol okay. Got it.

I can't figure this out.
@MtDalPizzol how do we create multiple bundles with:
. entry file (i.e: apps/admin/index.js)
. vendor file (i.e: lodash, react, whatever)
. manifest file
?
Did you sort this out?

Hey @damianobarbati ! Just be clear: I'm not a Webpack wizard like other guys around here. I'm absolutely not the best person to explain how the CommonsChunk plugin work, for example. Another thing is that I don't know the status of this since Webpack 2 was released. That said, what I can tell you is to read this post on StackOverflow to understand how the plugin works in a more detailed way. Also, I made a repo using the  the configs described above where I split dekstop and mobile bundles that share plugins. You can take a look here.

@wzup , Sorry for the delay. It's a long time since I worked on this issue. I'm not sure if something has changed in newer Webpack versions. Bu notice: the repo I made only deals with multiple vendors with common chunks. It gives you big vendor chunks, but it gives you multiple vendor chunks (one for each endpoint I need). You'll have to optimize the code on your own, using other plugins. Here's an output after running npm run build on the repo.
Version: webpack 2.7.0
Time: 766ms
                                 Asset       Size  Chunks                    Chunk Names
  mobile/index.139001604b762995b658.js    1.42 kB    0, 2  [emitted]         mobile/index
 desktop/index.f466d38475c5ec0d93e1.js  885 bytes       1  [emitted]         desktop/index
desktop/common.f155698581328ad65fb1.js  138 bytes       2  [emitted]         desktop/common
  desktop/post.760f239cfe094cf0ecf0.js  522 bytes       3  [emitted]         desktop/post
desktop/vendor.685847226605410d6e37.js     278 kB    4, 5  [emitted]  [big]  desktop/vendor
 mobile/vendor.db44b7492d6d516c4ea9.js     278 kB    5, 4  [emitted]  [big]  mobile/vendor
   [0] ./js/plugins/plugin-2.js 61 bytes {0} {2} [built]
   [1] ./js/plugins/plugin-1.js 39 bytes {0} {1} [built]
   [2] ./~/jquery/dist/jquery.js 272 kB {4} {5} [built]
   [3] multi jquery 28 bytes {4} {5} [built]
   [4] ./js/plugins/plugin-3.js 39 bytes {0} [built]
   [5] ./js/desktop/index.js 88 bytes {1} [built]
   [6] ./js/desktop/post.js 69 bytes {3} [built]
   [7] ./js/mobile/index.js 132 bytes {0} [built]
Done in 1.30s.


[sw] SPI PAGE_PROGRAM command can trigger segfault in bootstrap

Description
I have been toying with a fuzzer that generates SPI commands in #18929, which discovered a segfault produced by a PAGE_PROGRAM command with a payload_byte_count. I converted this input to a unit test, patched it onto the master branch, and verified it still produces the segfault in #19131.
It looks like we're not properly bounds-checking in bootstrap_page_program().
$ ./bazelisk.sh test --config=asan --test_output=streamed //sw/device/silicon_creator/rom:bootstrap_unittest
[...]
AddressSanitizer:DEADLYSIGNAL
=================================================================
==12==ERROR: AddressSanitizer: SEGV on unknown address (pc 0x7f29cfdfc03b bp 0x7ffc950a5ae0 sp 0x7ffc950a5a60 T0)
==12==The signal is caused by a READ memory access.
==12==Hint: this fault was caused by a dereference of a high value address (see register values below).  Dissassemble the provided pc to learn which register was used.
    #0 0x7f29cfdfc03b in bootstrap_page_program sw/device/silicon_creator/rom/bootstrap.c:176
    #1 0x7f29cfdfc99e in bootstrap_handle_program sw/device/silicon_creator/rom/bootstrap.c:316
    #2 0x7f29cfdfce84 in bootstrap sw/device/silicon_creator/rom/bootstrap.c:386
    #3 0x561998b6049b in TestBody sw/device/silicon_creator/rom/bootstrap_unittest.cc:703
    #4 0x7f29cfbd6feb in void testing::internal::HandleSehExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*) external/googletest/googletest/src/gtest.cc:2607
    #5 0x7f29cfbca77d in void testing::internal::HandleExceptionsInMethodIfSupported<testing::Test, void>(testing::Test*, void (testing::Test::*)(), char const*) external/googletest/googletest/src/gtest.cc:2643
    #6 0x7f29cfb88ccb in testing::Test::Run() external/googletest/googletest/src/gtest.cc:2682
    #7 0x7f29cfb8a287 in testing::TestInfo::Run() external/googletest/googletest/src/gtest.cc:2861
    #8 0x7f29cfb8b3de in testing::TestSuite::Run() external/googletest/googletest/src/gtest.cc:3015
    #9 0x7f29cfbb1370 in testing::internal::UnitTestImpl::RunAllTests() external/googletest/googletest/src/gtest.cc:5855
    #10 0x7f29cfbda0a9 in bool testing::internal::HandleSehExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char const*) external/googletest/googletest/src/gtest.cc:2607
    #11 0x7f29cfbcd124 in bool testing::internal::HandleExceptionsInMethodIfSupported<testing::internal::UnitTestImpl, bool>(testing::internal::UnitTestImpl*, bool (testing::internal::UnitTestImpl::*)(), char const*) external/googletest/googletest/src/gtest.cc:2643
    #12 0x7f29cfbadec7 in testing::UnitTest::Run() external/googletest/googletest/src/gtest.cc:5438
    #13 0x7f29cfd37c75 in RUN_ALL_TESTS() external/googletest/googletest/include/gtest/gtest.h:2490
    #14 0x7f29cfd37b52 in main external/googletest/googlemock/src/gmock_main.cc:70
    #15 0x7f29cf229d8f in __libc_start_call_main ../sysdeps/nptl/libc_start_call_main.h:58
    #16 0x7f29cf229e3f in __libc_start_main_impl ../csu/libc-start.c:392
    #17 0x561998b47f14 in _start (/home/dan_zerorisc_com/.cache/bazel/_bazel_dan/e8b9a1cec6d6126ab719828a176ad974/execroot/lowrisc_opentitan/bazel-out/k8-fastbuild/bin/sw/device/silicon_creator/rom/bootstrap_unittest+0x62f14)

FYI @alphan

Per our chat offline, this is not a real concern. The SPI device driver will never generate values larger than 256, and the lowest value that can trigger this behavior is 257.
Before closing this issue, I'll add a comment to clarify the range of spi_device_cmd_t::payload_byte_count. I'll also update my fuzzer to stay within that range when it generates spi_device_cmd_t values.


Avoid sending PRACK with undefined rseq value in RAck header

Before sending a PRACK it checks that the response.getHeader("rseq") exists before sending out an PRACK. I had scenarios where a PRACK was send with undefined rseq parameter in the RAck header:
RAck: undefined 2641 INVITE
Reported in #672
Could not test the code yet, I am not a javascript nor typescript developer
But feel free to change this.

Background information:
In order to send the PRACK, an Rseq number recieved in one of the 1xx provisional responses.
If the provisional 1xx responses never contains an Rseq it makes no sence to send a PRACK because the RAck header must contain an resq.
The change avoids sending an PRACK whenever there was no Rseq recieved before.
We are using this fix in so far without issues. Meanwhile I also could test the code.

Hi Fabian,
Sorry for not responding earlier. This fix kind of just covers up the underlying issue which is how SIP.js handles 100rel. We believe that the underlying issue is actually fixed with the rewrite of the core SIP.js code. We are in the process of working on this, but the first pass at it is currently on master. You can test it by building master and turning on the UA option experimentalFeatures. We expect to have a 0.14.0 release with this stuff finalized in the next few weeks. If the issue is still happening with the experimentalFeatures turned on let us know and we will take a closer look.

This fix is now out in 0.14.0. We ended up pulling out the experimentalFeatures, so you should be able to upgrade and test with no changes. Can you let us know if this is fixed?

Closing this due to inactivity. Feel free to re-open or open a new issue if the issue persists, but we believe the issue has been resolved since 0.14.0 (suggested version is currently 0.14.2).


What is the best pretrained model?

Hi,
For now, it is GhostFaceNetV1-1.3-1 (A) trained on MS1MV3
https://github.com/HamadYA/GhostFaceNets/releases/download/v1.2/GhostFaceNet_W1.3_S1_ArcFace.h5GhostFaceNetV1-1.3-1 (A)	MS1MV3


Support for using DTOs (with autoMapper)

I searched for solution to this use case, and saw here that @sergkr ask for this. any news about this? where can I read docs? thanks!

We have no plans to implement this at this time. Putting on the backlog for now.

So, if I using DTO in my object (and map them to EF entities) I should look for another solution, or you have any work around to deal with this? thanks

@arielcloud can you share more about what you're trying to do? The AutoMapper NuGet package supports .NET Core so it should run fine within ASP.NET Core. But beyond that, it depends on the specific scenario you have (e.g. model binding, using Entity Framework Core, something else...).

I'm using EF core, I have web api project with crud operation. I want to be able to get from client changes for entities, and save them to the db. but in my controller I expose just DTOs, so I'm get exactly the problem I linked to above:
"The above example wouldn't actually work since we're trying to apply a patch document of type DTO.Expense against an entity of type Entities.Expense. I think being able to support this use case is kind of important, as otherwise we're limiting being able to use Patch in all but the simplest of APIs."
In this link @sergkr describe the exact problem I'm having (starting in: "Second, it's common to have separate classes for domain objects ("entities") versus what you send/receive over the wire ("DTOs")".
thanks

Hmm, looking at the code it looks like you could do this:
public IActionResult UpdateProduct(int id, JsonPatchDocument<ProductDto> productDto)
{
    Product existingProduct = db.GetProduct(id);
    productDto.ApplyTo(existingProduct);
}

Though I agree that using AutoMapper in this would be quite complex.
This is not an area we plan to make any significant changes in, but if you (or anyone) has a prototype or something we could look at, we can certainly reconsider.

Hi guess you mean
 ProductDto existingProduct = db.GetProduct(id);
otherwise
'productDto.ApplyTo(existingProduct);'
not compiled.
but then, if I want to verify the changes (for example - if I allow to change only part of
Account) - I should test it on the controller (and I want to avoid logic on controller), because business logic layer dont know DTOs.
eventually, I succeeded use it after I removed the type from JsonPatchDocument,
so that instead of 'JsonPatchDocument patch' I use just 'JsonPatchDocument patch'.
then I can pass patch to BL layer, and apply it (also) to Account instance.
What do u think about my solution? thanks!

Now I saw that if I use JsonPatchDocument without entity type, I can't validate it against ModelState
public AccountDTO Patch(int id, [FromBody]JsonPatchDocument patch)
        {
            var accountToTestValidation = new AccountDTO();
            patch.ApplyTo(accountToTestValidation, ModelState); //this wont compile if I dont pass `JsonPatchDocument <AccountDto> ` param

@arielcloud yeah looking at the API it looks like there's a generic version of ApplyTo that you can't use (because it uses T for both sides), but also a non-generic version of ApplyTo that can take any two types and it matches based on property names, etc.
Did you get this working? What is the validation issue you were having?

Yeah, it is working.
the problem is that if I use it so, I cant use the ApplyTo version which take the ModelState param  and fill it with the validation errors the patch cause (if exists) to use later with
if (!ModelState.IsValid)
            {
                throw new InvalidDataException(

but I still can manually check the patched object validation by
patch.ApplyTo(patched);
TryValidateModel(patched);

Now I saw that I can do that simply by using CreateMap(typeof(JsonPatchDocument<>), typeof(JsonPatchDocument<>));             CreateMap(typeof(Operation<>), typeof(Operation<>));
that's all!

@Eilon
An other way to work this out could be able to convert an Operation<TEntity> to its equivalent Action<TEntity>. or to Func<TEntity, TEntity
so that after converting from JsonPatchDocument to JsonPatchDocument as @arielcloud suggested, one can work with just a list of transformation to apply in a service.
In this way, only the controller layer would depends on the JsonPatchDocument whilst other layers won't

Closing because there are no plans to implement this.

This is a common scenario where your DTO/viewmodel could be different from the Entities. In this case how to use this JSONPatch with ApplyTo function.  If possible, I think you should work on providing this feature.

Hi, it looks like you are posting on a closed issue/PR/commit!
We're very likely to lose track of your bug/feedback/question unless you:

Open a new issue
Explain very clearly what you need help with
If you think you have found a bug, include detailed repro steps so that we can investigate the problem

Thanks!


How to properly move submodules to GPU?

I've coded up a TransformerEncoder that relies on submodules. Specifically, I have a main lightning module (MainTransfomer.py) which has 2 sub (regular torch)modules. 1 is BertModel and 1 is a custom TransformerEncoderLayer.
The TransformerEncoderLayer has 2 submodules of its own named MultiHeadAttn and PosFeedForward. When I try to run my code via the lightning trainer, it seems that all my tensors/params are being moved to GPU except for when it tries to do operations within the MultiHeadAttn and PosFeedForward modules. Specifically it seems the weight matrix of a nn.Linear() within MultiHeadAttn is still sitting on cpu.
So my question boils down to what is the correct way to ensure all submodules are moved to the (correct) GPU within the lightning framework? In normal PyTorch I would just explicitly call .to(device) but I've read that it is not recommended to do this within lightning.
if I explicitly set .to(device) on my TransformerEncoderLayer in my MainTransformerEncoder init I don't run into this issue.
I can supply code if need be, but the general setup is as described above. Main lightning module inits a torch module, which itself has 2 sub torch modules (attn & ffnn). The attn and ffnn modules don't seem to be moved to GPU by the lightning trainer.
Env:

Ubuntu16.04
conda/py3.8
Lightning v0.7.1

Strange. Lightning should move all your modules to the GPU given they are LightningModules or are part of one. To identify and fix a potential bug we would need a minimal example. Could you post it as a google colab notebook? That would be super helpful.

Hey, I tried moving my code into collab this morning. I tried to find a balance between leaving the code as is and putting in the neccessary bits to just show the error. In the colab notebook I've confirmed the code runs beyond the point of the original error when using CPU. However, when switching to GPU within the colab I'm getting a vague error of "CUDA error: An illegal memory access was encountered".
Here's the notebook containing the code: link


Fix bug in event monitor where subscriptions would not be terminated properly on WebSocket error

Closes: #293
Description
This PR fixes a potential bug where subscriptions would not be terminated properly if the WebSocket client encountered an error because
a) we were emptying the subscriptions vector when passing it to select_all, and
b) we were not closing the client explicitly
To test this PR:
Use the following configuration file:
title = "IBC Relayer Config Example"

[global]
timeout = "10s"
strategy = "naive"

[[chains]]
  id = "chain_A"
  rpc_addr = "localhost:26657"
  account_prefix = "cosmos"
  key_name = "testkey"
  store_prefix = "ibc"
  client_ids = ["clA1", "clA2"]
  gas = 200000
  gas_adjustement = 1.3
  gas_price = "0.025stake"
  trusting_period = "336h"

[[connections]]

In one terminal, spawn a Tendermint node running the KV Store app:
docker run -it --rm -v "/tmp/tendermint:/tendermint" tendermint/tendermint init
docker run -it --rm -v "/tmp/tendermint:/tendermint" -p 26657:26657 tendermint/tendermint node --proxy_app=kvstore

In another terminal, from the relayer-cli folder, run the listen command of the relayer:
$ cargo run -- -c path/to/config/file.toml listen

Expected output:
Oct 06 18:01:35.126  INFO relayer_cli::commands::listen: spawning event monitor for chain.id=chain_A
Oct 06 18:01:35.159  INFO relayer_cli::commands::listen: spawning main event handler
Oct 06 18:01:35.160  INFO relayer::event_monitor: running listener for chain.id=chain_A
Oct 06 18:01:35.160  INFO relayer::event_handler: running IBC Event Handler
Oct 06 18:01:35.343  INFO relayer::event_handler: Chain chain_A pushed {"NewBlock":{"height":"322"}}
Oct 06 18:01:35.863  INFO relayer::event_handler: Chain chain_A pushed {"NewBlock":{"height":"323"}}

…

For contributor use:

[ ] Unit tests written
[ ] Added test to CI if applicable
[ ] Updated CHANGELOG_PENDING.md
[ ] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[ ] Updated relevant documentation (docs/) and code comments
[ ] Re-reviewed Files changed in the Github PR explorer

Codecov Report

Merging #290 into master will increase coverage by 23.4%.
The diff coverage is 61.9%.

@@            Coverage Diff            @@
##           master    #290      +/-   ##
=========================================
+ Coverage    13.6%   37.1%   +23.4%
=========================================
  Files          69     120      +51
  Lines        3752    7729    +3977
  Branches     1374    2726    +1352
=========================================
+ Hits          513    2873    +2360
- Misses       2618    4428    +1810
+ Partials      621     428     -193

Impacted Files
Coverage Δ

modules/src/events.rs
0.0% <0.0%> (ø)

modules/src/ics02_client/events.rs
0.0% <ø> (ø)

modules/src/ics02_client/raw.rs
0.0% <0.0%> (ø)

modules/src/ics03_connection/context.rs
0.0% <0.0%> (ø)

modules/src/ics03_connection/error.rs
16.6% <0.0%> (-16.7%)
:arrow_down:

modules/src/ics04_channel/error.rs
75.0% <ø> (+50.0%)
:arrow_up:

modules/src/ics04_channel/packet.rs
0.0% <0.0%> (ø)

modules/src/ics07_tendermint/client_def.rs
0.0% <0.0%> (ø)

modules/src/ics07_tendermint/error.rs
75.0% <ø> (+75.0%)
:arrow_up:

modules/src/ics18_relayer/error.rs
0.0% <0.0%> (ø)

... and 218 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 5024717...9122723. Read the comment docs.

FYI, cargo build on master fails following the work in this PR:
   Compiling relayer v0.0.3 (/Users/adi/Hammers/ibc-rs/relayer)
error[E0432]: unresolved imports `tendermint_rpc::query`, `tendermint_rpc::query`
 --> relayer/src/event_monitor.rs:4:5
  |
4 |     query::EventType, query::Query, Subscription, SubscriptionClient, WebSocketClient,
  |     ^^^^^             ^^^^^ could not find `query` in `tendermint_rpc`
  |     |
  |     could not find `query` in `tendermint_rpc`

error: aborting due to previous error

For more information about this error, try `rustc --explain E0432`.
error: could not compile `relayer`.

To learn more, run the command again with --verbose.
warning: build failed, waiting for other jobs to finish...
error: build failed


Migrate Radicle -> auth to heartwood

Try:

create new identity
remove identity
select default
set up multi-delegate project (a project with 2 identities with write permissions)

Document what needs to change
Add label with estimate (hours, days, weeks)
Start working on the fixes

**Steps to create a new identity **

export RAD_PASSPHRASE=.....
rad auth --stdin

Steps to remove identity

rad self
delete the last three directories

Select default
Create multiple identities
To create new identities we have to change the RAD_HOME to a different path

export RAD_HOME=

Add identity

First, we have to export RAD_PASSPHRASE variable
Then run rad auth --stdin

Remove identity

We have to modify the existing regex to find the new directories

@Stelios123 so, the plan here was to:

remove the list of identities and identity management features we had pre-heartwood.
in the settings, on top of the path to the CLI, we need to be able to set RAD_HOME (which is what defines the identity used). Talk to me if you want to figure out the exact text, etc.
When a passphrase is required, we need to request and store it securely, with the means provided to us by the platform. Does this look like the right information page: https://plugins.jetbrains.com/docs/intellij/persisting-sensitive-data.html#how-to-use ?

@Stelios123 after our chat today, let me try to clarify a little further:

we should add another input field in the settings, immediately under the path to the rad cli.
the label should read: "Radicle Node storage path (RAD_HOME)"
a Test button can be added that will work like this:

if it doesn't find the key, it asks the user to provide a passphrase, in order to create one
if it finds the key but doesn't have the passphrase, it asks the user to provide it in order to unlock it (unless there is already integration with, e.g., the SSH agent and we can unlock it that way)
if it finds the key and a working passphrase, it tells the user that the path is ok and returns the DID of the identity that is found in that path.

Do we want this?

@Stelios123 this issue here is about auth so probably not the right place to talk about it.
Please see this comment instead.


T06.08-Solution-PreferenceSummary

My solution.

PR is requesting we merge changes from later lesson branches into earlier lesson branches.
We will not be doing this as these branches are set-up this way on purpose for the course.


hide table of contents on middle sceen sizes

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


feat(COMPONENT): Improve CdkTable keyboard accessibility when combined with MatSort

Feature Description
A way to easily access MatSort within the template when used in conjunction with cdk-header-cell
Use Case
Improve the implementation of CdkTable navigation for users.
Additional Detail
When using MatSort, an accessible sort header can be conveniently added to CdkTable. However, when implementing arrowkey accessibility per W3's datatable example,  it is hard to listen to keydown events from the sort header within the table template. It would be nice to have an easier way to control keydown events on the sort headers.
Also, this is my first feature request, so sorry in advance if I did something wrong!

What makes it difficult to listen to keydown events? It's a little hard to tell without knowing what you're trying to do.

That’s a great question.  I’m trying to implement a roving tabindex on CdkHeaderCell or MatSort (one or the other). For grids requiring a roving tabindex, I have been using a directive (the directive specifies the 2d coordinates of a cell within a table, and pressing the arrow key moves the focus and tabindex by querying the proper coordinates). The issue that I’ve run into is that every MatSort directive appends div elements that have a tabindex of 0. Because there is no direct DOM access to the MatSort directive, I’m currently listening for changes to MatSort via @ViewChildren and setting the tabindex of each MatSort header to -1. If there was direct DOM access to MatSort, I would imagine it would be easier to implement keyboard-accessible tables, but I also could be going about this the wrong way?  I hope this makes sense – and no worries if not…  From: Kristiyan KostadinovSent: Wednesday, 12 May 2021 2:21 amTo: angular/componentsCc: davidhicks980; AuthorSubject: Re: [angular/components] feat(COMPONENT): Improve CdkTable keyboard accessibility when combined with MatSort (#22667) What makes it difficult to listen to keydown events? It's a little hard to tell without knowing what you're trying to do.—You are receiving this because you authored the thread.Reply to this email directly, view it on GitHub, or unsubscribe.


Configuration Options

I tried tile38 in a local docker container and it works as intended. I now like to take the next step and deploy it to our server. However I can't find any documentation of the configuration options. I like to configure (at least) a basic auth option for the client and maybe limit which client (protocols) can connect.
Can anyone provide a link to the configuration options and how to apply them?

You can use the local doc window to remotely connect to the server's services and set the configuration .
for example：
First open the doc window, CD enter the directory of tile38_cli .
Second

Of course, you can also write code to control tile38 with commands in the program and initialize the configuration .
If you use Java, this link might be useful for you：https://github.com/HkMoyun/tile38-client

I'm not asking about how I can interact with the server. I'm asking how I can configure it. I like to set environment variables to define the auth password for example.


angularclient client routing not working for successful token response

When I press the button “Yes, Allow” at the screen “angularclient is requesting your permission” I have an error:
GET https://localhost:44347/authorized 404 (not Found) :44347/authorized#id_token=eyJhbGciOiJSUzI1NiIsImtp…

Need to remove this from the server routes in the startup
var angularRoutes = new[] {
                "/forbidden",
                "/authorized",
                "/authorize",
                "/unauthorized",
                "/dataeventrecords",
                "/dataeventrecords/create",
                "/dataeventrecords/edit",
                "/logoff",
                "/securefile",
                "/securefile/securefiles",
            };

            app.Use(async (context, next) =>
            {
                if (context.Request.Path.HasValue && null != angularRoutes.FirstOrDefault(
                    (ar) => context.Request.Path.Value.StartsWith(ar, StringComparison.OrdinalIgnoreCase)))
                {
                    context.Request.Path = new PathString("/");
                }

                await next();
            });

fixed in master


Streamline Logging Across TestNG

Closes #2646
We now use sl4j apis for our logging.
Internally TestNG makes use of SimpleLogger
(Slf4j-simple) bindings for logging purposes.
Fixes #2646  .
Did you remember to?

[ ] Add test case(s)
[X] Update CHANGES.txt
[X] Auto applied styling via ./gradlew autostyleApply

We encourage pull requests that:

Add new features to TestNG (or)
Fix bugs in TestNG

If your pull request involves fixing SonarQube issues then we would suggest that you please discuss this with the
TestNG-dev before you spend time working on it.
Note: For more information on contribution guidelines  please make sure you refer our Contributing section for detailed set of steps.

@juherr

But I'm not feeling it good in term of side-effect: it will need a documentation update.

We can take care of updating the TestNG documentation site once the release goes out.


added FDFullscreenPopGesture by @forkingdog

added FDFullscreenPopGesture by @forkingdog
Project URL
https://github.com/forkingdog/FDFullscreenPopGesture
Description
An UINavigationController's category to enable fullscreen pop gesture in an iOS7+ system style with AOP.
Why it should be included to awesome-ios (optional)
a nice idea to enable fullscreen pop gesture in simple code (just one class), GitHub stars >4000!
Checklist

[x] Only one project/change is in this pull request
[x] Addition in chronological order (bottom of category)
[x] Supports iOS 9 / tvOS 10 or later
[x] Supports Swift 3 or later
[x] Has a commit from less than 2 years ago
[x] Has a clear README in English

          1 Warning

:warning:
Found 4 link issues, a project collaborator will take care of these, thanks :)

Link issues by awesome_bot

Line
Status
Link

8
521
https://img.shields.io/badge/PRs-welcome-brightgreen.svg

400
Dupe
https://github.com/rob-nash/Records

639
301
https://github.com/lucdion/FlexLayout redirects tohttps://github.com/layoutBox/FlexLayout

2768
302
http://us11.campaign-archive1.com/home/?u=cd1f3ed33c6527331d82107ba&id=532dc7fb64 redirects tohttps://us1.campaign-archive.com/home/?u=cd1f3ed33c6527331d82107ba&id=532dc7fb64

  Generated by :no_entry_sign: Danger

Can you open a new pull request with only one commit? Thanks


Make Zipkin timeout configurable

Currently connecting to Zipkin has a hardcoded 1-second timeout, which for me isn't long enough. When I connect to a Zipkin server behind a VPN from my local PC on the other side of the world this times out.
The workaround that I did now was to create my own Reporter bean. This would have been much simpler if it was just a config property, like the message timeout.
I'll create a PR if the team thinks this is a good feature to add.

Makes sense, please file a pr :)

This hardcoded 1-second timeout  means that there's a pre-check on the sender,  and that check has a timeout of one second. If this check fails, it appears that only a WARN log will be printed.  Is it necessary to specifically make this timeout configurable?
If you are using the default sender, which use RestTempalte.  It looks like prior to version 3.1.0, RestTemplate had a hardcoded connect&read timeout of 500 milliseconds (You do need to create your own Reporter),  but after 3.1.0 provides ZipkinRestTemplateProvider, you can inject your custom RestTemplate.

Please upgrade to Micrometer Tracing. Spring Cloud Sleuth is feature complete and out of OSS support.


[fix #1512] add not in comparison into Trigger Agent

Add not in comparison for doing != on arrays. Also warning about != and arrays was added.
Fixes #1512.

Thanks @andrey-yantsen, would you mind updating the spec too?

Updating spec was a great idea :)
Give me an example for proper rewriting of rule_values.any? and I'll squash commits into one.

I can not agree with it :) It's quite strange to compare entire list inside rule_values.any? — according to your suggestion exactly same operation will be performed for each element of the list. Also — .include? operation should actually be performed for any ==.

You're right, I misunderstood.
Want to add in next to your not in option at the top of the if?

Actually in is quite strange operation because it is exactly the same thing as field==value. I've added it, but I'm confused a little by this decision :)

Yea, that's a good point.  I'm tired this evening.  We don't need it :)

Ok :)
I've squashed the commits into one, check it once again please.

Thanks @andrey-yantsen, works great!


Added query to the latest release3

Added query

select * from job details where criminal_records = "felony" is retrieving proper results. this change is valid


console.log in Webview only Logs First Argument

Steps to Reproduce:

Create a html preview that calls console.log(1, 2)

Bug
Note that only 1 is logged

Looks like an electron issue: https://github.com/electron/electron/issues/8090

Seems this got wontfixed by chromium: https://github.com/Microsoft/vscode/issues/30425


Custom y2 axis labels still clipped in some cases

It appears that if you update the chart data without updating any options, the widths are not recalculated and the y2 labels still get clipped. This behavior occurs on master.
Related to #85.

Hi @marshall007 Is this problem still existing?

I will close this for now.
However, please let us know if this problem still exists!

@chaosmail sorry for the delay! I haven't noticed this in a while so I assume it's fixed now.


Add toBeDateString matcher

Copy of https://github.com/jest-community/jest-extended/pull/252.

Added a new matcher to check whether a string is in a date format.
A use-case example: when serializing a JS object with date values, those become strings, and when comparing against the returned json, it can be useful to use this matcher.
Housekeeping

[x]  Unit tests
[x]  Documentation is up to date
[x]  No additional lint warnings
[x]  Typescript definitions are added/updated where relevant

@bompus commented

This is likely a dupe of #164 and would be super useful.


DTSO-15909 - Custom name functionality

Jira link (if applicable)
DTSPO-15909
Change description
Optional input variable to allow setting of fully custom resource name. Some existing resources can't be migrated to the new terraform module without being destroyed, due to incompatible resource name.

can you provide an example please?

pip-ss-ithc-java-appinsights, the env is in the middle of this and existing module wants to append an env on the end. Doing so will trigger a destroy on the existing resource.

can you provide an example please?

pip-ss-ithc-java-appinsights, the env is in the middle of this and existing module wants to append an env on the end. Doing so will trigger a destroy on the existing resource.

I would just check with the team and see if they are ok with re-creation, that's really far off our standard naming

can you provide an example please?

pip-ss-ithc-java-appinsights, the env is in the middle of this and existing module wants to append an env on the end. Doing so will trigger a destroy on the existing resource.

I would just check with the team and see if they are ok with re-creation, that's really far off our standard naming

Sounds sensible actually - will see what they say and have this as a backup I guess.


Update RyuJIT package

Rename references to ryujit to clrjit
Add verification of JIT-EE interface revision
Add .bat file to automate generation of JIT-EE interface wrapper

The name of the jit .dll is same for both CoreRT and other runtimes now. It does not collide in fatal way yet, but it will start colliding eventually once the CoreCLR update propagates all the way to CoreRT. It will need to be fixed up in CoreRT packaging, probably by installing the jit .dll for the CoreRT compiler into a different directory.


Search Page: Misc.

[x] options for fuzzy, starts-with, contains, exact  ES (kg)
[ ] result content display (kg)

[ ]      - modify ES search to get some child info
[x]      - differentiate, index vs db results visually
[x]      - do away with detail box?
[ ]      ~- explain index vs db results~

[x] - css fussing (kg)
[x] - team feedback in March

Closing; remaining task moved to #193


Cannot install - error related to linking

Hello!
Thank you for writing this library. I want to control the Go1 robot's low level controller using python.
But I cannot install this library (see attached log)
┌─[charlie@pop-os] - [~/Desktop/unitree_legged_latest_sdk/build] - [2022-07-13 04:59:43]
└─[0] <git:(master 1304ed0✈) > make
[ 10%] Building CXX object CMakeFiles/example_position.dir/example/example_position.cpp.o
In file included from /usr/include/boost/smart_ptr/detail/sp_thread_sleep.hpp:22,
                 from /usr/include/boost/smart_ptr/detail/yield_k.hpp:23,
                 from /usr/include/boost/smart_ptr/detail/spinlock_gcc_atomic.hpp:14,
                 from /usr/include/boost/smart_ptr/detail/spinlock.hpp:42,
                 from /usr/include/boost/smart_ptr/detail/spinlock_pool.hpp:25,
                 from /usr/include/boost/smart_ptr/shared_ptr.hpp:29,
                 from /usr/include/boost/shared_ptr.hpp:17,
                 from /home/charlie/Desktop/unitree_legged_latest_sdk/include/unitree_legged_sdk/loop.h:14,
                 from /home/charlie/Desktop/unitree_legged_latest_sdk/include/unitree_legged_sdk/unitree_legged_sdk.h:11,
                 from /home/charlie/Desktop/unitree_legged_latest_sdk/example/example_position.cpp:5:
/usr/include/boost/bind.hpp:36:1: note: ‘#pragma message: The practice of declaring the Bind placeholders (_1, _2, ...) in the global namespace is deprecated. Please use <boost/bind/bind.hpp> + using namespace boost::placeholders, or define BOOST_BIND_GLOBAL_PLACEHOLDERS to retain the current behavior.’
   36 | BOOST_PRAGMA_MESSAGE(
      | ^~~~~~~~~~~~~~~~~~~~
[ 20%] Linking CXX executable example_position
/usr/bin/ld: /home/charlie/Desktop/unitree_legged_latest_sdk/lib/cpp/amd64/libunitree_legged_sdk.a(loop.cpp.o): relocation R_X86_64_PC32 against undefined hidden symbol `_ZTCN5boost10wrapexceptINS_17bad_function_callEEE0_NS_16exception_detail10clone_implINS3_19error_info_injectorIS1_EEEE' can not be used when making a PIE object
/usr/bin/ld: final link failed: bad value
collect2: error: ld returned 1 exit status
make[2]: *** [CMakeFiles/example_position.dir/build.make:97: example_position] Error 1
make[1]: *** [CMakeFiles/Makefile2:91: CMakeFiles/example_position.dir/all] Error 2
make: *** [Makefile:136: all] Error 2

I think the /home/charlie/Desktop/unitree_legged_latest_sdk/lib/cpp/amd64/libunitree_legged_sdk.a provided was not compiled with the right option https://stackoverflow.com/questions/19768267/relocation-r-x86-64-32s-against-linking-error
If it's my fault that it won't compile, can you let me know how to compile it such that it works? thank you

The lib has -fPIC flags.
It seems like boost releated issue. You can check your own boost version at
/usr/include/boost/version.hpp

I've got a same error on Ubuntu 22.04.

Ubuntu 22.04
CMake 3.22.1
Boost 1.74
g++ 11.2.0

On Ubuntu 20.04, it compiles fine.

Ubuntu 20.04
CMake 3.16.3
Boost 1.71
g++ 9.4.0

Any news on this?

I've got a same error on raspberry pi 4B. It seemed that can not find the lib.
the log file:
bpesun@raspberrypi:~/Downloads/unitree_legged_sdk-master/build $  bpesun@raspberrypi:~/Downloads/unitree_legged_sdk-master/build $ cd /home/bpesun/Downloads/unitree_legged_sdk-master bpesun@raspberrypi:~/Downloads/unitree_legged_sdk-master $ ls CMakeLists.txt  example  example_py  include  lib  LICENSE  README.md bpesun@raspberrypi:~/Downloads/unitree_legged_sdk-master $ mkdir build bpesun@raspberrypi:~/Downloads/unitree_legged_sdk-master $ cd build bpesun@raspberrypi:~/Downloads/unitree_legged_sdk-master/build $ ls bpesun@raspberrypi:~/Downloads/unitree_legged_sdk-master/build $ cmake ../ -- The C compiler identification is GNU 10.2.1 -- The CXX compiler identification is GNU 10.2.1 -- Detecting C compiler ABI info -- Detecting C compiler ABI info - done -- Check for working C compiler: /usr/bin/cc - skipped -- Detecting C compile features -- Detecting C compile features - done -- Detecting CXX compiler ABI info -- Detecting CXX compiler ABI info - done -- Check for working CXX compiler: /usr/bin/c++ - skipped -- Detecting CXX compile features -- Detecting CXX compile features - done -- CMAKE_SYSTEM_PROCESSOR: armv7l -- Configuring done -- Generating done -- Build files have been written to: /home/bpesun/Downloads/unitree_legged_sdk-master/build bpesun@raspberrypi:~/Downloads/unitree_legged_sdk-master/build $ make Scanning dependencies of target example_walk [ 10%] Building CXX object CMakeFiles/example_walk.dir/example/example_walk.cpp.o In file included from /usr/include/boost/smart_ptr/detail/sp_thread_sleep.hpp:22,                  from /usr/include/boost/smart_ptr/detail/yield_k.hpp:23,                  from /usr/include/boost/smart_ptr/detail/spinlock_gcc_atomic.hpp:14,                  from /usr/include/boost/smart_ptr/detail/spinlock.hpp:42,                  from /usr/include/boost/smart_ptr/detail/spinlock_pool.hpp:25,                  from /usr/include/boost/smart_ptr/shared_ptr.hpp:29,                  from /usr/include/boost/shared_ptr.hpp:17,                  from /home/bpesun/Downloads/unitree_legged_sdk-master/include/unitree_legged_sdk/loop.h:14,                  from /home/bpesun/Downloads/unitree_legged_sdk-master/include/unitree_legged_sdk/unitree_legged_sdk.h:11,                  from /home/bpesun/Downloads/unitree_legged_sdk-master/example/example_walk.cpp:5: /usr/include/boost/bind.hpp:36:1: note: ‘#pragma message: The practice of declaring the Bind placeholders (_1, _2, ...) in the global namespace is deprecated. Please use <boost/bind/bind.hpp> + using namespace boost::placeholders, or define BOOST_BIND_GLOBAL_PLACEHOLDERS to retain the current behavior.’    36 | BOOST_PRAGMA_MESSAGE(       | ^~~~~~~~~~~~~~~~~~~~ [ 20%] Linking CXX executable example_walk /usr/bin/ld: cannot find -lunitree_legged_sdk collect2: error: ld returned 1 exit status make[2]: *** [CMakeFiles/example_walk.dir/build.make:103: example_walk] Error 1 make[1]: *** [CMakeFiles/Makefile2:103: CMakeFiles/example_walk.dir/all] Error 2 make: *** [Makefile:149: all] Error 2

sorry. I changed the raspberry pi4 OS form 32bits to 64bits and the exmaple worked.

https://github.com/unitreerobotics/unitree_legged_sdk/pull/56

hi Velythyl,
Thanks for your files.
You said 22.04 can run your repo.  I can run you repo,  but when i run the python example,  it can not find the lib. The log:
pi@pi-desktop:~/Desktop/unitree_legged_sdk-master/example_py$ python3 example_walk.py
Traceback (most recent call last):
  File "/home/pi/Desktop/unitree_legged_sdk-master/example_py/example_walk.py", line 8, in <module>
    import robot_interface as sdk
ModuleNotFoundError: No module named 'robot_interface'

platform: raspberry Pi4B
os: ubuntu desktop 22.04.1 LTS
python: Python 3.10.4
CMake: 3.22.1
Boost: 1.74
g++ : 11.2.0
It seemed it can not find the python lib， the version of python is too high？
Thanks

I have the same error on ubuntu 22.04:
/usr/bin/ld: /home/amb/Workspace/unitree_legged_sdk/lib/cpp/amd64/libunitree_legged_sdk.a(loop.cpp.o): relocation R_X86_64_PC32 against undefined hidden symbol   _ZTCN5boost10wrapexceptINS_17bad_function_callEEE0_NS_16exception_detail10clone_implINS3_19error_info_injectorIS1_EEEE' can not be used when making a PIE object /usr/bin/ld: final link failed: bad value collect2: error: ld returned 1 exit status make[2]: *** [CMakeFiles/example_position.dir/build.make:97: example_position] Error 1 make[1]: *** [CMakeFiles/Makefile2:91: CMakeFiles/example_position.dir/all] Error 2 make: *** [Makefile:136: all] Error 2
How to fix it? or when will the corrected version of the sdk be released?

anything new？

Hello, our SDK does not support ubuntu 22.04, you can try  ubuntu 18.04

Hi, here is my way to compile the sdk with ubuntu 22:

Download the older libboost-1_66
unzip the downloaded libboost
./bootstrap.sh
./b2
sudo ./b2 install

redo the cmake .. and make the sdk folder.


Hashing

Passlib framework

Documentation: https://passlib.readthedocs.io/en/stable/

This is on the back burner for now. Error handling first. Will open this when the time comes.

This is on the back burner for now. Error handling first. Will open this when the time comes.

No longer using passlib, using bcrypt library.
I have done some testing and have a modified branch of dev. Working to further implement the work I've done to main.py, but the splashscreen is almost done. Will have to modify the README.md and update the install.py file.

Hashing the main master key is finished.


If installed through mac app, present messaging about how to migrate

Detect if we were installed through the Fabric Mac App and then present some information about migrating to used a better-supported install method as described in https://github.com/fastlane/fastlane/issues/11586

Shipped


Introduce exclude_fields

Idea is that it should work the same as fields() but instead exclude selected fields from the query

added in #39


updating of processed files

If it’s there, don’t worry, it just wasn’t visible in the browser and they usually are.
j
From: Valentina Mignosa @.>
Sent: 04 June 2021 08:51
To: ISicily/ISicily @.>
Cc: Jonathan Prag @.>; Comment @.>
Subject: Re: [ISicily/ISicily] updating of processed files (#165)
@valentinamignosa commented on this pull request.

In inscriptions/ISic020476.xmlhttps://github.com/ISicily/ISicily/pull/165#discussion_r645365808:

@@ -170,7 +180,7 @@

     <body>

         <div type="edition" xml:space="preserve" xml:lang="grc" resp="#VM">

             <ab>

               <lb n="1"/>Ἀριστάρχο̄

               <lb n="1"/><persName><name>Ἀριστάρχο̄</name></persName>

Odd... I can see it, can't you?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHubhttps://github.com/ISicily/ISicily/pull/165#discussion_r645365808, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AF7LEQWRZFHPRO4G7YXYRL3TRCAV3ANCNFSM45JG5BIQ.


Upgrade to OpenSSL 1.1.1c

Looks like this MUSL break will be harder to hack around than the last one :(
https://github.com/openssl/openssl/blob/OpenSSL_1_1_1c/crypto/rand/rand_unix.c#L21-L23

r=me with a green build, although I don't know how best to fix musl.

MUSL build is green - does the patch logic look okay to you?


Event Type can't have special characters.

hey me again. 👋👋

An event Type can't have a special character in the string.  like  % , & , @ , # , § ...
again sorry to bother you. and again not a big issue. but it seems the Type needs validation.
Regards.

i dit it with a Regex.Replace(row["Service"].ToString(), @"[^0-9a-zA-Z]+", "") from the backend but don't really know where to put it in the front-end...

Hi, I think the problem is with jquery selector when using characters such as @, & #, etc.. Please check my latest commit :slightly_smiling_face:


Remove non-breaking text from popover examples

Remove non-breaking text, now that popover's have a max width.

@interactivellama, Tooltips (i.e. blue background Popover) shouldn't have a close x.


Cannot undo stash and controller pair (Ledger Nano S)

I have created a stash and a controller wallet with Ledger Nano S and the Polkadot-ui-js. Setting session keys with Ledger Nano S does not work with the latest version #74, hence I tried to set a new non-ledger controller to set the session keys however setController is also an unsupported txn version #79 (why is it closed, txn version does not work). So there is no way to undo associating the stash and controller nor to set the session key. Any advice?
Ledger Nano S
v5.30.0

requires this PR to be merged and released https://github.com/LedgerHQ/app-kusama/pull/9

@jleni thanks, the new Kusama XL version supports setController, so this ticket can be closed. I haven't checked #74 but I think it's a different problem than runtime version.
The same upgrade will happen to polkadot soon, please make sure the new versions are rolled out when it happens.

AFAIK polkadot upgrades have been merged and deployed already


Kafka Source Connector in pulsar cluster

Describe the bug
Exception in kafka source connector in pulsar cluster function worker
To Reproduce
Steps to reproduce the behavior:

Run Pulsar Cluster - one zookeeper, one broker and three bookie
Ensure Function worker enabled
Create Source for kafka
Running kafka container and produce message to it.
No Exception thrown instead getting hanged . while checking logs no more meaningful errors

Expected behavior
Running Kafka Source Connector through function worker mode in pulsar cluster
Screenshots

Desktop (please complete the following information):

OS: Windows

Additional context

@Esakkimuthu991
What version of the Kafka cluster are you running? What version of Pulsar are you using?

@Esakkimuthu991
What version of the Kafka cluster are you running? What version of Pulsar are you using?

pulsar image - apachepulsar/pulsar-test-latest-version:latest
kafka image - spotify/kafka:latest

There are a couple of things to check:

I am not sure what version of Kafka is spotify/kafka. I think the Kafka connector is using Kafka version <IP_ADDRESS>. You need to make sure your Kafka cluster for testing is compatible with this version.

how do you configure bootstrapServers in the Kafka source connector? Is it reachable in pulsar cluster?

There are a couple of things to check:

I am not sure what version of Kafka is spotify/kafka. I think the Kafka connector is using Kafka version <IP_ADDRESS>. You need to make sure your Kafka cluster for testing is compatible with this version.
how do you configure bootstrapServers in the Kafka source connector? Is it reachable in pulsar cluster?

Earlier I tried the same spotify/kafka with pulsar standalone mode . Its supporting and source connector working fine.
Do we have docker compose file for pulsar cluster running with pulsar 2.5.0 image?

@Esakkimuthu991 you can use this docker-compose file for reference. https://github.com/apache/pulsar/blob/master/docker-compose/standalone-dashboard/docker-compose.yml
I think the important thing is to make sure your function worker is able to connect to your Kafka cluster.

@Esakkimuthu991 you can use this docker-compose file for reference. https://github.com/apache/pulsar/blob/master/docker-compose/standalone-dashboard/docker-compose.yml
I think the important thing is to make sure your function worker is able to connect to your Kafka cluster.

I've already tried with standalone mode pulsar and it is working

Trying to run it on pulsar cluster mode . So it would be helpful if sharing any reference for pulsar cluster 2.5.0

@Esakkimuthu991 do you mean a docker-compose file for cluster setup?
For step-by-step instruction on setting up a cluster using docker, you can check out this demo:
https://github.com/streamnative/tgip/blob/master/episodes/001/demo.md
I walk through it in this TGIP video: https://www.youtube.com/watch?v=1RQSot5tTuU
Hope this is useful for you.

@Esakkimuthu991 do you mean a docker-compose file for cluster setup?
For step-by-step instruction on setting up a cluster using docker, you can check out this demo:
https://github.com/streamnative/tgip/blob/master/episodes/001/demo.md
I walk through it in this TGIP video: https://www.youtube.com/watch?v=1RQSot5tTuU
Hope this is useful for you.

Thanks for the link provided

@Esakkimuthu991 I am assuming you have gotten the information you need. I am closing this issue now. If you need any more help, feel free to leave a comment and reopen this issue.


test (provider): receiptProvider impl of BlockchainProvider2

closes https://github.com/paradigmxyz/reth/issues/10340

thanks @joshieDo for helping me debug & unblock this 🙏 https://github.com/paradigmxyz/reth/pull/10660


[Bug]<EMAIL_ADDRESS>declares overrides

What happened?
I'm trying to use opentelemetry by following
bazel_dep(name = "opentelemetry-cpp", version = "1.16.0", repo_name = "io_opentelemetry_cpp")

Got errr:
ERROR: Error computing the main repository mapping: The MODULE.bazel file of<EMAIL_ADDRESS>declares overrides

Version
Development (host) and target OS/architectures:
Output of bazel --version:
Version of relevant rules from the WORKSPACE or MODULE.bazel file:
Language(s) and/or frameworks involved:
How to reproduce
No response
Any other information?
No response

Bazel 6.3.0 and higher should ignore overrides in dependencies, could you try updating Bazel?

@fmeum thanks,  I will close this issue

Resolved by upgrading bazel


PTVS Python 3.12 support

Perform various actions and document features that are broken for python 3.12
General Editing

[x] Create a new python project, run code to see IntelliSense
[x] Run code in interactive REPL window

Environments

[x] Add/Delete/Activate virtual/Conda/existing environment
[x] View/Install/Uninstall packages using the Python Environments window

Debugging

[x] python debugging
[ ] mixed mode debugging
[ ] attach process

Misc.

[ ] Profiling
[x] Unit Test
[x] Cookiecutter
[x] Version control
[x] Live share

Ref: https://github.com/microsoft/PTVS/pull/6891/files

I was curious why my VS installer still shows only 3.9, and following the threads via profiling I end here. Presumably more work to do before full support for 3.12, but since then there's 3.13 on the horizon, with advanced notice of changed in the beta here:
https://docs.python.org/3.13/whatsnew/3.13.html

Hi @tyeth, thanks for your suggestions! You're right, debugging and profiling will require a significant amount of work to support. Please check out this link for the related profiling task: https://github.com/microsoft/PTVS/issues/7904

Bruh why not just release support without profiling so they can at least not get a message about using 3.12 or even 3.13 or even 3.14.0a0 (cpython main).


Update frontend libs

This builds on @theresaanna's branch to update several of the JS libs. It doesn't resolve the jQuery version problem, but does add a shrinkwrap file to ensure our dependencies stay still.

Thanks @theresaanna !


Fix rendering 0 as only child without attrs - fixes #1752

@StephanHoyer Tip: use next instead of master for your dev branch - really helps a lot. I didn't catch it until others did for me 😟 , but I fixed and reapplied it to the correct branch.


Fix infinite loop in getScoreFromClass

The while loop is infinite if the condition isn't met, as superclass value never changes.

Thanks for your contribution!


🐛 Error while switching the camera from front to back and vice versa, when video recording is in-progress

What's happening?
🐛 Error while switching the camera from front to back and vice versa, when video recording is in-progress.
Following error occurs when i switch the camera while video is recording:
{"cause": {"message": "[capture/inactive-source] The recording failed because the source becomes inactive and stops sending frames.
One case is that if camera is closed due to lifecycle stopped, the active recording will be finalized with this error, and the output will be generated, containing the frames produced before camera closing. Attempting to start a new recording will be finalized immediately if the source remains inactive and no output will be generated.", "stacktrace": "com.mrousavy.camera.InactiveSourceError: [capture/inactive-source] The recording failed because the source becomes inactive and stops sending frames.
One case is that if camera is closed due to lifecycle stopped, the active recording will be finalized with this error, and the output will be generated, containing the frames produced before camera closing. Attempting to start a new recording will be finalized immediately if the source remains inactive and no output will be generated.
at com.mrousavy.camera.CameraView_RecordVideoKt$startRecording$1.accept(CameraView+RecordVideo.kt:64)
at com.mrousavy.camera.CameraView_RecordVideoKt$startRecording$1.accept(CameraView+RecordVideo.kt:52)
at androidx.camera.video.Recorder$RecordingRecord.lambda$updateVideoRecordEvent$6$androidx-camera-video-Recorder$RecordingRecord(Recorder.java:2752)
at androidx.camera.video.Recorder$RecordingRecord$$ExternalSyntheticLambda5.run(Unknown Source:4)
at android.os.Handler.handleCallback(Handler.java:938)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loop(Looper.java:268)
at android.app.ActivityThread.main(ActivityThread.java:8019)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:627)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:997)
"}, "code": "capture/inactive-source", "message": "[capture/inactive-source] The recording failed because the source becomes inactive and stops sending frames.
One case is that if camera is closed due to lifecycle stopped, the active recording will be finalized with this error, and the output will be generated, containing the frames produced before camera closing. Attempting to start a new recording will be finalized immediately if the source remains inactive and no output will be generated.", "userInfo": null}.
Reproduceable Code
<CameraVision
            ref={camera}
            style={StyleSheet.absoluteFill}
            device={device}
            audio={hasMicrophonePermission}
            isActive={
              !showPhotoPreview && !showVideoEditor && appState === 'active'
            }
            photo
            video
            enableZoomGesture
            orientation="portrait"
            enableHighQualityPhotos
          />

Relevant log output
{"cause": {"message": "[capture/inactive-source] The recording failed because the source becomes inactive and stops sending frames.
One case is that if camera is closed due to lifecycle stopped, the active recording will be finalized with this error, and the output will be generated, containing the frames produced before camera closing. Attempting to start a new recording will be finalized immediately if the source remains inactive and no output will be generated.", "stacktrace": "com.mrousavy.camera.InactiveSourceError: [capture/inactive-source] The recording failed because the source becomes inactive and stops sending frames.
One case is that if camera is closed due to lifecycle stopped, the active recording will be finalized with this error, and the output will be generated, containing the frames produced before camera closing. Attempting to start a new recording will be finalized immediately if the source remains inactive and no output will be generated.
	at com.mrousavy.camera.CameraView_RecordVideoKt$startRecording$1.accept(CameraView+RecordVideo.kt:64)
	at com.mrousavy.camera.CameraView_RecordVideoKt$startRecording$1.accept(CameraView+RecordVideo.kt:52)
	at androidx.camera.video.Recorder$RecordingRecord.lambda$updateVideoRecordEvent$6$androidx-camera-video-Recorder$RecordingRecord(Recorder.java:2752)
	at androidx.camera.video.Recorder$RecordingRecord$$ExternalSyntheticLambda5.run(Unknown Source:4)
	at android.os.Handler.handleCallback(Handler.java:938)
	at android.os.Handler.dispatchMessage(Handler.java:99)
	at android.os.Looper.loop(Looper.java:268)
	at android.app.ActivityThread.main(ActivityThread.java:8019)
	at java.lang.reflect.Method.invoke(Native Method)
	at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:627)
	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:997)
"}, "code": "capture/inactive-source", "message": "[capture/inactive-source] The recording failed because the source becomes inactive and stops sending frames.
One case is that if camera is closed due to lifecycle stopped, the active recording will be finalized with this error, and the output will be generated, containing the frames produced before camera closing. Attempting to start a new recording will be finalized immediately if the source remains inactive and no output will be generated.", "userInfo": null}

Camera Device
<CameraVision
            ref={camera}
            style={StyleSheet.absoluteFill}
            device={device}
            audio={hasMicrophonePermission}
            isActive={
              !showPhotoPreview && !showVideoEditor && appState === 'active'
            }
            photo
            video
            enableZoomGesture
            orientation="portrait"
            enableHighQualityPhotos
          />

Device
All devices
VisionCamera Version
2.16.1
Can you reproduce this issue in the VisionCamera Example app?
No, I cannot reproduce the issue in the Example app
Additional information

[X] I am using Expo
[ ] I have enabled Frame Processors (react-native-worklets-core)
[X] I have read the Troubleshooting Guide
[X] I agree to follow this project's Code of Conduct
[X] I searched for similar issues in this repository and found none.

Hey! I just released a new V4 beta (v4.0.0-beta.7) where I fixed a bunch of issues! 💪🚀 Can you test that and see if that fixes the issue for you? 😅
yarn add react-native-vision-camera@beta

You might need to increase the compileSdk to the newest Android SDK (34) if you are behind.

Hey - I think this issue has been fixed in VisionCamera 4.0.0. 🥳
Please try V4 and let me know if you still experience this issue;

if not, please consider 💖 sponsoring me on GitHub 💖 to support the development of VisionCamera and thank me for my time spent on fixing bugs and building new features.
if you still see this issue, please comment and we can re-open this. But please update your native logs with the native (Xcode/Android Studio) logs from running VisionCamera V4 so I can investigate this.


Some click handlers on the maps object stop working when using the cluster-manager

Thanks for stopping by to let us know something could be better!
Summary
As described in the question on stack overflow
https://stackoverflow.com/questions/62397241/supportmapfragment-v2-some-clicklisteres-like-the-oninfowindowclicklistener
I had problems with some click handlers on the maps object after I migrated my code to androidx (and from java to kotlin)
The user "rafon" from there seems to have deeper insights and suggested to file a bug here.
Expected behavior
InfoWindowClickListener and InfoWindowAdapter still be called as before my refactoring.
Observed behavior
InfoWindowClickListener and InfoWindowAdapter not be called any more whereas DragListerer and MapsLongClickListener continue to work as before.
Environment details
Observed on Nexus 5x / Android 8.0.1 and Pixel 3a Android 10
Observed with
implementation "com.google.android.gms:play-services-maps:17.0.0"
implementation "com.google.maps.android:android-maps-utils:1.3.3"
as well as with
implementation 'com.google.android.libraries.maps:maps:3.1.0-beta'
implementation 'com.google.maps.android:android-maps-utils-v3:1.3.1'
Steps to reproduce
Unfortunately my app is quite big and in the middle of refactoring. So demo code from there is not that easy.
But the following should reproduce the error:
Create a maps object with LongClickHandler, MarkerDragListener, InfoWindowClickListener, InfoWindowAdapter
-> Everything should work as expected
As soon as a ClusterHandler is added to the map, InfoWindowClickListener and InfoWindowAdapter are not called any more.
Feel free to ping me if you need more information.
HTH

@sqrt-1764 Thanks for reaching out! To support multiple types of markers/shapes on the map at the same time, v1 and up of this library uses a MarkerManager (and other *Manager objects) for adding markers, instead of adding them directly to the GoogleMap.
There is a migration guide here:
https://github.com/googlemaps/android-maps-utils#adding-a-custom-info-window
Could you please take a look and see if this resolves your problem?

To clarify:
I have more than one "type" of markers in my map:

Positions of geocoded pictures that are to be displayed clustered.
And
POIs and positions of items that are to be displayed unclustered (yet)
For 1) I had implemented the cluster manager. Those markers are not expected to react to click-events.

For 2) I add the markers directly to the map
So from the documentation I would expect that only the clustered markes are to be handled by a MarkerManager. I expect the markes added directly to the map to react to the handlers in the map object. This was the case in my old implementation and still is the case for e.g. the DragListener and the onMapLongClickListener.
So in order to also handle the markers from 2) with a MarkerManager I would have to create an additional Layer on the map and put them there?
Might anyway be the better solution for the future - work for my weekend ;-)

I forgot to clarify:
Once I removed the clusterManager for 1) the markes from 2) started to behave as before ...

So in order to also handle the markers from 2) with a MarkerManager I would have to create an additional Layer on the map and put them there?

For 2), you would just need to add those unclustered markers via a MarkerManager.Collection instead of adding them directly to the GoogleMap object.
See an example in the MultiLayerDemoActivity:
        // Unclustered marker - instead of adding to the map directly, use the MarkerManager
        MarkerManager.Collection markerCollection = markerManager.newCollection();
        markerCollection.addMarker(new MarkerOptions()
                .position(new LatLng(51.150000, -0.150032))
                .icon(BitmapDescriptorFactory.defaultMarker(BitmapDescriptorFactory.HUE_AZURE))
                .title("Unclustered marker"));
        markerCollection.setOnMarkerClickListener(marker -> {
            Toast.makeText(MultiLayerDemoActivity.this,
                    "Marker clicked: " + marker.getTitle(),
                    Toast.LENGTH_SHORT).show();
            return false;
        });

@sqrt-1764 Thanks for the feedback! I'll look at updating the migration guide to make this behavior more obvious.

So in order to also handle the markers from 2) with a MarkerManager I would have to create an additional Layer on the map and put them there?

For 2), you would just need to add those unclustered markers via a MarkerManager.Collection instead of adding them directly to the GoogleMap object.
See an example in the MultiLayerDemoActivity:
        // Unclustered marker - instead of adding to the map directly, use the MarkerManager
        MarkerManager.Collection markerCollection = markerManager.newCollection();
        markerCollection.addMarker(new MarkerOptions()
                .position(new LatLng(51.150000, -0.150032))
                .icon(BitmapDescriptorFactory.defaultMarker(BitmapDescriptorFactory.HUE_AZURE))
                .title("Unclustered marker"));
        markerCollection.setOnMarkerClickListener(marker -> {
            Toast.makeText(MultiLayerDemoActivity.this,
                    "Marker clicked: " + marker.getTitle(),
                    Toast.LENGTH_SHORT).show();
            return false;
        });

Is this supported on Kotlin? I was facing the same issue but it doesn't seem to work with Kotlin

@theshid: I am not the expert here, but I can confirm that Kotlin is supported. My App is in Kotlin and I was able to solve my probems with this information.

It is still not working, I am still having trouble using clustering and CustomInfoWindow at the same time. Here is a sample app that shows the problem ... (Removed all the security features, jus navigate to the "Mapa De Exposiciones Drawer / Fragment").
Here is a sample.
https://drive.google.com/file/d/16Wf2Ei8px49xGzRoMu8C_ATul0WkeJU_/view?usp=sharing

@betolix Any luck on this? I can't seem to use clustering with a custominfowindow either.

The problem is that any ClusterManager's MarkerManager "steals" the GoogleMap's sole OnMarkerClickListener:
https://github.com/googlemaps/android-maps-utils/blob/cb5c01130f2b349ad58c26a9d9be30a7ed1be3db/library/src/main/java/com/google/maps/android/collections/MarkerManager.java#L48
In our app we use android-maps-utils <1.0 with two ClusterManagers, that show two independently clustered data sets. I just tried updating to v2.2.0, and dug for hours why our OnClusterClick/OnClusterItemClickHandlers weren't working anymore for one of the ClusterManagers. Whichever ClusterManager is created last, will take all map click events.


Better validation of action events

Rasa version: Tested on 2.3.4, bug should still exist on currently latest main@eb408bb2d216de3a1cd1f72799d7b80eceeb1fd2
Rasa SDK version (if used & relevant): 2.3.1
Rasa X version (if used & relevant): 0.36.0 and later (not relevant however)
Python version: 3.8.6
Operating system (windows, osx, ...): Mac OS 11.2.1
Issue:
This came out of this Rasa X issue (see @tmbo's comment here for a summary).
The validation of action responses are quite optimistic, allowing actions to create invalid events (i.e. "entities": {} instead of the valid "entities": []), resulting on a bunch of different issues downstream in Rasa X, since so far Rasa X seems to trust that events sent over the broker are valid.
Here's a handy example repo made by @rgstephens to illustrate the issue: RasaHQ/action-execute-convo-issue. Note that this is actually a bug in the action code that should, however, be caught by Rasa OSS and have the error (invalid event) logged.

Same validation should be applied to events getting send to the http server of rasa open source (eg in the append events endpoint)

@TyDunn A customer is reporting that this issue is affecting them, any chance this can get bumped up in priority?


add codecov

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@4763821). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main      #79   +/-   ##
=======================================
  Coverage        ?   47.30%
=======================================
  Files           ?        5
  Lines           ?      241
  Branches        ?        0
=======================================
  Hits            ?      114
  Misses          ?      117
  Partials        ?       10

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 4763821...32ccff6. Read the comment docs.


Default file extension

Would be nice to have cleaner URLs such as
http://server/support    (really support.md on disk)
http://server/contact    (really contact.md on disk)
http://server/todo       (really todo.md on disk)
and when Wikitten notices there's no extension, it implies ".md" at the end. This allows a "pretty" URL and on disk, still stored as MarkDown. This would require tweaks of the tree.php as well. I made this change in index.php, just after $request_uri = parse_url($_SERVER['REQUEST_URI']);
if( strlen(pathinfo($request_uri['path'], PATHINFO_EXTENSION)) === 0) {
    $request_uri['path'] .= ".md";
}

but I think there's more work to be done

Hi,
I have implemented this on a forked project below.
https://github.com/nicosomb/Wikitten/pull/8
Hope it will help.

THANK YOU!!!!!!

We can't :(
Maybe @victorstanciu will!

@stevenandres you can close your own issue here in reply.. press close on comment instead of comment

closing


openapi-generator-cli: Use headless jre

Motivation for this change

Remove unnecessary dependency on linux gui libs.
Add a test to ease maintenance

Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[x] Tested via one or more ~NixOS~ test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 119405 at cf070262 run on x86_64-linux 1

2 packages built successfully:

openapi-generator-cli
openapi-generator-cli-unstable

3 suggestions:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/tools/networking/openapi-generator-cli/default.nix:20:3:
   |
20 |   installPhase = ''
   |   ^

warning: unused-argument
Unused argument: runCommand.
Near pkgs/tools/networking/openapi-generator-cli/default.nix:1:57:
  |
1 | { callPackage, lib, stdenv, fetchurl, jre, makeWrapper, runCommand }:
  |                                                         ^

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/tools/networking/openapi-generator-cli/unstable.nix:20:3:
   |
20 |   installPhase = ''
   |   ^

Rebased, ready for review and checks have passed.


Fix the disabled test and some minor cleanups

[x] Make sure there is a JIRA issue filed for the change (usually before you start working on it).  Trivial changes like typos do not require a JIRA issue.  Your pull request should address just this issue, without pulling in other changes.
[x] Each commit in the pull request should have a meaningful subject line and body.
[x] If you're unsure, you can format the pull request title like [CAMEL-XXX] Fixes bug in camel-file component, where you replace CAMEL-XXX with the appropriate JIRA issue.
[x] Write a pull request description that is detailed enough to understand what the pull request does, how, and why.
[x] Run mvn clean install -Psourcecheck in your module with source check enabled to make sure basic checks pass and there are no checkstyle violations. A more thorough check will be performed on your pull request automatically.
Below are the contribution guidelines:
https://github.com/apache/camel/blob/master/CONTRIBUTING.md

I guess CAMEL-15747 can be closed as BlobContainerOperations#listBlobs API is used both by the consumer as well as the producer with the corresponding regex support.
The main issue in the tests (including the @Disabled one) was that we asserted on different MockEndpoints after sending messages using the corresponding ProducerTemplate instead of before which caused race condition.


UnknownMemberId after brod_sup restarts the client

start up the application, containing a group subscriber
kill the brod_client under brod_sup
brod_sup will restart brod_client, and eventually the group subscriber will notice and try to rejoin the group

What happens: the member id is not reset, causing the group coordinator to rejoin again and again, issuing re-joining group, reason::UnknownMemberId. However, messages seem to be consumed normally.

I have not tested it yet, but restarting brod_client (only) should not affect group coordinator.
But I might have missed some corner cases, could you paste some logs here ?

Frankly, I'm not sure anymore if that is specifically related to the expected termination of brod_client: While testing, I noticed that even on a freshly started application I sometimes get output like this:
[info] group coordinator (groupId=consumer-group-id,memberId=foo@mypc/<0.276.0>-ef0b4e06-8e91-41d0-a1f0-d4b6db6cc99e,generation=75,pid=#PID<0.276.0>):
re-joining group, reason::RebalanceInProgress
[info] group coordinator (groupId=consumer-group-id,memberId=foo@mypc/<0.276.0>-ef0b4e06-8e91-41d0-a1f0-d4b6db6cc99e,generation=76,pid=#PID<0.276.0>):
elected=true
[info] group coordinator (groupId=consumer-group-id,memberId=foo@mypc/<0.276.0>-ef0b4e06-8e91-41d0-a1f0-d4b6db6cc99e,generation=76,pid=#PID<0.276.0>):
assignments received:
message:
    partition=0 begin_offset=undefined
[info] client :gateway_brod_client metadata socket down <IP_ADDRESS>:9092
Reason:{:shutdown, :tcp_closed}
[info] group coordinator (groupId=consumer-group-id,memberId=foo@mypc/<0.276.0>-ef0b4e06-8e91-41d0-a1f0-d4b6db6cc99e,generation=76,pid=#PID<0.276.0>):
re-joining group, reason::UnknownMemberId
[info] group coordinator (groupId=consumer-group-id,memberId=foo@mypc/<0.276.0>-3c22039b-7c0a-48e9-9750-06b69c5ded09,generation=78,pid=#PID<0.276.0>):
elected=true
[info] group coordinator (groupId=consumer-group-id,memberId=foo@mypc/<0.276.0>-3c22039b-7c0a-48e9-9750-06b69c5ded09,generation=78,pid=#PID<0.276.0>):
assignments received:
message:
    partition=0 begin_offset=undefined

Note that there are no other clients connected to the (single) broker. Still, brod_client often reconnects due to "RebalanceInProgress", and sometimes due to "UnknownMemberId".
(I've got quite a tight schedule this week at work, so I won't be able to look into that for a few days.)

(it would be nicer if you have timestamp in the logs).
by default, the heartbeat rate is 2 seconds, and session timeout is 10 seconds, could you try set it to 60 see if it helps ?

So, another attempt, hopefully more informative this time:
iex --sname bar -S mix
Erlang/OTP 19 [erts-8.2.2] [source] [64-bit] [smp:8:8] [async-threads:10] [hipe] [kernel-poll:false] [dtrace]

16:54:57.205 module=MyApp.Kafka.SupWrapper [debug] Starting Kafka supervisor (attempt #1)
Interactive Elixir (1.4.2) - press Ctrl+C to exit (type h() ENTER for help)
iex(foo@mypc)1> 16:54:57.248 module=MyApp.Kafka.GroupSubscriber [info] Starting Kafka group subscriber (group="MyApp.consumer-group", client=:myapp_brod_client, topics=["message"])
16:54:57.280 [info] group coordinator (groupId=MyApp.consumer-group,memberId=,generation=0,pid=#PID<0.277.0>):
connected to group coordinator localhost:9092
16:54:57.280 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.277.0>-f567525c-8d29-4868-9a82-50353847b363,generation=22,pid=#PID<0.277.0>):
elected=true
16:54:57.280 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.277.0>-f567525c-8d29-4868-9a82-50353847b363,generation=22,pid=#PID<0.277.0>):
assignments received:
message:
    partition=0 begin_offset=undefined
16:54:57.280 [info] client :myapp_brod_client connected to localhost:9092

We're connected.  Doesn't work... Maybe I should add brod_client to the subscriber's supervisor instead of using brod_sup. Anyway, I'll just kill that one too:
16:59:10.887 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.277.0>-f567525c-8d29-4868-9a82-50353847b363,generation=22,pid=#PID<0.277.0>):
leaving group, reason :killed

16:59:10.896 [error] GenServer #PID<0.277.0> terminating
** (stop) killed
Last message: {:EXIT, #PID<0.274.0>, :killed}
State: {:state, :myapp_brod_client, "MyApp.consumer-group"<EMAIL_ADDRESS><EMAIL_ADDRESS>22, ["message"], {'localhost', 9092}, #PID<0.278.0>, :undefined<EMAIL_ADDRESS>{:kafka_group_member_metadata, 0, ["message"]<EMAIL_ADDRESS>true, #PID<0.274.0>, :brod_group_subscriber, [], #Reference<<IP_ADDRESS>7>, :roundrobin, 10, 2, 5, 5, :undefined, :commit_to_kafka_v2, 5}
16:59:20.888 module=MyApp.Kafka.GroupSubscriber [info] Starting Kafka group subscriber (group="MyApp.consumer-group", client=:myapp_brod_client, topics=["message"])
16:59:20.894 [info] group coordinator (groupId=MyApp.consumer-group,memberId=,generation=0,pid=#PID<0.383.0>):
connected to group coordinator localhost:9092
16:59:20.898 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.383.0>-9bce6120-5782-4bd0-b789-3fca74825295,generation=24,pid=#PID<0.383.0>):
elected=true
16:59:20.903 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.383.0>-9bce6120-5782-4bd0-b789-3fca74825295,generation=24,pid=#PID<0.383.0>):
assignments received:
message:
    partition=0 begin_offset=undefined
16:59:20.905 [info] client :myapp_brod_client connected to localhost:9092

Now everything is good for almost three minutes. Then, without touching anything:
17:02:14.011 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.383.0>-9bce6120-5782-4bd0-b789-3fca74825295,generation=24,pid=#PID<0.383.0>):
re-joining group, reason::UnknownMemberId
17:02:14.016 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.383.0>-66ad3634-2ddf-4ad1-8abe-1559faaca46c,generation=26,pid=#PID<0.383.0>):
elected=true
17:02:14.019 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.383.0>-66ad3634-2ddf-4ad1-8abe-1559faaca46c,generation=26,pid=#PID<0.383.0>):
assignments received:
message:
    partition=0 begin_offset=undefined

There! And now it happens faster:
17:02:27.244 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.383.0>-66ad3634-2ddf-4ad1-8abe-1559faaca46c,generation=26,pid=#PID<0.383.0>):
re-joining group, reason::UnknownMemberId
17:02:27.248 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.383.0>-308d84b8-b74b-49cb-b36a-5b69999a6e35,generation=28,pid=#PID<0.383.0>):
elected=true
17:02:27.252 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.383.0>-308d84b8-b74b-49cb-b36a-5b69999a6e35,generation=28,pid=#PID<0.383.0>):
assignments received:
message:
    partition=0 begin_offset=undefined

...and keeps on happening...
17:02:37.292 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.383.0>-308d84b8-b74b-49cb-b36a-5b69999a6e35,generation=28,pid=#PID<0.383.0>):
re-joining group, reason::UnknownMemberId
17:02:37.296 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.383.0>-18adfe42-83f0-4e93-9d0c-dcc09fc3b10a,generation=30,pid=#PID<0.383.0>):
elected=true
17:02:37.300 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.383.0>-18adfe42-83f0-4e93-9d0c-dcc09fc3b10a,generation=30,pid=#PID<0.383.0>):
assignments received:
message:
    partition=0 begin_offset=undefined
17:02:49.091 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.383.0>-18adfe42-83f0-4e93-9d0c-dcc09fc3b10a,generation=30,pid=#PID<0.383.0>):
re-joining group, reason::UnknownMemberId
17:02:49.096 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.383.0>-7ed6266a-d705-4851-8890-3574ca6fa231,generation=32,pid=#PID<0.383.0>):
elected=true
17:02:49.100 [info] group coordinator (groupId=MyApp.consumer-group,memberId=foo@mypc/<0.383.0>-7ed6266a-d705-4851-8890-3574ca6fa231,generation=32,pid=#PID<0.383.0>):
assignments received:
message:
    partition=0 begin_offset=undefined

I hope that helps :) Let me know if and how I can provide more information.

I'll test it see if I can reproduce it.
@kevinbader thanks for the detailed logs.

Hi @kevinbader
I failed to reproduce it.
Here is the code I ran:
https://gist.github.com/zmstone/5253c55e23b3d5385ed45684ec927614
which is based on the demo code in this wiki page
https://github.com/klarna/brod/wiki/Brod-group-subscriber-demo-in-Elixir
Tested with latest master because I needed #178 to simplify the restart logic.
which should be equivalent as kill client then kill group subscriber.
My test steps:

evaluate BrodDemo.bootstrap in iex.
Kill the client :brod_demo_client
Client and subscriber were both restarted after the delay.
And I didn't see any restarts after that.

Could you please:

Test the code I pasted in the gist against your kafka broker see if there is any different observation than I had ?
If everything works fine, compare my implementation to yours to see if there is some pitfall that worth fixing in brod?


Text selection broken

Hello,
I'm running a file upload service (just for fun) and I've been using paste.js to allow users to CTRL+V images or text to the website. It'll then show "Pasted Image" and give an image preview or "Pasted Text". These can then be uploaded. Works perfectly - BUT:
To be able to paste the data no matter where the focus is on the website, I did
$('body').pastableNonInputable();
$('*').on('pasteImage', function(ev, data) {
    // not relevant
});

The problem is, that my website now has a broken text selection model. It took me quite long to figure this out, but it has something to do with pastableNonInputable() on the body. If I start selecting the text in a div for example, it works as expected only if I start selecting from the left or right side of the div (start selecting outside the div and then enter it). If I select text randomly inside the div it always deselects after releasing the mouse.
I want paste.js to do practically nothing except providing the paste data in a variable, no matter where the paste was initiated.
Am I doing something wrong or is this use case not supported?
Thanks in advance!
Edit1: See this small fiddle: https://jsfiddle.net/n43f4mor/4/
Edit2: Seems only to behave this way in Firefox, in IE11 and Edge I couldn't select at all, always drops.
Edit3: Dug a bit into the code, it's probably line 150 that causes this behaviour:
return paste._container.focus();

This line gets fired everywhere after clicking/selecting in the element the pastableNonInputable() was attached to.
It also happens on the first example at http://layerssss.github.io/paste.js/. The focus requesting also causes mobile browsers to show the keyboard, because the invisible input element created requests focus.

I can confirm the same behavior. If I comment out the snippet you mention on line 150, the text selection no longer is broken, but the paste action itself no longer works either...
I have a work-around involving detection of whether text is selected per this StackOverflow answer relating to a click handler: https://stackoverflow.com/a/31982533/1301139
paste.js:147
      $(nonInputable).on('click', (function(_this) {
        return function(ev) {
          if (!isFocusable(ev.target, false) && window.getSelection().type != "Range") {
            return paste._container.focus();
          }
        };
      })(this));

I have some more testing to do, but if it seems to be stable without compromising any other functionality of the plugin, I'll see about submitting a PR.


Include <signal.h> header to define SIGTERM symbol

Tested using gcc version 7.2.0, Galendae does not compile because 'SIGTERM' is undeclared. The header file <signal.h> needs to be included in src/instance.c.

Thanks a-roy. For some reason I had included SIGTERM from <sys/types.h> but this seems to have stopped working. I've changed the code as per your more sensible suggestion.


pieces with arbitrary side count

It would be nice to allow pieces with more than two sides to handle things like Steve Jackson Games Ogre infantry (3 sides), or GMT Commands and Colors (often 4 sides).  It would also allow pieces with 6 sides to emulate the War of the Ring custom dice (although some other features would also be needed).

#81 supports pieces with up to 100 sides, so close this.


Edit mod menu

Adds an in between step to actually add the relics.
Supports boss relics and shop relics so far.
Could possibly add support for other filters in the future

Looks great! I think it's ready at least for a github release and then if all goes well in a day or two we can submit the update onto the workshop!


Update timeout for .github/workflows/_kongintegration_tests.yaml

This is an automated pull request to update default timeouts for all jobs in .github/workflows/_kongintegration_tests.yaml that do not currently contain a default timeout property.
Please review these changes closely to ensure that all jobs have sufficient time to run. If a job needs more time, please add a timeout property to that job to override the organization default.
Additionally, please be attentive to unintended changes to the workflow YAML file, as the parser is not perfect.
Lastly, note that it is up to the reviewer to merge this pull request and delete the PR branch once they are satisfied with the changes.

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

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

Closing in favour of already merged #5733


Bug 2090774: Add Readme to plugin folder

documentation to explain how ptp plugin works are added to readme under plugin folder
Signed-off-by: Aneesh Puttur<EMAIL_ADDRESS>

/bugzilla refresh

/test e2e-aws


update tests to dsl2

Description
A description of the PR, should include a decent explanation as to why this change was needed and a decent explanation as to what this change does
Review Instructions
Describe if this ticket needs review and if so, how one may go about it in qa and/or staging environments.
For example, a ticket based on Security Hub, Snyk, or Dependabot may not need review since those services
will generate new warnings if the issue has not been resolved properly. On the other hand, an infrastructure
ticket that results in visible changes to the end-user will definitely require review.
Many tickets will likely be between these two extremes, so some judgement may be required.
Issue
https://ucsc-cgl.atlassian.net/browse/SEAB-6283
Security
If there are any concerns that require extra attention from the security team, highlight them here.
Please make sure that you've checked the following before submitting your pull request. Thanks!

[ ] Check that you pass the basic style checks and unit tests by running ./mvnw clean install
[ ] If this PR is for a user-facing feature, create and link a documentation ticket for this feature (usually in the same milestone as the linked issue). Style points if you create a documentation PR directly and link that instead.

note to self: seems to work with nextflow update, maybe need alpha webservice release


Update Implementations.md

Since v3.0.0 has been already released, I propose to change wording a little bit to be more accurate.

See also PR #1312 (already merged into master).

Thank you for the updates.  These changes were applied via a different PR.


cmake fails: Unknown CMake command "eth_install_executable"

I run
% cd cpp-ethereum
% mkdir build
% cd build
% cmake .. -DGUI=0 -DETHASHCL=1

and cmake fails with
CMake Error at CMakeLists.txt:48 (eth_install_executable):
  Unknown CMake command "eth_install_executable".

CMake Warning (dev) in CMakeLists.txt:
  No cmake_minimum_required command is present.  A line of code such as

    cmake_minimum_required(VERSION 3.4)

  should be added at the top of the file.  The version specified may be lower
  if you wish to support older CMake versions for this project.  For more
  information run "cmake --help-policy CMP0000".
This warning is for project developers.  Use -Wno-dev to suppress it.

-- Configuring incomplete, errors occurred!
See also "/home/XXX/cpp-ethereum/CMakeFiles/CMakeOutput.log".

even though cmake/EthExecutableHelper.cmake contains eth_install_executable.
This is after a fresh clone of this repo today.

Try using -DBUNDLE=miner or =cudaminer. Note that the latter is currently broken for Compute 5.2

Thanks, cmake now runs successfully with the -DBUNDLE={,cuda}miner options.
Do you think you could drop a little BUILD file with build instructions in the root of the repo? Since the upstream wiki has gone offline, it would be helpful to have a short summary of how to build this repo.
Or do you plan to migrate to the new webthree codebase any way?
What is Compute 5.2?


compress plugin compresses twice in case of successful POST request to a fresh cache existing URL

Hi !
I got a double-gzipped response by compress plugin in case of POST request to a fresh cache existing URL.
One is triggered in TS_EVENT_HTTP_READ_RESPONSE_HDR, for the response from the server.
https://github.com/apache/trafficserver/blob/d0a44f86345c9125946978efbbf837a720fc9537/plugins/compress/compress.cc#L847-L863
Another is triggered in TS_EVENT_HTTP_CACHE_LOOKUP_COMPLETE, for the existing fresh cash.
https://github.com/apache/trafficserver/blob/d0a44f86345c9125946978efbbf837a720fc9537/plugins/compress/compress.cc#L884-L891
Normally, if no fresh cache exists, only the former is triggered.  And if a fresh cache exists, only the latter is triggered.
But in case of (successful) POST request, existing cache will be deleted even if a fresh cache exists. This is an expected behaviour (see #7839). But in such case, TS_CACHE_LOOKUP_HIT_FRESH == obj_status condition is true, thus BOTH above are triggered.
Thanks in advance !
Kazuhiko

Also, remove-accept-encoding true does NOT work for this 'POST request to a fresh cache existing URL' case.

GET (no cache) -> Accept-Encoding removed
GET (cache hit) -> Accept-Encoding removed
POST (cache hit, that will be invalidated) -> Accept-Encoding NOT removed
POST (no cache) -> Accept-Encoding removed

This issue happens only with remove-accept-encoding true configuration, so the description above might be wrong.


Add a Preference to allow a user to set the default merge style on a PR

Description
Would be nice to be able to globally define the preferred merge style to use by default on an open mergable PR e.g.
"create a merge", "rebase and merge" "squash and merge".  Github does not have a way to define this it will default to whatever the last used merge style was on a PR, so one off changes mean you have to remember to revert back to your preferred method next time you merge. If configured, the refined github setting would override whatever github sets on the PR automatically.

Example URLs
https://github.com/refined-github/refined-github/pull/6740

No preferences allowed in Refined GitHub:
https://github.com/refined-github/refined-github/wiki/"Can-you-add-this-feature%3F"#3-it-doesnt-require-options
Related request:

https://github.com/refined-github/refined-github/issues/5467


fix(deps): update terraform-module

This PR contains the following updates:

Package
Type
Update
Change

terraform-ibm-modules/cos/ibm (source)
module
patch
8.11.3 -> 8.11.4

terraform-ibm-modules/iam-account-settings/ibm (source)
module
patch
2.10.6 -> 2.10.7

Release Notes

terraform-ibm-modules/terraform-ibm-cos (terraform-ibm-modules/cos/ibm)
v8.11.4
Compare Source
Bug Fixes

deps: update terraform ibm to latest for the deployable architecture solution (#713) (e7b48a4)

terraform-ibm-modules/terraform-ibm-iam-account-settings (terraform-ibm-modules/iam-account-settings/ibm)
v2.10.7
Compare Source
Bug Fixes

deps: update terraform terraform-ibm-modules/cbr/ibm to v1.24.1 (#346) (730ff44)

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, check this box

This PR has been generated by Renovate Bot.

/run pipeline

/run pipeline

:tada: This PR is included in version 1.12.5 :tada:
The release is available on:

GitHub release
v1.12.5

Your semantic-release bot :package::rocket:


Update readme.adoc

Update based on features discussed

Could you check if there is any other documents that still have the name "FOP Management App"? Otherwise this PR can be merged already


Language Inconsistency in Enatega Multivendor Web for Restaurants with Variations

Issue Summary:
When changing the language to Arabic in the Enatega Multivendor Web, a language inconsistency is observed for restaurants that offer menu variations. Specifically, the required tags, placeholders, and "Add to Cart" buttons associated with menu variations remain in English, causing a discrepancy in the translated content.
Steps to Reproduce:
Open the Enatega Multivendor Web.
Navigate to the language settings.
Change the language to Arabic.
Browse restaurants with menu variations.
Observe the required tags, placeholders, and "Add to Cart" buttons for menu variations.
Expected Behavior:
The required tags, placeholders, and "Add to Cart" buttons for menu variations should be translated into Arabic to provide a consistent and user-friendly experience in the selected language.
Actual Behavior:
The required tags, placeholders, and "Add to Cart" buttons for menu variations remain in English even after switching to Arabic, leading to a language inconsistency for users.

Resolved!

waiting for publish