help(google-maps/polygon): googlemaps-polygon draw live

Ive tried to update the LatLngLiteral[] for the polygon draw in the google map, it only loads preloaded value,
need help to update the polygon selection to be updates after each select, followed the document, but still it only loads after page refresh.

Closing this issue as it does not follow the issue template. Please file a new issue that follows the template, which gives the team the information needed to investigate.


NameError during Runtime#require raises unexpected error

$ bundle -v
Bundler version 1.10.6

$ bundle console
--- ERROR REPORT TEMPLATE -------------------------------------------------------
- What did you do?

  I ran the command `/Users/andre/.gem/ruby/2.2.2/bin/bundle console`

- What did you expect to happen?

  I expected Bundler to...

- What happened instead?

  Instead, what actually happened was...

Error details

    NameError: uninitialized constant Postgres
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler/runtime.rb:76:in `const_get'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler/runtime.rb:76:in `require'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler/runtime.rb:76:in `block (2 levels) in require'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler/runtime.rb:72:in `each'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler/runtime.rb:72:in `block in require'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler/runtime.rb:61:in `each'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler/runtime.rb:61:in `require'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler.rb:134:in `require'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler/cli/console.rb:10:in `run'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler/cli.rb:308:in `console'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler/vendor/thor/lib/thor/invocation.rb:126:in `invoke_command'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler/vendor/thor/lib/thor.rb:359:in `dispatch'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler/vendor/thor/lib/thor/base.rb:440:in `start'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler/cli.rb:10:in `start'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/bin/bundle:20:in `block in <top (required)>'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/lib/bundler/friendly_errors.rb:7:in `with_friendly_errors'
      /Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/bin/bundle:18:in `<top (required)>'
      /Users/andre/.gem/ruby/2.2.2/bin/bundle:23:in `load'
      /Users/andre/.gem/ruby/2.2.2/bin/bundle:23:in `<main>'

Environment

    Bundler   1.10.6
    Rubygems  2.4.5
    Ruby      2.2.2p95 (2015-04-13 revision 50295) [x86_64-darwin14]
    GEM_HOME  /Users/andre/src/bundler/bundler-api/.bundle/ruby/2.2.0
    GEM_PATH
    Git       2.3.0

Bundler settings

    path
      Set for your local app (/Users/andre/src/bundler/bundler-api/.bundle/config): ".bundle"
    disable_shared_gems
      Set for your local app (/Users/andre/src/bundler/bundler-api/.bundle/config): "1"
    jobs
      Set via BUNDLE_JOBS: "8"
    bin_path
      Set via BUNDLE_BIN_PATH: "/Users/andre/.gem/ruby/2.2.2/gems/bundler-1.10.6/bin/bundle"
    gemfile
      Set via BUNDLE_GEMFILE: "/Users/andre/src/bundler/bundler-api/Gemfile"
    orig_manpath
      Set via BUNDLE_ORIG_MANPATH: "/usr/local/share/man:/usr/share/man:/usr/local/share/man:/opt/X11/share/man:/usr/local/MacGPG2/share/man"
--- TEMPLATE END ----------------------------------------------------------------

Unfortunately, an unexpected error occurred, and Bundler cannot continue.

First, try this link to see if there are any existing issue reports for this error:
https://github.com/bundler/bundler/search?q=uninitialized+constant+Postgres&type=Issues

If there aren't any reports for this error yet, please create copy and paste the report template above into a new issue. Don't forget to anonymize any private data! The new issue form is located at:
https://github.com/bundler/bundler/issues/new

We should handle LoadErrors during Bundler.require with a warning at the most, and probably no printed message?

@indirect gemfile?

Bundler-Api repo, new index branch

All we do there is call require -- the const_get isn't ours

We shouldn't be showing the unexpected error template regardless, though


Extend depiction params to reference images params

We need to extend depiction params to get depictions by image

@mjy status please? If not done, does this need prioritization?

Cross reference to #1633.


[1.20.4] [Neoforge] Were Ritual recipes removed from JEI screen?

Describe the bug
I can't see the Ritual recipe on the JEI/EMI menu
To Reproduce
Steps to reproduce the behavior:

Left Click on a Ritual (Crushing ritual)

Expected behavior
When L-clicking on a Ritual icon on the JEI menu, it should open the menu to the bowl items and output screen.
Like in 1.20.1 version
Screenshots

System (please complete the following information):

Occultism Version: occultism-1.20.4-neoforge-1.125.0
OS: Windows
Minecraft Version: 1.20.4
Modpack Link and Version, or list of mods:
jei-1.20.4-neoforge-<IP_ADDRESS>
emi-1.1.4+1.20.4+neoforge
ImmersiveEngineering-1.20.4-11.1.0-172

Additional context
I recall that this used to work on 1.20.1

