Global regex search with "Not matching character" doesn't match newline

Issue Type: Bug
I was trying to search for
Promise\.all\(\[[^\]]*await[^\]]*\]\)

but I had no way to turn on multiline support so that ^\] would match newlines too. So searching for below would not be found.
return await Promise.all([
  await this.someFunction1(),
  await this.someFunction2(),
]);

I later put in the below and was happy to find that it suddely worked, but knew that it would only find if await was on the first line after the bracket, and also if there was no chars other than newline space or tab.
Promise\.all\(\[[^\]]*[\n\r \t]await[^\]]*\]\)

But, I then started to find others that actually SHOULDN'T have worked..
It was at this point, I realsed that ^\] was working as expected the first time, and was matching on newline chars.. (Perfect, I removed my tab/space hack).. To find, it stopped working again..
It seems I need atleast 1 \n in the regex to make it work.
My quick fix was to add this at the very end as such
Promise\.all\(\[[^\]]*await[^\]]*\]\)\n{0}

But this is obviously a dirty hack and not an obvious one, be far better to test for Not Matching and turn multiline back on, similar to what ever test you are doing for searching for \n and turning on multiline
VS Code version: Code 1.35.0 (553cfb2c2205db5f15f3ee8395bbd5cf066d357d, 2019-06-04T01:17:12.481Z)
OS version: Windows_NT x64 10.0.10240

System Info

Item
Value

CPUs
Intel(R) Xeon(R) CPU E5-1620 v3 @ 3.50GHz (8 x 3492)

GPU Status
2d_canvas: enabledchecker_imaging: disabled_offflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onnative_gpu_memory_buffers: disabled_softwarerasterization: enabledsurface_synchronization: enabled_onvideo_decode: enabledwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
15.92GB (5.33GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions (19)

Extension
Author (truncated)
Version

markdown-preview-github-styles
bie
0.1.6

npm-intellisense
chr
1.3.0

ssh
chr
0.0.4

vue-peek
dar
1.0.2

vscode-eslint
dba
1.9.0

vscode-ts-auto-return-type
ebr
1.0.1

tslint
eg2
1.0.43

RunOnSave
eme
0.0.18

prettier-vscode
esb
1.9.0

todo-tree
Gru
0.0.134

node-module-intellisense
lei
1.5.0

camelcasenavigation
map
1.0.1

rainbow-csv
mec
1.1.1

node-modules-resolve
nau
1.0.2

uuid-generator
net
0.0.4

vetur
oct
0.21.0

gitconfig
sid
2.0.0

open-in-browser
tec
2.0.0

sort-lines
Tyr
1.8.0

Yeah, currently multiline mode is only turned on when a literal newline is in the regex, for perf.

I think the inability to force multiline string or not accounting for for all the possibilities for needing it turned on should be reviewed.
Newline is indeed not a literal bracket, so should be matched. And multiline needs to be on for that. Even if it's just showing an extra button on the right, nest to the regex search" when someone does a not match [^ only.
You could even just silently bolt on \n{0} at the end internally after the input value is read. You wouldn't need to do any additional search code alterations. All edits would be at the UI component level, and multiline can be turned on by the user only if needed to keep perf by default.

The main reason we haven't done this is that we want to keep the number of possible switches under control, but it's a fair request.

Wanted to create a new issue that \s does not match newlines, but found this.

@roblourens Yeah, currently multiline mode is only turned on when a literal newline is in the regex, for perf.

How about better newline-regex detection? Something like this:
const { assert } = require("chai");

// Just came up with this, thus requires more tests. Doesn't match some edge cases.
const IS_MULTILINE = /(?<!(?<!\\)\[[^\]]*)(?<!\\)\\[ns]|(?<!\\)\[\^(.(?<!(?<!\\)\\[ns]))*(?<!\\)\]|(?<!\\)\[(?!\^)[^\]]*(?<!\\)\\[ns]/;

assert.notOk(IS_MULTILINE.test(/[^\n]/));
assert.notOk(IS_MULTILINE.test(/[^\s]/));
assert.notOk(IS_MULTILINE.test(/[s]/));
assert.notOk(IS_MULTILINE.test(/\[^a]/));
assert.notOk(IS_MULTILINE.test(/[\\s]/));

assert.ok(IS_MULTILINE.test(/[^\\s]/));
assert.ok(IS_MULTILINE.test(/[^\\n]/));
assert.ok(IS_MULTILINE.test(/[^s]/));
assert.ok(IS_MULTILINE.test(/[\s]/));
assert.ok(IS_MULTILINE.test(/\[\s/));
assert.ok(IS_MULTILINE.test(/[^\s]\s\]/));
assert.ok(IS_MULTILINE.test(/\[\s\]/));

I am assuming in this test below, s is just a substitute of any writeable char
assert.ok(IS_MULTILINE.test(/[^s]/));

@WORMSS in this case it's anything other than literal s, which for this purpose seems an irrelevant test case which I must have mistakenly kept in the test suite when I was fuzzing all manner of cases.

Regex Buddy confirms.
Though, I still had to do my hack to get it to match newline as a replaceable char. (ignoring the fact its treating \n\r as a single char.

Windows obviously.

I'm probably missing the point. /[^s]/ should match newline, hence:
assert.ok(IS_MULTILINE.test(/[^s]/));

is passing

Sorry, I read the sentence back and it makes little sense to myself.
Regex Buddy agrees would have been a better way for me to say it.
But I wanted to point out that RegexBuddy seems to count \n as a "not an s" even with multiline turned off. I don't know if that makes much of a difference.
And strangeness that \r\n is counted as a single char in vsc. (which I actually prefer to be honest)

\n is "not an as". Am I still missing the point? What are you suggesting?

I think there might be a bigger issue with "multiline" term in the first place if we are trying to agree with each other but seem to be missing the mark.
I've just re-re looked up my understanding multiline is for ^ and $ usage.
It seems javascript agrees with me, that "not an s" should match new line by default and not need multilined turned on.

s flag would be . matches newline.
I am guessing IS_MULTILINE is just a completely different type of allow multiline that is not directly related to a regex multiline?

I see --- yes, we're using the term to mean different things. When I say multiline in this context, I'm talking about ripgrep --multiline switch, which is what vscode uses to enable a regexp to match across multiple lines. From ripgrep's manual:
-U, --multiline
    Enable matching across multiple lines.

    When multiline mode is enabled, ripgrep will lift the restriction that a match
    cannot include a line terminator. For example, when multiline mode is not
    enabled (the default), then the regex '\p{any}' will match any Unicode
    codepoint other than '\n'. Similarly, the regex '\n' is explicitly forbidden,
    and if you try to use it, ripgrep will return an error. However, when multiline
    mode is enabled, '\p{any}' will match any Unicode codepoint, including '\n',
    and regexes like '\n' are permitted.

    An important caveat is that multiline mode does not change the match semantics
    of '.'. Namely, in most regex matchers, a '.' will by default match any
    character other than '\n', and this is true in ripgrep as well. In order to
    make '.' match '\n', you must enable the "dot all" flag inside the regex.
    For example, both '(?s).' and '(?s:.)' have the same semantics, where '.' will
    match any character, including '\n'. Alternatively, the '--multiline-dotall'
    flag may be passed to make the "dot all" behavior the default. This flag only
    applies when multiline search is enabled.

Then 👍 Big thumbs up that [^s] should indeed turn that mode on.. Ok.. I am up to date and agreeing. Good job everyone.. take a 5 minute break and enjoy your evening.

It seems like you are just saying that "if a regex can match \n then the multiline flag should be turned on". Isn't this the same as having the multiline flag turned on all the time?
Anyway to me the whole point is that the majority of regexes are not intended to match multiple lines, it's quite common to include \s when you really are looking for spaces and tabs within a line. And including \s should not blow up your search results by matching a bunch of \r and \n when you did not explicitly ask for it.
This heuristic of using \n is my alternative to actually having a button or checkbox to enable multiline mode.

I explicitly asked it to match any char that was not a ]
To the best of my knowledge \n and \r are not ] so therefore should have been matched.
There is no visible indication anywhere that multiline is turned on or off. It is just some magic that goes on behind the scenes. Only way to get consistent results would be to add \n{0} to every regex I do.
You may feel that \s should match only spaces and tabs but can you guarantee that every single user of VSCode would (1) know that that is indeed what is going on, and (2) know how to bypass the limitation if they needed to?

All I can do is what I think is right for the majority of users and hope that the rest can google their way to a solution. Because of the perf impact and possibility for unexpected behavior I think multiline search should always be an explicit opt in. My guess is that we would add a button for this some day but it will take careful UX thinking because we have too many options crammed into a small space already.

This heuristic of using \n is my alternative to actually having a button or checkbox to enable multiline mode.

IMO it's much better to have slower search than to potentially cause bugs (among other things), e.g. when someone refactors code and the search doesn't match all it should, in which case you better hope your CI catches it.
Nothing worse than a regex engine that doesn't comply to spec in such a glaring way as redefining \s to exclude \n, which is pretty unintuitive. I'd at least show a warning message when the regex contains \s.
Also, having an option to opt-in for \s to include \n seems reasonable to me.

That doesn't help in my case when I am not searching for \s but searching for not ]

That doesn't help in my case when I am not searching for \s but searching for not ]

True. I'd say your case is even more insidious, because it contains no whitespace characters in any way.
More generally, how should a user know that in vscode alone, you need to write (?:[^\]|\n) instead of [^\]].
Problem is, the above solution doesn't generalize because apparently you can't use (?:a|b) in negative-lookbehind because the vscode regex parser thinks it's variable length (which it isn't -- works in rg).
we should definitely tell user in some way that vscode search redefines how newline matching works.

Just ran into the same issue with the following use case:
I have a large JSON file in the following format:
[
    {
        "key": []
    },
    {
        "key": []
    },
    ...
    {
        "key": [
            1
        ]
    },
    ...
    {
        "key": []
    },
    {
        "key": []
    },
]

I wanted to find the key which contained a non-empty array. Naturally I tried to search for "key": \[[^\]] in the file but was rewarded with 0 matches. Luckily I thought of searching for "key":  (X matches) and then for "key": \[\] (X-1 matches) and figured out there's newline shenanigans. Took me the good part of an hour though. Would definitely like to see an option to enable multiline.

Hey mates @WORMSS Why not using structural search for your case?
I've created an extension that make it super easy to search such cases without having to fight with regexes!
Check CodeQue
The query would as simple as that:
Promise.all([
  await $$$(),  // $$$ is any statement wildcard
]);

I'm using next.js repo as test case here
https://user-images.githubusercontent.com/11561585/204633410-d022d47e-4219-4d3a-8d3a-654ad0f7b31d.mp4

