Performance investigation

For Reviewers
I changed some things, I would like to have reviewer input on what is acceptable:

more fine grained triggering of agents on put_partial: only notify agents that are in the increment
put_version and put_partial no longer wait for auto deploy to be completed this may break wait conditions in tests everywhere
increment cache is pre sorted per agent (slower on release (done once), faster for every agent (done often))
increment cache now refuses to move back to older versions
micro optimizations to use the DB more efficiently

** Do I need more tests anywhere? **
close #7262
Self Check:
Strike through any lines that are not applicable (~~line~~) then check the box

[x] Attached issue to pull request
[x] Changelog entry
[x] Type annotations are present
[x] Code is clear and sufficiently documented
[ ] No (preventable) type errors (check using make mypy or make mypy-diff)

src/inmanta/config.py:219: error: Argument 1 to "CronTab" has incompatible type "str | int"; expected "str"  [arg-type]
src/inmanta/config.py:311: error: Incompatible default for argument "validator" (default has type "Callable[[str], str]", argument has type "Callable[[str | T], T]")  [assignment]
src/inmanta/data/__init__.py:4982: error: Argument 1 to "loads" has incompatible type "object"; expected "str | bytes | bytearray"  [arg-type]
src/inmanta/data/__init__.py:5441: error: Signature of "get_list" incompatible with supertype "BaseDocument"  [override]
src/inmanta/data/__init__.py:5441: note:      Superclass:
src/inmanta/data/__init__.py:5441: note:          @classmethod
src/inmanta/data/__init__.py:5441: note:          def get_list(cls, *, order_by_column: str | None = ..., order: str | None = ..., limit: int | None = ..., offset: int | None = ..., no_obj: bool | None = ..., lock: RowLockMode | None = ..., connection: Connection | None = ..., **query: object) -> Coroutine[Any, Any, list[ConfigurationModel]]
src/inmanta/data/__init__.py:5441: note:      Subclass:
src/inmanta/data/__init__.py:5441: note:          @classmethod
src/inmanta/data/__init__.py:5441: note:          def get_list(cls, *, order_by_column: str | None = ..., order: str | None = ..., limit: int | None = ..., offset: int | None = ..., no_obj: bool | None = ..., lock: RowLockMode | None = ..., connection: Connection | None = ..., no_status: bool = ..., **query: object) -> Coroutine[Any, Any, list[ConfigurationModel]]
src/inmanta/data/__init__.py:5509: error: Argument 2 to "_get_status_field" of "ConfigurationModel" has incompatible type "object"; expected "str"  [arg-type]
src/inmanta/data/__init__.py:5513: error: Argument 1 to "append" of "list" has incompatible type "ConfigurationModel"; expected "dict[str, object]"  [arg-type]
src/inmanta/data/__init__.py:5514: error: Incompatible return value type (got "list[dict[str, object]]", expected "list[ConfigurationModel]")  [return-value]
src/inmanta/data/__init__.py:5796: error: "object" has no attribute "__iter__"; maybe "__dir__" or "__str__"? (not iterable)  [attr-defined]
src/inmanta/data/__init__.py:5886: error: "object" has no attribute "__iter__"; maybe "__dir__" or "__str__"? (not iterable)  [attr-defined]
src/inmanta/data/__init__.py:5962: error: Incompatible types in assignment (expression has type "object", variable has type "int")  [assignment]
src/inmanta/server/services/orchestrationservice.py:849: error: Argument 1 to "add_background_task" of "TaskHandler" has incompatible type "Coroutine[Any, Any, tuple[int, dict[str, Any] | None]]"; expected "Coroutine[object, None, Result | None]"  [arg-type]
src/inmanta/server/services/compilerservice.py:795: error: Incompatible types in assignment (expression has type "bool | int | float | str | dict[str, str | int | bool]", variable has type "int")  [assignment]
Generated HTML report (via XSLT): /home/wouter/projects/inmanta/mypy/index.html

[x] Sufficient test cases (reproduces the bug/tests the requested feature)
[x] Correct, in line with design
[ ] End user documentation is included or an issue is created for end-user documentation (add ref to issue here: )
[ ] If this PR fixes a race condition in the test suite, also push the fix to the relevant stable branche(s) (see test-fixes for more info)

Preliminary results
(on 15k resources)

we cause a storm of agent pulls (each put_partial makes every agent pull, we recompile fast)

[x] make it smarter

increment calculation is both on agent pull path and the release version

very performance sensitive
[x] it pulled in all attributes, so large config, slow increment
[ ] it interferes with itself somehow or mucks up the cache (to be investigated)

[x] make cache invalidation monotonic (only ever allow newer versions)

[x] auto deploy triggering is done in-line: the compile has to wait for it, we could change that

pyinstrument works somewhat as an async profiler, but not to the point where the numbers add up
slow query log, lock timing log and all query log on postgresql help somehwat
we still have some slow queries, basically everything related to the whole version or the part that remains the same

UPDATE resource SET status='deployed' WHERE environment=$1 AND model=$2 AND resource_id =ANY($3)

INSERT INTO resource(
                   environment,
                   model,
                   resource_id,
                   resource_type,
                   resource_id_value,
                   agent,
                   status,
                   attributes,
                   attribute_hash,
                   resource_set,
                   provides
               )(
                   SELECT
                       r.environment,
                       $3,
                       r.resource_id,
                       r.resource_type,
                       r.resource_id_value,
                       r.agent,
                       (
                           CASE WHEN r.status='undefined'::resourcestate
                           THEN 'undefined'::resourcestate
                           ELSE 'available'::resourcestate
                           END
                       ) AS status,
                       r.attributes AS attributes,
                       r.attribute_hash,
                       r.resource_set,
                       r.provides
                   FROM resource AS r
                   WHERE r.environment=$1 AND r.model=$2 AND r.resource_set IS NOT NULL AND NOT r.resource_set=ANY($4)
               )
               RETURNING resource_id, resource_set

  SUM(CASE WHEN r.status NOT IN($1,$2) THEN 1 ELSE 0 END) AS done,
	                           to_json(array(SELECT jsonb_build_object('status', r2.status, 'id', r2.resource_id)
	                                         FROM resource AS r2
	                                         WHERE c.environment=r2.environment AND c.version=r2.model
	                                        )
	                           ) AS status
	                    FROM configurationmodel AS c LEFT OUTER JOIN resource AS r
	                    ON c.environment = r.environment AND c.version = r.model
	                    WHERE c.environment=$3
	                    GROUP BY c.environment, c.version

INSERT INTO public.resourceaction_resource (resource_id, resource_version, environment, resource_action_id) SELECT unnest($1::text[]), unnest($2::int[]), $3, $4

Current time taken over parts of put_partial
2024-02-28 17:41:57,308 performance              WARNING STARTING PUT PARTIAL
2024-02-28 17:41:57,312 performance              WARNING INPUT VALIDATION: 0.0035941700043622404
2024-02-28 17:41:57,441 performance              WARNING LOAD STAGE: 0.1291558850207366
2024-02-28 17:41:57,802 performance              WARNING MERGE STAGE: 0.3613146049901843
2024-02-28 17:41:59,651 performance              WARNING PUT STAGE: 1.849367157992674
2024-02-28 17:42:01,870 performance              WARNING AUTO DEPLOY STAGE: 2.218535807012813

Processing this pull request

Merged into branches master in 3c05ea24fe9f4679a7761e00fb2e64f279d72c90


The jobs.*.runs-on documentation seems incomplete

Code of Conduct

[X] I have read and agree to the GitHub Docs project's Code of Conduct

What article on docs.github.com is affected?
The description of jobs.*.runs-on in the main article on workflow syntax reads, to me at least, as though the valid forms for runs-on are either:

a string, being one of the names of a GitHub-provided runner; or
an array, whose first element is the string self-hosted, followed by other arbitrary strings.

All of the examples I can find in the other pages relating to labels on self-hosted runners appear to follow this specification.
What changes are you suggesting?
I would appreciate it if the documentation matched observed behaviour, describing what the actual requirements are for the runs-on field, so that the validation tool I maintain can accurately detect valid and invalid syntax.  From what I've seen, the valid syntax appears to be "any arbitrary string or array of strings", but I'm extrapolating from a very limited set of data points, as I don't use self-hosted runners myself.
Additional information
I maintain a tool which validates GitHub workflows against a JSON schema (amongst other checks), and I recently received a bug report that workflows that GitHub accepts as valid were being rejected by my tool, because the runs-on specification doesn't match the documented syntax.
As far as I can tell, the bug report is valid, insofar as a workflow that does not match the published documentation does, indeed, appear to be accepted by GitHub and run successfully.

@mpalmer
Thanks so much for opening an issue! I'll triage this for the team to take a look :eyes:

Hey there @ramyaparimi I was the one that initially filed the bug report to @mpalmer
Please let me know if there is any other information I can give about this issue.

Hi @mpalmer! Thanks for letting us know about this. It looks like these docs were written for how we would want users to use this key, rather than what's possible. I agree that this isn't great, and I'll open a PR to fix what is the required syntax vs what we recommend for users.

PR opened: https://github.com/github/docs/pull/12504

Hey @lucascosti thanks for looking into this. I'm still a little confused... If I have two self-hosted runners tied to the same repository, and I want to install different dependencies on each runner, I have been using the 'labels' parameter in the entrypoint of my runners when spinning them up, to give each runner a unique label. Then I'll use those unique labels in the 'runs-on' section of an action that I want to run on a self-hosted runner. If the desired functionality is to use self-hosted as a label, would I use both self-hosted and my custom label in the runs-on array? Is that array being evaluated as an OR or an AND between these values?
For instance, I'll have a runner with only linting tools that is small and light weight, and another runner dedicated to building containers that will have any and all tools my jobs might need.

Hi @mathew-fleisch!  It's an AND.
All of your runners implicitly have a self-hosted label added to them.  You can add additional unique labels to your runners when you configure them.
The GitHub hosted runners will never have self-hosted, so if you want to target any self-hosted runner, regardless of configuration, you can specify only runs-on: self-hosted.  This is really only useful if you have a single set of runner types, maybe even just a single runner, and you want to run your workflows on it.
You can also specify runs-on: [ self-hosted, custom ] to run on any self-hosted runner with the custom label (IOW, both the self-hosted AND custom labels).

Sweet. thanks for the clarification. I'll give that syntax a try!

Thanks @lucascosti I have confirmed this syntax works to isolate specific self-hosted runners by label.
@mpalmer This syntax also passes the action-validator 🎉


Add ability to initialize ART for android java fuzzers

Friendly bump @fmeum
And while I have your attention, wondering if it's okay if I open a separate PR to add android as a supported platform in the README, as well as including an example.

@marktefftech Could you resolve the conflicts with @TheCoryBarker's PR? README changes, especially including examples, are highly appreciated!

@marktefftech Some conflicts with main remain. I don't know why the CI checks aren't running, hopefully they will when you rebase.

@fmeum Apologies for the delay, I had to connect with Cory for rebasing strategy.  Looks like the checks ran successfully now.


Build failure: python312Packages.triton-bin (nixpkgs-unstable)

Steps To Reproduce
Steps to reproduce the behavior:

Checkout nixpkgs branch nixpkgs-unstable on latest revision (86e78d3d2084ff87688da662cf78c2af085d8e73 at the time)
Build python312Packages.triton-bin with NIXPKGS_ALLOW_UNFREE=1 nix build .#python312Packages.triton-bin --impure
The build fails

