How to start a 'normal' Chrome instance? Chrome extensions are needed for development

Some use-cases need extensions for development (e.g. the MetaMask extension injects a web3 object into every page).
Is there a way to start the 'normal' Chrome instance with flutter run -d chrome?

duplicate of https://github.com/flutter/flutter/issues/43809

Honestly, this did not help at all. How can I use extensions when I put my program in debugging mode?

Hello @hosseinkhojany. You can use flutter run -d web-server to run the application, which then you'll be able to open in any browser, including extension enabled. You can then use the Dart Debug Extension to debug your application. Let me know if this solved your issue

I have the same issue: I desire to use a “normal” Chrome instance, not one which Flutter spins up. Furthermore, using flutter run -d web-server, which appears like an alternative, there is no “hot restart” functionality.

Marking as duplicate of https://github.com/flutter/flutter/issues/43809. If you disagree, please comment and we can reopen it

Hi Nabeel (@exaby73), the fact that there is a duplicate issue (#43809) does not mean that this issue should be closed. That issue was closed due to non-activity; this issue was presumably opened again to address the same underlying problem (namely that flutter run -d chrome launches its own Chrome instance, and so presents certain limitations such as that extentions aren’t available).

@rmschindler https://github.com/flutter/flutter/issues/43809 was closed not because of inactivity but rather by @yjbanov. See their comment.
I will reopen this issue for further insights from the team. Thank you for the feedback

Looks like --no-web-browser-launch option is gone. However, extensions specifically are being addressed in https://github.com/flutter/flutter/pull/109157. Would it be fair to deduplicate this with https://github.com/flutter/flutter/issues/109156?


[fix]:README Typo

[x] fixes #264
-> small typos fixed
[x] fixes #252
-> Updated markdown code
[x]  Other previous markdown errors fixed
[x] fixes #259

Hi @SamsShow
All the fixes are from issues assigned to different contributors. Kindly ask to be assigned an issue before working on it.


cannot call on a un-initialize UniTensor_base

Hi,
I tried to run DMRG and got the following error:
RuntimeError:
Cytnx error occur at virtual bool cytnx::UniTensor_base::is_blockform() const
error: [ERROR] fatal internal, cannot call on a un-initialize UniTensor_base

This error did not occur in version 0.9.2 but it does in v0.9.5. Was is_blockform() removed for UniTensors?

This error did not occur in version 0.9.2 but it does in v0.9.5. Was is_blockform() removed for UniTensors?

There are some API changes in the Network and the example codes are not updated, we will fix them soon.

@yaotingsu  @manuschneider  Which DMRG codes that your are running?

@yaotingsu @manuschneider Which DMRG codes that your are running?

ok it seems like it's the NCTS summer school code.


Updated to Jacoco 0.8.8

Ticket Link
Ticket
List of squashed commits

Updated to Jacoco 0.8.8

Test Checklist

[x] Unit Tests Passed:
[x] Jasmine Tests Passed:
[x] General Visual Inspection:

s/b java 17, not 18

These changes look fine. I will go ahead and merge these changes


Setup Sphinx + RTD

Need to go to Advanced Settings on RTD and change the Default branch. This is defined as:

What branch "latest" points to. Leave empty to use the default value for your VCS.

Switch this to main from setup-docs once this PR is merged!

Docs visible at saul.rtfd.io.


Extra characters while inputting Tamil Text

While Inputting Tamil text with compound letters (கா, கி, etc.), Xi-editor generates additional characters.
The rpc log collected shows that xi-mac making redundant duplicate rpc calls for each compound character insertion.
This causes extra characters to be left in the text.
Steps to recreate:

Enable Tamil99 Keyboard in Input sources
Press following keys -> hqhslkf
Expected text -> காகிதம் (U+0B95 U+0BBE U+0B95 U+0BBF U+0BA4 U+0BAE U+0BCD)
Generated text -> ககாககிததம் (U+0B95 U+0B95 U+0BBE U+0B95 U+0B95 U+0BBF U+0BA4 U+0BA4 U+0BAE U+0BCD)

Logs:
The duplicate calls are marked bold.
{"method":"edit","params":{"method":"insert","view_id":"view-id-2","params":{"chars":"க"}}}
{"method":"edit","params":{"method":"delete_backward","view_id":"view-id-2","params":[]}}
{"method":"edit","params":{"method":"insert","view_id":"view-id-2","params":{"chars":"கா"}}}
{"method":"edit","params":{"method":"delete_backward","view_id":"view-id-2","params":[]}}
{"method":"edit","params":{"method":"insert","view_id":"view-id-2","params":{"chars":"கா"}}}
{"method":"edit","params":{"method":"insert","view_id":"view-id-2","params":{"chars":"க"}}}
{"method":"edit","params":{"method":"delete_backward","view_id":"view-id-2","params":[]}}
{"method":"edit","params":{"method":"insert","view_id":"view-id-2","params":{"chars":"கி"}}}
{"method":"edit","params":{"method":"delete_backward","view_id":"view-id-2","params":[]}}
{"method":"edit","params":{"method":"insert","view_id":"view-id-2","params":{"chars":"கி"}}}
{"method":"edit","params":{"method":"insert","view_id":"view-id-2","params":{"chars":"த"}}}
{"method":"edit","params":{"method":"delete_backward","view_id":"view-id-2","params":[]}}
{"method":"edit","params":{"method":"insert","view_id":"view-id-2","params":{"chars":"தம"}}}
{"method":"edit","params":{"method":"delete_backward","view_id":"view-id-2","params":[]}}
{"method":"edit","params":{"method":"delete_backward","view_id":"view-id-2","params":[]}}
{"method":"edit","params":{"method":"insert","view_id":"view-id-2","params":{"chars":"தம்"}}}
{"method":"edit","params":{"method":"delete_backward","view_id":"view-id-2","params":[]}}
{"method":"edit","params":{"method":"delete_backward","view_id":"view-id-2","params":[]}}
{"method":"edit","params":{"method":"insert","view_id":"view-id-2","params":{"chars":"தம்"}}}

Okay looking at this a bit more, there's something strange happening in our IME handling: our fn replaceCharactersInRange: is being called twice, first by setMarkedText and then again by insertText; and since the actual insertion is handled in replaceCharactersInRange, that's being called twice as well.
The next step here is to go re-read whatever docs are available on the NSInputClient protocol, to figure out exactly what the expected behaviour of these two functions is, and update them accordingly.


Author pages

A page that lists all team members
Improved page template for customized homepage

TODO:

Styling
Descriptions for each team member
Link from homepage

Currently there's no specific style used in the team page or member pages. @thevangelist could we get your input for the style and structure for those pages?.

I can check the styles, I'll probably do a SCSS rewrite with BEM #8

@thevangelist cool, thanks!


Fix typos in props and computed property

Hi,
Thank you for making vue-comment-grid. It saved me a lot of time 👍
I fixed some typos in props and computed property.

Hi,
Thanks for fixes 👍


Add Shortcut to Cut--Button

see https://github.com/joeladdison/voctomix/commit/6582e5ec648574fc84ccefe3258759e3543a0647

Seems this was already merged in #55


just a question about frequency of  .DAILY

I just want to know exactly how often the .DAILY checks (or WHEN it checks).
My issue is the following:
I have the checker set to .IMMEDIATELY right now. I store the version.json file on my github in /docs folder and commit it to the repo when I'm ready to release the update. The problem with this is that it causes people in my app to get the update message before the update is approved by Google Play (takes  a few hours).
SO I thought .DAILY would be better. But, when I had it set to .DAILY, not matter how often I opened the app, it never seemed to notify me that there was an update. (this is why I set it to .IMMEDIATELY).
So what exactly is the frequency that triggers the .DAILY setting to display the dialog to the users of the app?

FWIW, I usually update the json file a couple of days after release, after I'm certain the new version isn't causing problems for the early/automatic updaters. So that might explain why I haven't had this particular experience, but it definitely makes sense.

(Closed due to lack of response, and I think the explanation is complete...)


Frosted glass

Should we have mixins for common effects, like frosted glass, @evyperez ? @Eduruiz ?

looks really cool, and since it's a mixing it doesn't spit out any CSS if not used so :+1:

Yes, I love it.

So, let's do it! The branch next is already working on Sass 4.
Do you do it, @evyperez ? Just remember to use always git pull --rebase before to push.


Automated cherry pick of #1570: fix: quota encode json allowing zero value

Cherry pick of #1570 on release/2.11.0.
#1570: fix: quota encode json allowing zero value

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.11.0@4c9ea9d). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             release/2.11.0   #1572   +/-   ##
================================================
  Coverage                  ?   7.69%
================================================
  Files                     ?     411
  Lines                     ?   62922
  Branches                  ?       0
================================================
  Hits                      ?    4840
  Misses                    ?   57673
  Partials                  ?     409

Flag
Coverage Δ

#aFlag
7.69% <0%> (?)

Impacted Files
Coverage Δ

pkg/compute/models/quotas.go
3.33% <0%> (ø)

Continue to review full report at Codecov.

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


Ready state changes require a bump to the message to re-sequence it

Fixes the core symptom reported in #421

Note that this does not implement the full change to batches described in the discussion on that issue.
However, when that change happens, this fix will still be applicable.

Could also call it simply "ReplaceMessage" - will leave it up to you if you prefer the current name though.

Codecov Report

Merging #438 (4b62785) into main (afb4c84) will not change coverage.
The diff coverage is 100.00%.

:exclamation: Current head 4b62785 differs from pull request most recent head 3a32ae6. Consider uploading reports for the commit 3a32ae6 to get more accurate results

@@            Coverage Diff            @@
##              main      #438   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files          277       277
  Lines        14939     14955   +16
=========================================
+ Hits         14939     14955   +16

Impacted Files
Coverage Δ

internal/database/sqlcommon/message_sql.go
100.00% <100.00%> (ø)

internal/events/tokens_transferred.go
100.00% <100.00%> (ø)

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 afb4c84...3a32ae6. Read the comment docs.


