cmd/go: vendoring fail when put on $GOPATH root with Import Path Checking

I have an issue with vendoring when placed on $GOPATH/src/vendor and "vendored" file use Import Path Checking.
$GOPATH
.
└── src
    ├── main
    │   └── main.go
    └── vendor
        └── vnd.org
            └── a
                └── a.go

with main.go :
package main

import _ "vnd.org/a"

func main() {}

and a.go :
package a // import "vnd.org/a"

go build throw many errors like this :
main.go:3:8: code in directory /[...]/src/vendor/vnd.org/a expects import "vnd.org/a"

According to the spec : "Code inside vendor/ subtrees is not subject to import path checking.".
However import path checking is not properly disabled if the vendored package is at the root of GOPATH since the package name doesn't contain "/vendor/" but starts with "vendor/"
The fix looks easy enough ( replace the condition in file cmd/go/pkg.go line 371 with (!go15VendorExperiment || (!strings.Contains(path, "/vendor/") && !strings.HasPrefix(path, "vendor/"))) ), I can submit a pull request if necessary.

Putting a vendor directory at the root of a workspace directory $GOPATH/src is somewhat nonsensical. There's barely a distinction between doing that and not creating the vendor directory at all.
We should probably disallow it.

This bug make it impossible to locally vendor many golang.org/x/... packages for example. It would be nice to have this fixed in 1.5.x

cc @davecheney

I don't see anything wrong with putting a vendor directory at the root of a workspace directory (eg if the intended usage is to make use of external tools to manage this part of the code).
In this particular case, the fix looks also especially obvious and in line with the rest of the code (see func findVendor, in cmd/go/pkg.go line 639:
	switch {
	case strings.Contains(path, "/vendor/"):
		return strings.LastIndex(path, "/vendor/") + 1, true
	case strings.HasPrefix(path, "vendor/"):
		return 0, true
	}

Can someone explain the use case for vendor/ being right inside $GOPTH/src? I'm attempting to understand when one would put it there.

yea, this sounds strange to me too.. sounds like its useful when doing $GOPATH rewriting for project workspaces...?
I personally wish for vendor/ to have the subdirectories: src/ and pkg/

@pkieltyka why have those subdirectories inside vendor/? The vendor/ directory and its subdirectories live inside the other directories off the GOPATH, for example pkg.

@mattfarina I'd like to see src/ and pkg/ in the vendor/ of my project so I could use vendor/ dependencies for development. In order to use them for development, I need the pkg/ folder so that the vendored deps won't be rebuilt each time. With vendor/pkg, I can keep doing incremental builds of my project and its deps which is essential for keeping development productive.

@pkieltyka Try going into your vendor/ directory and running go install ./.... I think that prebuilds them for you. I use Glide (the package manager) and my cohort wrote something similar to this with glide rebuild.

I guess go install ./... in vendor/ would work, thanks

Marking this as 1.5.1 so that we can make a decision on whether to include it.  I can't tell whether this is a real bug or not.  If it's a real bug, it's new in 1.5, so fixing it in 1.5.1 is not entirely out of bounds.

lol.. all that discussion for a 1 line fix.. nice :) thanks guys


Get Yew routing merged upstream

[x] develop prototype
[x] clean up code
[x] create PR in Yew
[ ] merge or make a local fork for this project depending on the reaction

Migrate the router away from the Router trait. Move that into w@j. Have the set route take a string.


run_tests_python_linux_opt fails on command "python setup.py egg_info"

seen in https://grpc-testing.appspot.com/job/gRPC_pull_requests_linux/1453/testReport/junit/(root)/aggregate_tests/run_tests_python_linux_opt/
copy of log in https://gist.github.com/apolcyn/4a2d83fd7880f04f3e57856e522b0add

note the PR where this failed was on v1.0.x branch

The weirdness about this is that it isn't just an ephemeral network failure; it's downloading things it absolutely shouldn't be downloading.


Instructions that use the PC are considered movable

    def is_movable_instruction(self, addr: int) -> bool:
        is_thumb = self.p.binary_analyzer.is_thumb(addr)
        insn = self.p.binary_analyzer.get_instr_bytes_at(addr)
        asm = self.p.disassembler.disassemble(insn, addr, is_thumb=is_thumb)[0]
        asm = self.p.disassembler.to_asm_string(asm)
        for addr in [0x0, 0x7F00000, 0xFE000000]:
            if self.p.assembler.assemble(asm, addr, is_thumb=is_thumb) != insn:
                return False
        return True

i don't believe the above check is sufficient. Some examples found in the arm printf pie binary are
0000050e  064b       ldr     r3, [pc, #0x18]  {data_528}
00000510  7b44       add     r3, pc  {data_5cc, "Hi"}

which would pass the above check since the disassembly does no change if the base address changes, but these aren't safe to move since they depend on the value of the PC.

Thank you for spotting this issue, fixed in latest commit :)


I think you guy's might've deregistered the festering desire or swapped the registration files or something, maybe it was just a scanning issue, but it registered the festering as a dull blade for me

Prob was a scanning issue. Festering desire is still in the weapon list. Let me know if this still occurs


Update required: Permissions pertaining to accessing device capabilities not present

Feedback:

As this page appears to be an aggregate of all permissions an app could ask for in Teams, I believe that documentation around device permissions too should be added here. For now having a redirection link would also work.
Link to detailed documentation: https://docs.microsoft.com/en-us/microsoftteams/platform/concepts/device-capabilities/native-device-permissions?view=msteams-client-js-latest&tabs=desktop
Page structure is very confusing. The headings and sub-headings don't look very apparent. Clear definition of sections would help in readability.
Hyperlinking items to respective sections at the introductory para would be helpful:

Possibly checking if any other sections also need an update since the documentation was last updated in 2019.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: cd0f81b0-eb71-1c7c-aa8c-25f114e57fd0
Version Independent ID: 9103c060-c536-b3cc-170a-10c2fbfbc834
Content: Microsoft Teams apps permissions and considerations - Microsoft Teams
Content Source: Teams/app-permissions.md
Service: msteams
GitHub Login: @rmw2890
Microsoft Alias: rowille

@shuchauh Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@shuchauh Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

Thank you for submitting feedback. We understand that this issue has been resolved.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Investigate getting Codecov to fail builds on coverage percentage

As shown in PR #68 , a coverage drop wasn't marked as a build failure. I need to investigate configuring Codecov to fail any loss of test coverage automatically.

Landed on develop


error with docker-compose

I can't make the project properly work locally with the given docker-compose.
Actually there is no mongo environnement variable injected into the web container but I can't see why.

I haven't touched docker in a while as I'm not currently using it for my production or staging apps.


Change Result in Vulnerability Attestation to interface{}

Summary
Please feel free to close the PR if there is any requirement on Result in the Vulnerability Scan Record Attestation.
Close https://github.com/sigstore/cosign/issues/2095
Release Note
None
Documentation
None

Codecov Report

Merging #2096 (8f9d314) into main (3e49138) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main    #2096   +/-   ##
=======================================
  Coverage   26.33%   26.33%
=======================================
  Files         129      129
  Lines        7564     7564
=======================================
  Hits         1992     1992
  Misses       5317     5317
  Partials      255      255

Help us with your feedback. Take ten seconds to tell us how you rate us.

I think this is fine, do we need to change the docs anywhere?

It is mentioned as object. I think it is fine. Do you want to change it to any or something like that?
https://github.com/sigstore/cosign/blob/95b74db89941e8ec85e768f639efd4d948db06cd/specs/COSIGN_VULN_ATTESTATION_SPEC.md?plain=1#L83


Die when you hit water

Redact this, we don't want water death anymore, at least not for shallow water
Instead I vote, being in water will halve your jump distance.
Not sure how you'll manage that if for example jumping through "falling" water (I think that may be a possibility)


Changes for cloud based Redis instances that need config-tls

Fixes #65 and #50

Proposed Changes

Moved config-tls config map from samples/source to source/config to install config map during source install,
Modified README to describe instructions for cloud based Redis instances with TLS Certs and a different connection string in address.

This is needed because the config map watcher, when an config update is detected, doesn't re-install the adapter for a new Redis pool connection.

Release Note

 🧽 Update or clean up current behavior of installing config-tls during source install.

/cc @lionelvillard @beemarie

/cc @lionelvillard @beemarie

Ready for merge.

Ready for merge.


Allows Paste in PortableTextEditor

This brings back the Paste functionality for the Portable Text Editor for Firefox.
While https://github.com/sanity-io/sanity/pull/6855 is still pending.

Looks like https://github.com/sanity-io/sanity/pull/6855 has been closed. Can you confirm the paste functionality is now working in Firefox?


Request_time > upstream_response_time ,difference between 2 seconds sometimes, same most of the time

kong :0.14.1
$request_time>$upstream_response_time ,difference between 2 seconds sometimes, same most of the time，return data is small (19k)
$request_time: request processing time in seconds with a milliseconds resolution; time elapsed between the first bytes were read from the client and the log write after the last bytes were sent to the client。
$upstream_response_time ： keeps time spent on receiving the response from the upstream server; the time is kept in seconds with millisecond resolution.

@taotao2014,
I am not sure what you are asking. Is your question about why Kong may add 2 seconds to latency and usually 0 ms latency? Perhaps it is because Kong needs to load something from database? 2 seconds sure is a lot, perhaps DNS resolution, perhaps you have some plugins running there? With this little information it is hard to give exact answer.

log_format json '{"dt":"$time_iso8601","ra":"$remote_addr","ip":"$http_x_forwarded_for","c":"$status","m":"$request_method","up":"$upstream_addr","rt":"$request_time","urt":"$upstream_response_time","h":"$http_host","qs":"$query_string","uri":"$uri","rs":"$body_bytes_sent","ref":"$http_referer","ua":"$http_user_agent","ssl":"$ssl_protocol"}';
$equest_time > $upstream_response_time


Added nix flake and nix shell

Description
Added a nix flake with defaultPackage and app, added default.nix for non flake systems. also added a shell.nix that works on flake and non flake systems
Additional Notes
Works perfectly on my system

Will close #140, sorry for all the hard work on that PR :/

Looks epic! thanks a ton!


Can't call 'graphHopper.clean' successfully after calling 'close' on Android

We are discussing one important issue in this theme
Do you have any idea how this can be fixed?

As discussed: the workaround is to create a different instance and load from a different folder. The problem is probably related to the fact that the memory mapped file cannot be unloaded via the Java API and some native magic would be required.

It could be that this is fixed with this commit where we explicitely unload storage files on Android too. Would be nice if someone (@ArtemNeFRiT :)) could try this?


Journal article 1892081: classification does not match in front office with back office

Bug
Journal article 1892081: classification does not match in front office with back office
Journal article classification:

"U" in back office: https://backoffice.biblio.ugent.be/publication?q=1892081&sort=date-updated-desc
"C1" in front office: https://biblio.ugent.be/publication/1892081

Screenshots:
Back office

Front office

@mietcls fixed by reindexing
was one of a small set of records that had a wrong classification and was autocorrected


Release version 0.8.1

Per comment in #421, we should release the fix for KML / GeoJSON coordinate order.
Looks like we can get #425, #426 in now as well, and otherwise this should be ready to go.
Will update the changelog with date and cut the release shortly.

@jorisvandenbossche since #417 brought in the limit of MacOS 12+ for pre-built wheels, I'm wondering if we should instead release this as 0.9.0 instead of 0.8.1.  That seems like a bigger change than we might want to make for a patch version release.

