fix: removed extra blank link between sections

Checklist:

[x] I have read freeCodeCamp's contribution guidelines.
[x] My pull request has a descriptive title (not a vague title like Update index.md)
[x] My pull request targets the main branch of freeCodeCamp.
[x] I have tested these changes either locally on my machine, or GitPod.

Closes #XXXXX

Hey @quinimi
Thank you for opening this pull request.
This is a standard message notifying you that we've reviewed your pull request and have decided not to merge it. We would welcome future pull requests from you.
Thank you and happy coding.


Add support for adding config to BALENA_HOST_CONFIG_dtoverlay

This was an oversight on my part, since my Ribbit network had RESIN_HOST_CONFIG_dtoverlay set up by default. I decided to keep compatibility with the old variable name since anyone starting up their own network clone could run into the same problem.
This also contains a bug fix to create the balena variable with a correct name, rather than the uninitialized variable.

Meant to get this in sooner, since I really want to get our sensor up and running on the network. I accidentally broke my laptop charger and consequently the charging port for my laptop. Had to spend some time debugging and ordering a new part :)
I believe this should fix the issue. I tested running on my personal network with the same Balena variables configured as your network. No problems with the extra overlays specified, although I agree with your point from Discord that it would be useful to programmatically set up the device tree configuration. This allows us to be more open and also more precise since we can fine-tune configuration based on device type.

Looks good to me! Yes I created a separate issue #66 to track setting the variable programtically.

We've confirmed that this works on @djgood 's Raspberry Pi 3 with a serial GPS module.
I'm testing now on a USB GPS CM4 and will merge once verified that everything still works :)

@djgood running this on a CM4 board here, it seems that this branch has broken something as the USB Power is disabled when I run this. I'm digging into it further to see what the matter is.

@djgood running this on a CM4 board here, it seems that this branch has broken something as the USB Power is disabled when I run this. I'm digging into it further to see what the matter is.

That's weird, I'm not sure why it's effecting the USB power. My code shouldn't change the DT overlay configuration for a CM4. What is BALENA_HOST_CONFIG_dtoverlay for your CM4 board?

Yeah I’m still trying untangle exactly the issue.
The overlay needs to be dtoverlay=dwc2,dr_mode=host for the carrier but it
seems like it’s getting reset. I’ll try a few things and see what I can
find in the morning.
We’re you able to read out the variable via echo $var name or how did you
check your variables in testing?
Thanks!
On Wed, Nov 10, 2021 at 6:53 PM Desmond Good @.***>
wrote:

@djgood https://github.com/djgood running this on a CM4 board here, it
seems that this branch has broken something as the USB Power is disabled
when I run this. I'm digging into it further to see what the matter is.
That's weird, I'm not sure why it's effecting the USB power. My code
shouldn't change the DT overlay configuration for a CM4. What is
BALENA_HOST_CONFIG_dtoverlay for your CM4 board?
—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/Ribbit-Network/ribbit-network-frog-sensor/pull/65#issuecomment-965943349,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATQ3FQ74BCQR7PUWSYLXB3ULMVZNANCNFSM5HW4U37A
.

The problem probably has to do with how I tried to get fancy and disable the UART if a CDC device is detected at gpsd.py:150. It should just be connecting to the SDK, seeing that there is no disable-bt overlay set and returning.
If it's not returning, something that may be breaking things is that my code currently parses dwc2,dr_mode=host into "dwc2","dr_mode=host","disable-bt" when it adds in the disable-bt overlay. I didn't realize that dt overlays can have a comma in them. Right now, you can get around this by surrounding your dt overlay with quotes, but I'll push a fix for that.
I was checking the value of BALENA_HOST_CONFIG_dtoverlay through "Device Configuration" on the Balena dashboard.

Here's the logs from a CM4 Test device. From the container I see:
 gpsd  USB CDC device detected.
 gpsd  Settings file not found or not in proper format. Rewriting default settings to: /root/.balena/balena.cfg
 gpsd  Error creating dtoverlay
 gpsd  UART0 disabled.
 gpsd  Starting gpsd attached to /dev/ttyACM0...

then I see
Applying boot config: {"dtoverlay":["disable-bt"],"arm_64bit":"1","avoid_warnings":"1","disable_splash":"1","dtparam":["i2c_arm=on","spi=on","audio=on"],"gpu_mem":"16"}
So you can see it's definitely removing the "dwc2","dr_mode=host"

Ok, yeah, that’s clearly an issue. I’ll go back and test the USB CDC case on my set up again to figure out why it’s overwriting the overlay.

On Nov 11, 2021, at 12:39 PM, Keenan Johnson @.***> wrote:

Here's the logs from a CM4 Test device. From the container I see:
gpsd  USB CDC device detected.
gpsd  Settings file not found or not in proper format. Rewriting default settings to: /root/.balena/balena.cfg
gpsd  Error creating dtoverlay
gpsd  UART0 disabled.
gpsd  Starting gpsd attached to /dev/ttyACM0...
then I see
Applying boot config: {"dtoverlay":["disable-bt"],"arm_64bit":"1","avoid_warnings":"1","disable_splash":"1","dtparam":["i2c_arm=on","spi=on","audio=on"],"gpu_mem":"16"}
So you can see it's definitely removing the "dwc2","dr_mode=host"
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Thanks! Yeah I have a few other things I need to sort out today but I can help take a look at the source more directly sometime in the next few days to see exactly what the error is.

I'm not able to reproduce it in my fleet, unfortunately.
But my current theory for the log line you posted is that the existing BALENA_HOST_CONFIG_dtoverlay value was empty or otherwise considered "false" in Python. It might have cleared itself on a previous boot, so we might only be seeing a symptom of the issue.
I've uploaded some code to hopefully make the config variable detection a bit more reliable and also parse the dt_overlay variable more carefully.
When you have a moment, can you try pushing my branch to your device? Try resetting BALENA_HOST_CONFIG_dtoverlay to dwc2,dr_mode=host and watching to see what happens.

Ok I just deployed it to a test fleet here. I can give you access to the device if you would like @djgood, but it seems like the same behavior. After pushing this branch, the BALENA_HOST_CONFIG_dtoverlay is reset to just have disable-bt.
I haven't looked at the source code closely yet with this change in mind, but I will have some time tomorrow to do so.

Ok, thanks! Looking at it in the dashboard should be a bit easier to debug. My Balena username is gh_djgood

Ok I just added you to the test-ribbit fleet. The device 5582468ac31e723a6def4580c131f69a (mean-photo) is now running the latest from this branch. Feel free to push to that fleet all your want. The device is inside, so the GPS reception might be a bit spotty, but the connectivity issues will be obvious.

Amazing! I should be able to figure out what’s happening, will report back.

On Nov 11, 2021, at 10:53 PM, Keenan Johnson @.***> wrote:

Ok I just added you to the test-ribbit fleet. The device 5582468ac31e723a6def4580c131f69a (mean-photo) is now running the latest from this branch. Feel free to push to that fleet all your want. The device is inside, so the GPS reception might be a bit spotty, but the connectivity issues will be obvious.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Awesome! Thank you so much for tackling this!
On Thu, Nov 11, 2021 at 8:08 PM Desmond Good @.***>
wrote:

Amazing! I should be able to figure out what’s happening, will report back.

On Nov 11, 2021, at 10:53 PM, Keenan Johnson @.***> wrote:

Ok I just added you to the test-ribbit fleet. The device
5582468ac31e723a6def4580c131f69a (mean-photo) is now running the latest
from this branch. Feel free to push to that fleet all your want. The device
is inside, so the GPS reception might be a bit spotty, but the connectivity
issues will be obvious.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/Ribbit-Network/ribbit-network-frog-sensor/pull/65#issuecomment-966804116,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATQ3FXMOFRGNWTI3ENL263ULSHKJANCNFSM5HW4U37A
.

I reset the variable to the override value and it seems to be working ok.
Maybe my fix fixed it? Or it’s a problem with initial boot up.

On Nov 11, 2021, at 11:18 PM, Keenan Johnson @.***> wrote:

Awesome! Thank you so much for tackling this!
On Thu, Nov 11, 2021 at 8:08 PM Desmond Good @.***>
wrote:

Amazing! I should be able to figure out what’s happening, will report back.

On Nov 11, 2021, at 10:53 PM, Keenan Johnson @.***> wrote:

Ok I just added you to the test-ribbit fleet. The device
5582468ac31e723a6def4580c131f69a (mean-photo) is now running the latest
from this branch. Feel free to push to that fleet all your want. The device
is inside, so the GPS reception might be a bit spotty, but the connectivity
issues will be obvious.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/Ribbit-Network/ribbit-network-frog-sensor/pull/65#issuecomment-966804116,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATQ3FXMOFRGNWTI3ENL263ULSHKJANCNFSM5HW4U37A
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

Oh interesting. Have you tried rebooting the device? Does your overridden
value stay in that case? Yeah it must be either something that only occurs
during a software update or a reboot.
On Thu, Nov 11, 2021 at 8:34 PM Desmond Good @.***>
wrote:

I reset the variable to the override value and it seems to be working ok.
Maybe my fix fixed it? Or it’s a problem with initial boot up.

On Nov 11, 2021, at 11:18 PM, Keenan Johnson @.***> wrote:

Awesome! Thank you so much for tackling this!
On Thu, Nov 11, 2021 at 8:08 PM Desmond Good @.***>
wrote:

Amazing! I should be able to figure out what’s happening, will report
back.

On Nov 11, 2021, at 10:53 PM, Keenan Johnson @.***> wrote:

Ok I just added you to the test-ribbit fleet. The device
5582468ac31e723a6def4580c131f69a (mean-photo) is now running the latest
from this branch. Feel free to push to that fleet all your want. The
device
is inside, so the GPS reception might be a bit spotty, but the
connectivity
issues will be obvious.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<
https://github.com/Ribbit-Network/ribbit-network-frog-sensor/pull/65#issuecomment-966804116
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AATQ3FXMOFRGNWTI3ENL263ULSHKJANCNFSM5HW4U37A

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/Ribbit-Network/ribbit-network-frog-sensor/pull/65#issuecomment-966813528,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATQ3FR6POZZRUA2V5ZUJ5TULSKMJANCNFSM5HW4U37A
.

Yeah, the right value persists through a reboot. I’ll try updating the device and seeing what happens.

On Nov 11, 2021, at 11:43 PM, Keenan Johnson @.***> wrote:

Oh interesting. Have you tried rebooting the device? Does your overridden
value stay in that case? Yeah it must be either something that only occurs
during a software update or a reboot.
On Thu, Nov 11, 2021 at 8:34 PM Desmond Good @.***>
wrote:

I reset the variable to the override value and it seems to be working ok.
Maybe my fix fixed it? Or it’s a problem with initial boot up.

On Nov 11, 2021, at 11:18 PM, Keenan Johnson @.***> wrote:

Awesome! Thank you so much for tackling this!
On Thu, Nov 11, 2021 at 8:08 PM Desmond Good @.***>
wrote:

Amazing! I should be able to figure out what’s happening, will report
back.

On Nov 11, 2021, at 10:53 PM, Keenan Johnson @.***> wrote:

Ok I just added you to the test-ribbit fleet. The device
5582468ac31e723a6def4580c131f69a (mean-photo) is now running the latest
from this branch. Feel free to push to that fleet all your want. The
device
is inside, so the GPS reception might be a bit spotty, but the
connectivity
issues will be obvious.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<
https://github.com/Ribbit-Network/ribbit-network-frog-sensor/pull/65#issuecomment-966804116
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AATQ3FXMOFRGNWTI3ENL263ULSHKJANCNFSM5HW4U37A

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/Ribbit-Network/ribbit-network-frog-sensor/pull/65#issuecomment-966813528,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATQ3FR6POZZRUA2V5ZUJ5TULSKMJANCNFSM5HW4U37A
.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

I wonder if there is some difference in the way the Balena supervisor
handles updates.
In my case I was applying the override at the fleet level originally. Then
when the script initially sets the variable to bt_disable I would just
delete the override in the Balena UI assuming it would force a reset.
However it looks like you went in and positively asserted the right overlay
paramaeters at the device override level, which must be subtly different
than what I did. Interesting.
On Thu, Nov 11, 2021 at 8:52 PM Desmond Good @.***>
wrote:

Yeah, the right value persists through a reboot. I’ll try updating the
device and seeing what happens.

On Nov 11, 2021, at 11:43 PM, Keenan Johnson @.***> wrote:

Oh interesting. Have you tried rebooting the device? Does your overridden
value stay in that case? Yeah it must be either something that only
occurs
during a software update or a reboot.
On Thu, Nov 11, 2021 at 8:34 PM Desmond Good @.***>
wrote:

I reset the variable to the override value and it seems to be working
ok.
Maybe my fix fixed it? Or it’s a problem with initial boot up.

On Nov 11, 2021, at 11:18 PM, Keenan Johnson @.***> wrote:

Awesome! Thank you so much for tackling this!
On Thu, Nov 11, 2021 at 8:08 PM Desmond Good @.***>
wrote:

Amazing! I should be able to figure out what’s happening, will
report
back.

On Nov 11, 2021, at 10:53 PM, Keenan Johnson @.***> wrote:

Ok I just added you to the test-ribbit fleet. The device
5582468ac31e723a6def4580c131f69a (mean-photo) is now running the
latest
from this branch. Feel free to push to that fleet all your want.
The
device
is inside, so the GPS reception might be a bit spotty, but the
connectivity
issues will be obvious.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or
Android.

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<

https://github.com/Ribbit-Network/ribbit-network-frog-sensor/pull/65#issuecomment-966804116

,

or unsubscribe
<

https://github.com/notifications/unsubscribe-auth/AATQ3FXMOFRGNWTI3ENL263ULSHKJANCNFSM5HW4U37A

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
<
https://github.com/Ribbit-Network/ribbit-network-frog-sensor/pull/65#issuecomment-966813528
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AATQ3FR6POZZRUA2V5ZUJ5TULSKMJANCNFSM5HW4U37A

.

—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.
Triage notifications on the go with GitHub Mobile for iOS or Android.

—
You are receiving this because your review was requested.
Reply to this email directly, view it on GitHub
https://github.com/Ribbit-Network/ribbit-network-frog-sensor/pull/65#issuecomment-966819422,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AATQ3FT2BEETMSM2FMKX7OLULSMQFANCNFSM5HW4U37A
.

My code doesn't detect that a variable exists when the device supervisor is inheriting the fleet override and does its own creation of the variable. So fleet overwrites behave a bit different than variables. I probably have to access another API to find out if there are any overrides specified for a given variable.
Another problem is that we always create the configuration to enable the UART if the configuration variable doesn't exist. So even when it's trying to disable the UART, it will create a new variable with "disable-bt". Easy enough fix for this though.

@all-contributors please add @djgood  for his great ideas and code contributions to this project.

Thanks for the shout out! Glad to be helping out with cool project! 😄
I did some poking around to see what calls the balena dashboard is using and found the API that I need to check if a fleet-wide variable is defined. Turns out I need to first check application_config_variable to see if it's defined fleet wide and then check out device_config_variable to see what the individual device configuration is. Fix is in progress now and hopefully should resolve our issues.

This is ready to go pending review right @djgood ?

This is ready to go pending review right @djgood ?

Yup, everything should be working now!


Firebase ios - no known class method for selector 'databaseWithURL:'

Steps to Reproduce

create a project
add the firebase dependencies
run flutter build ios

(upgrade dependencies to latest if needed, some versions may state behind, but are latest)
dev_dependencies:
  flutter_test:
    sdk: flutter
  intl: "^0.15.2"
  shared_preferences: "^0.3.1"
  google_sign_in: "^2.0.1"
  firebase_analytics: "^0.2.1"
  firebase_auth: "^0.4.2"
  firebase_database: "^0.3.3"
  firebase_messaging: "^0.1.2"
  firebase_admob: "^0.2.3"

Logs

    /Users/branflake2267/git/family-economy/economy/ios/Pods/Headers/Public/Flutter/Flutter/FlutterPlugin.h:129:35: warning: 'UIApplicationShortcutItem' is partial: introduced in iOS 9.0 [-Wunguarded-availability]
        performActionForShortcutItem:(UIApplicationShortcutItem*)shortcutItem
                                      ^
    In module 'UIKit' imported from /Users/branflake2267/git/family-economy/economy/ios/Pods/Target Support Files/firebase_database/firebase_database-prefix.pch:2:
    /Applications/Xcode.app/Contents/Developer/Platforms/iPhoneOS.platform/Developer/SDKs/iPhoneOS11.2.sdk/System/Library/Frameworks/UIKit.framework/Headers/UIApplicationShortcutItem.h:60:12: note: 'UIApplicationShortcutItem' has been explicitly marked partial here
    @interface UIApplicationShortcutItem : NSObject <NSCopying, NSMutableCopying>
               ^
    In file included from /Users/branflake2267/.pub-cache/hosted/pub.dartlang.org/firebase_database-0.3.4/ios/Classes/FirebaseDatabasePlugin.m:5:
    In file included from /Users/branflake2267/.pub-cache/hosted/pub.dartlang.org/firebase_database-0.3.4/ios/Classes/FirebaseDatabasePlugin.h:5:
    In file included from /Users/branflake2267/git/family-economy/economy/ios/Pods/Headers/Public/Flutter/Flutter/Flutter.h:34:
    In file included from /Users/branflake2267/git/family-economy/economy/ios/Pods/Headers/Public/Flutter/Flutter/FlutterAppDelegate.h:11:
    /Users/branflake2267/git/family-economy/economy/ios/Pods/Headers/Public/Flutter/Flutter/FlutterPlugin.h:128:1: note: annotate 'application:performActionForShortcutItem:completionHandler:' with an availability attribute to silence
    - (BOOL)application:(UIApplication*)application
    ^
    /Users/branflake2267/.pub-cache/hosted/pub.dartlang.org/firebase_database-0.3.4/ios/Classes/FirebaseDatabasePlugin.m:161:29: error: no known class method for selector 'databaseForApp:URL:'
        database = [FIRDatabase databaseForApp:[FIRApp appNamed:appName] URL:databaseURL];
                                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    /Users/branflake2267/.pub-cache/hosted/pub.dartlang.org/firebase_database-0.3.4/ios/Classes/FirebaseDatabasePlugin.m:165:29: error: no known class method for selector 'databaseWithURL:'
        database = [FIRDatabase databaseWithURL:databaseURL];
                                ^~~~~~~~~~~~~~~
    3 warnings and 2 errors generated.

Flutter Doctor
brandons-mbp:economy branflake2267$ flutter doctor
[✓] Flutter (on Mac OS X 10.13.2 17C205, locale en-US, channel master)
    • Flutter version 0.0.21-pre.325 at /Users/branflake2267/git/flutter
    • Framework revision b2c65e7653 (2 days ago), 2018-01-27 14:21:58 +0100
    • Engine revision 6921873c71
    • Tools Dart version 2.0.0-dev.16.0
    • Engine Dart version 2.0.0-edge.da1f52592ef73fe3afa485385cb995b9aec0181a

[✓] Android toolchain - develop for Android devices (Android SDK 27.0.2)
    • Android SDK at /Users/branflake2267/Library/Android/sdk
    • Android NDK at /Users/branflake2267/Library/Android/sdk/ndk-bundle
    • Platform android-27, build-tools 27.0.2
    • Java binary at: /Applications/Android Studio.app/Contents/jre/jdk/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] iOS toolchain - develop for iOS devices (Xcode 9.2)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Xcode 9.2, Build version 9C40b
    • ios-deploy 1.9.2
    • CocoaPods version 1.3.1

[✓] Android Studio (version 3.0)
    • Android Studio at /Applications/Android Studio 3.0.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] Android Studio (version 3.0)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Java version OpenJDK Runtime Environment (build 1.8.0_152-release-915-b08)

[✓] IntelliJ IDEA Ultimate Edition (version 2017.3.3)
    • Flutter plugin version 21.1.3
    • Dart plugin version 173.4301.22

[✓] IntelliJ IDEA Community Edition (version 2017.2.6)
    • Flutter plugin version 21.1.2
    • Dart plugin version 172.4343.25

[✓] Connected devices
    • Android SDK built for x86 • emulator-5554                        • android-x86 • Android 8.0.0 (API 26) (emulator)
    • Nexus 6                   • ZX1G22RV45                           • android-arm • Android 7.0 (API 24)
    • iPhone X                  • 8463530E-065D-4A3A-B7A7-3E1617D862C4 • ios         • iOS 11.2 (simulator)

Just tried to reproduce with same version of Flutter and Xcode as you are using according to flutter doctor above. I get zero errors:

Any chance you have configured Xcode to use a stricter-than-default static analysis?