Please Release the current mainline to support Custom Port

Is your feature request related to a problem? Please describe.
Currently you are not support another Port as the default port
Describe the solution you'd like
8 months ago would added the custom port support
Describe alternatives you've considered
Additional context
Thanks for Creating this mega Module it made my implementation easier

Forgot to close this when port support was released.


[MvRx & Epoxy] State is outdated in OnClickListener

I have my button declared like that
    override fun epoxyController() = simpleController(viewModel) { state ->
        ...
        button {
            id("button")
            Timber.d("state in button = %s", state) // here my state is updated and works as expected
            title(formatButtonText(state.picksCounter))
            onClick { _ ->
                Timber.d("state in onClick = %s", state) // here I have the old state and my alert shows old info
                showConfirmationAlert(state)
            }
        }

For some reason, my state in onClick isn't refreshed until I will scroll my RecyclerView.
The state in the button is always up to date.
Do you have an idea of what could be an issue or shall I prepare a simplified sample project?
[EDIT]
I solved it by adding val showConfirmation: Boolean to my state.
My epoxy config looks now like that
        button {
            id("button")
            onClick { _ ->
                viewModel.showConfirmation(true)
            }
        }

        if(state.showConfirmation) {
            confirmSelectionsDialog = showConfirmationAlert(state)
            viewModel.showConfirmation(false)
        }

Let me know if it's a good practice what I did and I'm still curious why onClick has outdated state.

@qbait when you do
onClick { _ -> state.doSomething }, you are implicitly capturing the state inside the lambda, and since state is a data class, it's making a copy. When epoxy rebuild the model, callbacks are not hashed, so it still has the old lambda capturing the old state.
Here's 2 solutions you can consider

do not pass the state to showConfirmation. call withState inside showConfirmation to get the most up to date state
Hash the callback. Something like

  class KeyedOnClickListener {
    val key: WhateverKeyThatHasProperDefinedHashValue
    val listener: OnClickListener
    func hashCode() {
        return key.hashCode
    }
  }

^^ (pseudocode above)
Not sure if option 2 is already supported in epoxy? @elihart


Softmax dim

I concern that softmax in routing process performs incorrectly.
c_ij = F.softmax(b_ij.unsqueeze(0))
The current softmax automatically choices dimension 1, which is input capsule (size: 1152 for MNIST dataset). The dimension for softmax should be dimension 2, which is output capsule (size: 10 for MNIST).
c_ij = F.softmax(b_ij.unsqueeze(0), dim=2)
However, curiously, this revision makes training and test margin losses very large. In my validation, I succeeded learning by initialization of W with uniform_(-stdv, stdv) and reducing initial learning rate to 0.001. I would appreciate if you could confirm this issue.

I'm so sorry for my late response..
Thank you so much for your pull request.
 I trained the network with your fix and found that the test accuracy is improved a lot (from 99.51% to 99.67%).
I have merged your fixes to the master.
Thank you again for your pull request!


Summary return types fixed, Server Tests added

Return type from AnalysisResult to a String wrapper class (since we cannot return as a String).
@lukechurch

@Georgehe4 This needs a test before we merge.

Pushing this back until the week is done.

@devoncarew @lukechurch
Ready for review.

I opened a bug for the test timeouts.
lgtm! @lukechurch?

lgtm

Merging.


Can not run container after libnetwork integrated to Docker

It's a big problem but no one reported because I got it on ARM64. Anyway I don't think it's an architecture related problem.
Error message from daemon:
ERRO[0006] leaving endpoint failed: no container attached to the endpoint
ERRO[0006] Handler for POST /containers/{name:.*}/start returned error: Cannot start container 729e0b1a4b68c970b1067be56c9acd741dc9260436aa41a893b354ecb61a5246: Link not found
ERRO[0006] HTTP Error                                    err=Cannot start container 729e0b1a4b68c970b1067be56c9acd741dc9260436aa41a893b354ecb61a5246: Link not found statusCode=404

Not much useful, after some digging, I found error occured in libnetwork:
container.AllocateNetwork()
  -> ep.Join()
    -> sb.AddInterface()
      -> netlink.LinkByName()
        -> execGetLink()
          -> req.Execute()

here it failed in this block:
239                         if m.Header.Type == syscall.NLMSG_ERROR {
240                                 native := NativeEndian()
241                                 error := int32(native.Uint32(m.Data[0:4]))   // error = -19
242                                 if error == 0 {
243                                         break done
244                                 }
245                                 return nil, syscall.Errno(-error)
246                         }

This kind of error (-19 NODEV) looks like lack of some kernel support, when I run Docker on arm64 (that's before libnetwork integrated to Docker), I met exactly same error (that time was using netlink code in libcontainer), finially I found it's because I enabled veth as module and not loaded it, after enable veth as y, Docker worked well.
After libnetwork integrated to Docker (I bisected, exactly after this integration), I got this error again, I thought maybe libnetwork using some features depend on some kernel config, which is enabled by default in most distributions and not mentioned in check-config.sh. Since I worked on a fresh kernel, got this problem.
I'm not a network expert, hope can get some help here. Thanks.
PS: check-config.sh is all green in my box, they are all enabled (not as module). And I already tried to enable all netlink related configs, no lock.
$ cat .config | grep NETLINK
CONFIG_NETFILTER_NETLINK=y
CONFIG_NETFILTER_NETLINK_ACCT=y
CONFIG_NETFILTER_NETLINK_QUEUE=y
CONFIG_NETFILTER_NETLINK_LOG=y
CONFIG_NF_CT_NETLINK=y
CONFIG_NF_CT_NETLINK_TIMEOUT=y
CONFIG_NF_CT_NETLINK_HELPER=y
CONFIG_NETFILTER_NETLINK_QUEUE_CT=y
CONFIG_NETLINK_MMAP=y
CONFIG_NETLINK_DIAG=y
CONFIG_SCSI_NETLINK=y
CONFIG_QUOTA_NETLINK_INTERFACE=y

@hqhq Thanks for bringing this to our attention.
We dont have access to this specific environment to try it out.
libnetwork uses https://github.com/vishvananda/netlink as our netlink library.
Can you please do some investigation on the assumptions this library has for your environment ?
Also, it would be helpful if you can narrow down the specific issue that you mentioned that got resolved in earlier version of docker that is resurfacing with libnetwork. That will help quickly isolate the problem and probably a PR on the above netlink library.

@hqhq Thanks for report though. Can you try to run vishvananda/netlink tests on your machine?

@mavenugo Last time, I got this error and fixed by enable CONFIG_VETH, then with the same kernel, I updated Docker code, I got this error again.

 Can you please checkout my patch and re-run the vishvananda/netlink tests? Thanks in advance!

@hqhq fyi the two tests failing on x86 are likely because your kernel is too old to support ipvlans. Support was added in 3.19

Closing this as I believe the issue has been resolved


NIX.md: update link to point to official wiki

thank you, and sorry for the delay in the response, this repo now  is changed to HVM2 and we don't have a NIX set yet for the HVM2, however, HVM1 still exists, and i'll change make this change to the official Nix wiki there


Remove render-texture dependency from the backend

GearVRF DCO signed off by: Roshan Chaudhari<EMAIL_ADDRESS>

Need rebase

Planning to merge on Thursday

I think we should wait as GVRRendleBundle needs refactoring and there is dependency on backends.

I removed IRendleBundle interface so we can have backend specific RendleBundle.

Looks good to me. Can be merged when the dependencies are ready.

Planning to merge on Tuesday


[PE-D][Tester C] Find command message not updating

The find command should return a corresponding results display message, but it is not showing. It shows the previous command display message instead.

Labels: severity.Medium type.FeatureFlaw
original: kaitinghh/ped#10

Closing as duplicate of #140.


where academy/academy.csv

hey @xiaokuili  i didn't upload a copy of academy.csv as that's actually real operational data from my company Algoritma Data Science Academy (https://algorit.ma).
I thought it's good to show how I build my LLM workflow and processes using actual data, from my actual work, rather than some dummy data. Unfortunately I can't share academy.csv --- the lessons should still apply to pretty much any csv files though!


allow simple casting to IComputeShaderPath

Without this change, casting to IComputeShaderPath would not allow to use the properties from the object. so if we do this:
(computeShader.shaderPath as IComputeShaderPath).computeSource

it will still fail, saying it doesn't exist on string. I avoid using generics here, this is just a simple solution to a simple problem :-)

Please make sure to label your PR with "bug", "new feature" or "breaking change" label(s).
To prevent this PR from going to the changelog marked it with the "skip changelog" label.

Visualization tests for WebGPU (Experimental)
Important - these might fail sporadically. This is an optional test.
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14924/merge/testResults/webgpuplaywright/index.html

WebGL2 visualization test reporter:
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14924/merge/testResults/webgl2playwright/index.html

There's the same problem in Effect.IShaderPath :)

Done! :-)

Visualization tests for WebGPU (Experimental)
Important - these might fail sporadically. This is an optional test.
https://babylonsnapshots.z22.web.core.windows.net/refs/pull/14924/merge/testResults/webgpuplaywright/index.html


Unexpected behavior with exists when list promotion is enabled

@brynrhodes:
With list promotion turned on, a boolean that is fed into the exists operator is turned into a list. Doing this will always cause the exists operator to return 'true'. As a result, something like the following will always true:
exists(exists(null))
or
exists(false)
Hopefully an author would never do this, but if an author did, it would have unintended consequences.  Earlier versions of CQL-to-ELM prevented the above scenario from happening.
Is list promotion working as intended?

List promotion is implemented with a ToList() operator, which is explicitly handling the null case to return an empty list, so exists(null) would return false. I agree thought that exists(false) would return true, which may be a surprising result. Can you give some examples of where you were thinking it would have unintended consequences though?

During testing, one of our QA people created a child library that did something like the following:
library child version '1.0.000'

using QDM version '5.0.2'

context Patient

define "hasEncounter":
    exists(["Encounter, Performed"])

They then used the child library in a parent library:
library parent version '1.0.000'

include child version '1.0.000' called ChildAlias

context Patient

using QDM version '5.0.2'

define "Initial Population":
    exists(ChildAlias.hasEncounter)

We expected this would return an error, but it did not. In this particular case, the author might think that this would only return true if the patient had an Encounter, but it would always return true and the patient would be included in the population even though the patient did not have an Encounter. If we allow this through list promotion, then we will have to be extra specially careful that we catch something like this during a review process.

I agree that's a potential issue. Strictly speaking, it is correct though. Is this a good case for a warning? It would be easy to add a warning when a Boolean-valued expression is promoted to a list.

I think it would be a good use case for a warning message.


gpssim: populate s_variance_m_s with 0.25

Describe problem solved by the proposed pull request
Solves https://github.com/PX4/sitl_gazebo/issues/285
gps->s_variance_m_s was published as zero in SITL which is bad since the EKF is using it at several places e.g. to initialize the uncertainty in the velocity states
Plots:
PR (left) vs Master (right)

@CarlOlsson the issue you reference has nothing to do with the internal GPS sim driver.
Also, would make much sense to compute the EPH and EPV from HDOP and VDOP.

the issue you reference has nothing to do with the internal GPS sim driver.

Yeah I noticed that, thought it was coming from gazebo

Also, would make much sense to compute the EPH and EPV from HDOP and VDOP.

I guess if it is possible, how do you propose to do that?

How do you calculate variances from the dilution of precision measure?

How do you calculate variances from the dilution of precision measure?

Using an acceptable value of User Equivalent Range Error (UERE).

This seems fine as an incremental improvement until we have a better solution.


Method createSignoutRequest not in the DOC

Usefull method createSignoutRequest is not explicit into docs

Is it possible to submit PRs for Wiki updates? Sorry for my newbie question but I may be able to help here now that I'm trying to understand how createSignoutRequest works 😎

The wiki doesn't allow that, IIRC, but feel free to open an issue with the proposed text/changes.

@brockallen How are your feelings to add JSDOC comments via pull requests to the most common functions of the library? This way it's self documented and most IDEs can pick it up for the intellisense.
I understand that it will add a burden to maintain those comments from that point on, but maybe that's something that can be outsourced to the community as well.
Otherwise I'll create an issue with the text as soon as I get some spare minutes.

JSDOC -- I'm not familiar with that.

Not sure how that's called in other languages. There's one for Java also called Javadoc. I believe in C# they're just called XML comments.
Example:
  /**
   * Implements the Fisher-Yates Shuffle algorithm that returns an array of randomized elements ensuring no duplicates
   * @param array The array to shuffle
   * @author https://stackoverflow.com/a/2450976/5866637
   */
  private shuffle(array: number[]): number[] {
    const cpArray = [...array];
// ...

It will later look like this on IDEs:

And can also generate an automatic documentation website with some software like:
https://github.com/jsdoc/jsdoc
Of course, it will require that all the public api for the library to be annotated in this way for the current and future versions, that's why I have asked before doing any PR as I don't know if I will be here tomorrow to keep contributing and may become a burden for you or your other contributors.
It's also not fully standarized, hopefully it shouldn't become another Gif-Jif discussion but you know how the internet works.


Cannot find tsserver executab , but tserver is available

Directory:
HY469-fullstack-React/frontend on  main [!?] via  v20.8.0 via  impure (nix-shell-env)
❯ ls
src             package.json        tsconfig.json       public             index.html
flake.nix       Dockerfile          tsconfig.node.json  package-lock.json  LICENSE
vite.config.ts  tailwind.config.js  postcss.config.js   flake.lock         README.md

Ofcoure , i will try later to provide one.
For now , installing localy , using npm i , having already set up typescript solves this.

Am also facing this problem, and i don't want to install locally and am also not using nix and i have set the path tsserver_path but it is still not working, i noticed that if i run zsh and then open the file it would work.


[Core] Simplifications to executor classes

Move the common executor field-setting __init__ logic to the ExecutorBase class.
Add missing tensorizer_config arg to ExecutorBase.__init__()
Fix incorrect return type of executor list_loras() method
Have check_health_async() implementation default to call check_health()

@zhuohan123 these are minor fixes/simplifications that I've pulled out of https://github.com/vllm-project/vllm/pull/3466 which aren't directly related to the multiprocessing. Having this merged would help reduce the continual rebasing pain with that one! :pray: (unless there are any of these changes that you don't like of course, which is fine)


NullPointerException on any use

Hey, I'm getting a NullPointerException whenever I try to use the compiler. I tried it with your example:
StringBuffer sourceCode = new StringBuffer();
sourceCode.append("package org.mdkt;\n");
sourceCode.append("public class HelloClass {\n");
sourceCode.append("   public String hello() { return \"hello\"; }");
sourceCode.append("}");

try{
        Class<?> helloClass = InMemoryJavaCompiler.compile("org.mdkt.HelloClass", sourceCode.toString()); // <--- Line 16
    }catch(Exception e){
        e.printStackTrace();
    }

And it doesn't work. The exception:
java.lang.NullPointerException
    at org.mdkt.compiler.InMemoryJavaCompiler.compile(InMemoryJavaCompiler.java:19)
    at me.nonamesldev.javacompilerexample.JavaCompilerExample.main(JavaCompilerExample.java:16)

I tried it with different source codes, all don't work.

After a little debugging, I found out that in the InMemoryJavaCompiler class, javac is null for a reason - I didn't have JDK installed. Silly me :P Add a note in the readme.md that you need JDK tho :)

I have installed JDK, even I am getting NullPointerException.When I debug I got javac is null.
I configure the application in eclipse.
static JavaCompiler javac = ToolProvider.getSystemJavaCompiler();
is not able to create instance of javac
If any idea please provide me.

Related to https://github.com/trung/InMemoryJavaCompiler/issues/12


Active Support dependency warning when using Ruby 2.2.0

I just started a new project with padrino-0.12.4 and ruby-2.2.0 and I am receiving this warning:
active_support/values/time_zone.rb:270: warning: circular argument reference - now
I looked into it and the issue is identified here (since padrino-support depends on active-support 3.2.*).
Will future releases bump the active support dependency?

padrino-support depends on activesupport >= 3.1. This should allow you to upgrade to 4.* without problems. There probably will be no bump in active-support requirement any time soon.

Sorry, you're right. I had this problem because I created the app through the project generator with the Mongoid ORM, that also depends on active support. For some reason running bundler installed Mongoid 3.x, that locked active support on the version I mentioned.

We could lock the generator on the latest mongoid if it's working.
On Tuesday, February 3, 2015, quimrstorres<EMAIL_ADDRESS>wrote:

Sorry, you're right. I had this problem because I created the app through
the project generator with the Mongoid ORM, that also depends on active
support. For some reason running bundler installed Mongoid 3.x, that locked
active support on the version I mentioned.
—
Reply to this email directly or view it on GitHub
https://github.com/padrino/padrino-framework/issues/1876#issuecomment-72647196
.

Let me bump the mongoid version and I'll get back to you.


support brpc baidu-std protocol

@smwyzi It would be useful to validate the trpc codec if we can add testing scripts in the test directory.
Are you planning to add the test in another PR?

@smwyzi It would be useful to validate the trpc codec if we can add testing scripts in the test directory.
Are you planning to add the test in another PR?

yes, i will add test case soon


splitpane_divider color

Hello,
I would like to suggest a different color for the split panes. The borders are now almost invisible. I've chosen for the foregroundColor als color, but it can definitely be another one.
My suggestion would be to add the following to the config:
// Config
exports.decorateConfig = config => {
    return Object.assign({}, config, {
        ...,
        css: `
            ${config.css || ''}
            ::selection {
                background: #9198A2 !important;
            }
            .splitpane_divider {
                background-color: ${foregroundColor};
            }
        `
    });
};

Thanks for your suggestion.
I feel like this is more of a border issue considering it'll look odd if the border and divider don't match. So I just pushed a version with a lighter border.
Hope that works for you ✌

Do you mean this commit https://github.com/henrikdahl/hyper-chesterish/commit/0c3188ec45117edc0e7eda9837859365ad0e30cd ?
That actually made the border even darker not lighter.

@joggienl
Commit message is wrong, the color is indeed lighter, you can compare them here
Left is old color, right is new one ✌

I see indeed, I did not compare the actual colors itself ;-)
For me the border and thus divider color still doesn't stand out enough. I do notice it little better on my retina screen, but on my non-retina screen it still is almost invincible.
The border color goes for windows/tabs and the split divider (as far as I know). For the window and tab this goes fine, it's a subtle difference which makes it look pretty. The border in this instance is not something you need to interact with (no dragging or so with the border).
The divider on the other hand is actually actionable. Thats why I made it way more white in my PR proposal: you need to be able to interact with it. Which is hard if you can't see it.
So, in short, do you see possibilities to make it even more lighter?
Aside from that it might also be nice to extend the example images with an image with some split panes in.

Well, to be honest, this "narrow contrast color palette" is what makes me want to use this scheme so much :-)
Thanks in advance, curious to see the results!

Hey hello!
Curious, have you had any time for this yet?
I was also thinking what might be good addition for this: make the active pane "jump" out (or give the inactive ones a different color...
Not sure if that will fit in the theme, but it could be a nice addition..


Printers are not found (Bluetooth printer not discoverable in android)

@tr3v3r  does this package support HOP-H58 ? i am not able to found printers or any status

SAME ISSUE

Sorry guys the lib supports only printers produced by EPSON

@tr3v3r do we have any lib that support all type of Thermal receipt printers ?

https://github.com/AllInOneYT/react-native-thermal-printer try this one


ASIM-SchemaDataTester-NewValidation

Required items, please complete
Change(s):

New Validation for ASIM Data Tester

Reason for Change(s):

New Validation for ASIM Data Tester

Version Updated:

NA

Testing Completed:

Yes

Checked that the validations are passing and have addressed any issues that are present:

Yes

Hi @anki-narravula @mkchiliveri , please help reviewing this new workflow. This workflow will be triggered only when ASIM Parser YAML files are added or modified under 'Parsers' folder. This workflow is added to help automating the ASIM Parsers PR review process.


bootstrap.sh fails when installing apache-airflow 2.0.2

To reproduce:
1.) run ./mwaa-local-env build-image
Is there any other information that would help debug this issue?

It works now.
The trick was I removed the constraints for prison and flask-openid from the constraints.txt file. But I'm sure this is not the right way to fix this. Curious to hear others' thoughts.

https://github.com/aws/aws-mwaa-local-runner/issues/78#issuecomment-1005224068


null is rendered at the first line in JS when header field is left blank

When I enter something like // in the header field, it is reflected on the first line.
Might relate to #14

Hey @jepsar can you please be more specific and maybe give a code sample? Will check it. Thx. I moved from YUICompressor to googl closure compiler. Maybe it will be fixed due to the switch but, to less information for that.

Test code:
// Test.

Output:
'use strict';

So I couldn't see any problems here, only that it will always added use strict, but this will be another issue. So it seems to be fixed in 2.4.0.


Add reversible encoding tests

Adding integration tests for reversible encoding and fix the pending alarms' test.

Looking into the failing build...


Not Able To Connect To Cluster

Describe the bug
Lens was working for me for awhile until I deleted everything and started over. Not sure why it is stuck with the Connecting... message. I am able to run kubectl get pods and other kubectl commands
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See error

Expected behavior
Should be able to connect or better error message
Screenshots

Environment (please complete the following information):

Lens Version: 3.5.2
OS: 10.15.6 (19G73)
Installation method (e.g. snap or AppImage in Linux): Mac OS

Kubeconfig:
apiVersion: v1
clusters:
- cluster:
    server: https://xxxxxx.sk.us-east-1.eks.amazonaws.com
    certificate-authority-data: xxxxx
  name: kubernetes
contexts:
- context:
    cluster: kubernetes
    user: aws
  name: aws
current-context: aws
kind: Config
preferences: {}
users:
- name: aws
  user:
    exec:
      apiVersion: client.authentication.k8s.io/v1alpha1
      command: aws-iam-authenticator
      args:
        - "token"
        - "-i"
        - "terraform-eks-demo"

I am a bit confused. If I launch it via the CLI, it works. Here are the logs. Nothing too useful
debug: Set kubectl version 1.17.4 for cluster version 1.17.4 using version map
info: Checking for update
debug: getFreePort() start
debug: getNextAvailablePort() start
debug: *** server listening event ***
debug: got port : 51190
debug: Creating temporary kubeconfig: /var/folders/9w/cxpyh7d96yq7jy21lvjkj5q17jlpvt/T//pl04tlkze3b-h1qsqqbvifw-kubeconfig
debug: Starting to load target definitions for aws
debug: Creating temporary kubeconfig: /var/folders/9w/cxpyh7d96yq7jy21lvjkj5q17jlpvt/T//563ielcl8je-rcv6j4ok3n-kubeconfig
debug: Created cluster[id: d8b1bdca-dcdb-469a-b27b-caa7ac0fb227] for context aws
debug: clusters after constructor:1
info: Update for version 3.5.2 is not available (latest version: 3.5.2, downgrade is disallowed).
debug: checkForUpdatesAndNotify called, downloadPromise is null
debug: IPC: preferencesSaved
debug: IPC: getClusters, workspace default
debug: IPC: getCluster
debug: Creating temporary kubeconfig: /var/folders/9w/cxpyh7d96yq7jy21lvjkj5q17jlpvt/T//k4bq1seb1md-5qr4hvjkz1h-kubeconfig
debug: getFreePort() start
debug: getNextAvailablePort() start
debug: *** server listening event ***
debug: got port : 51198
debug: Acquired a lock for 1.17.4
debug: Releasing lock for 1.17.4
debug: proxy aws stdout: Authentication proxy started

debug: features for aws
debug: feature metrics
debug: getting feature status...
debug: feature user-mode
debug: getting feature status...
debug: getFeatures resolving with features: {"metrics":{"currentVersion":null,"installed":false,"latestVersion":"v2.17.2-lens1","canUpgrade":false},"user-mode":{"currentVersion":null,"installed":false,"latestVersion":"v2.0.0","canUpgrade":false}}
debug: Set kubectl version 1.14.10 for cluster version v1.14.9-eks-f459c0 using version map
debug: Acquired a lock for 1.14.10
debug: checking pod default/loki-chart-2
debug: Local kubectl is version 1.14.10
debug: Releasing lock for 1.14.10
debug: checking pod default/memcached-744df86556-lvnkv
warn: PrometheusLens: failed to list services: services "prometheus" not found
warn: PrometheusLens: failed to list services: services "prometheus" not found
warn: PrometheusLens: failed to list services: services "prometheus" not found
warn: PrometheusLens: failed to list services: services "prometheus" not found
info: using lens as prometheus provider
info: using lens as prometheus provider

Attached is a screenshot of 2 lens side by side. The left one is the one opened via CLI and the other one not through CLI.

@brianpham my guess is that this command: aws-iam-authenticator command is only in PATH within a terminal environment. It's good to remember that "launcher" does not have exactly the same environment variables set vs terminal. It should work if you set full path to aws-iam-authenticator binary.

@jakolehm Hmm same issue even after setting the env variable for the launcher. I just added the following to my kube config hoping that it would help, but it didn't. Any other suggestions I could try?
      - name: "AWS_PROFILE"
        value: "dev"

Not sure what happened, but it started working today. Going to close this out, since I think its a one off. I been launching it from the CLI since I know it works.

@brianpham How do you start it via the terminal? I am experiencing the same issue.

@arditshala On Mac, I ran the following ./Applications/Lens.app/Contents/MacOS/Lens --debug


Suggestion: Palm Detection Mediapipe jS api

Firstly, kudos on the jS API! Its enabled a lot of us to get higher perf than TFjS
I wanted to suggest also releasing a Palm Detector for web, as this would be greatly beneficial due to possible speed and otherwise increases in performance.
This shouldn't be too hard as its just removing a calculator from the hands version.

yes please !

@mgyong can this be done?

@chuoling or @tyrmullen ?

@djthegr8 Sorry for the late response. We've added palm-detection solution APIs as a feature request, and will update this issue with progress.

Amazing!

uhh @sgowroji could we keep this opened till its been added?

Has there been any movement on this particular issue?

@mgyong @chuoling could you update here?

Hi @djthegr8 ,
Plam detection is available in the latest MediaPipe version , please verify and close this issue.

Great!


Fix generating null return type from reflection in TypeGenerator

The TypeGenerator failed with "type 'null' cannot be nullable" when fromReflectionType() was called for a method with a return type "null" due to a incorrect comparison with the string "null". This change corrects the comparison so that such methods are reflected without errors.
Obvious fix.
Resolves: #185

Thank you @robertlemke :)


New rule to check if @Preview (Jetpack Compose) functions end with 'Preview' suffix and are also private

This is a common issue with Jetpack Compose and would be really convenient. Actually, this should be a resposnibility for android lint I guess but maybe you see potential here.
Description:
In Jetpack Compose it is common to give your @Preview functions a 'Preview' suffix for example
@Preview
@Composable
private fun BannerPreview() {}

It's not so common to make them private but it should be. I had a discussion on slack, check https://kotlinlang.slack.com/archives/CJLTWPH7S/p1652276247920509
Sooner or later this will be adopted by android. If you think this is a good idea for 2 new rules, I'm glad to see it in a future version :)

Good rules for the future. Will be added after 1.2.0


Upgrade Ember + deps

Updates Ember & friends to 2.7.0
Sets up Ember Try to run all scenarios in one run
Updates the scenarios Ember Try runs, including Ember 2.9.0-alpha
Fixes a bug with Ember 2.9.0-alpha
Tests with the Chrome browser

Awesome, thanks!


🐛 Variable und unreliable power consumption and power settings

branch
unstable
Architecture
x86_64
Description
The power consumption on my laptop heavily fluctuates between 8W and 32W, with the same top-level applications running.
Sometimes, when I check powertop, it shows a ~32W power consumption, with all "Tunables" showing "Bad" (and, for example, "Integrated camera" on and consuming lots of energy despite it not being used).
Sometimes, it shows ~8W power consumption, with all "Tunables" showing "Good" (and "Integrated camera" off, as expected).
How's power managed in manjaro-sway? And any pointers as to why this is not reliably working? This is super annoying since it makes a world of difference (sometimes 5h) in terms of battery life. Thank you!
Reproduction steps
See above

Screenshots
With laptop in a "bad" state:

Logs
No response

Afaik there are no exotic things manjaro adds on top of arch. We have laptop-mode in addition. That you can disable or configure to your needs: https://wiki.archlinux.org/title/Laptop_Mode_Tools


Removed deprecated PropTypes

React version 15.5.0 add deprecated warning for React.PropTypes.
More info on React blog.

Thanks, @kubens!

Any chance of this getting merged?

ping @xavierelopez

Hey @SpainTrain, unfortunately I don't maintain this repo anymore and I don't have the needed permissions. Maybe @lindslev does?

ping @lindslev
React 16 was released, so this is now a roadbloack to upgrading to latest stable react version.

Just wonder if anybody still has access to this repo or if it's officially dead? As mentioned before this PR is a showstopper for upgrading to React 16.


I moved some MCscripts files

MCscripts has a new directory named /opt/MCscripts. I moved Bash script SH files, the backup_dir symlink, and the .mcbe_log directory to there. I did this because /opt/MCscripts is owned by root and read only for the mc user. It's good security practice to limit the mc user's permissions on script files root executes and config files for those scripts. If you have any questions let me know. Sorry for the inconvenience.
https://github.com/TapeWerm/MCscripts/compare/118e191...8460724 shows the user facing changes.

Thanks for letting me know!
I'll deal with changes.


Remove superfluous "the" from README.md

Coverage remained the same at 86.514% when pulling b1863bdedfac65a0533b52b6212b193c92b753c7 on jonathanp:patch-1 into cdcc1cbf89e0a46634fe4aaeba4793bfe2fdf9b2 on reactjs:master.

Thanks @jonathanp. 👍


Add support for the multiple attribute

If you need to upload several files at once, this patch lets you do that.
Fixes #2

This looked a bit shifty I must admit. Closed the PR for now, perhaps I'll create a new one in the future :)


Create a pull request on Neurohack repo

Pull request on Neurohack repo with project title, group members, relevant links, and a project description.
Link to repo: https://github.com/neuroHackademy2024/projects

This has now been added to the neurohackademy project description READme file


Implement If-Modified-Since when downloading images

Our HTTP client currently downloads the image specified on source even when the image in the server is the same as a local copy previously downloaded.
For ths case where a local copy is already in the pool, we could perform the HTTP request with the If-Modified-Since header (using the modification time of the local image), so it would be downloaded only if the remote file has been updated.

@inercia good idea


Answer for challenge Target the Parent of an Element Using jQuery not accepted

Challenge http://www.freecodecamp.com/challenges/waypoint-target-the-parent-of-an-element-using-jquery has an issue.
I believe that the code that i write is correct but the system doesn't accept it.
Issue need's to be fixed.

I also tried $("#target1").parent().css("background-color", "red"); as an answer...


add methods to retrieve and set baseline

just simple get and set the baseline value

@coding-guy Hi, do you have an example of code using read and write baseline?

Hi is there any progress in this implementation? I would like to save and restore baseline for CSS811.

CI-passing code was recently merged, check it out!


patch version bump to trigger new ci/cd

Codecov Report

Merging #199 into master will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #199      +/-   ##
==========================================
- Coverage   78.42%   78.36%   -0.06%
==========================================
  Files          11       11
  Lines         774      772       -2
  Branches      159      159
==========================================
- Hits          607      605       -2
  Misses        138      138
  Partials       29       29

Impacted Files
Coverage Δ

kmapper/_version.py
100.00% <ø> (ø)

kmapper/utils.py
90.00% <0.00%> (-0.48%)
:arrow_down:

kmapper/plotlyviz.py
43.87% <0.00%> (-0.36%)
:arrow_down:

Continue to review full report at Codecov.

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


running nixos-rebuild switch in fresh virtualbox image bricks the vm by removing demo user

Issue description
Running nixos-rebuild switch in a fresh VirtualBox appliance removes the demo user and logs you out, and there is no way to log in.
Apparently, the virtualbox image's active configuration includes https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/profiles/demo.nix via https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/installer/virtualbox-demo.nix but it is not present in /etc/nixos/configuration.nix.
It is unexpected that the active configuration does not match /etc/nixos/configuration.nix. Worse is that rebuilding bricks the vm.
Suggested solution: add profiles/demo.nix to /etc/nixos/configuration.nix in the image. Not sure if nixos-generate-config is involved. Also, run nixos-rebuild switch in the vm before shipping, so that running it on a freshly downloaded image does nothing.
Steps to reproduce

Download VirtualBox appliance, in my case:
http://releases.nixos.org/nixos/16.03/nixos-16.03.616.4ac4b35/nixos-16.03.616.4ac4b35-x86_64-linux.ova
Import appliance in virtualbox (I used default settings) and start it
log in with user: demo, password: demo
run sudo nixos-rebuild switch
When it finishes, you are logged out, the demo user is removed, and there is no way to log in.

Technical details

System: NixOS 16.03.616.4ac4b35 (Emu)
Nix version: 1.11.2
Nixpkgs version: 16.03.616.4ac4b35

The error is not fixed in nixos-16.03.714.69420c5-x86_64-linux
Very annoying if you just want to evaluate NixOS.

Demo user is still removed in 16.03.1162

Looks like this is fixed on master in https://github.com/NixOS/nixpkgs/commit/58db7f508f815ac7c1e12e3abd8f7d0fcf25a0cb . I'd like confirmation but I don't know how to build the image. Can someone help me out with a command?
Thanks

Duplicate of #13927
Fixed by 58db7f5
Image can be created with nix-build '<nixpkgs/nixos/release.nix>' -A ova
Pls reopen if it turns out I'm wrong.

This is still an issue in the latest ova (nixos-16.03.1299.a8e0739).

it's fixed in master and 16.09 but not 16.03.

Yeah - so the issue is that if you download the 16.03 ova, then upgrade to unstable, you're still toast.


Globe not rotating with world-50m.json

Hi there,
I'm trying to make a globe that rotates and zooms in on a country when it is searched by the user.
However, I'm running into a slight problem.
I want to use the "world-50m.json" file as it gives a geographies name and iso3 - but when used for some reason the globe will not rotate.
The globe rotates when I use the "world-110m.json" file but only provides an ID of the geography and not the geographies name or iso3.
Is there any way the globe is able to rotate using the "world-50m.json" file, or alternatively is there a way I can modify the "world-110m.json" file to include each geography's respective name and iso3.
Thank you :)

