Using a custom theme

Hi there guys,
I bought a bootstrap theme and have been using it with Laracogs. I recently came across the select element and can't seem to generate a select element that uses the theme within blade.
This is because when I use @foreach directive I can't seem to be able to create a roles selection. I can only achieve it using the default installation and normal bootstrap.
Regards

Are you trying to make the input individually or with the form maker? Here are two examples:
FormMaker::fromTable('users', ['roles' => ['type' => 'select', 'class' => 'something-different', 'options' => $options]])
Or
InputMaker::create('roles', ['type' => 'select', 'options' => $options], null, 'something-different')

I've improved some of the customizability of the FormMaker parts - in the config you'll see:
        'group-class' => 'form-group',
        'error-class' => 'has-error',
        'label-class' => 'control-label',```

which can be used by default - then for custom inputs you can pass whatever custom classes are needed :)

Thank You so much! Laracogs is awesome.


Weird linking (?) issues

I have experiencing weird crashes when attempting to load keys to Rnp:
Crash 1
homedir = "/Users/skale/Projekty/ribose/enmail/tmp/gpgme"
Rnp.homedir_info(homedir)
#=> {:public=>{:format=>"KBX", :path=>"/Users/skale/Projekty/ribose/enmail/tmp/gpgme/pubring.kbx"}, :secret=>{:format=>"G10", :path=>"/Users/skale/Projekty/ribose/enmail/tmp/gpgme/private-keys-v1.d"}}
info = Rnp.homedir_info(homedir)
#=> {:public=>{:format=>"KBX", :path=>"/Users/skale/Projekty/ribose/enmail/tmp/gpgme/pubring.kbx"}, :secret=>{:format=>"G10", :path=>"/Users/skale/Projekty/ribose/enmail/tmp/gpgme/private-keys-v1.d"}}
format = info[:public][:format]
#=> "KBX"
path = info[:public][:path]
#=> "/Users/skale/Projekty/ribose/enmail/tmp/gpgme/pubring.kbx"
Rnp.new.load_keys(format: format, input: Rnp::Input.from_path(path))
# Crashes Ruby interpreter with:
# dyld: lazy symbol binding failed: Symbol not found: _pgp_new
#   Referenced from: /usr/local/Cellar/rnp/HEAD-f45f254/lib/librepgp.0.dylib
#   Expected in: flat namespace
#
# dyld: Symbol not found: _pgp_new
#   Referenced from: /usr/local/Cellar/rnp/HEAD-f45f254/lib/librepgp.0.dylib
#   Expected in: flat namespace
#
# fish: 'bin/console' terminated by signal SIGABRT (Abort)

Crash 2
Rnp.homedir_info(homedir)
#=> {:public=>{:format=>"KBX", :path=>"/Users/skale/Projekty/ribose/enmail/tmp/gpgme/pubring.kbx"}, :secret=>{:format=>"G10", :path=>"/Users/skale/Projekty/ribose/enmail/tmp/gpgme/private-keys-v1.d"}}
info = Rnp.homedir_info(homedir)
#=> {:public=>{:format=>"KBX", :path=>"/Users/skale/Projekty/ribose/enmail/tmp/gpgme/pubring.kbx"}, :secret=>{:format=>"G10", :path=>"/Users/skale/Projekty/ribose/enmail/tmp/gpgme/private-keys-v1.d"}}
format = info[:secret][:format]
#=> "G10"
path = info[:secret][:path]
#=> "/Users/skale/Projekty/ribose/enmail/tmp/gpgme/private-keys-v1.d"
Rnp.new.load_keys(format: format, input: Rnp::Input.from_path(path))
# Crashes Ruby interpreter with:
# dyld: lazy symbol binding failed: Symbol not found: _rnp_get_debug
#   Referenced from: /usr/local/Cellar/rnp/HEAD-f45f254/lib/librekey.0.dylib
#   Expected in: flat namespace
#
# dyld: Symbol not found: _rnp_get_debug
#   Referenced from: /usr/local/Cellar/rnp/HEAD-f45f254/lib/librekey.0.dylib
#   Expected in: flat namespace
#
# fish: 'bin/console' terminated by signal SIGABRT (Abort)

Background
The /Users/skale/Projekty/ribose/enmail/tmp/gpgme is a homedir created with GPGME.
There are six keys (three secret and three public) created with following script, neither is password-protected:
    <GnupgKeyParms format="internal">
    %no-protection
    Key-Type: DSA
    Key-Length: 2048
    Subkey-Type: ELG-E
    Subkey-Length: 2048
    Name-Real: Some Arbitrary Key
    Name-Email<EMAIL_ADDRESS>    Name-Comment: Without passphrase
    Expire-Date: 0
    %commit

    Key-Type: DSA
    Key-Length: 2048
    Subkey-Type: ELG-E
    Subkey-Length: 2048
    Name-Real: Cato Elder
    Name-Email<EMAIL_ADDRESS>    Name-Comment: Without passphrase
    Expire-Date: 0
    %commit

    Key-Type: DSA
    Key-Length: 2048
    Subkey-Type: ELG-E
    Subkey-Length: 2048
    Name-Real: Roman Senate
    Name-Email<EMAIL_ADDRESS>    Name-Comment: Without passphrase
    Expire-Date: 0
    </GnupgKeyParms>

When called from CLI, Rnp can list them:
$ rnpkeys --list-keys --homedir=/Users/skale/Projekty/ribose/enmail/tmp/gpgme
6 keys found

pub   2048/DSA aa33c08af1b83e7e 2018-05-09 [SCA]
      149382e38729db6a5ac973faaa33c08af1b83e7e
uid           Some Arbitrary Key (Without passphrase<EMAIL_ADDRESS>sub   2048/Elgamal (Encrypt-Only) e1933c0d83c1e656 2018-05-09 [E]
      e6322905cfd9aeed4332a597e1933c0d83c1e656

pub   2048/DSA 01ff281c27f91b23 2018-05-09 [SCA]
      4144d8815fc3babfd9adc13c01ff281c27f91b23
uid           Cato Elder (Without passphrase<EMAIL_ADDRESS>sub   2048/Elgamal (Encrypt-Only) e40f56ec43088345 2018-05-09 [E]
      14e6ada7574094e675d6976ce40f56ec43088345

pub   2048/DSA 1d1a11f21ff10fec 2018-05-09 [SCA]
      f3841141c5b9b243b94adb0f1d1a11f21ff10fec
uid           Roman Senate (Without passphrase<EMAIL_ADDRESS>sub   2048/Elgamal (Encrypt-Only) 1daaf37e8ad2bcb6 2018-05-09 [E]
      48cdfb56aeb80d4e54f610e41daaf37e8ad2bcb6

Also, an attempt to run tests for ruby-rnp project results with similar failure.
Rnp library has been installed via Homebrew. rnp --version says:
rnp 0.8.0~[f45f254]
All bug reports, praise and chocolate, please, to:
Ribose Inc<EMAIL_ADDRESS>
cc @dewyatt @ronaldtse

All the tests should pass (as they do in CI), so we should focus on that and ignore the rest.
pgp_new is in librnp. If I had to guess, this is probably because of the strange interdependent libraries (related to https://github.com/riboseinc/rnp/issues/579) that we have not taken the time to resolve yet.
Everything works fine for Linux, so I would think we would be able to get things working on macOS too. Unfortunately, I have no macOS systems, so I really can't help much with that. Maybe @ni4 can?

Neither command line rnp seems to work.
rnpkeys produces the expected output, as shown above, however rnp --sign <some-file> quits without producing any output (be it stdin, stderr, or stdlog), neither creates or modifies any files.
Already tried with rnp installed via Homebrew, and built from source.
gcc -v says:
gcc -v
Configured with: --prefix=/Library/Developer/CommandLineTools/usr --with-gxx-include-dir=/usr/include/c++/4.2.1
Apple LLVM version 9.1.0 (clang-9<IP_ADDRESS>)
Target: x86_64-apple-darwin17.5.0
Thread model: posix
InstalledDir: /Library/Developer/CommandLineTools/usr/bin

This is a standard Apple's compiler symlinked as gcc.
I have tried with regular GNU CC as well:
Using built-in specs.
COLLECT_GCC=gcc-8
COLLECT_LTO_WRAPPER=/usr/local/Cellar/gcc/8.1.0/libexec/gcc/x86_64-apple-darwin17.5.0/8.1.0/lto-wrapper
Target: x86_64-apple-darwin17.5.0
Configured with: ../configure --build=x86_64-apple-darwin17.5.0 --prefix=/usr/local/Cellar/gcc/8.1.0 --libdir=/usr/local/Cellar/gcc/8.1.0/lib/gcc/8 --enable-languages=c,c++,objc,obj-c++,fortran --program-suffix=-8 --with-gmp=/usr/local/opt/gmp --with-mpfr=/usr/local/opt/mpfr --with-mpc=/usr/local/opt/libmpc --with-isl=/usr/local/opt/isl --with-system-zlib --enable-checking=release --with-pkgversion='Homebrew GCC 8.1.0' --with-bugurl=https://github.com/Homebrew/homebrew-core/issues --disable-nls
Thread model: posix
gcc version 8.1.0 (Homebrew GCC 8.1.0)

but it has crashed during build with following output:
In file included from /usr/local/Cellar/gcc/8.1.0/lib/gcc/8/gcc/x86_64-apple-darwin17.5.0/8.1.0/include-fixed/stdio.h:425,
                 from rnp_key_store.c:36:
rnp_key_store.c: In function 'rnp_key_store_load_from_file':
rnp_key_store.c:187:40: error: '__builtin_snprintf' output may be truncated before the last format character [-Werror=format-truncation=]
             snprintf(path, MAXPATHLEN, "%s/%s", key_store->path, ent->d_name);
                                        ^~~~~~~
rnp_key_store.c:187:13: note: '__builtin_snprintf' output 2 or more bytes (assuming 1025) into a destination of size 1024
             snprintf(path, MAXPATHLEN, "%s/%s", key_store->path, ent->d_name);
             ^~~~~~~~
cc1: all warnings being treated as errors

cc @dewyatt @ni4

I am building with:
git clone git@github.com:riboseinc/rnp.git
./build_macos.sh
make install

This is config.log: https://gist.github.com/skalee/b9df3a98ac571c6dcb72993f39686a80

@skalee I'll check what is wrong on macOS, but a bit later.
It should a be problem with dylib loading/pathes or compilation, most likely pgp_new body is not included into dylib.
Can you please show the output of nm /usr/local/Cellar/rnp/HEAD-f45f254/lib/librnp.0.dylib |grep pgp_new ?
rnp --sign file without username seems to be broken, you need to specify -u parameter with id of the signing key (or email which corresponds to that key).

@ni4 Adding -u parameter to rnp --sign file helped. Regarding nm /usr/local/Cellar/rnp/HEAD-f45f254/lib/librnp.0.dylib |grep pgp_new, it says:<PHONE_NUMBER>00be1a T _pgp_new

@skalee I spun up a macOS system and came up with a lame workaround:
diff --git a/lib/rnp/ffi/librnp.rb b/lib/rnp/ffi/librnp.rb
index 3ec7088..a979b34 100644
--- a/lib/rnp/ffi/librnp.rb
+++ b/lib/rnp/ffi/librnp.rb
@@ -6,6 +6,16 @@

 require 'ffi'

+module LibRnpkeys
+  extend FFI::Library
+  ffi_lib 'rnpkeys'
+end
+
+module LibRepgp
+  extend FFI::Library
+  ffi_lib 'repgp'
+end
+
 # @api private
 module LibRnp
   extend FFI::Library

Looking at objdump -lazy-bind /usr/local/lib/librnp.dylib I'm guessing (macOS newb here) the librnp dylib is missing a few links to the other libs.
I'm not really sure, but I do think it's a good time to switch to CMake (which I've been playing with today) and rethink the library organization (as @ni4 was mentioning today I think).

After applying this change, all tests do pass for me as well. I believe I can use a patched library for local development, and run tests against regular gem in CI. Thanks a lot!
 I guess they're just warnings, and are expected.

Yep


Feedback widget

Switch Report a problem widget for feedback widget
The report a problem component has been decommissioned in favour of using the feedback widget (read the design specs in the attached ADO card link above).  This widget has a few different states.  Initially it should display an interactive component that asks the user if they would like to submit feedback.  If yes is clicked, the widget displays a thank you message.  If no is clicked, a form is displayed where users must input one of the radio buttons, the extra details textarea is optional.  After submitting, the same thank you message is displayed as in if the user clicked the yes button.
There is client-side validation that checks if the radio group has an input that's been clicked.  There is also server-side validation that checks this as well.  If no input has been checked, the server returns a 400 bad request, but still displays the thank you message.  If the submission is successful, the widget returns a 200 status and shows the thank you.  If there is a failure because an error is caught, the server returns a 500 status and also displays the thank you message.  These are tested in a unit test.
Note:
the following env var have been added/updated (check the ADO pipeline to get their values):

NOTIFY_FEEDBACK_TEMPLATE_ID
NOTIFY_API_KEY

Check out a preview of your pull request here: https://app-servicecanadalabs-dev-pr-preview-3.azurewebsites.net

Check out a preview of your pull request here: https://app-servicecanadalabs-dev-pr-preview-3.azurewebsites.net


Backspace not working in new SimpleMDE on Issue/Commment

Gitea version (or commit ref): 1.11.0+dev-494-gf88715e29
Can you reproduce the bug at https://try.gitea.io: Yes

Description
When editing an issue body or comment the backspace key will not work to delete characters once the caret has been moved with arrow keys or mouse click.
Typing new characters works and the new characters can be deleted all the way back to where the last caret position was but then it will not delete further back.
Shift+backspace works though!
Tried in Firefox and Chrome.

@lunny Did you mean selecting the caret cause this bug?

I can confirm this.
The troublesome line is:
https://github.com/go-gitea/gitea/blob/d59536c222857713d95846d77903282afe608f23/web_src/js/index.js#L1493
I'm not sure what the trigger is there for, but deleting that line makes deletion work again.

@blueworrybear what was the purpose of that line?


Server 2019?

Hi, What is the limiting factor of running Webcommander on later operating systems?
Thanks Jerry,
Jen

Sorry Jen. I'm not sure. I don't have a 2019 environment to try it.
However the only concern is around the compatibility between IIS and PHP.
If you could install the same version of IIS (10) and PHP (5) on 2019, it should work.

Thanks Jerry, have you tried it on any other version of Windows server?
Jen


UITextView in ScrollView

Firstly, Thank you for the great library!
I was found a problem. When i add TextView have long content in Scrollview. I scroll TextView and the Navigation bar scroll follow it. So can you help me resolve this bug.

Can you post a sample showing the issue?

I guess not. Feel free to reopen if needed.


feat: add project list command

What type of PR is this?

What this PR does / why we need it:
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:
Does this PR introduce a user-facing change?

Additional documentation e.g., design docs, usage docs, etc.:

Pull Request Test Coverage Report for Build<PHONE_NUMBER>6
Details

244 of 292 (83.56%)  changed or added relevant lines in 11 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.5%) to 52.939%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

pkg/backend/storages/local.go
0
3
0.0%

pkg/backend/storages/oss.go
0
3
0.0%

pkg/backend/storages/s3.go
0
3
0.0%

pkg/cmd/workspace/list/options.go
5
9
55.56%

pkg/project/storages/local.go
17
21
80.95%

pkg/project/storages/oss.go
34
39
87.18%

pkg/project/storages/s3.go
44
49
89.8%

pkg/cmd/project/list/options.go
92
113
81.42%

Totals

Change from base Build<PHONE_NUMBER>3:
0.5%

Covered Lines:
10115

Relevant Lines:
19107

💛  - Coveralls

Done fixing comments.


docs syntax wrong on faker.list.cycle

Should be:
  // factories/meterRead.js
  classification() {
    return faker.random.arrayElement([
      'Actual',
      'Estimated'
    ])
  }

or
  // factories/meterRead.js
  classification: faker.list.cycle(
    'Actual',
    'Estimated'
  )

but not
  // factories/meterRead.js
  classification() {
    return faker.list.cycle(
      'Actual',
      'Estimated'
    )
  }

the last one is the one in the docs, not working

Ah yes, this is super confusing & bad that we introduced a basically inconsistent API. There are reasons for it but yah, for now let's just update.
Would you mind clicking the "Improve this page" link on the relevant doc page & submitting a PR with the fix?

done thanks

Super helpful, thank you sir


Refresh access token failed with status: 401 when opening .bot created with msbot clone services in Emulator

Describe the bug
When opening bot file in Emulator, we get a "Refresh access token failed with status code: 401"
To Reproduce
Steps to reproduce the behavior:
Create  required services for basic bot:
C:\Users\v-asho\source\repos\BugBash2\BotBuilder-Samples\javascript_nodejs\13.basic-bot>msbot clone services -n AshleyHoBugBash2Bot -f deploymentScripts/msbotClone --location westus --luisAuthoringKey MY_LUIS_KEY --verbose
[msbot] Checking az botservice version
[msbot] Fetching subscription account
[msbot] Creating resources in subscription: Bot Service Testing (Scratch) MY_LUIS_KEY
[msbot] Creating Azure group [AshleyHoBugBash2Bot]
[msbot] Creating LUIS Cognitive Service [AshleyHoBugBash2Bot-LUIS]
[msbot] Fetching LUIS Keys [AshleyHoBugBash2Bot-LUIS]
[msbot] Creating Azure Bot Service [AshleyHoBugBash2Bot]
[msbot] [az bot] To sign in, use a web browser to open the page https://microsoft.com/devicelogin and enter the code FT79EPKJ3 to authenticate.

[msbot] [az bot] obtained msa app id and password. Provisioning bot now.
 (this will take several minutes)
[msbot] Fetching bot extended information [AshleyHoBugBash2Bot]
[msbot] Fetching co-created resources [AshleyHoBugBash2Bot]
[msbot] Creating and importing LUIS application [AshleyHoBugBash2Bot-basic-bot-LUIS]
[msbot] Training LUIS application [AshleyHoBugBash2Bot-basic-bot-LUIS]
[msbot] Publishing LUIS application [AshleyHoBugBash2Bot-basic-bot-LUIS]
[msbot] Updating LUIS settings [AshleyHoBugBash2Bot-basic-bot-LUIS]
[msbot] Fetching storage connection string [shleyoup2oz]
[msbot] AshleyHoBugBash2Bot.bot created.
[msbot] Done cloning.

AshleyHoBugBash2Bot.bot successfully created:
{
    "name": "AshleyHoBugBash2Bot",
    "description": "",
    "services": [
        {
            "type": "endpoint",
            "id": "http://localhost:3978/api/messages",
            "name": "development",
            "appId": "...",
            "appPassword": "...",
            "endpoint": "http://localhost:3978/api/messages"
        },
        {
            "type": "luis",
            "name": "AshleyHoBugBash2Bot-basic-bot-LUIS",
            "appId": "...",
            "authoringKey": "...",
            "subscriptionKey": "...",
            "version": "0.1",
            "region": "westus",
            "id": "5"
        },
        {
            "type": "abs",
            "name": "AshleyHoBugBash2Bot",
            "tenantId": "...",
            "subscriptionId": "...",
            "resourceGroup": "AshleyHoBugBash2Bot",
            "serviceName": "AshleyHoBugBash2Bot",
            "appId": "...",
            "id": "220"
        },
        {
            "type": "endpoint",
            "name": "AshleyHoBugBash2Bot",
            "appId": "...",
            "appPassword": "...",
            "endpoint": "https://shleyougashlo27.azurewebsites.net/api/messages",
            "id": "175"
        },
        {
            "type": "appInsights",
            "tenantId": "...",
            "subscriptionId": "...",
            "resourceGroup": "AshleyHoBugBash2Bot",
            "name": "AshleyHoBugBash2Botmvsg6g",
            "serviceName": "AshleyHoBugBash2Botmvsg6g",
            "instrumentationKey": "...",
            "applicationId": "...",
            "apiKeys": {},
            "id": "148"
        },
        {
            "type": "blob",
            "name": "shleyoup2oz",
            "serviceName": "shleyoup2oz",
            "tenantId": "...",
            "subscriptionId": "...",
            "resourceGroup": "AshleyHoBugBash2Bot",
            "connectionString": "DefaultEndpointsProtocol=https;EndpointSuffix=core.windows.net;AccountName=shleyoup2oz;AccountKey=ACCOUNT_KEY",
            "container": null,
            "id": "60"
        }
    ],
    "secretKey": "",
    "version": "2.0"
}

Running the clone command successfully created it in Azure portal, however note that it does not have appId/password in Application Settings:

npm config set registry https://botbuilder.myget.org/F/botbuilder-v4-js-daily/npm/
npm install
add to .env: botFilePath = .\AshleyHoBugBash2Bot.bot
edit line 20 in bot.js: const LUIS_CONFIGURATION = 'AshleyHoBugBash2Bot-basic-bot-LUIS';
in terminal: npm start
in Emulator: File -> open AshleyHoBugBash2Bot.bot

at both development and AshleyHoBugBash2Bot endpoints, 401 authorization errors:

Expected behavior
to be able to connect to bot at both development and AshleyHoBugBash2Bot endpoints without 401 errors
Additional context
Add any other context about the problem here.
SDK Platform:
 Node
URL of Code Sample
 https://github.com/Microsoft/BotBuilder-Samples/tree/v4/javascript_nodejs/13.basic-bot

@Zerryth - was this resolved with @tonyanziano ? It might be better to track this in botframework-emulator repo. If you agree, can you move this issue and close this one?

I am seeing the same issue after msbot clone services. Might be related to https://github.com/Microsoft/botbuilder-tools/issues/528

Per Chat with Andrea, this might be related to our scratch and ppe environment. Jeff has checked in a fix and Andrea can no longer repro this.
@Zerryth Closing this as fixed; please reactivate if you continue to see this.

I can repro this again this afternoon, v4 emulator doesn't work but v3 does. I'm going to tear down the bot and re-create with latest template and will reopen if I see it

I am having the same issue @darrenj what did you do to resolve this?

You might have an old version do the emulator. Try the latest version from here: https://github.com/Microsoft/BotFramework-Emulator

From: mickallen<EMAIL_ADDRESS>Sent: Wednesday, November 14, 2018 6:33:03 AM
To: Microsoft/botbuilder-tools
Cc: Darren Jefford; Mention
Subject: Re: [Microsoft/botbuilder-tools] Refresh access token failed with status: 401 when opening .bot created with msbot clone services in Emulator (#512)
I am having the same issue @darrenjhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fdarrenj&data=02|01|darrenj%40microsoft.com|7d2fd0803cdf429c28cf08d64a3e1631|72f988bf86f141af91ab2d7cd011db47|1|0|636778027857974114&sdata=x1xh0%2FQPVGd%2FV5YUvq673Va%2FDqsuzHjqbM1lxxVZ6Mc%3D&reserved=0 what did you do to resolve this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fbotbuilder-tools%2Fissues%2F512%23issuecomment-438681940&data=02|01|darrenj%40microsoft.com|7d2fd0803cdf429c28cf08d64a3e1631|72f988bf86f141af91ab2d7cd011db47|1|0|636778027857984127&sdata=mx%2FrHvVRb5%2FbZyrUQlA6NYEVVaYCTkFzYmqCGxrTZWI%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAf93oOThUiXyKrnFirRb_Ol4GFynkQEEks5uvCmfgaJpZM4WspWS&data=02|01|darrenj%40microsoft.com|7d2fd0803cdf429c28cf08d64a3e1631|72f988bf86f141af91ab2d7cd011db47|1|0|636778027857984127&sdata=L9YhuoP8PKHUyIsmTAodGR2W4yf4d86tbZrvQqDzhHQ%3D&reserved=0.

THis is down to the emulator not having the new changes for the .bot file padlock

Thanks @darrenj I'll give that a try and post back

@darrenj, I downloaded the latest release (4.1.0)  from the releases page but still get the error, I am trying to run the NLP with Dispatch example from the samples repo  https://github.com/Microsoft/BotBuilder-Samples/tree/master/samples/csharp_dotnetcore/14.nlp-with-dispatch

@mickallen, getting same error for javascript (node) version of "NLP with Dispatch" sample. Were you able to resolve?

got the same issue after following this tutorial:
Tutorial: Create and deploy a basic bot


Implement Unit Test for cleanNumber.js

Coverage remained the same at 29.227% when pulling 8ec14a8f151408870839558494c6380f3dde1019 on LingForCC:unit-test-cleanNumber into f6203d6655faa1557d75a8818440aed05df0af3a on ringcentral:master.


[2pt] Setup Kanboard

Setup Kanboard
we are two members
@george827
@tanveerisonline
Link to the kanban

Hi Team,
While you made a great effort in this project, unfortunately, I cannot proceed to review your code.
Invalid Code Review Request
Link to the board is missing. Please include that in your description above.
Your Code Review Request will be marked as invalid in your Dashboard, so please submit a new one once you are ready 🙏
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Pull Request for re-reviews. You should use the same Pull Request submitted for the first review, either valid or invalid unless it is requested otherwise.

Invalid Code Review Request does not count into the code reviews limit.

STATUS: CHANGES REQUIRED ♻ ♻
Hi @george827 and @tanveerisonline,
Good job so far!
✔ Kanban columns are complete
However, there are some issues that you still need to work on to go to the next project but you are almost there!
Required Changes ♻

[ ] You seem to have a lot of duplicate (and should I say unnecessary) tasks. From the issue titles, you have different tasks to accomplish the same requirement. Kindly look into your tasks and remove duplicate and unneeded tasks.
I recommend you stick only to the tasks listed in the requirements to avoid confusion. Also, ensure the tasks have descriptions matching the ones in the requirements.

duplicate issues like N+1 and so on
owner of recipe, general shopping list, and so on
duplicate food list, recipe details, and so on

more
more

Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the Issue thread if something is not 100% clear. And please, remember to tag me in your question so I can receive the notification.
Please, do not open a new Issue for re-reviews. You should use the same Issue submitted for the first review, either valid or invalid unless it is requested otherwise.

As described in the Code reviews limits policy you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.

Hi team,
STATUS: APPROVED ✔ ✔
Your project is complete!  There is nothing else to say other than... it's time to start your capstone 👍
Congratulations and well done! 🎉

Highlights

All tasks have been fairly assigned ✔ ✔

Optional suggestions
Every comment with the [OPTIONAL] prefix won't stop the approval of this PR. However, I strongly recommend you to take them into account as they can make your code better. Some of them were simply missed by the previous reviewer and addressing them will really improve your application.
Cheers and Happy coding!👏👏👏
Feel free to leave any questions or comments in the PR thread if something is not 100% clear.
Please, remember to tag me in your question so I can receive the notification.

As described in the Code reviews limits policy  you have a limited number of reviews per project (check the exact number in your Dashboard). If you think that the code review was not fair, you can request a second opinion using this form.


help,what are the shutdown and restart commands?

item(title='关机' image=\uE12F cmd='C:\Windows\System32\shutdown.exe -s -t 0')
item(title='重启' image=\uE1DB cmd='C:\Windows\System32\shutdown.exe -r -t 0')
I want to add shutdown and restart commands in the taskbar, I use cmd command, but it doesn't work, I want to know in this software to shut down and restart the system is what command, thank you

I want to know the command of shut down, restart, sleep, hibernate

// try
cmd='C:\Windows\System32\shutdown.exe' args='-r -t 0')
// or
cmd='shutdown.exe'  args='-r -t 0')

https://discord.com/channels/1106387012707168318/1233094583953653830/1233094583953653830

// try
cmd='C:\Windows\System32\shutdown.exe' args='-r -t 0')
// or
cmd='shutdown.exe'  args='-r -t 0')

https://discord.com/channels/1106387012707168318/1233094583953653830/1233094583953653830

It works! However, I have a network problem here and it is not convenient to connect to discord, thank you very much, this software is very useful and has solved many problems for me!


[Brackets auto-update Linux] Crash on Quit

Description
[Brackets auto-update Linux]  Crash on Quit
Steps to Reproduce

Launch brackets 1.13.
Click on Debug > Open preference file.
Click on Help > Check for updates.
Click on OK in the update dialogue shown.
Close brackets.
Brackets crashes.

Expected behavior: Brackets should not crash.
Actual behavior: Brackets crashes on quit.
Versions
Ubuntu 17.10 64 Bit
Release 1.13 build 1.13.0-17665

The issue only occurred in Linux Ubuntu 17.10 which is not a LTS release.
Hence closing the issue.


automatically check for attachments and auto-ack

attachments usually take long enough to upload that they will always require a defer, so automatically ack'ing them would be good

i'd like to generalize this to acking if it'd probably fail otherwise, the actual implementation is a bit tricky but im thinking something like this

[ ] make Bot::interaction_handle spawn a thread that waits 2 seconds (optimistically assumes next request will take less than a second) and if no request was sent before, defers the interaction

[ ] because both defer and Bot::interaction_handle need them, add builder-style ephemeral and update methods to InteractionHandle that use DeferVisibility::Ephemeral and DeferBehavior::Update respectively

[ ] remove these parameters from defer method, remove defer_component altogether

[ ] add Bot::interaction_handle_manual_defer (i hate the name, ideas?) which has the current behavior
[ ] test race conditions (oh no)

enforcement of safety
it'd probably be more idiomatic to have a InteractionHandle::prepare method instead of two variants of the Bot::interaction_handle method, but that way Bot::interaction_handle has no mention of the possible failures if InteractionClient::prepare isn't called besides documentation, so i think this is something like unwrap and unwrap_unchecked (imagine unwrap methods returning Unwrap<T>: Deref<Target = T> with an unsafe method Unwrap::unchecked)
alternatively, Bot::interaction_handle could take an enum/bool telling whether to handle deferring, but this is a hot method and i think writing out the parameter isnt worth it because many people would avoid Bot::interaction_handle_manual_defer since the point of this lib is to abstract over things
lastly, we could take the opposite approach, adding a InteractionHandle::defer_manually method that cancels the spawned thread, tho it looks the best, it means having the overhead of spawning unnecessary tasks and the complexity of holding a thread handle and making it safely cancel a task


Document breaking changes in 1.0 CLI

What You Are Seeing?
CLI in 1.0 contains some breaking changes. For example when using .NET Framework runner -target Foo works with 0.38.5, but fails with 1.0 RC 1:
Error: Unknown command 'Foo'.
       build.cake -target=Foo
                          ^^^^^^ No such command

--target=Foo works in all versions and is what we also have documented since recently on cakebuild.net.
What is Expected?
Breaking changes should be documented in release notes.
What version of Cake are you using?
1.0.0-rc0001
Are you running on a 32 or 64 bit system?
64-Bit
What environment are you running on?  Windows? Linux? Mac?
Windows
Are you running on a CI Server?  If so, which one?
No
How Did You Get This To Happen? (Steps to Reproduce)

Setup Cake project using .NET Framework runner and default bootstrapper containing a task with name Foo
Use Cake 0.38.5
Call it with build.ps1 -target Foo
Use Cake 1.0 RC1
Call it with build.ps1 -target Foo

Output Log
N/A

Another difference: With pre 1.0 it was possible to pass arguments like Foo=, with 1.0 this will now throw an error:
Error: Expected an option value.

This was fixed in https://github.com/cake-build/website/pull/1312


Meetup organizing

Hi @busrasengul, thanks for your PR to add your contribution to Hacktoberfest 2023 for Umbraco in organizing the meetup event!
This has now been merged 👍


fix heading inconsistency

@BillWagner this is just a quick styling fix to an inconsistency I noticed in the codedoc article

<seealso> shouldn't be part of this

Open Publishing Build Service: The pull request content has been published and here are some sample preview links:

Docset 1: core-docs

@qinezh I don't have access to the link, also it's just a slight update to existing content

@tsolarin Thanks for spotting this, and updating the topic. Merging now.


MapEntry

I like the general direction of https://github.com/munificent/ui-as-code/blob/master/ideas/control-flow-elements.md. One random idea to throw out is whether maybe MapEntry would be a useful class to use in the case of map literals containing control flow. It would let you easily call into other methods and get back name/value pairs; currently, it's not clear how you would programmatically control the name side.
If the language supported tuples, this would also be a place to use those.

Oh, that is an interesting idea. That opens up some questions around whether you could use ... to spread a List into a map literal too.
My general impression is that MapEntry is rarely used, so this probably isn't worth the extra complexity. But it might be. I agree that tuples would be a natural fit if we had those. (I think Lasse also feels that if we had tuples, MapEntry would no longer need to exist except for compatibility reasons.)

Map may be retrofitted with a constructor with vararg list of MapEntries:
Map.fromEntries(MapEntry<K,V> entries...)
Then it will be possible to write (see the discussion in #15)
var map = Map.fromEntries {
  +MapEntry("foo", "bar");
  if (cond) {
    +MapEntry("baz", "haha");
  }
}


Equallity of vote arays in different elections as pre prop still prooduced different vote arrays

When having votes1 == votes2 as pre property, the counter examples would still show voting arrays
differing in the last slots.

fixed in commit 60b611c34ce1662a6b1d7c82410c6d8d34c35caf


add configurable ColumnTypeMergePolicy to SegmentMetadataCache

Description
This PR adds a new interface to control how SegmentMetadataCache chooses ColumnType when faced with differences between segments for SQL schemas which are computed, exposed as druid.sql.planner.metadataColumnTypeMergePolicy and adds a new 'least restrictive type' mode to allow choosing the type that data across all segments can best be coerced into. The existing "newest first" behavior remains the default, primarily because this is a behavior change around when schema migrations take effect for the SQL schema. With {"type":"newestFirst"}, the SQL schema would be updated as soon as the first job with the new schema has published segments, while using {"type":"leastRestrictive"}, the schema would only be updated once all segments are reindexed to the new type. The benefit of leastRestrictive is that it eliminates a bunch of type coercion errors that can happen in SQL when types are varied across segments with newestFirst because the newest type is not able to correctly represent older data, such as if the segments have a mix of ARRAY and number types, or any other combinations that lead to odd query plans.
I am not at all attached to these names, so if they should be called something else more intuitive then feel free to suggest.
Release note
A new broker configuration, druid.sql.planner.metadataColumnTypeMergePolicy adds configurable modes to how column types are computed for the SQL table schema when faced with differences between segments. A new 'least restrictive type' mode allows choosing the most appropriate type that data across all segments can best be coerced into. The existing "newest first" behavior remains the default, primarily because this is a behavior change around when schema migrations will take effect for the SQL schema. With {"type":"newestFirst"}, the SQL schema would be updated as soon as the first job with the new schema has published segments, while using {"type":"leastRestrictive"}, the schema would only be updated once all segments are reindexed to the new type. However, {"type":"leastRestrictive"} is likely to have "better" query time behavior and eliminates some query time errors that can occur when using newestFirst.

This PR has:

[ ] been self-reviewed.

[ ] using the concurrency checklist (Remove this item if the PR doesn't have any relation to concurrency.)

[ ] added documentation for new or modified features or behaviors.
[ ] a release note entry in the PR description.
[ ] added Javadocs for most classes and all non-trivial methods. Linked related entities via Javadoc links.
[ ] added or updated version, license, or notice information in licenses.yaml
[ ] added comments explaining the "why" and the intent of the code wherever would not be obvious for an unfamiliar reader.
[ ] added unit tests or modified existing tests to cover new code paths, ensuring the threshold for code coverage is met.
[ ] added integration tests.
[ ] been tested in a test Druid cluster.

Regarding naming:
For newestFirst, "newest" without context might be confusing because it can refer either to when the segment was created/published OR the interval it covers. I believe the behavior is to select the segment with the latest interval (since the segments are sorted by segment ID, and the interval comes first in the ID), so a policy name like latestInterval might be clearer?
For leastRestrictive, perhaps bestFit is clearer? My concern is that "least restrictive", while correct, can also refer to a policy that just chooses the widest possible type.

For newestFirst, "newest" without context might be confusing because it can refer either to when the segment was created/published OR the interval it covers. I believe the behavior is to select the segment with the latest interval (since the segments are sorted by segment ID, and the interval comes first in the ID), so a policy name like latestInterval might be clearer?

latestInterval probably is clearer, i can make the change soon unless anyone has better ideas/different opinions


Adding Sean Linux Art

Description
Adding neofetch art for my distro, Sean Linux

icon-ascii-art.txt

Please open another PR.


Provide a better solution for scrollable panels

We constantly run into panel scrolling issues such as #272. The problem is that we rely on Panel implementers to follow the markup described here:
div.sc-panel
  // overflowing container
  div.se-panel-content
    // has position: relative set, which is important to capture the position().top correctly
    // if for
    div.se-panel-content-inner

We should maybe have a different approach to Panels. Instead of implementing them using conventions we should provide a wrapper implementation that you just inject the content you want to display. This would be analog to what we are already doing with the Toolbar component.
This would mean instead of:
$$(TOCPanel)

We do:
// by default Panels will be rendered with our substance toolbar
$$(Panel, {nativeScrollbar: true}).append(
  $$(TOC)
)

Need to think a bit more but this could give us some flexibility over where we want to show certain views and implementing them independently. E.g. we could display the TOC component within a Modal:
// by default Panels will be rendered with our substance toolbar
$$(Modal).append(
  $$(TOC)
)

Also we could ditch the ContentPanel implementation which is not too generic and currently hard-bound to the TableOfContents.
Thoughts?

Done.


Test added and findNode fixed with O(1) complexity

Thanks @virenvarma007 !
Some observations:

Could you find another name for the instance variable dictNodes ? please
Can you format the code?
The test is good, but can you rename testdictNodes to something else? Also, can you add other tests that.

The test does not call the run method. Only test that setting and getting the nodes works well.
Some other tests that test only one simple thing


streamline documentation for installation and quickstart

These were written with expert users in mind -let's go through these instructions very carefully and make sure that a new user who may be less experienced with github, working on the command line, etc.

Installation is hopefully well documented now, but quickstart indeed needs a significant overhaul

@taylorbell57, @lkreidberg I can try to rewrite the quickstart this week if you haven't already started?

That'd be awesome - I haven't doing anything there yet


Add validation logic to backend

Backend should be validating fields, e.g., checking that the participant ID is an alphanumeric string of 100char.

Hi Rob, does anything actually need to be done here?
If so, can you take care of it since you're handling the backend?

This seems like a nice-to-have, but not necessary; I haven't found any problems in the backend data. Do you remember why you filed it?

because i'm a paranoid security engineer

So, you imagine this validation happening on post-processing, not before results are written to backend store? If so, I could add some validation to my python processing script.

I am remembering now that I think I saw that the backend is already doing some filtering along the liens of waht you jut described, but I am forgetting for sure. Could you double check that it does?

The NDB models should enforce the types when converting a message sent to the API to a datastore object. See https://github.com/GoogleChrome/experience-sampling/blob/master/backend/models.py
This is very course-grained though. For example, participant-id is just restricted to being a StringProperty (unicode, <=500chars).

I'm looking into this
On Thu, Apr 2, 2015 at 7:58 PM, Chris Thompson<EMAIL_ADDRESS>wrote:

The NDB models should enforce the types when converting a message sent to
the API to a datastore object. See
https://github.com/GoogleChrome/experience-sampling/blob/master/backend/models.py
This is very course-grained though. For example, participant-id is just
restricted to being a StringProperty (unicode, <=500chars).
—
Reply to this email directly or view it on GitHub
https://github.com/GoogleChrome/experience-sampling/issues/93#issuecomment-89084768
.

Ok, here's what I've found:

Chris is right; for StringProperty, the 500 char limit is checked; if a request exceeds the limit, it returns a server error
But you can still send an illegal participantId, e.g., with != 100 chars or with chars outside the allowed set
we could add validation for participantId on the backend, but I think this buys nothing, because:
a) someone could still send an incorrect participantId that passes validation, but just isn't their ID
b) other fields, especially the TextProperty fields for question and answer, have no length limit, and I don't know how we'd validate them
c) I don't think we have a problem here to begin with, because we never treat the user-provided data in an executable context; i.e., it gets written as valid JSON to the backend (the server gives an error if the data provided is not valid JSON), then we read it as JSON into our post-processing script, convert it to Python dicts (where the strings are just strings, not evaluated as code), then output the dicts to CSV files, where they are, again, just strings.

So my conclusion is there's nothing to do here. I'm hardly a code injection expert, so I'm open to arguments about why I'm wrong, but if there aren't any, I'll mark this bug closed.

Hearing no objections, I'm closing this.


K3ai Triage Management Workflow fails on PRs

See: #44
https://github.com/kf5i/k3ai-core/pull/44/checks?check_run_id=1328375820
https://github.com/kf5i/k3ai-core/pull/44/checks?check_run_id=1328376617

it should be fixed now. PR are not applied yet


Roll engine b1b388f1c235..3c93817839bd (1 commits)

https://github.com/flutter/engine/compare/b1b388f1c235...3c93817839bd
git log b1b388f1c2356a865240e706acdb37af982991ad..3c93817839bd557a045b17322b797fb4772c18f4 --no-merges --oneline
3c9381783 Add “full-dart-debug” that disabled optimizations in the Dart VM. (flutter/engine#8147)
The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll
Documentation for the AutoRoller is here:
https://skia.googlesource.com/buildbot/+/master/autoroll/README.md
If the roll is causing failures, please contact the current sheriff (jsimmons@google.com), and stop
the roller if necessary.

Trybots failed. These were the failed builds: https://cirrus-ci.com/task/5286943247040512

Commit queue failed; closing this roll.


Update dependencies to latest.

This PR updates all dependencies of cargo-component to the latest versions.

The clippy error message expected in one of the tests changed with the release of 1.68.0 (a few hours ago). Pushing up a fix.


[TRIVIAL] refer to std.meta not std.typetuple

Wow this is the first time I read this.
NoDuplicates is a horrible source of template bloat .... We do recommend this ?

Wow this is the first time I read this.
NoDuplicates is a horrible source of template bloat .... We do recommend this ?

I think we should stop the irrational fear of using templates and fix the implementation shortcommings instead. Templates are here to stay.


<EMAIL_ADDRESS>exception return undesired

Summary
Denial in<EMAIL_ADDRESS>causes a "Ignoring exception in command {command}"
Reproduction Steps
If the command caller does not meet the criteria for a<EMAIL_ADDRESS>an exception is thrown.
Minimal Reproducible Code
@commands.check_any(commands.has_role(role1id))

Expected Results
nextcord.ext.commands.errors.CheckAnyFailure: You do not have permission to run this command.
Actual Results
Ignoring exception in command levels:
Traceback (most recent call last):
File "/usr/local/lib/python3.9/site-packages/nextcord/ext/commands/bot.py", line 995, in invoke
await ctx.command.invoke(ctx)
File "/usr/local/lib/python3.9/site-packages/nextcord/ext/commands/core.py", line 887, in invoke
await self.prepare(ctx)
File "/usr/local/lib/python3.9/site-packages/nextcord/ext/commands/core.py", line 808, in prepare
if not await self.can_run(ctx):
File "/usr/local/lib/python3.9/site-packages/nextcord/ext/commands/core.py", line 1125, in can_run
return await nextcord.utils.async_all(predicate(ctx) for predicate in predicates)  # type: ignore
File "/usr/local/lib/python3.9/site-packages/nextcord/utils.py", line 522, in async_all
elem = await elem
File "/usr/local/lib/python3.9/site-packages/nextcord/ext/commands/core.py", line 1792, in predicate
raise CheckAnyFailure(unwrapped, errors)
nextcord.ext.commands.errors.CheckAnyFailure: You do not have permission to run this command.
Intents
default + guilds + members
System Information

Python v3.9.6-final
nextcord v2.0.0-alpha

nextcord pkg_resources: v2.0.0a3

aiohttp v3.7.4.post0
system info: Linux 5.4.0-88-generic #99-Ubuntu SMP Thu Sep 23 17:29:00 UTC 2021

Checklist

[X] I have searched the open issues for duplicates.
[X] I have shown the entire traceback, if possible.
[X] I have removed my token from display, if visible.

Additional Context
No response

Intended behavior

Your expected result is right at the bottom

Happy to know thanks for the confirmation


Added one step install

I updated the setup.py so that you can pip install cython numpy benepar all in one command.

This seems like a really useful one, as currently it takes my users additional install steps just because of the order of install issue. @nikitakit can you merge? Perhaps just add the newline at end of file here in the browser too?

Done @interrogator

Thank you for taking the time to look at this installation issue, and sorry that I haven't been monitoring this GitHub repo in a long long time.
In 2021, the wonderful pytorch-struct library has a nice implementation of CKY, and I've updated benepar to not use Cython at all. This has the added benefit of being much faster when using a GPU. Sadly some installation dependency issues remain because pytorch-struct is not on PyPI, but maybe I'll be able to work through those in the future.


dev branch - mosquitto_pub to multiple topics

mosquitto_pub does not support publishing a message to multiple topics
See
cyphernode/cyphernodeconf_docker/templates/bitcoin/blocknotify.sh
cyphernode/cyphernodeconf_docker/templates/bitcoin/walletnotify.sh
A fix has to be made to invoke mosquitto_pub 2 times - on each topic:
  # Topic 'newblock' will be removed eventually
  echo "[blocknotify-$$] mosquitto_pub -h broker -t newblock -t bitcoinnode/newblock -m \"{\"blockhash\":\"${blockhash}\",\"blockheight\":${blockheight}}\""
  mosquitto_pub -h broker -t newblock -m "{\"blockhash\":\"${blockhash}\",\"blockheight\":${blockheight}}"
  mosquitto_pub -h broker -t bitcoinnode/newblock -m "{\"blockhash\":\"${blockhash}\",\"blockheight\":${blockheight}}"

#310 linked


Supervisor still failing in storm:2.4.0-temurin because of unrecognized Java 11 VM option

This issue follows up on #32. PR #3504 fixes the bug, but, from what I'm seeing, a patch has yet to be released. Therefore, the supervisor process is still failing in the latest image build.
Expected behavior
Supervisor ignores unrecognized Java 11 VM options.
Actual behavior
Supervisor fails because of deprecated VM options.
2023-02-26 12:11:52.741 STDERR Thread-0 [INFO] Unrecognized VM option 'PrintGCDateStamps'
2023-02-26 12:11:52.741 STDERR Thread-1 [INFO] [0.000s][warning][gc] -Xloggc is deprecated. Will use -Xlog:gc:artifacts/gc.log instead.
2023-02-26 12:11:52.743 STDERR Thread-0 [INFO] Error: Could not create the Java Virtual Machine.
2023-02-26 12:11:52.743 STDERR Thread-0 [INFO] Error: A fatal exception has occurred. Program will exit.

However, adding the IgnoreUnrecognizedVMOptions option in /conf/storm.yaml to override the default worker.childopts config fixes the issue:
worker.childopts: "-Xmx%HEAP-MEM%m -XX:+IgnoreUnrecognizedVMOptions -XX:+PrintGCDetails -Xloggc:artifacts/gc.log -XX:+PrintGCDateStamps -XX:+PrintGCTimeStamps -XX:+UseGCLogFileRotation -XX:NumberOfGCLogFiles=10 -XX:GCLogFileSize=1M -XX:+HeapDumpOnOutOfMemoryError -XX:HeapDumpPath=artifacts/heapdump"

Are you guys planning to release a patch soon?

Adding a workaround seems like a good idea. At least until the patched version is released by the upstream.
I was going to patch conf/defaults.yaml similarly to STORM-3886. But there's no such file in the distribution for some reason.
$ docker run -it --rm storm:2.4.0-temurin sh -c 'pwd; ls conf'
Unable to find image 'storm:2.4.0-temurin' locally
2.4.0-temurin: Pulling from library/storm
Digest: sha256:d91fa2e8254f22ddbf0b477f228c690f392fe8d359a6be3b9f9094fb54f6b16c
Status: Downloaded newer image for storm:2.4.0-temurin
/apache-storm-2.4.0
seccomp.json.example  storm_env.ini  storm-env.ps1  storm-env.sh  storm.yaml

Although, this file exists in the source distribution apache-storm-2.4.0-src.tar.gz. But the Docker image uses apache-storm-2.4.0.tar.gz.
So I'd doubt that changes in conf/defaults.yaml have the desired effect. We need to try to figure this out with the upstream maintainers.
As an alternative solution it's possible to include fixed worker.childopts to the default storm.yaml that is generated by docker-entrypoint.sh. But it will only work for the cases when users don't provide custom config.

I tried to put defaults.yaml to $STORM_CONF_DIR but got the following error during startup:
Exception in thread "main" java.lang.ExceptionInInitializerError
	at org.apache.storm.command.ConfigValue.main(ConfigValue.java:38)
Caused by: java.lang.RuntimeException: java.io.IOException: Found multiple defaults.yaml resources. You're probably bundling the Storm jars with your topology jar. [jar:file:/apache-storm-2.4.0/lib/storm-client-2.4.0.jar!/defaults.yaml, file:/conf/defaults.yaml]
	at org.apache.storm.utils.Utils.findAndReadConfigFile(Utils.java:198)
	at org.apache.storm.utils.Utils.readDefaultConfig(Utils.java:241)
	at org.apache.storm.utils.Utils.readStormConfig(Utils.java:304)
	at org.apache.storm.utils.Utils.<clinit>(Utils.java:132)
	... 1 more
Caused by: java.io.IOException: Found multiple defaults.yaml resources. You're probably bundling the Storm jars with your topology jar. [jar:file:/apache-storm-2.4.0/lib/storm-client-2.4.0.jar!/defaults.yaml, file:/conf/defaults.yaml]
	at org.apache.storm.utils.Utils.getConfigFileInputStream(Utils.java:228)
	at org.apache.storm.utils.Utils.findAndReadConfigFile(Utils.java:176)
	... 4 more

So it looks like defaults.yaml gets packed into JARs during distribution phase and changes in its source should have the desired effect.
It seems that the only feasible option now is to include the workaround to the auto-generated storm.yaml (see https://github.com/31z4/storm-docker/pull/40) and wait until upstream releases a distribution with the proper fix.
What do you think @raulnegrean? Maybe I'm missing some better ways to address the issue?

Thanks for addressing this, @31z4.

What do you think @raulnegrean? Maybe I'm missing some better ways to address the issue?

Indeed, from what I'm seeing, the defaults.yaml is packaged in the storm-client JAR (see here), so we could directly patch the JAR to avoid having this issue if users provide their own storm.yaml (see details in PR https://github.com/31z4/storm-docker/compare/master...raulnegrean:storm-docker:patch-1).
Definitely less orthodox than what you're proposing, but, assuming I'm not missing something, this could fix the problem until a patch is released.

Thanks for the suggestion @raulnegrean 🙏
I made a PR inspired by your patch https://github.com/31z4/storm-docker/pull/41. I'm happy to merge after your review ☺

Looks good! Thanks for fixing this, @31z4.

Thanks for addressing this, @31z4.

What do you think @raulnegrean? Maybe I'm missing some better ways to address the issue?

Indeed, from what I'm seeing, the defaults.yaml is packaged in the storm-client JAR (see here), so we could directly patch the JAR to avoid having this issue if users provide their own storm.yaml (see details in PR master...raulnegrean:storm-docker:patch-1).
Definitely less orthodox than what you're proposing, but, assuming I'm not missing something, this could fix the problem until a patch is released.

this was really helpful, Thank you!


Packaged runners incorrectly named

If a runner is included in a package, it ends up being registered as it's filename (e.g. my-runner.run) instead of the caption specified in the runner.
This will work the first time the build is run (although it appears with '.run' at the end), but if the workspace is refreshed, the output panel can no longer find the correct runner and consequent runs will fail unless the output window is closed first.

The PR referenced above ensures runners are loaded with the correct names and found when searched for. Even when these PRs are merged, however there is still an issue here in that the user plugins (and hence the runners) are loaded after the run output has been loaded meaning the runner currently selected is never found.
I recommend a way to load plugins before the panels/panes are rendered is considered.

@thegecko i have merged https://github.com/c9/core/pull/142, but on second thought i am not entirely sure if it was a good idea or not. Allowing runners to set their names independent of filename, will make it harder to find where the runner comes from. Why do you need a runner with different caption and filename?

I recommend the loading system is changed to load plugins before the panels/panes are rendered.

Loading plugins before drawing panels is possible for core plugins that are preinstalled for a workspace type, but other plugins need to be able to load/unload without reloading whole ide.

I assumed the caption in the runner was the supported way of naming the runner. If it is an internal mechanism, then I'm happy to use the name of the file as the runner name, albeit without the '.run' extension from the file name visible.
If third party plugins (specifically the runners in those plugins) cannot be loaded before the build output panel, the panel would need to be aware that the runner it needs may not be loaded when it renders after a reload and recover from this situation in another way.


Differential w.r.t. multiple variables Any[] not allowed during MOLFiniteDifference discretize

I'm trying to discretize the following Nerst-Plank system in a single spatial dimension; however, I've received the following error during the check_equations() call:
ERROR: ArgumentError: Differential w.r.t. multiple variables Any[τ, Z] are not allowed.
Full code:

using ModelingToolkit, MethodOfLines, OrdinaryDiffEq, DomainSets

@parameters Z τ
@variables c(..) ϕ(..) S(..)
Dz = Differential(Z)
Dzz = Differential(Z)^2
Dτ = Differential(τ)

Z_min = Z_min = τ_min = 0.0
Z_max = 1.0
τ_max = 50.0

# Parameters
iₐ = 10.
i₀ = 20.
D₁ = 1e-11
F = 96487.
R = 8.314
T = 298.15
Lₓ = 50e-6
c₀ = 1000.
Mᵥ = 1.2998e-5
Cᵣ = cᵥ = 1.

# Dimensionless Parameters
δ = iₐ*Lₓ*Cᵣ/(2*F*D₁*c₀)
tD = abs(Cᵣ)*Lₓ^2/(3600*D₁)
Da = i₀*Lₓ/(2*F*D₁*c₀)
Pe = Mᵥ*i₀*3600/(Lₓ*Cᵣ*F)
ϕ₀ = R*T/F

# PDE
eq = [
       Dz(c(Z,τ))*Dz(ϕ(Z,τ))+c(Z,τ)*(Dzz(ϕ(Z,τ)))/(1.0-S(τ))^2 ~ 0.0,
       Dτ(c(Z,τ)) ~ (-Pe*(c(Z,τ)/c₀)^(1/2)*(-ϕ(Z,τ)/ϕ₀)*(1.0-Z)*(Dz(c(Z,τ))))/(tD*(1.0-S(τ)))+(Dzz(c(Z,τ))/(tD*(1.0-S(τ))^2))
]

domains = [Z ∈ Interval(Z_min, Z_max),
           τ ∈ Interval(τ_min, τ_max)]

# BCs
bcs =  [
        c(Z,0) ~ 1.0, #Concentration @ τ=0
        c(Z_max,τ)*(Dz(ϕ(1,τ))) ~ -δ*(1-S(τ)), #ϕ @ Z = 1
        ϕ(Z_max,τ) ~ 0.0, #ϕ @ Z = 1
        Dz(c(Z_max,τ)) ~ -δ*(1-S(τ)), #Concentration @ Z = 1
        Dz(ϕ(Z_max,τ)) ~ 0.0, #ϕ @ Z = 0
        Dz(ϕ(Z_min,τ))/(1-S(τ)) ~ -Da*(c(Z_min,τ)^(1/2))-ϕ(Z_min,τ), #ϕ BC @ Z = 0
        Dz(c(Z_min,τ))/(1-S(τ)) ~ -Da*c(Z_min,τ)^(1/2)*(-ϕ(Z_min,τ))+cᵥ*Pe*c(Z_min,τ)^(3/2)*(-ϕ(Z_min,τ)) #Concentration @ Z = 0
       ]

@named pdesys = PDESystem(eq,bcs,domains,[Z,τ],[c(Z,τ), ϕ(Z,τ)])

N = 10

dz = (Z_max-Z_min)/N

order = 2

discretization = MOLFiniteDifference([Z => dz], τ, approx_order=order, grid_align=center_align)

# Convert the PDE problem into an ODE problem
prob = discretize(pdesys,discretization)

Full error:
Discretization failed at structural_simplify, please post an issue on https://github.com/SciML/MethodOfLines.jl with the failing code and system at low point count.

ERROR: ArgumentError: Differential w.r.t. multiple variables Any[τ, Z] are not allowed.
 If anyone has a better approach to this, I'm definitely open to suggestions.

You'll want to include your equation for Dτ(S(τ)) in eq. Could you share the system of equations at low N, say 4? Should be output above the error.

Ah sorry I misspoke - S(tau) is what should go in the pdesys.
I can see the problem immediately, it stems from an issue with rules matching with odd ordered derivatives, specifically the case where you have 2 odd ordered derivatives multiplying each other - defining an auxiliary variable for Dz(c(Z,τ)) with an equation dzc(Z, tau) ~ Dz(c(Z,τ)), supplying ics and bcs for this variable should be a workaround.
I will take a close look at this soon, likely will have a branch that fixes this within a week -I'll ping you when that happens. in the meanti

After creating the auxiliary variable, I've encountered a similar error to #78:

ERROR: AssertionError: Variable dzc(Z, τ) does not appear in any equation, therefore cannot be solved for
Stacktrace:
[1] build_variable_mapping(m::Matrix{Int64}, vars::Vector{Any}, pdes::Vector{Equation})
@ MethodOfLines ~/Documents/Git/MethodOfLines.jl/src/interiormap.jl:69
[2] MethodOfLines.InteriorMap(pdes::Vector{Equation}, boundarymap::Dict{Sym{SymbolicUtils.FnType{Tuple, Real}, Nothing}, Dict{Sym{Real, Base.ImmutableDict{DataType, Any}}, Vector{Any}}}, s::MethodOfLines.DiscreteSpace{1, 4, MethodOfLines.CenterAlignedGrid})
@ MethodOfLines ~/Documents/Git/MethodOfLines.jl/src/interiormap.jl:20
[3] symbolic_discretize(pdesys::PDESystem, discretization::MOLFiniteDifference{MethodOfLines.CenterAlignedGrid})
@ MethodOfLines ~/Documents/Git/MethodOfLines.jl/src/discretization/MOL_discretization.jl:52
[4] discretize(pdesys::PDESystem, discretization::MOLFiniteDifference{MethodOfLines.CenterAlignedGrid})
@ MethodOfLines ~/Documents/Git/MethodOfLines.jl/src/discretization/MOL_discretization.jl:146
[5] top-level scope
@ REPL[35]:2

I've also pulled the recommend branch for this issue, but ran into the same error. There appears to be nothing returned from the j = findfirst(isequal(0), cols) line in build_variable_mapping() located in interiormap.jl. This appears to stem from the creation of m from pdesys.eqs and the s = DiscreteSpace(domain, alldepvars, allindvars, discretization). Any thoughts?
Full code:
using ModelingToolkit, MethodOfLines, OrdinaryDiffEq, DomainSets

@parameters Z τ
@variables c(..) ϕ(..) S(..) dzc(..)
Dz = Differential(Z)
Dzz = Differential(Z)^2
Dτ = Differential(τ)

Z_min = Z_min = τ_min = 0.0
Z_max = 1.0
τ_max = 50.0

# Parameters
iₐ = 10.
i₀ = 20.
D₁ = 1e-11
F = 96487.
R = 8.314
T = 298.15
Lₓ = 50e-6
c₀ = 1000.
Mᵥ = 1.2998e-5
Cᵣ = cᵥ = 1.

# Dimensionless Parameters
δ = iₐ*Lₓ*Cᵣ/(2*F*D₁*c₀)
tD = abs(Cᵣ)*Lₓ^2/(3600*D₁)
Da = i₀*Lₓ/(2*F*D₁*c₀)
Pe = Mᵥ*i₀*3600/(Lₓ*Cᵣ*F)
ϕ₀ = R*T/F

# PDE
eq = [
       dzc(Z,τ) ~ Dz(c(Z,τ)),
       dzc(Z,τ)*Dz(ϕ(Z,τ))+c(Z,τ)*(Dzz(ϕ(Z,τ)))/(1.0-S(τ))^2 ~ 0.0,
       Dτ(c(Z,τ)) ~ (-Pe*(c(Z,τ)/c₀)^(1/2)*(-ϕ(Z,τ)/ϕ₀)*(1.0-Z)*(dzc(Z,τ)))/(tD*(1.0-S(τ)))+(Dzz(c(Z,τ))/(tD*(1.0-S(τ))^2)),
       Dτ(S(τ)) ~ (Pe*ϕ(Z,τ)*c(Z,τ)^(1/2))/(tD)
]

domains = [Z ∈ Interval(Z_min, Z_max),
           τ ∈ Interval(τ_min, τ_max)]

# BCs
bcs =  [
        #Initial
        c(Z,0) ~ 1.0, # Concentration @ τ=0
        S(0) ~ 0.02, # Moving Boundary @ τ=0

        #Boundary
        c(Z_max,τ)*(Dz(ϕ(1,τ))) ~ -δ*(1-S(τ)), # ϕ @ Z = 1
        ϕ(Z_max,τ) ~ 0.0, # ϕ @ Z = 1
        Dz(c(Z_max,τ)) ~ -δ*(1-S(τ)), # Concentration @ Z = 1
        Dz(ϕ(Z_max,τ)) ~ 0.0, # ϕ @ Z = 0
        Dz(ϕ(Z_min,τ))/(1-S(τ)) ~ -Da*(c(Z_min,τ)^(1/2))-ϕ(Z_min,τ), # ϕ BC @ Z = 0
        Dz(c(Z_min,τ))/(1-S(τ)) ~ -Da*c(Z_min,τ)^(1/2)*(-ϕ(Z_min,τ))+cᵥ*Pe*c(Z_min,τ)^(3/2)*(-ϕ(Z_min,τ)), # Concentration @ Z = 0
        dzc(Z_max,τ) ~ -δ*(1-S(τ)),
        dzc(Z_min,τ)/(1-S(τ)) ~ -Da*c(Z_min,τ)^(1/2)*(-ϕ(Z_min,τ))+cᵥ*Pe*c(Z_min,τ)^(3/2)*(-ϕ(Z_min,τ)) # Concentration @ Z = 0
       ]

@named pdesys = PDESystem(eq,bcs,domains,[Z,τ],[c(Z,τ), ϕ(Z,τ), S(τ), dzc(Z,τ)])

N = 4

dz = (Z_max-Z_min)/N

order = 2

discretization = MOLFiniteDifference([Z => dz], τ, approx_order=order, grid_align=center_align)

# Convert the PDE problem into an ODE problem
prob = discretize(pdesys,discretization)

I've tracked the issue to this equation Dτ(S(τ)) ~ (Pe*ϕ(Z,τ)*c(Z,τ)^(1/2))/(tD). I don't know what to do about this since the lhs has no Z dependence, but the rhs does.


EKS cluster with Calico networking webhooks are not reachable if not running in NodePort

(This is used to request new product features, please visit https://discuss.istio.io for questions on using Istio)
Describe the feature request
Describe alternatives you've considered
Affected product area (please put an X in all that apply)
[ ] Ambient
[ ] Docs
[ X] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Extensions and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Affected features (please put an X in all that apply)
[ ] Multi Cluster
[ ] Virtual Machine
[ ] Multi Control Plane
Provide option to allow users to provide webhook proxy service name and namespace when installing via Helm for base and istiod charts

IMO, NodePort is not recommended in Production. I do not think we should provide things in official helm chart.
Special use case can try with https://istio.io/latest/docs/setup/additional-setup/customize-installation-helm/

Yeah, that's why we need to have an option to allow users to have proxy service which might be running in different namespace as different service in same cluster.

@ranga543 how can I reproduce?

This can be closed, since we can use kustomization to modify specific resource before helm do actual installation. https://istio.io/latest/docs/setup/additional-setup/customize-installation-helm/


'Ctrl + h' not working on Chrome + Linux

I searched a little and just discovered that Chrome always return false to event.ctrlKey and other keys (e.g. event.shiftKey) on keypress.
So, the hide function doesn't work on Chrome + Linux (34+ as far as I know).

Interesting. Did anyone mention a work-around? I have implemented key combinations like this before and haven't had any issues in Chrome.

I'll look for workarounds (Having "ctrl" shotcuts in my actual project would be great). But, so far, none of them worked.

I've managed to fix by just changing the 'keypress' event to 'keydown' event (yeah, hours looking for workarounds and just tried that now).

Cool. Looks like it was merged in so you should be :100: here.

Out in 0.1.2, thank you a lot.


Add Spring Boot snippets

The snippets of materials around Spring / Spring Boot should be collated into one place along with tests.

Done


Unpin Mypy

Let's start fixing Mypy 1.4.

Now we have two problems! ✨
Wanna unlock 3.11 too?

I unlocked it on all CPythons, looks ✅


bld extension for things that generates code (ex. dagger2)

how one could create an extension to perform such task?
i was troubleshooting a dagger2 issue and was using bld for fast investigation but got stuck on that part.

Not sure an extension would be needed, but I'm not that familiar with Dagger, so bare with me.
Isn't Dagger using an annotation processor? In that case, take a look at:

https://forum.uwyn.com/post/36

If an extension is needed, we have many extensions that call other tools.
If you can outline what you'd like the extension to do, I'd be happy to help.

Closing since it doesn't seem like this should be a generic extension specifically for code generation.


Check if user is logged

i want to print user first name and last name in navigation after login
i use but it is not working. Can you help?
Thanks
$rootScope.$watch($auth.isAuthenticated(), function() {
      if($auth.isAuthenticated() && !angular.isDefined($scope.user)) {
        Account.getProfile()
        .success(function(data) {
          $scope.user = data;
        })
        .error(function() {
        });
      }
    });

EDIT: Be sure to format the code corretctly next time @marijang . Hard to read otherwise.

what is in api/me ?

@marijang what is "Account" module?
I am also wondering about get user data on success authentication..
In my login.controller.js file i have written below to get data but not working..
 vm.authenticate = function (provider)
        {
            console.log("login initial");
           var res= $auth.authenticate(provider);
           console.log(res);
        };

In my login.tmpl.html
<button ng-click="vm.authenticate('facebook')"  class="btn btn-facebook btn-sm btn-block">Register with Facebook</button>
                                             <button ng-click="vm.authenticate('google')"  class="btn btn-google-plus btn-sm btn-block">Register with Google Plus</button>

can you help me?


update with modern wrapper and style

This PR comes with two major changes.

I rewrote the CLI with a typer, which provides a more modern and robust command line interface.
For a big part of the kernel, rewrote with numpy style and in a more pythonic way to enhance readability.

I addressed most comments with a new commit except for the add_OP function. The if statement makes it hard to consolidate those matrix constructions without introducing a new function.
I agree with adding more tests. Maybe some synthetic data for sanity checks.


DROOLS-733 - Added test verifying task comments.

Shows marshalling error in JAXB marshaller - cannot marshall TaskComment.

Refactored according to comments, added deleteTaskComment() verification.

merged and fixed the marshaling issues. thanks @sutaakar


Prepare release 1.0.5

Current coverage is 58.78% (diff: 100%)

No coverage report found for master at 37ad657.

Powered by Codecov. Last update 37ad657...a96b644


cargo clean failure with dangling readonly dir-symlinks on windows

Problem
Rustup creates symlinks within target/ in its test suite.
Here is an example:
stat target/tests/running-test-5D1PNf/rustupiIcEHF/toolchains/default-from-path
  File: target/tests/running-test-5D1PNf/rustupiIcEHF/toolchains/default-from-path -> /c/Users/username/Documents/src/rustup.rs/target/tests/running-test-5D1PNf/rustup-customsOflBt/custom-1
  Size: 102             Blocks: 0          IO Block: 65536  symbolic link
Device: de41ba52h/3728849490d   Inode:<PHONE_NUMBER>791307  Links: 1
Access: (0777/lrwxrwxrwx)  Uid: (197609/ robertc)   Gid: (197609/ UNKNOWN)
Access: 2023-02-27 11:44:54.557419600 +0100
Modify: 2023-02-27 11:44:54.557419600 +0100
Change: 2023-02-27 11:44:54.559622400 +0100
 Birth: 2023-02-27 11:44:54.557419600 +0100

and in cmd:
dir
 Volume in drive C has no label.
 Volume Serial Number is DE41-BA52

 Directory of C:\Users\username\Documents\src\rustup.rs\target\tests\running-test-5D1PNf\rustupiIcEHF\toolchains

27/02/2023  11:44 AM    <DIR>          .
20/03/2023  11:22 PM    <DIR>          ..
27/02/2023  11:44 AM    <JUNCTION>     default-from-path [\??\C:\Users\username\Documents\src\rustup.rs\target\tests\running-test-5D1PNf\rustup-customsOflBt\custom-1]
               0 File(s)              0 bytes
               3 Dir(s)  127,863,406,592 bytes free

Inspecting with explorer shows that the link is a readonly dir link.

This is a dangling link (likely because cargo clean deleted the target first).
stat /c/Users/username/Documents/src/rustup.rs/target/tests/running-test-5D1PNf/rustup-customsOflBt/
custom-1
stat: cannot stat '/c/Users/username/Documents/src/rustup.rs/target/tests/running-test-5D1PNf/rustup-customsOflBt/custom-1': No such file or directory

cargo clean fails:
cargo clean
error: failed to remove build artifact

Caused by:
  failed to remove file `C:\Users\username\Documents\src\rustup.rs\target\tests\running-test-5D1PNf\rustupiIcEHF\toolchains\default-from-path`

Caused by:
  Access is denied. (os error 5)

rm from within bash deletes the file successfully, which permits cargo clean to proceed.
I haven't tested yet, but I suspect the remove_dir_all function in the stdlib would handle this correctly. I have tested the remove_dir_all crate's equivalent API, and it does handle this correctly.
The code in _remove_file  calls a helper that doesn't use symlink_metadata, rather metadata, which will fail on dangling symlinks.
Separately I also note that the entire routine is based on path based calls rather than _at style calls, which will incur higher traversal costs in-kernel, and is serial, even though directory removal is embarrassingly parallel : if you're interested I could look at what changes would be needed to the remove_dir_all crate to support the UI cargo wants, to permit switching to that. Parallel deletion of installed toolchains had a measurable speed improvement for rustup, both in our test suite and for users.
Steps
No response
Possible Solution(s)
No response
Notes
No response
Version
$ cargo version --verbose
cargo 1.67.0-beta.2 (f6e737b1e 2022-12-02)
release: 1.67.0-beta.2
commit-hash: f6e737b1e3386adb89333bf06a01f68a91ac5306
commit-date: 2022-12-02
host: x86_64-pc-windows-msvc
libgit2: 1.5.0 (sys:0.15.0 vendored)
libcurl: 7.86.0-DEV (sys:0.4.59+curl-7.86.0 vendored ssl:Schannel)
os: Windows 10.0.22621 (Windows 10 Pro) [64-bit]

somewhat related https://github.com/rust-lang/cargo/pull/11442

Mentioned on #11442
Root cause is std::remove_dir_all.
The question is:
Do you continue to reimplement std::fs::remove_dir_all in cargo-util::paths::*
Or attempt to change std::fs::remove_dir_all to resolve?
I'm unfamiliar with how difficult submitting a change to std is in Rust.

#11442 has landed on beta channel. Is this issue still relevant?

Yes, the problem persists. The solution is to carefully go about every place where cargo decides how to delete a fs entry and check FileTypeExt::is_symlink_dir on Windows. I've dug a little bit, and it seems to me there are at least two places to patch: cargo_util::paths::remove_dir_all and CleanContext::rm_rf in cargo_clean.rs. I can come up with a patch.
BTW, std::fs::remove_dir_all does the right thing.


fixed typo in Check class

class Check(LLMChain):
    @classmethod
    def from_llm(cls, **kwargs) -> LLMChain:
        return cls(
            **kwargs,
            prompt=PromptTemplate.from_template(
                dedent(
                    """\
                    You are evaluating the result of an AI agent against its task to verify its accurate completion. # <=

                    Here is the relevant context: {context}

                    Here is the result: {learning}

                    If the result accomplishes its task, return "Complete".
                    Otherwise, return new task for the agent to complete.

                    Do not embellish.

                    Task: \
                    """
                ),
            ),
        )


Android client Characteristic name is always "Unknown"

Hello, how can I set the characteristic name ?
My function  :
var GetConfiguration = function()
{
GetConfiguration.super_.call(this,
{
uuid:       '31E98556-045A-4A22-9E88-6BF944352C21',
properties: ['read', 'write', 'indicate'],
value:      null
});
this._value                 = new Buffer(0);
this._updateValueCallback   = null;

};
In my android client I receive "Unknown" for the characteristic name :

Thanks,

I also would like to know how to set the name of both Characteristics and Services.
Thanks.

@ducmaxime this might not be possible on Android, the name might only be set for Bluetooth SIG standard UUID's:

https://developer.bluetooth.org/gatt/services/Pages/ServicesHome.aspx?_ga=1.44626528.591292021.1365037697
https://developer.bluetooth.org/gatt/characteristics/Pages/CharacteristicsHome.aspx?_ga=1.44626528.591292021.1365037697

You could trying adding a user description descriptor to the characteristic, see:
https://github.com/sandeepmistry/bleno/blob/master/test.js#L17-L21
@don do you have any info on this?

I'm closing this, as nothing can be done the bleno side.


USB Cable support

As a USB cable user, I want to be able to run "de Joa" on my USb cable.
This includes all USB cables.
Reaching from short to long.

Proposed edit: b


Lombok configuration and logs

This pull attempts to add initial support for #475
I only applied the changes to 3 modules just to present progress and query opinions. If the changes are satisfactory, I'll continually update this pull with other modules.
This pull only takes advantage of @Slf4j of Lombok. Other changes can be done in separate pulls, such as generating toString, etc. Loggers are configured to be static LOGGER elements so as to not conflict with parent logger field that we might have, or might inherit from other classes in Spring, etc.
I have modified the build process so that the aspectj plugin can take in the compiled code from javac+lombok and weave those artifacts.
I also realized that the aspectj plugin only applies the aspect to the core module and not the rest. This is because the Aspect file is only available to the cas-server core. I have added a few more aspect files to make sure we can produce aspects for all modules.
(Note that there are other alternatives, but this seemed like the simplest case for now. I will open up a separate issue for the fix which is: create a separate module that only contains the aspect, and reference the module as an aspectj library in the plugin configuration)
Note that for this to work in your IDE, specially IDEA, you should:

Enable annotation processing for Javac
Make sure Javac is the compiler
Install the lombok plugin for IntelliJ.

Sorry for the time spent on this, but I'm not in favor of this change (adding Lombok). To be short: little value, more magic and a more complicated build.

No worries. I sort of wanted to demonstrate the functionality. I was hoping that using annotations would certainly reduce code clutter, but that may not be worth it since as you pointed out, it's just too magical :) I am also curious to see what others think.
There is however another issue which I think we should address in a separate PR and that is the application of aspect functionality to all modules, as opposed to just core.


Found Inf or NaN global norm.

InvalidArgumentError (see above for traceback): Found Inf or NaN global norm. : Tensor had Inf values
I've been getting this error in the latest commit making the loss absurdly negative number or Nan. I tested with the previous versions the computation of loss is fine.

me too，do u resolve this question?

The issue was not resolved in the latest commit, instead I used the previous commit which works without this error

which code? can u send the link to me？ thank you

It should be ok by now :)

which code? can u send the link to me？ thank you

I think I used 2018 commintted code. Now Dany has fixed the issue for the latest release.


Add description to projects and forms

Non-urgent feature suggestion: a plain text (or markdown) field "description" for projects and forms to let admins add a short but friendly description for users.
Detail views could show description next to project/form title (positioning see "let's get some things done" catchphrase).
List views could show description as a column after title.
Update could happen in respective project/form settings tab.

hey florian!
in general, we prefer feature requests to go on the forum so more people can participate in them.
in the case of this one, it's been planned! i'll try to work it in soon. thanks for the feedback. :)

Sweet, thanks! Will post to forum next time ;-)
re story 5.33, working on submission to ROpenSci (process at  https://github.com/ropensci/software-review/issues/335) hoping to get adopted before mid October.

:) neat!

Now that we're running prod and UAT sites for ODK Central, setting the colour and title of the navbar would be a useful visual cue. Currently I have to squint at the URL to see whether I'm in prod or UAT.


Create 37i9dQZF1E37gRKKOu8Pkb

daily mix 1 for spotify user katzrockso

Also would note that since daily mixes are generated individually based on the account, if more than one person wants one, the names for the pretty files would have to be altered somehow.

it's autogenerated by spotify, so i don't think changing the title is possible

i think it'd be possible if you included a line in the plain file for custom titles
and then generated the pretty file name based off of that custom title if it exists
i'm trying to read through the code right now to see where a potential change could be made

where are you actually calling plain() from the formatter class, sorry

closed until i can think of another solution


Update PulseColorationShader to Foundry VTT 0.7.9

Which class should be updated?
PulseColorationShader
For which version should the class be updated?
Foundry VTT 0.7.9

taking this

Fixed by #276


Minor visual changes

Fixed panel borders/shadow/padding
Removed double margin from .table-details > .row

Before:

After:


add a markdown math intro

What does this PR do?
Add Resource(s)
For resources
Description
add a markdown math intro
Why is this valuable (or not)
当前的zh版本markdown没有添加对公式处理的说明，新增的这篇填补了这块的空白
How do we know it's really free?
来源于自己的博客地址
For book lists, is it a book?
no
Checklist:

[ ] Not a duplicate
[ ] Included author(s) if appropriate
[ ] Lists are in alphabetical order
[ ] Needed indications added (PDF, access notes, under construction)

so if it's not a book...

你最好 在github托管一个仓库，或者做成gitbook那种，这样全面点

The link is dead, I guess we can close this PR.


Add timestamper plugin

This plugin add timestamps to console output.
It is useful for longer running or multi-step jobs, especially
when they are not doing what they are supposed to.
See https://wiki.jenkins-ci.org/display/JENKINS/Timestamper

:+1: Looks useful


error on openOptionsPage

Describe the bug
When clicking the "Open Options" button of pop up, error Uncaught TypeError: Cannot read properties of undefined (reading 'openOptionsPage') is thrown on browser.runtime.openOptionsPage().

Reproduction

pnpm dev 2. open extension pop up 3. click "Open Options" 4. option page won't open, observed error Uncaught TypeError: Cannot read properties of undefined (reading 'openOptionsPage')

System Info
System:
    OS: Linux 6.5 Manjaro Linux
    CPU: (4) x64 Intel(R) Core(TM) i5-6200U CPU @ 2.30GHz
    Memory: 504.36 MB / 7.66 GB
    Container: Yes
    Shell: 5.9 - /bin/zsh
  Binaries:
    Node: 21.6.2 - /usr/bin/node
    Yarn: 1.22.21 - /usr/bin/yarn
    npm: 10.5.0 - /usr/bin/npm
    pnpm: 8.15.2 - /usr/bin/pnpm
    bun: 1.0.35 - ~/.bun/bin/bun
  Browsers:
    Chromium: 123.0.6312.86

Used Package Manager
pnpm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guide.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A, please open a GitHub Discussion instead.
[X] The provided reproduction is a minimal reproducible of the bug.

I don't quite understand the error behind. But I played around a bit, manually importing import browser from 'webextension-polyfill' in Popup.vue or updating the auto import config 'webextension-polyfill': [             ['*', 'browser'],           ],  to 'webextension-polyfill': [             ['default', 'browser'],           ], seems to fix the undefined error

updating the auto import config 'webextension-polyfill': [ ['*', 'browser'], ], to 'webextension-polyfill': [ ['default', 'browser'], ], works in dev, but breaks in production.
it seems like a bug in the unplugin-auto-import (not sure), PRs welcome :)

hi @tmkx thanks for following up. It does look like a limitation of unplugin-auto-import.
Based on the generated code from unplugin-auto-import, when 'webextension-polyfill': [ ['*', 'browser'] ] is configured, the Browser type is generated correctly (however, the browser object imported is not - the actual browser alias needed is in fact browser.default, and thus the reason browser.runtime is undefined). Likewise, if 'webextension-polyfill': [ ['default', 'browser'] ] is used, the imported object is correct (while Browser type is not).

updating the auto import config 'webextension-polyfill': [ ['*', 'browser'], ], to 'webextension-polyfill': [ ['default', 'browser'], ], works in dev, but breaks in production.将自动导入配置 'webextension-polyfill': [ ['*', 'browser'], ], 更新为 'webextension-polyfill': [ ['default', 'browser'], ], 在 dev 中有效，但在生产中中断。
it seems like a bug in the unplugin-auto-import (not sure), PRs welcome :)这似乎是 unplugin-auto-import 中的一个错误（不确定），PR 欢迎:)

That works for me

updating the auto import config 'webextension-polyfill': [ ['*', 'browser'], ], to 'webextension-polyfill': [ ['default', 'browser'], ], works in dev, but breaks in production.
it seems like a bug in the unplugin-auto-import (not sure), PRs welcome :)

That works for me

me too

I have another problem.
After updating "webextension-polyfill": [["*", "browser"]], to "webextension-polyfill": [["default", "browser"]],, it can run without error, but the type prompt for browser is no longer available.

I have another problem. After updating "webextension-polyfill": [["*", "browser"]], to "webextension-polyfill": [["default", "browser"]],, it can run without error, but the type prompt for browser is no longer available.

I restored "webextension-polyfill": [["default", "browser"]], to "webextension-polyfill": [["*", "browser"]],, then add import browser from "webextension-polyfill" in Popup.vue, it works.
But I'm not sure if this is correct, hope there is a better solution.

遇到同样的问题，本来改成了 default，但类型变成了 any
想着去掉 auto-import.d.ts 里面的类型，自己声明个类型，又发现 unimport 这个库有问题，配置 dtsDisabled 没反应，跑去改了改
但又感觉以后都要自己加类型太麻烦，转了一圈回来还是研究一下这个问题
目前小修了一下，应该可以了，不知道还有没有更好的方法


Expose Userprofile endpoint in Access-Management backend

Description
To support language, we need to get profile information to the frontend.
This is done through exposing a profile endpoint in Access Management backend
Tasks

[ ] Expose userprofile controller with userprofile information. Look at Profile controller in App

Acceptance Criteria

[ ] Userprofile is exosed
[ ] Only own userprofile is available
[ ] Uses Userprofile component as source

@Sondre-B denne er overtatt av @ekorra

Implementation has since been moved to altinn-access-manage and been tested and verified in both AT and TT environments


New Feature: Issue Sync

I've only just barely started work on this, but thought I'd drop a note in case anyone was interested in helping out with it or has some early feedback, ideas, suggestions, or recommendations...
I'm already using this plugin to keep git mirrors on Trac in sync with our authoritative repos on GitHub, but have found the lack of any Trac plugins with the ability to sync GitHub pull requests over to Trac, or sync activity on Trac back over to the appropriate GitHub pull requests to be painful. I'm sure others like us find GitHub issues severely lacking where Trac excels, or have just been using Trac issues for so long that it wouldn't be cost effective to switch have also wished for this kind of functionality. Everyone wants an official repository on GitHub for visibility and collaboration, but finds it annoying that pull requests can't be disabled when you still want patches submitted to Trac.
I've decided to extend this plugin with the ability to sync issue activity between Trac and GitHub. At least initially, I'm aiming mostly just for creating new Trac tickets for every new pull request, and automatically attaching patches generated from those pull requests to them, along with any updates to those pull requests. That's my primary priority.
After that, I'm planning to look into syncing ticket updates to those issues back to their original GitHub pull requests (which will likely require a GitHub account to post on behalf of, involving API keys), so developers only need to update the ticket on Trac, and the original contributor will still be notified on GitHub, and can still leave comments and updates there.
Others might be interested in turning on GitHub issues, and also syncing new Trac tickets over to GitHub if the plan is to eventually just switch to GitHub issues, but I'm not planning on doing that at all myself for now. Also, that workflow can mostly be avoided by just removing the TICKET_CREATE permission from Trac users, and directing them to GitHub.
I'll be working on this within my issue-sync branch here:
https://github.com/aaugustin/trac-github/compare/master...tierra:issue-sync

Thanks for letting me know!
Some preliminary advice:

Keep features optional so users can mix and match according to their use case.
Check out the "test framework". It's ad-hoc, but it works. Currently the plugin has almost 100% test coverage. Try to write tests for the new features.
Don't forget to update the documentation in the README.

Since this is a significant extension in the scope of trac-github, and not one I'm likely to use, perhaps I'll end up merging it and give you commit access to this repo :-)
If that doesn't work for whatever reason, and you don't want to fork, you could write a plugin that depends on this one and extends its classes. I'm willing to shuffle code around to make this easier.

I was planning on opening a pull request down the road when it's looking closer to finished (and yes, with adequate unit tests as well). I figured it would be best to combine more GitHub-related sync features into an existing plugin that already provides some GitHub-related sync features, otherwise I would have just started writing it as a new, independent plugin.
I guess that might end up back up for discussion though if I end up pulling in any 3rd party GitHub API python library down the road for that secondary feature allowing ticket updates to sync back to GitHub pull requests. Maybe it will end up being best to leave it in it's own plugin at that point.

It's hard to tell where this will end when you're just starting to write the code :-)
Feel free to ping me in this issue if you have questions!

If anyone wants to take @tierra's patch, update it to apply on the latest version of master, and add tests, that's certainly heplful.

If anyone wants to take @tierra's patch, update it to apply on the latest version of master, and add tests, that's certainly heplful.

OK, I started PR #138 for that.


Cant Move expander inside the content-wrapper

Hi
Am trying to move the expander inside the content-wrapper. So at render time, i see my own expander icon and title of the node inside the same rectangle.
<ng-template #treeNodeWrapperTemplate let-node let-index="index">

  **

    **<tree-node-expander class=" glyphicon glyphicon-chevron-down" style="display:inline-block" [node]="node"></tree-node-expander>**
    <tree-node-content [node]="node" [index]="index"></tree-node-content>

But i dont see it being honored at the render time, its reset to the default markup, back to being at sibling level to node-content-wrapper.

Is this possible at all ? Am i missing something?

Hi this is strange, can you share the entire file?
Please take a look at the example's code and see if you missed anything:
https://github.com/500tech/angular-tree-component/blob/master/example/cli/src/app/templates/templates.component.ts

Hi
Thanks, think it was my browser acting up. A restart helped.

I want to replace the existing expander with the font awesome icon fa-plus-square-o, how can I do that. Thanks

@nileshpatva did you find a solution to that, I am also trying to add a custom icon to the tree.


0E33: GET_PICKUP_THIS_COORD can break 0214: HAS_PICKUP_BEEN_COLLECTED

There are two issues related to 0E33: get_pickup_this_coord

If the Only_Available param is True, then the if (!onlyAvailable || !CPickups::IsPickUpPickedUp(pickupRef)) check can prevent a collected pickup from being detected as picked up again by the script that "owns" the pickup. When a pickup is detected as collected by opcode 0214, it's handle is cleared from the CollectedPickups array so it won't be detected as collected again - unless it respawns and is collected again. Save disks can be picked up but won't show the save screen. Property is payed for but the mini-mission won't unlock the property.
I suspect that the purpose of the Only_Available param is to filter out pickups that have been permanently collected, deleted, or never used so that only valid pickup handles are returned. Checking the type of the pickup might provide a better test as any pickup that is not type 0 is a valid pickup.

More information and a cleo repro script can be provided if needed. I suspect that you'll figure it out quickly enough. Seemann and I argued out the logic of the issue in the Sanny Discord server's #sdk-and-plugins channel. This discussion may provide additional details and context.
https://discordapp.com/channels/911487285990674473/938823120242999326/1106000841841848392

I use this on Buy Property Mod and works, or maybe my case is not influenced by this issue, I don't know, I don't remember how I used this and I have no idea when I will be able to verify this issue.

It could be that I misunderstood the purpose of the Only_Available parameter, that the opcode is functioning properly and won't cause any issues when used as intended, and that existing scripts might break if changes are made to the filter. Clarifying the purpose of the opcode might be the best option for avoiding conflicts - a warning against using the Only_Available option on any pickup not owned by the script.
I am in no rush for any changes, if necessary. I wrote a custom function modeled on 0E33, but it includes a search radius, and can continue the search after the previous handle. While testing this function I discovered that I could break the collection detection of standard and custom scripts if I intentionally used 0E33 incorrectly.

@OrionSR can you test 0E33 with this version. I replaced CPickups::IsPickUpPickedUp with the type check
CLEO+.zip

The updated Cleo+ linked above passed two types of tests with no issues.

Does 0E33 still break 0214's collection detection? No. The repro script I designed to force glitches with the collection of save disks, buy properties, and custom pickups did not encounter any errors with the linked Cleo+.

Does 0E33 return the appropriate pickup handles? Yes.   Five identical pickups were created in the same location. The 1st, 2nd, and 5th pickup were immediately destroyed. 0E33 returned the handle of the 1st pickup when Only_Available == 0. 0E33 returned the handle of the 3rd pickup when Only_Available == 1.

These results match my expectations for 0E33 using a pickup Type check.
Technicalities: These tests were conducted on a baseline save with a pristine pickup pool. Normally new pickups fill the first free slot, but it is possible for the 5th pickup created in sequence to have a lower index than the 1st pickup. When no pickup slots are free, the game will start recycling dropped pickups that are waiting to expire the next time the player is near.


Overpasss request and searcharea

i don't know how introduce searcharea in overpass request. when i try with overpass request which is ok, mapcontrib failed

Yes, areas are not supported right now, I have to rework the handling of OverPass requests.

@vinber Can you give me your OverPass request please?

for example :
[out:json][timeout:25];
// fetch area “Gironde” to search in
{{geocodeArea:Gironde}}->.searchArea;
// gather results
(
// query part for: “amenity=recycling”
node"amenity"="recycling";
way"amenity"="recycling";
relation"amenity"="recycling";
);
// print results
out body;

;
out skel qt;

Ok, it seems that the tricky part is {{geocodeArea:Gironde}}. When I test it on OverPass Turbo, and then I export it, it is turned into area(3600007405).
The new request is now:
[out:json][timeout:25];
// fetch area “Gironde” to search in
{{geocodeArea:Gironde}}->.searchArea;
// gather results
(
  // query part for: “amenity=recycling”
  node["amenity"="recycling"](area.searchArea);
  way["amenity"="recycling"](area.searchArea);
  relation["amenity"="recycling"](area.searchArea);
);
// print results
out body;
>;
out skel qt;

And it works in MapContrib! \o/
I have to search how the transformation happens.

You want to write :

[out:json][timeout:25];
// fetch area “Gironde” to search in
area(3600007405)->.searchArea;
// gather results
(
// query part for: “amenity=recycling”
node"amenity"="recycling";
way"amenity"="recycling";
relation"amenity"="recycling";
);
// print results
out body;
;
out skel qt;

It's not native in overpass query, it's a overload of overpass turbo.

@vinber Yeah my bad ;)
@frodrigo Oh good to know thanks!

This :
area["name"="Gironde"]->.Gironde;
(
node["amenity"="recycling"](area.Gironde);
way["amenity"="recycling"](area.Gironde);
relation["amenity"="recycling"](area.Gironde);
);
out body;
>;
out skel qt;
works, doesn't it ?
(btw nothing works for me if I add [out:json][timeout:25];)

yes, @PierreHachard you're right for request and [out:json][timeout:25];)

Thanks @PierreHachard!
@vinber The [out:...] and [timeout:...] thnigs are fixed in the current development version, and it will be released next week ;)

It seems that the @PierreHachard's solution fixed your problem @vinber.
Is it right ok you? I close the issue?


RTCMultiConnection support on mobile browser???

HiHi @muaz-khan !
I have a question to ask you !
RTCMultiConnection support on the phone ? If can , I want broadcast from desktop to the mobile browser?? How to fix it!
Thanks!

RTCMultiConnection-v3 supports phonegap/cordova based iOS+Android applications:

https://github.com/muaz-khan/RTCMultiConnection/blob/master/docs/ios-android.md

You can EVEN broadcast video from iOS/Android as well.
THE ONLY LIMITATION is, your mobile app can NOT share its screen. Otherwise everything else works fine from mobile-to-PC and vice versa.

Hi @muaz-khan !
Thank you for your answer.
It means that from the web on your desktop can not broadcast on the mobile browser . Mobile only runs when developing app for it.