No camera image on rviz for op3_bringup_visualization.launch

No camera image is showing from OP3 in rviz when op3_bringup_visualization.launch is executed.
As a work around, it is possible to see the image by launching:
rosrun usb_cam usb_cam_node
then changing the image topic in rviz to /usb_cam/image_raw

Sometimes the rviz configuration file seems to be wrong.
After running op3_bringup_visualization.launch, how about adding a camera topic manually?
Click Add button -> select By topic tab -> choose a camera of /usb_cam_node/image_raw
Or If camera calibration file is missing, you can not see the camera topic in rviz.
check the file in the folder of /home/username/.ros/camera_info/camera.yaml
If it doesn't exist, you should calibrate the camera.
Refer this link please.

This issue will be closed since there were no actions for a while. You can reopen this issue to show this issue to the users whenever. Thanks.


[Addition]: Spoofing Cluster

Hotspot b58 Addresses
11nwrJje6fyXqAhfwjHwyKVuEj2Sz6ANZas4xKYfU5F6xLEzbSV
11z7EcaPK9V65ZHKBbtzezGCUfZKe2iapu5u36C3kJ2gJ1VS9gn
11gTZyaTPmWE5pfBvFQ3cfwfRSemACYEAMC6CpYbDkysZT9obHP
113x9ZXT48fyfzkeb1aeQxKiRYxbJf7xkBEreWeP5eCJpXdbBCg
11nwrJje6fyXqAhfwjHwyKVuEj2Sz6ANZas4xKYfU5F6xLEzbSV
11z7EcaPK9V65ZHKBbtzezGCUfZKe2iapu5u36C3kJ2gJ1VS9gn
Discord Handle
tr1plx#9421
Reason(s)
Members of spoofing clusters
Screws up the transmit scale. This is unfair to the legal installations.
Every other hotspot of the corresponding wallets are already on the deniylist.
Please add these hotspots. Thank you for your great work
involved wallets:
13a4J7TbYAimEMeRuCaM3AduUh4V2aCGtzkTBxbLF9tDnSHDfjr
13yjhtnW8e64Ui4dYTZonVyMXhY81daHz3SNH5rvsz3NQfFXmQq

Multiple clusters dotted around this area.

Although this cluster seems to have been offline for a while.
11nwrJje6fyXqAhfwjHwyKVuEj2Sz6ANZas4xKYfU5F6xLEzbSV

Spoofers have long been hiding in major metropolitan areas to remain undetected for longer. but since I run some hotspots in the Stuttgart area, it annoys me all the more. My issue hasn't been processed since February. I don't feel like working with the Denylist anymore!

PUSH:
11gTZyaTPmWE5pfBvFQ3cfwfRSemACYEAMC6CpYbDkysZT9obHP
113x9ZXT48fyfzkeb1aeQxKiRYxbJf7xkBEreWeP5eCJpXdbBCg
still cheating !


called Result::unwrap() on an Err value: Any { .. }

Hi Prisma Team! My Prisma Client just crashed. This is the report:
Versions

Name
Version

Node
v16.18.0

OS
windows

Prisma Client
5.13.0

Query Engine
b9a39a7ee606c28e3455d0fd60e78c3ba82b1a2b

Database
sqlserver

Logs
prisma:tryLoadEnv Environment variables not found at null
prisma:tryLoadEnv Environment variables loaded from D:\Learn\NestJS\api-gateway\.env
prisma:client checkPlatformCaching:postinstall
prisma:client checkPlatformCaching:ciName
prisma:tryLoadEnv Environment variables not found at null
prisma:tryLoadEnv Environment variables loaded from D:\Learn\NestJS\api-gateway\.env
prisma:client dirname D:\Learn\NestJS\api-gateway\node_modules\.prisma\client
prisma:client relativePath ../../../prisma
prisma:client cwd D:\Learn\NestJS\api-gateway\prisma
prisma:client clientVersion 5.13.0
prisma:client:libraryEngine internalSetup
prisma:client:engines:resolveEnginePath enginePath D:\Learn\NestJS\api-gateway\node_modules\.prisma\client\query_engine-windows.dll.node
prisma:client prisma.$queryRaw(UPDATE Item SET @P1 WHERE id = @P2, [["name = 'Laptop'","quantity = 2000"],"2"])
prisma:client:libraryEngine sending request, this.libraryStarted: false
prisma:client:libraryEngine library starting
prisma:client:libraryEngine library started

Client Snippet
// PLEASE FILL YOUR CODE SNIPPET HERE

Schema
// PLEASE ADD YOUR SCHEMA HERE IF POSSIBLE

Prisma Engine Query
{"X":{}}}

Hi @smg-helenyemima, can you please share the contents of schema.prisma? Can you paste the Prisma Client code that made your app crash, with explicit input arguments values that we can copy from the snippet and run internally to verify this issue?
Ideally, we kindly ask you to fill all the information requested in this bug template, to allow us to attempt a reproduction.
Thanks!

Hey @smg-helenyemima,
Did you get a chance to take a look at the previous comment?
Without further information, we'll have to close this issue. Thank you 🙏

Hey @smg-helenyemima ,
With the lack of information and our inability to reproduce this issue, we're now closing it. Feel free to comment if you have any updates about it.
Thank you 🙏


Standardize event data for borrowing event

Read our Contributor Guidelines and README.
Parse data from event which fetched on-chain to humman-readable format, for example, now we have user = data[0] and we want to have user = data.user

In method parse_borrowing_event
Event data structure - event data

Create Serializers model using pydantic in serializers.py
In method process_borrowing_event
substitute fetching data by index Example: user= data[0] to data.user using serializer class

I would like to take this task, I love 🍍
I am a fullstack developer with 3+ years of experience
ETA: 24hrs


Silent replacement of destroyed baseobject on emit

Description of the problem
If you send baseobject (vehicle, player, etc.) via network it can be received as another baseobject, which is a security hole.
Reproduction steps

Create serverside vehicle
Send it to client
Send client->server event to destroy vehicle and right after it create another one, it will get same id
Send vehicle (from step 2) back to server
Server receives imposter vehicle we created in step 3

JS code:
server
alt.onClient('test', (player, obj) => {
    alt.log('test:', obj, obj?.id)
    alt.log(
        'is bmx:', obj.model === alt.hash('bmx'),
        'is sultan:', obj.model === alt.hash('sultan')
    )
})

let veh
alt.onClient('destroy', (player) => {
    veh.destroy()
    veh = new alt.Vehicle('bmx', 0, 0, 0, 0, 0, 0)
    alt.log('recreated veh id:', veh.id)
})

veh = new alt.Vehicle('sultan', 0, 0, 0, 0, 0, 0)
alt.log('created veh id:', veh.id)

player.emit('test', veh)

client
alt.onServer('test', (obj) => {
    alt.log('on server test')
    alt.emitServer('destroy')
    alt.emitServer('test', obj)
})

Expected behaviour
Somehow prevent it in the distant future, since as Heron said, there is no way to fix it now.
Additional context
No response
Operating system
Windows 11
Version
15.0-dev404
Crashdump ID
No response
Reproduction tested

[X] I confirm that I have made sure that this issue is also present on the newest dev version

It's core issue, and Heron said it wont be fixed


Cannot provide backgroundColor without backgroundColor when item is selected

Describe the bug
If itemRenderer is predefined and backgroundColor is passed as in https://codesandbox.io/s/1p8m477w7 (CustomTimeline.jsx/line 41) , you need to to provide different color when item is selected, otherwise when you click on item and click away the background color is the default blue
To Reproduce

Open https://codesandbox.io/s/1p8m477w7
Go to CustomTimeline.jsx / line 49

    const backgroundColor = itemContext.selected
      ? itemContext.dragging
        ? "red"
        : item.selectedBgColor
      : item.bgColor;

Change code to:

const backgroundColor = item.bgColor;

Click on an item in the demo on the right (backgroundColor is changed to yellow)
Click on another item. The backgroundColor of the first item is not the original color but the default blue

Expected behavior
I expected to be able to predefine the backgroundColor in itemRenderer's getItemProps without having to calculate different backgroundColor when item is selected
Screenshots

Library Version
0.20.0
Desktop:

OS: Ubuntu 18.04
Browser: Chrome
Version 68.0.3440.75

Same issue here, fixed by using background instead of backgroundColor


[BUG] - Cli prevents spending from script with inline datum at same utxo

Internal/External
Internal   if an IOHK staff member.
Area
Plutus    Related to Plutus Scripts (Alonzo).
Summary
The cli enforces including datum with a spending script input. This combined with the limitation where inline datum cannot also exist in the witness set (see here) means it is not possible to consume/reference an output with inline datum to satisfy the spending script. Including datum for spending script should be optional.
Steps to reproduce
Steps to reproduce the behavior:

With on output with inline datum at a script address that satisfies the script
Attempt to spend the script output with the cli

Actual result
When using --tx-in-script-file you must provide --tx-in-datum-*. Doing so will result in the following ledger error:
ShelleyTxValidationError ShelleyBasedEraBabbage (ApplyTxError [UtxowFailure (FromAlonzoUtxowFail (NonOutputSupplimentaryDatums (fromList [SafeHash "9e1199a988ba72ffd6e9c269cadb3b53b5f360ff99f112d9b2ee30c4d74ad88b"]) (fromList [])))])

Expected behavior
It is possible to use --tx-in-script-file and omit --tx-in-datum-* so that only the inline datum is used for script execution, or perhaps you need to also reference the same input (without needing to attach datum).
System info (please complete the following information):

Node version d0b0fa10ac83dc1d0d40aaf294cfded1b455a0a4 vasil-testnet-v1

We should also fix this for reference inputs
It'll make --reference-tx-in-datum-* optional.

Resolved by #4006


AttributeError: 'PipelineEngine' object has no attribute 'flatten'

Describe the bug
AttributeError: 'PipelineEngine' object has no attribute 'flatten'
To Reproduce
Steps to reproduce the behavior:

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

Expected behavior
A clear and concise description of what you expected to happen.
ds_report output
Please run ds_report to give us details about your setup.
Screenshots
If applicable, add screenshots to help explain your problem.
System info (please complete the following information):

OS: [e.g. Ubuntu 18.04]
GPU count and types [e.g. two machines with x8 A100s each]
Interconnects (if applicable) [e.g., two machines connected with 100 Gbps IB]
Python version
Any other relevant info about your setup

Launcher context
Are you launching your experiment with the deepspeed launcher, MPI, or something else?
Docker context
Are you using a specific docker image that you can share?
Additional context
Add any other context about the problem here.

@suiyan538, can you share a repro for this issue? Thanks!

@suiyan538 Hi, thank you for your report.
flatten of DeepSpeedEngine (the superclass of PipelineEngine) is expected to be set at the constructor: https://github.com/microsoft/DeepSpeed/blob/43d58d99eba800a809f16efa30a90eb25f7a7003/deepspeed/runtime/engine.py#L393
Can you give us a reproducing code?