Never-mind, the world-110m.json used in the globe example, is different from the world-110m.json file in the topojson-maps folder. The file in the topojson-maps folder has the data i need.


about crop video

hi AliaksandrSiarohin,
so awesome project, congradulations for you. about cropping video, I found that it is a cropping method for offline videos(crop box computed by all frames), If I want to crop the real-time video frame in the camera，Is there a more reasonable way to crop?

@newExplore-hash did you able to get cropping in real-time?

yeah, I want to drive a picture in real time through the camera，but I can't seem to find a suitable crop method.

@newExplore-hash  this link can help.

actually in the old demo model, somebody was able to solve this issue but I am also finding it difficult to crop the video too, and I believe it should be possible to do this automatically at upload time


Avoid floating-point issues (decimal module)

We store very small float values when the users enters something like 3.14e-30. Avoid this and use an exact representation by making use of the decimal module.
Also see this test case.

Let users specify precision of the module as config flag.

Problem right now:
>>> from decimal import *
>>> Decimal(3.14)
Decimal('3.140000000000000124344978758017532527446746826171875')
>>> Decimal('3.14')
Decimal('3.14')

If we want to allow Decimals in the API, then how can users denote that they mean an exact value in the sense of: "should not be rounded" vs. "exact" in terms of floating-point precision?