Agree on that. Bumping OS requirements shall not happen in a patch release.

If it becomes a 0.9, possibly https://github.com/geopandas/pyogrio/pull/422 could be included as well?

Note: for folks that receive emails from PyPI, you'll probably get something shortly about the project having trusted publishing setup but using a private token for upload; this is from me manually uploading the Arm64 wheels created on CircleCI.  You can safely ignore the PyPI email.

Wheels now published to PyPI, including Arm64 wheels built on CircleCI.  🎉

Thanks @brendan-ward!


Actually fix #4801

#4801 is still valid: let's fix it.

If the core team is interested, I have some code that improves the editor configuration page:

adding a description of the plugins
offers an on-the-fly preview of the currently selected plugins

Here's a preview of it:

@mlocati those UI changes with the tooltip and the preview look great!

We'd be happy to accept those.

Here it is: #6720


CI: Improve release process so that SDKs used in examples directory are updated with each release

could add tests as well to ensure they continue to function as expected.

The examples directory is gone now :tada:
https://github.com/dagger/dagger/pull/7752


Add Visible Columns Annotation

Add visible columns annotation to schemas that has been used for test cases, to avoid usage of heuristics.

I talked to @RFSH about this PR today, he said it's okay to merge once updating it with master. It only changes test schemas for a change to ermrestJS. This PR is being merged first because it's compatible with ermrestJS both before and after that PR is merged.


Sign in doesn't work in https://bugs.chromium.org

New Issue Checklist (tick off like this: [x])

[x] I checked existing issues (open and closed) for possible duplicates
[x] I can reproduce the problem on the latest stable version (not nightly!)
[ ] The issue is not about adding entries to the default blacklist

What is your browser?

[x] Firefox
[ ] Chrome

What is your operating system?

[ ] Linux
[ ] Mac
[x] Windows

Description (please include examples/screenshots where applicable)
Pressing sign in in https://bugs.chromium.org/ redirects page back.
After disabling Skip Redirect sign in works.

You need to add it to the blacklist.


Thanks to the open source repository. Is there a C++ version of this driver or a C++ repository with similar parsing functions?

Please use https://answers.ros.org for questions like this.


billing: Edit helper text on sponsorship application

nonprofit -> non-profit, since we were being inconsistent about it. Cf. this reference for choosing the version with a dash. Our landing page uses the dashed form.
Edit pass on default helper text to use more user-oriented language.

Testing plan:
Manual testing.
GIFs or screenshots:

looks good, we can merge once CI passes.

Merged, thanks @alya!


add a new method in IDocumentIndex

LuceneDocumentIndex has already implemented this method, but not its interface.
and also it is useful for custom class in custom Module

I don't like this change for two reasons:

it's a breaking change
it doesn't really add any value, because both indexing and consuming the data would need to know how the data is formatted, which means you could just use a string directly to format what you want.

What is the reason for the change?


[FEATURE] : Make Kin independent from Node.js - Provide independent executables with installers

Is your feature request related to a problem? Please describe.
Most of people who're being introduced to programming doesn't have tools like Node.js installed.  Since Kin requires it to be installed, it adds extra layer of complexity.
Describe the solution you'd like
Provide Kin's executables as independent not depending to Node.js, also add installers for linux && windows OS with clear guide to do it.

Might help: https://www.geeksforgeeks.org/how-to-run-node-js-program-as-an-executable/


Revert "Peform DNF update on edpm node"

This reverts commit c3ec651573761d3ae11949c418074a0bc335fd28.
Attempting to solve system-wide conflicts for all OSP17.1 packages, to upgrade them into versions provided by RDO Antelope DLRN repos (deployed as a hack, until we figure a better source of OSP18 repos on rhel9 hosts used in the Adoption CI) yields problems with openvswitch packages upgrades.
It also seems that we might as well have to carry on OVS RPMs upgrade logic from TripleO [1], [2], and cannot simply 'ensure: latest' by Ansible.
[1] https://github.com/openstack/tripleo-ansible/blob/e281ae7624774d71f22fbb993af967ed1ec08780/tripleo_ansible/ansible_plugins/modules/tripleo_ovs_upgrade.py
[2] https://github.com/openstack/tripleo-heat-templates/blob/1393d39be367db3acb02508e0e858395a4e4fefa/deployment/undercloud/undercloud-upgrade.yaml#L146

note: this cannot be tested for adoption CI as we don't have edpm-ansible in the required-projects for that job.
And more importantly we don't build the operators in data-plane-adoption repo, so that change would not be applied anyway.
I'm going to test it with my custom AEEE operator setup on my adoption dev env

i think we shoudl be able to do https://github.com/openstack-k8s-operators/edpm-ansible/pull/532 instead.
that has the revert but it also has other changes to not regress the update of openstack-selinux
for adoption in think you could just define
edpm_ovs_packages:
 - openvswitch2.15.x86_64

or
edpm_ovs_packages:
 - openvswitch3.2.x86_64

as required instead of the default
edpm_ovs_packages:
 - openvswitch

i think we shoudl be able to do #532 instead.

I'l test this in my adoption setup


New version: Elgato.CameraHub version <IP_ADDRESS>0

[x] Have you signed the Contributor License Agreement?
[x] Have you checked that there aren't other open pull requests for the same manifest update/change?

[x] Does your manifest conform to the 1.2 schema?

Note: <path> is the name of the directory containing the manifest you're submitting.

Microsoft Reviewers: Open in CodeFlow

Uhoh, might have triggered bot protection
2022-09-12 10:55:48.795 [CLI ] Caught wil::ResultException: D:\a\_work\1\s\external\pkg\src\AppInstallerCommonCore\Downloader.cpp(64)\WindowsPackageManager.dll!00007FF851240B70: (caller: 00007FF851240F29) Exception(4) tid(2334) 80190193 Forbidden (403).

    Msg:[Download request status is not success.]

Uhoh, might have triggered bot protection
2022-09-12 10:55:48.795 [CLI ] Caught wil::ResultException: D:\a\_work\1\s\external\pkg\src\AppInstallerCommonCore\Downloader.cpp(64)\WindowsPackageManager.dll!00007FF851240B70: (caller: 00007FF851240F29) Exception(4) tid(2334) 80190193 Forbidden (403).

    Msg:[Download request status is not success.]

Can you please elaborate? I don't have a clue 😅

Can you please elaborate? I don't have a clue sweat_smile

the bot is seeing a Forbidden return code when trying to access the website which often indicates DDOS protection from bots.

@wingetbot run

@wingetbot run

@wingetbot run

@wingetbot run

@wingetbot run

@msftbot reset feedback

Close with reason: Replaced by PR 87235;


error max limit

existe a way to show more than 200 error list?
Or can you add a feature for user can set limit ?
"editor.error.maxLimit": 450,

thank guys
i know this can give maybe a performance issue, but it only for debugging and i have a huge memory pc.

That looks like it is probably an error thrown by the linter you are using rather than VSCode, maybe try using ESLint instead and seeing the output?
Also, Why have you left so many errors, surely you should be fixing them?

@TomSputz
it not me lol , it from the pixi engine
https://github.com/pixijs/pixi.js/releases
i need check if it bad error, or only not used variable for keep compatibility with older version.
thank i take a look in ESLint , not find way to change limit error.

@djmisterjon I am not sure what you are asking for by looking at the screenshot. May I know what is the message in the screenshot?

@sandy081 yep sorry.
By default, if an error has been found, vscode displays a list, but if the list is too large, it is sencured and I can not navigate the list of errors.
I would like the opportunity to see all list error references, even though it exceeds 241.
The message from vscode in translated in english say:

242 additional errors and warnings are not displayed

also am not sure if is the same request here ? but
https://github.com/Microsoft/vscode/issues/9826

Yeah it is related to that

/duplicate of https://github.com//Microsoft/vscode/issues/49687.

the limit is now 1000 errors per file

@jriedken
hi thank for your answer, how i can set it , because as you can see, my limit are 200 ?
is will available in a next update ?


Website update for 1.18.0

@Agirish
Thanks for doing this.  You might want to add the Druid plugin to the release notes: (https://github.com/apache/drill/pull/1888).  Also there was a format plugin for SPSS files.

SPSS Format plugin: https://issues.apache.org/jira/browse/DRILL-7716

@Agirish
Any update on this?
Thx!

Let's get this done..
+1


Fix Dataset.to_file HDF5 functionality

Description

Fixes broken Dataset.to_file HDF5 functionality.
Closes #714.
Changelog description

Fixed broken Dataset.to_file HDF5 functionality.
Status

[ ] Code base linted
[ ] Ready to go

Note, need to set the dataset units for use with anything but driver='energy' datasets. For example:
from qcfractal.interface import FractalClient

fc = FractalClient()

ds = fc.get_collection('Dataset', 'SPICE PubChem Set 1 Single Points Dataset v1.2')
ds.units = "hartree / bohr"

ds.to_file('test.hdf5', 'hdf5')

No longer applicable to v0.50, although we might add HDF5 back into it at some point


Support for requesting SoC temperature for nrf51822 application.

For to use/retrieve the temperature value from the nrf51822 SoC, it would be nice to have a:
public void requestTemperature() { this->_device->requestTemperature(); }

method for the BLEPeripheral class.
For to get notified with the value, we can create a new derived MyBLE class overloading BLEPeripheral virtual method:
void MyBLE::BLEDeviceTemperatureReceived(BLEDevice& device, float temperature);

method. An optional alternative would be to register a callback giving back (only) the temperature.

@bojh it would be great if there was a more generic API, the nRF8001 and nRf51822 both have temp. sensors on board, but future supported boards might not.
What do you think of?
enum BLEDeviceRequest {
  // ..
  BLEDeviceRequest Address,
  BLEDeviceRequest BatteryLevel,
  BLEDeviceRequest Temperature
};

bool BLEPeripheral::request(BLEDeviceRequest request); // returns true on success, false on failure (no sensors)

void setResposneHandler(BLEDeviceRequest request, BLEDeviceResponseHandler response);

Maybe, it's getting too fancy ..
Note: Right now, if you extend BLEPeripheral you can already override,
virtual void BLEDeviceAddressReceived(BLEDevice& device, const unsigned char* address);
virtual void BLEDeviceTemperatureReceived(BLEDevice& device, float temperature);
virtual void BLEDeviceBatteryLevelReceived(BLEDevice& device, float batteryLevel);

and have acces to the following BLEDevices API's:
    virtual void requestAddress() { }
    virtual void requestTemperature() { }
    virtual void requestBatteryLevel() { }

@sandeepmistry The question is cerainly justified and I allready had it realized, as you have proposed by a derived BLEPeripheral class.
On the other hand, I expect also following chipsets (..nrf52x) will also support such universal functions.
From my point of view the fancy way seems a little bit more intuitive, ... but BLEDeviceResponseHandler has to be defined and we have different response value types :-)
... but surely depends a little bit on taste.

@prjh we could make it a union?
union BLEDeviceResponse {
  float f;
  char[6] address;
  // ....
};
typedef void (* BLEDeviceResponseHandler)(BLEDevice& central, BLEDeviceResponse& response);

Slightly unrelated, I've been also thinking about making a breaking change to the API, to expose BLEDevice, changing:
BLEPeripheral                    blePeripheral       = BLEPeripheral(BLE_REQ, BLE_RDY, BLE_RST);