Closing for now. Please re-open as needed.


Integrate Tailwind

Based on these instructions.
Basically, install binary and run on CSS change.

I think it's already possible to use Tailwind with Leptos, but I thought it would be great to have a kind of --tailwind parameter to the cargo leptos watch command to automatically run tailwindcss to watch changes without having to use two terminals. If you think it would be a good feature to add I can start working on a PR.

Sure. I think it would be better to add it to the config:
[package.metadata.leptos]
tailwind = true

So that the LibPackage struct ends up with a tailwind: bool field.
A good second step would be to add support for automatic download and installation of the tailwind binary in the Exe enum.
The signalling part is a bit complex, but if you want I can do that. Just give me a shout once the above is done.

I am thinking about working on this issue. can you tell me where can I check if classes have been updates in view! macros,  because I think tailwind build command have to be called everytime a class attribute is updated. Having tailwind watch command running would probably interfere with cargo rebuild as it does when it tried it by just running both tailwind watch and cargo leptos watch in two terminal instances. I was getting the following error everytime I tried using a class which wasn't already cached:-
Blocking waiting for file lock on package cache
or am I misinterpreting.
I am pretty new to open source so please don't mind that I am not well versed with the open source way of doing things.

I have already made the neccessary changes required for adding tailwind:bool argument to the conifig file and updating the Exe enum as well as the ExeMeta struct in a local branch. Shall I publish it?

There's a rather complex signalling system in place. See https://github.com/leptos-rs/cargo-leptos/blob/main/src/readme.md but note that it is mainly for getting an approximate understanding of it.
Let's start with what you've already done and then I'll point you to the next steps. Could you do a PR to the tailwind-support branch? Like that, we can take it step-by-step there.

#63

@akesson Do you think after merging changes from #62, tailwind can be integrated into the watch process like I mention here:-
https://github.com/leptos-rs/cargo-leptos/issues/62#issuecomment-1405975986

Yes, you are definitely heading the right way. Better to use tailwind build only because cargo-leptos is already doing the fs watch. I think a good next step would be to integrate tailwind into the cargo build step with an implementation in src/compile/tailwind.rs. But let's finish the PR first.


Unhandled Exception: MissingPluginException for preparePlayer Method in audio_waveforms Plugin

After selecting an MP3 file in my Flutter application, I encountered an error in the terminal:
Unhandled Exception: MissingPluginException(No implementation found for method preparePlayer on channel simform_audio_waveforms_plugin/methods)
#0      MethodChannel._invokeMethod (package:flutter/src/services/platform_channel.dart:308:7)

#1      AudioWaveformsInterface.preparePlayer (package:audio_waveforms/src/base/audio_waveforms_interface.dart:100:18)

#2      PlayerController.preparePlayer (package:audio_waveforms/src/controllers/player_controller.dart:122:24)

#3      _AudioWaveState.initAudioPlayer (package:client/features/home/view/widgets/audio_wave.dart:25:5)

This error occurs when attempting to prepare the audio player using the preparePlayer method from the audio_waveforms plugin.
This is my code for preparePlayer:
void initAudioPlayer() async {
await playerController.preparePlayer(path: widget.path);
}
I have tried:
flutter clean
flutter pub get

Hello,
I am experiencing the same issue with the audio_waveforms plugin. Specifically, I am encountering the following error:
Here is a brief summary of what I have tried so far:
Dependencies: I have included the latest version of flutter_audio_waveforms in my pubspec.yaml file and ran flutter pub get.
Platform-Specific Configuration:
Android: Verified that the plugin is correctly configured in android/app/build.gradle and that there are no issues with the MainActivity.java or MainActivity.kt files.
iOS: Checked that the plugin is added to the Podfile and that pod install was run successfully.
Code Implementation: Implemented the plugin in my Dart code and ensured that I am calling methods like preparePlayer after initializing the plugin.
Despite following these steps, the issue persists. Here is the relevant code snippet where the issue occurs:
...
@override
void initState() {
super.initState();
controller = PlayerController();
_preparePlayer();
...
}
...
void _preparePlayer() async {
....
controller.preparePlayer(
path: _localPath,
shouldExtractWaveform: true,
);
...
}
...
Additional Information:
Flutter Version: 3.24.3
Plugin Version: 1.0.5
Platform: Android
I would greatly appreciate any guidance or solutions for resolving this issue. It seems that there might be an integration problem or a missing implementation in the plugin. Thank you for your assistance.

Hello, Can you please try to delete the app and again install the app Before installing the app, please do flutter clean.


Create .po file for volunteer to translate this plugin

same as title , make your plugin multilingual !!
you can gettext of python create the .pot file and use poedit (or other po file edit software )to  convert it to .po file ,than volunteer can translate it to other language
you can find more information by searching "python i18n"
and you can also create team on Transifex and （or crowdin）share link to others people to help you for translation
（sorry for my english haha）

I tried to make it but i just know a little python knowledge,emmmmm

emmm

lol

It's not enough to just create a po file and have some translations. Those translations need to work out of the box and detect kicad's and/or system language settings.
This is low on my list of priorities which is why there is no progress so far.
Unless you want to contribute working code please refrain from meaningless comments.


[ClangImporter] Resolve forward declarations before importing names

Explanation: If a bridging header forward-declares a class that originally came from a Swift framework, and that Swift class had a custom Objective-C name, Swift 3.0 managed to figure out that these two declarations were the same. Swift 3.1 broke that with its more structured logic for importing names; restore that behavior by canonicalizing to the class definition (@interface) instead of a forward-declaration (@class).
Scope: Affects importing of forward-declared classes, but likely has no effect unless they were defined in Swift.
Issue: SR-3798 / rdar://problem/30287966
Reviewed by: @milseman
Risk: Low. We're mostly just doing the same things, but in a different, more reliable order.
Testing: Added compiler regression tests, verified that the original project no longer fails.

@swift-ci Please test

Build failed
Jenkins build - Swift Test OS X Platform
Git Commit - 80d77fab95d51738b5b03a131882f18d4bae24f1
Test requested by - @jrose-apple