Build log
Full build log: https://pastebin.com/E3aXmcTJ
error: builder for '/nix/store/xff81hcml3vgdr6rnklqmb1sgz07q2nj-python3.12-triton-3.1.0.drv' failed with exit code 1;
       last 25 log lines:
       > /build
       > Running phase: installPhase
       > Executing pypaInstallPhase
       > Successfully installed triton-3.1.0-cp312-cp312-manylinux2014_x86_64.manylinux_2_17_x86_64.whl
       > Finished executing pypaInstallPhase
       > Running phase: pythonOutputDistPhase
       > Executing pythonOutputDistPhase
       > Finished executing pythonOutputDistPhase
       > Running phase: fixupPhase
       > shrinking RPATHs of ELF executables and libraries in /nix/store/3pgvjqnl5c5wxzxppy8p3xpcp2kb3j79-python3.12-triton-3.1.0
       > shrinking /nix/store/3pgvjqnl5c5wxzxppy8p3xpcp2kb3j79-python3.12-triton-3.1.0/lib/python3.12/site-packages/triton/_C/libproton.so
       > shrinking /nix/store/3pgvjqnl5c5wxzxppy8p3xpcp2kb3j79-python3.12-triton-3.1.0/lib/python3.12/site-packages/triton/_C/libtriton.so
       > shrinking /nix/store/3pgvjqnl5c5wxzxppy8p3xpcp2kb3j79-python3.12-triton-3.1.0/lib/python3.12/site-packages/triton/backends/nvidia/bin/cuobjdump
       > shrinking /nix/store/3pgvjqnl5c5wxzxppy8p3xpcp2kb3j79-python3.12-triton-3.1.0/lib/python3.12/site-packages/triton/backends/nvidia/bin/nvdisasm
       > shrinking /nix/store/3pgvjqnl5c5wxzxppy8p3xpcp2kb3j79-python3.12-triton-3.1.0/lib/python3.12/site-packages/triton/backends/nvidia/bin/ptxas
       > checking for references to /build/ in /nix/store/3pgvjqnl5c5wxzxppy8p3xpcp2kb3j79-python3.12-triton-3.1.0...
       > patching script interpreter paths in /nix/store/3pgvjqnl5c5wxzxppy8p3xpcp2kb3j79-python3.12-triton-3.1.0
       > shrinking RPATHs of ELF executables and libraries in /nix/store/5pi24hh71yafcxdvfzggviw018j2yshl-python3.12-triton-3.1.0-dist
       > checking for references to /build/ in /nix/store/5pi24hh71yafcxdvfzggviw018j2yshl-python3.12-triton-3.1.0-dist...
       > patching script interpreter paths in /nix/store/5pi24hh71yafcxdvfzggviw018j2yshl-python3.12-triton-3.1.0-dist
       > Rewriting #!/nix/store/wfbjq35kxs6x83c3ncpfxdyl5gbhdx4h-python3-3.12.6/bin/python3.12 to #!/nix/store/wfbjq35kxs6x83c3ncpfxdyl5gbhdx4h-python3-3.12.6
       > wrapping `/nix/store/3pgvjqnl5c5wxzxppy8p3xpcp2kb3j79-python3.12-triton-3.1.0/bin/proton'...
       > Rewriting #!/nix/store/wfbjq35kxs6x83c3ncpfxdyl5gbhdx4h-python3-3.12.6/bin/python3.12 to #!/nix/store/wfbjq35kxs6x83c3ncpfxdyl5gbhdx4h-python3-3.12.6
       > wrapping `/nix/store/3pgvjqnl5c5wxzxppy8p3xpcp2kb3j79-python3.12-triton-3.1.0/bin/proton-viewer'...
       > chmod: cannot access '/nix/store/3pgvjqnl5c5wxzxppy8p3xpcp2kb3j79-python3.12-triton-3.1.0/lib/python3.12/site-packages/triton/third_party/cuda/bin/ptxas': No such file or directory
       For full logs, run 'nix log /nix/store/xff81hcml3vgdr6rnklqmb1sgz07q2nj-python3.12-triton-3.1.0.drv'.

Additional context
Make python312Packages.torch-bin build fail, which is annoying because it supports cuda and is faster to build than python312Packages.torch with cudaSupport enabled (with cachix cuda-maintainers enabled).
Notify maintainers
@junjihashimoto
Metadata

system: "x86_64-linux"
host os: Linux 6.6.58, NixOS, 24.11 (Vicuna), 24.11.20241025.18536bf
multi-user?: yes
sandbox: yes
version: nix-env (Nix) 2.24.9
nixpkgs: /nix/store/wb6agba4kfsxpbnb5hzlq58vkjzvbsk6-source

Add a :+1: reaction to issues you find important.

Related to issue #350117 which did not fix the problem for *-bin packages

I am running git bisect to find the problematic commit. Note that the last time I updated my nixpkgs revision, python 3.12 was not supported by triton-bin yet. I am running git bisect on the python311Packages.triton-bin derivation, which exhibits the same problem as python312Packages.triton-bin right now. I'll edit this message once I'm done.

This commit c5eb288cc7c4ef49f10edf4a393b61c8d57dbf18 by @DerDennisOP seems to be the culprit, perhaps the new remote triton sources do not have the same folder structure, hence why the postFixup commands fail

Will be fixed by pr #351627

In the meantime, you can use this overlay
(final: prev: {
  python312 = prev.python312.override {
    packageOverrides = final: prevPy: {

      triton-bin = prevPy.triton-bin.overridePythonAttrs (oldAttrs: {
        postFixup = ''
          chmod +x "$out/${prev.python312.sitePackages}/triton/backends/nvidia/bin/ptxas"
          substituteInPlace $out/${prev.python312.sitePackages}/triton/backends/nvidia/driver.py \
            --replace \
              'return [libdevice_dir, *libcuda_dirs()]' \
              'return [libdevice_dir, "${prev.addDriverRunpath.driverLink}/lib", "${prev.cudaPackages.cuda_cudart}/lib/stubs/"]'
        '';
      });
    };
  };
  python312Packages = final.python312.pkgs;
})


Rx Buffer operator with count is not working as expected

When using the buffer operator (using count as a parameter) to batch messages from the message queue. My Subscribe implementation never fires unless the buffer number is equal to or less than the number of messages i.e. Buffer(2) and I publish 2 messages then it works. I have moved the implementation to use TimeSpan instead and that works fine.
            var busControl = Bus.Factory.CreateUsingRabbitMq(cfg =>
            {
                var host = cfg.Host(new Uri("rabbitmq://localhost/"), h =>
                {
                    h.Username("guest");
                    h.Password("guest");
                });

                cfg.ReceiveEndpoint(host, "customer_update_queue", e =>
                {
                    var observer = new ObservableObserver<ConsumeContext<Customer>>();
                    observer.Buffer(5).Subscribe(value =>
                    {
                        foreach (var consumeContext in value)
                        {
                            Console.WriteLine("Content: {0}", consumeContext.Message.Content);
                        }
                    });

                    e.Observer(observer);
                });
            });

            busControl.Start();

            busControl.Publish(new Customer() { Content = "2" });
            busControl.Publish(new Customer() { Content = "3" });
            busControl.Publish(new Customer() { Content = "4" });
            busControl.Publish(new Customer() { Content = "5" });
            busControl.Publish(new Customer() { Content = "6" });
            busControl.Publish(new Customer() { Content = "7" });
            busControl.Publish(new Customer() { Content = "8" });
            busControl.Publish(new Customer() { Content = "9"});

            Thread.Sleep(5000);

            busControl.Stop();

I'm not sure this is an issue with MassTransit. I'm unfamiliar with Rx, but if you tell it to buffer 5 messages, won't it wait until it has five messages? Using a TimeSpan seems like a much smarter choice here if your intent is to batch work to reduce round trips to something like a database.

Yeah, your right it's more about how you use Rx with MassTransit. Yes. it will buffer 5 at a time until the count is less than 5 and then wait for more items or if _observer.OnCompleted() is called then it will process the remaining items and finish.
The reason was that I was calling _observer.OnCompleted() in my previous implementation but since I've moved to MassTransit it would be a continuious stream, so Timespan is the right option.
Thanks for the feedback,
Calum

SO can this be closed?

Yes, thanks.

@calumbett, I found https://github.com/Reactive-Extensions/Rx.NET/issues/323 issue during working with Rx(coincidentally also with MassTransit).

@NikGovorov thanks for the heads up.


Add u64 ExecutionId type

Description

Testing Instructions

[!WARNING]
This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

#8771  👈
#8769
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @bgw and the rest of your teammates on  Graphite

Merge activity

Jul 17, 11:57 AM EDT: @bgw started a stack merge that includes this pull request via Graphite.


SpringRoll Seed Template Project Links Broken?

When I click on the links for "PIXI", "Phaser 3", and "CreateJS" in the "Seed Project" section of the README, I get taken to the same Springroll-Seed repo as if I had just clicked on the "Seed Project" link. Are there separate repositories for the PIXI, Phaser 3, and CreateJS template projects?

They are all within the same project but on different branches. The links do take you to the correct branches, but I'll admit it's not very obvious as the structure is mostly the same.

On that note, I think this would be a good opportunity to update the individual READMEs for the individual templates to reflect the differences. I'll leave this open for the ticket to encompass that bit of work. I'm also going to move this thread over to SpringRoll-Seed, since that's where the issue lies.


[Impeller] libImpeller: Wire up Metal and Vulkan backends.

(I converted this to an issue so that I can enable notifications on this item)

so that I can enable notifications on this item

Ah, yeah. I had made placeholders on the dashboard. Anyway, this is being completed in https://github.com/flutter/engine/pull/56906 to round of the support for the MVP.


Function type : custom usage

Can a function call .net  method using function type as 'custom' ?

Not really. Functions in the Serverless Workflow spec are actually "serverless" functions, not "local" functions of your code. That being said, you could probably implement your own Processor. Know that we invision a new version of Synapse that should be way more flexible on the type of actions you could run but it's still in a conceptual phase (and I know it's been a while but we want to take our time to put the right pieces together).
That being said, may I ask which .NET method are you trying to call ?

@mremadevi With 1.0.0 Serverless Workflows enables you to run any process you'd like, using containers, scripts or shell. Another couple of weeks and a new Synapse release supporting said feature will see the day ;)
Does that answer your question?

@mremadevi I'm gonna assume my last comment answered your question and will therefore close this issue.
Feel free to reopen if need be ;)


IE11 Seek not working

Expected: open http://animejs.com/documentation/#seekAnim in IE11 and works as in any other browser
Actual result: no animation when using progress bar to seek. if timeline controls are used, play, pause, restart work. However the progress bar does nothing.

Edge and Internet Explorer have an inbuilt transition animation of about 0.2 seconds on their range inputs. So each call to update the range value starts a new transition request.
update: function(anim) { controlsProgressEl.value = anim.progress; }
This results in a divergence from the position of the timeline and the position of the thumb in the input type=range.

You'll have to look for 'change' as well as 'input' to get this working on IE and edge
controlsProgressEl.addEventListener('change', function() {   TLcontrols.seek(TLcontrols.duration * (controlsProgressEl.value / 100)); });

Fixed.


test: enable r2r encoding test for gst-vaapi

Signed-off-by: Wang Zhanjun<EMAIL_ADDRESS>

