initial run errors - FirebaseOptions

I wanted run this application for the first time, so I installed flutter from https://docs.flutter.dev/get-started/install/linux, next run all steps from Development section, then run:
flutter run
Launching lib/main.dart on Chrome in debug mode...
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_riverpod-1.0.3/lib/src/framework.dart:275:26: Warning: Operand of null-aware operation '!' has
type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('../../snap/flutter/common/flutter/packages/flutter/lib/src/scheduler/binding.dart').
    if (SchedulerBinding.instance!.schedulerPhase ==
                         ^

of course Chrome with debug mode started, but stopped with error on console:
┌───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────
│ #0   packages/exerlog/src/utils/logger/logger.dart 29:58                  info
│ #1   packages/exerlog/src/utils/logger/riverpod_logger.dart 13:9          didUpdateProvider
├┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄┄
│ 💡 {
│ 💡   "provider": "FutureProvider<FirebaseApp>",
│ 💡   "newValue": "AsyncError<FirebaseApp>(error: Assertion failed:
file:///home/developer/snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/firebase_core_web-1.7.1/lib/src/firebase_core_web.dart:207:11
│ 💡 options != null
│ 💡 "FirebaseOptions cannot be null when creating the default app.", stackTrace: dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 251:49  throw_
│ 💡 dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 29:3    assertFailed
│ 💡 packages/firebase_core_web/src/firebase_core_web.dart 207:18                  initializeApp
│ 💡 dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 45:50            <fn>
│ 💡 dart-sdk/lib/async/zone.dart 1685:54                                          runUnary
│ 💡 dart-sdk/lib/async/future_impl.dart 147:18                                    handleValue
│ 💡 dart-sdk/lib/async/future_impl.dart 766:44                                    handleValueCallback
│ 💡 dart-sdk/lib/async/future_impl.dart 795:13                                    _propagateToListeners
│ 💡 dart-sdk/lib/async/future_impl.dart 566:5                                     [_completeWithValue]
│ 💡 dart-sdk/lib/async/future.dart 527:22                                         <fn>
│ 💡 dart-sdk/lib/async/zone.dart 1685:54                                          runUnary
│ 💡 dart-sdk/lib/async/future_impl.dart 147:18                                    handleValue
│ 💡 dart-sdk/lib/async/future_impl.dart 766:44                                    handleValueCallback
│ 💡 dart-sdk/lib/async/future_impl.dart 795:13                                    _propagateToListeners
│ 💡 dart-sdk/lib/async/future_impl.dart 566:5                                     [_completeWithValue]
│ 💡 dart-sdk/lib/async/future_impl.dart 639:7                                     callback
│ 💡 dart-sdk/lib/async/schedule_microtask.dart 40:11                              _microtaskLoop
│ 💡 dart-sdk/lib/async/schedule_microtask.dart 49:5                               _startMicrotaskLoop
│ 💡 dart-sdk/lib/_internal/js_dev_runtime/patch/async_patch.dart 166:15           <fn>
│ 💡 )"
│ 💡 }
└───────────────────────────────────────────────────────────────────────────────────────────────────────────────────────