I haven't changed any defaults in xcode. I did create a new project and moved all the code to it to see if I had something cached or old sitting around. With that said, I'm not aware of xcode configuration that might be doing this yet. I'll work on finding the issue and maybe a test case if I can find it. I'll break it down. Thanks for looking into it.

So far, this looks like a dependency issue where the classpath has a newer version or maybe older version of type FIRDataSnapshot. I'm not very good at reading ObjectiveC yet, and haven't work with the library configuration yet in xcode. I'll aim to figure out how it's loading dependent libraries, pods.
https://github.com/firebase/firebase-ios-sdk/blob/master/Firebase/Database/Public/FIRDataSnapshot.h

I found it, the pod was out of date. I found the firebase pod was not up to date.
https://cocoapods.org/pods/Firebase
So my solution was to go into the ios project director and run pod update.
Snippet from the run:

(just in case somebody else runs into the same problem)
I was having the same error and was unable to pod update due to a connection error to GitHub (something to do with a TLS update they did in feb 2018).  I had to remove the existing repo and clone it again.
Problem:
pod update
[...]
[!] Failed to connect to GitHub to update the CocoaPods/Specs specs repo - Please check if you are offline, or that GitHub is down

Solution:
pod repo remove master
pod setup
flutter clean

However, even after that I still had the annoying:
    /.pub-cache/hosted/pub.dartlang.org/firebase_database-0.4.6/ios/Classes/FirebaseDatabasePlugin.m:166:29: error: no known class method for selector 'databaseForApp:URL:'
        database = [FIRDatabase databaseForApp:[FIRApp appNamed:appName] URL:databaseURL];
                                ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    /.pub-cache/hosted/pub.dartlang.org/firebase_database-0.4.6/ios/Classes/FirebaseDatabasePlugin.m:170:29: error: no known class method for selector 'databaseWithURL:'
        database = [FIRDatabase databaseWithURL:databaseURL];
                                ^~~~~~~~~~~~~~~

The error only disappeared when I removed the dependency to firebase_database in pubspec.yaml and then placed in back.


triton-nightly package not updated since July 17

it seems like this problem is already known, but the latest triton-nightly package is from 3 months ago:
$ pip index versions --index-url https://aiinfra.pkgs.visualstudio.com/PublicPackages/_packaging/Triton-Nightly/pypi/simple/ triton-nightly
triton-nightly (3.0.0.post20240716052845)
Available versions: 3.0.0.post20240716052845, 3.0.0.post20240715060519, 3.0.0.post20240713005620, ...

The release of triton==3.1.0 yesterday causes triton-nightly to be more outdated than actual triton now.

Note that 3.1.0 is a cherry-pick branch that only adds a few commits on top of 3.0.0. If triton-nightly worked properly, it would be more recent than 3.1.0.


Fix linking issue due to missing inline keywords

Codecov Report

Merging #15 into master will increase coverage by 0.02%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master      #15      +/-   ##
==========================================
+ Coverage   87.96%   87.99%   +0.02%
==========================================
  Files           3        3
  Lines         407      408       +1
==========================================
+ Hits          358      359       +1
  Misses         49       49

Impacted Files
Coverage Δ

include/spdlog_setup/conf.h
83.8% <ø> (ø)
:arrow_up:

include/spdlog_setup/details/conf_impl.h
89.26% <ø> (ø)
:arrow_up:

include/spdlog_setup/details/setup_error.h
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 e08cdd6...6e63583. Read the comment docs.


 not working after gradle update

Expected Behavior
We have one target that uses a different activity then the others. We managed this by adding the following lines in the manifest of that target:
        <activity-alias
            android:name="com.twipemobile.twpublishing.ui.to.HomeScreenActivity"
            android:targetActivity="com.twipemobile.client.activity.ClientHomeScreenActivity" />

When we launch that tartget the activity ClientHomeScreenActivity should be used instead of HomeScreenActivity.
Current Behavior
When we use gradle 2.10 it works correctly.
When we upgrade to gradle 4.1 it does not work anymore and the standard HomeScreenActivity is used.
Context
We are trying to migrate our platform to a newer gradle version. Most features stay the same but the  suddenly get's ignored.

For Android issues, please use the Android issue tracker.


Fixed danger local crash when there are no recent PRs #trivial

Show an actual error message if there haven't been any PRs yet

        1 Warning

  </th>
 </tr>

:warning:
Tests were not updated

  Generated by :no_entry_sign: danger

@KrauseFx the local command will never work with repositories that use a rebase workflow, should we have a look at that some time? (Should I create a issue?)

Rest looks good! 👍

@jeroenvisser101 in this case it was just a new project I started working on, where I just haven't created a PR yet, but plan on doing so, since it's open source

👍🏾  good call

@KrauseFx Alright, but the issue that rebase workflows don't work persist, although it seems that GitHub can see which commit belonged to which branches, we should be able to retrieve that somewhere. I might have a look tonight and see if that's doable. Nevertheless, it's outside the scope of this PR :)

Cool, yeah, make an issue, that's an interesting problem


Sync eng/common directory with azure-sdk-tools for PR 1160

Sync eng/common directory with azure-sdk-tools for PR https://github.com/Azure/azure-sdk-tools/pull/1160  See eng/common workflow

/check-enforcer override


rendering of images doesnt work on module landing

Is this issue regarding the huge banner/background image on https://deno.land/x?

No, its related to ie going to https://deno.land/x/fresh; the image doesnt render. seems to be an issue in marked, unsure though


Clean up repository for Make It So on Apple

Use main as the default branch
Do feature development on feature branches and merge into main once ready
Once implementation work is finished, create separate branches for derivative work (videos, blog posts, interactive DocC tutorials)

[ ] Create a branch for the current version
[ ] Create README and point to the previous version(s)

Hi Peter, thanks very much for the example app, it was super helpful to see potential 'best practices'.. I've been wondering if you might be planning a Swift 6 update showing the latest approaches you'd take for firebase auth, states, dependency injection, etc. In other words replace Combine with the new concurrency model to have a hybrid of @Observable for state management and Actors for thread safety, etc. I ask because I've learned so much from you and want this Friese train to keep rollin!

Hi Peter, thanks very much for the example app, it was super helpful to see potential 'best practices'.. I've been wondering if you might be planning a Swift 6 update showing the latest approaches you'd take for firebase auth, states, dependency injection, etc. In other words replace Combine with the new concurrency model to have a hybrid of @observable for state management and Actors for thread safety, etc. I ask because I've learned so much from you and want this Friese train to keep rollin!

Hi @mikeymike9000 - yes, we're working on it. Feel free to use the GitHub discussions for this project to ask questions.

great thanks Peter and I'll use GitHub discussions going forward!


feat: import new widgets in Tactic.Common

Is there some reason we haven't done this already?

bors merge


Adding 6 python examples to rfsg, rfmxwlan, and rfmxnr

What does this Pull Request accomplish?
This PR adds 6 new python examples to RFSG, RFmx WLAN, and RFmx NR.
Why should this Pull Request be merged?
These examples help broaden the base of examples for RFSG, RFmx WLAN, and RFmx NR. The examples are modeled on the corresponding examples found in the shipped RFSG/RFmx examples in .NET and LabVIEW.
What testing has been done?
These were all tested on a Ubuntu VM, communicating with real NI hardware (5830 VST) over a network.

You can run black to fix up the broken build step validate_python. You can see how to run it from the command line or configure in VS Code in our Wiki

Looks good overall to me, but I'm going to add @yardov as a reviewer in case he has any additional perspective on the example content.
Also, you should probably look at getting your GitHub account added to the NI org and team: https://github.com/ni

@reckenro Thanks, I ran black on the files and committed the changes.

@reckenro Thanks, I ran black on the files and committed the changes.

Sure thing. Looks like your recent updates are still failing the validation check. I'm not sure why but I think when you ran Black it didn't pick up the 100 line-length specified in the pyproject.toml.
In any case, you should be able to force it on the command line with something like black source/codegen examples -l 100.
Besides that, they look good in general to me.

The formatting is fixed, but there are errors in the validate_python step. I ran modacc-noncontiguous-multicarrier.py after the formatting changes with real hardware without issue. I am not sure how to resolve validate_python errrors.

Ah, forgot about that. It got past the formatting and ni-python-styleguide checks but then it runs mypy on the examples. mypy gets some type-checking in python. I'm not super familiar but I ran the validate_examples.py locally and will point out what I think the problems are in each file.


Possible feature: JSON -> CSV

I used the JSON option to download RIB dumps into JSON files. I'm wondering if it would be possible to have the same keys as the JSON, but as a header for a CSV or PSV?
The reason being I'm working on a lot of MRT data, and outputting all the RIB dumps I have as a JSON file takes up too much space for my machine. Additionally, when analyzing the files that get output, it's a lot faster to load in a CSV than it is to load in a JSON. However, I don't want to lose the information provided in all of the JSON keys - it is so incredibly useful!

Can you take a look at PR #145?

Absolutely, tutoring someone rn but should be able to check it out in an hour or two. Thank you so much!

I just checked this out, this is totally awesome! Literally twice as fast, and a third the size of the JSON for the same info. It also includes the super nice only_to_customers column, which is super important.
A few questions:
It looks like there’s a few elements that are included in the JSON but not in the PSV. Specifically, origin_asns, peer_asn, peer_ip, deprecated, and unknown. Not critical but would be nice to have.
Also for those elements, what are the deprecated and unknown fields that are present in the JSON? Very curious about these. Same with the origin_asns, I was under the assumption that the last ASN in the path was the origin, or is it something different (perhaps due to AS sets?)
And if those other elements aren’t going to be included no worries, feel free to close the issue and/or merge the PR at any point. Thank you so much for this!

*edited, I just missed the peer_asn and peer_ip in the PSV header, oops!

To answer your questions:

origin_asns is a derived field from the AS_PATH. I can put it into the PSV as it is already in BgpElem. It is plural due to the existence of AS sets, exactly as you mentioned.
deprecated and unknown are for BGP path attributes that were known to be deprecated or unknown. The data for these fields were not parsed, i.e. remain raw bytes, and not very useful to write to output. It can be processed with Rust SDK if needed.

I added the new origin_asns field and tested it with importing the PSV file into sqlite3. See more details here:
https://github.com/bgpkit/bgpkit-parser/pull/145#issuecomment-1843597064

Super awesome, just tried it out, works great!
Makes sense regarding the deprecated and unknown fields (and origin asns). I think the deprecated and unknown fields might be nice to have for research purposes to be honest, because it might be an interesting result to be able to say X% of MRT Announcements still use deprecated BGP attributes, and those attributes may, in certain ASes, still affect routing decisions. Same with unknown attributes.
Super cool tool, so glad I came across this, and thanks a ton for providing this information in such a nice and fast way. It has significantly sped up my parsing of MRT announcements.
Feel free to close the issue, thanks again!