Please ensure all usage of gen_name have been updated accordingly. This detail was missed for ffmpeg and had to be fixed post-merge (#71)

have fixed it, thanks


Break long words or URLs to prevent horizontal overflow

Note word-wrap is an alias for the standard's overflow-wrap. Shouldn't need to be changed, though.


Simplify gotek wiring

Could you emulate a second pcf8574 with FF-OSD and feed Amiga keys  to it ?
Could you add support to FlashFloppy to support buttons on a pcf8574 ?
If so, we can eliminate 2 or 3 wires between them and everything is done over i2c.
As a bonus, you could add support for more buttons on the gotek

up down left right.
a shift key for page up/down

Since we don't emulate a pcf8574 when talking to latest FlashFloppy, the protocol can be extended to be bidirectional. We'd probably want to version the protocol at the same time, and continue to handle already-installed OSDs which don't support bidirectional transfer over I2C. That can't be hard to detect.
So yes, I suppose then I'd need to hook up the new 'buttons' to do something. And I'm not sure the extra three wires at the moment is so much of a pain. The FF-OSD board will probably getsintegrated into an enhanced Gotek, or glued on the back of a standard one, anyway: So the wiring disappears or is static.

I have a proof of concept going for this. So from next version of FF OSD I will disable Gotek button pins when in extended protocol mode, and send the button info via i2c instead. This will also require a new version of Flashfloppy of course, which will precede the next version of FF OSD.

I spent some time implementing this only to realise at the very end that having physical connection to the buttons is very useful for triggering firmware update. The bootloader makes a very early decision whether to execute or update based on reading GPIOs.
The only ways round I can see are:

You would only need one button wire, for SELECT/JA. That suffices to enter the updater.
Could add a menu option in the main firmware, to run the updater. This could work in a few ways:

Erase first page of firmware, then reset. Bootloader detects this already.
Write to the backup registers (preserved across reset) and then reset. Modify bootloader to read the backup registers.
Jump at the bootloader direct. Possibly harder than it seems since the chip state is not reset, as the bootloader can currently assume.

FWIW - and I’m a stm newbie “Write to the backup registers“ seems the cleanest.
You could also have the FF validate/verify that the USB stick has a valid UPD file.

This is all done as of FF OSD v1.7 and FlashFloppy v3.5a.
The old A3-A5 connections are no longer supported and must be removed.
Latest FF does not work with older FF OSD due to I2C changes.


Debugging apparent connection failure in node lib

I have been wanting to move a bunch of queries into a lib to start cleaning up some app code that has started to get complex and hard to read.  I might well have some generic JS issue here, but I can't see what it is or understand why I am getting no connection errors. They only clue I have is that if I print out the Mariasql object, I see this:
sql  { threadId: undefined,
connected: false,
connecting: false,
destroyed: false,
pingInterval: 60000,
_connecting: true,
_destroying: false,
_keepQueries: undefined,
_pinger: undefined,
....
_connecting:true.... It seems like something is interuppting the connect and that leads to no error report. I did try hard coding the connect values with no change.  The DB is up and running of course(is it plugged in :) ) and other apps can connect using MariaSQL. It is also a MariaSQL server:
Server version: 10.1.2-MariaDB-1~trusty-wsrep-log mariadb.org binary distribution, wsrep_25.10.r4123
I also tried debug and single steping through the code, but I don't see any errors that way either.
Any thoughts or "look here dummy" comments greatly appreciated.
Thanks,
Eric

dbInit.js
var config = require('config');
var inspect = require('util').inspect;
var sql = require('mariasql');

var dbQuery = function () {};

    dbQuery.prototype.log = function () {
      console.log('dbQuery!');
    };

dbQuery.prototype.init = function () {
   if ( !process.env.SERVER_ENV ) {
   console.log("Missing SERVER_ENV SERVER_ENV=env-name node service.js ");
   process.exit(1);
   }

   var apiConfig = config.get('apiConfig');

   if (!apiConfig[process.env.SERVER_ENV] ){
     console.log("Missing SERVER_ENV or not in config SERVER_ENV=env-name node service.js ");
     process.exit(1);
   }

   var myConfig = apiConfig[process.env.SERVER_ENV]; // now my custom config
   console.log("My Config ",myConfig);

   var c = new sql();
   c.connect({
     host: myConfig.dbConfig['host'],
     user: myConfig.dbConfig['user'],
     password: myConfig.dbConfig['pass'],
     db: myConfig.dbConfig['db'],
     connTimeout: 60
   });

   c.on('connect', function() {
     console.log('Client connected');
   })
   .on('error', function(err) {
     console.log('Client error: ' + err);
   })
   .on('close', function(hadError) {
     console.log('Client closed');
   });

   global.vars = {updated:false,value:[],List: []};

   c.query('select Idx, Name FROM table where rs_id="main"')
    .on('result', function(res) {
      res.on('row', function(row) {
        console.log("row");
        global.evars.value[row.Idx] = (row.Name);
      })
      .on('error', function(err) {
        console.log('Result error: ' + inspect(err));
      })
      .on('end', function(info) {
        console.log('Result finished successfully');
      });
    })
    .on('end', function() {
      console.log('Done with all results vars');
      global.evars.updated=true;
      c.emit("DONE");
    });
console.log("sql ",c,global.vars);

dbQuery.prototype.sqlConnection = c;
dbQuery.prototype.config = myConfig;

}

exports.dbQuery = dbQuery;

In the Main app:
var qfd = require('./dbInit.js').dbQuery;

var dbQuery = new qfd;
dbQuery.log();
dbQuery.init();

Can you please try the rewrite branch and let me know if you still encounter problems?

I can't seem to find a way to install just via that branch.  It seems like I should be able to use something like npm rebuld, but not finding the way that works yet.  Thanks.

You can use tarball urls with npm: npm install https://github.com/mscdex/node-mariasql/tarball/rewrite

I did git clone and then cd into the node-mariasql dir and got this:
eric@dev:node_modules$ git clone https://github.com/mscdex/node-mariasql.git
Cloning into 'node-mariasql'...
remote: Counting objects: 2171, done.
remote: Total 2171 (delta 0), reused 0 (delta 0), pack-reused 2171
Receiving objects: 100% (2171/2171), 6.05 MiB | 0 bytes/s, done.
Resolving deltas: 100% (875/875), done.
Checking connectivity... done.
eric@dev:node_modules$ cd node-mariasql/
eric@dev:node-mariasql$ npm install
\

<EMAIL_ADDRESS>install /home/eric/dbstuff/serviceAPI/node_modules/node-mariasql
node-gyp rebuild

gyp ERR!  System Linux 3.13.0-43-generic
gyp ERR! command "node" "/usr/local/lib/node_modules/npm/node_modules/node-gyp/bin/node-gyp.js" "rebuild"
gyp ERR! cwd /home/eric/dbstuff/serviceAPI/node_modules/node-mariasql
gyp ERR! node -v v0.10.32
gyp ERR! node-gyp -v v1.0.1
gyp ERR! This is a bug in node-gyp.
gyp ERR! Try to update node-gyp and file an Issue if it does not help:
gyp ERR!     https://github.com/TooTallNate/node-gyp/issues
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 7
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is most likely a problem with the mariasql package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node-gyp rebuild
npm ERR! You can get their info via:
npm ERR!     npm owner ls mariasql
npm ERR! There is likely additional logging output above.
npm ERR! System Linux 3.13.0-43-generic
npm ERR! command "/usr/local/bin/node" "/usr/local/bin/npm" "install"
npm ERR! cwd /home/eric/dbstuff/serviceAPI/node_modules/node-mariasql
npm ERR! node -v v0.10.32
npm ERR! npm -v 1.4.28
npm ERR! code ELIFECYCLE
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     /home/eric/dbstuff/serviceAPI/node_modules/node-mariasql/npm-debug.log
npm ERR! not ok code 0

Sorry just missed your comment:
eric@dev:node_modules$ npm install https://github.com/mscdex/node-mariasql/tarball/rewrite
npm WARN package.json<EMAIL_ADDRESS>No README data
FATAL ERROR: Evacuation Allocation failed - process out of memory
Aborted
This is only a 1G EC2 instance, but I wouldn't expect to actually need more memory for this path of install.

I'm not familiar enough with EC2, but it installs fine on my Linux box with node v0.10 installed. Maybe something else on your system is using up too much memory and/or there is no swap configured? I don't know...

It was really a lack of memory, got it building now after shuting down MySQL :) Will report back on results


Beta Preview

All Submissions:

[ ] Have you followed the guidelines stated in CONTRIBUTING.md file?
[ ] Have you checked to ensure there aren't any other open Pull Requests for the desired changed?

Type of change
Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

New Feature Submission:

[ ] Does your submission pass the tests?
[ ] Have you linted your code locally prior to submission?

Changes To Core Features:

[ ] Have you added an explanation of what your changes do and why you'd like us to include them?
[ ] Have you written new tests for your core changes, as applicable?
[ ] Have you successfully ran tests with your changes locally?

After these steps, you're ready to open a pull request.
a. Give a descriptive title to your PR.

b. Describe your changes.

c. Put `closes #XXXX` in your comment to auto-close the issue that your PR fixes (if such).

Looks like I need to rebase on main and recommit my changes, I'll put up a new pr for this.


Feat: Add retry to attestation cert fetch

Description
There have been cases where there is failure when attempting to remote fetch AMD SEV SNP certificates for attestation checks. executeWithRetry is no wrapping the request to allow for retry if the operation fails
Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change.
Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

putting to draft as i work out the failing connection tests

Closing for an alternate strategy of converging environments on the CORS proxy url to prevent AMD rate limiting certificate fetching.


Endpoint client API

Client code (for example, in Next.js or Gatsby) currently use libraries such as Axios to access the REST endpoints. Let's work towards an endpoint client API that's easier to work with to eliminate boilerplate code from clients.

This is interesting! What kind of boilerplate clients encounter? What kind of features should make our hypothetical client API different from a generic HTTP library? My routing PR evolves Chiselstrike towards a generic HTTP server, but maybe this is not a good idea and we should make our endpoints more restricted to provide a more convenient client interface.

@honzasp The boilerplate is often around "duplicating" the endpoints and models in the front-end because you want wrappers for Axios calls and some sort of structure for your data. Pulling the entity classes it bit hard because they depend on ChiselEntity.
I don't see a problem with #1618 because it's great for writing general purpose REST APIs. This issue is more about reducing boilerplate for the simple cases than restricting the ChiselStrike interface as such.


Broken Code Example in form-validation page

https://vuejs.org/v2/cookbook/form-validation.html
This one's not a big deal and I'd be happy to fix it.
At the link shown above, the second code example says
<form
  id="app"
  @submit="checkForm"
  action="https://vuejs.org/"
  method="post"
  novalidate="true"
>

    Please correct the following error(s):

{{ error }}

    Name
    <input
      id="name"
      v-model="name"
      type="text
      name="name"
    >

    Email
    <input
      id="email"
      v-model="email"
      type="email"
      name="email"
    >

    Favorite Movie
    <select
      id="movie"
      v-model="movie"
      name="movie"
    >
      <option>Star Wars</option>
      <option>Vanilla Sky</option>
      <option>Atomic Blonde</option>
    </select>

    <input
      type="submit"
      value="Submit"
    >

</form>

The key spot here is the
 <input
      id="name"
      v-model="name"
      type="text
      name="name"
    >

type="text" is missing its closing quotation mark.
Again, not a big deal, but for the convenience of others using the page, I'd be happy to fix it up.

Sure! We'd be happy to accept a PR for that. Thanks!

Merged the fix! Thanks @DrLeeroyPhD.


Spaces in path

These changes are a prerequisite for the respective PRs in rakudo and nqp.
See rakudo#3031.

I'm not merging as it's up to you to chose the appropriate moment.


fix: replace _neq by _eq in get last updated

fix #790

:tada: This PR is included in version 1.109.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


UI enhancement of the top right user profile entry