Infrastructure issue. :-(
@swift-ci Please test macOS


vtkVersionMacros.h needed to be included

Without it, VTK_MAJOR_VERSION and VTK_MINOR_VERSION macros were not
defined, leading to compile failures in VTK 8.1.

Nice, merged before I could even request your review!


hand tracking always lost when near the face

i guess the training separate the hand and face deliberately.

This is most likely due to the lack of contrast between the skin on the hand and the skin on the face. A A higher contrast background will almost always work better.

Hi, exactly like @shraiwi said, since we are detecting palm area so it is indeed difficult to eliminate false positive on faces. It's a trade-off, possible to get better precision with lower speed by increasing the model size.


How to get the corresponding original signal according to the event_index of eventalign

Hi,
Thank you very much for your excellent tools Nanopolish. Recently, I used nanopolish eventalign to align the raw signals to the contig. But I don't know how to get the raw signals of each event. Can I get some information such as start position on raw signal sequence, end position on raw signal sequence of each event?
Thanks for your help.
Neng

Hi,
Try adding the --samples flag to eventalign.
Jared

To Jared,
That is great and it works. Thanks so much.
Neng


Feature 170 stratosphere diag

Pull Request Testing

[x] Describe testing already performed for these changes:
generated all four plots on 'kiowa' and 'seneca'

[x] Recommend testing for the reviewer(s) to perform, including the location of input datasets, and any additional instructions:

on 'seneca' and 'kiowa'  there is a conda environment: /home/met_test/.conda/envs/feature_170_stratosphere_diag
clone the METcalcpy (develop)  and METplotpy (feature_170_stratosphere_diag) repositories
set the PYTHONPATH
$METCALCPY_SOURCE/METcalcpy:$METCALCPY_SOURCE/METcalcpy/metcalcpy:$METPLOTPY_SOURCE/METplotpy:$METPLOTPY_SOURCE/METplotpy/metplotpy
in the $METPLOTPY_SOURCE/METplotpy/metplotpy/contributed/stratosphere_diagnostics/stratosphere_diagnostics.yaml file, replace setting for input_data_path to one of the following:
  on kiowa:
  /d1/projects/METplotpy/METplopty_Data/stratosphere

 on seneca:
 /d1/projects/METplus/METplotpy_Data/stratosphere

and replace the setting for output_data_path to the directory where you want to save the output plots.
Run the plotting code from the $METPLOTPY_SOURCE/METplotpy/metplotpy/contributed/stratosphere_diagnostics directory:
python stratosphere_diagnostics.py stratosphere_diagnostics.yaml

You can also copy the stratosphere_diagnostics.yaml file to another directory and run it like the NOAA EMC
users would have to run it.

You should have four plots created:
zonal_mean_wind_contour.png

zonal_mean_temperature_contour.png

zonal_mean_wind.png

polar_cap_meridional_mean_temp.png

[x] Do these changes include sufficient documentation updates, ensuring that no errors or warnings exist in the build of the documentation? [Yes ] also updated spreadsheet of Python packages

[x] Do these changes include sufficient testing updates? Manual testing only. Uses a large netCDF file as sample data

[x] Will this PR result in changes to the test suite? [No]
If yes, describe the new output and/or changes to the existing output:

[x] Please complete this pull request review by [Tuesday Feb1].

Pull Request Checklist
See the METplus Workflow for details.

[x] Complete the PR definition above.
[x] Ensure the PR title matches the feature or bugfix branch name.
[x] Define the PR metadata, as permissions allow.
Select: Reviewer(s)
Select: Organization level software support Project or Repository level development cycle Project
Select: Milestone as the version that will include these changes
[x] After submitting the PR, select Linked issues with the original issue number.
[ ] After the PR is approved, merge your changes. If permissions do not allow this, request that the reviewer do the merge.
[ ] Close the linked issue and delete your feature or bugfix branch from GitHub.

I'm getting some kind of error, although I'm not using the conda environment, so maybe that's the issue.  Does anyone know what this means?
ImportError: Plotting of arrays of cftime.datetime objects or arrays indexed by cftime.datetime objects requires the optional nc-time-axis (v1.2.0 or later) package.

I thought I updated the conda environment to contain that.  Let me make
sure it's installed.

Minna Win
Pronouns: she/her
National Center for Atmospheric Research
Developmental Testbed Center
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
On Fri, Jan 28, 2022 at 1:39 PM Christina Kalb @.***>
wrote:

I'm getting some kind of error, although I'm not using the conda
environment, so maybe that's the issue. Does anyone know what this means?
ImportError: Plotting of arrays of cftime.datetime objects or arrays
indexed by cftime.datetime objects requires the optional nc-time-axis
(v1.2.0 or later) package.
—
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METplotpy/pull/191#issuecomment-1024618700,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4UJHT4LPTOYV4IBRY5STLUYL5G3ANCNFSM5M44R77A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

sorry, I should have read your message more carefully.  You need to install
the nc-time-axis in order to get the plots to work.  It is an extension of
the cftime module.  I just verified that this is indeed installed in the
feature_170_stratosphere_diag conda environment on both 'kiowa' and
'seneca'.
Minna Win
Pronouns: she/her
National Center for Atmospheric Research
Developmental Testbed Center
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
On Fri, Jan 28, 2022 at 4:32 PM Minna Win-Gildenmeister @.***>
wrote:

I thought I updated the conda environment to contain that.  Let me make
sure it's installed.

Minna Win
Pronouns: she/her
National Center for Atmospheric Research
Developmental Testbed Center
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
On Fri, Jan 28, 2022 at 1:39 PM Christina Kalb @.***>
wrote:

I'm getting some kind of error, although I'm not using the conda
environment, so maybe that's the issue. Does anyone know what this means?
ImportError: Plotting of arrays of cftime.datetime objects or arrays
indexed by cftime.datetime objects requires the optional nc-time-axis
(v1.2.0 or later) package.
—
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METplotpy/pull/191#issuecomment-1024618700,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4UJHT4LPTOYV4IBRY5STLUYL5G3ANCNFSM5M44R77A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Okay, thanks.  How do I activate the conda environment to try and test with
that version?
Tina
On Fri, Jan 28, 2022 at 4:44 PM bikegeek @.***> wrote:

sorry, I should have read your message more carefully. You need to install
the nc-time-axis in order to get the plots to work. It is an extension of
the cftime module. I just verified that this is indeed installed in the
feature_170_stratosphere_diag conda environment on both 'kiowa' and
'seneca'.
Minna Win
Pronouns: she/her
National Center for Atmospheric Research
Developmental Testbed Center
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
On Fri, Jan 28, 2022 at 4:32 PM Minna Win-Gildenmeister @.***>
wrote:

I thought I updated the conda environment to contain that. Let me make
sure it's installed.

Minna Win
Pronouns: she/her
National Center for Atmospheric Research
Developmental Testbed Center
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
On Fri, Jan 28, 2022 at 1:39 PM Christina Kalb @.***>
wrote:

I'm getting some kind of error, although I'm not using the conda
environment, so maybe that's the issue. Does anyone know what this
means?
ImportError: Plotting of arrays of cftime.datetime objects or arrays
indexed by cftime.datetime objects requires the optional nc-time-axis
(v1.2.0 or later) package.
—
Reply to this email directly, view it on GitHub
<https://github.com/dtcenter/METplotpy/pull/191#issuecomment-1024618700
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AA4UJHT4LPTOYV4IBRY5STLUYL5G3ANCNFSM5M44R77A

.
Triage notifications on the go with GitHub Mobile for iOS
<
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675

or Android
<
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub
.
You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METplotpy/pull/191#issuecomment-1024746944,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AFJHD4RKIBNTZNXFYKL4V7LUYMS5TANCNFSM5M44R77A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because your review was requested.Message ID:
@.***>

--

Christina Kalb
Associate Scientist
National Center for Atmospheric Research
3450 Mitchell Lane
Boulder, CO 80301
@.***
303-497-2819

My working day may not be your working day. Please do not feel obliged to
reply to this email outside of your normal working hours.

conda activate /home/met_test/.conda/envs/feature_170_stratosphere_diag
on kiowa
On Mon, Jan 31, 2022 at 11:17 AM Christina Kalb @.***>
wrote:

Okay, thanks. How do I activate the conda environment to try and test with
that version?
Tina
On Fri, Jan 28, 2022 at 4:44 PM bikegeek @.***> wrote:

sorry, I should have read your message more carefully. You need to
install
the nc-time-axis in order to get the plots to work. It is an extension of
the cftime module. I just verified that this is indeed installed in the
feature_170_stratosphere_diag conda environment on both 'kiowa' and
'seneca'.
Minna Win
Pronouns: she/her
National Center for Atmospheric Research
Developmental Testbed Center
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
On Fri, Jan 28, 2022 at 4:32 PM Minna Win-Gildenmeister @.***>
wrote:

I thought I updated the conda environment to contain that. Let me make
sure it's installed.

Minna Win
Pronouns: she/her
National Center for Atmospheric Research
Developmental Testbed Center
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
On Fri, Jan 28, 2022 at 1:39 PM Christina Kalb @.***>
wrote:

I'm getting some kind of error, although I'm not using the conda
environment, so maybe that's the issue. Does anyone know what this
means?
ImportError: Plotting of arrays of cftime.datetime objects or arrays
indexed by cftime.datetime objects requires the optional nc-time-axis
(v1.2.0 or later) package.
—
Reply to this email directly, view it on GitHub
<
https://github.com/dtcenter/METplotpy/pull/191#issuecomment-1024618700
,
or unsubscribe
<

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

.
Triage notifications on the go with GitHub Mobile for iOS
<

https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675

or Android
<

https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub

.

You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
<https://github.com/dtcenter/METplotpy/pull/191#issuecomment-1024746944
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AFJHD4RKIBNTZNXFYKL4V7LUYMS5TANCNFSM5M44R77A
.
Triage notifications on the go with GitHub Mobile for iOS
<
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
<
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub
.
You are receiving this because your review was requested.Message ID:
@.***>

--

Christina Kalb
Associate Scientist
National Center for Atmospheric Research
3450 Mitchell Lane
Boulder, CO 80301
@.***
303-497-2819

My working day may not be your working day. Please do not feel obliged to
reply to this email outside of your normal working hours.
—
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METplotpy/pull/191#issuecomment-1026070427,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AHAUIKAPTQQV33JZ2MAGBITUY3G4TANCNFSM5M44R77A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because your review was requested.Message ID:
@.***>

--
Hank Fisher       Research Applications Laboratory
303-497-2817      National Center for Atmospheric Research
@.***  P.O. Box 3000, Boulder CO 80307-3000

and if for some reason activating the conda environment in the met_test
directory doesn't work, here is the requirements.yaml file you can use to
create your own.

work in your bash shell
run this: conda env create -f requirements.yaml
you might get a warning about not having a pip dependency, but conda will
go ahead and add one for you.  I think it's because I used pip to install a
lot of these (due to using a pip list output from my Mac to get things to
work on the Linux hosts).

Regards,
Minna
Minna Win
Pronouns: she/her
National Center for Atmospheric Research
Developmental Testbed Center
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
On Mon, Jan 31, 2022 at 11:17 AM Christina Kalb @.***>
wrote:

Okay, thanks. How do I activate the conda environment to try and test with
that version?
Tina
On Fri, Jan 28, 2022 at 4:44 PM bikegeek @.***> wrote:

sorry, I should have read your message more carefully. You need to
install
the nc-time-axis in order to get the plots to work. It is an extension of
the cftime module. I just verified that this is indeed installed in the
feature_170_stratosphere_diag conda environment on both 'kiowa' and
'seneca'.
Minna Win
Pronouns: she/her
National Center for Atmospheric Research
Developmental Testbed Center
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
On Fri, Jan 28, 2022 at 4:32 PM Minna Win-Gildenmeister @.***>
wrote:

I thought I updated the conda environment to contain that. Let me make
sure it's installed.

Minna Win
Pronouns: she/her
National Center for Atmospheric Research
Developmental Testbed Center
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
On Fri, Jan 28, 2022 at 1:39 PM Christina Kalb @.***>
wrote:

I'm getting some kind of error, although I'm not using the conda
environment, so maybe that's the issue. Does anyone know what this
means?
ImportError: Plotting of arrays of cftime.datetime objects or arrays
indexed by cftime.datetime objects requires the optional nc-time-axis
(v1.2.0 or later) package.
—
Reply to this email directly, view it on GitHub
<
https://github.com/dtcenter/METplotpy/pull/191#issuecomment-1024618700
,
or unsubscribe
<

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

.
Triage notifications on the go with GitHub Mobile for iOS
<

https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675

or Android
<

https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub

.

You are receiving this because you authored the thread.Message ID:
@.***>

—
Reply to this email directly, view it on GitHub
<https://github.com/dtcenter/METplotpy/pull/191#issuecomment-1024746944
,
or unsubscribe
<
https://github.com/notifications/unsubscribe-auth/AFJHD4RKIBNTZNXFYKL4V7LUYMS5TANCNFSM5M44R77A
.
Triage notifications on the go with GitHub Mobile for iOS
<
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
<
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub
.
You are receiving this because your review was requested.Message ID:
@.***>

--

Christina Kalb
Associate Scientist
National Center for Atmospheric Research
3450 Mitchell Lane
Boulder, CO 80301
@.***
303-497-2819

My working day may not be your working day. Please do not feel obliged to
reply to this email outside of your normal working hours.
—
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METplotpy/pull/191#issuecomment-1026070427,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4UJHWI35PW4L33PA7NLKLUY3G4TANCNFSM5M44R77A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Now it's not finding metcalcpy:
File "stratosphere_diagnostics.py", line 17, in
from metcalcpy.pre_processing.directional_means import meridional_mean
ModuleNotFoundError: No module named 'metcalcpy'
I did a setenv PYTHONPATH $METCALCPY_SOURCE/METcalcpy:$METCALCPY_SOURCE/METcalcpy/metcalcpy:$METPLOTPY_SOURCE/METplotpy:$METPLOTPY_SOURCE/METplotpy/metplotpy
so, I'm not sure why it can't find metcalcpy.  It says I'm in a tcsh shell

That matches what my PYTHONPATH was set to as well.  I had to run in bash
shell in order to activate the conda environment.  I just checked and even
though I'm switched to  bash, when I do an 'echo $SHELL'  at the command
line, it shows I'm in tcsh!  Can you try using the export= syntax and see
if that works?

Minna Win
Pronouns: she/her
National Center for Atmospheric Research
Developmental Testbed Center
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
On Tue, Feb 1, 2022 at 12:28 PM Christina Kalb @.***>
wrote:

Now it's not finding metcalcpy:
File "stratosphere_diagnostics.py", line 17, in
from metcalcpy.pre_processing.directional_means import meridional_mean
ModuleNotFoundError: No module named 'metcalcpy'
I did a setenv PYTHONPATH
$METCALCPY_SOURCE/METcalcpy:$METCALCPY_SOURCE/METcalcpy/metcalcpy:$METPLOTPY_SOURCE/METplotpy:$METPLOTPY_SOURCE/METplotpy/metplotpy
so, I'm not sure why it can't find metcalcpy. It says I'm in a tcsh shell
—
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METplotpy/pull/191#issuecomment-1027208010,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4UJHV5POIGYHFOHZFPDPDUZAX7RANCNFSM5M44R77A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

When I try to do the export, I get the following error:
METCALCPY_SOURCE: Undefined variable.
What should METCALCPY_SOURCE be set to?

What do you have your METCALCPY_SOURCE set to?  I don't usually set a
METCALCPY_SOURCE, I just explicitly point to the METcalcpy directory that I
cloned:
/home/minnawin/METcalcpy:/home/minnawin/METcalcpy/metcalcpy:/home/minnawin/METplotpy:/home/minnawin/METplotpy/metplotpy

Minna Win
Pronouns: she/her
National Center for Atmospheric Research
Developmental Testbed Center
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
On Tue, Feb 1, 2022 at 2:39 PM Christina Kalb @.***>
wrote:

When I try to do the export, I get the following error:
METCALCPY_SOURCE: Undefined variable.
What should METCALCPY_SOURCE be set to?
—
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METplotpy/pull/191#issuecomment-1027313392,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4UJHVENTFMMR32DW3NXHTUZBHJZANCNFSM5M44R77A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

I'm still getting an import error.  I'm not sure why.  Does it matter which order I switch to the bash shell and activate the conda environment?
ImportError: Plotting of arrays of cftime.datetime objects or arrays indexed by cftime.datetime objects requires the optional nc-time-axis (v1.2.0 or later) package

I always switch to bash shell (necessary for using conda), then set the
path, then activate conda.  I hope that works.
Minna Win
Pronouns: she/her
National Center for Atmospheric Research
Developmental Testbed Center
Phone:<PHONE_NUMBER>
Fax:<PHONE_NUMBER>
On Wed, Feb 2, 2022 at 12:09 PM Christina Kalb @.***>
wrote:

I'm still getting an import error. I'm not sure why. Does it matter which
order I switch to the bash shell and activate the conda environment?
ImportError: Plotting of arrays of cftime.datetime objects or arrays
indexed by cftime.datetime objects requires the optional nc-time-axis
(v1.2.0 or later) package
—
Reply to this email directly, view it on GitHub
https://github.com/dtcenter/METplotpy/pull/191#issuecomment-1028272719,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AA4UJHR4NLVWXCTJYWIYRQLUZF6NZANCNFSM5M44R77A
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.
You are receiving this because you authored the thread.Message ID:
@.***>

Finally got it working!


No rule to make target 'image'

Describe the bug
Following the build steps at https://github.com/microsoft/azurelinux/blob/3.0/toolkit/docs/quick_start/quickstart.md:
sudo make image -j8 REBUILD_TOOLS=y REBUILD_PACKAGES=n CONFIG_FILE=./imageconfigs/core-efi.json
make: *** No rule to make target 'image'.  Stop.

I am on 3.0-stable, with HEAD pointed to:

commit 30d1349857d98c0c8710bd591d3a1676f502db20 (HEAD, tag: 3.0.20241101-3.0, tag: 3.0-stable)
Merge: ec7cffeff 75d1fc327
Author: jslobodzian<EMAIL_ADDRESS>Date:   Fri Nov 1 17:16:08 2024 -0400
Merge branch '3.0-dev' into 3.0

It seems to be the most recent merge borked something. But with that said, I have never gotten a successful build on an AArch64 build host

To Reproduce
Steps to reproduce the behavior:

Follow the instructions here for VHD or VHDX builds: https://github.com/microsoft/azurelinux/blob/3.0/toolkit/docs/quick_start/quickstart.md
run sudo make image -j8 REBUILD_TOOLS=y REBUILD_PACKAGES=n CONFIG_FILE=./imageconfigs/core-efi.json
See error: make: *** No rule to make target 'image'.  Stop.

Expected behavior
Builds and works

@billatarm you need to run the make command from within toolkit directory. If you try to invoke make from the top directory you'll get No rule to make target 'image' error:
elsaco@juno:~/azurelinux$ sudo make image -j4 REBUILD_TOOLS=y REBUILD_PACKAGES=n CONFIG_FILE=./imageconfigs/core-efi.json
make: *** No rule to make target 'image'.  Stop.

and from toolkit:
elsaco@juno:~/azurelinux/toolkit$ sudo make image -j4 REBUILD_TOOLS=y REBUILD_PACKAGES=n CONFIG_FILE=./imageconfigs/core-efi.json
cd /home/elsaco/azurelinux/toolkit/tools/ && \
        go test -ldflags="-X github.com/microsoft/azurelinux/toolkit/tools/internal/exe.ToolkitVersion=3.0.20241202.1732 -X github.com/microsoft/azurelinux/toolkit/tools/pkg/imagecustomizerlib.ToolVersion=0.6.0-dev.20241202.1732+ba13b4671 -X github.com/microsoft/azurelinux/toolkit/tools/internal/exe.DistroNameAbbreviation=azl -X github.com/microsoft/azurelinux/toolkit/tools/internal/exe.DistroMajorVersion=3" -test.short -covermode=atomic -coverprofile=/home/elsaco/azurelinux/build/tools/internal.test_coverage ./...
---cut---

Test done using 3.0-stable:
commit ba13b467104263b5740cdca3054ea5121d8160ab (HEAD, tag: 3.0.20240910-3.0, tag: 3.0.20240824-3.0, tag: 3.0-stable)
Author: Daniel McIlvaney<EMAIL_ADDRESS>Date:   Tue Aug 27 13:59:31 2024 -0700

    fix: use static search path for toolchain GPG keys during validation (#10269)

commit 9f8d0fad669392e8c53e7334da1f481f5306e673
Merge: 027873ec5 bde321f1e
Author: jslobodzian<EMAIL_ADDRESS>Date:   Fri Aug 23 21:42:25 2024 -0400

    Merge branch '3.0-dev' into 3.0

And it's clearly in the documentation. I was doing this, and then discarded that step.... Thank you!


Change the generated client resource name to plural

Purpose
$Subject
The following tables shows how long different words take to convert to plurals using this implementation.

Add minor comment

Fixed in 757e5da


Update README.md

Summary
Updated readme to reflect change in NextUrlqClientConfig
Set of changes
renamed neverSuspend flag to suspense

These aren't the Client's options, but the withUrqlClient options, so this instruction is still completely correct. The HOC does have a neverSuspend option that can be set to true, although I'm actually unsure whether it can be made obsolete and getClientConfig's config can be read instead 🤔 cc @JoviDeCroock

Hmm, I think we could actually do that 😅 the complexity when we originally added this feature is because we don't read suspense: true when using our prepass approach. The problem presents itself when we use getServerSideProps (no prepass) and a query is uncovered by getServerSideProps which was sometimes intended 😅 it couldn't throw.
https://github.com/FormidableLabs/urql/pull/923#issue-457698628

Ok thanks, I just realised that you can pass the HOC options as second argument.... all good :)


Improve profile for Organizations

Fixes some problems in #27955:

autofocus of the search box
before:
if access the home page will jump to the search box

after:
will not jump to the search box

incorrect display of overview tab
before:

after:

improve the permission check to the private profile repo
In #26295, we simply added access control to the private profile.
But if user have access to the private profile repo , we should also display the profile.

add a button which can jump to the repo list?
I agree @wxiaoguang 's opinion here: https://github.com/go-gitea/gitea/pull/27955#issuecomment-1803178239
But it seems that this feature is sponsored.
So can we add a button which can quickly jump to the repo list or just move profile to the overview page?

Semi-related: Regarding autofocus on search boxes, I would prefer to get rid of all of them eventually. There are certain unintended behaviours like IOS Safari zooming the page when a input element with font size less than 16px is focused.

Semi-related: Regarding autofocus on search boxes, I would prefer to get rid of all of them eventually. There are certain unintended behaviours like IOS Safari zooming the page when a input element with font size less than 16px is focused.

I also think it is better to get rid of all of them eventually. Some places have no autofocus and some places have.
For example, in projects list page we don't have autofocus on search box.

Semi-related: Regarding autofocus on search boxes, I would prefer to get rid of all of them eventually. There are certain unintended behaviours like IOS Safari zooming the page when a input element with font size less than 16px is focused.

I also think it is better to get rid of all of them eventually. Some places have no autofocus and some places have. For example, in projects list page we don't have autofocus on search box.

Yeah, let's do in another PR.


Activate Street Addresses tab when no geocoding credits left

Right now we're disabling this tab:

So the idea is to able the tab and show days left to renew your billing cycle.

@saleiva do the copy works for you?

@rafatower @xavijam do you where I could get this “X days left to renew your billing cycle” info?

I feel that info is not available (yet?) in the frontend side.

I could not find any clear answer when asking around, but after talking to @juanignaciosl (thx) I went with using the user.get('billing_period'), and calculating the remaining days from that, AFAIK the billing is the same for all, even if the quotas differs for different things.


Improve envoyfilter patch implementation

I think these changes help 1. Improve readability 2. Enhance robustness (finding the location and replacing or inserting is not an easy task, and LDS/RDS/CDS all have similar logic)
Currently, only part of it has been modified. Once this PR is merged, I will modify other places in the next PR.

Skipping CI for Draft Pull Request.
If you want CI signal for your change, please convert it to an actual PR.
You can still manually trigger a test run with /test all

/test all

@wulianglongrd: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

lint_istio
09c173468812ceeefc9422a6df059fd605e72919
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test all

@wulianglongrd: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
47ac8305e294f3a89515030abc38780c42d116b7
link
true
/test gencheck

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@wulianglongrd: The following tests failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name
Commit
Details
Required
Rerun command

gencheck_istio
47ac8305e294f3a89515030abc38780c42d116b7
link
true
/test gencheck

lint_istio
47ac8305e294f3a89515030abc38780c42d116b7
link
true
/test lint

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.


Add POST Api "become a candidate"

So coin holder can call this api to become a candidate
API should be /api/applyCandidate
Params: private_key, amount

Done #129


Demonstrate how to cache asset handles using Local system parameters

How can Bevy's documentation be improved?
Strong handles can be retained in system-local storage. This can be quite useful to avoid having to track handles in dedicated resources. None of our examples use this pattern; we should at least demonstrate it in a standalone example.
To show what I mean, here's a snippet of real code that I was working on today.
#[derive(Bundle)]
struct PingBundle {
    ping: Ping,
    #[bundle]
    mesh_bundle: MaterialMesh2dBundle<ColorMaterial>,
}

impl PingBundle {
    fn new(
        ping: Ping,
        location: WorldCoordinates,
        mesh_handle: Mesh2dHandle,
        material_handle: Handle<ColorMaterial>,
    ) -> PingBundle {
        PingBundle {
            ping,
            mesh_bundle: MaterialMesh2dBundle::<ColorMaterial> {
                mesh: mesh_handle,
                material: material_handle,
                transform: Transform::from_xyz(location.x, location.y, 0.0),
                ..default()
            },
        }
    }
}

pub fn ping_at_instruction(
    cached_mesh_handle: Local<Option<Mesh2dHandle>>,
    cached_material_handle: Local<Option<Handle<ColorMaterial>>>,
    mut meshes: ResMut<Assets<Mesh>>,
    mut materials: ResMut<Assets<ColorMaterial>>,
    action_state: Query<&ActionState<PlayerAction>>,
    cursor_query: Query<&WorldCoordinates, With<Cursor>>,
    mut commands: Commands,
) {
    if action_state
        .single()
        .just_pressed(PlayerAction::Instruction)
    {
        let mesh_handle = match cached_mesh_handle.clone() {
            Some(cache) => cache,
            None => meshes.add(shape::Circle::new(50.).into()).into(),
        };

        let material_handle = match cached_material_handle.clone() {
            Some(cache) => cache,
            None => materials.add(ColorMaterial::from(Color::PURPLE)),
        };

        let &location = cursor_query.single();
        commands.spawn_bundle(PingBundle::new(
            Ping::default(),
            location,
            mesh_handle,
            material_handle,
        ));
    }
}

this pattern looks a lot like re-implementing FromWorld inside a system rather than inside a specific struct impl
struct PingHandles {
    mesh_handle: Mesh2dHandle,
    material_handle: Handle<ColorMaterial>,
}

impl FromWorld for PingHandles {
    fn from_world(world: &mut World) -> Self {
        let mut meshes = world.resource_mut::<Assets<Mesh>>();
        let mesh_handle = meshes.add(shape::Circle::new(50.).into()).into();
        let mut materials = world.resource_mut::<Assets<ColorMaterial>>();
        let material_handle = materials.add(ColorMaterial::from(Color::PURPLE));

        Self {
            mesh_handle,
            material_handle,
        }
    }
}

and then you can use a Local< PingHandles> in your system that will hold the handles

I like your pattern better! I'm going to leave the issue open though; we still need to demo this more realistically.


yeelink.light.ceiling20 vs. yeelink.light.ceiling22

Hi,
I have a Xiaomi Mi Ceiling Light (this is the name on the box).
Originally the device is made by Yeelight. It seems a re-branded (any maybe slightly modified) version of a Yeelight device.
The hostname / DNS of the light is yeelink-light-ceiling22-mibt1ADB (identifies as ceiling22).
On the device itself I can find two slightly different model no.:

MJXDD001
MJXDD01SYL

So far everything looks like it is a yeelink.light.ceiling22
BUT
According to your published pinout and the photos it fits more the YLXD50YL
So some details suggest it is yeelink.light.ceiling20 and some details suggest it is yeelink.light.ceiling22
What do you think?

It looks like both versions uses the same WiFi daughter board. The difference between the two YAML example configuration is the RGB support (ceiling20 = with RGB LEDs, ceiling22 = without RGB LEDs). You should use the ceiling22 as starting point because your device is CWWW only.

Could you provide a photo of your ceiling lamp (LED driver board) without the WiFi daughter board attached? I would like to see the socket labels on the mainboard.
The photo should show something like this:https://raw.githubusercontent.com/syssi/esphome-yeelight-ceiling-light/main/images/ylxd50yl/003.jpg

Hi,
yes I can and will provide.
But I can tell you already: It looks exactly like your photo and the labels are the same (R G B is on my board as well).

Do you agree to add your photos to the project under Apache 2.0 license? The pull request of ceiling22 didn't provide any photos. So that's the chance to illustrate/improve the ceiling22 support.

Yes, sure. Feel free to add de photos to the project or do you want me to create a PR?

I'm happy about a contribution too. :-)

Feel free to close this issue as soon your setup is up and running.

Hi,
with the current documented pinout for ceilling22, I was not able to flash the firmware.
TXD = TP4
RXD = TP5
The correct pinout is
TXD = TP5
RXD = TP3
I will create another PR

I was able to flash the ceiling22 firmware / configuration.
Looks very well so far! I am very happy.


No variable with name "reference' in Debugger when watching variable

Also showing wrong type of "reference" see testString

ComponentGroupReference and ComponentReference are subclass of ReferenceBase

Already fixed with parameters

Not fixed. I've latest update. Doesn't like word reference :dancer:

this method contains yield ?

No

Idk how repeat it

`class SomeEvent:UnityEvent{}
class ReferenceBase:ScriptableObject{}
class ComponentReference:ReferenceBase{}
class ComponentGroupReference:ReferenceBase{}
class MyMono:MonoBehaviour{
void Awake(){
var someEvent=new SomeEvent();
someEvent.AddListener(Handler);
someEvent.Invoke(ScriptableObject.CreateInstance());
//Scriptable Object aren't created without "new"
}
public void Handler(ReferenceBase reference){
//Add watch
Debug.Log(reference.GetType());
reference=ScriptableObject.CreateInstance();
Debug.Log(reference.GetType());
}
}
`
Try this!

Strange - i can't enter Handler method

when it call Invoke - no pause inside Handler

what Logs print?

Hm. That interest.

Why not paused. O_o

Put breakpoint also on someEvent.Invoke then try get in or "Play" to next breakpoint

Strange. I can't pause at any line, except var someEvent = new SomeEvent();

ref https://github.com/consulo/consulo-dotnet/issues/69

Don't close the previous issue as it is very important

if i can't reproduce - i can't fix. There no other peoples to can test - except you


State-switch card isn't displayed in v1.6.0 and above with TypeError: Cannot read properties of undefined (reading 'states')

Describe the bug
The state switch card is not working in the pop-up.
I'm using a template to show a card depending of states.light. It's working on v1.5.3 but it's not working since  v1.6.0
To Reproduce
Add a state-state-switch card in a pop-up view.
Config for the state-switch card inside the vertical-stack:
type: custom:state-switch
entity: template
template: |
  {% set areas = states.light
    | selectattr('name')
    | rejectattr('state','match','unavailable')
    | rejectattr('state','match','unknown')
    | map(attribute='entity_id')
    | map('area_name') | unique | reject('none') | list %}
  {% if "Cuisine" in areas %}
   true
  {% endif %}
states:
  'true':
    type: custom:bubble-card
    card_type: separator
    name: Lumières
    icon: mdi:lightbulb-group

The separator of the state-switch card is not building in the dashboard:

Expected behavior
Result with v1.5.3:

Curiously, with v.1.6.4 we can see the expected behavior when in editing mode:

Desktop (please complete the following information):

OS: Windows 10
Browser: Chrome
Version: I tried on v1.6.0 and v1.6.4 and both version have this bug.

Smartphone (please complete the following information):

Device: iPhoneXR
OS: iOS17.3
Browser: Home Assistant Companion App
Version: v1.6.4

Additional context
Here are some of the errors displayed by the browser console in v1.6.4:

Hi, can you tell me if this is fixed in the new v1.7.1? And thanks for the feedback!
https://github.com/Clooos/Bubble-Card/releases/tag/v1.7.1

Sometimes I can see the state card but it stop working after a while. I also tried with the newest version. I can't get it to work with v1.7.2 and v1.7.3.

Doesn't show the separator card when the state is changed. Shows in editing mode but not on dashboard.
Log:
ogger: frontend.js.latest.202403060
Source: components/system_log/init.py:306
First occurred: March 24, 2024 at 11:07:27 PM (321 occurrences)
Last logged: 12:04:44 AM
Uncaught error from WebKit 605.1.15 on iOS 16.7.7 TypeError: undefined is not an object (evaluating 'e.states') /hacsfiles/Bubble-Card/bubble-card.js:1:4363 k (/hacsfiles/Bubble-Card/bubble-card.js:1:4598) /hacsfiles/Bubble-Card/bubble-card.js:520:22386 updateBubbleCard (/hacsfiles/Bubble-Card/bubble-card.js:520:24069) hass (/hacsfiles/Bubble-Card/bubble-card.js:520:1103) connectedCallback (/hacsfiles/lovelace-state-switch/state-switch.js:1:18872) this (src/scoped-custom-element-registry.js:248:47) insertBefore ([native code]::) t (src/lit-html.ts:1454:43) i (src/lit-html.ts:1563:23)
Uncaught error from WebKit 605.1.15 on Mac OS 10.15.7 TypeError: undefined is not an object (evaluating 'e.states') /hacsfiles/Bubble-Card/bubble-card.js:1:4363 k (/hacsfiles/Bubble-Card/bubble-card.js:1:4598) /hacsfiles/Bubble-Card/bubble-card.js:520:22386 updateBubbleCard (/hacsfiles/Bubble-Card/bubble-card.js:520:24069) hass (/hacsfiles/Bubble-Card/bubble-card.js:520:1103) connectedCallback (/hacsfiles/lovelace-state-switch/state-switch.js:1:18872) this (src/scoped-custom-element-registry.js:248:47) insertBefore ([native code]::) t (src/lit-html.ts:1454:43) i (src/lit-html.ts:1563:23)
Uncaught error from Chrome <IP_ADDRESS> on Mac OS 10.15.7 TypeError: Cannot read properties of undefined (reading 'states') /hacsfiles/Bubble-Card/bubble-card.js:1:4364 k (/hacsfiles/Bubble-Card/bubble-card.js:1:4598) /hacsfiles/Bubble-Card/bubble-card.js:520:22385 V.updateBubbleCard (/hacsfiles/Bubble-Card/bubble-card.js:520:24069) set hass (/hacsfiles/Bubble-Card/bubble-card.js:520:1087) pt.connectedCallback (/hacsfiles/lovelace-state-switch/state-switch.js:1:18877) apply (src/scoped-custom-element-registry.js:248:41) insertBefore (src/lit-html.ts:1411:52) k (src/lit-html.ts:1454:35) $ (src/lit-html.ts:1563:11)
Uncaught error from Chrome <IP_ADDRESS> on Mac OS 10.15.7 TypeError: Cannot redefine property: googletag :1:196 :1:524
Uncaught error from WebKit 605.1.15 on Mac OS 10.15.7 TypeError: null is not an object (evaluating 't.popUp.classList') /hacsfiles/Bubble-Card/bubble-card.js:520:1626

The next release should finally fix that, I will release it in beta at first in a week or two.


[AutoComplete] with 16.6 and 16.5 autocomplete overflow scrolls down but not up

Problem description
Autocomplete dropdown overflow scrolls down but not up with v 16.6 and 16.5
That's a showstopper.
I've had to revert to v 16.4 to avoid the problem.
                <AutoComplete
                  style = {{width:'100%'}}
                  ref={'autocomplete'}
                  floatingLabelText="type in a key word, then select a list item"
                  filter={AutoComplete.caseInsensitiveFilter}
                  dataSource={this.findAspectChartLookups || []}
                  dataSourceConfig = {{text:'name',value:'value'}}
                  fullWidth = {true}
                  menuStyle = {{maxHeight:"300px"}}
                  openOnFocus = {false}
                  maxSearchResults = {60}
                  onNewRequest = {this.findOnNewRequest}
                  onUpdateInput = {this.findOnUpdateInput}
                />

Versions

Material-UI: 16.6, 16.5, problem absent with 16.4
React: 15.4
Browser: Chrome (mac)

I'm not sure to understand where is the issue.
Could you provide a live reproduction test case? That would help a lot 👷 .
You can use this playground to do so: http://www.webpackbin.com/Vy6jAlXbG.

You can't reproduce or you don't understand?
Quick try to clarify, in case it's the latter: I can scroll down with 16.6, but I can't scroll up. So I can scroll to the bottom of the list, but I can't scroll back to the top.
If you get an over-long list of autocomplete options with max height as I've set, you should be able to reproduce. (I'm presuming you have such a thing readily available).
Would you like a video of the problem? (it's really dead simple - can scroll down, but can't scroll up)
Another symptom: If I place the mouse over the slider of the dropdown, then scroll up works. But if I place the cursor over the body, then scroll up doesn't work. Apparently not listening to scroll up, or some side effect.
Learning the playground and reproducing there not really time I have at the moment...

