How to link different IlmBase library names according to Debug/Release configuration, when building OpenEXR with CMake + VS2015?

My environment:

Windows 10 version 1803
Visual Studio 2015 Update 3
CMake 3.11.4
dev branch downloaded here.

I want to use different output names of IlmBase libraries for better management according to Debug/Release configuration in VS. Specifically, if built under Debug mode, the output name will be suffixed with "_d", say, Half-2_2_d.lib, Iex-2_2_d.dll. Otherwise, the names are left unchanged. For IlmBase, this is done by the following CMake command line:
cmake -DCMAKE_DEBUG_POSTFIX=_d -DBUILD_SHARED_LIBS=ON -DCMAKE_INSTALL_PREFIX=C:\...\openexr-develop\IlmBase\install_ -G"Visual Studio 14 2015 Win64" ..
Then I proceed to building OpenEXR and problem arises here. What I want is to set the IlmBase library names to be linked to OpenEXR project on the CMake command line only, instead of modifying existing CMakeLists.txt scripts, or changing names manually inside VS afterwards. I searched google for hours but the answers are all using target_link_libraries or add_library with IMPORTED flag, which requires modification of CMakeLists.txt. So my question is, does CMake have a general mechanism to set the suffix of linked library names, like the CMAKE_INSTALL_PREFIX I used above? Is there any flags I can use on CMake command line to specify the suffix of linked libraries for OpenEXR based on Debug/Release configuration, or, aiming at the final goal, how to link different IlmBase library names according to Debug/Release configuration? As a side note, the CMake command line I used to build OpenEXR is as follows:
cmake -DBUILD_SHARED_LIBS=ON -DILMBASE_PACKAGE_PREFIX=C:\...\openexr-develop\IlmBase\install_ -DCMAKE_INSTALL_PREFIX=C:\...\openexr-develop\OpenEXR\install_ -DZLIB_LIBRARY=C:\...\zlib-1.2.11\install\lib\zlib.lib -DZLIB_INCLUDE_DIR=C:\...\zlib-1.2.11\install\include -G"Visual Studio 14 2015 Win64" ..
I tried something like IMPORTED_LOCATION_DEBUG but of no avail.
Thank you for help.

Hi Zhou, if you try the develop branch, there is work in progress to generate debug libraries for windows with a different name than release, and for the debug and release OpenEXR libraries to appropriately link the IlmBase libraries.
There is a bit more work to do to clean it up, In particular, there should probably be ALIAS libraries to simplify linking, but perhaps you can start with what is on the develop branch already.

So, generally speaking, we need to add this feature by manually writing scripts in CMakeLists.txt? I had thought there is some existing option/flag provided by CMake that can do the work on command line.
Thank you for your reply and excellent work on the development of OpenEXT&IlmBase.

You can do it from the command line. After you have gone through the configuration steps, to build Release from the command line -
cmake --build . --target install --config Release -- /maxcpucount:16

Debug -
cmake --build . --target install --config Debug -- /maxcpucount:16

(set cpucount to a realistic number)

Sorry I just noticed the reply made 6 days ago. I didn't quite understand the reply however, perhaps because we are not on the same page. I read some more CMake documentations these days and confirmed my hypothesis that CMake does provide a mechanism for linking different filenames based on Debug/Release configuration. This is written in the doc of target_link_libraries function. As I mentioned in the original post, we can use CMAKE_DEBUG_POSTFIX to differentiate output library filename, but I was stuck by how to differentiate the input library filename. Now I know that this is done by specifying debug or optimized keyword when calling target_link_libraries() function; we can specify different library filenames after these keywords respectively. For example, Field3D are using this mechanism to link different versions of Boost libraries. Of course we need to modify CMakeLists.txt to make use of this mechanism. Is the ongoing work using this way to link to different library filenames according to Debug/Release configuration of OpenEXR?

I thought the following is true, please correct me if I am wrong. I have read the documentation page, and it seems that this is true:

make library Foo
specify debug postfix _d to differentiate name
make library Bar
specify target_link_library of Foo on Bar NOT using debug or optimized keyword
build Bar debug. Foo_d.lib will be linked
build Foo release. Foo.lib will be linked

As commented above, we believe there are all the controls necessary for setting suffixes on the generated libraries. In addition, the cmake system has been overhauled to use the "modern" cmake which should be a bit more robust. Create a new issue if there are any new issues. thank you for the report!


add German QWERTZ layout

@skyspy20 are you working on this?

NO my friend, i got busted  up on other stuffs