UI improvement.
Follow the image below from Transferwise.com.

duplicated


Test connection: Encrypted private key detected, but no passphrase given

On Ubuntu the Private Key Auth for the "test connection" doesn't work.
The actually connection itself works fine.

Can you give a bit more of details?
I don't use private keys often, so maybe a bit more of description might help others to help solve this issue 👍

I am sorry, I have already uninstalled it. I think it was the button "test connection". It just did nothing

I'm encountering this issue as well, also on Ubuntu. Perhaps I can provide some additional information. I'm attempting to connect to a server via ssh, using my private key. Here is a screenshot with the error.

The passphrase toggle to the right of the private key path does not seem to do anything. It is unclear how to input the passphrase. The password field on the left of the private key path becomes disabled when using a private key, which makes sense, but I don't see any input for the private key passphrase.

Looks like this issue is a duplicate of https://github.com/sqlectron/sqlectron-gui/issues/387


Kmeans fix

What does this PR do?
Fix the partial_fit bug introduced in kmeans.py  which causes only the last batch used for training the k-means. It is related to this bug:
https://github.com/speechbrain/speechbrain/issues/2602
Based on the new experiments with the kmeans trained on all 960 datasets, there is not much difference in the performance of the downstream task. The result of the ablation study will be posted. The kmeans models trained on <1% data, 10%(only using LibriSpeech100 and 100% of data are uploaded in the HF repo:
https://huggingface.co/speechbrain/SSL_Quantization/tree/main

Fixes #<issue_number>

Before submitting

[ ] Did you read the contributor guideline?
[ ] Did you make sure your PR does only one thing, instead of bundling different changes together?
[ ] Did you make sure to update the documentation with your changes? (if necessary)
[ ] Did you write any new necessary tests? (not for typos and docs)
[ ] Did you verify new and existing tests pass locally with your changes?
[ ] Did you list all the breaking changes introduced by this pull request?
[ ] Does your code adhere to project-specific code style and conventions?

PR review

Reviewer checklist

[ ] Is this pull request ready for review? (if not, please submit in draft mode)
[ ] Check that all items from Before submitting are resolved
[ ] Make sure the title is self-explanatory and the description concisely explains the PR
[ ] Add labels and milestones (and optionally projects) to the PR so it can be classified
[ ] Confirm that the changes adhere to compatibility requirements (e.g., Python version, platform)
[ ] Review the self-review checklist to ensure the code is ready for review

@mravanelli  and @TParcollet  I think this PR should be merged.

@poonehmousavi, it looks like the docstring of the new function is not formatted in the same way as the others one. Could you please fix it?

@mir

@poonehmousavi, it looks like the docstring of the new function is not formatted in the same way as the others one. Could you please fix it?

@mravanelli It seems all the checkpoints are passed... I am not sure what to fix

My comment regards the format of the process_chunk.py function. It is different from the standard format for doctring used in speechbrain.

@mravanelli I think it is resolved now


Mouse without borders - Switch device shortcut

Description of the new feature / enhancement
I would like to propose a shortcut to switch to the next device. Once pressed the next device in the list will be activated switching back to the first device once the end of the list is reached.
Scenario when this would be used?
This would emulate the classic KVM switch (apart from the V part). You press a button (my keyboard has assignable keys which I would like to use one for this) and you control another device.
Supporting information
No response

/dup #27713


HandbrakeCLI: "MP4ERROR: File size exceeded 4 GB; output unplayable."

Just ran the latest transcode-video.sh on a Blu-ray movie rip (Snatch, 102 min) with settings --big --no-surround --burn 12 --crop 20:20:0:0, and the transcode ended with the log below. The output MP4 is 5.74 GB and neither QuickTime nor VLC are willing to play it.
Is enabling "Large file size" my bug or yours?
x264 [info]: ref P L0: 58.5% 12.3% 18.7% 10.0%  0.5%
x264 [info]: ref B L0: 81.8% 17.0%  1.3%
x264 [info]: ref B L1: 97.6%  2.4%
x264 [info]: SSIM Mean Y:0.9723527 (15.583db)
x264 [info]: PSNR Mean Y:44.493 U:50.391 V:52.480 Avg:45.647 Global:44.848 kb/s:7315.16
[00:33:13] dca-decoder done: 0 frames, 0 decoder errors, 0 drops
Muxing: this may take awhile...MP4ERROR: File size exceeded 4 GB; output unplayable. Enable "Large file size" to fix it.
MP4ERROR: (m_size <= (uint64_t)0xFFFFFFFF): assert failure
[00:33:13] mux: track 0, 147232 frames,<PHONE_NUMBER> bytes, 7315.03 kbps, fifo 1024
[00:33:13] mux: track 1, 287851 frames, 125712951 bytes, 163.77 kbps, fifo 2048
[00:33:15] libhb: work result = 0

Encode done!
HandBrake has exited.

real	97m38.779s
user	743m5.884s
sys	5m48.836s

Never mind, my bug. Old version of HandBrakeCLI was lingering in PATH.

What version of HandBrakeCLI are you using? This often happens with versions prior to 0.10.0, the current stable release.
I actually removed --large-file support in an update on December 30, 2014 because the HandBrake developers removed the option instead of just deprecating it.
You can check your version number on OS X with this command:
strings ~/bin/HandBrakeCLI | grep ^HandBrake | sed -n 1p

Aaaaand... nevermind my comment which was posted 10 seconds after you closed this. :)


Make Basic example from the README runnable in CommonJS environment

The basic example in the README makes many assumptions about user environment and skills like a proper type field set in the closest package.json, or the user has copy-pasted the example to a file with .mjs extension.
In most cases, users will face the following problems:

(node:59910) Warning: To load an ES module, set "type": "module" in the package.json or use the .mjs extension.
SyntaxError: await is only valid in async functions and the top level bodies of modules
No clear output is made, so it's hard to determine whether the example was executed successfully or not

Environment:

Node.js Version:  v19 or any
Redis Server Version: Any
Node Redis Version: Any
Platform: Any

@leibale @shacharPash  Can you guys pair on this?

@uglide You mean to write a more complete code example or what?

Can I work on this?


Adds support for creating/updating entries

Fixes https://github.com/contentful-labs/contentful-go/issues/15
Borrows code from https://github.com/contentful-labs/contentful-go/pull/25
Didn't see any reason why that was closed...so trying again!

Codecov Report

Merging #48 into master will increase coverage by 0.04%.
The diff coverage is 56.52%.

@@            Coverage Diff            @@
##           master     #48      +/-   ##
=========================================
+ Coverage   54.46%   54.5%   +0.04%
=========================================
  Files          13      13
  Lines        1120    1143      +23
=========================================
+ Hits          610     623      +13
- Misses        454     459       +5
- Partials       56      61       +5

Impacted Files
Coverage Δ

entry.go
14.94% <56.52%> (+14.94%)
:arrow_up:

Continue to review full report at Codecov.

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

@Khaledgarbaya please consider reviewing this feature as currently there is no convenient way to create an Entry using this SDK. CMIIW?

Codecov Report

Merging #48 into master will increase coverage by 0.04%.
The diff coverage is 56.52%.

@@            Coverage Diff             @@
##           master      #48      +/-   ##
==========================================
+ Coverage   54.46%   54.50%   +0.04%
==========================================
  Files          13       13
  Lines        1120     1143      +23
==========================================
+ Hits          610      623      +13
- Misses        454      459       +5
- Partials       56       61       +5

Impacted Files
Coverage Δ

entry.go
14.94% <56.52%> (+14.94%)
:arrow_up:

Continue to review full report at Codecov.

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


bug(mat-radio-button): mat-radio-inner-circle (selected) mat-radio-button not correctly centred in mat-dialog

Is this a regression?

[ ] Yes, this behavior used to work in the previous version

The previous version in which this bug was not present was
No response
Description
When a mat-radio-button is in a scrollable/overflown mat-dialog the mat-radio-inner-circle (selected) mat-radio-button loses centring at certain resolutions/scroll heights.

https://stackblitz.com/edit/angular-material-13-starter-x1xj4z-yamfte?file=src%2Fapp%2Fapp.component.html,src%2Fapp%2Fapp.component.css,src%2Fapp%2Fapp.component.ts,src%2Fapp%2Fapp.module.ts,src%2Fapp%2Fdialog-overview-example-dialog.html,package.json
Reproduction
Steps to reproduce:

Open StackBlitz in Chrome or Edge.
Open developer tools and 'device view'.
Click 'Open the dialog' button
Select a radio button
Adjust resolution/screen height and width.

Expected Behavior
mat-radio-inner-circle centred in the mat-radio-button when adjusting the height/dimensions.
Actual Behavior
mat-radio-inner-circle moves to off-centre in the mat-radio-button
Environment

Angular: 13.3
CDK/Material: 13.3
Browser(s): Chrome/Edge
Operating System (e.g. Windows, macOS, Ubuntu): Windows

We don't backport fixes to Angular v13 anymore. Please try it with Angular 15 or newer.


Upgrade requirements

Codecov Report

Merging #83 (e95e49d) into main (ae57d3f) will decrease coverage by 0.76%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##              main      #83      +/-   ##
===========================================
- Coverage   100.00%   99.23%   -0.77%
===========================================
  Files            6        6
  Lines          260      260
  Branches        51       51
===========================================
- Hits           260      258       -2
- Misses           0        1       +1
- Partials         0        1       +1

Impacted Files
Coverage Δ

src/django_linear_migrations/compat.py
92.85% <0.00%> (-7.15%)
:arrow_down:

Continue to review full report at Codecov.

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


arrow labels in \scriptstyle

Would it be ok to render arrow labels in a smaller font that would correspond to what the Tikz export currently does (\scriptstyle)?

yes, it should be easy to fix (e.g., add \scriptstyle in the generated edge label, either in the elm code or in the custom-element HTML element defined in js)


Provide console native app and web app samples

Repo is created here: https://github.com/casdoor/casdoor-dotnet-sdk-example/issues/1

Maybe fixed by https://github.com/casdoor/casdoor-dotnet-sdk-example/pull/2


Can tensorflow use multiple ps servers to store an embedding variable?

Thank you for reading.
Can tensorflow use multiple parameter servers to store an embedding variable?
My embedding is too big, the bandwidth of a server is not enough.
Which module should I use?
Looking forward to your reply！
Thanks！

@Ljl-Jdsk, Tensorflow supports distribution strategy please take a look at guide for parameter server strategy. Thanks!

Yes, the following code can distribute v1, v2 and v3 to two different parameter servers. But now I want to separate the v1 and store it on two servers. Sorry, can you tell me the specific function? I ’m abroad and I can’t open the guide.
cluster_spec = {
"ps": ["ps0:2222", "ps1:2222"],
"worker": ["worker0:2222", "worker1:2222", "worker2:2222"]}
with tf.device(tf.train.replica_device_setter(cluster=cluster_spec)):
v1 = tf.Variable(...)  # assigned to /job:ps/task:0
v2 = tf.Variable(...)  # assigned to /job:ps/task:1
v3 = tf.Variable(...)  # assigned to /job:ps/task:0

@Ljl-Jdsk, Please find the example for Parameter server strategy
strategy = tf.distribute.experimental.ParameterServerStrategy()
run_config = tf.estimator.RunConfig(
    experimental_distribute.train_distribute=strategy)
estimator = tf.estimator.Estimator(config=run_config)
tf.estimator.train_and_evaluate(estimator,...)

For more read here. Thanks

@Ljl-Jdsk, Did you get a chance to look at the above given link. Thanks!

