Entity returns null when used on the model

When i extends the class with Entity it returns null when used in model query as  returnType also when use in insert it insert null values but when i removed the CodeIgniter\Entity class from the Entity class it will display a good object is there any reason why this is happening ? i really appreciate any kind of help thanks.
object(App\Entities\User)#66 (16) {
  ["id"]=>
  NULL
  ["email"]=>
  NULL
  ["username"]=>
  NULL
  ["password":protected]=>
  NULL
  ["created_by"]=>
  NULL
  ["deleted"]=>
  bool(false)
  ["disabled"]=>
  bool(false)
  ["role"]=>
  NULL
  ["created_at"]=>
  NULL
  ["updated_at"]=>
  NULL
  ["datamap":protected]=>
  array(0) {
  }
  ["dates":protected]=>
  array(3) {
    [0]=>
    string(10) "created_at"
    [1]=>
    string(10) "updated_at"
    [2]=>
    string(10) "deleted_at"
  }
  ["casts":protected]=>
  array(0) {
  }
  ["attributes":protected]=>
  array(10) {
    ["id"]=>
    string(1) "1"
    ["email"]=>
    string(14<EMAIL_ADDRESS>    ["username"]=>
    string(4) "test"
    ["password"]=>
    string(60) "$2y$10$kDCaVVL2DpumXrRcV4pKqO2QjAqIl8g6NtVoPdRoFIKVGRFS9n3VK"
    ["created_at"]=>
    string(19) "2020-04-17 16:55:48"
    ["created_by"]=>
    string(1) "1"
    ["deleted"]=>
    string(1) "0"
    ["disabled"]=>
    string(1) "0"
    ["role"]=>
    string(10) "ADMIN,USER"
    ["updated_at"]=>
    string(19) "2020-04-17 16:55:48"
  }
  ["original":protected]=>
  array(10) {
    ["id"]=>
    string(1) "1"
    ["email"]=>
    string(14<EMAIL_ADDRESS>    ["username"]=>
    string(4) "test"
    ["password"]=>
    string(60) "$2y$10$kDCaVVL2DpumXrRcV4pKqO2QjAqIl8g6NtVoPdRoFIKVGRFS9n3VK"
    ["created_at"]=>
    string(19) "2020-04-17 16:55:48"
    ["created_by"]=>
    string(1) "1"
    ["deleted"]=>
    string(1) "0"
    ["disabled"]=>
    string(1) "0"
    ["role"]=>
    string(10) "ADMIN,USER"
    ["updated_at"]=>
    string(19) "2020-04-17 16:55:48"
  }
  ["_cast":"CodeIgniter\Entity":private]=>
  bool(true)
}

Please ask on the forums where others can help out. Providing code samples is always good, too, otherwise no one really knows. I can say that I've never run into this before so pretty sure it's not a bug. Closing unless we decide otherwise.

actually when extend the model in this it will occur the error i  i removed it.


geojson tiles giving errors when trying to see a map in vectors

https://team.cartodb.com/u/saleiva/editor/07ae2e5a-22a3-11e6-a0cb-0e674067d321/embed?vector=true doesn't work but https://team.cartodb.com/u/saleiva/editor/07ae2e5a-22a3-11e6-a0cb-0e674067d321/embed works like a charm!

Associated named map is not well formed, aggregation column is set to null in some cases, for instance:
"...": {
            "options": {
                "aggregationColumn": null,
                "column": "exp_job_loss_fraction",
                "operation": "avg",
                "prefix": "",
                "schema": {},
                "suffix": "",
                "sync_on_bbox_change": true,
                "sync_on_data_change": true
            },
            "source": {
                "id": "a0"
            },
            "type": "formula"
        }

This was fixed as per https://github.com/CartoDB/Windshaft-cartodb/issues/476.


Giving bundle of errors - SystemChrome.setEnabledSystemUIOverlays and ThemeData.accentColor etc.

I am NOT using chewie package / plugin in my Flutter project, but this plugin is giving me the following bundles of errors:
`Launching lib\main.dart on RMX3395 in debug mode...
/C:/Users/john/AppData/Local/Pub/Cache/hosted/pub.dev/chewie-1.2.2/lib/src/material/material_controls.dart:549:46: Error: The getter 'accentColor' isn't defined for the class 'ThemeData'.

'ThemeData' is from 'package:flutter/src/material/theme_data.dart' ('../../flutter/packages/flutter/lib/src/material/theme_data.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'accentColor'.
playedColor: Theme.of(context).accentColor,
^^^^^^^^^^^
/C:/Users/john/AppData/Local/Pub/Cache/hosted/pub.dev/chewie-1.2.2/lib/src/material/material_controls.dart:550:46: Error: The getter 'accentColor' isn't defined for the class 'ThemeData'.
'ThemeData' is from 'package:flutter/src/material/theme_data.dart' ('../../flutter/packages/flutter/lib/src/material/theme_data.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'accentColor'.
handleColor: Theme.of(context).accentColor,
^^^^^^^^^^^
/C:/Users/john/AppData/Local/Pub/Cache/hosted/pub.dev/chewie-1.2.2/lib/src/chewie_player.dart:158:18: Error: Member not found: 'SystemChrome.setEnabledSystemUIOverlays'.
SystemChrome.setEnabledSystemUIOverlays(
^^^^^^^^^^^^^^^^^^^^^^^^^^
/C:/Users/john/AppData/Local/Pub/Cache/hosted/pub.dev/chewie-1.2.2/lib/src/chewie_player.dart:171:20: Error: Member not found: 'SystemChrome.setEnabledSystemUIOverlays'.
SystemChrome.setEnabledSystemUIOverlays(
^^^^^^^^^^^^^^^^^^^^^^^^^^
/C:/Users/john/AppData/Local/Pub/Cache/hosted/pub.dev/chewie-1.2.2/lib/src/chewie_player.dart:175:20: Error: Member not found: 'SystemChrome.setEnabledSystemUIOverlays'.
SystemChrome.setEnabledSystemUIOverlays([]);
^^^^^^^^^^^^^^^^^^^^^^^^^^
/C:/Users/john/AppData/Local/Pub/Cache/hosted/pub.dev/chewie-1.2.2/lib/src/material/material_desktop_controls.dart:554:46: Error: The getter 'accentColor' isn't defined for the class 'ThemeData'.
'ThemeData' is from 'package:flutter/src/material/theme_data.dart' ('../../flutter/packages/flutter/lib/src/material/theme_data.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'accentColor'.
playedColor: Theme.of(context).accentColor,
^^^^^^^^^^^
/C:/Users/john/AppData/Local/Pub/Cache/hosted/pub.dev/chewie-1.2.2/lib/src/material/material_desktop_controls.dart:555:46: Error: The getter 'accentColor' isn't defined for the class 'ThemeData'.
'ThemeData' is from 'package:flutter/src/material/theme_data.dart' ('../../flutter/packages/flutter/lib/src/material/theme_data.dart').
Try correcting the name to the name of an existing getter, or defining a getter or field named 'accentColor'.
handleColor: Theme.of(context).accentColor,
^^^^^^^^^^^
Target kernel_snapshot failed: Exception

FAILURE: Build failed with an exception.

Where:
Script 'D:\john\flutter\packages\flutter_tools\gradle\src\main\groovy\flutter.groovy' line: 1302

What went wrong:
Execution failed for task ':app:compileFlutterBuildDebug'.

Process 'command 'D:\kamlesh\flutter\bin\flutter.bat'' finished with non-zero exit value 1

Try:

Run with --stacktrace option to get the stack trace.
Run with --info or --debug option to get more log output.
Run with --scan to get full insights.

Get more help at https://help.gradle.org

BUILD FAILED in 7s
Exception: Gradle task assembleDebug failed with exit code 1
Exited (1).
`
Kindly suggest me how can solve this issue to compile my Flutter application.
I am 100% sure, I am not using this chewie plugin. Checked pubspec.yaml and pubspec.lock files also.

flutter doctor in command line output:
`$ flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel beta, 3.14.0-0.1.pre, on Microsoft Windows [Version 10.0.19045.3324], locale en-US)
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
[√] Chrome - develop for the web
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.7.3)
[√] Android Studio (version 2022.3)
[√] VS Code (version 1.82.0)
[√] Connected device (4 available)
[√] Network resources
• No issues found!`


chore: fix getting file path for lint stage

Wrap lint stage filename path with double quotes
Solution from: https://github.com/lint-staged/lint-staged/issues/773#issuecomment-1260416318
Fixes #1646

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

@krissvaa For improvements to the repo setup you usually want to tag the PR with chore: .... In order to tag an issue, use Fixes #... (without parenthesis), otherwise the issue will not be closed automatically.


[connectors.Slack] Need to be updated

Needs to be updated to reflect connector changes.

Whispering
Upload

No longer gonig to support other connectors. Want to make this more specific to discord


netlink: Issue #60, fix onewire not present, and hanging when it is.

Fixes Issue #60, periph-cmd/onewire-list not working because the netlink package wasn't initialized.
There is a problem with loading netlink when the onewire bus drivers aren't loaded. If you run an executable with the netlink package loaded, and the onewire driver isn't loaded, the application hangs indefinitely. This was causing pipeline failures. It appears that netlink.Init() is sending a request on the netlink interface for onewire masters, but nothing is responding so it hangs.
My initial thought to resolve the hang was to put a network timeout on the request for onewire bus master. However, I think the approach of looking at the /sys/bus pseudo-filesystem is cleaner.
I'll follow up this PR with one to periph-cmd to do small fixes to onewire-list and add and ds18b20 program.

:warning: Please install the  to ensure uploads and comments are reliably processed by Codecov.
Codecov Report
Attention: Patch coverage is 66.66667% with 3 lines in your changes missing coverage. Please review.

Project coverage is 27.6%. Comparing base (b69b28c) to head (e05a20a).

Files with missing lines
Patch %
Lines

netlink/netlink_other.go
0.0%
2 Missing :warning:

netlink/netlink_linux.go
80.0%
1 Missing :warning:

Additional details and impacted files
@@           Coverage Diff           @@
##            main     #61     +/-   ##
=======================================
+ Coverage   26.7%   27.6%   +0.9%
=======================================
  Files         92      92
  Lines      11538    9376   -2162
=======================================
- Hits        3081    2589    -492
+ Misses      8322    6652   -1670
  Partials     135     135

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

Thanks again!


Low fps during recording on iPhone 12 Pro

Hello,
I use your library for recording ARSCNView and it works great. I record on the iPhone 11 and iPhone SE and it has 60 fps during recording. But when I try to record in new iPhone 12 Pro, fps drops down to 27-35 FPS. Do you know about that problem ? How can I fix it ?

Interesting, what version do you use?
I fixed some fps drop in 2.1.0.
Is the issue reproducible with the example app?

Oh, thanks man. I used 2.0.1, but when I upgraded to 2.1.0 it was okay. Thanks

@juraam I'd recommend to use the latest 2.6 :)

Okay, I will try it)


Only send ssl metrics if a connection succeeded