here's a video of the bug https://youtu.be/ejMddSAVeuw
lemme know when you're done and I'll take the video down.
Hope this helps

That regression is linked to https://github.com/callemall/material-ui/pull/4168.
In your case, it's the Popover shadow dom that handle the scroll instead of the Menu.
You need to set the overflowY: 'auto' or use the menuProps.maxHeight property.

Excellent. Glad it's been clarified. Thanks for looking into it. I think I'll stick with 16.4 for a bit...

You are welcomed. I'm closing the issue. It's more like a breaking changes we haven't anticipated.

You are welcomed. I'm closing the issue. It's more like a breaking changes we haven't anticipated.

@oliviertassinari to confirm I've added  overflowY:'auto' to menuStyle, and that resolved the issue (can now scroll up as well as down). I already had maxHeight set, and that had not by itself resolved the problem. So now I have:
   menuStyle = {{maxHeight:"300px",overflowY:'auto'}}

I don't understand why you'd consider this a 'feature' though. The widget fails out of the box without this workaround, which I would think of as a bug.
IAC thanks for the help!

I already had maxHeight set

I was referring to the component property, not the CSS property.

I don't understand why you'd consider this a 'feature' though.

Because I don't think that we can fix without introducing another breaking change.
When overriding component styles, everything may break.
There is no clear contract nor guarantee that the style override is going to behave the same way.