Closing this issue since the title of the issue is resolved. Thanks!


Adds support for slices parameter

This PR adds support for slices. Simple c&p from the reflect.Array case.
Working with arrays only is possible if you know the size upfront. Which in my case I don't and I have not found any way to convert a slice to an array.

Thank you

Awesome, thank you mattn!


[Calling] Calling-Component-Bindings for Reaction receive component

What

The PR includes the component binding changes for send reaction button
Why

How Tested

Process & policy checklist

[ ] I have updated the project documentation to reflect my changes if necessary.
[ ] I have read the CONTRIBUTING documentation.

Is this a breaking change?

[ ] This change causes current functionality to break.

The pull request is not an idea one. Will update with another PR.


Chinese Localization Summary Issue

Hi! This thread is now closed because it lives here: microsoft/TypeScript-Website-Localizations#5

This issue is for keeping track of the localization effort for zh. You can learn about the whole roadmap in #100. If you see an un-ticked area below, that means there isn't an version of that file in this language.
For this language there are 566 translated files, with 353 TODO.
tsconfig
Done: 51, TODO: 69.

[x] intro.md
[x] Additional_Checks_6176.md
[x] Advanced_Options_6178.md
[x] Basic_Options_6172.md
[x] Command_line_Options_6171.md
[x] Experimental_Options_6177.md
[x] Module_Resolution_Options_6174.md
[x] Project_Files_0.md
[x] Source_Map_Options_6175.md
[x] Strict_Type_Checking_Options_6173.md
[x] Watch_Options_999.md
[x] allowJs.md
[x] allowSyntheticDefaultImports.md
[x] allowUmdGlobalAccess.md
[x] alwaysStrict.md
[x] baseUrl.md
[x] checkJs.md
[x] composite.md
[x] declaration.md
[x] downlevelIteration.md
[x] esModuleInterop.md
[x] importHelpers.md
[x] incremental.md
[x] isolatedModules.md
[x] jsx.md
[x] lib.md
[x] module.md
[x] moduleResolution.md
[x] noEmit.md
[x] noImplicitAny.md
[x] noImplicitThis.md
[x] outDir.md
[x] outFile.md
[x] paths.md
[x] plugins.md
[x] preserveSymlinks.md
[x] removeComments.md
[x] rootDir.md
[x] rootDirs.md
[x] sourceMap.md
[x] strict.md
[x] strictBindCallApply.md
[x] strictFunctionTypes.md
[x] strictNullChecks.md
[x] strictPropertyInitialization.md
[x] target.md
[x] tsBuildInfoFile.md
[x] typeRoots.md
[x] types.md
[x] compilerOptions.md
[x] top_level.md
[ ] allowUnreachableCode.md
[ ] allowUnusedLabels.md
[ ] assumeChangesOnlyAffectDirectDependencies.md
[ ] charset.md
[ ] declarationDir.md
[ ] declarationMap.md
[ ] diagnostics.md
[ ] disableReferencedProjectLoad.md
[ ] disableSizeLimit.md
[ ] disableSolutionSearching.md
[ ] disableSourceOfProjectReferenceRedirect.md
[ ] emitBOM.md
[ ] emitDeclarationOnly.md
[ ] emitDecoratorMetadata.md
[ ] exclude.md
[ ] experimentalDecorators.md
[ ] explainFiles.md
[ ] extendedDiagnostics.md
[ ] extends.md
[ ] fallbackPolling.md
[ ] files.md
[ ] forceConsistentCasingInFileNames.md
[ ] generateCpuProfile.md
[ ] importsNotUsedAsValues.md
[ ] include.md
[ ] inlineSourceMap.md
[ ] inlineSources.md
[ ] jsxFactory.md
[ ] jsxFragmentFactory.md
[ ] jsxImportSource.md
[ ] keyofStringsOnly.md
[ ] listEmittedFiles.md
[ ] listFiles.md
[ ] locale.md
[ ] mapRoot.md
[ ] maxNodeModuleJsDepth.md
[ ] newLine.md
[ ] noEmitHelpers.md
[ ] noEmitOnError.md
[ ] noErrorTruncation.md
[ ] noFallthroughCasesInSwitch.md
[ ] noImplicitReturns.md
[ ] noImplicitUseStrict.md
[ ] noLib.md
[ ] noPropertyAccessFromIndexSignature.md
[ ] noResolve.md
[ ] noStrictGenericChecks.md
[ ] noUncheckedIndexedAccess.md
[ ] noUnusedLocals.md
[ ] noUnusedParameters.md
[ ] out.md
[ ] preserveConstEnums.md
[ ] preserveWatchOutput.md
[ ] pretty.md
[ ] reactNamespace.md
[ ] references.md
[ ] resolveJsonModule.md
[ ] skipDefaultLibCheck.md
[ ] skipLibCheck.md
[ ] sourceRoot.md
[ ] stripInternal.md
[ ] suppressExcessPropertyErrors.md
[ ] suppressImplicitAnyIndexErrors.md
[ ] traceResolution.md
[ ] typeAcquisition.md
[ ] useDefineForClassFields.md
[ ] watchDirectory.md
[ ] watchFile.md
[ ] watchOptions.md

playground
Done: 48, TODO: 26.

[x] sections.json
[x] TypeScript with Deno.ts
[x] TypeScript with Node.js
[x] TypeScript with React.tsx
[x] TypeScript with Web.js
[x] TypeScript with WebGL.js
[x] Function Chaining.ts
[x] Generic Functions.ts
[x] Typing Functions.ts
[x] Errors.ts
[x] Quick Fixes.ts
[x] Code Flow.ts
[x] Functions.ts
[x] Hello World.ts
[x] Objects and Arrays.ts
[x] Async Await.ts
[x] Immutability.ts
[x] Import Export.ts
[x] JSDoc Support.js
[x] Classes 101.ts
[x] Generic Classes.ts
[x] Mixins.ts
[x] This.ts
[x] JavaScript Playgrounds.js
[x] New Compiler Defaults.ts
[x] Automatic Type Acquisition.ts
[x] Fixits.ts
[x] Mobile Support.ts
[x] Sharable URLs.ts
[x] TypeScript Versions.ts
[x] Soundness.ts
[x] Structural Typing.ts
[x] Type Guards.ts
[x] Type Widening and Narrowing.ts
[x] Enums.ts
[x] Nominal Typing.ts
[x] Types vs Interfaces.ts
[x] Conditional Types.ts
[x] Discriminate Types.ts
[x] Indexed Types.ts
[x] Mapped Types.ts
[x] Any.ts
[x] Literals.ts
[x] Union and Intersection Types.ts
[x] Unknown and Never.ts
[x] Built-in Utility Types.ts
[x] Nullable Types.ts
[x] Tuples.ts
[ ] Big number literals.ts
[ ] Const to let.ts
[ ] Infer From Usage Changes.ts
[ ] Flattened Error Reporting.ts
[ ] Nullish Coalescing.ts
[ ] Optional Chaining.ts
[ ] Assertion Functions.ts
[ ] Recursive Type References.ts
[ ] Uncalled Function Checks.ts
[ ] Checking Unions with Index Signatures.ts
[ ] Accessibility Modifiers.js
[ ] Export Modules From.ts
[ ] Private Class Fields.ts
[ ] Type Imports.tsx
[ ] Class Constructor Code Flow.ts
[ ] JSDoc Deprecated.ts
[ ] Logical Operators and Assignment.ts
[ ] Nullish Coalescing.ts
[ ] Named Tuples.ts
[ ] Unknown in Catch.ts
[ ] Variadic Tuples.ts
[ ] New JSX Support.ts
[ ] See in JSDoc.ts
[ ] Intro to Template Literals.ts
[ ] Mapped Types with Template Literals.ts
[ ] String Manipulation with Template Literals.ts

app
Done: 8, TODO: 1.

[x] community.ts
[x] documentation.ts
[x] handbook.ts
[x] head-seo.ts
[x] index.ts
[x] nav.ts
[x] playground.ts
[x] tsconfig.ts
[ ] dt.ts

docs
Done: 1, TODO: 25.

[x] Nightly Builds.md
[ ] Creating DTS files From JS.md
[ ] Intro to JS with TS.md
[ ] JSDoc Reference.md
[ ] Configuring Watch.md
[ ] Project References.md
[ ] tsconfig.json.md
[ ] Declaration Merging.md
[ ] Decorators.md
[ ] Iterators and Generators.md
[ ] JSX.md
[ ] Mixins.md
[ ] Module Resolution.md
[ ] Modules.md
[ ] Namespaces and Modules.md
[ ] Namespaces.md
[ ] Symbols.md
[ ] Triple-Slash Directives.md
[ ] Type Compatibility.md
[ ] Type Inference.md
[ ] Utility Types.md
[ ] Variable Declarations.md
[ ] Babel with TypeScript.md
[ ] DOM Manipulation.md
[ ] React.md
[ ] TypeScript Tooling in 5 minutes.md

@orta Glad to reach out, can I take some of the tsconfig/? 😄

Yes

Hey folks, I'm about to press the buttons moving localizations to a new repo, and so I'm closing this thread - this issue has a replacement over at https://github.com/microsoft/TypeScript-Website-Localizations/issues/5


fix reliaca Status bug

fix ckman table page table replica status is wrong
because the clickhouse is upgrade
// Remove leader election, step 2: allow multiple leaders https://github.com/ClickHouse/ClickHouse/pull/11639

Which version have this feature？ Maybe it would be better to use the following code：
if common.CompareClickHouseVersion(currentVersion,  featureVersion) >= 0 {
    //new logic
} else {
   //old logic
}

Which version have this feature？ Maybe it would be better to use the following code：
if common.CompareClickHouseVersion(currentVersion,  featureVersion) >= 0 {
    //new logic
} else {
   //old logic
}

How can i get currentVersion？The version is 20.5 which having this feature.

Which version have this feature？ Maybe it would be better to use the following code：
if common.CompareClickHouseVersion(currentVersion,  featureVersion) >= 0 {
    //new logic
} else {
   //old logic
}

How can i get currentVersion？The version is 20.5 which having this feature.

ckman will store clickhouse version in cluster config， you can get it from CKManClickHouseConfig.Version

already change Judgment logic


test: improve coverage for array/view.mbt

Disclaimer: This PR was generated by an LLM agent as part of an experiment.
Summary
coverage of `array/view.mbt`: 91.7% -> 100%

Pull Request Test Coverage Report for Build 4236
Details

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

Totals

Change from base Build 4231:
1.3%

Covered Lines:
4538

Relevant Lines:
5582

💛  - Coveralls


Incorrect parsing in radio URL

What version of Music Assistant has the issue?
2.0.7
What version of the Home Assistant Integration have you got installed?
None yet, I'm trying to play directly through music assistant
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
I'm trying to add a custom radio station URL that's subscription based and therefore each users URL is personal.
An obfuscated URL for the stream is: https://stream.primordialradio.com/proxy/primordial?mp=/stream%3Fname=
However when looking at the logs, music assistant is parsing the %3F in the URL as a ? - unfortunately this leads to a 401 unauthenticated error.
I can verify this in chrome, where I can play the stream with %3F present, but when using ? instead, chrome also returns a 401 error.
2024-07-16 13:35:51.426 WARNING (MainThread) [music_assistant.audio] Error while parsing radio URL https://stream.primordialradio.com/proxy/primordial?mp=/stream%3Fname=: 401, message='Authentication Required', url=URL('https://stream.primordialradio.com/proxy/primordial?mp=/stream?name=')
How to reproduce
Unfortunately you will not be able to directly reproduce this error without my authenticated URL which I'm not able to provide publicly. You will not be able to enter a dummy URL as it will return the 401 immediately and not allow you to add the URL to music assistant.
I can add my URL to music assistant fine, it's only when attempting to play it that it parses it wrong.
Music Providers
Inputted the URL directly into music assistant as a radio station.
Player Providers
SONOS
Full log output
I believe the single line above shows the problem.
Additional information
No response
What version of Home Assistant Core are your running
2024.7.2
What type of installation are you running?
Home Assistant OS
On what type of hardware are you running?
Generic x86-64 (e.g. Intel NUC)