From: Aman Deep<EMAIL_ADDRESS>Sent: Tuesday, February 27, 2018 8:40:53 PM
To: hiamandeep/speedcoder-keyboard-layouts
Cc: Surajsing Keenoo; Mention
Subject: [hiamandeep/speedcoder-keyboard-layouts] add German QWERTZ layout (#3)
@skyspy20https://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fskyspy20&data=02|01|sk1813%40live.mdx.ac.uk|a97da25b19bf4372694d08d57e00df4b|38e37b88a3a148cf9f056537427fed24|0|0|636553464557836041&sdata=6FRxgOSd6tjrc0t75gG4lOdZ%2F74tO6GScvWZ6gNItEc%3D&reserved=0 are you working on this?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fhiamandeep%2Fspeedcoder-keyboard-layouts%2Fissues%2F3&data=02|01|sk1813%40live.mdx.ac.uk|a97da25b19bf4372694d08d57e00df4b|38e37b88a3a148cf9f056537427fed24|0|0|636553464557836041&sdata=B7GXK0UIOmMdqJ17O86tGLW3%2BwyI88ndbTdewDcUj1k%3D&reserved=0, or mute the threadhttps://emea01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAfU646GfpQ2NtfO3nzSx-iPe46-xIh5Tks5tZDAOgaJpZM4SB9ku&data=02|01|sk1813%40live.mdx.ac.uk|a97da25b19bf4372694d08d57e00df4b|38e37b88a3a148cf9f056537427fed24|0|0|636553464557836041&sdata=%2FXKd9lWsSc343pgGneMjwxt6mb58xQiSus53fYn7RDY%3D&reserved=0.

I'll take this as part of #Hacktoberfest Paris.

I'd like to contribute the german QWERTZ layout. Is there a way I could test the layout for correctness?


Release Google.Cloud.Batch.V1 version 1.3.0

Changes in this release:
New features

Support custom scopes for service account in v1 (commit 60d746a)
Add boot disk field in InstanceStatus (commit 60d746a)
Add boot disk and image source fields to v1 InstancePolicy (commit 60d746a)

Created release for Google.Cloud.Batch.V1-1.3.0


Error with openliberty-<IP_ADDRESS> Unsatisfied dependencies

I've got an error trying to reproduce your lesson implementing this archetype
my environment is with java se 1.8.0-242
Server creation:
        rm -rf $SERVERS/wlp
        #mkdir $SERVERS/wlp
        unzip $SERVERS/sources/openliberty-<IP_ADDRESS>.zip -d $SERVERS/
#       $SERVERS/wlp/bin/server create defaultServer
        $SERVERS/wlp/bin/server run

this zip is from All GA Features
I change the server.xml file with credentials following: https://www.youtube.com/watch?v=J7x9YlzMxj4&t
But I got stuck here ;c
[ERROR   ] CWWKZ0004E: An exception occurred while starting the application test. The exception message was:
com.ibm.ws.container.service.state.StateChangeException: org.jboss.weld.exceptions.DeploymentException:
WELD-001408: Unsatisfied dependencies for type String with qualifiers @Default
  at injection point [BackedAnnotatedField] @Inject com.airhacks.ping.boundary.PingResource.message
  at com.airhacks.ping.boundary.PingResource.message(PingResource.java:0)
WELD-001475: The following beans match by type, but none have matching qualifiers:
  - Producer Method [String] with qualifiers
[@BatchProperty @Any] declared as [[UnbackedAnnotatedMethod] @Produces @BatchProperty @Dependent
public com.ibm.ws.jbatch.cdi.BatchProducerBean.produceProperty(InjectionPoint)]

Please Help, what am I missing to configure ;c

Fixed adding <feature>microProfile-3.2</feature> on server.xml
I have to add each provided scope https://openliberty.io/docs/ref/feature/#microProfile-3.2.html
I'm new to was, my bad ;c
Hope this help someone new


Autobuild Does Not Work

Steps to reproduce:
About:
Version: 1.52.1 (user setup)
Commit: ea3859d4ba2f3e577a159bc91e3074c5d85c0523
Date: 2020-12-16T16:34:46.910Z
Electron: 9.3.5
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: <IP_ADDRESS>-electron.0
OS: Windows_NT x64 10.0.19041

Run this in wsl:

mkdir .devcontainer
cd .devcontainer
wget -q https://raw.githubusercontent.com/qdm12/latexdevcontainer/master/.devcontainer/devcontainer.json
wget -q https://raw.githubusercontent.com/qdm12/latexdevcontainer/master/.devcontainer/docker-compose.yml

Create foo.tex with this content:

\documentclass{article}

\begin{document}
hello world 123
\end{document}

Click on "Build LaTeX" project in LaTeX Workshop. It builds the pdf.
Change foo.tex and save the file. Nothing happens.

Probably something needs to be configured.
This setting does not help though:
"latex-workshop.latex.autoBuild.run": "onSave"

Probably something needs to be configured.

I just tried, running on a Windows host with VSCode 1.52.1 and it works. The big difference here is I still run Docker with Docker Desktop in its (hidden/behind the scene) VM and don't use WSL. I believe I had the problem where file changes notifications are not propagated correctly with WSL and Docker containers. I don't think there is really a solution except dropping WSL unfortunately. You could do some more testing with containers and bind mounts.

Run this in wsl:

Tiny FYI you can now use devtainr with
devtainr -dev latex -name projectname

😉

Probably something needs to be configured.

I just tried, running on a Windows host with VSCode 1.52.1 and it works. The big difference here is I still run Docker with Docker Desktop in its (hidden/behind the scene) VM and don't use WSL. I believe I had the problem where file changes notifications are not propagated correctly with WSL and Docker containers. I don't think there is really a solution except dropping WSL unfortunately. You could do some more testing with containers and bind mounts.

Run this in wsl:

Tiny FYI you can now use devtainr with
devtainr -dev latex -name projectname

😉

The big difference here is I still run Docker with Docker Desktop in its (hidden/behind the scene) VM and don't use WSL.

Sorry, I was not very precise. I only used WSL to download the devcontainer config files.
I opened the folder in a VS Code Dev Container (using Docker).
Surprisingly, it works now. I guess something restarted in between, I don't remember what I did differently.

Tiny FYI you can now use devtainr with

That is a really good idea!
I hope eventually VS Code will offer an easier way to setup dev containers.

The big difference here is I still run Docker with Docker Desktop in its (hidden/behind the scene) VM and don't use WSL.

Sorry, I was not very precise. I only used WSL to download the devcontainer config files.
I opened the folder in a VS Code Dev Container (using Docker).
Surprisingly, it works now. I guess something restarted in between, I don't remember what I did differently.

Tiny FYI you can now use devtainr with

That is a really good idea!
I hope eventually VS Code will offer an easier way to setup dev containers.

using Docker

But are you using Docker with the hyperv VM or WSL? 🧐

Surprisingly, it works now

yes sometimes just rebuilding the container from the VS Code command palette unclogs things 😉

I hope eventually VS Code will offer an easier way to setup dev containers.

Yeah, although it's quite awesome what they've done with dev containers already. Plus there is a lot of different ways to structure your files in that directory. I picked one that I share across my repositories so my tool works great with them, but that's not the case for everyone

using Docker

But are you using Docker with the hyperv VM or WSL? 🧐

Surprisingly, it works now

yes sometimes just rebuilding the container from the VS Code command palette unclogs things 😉

I hope eventually VS Code will offer an easier way to setup dev containers.

Yeah, although it's quite awesome what they've done with dev containers already. Plus there is a lot of different ways to structure your files in that directory. I picked one that I share across my repositories so my tool works great with them, but that's not the case for everyone

I understand now. Yeah, I guess I upgraded to WSL 2 some time ago, so I think my docker installation uses wsl and not hyper-v. I find all this virtualization stuff crazy :D

Yeah, although it's quite awesome what they've done with dev containers already.

Yes! I see a bright future for dev containers!

Plus there is a lot of different ways to structure your files in that directory

You mean how the latex sources are structured? Or how the dev container is structured?

I understand now. Yeah, I guess I upgraded to WSL 2 some time ago, so I think my docker installation uses wsl and not hyper-v. I find all this virtualization stuff crazy :D

Yeah, although it's quite awesome what they've done with dev containers already.

Yes! I see a bright future for dev containers!

Plus there is a lot of different ways to structure your files in that directory

You mean how the latex sources are structured? Or how the dev container is structured?

I upgraded to WSL 2 some time ago, so I think my docker installation uses wsl and not hyper-v. I find all this virtualization stuff crazy

I was a bit disappointed with Docker and WSL 2. It first tried WSL 1 which didn't support Docker so that a big no-no for me. And then WSL 2 has bind mount issues and memory leaks issue too (because of it caching files). So I'm still stuck on the pre-WSL days of running a full hyperv machine (but no setup required though, Docker Desktop handles that for you). And there again, it's not perfect either, I have file permission with bind mounts not working correctly A LOT 😄
I also have a Linux machine which works beautifully with development containers though.

Yes! I see a bright future for dev containers!

Yeah plus there is that Github codespaces I haven't tried yet which is another aspect of it. Having your dev environment in your browser and benefiting from fast cloud hardware is definitely a great thing too.

Plus there is a lot of different ways to structure your files in that directory

No 😄 How to files are structured in the .devcontainer directory.

I upgraded to WSL 2 some time ago, so I think my docker installation uses wsl and not hyper-v. I find all this virtualization stuff crazy

I was a bit disappointed with Docker and WSL 2. It first tried WSL 1 which didn't support Docker so that a big no-no for me. And then WSL 2 has bind mount issues and memory leaks issue too (because of it caching files). So I'm still stuck on the pre-WSL days of running a full hyperv machine (but no setup required though, Docker Desktop handles that for you). And there again, it's not perfect either, I have file permission with bind mounts not working correctly A LOT 😄
I also have a Linux machine which works beautifully with development containers though.

Yes! I see a bright future for dev containers!

Yeah plus there is that Github codespaces I haven't tried yet which is another aspect of it. Having your dev environment in your browser and benefiting from fast cloud hardware is definitely a great thing too.

Plus there is a lot of different ways to structure your files in that directory

No 😄 How to files are structured in the .devcontainer directory.


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

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

/check-enforcer override


[react-query] Type Inference Bug When Defining getNextPageParam Before queryFn in useInfiniteQuery

Describe the bug
When Defining getNextPageParam Before queryFn in useInfiniteQuery (same issue with infiniteQueryOptions)
lastPage is inffered as unknown
It works well
  useInfiniteQuery({
    queryKey: ["data"],
    initialPageParam: 1,
    queryFn: ({ pageParam }) => getData(pageParam),
    getNextPageParam: (lastPage) =>
      lastPage.hasNext ? lastPage.page + 1 : undefined,
  });

It doesn’t work well.
lastPage in getNextPageParam is inffered as unknown
  useInfiniteQuery({
    queryKey: ["data"],
    initialPageParam: 1,
    getNextPageParam: (lastPage) =>
      lastPage.hasNext ? lastPage.page + 1 : undefined,
    queryFn: ({ pageParam }) => getData(pageParam),
  });

Your minimal, reproducible example
https://codesandbox.io/p/devbox/dkhrfg
Steps to reproduce

Go to codesandbox link

Expected behavior
I expect that lastPage is inferred as the return type of getData.
  useInfiniteQuery({
    queryKey: ["data"],
    initialPageParam: 1,
    getNextPageParam: (lastPage) =>
      lastPage.hasNext ? lastPage.page + 1 : undefined,
    queryFn: ({ pageParam }) => getData(pageParam),
  });

How often does this bug happen?
None
Screenshots or Videos
No response
Platform
not affect
Tanstack Query adapter
None
TanStack Query version
latest
TypeScript version
v5.6.3
Additional context
No response

This is a known issue because of how TypeScript currently infers types. We have a lint rule that should catch this:
https://tanstack.com/query/v5/docs/eslint/infinite-query-property-order


Always allow JSON fragments

This updates the public versions of JSONEncoder and JSONDecoder to match the behavior shipped with the stdlib in iOS 13. More specifically, always passing .fragmentsAllowed to JSONSerialization's reading/writing options.
For more information see:

https://bugs.swift.org/browse/SR-6163
https://drewag.me/posts/2019/09/11/json-encoder-change-in-swift-5

Resolves SR-6163.

@shahmishal Are we missing something here in getting the changes properly merged to GitHub?

Let me start testing, after that we can merge it if you're ok with the changes.

@swift-ci test

What I mean is that there were probably other changes made that should have been merged here, but it doesn't appear it ever happened.

Can one of the admins verify this patch?

This pull request can be closed. The issue was fixed in apple/swift#30615 for Darwin.
SR-6163 can remain open for swift-corelibs-foundation, which doesn't support the fragmentsAllowed options yet.


feat(maps): Add Italy regions code to the map generator notebook

SUMMARY
Italy regions geojson file creation was constructed outside of the map generator jupyter notebook. Up on completion of #27455 pr it is suggested to add italy regions process to the map generator notebook file. Regions are created from dataframe itself since it provides region - region code information. In 2019 major changes applied on italy region information thus natural earh data directly used to create regions. This is more dynamic than creating custom dictionary as the natural-earth data updated maps will follow (this is only valid for countries that has ISO codes for all granulartiy, countries such as Turkey uses NUTS codes thus it requires custom dictionary).
Along with italy regions some code improvements also added. When plotting all the maps in earlier stages of notebook, process fails due to missing country-regions. This was fixed and those countries are shown in the center of the mini-plot for corresponding country.  New italy gejson file committed , ".ts" file remained same.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
The old italy regions geojson file plot

New italy regions geojson file plot

TESTING INSTRUCTIONS

ADDITIONAL INFORMATION

[ ] Has associated issue:
[ ] Required feature flags:
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

Hi rusackas, added italy regions to the country map generator notebook. before merging I wanna do some tests on france regions in case i can use region - region_cod columns to create regions map for france w/o using custom dictionary. This would bring more dynamic approach and will be future proof. Also should we delete italy_regions file inside the countries_custom folder since it is added to the notebook flow.

Thank you for this! I was going to take a look at those early cell failures as well, and at least let them fail gracefully. Thanks for looking into that. I'll pull and review when I'm home after a weekend trip.

hi, did some minor code change on italy region section to be more clear on the dataframe name. Also some code improvements applied on france regions. rather than using fixed region dictionary, now process infers regions codes and names from natural earth data. This will bring dynamic flow in case france region information changes. As the natural earth data updated, region map will be updated too. There was small file size difference but plots shows no majora diff with the old and new region files.
Previous france region map plot

New plot after code changes

This looks great, thank you so much!


Empty Groups = Crash

Home assistant version:
0.17
Python3 version:
3.4 (8.1.1)
Component:
The problem is reported to be about mqtt but it is actually about the groups feature.
Description of problem:
When using my exact config file (below), you get an error about mqtt:
Invalid config for [mqtt]: expected a list for dictionary value @ data['mqtt']['protocol']
Then, the entire thing crashes and you have to start it again with sudo hass :cry:
Expected:
I expected Home Assistant to start as normal but it stops and crashes :disappointed:
Problem-relevant configuration.yaml entries and steps to reproduce:
homeassistant:
  # Name of the location where Home Assistant is running
  name: Home

  # Latitude and Longitude for determining when the sun sets
  latitude: _MegaHackers_
  longitude: _WillBSad_

  # C for Celcius, F for Fahrenheit
  temperature_unit: C

  # Pick yours from here: http://en.wikipedia.org/wiki/List_of_tz_database_time_zones
  time_zone: Australia/Melbourne

  # Customization options for the GUI
  customize:
    media_player.chromecast9722:
      friendly_name: TV
      icon: mdi:monitor
      app_name: Chromecast
    media_player.chromecastaudio3349:
      friendly_name: Speakers
      icon: mdi:music-note
      app_name: Chromecast Audio
    media_player.apple_tv:
      friendly_name: Apple TV
      icon: mdi:apple
      app_name: TV OS

# Zone setup for people tracking
zone:
  name: School
  latitude: _PlzDont_
  longitude: _HackMe_
  radius: 250
  icon: mdi:crown

zone 2:
  name: School
  latitude: _GoAway_
  longitude: _SrslyDude_
  radius: 250
  icon: mdi:book-open-variant

zone 3:
  name: Home
  latitude: _Nope_
  longitude: _DieInAVeryBigHole_
  radius: 250
  icon: mdi:home

# View all events in a logbook
logbook:

# Enables the frontend
frontend:

# Allows you to issue voice commands from the frontend
conversation:

# Enables support for tracking state changes over time.
history:

# Track the sun
sun:

# Checks for available updates
updater:

# Discover some devices automatically
discovery:

# Settings for accessing Home Assistant over WiFi
http:
  api_password: _LolNopeM8_

# Settings for mosquitto MQTT broker
mqtt:
  broker: _Nah_
  port: 1883
  keepalive: 60

# Device tracking settings
device_tracker:
  platform: owntracks
  max_gps_accuracy: 200

# Settings for groups of devices
group:
  media:
    view: no
    name: Media
    entities:
      - media_player.chromecast9722
      - media_player.chromecastaudio3349
      - media_player.apple_tv

  shed:
    view: yes
    name: Shed
    entities:

  hallway:
    view: yes
    name: Hallway
    entities:

  front-garden:
    view: yes
    name: Front Garden
    entities:

  back-garden:
    view: yes
    name: Back Garden
    entities:

  kitchen:
    view: yes
    name: Kitchen
    entities:
      - media_player.chromecastaudio3349

  upstairs:
    name: Upstairs
    view: yes
    entities:
      - media_player.chromecast9722

Use this config file with real values instead of _RandomOnes_ for private info
It will crash and give you an error
Boom it has been reproduced.

Additional info:
You can fix it by removing all the groups that have no entities in them. But it is still a bug. :+1:

That error message is actually correct and indicates that the validation code is wrong. It expects a list of all possible values where it should test if the protocol you specify is one of the valid values.
I still remember someone (hi @balloob) saying that the list was doing the wrong thing there and I argued that it was right and even linked to the voluptuous source to make my point, sigh.
Will be opening a PR with a fix once I get to a computer, but I am going to have to check if I made the same mistake in other places as well.

Groups with no entitities are definitely failing validation. I noticed they are all views, what is the desired behaviour? Should it validate correctly and show an empty tab/card, or validate correctly and not show anything?

Both issues are fixed.


https://github.com/Azure/Azure-Sentinel/pull/3957

Hello, I'm not sure what the error is with the JSON file - the error text isn't helpful and the code looks ok
1 files changed in current PR after filter. File Type Filter: .json, File path Filter: Workbooks, File Kind Filter: Modified
WorkbooksMetadata Validation Failed. File path: Workbooks/WorkbooksMetadata.json. Error message: Cannot read properties of undefined (reading 'startsWith')
Please see https://github.com/Azure/Azure-Sentinel/pull/3957

Any updates on this please?

Closing the issue. As the check failure for workbook is resolved in another new PR #4089 and the PR #3957 is no more required.


Exclude PRs on the user's own repo

When retrieving pull requests created by a specific user to other repositories, we can use -user:username syntax to exclude pull requests made to our own repository. This can reduce irrelevant search results.
Reference: https://docs.github.com/en/search-github/searching-on-github/searching-issues-and-pull-requests#search-within-a-users-or-organizations-repositories

@j178 thanks
but we have already filtered this do we need to drop old filter way too?
https://github.com/yihong0618/github-readme-stats/blob/323f6b83bedce62b4e1435da3695b48b1acb73b8/github.go#L180

The -user:username filter is applied before returning the results, allowing us to retrieve more relevant results within the specified limit.
The client-side filter if *issue.AuthorAssociation == "OWNER" { can be keeped as a fallback measure in case the server does not respect the -user:username filter.

make sense thanks


DocsArchive-<IP_ADDRESS>Auto archive content

Please don't merge this PR until target archive repo PR https://github.com/OPS-E2E-PPE/docs-archive-test-source-version-target/pull/34 is merged into live branch.
Auto archive content to https://github.com/VSC-Service-Account/docs-archive-test-source-version-target.git

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

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

versioningArchiveSourceFolder/TOC.yml
:white_check_mark:Succeeded
View (sql-server-2014 || azuresqldb-current || azuresqldb-mi-current || azure-sqldw-latest || >= aps-pdw-2016 || >= sql-server-2016 || >= sql-server-linux-2017 || sql-server-previousversions || sqlallproducts-allversions)

versioningArchiveSourceFolder/docfx.json
:white_check_mark:Succeeded

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

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

File
Status
Preview URL
Details

.openpublishing.redirection.json
:white_check_mark:Succeeded

versioningArchiveSourceFolder/TOC.yml
:white_check_mark:Succeeded
View (sql-server-2014 || azuresqldb-current || azuresqldb-mi-current || azure-sqldw-latest || >= aps-pdw-2016 || >= sql-server-2016 || >= sql-server-linux-2017 || sql-server-previousversions || sqlallproducts-allversions)

versioningArchiveSourceFolder/docfx.json
:white_check_mark:Succeeded

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


Bug

Summary:
Please edit this to explain the summary of the bug.
`` needs to return ... but returns ...
CovsirPhy version
Related classes:

covsirphy.
(optional)

Codes and outputs:
import covsirphy as cs
# Dataset preparation
data_loader = cs.DataLoader("input")
jhu_data = data_loader.jhu()
population_data = data_loader.population()

This code returns
Environment:
Python 3.8, pipenv, WSL.
Dear Lisphilar,
Thankyou for the updates.As You have  made a single scenario phase of India,we are unable to predict the future data...,I am getting following error now while executing predicting model.
ind_scenario.clear()
ind_scenario.add_phase(days=7)
ind_scenario.simulate().tail(7).style.background_gradient(axis=0)
NameError: Initial value of Susceptible must be specified in @y0_dict.
Please check it once...,if possible..
Thankyou.

Dear Rakesh,
Thank you for your feed-back.
I changed the code to fix #110 in GitHub, but new error occurred as you mentioned. I will fix this issue today and update the package in PyPI.
By the way, please edit "summary", "Codes and outputs" section etc. in the issue template. I think this is more useful for you.

Thanks...,i did the mentioned..
With regards,
Rakesh

Dear Rakesh,
Thank you for your feed-back.
I changed the code to fix #110 in GitHub, but new error occurred as you mentioned. I will fix this issue today and update the package in PyPI.
By the way, please edit each "summary", "Codes and outputs" section etc. in the issue template. I think this is more useful for you.

Dear Lisphilar,
Please let me know how to divide the different phases in case of scenario matter as per own need.Kindly help me if possible.
With regards,
Rakesh

Dear Rakesh,
Thank you. I created issue #118 because this includes a new bug.

Dear Rakesh,
I fixed this issue in version 2.5.4
Please kindly confirm it.
Thank you!


Iptvbdfree

Hello

Hello,
I have problems to reply to github, if you can contact me on Codeberg Thanks!     On Wednesday, June 26, 2024 at 10:07:34 PM GMT+2, Iptvbdfree @.***> wrote:
Hello
—
Reply to this email directly, view it on GitHub, or unsubscribe.
You are receiving this because you are subscribed to this thread.Message ID: @.***>


[Bug][Worker] Remove service dependency from worker module

Purpose of the pull request
fix bug in https://github.com/apache/dolphinscheduler/pull/10994

Brief change log

Verify this pull request

This pull request is code cleanup without any test coverage.
(or)
This pull request is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:

Codecov Report

Merging #11008 (249c306) into dev (083ab2b) will decrease coverage by 0.00%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##                dev   #11008      +/-   ##
============================================
- Coverage     40.36%   40.35%   -0.01%
+ Complexity     4865     4864       -1
============================================
  Files           943      943
  Lines         37061    37055       -6
  Branches       4068     4062       -6
============================================
- Hits          14958    14953       -5
+ Misses        20600    20599       -1
  Partials       1503     1503

Impacted Files
Coverage Δ

...erver/master/processor/queue/TaskEventService.java
75.00% <0.00%> (-5.36%)
:arrow_down:

...e/dolphinscheduler/remote/NettyRemotingClient.java
52.08% <0.00%> (-0.70%)
:arrow_down:

...rver/master/runner/task/BlockingTaskProcessor.java
75.60% <0.00%> (-0.30%)
:arrow_down:

...eduler/api/service/impl/DataSourceServiceImpl.java
29.49% <0.00%> (ø)

.../dolphinscheduler/plugin/task/datax/DataxTask.java
0.00% <0.00%> (ø)

...cheduler/plugin/task/api/ShellCommandExecutor.java
0.00% <0.00%> (ø)

...er/api/service/impl/TaskDefinitionServiceImpl.java
24.01% <0.00%> (+0.05%)
:arrow_up:

...hinscheduler/plugin/alert/wechat/WeChatSender.java
29.78% <0.00%> (+0.20%)
:arrow_up:

...olphinscheduler/plugin/alert/email/MailSender.java
51.38% <0.00%> (+0.35%)
:arrow_up:

Continue to review full report at Codecov.

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

Good job, could you please remove the excludeFilters in WorkerServer
https://github.com/apache/dolphinscheduler/blob/e1c78d1a49628f75c05904c5659844dae0fa137f/dolphinscheduler-worker/src/main/java/org/apache/dolphinscheduler/server/worker/WorkerServer.java#L54-L61
and I find the dolphinscheduler-worker module relies on dolphinscheduler-server, and dolphinscheduler-server relies on dolphinscheduler-service can we remove the dolphinscheduler-service in dolphinscheduler-server(You can do this in another PR if you want)?

If this dependency tree exists, we have to remove the excludeFilters AND the dolphinscheduler-service in dolphinscheduler-server at the same time.

Good job, could you please remove the excludeFilters in WorkerServer
https://github.com/apache/dolphinscheduler/blob/e1c78d1a49628f75c05904c5659844dae0fa137f/dolphinscheduler-worker/src/main/java/org/apache/dolphinscheduler/server/worker/WorkerServer.java#L54-L61
and I find the dolphinscheduler-worker module relies on dolphinscheduler-server, and dolphinscheduler-server relies on dolphinscheduler-service can we remove the dolphinscheduler-service in dolphinscheduler-server(You can do this in another PR if you want)?

If this dependency tree exists, we have to remove the excludeFilters AND the dolphinscheduler-service in dolphinscheduler-server at the same time.

Yes, I just find we still rely on dolphinscheduler-service and dolphinscheduler-dao since we have the rely on dolphinscheduler-server

@ruanwenjun  submit a new PR ? or in this PR? move alertClientService to worker, delete dolphinscheduler-service in dolphinscheduler-server

@ruanwenjun submit a new PR ? or in this PR? move alertClientService to worker, delete dolphinscheduler-service in dolphinscheduler-server

Please do this in this pr.

@ruanwenjun I just find HeartBeatTask in dolphinscheduler-server  is using RegistryClient of dolphinscheduler-service, can not delete dolphinscheduler-service dependency in dolphinscheduler-server

@ruanwenjun I just find HeartBeatTask in dolphinscheduler-server is using RegistryClient of dolphinscheduler-service, can not delete dolphinscheduler-service dependency in dolphinscheduler-server

So we also need to split HeartBeatTask to MasterHeartBeatTask and WorkerHeartBeatTask, and move WorkerHeartBeatTask to worker 😢

service

I think the split operation is useless, WorkerHeartBeatTask still need RegistryClient

service

I think the split operation is useless, WorkerHeartBeatTask still need RegistryClient

Ok, I also notice this, we need to do much work to remove the dolphinscheduler-service in worker. Please rollback excludeFilters and I will approve this PR.

Done


mobx-react 7

See also #640

[ ] separate propTypes into separate package
[ ] move Provider / inject into separate page

This release is not scheduled yet

Explanation: The reason why this is not done in mobx-react 6 is that it would create a snowball effect migration: "I want to upgrade to react 16.8 to use hooks" -> I need to upgrade to mobx-react 6" -> "Oh, now I need to refactor propTypes and provider / inject" .
So to keep the upgrade path friendly, let's separate these goals:

mobx-react@6: Add support for hooks, increase minimum supported version to React 16.8
mobx-react@7: Make mobx-react more lean and mean

Are you sure that changing import is such a huge burden? As long as the API stays the same, it's pretty much fixable by find and replace or a simple codemod.
Either way, I think that V6 should include deprecated messages and make those new packages now so people can actually start migrating and not to wait for V7 dropping a bomb :)


Add unit tests for caliper-ethereum

This package has no unit tests at all

Hey @davidkel . I'd like to work on this issue if this is available !

@Sweetdevil144 I've assigned it to you

@Sweetdevil144 all I can suggest is review how the caliper-fabric does it's tests. I'm not actually familiar with the ethereum package sorry. The caliper-fabric tests are the pattern to follow.

Created tests for connector-Factory.js. I'm trying to create tests for ethereum-connector.js but it keeps throwing logs as below when I logged the info as follow :
let configPath = CaliperUtils.resolvePath(ConfigUtil.get(ConfigUtil.keys.NetworkConfig));
        console.log(`Config Path: ${configPath}`);
        let ethereumConfig;
        try {
            ethereumConfig = require(configPath).ethereum;
        } catch (error) {
            console.error(`Error requiring configPath: ${error}`);
            throw error;
        }

Original code lines were as follow : https://github.com/hyperledger/caliper/blob/ae23774d7bba51b55980cdc9110b11729aab96f7/packages/caliper-ethereum/lib/ethereum-connector.js#L46
Error logs :
 TypeError [ERR_INVALID_ARG_TYPE]: The "id" argument must be of type string. Received undefined
at Module.require (node:internal/modules/cjs/loader:1223:3)
      at require (node:internal/modules/helpers:179:18)
      at new EthereumConnector (lib/ethereum-connector.js:50:30)
      at Context.<anonymous> (test/ethereum-connector.js:100:27)
      at process.processImmediate (node:internal/timers:478:21)

Take a look at my current test file from : here to know how I'm mocking the data. (I found some test network configs for ethereum pre-created, utilising the same for now).


Add Beta feature gates to Projected Workspaces

Changes

This commit adds the beta feature gates to the Projected Workspace type.
/kind bug
Submitter Checklist
As the author of this PR, please check off the items in this checklist:

[x] Has Docs included if any changes are user facing
[x] Has Tests included if any functionality added or changed
[x] Follows the commit message standard
[x] Meets the Tekton contributor standards (including
functionality, content, code)
[x] Has a kind label. You can add one by adding a comment on this PR that contains /kind <type>. Valid types are bug, cleanup, design, documentation, feature, flake, misc, question, tep
[n/a] Release notes block below has been updated with any user facing changes (API changes, bug fixes, changes requiring upgrade notices or deprecation warnings)
[n/a] Release notes contains the string "action required" if the change requires additional action from users switching to the new release

Release Notes
NONE

Closing as the upstream is not updated.


Add support for python 3.5+

MkDocs still supports python 3.5+:
https://github.com/mkdocs/mkdocs/blob/ff0b7260564e65b6547fd41753ec971e4237823b/setup.py#L65
This PR lowers the requirements of the python version. I added unit tests and a github actions workflow to prove that it works from python 3.5 -3.8 and on osx, windows and linux.

Thanks :)
And no, that was a mistake. I removed that file (and .gitignore-d it) in afe93c3ac4915cf7d9ba0961982e70dd83509ab8