@Mitko-Kerezov in your case, the query could bee like this
({
   "key": [$$$]
 })

You can give it a try in CodeQue Playground

I just use \n{0} a the end of all my searches.. been working fine so far..

How VSCode developers dared to violate the common standards among regex flavours? Any regex engine (except for the old shell command sed) finds newline when I search for whitespace \s or negative character classes which don't include a newline. This is the definition.
negated character class reference
not matching newline in negated character classes is not a comparison criterion among important flavours
My problem, I want to search for multiline comments. Any javascript engine would match multiple lines with this regex:
/\/\*[^*]*\*\//g
Only in VSCode, I need this:
/\*(?:[^*]|\n)*\*/
(The \n{0} trick does not work for me.)
It is less convenient to type.
In this case, /\*[.\n]*?\*/ would be even better (and works for VSCode and JavaScript) but it has different semantics in the general case.
It's a common advice to use [\s\S] for matching all characters but it doesn't work with VSCode.
VSCode's regex interpretation is unintuitive regarding newlines because no other relevant regex engine behaves this way. Most languages consist of multi-line structures where users will need to search for multi-line structures at some time (when matching code pieces).
Usability is always more important than program speed. Because usability is the bottle neck. In the opposite case, the work that the code would do is imposed on the user who is slower than the code.
Sometimes, VSCode cannot even find multiline matches in the way I described above. (I found a case that led to catastrophic backtracking in VSCode.) Then I copy the text to another editor or to the browser to find/replace with their regex engine.
If VSCode cannot do it "because of the perf impact" then it is not the newline to blame but the implementation of VSCode. I never had any speed problems with other regex engines which allow me to match many lines at once.
The actual performance impact is catastrophic backtracking which is easy to bypass with a good regex style.


Update dasm and support new AddToSets functionality

Todo before merge:

[ ] Test it
[ ] Write documentation on the dasm side

Closing in favour of dasm2


Site 'Sprungbrett' could not be found

Describe the Bug
Inside 'Offers' the link to 'Sprungbrett' leads to "Sorry, but the requested page could not be found"
Steps to Reproduce

In 'Testumgebung' or  'Augsburg (Stadt)' open 'Offers'
klick 'Sprungbrett'

@svenseeberg That looks familiar. I am again getting a 301 and then a CORS missing allow origin.

If you access https://integreat.app/proxy/sprungbrett/app-search-internships?location=86150 instead of https://web.integreat.app/proxy/sprungbrett/app-search-internships?location=86150 then there would be no 301. But the page itself seems broken.

This is fixed now. Wasn't anything we are responsible for.


Fix gradle warnings

Closes #116

Thanks @DeKaN !
There is some warnings left that I'm mentioned here:

Please delete this empty file:

And remove these dead codes:

@Kaaveh Fixed missed warnings

Thanks, @DeKaN for your Contribution! We hope to see more PR from you in the future! ❤🙌🏻


Allow custom inline rules.

There is no public API that allows you to add a custom LayoutBlockBuilder into MarkDownToInlineAttributedStringConverterConfiguration. In a result, I don't know a convenient way of adding a custom inline rule.
Maybe add an option to inject it into MarkdownToViewConverterConfiguration or into MarkDownToAttributedStringConverterConfiguration?
There might exist other issues related to internal access controls.

Will be solved by https://github.com/M2mobi/Marky-Mark/pull/114


HWU64 odd number of samples

The HWU64 dataset contains 25k samples according to the original paper. The DialoGLUE paper stats the same number of samples.
However, the Readme states 11k samples.
If I count the number of samples which are actually in the HWU64 part of DialoGLUE then I get 12,112 samples (12k).
My questions:

Is there a reason for the difference in numbers in the original HWU64 and in the DialoGLUE HWU64? Or is it a bug?
Did you compute the performance of the intent prediction models on 25k, 12k or 11k samples?

Thank you for your answers :)

Thank you for bringing this discrepancy to our attention. We use the data downloading scripts provided by [https://arxiv.org/pdf/2009.13570.pdf] to get all of the intent prediction datasets, including HWU, in order to maintain consistency. The performance is reported on the data that is obtained by the data downloading scripts (i.e., the ~11k data points). The error here is the 25k number cited in the paper, which I will attempt to get corrected in the near future.
I'm not sure how you're counting 12,112 samples. I just ran the data downloading scripts and see:
1077 test.csv
9961 train.csv

thank you for the quick answer :) I counted 12,112 by adding
9960 train.csv
1076 test.csv
1076 val.csv
so I counted the val file also. There is still a discrepancy of 1 sample but that's ok for me.


Surprise Build Tool Issues

There was an issue with the previous iteration of the settings.gradle.kts where building wasn't properly attaching the main manifest. This has been fixed now.

Related to #6.


Bundle Install not working

$ bundle install
Ignoring nokogiri-<IP_ADDRESS> because its extensions are not built.  Try: gem pristine nokogiri --version <IP_ADDRESS>
Ignoring redcarpet-3.3.4 because its extensions are not built.  Try: gem pristine redcarpet --version 3.3.4
--- ERROR REPORT TEMPLATE -------------------------------------------------------

What did you do?
I ran the command /usr/local/bin/bundle install

What did you expect to happen?
I expected Bundler to...

What happened instead?
Instead, what actually happened was...

