Fix delete actor bug

When deleting an actor, it is not removed from the application's list of actors in the storage.
To reproduce:
csruntime --host localhost --port 5001 --controlport 5002 --keep-alive -l INFO
cscontrol http://localhost:5002 deploy calvin/examples/sample-scripts/actions.calvin
cscontrol http://localhost:5002 actor delete <actor_id>
csweb
Then, if you select the application in the web interface, you'll get an error because the actor id is still in the application's list of actors, so it will try to fetch that actor, but since it no longer exists, it'll fail.

Currently the app is static when it comes to actors and connections, which could be changed as you suggest. But note that the storage chosen for application assumes static actors. Your changes will end in race condition if several runtimes makes changes to the application. The proper change of this is to store the set of actors for an application separately e.g. with prefix "app-actors-" and using append, remove and get_concat storage methods. This change should not affect the storage api for the application or actor, i.e. add/get/delete_application and add/delete_actor, they just also maintain the set of actor for an application. Could you break out these changes in a PR separate from replication etc.

Change to use storage append and remove "app-actors-" methods. Therefore also added a new function for getting application actors, also updated the web interface to use this.
It would be nice to change though so that only one request is needed from the web interface, e.g. getApplication instead of loadApplicationData and loadApplicationActors.
Not completely sure about the case when actors are deleted during migration. Do you mean that the delete request takes place when the migration is in place, before it creates it on the new node? in this case, what is the desired behaviour?

Parts of this request have been considered and (optionally) included during rewrites and updates. Thanks!


Created new Resources page and also made titling update to the widget…

Created new Resources page and also made titling update to the widget showing police departments with the most killings.
Description
Created new Resources page and also made titling update to the widget showing police departments with the most killings.
For the resources page, I tried to cover the following:

Where people can access our dataset.

How people can access our repository.

How people can report app and data issues.

Alluded to releasing resources for technical audiences to use (e.g. APIs).

See screenshot below.

For the widget that shows the police forces with the most killings, I updated titling based on this feedback from one of the data sweepers (Delajean).
Fixes # (issue)
Type of change

[ x] New feature (non-breaking change which adds functionality)

Checklist:

[ x] My code follows the style guidelines of this project
[ x] I have performed a self-review of my own code
[ x] I have commented my code, particularly in hard-to-understand areas

Forgot to include link to both repos. Added link for data exploration/science repo


Add support for .graphqlconfig

This adds .graphqlconfig support, fixing #235. Notes:

The implementation is basically copy-paste from https://github.com/apollographql/eslint-plugin-graphql/blob/45b3f95dcdaa5154233c10fb90a21da20c8ad1f5/src/index.js#L265-L274

This should be backwards compatible. Previously, the --schema option was required and defaulted to schema.json. Now it is not "required" anymore and works like this:

If specified, the --schema option wins.
Otherwise we look for a .graphqlconfig file.
Finally we look for a schema.json file.
If nothing can be found, an error is thrown explaining that a schema is required and how to provide one.

So it should be backwards compatible.

I have not added tests yet, because I wanted to ask if you think this is the right approach first. Also, I'm unsure where you'd like me to add the tests and how: unit test the loadSchemaFromConfig function?

I also get this error when building the TypeScript files. The files are still compiled somehow, because I've been able to test this PR manually by creating a .graphqlconfig file an running node ./lib/cli.js generate. I'm new to TypeScript, so I'm unsure how to fix this.

$ npm run compile

><EMAIL_ADDRESS>compile /home/lydell/forks/apollo-codegen
> tsc

node_modules/graphql-request/dist/src/types.d.ts(5,14): error TS2304: Cannot find name 'RequestInit'.
node_modules/graphql-request/dist/src/types.d.ts(9,12): error TS2304: Cannot find name 'RequestInit'.
node_modules/graphql-request/dist/src/types.d.ts(10,19): error TS2304: Cannot find name 'RequestInit'.
node_modules/graphql-request/dist/src/types.d.ts(11,13): error TS2304: Cannot find name 'RequestInit'.
node_modules/graphql-request/dist/src/types.d.ts(12,16): error TS2304: Cannot find name 'RequestInit'.
node_modules/graphql-request/dist/src/types.d.ts(13,16): error TS2304: Cannot find name 'RequestInit'.
node_modules/graphql-request/dist/src/types.d.ts(14,22): error TS2304: Cannot find name 'RequestInit'.
node_modules/graphql-request/dist/src/types.d.ts(15,17): error TS2304: Cannot find name 'RequestInit'.

TL;DR It works! (I just tested using this PR in a real project.) Just need some advice to get across the finish line.