I think this is an example of this happening: https://github.com/aio-libs/aiohttp/issues/1725
It might work to do as they say and add encoded=True to the code, but I'm a little cautious about what the desired behaviour is. Need to check with @marcelveldt

should be fixed in next release

@tgozed You need to try the beta 2.1.0rc3 to see this fix

Very happy to confirm this issue is now fixed in 2.1.0rc3. Thank you so much!


Ability to style input when autocompleteContainer is open

I don't seem to find an option that would allow me to apply a specific style to the input when the results container is open.
What I am trying to achieve, is, having styled my input with rounded corners, I would like the bottom-left and bottom-right corners to become flat when opening the results container.
Is this option absent or am I missing something?

Hopefully render props in v.7.x will solve your issue @antoniobrandao! Any feedback will be very appreciated!! Closing this issue 👍


Added GtkCssProvider example 1

A simple example of how to link to a css style

Very nice, thanks!


support redis clusters

I re-wrote a whole bunch of the top-level state stuff now. I had avoided some parts of this, because you need eslint config ecmascript 2022 for class fields. But I guess it's ok. The IDE unfortunately still doesn't understand these fields :(
Also, the singletons that get constructed top-level are also a top-level invocation, so it feels odd to like one and not the other. But I agree that the implementation via the cfEnv singleton looks cleaner :)


[DO NOT MERGE] Test tomlplusplus ci installation

⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠
DO NOT MERGE THIS PR
⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠ ⚠

The investigation brings to this nice result: https://github.com/dic-iit/bipedal-locomotion-framework/pull/381#issuecomment-896841571


fix(monitor): sore reture info：resourceType,measurement

这个 PR 实现什么功能/修复什么问题:
1.对资源类型和对应的measurement进行排序输出
是否需要 backport 到之前的 release 分支:

release/3.6
release/3.7

/cc @zexi
/area monitor

Codecov Report

Merging #9698 (567fc99) into master (79e6d16) will increase coverage by 2.07%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master   #9698       +/-   ##
==========================================
+ Coverage    5.99%   8.07%    +2.07%
==========================================
  Files         845     107      -738
  Lines      131247   12968   -118279
==========================================
- Hits         7873    1047     -6826
+ Misses     122673   11801   -110872
+ Partials      701     120      -581

Flag
Coverage Δ