@GaeaKat can you comment on the "happy path" of how that should work with EMI present?
Without EMI jei should show the rituals still (but haven't tested, may be bugged!)

@GaeaKat can you comment on the "happy path" of how that should work with EMI present?
Without EMI jei should show the rituals still (but haven't tested, may be bugged!)

I think I see the difference between EMI and JEI here, and I am currently working on fixing it

@GaeaKat can you comment on the "happy path" of how that should work with EMI present?
Without EMI jei should show the rituals still (but haven't tested, may be bugged!)
Updated and did not change anything.
Well, is more that I can find the rituals on EMI, but the Ritual icons on the item list does not send me to the ritual page.
Video: https://github.com/klikli-dev/occultism/assets/44974827/7dbc3a38-6aeb-4c90-ba24-023a4bb6386f


PW-2359: new paypal in SG

Description
Added paypal smart buttons component
Tested scenarios
Making paypal payments and also Cancelling halfway
Fixed issue:

@zaiddreakh check LGTM


Update hdinsight-hadoop-use-data-lake-storage-gen2.md

Updated the note around RA-GRS and RA-ZRS secondary endpoints

@ashishthaps1 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

#sign-off


Reconsider SFENCE.VM/ASIDs

Presently, SFENCE.VM takes its ASID argument from the sptbr.  Since any PTE the sptbr points to can be speculatively loaded into the TLB, you don't want to just modify sptbr.asid arbitrarily.  So, in an OS like Linux, you might flush entry X in ASID Y as follows: (recall ASID 0 is the global ASID, and assume the kernel maintains a page table that consists only of its global mappings)
tmp <- sptbr
sptbr <- asid = Y, ppn = global page table
sfence.vm X
sptbr <- asid = tmp.asid, ppn = global page table
sptbr <- tmp

It's not ideal for simple implementations that always flush the TLB on sptbr writes, and it imposes the (minor) constraint that this code sequence is mapped into all address spaces.  And ASID 0 being special is an annoying quirk.
Here is an alternative proposal:

ASID 0 is no longer special.
SFENCE.VM takes two arguments: rs1=addr, rs2=asid.
When rs1=x0 and rs2=x0, it orders prior stores with all subsequent translations.
When rs1=x0 and rs2!=x0, it orders prior stores to non-Global PTEs with subsequent translations for the specified ASID.
When rs1!=x0 and rs2=x0, it orders prior stores to PTEs corresponding to virtual address rs1 with all subsequent translations.
When rs1!=x0 and rs2!=x0, it orders prior stores to non-Global PTEs corresponding to virtual address rs1 with all subsequent translations for the specified ASID.

This requires us to use funct3=4 in the SYSTEM opcode, since funct3=0 has opcode bits within rs2.

Merged into spec.

0b8a8e0c2a4385005e2ef6d92f3b4feee822fd25

in sbi.tex a small correction:
line 57 SFENCE.VM to SFENCE.VMA
& \multirow{4}{*}{\parbox{8cm}{Instruct other harts to execute SFENCE.VM. {\tt harts} points to a bitmask of remote hart IDs; NULL indicates all harts. {\tt asid} holds the address-space ID; 0 indicates all address spaces.}} \
& \multirow{4}{*}{\parbox{8cm}{Instruct other harts to execute SFENCE.VMA. {\tt harts} points to a bitmask of remote hart IDs; NULL indicates all harts. {\tt asid} holds the address-space ID; 0 indicates all address spaces.}} \
I found a place to note this - so you can disregard the email.

Thanks.


Update My-PR-Contribution-Practice.md

I have updated the my details in the alphabetical order.
Thank you ^_^.


Closes #137 - Added Simple Article Header

Closes #137 - Added Simple Article Header
This also includes the posts and comments section
Poke @canterberrie @rafibomb for review!
Quick Codepen => http://codepen.io/IamManchanda/full/OmNwWr/

This is a good looking building block! Left some feedback. Also, we'll need a preview image that has more padding around it so that it shows up on the site like this (not cut off on the edges!):

@canterberrie i have updated the padding .... also updated preview image with some editing. Is this good now!

@canterberrie @rafibomb I think the comments section should be there and that's why i also added smooth scroll but will keep the decision on you guys!

@IamManchanda Everything looks good to me. Talked to @rafibomb about it and we think it's worth keeping the comment section in here if it looks different (right now it looks too similar to an existing BB). It should still be kept pretty simple, but here are some suggestions!
Card based comments (credits: https://dribbble.com/shots/2069235-Feeds-Section-Card-Style)

Comments with actions (credits: https://dribbble.com/shots/2470220-Iphone-Dribbble-App-Comment-Section)

@canterberrie If this is urgent .... can we remove the comment section for now ....
I will come back and add this back in an another PR
What you say ??

@IamManchanda That sounds good! I'll merge this once you remove the comment area.

done @canterberrie !


Loosen assertions in TestNewSignalsForwarderMultiple.

See https://github.com/gruntwork-io/terragrunt/issues/196.
I ran the tests a couple times and they passed.

Great, thanks!


feat(Toggle)!: colors + labels

Overview

feat(Toggle)!: colors + labels

PR Checklist

[ ] Related to designs:
[ ] Related to JIRA ticket: ABC-123
[ ] I have run this code to verify it works
[ ] This PR includes unit tests for the code change

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.83-alpha.554504.0@codecademy/gamut-labs@30.3.2-alpha.554504.0@codecademy/gamut@45.2.2-alpha.554504.0@codecademy/monaco@0.6.27-alpha.554504.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6238f00a82379924d2727a69--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.83-alpha.210796.0@codecademy/gamut-labs@30.3.2-alpha.210796.0@codecademy/gamut@45.2.2-alpha.210796.0@codecademy/monaco@0.6.27-alpha.210796.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://623a0e5c953f4a009d7afa9b--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.86-alpha.6adc5a.0@codecademy/gamut-labs@30.5.1-alpha.6adc5a.0@codecademy/gamut@45.3.1-alpha.6adc5a.0@codecademy/monaco@0.6.28-alpha.6adc5a.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-kit@0.6.86-alpha.585eac.0@codecademy/gamut-labs@30.5.1-alpha.585eac.0@codecademy/gamut@45.3.1-alpha.585eac.0@codecademy/monaco@0.6.28-alpha.585eac.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://623cae8768cd5215e3f68af0--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.86-alpha.60324f.0@codecademy/gamut-labs@30.5.1-alpha.60324f.0@codecademy/gamut@45.3.1-alpha.60324f.0@codecademy/monaco@0.6.28-alpha.60324f.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://623cc1e6b34c4b23e941f343--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.87-alpha.ee27f8.0@codecademy/gamut-labs@30.6.1-alpha.ee27f8.0@codecademy/gamut@45.3.1-alpha.ee27f8.0@codecademy/monaco@0.6.28-alpha.ee27f8.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://623ccda900b1b136f8eafb37--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.89-alpha.958da9.0@codecademy/gamut-labs@31.0.1-alpha.958da9.0@codecademy/gamut@46.0.1-alpha.958da9.0@codecademy/monaco@0.6.30-alpha.958da9.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://623e4acd58aa8510175d73f1--gamut-preview.netlify.appDeploy Logs

🚀 Styleguide deploy preview ready!https://6241b59c4b4933186d4e90cc--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.90-alpha.cdbd04.0@codecademy/gamut-labs@31.0.2-alpha.cdbd04.0@codecademy/gamut@46.0.2-alpha.cdbd04.0@codecademy/monaco@0.6.31-alpha.cdbd04.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6241fba8ee70db1e87c52909--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.91-alpha.23a7f4.0@codecademy/gamut-labs@31.0.3-alpha.23a7f4.0@codecademy/gamut@46.0.2-alpha.23a7f4.0@codecademy/monaco@0.6.31-alpha.23a7f4.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6242113ef854de0767166df7--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.91-alpha.97ca77.0@codecademy/gamut-labs@31.0.3-alpha.97ca77.0@codecademy/gamut@46.0.2-alpha.97ca77.0@codecademy/monaco@0.6.31-alpha.97ca77.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://624219cea3b35f00c211db34--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.91-alpha.3a1eff.0@codecademy/gamut-labs@31.0.3-alpha.3a1eff.0@codecademy/gamut@46.0.2-alpha.3a1eff.0@codecademy/monaco@0.6.31-alpha.3a1eff.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://62422b7d1c860d0e441245af--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.91-alpha.e83fca.0@codecademy/gamut-labs@31.0.3-alpha.e83fca.0@codecademy/gamut@46.0.2-alpha.e83fca.0@codecademy/monaco@0.6.31-alpha.e83fca.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://624339b01923eb358a643707--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.91-alpha.e49906.0@codecademy/gamut-labs@31.0.3-alpha.e49906.0@codecademy/gamut@46.0.2-alpha.e49906.0@codecademy/monaco@0.6.31-alpha.e49906.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6243415506df68007570fbf6--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.92-alpha.33557a.0@codecademy/gamut-labs@31.0.4-alpha.33557a.0@codecademy/gamut@46.0.3-alpha.33557a.0@codecademy/monaco@0.6.32-alpha.33557a.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-kit@0.6.92-alpha.ee2f81.0@codecademy/gamut-labs@31.0.4-alpha.ee2f81.0@codecademy/gamut@46.0.3-alpha.ee2f81.0@codecademy/monaco@0.6.32-alpha.ee2f81.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6244e031a862cc2d53f478b8--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.102-alpha.c20fc4.0@codecademy/gamut-labs@31.6.1-alpha.c20fc4.0@codecademy/gamut@46.1.2-alpha.c20fc4.0@codecademy/monaco@0.6.34-alpha.c20fc4.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://625821109388dc1aaf251d1a--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.103-alpha.5a5a08.0@codecademy/gamut-labs@31.6.2-alpha.5a5a08.0@codecademy/gamut@46.1.2-alpha.5a5a08.0@codecademy/monaco@0.6.34-alpha.5a5a08.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://6259816d3c14c10066c13684--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.103-alpha.47b5fb.0@codecademy/gamut-labs@31.6.2-alpha.47b5fb.0@codecademy/gamut@46.1.2-alpha.47b5fb.0@codecademy/monaco@0.6.34-alpha.47b5fb.0<EMAIL_ADDRESS>
📬Published Alpha Packages:@codecademy/gamut-kit@0.6.108-alpha.41a053.0@codecademy/gamut-labs@31.8.4-alpha.41a053.0@codecademy/gamut@46.1.4-alpha.41a053.0@codecademy/monaco@0.6.36-alpha.41a053.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://626188ac2f7fad19d5e1fe0e--gamut-preview.netlify.appDeploy Logs

📬Published Alpha Packages:@codecademy/gamut-kit@0.6.114-alpha.f60980.0@codecademy/gamut-labs@31.11.3-alpha.f60980.0@codecademy/gamut@46.1.5-alpha.f60980.0@codecademy/monaco@0.6.37-alpha.f60980.0<EMAIL_ADDRESS>
🚀 Styleguide deploy preview ready!https://626702f79da8af046c382e2d--gamut-preview.netlify.appDeploy Logs


AddSPField to existing Documents library is not working.

The script is not throwing an error.  When I run this in a site script, create script/design - then create the site.  I do not see my fields in the library.
I have also tried this list name:
"listName":"Shared Documents"
I have also tried this verb:
{
"verb": "addSiteColumn",
"internalName": "ClientID",
"addToDefaultView": true
}
Here is the code below:
{
"verb": "createSPList",
"listName": "Documents",
"templateType": 101,
"subactions": [
{
"verb": "addSPField",
"fieldtype": "Text",
"displayName": "Client ID",
"addToDefaultView":true
},
{
"verb": "addSPField",
"fieldtype": "Text",
"displayName": "Project ID",
"addToDefaultView":true
}
]
},

Thanks for the question.
So your site script is erroring for me. A couple of issues -

The value of 'fieldtype' should be 'fieldType' with a capital T. Be careful with the JSON schema case sensitivity.
isRequired is a required property (the irony)

Please try this script

{
  "$schema": "schema.json",
  "actions": [
    {
        "verb": "createSPList",
            "listName": "Documents",
            "templateType": 101,
            "subactions": [
                {
                    "verb": "addSPField",
                    "fieldType": "Text",
                    "displayName": "Client ID",
                    "addToDefaultView":true,
                    "isRequired":false
                    },
                    {
                    "verb": "addSPField",
                    "fieldType": "Text",
                    "displayName": "Project ID",
                    "addToDefaultView":true,
                    "isRequired":false
                }
            ]
        }
  ],
  "bindata": { },
  "version": 1
}

WORKED!  Thanks
From: Beau Cameron<EMAIL_ADDRESS>Sent: Monday, July 29, 2019 2:46 PM
To: SharePoint/sp-dev-docs<EMAIL_ADDRESS>Cc: Casey Root<EMAIL_ADDRESS>Author<EMAIL_ADDRESS>Subject: Re: [SharePoint/sp-dev-docs] AddSPField to existing Documents library is not working. (#4387)
Thanks for the question.
So your site script is erroring for me. A couple of issues -

The value of 'fieldtype' should be 'fieldType' with a capital T. Be careful with the JSON schema case sensitivity.
isRequired is a required property (the irony)

Please try this script
{
"$schema": "schema.json",
"actions": [
{

    "verb": "createSPList",

        "listName": "Documents",

        "templateType": 101,

        "subactions": [

            {

                "verb": "addSPField",

                "fieldType": "Text",

                "displayName": "Client ID",

                "addToDefaultView":true,

                "isRequired":false

                },

                {

                "verb": "addSPField",

                "fieldType": "Text",

                "displayName": "Project ID",

                "addToDefaultView":true,

                "isRequired":false

            }

        ]

    }

],
"bindata": { },
"version": 1
}
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/SharePoint/sp-dev-docs/issues/4387?email_source=notifications&email_token=AF3P2R6PDAICAXSO5WSR7FDQB43GPA5CNFSM4IHVAJJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD3BUVNQ#issuecomment-516115126, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AF3P2R5C3WER62NPKATK4X3QB43GPANCNFSM4IHVAJJA.
Baker Tilly Virchow Krause, LLP Confidentiality Notice: This message is being sent by Baker Tilly Virchow Krause, LLP. It is intended exclusively for the individuals and entities to which it is addressed. This communication, including any attachments, may contain information that is proprietary, privileged, confidential, including information that is protected under the HIPAA privacy rules, or otherwise legally exempt from disclosure. If you are not the named addressee, you are not authorized to read, print, retain, copy or disseminate this message or any part of it. If you have received this message in error, please notify the sender immediately by email and delete all copies of this message. This message is protected by applicable legal privileges and is confidential. Tax advice, if any, contained in this communication was not intended or written to be used by any taxpayer for the purpose of avoiding penalties.


Revert Quarkus to a Debezium compatible version

Fixes #446

Thank you @panga for fixing this.
Is it possible to use dependencyManagement and downgrade mongodb driver instead? Because Quarkus 3.15.x is LTS version. I was planing to keep it at this version for a while. what do you think?

@ismailsimsek yes, I did that in https://github.com/memiiso/debezium-server-iceberg/pull/447/commits/e42fb9a07e32f05c38ff3df9b702e9ddca4d1594
Notes:

dependencyManagement order wins, Quarkus is imported before Debezium, I tried changing order but it fails build.
dependencyManagement does not allow exclusions.
Override need to be added for all Mongo dependencies since there's no BOM.
I think it is best keep Quarkus version in sync with Debezium because you might find other incompatibilities.

Thank you for the notes, i will check 1.

For 4. with debezium release 3.x we can upgrade and align quarks + debezium versions. so far i was waiting for debezium 3.1 release but we can as well go head with 3.0


net/http: ReadRequest should reject requests with multiple Host headers

What version of Go are you using (go version)?

$ go version
go version go1.16 darwin/amd64

Does this issue reproduce with the latest release?
Yes
What operating system and processor architecture are you using (go env)?
go env Output
$ go env
GO111MODULE=""
GOARCH="amd64"
GOBIN="/Users//go-workspace/bin"
GOCACHE="/Users//Library/Caches/go-build"
GOENV="/Users//Library/Application Support/go/env"
GOEXE=""
GOFLAGS=""
GOHOSTARCH="amd64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users//go-workspace/pkg/mod"
GOOS="darwin"
GOPATH="/Users//go-workspace"
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/Users//.asdf/installs/golang/1.16/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users//.asdf/installs/golang/1.16/go/pkg/tool/darwin_amd64"
GOVCS=""
GOVERSION="go1.16"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch x86_64 -m64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/_z/0tl9x4tj43981q938qh7xqhw0000gp/T/go-build492873196=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?
http.ReadRequest doesn't account for potentially multiple Host headers in the request. This is a snippet of the code to reproduce this issue - https://play.golang.org/p/vZ71i09DAHb
According to the HTTP Spec (https://tools.ietf.org/html/rfc7230#section-5.4),

A server MUST respond with a 400 (Bad Request) status code to any
HTTP/1.1 request message that lacks a Host header field and to any
request message that contains more than one Host header field or a
Host header field with an invalid field-value.

However, this is appropriately handled in https://github.com/golang/go/blob/master/src/net/http/server.go#L1007. But clients attempting to parse an HTTP request using http.ReadRequest have no means to detect the presence of multiple Host headers since the Host header is promoted to http.Request.Host and is not part of the http.Request.Header map.
What did you expect to see?
Expect to have a means by which one can detect the presence of multiple Host headers in the HTTP request when using http.ReadRequest.
What did you see instead?
The http.Request returned by the method does not have any indication to the presence of multiple Host headers. It also returns a nil error.

Does ReadRequest return an error if there are multiple Host headers in the request?

Thank you for your sample. Given that ReadRequest rejects the request, are you asking for the error to be changed to perhaps indicate the multiple header values presented?
Asked another way, if we changed http.Request to capture multiple Host headers, how would that change the program you are trying to write?

@davecheney - it does not return an error when there are multiple Host headers present. The code sample's err == nil code runs.
The issue is it does not return an error nor reflect in the req.Header fields that there were multiple Host headers present.

To answer your latter question, I want to be able to use the API to parse an incoming byte stream and return an error if it has multiple Host headers.

@borncrusader thank you for your reply. Given this is a violation of the RFC, maybe the best resolution of this issue is to make ReadRequest reject such requests.

Thanks @davecheney for your prompt reply!

Looks like https://github.com/golang/go/issues/45513 addresses this as well. I believe this will be considered for 1.16.4. Please correct me if I'm wrong, @davecheney

Missed by triage?
cc @neild @fraenkel @empijei

1.17 returns an error, too many Host headers
issue #45513

I'll close this issue because it is a duplicate of issue #45513. Let's continue the conversation there.


refactor: inline _get_all

ClientEntityBase._get_all was only called from ClientEntityBase.get_all and get_all did nothing besides calling _get_all.
Related to https://github.com/hetznercloud/hcloud-python/pull/251#discussion_r1270785152

Welp, seems that my brain is already in the Friday mood. Did not see the call to self._get_all in ClientEntityBase.get_actions. Nevermind then!


Exec tachikoma update<PHONE_NUMBER>3350

:hamster::hamster::hamster:   Strategy none
Powered by Interval Pull Request as a Service - Tachikoma.io Announcement: 2015-06-28 Updated permissions - Tachikoma.io

Coverage remained the same at 68.0% when pulling 3b5c3e19f753c946fab6c0c5a2bd8ff7effe34a5 on pazjacket:tachikoma/update-20160925163350 into 391307db508f29cde46957edf9baba38c7a2933e on sanemat:master.


Clicking on a related record in a record list does not bring you to the related record

Describe the bug
In a record list you can show values of a related record field. For example, in the opportunity list in the CRM (https://latest.cortezaproject.org/compose/ns/crm/pages/69199987912736785):

If you click on the table row, you are correctly brought to the record of the row. In this case, the opportunity. However, when you click on the link in the "Account name" column, you are not brought to the account, but to the opportunity record as well.
In Chrome, if you click the back button in the browser, you return to the opportunity list.
In Firefox, if you click the back button in the browser, you are brought to the account page (the account you clicked on), and if you hit back again, then you are brought to the opportunity list.
To Reproduce
Steps to reproduce the behavior:

Go to https://latest.cortezaproject.org/compose/ns/crm/pages/69199987912736785
Click on any value in the "Account name" record, to go to any account
Instead of the account, you will see the opportunity
There is one important thing to mention here: in the opportunity viewing here, the field "Account Name" is empty, and this is unexpected. If you would click anywhere else in the record list to see the account, then you do see the "Account Name" value in the account record.
Hit back in the browser. In Firefox you'll see the behaviour described above.

Expected behavior
If you click on a value in the "Account Name" column - a related record field - I expect to go to the related record.
Desktop (please complete the following information):

Browser: Frirefox and Chrome

Additional context
This didn't happen in other previous releases.

@Fajfa @tjerman  any idea how to tackle this? Both events are triggered - if you go "back" in a browser, you'll notice you land on the clicked record.

Probably worth to merging 2021.3.x-design first due to changes in the layout

Added a small update (step 4):

There is one important thing to mention here: in the opportunity viewing here, the field "Account Name" is empty, and this is unexpected. If you would click anywhere else in the record list to see the account, then you do see the "Account Name" value in the account record.

I will take a look today. Something seems to be triggering the redirect

Fixed: https://github.com/cortezaproject/corteza-webapp-compose/commit/0f05f8314b4443091027dfecd1836a10f06b8d11


Swarm: remove artifactory-pro.yml deprecated props

As of Docker Engine 17.0.6, we get the following warning message:
Ignoring deprecated options:
container_name: Setting the container name is not supported.

The container name in a stack is defined by the service name in the stacks file.
So, removing the name definition removes the

Hi @marcellodesales - can you sign JFrog's CLA so I can review and merge?
Thx!

@eldada the link above requires login... How can I get access?

@marcellodesales - not sure about the account. Go directly to the form

@eldada DONE! I sent the signed form.


tidypandas pivot_wider behaves differently from tidyr::pivot_wider

I am illustrating this with an example.
tidyr::pivot_wider behaves as follows:
> tibble::tibble(a=c(1,2,3,1)
+                , b=c(1,2,3,1)
+                , c=c("aa", "bb", "aa", "bb")
+                , d=c(1,2,3,1)) %>%
+   tidyr::pivot_wider(id_cols=c("a", "b")
+                      , names_from = "c"
+                      , values_from = "d")
# A tibble: 3 × 4
      a     b    aa    bb
  <dbl> <dbl> <dbl> <dbl>
1     1     1     1     1
2     2     2    NA     2
3     3     3     3    NA

But in tidypandas, pivot_wider computes cross products of all the index columns(doesn't matter if some combination is present in the original dataframe) and then applies pivot operation.
import pandas as pd
from tidypandas import tidyframe, tidy_accessor as tp
from tidypandas.tidy_utils import simplify
from tidypandas.tidyselect import *
>>> pd.DataFrame({"a": [1,2,3,1]
             , "b": [1,2,3,1]
             , "c": ["aa", "bb", "aa", "bb"]
             , "d": [1,2,3,1]}).tp.pivot_wider(id_cols=["a", "b"]
                                              , names_from=["c"]
                                              , values_from=["d"])

	a	b	aa	bb
0	1	1	1.0	1.0
1	1	2	NaN	NaN
2	1	3	NaN	NaN
3	2	1	NaN	NaN
4	2	2	NaN	2.0
5	2	3	NaN	NaN
6	3	1	NaN	NaN
7	3	2	NaN	NaN
8	3	3	3.0	NaN

While pandas pivot_table results without the creating additional rows that coming from a cross product and thus matches with tidyr::pivot_wider result
import pandas as pd
from tidypandas.tidy_utils import simplify
>>> simplify(pd.DataFrame({"a": [1,2,3,1]
             , "b": [1,2,3,1]
             , "c": ["aa", "bb", "aa", "bb"]
             , "d": [1,2,3,1]})
    .pivot_table(index=["a", "b"]
                , columns=["c"]
                , values=["d"]).reset_index(drop=False)
)
	a	b	d__aa	d__bb
0	1	1	1.0	1.0
1	2	2	NaN	2.0
2	3	3	3.0	NaN

@talegari Should we keep auto-filling with cross-product as an optional argument?

@grahitr I agree, we should not have combinations which have no value corresponding to at least one level.
I plan to implement this as follows: I will expose argument dropna in pandas.pivot_table as id_expand with default as False (aka dropna is True).
Agree?

@talegari  Yeah, I agree.

https://github.com/talegari/tidypandas/pull/46/commits/96623dff1b40136eddc38e8f0323d40d970c9443

correction: https://github.com/talegari/tidypandas/commit/23821b8405a47101b8c96028503d7beba9686e55


Added an editmode to reuse existing code

Hi, i was proudly using this nice bundle and missed something like edit mode, to use my flow in edit manner too ...
i managed to get this working without changing your code (in my subflow) but probably its nice to have it in the bundle?
i am working on more things of that at the moment, liek changing templates a bit etc, but i think nothing necessary to be merged now.
Cheers phil

Is there a use case of having different validation group prefixes for steps in one flow? I can imagine being in edit mode for the entire flow and setting the prefix with setValidationGroupPrefix(), but I can't imagine having mixed prefixes.

I have that atm, i am using multiple file uploads of a collection of another entity in a step and depending on beeing in edit mode or not i needed to validate the "hidden" path value and not the file upload value.
i am currently under heavy construction, probably when i finished i can validate new and edit in the same manner ...
I am unsure right now. i also use the same validation group for 10 out of 11 steps so mightbe better that way.

Do you still see a need for this?

Hmm i finished the thing and still have the is editmode thing in it cause it allows me to determine things better inside the template and the controller ... not sure if i can get rid of it easily, but i will have a look into it next week. the validation group i dont use :)

You don't even have to get rid of it. ;) You can easily add your edit mode by extending the FormFlow class like you do in this PR code. But I don't think this should be part of the bundle itself. Feel free to convince me otherwise.

no need to i think ;D


Add red shadow to robot icon when there are conflicting trajectories

Task: Make robot bigger when it has conflicting trajectories
https://app.asana.com/0/1141736835949017/1183830989063585

@matiasbavera for your review!

I'm guessing this is to highlight robots currently in conflict? If/When we have the robot dimensions and render each robot with different sizes, this could make things confusing. I think we can use better indicators, adding a warning badge for example.

I'm guessing this is to highlight robots currently in conflict? If/When we have the robot dimensions and render each robot with different sizes, this could make things confusing. I think we can use better indicators, adding a warning badge for example.

Oh I see, that sounds good. What do you think about replacing the current robot’s black shadow for a red one too (In case of conflict of conflict of course)?

Hm okay just so I understand the conclusion:
we are looking at using a highlight/shadow to indicate the robot being in conflict instead of a change in size since it will be messy once we have many robots with different dimensions.

Could you create a function with the common logic?
Just checking, are we making sure that the components are only rendered when we want? Are we making sure it doesn't go into a state update loop?
@matiasbavera sorry what do you mean by creating function with common logic? and also, I'm not sure how to check and prevent a state update loop.

we are looking at using a highlight/shadow to indicate the robot being in conflict instead of a change in size since it will be messy once we have many robots with different dimensions.

Yeah, I understand that too, but let's check with @koonpeng .

@matiasbavera sorry what do you mean by creating function with common logic? and also, I'm not sure how to check and prevent a state update loop.

Sorry for not being more clear. Maybe we should discard this logic after the discussion but I'll explain it either way, let me try this way, the following logic is repeated for both test Robot overlay increases in size when it is in trajectory conflict  and Robots Overlay. I was just saying to create a function to set this to default values, not to copy and paste these values. Something like what you did in the loopRequest.test.tsx
const robots = fakeFleets()[0].robots;
    const bounds = new L.LatLngBounds([0, 25.7], [-14, 0]);
    const colorManager = new ColorManager();
    // TextEncoder is not available in node
    colorManager.robotColor = jest.fn(async () => 'black');
    colorManager.robotColorFromCache = jest.fn(() => 'black');
    const conflictRobotNames = [fakeFleets()[0].robots[0].name];

    const wrapper = mount(
      <LMap>
        <RobotsOverlay bounds={bounds} colorManager={colorManager} robots={robots} conflictRobotNames={conflictRobotNames} />
      </LMap>,
    );

Task: Make robot bigger when it has conflicting trajectories
https://app.asana.com/0/1141736835949017/1183830989063585

@matiasbavera for your review!

Maybe you should update the PR description.

thanks for all the thorough checking and comments @matiasbavera!
@koonpeng ready for merging


Add prominent Browse Collections button above footer on the homepage

This feature is part of Phase 1 of the DAMS v1.5 update (#33).
Add a "Browse All Collections" button above the footer that links to the Browse Collections page (#35)
This button itself will not change size at different viewport sizes, but the padding around it will.

The background is a sliver of feature image with a semi-transparent aggie-blue fill over it. I'll think I'll be making custom image crops for this feature, though we could try filling with the same image used in main feature image and see how it looks.

Skipping v1.5. Not included in v2 design.


Bump minimum Swift version to 5.7

Motivation:
Now that Swift 5.9 is GM we should update the supported versions and remove 5.6
Modifications:

Update Package.swift
Remove #if swift(>=5.7) guards
Delete the 5.6 docker compose file and make a 5.10 one
Update docs

Result:
Remove support for Swift 5.6, add 5.10

@yim-lee Can we update API breakage here to stop using 5.6?

CI updated. @swift-server-bot test this please


Use API workaround to existing TFS bug that results in only 50 work items being displayed

VSTS Extension you are using
Generate Release Notes
Where are you running it?
TFS On-Prem 16.122.26918.3
Version of Extension/Task
2.10.2
Expected behavior and actual behavior
Current bug in TFS results in only 50 associated work items being returned for a build.
See https://developercommunity.visualstudio.com/content/problem/38322/tfs-2017-update1-associated-work-items-for-a-build.html for reference.
Steps to reproduce the problem
Queue a build with more than 50 items, the build page (and the generate release notes task) will only return the first 50 work items.
We've found a workaround to this by passing $top=100 (etc.) in the API call to return 100 items instead of 50 but there is no way to customise this TFS build definition options, would it be possible to pass this parameter in the tasks API call? Could it possibly be a configurable option in the tasks options page?
Thank you.

That should be possible. I'll try to look into that soon.

Thanks Chris

@ChrisLGardner I have a bit more time, I will have a look at this and if it is OK make similar changes on the CrossPlatform version

Released as 2.11.4 for this Powershell version of generate release notes
I previously mentioned looking at the cross platform version, this is a bit more complex as it uses currently undocumented API, so I am leaving it alone. Also where it is a valid setting in the xplat version it defaults to 250 or 1000. Which I guess is enough for most!


Allow for nested lists in the config

This is a work in progress!  Let me know what you think.
I've been looking at this since I wanted to define a pipeline purely through the configuration file.  This change should allow you to do something like this:
    'model': {
        '__factory__': 'sklearn.pipeline.Pipeline',
        'steps': [['clf', {'__factory__': 'sklearn.linear_model.LinearRegression'}],
        ],
    },

Coverage remained the same at 100.0% when pulling 0cf2563a31fd060ae3a66658bc1d9803dc88b588 on samsungaccelerator:feature-config-with-nested-lists into 7ad551e9042ad1996d1d21872e2ce4d1b1b89454 on ottogroup:develop.

Very useful. Is there anything else you want to do for this feature (besides renaming of variable names)?

Very useful. Is there anything else you want to do for this feature (besides renaming of variable names)?

The one thing that comes to mind is update the pipeline example at the end of the tutorial.  But right now that example serves both as an example for:

how to use a pipeline
define a model in your own Python file

I'm happy for you to decide what's the right way to comment this pull request's feature in the docs.

Coverage remained the same at 100.0% when pulling<PHONE_NUMBER>8b667fd3f3925a5ba7cad9a073a4ae on samsungaccelerator:feature-config-with-nested-lists into 7ad551e9042ad1996d1d21872e2ce4d1b1b89454 on ottogroup:develop.

@alattner Can we merge this and add a ticket for updating the docs?


uhdm: 1.74 -> 1.75

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://repology.org/project/uhdm/versions.
meta.description for uhdm is: Universal Hardware Data Model
meta.homepage for uhdm is: https://github.com/chipsalliance/UHDM
Updates performed

Version update

To inspect upstream changes

Compare changes on GitHub

Impact

Checks done (click to expand)

built on NixOS
The tests defined in passthru.tests, if any, passed
found 1.75 with grep in /nix/store/ca02g7l4hdmxngbkic9ssxvcvhk12y58-UHDM-1.75
found 1.75 in filename of file in /nix/store/ca02g7l4hdmxngbkic9ssxvcvhk12y58-UHDM-1.75

Rebuild report (if merged into master) (click to expand)

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath
surelog
uhdm

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/ca02g7l4hdmxngbkic9ssxvcvhk12y58-UHDM-1.75 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A uhdm https://github.com/r-ryantm/nixpkgs/archive/37f0787ab9e3b336b011eace2afab547900ba18d.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/37f0787ab9e3b336b011eace2afab547900ba18d#uhdm

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/ca02g7l4hdmxngbkic9ssxvcvhk12y58-UHDM-1.75
ls -la /nix/store/ca02g7l4hdmxngbkic9ssxvcvhk12y58-UHDM-1.75/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package failed to build:

surelog

1 package built:

uhdm

Maintainer pings
cc @matthuszagh for testing.

@ofborg build surelog


fix GitHub ci

wait for slow machine to change state

Superseded by https://github.com/sparta-science/connect/commit/7b2f3d0d6743216513b7fd01d74e0ccc5e9b9080


This extension seems to be hijacking CTRL+ENTER in the A1111 img2img tab.

With this extension enabled, CTRL+ENTER stops working in A1111 img2img.  Instead, SHIFT+ENTER works.
Disabling this extension results in CTRL+ENTER again working normally in img2img  Behavior confirmed by multiple people.
Is this extension doing anything that could affect CTRL+ENTER?

Closing because no further information - and this extension indeed does not touch keybinds.


Creating one Fan..two detected!

Hi..
first of all, thanks for that mighty tool. I really appreciate your work!
This is may one of the minor issues, but when I create the "fan" (just one), everything is working fine but Siri was telling me, that "both" fans are ON right know... BOTH?!? So I started troubleshooting.. the HomeKit App itself is not showing a second one, expecting the Elgato Eve app (see below). I did not implement the SwingMode line im my config, because it is a ceiling fan. However it  seems that it has something to do with that. Take a look.

The first one implements the option for swing mode (schwenken). So, how can I get rid of that one? Also the log is showing that Broadlink is always requesting the swing mode. Any option for disabling?
[7/5/2017, 10:24:56 PM] [Broadlink RM] Ventilator getFanSpeed: 16
[7/5/2017, 10:24:56 PM] [Broadlink RM] Ventilator getSwitchState: false
[7/5/2017, 10:24:56 PM] [Broadlink RM] Ventilator getSwingMode: 0
[7/5/2017, 10:24:56 PM] [Broadlink RM] Ventilator getFanSpeed: 16
[7/5/2017, 10:24:56 PM] [Broadlink RM] Ventilator getSwitchState: false
"name": "Fan",
"type": "fan",
"data": {
"on": "26004a.....",
"off": "26004e0...",
"fanSpeed10": "26004a...",
"fanSpeed50": "260068...",
"fanSpeed100": "26004a..."
}
Thanks for your support in advance!

Same question here. Fan does not have a swing option, so it would be great to exclude this somehow in the config.json Thx.

Check out 2.5.14

@lprhodes You are awesome! Thanks a lot for this implementation. Works great here. 👍🏻

Just like this! @lprhodes Thanks for the support.


fixed vivisual key binding

v is used as default for character wise visual mode since zsh 5.0.8; using emacs style C-x C-e to edit current command in full editor in editor module instead, which allows to use v as meant to

Ah, interesting... maybe there would be a better bind than C-x C-e though? Especially if this is for vimode... this seems like a very non-vi style key binding.

I think C-x C-e was picked because that's the default command for doing this in bash, which a lot of people got used to before we all found zsh and prezto and vimode.  When I saw this PR and realized that visual mode works perfectly fine on the command line as long as you have a key for it, I put this in my personal config as well.  And I couldn't think of any better keybinding for it than C-x C-e either.

Thanks for submitting this! Sorry it took so long to merge.


Allow disabling Reaper UI auth

Disabling Reaper's auth can theoretically be done by adding an empty name for Reaper.spec.UiUserSecretRef. In k8ssandra-operator though, we create a secret with credentials that's then added as a ref.
We need to introduce a way of disabling the auth for Reaper's UI to accommodate some use cases.

Quoted from another ticket:

Is that correct? If we are creating a standalone Reaper instance here (via a Reaper CR, not the k8ssandraCluster CR) then can't we just set Reaper.spec.UiUserSecretRef == ""?

We could if we created a Reaper CR, but that doesn't work with k8ssandra-operator and we're relying on k8ssandra-operator.


Comments left via the plugin hide line contents in AzDO preview

Extension version: 0.0.21
VSCode Version: 1.55.2
OS: Windows 10

Steps to Reproduce:

Checkout a PR in this plugin
Leave a comment on a line

Leaving a comment via this plugin in vscode causes the preview in AzDO to be missing all but the last character of the line the comment is set on

I did not understand it. Is the actual line max_line_length=100 not present in AzDO UI? Have you commited and pushed your changes?

Yeah there are no changes locally.  This screenshot is from a PR that I checked out via the plugin and added the comment on line 36.  When I view the comment in AzDO you can see it only shows the last character (0) for the line I left the comment on.  It didn't modify the code, just whatever is used to show the comment location in the file.   It doesn't break anything, but makes it hard to review comments in AzDO because you lose the line that the comment was left on.

Adding a comment will not change the line itself. The AzDO UI will show what is the latest commit in the branch in remote. And VS Code will show whatever is the latest commit in local. Maybe you don't have the latest change? Could you do git fetch and then git pull.


Add hover animations to Project components

Description
I would like to add some hover animations to Project components (probably scale them with the transform property). It's a well-known and commonly used pattern to improve user experience.
Screenshots
No response
Additional information
No response
Code of Conduct

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

Sure @Petrosdevri

I am working on the hover animations and have developed most of them, however they don't appear for some errors I don't understand. One of the main explanations is that there hasn't been a styles.css file generated in the public folder. I would like to ask how do you update these changes in Tailwind as rest of the code seems to work fine.

Hey @Petrosdevri
I am unable to understand the error
Could you raise a PR for the changes you have made? It would be much easier to understand the error with the code


Check for nil instead of truthiness when making a diff

Previous Behavior
The following lines create different output:
expect(true).to match a_falsy_value

# =>
       expected true to match (a falsey value)
       Diff:
       @@ -1,2 +1,2 @@
       -(a falsey value)
       +true

expect(false).to match a_truthy_value

# =>
       expected false to match (a truthy value)

The first generates a diff, whereas the second does not. This is because the actual and expected values are checked for presence with if actual && expected when creating the diff. In the first example, the actual is true, so it passes the check. In the second, the actual is false and the diff returned is an empty string.
New Behavior
These should be consistent with each other. The two options would be to make them both create a diff or modify the matcher so neither do. However, since all other match(something) matchers produce a diff output, it makes sense to do the same here.
If it is necessary for that check to also check for truthiness, then another solution could be found. But, I cannot find a situation where false would be passed in where we wouldn't want to create a diff.

I went ahead and checked that a diff is produced for both. I can separate them into their own it instances if you want. Also, CI failed because it timed out, is there a way to retry the job?

@lnestor I suggest you to:
git commit --amend
git push --force-with-lease lnestor diff-generation-nil-check

to re-run the build.


Make gutter icons configurable

Type of JetBrains IDE (IntelliJ, PyCharm, etc.) and version
any
TeXiFy IDEA version
0.6.3
What I did (steps to reproduce)
I want to disable e.g. "include" gutter action
Expected behavior
Can configure in Preferences | Editor | General | Gutter Icons
Actual behavior
n/a
fix: implement getName()/getIcon() in GutterIconDescriptor

@YannCebron How do you use GutterIconDescriptor? I cannot find out how to incorporate it into the framework.
The current gutter icons are provided via LatexCompileGutter (: RunLineMarkerContributor) for creating run configurations and LatexLineMarkerProvider (: RelatedItemLineMarkerProvider) to navigate to inclusions. I cannot find the link with GutterIconDescriptors here. Could it be that I completely missed it?

RelatedItemLineMarkerProvider extends LineMarkerProviderDescriptor which extends GutterIconDescriptor which has default no-op implementations
LatexCompileGutter will be used by RunLineMarkerProvider which in turn has its own implementation of GutterIconDescriptor, so it's a global setting here


[core / Docker] Add  workflow to build TRL docker images

What does this PR do?
As discussed internally, this is a first step towards creating our CI system that will run on self-hosted runners on GPU. I want to test everything on two scenarios:

transformers / accelerate / peft all installed from source
all the libraries above installed from latest pypi
We currently use similar docker images in transformers & peft and this is very convenient for testing purpose and tracking down where the issues do come from.
The dockerfile files are mostly copied from PEFT: https://github.com/huggingface/peft/blob/main/docker/peft-gpu/Dockerfile

cc @lvwerra

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Can't build example

Rust version: rustc 1.31.0-nightly (96cafc53c 2018-10-09)
I get the following output when trying to compile the example on a raspi 3B with cargo run --example rpi:
error[E0463]: can't find crate for `hal`
  --> examples/rpi.rs:14:1
   |
14 | extern crate linux_embedded_hal as hal;
   | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ can't find crate

error: aborting due to previous error

For more information about this error, try `rustc --explain E0463`.
error: Could not compile `mfrc522`.

To learn more, run the command again with --verbose.

You have to put linux_embedded_hal into Cargo.toml.
See here: https://crates.io/crates/linux-embedded-hal


Linux performance tuning - description of vm.swappiness parameter

This section of the Linux tuning guide says:

vm.swappiness: This parameter controls relative weight given to swapping out runtime memory by limiting the kernel to swap out SQL Server process memory pages.

I found this phrasing confusing and wanted to understand more about what the setting does and why 1 is the recommended value. Some references are below - would it be possible to use these to expand and clarify this part of the documentation?
I have a rough draft description of what might have helped me, though it can surely be improved:

vm.swappiness: When a process on Linux requests memory and none is available, there is a choice of what happens next: removing cached filesystem data (a quick task), or writing memory pages to the disk swapfile (a slower task). vm.swappiness adjusts this balance by setting one side of a ratio that totals 200, so the default setting of 60 prefers removing items from the filesystem cache at a ratio of 140:60. A setting of 1 is the strongest preference towards clearing down the filesystem cache and away from swapping process memory to disk. This setting is optimal because SQL Server has its own internal caching that can use information about the contents of the database files to make better caching decisions than the operating system, so the priority is to avoid the slowness of SQL Server memory being written to swap disk. (Setting vm.swappiness to 0 blocks the use of swap even in extreme situations where it would avoid a crash, and increases the risk of an Out Of Memory handler forcibly ending SQL Server instead; this is not recommended).

(I am not certain of this reasoning, it is my best current understanding)

Linux kernel memory management concepts (in the Linux kernel source code) - "As the [memory] load increases, the amount of the free pages goes down and when it reaches a certain threshold (low watermark), an allocation request will awaken the kswapd daemon. It will asynchronously scan memory pages and either just free them if the data they contain is available elsewhere [drop cached filesystem data], or evict to the backing storage device [swap process memory out to disk]"
Documentation for /proc/sys/vm/ at kernel.org - "Swappiness: This control is used to define the rough relative IO cost of swapping and filesystem paging, as a value between 0 and 200. At 100, the VM [Virtual Memory manager] assumes equal IO cost and will thus apply memory pressure to the [filesystem] page cache and swap-backed pages equally"
The Linux kernel source code comments where the swappiness measure is taken into account - "The amount of pressure we put on each LRU [Least Recently Used list, those being filesystem cached pages and process memory pages] is inversely proportional to the cost of reclaiming each list, as determined by the share of pages that are refaulting, times the relative IO cost of bringing back a swapped out anonymous page vs reloading a filesystem page (swappiness)." and the equation fp = (200 - swappiness) * (total_cost + 1) giving the ratio involving 200.

Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: dc91cc25-7d13-a975-76ee-6d97bdcf9416
Version Independent ID: ae8d2f7e-0c69-1269-a438-d6d36594dd27
Content: Performance best practices for SQL Server on Linux - SQL Server
Content Source: docs/linux/sql-server-linux-performance-best-practices.md
Product: sql
Technology: linux
GitHub Login: @tejasaks
Microsoft Alias: tejasaks

@HumanEquivalentUnit -- thank you for your well-researched feedback.  Excellent alias too.
@tejasaks -- please look into this issue.

@HumanEquivalentUnit -- thank you for your well-researched feedback.  Excellent alias too.
@tejasaks -- please look into this issue.

@HumanEquivalentUnit , thank you for your research and feedback. We were trying to keep the individual description concise while trying to give some context into recommendation. I agree with you that our text could be improved and we will look into it. It may not be too elaborate though as we would like to keep it just descriptive without making it a tutorial on that one parameter.
@amvin87 to help review the comment and content.

@HumanEquivalentUnit , thank you for your research and feedback. We were trying to keep the individual description concise while trying to give some context into recommendation. I agree with you that our text could be improved and we will look into it. It may not be too elaborate though as we would like to keep it just descriptive without making it a tutorial on that one parameter.
@amvin87 to help review the comment and content.

Thank you :)
I understand it's not a place for a huge tutorial, it would be a very long page if that happened for every parameter. The parts I wanted to understand more about were:

If the relative weight has "swapping out runtime memory" on one side, what is on the other side - what are we trading away?
If the setting can have hundreds of possible values, why is 1 a value that can always be confidently recommended - and yet it's not the default value?

Trying to focus on these, I can rewrite to half the size, is that any better and is it clear?

vm.swappiness: adjusts how the Linux kernel reclaims used memory, either by writing some memory pages out to disk (swapping), or by removing data from its filesystem cache. The default is to compromise between them, weighted against the extra IO cost of swapping to disk. A database workload performs best when avoiding swapping, and setting 1 most strongly prioritises reclaim from filesystem cache instead. (Setting 0 increases the risk of reclaiming memory by stopping processes, and is not recommended)

Thank you :)
I understand it's not a place for a huge tutorial, it would be a very long page if that happened for every parameter. The parts I wanted to understand more about were:

If the relative weight has "swapping out runtime memory" on one side, what is on the other side - what are we trading away?
If the setting can have hundreds of possible values, why is 1 a value that can always be confidently recommended - and yet it's not the default value?

Trying to focus on these, I can rewrite to half the size, is that any better and is it clear?

vm.swappiness: adjusts how the Linux kernel reclaims used memory, either by writing some memory pages out to disk (swapping), or by removing data from its filesystem cache. The default is to compromise between them, weighted against the extra IO cost of swapping to disk. A database workload performs best when avoiding swapping, and setting 1 most strongly prioritises reclaim from filesystem cache instead. (Setting 0 increases the risk of reclaiming memory by stopping processes, and is not recommended)

Issue was resolved under https://github.com/MicrosoftDocs/sql-docs/commit/6a7f1b7b586ab179620f48fe7529eb10d60c537a#diff-675da0d95633a88899d47bb6c920d65f5497577f81950238a17e8bffab2088c5


Validate links when publishing guides

Ensure links in guide body are 200. Allow user to publish even if there
are broken links.
Cache link checking, but expire it after five minutes.

Guide couldn't be published because it had broken links:

But, we can just publish anyway:

I like the UI and the option to force publish. The validation errors are a good place to display the errors too.
Could we show the screenshots to the content lot to see what they think?
The worry is that a page with a lot of links could take a long time to save. I think that's worth mentioning to the content people too. If they're pressing that save button a lot over a 2 hour period then the slow down could be too much for them.
I'll have a look at the code..

@thehenster the links are only checked when Publish is clicked. This is because I was also thinking that the user would be clicking Save more often. It shouldn't be too slow, though perhaps we could do what whitehall does and use Typhoeus for parallel http requests.

Looks good although maybe we could show this to everyone before merging? It's quite a major bit of publishing flow.

Reminder: do a demo tomorrow

This isn't going to work on environments with http basic auth :cry:

Maybe this should be done entirely in javascript.

LGTM


hanlp 2.0.0a26 使用 CTB5_POS_RNN 报错

Describe the bug
hanlp 2.0.0a26 使用 CTB5_POS_RNN 报错 .
Code to reproduce the issue
Provide a reproducible test case that is the bare minimum necessary to generate the problem.
tokenizer = hanlp.load('CTB5_POS_RNN')
(tokenizer('商品和服务')

Describe the current behavior
运行上述代码出错
Expected behavior
能够运行测试代码
System information

Ubuntu 18.04
Python version: 3.6
HanLP version: 2.0.0a26

Other info / logs
Traceback (most recent call last):
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/eager/context.py", line 1897, in execution_mode
    yield
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/data/ops/iterator_ops.py", line 659, in _next_internal
    output_shapes=self._flat_output_shapes)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/ops/gen_dataset_ops.py", line 2479, in iterator_get_next_sync
    _ops.raise_from_not_ok_status(e, name)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/framework/ops.py", line 6606, in raise_from_not_ok_status
    six.raise_from(core._status_to_exception(e.code, message), None)
  File "<string>", line 3, in raise_from
tensorflow.python.framework.errors_impl.InvalidArgumentError: ValueError: `generator` yielded an element of shape () where an element of shape (None,) was expected.
Traceback (most recent call last):

  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/ops/script_ops.py", line 236, in __call__
    ret = func(*args)

  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/data/ops/dataset_ops.py", line 825, in generator_py_func
    "of shape %s was expected." % (ret_array.shape, expected_shape))

ValueError: `generator` yielded an element of shape () where an element of shape (None,) was expected.

	 [[{{node PyFunc}}]] [Op:IteratorGetNextSync]

During handling of the above exception, another exception occurred:

Traceback (most recent call last):
  File "load-sem.py", line 5, in <module>
    tokenizer('商品和服务')
  File "/usr/local/lib/python3.6/dist-packages/hanlp/common/component.py", line 51, in __call__
    return self.predict(data, **kwargs)
  File "/usr/local/lib/python3.6/dist-packages/hanlp/components/taggers/rnn_tagger.py", line 47, in predict
    return super().predict(sents, batch_size)
  File "/usr/local/lib/python3.6/dist-packages/hanlp/common/component.py", line 454, in predict
    for idx, batch in enumerate(dataset):
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/data/ops/iterator_ops.py", line 630, in __next__
    return self.next()
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/data/ops/iterator_ops.py", line 674, in next
    return self._next_internal()
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/data/ops/iterator_ops.py", line 665, in _next_internal
    return structure.from_compatible_tensor_list(self._element_spec, ret)
  File "/usr/lib/python3.6/contextlib.py", line 99, in __exit__
    self.gen.throw(type, value, traceback)
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/eager/context.py", line 1900, in execution_mode
    executor_new.wait()
  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/eager/executor.py", line 67, in wait
    pywrap_tensorflow.TFE_ExecutorWaitForAllPendingNodes(self._handle)
tensorflow.python.framework.errors_impl.InvalidArgumentError: ValueError: `generator` yielded an element of shape () where an element of shape (None,) was expected.
Traceback (most recent call last):

  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/ops/script_ops.py", line 236, in __call__
    ret = func(*args)

  File "/usr/local/lib/python3.6/dist-packages/tensorflow_core/python/data/ops/dataset_ops.py", line 825, in generator_py_func
    "of shape %s was expected." % (ret_array.shape, expected_shape))

ValueError: `generator` yielded an element of shape () where an element of shape (None,) was expected.

	 [[{{node PyFunc}}]]

[x] I've completed this form and searched the web for solutions.

Taggers take lists of tokens as input, then outputs one tag for each token.


WebRunner::start should accept canvas object

It would be nice if it was possible to initialize eframe on web without giving the canvas an id and appending it to the DOM. It should be possible if the WebRunner::start accepted an HTMLCanvasElement, and passed it along directly:
https://github.com/emilk/egui/blob/d2717180656d640e606a46932718dbef59fd5ef7/crates/eframe/src/web/web_runner.rs#L58-L63
Right now for apps using eframe with multiple canvases on the same page, each one needs to be given a unique ID. The painter already holds onto the canvas element directly rather than its ID, so this feels like an obvious and easy change.

Context: https://github.com/rerun-io/rerun/pull/6680#discussion_r1659408887


Roll Engine from 859d892f1fca to a27336bcbf34 (16 revisions)

https://github.com/flutter/engine/compare/859d892f1fca...a27336bcbf34
2020-06-04<EMAIL_ADDRESS>Roll Dart SDK from acb552d056be to ac45f23134f2 (4 revisions) (flutter/engine#18821)
2020-06-04<EMAIL_ADDRESS>Roll Fuchsia Mac SDK from YmMjL... to aEbAT... (flutter/engine#18818)
2020-06-04<EMAIL_ADDRESS>Roll Dart SDK from e5b2cb0b02ae to acb552d056be (1 revision) (flutter/engine#18817)
2020-06-04<EMAIL_ADDRESS>Roll Dart SDK from 54adfeb93f0b to e5b2cb0b02ae (53 revisions) (flutter/engine#18806)
2020-06-04<EMAIL_ADDRESS>Mark symbols that should be public (flutter/engine#18802)
2020-06-04<EMAIL_ADDRESS>Live region announcements for iOS (flutter/engine#18798)
2020-06-04<EMAIL_ADDRESS>Don't export embedder API in desktop embeddings (flutter/engine#18735)
2020-06-03<EMAIL_ADDRESS>removing firefox tests from cirrus. since yesterday they are running on LUCI. (flutter/engine#18790)
2020-06-03<EMAIL_ADDRESS>ios accessibility: started ignoring route changes when presenting modal view controllers (flutter/engine#18544)
2020-06-03<EMAIL_ADDRESS>[web] Remove connection close on blur for desktop browsers (flutter/engine#18743)
2020-06-03<EMAIL_ADDRESS>Implement GetAllocationSize for Vertices (flutter/engine#18756)
2020-06-03<EMAIL_ADDRESS>Revert "null-annotate SemanticsUpdateBuilder.updateNode (#18552)" (flutter/engine#18785)
2020-06-03<EMAIL_ADDRESS>Pass SurfaceFrame to SubmitFrame (flutter/engine#18709)
2020-06-02<EMAIL_ADDRESS>Pin the analyzer version as a temporary workaround for https://github.com/dart-lang/sdk/issues/42163 (flutter/engine#18755)
2020-06-02<EMAIL_ADDRESS>Make images contribute to Picture allocation size, more aggressively release references when dispose is called (flutter/engine#18706)
2020-06-02<EMAIL_ADDRESS>null-annotate SemanticsUpdateBuilder.updateNode (flutter/engine#18552)
Also rolling transitive DEPS:
fuchsia/sdk/core/mac-amd64 from YmMjLp_hSbZy to aEbATANh6MfA
If this roll has caused a breakage, revert this CL and stop the roller
using the controls here:
https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Please CC<EMAIL_ADDRESS>on the revert to ensure that a human
is aware of the problem.
To report a problem with the AutoRoller itself, please file a bug:
https://bugs.chromium.org/p/skia/issues/entry?template=Autoroller+Bug
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+doc/master/autoroll/README.md

Commit queue failed; closing this roll.


blockScraper implementation

Is your feature request related to a problem? Please describe.
A scraper pipeline capable of retrieve all the similar blocks in a page, like ecommerce, weather, fly websites
Describe the solution you'd like
I have found this paper https://www.researchgate.net/publication/261360247_A_Web_Page_Segmentation_Approach_Using_Visual_Semantics
It deals specifically wti this issue
Describe alternatives you've considered
nope
Additional context

Neat idea but would it be simpler to just group web elements with the same css tags? A computer vision approach seems a bit over-engineered.

@epage480 This isn't a CV approach, it's a grouping similar object from HTML, if you want to help to implement this paper with us let me know, here the reference A Web Page Segmentation Approach Using Visual Semantics

@lurenss I would much rather focus on this paper, https://doi.org/10.1007/978-3-030-72240-1_5
much more recent and detailed comparison of all major web page segmentation algorithms.
tl;dr - microsoft's VIPS algorithm is still the best one out there; you can find implementations in java, JS or python.


Not finding any sass files

Thank you for this plugin!
I'm working on windows with ember cli 2.11.1
Your docs are outdated and don't cover newer versions of ember-cli.
Creating an addon/styles dir does not work on the newer ember-cli versions, and they don't even have an index.js file anymore.
After a whole day bashing my head against this, I found a solution:
Here's the relevant part of my ember-cli-build.js:
var EmberApp = require('ember-cli/lib/broccoli/ember-app')
var production = EmberApp.env() === 'production';
var basePath = process.cwd().replace(/\\/g,"/") + "/";
var sassDir = basePath  + "app/styles";

module.exports = function(defaults) {
  var app = new EmberApp(defaults, {
    compassOptions: {
      outputStyle: production ? 'compressed' : 'expanded',
      getSassDir: () => sassDir
    }
  });
  return app.toTree();
};

Should work on any os and with any version of ember-cli.
It's a bit of a hack, but it works like a charm.
Hope this helps someone. :)

Out of curisoity why did you close this?
I found it useful!


Improve Badges/src/badge-0.html

Need to improve this component on Badges/src/badge-0.html
Improvements required :
add padding
Any reference image?

Any external reference?
You can checkout this one for better understanding.

This looks good.


[Update] Watermark

[Removed] text watermark
[Added] image watermark
So it was planned to remove the text.
I think way more servers would use a logo as a watermark.

One thing worth pointing out is that we intentionally chose to show the everon.life branding so screenshots etc show it and people get pointed to the website for infos. I am not sure if a logo will provide the same effect.

What if you do an image logo that has the weblink like under it just smaller text. (All in one image)

I know the quality isn't 100% but as proof of concept. Maybe something like this.

I know the quality isn't 100% but as proof of concept. Maybe something like this.

I just actually looked at what you made lol should have done that before, I actually like it a alot, I think having it would be fine, if you look up everon life the website is #2 that comes up so personally, I think its a good change and will look more professional then a text url.

The "watermark" is used only during early development and will be removed completely in the near future. The website URL is shown intentionally, and a logo would not add much information. So I would close this. Thoughts? @Yawalla221

Yes, I'm going to change it.

Closing this, as any kind of watermark will be removed in the near future and until then the plain URL is all that is really needed.


feat: add comment into the starter template for required title element editor hint

What is it?

[X] Feature / enhancement
[ ] Bug
[ ] Docs / tests

Description
add required title element into the starter template, for better SEO.
Use cases and why

One use case

Another use case

Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have made corresponding changes to the documentation
[ ] Added new tests to cover the fix / functionality

The title is already in the starters:
https://github.com/BuilderIO/qwik/blob/b45bb4668d219b8db84828b746c82e49fa84eddd/starters/apps/basic/src/components/router-head/router-head.tsx#L13
Also, <meta charSet="utf-8" /> has to be always the first tag!
Wonder if this PR can be a comment instead.

damn... idk, gonna think about this. it's good to keep <title> in the other component because it allow to acces the qwikcity state


Support Scala 2.12

This PR bumps dependencies versions to most recent ones with Scala 2.12 support.
Algebird fixes based on #715.
This PR also removes support for Scala 2.10, because scalding doesn't support it.

This PR will be mergeable as soon as we release scalding 0.17.1 with fix for KryoHadoop https://github.com/twitter/scalding/pull/1685.

Now, when scalding release is out we can merge this 2.12 upgrade for SB. What do you think @pankajroark @piyushnarang @johnynek ?

Update: false alarm, tests are failing, digging into this now.

looks like a legit failure:
https://travis-ci.org/twitter/summingbird/jobs/252265982#L3657
Caused by: com.esotericsoftware.kryo.KryoException: java.lang.RuntimeException: Could not serialize lambda

Serialization trace:

fn (com.twitter.scalding.typed.FlatMappedFn)

	at com.esotericsoftware.kryo.serializers.ObjectField.read(ObjectField.java:144)

	at com.esotericsoftware.kryo.serializers.FieldSerializer.read(FieldSerializer.java:551)

	at com.esotericsoftware.kryo.Kryo.readClassAndObject(Kryo.java:790)

	at com.twitter.chill.SomeSerializer.read(SomeSerializer.scala:25)

	at com.twitter.chill.SomeSerializer.read(SomeSerializer.scala:19)

	at com.esotericsoftware.kryo.Kryo.readClassAndObject(Kryo.java:790)

	at com.twitter.chill.SerDeState.readClassAndObject(SerDeState.java:61)

	at com.twitter.chill.KryoPool.fromBytes(KryoPool.java:94)

	at com.twitter.chill.Externalizer.fromBytes(Externalizer.scala:145)

	at com.twitter.chill.Externalizer.maybeReadJavaKryo(Externalizer.scala:158)

	at com.twitter.chill.Externalizer.readExternal(Externalizer.scala:148)

	at java.io.ObjectInputStream.readExternalData(ObjectInputStream.java:1840)

This could be that we don't have the right chill version? It could be this exposes a bug in chill.

@johnynek yes, it is legit =( Unfortunately this time it comes from scalacheck and it's same issue as we saw in chill, yes - too many lambdas in one class. I tried to workaround this by creating this lambdas in summingbird code but there are couple of things which makes everything more complicated - they declare everything as private[scalacheck] (thanks not in general) and all classes are sealed. At some point I decided to stop and now I'm going to make a PR to scalacheck instead. If they do release quickly we can use new version otherwise I will continue to work on workaround.

@ttim I'm happy to help get this fixed, either in this repo or ScalaCheck? What were you planning to do to work around this? Restructure the ScalaCheck code a bit? Create your own custom Gen instances?

@ttim I'm happy to help get this fixed, either in this repo or ScalaCheck? What were you planning to do to work around this? Restructure the ScalaCheck code a bit? Create your own custom Gen instances?

@non that's awesome! If you can also help with releasing new version it would be twice awesome =)
In short term for SB I thought just to add code like this:
object ArbitraryWorkaround {
 implicit val f1: Arbitrary[Int => Int] = Arbitrary(Gen.const(x => x * 2))
 implicit val f2: Arbitrary[Int => List[(Int, Int)]] = Arbitrary(Gen.const(x => List((x, x * 3))))
 implicit val f3: Arbitrary[Int => Option[Int]] = Arbitrary(Gen.const(x => {
   if (x % 2 == 0) None else Some(x * 4)
 }))
 implicit val f4: Arbitrary[Int => List[Int]] = Arbitrary(Gen.const(x => List(x * 5)))
 implicit val f5: Arbitrary[((Int, (Int, Option[Int]))) => List[(Int, Int)]] = Arbitrary(Gen.const {
   case (x, (y, optZ)) => List((x, y), (x, optZ.getOrElse(42)))
 })
 implicit val f6: Arbitrary[((Int, Int)) => List[(Int, Int)]] = Arbitrary(Gen.const(x => List(x, x)))
}

But if we can do release reasonably soon I will do PR to scalacheck as soon as possible (today/tomorrow). Plan is to add a test on that + divide ArbitraryArities and GenArities into two files. Wdyt?

@ttim My suggestion is that if the above work-around solves the issue for Summingbird we start there.
I'm happy to try to expedite the suggested change to ScalaCheck, but unfortunately the arities files are traits right now, so restructuring them will break binary compatibility. This doesn't mean we can't do it, but might make a quick release more difficult.

(Also, if you'd prefer help I can make a branch off of this one and try the approach out myself later tonight.)

@non let's do this then. I'll merge my workaround. Meanwhile: this is a scala issue - https://issues.scala-lang.org/browse/SI-10232.
Here is repro I used:
import org.scalacheck.Arbitrary._
val f = arbitrary[Int => Int]
val outputStream = new java.io.ByteArrayOutputStream()
val objectOutputStream = new java.io.ObjectOutputStream(outputStream)
objectOutputStream.writeObject(f)
objectOutputStream.close
val inputStream = new java.io.ByteArrayInputStream(outputStream.toByteArray)
val objectInputStream = new java.io.ObjectInputStream(inputStream)
objectInputStream.readObject()

Causes:
Caused by: java.lang.BootstrapMethodError: too many bootstrap method arguments
  at java.lang.invoke.CallSite.makeSite(CallSite.java:320)
  at java.lang.invoke.MethodHandleNatives.linkCallSiteImpl(MethodHandleNatives.java:307)
  at java.lang.invoke.MethodHandleNatives.linkCallSite(MethodHandleNatives.java:297)
  at org.scalacheck.GenArities.$deserializeLambda$(GenArities.scala)

@non created a bug in scalacheck repo https://github.com/rickynils/scalacheck/issues/342, thank you very much for looking on this!

One other thing to note: it seems like the bug suggests that using -Ydelamdafy:inline is a work-around, so it's possible we could include that in the scalac arguments used for testing to avoid the Scala bug.
(Thanks for giving me a repro -- I'll start seeing what I have to do to fix this issue in ScalaCheck.)

@non I'm afraid -Ydelambdafy:inline should be used at the scalacheck compile time, so it's not possible to use as a workaround =(

I've got a fix on the ScalaCheck side that should let us publish an 0.13.6 release there that will resolve the issue without the workaround here.

Scalacheck release isn't out therefore I think to merge this PR now, and I created issue to remove it later as soon as scalacheck will be released - https://github.com/twitter/summingbird/issues/741.
Wdyt @johnynek @pankajroark @piyushnarang ?

I think merging this now and removing the workaround later sounds reasonable, but I think @non is planning to publish the ScalaCheck release either this weekend or next week.

Codecov Report

Merging #724 into develop will increase coverage by 0.04%.
The diff coverage is 100%.

@@             Coverage Diff             @@
##           develop     #724      +/-   ##
===========================================
+ Coverage    72.21%   72.26%   +0.04%
===========================================
  Files          153      154       +1
  Lines         3736     3742       +6
  Branches       204      209       +5
===========================================
+ Hits          2698     2704       +6
  Misses        1038     1038

Impacted Files
Coverage Δ

.../twitter/summingbird/scalding/VersionedState.scala
0% <ø> (ø)
:arrow_up:

...cala/com/twitter/summingbird/batch/Timestamp.scala
65.71% <100%> (ø)

...cala/com/twitter/summingbird/example/Storage.scala
77.77% <100%> (+1.3%)
:arrow_up:

.../com/twitter/summingbird/ArbitraryWorkaround.scala
100% <100%> (ø)

...ain/scala/com/twitter/summingbird/TestGraphs.scala
86.62% <0%> (-1.92%)
:arrow_down:

...ala/com/twitter/summingbird/scalding/Service.scala
76.92% <0%> (-1.54%)
:arrow_down:

Continue to review full report at Codecov.

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

@travisbrown-stripe I prefer to keep it merged and do not put any pressure on scalacheck =) I'll create removal PR immediately after merge thou.


Create an example app

Some initial work was already done in https://github.com/ably-labs/ably-chat-swift/pull/34 while implementing #4
┆Issue is synchronized with this Jira Task by Unito

We have an example app now!


Release <IP_ADDRESS>-rc1

OpenSearch 2.0.0-rc1 has been released, see release notes.
Official OpenSearch release ticket: https://github.com/opensearch-project/opensearch-build/issues/1624
Checklist:

[x] Compatibility Matrix is updated
[x] Release <IP_ADDRESS>-rc1 tag is present in the repo
[x] Release artifacts are available and downloadable
[x] Release notes published
[x] Forum announcement published

Done.


Fix folders for docker

does what it  says on the tin.

This only fixes the python3 version. Can you apply the same fixes to the python2 files?


Order is not placed if order is lower than minimum lot size

Added missing check after rounding of quantity
Added Moq package to unit testing so it's possible to mock dependencies
Fixed tests for BrokerageTransactionHandler that now tests what component should do instead of how it operates inside (larger test coverage and less brittle tests).

@mchandschuh - welcome feedback as lightweight submission for the issue #457

I've made a few comments, but overall the approach and design are good. I enjoy seeing things refactored to enable better testing. Great contribution @Phoenix1271!

Removed mentioned internal modifiers and used SetFinishedWarmingUp() method.


[Feature][SDK] Transform SQL support Round function

Description
ROUND(x [,y])：

x: Numbers that need to be rounded. It can be an integer or a floating point number.
y: Optional parameter, indicating the number of decimal places to be rounded.

If y is omitted, it means rounding x to an integer.
If y is a positive number, it means rounding x to y decimal places.
If y is negative, it means rounding x to the nearest y decimal place.

Use case
No response
Are you willing to submit PR?

[X] Yes, I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

FYI, the pull request can be modified many times. You don't need to create a new pull request if you want to make some modifications.

 FYI, the pull request can be modified many times. You don't need to close and create a new pull request if you want to make some modifications.
仅供参考，拉取请求可以修改多次。如果您想进行一些修改，则无需关闭并创建新的拉取请求。

Thank you for your suggestion. I will pay attention to it in the future.


Do not allow invalid save requests to crash test app

Fixes https://github.com/openid/OpenYOLO-Android/issues/27
PR on top of https://github.com/openid/OpenYOLO-Android/pull/74

Codecov Report

Merging #75 into master will increase coverage by 0.01%.
The diff coverage is 50%.

@@             Coverage Diff              @@
##             master      #75      +/-   ##
============================================
+ Coverage     70.94%   70.95%   +0.01%
  Complexity      436      436
============================================
  Files            58       58
  Lines          2437     2438       +1
  Branches        241      242       +1
============================================
+ Hits           1729     1730       +1
+ Misses          641      640       -1
- Partials         67       68       +1

Impacted Files
Coverage Δ
Complexity Δ

...java/org/openyolo/protocol/HintRetrieveResult.java
90.16% <0%> (+1.63%)
9 <0> (ø)
:arrow_down:

...rg/openyolo/protocol/CredentialRetrieveResult.java
91.8% <66.66%> (-1.54%)
10 <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 315c11f...57e469b. Read the comment docs.


dataprocessing should initialize pg tables if not exists

let's do it when postgresql starts for 1st time, create the sql into a ConfigMap and mount it to pg


Bump versions to v2.21.0

DCN config has been split into DCN and DCNv2. Do we need to change the supported methods in README.md?


Managed App Service Certificate Limitations

We do not issue managed certificates for domains with special characters such as фопкасаx.com. Hyphens are allowed though.
can you please update the document to reflect this
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: cc2ee874-df45-1de2-1b30-1fd75c7fd709
Version Independent ID: ee181722-8386-9842-407f-d0549012d2e9
Content: Add and manage TLS/SSL certificates - Azure App Service
Content Source: articles/app-service/configure-ssl-certificate.md
Service: app-service
GitHub Login: @cephalin
Microsoft Alias: cephalin

@motasem13 Thanks for your feedback! We will investigate and update as appropriate.

We hit this as well, but the error message we're saying indicates hyphens are not acceptable and only letters and numbers are allowed. Would be nice to have this called out in the documentation.

Addressed by MicrosoftDocs/azure-docs-pr#186421. #please-close


chore: add pre-commit hooks

Hello,
I would like to propose to add a pre-commit hooks file to the repo, so people can easily import the hook in their pre-commit configuration.
Thx,
Fred

Hi, any chance this gets a review? Thx

Hi @quintush, any chance we can merge this? Thx

Or something like this https://pre-commit.com/

It's the same, the hook file proposed in this PR is to be used by pre-commit.


Fixes #93. Remove #PBS pragmas at NCCS

In case of SLES12 upgrade removing support for #PBS under SLURM 19, this commit should do all SBATCH at NCCS

I've added @GEOS-ESM/chemistry-gatekeepers as a reviewer to this as I've touched the gmichem_setup et al that they deal with. My changes were pretty rote between all the setup scripts.

Also note: probably a good idea for @sdrabenh or @lltakacs to run a forecast/EMIP/plots and the like with this. I demonstrated sbatch gcm_run.j still works as expected, but the forecast and EMIP are much more exciting when it comes to SLURM!


Key press not registered by the Event Viewer?

Hello,
My keyboard does have a 'Fn' key at the right of the right Control key. When activating Karabiner debug, I'm getting the following output when pressing + releasing the key:
Mar 10 09:42:04 Goles kernel[0]: org.pqrs.Karabiner --Debug-- KeyboardEventCallback [ caught]: eventType 12, flags 0x80000000, key 0x00ff, kbdType  40, repeat = 0
Mar 10 09:42:04 Goles kernel[0]: org.pqrs.Karabiner --Debug-- An unknown modifier is pressed (KeyCode:0xff, Flags:0x80000000). Ignore it.
Mar 10 09:42:04 Goles kernel[0]: org.pqrs.Karabiner --Debug-- KeyboardEventCallback [ caught]: eventType 12, flags 0x80000000, key 0x00ff, kbdType  40, repeat = 0
Mar 10 09:42:04 Goles kernel[0]: org.pqrs.Karabiner --Debug-- An unknown modifier is pressed (KeyCode:0xff, Flags:0x80000000). Ignore it.

However when I run the EventViewer nothing is going on on that Key press.. Any ideas of what could be going on here?

Your key (KeyCode:0xff) is ignored by Karabiner.
That key sends same key event at both key up and key down.
Therefore, we cannot use that key to normal usage because we cannot detect physical key up.

@tekezo That's strange, I have another MacBook that I use with this keyboard and I didn't notice this problem. Do you think there's something I can do about it?

I guess it is caused by the third vendor keyboard driver or your keyboard is broken.

@tekezo I'm going to debug this issue in the MacBook were this keyboard is working properly. Is there a way to actually pick the driver myself? Haven't installed anything so I guess OS X did it for me when it tried to recognize this keyboard model.

I don't know well.
Does your fn key work properly in Safe Mode?
If not, you should check your MacBook in Genius Bar.
Mac OS X: Starting up in Safe Mode
http://support.apple.com/kb/ht1455

Maybe I didn't explain myself correctly,
This is an external non-apple keyboard, not my MacBook's keyboard.
Does that change things?
Cheers!
On 3/15/15 7:38 PM, Takayama Fumihiko wrote:

I don't know well.
Does your fn key work properly in Safe Mode?
If not, you should check your MacBook in Genius Bar.
Mac OS X: Starting up in Safe Mode
http://support.apple.com/kb/ht1455
—
Reply to this email directly or view it on GitHub
https://github.com/tekezo/Karabiner/issues/357#issuecomment-81356897.

I see.
Unfortunately, Karabiner does not support the fn key in non-apple keyboards.


Update manage-backup.md

Please reconfirm the added info with the PM group. All I have is an engineer saying "It should be indefinite". @cephalin and @AjayKumar-MSFT have email with that link (although they may not have access to the Teams).

@dcbrown16 : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Thanks @dcbrown16 for raising this PR. @cephalin, for review and help merge the appropriate changes.

Hi @cephalin - Is this pull request from Dec 2019 in progress? Thanks

@cephalin (to refresh notification)

#sign-off

#sign-off


Use API plots data

Use the newly available API plots data instead of sudo JSON
https://issues.redhat.com/browse/COST-4619

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 72.00%. Comparing base (c0ffe40) to head (d675517).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #321   +/-   ##
=======================================
  Coverage   72.00%   72.00%
=======================================
  Files          81       81
  Lines        1050     1050
  Branches      232      232
=======================================
  Hits          756      756
- Misses        282      294   +12
+ Partials       12        0   -12

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


Check if NodeConfiguration matches Status in the database

When you connect INX you can query information about the node. We should store that in the database and check that it matches on each new connection to INX.

Fixed by #241


Replace sql literal regex replace

Description
Replace sql literal regex replace of single quote by using SqlLiteral.toValue() method.

Codecov Report

Merging #6258 (4fac941) into master (1beaab5) will decrease coverage by 2.32%.
The diff coverage is 45.12%.

@@            Coverage Diff             @@
##           master    #6258      +/-   ##
==========================================
- Coverage   66.44%   64.11%   -2.33%
==========================================
  Files        1075     1243     +168
  Lines       54773    58949    +4176
  Branches     8168     8739     +571
==========================================
+ Hits        36396    37798    +1402
- Misses      15700    18392    +2692
- Partials     2677     2759      +82

Flag
Coverage Δ

unittests
64.11% <45.12%> (?)

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

Impacted Files
Coverage Δ

...e/pinot/broker/api/resources/PinotBrokerDebug.java
0.00% <0.00%> (-79.32%)
:arrow_down:

...ot/broker/broker/AllowAllAccessControlFactory.java
71.42% <ø> (-28.58%)
:arrow_down:

.../helix/BrokerUserDefinedMessageHandlerFactory.java
33.96% <0.00%> (-32.71%)
:arrow_down:

...ker/routing/instanceselector/InstanceSelector.java
100.00% <ø> (ø)

...ava/org/apache/pinot/client/AbstractResultSet.java
66.66% <0.00%> (+9.52%)
:arrow_up:

.../main/java/org/apache/pinot/client/Connection.java
35.55% <0.00%> (-13.29%)
:arrow_down:

...inot/client/JsonAsyncHttpPinotClientTransport.java
10.90% <0.00%> (-51.10%)
:arrow_down:

...not/common/assignment/InstancePartitionsUtils.java
73.80% <ø> (+0.63%)
:arrow_up:

.../apache/pinot/common/exception/QueryException.java
90.27% <ø> (+5.55%)
:arrow_up:

...pinot/common/function/AggregationFunctionType.java
98.27% <ø> (-1.73%)
:arrow_down:

... and 1082 more

Continue to review full report at Codecov.

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

For a query with large IN clause, I was able to confirm that we reduced the mean query compilation time from 12ms to 8ms. Thanks for the patch.


Event Handling

In regards to this,
I'd rather not have my events in the index.js. Is there a way, through DKRCommands, to move my event code into a separate file/ folder? I liked how WOKCommands used to have a Features directory where I could store my event listeners. Helped a lot with file/ folder organization and keeping the index.js lean.
Else, I'm open to any ideas you may have :)

Hello,
I have currently published version 1.0.0-beta.3, which allows to load events from a separate folder with the following hierarchy:
> events
  > messageCreate
    > eventFile1.ts/.js
    > eventFile2.ts/.js
    > ...
  > messageDelete
    > eventFile1.ts/.js
    > eventFile2.ts/.js
    > ...

events = events folder
messageCreate/Delete = event folder - must have correct name
each folder can have any number of files (when a given event is called, all of them are triggered)

I'll also update the documentation to reflect these changes in a few minutes.
I'll ask you to test and report any problems 🙂.
Have a nice Friday afternoon

P.S. DKRCommands now use DJS v14.7.1 😉


bump build to hopefully get newer libpq

Rebuild to hopefully get a newer libpq=12 to fix #32
@conda-forge-admin, please rerender
Checklist

[x] Used a fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[ ] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[ ] Ensured the license file is being packaged.

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

checks green and from what I can tell from the azure logs, appears to have picked up libpq=12. Mission Accomplished?

Let's give it a try!


Update passport-github and passport-instagram to 1.0.0

Older versions of these two strategies depend on Passport core 0.1.x, causing outdated versions of req.login to be added to the HttpRequest prototype. This interferes with newer versions of Passport that have different internals.

Let me know if this fixes the issue you mentioned here

Thanks :+1:. That solved the problem.


broken order of uppercase members

In affineplane docs, somehow helm2.HALF and helm3.HALF are next to each other regardless of all lower case members of helm3 are well positioned.

Could not reproduce. Probably fixed now or earlier. Closed for now.


When selecting image from library on Android, uri is wrong.

Description
I'm not sure what's wrong, but when selecting image from my Samsung s20 gallery, uri from callback does not present anything in Image component. It works perfectly with camera and selecting form Google Photos app.
How to repeat issue and example

Use launchImageLibrary({ mediaType: 'photo' }, imagePickerCallback)
It returns uri like content://media/external/images/media/2354
Tried also with base64, still does not work.
It worked on version 2.3.1 with showImagePicker

Solution
Let me know what should I do to have proper uri returned.
Additional Information

Image Picker version: 3.0.0-vnext.3
React Native version: 0.63.2
Platform: Android
Development Operating System: MacOS
Dev tools: Visual Studio Code, Android SDK: 10

What uri are you getting in version 2.3.1 for the same file? For testing purpose don't use any other option other than mediaType: 'photo'.
I have tested this in samsung phone and picking from gallery it seems to work.

Strange, but when juggling through versions I started to work! :) Probably some issue with modules in my app.
Thanks for help :)

I just tested it on apk, and it still not work for release version.
This is what both version returns:
2.3.1
{data: "/9j/4d1uRXhpZgAASUkqAAgAAAANAAABBAABAAAAwA8AAAEBBA…”
fileName: "20201102_131051.jpg",
fileSize: 2417061,
height: 2268,
isVertical: false,
latitude: 49.67026901245117,
longitude: 19.22437286376953,
originalRotation: 90
,path: "/storage/emulated/0/DCIM/Camera/20201102_131051.jpg",
timestamp: "2020-11-02T12:10:52Z"
, type: "image/jpeg"
, uri: "content://media/external/images/media/2354, "
width: 4032
}
3.0.0-vnext.3
{fileName: "20201102_131051.jpg"
, fileSize: 2417061
height: 2268,
type: "image/jpeg"
, uri: "content://media/external/images/media/2354, "
width: 4032}
and uri is the same 🙈

And again I found solution. Image was too big!

@malgorzatamaz please tell me your solution with code! thanks a lot!

@nguyenhoanglam I think that I changed this in build.gradle
com.android.tools.build:gradle:3.5.2