The problem I saw there, is actually not a problem. If the user passes in a string, we know we treat this value as exact (i.e. don't round the value). If the user passes in anything else (Decimal, int, float), we know the value is inexact, i.e. we will round it. This is the case no matter what calling API is used for Decimal, e.g. Decimal(3.14) as well as Decimal('3.14') will be treated as inexact values that we will round.
Maybe we can clarify in the docs that the mantra "passing in as string means you want the value to not be rounded" only holds true if one passes in a value directly as string. And not if one passes the value as string to the Decimal constructor (this would be a non-direct way).


Save and load project does not combine

When I save a project it is saved, but when I go to the folder with the load option the saved project does not show up in the list.

Works absolutely fine on my PC

@martijn9612  can you reproduce this bug?

I suppose that this is a thing that linux does, so we will not fix this. We have made a way to work around this by implementing a filter for all files in #87 .

I suppose that this is a thing that linux does, so we will not fix this. We have made a way to work around this by implementing a filter for all files in #87 .


Link to source code has typo

Quick note: the link to the source code ends with 'Program.css' instead of 'Program.cs', causing a Page not found error.

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

ID: 0975f25d-b20d-e4df-5ac6-055b7c4ad5b9
Version Independent ID: 6b164c45-ca36-b4a5-1b3c-40df791af67e
Content: Quickstart: QnA Maker client library for .NET - Azure Cognitive Services
Content Source: articles/cognitive-services/QnAMaker/Quickstarts/create-publish-kb-csharp-sdk.md
Service: cognitive-services
Sub-service: qna-maker
GitHub Login: @diberry
Microsoft Alias: diberry

Thanks for the feedback! We are currently investigating and will update you shortly.

@tarockey Thank you for catching this! I have created a PR to fix and it should be live soon. ^^


[Backport v2.1] net: coap: Fix possible overflow

Fix possible integer overflow when parsing a CoAP packet.

@ceolin this PR is against master, is that intended?

Hi @carlescufi  nope, should be 2.1. Will fix it.

Hey @carlescufi, could you take a look again ?


Implement feature: entries

Returns a new Sparray Iterator object that contains the key/value pairs for each index in the sparray.

done


Project not working with Android Studio 2.3 or even 3.0

Thank you for the much needed indoor solution for Android. However, I wasn't able to run it on neither Android Studio 2.3 nor 3.0.
It says that the project "Does not use the Gradle build system. I tried using the import wizard but to no avail.
So, what shall I do to get this project running?

Hello @alaabualkheir
make sure you are using example app folder and not the main project folder (the one with README and CHANGELOG). It is obvious that it doesn't contain build.gradle file, so your Android studio can't recognize it as a Gradle project. Simply go to the example app subfolder (the one where build.gradle file is) and launch the project.
Hope it helps :)
Cheers!