Yes, I see it now. Great. Will Merge. Thank you!!!!


bug: fix policy structure and remove policyType

@GitHubUser1339 The policy rules file is unchanged - I just changed the structure of the policy file and fixed a minor issue in the parameter file (using Out-FormattedPolicyDefinition.ps1)


eslint - final version

why
https://github.com/browserbase/stagehand/pull/269 got to a pretty gnarly diff, so in order to resolve comments there, we made https://github.com/browserbase/stagehand/pull/273 and merged it into a branch called eslint-final. This PR should theoretically add eslint and much more rigorous typing to Stagehand.
what changed
Rigorous internal type system. No more any, very limited as casting
test plan
CI pass

Merging without approval since all of us committed to this branch so none of us can approve it. There's no meaningful changes here besides linting + evals, and all tests passed


Support for TileUTFGrid layers using Mapbox API v4

Gents - any particular reason why OL currently does not support Mapbox's latest (v4) API for TileUTFGrid layers?
Getting Uncaught SyntaxError: Unexpected token :, which suggests that this is some sort of a parser error?
Fiddle here.
Cheers

The problem is that the v4 API doesn't support JSONP anymore

We need to add a jsonp option to ol.source.TileUTFGrid (same as the one in ol.source.TileJSON)

I think it is already there, isn't it?

