Add get kamino positions to sdk

Returns something like this:
      {
        shareMint: 'DcLcLcZsdMRrHo87CJPP4eagiMiFSDuKehkPeiQYSnXn',
        strategy: '7KqB3vRJQDdGwK7ewiDpAxXpTeMmgGicdDdbftQH41XC',
        sharesAmount: '14.459498'
      },
      {
        shareMint: '5BmZgW7dk1kximGfn7MPvDigp3yRmgT64jS9Skdq4nPY',
        strategy: 'Cfuy5T6osdazUeLego5LFycBQebm9PP3H7VNdCndXXEN',
        sharesAmount: '6.604654'
      }

Looks good - some small thoughts -
* This is a very heavy call

* spl-token-2022 will have  new program id when it is deployed

* TOKEN_PROGRAM_ID - is available as a constant in the spl-token lib

Ah, you're absolutely right - will use the TOKEN_PROGRAM_ID constant.
Regarding being a very heavy call, what do you mean exactly? Do you have an alternative in mind?

It depends where the call is being used but I think it will be ok for a first pass - will have a think if it can be done better
Someone explained to me that the issue with too many getProgramAccounts calls is that it is so heavy it can cause the RPC nodes to fall behind if you overuse it

@elliotkennedy

It depends where the call is being used but I think it will be ok for a first pass - will have a think if it can be done better
Someone explained to me that the issue with too many getProgramAccounts calls is that it is so heavy it can cause the RPC nodes to fall behind if you overuse it

Interesting - for now, this will be used by external aggregators that show the user's position, think Sonarwatch or Step finance.

No worries then - we will see if they complain! :)

Reasonable and pragmatic :smile:
Will keep it in mind though, thanks for the comments.

Reasonable and pragmatic :smile:
Will keep it in mind though, thanks for the comments.

https://solanacookbook.com/guides/get-program-accounts.html#dataslice
Looks like your impl is perfect 👍

Nice, glad to hear it!


Support Policy for .NET Packages

Rendered
The purpose of the proposed document is to describe the support policy of the NuGet packages we produce.
The idea is to help customers understand the lifecycle of our packages based on the way they are produced, the .NET version from which they are produced, and the frameworks they target.

Do we need this in the marketing website?

Maybe. We need to close on what this file should be first and then it should be more obvious what to put on dot.net.
This may be enough: https://github.com/dotnet/core/pull/9206#discussion_r1505382804

Can I get another round of review? I added some more info from the perspective of "Servicing" and "End of support" @richlander @leecow

From our meeting yesterday, I will close this PR since we will show this information in other documents.
The information in this PR might be better located in the maintenance-packages repo directly.


breaking: Using Events instead of Message for connect and disconnect

Using events is simpler to understand than InvokeHandler
Allows multiple objects to listen for connected events to add/remove message handlers to NetworkClient/Server
Stops attackers from sending Connect/Disconnect Message

this change does cause client OnClientConnect in host mode to be called the same
BREAKING CHANGE: use events in NetworkServer/Client instead of handlers for Connect/DisconnectMessage

3 things that keep doing

[ ] fix tests
[ ] find better names for OnConnectedEvent
[ ] can we remove public bool InvokeHandler<T>(T msg, int channelId) where T : NetworkMessage

I dont like the name of the events OnConnectedEvent. but OnConnected is already a method inside NetworkServer. One option is to rename the methods as they are private/internal
        public static event ConnectionEvent OnConnected;

        public static event ConnectionEvent OnDisconnect;

the public bool InvokeHandler<T>(T msg, int channelId) where T : NetworkMessage method seems to only be used to invoke message locally. I think this is un-needed as you could just call a method instead of invoking it indirectly.

Since you're obsoleting the messages, I don't think this is a breaking change, unless I missed something else.

Since you're obsoleting the messages, I don't think this is a breaking change, unless I missed something else.

they are not longer being invoked, so if someone manually replaced the handlers this would break

as far as I can tell, UNET team did it this way so that there is only one way for all events (message + handler).
with this change there are now two ways.
let me think about it a bit. it's not necessary bad either way.

I think this is one of the few cases where even though your PR is more simple, we should keep the previous solution with the big picture in mind.
this way there is only one way to react to messages.
it will also allow us to fully decouple the authenticator form core mirror later, where we can set authenticated = true in the default connectmessage handler, and authenticator can overwrite the handler and do it's own authentication method.
I do that in DOTSNET too and I think it's worth it.

It isnt added a 2nd way to react to message, it is removing the ability to invoke message locally. The 2 main benefits are being more obvious what is calling what (hard to know what InvokeHandler is really invoking), and allows multiple things to listen for the events.

reopening as discussed on discord

@James-Frowen please rebase, fix sonar errors and switch from draft to live :)

I'm in the process of fixing the tests, it seems some of them aren't clean up after themselves, I will make it live PR once I sort it out

This change also stops attackers from sending Connect/Disconnect Message

This change also stops attackers from sending Connect/Disconnect Message

Please elaborate :)

This change also stops attackers from sending Connect/Disconnect Message

Please elaborate :)

An attacker can send the ConnectMessage and Mirror will treat it the same as if it came from conn.InvokeHandler(new ConnectMessage(), -1);
I'm not sure exactly how NetworkManager would handle receiving multiple of these from the same client but It will likely call the OnServerConnect virtual method that user's override.

reading through changes again:

the PR does too much. like last time, please don't rename the internal functions from OnConnected to OnConnectedTransport etc.
wasn't the point to remove the internal invokehandler function? seems like it's still there?

the PR does too much. like last time, please don't rename the internal functions from OnConnected to OnConnectedTransport etc.

They have to be renamed so that the events can be called OnConnected, what name do you think would be better?
I'm not sure what less this this PR can do? the main point is now to remove ConnectMessage and DisconnectMessage

wasn't the point to remove the internal invokehandler function? seems like it's still there?

InvokeHandler has been marked with Obsolete, we don't need to fully remove the function now. The InvokeHandler itself isnt a security risk, it is that message handlers get registered so that an attacker could invoke them (when they should only be called locally on the server).
Its been about a month since I started hat and since then we have found that ConnectMessage and DisconnectMessage can be used by attacker to break the server. so the original idea of the PR is slightly different.

closing in favor of https://github.com/vis2k/Mirror/commit/52747a4aa10066299a586001243c724f879e18ca


Fix docs urls

Codecov Report
Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (21dff81) compared to base (1a3bb73).
Patch has no changes to coverable lines.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff            @@
##              main       #17   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files            8         8
  Lines          301       301
=========================================
  Hits           301       301

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


[EUWE] UI for add/remove interface on network router

UI for add/remove interface on network router
This commit adds UI code for adding and removing router interfaces.
BZ for this is:
https://bugzilla.redhat.com/show_bug.cgi?id=1394284
This is the Euwe backport PR corresponding to https://github.com/ManageIQ/manageiq-ui-classic/pull/60

@tzumainn @blomquisg It looks like this is failing because https://github.com/ManageIQ/manageiq/pull/13032 hasn't been backported to euwe yet. Things fail locally too, because the model bits aren't in place. I'll revisit this PR once the model is backported.

@sseago It looks like I can backport ManageIQ/manageiq-ui-classic#60 without conflict. This should still be used instread?

@simaishi ahh, I hadn't realized you could backport from manageiq-ui-classic.  This PR should be unnecessary then.

Backported https://github.com/ManageIQ/manageiq-ui-classic/pull/60, closing this PR.


Class C operation (AS_923).

Looking fwd for this .. thanks

sorry. typo :-)

We are working on that #28

there is an example of class C apparently working #32

class c working with chirpstack and ttn? can you an example?

I've still seen no Class C working example yet. Lot's of links to here is a class c example but I haven't seen anything substantial!


Added lushka.al

This PR is:

[x] Adding a new domain

[ ] Updating existing domain size

[ ] Changing domain name

[ ] Removing existing domain from list

[ ] Website code changes (512kb.club site)

[ ] Other not listed

[x] I used the uncompressed size of the site

[x] I have included a link to the GTMetrix report

[x] The domain is in the correct alphabetical order

[x] This site is not a ultra lightweight site

[x] The following information is filled identical to the data file

I confirm that I have read the FAQ section, particularly the two red items around minimal pages and inappropriate content and I attest that this site is neither of these things.

[x] Check to confirm

- domain: lushka.al
  url: https://lushka.al/
  size: 20.5
  last_checked: 2022-03-09

GTMetrix Report: https://gtmetrix.com/reports/lushka.al/VzsrAdPk/

Thanks!


Webpack -  Module not found: Error: Cannot resolve module 'nvd3' in node_modules\angular-nvd3\dist

Hello, I'm using the angular-nvd3 library in a project,  I use webpack to manage dependencies, my application was functioning normally but today I get this error:
ERROR in ./~/angular-nvd3/dist/angular-nvd3.js
Module not found: Error: Cannot resolve module 'nvd3' in node_modules\angular-nvd3\dist
@ ./~/angular-nvd3/dist/angular-nvd3.js 13:13-28
you can give me an idea about the solution?

I added this dependence in my project  and the problem was solved to package.json  "nvd3": "^1.8.4"

We have the same issue, senks for your workaround, it helps.

We also have the same issue, adding nvd3 as a project dependency has not resolved it.

The problem here is that the nvd3 package in npm doesn't not export the nv variable
I fixed this by defining an alias in the webpack config
resolve: {
  alias: {
    nvd3: "nvd3/build/nv.d3.min.js"
  }
}

So now i can do import 'nvd3' and nvd3 gets imported into window.nv but the angular-nvd3 code requires it again using nv = require('nvd3') so it gets overwritten with a blank object
Any clue on how to continue ?

Looks like something happened with the nvd3 package, which says it was updated 6 days ago.
Weirdly, npm view nvd3 shows that version 1.8.4 exists, just is treated as an earlier version than 1.7.1 (the "latest" and also what angular-nvd3 depends on).
As a temporary workaround, installing<EMAIL_ADDRESS>worked for me.

Well, that solved the problem for me. Thanks a lot, @johncomposed ! :+1:


Lexer.stringLiteral accept regex as start/close.