══╡ EXCEPTION CAUGHT BY WIDGETS LIBRARY ╞═══════════════════════════════════════════════════════════
The following TypeErrorImpl was thrown building MyApp(dirty, dependencies:
[UncontrolledProviderScope], state: _ConsumerState#c34bc):
Expected a value of type 'String?', but got one of type 'AssertionErrorImpl'

The relevant error-causing widget was:
  MyApp MyApp:file:///home/developer/git-repos/exer_log/lib/main.dart:44:14

When the exception was thrown, this was the stack:
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 251:49      throw_
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/errors.dart 84:3        castError
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/operations.dart 452:10  cast
dart-sdk/lib/_internal/js_dev_runtime/private/ddc_runtime/types.dart 367:9        as
packages/exerlog/main.dart 87:15                                                  <fn>
packages/riverpod/src/common.dart 265:26                                          <fn>
packages/riverpod/src/common.dart 391:17                                          [_map]
packages/riverpod/src/common.dart 263:12                                          AsyncValueX.when
packages/exerlog/main.dart 62:22                                                  build
packages/flutter_riverpod/src/consumer.dart 371:19                                build
packages/flutter/src/widgets/framework.dart 4919:27                               build
packages/flutter_riverpod/src/consumer.dart 431:20                                build
packages/flutter/src/widgets/framework.dart 4806:15                               performRebuild
packages/flutter/src/widgets/framework.dart 4977:11                               performRebuild
packages/flutter/src/widgets/framework.dart 4529:5                                rebuild
packages/flutter/src/widgets/framework.dart 2659:18                               buildScope
packages/flutter/src/widgets/binding.dart 891:9                                   drawFrame
packages/flutter/src/rendering/binding.dart 370:5                                 [_handlePersistentFrameCallback]
packages/flutter/src/scheduler/binding.dart 1146:15                               [_invokeFrameCallback]
packages/flutter/src/scheduler/binding.dart 1083:9                                handleDrawFrame
packages/flutter/src/scheduler/binding.dart 997:5                                 [_handleDrawFrame]
lib/_engine/engine/platform_dispatcher.dart 1090:13                               invoke
lib/_engine/engine/platform_dispatcher.dart 160:5                                 invokeOnDrawFrame
lib/_engine/engine/initialization.dart 194:45                                     <fn>

════════════════════════════════════════════════════════════════════════════════════════════════════
Application finished.

of course this is my first run, and maybe I should install some additional libs (or make additional config) to avoid these errors
I found some article connected with initialization of firebase in app, maybe it will help: https://fluttercorner.com/firebaseoptions-cannot-be-null-when-creating-the-default-app/

Im pretty sure this is just because you ran it in chrome. The app currently isn’t optimised for web, only iOS and android. Can you try again on one of these platforms?

@jorre127 did you install the stable version of flutter?

Ok I finished flutter configuration:
flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[✓] Flutter (Channel stable, 3.0.5, on Ubuntu 20.04.4 LTS 5.14.0-1047-oem, locale en_IN.UTF-8)
[✓] Android toolchain - develop for Android devices (Android SDK version 33.0.0)
[✓] Android Studio (version 2021.2)
[✓] VS Code
[✓] Connected device (1 available)
[✓] HTTP Host Availability

• No issues found!

and after launch selected android emulator, application stared:
flutter run
Using hardware rendering with device Android SDK built for x86. If you notice graphics artifacts, consider enabling software rendering with "--enable-software-rendering".
Launching lib/main.dart on Android SDK built for x86 in debug mode...
Running Gradle task 'assembleDebug'...                              9.0s
✓  Built build/app/outputs/flutter-apk/app-debug.apk.
W/FlutterActivityAndFragmentDelegate( 4420): A splash screen was provided to Flutter, but this is deprecated. See flutter.dev/go/android-splash-migration for migration steps
.
../../snap/flutter/common/flutter/.pub-cache/hosted/pub.dartlang.org/flutter_riverpod-1.0.3/lib/src/framework.dart:275:26: Warning: Operand of null-aware operation '!' has
type 'SchedulerBinding' which excludes null.
 - 'SchedulerBinding' is from 'package:flutter/src/scheduler/binding.dart' ('../../snap/flutter/common/flutter/packages/flutter/lib/src/scheduler/binding.dart').
    if (SchedulerBinding.instance!.schedulerPhase ==
                         ^
Syncing files to device Android SDK built for x86...               370ms

Flutter run key commands.
r Hot reload. 🔥🔥🔥
R Hot restart.
h List all available interactive commands.
d Detach (terminate "flutter run" but leave application running).
c Clear the screen
q Quit (terminate the application on the device).

Running with unsound null safety
For more information see https://dart.dev/null-safety/unsound-null-safety

Maybe it will be good idea to add tip in Development section about iOS and Android compatibility

@mfederowicz we can close this issue right?
CC: @KalleHallden


pydevd Debug TypeError 'NoneType' object is not callable

Issue with running debug in Visual Studio Code. Hard to reproduce exact conditions, but it seems to effect breakpoints placed on else statements far into code. Exact steps to reproduce below. I can provide any other information as needed.
Environment data

VS Code version: 1.29.0
Extension version (available under the Extensions sidebar): 2018.11.0
OS and version: CentOS 7.6.1810
Python version (& distribution if applicable, e.g. Anaconda): 2.7.5
Type of virtual environment used (N/A | venv | virtualenv | conda | ...): N/A
Relevant/affected Python packages and their versions: pydevd - unknown

Expected behaviour
Run Debug and stop at breakpoint
Actual behaviour
Exits with TypeError 'NoneType' object is not callable
Steps to reproduce:

Set breakpoint at else statement far into the script
Run debug

Exact steps:

git clone https://github.com/infphilo/hisat2.git
load hisatgenotype_extract_reads.py into vscode
set breakpoint at line 152
run debug

Logs
Traceback (most recent call last):
File "/home/user/.vscode/extensions/ms-python.python-2018.11.0/pythonFiles/experimental/ptvsd_launcher.py", line 45, in
main(ptvsdArgs)
File "/home/user/.vscode/extensions/ms-python.python-2018.11.0/pythonFiles/experimental/ptvsd/ptvsd/main.py", line 265, in main
wait=args.wait)
File "/home/user/.vscode/extensions/ms-python.python-2018.11.0/pythonFiles/experimental/ptvsd/ptvsd/main.py", line 258, in handle_args
debug_main(addr, name, kind, *extra, **kwargs)
File "/home/user/.vscode/extensions/ms-python.python-2018.11.0/pythonFiles/experimental/ptvsd/ptvsd/_local.py", line 45, in debug_main
run_file(address, name, *extra, **kwargs)
File "/home/user/.vscode/extensions/ms-python.python-2018.11.0/pythonFiles/experimental/ptvsd/ptvsd/_local.py", line 79, in run_file
run(argv, addr, **kwargs)
File "/home/user/.vscode/extensions/ms-python.python-2018.11.0/pythonFiles/experimental/ptvsd/ptvsd/_local.py", line 140, in _run
_pydevd.main()
File "/home/user/.vscode/extensions/ms-python.python-2018.11.0/pythonFiles/experimental/ptvsd/ptvsd/_vendored/pydevd/pydevd.py", line 1936, in main
if setup['cmd-line']:
TypeError: 'NoneType' object is not callable

same
still find no solution

Dup of Microsoft/vscode-python#3520
Upstream issue https://github.com/Microsoft/ptvsd/issues/987

I have the same problem with the latest version: VS Code Mac 1.30.1 on macOS 10.14.1 and Python extension 2018.12.1. This renders debugging Python code basically useless. Please fix!


validator.v9: Add Spanish translations

Enhances # .
Make sure that you've checked the boxes below before you submit PR:

[X] Tests exist or have been written that cover this particular change.

Change Details:

Spanish translations added

@go-playground/admins

Hey @alfonsoprado
Thanks for the PR. I'm going to be putting v9 into maintenance mode soon, I just don't have the time to maintain it alongside v10.
Going to close this PR for now as a result. If it's really needed please push the issue :)


App store@remove repo confirm dialog

Coverage remained the same at 59.659% when pulling 79a7d6aa8350d4215eb540d3c1e6daecf540aa60 on nikage:app-store@remove-repo-confirm-dialog into 095e0b2b29cd10823b06a67af8af97aa255c8ce3 on supergiant:master.


Update README.md

I just added 2 demos

For  element
Link

For  element
Link

Codecov Report

Merging #4246 (54649c3) into master (abaf123) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master    #4246   +/-   ##
=======================================
  Coverage   99.75%   99.75%
=======================================
  Files         180      180
  Lines        3241     3241
=======================================
  Hits         3233     3233
  Misses          8        8

Continue to review full report at Codecov.

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

Sorry, but are not going to link third party websites in our README.


Is there anything like http_server?

Hi, I'm new to SCTP and I found usrsctp a very solid codebase.
http_client.c is where I start from, but I didn't find anything like http_server,
while there are discard_server/client and echo_server/client which can communicate with each other.
I considered to set up an HTTP server over kernel SCTP stack, but as far as I know, Linux kernel stack does not support the UDP encap, so it may not work with the encap-usrsctp.
Is there any way we can fast build a simple testbed to send and receive HTTP messages over usrsctp (especially with UDP-encap)?
Many thanks.

We use thttpd running on the FreeBSD kernel implementation, which also supports UDP encapsulation.

Thanks for your quick response and I will take a look.
BTW I think this information is very helpful and can be posted in README or manual.
Thanks again!


update PAM50_cord

in Charley's DB/results/20160812_PAM50_cord
to incorporate for next release

I'm trying to regularize along z the segmentation done by ITK-Snap:

ok thanks-- looks like there is also a slight undersegmentation compared to propseg

A new version of the L1-L2 cord segmentation is now available, same folder

fixed in v3.0_beta21


Ability to copy one canvas onto another.

Works on #47

Pull Request Test Coverage Report for Build 191

21 of 28 (75.0%)  changed or added relevant lines in 1 file are covered.
1 unchanged line in 1 file lost coverage.
Overall coverage decreased (-0.02%) to 85.97%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

canvas/canvas.go
21
28
75.0%

Files with Coverage Reduction
New Missed Lines
%

canvas/canvas.go
1
82.93%

Totals

Change from base Build 186:
-0.02%

Covered Lines:
2163

Relevant Lines:
2516

💛  - Coveralls


How can you set this control to required?

Setting [required] doesnt seem to work.  I am using this as a component around my site, not just in 1 place and need to be able to set it to required using a varaible.
In ngx-bootstrap, there is a [required] property which can be used.

Please check the "Documentation"  here, there you can see that there is no required input property for this component.
On the same web site, on the "Examples -> Form validation" page, you can see how you can use a required validator in a reactive form.


Feature request: Wrap-around candidates list

There are times when you know that the candidate you want is near the end of the list which you could reach with a "Ctrl-end", but it will be great to have an option to just use the up arrow to go past the prompt and down to the end of the list.

Does M-> work?

Wrapping around would have to handle edge cases like if the prompt is selectable. As a user you might come into situations where you want to select the prompt and then it wraps around which would be confusing/annoying.

Does M-> work?

Yes, that's perfect. Didn't realise that would work as usual. Thanks!

Wrapping around would have to handle edge cases like if the prompt is selectable.

Wrap only if already in the prompt and always wrap when going down at the bottom. It would be nice to reuse the same keys for this.

You can't always select the prompt so sometimes you would need to wrap before it is selected, which would lead to situations where the intention was to select the prompt but then it wraps around. I would prefer not to allow such behavior and instead recommend to jump to the first/last candidate using M-< and M->.

You can't always select the prompt so sometimes you would need to wrap before it is selected, which would lead to situations where the intention was to select the prompt but then it wraps around

That's what my response was addressing. It's not a complicated edge case. If the prompt is selectable, wrap around only after selecting the prompt. This is how ivy handles it, and my preference would be to have it as an option. I don't mind implementing it just in my own config if it's possible though. I'll look into it. Is it easy to get the current candidate #, the total candidates, and whether the prompt is selectable?

It isn't complicated but it seems to be a bit confusing to me UI wise, you can use something like the following:
(defun selectrum-next-candidate (&optional arg)
  "Move selection ARG candidates down, stopping at the end."
  (interactive "p")
  (when selectrum--current-candidate-index
    (let ((selectable-prompt
           (not (and (selectrum--match-strictly-required-p)
                     (cond (minibuffer-completing-file-name
                            (not (selectrum--at-existing-prompt-path-p)))
                           (t
                            (not (string-empty-p selectrum--virtual-input)))))))
          (index (+ selectrum--current-candidate-index (or arg 1)))
          (max (1- (length selectrum--refined-candidates))))
      (setq selectrum--current-candidate-index
            (cond ((< index (if selectable-prompt -1 0))
                   max)
                  ((> index max)
                   (if selectable-prompt -1 0))
                  (t
                   index))))))

Yeah, it's just that as a user, it seems like there is no way to tell whether or not the prompt will be selectable in any given command, so it would be hard to predict whether pressing <up> would lead to wraparound. But, of course, nothing wrong with implementing the command manually.


Move runtime types out of plugins pkg

This moves the runtime types out of the plugin pkg and into a runtime package.
This also removes the Event type that the runtime and shim's use to be the same event type that is defined in the events service.

LGTM

Codecov Report

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

@@           Coverage Diff           @@
##           master    #1112   +/-   ##
=======================================
  Coverage   54.51%   54.51%
=======================================
  Files          16       16
  Lines        1229     1229
=======================================
  Hits          670      670
  Misses        386      386
  Partials      173      173

Continue to review full report at Codecov.

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

LGTM


[Designer] Allow data binding for all properties in preview mode

Related Issue
Fixes #6631
Description
Added an optional parameter to the getValue function that allows the user to get a property from _rawProperties instead of _propertyBag.
We pass GlobalSettings.enableFullJsonRoundTrip to getValue when generating JSON from the card.
Note: Data binding will only work while in preview mode.
Sample Card
{
    "type": "AdaptiveCard",
    "$schema": "http://adaptivecards.io/schemas/adaptive-card.json",
    "version": "1.5",
    "body": [
        {
            "type": "TextBlock",
            "text": "Image test:",
            "wrap": true
        },
        {
            "type": "Image",
            "url": "https://pbs.twimg.com/profile_images/3647943215/d7f12830b3c17a5a9e4afcc370e3a37e_400x400.jpeg",
            "width": "${imageSize}",
            "height": "${imageSize}"
        }
    ]
}

Data JSON
{
    "imageSize": "100px"
}

How Verified
Verified manually on the AdaptiveCards site
Questions
Are there any other cases where we use GlobalSettings.enableFullJsonRoundTrip that I should be aware of?
Microsoft Reviewers: Open in CodeFlow

{
"imageSize": "100px",
"shouldWrap": false
}

@jwoo-msft I tested tried those payloads, and it worked for me. We're you using preview mode?
I'll update the description, but the data binding only works in preview mode.

{
"imageSize": "100px",
"shouldWrap": false
}

no, it doesn't work for me in preview mode. can you share the pr site you used?

{
"imageSize": "100px",
"shouldWrap": false
}

no, it doesn't work for me in preview mode. can you share the pr site you used?

{
"imageSize": "100px",
"shouldWrap": false
}

no, it doesn't work for me in preview mode. can you share the pr site you used?

it works now; could be caching issue on my side.


docker latest master branch exit immediately after starting without -H

If not specified a -H on docker daemon staring, the latest master branch docker daemon will exit immediately it should use default /var/run/docker.sock
root@ubuntu:/home/lei# docker -d -s overlay
Warning: '-d' is deprecated, it will be removed soon. See usage.
WARN[0000] please use 'docker daemon' instead.
INFO[0000] Option DefaultDriver: bridge
INFO[0000] Option DefaultNetwork: bridge
s.servers is [], len is 0
INFO[0000] Firewalld running: false
INFO[0000] Loading containers: start.

INFO[0000] Loading containers: done.
INFO[0000] Daemon has completed initialization
INFO[0000] Docker daemon                                 commit=c45ad0b-dirty execdriver=native-0.2 graphdriver=overlay version=1.9.0-dev

seems it caused by #15753

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

#dibs


Support for HTTP/1 agent reporter for communication with Collector

Requirement - what kind of business use case are you trying to solve?
If some platforms do not support either TCP or HTTP/2, will need a HTTP/1 Agent Reporter.
Problem - what in Jaeger blocks you from solving the requirement?
Currently, communication between Agent and Collector can be achieved only via TChannel or grpc.
Proposal - what do you suggest to solve the problem or improve the existing situation?
Having a reporter.type="http" option which sends spans over HTTP/1.
https://github.com/jaegertracing/jaeger/compare/master...piyush-pruthi:agent-reporter-http-support?expand=1
Most of the work is done in this repository.
If it sounds good, I can raise a separate PR after having discussions.
Thanks
Any open questions to address

Please elaborate why your environment cannot support http/2.

@yurishkuro
We have a PAAS environment in which we use Cloud Foundry and gorouter for routing purposes.
gorouter currently doesn't support HTTP/2 packets.
https://github.com/cloudfoundry/gorouter/issues/195
So, if anyone wants to run jaeger on current CF installations, a HTTP/1 based reporter will help.

There is still an option to use HTTP directly from Jaeger clients to collector. Or using Envoy instead of gorouter. It feels unwise developing new feature in Jaeger to support clearly legacy issue in CF.
Did you already make the change? How big is it? How much does it reuse existing code?

Here are the changes @yurishkuro
https://github.com/jaegertracing/jaeger/compare/master...piyush-pruthi:agent-reporter-http-support?expand=1
This pretty much aligns with existing reporters design and http client from jaeger-client-go.

The problem is that strategically we want to have fewer comms method, not more. We want to converge all comms to grpc and decomm tchannel; this change would take us in the opposite direction, supporting flexibility for transports where we don't want it.

Sure..
Having a single communication channel makes sense.
We will try to explore some other things then for old CF installations.
Thanks

Hi, I like to +1 the request, b/c I‘d like to run the collector as a svc behind an OpenShift route. In V4 there is apparently no support to expose gRPC or http2 endpoints.

The problem is that strategically we want to have fewer comms method, not more. We want to converge all comms to grpc and decomm tchannel; this change would take us in the opposite direction, supporting flexibility for transports where we don't want it.

I understand that intention, however it appears that there is a real need for http communication and it can be seen with practically all official clients supporting http communication. Moreover the server, i.e. the collector supports that, why then not it’s own client?
All language clients needed to be implemented with http separately, imho b/c the agent won’t do it. My argument is, why should some instrumented Java application be more versatile in communication than the product itself?


Missing mbedtls_ecdh_init call in ec_new_by_nid

According to mbedtls documentation, mbedtls_ecdh_init should be called before mbedtls_ecdh_setup. This is not done when used in ec_new_by_nid:
https://github.com/DMTF/libspdm/blob/df3db1ba161f4164d5aca21dd394a940411b156b/os_stub/cryptlib_mbedtls/pk/ec.c#L53-L55
See the relevant mbedtls documentation here:
https://tls.mbed.org/api/ecdh_8h.html#a296f6ce7b1b8b53d048a6c28ff3420cc
"This function should be called after mbedtls_ecdh_init()".

Approved.


Upversion.

What? and Why?

What does this pull request change/fix? Why was it nessessary?

Checklist

CHANGELOG.md updated?

Codecov Report

Merging #60 into master will not change coverage.
The diff coverage is 100%.

@@           Coverage Diff           @@
##           master      #60   +/-   ##
=======================================
  Coverage   79.56%   79.56%
=======================================
  Files           9        9
  Lines         279      279
=======================================
  Hits          222      222
  Misses         57       57

Impacted Files
Coverage Δ

app/__init__.py
100% <100%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


Support for Arduino Micro (ATMEGA32U4)

Hello, would it be possible to add the support for Arduino Micro (ATMEGA32U4)? It would be very helpful.
Thanks in advance. Jenik5

Hi Jenik5,
Thank you for your interest in this project. I just ordered a Micro off Amazon in order to be able to implement (and test) support for it (basically: AVR109 type bootloaders). Given delivery time (and finding some free time for implementation) you can likely expect this to be completed in approximately 2 to 3 weeks.

Hi Jenik5,
Thank you for your interest in this project. I just ordered a Micro off Amazon in order to be able to implement (and test) support for it (basically: AVR109 type bootloaders). Given delivery time (and finding some free time for implementation) you can likely expect this to be completed in approximately 2 to 3 weeks.

Hi Christoph,
it is very kind from you. If you have some donation page, please let me know. I can cover at least your hardware expenses.
Jenik 5

Hi Jenik5,
Support for Arduino Micro (ATMega32U4 + the AVR109 protocol) has now been implemented. Let me know if you encounter any issues.
And no worries about hardware expenses, I can always use an extra Arduino!

Hi Christo.phe,
I tested it and works great.
Thank you
Jenik5


Demo App can load from local csv files.

Fixes #36. Allows for loading from the provided local .csv files or connecting to a locally running copy of the API. The distinction between the two is set in the .env file.
On that note ... I allowed this .env to be committed to the repo, but it doesn't contain secrets. Probably a bit sketchy to do so, but it is only this specific file that is allowed in the .gitignore.
It includes another package in the requirements file, python-dotenv so the virtual environment will need to be rebuilt.
@ben-chaney this fixes the problem you ran into Monday night as well!
Also ... ignore the branch name.

The .csv files are added into the repo. They're just snippets for demonstration of the app.


get_bookmarks() - User Fields not working despite proper Arguments

I am using the tweepy library to retrieve user bookmarks, but I am unable to retrieve the profile picture URL despite passing the correct user_field.
According to the tweepy documentation, the only available expansion for this endpoint is "expansions=owner_id". I am not sure what this means or how to retrieve the profile picture URL.
Can someone help me understand this and provide a solution?
bookmarks = client.get_bookmarks(expansions=["author_id"],
    user_fields=["profile_image_url"])

The response object is as follows:
{
  "data": [
    {
      "id":<PHONE_NUMBER>334483456,
      "text": "The majesty of nature is on full display at Yosemite National Park. Visit now to experience it for yourself: https://natureloversparadise.com"
    },
    {
      "id":<PHONE_NUMBER>569945091,
      "text": "Discover the beauty of the Great Barrier Reef with a snorkeling adventure. Book now at https://greatbarrierreefadventures.com"
    }
  ],
  "includes": {
    "users": [
      {
        "id": 123456789,
        "username": "naturelover",
        "name": "Nature Enthusiast"
      },
      {
        "id": 987654321,
        "username": "oceanadventurer",
        "name": "Ocean Adventurer"
      }
    ]
  }
}

As you can see, the includes section only contains basic information about the authors of the tweets (id , username , and name ), and no information about profile pictures.
Thank you for your help.

I'm unable to reproduce this.
In fact, those two Tweet IDs don't match the text there:
>>> response = client.get_bookmarks(expansions=["author_id"], user_fields=["profile_image_url"])
>>> response
Response(data=[<Tweet id=1620813010569945091 text="Why I spent 0.2 BTC on this Ordinal's-inscribed Bitcoin Rock\n\n(and why you should pay attention to Bitcoin NFTs):\n\n/🪨 https://t.co/uJwsRHXyfr">, <Tweet id=1621356248334483456 text='The real incarnation of crypto is getting cheaper https://t.co/RN5gV2aPLk'>], includes={'users': [<User id=906237309053480961 name=dotta username=dotta>, <User id=929223032467525633 name=Avocado Toast (💙,🧡) username=avocado_toast2>]}, errors=[], meta={'result_count': 2})
>>> response.includes["users"][0].data
{'name': 'dotta', 'id': '906237309053480961', 'username': 'dotta', 'profile_image_url': 'https://pbs.twimg.com/profile_images/1587449365014142979/y0p8yEp5_normal.jpg'}
>>> response.includes["users"][1].data
{'name': 'Avocado Toast (💙,🧡)', 'id': '929223032467525633', 'username': 'avocado_toast2', 'profile_image_url': 'https://pbs.twimg.com/profile_images/1609778151369883648/dgvqvUMB_normal.jpg'}

One of the user IDs doesn't even exist:
>>> client.get_user(id=987654321)
Response(data=None, includes={}, errors=[{'value': '987654321', 'detail': 'Could not find user with id: [987654321].', 'title': 'Not Found Error', 'resource_type': 'user', 'parameter': 'id', 'resource_id': '987654321', 'type': 'https://api.twitter.com/2/problems/resource-not-found'}], meta={})

and the other doesn't match the username or the name:
>>> client.get_user(id=123456789)
Response(data=<User id=123456789 name=Some Test User username=some_test_user>, includes={}, errors=[], meta={})

Neither of the usernames (one of which doesn't even have any Tweets) match the names either:
>>> client.get_user(username="naturelover")
Response(data=<User id=4039 name=Peter username=naturelover>, includes={}, errors=[], meta={})
>>> client.get_user(username="oceanadventurer")
Response(data=<User id=42259256 name=Terri Gentry username=Oceanadventurer>, includes={}, errors=[], meta={})


Publish and retract a v0.9.0

This should have the effect of convincing pkg.go.dev to show the latest
version from the default branch.
What does this PR do?
Alone, nothing.  But when tagged as v0.9.0, it should eventually result in https://pkg.go.dev/github.com/DataDog/datadog-agent displaying docs from a recent commit to the repo
Motivation
https://pkg.go.dev/github.com/DataDog/datadog-agent currently shows docs from over a year ago.

Recently saw that there was no "stable release" per go.dev's expectation of semver tags starting with a v -- and saw this change to the go.mod as the only reference as to why. Is there some explanation as to why "We do not want any tagged version to appear there"?
I appreciate the comment that at the time it showed outdated docs, but with frequent releases from here, if they were tagged according to what go.dev expects, then the docs on there would be relevant to the most recent release
And there would be docs searchable over the history of releases, not just the latest, with no prior history of docs if someone wanted to search the docs of an older version.
So I'm just wondering if there's any deeper reason to "We do not want any tagged version to appear there" than getting them to be fresher, would could be achieved by tagging semver with a v prefix.

I'm not sure why this repo is tagged without a v prefix.  We already do a lot of tagging, so adding duplicate x.y.z and vx.y.z might just be too much?  But, it's a good point -- I can check and see what we could do.

Ah, I have a little context: this repo is not meant for external consumption as Go packages -- it does not export any stable API, nor follow semver in terms of its exported items.  The choice to not include v-prefixed versions was explicitly to recognize that -- to avoid external users of go paths like github.com/DataDog/datadog-agent/v7 and expecting all exported items to obey semver.
That worked OK except that somewhere along the line there was a v0.9.0 tag, which caused go.dev to stop updating, which is what this PR addressed.
All things being equal, I think it makes most sense to have the docs tracking (loosely) the main branch of the repo, as that's the only version available via its Go API (and then only available to other devs in this repo).


Can not use the latest version of sklearn

python3.6/site-packages/sklearn/base.py:315: UserWarning: Trying to unpickle estimator ExtraTreeClassifier from version 0.18.1 when using version 0.18.2. This might lead to breaking code or invalid results. Use at your own risk.
UserWarning)
python3.6/site-packages/dragnet/models.py:51: UserWarning: Unable to unpickle ContentExtractionModel! Your version of scikit-learn (0.18.2) may not be compatible. Setting extractors to None.
UserWarning)