The jsonp option tells if the json file should be loaded via jsonp or xhr.
See https://github.com/openlayers/ol3/blob/v3.15.1/src/ol/source/tilejsonsource.js#L49

Oh, I see, thanks. If I can carve out some time this week, I'll try to send a PR.

fixed with #5329


Final CWL workflow step to move files.

We need a final CWL workflow step that moves the final output files to the output directory. In addition, this tool should take an optional prefix that would be appended to each output file.

Basic version is working, but some cleanup and features still need to be taken care of. See issues #499  and #500


Set the first_user_admin warning with the proper color

Signed-off-by: Miquel Sabaté Solà<EMAIL_ADDRESS>

Before:

After:

why dont we use an alert-info here instead?

@cyntss good idea! I'll change this tomorrow :)

Changed to @cyntss' suggestion:


problem with db connection

Can somebody please explain step by step how do i change the data-base to mysql for the following command
docker run -e MYSQL_ROOT_PASSWORD=petclinic -e MYSQL_DATABASE=petclinic -p 3306:3306 mysql:5.7.8
because every time i try to change something in the files the "mvn package" i get a build fail.
Thanks

See https://github.com/spring-projects/spring-petclinic/pull/323

Thanks @trisberg.
@ormam thank for the report, this is going to be taken care of in #323


Allow workshop content to open specific file and line in editor

This addresses bullet point 1 from https://github.com/eduk8s-tests/lab-spring-boot-k8s-getting-started/issues/1
I.e. opening a specific file in the IDE.
See the top of 'intro.md' for an example of this is supposed to be used.
Thanks to @GrahamDumpleton for helping us sort out proper way to 'glue' the eduk8s-vscode-helper extension into eduk8s.
(Details here: https://github.com/eduk8s-tests/lab-spring-boot-k8s-getting-started/pull/6)
His PR was merged into our branch and is included into this PR.
@dsyer I did not want to merge this code directly into master as I am uncertain how you are using the master branch ATM and waned to avoid accidentally disrupting your workflow. If you feel okay with it... please go ahead and merge this into master (or let me know and I'll do it myself).

Need to remove the pesky .DS_Store file which got added to git. Not sure how that managed to get added since usually that file type is in global git ignore defaults.

@dsyer okay to merge this? Or do you prefer to keep this and future work on 'editor remote control' features in branch for the time being?

Stick it on master. That way I can try it.

You will want to ensure on latest eduk8s operator and revise your budget setting, using instead new ability to override limit ranges.
https://docs.eduk8s.io/en/latest/runtime-environment/workshop-definition.html#resource-budget-for-namespaces

@GrahamDumpleton What is the difference between this (from your fixes to our branch):
  session:
    budget: large

And (From the docs link):
  session:
    namespaces:
      budget: small

And why do we want to change it?
Also, I think Dave is using the eduk8s instance hosted by eduk8s team. So probably doesn't need to worry about updating the operator. However, the instance we are/were using is our own, and I'm not sure on the best way to update the operator. Is it sufficient to just re-run all the various 'kubectl apply' commands? Or is there a nice script somewhere to update the operator?

Use of:
  session:
    budget: large

is going away. Deprecated in favour of:
  session:
    namespaces:
      budget: large

In adding limits stuff, it made better naming sense to group under both settings under namespaces section so was clearer what it applied to.
As to upgrade, running:
kubectl apply -k "github.com/eduk8s/eduk8s?ref=master"

again should technically update things okay. The existing TrainingPortal which is deployed will be using the same portal image it was though, so would need to delete/recreate the TrainingPortal to update it.


Page Header: Update props

This component some implicit API items in it's props (type checking :-( ), since this component is one of the older ones in the project and needs to align with the architecture of the rest of the project.
#1687 highlights some of the issues and does some needed markup changes.

Deprecate contentRight and navRight completely (but keep backwards compatibility)
Better name them such as onRenderActions and onRenderControls
Variant update #1702

Hey @interactivellama , I would like to take up this issue, can you please tell me more about the changes needed.
Thank you

Thank you for asking. We are modifying the cars engine while it is driving. :-)

Please allow the current API to exist without changes (internal code can be refactored if needed).
Keep SLDS markup changes from #1687 (or start over). Here is where the current markup came from https://www.lightningdesignsystem.com/components/page-headers/
Add render props onRenderActions and onRenderControls which basically do the same thing as contentRight and navRight but are just better names. These are proptype functions that you just pass a React node into https://github.com/salesforce/design-system-react/blob/master/docs/codebase-overview.md#prefix-render-props-with-onrender Here is Combobox menuItem for instance, but I don't think these custom rendering passed in from consumers need any extra props from their parent. Please don't use type !== 'string' or the props spread ...navRight.props that you currently see in page-header/index.js, but keep those for the existing props.
Add a checkProps call deprecating contentRight and navRight, so a console warning fires when those are used. An example https://github.com/salesforce/design-system-react/blob/master/components/app-launcher/check-props.js#L27
Remove contentRight and navRight from proptypes (which removes them from docs site eventually). You'll need eslint disable rules then.
#1702 changes the variants names in the proptypes, but allows the old ones to work and should also fire a checkProps console warning deprecated the current values.

Hi @interactivellama, If no one is working on it currently, can I work on this? I have already submitted a PR pending for review on the issue #1702.

@interactivellama, I have pushed some changes on already existing PR on pageHeader props  #1832 which resolves this issue with #1702 too. Kindly review it and suggest any changes as required.

#1832 has been reviewed.

This should be fixed now.


bug: importScripts is not defined error inside web worker script

I have a web worker worker.js that references a file e.g:
importScripts('file.js');

However Karma does not seem to like it:
INFO [karma]: Karma v0.12.37 server started at http://localhost:9877/
INFO [launcher]: Starting browser Chrome
INFO [Chrome 44.0.2403 (Linux 0.0.0)]: Connected on socket ZecIQTwPUPehOMzVhz-8 with id 85867718
Chrome 44.0.2403 (Linux 0.0.0) LOG: 'pre-main prep time: 2 ms'

Chrome 44.0.2403 (Linux 0.0.0) ERROR
    Uncaught ReferenceError: importScripts is not defined at /home/.../worker.js:1

importScripts is a standard API of the Worker global scope so it should not throw this error

This sounds like the script is not executed in a web worker, but rather in the general browser scope. Try opening the debug mode and look at what actually happens on execution. (This error is not coming from karma, it's just relaying the error that Chrome sends)

@Dignifiedquire I tried with --log-level debug but it doesn't provide any further information on this.
I did however find another user facing the same problem who solved this with a hack:
if( 'undefined' === typeof window){
    importScripts('file.js');
}

Because this works, it would indicate that when Karma and/or Jasmine is loading the web worker script file, the window object is not available?

I meant opening the debug mode in the browser window:

Run karma start --single-run false
Click on the big DEBUG button in the Chrome window
Open up the developer console and set breakpoints and activate "Break on Error"
Refresh the tab to trigger a run
See what happens

How are you loading this file? You might have to set included: false to avoid it being executed in the wrong context. You can read more about file options here: http://karma-runner.github.io/0.13/config/files.html

Again, the additional debug did not show me anything extra, except that Chrome is throwing this error just on the worker file but all the rest of the files/tests are running fine.
I think the right approach is to just NOT include workers as part of the tests as they need to be executed by a script e.g. var worker = new Worker("worker.js"); in order for worker functions like importScripts to become available. In this case, I just added the following rule to karma.conf.js to ignore the worker file from being executed:
{pattern: 'js/worker.js', included: false},

Importantly, I put this rule above where I include all my js files with a * rule:
{pattern: 'js/worker.js', included: false},
'js/**/*.js',

A better approach for me would be to just keep workers and their related scripts in a separate folder which is not included in karma.conf.js at all.
Thoughts?

Good, what you've described confirms what I thought. Re best practice, to be on the safe side how about you name all your worker scripts like this myscript.worker.js, then you can do
{pattern: 'js/**/*.worker.js', included: false}

without having to maintain a different folder structure

That also sounds like a good approach ! Thanks @Dignifiedquire

Where do you include this pattern {pattern: 'js/**/*.worker.js', included: false}. I checked karma.conf.js but I couldn't find an option there to exclude files.

Please see the doc page http://karma-runner.github.io/5.0/config/files.html

Thanks. My main issue is https://stackoverflow.com/questions/61762954/importscript-and-worker-html-js-are-failing-jasmine-karma-test-cases. Would it be possible for you to look at it please?

I don't see anything in the stackoverflow post related to karma issues.
The karma test is just a web page and if your test configuration includes bad code then the page will fail with errors. The TroubleShooting page gives the recommended approach.
http://karma-runner.github.io/5.0/intro/troubleshooting.html
Basically you start your tests but the server does not exit. Then you use the browser debugger to find issues.


Remove deprecated configuration options

Remove configuration options that are marked as deprecated.

Looks like a nice place to start contributing to this super tool!

As of today:

show-source
tab-size
Top level lint settings
extend-ignore
extend-unfixable
ignore-init-module-imports

ignore-init-module-imports

This option has only been deprecated with Ruff 0.4.4 (https://github.com/astral-sh/ruff/pull/11436) and the new behavior is still gated behind preview (we may want to promote it to stable?). I think we should wait with removing the option until after the new F401 behavior has been stabilized

There's also the output-format text that is deprecated and that we probably should remove

I'll close this and will create specific deprecation tasks to ease tracking of what needs to be deprecated/errored/removed when.


Solved fillin with 0s in the smaller sll

It seems that brute force solution is changing de slls to a integer number, sum these and convert it in sll. The hints says
"The issue is that when the linked lists aren't the same length, the head of one linked list represent the 1000´s place while the other represents the 10's place. What if you made them the same length? Is there a  way to modify the linked list to do that without changing the value it represents?"

After thinking in solution I found a program in less lines than you, i have 53 lines as you but i have 4 lines in comments, please check it. I eliminated the ten and in the other hand  i compare the lengths of the sll and increase the smaller in the same cycle.   ; )

New pr


pkg-config: exec: "pkg-config": executable file not found in $PATH

Following install steps.
1) go get -u -d github.com/cloudson/gitql
2) cd $GOPATH/src/github.com/cloudson/gitql
3) make
4) sudo make install
5) export LD_LIBRARY_PATH=$PWD/libgit2/install/lib on linux or export DYLD_LIBRARY_PATH=$PWD/libgit2/install/libon Mac OS.