Then we can use stringLiteral('`', { close: /`|\$\{/ }) to match multi open/end at the same time.

No. Just write a new Action.


Error

I have this error
Failed to load resource: the server responded with a status of 404 (Not Found)
localhost/:51 (SystemJS) XHR error (404 Not Found) loading http://localhost:5555/node_modules/angular-oauth2-oidc.js
Error: XHR error (404 Not Found) loading http://localhost:5555/node_modules/angular-oauth2-oidc.js
at XMLHttpRequest.wrapFn [as _onreadystatechange] (http://localhost:5555/node_modules/zone.js/dist/zone.js?1486029206310:1039:29) []
at Zone.runTask (http://localhost:5555/node_modules/zone.js/dist/zone.js?1486029206310:151:47) [ => ]
at XMLHttpRequest.ZoneTask.invoke (http://localhost:5555/node_modules/zone.js/dist/zone.js?1486029206310:345:33) []
Error loading http://localhost:5555/node_modules/angular-oauth2-oidc.js as "angular-oauth2-oidc" from http://localhost:5555/app/app.module.js Not expecting this error? Report it at https://github.com/mgechev/angular-seed/issues
(anonymous) @ (index):51
http://localhost:5555/node_modules/angular-oauth2-oidc.js Failed to load resource: the server responded with a status of 404 (Not Found)

Seems like, SystemJS isn't configured correctly. Please have a look at https://github.com/manfredsteyer/angular-oauth2-oidc/issues/1


SUBMARINE-617. Update the apache/notebook:jupyter-notebook docker image

What is this PR for?

Update the apache/notebook:jupyter-notebook docker image
Preinstall notebook & apache-submarine in jupyter notebook
Set two environment variable (SUBMARINE_SERVER_DNS_NAME & SUBMARINE_SERVER_PORT) in notebook container for ease of internal communication between submarine-server and notebook.

What type of PR is it?
[Improvement]
Todos

[ ] - Task

What is the Jira issue?
SUBMARINE-617
How should this be tested?
Travis CI
Screenshots (if appropriate)

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

I' m working on that inject submarine server DNS into notebook to easy the  internal communication and SDK initialization. Please do not commit it temporarily. Thanks.

In the first commit, apache-submarine package will be installed whenever creating notebook server. It spend a lot of time. I suggest that apache-submarine package should be installed before notebook container was running, and spend a lot of time pulling image when creating notebook server first time inevitably.
c.c. @jiwq @pingsutw

@jiwq @pingsutw
I set two environment variables (SUBMARINE_SERVER_DNS_NAME and SUBMARINE_SERVER_PORT) in notebook container. Notebook can connect to submarine-server with Submarine SDK without entering the URL.
Please review this PR again. Thanks!

Could you also set the environment used by Jupyter Notebook in https://github.com/apache/submarine/blob/master/docs/database/submarine-data.sql?
So we don't need to manually create this environment

@pingsutw Thanks for your review! I pushed a fixed-commit.


Replicate NMT scores of Bahdanau et al. (2015)

Use neuralmonkey to replicate the scores of Bahdanau et al. (2015) on English-to-French, WMT-14 translation task
This would mean to do the following things:

[x] Include the domain adaptation of Axelrod et al. (2011)
[x] Include the tokenization script from Moses (somehow related to #32)
[x] Adapt the code to the equations in the paper
[ ] Make sure we are able to run beam search with the trained model

Anything else?

For replicaton, the domain adaptation is not required - the filtered WMT data are available on http://www-lium.univ-lemans.fr/~schwenk/cslm_joint_paper/

I would expect the pseudo-in-domain-selection to have some scripts (probably in Perl) in the Moses code base. @ales-t was using it, he probably knows more.
It would be also good to claim in the paper, we can replicate a state-of-the-art result and provide a script that will prove it.

It would be also good to claim in the paper, we can replicate a state-of-the-art result and provide a script that will prove it.

Yes, that was my motivation for opening this issue.

It can prove challenging, though. I fear the model they used will not fit into any of our GPU's memory (I have a 6GB titan, same size as the cards on UFAL)

About the in-domain selection: you need two language models compatible with
SRILM, one trained on "guaranteed" in-domain data, the other trained on
general/out-of-domain data. They should both have identical n-gram length
and similar training data sizes. People also typically filter out
singletons (replacing them with ) when doing this.
Once you have the LMs, you can run this simple tool to score your sentences:
https://redmine.ms.mff.cuni.cz/projects/ufal-smt-playground/repository/revisions/master/show/playground/tools/lmppl
On Wed, Jul 13, 2016 at 1:58 PM, Jindra Helcl<EMAIL_ADDRESS>wrote:

It can prove challenging, though. I fear the model they used will not fit
into any of our GPU's memory (I have a 6GB titan, same size as the cards on
UFAL)
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ufal/neuralmonkey/issues/68#issuecomment-232334009,
or mute the thread
https://github.com/notifications/unsubscribe/AAPcpnIoNwDl_wLB2WPyOpI4yi7Vj1zgks5qVNLggaJpZM4JLR0m
.

Thanks. Do you think it is applicable when the only real in-domain data I have is a very small dev set (~2k sentences)? (The in-domain part of the training dataset I have is very noisy and probably wouldn't do good)
Also, given the size of the in-domain corpus, is it then better to train the general language model on a random 2k-sentence-subset of the whole data than on a different subset?

Hm, you can try it but I don't think this is enough data. Your OOV rate
will be very high and generalization might be poor. It's hard to guess in
advance though.
Ales
On Wed, Jul 27, 2016 at 4:34 PM, Jindra Helcl<EMAIL_ADDRESS>wrote:

Thanks. Do you think it is applicable when the only real in-domain data I
have is a very small dev set (~2k sentences)? (The in-domain part of the
training dataset I have is very noisy and probably wouldn't do good)
Also, given the size of the in-domain corpus, is it then better to train
the general language model on a random 2k-sentence-subset of the whole data
than on a different subset?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/ufal/neuralmonkey/issues/68#issuecomment-235604452,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAPcpp7sD75eufL7xv6-S3fd7sFqrSnLks5qZ2yAgaJpZM4JLR0m
.

Attention decoder implementation now follows equations in the paper (PR #77).

As for the domain adaptation and tokenizer, I would just put a note (or possibly a script) in the experiment directory with instructions of how to obtain the en-fr wmt task filtered data (these are available) and how to tokenize the data.
The product of this issue will be a directory inside examples/ with ini file and scripts/instructions how to prepare the data.

Issues s beam searchem je tu hromada, tohle je jinak už hotový.


Swarm adds prerouting iptables rule which hides existing rules

By default, swarm has a very open rule that looks like:
-A PREROUTING -m addrtype --dst-type LOCAL -j DOCKER-INGRESS
This in effect takes all traffic from anywhere and sticks in in the DOCKER-INGRESS chain, where the rules to route traffic for running services are dyanically added.  If you attempt to limit the globalness of this rule to:
-A PREROUTING -s <IP_ADDRESS>/16 -m addrtype --dst-type LOCAL -j DOCKER-INGRESS
-A PREROUTING -s <IP_ADDRESS>/12 -m addrtype --dst-type LOCAL -j DOCKER-INGRESS
-A PREROUTING -s <IP_ADDRESS>/8 -m addrtype --dst-type LOCAL -j DOCKER-INGRESS
When a container is started on a host by swarm, it re-adds its default rule to position 1 in the chain.
Chain PREROUTING (policy ACCEPT)
num  target     prot opt source               destination
1    DOCKER-INGRESS  all  --  <IP_ADDRESS>/0            <IP_ADDRESS>/0            ADDRTYPE match dst-type LOCAL
2    DOCKER-INGRESS  all  --  <IP_ADDRESS>/16       <IP_ADDRESS>/0            ADDRTYPE match dst-type LOCAL
3    DOCKER-INGRESS  all  --  <IP_ADDRESS>/12        <IP_ADDRESS>/0            ADDRTYPE match dst-type LOCAL
4    DOCKER-INGRESS  all  --  <IP_ADDRESS>/8           <IP_ADDRESS>/0            ADDRTYPE match dst-type LOCAL
Which then allows traffic from all sources, hiding the limited rules.  Deleting that rule returns the host to its desired behaviour.
This is very easy to replicate, start with a swarm host that has two interfaces, start a service with a simple web container, and then attempt to curl the container over both interfaces, should work.  Replace the default rule with the source limited rules, and filtering works as expected, traffic on the non-filtered interface works, on the filtered interface it does not.  Stop the container running on the host, swarm will start a new instance on that host and re-adds the global rule to position 1.
The behaviour of this could be modified, so swarm tests with there is already a -j DOCKER-INGRESS rule in play within PREROUTING, and if so, respects that rule is accurate.
I attempted to re-create the docker swarm network overlay using the --internal option, this doesn't appear to have any effect.  However, if it did, it only partly corrects the issue as we want to be able to limit interactions with the swarm to private and select public subnets.  Maybe there is some other way to achieve?

Hi @zk101  Thanks for your feedback.
I am afraid that your issue is related to Swarm Mode of docker engine, maybe it is more proper to post issue here https://github.com/docker/docker/issues/new . This project is about Swarm which is different from swarm mode in docker engine.
Here are more details about the difference https://github.com/docker/swarm#swarm-disambiguation
I am closing this. If you need more help, feel free to let us know. Thanks again for your contribution.


Add agentID in AppliedManifestWork

This is handle the case that a work agent may
need to handle appliedmanifestworks with different hubhash values.
Signed-off-by: Jian Qiu<EMAIL_ADDRESS>

/hold

/lgtm

/unhold


Live stage

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

InnovationTalk/index.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


refactor!(core): update AttributeUpdate methods returns

Description
Associated issue: #232
Content description: TODO
Additional information

[x] Breaking change

Codecov Report
Attention: Patch coverage is 54.54545% with 15 lines in your changes missing coverage. Please review.

Project coverage is 79.26%. Comparing base (a924645) to head (d3ca95c).

Files with missing lines
Patch %
Lines

honeycomb-core/src/attributes/collections.rs
38.88%
11 Missing :warning:

honeycomb-core/src/attributes/traits.rs
71.42%
2 Missing :warning:

honeycomb-kernels/src/grisubal/model.rs
0.00%
2 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #237      +/-   ##
==========================================
+ Coverage   79.21%   79.26%   +0.04%
==========================================
  Files          52       52
  Lines        6953     6950       -3
==========================================
+ Hits         5508     5509       +1
+ Misses       1445     1441       -4

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

🚨 Try these New Features:

Flaky Tests Detection - Detect and resolve failed and flaky tests


Using custom models with their own output format

Hello. This package is realy cool.
I want to use my own tflite model for pose estimation. Can I get output with my own format? I want to get heatmap from my model outputs for pose estimation, where count of keypoints only is 14.

Hi,
I think it would work if you update the part_names and pose_chain to match with your model.
https://github.com/shaqian/flutter_tflite/blob/master/android/src/main/java/sq/flutter/tflite/TflitePlugin.java#L58-L73
https://github.com/shaqian/flutter_tflite/blob/master/ios/Classes/TflitePlugin.mm#L980-L995
Thanks,
Qian

@shaqian Thanks, but, can I get raw heatmaps from model?

@garipovazamat
The raw heatmaps are here:
https://github.com/shaqian/flutter_tflite/blob/master/ios/Classes/TflitePlugin.mm#L1193-L1196
https://github.com/shaqian/flutter_tflite/blob/master/android/src/main/java/sq/flutter/tflite/TflitePlugin.java#L1250-L1253
If you want to return the outputs to Flutter then you will need to implement a method call.


New command: Remove member from administrative unit

Usage
m365 aad administrativeunit member remove
Description
Remove a specific member (user, group, or device) from an administrative unit
Options

Option
Description

-i, --id [id]
The id of the member to be removed. Specify either id, userId, userName, groupId, groupName, deviceId or deviceName.

--userId [userId]
The id of the user to be removed. Specify either id, userId, userName, groupId, groupName, deviceId or deviceName.

--userName [userName]
The user principal name (UPN) of the user to be removed. Specify either id, userId, userName, groupId, groupName, deviceId or deviceName.

--groupId [groupId]
The id of the group to be removed. Specify either id, userId, userName, groupId, groupName, deviceId or deviceName.

--groupName [groupName]
The name of the group to be removed. Specify either id, userId, userName, groupId, groupName, deviceId or deviceName.

--deviceId [deviceId]
The id of the device to be removed. Specify either id, userId, userName, groupId, groupName, deviceId or deviceName.

--deviceName [deviceName]
The name of the device to be removed. Specify either id, userId, userName, groupId, groupName, deviceId or deviceName.

--administrativeUnitId [administrativeUnitId]
The id of the administrative unit. Specify either administrativeUnitId or administrativeUnitName.

--administrativeUnitName [administrativeUnitName]
The name of the administrative unit. Specify either administrativeUnitId or administrativeUnitName.

-f, --force
Don't prompt for confirmation.

Examples
Remove a member specified by id from an administrative unit specified by id
m365 aad administrativeunit member remove --id 64131a70-beb9-4ccb-b590-4401e58446ec --administrativeUnitId 03c4c9dc-6f0c-4c4f-a4e6-0c9ed80f54c7

Remove a user specified by id from an administrative unit specified by id
m365 aad administrativeunit member remove --userId 64131a70-beb9-4ccb-b590-4401e58446ec --administrativeUnitId 03c4c9dc-6f0c-4c4f-a4e6-0c9ed80f54c7

Remove a group specified by id from an administrative unit specified by id
m365 aad administrativeunit member remove --groupId b2307a39-e878-458b-bc90-03bc578531d6 --administrativeUnitId 03c4c9dc-6f0c-4c4f-a4e6-0c9ed80f54c7

Remove a device specified by id from an administrative unit specified by id
m365 aad administrativeunit member remove --deviceId 810c84a8-4a9e-49e6-bf7d-12d183f40d01 --administrativeUnitId 03c4c9dc-6f0c-4c4f-a4e6-0c9ed80f54c7

Remove a user specified by UPN from an administrative unit specified by name
m365 aad administrativeunit member remove --userName<EMAIL_ADDRESS>--administrativeUnitName 'Marketing Division'

Remove a group specified by name from an administrative unit specified by id
m365 aad administrativeunit member remove --groupName 'Marketing Group' --administrativeUnitId 03c4c9dc-6f0c-4c4f-a4e6-0c9ed80f54c7

Remove a device specified by name from an administrative unit specified by name
m365 aad administrativeunit member remove --deviceName 'JohnDoe-PC' --administrativeUnitName 'Marketing Division'

Default properties
No response
Additional Info
https://learn.microsoft.com/en-us/graph/api/administrativeunit-delete-members?view=graph-rest-1.0&tabs=http
I will work on this issue.

Hi @MartinM85 thank you for the spec!
Few considerations:

For the id option, let's be a bit more specific: The id of the member to be removed.
Let's align this with the other commands and include userId, groupId, and deviceId.
Because of #5663, we're planning to rename all aad commands to entra. So let's update this spec to use entra as well. This means that it's probably best to wait until this issue is closed to avoid unneeded merge conflicts. We will try to prioritize this PR so we don't block other aad commands.

@milanholemans Do you suggest to have id option together with userId, groupId, and deviceId options or only userId, groupId, and deviceId options and remove id option?

Hi @MartinM85, the id option is definitely useful, I'm just suggesting to add these other options as well. Since we have these options at the aad administrativeunit member add, the remove command should have similar options (with addition to id and force).

@milanholemans Spec updated.

Great, could you also replace all aad occurrences with entra?

@milanholemans Replaced

Great! Looks good now!
As mentioned before, I think you better wait until #5663 is finished to avoid unneeded merge conflicts.

For me it makes more sense to merge this small ticket first before #5663, because there are a lot of changes in #5663 and it will take some time to check that everything is working after renaming. But I can postpone the delivery of this.

OK, there is already opened PR for #5663. I will postpone the delivery.

Yes indeed, we marked it as priority to unblock other issues.


Use CPython 3.8.0 mechanism to find msvc 14+

Summary of changes
Update msvc.py to use CPython 3.8.0 mechanism to find msvc 14+ as recommended in this comment.
Closes #1903
This PR has been tested as part of cibuildwheel tests using various CI providers.
Please note that I can't test it more thoroughly cause I don't have any Windows environment at hand (except for CI which requires very long iterations).
Pull Request Checklist

[x] Changes have tests
[x] News fragment added in changelog.d. See documentation for details

Using a Python 3.8 back port instead of the current mechanism seem to be the good way to do.
This also allow to completely drop the old mechanism that become unused. This will make this file far more easy to maintain. Note that this will remove the following:

VC 2010 support, only used by Python 3.4 that is deprecated.
Microsoft Windows SDK 6.1 and 7.0 support, tools that are deprecated and where removed from download by Microsoft. (And there is still the "VC for python 2.7" that replace them)

I sent you a PR with the following changes:

Drop the old mechanism.
Fixed a missing import.
PEP8 + Add python 3.8 backport comment.

I do not have a Windows right now to do more tests on it.

Thanks @JGoutin, this is much cleaner indeed. All CI tests of cibuildwheel are passing with your version of msvc.py. I will correct the missing import in the meantime (itertools.count() right ?)
If @pypa/setuptools-developers are ok with your notes (Microsoft Windows SDK 6.1 and 7.0, Python 3.4 support dropped), I'll merge that in this PR.

There are maybe tests with Python 3.8 that can be back-ported too.

Indeed, I'll check if I can add those:
https://github.com/python/cpython/blob/master/Lib/distutils/tests/test_msvccompiler.py

(itertools.count() right ?)
It's exactly that.

Concerning the drop of the VC2010 support, Python 3.4 will be dropped with #1908.

If @pypa/setuptools-developers are ok with your notes (Microsoft Windows SDK 6.1 and 7.0, Python 3.4 support dropped), I'll merge that in this PR.

#1908 is merged, Python 3.4 support is dropped from setuptools.

I will drop python 3.4 support in msvc.py from the PR

Edit: I won't drop python 3.4
Since python 3.4 uses the same code base as python 2.7, I'd rather not change it. It'll be easier to drop as a whole once python 2.7 support is really removed from setuptools.

Removing the python 3.4 support will not remove the Python 2.7 support. It will only remove the
Microsoft Windows SDK 6.1 and 7.0 support, that are deprecated and not available anymore.
Python 2.7 requires msvc9_find_vcvarsall & msvc9_query_vcvarsall functions.
In the PR code, theses functions are still available: https://github.com/JGoutin/setuptools/blob/msvc-python3.5/setuptools/msvc.py#L56.

@JGoutin,
I'd rather not change python 2.7 support at all to prevent any regressions, even if unlikely. So, in my opinion, removing  Microsoft Windows SDK 6.1 and 7.0 support is not an option.

@jaraco,
can you review this PR please ?

Released as 45.3.0


[LIVY-703] Show SparkUI link on Livy UI when running on Kubernetes

What changes were proposed in this pull request?
Jira
This PR is one of the PRs in the series related to the splitting of the PR https://github.com/apache/incubator-livy/pull/167 to multiple PRs to ease and speed up review and merge processes.
This PR targets branch of the base PR https://github.com/apache/incubator-livy/pull/249 and should be merged after it.
This PR implements a way to build Spark UI links on Livy UI when running on Kubernetes. This patch is required to unlock further integrations with Kubernetes Ingress resources (ref: https://github.com/apache/incubator-livy/pull/167).
Refer https://github.com/jahstreet/incubator-livy/pull/35 to compare changes.
How was this patch tested?
Unit tests.
Manual testing with Kubernetes on Docker Desktop for Mac v<IP_ADDRESS>.
Environment - Helm charts:

cluster-base with custom-values.yaml:

nginx-ingress:
  controller:
    service:
      loadBalancerIP: <IP_ADDRESS> # my-cluster.example.com IP address (from /etc/hosts)
      loadBalancerSourceRanges: []
cluster-autoscaler:
  enabled: false
oauth2-proxy:
  enabled: false

spark-cluster with custom-values.yaml:

livy:
  image:
    pullPolicy: Never
    tag: 0.7.0-incubating-spark_2.4.3_2.11-hadoop_3.2.0-dev
  ingress:
    enabled: true
    annotations:
      kubernetes.io/ingress.class: nginx
      kubernetes.io/tls-acme: "true"
      nginx.ingress.kubernetes.io/rewrite-target: /$1
    path: /livy/?(.*)
    hosts:
    - my-cluster.example.com
    tls:
    - secretName: spark-cluster-tls
      hosts:
      - my-cluster.example.com
  persistence:
    enabled: true
  env:
    LIVY_LIVY_UI_BASE1PATH: {value: "/livy"}
    LIVY_SPARK_KUBERNETES_CONTAINER_IMAGE_PULL1POLICY: {value: "Never"}
    LIVY_SPARK_KUBERNETES_CONTAINER_IMAGE: {value: "sasnouskikh/livy-spark:0.7.0-incubating-spark_2.4.3_2.11-hadoop_3.2.0-dev"}
    LIVY_LIVY_SERVER_SESSION_STATE0RETAIN_SEC: {value: "300s"}
    LIVY_LIVY_SERVER_KUBERNETES_ALLOWED1NAMESPACES: {value: "default,test"}
    LIVY_LIVY_UI_KUBERNETES_SPARKUI_ENABLED: {value: "true"}
    LIVY_LIVY_UI_KUBERNETES_SPARKUI_LINK0FORMAT: {value: "/spark-ui/%s"}
historyserver:
  enabled: false
jupyterhub:
  enabled: true
  ingress:
    enabled: true
    annotations:
      kubernetes.io/ingress.class: nginx
      kubernetes.io/tls-acme: "true"
    hosts:
    - my-cluster.example.com
    pathSuffix: ''
    tls:
    - secretName: spark-cluster-tls
      hosts:
      - my-cluster.example.com
  hub:
    baseUrl: /jupyterhub
    publicURL: "https://my-cluster.example.com"
    activeServerLimit: 10
    # $> openssl rand -hex 32
    cookieSecret: 41b85e5f50222b1542cc3b38a51f4d744864acca5e94eeb78c6e8c19d89eb433
    pdb:
      enabled: true
      minAvailable: 0
  proxy:
    # $> openssl rand -hex 32
    secretToken: cc52356e9a19a50861b22e08c92c40b8ebe617192f77edb355b9bf4b74b055de
    pdb:
      enabled: true
      minAvailable: 0
  cull:
    enabled: false
    timeout: 300
    every: 60

Interactive sessions - Jupyter notebook on JupyterHub with Sparkmagic
Batch sessions - SparkPi:

curl -k -H 'Content-Type: application/json' -X POST \
  -d '{
        "name": "SparkPi-01",
        "className": "org.apache.spark.examples.SparkPi",
        "numExecutors": 2,
        "file": "local:///opt/spark/examples/jars/spark-examples_2.11-2.4.3.jar",
        "args": ["10000"],
        "conf": {
            "spark.kubernetes.namespace": "<namespace>"
        }
      }' "https://my-cluster.example.com/livy/batches"

Codecov Report

Merging #252 into master will decrease coverage by 1.29%.
The diff coverage is 43.26%.

@@             Coverage Diff             @@
##             master     #252     +/-   ##
===========================================
- Coverage     67.92%   66.62%   -1.3%
- Complexity      939      956     +17
===========================================
  Files           102      103      +1
  Lines          5876     6191    +315
  Branches        891      945     +54
===========================================
+ Hits           3991     4125    +134
- Misses         1314     1469    +155
- Partials        571      597     +26

Impacted Files
Coverage Δ
Complexity Δ

...ain/java/org/apache/livy/rsc/driver/RSCDriver.java
80.08% <0%> (-0.68%)
46 <0> (+1)

rsc/src/main/java/org/apache/livy/rsc/RSCConf.java
88.18% <100%> (+0.33%)
9 <1> (+1)
:arrow_up:

...main/scala/org/apache/livy/server/LivyServer.scala
32.88% <20%> (-0.46%)
11 <1> (ø)

...e/livy/server/interactive/InteractiveSession.scala
68.93% <33.33%> (-0.44%)
46 <0> (ø)

...ala/org/apache/livy/utils/SparkKubernetesApp.scala
43.01% <43.01%> (ø)
14 <14> (?)

...rc/main/scala/org/apache/livy/utils/SparkApp.scala
45.23% <5.55%> (-30.77%)
1 <0> (ø)

...rver/src/main/scala/org/apache/livy/LivyConf.scala
95.02% <85%> (-1%)
22 <1> (+1)

...cala/org/apache/livy/scalaapi/ScalaJobHandle.scala
50% <0%> (-2.95%)
7% <0%> (ø)

...c/src/main/java/org/apache/livy/rsc/RSCClient.java
72.28% <0%> (-1.21%)
20% <0%> (ø)

... and 1 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 c7313c5...2090bcb. Read the comment docs.

Codecov Report
Attention: Patch coverage is 42.58675% with 182 lines in your changes missing coverage. Please review.

Project coverage is 66.69%. Comparing base (ba12b51) to head (4050c0d).
Report is 74 commits behind head on master.

Files with missing lines
Patch %
Lines

...ala/org/apache/livy/utils/SparkKubernetesApp.scala
42.20%
130 Missing and 22 partials :warning:

...rc/main/scala/org/apache/livy/utils/SparkApp.scala
5.55%
15 Missing and 2 partials :warning:

...main/scala/org/apache/livy/server/LivyServer.scala
20.00%
3 Missing and 1 partial :warning:

...e/livy/server/interactive/InteractiveSession.scala
33.33%
1 Missing and 3 partials :warning:

...rver/src/main/scala/org/apache/livy/LivyConf.scala
85.00%
3 Missing :warning:

...ain/java/org/apache/livy/rsc/driver/RSCDriver.java
0.00%
1 Missing and 1 partial :warning:

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #252      +/-   ##
============================================
- Coverage     68.06%   66.69%   -1.37%
- Complexity      940      956      +16
============================================
  Files           101      103       +2
  Lines          5855     6189     +334
  Branches        886      946      +60
============================================
+ Hits           3985     4128     +143
- Misses         1299     1464     +165
- Partials        571      597      +26

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Only getting highlights from books in the root folder of the KOreader device

I use calibre to add books to my KOreader device, which by default makes a separate folder for each author on the device. Therefore almost all my books are in sub-folders corresponding to authors. When I use the KOreader sync plugin, I only get the highlights from the handful of books in the root folder of the device.
Is there a way around this or could you add a recursive folder sync option?

Same for me. I could maybe open a PR to fix the problem if you show me where it could be fixed

From what I understood from the code  @Edo78 you're looking for files called metadata*.lua on the reader https://github.com/Edo78/obsidian-koreader-sync/blob/master/src/koreader-metadata.ts#L23
The problem is that I don't have such files on my reader.


Use Structs for all stdlib Iterators

This change moves all stdlib Iterators to structs instead of classes. It's unclear to me whether we'd call this a breaking change, since technically somebody could take one of these Iterators, iterate it in once context, and expect it to exist in its iterated state elsewhere. This seems like unlikely to me, though. That said, the case in which this is beneficial also seems unlikely.

All iterators should have reference-like semantics, they are only structs when their instance variables are never assigned to.

 With it being a struct it's 1 then 1, because the state wasn't preserved.
So I don't think we should merge this PR.
That said, it would be great if we could use structs to reduce memory usage, but I can't see how.

All iterators should have reference-like semantics, they are only structs when their instance variables are never assigned to.

Not all iterators today follow this principle. For example:
https://github.com/crystal-lang/crystal/blob/84977e48c784811e885f084ae0270c2dd47f14e5/src/iterator.cr#L398-L421
Should these be updated to use classes?

With ItemIterator being a class, the output is 1 then 2. With it being a struct it's 1 then 1, because the state wasn't preserved.

I worded it poorly in my PR description, but this is what I intended to refer to (and what makes this technically a breaking change). That said, do you think it's a common case?

The only assignments to Cons's instance variables come from the constructor, there are none inside #next.

The only assignments to Cons's instance variables come from the constructor, there are none inside #next

Right. Essentially, if the iterator is immutable after it was constructed (meaning: instance vars don't get reassigned, but it's okay to change their value by, say, appending an item to an array) then we used a struct. Otherwise we use a class.

Ah sorry right, I grabbed a bad example. The next two iterators in that file are ConsTuple and Cycle. which do assign to their instance variables in next. I'm happy to throw together a separate PR if the direction you'd rather take this is to instead make those use reference semantics.
https://github.com/crystal-lang/crystal/blob/84977e48c784811e885f084ae0270c2dd47f14e5/src/iterator.cr#L441-L519

Yeah those are buggy. Here is a reproduction for #cycle:
a = [1, 2, 3]
i1 = a.each.cycle
i2 = i1

i1.next # => 1
i1.next # => 2
i1.next # => 3
i1.next # => 1
i1.next # => 2
i2.next # => 1 # should be 3

For #cons_pair:
a = [1, 2, 3]
i1 = a.each.cons_pair
i2 = i1

i1.next # => {1, 2}
i2.next # => #<Iterator::Stop:0x7fb22c6cbfe0>

Iterators which assign to their instance variables inside #next despite being structs:

Iterator::ConsTupleIterator
Iterator::CycleIterator
Iterator::FlattenIterator
StructIter in spec/std/iterator_spec.cr (this type was introduced in #10096 specifically to support iterators that don't have reference semantics, so it might be reverted)

Iterators which don't do that despite being classes:

Indexable::CartesianProductIteratorT
Indexable::CartesianProductIteratorN


IBA annotations with source = GOC

Hello,
I thought we shouldn't have any IBAs with source = GOC.
Any idea where these come from ? And I suppose they need to be filtered out.

@cmungall @huaiyumi @dougli1sqrd
Thanks, Pascale

Correction : They seem to be coming from TAIR. I suppose that's covered in another ticket ?

That is how the redundant annotations come from. I think we decided at the New York meeting to only use the IBAs from the paint gaf from now on.

Oh yes I remember now. Will they disappear completely, or will only the redundant ones be filtered though?
@cmungall

Closing as redundant with https://github.com/geneontology/go-site/issues/587

Will they disappear completely, or will only the redundant ones be filtered though?

They will disappear completely, otherwise there would be no way of purging old entries. PAINT will be the only source of IBAs


Fullscreen playback is broken #10

Should fix Fullscreen display for youtube videos. i ended up not using clappr's event delegation system because they didnt seem to propagate to the plugin if triggered in a base plugin.
I also added dist to the repository, so no building will be necessary after bower/npm installs

Wow, lots of changes. Let me take a look at this later. Thank you very much!


package.xml: Add catkin/ament_cmake runtime dependency for packaging

Required by REP-136
Also adds build_depend on Pinocchio as the default for BUILD_TESTING=ON - this prevents a configuration error (the error message in unittest/CMakeLists.txt) on CI.

Thanks :)


Add links column to CFF partnerships table

Dummy links for now -
Might be to raw data, but could also be analytics file. due to size of raw data.

@taz-chiles Is this actionable - or are we blocked on something?


feature request - pandas wrapper

hello,
is it possible to have a simple pandas wrapper around all these functions? To make it easier to integrate with people, who use pandas dataframes a lot?
I know it's just nicety and I'm happy to provide these, if this is desired.
kind regards

Hi! Do you mean that you want to pass to functions pandas Series?


[💡 FEATURE REQUEST]: Jobs plugin pipelines check not works for pipelines without queues

Plugin
JOBS
I have an idea!
Hi, @rustatian
According to documentation https://roadrunner.dev/docs/lab-health/current#jobs-plugin-pipelines-check
In my project, I have two types of pipelines (push only, and process-only)
for example:
status:
  address: <IP_ADDRESS>:8081
jobs:
  consume:
     - process_pipeline
  pipelines:
    push_pipeline: # only push pipeline
      driver: amqp
      config:
        exchange: default
        routing_key:  product.created
        exchange_type: direct

     process_pipeline: # only process pipeline
      driver: amqp
      config:
        exchange: default
        routing_key:  something.happened.outside
        exchange_type: direct
        queue:  service-name.processing.something.happened.outside

But when I'm trying to execute
curl localhost:8081/jobs
jobs plugin not found

and log
{"level":"error","ts":1694590936456822482,"logger":"status","msg":"jobs state","error":"jobs_plugin_drivers_state: empty queue name, add queue to the AMQP configuration"}

So is it possible to return pipeline statuses only from the jobs.consume section?

Hey @iborysenko 👋🏻
What is your RR version?

@rustatian
 rr version 2023.2.2 (build time: 2023-08-10T16:38:29+0000, go1.21.0), OS: linux, arch: amd64

If I drop all pipelines without queue-names everything is ok,
But at the same time my jobs.consume array only consists of correct-pipelines

@iborysenko Thanks 👍🏻
So the problem is that without a queue, you can't check the health of the pipeline. I think we can skip such a pipeline and check the health of the connection instead.
I'll fix that, thanks for the FR ⚡

@iborysenko Note that in 2023.3 a new feature would be available - custom routing_key when pushing the JOB. So instead of creating the new pipeline (push-only pipeline), you can avoid this and push jobs with the custom routing key. To any existing pipeline.
More here: https://github.com/roadrunner-php/jobs/pull/58

Hi @rustatian
I tried to reproduce it again in
rr version 2023.3.0 (build time: 2023-10-05T23:30:26+0000, go1.21.2), OS: linux, arch: amd64

Looks like nothing changed in these case
After  shutting down Rabbitmq ,
curl localhost:8081/jobs
curl: (52) Empty reply from server

Or maybe this is expected behavior?
Thanks

Hey @iborysenko 👋🏻
Is there anything in the DEBUG logs?

@iborysenko Double tested this case, result is on the screenshot. I guess you don't have RR on localhost. If it is a container, might be you made a mistake in the host:port mapping between the host and container network.

@rustatian
---
version: "3"
server:
  command: php public/index.php --env=prod
  env:
    APP_RUNTIME: Baldinof\RoadRunnerBundle\Runtime\Runtime
logs:
  mode: production
  encoding: json
  channels:
    http:
      level: info
    server:
      level: info
      mode: raw
rpc:
  listen: tcp://<IP_ADDRESS>:6001
status:
  address: <IP_ADDRESS>:8081
metrics:
  address: <IP_ADDRESS>:2112
amqp:
  addr: amqp://${MESSAGE_BUS_USERNAME}:${MESSAGE_BUS_PASSWORD}@${MESSAGE_BUS_HOST}:${MESSAGE_BUS_PORT}/${MESSAGE_BUS_VHOST}
jobs:
  num_pollers: 12
  pipeline_size: 100000
  pool:
    num_workers: 2
    max_jobs: 500
    supervisor:
      ttl: 18000s
  pipelines:
    produce_delivery_event_kitchen-created:
      driver: amqp
      config:
        exchange: internal
        routing_key: delivery.event.kitchen-created
        exchange_type: direct
        exchange_durable: true
        redial_timeout: 600
    produce_delivery_event_kitchen-activity-updated:
      driver: amqp
      config:
        exchange: internal
        routing_key: delivery.event.kitchen-activity-updated
        exchange_type: direct
        exchange_durable: true
        redial_timeout: 600
    produce_delivery_event_kitchen-opening-time-updated:
      driver: amqp
      config:
        exchange: internal
        routing_key: delivery.event.kitchen-opening-time-updated
        exchange_type: direct
        exchange_durable: true
        redial_timeout: 600
    produce_delivery_event_status_updated:
      driver: amqp
      config:
        exchange: internal
        routing_key: delivery.event.status.updated
        exchange_type: direct
        exchange_durable: true
        redial_timeout: 600
    produce_command_logging_add-log:
      driver: amqp
      config:
        exchange: internal
        routing_key: command.logging.add-log
        exchange_type: direct
        exchange_durable: true
        redial_timeout: 600
    process_delivery_command_wolt-drive_webhook:
      driver: amqp
      config:
        queue: delivery.processing.delivery.command.wolt-drive.webhook
        exchange: internal
        routing_key: delivery.command.wolt-drive.webhook
        exchange_type: direct
        exchange_durable: true
        durable: true
        consume_all: true
        redial_timeout: 600
    process_delivery_event_kitchen-created:
      driver: amqp
      config:
        queue: delivery.processing.delivery.event.kitchen-created
        exchange: internal
        routing_key: delivery.event.kitchen-created
        exchange_type: direct
        exchange_durable: true
        durable: true
        consume_all: true
        redial_timeout: 600
    process_oms_event_order-courier-assigned:
      driver: amqp
      config:
        queue: delivery.processing.oms.event.order-courier-assigned
        exchange: internal
        routing_key: oms.event.order-courier-assigned
        exchange_type: direct
        exchange_durable: true
        durable: true
        consume_all: true
        redial_timeout: 600
    process_oms_event_order-courier-canceled:
      driver: amqp
      config:
        queue: delivery.processing.oms.event.order-courier-canceled
        exchange: internal
        routing_key: oms.event.order-courier-canceled
        exchange_type: direct
        exchange_durable: true
        durable: true
        consume_all: true
        redial_timeout: 600
    process_oms_event_order-status-changed:
      driver: amqp
      config:
        queue: delivery.processing.oms.event.order-status-changed
        exchange: internal
        routing_key: oms.event.order-status-changed
        exchange_type: direct
        exchange_durable: true
        durable: true
        consume_all: true
        redial_timeout: 600
    process_partner_event_partner-user-created:
      driver: amqp
      config:
        queue: delivery.processing.partner.event.partner-user-created
        exchange: internal
        routing_key: partner.event.partner-user-created
        exchange_type: direct
        exchange_durable: true
        durable: true
        consume_all: true
        redial_timeout: 600
  consume:
  - process_delivery_command_wolt-drive_webhook
  - process_delivery_event_kitchen-created
  - process_oms_event_order-courier-assigned
  - process_oms_event_order-courier-canceled
  - process_oms_event_order-status-changed
  - process_partner_event_partner-user-created
...

Started
{"level":"debug","ts":1696679571819321659,"logger":"jobs","msg":"initializing driver","pipeline":"process_delivery_command_wolt-drive_webhook","driver":"amqp"}
{"level":"debug","ts":1696679571819057624,"logger":"jobs","msg":"initializing driver","pipeline":"process_delivery_event_kitchen-created","driver":"amqp"}
{"level":"debug","ts":1696679571819657018,"logger":"jobs","msg":"initializing driver","pipeline":"process_oms_event_order-courier-canceled","driver":"amqp"}
{"level":"debug","ts":1696679571819043001,"logger":"jobs","msg":"initializing driver","pipeline":"process_oms_event_order-status-changed","driver":"amqp"}
{"level":"debug","ts":1696679571819822386,"logger":"jobs","msg":"initializing driver","pipeline":"process_oms_event_order-courier-assigned","driver":"amqp"}
{"level":"debug","ts":1696679571819809560,"logger":"jobs","msg":"initializing driver","pipeline":"produce_delivery_event_status_updated","driver":"amqp"}
{"level":"debug","ts":1696679571819843409,"logger":"jobs","msg":"initializing driver","pipeline":"produce_delivery_event_kitchen-activity-updated","driver":"amqp"}
{"level":"debug","ts":1696679571819866214,"logger":"jobs","msg":"initializing driver","pipeline":"process_partner_event_partner-user-created","driver":"amqp"}
{"level":"debug","ts":1696679571819829357,"logger":"jobs","msg":"initializing driver","pipeline":"produce_command_logging_add-log","driver":"amqp"}
{"level":"debug","ts":1696679571819880741,"logger":"jobs","msg":"initializing driver","pipeline":"produce_delivery_event_kitchen-opening-time-updated","driver":"amqp"}
{"level":"debug","ts":1696679572067500687,"logger":"jobs","msg":"driver ready","pipeline":"produce_delivery_event_status_updated","driver":"amqp","start":1696679571823753431,"elapsed":0.243744397}
{"level":"debug","ts":1696679572067554891,"logger":"jobs","msg":"initializing driver","pipeline":"produce_delivery_event_kitchen-created","driver":"amqp"}
{"level":"debug","ts":1696679572067592934,"logger":"jobs","msg":"driver ready","pipeline":"process_delivery_event_kitchen-created","driver":"amqp","start":1696679571820030321,"elapsed":0.24756206}
{"level":"debug","ts":1696679572067663144,"logger":"jobs","msg":"driver ready","pipeline":"process_partner_event_partner-user-created","driver":"amqp","start":1696679571827186692,"elapsed":0.240476004}
{"level":"debug","ts":1696679572067770984,"logger":"jobs","msg":"driver ready","pipeline":"produce_delivery_event_kitchen-activity-updated","driver":"amqp","start":1696679571826961150,"elapsed":0.240809134}
{"level":"debug","ts":1696679572067802734,"logger":"jobs","msg":"driver ready","pipeline":"process_oms_event_order-courier-canceled","driver":"amqp","start":1696679571820169355,"elapsed":0.24763312}
{"level":"debug","ts":1696679572067806672,"logger":"jobs","msg":"driver ready","pipeline":"process_oms_event_order-courier-assigned","driver":"amqp","start":1696679571823004572,"elapsed":0.244801595}
{"level":"debug","ts":1696679572067533570,"logger":"jobs","msg":"driver ready","pipeline":"produce_delivery_event_kitchen-opening-time-updated","driver":"amqp","start":1696679571827542660,"elapsed":0.239989951}
{"level":"debug","ts":1696679572067865591,"logger":"jobs","msg":"driver ready","pipeline":"process_oms_event_order-status-changed","driver":"amqp","start":1696679571820377397,"elapsed":0.247487325}
{"level":"debug","ts":1696679572067505654,"logger":"jobs","msg":"driver ready","pipeline":"process_delivery_command_wolt-drive_webhook","driver":"amqp","start":1696679571819935221,"elapsed":0.247569213}
{"level":"debug","ts":1696679572072667695,"logger":"jobs","msg":"driver ready","pipeline":"produce_command_logging_add-log","driver":"amqp","start":1696679571827383254,"elapsed":0.245281892}
{"level":"debug","ts":1696679572089940787,"logger":"jobs","msg":"driver ready","pipeline":"produce_delivery_event_kitchen-created","driver":"amqp","start":1696679572067562080,"elapsed":0.022376102}
{"level":"debug","ts":1696679572114051084,"logger":"amqp","msg":"pipeline was started","driver":"amqp","pipeline":"process_oms_event_order-courier-canceled","start":1696679572067923940,"elapsed":0.0461256}
{"level":"debug","ts":1696679572114064080,"logger":"amqp","msg":"pipeline was started","driver":"amqp","pipeline":"process_delivery_event_kitchen-created","start":1696679572067632865,"elapsed":0.046430161}
{"level":"debug","ts":1696679572114016001,"logger":"amqp","msg":"pipeline was started","driver":"amqp","pipeline":"process_oms_event_order-courier-assigned","start":1696679572067977662,"elapsed":0.046035556}
{"level":"debug","ts":1696679572114073454,"logger":"amqp","msg":"pipeline was started","driver":"amqp","pipeline":"process_oms_event_order-status-changed","start":1696679572068007931,"elapsed":0.046064555}
{"level":"debug","ts":1696679572114530881,"logger":"amqp","msg":"pipeline was started","driver":"amqp","pipeline":"process_delivery_command_wolt-drive_webhook","start":1696679572068072466,"elapsed":0.046457833}
{"level":"debug","ts":1696679572114568649,"logger":"amqp","msg":"pipeline was started","driver":"amqp","pipeline":"process_partner_event_partner-user-created","start":1696679572067684291,"elapsed":0.046883677}
{"level":"debug","ts":1696679573056307977,"logger":"jobs","msg":"exited from jobs pipeline processor"}
{"level":"debug","ts":1696679573056398608,"logger":"jobs","msg":"exited from jobs pipeline processor"}
{"level":"debug","ts":1696679573056409557,"logger":"jobs","msg":"exited from jobs pipeline processor"}
{"level":"debug","ts":1696679573056415198,"logger":"jobs","msg":"exited from jobs pipeline processor"}
{"level":"debug","ts":1696679573056424690,"logger":"jobs","msg":"exited from jobs pipeline processor"}
{"level":"debug","ts":1696679573056421755,"logger":"jobs","msg":"exited from jobs pipeline processor"}
{"level":"debug","ts":1696679573056434230,"logger":"jobs","msg":"exited from jobs pipeline processor"}
{"level":"debug","ts":1696679573056429957,"logger":"jobs","msg":"exited from jobs pipeline processor"}
{"level":"debug","ts":1696679573056440407,"logger":"jobs","msg":"exited from jobs pipeline processor"}
{"level":"debug","ts":1696679573056443418,"logger":"jobs","msg":"exited from jobs pipeline processor"}
[INFO] RoadRunner server started; version: 2023.3.0, buildtime: 2023-10-05T23:30:26+0000
[INFO] sdnotify: not notified

CURL
curl localhost:8081/jobs
plugin: jobs: pipeline: produce_delivery_event_status_updated | priority: 10 | ready: false | queue:  | active: 0 | delayed: 0 | reserved: 0 | driver: amqp | error:
plugin: jobs: pipeline: process_delivery_command_wolt-drive_webhook | priority: 10 | ready: true | queue: delivery.processing.delivery.command.wolt-drive.webhook | active: 0 | delayed: 0 | reserved: 0 | driver: amqp | error:
plugin: jobs: pipeline: process_partner_event_partner-user-created | priority: 10 | ready: true | queue: delivery.processing.partner.event.partner-user-created | active: 0 | delayed: 0 | reserved: 0 | driver: amqp | error:
plugin: jobs: pipeline: produce_delivery_event_kitchen-activity-updated | priority: 10 | ready: false | queue:  | active: 0 | delayed: 0 | reserved: 0 | driver: amqp | error:
plugin: jobs: pipeline: process_oms_event_order-status-changed | priority: 10 | ready: true | queue: delivery.processing.oms.event.order-status-changed | active: 0 | delayed: 0 | reserved: 0 | driver: amqp | error:
plugin: jobs: pipeline: produce_command_logging_add-log | priority: 10 | ready: false | queue:  | active: 0 | delayed: 0 | reserved: 0 | driver: amqp | error:
plugin: jobs: pipeline: produce_delivery_event_kitchen-created | priority: 10 | ready: false | queue:  | active: 0 | delayed: 0 | reserved: 0 | driver: amqp | error:
plugin: jobs: pipeline: produce_delivery_event_kitchen-opening-time-updated | priority: 10 | ready: false | queue:  | active: 0 | delayed: 0 | reserved: 0 | driver: amqp | error:
plugin: jobs: pipeline: process_delivery_event_kitchen-created | priority: 10 | ready: true | queue: delivery.processing.delivery.event.kitchen-created | active: 0 | delayed: 0 | reserved: 0 | driver: amqp | error:
plugin: jobs: pipeline: process_oms_event_order-courier-canceled | priority: 10 | ready: true | queue: delivery.processing.oms.event.order-courier-canceled | active: 0 | delayed: 0 | reserved: 0 | driver: amqp | error:
plugin: jobs: pipeline: process_oms_event_order-courier-assigned | priority: 10 | ready: true | queue: delivery.processing.oms.event.order-courier-assigned | active: 0 | delayed: 0 | reserved: 0 | driver: amqp | error:

Shutdown Rabbitmq
{"level":"debug","ts":1696679970765113866,"logger":"amqp","msg":"exited from redialer"}
{"level":"debug","ts":1696679970766854528,"logger":"amqp","msg":"exited from redialer"}
{"level":"error","ts":1696679970766955700,"logger":"amqp","msg":"amqp connection closed","error":"Exception (504) Reason: \"channel/connection is not open\""}
{"level":"error","ts":1696679970769229423,"logger":"amqp","msg":"amqp connection closed","error":"Exception (504) Reason: \"channel/connection is not open\""}
{"level":"debug","ts":1696679970769023551,"logger":"amqp","msg":"exited from redialer"}
{"level":"error","ts":1696679970769850577,"logger":"amqp","msg":"publish channel close","error":"Exception (504) Reason: \"channel/connection is not open\""}
{"level":"error","ts":1696679970769990255,"logger":"amqp","msg":"amqp connection closed","error":"Exception (504) Reason: \"channel/connection is not open\""}
{"level":"error","ts":1696679970766968498,"logger":"amqp","msg":"amqp connection closed","error":"Exception (504) Reason: \"channel/connection is not open\""}
{"level":"error","ts":1696679970770062233,"logger":"amqp","msg":"pipeline connection was closed, redialing","error":"Exception (320) Reason: \"CONNECTION_FORCED - broker forced connection closure with reason 'shutdown'\"","pipeline":"produce_delivery_event_kitchen-created","driver":"amqp","start":1696679970769282473}
{"level":"debug","ts":1696679970768666107,"logger":"amqp","msg":"exited from redialer"}
{"level":"debug","ts":1696679970771293928,"logger":"amqp","msg":"exited from redialer"}
{"level":"error","ts":1696679970771315161,"logger":"amqp","msg":"amqp connection closed","error":"Exception (504) Reason: \"channel/connection is not open\""}
{"level":"debug","ts":1696679970771268833,"logger":"amqp","msg":"delivery channel was closed, leaving the rabbit listener"}
{"level":"debug","ts":1696679970771381934,"logger":"amqp","msg":"exited from redialer"}
{"level":"debug","ts":1696679970771409116,"logger":"amqp","msg":"number of listeners","listeners":0}
{"level":"error","ts":1696679970771441604,"logger":"amqp","msg":"amqp connection closed","error":"Exception (504) Reason: \"channel/connection is not open\""}
{"level":"error","ts":1696679970771392302,"logger":"amqp","msg":"pipeline connection was closed, redialing","error":"Exception (320) Reason: \"CONNECTION_FORCED - broker forced connection closure with reason 'shutdown'\"","pipeline":"process_delivery_event_kitchen-created","driver":"amqp","start":1696679970771389326}
{"level":"error","ts":1696679970771483620,"logger":"amqp","msg":"pipeline connection was closed, redialing","error":"Exception (320) Reason: \"CONNECTION_FORCED - broker forced connection closure with reason 'shutdown'\"","pipeline":"produce_delivery_event_kitchen-activity-updated","driver":"amqp","start":1696679970771472333}
{"level":"debug","ts":1696679970771015836,"logger":"amqp","msg":"exited from redialer"}
{"level":"error","ts":1696679970772151214,"logger":"amqp","msg":"amqp connection closed","error":"Exception (504) Reason: \"channel/connection is not open\""}
{"level":"error","ts":1696679970772182321,"logger":"amqp","msg":"pipeline connection was closed, redialing","error":"Exception (320) Reason: \"CONNECTION_FORCED - broker forced connection closure with reason 'shutdown'\"","pipeline":"process_oms_event_order-courier-canceled","driver":"amqp","start":1696679970772179671}
{"level":"error","ts":1696679970770124336,"logger":"amqp","msg":"pipeline connection was closed, redialing","error":"Exception (320) Reason: \"CONNECTION_FORCED - broker forced connection closure with reason 'shutdown'\"","pipeline":"produce_delivery_event_status_updated","driver":"amqp","start":1696679970770120283}
{"level":"debug","ts":1696679970768854996,"logger":"amqp","msg":"exited from redialer"}
{"level":"error","ts":1696679970768883123,"logger":"amqp","msg":"amqp connection closed","error":"Exception (504) Reason: \"channel/connection is not open\""}
{"level":"debug","ts":1696679970772569683,"logger":"amqp","msg":"exited from redialer"}
{"level":"error","ts":1696679970772602051,"logger":"amqp","msg":"amqp connection closed","error":"Exception (504) Reason: \"channel/connection is not open\""}
{"level":"error","ts":1696679970772946845,"logger":"amqp","msg":"pipeline connection was closed, redialing","error":"Exception (320) Reason: \"CONNECTION_FORCED - broker forced connection closure with reason 'shutdown'\"","pipeline":"process_oms_event_order-status-changed","driver":"amqp","start":1696679970772943328}
{"level":"error","ts":1696679970772901986,"logger":"amqp","msg":"pipeline connection was closed, redialing","error":"Exception (320) Reason: \"CONNECTION_FORCED - broker forced connection closure with reason 'shutdown'\"","pipeline":"produce_command_logging_add-log","driver":"amqp","start":1696679970769256901}
{"level":"debug","ts":1696679970772507474,"logger":"amqp","msg":"delivery channel was closed, leaving the rabbit listener"}
{"level":"debug","ts":1696679970773322918,"logger":"amqp","msg":"number of listeners","listeners":0}
{"level":"debug","ts":1696679970768497645,"logger":"amqp","msg":"delivery channel was closed, leaving the rabbit listener"}
{"level":"debug","ts":1696679970773345924,"logger":"amqp","msg":"number of listeners","listeners":0}
{"level":"debug","ts":1696679970769134094,"logger":"amqp","msg":"exited from redialer"}
{"level":"error","ts":1696679970767615173,"logger":"amqp","msg":"amqp connection closed","error":"Exception (504) Reason: \"channel/connection is not open\""}
{"level":"error","ts":1696679970773366622,"logger":"amqp","msg":"pipeline connection was closed, redialing","error":"Exception (320) Reason: \"CONNECTION_FORCED - broker forced connection closure with reason 'shutdown'\"","pipeline":"produce_delivery_event_kitchen-opening-time-updated","driver":"amqp","start":1696679970773364046}
{"level":"error","ts":1696679970770096535,"logger":"amqp","msg":"pipeline connection was closed, redialing","error":"Exception (320) Reason: \"CONNECTION_FORCED - broker forced connection closure with reason 'shutdown'\"","pipeline":"process_delivery_command_wolt-drive_webhook","driver":"amqp","start":1696679970770014304}
{"level":"debug","ts":1696679970770851658,"logger":"amqp","msg":"exited from redialer"}
{"level":"debug","ts":1696679970770953935,"logger":"amqp","msg":"delivery channel was closed, leaving the rabbit listener"}
{"level":"debug","ts":1696679970773682411,"logger":"amqp","msg":"number of listeners","listeners":0}
{"level":"debug","ts":1696679970770693036,"logger":"amqp","msg":"delivery channel was closed, leaving the rabbit listener"}
{"level":"debug","ts":1696679970774357026,"logger":"amqp","msg":"number of listeners","listeners":0}
{"level":"error","ts":1696679970770998389,"logger":"amqp","msg":"amqp connection closed","error":"Exception (504) Reason: \"channel/connection is not open\""}
{"level":"error","ts":1696679970772470104,"logger":"amqp","msg":"pipeline connection was closed, redialing","error":"Exception (320) Reason: \"CONNECTION_FORCED - broker forced connection closure with reason 'shutdown'\"","pipeline":"process_partner_event_partner-user-created","driver":"amqp","start":1696679970772467570}
{"level":"debug","ts":1696679970768114020,"logger":"amqp","msg":"delivery channel was closed, leaving the rabbit listener"}
{"level":"debug","ts":1696679970775771830,"logger":"amqp","msg":"number of listeners","listeners":0}
{"level":"error","ts":1696679970777033128,"logger":"amqp","msg":"pipeline connection was closed, redialing","error":"Exception (320) Reason: \"CONNECTION_FORCED - broker forced connection closure with reason 'shutdown'\"","pipeline":"process_oms_event_order-courier-assigned","driver":"amqp","start":1696679970777027182}

CURL
curl localhost:8081/jobs
curl: (52) Empty reply from server

This is expected behavior if the connection was interrupted.


"Proposed API is only available..." error in v1.47

VSCode Version: 1.47.0
OS Version: macOS 10.15.4, Darwin x64 19.4.0

This may happen with more extensions, but I've only confirmed two.
With 1.47 installed, when I switch to the view container for Todo+ or TODO List the view breaks and nothing is shown. I receive the following error:
"[$extension]: Proposed API is only available when running out of dev or with the following command line switch: --enable-proposed-api $extension"
When I switch back to 1.46 I no longer receive the error and the extensions work as expected.
Steps to Reproduce:

Install 1.47
Install fabiospampinato.vscode-todo-plus or tzachovadia.todo-list
Restart VSCode
Switch to extension specific tree view

Does this issue occur when all extensions are disabled?: No

@fabiospampinato Do have a different extension targeting 1.47 that uses proposed API or a new API for which we have forgotten to remote the proposed check?

/needsMoreInfo
Works me, with 1.47 and latest insider

Sorry, I did leave out one important part.
Before switching to the tree, create a file named "TODO" and put something like the following in it
TODO:
- item
Then switch to the tree view. You should see the notification and in the tree it will say "TODO", but not list "item" under it. In 1.46, it will show "item" under "TODO" in the tree
I tried it in both insiders and 1.47.

Thanks. I can now reproduce. Seeing the following error
/Users/jrieken/Code/vscode-all-stable-versions/Visual Studio Code-1.47.0.app/Contents/Resources/app/out/vs/workbench/workbench.desktop.main.js:2947 Error: [fabiospampinato.vscode-todo-plus]: Proposed API is only available when running out of dev or with the following command line switch: --enable-proposed-api fabiospampinato.vscode-todo-plus
	at a (/Users/jrieken/Code/vscode-all-stable-versions/Visual Studio Code-1.47.0.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:594)
	at Object.t.checkProposedApiEnabled (/Users/jrieken/Code/vscode-all-stable-versions/Visual Studio Code-1.47.0.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:594)
	at m.getTooltip (/Users/jrieken/Code/vscode-all-stable-versions/Visual Studio Code-1.47.0.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:611)
	at m.createTreeNode (/Users/jrieken/Code/vscode-all-stable-versions/Visual Studio Code-1.47.0.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:611)
	at m.createAndRegisterTreeNode (/Users/jrieken/Code/vscode-all-stable-versions/Visual Studio Code-1.47.0.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:610)
	at /Users/jrieken/Code/vscode-all-stable-versions/Visual Studio Code-1.47.0.app/Contents/Resources/app/out/vs/workbench/services/extensions/node/extensionHostProcess.js:609
	at processTicksAndRejections (/Users/jrieken/Code/vscode-all-stable-versions/Visual%20Studio%20Code-1.47.0.app/Contents/Resources/app/out/vs/code/electron-browser/workbench/internal/process/task_queues.js:85)
	at async Promise.all (/Users/jrieken/Code/vscode-all-stable-versions/Visual%20Studio%20Code-1.47.0.app/Contents/Resources/app/out/vs/code/electron-browser/workbench/index 0)

https://github.com/microsoft/vscode/blob/b086502c8d844aa010898a96f006ff8060a092cf/src/vs/workbench/api/common/extHostTreeViews.ts#L535
This code is correct but it seems a little too strict. From my understanding the todo extension returns something which isn't a markdown string (but also no other supported types). We should be graceful and only enforce proposed API when receiving a markdown string

Sorry again for leaving that out.
This also occurs with "tzachovadia.todo-list". I know Todo+ has an option to have formatting, which relates to what you found.  I don't know what Todo List is doing though, I have never seen it show any formatting in the tree.

@fabiospampinato Do have a different extension targeting 1.47 that uses proposed API or a new API for which we have forgotten to remote the proposed check?

@jrieken I don't quite remember now, but I haven't published any meaningful update to that extension in like a year so updating vscode at this point probably shouldn't break it.

Thank you for investigating @jrieken. I agree that this shouldn't be as strict if extensions are returning non-markdown unsupported types.

Is there a projected date for when a new version will be released that includes the fix for this issue?

We usually release a recovery build the Wednesday or Thursday about one week after the normal release.


Failed to read mpf generated by mpfi

Due to mpfi not creating the trash footer lines, but expected them while reading

(not yet reproducible)


feat: Expose Swiper's updateAutoHeight() method for autoHeight enabled slides

Ionic version:

[x] 4.x
I'm submitting a ...

[ ] bug report
[x] feature request

Current behavior:
Currently the ion-slides component allows the ability to pass in any valid Swiper options, one of which is the autoHeight option. When set to true this will nicely adjust the height of the slides to match the height of the active slide.
The issue is that if your slides contain data that could change, like waiting for a network request before loading items, the height of the active slide can change but the swiper will not update the height. There is a method available on the Swiper instance updateAutoHeight() that will update the height correctly after the fact.
Expected behavior:
The updateAutoHeight() method should be exposed on ion-slides component so a developer can trigger it after view changes to the active slide.
Steps to reproduce:
Create an ion-slides with 2 slides and set autoHeight to true. Have slide 1 contain nothing but a spinner and then after a set timeout switch to a list of 10 items. The height of the container will not update and will be whatever the height of the spinner was.
Related code:
<ion-slides [options]="{ autoHeight: true }">
...
</ion-slides>

Other information:
Swiper API: http://idangero.us/swiper/api/
Ionic info:

Ionic:

   ionic (Ionic CLI)             : 4.5.0 (/Users/daem0n/.nvm/versions/node/v10.11.0/lib/node_modules/ionic)
   Ionic Framework               : @ionic/angular 4.0.0-beta.11
   @angular-devkit/build-angular : 0.8.3
   @angular-devkit/schematics    : 0.8.3
   @angular/cli                  : 6.2.3
   @ionic/angular-toolkit        : not installed

Cordova:

   cordova (Cordova CLI) : 8.0.1-nightly.2018.9.20.f8ed27c6<EMAIL_ADDRESS>   Cordova Platforms     : Warning: using prerelease version 8.0.1-nightly.2018.9.20.f8ed27c6 (cordova-lib@9.0.0-nightly.2<IP_ADDRESS>b26e63), android 7.1.1, ios 4.5.5
   Cordova Plugins       : cordova-plugin-ionic-keyboard 2.1.2, cordova-plugin-ionic-webview 2.1.4, (and 20 other plugins)

System:

   Android SDK Tools : 26.1.1 (/Users/daem0n/Library/Android/sdk)
   ios-deploy        : 2.0.0
   NodeJS            : v10.11.0 (/Users/daem0n/.nvm/versions/node/v10.11.0/bin/node)
   npm               : 6.4.1
   OS                : macOS High Sierra
   Xcode             : Xcode 9.3 Build version 9E145

duplicate of https://github.com/ionic-team/ionic/issues/15079


Not display those school in list which is unsupported

As title

06:48 Updated

Add missing fonts (those font will be import in css)
Update setting page
Update setting js

Make sure there is no error message in console of developer tools.


DRILL-7790 : Build Drill with Netty version 4.1.50.Final

DRILL-7790: Build Drill with Netty version 4.1.50.Final
Updated Netty version to 4.1.50.Final and have modified the code accordingly
Documentation
(Please describe user-visible changes similar to what should appear in the Drill documentation.)
Testing
(Please describe how this PR has been tested.)

@alkakumari42
Can you please take a look at the merge conflict?
Thanks!

@alkakumari42
Can you please take a look at the merge conflict?
Thanks!

Hi,
Resolved the conflicts.
Thanks,
Alka

@alkakumari42
There are still branch conflicts.  Can you please take a look?
Thanks!

@alkakumari42
There are still branch conflicts. Can you please take a look?
Thanks!

Hi @cgivre ,
Please note that the changelist I have added is currently not with the intent of merging, but rather as a frame of reference. The actual issue is that the up-gradation which I tried did not seem to work, so was wondering if someone from the open-source community could provide some assistance by looking at the changelist, or, just generally suggesting what needs to be done in order to upgrade netty, since the one being used contains vulnerabilities.
Regards,
Alka

Hi @cgivre,
Did you get a chance to look into this issue?
Kindly, let me know if you find something wrong with my changelist or if there are some other changes needed as well.
Regards,
Alka

Hi @cgivre,
Did you get a chance to look into this issue?
Kindly, let me know if you find something wrong with my changelist or if there are some other changes needed as well.
Regards,
Alka

@alkakumari42
Thank you for the contribution.  We're happy to commit this, but we can't merge this until the conflicts are resolved.  Can you please rebase this PR on the current master, and fix any merge conflicts?  Finally, we'll need you to squash all commits.  If you need help with this, let me know.
Thanks and Happy Holidays!

Hi @cgivre,
Actually, the changes that I have made is not with the intent of merging to the codebase. It's more of a demonstration of Netty jar up-gradation. As we tried to upgrade the Netty jar, we found there are many breaking changes that have been introduced.
So, the issue is not to resolve the conflict and merge the code. It's about how can we upgrade some jars that are being used in opensource code.
We have tried to upgrade from our end and modify the code accordingly. But, it didn't seem to work. If there is some suggestion on how to upgrade some of the jars used by open-source code or should we wait for the Netty jar up-gradation by the open-source community?
Regards,
Alka

This has been resolved in https://github.com/apache/drill/pull/2185.  Closing PR.


[dino-park] Expose external accounts in GraphQL.

External Accounts are part of uris and the new schema is already merged.


Revisit/Add customized metadata

BTW, it seems that we have already supported the customized feature's functionality for (multiple_choice_qa](https://github.com/neulab/ExplainaBoard/blob/c3af2de7e06d7147b184efafa7b68c68d29f3b9a/explainaboard/tests/test_qa_multiple_choice.py#L66)


Make it more obvious the example needs rabbitmq

The example service on the index page encourages you to just try to run it, which fails if you haven't installed RabbitMQ. Make it more obvious you need to follow the installation process first.
This should perhaps get lumped into a larger piece of work to separate out the project page from the docs page.

Just about the doc. I only find code in test uinsg py.test framework to test service. Can we have some examples don't using py.test. In real project we may don't use py.test helper as client code to call service.

Nameko uses py.test for its own tests, and we encourage others to do so as well. It's also basically compatible with the other testing frameworks, so I don't see much value in adding non-pytest examples. You should be able to port the same concepts to another testing framework if you really need to.

It's obviously, but please add something like this to the example of index page:

To run above code, insure that you have RabbitMQ running, for example:
$ docker run  -p5672:5672 rabbitmq:latest

@mattbennett I can handle this addition on the docs if it's ok.

@tyler46 yes please!

@mattbennett great, I will prepare a PR for review.

Fixed by https://github.com/nameko/nameko/pull/591


Update install.mdx

revising many typos. It looks like you are using some kind of machine translating to get the English version?

revising many typos. It looks like you are using some kind of machine translating to get the English version?

No machine, just not native English 😅


⚠ Telly has degraded performance

In 9e1d797, Telly ($TLY) experienced degraded performance:

HTTP code: 200
Response time: 1225 ms

Resolved: Telly performance has improved in 6ee7ed8.


2.4.3

Mainly bug fixes, API improvements, new events, and also new placeholders.

2.4.3 will most probably be released in a fortnight or so, along with 1.19 support.

API developers, please see the information in the nl.mtvehicles.core.infrastructure.models package (at Javadoc). As of v2.5.0, classes in this package may be moved and/or renamed. You'll be informed when and how.
Furthermore, some of the deprecated methods, fields, or classes may be removed in v2.5.0.

P.S. - You'll see all the comments and warnings which I've mentioned as soon as this PR is merged, not yet. The same applies to the deprecated list.


[breaking change] <title

Change Intent
No response
Justification
No response
Impact
No response
Mitigation
No response
Change Timeline
No response
Associated CLs
No response

This seems an error...


Cursor not following text

When you're typing text, the cursor on the remote side stay at the position of the first letter included and doesn't "follow" the text as it's being typed.

Addressed in PR #63 .


perf(client): fail fast when an authentication error is detected

Based on top off #57
Diff

Tagged 0.8.2.


Publish to live

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

README.md
:white_check_mark:Succeeded

docs/develop/convert-javascript-to-typescript.md
:white_check_mark:Succeeded

docs/excel/excel-add-ins-events.md
:white_check_mark:Succeeded

docs/includes/file-get-started-excel-angular.md
:white_check_mark:Succeeded

docs/includes/file-get-started-excel-jquery.md
:white_check_mark:Succeeded

docs/includes/file-get-started-powerpoint.md
:white_check_mark:Succeeded

docs/includes/file-get-started-word.md
:white_check_mark:Succeeded

docs/includes/powerpoint-tutorial-setup.md
:white_check_mark:Succeeded

docs/index.yml
:white_check_mark:Succeeded

docs/overview/office-add-ins.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Task: support links

Overview
Adds support for the links provided by the open api description
Demo

Notes
Keyboard navigation is flimsy at the moment. Use your mouse to select the options that you want
Testing Instructions

How to test this PR
Prefer bulleted description
Start after checking out this branch
Include any setup required, such as bundling scripts, restarting services, etc.
Include test case, and expected output

@thewahome, we should track flimsy keyboard navigation so that it is not lost.

@thewahome, we should track flimsy keyboard navigation so that it is not lost.

It's hand;ed in this PR


[Release] sdk/resourcemanager/azurestackhci/armazurestackhci/2.0.0

https://github.com/Azure/sdk-release-request/issues/5369

The breaking changes are introduced and approved in this PR.

@JeffreyRichter , please approve this major version release by adding label of Approved Major Version, thanks.


ViewModel DataAnnotation translations not found is base class

Background
I did setup a simple mvc site and enable localization for both EN and FR languages.
The resources are located in a folder named Resources where Views and Models ressource files are stored. ViewLocalization and DataAnnotationLocalization are both enabled. I use QueryStringRequestCultureProvider to set the current culture. HomeController has a simple Index action that is associated with a model class HomeViewModel. This view model inherits from a base classe BaseViewModel.
Those classe are defined as such in their respective file:
   public class HomeViewModel: BaseViewModel     {         [Display(Name = "TITLE")]         [Required]         public string Title { get; set; }     }
       public class BaseViewModel     {         [Display(Name = "COMMON")]         [Required]         public string Common { get; set; }     }
Under Resources/Models folder I created 4 resource files (HomeViewModel.en.resx, HomeViewModel.fr.resx, BaseViewModel.en.resx, BaseViewModel.fr.resx).
Where translations for Title property are defined in HomeViewModel..resx and translations for Common are defined in BaseViewModel..resx
Issue
HomeViewModel Title property is properly translate depending on the current culture
However, Common is not translated and its key is displayed
Expected result
Translation of properties with DataAnnotation are looked down the inheritance chain.
Am I missing something?
Sample repro project available here: https://github.com/aguerot/l10n-dataannotation

Thanks for contacting us. We're closing this issue as there was no community involvement here for quite a while now.


Library calls setuid and/or setgid without setgroups or initgroups

When building package for openSUSE, rpmlint warns me:

This executable is calling setuid and setgid without setgroups or initgroups. There is a high probability this means it didn't relinquish all groups, and this would be a potential security issue to be fixed. Seek POS36-C on the web for details about the problem.

Some more info:
setuid/setgid are called from luv_setuid/luv_setgid here:
https://github.com/luvit/luv/blob/a0a98ec424e368cc2481faa03b99a25f403a5832/src/misc.c#L337-L353
The only usage I can find of uv.setuid/uv.setgid on Github is here: https://github.com/creationix/gardener/blob/b9e2d590cb48a01f5310f35516e68eb52bfc6f02/bridge/server.lua#L56-L58
Here's how Libuv handles this when spawning a new process: https://github.com/libuv/libuv/blob/abe4f3d58d1f89f7b9c3092a917486832ceff7a2/src/unix/process.c#L354-L373
(it calls setgroups(0, NULL) and then setgid and then setuid).
I don't know enough about this stuff to know what should be done here, but hopefully that can provide a starting point.
Some more reference links:

https://security.stackexchange.com/questions/122141/always-setgroups-before-setuid
https://www.oreilly.com/library/view/secure-programming-cookbook/0596003943/ch01s03.html
https://wiki.sei.cmu.edu/confluence/display/c/POS36-C.+Observe+correct+revocation+order+while+relinquishing+privileges


Possible typo in fatal logger name

When looking at the constant names in slf_model.go, I noticed that the fatal level name has only a single "L". It is spelled as "Fata" and "Level" combined. This is confusing when compared with the names of the other levels.
const (
	TraceLevel Level = iota
	DebugLevel
	InfoLevel
	WarnLevel
	ErrorLevel
	PanicLevel
	FataLevel
)

I suggest that another constant be added called FatalLevel with the same numeric value as FataLevel.

OMG, this is really a huge typo...
Thanks~

This typo has been fixed in v1.0.7~


Rich outputs missing on ReadTheDocs

Describe the bug
All the matplotlib figures on https://myst-nb.readthedocs.io are not rendering, along with the examples of displaying pd.DataFrame and IPython.display.Math, and the whole "Widgets and interactive outputs" notebook.
To Reproduce
Steps to reproduce the behavior:

Go to https://myst-nb.readthedocs.io/en/latest/use/glue.html, https://myst-nb.readthedocs.io/en/latest/examples/basic.html, or https://myst-nb.readthedocs.io/en/latest/examples/interactive.html.
Observe the missing outputs.

thanks @mje-nz, yes indeed some notebook executions are failing on RTD: https://readthedocs.org/projects/myst-nb/builds/11469995/
I've opened a PR above to look into fixing this #218

This may be part of the problem (https://github.com/executablebooks/MyST-NB/pull/218/checks?check_run_id=944445317):
----------------------------- Captured stdout call -----------------------------
## deleted test_complex_outputs_unrun.ipynb/cells/0/outputs/0:
-  output:
-    output_type: stream
-    name: stderr
-    text:
-      Matplotlib is building the font cache; this may take a moment.

done! also what's your RTD handle? I tried chrisjsewell but it said that didn't exist

done! also what's your RTD handle? I tried chrisjsewell but it said that didn't exist

I'm there as cjsewell, but it doesn't matter about me, the bot is our new ruler

FYI I enabled some versions on RTD: The plots are there in https://myst-nb.readthedocs.io/en/v0.8.3/use/hiding.html but missing in https://myst-nb.readthedocs.io/en/v0.8.4/use/hiding.html

huh this is really weird, I haven't noticed any plots missing in the jupyter book docs, which I believe use the latest myst-nb? https://jupyterbook.org/interactive/hiding.html#hide-cell-inputs

I'm just updating jupyter-cache first (to use nbclient), then I'll give it another try

Err, I solved the main bug afeb1b0c675b18ef90c63a897789f20aba1c1ae3, but there is still this RTD specific problem: https://github.com/jupyter/nbclient/issues/95

That's pretty weird... I wonder why rtd is behaving differently from the other ci/cd envs. Maybe a dependency they pre install or something?

Think I've found the difference: https://github.com/readthedocs/readthedocs.org/issues/7360

This is now fixed for our documentation 🎉
Apparently RTD was using a "larger" builder for the PR build (see readthedocs/readthedocs.org#7360).
I'm getting clarification on what the specs are for small/large builders to feed back to jupyter/nbclient#95) and if its an issue they could/should be handling.
Otherwise, I guess it might be good to document this potential issue for other RTD users.

This should also help https://github.com/jupyter/nbclient/issues/101, but will have to wait until 0.4.3 is released

nbclient 0.5 should hopefully now stop this from happening


click listener to items

Thanks for lib. :)
but how to add onclicklistener to items ?

@MarsXan  Did you find the decision?

@elaman16  Unfortunately Noooo :(


Feat/guide new plex guide what does my media player support

Pull Request
Purpose

a excel sheet with several tested Media Player devices with info that has been provided by our community.
Approach

[x] Created a excel sheet with several tested Media Player devices with info that has been provided by our community.
[x] Add a link in the how to setup a quality profile guide to the media player guide

Open Questions and Pre-Merge TODOs

Requirements

[x] These changes meet the standards for contributing.
[x] I have read the code of conduct.

@zakkarry
perhaps we should add something between here

that it might be also useful for JF/Emby users ?

@zakkarry perhaps we should add something between here

that it might be also useful for JF/Emby users ?

changes after the first review lgtm....not sure what you want to add? I think if anything maybe just a admonition that states that while these are specific for plex, it is possible that the same principle can be applied to JF/Emby?
While these principles are based on Plex and the capabilities of certain devices, the capabilities of devices apply to JellyFin and Emby.  Assuming that it's possible to do the equivalent streaming method (direct or indirect play) with your media server, the hardware should act the same as if Plex was used.
I'm not sure of the messaging you intended, but that's a rough draft of what I think you're trying to fit in there?


CFFL

CFFL

Linter check found the following problems:
The following problems have been found:
ERROR: /tmp/abs_bbshdsvwe3/clone/recipe/meta.yaml:14: pip_install_args: pip install should be run with --no-deps.
ERROR: /tmp/abs_bbshdsvwe3/clone/recipe/meta.yaml:20: missing_wheel: For pypi packages, wheel should be present in the host section
ERROR: /tmp/abs_bbshdsvwe3/clone/recipe/meta.yaml:33: missing_pip_check: For pypi packages, pip check should be present in the test commands
ERROR: /tmp/abs_bbshdsvwe3/clone/recipe/meta.yaml:45: missing_documentation: The recipe is missing a doc_url or doc_source_url
ERROR: /tmp/abs_bbshdsvwe3/clone/recipe/meta.yaml:45: missing_dev_url: The recipe is missing a dev_url
Errors were found


MicroPython: Magic C++ Memory Management.

This is more a test to see if I'm barking up the right tree than a fix... yet.

Okay as a result of sleuthing for #720 I ended up raising https://github.com/micropython/micropython/issues/11116
After raising my concerns in https://github.com/micropython/micropython/commit/c80e7c14e6305e50e3b39f97172d4d8fe1214d3b
And spending a large amount of time trying to avoid new and delete.
Unfortunately, C++ does a lot of heap allocation that's difficult to control. In fact our project does ~10k and it's a job for Future Phil to figure out why on earth we're using so much heap because that seems extreme.
As a result of hours upon hours of tinkering I finally came up with https://github.com/Gadgetoid/micropython-cppmem which is integrated directly into pimoroni-pico in this PR.
This new module just needs to be linked into a MicroPython build and it takes over all C++ memory allocations in two ways:

All pre-init allocations end up in a 10k, linear, non-freeing memory pool that survives until hard-reset
Once init is done, the module switches into "MICROPYTHON" mode, where new and delete map to m_malloc and m_free respectively.

A side-effect of this approach is that any memory resulting from an implicit or explicit use of new in C++ modules cannot be relied upon to not be garbage collected.
This is to say that all our best practices still apply- if you're allocating something that you want to last for more than one C++ function call then it needs to be allocated with m_new and a pointer to the resulting memory region must be stored on an active MicroPython object. Like how our classes work with pointers to buffers and i2c objects.

The builds breaking and failing to start up when the cppmem/micropython module is included before other modules suggest that __attribute__ ((init_priority (65535))) is not global. If this is the case, then moving the wakeup module further up the list of C++ modules could make Badger 2040 / Badger 2040 W wakeup button detection a little more responsive.

Should probably move the switch to MicroPython heap mode into boot.py since we can be sure it will always run after the MicroPython heap, and C++ objects, have initialised.