hey @mahaoyang, this issue will be resolved when #52 is merged. We're trying to see that this happens soon!

It looks like sklearn>=0.18 will complain when loading a model about any difference in the installed version of sklearn and the one used to train a model. As far as I can tell, Any version after 0.18 can load the models just fine, despite the warning.
Would an explanation of this behavior in the README be enough to consider this resolved? If so, I'd be happy to add a section there on the warning. Or is there a better way of handling this?

I'm going to assume this was fixed by #52.


spawn limit

Hello!
We have 2500 connections on memcached server now, and i try to implement similar server.
Then i check reactor - looks like it have spawn limit on 2048 connections on my server (limits in config on server arround 500k)
Is this limitation of libuv or any hardcoded constant?
I try like this
import reactor/loop, reactor/async, reactor/threading, os

proc hardWork(): int =
  echo "hard work started"
  sleep(10000)
  echo "hard work finished"
  return 5

proc main() {.async.} =
  for i in 1..2500:
    let f = spawn hardWork()

main().runLoop()

This is most likely limitation of threadpool from stdlib (reactor spawn is a thin wrapper around it).

https://github.com/nim-lang/Nim/blob/devel/lib/pure/concurrency/threadpool.nim#L286
Anyway, you probably don't have 256-core processor, so (unless you are doing sync IO) it doesn't make sense to run more threads concurrently.

