Adding order arguments to search queries.

Is your proposal related to a problem?
I am working on ordering in wagtail grapple. I've had to do a bit of work in grapple to explicitly order queries. It took me a while to find I need to set order_by_relevance to false.  I found it really awkward that I needed to do more than just specify an explicit order on a query I provided as a basis for the search.
After some digging I figured out that order_by_relevance can't be inferred because Page.objects.all() is ordered by path by default.  That was after trying to order the result get_search_backend().search(...), which doesn't work because it doesn't return a queryset. It returns an implementation of BaseSearchResult, which makes sense given search support elasticsearch. Annotating all a queryset  through a proxy with the scores from ES would undo any performance benefit, so it all makes sense...
Passing a queryset into search is a way to pre-filter the results that will need to be scored and ranked for the search engine.  Which makes sense.
Describe the solution you'd like
I think it would be nice if there were an order argument in place of order_by_relevance that defaults to ['relevance']
We should allow any string and just check if the model has the field before adding it to the ultimate order_by an ignore or want on fields that aren't on the model.

Thanks for the suggestion @dopry! To summarise, what you're proposing is for:
MyModel.objects.search("term", order=["some_field", "other_field"])

to have equivalent functionality to:
MyModel.objects.order_by("some_field", "other_field").search("term", order_by_relevance=False)