Thanks @pawelDylag , that did help and now the code is working.


Add a way for dot spells to be connected to their impact spells

Makes it possible to use the multidot action with dot spells that have their dot component as separate

@Tharre I dont know why warlock tests got changed. Maybe you can spot something but from what code I changed I did not expect any of the test results to change. Shaman and Mage both show same results as before this change but yet warlock does not. I wont merge this PR until we figure out what causes it.

Thanks for doing this, looks good from a Shaman PoV! Confirmed it works as expected in the Multi Dot action which was my original reason for asking.
One relatively minor thing is that now both the Direct and DoT spells are shown in in "DoT Spell" APL fields, e.g. DoT Remaining Time:

Selecting either one works fine, since they now both refer to the same DoT. Not sure which one makes more sense to keep.

@raethkcj fixed the dot menus to show one of the 2 spells that have the dot linked.


fix(vue-app): handle layout on client error

Types of changes

[x] Bug fix (a non-breaking change which fixes an issue)
[ ] New feature (a non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Description

Resolves https://github.com/nuxt/nuxt.js/issues/7567#issuecomment-651683297
Checklist:

[ ] All new and existing tests are passing.

Codecov Report

Merging #7654 into dev will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #7654   +/-   ##
=======================================
  Coverage   70.18%   70.18%
=======================================
  Files          88       88
  Lines        3756     3756
  Branches     1020     1020
=======================================
  Hits         2636     2636
  Misses        911      911
  Partials      209      209

Flag
Coverage Δ

#unittests
70.18% <ø> (ø)

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 2302ffa...c2c048c. Read the comment docs.


ServiceMeshMemberRoll stops updating

I installed the OpenShift mesh operator, v1.1.0 on an openshift 4.3.9 cluster. Initially everything seemed to work and I got the bookinfo demo working as well as a project in a second namespace. But then the memberroll stopped updating. I ended up with:
apiVersion: maistra.io/v1
kind: ServiceMeshMemberRoll
metadata:
  annotations:
    kubectl.kubernetes.io/last-applied-configuration: >
      {"apiVersion":"maistra.io/v1","kind":"ServiceMeshMemberRoll","metadata":{"annotations":{},"finalizers":["maistra.io/istio-operator"],"name":"default","namespace":"istio-system"},"spec":{"members":["seldon","bookinfo","seldon-system"]}}
  selfLink: /apis/maistra.io/v1/namespaces/istio-system/servicemeshmemberrolls/default
  resourceVersion: '8137725'
  name: default
  uid: 371391c3-07ba-4aad-b308-5c034062e688
  creationTimestamp: '2020-04-08T16:17:37Z'
  generation: 12
  namespace: istio-system
  ownerReferences:
    - apiVersion: maistra.io/v1
      kind: ServiceMeshControlPlane
      name: basic-install
      uid: f80bb366-f5f5-4c36-b437-88908082ed7f
  finalizers:
    - maistra.io/istio-operator
spec:
  members:
    - bookinfo
    - seldon
    - seldon-system
status:
  annotations:
    configuredMemberCount: 2/2
  conditions:
    - lastTransitionTime: '2020-04-09T06:47:11Z'
      message: All namespaces have been configured successfully
      reason: Configured
      status: 'True'
      type: Ready
  configuredMembers:
    - bookinfo
    - seldon
  meshGeneration: 1
  meshReconciledVersion: 1.1.0-9.el8-1
  observedGeneration: 4

That lastTransitionTime is two weeks ago. Even if I remove bookinfo as a member, it still remains in the configuredMembers and I can still access it through the istio ingress. Adding new members into the member roll does not get them into currentMembers and those projects don't become accessible through ingress. I've tried restarting all the pods in the istio-system namespace but that doesn't bring things to life either. Seems to get stuck somehow.
I then uninstalled the operator and reinstalled it and it worked. Good that it worked but I don't think I should have had to reinstall the operator.

@ryandawsonuk thanks for reporting the problem. Any chance you can provide the logs of the Maistra operator?
Also note that bug reports should be filed in our Jira at https://issues.redhat.com/MAISTRA.

Is that the istio-operator pod in the openshift-operators namespace? Unfortunately I reinstalled it so I now don't have those logs. I guess I'll need to close this and open in jira if I recreate it.


Grammar and style review edits for Developing-LibertyDevTools.md

Edits for this issue: https://github.com/eclipse/codewind/issues/519

Looks great! Merging.


Refresh elasticsearch nodes periodically

The first two commits were merged in release 320, but I forgot to push the last two commits


PostgreSQL 9.5 support.

Signature of PostgreSQL' pglz_compress and pglz_decompress
change in PostgreSQL version 9.5. This commit introduces
new functions pglz_cstore_compress and pglz_cstore_decompress
to accomodate 9.5 changes.

this no longer compiles in pg9.5 existing work on another branch is updated.


Language getting automatically set to English while signing in

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:

Your language should be set to Spanish
Go to signin page
Enter any email address or phone number
Try to go back to the signin page by clicking on the link below
Language will be set to English

Expected Result:
Language should not change
Actual Result:

Workaround:
No, Spanish Users might find it hard to navigate.
Platform:

Where is this issue occurring?

Web
iOS
Android
Desktop App
Mobile Web

Version Number:
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Expensify/Expensify Issue URL:
View all open jobs on Upwork

Triggered auto assignment to @sonialiap (AutoAssignerTriage), see https://stackoverflow.com/c/expensify/questions/4749 for more details.

Proposal
We need to preserve the locale inside restartSignin function
I can do this by connecting to Onyx for current locale and use this block of code inside restartSignin function
const preferredLocale = currentPreferredLocale;
Onyx.clear().then(() => {
if (preferredLocale) {
Onyx.set(ONYXKEYS.NVP_PREFERRED_LOCALE, preferredLocale);
}
});

Triggered auto assignment to @deetergp (Engineering), see https://stackoverflow.com/c/expensify/questions/4319 for more details.

Triggered auto assignment to @arielgreen (External), see https://stackoverflow.com/c/expensify/questions/8582 for more details.

Definitely something a external contributor can do, and @thesahindia's proposal sounds good.

https://www.upwork.com/jobs/~01f7d2db5bd805d8ca

Triggered auto assignment to @tgolen (Exported), see https://stackoverflow.com/c/expensify/questions/7972 for more details.

@tgolen can you review @thesahindia's proposal?
@thesahindia please also be sure to apply to the Upwork job here.

@arielgreen Submitted the proposal :smile:

I would like to request some changes to your proposal to increase the quality of the code.
I would suggest refactoring this code slightly so that there is only one method in the entire codebase in actions/Session.js which calls Onyx.clear() and it would be a place to handle all of the "need to keep this after clearing Onyx". We are currently only doing this properly in redirectToSignIn() by saving the local and the active clients key.

@tgolen
Hi, I want to know if it'd be a bad idea to just use redirectToSignIn instead of restartSignin since redirectToSignIn is handling this situation in a better way.
https://github.com/Expensify/App/blob/69ee2fe21bf825318a9e717096293aedc51bda4a/src/libs/actions/SignInRedirect.js#L28-L34
Here it says that If we move this function to Session.js then it can cause circular dependency. So, maybe moving the logic to Session.js is not a good idea. Let me know your thoughts.

Yeah, I'd be fine with getting rid of restartSignin completely. Good suggestion! I understand about the circular dependency. It is OK to stay in its own file like that.
With that then, I think it's a complete proposal and I give the 🟢 to be hired for this @arielgreen.

@thesahindia just sent over an offer! I also raised the price to include the bonus you'll receive for proposing AND fixing this.

Thanks @arielgreen, I have accepted the offer!

@tgolen
I have one last query about the use of currentURL inside src/libs/actions/SignInRedirect.js  .
We are using this at only place which is here.
https://github.com/Expensify/App/blob/e5957e1ffff7e111a459b05643bac18ffab20619/src/libs/actions/SignInRedirect.js#L41-L43
And then there's no use case of it.
However, on the login screen the currentURL will not be set to any value and will fail this condition.
We can fix this by giving it a default value of  "/" or we can just remove currentURL from the file as I don't see any other use case of it.
Let me know what should be done here. Thanks!

It's entirely possible that currentURL can be removed completely. It was added in this PR to fix this GH. I think we need to be a little bit careful here because exitTo is still used in the VALIDATE_CODE_URL but it's used as a query parameter now as opposed to be part of the path. However, the VALIDATE_CODE_URL isn't used to construct a "new expensify" URL, it's used to construct a URL pointing to expensify.com.
So, I still think we are pretty safe to remove currentURL from this file, but you might want to double-check my investigation.

In fact, I really don't like that VALIDATE_CODE_URL lives in ROUTES.js since it isn't related to a route for New Expensify. Since it's only referenced once, I would just move the logic from ROUTES.js into App.js where it was being referenced. Would you mind cleaning that up?

Sure, openSignedInLink will look like this after the clean-up.
function openSignedInLink(url = '') {
     API.GetAccountValidateCode().then((response) => {
     const exitToURL = url ? `exitTo=${url}` : '';
     const validateCodeUrl = `v/${currentUserAccountID}/${response.validateCode}${exitToURL}`;
     Linking.openURL(CONFIG.EXPENSIFY.URL_EXPENSIFY_COM + validateCodeUrl);
     });
}

Sure, openSignedInLink will look like this after the clean-up.
function openSignedInLink(url = '') {
     API.GetAccountValidateCode().then((response) => {
     const exitToURL = url ? `exitTo=${url}` : '';
     const validateCodeUrl = `v/${currentUserAccountID}/${response.validateCode}${exitToURL}`;
     Linking.openURL(CONFIG.EXPENSIFY.URL_EXPENSIFY_COM + validateCodeUrl);
     });
}

Beware, This will not work in Safari Mobile. Linking.openURL does not work here

Oh, if this can break something then I think it's better to just leave it as it is?

I don't think we need to worry about Linking.openURL() in this issue. It was already there previously, so nothing is breaking worse than it was already broken before. @parasharrajat Is there an open issue yet for fixing openURL() for mobile safari? If not, let's get one created.

Found this issue while testing the code.
For some reason, there's a delay in Android (but Web looks fine)
Android :
https://user-images.githubusercontent.com/83179295/130642600-37baafb9-779a-402c-afb7-e6a6c24414d7.mp4
Web :
https://user-images.githubusercontent.com/83179295/130502663-2776d3dc-4b0b-4d83-a864-0f68021ac4d5.mp4
Not sure but it could be related to this issue https://github.com/Expensify/App/issues/4513

It is not related to #4513

@thesahindia can you please provide an update here?

The PR is up. Just waiting for a reply here

:hand: This PR was not deployed to staging yet because QA is ongoing. It will be automatically deployed to staging after the next production release.

🚀 Deployed to staging by @tgolen in version: 1.0.93-2 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

🚀 Deployed to production by @yuwenmemon in version: 1.0.94-1 🚀

platform
result

🤖 android 🤖
success ✅

🖥 desktop 🖥
success ✅

🍎 iOS 🍎
success ✅

🕸 web 🕸
success ✅

It's OK melvin. It's not overdue

Paid.


Paths fail when executing file on macOS

After building an executable file paths fail. It started happening after removing BASE_DIR, but adding BASE_DIR now will make it worse, because now we work from .exe file in another directory.

Fixed with adding this in settings.py

Now in all paths we use prefix.


Improve/fix senior physician beret

About the PR

The current senior physician beret was originally the brigmedic one, so it totally mismatched with the whole outfit, looked bad and even the id in the files was wrong. This PR fixes those issues. The color scheme now perfectly matches with the coat and also has the orange/blue stripe on the side to match the aesthetic.
Media

[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

BEFORE:

AFTER:

Changelog
:cl: Ubaser

tweak: The senior physician beret now matches their outfit more.

ok ill fix the issue soon

oh turns out meta has a brigmedic beret spawn


Add separate keymap settings for cmdline completions.

Feature Description
Since the addition of cmdline completions, it would be really nice to have separate keymap settings for cmdline completions as they may clash with keymaps for normal completions.

If/when this is implemented, it would be nice if <Up> and <Down> are left unmapped for cmdline by default, because those keys are used for history. More generally, it might be nice to have cmdline-specific overrides, at least for some settings. For instance I use completion.menu.direction_priority = { "n", "s" } but for my cmdline (noice popup) I'd always like the menu to be below.

If/when this is implemented, it would be nice if  and  are left unmapped for cmdline by default, because those keys are used for history.

yeah this would be nice too

For instance I use completion.menu.direction_priority = { "n", "s" } but for my cmdline (noice popup) I'd always like the menu to be below.

@stefanboca Is this because the menu starts below but eventually goes above when there's not many options? I'm thinking the solution could be to always pick the direction that can fit the maximum height

Yes, or when I manually trigger and there's less options than however many lines fit above.

I'm thinking the solution could be to always pick the direction that can fit the maximum height

That should solve it. I also noticed in completion.windows.menu there's a TODO: Don't switch directions if the context is the same which might be related.

Yeah, although that solution is wrong. We should pick the direction that can fit the max_height, falling back to the largest direction if none of them can fit the max_height


Translations not reflecting

Dear Wael,
When do the import from Import process , The translations are not reflecting on Target.
Are we missing any parameter during the import ?
Note : I have put the async call timeout parameter 3600 (1 Hr) is that because of it that it would reflect after 1 hour? Since solution import take more time during to kept it to max .
-AsyncWaitTimeout $AsyncWaitTimeout  >>>> 3600
Have commented code as well as were getting timeout when we use below code in import:-
$solution = Get-XrmSolution -ConnectionString "$CrmConnectionString" -UniqueSolutionName $solutionInfo.UniqueName

Appreciate your input and suggestion.?
Thanks
Sanjay

@sanjay1913 the timeout has nothing to do with the translations. There is no additional parameters for this. This post might help you: https://www.inogic.com/blog/2018/04/working-with-translation-for-managed-solution-in-dynamics-365/


Slight wording changes

And added a quote from me :)

@navveenb made a few changes and ran the copy through Grammarly and added a few FAQs.
I'm not sure if you have perms to add FAQ answers directly to this PR, you can add them as comments and I can commit them back in.
One thing I think you should consider is a name change:
1/ SCI-ONTOLOGY sounds like Scientology and I'm not sure if you want the connection to be made there: https://www.scientology.org.uk/
2/ I'm personally not familiar with the term ontology. Perhaps the term software-boundary or SCI-Boundary or something like that might help people more quickly understand what this project is about?

Hi @jawache , I am adding the FAQ response below
How long does this tool take to create an SCI boundary?
The time required depends on the software boundary being created. Typically it should take a couple of hrs to download the tool, read the documentation, understand the tool and create the software boundary. The quick start recipe provided with the tool should help to kick start the process.
Who is the audience for this tool - PMs, Devs, Ops?
The tool is targeted at software practitioners who would like to define and implement the software boundary as part of the SCI Specification. The process can be integrated as part of the Organisation DevOps process.
Does the tool require any data feeds?
The tool expects the customer to provide the data while creating their software boundary -like VM types, hardware configurations etc.
Can the tool examine an existing application and output the ontology?
This is planned for a later release, where the idea is we can develop extensions which can inspect an existing environment (like a cloud environment) to generate the SCI open ontology model, which can be refined based on your application
How do you store the definition of an ontology?
The definitions of ontology can be outputted in any representative format. The current version would support JSON format.
Can a human use the ontology tool from a web interface, or can it only be interacted via code?
The current scope is building the sci ontology model for your application from code. In a later release, we would provide an interactive web interface.

On the naming part, Ontology is basically a definition and classification of entity. We can call it knowledge base or something which is easier to resonate for all. Let me come back on this. We can commit this if its looks good.

Hi @jawache , I have added the FAQ, please merge this. I don't have access.


[rke2] windows rke2 images should be included in manifests and rancher-images

Rancher Server Setup

Rancher version: v2.6.5-rc6
Installation option (Docker install/Helm Chart): helm

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): rke1, 1.3.10

