Caused by: com.fasterxml.jackson.databind.exc.InvalidTypeIdException: Could not resolve type id 'http' as a subtype of [simple type, class io.dropwizard.jetty.ConnectorFactory]: known type ids = [] (for POJO property 'applicationConnectors')

Still getting this error in V1.3.8
I have configured shadowJar too.
I am deploying it on Heroku
Command used to run - web    java -Ddw.server.applicationConnectors[0].type="http" -Ddw.server.applicationConnectors[0].port="8080" -jar build/libs/ecommerce-1.0-SNAPSHOT.jar server ecommerce.yml

yaml file -
template: ecommerce of, %s!
defaultName: Stranger
logging:
  level: INFO
  loggers:
    com.prakhar: DEBUG

server:
  applicationConnectors:
    - type: http
      port: 8090
  adminConnectors:
    - type: http
      port: 8091
  applicationContextPath: /*
  rootPath: /app/*

database :
  driverClass: org.postgresql.Driver
  user: ${DATABASE_USER}
  password: ${DATABASE_PSWD}
  url: ${DATABASE_URL}
  properties:
    charSet: UTF-8
    user: ecommerce
    password: password
    url: jdbc:postgresql://localhost:5432/ecommerce_dev
    # the maximum amount of time to wait on an empty pool before throwing an exception
    maxWaitForConnection: 1s

    # the SQL query to run when validating a connection's liveness
    validationQuery: "/* MyService Health Check */ SELECT 1"

    # the timeout before a connection validation queries fail
    validationQueryTimeout: 3s

    # the minimum number of connections to keep open
    minSize: 8

    # the maximum number of connections to keep open
    maxSize: 32

    # whether or not idle connections should be validated
    checkConnectionWhileIdle: false

    # the amount of time to sleep between runs of the idle connection validation, abandoned cleaner and idle pool resizing
    evictionInterval: 10s

    # the minimum amount of time an connection must sit idle in the pool before it is eligible for eviction
    minIdleTime: 1 minute

I'm unable to reproduce the issue with Dropwizard 1.3.16:
https://github.com/joschi/dropwizard-issue-3017
# ./gradlew shadowJar

BUILD SUCCESSFUL in 2s
2 actionable tasks: 2 executed

# java -Ddw.server.applicationConnectors[0].type="http" -Ddw.server.applicationConnectors[0].port="8000" -jar build/libs/dropwizard-issue-3017-all.jar server configuration.yml

# curl localhost:8000/test
Hello World!

Please try with the latest stable version (Dropwizard 1.3.16) or provide a minimal project to reproduce the issue.


Detect competing force from resolutionStrategy

Prior to this change, a ResolutionStrategy.force based force was not
handled the same as setting force = true on a dependency.
This caused virtual platforms alignment to not properly detect there
were competing force in action.
With this change, a SelectorState is marked as force also for a forced
substitution.

Yes, I think it was the first if in there and did not think too hard about its validity when I added the other ones. However the moment a SelectorState can be marked as forced from another source than the DependencyMetadata, this check was causing the invalid case detection to fail.


[DOCS] Reorganize explanation of Accumulators and Broadcast Variables

What changes were proposed in this pull request?
The discussion of the interaction of Accumulators and Broadcast Variables should logically follow the discussion on Checkpointing. As currently written, this section discusses Checkpointing before it is formally introduced. To remedy this:

Rename this section to "Accumulators, Broadcast Variables, and Checkpoints", and
Move this section after "Checkpointing".

How was this patch tested?
Testing: ran
$ SKIP_API=1 jekyll build
, and verified changes in a Web browser pointed at docs/_site/index.html.

Can one of the admins verify this patch?

Thanks - this change makes sense.
Did you change any of the content besides moving them?

Looks fine but I second Reynold's question. Sounds like no modifications from your summary.

There is just one change and it is to the section name:
Strike "Accumulators and Broadcast Variables"
Replace with "Accumulators, Broadcast Variables, and Checkpoints"

I assume there's no need to run tests for doc changes, so merging to master / 2.0.

Did not merge to branch-2.0, so just left it in master.


Right-click menu goes off screen if you open it too close to the window's borders

Describe the bug
If the menu is opened too far to the right, some of it gets cut off.
To Reproduce
Steps to reproduce the behavior:

Right-click near the right side of the window
Menu gets cut in half

Expected behavior
The menu to clip back in the window
Screenshots
If applicable, add screenshots to help explain your problem.

Additional context
The menu doesn't get cut off at the bottom because of the sliding animation that "pushes" the menu back up.

Commit 5ab8d593baeeeee2a912fb68aa8fc5b67568b73a pushes offending menus to the left, but they close as soon as you try to reach them with the mouse. I'm reopening this so we can investigate a proper fix.
https://user-images.githubusercontent.com/29354120/141674310-4e719bbf-3f42-42e3-89c9-1774160b8b19.mp4


Segfault when multiple clients connect to listening socket

I use examples/server to set up a TCP server listening on port 12345 and then connect two clients. The first goes well but when the second connects, the server dies with a segfault.
Running in gdb:
(gdb) run
Starting program: /home/kll/terastream/acton/examples/server
[Thread debugging using libthread_db enabled]
Using host libthread_db library "/lib/x86_64-linux-gnu/libthread_db.so.1".

[Switching to Thread 0x7ffff6410640 (LWP 1097054)]
0x000000000041f63e in ENQ_msg (m=0x7fffd4000bd0, a=0x7fffdc001a70) at rts/rts.c:522
522	        while (x->$next)
(gdb) bt
#0  0x000000000041f63e in ENQ_msg (m=0x7fffd4000bd0, a=0x7fffdc001a70) at rts/rts.c:522
#1  0x000000000041ffbe in FLUSH_outgoing (self=0x4ec990, txnid=0x0) at rts/rts.c:776
#2  0x0000000000422513 in main_loop (idx=0x3) at rts/rts.c:1225
#3  0x00007ffff7f2beae in start_thread (arg=0x7ffff6410640) at pthread_create.c:463
#4  0x00007ffff7d16a5f in clone () at ../sysdeps/unix/sysv/linux/x86_64/clone.S:95

If I run with ./server --rts-verbose --rts-debug I haven't been able to reproduce it. Could it be timing related? Are we modifying x->$next outside of a lock?

Does it always (often) fail at the same spot? If so I'd like to do a few gdb printouts after it has stopped:
a->$class->$GCINFO
a->$msg
a->$msg->$next
a->$msg->$next->...->$next     (until it becomes NULL)
It would also be interesting to know if any other threads are awake at this time, and where they are working if that's the case.
The error is very likely timing related, which is why I hope you'll be able to provide the above printouts next time it fails. All manipulation of the message queues are (meant to be) protected by spinlocks, which includes writing to the $next fields. But I'll do another scan of the code to see if this has been accidentally broken.


Update fmt implementation to fix windows warnings

Resolves #2276
I don't expect anybody to "review" the code itself, it's just syncing with the latest fmtlib.
But perhaps @yann-lty can try building from this PR and see if it avoids the error? (If it works, I will also backport to RB-2.0 if you need it there.)

Maybe we should just accept that fmt is a dependency and handle it externally?
Like pybind11, its not a particularly onerous dependency since its header only. I'm guessing we could handle it in a similar way.

Turns out that the link errors were actually due to another external library that was redefining "isnan", more details here: https://github.com/OpenImageIO/oiio/issues/2276#issuecomment-508040022.
This PR builds and runs fine on my side, but since updating fmt is not required after all - and not in the "bugfix" scope anymore -, I don't know if it's still wanted to make changes about this right now ?

I think since I've already done the (small amount of) work, and it passes the tests, and presumably has some improvements to fmt's internals, I may as well merge it. I mean, I would/should have done it sooner or later anyway.
But I certainly won't backport it to 2.0 if there's not a pressing need. (And I also realized now, maybe if there was a need for a fix in 2.0, it would have been better to just #undef isnan in strutil.h and not update any code.)

@fpsunflower : I'm still trying to decide what to do long term. pybind11 is strictly a build-time dependency, but with fmt it needs to have the headers available when people (it's included by strutil.h) so we need to copy some of the headers into wherever include/OpenImageIO dists to. So I'm a little leery of just relying on a system install at build time.
Maybe it's still better to have it external like we do pybind11 (even if we copy parts to dist), rather than actually check the files into our repo. I'm not sure I want to tackle that today, need to think about it. Do you have any current thinking/preferences about "downloading" (like we do with pybind11) versus using git submodules?

Oh right, I forgot we have to expose that API as part of ours. That's definitely unfortunate (but unavoidable since its all template driven).
I'm not sure I want to get into recursive submodules at the moment. Its few enough headers that copying is probably the best solution for now. I'm just anticipating that at some point fmtlib will be popular enough that conflicts may arise for people that use their own copy elsewhere? It doesn't look like they allow for customizing the namespace wrapping.
I do miss the simplicity of printf  ...

We could change the namespace just in our copy, I guess.

There is an inner namespace with a version number, so presumably it should not be a problem for multiple libraries to be using fmtlib, assuming that they are properly updating the version if ABI compatibility breaks.

I think its more complicated than that since the headers are using include guards instead of #pragma once, so just including the headers from two different copies in the wrong order could mix up which ones get used.
To be safe, I think both things should be patched (header guards and namespace).

I think I'm going to merge what I have (nothing wrong with updating but keeping the status quo on inclusion strategy), and punt for now on changes to fmt. Will revisit another day if it pops up as an actual problem.


python310Packages.ratelimiter: fix check part

