Lombok is not comatible with Android Studio 2021.1.1 Patch 2

I've recently updated my android studio and it says that lombok is not compatible , even when I installed it manually.
My android studio version is: 2021.1.1 Patch 2 Build #AI-211.7628.21.2111.8193401 Feb 17 ,2022

me too

me too

me too

+1 me too

updated zip from idea 2022.1 which seems to work (place inside studio/plugins/ )
lombok.zip

@androidlover5842 I tried it, but it's still not working, maybe Lombok stopped support for Android Studio with the new versions.

@thangtx86 please make sure you have placed lombok inside androidStudio/plugins/lombok/ like this .
Some archive manager make nested folder pleace make sure its extracted like shown in image below

I have a question, if my gradle version is com.android.tools.build:gradle:3.5.4 ,it working ,if it's com.android.tools.build:gradle:4.0.0, it can't generate getter setter, and what you gradle version is?

@jiyuren 7.1.3

your plugin not work for me
lombok-plugin-0.34.1-2022.2.zip this plugin work for me ,but just below gradle version 4.0.0, I want upgrade my gradle version ,and lombok-intellij-plugin build error

@jiyuren you dont have to install it from disk you need to place it inside installed android studio folder /plugins  . It should show as bundled if done correctly

As promised, here is the video with step by step instructions on how to perform the edit and install the plugin. https://youtu.be/A14hrtUeNWU

updated zip from idea 2022.1 which seems to work (place inside studio/plugins/ )
lombok.zip

Thanks a lot!!!

I created a repository for this problem, which automatically exports the built-in Lombok plugin from IDEA Ultimate, hope it can help you: lombok-plugin-repository

I created a repository for this problem, which automatically exports the built-in Lombok plugin from IDEA Ultimate, hope it can help you: lombok-plugin-repository

Thanks, you version fix the issue that android studio could not fetch methods from instance/class which the intellij version has

Thank you sgpublic.


chore(cdc): manually bump tools/check dependency varnamelen

What problem does this PR solve?

Issue Number: close #5964
What is changed and how it works?
On my local development environment, make check always fail caused by varnamelen v0.6.1 checksum mismatch.
manually bump varnamelen to v0.8.0, this can help me solve the problem, please take a try to verify it.
Check List
Tests

No code

Questions
Will it cause performance regression or break compatibility?
Do you need to update user documentation, design documentation or monitoring documentation?
Release note
`None`

I've been confused by the problem for quite a long time, and I've tried some solutions but does not work. for example, go clean -modcache before run make check, this does not work.
I think this should be harmless

/merge

/run-integration-test
/run-leak-test


Add native contract debugging capability.

These changes allow contracts to be compiled natively into a shared library that can then be dynamically linked directly into nodeos. This allows contract code to be debugged using a native platform debugger.  In order to get this working a number of changes to code in eosiolib was necessary.  Most of these are marked with a conditional compilation flag that is tied to the compilation mode (native vs. wasm). The most intrusive of these changes is the renaming of the "time" type to "eosio_time" because of conflicts with a system type of "time". This affects the contract programming interface and all existing contracts that use this type.
Instructions for how to debug contracts can be found in plugins/debug_plugin/README.md.

I spent a lot of time trying to reuse the existing parameters. A number of
factors combined to make me abandon this approach:

The native versions of the intrinsics needed different types than the
non-native versions. The current intrinsic definitions are fairly sloppy
(use of int for pointers and other platform-dependent types) and don't
always agree with the function declarations in eosiolib (the native
intrinsic functions need a more exact match for the linker). A basic goal
of mine was to avoid changing the existing intrinsic macro arguments which
increased the difficulty.
I was unable to use templates in the generation of the native
intrinsic functions because these are defined in an extern "C" context and
templates are not allowed there. Macros are unable to pull apart parameters
like void(int32_t, int32_t).
Because the native intrinsic functions are type-safe (unlike the wasm
intrinsics) I am forced to do some type casting to get things to compile.

You might be able to combine things by completely redefining how the macro
arguments work (as opposed to just adding some new ones), but this seemed a
higher risk approach.
On Thu, Apr 26, 2018 at 12:50 PM, wanderingbort<EMAIL_ADDRESS>wrote:

@wanderingbort requested changes on this pull request.
I would like to take a few hours and see if we can remove the need to
specify more parameters when registering intrinsics.
Other than that, I have no issues so, its worth fixing the merge conflicts
in parallel as I will either retract my review or push changes that
shouldn't conflict with those merges resolutions.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/EOSIO/eos/pull/2577#pullrequestreview-115682704, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AJsuzi4rhvq1DnbL0GQjYMw4Mvi-k8Y-ks5tsgjLgaJpZM4Th0Iw
.

Out of date and a new approach to contact debugging is being explored.