Proxy/Cert Details: letsencrypt valid certs

Information about the Cluster

Kubernetes version: v1.22.9+rke2r1
Cluster Type (Local/Downstream): downstream

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider): custom rke2, windows

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom): standard user, cluster owner

Describe the bug
when adding a windows node to the cluster, it gets stuck waiting for probes: kubelet. Upon further investigation, this was because the rke2 manifest doesn't contain windows images anymore. after manually pulling the image into my registry, the node was able to finish registering
To Reproduce

in an airgap setup, deploy a custom cluster (default k8s, 1.22.9)
once there are linux workers in the custom cluster, add a windows 2022 node to the cluster
wait for node to register with the custom cluster

Result
fails with message waiting for probes: kubelet. After pulling the following image to my registry, the node is able to register.
https://hub.docker.com/layers/rke2-runtime/rancher/rke2-runtime/v1.22.9-rke2r1-windows-amd64/images/sha256-511e63eac71807e1d0f79bccac3e732801e0f1f8267b97e60aa4e29683a95dbe?context=explore

Expected Result
these images should be listed in rancher-windows-images.txt either as their own requirement, or added to the manifest.
Additional context
I suspect this is also an issue for the other k8s versions for rke2. please update these as well

Will be fixed by PR https://github.com/rancher/rancher/pull/37586


