Error login

Hi, thanks for inventing this integration. I tried it but I get this message in the logging during integration (after entering username and password):
Logger: custom_components.kia_uvo.config_flow
Source: custom_components/kia_uvo/config_flow.py:46
Integration: Kia Uvo (documentation)
First occurred: 10:01:22 PM (3 occurrences)
Last logged: 10:03:14 PM
kia_uvo Exception in kia_uvo login : 'redirectUrl'
I login from Nederland. Can you help me?
William

Thanks, thats loves the issue. Great job Fuatakgun


opencv3: don't build with CUDA newer than 11

Description of changes

Due to API changes in CUDA 12, OpenCV 3 won't build. This PR pins the version of CUDA used to 11 (right now 11.8), the latest supported.
Things done

Built on platform(s)

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

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 314712 run on x86_64-darwin 1

Result of nixpkgs-review pr 314712 run on aarch64-darwin 1

Result of nixpkgs-review pr 314712 --extra-nixpkgs-config '{ allowUnfree = true; cudaSupport = true; cudaCapabilities = [ "7.5" ]; }' run on aarch64-linux 1

Result of nixpkgs-review pr 314712 --extra-nixpkgs-config '{ allowUnfree = true; cudaSupport = true; cudaCapabilities = [ "7.5" ]; }' run on x86_64-linux 1


pod install failing in Example directory

Error received is:
[!] The dependency "AGEmojiKeyboard (from "../AGEmojiKeyboard.podspec")" is not used in any concrete target.

Fixed in master. Please take a pull and try again.
Closing this now.


Edit hdinsight-analyze-flight-delay-data.md

Edit complete.
In the diagram at the beginning of the article, "an Hadoop" and "an Hive" should be "a Hadoop" and "a Hive."
The link "Install and configure Azure PowerShell" didn't work when I tried it.
Per naming guidelines, I changed instances of "PowerShell" by itself to "Azure PowerShell." However, this article uses a mix of "Windows PowerShell" and "Azure PowerShell," even when the acronym "ISE" immediately follows the name. This may be an inconsistency. Please check all instances of "Windows PowerShell" and "Azure PowerShell" and confirm that they're accurate in context.
Per naming guidelines, I changed instances of "storage account" to "Storage account" (except in instances of "Blob storage account," because "Blob storage" is a separate term). Please confirm that this change is accurate, even in instances of "default Storage account".
The article contains multiple references to the year 2013. This information may need an update.
Please confirm that the screenshots don't reveal any potentially sensitive information.
The note on line 33 isn't formatted like other notes.
In the text "The CREATE EXTERNAL TABLE command doesn't allow any folders in the LOCATION" (line 102), it isn't clear what "LOCATION" refers to. If "LOCATION" was meant to be a generic word, it should be lowercase. If it was meant to be a command name, it should be bold. Some text may be missing here as well.
Please confirm that the link "Provision HDInsight cluster and run Hive/Sqoop jobs" (line 392) is rendering correctly. It doesn't seem to be formatted as link text in the GitHub preview.
In the table on line 405, the field names have a single asterisk on either side of them. That usually formats text as italic, but not in this table. (Even in the published article, the asterisks appear.) If you meant to use italic formatting, keep in mind that you don't have to apply it to these items because other items in the table don't have any special formatting. (In general, UI text is bold.)
On line 677, it would be clearer (and more consistent with previous bullet points) to replace "= "tutorials/flightdelays/data" = "tutorials/flightdelays/output"" with a complete sentence.

Hi @ShawnJackson, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (v-shawja). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, or work for Microsoft Open Technologies, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;


Fix missing companion non-active reads for active read calls