to something like:
// nRF8001
BLENRF8001Device nRF80001            = BLENRF8001Device(BLE_REQ, BLE_RDY, BLE_RST);
BLEPeripheral    blePeripheral       = BLEPeripheral(nRF80001);

// nRF51822
BLENRF51822Device nRF51822            = BLENRF51822Device();
BLEPeripheral    blePeripheral       = BLEPeripheral(nRF51822);

It would make sketches less portable (require one line change), but would support other slave boards like the nRF8001.

@bojh @prjh anything else to discuss on this?

@sandeepmistry OK the union would al be fine, but I we than have to use a discriminator value (may be  BLEDeviceRequest ) to know the callback reason.
Your ideas to the API is OK for me, because the hardware does no change so often for real projects. From compatibility view, I think it's OK to have a individual HW dependend constructor.

OK for me

Re-opening, as this is not implemented yet :)

I will provide an minimal, incremental solution as PR, making the BLEDevice::requestTemperature() function accessible from the BLEPeripheral class.


Support drawing circuit with vertical wires

What should we add?
QuantumCircuit.draw should support drawing the circuit with vertical wires instead of horizontal. This is useful when the length of the circuit exceeds the width of the display area. With horizontal wires, this situation results in line wrapping, but with vertical wires no wrapping would be needed because one can scroll, and it's easier to read the circuit contents that way.

vertical wires no wrapping would be needed because one can scroll
well.. until the amount of qubits gets too big for your screen :)

This is a mayor change. So, here are some alternatives:

Did you try the option fold=...? Most (all?) of the drawers support this option to break lines nicely.
For latex and mpl, we could just flip the result before showing it. This would also change the orientation of the text, of course. Would this be good enough?

Anything beyond that implies rewriting a drawer fully:

for text, it would have to be totally redone.
for latex, the qcircuit package we are using does not fully support vertical circuits (for example, with measurement as described section IV.B of the documentation), so an alternative package would be needed.
for mpl, it might be possible to change orientation of the non-text objects. But even in this case the change wont be small.

Therefore, I think this could be closed as wontfix. The suggestion is good and it comes from an understandable need. I think it could be a good external circuit drawing module. If you think that the mpl options are doable, please feel free to reopen this issue and add change the title accordingly.

@1ucian0 If you agree that the suggestion is good, why close the issue? I understand that you might not think the core developers should spend their time on this, but what if an external contributor wants to? Or even a core contributor on their free time? Would that not be a welcome contribution?

well.. until the amount of qubits gets too big for your screen :)

Indeed, the vertical drawer is most useful for circuits with qubits that fit on the screen. But that is a very important use case.

Did you try the option fold=...? Most (all?) of the drawers support this option to break lines nicely.

Thanks, but that does not achieve the effect I want.

For latex and mpl, we could just flip the result before showing it. This would also change the orientation of the text, of course. Would this be good enough?

That would certainly be a good start. But ideally the text would remain horizontal. So perhaps a transpose option could be easily added for these two cases, with vertical being a longer term project.

for text, it would have to be totally redone.

Perhaps, but let me just point out that Cirq's text diagram drawer supports vertical display and the code is not that complicated. Indeed, Cirq's drawer partly inspired this issue because I find it very useful and nice.

for latex, the qcircuit package we are using does not fully support vertical circuits (for example, with measurement as described section IV.B of the documentation), so an alternative package would be needed.

I've had good experiences using qpic.

If you think that the mpl options are doable, please feel free to reopen this issue and add change the title accordingly.

I don't have permission to reopen the issue.

Would [vertical drawers] not be a welcome contribution?

To put it a little bluntly, yes, it wouldn't be welcome in Terra, but that's not at all because it's not useful.  We'd love to see a more complete external library for circuit visualisation, where a separate set of maintainers could give it the time it needs, but that's not what Terra is.  We already struggle enough to maintain the limited visualisation support we offer, and only cope through the contributions of a very dedicated external contributor.
We're open source, but we still need to manage our priorities, and somebody putting in a lot of effort doesn't make Terra the best fit for a feature.  We have a list of contributions that we'd find useful, including features we'd be happy to accept - those are the issues tagged with "help wanted" or assigned to somebody.  We need to close issues that we wouldn't be happy to accept, to make it clear for people who come looking that this isn't something we'd like to add.

Indeed, the vertical drawer is most useful for circuits with qubits that fit on the screen. But that is a very important use case.

This isn't meant as discouragement at all, just a design point to consider: even with only small numbers of qubits, things like register names and instruction names would probably be displayed horizontally in a text-mode drawer, and those can be arbitrarily wide (and for high-level instructions often are).  A vertical-mode drawer probably still needs to consider how it will handle wrapping, even if its intent is only to support small numbers of qubits.


github_release_notes task adds a parenthesis to PR link