on the the 3rd step it goes
Building...
# pkg-config --cflags libgit2 libgit2
pkg-config: exec: "pkg-config": executable file not found in $PATH
make: *** [build] Error 2

Hi @Zhambul ,
You will need to follow this link here.
ruby -e "$(curl -fsSL https://raw.githubusercontent.com/Homebrew/install/master/install)" < /dev/null 2> /dev/null
brew install pkg-config

Then it is going to be fine...


ISSUE: Error searching emails: command: SEARCH => socket error: EOF

Describe the bug
A few new errors have popped up since updating to 0.3.29.
Environment (please complete the following information):

OS: HA OS
Type of device: Unraid VM
Home Assistant version: 2024.6.4
Component version: 0.3.29

Logs
This error originated from a custom integration.

Logger: custom_components.mail_and_packages.helpers
Source: custom_components/mail_and_packages/helpers.py:592
integration: Mail and Packages (documentation, issues)
First occurred: 7:02:13 AM (59 occurrences)
Last logged: 7:02:13 AM

Error searching emails: command: SEARCH => socket error: EOF
Error searching emails: socket error: [SSL: BAD_LENGTH] bad length (_ssl.c:2406)
Error searching emails: socket error: EOF occurred in violation of protocol (_ssl.c:2406)

This error originated from a custom integration.

Logger: custom_components.mail_and_packages
Source: custom_components/mail_and_packages/__init__.py:258
integration: Mail and Packages (documentation, issues)
First occurred: 7:02:13 AM (1 occurrences)
Last logged: 7:02:13 AM

Problem updating sensors: 'count'

This error originated from a custom integration.

Logger: custom_components.mail_and_packages
Source: helpers/update_coordinator.py:318
integration: Mail and Packages (documentation, issues)
First occurred: 12:28:34 AM (1 occurrences)
Last logged: 12:28:34 AM

Timeout fetching Mail and Packages (imap.gmail.com) data

This error originated from a custom integration.

Logger: custom_components.mail_and_packages
Source: helpers/update_coordinator.py:344
integration: Mail and Packages (documentation, issues)
First occurred: 7:02:13 AM (1 occurrences)
Last logged: 7:02:13 AM

Error fetching Mail and Packages (imap.gmail.com) data: 'count'

What settings do you have enabled for step 1?

What settings do you have enabled for step 1?

@firstof9 now the addon is failing to even Initialize.
Logger: homeassistant.bootstrap
Source: bootstrap.py:726
First occurred: 11:21:50 PM (1 occurrences)
Last logged: 11:21:50 PM

Waiting on integrations to complete setup: {('mail_and_packages', 'f47d2b16b5ca3f4272502e3cb8994c07'):<PHONE_NUMBER>04}

This error originated from a custom integration.

Logger: custom_components.mail_and_packages
Source: helpers/update_coordinator.py:318
integration: Mail and Packages (documentation, issues)
First occurred: 11:21:52 PM (3 occurrences)
Last logged: 11:24:02 PM

Timeout fetching Mail and Packages (imap.gmail.com) data

This error originated from a custom integration.

Logger: custom_components.mail_and_packages
Source: custom_components/mail_and_packages/__init__.py:96
integration: Mail and Packages (documentation, issues)
First occurred: 11:21:52 PM (3 occurrences)
Last logged: 11:24:02 PM

Error updating sensor data:

If I try changing IMAP to none, I get the following error:

Need actual tracebacks from the log, not the UI stuff.

@firstof9 I looked through the debug logs and it didn't look like there was anything useful, it was just checking for emails.

@clowncracker Here's what's happening on this:
2024-06-28 19:27:22.845 ERROR (MainThread) [custom_components.mail_and_packages] Timeout fetching Mail and Packages (imap.gmail.com) data
2024-06-28 19:27:22.845 DEBUG (MainThread) [custom_components.mail_and_packages] Finished fetching Mail and Packages (imap.gmail.com) data in 60.001 seconds (success: False)

You're hitting a 60s timeout, increase the IMAP timeout to 120.
I'm working on refactoring

@firstof9 changing the timeout to 120 ended up solving the problem.


funky weird thing

**Newest Version **
Config used:

https://files.catbox.moe/gfadvl.fed
Description:

Cheat crashes on inject due to THIS
Steps to reproduce:

[x] I am able to consistently reproduce this error

yes you can consistently get this error, just inject cheat lol
Steps to reproduce the behavior:

open tf2
open injector
inject fware(release).dll

Additional context

[x] I have included the steps to reproduce to the best I am able to share.

cry about it lol

I can't tell if you are joking or not.
SetFontGlyphSet (the vfunc you happened to cut the name out of in your screenshot) is only called by the cheat when fonts are loaded/reloaded.
do you have the DEFAULT windows font called Verdana installed on your computer (C:/WINDOWS/FONTS/VERDANA.TTF)?
do you have any settings for TF2 that have anything to do with fonts?
does this issue occur with the default fedoraware config?

yes i have the default windows font called verdana
no i do not have any tf2 settings that do anything with fonts
i cant inject the cheat without it crashing so idk if it does it with default cfg

reinstall your game


Could you please upload config file to the model dmis-lab/phrase-reranker-multi

Hi, I find there are no model files in dmis-lab/phrase-reranker-multi and the code in this repertory relies on the reranker model , could you upload the config file, please?

Hi @YanJiangJerry
Thank you for reporting the issue!
We have uploaded phrase re-ranker checkpoints, which can be found at the following link:
https://github.com/dmis-lab/TouR#model-list
Additionally, we have provided scripts for training the re-rankers for your reference:
https://github.com/dmis-lab/TouR/tree/main/cross_encoder
If you have any further questions, please don't hesitate to ask.


Not found by index in Help viewer 2.3

Not found by index in Help viewer 2.3 but can be found by search

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

ID: f589015f-22d2-45b3-dee0-9da442a8b51f
Version Independent ID: d12eee18-ba72-8a94-cd5f-578431697c52
Content: MmCopyMemory function (ntddk.h) - Windows drivers
Content Source: wdk-ddi-src/content/ntddk/nf-ntddk-mmcopymemory.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @tedhudek
Microsoft Alias: tedhudek

Hi @PisarevSergey, it looks like that page was added in 2018.  In the Help Viewer, if you have the "Windows Driver Development" modules, those are from 2017.  But the "Windows Driver Kit(WDK)" modules are from 2019, and do contain that page.

I had misunderstood.  I do repro that it is not in the index, but can be found using the Help Viewer search.  Comparing the metadata in the .md between that page and, say, IoCallDriver, I don't see a difference that would explain that.  Let me see if I can find anyone who might know why we're seeing this.

It turns out that the whole index is from the "Windows Driver Kit(WDK)" module, and that the "Windows Driver Kit(WDK) Reference" module has no index.  The fix is to rebuild that book and republish it.  Thanks for alerting us to this, @PisarevSergey.  I'll update this issue when the new book is available.

This is now fixed @PisarevSergey.  If you go into the Help Viewer, you'll see updates available for the WDK books.  That will fix the index in the Ref.  Thanks again for letting us know about the problem!

Thank you !


269: POC Security rule testing

PolicyAnalyzerServiceImplTest.testSecurityRules:101 expected: <0> but was: <31>

AWS:

rule opnrvn-r-12 is not testing anything


The environment variable BEE_CORS_ALLOWED_ORIGINS does not work under docker

The environment variable BEE_CORS_ALLOWED_ORIGINS does not work under docker
I tried to modify the command and it only worked after adding --cors-allowed-origins as follows:
command: start --cors-allowed-origins="*"

Did you uncomment it in the .env file?

I confirm that I have uncommented and all other environment variables are in effect

I was able to correctly set env variables as long as the file had the "." in the ".env". Also make sure to set it as a slice
BEE_CORS_ALLOWED_ORIGINS=["*"]

And make sure you are using compose if you want to easily use the .env file. I think you might be pulling the images from hub, instead of "https://docs.ethswarm.org/docs/installation/docker/#docker-compose". I might be wrong though.