Add Line numbers

I this pair of patches, I added line numbers both to files and to code in the prompt.
When creating SEARCH/REPLACE blocks aider write code with line numbers as well, and I chanegd the parser to handle it. When parsing I ignore the numbers and use existing SEARCH/REPLACE.
When benchmarking, it gives a small positive performance difference, and additional small improvement in the cases of "Malformed response".
This change allow me to tell aider things like this.
You can use the following commands to get information from files outside the chat:
\\GetDefinition of <symboll> used in <file>,<line>

I am writing a vscode extension that will be able to perform this kind of commands, soon I will make it public and open source.

Thanks for suggesting these changes. Have you run the full benchmark suite against this prompting? Can you share the detailed results?
I have extensively explored this concept in the past, and found line numbers were not helpful to overall code editing success. I seemed to find they actually worsened performance.
But perhaps you're seeing something different in the benchmarks?

I run the full suit only with gpt-4-turbo.
Results are detailed below.
With line numbers:

Malformed chats: 6
Pass 1:  52.59%
Pass 2:  62.96%

Community:

Malformed chats: 12
Pass 1: 50.04%
Pass 2: 61.5%

Results are not too reproducible - when I run the community version a second time I got 48% and 58%, but I chose to use the ones consistent with your measurments.
I performed some ablation study. Some of my performance boost comes from this trick:
https://github.com/paul-gauthier/aider/pull/346
I changed the example SEARCH/REPLACE block,

Without this change we can see ~1% performance drop when introducing line numbers.
With this change, without line numbers - Pass 1: 50.04%, Pass 2: 64.4%
So the difference don't come from the line-numbers alone.

Note about user experiance:
When using line numbers, the user can't copy code from the chat and paste it without removing the numbers first. Relevant for cases of malformed responses

I didn't keep the files from the ablation study benchmarking, when I used benchmarking for the first time I wasn't organized enough and I tested uncommited changes. I will re-run it to be sure.
I will also take a look on previous implementation to have better context of what I am doing. Do you have old benchmarking of line numbers documented somewhere?

The implementation is different

The new implementation use pipe | and not space to seperate numbers from code
In the new implementation, code that appears in the system promps is also numbered. The assistant is expected to produce numbered code as well.