Glad that I could help!


Update Dummy Officers Data

User Story
As a user who wants keep my data in the system accurate and up to date
I want to be able to see fields I understand in my data table
So that I am clear about my data in the system
Definition of Done
The following tasks have been completed:

[ ] Update the dummy data for the officers seeders so that most officers have supervisors and the supervisor_employee_id column actually points to another officer's employee_id a script could be useful for this (see reroute-supervisor-officer-number.js). but this could also be done manually if desired.
[ ] Ensure that when an officer is attached to a case their supervisor's name and employee_id is replicated onto their cases_offciers reference

Acceptance Criteria
Given I am in any non prod client environment
When I make a new case and attach an officer that has a supervisor
Then I can see who their supervisor is on the incident details page.

Update:
We created a new officer seed data CSV. It's identical to the previous seeder, except the supervisor_employee_id column is filled in for most officers now (per this card's requirements).
Doing a down & up worked with the new seeder, and tests pass.
We've uploaded this new CSV into the instance files for both NOIPM and Hawaii and pushed those changes.
I've also added this new CSV in S3 buckets in local, CI, and staging.
Some tests are still failing in the pipeline, and I think that is because an earlier push failed the pipeline and I (accidentally) pushed my commit before making sure the pipeline is green, so may need to collaborate to figure out what broke the pipeline in the first place.

Verified appropriate supervisors are displayed on incident details page.


Added Suite

Unfortunately, I have to close this PR without merging because your branch now has unrelated changes (the Interp changes). I'll make a new branch called Marsh-suite from the previous commit of your branch, and then you can make a new PR for that branch and make the final change to suite on there.


Update chroma-hnswlib to 0.7.5

When I install crewai,  chroma-hnswlib 0.7.3 counld not installed, The msg show:
clang: error: unsupported argument 'native' to option '-march='.
I install chroma-hnswlib 0.7.5, it successed.
CrewAI Version:  0.36.0
macOS Version: Sonoma 14.5
Python Version: 3.12.4
Full log:
  Building wheel for chroma-hnswlib (pyproject.toml) ... error
  error: subprocess-exited-with-error

  × Building wheel for chroma-hnswlib (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [17 lines of output]
      running bdist_wheel
      running build
      running build_ext
      creating var
      creating var/folders
      creating var/folders/hb
      creating var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn
      creating var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn/T
      clang -fno-strict-overflow -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -O3 -Wall -arch arm64 -arch x86_64 -g -I/Library/Frameworks/Python.framework/Versions/3.12/include/python3.12 -c /var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn/T/tmpldlzi1cr.cpp -o var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn/T/tmpldlzi1cr.o -std=c++14
      clang -fno-strict-overflow -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -O3 -Wall -arch arm64 -arch x86_64 -g -I/Library/Frameworks/Python.framework/Versions/3.12/include/python3.12 -c /var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn/T/tmpuucg6izh.cpp -o var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn/T/tmpuucg6izh.o -fvisibility=hidden
      building 'hnswlib' extension
      creating build
      creating build/temp.macosx-10.9-universal2-cpython-312
      creating build/temp.macosx-10.9-universal2-cpython-312/python_bindings
      clang -fno-strict-overflow -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -O3 -Wall -arch arm64 -arch x86_64 -g -I/private/var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn/T/pip-build-env-c83o18ap/overlay/lib/python3.12/site-packages/pybind11/include -I/private/var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn/T/pip-build-env-c83o18ap/overlay/lib/python3.12/site-packages/numpy/_core/include -I./hnswlib/ -I/Library/Frameworks/Python.framework/Versions/3.12/include/python3.12 -c ./python_bindings/bindings.cpp -o build/temp.macosx-10.9-universal2-cpython-312/./python_bindings/bindings.o -O3 -march=native -stdlib=libc++ -mmacosx-version-min=10.7 -DVERSION_INFO=\"0.7.3\" -std=c++14 -fvisibility=hidden
      clang: error: unsupported argument 'native' to option '-march='
      error: command '/usr/bin/clang' failed with exit code 1
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for chroma-hnswlib
Failed to build chroma-hnswlib
ERROR: ERROR: Failed to build installable wheels for some pyproject.toml based projects (chroma-hnswlib)

What version Crewai are you using?

The latest version, 0.36.0

When I used:
CC=gcc CXX=g++ pip3 install chroma-hnswlib==0.7.3
It awayls build fail, full log:
Collecting chroma-hnswlib==0.7.3
  Using cached chroma-hnswlib-0.7.3.tar.gz (31 kB)
  Installing build dependencies ... done
  Getting requirements to build wheel ... done
  Preparing metadata (pyproject.toml) ... done
Requirement already satisfied: numpy in /Library/Frameworks/Python.framework/Versions/3.12/lib/python3.12/site-packages (from chroma-hnswlib==0.7.3) (2.0.0)
Building wheels for collected packages: chroma-hnswlib
  Building wheel for chroma-hnswlib (pyproject.toml) ... error
  error: subprocess-exited-with-error

  × Building wheel for chroma-hnswlib (pyproject.toml) did not run successfully.
  │ exit code: 1
  ╰─> [17 lines of output]
      running bdist_wheel
      running build
      running build_ext
      creating var
      creating var/folders
      creating var/folders/hb
      creating var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn
      creating var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn/T
      gcc -fno-strict-overflow -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -O3 -Wall -arch arm64 -arch x86_64 -g -I/Library/Frameworks/Python.framework/Versions/3.12/include/python3.12 -c /var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn/T/tmp6qa9i5s7.cpp -o var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn/T/tmp6qa9i5s7.o -std=c++14
      gcc -fno-strict-overflow -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -O3 -Wall -arch arm64 -arch x86_64 -g -I/Library/Frameworks/Python.framework/Versions/3.12/include/python3.12 -c /var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn/T/tmpe3eznqbv.cpp -o var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn/T/tmpe3eznqbv.o -fvisibility=hidden
      building 'hnswlib' extension
      creating build
      creating build/temp.macosx-10.9-universal2-cpython-312
      creating build/temp.macosx-10.9-universal2-cpython-312/python_bindings
      gcc -fno-strict-overflow -Wsign-compare -Wunreachable-code -fno-common -dynamic -DNDEBUG -g -O3 -Wall -arch arm64 -arch x86_64 -g -I/private/var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn/T/pip-build-env-l9ct_g4l/overlay/lib/python3.12/site-packages/pybind11/include -I/private/var/folders/hb/5bz0nsy56f5f70xx5s2qld600000gn/T/pip-build-env-l9ct_g4l/overlay/lib/python3.12/site-packages/numpy/_core/include -I./hnswlib/ -I/Library/Frameworks/Python.framework/Versions/3.12/include/python3.12 -c ./python_bindings/bindings.cpp -o build/temp.macosx-10.9-universal2-cpython-312/./python_bindings/bindings.o -O3 -march=native -stdlib=libc++ -mmacosx-version-min=10.7 -DVERSION_INFO=\"0.7.3\" -std=c++14 -fvisibility=hidden
      clang: error: unsupported argument 'native' to option '-march='
      error: command '/usr/bin/gcc' failed with exit code 1
      [end of output]

  note: This error originates from a subprocess, and is likely not a problem with pip.
  ERROR: Failed building wheel for chroma-hnswlib
Failed to build chroma-hnswlib
ERROR: ERROR: Failed to build installable wheels for some pyproject.toml based projects (chroma-hnswlib)

So, clang or gcc counld not compilable on my computer.
clang version: Apple clang version 15.0.0 (clang-15<IP_ADDRESS>)


"More results"

Is your feature request related to a problem? Please describe.
So I set my maxResults to 10 and obviously the 11th item disappeared.
Thoroughly Describe the solution you'd like
I would like to know if there's a feature yet to add a more results button on the resultsList; which adds some kind of pagination to go to the 11th item and make the 10 items dissapear for the viewport.
Please provide a few use cases for this feature

... Responsiveness
... No more lost results

I will work on this

Here we go, since autoComplete.js 9.1.0 introduced const {input, query, matches, results} = data to resultsList.container, this is now easy to accomplish:
https://user-images.githubusercontent.com/233791/116778567-26d5e100-aa62-11eb-8ec7-f5c9a1c66acf.mp4
Some of the relevant code below. This can't easily be copied and pasted, you need to understand the code. Also I only wrote this as an example, I do not plan to use this code in my app so it is not well tested or complete.

var currentResults
var acMax = 8
var acCurrent = 1

function redrawAutocompleteResults(container) {
	let add = 0
	for (let c of container.children) {
		if (c instanceof HTMLLIElement && c.classList.contains("autoComplete_result")) {
			bedriftAutocompleteContent(currentResults[acCurrent + add], c)
			add = add + 1
		}
	}
	$("autoComplete_stats").innerHTML = acCurrent + "-" + (acCurrent + acMax - 1) + " / " + currentResults.length
}

function autocompletePrev(ev, container) {
	if ( acCurrent > 1 ) {
		acCurrent = acCurrent - acMax
		redrawAutocompleteResults(container)
	}
	ev.stopPropagation()
}

function autocompleteNext(ev, container) {
	if ( (acCurrent + acMax) < currentResults.length ) {
		acCurrent = acCurrent + acMax
		redrawAutocompleteResults(container)
	}
	ev.stopPropagation()
}

function lagAutocompleteHjelp(element, tekst) {
	const hjelp = document.createElement("p")
	hjelp.classList.add("autoComplete_info", "small")
	hjelp.innerHTML = `<span class="fa fa-question-circle"></span>${tekst}`
	hjelp.addEventListener("click", ev => {
		ev.stopPropagation()
	})
	element.prepend(hjelp)

	acCurrent = 1
	acMax = 8

	const pag = document.createElement("div")
	pag.classList.add("d-flex", "justify-content-between")
	pag.onclick = (ev) => { ev.stopPropagation() }

	const prev = document.createElement("span")
	prev.classList.add("ff", "fa-arrow-left", "p-1")
	prev.onclick = function(ev) { autocompletePrev(ev, element) }
	pag.append(prev)

	const stats = document.createElement("p")
	stats.classList.add("text-muted", "small", "p-1", "m-0")
	stats.setAttribute("id", "autoComplete_stats")
	stats.innerHTML = "1-8 / " + currentResults.length
	pag.append(stats)

	const next = document.createElement("span")
	next.classList.add("ff", "fa-arrow-right", "p-1")
	next.onclick = function(ev) { autocompleteNext(ev, element) }
	pag.append(next)

	element.append(pag)
}

autoComplete = {
	resultsList: {
		container: (element, data) => {
		currentResults = data.matches
			lagAutocompleteHjelp(
				element,
				"Du må velge en bedrift fra denne listen. Det er ikke nok å skrive inn navnet eller organisasjonsnummeret."
			)
		},
		maxResults: 8,
	}
}

``

I see now there is a bug because i do not replace the click event on the li element, so the autocomplete still calls onSelection with the data from the initial list when you click them. This is easy to fix - there are at least 3 ways of fixing it that I can think of off the top of my head. Since I will not use this code myself I can't be bothered to explain further unless someone actually asks.


2.0.0-next.3e165448 "No parser or filepath given, using 'babylon'..."

Is this a bug report?
This is sort of a bug.

Did you try recovering your dependencies?
Yes, removed node_modules and my yarn.lock and did a fresh install and build, same result

(Write your answer here.)
Which terms did you search for in User Guide?
babylon, no parser or filepath given

(Write your answer here if relevant.)
Environment

Environment:
OS:  OS X El Capitan 10.11.6
Node:  10.3.0
Yarn:  1.7.0
npm:  6.1.0
Watchman:  4.9.0
Xcode:  Xcode 8.2.1 Build version 8C1002
Android Studio:  Not Found
Packages: (wanted => installed)
react: ^16.4.0 => 16.4.1
react-dom: ^16.4.0 => 16.4.1
react-scripts: 2.0.0-next.3e165448 => 2.0.0-next.3e165448
Steps to Reproduce

Upgrade to<EMAIL_ADDRESS>Run yarn build which in my case translates to NODE_PATH=src react-scripts build

Expected Behavior

No deprecation errors to be thrown as this is the next version and should be up-to-date
Actual Behavior

Deprecation errors appeared
Reproducible Demo

if this is not something that I am doing wrong with going from the previous @next version to the latest, I will try and make a repro.
I think this might be an issue with prettier, they removed the default parser and made the setting empty be default, and if it is empty it will throw that warning:
https://prettier.io/blog/2018/05/27/1.13.0.html#don-t-default-to-the-javascript-parser-4528-by-duailibe
However, not sure why prettier is running on react-scripts build, I tried adding "parser": "babylon" to my "prettier" config in package.json but no luck.

closing due to 2.0 release not having this issue


WindowsMachineFileCopyV2 fails in a pipeline when using DFS

Required Information
Bug
Type: YAML
Enter Task Name: WindowsMachineFileCopyV2
list here (V# not needed):
https://github.com/Microsoft/azure-pipelines-tasks/tree/master/Tasks
Environment

Server - TFS on-premises

If using TFS on-premises, provide the version: 2020.1

Agent - Hosted or Private:

If using Hosted agent, provide agent queue name: default (shared pool)

Issue Description
The pipeline runs under user1, the task uses user2 to copy files to a protected directory.
    - task: WindowsMachineFileCopy@2
      displayName: 'Copy the installer to file share'
      inputs:
        SourcePath: 'unpacked'
        MachineNames: 'mycompany.net'
        AdminUserName: '$(CopyUser)'
        AdminPassword: '$(CopyPassword)'
        TargetPath: '$(CopyPath)'
        CleanTargetBeforeCopy: false
        CopyFilesInParallel: false

When the directory is given as a direct server path, the task works. When given as a DFS namespace, fails.
Task logs
##[error]Failed to Create PSDrive with Destination: '\\mycompany.net\...\Applications', ErrorMessage: 'Multiple connections to a server or shared resource by the same user, using more than one user name, are not allowed. Disconnect all previous connections to the server or shared resource and try again'

Can you please use the IP address of the remote server when you try to connect to the network share?

Since long time we didn't get any response. We are temporarily closing this issue now. Please feel free to reopen the issue if you are still facing the issue.


upgrade dependencies to be ready for babel 7

This upgrades the plugin to be ready for babel 7.
Fixes #21
This branch is released on npm. Install via:
npm i -D @axe312/babel-plugin-inline-import@beta

I added you as contributor of this repo. What is your npm username?

Ohh wow nice, thank you.
My npm username is https://www.npmjs.com/~axe312

We should make sure to release this as v4.0.0 since its a breaking change!

it sounds right. could you take care of publishing the new version for us? :)

@zvictor @axe312ger can one of you publish 4.0.0 thanks

@zvictor @axe312ger can one of you publish 4.0.0 thanks


[ns2] Unable to load game_setup.xml on centos 7

Not sure if this is the appropriate place for this. There is a thread on unknown worlds forum but it wasn't very helpful: http://forums.unknownworlds.com/discussion/136479/unable-to-load-game-setup-xml
I'm running this in a VM with CentOS 7 installed. Here is my console output:
GetInstanceCount currently unimplemented
[  0.000] Unbound/Unknown : Error: Invalid command line ()
[  0.006] Main : Filesystem initialized, cache enabled
Press Tab to open the console window
Press Ctrl-C to exit
Build 279
Linux
Server tickrate 30, client sendrate 20, bandwidth limit per player 51200
[S_API FAIL] SteamAPI_Init() failed; SteamAPI_IsSteamRunning() failed.
Setting breakpad minidump AppID = 4920
Connected to Steam servers
Web server running at XXX.XXX.XXX.XXX:8080
[  1.662] Worker 00 : Error: Unable to open 'game_setup.xml' (usage 0x1)
[  1.662] Worker 00 : Error: Unable to load 'game_setup.xml'!
VAC Enabled

The web admin server responds with a page that says "Server loading..."
I can find the server in the games list, but it I can't fully join the server. It stays "Connecting..." for a while and eventually times out.

Hi there. I'm sorry for your issue. Seems like you're not the only one experiencing this, either windows and linux. I can guess that it's not a script issue and that you should try to address this to game devs.

Btw, "invalid command line" makes me think it could be a start option issue. Could you please copy your main script's parms="" line ?
Maybe they changed some start options. Do you have a changelog of their last update (or preferably their last server update).

Maybe try with no parms and add them one by one to hopefully get one that's causing issues ?
Could possibly be a quote symbol issue as i got some.

Maybe try with no parms and add them one by one to hopefully get one that's causing issues ?
Could possibly be a quote symbol issue as i got some.

Roland, I tried your runline exactly - substituting my IP address.
Having an empty mods field gave me "Error: Invalid command line ()" with other errors. Please remove it  for now and post your output. Mine started successfully without it.
./server_linux32 -name "NS2 Server" -port 27015 -webadmin -webdomain <IP_ADDRESS> -webuser admin -webpassword "admin" -webport 8080 -map ns2_summit -limit 18 -config_path "/home/ns2server/server1" -modstorage "/home/ns2server/server1/Workshop"

Is this server available for free? I can try to reproduce it, I have CentOS 6 and 7 boxes here.

Hosting is free, the instructions are under the "Linux Setup" section of: http://wiki.unknownworlds.com/ns2/Dedicated_Server

Just tried installing, seems like you need a steam account owning this game to get it working.

Sorry, I'll clarify - any Steam account will work, it does not have to own the game for the dedicted server.

Thanks.
So i just tried under Debian 8.2 which is my server's distro, and got it working without an issue.
Here is my ns2server config :
Start Variables
defaultmap="ns2_summit"
port="27150"
maxplayers="24"
ip="<IP_ADDRESS>"
updateonstart="off"
servername="NS2 Server"
webadminuser="admin"
webadminpass="admin"
webadminport="8010"
configpath="server1"
modstorage="server1/Workshop"
mods=""
password=""
Add the following line to the parms if you want a private server. Ensuring
that the password variable above is not left empty.
-password "${password}"
Do you have anything significatively different compared to that ?
Maybe your port 8080 is already used ? That's why i changed mine, i thought it was already in use (but it's actually not).