What changes does this PR introduce?
Bug fix, sort of
What is the current behaviour?
Some calls to S/R active_read in pkg/ecco are not enclosed in #ifdef ALLOW_AUTODIFF block, and there are missing non-active reads. A model run that does not have pkg/autodiff turned on would crash.
What is the new behaviour
Blocks of #ifdef ALLOW_AUTODIFF are created, and non-active reads corresponding to active reads are added. if ALLOW_AUTODIFF is defined, call active_read; otherwise, call non-active reads.
Does this PR introduce a breaking change?
No
Other information:
Suggested addition to tag-index
(To avoid unnecessary merge conflicts, please don't update tag-index. One of the admins will do that when merging your pull request.)

oh thanks Ou. i've just discovered this problem last week as well when needing to compile ecco without autodiff, and was going to ask you to check it , so timing is perfect!

@owang01 I took a quick look and have 2 comments/issues:

the AD experiment obcs_ctrl  does not compile, and looks like a typo (missing separating comma) in cost_obcsw.F.
there are few very similar active_read call in pkg/ctrl that could also be changed the same way you did for pkg/ecco. They seems to be related to OBCS, as, e.g., in ctrl_getobcse.F but there might be more (when I tried, it stopped at the first one so hard to tell). May be you could think of also changing these in pkg/ctrl since it involves a very similar modification there ?

@jm-c Thank you for your review. I have fixed the typo and will add the non-active reads to files in pkg/ctrl shortly.

@owang01 Your changes are good, and was now able to compile exp. obcs_ctrl without pkg/autodiff.
I made further changes in cost_obcs.F (testing for iobcsX before accessing ncvarrecstart & ncvarrecsend) so that I can now also run this FWD test with -devel.
I also made changes to argument ordering (the unconsistent argument order has been a source of mistake in the past) in the routines that got updated here and hope it's OK with you (if not, could revert some, at least).
Please take a look and see what you think.
Note: I am not motivated to clean-up further cost_obcsvol.F (with a stop as the top) or cost_obcs_ageos.F (did not find any call to this S/R).

@jm-c Thanks for fixing the arguments in the READ_REC_X/YZ_RL calls, as well as making the other changes. Your changes all look good to me. It makes sense to change the argument ordering in cost_obcs.F etc. by putting myTime, myIter, and myThid at the end of the argument list .

@owang01 I think, from my side, this PR is ready to be merged (no  major changes but usefull in disentangling pkgs), and could merge it in the coming days, unless someone wants to add something. Do you agree ?

Sounds good to me. Thanks @jm-c.

Will merge this PR later today or tomorrow.


How to use canvas and paint image ? how to save a image file use flet

please see https://github.com/flet-dev/flet/issues/317 and upvote. There is no present support for this but its possible in flutter
also see https://github.com/hololeo/flet-hacks (screenshot hack) for method to take a screenshot of entire window with python. You can narrow it down to a part in the window


update package check for trusty 12.04

this fixes #23 and #22

Tim,
Are you planning to do this PR and re-make the OVJ 1.1A release and then tag it? If so, I have two minor fixes and 1 significant fix (qtune on Mercury does not work) to add. I can't add them until Monday.

Hi Dan
I'll add your fixes and the openvnmr-utils directory to OpenVnmrJ Monday evening, tag and rebuild. I added Michael's PR already (he added an Apache license header)
Cheers
Tim
Sent from my iPhone

On Sep 23, 2016, at 11:43 AM, Dan Iverson<EMAIL_ADDRESS>wrote:
Tim,
Are you planning to do this PR and re-make the OVJ 1.1A release and then tag it? If so, I have two minor fixes and 1 significant fix (qtune on Mercury does not work) to add. I can't add them until Monday.
―
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

i'm not sure why there's 5 commits on this, I think it'be because I started with the master branch.  hopefully squashing this into a single commit when doing the merge will get rid of them.

I am still a little confused by this process. I just finished pulling my changes from the development branch of DanIverson/OpenVnmrJ into the development branch of OpenVnmrJ/OpenVnmrJ. If I select the development branch, it says something like 22 commits ahead and 6 commits behind master. I have not yet made a PR from development to master. If I do a compare, most of the commits have already been pulled into master. I don't know why they are still showing up in development

Github just added a "rebase and merge" option to the "Merge pull request" button on pull requests, which I think would do the right thing for this situation.

I did the new "rebase and merge" on this PR

Hm, well it seems to have only brought over the two commits from me, so that's good.  In retrospect though, probably should have squashed them together.

I'll add some of my changes into this dev branch, then merge the development branch with master.
Squash & Merge? (what I'd usually do)
Rebase and Merge?
Just Merge and keep the history?

For this one, it doesn't matter.
For the future, to clarify, every PR that goes into development gets squashed, and every PR to master gets rebased and merged (or just merged?)


Add some tests around zombie stack handling

https://github.com/glassechidna/stackit/blob/master/stackit/stackit.go#L212-L215

Obsolete


[mempool experiment]

Motivation
(Write your motivation for proposed changes here.)
Have you read the Contributing Guidelines on pull requests?
(Write your answer here.)
Test Plan
(Share your test plan here. If you changed code, please provide us with clear instructions for verifying that your changes work.)
Related PRs
(If this PR adds or changes functionality, please take some time to update the docs at https://github.com/libra/website, and link to your PR here.)

:umbrella: The latest upstream changes (presumably 156191cb504ca4295d815f7def169aef014f3818) made this pull request unmergeable. Please resolve the merge conflicts.


API Manual is wrong for awswrangler.catalog.get_columns_comments

Describe the bug
The contents is wr.catalog.get_table_parameters
https://aws-data-wrangler.readthedocs.io/en/2.9.0/stubs/awswrangler.catalog.get_columns_comments.html#awswrangler.catalog.get_columns_comments
Environment
NA
To Reproduce
NA

Thanks for raising this, now fixed in latest version of documentation (please make sure to hard refresh the browser to see the change)


Golang: Out of Sync / right-click test configurations

Versions: IDEA Ultimate 2017.3.3 / 2<IP_ADDRESS>.1
          IDEA Ultimate 2017.3.1 / 2<IP_ADDRESS>.4

We are missing the right-click Run/Debug options in Golang projects that we have for Java/Scala.
We can execute tests form the corresponding BUILD file, but not from the context of the test file itself either as a whole unit or an individual method.
It also appears that the Workspace is frequently out-of-sync (via the light in the toolbar).
Sample .bazelproject:
directories:
  .

targets:
  //...

additional_languages:
  go

I am interesting in contributing, would we just need to port somethings over from the Java/Scala invocations?

It works as of the latest plugin and IDE.


[55-FIX] Update link for downloading all cases

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

Project coverage is 58.50%. Comparing base (7feafa1) to head (ce6bb7c).
Report is 6 commits behind head on main.

Files
Patch %
Lines

...ation/curator-service/api/src/controllers/cases.ts
0.00%
1 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #67      +/-   ##
==========================================
- Coverage   63.41%   58.50%   -4.91%
==========================================
  Files         171       21     -150
  Lines        6278     1405    -4873
  Branches     1722      217    -1505
==========================================
- Hits         3981      822    -3159
+ Misses       2297      583    -1714

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


Vulnerabilities

Hi, sorry about this, but I can't use my forked version of your library because it has 3 vulnerabilities and I can't seem to fix them?
found 3 vulnerabilities (2 moderate, 1 high) in 1464 scanned packages
  3 vulnerabilities require manual review. See the full report for details.

C:\src\jasmine-dom>npm audit fix
up to date in 8.942s

95 packages are looking for funding
  run `npm fund` for details

fixed 0 of 3 vulnerabilities in 1464 scanned packages
  3 vulnerabilities required manual review and could not be updated

I've tried updating kcd-scripts but it didn't make any difference?

Hi!
I had a look at kcd-scripts and the vulnerabilities come from rollup-plugin-node-builtins which seems to be abandoned.
Since kcd-scipts is a devDependency, I guess you shouldn't have an issue using your fork on a project? 🤔


Marshal's Coordinated Attack Feature not procced by the Marshal's Attack of Opportunity

Game version 1.4.33, Mod version <IP_ADDRESS>, no other mods
This may be intended but as written, it should proc on AoO. Change should either make the feature proc on AoO or modify the wording of the feature to reflect that not proccing on AoO is the intended design, for example:
"When you hit with a melee attack on your turn"
https://i.imgur.com/yaJqcm8.png
The feature otherwise works as written with main action and bonus action attacks.

It is intended because the game softlock if you triggers reaction within the context of another reaction. The description will be updated.

Cannot fix because of engine limitations.


fix(versions): update Activiti/example-runtime-bundle versions into master

UpdateBot pushed maven dependency: org.activiti.cloud.rb:activiti-cloud-runtime-bundle-dependencies to: 7.1.205

UpdateBot commands:
updatebot push-version --kind maven org.activiti.cloud.rb:activiti-cloud-runtime-bundle-dependencies 7.1.205


Doesn't properly parse defines with quotes

When trying to create a compiler definition called VERSION set to the string "1.0.0" (with the quotes), cmake-ide doesn't seem to parse it correctly. Here's an example that compiles fine, but gets flycheck errors constantly when trying to edit it in emacs:
CMakeLists.txt
cmake_minimum_required (VERSION 2.6)
cmake_policy(SET CMP0005 NEW)
add_definitions(-DVERSION="1.0.0")

add_executable(test test.cpp)

test.cpp
#include <stdio.h>

int main(int argc, char* argv[]) {
  printf("Version is %s\n", VERSION);
}

I get this for flycheck-clang-definitions:
flycheck-clang-definitions is a variable defined in `flycheck.el'.
Its value is ("VERSION=")

And running flycheck-compile gives me:
-*- mode: compilation; default-directory: "~/test/" -*-
Compilation started at Wed Sep  5 17:31:51

clang -fsyntax-only -fno-color-diagnostics -fno-caret-diagnostics -fno-diagnostics-show-option -iquote /home/cole/test/ -Wall -Wextra -DVERSION\= 1.0.0 -o CMakeFiles/test.dir/test.o -c -x c\+\+ - < /home/cole/test/test.cpp
clang: error: no such file or directory: '1.0.0'

Compilation exited abnormally with code 1 at Wed Sep  5 17:31:51

The issue is fixed if you change split-string-and-unquote to split-string in cide--split-command, but I'm not sure what the implications are of doing this.


add json mapping extra fields

this is extracted for json from https://github.com/crystal-lang/crystal/pull/4411

If we do this, I guess to_json should also take this into account.

I'm not sure if this feature is a good idea.
What's the use case? Why do we need this in JSON.mapping? To me it doesn't seem to be commonly needed. And it's relatively easy to implement in a custom to_json method.
There are ideas to change the way mappings are defined by using meta attributes (see #3620) which will probably mean removing JSON.mapping at some point. It is still open how a future solution will look like. Unless there is a pressing reason to implement this, it might make sense to figure out the general mechanics first.

For me, this is often needed, and needed mostly for from_json method (not to_json). When you have api, which receive some defined data structure, it also sometimes add not defined fields (different every time), which you want to receive. Before that, there is only way to parse it manually, which is pain.

which you want to receive

Pardon me insisting, but why? You described how this is used (which is quite obvious) but not a real reason.

there is only way to parse it manually, which is pain.

Would it perhaps be an option to improve the manual implementation and make it easier to use?

I'm still not convinced this is useful enough. But I don't have to decide that either ;)
But I have an idea to solve your problem in a more versatile and less intrusive way: JSON.mapping could have more advanced handling of keys not included in the mapping (the else branch in the big case statement produced by the macro). Currently, there is strict which configures to either raise or skip. What if there was a third way, a custom handler?
JSON.mapping(strict: ->(key : String, parser : JSON::PullParser) { @extras[key] = JSON::Any.new(parser) })

callback is nice, but there is also should be way to extend to_json method. May be
to_json(js) { |js| @extras.each { |k, v| js.field(key) { v.to_json(js) } } }

so, should i implement callback? or it not decided.

The idea of a callback is nice. Maybe JSON.mapping could define a method that by default raises an error, and which you can override to deserialize unknown keys. And same goes for to_json.
If eventually we have meta attributes, it would work the same way.

But let's first design it: what's the name of the method, etc.


Migrate to Laravel

[x] Migrate Base API to Laravel
[ ] Add Authentication Base
[x] Migrate Database to PostgreSQL
[ ] Prepare Dockerfile

Docker and authentication will move to another task


Failed to compile in 0.21.1

Hello,
The latest commit seems to have added a bug in the compilation. I am not familiar with cpp so I have no clue what's happening, I've attached the error and the g++ version below.
g++ -c src/stats.cpp -o obj/stats.o -std=c++11 -g -O3 -I./inc
g++ -c src/htmlreporter.cpp -o obj/htmlreporter.o -std=c++11 -g -O3 -I./inc
In file included from src/htmlreporter.cpp:1:
src/htmlreporter.h:20:24: error: ‘atomic_long’ has not been declared
     void setInsertHist(atomic_long* insertHist, int insertSizePeak);
                        ^~~~~~~~~~~
In file included from src/htmlreporter.cpp:1:
src/htmlreporter.h:42:5: error: ‘atomic_long’ does not name a type
     atomic_long* mInsertHist;
     ^~~~~~~~~~~
src/htmlreporter.cpp:22:34: error: variable or field ‘setInsertHist’ declared void
 void HtmlReporter::setInsertHist(atomic_long* insertHist, int insertSizePeak) {
                                  ^~~~~~~~~~~
src/htmlreporter.cpp:22:34: error: ‘atomic_long’ was not declared in this scope
src/htmlreporter.cpp:22:47: error: ‘insertHist’ was not declared in this scope
 void HtmlReporter::setInsertHist(atomic_long* insertHist, int insertSizePeak) {
                                               ^~~~~~~~~~
src/htmlreporter.cpp:22:47: note: suggested alternative: ‘sys_errlist’
 void HtmlReporter::setInsertHist(atomic_long* insertHist, int insertSizePeak) {
                                               ^~~~~~~~~~
                                               sys_errlist
src/htmlreporter.cpp:22:59: error: expected primary-expression before ‘int’
 void HtmlReporter::setInsertHist(atomic_long* insertHist, int insertSizePeak) {
                                                           ^~~
src/htmlreporter.cpp: In member function ‘void HtmlReporter::reportInsertSize(std::ofstream&, int)’:
src/htmlreporter.cpp:219:21: error: ‘mInsertHist’ was not declared in this scope
         allCount += mInsertHist[i];
                     ^~~~~~~~~~~
src/htmlreporter.cpp:219:21: note: suggested alternative: ‘setInsertHist’
         allCount += mInsertHist[i];
                     ^~~~~~~~~~~
                     setInsertHist
src/htmlreporter.cpp:221:17: error: ‘mInsertHist’ was not declared in this scope
     allCount += mInsertHist[mOptions->insertSizeMax];
                 ^~~~~~~~~~~
src/htmlreporter.cpp:221:17: note: suggested alternative: ‘setInsertHist’
     allCount += mInsertHist[mOptions->insertSizeMax];
                 ^~~~~~~~~~~
                 setInsertHist
make: *** [Makefile:27: obj/htmlreporter.o] Error 1

Here's the g++ version used
g++ (Debian 8.3.0-6) 8.3.0
Copyright (C) 2018 Free Software Foundation, Inc.
This is free software; see the source for copying conditions.  There is NO
warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.

Thanks for your report. Bug fixed.


Feature/render tag

Add render tag
See: https://shopify.github.io/liquid/tags/template/

Cool, looks similar to how I implemented it as well.
I haven't read through / fully understood the changes from #37 yet, so it may answer this, but for example, for a render tag like:
{% render 'foobar'.html, bar: bar %}

We want to limit the context scope to just bar, but there's also some subset of global variables we want to keep (for example, using shopify as an example, request). The way I handle it right now is that I track a list of the global variables in the context and clone those into the child context automatically. Is it possible to have some notion of globals as part of this?

Based on how I'm reading their documentation, you would not have access to global variables. The only variables you would have access to are those passed in at the render call.
However, I'm with you that Shopify itself is doing something different. The context object is quite complex and I don't fully understand it, but there are many scopes that can contain variables. There is Context.environment that is basically your starting variables that are defined when you create a context. There is also Context.scope which is used to track current variables defined within a "scope" such as a loop. Nesting loops creates nesting scopes as seen here. Lastly, there is a 3rd location specific to cycles/iterators.
Now, Shopify's documentation says this:
https://shopify.github.io/liquid/tags/template/

The code within the rendered template does not automatically have access to the variables assigned using variable tags within the parent template. Similarly, variables assigned within the rendered template cannot be accessed by code in any other template.

... which is very vague. I bolded the vague part.
What I currently do is create a new context and copy over everything besides the multiple scopes. I feel like maybe the environment variables should be kept around.
To add more complexity, the scopes also have something that is defined as a "global" which just puts the variable at the top level scope. Understanding the many uses of the context and scopes is on my todo list.
I know this was a lot of information, but it's complicated. I need to dig deeper into the scoping rules again before I can fully release this.

I've been deep in Shopify's Liquid code to figure out what they do here and it appears that there is yet another scope of variables called the static_environment. These variables are truly global and accessible within nested render calls.
I describe it here. In your case, when you create the context, you should do something like the following:
context = Liquex.Context.new(%{not_accessible_in: :nested_render_tags},
  static_environment: %{request: my_request_variable})

Cool - that solves my use case perfectly and will be a drop in replacement for my custom tag.
One last thing - we cache the liquid ast heavily at runtime (and actually run Liquid.parse at compile time via a macro for our "default" templates for the base case of templates that haven't been modified by the user). Would it be possible to have read_template_file return a string OR the AST & have the render tag skip parsing if it gets an AST?

Caching is going to be built in to the render tag, outside of the FileSystem. The render call will cache the results after parsing in the next code push. There's actually plans on caching a whole lot more within the context object, such as memoizing some of the variables when I decide to tackle drops.
Are you caching the AST between render calls? The plan was to cache the AST for the partials within the context, but that won't help you caching the partial across multiple render calls. Maybe there's something with reusing the context but clearing out variables between renders. I'm interested in any ideas you have.
Reading through the Liquid code again, they have a concept called Registers, and the registers have a static component that's shared across all the contexts created. This includes the partial cache. I could probably mimic this with a genserver or something similar. The static registers also include the configurations for file systems and the template factory.
I'd like to keep the FileSystem protocol pretty dumb here and not have parsing built into it.

Yeah, we cache the AST's between render calls in an ets table. In theory, each individual liquid template is only ever parsed once during the lifetime of the app unless it is modified by the end-user.
We do this for the main template (outside of Liquex/render tag), and before calling Liquex.render, we set a loader function on the context that the render_tag uses to load templates at runtime. The loader function returns the AST from the cache. This is similar to how you have the implemented the filesystem call, minus it returning an AST. Basic example below.
I've also debated the idea of scanning the AST after parsing and just expanding all of the render tags (essentially replacing the render tag w/ it's parsed AST) at parse-time. The downside is that you'd only be able to do this for templates that are included as string literals, but that's 99% of the case for us. The benefit for my use case would be that you can manipulate the whole tree, since we actually pre-process the AST before rendering to extract the static literals + capture the dynamic tags + assigns and transform it into a %Phoenix.LiveView.Rendered{} struct, which allows us to get LiveView diff-tracking on the liquid template - but since we can't see "inside" the render tag until runtime/render time, we're forced to do the transformation in the render tag at runtime.
Anyways, just food for thought, I know that liveview use-case is pretty atypical.
{:ok, ast} = get_template(name)
context = Liquex.Context.new(filter_module: FilterModule)
private = context.private |> Map.put(:loader, fn template_name, _ -> get_template(template_name) end)
context = %{context | private: private}

{result, _} = Liquex.render(ast, context)

def get_template(name) do
  case get_template_from_ast_cache(directory) do
    {:ast, ast} -> {:ok, ast}
    {:template, template} -> Liquid.parse(template)
  end
end

# render_tag.ex

def render(template_name, context) do
  ast = context.private.loader.(template_name)
  # ... bunch of stuff to build child context
  Liquex.render(ast, context)
end

I've added a behavior for caching and two basic caching modules.
My original take at this was that I would add a behavior so you could plug in something like Cachex because I don't want to try to make a full featured caching mechanism, however I felt like it should have something out of the box. By default, caching is disabled.
To use the ETS based cache, you would add it to your context:
:ok = Liquex.Cache.SimpleCache.init()
Context.new(%{...}, cache: Liquex.Cache.SimpleCache)

Right now, caching is limited to only render partials, but I plan to expand on that.
Since you have your own caching mechanism, let me know if the behavior I defined is enough to plug yours in. Check out the SimpleCache module for how to use it.


Document the Kube RBACs required to run odo experimental

/kind documentation

What mistake did you find / what is missing in the documentation?
If we're running odo in experimental mode in a Kube platform other than OpenShift, Minishift; we need to document the Roles required for the user to run odo in experimental mode successfully.
What is the relevance of it?
Odo experimental mode, is Kube platform agnostic and thus, the user would require the appropriate Roles to create the various Kubernetes resources.

Debating on the best way to document this:

Do we provide a role that cluster administrators can apply to users that need to use odo?

How do we keep this up to date?

Do we just document the permissions that odo needs and leave it up to the user?

Documenting each individual role that odo needs seems messy and again, would need to be kept up to date

@kadel @girishramnani these docs should probably reside only in GH?

Hey @maysunfaisal bit confused regarding this. I have updated all documentation on the odo.dev site to reflect that it's Kubernetes / OpenShift agnostic, so you can deploy on either one.
Do you mean RBAC roles or anything else we need to apply?


Support GitHub Desktop authentication

I have scm git installed and GitHub for Windows.
When I try to push I get prompted for username/password then get "Authentication failed on the git remote."
I am able to push from GitHub for Windows and command line using same credentials

What do you get when running git remote -v?

Hi, opening GIT bash in the folder & running I get
$ git remote -v
origin  https://github.com/joe912/taggle.git (fetch)
origin  https://github.com/joe912/taggle.git (push)

Yeah this is known. Hope you don't mind that I renamed this issue.

GitHub Desktop clones every repository using the https protocol and creates ~/.ssh/github_rsa and ~/.ssh/github_rsa.pub which it somehow uses for auth. git (or ssh) by default only looks for ~/.ssh/id_rsa and ~/.ssh/id_rsa.pub when using authentication.
We need to:

[ ] Find out how GitHub Desktop is able to use pubkey auth with the https protocol
[ ] Make VSCode use that github_rsa key when pulling/pushing from a GitHub repository.

ok - thanks for the update

I can confirm I am having this same issue. I can use GitHub Desktop or CLI to push, but I can not sync with VS Code. I tried again with the December update and it was not supported still.

I found the following Stackoverflow post that solved this issue.
Basically, I ran git config --global credential.helper wincred from the command line, and restarted VC Code. When I reopened the project, I was now able to sync from VS Code.
Do not know how this persists, and if it is only for that repo (--global makes me thing it will work elsewhere).
Can we run this command as part of installation, if this solves the issue?

Hello, I'm having the same issues, already followed the Stackoverflow post but it didn't work.
I don't have any problem with editing the code in VSC and then using the CLI or the GitHub desktop to sync my repos but It would be great if this issue is solved, the last update didn't solved it.
thanks

Hi, I'm experience this issue too. I'm willing to help. Please let me know how.

I can elaborate on what is causing issue for people.
If GitHub desktop is opened, it will break VS Code ability to push/pull. Closing GitHub desktop, DOES NOT fix VS Code GitHub sync.
Instead you have to -

Close GitHub Desktop and VS Code.
Run git config --global credential.helper wincred from Powershell or Cmd.
Reopen VS Code, you now have ability to push/pull without entering in username and password.

Opening GitHub Desktop will break this. There seems to be a conflict of some sort that is causing this issue.

I can elaborate on what is causing issue for people.
If GitHub desktop is opened, it will break VS Code ability to push/pull. Closing GitHub desktop, DOES NOT fix VS Code GitHub sync.
Instead you have to -

Close GitHub Desktop and VS Code.
Run git config --global credential.helper wincred from Powershell or Cmd.
Reopen VS Code, you now have ability to push/pull without entering in username and password.

Opening GitHub Desktop will break this. There seems to be a conflict of some sort that is causing this issue.

@khrome83 I tried several times your workaround and it is not working.

I was having this exact problem. Closing GitHub Desktop and running the config command did nothing for me. I started backtracking and remembered that I had recently added a simple .gitattributes file.
Most of the file was simply comments, and once I removed it and followed the instructions above once again, things started was working. Every time I open the desktop app, it breaks again. But at least the aforementioned fix works as long as I don't add this file to the project.
# # Auto detect text files and perform LF normalization
# *        text=auto
#
# *.cs     text diff=csharp
# *.java   text diff=java
# *.html   text diff=html
# *.css    text
# *.js     text
# *.sql    text
#
# *.csproj text merge=union
# *.sln    text merge=union eol=crlf
#
# *.docx   diff=astextplain
# *.DOCX   diff=astextplain
#
# # absolute paths are ok, as are globs
# /**/postinst* text eol=lf
#
# # paths that don't start with / are treated relative to the .gitattributes folder
# relative/path/*.txt text eol=lf

* text=auto

Looks like the age old battle between \n & \r\n has returned to bite us once again ;)

Why would having the file offer a conflict if the issue is just credentials? Is it because it is a conflict with the rules within VS Code (LF / CRLF). This is very confusing, but it does make sense. I did not have this file.
Is the solution to make VS Code sensitive to this file for it's GIT Implementation?

May not be related at all, really. One this is pretty clear, when VSCode sends credentials manually via a prompt, it doesn't work. So the easiest way to fix it is to eliminate the prompt, and that is what the command above seems to do. I did see some strange formatting in the git settings file too, but it may be unrelated. This is a file that seems to get touched when you open github desktop though.
git config --global --e

Opens the following file for me..(noticed multiple credential headings)
[user]
		name = djabr..
		email = <redacted>
[filter "lfs"]
		clean = git-lfs clean %f
		smudge = git-lfs smudge %f
		required = true
[credential]
[credential]
[credential]
		helper = wincred

Hi
For those who still have this problem, this worked for me
https://social.msdn.microsoft.com/Forums/vstudio/en-US/eb6853f3-b710-4be5-8b82-d55fa568e2b9/problem-with-git-on-vs-code?forum=vsarch
Regards

I don't even have an authorize window in VS Code - but the error message is shown after I click either push to or pull from the remote repository.
But if I go to the console and run commands manually, enter credentials - everything works.

Hi~I have this problems as well. I'm using Coding instead of GitHub. Is it because of https protocol? How can I make the same credentials work in vs code?

Since I don't want to Hijack this thread for Mac, I opened a separate issue for Mac OS:
https://github.com/Microsoft/vscode/issues/46964

FYI found a workaround that seems to work on all OS systems, Mac and Windows:
Visit your github developer settings and generate a personal access token.
That token can be used as your password with your user name to login from VSCode and it seems to work normally from that point on without any special configuration.

It still doesn't work for me, I managed to login via the terminal, but using your username + password = token didn't workout very well with 2FA enabled

I noticed that even though there is a pop-up GUI window that takes the Git username and password, there is another Visual Code pop-up, under the top main menu bar in Visual Code, that asks for the Git username and password.  After you enter your credentials there too, it works.

in case if you set up 2 stage auth you can use this https://help.github.com/articles/creating-a-personal-access-token-for-the-command-line/

I came across to this problem too just now after enabling 2FA on my account. I've regenerated the access token for VSC, and used it as the password. I was prompted twice but it worked. (OS: Windows 10)

this "git config --global credential.helper wincred" might work on windows, but I'm using a mac and Microsoft forces anyone on their repositories to activate 2 factor authentication.
I cannot login from VSCode to my GitHub in order to sync with origin.
Does anyone have a workaround for a Mac?

Exactly same issue here. I temporarily disabled 2FA in GitHub's account Security tab and it works. Its workaround and not fix.

FYI found a workaround that seems to work on all OS systems, Mac and Windows:
Visit your github developer settings and generate a personal access token.
That token can be used as your password with your user name to login from VSCode and it seems to work normally from that point on without any special configuration.

Work for me too, great job !

Code now supports automatic GitHub authentication, when using HTTPS repositories, so there's no need to support GH Desktop. This is already working in Insiders and will come out soon in the next stable.


[RBAC] Admin user configured doesn't have access to all permission policies by default

Describe the bug
When I simply add admins in the permission configuration as below and not policies in rbac-policy.csv as readme specify https://github.com/janus-idp/backstage-plugins/blob/main/plugins/rbac-backend/README.md#configure-policy-admins
permission:
 enabled: true
 rbac:
  admin:
   users:
    - name: user:default/<user>

then the admin only has access to policy-entity and not other plugin entities like catalog-entity
I explicitly need to specify each policy in rbac-policy.csv to get access to other plugin entities even for admin
p, user:default/<user>, catalog.entity.read, use, allow
Expected Behavior
Admin should have access to all the permission policies by default
What are the steps to reproduce this bug?

Setup backend plugin following https://github.com/janus-idp/backstage-plugins/blob/main/plugins/rbac-backend/README.md
add admins in the permission configuration as below in app-config.yaml and not policies in rbac-policy.csv as readme specify https://github.com/janus-idp/backstage-plugins/blob/main/plugins/rbac-backend/README.md#configure-policy-admins

permission:
 enabled: true
 rbac:
  admin:
   users:
    - name: user:default/<user>

start the instance
Admin users do not have access to Read/create a catalog and this may happen with other permission policy as well

Versions of software used and environment
"@janus-idp/backstage-plugin-rbac-backend":  1.6.6

cc @PatAKnight @AndrienkoAleksandr

I would agree that this is an important item. Ideally, we don't want to prime the pump or depend on the CSV file. With the large number of users and permission configs we believe we may end up with, leveraging the database solely would be preferred. The key is having the admin users automatically having the ability to create/manage permissions policies.


docs: add information about limitation that can on github.dev when exceeding GraphQL rate limits

Why:
This will help users to be aware of what can happened when they code too long on the web editor.
Closes #32854

What's being changed (if available, include any code snippets, screenshots, or gifs):

Check off the following:

[x] I have reviewed my changes in staging, available via the View deployment link in this PR's timeline (this link will be available after opening the PR).

For content changes, you will also see an automatically generated comment with links directly to pages you've modified. The comment won't appear if your PR only edits files in the data directory.

[x] For content changes, I have completed the self-review checklist.

@CBID2 Thanks so much for opening a PR! I'll get this triaged for review ✨


Apply 'Timeline should immediately start moving play head when starti…

…ng a new recording' commit on our side.


$__unixEpochFrom() and $__unixEpochTo() don't work in variable queries

$__unixEpochFrom() and $__unixEpochTo() don't appear to resolve when being used as query params in a variable query.

On a simple node express app with this route:
  console.log('query:', req.query);
  res.status(200).send('ok');
}

Yields:
  time1: [ '$__unixEpochFrom()', '$__unixEpochFrom()' ],
  time2: [ '$__unixEpochTo()', '$__unixEpochTo()' ]
}

They're coming through as string literals.  Hopefully I'm not just doing something wrong!

Found the problem:
datasource.ts:
Problem:
  const frames = await this.doRequest(query);

Solution:
  const frames = await this.doRequest(query, options.range);

Will send a PR tomorrow.

Fixed in v.1.2.0


Property permissions is not allowed.

Description of the bug
Problems tab message:
Property permissions is not allowed.
When using in app.json
"permissions" : []

This should be allowed for using minimum permissions.
To Reproduce
Add "permissions" : [] to app.json
Expected behavior
No problem message in VS Code
Actual behavior
Problem message Property permissions is not allowed.

Additional context
expo v39
MacOS v10.15
VS Code Version: 1.49.1

My mistake. Closing issue. It should be android.permissions:
android: {
  permissions: []
}


commitizen: 3.22.0 -> 3.24.0

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for commitizen is: Tool to create committing rules for projects, auto bump versions, and generate changelogs
meta.homepage for commitizen is: https://github.com/commitizen-tools/commitizen
meta.changelog for commitizen is: https://github.com/commitizen-tools/commitizen/blob/v3.24.0/CHANGELOG.md
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 3.24.0 with grep in /nix/store/4dqlkap1w9l5d5xfmd9cc7ya9xwnnmiz-commitizen-3.24.0
found 3.24.0 in filename of file in /nix/store/4dqlkap1w9l5d5xfmd9cc7ya9xwnnmiz-commitizen-3.24.0

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

2 total rebuild path(s)

2 package rebuild(s)

First fifty rebuilds by attrpath

commitizen

Instructions to test this update (click to expand)

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

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

Or:
nix build github:r-ryantm/nixpkgs/169fb3086fca7908c05ccb631aecff59fc55f159#commitizen

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

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

2 packages built:

commitizen
commitizen.dist

Maintainer pings
cc @lovesegfault @anthonyroussel for testing.

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

2 packages built:

commitizen
commitizen.dist


Add simple tests to make sure themes don't contain language errors.

I made a very basic test to just source and apply each theme and cycle through various modes, ensuring that there is no exception. As an aside, if the role of maintainer for this project is still open, I am interested in taking on the job.

Thanks, this is very much appreciated and help is always welcome. Let me see how this works.

@chrisbra My apologies! The readme still points to my travis build status. I've updated it to point to the master branch of this main repo. The repo owner needs to authorize travis ci app through the travis ci website, then enable it for this repository. I wasn't sure if I should open a new pull request for this so again sorry if this is the wrong way to go about it.

ah yes, I see. I think I have fixed that now...


Update BATSModel, TBATSModel to handle integer timestamp

🚀 Feature Request
In #178 it was found that tbats models fail on data with integer timestamp. It could be fixed.
Proposal
Fix tests tests/test_models/test_inference/test_forecast/TestForecastOutSample for models:

BATSModel
TBATSModel

Error:
ValueError: Model is not fitted! Fit the model before calling predict method!

The problem is with _TBATSAdapter.forecast which checks if self._freq is None which is now a valid value.

The default value for self._freq should be changed.
The check on fitted model should be changed.

Test cases

Move fixed tests in tests/test_models/test_inference/test_forecast/TestForecastOutSample from fail to success category
Move fixed tests tests/test_models/test_inference/test_predict/TestPredictInSampleSuffix from fail to success category

Additional context
Blocked by #178.

Closed by #195.


WinRM has failure binding SSL do to copy and pasted thumbprints.

As of the latest version of WinRM, all my SSL binding tasks are failing during a release. This appears to be due to special characters present in the SSL Thumbprints that were copied and pasted during my initial configuration. This was working correctly previously, but now it seems I'm going to have to re-enter all my values it seems in order to get this to work.

It also seems that a thumbprint with its spaces (copied from the browser) worked prior to the current version, but now the thumbprint requires removal of the spaces as well to meet the 40 character limit.

We have seen some instances where because of invisible characters ssl bindings is not working and hence we added this validation. If you remove extra characters etc is the task working?

@sumo300
Previously with spaces in thump print your task may be succeeding but binding might not have happened. The additional checks that we have added addresses that kind of scenarios.

Just ran into this as well... was banging my head for over an hour... Since we now know that the invisible characters are likely to always be there after a copy paste, can we get the extension updated to strip any characters that are not valid for us? (e.g. strip non-valid characters THEN test the length).


Allow specifying installation root other than ~/.cargo/

What are you thinking more specifically? rustup is intended (for now at least) to be run out of $CARGO_HOME/bin, so if we were going to make this configurable we would effectively be changing the location of CARGO_HOME.
This is doable, but it means that the way PATH is setup will also need to change, that the proxies will need to tell cargo where it's home is (though it already incidentally does this). The location of CARGO_HOME is also closely related to the location of RUSTUP_HOME - it's likely that if you want to change one you want to change both.
FWIW if CARGO_HOME is set at install time rustup-init will obey it, and it'll probably just work.

I mostly want tools to not put configuration along with the binaries.
For rustup installations I would like its configuration to live in some path specified by an environment variable (RUSTUP_CONFIG maybe) and all binary things it installs should go to $XDG_DATA_HOME and not along with the config – libraries into $XDG_DATA_HOME/lib, binaries to $XDG_DATA_HOME/bin, docs to $XDG_DATA_HOME/shared/doc etc etc. Basically what I’m looking for is at least a --install-root=whatever I want option.

I think the default shouldn't be $HOME/.cargo either, that's not following XDG base directory spec and is polluting $HOME.
It should go into $XDG_DATA_HOME/cargo or $HOME/.local/share/cargo if $XDG_DATA_HOME is not defined.

@shmerl that's https://github.com/rust-lang/rfcs/pull/1615

@steveklabnik: Thanks, I didn't realize it was such a thorny issue. I though using XDG shouldn't cause any such trouble. Still, I'm strongly in favor of it.

I would love to be able to install rust on a computer as root so that multiple users can all use rust.  I think this requires separating rustup's location from the location where each user running cargo will have cargo download packages.

What if users are using a NFS such a lustre? it cannot installed into NFS and be distributed.

Would be nice if rustup followed the same patterns as cargo install.

Provide a cli option (or rather a rustup_init option)
Provide an environment variable, like $CARGO_INSTALL_ROOT
A configuration key, like install.root of $CARGO_HOME/config.

The rust repo's x.py and cargo install both support manually changing the prefix, but the "official install tool" has no such concept.

An example of why someone might like to install elsewhere: Ones' home dir may be on an attached network share (not uncommon in many a corporate environment). These network drives are not normally known for their speed and would be a sub-optimal location for a cache.

rustup-init honours the CARGO_HOME and RUSTUP_HOME environment variables if they're set before the start of the installer.

Great, so we can close this out?
Sent with GitHawk

Is XDG base directory support already implemented though?

rustup doesn't use XDG dirs, no.

So it's still a problem then.

#247 is the ticket for XDG support; it is blocked on cargo (read through the ticket, links are in there). This ticket is about customisation, and as we have that, yes, it can be closed.

I just hit this issue where my NFS home is in a cross platform environment (x86, Arm, others), but there's only one bin directory.

@tommythorn as explained earlier in this issue, CARGO_HOME is obeyed from the environment for both cargo and rustup.


Find out why some of the latest dependencies from corefx are not compatible the tfms we use in corefxlab

I had to pin the version of some of our dependencies because they were incompatible with the tfms we use (dotnet5.2, 5.6). We need to understand why they became incompatible and if we are generating these libraries for the right tfms.

I suspect we need to change the project files to use:
    "netstandard1.x": {
        "imports": [ "dotnet5.y" ]
    }

@KrzysztofCwalina yes, where x = y - 1 :wink: see https://github.com/aspnet/Mvc/pull/4170 for an example

@livarcocc, I think I fixed this. Could you verify?

seems to be fixed


Dashboard page - Single Course Project View

This is a three-part task:

[x] Design the new page
[ ] Create the proof of concept
[ ] Implement the proof of concept

Design
What is the new component or item:
Create a design doc for developers to create a static copy with code. You will be using UIKit components along with Fontawesome PRO icons. Any comments you need to make, make them in the discussion of the issue or the implementation issue for this component.
The expected deliverable will be a design document with specs that shows all the possible states it has. Provide notes as necessary.
** Proof of Concept**
Create a proof of concept - take the design doc, specs sheet and convert it to code. You will be using UIKit as the CSS library along with Fontawesome PRO icons. Do not worry about logic during this step, just build out the component.
The expected deliverable will be a component that is responsive and mobile-ready. Make sure you address all stories requested (states) - if time permits. The next task will deal with adding any additional logic that might be needing. For example, creating API requests, iterating through data, etc.
Implement the Proof of Concept
Take the proof of concept and add missing logic. This means take it from static and make it dynamic if it requires it. Build it to be modular, add comments and make sure component is polished.
Design:

Is this task available?


ValueError: Variable kernel already exists, disallowed. Did you mean to set reuse=True or reuse=tf.AUTO_REUSE in VarScope?

I'm currently working on an LSTM based research problem. However, while using the RNNs in keras, as I will demonstrate below, I'm running into the above-mentioned error.
I use TF version 1.12.0 and keras 2.2.4.
This seems to work with cells like LSTMCell but fails to work with UGRNNCell.. No idea how to fix this issue.
cell1=tf.contrib.rnn.UGRNNCell(64)
cell2=tf.contrib.rnn.UGRNNCell(64)
cell3=tf.contrib.rnn.UGRNNCell(64)
cell4=tf.contrib.rnn.UGRNNCell(64)

And this is my model:
model = Sequential()

model.add(RNN(cell1, input_shape=(train_X.shape[1:]),return_sequences=True))
model.add(BatchNormalization())
model.add(Dropout(0.5))

model.add(RNN(cell2,return_sequences=True))
model.add(BatchNormalization())
model.add(Dropout(0.5))

model.add(RNN(cell3, return_sequences=True))
model.add(BatchNormalization())
model.add(Dropout(0.5))

model.add(RNN(cell4,return_sequences=False))
model.add(BatchNormalization())
model.add(Dropout(0.5))

model.add(Dense(128, activation='relu'))
model.add(BatchNormalization())
model.add(Dropout(0.5))

model.add(Dense(1, activation='sigmoid'))

When run this is the error obtained:
Traceback (most recent call last):
  File "fastcell.py", line 43, in <module>
    model.add(RNN(cell2,return_sequences=True))
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\python\training\checkpointable\base.py", line 474, in _method_wrapper
    method(self, *args, **kwargs)
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\python\keras\engine\sequential.py", line 175, in add
    output_tensor = layer(self.outputs[0])
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\python\keras\layers\recurrent.py", line 619, in __call__
    return super(RNN, self).__call__(inputs, **kwargs)
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\python\keras\engine\base_layer.py", line 757, in __call__
    outputs = self.call(inputs, *args, **kwargs)
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\python\keras\layers\recurrent.py", line 750, in call
    input_length=timesteps)
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\python\keras\backend.py", line 3201, in rnn
    outputs, _ = step_function(inputs[0], initial_states + constants)
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\python\keras\layers\recurrent.py", line 737, in step
    output, new_states = self.cell.call(inputs, states, **kwargs)
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\contrib\rnn\python\ops\rnn_cell.py", line 1683, in call
    self._linear = _Linear(cell_inputs, 2 * self._num_units, True)
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\contrib\rnn\python\ops\core_rnn_cell.py", line 99, in __init__
    initializer=kernel_initializer)
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\python\ops\variable_scope.py", line 1487, in get_variable
    aggregation=aggregation)
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\python\ops\variable_scope.py", line 1237, in get_variable
    aggregation=aggregation)
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\python\ops\variable_scope.py", line 540, in get_variable
    aggregation=aggregation)
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\python\ops\variable_scope.py", line 492, in _true_getter
    aggregation=aggregation)
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\python\ops\variable_scope.py", line 861, in _get_single_variable
    name, "".join(traceback.format_list(tb))))
ValueError: Variable kernel already exists, disallowed. Did you mean to set reuse=True or reuse=tf.AUTO_REUSE in VarScope? Originally defined at:

  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\python\framework\ops.py", line 1770, in __init__
    self._traceback = tf_stack.extract_stack()
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\python\framework\ops.py", line 3274, in create_op
    op_def=op_def)
  File "d:\Users\prane\Anaconda3\lib\site-packages\tensorflow\python\util\deprecation.py", line 488, in new_func
    return func(*args, **kwargs)

This results in the above error but works seamlessly and without any error when the UGRNNCell is replaced by an LSTMCell.
Any help would be appreciated.

I'm having the same error

So the issue is that the contrib UGRNNCell has been quite out of date, and was using the latest api to create its weights. we probably won't update the contrib code anymore since contrib is going away in TF 2.0. Having said that I could move the implementation of UGRNNCell to tensorflow/addons, which is a new repository for extended tf features.
Will do this in a week or two.

Any solution for this?

Maybe using the parameter stateful=True can solve the issue.


Broke Image Links

Images files are not able to download from given resources. I need them as soons as possible. If somehow I can get them, I will make them reachable for everyone.

I tried Waybak Machine snapshots but them also empty.

This problem might be related with my locations, I will aslo try with VPN.

Indeed, location is the problem here. I've just tried to download the image and there is no problem whatsoever.
I will place a copy on Gdrive. Hopefully you can download it. It will take another hour before the upload of 9 GB is completed.....

Please find a copy at GDrive.

Appreciate it. Thank you for your efforts 💖


Config settings for MongoDB Backup Agent

Codecov Report

Merging #1442 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #1442   +/-   ##
=======================================
  Coverage   98.37%   98.37%
=======================================
  Files          56       56
  Lines        2592     2592
=======================================
  Hits         2550     2550
  Misses         42       42

Does this work?

No errors so far. Backups are active on Cloud Manager and I can see logsthat data is being sync'ed.


direct connection to Modbus TCP

What happened?
Hello,
I am currently trying to integrate my Growatt SPH-10000 BL-UP into Loxone (a smart home system) which itself acts as a Modbus client. Is it possible to obtain direct access to the Modbus?
Sorry for the probably stupid question, but I am just not making any progress.
Thank you for your help.

Shine X

Which Inverter are you using?

SPH-10000 BL-UP

You cannot. May I kindly ask you to take feature requests to the forum?


AMBARI-23749. Support POST on all service / audit logs endpoint.

What changes were proposed in this pull request?
Support POST for every service / audit endpoints (some of them are missing ... it would not make that much sense to support POST, but for UI it will be easier to use POST everywhere)
How was this patch tested?
Please review @g-boros @kasakrisz

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/Ambari-Github-PullRequest-Builder/2103/
Test PASSed.


Android does not re-measure its layout on Height Changed

I have an absolute layout that takes up the whole screen which contains a carousel view (which contains a listview).
I then have a button that i press and this halves the height of the absolute layout (and subsequently carousel view)
But the view inside the carousel view does not remeasure itself meaning my listview does not re-evaluate whether scrolling should be enabled (i.e. when whole screen only half the listview on the screen is filled with items, hence no scroll enabled, when halfed now items are cut off so should enable scrolling but as carousel view does not re-measure itself scroll is not enabled).
If I look at Android
I can see it doesnt relayout unless the width of the control changes
if (ElementWidth.Equals(rect.Width))
      return;

On ios it remeasures every time size has changes as long as height > 0 which makes sense

I notice that ElementHeight has been commented out. To fix this possibly should it be:
if (ElementWidth.Equals(rect.Width) || ElementHeight.Equals(rect.Height))
      return;

ElementWidth = rect.Width;
ElementHeight = rect.Height;

Can you provide a sample code of the scenario you are using the Carousel?

If i do this change, then when soft keyboard appears, the entire thing will resize which is wrong.

CSSizeTest.zip
I've attached a sample.
Steps to reproduce:

Click "Add to list button" untill you have over half the list full
Press the "Resize" button and that will half the size of the carousel view
the listview does not resize itself as the carousel has not resized so therefore scrolling is not enabled

I'd imagine this would be a problem for other controls inside a Carousel not just a list.
You say it would re-layout when the soft keyboard appears, I think that the carousel should re-layout when the keyboard appears. If the height or width of the carousel has changed then the content inside should also change regardless of why it changed

Yeah but then the input will lose focus and the keyboard will be gone.

@alexrainman so from what I understand the problem comes when you click an Entry and it pops up the keyboard - which would resize the view and relayout the view and unfocus the Entry?
I dont think not resizing the View on Height changed is a solution for the Entry bug. I think it will cause more issues in the long run

Agree. Im trying to find a way to resize the view and focus the previously focused entry.

I think that is the correct solution. I will have a think about it to and let you know if I get a solution to re-focusing the Entry after keyboard appeared

I just released a new version but a fix for this still not there as i have to handle a complicated scenario related to soft-keyboard.

I also have this issue. Waiting for the fix.

Updated in next release.


doesn't work on Core 2024.5

Hi Marko,
thank you first for the great integration, which has worked wonderfully so far. Unfortunately, it is no longer compatible with Core 2024.5. The lights no longer indicate the correct state, and the binary sensors have completely gone. I had to revert to 2024.4.4 so that the house is not completely dead:(
Thanks in advance,
Manuel

I have the same issue, inputs not detected in HA 2024.5 and light status no longer correct (but it does switch them on or off).
Log:
Logger: homeassistant.util.logging
Bron: util/logging.py:98
Eerst voorgekomen: 14:53:29 (745 gebeurtenissen)
Laatst gelogd: 16:57:06
Exception in _update_callback when dispatching 'unipi_neuron_L207_PLC_input_UART_3_3_02': () Traceback (most recent call last): File "/config/custom_components/unipi_neuron/binary_sensor.py", line 105, in _update_callback self.async_write_ha_state() File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1008, in async_write_ha_state self.hass.verify_event_loop_thread("async_write_ha_state") File "/usr/src/homeassistant/homeassistant/core.py", line 440, in verify_event_loop_thread frame.report( File "/usr/src/homeassistant/homeassistant/helpers/frame.py", line 162, in report _report_integration(what, integration_frame, level, error_if_integration) File "/usr/src/homeassistant/homeassistant/helpers/frame.py", line 203, in _report_integration raise RuntimeError( RuntimeError: Detected that custom integration 'unipi_neuron' calls async_write_ha_state from a thread at custom_components/unipi_neuron/binary_sensor.py, line 105: self.async_write_ha_state(). Please report it to the author of the 'unipi_neuron' custom integration.
Exception in _update_callback when dispatching 'unipi_neuron_L207_PLC_input_UART_3_3_04': () Traceback (most recent call last): File "/config/custom_components/unipi_neuron/binary_sensor.py", line 105, in _update_callback self.async_write_ha_state() File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1008, in async_write_ha_state self.hass.verify_event_loop_thread("async_write_ha_state") File "/usr/src/homeassistant/homeassistant/core.py", line 440, in verify_event_loop_thread frame.report( File "/usr/src/homeassistant/homeassistant/helpers/frame.py", line 162, in report _report_integration(what, integration_frame, level, error_if_integration) File "/usr/src/homeassistant/homeassistant/helpers/frame.py", line 203, in _report_integration raise RuntimeError( RuntimeError: Detected that custom integration 'unipi_neuron' calls async_write_ha_state from a thread at custom_components/unipi_neuron/binary_sensor.py, line 105: self.async_write_ha_state(). Please report it to the author of the 'unipi_neuron' custom integration.
Exception in _update_callback when dispatching 'unipi_neuron_L207_PLC_input_UART_2_2_12': () Traceback (most recent call last): File "/config/custom_components/unipi_neuron/binary_sensor.py", line 105, in _update_callback self.async_write_ha_state() File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1008, in async_write_ha_state self.hass.verify_event_loop_thread("async_write_ha_state") File "/usr/src/homeassistant/homeassistant/core.py", line 440, in verify_event_loop_thread frame.report( File "/usr/src/homeassistant/homeassistant/helpers/frame.py", line 162, in report _report_integration(what, integration_frame, level, error_if_integration) File "/usr/src/homeassistant/homeassistant/helpers/frame.py", line 203, in _report_integration raise RuntimeError( RuntimeError: Detected that custom integration 'unipi_neuron' calls async_write_ha_state from a thread at custom_components/unipi_neuron/binary_sensor.py, line 105: self.async_write_ha_state(). Please report it to the author of the 'unipi_neuron' custom integration.
Exception in _update_callback when dispatching 'unipi_neuron_L207_PLC_relay_3_13': () Traceback (most recent call last): File "/config/custom_components/unipi_neuron/light.py", line 169, in _update_callback self.async_write_ha_state() File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1008, in async_write_ha_state self.hass.verify_event_loop_thread("async_write_ha_state") File "/usr/src/homeassistant/homeassistant/core.py", line 440, in verify_event_loop_thread frame.report( File "/usr/src/homeassistant/homeassistant/helpers/frame.py", line 162, in report _report_integration(what, integration_frame, level, error_if_integration) File "/usr/src/homeassistant/homeassistant/helpers/frame.py", line 203, in _report_integration raise RuntimeError( RuntimeError: Detected that custom integration 'unipi_neuron' calls async_write_ha_state from a thread at custom_components/unipi_neuron/light.py, line 169: self.async_write_ha_state(). Please report it to the author of the 'unipi_neuron' custom integration.
Exception in _update_callback when dispatching 'unipi_neuron_L207_PLC_relay_3_11': () Traceback (most recent call last): File "/config/custom_components/unipi_neuron/light.py", line 169, in _update_callback self.async_write_ha_state() File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1008, in async_write_ha_state self.hass.verify_event_loop_thread("async_write_ha_state") File "/usr/src/homeassistant/homeassistant/core.py", line 440, in verify_event_loop_thread frame.report( File "/usr/src/homeassistant/homeassistant/helpers/frame.py", line 162, in report _report_integration(what, integration_frame, level, error_if_integration) File "/usr/src/homeassistant/homeassistant/helpers/frame.py", line 203, in _report_integration raise RuntimeError( RuntimeError: Detected that custom integration 'unipi_neuron' calls async_write_ha_state from a thread at custom_components/unipi_neuron/light.py, line 169: self.async_write_ha_state(). Please report it to the author of the 'unipi_neuron' custom integration.

Okay I did some searching and replacing async_write_ha_state() with schedule_update_ha_state() seems to do the trick.
Changed it in binary_sensor.py and light.py. Cover.py already had this for some reason?

I'll try to use your suggested solution in the next few days (half of my house didn't work after the update). A big thank you for the quick support, Manuel

I am not sure how active the main contributor still is in this github code, but in my logs I see a few alerts a out code that will be deprivated in 2025, so be aware of that...

Hi @anline380  and @Geunsjl.
Sorry for late response. I am still on an older version of HA and do not update it regularly. I will look into this issue as soon as I find some time (Hopefully by the end of this week).
Regards
Marko

Hi @anline380 and @Geunsjl.
I have made required updates and fixes to work with latest HA core. Please check it out.

@marko2276 I just tested it and it lights/inputs seem to work. I use 2x the XS11 so it complained about the config not being valid (see image). I changed the regex back to the previous one to allow UART in the config.

Hi @Geunsjl
Which regex did you change to previous one (as in my commit, I do not recall of changing any regex?. Was it perhaps changed in some older commits?
Anyhow, can you please open a formal pull request, so that we can include your fixes?
Regards,
Marko


Search And replace history

Godot version:
3.1
OS/device including version:
Not relevant
Issue description:
It would be great to have search & replace history so we won't need to type the same search terms repeatedly.
It should probably save the last X search/replace texts.
Steps to reproduce:
Minimal reproduction project:

Feature and improvement proposals for the Godot Engine are now being discussed and reviewed in a dedicated Godot Improvement Proposals (GIP) (godotengine/godot-proposals) issue tracker. The GIP tracker has a detailed issue template designed so that proposals include all the relevant information to start a productive discussion and help the community assess the validity of the proposal for the engine.
The main (godotengine/godot) tracker is now solely dedicated to bug reports and Pull Requests, enabling contributors to have a better focus on bug fixing work. Therefore, we are now closing all older feature proposals on the main issue tracker.
If you are interested in this feature proposal, please open a new proposal on the GIP tracker following the given issue template (after checking that it doesn't exist already). Be sure to reference this closed issue if it includes any relevant discussion (which you are also encouraged to summarize in the new proposal). Thanks in advance!


Test multiple CI keywords

This pull request should not be merged. I created it to test multiple keywords for CI and planning to just close it afterwards.

/intelci: run default, ml_benchmarks

/intelci: run ml_benchmarks

/intelci: run ml_benchmarks

/intelci: run license-analysis

/intelci: run license-analysis

/intelci: run license-analysis


Need for detailed wireless settings.

We need a detailed list of reccomended wireless settings besides 5ghz only/different ssid's and 40mhz channel width.
Please provide detailed settings for commonly available properties.
for instance for channel width i can choose 20mhz or AUTO, there is no 40mhz.
fat channel intolerant?
Mixed mode protection?
packet coalescing?
roaming agressivness?
throughput booster?
etc.

one thing i have found is that too high a channel will cause the quest to loose connection.. my current channel of 112 is perfect and has yet to suddenly loose connection. but above 130 it looses connection a ton.
as for specific settings, i don't now besides using an AP instead of a wifi card(usb of pcie), as the AP is easier to configure, depending on your existing network of course. i for instance use an RT-N66U for my networking, and it takes care of the wireless signal upstairs(where i game and sleep) and i use my modem for downstairs.


Move all example manifests in us-west-1 to us-east-2

Description of your changes
The us-west-1 AWS region, like most regions, has three availability zones, but only two are available to new accounts. This means that depending on which AWS account you're running in, some of the example manifests may reference an az code that doesn't exist in your account (I have an account that only allows us-west-1b and us-west-1c, but I think the upbound account used by github actions only allows us-west-1a and us-west-1b).
The simplest way to avoid this is to just avoid the us-west-1 region entirely. This PR replaces all instances of "us-west-1" in the examples directory with "us-east-2". us-east-2 is a region that is available to all accounts by default, which always has three AZs available, which doesn't have some of the "special" behavior associated with us-east-1, and which is only referenced in a few of the example manifests.
I have:

[ ] Read and followed Crossplane's contribution process.
[ ] Run make reviewable to ensure this PR is ready for review.
[ ] Added backport release-x.y labels to auto-backport this PR if necessary.

How has this code been tested

Before this change, I found the following distribution of regions in the example manifests:
us-west-1: 640 files
us-east-1: 201 files
us-east-2: 20 files
us-west-2: 32 files
eu-west-1: 6 files
After this change, the distribution was
us-west-1: 0 files
us-east-1: 201 files
us-east-2: 659 files
us-west-2: 33 files
eu-west-1: 6 files
The counts are off by one from what you'd expect from a straight replacement because I additionally edited the db instance automated backup replication manifest, which had previously used both us-east-2 and us-west-1, to use us-east-2 and us-west-2.
Uptest runs:
:heavy_check_mark:  db instance automated backup replication: https://github.com/crossplane-contrib/provider-upjet-aws/actions/runs/8774463999

/test-examples="examples/rds/v1beta1/dbinstanceautomatedbackupsreplication.yaml"

/test-examples="examples/kms/v1beta1/replicaexternalkey.yaml"

Thank you for your effort and interest in this PR @mbbush, as far as I can see here, we have changed the region in many of the existing resource examples. My only doubt here is the possibility of uptestable resources failing due to the region change. So it would be great if we could trigger to uptest from commonly used resources before moving this PR forward.


build/Makefile: update ngfx-install rule

updated ngfx build: ported gnu makefile for building 3rd party libs to ninja
this is the corresponding change


Lazy Import Bundling

I'm wondering if there is a better way to handle lazy-imports with bundling other than requiring that they all be fragments. Please bear with me, this is kinda half-baked lol
Bundle each group of lazy-imports into their own bundle with the exception of; a single import in the group, imports that are included in fragments, and imports that show up in multiple places, which would be treated as implied fragments. These lazy-import links, which make up the bundle, would be replaced with the bundled href.
Again, this was just something I thought of and thought it might be cool to implement. Let me know what you think! @usergenic
PSA: Copious amounts of partial code examples
Examples
Single Import
Before:
<dom-module id="my-app">
    <link rel="lazy-import" group="view1" href="my-view1.html">
    <link rel="lazy-import" group="view2" href="my-view2.html">

After:
<dom-module id="my-app">
    <link rel="lazy-import" group="view1" href="my-view1.html">
    <link rel="lazy-import" group="view2" href="my-view2.html">

Multiple Imports
Before:
<dom-module id="my-app">
    <link rel="lazy-import" group="lazy" href="component-1.html">
    <link rel="lazy-import" group="lazy" href="component-2.html">

After:
<dom-module id="my-app">
    <link rel="lazy-import" group="lazy" href="lazy-bundled-components.html">

Multiple Imports with Fragment
Before:
{
  "fragments": [
    "components-1.html"
  ]
}

<dom-module id="my-app">
    <link rel="lazy-import" group="lazy" href="component-1.html">
    <link rel="lazy-import" group="lazy" href="component-2.html">

After:
<dom-module id="my-app">
    <link rel="lazy-import" group="lazy" href="component-1.html">
    <link rel="lazy-import" group="lazy" href="component-2.html">

Multiple Imports with same import in another file
<dom-module id="my-app">
    <link rel="lazy-import" group="lazy" href="component-1.html">
    <link rel="lazy-import" group="lazy" href="component-2.html">

<dom-module id="view1">
    <link rel="lazy-import" group="lazy" href="component-2.html">

After:
<dom-module id="my-app">
    <link rel="lazy-import" group="lazy" href="component-1.html">
    <link rel="lazy-import" group="lazy" href="component-2.html">

<dom-module id="view1">
    <link rel="lazy-import" group="lazy" href="component-2.html">

or with enough to bundle
Before:
<dom-module id="my-app">
    <link rel="lazy-import" group="lazy" href="component-1.html">
    <link rel="lazy-import" group="lazy" href="component-2.html">
    <link rel="lazy-import" group="lazy" href="component-4.html">

<dom-module id="view1">
    <link rel="lazy-import" group="lazy" href="component-2.html">

After:
<dom-module id="my-app">
    <link rel="lazy-import" group="lazy" href="lazy-bundled-components.html">
    <link rel="lazy-import" group="lazy" href="component-2.html">

<dom-module id="view1">
    <link rel="lazy-import" group="lazy" href="component-2.html">

@stramel I do like the idea of bundling lazy imports together that are part of the same group and are exclusive to that group.  Seems like a cool optimization, thought I wonder how frequently that scenario appears in the wild?

One additional caveat, wrt to the last example, is that the order of the imports changes if you bundle component-1 and component-4 together and then load component-2.  That might violate the expectation of someone who constructed their code (even unintentionally) to be order-dependent.
We try very hard to preserve the execution order of imports in the bundler, though in the case of lazy imports that's probably something that might need to be more relaxed anyways.

wrt import order, that is a good point that I didn't think about.
Another thing to keep in mind that we also support :not([group]) as a "group" of lazy-imports.
Thank you for the feedback, it would be interesting to know how often this kind of optimization could be applied. I will try to keep an eye out locally but I'm not sure how to get more detail about it.

I'm going to go ahead and close this. With P3 moving towards ES Modules, lazy-imports will be going away.


Bigtable API Client: 10 times more requests starting with release 2.10.0 resulting in 50% more cloud costs

Environment details

Bigtable Client Library for Java (v 2.2.0 --> 2.10.0)
OS type and version: Appengine 2nd Generation
Java version: Java OpenJDK 11
bigtable version(s):  (v 2.2.0 --> 2.10.0)

Steps to reproduce

Update Bigtable Client Library from version 2.2.0 to 2.10.0 or higher
Deploy code sample using bulkMutateRows method of com.google.cloud.bigtable.data.v2.BigtableDataClient
Observation Bigtable API Requests increase by factor 10 via APIs & Services
Version 2.2.0: 1 requests
Version 2.10.0: ~ 10 requests per bulk mutate row call

Code example
	BulkMutation bulkMutation = BulkMutation.create(tableId);
	for (int row = 0; row < data.getValues().size(); row++) {
		...
		bulkMutation.add(rowKey, rowMutation);
	}
	dataClient.bulkMutateRows(bulkMutation);

External references such as API reference guides

release-notes#August_01_2022

Any additional information below
We are using a bom based approach which keeps the google java client libraries in sync across all used services (Datastore, Bigquery, Cloud Storage, Bigtable, Cloud Tasks)
The specific method that we use to ingest data to bigtable is bulk mutate rows.
After updating & deploying we noticed that the bigtable api requests were 10 times higher compared to the previous deployed version.
We narrowed it down being a problem starting with version 2.10.0 of the bigtable API client for Java https://cloud.google.com/bigtable/docs/release-notes#August_01_2022
It seems to be related to PingAndWarm requests being introduced before each interaction with bigtable in that specific version.
Impact: 300% Latency increase on bigtable operations --> 50% More AppEngine Instances Spawned --> 40 - 50% more cloud costs.
We now decided to downgrade to Version 2.9.0 - but we are no longer in sync with other google client libraries since this is a manual maintenance step.
Because of this we want to understand:

What is the reason for this change in the big table client library?
What is the suggested way of addressing this Issue - since 10x more API requests is not suitable for us?
Are there any changes we could implement to be able to use the big table API efficiently without downgrading?

Any insights on this topic is much appreciated.

Hi,
PingAndWarm requests are sent once per channel during channel creation. Which happens when the client is initialized and every 50 mins there after. The purpose of PingAndWarm is to minimize perceived latency of a cold connection.By sending an RPC on each channel  we ensure that it is established and all of the caches along the way have been warmed.
The lifecycle of the ChannelPool is tied to the client, which should be tied to life of your application process and should not be re-created per request. So to avoid 10x blow, please make sure you keep the client alive. 1 rpc per channel / 50 mins should not have a meaningful impact on a service thats expected to handle 10k qps per node.
If there is something that I'm missing and there is a usecase for shortlived clients or disabling channel priming, please reopen the ticket.
Thanks


build: Update to Vulkan-Headers 1.3.247

There are no changes to the generated code from this update.

CI Vulkan-ExtensionLayer build # 331 running.

Do we produce Android binaries for the extension layers?


Refactoring tutorials for the new changes in the prometheus-service

Refactoring doc for the new changes in the prometheus-service
Issue- https://github.com/keptn-contrib/prometheus-service/issues/53
Signed-off-by: Raj Das<EMAIL_ADDRESS>

It looks like something is broken. I looked at the preview and the tutorial has some steps missing. The root cause here is most probably that the formatting is broken and the next pages could not be rendered.

Once this tutorial is fixed, we also need to replicate the updates in the other tutorial, such as

https://tutorials.keptn.sh/tutorials/keptn-multistage-qualitygates-08/index.html?index=..%2F..index#0
https://tutorials.keptn.sh/tutorials/keptn-litmus-08/index.html?index=..%2F..index#0

Please remove the instructions that are not necessary from this step and only keep the ones that need to be executed.

We'll wait for the merge of Prometheus-service and Prometheus-sli-service to be completed and then will upgrade this tutorial to reflect the new changes


put .apk in releases for obtainium

I added the latest release here: https://github.com/andstatus/game2048/releases/tag/1.14.3-release

that works, jusf keep posting them every update. thanks


Nested series of assignments takes ~quadratic time to parse

TypeScript Version: 3.5.0-dev.20190409 (also 3.4.2)

Search Terms: nested, parsePossibleParenthesizedArrowFunctionExpressionHead
Code
a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=a.a).a).a).a).a).a).a).a).a).a).a).a).a).a).a).a).a).a).a).a

Alternate test bench:
const ts = require("typescript");
const source = `a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=(a=a.a).a).a).a).a).a).a).a).a).a).a).a).a).a).a).a).a).a).a).a`;

ts.transpileModule(source, {});

Expected behavior:
Parses in a timely manner.
Actual behavior:
tsc takes 5 seconds, and this time increases ~quadratically as the repetition is increased. In our case this quickly grew to hours.
Playground Link:
Playground (Warning, this is a little intense and might hang the browser)
Related Issues:
Not that I could find
Some investigation:
Looking at profile output, it looks like the following might be happening:

Arrive at first (
Could be a parenthesized arrow function (based on (a=...)
Pursue that theory, start consuming first default assigned property
Arrive at second (
Could be a parenthesized arrow function (based on (a=...)
Repeat until we finally reach the middle, realise there was no arrow function, backtrack
We now know the outermost set of parens wasn't an arrow, but an assignment
Arrive at second ( (again)
Repeat the whole process

But, why?:
Received output of this style from terser (formerly uglify-es) from original code of the form:
str = str.replace("a", "b");
str = str.replace("c", "b");

We happen to need to do some AST based post-processing.

This was fixed by #32640; #31987 turned out to be a duplicate of this issue (and has effectively the same test case).


KRR install missing python modules aiostream and slack-sdk

After manually installing KRR https://github.com/robusta-dev/krr#manual-installation, I noticed the following modules were missing:

aiostream
slack-sdk

To Reproduce

Follow manual install instructions
See the error for slack-sdk. (if you manually install via pip the error goes away https://pypi.org/project/slack-sdk/)
If step 2 is resolved using workaround you will then see the error for aiostream. (if you manually install via pip error goes away https://pypi.org/project/aiostream/)

Expected behavior
slack-sdk and aiostream installed properly via requirements.txt file.
Suggested Fix
Add these 2 dependencies slack-sdk and aiostream  to the requirements.txt file
Error Messages
slack-sdk
python3 krr.py --help

Traceback (most recent call last):
  File "/Users/calvincarter/_programming/docker/development/krr_install/krr/krr.py", line 1, in <module>
    from robusta_krr import run
  File "/Users/calvincarter/_programming/docker/development/krr_install/krr/robusta_krr/__init__.py", line 1, in <module>
    from .main import run
  File "/Users/calvincarter/_programming/docker/development/krr_install/krr/robusta_krr/main.py", line 16, in <module>
    from robusta_krr.core.runner import Runner
  File "/Users/calvincarter/_programming/docker/development/krr_install/krr/robusta_krr/core/runner.py", line 6, in <module>
    from slack_sdk import WebClient
ModuleNotFoundError: No module named 'slack_sdk'

aiostream
 python3 krr.py --help
Traceback (most recent call last):
  File "/Users/calvincarter/_programming/docker/development/krr_install/krr/krr.py", line 1, in <module>
    from robusta_krr import run
  File "/Users/calvincarter/_programming/docker/development/krr_install/krr/robusta_krr/__init__.py", line 1, in <module>
    from .main import run
  File "/Users/calvincarter/_programming/docker/development/krr_install/krr/robusta_krr/main.py", line 16, in <module>
    from robusta_krr.core.runner import Runner
  File "/Users/calvincarter/_programming/docker/development/krr_install/krr/robusta_krr/core/runner.py", line 10, in <module>
    from robusta_krr.core.integrations.kubernetes import KubernetesLoader
  File "/Users/calvincarter/_programming/docker/development/krr_install/krr/robusta_krr/core/integrations/kubernetes.py", line 4, in <module>
    import aiostream
ModuleNotFoundError: No module named 'aiostream'

Desktop (please complete the following information):

OS: [Mac]
Version [Ventura 13.5]

Fixed in https://github.com/robusta-dev/krr/pull/124


Improve clarity and fix Docs for Logging

What were you initially searching for in the docs?

[ ] Don't use service_undefined in Java, use Null or String.Empty.
[x] Use units to represent the percentage, clarity over the 0.1. (http://jsr-108.sourceforge.net/javadoc/javax/units/NonSI.html#PERCENT) I'll review the javadocs of it to make it explicitly whether we should be using 0.01, 0.1, 10, etc.
[ ] Message, Unserializable JSON values will be casted to string, question: How? calling toString()? if so, tell that toString() will be called. Also, what about exceptions there? Will that be swollen?
[ ] Version is great, what about logging the alias?
[ ] Please, make this a warning/info box, this is too subtle the way it is today and if this precedence is respected elsewhere, create a section for know before you go. Configuration on environment variable is given precedence over sampling rate configuration on annotation, provided it's in valid value range.

Is this related to an existing part of the documentation? Please share a link
https://awslabs.github.io/aws-lambda-powertools-java/core/logging/#standard-structured-keys
Describe how we could make it clearer
Address the points listed
If you have a proposed update, please share it here

I don't think it's possible to get the alias from the context - https://docs.aws.amazon.com/lambda/latest/dg/java-context.html
Do you have any other suggestions?

I think service_undefined works well here, it matches the python implementation and gives us a service name to use when we implement Metrics. We'll need a reasonable default for that core util as well.

For sampling, I'm happy with the String / Double approach. If this was a public method I think it would be best as a unit. But because it's an environment variable I think it would only increase complexity.

I think service_undefined works well here, it matches the python implementation and gives us a service name to use when we implement Metrics. We'll need a reasonable default for that core util as well.

This is misleading, if we don't know the name, I would assume that is better to default to the FunctionName or even FunctionName:Version, until the customer properly configures it.

I don't think it's possible to get the alias from the context - https://docs.aws.amazon.com/lambda/latest/dg/java-context.html
Do you have any other suggestions?

getInvokedFunctionArn() – Returns the Amazon Resource Name (ARN) that's used to invoke the function. Indicates if the invoker specified a version number or alias.

For sampling, I'm happy with the String / Double approach. If this was a public method I think it would be best as a unit. But because it's an environment variable I think it would only increase complexity.
I think the point here is readability, if the argument is called samplingRate, I have to lookup/assume which is the actual unit, but this is totally style. Hopefully we have more inputs here.

W.r.t - service_undefined provides an opportunity for the customer to fix it. Service has also a deeper meaning as it acts as a namespace when looking for data across multiple functions be that logs, metrics or traces.
As to context, I don't know about Java, but in the Python context object it'll fetch a FQDN -- If it's a given published version or alias it'll be the last value <function_name>:<version_or_alias>

I'd be really happy to look at a PR for adding alias information.

W.r.t - service_undefined provides an opportunity for the customer to fix it. Service has also a deeper meaning as it acts as a namespace when looking for data across multiple functions be that logs, metrics or traces.

100%, but as soon as a customer incorporates that into multiple functions they will see service_undefined in multiple logstreams, or am I missing something? This goes hand-in-hand with tracing span naming which I'm in full agreement.
Now for the Alias, I'm seeking guidance on whether there's any logic behind or simply callee information.

That is correct - Hence the emphasis on POWERTOOLS_SERVICE_NAME in the docs
for all core utilities, including our blog post and examples.
I’d be happy to log a warning if service name is not found. I would not use
function name as the service name if not found as that will break many
workflows, and their current understanding of service being a namespace
they can search across core utilities too
On the alias bit, it should be easy to test with a deployed function -
Publish a version from the Console, and invoke that function from that
version — It should reflect in the context object, as it’s injected by the
bootstrap runtime process
On Fri, 4 Sep 2020 at 03:57, Diego Magalhães<EMAIL_ADDRESS>wrote:

W.r.t - service_undefined provides an opportunity for the customer to fix
it. Service has also a deeper meaning as it acts as a namespace when
looking for data across multiple functions be that logs, metrics or traces.
100%, but as soon as a customer incorporates that into multiple functions
they will see service_undefined in multiple logstreams, or am I missing
something? This goes hand-in-hand with tracing span naming which I'm in
full agreement.
Now for the Alias, I'm seeking guidance on whether there's any logic
behind or simply callee information.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/awslabs/aws-lambda-powertools-java/issues/69#issuecomment-686854231,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAZPQBFAQ3MSOTX3KQKJCATSEBCQFANCNFSM4QRKWE3A
.

Alias as part of the context was something discussed in the aws-lambda-java-libs project.
https://github.com/aws/aws-lambda-java-libs/issues/10
You might want to continue the discussion there.

Message, Unserializable JSON values will be casted to string, question: How? calling toString()? if so, tell that toString() will be called. Also, what about exceptions there? Will that be swollen?

This should be fixed as part of #113

@dgomesbr Can this issue be closed now based on some of the discussions above? 🤔

For sure. Thank you all!


Missing value

I wonder why I am the only person complaining about it, but plugin doesn't return value

Nevermind, I'm blind


Make rbac not dependent on framework core pacakges

What?
rbac package should not depend on any other package except PSR interfaces such as PSR-3 for logging or PSR-15 for cache.
It may depend on https://github.com/yiisoft/cache and https://github.com/yiisoft/log for development via require-dev composer section for running tests and development.
Why?
The RBAC package is general enough not to require anything from the core framework. If we'll make it dependency-free it could be used as a library thus expanding its adoption.
What do with DbManager?
Try relying on PDO.

to be noted: This package still requires yii-core for \yii\helpers\VarDumper and \yii\base\Component

Is there any need for \yii\base\Component?


Plan breaks when unifying with dagger.#Plan multiple times

If the dagger.#Plan definition is merged multiple times, it breaks the plan (I don't have the error handy).
For instance:

a.cue

dagger.#Plan & {
  actions: a: ...
}

b.cue

dagger.#Plan & {
  actions: b: ...
}

dagger do -p ./a.cue will work, but dagger do won't (e.g. loading twice).
I believe @marcosnils hit the same issue in a different way (e.g. a.cue and subdir/b.cue, running dagger do from subdir breaks because #Plan is unified twice).
Removing #Plan from every file except one (e.g. just using actions: b: ... rather than #Plan & { actions: b: ... }) fixes the problem.
I assume it's because we're doing stuff in #Plan that breaks when done multiple times.
/cc @helderco

I believe @marcosnils hit the same issue in a different way (e.g. a.cue and subdir/b.cue, running dagger do from subdir breaks because #Plan is unified twice).

Yep, exactly this behavior. IIRC unification breaks with client filesystem mounts only  IIRC since the CUE evaluator can't resolve to what it should ultimately be.

This has come up a few times. I've just recomended to specify the plan with -p or keep only one plan in the same dir. Do we want to support unifying multiple plans together? I think that'll be confusing, but also if it's not on purpose and there's unexpected behavior because of that. So we should at least detect this situation and error early.
I first tried this back in january when trying to break a big plan:

https://github.com/dagger/dagger/issues/1316

@helderco Do we know what the root cause is? In theory there shouldn't be an issue unifying #Plan multiple times

At the time I ruled it down to scoping issues. But I can give it another look.

@aluzzardi , can we close this one out now?

Yep


.itervalues() => .values() python3 compatibility

not sure how to target compatibility for both python2 and python3 since the drawbacks of using .values() in python2 are obvious. otherwise, this is a patch for python3.

#31

@blake, the drawbacks are not so obvious in this case; the number of labels on a single resource is unlikely to be large enough to cause a problem.
Your patch will not work on Python 3, however: next() is no longer a method. I just tested and submitted this PR.

Thanks to both of you for discovering and releasing a fix, I've added a bugfix in the 1.3.0 branch, now in testing phase.
I finally used popitem(), which seems to work in Python 2 & 3 and it is straightforward,


Links can't contain '?' character

If a URL contains a % character, then the markup isn't converted to a link.
Example:
Click Me

Should be fixed with #18
Released version 1.1.2


Feat/broker register

features

Add Delete method for broker repository
Add Delete method for broker controller
Add unit testing

Codecov Report

Merging #7 (e66ef92) into main (5774057) will increase coverage by 12.89%.
The diff coverage is 77.77%.

@@             Coverage Diff             @@
##             main       #7       +/-   ##
===========================================
+ Coverage   73.25%   86.15%   +12.89%
===========================================
  Files           3        3
  Lines          86      130       +44
===========================================
+ Hits           63      112       +49
+ Misses         22       18        -4
+ Partials        1        0        -1

Impacted Files
Coverage Δ

...rvices/inventory/broker/repository/broker_mocks.go
0.00% <0.00%> (ø)

...rc/services/inventory/broker/controllers/broker.go
96.55% <100.00%> (+36.08%)
:arrow_up:

...rvices/inventory/broker/repository/broker_mysql.go
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more