Ability to pass referer information

Is your feature request related to a problem? Please describe.
The auro-hyperlink does not allow the referer attribute to be sent.
Describe the solution you'd like
What would be nice is to have the ability to send the referer domain and path when a user clicks on certain links.  This would be an opt in feature but could help with tracking.  This would enable teams to be able to tailor their experience based off of where the user came from.
Describe alternatives you've considered
The alternative is to not use auro-hyperlink, when this is needed so they have the option to send this type of information.
Additional context
This story is on the content teams backlog.  This page has additional details...
https://dev.azure.com/itsals/E_Retain_Content/_backlogs/backlog/E_Retain_Content Team/Stories/?workitem=645265
This is a request to be able to send the page referer information to a 3rd party website that they could use for analytics.
https://developer.mozilla.org/en-US/docs/Web/HTTP/Headers/Referer
Security would be a concern as well
https://developer.mozilla.org/en-US/docs/Web/Security/Referer_header:_privacy_and_security_concerns

What we are talking about here is supporting a scenario where when _blank is used in conjunction with the referrerpolicy attribute, there is a scenario where users can manage their referral policies.
We can use the referrerpolicy attribute as a trigger to allow for opt in support and set the rel to external and remove the noreferrer default scenario.
Note
While researching, the following was discovered. It is recommended that we not change support for adding this value to the rel attribute at this time as this support is realitivelly new.

As of Chromium version 88, anchors with target="_blank" automatically get noopener behavior by default. Explicit specification of rel="noopener" helps protect users of legacy browsers including Edge Legacy and Internet Explorer.


ao configmap: fixes for handling boolean and numeric values

Summary
This is a small fix to allow the helm-generated configmap to accommodate boolean and numeric values -- it forces extra quotes into the configmap.
How to produce the error
The following snippet in values.yaml will cause helm deployment failure. It does not matter whether the word false is quoted, doublequoted, escaped in any way -- as long as it's recognizable as a boolean the failure will happen.
Similar problems happen with any value that is JSON/YAML parseable as an integer.
global:
  configmap:
    configParams:
      KEYLIME_TENANT_REQUIRE_EK_CERT: false

Gist of the fix
Produce extra quotes in the configmap template[s] to prevent parsing of affected values as anything but string.
Affected locations
Two places I noticed that are using .Values.global.configmap.configParams:

charts/keylime-init/templates/ca-job.yaml -- global environment variables used to direct CA generation
templates/configmap.yaml -- the actual copying of environment variables into the keylime configmap

Why we need this
I cannot start to produce meaningful CI tests without disabling EK cert requirements on AWS machines (since AWS TPMs don't have EK certificates). Expect a second, much larger PR for meaningful CI tests with AWS EC2 VMs as soon as this is merged.

Obvious but sorely needed fix. LGTM


Swagger API docs for getting container logs missing an input parameter

Using the REST api to get logs for a container running in a multi-container pod requires using "container" URI parameter.  That parameter doesn't not show up at all in the Swagger api docs.

@nikhiljindal

Thanks for the report, @perljedi

Being fixed by https://github.com/GoogleCloudPlatform/kubernetes/pull/10426

Cool, thanks!

Fixed by #10426


The default render function should show more info on dataset

So least the "shape" so the dimensions,
and maybe even the column names.

That's a good suggestion, it will be attempted on the next release. Happy to accept a PR as well.


PDF contents turns redacted (blackout) upon zoom-out to thumbnail size.

Attach (recommended) or Link to PDF file here:
Sample Document :
Sample_blackout.pdf
Configuration:

Web browser and its version: Chrome, Firefox, MS Edge (latest versions)
Operating system and its version: Windows, Mac
PDF.js version: 2.12.313 (latest)
Is a browser extension: NA

Steps to reproduce the problem:

Open attached Sample_blackout.pdf in pdfjs viewer
Zoom out the document to lesser than 30% or verify the content in thumbnail view
You will notice portion of the document content are turned black.

What is the expected behavior? : Document content should not change its style in thumbs view or while zoom out.
What went wrong? : document content are turned black and even some portions are redacted.
Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension): https://mozilla.github.io/pdf.js/web/viewer.html
Output in pdfjs viewer on zoom out :

I wonder if this is effectively a duplicate of e.g. issue #13211.

Sample_blackout.pdf
I have attached the file that we were able to recreate the issue with


Fixes #1449 -- duplicate profile fields in admin

Same as the previous fix in #1450, but with min_num = 1 added to ProfileInline.

I've moved the empty-form CSS rule to grappelli-safe, where I think it belongs. The CSS rule is already in vanilla Django, and the reason we don't see it is because of grappelli's overrides, so that's where the fix should live IMO.
See stephenmcd/grappelli-safe#65

Testing this and it seems to prevent adding new users - when I try to, I'm shown the add user form again but without any error messages.
I've a profile model with a required field - my rough guess is that the form for adding users thinks that profile field should have a value, even though it shouldn't appear on the add user form.

Damn, I'll have another look. Adding worked ok on my setup but with no
required profile fields.
So it's the initial username/password form you can't get past?
On Friday, 29 January 2016, Stephen McDonald<EMAIL_ADDRESS>wrote:

Testing this and it seems to prevent adding new users - when I try to, I'm
shown the add user form again but without any error messages.
I've a profile model with a required field - my rough guess is that the
form for adding users thinks that profile field should have a value, even
though it shouldn't appear on the add user form.
—
Reply to this email directly or view it on GitHub
https://github.com/stephenmcd/mezzanine/pull/1517#issuecomment-176485663
.

Yep
On 29 Jan 2016 11:16 AM, "Alex Hill"<EMAIL_ADDRESS>wrote:

Damn, I'll have another look. Adding worked ok on my setup but with no
required profile fields.
So it's the initial username/password form you can't get past?
On Friday, 29 January 2016, Stephen McDonald<EMAIL_ADDRESS>wrote:

Testing this and it seems to prevent adding new users - when I try to,
I'm
shown the add user form again but without any error messages.
I've a profile model with a required field - my rough guess is that the
form for adding users thinks that profile field should have a value, even
though it shouldn't appear on the add user form.
—
Reply to this email directly or view it on GitHub
<
https://github.com/stephenmcd/mezzanine/pull/1517#issuecomment-176485663>
.

—
Reply to this email directly or view it on GitHub
https://github.com/stephenmcd/mezzanine/pull/1517#issuecomment-176493001
.

Ok, well that's definitely what's happening.
Fortunately InlineModelAdmin provides a get_min_num() method which we can override, returning 0 when creating and 1 when updating. That should sort it out, can you give it a go Steve?

Works! Thanks as always for diving deeper than me.


Message feature not working

m "Mike Smith" hello TypeError: Cannot read property 'name' of undefined     at Messer.getThreadByName (/usr/local/lib/node_modules/messer/index.js:186:33)     at Promise (/usr/local/lib/node_modules/messer/src/command-handlers.js:83:29)     at Promise (<anonymous>)     at Messer.message (/usr/local/lib/node_modules/messer/src/command-handlers.js:73:12)     at Messer.processCommand (/usr/local/lib/node_modules/messer/index.js:150:25)     at REPLServer.eval (/usr/local/lib/node_modules/messer/index.js:128:54)     at bound (domain.js:301:14)     at REPLServer.runBound [as eval] (domain.js:314:12)     at REPLServer.onLine (repl.js:433:10)     at emitOne (events.js:115:13)     at REPLServer.emit (events.js:210:7)     at REPLServer.Interface._onLine (readline.js:278:10)     at REPLServer.Interface._line (readline.js:625:8)     at REPLServer.Interface._ttyWrite (readline.js:904:14)     at REPLServer.self._ttyWrite (repl.js:502:7)     at ReadStream.onkeypress (readline.js:157:10)
For some reason message or m isn't working for a friend named Mike Smith. What could be going on here? Did Facebook update something?

See #48 and #52


Quiet debug test runner output when in machine mode.

See: https://github.com/flutter/flutter-intellij/issues/1176
@tvolkert

Thanks!


Removing compiler warning

Submisison Checklist

[x] Run unit tests: ./runTests.py test/unit
[x] Run cpplint: make cpplint
[x] Declare copyright holder and open-source license: see below

Summary:
Squashes a compiler warning.
How to Verify:
Run:
./runTests.py test/unit/math/fwd/arr/meta/OperandsAndPartials_test.cpp
There should be no compiler warnings. In the current develop branch, there is a warning.
Side Effects:
This isn't actually used in Stan yet, so there should be no side-effects.
Documentation:
None required.
Reviewer Suggestions:
Anyone.
Copyright and Licensing
Please list the copyright holder for the work you are submitting (this will be you or your assignee, such as a university or company): Columbia University
By submitting this pull request, the copyright holder is agreeing to license the submitted work under the following licenses:

Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

Looks good.


How to force-nl on -> only when exceeds the width

Added below config to the :fn-map, don't know why it won't work.
  "->"              [:noarg1-body
                     {:list               {:constant-pair? false :force-nl? false}
                      :next-inner-restore [[:list :constant-pair?]]}]

code showing where it won't work https://github.com/status-im/status-mobile/pull/14128/commits/9ba1f28dbf14215a39833a34d346dd746d58ce3b#diff-acc2864028c846a967aae52fa52b23048f98d0b5cee98b2e118047f2dbb52171R94-R96