(would be great if could be turned on/off not only via ENVIRONMENT variable (and eventually flag) , but also during REPL (e.g. /config nonu or /config nu (yes, you see my bias toward Vim short codes...)

An update about this feature: I want to write a new benchmark, relevant to some problems I saw with this feature, to re-write some logic (searching for lines to replace) and to separate it as a third coder. I hope to find the time for it in the next few days.

Thanks for taking the time to experiment with new edit formats and share your results. I've also experimented with line numbers for code editing quite a bit. All the benchmarking I've done convinces me that it's worse to ask GPT to use line numbers.
I'm going to close this PR as I'm not likely to add line number support at this time.


Remove 5 minutes

I removed domain with 2 users only and it takes more than 5 minutes. Dont estimate time if you cant keep a promise.  I can update to this ticket how long it takes with simple DNS records and 2 active users later. Now it has took over 20 minutes.  Your estimate is probably in some laboratory environment ?

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

ID: 780db236-9235-2ce7-acbb-953fd997215b
Version Independent ID: a5b80ea1-fb9f-d381-4fa6-7d4d8cce3f82
Content: Remove a domain - Microsoft 365 admin
Content Source: microsoft-365/admin/get-help-with-domains/remove-a-domain.md
Service: o365-administration
GitHub Login: @pebaum
Microsoft Alias: pebaum

@eino-makitalo  Thank you for your feedback. The Time mentioned in the document is accurate in most cases. The delay Could be due to any other outages or Service incidents in the Geolocation or tenant.
Hope this helps!
Thanks
Sri

@eino-makitalo - Thank you for submitting feedback.
From our understanding, the issue you raised has been answered by @msbemba so we will close this issue.
Thank you for your contribution to make the docs better! Much appreciated!


Large read and write operations may be too slow

Currently read and write operations are in an iterative loop over each character. If there's a sufficient enough amount of data transferred (>2MB) then CC may terminate your program and say that CC may be installed incorrectly. This is a known issue and the current workaround is just to transfer smaller amounts of data for now.

How DARE there be bugs in the Alpha. HOW DARE YOU! I didn't pay for this!? (/j)

Hah this is nothing, 0.1.1 didn't have hardness values set so the blocks would break instantly (and the reel to reel didn't drop itself when broken)

I've done more testing. This issue only seems to appear above 20MB, which is much larger than the default tape sizes, and larger than any possible future expansions. This issue won't apply to sector media, as those will be read using a different method. I don't think this issue needs to be fixed.

You could speed this up a lot just by reading/writing multiple bytes at once using a ByteBuffer rather than converting to/from a String. This way it's just a memory copy, which should be as fast as you can get.

This is now fixed


broken link for Standalone EOP

Link at start of article for standalone EOP is https://docs.microsoft.com/en-us/exchange/standalone-eop/standaonline-eop but should be https://docs.microsoft.com/en-us/exchange/standalone-eop/standalone-eop

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

ID: a8878304-7f4b-15c3-d2b6-8a5583930199
Version Independent ID: 329f3b45-a087-6743-0664-e74dd9e0fc46
Content: Exchange Online Protection (EOP) overview - Office 365
Content Source: microsoft-365/security/office-365-security/exchange-online-protection-overview.md
Product: m365-security
Technology: mdo
GitHub Login: @chrisda
Microsoft Alias: chrisda

@softwarecraft Thank you for your feedback. We have made changes to the document  via PR#https://github.com/MicrosoftDocs/microsoft-365-docs/pull/5315
Thanks
Sri

@softwarecraft, in the future, you, too can make fixes like this yourself directly; no need to open a GitHub Issue first. For instructions, see https://github.com/MicrosoftDocs/office-docs-powershell/blob/master/README.md.
Thanks again for the feedback.
Closing (related PR approved)


Fix TiDB behavior: rename column

Index issue: #598
SQL example
ALTER TABLE t RENAME COLUMN a TO b;

Current implementation
AlterTableRenameColumn is ignored by TiDB.
Expected implementation
Report an error like ErrUnsupportedRenameColumn.

Hi, I find this issue is fix by tidb#14281.
So this can be closed.


Composer does not trigger intent when using list type entity

Describe the bug
When using list type entity in a trigger the luis trace shows the correct intent but the composer triggers "intent unknown" trigger. when checking the trace log the value is: {\r\n  \"lgType\": \"Activity\",\r\n  \"text\": \"ChooseIntent\"\r\n}",
Version
Version:  2.1.0
Electron: 8.2.4
Chrome: 80.0.3987.165
NodeJS: 12.13.0
V8: 8.0.426.27-electron.0
Browser

[ ] Electron distribution
[x] Chrome
[ ] Safari
[ ] Firefox
[ ] Edge

OS

[ ] macOS
[x] Windows
[ ] Ubuntu

To Reproduce

Define utterances
Define a list entity type
Use the list entity in the utterance

Expected behavior
Triggering correct intent
Screenshots
The utterance and the list entity:

When using exact match from utterance we get this luis trace:

And this bot trace:

When using the utterance with the {SportType} we get these traces:

Hi @JelleDK98 , are you using QnA Maker in this dialog as well?
If you are using QnAMaker, can you also attach a trace here for when you see the ChooseIntent?

Hi @taicchoumsft , yes we are using QnA Maker as well. This is the trace when we see the ChooseIntent
{   "activity": {     "type": "trace",     "serviceUrl": "http://localhost:5000",     "channelId": "emulator",     "from": {       "id": "bc55d027-b295-4927-8687-600d59bdb15d",       "name": "Bot",       "role": "bot"     },     "conversation": {       "id": "77cc379f-88db-4c0e-a109-a7720ab98687|livechat"     },     "recipient": {       "id": "42604335-34f8-47f5-aab9-76c72e7140a4",       "role": "user"     },     "locale": "nl-nl",     "replyToId": "11c0e7f9-c9cf-4e64-afa1-4ba5ec5b9a0c",     "label": "QnAMaker Trace",     "valueType": "https://www.qnamaker.ai/schemas/trace",     "value": {       "message": {         "type": "message",         "id": "11c0e7f9-c9cf-4e64-afa1-4ba5ec5b9a0c",         "timestamp": "2021-07-27T17:55:32.3+00:00",         "localTimestamp": "2021-07-27T19:55:32+02:00",         "localTimezone": "Europe/Berlin",         "serviceUrl": "http://localhost:5000",         "channelId": "emulator",         "from": {           "id": "42604335-34f8-47f5-aab9-76c72e7140a4",           "name": "",           "role": "user"         },         "conversation": {           "id": "77cc379f-88db-4c0e-a109-a7720ab98687|livechat"         },         "recipient": {           "id": "bc55d027-b295-4927-8687-600d59bdb15d",           "name": "Bot",           "role": "bot"         },         "textFormat": "plain",         "locale": "nl-nl",         "text": "ik heb een vraag ivm atletiek",         "entities": [           {             "type": "ClientCapabilities",             "requiresBotState": true,             "supportsListening": true,             "supportsTts": true           }         ],         "channelData": {           "clientActivityID": "1627408532278nokjas4gfjb",           "clientTimestamp": "2021-07-27T17:55:32.278Z"         },         "callerId": "urn:botframework:azure"       },       "queryResults": [         {           "questions": [             "Hallo",             "Hallo Robin",             "Dag Robin",             "Dag, ik heb een vraag"           ],           "answer": "Stel je vraag maar, ik probeer te antwoorden of door te verwijzen naar de juiste info of contactpersoon.",           "score": 0.88809997,           "metadata": [             {               "name": "dialogname",               "value": "robin"             }           ],           "source": "custom editorial",           "id": 1892,           "context": {             "prompts": []           }         },         {           "questions": [             "Wetenschappelijk onderzoek",             "contactpersoon ivm research",             "onderzoeksrapportage",             "publicaties",             "samenwerking",             "benchmark buitenland"           ],           "answer": "VTS baseert haar beleid, opleidingsvisie en opleidingsstructuren op heel wat internationaal onderzoek. VTS werkt ook samen met NOC*NSf in Nederland, International Council for Coaching Excellence (ICCE) en heel wat andere internationale partners. Wil je hier meer over weten ? ++RESEARCH++",           "score": 0.49130002,           "metadata": [             {               "name": "dialogname",               "value": "robin"             }           ],           "source": "custom editorial",           "id": 1909,           "context": {             "prompts": []           }         }       ],       "knowledgeBaseId": "XXX",       "scoreThreshold": 0.3,       "top": 3,       "strictFilters": [         {           "name": "dialogName",           "value": "Robin"         }       ],       "qnaId": 0,       "isTest": false,       "rankerType": "Default"     },     "name": "QnAMaker",     "id": "XXX",     "localTimestamp": "2021-07-27T19:55:32+02:00",     "timestamp": "2021-07-27T17:55:32.936Z"   },   "id": "XXX",   "timestamp":<PHONE_NUMBER>936 }

Thanks @JelleDK98.
If you don't have one already, can you add a "Duplicate Intents" trigger to the dialog (it will be in the dropdown list of triggers you can add)?  Because both Luis and QnAMaker are in the mix, and both of them have results for the customer based on this utterance, the duplicate intents trigger is needed to ask the user which intent he/she actually meant.
If this doesn't resolve your issue, we can dive in deeper.

@taicchoumsft that did the trick. so to fix this we need to better train LUIS/QnA?

Hi @JelleDK98 , yes, If you can fix the training so that this utterance will not cause both recognizers to return an intent simultaneously, this is ideal.
In some cases, ambiguity cannot be removed.  It should be ok to keep this duplicate intent trigger, and we recommend using it whenever you have QnA and LUIS used together in the same dialog.


Added pymssql & requirements

It adds the dependencies before the autoclean, so this avoids needing gcc post-install (e.g. for pip install pymssql).
Solves #118 #185 #217

Thanks, please use airflow subpackages to install pymssql https://github.com/apache/incubator-airflow/blob/6fd4e6055e36e9867923b0b402363fcd8c30e297/setup.py#L210

Ah, thanks... that link showed me why [pymssql] was not working... it should be [mssql].


Enable storing special characters such as |, @ in username column to support userid(sub) or email option

Feature Description
We are federating with multiple identity providers (IDPs). We use auth0 IDP which federates with multiple other IDPs.
We configured  oauth2_client to use userid which is mapped to userid/sub attribute (https://docs.gitea.io/en-us/config-cheat-sheet/#oauth2-client-oauth2_client)
Auth0 provides "sub" attribute in this format "waad|". Example: waad|mUiNk3eeE-gtE_tyb
We can't change the sub field as we are federating with multiple other IDPs.
Based on the error log, it looks like user name only supports alpha numeric, dot(.), underscore( _) and dash(-) characters.
Can you please enable special characters such as pipe (|) in username column?
The other option is to use email. It looks like Gitea removes email domain and just stores username. Since we federate with many IDPs, using just username will cause collision. Can you please enable storing full email? May be we can add another option called "full-email" and store the full email address, and this would require storing "@" character in username column.
Do you see any issues in supporting special characters in username field?
Screenshots
No response

That's impossible for Gitea, a lot of code was written with the current username rule.