Error details
Errno::EACCES: Permission denied @ dir_s_mkdir - /home/ubuntu/.bundle/cache/compact_index/rubygems.org.443.29b0360b937aa4d161703e6160654e47/versions20160624-24856-1enpvx6
  /usr/lib/ruby/2.2.0/tmpdir.rb:85:in `mkdir'
  /usr/lib/ruby/2.2.0/tmpdir.rb:85:in `block in mktmpdir'
  /usr/lib/ruby/2.2.0/tmpdir.rb:129:in `create'
  /usr/lib/ruby/2.2.0/tmpdir.rb:85:in `mktmpdir'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/vendor/compact_index_client/lib/compact_index_client/updater.rb:29:in `update'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/vendor/compact_index_client/lib/compact_index_client.rb:64:in `update'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/vendor/compact_index_client/lib/compact_index_client.rb:55:in `update_and_parse_checksums!'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/fetcher/compact_index.rb:65:in `available?'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/fetcher/compact_index.rb:15:in `call'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/fetcher/compact_index.rb:15:in `block in compact_index_request'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/fetcher.rb:154:in `use_api'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/source/rubygems.rb:331:in `block in api_fetchers'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/source/rubygems.rb:331:in `select'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/source/rubygems.rb:331:in `api_fetchers'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/source/rubygems.rb:336:in `block in remote_specs'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/index.rb:10:in `build'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/source/rubygems.rb:335:in `remote_specs'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/source/rubygems.rb:82:in `specs'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/definition.rb:211:in `block (2 levels) in index'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/definition.rb:209:in `each'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/definition.rb:209:in `block in index'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/index.rb:10:in `build'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/definition.rb:206:in `index'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/definition.rb:200:in `resolve'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/definition.rb:140:in `specs'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/definition.rb:129:in `resolve_remotely!'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/installer.rb:195:in `resolve_if_need'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/installer.rb:70:in `run'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/installer.rb:22:in `install'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/cli/install.rb:102:in `run'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/cli.rb:175:in `install'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/vendor/thor/lib/thor/command.rb:27:in `run'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/vendor/thor/lib/thor/invocation.rb:126:in `invoke_command'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/vendor/thor/lib/thor.rb:359:in `dispatch'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/vendor/thor/lib/thor/base.rb:440:in `start'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/cli.rb:11:in `start'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/exe/bundle:27:in `block in <top (required)>'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/lib/bundler/friendly_errors.rb:98:in `with_friendly_errors'
  /var/lib/gems/2.2.0/gems/bundler-1.12.5/exe/bundle:19:in `<top (required)>'
  /usr/local/bin/bundle:23:in `load'
  /usr/local/bin/bundle:23:in `<main>'

Environment
Bundler   1.12.5
Rubygems  <IP_ADDRESS>
Ruby      2.2.4p230 (2015-12-16 revision 53155) [x86_64-linux-gnu]
GEM_HOME  /var/lib/gems/2.2.0
GEM_PATH  /var/lib/gems/2.2.0:/home/ubuntu/.gem/ruby/2.2.0:/usr/share/rubygems-integration/2.2.0:/usr/share/rubygems-integration/all
Git       1.9.1
  Bundler settings

orig_path
Set via BUNDLE_ORIG_PATH: "/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games"
--- TEMPLATE END ----------------------------------------------------------------
Unfortunately, an unexpected error occurred, and Bundler cannot continue.
First, try this link to see if there are any existing issue reports for this error:
https://github.com/bundler/bundler/search?q=Permission+denied+%40+dir_s_mkdir+-+%2Fhome%2Fubuntu%2F.bundle%2Fcache%2Fcompact_index%2Frubygems.org.443.29b0360b937aa4d161703e6160654e47%2Fversions20160624-24856-1enpvx6&type=Issues
If there aren't any reports for this error yet, please create copy and paste the report template above into a new issue. Don't forget to anonymize any private data! The new issue form is located at:
https://github.com/bundler/bundler/issues/new

The error is because bundler does not have permissions to access that folder. Please check your folder permissions and try again. Also, check out how the other issues with permission errors were resolved, like #4458, #4429. Let us know if you still have problems after you've got the right folder permissions set.

Closing as I think there's enough context in @hmistry's linked issues to alleviate this. Please let us know if this isn't the case.


Admin Router: Generalise endpoint tests

High Level Description
This PR:

adds reflecting capability to mesos/iam/mesos_dns/marathon mocks
test-harness fixes:

generic_no_slash_redirect_test should check the Location header
add agent_ar_process_perclass fixture

refactors common tests into generalized sub-framework based on YAML config files (*/test_generic.config.yml) Adding standard test cases is now based soley on reconfiguring this file. Documentation is comming
clears up Marathon upstream - re-use /service/marathon instead of local marathon instance. This improves reliability

Related Issues

DCOS_OSS-1016 Adminrouter talks to localhost Marathon instead of leading Marathon
DCOS-14998 Admin Router: Generalize endpoint tests in AR

Dependencies
Related EE PR: https://github.com/mesosphere/dcos-enterprise/pull/892
Old Open PR: https://github.com/dcos/dcos/pull/1514

@gpaul @mhrabovcin Thanks for the review!
@mhrabovcin Ready for the second pass!


Usability after update on server requires logout and login. Only running login does not work

I have updated keybase client on my server. Few issues I have observed

After update running cd to /keybase hangs forever.
Currently every update requires user to log in again in all systems(windows, linux).  So when I ran "keybase login", nothing happens. Access to kbfs is possible only after running keybase logout and keybase login

This I consider only as usability issue for headless systems.
my log id: 71d061f1c543963f73bd041c

Does 'keybase unlock' work?
On Wednesday, April 13, 2016, SirishKumar<EMAIL_ADDRESS>wrote:

I have updated keybase client on my server. Few issues I have observed

After update running cd to /keybase hangs forever.
Currently every update requires user to log in again in all systems.
So when I ran "keybase login", nothing happens. Access to kbfs is possible
only after running keybase logout and keybase login

This is consider only as usability issue for headless systems.
my log id: 71d061f1c543963f73bd041c
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub
https://github.com/keybase/client/issues/2600

When cd to keybase hangs, I didn't had the control of the terminal, Ctrl+C does not work, I had to close the terminal manually.
I didn't tried running keybase unlock from another terminal session. In next update I can try running "keybase unlock".

Resolving this, I no longer see this issue


Travisci

Hi @xakraz!
I'm happy to merge changes upstream here if you want. Only thing that needs to be fixed is to rebase. Sounds good? Thanks!


Possible to use custom function on expression inside repeat table?

I'm trying to call a custom function on an expression inside the repeat table and it doesn't work. According to the document, "Expressions found in the cells of the table row are evaluated against the object from the list". It can be inferred from the quote that in my case, the expression is evaluated against Map<String,String> in which the custom function is not present. Am i missing something or this feature hasn't been implemented yet?
Here are the codes:
public interface DateFormatFunction {
    String formatDate(final String dateString, final String format) throws ParseException;
}

public static class DateFormatFunctionImpl implements DateFormatFunction {

    private static final String INPUT_FORMAT = "dd/MM/YYYY";

    @Override
    public String formatDate(final String inputDateString, final String outputFormat) throws ParseException {
        final SimpleDateFormat inputFormatter = new SimpleDateFormat(INPUT_FORMAT);
        final Date inputDate = inputFormatter.parse(inputDateString);

        final SimpleDateFormat outputFormatter = new SimpleDateFormat(outputFormat);
        return outputFormatter.format(inputDate);
    }

}

public static class TemplateContext {

    private List<Map<String, String>> items;

    public List<Map<String, String>> getItems() {
        return items;
    }

    public void setItems(List<Map<String, String>> items) {
        this.items = items;
    }

}

@Test
public void testMap2() throws FileNotFoundException, IOException {
    TemplateContext context = new TemplateContext();

    List<Map<String, String>> csvRecords = parseCSV(new File("sample_1.csv"));
    context.setItems(csvRecords);

    try (FileInputStream inputStream = new FileInputStream("report_template.docx")) {
        DocxStamper<TemplateContext> stamper = new DocxStamperConfiguration()
                .exposeInterfaceToExpressionLanguage(DateFormatFunction.class, new DateFormatFunctionImpl())
                .build();

        final ByteArrayOutputStream reportOutputStream = new ByteArrayOutputStream();
        stamper.stamp(inputStream, context, reportOutputStream);

        final ByteArrayInputStream reportInputStream = new ByteArrayInputStream(reportOutputStream.toByteArray());
        try (FileOutputStream pdfFileOutputStream = new FileOutputStream("report_result.pdf")) {
            convertDocxToPdf(reportInputStream, pdfFileOutputStream);
        }
    }
}

My template:
Thanks, Le

copied from https://github.com/thombergs/docx-stamper/issues/60
Will probably be solved when we remove the current working based on proxyfication of the context.

As per the test fetured in CustomFunctionTests.java it seems to be working correctly now.


feat(js-client): Add fire-and-forget flag [DXJ-562]

https://github.com/fluencelabs/cli/pull/652
Test passed in updated CLI


[Question]: Should #990 be tracked upstream

If the automatic wordlist updater feature gets merged, the pull request will get overwritten. I feel that it should be tracked so that we can know if it will get added.
https://github.com/wallarm/jwt-secrets/pull/21

OK no worries, it works
Le lun. 8 avr. 2024 à 05:16, Mo Langning @.***> a
écrit :

If the automatic wordlist updater feature gets merged, the pull request
will get overwritten. I feel that it should be tracked so that we can know
if it will get added.
wallarm/jwt-secrets#21 https://github.com/wallarm/jwt-secrets/pull/21
—
Reply to this email directly, view it on GitHub
https://github.com/danielmiessler/SecLists/issues/1006, or unsubscribe
https://github.com/notifications/unsubscribe-auth/A5MEH2IFK4NO36XJQAKTBNLY4IDZJAVCNFSM6AAAAABF34P7B6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGIZTAMJWGU3TOOI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>

It has been merged upstream


RCTThirdPartyFabricComponentsProvider always generated in React-Fabric filepath

Description
RCTThirdPartyFabricComponentsProvider always generated in React-Fabric filepath
not in the -outpath param specified path
Steps to reproduce
node node_modules/react_native/scripts/generate-codegen-artifacts.js
--path SampleApp/
--outputPath <an/output/path> \
React Native Version
0.72.5
Affected Platforms
Runtime - iOS
Areas
Codegen
Output of npx react-native info
System:
  OS: macOS 14.2.1
  CPU: (8) arm64 Apple M1 Pro
  Memory: 147.52 MB / 16.00 GB
  Shell:
    version: "5.9"
    path: /bin/zsh
Binaries:
  Node:
    version: 20.5.1
    path: /opt/homebrew/bin/node
  Yarn:
    version: 1.22.21
    path: /opt/homebrew/bin/yarn
  npm:
    version: 9.8.0
    path: /opt/homebrew/bin/npm
  Watchman:
    version: 2<IP_ADDRESS>
    path: /opt/homebrew/bin/watchman
Managers:
  CocoaPods:
    version: 1.11.3
    path: /usr/local/bin/pod
SDKs:
  iOS SDK:
    Platforms:
      - DriverKit 23.2
      - iOS 17.2
      - macOS 14.2
      - tvOS 17.2
      - visionOS 1.0
      - watchOS 10.2
  Android SDK: Not Found
IDEs:
  Android Studio: Not Found
  Xcode:
    version: 15.2/15C500b
    path: /usr/bin/xcodebuild
Languages:
  Java:
    version: 20.0.1
    path: /usr/bin/javac
  Ruby:
    version: 2.6.10
    path: /usr/bin/ruby
npmPackages:
  "@react-native-community/cli": Not Found
  react:
    installed: 18.2.0
    wanted: 18.2.0
  react-native:
    installed: 0.72.5
    wanted: 0.72.5
  react-native-macos: Not Found
npmGlobalPackages:
  "*react-native*": Not Found
Android:
  hermesEnabled: true
  newArchEnabled: false
iOS:
  hermesEnabled: false
  newArchEnabled: true

Stacktrace or Logs
nothing

Reproducer
https://github.com/nanatlantica/RNDynamic
Screenshots and Videos
Can I put the files generated by codegen into another repo, to make react-native repo no need to update frequently

Actually, this file ends up in React-RCTFabric, slightly different.
Anyway, that's correct for the time being.
That's the right path for that file. The file needs access to RCTFabric files and RCTFabric uses this file, that's why we generate it there. Otherwise, generating it in the React-Codegen path, that will create a circular dependency:

React-Codegen will depend on React-RCTFabric.
React-RCTFabric will depend  React-Codegen.

We are already working toward disentangling this, but for the time being, that's the correct behavior.


WIP: Test Bump Tekton to v0.21.0

Changes
This is just a TEST PR fore Tekton v0.21.0 bump!!!

FYI @xiujuan95 ^^

the go vet test passed in my PR.
FYI @xiujuan95 ^^

It's a magic!!!!!! I suspect I did go mod vendor in mac OS, however, our tests run on linux OS. Now, I revert everything and go mod vendor on linux OS, seem like below error disappears:
vendor/k8s.io/kube-openapi/pkg/util/proto/document.go:24:2: cannot find package "." in:
	/home/runner/work/build/build/vendor/github.com/googleapis/gnostic/openapiv2

Zoe already fixed the go vet problem in the PR: https://github.com/shipwright-io/build/pull/663
Let me close this test PR.


Refactor io.monitor to support alternativ file entities

I want to combine the io.monitor with the https://github.com/hierynomus/smbj project, to realize a SMB capable io monitor. In order to achieve this, I have to replace certain hard-coded handles like File with the resp. SMBJ handles.
To this end, I started refactoring the code so that I may realize this alternative implementation. Within my IDE i have a prototype of this combination (realized within a separate project) that basically works.
My question is: would the commons-io community care for accepting a PR that "opens-up" the monitor for alternative implementations?
The PR at hand serves as a basis for discussion.

May you please rebase on master?

@col-panic
May you please rebase on master?

I did a dumb rebase on the current master, only accepting the incoming changes. I did not have a look at the code, as it's been a while I haven been creating this PR and we realized a separate solution for this in the meantime! I leave it up to you what to do with it, as I can't spend time more time on this - sorry! :(

This PR now changes 463 files, obviously not good or intended I am sure. Please review rebase on git master.

Closing, no feedback and 400+ files changes is impossible to review.


Add check to see if PRAW is being ran in an asynchronous environment

Feature Summary and Justification
This adds a check to see if PRAW is being ran in an asynchronous environment (more specifically, if there is a running asyncio event loop) and advise users to use Async PRAW if there is one running.
This adds a check that is ran on the initialization of praw.Reddit and every time Reddit.request() is called. It can be suppressed by passing check_for_async=False anywhere config options are accepted.

In the first revision, I had on just the initialization of praw.Reddit since that is when the requestor is initialized. But then I got thinking about it and thought it would be better have it on Reddit.request as well since that's where the blocking requests are made and because praw.Reddit can be initialized outside of the asynchronous environment.

In the first revision, I had on just the initialization of praw.Reddit since that is when the requestor is initialized. But then I got thinking about it and thought it would be better have it on Reddit.request as well since that's where the blocking requests are made and because praw.Reddit can be initialized outside of the asynchronous environment.

Perhaps we should only do it only request then since having it in both seems a little redundant.

Perhaps we should only do it only request then since having it in both seems a little redundant.

Sounds good! I'll get that changed later tonight.

Sounds good! I'll get that changed later tonight.


FTS5 Support

Hello everyone,
I'm using the fts4 extension and I noticed that since SQLite version 3.9.0 a more mature version of fts5 was  released as well. Since this jdbc driver rocks SQLite version 3.14.2, already makes me wonder why isn't included.
I tried to create an fts5 table but I get an SQLITE_ERROR:
[SQLITE_ERROR] SQL error or missing database (no such module: fts5)
Interestingly enough, the build flag was enabled since this commit https://github.com/xerial/sqlite-jdbc/commit/28882dd0ecbdf4af20c146a2f9c3a96096b99e6f which makes me wonder why I'm receiving this error.

@compengi FTS5 actually in-place and works fine. I saw this error when you specify column names for the keyword search. FTS5 requires using (table name) MATCH 'keyword'.

@compengi See https://github.com/xerial/sqlite-jdbc/blob/8df0b6ccee1a58958cb3b8724a25a0640b2c547b/src/test/java/org/sqlite/ExtensionTest.java#L53 for example.

@xerial no the it errors on CREATE.
I ran the test you mentioned on macOS 10.12.1 it went through, on Windows 10 it failed for FTS5.
java.sql.SQLException: [SQLITE_ERROR] SQL error or missing database (no such module: fts5)
at org.sqlite.core.DB.newSQLException(DB.java:890)
at org.sqlite.core.DB.newSQLException(DB.java:901)
at org.sqlite.core.DB.execute(DB.java:810)
at org.sqlite.core.CoreStatement.exec(CoreStatement.java:74)
at org.sqlite.jdbc3.JDBC3Statement.execute(JDBC3Statement.java:61)
at com.exocad.bra.report.parser.test.ExtensionTest.extFTS5(ExtensionTest.java:52)

Please re-open this ticket.

OK. Then the title should be FTS5 support on Windows.

@compengi I cannot reproduce this error with mvn test, which includes test cases for FTS5. I'm using Windows 8.

@compengi Could you try using the latest snapshot jar? https://oss.sonatype.org/content/repositories/snapshots/org/xerial/sqlite-jdbc/3.15.0-SNAPSHOT/sqlite-jdbc-3.15.0-20161031.182337-4.jar

Yup, the tests execute fine on 3.15.0
I thought it was already added since 3.9.0?

I apologize. Something interesting just happened.
I commented out the gradle dependency to try your library. After uncommenting it back, I ran the tests and they went through!
I guess it was a gradle/maven/intellij dependency issue.
Sorry for the confusion :/
I think it's safe to close this ticket.

Thanks for the confirmation!


Automated cherry pick of #4420: update rbac default policy

Cherry pick of #4420 on release/2.12.
#4420: update rbac default policy

Codecov Report

:exclamation: No coverage uploaded for pull request base (release/2.12@20468aa). Click here to learn what that means.
The diff coverage is n/a.

@@              Coverage Diff               @@
##             release/2.12   #4422   +/-   ##
==============================================
  Coverage                ?   6.83%
==============================================
  Files                   ?     501
  Lines                   ?   85410
  Branches                ?       0
==============================================
  Hits                    ?    5839
  Misses                  ?   79078
  Partials                ?     493

Flag
Coverage Δ

#aFlag
6.83% <ø> (?)

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 20468aa...bf94570. Read the comment docs.

/lgtm

/approve

/approve


Make it clear which tab Pixel Perfect is applying to

Maybe hide the Pixel Perfect window when the user switches tabs, then show it back if the tab has Pixel Perfect on.

Yep I like the idea.
Honza


Be able to run postmortem with --since=xh or xm flag

Would be nice if postmortem had option to collect logs just for the last hour vs full logs etc.
similar to kubernetes logs --since=xx option

@micoletta Does log_limit help with your requirement ?

Any news on this idea? I was about to post the same idea, also allow specific timestamp input for those scenarios when the customer is aware when the problem initiated.
Thanks.

@micoletta @dfibm  - Not sure I see the benefit of this issue - from perspective of those analysing the logs - I think it is best to see as much as possible. If goal here is how long it takes to run - we have other plans to speed up the running of post mortem.


Fix list -trashed

Fixes #695.

Apologies for the late reply, I'll comment more on this PR in about 9 hours when I get back home.

What broke list --trashed was two separate small bugs. First, it wasn't working in nested directories because it was looking for the directory in trash rather than just its containing files in trash. So I switched list to use its default path resolver FindByPath instead of FindByPathTrashed.
Second, list --trashed returned all trashed items in all directories. I found in buildExpression that a parent directory was not specified if searching inTrash. Adding it back to match the regular case fixed the issue. I also removed the (typeMask&InTrash) != 0 check because I felt it shouldn't contradict the inTrash parameter and simplified the expression even more.

Dope! Thanks @njbbaer. Just one comment here, would you mind updating your commit message to reflect the change with something like this
list: fix --trashed to return all items in the current working directory

Fixes #695

<Any description of the solution if you'd like to make one>

Thanks for the tip. I've updated my commit message.

Oh, my bad, I meant in the actual git commit like this
$ git commit --amend

and then when it opens up the editor
list: fix --trashed to return all items in the current working directory

Fixes #695

<Any description of the solution if you'd like to make one>

and after force push it to overwrite this current commit.
$ git push origin <branch_name> -f

That's what I did. I changed both the commit message and the pull request title.

Awesome. Thank you. LGTM! Merging in.


DEV-7763 Expand/Collapse Buttons

Testing Instructions
New Dashboard

Open CDC Editor, choose Dashboard, and import data.
In the Dashboard editor, hover over the hamburger icon on the top left of the visualization box and select the "Toggle between up to three visualizations" icon all the way to the right
Drag and drop up to 3 visualizations into the box
Click on the Gear icon in the top right corner of the visualization row box
Choose the dataset and select the "Configure Multiple Visualizations" checkbox
Choose an option from the drop down
Click the "Add Expand/Collapse All Buttons" checkbox
Click Continue
Configure the visualizations if needed
Click on, Dashboard Preview Tab
In preview click the Expand All and Collapse All buttons to see that they are functioning properly
Click on the "+" and "-" buttons on the individual boxes to see that they are functioning properly

Existing Dashboard

Open an existing dashboard, open the editor, and go back to the Dashboard Description tab
Click the Gear icon in the top right hand section of the visualization row box
Click the "Add Expand/Collapse All Buttons"
Click Continue button
Click the Dashboard Preview Tab and test the buttons

@Matt-DiPaolo is this ready to be merged? Can you catch your branch up and merge it into dev?


Updated drawer movement

Context

Ticket: https://www.notion.so/thomashudsonnotes/a1f3e7cd3bf74c62b06dbda78b2c9c7c?v=43d3f2b9730045f691cd254967c6949d&p=82f848b368ee4d56a7b57569caeadc61

Related PRs
https://github.com/SocexSolutions/agenda-v2/pull/12

Blocks/Is Blocked by Ticket
No

Who uses this and for what?
The user's will use this (the people with accounts not people invited to meetings)

Is this part of a greater feature? What is it?
This is part of the great feature or out website yes.

Implementation
Changes
Made the bar a bit smaller and changed how it opens and closes.  It currently leaves a little bit of space shown.  I have 2 ideas:

Add a hamburger thing in the navbar to the left of the home button and rearange how the navbar looks.
Made the right hand that is shown still some sort of barlike thing so it looks like just a nice slime icon that spans the length of the drawer to show that it is there.

Feedback Request (optional)

Are there any parts of the implementation you are unsure about?
Which of the two options from above to do.

Preview (optional)

Merge Checklist

[ ] Correct Target Branch
[ ] Pre-Review Diff Check
[ ] PR Description
[ ] Add Reviewers and Assignees
[ ] Review Complete
[ ] Rebase
[ ] Rebase Approved

This is now a duplicated pull request. When a node in a system is deemed a duplicate by the other nodes it must self destruct.


[Resolved] text_on_image Creator - Web design #136 #636

[Resolved] text_on_image Creator - Web design #136 #636
Hi, I have solved this issue, Please review and accept my pr soon.

Heartfelt Thanks for Your Contribution to Our Open Source Project!
            Dear @RanadeAtharva,

            I trust this message finds you in good health and high spirits. I am reaching out with a heart full of gratitude for your recent contribution to our open source project. Your dedication has not only elevated the functionality of our project but has also exemplified the collaborative spirit that lies at the core of thriving open source communities.

            A heartfelt congratulations are in order for your exceptional work! Your commitment and expertise have played a pivotal role in the success of our project, and we consider ourselves truly fortunate to count you among our community members.

            I must extend my sincere apologies for the delay in responding to your pull request. Your patience and understanding during this time are immensely appreciated. As our project continues to grow, we acknowledge the imperative need to enhance the quality and response time for contributions. To address this, we've kickstarted a call for maintainers and reviewers who can help us uphold the high standards of our project and ensure swift responses to pull requests.

            Your involvement and insights would be invaluable in this initiative. If you, or someone you know, is interested in taking on a more active role in the project, please don't hesitate to reach out. We firmly believe that with a robust team of maintainers and reviewers, we can elevate both the quality of the project and the speed of our responses.

            On an exciting note, I am thrilled to share that we have activated the sponsorship button for our project. This development allows supporters like yourself to contribute to the sustainability and growth of the project. Your ongoing support, whether through code contributions or sponsorship, is genuinely appreciated.

            Once again, congratulations on your outstanding contribution, and thank you for being an indispensable part of our open source community. We eagerly anticipate your continued collaboration and shared success.

            Best regards,

            Ntem Kenyor K.
            Lead
            Kwickerhub


fix golint errors

Signed-off-by: sph<EMAIL_ADDRESS>
What this PR does / why we need it:
Which issue(s) this PR fixes (optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close the issue(s) when PR gets merged):
Fixes #
Special notes for your reviewer:
Release note:

/ok-to-test

/lgtm
/approve


Add a resource for managing the motd

Core Chef Resource Checklist
Before suggesting a resource for inclusion please make sure your suggestion meets these criteria for resources built into Chef:

[ ] Automates an operating system component that ships by default on systems such as authentication, raid, disk partitions, firewalls, containers, or virtualization systems.
[ ] Does not attempt automate 3rd party applications such as database, web, or application servers, which are best suited for cookbooks due to their fast moving nature.

Describe the resource:
A resource that would allow you to set the content of the system's MTOD
Why should this be included out of the box?:
A lot of organizations want to set a security warning here and we should make that super simple.
What operating systems would it run on?
Linux at least

@tas50 I think there are two potential used cases for motd.

Providing the content inline with the resource for the motd (give a simple motd)
Providing a path to a file or template with the content for the motd (complex motd)

Example of providing a simple motd
motd 'message name' do
  content "This is a secure system. blah blah blah"
  action :create # default action; other action :delete
end

Example of providing  a source for a more complex motd
motd 'message name' do
  provide_source true # set to true to pass template/file (defaults to false)
  cookbook 'current_cookbook' # defaults to current cookbook
  source 'path/to/template.erb'
  action :create # default action; other action :delete
end

@tas50 simple motd resource functional for linux + windows: https://github.com/devopsdina/motd
Just need help moving this into core!

You should be able to skip the provide_source bit and just use the source if a source is set. Other than that the design in the docs here lookos good to me.

@tas50 Simple and Complex motd should be working now: https://github.com/devopsdina/motd
Tested with the latest chef client (16.2), with the new resource changes https://docs.chef.io/release_notes/#breaking-change-in-resources, and backwards down to chef 14


Document that systemd.services..script is the same as systemd.services..serviceConfig.ExecStart

Motivation for this change
To reduce potential confusion.
#nixos irc interaction that motivated this:
xd1le        ┃ What's the difference between systemd.services.<name>.script and
             ┃ systemd.services.<name>.serviceConfig.ExecStart ? Not sure which
             ┃ to use.
dash         ┃ xd1le: nothing really
xd1le        ┃ dash: hmm ok then thank you
dash         ┃ xd1le: aha, here is the answer:
             ┃ https://github.com/NixOS/nixpkgs/blob/master/nixos/modules/system/boot/systemd.nix#L256
dash         ┃ xd1le: 'script' gets written to a file and the filename put in
             ┃ ExecStart.
xd1le        ┃ dash: nice, thank you!
xd1le        ┃ dash: I feel like this should be documented to reduce future
             ┃ confusion
dash         ┃ xd1le: awesome, you should do that :)

Things done

[ ] Tested using sandboxing
(nix.useSandbox on NixOS,
or option build-use-sandbox in nix.conf
on non-NixOS)
Built on platform(s)

[ ] NixOS
[ ] macOS
[ ] Linux

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Nitpick: saying one is an alias for the other is strictly not true.

Actually agreed, it was late :/
I'll update this later don't merge.

"Functionally equivalent"?

Actually I'm not even sure if serviceConfig.ExecStart is 'functionally' the same either, because I'm writing a service, and in there when I use serviceConfig.ExecStart, the service fails to run with Exec format error. But when I use script with the same command it works. I might be wrong about that (lmk if I am), but closing based on this.

@vyp: "ExecStart" must be the absolute path to a program, plus arguments (see https://www.freedesktop.org/software/systemd/man/systemd.service.html#ExecStart=). "script" are shell commands that NixOS places in a shell script and then passes the abosolute path to that script to "ExecStart".
In nixpkgs/nixos/modules/system/boot/systemd.nix you'll find this code:
          { serviceConfig.ExecStart = makeJobScript "${name}-start" ''
              #! ${pkgs.stdenv.shell} -e
              ${config.script}
            '' + " " + config.scriptArgs;
          })

if that makes it clearer.

Ah thanks so much @bjornfor, that makes it much clearer. I saw that but I didn't realize ExecStart commands have to use absolute paths. So in that case script is clearly different to ExecStart, and my confusion or curiousity came from my lack of familiarity with systemd rather than nixos making anything confusing or anything. So still nothing here to fix, but again, thanks for clearing things up!


Site not working since 3-4 days

When I enter my name in box, It shows Error 500.
http://spojtool.appspot.com/user/deependrapatel

Spoj changed its interface recently causing the scraping logic to fail. Am on way to fix it.

The plaintext version of user submissions has been restricted to private access by SPOJ. This was earlier used to compute almost all the stats of the user using just a single HTTP request. I have mailed spoj asking clarification if this change was a result of a bug or deliberate. waiting for reply.

Since fetching and parsing submission history pages is not feasible, I can modify spojtool to use whatever user data which is available for comparison between users.


Add retries with autostartToken

Is your feature request related to a problem? Please describe.
Authentication with autostartToken fails too fast.
What area is it related to
BankID
Describe the solution you'd like
Retry with new token 5 times as described by @smuda

When requireAutostartToken is used the timeout for BankID is currently 30 seconds. Checkout https://demo.bankid.com/Logon.aspx?environment=Produktion where they handle the timeout by generating a new transaction automatically up to five times.

https://github.com/ActiveLogin/ActiveLogin.Authentication/pull/138#issuecomment-549234589
Additional context
We should do this both for QR codes and same device authentication since they both use the autostartToken.

Sorry but I had to push a naive approach 😇

Fixed in #154


[Bug]: GroupChatManager.a_run_chat does not handle NoEligibleSpeaker Exception

Describe the bug
According to the code in GroupChatManager.run_chat when GroupChat.select_speaker raise NoEligibleSpeaker , the groupchat will be terminated. This feature enables coders to define termination conditions in customized speaker_selection_method.
def run_chat(
        self,
        messages: Optional[list[dict]] = None,
        sender: Optional[Agent] = None,
        config: Optional[GroupChat] = None,
    ) -> tuple[bool, Optional[str]]:
        """Run a group chat."""

         # other codes before ...

        for i in range(groupchat.max_round):
            self._last_speaker = speaker
            groupchat.append(message, speaker)
            # broadcast the message to all agents except the speaker
            for agent in groupchat.agents:
                if agent != speaker:
                    self.send(message, agent, request_reply=False, silent=True)
            if self._is_termination_msg(message) or i == groupchat.max_round - 1:
                # The conversation is over or it's the last round
                break
            try:
                # select the next speaker
                speaker = groupchat.select_speaker(speaker, self)
                if not silent:
                    iostream = IOStream.get_default()
                    iostream.print(colored(f"\nNext speaker: {speaker.name}\n", "green"), flush=True)
                # let the speaker speak
                reply = speaker.generate_reply(sender=self)
            except KeyboardInterrupt:
                # let the admin agent speak if interrupted
                if groupchat.admin_name in groupchat.agent_names:
                    # admin agent is one of the participants
                    speaker = groupchat.agent_by_name(groupchat.admin_name)
                    reply = speaker.generate_reply(sender=self)
                else:
                    # admin agent is not found in the participants
                    raise
            except NoEligibleSpeaker:
                # No eligible speaker, terminate the conversation
                break

        # other codes after ...
        return True, None

However, it seems that GroupChatManager.a_run_chat do not have this feature.
I am not sure whether it is a feature or bug.

async def a_run_chat(
        self,
        messages: Optional[list[dict]] = None,
        sender: Optional[Agent] = None,
        config: Optional[GroupChat] = None,
    ):
        # other codes before ...
        for i in range(groupchat.max_round):
            groupchat.append(message, speaker)

            if self._is_termination_msg(message):
                # The conversation is over
                break

            # broadcast the message to all agents except the speaker
            for agent in groupchat.agents:
                if agent != speaker:
                    await self.a_send(message, agent, request_reply=False, silent=True)
            if i == groupchat.max_round - 1:
                # the last round
                break
            try:
                # select the next speaker
                speaker = await groupchat.a_select_speaker(speaker, self)
                # let the speaker speak
                reply = await speaker.a_generate_reply(sender=self)
            except KeyboardInterrupt:
                # let the admin agent speak if interrupted
                if groupchat.admin_name in groupchat.agent_names:
                    # admin agent is one of the participants
                    speaker = groupchat.agent_by_name(groupchat.admin_name)
                    reply = await speaker.a_generate_reply(sender=self)
                else:
                    # admin agent is not found in the participants
                    raise
           # It does not have the following exception handler
           #  except NoEligibleSpeaker:
           #     break

            if reply is None:
                break

       # other codes after ...

Steps to reproduce
Define a speaker_selection_method returning None under some conditions. ( That should be a proper case when we try to define the termination condition
Model Used
No response
Expected Behavior
No response
Screenshots and logs
No response
Additional Information
No response

Thanks @linmou, I have addressed this in my Telemetry Phase 1 code, if you need it more urgently then I'll create a new PR. Telemetry Phase 1 #296

Thanks @linmou, I have addressed this in my Telemetry Phase 1 code, if you need it more urgently then I'll create a new PR. Telemetry Phase 1 #296

Not so urgent , I can change my code locally.

I'm also addressing this in #315, as I think that will merge earlier than #296.


Relative build.yml and Dockerfile paths not generating the Dockerfile.generated at the right location

Having the following (simplified) build.yml:
build:
  version: 2016-03-14
  steps:
    builder:
      name: builder
      dockerfile: builder/Dockerfile

And when using the following command:
sudo habitus --host=unix:///var/run/docker.sock --binding=<IP_ADDRESS> --keep-all --no-cache -f builder/build.yml (and probably a simpler version of it as well, didn't try out)
the following occurs:
2017/01/06 17:00:49 ▶ Using 'builder/build.yml' as build file
2017/01/06 17:00:49 ▶ Collecting artifact information
2017/01/06 17:00:49 ▶ Building 2 steps
2017/01/06 17:00:49 ▶ Starting API on 8080
2017/01/06 17:00:49 ▶ Step 0 - builder: builder
2017/01/06 17:00:49 ▶ Step 1 - app-base: app-base
2017/01/06 17:00:49 ▶ Parallel build for builder
2017/01/06 17:00:49 ▶ Building builder
2017/01/06 17:00:49 ▶ Parsing and converting 'builder/Dockerfile'
2017/01/06 17:00:49 ▶ Writing the new Dockerfile into builder/Dockerfile.generated
2017/01/06 17:00:49 ▶ Building the builder image from Dockerfile.generated
2017/01/06 17:00:49 ▶ Build for step builder failed due to API error (500): {"message":"Cannot locate specified Dockerfile: Dockerfile.generated"}

We can clearly see in the above log that Habitus is writing to builder/Dockerfile.generated but then tries to read from Dockerfile.generated instead.
Being able to run Habitus from a parent directory using the -f argument would allow to share resources (like configuration files) in a common top-level folder.

thanks for the bug report @chdeliens

@chdeliens I found the issue with your example, because your build.yml is in the builder directory you need to change your build.yml to the following example and it works:
build:
  version: 2016-03-14
  steps:
    builder:
      name: builder
      dockerfile: Dockerfile

-f is telling to use builder/build.yml but you at the moment it's using /builder as the reference, you don't need to prefix the Dockerfile with the /builder directory.

Related issue, not sure if it should be a new issue.
Wanted to have individual Dockerfiles exist in their own subdirectories, so they could all keep the name "Dockerfile" rather than be "Dockerfile.". This allows for syntax highlighting of the Dockerfiles in editors like PyCharm and Sublime Text.
However, if I have the following directory structure:

.
├── build.yml
├── mf_builder
│   ├── Dockerfile
│   └── Dockerfile.generated
├── run.sh
├── scripts
│   ├── cloneRepos.sh
│   ├── installruby.sh
│   ├── installruby_docker.sh.bak
│   └── ucplayer_build.py
└── ucplayer_builder
└── Dockerfile

And build.yml:
build:
  version: 2016-03-14 # version of the build schema.
  steps:
    mf_builder:
      name: mf_builder
      dockerfile: mf_builder/Dockerfile
    ucplayer_builder:
      name: ucplayer_builder
      dockerfile: ucplayer_builder/Dockerfile
      depends_on:
        - mf_builder
      secrets:
        id_rsa:
          type: file
          value: _env(HOME)/.ssh/id_rsa

I end up with an issue similar to @chdeliens.
Was wondering if Habitus can already support this directory structure, perhaps with some changes to the build.yml or having separate build.yml files with each Dockerfile, or if I must stick with one build.yml in the same directory as all Dockerfiles.

I second the previous poster's request. Please reopen this issue.
I have complex Dockerfiles each in it's own directory.
Will Habitus support that anytime soon?

Hi @GertThiel,
I did some work on the next release to fix this issue. Can you try your example with the pre-release please. You can download the pre-release here: https://github.com/cloud66/habitus/releases/tag/0.4.11_pre
Please let me know.
Daniël,

That helped @kamerblauwlicht. Thanks a lot.


Add programme for Day 2 of 2018 Users meeting

Couple of initial decisions open for discussion in addition to the usual spelling/formatting review:

I re-used the same CSS layout as the previous Users meeting
for the morning workshops, the event-title class is used to highlight the workshop theme
the program only includes workshops titles for now. Bullet points can be added in a second step or once final
the last workshop is expected to grow as we receive interest from the community

Staged at https://snoopycrimecop.github.io/www.openmicroscopy.org/events/13th-annual-users-meeting-2018.html.

Staged at https://snoopycrimecop.github.io/www.openmicroscopy.org/events/13th-annual-users-meeting-2018.html for review

Looks good 👍

R notebook has been moved to handle metadata

just an idea while driving back to get kids
for metadata workshop: The journey of metadata


Set mime type for template

Hello,
I would like to know if there is a way to specify some HTTP properties, like the mime type.
AFAIC, I have a feed template that generates a RSS feed (I know there is a plugin for that, but I prefer the template way, mainly because I use a theme that does it that way). I with there would be a way to indicate that this template should use the header Content-Type: application/rss+xml.

You can't modify headers using Twig, you'll have to write a Pico plugin for that.

I can imagine a dictionnary in the configuration enabling to specify various settings relative to certain templates, not only HTTP headers. For instance, title levels shift could be defined that way (see issue #543).
This could be like this:
templatevars:
    post:
        type: text/html
        titleshift: 2
    feed:
        type: application/rss+xml
    default:
        [...]


Update ListOperation.php

https://github.com/Laravel-Backpack/CRUD/pull/3015/commits/df4ff78408084b5d6bf339b7f6575f1dbb155b8c commit crashed our app because we use database prefix (i know, i will never do it again).
This PR fix it. This method return empty string in case not using prefixed database names.

A new inspection was created.

The inspection completed: No new issues

Hi @mgralikowski
So, ->order('field','direction') and ->orderByRaw('table.field','direction') produce different results ?
From my understanding, and I might be wrong, but if you have a 'prefix'    => 'some_prefix_' in your connection array configuration, when using ->getTable() it should return some_prefix_«table_name_in_model»
You can define multiple connections with different prefixes and use in model $connection = 'connect_with_different_prefix
Let me know,
Pedro

getTable return only table name without prefix so it is not real column name in database.

Yes, we have fatal db error. All tables are prefixed besides in raw part.
getTable return only table name without prefix so it is not real column name in database.

Do you have that prefix configured in your connection configuration file ? config\database.php

  'mysql' => [
            'driver'                  => 'mysql',
            'url'                     => env('DATABASE_URL'),
            'host'                    => env('DB_HOST', '<IP_ADDRESS>'),
            'port'                    => env('DB_PORT', '3306'),
            'database'                => env('DB_DATABASE', 'forge'),
            'username'                => env('DB_USERNAME', 'forge'),
            'password'                => env('DB_PASSWORD', ''),
            'charset'                 => 'utf8mb4',
            'collation'               => 'utf8mb4_unicode_ci',
            'prefix'                  => 'laravel_',
            'prefix_indexes'          => false,
            'foreign_key_constraints' => env('DB_FOREIGN_KEYS', true),
            'strict'                  => true,
        ],

Ugh... Thank you @mgralikowski . Sorry for breaking your app 😄 😇
@pxpm now I remember there's this bug in Laravel where getTable() doesn't actually get the prefixed table name. It's a weird one, that apparently never got fixed and never will - can't really remember why they chose to do it like this. Maybe the local prefix gets applied but not the global one? Can't remember. But yeah...
As mentioned in my other comment - we have getTableWithPrefix() inside CrudTrait which should, in theory, also fix this. Without needing to call DB.
@mgralikowski - can you please confirm that using getTableWithPrefix() instead of getTable() fixes this for you too?
Cheers!

Sorry guys, didn't know about that Laravel limitation, it just does not make sense if you have a global database prefix that when you call model get table it does not get the prefixed one. Technically the one returned would never be the model table, that table name without prefix would simply not exist , why on hell would laravel return it ?
Always learning. In this scenario I agree with @tabacitu and we should use getTableWithPrefix()
Submitting a PR to address this.
Best,
Pedro

I also agree that using global DB is not the best option so i fixed method getTableWithPrefix
https://github.com/Laravel-Backpack/CRUD/pull/3126
I same moment I send PR 1:1 as @pxpm so I am closing it.


Added a scroll down button

Added a scroll down button to scroll down the page

There is no pr template or issue attached to this. Closing pr.


Cifar-10 global average pooling

In imagenet_model you defined resnet_v2 with global_pool,
is there any reason that cifar model doesn't contain global average pooling?
Am I missing something?
or the part
https://github.com/mfigurnov/sact/blob/6c45317228705cc6a0b27c48574ead08e7dfccda/cifar_model.py#L86
is doing what I am looking for?

The global average pooling for CIFAR is done on this line: https://github.com/mfigurnov/sact/blob/master/cifar_model.py#L134

Thanks!


Using sleepy makes the dynamic body stop and shake continuously

Hi,
I am creating a dynamic body this way -
const el = document.createElement('a-entity')
        el.setAttribute('id', this.data.id);
        el.setAttribute('gltf-model', this.data.src)
        const offset = new THREE.Vector3(rg.real(-jx,jx), rg.real(0.5,jy), rg.real(-jz,jz));
        el.setAttribute('position', center.clone().add(offset));
        el.object3D.scale.set(0.01,0.01,0.01);
        el.setAttribute('dynamic-body', 'shape:sphere; sphereRadius: .3; mass: 5');
        el.setAttribute('velocity', '0 0 23');
        el.setAttribute('sleepy', true);
        console.log(el);
        sceneEl.appendChild(el);

But, the body becomes very slow, falls to the ground and starts to shake.
If I don't use "sleepy" then the body moves quickly, does not stop after collision and shaking happens in this case as well.
I am looking for a solution to make the body move quickly before the collision, come to a stop after a collision and the shaking needs to stop ( I realize that the shaking is independent of using the sleepy component)
I have spend countless hours trying to solve this. Could you please help?
Thanks,
Niraj

Sleep has a default setting to dampen movement.
Change it by decreasing the linearDamping and angularDamping attributes.
The shaking I suspect is caused by using the velocity component on a dynamic body. I would try using the applyImpulse/applyLocalImpulse Cannon body methods instead. See example here: https://github.com/quinn-madson/ars-attacks/blob/master/server/static/js/projectile.js


Refactor error messages

This PR:

refactors the <push> tag error so that it takes into account empty name attributes
updates some path-related error messages to be more succint and give a little more context

Let me know what you think, thanks!

@cossssmin thank you for this enhancements!
To me looks all good.
I would just change how to include get function from lodash.
From:
const get = require('lodash');

to:
const get = require('lodash/get');

Since according to docs and this article here this method seem to reduce bundle sizes:

Full import: 256.4 KB
Curly brackets: 256.54 KB
Module imports: 142.39 KB

I didn't test but I notice a change in bundle size after this change https://github.com/thewebartisan7/posthtml-components/commit/ecd7d54bffad05d8b92e4b1ebc7b5a881fc9dc42
Although there is not much to worry about the bundle size in this case. It is a fixation of mine.
I will merge the PR and release by Monday the stable release if there are not any other feedback from you.
Thank you!

Sure thing, will fix it now 👍


Issues

Hi, I didn't  download the Pre-trained model files [Shanghai Tech A], [Shanghai Tech A],Can you seed me the files to the email, My email is<EMAIL_ADDRESS>Thank you  very much

You can download the models from here:
https://www.dropbox.com/s/irho4laltre9ir5/cmtl_shtechA_204.h5?dl=0
https://www.dropbox.com/s/lkt5ipshibs027w/cmtl_shtechB_768.h5?dl=0
Let me know if you have any issues.

svishwa:
Thank you very much. In fact, I can't connect to this site. So I hope you can send this two models to my email. I want to learn this article better thought code. My email is caacsri_dangwanli@163.com,Thank you very much.

Sent!

Thank you very much!


Favicon not present

Add Favicons to Website
I'm always frustrated when I can't easily identify our website in my browser tabs because it lacks a favicon.
Describe the solution you'd like
Add a favicon to the website to make it easily recognizable in browser tabs and bookmarks.
Please assign this issue to me under SSOC

@Trident09  Someone already fixed this issue.


Body text or  CSS styling for web content

[ I cannot find in the style guide best practice (CCS) for body font size, weight, and color. I DO see  "minimum" line spacing for various font sizes (e.g. 14 pt body text - 120% line spacing minimum).
As an example, for a SharePoint Text web part, what are some guidelines for  body text  style and font-weight?
I believe the online Microsoft Writing Style Guide, styles the  text/ body text as following:
font-size: 16px
font-weight: 400
line-height: 160%
color: rgb (0,0,0)
I am thankful for any insight, opinions, or guidance you can provide.
Tallis Lockos]

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

ID: 1c4ab3f5-d7de-1853-6f11-16e761b9698e
Version Independent ID: 3901f044-2b3e-5f86-79b8-0cce3382c49c
Content: Welcome - Microsoft Style Guide
Content Source: styleguide/welcome/index.md
Product: non-product-specific
GitHub Login: @pallep
Microsoft Alias: pallep

Hi Tallis,
In general, the Microsoft Writing Style Guide covers editorial issues like word choice, capitalization, and punctuation. A site's web or UI designer is the best resource for the details of text styling, like font and font size.
I'm also replying to your question in a separate email in more detail.


[CSS] Reusing Clarity colors

Hi,
In the previous version of Clarity there was a file called "dependencies.clarity" which included all clarity variables/mixins at one place. We used it to include it in our own components' sass files in order to use clarity's variables/mixins.
In the latest version there is such file. What is your recommendation for consuming Clarity's variables/mixins?

What version are we talking about here? We have issue templates to help fill out useful information like a reproduction if there is a bug or answer questions like what versions of Clarity and Angular you are using, please use them in the future to help organize content better. There is some theming documentation coming for v3 which hasn't landed yet, and probably will address these questions unless its about another version of Clarity.

Hello @ttsvetko, due to inactivity on this issue we're closing it per our issue management policy. If you are able to provide additional details and a reproduction of the issue, we can reopen the ticket as necessary.


[TW#18869] BLE Passive Scan Channels

BLE devices advertises on 3 channels, using frequency hopping. Does ESP32 scan the 3 channels simultaneously, or just monitor 1 channel? It seems that I can't scan some devices continously as it seems they disappear after few seconds, probably hopped to another channel, then come back up visible by ESP32 after few secs.

@hopkinskong Our ESP32 can scan the all adv channels. of course you can scan one of the channels by setting scan params. you can refer to gatt_client demo in ESP-IDF.
static esp_ble_scan_params_t ble_scan_params = {
.scan_type              = BLE_SCAN_TYPE_ACTIVE,
.own_addr_type          = BLE_ADDR_TYPE_PUBLIC,
.scan_filter_policy     = BLE_SCAN_FILTER_ALLOW_ALL,
.scan_interval          = 0x50,
.scan_window            = 0x30
};
If "scan_interval" is equal to "scan_window", the scanner will scan  continously.

@Weijian-Espressif Does it also work on BLE_SCAN_TYPE_PASSIVE?

Advertising and scanning both rotate through the channels sequentially so there is a chance you could miss advertisements.
https://devzone.nordicsemi.com/cfs-file/__key/communityserver-blogs-components-weblogfiles/00-00-00-00-04-DZ-782/1273.Scan.PNG

@negativekelvin Is there no workaround for this issue, except deploying multiple scanners?

If your scan window is > longest advertising interval of devices you are scanning and you are continuously scanning then you should receive them

Hi @hopkinskong , did the above suggestion help you resolve the issue?

@FayeY Need to test, closed the issue for now. Will reopen if there is still issues.

@FayeY @Weijian-Espressif
If .scan_interval == .scan_window, a persistent TCP connection will be timed out, and wifi connection might loss:
I (26642) wifi: bcn_timout,ap_probe_send_start
HTTP throughput is unacceptable (200KB transfer > 120s, and disconnection in the middle)
I am referencing to this issue #1639:
Optimizing BLE scan will affect TCP performance a lot
Optimizing TCP connection will affect BLE performance too...
I need to deploy my application on 16x ESP32-WROVER and I really need to get this issue solved ASAP.

@hopkinskong , if you set .scan_interval == .scan_window, ble always use the  phy, it will case wifi disconnection.

@Weijian-Espressif Exactly, how to optimize the parameters for best performance of both Wi-Fi + BLE?
@negativekelvin I concur. I tried SW/HW COEXIST option in sdkconfig but it doesn't seem improve much performance.

if you want to use both Wi-Fi + BLE, You need to pay attention to the scan parameter or adv parameter.
BLE and sharing a phy and BLE has a higher priority. When ble is using the phy, wifi will wait until the phy is idle. So You need to pay attention to the scan parameters, .scan_interval == .scan_window is not allowed in ble & wifi coexistence .

@Weijian-Espressif phy-controlled scan would be better, it could scan continuously except for rx dtim or tx.  Basically modem sleep but instead of sleep, scan.  Beacon scanner is a common use case.

@jkhax0r Good to know I am not alone.
@Weijian-Espressif Is there any recommended configuration parameters to maximize the performance of both BLE and Wi-Fi?

related to #1639

@TianHao-Espressif Is it this commit?
https://github.com/espressif/esp32-wifi-lib/commit/608bf369c66c33c15b0cc198e573e421c7103e1f
What is the acceptable range for testing?  .scan_interval == .scan_window is still not expected to work, right?

@TianHao-Espressif Also, do I need to enable CONFIG_SW_COEXIST_ENABLE?

@TianHao-Espressif With CONFIG_SW_COEXIST_ENABLE = YES, the performance is very bad. After application start, TCP connection disconnects* in 30-45s.
With CONFIG_SW_COEXIST_ENABLE = NO, the performance is much more better, now running at least 3 minutes without issues.
Note that my BLE scans every 1s, with the following parameters:
.scan_type              = BLE_SCAN_TYPE_PASSIVE, 		.own_addr_type          = BLE_ADDR_TYPE_PUBLIC, 		.scan_filter_policy     = BLE_SCAN_FILTER_ALLOW_ALL, 		.scan_interval          = 0x550,  		.scan_window            = 0x380
*Disconnects means the TCP data stream is idle for more than 11s (with is REALLY high value)

@hopkinskong ,
two choices:

if disable CONFIG_SW_COEXIST_ENABLE, you should keep "scan interval - scan windows > 150ms", it can works.
if enable CONFIG_SW_COEXIST_ENABLE, any scan interval and scan windows value. And for your requirement, you can try "prefer to wifi".

For our  internal test, the tcp do not disconnect while bluetooth run A2DP for serval hours.

@hopkinskong , and do you do git submodule update or could you provide me your whole log?

@TianHao-Espressif
What do you mean "prefer to wifi"? Where can I set "prefer to wifi"?
Now my problem remains poor TCP performances. I need to send ~200KB data every 0.5~1s, they are JPEG data from the camera, the connection will just stuck due to delay and low throughput. The performances are just very bad.

https://github.com/espressif/esp-idf/commit/9b7454de0c171a7f94dd8f42a5453ef6c75370e0#diff-0b068dd317b997485fff4a56e7fb7ee8

@negativekelvin, thanks.
@hopkinskong , as @negativekelvin wrote, please update your esp-idf and choose
-> Component config                                                                                -> Wi-Fi                                                                                -> Software controls WiFi/Bluetooth coexistence (SW_COEXIST_ENABLE [=y])                                                                                -> WiFi/Bluetooth coexistence performance preference (SW_COEXIST_PREFERENCE [=y])

hi @TianHao-Espressif i just want to ask if there's  an example to test the performance of WiFi/BLE coexistence provide in idf or would be okay to share the code you used to test the performance WiFi/BLE coexistence , as currently i have stability problem in both of them , my code is just scanning for beacons and have MQTT for another usage

I've tried all combinations of settings suggested in this thread and cannot get any reliability. I wish to continuously scan for BLE broadcasts, filter and decode some of them, and transmit over MQTT.
Typically this works for about a minute, then the MQTT connection goes down, presumably because of interference from the bluetooth radio.
Does anyone have an example that is working reliably?

@lukecyca What scan_interval and window_interval values did you try? Usually beacons and mid performance ble device should have advertising interval around 250ms, so lets assume you may want to scan for devices with scan_interval around 250-350. Now we have 3 channels, so you can set window_interval around 3*350~=1200ms. I would start with those values:

scan_interval = 350ms
window_interval = 1200ms

Thanks for the reply @chegewara.
I've tried:

.scan_interval = 0x50, .scan_window = 0x30 as per the example
.scan_interval = 0x550, .scan_window = 0x380 as per @hopkinskong's comment and following
@TianHao-Espressif's advice for scan_interval - scan_window > 150.
.scan_interval = 0x350, .scan_window = 0x1200 but I'm pretty sure the interval must be larger than the window (per the BLE 4.0 core spec). So I also tried the reverse in case that's what you meant.

In all cases it fails in one of two ways:

No BLE packets are captured at all (but MQTT is working and is transmitting other telemetry); or
Packets are captured and transmitted over MQTT for 10-20s and then MQTT connect fails. I have it reconnect after 10s and it works again for 10-20s, then fails again.

I am still using esp-idf 3.0.3 (so no software co-existance) for the above.
I also tried esp-idf 3.1 with software co-existance (preferring wifi) and repeated all of the parameters above, with the same result.
Setting a larger interval reduces the number of BLE packets received (makes sense) but otherwise seems to have no effect on whether the MQTT connection gets clobbered.

You're right, scan_interval should be greater value, my bad.
When you set scan_interval to 0x50 (50ms) then you will miss most of advertising packets/devices,but its your app and you know better what you need.
I have arduino app on esp32 that is connected with aws IoT over mqtt and when is connected i am performing scan for ble thermal printer and connect to it. My app is subscribing to individual topic, then every few seconds i am publishing some value and receiving response on subscribed topic. Depends on response value i am printing or not some message on thermal printer. I had no issue with this solution except it requires a lot of heap and its very hard to fit it on wroom module.
There is few difference between my and your app, i scan only for few seconds and send just a few bytes over wifi.


Function for pruning missing data

I would like to propose a function that removes features and languages in a ValueTable that don't meet a certain cut-off for missing data. The approach is similar but not identical to what we did in grambank-analysed for the release paper. I think this approach is more elegant, easier to understand and overall better. Instead of doing it step-wise, the cropping is done in one sweep in one pipe.
I couldn't get the testthat test to run without using library() at the start and source the relevant script with the function. I welcome help in remedying this. For now I've commented out the library and sourcing in the test script.

Removing data based on two orthogonal criteria is problematic, I think. Naively, I'd assume that after pruning with cutoffs x and y, all languages in the dataset will have a coverage >= x% features, and all features will have a coverage >= y% languages. However, that isn't necessarily true, because whatever is done last may also remove datapoints which were necessary for the other threshold to be cleared.
So I'd think, the only way pruning based on low coverage can be supported in a transparent way is by having two functions, one for each criterion, and letting the user deal with issues due to applying both.

@SimonGreenhill  perhaps we should not have this function at all and instead encourage users to crop as they see fit for each function necessary. In the meantime, I'll implement an approach to make_theo_scores() that crops per set of features missing values. I'll make a PR for it in a sec.

It seems @SimonGreenhill doesn't want this function from the grambank release paper in rgrambank, I'm closing the PR.


Emit ETag header to result

It would be useful to get the ETag header in all reading requests. Thus, caching information can be used on application level.
Perhaps you could add a third optional parameter to the result callback: function(err, doc, metadata).
This could encapsulate additional meta data like the request ETag header value.

Isn't the ETag basically equal to the document's _rev attribute?

That's right for documents. But it is not available in view result sets.

Ah, I see.  Didn't know that's what you were talking about.
On Fri, Feb 17, 2012 at 4:00 PM, Benjamin Erb <
<EMAIL_ADDRESS>
wrote:

That's right for documents. But it is not available in view result sets.

Reply to this email directly or view it on GitHub:
https://github.com/cloudhead/cradle/issues/7#issuecomment-4016584


Cannot add version 2.0.0

this Error happen when use v2.0.0:
Manifest merger failed : Attribute application@appComponentFactory value=(android.support.v4.app.CoreComponentFactory) from [com.android.support:support-compat:28.0.0] AndroidManifest.xml:22:18-91
is also present at [androidx.core:core:1.0.1] AndroidManifest.xml:22:18-86 value=(androidx.core.app.CoreComponentFactory).
Suggestion: add 'tools:replace="android:appComponentFactory"' to  element at AndroidManifest.xml:11:5-49:19 to override.

Image Picker Version:2.0.0
Android OS:23
Phone:

I think this happen because your projects haven't yet migrate to AndroidX
Can you try adding these two lines in gradle.properties
android.useAndroidX=true
android.enableJetifier=true

Closing the issue do inactivity.
Will add some info in readme


goPackages.buildFromGitHub missing

Hey, go users, is there a reason why buildFromGitHub isn't exported? I would like to have an externally defined/overridden package and this is missing for me. I'll push it in a few days...

No reason, as usual :)

@vcunat I wrote it mostly because I saw the exact same pattern repeated a thousand times in goPackages, so I just factored it out without giving much thought to go packages specified elsewhere. Please export/generalize/do whatever is needed to improve things :smile:

It was rather pointless of me to open this issue instead of pushing directly...

:)
On Thu, Nov 5, 2015 at 10:11 Vladimír Čunát<EMAIL_ADDRESS>wrote:

It was rather pointless of me to open this issue instead of pushing
directly...
—
Reply to this email directly or view it on GitHub
https://github.com/NixOS/nixpkgs/issues/10805#issuecomment-154087621.


UV Layers not properly displaying/working after updating the model

Not sure if it's a blender problem or a TS problem but when exported from blender and the model has 2 UV layers, only one can display while the other renders null
Road texture UV:

Shadow bake UV:

The road texture UV being displayed and working properly but not the shadow uv:

Blender 3.5 and higher issue. Their dae exporter exported to the wrong UV layer. A work around with update https://github.com/MapStudioProject/Track-Studio/commit/2f1d02a898155aaa7ee2d9af9e6e5f0bded84040


show who posts what

It's awesome if we can see who post what right?
Good to use these?
https://docusaurus.io/docs/en/site-config#enableupdateby-boolean
https://docusaurus.io/docs/en/site-config#enableupdatetime-boolean

Thanks for https://github.com/baby-degu/portal/pull/14 I'll close this issue.


Remove "Portfolio" Text from website

It seems unnecessary as point out by sreekaransrinath at this issue

Completed

Completed


MSSQL Support

Would be nice to see MSSQL support, though this may well require adaptation of knex.  The structure isn't particularly more complex than the differences with the already supported data stores.

This is entirely a knex issue, please open an issue there. If you would like to add support you can check out how other dialects are impemented.


facing this error while installing aks cli

This is autogenerated. Please review and update as needed.
Describe the bug
Command Name
az aks install-cli
Errors:
The command failed with an unexpected error. Here is the traceback:
HTTP Error 403: rate limit exceeded
Traceback (most recent call last):
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\knack/cli.py", line 231, in invoke
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 657, in execute
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 720, in _run_jobs_serially
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 691, in _run_job
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/__init__.py", line 328, in __call__
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/core/commands/command_operation.py", line 121, in handler
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/acs/custom.py", line 355, in k8s_install_cli
  File "D:\a\1\s\build_scripts\windows\artifacts\cli\Lib\site-packages\azure/cli/command_modules/acs/custom.py", line 442, in k8s_install_kubelogin
  File "urllib\request.py", line 222, in urlopen
  File "urllib\request.py", line 531, in open
  File "urllib\request.py", line 640, in http_response
  File "urllib\request.py", line 569, in error
  File "urllib\request.py", line 502, in _call_chain
  File "urllib\request.py", line 649, in http_error_default
urllib.error.HTTPError: HTTP Error 403: rate limit exceeded

To Reproduce:
Steps to reproduce the behavior. Note that argument values have been redacted, as they may contain sensitive information.

Put any pre-requisite steps here...
az aks install-cli

Expected Behavior
Environment Summary
Windows-10-10.0.19041-SP0
Python 3.8.9
Installer: MSI

azure-cli 2.29.0 *

Additional Context

route to service team

@phealy

I'm assuming you're using az aks install-cli in a script or CI/CD pipeline of some sort that's running it repeatedly from the same system

In our case, we're running this on bitbucket cloud ci pipelines. We are only running this occasionally, though you're probably right, that other companies have probably also been running the same command repeatedly on the same shared runner.
We're using the Azure CLI Docker container.

Preferred: get kubectl/kubelogin into your pipeline using an alternate method that doesn't download them every time, incurring bandwidth and processing on the hosting servers for every run.

We'd love that as well. The reason why we need to call az aks install-cli in the first place, is that the azure-cli container doesn't come with it pre-installed.
Our much preferred option would be Microsoft publishing a second variant of the azure-cli container that already includes all of the related addons (like in this case the aks cli). Then we would just use that docker container as the base for our pipeline (and have that cached in Bitbucket, without the outbound network traffic for each pipeline run).


fix bug that stuck auto fetch when pull failed & make stage area resizeable

fix bug that stuck auto fetch when pull failed.
make stage area resizeable.

I've manually picked your changes. Please make sure your PR is based on latest develop branch and the target branch of your PR is develop


Using it on Quest 3 ?

hello, do u have a how to using it the demo on a quest 3 ?
thanks

Hello! Yes, all works fine on Quest 3

Hello, did you download the apk from the latest release and it doesn't work? Or you tried to build the apk and something went wrong?

Sorry where is the apk for quest 3 and how to install it on quest 3 Thanks

Hello,
ok so i download the version here :
https://github.com/jetstyle/Apple-Vision-Pro-UI-Kit-Demo/releases/tag/v1.2.0
look greats


Added option typer arguments

Similar to selector arguments, I added the option to define arguments for the typer. This allows for setting options such as the delay between key presses.
I also updated the README to include ydotool and describe the usage of --typer-args

Hey, thank you! 🙂
You mentioned delay between key presses. Is this something we could set by default? Do you have any other use cases in mind?
I'm just asking because configuration always adds complexity, and if we can avoid it with good defaults, that's often preferable.

Hi,
Yeah, no worries, the only use case I had in mind was the delay between key presses. I only went with allowing any arguments because it would be more consistent with what was done with --selector-args? But I am more than happy to reduce the complexity to just control the delay between key presses :slightly_smiling_face:
I am unsure what a sensible default would be, though. xdotool and ydotool have a default of 12ms, but wtype has a default of 0ms. I am unsure why 12ms was chosen, and cannot seem to find anything on Google that explains it. Personally, I use xdotool and would like the delay to be 0ms (as this prevents me accidentally hitting enter, to log in, before the password is typed). I do not see why the default would not be 0ms, unless it causes some issues? What would you suggest is a sensible default?

I agree, if we offer this parameter, it's better to make it generic for all kinds of arguments.
I could imagine that there are problems if you set it to 0, effectively sending everything without even a tiny break. Have you tried this, and did it work?
If there are no problems, I think we can just set it to 0 everywhere.

I have tried it as 0, and so far there have been no problems. I am just concerned about edge cases that I have missed.
Also, I am a bit confused how we would make it generic for any typer argument, but also have a default for just the delay? Would you like one option for key delay, and one for generic arguments? Sorry, this is my first proper pull request.
I have also tried using ydotool, but there is currently an issue with it, which means the key-delay doesn't work. I will have a look at trying to fix this though.

Sorry, this is my first proper pull request.

Awesome, I feel honored!  Would you like one option for key delay, and one for generic arguments?

If a general delay of 0 works, my plan would be to hardcode it and not offer any arguments to the typer, to reduce complexity (until someone needs it 😉).

I have tried it as 0, and so far there have been no problems. I am just concerned about edge cases that I have missed.

This is one of the cases where we can't try everything, so I'd be fine with merging a hardcoded 0 and waiting for bug reports.

Perfect, I just pushed the update to only offer the key delay parameter with a default of 0 (I wrote the code while we were discussing, so it's showing up earlier in the comments)

I haven't forgotten this PR, but I want to think a bit more if the added complexity with a new parameter is worth it, or if we should just hardcode 0 into the typers.

No worries, thank you

My current impression is that the delay is not a problem for most people. Still, we could reduce it to avoid the problems you described (and to make it faster).
Since this is also the only use case for --typer-args, that argument would add complexity for little benefit. --key-delay is a bit less complex, but still a new argument. So, before we add this complexity, I'd like to try hardcoding a delay of 0 (or a low value like 2). I expect this to solve the mentioned problems and work for everyone while having a simpler UX.
Only if that introduces new problems would I want to make the delay configurable.
So, do you want to hardcode 0 in this PR? I'm also not against keeping the previous commit (for --typer-args and for --key-delay) in case we'd need one of them later.

Would you prefer I use a variable for the hardcoded 0 and pass this through to the type_characters method, or just have the 0 hardcoded into the run parameters?
If using a variable is better, how would you like me to define it? I'm not too sure on conventions - would I use upper case with underscores to show it is a constant? And where would I define it - in the main method, the RofiRbw class or just in the rofi_rbw.py file?

Having it directly in the run parameters should be clear enough.

Thanks a lot! 🙂


Invalid results file when using allure report file in publish results test step

My main goal is to be able to build a report that includes attaching a screenshot from a failed test case to a report which can be uploaded in the Publish Test Results step.  I have used the allure reports before and it is set up to easily attach screenshots to a failed test result in the report file.
However after I run my build in TFS the Publish Test results step returns "Invalid results file. Make sure the Test Result Format field in the task matches the result format of the file:" I know that the allure reports follow the xUnit format however they also add in some supplementary data as well.  Attached
is the resulting xml file.
testsuite.txt
In short my question is does should TFS be able to read this report file?  Am I missing a step to allow TFS to read the file?

Hi @jwerkau,
In the Publish Test Results task, we parse the results file to get to the relevant nodes to get test cases information. For that we strictly follow the XSD schema for Xunit (http://xunit.github.io/docs/format-xml-v2.html). The file you have attached does not conform to the Xunit schema and hence the error.
Please let us know if you have any other questions.

Thanks @prawalagarwal.  Makes sense.  It is odd that they claim to use a standard xunit format but as you point out it is not following the XSD schema.  Rather than building the report and publishing through the publish results step I will install the Allure plugin in TFS and try generating the report that way in order to display the results.

Thanks @jwerkau. Closing the thread.


Update corectl-app to 0.2.7

Checklist

[x] The commit message includes the cask’s name and version.
[x] brew cask audit --download {{cask_file}} is error-free.
[x] brew cask style --fix {{cask_file}} left no offenses.

Merged as 41de782ff921abd9aeeb23ded70acb5a84cbfff4.