Handle "Service connection disposed" error from VmService disconnecting while requests are outstanding

Fixes umbrella issue https://github.com/flutter/flutter/issues/153471, including its children: https://github.com/flutter/flutter/issues/153472, https://github.com/flutter/flutter/issues/153473, and https://github.com/flutter/flutter/issues/153474.
The VM service can be disposed at any time during requests (e.g. the user closes the app or stops debugging in VSCode)^1. https://github.com/dart-lang/sdk/commit/a479f91e80875dd6661b12108c9b81bdaeb2af65 and https://github.com/dart-lang/sdk/commit/a7d8707a5964e174f2ce321d075e96e37a9a16ad updated package:vm_service to throw new RPCErrors when the service disconnects while requests are inflight. Therefore, we need to handle these exceptions in the tool. See umbrella issue for more details.
I plan on cherry-picking this change to the stable channel.
Pre-launch Checklist

[x] I read the Contributor Guide and followed the process outlined there for submitting PRs.
[x] I read the Tree Hygiene wiki page, which explains my responsibilities.
[x] I read and followed the Flutter Style Guide, including Features we expect every widget to implement.
[x] I signed the CLA.
[x] I listed at least one issue that this PR fixes in the description above.
[x] I updated/added relevant documentation (doc comments with ///).
[x] I added new tests to check the change I am making, or this PR is test-exempt.
[x] I followed the breaking change policy and added Data Driven Fixes where supported.
[x] All existing and new tests are passing.

fyi @bkonyi @DanTup


Prepare ctypes 0.9.2

Would you mind having a look at opkg ocamldoc ctypes to see if the errors are fixable (they may not be) ?

> opkg ocamldoc ctypes
File "/Users/dbuenzli/.opam/4.03.0/lib/ctypes/cstubs_internals.mli", line 15, characters 13-29:
Error: Unbound module Ctypes_ptr
1 error(s) encountered
File "/Users/dbuenzli/.opam/4.03.0/lib/ctypes/ctypes_static.mli", line 39, characters 22-47:
Error: Unbound module Ctypes_bigarray
1 error(s) encountered
File "/Users/dbuenzli/.opam/4.03.0/lib/ctypes/dl.mli", line 44, characters 48-64:
Error: Unbound module Ctypes_ptr

I'm not sure that they errors are fixable, unfortunately: the problems are with references to internal modules for which the cmi/cmt/cmti files are deliberately not installed.

Ok but are these interfaces supposed to be used by end-users ? If not you could avoid installing them. FWIW here's the current rendering you get with ctypes.

In general opkg will consider whatever cmti or mli file installed as a wish from the package to be included in the API documentation.

The difficulty arises with modules whose types need to be exposed so that generated code can be type-checked but that are not intended to be used directly by end users.  It sounds like the best thing is to install only cmi (not cmti) files for those modules.  #445 makes that change.

Note that eventually I plan to add OPAMv2 directives to be specified in your package file to guide a bit what opkg does e.g. we could have an x-opkg-doc-exclude: <list of cmti/mli> but if the use cases are low I prefer not to.

Just one question about about the changes you made to Dl. Do the changes make sense from an API point of view or did opkg force something on you you didn't want to do ?

The changes make sense from an API point of view.  It seems unreasonable to expose functions in the public interface if those function have types whose names aren't also exposed.  So I changed the type of Dl.dlsym to only use public types.

Ok thanks.


Textmate: improvement on record entries and closure

Improvement on closure parameters parsing ({|...| })
Improvement on record entries (fields name) parsing (field:)

Known issue :
There are issues in interpolated strings if there is a quote following by a colon.

I tested it. Seems good enough for now. Thank you again!


Checkpoint.docFromText is broken for bsize=None

It looks like the bsize=None path hasn't been exercised in some time.  At the very least it is broken for keep_dims="flatten", because self.doc expects "flatten" to be transformed into "return_mask", and that only happens on the bsize path.  It also looks like the rest of the processing in the bsize path is also important for the return value to be transformed correctly but I have not looked more deeply into that.
Here is the exception for the keep_dims issue:
  File "/home/jonathan/Projects/colbert-live/colbert_live/colbert_live.py", line 98, in encode_chunk
    embeddings = self._cp.docFromText([content], keep_dims="flatten", pool_factor=None)
  File "/home/jonathan/miniforge3/envs/colbert-live/lib/python3.10/site-packages/colbert/modeling/checkpoint.py", line 192, in docFromText
    return self.doc(input_ids, attention_mask, keep_dims=keep_dims, to_cpu=to_cpu)
  File "/home/jonathan/miniforge3/envs/colbert-live/lib/python3.10/site-packages/colbert/modeling/checkpoint.py", line 92, in doc
    D = super().doc(*args, **kw_args)
  File "/home/jonathan/miniforge3/envs/colbert-live/lib/python3.10/site-packages/colbert/modeling/colbert.py", line 96, in doc
    assert keep_dims in [True, False, 'return_mask']

More generally, the options in docFromText don't all play nice together.  Eg., pool_factor is ignored unless keep_dims='flatten'.


serializers: datacite tests

Adds tests for DataCite serializer.

Fixes issue with OAI-PMH serialization.

Signed-off-by: Lars Holm Nielsen<EMAIL_ADDRESS>

Coverage increased (+8.5%) to 99.444% when pulling 331d73c1113ec76abd6a89c5dd129bdf4a0b34ff on lnielsen:datacitefix into 3975fd7c99745a9aa6362386936a918c0f878940 on inveniosoftware:master.

Coverage increased (+8.5%) to 99.444% when pulling cc69072269f5b5bea898ae853521fd1b46e37ba6 on lnielsen:datacitefix into 3975fd7c99745a9aa6362386936a918c0f878940 on inveniosoftware:master.

@lnielsen looks good for me. Can I merge or are you still working on it?
Maybe we should create an issue for removing fields from the records' metadata returned by search. I don't think we have such a mechanism yet. It will be needed to remove sensitive information before returning them to users, whatever the serializer.

It's ready from my site. Thanks for the review.

Removing sensitive data

This ensures that sensitive data cannot be searched on (when using bare ES syntax).
This serialiser removes stuff which is not for the public eye.


Live2Vod contents playback issue

Have you read the Tutorials?
Yes
Have you read the FAQ and checked for duplicate open issues?
Yes
What version of Shaka Player are you using?
4.3.5
Please ask your question
we are encountering issues with Live2Vod contents stuttering/flashing while streaming at content playback launch.
if you skip a few frames forward or restart the event the issue looks to resolve itself.
This problem has been observed on the following devices:
Device Google Chromecast 4K with Android TV
Player Shaka v 4.3.5
OS - Android 13
This issue is mainly observable on GoogleTV Chromecast 4K, but please take in count that it can be also replicate in a less evident way also on other platforms (e.g. Chromecast casting from ios or android application, LG smarttv application)
about the contents generation workflow.
We have a cluster of elemental encoder, but for a single profile, we use one single encoder to publish the content on two different Publishing Endpoint on two different Live Origins.
Then no sync issue on source side because the encoder is the same.
Live Origins (1 and 2) have the same configuration and the same live archive (4days).
Live Origin 1 is used for live streaming.
Live Origin 2 is used to generate capture using Unified tools installed on RMP and triggered by external WF manager.
the issue seems to be related to the frame_accurate options used during the l2v capturing process. When the --frame_accurate option is used, Unified Capture transcodes the first and last fragments of a clip to allow for sample accurate timing.
we identified differences between the streams generated using frame accurate.
Following two contents generated using capture unified tool with and without frameccurate option.
We analyze them using ffmprobe tool and what can be observed is that the stream structure for the two cases is different.
FRAME ACCURATE OFF
https://vod.rte.ie/vod-d/RTE_Test_-_Integration/937/280/IH10002380-24-0001/IH10002380-24-0001-1704193591844_nodrm.ism/.mpd
ffprobe -i "https://vod.rte.ie/vod-d/RTE_Test_-Integration/937/280/IH10002380-24-0001/IH10002380-24-0001-1704193591844_nodrm.ism/.mpd" -show_frames -select_streams v:5 | grep "pict_type" >> /tmp/struct_frame_accurate_off.txt
all GOPs have IBBPBBP scructure
[FRAME]
key_frame=1
pkt_pts=24
pkt_pts_time=0.040000
pkt_dts=24
pkt_dts_time=0.040000
pict_type=I
coded_picture_number=0
[/FRAME]
[FRAME]
key_frame=0
pkt_pts=48
pkt_pts_time=0.080000
pkt_dts=48
pkt_dts_time=0.080000
pict_type=B
coded_picture_number=2
[/FRAME]
[FRAME]
key_frame=0
pkt_pts=72
pkt_pts_time=0.120000
pkt_dts=72
pkt_dts_time=0.120000
pict_type=B
coded_picture_number=3
[/FRAME]
[FRAME]
key_frame=0
pkt_pts=96
pkt_pts_time=0.160000
pkt_dts=96
pkt_dts_time=0.160000
pict_type=P
coded_picture_number=1
[/FRAME]
FRAME ACCURATE ON
https://vod.rte.ie/vod-d/RTE_Test_-_Integration/655/424/IP10000879-01-0014/IP10000879-01-0014-1686931010344_nodrm.ism/.mpd
ffprobe -i "https://vod.rte.ie/vod-d/RTE_Test-_Integration/655/424/IP10000879-01-0014/IP10000879-01-0014-1686931010344_nodrm.ism/.mpd" -show_frames -select_streams v:5 | grep "pict_type" >> /tmp/struct_fram_accurate_on.txt
(only) the first GOP is composed by only from I and P frames with different dts and pts
the others have IBBP scructure as expected
[FRAME]
key_frame=1
pkt_pts=24
pkt_pts_time=0.040000
pkt_dts=0
pkt_dts_time=0.000000
pict_type=I
coded_picture_number=0
[/FRAME]
[FRAME]
key_frame=0
pkt_pts=48
pkt_pts_time=0.080000
pkt_dts=24
pkt_dts_time=0.040000
pict_type=P
coded_picture_number=1
[/FRAME]
[FRAME]
key_frame=0
pkt_pts=72
pkt_pts_time=0.120000
pkt_dts=48
pkt_dts_time=0.080000
pict_type=P
coded_picture_number=2
[/FRAME]
[FRAME]
key_frame=0
pkt_pts=96
pkt_pts_time=0.160000
pkt_dts=72
pkt_dts_time=0.120000
pict_type=P
coded_picture_number=3
[/FRAME]
summarizing, it seems that, when the frame_accurate option is used, the structure of the first GOP is slight different and it could cause the stuttering issue reported. do you have any suggestions how we can solve or mitigate this issue on client side? following the current shakaplayer configuration used on client side.
these are the configurations passed by us:
AD_MIME_TYPES: ['dash', 'hls', 'mp4'],
SHAKA_DRM_TYPES: ['com.widevine.alpha'],
SHAKA_CONFIG_STREAMING: {
bufferingGoal: 15,
rebufferingGoal: 5,
retryParameters: { baseDelay: 2000, maxAttempts: 1, backoffFactor: 1, timeout: 40000, fuzzFactor: 0 },
},
SHAKA_RETRY_CONFIG_MANIFEST: {
maxAttempts: 3,
baseDelay: 2000,
backoffFactor: 1,
fuzzFactor: 0
},
frameaccurate ON content:
https://vod.rte.ie/vod-d/RTE_Test_-_Integration/655/424/IP10000879-01-0014/IP10000879-01-0014-1686931010344.ism/.mpd
frameaccurate OFF content:
https://vod.rte.ie/vod-d/RTE_Test_-_Integration/937/280/IH10002380-24-0001/IH10002380-24-0001-1704193591844_nodrm.ism/.mpd

From what you say, it seems that the problem is in the decoding, which in this case is handled by the browser and not by us. Since it is not a problem with the generated manifest but with the segment itself, I cannot be of more help to you... sorry

Any way to process and manage the first GOP frames prior to send them to the decoder?
Thanks!


bug: Not send tracing data headers to upstream

Issue description
Not send tracing data headers to upstream
Environment

apisix version (cmd: apisix version):
OS: centos

Minimal test code / Steps to reproduce the issue
your steps

What's the actual result? (including assertion message & call stack if applicable)
What's the expected result?

fixed by https://github.com/apache/incubator-apisix/pull/713


Unable to setup COE Starter Kit Power BI

Hi,
We're stuck on Step 6: Configure the Power BI Dashboard in CoE Starter Kit - Setup Instructions.docx. We tried both the "Get the Power BI Dashboard from App Source", and "Configure the Power BI Dashboard using the PBIX file" options.
A screen capture history for the App Source option is in COEStarterKitPowerBISetup.zip.
It looks like login credentials are accepted, and the setup appears to follow what is described in the instructions. However, we only see the sample data. It appears there is no connection to our data after repeated attempts to connect.
The pbix file option uses the Edit Queries button in the Ribbon of Power BI Desktop. However, this button does not seem to exist in our recently updated version of Power BI Desktop.
COEStarterKitPowerBISetup.zip

In the new version of Power BI, can you try going to Transform Data

And then selecting OrgUrl there and updating it (from there the steps should be similar to the ones outlined in the documentation)

It looks like the ribbon has recently changed with a new version of Power BI Desktop, so we do need to update our documentation to reflect that.

Hi,
Was able to set the OrgUrl, and Publish, the .pbix. Published it to the existing New Power Platform CoE Toolkit Tenant Adoption workspace. The new screen capture shows our data with me as the #1 App Maker. :-)
However, it also still shows, "You're viewing sample data. After the data is connected and refreshed, reload the page."
We removed the sample report "Dashboard-PowerPlatformAdminDashboard_2020-02-12" from Reports, but are not allowed to Delete the sample dataset, "Dashboard-PowerPlatformAdminDashboard_2020-02-12" from Datasets.
So the "Get the Power BI Dashboard from App Source" option won't work for us, and the "Configure the Power BI Dashboard using the PBIX file" sort of works for us.
Thanks,
Ron

The AppSource option is still very new, looks like we might need to fix some bugs there! When you refresh the page, does the warning message go away?
It does look like it refreshed properly with your environments, so might just be an issue with the warning message being displayed incorrectly.
Congrats on being Top Maker!

Hi,
Yup ctrl-F5 did it.
The AppSource option definitely seems more user friendly. I'll be looking for a chance to try it again,
Thanks again for all your help!
Ron

Awesome, really appreciate your detailed error messages and explanations - very helpful for us updating the documentation!


otg mode screen orientation

[x] I have read the FAQ.

[x] I have searched in existing issues.

Environment

OS: Windows 11
scrcpy version: v2.3.1
installation method: Windows Release
device model: Motorola Edge Plus 2023
Android version: 14

Describe the bug
I can use scrcpy perfectly fine with ( --otg ) when my phone is in portrait mode. I can scroll, click, etc. all as intended on all parts of the screen. When I switch my phone to landscape mode, I lose all mouse input on the right hand side of the phone screen as if scrcpy hasn't recognized my phone has switched to landscape mode.
Is there any way to change the screen orientation that scrcpy views the phone as when using (--otg)?

What happend with a real mouse (a bluetooth mouse for example)?

What happend with a real mouse (a bluetooth mouse directly connected to the device for example)?

I'd like to be able to use a keyboard as well. My laptop is not as powerful as my phone, funnily enough.


Auto-discover EKS cluster endpoint by cluster name via IRSA

Tell us about your request
Karpenter already has a IRSA to manage the EC2 instances. By increasing the scope of that role, the cluster endpoint could be automatically discovered, removing the need to provide the value during creation of the helm chart.
Endpoint to call: https://docs.aws.amazon.com/eks/latest/APIReference/API_DescribeCluster.html
Additional IAM Permission required: "eks:DescribeCluster" which can (and should) be scoped to the single cluster.
A suggestion would be to add another value for the helm chart autodiscoverClusterEndpoint which can be used for that feature.
Tell us about the problem you're trying to solve. What are you trying to do, and why is it hard?
We are not creating the Helm chart with terraform, but with a purely GitOps based workflow after the cluster has been created. During that phase, we usually only have the name of the cluster, but not the actual endpoint address, requiring an additional lookup and mapping of that value.
Are you currently working around this issue?
Creating a custom mapping from cluster name to endpoint.
Additional Context
No response
Attachments
No response
Community Note

Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
Please do not leave "+1" or "me too" comments, they generate extra noise for issue followers and do not help prioritize the request
If you are interested in working on this issue or have submitted a pull request, please leave a comment

The AWS implementation of Karpenter intentionally does not rely on EKS APIs and instead relies on assumptions of the AWS Cloud Provider for Kubernetes.

It's still using the aws sdk for go, isn't it? At least it's used for controlling the ec2 instances. Why can't that sdk be used for also resolving the aws specific cluster endpoint configuration?

It's straightforward to make the implementation do so, the problem is that it would break for any user with a cluster on EC2 who is not using EKS. You could do partial behavior, where we attempt to discover the cluster if it's EKS and fail otherwise, but this seems like it may be more confusing for users than other users are helped by the discovery.

Thanks I see, what are your thoughts on adding a eks specific subsection (under aws) into the helm chart for that flag?

Can you elaborate on the use case a little bit? How much simpler is name than name/endpoint/cabundle? Does it change your installation significantly? One guess is that name is well known at config time, where endpoint/cabundle are dynamic. Does this enable you to use less tools during installation?
We've heard this ask enough times that I wonder if it's worth rethinking our stance (above), but I want to understand the use case better.
Consider me open minded to doing this either in helm or as a runtime arg (incl. dependency on eks:DescribeCluster)

Sure @ellistarn :
In this case, (for now) won't make the manual mapping a significant less effort, but it could make one in the future. The main concern is that the first usage experience. Karpenter can auto-discover "everything", except the endpoint address (for EKS), which feels strange.
Regarding less tools: For my current usage, we won't require calling the aws cli to read the cluster endpoint address during setup to populate a local mapping file, which we currently only require for this use case.

👍  Chatting with the team about this today.

@ellistarn is there an outcome from your discussion? Is this something you would like to support in the future? Would you be interested in a PR for this?

Supportive of a PR for this. Perhaps @jonathan-innis can chime in, too?

cluster-endpoint is now optional
If cluster-endpoint is not set, discover at startup using cluster name

Agree with @ellistarn. This seems reasonable to me. The default install could have the DescribeCluster on the single resource and auto-discover the api-endpoint.

@woehrl01 If you're interested in opening a PR around this, I'm happy to assign this issue to you 👍

@jonathan-innis Sure, would definitely give this a try. I already tried to make myself familiar in the repo. It looks like that https://github.com/aws/karpenter/blob/main/pkg/apis/settings/settings.go#L72-L93 would be the place to add the lookup, do you agree?

@woehrl01 Yes, I think you are right. We would want to encapsulate this lookup in the settings package. I think in order to make our dependency graph look right, we may want to create a session package so that we generate sessions that all look the same as this session. We can instantiate the eks api using that session and call-out to it to describe the cluster API endpoint.

@jonathan-innis I looked a bit more through the repo and found the "launchtemplate" package. It looks like that this is the right place in my opinion to add the logic:
https://github.com/aws/karpenter/blob/main/pkg/cloudprovider/launchtemplate.go#L136-L160
Here, we already have access to the ec2api (so we simply add the additional API), and it won't require the whole controller to restart if there are any changes in the infrastructure. Thoughts?

+1 to launchtemplate and not in settings.


Execute wasm contract failed when using extension

When I try to executing contract using Extension, I'm got an error: "rpc error: code = InvalidArgument desc = failed to execute message; message index: 0: Error parsing into type contract::msg::HandleMsg: unknown variant eyJhZGRfcGxheWVyIjp7fX0=, expected add_user: execute wasm contract failed: invalid request".
What's the correct way of using the extension?
Code:
`
this.client = new LocalTerra();
this.extension = new Extension();
this.extension.connect();
this.extension.on("onConnect", (w: Wallet) => {
this.wallet = w;
});
const execute = new MsgExecuteContract(
this.wallet.address, // sender
"terra1tndcaqxkpc5ce9qee5ggqf430mr2z3pefe5wj6", // contract account address
{ add_user: {} }, // handle msg
{ uluna: 100000} // coins
);
this.extension.post({
msgs: [execute],
});
this.extension.once("onPost", async (data: any) => {
console.log("post", data);
});
`

The latest terra.js and LocalTerra is compatible with bombay network

@hanjukim could you please clarify, what should I do, if I need to test the contract execution with the extension on my local machine before I go to the Testnet?


Changing engine properties inside slots

I'm looking into replacing qmlrs with qml-rust in one of my projects. The backend of the application exposes its data using a bunch of QListModel instances to the fontend written in QML. When to user kicks off an action the QML side calls a slot of a singleton object. The slot handler code on the Rust side will do his thing and then update some of the QListModel instances and return. My problem is that I can't figure out how to get a hold of a reference to the QListModels. I tried to make them properties of the singleton which failed, I also had the idea to put the QmlEngine instance into a global static RwLock. Any ideas?

@flanfly Well, one of my app (it's pretty simple) Kefia uses QListModel as a data storage.
It uses a following approach:

There is a field list in a Packages struct which is essentially a QListModel instance inside. (It is generated though Q_LIST_MODEL macro here
When user interacts with a list, QML rings a slot of QPackages (that is a wrapper on the Packages struct).
Rust code modifies a list with a calculated data via set_data
So, the QListModel instance should be stored inside a model, but not as a property, but as a field of a struct.

That is what we have for now. I hope I described it okayish.

Also, properties are supported, but it is not really comfortable to use them due to limitations of macro system on stable Rust.

Thanks for the quick answer! I did not know I have to write list instead of QListModel into the macro. I'll try that.

@flanfly you don't have to name it list, actually, any valid Rust identifier is good. Here are the docs with an example

I got it to work, thanks.


Testable stacks

The attached have script problems, but are not corrupted and can be imported.

All zips below are part of the HyperCard 2.4 release
root folder: not including corrupted files
stacks-in-root-hc2.4-folder.zip
More Stacks: has script bugs:
More Stacks.zip
HyperCard Stacks folder: has script bugs, and include some corrupted stacks for inspection (renamed as such. See issue #100)
HyperCard Stacks.zip
Your Tour of HyperCard
I think these are a good start in tackling some of the scripting issues

Thanks, @ooper-zz !
Which stacks are corrupt (meaning you cannot actually import them, not that their scripts don't work correctly)? I thought I used all (most?) of these to validate the stack import feature, so I'd be surprised if some couldn't be imported.
Let me know what script errors you discover as you play with these. I fixed quite a few bugs that I found by importing existing stacks, but not all of them. The most glaring issue, I think, is the lack of support for the commands and functions that were provided by the baked-in XCMDs and XFCNs.


BUG: fix basename

This should be ready to go again :)

Thank goodness for 3-way merges!

Pull Request Test Coverage Report for Build 35

17 of 17 (100.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage increased (+4.6%) to 90.526%

Files with Coverage Reduction
New Missed Lines
%

q2_shogun/_shogun.py
1
81.25%

Totals

Change from base Build 19:
4.6%

Covered Lines:
76

Relevant Lines:
82

💛  - Coveralls


Introduce New Canceled Status for Cancelled Actions

Feature Description
When canceling an action, it will get the status Failed.
I would like to suggest that cancelled workflow runs will get their own "Canceled" status.
Screenshots
I canceled the workflow run:

It has the status failed.

Fixed by  Fix incomplete Actions status aggregations #32859


Add reserve compute resources kubelet flags

adds kubelet flags required to set values necessary to configure resources available for system daemons. Relevant reading:
Reserve Compute Resources for System Daemons
Configure Out Of Resource Handling

ci check this

@chadswen  Thanks for your reviwed.
Different operating systems are not exactly the same, and it is difficult to achieve uniformity.


PlugStatus showed error with wildcard tag

:PlugStatus was reporting an invalid tag when using e.g.Plug 'fatih/vim-go', { 'tag': '*' }

Thanks a lot!


whatsapp_automation

Pull Request Template
script name -
whatsapp_automation
Brief about script
This is used to extract the last text from WhatsApp and send a text using the same script.
This script can be used for main complex projects namely sentiment analysis, cyberbullying detection, etc using real-time data/content scraping.
I request the contributors to assign the issue to me.
Thanks...
Happy learning!
:)
Issue no.(must) -
Self Check(Tick After Making pull Request)

[ ] This issue was assigned to me.
[x] One Change in one Pull Request
[x] My file is in the proper folder (Name of the folder should be in lowercase with no space in between)
[x] I am following clean code and Documentation and my code is well linted with flake8.
[ ] I have added README.md and requirements.txt with my script

If the issue was not assigned to you Please don't make a PR. It will mark as invalid.

Okay cool, @banerjee-tuhina go ahead :)


Check version of baseserver at runtime

We developers want to know if there is a baseserver update we are missing.

implemented by #411


Make the default key bindings optional

I like using f/F for flit and t/T for Leap.  Can we make flit have optional keybindings very much like Leap does?
Right now my config is:
    use {
        'ggandor/flit.nvim',
        requires = { 'ggandor/leap.nvim' },
        config = function()
            require('flit').setup {
                keys = { f = 'f', F = 'F', t = 'ggt', T = 'ggT' },
            }

        end,
    }

I would like to just define:
  require('flit').setup {
                keys = { f = 'f', F = 'F'},
            }

Or just have the ability to call (flit-forward-to) like I do with Leap!

I would like to just define:

require('flit').setup { keys = { f = 'f', F = 'F'}, }

This already works, you're free to leave out those keys you don't want to remap.

Or just have the ability to call (flit-forward-to) like I do with Leap!

Long story short: it is because of the "clever-f" functionality - for that, the call should know what key has been mapped to it, that's why we need a wrapper function. (Lightspeed solved this in a hacky way, by scanning through all the user-defined mappings itself, so that it could still use <Plug> keys, but I considered that overengineering here.)

Are you sure that leaving keys out works? I tried it (before and after opening the ticket) and it results in:
table index is nil
Looking at your code, you do hardcode the [key.t] .... So I'm assuming you cannot leave them out of the setup. Can you confirm? Maybe I'm doing something wrong?

you do hardcode the [key.t] ...

Oh, indeed. I forgot about setup itself :)


Atom v1.48.0 (2020-06-10) Broke this Package.

With Atom v1.48.0 you can now only highlight single words with this package. I wanted to bring that to your attention so you can fix the problem. I am not educated enough in code to report to you why it is doing this, I just know I rolled back to 1.47.0 and it works again. Thank you for your time, and thank you for this Package. It is the #1 package I use in Atom. It helps me immensely.

Bump, just wanted to know someone is aware of this?

I believe you're interested in this Fork https://github.com/nathanpotolsky/highlight-selected @nathanpotolsky

Looks like the guy who did it is MIA. Is there any change this function
could be updated to work with new versions on Atom?
On June 12, 2020 4:10:47 AM Richard Race<EMAIL_ADDRESS>wrote:

I believe you're interested in this Fork
https://github.com/nathanpotolsky/highlight-selected @nathanpotolsky
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or unsubscribe.


Fix dates

Revert b262e57
Fix numberOfFullMonthsBetween and parse in DateUtils
In tests compare UTC dates to UTC dates

Codecov Report

Merging #29 into master will decrease coverage by 0.08%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #29      +/-   ##
==========================================
- Coverage   99.82%   99.74%   -0.09%
==========================================
  Files          45       45
  Lines        1162     1162
  Branches      144      144
==========================================
- Hits         1160     1159       -1
- Misses          0        1       +1
  Partials        2        2

Impacted Files
Coverage Δ

src/DateUtils.ts
95.91% <100%> (-2.05%)
:arrow_down:

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 2625205...0b9324e. Read the comment docs.


egl: failed to create dri2 screen

Background
I have encountered a problem just ran the early stages of the Roguelike tutorial, specifically the two initial map examples.

2.2. Walking A Map
2.3. A More Interesting Map

To be sure, I've downloaded the source from the github example:

amethyst / rustrogueliketutorial

Problem

Running both examples run  the mouse moves the 'player' around the screen OK
If the mouse is moved, we get this error as shown.

Error
 $  cargo run
    Compiling cfg-if v1.0.0
   Compiling smallvec v1.11.2
   Compiling bitflags v1.3.2
       :
   Compiling rogue-two v0.0.1 (/work/dev/Utils/Rogue-two/rogue-two)
    Finished dev [unoptimized + debuginfo] target(s) in 34.75s
     Running `target/debug/rogue-two`
libEGL warning: egl: failed to create dri2 screen
Initialized OpenGL with: 4.5 (Core Profile) Mesa 23.2.1-1ubuntu3.1, Shader Language Version: 4.50
Error sending request: Resource temporarily unavailable

Set-up
The tutorial version  was using earlier builds of the rltk and specs.  I reconfigured the toml file, ran cargo update to test with the newer versions (see below), same problem.
Cargo.toml
[dependencies]
##  rltk            = "0.8.0"
    rltk            = "0.8.7"
##  specs           = "0.17.0"
    specs           = "0.20.0"
    specs-derive    = "0.4.1"

LSB
The error seems to be with the Mesa 3D graphics using Wayland on Kubuntu 23.10.
 $ lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 23.10
Release:	23.10
Codename:	mantic

Desktop: KDE Plasma v: 5.27.4

API: OpenGL v: 4.6.0 NVIDIA 535.113.01 renderer: NVIDIA GeForce RTX
    2080/PCIe/SSE2 direct-render: Yes

Summary

For others, this only seems to happend if you touch the mouse.  So don't touch the mouse ;-)
I was wondering if there is some way to configure the RLTK to avoid the problem or to just use a more basic graphics layer
Also, does anyone know how to get logs or investigate this:
libEGL warning: egl: failed to create dri2 screen problem?

Suggestions welcome.

I ran across some relevant documentation:

EGL Drivers

It seems that the dri2 mentioned is a fallback if DRI3 can't be found (I think).  The driver is supposed to work for X11, Android and Wayland.
Is this something that can be configured ito improve usage by the RLTK ?

I ran across some relevant documentation:

EGL Drivers

It seems that the dri2 mentioned is a fallback if DRI3 can't be found (I think).  The driver is supposed to work for X11, Android and Wayland.
Is this something that can be configured ito improve usage by the RLTK ?
The root cause appears to lie within libEGL itself, see:

libEGL "failed to create dri2 screen" and coredump
For now just don't move a mouse ;-)

Update ...
The next stage of the tutorial: "2.4. Field of View" fails with said error: "libEGL warning: egl: failed to create dri2 screen".
I'm guessing progress on the tutorial is blocked until I find a workaround or perhpas a way to avoid using EGL temorarily.  Full transcript:
 $ cargo run
    Finished dev [unoptimized + debuginfo] target(s) in 0.08s
     Running `target/debug/rogue-two`
libEGL warning: egl: failed to create dri2 screen
Initialized OpenGL with: 4.5 (Core Profile) Mesa 23.2.1-1ubuntu3.1, Shader Language Version: 4.50
Error sending request: Resource temporarily unavailable

previously found problem on both Wayland and X.11

Keywords:
EGL, DIR2, Mesa, OpenGL, error, graphics, Ubuntu, NVIDIA

Recently I've found that the eglinfo -B core dump has become now, another example of the original problem:
   :
Surfaceless platform:
libEGL warning: egl: failed to create dri2 screen
EGL API version: 1.5
EGL vendor string: Mesa Project
EGL version string: 1.5
EGL client APIs: OpenGL OpenGL_ES
OpenGL core profile vendor: Mesa
   :

Also, it seems that this issue is a problem with NVIDIA graphics.
Personally, for a text-box demo example having the Rendering fail is still a FAIL; better if there were some  kind of cut-out or an opt-out config option/setting (imho).


Solution_v1.0

Unfortunately when I'm trying to rename headless service for statefulSet getting error of naming as I understand, could you please explain what I'm doing wrong?

Unfortunately when I'm trying to rename headless service for statefulSet getting error of naming as I understand, could you please explain what I'm doing wrong?

Зміни, які ти намагаєшся зробити у маніфесті типу StatefullSet заборонені. Можна лише повністю передеплоїти цей ресурс.


Improve nullability of Arcs code #paydown

Introduce testing/preconditions.ts to allow inline checking of null/undefined.
Introduce types in lots of local variables.

1614 -> 1453 == 16.1 points

sorry, it's a bit patch....

Nice job!


Add spec: Struct#keyword_init?

Issue
A part of https://github.com/ruby/spec/pull/947
Changes
I'm trying to cover the following part of the issue.

StructClass#keyword_init? is added [Feature #18008]

BTW, this tweet encouraged me to create this pull request.
Please tell me if anything is wrong with this pull request since I'm almost new to contribution to this project!

Thank you for the PR.
Unfortunately I already merged https://github.com/ruby/spec/pull/953 before seeing this one, I am sorry.
Maybe you can pick another spec? Feel free to mention it on https://github.com/ruby/spec/pull/947 to avoid conflicts.

FWIW I tried to process my emails in order but somehow I missed this one, sorry.

@eregon No worries!
I'll try another one and will mention my target in the issue. 💪


Same nuget package works in VS Enterprise but doesnt in VS Code via CakeScript.

dotnet.exe, nuget.exe (Visual Studio Code - Cake Script)
and
Visual Studio Enterprise 2017
NuGet version: <IP_ADDRESS>48
dotnet.exe: 2.1.302
VS version : 15.7.5
Cake version: 0.29.0
Detailed repro steps:
SHFB(Sandcastle Help File Builder): A tool to create documentation out of XML-Comments
Installed following nuget packages with the Visual Studio nuget-package-manager:
EWSoftware.SHFB --> version 2018.7.8
EWSoftware.SHFB.NetFramework --> version 4.7.2
-> worked properly and i had no problems to use it.
Now i used a cake script in Visual Studio Code to automate documentation creation.
i tried to add both packages like this:
#tool nuget:?package=EWSoftware.SHFB&version=2018.7.8
#tool nuget:?package=EWSoftware.SHFB.NETFramework&version=4.7.2
on command "./build" the first package installed without problems.
the second package didn't, i got this error: "E
rror: Failed to install tool 'EWSoftware.SHFB.NETFramework'"
I tried all possible Versions of the .NETFramework nuget package (4.6.0; 4.6.1; 4.6.2; 4.7.0; 4.7.2)
Same Error on every Version.
I also contacted the product owner but he couldn't help me.
And i already posted this issue in the nuget issue forum.
So my Question is, why is the same package working for VS and not working for cakescript in VS Code?
Here my cakescript etc. to reproduce my problem in VS Code:
cakescript.zip
TL;DR:
Installing Nugets:
EWSoftware.SHFB --> version 2018.7.8
EWSoftware.SHFB.NetFramework --> version 4.7.2
Works in VS Enterprise.
Doesnt work via Cakescript in VS Code.
Thanks in advance!
Greetings
Jay

@gep13 thanks for the fast answer. As far as i know the second package is to specifiy the platform on which the first one is running.
The Description for the nuget EWSoftware.SHFB.NetFramework is:
This package contains the .NETFramework/.NETCore/.NETStandard reflection information data set for use with the EWSoftware.SHFB package.
I now deleted the second package in visual studio and it seems to still work. Im a bit confused now, cause in the tutorial it was said to install both packages.
Well but that explains why there is no exe in the tool folder if installed and why Cake is throwing an Error.
Thank you

If you find that you "need" that package, then I would suggest using the NuGetInstall alias to install the required NuGet package:
https://cakebuild.net/api/Cake.Common.Tools.NuGet/NuGetAliases/343A93DB
For now, I am going to go ahead and close this issue.


Reduce logging

We have received complaints that our service is writing too much to the logs. Notably, can the service please stop logging :
using daemon endpoints from environment variable AWS_XRAY_DAEMON_ADDRESS
Emitter using address
which come from aws-xray-sdk-go
How is it possible to stop these?

Its log level is LogLevelInfo, so set the log level lower than LogLevelInfo.
xray.SetLogger(xraylog.NewDefaultLogger(os.Stderr, xraylog.LogLevelWarn))
// or xray.SetLogger(xraylog.NewDefaultLogger(os.Stderr, xraylog.LogLevelError))

Also, you can use logger section in README.MD (https://github.com/aws/aws-xray-sdk-go) to get more information about how X-RAY Go SDK implements logger.

If I understand this correctly, if we set the log level to Warn, then we will lose the Info logging in our service?
In order to log at Info in our service, but not log Info in aws-xray-sdk-go, do we have to add logic to a custom logger?

Hi @martin-flower,
I believe if you set log level to Warn then you will not lose Info logging in your service. I think it will emit x-ray specific logging information and not your service specific. I have assumed you are not using X-Ray's default logger implementation in your service.

Thanks for all the comments - I'll need to spend more time understanding custom logging, but meanwhile I'll set the level to Warn. This is the current code which I'm working with:
_ = xray.Configure(xray.Config{})
xray.SetLogger(utils.NewDefaultLogger(os.Stderr, xraylog.LogLevelWarn))

Looks great! I am closing this issue for now feel free to reopen this if you still see any issue.


MqttServer.StartAsync  Unable to actually start the listening port

Hello, I install MQTTnet under the ASP core6-API project, version number: <IP_ADDRESS>2
Use the code under the IHosted Service interface:
var optionsBuilder = new MqttServerOptionsBuilder().WithConnectionBacklog(100).WithDefaultEndpointPort(87941)
var mMqtt = new MqttFactory(new MyLogger()).CreateMqttServer(optionsBuilder.Build())
await mMqtt.StartAsync()
On the premise that there is no abnormal error and the startup is successful, the actual mMqtt. StartAsync () is not started and does not monitor the specified port
How do I start an MQTT server side within a defined IHosted Service?

I would recommend to build a background service following https://learn.microsoft.com/en-us/dotnet/core/extensions/windows-service and provide it within the startup:
service.AddSingleton<MyBackgroundService>();
service.AddHostedService(p => p.GetRequiredService<MyBackgroundService>());

I would recommend to build a background service following https://learn.microsoft.com/en-us/dotnet/core/extensions/windows-service and provide it within the startup:
service.AddSingleton<MyBackgroundService>();
service.AddHostedService(p => p.GetRequiredService<MyBackgroundService>());

Hello, test code output: StartAsync still does not open the port, IMqttNetLogger log has output: Started, and then there is no active content output.
`public class IotMqttServer : IHostedService
{
private MqttServer mMqtt;
public IotMqttServer()
{
}
public async Task StartAsync(CancellationToken cancellationToken)
{
    try
    {

        var optionsBuilder = new MqttServerOptionsBuilder().WithConnectionBacklog(100).WithDefaultEndpointPort(87941);
        var mMqtt = new MqttFactory(new MyLogger()).CreateMqttServer(optionsBuilder.Build());
        await mMqtt.StartAsync();
        System.Diagnostics.Debug.WriteLine("StartAsync");
    }
    catch (Exception)
    {

    }

}
public async Task StopAsync(CancellationToken cancellationToken)
{
    if (mMqtt != null)
    {
        if (mMqtt.IsStarted)
        {
            await mMqtt.StopAsync();
        }
    }
}}`

builder.Services.AddSingleton<IotMqttServer>(); builder.Services.AddHostedService(P=>P.GetRequiredService<IotMqttServer>());

@shouhujishu This is how I use it:
using MQTTnet;
using MQTTnet.Protocol;
using MQTTnet.Server;

/// <inheritdoc cref="BackgroundService"/>
public sealed class MqttBrokerService : BackgroundService
{
    /// <summary>
    /// The MQTT Server.
    /// </summary>
    public MqttServer MqttServer { get; internal set; }

    /// <summary>
    /// Initializes a new instance of the <see cref="MqttBrokerService"/> class.
    /// </summary>
    public MqttBrokerService()
    {
        var mqttFactory = new MqttFactory();
        var mqttServerOptions = mqttFactory
            .CreateServerOptionsBuilder()
            .WithDefaultEndpoint()
            .WithDefaultEndpointBoundIPAddress(IPAddress.Parse("<IP_ADDRESS>"))
            .WithDefaultEndpointPort(1883)
            .Build();
        this.MqttServer = mqttFactory.CreateMqttServer(mqttServerOptions);
    }

    /// <inheritdoc cref="BackgroundService"/>
    public override async Task StartAsync(CancellationToken cancellationToken)
    {
        this.logger.Information("Starting MQTT server on port {Port}", 1883);

        // Start the MQTT server
        await this.MqttServer.StartAsync();

        this.logger.Information("MQTT server has started");
        await base.StartAsync(cancellationToken);
    }

    /// <inheritdoc cref="BackgroundService"/>
    public override async Task StopAsync(CancellationToken cancellationToken)
    {
        this.logger.Information("Stopping MQTT server");
        await this.MqttServer.StopAsync();
        await base.StopAsync(cancellationToken);
        this.logger.Information("MQTT server is stopped");
    }

    /// <inheritdoc cref="BackgroundService"/>
    protected override async Task ExecuteAsync(CancellationToken stoppingToken)
    {
        while (!stoppingToken.IsCancellationRequested)
        {
            await Task.Delay(TimeSpan.FromSeconds(30), stoppingToken);
            this.logger.Information("MQTT server is running");
        }
    }
}

I guess, you were missing .WithDefaultEndpoint() and .WithDefaultEndpointBoundIPAddress(IPAddress.Parse("<IP_ADDRESS>")).

@shouhujishu This is how I use it:
using MQTTnet;
using MQTTnet.Protocol;
using MQTTnet.Server;

/// <inheritdoc cref="BackgroundService"/>
public sealed class MqttBrokerService : BackgroundService
{
    /// <summary>
    /// The MQTT Server.
    /// </summary>
    public MqttServer MqttServer { get; internal set; }

    /// <summary>
    /// Initializes a new instance of the <see cref="MqttBrokerService"/> class.
    /// </summary>
    public MqttBrokerService()
    {
        var mqttFactory = new MqttFactory();
        var mqttServerOptions = mqttFactory
            .CreateServerOptionsBuilder()
            .WithDefaultEndpoint()
            .WithDefaultEndpointBoundIPAddress(IPAddress.Parse("<IP_ADDRESS>"))
            .WithDefaultEndpointPort(1883)
            .Build();
        this.MqttServer = mqttFactory.CreateMqttServer(mqttServerOptions);
    }

    /// <inheritdoc cref="BackgroundService"/>
    public override async Task StartAsync(CancellationToken cancellationToken)
    {
        this.logger.Information("Starting MQTT server on port {Port}", 1883);

        // Start the MQTT server
        await this.MqttServer.StartAsync();

        this.logger.Information("MQTT server has started");
        await base.StartAsync(cancellationToken);
    }

    /// <inheritdoc cref="BackgroundService"/>
    public override async Task StopAsync(CancellationToken cancellationToken)
    {
        this.logger.Information("Stopping MQTT server");
        await this.MqttServer.StopAsync();
        await base.StopAsync(cancellationToken);
        this.logger.Information("MQTT server is stopped");
    }

    /// <inheritdoc cref="BackgroundService"/>
    protected override async Task ExecuteAsync(CancellationToken stoppingToken)
    {
        while (!stoppingToken.IsCancellationRequested)
        {
            await Task.Delay(TimeSpan.FromSeconds(30), stoppingToken);
            this.logger.Information("MQTT server is running");
        }
    }
}

I guess, you were missing and . Note: as IP address is needed if you want to have the broker accessible from outside the machine as well, else you can use as well. You can change the port of to any other port, of course..WithDefaultEndpoint()``.WithDefaultEndpointBoundIPAddress(IPAddress.Parse("<IP_ADDRESS>"))``<IP_ADDRESS>``<IP_ADDRESS>``1883
Thanks for the help, I added WithDefaultEndpoint () and it works fine

I guess, we can close this :)


Impossible hooking

Hooking init in after_setup_theme is not possible
https://github.com/devgeniem/wp-no-admin-ajax/blob/ebee2d29870e03dc7db0b584acbaa891c08c48b0/plugin.php#L41
as init is an earlier hook.

No!
https://codex.wordpress.org/Plugin_API/Action_Reference
Sorry for the noise.


[DNM] enlarge default region size from 96MB to 256MB

What is changed and how it works?

Issue Number: Close https://github.com/tikv/tikv/issues/16732

What's Changed:
Change the default region size from 96MB to 256MB to reduce TiKV meta memory usage and PD meta management pressure.

Related changes

[ ] PR to update pingcap/docs/pingcap/docs-cn:
[ ] Need to cherry-pick to the release branch

Check List
Tests

[x] Unit test
[x] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Release note

Change the default region size from 96MB to 256MB to reduce TiKV meta memory usage and PD meta management pressure.

I think it may be better to make it only applicable to new clusters. Because in some well-tuned environment, region-size increase could have negative impact such as more CPU usage by load balance (under same store limit config).

Hold it until 8.1 LTS version released.


Fix typo

Description
Fixed a typo in the readme.
General Checklist

[ ] Tests added for this feature/bug
if it was a bug, test must cover it.
[x] Conform by the style guides, by using formatter
[x] Documentation updated
[ ] (optional) Please add a label to your PR

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
3744

Relevant Lines:
4536

💛  - Coveralls

Merged your fix, @alex-zenml. Thanks again for the catch. We have some whylabs swag for contributors to whylogs if you're interested. Just share your email and I'll reach out!


Enhanced Visibility of Completed Items in Purchase Orders (PO)

What is the motivation for adding/enhancing this feature?
When a PO contains multiple items to be received, it's essential for users to have a clear understanding of which products have already been received or completed. The completed items will be clearly distinguishable from pending items, ensuring that users can quickly identify which products are no longer outstanding. This feature is crucial for users dealing with large and complex POs, as it significantly improves their ability to manage and track received items.
What are the acceptance criteria?

[ ] ...

Can you complete this feature request by yourself?

[ ] YES
[ ] NO

Additional information

Please refer the UI: https://www.figma.com/file/wK5FMs4z2zXQR9VfHV9ycE/Dhiraj's-Ionic-6-Material-UI-Kit-(Community)?type=design&node-id=28361%3A113187&mode=design&t=doH7dMQSqXDJaGZq-1


Object Explorer API differences

Inspecting through the Object Explorer gives me a different types and number of arguments then when hooking to the method.
Object Explorer:
snow.enemy.EnemyCharacterBase.afterCalcDamage_DamageSide(snow.hit.DamageFlowInfoBase, snow.DamageReceiver.HitInfo)
sdk.hook(sdk.find_type_definition("snow.enemy.EnemyCharacterBase"):get_method("afterCalcDamage_DamageSide"),
	function(args)
		log.debug("afterCalcDamage_DamageSide")
		local enemyCharacterBase = sdk.to_managed_object(args[2])
		local afterCalcInfo = sdk.to_managed_object(args[3])
		log.debug(afterCalcInfo:get_type_definition():get_full_name()) --[11796] snow.hit.EnemyCalcDamageInfo.AfterCalcInfo_DamageSide
	end,
	function(retval)
		return retval
	end)

This isn't an issue. You're just seeing inheritance in action.
Find snow.hit.EnemyCalcDamageInfo.AfterCalcInfo_DamageSide in the object explorer and you'll see that it inherits from:

snow.hit.AfterCalcInfo_DamageSideBase
and finally snow.hit.DamageFlowInfoBase, which is what your method takes.


Fixed Profiler's unit tests bugs.

Problem:
The TestProfiler class was using Java Thread methods to test the
Profiler's functionality. That was causing the tests to fail sometimes
since the JVM's thread priority could be low on some OS.
Fix:
Using PowerMockito to mock the System calls, the threads could be
removed. This makes the tests considerably faster, OS independent and
still guarantees the correct implementation of the Profiler class.
The changes on the Profiler's class was only to shorten the class's line
size by not assigning the return value to a variable returning it
straight out.

This LGTM.  @pdube, would you mind reviewing this for me?

Built and ran the unit tests. LGTM
[INFO] Apache CloudStack Developer Tools - Checkstyle Configuration  SUCCESS [  1.106 s]
[INFO] Apache CloudStack ................................. SUCCESS [  0.975 s]
[INFO] Apache CloudStack Maven Conventions Parent ........ SUCCESS [  1.918 s]
[INFO] Apache CloudStack Framework - Managed Context ..... SUCCESS [  1.485 s]
[INFO] Apache CloudStack Utils ........................... SUCCESS [ 11.555 s]
[INFO] Apache CloudStack Framework ....................... SUCCESS [  0.047 s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [  1.561 s]
[INFO] Apache CloudStack Framework - Configuration ....... SUCCESS [  1.210 s]
[INFO] Apache CloudStack API ............................. SUCCESS [ 12.870 s]
[INFO] Apache CloudStack Framework - REST ................ SUCCESS [  0.644 s]
[INFO] Apache CloudStack Framework - IPC ................. SUCCESS [  0.640 s]
[INFO] Apache CloudStack Cloud Engine .................... SUCCESS [  0.030 s]
[INFO] Apache CloudStack Cloud Engine API ................ SUCCESS [  3.529 s]
[INFO] Apache CloudStack Framework - Security ............ SUCCESS [  0.499 s]
[INFO] Apache CloudStack Core ............................ SUCCESS [  4.886 s]
[INFO] Apache CloudStack Agents .......................... SUCCESS [  1.546 s]
[INFO] Apache CloudStack Framework - Clustering .......... SUCCESS [  1.026 s]
[INFO] Apache CloudStack Framework - Event Notification .. SUCCESS [  0.332 s]
[INFO] Apache CloudStack Cloud Engine Schema Component ... SUCCESS [ 10.903 s]
[INFO] Apache CloudStack Framework - Jobs ................ SUCCESS [  2.049 s]
[INFO] Apache CloudStack Cloud Engine Internal Components API  SUCCESS [  0.551 s]
[INFO] Apache CloudStack Server .......................... SUCCESS [ 25.862 s]
[INFO] Apache CloudStack Framework - Quota ............... SUCCESS [  1.987 s]
[INFO] Apache CloudStack Usage Server .................... SUCCESS [  1.694 s]
[INFO] Apache CloudStack Cloud Engine Orchestration Component  SUCCESS [  0.648 s]
[INFO] Apache CloudStack Cloud Services .................. SUCCESS [  0.016 s]
[INFO] Apache CloudStack Secondary Storage ............... SUCCESS [  0.125 s]
[INFO] Apache CloudStack Secondary Storage Service ....... SUCCESS [  0.678 s]
[INFO] Apache CloudStack Engine Storage Component ........ SUCCESS [  5.129 s]
[INFO] Apache CloudStack Engine Storage Volume Component . SUCCESS [  0.588 s]
[INFO] Apache CloudStack Engine Storage Image Component .. SUCCESS [  0.472 s]
[INFO] Apache CloudStack Engine Storage Data Motion Component  SUCCESS [  0.592 s]
[INFO] Apache CloudStack Engine Storage Cache Component .. SUCCESS [  0.545 s]
[INFO] Apache CloudStack Engine Storage Snapshot Component  SUCCESS [  3.074 s]
[INFO] Apache CloudStack Cloud Engine API ................ SUCCESS [  0.567 s]
[INFO] Apache CloudStack Cloud Engine Service ............ SUCCESS [  0.596 s]
[INFO] Apache CloudStack Plugin POM ...................... SUCCESS [  0.266 s]
[INFO] Apache CloudStack Plugin - API Rate Limit ......... SUCCESS [  3.276 s]
[INFO] Apache CloudStack Plugin - Storage Volume default provider  SUCCESS [  0.485 s]
[INFO] Apache CloudStack Plugin - Storage Volume SolidFire Provider  SUCCESS [  0.574 s]
[INFO] Apache CloudStack Plugin - API SolidFire .......... SUCCESS [  0.483 s]
[INFO] Apache CloudStack Plugin - API Discovery .......... SUCCESS [  1.377 s]
[INFO] Apache CloudStack Plugin - ACL Static Role Based .. SUCCESS [  0.497 s]
[INFO] Apache CloudStack Plugin - Host Anti-Affinity Processor  SUCCESS [  0.428 s]
[INFO] Apache CloudStack Plugin - Explicit Dedication Processor  SUCCESS [  0.404 s]
[INFO] Apache CloudStack Plugin - User Concentrated Pod Deployment Planner  SUCCESS [  0.429 s]
[INFO] Apache CloudStack Plugin - User Dispersing Deployment Planner  SUCCESS [  0.424 s]
[INFO] Apache CloudStack Plugin - Implicit Dedication Planner  SUCCESS [  2.820 s]
[INFO] Apache CloudStack Plugin - Skip Heurestics Planner  SUCCESS [  0.635 s]
[INFO] Apache CloudStack Plugin - Host Allocator Random .. SUCCESS [  0.483 s]
[INFO] Apache CloudStack Plugin - Dedicated Resources .... SUCCESS [  2.748 s]
[INFO] Apache CloudStack Plugin - Hypervisor OracleVM .... SUCCESS [  0.560 s]
[INFO] Apache CloudStack Plugin - Open vSwitch ........... SUCCESS [  0.566 s]
[INFO] Apache CloudStack Plugin - Hypervisor XenServer ... SUCCESS [ 21.803 s]
[INFO] Apache CloudStack Plugin - Hypervisor KVM ......... SUCCESS [  3.487 s]
[INFO] Apache CloudStack Plugin - RabbitMQ Event Bus ..... SUCCESS [  0.526 s]
[INFO] Apache CloudStack Plugin - In Memory Event Bus .... SUCCESS [  1.354 s]
[INFO] Apache CloudStack Plugin - Kafka Event Bus ........ SUCCESS [  0.396 s]
[INFO] Apache CloudStack Plugin - Hypervisor Baremetal ... SUCCESS [  0.454 s]
[INFO] Apache CloudStack Plugin - Hypervisor UCS ......... SUCCESS [  0.444 s]
[INFO] Apache CloudStack Plugin - Hypervisor Hyper-V ..... SUCCESS [  0.792 s]
[INFO] Apache CloudStack Plugin - Hypervisor OracleVM3 ... SUCCESS [ 17.310 s]
[INFO] Apache CloudStack Plugin - Network Elastic Load Balancer  SUCCESS [  1.753 s]
[INFO] Apache CloudStack Plugin - Network Internal Load Balancer  SUCCESS [  4.308 s]
[INFO] Apache CloudStack Framework - Spring Life Cycle ... SUCCESS [  0.280 s]
[INFO] Apache CloudStack Plugin - Network Juniper Contrail  SUCCESS [  5.478 s]
[INFO] Apache CloudStack Plugin - Palo Alto .............. SUCCESS [  2.308 s]
[INFO] Apache CloudStack Plugin - Network Netscaler ...... SUCCESS [  0.555 s]
[INFO] Apache CloudStack Plugin - Network Nicira NVP ..... SUCCESS [  5.086 s]
[INFO] Apache CloudStack Plugin - BigSwitch Virtual Network Segment  SUCCESS [  2.470 s]
[INFO] Apache CloudStack Plugin - Network Brocade VCS .... SUCCESS [  2.579 s]
[INFO] Apache CloudStack Plugin - Midokura Midonet ....... SUCCESS [  1.744 s]
[INFO] Apache CloudStack Plugin - Stratosphere SSP ....... SUCCESS [  2.396 s]
[INFO] Apache CloudStack Plugin - Network Opendaylight ... SUCCESS [  1.376 s]
[INFO] Apache CloudStack Plugin - Storage Allocator Random  SUCCESS [  0.473 s]
[INFO] Apache CloudStack Plugin - User Authenticator LDAP  SUCCESS [  1.030 s]
[INFO] Apache CloudStack Plugin - User Authenticator MD5 . SUCCESS [  1.579 s]
[INFO] Apache CloudStack Plugin - User Authenticator PBKDF2-SHA-256  SUCCESS [  3.007 s]
[INFO] Apache CloudStack Plugin - User Authenticator Plain Text  SUCCESS [  0.546 s]
[INFO] Apache CloudStack Plugin - User Authenticator SAML2  SUCCESS [ 18.772 s]
[INFO] Apache CloudStack Plugin - User Authenticator SHA256 Salted  SUCCESS [  1.376 s]
[INFO] Apache CloudStack Plugin - Dns Notifier Example ... SUCCESS [  0.438 s]
[INFO] Apache CloudStack Plugin - Storage Image S3 provider  SUCCESS [  0.447 s]
[INFO] Apache CloudStack Plugin - Storage Image Swift provider  SUCCESS [  0.451 s]
[INFO] Apache CloudStack Plugin - Storage Image default provider  SUCCESS [  0.440 s]
[INFO] Apache CloudStack Plugin - Storage Image sample provider  SUCCESS [  0.488 s]
[INFO] Apache CloudStack Plugin - Storage Volume Nexenta Provider  SUCCESS [  0.570 s]
[INFO] Apache CloudStack Plugin - Storage Volume CloudByte Provider  SUCCESS [  0.536 s]
[INFO] Apache CloudStack Plugin - Storage Volume sample provider  SUCCESS [  0.512 s]
[INFO] Apache CloudStack Plugin - SNMP Alerts ............ SUCCESS [  1.302 s]
[INFO] Apache CloudStack Plugin - Syslog Alerts .......... SUCCESS [  0.969 s]
[INFO] Apache CloudStack Plugin - Network VXLAN .......... SUCCESS [  1.547 s]
[INFO] Apache CloudStack Plugin - GloboDNS ............... SUCCESS [  3.733 s]
[INFO] Apache CloudStack Plugin - Quota Service .......... SUCCESS [  2.127 s]
[INFO] Apache CloudStack Framework - Spring Module ....... SUCCESS [  2.605 s]
[INFO] Apache CloudStack Secondary Storage Controller .... SUCCESS [  0.376 s]
[INFO] Apache CloudStack Client UI ....................... SUCCESS [  1.352 s]
[INFO] Apache CloudStack Console Proxy - RDP Client ...... SUCCESS [  5.639 s]
[INFO] Apache CloudStack Console Proxy ................... SUCCESS [  0.122 s]
[INFO] Apache CloudStack Console Proxy - Server .......... SUCCESS [  0.729 s]
[INFO] Apache CloudStack Framework - QuickCloud .......... SUCCESS [  0.112 s]
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 04:08 min
[INFO] Finished at: 2016-03-22T11:04:28-05:00
[INFO] Final Memory: 94M/763M
[INFO] ------------------------------------------------------------------------

@swill this looks ready, no?

@pdube we need to run CI against it first, but otherwise, yes it is ready...

I would say it is missing an LGTM.
But, it is very good the code.

@rafaelweingartner @swill The only thing changed was the unit tests, which I ran. And the only code change he did was to remove useless variables. @swill also gave his LGTM higher

I did nto noticed that @swill  LGTM.
So, it is OK.

@pdube sorry, my brain is into something else right now and I didn't review the scope of the code.  Yes, your tests are valid tests for this change.  So yes, I will add this to my queue to be merged.  Thx guys...

Just trying to help get things merged :)

I appreciate it.  :+1:

@swill thanks, I was expecting for this one a long time


PullPush - replacement service for PushShift data

Hello!
I created a replacement service for PushShift functionality that's now restricted. See https://pullpush.io/ for details. Overall it will aim to be compatible to Dec-2022 version of PushShift API.
In case you feel burned out by the whole 3rd party tool affair; I would like to ask for permission to fork and rebrand your project :)
PullPush Actual

Great to hear! I've been in the Discord server since it was first announced. I've been trying out integrating with other sources but this is the only one I've seen that actually seems to be attempting to host all subreddits.
I've created a branch for it here - https://github.com/DenverCoder1/unedit-for-reddit/tree/pullpush-source
It doesn't seem like the API is fully working yet.
I tested with a random comment here. It's coming back with a CORS error when fetching from Reddit's domain and when opening the actual URL itself, it just shows some comments from r/Jokes instead of the comment with the given ID.
Access to fetch at 'https://api.pullpush.io/reddit/search/comment/?ids=t1_himv9l3&fields=body,author,id,link_id,created_utc,permalink'
from origin 'https://www.reddit.com' has been blocked by CORS policy:
Response to preflight request doesn't pass access control check:
No 'Access-Control-Allow-Origin' header is present on the requested resource.
If an opaque response serves your needs, set the request's mode to 'no-cors'
to fetch the resource with CORS disabled.

ids is not implemented yet. This is because if is a completely different query structure to the rest of the bunch, I will actually do it today now what I have an index that will (hopefully, testing on 5% of dataset doesn't always extrapolate to the whole) solve the performance issues for large queries

So implementation of ?ids= is done. Your particular example t1_himv9l3  (subreddit: pushshift) will need to wait as for privacy reasons we are testing in /r/jokes only (this is to give people a chance to submit PII removals, while having a long-standing and keyword rich subreddit to test on).
Limit is implemented on the back-end as subreddit=jokes that gets added to your query.
Examples:
https://api.pullpush.io/reddit/search/submission/?q=microsoft  [ Find jokes about Microsoft ]
https://api.pullpush.io/reddit/search/submission/?ids=t3_ttk4ho,u4qa7i,t3_v3jr5m [ Pull up 3 different submission ids ]


Contract deployment action

It should be possible to create smart contracts from within the extension.

[ ] Create interface for deploying a smart contract from raw bytecode
[ ] Create toggle in settings for making this feature visible

1f44759 added the ability to deploy contracts using eth_sendTransaction from the injected provider. It may be easier to leave the implementation of an interface to dapps


CSS selector issue

Issue
Changedetection returns blank content for a valid CSS selector. I've copied the css selector from developer tools, but changedetection is not able to follow it through to the end. I'm not sure if it's something specific with tables or something else. I've been able to reproduce this issue on a table in Yahoo.com as well.
Can someone reproduce or troubleshoot?
Example
Link: https://www.ettvcentral.com/user/smcgill1969
CSS selector: #main > div > div.right_column > div:nth-child(3) > div.myFrame-content > div > table > tbody > tr:nth-child(1) > td:nth-child(2) > a > b
Description:
Changedetection reads content up until the table selector. From the tbody selector onwards it reads as blank.

@guipace the problem is that the HTML of the site does not include a tbody, the tbody is a virtual element supplied by the browser because the table is not constructed properly
look here, there is no tbody in the source

This is related to #127

@dgtlmoon You taught me something today. Thank you for the explanation, that makes sense. What is the tool you're using to view the original html and do you know how I could copy the CSS selector from that?

@dgtlmoon Still having issues with this. Do you know how I could obtain the correct CSS selector to use in changedetection from the source HTML?

no sorry


Quality and coverage of unit tests

Unit tests created and conducted successfully, all test are checked and approved by team.


Added entry for Solr for newbies

What does this PR do?
Adds entry for book "Solr for newbies"
For resources
Book "Solr for newbies"
Description
A book on how to get started with Apache Solr.
Why is this valuable (or not)?
I thought the content is relevant to programmers. There are a couple of entries already for ElasticSearch but I didn't see anything for Solr.
How do we know it's really free?
I wrote it and licensed under the Creative Commons Attribution 4.0 International License.
For book lists, is it a book? For course lists, is it a course? etc.
It's the materials for a workshop, in book format.

This is a great addition!
We've been trying to avoid single-item categories. Are there no other free books on Solr? Would this be better under Java? Should we have a category for things like Solr and Elasticsearch? (on the list of things to do is moving MySQL and Postgres into an SQL category.)

This is a great addition!
We've been trying to avoid single-item categories. Are there no other free books on Solr? Would this be better under Java? Should we have a category for things like Solr and Elasticsearch? (on the list of things to do is moving MySQL and Postgres into an SQL category.)

I think grouping Solr and ElasticSearch books into a single category ("Search Tools/DBs/Engines"?) would be a good idea. I didn't feel like I had enough authority/knowledge on how the organization was setup to make that kind of change.

I think grouping Solr and ElasticSearch books into a single category ("Search Tools/DBs/Engines"?) would be a good idea. I didn't feel like I had enough authority/knowledge on how the organization was setup to make that kind of change.

Let's create "Search" under "Language Agnostic". We're looking at ways to improve usability of the too-big books page and I think we'll rename "Language Agnostic" to something like "Topics" or "Subjects". Appreciate your help on this.

Let's create "Search" under "Language Agnostic". We're looking at ways to improve usability of the too-big books page and I think we'll rename "Language Agnostic" to something like "Topics" or "Subjects". Appreciate your help on this.

#5249

#5249


Add nullcheck to breadcrumb metadata

Adds a null check when reading breadcrumb metadata values.
Also updates build.gradle file to allow testing of local changes to the NDK library by depending on an AAR.

Can we add a test for this? @fractalwrench

@snmaynard we don't currently have a test framework setup for this project so that would be the first thing to add. GoogleTest appears to be one solution, although I'm not too sure how well that will interact with the JNI components which this PR touches. If you know of any other suitable test frameworks then it would be good to discuss that with you. I've created a ticket in JIRA so that we can estimate and prioritise this work.

I dont know of any, even maze would be fine with me


fix multiplicity for pure platform expression node

What does this PR do / why is it needed ?
This PR fixes a bug to set the multiplicity for pure platform expression nodes and in turn correctly build out execution plans

Which issue(s) this PR fixes:
NA

Other notes for reviewers:
NA
Does this PR introduce a user-facing change?
No

Thanks for fixing.
Can we also fix it here at same time - https://github.com/finos/legend-engine/blob/master/legend-engine-pure-code-compiled-core/src/main/resources/core/store/platform/executionPlan/executionPlan_generation.pure


Type instantiation is excessively deep and possibly infinite.

I get a typescript error after upgrading to the last version:
Type instantiation is excessively deep and possibly infinite.
for the following line:
app.use('*', sentry())
Not sure if this belongs to the repo or to the sentry one.

Is this the error you are talking about?

It's caused by @honojs/sentry not following the update of Hono core.

Nope, this is how it looks like:

How about rebooting the VSCode or re-installing the packages?

I've updated vscode now, restarted and reinstalled all packages, no change. Could it be related to my tsconfig?
I'm using the wrangler defaults:
"target": "es2021",
"lib": [
  "es2021"
],
"module": "es2022"

Could it be related to my tsconfig?

Hmm. It seems to be not related. Here is a minimal project using @honojs/sentry. It does not show the errors:
https://gist.github.com/yusukebe/cf910ca1ce90ca98f9db05b1bff0f2ac
Can you try it?
Or, perhaps, if you remove the node_modules and re-install modules, it may work.

Removing node_modules didn't work.
I'll try it with the minimal project.

https://github.com/honojs/hono/issues/675#issuecomment-1325517460 fixed it.


installed_check script fails in mulit-root workspace with virtual environment

Type: Bug

Behaviour
Expected vs. Actual
I expect no exceptions to be raised when provided wth a valid python interpreter path and requirements file path.
Instead, I see an error (see logs below)
Steps to reproduce:

Create a multi-root workspace
In a folder, set the python.defaultInterpreterPath to .venv/bin/python
Create the venv with python -m venv .venv (with that folder as the working directory)
Create a requirements.txt in the folder
Open your Python output and you should show the error below

Repro multi-root workspace:
workspace_root.zip

Diagnostic data

Python version (& distribution if applicable, e.g. Anaconda): 3.9.11
Type of virtual environment used (e.g. conda, venv, virtualenv, etc.): Pyenv
Value of the python.languageServer setting: Pylance

Output for Python in the Output panel (View→Output, change the drop-down the upper-right of the Output panel to Python)

2023-11-10 17:11:22.654 [info] Running installed packages checker:  .venv/bin/python /Users/username/.vscode/extensions/ms-python.python-2023.20.0/pythonFiles/installed_check.py /tmp/workspace_root/one/requirements.txt
2023-11-10 17:11:22.655 [info] > .venv/bin/python ~/.vscode/extensions/ms-python.python-2023.20.0/pythonFiles/installed_check.py /tmp/workspace_root/one/requirements.txt
2023-11-10 17:11:22.656 [error] Error while getting installed packages check result:
 Error: spawn .venv/bin/python ENOENT
    at Process.ChildProcess._handle.onexit (node:internal/child_process:283:19)
    at onErrorNT (node:internal/child_process:476:16)
    at processTicksAndRejections (node:internal/process/task_queues:82:21) {
  errno: -2,
  code: 'ENOENT',
  syscall: 'spawn .venv/bin/python',
  path: '.venv/bin/python',
  spawnargs: [
    '/Users/username/.vscode/extensions/ms-python.python-2023.20.0/pythonFiles/installed_check.py',
    '/tmp/workspace_root/one/requirements.txt'
  ]
}

User Settings

Multiroot scenario, following user settings may not apply:

envFile: "<placeholder>"

languageServer: "Pylance"

testing
• pytestArgs: "<placeholder>"

Extension version: 2023.20.0
VS Code version: Code 1.84.2 (Universal) (1a5daa3a0231a0fbba4f14db7ec463cf99d7768e, 2023-11-09T10:52:33.687Z)
OS version: Darwin arm64 23.0.0
Modes:

System Info

Item
Value

CPUs
Apple M1 Max (10 x 24)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: enabled_ondirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
6, 5, 5

Memory (System)
32.00GB (0.05GB free)

Process Argv
--crash-reporter-id 9c8b74e5-efe2-4756-b68a-91b9fde0fa03

Screen Reader
no

VM
0%

A/B Experiments
vsliv368:30146709
vsreu685:30147344
python383:30185418
vspor879:30202332
vspor708:30202333
vspor363:30204092
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593cf:30376535
pythonvs932:30410667
py29gd2263cf:30880073
vsclangdf:30486550
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
2e4cg342:30602488
f6dab269:30613381
a9j8j154:30646983
showlangstatbar:30737416
pythonfmttext:30731395
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
welcomedialog:30882109
pythonnosmt12:30797651
pythonidxpt:30866567
pythonnoceb:30805159
synctok:30869157
dsvsc013:30795093
dsvsc014:30804076
dsvsc015:30845448
pythontestfixtcf:30871695
pythonregdiag2:30871582
pyreplss2:30886141
pythonmypyd1:30879173
pythoncet0:30885854
h48ei257:30885898
pythontbext0:30879054
dsvsc016cf:30886111
dsvsc017cf:30886113
dsvsc018:30886114
aa_t_chat:30882232

@karrtikr it looks like we are getting the wrong python path here?

It does not look like the path to interpreter is valid. Either use
'./.venv' or use ${workspaceFolder}/.venv notation to configure relative interpreter paths, the latter is preferred.

Thanks for your help with this!
In my experience, .venv/bin/python  has been valid for test discovery, test execution, test debugging, run file, debug file, pylint, and intellisense. I believe this is because they all assume a cdw of the “nearest” current workspace folder.
Does installed_check.py follow a different standard or have different requirements?
I’ve struggled a bit to understand what this script does and what it’s requirements/expectations are.
Would it be reasonable to have installed_check.py use the same contextual details as the other extension tools to resolve a relative interpreter path? I worry about different pieces of the extension having different requirements for that kind of thing.

Unfortunately due to technical reasons and simplicity of implementation, that is not possible for now, especially when there're reliable ways to correctly specify relative paths. The thing which makes it difficult it's up to the each tool how that is interpreted, which can be unpredictable (imagine a tool using Python extension which we do not control).
If there's an issue with existing functionality do let us know and we'll happy to have a look again, closing for now.

Thanks for your message @karrtikr. I have a different interpretation of this issue, the configuration options of the extension, and the documented expectations for defaultInterpreterPath. Respectfully, I ask you to reconsder.

The thing which makes it difficult it's up to the each tool how that is interpreted, which can be unpredictable (imagine a tool using Python extension which we do not control).

I don't see "multiple tools" at play here. I believe that getInstalledPackagesDiagnostics is using concepts and utilities that are purely internal to this extension and/or part of the public extensions API.
As a matter of fact, the vscode documentation (the extension's documentation?) provides an example (emphasis is mine) that's identical to the one described in this issue (the documentation does say "full path", but it's clear that does not mean "absolute path" in the context of the provided example):

You can then either enter the full path of the Python interpreter directly in the text box (for example, ".venv/Scripts/python.exe") or [...]
--https://code.visualstudio.com/docs/python/environments#_manually-specify-an-interpreter

I believe I was actually inspired by this example when I configured my workspace and my setup has been generally well supported by the extension (other than in the case of getInstalledPackagesDiagnostics).

Unfortunately due to technical reasons and simplicity of implementation

I'm sorry, but it's not clear to me what "technical reasons" would prevent two parts of the same extension (vscode-python) from being able to share common interpreter path resolution... I also don't see much complexity in the requested fix.
Other parts of the same codebase seem quite capable of doing resolving the interpreter path relative to the workspace folder:
https://github.com/microsoft/vscode-python/blob/a1fac8125f4841a14c8a8aa599c0f360873b17c3/src/client/common/configSettings.ts#L242
https://github.com/microsoft/vscode-python/blob/a1fac8125f4841a14c8a8aa599c0f360873b17c3/src/client/common/configSettings.ts#L471
https://github.com/microsoft/vscode-python/blob/a1fac8125f4841a14c8a8aa599c0f360873b17c3/src/client/interpreter/display/index.ts#L150
It seems to me that installedCheckScript could easily be updated to be consistent with the rest of the extension:
https://github.com/microsoft/vscode-python/blob/a1fac8125f4841a14c8a8aa599c0f360873b17c3/src/client/pythonEnvironments/creation/common/installCheckUtils.ts#L62

My general concern is that as a user of vscode-python, I expect the extensions settings have fixed, consistent requirements throughout the extension - they should mean and express one (and only one) thing. I can certainly see why a fix would not be a high priority, given the availability of a workaround. However, I see no evidence that this is "not possible now."
It's clear that "python.defaultInterpreterPath": ".venv/bin/python" is resolved relative to the workspace folder by most areas of the extension, so I propose that aligning getInstalledPackagesDiagnostics with the majority of the rest of the extension is the best and easiest path to a consistent behavior.
Thanks anyway for your consideration. I hope you may reconsider a fix here to improve the consistency of handling defaultInterpreterPath throughout the extension.

Reopening as we should correct the docs if we don't support this scenario. Is there a reason why the other notations do not work for you?

Reopening as we should correct the docs if we don't support this scenario.

Thanks, @karrtikr! However, I think it would be more accurate to say that the extension does support this scenario in every area I've found exception this one. That's why I suspect fixing getInstalledPackagesDiagnostics is both easiest and probably most accurately reflects the intent of the extension as a whole.

Is there a reason why the other notations do not work for you?

I honestly wasn't worried about them when you first made the alternative suggestions, however I've tested them and they resulted in the same failures as my original syntax:

./.venv/bin/python also does not work at the moment.
Additionally, though not surprisingly, it seems that ${workspaceFolder}/.venv/bin/python also does not work if the folder's path (in the.code-workspace) is defined as ..
Finally, and more surprisingly, ${workspaceFolder}/.venv/bin/python also does not work when the folder path (in the .code-workspace file) is defined as one.

NOTE: multiroot workspaces explicitly call out the value of and their support for using relative paths so that the same .code-workspace file may be shared between developers.
All of these results continue to suggest that in order to support multiroot workspaces like the rest of the extension, getInstalledPackagesDiagnostics will need to perform similar path resolution to the rest of the extension.

Thanks for getting back, let's look at each concern one at a time:
For multiroot workspaces, try using notation mentioned here https://github.com/microsoft/vscode-python/issues/18650 (for eg. ${workspaceFolder:python-a}). Using relative paths or just ${workspaceFolder} leads to ambiguity as described here: https://github.com/microsoft/vscode-python/issues/18650#issuecomment-1136268585.

NOTE: multiroot workspaces explicitly call out the value of and their support for using relative paths so that the same .code-workspace file may be shared between developers.

"Relative paths for folder section" to be exact, settings are not resolved that way.

I think it would be more accurate to say that the extension does support this scenario in every area I've found exception this one.

As I noted earlier, there could be more scenarios easily, we cannot force every tool to use workspace folder as their cwd, hence it's better to be explicit using a syntax such as ${workspaceFolder} instead when specifying interpreters.

If ${workspaceFolder}/.venv/bin/python does not work for you in single workspace folder scenario (when not using .code-workspace file), do let us know and we can dig further into it.

I think you're referring to the different sub-components of the vscode-python package

Not necessarily, it can be different tool extensions.

I don't see how that will work for the folder defined as { "path": "." }. I'm getting an error with that:

Should be solved with: https://github.com/microsoft/vscode-python/issues/22452.
Reading settings using ${workspaceFolder} can technically be supported as you pointed out, but it's a general issue for all extensions whose support should ideally come from VS Code. At this point we can work on fixing support for ${workspaceFolder:name} as that is already required to be supported for other scenarios.

I've opened https://github.com/microsoft/vscode-python/issues/22472 on your behalf to support resolving ${workspaceFolder} correctly where possible, feel free to add/clarify in comments if anything else is required.

I think you're referring to the different sub-components of the vscode-python package

Not necessarily, it can be different tool extensions.

I see what you mean. I've definitely been focused on intra-extension interpreter path resolution consistency.

At this point we can work on fixing support for ${workspaceFolder:name}

That will be a helpful fix to have!

I've opened #22472 on your behalf to support resolving ${workspaceFolder} correctly where possible, feel free to add/clarify in comments if anything else is required.

Thank you very much, @karrtikr! I was feeling a little unsure of where to go next with my concern since the only solution path I feel confident enough about was within vscode-python. #22472 definitely summarizes my concerns well and if there's a common solution that will allow all parts of the vscode-python extension (including getInstalledPackagesDiagnostics ) to benefit from, that's wonderful!


Pin coverage to < 7 when running coveragepy-lcov

coverage 7.0.0 removed FnmatchMatcher, which coverage-lcov currently depends on.
https://github.com/chaychoong/coveragepy-lcov/issues/5
We can unpin this once coverage-lcov has been fixed.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.0%) to 84.769%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
1525

Relevant Lines:
1799

💛  - Coveralls


Allow blankConnection name

We're currently hard-coding a "Blank Connection Name" string - we should expose this as an optional prop.

https://github.com/iTwin/itwinjs-core/pull/6003


Unmanage unnecessary google dependencies

https://github.com/apache/camel-quarkus/pull/2688 imports the new io.quarkiverse.googlecloudservices:quarkus-google-cloud-services-bom which most likely manages many of the google cloud dependencies we now have in our bom.

A good occasion to do this could be when upgrading Quarkiverse Google Services BOM to a new version based on Quarkus 2.0.0.CR3+.

A fix is underway, but it is not perfect yet - see the followup https://github.com/apache/camel-quarkus/issues/2816


⚠ Samsung has degraded performance

In e3bdd6e, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2282 ms

Resolved: Samsung performance has improved in 57f148b.


entrypoint.sh can only use a single etcd host

With the current structure of entrypoint.sh only a single etcd host can be added.  Porentially this could be overridden by supplying a string of hosts as ETCD_HOST but in this case it will also require setting ETCD_PORT="" to avoid the automatic assignment of that variable.

In internal/flagconfig


enable GitHub Actions CI

https://github.com/features/actions

Goodbye Travis :-) We used to have 4 years in common.


Faculty View in CELTS

If the user is a faculty, the landing page they get is the event list, and they do not have access to their course proposal management page in the sidebar.  We should change this to make the course proposal management page the page they see when they log in, and that is the only page they should have access to in the sidebar.

PR 272 solved this issue and this has been merged. Closing issue.


Improve: Logging system

Currently, the logging system logs every command that is sent to the bot using the prefix. The logging system does not log any slash commands used. Log the slash commands as shown below:

@z404 I can work on this issue. Please assign it to me.

Assigned!

Hi @z404, added log for slash command, but not pushed the changes yet.

For the commands with bot prefix, should we add some prefix in the log like [BotPrefix] ?

Sure! It may come in handy later


Multiline Secret does not work

Component
server, cli
Describe the bug
I created a secret using:
woodpecker secret add -repository me/testrepo -name SSH_ED25519 -value @/root/privatekey
according to the secrets docs
That worked fine.
Yet connecting with this key does not.
woodpecker.yml:
pipeline:
  deploy-app:
    image: alpine:latest
    privileged: true
    commands:
      - apk add --update openvpn openssh-client
      - echo $SSH_ED25519 > privatekey
      - ssh -i privatekey -o StrictHostKeyChecking=no root@myserver "echo hello"

so i added a sleep 600 to the woodpecker.yml, ran the pipeline again and executed the container using:
docker exec -it 0_3056348731034984128_step_0 sh
inside the container i ran env which showed everything just fine with a multiline:

but running echo $SSH_ED25519 > privatekey prints it as a single line.
System Info
{"source":"https://github.com/woodpecker-ci/woodpecker","version":"0.15.6"}

Additional context
No response
Validations

[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A join our Discord Chat Server or the Matrix room.

As alpine is using busybox, print behaves differently.
Using prinft helped here. But one needs to add quotes for the multiline string:
https://stackoverflow.com/a/32371729
A pipeline to connect to OpenVPN and run a command on that server as a deployment can be seen here:
pipeline:
  deploy-app:
    image: alpine:latest
    privileged: true
    commands:
      - apk add --update openvpn openssh-client
      - printf "$SSH_ED25519" > privatekey
      - printf "$CLIENT_OVPN" > config.ovpn
      - chmod og= privatekey
      - chmod og= config.ovpn
      - echo $VPN_USER > /etc/openvpn/cred.txt
      - echo $VPN_PWD >> /etc/openvpn/cred.txt
      - openvpn --config config.ovpn --auth-user-pass /etc/openvpn/cred.txt --daemon
      - sleep 20s

      - ssh -p $SERVER_PORT -i privatekey -o StrictHostKeyChecking=no $SERVER_USER@$SERVER_IP "cd deployment && docker-compose pull && docker-compose down && docker-compose up -d && docker system prune -f"
    secrets: [ ssh_ed25519, client_ovpn, vpn_user, vpn_pwd, server_user, server_ip, server_port ]

Maybe this can be added to the docs - would probably help a lot.


VMD plugin does not install

I have followed the steps for installing the VMD plugin, but its tab does not appear under Extensions-Modeling.

Hello Muhammad,
Thanks for your interest in our program. By copying I am requesting Dr. Haoqi Wang, who is the developer of the algorithm to respond when he can find some time.
We will try to address this issue in the steps.
Best Regards, San
Sandun (San) Fernando, Ph.D., P.E.
Professor and Associate Department Head for Graduate Programs and Research
Biological and Agricultural Engineering Department
303C Scoates Hall
2117 TAMU
Texas A&M University
College Station, TX  77843-2117
Tel:<PHONE_NUMBER> ;  fax<PHONE_NUMBER>
Email<EMAIL_ADDRESS>From: Muhammad I. Ismail @.>
Sent: Monday, October 24, 2022 8:51 AM
To: sfernando-BAEN/ELIXIR-A @.>
Cc: Subscribed @.***>
Subject: [sfernando-BAEN/ELIXIR-A] VMD plugin does not install (Issue #1)
I have followed the steps for installing the VMD plugin, but its tab does not appear under Extensions-Modeling. - Reply to this email directly, view it on GitHub, or unsubscribe. You are receiving this because you are subscribed to this
ZjQcmQRYFpfptBannerStart
This Message Is From an External Sender
This message came from outside your organization.
ZjQcmQRYFpfptBannerEnd
I have followed the steps for installing the VMD plugin, but its tab does not appear under Extensions-Modeling.

Reply to this email directly, view it on GitHubhttps://urldefense.com/v3/__https:/github.com/sfernando-BAEN/ELIXIR-A/issues/1__;!!KwNVnqRv!GBEiVthmwL4FJzWY215bXrPvSpFJICMT5je6QYDJE-L1YmAUOUD9T8P-p1IFQJiFXrDwRvT_Yufou3zrq8fRTSmmdQ$, or unsubscribehttps://urldefense.com/v3/__https:/github.com/notifications/unsubscribe-auth/AQEBDAFBAH2YG24PZ53ANOLWE2H4NANCNFSM6AAAAAARM77D3M__;!!KwNVnqRv!GBEiVthmwL4FJzWY215bXrPvSpFJICMT5je6QYDJE-L1YmAUOUD9T8P-p1IFQJiFXrDwRvT_Yufou3zrq8cuuQtmqA$.
You are receiving this because you are subscribed to this thread.Message ID<EMAIL_ADDRESS>

Hi Muhammad,
Please try again with the following steps.

Download the EXLIXIR-A package from the link below.
https://github.com/sfernando-BAEN/ELIXIR-A/releases/tag/v2.0

unzip the file and copy the elixir2.0 folder to VMD TCL plugins directory $VMD/plugins/noarch/tcl/
if your vmd folder is /user/home/VMD, then TCL plugins directory will be /user/home/VMD/plugins/noarch/tcl/

Add the command in file /scripts/vmd/loadplugin.tcl.4.
if your vmd folder is /user/home/VMD, then file path will be /user/home/VMD/scripts/vmd/loadplugin.tcl

Under  line
### Modeling menu

Add
vmd_install_extension elixir elixir_tk  "Modeling/ELIXIR-A"

Open VMD.

If this doesn't work for you, please send us the screenshot of your Command Prompt (Windows) or Terminal (Mac/Linux).
Thanks,
Haoqi


Run PR Build based on merge commit

Sometimes it can be an issue that the PR Build is not running with the latest changes from the base branch. This build https://github.com/microsoft/ALAppExtensions/actions/runs/4500507606 is complaining about breaking changes even though, the PR doesn't touch the SignedXml codeunit. The issue here is that the head branch doesn't contain all the latest changes from the base branch.
A fix for this could be to use the merge branch created by GitHub since it contains the changes from both head and base branch.

Does this always exist when creating a PR?

The ref should always exist, unless the PR is created with a merge conflict. Once the merge conflict is resolved, the ref will be created.
Once the ref is created it is updated on every push into the head branch, unless there are merge conflicts between the head and base branch. Once the merge conflicts are resolved it will again contain the latest changes from the head branch and the base branch.
If there are merge conflicts, the default behavior of the pull_request_target trigger is to trigger a build anyway. Once/if we switch to the pull_request trigger, a workflow won't be submitted until the merge conflict is resolved (https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request).

The ref should always exist, unless the PR is created with a merge conflict. Once the merge conflict is resolved, the ref will be created.
Once the ref is created it is updated on every push into the head branch, unless there are merge conflicts between the head and base branch. Once the merge conflicts are resolved it will again contain the latest changes from the head branch and the base branch.
If there are merge conflicts, the default behavior of the pull_request_target trigger is to trigger a build anyway. Once/if we switch to the pull_request trigger, a workflow won't be submitted until the merge conflict is resolved (https://docs.github.com/en/actions/using-workflows/events-that-trigger-workflows#pull_request).

So, what happens right now (with PR_target) if there is a PR conflict?
What is the experience?


bumped api version to v1beta2

https://github.com/timeplus-io/neutron/issues/1238

(Jove Github Bot) moved this ticket out of the GitHub project(up to 1200 tickets for one project).


Use the current model among different people.

When i apply it to the yellow race， I got a lower accuracy . The impact of race on the outcome is certainly great. So I must train my own models. Is that right?

Can you tell how you tested that you get a lower accuracy on asian people? I'm quite interested to know if race actually makes a difference on the accuracy.

@MaartenBloemen  When i use the model provided by MS-Celeb-1M，I got a lower accuracy on asian people, but there is a higher score on Caucasian.

There's obviously differences in facial structure between caucasian and asian people. Since FaceNet extracts features that distinguish between people, it makes sense that if trained on a dataset primarily of caucasians (like ms celeb 1m), it would extract features that help in distinguishing between caucasians. These features are probably slightly different than what would be used to distinguish between asians, and would be less accurate for different races; in fact, humans often suffer from seeing members of different races as all looking the same, because they have not been "trained" to distinguish with features for different races. @ouyangbei you probably need to train FaceNet on your own dataset with triplet loss.

@avikj
How to train FaceNet with my own dataset?
It's ok to follow the instruction of this article "https://github.com/davidsandberg/facenet/wiki/Triplet-loss-training"?


cmd: Moved all non-service sub commands under new tools subcommand.

This will allow better extensibility for the future for non-bucket related tools we plan to add.
Signed-off-by: Bartlomiej Plotka<EMAIL_ADDRESS>

Done! :hugs: @kakkoyun @squat

@kakkoyun @brancz @squat does it make sense now? (:


Add job with check bounds

This PR adds a job that runs our perf target with --check-bounds=yes to make sure that we're not accidentally accessing invalid memory.

@charleskawczynski,  is this current or should be closed?

@charleskawczynski, is this current or should be closed?

It's current.

Merging with soft-fail

bors r+