You can open issue on main Nim repo if you want threadpool module to support more threads. (unless you want to implement thread pool inside reactor.nim :) )


Change enketo/* overridable modules

enketo/config
enketo/translator
enketo/widgets
enketo/dialog
enketo/file-manager
To prepare for a future when we can simply load ES6 modules without any bundling (including from node_modules folder), we should change the approach for these placeholder modules.
Maybe add static setter functions like:

Form.config = ...., Form.dialog = ...

In app.js, we could set these properties with the placeholder files. In an application that use enketo-core, app.js would be replaced by its own entry point.

Considering we need those things in many modules (e.g. in a widget), using a static property would mean we need to import Form leading to circular references.

Have not been able to come up with a good approach. Will save for later and not include in 5.0.0 release


Definitions of Signal Detection Artifact Function and Signal Processing Artifact Function too broad

Given the definition of Signal Detection Artifact Function, if an artifact is designed to discern between information-bearing patterns and random patterns, or noise, that distract from the information, then all of that artifact's Artifact Functions are Signal Detection Artifact Functions, which needn't be the case. Similar remarks apply in the case of Signal Processing Artifact Function.
The definitions could probably be fixed if they were revised to appeal to realization.

Signal Detection Artifact Function = An Artifact Function that inheres in Artifacts that are designed to discern between information-bearing patterns and random patterns, or noise, that distract from the information.
could be turned into:
Signal Detection Artifact Function = An Artifact Function that is realized in the process of discerning between information-bearing patterns and random patterns, or noise, that distract from the information`
And
Signal Processing Artifact Function = An Artifact Function that inheres in Artifacts that are designed to process or transfer information contained in signals.
Turns into:
Signal Processing Artifact Function = An Artifact Function that is realized in the process of transferring or processing information contained in signals.
@mark-jensen where can I start working on some of these changes? On this issue and ones similar like 246

@cameronmore: Your proposed definitions sound good, though I'd suggest some minor changes that would make them more similar to most other definitions of Artifact Function subclasses:

Signal Detection Artifact Function: An Artifact Function that is realized in processes in which some Artifact is used to discern between information-bearing patterns and random patterns, or noise, that distract from the information.

Signal Processing Artifact Function: An Artifact Function that is realized in processes in which some Artifact is used to process or transfer information contained in signals.

The ancestor class Artifact Function already covers the 'inheres in an artifact that' portion of the definition, so although I like your suggestion, I think it could be conciser, and the other definition (that have the same style) also don't have that 'inheres in artifact that' portion.

@cameronmore: I'm not sure what you mean. The slightly changed versions of your proposed definitions that I suggested don't mention inherence at all.

Sorry, I meant that the 'in which some artifact is used to' portion of the definition isn't necessary, that's more of the definition of the process in which the artifact is employed

@cameronmore: OK. Although I'd note that most of the other definitions of Artifact Function subclasses do contain the "in which some Artifact is used to" portion.

An update has been merged into development in response to this issue. Closing.


Added MacOS files that should not be checked in

Added MacOS files that should not be checked in like .DS_Store for example.
We need to ignore MacOS specific files too.
https://en.wikipedia.org/wiki/.DS_Store
https://en.wikipedia.org/wiki/AppleSingle_and_AppleDouble_formats

You should use the global gitignore for Mac OS files: https://github.com/github/gitignore/blob/master/Global/macOS.gitignore
This guide may help: http://egorsmirnov.me/2015/05/04/global-gitignore-file.html

Thanks for the clarification.

Done 👍


Feature request: warn if a depends directory or setting overrides pack's dependecies unintentionally

pack gets a bit confused if there's a local depends directory.
Would it make sense to issue a warning if there's a depends directory that overrides/changes the dependencies in pack?
I think this ought to be a warning, not an error, and we ought to have the ability to suppress it by a flat and a local pack.toml option, because some projects might want to have more complex dependency structures than we expect, and ignoring warnings is a slipper slope :).

Yes, we should definitely have this. I guess a --no-warn-depends command line option (plus similar option in pack.toml) would be ok to turn this off?
Do you know if Idris even considers the packages installed in PACKAGE_PATH if there is a local depends folder?


How can specified the k8s job‘s namespace and serviceaccount with drone ?

When a pipeline run ,the drone server will create a job in k8s.But the job always create in the default namespace ,and the job will create namespace to execute pipeline steps by the user system:serviceaccount:default:default.

So,I have to give the permission about create namespace to system:serviceaccount:default:default.
For isolation I want create a namespace ,such as "devops" ,then put the drone server and all of the drone job in this namespace.If the drone can do this?
I would be very grateful if you have good opinions.

:wave: hey there, per our issue template, for end-user support please use one of our support channels, such as our chatroom or our mailing list.


Not able to parse DateTimeSpec value "the hour" while reading response from luis in bot app

Ask
May I know why "the hour" text could not be converted in DateTimeSpec type? Do I need to tweak something from my c# model side to allow it?
Package
<PackageReference Include="Microsoft.Bot.Builder.AI.Luis" Version="4.12.0" />
Luis response JSON
{
  "useralias": [
    "abcd"
  ],
  "number": [
    2
  ],
  "datetime": [
    {
      "timex": [
        "(T14, T15,PT1H)"
      ],
      "type": "timerange"
    },
    "the hour"
  ],
  "personName": [
    "Tony"
  ],
  "$instance": {
    "useralias": [
      {
        "type": "useralias",
        "text": "abcd",
        "startIndex": 0,
        "endIndex": 10,
        "modelType": "Entity Extractor",
        "recognitionSources": [
          "model"
        ]
      }
    ],
    "number": [
      {
        "type": "builtin.number",
        "text": "2",
        "startIndex": 31,
        "endIndex": 32,
        "modelType": "Prebuilt Entity Extractor",
        "recognitionSources": [
          "model"
        ]
      }
    ],
    "datetime": [
      {
        "type": "builtin.datetimeV2.timerange",
        "text": "2 to 3pm",
        "startIndex": 31,
        "endIndex": 39,
        "modelType": "Prebuilt Entity Extractor",
        "recognitionSources": [
          "model"
        ]
      },
      {
        "type": "builtin.datetimeV2.duration",
        "text": "the hour",
        "startIndex": 99,
        "endIndex": 107,
        "modelType": "Prebuilt Entity Extractor",
        "recognitionSources": [
          "model"
        ]
      }
    ],
    "personName": [
      {
        "type": "builtin.personName",
        "text": "Tony",
        "startIndex": 59,
        "endIndex": 65,
        "modelType": "Prebuilt Entity Extractor",
        "recognitionSources": [
          "model"
        ]
      }
    ]
  }
}

Error
{"Error converting value \"the hour\" to type 'Microsoft.Bot.Builder.AI.Luis.DateTimeSpec'. Path 'entities.datetime[1]', line 1, position 362."}

   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.EnsureType(JsonReader reader, Object value, CultureInfo culture, JsonContract contract, Type targetType)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateValueInternal(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateList(IList list, JsonReader reader, JsonArrayContract contract, JsonProperty containerProperty, String id)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateList(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, Object existingValue, String id)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateValueInternal(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.SetPropertyValue(JsonProperty property, JsonConverter propertyConverter, JsonContainerContract containerContract, JsonProperty containerProperty, JsonReader reader, Object target)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateObject(Object newObject, JsonReader reader, JsonObjectContract contract, JsonProperty member, String id)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateValueInternal(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.SetPropertyValue(JsonProperty property, JsonConverter propertyConverter, JsonContainerContract containerContract, JsonProperty containerProperty, JsonReader reader, Object target)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.PopulateObject(Object newObject, JsonReader reader, JsonObjectContract contract, JsonProperty member, String id)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateObject(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.CreateValueInternal(JsonReader reader, Type objectType, JsonContract contract, JsonProperty member, JsonContainerContract containerContract, JsonProperty containerMember, Object existingValue)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.Deserialize(JsonReader reader, Type objectType, Boolean checkAdditionalContent)
   at Newtonsoft.Json.JsonSerializer.DeserializeInternal(JsonReader reader, Type objectType)
   at Newtonsoft.Json.JsonSerializer.Deserialize(JsonReader reader, Type objectType)
   at Newtonsoft.Json.JsonConvert.DeserializeObject(String value, Type type, JsonSerializerSettings settings)
   at Newtonsoft.Json.JsonConvert.DeserializeObject[T](String value, JsonSerializerSettings settings)
   at Newtonsoft.Json.JsonConvert.DeserializeObject[T](String value)
   at Luis.GeneralLuis.Convert(Object result)
   at Microsoft.Bot.Builder.AI.Luis.LuisRecognizer.<RecognizeAsync>d__26`1.MoveNext()
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at OOFAssistant.Controllers.NotifyController.<PullAliasRecord>d__17.MoveNext()
   at System.Runtime.ExceptionServices.ExceptionDispatchInfo.Throw()
   at System.Runtime.CompilerServices.TaskAwaiter.ThrowForNonSuccess(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter.HandleNonSuccessAndDebuggerNotification(Task task)
   at System.Runtime.CompilerServices.TaskAwaiter`1.GetResult()
   at OOFAssistant.Controllers.NotifyController.<BotCallback>d__15.MoveNext()

Inner exception
Could not cast or convert from System.String to Microsoft.Bot.Builder.AI.Luis.DateTimeSpec.

   at Newtonsoft.Json.Utilities.ConvertUtils.EnsureTypeAssignable(Object value, Type initialType, Type targetType)
   at Newtonsoft.Json.Utilities.ConvertUtils.ConvertOrCast(Object initialValue, CultureInfo culture, Type targetType)
   at Newtonsoft.Json.Serialization.JsonSerializerInternalReader.EnsureType(JsonReader reader, Object value, CultureInfo culture, JsonContract contract, Type targetType)

Convert logic
        public void Convert(dynamic result)
        {
            var app = JsonConvert.DeserializeObject<GeneralLuis>(
                JsonConvert.SerializeObject(
                    result,
                    new JsonSerializerSettings { NullValueHandling = NullValueHandling.Ignore, Error = OnError }
                )
            );

Hi @PurnaChandraPanda I'm looking into this and will share my findings.

@dmvtech any updates on this?

Hi @PurnaChandraPanda I'm not quite getting the same results as you. Can you tell me what the full utterance is that you are using?

Hi @dmvtech
Here is the utterance
jode - away 2 to 3pm >>> John is in an offline meeting between the hour. He is not available for issues in that time. Just FYI!

Thanks!

@PurnaChandraPanda
So, there are two aspects to this.
First of all, the utterance 'between the hour' doesn't really fit in a proper English vernacular. Something like 'between the hours of 2 to 3', 'an hour' or 'that hour' are.
Testing the following utterances using the Timex sample, I debugged to find the recognized results.
The following utterance (yours):
"jode - away 2 to 3pm >>> John is in an offline meeting between the hour.";
Gives two results. One for range, one for duration.
{Microsoft.Recognizers.Text.ModelResult}
    End: 19
    Resolution: Count = 1
    Start: 12
    Text: "2 to 3pm"
    TypeName: "datetimeV2.timerange"

{Microsoft.Recognizers.Text.ModelResult}
    End: 70
    Resolution: null
    Start: 63
    Text: "the hour"
    TypeName: "datetimeV2.duration"

Here we see that the duration has no resolution. Meaning no values that are retrieved/extracted from that. When running in the Timex example, this results in a null reference exception. I believe this is what is contributing to the LUIS/Serialization exception.
The following utterance (mine):
"jode - away 2 to 3pm >>> John is in an offline meeting for that hour.";
Only gives a single recognizition (the range):
{Microsoft.Recognizers.Text.ModelResult}
    End: 19
    Resolution: Count = 1
    Start: 12
    Text: "2 to 3pm"
    TypeName: "datetimeV2.timerange"

And finally, this utterance:
"jode - away 2 to 3pm >>> John is in an offline meeting for an hour.";
Gives two valid recognized, both duration and range. With the duration having resolution/values:
{Microsoft.Recognizers.Text.ModelResult}
    End: 23
    Resolution: Count = 1
    Start: 17
    Text: "an hour"
    TypeName: "datetimeV2.duration"
{Microsoft.Recognizers.Text.ModelResult}
    End: 37
    Resolution: Count = 1
    Start: 25
    Text: "from 2 to 3pm"
    TypeName: "datetimeV2.timerange"

So, I believe that this should/could be handled more gracefully in code. But be aware that that sample utterance you gave is not correct English and should be avoided if possible.
I'll continue on this and let you know as I have more info.

@dmvtech : I agree to your point. But, my users can file in such message. So, I would have to find a way to prepare for it.
On C# end, I have the luis option set to read as instance data.
                        luisOptions = new LuisRecognizerOptionsV3(luisApp)
                        {
                            TelemetryClient = telemetryClient,
                            LogPersonalInformation = true,
                            // added includeInstanceData property in prediction option
                            // to read the raw identified text as datetimev2 entity
                            PredictionOptions = new LuisV3.LuisPredictionOptions {
                                IncludeInstanceData = true
                            }
                        };

Would things differ if reading datetimev2 results as instance data on client ent??

@dmvtech is this issue with the SDK code or due to an error on the LUIS API side? That will help us determine where we need to route this issue. Thanks!

@peterinnesmsft I kinda see it as both.
When using the 'bad' phrase, LUIS/Timex does recognize two entities. One for duration and one for time range in the instance:
        "datetimeV2": [
          {
            "type": "builtin.datetimeV2.timerange",
            "text": "2 to 3pm",
            "startIndex": 12,
            "length": 8,
            "modelTypeId": 2,
            "modelType": "Prebuilt Entity Extractor",
            "recognitionSources": [
              "model"
            ]
          },
          {
            "type": "builtin.datetimeV2.duration",
            "text": "the hour",
            "startIndex": 63,
            "length": 8,
            "modelTypeId": 2,
            "modelType": "Prebuilt Entity Extractor",
            "recognitionSources": [
              "model"
            ]

But for entities, the duration is just the string the hour with no resolution:
      "datetimeV2": [
        {
          "type": "timerange",
          "values": [
            {
              "timex": "(T14,T15,PT1H)",
              "resolution": [
                {
                  "start": "14:00:00",
                  "end": "15:00:00"
                }
              ]
            }
          ]
        },
        "the hour"
      ],

But when using a more valid English phrase an hour, then we get:
      "datetimeV2": [
        {
          "type": "timerange",
          "values": [
            {
              "timex": "(T14,T15,PT1H)",
              "resolution": [
                {
                  "start": "14:00:00",
                  "end": "15:00:00"
                }
              ]
            }
          ]
        },
        {
          "type": "duration",
          "values": [
            {
              "timex": "PT1H",
              "resolution": [
                {
                  "value": "3600"
                }
              ]
            }
          ]
        }
      ],

So, I do see this as being a bug in LUIS (if it recognizes as a duration, it should be giving resolution as such. with timex and resolution value). I'll look into filing that. As i've tested with Timex classes, i'm not even sure if it's entirely a LUIS issue, as it pretty much starts with the datetime recognizers.
But I'm wonder if the DateTimeSpec class shouldn't' be changed to gracefully handle this (or another similar) scenario.

Still investigating on the LUIS side of things. Will update when have more details.

Fix is still in works for the recognizer. After that LUIS fixes will be implemented. I'll update as I hear more information.

Still waiting on fixes for Recognizers and LUIS.

Related: https://github.com/microsoft/botbuilder-dotnet/issues/5620

Note that due to the recognizers dropping .Net core version 2.0 compatibility, we cannot take the recognizers fix until next cycle when the SDK does the same. (.NEt Core 2.0 is no supported past that date). Pushing this to R15 so that once the dependencies are updated we can merge.

This issue is solved in the new version of Recognizers.
We are updating the .NET SDK to use these packages in PR #5992.


The newly created tab with no buffer have a current dir as a window name in tabline

Hi!
The newly created tab with no buffer (:tabnew) have a current dir as a window name in tabline:

I'm pretty sure it should be 'No name' instead.
(macOS, iTerm2, Neovim nightly)

Oh, that is a miss. I will fix it

Fix in latest commit.


mobileSize

When i use the v-lazy-image in size of mobile with android 7 or less than one
v-lazy-image does not show image in my mobile
please help me
thank you

It'd need to have a repro issue. Feel free to reopen with it


[BUG] Difficulty setting OpenAI API key environment variable on Windows

Required prerequisites

[X] I have read the documentation https://camel-ai.github.io/camel/camel.html.
[X] I have searched the Issue Tracker and Discussions that this hasn't already been reported. (+1 or comment there if it has.)
[X] Consider asking first in a Discussion.

What version of camel are you using?
0.1.0
System information
OS: Windows
Problem description
Users attempting to use the camel project on Windows have reported issues with setting the OpenAI API key as an environment variable. The current instructions in the README seem to be more suited for Unix-like environments (like Linux or macOS), causing confusion among Windows users.
Reproducible example code

Clone and install the camel project as per the README.
Attempt to set the OpenAI API key as an environment variable using the export command in Windows PowerShell or Command Prompt.
The export command is not recognized, and the environment variable is not set.
As a result, running scripts that require the OpenAI API key (like examples/ai_society/role_playing.py) results in errors.

Traceback
The `export` command is not recognized in Windows PowerShell or Command Prompt. Users have to use a different command (`$env:OPENAI_API_KEY = "your-api-key"`) to set the environment variable. This is not currently mentioned in the README.

Expected behavior
Windows users should be able to set the OpenAI API key as an environment variable in their terminal, allowing scripts to access the key as needed.
Additional context
Update the README and any other relevant documentation to include specific instructions for setting environment variables on Windows. This should cover both Command Prompt and PowerShell syntax.

Thanks for reporting the issue! Would you like to help with updating the readme with the instructions for setting up the OpenAI key for different operating systems?

Yeah, for sure.

In Windows PowerShell the following works for me:
$Env:OPENAI_API_KEY = 'sk-XXXXXXXXXXXXXXXXXX'; python examples/ai_society/role_playing.py

In cmd prompt, you should do SET OPENAI_API_KEY=sk-xxxxx


Unable to pan in Chrome

Describe the bug
I am unable to pan or click in latest Openlayers on Windows XP (Chrome version 49.0.2623.112).  Please note that some of my clients still uses XP with this version of Chrome.  This used to work 100% on older versions of Openlayers.
To Reproduce
Load openlayers in older version of Chrome and try to pan / click.
Expected behavior
Map needs to pan and accept clicks.

Maybe related to  #10712: please make sure you have ol/ol.css from v6.2.1 included in your build or html page.

Same versions of js and css are in use, but still not working.

Maybe it is a cache issue.  I will investigate further.

No, definitely not a cache issue.  On XP it works on Firefox, but not in Chrome.

That’s a pre pointer events version of Chrome. Maybe elm-pep, the polyfill we use now, does not work with that old Chrome Version. You may need to load PEP before ol.js.

Thanks, that did the trick.


[WIP] We always want London date comparisons, not UTC – make this explicit

The remaining uses of isSameDay() were all for events or opening times, which are based in London. This means we can remove the UTC handling, and be clear that whenever we mean "isSameDay", we mean "in London".
Closes https://github.com/wellcomecollection/wellcomecollection.org/issues/7831
Who is this for?
Devs.
What is it doing for them?
Making it clear how our date comparison logic works, and more importantly how it's meant to work.

Closing – I’ll revisit this later.


Letter mappings: missing œ

Seems I was drunk, it is there in letter mapping, unlike in its sibling projects.


catch template errors and return an informative message in the response body

Does the 'catch template errors' in the title refer to a request not being a templateable_request? https://github.com/binkhq/api-reflector/blob/07f89b387cf3700eabf7b7f06ccb21d36d8c76da/api_reflector/views.py#L108

this issue could certainly use some more detail! :)
it's mostly referring to this line here:
https://github.com/binkhq/api-reflector/blob/07f89b387cf3700eabf7b7f06ccb21d36d8c76da/api_reflector/rules_engine.py#L101
when we call .render(...) on the rule template, and here:
https://github.com/binkhq/api-reflector/blob/07f89b387cf3700eabf7b7f06ccb21d36d8c76da/api_reflector/views.py#L113-L118
when we call .render(...) on the response template.
these template renders can fail, and at the moment if that happens you just get back a generic error. it would be nice to catch those errors and return some useful information about what went wrong and where.

sounds good, thank you!
in the end, any error handling is better than none. returning detailed information about what exactly went wrong would be great, but even just returning a generic 400 error saying "error rendering template" is much better than it crashing as it does right now!


Store the formation changes of a team in Team object

I would like to create a formation_changes attribute in our Team class. I would use it to make it easier to calculate how much a team has played in a given formation in downstream usage of the EventsDataset.
My proposal would be to make a similar change as we are planning to do for player positions, where we are moving from having a (starting) position attribute to a position_changes attribute, with a starting_position and position method to easily retrieve the position at any given moment. The details of these changes are discussed in: https://github.com/PySport/kloppy/issues/277.
So the changes to our Team class, would look something like this:
@dataclass(frozen=True)
class FormationChange:
    timestamp: timedelta
    period_id: int
    formation: Formation

@dataclass
class Team:
    team_id: str
    name: str
    ground: Ground
    formation_changes: List[FormationChange] = field(default_factory=list)
    players: List[Player] = field(default_factory=list)

    @property
    def starting_formation(self):
        if self.formation_changes:
            return self.formation_changes[0].formation

    def formation(self, period_id: int, timestamp: timedelta):
        if self.formation_changes:
            formation = self.formation_changes[0].formation
            for formation_change in self.formation_changes:
                # Check if the current change is before or exactly at the given period and timestamp
                if period_id > formation_change.period_id or (
                    period_id == formation_change.period_id
                    and timestamp >= formation_change.timestamp
                ):
                    formation = formation_change.position
                else:
                    # Since the list is assumed to be in chronological order, break on finding the first future change
                    break

            return formation

However, I am bit struggling with how to think about the overlap of functionality between the add_state method with 'formation' argument and the new formation method.  They seem to be both serving a similar functionality, determining the formation of team at a given timepoint in the game. Maybe, their purposes are slightly different and it is deseriable to have the both of them in kloppy?
@probberechts @JanVanHaaren @koenvo thoughts?

Fixed in https://github.com/PySport/kloppy/pull/332


outline linux terminal client

What would you like the app to enable you to do?
I want to use outline from docker or on linux server for CI/CD purpose. my git server can be accessed using vpn only.
Describe alternatives you've considered
Nothing at the moment. I need to explore

The quick hint (after 5 minutes looking at the code):
https://github.com/Jigsaw-Code/outline-client/blob/master/src/electron/sslibev_badvpn_tunnel.ts
// Establishes a full-system VPN with the help of Outline's routing daemon and child processes
// ss-local and badvpn-tun2socks. ss-local listens on a local SOCKS server and forwards TCP and UDP
// traffic through Shadowsocks. badvpn-tun2socks processes traffic from a TAP device and relays to
// a SOCKS proxy. The routing service modifies the routing table so that the TAP device receives all
// device traffic.
//
// |TAP| <-> |badvpn-tun2socks| <-> |ss-local| <-> |Shadowsocks proxy|
ss-local -s x.x.x.x -p 65336 -k mypassword -m chacha20-ietf-poly1035 -l 1081 -u
    // ./badvpn-tun2socks.exe \
    //   --tundev "tap0901:outline-tap0:<IP_ADDRESS>:<IP_ADDRESS>:<IP_ADDRESS>" \
    //   --netif-ipaddr <IP_ADDRESS> --netif-netmask <IP_ADDRESS> \
    //   --socks-server-addr <IP_ADDRESS>:1081 \
    //   --socks5-udp --udp-relay-addr <IP_ADDRESS>:1081 \
    //   --transparent-dns

@blshkv can you tell how to do it from linux server

Ok, I figured it out. Here is how.
Get your key:

ss://<strBase64@vpnServer:vpnPort/?outline=1
decode strBase64 into: cipher:accessKey, get vpnServer ip address
start

sudo OutlineProxyController --socket-filename=/var/run/outline_controller

start

ss-local -s vpnServerIP -p vpnPort -k accessKey -m cipher -l 1081 -u

start

badvpn-tun2socks --tundev outline-tun0 --netif-ipaddr <IP_ADDRESS> --netif-netmask <IP_ADDRESS> --socks-server-addr <IP_ADDRESS>:1081 --transparent-dns --socks5-udp --udp-relay-addr <IP_ADDRESS>:1081 --loglevel error

Fix routing by running

echo '{"action":"configureRouting","parameters":{"proxyIp":"<IP_ADDRESS>","isAutoConnect":false}}' | nc -U /var/run/outline_controller

to stop, run:
echo '{"action":"resetRouting","parameters":{}}' | nc -U /var/run/outline_controller

and kill all badvpn,ss-local and OutlineProxyController
P.S. OutlineController is not really required, you can fix routing table and /etc/resolv.conf manually

Hi, is this still legit?
I need outline terminal client for a computer located inside Iran.
I do not have access to it's gui.

We have a prototype Linux command-line client at a branch of our SDK: https://github.com/Jigsaw-Code/outline-internal-sdk/tree/junyi-outline-cli/x/outline-cli
/cc @jyyi1

@jyyi1 unable to find https://github.com/Jigsaw-Code/outline-internal-sdk/blob/junyi-outline-cli/x/outline-cli/main.go#LL15C41-L15C41
"github.com/Jigsaw-Code/outline-internal-sdk/tun2socks/lwip"

could you help please?

Hi @blshkv , the CLI PR is not ready yet (so it's still in draft), but I'm actively working on that. The package (and all the APIs) have been totally redesigned, and the new package is "github.com/Jigsaw-Code/outline-internal-sdk/network/lwip2transport". Please feel free to try the API there.

./outline-vpn-cli-poc
OutlineVPN CLI (experimental-01271815)
panic: runtime error: index out of range [3] with length 2

any ideas?

./outline-vpn-cli-poc
OutlineVPN CLI (experimental-01271815)
panic: runtime error: index out of range [3] with length 2

any ideas?

sudo ./outline-vpn-cli-poc "<shadowsocks-server-ip-address>" <shadowsocks-server-port-number> "<shadowsocks-server-password>"

@jyyi1 when I run outline-vpn-cli-poc, it kills local network and internet

Hi @dmatora , the comment above is out-dated. Please refer to the latest outline-cli document: https://github.com/Jigsaw-Code/outline-sdk/tree/main/x/examples/outline-cli

@jyyi1
go run github.com/Jigsaw-Code/outline-sdk/x/examples/outline-cli@latest -transport "ss://xxx@<IP_ADDRESS>:16654/?outline=1"
OutlineVPN CLI (experimental)
[INFO] 2023/10/29 08:19:51 updated global IPv6 support: false
[INFO] 2023/10/29 08:19:51 remote server supports UDP, we will delegate all UDP packets to it
[INFO] 2023/10/29 08:19:51 OutlineDevice -> tun stopped: 0 <nil>
[INFO] 2023/10/29 08:19:51 tun -> OutlineDevice stopped: 48 network device already closed
[INFO] 2023/10/29 08:19:51 updated global IPv6 support: true
[ERROR] 2023/10/29 08:19:51 failed to configure system DNS: failed to backup DNS config file '/etc/resolv.conf.head' to '/etc/resolv.head.outlinecli.backup': rename /etc/resolv.conf.head /etc/resolv.head.outlinecli.backup: no such file or directory

go build -o outline-cli  -ldflags="-extldflags=-static" ./outline-cli
# github.com/Jigsaw-Code/outline-sdk/x/examples/outline-cli
/usr/bin/ld: /tmp/go-link-1001848889/000004.o: in function `_cgo_9c8efe9babca_C2func_getaddrinfo':
/tmp/go-build/cgo-gcc-prolog:58: warning: Using 'getaddrinfo' in statically linked applications requires at runtime the shared libraries from the glibc version used for linking

Thanks @dmatora ,
The compilation warning and the local network routing are both expected. You can ignore the compilation warning.
The local network routing is also expected because we are adding "from all not to <outline-server-ip> via table <outline-tun-device-table>". This means that all traffic except traffic to the Outline server will be routed through the Outline's routing table (which is 233). As a result, the "192.168.1.x" subnet will be routed to the Outline server, and the server will not be able to access its own local network IP address. This behavior is the same for all of our products, including Outline Client and Intra.
To resolve this issue, we would need to implement split-tunneling. Split-tunneling would allow us to add more exception entries to the routing rules so that local network traffic would be routed to the default routing table instead of the Outline routing table. However, implementing split-tunneling would require a lot of work, and we have very limited bandwidth to implement that at this time.
As a workaround, you can use the command ip rule add to add exceptions with higher priority to route all local network traffic to the default routing table (default) instead of the Outline routing table (233).

Local network is going down only when using outline cli. When using GUI client - this is not an issue. I've temporarily "solved" it by increasing VM storage/memory, installing XFCE and switching to GUI client. It is huge resource waste and an overkill but it works

Can we show a bit more love for outline-cli? I didn't even know it existed until I stumbled upon it entirely by accident in a completely unrelated issue (specifically, #1794). It isn't advertised in either this repository or the outline-sdk one, where it's actually located. At the very least, please consider including it in the README to enhance its discoverability, because it's highly unlikely that anyone will search for it in the examples directory of the SDK repo's "experimental" section. I certainly didn't.
Despite minor, easily fixable annoyances, outline-cli offers a superior user experience compared to the GUI client. It doesn't deserve to be buried the way it is now.
As for the annoyances, I could only discover two:

The DNS resolver is hardcoded. This is also true for the GUI client, but given we compile outline-cli from the source anyways, I would say here the situation is somewhat better - we can alter the hardcoded string before the actual compilation. In my setup, I've implemented a simple 8-line patch enabling me to specify my preferred DNS through a custom &dns=<ip> URL parameter.
LAN becomes inaccessible as requests to it are also redirected to the VPN server. The solution to this issue is pretty straightforward and has been described pretty well by @jyyi1 (one small addendum, for the most part, requests should use the main routing table, not the default one).

Even these present issues are manageable and, once again, easily fixable. And after dealing with those, we can tailor outline-cli and integrate it into our systems as we see fit. For instance, my setup, which includes like 2 shell scripts on top of the CLI itself, offers all the same features as the GUI (i.e., access key management and automatic reconnection to the server when your connection goes down) and even adds a few pleasant bonuses like global hotkeys (for example, I can toggle the VPN state via Super + v, and so on). For anyone interested, my setup can be found here: Kir-Antipov/outline-cli.


'systemctl list-unit-files' lists 99% of services as "bad" on aws nixos 16 fresh instance

Issue description
Just starting up any of the nixos v16 AMIs on a fresh EC2 machine, I can't get my service to run from the configuration file. So I ran 'systemctl list-unit-files' and nearly every service's state is "bad", and a few are "masked".
I can't get any systems to start either.
I'm not sure if this is just an EC2 thing or what.
Steps to reproduce
AWS EC2 -> Launch Instance -> Select a nixos v 16 instance from the Community Instances
Start it up.
SSH in and try to get any service running.
Technical details

System: 16.03.509.002d151 (Emu) - Occurs on any of the v 16 amis
Nix version: nix-env (Nix) 1.11.2
Nixpkgs version: "16.03.509.002d151"

Seems this wasn't why my service wasn't running. Also from others I've heard this is default behavior. Closing.


Multiple else blocks

In strict mode, Python Liquid raise a LiquidSyntaxError if an {% if %} block contains multiple else tags. Shopify/Liquid silently ignores all but the first else tag and its block.
{% if false %}1{% else %}2{% else %}3{% endif %}

Expected output
2

Extraneous elsif blocks are silently ignored by Shopify/Liquid too, even in strict mode.
{% if false %}1{% else %}2{% else %}3{% elsif true %}4{% endif %}

Expected output
2


Move from Travis to GitHub actions

This repository is still using Travis but everything else has moved to using GitHub actions.

@daniellimws - Want to take a look?

Sure


chromedriver,chromium: 125.0.6422.76 -> 125.0.6422.112

Description of changes
https://chromereleases.googleblog.com/2024/05/stable-channel-update-for-desktop_23.html
This update includes 1 security fix. Google is aware that an exploit for
CVE-2024-5274 exists in the wild.
CVEs:
CVE-2024-5274
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 314144 run on x86_64-linux 1 22 packages built:  aws-azure-login chromedriver chromium chromium.sandbox fast-cli grafana-kiosk mermaid-cli mermaid-filter percollate playwright-test puppeteer-cli python311Packages.pytest-playwright python311Packages.pytest-playwright.dist python312Packages.pytest-playwright python312Packages.pytest-playwright.dist selendroid selenium-server-standalone sharedown single-file-cli sitespeed-io vhs wayback


[Snyk] Security upgrade @aws-sdk/client-s3 from 3.592.0 to 3.606.0

Snyk has created this PR to fix 1 vulnerabilities in the yarn dependencies of this project.
Snyk changed the following file(s):

services/app-api/package.json
services/app-api/yarn.lock

Note for zero-installs users
If you are using the Yarn feature zero-installs that was introduced in Yarn V2, note that this PR does not update the .yarn/cache/ directory meaning this code cannot be pulled and immediately developed on as one would expect for a zero-install project - you will need to run yarn to update the contents of the ./yarn/cache directory.
If you are not using zero-install you can ignore this as your flow should likely be unchanged.
Vulnerabilities that will be fixed with an upgrade:

Issue
Score

Regular Expression Denial of Service (ReDoS) SNYK-JS-FASTXMLPARSER-7573289
  50

[!IMPORTANT]

Check the changes in this PR to ensure they won't cause issues with your project.
Max score is 1000. Note that the real score may have changed since the PR was raised.
This PR was automatically created by Snyk using the credentials of a real user.

Note: You are seeing this because you or someone else with access to this repository has authorized Snyk to open fix PRs.
For more information:
🧐 View latest project report
📜 Customise PR templates
🛠 Adjust project settings
📚 Read about Snyk's upgrade logic

Learn how to fix vulnerabilities with free interactive lessons:
🦉 Regular Expression Denial of Service (ReDoS)

This is addressed in #715


MMI borg roletime tracking

About the PR
Brains put into an MMI now count as Borg for purposes of playtime tracking.
Note: due to an unrelated issue, MMI brains (unlike positronic brains) will not track borg playtime while outside of a borg.
Why / Balance
Not only was MMI borg playtime not tracked, it actually counted as the mob's original job
Fixes #31760
Technical details
JobComponent.Prototype is changed to Borg upon being inserted into an MMI. The old job prototype is saved on MMILinkedComponent, and restored if the brain is taken out of the MMI. This doesn't currently matter, since the brain will be dead and not tracking playtime, but it will be needed by potential future features like brain cloning or transplantation
Issue: Brains still count as "dead" while in an MMI so they only start tracking playtime after inserted into a borg. this is inconsistent with Positronic Brains, which start tracking as soon as they are possessed, regardless of body. I'm not fixing that here yet, because I'm not actually sure which of the two behaviors is "correct", and it can be handled on its own
Requirements

[X] I have read and I am following the Pull Request Guidelines. I understand that not doing so may get my pr closed at maintainer’s discretion
[X] I have added screenshots/videos to this PR showcasing its changes ingame, or this PR does not require an ingame showcase

Changelog
:cl: Errant

fix: Organic brains turned into a borg now count borg playtime.

Note: due to an unrelated issue, MMI brains (unlike positronic brains) will not track borg playtime while outside of a borg.

i think it should be intended

Positronic Brains [...] start tracking as soon as they are possessed


Disable Biome LSP if biome.json or @biomejs/biome package is not found.

Currently, if a project doesn't have a biome.json or does have a @biomejs/biome package installed, the extension will download a separate biome binary instead. However, it's not friendly for projects that are yet to migrate to the biome.
It would be good if the biome LSP enable when either biome.json or @biomejs/biome is found (or maybe both).

The only workaround I know right now is to uninstall the biome extension every time I work on a non-biome project.

This is a good question,
I think the vscode extension requires biome to be installed in the project. Though I find it also nice to be able to edit any TS file, outside of a project, and have biome formatting etc. with default configuration.
I think a good solution would be to have the option in zed settings to enable or disable biome. Then you can choose to enable the extension in only the projects that it is used in (via .zed/settings.json), otherwise disable it globally.
But an api for settings like that doesn't exist yet...

I think a good solution would be to have the option in zed settings to enable or disable biome.

I'm still pretty new in Rust, they seem to have exposed the LSP settings to the extension author, but I'm not so sure.
And also, correct me if I'm wrong, after digging into the extension code, it seems like it will always use the latest biome binary regardless of which biome version I'm using in my project?

I'm still pretty new in Rust, they seem to have exposed the LSP settings to the extension author, but I'm not so sure.

I was talking about the zed user settings, not LSP settings.

And also, correct me if I'm wrong, after digging into the extension code, it seems like it will always use the latest biome binary regardless of which biome version I'm using in my project?

It will use the binary from your node_modules if it exists. Otherwise downloads the latest version.

This is a question that has been asked for a long time. I think, for now, it's acceptable to disable the extension if the binary isn't installed.
Although, in the future, I think we will plan to ship the binary again, because we want to provide more features that are more appropriate to a LSP (refactors, rename, etc.)

In my case, the language server doesn't use the biome.json configuration in my monorepo, instead formatting with the default.

I was talking about the zed user settings, not LSP settings.

Yeah, I understand that, but the user could also configure their LSP setting too. https://zed.dev/docs/configuring-zed#lsp

It will use the binary from your node_modules if it exists. Otherwise, download the latest version.

That's what I thought when I saw the logic, but it doesn't seem to be the case for me after I checked the Zed log, it always downloads the latest binary regardless.

In my case, the language server doesn't use the biome.json configuration in my monorepo, instead formatting with the default.

From what I know so far, biome doesn't support monorepo at the moment, it only takes the biome.json at the root. But you could override it for subrepo, https://biomejs.dev/guides/big-projects/#monorepos

From what I know so far, biome doesn't support monorepo at the moment, it only takes the biome.json at the root. But you could override it for subrepo, https://biomejs.dev/guides/big-projects/#monorepos

I have only defined one biome.json at the root of my repository, which works on both VSCode and when running biome format/check in the terminal. I am currently sticking with this configuration.
      "formatter": {
        "external": {
          "command": "bunx",
          "arguments": [
            "@biomejs/biome",
            "check",
            "--apply",
            "--stdin-file-path={buffer_path}"
          ]
        }
      }
    },

Maybe this should be a separate issue btw

I was talking about the zed user settings, not LSP settings.

Yeah, I understand that, but the user could also configure their LSP setting too. https://zed.dev/docs/configuring-zed#lsp

Ah sorry! I actually didn't know that ^^ I need to try that, that would be great.

That's what I thought when I saw the logic, but it doesn't seem to be the case for me after I checked the Zed log, it always downloads the latest binary regardless.

Hmm I guess the extension doesn't find the binary for some reason...

Hmm I guess the extension doesn't find the binary for some reason...

I think it only checks the existence of the binary relatively somewhere in the Zed extension directory. I was wondering if we could utilize the worktree, similar (but not the same) to how the zig extension did. https://github.com/zed-industries/zed/blob/2c78cf349b94d69a4b027400d9d388992a4e4517/extensions/zig/src/zig.rs#L21

That's a good approach. Any PR is very welcome!

So I was able to get the extension to use the project binary, then only realized that the LSP will live across the workspaces.
For instance, it will always use the LSP binary from the first opened workspace, subsequence opened workspaces will still use the LSP binary from the first one, unless you restart the Zed manually. I'm not so sure if this is the current limitation of Zed extensions, or if I did something wrong as I am pretty new to Rust.
Here's the repo I hacked things together, https://github.com/chungweileong94/biome-zed/tree/local-biome-server

So I was able to get the extension to use the project binary, then only realized that the LSP will live across the workspaces.
For instance, it will always use the LSP binary from the first opened workspace, subsequence opened workspaces will still use the LSP binary from the first one, unless you restart the Zed manually. I'm not so sure if this is the current limitation of Zed extensions, or if I did something wrong as I am pretty new to Rust.
Here's the repo I hacked things together, https://github.com/chungweileong94/biome-zed/tree/local-biome-server

Seems odd. Try if worktree.root_path() returns the correct path in langague_server_command(). Maybe it's still using cached_server_path. Extensions may be inited per session not per window.

Seems odd. Try if worktree.root_path() returns the correct path in langague_server_command(). Maybe it's still using cached_server_path. Extensions may be inited per session not per window.

I actually did try to disable the cached_server_path part, and it's the same, likely the langague_server_command() doesn't even get called twice.
Anyway, mind if I ask how can I possibly "log" anything? As I wasn't able to figure out how to log anything in Zed.

I don't know about zed, but the LSP does emit logs once it is launched: https://biomejs.dev/guides/integrate-in-editor/#daemon-logs

Seems odd. Try if worktree.root_path() returns the correct path in langague_server_command(). Maybe it's still using cached_server_path. Extensions may be inited per session not per window.

I actually did try to disable the cached_server_path part, and it's the same, likely the langague_server_command() doesn't even get called twice.
Anyway, mind if I ask how can I possibly "log" anything? As I wasn't able to figure out how to log anything in Zed.

I believe you can just use println!("hello there!"); will be printed to zed: logs.

I am trying to playing around with the extension, I can't see any logs. Even with tracing, nothing gets printed.

Even a panic! doesn't trigger anything when locally building the extension

I actually did try to disable the cached_server_path part, and it's the same, likely the langague_server_command() doesn't even get called twice.

So turns out the language_server_command() does get called again every time we open a new workspace, and I manage to get the extension to use the workspace binary instead, the lack of logging ready confused me a lot😵💫

Unfornately, for some reason, I'm not able to check if the workspace binary exists.
This code right keep returning error for some reason. Not sure if this is because of the Zed wasm thing.

FYI if you build zed from source and run with cargo, you do get logs like this:

and println's too... which is why I thought logs work :)


Patroni creds should be changed from superuser to database username/pwd for DB backup verification

Currently our Application is accessing Patroni using the superuser and superpassword, as per this story, the accessing of the application should be done using the database username/pwd
Acceptance Criteria

[x]  Implement backups in PROD
[x] Change the user in database-user in patroni-pre-req.yml, to be constant and not trailing autogenerated.
[x] Add scripts to assign permission of database user to access sims schema.
[x] Change the backup-deploy.yml to use the DATABASE_USER_KEY_NAME and DATABASE_PASSWORD_KEY_NAME from the database user/pwd by changing the makefile db-backup-deploy-patroni job.
[x] Create verifiation scheduler in patroni-simsdb-backup.conf

Two ACs removed and new tickets created for follow up:

[ ] Jason to do: Determine if OCIO provides off server storage of backups (Currently being stored on PV in OpenShift)
[ ] Jason to do: To check with the team on the outage for the backup and restore


[BUG] Dropouts in IMU data publishing on OAK-D PRO W

Issue description
We are trying to run a VIO algorithm in ROS using the stereo IR images from OAK-D PRO W with mixed success. While analyzing the IMU data, we have found out that the data is published quite sporadically in batches. This is not ideal for VIO, which preintegrates IMU data between image frames to provide a stable high-frequency stream of state estimation. I'm not sure whether the problem is in the way the IMU messages are published to ROS or if the problem is in the reading of data from the sensor itself.
I will try to better explain the issue on these plots:

The OAK-D IMU data in this plot arrives in batches with periods in the tens of ms without any data. Notice the other IMU in the plot, which publishes evenly spaced data points on 100 Hz (OAK-D IMU runs on 400 Hz in this plot). That would be the desired behavior. Can it be achieved or is there something preventing getting evenly-spaced IMU data points from OAK-D?
Let's see the problem in rqt_bag:

The bins are 0.005s long, so each bin should contain 1 IMU message (IMU running at 200 Hz) but there seem to be a window without any message for over ~30 ms.
Solutions attempted
We tried various settings of the IMU parameters to improve this behavior but so far have been unsuccessful in eliminating the problem entirely. The parameter having the biggest effect is the batch_report_threshold and potentially max_batch_reports. If I understand the parameters correctly, OAK waits until batch_report_threshold data points (IMU and camera images combined) before sending over USB and this batch can only have max_batch_reports IMU data points. So unless max_batch_reports < batch_report_threshold  the max_batch_reports parameter should have no effect right? I included both parameter values in case I understand it wrong. Let's see how the parameters influence the consistency of IMU data frequency. The plots show the ROS time of message arrival on the X-axis and the message time stamp (from its header) on the Y-axis.
batch_report_threshold = 5; max_batch_reports = 10 (default)

I see periods of ~40 ms without IMU data. Also, the periodicity of the dropout is suspicious. Could something else be blocking the sending of IMU data?
batch_report_threshold = 20; max_batch_reports = 20

Much worse but expected. OAK waits until 20 data points are available before sending. I included this test to show that changing the parameters in this direction won't help. Also, I saw somewhere in code comment that this setting might help in high-throughput applications, so I wanted to try it.
batch_report_threshold = 1; max_batch_reports = 10

This is the best performance achieved so far. The messages are sent much more often but still, there are ~40ms dropouts. Seems that the dropout has a ~100ms period. Could the communication be blocked by sending camera images at these moments? I don't have access to the camera today but I will check if this happens also when running the IMU only.
The dropouts are periodic and seem to happen when images from all cameras arrive at the same time so indeed it seems that the communication is blocked by the images. Can IMU messages be somehow prioritized?

System info

OAK-D PRO W (IMU type: BNO086, firmware version: 3.9.9, embedded firmware version: 3.9.9)
Recent generation of Intel NUC i7
Ubuntu 20.04
ROS Noetic
depthai-ros build in a ros workspace with-O2 optimization
depthai/depthai-core installed from rosdep
This is all I can provide ATM (no access to OAK-D today) but happy to provide more information tomorrow upon request.

Hi, sorry for the delay and thanks for the detailed reports, we are currently discussing these issues internally and I will get back to you with the conclusions hopefully this/next week.

Any updates on this issue?

@petrlmat
Hi - just one thing to cross check, are you referring to When the reports arrive or What timestamp they hold?
Due to comms channel we cannot have eg "realtime" IMU updates, but batched IMU readings should still hold correct individual timestamp per report.
Those reports should be feed into the system with the timestamp component

Hi, I am referring to "When the reports arrive". Realtime IMU updates are necessary for running realtime state estimation algorithms. ~40 ms long period without data is just too much for state estimation of unstable systems such as drones. We would like to use your camera as a "do it all" camera on our drones but due to this issue we have to rely on our custom solution with realtime IMU.
How can the time between batches be influenced? Would e.g. sending less camera streams through the comm channel increase the batch frequency?

@petrlmat

How can the time between batches be influenced? Would e.g. sending less camera streams through the comm channel increase the batch frequency?

Yes, this is likely the biggest factor. You may quickly check it by only sending the IMU updates instead of camera stream as well

Any updates on this issue? Is there a plan or possibility to obtain the IMU measurements ASAP instead of getting them in batches or is it infeasible due to hardware limitations?


Dev

add confusion matrix and ROC curve

I believe the y are the actual responses (0 or 1), @boyinggong can you make some comments to the function

Sorry I've updated the comments.

Check #112

@BenjaminHsieh I've combined the code into the logistic_function.py and logistic.py file. I also update the code to subplot all figures into one. Please check it and let me know if there is any error.

Cool, can you also add a subject label at the top of each ROC graph?

I am going to merge this for now, we can add the subject label later


Update support link as it was requested by Salesforce.

Signed-off-by: Yury Fridlyand<EMAIL_ADDRESS>Description
Salesforce has requested that the support link be updated from a GitHub link to OpenSearch .
Issues Resolved
Check List

[ ] New functionality includes testing.

[ ] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Upstream PR: https://github.com/opensearch-project/sql/pull/643


[JOSS] Mention binder availability in README

This issue was opened as part of my JOSS review: https://github.com/openjournals/joss-reviews/issues/6889
The documentation of the tutorials offer binder integration which is quite nice. However, this is not mentioned in the README, meaning some users will not see it. I think this is a really nice feature that it is worth mentioning in the README. In my experience it lowers the barrier to entry, but only if people know it is there. For example, I'd already made an environment before I realized the binder integration was an option. You might consider either adding a binder badge to the README, or mentioning the option when you introduce the tutorials.

I have now added a badge to the README showing the binder integration, but haven't yet considered how (or if) to include it elsewhere in the README. I just wanted to provide an update here since there is no linked pull request for that change.

On reflection, I think adding the badge to the README seems sufficient. I don't want to further complicate the documentation with another topic. I'm happy to add some descriptive text to the tutorial gallery if you think it's worthwhile. Due to the need to download data to get the most out of interactive use of scores, the binder experience is somewhat limited, and I think a simple approach is suitable. As a result, I'm going to close this issue, but please do feel free to re-open it or leave further comment and I'll happily revisit it.


when Mekanism running on the SpongeForge Server cause Error

Error logs:https://pastebin.com/rVKY4XxB
Forge Version:1.12.2-<IP_ADDRESS>05
SpongeForge Version:1.12.2-2705-7.1.0-BETA-3291
Mod list:https://pastebin.com/Zsf5A1wL
Plugins List:https://pastebin.com/bfLvYCpr
Mekanism Version:<IP_ADDRESS>9 for 1.12.2

The function mentioned in the stack loops the player list provided by the server. If it's null by the time it tries to send it there's nothing we can do. If there's a null entry in the list, that's a sponge problem. Not enough info to tell.
Either the player managed to disconnect at just the wrong moment or some internal Forge/Sponge stuff is going wonky.