aFlag
8.07% <ø> (+2.07%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/appsrv/dispatcher/dispatcher.go

pkg/util/netutils2/netutils.go

pkg/util/influxdb/influxdb.go

pkg/compute/guestdrivers/base.go

pkg/baremetal/tasks/bm_register.go

pkg/compute/models/storageresource.go

pkg/util/rbacutils/policycore.go

pkg/util/iproute2/link.go

pkg/compute/models/guest_actions.go

pkg/compute/models/guestjoints.go

... and 726 more

Continue to review full report at Codecov.

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


Exclude field(s) from query

Hello,
say that I have a class with 20 fields that store course information and one of these field is a longtext. If I want to create a query that will return the approved courses sorted by reading count, I would have to do something like:
return dbContext.Courses.Where(c => c.Approved == true).OrderByDescending(c => c.ReadingCount);
The issue with that query is that the longtext field will be retrieved and it could be quite big, so if I have 100 courses, the data transfered from the database to the server could be quite big. Of course, I could do:
return dbContext.Courses.Where(c => c.Approved == true).Select(c => new { c.Title, c.Description, ... , c.ReadingCount }.OrderByDescending(c => c.ReadingCount);
But as I have 20 fields, it's pretty fastidious, especially if I have other queries that follow the same principle. That's why I was wondering if it would be possible to implement an Exclude method that would work a bit like the Include one except that it would be the exact opposite. The query would then become:
return dbContext.Courses.Where(c => c.Approved == true).Exclude(c => c.Description).OrderByDescending(c => c.ReadingCount);
There could also be an override with a Func returning an array to be able to exclude more fields (or something like that).
Of course, it would be up to the user not to use the excluded field in subsequent query. For example, this:
return dbContext.Courses.Where(c => c.Approved == true).Exclude(c => c.ReadingCount).OrderByDescending(c => c.ReadingCount);
Would fail as the ReadingCount field has been excluded by the user.
I'm sorry if this has already been proposed, I searched and didn't find anything.

@ssougnez Thanks for your comments. I've added a note to #1387 to look here for additional ideas and I'll close this as a duplicate of #1387 so we can track this in one place.


Test / fix for concurrency issue when using bddfy with parallel runners

(see issue https://github.com/TestStack/TestStack.BDDfy/issues/222)
Pretty lo-fi solution but I've added a lock around the ContextLookup - seems to fix the issue.
Forgive not using Should.Throw - but it doesn't give me meaningful information about the exception in-order to verify it's the correct one. It just reports 'AggregateException' which is not much use.

Thanks @graemefoster . Looks good!


How to show full stack traces?

Expected behaviour
I'd like to see full stack traces
Actual behaviour
I've tried Mocha, Jasmine, Chrome headless, electron, and various preprocessors, but in every case I see errors of this format:
Electron 2.0.0 (Node 8.9.3) ERROR
  {
    "message": "Uncaught TypeError: Illegal constructor\nat /home/trusktr/Downloads/src/trusktr+lowclass/src/index.js:1:11\n\nundefined",
    "str": "Uncaught TypeError: Illegal constructor\nat /home/trusktr/Downloads/src/trusktr+lowclass/src/index.js:1:11\n\nundefined"
  }

How do I enable full stack traces?
Environment Details

Karma version (output of karma --version): 2.0.2
Relevant part of your karma.config.js file: Doesn't matter, the error format seems to be the same after trying different preprocessors, launchers, browsers, and frameworks.

Steps to reproduce the behaviour

Run any test

My config looks like this (at the moment):
const CWD = process.cwd()
const glob = require('globby')
const fs = require('fs')
const mkdirp = require('mkdirp')

let testFiles = null

glob([
    CWD+'/src/**/*.test.js',
    CWD+'/tests/**/*.js',
]).then(paths => testFiles = paths)

// wait for glob call to finish
while (!testFiles) {
    require('deasync').sleep(100)
}

testFiles.forEach(file => {
    const relativeFile = file.replace(CWD, '')
    const relativePath = dirname(relativeFile)

    mkdirp( CWD + '/.test-build' + relativePath )

    fs.writeFileSync( CWD + '/.test-build' + relativeFile, `
        require('@babel/register')({
            presets: [ ['@babel/preset-env', { targets: { node: 9 } }] ],
            sourceMap: 'inline',
        })

        require( '${ file }' )
    ` )
})

function dirname(fullname) {
    const parts = fullname.split('/')
    parts.pop()
    return parts.join('/')
}

module.exports = function(config) {

    config.set({

        frameworks: ['jasmine', 'stacktrace'],
        reporters: ['spec'],
        port: 9876,  // karma web server port
        colors: true,
        logLevel: config.LOG_INFO,
        autoWatch: false,
        concurrency: Infinity,
        basePath: CWD,

        browsers: ['Electron'],
        files: [
            '.test-build/**/*.js',
        ],
        preprocessors: {
            '.test-build/**/*.js': ['electron'],
        },
        client: {
            // otherwise "require is not defined"
            useIframe: false,
            loadScriptsViaRequire: true,
        },

    })

}

I guess your stack has one frame,
/home/trusktr/Downloads/src/trusktr+lowclass/src/index.js:1:11
I think a good next step would be to create an simple case in chrome where you have two frames and then throw an error.  If you see the two frames in the browser but not the karma-runner output, we'll know more.


Cilium Backports 1.2- 28-11-2018

PR:
#6295
#6299

test-me-please

test-me-please

Timeout issue: https://jenkins.cilium.io/job/Cilium-PR-Ginkgo-Tests-K8s/1349/console

test-missed-k8s

test-missed-k8s


fixes issue #105 by removing try: except completely

This solution fixes incorrect AttributeError catches by removing the corresponding try: except block completely.
I assume the reason this block was added in the first place was to distinguish between a method not existing and a method returning a falsy value without invoking hasattr, for performance reasons.
My solution verifies if the method was invoked by first creating a unique object (tuple) that is then returned as a default argument from getattr. To check if a method was invoked, that object is then compared to the return value by using the is comparator.

I just noticed that LGTM uncovered an error in my implementation that is not caught by tests yet - the case where event handlers don't handle event by returning pyglet.event.EVENT_UNHANDLED.
Will add tests that catch it along with a fix later.

Thanks for the PR! I like your solution it's similar to what they do in the standard library lru_cache, with using a "sentinal" object.
Unfortunately there is still a race condition that we hit when not hijacking AttributeError. It often happens in slightly complex projects. One way to reproduce this consistently is to try the examples/text_input.py example in the master branch. Paul and I were talking about this on Discord a bit.


Issues with retaining default oneof values

I'm working on implementing support for the protobuf API of Google Cloud Datastore into gcloud-node. I've hit a sticking point, and I'm not sure exactly where to look in the stack to find the problem.
I made a fork to reproduce: https://github.com/stephenplusplus/protobuf.js. I eliminated all of the tests other than mine that fails: https://github.com/stephenplusplus/ProtoBuf.js/blob/master/tests/suite.js
When I mocked the structure of the Google proto files into one test proto file, the tests passed. But when I plug in the real files, it fails.
Sorry to dump a bunch of code on you; if you think this should be asked somewhere else, feel free to close. Thanks for any help!

I think this is related to a oneof inside of map values. Here is a PR which has a test that fails on decoding a default value inside a map but not on decoding a default value outside of a map.

Ah sorry, I messed up my test, they actually pass as expected.

Currently debugging this without success. Is there still an issue?

Thanks for checking it out. It's still reproducible for me, but only with my setup from the first post when loading the real proto files, no mocks.
The specific issue is any falsy values in the Value.value_type oneof (integer_value of 0, boolean_value of false), end up getting a null "value_type" when decoded: https://github.com/stephenplusplus/ProtoBuf.js/blob/c88a581dee8395f68c9e0ef28bb267e0a70c29e6/tests/suite.js#L97

Still pinpointing. A minimal .proto causing the error seems to be:
syntax = "proto3";

message Entity {
    map<string, Value> properties = 3;
}

message Value {
    oneof value_type {
        // A boolean value.
        bool boolean_value = 1;
    }
}

With that test:
        var builder = ProtoBuf.loadProtoFile("./test.proto");
        var Entity = builder.build("Entity");
        var entity = new Entity({
            properties: {
                isTrue: {
                    boolean_value: true
                },
                isFalse: {
                    boolean_value: false
                }
            }
        });
        var bb = entity.encode();
        var entity2 = Entity.decode(bb);
        test.log("isTrue");
        test.strictEqual(entity2.properties.map.isTrue.value.value_type, "boolean_value");
        test.log("isFalse");
        test.strictEqual(entity2.properties.map.isFalse.value.value_type, "boolean_value");
        test.done();

Seems to be an issue with Reflect.Message.Field#hasWirePresence. That's contributed code, going to debug this now.

This is what happens: When boolean_value is set to false, it is set to the default value of the field (proto3), which results in the field not being encoded on the wire.
When decoding, the message is empty on the wire, so it will decode with boolean_value being false again - but ProtoBuf.js doesn't know which field was actually set explicitly to it's default value.
Do you know how this is handled in the official implementation? Is it even possible?

I believe it does encode the values. I'm not totally familiar with the implementation, but I think the relevant portion is here:
https://github.com/google/protobuf/blob/master/python/google/protobuf/internal/python_message.py#L656

Thanks very much! It works perfectly. Any chance of tagging a release soon?

4.1.0 is now on npm


Update to the latest mrview

This includes all the changes made to couch_mrview since the last deps update

@kxepal can you take a look, please? this bumps mrview up to HEAD and should resolve one of those pesky test failures we have on travis.

@eiri
LGFM, except failures on travis. Seems like we have new flaky tests regardless @nickva work /:
IIRC, there were some work in apache/couchdb-couch around delete - does it already on rebar.config?

@kxepal Yes, apache/couchdb-couch part got in rebar.config as it happened to be downstream of admin_local work, so mrview's bit got left behind.
The test failures, as far as I can see, are unexpected oauth 403 in couchdb_vhosts_tests and timeouts in couch_changes_tests. I've seen timeout is couchdb_compaction_daemon_tests once, but it seems transient.

@eiri Good then. +1. Those tests need in additional, unrelated work then.


Bug 2073220: routes/status resources can leak sensitive data

Fixes: https://bugzilla.redhat.com/show_bug.cgi?id=2073220

/assign @sttts @soltysh @mfojtik

/lgtm

/retest

TestGetAuditPolicy is failing, i think you will need to add the same in the test files as well

/retitle Bug 2073220: routes/status resources can leak sensitive data

/retest-required

@aojea I think you will need to add the change in the test data as well
https://github.com/openshift/library-go/blob/330d9097d0f08c86aedafc8ec350b25d4a9b4f68/pkg/operator/apiserver/audit/testdata/writerequestbodies.yaml#L35
maybe here as well?
https://github.com/openshift/library-go/blob/330d9097d0f08c86aedafc8ec350b25d4a9b4f68/pkg/operator/apiserver/audit/testdata/allrequestbodies.yaml#L35

found another one
https://github.com/openshift/library-go/blob/330d9097d0f08c86aedafc8ec350b25d4a9b4f68/pkg/operator/apiserver/audit/testdata/audit-policies-cm-scenario-1.yaml#L124

I don't have a clue what this fails, if I run it locally the failing test looks unrelated
 /usr/local/go/bin/go test -timeout 120s -run ^Test_newFromConfigMapData_slow_sigstore$ github.com/openshift/library-go/pkg/verify -v

=== RUN   Test_newFromConfigMapData_slow_sigstore
    /home/aojea/go/src/github.com/openshift/library-go/pkg/verify/configmap_test.go:153: expected ^[[][0-9TZ:-]*: Get "https://example.com/signatures/sha256=123/signature-1": context deadline exceeded, [0-9TZ:-]*: context deadline exceeded]$, got [2022-09-01T19:39:04+02:00: Get "https://example.com/signatures/sha256=123/signature-1": context deadline exceeded, 2022-09-01T19:39:04+02:00: context deadline exceeded]
--- FAIL: Test_newFromConfigMapData_slow_sigstore (0.10s)
FAIL
FAIL	github.com/openshift/library-go/pkg/verify	0.114s

$ go test -v -race -count=1 ./pkg/verify -run=Test_newFromConfigMapData_slow_sigstore
=== RUN   Test_newFromConfigMapData_slow_sigstore
    configmap_test.go:153: [2022-09-02T07:57:51-04:00: Get "https://example.com/signatures/sha256=123/signature-1": context deadline exceeded, 2022-09-02T07:57:51-04:00: context deadline exceeded]
--- PASS: Test_newFromConfigMapData_slow_sigstore (0.10s)
PASS
ok  	github.com/openshift/library-go/pkg/verify	0.183s

If I run it locally, it passes for me, (i am logging the error ^), and my go version is go version go1.19 linux/amd64
and the timestamp i see is 2022-09-02T07:57:51-04:00 ( -04:00 is my time zone)
This line is creating the error with timestamp time.Now().Format(time.RFC3339)
https://github.com/openshift/library-go/blob/330d9097d0f08c86aedafc8ec350b25d4a9b4f68/pkg/verify/verify.go#L192
time.RFC3339 translates to 2006-01-02T15:04:05Z07:00
maybe in ci, this test is running in a +00:00 timezone now? @aojea the change looks good to me

/lgtm

/cherry-pick release-4.11
/cherry-pick release-4.10

/cherry-pick release-4.10


Add configuration module

This PR adds the [config] module. This isn't a native module nor an external module, it is a parser module which is handled at the parsing step immediately. Currently, there is support for import, hide and set statements.
Syntax:
To use the config module, put a multiline [config] module at the very top of the document. [config] modules are not allowed anywhere else. Thereafter, each line of the module body is a config statement. There are three types of statements described below, and here is an example of a config module expression:
[config]
hide table
import bettertable
set bettertable_colours gray white gray black

Import statements:
Import statements come in three flavours:
import foo
import bar using a, b, c
import baz hiding x, y, z

They import the given package, possibly only importing some transforms (using) or possibly skipping some transforms (hiding).
Hide statements:
You can hide a built-in package by doing hide html. This has the same behaviour as doing import html using <none> or import html hiding * (none of which is allowed).
Set statements
You may do set key value to set the variable key to the value value. value may contain spaces (so set a abc def works), but it strips its initial spaces; quotes can be used to include initial whitespace
State of this PR
This is currently a draft PR. To make this merge-ready, it needs to have better support in Core, to actually do something with errors rather than just printing them to the console, and hook it up to the package manager (for the CLI) which will hopefully be merged in GH-126.
I think this PR is ready for merging. Note that these config statements are not supposed to be the only ones which exists but I think it might be good to have a foundation to build upon on main. More statements may be added as part of future issues/PRs.
Note that this PR doesn't add package resolving to the Playground; if you want to test it out, you may use the CLI

I think this PR is ready for review now. In summary, it adds the [config] module allowing for configuring how loaded packages are used (what transforms they allow for and so on), and resolving new packages using a Resolver (currently implemented for the Cli but not for Playground). Here are some test cases:
You may test importing standard packages with reduced functionality (works in Cli + Playground):
[config]
import html hiding __bold, __math

// Now the __bold and __math transformation is disabled

You may test hiding standard packages completely (works in Cli + Playground):
[config]
hide table

// Now, the table package isn't available

You may test importing additional packages from the registry (works in Cli only):
[config]
import pkgs:hello

Now, we invoke a new module from the imported package Hello: [hello]

Note that the hello package just outputs "Hello, World!" and is a part of the registry
You may test importing additional packages locally (works in Cli only):
[config]
import hello

Now, we invoke a new module from the imported package Hello: [hello]

You may download the hello package from the registry here
You may also test entering invalid packages, configuring the same package multiple times etc to see that you get correct errors


Multiple DbContexts per tenant

Does Finbuckle.MultiTenant support multiple connection strings per tenant?
(i.e. for an infrastructure database, an auth database, and an app database)
If not, any thoughts and how this could be accomplished "manually?"
Thanks,
Tyson

hi @tcorvin
The next version releasing soon will support a custom Tenant Info object so you can have any number of connection strings. Then in the configuration for the database context you would have to somehow choose the connection string you wanted to use.
In the current version you could store them in Items collection, but its just a <string, object> dictionary and not strongly typed.

Hey @achandlerwhite, perfect ... I'll use the items collection for now as we wait for next release ... thanks!


Update GoogleAppMeasurement version to avoid macOS version error.

[REQUIRED] Step 1: Describe your environment

Xcode version: 10.25
Firebase SDK version: 8.2.0
Installation method: Swift Package Manager (select one)
Firebase Component: Messaging (Auth, Core, Database, Firestore, Messaging, Storage, etc)

[REQUIRED] Step 2: Describe the problem
Steps to reproduce:
What happened? How can we make the problem occur?
When you run swift swift build you get an error
error: the library 'GoogleAppMeasurementTarget' requires macos 10.10, but depends on the product 'GULAppDelegateSwizzler' which requires macos 10.12; consider changing the library 'GoogleAppMeasurementTarget' to require macos 10.12 or later, or the product 'GULAppDelegateSwizzler' to require macos 10.10 or earlier.
After investigating the problem, I discovered that the problem is using an app measurement .package(
name: "GoogleAppMeasurement",
url: "https://github.com/google/GoogleAppMeasurement.git",
.exact("8.3.0")
)
using the exact version means macOS version would be 10.10. If you could bump this version up all would be fine.
If you have a downloadable sample project that reproduces the bug you're reporting, you will
likely receive a faster response on your issue.
Relevant Code:
// TODO(you): code here to reproduce the problem

Thanks for the report and sorry for the troubles.
It looks like there was an attempt at a fix here: https://github.com/google/GoogleAppMeasurement/pull/12 which is included in the 8.3.0 release: https://github.com/google/GoogleAppMeasurement/blob/06e74ef7ee7326e1af724d462091eed1e5c6fb4a/Package.swift#L22
Are the reproduction steps the same for you as the issue linked above, or did you follow a different set of steps?

Yes, it works changing the branch to master. but then this error pops up.
error: artifact 'GoogleAppMeasurement' does not support the target platform and architecture ('Triple(tripleString: "arm64-apple-macosx", arch: TSCUtility.Triple.Arch.arm64, vendor: TSCUtility.Triple.Vendor.apple, os: TSCUtility.Triple.OS.macOS, abi: TSCUtility.Triple.ABI.unknown)')
swift build -Xswiftc "-sdk" -Xswiftc "xcrun --sdk iphonesimulator --show-sdk-path" -Xswiftc "-target" -Xswiftc "x86_64-apple-ios11.0-simulator"

GoogleAppMeasurement (a FirebaseAnalytics dependency) does not currently support macOS.  See #1732.  You may need to use xcodebuild instead of swift build to build from the command line.

Going to close since the version error is addressed in 8.3.0.  We can continue the discussion here though.

Probably related, but after upgrading to Firebase 8.3.0 via SPM, I'm not able to run the project anymore. Should I create new issue?

@Kaspik I've seen that happen even outside of the upgrade cycle, it seems to be an Xcode flake that happens occasionally. Typically going to File -> Swift Packages -> Reset Package Cache works for me.


[jdbc-v2] Metabase testing issues

### Latest issues
- [x] TABLE_TYPES should be broad categories rather than table engines
- [x] SQL Comments cause issues parsing statement type
- [x] Unwrap not unwrapping
- [x] ResultSet.next() throws exception other than SQLException
- [x] SimpleAggregateFunction not supported (See https://github.com/ClickHouse/clickhouse-java/pull/2022)
- [x] IPv6 is showing as IPv4
- [x] Set no-op for all instances of "Unsupported"
- [x] Enums should return String not Number
- [ ] We seem to not close connections - need to make sure it gets closed properly

We seem to not close connections - need to make sure it gets closed properly

The related issue: https://github.com/ClickHouse/clickhouse-java/issues/1741


Should the validated property of a subcription be exposed to the end user?

Description
Currently an end user can retrieve all subscriptions or a single susbcription, but the validated property isn't exposed to them.
The only way to know if a subscription is valid is to confirm that the call back when registering the subscription has been received.
In scope
No response
Out of scope
No response
Additional Information
No response
Analysis
No response
Conclusion
No response

Implemented in #129


Create enabling-dma-remapping-for-device-drivers.md

New document for providing guidance on enabling DMA Remapping for device drivers

@AaDake this passed all tests. Is it ready to go live?

@EliotSeattle this is ready to go live. I just made a typo fix this morning.
Thanks for your help!


JENKINS-50046: set the parse interval, JENKINS-50048: progress bar

Hi,
This pull request implements two new features:
https://issues.jenkins-ci.org/browse/JENKINS-50046: In pipeline projects it is possible to set the parse interval to a fixed value. When it is not set or for Freestyle projects the current logic is kept (every 1/100 of the estimated build time and never more often than every 5 seconds).

https://issues.jenkins-ci.org/browse/JENKINS-50048: Adding a progress bar to the summary view. The tooltip shows the number of completed tests vs expected tests and the estimated remaining time. This is based on the latest test results from previous builds.
New project (or cannot find previous test results):

Progress with tooltip:

This pull request implements two new features

Generally best to split into two PRs then.
I have contributed to this plugin in the past but I am not sure if it has a maintainer currently. Are you interested in adopting it?

@jglick
Yes I like to adopt it, I've been contributing to a lot of plugins already it's probably time I get involved a bit more with all the release lifecycle.
The plugin is not marked for adoption, do I still go through with the procedure here: https://www.jenkins.io/doc/developer/plugin-governance/adopt-a-plugin/ ?

Yes. https://github.com/jenkinsci/junit-realtime-test-reporter-plugin/commit/cbd19bb6c3920f6f4a0cb4f524f3265cf80ab1ae suggests it is unmaintained, as does https://github.com/jenkins-infra/repository-permissions-updater/blob/db153f3cde58b13b9b31e8c8a0732114f2307272/permissions/plugin-junit-realtime-test-reporter.yml#L6.

Please split into 2 PRs and resolve the conflicts.
Both are likely fine but the PR is a bit big right now.

Closing in favor of #32 and #33


Incorrect query results, possibly due to incorrect inlining of UUID function

Steps to reproduce the behavior (Required)
unzip [bug.zip](https://github.com/StarRocks/starrocks/files/12701386/bug.zip)
cd bug-repro
./bug.sh

Expected behavior (Required)
If you diff correct.sql and bug.sql, you can see that there's a small change that should not affect the results. I expect the to SQLs to produce equivalent results:
Sleeping for 1 minute
Creating database
Running Correct.sql
JoinColumnResolver_Column_ValueTable_000
9
1
Running Bug.sql
9
1
Removing docker container
starrocks-test
starrocks-test

Real behavior (Required)
They produce different results. My guess is that the UUID is getting inlined somewhere and causing it to produce different UUIDs for the same CTE and thus we are not joining as we expect to.
Sleeping for 1 minute
Creating database
Running Correct.sql
JoinColumnResolver_Column_ValueTable_000
9
1
Running Bug.sql
Removing docker container
starrocks-test
starrocks-test

StarRocks version (Required)
starrocks/allin1-ubuntu:3.1.0-rc01

This is a fantastic discovery.
Currently, when deciding whether to use CTEs (Common Table Expressions), we do not take into account whether non-deterministic functions are present within the operators. This can result in unexpected outcomes.
Solutions

use set cbo_cte_reuse_rate=0 to enforce cte for reuse to avoid this bug.
I will add a rule to enforce cte when non-deterministic functions are present within the operators later.


g:Lf_MruFileExclude not work

vim or neovim?

[x] vim
[ ] neovim

vim --version or nvim --version:
VIM - Vi IMproved 8.0 (2016 Sep 12, compiled Apr 18 2018 15:30:48)
macOS version
Included patches: 1-1700
Compiled by Homebrew

Output of :echo has("python"): 0

Output of :echo has("python3"): 1

Output of :echo &pythondll(only vim, not neovim): nothing printed

Output of :echo &pythonthreedll(only vim, not neovim): nothing printed

Output of :py print(sys.version): Sorry, the command is not available in this version

Output of :py3 print(sys.version): 3.6.5

Operating system:

[ ] Linux
[x] Mac OS X
[ ] Windows
[ ] Etc.

Configurations related to LeaderF in vimrc:

let g:Lf_WildIgnore = {
            \ 'dir': ['.svn','.git','.hg'],
            \ 'file': ['*.sw?','~$*','*.bak','*.exe','*.o','*.so','*.py[co]']
            \}
let g:Lf_MruFileExclude = ['*.sw?','~$*','*.bak','*.exe','*.o','*.so','*.py[co]', '*.cnx']

Describe your question, feature request, or bug.
Mrus still show .cnx files when I set g:Lf_MruFileExclude. By the way, how can I exclude files in specific directory? I have add .git direcotry in g:Lf_WildIgnore but mrns still show files in .git directory.

g:Lf_WildIgnore does not work for MRU, it only works for LeaderfFile.
g:Lf_MruFileExclude indicates that files match pattern defined in it won't be recorded in MRU cache. If you have opened the .cnx file before setting g:Lf_MruFileExclude, the .cnx file can still be shown, the .cnx file after the setting won't show in the MRU.
Maybe I need add a g:Lf_MruWildIgnore option to meet your requirement.

How can I refresh MRU cache? I have tried use F5 but not work.

Why do you want to refresh the cache?

Sorry for misunderstanding  your reply. There is no need to refresh the cache.
By the way, I think it may be better to make g:Lf_WildIgnore works for all modes than add a new g:Lf_MruWildIgnore

Adding g:Lf_MruWildIgnore  +1

I have updated LeaderF and replaced g:Lf_MruFileExclude with g:Lf_MruWildIgnore.
Thanks a lot for making the charming plugin!


agent: remove more hardcoded paths, ports

Signed-off-by: Kuat Yessenov<EMAIL_ADDRESS>Fixes: #32920
Removed constants to keep the trouble away.

@kyessenov: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

release-notes_istio
5f1b35f3146db8b4c9cdae665869c31c6feb606f
link
/test release-notes_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@kyessenov: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
5221d5111bf929fa05084f3c31eb76007b7bb75b
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@kyessenov: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-security-multicluster-tests_istio
5221d5111bf929fa05084f3c31eb76007b7bb75b
link
/test integ-security-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest


how to set class RpcClientConf member variable Endpoints

we use nacos cluster,actually we all use this frame in pro enviroment.
my yaml content is below like that:
Nucleus:
Endpoints:

<IP_ADDRESS>:8848/nucleus-grpc

i did not set App and token  params(although i havn't found any introduction of the two params)
Endpoints,this param i set <IP_ADDRESS>:8848 or nacos://<IP_ADDRESS>:8848/nacos.rpc?timeout=5000s like demo seted
it did not work。
when i launch the app  it warned
":rpc dial: direct:///<IP_ADDRESS>:8848/nucleus-grpc, error: context deadline exceeded, make sure rpc service "nu
cleus-grpc" is already started","level":"error

https://github.com/zeromicro/zero-contrib/tree/main/zrpc/registry/nacos

You can refer to the practice here: https://github.com/zeromicro/zero-contrib/tree/main/zrpc/registry/nacos

it did not work. import (
"flag"
"github.com/zeromicro/go-zero/core/conf"
"github.com/zeromicro/go-zero/zrpc"
_ "github.com/zeromicro/zero-contrib/zrpc/registry/nacos"
)
func main() {
flag.Parse()
var c zrpc.RpcClientConf
conf.MustLoad(*configFile, &c)
zrpc.MustNewClient(c)

}
i tested the code like demo. always warned  rpc dial: nacos://<IP_ADDRESS>:8848/nacos.rpc?timeout=5000s, error: context deadline exceeded, make sure rpc service "nacos.rpc?timeout=5000s" is already started.
but it's strange that the client demo could substribe the server service ,even my own server service program.

行了，自行通过nacos官方sdk搞定了。

行了，自行通过nacos官方sdk搞定了。

请问下怎么解决的，我也遇到了，求大佬解惑


[finishes #133610025] Fork from apigee to unitive-inc

In preparation for npm publish
Tests pass:
arcus:swagger-tools damon$ git log -1 && npm test
commit 56655709ea523e3a3bd9c49eab45a0124e73b913
Author: Damon Cokenias<EMAIL_ADDRESS>Date:   Wed Nov 2 09:54:42 2016 -0700

    Fork from apigee to unitive-inc
(node:10597) fs: re-evaluating native module sources is not supported. If you are using the graceful-fs module, please update it to a more recent version.

><EMAIL_ADDRESS>test /Users/damon/Unitive/devel/swagger-tools
> ./node_modules/gulp/bin/gulp.js

[09:57:57] Using gulpfile ~/Unitive/devel/swagger-tools/gulpfile.js
[09:57:57] Starting 'lint'...
[09:57:58] Starting 'test'...
[09:57:58] Starting 'test-node'...

  CLI Global

...

        ✔ should handle schema items definition

Finished in 4.872 secs / 4.529 secs

SUMMARY:
✔ 99 tests completed```

Don't you have to rename the package in package.json?

ooops. will redo


Improve default handling for filters and groups

If a user passes in an array of text (e.g. groups:["sex","race"]) then setDefaults should .map() it in to an appropriate array of objects.

try renderering in version = dev-v1.0.0 with the following settings:{filters:["SEX","RACE"], groups:["SEX","ARM"]}

CAT requires JSON syntax:
{
  "filters": ["SEX", "RACE"],
  "groups": ["SEX", "ARM"]
}

Expected Result: Used the following settings in the CAT to render only those filters and groups:
{
"filters": ["SEX", "RACE"],
"groups": ["SEX", "ARM"]
}
Outcome: Pass


Gain Homebridge Verification

Is your feature request related to a problem? Please describe:
This is not related to a problem, rather an enhancement.
Describe the solution you'd like:
I would love if the plugin could become Homebridge verified. (https://github.com/homebridge/homebridge/wiki/Verified-Plugins).  It seems you already meet all the requirements.
Describe alternatives you've considered:

Additional context:
This helps with the discovery of quality Homebridge plugins.

I've already planned this, just wanted to wait a bit for tests before an v1.0.0 release. After that I will start the process to verify the plugin.

https://github.com/homebridge/verified/issues/703


how to install an extension for  python

please  help

Thanks for the bug report, but unfortunately, we need some more information from you in order to triage this issue appropriately. Please follow instructions here to create a new bug with all the critical details we need for diagnosis.

If you want to install an extension, you can follow the steps here.
If you need more help please create an issue explaining the problem.


debug/gosym: mark LineTable.LineToPC and LineTable.PCToLine as deprecated

Both methods appear to be deprecated, according to their comments. We have a standard way of marking methods as deprecated; let's apply it here.
This may be a good first issue for someone who wants to try out contributing.

I've picked this up!


workbook.Write() should not close stream

Workbook does not own the stream, it should not be closing it.

This issue is still present. We can`t use Write to MemoryStream in any ways.

This is fixed in NPOI official release. Please use https://www.nuget.org/packages/NPOI/ instead.

Not ideal, but if you face this issue, copy content of stream to byte array and create a new memory stream from it.


[BEAM-8387] Remove sdk-worker-parallelism option from JobServerDriver

This option was added when it wasn't possible to specify the worker count as pipeline option, which is no longer the case. The pipeline option has a special value of 0, which means that the runner should pick a suitable value. Unfortunately this was overridden in FlinkJobInvoker with 1 (because that's the default in the  JobServerDriver config). There is no need to have this option in the job server.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

LGTM, I always thought having the same argument set in both places added unnecessary complexity (like BEAM-7657).