Thanks a lot for working on this!
I'm ok with not having any new tests for this, because we're not really testing the CLI anyway (we probably should...). Also, we'll probably want to extract the CLI code into a common package so we don't have to copy/paste code between the different projects.
The TypeScript error turned out to be pretty hairy, and the only easy workaround was to add the dom lib to tsconfig.json. Hopefully we can solve this soon upstream (see https://github.com/graphcool/graphql-request/issues/15).

Awesome! Let me know if there's anything you'd like me to do before merging.

Merge conflicts fixed. Let me know if there's anything else  you'd like me to do before merging.

Merge conflicts fixed (again!). Let me know if there's anything else you'd like me to do before merging.

Sorry, I seem to have missed this was ready to merge before. Thanks for working on this!


Content negotiation fails in subsequent requests accepting same media type

#3140
clone MediaTypeHeaderValue instance before updating it when content negotiation succeeds

avoids changes to MediaTypeConstants properties and OutputFormatter.SupportedMediaTypes entries
MediaTypeHeaderValue.Clone() does not exist in our DNX Core fork of this class
in previous implementation, was called defensively rather than when required

update WebApiCompatShimBasicTest functional tests to use MvcTestFixture<TStartup> everywhere

#3140 blocked that final migration
remove TestHelper since it's no longer referenced

nits:

remove comments mentioning TestHelper
correct spelling of "negotiation"

remove TestHelper since it's no longer referenced
Just in case you missed it.

/cc @rynowak @ryanbrandenburg (can't keep these Ryans straight)

:shipit:

53060be


[1.9.x] Bump KIE 7.57.0.Final

Many thanks for submitting your Pull Request :heart:!
Please make sure that your PR meets the following requirements:

[ ] You have read the contributors guide
[ ] Your code is properly formatted according to this configuration
[ ] Pull Request title is properly formatted: KOGITO-XYZ Subject
[ ] Pull Request title contains the target branch if not targeting master: [0.9.x] KOGITO-XYZ Subject
[ ] Pull Request contains link to the JIRA issue
[ ] Pull Request contains link to any dependent or related Pull Request
[ ] Pull Request contains description of the issue
[ ] Pull Request does not include fixes for issues other than the main ticket

How to retest this PR or trigger a specific build:

Run all builds
Please add comment: Jenkins retest this

Run (or rerun) specific test(s)
Please add comment: Jenkins (re)run [runtimes|optaplanner|apps|examples] tests

Quarkus LTS checks
Please add comment: Jenkins run LTS

Run (or rerun) LTS specific test(s)
Please add comment: Jenkins (re)run [runtimes|optaplanner|apps|examples] LTS

Native checks
Please add comment: Jenkins run native

Run (or rerun) native specific test(s)
Please add comment: Jenkins (re)run [runtimes|optaplanner|apps|examples] native

Full Kogito testing (with cloud images and operator BDD testing)
Please add comment: Jenkins run BDD
This check should be used only if a big change is done as it takes time to run, need resources and one full BDD tests check can be done at a time ...

Jenkins retest this

Jenkins retest this

The (build) Runtimes check has failed. Please check the logs.

The (build) Apps check has failed. Please check the logs.

@danielezonca cherry picked https://github.com/kiegroup/kogito-runtimes/pull/1452

jenkins rerun apps tests

jenkins rerun runtimes tests


docs: fix link to GitHub workflow badge

see https://github.com/badges/shields/issues/8671
Before

After

Many thanks!


Use mime.json instead lots of cases

Instead of using all these thousands of cases consider using mime.json.

👍

I’ll make a PR the next days that’ll improve the current behaviour.

Just created Pull Request #14 which does the suggested changes.


Including minor details about flatpickr

Changes include details like:

Where to get started with Stimulus.js for Rails.
Installing Flatpickr package is a must
Importing Flatpickr styles are required for the calendar to display correctly.

Thanks this will be very helpful
Close #50


Add code coverage automation

Use this as a template : https://github.com/codecov/example-scala

https://github.com/Khalian/Modulo12/pull/15. Scoverage is not supported for scala 3


Fix the auto_reloader to work when the executable was launched with a module

Fixes #1492
Implements a detection routine when determining arguments for launching the child application, it checks if the executable was launched as a module, rather than a script, and if so launch the same module with the -m commandline argument again.
This PR also includes a different auto_reloader script, which passes the current cwd to the subprocess.call() call.
This is a basic workaround to get bugs in the current auto_reloader fixed for the 2019.03 sanic release.
We do intend to replace this simple naive auto_reloader in the future to be more robust one, that will potentially be implemented as a plugin.

Can anyone work out what is causing the failure in the py35-no-ext test in travis?
The failure is in logging tests and is not related to this PR.
It seems that for some reason in py35 with no uvloop, the asyncio module is emitting a message to the caplog causing the caplog logging test to fail its assertion.

Codecov Diff is negative because there are no tests that touch the auto_reloader. So any modifications to the auto_reloader that add additional lines will decrease codecov further.


Information about relative positional encoding

Hi thanks for sharing your work.
I was wondering if someone can provide we with some additional information about the following line (and the two lines after that):
https://github.com/microsoft/SimMIM/blob/519ae7b0999b9d720daa61e3848cd41b8fbd9978/models/vision_transformer.py#L78
Why exactly do we subtract 3, 2 and 1 respectively? And isn't the statement on line 80 overwriting the statement two lines prior on line 78?
What exactly does the relative_position_index matrix hold?

Hi thanks for sharing your work. I was wondering if someone can provide we with some additional information about the following line (and the two lines after that):
https://github.com/microsoft/SimMIM/blob/519ae7b0999b9d720daa61e3848cd41b8fbd9978/models/vision_transformer.py#L78
Why exactly do we subtract 3, 2 and 1 respectively? And isn't the statement on line 80 overwriting the statement two lines prior on line 78?
What exactly does the relative_position_index matrix hold?

This is for tackling [CLS] token. This was used for ViT. For Swin Transformer, [CLS] token is not used by default.


add gitter link to README.md Closes #56

Added link to gitter chat to README.md
PS: permission is needed to open the chat.

@datapythonista  I can see that Outdated"next to the README.md file above. Is this good to merge the changes from this PR or will it result in conflicts? At the same time, I see the messages that "This branch has no conflicts with the base branch"

The Outdated refers to the review comment. When someone comment on some changes, and then the line that the comment refers to changes, github assumes that the fix requested in the comment is applied.
In this case I commented in the line of the link to ask for the </>, you added that, and github correctly assumed that you already implemented the change I proposed, and considers my comment outdated, which is correct.

@datapythonista  Can I merge this request? But looks like I don't have write access?

@smiaa invited you

@datapythonista, Thank you!


Error on import: posix/glob.so: wrong ELF class: ELFCLASS64

My guess is that it's compiling something for the wrong system width (32 bit instead of 64 bit?)
wget https://luarocks.org/releases/luarocks-3.9.2.tar.gz
tar -zxvf luarocks-3.9.2.tar.gz
./configure

Results in
Configuring LuaRocks version 3.9.2...

Lua version detected: 5.3
Lua interpreter found: /usr/bin/lua5.3
lua.h found: /usr/include/lua5.3/lua.h
unzip found in PATH: /usr/bin

Done configuring.

LuaRocks will be installed at......: /usr/local
LuaRocks will install rocks at.....: /usr/local
LuaRocks configuration directory...: /usr/local/etc/luarocks
Using Lua from.....................: /usr

* Type make and make install:
  to install to /usr/local as usual.
* Type make bootstrap:
  to install LuaRocks into /usr/local as a rock.

Then
sudo make install
luarocks install --local     http://raw.github.com/luaposix/luaposix/master/luaposix-git-1.rockspec
# ... add paths from luarocks path ...
lua -e 'require"posix"'

Returns
lua: error loading module 'posix.glob' from file '/home/rett/.luarocks/lib/lua/5.3/posix/glob.so':
        /home/rett/.luarocks/lib/lua/5.3/posix/glob.so: wrong ELF class: ELFCLASS64
stack traceback:
        [C]: in ?
        [C]: in function 'require'
        /home/rett/.luarocks/share/lua/5.3/posix/init.lua:23: in main chunk
        [C]: in function 'require'
        (command line):1: in main chunk
        [C]: in ?

Not sure if this helps
objdump /home/rett/.luarocks/lib/lua/5.3/posix/glob.so -x
/home/rett/.luarocks/lib/lua/5.3/posix/glob.so:     file format elf64-x86-64
/home/rett/.luarocks/lib/lua/5.3/posix/glob.so
architecture: i386:x86-64, flags 0x00000150:
HAS_SYMS, DYNAMIC, D_PAGED
start address 0x0000000000000000

...

Version References:
  required from libc.so.6:
    0x06969187 0x00 03 GLIBC_2.27
    0x09691a75 0x00 02 GLIBC_2.2.5

...

It looks like my lua is 32 bits
$ luarocks install bitness --local
$ lua -e 'print(require"bitness"())'
32

How do I compile lua.posix to be 32 bit?

I haven't seen this particular problem, but often after upgrading luarocks, I end up having to reinstall all the rocks again.
also, if you're using luarocks to manage your rocks, especially a self build luarocks, you should use only luarocks installed rocks (mixing with rocks from your package manager iss going to make life difficult).
In any case, I don't think this is related to luaposix. You might try asking the luarocks maintainers what to do if reinstalling all your rocks doesn't wprk.

As long as your luarocks installation is pointing to the configuration that matches the Lua binary your are using, luarocks will pass the same compiler options when building rocks.

I'm new to lua. This is the first (and only) package I've installed and I have no rocks installed from the PM.
I'll open an issue on luarocks, thanks!

I figured out the issue. I had installed i386 lua but not i386 lua-dev
Fixed with
sudo apt install lua5.3          # NOT lua5.3:i386
sudo apt install liblua5.3-dev

Then re-installing luarocks and luaposix

thanks for the update!


[feature request] add possibility to get all channels info in one api call.

Is your feature request related to a problem? Please describe.
For now to get all channels info we need to make publicIdentifier/channels call, and then get info for each channel separately by it's ID.
Describe the solution you'd like
Would be very helpful to have something like <publicIdentifier>/getallchannels to receive :
[
"channelID1" :
{info},
"channelID2" :
{info},
]

There is an API to get all the multisigAddresses that are open, but returning all the channel information would be a prohibitively large response in many cases


Add sqlite3_file_control() support

This pull request adds the SQLiteConn.FileControl() method which calls the underlying sqlite3_file_control() function. This can be used for low-level operations on SQLite databases such as persisting the WAL file after database close.
Fixes https://github.com/mattn/go-sqlite3/issues/999

@rittneje I fixed up the PR. Can you take another look please?

@rittneje @mattn Are there any other changes I should make to get this merged? Thanks!

Codecov Report

Merging #1000 (67844d2) into master (671e666) will increase coverage by 0.05%.
The diff coverage is 77.77%.

@@            Coverage Diff             @@
##           master    #1000      +/-   ##
==========================================
+ Coverage   45.98%   46.04%   +0.05%
==========================================
  Files          11       11
  Lines        1481     1490       +9
==========================================
+ Hits          681      686       +5
- Misses        665      667       +2
- Partials      135      137       +2

Impacted Files
Coverage Δ

sqlite3.go
52.97% <77.77%> (+0.02%)
:arrow_up:

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 671e666...67844d2. Read the comment docs.

@rittneje good to merge?

Thank you

@mattn @rittneje Thanks so much for all the reviews! 🙏


Add isopen(::Server)

This used to work but disappeared somewhere in the last few versions.

Looks like TLS failures on nightly?

To :408. Assuming
    servertask = @schedule run( server, portno)

...client connections established...
    close(server)

the established client connections remains open
websocket connections (if present) remains open
any new connections are refused
se.http.sock.status = 6

This is not the only intuitive behaviour of close(server). This may be why isopen(server) was removed?

I don't have strong feelings about it either way. I use isopen(server.http.sock) to check if I have already closed a server (partly because closing an already-closed server throws an exception).


Arch Board Intro Meeting - Release Planner App and Scheduling Tool Integration

The Introduction to the Azure Developer Experience Architecture Board meeting should be a requirement for a service or a product that will be releasing new client libraries (and SDKs).  For example, new Cognitive Services products are skus.  These skus typically have a new API which will result in new client libraries which will be added to the Cognitive Services SDKs.
The Public Preview release plan needs to include a task for Introduction to the Azure Developer Experience (DevEx) Architecture Board.  This task should always appear for new products that will be releasing new client libraires (never released previously, not a new version).

@kyle-patterson @ronniegeraghty Please confirm scenarios that require Azure SDK Intro sessions by default.  The release planner can include this as a step and the scheduling tool can pop-up pre-populated with all of the information we have gathered so far.  Support would be for Cadl + DPG first but we could also support for other scenarios if we can clearly identify the scenarios. For example, if a service partner has a new API (not a new API version) that will result in new client libraries, then should there be a service intro for this new API.

Design and requirements as follow based on discussions with Ronnie, Kyle, and the arch board members who participated in the arch board admin meeting (scheduled by Ronnie and Klye).

New pre-requisite for arch board introduction meeting.  This should invoke the scheduling tool to schedule the introduction meeting.
This should be done as a pre-requisite and on a different blade than the current one below.

3. New blade should preview the prerequisites for the Introduction meeting with the opportunity for user to provided links.

[ ] Hero/Champion scenario (link)
[ ] Overview of the service or product associated with the new initial SDK releases (link)
[ ] Samples (can be pseudocode) (link)
[ ] Generated SDKs and APIViews (links in a table similar to below)

[ ] Prompt to schedule the Introduction meeting with the architecture board (invokes the scheduling app).  Should be greyed out until all prerequisites are provided.

Scheduling Tool Requirements

[ ] Include prerequisite data in the scheduling app and ensure that only Introduction meeting type is enabled.
[ ] Ensure that all of the data is prerequisite data is in the GitHub issue that is created by the scheduling app.
@ronniegeraghty @kyle-patterson Is there a specific GitHub issue template that the scheduling app should use/look at for the Introduction meeting?  Are there any labels or any additional information you want in the GitHub issue?

@ronniegeraghty @kyle-patterson How will the release planner app know that the Introduction meeting has been completed?
Add the Introduction with Arch Board as a requirement for SDK release readiness.  This blade should include all related SDK release readiness criteria, including the introduction meeting.

@ladonnaq
Clarifying questions:

New blade should preview the prerequisites for the Introduction meeting with the opportunity for user to provided links.

Hero/Champion scenario (link)
Overview of the service or product associated with the new initial SDK releases (link)
Samples (can be pseudocode) (link)
Generated SDKs and APIViews (links in a table similar to below)

These prerequistes are different than the ones present in the documentation here: https://dev.azure.com/azure-sdk/internal/_wiki/wikis/internal.wiki/667/Azure-SDK-Archboard-Review?anchor=informational-meeting
We need to make sure that names and types of documents are the same between tools and documentation. Which one do you want to use? Or should the documentation be updated?

Any reason to ask for the prerequisites in the Data Plane SDK Release App and not in the Scheduler App?

For Generated SDKs and APIViews => Is the idea to go to the Introduction meeting with the SDK generated from the API Spec PR that was approved by the API Board? So looking at the Apps, this means the Data Plane API Readiness app like you show?

If we already have them, we should populate the requisites with this information instead of asking the user, right?

APIView has the ability for the user to upload samples (example) so we can ask the users to use that instead of uploading a new file for us to mantain. What do you think?

Scheduling Tool Requirements

Ensure that all of the data is prerequisite data is in the GitHub issue that is created by the scheduling app.
@ronniegeraghty @kyle-patterson Is there a specific GitHub issue template that the scheduling app should use/look at for the Introduction meeting? Are there any labels or any additional information you want in the GitHub issue?

The GitHub issue createad already includes all the information we collect from the Scheduler App. If the pre-requisites changed (above conversation) then we will modify this too.
Example:

I forgot to add, the idea to include the APIViews in the introduction meeting is to review the APIs? do we want the architects to comment in those APIViews?
If so, then how is this different than a follow up Arch board? should we wait until the service actively works on the SDKs that are generated in the following steps (2) of the Data Plane SDK Release ?

@ladonnaq Clarifying questions:

New blade should preview the prerequisites for the Introduction meeting with the opportunity for user to provided links.

Hero/Champion scenario (link)
Overview of the service or product associated with the new initial SDK releases (link)
Samples (can be pseudocode) (link)
Generated SDKs and APIViews (links in a table similar to below)

These prerequistes are different than the ones present in the documentation here: https://dev.azure.com/azure-sdk/internal/wiki/wikis/internal.wiki/667/Azure-SDK-Archboard-Review?anchor=informational-meeting
**(La Donna) I was looking at the GitHub page for the arch board introduction meeting - https://azure.github.io/azure-sdk/policies_reviewprocess.html.  These requirements are not the same a found on the wiki page. (Copied screenshot below)._**

(Mariana) We need to make sure that names and types of documents are the same between tools and documentation. Which one do you want to use? Or should the documentation be updated?

@ronniegeraghty  @kyle-patterson - What are the expectations for prerequisites the service partner needs to have completed and provide during the Introduction to the Arch Board meeting?  Once you have decided, would you please update the wiki and GitHub page so that the information is consistent?

Any reason to ask for the prerequisites in the Data Plane SDK Release App and not in the Scheduler App?

For Generated SDKs and APIViews => Is the idea to go to the Introduction meeting with the SDK generated from the API Spec PR that was approved by the API Board? So looking at the Apps, this means the Data Plane API Readiness app like you show?
(La Donna)  The initial Data Plane API Readiness should be complete before entering SDK Release. I would expect that we would store the links to the any completed assets of the Data Plane API Readiness milestone.  This would include Hero Scenarios, the generated APIViews, generated SDKs, etc.  The samples are not requested in the Data Plane API Readiness app, so we would need to ask for those.  I do think that we should make sure that the service partner has completed all pre-requisites before scheduling the Introduction Meeting.  This would mean that we may want to have a blade for scheduling the Introduction meeting that shows all of the pre-requisites and if we have data, it would be on the blade.

If we already have them, we should populate the requisites with this information instead of asking the user, right?

APIView has the ability for the user to upload samples (example) so we can ask the users to use that instead of uploading a new file for us to mantain. What do you think?
(La Donna) Perfect!

Scheduling Tool Requirements

Ensure that all of the data is prerequisite data is in the GitHub issue that is created by the scheduling app.
**

@ronniegeraghty @kyle-patterson Is there a specific GitHub issue template that the scheduling app should use/look at for the Introduction meeting? Are there any labels or any additional information you want in the GitHub issue?

The GitHub issue createad already includes all the information we collect from the Scheduler App. If the pre-requisites changed (above conversation) then we will modify this too. Example:

@ronniegeraghty @kyle-patterson We need your feedback so that engineering can work on the implementation for this epic/feature.  We will begin piloting the release planner with service partners in scope for Cadl + DPG in January.  If we are not prompting the service partner when to schedule the Introduction within the release planner, then they will not schedule the meeting and that will be problematic.

@ladonnaq thank you for the clarifications :) I will wait on Kyle and Ronnie for the open questions:

prerequissites:
(previously posted) I forgot to add, the idea to include the APIViews in the introduction meeting is to review the APIs? do we want the architects to comment in those APIViews?
If so, then how is this different than a follow up Arch board? should we wait until the service actively works on the SDKs that are generated in the following steps (2) of the Data Plane SDK Release ?

@kyle-patterson @ronnie - How does an Intro meeting for Cadl + DPG differ from other Intro meetings?   From discussions with the arch board, it seems they do want an intro meeting for services that are bringing forward new APIs (likely associated with a new product).
Is this list valid?  Anything missing?

[ ] APIViews (should already be generated from the PR for the new Cadl/API)
[ ] Charts/Info in the new service/product
[ ] If the new service/product has a name, then naming approval for namespace for each language
[ ] Hero/Champion scenario

Would a follow-up meeting be a specific topic?  For example, follow-up Java and JavaScript SDK review.  Or follow-up for naming approval for Python.

@maririos Please let me know once you finalize the missing information from Release Planner App.
@ladonnaq Should this issue belong to Scheduler Tool instead of Release Planner?

@maririos Please let me know once you finalize the missing information from Release Planner App.

@live1206 I am still waiting for PM feedback on the information to include in a meeting and the Wiki to link to. But from what I see, I don't expect too much work to happen on the Scheduler Side.
For Release Planner the work will be to add a step in one fo the apps that will prompt the Intro meeting.
As for scheduling a follow up, maybe the release planner should have that logic and let the scheduler meeting know the type.

@maririos Please let me know once you finalize the missing information from Release Planner App.

@live1206 I am still waiting for PM feedback on the information to include in a meeting and the Wiki to link to. But from what I see, I don't expect too much work to happen on the Scheduler Side.
For Release Planner the work will be to add a step in one fo the apps that will prompt the Intro meeting. As for scheduling a follow up, maybe the release planner should have that logic and let the scheduler meeting know the type.

I updated the description for the requirements for the current release planner.  We will address any new or additional feedback in the new UI.  I also hid all of the other comments so that it will be less confusing.

For  Data plane SDK release type = "new initial client libraries, no existing SDKs for service" or "new initial client libraries, existing SDKs for service"

For  Data plane SDK release type = "new version of client libraries, new SDK version"

CC @maririos
As we move Schedule Arch Board Meeting to first step in the Generate SDK Code and Write Test section. We will not write SDK apiView info to the Artifacts table. The SDK apiView will be generated in Grow Up SDK  step, and then the service team can review or updates the fields in the scheduler for Intro or SDK Review meeting.

@zzvswxy sounds good. If the user is not going to be blocked on generating the SDKs because they haven't had the archboard, then I agree

deployed to Prod


add dot_product example

Scratching this off the #110 list.
Special thanks to @Lokathor, @workingjubilee and the Zulip chat for answering questions on this one.

Failure on riscv64gc seems unrelated because something couldn't pull the proptest lib.

I think this is good to merge. Any objections @workingjubilee ?

Todo:

[ ] Rebase to master to exploit #138
[ ] Add Caleb's method, discuss tradeoffs
[ ] Add a mul_add method from #138
[ ] Benchmark, at some point.

With #138 that is now unblocked~
( it should be zero-cost, tbh )

I believe this can proceed if it is rebased/updated, with a minimal effort. @miguelraz Would you like to do so?

Alright, I think I have some bandwidth now... once more unto the breach my friends! 🎉

I've addressed all of my TODOs from above, and I'm punting the benchmarking to a future PR once more examples take shape and the proper benchmarking infra makes sense.

@calebzulawski I believe your review is required.

@calebzulawski This should be good to go now.

Would you kindly rebase commit https://github.com/miguelraz/stdsimd/commit/340491b2ce5dbc70ea4ebf75e912dd396901678e out of existence?


Add presubmit config.

Fixes https://github.com/GoogleCloudPlatform/healthcare-data-protection-suite/issues/10

/gcbrun

The files LGTM but where does it run? Do we also need a .github/workflows file, or a cloudbuild.yaml somewhere?

The files LGTM but where does it run? Do we also need a .github/workflows file, or a cloudbuild.yaml somewhere?

Check the Trigger in the Checks. It's running on GCB and it's using the build/presubmit.yaml.

The files LGTM but where does it run? Do we also need a .github/workflows file, or a cloudbuild.yaml somewhere?

Check the Trigger in the Checks. It's running on GCB and it's using the build/presubmit.yaml.

I see, thank you.


Reactivity with Console App

I'm trying this for the first time.
This does not call @render function.
Am I missing something?
Arg, I cannot figure out how to preserve indentation.
from mopyx import model, render
@model
class PointModel:
def init(self):
self.x = 0
@render
def render_x(model):
print(f"x: {model.x}")
model = PointModel()
while True:
model.x = input("Enter x: ")

You need to change the variable inside an @.***` annotated function.
See here:
https://github.com/germaniumhq/mopyx/blob/master/test/test_render.py#L106-L110
or:
https://github.com/germaniumhq/mopyx/blob/master/test/test_action_on_model.py
On Sat, Aug 31, 2024 at 2:29 AM jbritton-iti @.***>
wrote:

I'm trying this for the first time.
This does not call @render function.
Am I missing something?
from mopyx import model, render
@model https://github.com/model
class PointModel:
def init(self):
self.x = 0
@render
def render_x(model):
print(f"x: {model.x}")
model = PointModel()
while True:
model.x = input("Enter x: ")
—
Reply to this email directly, view it on GitHub
https://github.com/germaniumhq/mopyx/issues/11, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADPZCQNGEGXA2LYOIPZHF3ZUEE5RAVCNFSM6AAAAABNNKXKVCVHI2DSMVQWIX3LMV43ASLTON2WKOZSGQ4TQNBWGMZTEMQ
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

I was able to get the following to work.
As you noted, I had to modify the model under an action decorator.
I also had to make an initial call to the render function in order to get the reactivity wired up.
This is not a bug and can be closed.
However, I missed the detail of needing to the initial render call to get things wired up.
In the README.md under @render is the following:
"You decorate your UI rendering functions with @render, or invoke them with render_call. MoPyX will map what render method used what properties in the model. The parameters for the function will be also recorded and sent to the renderer function."
This hints at that wiring up, but I think it could be clearer that the initial render call is necessary.
from mopyx import render, model, action

@model
class PointModel:
    def __init__(self):
        self.x = 0

@action
def set_x(x):
    model.x = x

@render
def render_x():
    print(f"x: {model.x}")

model = PointModel()
render_x()
while True:
    x = input("Enter x: ")
    set_x(x)


update urdf mesh filenames to use package path instead of file find, for general remote usage of rviz

Hello!
I have a use case that the driver is running on the robot (inside a docker) , and i monitor the robot in rviz from my laptop.
I noticed that when using file://$(find robotiq_description) instead of package://robotiq_description the workspaces on the two computers need to have the same absolute path in order for rviz to show the meshes properly, this pr solves this issue.
Thanks

FYI this PR seems to have broken Gazebo workflows. Raising an issue now.


Support for Response Headers

Related to: https://github.com/aliencube/AzureFunctions.Extensions/issues/110
Open API spec: https://github.com/OAI/OpenAPI-Specification/blob/master/versions/3.0.1.md#responseObject

This has been released to v0.4.0-preview.


Added no data for all feeds. Moved label to collectionView. #472.

Partial fix for #472.

 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.:x: IgorPopov-MS sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.


ngrok

Requested Feature
ngrok is widely used for tunnelling localhost where HTTPS is enforced (eg. OAuth redirects / webhooks).
The following would be the install script, as per the docs:
https://dashboard.ngrok.com/get-started/setup/linux
curl -s https://ngrok-agent.s3.amazonaws.com/ngrok.asc \
  | sudo tee /etc/apt/trusted.gpg.d/ngrok.asc >/dev/null && echo "deb https://ngrok-agent.s3.amazonaws.com buster main" \
  | sudo tee /etc/apt/sources.list.d/ngrok.list && sudo apt update && sudo apt install ngrok

This project is not associated with the devcontainer project and appears to have been abandoned https://github.com/devcontainers-contrib/features/issues/628 . The official project has removed all devcontainer-contrib features from their listing. https://github.com/devcontainers/devcontainers.github.io/issues/451. There is a fork that is being worked on to update and maintain the features https://github.com/devcontainers-extra/features


liberty support for bus, what extent supported? or bug?

Earlier I hit this: https://stackoverflow.com/questions/47517707/yosys-read-liberty-syntax-error-on-pindatabus
Today:
1. Executing Liberty frontend.
ERROR: Missing or invalid direction for bus A on cell SP64x64_m4.

For a liberty file resembling this:
    bus (A) {
      bus_type : A_5_0;
      # direction label does not appear here
      pin ("A[5:0]") {
        related_power_pin : VDD;
        related_ground_pin : VSS;
        direction : input;

@cliffordwolf Are you looking at a specific spec/BNF for liberty cell format?  I would like to determine who is "at fault".  Then, I can decide on the next step.

https://github.com/YosysHQ/yosys/blob/master/frontends/liberty/liberty.cc#L583-L586
Here is my initial hack.
LibertyAst *dir = node->find("direction");
if (!dir) dir = node->find("pin")->find("direction");
if (!dir || (dir->value != "input" && dir->value != "output" && dir->value != "inout" && dir->value != "internal"))
    log_error([..])

I am surviving with the following patch, a single line fix.
diff --git a/frontends/liberty/liberty.cc b/frontends/liberty/liberty.cc
index 5458c60..8e9f8cc 100644
--- a/frontends/liberty/liberty.cc
+++ b/frontends/liberty/liberty.cc
@@ -581,6 +581,7 @@ struct LibertyFrontend : public Frontend {
                                                log_error("Error in cell %s: bus interfaces are only supported in -lib mode.\n",

                                        LibertyAst *dir = node->find("direction");
+                                       if (!dir) dir = node->find("pin")->find("direction");

                                        if (!dir || (dir->value != "input" && dir->value != "output" && dir->value != "inout" &&
                                                log_error("Missing or invalid direction for bus %s on cell %s.\n", node->args.at

Recommend we keep this open someone else faces the issue or, perhaps, I hit another roadblock.

Does 2c95dfc fix the issue for you?

Fix is confirmed.  Closing.


Updated cell value is showing two times in onOp

After updating the value it is showing two times in data object {"op":"replace","value":"good","path":["data",7,0,"v"],"id":0}
{"req":"op","data":[{"op":"replace","value":"good","path":["data",7,0,"m"],"id":0},{"op":"replace","value":{"fa":"General","t":"g"},"path":["data",7,0,"ct"],"id":0},{"op":"replace","value":"good","path":["data",7,0,"v"],"id":0}]}
And one more question How can identify the actual data that mean "good". because data object have 3 more objects and each object having value.

We use produceWithPatches to record operations on data. In this example, the update func modifid m, ct, v of the cell[7,0], thus there were 3 ops. To identify the actual data, just check the object whose last element of path is "v".

@shuanhukexuejia  am clear good...!


Client Allocation Request for: 北京中经天平科技有限公司

Client Allocation Questions
Core Information

Name: 北京中经天平科技有限公司
Website / Social Media: https://www.bjzjtp.com
Region: Greater China
DataCap Requested: 10TiB
Addresses to be Notarized: f3wwbylr4mxlqcbqvoj3qct2smns4oqk6ewtjwh6cexytv2bqx5w33mw3wpjbtaxnht5az575ia7amj4dtexsq
Notary Requested: IPFSUnion

@szcrzx201 Please subscribe to notifications for this Issue to be aware of updates. Notaries may request additional information on the Issue.

Thank you for your request. Can you kindly answer my following questions?
individual and organization
    1.	What's your name? What's your duty in your company? Please introduce yourself and your team briefly, and show me some your social media accounts such as GitHub, email address, WeChat official account etc.
    2.	For your organization, Please describe its time since inception, registration country，scale, named officers, project, industry reputation etc.
track record
    1.	How did you know about Filecoin? Please describe your understanding of Filecoin storage.
    2.	Please describe any of your honor and disputes in the field of Filecoin.
    3.	Regarding your data you intend to store, What’s the original source of those data, and do you have any proof of ownership or authorization?
    4. Has your organization obtained DataCap before? if so, please answer the following sub-questions.
        * Please share all links and recording of your application before.
        * How about your DataCap usage? Is there any remaining? Is there any information on the deals you've created and miners you've used with your previously allocated?
use plan
    1.	Please describe your plan to use DataCap as detailed as possible? (including use case, amount, timespan, geography, miner, partner etc.)
    2.	Do you plan to record your allocation decisions and execution details in public?
    3.	How do you plan to protect the DataCap you have obtained to make sure that it is used to its best advantage?
Email if you think privacy is involved.

Deals.xls

After comprehensive consideration, 5T datacap is allocated to you. The related message is here.
https://filscout.com/zh/message/bafy2bzacecs46gr7e5zwr6tef7gl6f4kgkaywlx7qapvmf7kbmhj55ktwtiky


Add guide to solve some common installation issues

Some issues I had during installation, noted the solution that solved them, hope it will help.

Great thanks.


feat(cmdline): Add format for := lua expr

Hey @folke there was a new commit on nightly https://github.com/neovim/neovim/commit/a92b38934a2d00c13ee4d1969d994da15e0857ab with a shorthand syntax := for :lua cmdline. This feature is to match the pattern and format with the lua icon.
My autoformatter added the whitespace, let me know if you want me to remove it.

The := {expr} syntax can be used to evaluate a lua expression, as
a shorter form of :lua ={expr}. := and :[range]= without argument
are unchanged. However :=# and similar variants using |ex-flags|
are no longer supported.

Thank you for this PR!
I've changed the implementation quite a bit so that multiple patterns are now allowed for a cmdline item.
Also added better support for :lua=

Thank you for this PR!
I've changed the implementation quite a bit so that multiple patterns are now allowed for a cmdline item. Also added better support for :lua=

awesome, 👍 just gave it test and looks good to me.


[bug]  Accessibility: arrow keys inside nested accordions don't work as expected

Describe the bug
When navigating an accordion inside another accordion, the arrow keys don't move through the nested accordion. Instead, they jump up to the parent accordion.
Related: https://github.com/patternfly/patternfly-elements/issues/2490
Which Component?
rh-accordion
Steps to reproduce

Run accordion demo locally
Find nested example
Expand first accordion item
Tab inside to the child accordion
Hit the down arrow (assuming it should go to the next item in the child accordion)
Focus jumps to the parent accordion header

Expected behaviour
Arrow keys should navigate the currently focused accordion
Screenshots
No response
Element HTML
No response
Operating System (OS)
macOS
Browser
No response
More browser information
No response
Additional Info
No response

See same bug in PFE: https://github.com/patternfly/patternfly-elements/issues/2490

PR has been merged to address this


fix: 升级vue-demi版本

Description

Motivation and Context

Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Enhancement (changes that improvement of current feature or performance)
[ ] Refactoring (changes that neither fixes a bug nor adds a feature)
[ ] Test Case (changes that add missing tests or correct existing tests)
[ ] Code style optimization (changes that do not affect the meaning of the code)
[ ] Docs (changes that only update documentation)
[ ] Chore (changes that don't modify src or test files)

Self Check before Merge

[x] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[x] I have read the CONTRIBUTING document.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
0

Relevant Lines:
0

💛  - Coveralls

@NewByVector 还有一个原因使用vue-demi的latest版本，因为别的库都使用的latest，但是antv却使用了较低版本，就会出现依赖过个vue-demi的版本问题；大概什么时候可以发布一个x6-vue-shape的包呢，现在升级到了vue2.7，这个x6-vue-shape不升级项目跑不起来了，谢谢

@lijing666 我今天发布一个新的版本。

@lijing666 我今天发布一个新的版本。

你早上五六点就上班了，辛苦了，谢谢

不好意思，后面忙忘了， 更新到 1.4.2 版本就好了。

不好意思，后面忙忘了， 更新到 1.4.2 版本就好了。

没事，谢谢了

不好意思，后面忙忘了， 更新到 1.4.2 版本就好了。

没事，谢谢了


feat: Track generated CDK context

As per
https://aws.amazon.com/blogs/devops/best-practices-for-developing-cloud-applications-with-aws-cdk/.
This partially reverts commit e118a393626e7436a11257ea2a91f445c3b3a36a.

Reference
Code review checklist

It applies best practices but our cdk.context.json is not applicable for general public users, so it is questionable.

It applies best practices but our cdk.context.json is not applicable for general public users, so it is questionable.

General public users will have to create their own VPC. I'm not sure whether that will result in regenerating cdk.context.json, but if not we might want to add a note to the documentation about removing it.


Export hive data to clickhouse cluster by seatunnel, and the data is always imported to only one clickhouse node.

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
Export hive data to clickhouse cluster by seatunnel, and the data is always imported to only one clickhouse node.
SeaTunnel Version
seatunnel: apache-seatunnel-2.3.2
spark: 3.3.1
clickhouse: <IP_ADDRESS>
SeaTunnel Config
env {
  execution.parallelism = 3
  job.mode = "BATCH"
  spark.sql.catalogImplementation = "hive"
  spark.app.name = "seatunnel-hive-to-ck_xxx"
  spark.yarn.queue = "default"
  spark.executor.instances = 16
  spark.executor.cores = 2
  spark.driver.memory = "3g"
  spark.executor.memory = "14g"
  spark.hadoop.hive.exec.dynamic.partition = "true"
  spark.hadoop.hive.exec.dynamic.partition.mode = "nonstrict"
  spark.sql.sources.partitionOverwriteMode = "dynamic"
  spark.executor.extraJavaOptions = "-Dfile.encoding=UTF-8"
  spark.driver.extraJavaOptions = "-Dfile.encoding=UTF-8"
}

source {

  Hive {
    metastore_uri = "thrift://xx01:9083,thrift://xx02:9083,thrift://xx03:9083"
    table_name = "dm.xxx"
    result_table_name = "soure_table"
    parallelism = 16
  }

}

transform {
Sql {
    source_table_name = "soure_table"
    result_table_name = "sink_table"
    query = "select * from soure_table"
}
}

sink {
Clickhouse {
    host = "xx01:8123,xx02:8123,xx03:8123,xx04:8123,xx05:8123"
    database = "dm"
    table = "xxx"
    username = xx
    parallelism = 16
    password = xxxx
    clickhouse.confg = {
      max_rows_to_read = "100"
      read_overflow_mode = "throw"
          bulk_size = 100000
      retry = 3
    }
   }
}

Running Command
/usr/local/apache-seatunnel-2.3.2/bin/start-seatunnel-spark-3-connector-v2.sh --master yarn --deploy-mode client --config  /usr/local/apache-seatunnel-2.3.2/config/xxx.conf

Error Exception
I have tried multiple times to import data into ClickHouse, but each time it only writes the data to the first node in the ClickHouse cluster list

Zeta or Flink or Spark Version
No response
Java or Scala Version
No response
Screenshots
No response
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

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

seatunnel-connectors-v2 connector-clickhouse 中 org.apache.seatunnel.connectors.seatunnel.clickhouse.util.ClickhouseUtil代码写死只获取了clickhouse集群列表的第一个节点

org.apache.seatunnel.connectors.seatunnel.clickhouse.sink.client.ClickhouseSink 中写死只使用nodes中的第一个节点nodes.get(0)

添加代码：        // 随机生成一个索引值
Random random = new Random();
ClickHouseNode clickHouseNode = nodes.get(random.nextInt(nodes.size()));
并替换所有的nodes.get(0)

@Toroidals hi， The ClickHouse sink only supports sharding when the table is a distributed table,you can refer to the "split_mode" parameter.

@Toroidals hi， The ClickHouse sink only supports sharding when the table is a distributed table,you can refer to the "split_mode" parameter.
When using distributed tables for writing in ClickHouse, it is necessary to rely on ZooKeeper. In earlier versions, when there is a large amount of data, it could lead to ZooKeeper crashes. Is it still possible to encounter the same issue in the current version?

@Toroidals hi， The ClickHouse sink only supports sharding when the table is a distributed table,you can refer to the "split_mode" parameter.
When using distributed tables for writing in ClickHouse, it is necessary to rely on ZooKeeper. In earlier versions, when there is a large amount of data, it could lead to ZooKeeper crashes. Is it still possible to encounter the same issue in the current version?

I don't think there will be this problem because essentially Clickhouse Sink will write to the local table rather than directly from a distributed table


#488 Pre-fill participant activities for existing users upon calendar page load

Proposed changes
These changes resolve #488 by doing the following:

This removes Cohort 5 from the database migration.
This pre-fills the participant_activities table for each user when they load the calendar page for the first time.

The latter functionality was achieved in code rather than with a stored procedure / migration. This was done in order to adapt best to both environments—live and development—and for code maintainability and testing purposes. Additionally, this minimizes the number of database inserts that need to be done by doing them only on an as-needed basis.
To reiterate, this functionality is only here until we build an admin feature for the participant activities and can associate users with specific programs. When that happens, this code can be removed.
This pull request also updates some out-of-date sample data in the programsService.ts test file, and it updates the name of one of the models to better conform to the naming conventions the other models follow.
Checklist

[x] Are the issues being addressed linked to this PR?
[x] Do all commit messages start with the issue number?
[x] Are all code changes sufficiently tested?
[x] Are there screenshots for UI changes?

Thank you for all your valuable feedback everyone!


Break all (pause) not working for launch/attach scenarios in 1.7.1

VSCode Version: 1.7.1
OS Version: Ubuntu 14.04, Windows 10 Anniversary

Steps to Reproduce:

Install VSCode version 1.7.1
Install C/C++ extension ( or C# extension)
attach and debug an application
Attempt to hit the pause  (break all) button

When I enable logging for the C/C++ extension I don't see a pause command coming from VSCode.

I tried it with launch also, and it doesn't seem to be sending pause.
@chuckries

looks like this is a duplicate of #15165 and fixed in 1.7.2


chore: bump deps

Update:
Github Actions deps
Talos 1.6.7
COSI runtime 0.4.1
Pull Request

What? (description)
Why? (reasoning)
#141
Acceptance
Please use the following checklist:

[ ] you linked an issue (if applicable)
[ ] you included tests (if applicable)
[ ] you ran conformance (make conformance)
[x] you linted your code (make lint)
[x] you linted your code (make unit)

See make help for a description of the available targets.

/m


Prevent middleware for replace response when applying filters

If a Filter applies, the response is replaced with a 200 OK instead of the actual response.
This change ensures the original err is returned when logging is being filtered, which will not affect the response body or response code.
Observed with
github.com/gofiber/fiber/v2 v2.50.0

Thanks for the fix. Let's merge.


Add support for overwriteMode introduced in 3.7

The REST API endpoint for inserting documents at POST /_api/document/ will now handle the URL parameter overwriteMode.

https://github.com/arangodb/docs/blob/master/3.7/release-notes-api-changes37.md#endpoints-augmented

This issue is solved

Hello, discovered the issue while was searching for mentions of overwriteMode parameter in the document creation API and couldn't find any in this repo. Is the issue actually solved?. P.S. Thanks for your work 🙂

Reopened this issue after further investigation.


Phantomjs

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Chrome was used as environment for running the unit tests with karma.

What is the current behavior? (You can also link to an open issue here)

PhantomJS is used as environment for running the unit tests with karma.

What is the new behavior (if this is a feature change)?

Other information:

In Travis both, Chromium and PhantomJS are run.

I'm not a fan of PhantomJS and soon there will be headless chrome but this is fine

🙈


JAMES-2586 Rename Postgres Subscription

rename JPA in prefix to Postgres

Maybe actually we should just move all jpa subpath in packages to postgres already (in all postgres modules), even if some classes are still using jpa. Would maybe help avoid mistakes on forgetting it later?

When I rename package from jpa -> postgres, ci fail because can't not initial some jpa spring bean.
I tried debugging but was not sure could fix it quickly, for ease, I dropped some "un-used" implement of AbstractJPAMailboxMessage
JPAEncryptedMailboxMessage.java
JPAMailboxMessageWithAttachmentStorage.java
JPAStreamingMailboxMessage.java

(that we discussed will not implemented)

https://ci-builds.apache.org/job/james/job/ApacheJames/job/PR-1787/3/testReport/junit/org.apache.james.backends.postgres/ConnectionThreadSafetyTest/connectionShouldWorkWellWhenItIsUsedByMultipleThreadsAndQueriesIncludeBothSelectAndInsert/
Related or not?

https://ci-builds.apache.org/job/james/job/ApacheJames/job/PR-1787/3/testReport/junit/org.apache.james.backends.postgres/ConnectionThreadSafetyTest/connectionShouldWorkWellWhenItIsUsedByMultipleThreadsAndQueriesIncludeBothSelectAndInsert/
Related or not?

not related

Restarted the build, let's see


Adjust Person Info Panel sizes to show full address and remark

Help me check if the sizes of the personInfoPanel look ok in your screens as well.

oh, can we move the address part to the last of person panel? now it looks a bit strange...:)


Option to not display non-cached torrents

As per title. Jackettio is better than anything we have, would love to be able to hide/not display torrents that haven't been cached.

Added in v1.6.0


Adding KafkaStreamFactory for ingesting spans from kafka transport

stream-kafka
KafkaStream
This stream ingests Spans form a Kafka topic advertised by Zookeeper, using Spark Kafka libraries.
Kafka messages should contain a list of spans in json or TBinaryProtocol big-endian encoding.
Details on message encoding is available here
Configuration
KafkaStream can be used as a library, where attributes are set via
KafkaStreamFactory.Builder. It is more commonly enabled with Spring via autoconfiguration.
Here are the relevant setting and a short description. All properties
have a prefix of "zipkin.sparkstreaming.stream.kafka"

Attribute
Property
Description

topic
topic
Kafka topic zipkin spans will be consumed from. Defaults to "zipkin"

zookeeper
zookeeper
Comma separated host:port pairs, each corresponding to a Zookeeper server. e.g. "<IP_ADDRESS>:3000,<IP_ADDRESS>:3001,<IP_ADDRESS>:3002". No default

zkConnectionPath
zk-connection-path
Connection path for Zookeeper. Used as suffix for zk connection string. No default

zkSessionTimeout
zk-session-timeout
Zookeeper session timeout. Defaults to “10000”

I know it is insanely late your time, so I'll fix the minor points on the way in.
good job @naoman!

did quite a bit of rebuffing as you'll notice on the way in. Particularly, making the indirect lookup of the bootstrap servers optional, as in many cases we can supply our own. When things quiet down, the code needs to get some unit test love. you can look at similar integration tests in zipkin-report or zipkin which have KafkaRule.


Improve MSStore installation success rate by trying Restart or Cancel when applicable

After calling to InstallProductAsync, try to check status of existing install items and call Restart or Cancel if applicable.
After our operation failed, clean up the installation queue by calling cancel.
I manually validated by opening Store app, install a test app and pause the installation. Then use winget to install the same app. App installation failed before the change and succeeded after the change.
Microsoft Reviewers: Open in CodeFlow

/azp run

/azp run