Permissions and Modules

Hey, thanks a lot for the program, seems great indeed.
I was wondering if there's any plan to add Permissions (can_list, can_edit etc) and Modules (moduleA, moduleB) to the roadmap. I would imagine being able to register the application's modules, and while registering the user being able assign permission to modules (if I'm not mistaken Flask App Builder does something similar). Probably could be used with a decorator on the routes that would check if the user has the necessary permission to access that particular module. Dependencies would be another option, albeit I don't really like those huge parameter lines being passed to the route function, but that's just me. Anyways, just an idea.
Thanks a lot!

Hi @lowercase00!
Currently, this is not on the roadmap. I feel this is not in the scope of this project (see https://github.com/frankie567/fastapi-users/issues/217#issuecomment-644100504).
Cheers!


Need equivalent vue build (poi)

Hello, i need equivalent of these command
vue build .\src\components\Component.vue --prod --lib

If that question is about poi, why do you ask here?

because that command removed from vue, is there any way to do this?

https://poi.js.org/#/recipes/cjs-and-umd-format


Certifying SBC's

Could someone please point us in the direction of who we need to make contact with in order to certify our SBC technology for use with Microsoft Teams, have looked online but so far have been unsuccessful.

@GymTheGeek thank you for submitting feedback.
I think the best way forward is if you open a service ticket in your tenant so this can get resolved ASAP. Based on the outcome let me know if it is something that can be called out in the docs.
Please follow this link to contact support for business products:https://docs.microsoft.com/office365/admin/contact-support-for-business-products
Please keep us posted here on the resolution so we can feed whatever information you discover into the content.


[FR] Allow multiple files drag and drop into attachment & allow rename files

Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find a similar feature request

Problem statement
Currently  we need to drag and drop files 1 by 1, it is also impossible to rename them.

Suggested solution
Allow multiple drag and drop.
It would also be great to allow renaming files.
Describe alternatives you've considered

Examples of other systems

Do you want to develop this?

[ ] I want to develop this.

@damien-ls funnily enough I was looking at this recently too.
Given that we are working towards our major UI refactor I would hold off until then. The new UI will have a much better drag-and-drop upload interface

The "multiple drag and drop" feature now works - but implementing a "rename uploaded file" function would require a backend change


Support for only show message in diff

Hi  there, currently the inline comment will try to set message directly to the line in diff, when it's out of the diff's range, goes back to a comment.
But sometimes I just want to let them show in diff, when out of range, it means we don't care them in this pull request.(something like swiftlint).
I'm trying to work on this, add an option like strict_inline to the message plugin.
Any ideas about this?

I think the API specifically does not allow this, which is why we have to do such a weird dance around putting it in the main message

yes, so can we have an option to ignore those out-of-diff messages when they are out-of-my-concern too, like i don't want the whole file's lint violations result but only in my changes?

oh, I see, interesting.
I can see a case when you still have a lot of ignored warnings that could be outside of that range. Sure, I think that makes sense as an option on messaging, perhaps messaging.dismiss_out_of_range_violations = true ? yeah, it's verbose, but it makes it very obvious

Fixed in https://github.com/danger/danger/pull/747


Unzip fails with Gateway timeout

I get an error (see below) when extracting a zip file within HydroShare. This archive can be extracted locally without any issues, but when using HydroShare's "unzip" functionality I get HTTP 504. The strange thing is that the data appears to be extracted successfully.

A 504 is a timeout.  This is because your browser doesn't receive a response before timing out due to django being busy unzipping the file.  This will be fixed with the async tasks work that is nearly complete.

@hyi Is this ticket still active or should it be closed?

@Castronova can you please test and confirm that this works now?

@martinseul and @Castronova I tested manually and also added to a PR for our QA suite and can confirm that this works now


Size of futlength

Great job! I want to know why the val loss output is a 10 dim tensor, what does that mean? When i change the futlength to 50, some errors occurred.

Hi, the future length is 5 seconds. The original data is sampled at 10Hz, while in training, the predictor downsamples the future trajectory into 2Hz. So that 10 dim tensor means the locations at [0.5, 1.0, 1.5,..., 5.0] seconds.
Hope the above answers this question.


Add an upload feature

This application is great.
An upload page would be great.
Regards.

Sorry but this does not align with the current setup of the app, see readme:

Its simple. Shows what you have that's it. No gallery changes (photo delete, rotate, enhance, tag, organize, etc), your gallery folder is read-only.


[WIP] file array rule, validate nested items

WHY
BEFORE - What was wrong? What was happening before this PR?
Our validators didn't take into account the repeatable scenario.
There will probably be more changes related to this, so NO MERGE, NO TEST, NO LOOK AT THIS!

There will probably be more changes related to this, so NO MERGE, NO TEST, NO LOOK AT THIS!

❤ love this note! ❤ 😅


Typeahead selection by mouse click doesn't work

Using Bootstrap 0.11.0 and angular js v1.4.8, the Typeahead selection by mouse click doesn't work. It works when keyboard enter is used.
There was a similar problem discussed for v1.3.0, which was fixed but problem seems to be there in v1.4 -> https://github.com/angular-ui/bootstrap/issues/2293

@ashigl, If you think you've found a bug, please follow these instructions.

Keep in mind, UI Bootstrap is now on 1.0.3 - 0.11.0 is almost 2 years old, so we can't really help with any issues on a version that old.

Related Plunkr link below:
http://plnkr.co/edit/TokpsC12d8rhzBPQojvc?p=preview

@any-hn, to reiterate what @wesleycho said, given the age of the library being used, I'm pretty certain it doesn't make any sense for us to spend time on this.  We are not going to release a patch to the 0.11 version.  As an aside, I check that plunker with 0.12.0 and the issue goes away so if you're absolutely stuck on an old version, perhaps upgrading to 0.12.0 will work for you.

cool. thanks @icfantv


Resource controller doesn't return a route, and doesn't have a middleware method.

Describe the bug
Using a Route::resource() method doesn't return a Route so there is no middleware method available.

In addition to not working in the same way as the other routes, resource controllers don't allow for parameter naming, excluding/including methods or any other features that are/could be included on the normal routes.
A decision needs to be made as to whether the resource controller serves a big enough purpose to be worthwhile - the end result can, after all, be achieved with standard routes. Using resource controllers decrease boilerplate code, but are (currently) way less flexible.


Performance gap in reproducing results

The reproduced results show that DS is much lower than the numerical value in the paper. Is this result jitter normal?

Hi, thanks for your interests.
This codebase has encountered a major refactor and it is mixed with some implementations in our next work  (accepted by ICCV 2023 and this time we will release both the original 189K and 2M checkpoints, please stay tuned!). without thorough inspection and tuning due to limited time and computational resource. The model is also retrained.
Thus, there might be some bugs. I check out one of our old team_agent files and write a similar one for this codebase:
old_team_agent.py
Could you please try this one? (Just replace the original_team_agent) In the meantime, I will cross-check with the private codebase when having time.
Thanks for your time.

 The reproduced results show that DS is much lower than the numerical value in the paper. Is this result jitter normal?

Hello, which scenarios file do you use, just the "all_towns_traffic_scenarios_no256.json" provided in the codebase?

Hi, I'd like to ask can you provide the inference time when you evaluate this code. How long it takes ?


[snapshot] Failed to generate snapshot

$ dart --version
Dart VM version: 2.1.0-dev.7.0 (Unknown timestamp) on "linux_x64"

$ cat main.dart
import 'dart:io';

main(List<String> args) {
  print(args);
  HttpServer.bind(InternetAddress.anyIPv4, 2018).then((server) {}).catchError(print);
}

$ dart --snapshot=main.dart.snapshot --snapshot-kind=app-jit main.dart
[]
^C

Execute first and then produce a snapshot?
Why do I need to run the program? The previous version did not need to run and the snapshot was successful.

The program will run when using --snapshot-kind=app-jit, this has been the case since that snapshot kind was introduced. The VM executes the program in order to get some JIT optimizations. You can use --snapshot-kind=script to get a snapshot built without running the program.

@natebosch
$ dart --snapshot=main.dart.snapshot --snapshot-kind=script main.dart
Unrecognized value for snapshot_kind: 'script'
Valid values are: none, kernel, app-jit

Are you sure you have this parameter?
I have a snapshot generated by the previous version, 3.5M file, which is generated by app-jit method.

Ah I had the name wrong. It should be --snapshot-kind=kernel.
If you have a file generated with app-jit then it was generated while running the program, which is a great way to do it when possible.


Remove userId null checks in src/main/java/app/fitbuddy/controller/crud/ExerciseController.java #177

Removed the userId null checks in ExerciseController and indented the code as per standards.
Also eliminated the unused imports.
Close #177

@all-contributors add @iravimandalia for code


backing error when nesting forward-mode

Here is a trivial example of nested forwards mode
using Diffractor
f(x) = 3x^2
g(x) = Diffractor.∂☆{1}()(Diffractor.ZeroBundle{1}(f), Diffractor.TaylorBundle{1}(x, (1.0,)))
Diffractor.∂☆{1}()(Diffractor.ZeroBundle{1}(g), Diffractor.TaylorBundle{1}(10, (1.0,)))

Here is the output from running that:

julia> Diffractor.∂☆{1}()(Diffractor.ZeroBundle{1}(g), Diffractor.TaylorBundle{1}(10, (1.0,)))
ERROR: ArgumentError: Tangent for the primal Diffractor.UniformTangent{ChainRulesCore.ZeroTangent} should be backed by a NamedTuple type, not by Tuple{ChainRulesCore.ZeroTangent}.

I suspect it is possible to rewrite some (maybe all?) case to do ∂☆{2}
but I would need to do some thinking.
we definately shouldn't be just erroring though.

So I believe the cause of this is that CompositeBundle works (/is tested) to represent the tangent bundle for Tuples only.
But its being asked to represent the tangent of structs,
but when you pull the partial out for a struct, it pulls out a CRC.Tangent(P, <:Tuple) and that is what gives the error

Fixed by #137


Unexpected "docs-mdx" section in the doc sidebar

/area documentation
/kind bug
What mistake did you find / what is missing in the documentation?
Example: https://odo.dev/docs/user-guides/quickstart/docs-mdx/connecting_to_the_cluster_description

What is the relevance of it?
"docs-mdx" should not be visible.

cc @valaparthvi

I thought I had handled this thing :facepalm:
I'll look into it. Thanks for opening this.

I thought I had handled this thing facepalm I'll look into it. Thanks for opening this.

I fixed this as a part of #6442.

Ah, fine then. Okay - let's wait for this PR to be merged then. Thanks.


Support get box

Implemented for Gaussian grids. Can add more if there are the users to justify it.


Invisible Characters Remains While Editing in Non-English ( Chinese )

VSCode Version: Code 1.19.0 (816be6780ca8bd0ab80314e11478c48c70d09383, 2017-12-14T09:51:12.123Z)
OS Version: Darwin x64 17.3.0
Extensions:

Extension
Author (truncated)
Version

xml
Dot
1.9.2

beautify
Hoo
1.1.1

vscode-docker
Pet
0.0.22

GBKtoUTF8
buk
0.0.2

plsql
cas
0.0.3

vscode-eslint
dba
1.4.3

githistory
don
0.2.3

tslint
eg2
1.0.24

code-runner
for
0.8.5

ansible
haa
0.2.5

restructuredtext
lex
44.0.0

Go
luk
0.6.69

VS-code-vagrantfile
mar
0.0.7

python
ms-
0.9.0

azure-account
ms-
0.2.2

debugger-for-chrome
msj
4.0.0

java
red
0.16.0

vscode-icons
rob
7.19.0

vscode-java-debug
vsc
0.4.0

vscode-java-pack
vsc
0.2.0

Steps to Reproduce:

Edit .md files using VS Code
Switching input methods between default input method and Sougou Input Method during editing Chinese characters.

Reproduces without extensions: Yes/No

Please provide screenshots and sample text to add to the md file so that we can try and repro the issue

Hello, Here's the quick links to the problemnic md files edited using VS Code.
https://github.com/eliu/openshift-vagrant/blob/7bc6a4876956fc9c49a6095068eeb2e34fcd9194/README_CN.md
Screenshot as below:
in the screenshot above, emphasised in green, with quesion mark, that is the invisible characters. they appears sometimes when im type among Sogou Input Method and default one
Cheers,
Eric
On 12/17/2017 04:37，Ramya<EMAIL_ADDRESS>wrote：
Please provide screenshots and sample text to add to the md file so that we can try and repro the issue
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub, or mute the thread.

@ramya-rao-a are you still following up this issue right now? Since these characters are not visible, therefore all i can provided are screenshots links and original text file. let me know if you want anything else to provide from me. Thanks.

Thanks for providing the text file and screenshots. @mjbvz will be following up soon

Here's what I see:

What is the encoding of the file in vscode?

The encoding of the respected file shows UTF-8 in vscode.

Any update on this?

I have a every similar issue.
The markdown file was showed correctly in visual studio code. But there're several ^H , when I open that file by Vim.
To reproduce:
It's every easy to reproduce if you're editing mix-language file (e.g. Chinese-English ) especially after some deletions and insertions.
system: macOS 10.13.5
vscode version: Version 1.23.1 (1.23.1)

Duplicate of https://github.com/Microsoft/vscode/issues/37114


chore: Add Nix Flake

Description
Adding Nix support for those of us that Live by the Flake Code ⚔❄
Link to issue
None!
Type of change

[x] Chore

Test plan (required)

[ ] Set up Nix GitHub Actions

Screenshots/Screencaps
N/A

Being super lazy, and just copy/pasting from the Homestar repo


Override printSchema

Description
We'd like to be able to re-order how the schema is printed, mostly to put the Query and Mutation type on top of the file vs. the alphabetical order that is made at the moment.
As we're importing types from prisma with nexus-prisma, it creates a lot of noise especially with input types being huge. For that reason, it'd be useful for us to be able to decide how the schema is printed

I'm actually thinking of adding more config options for makeSchema to tap into important pieces of the schema construction or execution processes, e.g. the missingType that nexus-prisma uses to catch missing types. Or for the authorization pieces, a way to handle/format the error. Or for non-null errors ways to gracefully recover from it without crashing the entire Query.
Basically a more well defined middleware system.
Perhaps printSchema could be one of these configs?
makeSchema({
   types: [ /* ... */ ],
   outputs: { /* ... */ },
   printSchema(schema) {
      return myCustomPrintSchema(schema)
   }
})

Hey Tim,
Sounds great! That would indeed work perfectly for me. That would let me override printSchema from makePrismaSchema if none is provided 🙌

@Weakky is this still something you're looking for?


Stable 0.13.0-final doesn't build on Ubuntu 18.04

I have boost 1.65 installed outside this directory, along with 1.55.
`CMake Warning at /usr/share/cmake-3.10/Modules/FindBoost.cmake:801 (message):
New Boost version may have incorrect or missing dependencies and imported
targets
Call Stack (most recent call first):
/usr/share/cmake-3.10/Modules/FindBoost.cmake:907 (_Boost_COMPONENT_DEPENDENCIES)
/usr/share/cmake-3.10/Modules/FindBoost.cmake:1558 (_Boost_MISSING_DEPENDENCIES)
CMakeLists.txt:53 (find_package)
CMake Warning at /usr/share/cmake-3.10/Modules/FindBoost.cmake:801 (message):
New Boost version may have incorrect or missing dependencies and imported
targets
Call Stack (most recent call first):
/usr/share/cmake-3.10/Modules/FindBoost.cmake:907 (_Boost_COMPONENT_DEPENDENCIES)
/usr/share/cmake-3.10/Modules/FindBoost.cmake:1558 (_Boost_MISSING_DEPENDENCIES)
CMakeLists.txt:53 (find_package)
CMake Warning at /usr/share/cmake-3.10/Modules/FindBoost.cmake:801 (message):
New Boost version may have incorrect or missing dependencies and imported
targets
Call Stack (most recent call first):
/usr/share/cmake-3.10/Modules/FindBoost.cmake:907 (_Boost_COMPONENT_DEPENDENCIES)
/usr/share/cmake-3.10/Modules/FindBoost.cmake:1558 (_Boost_MISSING_DEPENDENCIES)
CMakeLists.txt:53 (find_package)
CMake Warning at /usr/share/cmake-3.10/Modules/FindBoost.cmake:801 (message):
New Boost version may have incorrect or missing dependencies and imported
targets
Call Stack (most recent call first):
/usr/share/cmake-3.10/Modules/FindBoost.cmake:907 (_Boost_COMPONENT_DEPENDENCIES)
/usr/share/cmake-3.10/Modules/FindBoost.cmake:1558 (_Boost_MISSING_DEPENDENCIES)
CMakeLists.txt:53 (find_package)
CMake Warning at /usr/share/cmake-3.10/Modules/FindBoost.cmake:801 (message):
New Boost version may have incorrect or missing dependencies and imported
targets
Call Stack (most recent call first):
/usr/share/cmake-3.10/Modules/FindBoost.cmake:907 (_Boost_COMPONENT_DEPENDENCIES)
/usr/share/cmake-3.10/Modules/FindBoost.cmake:1558 (_Boost_MISSING_DEPENDENCIES)
CMakeLists.txt:53 (find_package)
-- Boost version: 1.68.0
-- Found the following Boost libraries:
--   system
--   thread
--   chrono
--   date_time
--   atomic
CMake Error at CMakeLists.txt:128 (add_subdirectory):
The source directory
/home/evan/projects/cpp-netlib/cpp-netlib-0.13.0-final/deps/googletest

does not contain a CMakeLists.txt file.
-- Configuring incomplete, errors occurred!
See also "/home/evan/projects/cpp-netlib/cpp-netlib-0.13.0-final/build/CMakeFiles/CMakeOutput.log".
See also "/home/evan/projects/cpp-netlib/cpp-netlib-0.13.0-final/build/CMakeFiles/CMakeError.log".
evan@evan-Gazelle-Pro:~/projects/cpp-netlib/cpp-netlib-0.13.0-final
`

Not hugely helpful for resolving the root issue, but I've managed to get around this build issue by switching the CMakeLists.txt option CPP-NETLIB_BUILD_TESTS to OFF.


It turns out that a lot of sites do not work with

 Issue by alekssamos
Fri Feb 25 08:35:22 2022
Originally opened as https://github.com/codelucas/newspaper/issues/937

I am completely disenchanted.
Why these dictionaries, key stop words? From many sites, instead of the text of the article, there is an empty line. I definitely didn't expect this.

This should be marked wont fix or resolved. @AndyTheFactory


Filtering '\r' on windows, was causing double endline on code blocks.

Related to haskell/haddock#356

Looks OK, I wonder if we could get away with just one filter in appropriate place. This does confirm that \r is a problem at the very least. I will merge it closer to the evening of my time when I can run it.
Thanks for the PR!

Pulled in with a slightly better (IMHO) commit message, thanks.


⚠ Blutopia has degraded performance

In 6901e45, Blutopia ($BLU) experienced degraded performance:

HTTP code: 200
Response time: 3976 ms

Resolved: Blutopia performance has improved in 86e69bb after 6 minutes.


Support str

Exceptions use __str__ instead of __repr__ for formatting; this is unfortunate for defining exceptions with attrs, because the default __str__ will print out nothing which is very useless.
What do you think about an extra option to define __str__ the same as __repr__?

Yeah I’ve run into that too & resorted to setting __str__ to __repr__ by hand before.  I guess we can add a str=False.

I presume this is what you wanted?

Yup! :+1:


Delete users with existing Sierra accounts who logged on for the first time before 8 September 2021

There is a fix for users logging in with existing accounts in Sierra such that they are marked as verified (#174 ), which was closed on 7 September.
This does not help people who logged in before that fix was made (7 September): they still see the message that their email isn't validated on their account page.
For the small number of affected users, we can fix it by deleting their Auth0 accounts (this should have no adverse affects as their Sierra accounts will just be re-imported). (comment)

This is done.
I've reviewed the list of users in both prod and stage; there are no external users who first logged in before 8th September. Either nobody logged in before then, or somebody has already sorted this.
There are some staff users, but deleting those accounts is a bunch of point-and-clicking that's probably unnecessary.


[SPARK-5821] [SQL] ParquetRelation2 CTAS should check if delete is successful

Do the same check as #4610 for ParquetRelation2.

Test build #28922 has started for   PR 5107 at commit 7092c8d.

This patch merges cleanly.

Test build #28922 has finished for   PR 5107 at commit 7092c8d.

This patch passes all tests.

This patch merges cleanly.

This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/28922/
Test PASSed.

LGTM. I don't think we need a test case for this, so I'm gonna merging it to master and branch-1.3. Thanks!


[Event Request] codeunit 64 "Sales-Get Shipment" - OnBeforeTransferLineToSalesDocLine

We would like to add a new event OnBeforeTransferLineToSalesDocLine in the following procedure:
procedure CreateInvLines(var SalesShptLine2: Record "Sales Shipment Line")

.
.
.
                end;
                //A108F0019.Event.ns
                TransferThisLine := true;
                OnBeforeTransferLineToSalesDocLine(SalesShptHeader, SalesShptLine2, SalesHeader, TransferLine);
                if TransferThisLine then
                    //A108F0019.Event.ne
                    if TransferLine then begin
                        SalesShptLine := SalesShptLine2;
                        SalesShptLine.InsertInvLineFromShptLine(SalesLine);
                        CalcUpdatePrepmtAmtToDeductRounding(SalesShptLine, SalesLine, PrepmtAmtToDeductRounding);
                    end;

Event Signature:
[IntegrationEvent(false, false)]
local procedure OnBeforeTransferLineToSalesDocLine(SalesShipmentHeader: Record "Sales Shipment Header"; SalesShipmentLine: Record "Sales Shipment Line"; var SalesHeader: Record "Sales Header"; var TransferLine: Boolean)
begin
    //A108F0019.Event
end;

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


Cancel async functions inside useEffect hooks

Improvement related to performance.
Links:

https://github.com/privacy-scaling-explorations/bandada/pull/214#discussion_r1232426046

Hi @cedoor can I work on this ?

Delegate to @vplasencia

Hey @ScottyDavies! Do you want me to assign this issue to you?

yes, thanks


Performance degrades over time

I've noticed on a standalone cluster (using tip of tree) that the atlas process performance degrades over time.  The main indicator is the "rebuilding index", which incrementally increases in time.
Here's the setup:
Server with 16GB ram, startup params specify 12GB
Roughly 1 million metrics reported by index (retaining 3 hours at 60 step)
index rebuild time set to 30 seconds
The index rebuild time when "full" is around 15 seconds. Leaving this server up ingesting data for 24 hours, the index rebuild time is 7 minutes, and the queries are very slow or fail/timeout.
What is the recommended "size" for a standalone atlas server?
Is there a way to view the Spectator global registry metrics?
What memory size would be good for a clustered/sharded solution (I'm seeing similar degradation behavior with a 5-node 8GB cluster sharding implementation)
Thanks!
Brian

My guess from what your describing is that GC is becoming a problem. What are the JVM parameters?
We typically run using G1 with a recent java 8 build (there are some known G1 issues on older versions, e.g., #81). CMS is prone to fragmentation and we have generally avoided it. In most cases we were better off using the old parallel collector than CMS prior to G1.
We typically run on r3.2xlarge instances assuming at most 2M time series per instance with 6h window and 60s step (we alert if the index size hits 2M and we can get up to 2.5 to 3 before major problems with this configuration depending on data characteristics).

A number of those are just for getting diagnostic information. Here is some overview on why we use some of the settings:

-XX:+PreserveFramePointer: better flame graphs
-XX:MetaspaceSize: avoid full GC due to metaspace, see #81
-XX:+UseG1GC: generally works best for us when using recent java 8 builds, CMS is prone to fragmentation and we generally avoid it
-XX:StringTableSize=1000003: we use string interning heavily to reduce memory for tags, if the table fills up the intern calls can get really slow
-Xms52g, -Xmx52g: pin the heap size
others: mostly debugging and things like GC logs

There are a few other configurations, but that is the main one we use. We haven't done a whole lot of testing recently on varying those or using other heap sizes. Note, data characteristics matter quite a bit, so these settings have been tuned for the data patterns we have at Netflix. In artificial tests we can get much more data using the same config with more optimal data patterns or less with some patterns.
I started setting up a basic load test to make it easier to verify a number of configurations for both finding regressions and creating reasonable docs on recommended sizing and configuration, but don't have a lot of time to spend on it right now.

Is there a way to view the Spectator global registry metrics?

There are a few ways to view them. In our internal setup they get reported to an Atlas cluster. There is also a simple admin component we add on a number of our services when running them to get some additional debugging information. You can also configure a registry to send them to JMX if that is preferable.

Out setup is really simple (looks like the GC is the likely culprit, I did see a GC error in the logs at one point).  I'll spin it up with your parameters adjusted for a 16GB box and 3hours and see how it holds up.
I'll check out the spectator admin component also, I'm not familiar with spectator at all.
Sounds like the formula is ballparked around 4GB ram per million metrics per hour of retention, If so, I should be able to handle about 3 hours safely only a 16GB system (with jvm pinned to 12gb).
Many thanks for the reply!
Brian
Here's our setup:
Running on an m4.xlarge (4vCPU, 16GB RAM)
JDK: jdk-8u91-linux-x64
Startup with:
java -Xmx12g -Xms12g
-Dcom.sun.management.jmxremote.port=9999
-Dcom.sun.management.jmxremote.authenticate=false
-Dcom.sun.management.jmxremote.ssl=false
-Dlog4j.configurationFile=/usr/local/atlas/log4j2.xml
-XshowSettings:all
-jar /usr/local/atlas/atlas.jar /usr/local/atlas/atlas.conf
Here are some errors from old logs:
atlas: #
atlas: # A fatal error has been detected by the Java Runtime Environment:
atlas: #
atlas: #  SIGBUS (0x7) at pc=0x00007fe1f66578ce, pid=15045, tid=140607043139328
atlas: #
atlas: # JRE version: Java(TM) SE Runtime Environment (8.0_91-b14) (build 1.8.0_91-b14)
atlas: # Java VM: Java HotSpot(TM) 64-Bit Server VM (25.91-b14 mixed mode linux-amd64 compressed oops)
atlas: # Problematic frame:
atlas: # C  [libc.so.6+0x14e8ce]  __memmove_ssse3_back+0xbfe
and a GC OOM:
atlas: 2016-08-09T07:32:02.708 ERROR [atlas-akka.actor.default-dispatcher-3279] akka.event.slf4j.Slf4jLogger$$anonfun$receive$1$$anonfun$applyOrElse$1: Uncaught error from thread [atlas-akka.actor.default-dispatcher-3265] shutting down JVM since 'akka.jvm-exit-on-fatal-error' is enabled
atlas: java.lang.OutOfMemoryError: GC overhead limit exceeded

Based on the volume you mention, I would also recommend being explicit about the string table size (-XX:StringTableSize=1000003).

Reporting back - the standalone setup has been running flawlessly for 24hrs now, no degradation at all.  It's holding 3hrs of metrics, with an index hovering around 1.2million.
Many thanks!


How to use ARG ELK_VERSION when executing docker-compose up?

thanks

sorry,I find the answer in closed issue list,it should config in .env file

You could also have checked the README: https://github.com/deviantony/docker-elk#version-selection


Updating Tekkenbot with cheat engine

I've seen everyone say to fix the issue with the overlay only showing question mark , all you need to do it change the offset or memory address. I tried everything and it still seems I'm doing something wrong. Can some one please make a step by step walk-through on what to do to get the overlay working for those of us that are not savvy with using cheat engine. Or would it just be better to wait until another update comes out?

I used cheat engine to find the data offset, however adding that to the config file does not solve the issue for me. TekkenBot was working for me in previous builds but no longer works,  If possible, I could recompile the program if instructions were given.

Has anyone been able to verify if the issue is Tekken bowling? Part of me think that if I buy it that will take the issue away and the bot will go back to working like normal.

Previous threads demonstrated that bowling was not the issue.

Well that's good to know. So for us who cant use cheat engine to save our life, it's pretty much a waiting game of the creator to update the program? I just hope the creator don't feel too much pressure from us.

I've just been checking this page every day for the last week in hopes of it getting updated lol.  Glad someone started a conversation about it.  It probably is just taking a lot of time.  He has made Tekken so much easier to learn with this program I hope it doesn't stop here.  Thanks to the creator...

Guys, I'm complete noob in coding. How to update offsets? From readme:
"Find the offset to the pointer to the pointer of any of the first 8 and replace the 'player_data_pointer_offset' value in MemoryAddressEnum.py"
Pobega wrote what offsets have to be updated here:
https://github.com/roguelike2d/TekkenBot/pull/53/commits/cf6b55caab209cf59f835b838d66f7041ffcb4a4
I've got no clue how to do it. Anyone?

I don't know either. I've just been modifying the code and running it with python GUI_TekkenBotPrime.py
Don't forget to pip install requests otherwise it won't run properly.

Thank you Pobega!

@Pobega can you explain how you got the offset for future reference?


CLU option not displaying in Bot Composer

Hi team,
Earlier we are using LUIS and planning to migrate to CLU.
We have trying to publish the profile with CLU but we are not getting the option.
PFA

@ykr22,
This is not the intended deployment process for CLU.
You would instead install the CLU Recognizer, which would itself need to be deployed before you can configure it in your bot. The README in the recognizer explains how to install the package and configure it for your Composer bot. You will not be able to re-deploy CLU changes through Composer.


Using IdP for web apps that do not reside on Azure

Hi - we have an internal (Windows) web server with a web-based application. We want to be able to provide an Identity Provider solution for this internal application to both our internal and external users. Do you have a solution for that and/or can you point me to more information for this on the Azure website? Thanks.

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

ID: 5cceddcd-1431-ab61-df75-7ccf74a84fba
Version Independent ID: d5833a30-c43e-8048-63b3-d94d26410a3b
Content: Microsoft identity platform Windows UWP quickstart
Content Source: articles/active-directory/develop/quickstart-v2-uwp.md
Service: active-directory
Sub-service: develop
GitHub Login: @jmprieur
Microsoft Alias: jmprieur

Is this IDP OAuth2.0/OIDC compliant? @jpitcherCMH

@jpitcherCMH : please have a look at https://docs.microsoft.com/en-us/azure/active-directory/develop/authentication-flows-app-scenarios#web-application-signing-in-a-user

#please-close (as the question is answered)


new notebook explaining the orbit distribution plots

I've added a new tutorial notebook to the docs: 7_orbital_distributiuons.ipynb which explains how to get to the orbit distribution data, which we'll need for orbit colouring.
@maindlt - please test this new notebook and merge if it runs successfully. I'm having some trouble rendering the docs with using sphinx... something to do with my nbsphinx installation. If you can also manage to render this, and upload this to the website (w/ Alina's help?) that would be great.
Also @maindlt if this notebook gives you enough of a starting point to get going with the voronoi binning (with vorbin) then please feel free to get going with that. Otherwise, let's wait time till @boeckera and I are back and available to pick this up again.

Hi Prash, the new tutorial is great! After fixing a few minor things I am happy to merge :-)
About making the docs:
I also ran into problems with nbsphinx, which required some fiddling around. I'll clean up things, do a fresh sphinx install and update the instructions in PR #383 (which is about the new installation procedure, including making the docs).
Upload to the website:

On the website we only link to the DYNAMITE releases. As the new tutorial is not part of an "official" release yet and the "master branch docs" contain references to code not yet included in a release, I would not put them on the website just now, but we can of course create a new release (it's been half a year since the last one).
Connected to that: what about offering two alternatives on https://dynamics.univie.ac.at/#DYNAMITE:

Links to the DYNAMITE releases + the release version docs (just like now)
Links to the master branch version ("Release candidate" or "Tested development version" or so) + its docs + changelog

This would allow the users to access the latest bugfix pre-release (bugfixes + new features) from the dynamics website, but we would need to compile and publish the docs every time we merge a PR. Perhaps we can discuss this at the next hack session.

Vorbin: I'll happily give it a try ;-)


My portfolio moblie Using JS DOM and events

In this project, I learned the following:-

Using basic JavaScript syntax.
Using JavaScript to manipulate DOM elements.
Using JavaScript events.

@Reem-lab please let me create another pull request.


[SYSTEMML-1907] Rename python package to systemml-*-python.tar.gz

Switched python artifact to tar.gz for PyPi upload compatibility with twine.
Also made corresponding updates to release validation utilities and documentation.

Tested the following on linux:
mvn clean package -DskipTests=true -P distribution
./release-verify.sh --compile
./release-verify.sh --verifyLic --distDir="../../target"
pip install ./target/systemml-1.0.0-SNAPSHOT-python.tar.gz
Ran matrix example in PySpark. Verified distribution built on Windows from command line.

If other artifacts need to be similarly switched from tgz to tar.gz, then a separate PR/JIRA can be opened.

Note the website's install page "Dev Python (Latest code)" tab which references python artifact will be updated after PR merged.

Refer to this link for build results (access rights to CI server needed):
https://sparktc.ibmcloud.com/jenkins/job/SystemML-PullRequestBuilder/2103/

Thanks @gweidner
LGTM

Thank you Arvind for reviewing!  I will merge.


Fix for wrong connection count on remote nodes

Description
Issue: Even after setting CONNECTION_POOL_REMOTE_SIZE to zero, connections were getting created to remote nodes. Number of connections on REMOTE nodes were equal to Number of connections on LOCAL nodes.
On some investigation we found out that the private localDc variable of YugabyteDefaultLoadBalancingPolicy class [child class] was hiding the localDc variable of the BasicLoadBalancingPolicy class [parent class], due to which the driver was not properly identifying Local AND Remote nodes. It was considering all Up nodes as Local.
Fix
Removed the private localDc from YugabyteDefaultLoadBalancingPolicy and made the localDc variable in BasicLoadBalancingPolicy protected.
Also did this change for another private member variable of YugabyteDefaultLoadBalancingPolicy class: distanceReporter
Note
This PR also includes some formatting changes in PartitionAwarePolicy.java
Testing

Tested locally using a sample app to verify remote and local node connections.
nosqlbench  test runs with different consistency level and local datacenter specifications has been done, and load routing behaviour is as expected.
Tests in ycql-4.x-tests repo: 7 Tests, 5 Passed, 2 Errors in TestDMLRoutingLocality(these occur with 4.15.0-yb-1 also).
Tests in Yugabyte-db repo: 13 Tests, all Passed.

Also, please run the formatting command and commit the changes
mvn com.coveo:fmt-maven-plugin:format

Should get a clean run of nosqlbench with correct routing behaviour before merging.

@kneeraj nosqlbench run with different consistency level and local datacenter specifications has been done, and load routing behaviour is as expected. cc: @ashetkar

@HarshDaryani896 Let's get this PR cleaned of all the debugging logs and merge it, if we have successfully completed all the testing.
We can work on debugging logs on another branch.
Also, please mention the test case which was failing earlier but is fixed by your change, for reference.


Feature/i18n

[x] Fix the last missing "old" translation calls
[x] Test the docker build/deployment
[x] Maybe implement a tool to manage the locales. For example: https://localazy.com/p/unsaged

Fix #89 #47 #73

Just added a issue with the optimizations that are still needed but the current i18n fix should be merged to fix some ongoing issues I think we can push it and I will have a look at #147 later on 👍🏼


How to support any of the audio format flac/mp3/mpeg/ogg/wav/basic etc?

I need feed the recorded file into https://cloud.ibm.com/apidocs/speech-to-text/speech-to-text#recognize-audio , which does not take either aac or mp4. How do connect the two, please?

I'd love to see FLAC supported - or at least SOME cross-platform royalty-free encoding (LPCM/WAV, FLAC, OPUS, etc). Without this, it is very difficult to use this library for anything beyond basic sound recording and playback.


Update UX8_Correlation_Regression.ipynb

In section 1 first code block: what is the reason for "index=[1,2,3,…"? Is it even needed? It will index it from 0 by default and I thought that was fine.

It might be confusing of how to interpret covariance matrix since at first glance it is just very high numbers. Is it just that you check if they are positive, or can you also tell something from how high the numbers are? I think it is a bit confusing since it depends on what values the data are.

"....(drumroll)", that one’s really funny

Maybe highlight that linear regression is advanced topic they will learn later? the word ‘model’ won’t likely make sense for them, since it is more machine learning vocabulary. Just to keep in mind when you are having the lecture, to maybe tell them that the error e is sort of a black box, for now. My main concern is that they will want to really know everything about it, which is a pattern I've seen in some students, so it might be worth reiterating what they should know about linear regression, and as well what they shouldn't really know for this class.

Rephrase in 1st paragraph of section 5: "A recurrent question is how much of the variability in data that the model accounts for, the variance explained." I got really confused reading that line


Feature: add option to export multiple pgbouncer instances

Why this PR
We TCP load balance our connections over multiple (4+) pgbouncer instances/processes to increase stability and performance. We would like to switch from our own exporter to the prometheus-community exporter but would like to run only one exporter instance instead of an exporter per pgbouncer instance.
What does the PR do
This PR will add the option to export multiple pgbouncer instances with one exporter instance using a yaml config file.
All the options are documented within the example config.yaml within this repo.
This config file also add the option to start the exporter even if the connection fails on startup.
Extra labels
To differentiate the metrics for each pgbouncer instances, you MUST configure a unique label/value set for each instance.
As shown in the example, this can be as simple as setting a unique pgbouncer_instance value for each instance.
The extra labels per instance are merged with the common extra_labels where the instance value has precedence over the common value
Requirements:

Every instance must define the same labels
Every instance must define at least have one label with an unique value.

If there requirements are not met the exporter will fail startup.
Example
When using the config file:
must_connect_on_startup: false
extra_labels:
  environment: prod
pgbouncers:
  - dsn: postgresql://user:pass@localhost:6432/pgbouncer?sslmode=disable
    extra_labels:
      pgbouncer_instance: set1-0
  - dsn: postgresql://user:pass@localhost:6433/pgbouncer?sslmode=disable
    extra_labels:
      pgbouncer_instance: set1-1
  - dsn: postgresql://user:pass@localhost:6434/pgbouncer?sslmode=disable
    extra_labels:
      pgbouncer_instance: set1-2

you now get 3 pgbouncer_up metrics:
% curl -s http://localhost:9127/metrics | grep "_up"
# HELP pgbouncer_up The pgbouncer scrape succeeded
# TYPE pgbouncer_up gauge
pgbouncer_up{environment="prod",pgbouncer_instance="set1-0"} 1
pgbouncer_up{environment="prod",pgbouncer_instance="set1-1"} 1
pgbouncer_up{environment="prod",pgbouncer_instance="set1-2"} 0

Force-push was to add sign-off to the commit

@SuperQ Can i do something to add some progress the review process?

@SuperQ : This is definitely a useful and most wanted feature.
Or is there any other PR which is going to provide same feature. i.e. "Support to export metrics from multiple pgbouncer instances"
Please, can we expedite support for this feature?


run build generate index.html and build.js, but changes doesn't reflect

I downloaded the SPA.Demo and followed the documentation steps, Build Action is set to Content and Copy Always. After making changes to main.vue and running npm run build (I made sure it generates the files), running the project from VS does not show any changes (run dev however shows the changes).
The only suspicion I have is a bug in copying "obj\Debug\Neutronium SPA Demo.exe" to "bin\Debug\Neutronium SPA Demo.exe".

Thank you so much, that fixed the problem!


error for openstack-keystone

I am trying to use "jclouds.provider=openstack-keystone" to access swift, but get the following exception. I see someone mentioned "swift-keystone", but that's not recognized by s3proxy.
Exception in thread "main" java.lang.IllegalArgumentException: api {id=openstack-keystone, name=OpenStack Keystone Essex+ API, views=[], endpointName=Keystone base url ending in /v${jclouds.api-version}/, identityName=${tenantName}:${userName} or ${userName}, if your keystone supports a default tenant, credentialName=Optional.of(${password}), documentation=http://api.openstack.org/, api=interface org.jclouds.openstack.keystone.v2_0.KeystoneApi} not wrappable as org.jclouds.blobstore.BlobStoreContext; context: org.jclouds.rest.ApiContext<org.jclouds.openstack.keystone.v2_0.KeystoneApi>, views: []
at org.jclouds.ContextBuilder.buildView(ContextBuilder.java:611)
at org.jclouds.ContextBuilder.buildView(ContextBuilder.java:595)
at org.jclouds.ContextBuilder.build(ContextBuilder.java:588)
at org.gaul.s3proxy.Main.createBlobStore(Main.java:257)
at org.gaul.s3proxy.Main.main(Main.java:140)

@mikewang2016 did you make any progress with this and get s3proxy to work with Openstack Swift with Keystone?

Sorry I missed this earlier -- S3Proxy works with Swift and Keystone although the latter can require configuration.  First you must set the jclouds.provider=openstack-swift.  Next you can configure Keystone via additional jclouds properties:
https://github.com/jclouds/jclouds/blob/4bbca9edf943852ce1ea5aa579fa0554f770a3ea/apis/openstack-keystone/src/main/java/org/jclouds/openstack/keystone/v2_0/config/KeystoneProperties.java
The wiki has some examples but not exactly what you want:
https://github.com/andrewgaul/s3proxy/wiki/Storage-backend-examples#softlayer

Closing due to lack of activity.  Please reopen if you continue to have issues.


Correction for raw affiliation Moulay Ismaïl University, National High School of Arts and Crafts of Meknes (ENSAM-Meknès),  Team of Modelling, Information Processing and Control Systems (MPICS)

Correction needed for raw affiliation Moulay Ismaïl University, National High School of Arts and Crafts of Meknes (ENSAM-Meknès),  Team of Modelling, Information Processing and Control Systems (MPICS)
raw_affiliation_name: Moulay Ismaïl University, National High School of Arts and Crafts of Meknes (ENSAM-Meknès),  Team of Modelling, Information Processing and Control Systems (MPICS)
new_rors: 02b3gqy34
previous_rors: 018pp1107;02b3gqy34
works_examples: W4361204850
contact: 25eb831c043a78d63208a11f67904c2c:f045fef5cdf214092fb3d7e75c @ ensam.eu

This issue was accepted and ingested by the OpenAlex team on 2024-10-09. The new affiliations should be visible within the next 7 days.


Fix #1841: Sane error for existentials in use(x).as[T]

Review by @sjrd

LGTM

Refer to this link for build results (access rights to CI server needed):
https://scala-webapps.epfl.ch/jenkins/job/scalajs-pr/2059/
https://scala-webapps.epfl.ch/jenkins/job/scalajs-matrix-build/2669/
Test PASSed.


Petrified Fuel Generator intended that way?

MC 1.12.1
Forge 2446
IF 1.1-25
Sorry, but I was wondering if the Petrified Fuel Generator is intended the way it is, that it doesn't extend the burn time of the fuel, rather keep the burn time always the same and instead change the power produced.
Because the way it is, I feel that this is very unbalanced. If it is intended, I wonder if you could at least add a config option to rather extend the burn time. This would be very helpful for my pack. Thanks in advance.

Its not unbalanced, a stick produces a little power and a block of coal a big burst of power.

So, it's intended that way? It's for sure inbalanced as you can generate multiple thousands of RF from a generator, I can't think of another single block gen in modded that can generate that much power in a short amount of time. Betatesters of mine agree, Anyway, if it's intended that way, fair enough, just wanted to check as I wasn't sure.
Any chance there will a config option to change that?

It is intended but I will add a config too

Thanks very much.

So I updated to 1.1.1-27 which is supposed to add it, but I can't find the config.
I looked at the code and it is supposed to be timeModifier, but searching for that in the config file didn't result in a find, also deleting the config and regenning didn't work.

It's in the first line, hard to miss...

petrified_fuel_generator {
        # Time modifier for the burning time. (FuelBurningTime/%value%) [range: 1 ~<PHONE_NUMBER>, default: 20]
        I:timeModifier=20

        # Machine can perform a work action [default: false]
        B:workDisabled=false
    }

Does not show up for me, weird.

Are  you sure you are using the  correct version, a clean instance generates it.

What? It did now, sorry for bothering you, I might have messed up somehow.

Don't worry about it.


Can't find eval.txt

In the inference part of the readme you mention a file called eval.txt, I can't find it.

--metapath: The txt file contains the source and target speech paths, see eval.txt for an example.

Sorry. I just pushed it. It's a template of a very simple structure.

Thank you!


no power to resume after pause

1.please add scheduler
2.Download video from a playlist in this method
Create a folder (channel name) and create another folder (playlist name)  inside it and a third folder (video number)  inside it
Video has to be downloaded in third folder
Create a third folder (video number) for each video

Scheduler is coming to ytdl unlocked version
Change default output template for playlist to %(uploader)s/%(playlist_title)s/%(playlist_index)s/%(title)s.%(ext)s and save it

If you want to use scheduler now then you can use Task scheduler
Step 1: Add C:\Users\soura\AppData\Local\Youtube-dl GUI\Youtube-dl GUI to Path
Step 2: Add task to Task scheduler with parameters like URL, output, format . . .
Remember you have to change parameters each time manually.

No option

You are using old version

Wait for a few days for a newer version

Ok

Fixed in #28
This works if application is open


Sorting of data in grid is not accurate

After clicking around on the column headings.  Fix in the controller.

Should be fixed by https://github.com/softlayer/sl-ember-components/pull/846


Stargaze New Error Message

I am trying to claim my stargaze and after the first initial error message, I am now able to connect my ledger with keplr. However, when I approve transaction through my ledger to get the initial 20% Stargaze claim, a new error message shows up. Is it happening to anyone else?

I'm having the same issue, but with a different account number and sequence number (1). I'm using a different physical Ledger device than the one I used to set up the wallet, but with the same wallet imported on to it. Not sure if that would make a difference at all?

The error message is a complicated and confusing way of saying: "signature invalid". There is nothing you can do about it other than waiting for a fix. See also https://github.com/cosmos/cosmjs/issues/987#issuecomment-1014878386

Alright, well we have to wish for the best


calcualtions of adapter show strange values

Hi,
It seems that the calculation function of the adapter shows strange values.
For sure I can tell that SOC and solarInputEnergyTodayWh under "Calculations" show values that do not fit...
Is there anything I need to configure?
For SOC the value in calcualtion differs minimum by 20% from the value that is shown in the main tree
Currently I am running 1.7.7 of adapter
Thank you for some feedback / guidance...
best regards

Should be fixed with 1.8.1. Please try and report if issue is solved.

Hi, seems to be OK! Thanks a lot!


Remove httplib2, replace with Requests

Resolves #1998
Going through each package:

[x] core
[x] bigquery (dependency on storage)
[x] bigtable (no changes needed, grpc-only)
[x] datastore
[x] dns
[x] error_reporting
[x] language
[x] logging
[x] monitoring
[x] pubsub
[x] resource_manager
[x] runtimeconfig
[x] spanner (no changes needed, grpc-only)
[x] speech
[x] storage
[x] translate
[x] videointelligence (no changes needed, grpc-only)
[x] vision

The core changes (the first commit) should be ready for review @dhermes.

THE BUILD IS GREEN.
This is ready for final review. I'm apologize that I couldn't keep the history pretty towards the end.
@dhermes I will be trapped in meetings and interviews until tomorrow afternoon, can you push a non-remote branch to run the system tests? I ran most of them locally, but it would be great to get validation on Circle.

I'm apologize that I couldn't keep the history pretty towards the end.

We squash and merge, so who cares? :-)

@dhermes Thoughts on getting a green build for system tests? I can not think of how to do it without getting stuck on Stackdriver.

By the way, since we have a Speech release tomorrow, let's try to get this merged today if at all possible so we can get it out.

I can not think of how to do it without getting stuck on Stackdriver.

As long as they fail the same way then this is a success. :P

It's worth noting that the system tests do not exercise HTTP if there is a gRPC transport, so a local test by the committer / maybe others is more relevant.

@dhermes I don't have access to the usual project for system tests, do you think you could spin this up?
I can also attempt to run most of them against my own project.

Actually, scratch that- I'll run the system tests locally if you can get this otherwise reviewed.

@jonparrott Yup

Okay - system testing seems good (modulo expected issues). Shall we merge?

:tada:

Ahhhhhhh! :raised_hands: :confetti_ball: :1st_place_medal: :tada: :taco:

This is fantastic! 👏 I've been waiting for this to be thread-safe as I want to use it in a project I'm deploying soon. When is the next release?

Great question @alexcwatt. @lukesneeringer WDYT?


Support other implementations of Promises inside nuxtServerInit (currently only HTML5 Promises works correctly)

Fix for using for exapmle Axios promises in nuxtServerInit() without wrapping it to HTML5 Promise

Codecov Report

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

@@          Coverage Diff          @@
##           master   #456   +/-   ##
=====================================
  Coverage     100%   100%
=====================================
  Files          11     11
  Lines         449    449
=====================================
  Hits          449    449

Continue to review full report at Codecov.

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

Hi @taldy
Can you show an example where our code does not work?

@Atinux
// store/index.js

export const actions = {
  nuxtServerInit ({ commit }) {
    return axios.get('/api/products')
      .then(res => commit(SET_PRODUCTS, res.data))
  }
}

@Atinux I checked one more time and found that async/await solves my problem without any changes in Nuxt.
But Vuex documentation describe returning promise from action as a legal way to use the system. That's why I thought this is a some kind of bug.

I have same issue, maybe this code may be a better way to make sure the result is a promise. Not a object which has a property named 'then'
<% if (store) { %>
        let promise = (store._actions && store._actions.nuxtServerInit ? store.dispatch('nuxtServerInit', omit(getContext(context), 'redirect', 'error')) : null)
      promise = Promise.resolve(promise || undefined);

What about updating this line as well @taldy to make every method which should return a Promise to not be only Promise instances? https://github.com/nuxt/nuxt.js/blob/master/lib/app/utils.js#L87


fix(form): prevent reload loop with rjsf v5.20

Closes #72
rjsf 5.20 was causing issues with our e2e tests since sometimes, when dragging new (e.g. select) fields to the tree some of them would be typed as text fields. This is related to our use of rawErrors in the our custom validation function to pass down the path to FieldTemplate.
It turns out the error was being caused by this change in rjsf. That mergeObjects there with the preventDuplicates argument returns a deep copy of the object when called with two equal objects, meaning that the errorSchema object returned is essentially a new one. properties.content.props in ObjectFieldTemplate contains errorSchema, therefore it reloads when the errorSchema changes (after that change in rjsf it now does so on every validation call), causing cards to change, causing the uiSchema to be updated in the store, causing the Form in SchemaTree to refresh (as uiSchema is an object and useSelector uses a shallow equality check*), triggering a new validation, and continuing the endless loop.
The solution has been to add an extra check in ObjectFieldTemplate so that changes in cards only dispatch an update to the uiSchema when the order of the cards changes, and not in every situation when the cards array changes (which in fact happens on every re-render as it's an array and arrays are compared by reference, returning false even if both arrays have the same content, which is generally the case here). This breaks that loop.
Note: The way we use rawErrors to pass the path to the FieldTemplate is a bit hackish, but seeing that it keeps working after this change I would maintain it for the time being. There is no rjsf way of getting the path information functions such as toPathSchema only return the "path" from the current element to its children, but not from the root to the current element, which is what we need. So, the alternative, if we end up wanting to change this, could be to have a paths object in the redux store and populate it in the addByPath action, and we also would have to populate it on startup when loading an existing schema by traversing it completely. We would also have to disallow duplicate field ids everywhere in the form.
*We should consider using Immer for the Redux state to better protect ourselves against these situations

Merged with #77


Edit Conventional Commits

TL;DR
We need more conventions for our development team. We need a way to modify the conventional commits.
Detailed design
I couldn't find a description for how to change the way code reads the commits. For Example, I want to add improvement and package change titles for the change log file. Is there a way to do it with the current version?
Additional information
No response

Also, can I use commit descriptions and commit titles for changelog?

This action wraps the underlying release-please library. There is an option for updating the changelog sections.
I would suggest that you switch to a "manifest" configuration for any complex customizations (not all customization options will be propagated to action inputs in the future). Additionally, the manifest config files have a published JSONSchema which can be used for validation and documentation of every available option.
If you're looking to rewrite a commit message and you are using pull requests, there's also a feature for that: https://github.com/googleapis/release-please#how-can-i-fix-release-notes


Add gcloud-java

Add links to the gcloud-java project from the site and the documentation.

This is completed. The link now shows up on the site and all documentation pages.


u-datalist: Restore original read-only

Resolved by https://github.com/u-elements/u-elements/commit/80e690e8f337fbfa346bed558817600630a5571a


BallModel should be using inverse distance, but it isn't

There is a specific function in question, in module/localisation/BallLocalisation/src/BallModel.cpp:
arma::vec3 BallModel::predictedObservation(const arma::vec::fixed<size>& state,
                                              const FieldDescription& field,
                                              const Transform3D& Hcw) const {

       arma::vec3 rBWw      = {state[PX], state[PY], field.ball_radius};
       arma::vec3 rBCc_cart = Hcw.transformPoint(rBWw);
       arma::vec3 rBCc_sph1 = cartesianToSpherical(rBCc_cart);             // in r,theta,phi
       arma::vec3 rBCc_sph2 = {rBCc_sph1[0], rBCc_sph1[1], rBCc_sph1[2]};  // in roe, theta, phi, where roe is 1/r

       return rBCc_sph2;
   }

rBCc_sph2 appears to have been made so that it uses inverse distance, but that has since changed, making that line redundant.
The place where this function is called might make adjustments so that the end result is the same, but this is where those changes should be made.

@TrentHouliston @Bidski @danielginn


Unify pipeline status icons

I know this might be controversial and there were already a lot of back and forth by the pipeline status icons. But right now it is kind of inconsistent, e.g. status success, running and skipped are circled icons while others are not. I would really like to harmonize it and use circleOutline icons for all pipeline status icons.

Why are we migrating to the circle icons anyways we had: #2783 🤔

👋 hey guys, just trying to help. How about doing polls among maintainers on such subjective UI things? This would eliminate the 1:1 frictions and allows us to have more of a democratic way of moving forward.

Polls are fine for me, but often hard to get a majority as some maintainers are not participating or only read messages infrequently.

The reason we changed it before was that there are circle versions missing for some icons and therefore we sometimes mixed circle and none circle icons which we wanted to avoid. Seems like #3809 changed them unrelated to the plugin migration. Can't see a reason for any new discussions 🙃

For the status, not a single icon I can see is missing a circleOutline version.

Can't see a reason for any new discussions 🙃

There was never a discussion. You have not explained your reasons on the linked PR and feedback on this PR was ignored :shrug:  However, feel free to close this PR...

There was never a discussion.

We had it in another PR or chat (not interested in searching the comment / message atm 😅).
The service gear would stay without a circle?

However, feel free to close this PR... I feel a bit disappointed now as I have a feeling that efforts put into UI got blocked quite often...

Please know that review comments are meant to improve the final outcome and ensure we’ve considered all perspectives.

Polls are fine for me, but often hard to get a majority as some maintainers are not participating or only read messages infrequently.

I see the issue but this is similar to the other governance-related issue we had: if we just continue to merge such PRs which are actively debated or contain subjective parts like UI changes with one maintainer approval, we might make the majority unhappy or force even a revert with the argument "I didn't see it and couldn't even veto it". In addition, this causes friction among maintainers and eventually leads to "everyone quickly merges subjective changes to not even allow much of a discussion".
The same applies to opinions of owners IMO, i.e. they shouldn't be a hard veto for such changes if "enough" people liked it/agreed on it.
All the above is  without any opinion on the changes of this PR!
So we need to find a way to

allow all maintainers to cast a vote in a reasonable time
decide what the (majority) count should be that is required for merge (doesn't necessarily need to be > 50% of possible votes IMO)

(Yes I know this is off-topic and should be moved to a discussion but there it will likely go stale (?)).
Currently, the maintainers team consists of 8 real people of which 6 are reasonably active in such discussions.
As we can't enforce votes in timespan X from all members, I think we need to work with relative votes. In addition, the tagging of the maintainers team is used inflationary and most don't even look at the issues therefore.
Hence I propose the following:

No maintainer group tags anymore by default in PRs
Every UI-related change is coupled by a comment requesting to vote on the changes and tags the maintainers team (thumbs up/down)
The vote is kept open for three days and if there's a majority in favor of the changes, they get merged, otherwise not.
Only maintainer votes are counted, other user votes are discarded

Sorry for the wall of text. Yet, I think it is needed to find to reduce anger/emotions in decision making (I'll include myself here as well) and improves governance overall. @xoxys @anbraten what do you think?

We had it in another PR or chat (not interested in searching the comment / message atm 😅).

Just add context to the PR description could solve it (we should improve in this area anyway)

The service gear would stay without a circle?

The gear is already a circle shape, which fits optically for me. Technically, I get your point, and we could search for another circleOutline icon (IMO a gear is not really symbolic of services)

Please know that review comments are meant to improve the final outcome and ensure we’ve considered all perspectives.

You're right, sorry for my reaction, I'm trying to improve here.

My criteria are just: contrast between them is high enouth; symboles can be spotted as different one evenif monocrome; the bird for running pipelines stays :)
So a vote is fine by me, i will just intentionaly abstain to let people witj better UI/UX experience deside it :)

@woodpecker-ci/maintainers please vote on this post :+1: use circled pipeline icons :-1: use not circled icons from https://github.com/woodpecker-ci/woodpecker/pull/2783


Simplify platform abstraction

Brief feature description
Currently one needs to provide all headers for all platform, even if they are simply forwarding to the system header. This can be simplified by the C++17 feature __has_include
Detailed information
The platform would provide a generic implementation of the header. If the specific platform does not provide a custoom header, the generic one would be used.
This is an example for iceoryx_platform/include/iox/platform/semaphor.hpp
#ifndef IOX_PLATFORM_SEMAPHORE_HPP
#define IOX_PLATFORM_SEMAPHORE_HPP

#if __has_include("iox/platform/override/semaphore.hpp")
#include "iox/platform/override/semaphore.hpp"
#else
#include "iox/platform/generic/semaphore.hpp"
#endif // __has_include

#endif // IOX_PLATFORM_SEMAPHORE_HPP

The generic implementation in iceoryx_platform/generic/include/iox/platform/generic/semaphore.hpp would look like this
#ifndef IOX_PLATFORM_GENERIC_SEMAPHORE_HPP
#define IOX_PLATFORM_GENERIC_SEMAPHORE_HPP

#include <cstdint>
#include <semaphore.h>
#include <sys/types.h>

using iox_sem_t = sem_t;

#define IOX_SEM_FAILED SEM_FAILED
constexpr uint32_t IOX_SEM_VALUE_MAX = SEM_VALUE_MAX;

#ifndef IOX_PLATFORM_OVERRIDE_SEM_GETVALUE
inline int iox_sem_getvalue(iox_sem_t* sem, int* sval)
{
    return sem_getvalue(sem, sval);
}
#endif

#ifndef IOX_PLATFORM_OVERRIDE_SEM_POST
inline int iox_sem_post(iox_sem_t* sem)
{
    return sem_post(sem);
}
#endif

// ...

#ifndef IOX_PLATFORM_OVERRIDE_SEM_UNLINK
inline int iox_sem_unlink(const char* name)
{
    return sem_unlink(name);
}
#endif

#endif // IOX_PLATFORM_GENERIC_SEMAPHORE_HPP

For Linux, Unix, QNX and potentially other POSIX operating systems it would work out of the box.
FreeRTOS would specify a iceoryx_platform/freertos/include/iox/platform/override/semaphore.hpp header and partially reuse the generic header
#ifndef IOX_PLATFORM_FREERTOS_SEMAPHORE_HPP
#define IOX_PLATFORM_FREERTOS_SEMAPHORE_HPP

#define SEM_FAILED nullptr
#define IOX_PLATFORM_OVERRIDE_SEM_UNLINK

#include "iox/platform/generic/semaphore.hpp"

inline int iox_sem_unlink(const char*)
{
    // Named semaphores are not supported in FreeRTOS+POSIX
    configASSERT(false);
    return 0;
}

#endif // IOX_PLATFORM_FREERTOS_SEMAPHORE_HPP

Windows (and macOS) would have a full re-implementation in iceoryx_platform/windows/iox/platform/override/semaphore.hpp
#ifndef IOX_PLATFORM_WINDOWS_SEMAPHORE_HPP
#define IOX_PLATFORM_WINDOWS_SEMAPHORE_HPP

// fully custom implementation without using the generic header

#endif // IOX_PLATFORM_WINDOWS_SEMAPHORE_HPP

To use an out-of-tree platform one just needs to specify the path to iceoryx_platform/os/ like it is nowadays but it would be much simpler to add an maintain a platform which is mostly compatible with the generic platform.

@budrus @elfenpiff @FerdinandSpitzschnueffler @mossmaurice @MatthiasKillat @dkroenke any objections to the proposed refactoring of the platform? It would be part of a 4.0 release and nothing in the short term but we might make it a prerequisite for Android support.


position: off-screen tooltips in stories scroll with viewport

Bug
Expected behavior
Positioned tooltips stay positioned relative to the offscreen elements.
Actual behavior
If an element if offscreen (below the fold), the tooltip for it sits at the bottom of the viewport and stays there until the offscreen element scrolls into view. Then it stays in the correct position.
See: https://github.com/pluralsight/design-system/pull/1786
Steps to repro

Run Position stories
Ensure some positioned tooltips are below the fold in the viewport
Scroll
See tooltips move

Position is used by our dropdown menu type elements (MultiSelect, Select, Typeahead). So we have a function called clipAdjust in Position that keeps the menus on the screen when they'd otherwise go offscreen given their default position. I'm assuming Position is more often used in dev projects with Tooltip so in that case we should have a clipY (name pending) prop that defaults to false and then set to true when we want to keep portal element onscreen like in those dropdown edge cases.
The real question is for those tooltip targets that are on the edge of onscreen or offscreen do we want to clip the y position by adding some buffer zone or just ignore and let the user scroll to see tip. I'd like your opinion @jaketrent?


[FEATURE][SanityTest] Transmit the PPL query statement to Spark query/job description

Is your feature request related to a problem?
Flint doesn't transmit the PPL query statement to the underly Spark job/query description.
SQL Page:

Job Page:

What solution would you like?
Display the PPL query statement in Spark job/query description.
What alternatives have you considered?
A clear and concise description of any alternative solutions or features you've considered.
Do you have any additional context?
Add any other context or screenshots about the feature request here.

[Catch All Triage - 1, 2]


Templating probe address

Given
          args:
            - --health-probe-bind-address=:8081
            - --metrics-bind-address=<IP_ADDRESS>:8080
            - --leader-elect
          livenessProbe:
            httpGet:
              path: /healthz
              port: 8081
            initialDelaySeconds: 15
            periodSeconds: 20

how would I generate this output
          args:
            - --health-probe-bind-address=:{{ .Values.mydeployment.manager.probePort }}
            - --metrics-bind-address=<IP_ADDRESS>:8080
            - --leader-elect
          livenessProbe:
            httpGet:
              path: /healthz
              port: {{ .Values.mydeployment.manager.probePort }}
            initialDelaySeconds: 15
            periodSeconds: 20

I can see how to set the port in livenessProbe, but how to adjust the argument when only part of the argument value is to be templated?

It's impossible right now as the args is a list of strings and I didn't parse them into a map...
Here is how I set args in my project:
- strategy: control-if-yaml
  key: controller.args
  value:
    - --log-level={{ .Values.policy.controller.logLevel }}
    - --refresh-period=5s

Hadn't thought about putting helm templates directly to the values of the config file.
I'll give that a go on my project, and see how it is. Maybe I'll PR the readme's to include some advanced usage scenarios, and close this issue on the PR
Didn't take me long from finding this project (a day or two before I started raising issues) to find this as a potential problem.


Hover card should cause slowdown instead of pause

When hovering the card we should try to implement functionality so it slows down the game instead of pausing it.

Done 2a1034e


Implement Pino logger (production ready) + pre-commit added to dockerfile

This branch implements Pino logger using this dependecy.
By default Pino logger prints out json formatted logs that are suitable for prod monitoring systems.
For now, for development I added the pino-pretty add-on, that makes one-line, colorful logging possible.
From now on, every service that is to log something should have private readonly logger = new Logger(CLASS_NAME.name); attribute.
To log, use: this.logger.log('Message to log');
As a side feature, this branch adds pip & pre-commit installation to the dockerfile, so pre-commit works at the VsCode DevContainer.

I like what it looks like. 👍🏻
The question is, why is this message INFO? Shouldn't that be ERROR?

If you can improve that - it will be awesome. If it's planned for now then it's okay and you can merge this MR.

Generally it is not obvious to log HTTP requests as ERROR even if the status code is >400. ERROR is a pretty serious log level and one does not neccessarily need to store 401 Unauthenticated error that way.
Anyway for our development-to-prod purposes I changed the log level for HTTP errors, so It's clear when debugging.


Remove Nginx Build & Deploy from CI/CD

Type

[ ] Bug fix
[ ] New feature
[ ] Improvement to existing feature
[x] CI/CD

Related JIRA Issue(s)
https://www.ebi.ac.uk/panda/jira/browse/ENSWBSITES-1379
Importance
Cleanup CI/CD pipeline.
Description
The static assets are now published directly to PVC ( https://github.com/Ensembl/ensembl-client/pull/528 )
This doesn't need rebuilding & deployment of the Nginx container. This PR removes jobs that build & deploys the Nginx container.
Deployment URL

Views affected

Other effects

[ ] I have checked any requirements for Google Analytics
[ ] I have created any required tests
[ ] The PR has as its final commit a WASM/JS update commit if any Rust files were updated.

Possible complications

@azangru  I would like to keep all the deployments as identical as possible to avoid surprises in production.


Add support for HTTPS

https://tinyurl.com/api-create.php?url=${encodeURIComponent(url)}

@mrister I just did a PR to support HTTPS since it's finally supported 🎉
Thank you!

@mrister Thank you!
I just did a PR to support HTTPS since it's finally supported, unfortunately their API doesn't return HTTPS short links for now, I contacted them to know if they support the feature

Response from TinyUrl support... 🤦♂

All TinyURLs can be accessed using either http or https. For the output of the API, you can add in the S in your app.


Fix for https://kylo-io.atlassian.net/browse/KYLO-1192

If a Hive table exists with a name that ends with one of "_valid", "_invalid", "_profile", "_feed" it is not possible to browse the table using the "Tables" feature of Kylo. To reproduce, create a Feed using the standard ingest template with the name UserdataFeed. After successfully running an ingestion, no table associated to the feed is shown in the "Tables" feature of Kylo.

It looks like this was fixed by another change.

Ok thanks, will check the fix and close the PR if everything is ok

Please resubmit if the fix does not work for you.


##.banner is too strong

Word Banner means a lot of things. Just having CSS class "##.banner" does not mean it's an advertisement, it can be any graphical element on a site, e.g. a logo! Mainly referring to "2. a headline in large type running across a newspaper page" from Meriamm Webster dictionary. If you agree, please remove that filter. (It breaks certain sites' valid graphical elements, that I don't wish to list here.) Thanks for considering.
Definition of banner
(Entry 1 of 3)
1a : a piece of cloth attached by one edge to a staff and used by a leader (such as a monarch or feudal lord) as a standard (see standard entry 1 sense 1)
b : flag entry 2 sense 1 our nation's beloved banner
c : an ensign displaying a distinctive or symbolic device or legend (see legend sense 2a) especially : one presented as an award of honor or distinction
2 : a headline in large type running across a newspaper page The banner read "Kennedy Wins."
3 : a strip of cloth on which a sign is painted welcome banners stretched across the street
4 : a name, slogan, or goal associated with a particular group or ideology the new banner is "community control"— F. M. Hechinger —often used with under every new administration arrives … under the banner of change— John Cogley
5 or banner ad or banner advertisement : an advertisement graphic (see graphic entry 2 sense 2b) that runs usually across the top of a World Wide Web page

Hi,
thanks for the report, the affected rule will be removed in the coming commit.


Fix unblock read only file

Fix #4390.
Problem
If a file is read only it is skipping silently.
Fix
Analize GetLastError() and throw if a file stream hasn't been removed.
First commit only refactor tests to use ShouldBeErrorId.

@iSazonov what is the error message the user sees now?

MSDN says that GetLastError can return either "Access denied" or "File not found".
Unblock-File : Access is denied
At line:1 char:1
+ Unblock-File -Path $Testpath
+ ~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    + CategoryInfo          : NotSpecified: (:) [Unblock-File], Win32Exception
    + FullyQualifiedErrorId : System.ComponentModel.Win32Exception,Microsoft.PowerShell.Commands.UnblockFileCommand

@lzybkr @SteveL-MSFT Please continue the code review.


feat: allow a custom out dir from forge config

[X] I have read the contribution documentation for this project.
[X] I agree to follow the code of conduct that this project follows, as appropriate.
[X] The changes are appropriately documented (if applicable).
[X] The changes have sufficient test coverage (if applicable).
[X] The testsuite passes successfully on my local machine (if applicable).

Summarize your changes:
This PR allows using a custom out directory when packaging and making an app.
#2714
#1041
#214

@MarshallOfSound

How has this project existed so long without a custom output directory?

Its interesting how this was surfaced in in 2019 (https://github.com/electron/forge/issues/1041) and there has yet to be any action taken on this even though a pr exists with tests and is approved. Multiple people have commented how this would be super beneficial to them, what exactly is the hold up for this?
Why is MarshallOfSound the gatekeeper of what should or shouldn't be merged into this project? I would expect because this project is open source, if someone comes with a very useful pr with tests that has been requested by dozens of people there would be some kind of action eventually taken and it merged.

@luke-hagar-sp @galens Hey both, I appreciate that this PR may contain a change that you feel passionate about, but I'm going to ask you to please be respectful and also mindful of the fact that this is an open source project with limited maintainer resources. The opinion voiced by MarshallOfSound here is also shared by others in the maintainer group for Electron Forge. A valid technical opinion raised by one maintainer on the project should not be misunderstood as gatekeeping.
We'll review this PR at our next working group meeting and will comment back here at that time.

Hi, just to be clear: this PR adds the option only when using the configuration file and not through CLI or package.json, right?
This is not documented at all on the docs site.
Thanks.

It can also be used in the forge.config.js (or forge.config.ts if transpiling from TypeScript) file by adding: outDir: "path/here" to the top level config that is exported.


DO NOT MERGE:  Add strimi crd api

some updates for getting us onto Strimzi 0.6.0 and later.
currently in the state of testing, but issuing PR already

For got the API class - will follow soon ;-)

closing for now - sorry for the noise !


Missing line breaks when copying selected texts

Hi @adrianjagielak
I used flutter_markdown_selectionarea plugin in my Flutter app to support Selectable texts in the app.
But I found that after clicking 'copy', all line breaks are gone.
For example: If you copy selected content in the app:

The content in the clipboard would be
Heading 3This is a paragraph of text. You can make text bold or italic.You can create lists:•Item 1•Item 2•Item 3Or numbered lists:

I wonder if it is a bug in the library or not. The expected behavior is that the content with line breaks can be copied normally.

Its expected behavior, at least for now, as each paragraph is rendered into its own RichText widget and Flutter does not yet nicely handle inserting spaces/line breaks into text copied from multiple widgets.
I refer you to this: https://github.com/flutter/flutter/issues/104548
There is a work in progress to add support for text separators, PR: https://github.com/flutter/flutter/pull/121112
You could probably try to copy those changes to your local Flutter copy (basic.dart, framework.dart, scrollable.dart, and selectable_region.dart)

Thank you!

Check here maybe you can find solution. https://github.com/flutter/flutter/issues/104548#issuecomment-2051481671


I Found That setCancelled Cannot Be Used When Calling the WeaponEquipEvent Event Through the API

Description
Can the method e.setCancelled(Boolean) be added to WeaponEquipEvent?
Because I have discovered that setting the item content in events related to WeaponShootEvent will trigger WeaponEquipEvent
Configuration
public void onequip(WeaponEquipEvent e){
  if(somthiing){
e.setCancelled(true);
return;
      }
  }

Payment (optional)

[ ] I'm willing to pay for this feature

You can try to implement it! Good luck!
Sorry to sound rude... but it is impossible to add. You'll notice that WeaponEquipEvent works even when a dispenser equips the item into your hand. To understand why it is impossible, try to understand how we implemented the equipment event in the first place.


Add a test to verify that the user directory is properly updated after a lazy load join finishes syncing

As the title states.

Related: https://github.com/matrix-org/synapse/issues/12815

I forgot to mention: this passed when I ran it locally with workers, modulo the missing Notifier.notify_replication call of course.

If you're feeling up for it, you could try factoring out the common code from the 3 checks into an assertUserDirectoryContains local function. createJoinEvent has an example of the syntax.

In the interests of getting faster joins out there I'm going to merge this as-is; though by all means do feel free to factor this out.


Add operator image workflow

This PR switches the GitHub workflow to listen to the master branch so that we get a new operator image on any new commits on the master branch. We also added jobs to build bundle and openscap images to the same workflow. Also, we added a special bundle image dockerfile, so that the bundle image can point to the correct operator image and openscap image.

/retest

cluster issue, retest

/retest

/retest-required
Timeout issue with AWS, which we're seeing across several patches.

/retest


pm4 debug

add more syncs and tests passed. will debug from this working state.

tests pass. hlb works, testing resnet


Error while updating property blurRadius

Is this a bug report?
Yes , I think
The following code is working well on android 6 and 7 but on android 4.2.2 we have a problem

Have you read the Contributing Guidelines?
Yes
Environment
OS:  Windows 10
Node: v8.1.3
Yarn:  1.2.0
npm:  5.0.3
Watchman: Not Found
Xcode:  Not Found
Android Studio:  2.3.3

Packages: (wanted => installed)
    react-native: 0.49.3 => 0.49.3
    react: 16.0.0-alpha.12 => 16.0.0-alpha.12

Target Platform: Android(4.2.2)

Steps to Reproduce

import React, { Component } from 'react';
import { AppRegistry, Image } from 'react-native';

export default class Bananas extends Component {
  render() {
    let pic = {
      uri: 'https://upload.wikimedia.org/wikipedia/commons/d/de/Bananavarieties.jpg'
    };
    return (
      <Image source={pic} style={{width: 193, height: 110}} blurRadius={0.5}/>
    );
  }
}

Expected Behavior

Blur image
Actual Behavior

UP

I am seeing this 7.1.0...

Please fix the issue or at least add a method that let us blurRadius is supported or not

This may be fixed by #16845.


Position API unification

Is there any reason why Tree.meta uses start_pos while Token uses pos_in_stream? Do the two mean different things?  If not it would be handy to have the API unified.
Use case is related to issue. After unification it would be easy to construct Token borrowing rule's position: Token.new_borrow_pos("type", "value", tree.meta).

No, they are the same thing in different contexts.
I agree it would be better to unify them into one name.

Recently fixed this.


v1.9.2

Patch release, before merging feature PRs.

[ ] once-over from @jfmengels
[x] check against test set
[ ] publish to latest on npm

Those wouldn't be part of this anyway, I need to publish the Node resolver separately to fix that. I had marked that as part of milestone 1.10, but really it's just a pre-req (since there are tests that require it now).

Ok, sounds good to me. Btw, you're on a roll today :smile:


Rollout restart Virt-handler only restarts one pod.

What happened:
When rolling out virt-handler daemonset, only one pod got restarted and status shows it was successfully rolled out.
What you expected to happen:
All virt-handler pods should be restarted.
How to reproduce it (as minimally and precisely as possible):
Simply run kubectl rollout restart ds virt-handler -n kubevirt
Environment:

KubeVirt version (use virtctl version): v1.2.0

/cc @enp0s3

/remove-lifecycle stale

@ProblemTryer Hey!
That's indeed an issue, since we do use canary upgrade method, but it should happen only upon Kubevirt upgrade, and
not during simple rollout.

@aburdenthehand Hi, sorry but fix is yet to come due to lack in bandwidth.


Fix  some documentation issues

Pull Request Test Coverage Report for Build 603047793

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 94.669%

Totals

Change from base Build 602734269:
0.0%

Covered Lines:
1172

Relevant Lines:
1238

💛  - Coveralls


Every extras property seems to be stringified a second time

(not sure if bug or feature request)
I view reported issues in GlitchTip. In the Additional Data section, which I populate using setExtra every string is wrapped in quotes, and every JSON object (which I already stringified because setExtra only accepts strings) is escaped with backslashes, leading to very unreadable JSON data.

Request:
Either allow passing objects to setExtra or do not stringify string properties a second time

Hmm, it seems that it's not actually micro-sentry but me who passes an escaped string to micro-sentry 🤔

Closing as it's an issue on my side, sorry for the confusion.


Abstract playground files under npm package

Problem
The Cosmos setup requires you to add two playground-specific files to your project:

Webpack entry file
Function for traversing through your project files and builds a tree of component with fixtures

I was already thinking of abstracting these somehow as they make the integration less pleasant, when @goatslacker also suggested it on Gitter.
The reason I was hesitant to hide these files was because they make customization options obvious. However, since I don't remember anybody customizing the way Cosmos expects to find components and fixtures within a project, I'm going to make the assumption that customizing the files above is more an exception than a rule.
Solution

[x] Bring back cosmos-js package (now becoming 0.7), containing entry.js and get-component-fixture-tree.js
[x] Use cosmos-js as entry inside the playground webpack config
[x] cosmos-js dependency replaces react-component-playground and react-querystring-router dependencies
[ ] Update README section to reflect how would one go about customizing these files once they're no longer part of the host project.
[ ] Update react-0.14 boilerplate and add note under react-0.13 stating that it no longer receives update and latest setup is found under react-0.14

@goatslacker wdyt?

This is good I just have a note on React 0.14/0.13 compatibility. How about you support both?
You can create a getDOMNode function like
function getDOMNode(node) {
  return React.version.indexOf('0.14') > -1 ? node : node.getDOMNode();
}

You can then also conditionally load ReactDOM and use it to render. That seems to be all you're using React 0.14 for.
Also need to update the peer dependency requirements. You can probably even get away with 0.12 support.

This is good I just have a note on React 0.14/0.13 compatibility. How about you support both?

Good idea, created #179 for this.


"Maglavit" , "Hunia" are listed in the drop-down menu

Description:
When user writes ”Maglavit" , "Hunia" the locations appear in the drop-down menu next to the comune and the county which is a part according to the law.
Precondition:
The website is up an running.

Step 1
Write in the search bar "Maglavit"

Expected results
The location appears in the drop-down menu as "Sat Maglavit, DOLJ (Maglavit)".

Step 2
Press "x" button.

Expected results
The location was deleted from the search bar.

Step 3
Write in the search bar "Hunia"

Expected results
The location appears in the drop-down menu as "Sat Hunia, DOLJ (Maglavit)".


Candlestick: draw horizontal line if open equals close

antilles79 from the Discord (#1966) noted that candlestick plots have two issues:

[ ] candles at X=0 have unexpected width
[ ] candles where open equals close do not display a horizontal line

List<OHLC> price = new List<OHLC>();

price.Add(new OHLC(100, 103, 99, 102, new DateTime(1899, 12, 30), TimeSpan.FromDays(1)));
price.Add(new OHLC(100, 103, 99, 102, new DateTime(1899, 12, 31), TimeSpan.FromDays(1)));
price.Add(new OHLC(101, 103, 99, 101, new DateTime(1900, 1, 1), TimeSpan.FromDays(1)));
price.Add(new OHLC(100, 103, 99, 102, new DateTime(1900, 1, 2), TimeSpan.FromDays(1)));
price.Add(new OHLC(100, 103, 99, 102, new DateTime(1900, 1, 3), TimeSpan.FromDays(1)));

formsPlot1.Plot.Add.Candlestick(price);

These issues can probably be resolved by editing this section of code:
https://github.com/ScottPlot/ScottPlot/blob/2a575c80b6a7a31d00ff57d8de956e288abc22dc/src/ScottPlot5/ScottPlot5/Plottables/CandlestickPlot.cs#L63-L109

I have also faces this issue and I will higly appreciate it if it will be considered soon.
Moreover, when close=high=low=open, they become invisible.

Fixed! This will be in the next release.

Hello, Scott!
Thank you for solving this issue!
I wanted to ask: What if open=close=high=low? Do they become invisible as previously?
Thanks!


Possible to extract selected items in a combox?

Been trying to create a Solara component with reacton.ipyvuetify elements. Essentially a Combobox in which I eventually want to be able to select items and dynamically store them in a variable - or store them at the click of a button.
I tried this but got stuck:
import solara as s
import reacton.ipyvuetify as v

items = "item1 item2 item3".split()
chips = s.reactive([items[0]])

@s.component
def Page():

    def on_change(widget, event, data):
        chips.set([???])

    with s.ColumnsResponsive(10, large=[6,6]):
        cb = v.Combobox(
            solo_inverted=True,
            background_color="success",
            solo=True,
            items=items,
            chips=True,
            deletable_chips=True,
            multiple=True,
            autofocus=True,
            prepend_inner_icon="mdi-filter-variant",
            placeholder="Select relevant items",
        )

        btn = v.Btn(children=['CHECK'])
        v.use_event(btn, 'click', on_change)
        s.Markdown(f"Selected: {chips.value}")

This should work:
import solara as s
import reacton.ipyvuetify as v

items = "item1 item2 item3".split()
items_selected = s.reactive([])

chips = s.reactive([items[0]])

@s.component
def Page():

    def on_change(widget, event, data):
        chips.set(items_selected.value)

    with s.ColumnsResponsive(10, large=[6,6]):
        cb = v.Combobox(
            v_model=items_selected.value,
            on_v_model=items_selected.set,
            solo_inverted=True,
            background_color="success",
            solo=True,
            items=items,
            chips=True,
            deletable_chips=True,
            multiple=True,
            autofocus=True,
            prepend_inner_icon="mdi-filter-variant",
            placeholder="Select relevant items",
        )

        btn = v.Btn(children=['CHECK'])
        v.use_event(btn, 'click', on_change)
        s.Markdown(f"Selected: {chips.value}")

Thank you Mario!


fix grammar in documentation

Reduction of typos and correction of German grammar

@ruderphilipp cheers for the fixes! That was all "written on the go" so you just saved me a mass of work!


Windows 10 - Keybase installer will start but after a while it disappears

I have this weird problem on my win 10 machine, that when i launch the .msi installer it starts and when it reaches the: "Please wait while Windows configures Keybase" the installer window disappears in the middle of the loading. But Keybase program appears in "Programs and Features" in the control panel.
Please help!

Newermind it got installed on my system but i have to locate the .exe file myself to launch it.
The location of .exe file is in %LOCALAPPDATA%\Keybase\Gui\Keybase.exe

Mgfd36.001

%LOCALAPPDATA%\Keybase\Gui\Keybase.exe

@goeb36


Check for provided config file

Implementation of #1264

Coverage remained the same at 66.05% when pulling c4ff5e0bc240377a0ec61d7844e6bfedd58415f1 on feature/script_check_config into 6bf8ff28dd24c06636a2549615e466a17d8bd3e3 on develop.


added vue-types reference

Hi,
I've added to the list my latest package that collects some prop definition in a React.Proptypeslike interface.
Relevant discussion in the forum: http://forum.vuejs.org/t/vue-prop-types-definitions-package/2264/6

Thank you for your PR.
Oh, conflicting... sorry,
Can you resolve the conflict please?

Hi,
It should be fixed now.
Bye!


Add Justify as a suite user to README.md

Add the Java-implemented validator as a suite user to README.md.

Great, thanks!


Landing Page is not responsive

Description
Landing Page doesn't seems to be responsive for different screen sizes.
Concerning Folder

https://github.com/houseofgeeks/visualrithms/tree/master/src/Pages/Home
https://github.com/houseofgeeks/visualrithms/tree/master/src/components/Navbar

Screenshot

Labeled, thanks for reporting!

I can work on this one. 😁

You can go for it @pablowbk


Run build on target docusaurus site before update

Work for this issue will begin begin on branch work-liquid-labs/cloudsite/18.


Camel 10086 - Moved Pattern.compile() usages into static variables

This PR covers a first set of changes for CAMEL-10086 which targets extracting some Pattern.compile() usages from within methods into static variables.
Couple of things to note:

I did not pay attention and ended up with 3 commits. if this is a problem let me know and I'll close this PR, squash the commits on a new branch and re-raise
This PR also has some typo fixes and some license fixes (I found those by chance and could not leave them as they were); I hope that's ok

Thanks this looks good.

Thanks for the PR it has been merged. Do you mind closing this?

Closing this PR as it was merged


Viewmodel is nonreactive to changes in template data

Here's an example
Template.test.viewmodel 'test',
	name: "John"
	newName: ''
	changeName: ->
		@name @newName()
, "name"

Template.child.viewmodel 'child', (data) ->
	name: data
, "name"

template(name="test")
	input(data-bind="value: newName, returnKey: changeName")
	p #{name}
	with name
		+child

template(name="child")
	p #{name}
	p(data-bind="text: name")

The first output will be
John
John
John

Entering 'Mike' and pressing enter yields
Mike
John
John

Please make a repro (something I can clone and run). That way I don't have to guess what it should do.

Ok, here it is: https://github.com/nicolamr/test
I also added the test for the other issue I posted.
I never had to do this, so if there's a more efficient way to make a repro please let me know.

That's good. The first problem is a little bit confusing so I'll get back to you on that one. The second one is simple: when you use 'helper' as a helper method then @ (this) refers to the helper context, not the view model. When you use 'helper' in a data bind then @ (this) refers to the view model and it does have a parent.

Here's the deal, Blaze is kind of cheating in that the name property is reactive only for helper purposes. It knows that the helper comes from the parent template and thus hooks/binds it the parent. To give you an idea, you can't get the same behavior if you try to put name in a Tracker.autorun.
On the other hand the view model is only getting a non-reactive object when it's initialized ({name:"John"}). You have 2 options: 1) pass an object to the child with a function that returns the parent's name or 2) reference @parent().name() from the child. #2 is the simplest/cleanest one.

Thank you for your time. I appreciate your efforts. I spent a couple of days exploring your module, which is very nice. But I got into these complex situations because I have forms with parts that change depending on the user input. As soon as you have to communicate between different levels of viewmodels/helpers, it starts to feel hacky and not intuitive (a lot of time is spent trying things out without knowing whether they will work). I wonder why the meteor development team didn't go all the way to implement full two-way data binding. I'd have preferred your module, which adheres much more to the meteor philosophy, but I think I'm going to take a look at angular-meteor for my current project. Thank you again, and keep up the good work.

The problem is that you're expecting your view models to have dynamic contexts like Blaze does. In Blaze the context changes every two steps. If you look at Blaze 2 proposals they're trying to move away from it and standardize the context. In ViewModel the context is always the same (the view model, however you create it).
If you're going with Angular I suggest you take a look at the Aurelia project (http://aurelia.io/). It's to Angular as Coffeescript is to Javascript.


Should We Show Cards When Everyone is All In?

ya that might be fun. is that what poker starts does?