Good question.  This is clearly a bug, though whether of code or documentation is a open question.  It turns out that :noarg1-body is in the set :fn-force-nl, which causes it to never be formatted onto on line.  Since -> is the only "user" of :noarg1-body, you could get what you want by simply removing :noarg1-body from :fn-force-nl, thus:
(czprint i272 {:parse-string? true :remove {:fn-force-nl #{:noarg1-body}}})
(defn pressable-hooks
  [props]
  (let [{background-color :bgColor,
         border-radius :borderRadius,
         border-color :borderColor,
         border-width :borderWidth,
         type :type,
         disabled :disabled,
         on-press :onPress,
         on-long-press :onLongPress,
         on-press-start :onPressStart,
         accessibility-label :accessibilityLabel,
         children :children,
         :or {border-radius 0, type "primary"}}
          (bean/bean props)
        long-press-ref (react/create-ref)
        state (animated/use-value (:undetermined gesture-handler/states))
        active (animated/eq state (:began gesture-handler/states))
        gesture-handler (animated/use-gesture {:state state})
        animation (react/use-memo
                    (fn []
                      (animated/with-timing-transition
                        active
                        {:duration (animated/cond* active time-in time-out),
                         :easing (:ease-in animated/easings)}))
                    [])
        {:keys [background foreground]}
          (react/use-memo (fn []
                            (type->animation {:type (keyword type),
                                              :animation animation}))
                          [type])
        handle-press (fn [] (when on-press (on-press)))
        long-gesture-handler
          (react/callback
            (fn [^js evt]
              (let [gesture-state (-> evt .-nativeEvent .-state)]
                (when (and on-press-start
                           (= gesture-state (:began gesture-handler/states)))
                  (on-press-start))
                (when (and on-long-press
                           (= gesture-state (:active gesture-handler/states)))
                  (on-long-press)
                  (animated/set-value state
                                      (:undetermined gesture-handler/states)))))
            [on-long-press on-press-start])]
    (animated/code!
      (fn []
        (when on-press
          (animated/cond* (animated/eq state (:end gesture-handler/states))
                          [(animated/set state
                                         (:undetermined gesture-handler/states))
                           (animated/call* [] handle-press)])))
      [on-press])
    (reagent/as-element
      [gesture-handler/long-press-gesture-handler
       {:enabled (boolean (and on-long-press (not disabled))),
        :on-handler-state-change long-gesture-handler,
        :min-duration-ms long-press-duration,
        :max-dist 22,
        :ref long-press-ref}
       [animated/view
        {:accessible true, :accessibility-label accessibility-label}
        [gesture-handler/tap-gesture-handler
         (merge
           gesture-handler
           {:shouldCancelWhenOutside true,
            :wait-for long-press-ref,
            :enabled (boolean (and (or on-press on-long-press on-press-start)
                                   (not disabled)))})
         [animated/view
          [animated/view
           {:style (merge absolute-fill
                          background
                          {:background-color background-color,
                           :border-radius border-radius,
                           :border-color border-color,
                           :border-width border-width})}]
          (into [animated/view {:style foreground}]
                (react/get-children children))]]]])))

I believe this will get you what you want. I'll fix the documentation of this function style to include the fact that it is included in :fn-force-nl.  Thanks!
Thanks for asking!

Thanks. It works now. I tried below before and it won't work. Seems the :remove option has a higher priority.
:fn-force-nl #{:noarg1 :force-nl-body :force-nl :flow
                  :arg1-force-nl :arg1-force-nl-body :flow-body
                  :arg2-force-nl-body :arg2-force-nl}

Yes, the above where you defined :fn-force-nl yourself doesn't remove things.  Handling sets in the options map is complicated, and so what I ended up doing was allowing anything you specify to add to the set, but it doesn't redefine the entire set.  You have to use :remove to get default things out of the set.  Which may not be clear.  Maybe this from the zprint reference might be a little clearer.


Regex for pool names does not contain dash character

ISSUE TYPE

Bug Report

AS3 BUILD/ VERSION
3.7.0

BIGIP VERSION

Sys::Version
Main Package
  Product     BIG-IP
  Version     <IP_ADDRESS>
  Build       0.0.1
  Edition     Point Release 3
  Date        Wed Nov 28 18:50:45 PST 2018

SUMMARY
Regex for pool name does not include dash character, which is a valid character.

STEPS TO REPRODUCE

EXPECTED RESULTS
Dash "-" is a valid character for F5 pool name, but is not included in regex.
ACTUAL RESULTS

{'code': 422,
 'declarationFullId': '',
 'errors': ['/POD31/lvsp31_pools: propertyName "BLAH-BLA_443" should match '
            'pattern "^[A-Za-z][0-9A-Za-z_]{0,47}$"'],
 'message': 'declaration is invalid'}

Hi @krambone95,
This exclusion of hyphen is deliberate / by design and is not a bug. This pattern check is applied to other big-ip components and not just the pool name.
We have opened internal issue 805 to improve our documentation regarding this.
We do have a bit of explanation, currently available on clouddocs: as3-declarations-and-big-ip-configuration-component-names
Thanks,
Kat

This has been added to our cloud docs page. See https://cloudsolutions.pages.gitswarm.f5net.com/f5-appsvcs/userguide/tips-warnings.html?highlight=dash/hyphen (-)#notes-and-tips


Implement Tendermint migration conclusion

See the latest update to the BEP-42.
The logic is triggered by the last election vote.

[x] Generate a new chain ID and update the currently set value.
[x] Turn on a flag that prevents BigchainDB from accepting new transactions.
[x] Turn the flag off upon receiving the matching chain_id in InitChain.
[x] Update the ABCI interface (e. g. the info command) to return height - chain offset to Tendermint instead of just height.

See https://github.com/bigchaindb/bigchaindb/pull/2488.


[JTC] set_hold_position results into not exactly reaching desired target

Describe the bug
In #558  (included in rolled out version 3.14.0)  set_hold_position was added which commands the current position as the target after sampling the last point of the trajectory (https://github.com/ros-controls/ros2_controllers/blob/1dac309979fb9392dc12feb6b46555582fed6fc6/joint_trajectory_controller/src/joint_trajectory_controller.cpp#L346).
This results into not reaching the target exactly. (In our case a UR16e arm).
As far as I see there is no way to disable or tune this behavior
To Reproduce
Take some robot arm. Move rather fast from A to B. Check if the desired target has been reached within a tolerance of for example 0.001rad
Expected behavior
In case the checks (e.g. arm is within goal tolerance) were successful set the last point of the trajectory as target
Environment (please complete the following information):

iron
ros2control : 3.14.0

@christophfroehlich  If I understand the code correctly the goal tolerances are just there for checking if the goal has been reached after sampling the last point of the trajectory. If the arm takes just a tiny bit of time to precisely reach the target this check would fail.
I worked around this in the previous version of the JTC by implementing our own check after the action has been executed successfully, but this only works under the precondition that the JTC actually sets the last point of the trajectory as desired target.

this is not 100% correct, it checks for he duration of the goal_time parameter, not only once.
this means, if you set a nonzero goal_time it should act as you desire

I did set the goal_time. Seems to work now. It was just confusing because there was a change in behavior compared to the previous version.
Nevertheless I think

We could add an option for using the desired states instead of the measured ones.

would be rather nice to have.


Some weird color artefacts

Hello! I use that code:
model = ModelLoader().load_from_file(r"4x_RealisticRescaler_100000_G.pth")
assert isinstance(model, ImageModelDescriptor)
model.to("cuda:0")
model.eval()

preprocess = transforms.Compose([
    transforms.ToTensor(),
])

image = Image.open("input.jpg")
image_tensor = preprocess(image).unsqueeze(0)

image_tensor = image_tensor.to("cuda:0")

processed_image_tensor = process(image_tensor)
processed_image = transforms.ToPILImage()(processed_image_tensor.squeeze())

processed_image.save("output.png")

I've used different models, different CPUs/GPUs, different computers.

Looks like an overflow issue. Models do not guarantee exact 0-1 output values. They might slightly off (e.g. -0.0001). So when you convert those values directly to integers, you might get -1, which wraps around to 255.
To fix this, clamp the output tensor .clamp_(0, 1) and that should hopefully fix the issue.

Maybe we should consider this a bug in spandrel. Our call API says: "The range of the output tensor will be [0, 1]." So a model returning values outside this range is a bug as per this description.
The fix would for us to always call .clamp_(0, 1) on the output tensor (within our call API). This wouldn't even be a performance impact, because callers have to do the clamp anyway since they can't trust spandrel right now. So this would make spandrel easier to use correctly.
What do you think @joeyballentine?

Also, @unblock7 did .clamp_(0, 1) fix your problem?

I think that would make sense to do. The only thing is that technically speaking then we're not giving the true output of the model. But if you want that you can use the model directly rather than the call API, so yeah that makes sense to me.
We just need to make sure we respect range. Some swinir models (the jpeg ones) use 0-255

We just need to make sure we respect range. Some swinir models (the jpeg ones) use 0-255

Our call API says that we input and output 0-1 images. If some SwinIR models expect 0-255, we have to do the conversion inside the call API.

I just looked and swinIR already handles the image range internally, so there's no need for us to worry about it.

Also @unblock7, did .clamp_(0, 1) fix your problem?

Yes, that worked, thank you!


Migrating to create-react-app

Is there a simpler way to migrate a project from this structure to create-react-app one?

There is not currently, sorry.


Elide url passwords in cli arguments

Fixes #2365

Really nice!

@rndstr Feel free to ignore the netlify errors, it's flakey right now (CC @errordeveloper )


Retrieve not allowed on channel ActivityEngagementVirtualChannel - HVS in scratch org

Summary
When a scratch org (with HighVelocitySales feature enabled) gets created with the following definition file, attempting to pull (without even changing anything in the org) ends up giving the following error:
Retrieve not allowed on channel ActivityEngagementVirtualChannel.

Steps To Reproduce:

Create a scratch org with HighVelocitySales and DevelopmentWave features.
Open the org, navigate to Setup > High Velocity Sales, and enable the feature. (no way of doing it over a definition file as stated here: "High Velocity Sales can be deployed in scratch orgs, but the settings for the feature can’t be updated through the scratch org definition file. Instead, configure settings directly in the High Velocity Sales app." https://help.salesforce.com/articleView?id=hvs_considerations.htm&type=5)
Without doing anything else, run sfdx force:source:pull

Expected result
I should be able to retrieve automatically created metadata of type WaveDashboard, WaveDataset, WaveDataflow, WaveApplication, WaveXmd, etc
Actual result
The following automatically gets added to the org:
STATE /                               TYPE                                                    /                                              PROJECT PATH
Remote Add  / ActivityEngagementVirtualChannel_LeadChangeEvent            /         PlatformEventChannelMember
Remote Add  / ActivityEngagementVirtualChannel_TaskChangeEvent                    /   PlatformEventChannelMember
Remote Add  / ActivityEngagementVirtualChannel_EmailMessageChangeEvent     / PlatformEventChannelMember
Remote Add  / ActivityEngagementVirtualChannel_ContactChangeEvent             /    PlatformEventChannelMember
Remote Add  / ActivityEngagementVirtualChannel_ListEmailChangeEvent          /     PlatformEventChannelMember
And when you pull;
If your API Version greater than 46.0
You will be given an error message saying Retrieve not allowed on channel ActivityEngagementVirtualChannel.
If your API Version less than 47.0
You will be given an error message saying Entity type 'PlatformEventChannelMember' is not available in this api version
(which is explained here: https://developer.salesforce.com/docs/atlas.en-us.api_meta.meta/api_meta/meta_platformeventchannel.htm)
Additional information
SFDX CLI Version: 7.82.1-0
SFDX plugin Version
@oclif/plugin-autocomplete 0.2.0
@oclif/plugin-commands 1.3.0 (core)
@oclif/plugin-not-found 1.2.4 (core)
@oclif/plugin-plugins 1.9.4 (core)
@oclif/plugin-warn-if-update-available 1.7.0 (core)
@oclif/plugin-which 1.0.3 (core)
@salesforce/lwc-dev-server 2.7.0
├─ @oclif/plugin-help 2.2.3
└─ @oclif/plugin-update 1.3.9
@salesforce/sfdx-diff 0.0.6
@salesforce/sfdx-scanner 2.4.0
@salesforce/sfdx-trust 3.6.0 (core)
alias 1.1.3 (core)
analytics 1.12.1 (core)
auth 1.4.5 (core)
config 1.2.1 (core)
generator 1.1.3 (core)
salesforcedx 50.7.1 (core)
├─ apex 0.1.4 (core)
├─ @salesforce/sfdx-plugin-lwc-test 0.1.7 (core)
├─ schema 1.0.2 (core)
├─ templates 50.1.0 (core)
├─ custom-metadata 1.0.10 (core)
└─ salesforce-alm 50.7.1 (core)
sfdx-cli 7.82.1-0 (core)
sfdx-destruction 1.0.0-beta.3
sfdx-git-delta 4.1.0
shane-sfdx-plugins 4.43.0
├─ @mshanemc/sfdx-sosl 1.1.0
└─ @mshanemc/plugin-streaming 1.1.7
OS and version: macOS Big Sur - Version 11.0.1

I am experiencing the same issue. Any help would be really appreciated 👍

@clairebianchi any suggestions for this?

could you try running the command with the --dev-debug flag and post the output here? That will help us narrow down the issue to a server side, metadata, or cli issue

could you try running the command with the --dev-debug flag and post the output here? That will help us narrow down the issue to a server side, metadata, or cli issue

Hi @WillieRuemmele,
i attached the requested output with the --dev-debug flag.
sfdx_error.txt

Dear All, I am also getting same kind of error- Any luck on this?

That type (ActivityEngagementVirtualChannel) is not in the metadata coverage report.
https://developer.salesforce.com/docs/metadata-coverage/
So if you don't need to use it and it's only causing problems, put it in your forceignore file


Feat(serving): add sklearn and xgboost simple serving

What this PR does / why we need it:
Feat(serving): add sklearn and xgboost simple serving
The first commit add mlserver source code for image build, see https://github.com/SeldonIO/MLServer
Which issue(s) this PR is related to (optional, link to 3rd issue(s)):
Fixes #
Reference to #

Special notes for your reviewer:
/cc @gaocegege @simon-cj

Release note:

NONE

/approve
/cc @gaocegege

Can write a internal documentation，let others can know some background and how to request the inference api.

Can write a internal documentation，let others can know some background and how to request the inference api.

I'll pull another request for the docs

/lgtm


Redirects for "Schlau-Werkstatt"

One of our content partners "Schlau-Werkstatt" would like to have 4 redirects to better promote the corresponding Serlo content.
serlo.org/schlau-größen -> https://de.serlo.org/183604
serlo.org /schlau-geometrie -> https://de.serlo.org/183603
serlo.org /schlau-rechenregeln-rechenbegriffe -> https://de.serlo.org/146948
serlo.org /schlau-ganzezahlen -> https://de.serlo.org/183602

Hm... Can't they do that on their end? E.g. redirect something like https://www.schlau-werkstatt.de/serlo-größen to https://de.serlo.org/183604? This way, they are responsible for supporting those as long as they need the URLs.

@yolo-bird When they cannot do this redirect by their own you can help them setting up schlau-werkstatt.link (similar to serlo.link). Then they can manage the redirects by themselves.

Thanx for the input. Then I'll further discuss the topic of redirects with them and will use one of the above mentioned solutions.


KAFKA-15817: Avoid reconnecting to the same IP address if multiple ad…

…dresses are available
This change is a follow-up to https://github.com/apache/kafka/pull/9902. When re-resolving DNS after disconnecting, it is possible (likely, even) that we will resolve the same set of addresses in the same order, meaning we could attempt to reconnect to the same IP that we just disconnected from. In the case where we disconnected from the IP address because it has become unhealthy (due to a load balancer going down or a network routing layer restarting, for example), the client will need to time out before connecting to the next IP. This essentially unifies the behavior before and after KAFKA-12193: re-resolve after disconnecting while still progressing through the IP list.
Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

Thanks for the PR. Let's cherry pick this to 3.6 after this is merged to master.

test failures are know flaky tests

@lucasbru Did you cherry-pick to 3.6?

@ijuma I did not. My understanding was that this shouldn't be included in 3.6.1, so I was waiting for 3.6.1 to go through before cherry-picking it.


ebpf warnings

The Hook kernel generates a hundred or so lines warning about ebpf.. we possibly could just disable it in the kernel config? Or investigate it in more detail
Expected Behaviour
A nice clean dmesg output on boot :-)
Current Behaviour
A lot of scrolling messages about ebpf, potentially pushing away any other warning messages.

merged.


Loading Unstable Nuclides from the RIPL-3 Database using the ARMI_RIPL_PATH Environment Variable

Adding an ARMI_RIPL_PATH environment variable to determine if
nuclide bases should read in unstable nuclide data from the RIPL-3
data files during the factory function call. These data files are not
distributed with the framework, so a user will have to download them
separately from https://www-nds.iaea.org/RIPL-3/levels/.

I'm excited about this feature. This will help anyone who wants to deal with all fission products. I have the in-line comments above, plus:

Add documentation to the installation section of the user manual telling people about setting this environmental variable

To explicitly highlight what this feature adds, i've written the nuclides that are processed within nuclideBases.py with and without the RIPL-3 data files. Working on addressing the other comments now.
nuclides-with-ripl-3.txt
nuclides-without-ripl-3.txt

looks good. in the squash commit msg be sure to note why you eliminated mc2 labels from the big table. Fine by me, just mention why.


The SEO tags are totally messed up due to the Digest/Journal split

Won't do as we'll maybe switch to another tooling with the new team member.


Support specification of docker image for distribution

Support
distribution {
    DOCKER="test/test-component"
}

Also should be supported in end-point
rest/api/2/components/{component}/versions/1.2.3/distribution

@aryabokon , @CamaradeRoman , please look are you ok with with configuration in CRS?

DOCKER in distribution section looks good, but I will appreciate anyone who add this parameter to distribution section of  rest/api/2/components/{component}/versions/1.2.3

@pgorbachev it is OK if we fix the agreement that image tag must be equal to version. But it means that we won't be able to use "standard" approach of image classification like postgres:13.12, postgres:13.12-alpine, postgres:13.12-alpine3.17, postgres:13.12-alpine3.18, postgres:13.12-bookworm etc.

Already implemented


Identify unique hardware ID

There's probably some kind of serial on the Pico Pi that can be used to identify specific devices for the purpsoses of linking to the companion site

byte_id = machine.unique_id()
hex_id = "".join("{:02x}".format(byte) for byte in byte_id)

Easy peasy


[zh-cn] Replace replace deprecated fullversion shortcode

Problem:
Based on https://github.com/kubernetes/website/issues/41005, I have created this issue/task to help update the [zh-cn] language documentation.

We've deprecated the fullversion param ({{< param "fullversion" >}}) and now English documents are using {{< skew currentPatchVersion >}} instead. (by #40692)
All the other documents need to change {{< param "fullversion" >}} into {{< skew currentPatchVersion >}} or v{{< skew currentPatchVersion >}} appropriatly.
Since several languages are involved, I'd like to share this issue to check all localizations complete the task.

Proposed Solution:

For each localization team, search fullversion and change appropriately. (See the reference, https://github.com/kubernetes/website/pull/40692/files)

Page to Update:
In the main branch, I found:
Results for "param "fullversion"" in "k8s-website/content/zh-cn"
/home/rdlugyhe/kubernetes/k8s-website/content/zh-cn/docs/tasks/tools/install-kubectl-linux.md:
96:     curl -LO https://dl.k8s.io/release/{{< param "fullversion" >}}/bin/linux/arm64/kubectl
/home/rdlugyhe/kubernetes/k8s-website/content/zh-cn/docs/tasks/tools/install-kubectl-windows.md:
58:  1. Download the [latest release {{< param "fullversion" >}}](https://dl.k8s.io/release/{{< param "fullversion"
58:  "fullversion" >}}](https://dl.k8s.io/release/{{< param "fullversion" >}}/bin/windows/amd64/kubectl.exe).

/label language-zh

cc: @seokho-son

/language zh

/sig docs
/sig localization

/triage accepted
/area localization
/priority backlog

Resolved by #41528
/close

/reopen
We should also update content/zh-cn/docs/tasks/tools/install-kubectl-windows.md


Labelled points

Would it be possible to add labelled points support much like this PyVista example

Good idea! Yes, I would also like to have this :+1:


kernel: enable PCIE_ROCKCHIP_EP

Motivation for this change
Things done

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

[ ] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Added a release notes entry if the change is major or breaking
[ ] Fits CONTRIBUTING.md.

According yo my research in https://github.com/NixOS/nixos-hardware/pull/259 the necessary flags are:
PHY_ROCKCHIP_PCIE y
PCIE_ROCKCHIP_HOST y


raftstore: fix redundant heartbeat

What have you changed? (mandatory)
This PR fixes the problem which may send redundant heartbeat to PD. If a peer is the leader, it will call heartbeat_pd, which will put the pending peer into peers_start_pending_time. And if the ConfChangeType is AddLearnerNode, it will be pushed again. So it might send two heartbeats to PD at the same time.
What are the type of the changes? (mandatory)

Bug fix (non-breaking change which fixes an issue)

How has this PR been tested? (mandatory)
manual tests

@rleungx Please cherry-pick it to release-2.1 branch.


use styled-jsx/css

As discussed in #14 and #15, styles might get too bloated in the components and can separated out to simplify things.

Hmm, seems to have failed on "Checkout code" 🤔

That's weird, I didn't close this  ¯\(ツ)/¯

I think something funky is happening since this repo went public while this PR is from my forked, private copy.

Whoop! Hopefully easier now. I pulled the trigger on public so that now.sh would hopefully work on forked PRs.


Modified Container output test to match uppercase input

Needed for the "should be consumable via the environment" tests to pass.

Hi @bclau. Thanks for your PR.
I'm waiting for a e2e-win member to verify that this patch is reasonable to test. If it is, they should reply with /ok-to-test on its own line. Until that is done, I will not automatically test new commits in this PR, but the usual testing commands by org members will still work. Regular contributors should join the org to skip this step.
I understand the commands that are listed here.

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


[Resource Explorer] Replace version buttons with a toggle button

Currently on Resource Explorer, the 2 version buttons are taking up too much space. The user is only to pick a version once but their main focus is on the actual resources tree.
Instead of the 2 big button, we can use a toggle button to switch to beta.
The by default view on the tree is V1.
The toggle button would say switch to Beta so the user knows that if they wish to explore resources on Beta, they will have to switch the button.
There are Fluent UI toggle buttons to use out of the box but the team should check and review with the UX team first.
Current view:

To replace with : (This isn't a final design)

Hi @thewahome thank you for quickly looking at this. When I tested, the design looks very different from what's on the ticket and it is confusing to have beta when the toggle is off. Does that mean I'm looking at the beta version or does it mean beta is off and what I'm seeing is v1?
Can we please have a design similar to what's on the ticket: keep it all to one line, default view is v1, should the user wish to view beta they would switch the toggle that says switch to beta on and then the tree would load beta . Thanks

Hi @thewahome is there a reason why this has been closed? Thanks


Fix stdout capture for test tearDown under failure scenario for Python <= 3.1

Fix #2929 Tests do not setUp/tearDown correctly
As mentioned in the bug, setUp and tearDown were done correctly, but the wrong output led you to believe that it wasn't.

👍  with whatever change you decide for the test message.


Making initial object for SomaticSignatures

Hi,
This is not a problem with this software rather my confusion
I have a list of .vcf files from WGS contains called SNV and indels. I have extracted information from them like below
> head(mutations)
  sampleID chr     pos ref mut
1    t_005   1  928006   G   A
2    t_005   1 1649842   G   T
3    t_005   1 2020408   G   A
4    t_005   1 2031677   T   A
5    t_005   1 2901878   G   A
6    t_005   1 3039338   G   A
>

I want to make VRanges object from my vcf files or the information I extracted from .vcf files, because I need to find signature in my WGS samples, could you please help me?
Thanks a lot in advance

You can have a look at the vignette of the package that describes this step using the VRanges function of the VariantAnnotation package.


Portworx bad volume name in IBM-portworx module

When shortening volume names, they sometimes end with a - character, depending on the length of the prefix.  When this happens the module fails with an error like this:
Error: "name" ("3-pwx-kube-c9fj1nnw0d0i0q7og6d0-masdev2-default-00000c0c") should match regexp ^([a-z]|[a-z][-a-z0-9]*[a-z0-9])$
│
│  with module.ibm-portworx.ibm_is_volume.volume[3],
│  on .terraform/modules/ibm-portworx/main.tf line 67, in resource "ibm_is_volume" "volume":
│  67:  name      = substr("${replace(var.name_prefix, "_", "-")}${length(var.name_prefix) > 0 ? "-" : ""}${count.index}-pwx-${var.workers[count.index].id}", 0, 61)
private channel
 length of 61 characters for volume name
│
╵

We need to make sure that the generated name does NOT end with a dash (-).

We also need to handle the case where it will fail b/c the volume name can't start with a number.  This happens if no name prefix is specified.


devscripts/buildserver.py:  format block comment

There is no difference. This file is not checked by flake8.

I ran it with latest flake8 from git and it show this warning.

This file is not checked by flake8.

Oh, sorry. I see.


Debuggable Builds [FEATURE]

Problem
I cannot use neopo to build debuggable builds for use with the vscode particle debugger
Neopo is much faster than the built-in vscode compile / flash commands from the Particle Workbench
Neopo is readily usable in CI/CD environments (thank you so much)
I use a gitlab job to automate and archive builds with neopo, but I have no way of automating builds for debuggable versions. Currently debug builds are not available from the neopo build /path/to/project command
Problem
When I read the neopo docs, I thought it was not possible to build debug builds. The "flags" examples show how to do this, but they do not explain that the result of neopo flags "-D DEBUG_BUILD" will be that the build will work with the VSCode Particle Workbench Debugger. My initial reaction to this, as someone only becoming familiar with the particle ecosystem, was that it was a custom flag with only relevance to some specific project.
Suggested Solution
neopo build -d /path/to/project
Use a neopo cli flag like -d to request a debug build
Alternatives

edit the docs https://neopo.xyz/docs/full-docs to indicate that neopo flags "-D DEBUG_BUILD" will enable compatibility with the VSCode Particle Workbench Debugger. Add this line of text beneath that example:
Note: the 'DEBUG_ENABLED' flag will enable compatibility with the VSCode Particle Workbench Debugger

Hey @briveramelo,
Thank you for using neopo. I'm glad that it has helped you with your Particle development needs.
Neopo has several options for building projects. The most common is build, which uses the compile-user target from the Workbench Makefile. However, there are also: flash, flash-all and clean.
To use other makefile targets you can use the run command, as hinted by the following in the neopo help output:
run <target> [project] [-v/q]             # Run a makefile target

To run a debug build you would use:
neopo run compile-debug /path/to/project

I apologize if this was unclear in the documentation.
The following served as a way to define DEBUG_ENABLED for use in application code:
neopo flags "-D DEBUG_ENABLED"

#ifdef DEBUG_ENABLED
Serial.println("some debugging message");
#endif

Lately, I have used the neopo manpage to hold the most up to date documentation which I also host on neopo.xyz. I've been meaning to incorporate the updated documentation into the other webpage.

Yes, it has helped greatly, and it's lightning quick compared to the vscode particle workbench, so props!
What do you think about the proposals then? Adding a flag would be nice, but I do recommend at least updating the docs to clarify that the "DEBUB_ENABLED" preprocessor is not some arbitrary preprocessor, but in fact a reserved particle toolchain preprocessor needed to create debug builds.
A note like this would do the trick
Note: the 'DEBUG_ENABLED' flag will enable compatibility with the VSCode Particle Workbench Debugger

Note: the 'DEBUG_ENABLED' flag will enable compatibility with the VSCode Particle Workbench Debugger

This is false to the best of my knowledge.
The preprocessor flag to enable debugging is DEBUG_BUILD, not DEBUG_ENABLED.
https://github.com/particle-iot/device-os/blob/develop/docs/debugging.md
In fact DEBUG_ENABLED isn't used anywhere in the deviceOS source. I'll admit that I had never checked for this, but you can see for yourself with:
$ grep -rI DEBUG_ENABLED ~/.particle/toolchains/deviceOS/2.0.0-rc.4

Because neopo uses the same makefile as Workbench (~/.particle/toolchains/buildscripts/1.9.2/Makefile), when using any of the debug targets in the makefile, DEBUG_BUILD=y is automatically set. An example of this would be in the compile-debug target.
I agree that the documentation is rough as I develop neopo in my spare time. When I revamp the documentation, I'll make sure to change that example to not contain DEBUG to avoid further confusion.
Have you tried using the neopo run command? You can use it to run any of the targets in the makefile, including the debug targets.

Oh this is bizarre. Not even the reserved DEBUG_BUILD flag is needed to connect via the debugger. I'm using deviceOS 2.0.0. I just deleted my target folder, ran neopo clean && neopo flags "" && neopo build . && neopo flash . (I am aware of the scripts feature, just did this for a quick run), and it hits breakpoints just fine. I recently got breakpoints working in CLion, so I've tested in both CLion and VSCode, and they both hit breakpoints and work as expected without any flags in the build. The strange thing is that I'm almost certain this did not work when I was first attempting, which is what prompted this entire thread.
I apologize for the distraction. The original issue must have been something else entirely.
Thank you for looking into this though


I get this error on Windows 10 "AttributeError: module 'os' has no attribute 'uname'. Did you mean: 'name'?"

I've successfully installed and used 'rawdog' on my personal workstation at home. But when I installed and ran it on my corporate workstation I get the error:
AttributeError: module 'os' has no attribute 'uname'. Did you mean: 'name'?
My quick google suggests that this has to do with Windows not using the 'os.uname' method but instead should use 'platform.name'.
Is there a way to make sure 'rawdog' is using the Windows method?
Thanks in advanced!

I should point out that i also got this error first:
ModuleNotFoundError: No module named 'readline'
But this error was fixed by:
pip install pyreadline

Just merged a fix! https://github.com/AbanteAI/rawdog/pull/40
Let me know if that works for you. If you installed with pip it's not updated there yet - if you clone the repo you could test it

Cool! Thank you! I'll give this a try.

I'm sorry to ask. How do I run 'rawdog' after I've cloned the repo?  There's no executable in the folders.

pip install -e . (you may want to do this in a venv). Then you should be able to run rawdog.

Thanks for that. I ran the pip install -e . in the git cloned folder and it installed without issue.  But I still get this error:
(venv) PS C:\Temp\rawdog> rawdog How many files are in this directory?
C:\Temp\rawdog\src\rawdog_main_.py:64: SyntaxWarning: invalid escape sequence '_'
/_____/   U    Rawdog v{version}"""
Traceback (most recent call last):
File "", line 198, in run_module_as_main
File "", line 88, in run_code
File "C:\Temp\venv\Scripts\rawdog.exe_main.py", line 4, in
File "C:\Temp\rawdog\src\rawdog_main.py", line 5, in
import readline
ModuleNotFoundError: No module named 'readline'

ah, seems readline is a python module that's not available on windows.
You fixed it before with pip install pyreadline, does that work? I'll look into options to make this work for everyone

Okay, I made some progress. I installed the pyreadline3 (need the version 3):
pip install --trusted-host pythonhosted.org --trusted-host files.pythonhosted.org pyreadline3
Then I saw another error with 'pkg_resources':
(venv) PS C:\Temp\rawdog> pip install --trusted-host pythonhosted.org --trusted-host files.pythonhosted.org pyreadline3
Collecting pyreadline3
Using cached pyreadline3-3.4.1-py3-none-any.whl (95 kB)
Installing collected packages: pyreadline3
Successfully installed pyreadline3-3.4.1
(venv) PS C:\Temp\rawdog> rawdog How many files are in this directory?
Traceback (most recent call last):
File "", line 198, in run_module_as_main
File "", line 88, in run_code
File "C:\Temp\venv\Scripts\rawdog.exe_main.py", line 4, in
File "C:\Temp\rawdog\src\rawdog_main.py", line 9, in
from rawdog.llm_client import LLMClient
File "C:\Temp\rawdog\src\rawdog\llm_client.py", line 9, in
from litellm import completion, completion_cost
File "C:\Temp\venv\Lib\site-packages\litellm_init_.py", line 520, in
from .utils import (
File "C:\Temp\venv\Lib\site-packages\litellm\utils.py", line 32, in
import pkg_resources
ModuleNotFoundError: No module named 'pkg_resources'
The 'pkg_resources' is in setuptools, so I:
pip install --trusted-host pythonhosted.org --trusted-host files.pythonhosted.org setuptools wheel
Then when I run rawdog. It actually starts kind of. But I get a huge wall of error text and at the bottom saying "
"openai.APIConnectionError: Connection error."
The wall of error messages shows a lot of self-signed certificate errors and SSL: CERTIFICATE_VERIFY_FAILED messages.
So I think my current problem is due to our firewall acting as man-in-the-middle with our corporate certs.  There is some troubleshooting I found related to it, but it will take awhile to figure out what to do. There is a way to disable Python cert verification, that's why I was using the --trusted-host flags when installing. But now that rawdog is reaching out to OpenAI, I have no way of 'trusting' that URL.

ah, seems readline is a python module that's not available on windows.
You fixed it before with pip install pyreadline, does that work? I'll look into options to make this work for everyone

requirements.txt:
pyreadline3==3.4.1; platform_system == "Windows"
But can't test it myself as I'm on mac

@jonatj does it work when you're not on the corporate firewall?

Yes, it works perfectly on my home computer (Windows 11). So I know it’s just our corporate network affected this. I also don’t have administrator rights on my work computer, but I’m not sure that is related. I’m really bummed because I love the functionality of Rawdog and wanted everyone to use it on my team. On Feb 8, 2024, at 12:03 PM, biobootloader @.***> wrote:
@jonatj does it work when you're not on the corporate firewall?
—Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: @.***>

I've successfully installed and used 'rawdog' on my personal workstation at home. But when I installed and ran it on my corporate workstation I get the error:
AttributeError: module 'os' has no attribute 'uname'. Did you mean: 'name'?
My quick google suggests that this has to do with Windows not using the 'os.uname' method but instead should use 'platform.name'.
Is there a way to make sure 'rawdog' is using the Windows method?
Thanks in advanced!

Being a Windows-exclusive error,
host = os.uname()[1]
had to be changed to:
host = os.name
That's the fix.


Recurrent neural networks and string arrays

Summary
Improvement in input/output formats when dealing with arrays of strings / tokens for RNN.
Basic example
Currently, the LSTM model for example, can accept different types of data as input and as training data, arrays and strings. While the support for strings out of the box is great, it has some defaults that don't seem to fit so many use cases.
Current behavior

Given raw strings:

const data = [
  {
    input: 'hello I am an input',
    output: 'hi I am the output'
  }
]

This will implicitly use character tokenization behaving as: input.split('') and output.join(''). See #799 which is a common case of LSTM usage I believe, string input and labels output. Because of the default behavior, the labels are being treated as actual text.
It is possible to preprocess data by splitting everything to arrays so the model will be mapping words to neurons instead of characters.

Given arrays of strings

const data = [
  {
    input: ['I', 'split', 'my', 'input', 'into', 'tokens']
    output: ['so', 'I', 'expect', 'tokens', 'as', 'output']
  }
]

Here, we are using words instead of characters. That helps having simpler data to learn from for our model, and it also assures us of the output being exactly what we expect in term of label names. The problem with this right now is my output would be soIexpecttokensasoutput because of the output.join('') behavior. Working with string arrays is not documented anywhere so it surely causes trouble to some users of the library.
Current workaround
For now the most basic workaround I can think of is to add a space to every word in an array so that the output is readable and processable.
const data = [
  {
    input: ['I ', 'split ', 'my ', 'input ', 'into ', 'tokens ']
    output: ['so ', 'I ', 'expect ', 'tokens ', 'as ', 'output ']
  }
]

Possible improvement
I think the best improvement would be to have the same output format as the input. In the case of #799 where strings are being used as input and arrays as output, it should probably throw an explicit error about the input being a string while the output is an array. Preprocessing the data by splitting it into words input.split(' ') would be a pretty easy step for the user, rather than figuring out both how the input and output are mapped to neurons and how the output formatted.
Motivation
Apart from the #799 issue, I have been dealing with that problem and spent some time on it until I could write a lot of extra code to get some workaround. String arrays using RNN are not really documented and don't really provide a lot of customization.
A lot of the RNN (specifically LSTM) usage seems to be for reading questions or requests and writing a response using words that are related to a topic in a pretty limited vocabulary, as well as usage for mapping labels to a specific textual input which is not really well supported at the moment and has very implicit behavior. We could provide a more flexible RNN that is more easily customizable regarding how it treats the input and output, as well as making it more adapted to more usage.

Working on this now.  Not as easy as it seems, but have started abstracting the DataFormatter into a different type that can work with typescript generics and the original api.  This is a priority for me because this is an existing feature that isn't alpha/beta.


Some meshes in scene file do not get updated when changing source file - other meshes do

Tested versions

Reproducable in Godot v4.3.stable.official [77dcf97d8]

System information
Windows 11 - Godot v4.3.stable.official
Issue description
Godot is not updating prefabs correctly.
The intended update architecture is:
.blend (raw meshes) -> .tscn (set up scene) -> .tscn (scene variant)
When editing a bodyshape of a 3d model the set up scene updates and SOME of the scene variants. Some just keep the old mesh shape.
I even tried to add a cube to the blendfile which appeared in ALL the scenes. The bodyshape however was still only updated in some scenes.

Steps to reproduce
There are files for a giraffe and a racoon with different variants.
The racoon should have a clearer half-moon-shaped head.
The giraffe should have a thicker tongue.
The screenshot shows the folder structure and the variants that do get updated an the ones that don't.
Updating other objects like accessoires works however. It seems to be the body object only.

Minimal reproduction project (MRP)
prefab_bug_project.zip

Sounds related to:

https://github.com/godotengine/godot/issues/75970


Marten.AspNetCore 4.0.0 hasn't been published to NuGet

As the title states, it's missing. It seems like the GitHub Action for package deployment has been updated for the release, but it failed to upload Marten.AspNetCore due to a permissions issue. This is probably because package ownership of the main Marten package is jeremydmiller and mysticmind, while Marten.AspNetCore (seems to be an legacy project?) is owned by jokokko.
@jokokko

Package ownerships adjusted.

Thanks!


Allow writing InputRowParser extensions that use hadoop/any libraries

This PR is a result of discussion https://github.com/druid-io/druid/pull/1682#discussion-diff-38207513
in #1682 . This patch

now allows users to write custom InputRowParser for specific cases while hadoop ingestion and can depend upon hadoop or any other libs
reverts the special handling introduced for BytesWritable in  #1682 and handles it via a HadoopyStringInputRowParser

most of the changes are updates due to constructor arg changes in DataSchema. changes to note are in DataSchema.java, HadoopDruidIndexerConfig.java, IndexingHadoopModule.java and HadoopyStringInputRowParser.java

@gianm pls see this when you can, it is the continuation to discussion in  #1682
I see there are some merge conflicts now, will resolve them when review is done/makes-progresses.

@himanshug had a couple minor comments but overall looks good to me. I don't see a better way of having hadoopy parsers and still respecting the classpath situation…

Discussed this on the dev sync, @cheddar thought that in the future it would make sense for the hadoop input stuff to be loadable from extensions, similar to how the firehoses can be loadable from extensions.
:+1: on this pr for now though, since that's a longer term approach

@gianm addressed review comments and rebased against latest master.

@gianm / @himanshug the problem of lazy converting of Map<String, Object> --> ClassOfInterest is not unique to here. It is also present in LoadSpec as well as QTL (of which QTL is not fixed yet, routers require it in order to serde the query)

@himanshug does this change introduce any performance overheads?

@xvrl none observable . That said in theory the InputRowParser is created each time getParser() is called (see https://github.com/druid-io/druid/pull/1695#discussion_r38948854 ) , however that never happens in a tight loop so it does not matter.

@drcrallen this PR is pending response on
https://github.com/druid-io/druid/pull/1695#discussion_r39096015
https://github.com/druid-io/druid/pull/1695#discussion_r39096315
https://github.com/druid-io/druid/pull/1695#discussion_r39097331
pls see whenever you have time. thanks.

Thanks @himanshug, responded.

@drcrallen all review comments have been addressed.

:+1:


Mixlib's shellout expects streams to have a shift operator

In metal we use the EventsOutputStream and pass that to shellout. Shellout does the following:
@live_stdout << chunk if @live_stdout

here: https://github.com/opscode/mixlib-shellout/blob/master/lib/mixlib/shellout/unix.rb#L256 - where @live_stdout is an EventsOutputStream object, which then bombs because the object doesn't have a shift operator.
cc @jkeiser

Is this still an issue?  Should we just whack merge?  Or should we close this?  If you can answer that question please just do the needful.


fix: verify

Attempt to fix latest failure on main branch

:tada: This PR is included in version 0.3.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Prints properly formatted compile exceptions to console

Compile-time exceptions are now printed to the console, and focus is sent there as soon as one arrives. There may be a more elegant way to surface these, but this approach should work at least for the short term.

Fixes #911 .


check: update stacker check to verify newuidmap

stacker check command now verifies if the newuidmap binary exists in PATH and if the binary is executable with setuid bit on.
Fixes #324
Signed-off-by: Ravi Chamarthy<EMAIL_ADDRESS>

What do you think about have check emit suggested fixes?

It's a good idea, but I don't know if we can suggest the correct fix that will work consistently with the distro.

What do you think about have check emit suggested fixes?

It's a good idea, but I don't know if we can suggest the correct fix that will work consistently with the distro.

What do you think about have check emit suggested fixes?

It's a good idea, but I don't know if we can suggest the correct fix that will work consistently with the distro.

Agreed.  For ubuntu and fedora the fixes are quite different.  Well, setuid-root will always work, but...

Can we just detect the distro and suggest?
https://github.com/zcalusic/sysinfo, for example.

Can we just detect the distro and suggest? https://github.com/zcalusic/sysinfo, for example.

I think this is a good start, after this its googling the error :)


cpplint/IWYU: Add atomic, bitset, complex and initializer_list

This change is

To test this PR: https://github.com/RobotLocomotion/drake/pull/12768

Would you like me to push a GitHub action to run the unit test? Should be trivial.

Sounds great.  Whatever the easiest solution is.

The tests fail with python3.  I am fixing them.

FYI I'm happy (and probably prefer) dealing with CI + py3 porting in a new PR.  Once the Drake #include fixes land, we can merge this as-is.

Agreed.


StickyHeaderComponent not working as expected

Description
I don't think the StickyHeaderComponent is working as expected, but I may be misunderstanding the docs. The component is not docking at the top on scroll.
React Native Version
0.70.8
Output of npx react-native info
OS: macOS 14.0
CPU: (12) arm64 Apple M2 Max
Memory: 52.73 MB / 32.00 GB
Shell: 5.9 - /bin/zsh

Binaries:
Node: 18.15.0 - ~/.nvm/versions/node/v18.15.0/bin/node
Yarn: 1.22.19 - ~/.nvm/versions/node/v18.15.0/bin/yarn
npm: 9.5.0 - ~/.nvm/versions/node/v18.15.0/bin/npm
Watchman: 2<IP_ADDRESS> - /opt/homebrew/bin/watchman
Managers:
CocoaPods: 1.12.1 - /Users/XXX/.rbenv/shims/pod
SDKs:
iOS SDK:
Platforms: DriverKit 23.0, iOS 17.0, macOS 14.0, tvOS 17.0, watchOS 10.0
Android SDK: Not Found
IDEs:
Android Studio: 2022.2 AI-222.4459.24.2221.10121639
Xcode: 15.0/15A240d - /usr/bin/xcodebuild
Languages:
Java: 11.0.18 - /usr/bin/javac
npmPackages:
@react-native-community/cli: Not Found
react: 18.1.0 => 18.1.0
react-native: 0.70.8 => 0.70.8
react-native-macos: Not Found
npmGlobalPackages:
react-native: Not Found
Steps to reproduce
Create a very simple Flatlist with a ListHeaderComponent and a StickyHeaderComponent. By setting the stickyHeaderIndices=[1] I expected the StickyHeaderComponent to be under the ListHeaderComponent and then by setting stickyHeaderHiddenOnScroll={true} I thought that would mean it would dock at the top when scrolling up. The StickyHeaderComponent is in the second position but it does not dock when scrolling up.
    <FlatList<number>       data={[1, 2, 3, 4, 5, 6, 7, 8, 9, 10]}       renderItem={() => <View style={{ height: 100, borderWidth: 1 }} />}       ListHeaderComponent={() => (         <View style={{ height: 200, borderWidth: 3 }} />       )}       stickyHeaderIndices={[1]}       stickyHeaderHiddenOnScroll={true}       StickyHeaderComponent={() => (         <View style={{ height: 100, backgroundColor: 'red' }} />       )}     />
Snack, screenshot, or link to a repository
https://github.com/facebook/react-native/assets/29592293/021b0add-887c-4007-a81e-ccb4ad536e13
Many thanks in advance!

I've made something work by transforming the Y translation of the sticky header component myself, but again, not sure if this is how the component is supposed to work. Thanks

@LydGol90 I'm not really sure why it doesn't work, but you can use the section list to get the same result. Here you'll find a similar discussion


Set a initial meeting time

Working with @cruzluna to determine a new meeting cadence for re-kickoff.

@cruzluna and I decided on the first Monday of each month at 21:05 UTC (5:05 ET). We think a monthly cadence is manageable to start. :calendar: add this to your calendar


Enhancement: Automatically merge update of development dependency

This PR

[x] automatically merges an update of a development dependency

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@896cf05). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             master   #170   +/-   ##
=======================================
  Coverage          ?   100%
  Complexity        ?      3
=======================================
  Files             ?      1
  Lines             ?      6
  Branches          ?      0
=======================================
  Hits              ?      6
  Misses            ?      0
  Partials          ?      0

Continue to review full report at Codecov.

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


[Bug] 'Unsupported file extension' exception

The CreateNamespaceBuilderService method in NamespaceBuilderFactory throws an exception for files that it cannot fix the namespace for (e.g. .jpg/.png) but the ignored extensions aren't used until FixNamespace does a check to IgnoreFile(string filePath)
I think the CreateNamespaceBuilderService should return a default c# builder instead of throwing an exception or the ignored file list should be used there.

Hello! It has been updated, and it won't trigger the error anymore :)
On Wed, Jun 3, 2020 at 3:19 PM Joris<EMAIL_ADDRESS>wrote:

The CreateNamespaceBuilderService method in NamespaceBuilderFactory throws
an exception for files that it cannot fix the namespace for (e.g.
.jpg/.png) but the ignored extensions aren't used until FixNamespace does a
check to IgnoreFile(string filePath)
I think the CreateNamespaceBuilderService should return a default c#
builder instead of throwing an exception or the ignored file list should be
used there.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/pauer24/VsNamespaceFixer/issues/38, or unsubscribe
https://github.com/notifications/unsubscribe-auth/AGBANKGJPWU7EIRRGXLOLL3RUZEXJANCNFSM4NRVF3VQ
.

Nice! Works as expected. Thanks for your work!


Revert "Support @noescape SIL function types. (#12420)"

This reverts commit d369aa40705d14a3ddf3bbbc026f77632f52e756.
It is failing in ASAN tests.

@swift-ci please test asan

@swift-ci please asan test

@swift-ci please smoke test

Fixed in https://github.com/apple/swift/pull/12505.

Good thing the tests took so long. Thanks, Andy.


set-output command  deprecated

When I run your action, I get the following warning:
The set-output command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
detailed log: https://pastebin.com/m0ahAQg8

@mikepenz
Unfortunately, this issue was not fixed by the 0.8.1 release :/
Run<EMAIL_ADDRESS>📘 Reading input values
  No 'gradle-build-module' passed, using ':'
🔨 Processing gradle dependencies for root module - ':'
  Completed retrieving the 'dependencies' for configuration 'compileClasspath' within 9385ms
📄 Parsing gradle dependencies graph - ':'
  Dependency output of 2[18](https://github.com/cech12/BucketLib/actions/runs/3338939803/jobs/5527276926#step:6:21) lines
  Completed parsing 189 dependency associations within 2ms
📦 Preparing Dependency Snapshot - ':'
  Connection 3 direct dependencies
  Connection 90 indirect dependencies

Warning: The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Notice: Submitting snapshot...

Opened a PR here: https://github.com/github/dependency-submission-toolkit/pull/34


Add samtools faidx sequence retrieval

Fixes #58

@clami66 @LeandroRitter I added a samtools faidx step and modified the Breadth_Of_Coverage rule accordingly. To ensure the indexing step runs, Breadth_Of_Coverage has to include the index file as its input. For some reason, we have throughout the workflow never used the malt database as an input but rather as a parameter. I can't remember if we had any explicit reason to not use it as an input? Fear of accidentally deleting it through snakemake in some way?

LGTM!


Add jabber.otr.im Hidden Service

Add the mapping "jabber.otr.im" -> "5rgdtlawqkcplz75.onion" as documented by
their webpage at https://otr.im/chat.html
Addresses agl/xmpp-client#71 issue by @ioerror

LGTM!


Adds tearDown and renames SetUp functions to setUp.

This matches what is required by unity test runners.

Thanks for your contribution.


May 24th, 2021 Community Meeting

Please add a topic in this thread and add a link to the Github issue associated with the topic.
Please make sure you give folks enough time to review/discuss the topic offline on Github before coming into the meeting
(optional) Paste the image of an animal 😸

SHIP enhancement proposals: https://github.com/shipwright-io/community/blob/main/ships/README.md

Minutes:

SHIP enhancement proposals - community will be the new home. Existing proposals will remain in shipwright-io/build until their respective PRs are merged or closed. @adambkaplan will create a "first" SHIP that links back to the proposals in shipwright-io/build
Runtimes feature - there is consensus that it should be deprecated and removed. "Runtime image" is a specific need for the source-to-image tool, and is not needed for Dockerfile-based build strategies or build packs. Deprecation should be discussed via an enhancement proposal.
CI - @adambkaplan will update permissions so reviewers (as defined in the OWNERS file) have the ability to restart GitHub actions. Unfortunately prow does not have a command that lets us re-run a GitHub action.


Fix for Winrm Authentication Failure issue during bootstrap

Added local user account into the Administrators group
In-Progress

:+1:


Add SPFx based outlook add-in in Office Store

Category

[x] Question
[ ] Typo
[ ] Additional article idea

Question
We have created an outlook add-in application using SPFX. Now we want to put this application with SPPKG in the Microsoft office 365 store as an outlook add-in. We are stuck as we have not found a convenient method. Is there any standard way of  doing this?
Environment details (development & target environment)

Your Developer Environment: Windows 10
Target Environment: SharePoint Online / Outlook Web Access
Framework: Node.js v6/v8/v10/v12]
Browser(s): Any
Tooling: VS Code | SPFx v1.11.0
Additional details: We want to know how to deploy on Office Store

At the moment, using SPFx in Outlook add-ins is in developer preview mode and not presently supported in the app store.

At the moment, using SPFx in Outlook add-ins is in developer preview mode and not presently supported in the app store.


feat: 5200 - currency selector

What

Added a currency selector to user preference settings.
It was the opportunity to fix a right padding issue among user preference settings, while keeping a specific display for the welcome page.

Screenshots

preferences
welcome

Fixes bug(s)

Closes: #5200

Files
New files:

currency_selector.dart: A selector for selecting user's currency.
user_preferences_currency_selector.dart: Currency selector within user preferences.

Impacted files:

app_en.arb: added 2 labels for the currency selector
country_selector.dart: fixed discrepancies between selector display
user_preferences.dart: new "user currency code" getter/setter
user_preferences_settings.dart: added currency selector
welcome_page.dart: fixed discrepancies between selector display

Codecov Report
Attention: Patch coverage is 0.85470% with 116 lines in your changes are missing coverage. Please review.

Project coverage is 8.63%. Comparing base (4d9c7fc) to head (ca0b68b).
Report is 109 commits behind head on develop.

Files
Patch %
Lines

...th_app/lib/pages/onboarding/currency_selector.dart
0.00%
84 Missing :warning:

...references/user_preferences_currency_selector.dart
5.55%
17 Missing :warning:

.../lib/data_models/preferences/user_preferences.dart
0.00%
6 Missing :warning:

.../smooth_app/lib/pages/onboarding/welcome_page.dart
0.00%
6 Missing :warning:

...b/pages/preferences/user_preferences_settings.dart
0.00%
2 Missing :warning:

...oth_app/lib/pages/onboarding/country_selector.dart
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##           develop   #5236      +/-   ##
==========================================
- Coverage     9.54%   8.63%   -0.92%
==========================================
  Files          325     329       +4
  Lines        16411   16684     +273
==========================================
- Hits          1567    1441     -126
- Misses       14844   15243     +399

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

Thank you @teolemon for your review!

We should be more pedagogical about the language, the country and the currency because currently users have no idea what they possibly do

Definitely. Could be done partly in the onboarding.


fix: use includes for .ply search instead of endswith

My ply files have a token in the query parameters so I cannot search for a path that ends with .ply. I have to use includes.

I'd be happy to update the file loading functions to add more control over format specification. I think maybe instead of inspecting the path or file name and guessing the format, I'll add a parameter to Viewer.loadFile() and Viewer.loadFiles() to force it to expect a certain format. If that parameter is omitted, then it can fall back on guessing based on file extension.

@mkkellogg updated based on your comments

@jarodabel I will try to get these changes into the next release. I may tweak it a bit, but the same functionality will be there :)

I ended up reworking the code a bit so I'll close this PR, but in the latest release (v0.2.2) you can now pass a format parameter as part of the options object to addSplatScene(). The value of format needs to be one of the values defined in the SceneFormat enum.


NimbleComboboxListOptionDirective should use text, not textContent.

🐛 Bug Report

💻 Repro or Code Sample
As described in #1135, for clients to have the ability to create a ListOption variant that hosts more complex content, the NimbleComboboxListOptionDirective must alter its implementation to use the text property on the ListboxOption class, instead of bypassing it to use textContent instead. The text property ultimately just wraps the textContent (in FAST's implementation), but by going through the text property we allow overrides of text to provide the necessary textContent from some other node in the DOM.
🤔 Expected Behavior
Selecting items in the combobox that have complex visual content still work within forms.
😯 Current Behavior
If a combobox contains options with complex content, then the form bindings will not function as expected.
💁 Possible Solution
Solution prototype branch
🔦 Context
Trying to unblock a client request.

I just wasted time trying to figure out why my selected option was coming up "Symbol(not found)" only to realize my HTML was formatted like:
<nimble-list-option *ngFor="const item of items$ | async" [ngValue]="item">
    {{ item?.name }}
</nimble-list-option>

vs.
<nimble-list-option *ngFor="const item of items$ | async" [ngValue]="item">{{
    item?.name
}}</nimble-list-option>

This caused my optionMap keys to have beginning/trailing spaces, so no options were being matched. I think using text vs textContent fixes this, but until then, we should trim textContent.

I think this is easy to trip up on and should be addressed sooner independently of complex content support. Marking for triage.


[Proposal] Performance/SelectFirst

We have built this cop internally, for identifying uses of .select {}.first or .select {}.count that could be written in a more performant way. My question is, would rubocop accept a PR to add this to the main gem (if I added tests etc) or is it too niche?
module RuboCop
  module Cop
    module TandaCustomCops
      # Don't call `.find_all { }.first`. Instead just use `.find { }`
      # The bad form iterates over every item in an array and runs the block on each.
      # The good form returns as soon as a match is found.
      class SelectFirst < Cop
        def_node_matcher :find_all_first_candidate?, <<-PATTERN
          (send (block $(send _ ${:find_all :select}) ...) ${:first} $...)
        PATTERN
        def_node_matcher :find_all_count_candidate?, <<-PATTERN
          (send (block $(send _ ${:find_all :select}) ...) ${:count} $...)
        PATTERN

        MSG_FIND = <<~MSG.freeze
          `.select { }.first` can be written as `.find { }`. This is faster because it returns as soon as a match is found.`
        MSG

        MSG_COUNT = <<~MSG.freeze
          `.select { }.count` can be written as `.count { }`. This is marginally faster.`
        MSG

        def on_send(node)
          add_offense(node, message: MSG_FIND) if find_all_first_candidate?(node)
          add_offense(node, message: MSG_COUNT) if find_all_count_candidate?(node)
        end
      end
    end
  end
end

Can you show a benchmark compared to alternative methods?

require 'benchmark/ips'

array = 100_000.times.map { rand > 0.1 ? 0 : 1 } # 10% of the array should be 1, 90% should be 0

Benchmark.ips do |x|
  x.report("slow") { array.select(&:nonzero?).first }
  x.report("fast") { array.find(&:nonzero?) }

  x.compare!
end

=begin
Warming up --------------------------------------
                slow     3.000  i/100ms
                fast    44.899k i/100ms
Calculating -------------------------------------
                slow     35.648  (±11.2%) i/s -    177.000
                fast    579.702k (±14.2%) i/s -      2.829M

Comparison:
                fast:   579701.7 i/s
                slow:       35.6 i/s - 16262.05x slower
=end

This one is less exciting, it's just neater code IMO:
require 'benchmark/ips'

array = 100_000.times.map { rand > 0.1 ? 0 : 1 } # 10% of the array should be 1, 90% should be 0

Benchmark.ips do |x|
  x.report("slow") { array.select(&:nonzero?).count }
  x.report("fast") { array.count(&:nonzero?) }

  x.compare!
end

=begin
Warming up --------------------------------------
                slow     3.000  i/100ms
                fast     3.000  i/100ms
Calculating -------------------------------------
                slow     37.756  (± 2.6%) i/s -    189.000
                fast     35.935  (±11.1%) i/s -    180.000

Comparison:
                slow:       37.8 i/s
                fast:       35.9 i/s - same-ish: difference falls within error
=end

Thank you for benchmarking with alternative methods. I agree to add this cop to performance cops.
I will convey some of points considered.

This proposed cop name Performance/SelectFirst looks good to me.
filter method has been added in Ruby 2.6. So filter method is also targeted when Ruby 2.6 or higher. https://ruby-doc.org/core-2.6.3/Array.html#method-i-filter
It would be preferable for this cop to replace with find method by auto-correct. Please see "Why did I choose this fix out of the possible options?" at the bottom.
Be careful not to cause false positives in Active Record's select method. For example, Model.select(fields).first should not be warned. It should not be warned when an argument is not a block. https://api.rubyonrails.org/classes/ActiveRecord/QueryMethods.html#method-i-select
Same for count. Please  replace select { ... }.count only if an argument is a block. Otherwise, a SQL changes.
https://github.com/JuanitoFatas/fast-ruby#enumerabledetect-vs-enumerableselectfirst-code would be useful as a reference.

Why did I choose this fix out of the possible options?
I think that coding context will be maintained by replacing find_all { ... }.first with find method and replacing select { ... }.first to detect method respectively. However, it will not be possible to replace it, as it will cause an error in an Active Record model when select { ... }.first method will be auto-corrected detect method. The following are examples.
# Use `select { ... }.first`
Model.select { |model| model.attr == value }.first #=> Returns an AR model object

# Use `find { ... }`
Model.find { |model| model.attr == value } #=> Returns an AR model object

# Use `delect { ... }`
Model.delect { |model| model.attr == value } #=> NoMethodError
NoMethodError: undefined method `delect' for #<Class:0x007f81bd065ae0>
Did you mean?  delete
               select

So I considered that it should be replaced by find method.
It means that performance is prioritized over representation by API name when using this cop. Because this cop is a performance cops.

It may be possible to further consider whether it is okay to include select { ... }.first and select { ... }.count in the same cop. Because it is named SelectFirst cop. It looked good to me, but there may be other appropriate cop name.
Cc @rubocop-hq/rubocop-core

it will cause an error in an Active Record model

I think that was caused by a typo: delect != detect.

Yeah, detect is correct, not delect. This is my typo.

filter method has been added in Ruby 2.6. So filter method is also targeted when Ruby 2.6 or higher. ruby-doc.org/core-2.6.3/Array.html#method-i-filter

Is there a recommended approach to having rules only apply to specific ruby versions?

Please use target_ruby_version. The following production code and spec code will be helpful.

https://github.com/rubocop-hq/rubocop/blob/v0.72.0/lib/rubocop/cop/layout/rescue_ensure_alignment.rb#L131
https://github.com/rubocop-hq/rubocop/blob/v0.72.0/spec/rubocop/cop/layout/rescue_ensure_alignment_spec.rb#L301

I think Performance/Detect and Performance/Count cover most if not all of this functionality. As mentioned in #69, I don't think these cops were being run for most people. There are likely some enhancements that can be made to each of the cops.

@ghiculescu Have you started work on this?

Ahh, no I haven't. @fatkodima if you're interested please go ahead, otherwise I will try and get to it soon.


Why aren't you in Atlanta?

You are adorable and we could hang out if you were in Atlanta.
It would be so cool! I could make you drinks and you could show me what hives look like!
😍  🍸  🎵  🎉  💯

In addition... I'd love to learn how this works @jamesdabbs @kingcons ^_^ first look through the files / first Haskell project I've really read into, and trying to figure out where to start :)

Heh. app/Main.hs? 😉
But for real though - have you looked at any Haskell yet? LYAH is where I started, but I hear great things about The Haskell Book. If you're starting-to-sort-of-be-okay-with-the-syntax, I'd be happy to walk you through the architecture here sometime. @matthiasak

@jamesdabbs
Completed LYAH, now reading into stack and yesod, perusing blogs for wisdom, and doing exercism problems; "starting-to-sort-of-be-okay-with-the-syntax" mode achieved. :)


Synthesis failed for java-analytics-admin

Hello! Autosynth couldn't regenerate java-analytics-admin. :broken_heart:
Here's the output from running synth.py:
ee the License for the specific language governing permissions and limitations under\n \\* the License.\n \\*/\n' in /tmpfs/tmp/tmpteip8jzv/google-analytics-admin-v1alpha-java/proto-google-analytics-admin-v1alpha-java/src/main/java/com/google/analytics/admin/v1alpha/AndroidAppDataStreamName.java.
2020-08-06 01:50:43,499 synthtool [INFO] > Replaced '/\\*\n \\* Copyright \\d{4} Google LLC\n \\*\n \\* Licensed under the Apache License, Version 2.0 \\(the "License"\\); you may not use this file except\n \\* in compliance with the License. You may obtain a copy of the License at\n \\*\n \\* http://www.apache.org/licenses/LICENSE-2.0\n \\*\n \\* Unless required by applicable law or agreed to in writing, software distributed under the License\n \\* is distributed on an "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express\n \\* or implied. See the License for the specific language governing permissions and limitations under\n \\* the License.\n \\*/\n' in /tmpfs/tmp/tmpteip8jzv/google-analytics-admin-v1alpha-java/proto-google-analytics-admin-v1alpha-java/src/main/java/com/google/analytics/admin/v1alpha/AccountName.java.
2020-08-06 01:50:43,505 synthtool [INFO] > Replaced '^package (.*);' in /tmpfs/tmp/tmpteip8jzv/google-analytics-admin-v1alpha-java/grpc-google-analytics-admin-v1alpha-java/src/main/java/com/google/analytics/admin/v1alpha/AnalyticsAdminServiceGrpc.java.
2020-08-06 01:50:43,715 synthtool [INFO] > Running java formatter on 10 files
2020-08-06 01:50:47,610 synthtool [INFO] > Running java formatter on 1 files
2020-08-06 01:50:51,086 synthtool [INFO] > Running java formatter on 159 files
2020-08-06 01:51:01,615 synthtool [DEBUG] > Using precloned repo /home/kbuilder/.cache/synthtool/synthtool
.github/CODEOWNERS
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.github/PULL_REQUEST_TEMPLATE.md
.github/release-please.yml
.github/trusted-contribution.yml
.github/workflows/ci.yaml
.kokoro/build.bat
.kokoro/build.sh
.kokoro/coerce_logs.sh
.kokoro/common.cfg
.kokoro/common.sh
.kokoro/continuous/common.cfg
.kokoro/continuous/dependencies.cfg
.kokoro/continuous/integration.cfg
.kokoro/continuous/java11.cfg
.kokoro/continuous/java7.cfg
.kokoro/continuous/java8-osx.cfg
.kokoro/continuous/java8-win.cfg
.kokoro/continuous/java8.cfg
.kokoro/continuous/lint.cfg
.kokoro/continuous/propose_release.cfg
.kokoro/continuous/samples.cfg
.kokoro/dependencies.sh
.kokoro/linkage-monitor.sh
.kokoro/nightly/common.cfg
.kokoro/nightly/dependencies.cfg
.kokoro/nightly/integration.cfg
.kokoro/nightly/java11.cfg
.kokoro/nightly/java7.cfg
.kokoro/nightly/java8-osx.cfg
.kokoro/nightly/java8-win.cfg
.kokoro/nightly/java8.cfg
.kokoro/nightly/lint.cfg
.kokoro/nightly/samples.cfg
.kokoro/populate-secrets.sh
.kokoro/presubmit/clirr.cfg
.kokoro/presubmit/common.cfg
.kokoro/presubmit/dependencies.cfg
.kokoro/presubmit/integration.cfg
.kokoro/presubmit/java11.cfg
.kokoro/presubmit/java7.cfg
.kokoro/presubmit/java8-osx.cfg
.kokoro/presubmit/java8-win.cfg
.kokoro/presubmit/java8.cfg
.kokoro/presubmit/linkage-monitor.cfg
.kokoro/presubmit/lint.cfg
.kokoro/presubmit/samples.cfg
.kokoro/release/bump_snapshot.cfg
.kokoro/release/common.cfg
.kokoro/release/common.sh
.kokoro/release/drop.cfg
.kokoro/release/drop.sh
.kokoro/release/promote.cfg
.kokoro/release/promote.sh
.kokoro/release/publish_javadoc.cfg
.kokoro/release/publish_javadoc.sh
.kokoro/release/snapshot.cfg
.kokoro/release/snapshot.sh
.kokoro/release/stage.cfg
.kokoro/release/stage.sh
.kokoro/trampoline.sh
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
codecov.yaml
java.header
license-checks.xml
renovate.json
samples/install-without-bom/pom.xml
samples/pom.xml
samples/snapshot/pom.xml
samples/snippets/pom.xml
2020-08-06 01:51:01,774 synthtool [DEBUG] > existing pom file found (samples/pom.xml) - keeping the existing
2020-08-06 01:51:01,775 synthtool [DEBUG] > existing pom file found (samples/snippets/pom.xml) - keeping the existing
2020-08-06 01:51:01,775 synthtool [DEBUG] > existing pom file found (samples/install-without-bom/pom.xml) - keeping the existing
2020-08-06 01:51:01,776 synthtool [DEBUG] > existing pom file found (samples/snapshot/pom.xml) - keeping the existing
2020-08-06 01:51:04,829 synthtool [DEBUG] > Wrote metadata to synth.metadata.
2020-08-06 01:51:04,910 autosynth [INFO] > Changed files:
2020-08-06 01:51:04,910 autosynth [INFO] > M synth.metadata
2020-08-06 01:51:04,910 autosynth [DEBUG] > Running: git log 8f64979cee9b4857650fea6c41543995aab06371 -1 --no-decorate --pretty=%s
2020-08-06 01:51:04,915 autosynth [DEBUG] > Running: git log 8f64979cee9b4857650fea6c41543995aab06371 -1 --no-decorate --pretty=%b%n%nSource-Author: %an <%ae>%nSource-Date: %ad
2020-08-06 01:51:04,919 autosynth [DEBUG] > Running: git add -A
2020-08-06 01:51:04,926 autosynth [DEBUG] > Running: git status --porcelain
2020-08-06 01:51:04,932 autosynth [DEBUG] > Running: git commit -m docs: recommend installation via pip -e

Closes #691

Co-authored-by: Jeffrey Rennie<EMAIL_ADDRESS>
Source-Author: Bu Sun Kim<EMAIL_ADDRESS>Source-Date: Wed Aug 5 15:23:24 2020 -0700
Source-Repo: googleapis/synthtool
Source-Sha: 8f64979cee9b4857650fea6c41543995aab06371
Source-Link: https://github.com/googleapis/synthtool/commit/8f64979cee9b4857650fea6c41543995aab06371
[autosynth-81 ea059e0] docs: recommend installation via pip -e
 1 file changed, 4 insertions(+), 4 deletions(-)
2020-08-06 01:51:04,941 autosynth [DEBUG] > Running: git reset --hard HEAD
HEAD is now at ea059e0 docs: recommend installation via pip -e
2020-08-06 01:51:04,948 autosynth [DEBUG] > Running: git checkout autosynth
Switched to branch 'autosynth'
2020-08-06 01:51:04,954 autosynth [DEBUG] > Running: git diff HEAD..autosynth-81 -- . :(exclude)synth.metadata
2020-08-06 01:51:04,958 autosynth [DEBUG] > Running: git diff HEAD..autosynth-81 -- synth.metadata
2020-08-06 01:51:04,963 autosynth [DEBUG] > Running: git diff HEAD..autosynth-82 -- . :(exclude)synth.metadata
2020-08-06 01:51:04,968 autosynth [DEBUG] > Running: git diff HEAD autosynth-82
2020-08-06 01:51:04,973 autosynth [DEBUG] > Running: git apply /tmpfs/tmp/tmpg6ei1fq9/autosynth-82.patch
2020-08-06 01:51:04,977 autosynth [DEBUG] > Running: git add -A
2020-08-06 01:51:04,983 autosynth [DEBUG] > Running: git status --porcelain
2020-08-06 01:51:04,990 autosynth [DEBUG] > Running: git commit -m cleanup: removes unused kokoro config files

* cleanup: removes unused kokoro config files

Removes unused kokoro files from the java library template. We have
stopped running some of these due to Github quota issues.

* fix:reverts back samples.cfg files

The files presubmit/samples.cfg and nightly/samples.cfg should remain in
the java template repository.

Co-authored-by: Jeffrey Rennie<EMAIL_ADDRESS>
Source-Author: Thiago Nunes<EMAIL_ADDRESS>Source-Date: Thu Aug 6 09:48:58 2020 +1000
Source-Repo: googleapis/synthtool
Source-Sha: 4530cc6ff080ef8aca258c1ec92c4db10a1bbfb4
Source-Link: https://github.com/googleapis/synthtool/commit/4530cc6ff080ef8aca258c1ec92c4db10a1bbfb4
[autosynth f9eec35] cleanup: removes unused kokoro config files
 12 files changed, 4 insertions(+), 176 deletions(-)
 delete mode 100644 .kokoro/continuous/dependencies.cfg
 delete mode 100644 .kokoro/continuous/integration.cfg
 delete mode 100644 .kokoro/continuous/java11.cfg
 delete mode 100644 .kokoro/continuous/java7.cfg
 delete mode 100644 .kokoro/continuous/java8-osx.cfg
 delete mode 100644 .kokoro/continuous/java8-win.cfg
 delete mode 100644 .kokoro/continuous/lint.cfg
 delete mode 100644 .kokoro/continuous/propose_release.cfg
 delete mode 100644 .kokoro/continuous/samples.cfg
 delete mode 100644 .kokoro/nightly/dependencies.cfg
 delete mode 100644 .kokoro/nightly/lint.cfg
2020-08-06 01:51:04,999 autosynth [DEBUG] > Running: git log -1 --no-decorate --format=%B
2020-08-06 01:51:05,003 autosynth [DEBUG] > Running: git push --force origin autosynth
remote:
remote: Create a pull request for 'autosynth' on GitHub by visiting:
remote:      https://github.com/googleapis/java-analytics-admin/pull/new/autosynth
remote:
To https://github.com/googleapis/java-analytics-admin.git
 * [new branch]      autosynth -> autosynth
2020-08-06 01:51:08,688 autosynth [DEBUG] > Running: git log -1 --pretty=%b
2020-08-06 01:51:11,395 autosynth [DEBUG] > Running: git clean -fdx
Removing __pycache__/
Traceback (most recent call last):
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 193, in _run_module_as_main
    "__main__", mod_spec)
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/runpy.py", line 85, in _run_code
    exec(code, run_globals)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 690, in <module>
    main()
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 539, in main
    return _inner_main(temp_dir)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 670, in _inner_main
    commit_count = synthesize_loop(x, multiple_prs, change_pusher, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 400, in synthesize_loop
    synthesize_inner_loop(fork, synthesizer)
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 444, in synthesize_inner_loop
    synthesizer, len(toolbox.versions) - 1
  File "/tmpfs/src/github/synthtool/autosynth/synth.py", line 273, in synthesize_version_in_new_branch
    synthesizer.synthesize(synth_log_path, self.environ)
  File "/tmpfs/src/github/synthtool/autosynth/synthesizer.py", line 120, in synthesize
    synth_proc.check_returncode()  # Raise an exception.
  File "/home/kbuilder/.pyenv/versions/3.6.9/lib/python3.6/subprocess.py", line 389, in check_returncode
    self.stderr)
subprocess.CalledProcessError: Command '['/tmpfs/src/github/synthtool/env/bin/python3', '-m', 'synthtool', '--metadata', 'synth.metadata', 'synth.py', '--']' returned non-zero exit status 1.

Google internal developers can see the full log here.

Autosynth passed, closing! :green_heart:


chore(prow/cluster): add ingress to prow.prow.net

What problem does this PR solve?
add ingress to prow.tidb.net
What is changed and how it works?
What's Changed: add a k8s ingress resource prow-net

/merge


Controller mode config (BTDM/BLE) question in newer idf version (IDFGH-1413)

Environment
Development Kit: ESP32-LyraT
Module or chip used: ESP32-WROOM-32
IDF version (run git describe --tags to find it): v4.0-dev-837-g58df1d93b
Build System: Make
Compiler version (run xtensa-esp32-elf-gcc --version to find it): xtensa-esp32-elf-gcc (crosstool-NG esp32-2019r1) 8.2.0
Operating System: Linux
Power Supply: USB
Problem Description
It seems like CONFIG_BTDM_CONTROLLER_MODE_BTDM
& CONFIG_BTDM_CONTROLLER_MODE_BLE_ONLY  are deprecated.
What is right config to select the mode now ?

Hi @kvbulusu. Here are the new options: https://docs.espressif.com/projects/esp-idf/en/latest/api-reference/kconfig.html#config-btdm-ctrl-mode (CONFIG_BTDM_CTRL_MODE_BTDM and CONFIG_BTDM_CTRL_MODE_BLE_ONLY).

Thanks @dobairoland


Update Kafka utils to use images from Maven properties

Summary
KafkaRegistry and KafkaVendor now using image defined in Maven properties. It helps track versions from one place.
Please check the relevant options

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[x] Dependency update
[x] Refactoring
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] This change requires a documentation update
[x] This change requires execution against OCP (use run tests phrase in comment)

Checklist:

[ ] Example scenarios has been updated / added
[x] Methods and classes used in PR scenarios are meaningful
[x] Commits are well encapsulated and follow the best practices

run tests

run tests


Exception parsing negative integer strings

Initial Checks

[X] I confirm that I'm using Pydantic V2

Description
Starting with pydantic 2.7.1 parsing an integer string like '-1.0' gives an error while '1.0' works fine. Probably related to pydantic/pydantic-core#1269 or pydantic/pydantic-core#1272.
Example Code
In [1]: import pydantic

In [2]: class A(pydantic.BaseModel):
   ...:     i: int
   ...:

In [3]: A(i='4.0')
Out[3]: A(i=4)

In [4]: A(i='-4.0')
---------------------------------------------------------------------------
ValidationError                           Traceback (most recent call last)
Cell In[4], line 1
----> 1 A(i='-4.0')

File ~/.cache/pypoetry/virtualenvs/hl-7m4wOqun-py3.12/lib/python3.12/site-packages/pydantic/main.py:176, in BaseModel.__init__(self, **data)
    174 # `__tracebackhide__` tells pytest and some other tools to omit this function from tracebacks
    175 __tracebackhide__ = True
--> 176 self.__pydantic_validator__.validate_python(data, self_instance=self)

ValidationError: 1 validation error for A
i
  Input should be a valid integer, unable to parse string as an integer [type=int_parsing, input_value='-4.0', input_type=str]
    For further information visit https://errors.pydantic.dev/2.7/v/int_parsing

Python, Pydantic & OS Version
             pydantic version: 2.7.1
        pydantic-core version: 2.18.2
          pydantic-core build: profile=release pgo=true
                 install path: /home/noam/.cache/pypoetry/virtualenvs/hl-7m4wOqun-py3.12/lib/python3.12/site-packages/pydantic
               python version: 3.12.3 (main, Apr 10 2024, 05:33:47) [GCC 13.2.0]
                     platform: Linux-<IP_ADDRESS>-microsoft-standard-WSL2-x86_64-with-glibc2.39
             related packages: typing_extensions-4.10.0 mypy-1.7.1
                       commit: unknown

I investigated this and the issue seems to be in strip_leading_zeros function where if any character occurs except (+, _, 1 ... 9) in the string then None is being returned. The - character is not handled here and  due to this it's throwing validation error.
https://github.com/pydantic/pydantic-core/blob/main/src/input/shared.rs#L149-L168

Hey folks,
Thanks for reporting this bug. Definitely something we want to fix in 2.7.2. Should be an easy fix if anyone is interested in contributing to pydantic-core. I'll pick this up soon if not!

I'll pick it up.


chevrons don't appear in iOS 5.0 simulator when loaded from localhost

If I load the demos/main/index.html example from jqtouch-1.0-b4-rc, all the chevrons are messed up if I load them like this in the iOS 5.0 simulator:
http://<IP_ADDRESS>/~pho0/test/jqtouch/demos/main/index.html
but it works if I load it like a local file:
file///Users/pho0/Sites/test/jqtouch/demos/main/index.html
I can provide you with screenshots if you need them, email me at<EMAIL_ADDRESS>if you want them.

I'm able to reproduce this in my project as well.

Sorry, can't see your internal network link. Can you put it up publicly?
Thanks,
Dave
On Jan 27, 2012, at 4:41 PM, pho0 wrote:

If I load the demos/main/index.html example, all the chevrons are messed up if I load them like this in the iOS 5.0 simulator:
http://127.0.01/~pho0/test/jqtouch/demos/main/index.html
but it works if I load it like a local file:
file///Users/pho0/Sites/test/jqtouch/demos/main/index.html
I can provide you with screenshots if you need them, email me at<EMAIL_ADDRESS>if you want them.

Reply to this email directly or view it on GitHub:
https://github.com/senchalabs/jQTouch/issues/371

Nice, I can't repro. I think it was a cached image issue with my browser.


Feedback for: machine/drivers/vsphere.md

I've tried to create a docker-machine on my local ESXi server with the following command:
docker-machine create --driver vmwarevsphere --vmwarevsphere-username=root --vmwarevsphere-password=secret --vmwarevsphere-vcenter=<IP_ADDRESS> vm
All went OK and my log was:
Running pre-create checks... Creating machine... (vm) Copying /Users/michal/.docker/machine/cache/boot2docker.iso to /Users/michal/.docker/machine/machines/vm/boot2docker.iso... (vm) Generating SSH Keypair... (vm) Creating VM... (vm) Uploading Boot2docker ISO ... (vm) Reconfiguring VM... (vm) Waiting for VMware Tools to come online... (vm) Provisioning certs and ssh keys... Waiting for machine to be running, this may take a few minutes... Detecting operating system of created instance... Waiting for SSH to be available... Detecting the provisioner... Provisioning with boot2docker... Copying certs to the local machine directory... Copying certs to the remote machine... Setting Docker configuration on the remote daemon... Checking connection to Docker... Docker is up and running! To see how to connect your Docker Client to the Docker Engine running on this virtual machine, run: docker-machine env vm
However, when I run docker-machine env vm then I get the following error:
Error checking TLS connection: Error checking and/or regenerating the certs: There was an error validating certificates for host "<IP_ADDRESS>:2376": dial tcp <IP_ADDRESS>:2376: i/o timeout
and nothing works. On docker-machine ls the state of the machine is Timeout. What is wrong and how can I fix this?
Remark: when I look to config.json and ssh SSHUser@IPAddress then I can login into vm. But I cannot use docker client with it.

Updating docker toolbox to docker-machine 0.8.0 build b85aac1 solved the issue.


auto run local karma installation if local karma not found

This automatically runs npm install when karma fails to run because it's not locally installed.


Razor Components do not include embedded resources from assemblies

Just upgraded a project to Razor Components from Blazor 0.7
Unfortunately, it depends on CSS/JS-Files located in a separate assembly.
With Blazor, after declaration of these CSS/JS-Files as embedded resource with a certain name (blazor:js:** and blazor:css:**), they were packed and referenced automagically by blazor.
After migraton to Razor Components, this functionality seems to be not available any more (at least, the resources are not shipped to the browser any more)
What must be done to make inclusion of JS/CSS from separate assemblies work again?
Thanks!

Duplicate of https://github.com/aspnet/AspNetCore/issues/6349

Closing as this is indeed a duplicate.