The OP seemed to have trouble with CentOS. Debian and Ubuntu seem to work off the bat.

The OP isn't responding anymore btw ://
Seems like you found the solution : remove the empty -mods ""
Anything else to add @Turbine1991 ?

I'm satisfied. :)

Thanks for your help mate :)

Sorry for the delayed response. I tried removing mods and it did not resolve the issue unfortunately. No need to re-open the bug. I'll try to find help from the ns2 community since this is not a problem with this project.
BTW, removing mods gave me the same output either way:
$ ./server_linux32 -name "NS2 Server" -port 27015 -webadmin -webdomain <IP_ADDRESS> -webuser admin -webpassword "admin" -webport 8080 -map ns2_summit -limit 18 -config_path "/home/ns2server/server1" -modstorage "/home/ns2server/server1/Workshop"
GetInstanceCount currently unimplemented
[  0.000] Main : Filesystem initialized, cache enabled
Press Tab to open the console window
Press Ctrl-C to exit
Build 279
Linux
Server tickrate 30, client sendrate 20, bandwidth limit per player 51200
[S_API FAIL] SteamAPI_Init() failed; SteamAPI_IsSteamRunning() failed.
Setting breakpad minidump AppID = 4920
Connected to Steam servers
Web server running at <IP_ADDRESS>:8080
[  1.819] Worker 00 : Error: Unable to open 'game_setup.xml' (usage 0x1)
[  1.819] Worker 00 : Error: Unable to load 'game_setup.xml'!
VAC Enabled

Thought you left forever :p
I'm interested in finding a solution as it could affect all CentOS users.
We could include a fix for that if you can figure out what's causing that issue.
Please, keep us informed !

Are you using selinux? Run the following command:
getenforce
Also, do
ls -lZ game_setup.xml
and that'll give us the SELinux context. And, you've got the files owned by the game server user, and permissions are OK? If you want to create an account for me on the box, I'd be glad to pop in and take a peek.

Closing for now, please keep us informed if anything relevant to fix.
Otherwise, i advise you to use the steam forum for general support : https://steamcommunity.com/groups/linuxgsm/discussions

I know this post was closed; but I am experiencing the same issue.
Picking up with the information that @jaredballou requested:
[ns2server@ns2 ns2]$ ls -lZ game_setup.xml
-rwxrwxr-x. ns2server ns2server unconfined_u:object_r:user_home_t:s0 game_setup.xml
Additional Information:
[ns2server@ns2 ns2]$ cat /etc/redhat-release
CentOS Linux release 7.3.1611 (Core)
Firewall dump:
[root@ns2 ~]# firewall-cmd --list-ports
27015/tcp 27015/udp 27016/udp 8080/tcp 8081/tcp 27016/tcp
[ns2server@ns2 ns2]$ sestatus
SELinux status:                 disabled
Running the command directly using:
./server_linux32 -name " Awesome NS2 Server" -port 27015 -webadmin -webdomain "<IP_ADDRESS>" -webuser ##admin## -webpassword ##password## -webport 8080 -map ns2_summit -limit 16
Documentation: https://wiki.unknownworlds.com/ns2/Dedicated_Server#CentOS