I assume this to be fixed.

This still appear to be an open issue @Eknir

This still appear to be an open issue @Eknir, at least from my last tests.

@tmm360 could you please open a new issue and provide as much detail as possible (the template will guide you).

This still appear to be an open issue @Eknir, at least from my last tests.

@tmm360 could you please open a new issue and provide as much detail as possible (the template will guide you).

Yes, I will in next days


Resuming Training

Hi, is there any option for resuming training?

hi,when I run the command "python -m spacy download en",I got the problem of requests.exceptions.ConnectionError: HTTPSConnectionPool(host='raw.githubusercontent.com', port=443): Max retries exceeded with url: /explosion/spacy-models/master/shortcuts-v2.json (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fb877df1390>: Failed to establish a new connection: [Errno 111] Connection refused')).do you encounter the problem? Thanks a lot!!!!

hi,when I run the command "python -m spacy download en",I got the problem of requests.exceptions.ConnectionError: HTTPSConnectionPool(host='raw.githubusercontent.com', port=443): Max retries exceeded with url: /explosion/spacy-models/master/shortcuts-v2.json (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fb877df1390>: Failed to establish a new connection: [Errno 111] Connection refused')).do you encounter the problem? Thanks a lot!!!!

No I didn't face this problem I installed spacy long time ago :)

hi,when I run the command "python -m spacy download en",I got the problem of requests.exceptions.ConnectionError: HTTPSConnectionPool(host='raw.githubusercontent.com', port=443): Max retries exceeded with url: /explosion/spacy-models/master/shortcuts-v2.json (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fb877df1390>: Failed to establish a new connection: [Errno 111] Connection refused')).do you encounter the problem? Thanks a lot!!!!

you can download the .tar add run: pip install .tar, the dress is :https://github.com/explosion/spacy-models

hi,when I run the command "python -m spacy download en",I got the problem of requests.exceptions.ConnectionError: HTTPSConnectionPool(host='raw.githubusercontent.com', port=443): Max retries exceeded with url: /explosion/spacy-models/master/shortcuts-v2.json (Caused by NewConnectionError('<urllib3.connection.HTTPSConnection object at 0x7fb877df1390>: Failed to establish a new connection: [Errno 111] Connection refused')).do you encounter the problem? Thanks a lot!!!!

hi, I got the same problem, have you solved it, and how? thanks


Fixed issuer url format

Issuer url has trailing slash after tenant id.

Hi @cdeange, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. Real humans will now evaluate your PR.

TTYL, MSBOT;

@cdeange : Thanks for your contribution to the Azure documentation! The author, @MikeWasson, has been notified to review your proposed change.

#sign-off


Allow user to specify custom client_arg_name

For vendored SDKs, the dynamic smartness of client_arg_name = 'client' if operation.startswith(('azure.cli', 'azext')) else 'self' is not reliable.
This PR allows the user to override this.

View a preview at https://prompt.ws/r/Azure/azure-cli/5139
This is an experimental preview for @microsoft.com users.
(It may take a minute or two for your instance to be ready)
Email feedback to 'azfeedback' with subject 'Prompt Feedback'.

@tjprescott In my specific case, there's an extension that has it's SDK under azext_NAME and the CLI was choosing client when I wanted self.


Update network-watcher-read-nsg-flow-logs.md

Changed initial and final block size to reflect version currently in production. Added MAC Address to NSG blob path

@mattreatMSFT : Thanks for your contribution to the Azure documentation! The author, @jimdial, has been notified to review your proposed change.

#sign-off


Spaces are not being preserved while copying in HTML format

Spacing between words are not being preserved while copying and pasting to a destination app via HTML format.
Steps for Reproduction

Visit https://codesandbox.io/p/sandbox/quill-copy-space-issue-5j6t64?file=%2Fsrc%2Findex.tsx
Type a word and add few spaces and type another word.
Copy the message from quill Editor
Paste to a app which reads the HTML format

<span style="color: rgb(0, 0, 0); font-family: Helvetica, Arial, sans-serif; font-size: 13px; font-style: normal; font-variant-ligatures: normal; font-variant-caps: normal; font-weight: 400; letter-spacing: normal; orphans: 2; text-align: left; text-indent: 0px; text-transform: none; widows: 2; word-spacing: 0px; -webkit-text-stroke-width: 0px; white-space: pre-wrap; text-decoration-thickness: initial; text-decoration-style: initial; text-decoration-color: initial; display: inline !important; float: none;">FirstWord                                         AnotherWord

Expected behavior:
Spacing between word should be preserved on pasting via HTML format

Actual behavior:
Spacing between word are not preserved on pasting via HTML format

Platforms:
Chrome, windows
Version:
Quill version 1.3.1

We are experiencing the same issue on 2.0.2.
It can be replicated via the Quill playground:

Type a  a
Copy the text
Paste the text back in to the editor
The text in the editor will be a a

I believe the issue is related to the html that ends up on the clipboard when copying from the quill editor.
If you run
setTimeout(async()=> {
    const clipboardContents = await navigator.clipboard.read();
    for (const item of clipboardContents) {
      if (item.types.includes("text/html")) {
        const blob = await item.getType("text/html");

        const html = await blob.text()
        console.log('html', html);
      }

      const plainText = await navigator.clipboard.readText();
      console.log('plain text', plainText);

    }
}, 3000)

Then the html is <meta charset="utf-8">a  a
Because spaces are collapsed by default in html, this results in the spaces being collapsed in to a single space.
In comparison, copying the 'same' text from google docs results in
<meta charset="utf-8"><meta charset="utf-8">a  a

With the seemingly important bit being a  a, i.e. using a non-breaking spaces to try and prevent the spaces collapsing.
Note: This still results in a single space when pasted in to the quill editor, the rule seems to be if you have n+1 spaces on the clipboard, then pasting in to the quill editor results in n spaces


Disk usage

I have running stashvision server for less than 24 hours and the stash_tabs.bleve is currently taking 4gb of space.
Is it possible to change the implementation to take less storage space?

Restarting stashvision "reset" the disk usage. After about half a day it's back at 1.5gb.

there's unfortunately not much I can do about this short of rewriting (which I plan on doing at some point when I get the time) and using a different library. for now the workaround I'd suggest is running stashvision after your dump tabs get full, and deleting the stash tab index between runs (the *.bleve folder). indexing from scratch generally shouldn't take long at all, even with a completely full dump tab.

Ok, no big deal for me. But found the disk usage strange and I belived it was over-using storage.


resolves #494

Modify TestForIssue488 and set chartBackgroundColor and plotBackgroundColor to white.
In the new BufferedImage object, the imageType parameter is restored to BufferedImage. TYPE _ INT _ RGB

Run TestForIssue488 class, generate images:


This circuit requires a channel width above 1000, probably is not going to route.

i am doing a custom design but it failed on routing.
I am using soft_adder example
here are the files that i am using
openfpga_shell_template=${PATH:OPENFPGA_PATH}/openfpga_flow/openfpga_shell_scripts/bitstream_setting_example_script.openfpga
openfpga_arch_file=${PATH:OPENFPGA_PATH}/openfpga_flow/openfpga_arch/k4_frac_N8_reset_softadder_register_scan_chain_caravel_io_skywater130nm_fdhd_cc_openfpga.xml
openfpga_sim_setting_file=${PATH:OPENFPGA_PATH}/openfpga_flow/openfpga_simulation_settings/fixed_sim_openfpga.xml
openfpga_bitstream_setting_file=${PATH:OPENFPGA_PATH}/openfpga_flow/tasks/fpga_verilog/adder/soft_adder/config/bitstream_annotation.xml
openfpga_vpr_circuit_format=eblif
external_fabric_key_file=
Have you seen this error before?
(00_counterSBox_MIN_ROUTE_CHAN_WIDTH) - OpenFPGA Shell Run run failed with returncode 1
ERROR (00_counterSBox_MIN_ROUTE_CHAN_WIDTH) - command /home/pc-1/zliu/OpenFPGA/openfpga/openfpga -batch -f counterSBox_run.openfpga
ERROR (00_counterSBox_MIN_ROUTE_CHAN_WIDTH) - -->>Error 1:
ERROR (00_counterSBox_MIN_ROUTE_CHAN_WIDTH) - -->>Error 2: Command 'vpr' execution has fatal errors
ERROR (00_counterSBox_MIN_ROUTE_CHAN_WIDTH) - Current working directory : /home/pc-1/zliu/OpenFPGA/openfpga_flow/tasks/fpga_verilog/adder/soft_adder/run010/k4_frac_N8_tileable_reset_softadder_register_scan_chain_nonLR_caravel_io_skywater130nm/counterSBox/MIN_ROUTE_CHAN_WIDTH
ERROR (00_counterSBox_MIN_ROUTE_CHAN_WIDTH) - Failed to run OpenFPGA Shell Run task
ERROR (00_counterSBox_MIN_ROUTE_CHAN_WIDTH) - Exiting . . . . . .
ERROR (00_counterSBox_MIN_ROUTE_CHAN_WIDTH) - Failed to execute openfpga flow - 00_counterSBox_MIN_ROUTE_CHAN_WIDTH
here is the openfpgashell.log file:
Warning 209: No routing path for connection to sink_rr 1355, retrying with full device bounding box
Cannot route from io_right.inpad[0] (RR node: 1836 type: SOURCE location: (5,4) class: 1 capacity: 1) to clb.clk[0] (RR node:  type:     SINK location: (4,3) class: 47 capacity: 1) -- no possible path
Failed to route connection from 'clk' to 'DUT1.s[4]' for net 'clk' (#21)
Routing failed.
Attempting to route at 1792 channels (binary search bounds: [896, -1])
Routing took 0.43 seconds (max_rss 24.2 MiB, delta_rss +7.7 MiB)
The entire flow of VPR took 0.59 seconds (max_rss 24.2 MiB)
Fatal error occurred!
OpenFPGA Abort
Finish execution with 2 errors
The entire OpenFPGA flow took 0.588068 seconds
Thank you for using OpenFPGA!
Error 1:
Type: Routing
File: /home/pc-1/zliu/OpenFPGA/vpr/src/base/place_and_route.cpp
Line: 165
Message: This circuit requires a channel width above 1000, probably is not going to route.
Aborting routing procedure.
Error 2: Command 'vpr' execution has fatal errors
1
i tried with this but the same error is showing:
[SCRIPT_PARAM_Fixed_Routing_1500]
fix_route_chan_width=1500
end_flow_with_test=
vpr_fpga_verilog_formal_verification_top_netlist=

Hi @zliu1Charlotte . How did you solve this issue? I'm facing the same error.

this did not resolved, i went with other task.conf file sorry.


Tracking audits for Embedded objects

Hi
I have a domain object with another domain object within it as one of the attributes. When using the plugin and persisting to the domain object along with embedded object, i am not seeing audits for the embedded object
For instance
class Employee {
String name;
EmployeeProfile profile;
}
class EmployeeProfile {
String nationality;
}
Below is just an example and not actual code
EmployeeProfile profile = new EmployeeProfile();
profile.nationality = "India";
Employee employee = new Employee();
employee.name = "John Doe";
employee.profile = profile;
employee.save(flush: true);
I have created 2 new domain objects each for the audit separately.
Expected Behavior:
When i save employee, i am expecting both the audit tables to be tracked separately for the attributes that has changed
Actual Behavior:
Employee audit table gets tracked for the attributes including profile where profile's id value is tracked.
EmployeeProfile audit table is not touched at all. No attributes of this domain is being tracked
How do I make sure both domain attributes are being tracked? What change should I make to incorporate that? Any help is much appreciated.
Environment Information
Operating System: Windows
GORM Version:
Grails Version (if using Grails): 4.0.10
JDK Version: Java 8
Thanks
Venkatesh

@robertoschwald - any help here on my question?


Add npm instructions to readme

Since shards-ui is a package on npm, it's worth mentioning how to use it as a dependency instead of with html. Personally, I like bundling all my css with webpack so that it doesn't need to be depend on another site.
Additions to README:

npm version badge
npm downloads per month badge
dependencies status badge
devDependencies status badge
Add npm package link to Getting Started
Instructions for installing with npm
Instructions for usage with webpack or another bundler

@hisk done

@vikr01 Great! 👍Thanks Vik! 😊


Images not loading in Brave Mobile

Images do not load in the Brave mobile browser. When I toggle the shield (regardless of current state) it works.

I've managed to reproduce this. It seems to solve itself on any change to page settings, whether turning something on or off.
We currently intend to move to external CDN hosting, so perhaps that will resolve the issue. Otherwise, it looks like an interesting bug to submit to Brave.

The images are served off Cloudinary on production now. I can't imagine what else we can do here. They are just images yeah? Not sure why they'd be blocked for any reason.


Option to have the time axis start on a date

Setting a start date and having the possibility to enter injection instances as dates instead of relative or absolute days.
Would make a lot more convenient to add instances from tracking calendar instead of having to do mental math to confirm which instance corresponds to which when using the same simulation updated semu regularly

I'm guessing you're also thinking about importing/exporting CSVs that use dates instead of days?

Didn't think this far, I was thinking of just having a day 0 field as a date for simplicity's sake, but having the CSV with dates would make it better for readability and other uses


Observations show upload button and incorrect no photos icon on Explore

Describe the bug
Explore grid view is showing the "no photos" icon for observations with photos, and both grid and list views are showing the upload button for all observations.
To Reproduce
Steps to reproduce the behavior:

Go to Explore
In list view, note that all observations have the upload button (up arrow surrounded by a dotted circle)
In grid view, note that all observations have the upload button (up arrow surrounded by a dotted circle) and all observations have the "no photos" icon in the upper right.
See error

Expected behavior
Explore should never provide the option to upload unless you have permission to edit that observation (e.g. you created the observation), and perhaps never (check with Abhas if he wants to support upload from Explore).
The "no photos" icon should only appear if the observation has no photos.
Screenshots

Context (please complete the following information):

Device: iPad Air (5th gen)
OS: iPadOS 16.6.1
App version: v0.15.0+61

Additional context
I realize Explore isn't fully designed yet, but I think the obs grid/list will remain pretty much the same so we should fix these.

@abhasinat Do we want to support upload from the Explore screen?

I'm assuming you mean the upload button in the tab bar, or the tab bar in general? Yep, I think we should support it on explore. Only screens that lay on top of everything else (obs edit, explore filters, etc) don't have the tab bar.

@abhasinat, this issue is about the upload buttons on individual observation items that appear in Explore (see the screenshots in the OP). We can discuss if you want, but I'm strongly opposed to to supporting that in Explore. As far as I can tell from the designs there is no upload button in the tab bar (maybe you're referring to the Add Obs button?).

Ah, I thought this was already explained in the issue and angie was asking about something in addition to it.
Yes, the upload buttons next to each obs item in explore are a mistake. On explore, a user should only ever see the obs list status (with quality grade, number of comments, and number of IDs). Those upload buttons should only ever appear on home screen/my observations.

@angielt you removed the upload button, but you also removed the observation status indicators, e.g. the quality grade, number of comments, IDs, etc. Could you please restore them, and write a test the confirms that they are present on Explore and that the upload buttons are not?


[Bug]: Select and MultiSelect have different height properties

Tremor Version
v3
Link to minimal reproduction
/sorry-no-link
Steps to reproduce
Should be straightforward: the Select component and MultiSelect components appear to have different default heights, which is difficult to manage with Tailwind conflicts. The MultiSelect search also doesn't respect the styling of MultiSelect or MultiSelectItem. For example, if you set className='text-xs', items will get styled by the search will not.
What is expected?
These components are often used next to each other as filters etc. in dashboards. Ideally, they would have the same default height.
What is actually happening?
It appears MultiSelect has a slightly larger height (in order to support selections). Search also doesn't respect styling (while the items do).
What browsers are you seeing the problem on?
Chrome, Firefox
Any additional comments?
Simplest solution would be to increase the height of the Select component to match that of MultiSelect which has to take items.

Hi @mohamm-ad, thanks for letting us know, we will investigate asap! In the meantime, you can harmonize the heights via className customization: Try to insert [&>button]:h-9 for example.

Amazing @christopherkindl! And good call, that fix worked! Any suggested workarounds for styling the search in MultiSelect?

@mohamm-ad you are welcome! :) What do you want to style exactly?

When styling MultiSelect + MultiSelectItem, changes with Tailwind are reflected correctly across the component except for in the search area. For example, in the image above I applied text-xs to both, which works fine except for search (where text stays the default size). Apologies if I'm missing something!

Hey @mohamm-ad, I my tests, they seem all seem to have the same height. Can you post a screenshot of your example?

Ad text: Why don't use use the Theme in the tremor config? You could easily overwrite tremor-default to your liking
 fontSize: {
        "tremor-label": ["0.75rem"],
        "tremor-default": ["0.875rem", { lineHeight: "1.25rem" }],
        "tremor-title": ["1.125rem", { lineHeight: "1.75rem" }],
        "tremor-metric": ["1.875rem", { lineHeight: "2.25rem" }],
      },

Interesting! I'm not quite sure what's going on in mine, I haven't made any changes:

Yep, when you adjust the default text size, the search text adjusts accordingly as intended. But for this particular use case, I needed to adjust the placeholder and item size to be different from the default -- that worked fine with tailwind, but i wasn't able to change the search placeholder text.

Alright, the select on the left is too small. Its the Select or Multiselect? I am having a hard time replicating your issue. Could you share a minimal reproduction?

@mohamm-ad Can I close this issue from your side?


Fixed check_ceph_osd for Ceph Octopus.

Removed unnecessary parentheses for Python 3, fixed regex variable import, and added a loop to the print statements to properly format the aggregate OSD states.
The findall regex function of python3 outputs a tuple when searching for multiple statements. This is in contrast to prior versions which output an array of strings, requiring us to loop through joining the tuples prior to printing.

Hello @HeroesLament,
The parentheses in the print statements are NOT optional. In python3 it is mandatory, print(...) is a function call...
Example:
Python 3.9.10 (main, Jan 15 2022, 11:48:00)
>>> print "hello world!"
  File "<stdin>", line 1
    print "hello world!"
          ^
SyntaxError: Missing parentheses in call to 'print'. Did you mean print("hello world!")?

Hello @valerytschopp ,
The work our team was doing on this was apparently not on track. The server this was being worked on was running python 2.7, so much of the syntax was not correct. We are working to re-factor this with python 3.8 now.


Feature request: Use arbitrary colours for components supporting the 'Color' parameter.

I asked this question on the Discord channel and was asked to open it as an issue.
For components that support the 'Color' parameter it would be great to be able to add arbitrary colours using HTML color code notation or similar.
For example, I use a dark background for my side drawer component so need to use light colours for the icons and text there. However, I use a light background for the rest of the app.
I'm aware I can use themes but it's a pity to have to waste one of the theme 'slots' for a setting that's only to be used a few times in a static context.
An alternative would be to allow extension of the Color enum but I don't know if it's even possible to extend enums at runtime?
Thanks for the great work to date

Just an idea: a parameter/property on a component like Theme: Inherit/Custom

Just an idea: a parameter/property on a component like Theme: Inherit/Custom

This is something that i and henon planned to add, im reworking the icons right now and then release 1.3.0 after that i can look into this one for a neat solution :)

This is something that i and henon planned to add, im reworking the icons right now and then release 1.3.0 after that i can look into this one for a neat solution :)