udated test

Coverage increased (+1.1%) to 37.931% when pulling 6a8439c8d19c7d44dd206111460a07bfaa87cad9 on JeffBarrette:pycon2016 into a5390a50b6e8ab83c47c6c64c379c4daf10aa4dc on keeppythonweird:master.


feat: manage groups through database

resolves #103

Codecov Report
Base: 28.17% // Head: 29.64% // Increases project coverage by +1.47% :tada:

Coverage data is based on head (643a7d8) compared to base (9699f5f).
Patch coverage: 45.61% of modified lines in pull request are covered.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #112      +/-   ##
==========================================
+ Coverage   28.17%   29.64%   +1.47%
==========================================
  Files          23       25       +2
  Lines        1377     1339      -38
==========================================
+ Hits          388      397       +9
+ Misses        989      942      -47

Impacted Files
Coverage Δ

src/lib/groups.py
76.66% <22.22%> (+41.24%)
:arrow_up:

src/lib/datahandler.py
20.44% <23.52%> (-1.24%)
:arrow_down:

src/lib/database/groups_database.py
51.16% <51.16%> (ø)

src/lib/database/groups_db_init.py
66.66% <66.66%> (ø)

src/lib/constants.py
100.00% <100.00%> (ø)

src/lib/tamer.py
14.42% <100.00%> (-0.82%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Api Memory Manipulation Test Suite

Added a new test suite that uses APIs that manipulates Entity memory contents to verify the coupling of these various pieces all work together end-to-end. The basic pieces came from our Pizza Demo and the test scenario follows a simplified version of those demos.

Merging into Master now as I got no response and Matt is out of office till Monday and I have another PR to submit. Will entertain any late comments and fix later if need be.

:tada: This PR is included in version 0.361.5 :tada:
The release is available on:

npm package (@next dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


[Feature Request] Create a benchmarking instance on Digital Ocean

Current Behavior

Currently benchmarking works very randomly.
Desired Behavior

Need to create a benchmarking place.
Screenshots / Mockups

Alternatives

techempower works well.


Padding

so far the input sentences need to be at least context_length. need to have padding to make sure model takes in at least a context_length (esp. GPT)

fixed in commit 131a9614963402fe0794795159b31571be708b10
turns out i don't really need padding. initially i set the attention mask using register_buffers, which means that if i set context _length = 8, the attention mask will also be of size 8, 8 although the current input is maybe less than that

fixed in commit 131a9614963402fe0794795159b31571be708b10
turns out i don't really need padding. initially i set the attention mask using register_buffers, which means that if i set context _length = 8, the attention mask will also be of size 8, 8 although the current input is maybe less than that

turns out i need it for finetuning lol

2 things needs to be considered:

make sure the padding embedding doesn't get updated (as simple as setting padding_idx in the embedding layer)
mask positions that are padding tokens during attention layer (this one might be alittle bit tricky)


WIP: Feature paymentorders first draft

An attempt to add support for the PaymentOrders api

🎉
Thanks for an awesome contribution!
You should run yarn generate-docs to generate some documentation. You don't have to do it before you are ready with this PR :)

Sure, I still has some work to do before I open this a pull request ready for review. I opened it as a draft so you and others can see what I've done and are planning to do.

Understood, it looks great! :)

This is way harder than I thougt it would be. Setting up classes and such is a bit confusing. Is there a guide or reference for how to do it correctly? 😂

Learning TypeScript may be a good idea, @NullableInt. 😃

Learning TypeScript may be a good idea, @NullableInt. 😃

Time to read!

@NullableInt What do you think, maybe we could merge this so we can start to iterate on it? :)