Description of changes
Fix build (https://hydra.nixos.org/build/173995488)

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
22.05 Release Notes (or backporting 21.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 171073 run on x86_64-linux 1

1 package marked as broken and skipped:

snakemake

2 packages built:

python310Packages.ratelimiter
python39Packages.ratelimiter


Security - Rake Update (CVE-2020-8130)

moderate severity
Vulnerable versions: <= 12.3.2
Patched version: 12.3.3
There is an OS command injection vulnerability in Ruby Rake before 12.3.3 in Rake::FileList when supplying a filename that begins with the pipe character |.

Fixed


TFKUBE-383: Fix the synchrony installation

This is just a temporary fix for now. After merging https://github.com/atlassian-labs/data-center-terraform/pull/186 this won't be necessary as we will setup synchrony in terraform.
Because this fix is simple and it might take some time for the ingress PR to get merged, I think we can merge this meanwhile to cover use case if customer doesn't have the helm repo installed.

Will be superseded by https://github.com/atlassian-labs/data-center-terraform/pull/186


added e2e for contact details page

Context
AB#197338 story AB#185601
Change proposed in this pull request
scenario to go onto contact details page of the school from school home page.
added assertions for the school page which will be incremental as we add more stuff in the page.
Guidance to review
I am going to work on setting up a standards document and approach about writing the tests and tests will be refactored accordingly.
Checklist

[x] Work items have been linked (use AB#)
[x] Your code builds clean without any errors or warnings
[x] You have run all unit/integration tests and they pass
[x] Your branch has been rebased onto main
[x] You have tested by running locally

closing this PR as a separate one has been added after refactoring work.


fix acc_type_switch macro with extra tests

Description
Fix the bug that reduction operators would not produce the correct value for integer type inputs due to bug in MXNET_ACC_TYPE_SWITCH.
Checklist
Essentials
Please feel free to remove inapplicable items for your PR.

[ ] The PR title starts with [MXNET-$JIRA_ID], where $JIRA_ID refers to the relevant JIRA issue created (except PRs with tiny changes)
[ ] Changes are complete (i.e. I finished coding on this PR)
[ ] All changes have test coverage:
Unit tests are added for small changes to verify correctness (e.g. adding a new operator)
Nightly tests are added for complicated/long-running ones (e.g. changing distributed kvstore)
Build tests will be added for build configuration changes (e.g. adding a new build option with NCCL)
[ ] Code is well-documented:
For user-facing API changes, API doc string has been updated.
For new C++ functions in header files, their functionalities and arguments are documented.
For new examples, README.md is added to explain the what the example does, the source of the dataset, expected performance on test set and reference to the original paper if applicable
Check the API doc at http://mxnet-ci-doc.s3-accelerate.dualstack.amazonaws.com/PR-$PR_ID/$BUILD_ID/index.html
[ ] To the my best knowledge, examples are either not affected by this change, or have been fixed to be compatible with this change

Changes

[x] Fix ACC_TYPE_SWITCH macro
[x] Extra test for integer norm

Comments
@reminisce

@eric-haibin-lin @reminisce @szha


findspark.init(spark_home) is never called

in pytest_configure, find_spark_home_var is called, which will raise an error if it can't find spark_home.  But, find_spark_home_var never calls findspark.init(spark_home), which previously could automatically find the spark_home without being set anywhere by looking in common locations. This means we cannot run any pytests that were previously running due to not being able to fine spark_home

UPDATE: The issue actually is that it raises an error without checking the PYTHONPATH. In our workflow, we specify the spark locations via the PYTHONPATH rather than with SPARK_HOME. This has been a valid way to be able to use pyspark with pytest and the configure should not raise an error if this is the case. The previous version of this code did not raise an error, which was allowed the code to run successfully.

@swhitelaw thanks for the report!
I have removed raising the error and this logic should be similar to the one in v 0.4.0 which I believe you were using before.
Please, check the new version 0.4.4 (already on pypi)

Hi @malexer,
I believe that fixed our issue. Tested with the following commands without SPARK_HOME defined:
pip install pytest-spark --upgrade
pytest --version
Previously this command would fail with the raised error. It now runs without issue. Thanks for the quick response and fix.
I have one more test to confirm. Will update when that test is completed.

Great!
BTW you can also override all settings by command line option pytest --spark_home=<path>

UPDATE: confirming that this fix did resolve the issue.


VerticalCollection causing Ember Acceptance tests to fail (run-loop's autorun disabled warning)

Hi!
I'm having difficulty getting my Ember Acceptance tests to pass once I start using the VerticalCollection in 0.3.6.
Here is a simplified version of my test
visit '/page' // Page loads and renders vertical collection for the first time
andThen ->
  click('.changeArrayContents')  // Changes the contents of the array bound to the vertical collection
andThen ->

On the final andThen I get the following
Assertion Failed: You have turned on testing mode, which disabled the run-loop's autorun.
                  You will need to wrap any code with asynchronous side-effects in a run

Adding a breakpoint shows that occulusion-collection#_updateChildStates gets called which in turn calls
this.sendActionOnce('firstReached', {
            item: component,
            index: bottomComponentIndex
          })

which triggers the error above.
Am I using the collection incorrectly?
Maybe I need to make use of this option?
  /**!
   * Set this to false to prevent rendering entirely.
   * Useful for situations in which rendering is
   * expensive enough that it interferes with a
   * transition animation.
   *
   * In such cases, set this to false, and switch it
   * to true once animation has completed.
   */
    shouldRender: true,

Thanks!

It's highly possible that this is the collection's fault and not yours. I use the run loop within a special container in order to make teardown less buggy.  This was necessary when dozens of calls were being scheduled into the loop, but now there are only a small handful.  This run loop container is on the chopping block for 0.3.7

This is likely fixed by #51, would be great if you could check out the latest master and report back

Sorry, meant the PR #60 that's now on master.

Hi @runspired
I am also facing same issue while running acceptance test cases. On debug, I found that its failing at
https://github.com/runspired/smoke-and-mirrors/blob/develop/addon/models/radar.js#L278
I am using the latest version 0.5.1
offers page has list of all offers and am using vertical-collection component:
        {{#vertical-collection
          content=arrangedContent
          alwaysUseDefaultHeight=true
          containerSelector="body"
          as |offer|
          }}
              {{partial 'offers/offer_details'}}
          {{/vertical-collection}}

Here is the test case:
test("redirect to offers page", function() {
  visit("/offers");
  andThen(function(){
    equal(currentURL(), "/offers/submitted");
    equal(find("ul.list li").length, 1);
    equal(find("ul.list img").length, 1);
});

Do I need to wrap the whole test inside Ember.run(() => {...}) ?
Am I missing anything?

Upgrading node-version and ember version fixed issue with test cases. Thanks!
cc: @runspired

@swatijadhav what ember version did you upgrade to? i am seeing this error on v2.4.1

@joeyjiron06 Previously I was facing this issue on ember v1.13.8 and after upgrading it to v1.13.13 worked for us.
But today after upgrading it to ember v2.1.0, am getting this issue again. and strange thing that you are facing it on v2.4.1 :(
Please let me know if you able to get the solution for it. Thanks.

I also see this issue on ember v1.13.13 - @swatijadhav @joeyjiron06 did you find a solution for this?

I have suspicions as to what is causing it and what will fix it, but won't have time to really dig in until next week.  I'm trying to get all my addons back in shape one at a time, and this week is Flexi and ember-hammertime's turn :D


Customize RetryHandler in Zuul

We are using Camden SR4. I have enabled ribbon.httpclient.enabled: true
Digging into the code, the retry handler is instantiating the RequestSpecificRetryHandler directly. Is there a way to override the retry handler? I tried to create a decorator and creating a BeanPostProcessor which would wrap the RibbonLoadBalancingHttpClient to return the retry handler that I wanted, but my bean post processor was not included as a post processor for that bean.

Is there any chance you can upgrade to Camden.SR7?  The logic to retry requests has completely changed in later releases.  What functionality are you trying to customize?

I wanted to only retry on ConnectTimeout or ConnectionRefused errors.

Let me know what you see after upgrading.


buddy_list: Show unsubscribed participants in participants list.

Followup to https://github.com/zulip/zulip/pull/32470#issuecomment-2504958549

Please add comments documenting the reasoning here, and ideally test coverage if that's easy (I expect it should be?) -- if there's a subtle detail like this that can be broken, it can be broken in refactoring too.

It's not the most straight forward to add a test. We don't actually even have any test coverage for participants in the buddy_list test file. To more easily test, we'd probably want to split the logic out more from the UI/JQuery code. I can also spend time setting up the necessary mocks and stuff to add more test coverage as is, if you want!
I've updated the commit to add the motivation, and also fixed a linter failure.

Merged, thanks @evykassirer!


improved validation in driver factory

Correção das factories dos drivers Annotations, Yaml e Xml quando na configuração é informado um array vazio. Os testes com configuração vazia falhavam

Temos um problema de dependências em cascata. Veja:

Este projeto tem depende de "doctrine/mongodb-odm": "^1.2"
O doctrine/mongodb-odm:1.2 depende de "doctrine/annotations": "~1.2" (ou seja, >=1.2 <2.0.0)
Já o doctrine/annotations: 1.6 requer php: ^7.1

Este é uma das causas para o erro no teste com php5.6:
https://github.com/doctrine/annotations/blob/1.5/lib/Doctrine/Common/Annotations/AnnotationRegistry.php#L50

@matwright Why did you merge this PR into your PR? could please revert this?

@helderjs because I received a pull request. There was a minor conflict in the composer file.
Did you send a PR to the wrong branch/fork?

@helderjs I've reverted the merge. Do I need to do anything else?

@matwright That's strage. I'm still seeing both PR merged into each other. We need to revert these changes to do a proper merge o master.

@matwright and @vinaocruz I really don't know what happened here. So I going to merge this PR and close the https://github.com/helderjs/doctrine-mongo-odm/pull/3. Before create a new release I'll do some checks to avoid BC breaks.


use median value when multiple parameters are available in Fishbase/SealifeBase

from @agruss2 email communication -
● When the OSMOSE API employs the data stored in the data archives to derive values for OSMOSE parameters, there can actually be several values for a given parameters. For example, there are many different values available for red snapper’s Linf, as we already observed here a few days ago: https://www.fishbase.ca/popdyn/PopGrowthList.php?ID=1423&GenusName=Lutjanus&SpeciesName=campechanus&fc=323
However, the OSMOSE API picks the first of these many values to produce an OSMOSE parameter estimate (63 cm in the case of red snapper’s Linf), instead of calculating a median value from the many values provided in the data archives (the median value would be around 94 cm in the case of red snapper’s Linf).

Please provide an exhaustive list of parameters for which you'd like to use the median values if multiple values are present. Also, please confirm that calculated values (e.g., "species.vonbertalanffy.threshold.age.sp") will use the median values as inputs.

@jhpoelen
The median values should be calculated from the FishBase/SeaLifeBase (FB/SLB) parameters before the OSMOSE API makes any calculation.
Thus, the exhaustive list of parameters for which I would like a median value to be calculated if multiple values are present are FB/SLB parameters, but changes need to be made in several different Java classes because a number of FB/SLB parameters are used to derive several different OSMOSE parameters.
Here is the exhaustive list of parameters:
(1) LongevityWild (FB/SLB table: species), used to calculate longevity for OSMOSE.
(2) K (FB/SLB table: popgrowth), used to calculate instantaneous growth rate at small size for OSMOSE.
(3) Loo (FB/SLB table: popgrowth), used to calculate Linf for OSMOSE.
(4) to (FB/SLB table: popgrowth), used to calculate the theoretical age of zero length for OSMOSE.
(5) a (FB/SLB table: poplw), used to calculate the constant of proportionality of the allometric length-weight relationship for OSMOSE.
(6) b (FB/SLB table: poplw), used to calculate the exponent of the allometric length-weight relationship for OSMOSE.
(7) Lm (FB/SLB table: maturity), used to calculate body size at sexual maturity for OSMOSE.
(8) tm (FB/SLB table: maturity), used to calculate age at sexual maturity for OSMOSE.
(9) LengthMin (FB/SLB table: poplw), to, Loo, and K (FB/SLB table: popgrowth), and LongevityWild (FB/SLB table: species), used to calculate age at sexual maturity for OSMOSE.
(10) Troph (FB/SLB table: estimate), used to calculate trophic level for OSMOSE.
(11) LengthMin (FB/SLB table: popll), used to calculate minimum body size for OSMOSE.
(12) LengthMax (FB/SLB table: popll), used to calculate maximum body size for OSMOSE.
(13) mortality (FB/SLB table: popqb), used to calculate diverse natural mortality rate for OSMOSE.
(14) Lm (FB/SLB table: maturity), used to calculate the body size threshold that separates the predator/prey size ratios for juvenile individuals from those for adult individuals  for OSMOSE.
(15) tm (FB/SLB table: maturity), used to calculate the age threshold that separates the accessibility coefficients of the juvenile and adult stages of focal functional groups for OSMOSE.
Please let me know if that makes sense and/or if you need anything else from me. Many thanks !!

I deployed a first pass at implementing the desired new functionality to calculate median values for specific provided parameters. Please confirm that the functionality has been implemented as desired.

@jhpoelen I just ran some tests and can confirm that the present issue has been fully solved. Many thanks !! I can now close this issue.


[opentitantool] Persistent sessions

Problem
OpenTitan tool allows for a variety of backend "Transports", some of which are drivers for hardware debuggers, others emulate OpenTitan or related chips.
By their nature, some of those drivers will maintain state in the Rust struct that implements the Transport trait, others may have USB connections to hardware, which gets disturbed if the USB connection is closed and restablished.
Rust programs which link directly with OpenTitan lib are able to keep the Transport object for a long as needed, making a sequence of method invocations to "stimulate" the device under test.  However, users of the command line OpenTitan tool does not have that ability, since each command line invocation generally results in the creation of a Transport object, and the one "useful" method invocation, before the Transport is closed as the process terminates.
Proposal
We will add a new optional persistent “OpenTitan session proxy” process (a daemon).  When used, all communication with the Transport backend will go through this proxy process.  The proxy process will listen on a TCP socket and contain the "real" Transport implementation, maintaining emulator state or connection to USB hardware.  The command line OpenTitan tool will use a "stub" Transport implementation, which forwards all requests via a TCP connection to the proxy.
Design document: https://docs.google.com/document/d/18JIqdYcmJKDzANheNS9JI1rEre6-iXPZhjMjliAS0TE/edit#
Tasks

[x] Make sure all error return values from Transport methods can be serialized (with serde) (#10964)
[x] Declare wire protocol by as a number of Rust structs/enums (#10994)
[x] Implement "stub" Transport (using serde JSON text via TCP) (#11023)
[x] Implement proxy server functionality (opentitansession binary) (#11054)
[x] Ability for opentitansession to run in background (as daemon) (#11173)

This proposal has significant overlap with previous issue #10217, which proposed a JSON TCP protocol for communicating with an emulator running in a separate process.  With this proposal, the interface to the emulator would instead be the Transport trait itself, that is, the top-level process of the emulator would either be opentitansession, or would be another binary that links with a library containing most of the functionality of opentitansession (the listening on TCP socket, and translation of protocol into calls to Transport trait.)

Final PR has merged a long time ago.


Recipe ReplaceLambdaWithMethodReference results in non compilable code cause it does not added an import for a needed class.

Running ReplaceLambdaWithMethodReference in this simpliefied example results in code which no longer compiles
Original code
String fullName = Optional.ofNullable("Turing")
		.map(n -> UserManager.INSTANCE.getUser(n))
		.map(user -> user.getFullName()) // works without needing an ímport User' line
		.orElse("");

Converted code
String fullName = Optional.ofNullable("Turing")
		.map(UserManager.INSTANCE::getUser)
		.map(User::getFullName) // Causes that code no longer compiles
		.orElse("");

It does not compile because a 'import User' line is missing. In the original code UserManager.INSTANCE.getUser(n) returns an object of type User. No import for it is needed. In the converted code now the class User is explicitly used and a method in it called. An 'import class' in IntelliJ fixes the problem. Hence ReplaceLambdaWithMethodReference should create an import when needed.

@jbessels Thanks for the report! I was able to reproduce the error using this test class:
              import java.net.URI;
              import java.nio.file.Paths;
              import java.util.Optional;

              class A {
                  void m() {
                      URI uri = Optional.ofNullable("path")
                            .map(Paths::get)
                            .map(path -> path.toUri())
                            .orElse(null);
                  }
              }

Thanks, that it is fixed. The majority (85%+) of 'breaking code aka not compiling' code after running recipes has to do with Lambda's for our organic/exotic repo.Another repo had very little Lambda issues.  The nasty repo  uses Wicket (web framework, where often 6 lines of code can be replaced by 1 Lambda line)  Ambiguity issues (more tickets incoming) is a real can of worms. And as you said in another ticket 'surprisingly difficult to detect'. Atm I have to exclude about 40 files wrt Lambda edge case.  Some issues affect 8+ files.
When will this fix finds its way in the 4.43.0-SNASHOT - which uses 7.39.0-SNAPSHOT?? Aka when can I test it?

Not sure if comments in closed tickets are read. Just reran OpenRewrite again and it still fails. No import for User is added.
    <!-- OpenRewrite -->
    <rewrite-maven-plugin.version>4.45.0</rewrite-maven-plugin.version>
    <rewrite-migrate-java.version>1.21.1</rewrite-migrate-java.version>
    <rewrite-testing-frameworks.version>1.37.0</rewrite-testing-frameworks.version>

@jbessels That is odd. Is this in a public repo somewhere? It looks similar enough to the test case I added, yet there must be a subtle difference somewhere...

Yes must be. I've double checked and it still fails. Can you point me to your test case and the changes made to code?

@jbessels Here is the test that I added: https://github.com/openrewrite/rewrite/blob/b9a0475eb72a5cdde5003a64f57bae0a9bf08cb4/rewrite-java-test/src/test/java/org/openrewrite/java/cleanup/ReplaceLambdaWithMethodReferenceTest.java#L966-L1003

@knutwannheden Missed your response. It DOES work with your example. Not with my code.
I've inserted your test code in the same class as where the User import was not added. Your Path import was correctly added. Paths and Path are in the same Java package. This got me thinking. class User is from another module (with another package name). Because running openRewrite on the whole big 100++ Maven modules repo is painfully slow I ran it for the module where it did go wrong. Hence the complete LST for the repo is NOT available and class User is unknown. I hoped that when running it for the whole repo it would work. Unfortunately NOT.
I checked and there is only one User.java class just to be sure that it would not break with multiple classes of the same name. Beneath a screenshot of  what IntelliJ shows. Java knows that the line above user -> user.getFullName() results in an User object.

@jbessels Thanks a lot for your investigation! It would seem like the recipe needs to make sure that the type attribution is complete before making the substitution. I will report back when I know more.

@jbessels I took another look at the code and still can't quite figure it out. What I did find however, is that the recipe doesn't properly deal with nested classes. So if your User class for example is a nested class in UserManager, I can see how this could go wrong. Can you check whether that is the case?

Did some digging. UserManager is an enum. No nested class in it.
public enum UserManager implements Caching { .... }
User is a normal class
public class User extends CRUDObject<User> implements Comparable<User>, Serializable {
If needed I can send you the involved  classes in a DM Slack.

@jbessels Yes, it would be great if you could send me all I need to reproduce the problem.

Will send it all this evening. That pesky sprint work is in the way ;-)

I've retested it. Somewhere between last comment in the thread and the latest version of the plugin it got solved. As in the User object is added. See screenshot. Hence 3071 can be closed.


[WIP] Adding linear synthesis algorithm for LNN

Summary
Adding a synthesis algorithm for linear reversible circuits (containing only CX gates) for linear-nearest-neighbor connectivity.
See #9036
Co-authored by Ben Zindorf.
Details and comments
According to the paper: https://arxiv.org/abs/quant-ph/0701194
The depth of an n-qubit linear reversible circuit is bounded by 5n.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

127 of 129 (98.45%)  changed or added relevant lines in 2 files are covered.
2 unchanged lines in 1 file lost coverage.
Overall coverage increased (+0.03%) to 84.564%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

qiskit/synthesis/linear/linear_depth_lnn.py
126
128
98.44%

Files with Coverage Reduction
New Missed Lines
%

qiskit/extensions/quantum_initializer/squ.py
2
79.78%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
62581

Relevant Lines:
74004

💛  - Coveralls


[Event Request] Codeunit 90 Purch.-Post - OnBeforeDivideAmountOnAfterInitAmout

Hi, would it be possible to get an event OnBeforeDivideAmountOnAfterInitAmout in Codeunit 90 Purch Post, procedure DivideAmounts, like this :

?
Thanks

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. We will update this issue with information about availability. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.


Changes to port back from FRCG

Estimates

[x] (8 hours) bring code changes back from FRCG, based on meetings with James and with Julie this week on "what is 'base'" (@JulesKhong )
[ ] [ 3 hours? ] Code review changes (@jamacon36)
[ ] [ 3 hours] make fixes based on code review (@JulesKhong or @jamacon36 )
[ ] (2 hours) QA fixes

Next step

[ ] #71 Demo Site so design has something to QA against
[ ] #65 QA while sync'ing code with Julie in Figma (@JulesKhong )

Assets

[x] Update css file structure to match atomic naming approach.

[x] Update base JS functions with final versions

Tokens

[x] Update button component file to bring in the rest of the base button styles
[x] Update base grids file with abstracted gap values
[x] Overall: update base token files to match finalized FRCG values and efficiencies
[x] Add p-site to site-spacing.json
[x] Add the rest of the base typography styles to match full set from FRCG
[x] Ensure any new styles are included in /style-guide

Views

[x] Update the views directory to match atomic naming conventions in FRCG

Buttons
[ ] btn-dropdown
[x] btn-social
[x] btn-tag
[ ] btn-tag--checkbox
[x] btn-text-link
[x] btn-underline

SVGS

[x] Add basic social svs, ensure social block is populated, add to style guide.

Cards
Update card styles to use the latest refinements from FRCG

[x] Card-event
[x] card-listing
[x] card-md
[x] card-lg
[x] card-xl
[x] card-overlay-link
[x] card-person
[x] example--card

Images
Bring over caption files so captions above/below functionality is working and has base styles

[x] caption--below
[x] caption--above
[x] media

Pagination
Bring over base pagination

[x] pagination.twig and pagination.css
[x] Add it to index.css
[ ] ensure it's working on the style guide

Section title

[ ] Bring over updates from section title component across layouts, including new properties

Blocks

[ ] BANNER: Bring over banner updates, including video banner, default, fullBleed, homepage, insetCentered, overlay, and side-text. Ensure hero vs cta logic and h1 vs h2 logic is working.
[ ] Confirm js and css are still working on the style guide, and with video.
[ ] Card Grid block updates
[ ] CTA block updates
[ ] Image-text block updates
[ ] News grid block updates
[ ] people cards block updates
[ ] Ensure all blocks are appearing on the style guide, and are accessible with automated and manual testing

Footer

[x] Port back footer and footer menu base work
[x] add footer and footer menus to the style guide

Header

[x] Port back header and header base menus (base and utility, desktop and mobile, submenus)
[x] port back css and js, test to confirm accessibility is intact

Style guide sections directory

[x] Update file structure to match atomic naming conventions from frcg

Other

[x] Update menu.twig
[x] Update template directory file structure and naming conventions, and update all references in .php files

[x] Update 404.twig to have base styles
[x] update pages/page-landing-page.twig
[x] update posts/single.twig
[x] update pages/page-style-guide.twig
[x] update archive/archive.twig
[x] update page/our-team.twig

@JulesKhong assuming the todos in this ticket are correct, what do you think is easiest to work with — I can pull each of them into their own tickets, you could finish them here, or...?


Import multiple actors in a single topic into a sub-folder with the name of the topic

Where a RW topic has more than one actor, arrange the actors in a sub-folder named after the topic. This should help with organisation and looking for tokens for scenes.

Implemented in 0.3.15


Update build.gradle

To update your build.gradle file at line 24, I'd need to know the context of what's currently in the file and what changes you want to implement.
Here’s a generic outline of what you might want to update in a typical build.gradle file:

Dependencies: Adding, removing, or updating dependencies.
Plugins: Modifying or adding plugins used in the project.
Repositories: Adding or changing repositories from which dependencies are fetched.
Configurations: Updating settings related to compilation, testing, or other configurations.


update per STO-6748

Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the CODEOWNERS.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://6595b3a83aedf307d91eb01f--harness-developer.netlify.app

Please check the Execution Link of the Pipeline for the Website Draft URL. This is located in the Preview Step behind the Harness VPN and also is available in #hdh_alerts. E.g Website Draft URL: https://unique-id--harness-developer.netlify.app. Current Draft URL is: https://659709d44d712b2e7b1198e9--harness-developer.netlify.app


ResponseBot repository got rebased after being published.

When this repository was initially published, it had one commit:
commit 8f01ce6f6f8d017246caf0e87dc81c3b45875121
Author: Huy Nguyen+Giang Nguyen <...omitted...>
Date:   Sat May 7 10:50:37 2016 +0700

    Initial commit

I submitted pull request 1 based on that commit.
But now when I look at the history, the initial commit is different:
commit 0aad906f7abcd5081f87776a11fbee3ec42ddb27
Author: Huy Nguyen+Giang Nguyen <...omitted...>
Date:   Sat May 7 10:50:37 2016 +0700

    Initial commit

And I can see that the contents (as compared to the old initial commit) are very different.  In the new version of the initial commit, there is a new file, tests/unit_tests/models/test_tweet_filter.py, and there are differences in 17 other files.
Now I'm worried :-).  Is it going to get rebased again?  Is this repository stable enough to use?
If a repository is rebased after it is published, that creates a problem everyone else who cloned that repository and has local branches or local changes in their clone.  See this section of the git-rebase manual page:
RECOVERING FROM UPSTREAM REBASE
    Rebasing (or any other form of rewriting) a branch that others
    have based work on is a bad idea: anyone downstream of it is
    forced to manually fix their history. This section explains
    how to do the fix from the downstream’s point of view. The
    real fix, however, would be to avoid rebasing the upstream in
    the first place.

@kfogel Sorry for the inconvenience, we did push a change in the initial commit, please rebase your branch with the new master. Thanks for noticing us, we will avoid doing that in the future.

Will do!  Thanks for clarifying.

@kfogel We'll re-apply the changes from your PR, since it's our fault that it got closed :)


x64 build fails on windows

Seeing this error when trying to build for x64 on Windows 10, with Visual Studio 2015 update 3
ninja: Entering directory `src\out-x64\static_library'                                                                                [11191/18516] CXX obj/media/media/decoder_selector.obj
FAILED: obj/media/media/decoder_selector.obj
C:/Python27/python.exe ../../build/toolchain/win/tool_wrapper.py cl-wrapper ninja -t msvc -e environment.x64 -- "C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\BIN\amd64/cl.exe" /nologo /showIncludes /FC @obj/media/media/decoder_selector.obj.rsp /c ../../media/filters/decoder_selector.cc /Foobj/media/media/decoder_selector.obj obj/media "media_cc.pdb"
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.cc(246): error C2220: warning treated as error - no 'object' file generated
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.cc(246): warning C4661: 'void media::DecoderSelector<media::DemuxerStream::VIDEO>::InitializeDecryptingDecoder(void)': no suitable definition provided for explicit template instantiation request
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.h(83): note: see declaration of 'media::DecoderSelector<media::DemuxerStream::VIDEO>::InitializeDecryptingDecoder'
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.cc(246): warning C4661: 'void media::DecoderSelector<media::DemuxerStream::VIDEO>::DecryptingDecoderInitDone(bool)': no suitable definition provided for explicit template instantiation request
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.h(84): note: see declaration of 'media::DecoderSelector<media::DemuxerStream::VIDEO>::DecryptingDecoderInitDone'
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.cc(245): warning C4661: 'void media::DecoderSelector<media::DemuxerStream::AUDIO>::InitializeDecryptingDecoder(void)': no suitable definition provided for explicit template instantiation request
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.h(83): note: see declaration of 'media::DecoderSelector<media::DemuxerStream::AUDIO>::InitializeDecryptingDecoder'
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.cc(245): warning C4661: 'void media::DecoderSelector<media::DemuxerStream::AUDIO>::DecryptingDecoderInitDone(bool)': no suitable definition provided for explicit template instantiation request
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.h(84): note: see declaration of 'media::DecoderSelector<media::DemuxerStream::AUDIO>::DecryptingDecoderInitDone'
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.cc(52): warning C4661: 'void media::DecoderSelector<media::DemuxerStream::VIDEO>::InitializeDecryptingDecoder(void)': no suitable definition provided for explicit template instantiation request
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.h(83): note: see declaration of 'media::DecoderSelector<media::DemuxerStream::VIDEO>::InitializeDecryptingDecoder'
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.cc(52): warning C4661: 'void media::DecoderSelector<media::DemuxerStream::VIDEO>::DecryptingDecoderInitDone(bool)': no suitable definition provided for explicit template instantiation request
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.h(84): note: see declaration of 'media::DecoderSelector<media::DemuxerStream::VIDEO>::DecryptingDecoderInitDone'
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.cc(52): warning C4661: 'void media::DecoderSelector<media::DemuxerStream::AUDIO>::InitializeDecryptingDecoder(void)': no suitable definition provided for explicit template instantiation request
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.h(83): note: see declaration of 'media::DecoderSelector<media::DemuxerStream::AUDIO>::InitializeDecryptingDecoder'
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.cc(52): warning C4661: 'void media::DecoderSelector<media::DemuxerStream::AUDIO>::DecryptingDecoderInitDone(bool)': no suitable definition provided for explicit template instantiation request
c:\users\denimuser\playground\libchromiumcontent\src\media\filters\decoder_selector.h(84): note: see declaration of 'media::DecoderSelector<media::DemuxerStream::AUDIO>::DecryptingDecoderInitDone'
[11200/18516] CXX obj/content/public/browser/browser_sources/web_contents_observer.obj
ninja: build stopped: subcommand failed.
Traceback (most recent call last):
  File "script\build", line 66, in <module>
    sys.exit(main())
  File "script\build", line 45, in main
    subprocess.check_call([NINJA, '-C', os.path.relpath(out_dir), target], env=env)
  File "C:\Python27\lib\subprocess.py", line 186, in check_call
    raise CalledProcessError(retcode, cmd)
subprocess.CalledProcessError: Command '['C:\\Users\\denimuser\\playground\\libchromiumcontent\\vendor\\depot_tools\\ninja.exe', '-C', 'src\\out-x64\\static_library', 'chromiumcontent:chromiumcontent']' returned non-zero exit status 1

Is any one able to compile 64 bit ?

From the slack channel, for electron I got to know that this can be the issue with Windows SDK version, the read me link for this repo is pointing to build the latest chromium in the prerequisite, which asks to install Visual Studio 2017, I think the version of Chromium requires Visual Studio 2015 update 3 and I have Windows SDK 10.0.14393.795. Can any body point me to the correct version of Visual Studio and Windows SDK to install?

@anupam

to build the latest chromium in the prerequisite

I don't have to build Chromium to build libcc.

Can any body point me to the correct version of Visual Studio and Windows SDK to install?

The latest libcc is based on Chromium 59.0.3071.115, here are its build instructions for Windows: https://chromium.googlesource.com/chromium/src/+/59.0.3071.115/docs/windows_build_instructions.md You can find system requirements and environment setup instructions there.
Closed.

Thanks it helped and I was able to build it.


Provide Shipyard shared infra in all submariner-io repos

Currently, some repositories don't provide the shared Shipyard infrastructure:

https://github.com/submariner-io/submariner-website
https://github.com/submariner-io/enhancements
~https://github.com/submariner-io/submariner-charts~

It would be useful to be able to run local linting in those repos, via the shared Shipyard tooling, instead of relying on GHAs in CI.
While adding Shipyard infra to these projects, the update/refactor the (now-removed) docs about the process of adding Shipyard to repos.

As a part of this effort, we should make sure we end up running the shared infra's gitlint tooling in the new-to-Shipyard repos. https://github.com/submariner-io/submariner/issues/799

Talking about this on the Automation sync today, it doesn't seem to make much sense, or at least not provide a lot of value, to do this for the website and enhancements. Charts was the main goal.


[7.x] fix 2 bugs that caused defaultTimezone not to be respected in scheduled Events

Version Bump dragonmantank/cron-expression
-> fixes a bug that causes cron schedules to drop timezone settings (see changelog: https://github.com/dragonmantank/cron-expression/blob/master/CHANGELOG.md)
Update CallbackEvent
-> CallbackEvents were not injected with the timezone on construction making them ignore the schedulers timezone on $event->between() settings

Hey @sodoardi can you send this to 6.x (current LTS version)? Thanks


fix: build sys

Description
Force base to rebuild when circuits changes in order to get the new wasm
Checklist:

[ ] I have reviewed my diff in github, line by line.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to the issue(s) that it resolves.
[ ] There are no unexpected formatting changes, superfluous debug logs, or commented-out code.
[ ] The branch has been merged or rebased against the head of its merge target.
[ ] I'm happy for the PR to be merged at the reviewer's next convenience.

superseded by https://github.com/AztecProtocol/aztec3-packages/pull/197


[Feature Request] Live boss kill timer

First of all, thanks for the plugin - seems to work great!
Any chance we could get a "live" kill timer in the panel (or, enabling changing the default infobox text to another value)? Took a quick look at the code, and figured that the killTimerStart would be the reasonable variable to show - but wasn't quite sure how it would be implemented. Sure, I could just use a stopwatch every time, but figured I'd ask, since the data seems to be available.
Cheers!

Yes please

I will look into adding this when I update the plugin to add party compatibility & TOA support.


Update peer dependencies?

I am requiring "@infinumjs/eslint-config": "^1.1.0" in a project and I got:

<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself

I also have "eslint": "^5.4.0" in my devDependencies, and the linter works fine.
Should the peerDependencies be set to >=4.x?
https://github.com/infinum/js-linters/blob/b83344a623f5566750926fd63d930b55c31b91fa/eslint-config/package.json#L24

Maybe the peer dependency should be updated to 5.x.

If there is no reason to deprecate eslint 4 support, we could go with"^4.x || ^5.x". Something like >=4.x is too dangerous, since we can't know if everything will work with eslint 6

I have published new package versions.


added inheritance for all huggingface models

issue #69 adds Bert to onsite llms
this PR also modifies each class for the local LLM models so that it inherits functionality from the Base LLM class

Looks good to me

This looks like good work @ajitalapati


The googleAdsRow.getCampaignCriterion().getBidModifier().getValue() both return 0 when Google Ads UI Bid adjustment = "--" or -100%

Which version of the client library are you using?

Which version of Java are you using?
jre1.8.0_211
Which operating system (Linux, Windows, ...) and version?
Linux Darwin 11.4.2 and Windows 7
Actions taken
Please provide a code snippet or steps to consistently reproduce the experienced behavior.
[DO NOT INCLUDE ANY PERSONAL OR SENSITIVE DATA IN YOUR SUBMITTED CODE]

query =
SELECT
customer.id,
customer.currency_code,
customer.descriptive_name,
campaign.id,
campaign.name,
campaign_criterion.criterion_id,
campaign_criterion.type,
campaign_criterion.status,
campaign_criterion.bid_modifier,
campaign_criterion.device.type,
campaign_criterion.negative
FROM
campaign_criterion
WHERE
campaign_criterion.type IN ('DEVICE','MOBILE_DEVICE')

Expected result
we expect the return value of bid modifier can distinguish between the "--" and -100%
But all the device bid modifier result are "0".  Is there any other way to recognize them?
As you can see in the UI of below image

Actual result

Make sure you include information that can help us debug (full error message, exception listing, stack trace, logs).
See troubleshooting for how to diagnose problems better.
Anything else we should know about your project / environment

Hey, thanks for reaching out. You can use CampaignCriterion.hasBidModifier() to detect if the bid modifier was present or not.
The general idea is that all scalar value type fields in a protocol buffer (v3) message have a default value. So let's say you have a message:
message CampaignCriterion {
  double bid_modifier = 1;
}

When you send a response not containing this field, it will always be 0.
CampaignCriterion c = CampaignCrtierion.newBuilder().build();
c.getBidModifier(); // returns 0.0

To represent presence in this message, we add a wrapper type DoubleValue.
message DoubleValue {
  double value = 1;
}

message CampaignCriterion {
  DoubleValue bid_modifier = 1;
}

Now when you access the wrapped value, you still get back 0.0, but you can also test to see if the value was present:
CampaignCriterion c = CampaignCrtierion.newBuilder().build();
c.getBidModifier().getValue(); // returns 0.0
c.hasBidModifier(); // returns false

All scalar values that are optional in Google Ads API are represented using wrapper types.
Hope this helps, please let me know if you have any questions.


[Issue]

[ ] Make sure you've installed the latest version using instructions in the wiki

Output from azd version
Run azd version and copy and paste the output here:
Describe the bug
Description of issue you're seeing...
To Reproduce
Steps to reproduce the behavior...
Expected behavior
A clear and concise description of what you expected to happen.
Environment
Information on your environment:
* Language name and version
* IDE and version : [e.g. Visual Studio 16.3]
Additional context
Add any other context about the problem here.

@james-britton can you add some details to the issue?


Update postprocessing example.

The setup name was not accessing the hfss app correctly.
Fix #1788.

Already implemented by #1798 .


remove subsystem

https://github.com/xmake-io/xmake-repo/issues/3082
https://github.com/xmake-io/xmake-repo/issues/3139

error: D:\a\xmake-repo\xmake-repo\packages\l\libsdl\xmake.lua:231: ...gramdir\core\sandbox\modules\import\core\tool\linker.lua:75: @programdir\modules\core\tools\link.lua:175:
LINK : fatal error LNK1561: entry point must be defined

What's the reason xmake keeps adding it even when "main" component isn't active?

What's the reason xmake keeps adding it even when "main" component isn't active?

I don't know, but I have not time to debug it.


Question for lecture 10

Hej Jeppe
My question is about solving non-linear equation: In section 4.2 you show how to solve a non-linear equation by e.g. using derivative based method, but in section 3.4 you also show a method for solving multiple equations (sol = sm.solve([Eq1,Eq2],[x,y])). The latter method can also be used to solve a single equation, but then what is the difference? In other words, is the method shown in section 3.4 a valid method for solving?
In advance, thank you very much for the help :)

Hi Jensen
I am not quite sure I understand the question:
Section 2.4 is using a symbolic solver. Will only work with algebraically formulated problems.
Section 4.2 is using a numerical solver. Will work for any computable problem.
Does this answer your question?
Best, Jeppe

Hello Jeppe
Thank you for the help !
It does answer my question :)
Best, Jens


Incompatibility with AWS API Gateway

Vendia's serverless-express package does not work as expected with AWS API Gateway, it only returns a blank screen instead the swagger screen.
This issue is forcing us to use the serverless-http package.
To reproduce the error, you basically have to:

Create an AWS Lambda function* with a simple project containing express, serverless-http or vendia and swagger;
Configure the AWS API Gateway to consume the Lambda function created in the step 1.

*Node versions tried: 14/16/18/20
BTW. Using AWS Lambda function URLs Vendia's serverless-express package works pretty fine with swagger.
Example of a simple project containing express, serverless-http or vendia and swagger:
const express = require('express')
const serverless = require('serverless-http')
const swaggerUI = require('swagger-ui-express')

const app = express()
app.use(
  '/swagger',
  swaggerUI.serve,
  swaggerUI.setup(null, {
    swaggerOptions: {
      url: "https://petstore.swagger.io/v2/swagger.json"
    }
  })
)

module.exports.handler = serverless(app)

Thanks for the report! I'll take a look and update you.

Same problem here, I solved using serverless-http instead.

Did this only just start happening? Is it only an issue with Swagger UI?

Did this only just start happening? Is it only an issue when using Swagger UI?

I had never used serverless-express with swagger-ui-express and API Gateway but when I tried it (monday) I had this problem with the blank screen on the swagger route.

Related to https://github.com/brettstack/serverless-express/issues/400?

Related to #400?

No, it's not!

This error can be caused by low lambda concurrency, below 5 the Swagger will not render.


Add 'make serve' to the detailed installation instructions

This was added to the "Quickstart" section of the Getting Started guide, but not the detailed instructions.  Hoping this will make it easier for beginners.

✨


Flashing icons without styles on load

Awesome work on this template. I loved how simple it was to get started with.
I've been experiencing flashing of content without styles on load. I tried the usual Vue tricks to prevent it but doesn't seem to work. Is there anything you guys can recommend?

Sorry, found the solution.

@milosdakic how did you fix it?

@sergejcodes thanks!

@sergejcodes hey! I still got this problem with a live-site on shopify https://snuffelbox.nl/. Any idea how I can fix this?

Hi 👋,
I just clicked through your site and all icons seem to be fine, no issues here. I'm on the latest Chrome version. You have to be a bit more specific. One thing you could try is to add an initial fixed width and height to your svgs like this:
<svg width="24" height="24" viewBox="0 0 24 24" >...</svg>

Hey! Thanks for the quick reply, much appreciated! I've deployed to production and now it's working indeed. One other issue I have everytime I push to production with: npm run deploy:live -- --allow-live it's overwriting my configured shopify images. Any idea why that's happening?

Do you mean images you set through the Shopify theme editor? If yes, you should run npm run settings:live to download the latest live settings_data.json file before deploying.
By default, the deploy task overrides everything on the remote store, so you have to download the settings file if someone makes separate adjustments on the live store which differ from the dev store.
Depending on your workflow you could incorporate a CI/CD workflow that does exactly that (described in the Shopify Theme Lab readme) and/or Shopify Settings Control for backing up the live settings_data.json to prevent data loss.

Thanks


Don't double-flush HTTP/1 Content-Length responses

This should fix the biggest single cause of the perf regression discussed in #11778. The regression was introduced by #11675 which caused PipeWriter.FlushAsync() to be called twice per response in the TechEmpower plaintext benchmark instead of once like before.
I fixed this by checking whether _unflushedBytes > 0 in Http1OutputProducer.WriteStreamSuffixAsync() for non-chunked responses. If there are no remaining unflushed bytes, we now skip the redundant flush.

Here are some before and after numbers from the Pipelined Plaintext TechEmpower benchmark:
Before PR
RequestsPerSecond:           4,701,874
Max CPU (%):                 94
WorkingSet (MB):             54
Avg. Latency (ms):           1.39
Startup (ms):                349
First Request (ms):          97.18
Latency (ms):                0.11
Total Requests:              70,998,775
Duration: (ms)               15,100
Socket Errors:               0
Bad Responses:               0
SDK:                         3.0.100-preview7-012769
Runtime:                     3.0.0-preview7-27825-08
ASP.NET Core:                3.0.0-preview7.19328.3

After PR
RequestsPerSecond:           5,018,954
Max CPU (%):                 90
WorkingSet (MB):             53
Avg. Latency (ms):           1.46
Startup (ms):                366
First Request (ms):          96.96
Latency (ms):                0.12
Total Requests:              75,784,120
Duration: (ms)               15,100
Socket Errors:               0
Bad Responses:               0
SDK:                         3.0.100-preview7-012769
Runtime:                     3.0.0-preview7-27825-08
ASP.NET Core:                3.0.0-preview7.19328.3

So it looks like we got about ~6.5% of our RPS back in this benchmark.

@jkotalik can you please sign off on this, if you feel comfortable with the fix? Thanks!

Is WebApiTemplateAsync a known flaky test? I've seen it fail once here https://github.com/aspnet/AspNetCore-Internal/issues/2220 but Template failures scare me as they showed problems in the past.

The template tests have caught some HTTP/2-related race conditions. Fortunately, this change is HTTP/1-only where we have much better test coverage. It also doesn't impact any synchronization, so I'm not worried about this introducing a race.

Merging as this is approved for Preview 7!

@dougbu this was approved for Preview 7 release. Please make sure this is included in the build we produce.

We don't have a coherent build yet but @wtgodbe is aware of this and started a couple of internal builds containing the fix to shorten the time-to-BAR


Object Variables

Hi, I'm using MathExecutor in combination with a money library to calculate prices (without floating point errors).
I've extended MathExecutor and overwritten all the operators to also work with with Money objects and it works great, however setting variables is not possible as it only allows for scalar types. Unfortunately I can't override the setVar function as $variables is private.
A simple fix would be to put the validation in a separate function so it's possible to override it, or make the $variables array protected. Not sure if this might cause problems in other areas though.
I'd be happy to create a PR for the above or for a different solution if you're willing to accept it.

$variables should probably not be private.  But open for a PR. I think we added the scalar test to try to make it more type safe. But what we really want is an object we can perform math on.  I don't know of a way to specify that in PHP, as PHP does not yet support operator overloading, which is really what we need.
Making properties protected might just be enough.
But I like an overridable validation function.  Then you can easily add valid types and still get some type protection.
Maybe do both?

Another possibility would be a validation callback.  It would default to what it does now, but you could pass a callback. That way you would not have to override a function.

Issue resolved with next release, closing.


Fix the frontend formatting

There are a few issues with the layout of the page:
Set the backlog to a fixed height
fix alignment issues with the backlog
fix alignment issues with the text boxes
fix alignment issues with the sanity check

Fix alignment issues with the sidebar

Number of background colors/layers is too high

Change Sanity Check so that font size is consistent across the page

Most of these seem fixed, I will separate the remaining ones into their own issues


downlaod mouse genome for esiting bcbio installation

Hi Brad,
We have a bcbio installation (3.4) for the past two years and we downloaded only GRCh37 at that time. We cannot upgrade as we have locked this version for our clinical studies. Now that we have requests for mouse data and we don't have mm10 genome in our initial installation, I tried getting it but looks like its going to upgrade entire bcbio package.
As an alternative I tried to install it in a different location and it ended up with errors. How can get mm10 on existing installation? If not can you help in fixing the below errors.
mv: cannot stat `master': No such file or directory
Setting up virtual machine
INFO: <cloudbio.flavor.Flavor instance at 0x2b543bb14a28>
INFO: This is a ngs_pipeline_minimal flavor
INFO: Reading default fabricrc.txt
DBG [config.py]: Using config file /mnt/speed/qc/sequencing/bcbio/tmpbcbio-install/cloudbiolinux/cloudbio/../config/fabricrc.txt
INFO: Distribution auto
INFO: Get local environment
INFO: CentOS setup
DBG [distribution.py]: NixPkgs: Ignored
[localhost] local: echo $HOME
[localhost] local: uname -m
INFO: Now, testing connection to host...
INFO: Connection to host appears to work!
DBG [utils.py]: Expand paths
INFO: List of genomes to get (from the config file at '{'install_liftover': False, 'genome_indexes': ['rtg'], 'genomes': [{'name': 'Mouse (mm10)', 'dbkey': 'mm10', 'annotations': ['problem_regions', 'dbsnp', 'transcripts', 'mirbase']}], 'install_uniref': False}'): Mouse (mm10)
[localhost] local: mkdir -p /coldstorage/vakad/genomes
[localhost] local: mkdir -p /coldstorage/vakad/genomes/Mmusculus/mm10
INFO: Genome preparation method ggd failed, trying next
INFO: Downloading genome from s3: mm10 seq
[localhost] local: pwd
[localhost] local: echo $HOME
[localhost] local: mkdir -p '/home/sequencing/tmp/cloudbiolinux/5f24256c-eecf-3715-aa28-2756cde9a517'
[localhost] local: wget --continue --no-check-certificate -O mm10-seq.tar.xz 'https://s3.amazonaws.com/biodata/genomes/mm10-seq.tar.xz'
--2017-03-15 13:04:50--  https://s3.amazonaws.com/biodata/genomes/mm10-seq.tar.xz
Resolving s3.amazonaws.com... <IP_ADDRESS>
Connecting to s3.amazonaws.com|<IP_ADDRESS>|:443... connected.
HTTP request sent, awaiting response... 200 OK
Length: 660653188 (630M) [binary/octet-stream]
Saving to: “mm10-seq.tar.xz”
100%[===============================================================================================================================>] 660,653,188  197K/s   in 64m 25s
2017-03-15 14:09:15 (167 KB/s) - “mm10-seq.tar.xz” saved [660653188/660653188]
[localhost] local: mv mm10-seq.tar.xz /coldstorage/vakad/genomes/Mmusculus/mm10
[localhost] local: rm -rf /home/sequencing/tmp/cloudbiolinux/5f24256c-eecf-3715-aa28-2756cde9a517
[localhost] local: xz -dc mm10-seq.tar.xz | tar -xvpf -
seq/
seq/mm10.fa
seq/mm10.fa.fai
seq/mm10.dict
seq/mm10-resources.yaml
tar: seq: Cannot change mode to rwxrwsr-x: Operation not permitted
tar: Exiting with failure status due to previous errors
Fatal error: local() encountered an error (return code 2) while executing 'xz -dc mm10-seq.tar.xz | tar -xvpf -'
Thanks,

Diya;
My suggestion would be to install a separate up to date bcbio to use for mm10 runs and any other non-clinical updates you need. That way you avoid any issue with your production version. bcbio can do a data-only install but will also try to update data for your GRCh37 install which I assume you don't want to happen.
The error you saw looks like a filesystem permissions error, so is likely separate from this -- maybe you have the genome directories in your production install read-only? Hope the separate install for new genomes works for you.


Translate item descriptions

As the name states. Not sure if the text will fit, so either the font size needs to be further changed or I need suggestions for shortening. The translations as they are right now are accurate to the original.
This also includes a fix for item names, as I previously used "Fall" instead of "Autumn" there.

these 3 were viewed from inventory or directly by tap holding, lengths limitation doing some work

this one was viewed from shop, you got more space here apparently

Are those three the only ones that don't fit?

Are those three the only ones that don't fit?
I assume the winner's garland problem would apply to all garland items.

if they viewed from awakening page, some of the winner's garland got cut off
but as long as the race name is there, its okay i think

I don't believe we can keep the flavor text of the translations for a lot of the items properly if we want the descriptions to fit. Specifically:

Shoes
Jewels
Alarm Clock
Horseshoes
Clovers
Money
Friendship Points
Goddess Statues
Team Stadium items
Follower Limit Up

I don't believe we can keep the flavor text of the translations for a lot of the items properly if we want the descriptions to fit. Specifically:

Shoes
Jewels
Alarm Clock
Horseshoes
Clovers
Money
Friendship Points
Goddess Statues
Team Stadium items
Follower Limit Up

we could just put their functionality as desc while throwing away the flavor texts
its just as you said it wont fit at this rate, i give up trying to shorten them lol

It's a shame since I like the flavor text, but seems like we will have to go ahead with that, yea.

Please try these changes out.


Re-enable FSDP+TP w/ strided sharding

Stack from ghstack (oldest at bottom):

-> #507

Summary

re-enable FSDP+TP 2D in torchtitan.
remove temporary re-enablement added in #460

Test
Command: python test_runner.py outputs --test pp_dp_tp --ngpu 8
GPUs: A100
Output:

without strided sharding:

[rank7]:2024-08-06 03:21:26,706 - root - INFO - step:  2  loss:  8.1652  memory:  0.51GiB(0.64%)  wps: 8,250  mfu: 0.25%
[rank7]:2024-08-06 03:21:27,013 - root - INFO - step:  3  loss:  8.0951  memory:  0.51GiB(0.64%)  wps: 13,358  mfu: 0.41%
[rank7]:2024-08-06 03:21:27,309 - root - INFO - step:  4  loss:  7.9748  memory:  0.51GiB(0.64%)  wps: 13,865  mfu: 0.42%
[rank7]:2024-08-06 03:21:27,582 - root - INFO - step:  5  loss:  7.8025  memory:  0.51GiB(0.64%)  wps: 15,057  mfu: 0.46%
[rank7]:2024-08-06 03:21:28,076 - root - INFO - step:  6  loss:  7.5612  memory:  0.51GiB(0.64%)  wps: 8,300  mfu: 0.25%
[rank7]:2024-08-06 03:21:28,608 - root - INFO - step:  7  loss:  7.3649  memory:  0.51GiB(0.64%)  wps: 7,705  mfu: 0.23%
[rank7]:2024-08-06 03:21:28,927 - root - INFO - step:  8  loss:  7.2946  memory:  0.51GiB(0.64%)  wps: 12,832  mfu: 0.39%
[rank7]:2024-08-06 03:21:29,251 - root - INFO - step:  9  loss:  7.1311  memory:  0.51GiB(0.64%)  wps: 12,669  mfu: 0.38%
[rank7]:2024-08-06 03:21:29,627 - root - INFO - step: 10  loss:  7.0540  memory:  0.51GiB(0.64%)  wps: 10,918  mfu: 0.33%
>>>>>>>>>>>>>>>>>Checkpoint save & load<<<<<<<<<<<<<<<<<<<
[rank7]:2024-08-06 03:21:59,723 - root - INFO - step: 11  loss:  7.0822  memory:  0.51GiB(0.64%)  wps: 1,139  mfu: 0.03%
[rank7]:2024-08-06 03:22:00,054 - root - INFO - step: 12  loss:  7.0508  memory:  0.51GiB(0.64%)  wps: 12,366  mfu: 0.38%
[rank7]:2024-08-06 03:22:00,340 - root - INFO - step: 13  loss:  6.9182  memory:  0.51GiB(0.64%)  wps: 14,370  mfu: 0.44%
[rank7]:2024-08-06 03:22:00,624 - root - INFO - step: 14  loss:  6.8948  memory:  0.51GiB(0.64%)  wps: 14,442  mfu: 0.44%
[rank7]:2024-08-06 03:22:00,907 - root - INFO - step: 15  loss:  6.8358  memory:  0.51GiB(0.64%)  wps: 14,514  mfu: 0.44%
[rank7]:2024-08-06 03:22:01,574 - root - INFO - step: 16  loss:  6.7653  memory:  0.51GiB(0.64%)  wps: 6,144  mfu: 0.19%
[rank7]:2024-08-06 03:22:02,209 - root - INFO - step: 17  loss:  6.7340  memory:  0.51GiB(0.64%)  wps: 6,453  mfu: 0.20%
[rank7]:2024-08-06 03:22:02,532 - root - INFO - step: 18  loss:  6.6874  memory:  0.51GiB(0.64%)  wps: 12,695  mfu: 0.39%
[rank7]:2024-08-06 03:22:02,863 - root - INFO - step: 19  loss:  6.6566  memory:  0.51GiB(0.64%)  wps: 12,406  mfu: 0.38%
[rank7]:2024-08-06 03:22:03,257 - root - INFO - step: 20  loss:  6.6629  memory:  0.51GiB(0.64%)  wps: 10,392  mfu: 0.32%

with strided sharding

[rank7]:2024-08-06 03:26:18,288 - root - INFO - step:  1  loss:  8.2069  memory:  0.50GiB(0.63%)  wps: 915  mfu: 0.03%
[rank7]:2024-08-06 03:26:19,084 - root - INFO - step:  2  loss:  8.1913  memory:  0.51GiB(0.64%)  wps: 5,144  mfu: 0.16%
[rank7]:2024-08-06 03:26:19,365 - root - INFO - step:  3  loss:  8.1148  memory:  0.51GiB(0.64%)  wps: 14,593  mfu: 0.44%
[rank7]:2024-08-06 03:26:19,698 - root - INFO - step:  4  loss:  7.9982  memory:  0.51GiB(0.64%)  wps: 12,328  mfu: 0.37%
[rank7]:2024-08-06 03:26:20,011 - root - INFO - step:  5  loss:  7.8382  memory:  0.51GiB(0.64%)  wps: 13,100  mfu: 0.40%
[rank7]:2024-08-06 03:26:20,498 - root - INFO - step:  6  loss:  7.6293  memory:  0.51GiB(0.64%)  wps: 8,423  mfu: 0.26%
[rank7]:2024-08-06 03:26:21,126 - root - INFO - step:  7  loss:  7.4454  memory:  0.51GiB(0.64%)  wps: 6,530  mfu: 0.20%
[rank7]:2024-08-06 03:26:21,472 - root - INFO - step:  8  loss:  7.3337  memory:  0.51GiB(0.64%)  wps: 11,843  mfu: 0.36%
[rank7]:2024-08-06 03:26:21,849 - root - INFO - step:  9  loss:  7.1960  memory:  0.51GiB(0.64%)  wps: 10,892  mfu: 0.33%
[rank7]:2024-08-06 03:26:22,229 - root - INFO - step: 10  loss:  7.1208  memory:  0.51GiB(0.64%)  wps: 10,798  mfu: 0.33%
>>>>>>>>>>>>>>>>>Checkpoint save & load<<<<<<<<<<<<<<<<<<<
[rank7]:2024-08-06 03:26:50,306 - root - INFO - step: 11  loss:  7.1222  memory:  0.51GiB(0.64%)  wps: 866  mfu: 0.03%
[rank7]:2024-08-06 03:26:50,632 - root - INFO - step: 12  loss:  7.1189  memory:  0.51GiB(0.64%)  wps: 12,589  mfu: 0.38%
[rank7]:2024-08-06 03:26:50,917 - root - INFO - step: 13  loss:  6.9646  memory:  0.51GiB(0.64%)  wps: 14,417  mfu: 0.44%
[rank7]:2024-08-06 03:26:51,217 - root - INFO - step: 14  loss:  6.9626  memory:  0.51GiB(0.64%)  wps: 13,680  mfu: 0.42%
[rank7]:2024-08-06 03:26:51,514 - root - INFO - step: 15  loss:  6.8694  memory:  0.51GiB(0.64%)  wps: 13,799  mfu: 0.42%
[rank7]:2024-08-06 03:26:52,207 - root - INFO - step: 16  loss:  6.7994  memory:  0.51GiB(0.64%)  wps: 5,910  mfu: 0.18%
[rank7]:2024-08-06 03:26:53,053 - root - INFO - step: 17  loss:  6.7634  memory:  0.51GiB(0.64%)  wps: 4,847  mfu: 0.15%
[rank7]:2024-08-06 03:26:53,370 - root - INFO - step: 18  loss:  6.7233  memory:  0.51GiB(0.64%)  wps: 12,915  mfu: 0.39%
[rank7]:2024-08-06 03:26:53,686 - root - INFO - step: 19  loss:  6.7054  memory:  0.51GiB(0.64%)  wps: 12,995  mfu: 0.39%
[rank7]:2024-08-06 03:26:54,059 - root - INFO - step: 20  loss:  6.7130  memory:  0.51GiB(0.64%)  wps: 10,991  mfu: 0.33%

When to merge:
when https://github.com/pytorch/pytorch/pull/130760 is in nightly build.

@XilunWu FSDP2 registers load state dict post hooks because the loaded DTensor may lose the padding an FSDP2 needs to reacquire the reference to the tensor in case of assign=True without swap_tensors:
https://github.com/pytorch/pytorch/blob/e7b870c88bc3b854a95399a96a274d2f1f908172/torch/distributed/_composable/fsdp/_fsdp_param.py#L242-L248

@awgu This detection is done before the first iteration. iirc, FSDP2 lazily initializes the load state dict hook?

I think it is done at construction time :/ (see the code pointer in https://github.com/pytorch/torchtitan/pull/507#issuecomment-2282832902).

That is post hook? The hooks we remove here are pre hook for state_dict and load_state_dict. The hooks I mentioned are https://github.com/pytorch/pytorch/blob/e7b870c88bc3b854a95399a96a274d2f1f908172/torch/distributed/_composable/fsdp/_fsdp_param_group.py#L505 but I think they are installed during the first iteration?

@fegin ah yes, those hooks are initialized on first iteration (lazy init).


Fixes for bugs in previous integration to master

Merging without CR since all the PRs included were CRed in Dev branch.


provider plugin fails with error "Incompatible API version with plugin. Plugin version: 2, Ours: 1"

when terraform-provider-opc is built against the latest terraform source the provider fails with the following error:
$ terraform plan
Error configuring: 1 error(s) occurred:

* Incompatible API version with plugin. Plugin version: 2, Ours: 1

The problem appears to be related to terraform v0.7.13.  Workaround is to compile the terraform-provider-opc against the v0.7.12 tagged source
cd $GOPATH/src/github.com/hashicorp/terraform
git checkout v0.7.12

https://github.com/hashicorp/terraform/issues/10351


GLIBC_2.34 not found

When compiling apache/arrow with the latest closure compiler on node 16 and Debian, we get this error
[03:34:57] Error: gulp-google-closure-compiler: /arrow/js/node_modules/google-closure-compiler-linux/compiler: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.32' not found (required by /arrow/js/node_modules/google-closure-compiler-linux/compiler)
[321](https://github.com/apache/arrow/actions/runs/4040323460/jobs/6945859429#step:5:322)
/arrow/js/node_modules/google-closure-compiler-linux/compiler: /lib/x86_64-linux-gnu/libc.so.6: version `GLIBC_2.34' not found (required by /arrow/js/node_modules/google-closure-compiler-linux/compiler)
[322](https://github.com/apache/arrow/actions/runs/4040323460/jobs/6945859429#step:5:323)
Error writing to stdin of the compiler. write EPIPE
[323](https://github.com/apache/arrow/actions/runs/4040323460/jobs/6945859429#step:5:324)

Ref https://github.com/apache/arrow/actions/runs/4040323460/jobs/6945859429#step:5:398

Debian 11 is "Bullseye" which ships with libc-bin 2.31-13+deb11u5 (bullseye-updates has the same version).
Debian 12 is "bookwork", which has libc-bin 2.36-8. Bookwork is still "testing", while bullseye is "stable", but in debian terms "stable" is often "very out of date, but at least we promise not to break anything".
Sid/unstable is running the same version of libc-bin as bookworm.
I could reasonably expect that closure-compiler would want to downgrade the required glibc version to be more generally usable, or there might be reasons to push to newer versions (in the same way that Java 8 support was recently dropped).


Retry for transient exceptions while doing cleanup for Hadoop Jobs

From : https://github.com/druid-io/druid/pull/1815#discussion_r42021133
If there is any transient exception while doing cleanup in Hadoop Jobs, right now we simply propagate the exception. It would be better add Retry logic for transient exceptions in this case.

@nishantmonu51 is this fixed with #1815?

@nishantmonu51 is this still an issue?


[Bug]: Card component does not respect border-none

Tremor Version
3.18.2
Link to minimal reproduction
https://github.com/lidermanrony/tremor-issue-repro
Steps to reproduce

npm i
npm run dev

or use your favorite package manger (pnpm / yarn)
see the card on screen - it has a shadow even though shadow is none.
What is expected?
no shadow to be shown since shadow-none was specified as a className.
What is actually happening?

What browsers are you seeing the problem on?
Chrome, Safari
Any additional comments?
No response

Hi @lidermanrony ! I suppose this connects to how tailwind-merge resolves the shadow styles. I cloned your reproduction and was able to get rid of the shadow by addind ! to the tailwind utility → !shadow-none. Let me know if that helps :)

Hi @lidermanrony again, I released a beta npm i<EMAIL_ADDRESS>where I updated tailwind-merge. Worked in my local tests. Can you give it a try and report back?

Hi @lidermanrony again, I released a beta npm i<EMAIL_ADDRESS>where I updated tailwind-merge. Worked in my local tests. Can you give it a try and report back?

Sure, Will test and report back.

Great, will release this soon!


fix app show human output format

Experience
The interactive experience
$ ecs-preview app show
? Which application would you like to show?
> my-app

General

    AppName   "my-app"
    Type      "lb-application"
    Project   "my-project"

Deploys

    EnvName     "test"
    AccountID   "123456789"
    Region      "us-west-2"
    Prod        false
    Port        "8080"
    Tasks       3
    CPU         "256"
    Memory      "512"
    Path        "*"
    URL         "flask-Ec2Se-KCH5OLWN57WM-992971783.us-west-2.elb.amazonaws.com"

We can always add more colors to make it fancier.

Brainstorming random stuff:

Removed data that's about environments (account, region) to make this application specific.
Displaying information with human friendly units (vCPU, MiB)
Instead of displaying displaying every stat under an environment, categorizing them?

$ ecs-preview app show
? Which application would you like to show?
> my-app
About
    Name      "my-app"
    Type      "lb-application"
    Project   "my-project"

Configurations
    Environment      CPU (vCPU)      Memory (MiB)     Port
    test             0.25            512              8080
    prod             0.5             1024             8080

Routes
    Environment      URL                                                           Path
    test             flask-Ec2Se-KCH5OLWN57WM.us-west-2.elb.amazonaws.com          *
    prod             flask-Ec2Se-KCH5OLWN57WM.us-west-2.elb.amazonaws.com          *

let me know what you think

@efekarakus - I like your version. In the About section, I'd probably put project first, followed by name, then type (and remove the quotes).
One other interesting we could do is provide an --infrastructure flag (optional). When this flag is provided, we add another category called Infrastructure:
$ ecs-preview app show --infrastructure
? Which application would you like to show?
> my-app
About
    Project   my-project
    Name      my-app
    Type      lb-application

Configurations
    Environment      CPU (vCPU)      Memory (MiB)     Port
    test             0.25            512              8080
    prod             0.5             1024             8080

Routes
    Environment      URL                                                           Path
    test             flask-Ec2Se-KCH5OLWN57WM.us-west-2.elb.amazonaws.com          *
    prod             flask-Ec2Se-KCH5OLWN57WM.us-west-2.elb.amazonaws.com          *

 Infrastructure
    test
        Type         ID
        Service      kudos-test-Cluster-10L7BP5EXEYB/ecsarnplz
        Target Group kudos-Targe-L2ZO0DGY38BP/95c46b6b469cd61e

These values will come from querying Resource Groups (this can be a follow up action, of course).


GH-33699: [C++] Increase timeout of c++ tests when running under valgrind

This partially addresses #33699.  Once these tests are passing we can analyze the runtime and try and scope our valgrind coverage to something appropriate.

Closes: #33699

@github-actions crossbow submit test-conda-cpp-valgrind

Looks like they are still timing out https://dev.azure.com/ursacomputing/crossbow/_build/results?buildId=43127&view=logs&j=0da5d1d9-276d-5173-c4c4-9d4d4ed14fdb&t=d9b15392-e4ce-5e4c-0c8c-b69645229181&l=6947

I think splitting arrow-compute-scalar-test is inevitable at this point.  There are 1600 tests.  Right now, under valgrind, each takes an average of 700ms, which is not extremely long.  I tried to identify some of the worst culprits and while there are some things I can do I don't see any real solution.
The rest of the failures I can improve the timing on.

Looks like I'll have to tackle both issues after all :)
I broke the scalar test into multiple tests.  Some kernel tests were still pretty long so as most of those tests run on a large combination of types so I reduced the combinations when testing with valgrind.
The scanner test was using too much data.  I turns out that sorting an array with 2Mi rows is pretty slow when running in valgrind.
The CSV tests were slow because they were testing many combinations of compression and writing CSV files (even 50B CSV files) is very slow under valgrind (the conversion to string in particular).  I figure the various compressions are independently tested already and so removed those when running under valgrind.
The hash tests were simply running too many iterations.

@github-actions crossbow submit test-conda-cpp-valgrind

Awesome looks like the timeout is fixed :D

Yes, though the appveyor test is concerning (and confusing :)

Yes, though the appveyor test is concerning (and confusing :)

From the error message I guess it's a MSVC bug where it somehow treats the anonymous namespace in scalar_arithmetis_test.cc and the anonymous namespace in test_util.cc (in which the four typed ValidateOutput are defined) as the same namespace. The unqualified name lookup stops once it finds the name ValidateOutput in the same namespace.

The fact that MSVC looks up names in source files seems even wilder to me....

Ah, that's probably it.  I will try fully qualifying.  I suspect that it isn't looking in different source files because it is a unity build so I think, in this case, scalar_arithmetic_test and test_util have been smashed together into the same file.

The fact that MSVC looks up names in source files seems even wilder to me....

Ah, that's probably it. I will try fully qualifying. I suspect that it isn't looking in different source files because it is a unity build so I think, in this case, scalar_arithmetic_test and test_util have been smashed together into the same file.

Oh I see. Makes sense.

@js8544 's suggestion worked.  Looks like the remaining test failures are spurious.  Any last thoughts @assignUser ?

Thanks a lot @westonpace !


False positive constant on native_constant_invocation rule

Bug report
Versions

PHP-CS-Fixer: from 3.2.0 to 3.5.0
PHP runtime : 8.0.9

The native_constant_invocation rules seems to interprete the 2nd object as a constant when it's in a catch statement with another object but without the exception variable (from PHP 8).
PHP runtime: 8.0.9
Loaded config default.
Using cache file ".php-cs-fixer.cache".
F                                                                   1 / 1 (100%)
Legend: ?-unknown, I-invalid file syntax (file ignored), S-skipped (cached or empty file), .-no changes, F-fixed, E-error
   1) test.php (native_constant_invocation)
      ---------- begin diff ----------
--- "test-phpcs/test.php"
+++ "test-phpcs/test.php"
@@ -6,13 +6,13 @@

 try {
     // Code
-} catch (MyException|\TypeError) {
+} catch (MyException|TypeError) {
     // Code
 } catch (MyException|\TypeError $e) {
     // Code
-} catch (\TypeError|\ErrorException) {
+} catch (\TypeError|ErrorException) {
     // Code
-} catch (\ErrorException|\TypeError) {
+} catch (\ErrorException|TypeError) {
     // Code
 } catch (\TypeError) {
     // Code

      ----------- end diff -----------

Checked all files in 0.017 seconds, 12.000 MB memory used

thanks for reporting and all the details provided :+1:
I think this is the same as https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/5684#issuecomment-1029766951 so the fix will solve this, let me know if not

@SpacePossum Sorry, I didn't see it!


Added output data changed signal

Introduced a new signal to indicate that the data on the server has been updated and therefore the components should refetch the data and rerender the components. This PR handles the signal in table-output-component and timegraph-output-component. We have a use case where data is dynamically modified on the server, and we want the components to rerender with the new data. Currently, the user would have to close the active trace panel and reopen it to see the changes.

Introduced a new signal to indicate that the data on the server has been updated and therefore the components should refetch the data and rerender the components. This PR handles the signal in table-output-component and timegraph-output-component. We have a use case where data is dynamically modified on the server, and we want the components to rerender with the new data. Currently, the user would have to close the active trace panel and reopen it to see the changes.

@ngondalia Interesting update. It sounds like that it is for some streaming / monitoring use case. If your server is modifying data, do you have some notification channel implemented from your custom server to the front-end? Please let me know.

Introduced a new signal to indicate that the data on the server has been updated and therefore the components should refetch the data and rerender the components. This PR handles the signal in table-output-component and timegraph-output-component. We have a use case where data is dynamically modified on the server, and we want the components to rerender with the new data. Currently, the user would have to close the active trace panel and reopen it to see the changes.

@ngondalia Interesting update. It sounds like that it is for some streaming / monitoring use case. If your server is modifying data, do you have some notification channel implemented from your custom server to the front-end? Please let me know.

Sorry, the word "dynamically" was misleading and wrong here. The data gets modified based on some trace type specific settings/filters coming from our custom webviews within our extension. The front end initiates the data modification, therefore we dont have the need for the server to notify the front-end of anything. It already knows that data has changed, as well as the type of data that has changed.


can't click "copy" on the commit hash in the about tab

Describe your issue

I can't click "Copy" for the commit hash in the about tab. The copy option also looks weird, kinda blurry.
Screenshots

OS and Chatterino Version

Windows 10, Chatterino 3b0d62f5c

additionally, the date of build missing after the comma.

This is because you can select the text. If you select the text you'll see the option is available

If you only want to copy the link, you can copy the link location

Generally, if you're asked to share what c2 version you're running, you'd be asked to copy all of the text


Add webhooks

When a user revokes the app from the interface, the app has no knowledge of that. We should offer webhooks (server to server calls) and testing capabilities.
Possible user actions: revoke tokens, revoke Auth, add auth

Closing, this is out of scope for now


added Consul enableTagOverride

In this pull request I address #347 . The changes seem to be quite basic.

@amanshin Can you add some documentation for this?

I certainly can. In what form will it work best?

Any update on this?

Can we please get this merged into mainline? It's a fairly simple patch and it's really useful.

@amanshin we have a docs folder, add it to the appropriate backend.


Fix for user private repos

Don't quite know why, additional permissions were needed: contents: read.
Example: https://severinbratus.github.io/testable-template-private/

If you don't know why I certainly don't know either... But if this fixes it, perfect!


Installation Fails

I followed the 3 installation steps on the project's homepage.
I get the following message error:
2015/05/12 16:18:53 # github.com/tylerb/graceful
../../tylerb/graceful/graceful.go:41: undefined: http.ConnState
../../tylerb/graceful/graceful.go:169: srv.Server.ConnState undefined (type *http.Server has no field or method ConnState)
../../tylerb/graceful/graceful.go:169: undefined: http.ConnState
../../tylerb/graceful/graceful.go:171: undefined: http.StateNew
../../tylerb/graceful/graceful.go:173: undefined: http.StateClosed
../../tylerb/graceful/graceful.go:173: undefined: http.StateHijacked
../../tylerb/graceful/graceful.go:271: srv.SetKeepAlivesEnabled undefined (type *Server has no field or method SetKeepAlivesEnabled)

Google doesn't help much here...
Thanks

Can I see the full output after you run the command?

So I was using go1.2. I installed 1.4 and started over. Now the server works.
But Migration don't seem to execute correctly. (Tables already exist, but that's normal since I already executed the commands for that).
Here are the whole outputs:
▶ $GOPATH/bin/go-bootstrap -dir github.com/aug-riedinger/go-bootstrap
2015/05/13 13:18:34 Creating /home/augustin/.gvm/pkgsets/go1.4/global/src/github.com/aug-riedinger/go-bootstrap...
2015/05/13 13:18:34 Copying a blank project to /home/augustin/.gvm/pkgsets/go1.4/global/src/github.com/aug-riedinger/go-bootstrap...
2015/05/13 13:18:34 Replacing placeholder variables on aug-riedinger/go-bootstrap...
2015/05/13 13:18:34 Creating database go-bootstrap...
createdb: database creation failed: ERROR:  database "go-bootstrap" already exists
Running migration on postgres://augustin@localhost:5432/go-bootstrap?sslmode=disable...
scripts/db-bootstrap: ligne 7: migrate : commande introuvable
Creating database go-bootstrap-test
createdb: database creation failed: ERROR:  database "go-bootstrap-test" already exists
Running migration on postgres://augustin@localhost:5432/go-bootstrap-test?sslmode=disable...
scripts/db-bootstrap: ligne 13: migrate : commande introuvable
2015/05/13 13:18:34 Running go get ./...
2015/05/13 13:18:54 Installing github.com/tools/godep...
2015/05/13 13:19:05 Running git init
2015/05/13 13:19:05 Running godep save ./...
2015/05/13 13:19:06 Running godep go test ./...
2015/05/13 13:19:18 ?   	github.com/aug-riedinger/go-bootstrap	[no test files]
--- FAIL: TestCreateDeleteGeneric (0.03s)
	base_test.go:91: Deleting user by id should not fail. Error: pq: Could not complete operation in a failed transaction
--- FAIL: TestCreateDeleteById (0.02s)
	base_test.go:118: Deleting user by id should not fail. Error: pq: Could not complete operation in a failed transaction
--- FAIL: TestCreateUpdateGenericDelete (0.02s)
	base_test.go:148: Updating existing user should not fail. Error: pq: relation "users" does not exist
	base_test.go:154: Deleting user by id should not fail. Error: pq: Could not complete operation in a failed transaction
--- FAIL: TestCreateUpdateByIdDelete (0.01s)
	base_test.go:183: Updating existing user should not fail. Error: pq: relation "users" does not exist
	base_test.go:189: Deleting user by id should not fail. Error: pq: Could not complete operation in a failed transaction
--- FAIL: TestCreateUpdateByKeyValueStringDelete (0.02s)
	base_test.go:220: Updating existing user should not fail. Error: pq: relation "users" does not exist
	base_test.go:226: Deleting user by id should not fail. Error: pq: Could not complete operation in a failed transaction
--- FAIL: TestUserCRUD (0.01s)
	user_test.go:18: Signing up user should work. Error: pq: relation "users" does not exist
	user_test.go:21: Signing up user should work.
FAIL
FAIL	github.com/aug-riedinger/go-bootstrap/dal	3.932s
?   	github.com/aug-riedinger/go-bootstrap/handlers	[no test files]
ok  	github.com/aug-riedinger/go-bootstrap/libenv	0.007s
ok  	github.com/aug-riedinger/go-bootstrap/libhttp	3.173s
ok  	github.com/aug-riedinger/go-bootstrap/libstring	0.014s
ok  	github.com/aug-riedinger/go-bootstrap/libunix	0.004s
?   	github.com/aug-riedinger/go-bootstrap/middlewares	[no test files]
godep: go exit status 1

Thanks for the help.

In this line: scripts/db-bootstrap: ligne 7: migrate : commande introuvable
It seems to me that migrate command is still compiled using Go 1.2. Can you try removing $GOPATH/bin/migrate and re-run the command again?
It's probably best to remove all $GOPATH/bin/* after upgrading Go.

Oh, I see what's going on, go get github.com/mattes/migrate was accidentally removed. This is a bug!

Can you update go-bootstrap to the latest one and try again?


chore(VitaliFamily): rename α → X

The module docstring uses X while the code used α.

Thank you for doing this. Prior to this PR, x was often used as an element of \alpha - this notation is much better! I have one follow-up question: this file now uses x : X, and a if there's another variable in X. Would you like to rename that one as well? (Apart from that, it's good to go!)

a was used for variables of type Set X. This doesn't follow our naming convention, so renamed to s or t.

Thanks, looks good to me now.
maintainer merge

Thanks!
bors merge


Fix building on Vercel

Fix the issue describe in https://app.dework.xyz/bounties?taskId=d81f43b0-abb2-4a20-b015-a94901e9b86a
Here is the new build log:

Thank you Thanh, the bounty reward given to another solution. But your solution tagged as "Honorable Mention".
See details:
https://app.dework.xyz/citydao/engineering-2?taskId=d81f43b0-abb2-4a20-b015-a94901e9b86a