Is that correct?
(If we do need a magic string value to pass to order indicate ordering by relevance - and I'm not sure that we do, since order=None would probably do the job - I'd suggest "__relevance__", to avoid the possibility of clashing with an actual field named relevance and keep consistency with Django's use of things like fields = "__all__" for ModelForms.)

@gasman you got it... although looking at
MyModel.objects.search("term", order=["some_field", "other_field"]) # order would default to __relevance__ in this scenario.

my concern is that it precludes .order_by(...) altogether. and would require changes to how ordering is setup in all code currently calling order_by_relevance=False.
Another option if it is feasible would be...
MyMode.objects.order_by("__relevance__", "some_field").search("term")

With this version there would be a synthetic relevance field that is only populated when searching and relevance would need to be explicitly requested by search. With this implementation code with order_by_relevance=false wouldn't not need to be changed and any calls to search that are implicitly expecting order by relevance will need to updated to explicitly request order by relevance.

I want to work on this issue.. Any updates on it?

@dopry My suggestion is for the order kwarg on .search to essentially act as a shorthand for inserting an .order_by clause and setting order_by_relevance=False. All existing behaviour when not passing an order kwarg would be preserved (namely, ordering by relevance unless order_by_relevance=False is passed, in which case it will respect the existing ordering on the queryset), so existing code would continue to work.
I don't think the MyModel.objects.order_by("__relevance__", "some_field").search("term") approach is feasible, because once we're inside the .search method, all we know is that the current queryset has an ordering defined on it - we don't know whether we got there from
MyModel.objects.order_by("some_field").search("term")

or from a straightforward
MyModel.objects.search("term")

where MyModel just happens to have "some_field" set up as its default ordering in Meta. In the latter case, we definitely don't want to depart from the current behaviour of ordering by relevance - which means the only appropriate behaviour (in the absence of any options passed to search to indicate otherwise) is to ignore the queryset's ordering, regardless of whether that's an explicit order_by or an implicit model-level one.
@Parthmadan-1 If @dopry is on board with this as the way forward, then this should be ready to pick up.

@gasman great, looking forward to contribute.

@gasman sure. the implicit model level search is what I think lead to this in the first place. My only concern is that it may not fix my use case where I'm not actually calling search. I'm using grapple and the paginated_results function that accepts a queryset and some other arguments. I'm extending the behavior of grapple for new types with additional filters and ordering options that are applied to the query set before passing on the to paginated results method.  I think you approach will work, but it will also required updating wagtail-grapple's paginated results function to better handle ordering.

@dopry It looks to me like this proposed change will simplify the implementation of resolve_paginated_queryset here, since the semantics of passing order or not, to either apply the explicit order or use the "natural" default, more or less match up. In pseudocode:
if search_term:
    # this handles both the order=None (order by relevance) and order=['some_field'] case
    qs = qs.search(search_term, order=order)
elif order:
    qs = qs.order_by(order)
else:
    pass  # keep the queryset's built-in ordering

Either way, whether it's an exact fit or not, I think we've established enough potential usefulness here that we're good to go ahead with this enhancement.

@gasman I agree. I also think it would be cleaner in the search documentation. replacing
>>> EventPage.objects.order_by('date').search("Event", order_by_relevance=False)

with
>>> EventPage.objects.search("Event", order=['date'])

Is this issue still open??
I would like to work on it.

@Gua00va Please see the discussions above - @Parthmadan-1 has already expressed an interest in this, so best to give them a chance to work on this first.

@Parthmadan-1 Do need any additional help to get started working on this issue?


Support older Yubikeys with TOTP

See: https://github.com/aripalo/ykmangoath/pull/6

Released in v0.14.0.


Nested endpoints

I have the following API:
type API
    =   "accounts"
        :> QueryParam "asset" Asset
        :> QueryParam "cursor" Text
        :> QueryParam "limit" Natural
        :> Get '[HalJson] (Records Account)
    :<|> "accounts" :> Capture "account_id" Address :> Get '[HalJson] Account
    :<|>
        "accounts" :> Capture "account_id" Address :> "transactions"
        :> QueryParam "cursor" Text :> QueryParam "limit" Natural
        :> Get '[HalJson] (Records Transaction)

And servant-cli give me this:
$ stack run -- horizon --help
Usage: horizon COMPONENT
Available options:
  -h,--help                Show this help text
Path components:
  accounts

$ stack run -- horizon accounts --help
accounts
Usage: horizon accounts [<account_id> | [--asset ASSET] [--cursor TEXT]
                          [--limit NATURAL]]
Available options:
  <account_id>             This account’s public key encoded in a base32 string...
  --asset ASSET            An issued asset represented as...
  --cursor TEXT            From official docs: A number that points to a...
  --limit NATURAL          The maximum number of records returned. The limit can...
  -h,--help                Show this help text

How to call the accounts/:account_id/transactions endpoint?

Hi, hope you were able to figure this out!  Sorry, have been inactive for a while. IIRC (and it's been a while), you would call with horizon accounts myaccount123 transactions.

It's non-intuitive for CLI. And it needs to be documented in --help.


Add support for DiffMatchPatch algorithm with Rope

Incorporate a Rope version of https://github.com/google/diff-match-patch for providing efficient diffing and patching of text.

Alternative is to just use Myers diff - https://blog.jcoglan.com/2017/02/12/the-myers-diff-algorithm-part-1/


XSD for Policy Report Outputs

Is there a xsd schema for the output xml policy reports that export when I run the below command. I found this link but it isn't the schema for the policy check: https://raw.githubusercontent.com/MediaArea/MediaAreaXml/master/mediaconch.xsd
mediaconch -p '/Users/jsciame/Downloads/ContributionSpec.xml' /Users/jsciame/Downloads/MezzVideoDVSIssue.mov  --Mediaconch -fx > ~/Downloads/reportmax2.xml
<MediaConch xmlns="https://mediaarea.net/mediaconch" xmlns:mmt="https://mediaarea.net/micromediatrace" xmlns:mi="https://mediaarea.net/mediainfo" version="0.3">
  <media ref="/Users/jsciame/Downloads/MezzVideoDVSIssue.mov">
    <policy name="Contribution Spec" type="and" rules_run="6" fail_count="0" pass_count="6" outcome="pass">
      <rule name="PCM" value="Format" tracktype="Audio" occurrence="*" operator="=" xpath="mi:MediaInfo/mi:track[@type='Audio'][*]/mi:Format='PCM'" outcome="pass"/>
      <rule name="Height" value="Height" tracktype="Video" occurrence="1" operator="=" xpath="mi:MediaInfo/mi:track[@type='Video'][1]/mi:Height='1080'" outcome="pass"/>
      <rule name="Width" value="Width" tracktype="Video" occurrence="1" operator="=" xpath="mi:MediaInfo/mi:track[@type='Video'][1]/mi:Width='1920'" outcome="pass"/>
      <rule name="Chrome Sub Sampling" value="ChromaSubsampling" tracktype="Video" occurrence="1" operator="=" xpath="mi:MediaInfo/mi:track[@type='Video'][1]/mi:ChromaSubsampling='4:2:2'" outcome="pass"/>
      <policy name="Codec" type="or" rules_run="2" fail_count="1" pass_count="1" outcome="pass">
        <rule name="JPEG2000" value="Format" tracktype="Video" occurrence="1" operator="=" xpath="mi:MediaInfo/mi:track[@type='Video'][1]/mi:Format='JPEG 2000'" actual="ProRes" outcome="fail"/>
        <rule name="ProRes" value="Format" tracktype="Video" occurrence="1" operator="=" xpath="mi:MediaInfo/mi:track[@type='Video'][1]/mi:Format='ProRes'" outcome="pass"/>
      </policy>
      <policy name="Frame Rate" type="or" rules_run="3" fail_count="2" pass_count="1" outcome="pass">
        <policy name="23.98" type="and" rules_run="2" fail_count="0" pass_count="2" outcome="pass">
          <rule name="23.98" value="FrameRate" tracktype="Video" occurrence="*" operator="=" xpath="mi:MediaInfo/mi:track[@type='Video'][*]/mi:FrameRate='23.976'" outcome="pass"/>
          <rule name="Scan Type" value="ScanType" tracktype="Video" occurrence="1" operator="=" xpath="mi:MediaInfo/mi:track[@type='Video'][1]/mi:ScanType='Progressive'" outcome="pass"/>
        </policy>
        <policy name="29.97" type="and" rules_run="2" fail_count="1" pass_count="1" outcome="fail">
          <rule name="29.97" value="FrameRate" tracktype="Video" occurrence="*" operator="=" xpath="mi:MediaInfo/mi:track[@type='Video'][*]/mi:FrameRate='29.970'" actual="23.976" outcome="fail"/>
          <policy name="Scan Type" type="or" rules_run="2" fail_count="1" pass_count="1" outcome="pass">
            <rule name="Scan Type_Progressive" value="ScanType" tracktype="Video" occurrence="1" operator="=" xpath="mi:MediaInfo/mi:track[@type='Video'][1]/mi:ScanType='Progressive'" outcome="pass"/>
            <rule name="Scan Type_Interlaced" value="ScanType" tracktype="Video" occurrence="1" operator="=" xpath="mi:MediaInfo/mi:track[@type='Video'][1]/mi:ScanType='Interlaced'" actual="Progressive" outcome="fail"/>
          </policy>
        </policy>
        <policy name="25" type="and" rules_run="2" fail_count="1" pass_count="1" outcome="fail">
          <rule name="25.00" value="FrameRate" tracktype="Video" occurrence="*" operator="=" xpath="mi:MediaInfo/mi:track[@type='Video'][*]/mi:FrameRate='25.000'" actual="23.976" outcome="fail"/>
          <policy name="Scan Type" type="or" rules_run="2" fail_count="1" pass_count="1" outcome="pass">
            <rule name="Scan Type_Progressive" value="ScanType" tracktype="Video" occurrence="1" operator="=" xpath="mi:MediaInfo/mi:track[@type='Video'][1]/mi:ScanType='Progressive'" outcome="pass"/>
            <rule name="Scan Type_Interlaced" value="ScanType" tracktype="Video" occurrence="1" operator="=" xpath="mi:MediaInfo/mi:track[@type='Video'][1]/mi:ScanType='Interlaced'" actual="Progressive" outcome="fail"/>
          </policy>
        </policy>
      </policy>
    </policy>
  </media>
</MediaConch>```

We are looking to bind this in Python using pyxb for parsing.

The MediaConch (input) policy XSD is mediapolicy.xsd. Tested and works fine from my point of view. We need to provide the link in the XML.
The MediaConch (output) result XSD is expected to be mediaconch.xsd,
MediaConch result XSD works well for the implementation checks (which is implemented for MKV and FFV1 only due to the priority of the sponsors).
MediaConch result  XSD does not work for the policy checks, the policy result is well defined in the XSD (policyChecks element defined) but the output XML has an undefined policy element.
Looks like that at some point in the development we updated both the XSD and the implementation checker results but we forgot to update the policy checker results, which leads to an incoherency. cc @dericed.
We definitely need to fix that, in my opinion the output should match the XSD instead of changing the XSD for matching the current output but the fix will break the compatibility with third party scripts, we'll need a transition period.

Hotfix for XSD validation would be to have the XSD compatible with the current output, and comment it as "legacy".

closed?

Closing, knowing that at some point we should (or not?) move to the policyChecks  elements, when we have to break compatibility for something else...


Develop

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

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

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
554

Relevant Lines:
797

💛  - Coveralls


UAVCAN footprint reduction

@LorenzMeier @davids5 @bendyer @mhkabir
This optimization feels like cheating.
ROM reduction (I swear I didn't touch the compiler flags): 980204 - 967772 = 12 K.
RAM reduction: it's complicated.
nsh> uavcan status
uavcan: application not running
nsh> free
             total       used       free    largest
Mem:        229760     147360      82400      78800
nsh> uavcan start
uavcan: Node ID 1, bitrate 1000000
uavcan: SW version vcs_commit: 0x588a7afa
uavcan: sensor bridge 'gnss' init ok
uavcan: sensor bridge 'mag' init ok
uavcan: sensor bridge 'baro' init ok
nsh> free
             total       used       free    largest
Mem:        229760     158304      71456      66544
nsh> uavcan start fw
uavcan fw srv: base: couldn't open /etc/uavcan/fw
nsh> free
             total       used       free    largest
Mem:        229760     170272      59488      52064

Connecting Zubax GNSS, which requires the driver to sponge more memory, yet we can see that  fragmentation does not increase (pay attention to the field largest):
nsh> uavcan_mag adding channel 124...
uavcan_mag channel 124 class instance 1 ok
uavcan: GNSS receiver node ID: 124
free
             total       used       free    largest
Mem:        229760     172112      57648      52064

Connecting another node (px4cannode-v1) with an outdated firmware - the driver initiates the update procedure; at the end of it we see this:
nsh> free
             total       used       free    largest
Mem:        229760     173840      55920      52064

In this particular test, worst case memory use was 82400 - 55920 = 26K.
What happened?

The driver used to maintain multiple (3 exactly) independent block pools for different tasks. This approach is wildly inefficient. Now the driver uses one huge allocator that is thread-safe and therefore can serve everything.
The old allocator used to allocate memory from an internal pool implemented as a class field. Therefore the driver had to allocate huge continuous chunks of memory, causing fragmentation and sadness. The new allocator uses a linked list of chunks that grows dynamically as memory needs increase, taking memory from the heap in 64-byte chunks (mm overhead included). Once a chunk has been allocated, the allocator keeps it for future use, therefore the driver doesn't malloc()/free() constantly (actually it never free()s anything on its own).
The new allocator is limited in growth, and the limit can be runtime adjusted. The limit is currently 500 blocks, or 32000 bytes.
It is possible to shrink() memory reserved by the allocator. The driver will penalize you for that with higher CPU usage for the following few allocations, because it will have to malloc() again. Possible usage scenario: shrinking before arming to provide more memory for log buffers.
The transport layer and some other stuff in the library has been changed also to avoid static allocation in favor of block allocation. It's complicated and I don't want to go into details here, the old solution was really not very efficient.

Stay tuned, work is still in progress.

https://github.com/UAVCAN/libuavcan/pull/73

According to https://github.com/UAVCAN/libuavcan/issues/33, RAM reduction in the case above is about 42K - 26K = 16K.

CPU load during firmware update at 1 Mbps:
 186 uavcan                       8269 10.105  1508/ 1792 240 (240)  w:sem
 197 uavcan fw srv                5471  6.259  3556/ 5992 120 (120)  w:sem

Testing with firmware update and some nodes connected shows that peak memory usage does not exceed 80 blocks, so the limit of 500 blocks seems enough.

Nice!

@bendyer @mhkabir Does anybody want to give it a try? Seems to work flawlessly for me so far, although it still leaks about ~300 bytes of memory per restart (@davids5 master does too). The problem is certainly not with libuavcan (it's unit tested for leaks), could be either the application, some part of PX4 or NuttX. I'm not sure how important this issue is, considering that the driver is not supposed to be stopped during normal operation.

Seems like I've fixed the leaks. It appears to be ready for merge now.
@bendyer @mhkabir so would you like to test it, or shall we merge it as is?

@pavel-kirienko A test flight with this change would be great.

Toggling to re-run hans-ci.

@pavel-kirienko @bendyer Is this flight tested? I'll rebase it on master to make it easier to handle.

Not yet. Some hardware needed for the test is still stuck in post. I'll update you once it's tested.

Good to go for a test flight.

I'm going to test fly this early next week. Should resolve https://github.com/PX4/Firmware/issues/3102.

This has been test flown on an Iris with UAVCAN ESC without issues.

I test flew this today as well. System loading / memory looks good! :)

Will someone volunteer to merge this? I have already updated this PR to the yesterday's master.


Blue LED blinks rapidly

hi......im back with another question ;)
what does it mean, if theres no Exploit WLAN after a fresh reflash.......
instead the blue led just goes on and off instantly......
Could it be the case that i have to wait longer between "ResetDefaultConfig:OpenNetwork" and the 32u4 flash?
I have 2 Cactus Dongles and did exactly the same with both........one works the 2nd doesnt!

Corey?

The blue LED blinks fast during the flashing process.
On a fresh unit there is a roughly 90 seconds wait upon power on where the SPIFFS file system is formatted.
Then the rest of the program happens, load config, attack, loop, etc...
Never try to test the ESP before flashing the proper sketch to the Atmega32u4 as the 32u4 sketch turns on an enable pin to power up the ESP.  This is a common mistake people make, asking questions about the ESP not working when the ESP is not powered on.
Open a serial connection with the device, I believe I added feedback when you send commands.
What part of the process are you stuck on this time?

Solved?  If so please close.

Not solved.....
I've tried wifiducky and when it did not work (compiler constantly reports errors) I wanted to switch back to esploit .... I first flashed the programmer and then tried to prepare with esptool erase_flash, was not ........ something with timeout and header ...... So I han with the windows-tool on 0x00000 the 0x00000.bin and 0x010000 the 0x010000.bin flashed and then the ofw on 0x00000 .......
so far so good......
if I try now to recompile the esploit.bin with German keyboard I constantly get errors ....... screenshot is coming!

You have modified the code, I do not support these situations.
Esplora.h?
Keyboard.h?
Hid.h?
Of course there are errors.
Why don't you try to compile the actual source?
The ESP is not a "keyboard", the 32u4 is the keyboard portion.  This is why it does not compile.

ok.....
i have downloaded a new esploitv2 to my desktop.....
then i reflashed this bin file here --->

next step was to open arduino ide to customize the ESP_SKETCH, right?
i did so and here are my used settings --->

after i clicked on "compile/export" it gave me following error --->

hope you can help, because i have no idea, i reinstalled the ide as well and i dont know what i can do ^^

How did you customize the ESP sketch and why?
The keyboard portion is the 32u4 sketch.  You need to customize the keyboard library globally or redefine the include and copy the keyboard library to the 32u4 sketch folder and modify it there.

You should just flash the precompiled ESP binary from here.
http://esploitv2.exploit.agency/releases

....and again, thanks alot mate ^^


Publish BIRT-related binaries

Several binaries cannot be found in maven repos, so we will have to publish them ourselves. I believe the EPL allows this.

Is there a list?

The dependencies are listed in #79


Modify /head test to fail with function based titleTemplate

Ref #1886

I updated the test an fixed it, thank you @homerjam
See https://github.com/nuxt/nuxt.js/commit/bf72a35634e084c08be6126a4281520caf259288


Refactor predictkube scaler config

Refactor predictkube scaler config
Checklist

[x] I have verified that my change is according to the deprecations & breaking changes policy
[x] Tests have been added
[x] Commits are signed with Developer Certificate of Origin (DCO - learn more)

Fixes #

Relates to #5797

/run-e2e predictkube


Vose Alia's Via Recursion

📥 Pull Request
issue :  #1186
Description
The Recursive Vose's Alias Method is an efficient algorithm designed for sampling from discrete probability distributions. It features a preprocessing time of O(n), allowing for quick setup, while enabling constant-time sampling at O(1), making it ideal for applications requiring rapid random sampling. The method effectively handles non-uniform distributions by categorizing probabilities into small and large lists, utilizing a recursive approach to build an alias table that accurately represents the distribution. Despite its recursive nature, the algorithm remains straightforward to implement and understand. Its versatility makes it applicable in various fields, including computer graphics, machine learning, and simulations, where fast and reliable sampling is essential. Overall, the Recursive Vose's Alias Method combines speed and flexibility, making it a valuable tool for efficient random sampling in discrete settings.
Fixes #1186
Type of change

[✅ ] New feature (non-breaking change which adds functionality)
[✅ ] Documentation update

Checklist:

[✅] My code follows the style guidelines of this project
[✅ ] I have performed a self-review of my code
[✅ ] I have commented my code, particularly in hard-to-understand areas
[✅ ] I have added tests that prove my fix is effective or that my feature works
[ ✅] New and existing unit tests pass locally with my changes
[✅ ] Any dependent changes have been merged and published in downstream modules

@ajay-dhangar , can you please check now ,,, i have updated the Description as the SEO Point of view


AVRO-3700: Publish SBOM artifacts

What is the purpose of the change
This PR aims to publish SBOM artifacts.
Here is an article to give some context.

https://www.activestate.com/blog/why-the-us-government-is-mandating-software-bill-of-materials-sbom/

Software Bill of Materials (SBOM) are additional artifacts containing the aggregate of all direct and transitive dependencies of a project. The US Government (based on NIST recommendations) currently accepts only the three most popular SBOM standards as valid, namely: CycloneDX, Software Identification (SWID) tag, Software Package Data Exchange® (SPDX).
This PR uses CycloneDX maven plugin, a lightweight software bill of materials (SBOM) standard designed for use in application security contexts and supply chain component analysis.
Verifying this change
Manually verify with the following procedure. avro-1.12.0-SNAPSHOT.jar will have avro-1.12.0-SNAPSHOT-cyclonedx.xml and avro-1.12.0-SNAPSHOT-cyclonedx.json BOM files.
$ mvn install -DskipTests
...
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  02:04 min
[INFO] Finished at: 2023-01-05T10:15:49-08:00
[INFO] ------------------------------------------------------------------------

$ ls -al ~/.m2/repository/org/apache/avro/avro/1.12.0-SNAPSHOT/
total 2184
drwxr-xr-x  9 dongjoon  staff     288 Jan  5 10:14 .
drwxr-xr-x  5 dongjoon  staff     160 Jan  5 10:14 ..
-rw-r--r--  1 dongjoon  staff     302 Jan  5 10:14 _remote.repositories
-rw-r--r--  1 dongjoon  staff   17421 Jan  5 10:14 avro-1.12.0-SNAPSHOT-cyclonedx.json
-rw-r--r--  1 dongjoon  staff   15178 Jan  5 10:14 avro-1.12.0-SNAPSHOT-cyclonedx.xml
-rw-r--r--  1 dongjoon  staff  443402 Jan  5 10:14 avro-1.12.0-SNAPSHOT-tests.jar
-rw-r--r--  1 dongjoon  staff  612699 Jan  5 10:14 avro-1.12.0-SNAPSHOT.jar
-rw-r--r--  1 dongjoon  staff    9510 Jan  5 10:03 avro-1.12.0-SNAPSHOT.pom
-rw-r--r--  1 dongjoon  staff    1329 Jan  5 10:14 maven-metadata-local.xml

Documentation
This is a build-only change.

Could you review this, @dkulp , @iemejia , @martin-g ?

Thank you, @martin-g and @iemejia .

One question @dongjoon-hyun I am kind of new in the SBOM world but looking around it seems like there are like 3 big standards, any reason to choose the Cyclone one over SPDX (which seems to be the one being pushed by the Linux Foundation)? I am ok with merging this as it is, just curious. Better to have one that none :)
I am also wondering what other Apache projects use. Just from a quick look it seems not even Log4j with all the mess of the last year is publishing their SBOM and there are not recommendations yet from the security group at the ASF

Yes, among those three standards, CycloneDX and SPDX formats are proper for us and Maven plugins exist in the same way.

https://github.com/CycloneDX/cyclonedx-maven-plugin
https://github.com/spdx/spdx-maven-plugin

Although this PR delivers CycloneDX-style BOM files first, we may want to add SDPX-style BOM files additionally later. They are not exclusive. The reason why I choose CycloneDX is simpler and focuses on mostly for application security and supply-chain component analysis, @iemejia . This fits my needs first.

For the second question, I also searched some references in ASF foundation, but I could not find. So, I'm proposing and leading in the following way, @iemejia .

ORC-1342: Publish SBOM artifacts (Merged, https://github.com/apache/orc/pull/1353)
AVRO-3700: Publish SBOM artifacts (Approved, This PR, https://github.com/apache/avro/pull/2046)
PARQUET-2224: Publish SBOM artifacts (Open, https://github.com/apache/parquet-mr/pull/1017)
SPARK-41893: Publish SBOM artifacts (Merged, https://github.com/apache/spark/pull/39401)
FLINK-30578: Publish SBOM artifacts (Open, https://github.com/apache/flink/pull/21606)


Modifier -> SwiftUI -> Technology-specific views: activityBackgroundTint(_:)

Doc: https://developer.apple.com/documentation/swiftui/view/activitybackgroundtint(_:)

[ ] Swift implementation
[ ] Elixir implementation

We won't support this according to the team


Channel List Screen UI

Make the UI of ChannelListScreen similar to this

Verified this issue on dev app v2.0.70(436)


[WIP] Prepare release notes for v1.5.6

Wanted to open this to get the convo started on a new 1.5.x release as seems like there's enough to justify a new patch. Noticed @dmcgowan has opened all of the previous patch bump PRs so feel free to steal my branch and re-open 😋
There's a couple outstanding PRs that might be nice to wait for.

https://github.com/containerd/containerd/pull/6024
https://github.com/containerd/containerd/pull/5981
https://github.com/containerd/containerd/pull/5104

cc @kevpar

Did a test generation of the release notes
https://gist.github.com/dmcgowan/1d891f8aa4d267ca178c9fe423e7e53d
Can you also rebase the prepare commit onto the latest from release/1.5
A couple comments on the notes...

Just mentioning hcsshim shim is updated is redundant to the dependencies update at the bottom, mention why it is updated or exclude it
Worth adding mention of https://github.com/containerd/containerd/pull/5982

Thanks for taking care of writing the notes!

Looks like we also need to get a mailmap entry for @wllenyj (if could provide your real name for future sign offs)

@dmcgowan Re: rebasing, sure! And seemed like on the others the notable updates descriptions were a bit short so tried to keep it the same; I can give a small blurb on the reason for the update and I'll also add the clone3 addition as mentioned. Seems like anytime runc is bumped it's present in the notable updates and the small blurb at the beginning as the bumps are probably general housekeeping (unless the bump was for a cve which doesn't seem to be the case this time around).

Yeah, most recently runc has been bumped mostly from related CVE. Many of those updates didn't even require a dependency update for runc, we just happen to include for convenience in the packaging. We try to avoid just updating
versions in our stable branches for no reason, so usually there is at least a one liner justification. I don't believe we were mentioning every bump runc previously though

@dmcgowan updated! The CGroupsV2 tests seem to be my sworn enemy


Playground: Preset Passed Via Search Param in URL

Closes #365

Got to a decent point, required some kinda wild useEffect stuff to keep the url + atom in sync. Jotai does provide a atomWithHash, but unfortunately doesn't support Next 13 / app router
Also found a bug where if you select a new framework while the current one is loading it will show the new files, but use the previous start command.
would appreciate any feedback on what I did so far. Thanks!

through some git stuff I dont understand the first pr got closed but at least now its synced with the the current state of the playground branch

address the comments, if all looks good I think we can look to merge @GuiBibeau thanks!


Coverage broken on master

Might be releated to the string of commits here: https://github.com/babel/generator-babel-boilerplate/commit/62cbe1884bc1d6661303ff756281a50bd050643a, but I'm not sure. Investigation underway!

This worked on a recent commit before the switch to Yeoman, e49f661954c30. I'm still investigating the cause...

This is caused by overriding Istanbul's require wrap. The order needs to go Babel => Istanbul, but the refactor in https://github.com/babel/generator-babel-boilerplate/commit/62cbe1884bc1d6661303ff756281a50bd050643a inadvertently made it go Babel => Istanbul => Babel.

I'll likely need to make a PR to fix this on Parsley as well.
//cc @marcandre
: P

Copy+pasting the old Gulpfile into the new fixes the problem, so it's def. something in the Gulpfile

The issue is described in this issue: https://github.com/douglasduteil/isparta/issues/45
Solution is here: https://github.com/douglasduteil/isparta/issues/45#issuecomment-134477558

Done!


elmPackages.*: update to latest versions

Motivation for this change
Intention is to update elm-language-server to 1.4.6. Executed update script.
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

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

Notify maintainers
cc @turboMaCk @Razzeee @BrianHicks

cc @domenkozar I think this can be merged.


Refactoring of AbstractSchema::getColumnPhpType() method

Q
A

Is bugfix?
❌

New feature?
❌

Breaks BC?
❌

Fixed issues
-

👍


Define Logical Source in RML-Core instead of RML-IO

Currently the concept of a Logical Source is defined and described in the RML-IO spec.
However, there are many mentions to logical source in definitions of RML-Core constructs. Not having Logical Source defined in core makes it difficult to describe these constructs properly.
Proposal:
Describe Logical Source in the abstract sense in core. Describe the notion of iterations and reference formulations here, in an abstract sense, such that these concepts can be used to properly define the relevant core concepts.
Note: Some discussion around this topic has also taken place in the context of https://github.com/kg-construct/rml-lv/issues/14

AGREEMENT 03/07: @pmaria will create a suggestion


extension: fixup ExtensionContents serialization

Ugh, a couple of new lints appeared due to new versions of Rust.
Do you mind fixing them in this PR? (if you do I can do that in a separate one and then we rebase this PR on top of the other).
Thanks for your PR! :wave:

Thanks for your work! I'll get to releasing a patch version soon...

Sorry for the delay but I've released version 0.2.6 which contains this fix: https://crates.io/crates/ssh-agent-lib/versions


Wrong number of padding-spaces in article about padding-function

Describe your problem and how to reproduce it:
On "3. String padding" there in the line of ".padStart(10)", there are 10 spaces but there should only be 9. In this scenario this was very misleading.
Add a Link to the page with the problem:
https://medium.freecodecamp.org/here-are-examples-of-everything-new-in-ecmascript-2016-2017-and-2018-d52fa3b5a70e
Recommended fix, suggestions (how would you update it?):
Correct the number of spaces.

Other article on /news that needs attention @raisedadead @AbbeyRenn

@Ieahleen @AbbeyRenn I edited the article to remove the extra space.  You may have to refresh the page to see the change.
I am closing this as resolved.


resource custom offhours schedules

refactored offhours support based on faisal's pull request (#150), new features

resource tag values can specify custom schedules
weekend only support
record opt outs
record parse errors
remove skew as config option
additional documentation

closes #13
closes #180


[treeview] style tab content

We need to have a tab on the website for Style content for the Tree view component.

Notion document in progress: https://www.notion.so/carbondesignsystem/Style-6ffd3ec0a2a44c34aee3b876ca118ea5

Hey team! Please add your planning poker estimate with ZenHub @laurenmrice @thyhmdo


Fix findbugs issue.

Related to:

https://github.com/wso2/product-is/issues/19928

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/8169697075

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/8169697075
Status: failure

PR builder is not support for this repo.


Take down of single orientdb instance in 3 node cluster results in read operations failing until timeout

OrientDB Server v2.1.13 (build 2.1.x@r9bc1a54a4a62c4de555fc5360357f446f8d2bc84; 2016-03-14 17:00:05+0000)
We have a cluster of 3 orientdb servers with a read quorum of 1 and write quorum of 2. If we power off one of the machines all the read requests fail with the following exceptions, until the heartbeat timeout happens.
Is this the expected behavior?
Is the entire cluster really expected to be un-useable for the duration of the timeout (which is 5mins in this case)?
Shouldn't the read operations keep going through since the read quorum is 1 and there is more than 1 node in the cluster?
These logs are from the device with ip <IP_ADDRESS>, the device that we took down was <IP_ADDRESS>
2016-03-31 17:09:45:859 WARNI [<IP_ADDRESS>]:2434 [orientdb] [3.5.3] No response for 120000 ms. InvocationFuture{invocation=Invocation{ serviceName='hz:impl:mapService', op=GetOperation{orientdb}, partitionId=19, replicaIndex=0, tryCount=250, tryPauseMillis=500, invokeCount=1, callTimeout=60000, target=Address[<IP_ADDRESS>]:2434, backupsExpected=0, backupsCompleted=0}, response=null, done=false} [Invocation]
2016-03-31 17:09:45:859 WARNI [<IP_ADDRESS>]:2434 [orientdb] [3.5.3] Asking if operation execution has been started: Invocation{ serviceName='hz:impl:mapService', op=GetOperation{orientdb}, partitionId=19, replicaIndex=0, tryCount=250, tryPauseMillis=500, invokeCount=1, callTimeout=60000, target=Address[<IP_ADDRESS>]:2434, backupsExpected=0, backupsCompleted=0} [Invocation][<IP_ADDRESS>]:2434 [orientdb] [3.5.3] While asking 'is-executing': Invocation{ serviceName='hz:impl:mapService', op=GetOperation{orientdb}, partitionId=19, replicaIndex=0, tryCount=250, tryPauseMillis=500, invokeCount=1, callTimeout=60000, target=Address[<IP_ADDRESS>]:2434, backupsExpected=0, backupsCompleted=0}
java.util.concurrent.TimeoutException: Call Invocation{ serviceName='hz:impl:mapService', op=com.hazelcast.spi.impl.operationservice.impl.operations.IsStillExecutingOperation{serviceName='hz:impl:mapService', partitionId=-1, callId=11468, invocationTime=1459469391551, waitTimeout=-1, callTimeout=5000}, partitionId=-1, replicaIndex=0, tryCount=0, tryPauseMillis=0, invokeCount=1, callTimeout=5000, target=Ad
dress[<IP_ADDRESS>]:2434, backupsExpected=0, backupsCompleted=0} encountered a timeout
at com.hazelcast.spi.impl.operationservice.impl.InvocationFuture.resolveApplicationResponse(InvocationFuture.java:366)
at com.hazelcast.spi.impl.operationservice.impl.InvocationFuture.resolveApplicationResponseOrThrowException(InvocationFuture.java:334)
at com.hazelcast.spi.impl.operationservice.impl.InvocationFuture.get(InvocationFuture.java:225)
at com.hazelcast.spi.impl.operationservice.impl.IsStillRunningService.isOperationExecuting(IsStillRunningService.java:85)        at com.hazelcast.spi.impl.operationservice.impl.InvocationFuture.waitForResponse(InvocationFuture.java:275)
at com.hazelcast.spi.impl.operationservice.impl.InvocationFuture.get(InvocationFuture.java:224)
at com.hazelcast.spi.impl.operationservice.impl.InvocationFuture.get(InvocationFuture.java:204)
at com.hazelcast.map.impl.proxy.MapProxySupport.invokeOperation(MapProxySupport.java:456)        at com.hazelcast.map.impl.proxy.MapProxySupport.getInternal(MapProxySupport.java:275)        at com.hazelcast.map.impl.proxy.MapProxyImpl.get(MapProxyImpl.java:82)        at com.orientechnologies.orient.server.hazelcast.OHazelcastPlugin.getNodeConfigurationById(OHazelcastPlugin.java:271)        at com.orientechnologies.orient.server.hazelcast.OHazelcastPlugin.getClusterConfiguration(OHazelcastPlugin.java:264)        at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.openDatabase(ONetworkProtocolBinary.java:808)        at com.orientechnologies.orient.server.network.protocol.binary.ONetworkProtocolBinary.executeRequest(ONetworkProtocolBinary.java:293)        at com.orientechnologies.orient.server.network.protocol.binary.OBinaryNetworkProtocolAbstract.execute(OBinaryNetworkProtocolAbstract.java:223)        at com.orientechnologies.common.thread.OSoftThread.run(OSoftThread.java:77)
2016-03-25 18:39:00:959 WARNI [<IP_ADDRESS>]:2434 [orientdb] [3.5.3] Removing Member [<IP_ADDRESS>]:2434 because it has not sent any heartbeats for 300000 ms. [ClusterService]
2016-03-25 18:39:01:959 WARNI [<IP_ADDRESS>]:2434 [orientdb] [3.5.3] This node does not have a connection to Member [<IP_ADDRESS>]:2434 [ClusterService]
2016-03-25 18:39:01:959 WARNI [<IP_ADDRESS>]:2434 [orientdb] [3.5.3] Removing Member [<IP_ADDRESS>]:2434 because it has not sent any heartbeats for 300000 ms. [ClusterService]
2016-03-25 18:39:01:959 INFO  [<IP_ADDRESS>]:2434 [orientdb] [3.5.3] Connecting to /<IP_ADDRESS>:2434, timeout: 0, bind-any: true [SocketConnector]
2016-03-25 18:39:03:960 INFO  [<IP_ADDRESS>]:2434 [orientdb] [3.5.3] Connection [Address[<IP_ADDRESS>]:2434] lost. Reason: Socket explicitly closed [TcpIpConnection]
2016-03-25 18:39:03:960 INFO  [<IP_ADDRESS>]:2434 [orientdb] [3.5.3] Removing Member [<IP_ADDRESS>]:2434 [ClusterService]

Are you using sharding?

No, all the nodes are suppose to have all the data.
Here is the default-distributed-db-config.json we use:
{"autoDeploy":true,"hotAlignment":false,"executionMode":"synchronous","readQuorum":1,"writeQuorum":2,"failureAvailableNodesLessQuorum":false,"readYourWrites":true,"servers":{"*":"master"},"clusters":{"internal":{},"index":{},"*":{"servers":["<NEW_NODE>"]}}}

We have serious problems in production because of exactly this (some nodes have started to fail due to unrelated OOM bugs in OrientDB, so it's critical that clustering/failover works!!!)
For my own sake, I really hope the team is working around the clock on this one.

It looks like that is an Hazelcast problem (the thread is stuck in Hazelcast). We already resolved this problem in v2.2.0-SNAPSHOT (develop branch). Applying this in v2.1 it's not easy. What release of Hazelcast do you have in OrientDB libs directory? Could you please replace the jars with the 3.6.1, the same we have under v2.2?

@lvca Thanks you for suggesting this, 3.6.1 jars solves the problem! I can now stop a node without issue.
Of course, the OOM issues remains.

Cool, what OOM? Do you have any log?

Can post a log next time it happens (should be soon enough) Happens when sorting on non indexed fields mostly, or bulk inserts.

After a while I'm closing this because not reproducible. In case, please reopen/comment it.


feat: Added Option for Oracle Connector to use Alias w/ Params

The use of aliases defined in tnsnames.ora is a common connection method found in Oracle, which also enables the specification of communication protocols like TCP/TCPS  (i.e. SSL-enabled) among other configurations through a DSN connection string.
This change adds parameter handling for the connection string defined for an Oracle DB connection. Specifically, the alias boolean parameter when set to true, parses the alias defined in the host directly to the oracle::Connector::new method to initialize the connector object.
For a connection using aliases, the connection string will look as such:
oracle://<username>:<password>@<alias>?alias=true
compared to the direct connection string method specifying host & port:
oracle://<username>:<password>@<host>:<port>/<database>
This will resolve issues #202, #420 & #484 — it is a requested feature for DSN connection strings in TNS aliases to be used.

Hey @wangxiaoying if this change is good to go, any chance we can get a minor release onto pypi? It'll be really helpful to get the MSSQL SSL support & Oracle alias connection in for downstream uses to test it out 😄

Thank you @pangjunrong for the PR! It looks good to me.

Hi @pangjunrong , I just released an alpha version for the new features. You can install by pip install connectorx==0.3.4a3. Please feel free to give it a try!


Re-run pending tasks

Currently Shield does not do anything with tasks that never come out of the pending state. To address this:
Add a function to ReschedulePendingTasks:

Find all tasks in the database listed as pending.
Cross-reference those tasks with the tasks in the scheduling queue of the supervisor. If there is a database-found task which is not in the scheduling queue, add it to the queue

Call this function at startup

This requires changes to the schema (hello, v3!) to track more important information about Task objects (like, what they are supposed to be doing).


[Enhancement] Decoupling AlterTableClauseVisitor and AlterTableAnalyzer

Fixes #issue
What type of PR is this:

[ ] BugFix
[ ] Feature
[x] Enhancement
[ ] Refactor
[ ] UT
[ ] Doc
[ ] Tool

Checklist:

[ ] I have added test cases for my bug fix or my new feature
[ ] This pr will affect users' behaviors
[ ] This pr needs user documentation (for new or modified features or behaviors)

[ ] I have added documentation for my new feature or new function

Bugfix cherry-pick branch check:

[x] I have checked the version labels which the pr will be auto-backported to the target branch

[x] 3.1
[ ] 3.0
[ ] 2.5
[ ] 2.4

[FE PR Coverage Check]
:disappointed: fail : 177 / 392 (45.15%)
file detail

path
covered_line
new_line
coverage
not_covered_line_detail

:large_blue_circle:
com/starrocks/sql/analyzer/AlterTableClauseVisitor.java
176
391
45.01%
[78, 108, 114, 120, 121, 124, 125, 127, 130, 133, 138, 139, 142, 143, 148, 149, 150, 151, 155, 156, 162, 163, 167, 168, 174, 175, 182, 183, 184, 188, 189, 197, 203, 204, 205, 208, 213, 214, 223, 229, 240, 244, 248, 254, 255, 264, 268, 269, 277, 278, 281, 282, 285, 286, 289, 291, 292, 294, 295, 297, 300, 301, 302, 303, 304, 306, 309, 310, 311, 312, 313, 314, 315, 317, 318, 320, 323, 324, 325, 326, 328, 329, 334, 335, 336, 337, 338, 339, 340, 348, 349, 355, 360, 364, 365, 379, 386, 387, 390, 394, 396, 397, 400, 401, 404, 405, 408, 409, 411, 412, 414, 417, 418, 419, 420, 421, 423, 426, 427, 428, 429, 430, 431, 432, 434, 435, 437, 440, 441, 442, 443, 445, 451, 455, 456, 457, 458, 459, 460, 461, 462, 478, 484, 485, 486, 487, 489, 491, 501, 505, 506, 514, 515, 518, 519, 522, 523, 526, 528, 529, 531, 532, 534, 537, 538, 539, 540, 541, 543, 546, 547, 548, 549, 550, 551, 552, 554, 555, 557, 560, 561, 562, 563, 565, 566, 573, 574, 580, 593, 597, 608, 612, 646, 650, 655, 656, 660, 661, 679, 683, 686, 687, 696, 697, 706, 707, 715, 716, 717, 718, 719, 724, 725, 726, 728]

:large_blue_circle:
com/starrocks/sql/analyzer/AlterTableStatementAnalyzer.java
1
1
100.00%
[]

@Mergifyio backport branch-3.1


Add optional post-build compression step

There are a few current and future appliances where the output of imagefactory is a non-compressed format. This patch adds support for an optional post-build compression step to reduce the size of those images.
The supported formats are "gzip" for cases where the users are mostly Unix/Linux people, and "zip" for Windows folks.
This PR doesn't yet enable any compression. However I would suggest we enable in a future PR:

ZIP compression for Azure (given the Windows audience). This will bring   down the appliance size of 3.3GB to a size similar to the other appliances  (currently roughtly 1GB for the nightly builds).
ZIP compression for HyperV. Same situation as Azure.
GZIP compression for EC2 (once EC2 PR has been enabled). The native image   format for EC2 is a sparse RAW file which will be 50GB in size. This is  not practical and must be compressed.

@miq-bot add_label euwe/yes

Seems reasonable but i'm not sure if this is desireable as you can't directly import them into the provider.

@miq-bot remove_label euwe/yes

@geertj Because of Azure VHD image issue, I'd need to create Azure image a bit differently now and that will require the final image to be compressed. I'd like to use this new compress option if possible - would you mind updating the PR?


Expose non-bundled files and named exports

so external webpack or node can do its job resolving dependencies.
Expose PalindromDOM for browser.

Sorry, my bad I messed up something with rebasing. Closing in favor of https://github.com/Palindrom/Palindrom/pull/257


[v0.0.2] Update docs, fix imports, fix unpkg links, move data files

Will bump version when changes are accepted.

Interestingly, both dash_renderer and dash-renderer would work, since they are recognized as the same package by pip. In fact, it would also work for Flask and flask:
C:\Users\xingh>pip show Flask
Name: Flask
Version: 1.0.1
Summary: A simple framework for building complex web applications.
Home-page: https://www.palletsprojects.com/p/flask/
Author: Armin Ronacher
Author-email<EMAIL_ADDRESS>License: BSD
Location: c:\users\xingh\appdata\local\continuum\anaconda3\lib\site-packages
Requires: Jinja2, Werkzeug, itsdangerous, click
Required-by: Flask-Cors, Flask-Compress, Flask-Caching, dash
You are using pip version 18.0, however version 18.1 is available.
You should consider upgrading via the 'python -m pip install --upgrade pip' command.

C:\Users\xingh>pip show flask
Name: Flask
Version: 1.0.1
Summary: A simple framework for building complex web applications.
Home-page: https://www.palletsprojects.com/p/flask/
Author: Armin Ronacher
Author-email<EMAIL_ADDRESS>License: BSD
Location: c:\users\xingh\appdata\local\continuum\anaconda3\lib\site-packages
Requires: Werkzeug, click, Jinja2, itsdangerous
Required-by: Flask-Cors, Flask-Compress, Flask-Caching, dash
You are using pip version 18.0, however version 18.1 is available.
You should consider upgrading via the 'python -m pip install --upgrade pip' command.

On the other hand, it wouldn't work for a package like scikit-learn (i.e. pip show sklearn doesn't print anything). From what I observed (e.g. Flask) it is more common to include the PyPi distribution of the package, rather than the repo name:
dash_renderer: https://pypi.org/project/dash_renderer/
dash-html-components: https://pypi.org/project/dash-html-components/
So in this case it would be dash_renderer and dash-html-components. However, I believe this is not strictly enforced, since - are directly converted to _, and vice versa. Similarly for lower case and upper case.
In fact pip show DASH-HtMl_comPONENTS would yield the same results as pip show dash-html-components, and https://pypi.org/project/DASH-HtMl_comPONENTS would redirect to https://pypi.org/project/dash-html-components/.


Popup should support controlled mode / ContextMenu component

Feature Request
Problem description
We need a way to be able to override the open state of the Popup. e.g. when clicking on the items in the popup, we need to close the popup -

Proposed solution
Provide a way to say that the popup is closed now. Note - We still want to re-use the document click behavior from the popup.
MVP
Not sure yet how to allow it. Probably, we can provide some callbacks through the component ref like fabric - https://github.com/OfficeDev/office-ui-fabric-react/blob/master/packages/office-ui-fabric-react/src/components/Popup/Popup.types.ts.
Or
May be support menu as a shorthand prop in the popup, and have a prop to close the popup on item select.
Or
I am sure this is a common requirement. Probably, create a separate ContextMenu component.

thanks for posting this issue - at the moment we are working on PR that will address this problem

closing the issue as now it is already possible - Popup supports controlled mode:
class Foo extends React.Component {
  ....
  render() {
     // Foo controls the state of the Popup
     <Popup open={this.state.isPopupOpen} ... />
  }
}


fix TestWalletRescanning NDF

This PR addresses an observed NDF in TestWalletRescanning by mining more blocks and sleeping less.

ouch. One of the issues here is that with our current set of DEBUG flags, adding blocks is quadratic but rescanning is only linear.
We might be able to fix that. I'm always nervous to disable sanity checks, but perhaps we can make the quadratic one a probabilistic check.


Laravel 5.2

Changes
Updates Northstar to Laravel 5.2! Not only do we get all the latest fanciest new goodies, Laravel 5.2 also allows us to easily override the Guard class which would have involved some hackery before. This will help us clean up some of the custom authentication work we've been doing in a future PR.

For review: @angaither

looks kewl

:sunglasses:


NBC-8 # Build.json validation

Added

Added buildfile base system
Added windows validation
Added documentation for 'build' command

Changed

Changed default profile value to 'default'
Changed 'upload' command to 'build'

LGTM 👍


Release build triggers libc++ pipeline on a wrong commit

https://buildkite.com/llvm-project/llvm-releases/builds/1358 on llvm 14.x branch has triggered libcxx pipeline on completely unrelated commit.

Gentle ping on this, I have no idea what's going on but it's creating a lot of traffic on the libc++ pipeline :(

Hi Louis! Yea, I remember! will try to look at that tomorrow. sorry for the
delay.
On Fri, Mar 4, 2022, 21:06 Louis Dionne @.***> wrote:

Gentle ping on this, I have no idea what's going on but it's creating a
lot of traffic on the libc++ pipeline :(
—
Reply to this email directly, view it on GitHub
https://github.com/google/llvm-premerge-checks/issues/389#issuecomment-1059478940,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAB6EOYT4AN5KV54KHLOTVDU6JUMLANCNFSM5OSLEDIQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

@ldionne I think I have fixed it. Sample build https://buildkite.com/llvm-project/llvm-releases/builds/1416 .

@metaflow Yes, this is fixed now indeed. Thank you so much for taking a look, BuildKite is a lot more usable now! I really appreciate the support -- I had to cancel old jobs every day previously 😅


Built-in comparator not used for SortedSet implementations

Summary
Assertions on implementations of SortedSet (like TreeSet) do not use the comparator used to construct the instance.
Example
If I wanted to create a class that returned a set of strings which allowed testing for case-insensitive inclusion, I'd use a TreeSet with a custom comparator. The following code example is greatly simplified, but demonstrates the basic mechanics (in a real example, the creation of the set would be encapsulated within the class under test).
TreeSet<String> mySet = new TreeSet<>(Comparator.comparing(String::toUpperCase));
mySet.add("FOO");
assertThat(mySet.contains("foo")).isTrue(); // passes
assertThat(mySet).contains("foo"); // fails

I found it surprising that the assertion didn't really call mySet.contains(), but looking into the AssertJ codebase I found that it was highly abstracted.
I can work around it by passing in an element comparator, like this:
assertThat(mySet).usingElementComparator(mySet.comparator()).contains("foo")

However, in a real application, the set would probably have come from a method just returning Set<String>, so accessing its comparator is mostly out of the question. And even if I did, it just wouldn't feel like I was actually testing that my class returned a case-insensitive set.

@tommcdo I think that this issue is the same as #1104

@tommcdo this is really interesting situation you have.
I encountered the same for a few times and was also surprised.
But what if you look at it this way:

by testing assertThat(mySet.contains("foo")) you test the concrete implementation of contains method of the given object
by testing assertThat(mySet).contains("foo") you test that given object contains exactly specific object.
as said in javadoc

Verifies that the actual group contains the given values, in any order

@filiphr I think it is slightly different

At first sight, the issue is in StandardComparisonStrategy#iterableContains:
  public boolean iterableContains(Iterable<?> iterable, Object value) {
    if (iterable == null) {
      return false;
    }
    for (Object next : iterable) {
      if (areEqual(next, value)) {
        return true;
      }
    }
    return false;
  }

It checks values one by one with areEqual instead of using the iterable contains method, this was made to be able to override areEqual to use a different comparison strategy.
I think this should be changed so that collection accepting a Comparator are correctly honored (mainly TreeSet).

It looks most general interface where a custom comparator can be supplied is SortedSet.
Would it be too naive to simply overload Assertions.assertThat(SortedSet)?
public static <ELEMENT> IterableAssert<ELEMENT> assertThat(SortedSet<? extends ELEMENT> actual) {
  return new IterableAssert<>(actual).usingElementComparator(actual.comparator());
}

@tommcdo that is a nicely short possibility, I think the only drawback is that if your SortedSet is declared as a Set that won't work, it would select assertThat(Iterable<? extends ELEMENT> actual) instead of assertThat(SortedSet<? extends ELEMENT> actual).

@joel-costigliola Good call, that would be non-ideal.

We might get away with the limitation I mentioned by checking if the runtime type of actual is a SortedSet.


Support for landlines

This adds support for landline numbers

Thanks king, makapenga. Let me run this and merge ✌🏿


[WIP] query OpenSearch via temporal extent of dataset (#633)

Please do not merge yet.
Overview
Update OpenSearch support to query temporal extent of the data with a metadata record.
Related Issue / Discussion
#633
Additional Information
Contributions and Licensing
(as per https://github.com/geopython/pycsw/blob/master/CONTRIBUTING.rst#contributions-and-licensing)

[x] I'd like to contribute [feature X|bugfix Y|docs|something else] to pycsw. I confirm that my contributions to pycsw will be compatible with the pycsw license guidelines at the time of contribution.
[x] I have already previously agreed to the pycsw Contributions and Licensing Guidelines

I tested this on my testing endpoint and works as expected. Temporary URL:  https://csw.epinux.com/?mode=opensearch&service=CSW&version=2.0.2&request=GetRecords&elementsetname=full&typenames=csw:Record&resulttype=results&time=2019-01-01/2020-09-01
Thanks!

Any updates on this?


Add draft workflow to Publication work types

Note that this PR also fixes a bug where submitted publications were failing to index when their "relationship" field consists of an array of empty strings. This but was uncovered in the course of writing feature specs for publication work types.

I confirmed the draft workflow works for Publication in my development environment.

I confirmed the draft workflow works for Publication in my development environment.


update String equality and stream close

update String equality
make stream close whatever happens

Hello,
Thank you for contacting us.
We appreciate your suggestions and merged PR.
Regards,
SendinBlue Team

Hello,
Thank you for contacting us.
We appreciate your suggestions and merged PR.
Regards,
SendinBlue Team


[Serial] Remove unnecessary member variables

We are keeping state for no reason (mostly begin() method); racks up unnecessary memory.
For example:
Windows::Devices::Bluetooth::Rfcomm::RfcommDeviceService ^_device_service;
Windows::Devices::Bluetooth::Rfcomm::RfcommServiceId ^_service_id;
Windows::Devices::Bluetooth::Rfcomm::RfcommServiceProvider ^_service_provider;
Windows::Networking::Sockets::StreamSocket ^_stream_socket;

Due to the way UWP API is implemented today, storing the member variables appears to be necessary in order to prevent the premature deallocation of those variables.


UI Bug: Accessibility of text in light theme

Which API Provider are you using?
OpenRouter
Which Model are you using?
Claude 3.5
What happened?
The text is unreadable in light theme.

Steps to reproduce

Use VS Code in Light+ Theme
Click on Cline Extension
Type @ and wait for auto-complete

Relevant API REQUEST output
No response
Additional context
No response

I would like to contribute and fix this

Fixed by https://github.com/cline/cline/pull/942


Running chatdocs ui on headless server

I'm attempting to run this on a headless server (Ubuntu 22.04) where I have considerably more resources, and can't get internet access to it.  I've attempted to modify the IP/Port numbers on the chatdocs.yml, tried modifying the ui.py, nothing.  Keeps returning the

Running on http://localhost:5000

local firewalls are disabled and the device is reachable via Ping.
Any suggestions?

I'm here for guidance on this as well

I just released a new version 0.2.5 which adds host and auth options.
Add the following to chatdocs.yml to allow access via network:
host: <IP_ADDRESS>

To connect, you should use the public IP of your server instead of localhost: http://SERVER_IP:5000
But this is not secure as it allows public access, so you can also add auth: true to chatdocs.yml which generates a random authenticated URL to access the UI.
For more security, you can limit your server/port access to your local machine's IP e.g. using security groups if you are using AWS.


cannot set height of open menu

Hi, How to set height of residemenu when opening the menu.. if i set 0.4, its height too small instead of 80% height. What i want to set the height is like below image..

how you have done this?


Could not open database

java.lang.RuntimeException: Unable to create application com.example.recipes.ReciepyApp: android.database.sqlite.SQLiteCantOpenDatabaseException: unknown error (code 14): Could not open database
A week ago i use this library and successfully run my app but
after a week ago when i modify *.sqlite file and giving me this error on
private static final String DATABASE_NAME = "/databases/recipev3.sqlite";
public RecipyHelperClass(Context context) {
        super(context, DATABASE_NAME, null, DATABASE_VERSION);
        db = this.getReadableDatabase(); //error here
    }
can u help me..

Are you changing the "v3" part of the DATABASE_NAME on each revision? Or is the name always recipev3.sqlite and you're using the versioning system by incrementing DATABASE_VERSION per the documentation?
If the former, please read the Database Upgrades documentation and follow the instructions there.)
If the latter, we'll need more context to solve probably: stack trace, device, has the app been reinstalled, what files are in your assets directory, what has changed in your source since it was working, etc.


Fatal error: [InvalidArgumentException] "@0" is not a valid date.

Here's the full output I  get when I attempt to run this tool:

  [InvalidArgumentException]
  "@0" is not a valid date.

generate:json [-s|--skip="..."] [-o|--only="..."] input output

OS: Windows 10
php --version: PHP 5.6.25 (cli) (built: Aug 18 2016 11:39:11)

@sbrl what is your command?
I tried bin/uml-generator-php generate:json ./src /tmp/output without trouble.

Huh. that command appears to work. I must have done something strange.


Watch does not have an event for terminated workloads

Summary
When creating a Watch using the argo go client, the event channel does not receive an event when a workload is terminated and will just hang open.
Motivation
I would like to do some cleanup when a workload finishes and I have no idea when a terminated workload is terminated. That being said, it does seem like the channel does close by itself after a while, but is there any way to make this more prompt?
Proposal
When a workload is terminated, an event should be generated which sets the workload status to terminated or something. I understand that there's a bit of an issue here because the workload status doesn't exist at all anymore, but something along these lines would be great.

Message from the maintainers:
If you wish to see this enhancement implemented please add a 👍 reaction to this issue! We often sort issues this way to know what to prioritize.

The argo-go client is a wrapper around the kubernetes native watch API, and should get exactly the same events of that. Can you confirm that the native k8s watch is receiving the event, but the argo go client is not?

Can I request more information:

What version of Argo CLI are you using?
Are you using Argo Server?
*. What exact command do you execute to terminate your workflow?
What happens when your run kubectl get -w wf my-my?


main window not opening since latest patch

since the patch today the main window wont open.  It didn't open automatically when entering HoH and the /necrolens command also didn't work to open it.  The config window does open and the options seems to all work.

Same for me

yes this is correct, there was a client update and i need to update the plugin. im at hospital rn and cant update it till weekend.

Its updated now. Sorry for the very long delay :(


Issue with image name directory not existing

If the wfi name to create doesn't have an existing directory with the same name, the std::filesystem::canonical() call fails.
Need to look into this and see if having a directory for the image with the same name is what i actually wanted and then fix it with an exists/mkdir code

no longer relevant now that i switched formats.


Add Burkina Faso

Coverage remained the same at 96.522% when pulling 3f787c91ee79765cb64396cae2f74b2a090927fb on addBurkinaFaso into 8fb788003581d9f908f7690a6303a5308db6ba70 on main.


docs/environment.md: Fix 'hwo' to 'how'

Signed-off-by: Denis Semakin<EMAIL_ADDRESS>

@macpijan Review please

Thanks for thix fix


Chibi Scheme does not start on Mac OS due to missing (probably misplaced) dylib

Describe the bug
When the package chibi is installed on Mac OS, running the main chibi-scheme program fails because the dynamic library containing most of the Scheme implementation is expected to be in the global /usr/lib or /usr/local/lib directories, but isn’t.
The specific error is:
> chibi-scheme -R
dyld[67877]: Library not loaded: '/usr/local/lib/libchibi-scheme.0.10.0.dylib'
  Referenced from: '/nix/store/gn5fj0bz79p8kimk1kccs8in2kzcap2h-chibi-scheme-0.10/bin/.chibi-scheme-wrapped'
  Reason: tried: '/usr/local/lib/libchibi-scheme.0.10.0.dylib' (no such file), '/usr/lib/libchibi-scheme.0.10.0.dylib' (no such file)
fish: Job 1, 'chibi-scheme -R' terminated by signal SIGABRT (Abort)

Steps To Reproduce
I installed Chibi with nix-env -iA nixpkgs.chibi on Mac OS 12.5 and ran chibi-scheme at the command line. The version of Chibi it installed was 0.10.
Expected behavior
chibi-scheme should just work when installed.
Notify maintainers
@DerGuteMoritz
Metadata
 - system: `"aarch64-darwin"`
 - host os: `Darwin 21.6.0, macOS 12.5`
 - multi-user?: `yes`
 - sandbox: `no`
 - version: `nix-env (Nix) 2.10.3`
 - channels(root): `"nixpkgs"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixpkgs`

Please format the commit as shown in this link. Thanks!


[Suggestion] Version number to check against

Would it be possible to add a version we can check against for those of us using the win32-dynamic build?
For example, if we implement something that requires at minimum v2.1.0, there should be a way to check that the user has a DLL that meets the minimum version. Here's how it could be potentially implemented:
DISCORD_EXPORT float Discord_GetVersion(void) const;

Usage:
if (Discord_GetVersion() < 2.1f) {
  Error("Your Discord RPC plugin is out of date, please update it and try again.");
}

This would ensure any projects relying on the dynamic branch (which may require users to download the DLL manually) will continue to work correctly, without having to resort to customer feedback of "why isn't X working?!" -- unless this is guaranteed to never happen, of course!

you should just deploy your app with the dll, any changes worth changing the dll for will most likely require changes to your app anyway

@devsnek It's an open-source mod for a game that we don't have the source code to. I did attempt to use the static version, but it didn't play well with the linker unfortunately.

Can't you ship the dynamic dll alongside your mod?

I think Bluexin has the right idea here. Rather than instruct users to update a third-party dependency (that they may or may not know how to to) for your third-party mod, it probably makes more sense to include a build script in whatever update process your mod has to pull the latest dll from our releases, or compile it yourself if that's preferable.
That means players will never have to worry about juggling dlls, and you don't have to worry about writing a help article instructing them how to do it and answering support questions of:
Error("Your Discord RPC plugin is out of date, please update it and try again.");

what does this mean

Does that seam reasonable?

Currently, the only build process involved is using AppVeyor to compile the latest changes made to the repository. I am not aware of the ability to include a 3rd party dependency with the build artifacts it generates.
The error message was just a simple, straight-forward example. I include more in-depth instructions on my repo, and regularly keep everyone updated via our Discord channel. There is no updater for the mod (other than the AppVeyor builds). The amount of users is very small, maybe a handful of people use it at the most. Just a small side hobby, really.
I only suggested this idea because, as mentioned, there is no way to verify that the dynamic version being loaded is compiled against the header file used in the mod. It's a very, very minor thing, and I doubt it'll be utilized by a lot of people. But it would be there if needed!
I don't think there's much more that I can say. I feel it's a harmless thing to implement and keep maintained, just for that extra degree of security. Developers may use it however they deem necessary, if even at all.

Moved to Feedback Site as per #290


help! view button Init

Type of issue

[ ] Bug report
[ ] Feature request
[ o] Support request

Uploader type

[o ] Traditional
[ ] S3
[ ] Azure

Support Request
Fine Uploader version
5.14.2
Question
After creating the view button in the template
I want to initialize it by calling the addInitialFiles function. But it is not as easy as I thought. Look at the source below.
Related code
template

                     .....
                    <button type="button" class="view-btn hide btn">View</button>
                     .....

Run when page loads
callbacks: {
	            	onStatusChange: function (id, oldStatus, newStatus) {

	                    var _uuid = this.getUuid(id);
	                    var result = $.grep(this._options.initFileData, function(e){ return e.uuid == _uuid; });

	                    if (newStatus == qq.status.UPLOAD_SUCCESSFUL && result.length > 0) {
	                        var serverPathToFile = result[0].path;
	                        //var fileItem = this.getItemByFileId(id);
	                        //var viewBtn = qq(fileItem).getByClass("view-btn")[0];
	                        console.log($('.qq-file-id-' + id + ' qq-upload-success')); //undefined
	                        console.log($('.qq-file-id-' + id + ' qq-upload-success').find('.view-btn')); //undefined

	                        //viewBtn.setAttribute("onclick", "location.href='" + serverPathToFile + "';");
	                        //qq(viewBtn).removeClass("hide");
	                    }
	                },

function getFileList(_data) {
for(var i = 0; i < _data.value.length; i++) {
			            		var val = _data.value[i];
			            		var _ext = val.new_txt_attachname.split('.');

			            		var _thumbnailurl = "";
			            		if(_ext[1] == "pdf") {
			            			_thumbnailurl = '../fileupload_/placeholders/PDF_64.png';
			            		}
			            		else if(_ext[1] == "txt") {
			            			_thumbnailurl = '../fileupload_/placeholders/TXT_64.png';
			            		}
			            		else if(_ext[1] == "xlsx" || _ext[1] == "xls") {
			            			_thumbnailurl = '../fileupload_/placeholders/Microsoft_Excel_64.png';
			            		}
			            		else if(_ext[1] == "docx" || _ext[1] == "doc") {
			            			_thumbnailurl = '../fileupload_/placeholders/Microsoft_Word_64.png';
			            		}
			            		else if(_ext[1] == "pptx" || _ext[1] == "ppt") {
			            			_thumbnailurl = '../fileupload_/placeholders/Microsoft_PowerPoint_64.png';
			            		}
			            		else if(_ext[1] == "png" || _ext[1] == "jpg" || _ext[1] == "gif" || _ext[1] == "jpeg") {
			            			_thumbnailurl = config.fileUrl + val.new_txt_attachname;
			            		}
			            		else {
			            			_thumbnailurl = '../fileupload_/placeholders/not_available_generic.png';
			            		}

			            		manualUploader._options.initFileData.push({
			            			uuid: val.new_txt_uuid,
			            			path: config.fileUrl + val.new_txt_attachname
			            		});

			            		_listarray.push({
			            			"name": val.new_name,
					                "uuid": val.new_txt_uuid,
					                "size": val.new_txt_size,
					                "thumbnailUrl": _thumbnailurl
			            		});
			            	}

			            	manualUploader.addInitialFiles(_listarray);
}

And the problem is?

I want to link to the click of the view button but can not hang it.

sorry, i don't understand

onStatusChange
console.log($('.qq-file-id-' + id + ' qq-upload-success')); //undefined
When can I bind?
viewBtn.setAttribute("onclick", "location.href='" + serverPathToFile + "';");

First off changing the page URL using a click handler makes no sense. That's why anchor links exist.  set the href of your anchor link when the file status is qq.status.UPLOAD_SUCCESSFUL and be sure your version of the lib is up to date.

 I'm sorry but I don't have much time to beg for information in order to provide support. You've provided nothing in terms of a runnable example or even information about what is and isn't happening when your issue occurs.

If you do find a specific issue in the lib, please open a pull request, with tests. The issue you are describing is already covered as far as I know


How do i place an image behind a video by using Transformer?

I have a video (360x720) and an image (1280x960). Please show me how to place the image behind the video like this:

it is possible to do via custom effects.
For reference, take a look at [OverlayShaderProgram](https://github.com/androidx/media/blob/release/libraries/effect/src/main/java/androidx/media3/effect/OverlayShaderProgram.java) and [OverlayEffect](https://github.com/androidx/media/blob/release/libraries/effect/src/main/java/androidx/media3/effect/OverlayEffect.java)


I tried to pass the startPageIndex as 10 but the pdf is not loaded as page 11

I tried to pass the startPageIndex as 10 but the pdf is not loaded as page 11
private func showDocument(_ document: PDFDocument) {         let image = UIImage(named: "")         let controller = PDFViewController.createNew(with: document, title: document.fileName, actionButtonImage: image, actionStyle: .activitySheet, startPageIndex: 10)         navigationController?.pushViewController(controller, animated: true)     }
Could you please advise?

u should move the code
" currentPageIndex = updatedPageIndex
thumbnailCollectionController?.currentPageIndex = currentPageIndex" on the func scrollViewDidScroll to the other place, like this:
public func collectionView(_ collectionView: UICollectionView, willDisplay cell: UICollectionViewCell, forItemAt indexPath: IndexPath) {
currentPageIndex = indexPath.row
self.title = String(format: "%d/%d",currentPageIndex,self.document.pageCount)
thumbnailCollectionController?.currentPageIndex = currentPageIndex
}


[Question] Bot works in the emulator but doesn't work on dev.botframework or on skype channel

System Information (Required)

SDK Language:  .NET/C#

My bot works fine in the emulator,even with Microsoft app id and password but when I'm testing it at https://dev.botframework.com i get a message "couldn't send message retry".I get a 401 unauthorized error when I'm trying to get a reply from the emulator with bot url https://botwebdeployment.azurewebsites.net/ (deployed in azure).I have added my  bot in microsoft bot directory and have enabled skype .But even on my machine i can send message on skype app to bot but i don't get any reply.

Makes sure that your MicrosoftAppId and MicrosoftAppPassword are set in Web.config
Sent from Mail for Windows 10
From: Chetanchezz
Sent: 22 July 2017 17:20
To: Microsoft/BotBuilder
Cc: Subscribed
Subject: [Microsoft/BotBuilder] Bot works in the emulator but doesn't work ondev.botframework or on skype channel (#3177)
System Information (Required)
• SDK Language: .NET/C#
My bot works fine in the emulator,even with Microsoft app id and password but when I'm testing it at https://dev.botframework.com i get a message "couldn't send message retry".I get a 401 unauthorized error when I'm trying to get a reply from the emulator with bot url https://botwebdeployment.azurewebsites.net/ (deployed in azure).I have added my bot in microsoft bot directory and have enabled skype .But even on my machine i can send message on skype app to bot but i don't get any reply.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

Duplicate of https://stackoverflow.com/questions/45256550/my-bot-works-fine-in-the-emulator-even-with-microsoft-app-id-and-password-but-wh


Add Recommendations API Spec

ARM API Information (Control Plane)
MSFT employees can try out our new experience at OpenAPI Hub  - one location for using our validation tools and finding your workflow.

Azure 1st Party Service can try out the Shift Left experience to initiate API design review from ADO code repo.  If you are interested, may request engineering support by filling in with the form https://aka.ms/ShiftLeftSupportForm.
Changelog
Add a changelog entry for this PR by answering the following questions:

What's the purpose of the update?

[ ] new service onboarding
[x] new API version
[ ] update existing version for new feature
[ ] update existing version to fix swagger quality issue in s360
[ ] Other, please clarify

When are you targeting to deploy the new service/feature to public regions? Please provide the date or, if the date is not yet available, the month. November
When do you expect to publish the swagger? Please provide date or, the the date is not yet available, the month. November
By default, Azure SDKs of all languages (.NET/Python/Java/JavaScript for both management-plane SDK and data-plane SDK, Go for management-plane SDK only ) MUST be refreshed with/after swagger of new version is published. If you prefer NOT to refresh any specific SDK language upon swagger updates in the current PR, please leave details with justification here.

Contribution checklist (MS Employees Only):

[x] I commit to follow the Breaking Change Policy of "no breaking changes"
[x] I have reviewed the documentation for the workflow.
[x] Validation tools were run on swagger spec(s) and errors have all been fixed in this PR. How to fix?

If any further question about AME onboarding or validation tools, please view the FAQ.
ARM API Review Checklist

Applicability: :warning:
If your changes encompass only the following scenarios, you should SKIP this section, as these scenarios do not require ARM review.

Change to data plane APIs
Adding new properties
All removals

Otherwise your PR may be subject to ARM review requirements. Complete the following:

[x] Check this box if any of the following apply to the PR so that the label "ARMReview" and "WaitForARMFeedback" will be added by bot to kick off ARM API Review.  Missing to check this box in the following scenario may result in delays to the ARM manifest review and deployment.

Adding a new service
Adding new API(s)
Adding a new API version
-[ ] To review changes efficiently, ensure you copy the existing version into the new directory structure for first commit and then push new changes, including version updates, in separate commits. You can use OpenAPIHub to initialize the PR for adding a new version. For more details refer to the wiki.

[x] Ensure you've reviewed following guidelines including ARM resource provider contract and REST guidelines. Estimated time (4 hours). This is required before you can request review from ARM API Review board.

[x] If you are blocked on ARM review and want to get the PR merged with urgency, please get the ARM oncall for reviews (RP Manifest Approvers team under Azure Resource Manager service) from IcM and reach out to them.

Breaking Change Review Checklist
If you have any breaking changes as defined in the Breaking Change Policy, request approval from the Breaking Change Review Board.
Action: to initiate an evaluation of the breaking change, create a new intake using the template for breaking changes. Additional details on the process and office hours are on the Breaking Change Wiki.
NOTE: To update API(s) in public preview for over 1 year (refer to Retirement of Previews)
Please follow the link to find more details on PR review process.

https://github.com/Azure/azure-rest-api-specs-pr/pull/7705
ARMSignedOff already given in this PR. The specs and files are identical.

Is this PR different from https://github.com/Azure/azure-rest-api-specs/pull/21052 ? Can you please close either one if it is a dupe ?

Is this PR different from #21052 ? Can you please close either one if it is a dupe ?

Closed #21052 as this merges to the correct branch.


Fix invocation of bump script command

When doing the latest release, we have noticed the --version flag in the patch releases guide is incorrect and it should be -v instead.

We can also use more verbose to-version, which is the alias for the v flag. Reference:

Fine with both, as long as we update all the scripts snippets to use the same

I've updated the command to use --to-version and --token to align the individual guides 👍


Adding the option to change the default list style via the config, de…

…faulting to ordered so this is fully backwards compatible, updating the README to reflect new config options
This PR adds functionality to make the default list type variable. I added a check to ensure the type was either ordered or unordered - so long as the option passed via the config.defaultListStyle is one of these valid options, it is considered the default style. If it is not a valid option, or does not exist, the fallback is ordered so that it is fully backwards compatible.
The README as been updated to reflect this change.
Per the comments on other PRs, this code has been linted with ESLint. I did not see any style linting rules such a .prettierrc so I did the style linting by hand. Please ensure it matches the code base style to your liking - if not, please let me know how to improve it. Thanks.

+1 to this. @editorjs/list supports a config: { defaultStyle: "unordered" } so it would be ideal to have the same config when switching over to this tool.

@sethmichaelbrown thanks for the PR, but this feature is updated via #50


Bridge site id 3700783 has no span or width

There is no span or width seen on the mergin form or the photo of the paper card. @NewGraphEnvironment did you perhaps take a picture of the site card before you filled that data in? Can you check the site card?

no need to @mention me as i get notifications of all issues as the owner (until I turn them off which hasn't happened yet)
There is a photo of the site card labelled as such in the photos on onedrive (no width or length though).
Not sure why I rushed through recording that. doh
looking at the photos I would estimate 5m x 13m

Thanks, changed it in Q and input into template


[gn] Vscode integration. Fixes #1526.

Problem
The new GN build system isn't integrated yet into vscode
Summary of Changes
Add vscode integration
fixes #1526

 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.

@turon can you sign the CLA (again) and fix the merge here?


SegmentedControl needs layout constraints

I got this working last time by adding the views programmatically (not hooking up MXSegmentedPager in the storyboard). I have some pretty complex view controllers in my app, and I am trying to hook up MXSegmentedPager to an outlet in storyboard, where the top and bottom constraints change.
I have found that the segmented control needs constraints. I added this to - (HMSegmentedControl *)segmentedControl method:
    _segmentedControl.translatesAutoresizingMaskIntoConstraints = NO;
    NSDictionary *binding  = @{@"v" : _segmentedControl};
    [self addConstraints:[NSLayoutConstraint
                          constraintsWithVisualFormat:@"H:|-0-[v]-0-|"
                          options:NSLayoutFormatDirectionLeadingToTrailing
                          metrics:nil
                          views:binding]];

    [self addConstraints:[NSLayoutConstraint
                          constraintsWithVisualFormat:@"V:|-0-[v(50)]"
                          options:NSLayoutFormatDirectionLeadingToTrailing
                          metrics:nil
                          views:binding]];

which works well. But only in the case that there is no ParallaxHeader.
I am having trouble adding the constraints to the MXSegmentedPager+ParallaxHeader.m file, when a parallax header exists. No matter where I add them, things blow up.

Hi @mommyme !
This does not work with the parallax category because the segmented control and the pager are added to a scroll view so your constraints will mess up with the parallax layout. Could you provide a example that show your issue, you can modify the example project with a complex controller and constraints and send it to my email. Or maybe add a gif that show what happens.
Because I also use the MXSegmentedPager with constraints in a storyboard and I don't have any issues...

Hi @maxime! I will try to provide an example when I can, but my use case is pretty complex. My issue happens when setting the cover to nil; you currently only set constraints for the parallax header/scrollview, but you do not set constraints when it is not used, and this actually does affect things. I think that when there is no parallax header (cover is nil) constraints should be set on the segmented control and pager. I will try to post an example when I can, but I think this needs to be done.
 On Thursday, May 28, 2015 8:24 AM, Maxime<EMAIL_ADDRESS>wrote:

Hi @mommyme !
This does not work with the parallax category because the segmented control and the pager are added to a scroll view so your constraints will mess up with the parallax layout. Could you provide a example that show your issue, you can modify the example project with a complex controller and constraints and send it to my email. Or maybe add a gif that show what happens.
Because I also use the MXSegmentedPager with constraints in a storyboard and I don't have any issues...—
Reply to this email directly or view it on GitHub.

Hi @mommyme !
FYI, I'm working on a pure autolayout approach. To do so, the standard class and the parallax category have been merge. I still got a lot of work to do but you can see the code on the parallax-header-merge branch.
Regards,
Maxime

Hi Maxime. How is the parallax-header-merge branch coming? I was working on some other things and I thought I would come back to this if it was ready.

Hi @mommyme
I have still some issues, mainly with ios 7. But you can try it :
pod 'MXSegmentedPager', :git => 'https://github.com/maxep/MXSegmentedPager.git', :branch => 'parallax-header-merge'

It's a completely different control. I don't know when I will have time to completely rework my project.

I tried it and it still doesn't work with Storyboard. I think that is part of why there are so many issues with the control. Just as you would hook up a UITableView in a storyboard, try to put the MXSegmentedPager in a storyboard. It simply fails every time. I appreciate your hard work on this control, but I think until you test it in storyboard it will have issues. Not all people will need to use it in storyboard, but some people will.

Hi @mommyme ,
The branch is not completely different, the interface and the category have only been merge. You just have to remove the '+ParallaxHeader.h' from your include, it is not a completely rework.
Well, I used it in storyboard with no issue. Since I can't reproduce your issue and you never provide any sample project, or at least a screenshot, I will close this issue.
Regards,
Maxime

Sorry, I was having a rough time getting things set with my last comment. I have everything up and running now. Thanks again for this control.


Add 'environment_variables' arguments to ContainerInstanceManagementClient.container_groups.begin_start()

Is your feature request related to a problem? Please describe.
I am trying to start a task-based container instance (within an Azure Trigger Function) that I previously deployed on Azure. However, I realize I cannot add env var when I start the container run withContainerInstanceManagementClient.container_groups.begin_start()
see docs
This limits the capability for me to customize each container run with additional flexibility. This issue should apply to anyone who has the need to programmatically start a container instance with the Azure Python SDK.
Describe the solution you'd like
Add 'environment_variables' arguments to ContainerInstanceManagementClient.container_groups.begin_start(), so that environment variables can be assigned to the container upon each run. Other Container settings (e.g. memory_in_gb, cpu) can also be added for more flexibility.
Ideally, the solution should look like this

env_var1 = EnvironmentVariable(name='ENV_NAME1', value='ENV_VALUE1')
env_var2 = EnvironmentVariable(name='ENV_NAME2', value='ENV_VALUE2')

client = ContainerInstanceManagementClient(
    credential=DefaultAzureCredential(),
    subscription_id=os.environ["AZURE_SUBSCRIPTION_ID"])

response = client.container_groups.begin_start(
    resource_group_name=os.environ["AZURE_RESOURCE_GRP_NAME"],
    container_group_name=os.environ["AZURE_CONTAINER_GRP_NAME"],
    environment_variables=[env_var1,env_var2]
    ).result()
logging.info('Cantainer Run: SUCCESSFUL')

Describe alternatives you've considered
I learned that there are ways to set up environment variables for a task-based container, however, it requires users to create a new Container Group each run, which takes a lot of time and requires feeding in all config of the container each time.
https://learn.microsoft.com/en-us/python/api/overview/azure/container-instance?view=azure-python#create-task-based-container-group
Additional context
N/A

Hey, thanks for the input. Seems like this is something that the ARM API would need to account for when starting the container group. I'll let @msyyc or someone else from the mgmt team chime in with some guidance or insights on the feasibility of this feature request.

Hi @eses-wk, Python SDK is generated automatically with swagger. Unfortunately, the swagger definition of ContainerInstanceManagementClient.container_groups.begin_start() doesn't accept environment variables. The only method is update the existing container groups like:
...
# update evnironment variables for each container under the container group
env_var1 = EnvironmentVariable(name='ENV_NAME1', value='ENV_VALUE1')
env_var2 = EnvironmentVariable(name='ENV_NAME2', value='ENV_VALUE2')

container_group = client.container_groups.get(resource_group_name="XXX", container_group_name="XXX")
for container in container_group.containers:
    container.environment_variables = [env_var1, env_var2]
container_group_parameters = client.container_groups.begin_create_or_update(
    resource_group_name="",
    container_group_name="",
    parameters=container_group
).result()

...

It update (not create) the existing container group so it shall faster than create. If you still want that begin_start can accept environment_variables, please open issue in https://github.com/Azure/azure-rest-api-specs/issues. Once swagger adds the feature, Python SDK can release a new version to support it.

@msyyc your recommendation works great for me!
Just pointing out one typo: parameters attribute in the begin_create_or_update() should be container_group. Referring to the doc
For others, you may check out this doc to learn about what other container settings can be changed
Closing this issue for now.


gridding.py No such file or directory: 'Desktop/MacroPlacement-main/CodeElements/Gridding/test/rtl_mp/ariane.hgr.outline'

hi ,
i've run up-to-dated macroplacement flow (add code flow),
openroad is install and pass recursive check in my machine, and openroad (setup.tcl) is pointed to my local build path.
I found error in:
gridding = GriddingLefDefInterface(gridding_src_dir, design, setup_file, tolerance, halo_width, openroad_exe)  line 297
self.ExtractInputs() line 345
error  is :
FileNotFoundError: [Errno 2] No such file or directory: '/home/*/Desktop/MacroPlacement-main/CodeElements/CodeFlowIntegration/rtl_mp/ariane.hgr.outline'
could you kindly help me check this issue?

here's file generated in rtl_mp
positon.txt
io_net.txt
ariane.hgr.blockage
top_glue_logic_macro_cluster_0_part_0.txt.block
ariane.hgr.net
ariane.hgr.block
top_glue_logic_macro_cluster_0_part_0.txt.net

by the way , i've used Desktop/MacroPlacement-main/CodeElements/Gridding/test/test_gridding.py
the error is the same

Hi diaoerlie,  I think it's related to the openroad binary.  Since we have got many feedbacks about OpenROAD.  Could you do a git pull and build your own OpenROAD binary (https://github.com/ravi-varadarajan/OpenROAD.git),  then retry ?

hi zhi ang,
I've build openroad locally from public git : https://github.com/The-OpenROAD-Project/OpenROAD
do you mean, i should use this git?  https://github.com/ravi-varadarajan/OpenROAD.git

Yes,  the codes are different.

thanks i will try it

it's seems work, let's wait and see

hi zhi ang,
here's what we generated, after python3 flow.py:
ariane.hgr.vertex
ariane.hgr.outline
ariane.hgr
ariane.hgr.part.637
my question is : how to combine (or feed) this output into P&R tools (openroad / innovus)
thank you so much

No.   You cannot do this.  the code elements are used to replicate the gridding/grouping/clustering.   I am not clear what you want.   Do you want to run Circuit Training ?  Or do you want to run Gridding/Grouping/Clustering ?

i‘d like to use circuit training, and use CT's output feed into openroad / innovus tools and run place and route.

@diaoerlie
Just for your reference:
You can use the plc_pb_to_placement_tcl.py script to generate the macro_place.tcl file. This you can source in the innovus shell to generate the macro placement.
Here is an example of generating the macro_place.tcl from the final *.plc file and the protobuf netlist.
python plc_pb_to_placement_tcl.py rl_opt_placement.plc netlist.pb.txt macro_place.tcl

Hi diaoerlie, I think it's related to the openroad binary. Since we have got many feedbacks about OpenROAD. Could you do a git pull and build your own OpenROAD binary (https://github.com/ravi-varadarajan/OpenROAD.git), then retry ?

a suggestion that you might need to change the description of "git clone --recursive https://github.com/The-OpenROAD-Project/OpenROAD.git" to "git clone --recursive https://github.com/ravi-varadarajan/OpenROAD.git"
:)


Faster access

Firstly I love the plugin, it makes my life easier. I was wondering if it would be possible by any means to have a shortcut or an easier way to enter the edit mode, without having to right click and select the option which is always one of the last ones

This is a good idea. I added a new slash command 'markdown-table-editor' in version 1.6.0. I hope it can help you.

.


Incomplete TaskExecutionGraph.allTasks reported in included build

Context
The tasks reported in gradle.taskGraph.whenReady { println("AllTasks: ${allTasks.map { it.path }}") }  are incomplete if running as an included build.
Steps to reproduce

In gradle/gradle repo root.
echo 'gradle.taskGraph.whenReady { println("AllTasks: ${allTasks.map { it.path }}") }' >> build-logic/build.gradle.kts
./gradlew :build-logic:check --dry-run | tee output.txt

You can see a lot of tasks executed, but not displayed in TaskExecutionGraph.allTasks, for example, :build-logic:basics:ktlintTestSourceSetCheck.

This is expected because the task graphs are for a single build and not for the entire set of included builds.
Are you trying to get test failures/code quality failures from the included builds?

the task graphs are for a single build and not for the entire set of included builds

What does single build mean here? We can get the Gradle instance of included build, but the Gradle.taskGraph is complete, this is expected?


Replace 'index.ts' with 'index.d.ts' for namespace declaration

If the namespace declaration is put into index.ts file it is not detected by TS properly. Putting it into index.d.ts file fixes the issue and TS stops complaining.

Closing as this seems no longer relevant with new text.


do not pin opentelemetry-* dependencies

Currently, it is not possible to install uptrace==1.10.1 alongside with opentelemetry-instrumentation-requests==0.30b1.
The conflicts is that opentelemetry-instrumentation-requests requires opentelemetry-api~=1.3 while this project requires opentelemetry-api==1.11.1

Thanks for the report. I've committed a change that does not pin the opentelemetry-api version any more - https://github.com/uptrace/uptrace-python/blob/master/setup.cfg#L30 . I guess we still need to pin the SDK, because OpenTelemetry still does incompatible changes there...
And I've also released v1.11.1 - please let me know how that works.

Thank you for such a fast update. You rule !
Installation now works in my project


feat: pass software version from abci to node info

Closes: #1035

What's the 122 mean?

TBH I'm not sure


Limit axes to the data domain

Currently both HoloViews and GeoViews allow zooming out beyond the actual domain of the data which is at best useless and sometimes annoying. While tile sources repeat the data is usually projected only within the -180 to 180 longitude range or similar. We should allow defining hard bounds on the axes and maybe enable them by default.

This has been done.

Can you point to the solution?

It's automatic, i.e. the user doesn't do anything.

There isn't even a way to turn it off? Seems like it still needs a link to some PR or to some section of the manual that tells people how to control it.

Oh, didn't read very closely since we were triaging so many issues. This issue shouldn't have been closed. There is not yet a way to set hard bounds, it just sets the hard bounds to the max extent of the projection.


Add clarifications to Contributing Guidelines

Description
After rust code was moved into the quickwit subdirectory some of the contributing guideline instructions no longer work. This commit tries to clarify these instructions by adding missing prerequisites, specifying directories where certain commands should be executed and adding config parameter where needed.
Closes #2668
How was this PR tested?
Just followed the instructions on my machine.

 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.