Does the server appear to connect to? It may not be an error, more a warning.

@Turbine1991 negative, I cannot see the server in the game server listing.
The server status page still displays "Loading server..." after 10 minutes.
Did a complete reinstall of the OS and followed: https://gameservermanagers.com/lgsm/ns2server/
Same result.

What's your runline?

@Turbine1991 by runline do you mean what command do I use to start the server?
After installing the script, I use the default command: ./ns2server start
executed from: /home/ns2server

@JLATORRE89 I forgot this is for a server manager. You should try set it up natively and run it, just to rule out whether it's a manager issue or a system issue.
Although I do run NS2 on Ubuntu.  Are you stuck to using this OS?

@Turbine1991 I pretty much went with CentOS 7 because it is based on RHEL and is fairly stable.
I spun up another VM that is running Ubuntu 16, used the server manager; I can now see the dashboard as expected.
A key difference I noticed was fixes were not applied during the CentOS 7 install.
The manual install for CentOS 7 did not work either. I will make a note on the NS2 forum.
Ubuntu start:
[ WARN ] Starting ns2-server: Configuration file missing!
/home/ns2server/server1
Downloading Natural Selection 2 Config
default configs from https://github.com/GameServerManagers/Game-Server-Configs
fetching check_ip.sh...OK
fetching info_config.sh...OK
fetching check_status.sh...OK
fetching fix_steamcmd.sh...OK
[ .... ] Fix ns2-server: Applying steamclient.so general fix: Natural Selection [ INFO ] Fix ns2-server: Applying steamclient.so general fix: Natural Selection [  OK  ] Fix ns2-server: Applying steamclient.so general fix: Natural Selection 2
fetching logs.sh...OK
[  OK  ] Starting ns2-server: Zuper Rookie Server
Thanks for all the help! =]

I wouldn't call small game servers mission critical, for NS2 I'd think performance is as important as stability. I've never had it crash/lock up on me since they fixed the crashes a long time ago.


Implement DNS collector

Description
Implemented DNS collector. DNS collector makes a DNS query (A record) to a hostname and records the time to finish the query. Multiple nameservers and hostnames can be configured.
Closes #15
Checklist

[x] Changelog has been updated.
[x] Unit tests have been updated.
[x] Related documentation in wiki has been updated.

@adarshk7, I'm merging this as well. I ordered Cargo.toml alphabetically (or VSCode did that automatically), so that might cause some conflicts in your PR.


Poor user experience installing Rust on Windows 7

See https://github.com/rust-lang/rust/issues/40054

I'm not a Windows guy, but needed to use it for a particular project and literally ran into this. exact. issue. Even if rustup couldn't do all the downloading and installing automagically, it could at least suggest those three tools in a readable fashion with links as an interim solution.

This is because the MSVC build of rustup presently dynamically links to CRT. The next release will fix that.
Still, I am surprised both of you ran into this. The main installation flow directs all Windows users to the -gnue builds (which work the exact same as the -msvc) builds, and those should not suffer this problem.

Could you clarify whether you were using the -msvc build of rustup or the -gnu build? Both builds of rustup are capable of installing both -msvc and -gnu versions of the rust toolchain, which is why rustup.rs always links to the -gnu version of rustup.
Currently the only requirement should be on the VC++ build tools, and that should only be required if you install the -msvc toolchain. MSVCR120.dll should not be required unless you explicitly chose "additional download options" and downloaded the -msvc version of rustup itself, which is currently not recommended, and provides no additional benefits.

I'm sorry but I can't remember now.

I, too, get the "missing MSVCR120.dll" error when running rustup-init.exe. I am definitely using the -gnu build of rustup-init.exe.
However, the error only pops up at the tail end of rustup-init -- rustc, rust-std, and cargo get successfully installed before the error.
It seems like rustup-init, at the very end, is trying to run "rustc --version" (so that it can print a success message), and that attempt to launch rustc is what triggers the error dialog.

It sounds like rustc itself may have an unexpected dependency on the MSVCRT which is not satisfied by the VC++ build tools?
@retep998 Do you know if this is the case? If so we can add whatever other dependencies rustc has to the instructions at install time.

I'm pretty sure the latest rustup rustc and cargo all statically link the CRT now so there shouldn't be any dependencies on VC++ redistributables.

@retep998 Thanks!
Please reopen if the issue re-occurs.

Rust 1.17.0 doesn't seem to have improved the Windows user installation experience. I downloaded and ran the rustup-init.exe and it gave me a console with a big red warning that I needed to install C++ build tools before proceeding. For me one of the whole points of using Rust is so I don't depend on the C/C++ ecosystem. Anyway, it gave a link to the needed build tools and I got their installer and that didn't work saying I needed .Net 4.5.1 or some such and I decided not to go round that loop again. (I do have Mingw64 installed but have no idea how to tell the Rust installer about it. I tried the 'custom' install option but when it asked for the 'default host triple' I could guess it wanted x86_64-pc-windows- but not what the end should be, so once again I've given up for now. Windows users generally expect things to just work: compare with Go's installer which is easy and fast and works.


Subject and AccessControlContext in spec/tck

Add spec and tck tests for the use of Subject and AccessControlContext inside the ThreadContext.SECURITY bucket.
@darranl can fill in the gaps where I've been vague

This could be related to https://us06st1.zoom.us/web_client/6ltmb5/html/externalLinkPage.html?ref=https://github.com/smallrye/smallrye-context-propagation/pull/260


element.remove() function is not working in Internet explorer

What part of Carbon does this bug correlate to?
General graph logic:

[x]  Graph attributes (axes, legends, regions...etc.)
[ ]  Other (please specify):

Describe the bug

When we try to destroy one of the tooltip div element in Carbon-graphs, we are getting error like "Object does not support property or method called remove" because IE does not support remove() function for DOM elements.

Expected behavior

div element should be removed during destroy without any errors in all browsers.

Screenshots

Desktop (please complete the following information):

Browser [internet explorer]

:tada: This issue has been resolved in version 2.14.1 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Type inference error for Enum

TypeScript Version:  2.8.0-dev.20180221
Search Terms:
enum type inference
Code
const enum Enum1 {a, b};

let e1: Enum1 = Enum1.a;

for (let i = 0; i < 10; i++) {
   const newE1 = (i % 2 == 0) ? Enum1.a : Enum1.b;
   setE1(newE1);
   switch (e1) {
      case Enum1.a:
         console.log("a");
         break;
      case Enum1.b:
         console.log("b");
         break;
   }
}

function setE1 (newE1: Enum1) {
   e1 = newE1;
}

Actual behavior:
Compiler message: "error TS2678: Type 'Enum1.b' is not comparable to type 'Enum1.a'."
Playground Link
Related Issue:
Stack Overflow entry

Simplification of example: An assignment done in a function call is not taken into account for code flow type inference:
const enum Enum1 {a, b};
let e1: Enum1 = Enum1.a;

setE1(Enum1.b);
// Error === cannot be applied to Enum1.a and Enum1.b
if (e1 === Enum1.b) console.log('yup'); // But it is

function setE1(newE1: Enum1) {
   e1 = newE1;
}

Duplicate of #9998

I understand that this topic was discussed in #9998. But it would probably be better to disable or relax type inference for enums. It causes more harm than good. It's sufficient to check whether the involved enum constants and variables are part of the same enum type.


feat: some trivial modifications

support special header format for Visual Fox Pro
support defferred data loading since some dbf is to large to load quickly
add some helper functions

Tag me when you're done @TenzT


Need to add "reasonable" default Decider to filter out hard failures from soft ones

The default decider implementation should throw when this gets raised currently - but shouldn't we just ignore this error by default? Because there's instances where a user can request partitions / topics that haven't been created yet.
Originally posted by @Aaronontheweb in https://github.com/akkadotnet/Akka.Streams.Kafka/pull/246#discussion_r749425902

We should not blow up Akka.Streams.Kafka graphs when "normal" errors occur:

Idle connections being reaped by broker;
Requesting topics / partitions that haven't been created yet;
What else from this list? https://github.com/akkadotnet/Akka.Streams.Kafka/issues/184#issuecomment-963468130

Serialization errors (requires some special handling - end users are going to need to decide what to do; skip message or crash. Need documentation on how to do both.)

This needs to be documented before we can release 1.2.0. Need to explain how to commit past a poisoned message on the consumer or how to let the stream blow up and be restarted via supervision.

@Arkatufus would a better way of doing this be: when should we blow up vs. when shouldn't we? Looks like most of these errors are informational at best.

@Aaronontheweb looks like the exception list in the Java API reference is a better source for research, they actually explain what the error code ment and what caused them

@Arkatufus let's reference it in this issue then.


DBR Optimizer Rule ignored

When running with the underlying lower and higher uuid longs  in the projection the optimiser rule is run with correct pushdown (also on Photon), so
select * from (select lower, higher, as_uuid(lower,higher) context from thetable) where context = ''

will optimise
select * from (select as_uuid(lower,higher) context from thetable) where context = ''

will not

This seems an oddity of DBR that the predicates trigger use of the rule, instead the issue seems to be with .show / display - oss show uses the logical plan not an optimized one, assumption is display works the same way.  If so the notebook performance results cannot be trusted and a collect or toLocalIterator.next must be used to measure against.

Thanks to help from Sandeep @ Databricks it seems a limit is added for displays which stopped the lineage tracking, general solution added and verified on dbr 12.2


Replace usages of deprecated pointer functions

How to categorize this PR?

/area open-source
/kind enhancement
What this PR does / why we need it:
This PR replaces the usages of deprecated pointer functions.
Which issue(s) this PR fixes:
Fixes #
Special notes for your reviewer:
Release note:

Utility functions `QuantityPtr`,`ProtocolPtr`,`TimePtr` and `TimePtrDeref`, `extensionsv1alpha1.UnitCommandPtr` and `ValueExists` are dropped. Use `k8s.io/utils/ptr.To`, `k8s.io/utils/ptr.Deref` and `slices.Contains` instead.

/retest

Awesome, thanks for cleaning this up. There is no single k8s.io/utils/pointer left 🚀
I found two additional custom pointer function you could delete too:

https://github.com/shafeeqes/gardener/blob/aa57f82c90831e30d797571dd9634a78f985d39c/pkg/utils/kubernetes/highavailability_test.go#L90-L92
https://github.com/shafeeqes/gardener/blob/aa57f82c90831e30d797571dd9634a78f985d39c/pkg/apis/extensions/v1alpha1/types_operatingsystemconfig.go#L141-L144

I found two additional custom pointer function you could delete too:

https://github.com/shafeeqes/gardener/blob/aa57f82c90831e30d797571dd9634a78f985d39c/pkg/utils/kubernetes/highavailability_test.go#L90-L92

I considered this, but then we have to use ptr.To(gardencorev1beta1.FailureToleranceType(<v>)) everywhere, which isn't any better. But for consistency reasons, I can do it.

https://github.com/shafeeqes/gardener/blob/aa57f82c90831e30d797571dd9634a78f985d39c/pkg/apis/extensions/v1alpha1/types_operatingsystemconfig.go#L141-L144

I considered this as well, but I saw that the extensions were using this and maybe let's keep it. I removed it and added to the release note.

Thanks 😄
/lgtm

/unhold
/lgtm


Drop Python 2

I think it's time.

+1 from me
On Fri, Apr 26, 2019 at 11:23 AM Tom Augspurger<EMAIL_ADDRESS>wrote:

I think it's time.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/dask/dask-ml/issues/499, or mute the thread
https://github.com/notifications/unsubscribe-auth/AACKZTEEREEA2MGG3BW5BULPSMT6VANCNFSM4HIW7CBA
.


[#188] Implemented notification for new group members

Extended MemberController::store() to trigger notification on user
Created BecameNewMember notification to send email to user with proper
message
Extended MemberTest to test that notification has been send

Codecov Report

Merging #196 into dev will increase coverage by 0.77%.
The diff coverage is 86.66%.

@@             Coverage Diff              @@
##                dev     #196      +/-   ##
============================================
+ Coverage     64.46%   65.23%   +0.77%
- Complexity        8       16       +8
============================================
  Files            71       72       +1
  Lines           636      653      +17
============================================
+ Hits            410      426      +16
- Misses          226      227       +1

Impacted Files
Coverage Δ
Complexity Δ

app/Http/Controllers/MemberController.php
100% <100%> (+4.54%)
4 <0> (+4)
:arrow_up:

app/Notifications/BecameNewMember.php
85.71% <85.71%> (ø)
4 <4> (?)

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 d75e693...62b4784. Read the comment docs.

Hey, so laravel notification will be used to send mail, but for frontend notification we're using spatie activitylog package so in the listener you need to log to it, hope it make sense :)

Ok, I might misunderstood the task here - was this about sending user an email notification about them being added to a team or to create new notification that's displayed on front in upper right corner?
If later then I guess the whole solution is wrong and need to be redone.

Well it was about showing in page but you reminded we also need to send mail so its ok. But now problem is if we use laravel default notification for both mail and frontend, can we choose in certain situation where we need only frontend notification and not mail, can it be done with default notification?

Take a look at this manual entry regarding Notifications - https://laravel.com/docs/5.7/notifications#specifying-delivery-channels
Based on that, you should be able to use single notification class to send notification via multiple channels - in this case I think you might be interested in mail and broadcast. I don't know whether you have or plan to implement frontend notifications via web sockets (using pusher or whatever else), but if you do, broadcast is the way how to do it. There's nice series about Laravel Echo - how it works and how to implement it relatively easily via pusher. I can look it up for you if you're interested.
Another option is to stay with event system - you can still define event and assign to it as many listeners as you want, so this solution might be reworked in a way that instead of calling $user->notify() you can simply dispatch event and set up listener for it to send email and another one to push frontend notification as well.
Let me know what's your opinion on this I don't mind extending/reworking the solution I developed here.

well that via delivery channel work by using user data but use case little different, for example I want to send mail & frontend notification when they are added but send only frontend notification when they are removed. that's why I'm not sure if default system will solve this problem

I don't see a problem there, you'll just need to implement new notification like RemovedFromMembers that will have it's own setting of delivery channels to send notification through, in this case most likely only broadcast

Oh yes, you're right, misunderstood it completely.

Ok so if we're settled on that, let's get back to this issue. What needs to be done in order to close it?


Better detection on Safari

Hello,
Using localStorage on Safari on iOS will result in JS errors when in private mode:
http://stackoverflow.com/questions/14555347/html5-localstorage-error-with-safari-quota-exceeded-err-dom-exception-22-an
http://stackoverflow.com/questions/13208878/localstorage-not-accessible-in-ios-6-safari
Would you please consider a better detection of localStorage actually working, and if not, handle as if not available instead of erroring?
e.g. simpleStorage.get() could return undefined
Thanks

Can you please post the error you get?


Submission doesn't appear in the Leaderboard

After clicking "Publish to Leaderboard", our submission is not shown in the Leaderboard even after nearly 1 hour. Here is our submission in the 2D tracking<EMAIL_ADDRESS>What could be the reason?

ok now it appears :) sorry for the inconvience.


Add flow-cli dev super command

Integrate flow-cli dev supper command with vscode

flow dev: Automatically watch any changes you make to Cadence files and make sure to continuously sync those changes on the emulator network

Enable the extension to watch cadence files and automatically keep the emulator updated
Super command readme:
https://github.com/sideninja/flow-basic-scaffold/blob/main/README.md
Later we can add flow exec: Send transactions or scripts easily and automatically search for that file in scripts and contracts and will execute it for you
Note: flow setup should be run to setup the project, probably before opening it in VSCode.

Decided not to integrate commands in vscode to keep the extension more lightweight.


Fix map field type error when value type is defined after the map

This fixes bug #181 and adds tests for the bug. With this change, now
NIL can be inserted into maps that have message values. This is a bug,
but it is the same behavior as any optional message field.

Please don't make descriptions like: Fix bug #181 #186
I have no idea what bug 181 is, instead say what it's doing and then link to the bug
in the description.

Please don't make titles like: Fix bug #181 #186
I have no idea what bug 181 is, instead say what it's doing and then link to the bug
in the description.

Edited it. The bug is a bit of a mouthful, hence why I linked it

Should this also update the json code?
Or is that in another cl?

Should this also update the json code?
Or is that in another cl?

What would change in the JSON code?

You commented out several well known types.

I see, this is cl-protobufs should import struct.proto #182
nvm

I see, this is cl-protobufs should import struct.proto #182
nvm
For some reason I thought this and that were related...

Oh right, I understand. That's next :)

merged as of https://github.com/qitab/cl-protobufs/commit/27d33c63b1516662e43e633fad15728d2034bebd


Add routing namespace descriptions

Addresses https://github.com/dotnet/aspnetcore/issues/27286

Docs Build status updates of commit 7aa231e:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Constraints.xml
:x:Error

Details

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Patterns.xml
:x:Error

Details

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Template.xml
:x:Error

Details

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Tree.xml
:x:Error

Details

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.xml
:x:Error

Details

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Constraints.xml

Line 0, Column 0: [Error-ECMA2Yaml_InternalError]

Intenal Several Error: System.Xml.XmlException: 'xref' is an undeclared prefix. Line 3, position 287.
   at System.Xml.XmlTextReaderImpl.Throw(Exception e)
   at System.Xml.XmlTextReaderImpl.LookupNamespace(NodeData node)
   at System.Xml.XmlTextReaderImpl.ElementNamespaceLookup()
   at System.Xml.XmlTextReaderImpl.ParseElement()
   at System.Xml.XmlTextReaderImpl.ParseElementContent()
   at System.Xml.Linq.XContainer.ReadContentFrom(XmlReader r)
   at System.Xml.Linq.XDocument.Load(XmlReader reader, LoadOptions options)
   at System.Xml.Linq.XDocument.Parse(String text, LoadOptions options)
   at ECMA2Yaml.ECMALoader.LoadType(FileItem typeFile)
   at ECMA2Yaml.ECMALoader.LoadTypes(String basePath, Namespace ns)

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Patterns.xml

Line 0, Column 0: [Error-ECMA2Yaml_InternalError]

Intenal Several Error: System.Xml.XmlException: 'xref' is an undeclared prefix. Line 3, position 58.
   at System.Xml.XmlTextReaderImpl.Throw(Exception e)
   at System.Xml.XmlTextReaderImpl.LookupNamespace(NodeData node)
   at System.Xml.XmlTextReaderImpl.ElementNamespaceLookup()
   at System.Xml.XmlTextReaderImpl.ParseElement()
   at System.Xml.XmlTextReaderImpl.ParseElementContent()
   at System.Xml.Linq.XContainer.ReadContentFrom(XmlReader r)
   at System.Xml.Linq.XDocument.Load(XmlReader reader, LoadOptions options)
   at System.Xml.Linq.XDocument.Parse(String text, LoadOptions options)
   at ECMA2Yaml.ECMALoader.LoadType(FileItem typeFile)
   at ECMA2Yaml.ECMALoader.LoadTypes(String basePath, Namespace ns)

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Template.xml

Line 0, Column 0: [Error-ECMA2Yaml_InternalError]

Intenal Several Error: System.Xml.XmlException: 'xref' is an undeclared prefix. Line 3, position 58.
   at System.Xml.XmlTextReaderImpl.Throw(Exception e)
   at System.Xml.XmlTextReaderImpl.LookupNamespace(NodeData node)
   at System.Xml.XmlTextReaderImpl.ElementNamespaceLookup()
   at System.Xml.XmlTextReaderImpl.ParseElement()
   at System.Xml.XmlTextReaderImpl.ParseElementContent()
   at System.Xml.Linq.XContainer.ReadContentFrom(XmlReader r)
   at System.Xml.Linq.XDocument.Load(XmlReader reader, LoadOptions options)
   at System.Xml.Linq.XDocument.Parse(String text, LoadOptions options)
   at ECMA2Yaml.ECMALoader.LoadType(FileItem typeFile)
   at ECMA2Yaml.ECMALoader.LoadTypes(String basePath, Namespace ns)

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Tree.xml

Line 0, Column 0: [Error-ECMA2Yaml_InternalError]