@cobraz sure, go ahead! I'll let you press the button 😄

It looks good. There is a lot of changes caused by the fact that you are adding so many entities. I have some ideas that I'd like to discuss with you about the interfaces/entities if you are available.

Just tell me if you want a meeting, or contact me on slack!

It looks good. There is a lot of changes caused by the fact that you are adding so many entities. I have some ideas that I'd like to discuss with you about the interfaces/entities if you are available.

Just tell me if you want a meeting, or contact me on slack!


fix: use networking.k8s.io/v1 Ingress instead of extensions/v1beta1

Removes warnings like

W0402 10:55:31.309809   17207 warnings.go:70] extensions/v1beta1 Ingress is deprecated in v1.14+, unavailable in v1.22+; use networking.k8s.io/v1 Ingress

Fixes #172

Are we not caring about kube clusters <1.18 now ?

I wouldn't care about <1.18, see here for the arguments.

Hmm, 1.18.17 also has problems with networking/v1 :thinking:

networks.k8s.io/v1 was introduced in Kubernetes v1.15+, the Ingress resource was promoted to v1 only in Kubernetes v1.19+

So, we need to bump the requirement to 1.19 to use v1.
However, with the way versioning works in k8s (stored version, served version), you can use any version to read it:

When you create an ingress object, it can be read via any version (the server handles converting into the requested version).  (https://github.com/kubernetes/kubernetes/issues/94761#issuecomment-691982480)

The end goal is for the epinio cli to only talk to the epinio server and not to the Kube api directly. That means, any warnings will only appear on the epinio server pod and the user won't see them. I'm closing this.

For as long as we still support <=1.18 : https://github.com/epinio/epinio/pull/354


Breaking change in v2.2.0 of MDX Loader

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[ ] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
The 2.2.0 release for Mermaid introduced a breaking change to the type for Options in @docusaurus/mdx-loader.
This commit/line: https://github.com/facebook/docusaurus/commit/9c92a79d23533a61f3ad1bf3ea4d0889f0e2710f#diff-545a046200b390be351720ea1e83af42ec7ed096a4351213383e1550467d8cdcR66
markdownConfig should be an optional type or have a default value assigned, to align with semver.
Reproducible demo
No response
Steps to reproduce

Use @docusaurus/mdx-loader in a plugin/app pre 2.2.0
Upgrade to 2.2.0
Observe: Build breaks with error:

[ERROR] TypeError: Cannot read properties of undefined (reading 'mermaid')
    at Object.mdxLoader (/Users/ben/flagsmith/docs/node_modules/@docusaurus/mdx-loader/lib/loader.js:115:43)
    at LOADER_EXECUTION (/Users/ben/flagsmith/docs/node_modules/loader-runner/lib/LoaderRunner.js:132:14)
    at runSyncOrAsync (/Users/ben/flagsmith/docs/node_modules/loader-runner/lib/LoaderRunner.js:133:4)
    at iterateNormalLoaders (/Users/ben/flagsmith/docs/node_modules/loader-runner/lib/LoaderRunner.js:250:2)
    at /Users/ben/flagsmith/docs/node_modules/loader-runner/lib/LoaderRunner.js:223:4
    at /Users/ben/flagsmith/docs/node_modules/webpack/lib/NormalModule.js:834:15
    at eval (eval at create (/Users/ben/flagsmith/docs/node_modules/tapable/lib/HookCodeFactory.js:33:10), <anonymous>:12:1)
    at processTicksAndRejections (node:internal/process/task_queues:83:21)
    at runNextTicks (node:internal/process/task_queues:65:3)
    at processImmediate (node:internal/timers:437:9)

Can be seen here: https://github.com/cloud-annotations/docusaurus-openapi/issues/222
Expected behavior
Breaking changes should be reserved for major release cycles. markdownConfig should either be optional or have a default value assigned as its value.
Actual behavior
Build process breaks when upgrading to 2.2.0.
Your environment

Docusaurus version used: v2.2.0
Environment name and version (e.g. Chrome 89, Node.js 16.4): Chrome Version 107.0.5304.87 (Official Build) (arm64), Node.js v18.11.0
Operating system and version (e.g. Ubuntu 20.04.2 LTS): Mac OS 13.0

Self-service

[X] I'd be willing to fix this bug myself.

We do assume only content plugins use mdx-loader, because it's not a documented public API. Changes to its API surface are not a breaking change. Is it possible for docusaurus-openapi to add markdownConfig: siteConfig.markdown, to its mdx-loader options?

Thanks for the reply @Josh-Cena. Yes, we can default a value for markdownConfig for this plugin to resolve the issue.
While I would expect changes to an undocumented API surface, I would not expect API additions to result in broken builds/types. This seems problematic as an unknown number of plugins could be impacted by this, without much visibility to this change in the CHANGELOG/commits.
This issue seems resolvable at both ends. The plugin can backfill this option to add support for 2.2.0. Docusaurus could either make markdownConfig optional (it appears to only be used for the mermaid feature currently?) or assign an empty object. Would you accept a contribution to make this change to docusaurus and if so, which one would you prefer? Alternatively if you have another suggestion, let me know.

I would maintain the ground that undocumented APIs are internal and we shouldn't send the message that we care about breaking users of these internal APIs. This is stated in the docs:

For non-theme APIs, any documented API is considered public (and will be stable); any undocumented API is considered internal.

We do normalize the siteConfig.markdown field and apply default values, so as long as you pass this field into markdownConfig, mdx-loader should be happy. It's a relatively easy fix. If it's not easy to work around, I might be okay with changes on our side; but as it stands, I would prefer no changes to our code.

Ok cool. I think we can close out this issue then 👍 I have a viable change to the plugin and others that run into a similar issue can come across this issue as a reference.
Thanks for the quick follow-up!

Welcome! Thanks for letting us know :)