Describe the bug
The github_release_notes task incorrectly adds a closing parenthesis to the PR link.
Example output is:
# Changes
                    123456 - Did the thing
                    [[PR537](https://github.com/foo/bar/pull/537)]
                    123457 - Did something else
                    [[PR539](https://github.com/foo/bar/pull/539)]

If I click the links, I am redirected to https://github.com/foo/bar/pull/537) and https://github.com/foo/bar/pull/539) (note the parenthesis).  We only keep our builds for a month but this has been happening since at least April 4th.
Reproduction steps
Run a flow that uses the github_release_notes task such as release_2gp_beta
Your CumulusCI and Python versions
CumulusCI: 3.56.0
Python: 3.8.12
Operating System
Linux
Windows environment
No response
CumulusCI installation method
brew
Error Gist
No response
Additional information
The CI job is running as an Azure DevOps Pipeline if that makes a difference.

@georgeslater I'm not able to repro this with your example output:
[[PR537](https://github.com/foo/bar/pull/537)]

Produces: [PR537]
Where is the markdown being rendered? Or are you attempting to click the URL from your CI logs instead?

Sorry, you're absolutely right - it only fails from within the Azure CI system.  In GitHub itself there is no issue.  I'll close this out.


Warped image after Oculus API change

Some games produce a warped image after oculus changed its required API beginning with Oculus Version 9. The result looks like the field of view is way to large.
The distortion is caused by the required <uses-feature android:name="android.hardware.vr.headtracking" android:required="true" android:version="1"/>
in the client manifest to enable controller tracking on later firmware versions
Known broken games are:

DCS World
Rise of the Tomb Raider
Rainfall Megaton
A-Tech Cybernetic VR

After the change from oculus, the reported fov changed from:
GetProjectionRaw Eye=0 (l,r,t,b)=(45.000000,45.000000,45.000000,45.000000)
to
GetProjectionRaw Eye=0 (l,r,t,b)=(52.000000,42.000000,53.000000,47.000004)
I tried to change it back manually, but this parameters change the rendering on the quest and forcing it to the old values did not help

Found the fix! #102 Before I proceed with the fix, I want to know: is DCS the only one that require the tracking system string to be changed? Is a "Emulate Vive headset" option enough or is it better to allow the user to set custom strings for tracking system, serial number, model number, etc... ? It happens that when I substitute all Oculus S strings with Vive ones (taken from polygraphene's repo), when I disable the controllers and I use a gamepad, in the SteamVR house the gaze pointer no longer works so I cannot teleport and cannot use the UI. But when using the controllers everything seems fine.

Maybe we should add the Vive as emulated headset like the Rift S Controller Mode in the other tab.
At first I was thinking that the faulty games should be fixed, but I realize now that that will never happen

@eyelykaynal please test ev9 with DCS World
@Letrange-g can you please test your games; Rainfall Megaton and A-Tech Cybernetic VR with ev9 and the warp fix? I'm not sure how Rainfall Megaton should look like and I can't play it for long without motion sickness

Hey man,
I will aim to do this tomorrow, time permitted.
On Sat, 29 Feb 2020, 14:27 JackD83<EMAIL_ADDRESS>wrote:

@eyelykaynal https://github.com/eyelykaynal please test ev9 with DCS
World
@Letrange-g https://github.com/Letrange-g can you please test your
games; Rainfall Megaton and A-Tech Cybernetic VR with ev9 and the warp fix?
I'm not sure how Rainfall Megaton should look like and I can't play it for
long without motion sickness
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/JackD83/ALVR/issues/88?email_source=notifications&email_token=AFPO6UNTL4YO2N4ILDJXVRDRFDYJLA5CNFSM4JXJ7WOKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENLXUSA#issuecomment-592935496,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFPO6UK3LZNYKI46EL4LFD3RFDYJLANCNFSM4JXJ7WOA
.

This is working quite well now. The warping is gone but the cockpit looks a little jittery when looking around. This could possibly fixed with some tweaking of the settings though, I will let you know when I've had a chance to try.
Nice work!


Revert "Run Router in a container in Staging"

Reverts alphagov/govuk-puppet#7299
We are making this change to test that the S3 buckets are usable as we would expect during a normal operation.

I'd recommend respinning the instances to ensure there are no resources left lying around.


[LO-Refactor][T09-A1]yanhao

This the refactor I did,but I am still confused about git and github usage...

@vane1992 Some comments added. Please close the PR after reading comments.


Fix table in README

The table was not properly formatted and therefore not typeset by Markdown.

Thanks!


An option to avoid prepending Path environment variable with directory containing 'git.exe' in Checkout step

In Build pipeline, before repo is checked, there is initialization procedure which prepends Path variable with git.exe.
In my self-hosted agent there is already git.exe present. This git.exe instance is linked to a config pointing to specific credential store. In my pipeline script I am cloning another helper repo. But as different git.exe is executed, different config is used so cloning this other repo fails.
I can specify the full path to my git.exe in my script, but as I'd like to avoid paths hardcoding, it would be nice to have an option:

To disable PATH variable modification
To somehow define that git.exe used by Azure should use my specific config

There is an existing Agent_Knob to fix this system.prefergitfrompath set this to true to accomplish what you want. You can either set this in the agent's config file or in the system's environment variables. In ome cases you can simply add this as a variable of your pipeline.
https://github.com/microsoft/azure-pipelines-agent/blob/1adfc03151ede59addaf6c5e9eaa55aad57b6a7a/src/Agent.Sdk/Knob/AgentKnobs.cs#L73-L78

Hey @jan-tosovsky-cz, @jessehouwing correctly mentioned that you could use system.prefergitfrompath set to true to achieve what you want. In this way, the agent will use git that is preinstalled in the system instead of bundled version.

Thanks for the hint. I am failing to find any mention in documentation WHERE actually this should be set. What is 'agent's config file'? Can you elaborate, please?

@jan-tosovsky-cz You can set this variable as an environment variable in the system, where the agent is executing

You can also add the variable to a .env file in the agents installation directory.
https://docs.microsoft.com/en-us/azure/devops/pipelines/agents/v2-windows?view=azure-devops#how-do-i-set-different-environment-variables-for-each-individual-agent
Make sure you restart the agent after changing.either the global systems environment variables or the env file.
And many of these knobs will simply trigger when a pipeline variable with the right name is set, since they are also exported to the environment when a job runs.

@jan-tosovsky-cz Is it still an actual issue? Could we close this work item?

@jan-tosovsky-cz Let me close this issue, in case of any future questions feel free to ping me


For Review Only Apex 247 - Implementing Delay operator

@davidyan74 Please review. I

@sandeshh What are the next steps for this PR?

@gauravgopi123 This implementation is outdated because there is already default implementation in Core which doesn't need WindowDataManager. So updating it for N-DelayOperator. Do you want me to create a new pull request ?

Please close this


test: Add a test for incorrect message signature in is_signed_by_address

In response to this comment: https://github.com/rust-bitcoin/rust-bitcoin/pull/819#discussion_r801477961
This should be straightforward. Let me know if there are any style issues. I tried to keep things similar to the existing test while cutting out any extra cruft to keep things small.

I think it'd be nice to have another test that has a correct signature for correct message but the address is different. This would protect us if someone refactors the code and forgets to check the address. I actually experienced something similar in unrelated cryptographic code and it's also a bit similar to the fraud faketoshi attempted.

@Kixunil Is that not what the existing test is doing (not this PR)?
let p2wpkh = ::Address::p2wpkh(&pubkey, ::Network::Bitcoin).unwrap();
assert_eq!(signature2.is_signed_by_address(&secp, &p2wpkh, msg_hash), Ok(false));
let p2shwpkh = ::Address::p2shwpkh(&pubkey, ::Network::Bitcoin).unwrap();
assert_eq!(signature2.is_signed_by_address(&secp, &p2shwpkh, msg_hash), Ok(false));

P2WPKH and P2SH_P2WPKH address are derived from the same key, but they fail due to only P2PKH being supported.
I'm not sure what other ways the addresses could be different. The network could be different but it should always pass with the same pubkey. I don't think I understand what you are trying to get at.

There is no difference between "valid signature with wrong message" and "invalid signature", assuming the siganture is formatted correctly.

Yes.


Hello，what can i do with this project?

is this project runnable?
can i train the representation network and generation network with the dataset provided by deepmind?
or with my own dataset?
or i just can inference the model with pre-trained model?
@musyoku @ktns

Hi.
The goal of this project is to get the same result as this video.
https://youtu.be/G-kWNQJ4idw?t=32

You can

create your own dataset
train a model
generate images

This project is currently under development.

Hello，thanks for your reply!
This is a wonderful project! I am also very interested in GQN, but I am not able to implement it.
So what can we do with this project now?
I saw you ticked in front of "implement GQN"
But did not tick the "implement training loop"
So is this project only able to inference but can't train?
but how can we inference without training? do you have a pre-trained model?

Hi @musyoku !
How many iterations can you finish in two weeks with 1080?
I just can finish 1 iteration per day with GTX TITAN X(maxwell) and I saw in your code the iteration range is 2*10**6 . Maybe I will never finish the train.
My dataset is generated by your “create_dataset.py”, it contains 2 millions samples. And it is around 500GB and stored in HDD(7200rpm) driver. Will it be much quicker with SSD?

I think that SSD is faster than HDD because my training code reads the file for each iteration and generates minibatch.
I finished 7 iterations (387,800 steps / 2,000,000) in 5 days on a single 1080.
Importantly, I have never finished training, so there may be a bug in the code and not get the same results reported by DeepMind.

Hello @musyoku
will you support maze scenario?

I will not but I want to support if I have time

The PIG is approximated at every point by averaging over 50 samples per heading directions.

PIG is computed by averaging over 50 x that are sampled from the generator network.
x_n ~ g(x|z,y)(z|y)
PIG = 1/50 * {IG(x_1, y) + IG(x_2, y) + ... + IG(x_50, y)}
I'm very sorry but I would like to discuss in issues.
And my English skill is not good enough to discuss :confused: (I'm using Google translation)

My understanding is

Input (v_q, r) to the generator and output x_n. (v_q is a random query viewpoint covering the maze)
Input x_n to the inference network and output (z_mean_q, z_var_q, z_mean_p, z_var_p).
Compute IG (KL divergence) between (z_mean_q, z_var_q) and (z_mean_p, z_var_p)

@musyoku
OK Thank you very much!
I have another question, I saw your create_dataset.py in the room scenario generates a Cornell box with every wall a solid color( each wall has just one color). I want to add a texture to the floor, how could I implement it? (I have my own picture and want to post it to the floor as a texture map)
Thank you again

I am implementing the texture shader.

ok! thank you~

Hello, @musyoku which license do you choose?
Apache, GPL, or MIT?

MIT


Can we integrate with mongodb

Can we use mongodb as backend instead of redis?

Simple answer is No.
There already exist a couple of mongo backed task queues on npm, however if they don't fit your, you should check Kue's code and propose a adapter-based redesign, with implementation specifics for job pulling in mind.


README example is missing a key parameter!

After many hours debugging, I realized the example shown was "missing a piece". Please update the README.md
Original post that took me home:
I was facing the same issue but realised I did not hide the modal manually in the onConfirm handler.
Make sure you set your isVisible variable to false in the onConfirm handler.
Example :
const [isDatePickerVisible, setDatePickerVisibility] = useState(false);

<DateTimePickerModal
        isVisible={isDatePickerVisible}
        mode="date"
        onConfirm={handleConfirm}
        onCancel={hideDatePicker}
      />
...

  const handleConfirm = (date) => {
    setDatePickerVisibility(false);
  };

Originally posted by @bvangrimberge in https://github.com/mmazzarolo/react-native-modal-datetime-picker/issues/251#issuecomment-961857624

What example in the README.md is missing not hiding the modal in the onConfirm? The main one looks fine to me.
Please double-check it and submit a PR with the fix if you still think there's an issue with it, thanks.

Hi Matteo,
My bad, I was working with https://github.com/react-native-datetimepicker which does not mention 'isVisible', but found the right way to do it on this repo's issues, from which I commented.
I just realized this while trying to provide the PR you suggested.
My apologies for wasting your time,
JMV

No worries! Have a nice day :)


Windows build panicks on unwrap()

Windows 10.
Latest (0.2.7) prebuilt .exe file I downloaded from the releases page.
Error:

@panstromek You are missing the winpty-agent from the releases page as the error indicates.
https://github.com/jwilm/alacritty/releases
Put the winpty executable in the same directory and it should run normally.
Alternatively, if you're using Windows 10 version 1803+, you can try setting enable_experimental_conpty_backend to true to use the Windows conPTY instead of winpty-agent. This feature is experimental as the value indicates, so beware.

Oh - thanks a lot, I missed that  ;) works now.
btw. is there a way to embed the window into IntelliJ terminal window? If I set it in intelliJ it always opens a new window.


Use a larger DH parameter

We need to change the current Diffie-Hellman parameter we are using, in order to get a "A" grade on SSSLabs and close H1-117458 + H1-131065.
To generate a now group with the recommended size, just run openssl dhparam -out dhparams.pem 2048 in /etc/nginx/. It will take some time.
(I don't really know how we should handle these kind of security reports, since nothing is specified on the how-to and the related configuration file don't have to be in this repository. I guess that making one issue and mentioning @whit537 is not the worst way.)
Tell me when it's done, so I can PR the related nginx configuration directive :-)

Like this?
root@grtp:~# cd /etc/nginx/
root@grtp:/etc/nginx# l
certs/   fastcgi.conf    koi-utf  mime.types  nginx.conf.dpkg-old  scgi_params       sites-enabled/  uwsgi_params
conf.d/  fastcgi_params  koi-win  nginx.conf  proxy_params         sites-available/  snippets/       win-utf
root@grtp:/etc/nginx# openssl dhparam -out dhparams.pem 2048
Generating DH parameters, 2048 bit long safe prime, generator 2
This is going to take a long time
............................................................+........................................................................................................................................................................................+.......................................................................................................+..........................................................................................

Yep 👍

Tell me when it's done, so I can PR the related nginx configuration directive :-)

It's not done yet, but you can start the PR whenever, and we'll simply wait to deploy. :)

Erp ... it's done! 💃

FTR, everything is working as expected and ssllabs is still giving us a B but for one other issue. I'm taking a look at it.


Fix typo (tree/three) in uart.rst (IDFGH-2054)

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

@bcran Thanks for your contribution, we'll put it into internal review queue.

Merged in https://github.com/espressif/esp-idf/commit/bbb4e0e8f84ef03933391615b4f97396f2626af9 Thanks!


[FP]: False positive on io.asyncer:r2dbc-mysql

Package URl
<EMAIL_ADDRESS>CPE
cpe:2.3:a:mysql:mysql:1.0.3:*:*:*:*:*:*:*)
CVE
No response
ODC Integration
{"label"=>"Maven Plugin"}
ODC Version
8.4.0
Description
it seems false positive where io.asyncer:r2dbc-mysql:* is being mistaken for for mysql:mysql:*.
related issue: https://github.com/asyncer-io/r2dbc-mysql/issues/148

approved


client hangs while running

I'm using Python 2.7.12, iperf 3.0.11 on a Ubuntu 16.04.1 x86_64 machine.
I configured the client to run as follows:
>>> import iperf3
>>> client = iperf3.Client()
>>> client.duration = 125
>>> client.bind_address = '<IP_ADDRESS>'
>>> client.server_hostname = '<IP_ADDRESS>'
>>> client.port = 9923
>>> client.bandwidth = (5096 * 1024)
>>> client.num_streams = 3
>>> client.protocol = "tcp"
>>> client.bulksize = (client.bandwidth * client.duration)/8
>>> client.run()

The bandwidth parameter is configured that way since I need 5096 kbps. The Bulksize parameter is kept as it is to ensure that the total throughput (total bits sent/total duration) matches the bandwidth value of (5096*1024). I've run the same code with smaller throughput and smaller duration values and it works great.
There are two problems:

When running this client, the client stalls. It does not terminate on my machine. For lower throughput/duration values, it works like a charm. Whats the problem here? I always have to kill it for this specific test!

I've noticed that the traffic is bursty in nature i.e. it attempts to send most of the data within the 1st second itself rather than evenly distributing the data over the entire test duration. Is there a way where I can achieve my expected throughput but have evenly distributed traffic as well?

Yes, this problem really exists.
I have been tested iperf3 wrapper using multiple combinations of parameters for TCP Client and concluded that issue not related to any parameters of traffic, only one has a weight - this is duration.
If I set duration MORE than 99 (seconds) - client won't return any data and hang.

Anyone know if there any fix on this?
I am encountering same issue on rhel7 and Ubuntu 16.04. When i use iperf3 module with python3 it hangs for interval > 130...
When I run iperf3 tool itself command line it works just fine with long test duration.
Thanks, any insight appreciated! :)
(iperf3env) # python3
Python 3.6.8 (default, Jun 11 2019, 15:15:01)
[GCC 4.8.5 20150623 (Red Hat 4.8.5-39)] on linux
Type "help", "copyright", "credits" or "license" for more information.

import iperf3
client = iperf3.Client()
client.duration = 10
client.server_hostname = '<IP_ADDRESS>'
client.bandwidth = 50000000
result = client.run()
result.sent_Mbps
49.561612938104
del client
client = iperf3.Client()
client.duration = 131
client.server_hostname = '<IP_ADDRESS>'
client.bandwidth = 50000000
result = client.run()
^CTraceback (most recent call last):
File "", line 1, in
File "/home/nick/iperf3env/lib/python3.6/site-packages/iperf3/iperf3.py", line 616, in run
error = self.lib.iperf_run_client(self._test)
KeyboardInterrupt
quit()
(iperf3env) #

(iperf3env) # iperf3 -c <IP_ADDRESS> -b 50000000 -t 180
Connecting to host <IP_ADDRESS>, port 5201
[  4] local <IP_ADDRESS> port 38590 connected to <IP_ADDRESS> port 5201
[ ID] Interval           Transfer     Bandwidth       Retr  Cwnd
[  4]   0.00-1.00   sec  5.46 MBytes  45.8 Mbits/sec   29    417 KBytes
[  4]   1.00-2.00   sec  5.88 MBytes  49.3 Mbits/sec    0    427 KBytes
[  4]   2.00-3.00   sec  6.00 MBytes  50.3 Mbits/sec    0    436 KBytes
...
[  4] 177.00-178.00 sec  6.00 MBytes  50.3 Mbits/sec    0    622 KBytes
[  4] 178.00-179.00 sec  5.88 MBytes  49.3 Mbits/sec    0    625 KBytes
[  4] 179.00-180.00 sec  6.00 MBytes  50.3 Mbits/sec    0    625 KBytes

[ ID] Interval           Transfer     Bandwidth       Retr
[  4]   0.00-180.00 sec  1.05 GBytes  50.0 Mbits/sec   82             sender
[  4]   0.00-180.00 sec  1.05 GBytes  50.0 Mbits/sec                  receiver
iperf Done.
(iperf3env) #

i am triying to test bandwidth using iperf3 in python but i am getting error
Traceback (most recent call last):
File "C:\Users\Administrator\Desktop\Python310\iperfm.py", line 2, in
client = iperf3.Client(r'C:\Users\Administrator\Downloads\iperf-3.1.3-win64\iperf-3.1.3-win64\iperf3.exe')
File "C:\Users\Administrator\Desktop\Python310\iperf3-python\iperf3\iperf3.py", line 418, in init
super(Client, self).init(role='c', *args, **kwargs)
TypeError: IPerf3.init() got multiple values for argument 'role'

how to run this code in windows
import iperf3
client = iperf3.Client(r'C:\Users\Administrator\Downloads\iperf-3.1.3-win64\iperf-3.1.3-win64\iperf3.exe')
client.duration = 1
client.server_hostname = '<IP_ADDRESS>'
client.port = 5201
client.run()


tooth ankylosis

tooth ankylosis
DEF: solid fixation of a tooth resulting from fusion of the dental cementum and alveolar bone, with obliteration of the periodontal ligament
Exact synonyms: ankylosis of teeth ;   dental ankyloses ;   dental ankylosis ;   dentoalveolar ankyloses ;   dentoalveolar ankylosis ;   teeth ankyloses ;  teeth ankylosis ;   tooth ankyloses
SOURCE: RDO:0001938
Annotation in J:50144

tooth ankylosis MP:0030449


Uploading  the same file for the second time

If you upload a file, then click X to remove the text, then after that try to upload the same file for the second time, the text is not shown on editor.

Can you please double check whether that still happens? If it works fine feel free to close this issue then (unless you still want to keep this open?)
Can you also check other scenarios, i.e.uploading different files and so on.

Sure, I am going to check other scenarios.


GLDM matrix calculation

How gldm matrix is calculated for the different angles for the given image. The output shown for this calculation is given as single matrix without specifying at which angle it is calculated. Can you please clarify the scope of calculating features in different angles and how it is implemented in this package?

All angles are used to build 1 GLDM matrix. This is similar to how NGTDM and GLSZM are calculated. This is explained in detail in the online documentation.


(In Code Review) Add created_at CaseService field

https://jira.desk.technology/browse/APIJ-50

:+1:


Make chobby config part of dynamic configuration

This could be useful to be able to select from launcher some parameters that then chobby can read, so that user can more dynamically configure chobby without having to deal with lua files.

Resolved via https://github.com/beyond-all-reason/spring-launcher/commit/02cafb079d950f6b38eccc1af2f4550781f48d6c


Flutter 3.0 : Execution failed for task ':pay_android:compileDebugKotlin'.

e: /Users/karmaln/Developer/flutter/.pub-cache/hosted/pub.dartlang.org/pay_android-1.0.6/android/src/main/kotlin/io/flutter/plugins/pay_android/PayMethodCallHandler.kt: (50, 69): Type mismatch: inferred type is Activity? but Activity was expected
e: /Users/karmaln/Developer/flutter/.pub-cache/hosted/pub.dartlang.org/pay_android-1.0.6/android/src/main/kotlin/io/flutter/plugins/pay_android/PayMethodCallHandler.kt: (69, 74): Type mismatch: inferred type is String? but String was expected
e: /Users/karmaln/Developer/flutter/.pub-cache/hosted/pub.dartlang.org/pay_android-1.0.6/android/src/main/kotlin/io/flutter/plugins/pay_android/PayMethodCallHandler.kt: (73, 34): Only safe (?.) or non-null asserted (!!.) calls are allowed on a nullable receiver of type Map<String, Any>?
e: /Users/karmaln/Developer/flutter/.pub-cache/hosted/pub.dartlang.org/pay_android-1.0.6/android/src/main/kotlin/io/flutter/plugins/pay_android/PayMethodCallHandler.kt: (74, 34): Only safe (?.) or non-null asserted (!!.) calls are allowed on a nullable receiver of type Map<String, Any>?

FAILURE: Build failed with an exception.

* What went wrong:
Execution failed for task ':pay_android:compileDebugKotlin'.
> Compilation error. See log for more details

* Try:
Run with --stacktrace option to get the stack trace. Run with --info or --debug option to get more log output. Run with --scan to get full insights.

* Get more help at https://help.gradle.org

BUILD FAILED in 26s
Exception: Gradle task assembleDebug failed with exit code 1

in android/build.gradle

dependencies {
        classpath 'com.android.tools.build:gradle:4.1.1'
        classpath "org.jetbrains.kotlin:kotlin-gradle-plugin:$kotlin_version"
    }

in app/build.gradle
compileSdkVersion 31
minSdkVersion 21

dependencies {
    implementation "org.jetbrains.kotlin:kotlin-stdlib-jdk7:$kotlin_version"
    implementation 'com.google.android.gms:play-services-wallet:19.1.0'
}

in androidMainfest.xml
<meta-data
            android:name="com.google.android.gms.wallet.api.enabled"
            android:value="true" />

For anyone else finding this, https://github.com/google-pay/flutter-plugin/issues/126#issuecomment-1125747859 fix the issue for me

For anyone else finding this, #126 (comment) fix the issue for me

solution did not work for me

Same problem for me

Same problem for me

wait for this pr get merge https://github.com/google-pay/flutter-plugin/pull/129

When can we expect a release with that issue fixed?

pay: 1.0.9 and pay_android: 1.0.7 are now published, addressing this issue.
Thank you.

Launching lib\main.dart on SM M022G in debug mode...
lib\main.dart:1
e: C:\src\flutter.pub-cache\hosted\pub.dartlang.org\tflite_flutter_helper-0.3.1\android\src\main\kotlin\com\tfliteflutter\tflite_flutter_helper\TfliteFlutterHelperPlugin.kt: (43, 1): Class 'TfliteFlutterHelperPlugin' is not abstract and does not implement abstract member public abstract fun onRequestPermissionsResult(p0: Int, p1: Array<(out) String!>, p2: IntArray): Boolean defined in io.flutter.plugin.common.PluginRegistry.RequestPermissionsResultListener
e: C:\src\flutter.pub-cache\hosted\pub.dartlang.org\tflite_flutter_helper-0.3.1\android\src\main\kotlin\com\tfliteflutter\tflite_flutter_helper\TfliteFlutterHelperPlugin.kt: (143, 2): 'onRequestPermissionsResult' overrides nothing
FAILURE: Build failed with an exception.

What went wrong:
Execution failed for task ':tflite_flutter_helper:compileDebugKotlin'.

Compilation error. See log for more details

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 51s
Exception: Gradle task assembleDebug failed with exit code 1
**any one can help me please ?
I have an error  with tflite_flutter_helper
i use it to pre-processr images  **

Getting the same error .Whats the solution


Add Support for the Testing API

Add support for the vscode testing API. This would allow for auto-discovering and running t/*.t tests tests directly in vscode. Open to design decisions, but I suspect we should run them via https://perldoc.perl.org/TAP::Harness as it is a core module and we can leverage the perl interpreter setting (as opposed to requiring a user to specify a location for the prove binary).
This is what the API looks like for an example set of tests in Python.
https://code.visualstudio.com/api/extension-guides/testing

Making a start on this. Using TAP::Harness is a good idea, but would ideally like to allow support for App::Yath/Test2::Harness.

Yeah, I second allowing some config for what to run tests with.
This is cool!

I'm going to investigate potentially using Test2::API for this, I believe its available as a core module.

Poking around with the details of TestController and figuring out how to piece everything together (I don't have much experience with writing VSCode plugins).
Looking to make use of Test2::API::intercept:
my $events = intercept {
    do '/path/to/test.t';
};

https://metacpan.org/pod/Test2::API::InterceptResult#SYNOPSIS
The goal would be to obtain the relevant data to pass into TestItem.
There may be a more effective way of achieving this but still experimenting and investigating the docs/code.

use Test2::API qw<intercept_deep>;
use JSON::PP qw<encode_json>;

my $events = intercept_deep {
    do $ARGV[0];
};

say encode_json($events->squash_info->flatten);

Example test file as input for above:
use strict;
use warnings;

use Test::More;

ok 1, 'yay';
ok 0, 'nay';

subtest 'nest' => sub {
    ok 1, 'subtest yay';
    ok 0, 'subtest nay';
    is_deeply {foo => 1}, {bar => 0}, 'subtest hash'
};

done_testing;

Resulting JSON:
[
  {
    "name": "yay",
    "pass": 1,
    "trace_file": "./t/hello.t",
    "causes_failure": 0,
    "trace_line": 8
  },
  {
    "trace_line": 9,
    "trace_file": "./t/hello.t",
    "causes_failure": 1,
    "pass": 0,
    "name": "nay",
    "diag": [
      "  Failed test 'nay'\n  at ./t/hello.t line 9.\n"
    ]
  },
  {
    "note": [
      "Subtest: nest"
    ],
    "causes_failure": 0,
    "trace_file": "./t/hello.t",
    "trace_line": 15
  },
  {
    "trace_line": 12,
    "trace_file": "./t/hello.t",
    "causes_failure": 0,
    "pass": 1,
    "name": "subtest yay"
  },
  {
    "trace_line": 13,
    "trace_file": "./t/hello.t",
    "causes_failure": 1,
    "pass": 0,
    "name": "subtest nay",
    "diag": [
      "  Failed test 'subtest nay'\n  at ./t/hello.t line 13.\n"
    ]
  },
  {
    "name": "subtest hash",
    "diag": [
      "  Failed test 'subtest hash'\n  at ./t/hello.t line 14.\n"
    ],
    "pass": 0,
    "trace_file": "./t/hello.t",
    "causes_failure": 1,
    "trace_line": 14
  },
  {
    "causes_failure": 0,
    "trace_file": "./t/hello.t",
    "trace_line": 14,
    "diag": [
      "    Structures begin differing at:\n         $got->{bar} = Does not exist\n    $expected->{bar} = '0'\n"
    ]
  },
  {
    "trace_file": "./t/hello.t",
    "causes_failure": 0,
    "trace_line": 15,
    "plan": "3"
  },
  {
    "diag": [
      "Looks like you failed 2 tests of 3.\n"
    ],
    "trace_line": 831,
    "trace_file": "/home/dango/.asdf/installs/perl/5.38.0/lib/5.38.0/Test/More.pm",
    "causes_failure": 0
  },
  {
    "trace_line": 15,
    "subtest": {
      "follows_plan": 1,
      "is_passing": 0,
      "failed": 2,
      "count": 3,
      "plan": 3
    },
    "causes_failure": 1,
    "trace_file": "./t/hello.t",
    "pass": 0,
    "name": "nest",
    "diag": [
      "  Failed test 'nest'\n  at ./t/hello.t line 15.\n"
    ]
  },
  {
    "trace_line": 17,
    "trace_file": "./t/hello.t",
    "causes_failure": 0,
    "plan": "3"
  }
]

Thanks @m-dango, this is great. Support for tests would be a great addition. I'll take a closer look later this week, but I also wanted to point out this extension that's worth looking at for design ideas. https://github.com/Nihilus118/vscode-perl-test-provider


Hybrid Search Tutorial

Could we turn this to use the hybrid search interface introduced in https://github.com/qdrant/qdrant-client/pull/553?

@Anush008 I'd prefer to do that in a separate notebook once a new Qdrant client release is made with these changes. If you'd like, please feel free to raise a PR for that as well?

@Anush008 I'd prefer to do that in a separate notebook once a new Qdrant client release is made with these changes. If you'd like, please feel free to raise a PR for that as well?

it is available as of qdrant-client == 1.8.2
AFAIK, @generall wanted to do some example with it as well, not sure what’s the status now

AFAIK, @generall wanted to do some example with it as well, not sure what’s the status now

I will add another notebook in a fresh PR with the latest Qdrant Client changes for hybrid and sparse support both. I need this for the docs as well as some questions from users
cc @joein @generall


Add aes-gcm support in afalg engine

Hi,
The following patch series adds aes-gcm support in openssl's afalg engine and a afalg gcm test code.
The support is added for key lengths 128, 192 and 256 bits.
Added nids of aes gcm, so that it is initialized during engine_init.
For encrytion and decrytion, the input is copied into iovec structure. The key, operation
type(encryption/decryption) and associated length is set via setsockopt options.
This inputs are send using sendmsg via afalg socket.
The output is collected asynchronously using aio apis, io_getevents() and io_read().
This patch is tested for openssl speed and openssl s_server and openssl s_client tests.
Please review the series and provide your comments.
Thanks,
Ayush
Checklist

[ ] documentation is added or updated
[x] tests are added or updated

reopening PR again for to update cla flag

I am not sure the background, but I have a question, what's the difference betweeen this afalg and the scoket's AF_ALG type in kernel?

I am not sure the background, but I have a question, what's the difference betweeen this afalg and the scoket's AF_ALG type in kernel?

The afalg code in this commit will utilize kernel's Socket type AF_ALG so that user space apps can use kernel crypto api.
cbc support is present in the existing afalg engine. This commits adds the support for aes(gcm).

I am not sure the background, but I have a question, what's the difference betweeen this afalg and the scoket's AF_ALG type in kernel?

The afalg code in this commit will utilize kernel's Socket type AF_ALG so that user space apps can use kernel crypto api.
cbc support is present in the existing afalg engine. This commits adds the support for aes(gcm).

Thanks for the explanation.  As far as I understand, there is another socket type named IF_KTLS in kernel, it also utilized aes-gcm and implement the record layer in kernel. So, with AF_ALG, what's the peroformance gain compared with IF_KTLS?

I am not sure the background, but I have a question, what's the difference betweeen this afalg and the scoket's AF_ALG type in kernel?

The afalg code in this commit will utilize kernel's Socket type AF_ALG so that user space apps can use kernel crypto api.
cbc support is present in the existing afalg engine. This commits adds the support for aes(gcm).

Thanks for the explanation. As far as I understand, there is another socket type named AF_KTLS in kernel, it also utilized aes-gcm and implement the record layer in kernel. So, with AF_ALG, what's the peroformance gain compared with AF_KTLS?

The AF_KTLS is not a valid type in mainline kernel, so i have not tried testing with that.

I'm taking this out of the 3.0 beta1 milestone. Its too close IMO for such a significant feature to be added.

This PR is in a state where it requires action by @openssl/committers but the last update was 30 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 61 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 92 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 123 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 123 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 154 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 185 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 216 days ago

This PR is in a state where it requires action by @openssl/committers but the last update was 247 days ago

As engines are deprecated in 3.0 I do not think we want to add new features to the existing engines shipped in 3.0. I am closing this.


Partial Evaluation for Huge Documents

Description
Brief
Provide a partial evaluation API, with semantics: generate PDF from page $i$ to page $i+m$.
This feature makes sense only if intermediate results can be offloaded to disk storage.
Implementation: Page to Source Mapping
To implement partial evaluation, the compiler has to determine which part of the source should be evaluated from the page range.
It requires all evaluated expressions to have a pre-computed layout height. By searching a prefix-sum like structure, the compiler can find the part of the code to evaluate for the rendered range and the context.
Implementation: Offload Intermediate Results
By peeking into typst codebase a little, I found typst relies on a one-pass VM execution over all expressions with partially cached results. This requires all documents and all intermediate results to be stored in memory, which is prohibitive for huge documents.
I believe some of the intermediate results can be offloaded to disk storage. It would be better if each file can be compiled into a linkable format and later linked together. This can be done by leveraging module parameters.
Implementation: Missing Bits
Functions like measure require the layout engine to run first.
Perhaps allow VM to execute multiple times may solve this problem.
Use Case
In plugins like typst-preview, implementing this feature can save a lot of rendering time and memory. I believe this enhancement can benefit user experience.
I've been writing notes in Typst for several days. I produce 4 paragraphs per day, so in one year, I will have 4*365 paragraphs. To evaluate if typst can still handle my notes after one year, I wrote a python script to generate some content. The preview plugin gets stuck.

implementing this feature can save a lot of rendering time

I highly doubt that. You would quickly be limited by storage. Even a modern nvme SSD has vastly higher latency and bandwidth than RAM. Not to mention wear.

implementing this feature can save a lot of rendering time

I highly doubt that. You would quickly be limited by storage. Even a modern nvme SSD has vastly higher latency and bandwidth than RAM. Not to mention wear.

This feature request is not really about disk offloading 🤣.
I'm talking about not to generate the whole PDF in-memory when the user is just previewing some of the pages.
For your concern, by specifying pages to render, only a very small part of the intermediate results will be loaded into memory.

implementing this feature can save a lot of rendering time

I highly doubt that. You would quickly be limited by storage. Even a modern nvme SSD has vastly higher latency and bandwidth than RAM. Not to mention wear.

This feature request is not really about disk offloading 🤣.
I'm talking about not to generate the whole PDF in-memory when the user is just previewing some of the pages.
For your concern, by specifying pages to render, only a very small part of the intermediate results will be loaded into memory.

FYI typst-preview does not use PDF, and doesn't render all pages.
@Enter-tainer

But still, all the intermediate records are held in-memory, and used in computation.
Is that correct?

i feel like this issue is about typst-preview but not typst/typst.
@Y-jiji i suggest you using --timings to measure which part of your document is slow before we discussing potential optimizations.

I'm still adjusting the Lorem ipsum size so that typst compiler doesn't crash.
This is the bench I mentioned: https://github.com/Y-jiji/notizbuch. On the bench branch, in the root directory, run python code/generate-huge-chap.py will generate a Lorem ipsum project.

Properly layouting page n requires to layout all previous page at the least. But the later pages are also required to make introspection work. And the compiler architecture is not at all built to accommodate such a use case. All in all, this is simply not feasible to implement.

@Y-jiji I tried your script, here's a couple of findings:

it is creating ~10000 "tags", each of which corresponding to a 100 words/refs paragraph. This doesn't seem very consistent with the "4 paragraphs a day for a year", since with that pace you would need almost 7 years to write this many paragraphs. It would also contain 200000 references on average, which generally seem to be pretty costly;

reducing the amount of tags and tasks to 1000 makes it much more manageable, taking 15 seconds to compile. If I remove the references (i.e. make the paragraphs only consist of 100 words) this reduces even further to just 5 seconds, which goes to show how costly references are. Does this number seem more realistic/reasonable to you?

going from 1000 to 2000 tags/task (with references) increases the compilation time from 15 to 40 seconds, showing that compilation doesn't fully scale linearly;

your script doesn't ensure labels are unique, which gave me some errors a couple of times (it seems both tasks and tags create labels, but you don't ensure that they are disjoint).


Azure Databricks data plane REST API specifications

Where can the REST API specifications be found? I looked for them at:
https://github.com/Azure/azure-rest-api-specs/blob/main/specification/databricks
But it only contain the control plane (resource manager) specifications. It would be great to add links.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 25b5fd38-af0a-12e9-e8f8-3e57bcbc273f
Version Independent ID: f3a0cfcb-99eb-77c4-c888-4045d5d93947
Content: REST API (latest) - Azure Databricks
Content Source: databricks/dev-tools/api/latest/index.md
Service: azure-databricks
GitHub Login: @mssaperla
Microsoft Alias: saperla

I requested the specifications be added to the standard location in https://github.com/Azure/azure-rest-api-specs/issues/16688. Meanwhile, I see that at least one is available at:
https://docs.microsoft.com/en-us/azure/databricks/_static/api-refs/queries-dashboards-2.0-azure.yaml
Where would I find the others? For example Clusters API 2.0?

@cataggar we'll review your feedback and get back to you shortly. Thanks.

Thanks for your dedication to our documentation. We include links to the API ref content on several pages throughout the documentation, including the top-level page and https://learn.microsoft.com/en-us/azure/databricks/dev-tools/api/latest/. The Clusters API 2.0 page is at https://learn.microsoft.com/en-us/azure/databricks/dev-tools/api/latest/clusters.
This issue has been resolved and we are closing the issue.
#please-close

#please-close


Improve Challenge Link Readability

Closes #6018
As per the above, and my personal observations, the green color of links is very hard to read in context, so I've tried changing them to slate blue and making them bold:

This does make all links in the instructions bold and blue:

@FreeCodeCamp/issue-moderators and @QuincyLarson - what do you think?

I think this is a HUGE improvement and will make people use the links much more!

Just a suggestion. We can add an external link icon after every link like this:
.bonfire-instructions a::after {
   font-family: FontAwesome;
   content: " \f08e";
}

Will look like this:

I like the icon idea. It becomes more obvious that the link is something that the user should interact with.

Here are some examples:

Here is is without the bold:

I had to reduce the font size on the FontAwesome icon:
.bonfire-instructions a::after {
  font-size: 70%;
  font-family: FontAwesome;
  content: " \f08e";
}

I love this idea, and I definitely think the links should be higher-contrast. We'll need to check and see how the outbound link symbol works (we have a lot of links, such as those in the navbar, and we wouldn't want them all to adopt the symbol).
Let's see whether this is still a problem with the redesign, and if it is, we can make links bold.

@QuincyLarson I limited the link symbol to ONLY the bonfire-instructions class, so they shouldn't leak out.  I don't think other links need them, just the in-line ones.
Do you want me to roll this out, or do you want to take care of it in the redesign?

I am glad to see, how this issue is handled. It is indeed helpful now. @/all

@SaintPeter Kindlydo not limit it to Bonfire-instructions. Roll it on whole website. @QuincyLarson

@QuincyLarson are we going to merge this PR?

This PR currently does not have the link indicator on it.  I was looking for feedback before I pushed that out.

@SaintPeter Definitely add the link indicator. If this PR is scoped just to these links lets merge it in.

Final:

Ready for QA/Merge.

Verified and merged. Looks really awesome. Thanks, @SaintPeter!


Getting unauthorized error while trying to upload schemas to Iglu with Iglu Server Swagger UI

While there is no problem in the keygen sections, getting unauthorized error while trying to upload schemas to Iglu. Also, there is no problem while uploading with igluctl when this problem occurs and this means that using wrong apiKey case is eliminated.

The master api_key cannot be used to upload schemas.  It can only be used to generate api_keys that can then upload schemas.  Under the hood in igluctl this is what it is doing.


Can't click on anything if I close linear editor by clicking on canvas.

My proposed fix can be found here: #8402
1.
If I close the linear editor using escape things work normally, but if I close it by clicking on the canvas this happens:
(Clicking on the buttons does nothing)
https://github.com/user-attachments/assets/a17d23c3-dd6d-4fdb-913f-d14e325fb584
This is because in onPointerUpFromPointerDownHandler the selectionElement is set to null but this is overwritten when actionFinalize is called.
2.
Another problem is that in actionFinalize the selectedLinearElement is set to null but selectedElementIds remain as is. This is inconsistent, if you want to move points of the linear element around after closing the linear editor, you need to unselect and reselect it:
https://github.com/user-attachments/assets/b1864fd2-06ab-4f3b-80c0-05ccd1856c61
3.
Linear editor is not always opened using actionToggleLinearEditor
Additionally the actionToggleLinearEditor is not being used to toggle the linear editor when being opened with (ENTER,CTRL_OR_CMD) or CTRL_OR_CMD and double click. The actionToggleLinearElement's predicate function is also implemented (slightly differently) in both of these places:

if (isLinearElement(selectedElement)) {
  if (
    !this.state.editingLinearElement ||
    this.state.editingLinearElement.elementId !==
      selectedElements[0].id
  ) {
    this.store.shouldCaptureIncrement();
    if (!isElbowArrow(selectedElement)) {
      this.setState({
        editingLinearElement: new LinearElementEditor(
          selectedElement,
        ),
      });
    }
  }
}

if (selectedElements.length === 1 && isLinearElement(selectedElements[0])) {
   if (
     event[KEYS.CTRL_OR_CMD] &&
     (!this.state.editingLinearElement ||
       this.state.editingLinearElement.elementId !==
         selectedElements[0].id) &&
     !isElbowArrow(selectedElements[0])
   ) {
     this.store.shouldCaptureIncrement();
     this.setState({
       editingLinearElement: new LinearElementEditor(selectedElements[0]),
     });
     return;
   }

Both of these could simply be replaced by:
 this.actionManager.isActionEnabled(actionToggleLinearEditor) &&
  this.actionManager.executeAction(actionToggleLinearEditor, "keyboard");

Hi, thanks for the PR!
Strange we haven't caught (1) and (3) yet. (2) we're tracking and were going to fix ourselves soon!
Yes, the state update race conditions are annoying. Looking forward to moving away from React for state management at some point.


Adding support for SDP-K1.

Description

Adding support for SDP-K1.
Pull request type

[ ] Fix
[ ] Refactor
[x] Target update
[ ] Functionality change
[ ] Docs update
[ ] Test update
[ ] Breaking change

Reviewers

@maclobdell

@ARMmbed/mbed-os-tools
Hello, can this PR be approved? Are there any additional requirements?


[BUG] Setting Parameter rag_pipeline_feature_enabled: true is not effect in 2.12.0 when restarting the cluster

What is the bug?
In Opensearch 2.12.0, I have set the parameter plugins.ml_commons.rag_pipeline_feature_enabled value to true, like is:
PUT _cluster/settings
{
  "persistent": {
    "plugins.ml_commons.rag_pipeline_feature_enabled": true
  }
}

Then I created a RAG pipline:
PUT /_search/pipeline/rag_demo
{
  "response_processors": [
    {
      "retrieval_augmented_generation": {
        "tag": "rag_pipeline_demo",
        "description": "Demo pipeline Using Azure OpenAI Connector for RAG application",
        "model_id": "EzpM0I0BVbLa0eH3b3an",
        "context_field_list": ["name","desc"],
        "system_prompt": "You are a helpful assistant",
        "user_instructions": "Generate a concise and informative answer in less than 100 words for the given question"
      }
    }
  ]
}

But the setting plugins.ml_commons.rag_pipeline_feature_enabled does not take effect when restarting the cluster and will throw an exception:
[2024-03-01T16:54:59,436][WARN ][o.o.s.p.SearchPipelineService] [opensearch-cluster_manager] failed to update search pipelines
org.opensearch.OpenSearchParseException: Error updating pipeline with id [rag_demo]
        at org.opensearch.search.pipeline.SearchPipelineService.innerUpdatePipelines(SearchPipelineService.java:198) ~[opensearch-2.12.0.jar:2.12.0]
        at org.opensearch.search.pipeline.SearchPipelineService.applyClusterState(SearchPipelineService.java:156) [opensearch-2.12.0.jar:2.12.0]
        at org.opensearch.cluster.service.ClusterApplierService.callClusterStateAppliers(ClusterApplierService.java:608) [opensearch-2.12.0.jar:2.12.0]
        at org.opensearch.cluster.service.ClusterApplierService.callClusterStateAppliers(ClusterApplierService.java:596) [opensearch-2.12.0.jar:2.12.0]
        at org.opensearch.cluster.service.ClusterApplierService.applyChanges(ClusterApplierService.java:563) [opensearch-2.12.0.jar:2.12.0]
        at org.opensearch.cluster.service.ClusterApplierService.runTask(ClusterApplierService.java:486) [opensearch-2.12.0.jar:2.12.0]
        at org.opensearch.cluster.service.ClusterApplierService$UpdateTask.run(ClusterApplierService.java:188) [opensearch-2.12.0.jar:2.12.0]
        at org.opensearch.common.util.concurrent.ThreadContext$ContextPreservingRunnable.run(ThreadContext.java:854) [opensearch-2.12.0.jar:2.12.0]
        at org.opensearch.common.util.concurrent.PrioritizedOpenSearchThreadPoolExecutor$TieBreakingPrioritizedRunnable.runAndClean(PrioritizedOpenSearchThreadPoolExecutor.java:283) [opensearch-2.12.0.jar:2.12.0]
        at org.opensearch.common.util.concurrent.PrioritizedOpenSearchThreadPoolExecutor$TieBreakingPrioritizedRunnable.run(PrioritizedOpenSearchThreadPoolExecutor.java:246) [opensearch-2.12.0.jar:2.12.0]
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1144) [?:?]
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:642) [?:?]
        at java.base/java.lang.Thread.run(Thread.java:1583) [?:?]
Caused by: org.opensearch.ml.common.exception.MLException: plugins.ml_commons.rag_pipeline_feature_enabled is not enabled.
        at org.opensearch.searchpipelines.questionanswering.generative.GenerativeQAResponseProcessor$Factory.create(GenerativeQAResponseProcessor.java:324) ~[?:?]
        at org.opensearch.searchpipelines.questionanswering.generative.GenerativeQAResponseProcessor$Factory.create(GenerativeQAResponseProcessor.java:248) ~[?:?]
        at org.opensearch.search.pipeline.PipelineWithMetrics.readProcessors(PipelineWithMetrics.java:157) ~[opensearch-2.12.0.jar:2.12.0]
        at org.opensearch.search.pipeline.PipelineWithMetrics.create(PipelineWithMetrics.java:98) ~[opensearch-2.12.0.jar:2.12.0]
        at org.opensearch.search.pipeline.SearchPipelineService.innerUpdatePipelines(SearchPipelineService.java:180) ~[opensearch-2.12.0.jar:2.12.0]
        ... 12 more

I tried to reset it to true, but it didn't work.
Surprisingly, I first set it to false, then set it to true, and it can work again without throwing any exceptions.
PUT _cluster/settings
{
  "persistent": {
    "plugins.ml_commons.rag_pipeline_feature_enabled": false
  }
}
PUT _cluster/settings
{
  "persistent": {
    "plugins.ml_commons.rag_pipeline_feature_enabled": true
  }
}

How can one reproduce the bug?
Steps to reproduce the behavior:

Setting plugins.ml_commons.rag_pipeline_feature_enabled value to true
Restarting Opensearch cluster
View cluster startup logs
See error

What is the expected behavior?
Setting Parameter rag_pipeline_feature_enabled: true is effect when restarting the cluster
What is your host/environment?

OS: Linux CentOS 7.9
Version: 2.12.0
Plugins: ml_commons

Do you have any screenshots?
If applicable, add screenshots to help explain your problem.
Do you have any additional context?
Add any other context about the problem.

@HenryL27 could you please take a look at this issue?

Fixed in 2.13.


Add parameter objects for dependencies of public services

Issue #7465
This change introduces parameter objects for the service dependencies of all public services. This allows new dependencies to be added without breaking derived classes that call the base class constructor. For factories, these dependency objects are passed down into the services created by the factory. This means that we also don't break by adding parameters to the constructors called by the factory. This is the main reason for using this approach rather than just the service rewriting approach.
The pattern is not used for internal services since no code should be inheriting from these classes. However, the service re-writer code is still in place, which means that we have some flexibility to add dependencies to internal services even in patch releases where we are being very strict about binary compatibility.
Some work remains still to do:

Add With methods to all ...Dependencies objects to allow for service replacement
Add sugar for registration
Re-visit service scopes

OMG ! 😄

:shipit:


Constraints: Express rolling window constraints in constraints eDSL

Copying https://jira.jpl.nasa.gov/browse/AERIEQS-396.
Currently there are 8 identified flight rules that rely on this type of check (and likely will be more).

The use case is a flight rule that states a piece of hardware can only be powered on for 200 minutes across any given 8 hour span.
So the existing lessThan node could flag any occurrence when it is powered on for more than 200 minutes at once, but I also need to catch cases like if its powered on for 120 minutes, off for 60, and then back on for 120
Reported by Sloan: https://jpl.slack.com/archives/C0163E42UBF/p1666903380420049

Criticality is major in long term, but okay to delay until FY 24

This will be implemented in https://github.com/NASA-AMMOS/aerie/issues/830


implement UiWidgetListTable

UiWidgetListTable is to be implemented. This will be tough especially for web (GWT). We have to consider the following requirements:

Support for scrolling tbody vertically as needed and the entire table horizontally.
Ability to scale the size of the entire table with the available size (without javascript layout managers)
Abilitly for the user to sort by column (via clicking on column header)
Ability for the user to resize columns (with e-resize cursor on the fly by dragging the right border of the column)
Ability for the user to reorder columns with drag and drop via column header
Ability for inline editing
Ability for multiple selection with checkboxes
Keyboard support to step through the column headers and cells with cursor keys
Accessibility support via WAI-ARIA
For large datasets it may be required that we only render the visible part and update lazily while scrolling

Great progress after long fight with HTML(5) and CSS. Now working:

list table with column model
tbody scrollable
columns resizable
columns sortable
showing of sort order via CSS icon
content of columns is clipped when to narrow using text-overflow: ellipsis
setting min/max width for column honored by resizing

Still lots of things to do:

inline editing (should be quite easy to implement now)
hover text for headers and cells to show full text (useful if clipped), configurable via API
grouping of headers
tab/focus management for keyboard control
horizontal scrolling
tfooter support including summary functions
accessibility/aria support for screenreaders
special features (all cells editable, custom editor e.g. via Popup, custom styles for cells, etc.)


invalid $HOME dir used on OSX?

bash-3.2$ wget "https://github.com/aminya/setup-cpp/releases/download/v0.13.0/setup_cpp_mac"
--2022-04-24 18:26:41--  https://github.com/aminya/setup-cpp/releases/download/v0.13.0/setup_cpp_mac
Auflösen des Hostnamens github.com (github.com)… <IP_ADDRESS>
Verbindungsaufbau zu github.com (github.com)|<IP_ADDRESS>|:443 … verbunden.
HTTP-Anforderung gesendet, auf Antwort wird gewartet … 302 Found
Platz: https://objects.githubusercontent.com/github-production-release-asset-2e65be/406308261/bfdce48f-687f-480a-b19f-89ac66d15973?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20220424%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20220424T162641Z&X-Amz-Expires=300&X-Amz-Signature=52c1a545655cdff667b66f3669de059dfa95df4fea31f69aeb4f524b6e177e81&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=406308261&response-content-disposition=attachment%3B%20filename%3Dsetup_cpp_mac&response-content-type=application%2Foctet-stream [folgend]
--2022-04-24 18:26:41--  https://objects.githubusercontent.com/github-production-release-asset-2e65be/406308261/bfdce48f-687f-480a-b19f-89ac66d15973?X-Amz-Algorithm=AWS4-HMAC-SHA256&X-Amz-Credential=AKIAIWNJYAX4CSVEH53A%2F20220424%2Fus-east-1%2Fs3%2Faws4_request&X-Amz-Date=20220424T162641Z&X-Amz-Expires=300&X-Amz-Signature=52c1a545655cdff667b66f3669de059dfa95df4fea31f69aeb4f524b6e177e81&X-Amz-SignedHeaders=host&actor_id=0&key_id=0&repo_id=406308261&response-content-disposition=attachment%3B%20filename%3Dsetup_cpp_mac&response-content-type=application%2Foctet-stream
Auflösen des Hostnamens objects.githubusercontent.com (objects.githubusercontent.com)… <IP_ADDRESS>, <IP_ADDRESS>, <IP_ADDRESS>, ...
Verbindungsaufbau zu objects.githubusercontent.com (objects.githubusercontent.com)|<IP_ADDRESS>|:443 … verbunden.
HTTP-Anforderung gesendet, auf Antwort wird gewartet … 200 OK
Länge: 29593873 (28M) [application/octet-stream]
Wird in »setup_cpp_mac« gespeichert.

setup_cpp_mac                                  100%[=================================================================================================>]  28,22M  6,30MB/s    in 4,7s

2022-04-24 18:26:47 (6,05 MB/s) - »setup_cpp_mac« gespeichert [29593873/29593873]

bash-3.2$ chmod +x setup_cpp_mac
bash-3.2$ sudo ./setup_cpp_mac --compiler llvm-14.0.0
Password:
::group::Installing llvm 14.0.0
Installing llvm 14.0.0 x64 via direct downloading

::set-output name=version::14.0.0
Download and extract llvm 14.0.0
::debug::Downloading https://github.com/llvm/llvm-project/releases/download/llvmorg-14.0.0/clang+llvm-14.0.0-x86_64-apple-darwin.tar.xz
::debug::Destination /tmp/347b2dd2-f1ba-4a58-8494-c6be266b67a7
::debug::download complete
tar: could not chdir to '/home/clausklein/llvm'

Add /home/clausklein/llvm/bin to PATH
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Failed to add /home/clausklein/llvm/bin to the percistent PATH. You should add it manually.
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Failed to export environment variable LLVM_PATH=/home/clausklein/llvm. You should add it manually.
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Failed to export environment variable LD_LIBRARY_PATH=/home/clausklein/llvm/lib:. You should add it manually.
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Failed to export environment variable DYLD_LIBRARY_PATH=/home/clausklein/llvm/lib:. You should add it manually.
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Failed to export environment variable LDFLAGS=-L/home/clausklein/llvm/lib. You should add it manually.
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Failed to export environment variable CPPFLAGS=-I/home/clausklein/llvm/include. You should add it manually.
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Failed to export environment variable CC=/home/clausklein/llvm/bin/clang. You should add it manually.
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Failed to export environment variable CXX=/home/clausklein/llvm/bin/clang++. You should add it manually.
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Failed to export environment variable LIBRARY_PATH=/home/clausklein/llvm/lib. You should add it manually.
[command]/usr/bin/xcrun --sdk macosx --show-sdk-path
2022-04-24 18:30:48.649 xcodebuild[22971:13218588] Requested but did not find extension point with identifier Xcode.IDEKit.ExtensionSentinelHostApplications for extension Xcode.DebuggerFoundation.AppExtensionHosts.watchOS of plug-in com.apple.dt.IDEWatchSupportCore
2022-04-24 18:30:48.649 xcodebuild[22971:13218588] Requested but did not find extension point with identifier Xcode.IDEKit.ExtensionPointIdentifierToBundleIdentifier for extension Xcode.DebuggerFoundation.AppExtensionToBundleIdentifierMap.watchOS of plug-in com.apple.dt.IDEWatchSupportCore
2022-04-24 18:30:52.989 xcodebuild[22973:13218640] Requested but did not find extension point with identifier Xcode.IDEKit.ExtensionSentinelHostApplications for extension Xcode.DebuggerFoundation.AppExtensionHosts.watchOS of plug-in com.apple.dt.IDEWatchSupportCore
2022-04-24 18:30:52.989 xcodebuild[22973:13218640] Requested but did not find extension point with identifier Xcode.IDEKit.ExtensionPointIdentifierToBundleIdentifier for extension Xcode.DebuggerFoundation.AppExtensionToBundleIdentifierMap.watchOS of plug-in com.apple.dt.IDEWatchSupportCore
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.3.sdk
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Failed to export environment variable SDKROOT=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.3.sdk. You should add it manually.
::endgroup::
took 1 minute, 40 seconds
✅ llvm was installed successfully:
- The installation directory is /home/clausklein/llvm
- The binary directory is /home/clausklein/llvm/bin
setup_cpp finished
Run `source ~/.cpprc` or restart your shell to update the environment.
bash-3.2$ echo $HOME
/Users/clausklein
bash-3.2$

Without sudo, it works:
bash-3.2$ ./setup_cpp_mac --compiler llvm-14.0.0
::group::Installing llvm 14.0.0
Installing llvm 14.0.0 x64 via direct downloading

::set-output name=version::14.0.0
Download and extract llvm 14.0.0
::debug::Downloading https://github.com/llvm/llvm-project/releases/download/llvmorg-14.0.0/clang+llvm-14.0.0-x86_64-apple-darwin.tar.xz
::debug::Destination /var/folders/wb/ckvxxgls5db7qyhqq4y5_l1c0000gq/T/50300665-df0c-4162-94fa-0846a4b3d88e
::debug::download complete
Add /Users/clausklein/llvm/bin to PATH
Added export SOURCE_CPPRC=0 to /Users/clausklein/.cpprc

# source .cpprc if SOURCE_CPPRC is not set to 0
if [[ "$SOURCE_CPPRC" != 0 && -f "/Users/clausklein/.cpprc" ]]; then source "/Users/clausklein/.cpprc"; fi
 was added to /Users/clausklein/.profile

# source .cpprc if SOURCE_CPPRC is not set to 0
if [[ "$SOURCE_CPPRC" != 0 && -f "/Users/clausklein/.cpprc" ]]; then source "/Users/clausklein/.cpprc"; fi
 was added to /Users/clausklein/.bashrc
/Users/clausklein/llvm/bin was added to "/Users/clausklein/.cpprc"
LLVM_PATH="/Users/clausklein/llvm was added to "/Users/clausklein/.cpprc"
LD_LIBRARY_PATH="/Users/clausklein/llvm/lib: was added to "/Users/clausklein/.cpprc"
DYLD_LIBRARY_PATH="/Users/clausklein/llvm/lib: was added to "/Users/clausklein/.cpprc"
CPATH="/Users/clausklein/llvm/lib/clang/14.0.0/include was added to "/Users/clausklein/.cpprc"
LDFLAGS="-L/Users/clausklein/llvm/lib was added to "/Users/clausklein/.cpprc"
CPPFLAGS="-I/Users/clausklein/llvm/include was added to "/Users/clausklein/.cpprc"
CC="/Users/clausklein/llvm/bin/clang was added to "/Users/clausklein/.cpprc"
CXX="/Users/clausklein/llvm/bin/clang++ was added to "/Users/clausklein/.cpprc"
LIBRARY_PATH="/Users/clausklein/llvm/lib was added to "/Users/clausklein/.cpprc"
[command]/usr/bin/xcrun --sdk macosx --show-sdk-path
/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.3.sdk
SDKROOT="/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.3.sdk was added to "/Users/clausklein/.cpprc"
::endgroup::
took 2 minutes, 14 seconds
✅ llvm was installed successfully:
- The installation directory is /Users/clausklein/llvm
- The binary directory is /Users/clausklein/llvm/bin
setup_cpp finished
Run `source ~/.cpprc` or restart your shell to update the environment.
bash-3.2$

Is SUDO_USER defined on MacOS? If not, what is the alternative?
The code is here:
https://github.com/aminya/setup-cpp/blob/fcfb09bb91efe6a51b6084331b38a4a7fa01e924/src/utils/path/untildify.ts#L6

Oh, I think on MacOS we should use  join("/Users/", process.env.SUDO_USER, path) instead.
https://github.com/aminya/setup-cpp/blob/fcfb09bb91efe6a51b6084331b38a4a7fa01e924/src/utils/path/untildify.ts#L8

But please note too: there was no correct error handling implemented?
sudo ./setup_cpp_mac --compiler llvm-14.0.0
# . . .
Error: ENOENT: no such file or directory, open '/home/clausklein/.cpprc'
Failed to export environment variable SDKROOT=/Applications/Xcode.app/Contents/Developer/Platforms/MacOSX.platform/Developer/SDKs/MacOSX12.3.sdk. You should add it manually.
::endgroup::
took 1 minute, 40 seconds
✅ llvm was installed successfully:
- The installation directory is /home/clausklein/llvm
- The binary directory is /home/clausklein/llvm/bin
setup_cpp finished
Run `source ~/.cpprc` or restart your shell to update the environment.

It is implemented. It says "Failed to export environment variable ... You should add it manually."


Don't redefine asprintf when building for Android

Bionic includes a definition and implementation of asprintf. We should
not include our own on Android.
See 'include/stdio.h' and 'libc/stdio/stdio.cpp'
https://android.googlesource.com/platform/bionic/+/refs/heads/master/libc/include/stdio.h#288
https://android.googlesource.com/platform/bionic/+/refs/heads/master/libc/stdio/stdio.cpp#736

@swift-ci please test

/cc @buttaface FYI

That works for me too.

I should say that it works for me with caveats.
Digging through the macros. It's already defined if you have __USE_GNU enabled, or STDC_WANT_LIB_EXT2 set, so we shouldn't re-declare it in that case. @compnerd will likely have more comments.