[#170] Rename SlotId and remove LocalSlotIndex

Instead of using LocalSlotIndex to represent the number of slots into an epoch a slot is, we use SlotCount. This has two benefits, the first, expanding the size to a Word64, which is the same size as the EpochSlots type, and second removing an extra unnecessary newtype.
Closes #170

bors r+


Fix: Respects that Leaf can become a Parent.

I ran into this issue when I was creating a tree of checkboxes. A checkbox can be a Leaf but can become a Parent (represents the whole category). In case a new checkbox is assigned to the existing one, I create a children array on the existing one, and put the new one there. That's how Leaf turns into a Parent.
The issue is - I'm not able to access the children of the wrapped Node (Leaf that became Parent) because it was cached without children. This PR illustrates a solution, where the decision - whether the children exist - is moved to the point when they are demanded. However, there is a drawback - enumerable is fixed on true. I believe in the wrapped tree that is not a big deal.
Cheers!

Hmm. I do think that the field “existing” has some value. So adding it everywhere, set to undefined, does seem like quite the break...

Can you replace your nodes when they change, instead of rewriting all their fields?

I'm not used to receiving answers so quickly :) Well, technically, thanks to this "util" I can, but it's a little weird (I wrote the following code  on spot, didn't run it):
addNode( node, parentId )
{
  const parent = getNodeById( parentId ); // finds node in wrapped tree
  if ( ! parent ){ ... }

  const superParent = parent.parent;
  // TODO: Add check that parent has parent, and none of them is the root elements

  const newParent = { ...parent.node, children: [ ...(parent.node.children || []), node] }

  const index = superParent.node.children.indexOf( parent.node );
  superParent.node.children.splice( index, 1, newParent ); // swap parent
}

It does solve the issue (at least I believe so, haven't tested it), but on the other hand, one can't work with wrapped (proxied) tree the same way as with the raw (unwrapped) tree. So this corner-case should be at least put in README.
Having children set to undefined can break node.hasOwnProperty( 'children' ), but does not break ! node.children. I mean yes ... not great :D Either nothing or an array is expected there, so I hope setting it to undefined is not such a big deal. Only affects wrapped tree and provides the benefit of handling it the same way as unwrapped. (Asking for wrappedNode.node.children is still simpler than finding a superParent).
I guess the only solution that doesn't have any of those defects would be implemented via Proxy. However, seeing WeakMap being polyfilled I guess, you're targeting for maximum compatibility.

Regarding manipulation of the .node member - I think I do that, what makes you think otherwise?

Having this discussion forced me to think more about the issue and I agree that swapping a node is the best solution here. However, being the only solution, util suddenly forces the "immutable" pattern, which might be unexpected and requires a deeper knowledge of the tool (initially I didn't know there is a cache, and rewrapping the tree didn't help). That's why I think it could be mentioned in README.
Anyway, I'm closing this one. Thank you for your time and have a great day!


Should CVD matrices be applied to linear RGB or gamma-encoded RGB?

Machado et al. (2009) does not specify exactly how its CVD simulation matrices should be applied to RGB. As pointed out by USAARL Report No. 2020-10, Machado et al. apply the transforms to gamma-encoded RGB values (I was also able to confirm this by trying to replicate the figures). The authors of the technical report apply the matrices to linear RGB values, as is done in Colorspacious. Applying a linear transform to gamma-encoded values doesn't make much sense to me, so I feel that Colorspacious's current approach is correct, but I wanted to see if anyone else had thoughts on this.

The following comment about this from Manuel M. Oliveira was forwarded to me:

According to our experience, applying the matrices to either linear or gamma-encoded RGB did not appear to make any significant difference. Thus, in our paper, we showed results directly on gamma-encoded RGB.

The fact that it didn't make a difference makes sense, since the experiments done in the paper did not use extremely dark colors, where the difference between linear and gamma-encoded RGB values is the largest.
However, as the CVD simulations matrices were derived from the LMS color space, the use of linear RGB values makes the most sense, since a linear transformation can be used to translate LMS values into linear RGB values, unlike the transformation to gamma-encoded RGB, which is non-linear.


After getting PKed being skulled at revs, repeating the trip is unskulled

Send your minion to kill revenants being skulled

Get PKed

Try repeating the trip, realize that the trip is being repeated but unskulled

Fixed in next update.


Ting fixed markdown syntax errors on Chinese README.md

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the master branch of freeCodeCamp.
[x] None of my changes are plagiarized from another source without proper attribution.
[x] My article does not contain shortened URLs or affiliate links.

If your pull request closes a GitHub issue, replace the XXXXX below with the issue number.
Closes #18310

Thank you for opening this pull request.
This is a standard message notifying you that we’ve reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Reason: This PR currently has merge conflicts and since you deleted the fork/branch, we can not easily resolve. Please create a new fork and review the file on master to see if the modifications made in this PR are still needed. If so, please create a new pull request.
Thank you and happy coding.


CONN-9901 | Azure Service Bus Connector AAD authentication

https://github.com/mulesoft/docs-connectors/pull/1781/files verified here


Improved support apply from in gradle files

For now work in progress, should fix https://github.com/dependabot/dependabot-core/issues/2138
I still need to add some tests etc, for now I'm just wondering if there is any interest in getting this merged as a stop-gap solution before the Gradle Tooling is integrated as proposed in https://github.com/dependabot/dependabot-core/issues/1164

@greysteil do you think it's worth it to finish and merge this? I understand #1164 is a better approach, but I don't have the knowledge or time needed to implement that and there seems to be no progress on that issue.

Hey @zbynek, I'm not the right person to ask! @asciimike / @hmarr are likely to have a better view.

@JLLeitschuh might be a good candidate to review this, if available; otherwise, someone on our eng team will take a look at this when free.

A set of unit/integration tests would help me better understand what you're trying to do here

You may want to check for recursion and make sure that your logic doesn't get stuck checking two files that are potentially self-referential.

@JLLeitschuh thanks for checking this! I've added some tests.
There aim of this PR is not to implement full support for apply, only to add support for some usecases.
Improvements in this PR:

resolve property defined in included file and used in (sub)project build file
resolve repositories defined in a file included from the main project file
basic Kotlin syntax apply(from = "dependencies.gradle.kts")

Existing limitations not addressed by this PR:

no support for more advanced syntaxes (e.g. filename defined by property, Koltin apply with mapOf etc.)
no recursive resolution of includes
no support for apply to: (to is ignored)
all included files that are actually fetched must contain dependencies in the filename

all included files that are actually fetched must contain dependencies in the filename

That will require some kind of documentation somewhere too

I don't think you should drop this PR, but I do want to point you towards the more idiomatic way of doing this with Gradle.
We're encouraging our users to move away from using apply from: to using either plugins, or using platforms.
https://docs.gradle.org/current/userguide/platforms.html

The test look sane, so a 👍 for me from here.

@JLLeitschuh thanks. For the docs I created https://github.com/github/docs/pull/10041/files .

The test look sane, so a 👍 for me from here.

@asciimike given the approval from @JLLeitschuh , what are the next steps to get this merged?


initial setup

Initial setup for Kristin to fix and make work

let's do this, ricky! get ready for the lights to flicker!!!


Add a cool melon slab house screenshot to promotional folder

make a cool looking house out of melon slabs and add it to the promotional folder


Update README examples for post-checkout@v1 fetch

This PR updates the docs for newer checkout actions.

Thanks!
Fixed #29


Is there a way to go to a symbol containing hyphens?

Some programming languages allow hypens in their identifiers. The .tags file has those definitions in, but the "go to" seems to "stop" at the hyphen.

Could you provide an example repo or a snippet of code I could reproduce this with?

Example repo: https://github.com/contextfreecode/cobol-explore
Ideally the following should be able to be opened by CTRL-click (the ctags file has those as content):

a file name COPY as in https://github.com/contextfreecode/cobol-explore/blob/814c7415ba94da0c9e0fa1bf5dbf72350ac25f4f/game.cbl#L6
a variable reference, like in https://github.com/contextfreecode/cobol-explore/blob/814c7415ba94da0c9e0fa1bf5dbf72350ac25f4f/game-option.cpy#L20 (has an hyphen)
a procedure reference like in https://github.com/contextfreecode/cobol-explore/blob/814c7415ba94da0c9e0fa1bf5dbf72350ac25f4f/game-stat.cpy#L29 (also with an hyphen)
a sub-program CALL like in https://github.com/contextfreecode/cobol-explore/blob/814c7415ba94da0c9e0fa1bf5dbf72350ac25f4f/hi.cbl#L9-11 (the first one is a "foreign" symbol, the second is a symbol that is defined in the same source)

@GitMensch is this issue still relevant?
Had a quick look today. Some findings:

Removing hyphens (or anything) from editor.wordSeparators does not seem to help. Cobol language definition is required. See down below.
I have to add  --langmap=cobol:+.cpy to the ctags command to make it index .cpy files, e.g.:"ctags-companion.command": "ctags -R --fields=+nKz -f .vscode/.tags --langmap=cobol:+.cpy",

This extension uses getWordRangeAtPosition API which by default uses language specific word definitions.
The problem is that out of the box vscode does not include Cobol language definition so it thinks that .cpy files are Python, .cbl files are Plain Text and so on (i.e. languages where hyphens are a word separator). You can see the detected language in the bottom right corner of the editor.

I can work around this by installing an extension which provides a Cobol language definition. I've picked this one at random: https://marketplace.visualstudio.com/items?itemName=broadcomMFD.cobol-language-support and it seems to work. @GitMensch have you tried using any of the Cobol extensions?

Is this issue still relevant?

Yes.
Looking at your answer, we need both a ctags configuration and support by the word definition.
I'm not sure about this but it seems that one may need to set both https://github.com/Microsoft/vscode-docs/blob/main/api/language-extensions/language-configuration-guide.md#word-pattern wordPattern in the language definition and editor.wordSeparators.
Are you sure that the working go to feature still uses ctags-companion and not the cobol extension?
In any case: should I move the part "Ideally the following should be able to be opened by CTRL-click (the ctags file has those as content" out to a separate issue?

I'm not sure about this but it seems that one may need to set both https://github.com/Microsoft/vscode-docs/blob/main/api/language-extensions/language-configuration-guide.md#word-pattern wordPattern in the language definition and editor.wordSeparators.

I think we can forget about the editor.wordSeparators setting. It does not seem to be relevant after all.
As for wordPattern, it does not look like a user setting. It is something that should be defined in a language extension. For this reason I was suggesting you to try installing an extension which provides Cobol language support.

Are you sure that the working go to feature still uses ctags-companion and not the cobol extension?

I am fairly sure, because if I delete the .tags file it says "no definition found". Also using debugger I can see that getWordRangeAtPosition returns a correct symbol such as option-time rather than just option or time.

In any case: should I move the part "Ideally the following should be able to be opened by CTRL-click (the ctags file has those as content" out to a separate issue?

No, it's okay. The problem scope has not changed.
So all in all @GitMensch I would suggest you explore some of the Cobol extensions and see if it works for you.
Otherwise the next best thing I can come up with is exposing the regex parameter of getWordRangeAtPosition as a setting for you to configure.

Otherwise the next best thing I can come up with is exposing the regex parameter of getWordRangeAtPosition as a setting for you to configure.

That would be useful in any case.

The problem scope has not changed.

Hm, the following has no hyphen in it and still does not work:

Example repo: https://github.com/contextfreecode/cobol-explore
Ideally the following should be able to be opened by CTRL-click (the ctags file has those as content): [...] a sub-program CALL like in https://github.com/contextfreecode/cobol-explore/blob/814c7415ba94da0c9e0fa1bf5dbf72350ac25f4f/hi.cbl#L9-11 (the first one is a "foreign" symbol, the second is a symbol that is defined in the same source)

I have made a release containing a fix for looking up symbols containing hyphens: https://github.com/gediminasz/ctags-companion/commit/235097d7b987333b5e4cb6c2881c2eeb5c0e43ef
In combination with another extension which provides Cobol language definition, I was able to get "jump to definition" to work.
Cobol language extension used: https://marketplace.visualstudio.com/items?itemName=bitlang.gnucobol
I just had to adjust the ctags command setting:
"ctags-companion.command": "ctags -R --fields=+nKz --langmap=cobol:+.cpy",

@GitMensch let me know if it works.


removing text which I think is redundant

we never have time to go through it before we start
we have the overview below and a wall of text moves it away from
screen
time estimates are outdated anyway
the instructor should be able to introduce the schedule in words
without this text based on the schedule and instructor guide

I think I will be going through this since Stefan takes over after the motivation part.


Issue: Not able to get the expected answers when asking answer of other column corresponding to other column

Issue you'd like to raise.
        if file_path.lower().endswith(".xlsx") or file_path.lower().endswith(".xls"):             loader = UnstructuredExcelLoader(file_path, mode="elements")             document = loader.load()             text_splitter = RecursiveCharacterTextSplitter(chunk_size=100, chunk_overlap=10)             texts = text_splitter.split_documents(documents=document)
what can i modify so that I get answer from the column?
Suggestion:
No response

UnstructuredExcelLoader

for your reference, below is my code
if file_path.lower().endswith(".xlsx") or file_path.lower().endswith(".xls"):
           loader = UnstructuredExcelLoader(file_path, mode="elements")
           document = loader.load()
           text_splitter = RecursiveCharacterTextSplitter(chunk_size=100, chunk_overlap=10)
            texts = text_splitter.split_documents(documents=document)```

🤖
Hello again @deepak-habilelabs! It's good to see you're still keen on working with LangChain. Let's dive into your issue.
Based on the provided context, it seems like you're trying to load an Excel file and then split the loaded document into chunks using the RecursiveCharacterTextSplitter class. However, you're facing issues when trying to get answers from a specific column in the Excel file.
Unfortunately, the UnstructuredExcelLoader class you're using is not present in the provided context, so I can't provide specific details about its functionality or how it handles Excel files with multiple columns.
The RecursiveCharacterTextSplitter class, on the other hand, is used to split text into chunks based on specified separators. It doesn't directly interact with an Excel file but is used to process the text data that may be extracted from an Excel file or any other source.
If you're trying to get answers from a specific column in the Excel file, you might need to modify the way you're loading the Excel file or processing the loaded data. For example, you might need to specify the column you're interested in when loading the Excel file, or you might need to process the loaded data in a way that preserves the column information.
Without more information about the structure of your Excel file and the specific issues you're facing, it's difficult to provide a more specific solution. Could you please provide more details about the Excel file (e.g., column names and their respective data types) and the specific issues you're facing? This will help me understand your problem better and provide a more accurate solution.
For more information about the classes and methods used in LangChain, you can refer to the LangChain repository.
Sources

libs/langchain/langchain/document_loaders/unstructured.py
libs/langchain/tests/unit_tests/test_text_splitter.py

This response is meant to be useful and save you time. It is not meant to be a precise solution, but rather a starting point for your own research.
Help me be more useful! Please leave a 👍 if this is helpful and 👎 if it is irrelevant.
If you want to continue the conversation, start your reply with @dosu-bot.

for your reference, below is my code
if file_path.lower().endswith(".xlsx") or file_path.lower().endswith(".xls"):
      loader = UnstructuredExcelLoader(file_path, mode="elements")
       document = loader.load()
       text_splitter = RecursiveCharacterTextSplitter(chunk_size=100, chunk_overlap=10)
        texts = text_splitter.split_documents(documents=document)```


[Docs]: Code Scanner Enhancements

Is there an existing issue for this?

[X] I have searched the existing issues

Engineering Ticket Link
https://github.com/appsmithorg/appsmith/issues/17367
Release Date
Merged into release on Nov 4
First Draft
PR for the changes here
Loom video
It's a pretty straightforward change. We have just added a new property, Scanner Layout, that sets how the code scanner looks.

Discord/slack/intercom Link if needed
No response
PRD
https://github.com/appsmithorg/appsmith/issues/17367
Test plan/cases
No response
Use cases or user requests
Link to Front conversations can be found here.

@spark-c - Could you please look into this?

Published the changes.


Missing try statement for increased validation

A try statement added before the following lines, would help to increase validation in the code because the switches can get disconnected at any time.
https://github.com/kytos-ng/flow_manager/blob/master/main.py#L445-L458
main.py
serializer = FlowFactory.get_class(switch)
flows = flows_dict.get('flows', [])
for flow_dict in flows:
flow = serializer.from_dict(flow_dict, switch)
if command == "delete":

Thank you for addressing this issue @viniarck and reporting on the commits that will fix it. In my opinion, I believe that those fixes and the unit tests you added should be enough for now. More test coverage is always something our team has in mind whenever possible, but I'll leave that decision to @italovalcy and @ajoaoff .


End the stream created when done

Fix #10 and #8
The reason why the test task never ends in watch mode, is that tape.getHarness()._tests.length increases each time the test task finishes. So from the second time, it never ends.
I end the tape output when tests are done, so that the reporter output will come before the gulp task messages and they won't mess.
However, the reporter will never receive messages created when results.close called. So I push them manually.
I'm not quite sure whether we should end the tape output and push the extra messages.
What do you say? @yuanqing

@zoubin Am OK with merging this and cutting a 0.0.7 until we can find a better solution

@yuanqing OK. I'll release 0.0.7 later today.

Done. 0.0.7 released.


List DDS on Bo Haglund's site

http://privat.bahnhof.se/wb758135/bridge/users.html

Email sent.
---------- Forwarded message ---------
From: Kiat Huang<EMAIL_ADDRESS>Date: Mon, 21 Sep 2020 at 22:54
Subject: Bridge Hackathon is a DDS user
To: Bo Haglund<EMAIL_ADDRESS>Cc: Adam Wildavsky<EMAIL_ADDRESS>Hi Bo,
Just wanted to let you know that we are happy users of your DDS code, if you wish to add us to your users page - http://privat.bahnhof.se/wb758135/bridge/users.html
We've put a wrapper around it and made it available as a web service.
https://github.com/online-bridge-hackathon/DDS
--
Kind regards,Kiat
BBO ID: kiatlinkedin.com/in/kiathuang
bit.ly/bridge-hackathon+44-7526-810553

Done


Remove etcd and etcd-backup-restore images

What this PR does / why we need it:
After the introduction of etcd-druid we decided that it does not make sense to let Gardener control the versions of the etcd and etcd-backup-restore images as their manifests/configuration are controlled by etcd-druid. It is necessary for a successful deployment that the version works well with the manifests/configuration.
Consequently, etcd-druid was meanwhile made capable of controlling these versions using the image vector concept (see gardener/etcd-druid#23 and gardener/etcd-druid#24).
Gardener already supports overwriting the image vector for components it deploys, see #2036 and https://github.com/gardener/gardener/blob/master/docs/deployment/image_vector.md#image-vectors-for-dependent-components.
This PR now removes the etcd and etcd-backup-restore images from Gardener so that etcd-druid can take over.
Special notes for your reviewer:
/cc @shreyas-s-rao @swapnilgm @georgekuruvillak - once this PR is merged you need to release a new version of etcd-druid when you want Gardener to use a newer/different version of etcd or etcd-backup-restore. There won't be PRs such as #2260 anymore to this repository. Instead, they should be opened to the gardener/etcd-druid repository. I opened gardener/etcd-druid#50 to track this.
❌Softly depends on #2263
Release note:

Gardener's image vector does no longer contain `etcd` and `etcd-backup-restore`. If you want to overwrite these versions you must do it via the `etcd-druid`. Please consult [this document](https://github.com/gardener/gardener/blob/master/docs/deployment/image_vector.md#image-vectors-for-dependent-components) for more information.

Important note: The version of etcd-druid that Gardener is using is still referencing etcd-backup-restore:0.8.0. Hence, merging this PR without a new etcd-druid version would result in Gardener not deploying the v0.9.0 version.
@shreyas-s-rao @georgekuruvillak @swapnilgm can you please release a new druid version now that https://github.com/gardener/etcd-druid/pull/51 is merged?

Thanks!

What do you mean? I have removed the image field of the Etcd resource, it's part of the PR, so after shoot reconciliation this field will be omitted and the version of druid will be taken.

Since druid will see that for exsiting shoot etcd/backup image is already confgirued in CRD, so if won't apply it from the imagevctor. We will have to delete and recreate old etcd's with image injection.

I don't understand - can you explain what you mean? Did you test it locally? In my tests, I was reconciling a shoot and it didn't restart (as expected, because I didn't specify an imagevector overwrite for druid). Then I added such an imagevector overwrite and I saw druid being restarted. In the subsequent shoot reconciliation also the etcds got restarted.

ok Ignore it. I had misunderstanding about how gardener applies the charts/templates.


Tweak autoClosingPairs to make JavaScript docblock insertion less intrusive

As soon as you type /** you immediately get /**𝖨 */ (where 𝖨 is the cursor). If you didn't intend to start a docblock you need to remove the extraneous */. I'd appreciate two enhancements:

Expand to docblock only after /**<space> or /**<enter>.
Disable the overall feature if you're already inside a comment and maybe if the line already has code before the caret. (Other invalid contexts seem to be detected nicely.)

This is a follow-up to #51047.

Duplicate of #1966


Android - cannot return correct available version

Hi,  thanks for your great work with this plugin.
I'm using version 1.3.0 with capacitor 3.3.3 for 2 android apps. In both cases the available version of my app returned by this code:
AppUpdate.getAppUpdateInfo().then(appUpdateInfo => {
      this.appUpdateInfo = appUpdateInfo;
      console.warn('app update info is ', this.appUpdateInfo)
    })

is not the latest version but an older version. The only thing noticeable about the version that is returned is that it was running Android SDK 29, all versions after this were changed to use Android SDK 30.
I can't see any way to find the logic that is used to get the available version (I assume this is in com.google.android.play:core which I'm using the latest version of 1.10.2). I've tried clearing the Google Play cache on the phone, checking the countries the app is available for, making sure there are no previous versions active in other tracks etc).
My apps are running on Google Play (one in open testing and one in production) and I can update them via Google Play when I push app updates (i.e. the play store recognises that updates are available and allows me to manually install them). I cannot figure out why the plugin does not see the latest version but only a specific older one?!
Example app: app.scafol.io.staging
Shown on the phone using this plugin:

Versions available in Google Play:

Any pointers for resolving this would be greatly appreciated!
thanks
Marcus

Hi @mrobst,
thank you for your interest in this plugin and your detailed request.
I'll try to help you as much as I can.
Unfortunately, I don't have much control over this output, as this output comes from the Play Core library.

In both cases the available version of my app returned by this code: ... is not the latest version but an older version.

In my personal experience, there can be two reasons for this behavior:

The Play Store on your device hasn't learnt about the new update yet (see https://stackoverflow.com/a/56383353).
The device does not support the app version and cannot receive the update.

However, according to your description, neither seems to apply. That is very strange.
I will research what else it could be and let you know.
Does this problem occur only on a few devices and on all devices?

I can't see any way to find the logic that is used to get the available version

See https://github.com/robingenz/capacitor-app-update/blob/38a1e5ad6cef80f96ef19241856ac46498adfe5e/android/src/main/java/dev/robingenz/capacitor/appupdate/AppUpdatePlugin.java#L58 and https://github.com/robingenz/capacitor-app-update/blob/38a1e5ad6cef80f96ef19241856ac46498adfe5e/android/src/main/java/dev/robingenz/capacitor/appupdate/AppUpdatePlugin.java#L71

Hi thanks for the quick reply. I've installed the app on 3 different phones now and I get different values for availableVersion on each one!? One says version 0, one says 10713 and one says 10613. Its very strange :-(
Do you know if there is a way I can raise this as a bug/issue with the Play Core Library team?
thanks
Marcus

As I said, that different devices show different versions is normal, because it may be that on some devices the Play Store background job to update has not yet run through. But after a few days the versions should be in sync (assuming all devices support the update etc.).
Also, to be on the safe side, check out this comment to make sure you're testing it correctly: https://github.com/robingenz/capacitor-app-update/issues/18#issuecomment-886706841

Do you know if there is a way I can raise this as a bug/issue with the Play Core Library team?

To be honest, I'm not 100% sure.
I would use the Google issuetracker.
If you report an issue there, you are very welcome to post the link here so we can track it.

Hi @mrobst,
Do you have any news? Were you able to solve the problem?

Hi - apologies for the lack of feedback from me. The plugin is functionally working, I've been able to check on a couple of phones and it seems that after some time the available update is recognised and I can trigger the in-app update. I have never been able to return a release date for the (new) available version (shown blank in the screenshot above) but this doesn't impact the ability to upgrade.
After the plugin is used to trigger an app update it correctly shows the current version and then the version of future releases. It seems on first install / use an old version is reported as the current version, I never solved why that was :-(
I'll close the issue now as far as I can tell there is nothing for you to change about this plugin, thanks!

@mrobst Thank you for your feedback! This will help me with similar questions in the future.


Disable smart quotes and dashes within embedded script editor

Smart quote and dash replacement is an option in OS X System Preferences. It can be helpful when typing text for humans, but generally frustrates writing scripts because most, if not all, parsers don't recognize the replacement characters as equivalent to the ones typed. The system option seems to affect the script editing text box within Platypus.
To reproduce:
Enable "Use smart quotes and dashes" in System Preferences > Keyboard > Text.
Open Platypus.
Select Bash as Script Type.
Click "New" under the Script Path box.
Delete the single quote at the end of the "Hello, World!" string.
Press the double/single quote key on the keyboard to try to retype it.
If you look closely, after a moment it will replace the plain single-quote/apostrophe character with one that curves to the left (closing). If you save and try to use that script, you'll get a syntax error from bash. (An easier "smart" replacement to see is the replacement of two adjacent hyphens with an "en" dash.)
If it's possible to opt out of this functionality in a particular OS-X–standard (Cocoa?) text box despite it being enabled at the system level, would you please do so?

This has been fixed on the current branch, and will be part of the improvements for the next version of Platypus. That being said, the built-in text editor is never going to be good for anything but very small scale stufff. Platypus is not a text editor and shouldn't aim to be.

Thanks, and I agree.

Verified as fixed in master, will be part of the changes and improvements in version 4.9.1


Usage inside vue-resource interceptor

Hi,
I'm using vue-resource and I'd like to use vue-spinner inside interceptors to show/hide the spinner on any ajax request. I don't know if it's possible to reference a vue-spinner component inside the interceptor. I have done the following but with no success: (I'm using vueify)
var PulseLoader = require('vue-spinner/dist/vue-spinner.min').PulseLoader;

Vue.http.interceptors.push((request, next) => {
    this.loading = true;
   next((response) => {
        this.loading = false;
    });
});

new Vue({
    el: '#grid-manager',
   components: { 'pulse-loader': PulseLoader},

  ....and in the root vue instance i have:

<pulse-loader :loading="loading" :color="color" :size="size"></pulse-loader>

Thanks!

Did you get this figured out?

Not yet @patrickbolle I think i'm going to use other options like this one http://ricostacruz.com/nprogress/

I guess this in the interceptor is in context of interceptor, not the vue main instance.

a good solution is to use a global mixin like
Vue.mixin({
	data() {
		return {
			loading: false
		}
	}
})

then loading is a reactive property and can be used in interceptors


W-GNT Order

2 w-gnt order filled
https://etherscan.io/tx/0xe916bb3b5ca8db758e40b337eafa6fafac96f9d1add8a689f94b0045dd5249c2
sell w-gnt does not show newly acquired 2 w-gnt
where did the w-gnt go?

Well, the w-gnt finally showed up. Looks like it took ~48hrs, is that expected/normal?

@charlieamadeus not really. It seems there was some issue in the network node.
Were you using metamask? That would explain the issue as they've been having problems lately with their nodes.


JSON.SET with NX/XX

Normally Redis has support to JSON.SET key path value [NX | XX] command.
I think this have to implement the dotnet library. We use raw command now but it is not best approach. InsertIfNotExist or InsertIfExist native support will be good. This result could return bool.
true if executed correctly, or false if the specified NX or XX conditions were not met.
Conditions
NX - only set the key if it does not already exist
XX - only set the key if it already exists

I will work on it soon.

@shacharPash Not sure if it is possible, can we also add: AddorUpdate ? this would be great since it would take care of all the checks

@shacharPash Not sure if it is possible, can we also add: AddorUpdate ? this would be great since it would take care of all the checks

Wouldn't AddOrUpdate() effectively be the same as an Insert()...  or are you think of this more for the statefull side where it would replace or patch on the request?

No, i've created https://github.com/redis/redis-om-dotnet/issues/212


Rename bug for derived class method

Brief Description
The generated code of the CSharp tests is uncompileable because one function signature isn't renamed to PascalCase.
Further details
The part which is wrongly generated is this:
class DLL_API HasPropertyWithDerivedType
{
public:
    HasPropertyWithDerivedType();
    HasOverrideOfHasPropertyWithDerivedType* hasPropertyWithDerivedTypeSubclass;
    virtual void causeRenamingError();
};

class DLL_API HasOverrideOfHasPropertyWithDerivedType : public HasPropertyWithDerivedType
{
public:
    HasOverrideOfHasPropertyWithDerivedType();
    virtual void causeRenamingError();
};

The function causeRenamingError in class HasOverrideOfHasPropertyWithDerivedType is not renamed to PascalCase. Furthermore the base class function is spelled wrong too (should be PascalCase too). The base class function is renamed correctly.
PS: The PR #812 cannot be merged due to a renaming bug in the CSharpTests.
Used headers and code files
From the CSharpTest project:
CSharp.h
CSharp.cpp
Stack trace or incompilable generated code
    public unsafe partial class HasOverrideOfHasPropertyWithDerivedType : global::CSharp.HasPropertyWithDerivedType, IDisposable
    {
        public override void causeRenamingError()
        {
            base.causeRenamingError();
        }

    }

    public unsafe partial class HasPropertyWithDerivedType : IDisposable
    {
        public virtual void CauseRenamingError()
        {

        }
    }

(This reduced to the important code)

should be fixed with pr #812

Fixed


gitian: Update broken links to now point to gitian-build.py

@MarcoFalke as per your comment on https://github.com/bitcoin/bitcoin/pull/13706#discussion_r203559422.

utACK 70629d839a4999b31ab33252916cbd8e923d76bf
Thanks!


aur-pkglist: add file integrity checks

https://gitlab.archlinux.org/archlinux/aurweb/-/merge_requests/440

As the checksums only apply to the compresed (.gz) archives, curl --compressed could no longer be used.


Config option to enable setting IP_TOS on the http client socket

Can you describe impact of this change? How this could be leveraged?

@wendigo I will add the impact this change to https://github.com/trinodb/trino this week.
The goal is to prioritize Trino internal traffic, e.g. heartbeats, over HDFS traffic.
Similar to this ceph issue: https://github.com/ceph/ceph/pull/3353
Thanks.

@wenjianhn can you rebase this change to resolve conflicts?


Incorrect comment format display

This is a Bug Report

Problem:
The page incorrect with the comment, such as: https://kubernetes.io/zh/docs/concepts/services-networking/ingress/#%E4%B8%93%E7%94%A8%E6%9C%AF%E8%AF%AD. It was correct in my mac, and the version of hugo is Hugo Static Site Generator v0.62.1/extended darwin/amd64.
Proposed Solution:
Update hugo in server.
Page to Update:
https://kubernetes.io/zh/docs/concepts/services-networking/ingress/#专用术语.

/kind bug
/priority important-soon
I think.

I think this is fixed. @rainingmaster can you confirm?


how to delete an index when removing it from main DB

i want to delete an index of a post when removing post from mysql

As I understood, You want to remove the post from ElasticSearch index after deleting it from DB? If it is so, you should run YourEloquentModel::reindex(); for updating the index's data. If you want to delete index completely, run YourEloquentModel::deleteIndex();

@Pashaster12 tnx man, but is it ok ? reindex after every delete a record from db.

@Blackops404 No, it's not ok))) The best solution is make a cron job with the reindexation which will be run with the periodicity that you want. 5 minute will be ok.

@Pashaster12 tnx again,  how can i delete a document using this Elasticquent, when i want to use deleteDocuments() method laravel says :Call to undefined method

@Blackops404 it's ok, because there is no such function in Elasticquent.

@Pashaster12 thanks helping me ;) i have final question : when i want insert arabic texts sometimes,below error is occured, i have no idea why it's happening, i tried htmlspecialchars() function for cleaning text input but the problem is the same.
it because � chracther but i don't know how to remove it

@Blackops404 what about your charset?


APIM running out of memory/endpoint not destroy issue in admin services

Description:
ScheduledFuture objects are created by the EndpointView instance and there is a destroy method in EnpointView class that should be triggered before deleting an endpoint. This destroys the method cancels the ScheduleFuture task and allows EndpointView class to let the garbage collected. When we are trying to remove an endpoint(Or an API from APIM) through the management console, we have noticed that the EndpointView destroy method does not get hit and it keeps on the heap. In admin service implementation we have noticed that endpoint removal does not call destroy events and due to that reason, these objects do not remove and keep on increasing.
Steps to reproduce:

Start the APIM server.
Create, publish, and delete an API for like 30min.
Take a heap dump and check the growth of the ScheduledFuture objects.

Affected Product Version:
APIM 2.6
Optional Fields
Related Issues:
Fix - https://github.com/wso2/carbon-mediation/pull/1463
Suggested Labels:
APIM, EI

Fixed in master https://github.com/wso2/carbon-mediation/pull/1463


Review: Kallman syndrome

We follow ORDO in placing a number of different OMIM entries as narrower than Kallman
https://www.ebi.ac.uk/ols/ontologies/ordo/terms?iri=http%3A%2F%2Fwww.orpha.net%2FORDO%2FOrphanet_478

consistent with
   is_a MONDO:0018800 ! Kallmann syndrome ***  [xref: DOID:3614] [xref: ICD10:E23.0] [xref: MedDRA:10053142] [xref: MESH:D017436] [xref: NCIT:C75479] [xref: Orphanet:478] [xref: SCTID:93559003] [xref: UMLS:C0162809]
      is_a MONDO:0007844 ! hypogonadotropic hypogonadism 2 with or without anosmia [xref: DOID:0090083] [xref: ICD10:E23.0] [xref: OMIM:147950] [xref: UMLS:C1563720]
      is_a MONDO:0009482 ! hypogonadotropic hypogonadism 3 with or without anosmia [xref: DOID:0090092] [xref: ICD10:E23.0] [xref: OMIM:244200] [xref: UMLS:C2930927] [xref: UMLS:C3550478]
      is_a MONDO:0010635 ! hypogonadotropic hypogonadism 1 with or without anosmia [xref: DOID:0090094] [xref: ICD10:E23.0] [xref: NCIT:C75480] [xref: OMIM:308700] [xref: UMLS:C1563719]
      is_a MONDO:0012528 ! hypogonadotropic hypogonadism 4 with or without anosmia [xref: DOID:0090077] [xref: ICD10:E23.0] [xref: OMIM:610628] [xref: UMLS:C1857720] [xref: UMLS:C3552343]
      is_a MONDO:0012880 ! hypogonadotropic hypogonadism 5 with or without anosmia [xref: DOID:0090084] [xref: ICD10:E23.0] [xref: OMIM:612370] [xref: UMLS:C2675302] [xref: UMLS:C3552553]
      is_a MONDO:0012988 ! hypogonadotropic hypogonadism 6 with or without anosmia [xref: DOID:0090086] [xref: ICD10:E23.0] [xref: OMIM:612702] [xref: UMLS:C2675188] [xref: UMLS:C3552574]
      is_a MONDO:0013910 ! hypogonadotropic hypogonadism 8 with or without anosmia [xref: DOID:0090074] [xref: ICD10:E23.0] [xref: OMIM:614837] [xref: UMLS:C3553841]
      is_a MONDO:0013911 ! hypogonadotropic hypogonadism 9 with or without anosmia [xref: DOID:0090085] [xref: ICD10:E23.0] [xref: OMIM:614838] [xref: UMLS:C3553842]
      is_a MONDO:0013913 ! hypogonadotropic hypogonadism 11 with or without anosmia [xref: DOID:0090071] [xref: ICD10:E23.0] [xref: OMIM:614840] [xref: UMLS:C3553844]
      is_a MONDO:0013926 ! hypogonadotropic hypogonadism 14 with or without anosmia [xref: DOID:0090087] [xref: ICD10:E23.0] [xref: OMIM:614858] [xref: UMLS:C3540450]
      is_a MONDO:0013946 ! hypogonadotropic hypogonadism 15 with or without anosmia [xref: DOID:0090075] [xref: ICD10:23.0] [xref: OMIM:614880] [xref: UMLS:C3553977]
      is_a MONDO:0013961 ! hypogonadotropic hypogonadism 16 with or without anosmia [xref: DOID:0090080] [xref: ICD10:E23.0] [xref: OMIM:614897] [xref: UMLS:C3554021]
      is_a MONDO:0014102 ! hypogonadotropic hypogonadism 17 with or without anosmia [xref: DOID:0090079] [xref: ICD10:E23.0] [xref: OMIM:615266] [xref: UMLS:C3808971]
      is_a MONDO:0014103 ! hypogonadotropic hypogonadism 18 with or without anosmia [xref: DOID:0090076] [xref: ICD10:E23.0] [xref: OMIM:615267] [xref: UMLS:C3808975]
      is_a MONDO:0014105 ! hypogonadotropic hypogonadism 19 with or without anosmia [xref: DOID:0090090] [xref: ICD10:E23.0] [xref: OMIM:615269] [xref: UMLS:C3808981]
      is_a MONDO:0014106 ! hypogonadotropic hypogonadism 20 with or without anosmia [xref: DOID:0090082] [xref: ICD10:E23.0] [xref: OMIM:615270] [xref: UMLS:C3808983]
      is_a MONDO:0014107 ! hypogonadotropic hypogonadism 21 with or without anosmia [xref: DOID:0090093] [xref: ICD10:E23.0] [xref: OMIM:615271] [xref: UMLS:C3808986]
      is_a MONDO:0014461 ! hypogonadotropic hypogonadism 22 with or without anosmia [xref: DOID:0090081] [xref: ICD10:E23.0] [xref: OMIM:616030] [xref: UMLS:C4014988]

However, there is an issue here; let's take https://omim.org/entry/610628 (HH4)
"In the presence of anosmia, idiopathic hypogonadotropic hypogonadism has been called 'Kallmann syndrome (KS),' whereas in the presence of a normal sense of smell, it has been termed 'normosmic idiopathic hypogonadotropic hypogonadism (nIHH)' (summary by Raivio et al., 2007). Because families have been found to segregate both KS and nIHH, the disorder is here referred to as 'hypogonadotropic hypogonadism with or without anosmia (HH)"
so it's seems incorrect to place the HHs under KS, as the true path rule is violated.
The ORDO def for KS is
Kallmann syndrome (KS) is a developmental genetic disorder characterized by the association of congenital hypogonadotropic hypogonadism (CHH) due to gonadotropin-releasing hormone (GnRH) deficiency, and anosmia or hyposmia (with hypoplasia or aplasia of the olfactory bulbs)
Note that GARD calls this OMIM ID "KAL4" and "Kallman syndrome 4"
https://rarediseases.info.nih.gov/diseases/10772/kallmann-syndrome-4
In HPOA we have an 'occasional' annotation of this OMIM to Anosmia
One possibility would be to make two subtypes of every HH[n]: KS[n] and niHH[n]. However, this seems like overkill.
It may be better to place HH[n] under the more general MONDO:0018555 ! hypogonadotropic hypogonadism
For now, we will follow ORDO and place the HH[n]s under KS, even though each may not have all the classic KS symptoms

🎂 happy 3rd birthday, ticket!
h/t @monicacecilia
we will continue to follow ORDO as mentioned above. We can consider re-reviewing at a later time. Closing for now.


terratest_log_parser does not retain a lot of useful test and failure information

Describe the bug
I would expect the report.xml generated by terratest_log_parser to contain the reason for the test failure.
However, terratest_log_parser loses the assertion results
To Reproduce

go test -count=1 -v | tee test_output.log

The above command created the following lines in the test_output.log:
--- FAIL: TestHelloWorldTerraform (0.74s)
    terraform_hello_world_test.go:193:
        	Error Trace:	/mnt/c/terratest/terraform_hello_world_test.go:193
        	Error:      	Not equal:
        	            	expected: "Hello foo eRcjAe"
        	            	actual  : "Hello eRcjAe"

        	            	Diff:
        	            	--- Expected
        	            	+++ Actual
        	            	@@ -1 +1 @@
        	            	-Hello foo eRcjAe
        	            	+Hello eRcjAe
        	Test:       	TestHelloWorldTerraform
FAIL
exit status 1

terratest_log_parser -testlog test_output.log -outputdir test_output --log-level info

The above command created the following report.xml:
<?xml version="1.0" encoding="UTF-8"?>
<testsuites>
	<testsuite tests="1" failures="1" time="0.804" name="github.com/company/terratest">
		<properties>
			<property name="go.version" value="go1.11"></property>
		</properties>
		<testcase classname="terratest" name="Failure" time="0.000">
			<failure message="Failed" type="">    terraform_hello_world_test.go:193: </failure>
		</testcase>
	</testsuite>
</testsuites>

Expected behavior
I would expect the report.xml to include

the testcase name == test i.e. the go test function name TestHelloWorldTerraform
the failure body == assertion error

Versions

Terratest version: 1.7.5
Environment details (Ubuntu 20.04, Windows 10, etc.): WSL2

Upon further investigation, I discovered that we are using the github.com/jstemmer/go-junit-report/parser package to generate the XML report. Unfortunately, this package does not offer an option to include assertion error details, and it appears that we are unable to do so either. There is an open issue related to this on GitHub (#181), which was created on March 26, but it has not yet been addressed. https://github.com/jstemmer/go-junit-report/issues/181
I'll close this issue as there's nothing we can do on this yet. Let's reopen this when this ticket is addressed.


[nlopt] Update to 2.6.2

Describe the pull request
Updates port NLopt to latest version 2.6.2.
Enables support for building shared libraries, as these should work fine since https://github.com/stevengj/nlopt/commit/101c69f5d4819e7445f8645456e744e14aaa2dd6.

Which triplets are supported/not supported? Have you updated the CI baseline?

CI baseline already contains entries for UWP.

Does your PR follow the maintainer guide?

Yes.

LGTM! Thanks @rickertm :)


fix the command output

I have verified on version v1.11

^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Please delete this note before submitting the pull request.
For 1.12 Features: set Milestone to 1.12 and Base Branch to release-1.12
Help editing and submitting pull requests:  https://deploy-preview-9510--kubernetes-io-master-staging.netlify.com/docs/contribute/start/#submit-a-pull-request.
Help choosing which branch to use, see
https://kubernetes.io/docs/contribute/start#choose-which-git-branch-to-use.
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

@tanshanshan  Thanks for the fix. I checked the output on: https://deploy-preview-10307--kubernetes-io-master-staging.netlify.com/docs/concepts/cluster-administration/manage-deployment/
/lgtm

@janetkuo @kow3ns can you lgtm for tech review please? Thanks!

/assign

/approve


Problems with npm install and windows

Node: 8.9.1
OS: Windows 10, 64bit, v1709
Microsoft Build Engine version: 14.0.24720.0
CMAKE: 3.11.0-rc1
I just tried to install face-recognition and I receive this error:
>  gyp ERR! build error gyp ERR! stack Error: C:\Program Files (x86)\Microsoft Visual Studio\2017\Community\MSBuild\15.0\Bin\MSBuild.exe` failed with exit code: 1
gyp ERR! stack     at ChildProcess.onExit (C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\lib\build.js:258:23)
gyp ERR! stack     at emitTwo (events.js:126:13)
gyp ERR! stack     at ChildProcess.emit (events.js:214:7)
gyp ERR! stack     at Process.ChildProcess._handle.onexit (internal/child_process.js:198:12)
gyp ERR! System Windows_NT 10.0.16299
gyp ERR! command "C:\Program Files\nodejs\node.exe" "C:\Program Files\nodejs\node_modules\npm\node_modules\node-gyp\bin\node-gyp.js" "rebuild"
gyp ERR! cwd C:\Users\Rafael Albuquerque\Documents\workspace\visao\node_modules\face-recognition
gyp ERR! node -v v8.9.1
gyp ERR! node-gyp -v v3.6.2
gyp ERR! not ok
npm WARN<EMAIL_ADDRESS>No repository field.
npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>install: node-gyp rebuild
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>install script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.
npm ERR! A complete log of this run can be found in:`
I have both node-gyp and rebuild installed globally.
Also, during the install, I receive a lot of warnings.
Ex:
c:\users\rafael albuquerque\documents\workspace\visao\node_modules\face-recognition\cc\FaceRecognizerNet.h(2): fatal error C1083: Not possible open the file include::  'dlib/dnn.h': No such file or directory (compilando arquivo de origem ..\cc\FaceRecognizerNet.cc) [C:\Users\Rafael Albuquerque\Documents\workspace\visao\node_modules\ face-recognition\build\facerec.vcxproj]

Did the auto build of dlib complete without any errors?

Yep. No errors and no warnings. I already tried to change SDK, change vs version(2017 -> 2015) and nothing seems to work.

Odd, so if the auto build really finished successfully then you should have a clone of the dlib repo in node_modules/dlib_build/dlib and a dlib.lib file should exist in node_modules/dlib_build/build/dlib/Release/lib.
If you do not have these, then I need the full npm log and you could try to npm install dlib-build again. If you do, then there is either something wrong with the dlib include path on your system or you have set a DLIB_INCLUDE_DIR env variable, which is pointing to a wrong location.
By the way vs2015 won't work, stick with vs2017.

Yep, like I say, no errors and no warnings.

Do you want dlib logs or face-recognition  logs?

Nops. I tried a lot of things and in the end, I just gave up.  I decided to use another solution to resolve my problem with face recognition.

+1 same exact problem.

Not sure if thats the issue, but whenever people report that issue, they have a whitespace in their path.

You were 100% correct. I had my project in a folder called "Visual Studio". That space is what did it. It's working now, thanks for the suggestion!

@justadudewhohacks i'm stuck with the same error.  Which path should I check for spaces?  I've tried everything from removing/reinstalling node node-gyp changing windows 10 sdk version etc...  Any idea what else I can check?

Actually I am talking about the path, where you execute node/npm, your current working directory. Apparently fs.exists doesn't like paths with whitespaces in it. So, you are also confronted with fatal error C1083: Not possible open the file include:: 'dlib/dnn.h': No such file or directory?

@justadudewhohacks it seemed to be the same error at first as it complained about gyp-rebuild but i'm not entirely sure.  I've attached my output log.  This seems to be a real pain to build in windows, it took me a day to try compile openBlas and I eventually found a pre-compiled (older version) but not sure at
node-face-recognition-errorLog.txt
all how to proceed from here with this node-gyp error...

i'm using the auto build dlib package.  Should I try build it myself?

That's odd, usually with the default cmake configuration dlib should automatically configure this for you. Maybe this issue  helps you. If you need to set cmake specific flags I would suggest you to build dlib on your own.

i've never used libpng, what I took from that was that I should make sure I build for 32/64bit?  Doesn't node handle the dlib build during auto build?  If so how can I see which architecture it's trying to build for?

What I wanted to say is, that it seems like on your machine dlib is linked against a custom libpng instance. I think by default dlib should just build libpng from it's externals folder, but it looks like it does not build or link it properly in your case.
I would try to build dlib on your own first, or watch the cmake output of the auto build for some libpng specific output.


usWinAscent & usWinDescent greater than bbox?

In some fonts, the usWinAscent, usWinDescent are greater than the yMax, yMin. Personally, I can't see this causing any problems. Could we change check com_google_fonts_test_040 to pass these cases? it should still fail if the values are less than yMax, yMin because clipping will occur on some Win apps.

Yeah afaik this will not cause a problem, just additional white space,
which is required for short family members to have consistent line heights
On Nov 24, 2017 3:21 AM, "Marc Foley"<EMAIL_ADDRESS>wrote:

In some fonts, the usWinAscent, usWinDescent are greater than the yMax,
yMin. Personally, I can't see this causing any problems. Could we change
check com_google_fonts_test_040 to pass these cases? it should still
fail if the values are less than yMax, yMin because clipping will occur on
some Win apps.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/googlefonts/fontbakery/issues/1655, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAP9y-4ajLlJGZ8_MAku9ueCdfoKDXS6ks5s5pi-gaJpZM4QpoJE
.

@davelab6 agreed. However, If a family has already been released with win metrics which are greater than the bbox, I think we should keep them as they are.

It is often required that win ascent/descent be greater than font bbox to account for combining marks. The check should test that the ascent/descent values are not smaller thank bbox, larger values can be a warning or just silently accepted.

thanks, @m4rc1e and @khaledhosny !


Allow the package to be required directly to template

template: require('html-webpack-template') should work now.
Closes #9.

This was just a matter of resolving the path to the template. So the implementation is simpler than what I had in mind initially. The change is backwards-compatible.

Any chance you could have a quick look at this over the weekend? It's a simple change, but it would help a lot with npm-install-webpack-plugin given it would allow the system to catch the require and get html-webpack-plugin installed automatically.

Could you have a quick look? Seems simple enough. Sorry to pester. :smile:.

Gentle poke.

Hey, cool feature.  Thanks for the add.  Nice work on the simpler implementation.  I was not seeing notifications for this, so thanks for poking me on twitter.

@bebraw Now published as v5.0.0.  Try it out, and let me know how your the new method works.

@jaketrent Works great. Thanks! 👍


MM-32162 Review and Update mmctl product docs

Documentation task: https://mattermost.atlassian.net/browse/MM-32162
Updated:

Reviewed all mmctl commands against the 'mmctl' repository and updated where needed.

Thank you so much, @justinegeffen, for reviewing this huge PR for the mmctl documentation.

@cwarnermm I've ported back to the mmctl autogenerated docs some of the changes of this PR on https://github.com/mattermost/mmctl/pull/348


Example in the readme for defining your own queries does not work for (built in) treesit

I think the example in the Readme for defining your own queries does not work for treesit because it expects slightly different queries.
(https://github.com/meain/evil-textobj-tree-sitter#custom-textobjects)
Says
evil-motion-range: Wrong type argument: treesit-query-p, [(import_statement) @import]

for example.
By exchanging the [] with () treesit-query-p is satisfied:
  (define-key evil-outer-text-objects-map "i" (evil-textobj-tree-sitter-get-textobj "cond"
                                              '(
                                                (c-ts-mode . ((if_statement) @cond))
                                                )))

Thank you. Looks like ( works for tree-sitter and treesit and so I've change the readme to use that instead.


Where should I download the latest release from?

I saw the release you did in github here: https://github.com/verronpro/docx-stamper/releases/tag/v1.6.5
But I cannot see the artifact on maven central. E.g. https://central.sonatype.com/artifact/pro.verron/docx-stamper/ redirects me to the previous version https://central.sonatype.com/artifact/pro.verron/docx-stamper/1.6.4

Normally by publishing into s01.oss.sonatype.org, the sync should be done to central in a few hours after my manual validation. I can't access the s01 server right now, it keeps sending me error message, so i'll checkout later when it will be less capricious.


Links to search queries no longer seem to work on Firefox

I have started encountering this issue recently. This reduces the convenience of using the website as an address bar search engine.
Example link:
https://source.dot.net/#q=Attribute (or any other search link, really)
Navigating to the link doesn't seem to automatically fill in the search field. No errors in the console.

Additional observations:

Navigating to the same link in a new tab doesn't help
Navigating to the same link in the same tab doesn't help
Navigating to the same link in the same tab and then in a new tab correctly fills in search only in the new tab

Firefox version: 65.0.1 (64-bit) (Win10-x64)
I haven't been able to reproduce this issue in Chrome, Edge or Vivaldi.

This issue was caused by the Consent Manager addon overriding window.onload.


Updating custom toolbar to display colour of 'selected' content.

Hello,
I'm trying to work out the best way of updating my toolbar to style the colour picker with the colour of the current selection. I noticed that it works in the main example, but I can't for the life of me work out how.
Ideally, I'd like an event so I can update the toolbar with the selected font size and colour.
Has anyone achieved this already, and if so, how?
Thanks
Joel

I managed to figure this out by delving into the source a bit more (though I'm on an older version than latest).
Here's how I did it;
// Set up an event watcher on any interaction so we can update the colour
editor.on('interaction', function(){
    var color = getEditorColor();
    // Set your toolbar colour here
});

getEditorColor: function(){
        var el = editor.composer.commands.state("foreColorStyle");
        // If it found elements, we want the first one
        if ( el ) {
            el = el[0];
        }
        // Couldn't find a forColorStyle
        else {
            return false;
        }

        // Get the styles using getComputedStyle
        var styles = window.getComputedStyle(el);

        return styles.color; // rgb(200,0,0)
}

I'm also going to implement this for font size too


Add import progress report

Related issue: 7220
Although the progress report is present for the export operation, it was missing for the import. It's been added for both for feature consistency and for allowing users to know how long the operation is going to take.

@pixcc @maximyurchuk , is that something that that should be fixed on my end ?
FYI, I did run the ya test command but it was taking so long that stopped and went ahead and pushed.

@NeimadTL Thank you for your contribution! Going to review it as fast as possible

@NeimadTL Thank you for your contribution! Going to review it as fast as possible.

Awesome ! you're welcome.

@pixcc, when reviewing please take into account my last commit's message. If my guess is wrong, the other suggestion I have is to do TPathId ImportPathId = InvalidPathId; on the TImportInfo struc defined in ydb/core/tx/schemeshard/schemeshard_info_types.h #L2776

@pixcc, just pushed the requested changes. Thanks again for your help. Hopefully this time around, it'll be good.

@NeimadTL Unfortunately the test you created failed again, could you please try to debug it?

@NeimadTL Unfortunately the test you created failed again, could you please try to debug it?

Yes, will try to.

@NeimadTL Unfortunately the test you created failed again, could you please try to debug it?

Yes, will try to.

Hey, how it is going? Need help?

@NeimadTL Unfortunately the test you created failed again, could you please try to debug it?

Yes, will try to.

@NeimadTL Hey, how it is going? Need help?

Hey @pixcc , thanks for reaching out. Been sick since last Monday so I haven't been programming. is the best way to get you live (instant messaging) Discord ? Will comme to this work soon.


Humidity sensors to connect to home automation

With the humidity in Georgia, mold is more of a concern than before. Just to begin with getting more data points I'm interested in measuring over time.

Got Aeotec aërQ, wireless, battery powered, SmartThings sensor, Z-Wave Plus and hooked with HoA. So far so good (been only a few hours). Running inclusion took a few attemps (z-wave standalone).
Looks good so will buy a few more.


feat(thermocycler-gen2): update hardware controller

Updates hardware controller to use AsyncResponseSerialConnection

I think you want to target this to edge

I think you want to target this to edge

No, unfortunately we're continuing to try and develop the TCGEN2 release independent of edge

Okay, we should target this on edge and once merged we'll cherry-pick to https://github.com/Opentrons/opentrons/pull/11607

Or we can target it on https://github.com/Opentrons/opentrons/pull/11607 and it'll come back in when we merge that back

Retargeting to edge since everything else for TCGEN2 is in there, and we're cherrypicking anyway.


Allow == operator for variant type conditions

Currently, to use branching on variant types, you have to use the === operator. If you change it to == in the below code Flow won't acknowledge the refinement.

type Opts = {bzt:{}}

type Variant = { type: 'a', opts_a: Opts } | { type: 'b', opts_b: Opts }

function f1 (val : Variant) : Opts {
  if(val.type === 'a') {
    return val.opts_a
  }
  else {
    return val.opts_b
  }
}

This is a gotcha. There should be a warning advising to use ===, or better yet Flow should allow == here and prevent cases where == and === would have different results.
Using version af67541

This is expected behavior


adds a dial method and associated spec to initiate a SIP call

Please consider this opentok.sip.dial() method, a spec and README.  I have tested this code with the Twilio SIP provider.  Please suggest any improvements.  Thanks!

thank you @chess1424. please let me know if there is any other work i can do to facilitate review.

👍 would love to see this get merged


Fix the responsiveness of the page

closes #133 [BUG] : Horizontal bar appearing on smaller screen

well done @md-abid-hussain


onEnter checks for routes and serverside api proxy?

As routing and server files are not exposed I was wondering if it is possible and how can we achieve following tasks?

Add onEnter check to the routes
Adjust server to proxy our api calls so we can use something like /api on client side to mask it.

Add onEnter check to the routes

I wonder if componentWillMount is enough ?

Adjust server to proxy our api calls so we can use something like /api on client side to mask it.

It's addressed on https://github.com/zeit/next.js/issues/25
I'm not sure if we support it, but you can run Nginx or such to proxy requests of next.js server and API server from a domain.

Closing in favor of #25.


android build results do not contain actually bound library

The resulting nuget file of the facebook android binding consists of mauifacebook-Release.aar, but does not contain the referenced aar or jar files. Because of this, the nuget fails when being loaded on runtime (complaining that classes can not be found). This is reasonable, as these dependencies are not included in the nuget.

Realized that this is by design. Native stuff can be added using the AndroidLibrary element.


Plug in stop after running overnight

I just try to install your plug in to my homebridge which is running on Raspberry Pi. However, after a while, home bridge just quit and show the following message:
crypto.js:160
var ret = this._handle.final();
^
Error: error:0606508A:digital envelope routines:EVP_DecryptFinal_ex:data not multiple of block length
at Decipheriv.final (crypto.js:160:26)
at Socket.cs.on (/usr/lib/node_modules/homebridge-broadlink-platform/node_modules/broadlinkjs-sm/index.js:234:27)
at emitTwo (events.js:106:13)
at Socket.emit (events.js:194:7)
at UDP.onMessage [as onmessage] (dgram.js:544:8)
Could you please help me with the issue?

are you using another broadlink plugin? I'm experiencing the same issue

The issue lies within the broadlinkjs-sm module https://github.com/smka/broadlinkjs-sm


[tool_crash] _CastError: (#0      describeLocale (package:flutter_tools/src/localizations/localizations_utils.dart:219:49))

Command
flutter --no-color run --machine --track-widget-creation --device-id=adb-RZCT50NXZTD-kDeMVx._adb-tls-connect._tcp. --start-paused --dart-define=flutter.inspector.structuredErrors=true lib/main.dart

Steps to Reproduce

...
...
...

Logs
_CastError: (#0      describeLocale (package:flutter_tools/src/localizations/localizations_utils.dart:219:49))
#0      describeLocale (package:flutter_tools/src/localizations/localizations_utils.dart:219:49)
#1      LocalizationsGenerator._generateBaseClassFile (package:flutter_tools/src/localizations/gen_l10n.dart:1137:34)
#2      LocalizationsGenerator._generateCode (package:flutter_tools/src/localizations/gen_l10n.dart:1238:46)
#3      LocalizationsGenerator.writeOutputFiles (package:flutter_tools/src/localizations/gen_l10n.dart:1313:47)
#4      generateLocalizations (package:flutter_tools/src/localizations/gen_l10n.dart:69:9)
#5      GenerateLocalizationsTarget.build (package:flutter_tools/src/build_system/targets/localizations.dart:56:5)
#6      _BuildInstance._invokeInternal (package:flutter_tools/src/build_system/build_system.dart:839:27)
<asynchronous suspension>
#7      FlutterBuildSystem.build (package:flutter_tools/src/build_system/build_system.dart:606:16)
<asynchronous suspension>
#8      generateLocalizationsSyntheticPackage (package:flutter_tools/src/dart/generate_synthetic_packages.dart:58:30)
<asynchronous suspension>
#9      FlutterCommand.verifyThenRunCommand (package:flutter_tools/src/runner/flutter_command.dart:1318:7)
<asynchronous suspension>
#10     FlutterCommand.run.<anonymous closure> (package:flutter_tools/src/runner/flutter_command.dart:1183:27)
<asynchronous suspension>
#11     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#12     CommandRunner.runCommand (package:args/command_runner.dart:209:13)
<asynchronous suspension>
#13     FlutterCommandRunner.runCommand.<anonymous closure> (package:flutter_tools/src/runner/flutter_command_runner.dart:281:9)
<asynchronous suspension>
#14     AppContext.run.<anonymous closure> (package:flutter_tools/src/base/context.dart:150:19)
<asynchronous suspension>
#15     FlutterCommandRunner.runCommand (package:flutter_tools/src/runner/flutter_command_runner.dart:229:5)

[✓] Flutter (Channel stable, 3.0.3, on macOS 12.4 21F79 darwin-arm, locale en-NP)
    • Flutter version 3.0.3 at /Users/kapil/flutter/flutter
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision 676cefaaff (6 days ago), 2022-06-22 11:34:49 -0700
    • Engine revision ffe7b86a1e
    • Dart version 2.17.5
    • DevTools version 2.12.2

[✓] Android toolchain - develop for Android devices (Android SDK version 32.0.0-rc1)
    • Android SDK at /Users/kapil/Library/Android/sdk
    • Platform android-32, build-tools 32.0.0-rc1
    • Java binary at: /Applications/Android Studio.app/Contents/jre/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 13.4)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • CocoaPods version 1.11.3

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2021.2)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 11.0.12+0-b1504.28-7817840)

[✓] VS Code (version 1.66.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.36.0

[✓] Connected device (3 available)
    • SM M336BU (mobile) • adb-RZCT50NXZTD-kDeMVx._adb-tls-connect._tcp. • android-arm64  • Android 12 (API 31)
    • macOS (desktop)    • macos                                         • darwin-arm64   • macOS 12.4 21F79 darwin-arm
    • Chrome (web)       • chrome                                        • web-javascript • Google Chrome 103.0.5060.53

[✓] HTTP Host Availability
    • All required HTTP hosts are available

• No issues found!

Flutter Application Metadata
Type: app
Version: 1.0.0+1
Material: true
Android X: false
Module: false
Plugin: false
Android package: null
iOS bundle identifier: null
Creation channel: stable
Creation framework version: 676cefaaff197f27424942307668886253e1ec35

Hi @kapilgritfeat,
This has been fixed as part of https://github.com/flutter/flutter/issues/87851 so the fix should be available in the latest master.
So I suggest you switch to the latest master and try the command again to see the fix.
flutter channel master
flutter upgrade
Closing as this is fixed and available on master already.


Error after upgrading packages from 1.2.1 to 1.3.0

Someone already raised a query regarding it, Below is the link:
https://stackoverflow.com/questions/62029936/i-have-a-couple-of-problems-when-i-upgraded-packages-in-my-flutter-app

Looks like your flutter sdk needs to be upgraded. In the old version, onTap property isn't defined in Dropdown.


Debugging features

Is your feature request related to a problem? Please describe.
It's really hard to investigate what's going on when things don't play the way I'd expect
Describe the solution you'd like
A method allowing to print a string to the OLED display would be ideal for debugging purposes.

Is your feature request related to a problem? Please describe.
It's really hard to investigate what's going on when things don't play the way I'd expect
Describe the solution you'd like
A method allowing to print a string to the OLED display would be ideal for debugging purposes.

or even printing to the 7-segment display might often be better since you will be using the OLED display for setting parameters.

or even printing to the 7-segment display might often be better since you will be using the OLED display for setting parameters.

Yes. Anything that can be used to display something :)

Hey @ptarabbia I managed to get the Korg waves demo oscillator compiled with web assembly using the alpha/wasm-builds branch code and following the documentation there. I can now add standard string output to the oscillator code with printf or std::cout, load it in the browser simulator, and it outputs strings in the Chrome debug console. It's probably not relevant here, but the documentation says it is untested in Windows Subsystem for Linux but I had no problem with that. I'm also successfully building oscillators with gcc under WSL.

Is there any success in configuring a way to build and debug in vs code?  I'll see about looking into alpha/wasm-builds branch to load it in the browser simulator but was just curious on any attempts to set this up in vs code.


FIx docstring and type hints

(thx mkdocs)

I have fixed more or less the same things in another PR #354

@ecomodeller, ok to close this?


HideUtilityClassConstructorCheck

Makes sure that classes that contain only static methods or fields do not have a constructor

Name is a bit odd though.
@AlexHaxe
@Gama11

any other good name?

How about UnnecessaryConstructorCheck? That could then be generalized to also check for unnecessary empty constructors in child classes, which was on my TODO-list anyway. :)

renamed to UnnecessaryConstructorCheck
Need a better description and message when this is extended for constructor check in sub classes.


Few improvements for constants in visual shader

Changes:

Removed drop-down list from FloatConstant - instead added a new option 'Float Constants' to the context menu:

maybe it should be named Predefined constants but I think it's also fine

Changed constants caption like: Boolean -> BooleanConstant, Vector -> VectorConstant etc. to prevent confusion for the users

Thanks!

@Chaosus Do you think this context menu and functionality is easy to use in VisualScript as well?
If you point me in the right direction I will try to take it from there.
PS: "Improvements to shaders and visual shaders in Godot 4.0"  Nice Job


Container interface changes

[x] IContainerInput::pushInfo -> IContainerInput::target
[x] IContainerInput -> ContainerInput
[x] Add GradeReport::status: string (and figure out the relationship between ContainerOutput and GradeReport because there seems to be more duplication there.
[x] Move the high level input and output types for the container into their own file in common/src so they are self contained for people implementing containers

This list will be updated as more minor edge cases are identified.

Fixed and long deployed.


CIV-10046 - BUG ITHC - Document Security

JIRA link
https://tools.hmcts.net/jira/browse/CIV-10046
Change description
Fixed document download security issue.
Does this PR introduce a breaking change? (check one with "x")
[ ] Yes
[x] No

This change then makes it impossible for the first journey to download claim form. is this what we want? and if so do we need to remove the link to pdf in the first journey (where the defendant is not logged in)

Yes.. That's correct... we are removing the download claim from form first-contact claim summary page https://github.com/hmcts/civil-citizen-ui/pull/2252


ci: add initial ci

What did you fix? (provide a description or issue closes statement)
starting to add tooling, linting, autorelease

Reproduction steps

Evidence/screenshot/link to line
Considerations

Does this contain a new dependency? [Yes]
Does this introduce opinionated data formatting or manual data entry? [No]
Does this pr include updated data files in a separate commit that can be reverted for a clean code-only pr? [No]
Have I run the linter? [Yes]
Is is a bug fix, feature request, or enhancement? [Enhancement]

lolwot, sorry, didn't mean to merge this!


Add TL;DR of demarcation

Something like this @pettersv ?

Something like this @pettersv ?

Perhaps add something on an even higher level, such as that this means that deploying operators most likely will be problematic.


Updates for dbt-utils to dbt-core cross-db macro migration

This pull request was created automatically 🎉
Before merging this PR (refer to Detailed Update Sheet 10/2022 for more information):

[x] Verify dbt_project.yml & integration_tests/dbt_project.yml versions are properly bumped
[x] Spot check for dispatch updates, dbt_utils.macro -> dbt.macro
[x] Verify that .circleci directory has been removed
[x] Verify integration_tests/requirements.txt adapters have been updated to 1.2.0 and dbt-databricks is added
[x] Verify .buildkite directory has been added with the following: hooks/pre-command, scripts/run_models.sh, pipeline.yml
[x] Update packages.yml, will need to bump source package (FT utils should be bumped)
[x] Update .buildkite/scripts/run_models.sh with vars as applicable, if N/A then remove relevant lines from script
[x] Remove databricks block from .buildkite/pipeline.yml if package is incompatible
[x] Update schema names in integration_tests/ci/sample.profiles.yml
[x] Update "spark" strings where applicable
[x] Update CHANGELOG template and remove surrogate keys if not applicable to package
[ ] Update README for dbt version badge, install package version range and dependencies for: Fivetran_utils, dbt-utils and source packages
[ ] Regenerate docs
[x] Follow Instructions for adding Buildkite
[ ] Follow instructions for removing Circleci

[ ] blocked because uses dbt_expectations, currently a dbt_utils version conflict because dbt-date is still on dbt_utils v0.9.*
[ ] update old readme
[ ] regen docs

Blocked by dbt-date update for dbt utils 1.0 compat


CUL livestock/stockpond algorithms

Signed-off-by: Andrew Gilmore<EMAIL_ADDRESS>

I think we might need to improve our filter for source data, which right now is looking for data not from 'CU_estimation_process'. Some data for livestock and stockponds will need to be treated as source data for disaggregations, even though it's already computed from lower level source data. This is a great start though.


(ASC-232) Backport PR 2878 to pike

This PR backports the updates in https://github.com/rcbops/rpc-openstack/pull/2878 to the pike branch.
Issue: ASC-219

recheck_cit_release
recheck_cit_dep_update

The missing test is being added in https://github.com/rcbops/rpc-gating/pull/667 - once that's merged, this can be rechecked.

recheck_cit_release
recheck_cit_issue-link

Issue link isn't working because the commit messages do not contain any reference to the JIRA issue.

recheck_cit_release

recheck_cit_artifact

recheck_cit_xenial_no_artifacts_swift_deploy

recheck_cit_artifact

recheck_cit_artifact

recheck_cit_artifact

@johnduarte The artifact job is not required and is generally not working regardless of this patch, so I wouldn't bother rechecking it.


can't add tap actions

This clock was perfect for my needs, wanted a very small clock for a small panel I have running. However I was trying to add a tap action, to navigate to another dashboard with more calendar info, but the tap options don't seem to actually work.
Any thoughts?
type: custom:digital-clock
tap_action:
  action: navigate
  navigation_path: /living-room/calendar

This card doesn't support tap-actions at the moment.

This clock was perfect for my needs, wanted a very small clock for a small panel I have running. However I was trying to add a tap action, to navigate to another dashboard with more calendar info, but the tap options don't seem to actually work. Any thoughts?
type: custom:digital-clock
tap_action:
  action: navigate
  navigation_path: /living-room/calendar

I was also trying to do the same thing this morning and came here to see if there was a solution. Hopefully this gets added as a feature. Would be a nice way to access my dashboard with my calendar, shopping list, notes/tasks.

I was also looking for a way to do this today - just to add another voice to the FR
However, I do recognise that this is not a trivial ask.
If you want a source, custom:button-card has this functionality, and it may be possible to re-use that code.

+1 for this enhancement

Another victim of no tap_action here. Would love for this to go to /calendar

+1 to tap action.  Great card!

I was also looking for a way to do this today - just to add another voice to the FR
However, I do recognise that this is not a trivial ask.
If you want a source, custom:button-card has this functionality, and it may be possible to re-use that code.

Can you please share the yaml of the button card as a workaround? I have it installed from HACS but can't get it to display cleanly, as I can't figure out how to overlay a clickable area over/under the time/date. My method has both next to each other.

As a work around, I ended up just using a custom stack in card in vertical mode. Added in this custom clock and then below added chips with a center alignment. The calendar chip navigates to a different dashboard with my family calendar and some other things. Below is what it looks like in the mobile app.

I also wanted the same functionality, ended up creating this work around, with layout-card and button-card, this will add a empty button card with tap actions on top of the hour card
type: custom:layout-card
layout_type: custom:grid-layout
cards:
  - type: custom:stack-in-card
    card_mod:
      style: |
        ha-card {
          border-width: 0px;
        }
    view_layout:
      grid-column-start: 1
      grid-row-start: 1
    cards:
      - type: custom:digital-clock
        timeFormat: HH:mm
        dateFormat: null
        card_mod:
          style: |
            ha-card {
              font-weight: 700 !important;
              font-size: 2em;
              border-width: 0px;
            }
            .second-line {
              display: none;
            }
  - type: custom:button-card
    show_name: false
    show_icon: false
    tap_action:
      action: toggle
    entity: input_boolean.screensaver_office_kiosk
    view_layout:
      grid-column-start: 1
      grid-row-start: 1
    styles:
      card:
        - background: none
        - border-width: 0px
        - height: 100%


docs: add column for governance modifiable params in resource_pricing

Overview
Addresses #2966 by using the governance modifiable params tests in PR #2973.
All four of the params are modifiable by governance, which matches the specs in params.md.
Checklist

[x] New and updated code has appropriate documentation
[x] New and updated code has new and/or updated testing
[x] Required CI checks are passing
[x] Visual proof for any user facing features like CLI or documentation updates
[ ] Linked issues closed with keywords

that can be modified via governance

Oh good point. I wasn't sure if they could all be modified by governance but since @fahimahmedx confirmed they can all be modified, I'm okay with merging this PR as-is or removing the column and just updating the "fourth" thing


For saving people's life, please make form pre-population easier.

I was trying to pre-populate the form which already had values(saved in database) as you said in another issue, fetching data through $http then put it to $scope. But I don't understand which variable/object of the $scope should been put, I assumed $scope.input and $scope.defaultValue.
Here is my test of each one:

Use $scope.input nothing happened.
Use $scope.defaultValue works fine but failed when form component is checkbox.

So I suggest that we shall not keep arrays like this ['value1', 'value2', 'value3'] in the $scope.input when creating forms, the checkbox component cannot use it directly, and it was horrible to check each component of the form is a checkbox, and it is hard to do form feedback statistics, right? Just use the id of component's option instead, like this:
[
  {"optionId": "1", "checked": "true"},
  {"optionId": "2", "checked": "false"}
]

then we can retrieve/store a more structural data, at least, use [true, false] please.
Thanks very much for sharing such a good project.

@kelp404 Poor response. Single demo with data population, would be good.

Poor response.


Improve event listener/handler interface

THE REASON FOR THIS CHANGE
The previous interface caused confusion. For example, setObjectIsHovered might be interpreted as a setter for a Boolean (which it is not), rather than as a setter for a method that returns a Boolean (which it is). This problem was surfaced by pull request #9.
CRITERIA FOR THE NEW INTERFACE
The interface in this PR is intended to alleviate confusion by satisfying the following design criteria:

flexible enough to handle a wide range of events
intuitive enough for beginners
consistent enough with p5

THE NEW INTERFACE
Here's the new interface:
setEventDetector(type, detector)
setEventResponder(type, responder)

type: a string (e.g. ‘mousepressed’ and ‘mousereleased’)
detector, responder: callback functions for the listener and handler, respectively[^1]

[^1]: "Detector" and "responder" are hopefully more intuitive to beginners than "listener" and "handler."
Note that the type parameter supports the exact names of p5 handlers, such as mousePressed().[^2] When there's no corresponding event handler in p5, we'll typically match Web API names (e.g. ‘mouseover’ and ‘mouseout’), but we may choose our own names if there's a good reason to do so. This pull request also renames some internal variables using a more intuitive naming scheme that matches the new interface.
[^2]: Since event types are passed as strings, they're all lowercase, but otherwise they have the same names as p5 event handlers when applicable.
COMPARISON TO OTHER INTERFACES
The approach in this pull request is similar to the addEventListener() method of the EventTarget Web API and the .on() method of Konva.js. One major difference is that setEventDetector() and setEventResponder() are attached to a Mathemagical interaction object, rather than the ultimate event target (a drawing object).
This way, the user only needs to create a single interaction object in order to apply multiple listeners and handlers (e.g. for a draggable square, multiple listeners and handlers are used to change the cursor on mouseover and mouseout events, and also to change the position on drag events).

TODO
Although we have more features planned (e.g. custom events and custom interaction objects), all that's left for this PR is to implement the following methods on Draggable:
deactivateDefaultResponder(type)
reactivateDefaultResponder(type)

We could do a basic test by calling these with the 'mouseover' type from multiple-event-responders-demo.js. If the deactivate method works, the cursor will no longer change to a move cursor but the stroke will still change; reactivating will restore the original behavior. Once that's done, we can do a quick code review and merge what we have.
Here's my reasoning on prioritizing deactivate/reactivate: Without this, we have no way of resetting default responders, which is something we were already able to do in the original design. (In the original design, we could also reset default detectors. However, that's a less common use case, and the planned features will provide other ways of customizing the default detectors. For example, we'll have customizable hit regions, and it will be possible to create default detectors and responders from scratch for custom events. That all may change, but for now, that's the idea.)


Allow user to specify fields of the item to save to bigquery

A setting to allow a user to set the fields of the incoming item to save to BigQuery since most of the fields are not needed(in my case).
I have written a patch to this and I would like to be assigned.

Created pull request. #3


feat: Add auto-legalization for Iceberg Schema column names.

This issue was auto-generated
PR: https://github.com/deephaven/deephaven-core/pull/5746
Author: lbooker42
Original PR Body
Will close #5714 when merged.

Relates to Iceberg PR


implement a machine watch to trigger a config reconcile

/kind feature
Describe the solution you'd like
As part of #16 we will have to watch machine objects and trigger the associated config object to get reconciled. This is using kubebuilder's MapFn feature.
/assign @amy
/lifecycle active
Anything else you would like to add:
We do something similar in CAPI here: https://github.com/kubernetes-sigs/cluster-api/blob/master/pkg/controller/machineset/machineset_controller.go#L110
That idea will need to be ported to kubebuilder v0.2.
The goal is that if you run the CABPK controller and create a machine with an associated kubeadm bootstrap config you will see that we reconcile the associated bootstrap config on a machine update.

It'd be great to add this upstream, like https://github.com/kubernetes-sigs/cluster-api/pull/1153. It has the benefit that the mapping functions can be shared between all providers

That sounds reasonable to me! thanks for the PR link.

/milestone v0.1
/priority important-soon

/close
please see #39 for details