Added in the Mermaid PR that we recommend third-party plugins using mdx loader to forward the new markdownConfig option:
const loader = {
  loader: require.resolve('@docusaurus/mdx-loader'),
  options: {
    markdownConfig: siteConfig.markdown,
  },
}

More options will be added later, to control the global markdown behavior of your site

While I understand your reasoning regarding private API changes, it does seem like an easily avoided breaking change (for plugins using mdx-loader directly).
Anyway, could you at least update the documentation for the mdx-loader to indicate this value and that it is required?
It is missing here:
https://github.com/facebook/docusaurus/blob/main/packages/docusaurus-mdx-loader/README.md

handled in https://github.com/facebook/docusaurus/pull/8419


Storing a large data set for later use

Hi There @lvx3 ,
I'm trying to store a large request from mysql database for ~ 3 minutes to use in other calculations (the query has about 200k lines) it it seems to work very very slow. ( 30-50 seconds to store it in the cache)
Does that make sense?

Hi @guypaskar
This is likely down to the way the data is serialised before being stored. I'm afraid I don't really have any good advice other than to try another caching library.
I wrote this as a wrapper around an existing wrapper so I could cache relatively small pieces of data where keeping load off the database was more critical than overall performance/speed - so it's very possible that it might just not quite fit your needs.


Server certificate Subject Alternative Name of type IP Address is not taken into account in server name verification

Driver version
JDBC driver version (e.g. 12.8.1).
SQL Server version
Microsoft SQL Server 2022 (RTM-CU15-GDR) (KB5046059) - 16.0.4150.1 (X64)
Sep 25 2024 17:34:41
Copyright (C) 2022 Microsoft Corporation
Developer Edition (64-bit) on Linux (Ubuntu 22.04.5 LTS)
JAVA/JVM version
21.0.4
Problem description
Server certificate subject alternative name of type iPAddress is not taken into account in server name verification.
Expected behavior
If server address is entered as IP address and server certificate contain subject alternative name extension with that IP address defined connection attempt will be successful.
Actual behavior
Server name check fails.
Error message/stack trace
javax.net.ssl.SSLHandshakeException: Failed to validate the server name "<IP_ADDRESS>"in a certificate during Secure Sockets Layer (SSL) initialization. Name in certificate "<DNS_NAME>"
        at java.base/sun.security.ssl.Alert.createSSLException(Alert.java:130)
        at java.base/sun.security.ssl.TransportContext.fatal(TransportContext.java:378)
        at java.base/sun.security.ssl.TransportContext.fatal(TransportContext.java:321)
        at java.base/sun.security.ssl.TransportContext.fatal(TransportContext.java:316)
        at java.base/sun.security.ssl.CertificateMessage$T12CertificateConsumer.checkServerCerts(CertificateMessage.java:647)
        at java.base/sun.security.ssl.CertificateMessage$T12CertificateConsumer.onCertificate(CertificateMessage.java:467)
        at java.base/sun.security.ssl.CertificateMessage$T12CertificateConsumer.consume(CertificateMessage.java:363)
        at java.base/sun.security.ssl.SSLHandshake.consume(SSLHandshake.java:393)
        at java.base/sun.security.ssl.HandshakeContext.dispatch(HandshakeContext.java:476)
        at java.base/sun.security.ssl.HandshakeContext.dispatch(HandshakeContext.java:447)
        at java.base/sun.security.ssl.TransportContext.dispatch(TransportContext.java:201)
        at java.base/sun.security.ssl.SSLTransport.decode(SSLTransport.java:172)
        at java.base/sun.security.ssl.SSLSocketImpl.decode(SSLSocketImpl.java:1506)
        at java.base/sun.security.ssl.SSLSocketImpl.readHandshakeRecord(SSLSocketImpl.java:1421)
        at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:455)
        at java.base/sun.security.ssl.SSLSocketImpl.startHandshake(SSLSocketImpl.java:426)
        at com.microsoft.sqlserver.jdbc.TDSChannel.enableSSL(IOBuffer.java:1854)
        ... 53 more
Caused by: java.security.cert.CertificateException: Failed to validate the server name "<IP_ADDRESS>"in a certificate during Secure Sockets Layer (SSL) initialization. Name in certificate "<DNS_NAME>"
        at com.microsoft.sqlserver.jdbc.SQLServerCertificateUtils.validateServerNameInCertificate(SQLServerCertificateUtils.java:269)
        at com.microsoft.sqlserver.jdbc.HostNameOverrideX509TrustManager.checkServerTrusted(SQLServerTrustManager.java:99)
        at java.base/sun.security.ssl.AbstractTrustManagerWrapper.checkServerTrusted(SSLContextImpl.java:1430)
        at java.base/sun.security.ssl.CertificateMessage$T12CertificateConsumer.checkServerCerts(CertificateMessage.java:631)
        ... 65 more

Any other details that can be helpful
com.microsoft.sqlserver.jdbc.SQLServerCertificateUtils.validateServerNameInCertificate
takes only constant 2 (DNS name) into account
JDBC URL: jdbc:sqlserver://<IP_ADDRESS>:<PORT>;databaseName=<DB_NAME>;socketTimeout=60000;encrypt=true;trustServerCertificate=false;trustStore=<PATH_TO_TRUSTSTORE>;trustStorePassword=<TRUSTSTORE_PWD>
possible work around using parameter hostNameInCertificate with DNS name used in certificate

hi @d-richter-qdt
I'm not sure what the question is, did the hostNameInCertificate property work for you? That is what this property is used for, it can be used to specify the host name in situations where the name, or names, used in the certificate doesn't match the name passed in to the serverName property. For more information on descriptions of connection properties for the driver, please see the doc Setting the connection properties.

Hello @lilgreenbird,
thank you for your response. It is not question but bug report. I believe that driver should be able to connect without using that property in this case, as the certificate contains IP address of DB server in subjAltName extension (Even tho I agree it is kind of an edge case). As to your question yes the property works.
To be exact:
Here: https://github.com/microsoft/mssql-jdbc/blob/ffa5e1fa9cc1a490ed10d86b39a7c8b887488d89/src/main/java/com/microsoft/sqlserver/jdbc/SQLServerCertificateUtils.java#L224
is missing handling of key == 7 https://docs.oracle.com/javase/6/docs/api/java/security/cert/X509Certificate.html#getSubjectAlternativeNames()

thank you, I will add this as an enhancement request


[opentitantool] Allow changing which pins to use for JTAG

HyperDebug by default does JTAG on five particular pins.  However, as it is all bit-banging with no true hardware support, it is able to use almost any combination of pins.
This PR introduces a transport-specific command to "move" the JTAG functionality to a different set of pins.
Usage:
opentitantool --interface=hyperdebug transport set-jtag-pins <TCLK> <TMS> <TDI> <TDO> <TRSTn>

Thanks Jes. We are still testing this.

The code looks good. Maybe keep the PR open until we've validated it works?

Looks good to me. Is proxy support necessary?

Maybe at some point.  For now, proxy support is not implemented for any JTAG functionality.

The code looks good. Maybe keep the PR open until we've validated it works?

Yes, I will wait for you Ben, to try this out on real hardware, before merging.

@BenBenderOT have you had a chance to try this out on actual hardware?

@jesultra we tried setting to pins we needed. Still unable to confirm it worked. Hopefully today or tomorrow.

I'm using the "hyper310" interface instead of the "hyperdebug" interface. Should that make a difference? In any case, as soon as I try to set the pins to their default settings using the command, no signals are seen anymore and examination fails.

I'm using the "hyper310" interface instead of the "hyperdebug" interface. Should that make a difference? In any case, as soon as I try to set the pins to their default settings using the command, no signals are seen anymore and examination fails.
That was unexpected.
It should not matter that you use --interface=hyper310, the JTAG support is identical between that and --interface=hyperdebug.

It is hard for me to understand, how setting JTAG pins to their default setting would cause JTAG to stop working.  Looking at the HyperDebug firmware code, I see a bug that can be triggered if you first do some JTAG communication, then switches JTAG pins, and then tries JTAG communication again.  Could you try hitting the black reset button on your HyperDebug board, and then seeing if OpenOCD can connect? Assuming that works, then please reset again, then set the pins to their default value, and then see if OpenOCD can connect.  Also, could you post the exact command you use to the pins to their default?  This one should do it, but using aliases of the given pins should result in the same: opentitantool --interface=hyperdebug transport set-jtag-pins CN7_1 CN7_7 CN7_3 CN7_5 CN7_16

This feature did not work due to a silly bug in the HyperDebug firmware.  With the below fix, I have observed JTAG on my CW310 work with some jumper wires to non-standard pins on HyperDebug.  In plan on merging this, once the updated firmware has been included in opentitantool.
https://chromium-review.googlesource.com/c/chromiumos/platform/ec/+/5568495


Avoid pascal casing in function names

Pascal casing should be avoided when naming functions. Instead, use descriptive and meaningful names that follow standard conventions.
For example: instead of CheckFormat, it should be checkFormat.
See utils folder

This PR has been created. @lennyAiko please check


Translate to other languages

A request to transifex is out

We now have a transifex account at https://www.transifex.com/none-581/zapp-android-app/
Help is always welcome!


Is there a way to implement this with TeamCity CI?

Can you spawn several CI containers in parallel on TeamCity?

@bahmutov I think it should be plausible using this functionality: https://www.jetbrains.com/help/teamcity/parallel-tests.html#custom-tests

TC has a separate feature called Parallel tests I believe


feat(fx-2639, fx-2650): adds artist nationalities filter

The type of this PR is: Feature
This PR resolves FX-2639
This PR resolves FX-2650
Description
https://user-images.githubusercontent.com/112297/119173103-c3254f00-ba34-11eb-8856-da447cf7bc55.mov
PR Checklist (tick all before merging)

[x] I have included screenshots or videos to illustrate my changes, or I have not changed anything that impacts the UI.
[ ] I have tested my changes on iOS and Android.
[x] I have added tests for my changes, or my changes don't require testing, or I have included a link to a separate Jira ticket covering the tests.
[x] I have added a feature flag, or my changes don't require a feature flag. (How do I add one?)
[x] I have documented any follow-up work that this PR will require, or it does not require any.
[x] I have added an app state migration, or my changes do not require one. (What are migrations?)
[x] I have added a CHANGELOG.yml entry or my changes do not require one.

I haven't tried booting up the android app in a while but I can give it a shot if that's blocking a merge.

I don't think it should be a problem — This is basically just copy paste from previous filters I've done. I mostly just think that this should be part of Q/A.

I don't think it should be a problem — This is basically just copy paste from previous filters I've done. I mostly just think that this should be part of Q/A.

Just took a look in the iOS simulator. Looks great to me :+1:


option to add horizontal line between negative & positive bar chart.

please add option to add horizontal line at 0 of x axis in bar chart. so that negative and positive bars look sperate