Yes, we absolutely need it. It feels strange having to resort to tricks to circumvent this restriction. Why is the Color parameter on elements an enum? What if it was simply a Type that supports all RGBA colors, and the theme was just a static class with readonly properties of that type, defining what is "Primary", what is "Secondary" and so on.

Would be a great use-case for discriminated unions.

I would like to vote for this as well, or at least to expand the Color enum to list all possible choices in a palette so one could choose ErrorLighten, etc when building a button or text component.
In my use case I am using Color.Success for form submit buttons, and Color.Error for cancel buttons.  In addition those colors are also used in dashboards/reports to denote success and errors, but I have found that the preferred colors work well for buttons, but do not have enough contrast for some text.
Obviously this can be addressed using the Style parameter, but having more options to choose from in the root implementation would make it easier to have colors change when a different theme is selected or dark mode is enabled.


[Bug]: Custom Element doen't exist

Requirements

[X] I checked the troubleshooting section in the README to verify that I have the latest Mushroom version.
[X] I did a search to see if there is a similar issue or if a pull request is open.
[X] I have read the state of card mod support (in pin issues) and understand card mod support is not provided in this repository.

Current Behavior
When i install version 4.0.0 the cards no longer load
Expected Behavior
loaded cards.
Steps To Reproduce

install version 4.0

Context
one of them:
YAML state
type: custom:mushroom-template-card
entity: sensor.afvalinfo_home_pbd
primary: PMD
secondary: >-
  {% if states('sensor.afvalinfo_home_trash_type_tomorrow') == "Geen" %}

  Geen informatie

  {% elif states('sensor.afvalinfo_home_trash_type_today') == "pdb" %} Vandaag
  geweest. Kliko weer ophalen!

  {% else %}
    {% set papier = (strptime(states(entity), '%d-%m-%Y').date()) %}
    {% set delta = (papier - now().date()).days %}
    {% if states(entity) == "Geen" %}Geen informatie
    {% elif delta < 0.9 %}Vanavond kliko aan de straat!
    {% elif delta < 1.9 %}Morgen ({{states(entity)}})
    {% else %}Over {{delta | round(0)}} dagen ({{as_timestamp(papier) | timestamp_custom("%A %d-%m")}})
    {% endif %}
  {% endif %}
icon: mdi:recycle
icon_color: blue
multiline_secondary: true

Environment
- Browser: edge / chrome
- HA Version: 2024.7.4
- Mushroom: 4.0.0

Anything else?
re-install 3.6.4 solves the problem

I have the same problem.

I have the same issue and going back to 3.6.4 did not fix the issue

Same issue has happened to me after updating to 4.0.0. Reverting back to 3.6.4 has not worked for me yet.

Same issue has happened to me after updating to 4.0.0. Reverting back to 3.6.4 has not worked for me yet.