What does this PR do?
Only send ssl metrics if a connection succeeded
Motivation

AGENT-5349

Additional Notes

Review checklist (to be filled by reviewers)

[ ] Feature or bugfix MUST have appropriate tests (unit, integration, e2e)
[ ] PR title must be written as a CHANGELOG entry (see why)
[ ] Files changes must correspond to the primary purpose of the PR as described in the title (small unrelated changes should have their own PR)
[ ] PR must have changelog/ and integration/ labels attached

I think that in the case of error the service check should not be sent at all

But that's the point of service checks

I think that in the case of error the service check should not be sent at all

But that's the point of service checks

I'm thinking about this as well. We still have the http.can_connect service check for connection related errors. Should the http.ssl_cert be more focused about the cert related info?
For context, the customer wants to alert on ssl expiry using the service check. However, connection to the endpoint can be intermittent causing the monitor to flap not because of the cert.

It could send AgentCheck.UNKNOWN instead of CRITICAL

It could send AgentCheck.UNKNOWN instead of CRITICAL

We have to test this behaviour. Why not to avoid sending any check instead so Service Check monitor can properly generate NO DATA alert?

@niamster I have made some changes to introduce Notify for Unknown status for http.ssl_cert check. It would look something like below

In the mean time, you can modify (via API) or recreate your existing monitor (via JSON) to add the "unknown" threshold as seen below. Note that "unknown" UI slider will not be viewable until this PR is merged but the monitor should still trigger on Unknown status

Great, we're on a freeze for 7.25 this week though. We cannot merge until next week. Sadly that means this fix wont be there until 7.26


[WIP] Add topology grouping for Argo and AppSet apps

Signed-off-by: Feng Xiang<EMAIL_ADDRESS>https://issues.redhat.com/browse/ACM-2469
https://issues.redhat.com/browse/ACM-2948
https://issues.redhat.com/browse/ACM-2900

Migrated over the AppSub topology grouping feature to Argo and AppSet apps. This should help with the performance issue that Amadeus was hitting as the UI was trying to render over 1400 node which is making the browser unresponsive
Also fixed a topology details scrolling issue
With this fix, I think the topology pagination issue for Argo/AppSet apps will be fixed as well

/test lint-check-unittests-sonarcloud


Replace ``` with ` when emphasizing something inline in docs/

Only handles things under docs/, the rest of the codebase still has a bunch of uses.

LGTM


Crash when reporting issue

When trying to report issue #1125 via PocketHub, the app always crashed. I think the relevant Exception is 07-28 15:45:07.392 E/AndroidRuntime(11581): java.lang.IllegalStateException: Missing required properties: labels assignees

Should be fixed in the next version, it's fixed in master but not released.


[discussion] FromVariant error refinement

Currently, from_variant's return value only indicates whether the conversion was successful, without providing any information on the error. This was sufficient for primitive values, but produces rather unhelpful error messages when working with structures. As a result, a non-compatibility-breaking way to introduce rich errors should be devised.
One such possibility is to add a provided function on FromVariant:
enum FromVariantError {
    Unspecified,
    /* - snip - */
}

trait FromVariant {
    fn from_variant(variant: &Variant) -> Option<Self>;
    fn try_from_variant(variant: &Variant) -> Result<Self, FromVariantError> {
        Self::from_variant(variant).ok_or(FromVariantError::Unspecified)
    }
}

Drawbacks
Hopefully none. This is certainly more code to compile and more strings to put in the binary, but the benefits to development should more than offset this cost.
Rationale and alternatives
Error as an enum instead of associated type
The reason the error is represented as an enum is because:

Associated type defaults (RFC 2532) are not stable: adding an associated type would be a breaking change.
Most of the time the errors should come from the same list of variants anyway.

Not a new trait, akin to TryFrom
A new TryFromVariant trait would be impractical. Although we can provide a blanket implementation for all T: FromVariant, we can't do the inverse, and existing user functions using FromVariant as a bound will no longer work with our types (since we'll want to improve their error messages, obviously).
Unresolved questions

[ ] What kinds of errors are needed, beside the blanket Unspecified and Custom(Box<dyn Display>)?
[ ] Whether (or when) the original from_variant function should be deprecated / removed.

This would constitute a relatively large change throughout the codebase, and there might be drawbacks or questions I'm not currently aware of. so I'd love some feedback before going into implementation!

Better error reporting is a good goal!

As a result, a non-compatibility-breaking way to introduce rich errors should be devised.

I am not sure how strict we should see compatibility breaking changes. The FromVariant trait has not been around for so long. I assume most people don't use it directly but via method parameters instead (I could be wrong here). If the output of the derive macro changes accordingly I would expect most code to keep working.
Since I haven't actually used the bindings for a project in a long time my assumptions probably are not the most precise ones. I just want to say that all this stuff is still pretty new and I personally don't think breaking compatibility should be such a big thing.

most people don't use it directly but via method parameters instead

There are also custom implementations. I have accumulated 10 of them (5 are macros) in a ~20kloc codebase, which imo puts it into the "fairly common" category, but maybe my use case is weird.
The number is not too big in an absolute sense anyway, and I don't expect there to be very large codebases in existence at this point in time (judging from our popularity), so it is indeed an option to just break compatibility here. I just preferred the non-breaking way originally because I didn't think the cost was very big here.
Maybe @tom-leys can provide some insight on how much FromVariant is used in his project?

I have written a helper to manage from variant in most cases, I don't have
many api functions exposed to Godot scripts so the pain is still low to
change.
Changes to things like variant are quite annoying as variant is kind of
magic and hard to debug for me so when the code gets broken it's a
nuisance.
Anything to make variant ergonomic is a good investment I think. I would
prefer some temporary pain for long term ease of use.
On Tue, 24 Dec 2019, 04:23 toasteater<EMAIL_ADDRESS>wrote:

most people don't use it directly but via method parameters instead
There are also custom implementations. I have accumulated 10 of them (5
are macros) in a ~20kloc codebase, which imo puts it into the "fairly
common" category, but maybe my use case is weird.
The number is not too big in an absolute sense anyway, and I don't expect
there to be very large codebases in existence at this point in time
(judging from our popularity), so it is indeed an option to just break
compatibility here. I just preferred the non-breaking way originally
because I didn't think the cost was very big here.
Maybe @tom-leys https://github.com/tom-leys can provide some insight on
how much FromVariant is used in his project?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GodotNativeTools/godot-rust/issues/265?email_source=notifications&email_token=ABIKNVIH6JZSVQ62YTEX3LTQ2DJXZA5CNFSM4J6LYME2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHRKXJI#issuecomment-568503205,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABIKNVLSRNHI5Q4MCZWZS2LQ2DJXZANCNFSM4J6LYMEQ
.

Oh,  also I have avoided defining to and from variant for my own types as
it has been unnecessary so far.
On Tue, 24 Dec 2019, 04:23 toasteater<EMAIL_ADDRESS>wrote:

most people don't use it directly but via method parameters instead
There are also custom implementations. I have accumulated 10 of them (5
are macros) in a ~20kloc codebase, which imo puts it into the "fairly
common" category, but maybe my use case is weird.
The number is not too big in an absolute sense anyway, and I don't expect
there to be very large codebases in existence at this point in time
(judging from our popularity), so it is indeed an option to just break
compatibility here. I just preferred the non-breaking way originally
because I didn't think the cost was very big here.
Maybe @tom-leys https://github.com/tom-leys can provide some insight on
how much FromVariant is used in his project?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/GodotNativeTools/godot-rust/issues/265?email_source=notifications&email_token=ABIKNVIH6JZSVQ62YTEX3LTQ2DJXZA5CNFSM4J6LYME2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEHRKXJI#issuecomment-568503205,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABIKNVLSRNHI5Q4MCZWZS2LQ2DJXZANCNFSM4J6LYMEQ
.

From what's seen here, it would probably be fine to just change the signature of from_variant. I will then be working in that direction instead. Thanks!


Fix open_command inside tmux

I want to use open_command inside tmux.
$ tmux
$ nohup open .
appending output to nohup.out
$ cat nohup.out
nohup: can't detach from console: No such file or directory

So, I modified open_command what not use nohup command inside tmux.
see also: http://stackoverflow.com/questions/23898623/nohup-cant-detach-from-console#comment36797854_23898623

:+1:

I can't reproduce this. The output of nohup is silenced in the current version of the open_command function. Let me see some diagnostic info, please post the output of:
which -a open
which -a open_command
nohup --version

$ which -a open
/usr/bin/open

$ which -a open_command
open_command () {
	local open_cmd
	local nohup
	case "$OSTYPE" in
		(darwin*) open_cmd="open"  ;;
		(cygwin*) open_cmd="cygstart"  ;;
		(linux*) open_cmd="xdg-open"  ;;
		(*) echo "Platform $OSTYPE not supported"
			return 1 ;;
	esac

	nohup $open_cmd "$@" &> /dev/null
}

$ nohup --version
nohup: illegal option -- -
usage: nohup [--] utility [arguments]

I did not know how to determine the version of nohup command
Sorry, I was wrong to write an example.
I fixed example
$ tmux
- $ open .
+ $ nohup open .
appending output to nohup.out
$ cat nohup.out
nohup: can't detach from console: No such file or directory

Oh, then you're not really using open_command. You should do open_command . which is the platform-agnostic function, or just open . since you're on OSX. The nohup in your case is not necessary.

This problem is that open_command is not working inside tmux .
open_command the web-search, jira, such as in and have been utilized and node plugin, these plugin uses open_command.
open_command and to open a URL in a web browser is being used.
However , it does not open a web browser and are inside tmux.
I am in the environment remains this open_command does not work , and would like to to move This is the purpose of this Pull Request.

I can reproduce, with OS X 10.9.5, zsh 5.0.2, tmux 2.1.
The problem isn't that @eiel is adding an unnecessary nohup: he's just demonstrating the breakage. The problem is that inside tmux, nohup fails, and the open doesn't happen. So the unconditional nohup that open_command is doing means it doesn't work properly when called inside a tmux session.
For example, here's a fresh tmux session I fired up locally inside iTerm2. The open works, opening the text file in BBEdit, my default text editor. The open_command call has no effect.
$ open omz_diagdump_20151008-223116.txt
[~]
$ open_command omz_diagdump_20151008-223116.txt
[✘ ~]
$

I don't know exactly what the semantics of open_command should be, so I'm not sure what the exact change should be, but it does seem that something like this PR is needed.

Ok, thanks for chiming in. I see now what he meant.
Weirdly, when I tested this on my setup it worked correctly. I'll see if we need this or there is some other alternative that works everywhere though, I think I added the nohup part to be sure, but it may be replaced by starting the process with &| / &!:

If a job is started with ‘&|’ or ‘&!’, then that job is immediately disowned. After startup, it does not have a place in the job table, and is not subject to the job control features described here.

Try this one-liner instead:
$open_cmd "$@" &>/dev/null &|

