NLog.Database  version 5.2.8 with net8 error

Hi! Thanks for reporting this bug!
Please keep / fill in the relevant info from this template so that we can help you as best as possible.
NLog version: 5.2.8
Platform:  .NET8
Current NLog config (xml or C#, if relevant)
<?xml version="1.0" encoding="utf-8"?>
<nlog xmlns="http://www.nlog-project.org/schemas/NLog.xsd" xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance" autoReload="true" internalLogLevel="Trace" internalLogFile="NlogRecords.log">
	<extensions>
		<add assembly="NLog.Web.AspNetCore" />
	</extensions>

	<targets async="true">

		<target name="log_database" xsi:type="Database" dbProvider="MySql.Data.MySqlClient.MySqlConnection, MySql.Data" connectionString="Server=<IP_ADDRESS>;Database=JQEMS360;User=Ray;Password=qcmean;Pooling=true;MAX Pool Size=512;Min Pool Size=50;Connection Lifetime=30;">
			<commandText>
				INSERT INTO NLogs
				(LogDate,LogLevel,LogType,Logger,Message,MachineName,MachineIp,NetRequestMethod
				,NetRequestUrl,NetUserIsauthenticated,NetUserAuthtype,NetUserIdentity,Exception)
				VALUES
				(@LogDate,@LogLevel,@LogType,@Logger,@Message,@MachineName,@MachineIp,@NetRequestMethod
				,@NetRequestUrl,@NetUserIsauthenticated,@NetUserAuthtype,@NetUserIdentity,@Exception);
			</commandText>
			<parameter name="@LogDate" layout="${date}" />
			<parameter name="@LogLevel" layout="${level}" />
			<parameter name="@LogType" layout="${event-properties:item=LogType}" />
			<parameter name="@Logger" layout="${logger}" />
			<parameter name="@Message" layout="${message}" />
			<parameter name="@MachineName" layout="${machinename}" />
			<parameter name="@MachineIp" layout="${aspnet-request-ip}" />
			<parameter name="@NetRequestMethod" layout="${aspnet-request-method}" />
			<parameter name="@NetRequestUrl" layout="${aspnet-request-url}" />
			<parameter name="@NetUserIsauthenticated" layout="${aspnet-user-isauthenticated}" />
			<parameter name="@NetUserAuthtype" layout="${aspnet-user-authtype}" />

			<parameter name="@NetUserIdentity" layout="${aspnet-user-identity}" />
			<parameter name="@Exception" layout="${exception:tostring}" />
		</target>

		<target name="log_file" xsi:type="File" fileName="${basedir}/logs/${shortdate}.log" layout="${longdate} | ${level:uppercase=false} | ${logger} | ${aspnet-session:item=username} | ${message} ${onexception:${exception:format=tostring} ${newline} ${stacktrace} ${newline}" />

		<target name="log_json" xsi:type="File" fileName="${basedir}/logs/${shortdate}.json" >
			<layout xsi:type="JsonLayout">
				<attribute name="time" layout="${longdate}" />
				<attribute name="level" layout="${level:upperCase=true}"/>
				<attribute name="userName" layout="${aspnet-session:item=username}"/>
				<attribute name="stracktrace" layout="${stracktrace}" />
				<attribute name="logger" layout="${logger}" />
				<attribute name="message" layout="${message}" />
				<attribute name="exception" layout="${exception:format=ToString}" />
			</layout>

		</target>

	</targets>

	<rules>
		<logger name="Microsoft.*" final="true" />
		<logger name="*"  writeTo="log_database,log_json,log_file" final="true"/>
	</rules>
</nlog>

There is no problem using nlog5.1 and nlog.database 5.1 in .net6
When using nlog5.2.8 and nlog.database 5.2.8 in .net8,
When executing add-migration or update-database to build a MySQL database, net-host will be used.
After removing nlog.database 5.2.8 and using add-migration or update-database to build the MySQL database, net-host can terminate normally.
But if nlog.database is not installed, the database cannot be written.
For this situation have any idea? thank you

I don't know much about MqSQL or your application, and have no idea what add-migration or update-database is.
Right now you are changing from NET6 to NET8, and also changing from NLog 5.1 to NLog 5.2.8:

What happens if using NLog v5.2.8 on NET6 ?
What happens if using NLog v5.1 on NET8 ?

Have you checked the output from NLog InternalLogger to see if there are any errors, or difference in output ?

.net8 is my new project, instead of using .net6 to migrate to .net8, ,
I have reported this problem before:[( https://github.com/NLog/NLog/issues/5492)]
and I suspect that the architecture of nlog 5.2.8 may be different from that of 5.1.

So you have Old App1 that is NET6 and then you have New App2 that is NET8, and those 2 application are implemented differently. And because you cannot get NLog working in your New App2, then you suspect NLog is the problem without providing any evidence, and expect the NLog maintainer to debug your application for you?
I'm sorry to let you down, but I cannot debug your application for you. You have to perform your own debugging and troubleshooting. If you find evidence of something that can be fixed/improved in NLog, then your feedback is most welcome.
NLog 5.2 adds method-attributes to remove build-warnings for application published with trimming. And also marks several exotic methods as obsolete. But should not change behavior/architecture. See also: https://nlog-project.org/2023/05/30/nlog-5-2-trim-warnings.html

hi , thank you ,
I just tried to change the nlog in my .net8 project to nlog 5.1.0 and nlog.database 5.1.0 . With the original nlog.config mentioned in  [(#5492)] , I can write nlog database /text log file /json log file ,
nlog 5.1  works fine in .net8 , just like it did in .net6

You need to collect some error messages or exception-details, so there is something to investigate.
Sent from my Galaxy
-------- Original message --------
From: hiwoei @.>
Date: 10/04/2024 08:53 (GMT+01:00)
To: NLog/NLog @.>
Cc: Rolf Kristensen @.>, Comment @.>
Subject: Re: [NLog/NLog] NLog.Database version 5.2.8 with net8 error (Issue #5505)
hi , thank you ,
I just tried to change the nlog in my .net8 project to nlog 5.1.0 and nlog.database 5.1.0 . With the original nlog.config mentioned in [(#5492https://github.com/NLog/NLog/issues/5492)] , I can write nlog database /text log file /json log file ,
nlog 5.1 works fine in .net8 , just like it did in .net6
—
Reply to this email directly, view it on GitHubhttps://github.com/NLog/NLog/issues/5505#issuecomment-2046656150, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACXZ7HAG6VLTTHBMPNP6RGTY4TOXHAVCNFSM6AAAAABF5VOAV6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBWGY2TMMJVGA.
You are receiving this because you commented.Message ID: @.***>

I have generated a nlog.trace file
, but without any errors, as follows:
2024-04-10 17:05:28.9451 Debug No file exists at candidate config file location: file path+ EMS24360API.exe.nlog
2024-04-10 17:05:28.9604 Debug No file exists at candidate config file location: file path+ EMS24360API.dll.nlog
2024-04-10 17:05:28.9726 Debug Reading config from XML file: file path+NLog.config
2024-04-10 17:05:28.9851 Info Loading NLog config from XML file: file path+ NLog.config

Again without any evidence of failure from an error-message or exception-details. Then it is hard to investigate,
Sent from my Galaxy
-------- Original message --------
From: hiwoei @.>
Date: 10/04/2024 11:55 (GMT+01:00)
To: NLog/NLog @.>
Cc: Rolf Kristensen @.>, Comment @.>
Subject: Re: [NLog/NLog] NLog.Database version 5.2.8 with net8 error (Issue #5505)
I have generated a nlog.trace file
, but without any errors, as follows:
2024-04-10 17:05:28.9451 Debug No file exists at candidate config file location: file path+ EMS24360API.exe.nlog
2024-04-10 17:05:28.9604 Debug No file exists at candidate config file location: file path+ EMS24360API.dll.nlog
2024-04-10 17:05:28.9726 Debug Reading config from XML file: file path+NLog.config
2024-04-10 17:05:28.9851 Info Loading NLog config from XML file: file path+ NLog.config
—
Reply to this email directly, view it on GitHubhttps://github.com/NLog/NLog/issues/5505#issuecomment-2047078894, or unsubscribehttps://github.com/notifications/unsubscribe-auth/ACXZ7HDM6TJNSXS3OPQRWODY4UEBVAVCNFSM6AAAAABF5VOAV6VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANBXGA3TQOBZGQ.
You are receiving this because you commented.Message ID: @.***>

Closing due to inactivity. If you find something that can be improved, then please re-open issue, or create pull-request.


Where to install the HANA client for windows?

Hi can you please mention how to install the HANA client. I got the SAR file from SAP marketplace but where to place the file?

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

ID: ea414a5c-c3ca-281e-46ca-3e677d27924b
Version Independent ID: 363a9c9c-13e2-58ce-cb79-9f897d83c307
Content: Use SAP HANA in Power BI Desktop - Power BI
Content Source: powerbi-docs/desktop-sap-hana.md
Service: powerbi
GitHub Login: @davidiseminger
Microsoft Alias: davidi

Thanks for your comment, @Robby29. We’ve forwarded your feedback to the writer, who should respond soon.

Thanks for your comment, @Robby29 . This looks like a support request (for SAP, too) rather than documentation feedback. You might find a solution in the Power BI Community, found at https://community.powerbi.com. If not, you can get support by going to https://support.powerbi.com. Or you can check the SAP documentation or support channels.


Update stable version in repo from 1.15.0 to 1.16.1

Think about update stable version to 1.16.1
Version: stable=1.15.0 (200 days old) => latest=1.16.1 (15 days old)
Installs: stable=11118 (68.08%), latest=1068 (6.54%), total=16330
Click to use developer portal
Click to edit
Do not close this issue manually as a new issue will be created if condition for update still exists.
Please drop a comment if any reason exists which blocks updating to version 1.16.1 at this time.
Note: This is an automatically generated message and not personally authored by bluefox!
@mcm1957 for evidence

@jens-maus
@Apollon77
Da ihr bei dem Adapter als Maintainer drinnen seid - gibts da nen Grund die aktuelle Release noch nicht in STABLE zu schieben?

Ja, die fehlende Zeit 😜

Wenns für dich OK ist schalt ich den dann mal durch. OK?

Klar, leg los..

This issue seems to be outdated.
This issue suggests to update the stable version of this adapter to 1.16.1 but the current stable version is already 1.16.1.
So this issue should be closed.
@mcm1957 for evidence


Add a distinct feature flag for RangeCheck1 in Kimchi

The kimchi side of things is a proof-systems ticket.  Closing this and opening new one -> https://app.zenhub.com/workspaces/core-crypto-621fcb82b7d49700156fd060/issues/gh/o1-labs/proof-systems/966


Persistent terminals in repositories with Workspace settings configurations

Hear me out.
For people who need more than one terminal, configuring a set of terminals for their repositories would be great. Instead of opening each terminal and then naming and colouring them from zero after restarting vscode, we could possibly add
the desired configurations to Workspace Settings json file and whenever this repository is opened, the terminals would start too.
Optionally, keeping the history of executed commands in specified terminals would help too.
Example use case:

Hello @Tyriar, I published an extension after opening the feature request. I believe it can be helpful to give you an idea.
The idea is simply adding these list of terminals and respective commands to workspace setting.json file in the repository you want.
{
    "persistentTerminals.terminals": [
        {
            "name": "BACKEND - UVICORN",
            "color": "terminal.ansiGreen",
            "commands": [
                "cd backend",
                "uvicorn app.main:app --reload"
            ]
        },
        {
            "name": "FRONTEND - NEXT",
            "color": "terminal.ansiBlue",
            "commands": [
                "cd frontend",
                "npm run dev",
            ]
        },
        {
            "name": "HEROKU CLOUD",
            "color": "terminal.ansiYellow",
            "commands": [
                "heroku logs"
            ]
        },
    ],
    "persistentTerminals.userRestrictedCommands": [
        "dangerous-custom-script",
        "rm -rf /",
        "sudo rm -rf /",
        "rm -rf /*",
        // "Add whatever you want here to prevent you from shooting yourself in the foot"
    ]
}

Related: https://github.com/microsoft/vscode/issues/224965#issue-2451311876


pull request for e2e test at 01/03/2017 16:15:11

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Special characters in 'enum' type will be compiled to unicode by default

TypeScript Version:  2.9.1

Search Terms: typescript unicode, typescript enum unicode
Code

export const TestObject = {
  WELCOME: '你好',
  TEST: 'тест',
}

export enum TestEnum {
  WELCOME = '你好',
  TEST = 'тест',
}

Expected behavior:
Fields in TestObject and TestEnum ought to be compiled to a consistent encoding format (utf8).
Actual behavior:
Special characters in Object were compiled to utf8 while those in Enum were compiled to unicode.
Playground Link:
Official Playground
Related Issues:
https://github.com/Microsoft/TypeScript/issues/10498

In JSX same problem

I think Typescript should have cli parameter to keep non Latin characters as is (and may be TRUE by default)

Any progress?


Support reporting filenames relative to PYTHONPATH

Is your feature request related to a problem? Please describe.
I run coverage on pytest under tox in my project, and I'm trying to move to a src layout.  That means that a simple import of my package will no longer find it in the current directory, but rather in the venv where tox installed it.
When I do that, then suddenly, I start getting really long filenames in the coverage report:
.tox/site/lib/python3.10/site-packages/systemd_ctypes/bus.py                126     11    91%   141-144, 181-185, 188-190
.tox/site/lib/python3.10/site-packages/systemd_ctypes/event.py               42      2    95%   67, 74
...

and so on.
Describe the solution you'd like
I want the first part of the path to be stripped off, leaving only the module name.
Something like
systemd_ctypes/bus.py                126     11    91%   141-144, 181-185, 188-190
systemd_ctypes/event.py               42      2    95%   67, 74
...

I don't understand why that doesn't work already.  It seems that the PythonFileReporter tries to generate a name based on the module's __name__:
        if hasattr(morf, '__name__'):
            name = morf.__name__.replace(".", os.sep)
            if os.path.basename(filename).startswith('__init__.'):
                name += os.sep + "__init__"
            name += ".py"

but for some reason, this code doesn't engage for me, and I just end up with the long path name.
Describe alternatives you've considered
I thought that maybe relative_files = true would help here, but a quick read through the source seems to indicate that this is about making filenames relative to the current working directory (which is as they get reported, indeed).
Additional context
Here's some relevant pyproject.toml fragments:
...

[tool.coverage.run]
source_pkgs = ["systemd_ctypes"]
relative_files = true

[tool.coverage.report]
fail_under = 85
show_missing = true
skip_covered = true

[tool.tox]
legacy_tox_ini = """
...
commands =
  python3 -m coverage erase
  python3 -m coverage run --module pytest {posargs}
  python3 -m coverage report
...

Hi, sorry it's taken me so long to get to this.
I'm making some improvements (described in #991, which isn't really about slash direction) that might make this easier.  I have a project using src layout (scriv). It's coverage measurement includes this [paths] setting:
[coverage:paths]
source =
    src
    */site-packages

others =
    .
    */scriv

Do you use coverage combine? Have you tried configuring the [paths]?  The latest version of coverage.py has a coverage combine --debug=pathmap option that can show what is being considered during combining.

Thanks for your reply!  It helped a lot.
As things are, I don't use coverage combine in my project.
In response to your suggestion, I've added the following to my pyproject.toml:
[tool.coverage.paths]
source = ["src", "*/site-packages"]

and adjusted my coverage setup to invoke run with -p and add a separate combine step.
python3 -m coverage erase
python3 -m coverage run --parallel-mode --source systemd_ctypes --module pytest {posargs}
python3 -m coverage combine
python3 -m coverage report --skip-covered  --fail-under=85 --show-missing

The result is that the reported paths now look like the following:
src/systemd_ctypes/bus.py                398     40    90%   265, 271-273, 316, 328-333, 459, 546-547, 558-559, 584, 672, 674, 677, 681, 688, 700-702, 705-710, 713-714, 790, 803, 808-809, 833, 856-857
src/systemd_ctypes/event.py               58     10    83%   68, 90-101, 106

which I consider to be acceptable.
That's with python3-coverage-6.4.2-2.fc37.x86_64 (ie: standard coverage package from Fedora) installed, so it doesn't depend on any fancy new feature, either.
So this is at least a partial solution to my problem, and I'd be happy enough to take it for now.  Some things which might make it nicer:

have this work without having to use the combine tool, since this isn't really what it was made for
mapping site-packages/* to some standard prefix might be a nice default

In case anyone else stumbles upon this: I just discovered the existence of pytest-cov, which makes this all quite a bit nicer.
It basically allows to replace the separate commands with a single pytest --cov invocation, and the paths merging works properly with this, as well (although it still needs to be explicitly given in the coverage config).

pytest-cov does work for most of the reports, but not with XML when not using coverage combine. I am still seeing an absolute path in my XML with a src layout project and relative_files = true:
<?xml version="1.0" ?>
<coverage version="6.5.0" timestamp="1667334452453" lines-valid="2110" lines-covered="2048" line-rate="0.9706" branches-valid="808" branches-covered="762" branch-rate="0.9431" complexity="0">
	<!-- Generated by coverage.py: https://coverage.readthedocs.io -->
	<!-- Based on https://raw.githubusercontent.com/cobertura/web/master/htdocs/xml/coverage-04.dtd -->
	<sources>
		<source>/abs/path/to/repo</source>
	</sources>
...

Any ideas on how to fix that?

Just found this after reporting #1674, which is slightly more general compared to this request, namely: Support reporting filenames relative to any configured base directory, and/or support applying arbitrary transformations to paths in a file post-processing step.


⚠ Samsung has degraded performance

In d892075, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2756 ms

Resolved: Samsung performance has improved in c046149.


Global critical points

CODE REVIEW
This implements a variable variance probability algorithm to try and isolate global vs local critical points.  The algorithm is outline here: http://www.iccm-central.org/Proceedings/ICCM16proceedings/contents/pdf/MonK/MoKA1-04ge_ghiasimh224461p.pdf
The basic premise behind it is that we initially select some relatively large number of n-dimensional points at random and then, after every iteration, we rank all of those points against our previously selected guess.  In this way, this is an evolutionary algorithm.  Every successive iteration attempts to choose points distant from the previously selected point using some statistics.  We currently iterate a constant number of times and at the end select a best guess.
This doesn't guarantee we select the true global critical point, it just increases the odds that we do so.  Included is a unit test that has an equation with global and local critical points across the closed set [-4,2].  Given that evaluation is probabilistic, I've commented that test out.  In the future, I'm going to modify it so that we can say confidently we can guess the correct global value some percent of the time and have that test be activated (although it will still fail some small percentage of the time).
In general, this package can use quite a bit of cleanup and probably some more finite unit tests.  Now that I have a stronger grasp on what's required here and where it's going, I can begin that work.
Part of that work will include making a lot of our current constants a variable that can be passed in via the config or make the algorithm itself more adaptive.
@alexandercampbell-wf @beaulyddon-wf @tannermiller-wf @rosshendrickson-wf @ericolson-wf @stevenosborne-wf @tylertreat-wf

+1

+1

+1


Handle nil opts in setup function

Hello, I just saw your plugin on that reddit post and decided to test it. It does look nice, but there is an issue that if the setup call is called without any arguments, it tries to access opts.card_style, which throws an error, since opts is nil.
Since this is very simple, I make a quick fix instead of just opening an issue. I hope that's ok!
Also, you can create a "default options" table in the future and merge with user passed data. The way I coded just handles a nil opts.  You can check for example this line in this plugin: https://github.com/j-hui/fidget.nvim/blob/main/lua/fidget.lua#L518
It uses this vim.tbl_deep_extend built in (in nvim) function to handle that merge table. I'm not experienced with it, but I think it is worth checking in the future if you need to expand your plugin with more options.
By the way, I think you plugin is super cool!

Thank you very much. I saw the extend being before, I'll use it when I have more options to configure. Thank you very much for the bug fix.


Unresponsive extension host for Python

Extension Name: python
Extension Version: 2018.9.0
OS Version: Windows_NT x64 10.0.17763
VSCode version: 1.28.0

We have written the needed data into your clipboard. Please paste:
{
	"activationTimes": {
		"startup": false,
		"codeLoadingTime": 24693,
		"activateCallTime": 2564,
		"activateResolvedTime": 23877,
		"activationEvent": "onLanguage:python"
	}
}

Duplicate of #2827


light sleep sensor has a class issue

System Health details
System Information

version
core-2024.2.1

installation_type
Home Assistant Container

dev
false

hassio
false

docker
true

user
root

virtualenv
false

python_version
3.12.1

os_name
Linux

os_version
6.1.0-rpi8-rpi-v8

arch
aarch64

timezone
Europe/Berlin

config_dir
/config

Home Assistant Community Store

GitHub API
ok

GitHub Content
ok

GitHub Web
ok

GitHub API Calls Remaining
5000

Installed Version
1.34.0

Stage
running

Available Repositories
1397

Downloaded Repositories
50

AccuWeather

can_reach_server
ok

remaining_requests
33

Home Assistant Cloud

logged_in
false

can_reach_cert_server
ok

can_reach_cloud_auth
ok

can_reach_cloud
ok

Dashboards

dashboards
2

resources
34

views
29

mode
storage

Recorder

oldest_recorder_run
January 13, 2024 at 07:00

current_recorder_run
February 11, 2024 at 08:04

estimated_db_size
7355.27 MiB

database_engine
mysql

database_version
11.2.2

Sonoff

version
3.5.4 (a4a8c5f)

cloud_online
3 / 3

local_online
1 / 1

Spotify

api_endpoint_reachable
ok

Checklist

[X] I have enabled debug logging for my installation and pasted in the debug logs below.
[X] I have filled out the issue template to the best of my ability.
[X] This issue only contains 1 issue (if you have multiple issues, open one issue for each issue).
[X] This issue is not a duplicate issue of currently previous issues..

Describe the issue
I got this waring from the regular logs
Integration Version
4.0.0
Reproduction steps

Read logs
see warning

Debug logs
Logger: homeassistant.components.sensor.recorder
Source: components/sensor/recorder.py:301
Integration: Sensor (documentation, issues)
First occurred: 03:05:10 (1 occurrences)
Last logged: 03:05:10

Entity sensor.light_sleep from integration google_fit has state class total_increasing, but its state is not strictly increasing. Triggered by state 17740.0990004539 (18551.0990004539) with last_updated set to 2024-02-12T02:00:45.644005+00:00. Please create a bug report at https://github.com/YorkshireIoT/ha-google-fit/issues

@AlexKusnezov - is this still frequently happening? If so, can you add your sensor history around the times you get the warning?

@YorkshireIoT sorry I just saw you comment. Yes it happens every 24 hours, basically when I wake up and my watch sychronizes sleep data, recent warning:
Entity sensor.sleep from integration google_fit has state class total_increasing, but its state is negative. Triggered by state -570.910999917984 with last_updated set to 2024-03-10T05:32:54.433451+00:00. Please create a bug report at https://github.com/YorkshireIoT/ha-google-fit/issues

That issue was closed in #194 and will be available in the next release. Your original issue mentioned a problem with the light sleep sensor. Are you still getting those warnings?

I am still getting those warnings every day when my watch synchronizes my sleep data. In fact, there are 3 warnings:
Entity sensor.light_sleep from integration google_fit has state class total_increasing, but its state is not strictly increasing. Triggered by state 18780.0 (19260.0) with last_updated set to 2024-05-06T22:43:51.668934+00:00.
Entity sensor.awake_time from integration google_fit has state class total_increasing, but its state is not strictly increasing. Triggered by state 1200.0 (1260.0) with last_updated set to 2024-05-07T00:25:01.152402+00:00.
Entity sensor.rem_sleep from integration google_fit has state class total_increasing, but its state is not strictly increasing. Triggered by state 1800.0 (1920.0) with last_updated set to 2024-05-07T06:44:16.725760+00:00.


Formatter breaking comments

Describe the bug
The formatter sometimes - but not always - breaks commented SQL queries.
I noticed this with a SQL file:
-- This is optimizing the query which is used to summarize the download history
-- in 10 second intervals:
--
-- SELECT COUNT(*) AS numrows
-- FROM (downloaddata)
-- WHERE moddate_m > DATE_SUB(NOW(), INTERVAL 10 SECOND);

Expected behavior
The input should be unmodified
Actual behaviour
-- This is optimizing the query which is used to summarize the download history
-- in 10 second intervals:
--
-- SELECT COUNT(*) AS numrows
-- FROM (downloaddata)
- -
WHERE moddate_m > DATE_SUB(NOW(), INTERVAL 10 SECOND);

Desktop (please complete the following information):

SQLTools Version: 0.23.0
VSCode Version: 1.48.2
OS: Mac

I ma having the same issue.

Sounds to be an old one #97, but still there for me too.

@mtxr I took a quick look at your code and it seems that you handle new comment lines correctly outside of query blocks but not inside.
I am not sure but do you think adding || this.getCommentToken(input) here could handle this case?
https://github.com/mtxr/vscode-sqltools/blob/7c382e8e02b77e7df6147cbc67d19339aabd3a10/packages/formatter/src/core/Tokenizer.ts#L313-L325

You can try to format this in your playground to reproduce 2 different issues with line comments:
select *
from table
where a >= some_date1
and a <  some_date2  -- to
-- and b >= some_date3  -- and
and a <  some_date2
-- and b >= some_date3  -- and
and b <  some_date4  -- where, select etc.
-- and 1 = 1;

It becomes:
SELECT *
FROM TABLE
WHERE a >= some_date1
AND a < some_date2 -- to
- -
AND b >= some_date3 -- and
AND a < some_date2 - -
AND b >= some_date3 -- and
AND b < some_date4 -- where, select etc.
-- and 1 = 1;

And it sounds like your playground use the class operator for the alone -, don't know if it is just rendering stuff or if it could leads to what is really tokenized by your parser.

Should be fixed by #791

Please re-test after updating to 0.25.0 and report back if the problem remains.

It's still broken... @gjsjohnmurray
before formatting

after auto format

@ortonomy is this #1197?

Looks like it @gjsjohnmurray


interactivewebserver example

I am getting 'class WiFiClass' has no member named 'mode' when compiling this sketch any ideas where i am going wrong?

reinstalled ESP32 libraries fixed this


Add multilingual interviews and responses in another language

Hello, it will be possible that soon we will have the option of doing interviews in another language.
My first language is Spanish but I want to access the American market and I just need you to transcribe and give me answers in both English and Spanish

My first language is not English either but I think maybe you can think about editing the Py file to add other languages. Just thinking loudly.

Is Tamil?


Added support for dump on lpar, network dump, dump with different smt…

… level and dumprestart testcases  - v2
Added test case for dump over NFS, SSH, dump with smt levels 1,2,4 and dump with single core and single cpu online.
Signed-off-by: Pavithra<EMAIL_ADDRESS>

Raised new PR, with comments addressed.

Raised new PR, with comments addressed.


Add task list to enhancements template

What type of PR is this?
/kind documentation
What this PR does / why we need it:
Lists possible artifacts that an enhancement entails
Which issue(s) this PR fixes:

Fixes #
Special notes for your reviewer:

/assign @ahg-g

/lgtm


Fix fullscreen UI button for iOS

Fullscreen button was not showing on iOS since it does not support the webkitRequestFullscreen API.
However, we can still support full screen using video.webkitEnterFullscreen.
Using this API has the drawback of not showing the custom UI overlay, just the fullscreen video with the standard controls.
In my opinion, it is still a good option to be able to go fullscreen on iOS.
Fixes https://github.com/AlexxIT/WebRTC/issues/300.

Thanks!


Prepare for 0.21 release

TODOs

[x] Version website
[x] Ask ChatGPT to write a blog post

Whoever merges this PR should publish the following packages:
yew yew-macro yew-router yew-router-macro yew-agent yew-agent-macro
using the publish action

Links to the authors of the changes in the changelog seem to be malformed, is the changelog autogenerated?

Links to the authors of the changes in the changelog seem to be malformed, is the changelog autogenerated?

Yes, it is automated. I would prefer to fix the tool and changelog after release though...

I am a little bit skeptical about the content that ChatGPT generates.

Fair. I reviewed it manually though, so everything that is in there is what got past me

I have this very small PR to include if possible :pray: #3419 (it's okay I understand if not! no worry)

I have this very small PR to include if possible 🙏 #3419 (it's okay I understand if not! no worry)

@cecton what if we published 0.21.1 with this change? I really want to get 0.21.0 out and if we merged it now, it would require redoing the changelog and stuff

@hamza1311 of course!! no worry I perfectly understand. Thank you for you work on this release btw!


How to set other field into the LogRecord for V3

Monolog version 1|2|3?
3
Write your question here.
In Monolog version 2, we could kindly call the pushProcessor method to add our custom processor, which gracefully adds some fields to the log. However, in version 3, the processor's callable has to respectfully accept a LogRecord object, and the LogRecord can only have seven keys - "message", "context", "level", "channel", "datetime", and "extra". We cannot set other fields in the LogRecord object. May I kindly ask if you provide an alternative method that allows us to elegantly customize the LogRecord content?

processors were always meant to store data in the extra array, which is still available in monolog 3.


[4.0] IRGen: Fix ObjectiveC partial apply forwarding stub

We need to map from swift parameter passing ABI to local explosions in partial
apply forwarding stubs to ObjectiveC methods.
• Explanation: Objective C partial apply forwarding thunks were missing code to adapt to the swift calling convention. As a consequence the compiler will assert for certain c types when a closure is built from an objective  c method.
• Scope of Issue: Introduced by the move to the swift calling convention. Forming closures of objective c methods with certain types as arguments will crash the compiler.

Risk: The code adds a map from the swift calling convention to the expected convention. This should only affect cases that were broken before this patch.

Testing: Tested by a unit test
SR-5569
rdar://33591146

@swift-ci Please test


[VT-35] vit; mouseability

David Patrick on 2013-09-17T05:37:52Z says:
You can tell that the mouse is active as it hovers over vit, because the mouse-roller moves the selection-bar 5 tasks up and down.
The first part of this feature-request is to figure out how to get that down to one-task-at-a-time when you roll the mouse-roller.
If we could then figure out leftclick= and rightclick=, then we would have mouseability!

Migrated metadata:
Created: 2013-09-17T05:37:52Z
Modified: 2014-02-09T01:16:51Z


[1.12.2] Crash when using "fill" command

I tried the mod with my server and made a grand new save for it so I can test the fill command. Unfortunately it makes my server crash with java.lang.IllegalAccessError.
I uploaded the full crash log here.

Similar problem on my server
Latest forge 1.12.2 - <IP_ADDRESS>66
java.lang.IllegalAccessError: tried to access field net.minecraft.world.gen.ChunkProviderServer.field_73248_b from class com.wimbli.WorldBorder.task.ChunkUtil 	at com.wimbli.WorldBorder.task.ChunkUtil.unloadChunksIfNotNearSpawn(ChunkUtil.java:20) 	at com.wimbli.WorldBorder.task.WorldFillTask.onServerTick(WorldFillTask.java:365) 	at net.minecraftforge.fml.common.eventhandler.ASMEventHandler_841_WorldFillTask_onServerTick_ServerTickEvent.invoke(.dynamic) 	at net.minecraftforge.fml.common.eventhandler.ASMEventHandler.invoke(ASMEventHandler.java:90) 	at net.minecraftforge.fml.common.eventhandler.EventBus.post(EventBus.java:179) 	at net.minecraftforge.fml.common.FMLCommonHandler.onPreServerTick(FMLCommonHandler.java:277) 	at net.minecraft.server.MinecraftServer.func_71217_p(MinecraftServer.java:655) 	at net.minecraft.server.MinecraftServer.run(MinecraftServer.java:524) 	at java.lang.Thread.run(Thread.java:745)

Thanks for reporting, i didnt do much testing on 1.12 so did not expect this issue, ill have it fixed asap

Did you advance on the problem ? Because i encounter the same error for trim command

Hmm, i cannot seem to reproduce this, testing as much as i can on my test server but it all seems to work fine, maybe some more info as to how you guys are attempting it? through console, player? did u change anything in the config?

I did not change any configuration of worldborder. I use the sponge but i tried without, but i still have this error ..
My server is empty as i execute the command.
The logs of the error :

Sill unable to reproduce, however i have attempted a fix for what the crash report had reported, if you guys could do me a favor and test it before i actually upload it on curse that'd be great!
https://github.com/abused/World-Border/releases/tag/1.0.6-beta

Your fix works perfectly ! The trim and fill is done without problems. Great job :)
I just noticed something in the log (patch 1.0.5 and 1.0.6) worldborder says that the /wb command is used by another mod (it was the only mod on the server). This does not prevent its operation but i wanted to make you go back

All right perfect! As for that log it is because vanilla also has its own worldborder command so I'm seeing what I can do to disable that before I post a release

I tried the new 1.0.6 version, server didn't crash anymore and console says that it fills like it should. But when I did fullrender for Dynmap there's no new chunks.

I can confirm that this fixed the issue with the crashing I also saw.
While the fill command does work, I got this warning in the console.

31.12 22:49:57 [Server] Server thread/ERROR [worldborder]: All WorldBorder commands are being handled elsewhere:
31.12 22:49:57 [Server] Server thread/ERROR [worldborder]: * /wborder (from com.feed_the_beast.ftbu.ranks.CmdOverride)
31.12 22:49:57 [Server] Server thread/ERROR [worldborder]: * /wb (from com.feed_the_beast.ftbu.ranks.CmdOverride)
31.12 22:49:57 [Server] Server thread/ERROR [worldborder]: * /worldborder (from com.feed_the_beast.ftbu.ranks.CmdOverride)
31.12 22:49:57 [Server] Server thread/ERROR [worldborder]: It may be that another mod is attempting to provide world border functionality. Consider removing or disabling that mod to allow WorldBorder-Forge to work properly.

Sill unable to reproduce, however i have attempted a fix for what the crash report had reported, if you guys could do me a favor and test it before i actually upload it on curse that'd be great!
https://github.com/abused/World-Border/releases/tag/1.0.6-beta

I can confirm this fixed it for me as well. Thank you and keep up the good work.


buildHtml to take multiple elements

Not sure if this is by design, but having multiple top level elements in buildHtml does not seem to work:
proc createDom(): VNode =
  result = buildHtml():
    tdiv():
      text "test"
    tdiv():
      text "test"

Leads to: Error: expression 'tmp94011' is of type 'VNode' and has to be discarded
Wrapping them into one top level element solve the problem.

Huh? Pretty sure I have code like this.

Hm, okay not sure what could be different. Here's my full example to reproduce:
include karaxprelude

proc renderer(): VNode =
  result = buildHtml():
    tdiv():
      text "test"
    tdiv():
      text "test"

setRenderer renderer

Running with updated karax & nim devel branches this afternoon.

Oh yeah, well that can't work, there is no root node to attach the 2 divs to.

Shouldn't there be the main/global entry point of the app

?
A use case for a "multiple children buildHtml" would be:
proc renderList(): VNode =
  # don't want to make a decision about the container here
  result = buildHtml():
    tdiv():
      text "child 1"
    tdiv():
      text "child 2"

proc renderer(): VNode =
  result = buildHtml():
    tdiv():
      renderList()
    aDifferentContainer():
      renderList()

This is by the way one of the limitations in Vue (each component must have a single root) and often results in wrapping things in slightly annoying dummy divs.

This is by the way one of the limitations in Vue (each component must have a single root) and often results in wrapping things in slightly annoying dummy divs.

We can perhaps introdue a special VNodeKind.unnest that is embedded into the parent node, but this is more complex than it sounds. :-(

Yes, this is probably not so easy. The question is whether a VComponent should allow for unnested nodes as well, which looks complicated to me. I think it would be an acceptable compromise the require VComponents to be nested, but solve the problem at least for buildHtml to make factoring out view elements easier. I had two ideas to solve this, but I didn't think them through yet.
The first idea was to fully switch to lists of nodes, because html itself is always a list of nodes within . This would mean for instance to define type VNodes: seq[VNode], change the signature to buildHtml(): VNodes, and the renderer would return VNodes. Whenever the karaxdsl comes across a VNodes element it iterates over it and appends all children. Not sure how feasible this is though.
The other idea is to stay with a single VNode return type, but make it easier to pass the context (or parent/currentNode) to sub-view elements. If the karaxdsl would inject a name like currentNode exposing the context, the example from above could be solved like that:
proc renderList(parent: VNode): VNode =
  # don't want to make a decision about the container here
  # solved by forwarding the parent to buildHtml
  result = buildHtml(parent):
    tdiv():
      text "child 1"
    tdiv():
      text "child 2"

proc renderer(): VNode =
  result = buildHtml():
    tdiv():
      renderList(currentNode)
    aDifferentContainer():
      renderList(currentNode)

This might clash (or rather confuse people) with the current overload of buildHtml to support a tag as its first parameter. Personally have never seen a big need for this, at least not, if the following two are equivalent anyway:
# Writing the root tag in the body looks good to me
result = buildHtml():
  tdiv(id="A"):
    tdiv(id="sub"):
      text "a"

# Do we need support for this, any benefit?
result = buildHtml(tdiv(id="A")):
  tdiv(id="sub"):
    text "a"

This second solution should work fine for inner levels -- on the outermost level it could be tricky if it requires e.g. a dummy parent.

FYI: Inspired by Karax, I wrote a similar DSL to generate SVG files/animations. As an experiment I modified the DSL in a way that the main buildSvg macro returns a seq[Node] making it possible to handle flat structures in general. Maybe this would work for Karax as well? If you are interested here is the macro.

I second this feature.

I'm going to close this for now due to inactivity. It sounds like this may be able to be supported in the future, but is not currently. Please feel free to re-open and comment if you want to discuss further!


TFSLIM UnknownError Input/Output Error.

Please go to Stack Overflow for help and support:
https://stackoverflow.com/questions/tagged/tensorflow
If you open a GitHub issue, here is our policy:

It must be a bug, a feature request, or a significant problem with documentation (for small docs fixes please send a PR instead).
The form below must be filled out.
It shouldn't be a TensorBoard issue. Those go here.

Here's why we have that policy: TensorFlow developers respond to issues. We want to focus on work that benefits the whole community, e.g., fixing bugs and adding features. Support only helps individuals. GitHub also notifies thousands of people when issues are filed. We want them to see you communicating an interesting problem, rather than being redirected to Stack Overflow.

System information

Have I written custom code (as opposed to using a stock example script provided in TensorFlow):
OS Platform and Distribution (e.g., Linux Ubuntu 16.04):
TensorFlow installed from (source or binary):
TensorFlow version (use command below):
Python version:
Bazel version (if compiling from source):
GCC/Compiler version (if compiling from source):
CUDA/cuDNN version:
GPU model and memory:
Exact command to reproduce:

You can collect some of this information using our environment capture script:
https://github.com/tensorflow/tensorflow/tree/master/tools/tf_env_collect.sh
You can obtain the TensorFlow version with
python -c "import tensorflow as tf; print(tf.GIT_VERSION, tf.VERSION)"
Describe the problem
Describe the problem clearly here. Be sure to convey here why it's a bug in TensorFlow or a feature request.
Source code / logs
Include any logs or source code that would be helpful to diagnose the problem. If including tracebacks, please include the full traceback. Large logs and files should be attached. Try to provide a reproducible test case that is the bare minimum necessary to generate the problem.
I have created tfrecord files and am getting the following error during my training:
INFO:tensorflow:global step 3338: loss: 0.4686 (2.67 sec/step)
step=1217
INFO:tensorflow:global step 3339: loss: 0.4468 (2.57 sec/step)
step=1218
INFO:tensorflow:global step 3340: loss: 0.4400 (2.66 sec/step)
step=1219
INFO:tensorflow:global step 3341: loss: 0.5029 (2.76 sec/step)
step=1220
INFO:tensorflow:global step 3342: loss: 0.3761 (43.48 sec/step)
INFO:tensorflow:Error reported to Coordinator: <class 'tensorflow.python.framework.errors_impl.UnknownError'>, /content/drive/app/images/train/fmri_train_00000-of-00001.tfrecord; Input/output error
[[Node: parallel_read/ReaderReadV2 = ReaderReadV2[_device="/job:localhost/replica:0/task:0/device:CPU:0"](parallel_read/TFRecordReaderV2, parallel_read/filenames)]]
Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py", line 1361, in _do_call
return fn(*args)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py", line 1340, in _run_fn
target_list, status, run_metadata)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/errors_impl.py", line 516, in exit
c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.OutOfRangeError: FIFOQueue '_3_batch/fifo_queue' is closed and has insufficient elements (requested 32, current size 0)
[[Node: batch = QueueDequeueUpToV2[component_types=[DT_FLOAT, DT_UINT8, DT_INT64], timeout_ms=-1, _device="/job:localhost/replica:0/task:0/device:CPU:0"](batch/fifo_queue, batch/n)]]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/training/supervisor.py", line 990, in managed_session
yield sess
File "train_alz.py", line 282, in run
summaries = sess.run(my_summary_op)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py", line 905, in run
run_metadata_ptr)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py", line 1137, in _run
feed_dict_tensor, options, run_metadata)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py", line 1355, in _do_run
options, run_metadata)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py", line 1374, in _do_call
raise type(e)(node_def, op, message)
tensorflow.python.framework.errors_impl.OutOfRangeError: FIFOQueue '_3_batch/fifo_queue' is closed and has insufficient elements (requested 32, current size 0)
[[Node: batch = QueueDequeueUpToV2[component_types=[DT_FLOAT, DT_UINT8, DT_INT64], timeout_ms=-1, _device="/job:localhost/replica:0/task:0/device:CPU:0"](batch/fifo_queue, batch/n)]]
Caused by op 'batch', defined at:
File "train_alz.py", line 304, in
run()
File "train_alz.py", line 184, in run
images, _, labels = load_batch(dataset, batch_size=batch_size)
File "train_alz.py", line 168, in load_batch
allow_smaller_final_batch = True)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/training/input.py", line 989, in batch
name=name)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/training/input.py", line 761, in _batch
dequeued = queue.dequeue_up_to(batch_size, name=name)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/data_flow_ops.py", line 527, in dequeue_up_to
self._queue_ref, n=n, component_types=self._dtypes, name=name)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/ops/gen_data_flow_ops.py", line 2557, in _queue_dequeue_up_to_v2
component_types=component_types, timeout_ms=timeout_ms, name=name)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/op_def_library.py", line 787, in _apply_op_helper
op_def=op_def)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/ops.py", line 3271, in create_op
op_def=op_def)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/ops.py", line 1650, in init
self._traceback = self._graph._extract_stack()  # pylint: disable=protected-access
OutOfRangeError (see above for traceback): FIFOQueue '_3_batch/fifo_queue' is closed and has insufficient elements (requested 32, current size 0)
[[Node: batch = QueueDequeueUpToV2[component_types=[DT_FLOAT, DT_UINT8, DT_INT64], timeout_ms=-1, _device="/job:localhost/replica:0/task:0/device:CPU:0"](batch/fifo_queue, batch/n)]]
During handling of the above exception, another exception occurred:
Traceback (most recent call last):
File "train_alz.py", line 304, in
run()
File "train_alz.py", line 300, in run
sv.saver.save(sess, sv.save_path, global_step = sv.global_step)
File "/usr/lib/python3.6/contextlib.py", line 99, in exit
self.gen.throw(type, value, traceback)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/training/supervisor.py", line 1000, in managed_session
self.stop(close_summary_writer=close_summary_writer)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/training/supervisor.py", line 828, in stop
ignore_live_threads=ignore_live_threads)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/training/coordinator.py", line 389, in join
six.reraise(*self._exc_info_to_raise)
File "/usr/local/lib/python3.6/dist-packages/six.py", line 693, in reraise
raise value
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/training/queue_runner_impl.py", line 252, in _run
enqueue_callable()
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/client/session.py", line 1259, in _single_operation_run
None)
File "/usr/local/lib/python3.6/dist-packages/tensorflow/python/framework/errors_impl.py", line 516, in exit
c_api.TF_GetCode(self.status.status))
tensorflow.python.framework.errors_impl.UnknownError: /content/drive/app/images/train/fmri_train_00000-of-00001.tfrecord; Input/output error
[[Node: parallel_read/ReaderReadV2 = ReaderReadV2[_device="/job:localhost/replica:0/task:0/device:CPU:0"](parallel_read/TFRecordReaderV2, parallel_read/filenames)]]
I am a newbie to tensorflow and would really appreciate any help!

Nagging Assignee @rohan100jain: It has been 14 days with no activity and this issue has an assignee. Please update the label and/or status accordingly.

It has been 14 days with no activity and the awaiting response label was assigned. Is this still an issue?

It has been 29 days with no activity and the awaiting response label was assigned. Is this still an issue?

We are closing this issue for now due to lack of activity. Please comment if this is still an issue for you. Thanks!


Q: contour from data

Hi,
I'm trying to plot the contour of an activation map.
I'm struggling to understand how I can convert my data array whose shape is n_vertices=20484 (10242 per hemisphere of fsaverage with oct=5) to an overlay array of shape 163842, (which seems to relate to brain._geo.curv?)
Any idea on how to do this?

stc.morph with the "None" grade designed to fill the high-density mesh

Oh sorry though this was MNE, heh.
Take a look at how STCs would be plotted using PySurfer, it would use the same code. You need to upsample it using smoothing steps. If you can't find it in the docs I'll look.

Nope, I'm getting stuck. I don't know the package well enough :/
Are you saying we can plot activation contours in MNE?

What do you mean by "activation contours"? You mean STCs? Yes stc.plot() wraps to PySurfer.

For non-STCs look at the STC example, it upsamples from the decimated mesh to the high-resolution one:
https://github.com/nipy/PySurfer/blob/master/examples/plot_meg_inverse_solution.py#L60

I would like to add a contour overlay on top of activation map similar. The contour would be similar to this, but using a single line:

This would be to show the effect size (color coded) and surround the significant clusters with a line analogous to this:

Do stc.morph in MNE-Python with grade=None to fill the high-resolution mesh
Find a PySurfer example of contour plots
Tweak the properties of the underlying Mayavi object if necessary to get the single contour to be at the right value

(after morph you can use stc.lh_data and stc.rh_data or similar to get the left and right hemi data, which PySurfer will want)

Works nicely, thanks.


Add storage common property configuration supports all Storage features

As title.
Add properties prefixed with spring.cloud.azure.storage, so that common properties such as account-name, connection-string, etc. can be shared.

API change check
APIView has identified API level changes in this PR and created following API reviews.
spring-cloud-azure-autoconfigure

https://github.com/Azure/azure-sdk-for-java/issues/28461

/azp run java - spring - ci

/azp run java - spring - tests

Hi @alzimmermsft, we changed some annotations for some AutoConfiguration classes in this PR, which aren't breaking changes in Spring. But now it seems like the java-code-quality-reports-ci failed because of this https://dev.azure.com/azure-sdk/public/_build/results?buildId=1615699&view=results. But we marked these changes in the revapi.json file as exceptions. Do you know how to make the java-code-quality-reports-ci ignore them too?

Hi @alzimmermsft, we changed some annotations for some AutoConfiguration classes in this PR, which aren't breaking changes in Spring. But now it seems like the java-code-quality-reports-ci failed because of this https://dev.azure.com/azure-sdk/public/_build/results?buildId=1615699&view=results. But we marked these changes in the revapi.json file as exceptions. Do you know how to make the java-code-quality-reports-ci ignore them too?

At a quick glance the new ignores added to RevApi should be ignoring what is being reported, I'll need to take a deeper look to understand why this isn't happening.

@alzimmermsft thanks!


Delete example.py

It is not related to the aims of the project so I believe it should be removed from the repo.

I believe single.py should be cleared as well since we don' t use it anymore (was just a first version)


How to run apply_net.py after model training?

Hello @rakhimovv
I am training the model with s0_bv2_bifpn_f64_s3x.yaml config.
When I try to run apply_net.py on sample images, predictions seem to be very off.
Command I am using = "python apply_net.py show configs/s0_bv2_bifpn_f64_s3x.yaml model_0104999.pth input.jpg bbox,dp_contour,dp_u -v"
Is there any specific instruction to run apply_net.py?

Example output.

Looks like the issue with the detectron2 version in my setup. Closing the issue.


map legend does not contain image service

The map legend does not contain the  image service used in the app.  See screen grab below.

@NatalieCampos can you add the screenshot you have for the map legend. This is the one from Layer Transparency.


NPM Automation

Fetch a list of all NPM releases with a script called npm.py and generate a release file: https://github.com/endoflife-date/release-data/tree/main/releases
See
https://github.com/endoflife-date/release-data/issues/15 and https://github.com/endoflife-date/release-data/issues/10 for example.

To work on this:

Clone this repo (git clone git@github.com:endoflife-date/release-data.git)
Clone the endoflife-date repository inside the website directory (git clone https://github.com/endoflife-date/endoflife.date website)
Make sure you have Python 3 installed
Install dependencies pip install -r requirements.txt
Run Existing scripts to verify (python src/pypi.py) for eg.
Update a page belonging to the npm ecosystem in website/products to use auto: npm
Write your npm.py script in the src directory and test it against this one project.

Have created a pull request for the same: https://github.com/endoflife-date/release-data/pull/36

Closed by #36.


SQLiteConstraintException in createMediaStoreUri

Describe the bug
I'm using modernstorage to save a file from the web to my Downloads folder.
In certain situations, the following error occurs:
java.util.concurrent.ExecutionException: android.database.sqlite.SQLiteConstraintException: UNIQUE constraint failed: files._data (code 2067 SQLITE_CONSTRAINT_UNIQUE)
        at androidx.work.impl.utils.futures.AbstractFuture.getDoneValue(AbstractFuture.java:516)
        at androidx.work.impl.utils.futures.AbstractFuture.get(AbstractFuture.java:475)
        at androidx.work.impl.WorkerWrapper$2.run(WorkerWrapper.java:321)
        at androidx.work.impl.utils.SerialExecutor$Task.run(SerialExecutor.java:91)
        at java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1167)
        at java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:641)
        at java.lang.Thread.run(Thread.java:923)
     Caused by: android.database.sqlite.SQLiteConstraintException: UNIQUE constraint failed: files._data (code 2067 SQLITE_CONSTRAINT_UNIQUE)
        at android.database.DatabaseUtils.readExceptionFromParcel(DatabaseUtils.java:178)
        at android.database.DatabaseUtils.readExceptionFromParcel(DatabaseUtils.java:142)
        at android.content.ContentProviderProxy.insert(ContentProviderNative.java:549)
        at android.content.ContentResolver.insert(ContentResolver.java:2149)
        at android.content.ContentResolver.insert(ContentResolver.java:2111)
        at com.google.modernstorage.storage.SharedFileSystem.createMediaStoreUri(SharedFileSystem.kt:266)

To Reproduce
As I managed to find out, this happens if you download a file in the application, then clear the application data and try to download the same file again.
Environment:

Android OS version: Android  11
Target SDK version: API 31
Device: Xiaomi Mi 8 (PE)
ModernStorage version: v1.0.0-alpha03

Additional context
Some peaces of code, witch I use:
        val destPath = fileSystem.createMediaStoreUri(
            filename = depFileName,
            directory = getExternalStorageDir().absolutePath
        )?.toPath()

    private fun getExternalStorageDir(): File {
        return if (Build.VERSION.SDK_INT >= Build.VERSION_CODES.Q) {
            applicationContext.getExternalFilesDir(Environment.DIRECTORY_DOWNLOADS)!!
        } else {
            @Suppress("DEPRECATION")
            Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DOWNLOADS)
        }
    }

Let me give a try to reproduce the issue. I might escalate the problem with the Android storage engineers if it's not related to ModernStorage

So I investigated a bit but wasn't able to reproduce the same issue you're getting (at least on an emulator).
Two things to note though, your getExternalStorageDir is pointing to two different folders. The first one is the app's own Download external folder (which isn't visible by other apps since Android 11, was never supposed to be public to be honest) and the second one is the shared Download folder (visible with a file manager):
# applicationContext.getExternalFilesDir(Environment.DIRECTORY_DOWNLOADS)
/storage/emulated/0/Android/data/com.google.modernstorage.storage.test/files/Download

# Environment.getExternalStoragePublicDirectory(Environment.DIRECTORY_DOWNLOADS)
/storage/emulated/0/Download

On Android 11 (I haven't tried yet on previous versions), it doesn't let me write to the app's own Download external folder. It doesn't trigger an exception but change the target folder by putting it in the device shared Download folder.
I've tried setting the same file name but end up having MediaStore adding a suffix automatically:
# First try
uri = content://media/external/file/65
path = /storage/emulated/0/Download/samefile.pdf

# Second try
uri = content://media/external/file/66
path = /storage/emulated/0/Download/samefile (1).pdf

Could you share me a snippet adding the same file twice from your project just to make sure I'm reproducing it in the same way? Thanks!


On-going maintenence after forking a pattern

I have a question I don't see an answer to anywhere in the docs. Happy to put an answer here into a PR!
Question: Let's say I've forked a pattern repo (multicluster-gitops for example) and deployed a hub cluster with a few managed clusters. A few months go by and the upstream project has had several changes. Am I expected to pull and merge those changes into my fork and redeploy?

Hi @RyanMillerC,
Thanks for the question, let me try and give this question an initial tentative answer:
In general I think the answer is "Unless you need some new features from the upstream project, there is no need to upgrade your repo". There is also a bit of nuance in that answer, namely by default most (all?) patterns define some subscription based on certain channels without pinning a specific version, so your cluster is updating to a new operator versions as soon as they are released in that channel.
In general this is almost never a problem, but we have seen the occasional compatibility broken, which then either got fixed by a subsequent update or by a change in code. They are quite rare though, so it really depends.
Hope this covers your question somewhat?

I am closing this one, let us know if there are any more questions/doubt about this


SharedRackApplicationFactory substitutes error application for real one

If the rack application fails to initialize properly, SharedRackApplicationFactory creates a new application that delegates to the error application, responding to every request with the initialization exception stack trace.
This behavior can be helpful during development, but it's awful for production, and there's no way to disable it. In production, it would be best for the server not to start listening on the specified port. The desired behavior should be configurable, perhaps via the servlet context.

as a compromise you can now override handleInitializationException
instead of having to copy paste the whole contextInitialized code ...

Thanks!


Added framework for license application payments

Added framework code for:

New interfaces project for BC Express Pay (BCEP) interface
New payment controller and unit test
Updated application submit component to call payment controller and re-direct to payment url
Added new component "payment-confirmation" to receive re-direct back from payment service with success/fail code

Functionality is still to be implemented, however would like a review for design, as well as updates to .csproj and .sln files.
Note this includes updates from rafael fork which are not yet PR'ed back to master.

@GeorgeWalker I expect this will break the build in OpenShift due to updates to .csproj and .sln files.


make maxUAMSMBBasalMinutes a preference

Currently, SMB is limited to delivering 30m worth of basal at a time unless IOB is insufficient to cover COB (generally after meal carbs have been entered, but only a partial bolus or no manual bolus has been given).  Some people have found it necessary to make SMB more aggressive even in the absence of meal announcements, so this provides an optional preference to control SMB's maximum bolus in the UAM case similarly to how maxSMBBasalMinutes controls it in the announced-meal case.
Special thanks to @dcacklam for identifying this as a problem and testing a similar fix.

Code-wise this looks fine to me. Will merge for part of 0.6.1 testing tomorrow if there are no objections.


How to narrow the range of azimuth angles

Hello,
We are evaluating picoScan with ROS2, but does not seem to be able to change the "Scan Range Filter".
Would you please tell us how to narrow the range of azimuth angles ?

ROS2 humble
picoScan V1.3.0 (PN1134609)
sick_scan_xd 3.4.0
The following site say.
"An azimuth filter is currently not supported for picoScan lidars."
https://github.com/SICKAG/sick_scan_xd/issues/266
When is it expected to improve ?
Thank you in advance.

Thanks for your feedback. As far as we know, an azimuth filter (LFPangleRangeFilter) is not supported by picoScan lidars. We will clarify with SICK whether support for this feature is planned.

As an alternative option, the picoScan150 offers native ROS2 (DDS) to connect to ROS2 Humble without the need for an external driver. With this feature, you can use the scan range filter. Please use https://supportportal.sick.com/ to contact SICK support.


core-api for Go implementation

I'm working on the go-ipfs-core-api in go-ipfs/core/coreapi (ipfs/go-ipfs#2876) for now, but I guess ideally it should move here? Should I move the js stuff into a subdirectory?
Also, should this repo be named something something core-api, instead of core interface? The terms API and interface aren't neccessarily synonymous. I know it's late for this concern, I just picked up the go-core-api work again and found it a bit irritating.

but I guess ideally it should move here?

yes

Should I move the js stuff into a subdirectory?

Probably best to move all things that can be moved related to js into a folder javascript and add another folder go with the go things. Should also have a note on this in the readme

interface-core-api/go roadmap
The PM tracking milestone for this is here for now: https://github.com/ipfs/go-ipfs/milestone/19
The preliminary roadmap looks as follows:

gateway powered by go-ipfs/core/coreapi
preliminary Unixfs API: Cat(), Ls(), with tests -- ipfs/go-ipfs#3207
preliminary Unixfs API: Add(), with tests
Object API: AddLink(), RmLink()
Unixfs API specs
go-ipfs/core/coreapi moved to interface-ipfs-core/go
gateway extracted from go-ipfs to go-ipfs-gateway
go-ipfs-api implements Core API
unixfs, object, block
etc.
go-ipfs commands use Core API
unixfs, object, block
etc.
more API specs and implementations in go-ipfs-api and interface-core-api

@lgierth anything that can be PR'ed to this repo? I'm interested to know if the current way of defining the API works for go land as well

This exists now and we even got a tutorial for it:

https://blog.ipfs.io/073-go-ipfs-as-a-library/
https://github.com/ipfs/go-ipfs/tree/master/docs/examples/go-ipfs-as-a-library


Queue management module and unit tests for socket listener

First version of the queue management module with all required functionality implemented.
Unit tests for the Queue management module implemented.
Unit tests for ReceiveSocketListener and SendSocketListener (client-side) implemented.

Put SocketManagement testing folder inside Networking

Somewhere you guys have used
`function(){
...

}`
while at some places
`function()
{
...

}`
Stick to the second style only.


Choose what version of packages you want to install in the GUI

Maybe implement a gui box to choose what version of package you want just like how Hot Chocolatey already does it.
https://imgur.com/a/raFQw7f

why is this not getting replies?

@nutterthanos apologies for not replying sooner about this, I didn't see the notifications about this issue.  This suggestion is really a duplicate of what was suggest here: https://github.com/chocolatey/ChocolateyGUI/issues/545.  There is an open PR for providing this functionality here: https://github.com/chocolatey/ChocolateyGUI/pull/823 would be great if you can provide feedback on that.  I am going to close this issue out as a duplicate.


Refactor: Architecture

closes #97
closes #200
closes #201
closes #203
closes #204
Easier to review one commit at a time.
Some commit messages are long, and their content should be reviewed (i.e. check that the changes are okay).
Other changes not mentioned in commit messages:

update_borrower in borrower_utils.py had different logic, but for no reason (the dict argument set exactly the same keys that the function accessed individually). It also called session.refresh() instead of session.add().

In one commit message I mention:

These functions previously didn't set created_at:

copy_application
copy_documents
create_application_action
create_credit_product
create_or_update_borrower_document
save_message_type

However, all created_at fields (except Statistic) have default=datetime.utcnow(), so created_at doesn't need to be explicitly set.
The functions that did set it right before flush() were:

insert_application
insert_message
create_message (in both application_utils and utils/applications)
_insert_award
_insert_borrower

Since these set default=datetime.utcnow(), the extra assignment can be removed.

@jpmckinney I see background_processes, core, schema and db are empty directories now, can I delete them as part of this PR?

They are gone from the PR, but you might need to delete yours locally.

#202 Is completed in #208


🚀 use the screens migrated by default

⚠ Do not merge must go with the 3.14 ⚠
Issue
https://github.com/gravitee-io/issues/issues/6540
Description
The end 🏁
Use new Org Settings screens by default
Keep legacy ones with ajs- prefix
Additional context

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.

for the commit : style: prefer to use "delete" instead of "remove" for element removal ... is view with @mouligno . But if you have any suggestion I take my English is not very subtle 💌

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.

✅  LGTM, we should just post a message in slack to share the info before merging this

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.

Gravitee.io Automatic Deployment
📚 View the storybook of this branch here

Gravitee.io Automatic Deployment
Congrats, all the CI steps were 🟢 . The Bot was able to deploy the build of this PR, so you can now try it directly here! 🚀
Notes: The deployed app is linked to the management API of the Element Zero team's environment.


[BEAM-822] Move the separate package-info.java compile to java7 profile

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

Move the separate package-info.java compile to java7 profile since it's not needed with java8
This fixes compiling in Eclipse (assuming Neon which require java8)
Slightly speeds up Java8 compile (one invoke of compiler plugin)

@kennknowles : please review as this somewhat redoes the work you did

Please merge and add the comments as you understand the need for those flags better than I.  I was just trying to get the effective pom for java7 equivalent to what you had and always use those flags for Java8.

OK. Waiting for the Travis test matrix, as variation in JDKs is key here.

Travis looks ok.  The Jenkins failure seems completely unrelated.  (and I cannot reproduce)

Yup, Travis is happy. That Jenkins error is so opaque there's nothing to be done. I'll merge.


Installation error

ERROR: Download of https://dl.bintray.com/lovell/sharp/libvips-8.4.2-linux-x64.tar.gz failed: Response code 401 (Unauthorized)

Please see #819


Synchronize nomenclature with CloudFlare dashboard

It's confusing to call have a section called "Image Optimization" under Speed. This is different nomenclature than used in the CloudFlare Dashboard which used Polish and Mirage.  Is this feature on of those, or both at the same time?
v3.0.3 (and prior versions)

Both at the same time. This was a conscious choice on our end to simplify the features for non-tech users.


npm error every time i log into my account

Describe the bug
npm error every time i log into my account it starts the install npm command but then it fails and i cannot see the preview of my application. this happens every and I don't get a chance to deploy what i have build.
Link to the Bolt URL that caused the error
https://bolt.new/~/sb1-4yhfiy
Steps to reproduce
i would like to log in and not have to get npm install errors every time.
Expected behavior
i would like to log in and not have to get npm install errors every time.
Screen Recording / Screenshot

Platform

OS: [e.g. macOS, Windows, Linux]
Browser: [e.g. Chrome, Safari, Firefox]
Version: [e.g. 91.1]

Additional context
No response

Appreciate the feedback! We are aware of this issue and are tracking it here: #2019


FI-3435: Documentation for DTR Light EHR release

Summary
This PR updates the documentation of the DTR Light EHR test suite and general DTR test kit and home page documentation in preparation for the initial publication of the Light EHR suite.
Code Changes

I tried to organize the RESTful capability profiles in a better way by following the structure of the USCore v3.1.1 fhir api test group. I created a group.rb file for each of the profiles so that I could include detailed descriptions for each without cluttering the dtr_light_ehr_suite.rb file.
docs/dtr_light_ehr_suite_description_v201.md

Added DTR QuestionnaireResponse and CDex Task explanations to limitations section (is this necessary? is this the spot for it?)

Testing Guidance

Read through updates to documentation and make sure there are no typos/errors.
Make sure details are relevant and not redundant.
Let me know if anything is unclear/missing.
Run the test kit as if you were a branch new user and make sure the instructions make sense.

Questions

Should I update the preset to use the public instance of the inferno reference server instead of localhost now that that patient has been merged?
What should I name test groups 2 and 3? One is for US Core API and the other is for the DTR Light EHR profile API.

@karlnaden Thanks for the feedback! Regarding your main comments:

Addressed in Input titles and descriptions.

For QuestionnaireResponse description, I was wondering if we could discuss this more? I am not totally sure what details are relevant here. I did a lot of copy-paste for this one from USCore because these tests were made with the USCore generator so I thought that made the most sense?

the repo README needs to be updated with information about the Light EHR tests.


Update deep-links.md

Added note to say topics is not supported on mobile & deep link to scheduling is not yet available on mobile.

OPS Build status updates of commit 169ace2:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/get-started/get-started-app-studio.md
:warning:Warning

Details

msteams-platform/concepts/task-modules/task-modules-tabs.md
:warning:Warning

Details

msteams-platform/concepts/deep-links.md
:white_check_mark:Succeeded

msteams-platform/get-started/get-started-app-studio.md

Line 15: [Warning] Invalid file link:(~/get-started/get-started-nodejs-app-studio).
Line 54: [Warning] Invalid file link:(~/get-started/~%5Cresources%5Cschema%5Cmanifest-schema.md).
Line 92: [Warning] Invalid file link:(~/get-started/~%5Cconcepts%5Cmessaging-extensions.md).

msteams-platform/concepts/task-modules/task-modules-tabs.md

Line 29: [Warning] Invalid file link:(~/concepts/task-modules/~%5Cconcepts%5Ctask-modules%5Ctask-modules-overview.md#the-taskinfo-object).
Line 32: [Warning] Invalid file link:(~/concepts/task-modules/~%5Cconcepts%5Ctask-modules%5Ctask-modules-overview.md#adaptive-card-or-adaptive-card-bot-card-attachment).
Line 36: [Warning] Invalid file link:(~/concepts/task-modules/~%5Cconcepts%5Ctask-modules%5Ctask-modules-overview.md#task-module-samples).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.

OPS Build status updates of commit dd7565f:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/get-started/get-started-app-studio.md
:warning:Warning

Details

msteams-platform/concepts/task-modules/task-modules-tabs.md
:warning:Warning

Details

msteams-platform/concepts/deep-links.md
:white_check_mark:Succeeded

msteams-platform/get-started/get-started-app-studio.md

Line 15: [Warning] Invalid file link:(~/get-started/get-started-nodejs-app-studio).
Line 54: [Warning] Invalid file link:(~/get-started/~%5Cresources%5Cschema%5Cmanifest-schema.md).
Line 92: [Warning] Invalid file link:(~/get-started/~%5Cconcepts%5Cmessaging-extensions.md).

msteams-platform/concepts/task-modules/task-modules-tabs.md

Line 29: [Warning] Invalid file link:(~/concepts/task-modules/~%5Cconcepts%5Ctask-modules%5Ctask-modules-overview.md#the-taskinfo-object).
Line 32: [Warning] Invalid file link:(~/concepts/task-modules/~%5Cconcepts%5Ctask-modules%5Ctask-modules-overview.md#adaptive-card-or-adaptive-card-bot-card-attachment).
Line 36: [Warning] Invalid file link:(~/concepts/task-modules/~%5Cconcepts%5Ctask-modules%5Ctask-modules-overview.md#task-module-samples).

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Added translation strings for Italian language

Added translation strings for Italian language.
Built and tested, working properly AFAICT (i.e., with java -Duser.language=it -jar epubcheck.jar ...)

Great, thanks for the contribution, Alberto! :)
Any chance you can squash the commits to a single one for i18n IT and push those to your master?
You should also have a look first at recently merged PR #497 which changes PKG_005

Sure, I forgot to rebase. I will look into this next Monday (2015-02-16). I will close this pull request, and open another one w/ the rebased commit.

Leave it open and just rebase your master branch... This will update this PR

That's a nice feature of GitHub I did not know about!
OK, I fetched the remote master update, updated the Italian string for PKG_005, squashed, and forced pushed.
It should be OK for you to merge, apologies for forgetting to squash in the first place.

#1

Great! But you probably did rebase on the wrong master commit as your master tree https://github.com/pettarin/epubcheck/commits/master misses two commits in comparison to https://github.com/IDPF/epubcheck/commits/master, right?
I don't know hwat happens when I'll merge this anyway so I leave it open for now...

I guess the problem was that the fetch from the upstream (which brought in your edits) in the middle of my rebase.
Now I fetched the upstream again, and pushed to my master, so the f092b86 is now "ahead" of the IDPF master, and good for you to merge --- the only diff files are the *_it message files.
Anyhow, if you prefer, just close this pull and I will open another one from a "cleaner state".

Great! This looks perfect now! Thanks for the contribution, Alberto! :+1:


Add a video or two to "fake" media library

It would be very handy to test videos without uploading on some situations ;)
Some suggestions to keep the cat theme 😸
https://pixabay.com/en/videos/cute-cat-funny-cat-kitten-domestic-3092/
https://pixabay.com/en/videos/cat-cat-thailand-small-indian-civet-1297/

Loved your suggestions :heart_eyes_cat:
2 videos have been added to v0.4.0-rc.1 and the preview mode of video has also fixed. Can you test it before the final release?


Re-initializing colorscheme when switching background 'light' and 'dark'

Is there a way to get lightline to reflect the current colorscheme when the background is switched between 'light' and 'dark'.
If I switch the background (light/dark) for solarized, lightline retains the initial colorscheme background colors.
lightline#init, #colorscheme, #enable etc do not seem to have any affect.

:h lightline-problem-13

My fault.. didn't see problem report #104. The outlined procedure works perfectly.
Thanks for lightline! It rocks!


Add breaking changes for WPF

Summary
Document two old breaking changes.
@gewarren I improvised on the designs to match the latest way of documenting these I could see.
Fixes #43282
Fixes #43285

Internal previews

📄 File
🔗 Preview link

docs/core/compatibility/3.0.md
docs/core/compatibility/3.0

docs/core/compatibility/7.0.md
docs/core/compatibility/7.0

docs/core/compatibility/wpf/3.0/drag-and-drop.md
docs/core/compatibility/wpf/3.0/drag-and-drop

docs/core/compatibility/wpf/7.0/drag-and-drop.md
docs/core/compatibility/wpf/7.0/drag-and-drop

Ouch! Lots of little fixes :) Thank you, I'll get on them!


Bundle both esm and cjs for compatibility

Clear and concise description of the problem
I work in a codebase where it is too much of a lift to switch to esm just so that we can use this package. It would be great if this package also emitted cjs code so that I can use version 7, which has some props that i need that are not in version 6.
Suggested solution
Make the bundling step emit both esm and cjs.
Alternative
No response
Additional context
No response
Validations

[X] Follow our Code of Conduct
[X] Read the docs.
[X] Check that there isn't already an issue that request the same feature to avoid creating a duplicate.

Would you like to open a PR for this feature?

[ ] I'm willing to open a PR

That is not an option, and dropping CJS modules was an intended move. You still have options to use version, for example by using Babel on it on your side


Add a way to sort the library (alphabetically, by most played or download order)

Build a simple sort option for the library:

hello, I would like to start contributing to the project. Can I take on this task?

Before doing so, I would like to suggest you to move the library to a separate tab, with search options too
Having all the games on the nav bar is a bit inconvenient, especially as the list grows

https://github.com/hydralauncher/hydra/assets/33422772/d36986d1-f7de-46c1-9d1c-8160410a3b44

I think having downloads always at the top it's a must.


Change entry of language French

Hello,
When I change a entry of french language, and submit i give a error of serveur ("Error Content Encoding")
Why ?
Thanks

hmm... at the moment I'm not sure what to answer, I'll give a try later and see what the problem could be.

I forgot to ask: is that with 1.0 or the 1.1 development version?

Not is in the 0.97 .....

hmm... we never released a 0.97 version. :confused:

sorry I'm wrong I'have a 1.0.7 version....

Sorry for the slow answer.
I tried to edit the language with French installed, running French or English, but I didn't get this error message.
Searching a bit on the internet, it may have something to do with Firefox some corrupted cache or cookie, try cleaning the cookies and the cache of your browser and see if anything changes.

I have try ro clean the cache but the same .... I don't understand why it done ....

I wonder if its due to the language local defined in the files vs whats available on the server?  Although I would think that would show up as a problem on other pages.  By default index.french.php has $txt['lang_locale'] = 'fr_FR.utf8'; could try that as $txt['lang_locale'] = 'fr_FR';  Just a shot in the dark.

@Paul75 : Just to confirm it's a browser related issue can you try with a different browser please?
https://support.mozilla.org/de/questions/945251

Hello,
I try in Firefox and Safari.
In Safari not hve message, it relaod but not value change ....

@Paul75 did you try what suggested by Spuds?

No more feedback and not enough info as far as I can see.
If you or anyone else has more info at some point, feel free to re-open and/or post here or open a new issue, etc.
Thanks for the report. :)


az sf cluster create - KeyVault not placed in location specifed by --location

Environment summary
Azure Cloud Shell

When creating a cluster using az sf cluster create and specifying a --location parameter, which is different from the location of the Resource Group, the KeyVault gets created in the Resource Groups location, which breaks the deployment.

Service Fabric

https://github.com/Azure/azure-cli/pull/29861


chore: expands manual release workflow

closes 2181

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.Christopher Lowenthal seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Add connection view implementation

Notes for Reviewers
This PR fixes (#10582) replaced by #10965
Signed commits

[X] Yes, I signed my commits.

@hexxdump look good?

@hexxdump look good?

yes, looks good, just had some typos, pointed it out.

Thank you, @hexxdump 👏

@Jougan-0 do you mind giving this a once-over?

@MUzairS15 if I am right you were going to do changes in connection_handler replace /api/integrations/connections/{connectionKind} by /api/integrations/connections/{connectionId}, am I  right ?

Yes I will be and your PR is complete already from your side

Uh-oh, a couple of merge conflicts have bubbled up.

@singh1203 or @Jougan-0 will you assist in seeing these merge conflicts eliminated and this PR merged?
We haven't heard from @lekaf974 recently, so please tie this off on his behalf.

Closing this as I am tying this off in #11490


fix: quick quantity change on item lists

PR Type

[x] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no API changes)
[ ] Build-related changes
[ ] CI-related changes
[ ] Documentation content changes
[ ] Application / infrastructure changes
[ ] Other:
Actual Behavior
When I change the quantity of an item in the shopping cart, right after I have already changed the quantity before, the price of the item is not updated
Issue Number: Closes #1380
Expected Behavior
If I have the possibility to change the item quantity, I expect a correct update of the item
Does this PR Introduce a Breaking Change?

[ ] Yes
[x] No
Other Information
AB#84676

A trackBy function is not needed for the retail set part component, because the initial data will not change, when e.g the product quantity increases.


go-junit-report: report junit failures for TestMain issues

A very common issue I see when debugging integration tests is for TestMain (install) to fail. The junit report has no logs, so users look at the prow highlighted errors, which have irrelevant warnings/errors.
it would be great to get Testmain logging. I think we can hack this into the test framework.
For example:

func TestMain(m *testing.M) {
	fmt.Println("=== RUN TestMain")
	fmt.Println("printmain")
	exitCode := m.Run()
	fmt.Println("--- FAIL: TestMain (0.00s)")
	os.Exit(exitCode)
}

Seems to trick go-junit-report into getting our TestMain info into the report
cc @chizhg

There is another inconsistent behavior I recently found for TestMain:
if the test package only contains TestMain, or all tests are skipped, go-junit-report will not include TestMain in the junit result if it passes, but will include it if it fails.
This is a different issue but just wanna track them together since they are both related to TestMain.


Clickable-hyperlinks don't work?

Environment
Windows build number: 10.0.19041.0
Windows Terminal version (if applicable): Latest store version at filing, default settings file says 1.4.3243.0

Any other software? Powershell Core 7.1.0 is the main shell I use. Also using the "powerline in powershell" per the instructions in the docs. But, I'd be surprised if either of those mattered: I can echo the same URL from below in a WSL shell and it is similarly unclickable.

Steps to reproduce
I push something to GitHub, and a URL is displayed. But, control-click on it does nothing. You might be able to dump this from a file to terminal to repro.
ryanp@RYZENSHINE-WIN  E:\src-ssd\hello_xr_hpp   pr26-210112 ≢                                              [11:53]
❯ git push -u origin pr26-210112
Enumerating objects: 89, done.
Counting objects: 100% (89/89), done.
Delta compression using up to 16 threads
Compressing objects: 100% (80/80), done.
Writing objects: 100% (83/83), 151.65 KiB | 11.67 MiB/s, done.
Total 83 (delta 56), reused 12 (delta 2), pack-reused 0
remote: Resolving deltas: 100% (56/56), completed with 6 local objects.
remote:
remote: Create a pull request for 'pr26-210112' on GitHub by visiting:
remote:      https://github.com/rpavlik/hello_xr_hpp/pull/new/pr26-210112
remote:
To https://github.com/rpavlik/hello_xr_hpp.git
 * [new branch]      pr26-210112 -> pr26-210112
Branch 'pr26-210112' set up to track remote branch 'pr26-210112' from 'origin'.

Expected behavior
Expected underlining on hover with ctrl key down, and upon click on link with ctrl key down, for the page to open in my default browser, just as happens in the vscode terminal.
Actual behavior
Nothing happens: acts just like I control-clicked any normal text.
I have tried re-generating config file and just transplanting changes, no luck. Changes are pretty minimal: I disabled ctrl-c and ctrl-v as copy/paste, re-arrange some profiles, add two new ones, and add some defaults for powerline and acrylic. (Turning those off doesn't fix it either)
I tried really hard to figure out how to get it to work: waited a long time after release, manually checked for updates in the store, dug through all the docs, but I can't figure out what I'm doing wrong.

Easier repro: just use the https://aka.ms/powershell that gets printed in each new pscore tab. It does work in terminal preview, which I just installed...

Easier repro: just use the https://aka.ms/powershell that gets printed in each new pscore tab. It does work in terminal preview, which I just installed...

Fortunately, there's nothing in the config that affects how hyperlinks work - they should "just work".
This is a wild guess - are you sure that the link didn't actually open in your existing browser window, but left the focus on the Terminal? We've had some weird z-ordering issues in the past, and I wouldn't be surprised if the system didn't know how to switch focus away from the Terminal onto the window it just opened a tab in.

Fortunately, there's nothing in the config that affects how hyperlinks work - they should "just work".
This is a wild guess - are you sure that the link didn't actually open in your existing browser window, but left the focus on the Terminal? We've had some weird z-ordering issues in the past, and I wouldn't be surprised if the system didn't know how to switch focus away from the Terminal onto the window it just opened a tab in.

Uhg, ignore me.
Turns out auto-detecting hyperlinks were only added in 1.5+. See the original PR. So this is "as expected" - 1.4 doesn't support them yet 😄

Uhg, ignore me.
Turns out auto-detecting hyperlinks were only added in 1.5+. See the original PR. So this is "as expected" - 1.4 doesn't support them yet 😄

oops, thanks 🤦

oops, thanks 🤦


Fixed wiley outputs

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 91.38%. Comparing base (3cba4c6) to head (16b0a97).
Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #146   +/-   ##
=======================================
  Coverage   91.38%   91.38%
=======================================
  Files          25       25
  Lines        2879     2879
=======================================
  Hits         2631     2631
  Misses        248      248

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


In version 3.10.1, encountering an error in  'Start in container' functionality while running in Podman.

When we run 'Start in container' using podman (v 4.9.3) we are getting the following error.
[INFO] Starting container...
[INFO] Container command: podman run --rm -p 9080:9080 -p 9443:9443 -p 7777:7777 -e WLP_DEBUG_SUSPEND=n -e WLP_DEBUG_ADDRESS=7777 -e WLP_DEBUG_REMOTE=y -v /Users/aparnacmichael/Documents/Test-24.0.3/Intellij/app-name-intellij-one/target/.libertyDevc/apps:/config/apps -v /Users/aparnacmichael/Documents/Test-24.0.3/Intellij/app-name-intellij-one/target/.libertyDevc/dropins:/config/dropins -v /Users/aparnacmichael/Documents/Test-24.0.3/Intellij/app-name-intellij-one:/devmode -v /Users/aparnacmichael/Documents/Test-24.0.3/Intellij/app-name-intellij-one/target/liberty/wlp/usr/servers/defaultServer/logs:/logs -v /Users/aparnacmichael/.m2/repository:/devmode-maven-cache --user 501  --name liberty-dev app-name-intellij-one-dev-mode server debug defaultServer -- --io.openliberty.tools.projectRoot=/devmode
[ERROR] Error: preparing container b6f0b079bed6815152e908f06a9a7297086935163efeeda0b4eb72231a6b1a15 for attach: unable to find user 501 : no matching entries in passwd file
[ERROR] An error occurred while running the container: Error: preparing container b6f0b079bed6815152e908f06a9a7297086935163efeeda0b4eb72231a6b1a15 for attach: unable to find user 501 : no matching entries in passwd file RC=126
[INFO] ------------------------------------------------------------------------
[INFO] BUILD FAILURE
[INFO] ------------------------------------------------------------------------
[INFO] Total time:  01:05 min
[INFO] Finished at: 2024-03-13T16:22:16+05:30
[INFO] ------------------------------------------------------------------------
[ERROR] Failed to execute goal io.openliberty.tools:liberty-maven-plugin:3.10.1:devc (default-cli) on project app-name-intellij-one: Error starting the server in dev mode.: An error occurred while running the container: Error: preparing container b6f0b079bed6815152e908f06a9a7297086935163efeeda0b4eb72231a6b1a15 for attach: unable to find user 501 : no matching entries in passwd file RC=126 -> [Help 1]
If we change the 'liberty-maven-plugin' to previous version (3.10) start in container is working without any issue.

The Start in container in Podman isn't functioning as expected on Linux (RHEL), using Podman version 4.6.1. This issue persists across Maven versions 3.10, 3.10.1, and 3.10.2. However, on Mac systems running Podman version 4.9.3, the "Start in container" feature works fine with Maven versions 3.10 and 3.10.2.

@aparnamichael Is the error identical using 3.10.2 on Linux? Can you please include the logging output that shows the podman commands that were run when using version 3.10.2?

@aparnamichael Is the error identical using 3.10.2 on Linux? Can you please include the logging output that shows the podman commands that were run when using version 3.10.2?

I apologize, @cherylking . After uninstalling and reinstalling Podman and attempting to run "start in container" again with both maven versions 3.10 and 3.10.2, everything seems to be working fine. Therefore, I  am closing the issue for now.

@aparnamichael Oh good! I was confused why the fix would not have worked for you, as I had someone test it on Linux specifically.


ondrive crash

Description
Hello, I see many file crashes when I work on ondrive files. File must restart often.
Steps To Reproduce
No response
Expected behavior
Hello, I see many file crashes when I work on ondrive files. File must restart often.
Files Version
Version : <IP_ADDRESS> OS Version: 10.0.19044.1586
Windows Version
windows 10
Relevant Assets/Logs
debug_fulltrust.log
debug.log

Merging with #8599


Releases

Hi @DigitalFlow ! I Hope to not bother you. Is there any plan to merge or release more updates to the project soon?
We would like to get more used to it to start contributing, if that is the case, is there any plan to release new versions and features more often ?
Thanks a lot, appreciate your help.

Hi @imartinflores,
yes sure :)
I was on vacation, sorry for not responding that fast.
I'll merge the PRs and update the package soon :)
Kind regards,
Florian

Hi @DigitalFlow  , thanks a lot!
Hope you had a really nice holidays!
Regards,
Nacho


chore(deps): upgrade dev dependencies

Upgrades project dependencies. See details in workflow run.

Automatically created by projen via the "upgrade-dev-dependencies-k8s-28-main" workflow

⚪ Backport skipped
The pull request was not backported as there were no branches to backport to. If this is a mistake, please apply the desired version labels or run the backport tool manually.
Manual backport
To create the backport manually run:
backport --pr 3909

Questions ?
Please refer to the Backport tool documentation