Follow up #476


Conditional measure fixes

Partially resolves #2567 (not the drawing issue) by the following:

Preventing randomized tests from generating conditional instructions (as they are unsupported by devices and Aer).
Allows conditional measures to be added to circuits, even if the measure will overwrite the bit used to control the measure (resolving the self-edge reported in #2567).
Raises an error when a user attempts to make a barrier conditional, as barrier is a compiler directive and cannot be altered by a run-time value.
Raises an error when unrolling a conditional circuit if that circuit would modify its controlling bits. (We could reconsider better supporting this in the future, if we end up with simulator or device support.)

Raises an error when a user attempts to make a barrier conditional, as barrier is a compiler directive and cannot be altered by a run-time value.

Can you do the same for snapshot


adi_driver: 1.0.3-0 in 'indigo/distribution.yaml' [bloom]

Increasing version of package(s) in repository adi_driver to 1.0.3-0:

upstream repository: https://github.com/tork-a/adi_driver.git
release repository: https://github.com/tork-a/adi_driver-release.git
distro file: indigo/distribution.yaml
bloom version: 0.6.8
previous version for package: 1.0.2-0

adi_driver
* Change to depend on imu_tools(#19 <https://github.com/tork-a/adi_driver/issues/19>)
  - Depend on imu_filter_madgwick and rviz_imu_plugin
* Add bias estimation service document(#20 <https://github.com/tork-a/adi_driver/issues/20>)
  - Update README.md
  - Fix incorrect sentence.
* Contributors: Ryosuke Tajima

Holding for sync


web domain specific theme preference

Currently the selected theme is applied to every tab.
We'd want to have the selected theme applied to only those websites that user has selected his/her preference.

@sharma66mahesh should be pretty straightforward i think. we just need to save the theme preference and load it per domain...
the only thing we need to figure out is how we are going to display this feature in the ui... how will the user enable/disable theming for a certain site but enable for other sites without a lot of hassle? also, will there be a reset button to reset all user preferences?
let me know if you have any ideas. we can organise a session to create some mockups for this if needed.

PR for this feature here: https://github.com/sharma66mahesh/ShadowShift/pull/3


Write lumina-fileinfo manual page.

lumina-fileinfo is used to view the properties of specific files. It also includes a simple XDG application registration creator.

Ken plans to combine lumina-fileinfo and lumina-xdg entry, so both these manual pages will be delayed until that work is complete.

lumina-fileinfo is being rebuilt: option flags will be removed, xdg desktop entry creation is incorporated (making lumina-xdgentry obsolete), and a new process to create and save a writable copy of a read only desktop entry file.

Ken put his changes in before he left for vacation last week. We should be good to go.


cargo install fails with cargo v1.80

This fails when installing cargo audit with cargo v1.80 due to an issue with the locked version of the time dependency. Removing --locked from the install command allows the command to work.
cargo install cargo-audit --vers 0.20.0 --no-default-features

Installing with --locked was previously necessary since otherwise v0.20 would have runtime problems #72.
It seems that currently it works without --locked. I will push an update.


Accordion end-to-end test

Summary of changes
Accordion end-to-end test
Context and reason for changes
Checked whether the correct signals and attributions are shown on the audit detailed view column. The e2e test output file is also attached to avoid having 2 identical attributions. Note: Please avoid uploading or modifying the e2e output file in the future as it might cause error in the e2e test
Fix: #950
Signed-off-by: Billwuzl<EMAIL_ADDRESS>Note: Please review the guidelines for contributing to this repository.

Looks good to me. Just squash commits.


[WIP] [DO NOT MERGE] In-flight kernel updates for various hardware platforms in our pipeline

This is a WIP replacement for #1149 (which I intend to close soon). It'll hang out here for now (while I debug various hardware platforms and come up with a best set of kernel settings for them). I plan to split things off from this PR and send separate PRs as time goes on -- but for now it is convenient to have one over here.

[the hostess took up the broom]
@rvs do you plan to return to this or we can close it? and btw why "do not merge"?
I try to go through old opened prs to understand what makes sense and what we can safely close.

@eriknordmark can we close this? once needed - we know where to find and reopen.


Created Information about DFS Algorithm

I have added information about DFS Algorithm with Program.

highlight heading and add 100+ lines , edit .md file to hight codes and headings


Move prop-dependent styles to variables

After reading your recent suggestion @Kikobeats (https://github.com/microlinkhq/microlinkjs/pull/72#pullrequestreview-98148844), I see it better-structured this way. So I've moved some of the other style blocks to follow suit.


remove all occurences of embeds from docs

@nikolasburk is there a reason why this has been removed? I know it was not implemented, but i was expecting that it will be released soon. i think embeds are important, what is the state of that?

Hey @macrozone 👋 embedded types are currently not on the roadmap unfortunately 😐
We currently don't have an open feature request for embedded types, if that's something you're looking for, please open a FR request here 🙂

hi @nikolasburk thank you, i opened e FR https://github.com/prisma/prisma/issues/3014


[Bug]: Code execution does not break in debugging tool when watch mode is set

Version
29.6.1
Steps to reproduce

Clone my repo at https://github.com/cdoublev/jest-issue
node --inspect-brk=<IP_ADDRESS>:9222 ./node_modules/.bin/jest --watchAll (npm test)
Open chrome://inspect/ in Chrome
Click inspect in the target list

Expected behavior
I expect the code to break at the debugger line in ./add.js.
Actual behavior
The code did not break.
Additional context
I run the above steps in a virtual machine. I connect to the VM from my terminal with a SSH tunnel localhost:9222 -> <IP_ADDRESS>:9222.
I have not checked if the problem occurs without the SSH tunnel because I am using a virtual machine so I do not have to install NodeJS on my machine's OS.
But I guess this problem is specific to this context, and is related to this change in version 29.6.0:

[jest-core] Always use workers in watch mode to avoid crashes (#14059)

Indeed, the code breaks when removing --watchAll from the command line arguments.
Environment
System:
  OS: Linux 6.1 Debian GNU/Linux 12 (bookworm) 12 (bookworm)
  CPU: (1) x64 Intel(R) Core(TM) i7-3612QM CPU @ 2.10GHz
Binaries:
  Node: 20.3.0 - /usr/bin/node
  npm: 9.6.7 - /usr/bin/npm
npmPackages:
  jest: ^29.6.1 => 29.6.1

Seems like you are right about #14059. After it landed workers are used always instead of sometimes in watched mode. This means that your reproduction now fails always, but before it was failing sometimes. If I didn’t miss something. Might be that all worked as expected with --runInBand flag, but you are not mentioning it.
All examples in debugging documentation use the --runInBand flag. The CRA example additionally sets --watchAll=false.
The configuration examples in vscode-jest repo also set --runInBand and --watchAll=false.
All this makes me think that it can be that the watch mode is not designed to work with debugger. For some reason. Can’t proof that.
@benjaminjkraft Could you take a look? I can’t recall if we did talk about respecting --runInBand in the watch mode? Does that make sense?

So the watch mode and debugging tools only worked together if --runInBand is set or if only one available core is detected? Curious to know why. There is nothing in the current version (v29.5.0) of the documentation about this.
The VM I am running Jest in has only one core (Virtual Box's default) therefore it always worked.


feat(MetricSpace/Ultra): TotallyDisconnectedSpace X when X is ultrametric

LGTM, thanks! Please fix the formatting comment above + then I'll send it on for merging

Thanks!
maintainer merge


can u please add an option for select tcp or udp?

that would be a lot helpful to me, thanks.

Can u please...
https://github.com/Nyr/openvpn-install/issues/174
https://github.com/Nyr/openvpn-install/issues/147
https://github.com/Nyr/openvpn-install/issues/130
https://github.com/Nyr/openvpn-install/pull/111
https://github.com/Nyr/openvpn-install/issues/110
https://github.com/Nyr/openvpn-install/pull/58

sorrymybad


refactor: simpler logic for border width

Since there will be showLastColumnBorderRight arg to the border func added with the column width, it started to become a lot of if statements. Turns out you could simplify it quite a lot

:tada: This PR is included in version 2.2.0 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Use describe_instance_types to get efa enabled instance types

This commit replaces the usage of the features json file with a call to describe_instanceTypes
to get the list of efa enabled instance types.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Codecov Report

Merging #1835 into develop will decrease coverage by 0.11%.
The diff coverage is 18.18%.

@@             Coverage Diff             @@
##           develop    #1835      +/-   ##
===========================================
- Coverage    54.28%   54.17%   -0.12%
===========================================
  Files           28       28
  Lines         4421     4430       +9
===========================================
  Hits          2400     2400
- Misses        2021     2030       +9

Impacted Files
Coverage Δ

cli/pcluster/utils.py
46.79% <10.00%> (-1.06%)
:arrow_down:

cli/pcluster/config/validators.py
83.54% <100.00%> (-0.04%)
:arrow_down:

Continue to review full report at Codecov.

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

Nice!


Add swift-log and swift-async-algorithms to checkouts

These repositories are used by https://github.com/apple/swift-sdk-generator, which needs to be built as a part of Swift CI jobs for creating a Swift SDK for WASI for CI testing purposes.

Please also update rebranch and next schemes

@swift-ci smoke test


Requirements have not been properly satisfied: MacOS Sierra

Keep on getting the same issue even after re installing everything.
ERROR: Requirements have not been properly satisfied. Please try running:
$ sudo pip install -r requirements.txt
If you still get the same error, please submit an issue here:
https://github.com/k4m4/kickthemout/issues
Requirement already satisfied: pcapy in /usr/local/lib/python2.7/site-packages
$ brew install libdnet scapy
Updating Homebrew...
Warning: libdnet-1.12 already installed
Warning: homebrew/python/scapy-2.3.3 already installed
Doing something wrong?

Check out Issue #42 and Issue #13. Please let me know if you don't manage to resolve your issue. Thanks!


API lacks a means to retrieve emitter flow

The EN_getnodevalue API function with EN_DEMAND as its property argument retrieves a node's current demand flow which consists of both the regular consumer demand and any emitter flow associated with the node. It would be useful to also be able to retrieve just the emitter flow, especially when using emitters to model fire flow or leakage. This is easily accomplished by adding a new node property named EN_EMITTERFLOW  for which EN_getnodevalue will return the value in the existing EmitterFlow array (in the Hydraul struct).

This issue was addressed by PR #739 and can be closed.


llvm_asm! -> asm!

Tiny change to stop using llvm_asm!, as everywhere else is using asm! and the feature is basically deprecated.

Thanks a lot!

Published together with #155 as v0.10.2.

Could you backport it to v0.9?


support for channel claims in web URL

The Issue
A url like https://beta.lbry.tv/@jiggytom/jpgfile should resolve (also may have something like @jiggytom:123/jpgfile when we get canonical URLs)
System Configuration

LBRY Daemon version:
LBRY App version:
LBRY Installation ID:
Operating system:

Anything Else

Screenshots

Closing for #2463