I've submitted that change in the form of a commit at https://github.com/mcornella/oh-my-zsh/commit/454e89cd31b38e56a7c03621c4cf6f240b15fa5d.

$open_cmd "$@" &>/dev/null &|
Awesome! :bow: :+1:
It was confirmed that it is working well. :smile:

Ok, I've submitted https://github.com/robbyrussell/oh-my-zsh/pull/4534 to test separately. +1 there if it works for you


0b11111111 returns -1 instead of 255

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
[convert]::ToString(255,2)
0b11111111
[convert]::ToInt32(11111111,2)

11111111
-1
255

Expected behavior
11111111
255
255

Actual behavior
0b11111111 returns -1

Error details
No Errors

Environment data
Name                           Value
----                           -----
PSVersion                      7.4.0-preview.1
PSEdition                      Core
GitCommitId                    7.4.0-preview.1
OS                             Linux 6.1.12-arch1-1 #1 SMP PREEMPT_DYNAMIC Tue, 14 Feb 2023 22:08:08 +0000
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

and 7.3.2

Visuals
No response

If it was a signed 8 bit byte then it would make sense as it would be twos complement sign extension.
PS> (0b11111111).GetType().Name
Int32
PS> 0b11111111
-1

And 65535

And 65535

I raise you<PHONE_NUMBER> for an unsigned 32 bit all ones

Per about_Numeric_Literals:

If a binary string is some multiple of 8 bits long, the highest bit is treated as the sign bit.
[...]
Specifying an unsigned suffix on binary and hex literals ignores sign bits.

Therefore, use a u suffix to ensure that you get an unsigned (positive) value:
PS> 0b11111111u
255  # [uint32]

PS> 0b11111111uy
255  # [byte]

PS> 0b11111111us
255  # [ushort] == [System.UInt16]

PS> 0b11111111ul
255  # [ulong] == [System.UInt64]

Therefore, use a u suffix to ensure that you get an unsigned (positive) value.

For 255 only? But for 254 and 256 this is not necessary.

24 is a multiple of 8 but this rule is not applied
PS > 0b111111111111111111111111
16777215
PS > (0b111111111111111111111111).GetType().Name
Int32

4 is a multiple of 8 but this rule is not applied

the "rule" of 255, this bug is for 255 and 65535:

for ($i = 1; $i -le ([int]::MaxValue * 2); $i *= 2)
{
    $m = ($n = $i - 1) % 255
    '{0,11}: {1,-11} {2}' -f $n, [int] $( '0b' + [convert]::ToString($n,2)), $m
}