Intenal Several Error: System.Xml.XmlException: 'xref' is an undeclared prefix. Line 3, position 62.
   at System.Xml.XmlTextReaderImpl.Throw(Exception e)
   at System.Xml.XmlTextReaderImpl.LookupNamespace(NodeData node)
   at System.Xml.XmlTextReaderImpl.ElementNamespaceLookup()
   at System.Xml.XmlTextReaderImpl.ParseElement()
   at System.Xml.XmlTextReaderImpl.ParseElementContent()
   at System.Xml.Linq.XContainer.ReadContentFrom(XmlReader r)
   at System.Xml.Linq.XDocument.Load(XmlReader reader, LoadOptions options)
   at System.Xml.Linq.XDocument.Parse(String text, LoadOptions options)
   at ECMA2Yaml.ECMALoader.LoadType(FileItem typeFile)
   at ECMA2Yaml.ECMALoader.LoadTypes(String basePath, Namespace ns)

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.xml

Line 0, Column 0: [Error-ECMA2Yaml_InternalError]

Intenal Several Error: System.Xml.XmlException: 'xref' is an undeclared prefix. Line 3, position 244.
   at System.Xml.XmlTextReaderImpl.Throw(Exception e)
   at System.Xml.XmlTextReaderImpl.LookupNamespace(NodeData node)
   at System.Xml.XmlTextReaderImpl.ElementNamespaceLookup()
   at System.Xml.XmlTextReaderImpl.ParseElement()
   at System.Xml.XmlTextReaderImpl.ParseElementContent()
   at System.Xml.Linq.XContainer.ReadContentFrom(XmlReader r)
   at System.Xml.Linq.XDocument.Load(XmlReader reader, LoadOptions options)
   at System.Xml.Linq.XDocument.Parse(String text, LoadOptions options)
   at ECMA2Yaml.ECMALoader.LoadType(FileItem typeFile)
   at ECMA2Yaml.ECMALoader.LoadTypes(String basePath, Namespace ns)

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 5db2c1d:
:x: Validation status: errors
Please follow instructions here which may help to resolve issue.

File
Status
Preview URL
Details

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.xml
:x:Error

Details

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.xml

Line 0, Column 0: [Error-ECMA2Yaml_InternalError]

Intenal Several Error: System.Xml.XmlException: The 'see' start tag on line 3 position 244 does not match the end tag of 'summary'. Line 3, position 351.
   at System.Xml.XmlTextReaderImpl.Throw(Exception e)
   at System.Xml.XmlTextReaderImpl.Throw(String res, String[] args)
   at System.Xml.XmlTextReaderImpl.ThrowTagMismatch(NodeData startTag)
   at System.Xml.XmlTextReaderImpl.ParseEndElement()
   at System.Xml.XmlTextReaderImpl.ParseElementContent()
   at System.Xml.XmlTextReaderImpl.Read()
   at System.Xml.Linq.XContainer.ReadContentFrom(XmlReader r)
   at System.Xml.Linq.XContainer.ReadContentFrom(XmlReader r, LoadOptions o)
   at System.Xml.Linq.XDocument.Load(XmlReader reader, LoadOptions options)
   at System.Xml.Linq.XDocument.Parse(String text, LoadOptions options)
   at ECMA2Yaml.ECMALoader.LoadType(FileItem typeFile)
   at ECMA2Yaml.ECMALoader.LoadTypes(String basePath, Namespace ns)

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit ec70d12:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Constraints.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Matching.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Patterns.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Template.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Tree.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.xml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit e6e5e06:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Constraints.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Matching.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Patterns.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Template.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Tree.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.xml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel

Docs Build status updates of commit 20ac8c1:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Constraints.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Matching.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Patterns.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Template.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.Tree.xml
:white_check_mark:Succeeded
View

aspnet-core/xml/ns-Microsoft.AspNetCore.Routing.xml
:white_check_mark:Succeeded
View

For more details, please refer to the build report.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
For any questions, please:Try searching in the Docs contributor and Admin GuideSee the frequently asked questionsPost your question in the Docs support channel


Add Lucas Cordeiro to Contributors list

@lucascmgs, I'm quite elated about your PR. I wanna evolve this project to addresses various problems faced by first-time contributors. I'd love to learn about your journey in open source community, the problems, pain points you had etc.
Could you explain how you felt when you went through the tutorial, made a PR and learned that I merged it?


Chapter 5 translated to Russian

The translation of chapter 5 has been completed.
Hello, @MKhalusova! I think I found a typo in the evaluation.mdx (line 203) I think the word "target" should be changed to "prediction".
Please let me know if I am wrong.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.

Thank you @MKhalusova !
File formatting was checked :white_check_mark:


clangd inserts internal std lib header <bits/ranges_algo.h> instead of

As the title states, clangd header insertion inserts the internal standard library header when it should instead insert the public header.
This can be replicated by completing transform in the following snippet:
#include <algorithm>
int main() {
    std::ranges::transform
}

After completion, an additional and unnecessary include for <bits/ranges_algo.h> is inserted. This file is already included via the <algorithm> header:
#if __cplusplus > 201703L
# include <bits/ranges_algo.h>
#endif

The C++ standard is set to C++20.
For completeness: the CMake project is configured with g++13.0.0 whereas the system's default is g++-12.1.1. The header paths in the attached log reflect the latter.
Logs
clangd-stderr.log
System information
Output of clangd --version: clangd version 15.0.0
Editor/LSP plugin: VSCode
Operating system: Gentoo Linux

I believe the way this works is that clangd has a mapping of implementation header to standard header.
The mapping likely needs to be updated to include new implementation headers introduced in new libstdc++ versions.

I figured there would be a mapping. Thanks for pointing me to it! I'll see if adding <bits/ranges_algo.h> works and submit a patch via Phabricator.

Patch submitted. https://reviews.llvm.org/D127184

(Reopened, the issue should only be closed once the patch merges)


Misc cleanup

* Update CSV
* Remove old manifests
* Use centos base images for init containers
* Use static compile and scratch images for Go images
* Remove references to removed components

@lulf I think we can remove also inventory for ansible from systetests package

I'll merge this now to unbreak master, and we can change the base image back to something that has the needed tools but with less size.


STEP_8_JRTB-4: added ability to send notifications about new articles.

STEP_8_JRTB-4: added ability to send notifications about new articles.
STEP_8_JRTB-8: added ability to set inactive telegram user.
STEP_8_JRTB-9: added ability to set active user and/or start using it.

PR Details
0.7.0-SNAPSHOT
JRTB-4: added ability to send notifications about new articles
JRTB-8: added ability to set inactive telegram user
JRTB-9: added ability to set active user and/or start using it.
Related Issue
Working on next issues: https://github.com/EllsaG/telegram-bot/issues/5 ,  https://github.com/EllsaG/telegram-bot/issues/9 ,    https://github.com/EllsaG/telegram-bot/issues/10
How Has This Been Tested
provided unit tests
Types of changes

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


'Accept suggestion on Enter' not working for a particular Unity class

Type: Bug
Issue
Accepting a suggestion, when inserting function calls or variables, using any form of input (e.g. enter, mouse) reverts the text back to the state it was in when the sugggestions list was first shown. However, this only happens for a particular class.
Expected result
Accepting a suggestion from the suggestions list inserts the suggestion into the script.
Actual result
Accepting a suggestion from the suggestions list reverts the code to the state when the suggestions list was initially shown.
Steps to reproduce

Configure VS Code for authoring Unity code by installing the Unity Extension v0.9.0 (see extension post)
Download the latest Visual Studio Editor package in Unity
Attempt to override SetPosition-function from the UnityEditor.Experimental.GraphView.Node class by typing the following:
override S

Extension version: 1.6.0
VS Code version: Code 1.81.0 (6445d93c81ebe42c4cbd7a60712e0b17d9463e97, 2023-08-02T12:37:13.485Z)
OS version: Windows_NT x64 10.0.19045
Modes:

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-9600K CPU @ 3.70GHz (6 x 3696)

GPU Status
2d_canvas: enabledcanvas_oop_rasterization: disabled_offdirect_rendering_display_compositor: disabled_off_okgpu_compositing: enabledmultiple_raster_threads: enabled_onopengl: enabled_onrasterization: enabledraw_draw: disabled_off_okvideo_decode: enabledvideo_encode: enabledvulkan: disabled_offwebgl: enabledwebgl2: enabledwebgpu: enabled

Load (avg)
undefined

Memory (System)
15.94GB (3.07GB free)

Process Argv
D:\Unity Projects\Chemical Beats -g D:\Unity Projects\Chemical Beats\Assets\Scripts\Editor\AIBehaviourTree\GraphNode.cs:1:0 --crash-reporter-id 09d8f480-131b-4380-9e29-c77f065f947c

Screen Reader
no

VM
0%

A/B Experiments
vsliv368cf:30146710
vsreu685:30147344
python383cf:30185419
vspor879:30202332
vspor708:30202333
vspor363:30204092
vswsl492:30256859
vslsvsres303:30308271
vserr242:30382549
pythontb:30283811
vsjup518:30340749
pythonptprofiler:30281270
vshan820:30294714
vstes263:30335439
vscoreces:30445986
vscod805:30301674
binariesv615:30325510
bridge0708:30335490
bridge0723:30353136
vsaa593:30376534
pythonvs932:30410667
vscaat:30438848
vsclangdf:30486550
c4g48928:30535728
dsvsc012cf:30540253
pynewext54:30695312
azure-dev_surveyone:30548225
vscccc:30803845
282f8724:30602487
f6dab269:30613381
2i9eh265:30646982
showlangstatbar:30737416
vsctsb:30748421
03d35959:30757346
pythonfmttext:30731395
pythoncmvfstrcf:30756944
f8hc8238:30694864
fixshowwlkth:30771522
showindicator:30805244
pythongtdpath:30769146
i26e3531:30792625
gsofb:30804716
pythonnosmt12:30797651
pythonidxpt:30805730
pythonnoceb:30805159
e537b577:30795824
dsvsc013:30795093
dsvsc014:30804076

This is a known issue with override completion.  Duplicate of https://github.com/dotnet/vscode-csharp/issues/6011


add option "go to line"

I would be very happy if you could add an option by right-clicking on the row numbers to go to a certain row

You want to go to the line on which you right-click?  If so, why not just left-click in the line itself?  Am I missing something?

I'm writing with google translate, probably that's why I'm not understood, I'll try to explain again:
I mean they will be able to right click and then a window will open for you to enter which line number you want to reach and by clicking save you will reach the line you entered
did you understand?

I doubt this is possible.  Right-clicking brings up the context menu but probably cannot trigger a command - like opening an input box, etc.  You could try, in an extension, adding a command to the line number context menu that when selected will do what you want.

There is a keyboard shortcut for that: ctrl+g. To type in the number you have to use the keyboard anyways.