I had to manually set the resource again and not my mushroom cards are working with 3.6.4
Add reference to mushroom.js in Dashboard. There's two way to do that:
Using UI: Settings → Dashboards → More Options icon → Resources → Add Resource → Set Url as /local/mushroom.js → Set Resource type as JavaScript Module. Note: If you do not see the Resources menu, you will need to enable Advanced Mode in your User Profile

Upgrading to 4.0.0 via HACS doesn't seem to pull down/install the mushroom.js appropriately (I don't see it in the appropriate directory) and the resource reference URL then includes "null" where mushroom.js should be - /hacsfiles/lovelace-mushroom/null?hacstag=444350375361.  If you then remove 4.0.0 and install 3.6.4, the resource reference above wasn't updated, so I went back and replaced null with mushroom.js and all is working again just like the post above...  I'll try 4.x again in a few days to see if this all gets resolved.

Apperantly it is caused by an incompatibility between HA 2024.7 and Mushroom 4.x.x
It now states that Mushroom 4.x.x should not be installed unless you have HA 2024.8

With the upgrade to 2024.8 and the update van Mushroom to 4.0.2 the problem is solved. I will close this ticket.


Add curiosity boxes explaining some of the unicode encoding schemes (at least UTF-8 and UTF-32)

(Note that this should be done once the main text has been refactored -- I'll update this saying when that is).
This is not essential for the standard, but some students will want to know how unicode characters were assigned bit representations in encoding schemes. We should aim to at least do UTF-32, which is trivial, and UTF-8, which is more complicated, but interesting, and still accessible.
Wikipedia gives an explanation, but we'll want to explain this as clearly as possible for the students.
https://en.wikipedia.org/wiki/UTF-8#Description
Assigned this to Tim, but if you are not keen, I'm happy to do it myself :-)

Closing this as I've decided all students should understand UTF-8. Will update with a new issue later.

@Heidirose Can you please link the new issue to this issue when it's created so it's easier to find.

Sure, although this issue is pretty much redundant now. If I could have hard deleted it, I would have.

The goal with having our repo on GitHub is everything is stored, even if is not implemented. Because if a few years down the line someone suggests we move UTF-8 info to curiosity boxes, we can show them this issue to as proof we had discussed it at some point and decided all students should understand it.


Support Concurrency Checking in Postgres

Support for concurrency token in Postgres with Npsql library is done by using the "xmin" system column (see more here).
However, this causes an "InvalidOperationException: The required column 'xmin' was not present in the results of a 'FromSql' operation." error being thrown when trying to update an entity that uses 'xmin' as a concurrency token with BulkSaveChanges.
My configuration is the following. I tried PropertiesToExclude in an effort to avoid the error to no avail.
var _config = new BulkConfig()
            {
                PropertiesToExclude = new List<string> {"xmin"},
                SetOutputIdentity = true,
                EnableShadowProperties = true,
                OmitClauseExistsExcept = true,
                WithHoldlock = false,
                BulkCopyTimeout = 0,
                BatchSize = _options.BatchSize,
            };

Can we support concurrency checks in Postgres as we do in SQL Server with the DoNotUpdateIfTimeStampChanged setting? I understand it is a different and more difficult case in Postgres with the hidden shadow property but i believe it would be really beneficial if we do support this.
At the very least could we have a way to avoid this error being thrown? BulkUpdate don't have this issue, only with BulkSaveChanges we see this.

Added Test TimeStampPGTest with BulkSaveChanges and PG TimeStamp.
Unable to reproduce the issue with latest version.


Does this issue also apply to ASP.Net?

I see similar issue in ASP.Net.WebApi.Core 5.2.4.

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

ID: 0b42a61c-cf3a-3bdb-0145-dc6295c5ba23
Version Independent ID: 5b8d2082-1e70-eb9a-1bbe-7b93f74bdd3a
Content: Migrate from ClaimsPrincipal.Current
Content Source: aspnetcore/migration/claimsprincipal-current.md
Product: aspnet-core
GitHub Login: @mjrousos
Microsoft Alias: scaddie

@hoyle1977, this should only apply to ASP.NET Core. Can you provide more details on the ASP.NET problem you're seeing? If ClaimsPrincipal.Current isn't populated, it's possible that authentication isn't setup properly or that there's a bug in recent ASP.NET builds.


Nonfunctioning back button in Android app

Only the bottom back button works after pressing on the "Find Dogs Near Me" button. The top back button currently doesn't do anything. This is a medium priority bug since the user can still return to the previous screen but it is really counter intuitive that one of the buttons does nothing. This will be finished by Monday my me (AJ).

This has been fixed now!


Joshproehl/linked list as array

Parse the linked list as an actual array, plus remove it from the output JSON

So tiny its almost funny, merged


InstructBLIP generates short and repeated sentence.

Thanks for your awesome work in InstructBLIP. When I want to reproduce the result in Figure 5 in your paper, the result is not ideal.
raw_image = Image.open("../docs/_static/Confusing-Pictures.jpg").convert("RGB")
question = "What is unusual about this image?"
# use "eval" processors for inference
image = vis_processors["eval"](raw_image).unsqueeze(0).to(device)
question = txt_processors["eval"](question)
samples = {"image": image, "text_input": question}
display(raw_image.resize((596, 437)))
output = model.generate(samples=samples, use_nucleus_sampling=True, repetition_penalty=2.0, min_length=50)[0]
print(output)

The output is:
man on top of car drying clothes in the middle of the street during rush hour with taxi cabs and other cars driving by it is unusual for a man to be doing laundry in the middle of the street during rush hour while there are taxi cabs and other cars driving by

I also try other images, most outputs are short. How can I reproduce the result as shown in your paper? Thanks in advance :)

What model did you use?

I used vicuna-13b:
model, vis_processors, txt_processors = load_model_and_preprocess(name="blip2_vicuna_instruct", model_type="vicuna13b", is_eval=True, device=device)

Do you observe similar behavior with other models such as vicuna7b?

Thanks for your awesome work in InstructBLIP. When I want to reproduce the result in Figure 5 in your paper, the result is not ideal.
raw_image = Image.open("../docs/_static/Confusing-Pictures.jpg").convert("RGB")
question = "What is unusual about this image?"
# use "eval" processors for inference
image = vis_processors["eval"](raw_image).unsqueeze(0).to(device)
question = txt_processors["eval"](question)
samples = {"image": image, "text_input": question}
display(raw_image.resize((596, 437)))
output = model.generate(samples=samples, use_nucleus_sampling=True, repetition_penalty=2.0, min_length=50)[0]
print(output)

The output is:
man on top of car drying clothes in the middle of the street during rush hour with taxi cabs and other cars driving by it is unusual for a man to be doing laundry in the middle of the street during rush hour while there are taxi cabs and other cars driving by

I also try other images, most outputs are short. How can I reproduce the result as shown in your paper? Thanks in advance :)

same here~
did you try increase the repeat penalty?

Thanks for your awesome work in InstructBLIP. When I want to reproduce the result in Figure 5 in your paper, the result is not ideal.
raw_image = Image.open("../docs/_static/Confusing-Pictures.jpg").convert("RGB")
question = "What is unusual about this image?"
# use "eval" processors for inference
image = vis_processors["eval"](raw_image).unsqueeze(0).to(device)
question = txt_processors["eval"](question)
samples = {"image": image, "text_input": question}
display(raw_image.resize((596, 437)))
output = model.generate(samples=samples, use_nucleus_sampling=True, repetition_penalty=2.0, min_length=50)[0]
print(output)

The output is:
man on top of car drying clothes in the middle of the street during rush hour with taxi cabs and other cars driving by it is unusual for a man to be doing laundry in the middle of the street during rush hour while there are taxi cabs and other cars driving by

I also try other images, most outputs are short. How can I reproduce the result as shown in your paper? Thanks in advance :)

hi, i also run into this problem. have you solved this?

Thanks for your awesome work in InstructBLIP. When I want to reproduce the result in Figure 5 in your paper, the result is not ideal.
raw_image = Image.open("../docs/_static/Confusing-Pictures.jpg").convert("RGB")
question = "What is unusual about this image?"
# use "eval" processors for inference
image = vis_processors["eval"](raw_image).unsqueeze(0).to(device)
question = txt_processors["eval"](question)
samples = {"image": image, "text_input": question}
display(raw_image.resize((596, 437)))
output = model.generate(samples=samples, use_nucleus_sampling=True, repetition_penalty=2.0, min_length=50)[0]
print(output)

The output is:
man on top of car drying clothes in the middle of the street during rush hour with taxi cabs and other cars driving by it is unusual for a man to be doing laundry in the middle of the street during rush hour while there are taxi cabs and other cars driving by

I also try other images, most outputs are short. How can I reproduce the result as shown in your paper? Thanks in advance :)

same here~ did you try increasing the repeat penalty?

could you solve it by increasing the repeat penalty? could you please tell me what value you use?


Invitation code works only for admin/app-built-in

I am trying to use invitation code. We do not use build-in organization and create own organization and application.
When I do Copy signup page URL and follow to this page there is message pop-up with Invitation code is invalid text. Sign-up page make requests to /api/get-application?id=admin/app-built-in and /api/get-invitation-info?code=12345678&applicationId=admin/app-built-in that I believe is not right.
Version: v1.681.0
UPD: when I change organization on built-in no error popups. Didn't try to signup to this organization through

@dacongda

I create a new org and application, then create a invitation code and not pop up invalid message

canyou provide a demo video that how to reporduce this issue ?

btw, use shared application may cause Invitation code is invalid

Application not shared. I actually compare built-in application and organization and can't find anything THAT different. Looking forward to capture screen. How to share video with you?

you can just capture the setting of invitation code and application

also, share the setting of that organization will be much better

https://youtu.be/hNN3oGgtKOw - please let me know when you check it, I'll close access

https://youtu.be/hNN3oGgtKOw - please let me know when you check it, I'll close access

ok, i check it

https://youtu.be/hNN3oGgtKOw - please let me know when you check it, I'll close access

can you show the link you copied, you can mask the domain

https://youtu.be/hNN3oGgtKOw - please let me know when you check it, I'll close access

emmm, according to the video, I think you can set the default application for your group like below

Is it requirement to have default application?

I set default application for organization - doesn't help

I set application for invitation too - after this Copy signup page URL adds application name to signup URL.
When I remove specific application for Invitation previous link (with app name in it) still works.
So, looks like there is problem with invite link generation

i try and can not reproduce it
https://github.com/user-attachments/assets/6f27f0e4-2d72-4595-bc83-13ec3924dd8e

Closed for not reproducible

Great way to handle issues! sarcasm


Add option to ignore unexecuted notebooks

Hello!
This pre-commit hook is used in instamatic, where some notebooks contain no executed cells.
In my opinion, these should pass an execution order check.
To avoid breaking anything, this PR adds an optional flag to allow these files (off by default).
If such a feature is unfitting for this hook, feel free to decline this PR. There are other ways to work around this, e.g. ignoring them in .pre-commit-config.yaml

Thanks for this, if you add a small line about this option to the readme I'm happy to merge this 👍

Done, thanks!

Updated the tag as well:
https://github.com/stefsmeets/nbcheckorder/releases/tag/v0.3.0