I should have clarified that I was merely citing the documentation and showing a workaround,
but let me weigh in on what I think of this behavior:
It is certainly surprising, and the true rules are obscure (and currently incorrectly documented, as @rhubarb-geek-nz  points out) and inconsistent with the rules for hex literals.
Let me try to document the current de facto behavior (inferred from experiments - haven't looked at the source code):

Binary literals without a type suffix or with signed integer-type suffixes (y, s, l, d, n) that have 8, 16, 32, or 64 binary digits (bits) and whose first digit is 1 (the high bit) become negative numbers: they are treated as bit patterns of a signed integer type of that width (see two's complement).

Note that this means that the high bit alone decides whether the resulting number is negative; e.g., 0b10000000 is -128

You would think that the resulting number types map onto signed types [sbyte], [int16] ([short]), [int32] ([int]) and [int64] ([long]), but binary numbers with 8, 16 digits  are converted to [int32] too.

Hex literals:

[int32] and [int64] are also the automatically chosen types used for non-suffixed hex literals up to 64 bits.
However, the negativity rules differ: only 32-bit values and 64-bit values with the high bit set  are negative by default (e.g., 0xffffffff and 0xffffffffffffffff) and smaller numbers can only be negative with an explicit type suffix for a smaller signed type (e.g,. 0xffy ([sbyte]) and  0xffffs ([short]==[int16])

Type suffixes for unsigned integer types prevent interpretation as a negative number, both for binary and hex literals: uy ([byte]), us ([ushort]==[uint16]), u ([uint32], widened to [uint64] if needed), andul ([ulong]==[uint64])

Beyond 64 bits, number literals fundamentally only work with suffixes d ([decimal]) and n ([bigint]) , both of which are signed types, so the non-suffixed negativity rules apply them as well, plus to every additional 32 bits of width, up to 96 for [decimal], and open-ended for [bigint], but for binary literals only.
The only way by prevent negative numbers for these type is  by prepending 0; e.g., 0b0111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111n ([decimal],<PHONE_NUMBER>4264337593543950335, which is [decimal]::MaxValue)

Hex literals:

With type suffix d ([decimal]):

Note: [decimal], strictly speaking, isn't an integer type, but it can be used like one and when parsing binary literals, it seems to behave like a 96-bit signed integer.
Unexpected behavior up to 64 bits, e.g., 0xfffffffD (7 hex digits) is -3(!) and 0xffffffffD (which without D would be -1) is<PHONE_NUMBER>3 (!).
Broken beyond 64 bits (e.g., 0xffffffffffffffffffffffffD (96 bits) fails to parse, even though it is the equivalent of binary 0b0111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111D, which works.

With type suffix n ([bigint]):

Hex. literals In the beyond-64-bit range are always positive, unlike binary literals; e.g., 0xffffffffffffffffffffffffN is<PHONE_NUMBER>4264337593543950335, whereas the equivalent binary literal, 0b111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111N, is -1 (and would require a leading 0 to be positive).

Finally, there's an outright bug:

Non-suffixed binary literals with 96, 128, ... digits should break (the widest auto-selected type is [int64], which cannot hold such numbers), but the parsing quietly overflows and returns -1 of type [int32]; e.g: 0b111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111111

Professional explanation. But this behavior, I believe, should be fixed. Maybe just by documenting and advising to add a zero to the 0b prefix ( 0b0 ) ?

Thanks, @237dmitry; I think there are several actions needed:

The outright bugs described should be fixed.

The current behavior should be revisited, especially in light of the discrepancies between binary and hex literals:

Backward compatibility is a concern, though perhaps binary literals have seen so little use so far and current uses probably employ the workarounds needed to get a positive number, so changes could fall into bucket 3, in which case they'd be acceptable.

Note that while the hex-literal behavior may be less surprising, it still is (I think most people's expectation is to never get a negative number), but that ship has probably sailed (a change was discussed pre-v6 - see https://github.com/PowerShell/PowerShell/issues/3313#issuecomment-297103481 - but it lead to the U suffix instead of changing the default behavior).

The documentation should be amended either way, both to correct the existing inaccuracies and to provide clear guidance.

The 0b0 technique is already mentioned, but it should certainly be made clearer when it is needed.
Using type suffix u is the better option as long as you're in the realm of 64-bit values (which, if I were to guess, applies to most use cases), because it also allows you to get an unsigned 64-bit value up to [uint64]::MaxValue; that is, 0xffffffffffffffffU and 0b1111111111111111111111111111111111111111111111111111111111111111u work, while 0x0ffffffffffffffff and 0b01111111111111111111111111111111111111111111111111111111111111111 do not.

Therefore, use a u suffix to ensure that you get an unsigned (positive) value.

Exclusively for 255? But for 254 and 256 this is not necessary.

if 254 is written as a 0b11111110 it will come back as -2  , 256 is OK because you have 9th bit.
Any number >= 128 written with exactly 8 bits becomes the 2's compliment negative (so does any number >= 32768 written with exactly 16 bits, or >=<PHONE_NUMBER> within with exactly 32 bits etc. )

But this behavior, I believe, should be fixed. Maybe just by documenting and recommending to add a zero to the 0b prefix ( 0b0 ) ?

This fixes one problem by ensuring that (e.g.) 128 is written with 9 bits and is seen as a 16 bit number implied leading zeros. It won't work for a 64 bit.  (0b01000000000000000000000000000000000000000000000000000000000000000) gives an error. (remove the first 0 and it works)
I get a different result from convert
PS>  [convert]::ToString(255,2)
11111111

(no leading 0b)  It seems to me that convert is wrong here.  It has received an Int32 and  trimmed all the leading zeros from it.  If the high bit is 1 in really should pad with zeros.

@jhoneill, to set the stage for potential further discussion and to make it clearer where PowerShell "does its own thing":
.NET and C# play by different, arguably better (simpler) rules:

C#:

All hex and binary literals are invariably positive numbers (except if you prefix them with -).

Without a type suffix, the type is automatically chosen, using the following on-demand widening: Int32 (int) -> UInt32 (uint) -> Int64 (long) -> UInt64 (ulong)

No integer literals beyond 64 bits are supported except if you use a decimal representation and use the m suffix to create a Decimal instance (e.g.<PHONE_NUMBER>3709551616m).

C# has no System.Numerics.BigInteger ([bigint]) literals, but you can construct them from other types, including Decimal, in which case the sign of the argument is honored. Thus, you need the leading-zero-byte technique to force interpretation as a positive number only if the value to pass to a constructor is beyond the range of Decimal, in which case initialization via a byte array is needed. E.g. (using a small number for brevity; note that due to little-Endian representation the leading 0 byte comes last), [bigint] [byte[]] 0xff yields -1, whereas [bigint] [byte[]] (0xff, 0x0) yields 255.

.NET:

[Convert]::ToString($num, 2), [Convert]::ToString($num, 8), and [Convert]::ToString($num, 16) (the only 3 number bases other than 10 that are supported):

These calls create representations strictly based on the bit pattern of the input number, with leading 0s omitted. That is, the input type is lost in the process, including whether or not the input number was negative.
Note that the base parameter is supported only for byte, short, int, and long, and unsigned types are quietly coerced to the next higher signed type as needed, with long.MaxValue being the highest number supported (that is, you cannot pass a ulong instance with a value higher than long.MaxValue).

It is only with [bigint]::Parse() that the leading-zero-byte technique comes into play, and note that all Parse methods of numeric types are limited to parsing hex representations (in addition to the default decimal ones): the System.Globalization.NumberStyles enum has AllowHexSpecifier but no AllowBinarySpecifier, i.e. parsing binary string representations is not supported.

Analogous to the [byte[]] constructor example above, [bigint]::Parse('ff', 'AllowHexSpecifier') yields -1, whereas [bigint]::Parse('0ff', 'AllowHexSpecifier') (leading 0) yields 255.

@mklement0  thanks for filling all that in.
I thought this was the crucial part.

These calls create representations strictly based on the bit pattern of the input number, with leading 0s omitted. That is, the input type is lost in the process, including whether or not the input number was negative.

In other words we don't know if 1111 1111 was a signed byte -128, an unsigned byte (+255), or a positive number written with 16 or 32 bits with the leading zeros dropped.  Similarly we don't know if  ffff was -1 in an int16 or 65535 in an int32.  Even if we had leading zeros we wouldn't know if the original was the signed or unsigned version of a type.
However- 0xffff returns 65535 - the lack of leading zeros doesn't matter for hex but does for binary. 8 digit hex numbers still follow 2s compliment so  I can see that this could be be misunderstood
PS>  -0xff
-255

PS>  -0xffff
-65535

PS>  -0xffffffff
1

TBH (1) I could argue for the behaviour of the binary conversion if 1-8 digits returned an sbyte and 9-16 digits returned an int16 but they don't: they return int32s ; the hex conversion also returns int32 but it assumes 2 and 4 digits are the non-zero part of a 32 bit number, unless they are suffixed to say otherwise (2) Even if shorter numbers became smaller types, binary and hex would still be inconsistent, and I wouldn't argue for having both ways of working. So net, 0b handling just seems wrong.   (3) 0b notation is not in Windows PowerShell, so although changing it would be a breaking change, there aren't scripts out there which have depended on current behaviour for ages. (That's another argument for leaving 0x alone and changing 0b to match it)

I agree, and this has mostly already been summarized above, preceded by a detailed explanation of the inconsistencies between 0b and 0x parsing.
As for:

So net [...] handling just seems wrong

I don't think [Convert]::ToString() is to blame here: its purpose is to create a string representation, not something that is guaranteed to round-trip as a number literal in all .NET-based languages.(which inherently could have different syntaxes for number literals, leaving the obvious difference of prefixes, such as 0x vs. &H, aside).
Creating a language-neutral round-trip representation at the .NET API level would require a new, opt-in  [Convert]::ToString() overload, combined with the introduction of a complementary [Convert]::FromString() method.

@mklement0   as this has gone on I've changed my view a bit. Initially I thought that this was exclusively a problem with [convert] the more I've read (thanks) and tested it, the more I think it is only part of the issue, and the 0b parsing is the bigger problem.
You're right that anyone who doesn't write dates as least-significant in the middle probably knows convert to string is not guaranteed to convert back.
PS>  $x=get-date   # today is the 8th day of march
PS>  $y=[convert]::ToString($x)
PS>  $y
08/03/2023 15:13:03  # to string in least significant first date format
PS>  [datetime]$y              #  from string in least significant in the middle format.
03 August 2023 15:13:03

IIRC [convert] uses local culture, and [datetime]"string" is culture invariant, and whilst the logic for doing this is sound, it is not always obvious.
One can write [convert]::ToString(200,2).PadLeft(32,"0") to ensure the binary literal will convert back. In an ideal world this would be a parameter in ToString.
Just as a we sloppily write "3:13:03" without specifying the timezone (or whether it is AM or PM), giving a meaningless time  0b11111111 without a suffix which says "signed/unsigned" and whether it is more than 8 bits is really not good practice.  I think the real answer is to tell people to use the suffix and be explicit - but I also think the default behaviour can be improved.

and the 0b parsing is the bigger problem.

I fully agree.
As you've previously observed, the only conceivable justification for the current behavior would be if the input bit width implied the specific (signed) data type to return, so that 0b11111111 would return an [sbyte], for instance.
As it stands, the input bit pattern is effectively modified through the type-widening to [int] (it effectively becomes 0b11111111111111111111111111111111).
And, of course, there's the inconsistency with how hex literals are parsed.
As discussed, we're probably stuck with the itself counterintuitive behavior of hex literals, given its long history, but hopefully a bucket-3 argument can be made to align the behavior of binary literals with that of hex literals, for at least consistent, less complex counterintuitiveness, if you will.
In the context of this alignment, ideally we'd get rid of the leading-0 logic altogether:

For values > 64 bits - types [decimal], [bigint] -  the leading 0 never made sense to begin with, given that neither type is a simple bit field; Instead:

Treat all values in this range as positive numbers, the way hex literals already do.
If the intent is to get a negative number, prefix the literal with -.

For values <= 64 bits, using type suffix u is the better choice (see below).

Separately (irrespective of whether 0b parsing will change), we should offer the following guidance in the docs:

If you need your number literal (whether decimal, hex, or binary) to be a specific type, use the appropriate type-specifier suffix (y/ uy, s/ us, (by default) / u, l / ul), or - beyond 64 bits - d (n/a to hex literals)  or n

If you simply want to ensure that your number literal is treated as a positive value, irrespective of the specific resultant type, use suffix u, for values up to UInt64 ([ulong])

As for [convert]::ToString():
There are two separate aspects that hamper round-tripping:

Culture-sensitivity - this can be bypassed by passing [cultureinfo] '' (representing the invariant culture)  as the System.IFormatProvider argument; e.g.  [convert]::ToString((Get-Date), [cultureinfo]  '')
Loss of accuracy / type identity - this would require a new notation, given that even preserving the exact bit pattern doesn't preserve the exact type identity (signed vs. unsigned).

WG-Engine discussed this today. We agree the current behaviour is unintuitive and it may be worth the breaking change in this instance.
We suggest the parsing should be modified to line up functionally with the hexadecimal parsing: all literals up to 32-bits wide should be considered effectively "unsigned" -- so that things like 0b11111111 should not resolve to a -1, and the only accepted sign bits should be 32, 64, and above that are currently accepted by the parsing.
Implementation wise this should be a relatively simple change, just removing these two switch cases from the binary parsing utility method:
https://github.com/PowerShell/PowerShell/blob/04b93af4ae333013e4fe00013a196268d314cb00/src/System.Management.Automation/engine/Utils.cs#L188-L189


String values in native ENUM column

Some RDBMS (e.g. MySQL) provides native ENUM column type. Propel however keeps using numeric values internally for the enumerated column even if you force the column type with sqlType="ENUM('one', 'two', 'three')" and phpType="string" in schema.xml.
Back in the day the solution was to modify the value set in Peer class (as discussed here: https://groups.google.com/forum/#!msg/propel-development/3DhvA-9bUDg/Q0Z3zY39DUkJ). With the advent of Peer classes, there is no such simple solution.
This PR modifies the behaviour of ENUM columns which are also set as native ENUM through sqlType attribute in schema by allowing the values actually inserted from Propel to database to be string.
The solution is bit hackish for my taste, but I am not very familiar with Propel internals, so I would use some guidance to see if there is a better place for doing this ENUM "magic".
Example column definition:
<column name="status" type="ENUM" valueSet="NEW,OPEN,COMPLETED,CANCELLED" sqlType="enum('NEW','OPEN','COMPLETED','CANCELLED')" phpType="string" />

It would be also nice to have sqlType ENUM values automatically filled from valueSet to avoid repetition, but since this is RDBMS specific, I don't know how it fits in Propel's philosophy (Behavior?).

@oujesky any news?

Sorry for the delay guys. I rebased and squashed the commits.
However I gave it a second look and I still think this is not the best solution. I did a little research on the supported DB platforms and how enum-like types works there. I think the generator can decide which actual DB column type should be used based on phpType setting in schema. The default phpType is integer and with that stays the current default enum behaviour. But when it would be set to string, the string based enum type can be enforced.
On MySQL and PostgreSQL, there is the native ENUM column type:
... column ENUM('small', 'medium', 'large') ...

On MSSQL, SQLite and Oracle it can be simulated to certain degree with CHECK constraint:
... column VARCHAR(10) CHECK(column IN ('small', 'medium', 'large')) DEFAULT 'small' ...

I guess it would be better to create a separate "request for comments" issue though, as it needs to be discussed a bit. The main purpose is not to change the default behaviour of how Propel approaches enums, but to allow legacy DB structure work well with Propel. What do you think?

After a second look I'd like to note following:

As I want this behavior as default I guess it's better to make it vice-versa, meaning phpType="string" is not necessary to have real enums in the database
phpType is not the right place to say the enum type if they should store it as integer or string, as phpType means the type of actual php code, not database schema.
We should make sure that strings are now at the object property, not that legacy hack with integers. Integers should only be read at runtime through array_search.

So this means making Propelv2 enum-aware is much more work, so I'd like either to close this PR or someone is taking the time to invest more energy in it to make it the right-way™ :smile:

I work on Native Enum usage and I work about more simple solution based on @oujesky solution
but with only :

default value (Table creation) based on schema if column sqlType is defined
getValue for Enum return 'content' (and not index) if domain sqlType != 'TinyInt' (workaround for create/update keep aware from internal structure and query raw value)

this work if I define a schema like :
<column name="state" type="ENUM" valueSet="draft,running,expired,cancel" sqlType="enum('draft','running','expired','cancel')" required="true" defaultValue="draft"/>
I think this is not perfect because I check original sqlType with "Type Checking" (TINYINT) but Oracle use (NUMBER) and PGSQL (INT2) but this is compatible with indexed actual approach (without sqlType)
If you consider my approach is good, I can implement platform checking and workaround if domain sqlType differ to waited ENUM default type and asked a PR
ref:
https://github.com/jeremydenoun/Propel2/tree/propel-enum-type

@jeremydenoun, looks pretty good! We are looking forward to your pull request :)

I saw @oujesky added also some tests, could you then also please add some tests? We should make sure this works as intended.

I'm closing this since we implemented enums in our data-mapper branch, which will be merged into master soon.


feat: add test_providers script to test ability of various providers …

What was wrong?
! There is a fair amount of re-org included in this pr !
If any reviewer thinks it's too much, then just lmk and I'll happily split it up into multiple prs.
It's hard to test whether our bridge is successfully creating valid data based on the responses from various providers. This script makes it easy to tell which providers are compatible, and which providers are not (and the reasons why they are not compatible). This could be converted into a unit test, but due to the reliance on live endpoints (pandaops/infura), which we want to avoid in our unit tests, it seemed suitable to just create a script that can be run whenever needed to test various providers.
How was it fixed?

added test_providers script
extracted validation logic from portal-bridge/bridge/history.rs into portal-bridge/api/execution.rs. This makes it easier to test the validation logic in the test_provider.rs script
Improved error handling in receipts deserialization
Added range functionality to sample_range script

To-Do

[ ] Clean up commit history and use conventional commits.

A suggestion on how to maybe make this better.
Have your script be used to generate static test vectors that represent the responses from a given provider...  save those... have unit tests that use the saved responses to validate we can indeed handle the responses from all of the providers that run in CI.
This way you guys don't have to manually spin up each type of provider each time something changes to be able to test those changes.

When Portal-Hive code is moved to hive. I am planning to write Bridge tests where we spin up a EL node fill it with the block data we are testing, set the EL as the data source for the bridge and gossip the content to something like trin. Then we can easily verify if all the EL clients are compatible.

@pipermerriam
In hindsight, this script accomplishes two things at once

Provider liveness
Provider compatibility

Testing liveness is a useful thing imo (ex. I used it multiple times this week to check if our erigon provider was live, which it was not) and not something that can be extracted into a unit test. However, this script is somewhat overkill for simply checking provider liveness.
As you suggested, provider compatibility is something that is better suited for a static, unit test: tracking it in #1117 .
I'm going to go ahead with this merge, because this pr also includes some useful refactoring that makes generating era1 files easier. Given that...

Provider compatibility will be addressed in an upcoming pr
Removing the test_providers.rs script is very simple and can be done anytime


Ecobee Issues in 0.100.1

Home Assistant release with the issue:

0.100.1
Last working Home Assistant release (if known):
0.99
Operating environment (Hass.io/Docker/Windows/etc.):

Docker
Integration:

Description of problem:
With Ecobee in version 0.100.1 I am seeing many errors which seem to be a string being forced to a float.
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
N/A - now an integration

Traceback (if applicable):
Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 408, in _async_add_entity
    await entity.async_update_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 273, in async_update_ha_state
    self._async_write_ha_state()
  File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 307, in _async_write_ha_state
    state = self.state
  File "/usr/src/homeassistant/homeassistant/components/ecobee/sensor.py", line 81, in state
    return float(self._state) / 10
ValueError: could not convert string to float: 'unknown'

Additional information:
Generally Ecobee seems to work but I have over 3300 events of the above in my logs, all the same.  The message over being line 81 seems odd as the source doesn't match that, but I do see this later in the file around ~120.

I will take a look! Not sure where the ‘unknown’ is coming from.

Very strange to me that your temperature sensor is returning the value 'unknown' - the API is supposed to return -5002 when a value is unknown, not the string 'unknown'. I can fix this but it is still mystifying.

As an FYI it seems to be caused by a remote sensor with a dead battery.  Must have died very recently, but did not see this error under 0.99.x.


Use free-style for styles

This project looks great. I think that styling would be nice with free-style. For responsive web design, we need media queries and it is pretty simple with this library, among other advantages.

@carloslfu I find that I use media queries less and less these days. Take a look at the responsive events documentation. http://garth.github.io/snabbdom-material/dist/#/responsiveEvents
It's better to render what the device needs rather than always rendering all html and using media queries to hide or restyle elements based on screen size and orientation. Depending on your app, this can make rending on phones faster than using media queries.
I have considered free-style and would like to revamp the styling mechanism of this lib and introduce better customisation. The main issue is packaging though. As a lib all css and js needs to be easy to import into your app, but since webpack doesn't parse node_modules content it's not so straightforward. For this reason in leaning towards having all default base styles embedded. But if you have any suggestions with examples of how this could be done better I would be happy to hear them?

Oh, you are right. The responsive events is a better approach, with they, you have the screen size as part of the app state. The customization part looks good to me, components are modular and use the style attribute to each one is a good way to customize them.


Failed to create "GitHub->cookiecutter-flask/cookiecutter-flask" template in Cookiecutter.

Describe the bug

Steps to Reproduce

Launch VS
Click "View->Cookiecutter Explorer"
Select "GitHub->cookiecutter-flask/cookiecutter-flask" to click "Next"
Wait for load successfully and then click "Create and Open Folder"

Expected behavior
It should create successfully.
Additional context and screenshots
Error occurs when create template.

Error detail info
----- Creating files using template 'cookiecutter-flask/cookiecutter-flask' -----
DEBUG main.py: context_file is C:\Users\vting\.cookiecutters\cookiecutter-flask\cookiecutter.json
DEBUG generate.py: Context generated is {'cookiecutter': OrderedDict([('full_name', ''), ('email', ''), ('github_username', ''), ('project_name', ''), ('app_name', ''), ('project_short_description', ''), ('use_pipenv', ''), ('python_version', ''), ('node_version', ''), ('use_heroku', '')])}
DEBUG utils.py: Making sure path exists: C:\Users\vting/.cookiecutter_replay/
DEBUG find.py: Searching C:\Users\vting\.cookiecutters\cookiecutter-flask for the project template.
DEBUG find.py: The project template appears to be C:\Users\vting\.cookiecutters\cookiecutter-flask\{{cookiecutter.app_name}}
DEBUG generate.py: Generating project from C:\Users\vting\.cookiecutters\cookiecutter-flask\{{cookiecutter.app_name}}...
DEBUG generate.py: Rendered dir  must exist in output_dir C:\Users\vting\AppData\Local\Temp\85d45923-8a2b-44be-97f7-3883dda14121
Traceback (most recent call last):
  File "c:\program files\microsoft visual studio\2022\preview\common7\ide\extensions\microsoft\cookiecutter\cookiecutter_run.py", line 46, in <module>
    sys.exit(int(main() or 0))
  File "c:\program files\microsoft visual studio\2022\preview\common7\ide\extensions\microsoft\cookiecutter\cookiecutter_run.py", line 37, in main
    cookiecutter(
  File "C:\Users\vting\AppData\Local\Microsoft\CookiecutterTools\env\lib\site-packages\cookiecutter\main.py", line 140, in cookiecutter
    return generate_files(
  File "C:\Users\vting\AppData\Local\Microsoft\CookiecutterTools\env\lib\site-packages\cookiecutter\generate.py", line 264, in generate_files
    project_dir = render_and_create_dir(
  File "C:\Users\vting\AppData\Local\Microsoft\CookiecutterTools\env\lib\site-packages\cookiecutter\generate.py", line 211, in render_and_create_dir
    raise OutputDirExistsException(msg)
cookiecutter.exceptions.OutputDirExistsException: Error: "C:\Users\vting\AppData\Local\Temp\85d45923-8a2b-44be-97f7-3883dda14121" directory already exists
C:\Users\vting\AppData\Local\Microsoft\CookiecutterTools\env\scripts\python.exe returned an exit code of 1.
----- Failed to create files using template 'cookiecutter-flask/cookiecutter-flask' -----

I think this would be a bug in the template code and should probably be logged there?

Logged a while ago: #6390

Yeah I get this is I do this more than once. The user has to specify a new directory here before creating:

Or delete the old folder.

@rchiodo But it was the first time I click "Next" >" Create", it's that suppose to happen?

Are you sure this directory doesn't exist from a previous test?

@rchiodo You can see this can repro in today's build. I just launch the VS and do this.

Yes that will happen if the directory already exists on your machine. Did you install on a completely fresh machine?
Before you run anything, verify that the <user>\source\repos\cookiecutter-flash directory doesn't exist.

@rchiodo Yeah, I install on a completely fresh machine and select that template, then click "Next" >" Create" (You  can see that "Installed" section has nothing at first)

Need to try and repro myself.

Repos on completely fresh machine. Logged this issue here:
https://github.com/cookiecutter-flask/cookiecutter-flask/issues/1183


Newline on every command

https://imgur.com/t5JAmMe

Give us your .zshrc file so we can help you to fix it

If you come from bash you might have to change your $PATH.
export PATH=$HOME/bin:/usr/local/bin:$PATH
Path to your oh-my-zsh installation.
export ZSH="/home/lorcalhost/.oh-my-zsh"
Set name of the theme to load --- if set to "random", it will
load a random theme each time oh-my-zsh is loaded, in which case,
to know which specific one was loaded, run: echo $RANDOM_THEME
See https://github.com/robbyrussell/oh-my-zsh/wiki/Themes
ZSH_THEME="spaceship"
Set list of themes to pick from when loading at random
Setting this variable when ZSH_THEME=random will cause zsh to load
a theme from this variable instead of looking in ~/.oh-my-zsh/themes/
If set to an empty array, this variable will have no effect.
ZSH_THEME_RANDOM_CANDIDATES=( "robbyrussell" "agnoster" )
Uncomment the following line to use case-sensitive completion.
CASE_SENSITIVE="true"
Uncomment the following line to use hyphen-insensitive completion.
Case-sensitive completion must be off. _ and - will be interchangeable.
HYPHEN_INSENSITIVE="true"
Uncomment the following line to disable bi-weekly auto-update checks.
DISABLE_AUTO_UPDATE="true"
Uncomment the following line to change how often to auto-update (in days).
export UPDATE_ZSH_DAYS=13
Uncomment the following line to disable colors in ls.
DISABLE_LS_COLORS="true"
Uncomment the following line to disable auto-setting terminal title.
DISABLE_AUTO_TITLE="true"
Uncomment the following line to enable command auto-correction.
ENABLE_CORRECTION="true"
Uncomment the following line to display red dots whilst waiting for completion.
COMPLETION_WAITING_DOTS="true"
Uncomment the following line if you want to disable marking untracked files
under VCS as dirty. This makes repository status check for large repositories
much, much faster.
DISABLE_UNTRACKED_FILES_DIRTY="true"
Uncomment the following line if you want to change the command execution time
stamp shown in the history command output.
You can set one of the optional three formats:
"mm/dd/yyyy"|"dd.mm.yyyy"|"yyyy-mm-dd"
or set a custom format using the strftime function format specifications,
see 'man strftime' for details.
HIST_STAMPS="mm/dd/yyyy"
Would you like to use another custom folder than $ZSH/custom?
ZSH_CUSTOM=/path/to/new-custom-folder
Which plugins would you like to load?
Standard plugins can be found in ~/.oh-my-zsh/plugins/*
Custom plugins may be added to ~/.oh-my-zsh/custom/plugins/
Example format: plugins=(rails git textmate ruby lighthouse)
Add wisely, as too many plugins slow down shell startup.
plugins=(
git
)
source $ZSH/oh-my-zsh.sh
User configuration
export MANPATH="/usr/local/man:$MANPATH"
You may need to manually set your language environment
export LANG=en_US.UTF-8
Preferred editor for local and remote sessions
if [[ -n $SSH_CONNECTION ]]; then
export EDITOR='vim'
else
export EDITOR='mvim'
fi
Compilation flags
export ARCHFLAGS="-arch x86_64"
ssh
export SSH_KEY_PATH="~/.ssh/rsa_id"
Set personal aliases, overriding those provided by oh-my-zsh libs,
plugins, and themes. Aliases can be placed here, though oh-my-zsh
users are encouraged to define aliases within the ZSH_CUSTOM folder.
For a full list of active aliases, run alias.

Example aliases
alias zshconfig="mate ~/.zshrc"
alias ohmyzsh="mate ~/.oh-my-zsh"

We don't have a spaceship theme. Where is it and what are it's contents? Most probably there is something wrong with it.

I was using this https://github.com/denysdovhan/spaceship-prompt

I have the same issue, and my theme is robbyrussell. A '~' symbol follows every command， and once execute ‘source .zshrc', the one more a  ‘~’ , like:
->: ls
lsDocument       Download      Picture       Desktop
~
->: source .zshrc
~
->:(press Ent)
~~
->:source .zshrc
~~
->:(press End)
->:

I have no idea about it, could you give me some advice?
Thank you

Can you post your zshrc file?


Verify validator metadata upon join request

In sui_system.move, we provide interface request_add_validator for a new validator to join. The metadata provided will eventually be used to create committee and connect network during reconfiguration.
Currently these data are not validated, and during reconfiguration if any of them is invalid we will panic and there is no way to recover. You can find most of the panic points in https://github.com/MystenLabs/sui/blob/main/crates/sui-types/src/sui_system_state.rs (look for expect or unwrap).
Hence we must validate them when we receive the request. We need to implement a native function that takes in a reference to the validator metadata, and attempt to convert all the bytes fields into actual value (e.g. keys, network addresses and etc), and if any of them fail we must reject the join request. This ensures that during reconfiguration we cannot fail.

I think that we should do as much as we can to do onchain verification of this data, although in addition to that we should also teach our validator software to tolerate when that data is incorrect and handle the case when it is incorrect properly instead of crashing.


Rewrite CGDataProvider and change DWrite to use it directly

Rewrites CGDataProviderRef to subclass from CppBase rather than being NSData.  Changes the DWrite classes to (un)register fonts to use CGDataProviderRef directly.
Fixes #1450

This change is

size_t memSize = sizeof(struct __CGFont) - sizeof(CFRuntimeBase);

it would be nice to have it as cpp based at some point

Refers to: Frameworks/CoreGraphics/CGFont.mm:129 in 9a3d204. [](commit_id = 9a3d204398ea7c113b82eacf4299d2f9c5cbd10a, deletion_comment = False)

:clock1:


Web Request SSL/TLS error during program execution.

Not sure if this is the best place to post. But I am running out of ideas.
I keep having the following error pop up when running my program: "The request was aborted: Could not create SSL/TLS secure channel."
The exception appears to be happening at a couple different places. It occurs once in Listing.cs at line 190 while my program is looking through comments. I am assuming this means that my OAuth token has expired?
The next time it happens is at WebAgent.cs line 108 when I try to log back in using AuthProvider(username,password) line 161
I have tried adding-
ServicePointManager.SecurityProtocol = SecurityProtocolType.Tls
| SecurityProtocolType.Tls11
| SecurityProtocolType.Tls12
| SecurityProtocolType.Ssl3;
to the top of my code in an attempt to fix the issue but it didn't seem to help. Is there something I am doing wrong here that can stop these exceptions from popping up?
Thank you,
Ac5000

Are you using the default WebAgent?
Also check whatever WebAgent you are using to see if it's making the request with https, I think RedditSharp still allows you to use non-https, but Reddit broke that so it needs removed.

I am creating a new webagent at the top of my program-
static IWebAgent myWebAgent = new WebAgent();
That is getting fed in when I am setting my authProvider with the oauth stuff for my login block:
authProvider = new AuthProvider(oauthID, oauthSecret, oauthRedirect, myWebAgent);
token = authProvider.GetOAuthToken(username, password);
myWebAgent.AccessToken = token;
reddit = new Reddit(token);
I'm using the current WebAgent from here. I'm not sure where I would find if it's using non-https.

You can try creating the AuthProvider without passing in a WebAgent
OR change static IWebAgent to Static WebAgent and check what the .Protocol property is.

Thank you greatly for working with me by the way. I am using this project as a way to learn C#.
I started with not passing in WebAgent to AuthProvider. Added it in an attempt to track this issue down.
Changed from IWebAgent to WebAgent. The protocol property is "https"

Any more ideas as to what is happening or how I can fix this?
Also, would you be able to explain how the GetOauthToken(string code, bool=refresh) method works? Mostly what is the code/where do I get it? I would rather my program just refresh the token than get a whole new one each time it hits the 401 unauthorized.

No worries! Umm lets see. So here's the general run down on how that works.
AuthProvider is the class that well, provides authentication, as you seem to have figured out haha. Reddit has a couple different kinds of authentication you can use. And as I type this, I just realized another issue it could be I'll get to that in a minute though.
Reddit can authenticate using a Username and Password like you are trying to do, or it can authenticate with OAuth and a popup window that actually opens a page for the user to accept the authentication request.
Both of these authentication patterns use "tokens", but only 1 makes use of refresh tokens. Refresh tokens are a concept of OAuth and basically allow you to get another valid security token after yours has expired (in 1 hour in the case of Reddit). Username and Password authentication does NOT give you a refresh token. This means you'll need to provide your username and password again after the hour is up to get a new token.
GetOAuthToken(string code, bool=refresh) is a method that is used by the OAuth pattern to either get a security token on the first request (refresh = false), or to get a new security token by providing a refresh token (refresh = true).
In your case, you'll want to be calling GetOAuthToken(username,password) like you indicated earlier.
Now, the other thing to check here. What type of app did you make in Reddit? There are 3 options, "Web App", "Installed App", and "Script". You can look at Reddit's OAuth documentation for a little more detail on that, but if you are doing username password authentication to create a bot which it seems like you are doing, you want it to be a "Script" application. A Web App will expect an OAuth redirect, and an Installed App won't give you a client secret, although it might be possible to work with that, I haven't tried it personally though.

I made a "script" app. So I have an id, secret, and redirect. Have been using the GetOAuthToken(username,password) method currently.
Where does the the "string code" on the other method come from? Is that something you get when using the other app type then?
The bot runs fine until it hits the hour mark, errors on 401 for the token expiring. I wait 30 seconds. I then try doing revoketoken but it fails with the SSL/TLS error, try logging in again after that and it gets the same error.
Sometimes though, I will get SSL/TLS error just while checking listings. Less than an hour after I log in or back in.

Ahhhhh okay I know what's going on then.
The "string code" is from the other authentication flow yes.
After an hour, the access token expires, and you have to get a new one. Since the access token already expired, you can't revoke it (not sure why it's giving you an SSL/TLS error tbh). You just need to request a new token like you did with the first token you got.
If you are getting the SSL/TLS error while just checking random things, it MAY have something to do with Reddit's servers then and them sending out 503's or something, but I'm not sure. What I would do to test it, is clone the source of this repo, add it as a project reference to your bot, and then you can step into the code and see what the actual server response is (it might be a different error that's getting masked as an SSL issue).

Ahh, that makes sense with the revoke situation. I will remove that then and just log in again.
I'll have to give that a shot with running it as a project reference. I was thinking about doing that but wasn't quite sure. I'll post back if I find anything unusual.
Thank you for your help. It was much appreciated.

Sounds good, glad I could help clear that up a bit for you.

I'm seeing the same error when requesting a token. It is odd that on the third retry, it works fine.
https://github.com/RockstarLabs/OwinOAuthProviders/issues/132
Were you able to resolve?

@jarrettv if you run into it again, try and quickly ping / load up Reddit and see if they are having server issues at the same time. I commented on your other issue for the Owin provider, but that's what my gut is telling me the issue is.


survey js submit survey with api call

Are you requesting a feature, reporting a bug or ask a question?
requesting a feature
What is the current behavior?
When survey is completed shows the html text indicating survey is completed
What is the expected behavior?
when submit a survey, may I have a customized post api call inside it, please?
Is there any chance that I could add a cancel button beside it, or do you have this function provided?
Provide the test code and the tested page URL (if applicable)
Tested page URL:
Test code
 model.onComplete.add(function(result) {
  document.querySelector('#container').innerHTML = "

Survey has been submitted
" ;
}

@xiuzhen1103 hello! If I understood you correctly, you need to control "onclick" of "complete" button?

@xiuzhen1103 Please take a look at survey.completedHtml property
Here is the demo of using it: http://surveyjs.org/examples/jquery/survey-processtext/

control "onclick" of "complete" button was one question,
second one is :
do you provide cancel button? to confirm if cancel , something like this

I think it makes sense. We could optionally provide "сonfirmation dialog" when "complete" button "onclick" from the box or just give event and possibility to handle it. @xiuzhen1103, @andrewtelnov what do you think about it?

hi Dmitry,
Thanks for the suggestion , is there any chance that I could have a cancel button beside submit button for our project.

@xiuzhen1103 hello! And what will "cancel" button do? Will it go to the first page and clean all answers or something like that?

cancel button pop up a dialog modal as the screenshot I posted, provides two option Yes and No, when click no return to survey form, click yes, direct to other page.

@xiuzhen1103 as a workaround you could try to do it by yourself. Here an example: https://plnkr.co/edit/DYmf4jOSSDWEu7uwHB7a?p=preview

and we will think about this feature

Thanks Dmitry,
unfortunately, this is not working

@xiuzhen1103 that is strange, I can't reproduce this problem in chrome, which browser do you use?

chrome

Version 60.0.3112.66

It worked now, very strange.

Is there any chance that I could rename button "complete" to "submit"

@xiuzhen1103 yes, you can use localization http://surveyjs.org/examples/jquery/survey-localization/

localization is not working
https://plnkr.co/edit/DYmf4jOSSDWEu7uwHB7a?p=preview

@xiuzhen1103 you forget about survey.locale = "my" and it should be defined before render: https://plnkr.co/edit/ZWAxird0m4PPZMucddEB?p=preview

@xiuzhen1103 Another way to implement it is to use the custom navigation. You may turn off showing the standard navigation button and use your own. Here is the demo: http://surveyjs.org/examples/jquery/survey-customnavigation/
As for button text, there is an easy solution as well. Here is the list of properties to change the buttons text:
survey.pagePrevText, survey.pageNextText and survey.completeText.
Thank you,
Andrew

@andrewtelnov, thanks millions!!!  this would make more sense, and works perfectly. I have tried in my local project, localization not work for me, error message its locales is not defined.

@xiuzhen1103 yes, I forgot about custom navigation! By the way, for using custom localization you need to redefine (or copy-past and modify) all strings, so it is more complex solution.

working excample with custom locale for someone else from issues search: https://plnkr.co/edit/ZWAxird0m4PPZMucddEB?p=preview

Thanks for all the help!!! @dmitrykurmanov  @andrewtelnov


how to populate text box based on change of value of another field.

I have a tabbed Dialog. I have managed to populate second tab.
The second tab has Members details which corresponds to the selected member.
Want to add new details to selected member. The member number is a dropdown field which am able to pass the details. However, Member Name field is auto populated based on the selected Member.
When select member detail dialog and add new details, selecting the dropdown works fine to populate the Fullname. But when accessing it from the tabbed and since the dropdown field is set as not Updatable, the select statemement below is not invoked.
this.form.MemberId.changeSelect2(e => {
                var memberId = Q.toId(this.form.MemberId.value);
                if (memberId != null) {
                    this.form.MemberName.value = MembersRow.getLookup().itemById[memberId].FullName;
                }
            });

How to I update the Member full Name when Member No value changes or is not null

see cascade value and cascade field.

Thanks so much @adhikamafran.


SAK-49615 Lessons: Title of a Question Item incorrectly displayed

https://sakaiproject.atlassian.net/browse/SAK-49615

@ottenhoff Do you think, we should strip off the HTML tags from CKEditor directly before saving it?

no, HTML is valid and ok


[2069] Allow publishing of courses in next cycle

Context
Allow publishing of courses in the next recruitment cycle
Changes proposed in this pull request
Render form to publish courses in the next recruitment cycle
Guidance to review
🚢

 Next cycle publishing (MCFE)

 Next cycle publishing (MCFE)


Update vocabulary to use watcher instead of cop

Trying this again - change should be against master this time.   Update vocab to change cop to watcher.

Verified slack support is working ok with these changes.

Can one of the admins verify this patch?

LGTM


(prod) DA-4013: Update list org api to first lookup and then search org name

Implementing Hybrid solution as suggested by @1010sachin here -
https://jira.linuxfoundation.org/browse/DA-4013?focusedCommentId=532141&page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#comment-532141
Verified and tested locally.

Merged this PR, @lukaszgryglicki request you to deploy.

Done.


Use RUST_LOG env to configure logging if present

We identified a perf impact when writing logs.  In this case it would be helpful to only run with logs set at a "warning" or "error" level.  We attempted to turn most logs off via containerd with containerd -l fatal but it still dumped logs.
After some investigation it appears containerd only sets a flag not a level (https://github.com/containerd/containerd/blob/7467d81987e1a54dd2a89e052ae9429a4c921e9e/core/runtime/v2/binary.go#L66-L68) and our code will default to info https://github.com/containerd/rust-extensions/blob/cc445f54c0fa8e9b35376550874422c932ad1566/crates/shim/src/logger.rs#L135-L139
changes
This reads the environment variable RUST_LOG and will use value from there if set. It can be set to any values in the log crate. Otherwise the behavior is the same. If the -debug flag is passed it will choose the highest of the two values.
This also introduces ability to configure the default setting through the shim Config object:
run::<Shim>(&id, Some(Config{default_log_level: "error", ..Default::default()}));

If all the configuration fails it will fallback to logging to to info level.

We identified a perf impact when writing logs.

Would be curious to learn more about costs of logging to pipe/fifo.

Would be curious to learn more about costs of logging to pipe/fifo.

@radu-matei @lann could you share some of your findings?

I suspect that the performance issues have more to do with mutex contention, but I haven't done any analysis to confirm.

mutex contention

I wonder if going this route would be more viable instead of introducing custom var? parking_lot's mutex or maintaining a message queue.

I think it should be both. There is an expectation from users that since they are using env crate they can adjust the log level.  In the current set up, it is impossible to get trace logs from the shim or other components since we only go to the debug level.

@mxpv I've logged #249 for further investigation on the logging issue.
Of the feedback in the PR above above which changes would you like to see before merging this?

Let's have this one in. We can follow up with further improvements.


net/wireguard: introduce warning before going stable

0.10 will be for 19.1.7 as a devel and CFT .. when there are no show stoppers we can include it for stable in 19.1.8

@mimugmail You have clamav things in this PR, can you remove please? Then should be ready for @fichtner after that

This is only a merge of master, no Changes to clamav code :)

Merged, thanks!


Quit with message if platform not macOS

Only OS X & macOS is supported for now. Sorry 🐧  & 🖼

Fixed in f43821ee16b09fffb81654af840b879f49af2a1f


Bug Fix: Pull latest version from given channel

Use the new channels/ API to pull the latest version from the configured channel.
Signed-off-by: Tom Duffield<EMAIL_ADDRESS>

Confirmed this fixed the issue I was seeing via the habitat-build cookbook:
[2017-07-21T17:36:49+00:00] INFO: hab_package[core/hab-studio] upgraded core/hab-studio to 0.26.1/20170715034337

    - upgrade package core/hab-studio from 0.27.0-dev/20170721072438 to 0.26.1/20170715034337

Need to fix the package test, the below will get it passing
hab_package 'core/bundler' do
  channel 'unstable'
  version '1.13.3/20161011123917'
end


Compile to a single library file ?

Hi Dougal,
Thanks for doing this. I'm pretty new to Cython and my C/C++ is from 20 years ago. I'd ideally like to wrap the FLANN C++ code in Cython and then compile it all to a single .so / .pyd file. Is this possible or will there always be two libs (one Cython, one FLANN native) ?
Thanks
Ian

Hi, I think that should be possible, but I don't see how to do it in a small amount of work.
You could probably adjust cyflann to use nanoflann, which  would definitely fit that bill.
Depending on your needs, you also might prefer using FALCONN or annoy.

Thanks - I looked at Annoy, but it doesn't have some of the features we need. I'll take a look at nanoflann though !

Just looked at nanoflann - seems it's just 2D and 3D optimised (we've got 100s of features). So...cyflann it is !  When I compile the pyd / so file, what dependencies do I need to package with it if I want to run it in a vanilla python/anaconda environment ? (I found a 1MB .o file - not sure if that's been compiled into the final file)

@MonkeyChap Sorry, I just found a partially-written reply to this in my drafts folder -- thought I had forever ago!
The .o file isn't necessary. It's easiest if you use setuptools to build everything, of yours; then setup.py install will only install the .so that you need. Same for cyflann itself. Check out https://github.com/dougalsutherland/skl-groups/blob/master/setup_accel.py for an example of doing that.
If you use conda, cyflann will hopefully be in conda-forge soon and then you can just install it that way. flann already is.

cyflann is now in conda-forge, so if you just conda install -c conda-forge cyflann then that should give you all the dependencies you need. Let me know if you need any help and I'll try to reply more promptly. :)


Document that v3 Route host can be a wildcard (*)

Private Domains can have host be set to a wildcard * to match on traffic to any host under that domain. It was unclear from the docs that this was an acceptable value for the host field.

[x] I have reviewed the contributing guide

[x] I have viewed, signed, and submitted the Contributor License Agreement

[x] I have made this pull request to the main branch

[x] I have run all the unit tests using bundle exec rake

[] I have run CF Acceptance Tests

Thanks!


Networking with incremental Rendering?

Question about Networking
I have a little design Question as a Network Engineer i am wondering about how you overcome this fact
That a Stream that not ends will simply make the Site load as long as you stream.
Little Importent base knowleg about TCP/IP -> HTTP Implamentation
Dependencys inharitance the express -> http -> net
The Problem with write Streams
When you write out some data via a stream to the direct net socket (The Browser Connection)
The Page is only loaded after you sended a end so the page will load until you end that is slow
i think loading that async as webcomponent bypasses all that limits and also your iframe way for syncing up the dom.
Fact:
a webcomponent returns a HTML Document like a iframe
I Think at present you want to render a Iframe to bypass that? and Stream to the iframe right?
Some More example code about sending files to a browser
//sendStream
function sendFileStream(req, res, path, opts) {
      // create send stream
    var stream = send(req, path, opts)

    // add directory handler
    stream.on('directory', onDirectory)

    // add headers listener
    if (setHeaders) {
      stream.on('headers', setHeaders)
    }

    // add file listener for fallthrough
    if (fallthrough) {
      stream.on('file', function onFile () {
        // once file is determined, always forward error
        forwardError = true
      })
    }

    // forward errors
    stream.on('error', function error (err) {
      if (forwardError || !(err.statusCode < 500)) {
        next(err)
        return
      }

      next()
    })

    // pipe
    stream.pipe(res)
}
//sendFile Kernel method
var open = process.binding('fs').open;
var sendfile = process.binding('fs').sendfile;
var bufSz = 1 << 10;
function sendfileKernel(req, path) {
  open(path, process.O_RDONLY, 0, function(err, fd) {
        // Track our offset in the file outside of sendData() so that its value
        // is stable across multiple invocations
        var off = 0;
        var s = req.connection
        var sendData = function() {
            // We only care about the 'drain' event if we're not done yet
            s.removeListener('drain', sendData);

            try {
                // Try to send file data until we either hit EOF, or fail the
                // write due to EAGAIN
                do {
                    nbytes = sendfile(s.fd, fd, off, bufSz);
                    off += nbytes;
                } while (nbytes > 0);

                s.end();
            } catch (e) {
                // Only EAGAIN is special; everything else is fatal
                if (e.errno !== process.EAGAIN) {
                    throw e;
                }

                // When the socket has room for more data, start pumping again
                s.on('drain', sendData);

                // Manually fire up the IOWatcher so that the 'drain' event
                // fires. The net.Stream class usually manages this for you,
                // but since we're going around it via sendfile(), we have to
                // do this manually.
                s._writeWatcher.start();
            }
        };

        if (err) {
            console.error(err);
            s.end();
            return;
        }

        // Kick off the transfer
        sendData();
    });
}

Is your question about the browser load event? The html request is ended before the incremental rendering is complete. The streams don't all have to end at the same time. html request ends, page's load event is fired, and mutations continue until they are finished.

@matthewp where will that mutation happen who is sender and reciver?

@matthewp is this only about incremental rendering of ndjson or incremental rendering in general?


ci(accessibility): update a11ywatch<EMAIL_ADDRESS>

update action to v1.12.0 fixing github action machine limits.

All good to merge, @j-mendez? 😊

All good to merge, @j-mendez? 😊

Hi @LauraLangdon - good to merge thanks!


Include declaration documents on service page

https://trello.com/c/h3dtJytJ/882-display-modern-slavery-document-uploads-on-service-page

The Service presenter class already combines supplier data with service data. We're now adding in declaration data (where the Modern Slavery doc URL is kept).
The hardcoded document list is moved to the top of the service_presenters module for easier future editing.
The section of the manifest that previously displayed the documents at the bottom of the page has been removed, and a new _service_documents.html template has been included (with the same content as the top panel).
Content changes of the document names need to be OK-ed with Stefan.

@benvand tests now added!


settings Pane doesn't capitalize settings correctly

VSCode Version: 1.34.0-insiders
OS Version: macOS

Steps to Reproduce:

See how the first "Powershell" only has the P capitalized and the second "Power Shell" has a space?
This is, I'm guessing, because the first "Powershell" is all lowercase and the second "Power Shell" is camelCase in the setting:
powershell.powerShellExePath

How can I get the desired casing of "PowerShell"?

Right now we don't give extensions any control over the formatting of this label. I have some formatting for proper nouns, I can try to fix PowerShell the same way.

Thanks @roblourens!


refactor: resolve EventSubscriber warnings

What this PR changes/adds
Change the EventSubscriber interface API
Why it does that
Avoid compiler warnings
Further notes

also EventEnvelope suffers from warnings, caused by the fact that the type of the payload is not known until it has been set with the payload builder method. I'd propose to add the payload as a member of the newInstance method to avoid this, eventually in another PR.

Linked Issue(s)
Closes #2740
Checklist

[ ] added appropriate tests?
[ ] performed checkstyle check locally?
[ ] added/updated copyright headers?
[ ] documented public classes/methods?
[ ] added/updated relevant documentation?
[x] assigned appropriate label? (exclude from changelog with label no-changelog)
[x] formatted title correctly? (take a look at the CONTRIBUTING and Etiquette for pull requests for details)

Codecov Report
Patch coverage: 66.66% and project coverage change: +0.03 :tada:

Comparison is base (6ee08b7) 64.31% compared to head (53bb4b0) 64.35%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2741      +/-   ##
==========================================
+ Coverage   64.31%   64.35%   +0.03%
==========================================
  Files         949      949
  Lines       19533    19526       -7
  Branches     1165     1165
==========================================
+ Hits        12563    12565       +2
+ Misses       6514     6505       -9
  Partials      456      456

Impacted Files
Coverage Δ

...ipse/edc/connector/core/event/EventRouterImpl.java
88.46% <ø> (ø)

...lipse/edc/junit/matchers/EventEnvelopeMatcher.java
0.00% <0.00%> (ø)

...pse/edc/event/cloud/http/CloudEventsPublisher.java
91.17% <ø> (ø)

...r/callback/dispatcher/CallbackEventDispatcher.java
87.50% <ø> (ø)

...k/dispatcher/CallbackEventDispatcherExtension.java
85.71% <100.00%> (ø)

... and 15 files with indirect coverage changes
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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


⚠ DDoS-Relay US-W-sfo3-02 has degraded performance

In 833fbcf, DDoS-Relay US-W-sfo3-02 (http://<IP_ADDRESS>:42069/;csv) experienced degraded performance:

HTTP code: 200
Response time: 138 ms

Resolved: DDoS-Relay US-W-sfo3-02 performance has improved in 732cb7d after 15 minutes.


Shift-clicking item into Dark Steel Anvil temporarily dupes item

If I shift-click an item into the anvil, it appears both in my inventory and in the anvil.  Once I complete the repair and close the UI, my inventory shows one item again.  I'm using build 350.

As far as I know, that's a vanilla bug.

Yes, yes it is


Remove (++~) and (++=) as (++) is removed from prelude

See #44

Looks good, thanks, but let's wait until the next major release since this is a breaking change.

Does this need anything else or should we merge it?

I'll merge it before the next major release.


Question: How to add URLs in Command Line on macOS? (the input device is not a TTY error)

I have archivebox running on my Mac with docker-compose. Everything is working so far, I can add URLs via the web GUI.
But when I try for example to ingest a textfile with URLs via the commandline, I always get "the input device is not a TTY".
The documentation is a bit sparse on that point, as it just says: "pass in URLs via stdin." I don't really understand what is meant by that. When I copy docker-compose run archivebox add < ~/Downloads/bookmarks.html (and change the path of course) I always get "the input device is not a TTY"
What do I need to do to be able to ingest URLs in the command line?

Please post your docker-compose.yml file and the output of docker-compose run archivebox version.

Closing as stale.

@pirate Is there any chance you can explain what this error means?  I too am getting this error, although it's on a fresh install of Ubuntu 24.04, v0.7.2.  I thought everything was working, but passing in my Firefox .json export it doesn't like:

Here is my docker compose run archivebox version info:

On my old server, I used "docker-compose" and it did work...but "docker compose" doesn't seem to want to play ball.  I will admit, I get pretty confused as the doc seems to flop back and forth between "docker compose" and "docker-compose" which from the tiny bit of reading I've done 'docker compose' seems like it's the newer/better option nowadays, so that's all I've installed on this new server I'm trying to spin up (if I try to run "docker-compose" I get the normal message telling me it's not installed).  Do I absolutely need to install this alongside docker-compose and memorize which commands need the dash vs which ones don't?  Or is there a way to make "docker compose" work for this task?  Sorry if I'm missing something obvious here.

docker-compose used to be a separate package but they merged it into docker, so as of ~2022 you should only run docker compose
As for the -T / -it stuff I agree it is a confusing common issue for new docker users. Basically the only thing to remember is:

docker run ... sometimes needs -it
docker compose run ... sometimes needs -T

(Never the other way around)
Basically the flags tell the docker daemon whether to expect interactive input on stdin, a pipe, or something else, and the default is different between plain docker and compose, hence the different args to change the default.
Hope that helps!

docker-compose used to be a separate package but they merged it into docker, so as of ~2022 you should only run docker compose
As for the -T / -it stuff I agree it is a confusing common issue for new docker users. Basically the only thing to remember is:
* `docker run ...` sometimes needs `-it`

* `docker compose run ...` sometimes needs `-T`

(Never the other way around)
Basically the flags tell the docker daemon whether to expect interactive input on stdin, a pipe, or something else, and the default is different between plain docker and compose, hence the different args to change the default.
Hope that helps!

Appreciate the crash course, definitely helps out!!


TC Test Search with both Leading and Trailing Spaces

Description: Verify the search functionality with both leading and trailing spaces.
Preconditions: The https://admin.peviitor.ro/ page is opened.

Step 1. Click the "Companii" button.
Expected results: The "Companii" page is displayed.

Step 2. Verify that the search bar works correctly when using both leading and trailing spaces.
Expected results: The search bar returns the correct information, not considering the leading and trailing spaces.


Revert checks for if graphics are deleted

Although this increases compat with GDI+, we’re backing this out because this is undefined behaviour.
See https://github.com/mono/libgdiplus/pull/95#discussion_r141030353

@hughbe,
Thanks for having already signed the Contribution License Agreement. Your agreement was validated by .NET Foundation. We will now review your pull request.
Thanks,
.NET Foundation Pull Request Bot


Create a Remote Procedure Call helper

This will be used for pylint, pep8, and introspection.

Create a QProcess with the executable of the current project, adding spyder and the appropriate dependencies (e.g. jedi) to the path.
Use ZMQ to pass pickled objects to a remote procedure call.
The remote end imports the desired modules(s) and performs the remote procedure calls, returning the result over the ZMQ socket pair.

RemoteClient(executable, dependencies, imports)
    onReply(request_number, result)
    send(func_name, *args, **kwargs)  # returns a request number
RemoteServer(imports, server_port, client_port)

Two things about this:

Pyflakes needs to be called like this too :-)
Fortunately all these packages work on both Python 2 and 3 at the same time (so importing from a Python 2 installation in a Python 3 remote process won't be a problem). The only missing piece was rope but its next version will be Python 3 compatible too.

@goanpeca, this is what you wanted :-)

:-)


docs: Fix VITE_CORE_API_URL in README.md

Should this section even be there? Doesn't the default .env.development already set this up correctly?

Should this section even be there? Doesn't the default .env.development already set this up correctly?

It does look like the combination of .env + .env.development files handle these 2 env vars. I deleted the .env.local files and was able to run things.

Should this section even be there? Doesn't the default .env.development already set this up correctly?

Might still need this for custom setups. Should I prepend it with something like
If your DHC address is different from the default `http://localhost:10000`, edit `.env.local` in each package to contain...

?

Yes that's better.


Formatting.

Issue #, if available:
Description of changes:
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Codecov Report

Merging #601 into master will decrease coverage by 0.00%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master     #601      +/-   ##
==========================================
- Coverage   83.74%   83.73%   -0.01%
==========================================
  Files         181      180       -1
  Lines       10284    10281       -3
==========================================
- Hits         8612     8609       -3
  Misses       1672     1672


zeitgeist: 1.0 → 1.0.1

Motivation for this change
Minor update and clean-up.
https://cgit.freedesktop.org/zeitgeist/zeitgeist/commit/?id=3cf082f10692ab8fd75a0a59dfa6a97b77a39bd6
Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option build-use-sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[x] Fits CONTRIBUTING.md.

Failure on x86_64-darwin (full log)
Partial log (click to expand)
Package ‘zeitgeist-1.0.1’ in /private/var/lib/ofborg/checkout/repo/38dca4e3aa6bca43ea96d2fcc04e8229/builder/lnl7-mac/pkgs/development/libraries/zeitgeist/default.nix:38 is not supported on ‘x86_64-darwin’, refusing to evaluate.

a) For `nixos-rebuild` you can set
  { nixpkgs.config.allowBroken = true; }
in configuration.nix to override this.

b) For `nix-env`, `nix-build`, `nix-shell` or any other Nix command you can add
  { allowBroken = true; }
to ~/.config/nixpkgs/config.nix.

Success on aarch64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/b47hd9pch41c2jxdg3j8ppmvpzx2qgki-zeitgeist-1.0.1/lib/libzeitgeist-2.0.so.0.0.0
gzipping man pages under /nix/store/b47hd9pch41c2jxdg3j8ppmvpzx2qgki-zeitgeist-1.0.1/share/man/
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/b47hd9pch41c2jxdg3j8ppmvpzx2qgki-zeitgeist-1.0.1/lib  /nix/store/b47hd9pch41c2jxdg3j8ppmvpzx2qgki-zeitgeist-1.0.1/bin
patching script interpreter paths in /nix/store/b47hd9pch41c2jxdg3j8ppmvpzx2qgki-zeitgeist-1.0.1
checking for references to /build in /nix/store/b47hd9pch41c2jxdg3j8ppmvpzx2qgki-zeitgeist-1.0.1...
strip is /nix/store/lvx1acn1ig1j2km8jds5x3ggh3f2wa8v-binutils-2.28.1/bin/strip
Moving /nix/store/b47hd9pch41c2jxdg3j8ppmvpzx2qgki-zeitgeist-1.0.1/lib/python2.7 to /nix/store/7fkwxlck1y08zv3g8vpz5x6ii7izxkfq-zeitgeist-1.0.1-py/lib/python2.7
rmdir: failed to remove '/nix/store/b47hd9pch41c2jxdg3j8ppmvpzx2qgki-zeitgeist-1.0.1/lib': Directory not empty
/nix/store/b47hd9pch41c2jxdg3j8ppmvpzx2qgki-zeitgeist-1.0.1

Success on x86_64-linux (full log)
Partial log (click to expand)
shrinking /nix/store/l7rvi48k0xzwdisw3s2530nq79gxlc3w-zeitgeist-1.0.1/bin/zeitgeist-daemon
shrinking /nix/store/l7rvi48k0xzwdisw3s2530nq79gxlc3w-zeitgeist-1.0.1/lib/libzeitgeist-2.0.so.0.0.0
gzipping man pages under /nix/store/l7rvi48k0xzwdisw3s2530nq79gxlc3w-zeitgeist-1.0.1/share/man/
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
stripping (with command strip and flags -S) in /nix/store/l7rvi48k0xzwdisw3s2530nq79gxlc3w-zeitgeist-1.0.1/lib  /nix/store/l7rvi48k0xzwdisw3s2530nq79gxlc3w-zeitgeist-1.0.1/bin
patching script interpreter paths in /nix/store/l7rvi48k0xzwdisw3s2530nq79gxlc3w-zeitgeist-1.0.1
checking for references to /tmp/nix-build-zeitgeist-1.0.1.drv-0 in /nix/store/l7rvi48k0xzwdisw3s2530nq79gxlc3w-zeitgeist-1.0.1...
strip is /nix/store/b0zlxla7dmy1iwc3g459rjznx59797xy-binutils-2.28.1/bin/strip
Moving /nix/store/l7rvi48k0xzwdisw3s2530nq79gxlc3w-zeitgeist-1.0.1/lib/python2.7 to /nix/store/yw68mj2l4hy1na0sda5y9n9v6r7pdmc1-zeitgeist-1.0.1-py/lib/python2.7
rmdir: failed to remove '/nix/store/l7rvi48k0xzwdisw3s2530nq79gxlc3w-zeitgeist-1.0.1/lib': Directory not empty


How to change the percentage value dynamically?

How to change the percentage value dynamically?

Hello @twob-shahid ,
thanks for using percircle.
My greatest apologies for having been not able to investigate your query earlier .
The requirement you asked for, is not currently available, but I would be happy to review a pull request for it.
Thanks,
TB

Hey @twob-shahid ,
as you can see, your request is not done!
Please update to v1.0.24!
You can check the updated documentation on how to achieve updating the value of the a percircle object dynamically.