--esModuleInterop changes meaning of declaration

TypeScript Version:  2.8.0-dev.20180201
Code
m.d.ts
declare function m(): void;
declare namespace m {}
export = m;

n.d.ts
import * as m from "./m";
export const n: typeof m;

a.ts
import { n } from "./n";
n();

Expected behavior:
No error.
Actual behavior:
No error  normally. With tsc --esModuleInterop:
src/a.ts(2,1): error TS2349: Cannot invoke an expression whose type lacks a call signature. Type '{ default: () => void; }' has no compatible call signatures.
src/n.d.ts(1,1): error TS7038: A namespace-style import cannot be called or constructed, and will cause a failure at runtime.

The user may not have control over the definition file, so I think we could a) omit the error in the definition file, and b) preserve the original meaning where typeof m is callable.

I'm not really a fan of eliding the error just because the source import is in a definition file. It's a legitimate shape (reexporting the namespace as a member) that would strip the call and construct signatures if the .d.ts truly reflects the source. I think doing this for compat reasons may hamstring our ability to catch real errors - each of these files could easily be generated from the user's own code and actually be indicative of an issue.

@andy-ms You suggestion in the associated dt PR:

That should probably be import serveStatic = require("serve-static") in express/index.d.ts -- could you see if that works for you?

Is really the correct fix. You shouldn't use an es6-style import if you don't want es6-style semantics.

Chatted with @weswigham offline, we need to go update definitelytyped to remove all references of this form to avoid users having to deal with the error.

@weswigham can you run the tests and let us know which ones need updating? i would be happy to help fixing them as needed.

I opened https://github.com/DefinitelyTyped/DefinitelyTyped/pull/23355 with fixes for the packages that I identified had issues. I can't seem to load @andy-ms 's PR that enables the flag on all packages at https://github.com/DefinitelyTyped/DefinitelyTyped/pull/23354 - it keeps Unicorn'ing.

@weswigham If it helps, I was able to load the page in a Chrome incognito window; maybe not being logged into GitHub has some effect.

It also would be great if we could get, in writing, in some DefinitelyTyped guidelines, that you always must write import x = require('x') in DefinitelyTyped code if the package export is commonjs.
As an example, a lot of the pull requests to react-loadable involved people changing the require-imports into star imports and sometimes my request to change to require-import were ignored because "it works", or when they were done people would eventually revert it in some pull request I didn't get to review.

So is this then not considered a bug? I think this is good behavior as @weswigham stated.

Had a chat about this offline, now that DefinitelyTyped has been updated, we will add a new lint rule to ensure that reexports of an export =  use import = require. We will not force the use of --esModuleInterop flag on DT packages in the time being.

DTSLint is now enforcing this on DT, so I think we're good

That's backwards; the problem is namespace imports, not default imports.

The namespace imports are flagged in the type checker, as all packages now have esModuleInterop on by default. The lint rule is "backwards" to ensure that you produce a type definition file that also works people not using allowSyntheticDefaultImports or esModuleInterop.

Is https://github.com/DefinitelyTyped/DefinitelyTyped/blob/master/types/json-stable-stringify/index.d.ts correct?

Yes. The (commonjs) package exports a single top-level function. You should import as import stringify = require("json-stable-stringify") (if targeting commonjs modules) or, if esModuleInterop and/or allowSyntheticDefaultimports is set, you may use import stringify from "json-stable-stringify"

Hmm, when I use import stableStringify from 'json-stable-stringify' with "esModuleInterop": true I get this error
src/util.ts:1:8 - error TS1192: Module '"/Users/domoritz/Developer/UW/vega-lite/node_modules/@types/json-stable-stringify/index"' has no default export.

1 import stableStringify from 'json-stable-stringify';
         ~~~~~~~~~~~~~~~

error Command failed with exit code 1.

I assume with module: es2015, yeah? We only consider esModuleInterop to imply allowSyntheticDefaultimports when the module target is commonjs, amd, umd, or system.

Yes.

Right, so you need to set allowSyntheticDefaultimports: true yourself to assert that whatever is running your modules will understand that you expect default imports for non-esm modules to exist.


[Dash] Use dash docset keywords as raycast command titles

Extension – Dash
Author: @sxn @RSO

Describe the feature and the current behavior/state.
I would like the workflow for looking up docs to be:

⌥ + space (open raycast)
"css" (choose docset)
⇥ (launch command)
"margin" (query docset)

Whereas right now there are 2 additional steps:

⌥ + space (open raycast)
"dash" (choose dash command)
⇥ (launch command)
"css" (find docset)
⇥ (choose docset)
"margin" (query docset)

Dash does offer the user the ability to set "docset keywords" in its interface, which it uses for its own search, as well as in its Alfred extension. It would be great to have the same ability in Raycast.
Who will benefit with this feature?

every Dash user who has already too many keyboard shortcuts to remember and would like to not have to remember one for opening dash in raycast.

Any Other info.

At this point, Raycast does not support dynamically defining commands in Extensions

To be fair, Alfred doesn't really support this either, but since its extensions are clear text (xml IIRC) this kinda is something that is doable. However I think this feature could be really neat for other "user defined" commands like for example a "translate" that allows the user to directly target "EN -> ES" by typing "ES".

Bumping this as well - I see that it's not possible to do it dynamically, but would it be possible to register 5-10 commands that start out disabled and can be given aliases and mapped to searching a docset through the settings? It likely wouldn't be optimal but it work in the meantime!

I am also trying to skip the first step, so I am making some changes to the extension, add two more commands to make it happen with the help of quicklink, here is the sneak peak of the change, when I am done, I will try to publish the code to this extension:
https://github.com/raycast/extensions/assets/2911620/4cdd24a8-50d4-44bc-bb78-e5909cd0713b

@JasonBoy that's dope! sorry i never saw this because i kinda went back to Alfred for a while to let raycast mature a bit, but I'm setting it up again and this was really one of the big missing pieces for me. Thanks!


recursive without indirection

I am using this script https://gist.github.com/fitzgen/187381e358f60efa8194d0b276b4d11a.
The hashtag for my bindgen version is 4dd4ac7 .
$ ./b.sh  rust-bindgen/target/debug/bindgen  abc.h
clang-4.0: warning: treating 'c-header' input as 'c++-header' when in C++ mode, this behavior is deprecated [-Wdeprecated]
error[E0072]: recursive type `C` has infinite size
 --> /tmp/bindings-kGYvi3.rs:3:123
  |
3 | # [ repr ( C ) ] pub struct C__bindgen_vtable ( :: std :: os :: raw :: c_void ) ; # [ repr ( C ) ] # [ derive ( Debug ) ] pub struct C { pub vtable_ : * const C__bindgen_vtable , pub a : C , }
  |                                                                                ^^^^^^^^^^^^ recursive type has infinite size            --------- recursive without indirection
  |
  = help: insert indirection (e.g., a `Box`, `Rc`, or `&`) at some point to make `C` representable

error: aborting due to previous error

Interesting: bindgen emitted Rust code that won't compile!

$ cat abc.h
template < class T > struct C
{
  explicit C (const C & t):a (t)
  {
  } virtual ~ T ()
  {
  } C a;
};

Huh. I'd expect a C++ compiler to refuse to compile this code... so I wouldn't expect this code to get past libclang in the first place.
Worth investigating how to exit earlier with a proper error message rather than emitting bindings that are sure to fail to compile.

And thanks, as always, for the bug report @helloqirun :)


#199 was merged into develop branch with conflicts.

The develop branch project no longer opens in XCode due to unresolved conflicts on L922-L926 and L951-L955.
- <<<<<<< Updated upstream
				CLANG_ENABLE_MODULES = YES;
- =======
				CODE_SIGN_IDENTITY = "iPhone Developer";
- >>>>>>> Stashed changes

Fixed, thank you

Sorry, but it looks like this is still not fixed in in the develop branch. #207 will resolve this bug.


It works on Simulator but not on iPhone!

I use cocoahttpserver in my demo app for iPhone. It works well in Simulator on Mac mini. I could open the web in the browser of my PC. But when I download the app into a real iPhone, I couldn't do that. I could use safari in the same iPhone to browse the web in cocoahttpserver, but I couldn't do that on a PC. How could it work in a REAL iPhone ? Help me!

How can I create a voting link


Get TabulatedRatingCurve water level info from elsewhere

Right now our TabulatedRatingCurve node uses the connected water level on the upstream side to determine the flow from upstream to downstream.
This issue suggests to allow using a waterlevel from elsewhere in the model instead. This is a drawing of the suggested feature:

I'm not sure I like complicating the TabulatedRatingCurve like this. Using DiscreteControl and an Outlet can almost cover this behavior, and can fully cover it if we have #720.
So I think I favor #720 here.

Discuss with Martijn & Peter

closed in favor of #1019


Wording

If this is in past tense then should probably be under _generate?
Originally posted by @sed-i in https://github.com/canonical/loki-k8s-operator/pull/48#discussion_r764059578

TL;DR
We are currently saying that we haved saved, although the actions have not been performed yet. In case it errors, the operator will likely be very confused by reading the logs.


node-webkit-builder is deprecated

I get the following warning when installing ember-cli-node-webkit:

npm WARN deprecated<EMAIL_ADDRESS>WARNING: This module has been renamed to nw-builder. Install using nw-builder instead, node-webkit-builder will no longer be updated.

Perhaps it's also a good idea to rename this project to ember-cli-nw as well?

I almost did rename it! Until I discovered there was already a package on NPM called ember-cli-nw. But I can't track down its source on GitHub, and the project is very bare. At any rate, it seems that I am stuck with the node-webkit name given the conflict.
I can update the node-webkit-builder dependency to nw-builder though. =)

@olivierlesnicki, according to npmjs.org you're the creator of ember-cli-nw. Did you have plans for that project?

@jonblack This project is now named ember-cli-nwjs. I've updated it to use the nw-builder package as well.

That also works. Great!


refactor(plugin): simplify ErrorHandler

Import circle... That's the reason why this wasn't simplified before, closing this.


A value of type 'HamburgerButtonInfo' cannot be added to a collection or dictionary of type 'ObservableCollection'.

I get a [rather busy] blue squiggly lines in Shell.xaml with a [benign] error: A value of type 'HamburgerButtonInfo' cannot be added to a collection or dictionary of type 'ObservableCollection'. shown as tooltip.
Surely, HamburgerButtonInfo containers as HamburgerMenu.PrimaryButtons and HamburgerMenu.SecondaryButtons are both ObservableCollection<HamburgerButtonInfo>,  so the error makes no sense, and most importantly compiles and runs okay. Based on the error, the compiler seems to be hinting for HamburgerButtonInfo to be a UIElement rather than DependencyObject.
Does anybody know why VS2017 triggers this error?

I do not. We might run into it when we start testing Template.Hamburger.


Add compatibility with binary_sensors Times of the day sensor options

Problem
Automations to be triggered by input_datetime sensor require templating.
with a binary sensors we can set times if the day where the sensor will be on. This can however only be set in the HA setting and it's a pain to arrive to the change screen. this card would be perfect for it.
Suggested feature / solution
Allow to change the times of the day sensor options of a binary sensor.
Additional context
Aditinaly UI options could be available to display only one time (either the on time or off time, allowing to hide one of them)

after all we can trigger automations with input_datetime without templating...
closing the request


Modify FQL reconcile function to be part of the ReconciliationContext

What this PR does:
CheckFullQuery will now be part of the ReconciliationContext, while the parsing and the validation will be in the CassandraDatacenter. This allows to reuse the validation and parsing logic in the webhook and in the handler's isValid functionality (which is ran before the actual reconcile)
Which issue(s) this PR fixes:
Fixes #214
Checklist

[ ] Changes manually tested
[x] Automated Tests added/updated
[ ] Documentation added/updated
[ ] CHANGELOG.md updated (not required for documentation PRs)
[x] CLA Signed:  DataStax CLA

All of my questions have been addressed here except for one:

I can see both ValidateFQLConfig() and dc.FullQueryEnabled() are tested in webhook_test.go. My only question is whether CheckFullQueryLogging() should have some unit tests too? I'm less interested in the happy path there, but I'd like to see what happens if it receives invalid configs etc. (obviously this connects to point 1 above).

If we can add some unit tests for CheckFullQueryLogging() then I'm happy to approve 🙂

CheckFullQueryLogging is the method you created ;)

Added unit tests and renames.

I think all of those are http connection issues, I don't really see the point in testing them as that's not really logic related. The only test really is that it requeues, but even that isn't such a big deal in the end.

The tests have started failing a lot after FQL reconcile addition in the master also. There's something going on, since this is the log in canary-upgrade (it does not enable FQL):
2021-11-04T01:29:12.646Z	ERROR	controllers.CassandraDatacenter	failed to call endpoint /api/v0/ops/node/fullquerylogging	{"cassandradatacenter": "test-canary-upgrade/dc1", "requestNamespace": "test-canary-upgrade", "requestName": "dc1", "loopID": "e87dbbe5-2da6-4c95-b38c-4ba9e55f056d", "namespace": "test-canary-upgrade", "datacenterName": "dc1", "clusterName": "cluster1", "error": "incorrect status code of 500 when calling endpoint"}
github.com/k8ssandra/cass-operator/pkg/reconciliation.(*ReconciliationContext).CheckFullQueryLogging
	/workspace/pkg/reconciliation/reconcile_fql.go:37
github.com/k8ssandra/cass-operator/pkg/reconciliation.(*ReconciliationContext).ReconcileAllRacks
	/workspace/pkg/reconciliation/reconcile_racks.go:2360
github.com/k8ssandra/cass-operator/pkg/reconciliation.(*ReconciliationContext).CalculateReconciliationActions
	/workspace/pkg/reconciliation/handler.go:160
github.com/k8ssandra/cass-operator/controllers/cassandra.(*CassandraDatacenterReconciler).Reconcile
	/workspace/controllers/cassandra/cassandradatacenter_controller.go:140
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).reconcileHandler
	/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.2/pkg/internal/controller/controller.go:298
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem
	/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.2/pkg/internal/controller/controller.go:253
sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func2.2
	/go/pkg/mod/sigs.k8s.io/controller-runtime@v0.9.2/pkg/internal/controller/controller.go:214

I'm surprised it tries to call these, as these are 3.11.7 pods.

Ah, now I know.. the pod reports it supports FQL properties (newer management-api) even for 3.11.*

Ah, now I know.. the pod reports it supports FQL properties (newer management-api) even for 3.11.*

So that's intended behaviour from management API because the server does not know which Cassandra version it is actually working against. It only talks to a generic CassandraAPI not a versioned CassandraAPI4x/CassandraAPI3x. I don't particularly like that fact, but that's how the project is structured, and it is major surgery to fix it.
This is why I was asking where we call ValidateFQL - doesn't that catch this condition in handler.go here? I was under the impression that we couldn't proceed to CheckFullQueryLogging() if the config wasn't valid, but I don't fully understand how the handler works?

We can proceed to CallFullQueryLogging. There's no enabled FQL or mention in any of the 3.11.* configs, but since CallFullQueryLogging wants to verify in any case, it tries to call the pod. As such, I added a check that if it's a dse/3.11 it just exits and doesn't verify the status.

That sounds fine in theory... I just want to confirm that this line is called after this line?
I am struggling to get my head around this because you are validating in the DC controller but you are reconciling in the racks controller. I'm not sure about this 😒

Yes, the IsValid is in the beginning of the reconciling.
The IsValid is called here:
https://github.com/k8ssandra/cass-operator/blob/65e47b5ac16adfccfbb335abf9723263554fc3ee/controllers/cassandra/cassandradatacenter_controller.go#L116
But the ReconcileAllRacks process starts only from:
https://github.com/k8ssandra/cass-operator/blob/65e47b5ac16adfccfbb335abf9723263554fc3ee/controllers/cassandra/cassandradatacenter_controller.go#L140
Which then leads to:
https://github.com/k8ssandra/cass-operator/blob/65e47b5ac16adfccfbb335abf9723263554fc3ee/pkg/reconciliation/handler.go#L160

Righto. I don't love having the validation in a different place to the reconciliation. But I guess the logic is that we want to validate the resource as a whole and then proceed to call different reconcilers for its component parts, and I can understand that argument.
I just need to remember that the reconciliation is occurring serially on a single resource, rather than multiple parallel reconciliation loops being run on component parts of the resource.
Once the tests pass I'm happy to approve (again 😅)


add arb_translation with last version

Hi leanCode team,
thanks for this package
so i want to add arb_translation with last version and this happen:
Note: meta is pinned to version 1.11.0 by flutter from the flutter SDK.
See https://dart.dev/go/sdk-version-pinning for details.
Because arb_translate >=0.0.2 depends on meta ^1.12.0 and every version of flutter from sdk depends on meta 1.11.0, arb_translate >=0.0.2 is incompatible with flutter from sdk.
So, because test_translate depends on both flutter from sdk and arb_translate ^0.1.0, version solving failed.
You can try the following suggestion to make the pubspec resolve:

Consider downgrading your constraint on arb_translate: flutter pub add arb_translate:^0.0.1.

this is my flutter doctor:
[√] Flutter (Channel stable, 3.19.0, on Microsoft Windows [Version 10.0.19045.3996], locale en-US)
• Flutter version 3.19.0 on channel stable at C:\dev\flutter
• Upstream repository https://github.com/flutter/flutter.git
• Framework revision bae5e49bc2 (3 days ago), 2024-02-13 17:46:18 -0800
• Engine revision 04817c99c9
• Dart version 3.3.0
• DevTools version 2.31.1
• Pub download mirror https://pub-web.flutter-io.cn/
[√] Windows Version (Installed version of Windows is version 10 or higher)
[√] Android toolchain - develop for Android devices (Android SDK version 33.0.1)
• Android SDK at C:\Users\moata\AppData\Local\Android\Sdk
• Platform android-33-ext4, build-tools 33.0.1
• Java binary at: C:\Program Files\Android\Android Studio1\jbr\bin\java
• Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)
• All Android licenses accepted.
[√] Chrome - develop for the web
• Chrome at C:\Program Files\Google\Chrome\Application\chrome.exe
[√] Visual Studio - develop Windows apps (Visual Studio Community 2022 17.5.3)
• Visual Studio at C:\Program Files\Microsoft Visual Studio\2022\Community
• Visual Studio Community 2022 version 17.5.33516.290
• Windows 10 SDK version 10.0.22000.0
[√] Android Studio (version 2023.1)
• Android Studio at C:\Program Files\Android\Android Studio1
• Flutter plugin can be installed from:
https://plugins.jetbrains.com/plugin/9212-flutter
• Dart plugin can be installed from:
https://plugins.jetbrains.com/plugin/6351-dart
• Java version OpenJDK Runtime Environment (build 17.0.7+0-b2043.56-10550314)
[√] VS Code (version 1.86.1)
• VS Code at C:\Users\moata\AppData\Local\Programs\Microsoft VS Code
• Flutter extension can be installed from:
https://marketplace.visualstudio.com/items?itemName=Dart-Code.flutter
[√] Connected device (3 available)
• Windows (desktop) • windows • windows-x64    • Microsoft Windows [Version 10.0.19045.3996]
• Chrome (web)      • chrome  • web-javascript • Google Chrome 121.0.6167.184
• Edge (web)        • edge    • web-javascript • Microsoft Edge 121.0.2277.112
[√] Network resources
• All expected network resources are available.
• No issues found!
Process finished with exit code 0
Thanks in advance for your help !

Hi,
I assume you want to add arb_translate as a dependency in pubspec.yaml, right? This isn't currently supported. arb_translate is intended to be used as a global tool so you don't need to add it to pubspec.yaml just enable it by running:
dart pub global activate arb_translate

Great thanks 🫡


1.4: Intermittent test hang: io.helidon.webserver.TestHttpParsingDefaults

Just had a pipeline job hang on:
Tests run: 9, Failures: 0, Errors: 0, Skipped: 0, Time elapsed: 0.014 sec - in io.helidon.webserver.HashResponseHeadersTest
Running io.helidon.webserver.TestHttpParsingDefaults

This seems intermittent since other times the pipeline has passed. Hung job was at: https://builds.helidon.io/53F3E72BE46274A8B665498126A9E580
Looks like this test was added by #2244

Looks like this issue is still occurring: https://builds.helidon.io/0BD243C1F1146E2A381220C79A909663

@romain-grecourt any idea how to

increase the chance for this to happen
get more information on running tests, such as which method is being executed?

Even if I reproduce it (such as in the linked problem), I just see it was a specific class, I cannot see which test was the cause of the problem...

Failed to reproduce (300000 repeats of tests).
I will update the tests to have all the possible timeouts, so at least we find out which specific method is causing the failure

Closing this issue again. If this ever occurs again, please update and reopen.


Django 4 compatibility

I recently tried to upgrade one of my Django apps to version 4 and found django-generic-positions to be blocking. The problem can be solved easily be replacing django.utils.translation.ugettext_lazy with django.utils.translation.gettext_lazy which first is depreacted in v4 of Django.

@mpibpc-mroose Thanks for posting. I'm afraid there will be more changes, since this app hasn't been maintained for years. Anyway, feel free to come up with a PR :)


Implement defaulted parameters

We can probably make use of the fact that JS allows calling functions with any number of arguments, setting the missing ones to undefined. Note that this also has to work properly with named argument method invocation.

Implemented with commit 677abd84ed02c50a2e9cd31945aefaafec8628ac

Great!


System.NullReferenceException: Object reference not set to an instance of an object.

Microsoft PowerToys version
0.55.2
Running as admin

[X] Yes

Area(s) with issue?
General, Settings
Steps to reproduce
I don't know what to do, the following error dialog pops up
"
System.NullReferenceException: Object reference not set to an instance of an object.
at Microsoft.PowerToys.PreviewHandler.Monaco.MonacoPreviewHandlerControl.<>c__DisplayClass6_1`1.<b__3>d.MoveNext()
--- End of stack trace from previous location ---
at System.Threading.Tasks.Task.<>c.b__140_0(Object state)
"
PowerToysReport_2022-02-12-09-46-21.zip
✔ Expected Behavior
Don't show error messages anymore
❌ Actual Behavior
error dialog pops up
Other Software
No response

Duplicate of #16081

Issue is a duplicate
Thank you for your issue! We've linked your report against another issue to centralize the thread.
Thanks for helping us to make PowerToys a better piece of software.


chore: update codeowner

List of changes

update list of codeowners

Thanks for this @MSzalowski
Since we are here, can fix the errors with the testing maybe change:
@IntersectMBO/govtool-test -> @spannercode @kneerose
Do you agree @spannercode ?

@Ryun1 Sounds good.
I'd also propose to add @mesudip.


Add table oci_MYSQL_shapes

Is it possible to add the MYSQL shapes list as a table?
https://docs.oracle.com/en-us/iaas/tools/python/2.85.0/api/mysql/client/oci.mysql.MysqlaasClient.html#oci.mysql.MysqlaasClient.list_shapes
This will allow you to resolve the actual OCPU and Memory for the running MySQL instances.

Welcome to Steampipe @AnykeyNL 👍 . Thank you for raising the feature request 😄.
If anyone would like to attempt to develop this table, the API is an excellent place to start.
Please let us know in case there are questions 👍 .

I will add this information. Does it make sense to add it as a separate table? Or just extent the mysql_db_system table? Given it's only 2 extra fields it might make more sense to extend the table rather than doing an extra SQL join?

@scotti-fletcher that's a good point. I don't think e need a new table for this, since it is just an extension of MySQL database.
To add to it, since the API doesn't have pagination, I think it is better to add it in the form of new columns than tables.

@misraved I've created a PR for these extra fields
https://github.com/turbot/steampipe-plugin-oci/pull/438/commits

Resolved in https://github.com/turbot/steampipe-plugin-oci/pull/483


macOS Catalina giving OS Error: Permission denied, errno = 13 when

flutter_02.log

FileSystemException: FileSystemException: Cannot open file, path = '/flutter/clone/flaguru/.flutter-plugins' (OS Error: Permission denied, errno = 13)

Does your user have permissions to edit that file?
$ ls -la /Users/hoangduy/Desktop/University/CNTT 192A/flutter/clone/flaguru/.flutter-plugins

I did a video how to fix it hope it will help some are new with flutter like me.
Go Flutter !!!
Link : https://www.youtube.com/watch?v=L98Y0UT8wuQ

I did a video how to fix it hope it will help some are new with flutter like me.
Go Flutter !!!
Link : https://www.youtube.com/watch?v=L98Y0UT8wuQ&feature=youtu.be&fbclid=IwAR3rVC_8wNMwSh8K8xNKnde5HqtaljRFsNsnBbXT5qL1LREb_qsmEu1Xgro

That should have been fixed with https://github.com/flutter/flutter/issues/36714.
What's the output of flutter doctor -v?

In output will be Http errors, can't download from pub repo. I changed pub repo server:
export PUB_HOSTED_URL=https://pub.flutter-io.cn
then works fine

Without additional information, we are unfortunately not sure how to resolve this issue. We are therefore reluctantly going to close this bug for now. Please don't hesitate to comment on the bug if you have any more information for us; we will reopen it right away!
Thanks for your contribution.
Could everyone who still has this problem please file a new issue with the exact descriptions what happens, logs and the output of ' flutter doctor -v'  please.
All system setups can be slightly different so its always better to open new issues and reference related issues.


Added pull handles to JWT editor view

In the current version, the JWT editor is inflexible and hard to read on small screens or with larger font sizes.
This PR adds two pull handles to the JWT editor tab (as shown in Burp request/response views). The user can use these to vertically shrink the serialized text field and the signature hex view, to focus on the contents of the token.
Current cramped view:

More helpful view (note the pull handles between serial, tabview and signature):

Thanks for the PR. It looks like a really nice improvement!
Do you think that the JWE panel should have them as well?

The top divider applies whichever tab is selected (the SplitPanel which provides it is a parent of the Tab panel).
I saw no obvious place to put a divider inside the JWE tab, it seems more balanced (multiple similar fields) than the JWT panel (more commonly useful stuff followed by the large Signature field).
I may have missed something, but placing two dividers in the JWE tab would (AFAICT) lead to a hierarchy of paired panels, which probably wouldn't behave as the user would expect.

Fair enough :+1:
I'll merge it now and email PortSwigger over the weekend to kick start getting it into the BAppStore.


"Failed to execute menu action. Cannot index into null array."

Hi!
I have tried to install this extension for the first time on Playnite 10.20 (just updated to the most recent version) and I am getting a weird error and the extension doesn't seem to work at all. I just installed the extension from the add on database

Any ideas what's going on?

Hmm, I can't think of anything that would cause that error specifically. You did select one or more games before right clicking and choosing "Add to Is There Any Deal Collection"?
Is there any chance you have a game with an empty name? I found a bug related to that that seems like a similar issue. Try this update and let me know if it fixes the issue:
https://github.com/bburky/isthereanydeal-playnite/releases/tag/v1.4.1
Otherwise, let me know if there are any isthereanydeal-playnite or PowerShell errors in the extensions.log or playnite.log files.
They're in the Playnite user data directory (you can find it via About Playnite -> Other -> User data directory)

I will check the log as soon as I have the chance, but that error appeared with everyone I tried. All of them with different names. I was trying one game at a time.
I never got to the point to actually log into IT AS either, if the can spark some ideas ...

This extension has been entirely rewritten to handle the ITAD site redesign. As part of the rewrite all the code changed, this specific error should no longer apply.
Let me know if you have any issues with the new version though.


Missing label for x_4 in figure 11.11

Describe the erratum
The label x_4 is missing from figure 11.11 on page 270. Reference is made to x_4.
Version and Location
Please provide:

27th of Jan 2022
p 270

Proposed correction
Just add in the label $x_4$
Screenshots
It's fairly obvious from the figure what's missing.
Additional context
None. Loving the book so far. Great job.

Thank you for this. I've updated the figure but the book is currently in a frozen state as it undergoes copyediting. So it might take a bit for this correction to go live.


chore: release actions

chore: setup release
ci: add release actions

true

@all-contributors please add @r17x  for Infrastructure


Update README and fix file permissions.

Some updates of README.md and README_CN.md to improve clarity.
Remove the executable mode of README.md, README_CN.md, and logo.png.

 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.

LGTM, thanks for contributing!

LGTM.


Generic has wrong runtimeType if compiled to JS

How you can reproduce it:
Take this sample: http://samples.material-design-lite.pub/mdld_repeat_data-table/index.html
Open the console and click on EVGA... (price 110.12)
You can see the result of this line:
_logger.fine("Input-Value: '$val' (${val.runtimeType}) -> '${_value}' (${_value.runtimeType})");

I use OvservableProperty for setting the value (https://goo.gl/ZoVsMe)
I take the untyped input value, check against the generic runtimeType and convert if necessary. In the above sample the input value (double) 110.12 should stay the same. This works fine with the DartVM but fails if compiled to JS.
The JS-Version returns int as runtimeType for a generic defined as double.
Here is the definition (main.dart where I define final ObservableProperty<double> total: https://goo.gl/c4QTtE
If you tick the checkbox then this line sets the value https://goo.gl/plF0sX and produces the log output.
Output in Dartium/Chromium:

An here is Chrome:

This is one of those few cases where the vm and dart2js differ by design. Dart2js'  runtime types of numbers is derived from the numeric representation we get in JavaScript, where 0.0 and 0 are equivalent.
In your example, this means that an initial value of 0.0 will set initial type as int, not double.
Here is a small example that also illustrates the situation:
main() {
  var x = 1.0;
  print(x.runtimeType);
  print(x.toDouble().runtimeType);
  print(x.toInt().runtimeType);
}

This will print double double int in the VM, but int int int in dart2js.
Making as a duplicate of https://github.com/dart-lang/sdk/issues/1533


Website not converting backup correctly

Steps to reproduce
Goto your conversion url-
https://lnreader.github.io/guides/upgrade
Upload my file
lnreader_backup_2024-10-15_03_51.json
You get this file-
migrated-backup.json
Also added converted file.
Expected behavior
Expected behaviour is, the new file to be around comparable size, ime. 69kb. And contain all the novel's, not just 3.
Actual behavior
The website to convert backup from 1.1.19 to 2.0.0 is not converting my backup file correctly.
I have more than 50 novels in the back but the converted version has just 3 novels (which is visible by the size as well, 69 kb old backup file turns into 2kb new backup file).
You can reproduce it by uploading my file
LNReader version
2.0.0 beta 2
Android version
Android 14
Device
POCO X6 PRO
Other details
This is not a device or app's problem. It's the conversion tool's bug. Please email me if there is any update or resolution. Much appreciated, thanks.
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open or closed issue.
[X] I have written a short but informative title.
[X] If this is an issue with a source, I should be opening an issue in the sources repository.
[X] I have updated the app to version 2.0.0.
[X] I will fill out all of the requested information in this form.

Not all plugins that were available in the stable version are present in the beta. Either they haven't been implemented yet, or the websites they rely on no longer work.
Could you check if the novels that aren't present have their respective plugins available in the beta?

Hi, thanks for a prompt reply.
I have checked, the plugins are available, most of my novels were from the source AllNovelFull. And it has been installed. Infact i even tried to load the backup after installing all the plugins, but still it just ends after 3 novels.
You can also load my backup in old version to verify the source, it is present in the new version as well.

@rajarsheechatterjee Hey guys, are you working on this? Would love to get some updates on this. Thanks for your hardwork.

upload your backup file here or in our discrod server

Hi @nyagami , i have uploaded my backup file in the original post. Re-attaching these-
My file for old version with all 50+novels +-
lnreader_backup_2024-10-15_03_51.json
My file after using your link for new version with just 3 novels-
migrated-backup.json
Thank you for looking into this.

allnovelfull.com has changed to allnovelfull.net
I pushed the fix, reload your page and try again

Hey @nyagami , it worked. Thanks for the quick fix.


LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20210720093413668 to main

LEGO: Pull request from lego/hb_04604851-bac4-4681-9f74-73de611d6e48_20210720093413668 to main with localized lcls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.006%) to 42.555%

Totals

Change from base Build<PHONE_NUMBER>:
0.006%

Covered Lines:
26727

Relevant Lines:
56924

💛  - Coveralls


Pixels become invisible when zooming out

In the original client, they are visible, but in this client, they become invisible.
As a last resort, it is possible to limit the zoom level.

caused by server not updating tile file properly/timely


Fix: Stubs, Sandboxes and ESLint

I would also like to drop the warning for functions without explicit returning types because TS can automatically validate that.

@kewitz thanks for taking care of that!
I personally like the fact that it enforces to set the return type for functions. But I'm not sure what we're referring to here, can you provide a short example as well as arguments on why we should drop it?

@Betree mainly because TypeScript is really good at deducing the return type by itself:

Especially when you use a lot of native operators and functions, which is pretty much our case.
The exception is when you use a library that is not typed, in that case, TS wouldn't be able to deduce the output type.

@kewitz I'm ok to have it just as a warning, but I don't like the idea of removing the check entirely because explicit returns type are a good practice. They make sure that the calling code can expect a stable API from your function.
Let's take an example:
// Implements this **in a typescript file**
const myFunc = arg => {
  return Promise.resolve(42);
}

// Implements some code **in a JS file**
myFunc().then(console.log);

// Update first example
const myFunc = arg => {
  return arg ? Promise.resolve(42) : 55;
}

Now if you use explicit types Typescript will prevent you from doing the latest, because you've specified that myFunc should always return a promise. If you don't use explicit types, the type check will pass and the failure will only happen at runtime.
Also if we ever want to move to strict typescript flags in the future (which I hope we'll be able to do one day) implicit returns types are not allowed so we'll have to go back and type everything. I'd rather keep a good habit from the start.

@Betree you're right, in this mixed JS/TS context it makes sense.


Disable color for no_tty

This gem has become a standard requirement for development. So much so, that I've even used the built in color methods under the String class.
https://github.com/amazing-print/amazing_print/blob/dbac402f12fee5c2b818b8f555d9f9ac2b035c45/lib/amazing_print/core_ext/string.rb
I do an an issue through. When I'm using a color in a string "this is my string".red I need to disable color when my script runs in no tty.
How do I go about disabling colour after the amazing_print module has already been required in my program?

Hi @ddarbyson and thanks for using AmazingPrint and asking your question here!
There's two ways that colors can get set. With Ruby 2.7+ IRB supports colorization natively. This can be disabled with
IRB.conf[:USE_COLORIZE] = true

See https://docs.ruby-lang.org/en/2.7.0/IRB.html for more info
When colors are off in IRB but on in AmazingPrint, we can turn them off for a single output providing the plain: true option to ap

We see that nil has color again because we only turned off colors for formatting the string and not the return value of the ap command itself.

Hey, @HarlemSquirrel thanks for getting back to me.
I should clarify my intention. We are not using IRB and Amazing Print in the traditional sense. For us, AP has become a staple for writing programs so we require the module always.
But doing this conflicts with the String mixin you have.
https://github.com/amazing-print/amazing_print/blob/master/lib/amazing_print/core_ext/string.rb
By design, AP is injecting colour methods directly into the String class (i.e. "my string".red). You can see, if we are using AP as a staple gem, then these assumed methods actually conflict and assume too much.
For example, this conflicts with other code which handles the colour of the string. In this case, we opted to drop our previous gems which handle colour in favour of AP String mixins. I suppose that is okay, however, when using our program in none tty, there is no control over the colour.
So I would propose 2 changes

Don't assume anything should be mixed into standard Ruby classes unless specified
Allow the string.rb mixing to detect no tty and do not use colour

@ddarbyson We are doing some detection for tty here
https://github.com/amazing-print/amazing_print/blob/5a1a94086c5e6efd75aaa0b6f04eb206649e2a1d/lib/amazing_print/inspector.rb#L98-L115
I also agree that we should shouldn't mix in to standard library. For colorizing output there's the more robust rainbow gem that can add those mix ins.
@paddor what do you think?
I'm happy to review a PR to refactor how we colorize here but I don't have the time to dig in myself at the moment.

hi @HarlemSquirrel I investigated the colourize? method and I do see ENV['TERM'] control, however, it has no effect when using the colours methods provided by the String mixin in such as "mystringexample".red
Having said that, .red is not influenced by ENV['TERM'] and therefore using the colour mixins provided by AP String class still has coloured output in none tty sessions - which produces unescaped character sequences in email or log files; that's why i'm looking to turn off colour.
To reiterate, in our case, we use AP String mixin for colour, b/c there's no way of turning it off, and there's no point in introducing another gem to handle colour.
What would be ideal for AP is to introduce two improvements:

Have AP colour String mixin optional
Have AP colour String be influenced by ENV['TERM']

I agree that we shouldn't extend the String class. We could use the Rainbow gem and refine String just for AP's use like described here. Then @ddarbyson can do the same in his scripts if needed. To detect whether the TTY supports colors, use tty-color.

I really like the idea and appreciate the thought you guys have put into it. It really does solve all my issues at once and furthermore, our developments will benefit from the Rainbow gem.
I would be more than happy to test once it's ready.

See: https://github.com/amazing-print/amazing_print/pull/91


for large number of pages the plugin is not working to create pdf

for pages more than 70 the plugin is not generating pdf also the css footer properties are getting tampered.

html2pdf.js creates one single html canvas with all of the content and then converts that to a pdf.  The problem is that browsers have a limit on how big a canvas can be, so for large documents the pdf ends up just being blank.
html3pdf, which is a fork of this library and shouldn't require any changes to your code, solves this by rendering each page on its own canvas (although this is much slower).

html2pdf.js creates one single html canvas with all of the content and then converts that to a pdf.  The problem is that browsers have a limit on how big a canvas can be, so for large documents the pdf ends up just being blank.
html3pdf, which is a fork of this library and shouldn't require any changes to your code, solves this by rendering each page on its own canvas (although this is much slower).

I tried to import it but it didn't work.
It always says html2pdf is not defined.


Fail to start SSH server - permission denied

When setting up Gogs, I checked "Use Builtin SSH Server". After finishing installation, the page redirect to 502 error. This is the error in the log file:
2017/12/24 23:40:06 [TRACE] Log Mode: File (Trace)
2017/12/24 23:40:06 [ INFO] Gogs <IP_ADDRESS>2
2017/12/24 23:40:06 [ INFO] Cache Service Enabled
2017/12/24 23:40:06 [ INFO] Session Service Enabled
2017/12/24 23:40:06 [ INFO] Git Version: 2.11.0
2017/12/24 23:40:06 [ INFO] Run Mode: Production
2017/12/24 23:40:06 [TRACE] Doing: CheckRepoStats
2017/12/24 23:40:06 [ INFO] SSH server started on <IP_ADDRESS>:22
2017/12/24 23:40:06 [TRACE] SSH server cipher list: [aes128-ctr aes192-ctr aes256-ctr<EMAIL_ADDRESS>arcfour256 arcfour128]
2017/12/24 23:40:06 [FATAL] Fail to start SSH server: listen tcp <IP_ADDRESS>:22: bind: permission denied

Seems like I do not have the permission to access SSH server.  Is there anyway to solve it?

Use:
sudo setcap CAP_NET_BIND_SERVICE=+eip /path/to/gogs

Thanks, I will close this issue.


Update "tomcrypt" to v1.18.2

I've followed this guide
step by step carefully. [Yes]

I've tested this package remotely and have excluded all broken builds.
Here is the links to the Travis/AppVeyor with status "All passed":

https://ci.appveyor.com/project/t0p4/hunter/builds/19324550
https://travis-ci.com/t0p4/hunter/builds/87132697

Update tomcrypt to 1.18.2 #4

Testing:

https://ci.appveyor.com/project/ingenue/hunter/builds/19372508
https://travis-ci.org/ingenue/hunter/builds/439156918?utm_source=github_status&utm_medium=notification

Testing:

https://ci.appveyor.com/project/ingenue/hunter/builds/19374300
https://travis-ci.org/ingenue/hunter/builds/439184387?utm_source=github_status&utm_medium=notification

This pull request is a big mess now, I will close it.
Please open new pull request, follow documentation carefully and fill accurately pull request template. Thanks.


Fix up top-level array diffing

Issue: When passing in 2 arrays to merge-deep, the diffing doesnt work as expected (due to array ordering). Converting arrays to an object-wrapped array works (i believe thats the intention of the old code that was there - but these objects still look like arrays)

@markjm Can you please describe what you mean by "the diffing doesn't work as expected". What are you expecting, and what are you seeing? These details will help me test this and get it merged. Thanks!


function ends without a return statement

Seeing this build error on go 1.0.3:
/usr/lib64/go/src/pkg/github.com/masterzen/winrm/winrm/http.go:15: function ends without a return statement

I added 'return' to the end of the function (outside the if / else) and it then built successfully.

FYI, saw this issue with go 1.0.2. Not seeing this issue with go 1.4.2.


Netcdf geo

Closes #21

Last commit should be a relatively consistent saving of the features in both NetCDF and Tif with correct projections, dimensions and metadata.


Update ExpoRouter.md

Add onPress to renderItem because without it the ShowroomListItem gave the error ReferenceError: Property 'onPress' doesn't exist.

@mchisolm0 thanks for catching this and contributing the fix!


Speed up Import-Module for MicrosoftTeams Module 3.0+

Summary of the new feature / enhancement
Importing the newer version of the Teams Online PowerShell module takes a long time.
PS> Measure-Command {Import-Module microsoftteams -RequiredVersion 3.1.0}
gives results from 15 to 35 seconds in my systems (PS7.2.1, but also tested on PS5.1)
The culprit is a section of this file:
3.1.0\net472\Microsoft.Teams.ConfigAPI.Cmdlets.psm1:
  if($exportsPath) {
    Get-ChildItem -Path $exportsPath -Recurse -Include '*.ps1' -File | ForEach-Object { . $_.FullName }
    $cmdletNames = Get-ScriptCmdlet -ScriptFolder $exportsPath
    #Export-ModuleMember -Function $cmdletNames -Alias (Get-ScriptCmdlet -ScriptFolder $exportsPath -AsAlias)
  }

PowerShell has to dot load each individual export file (700) and verify its signature.
On my test systems, this section takes from 14 to 30 seconds the first time I load the module in a new shell.
By combining the export files into a single file, stripping the signatures from the file, and dot-loading that single file, I could import the whole module in less than 5 seconds.
Proposed technical implementation details (optional)
I suggest that before release, you have the build process merge the export files into 1 file, and sign it, as that should significantly speed up whenever importing the module in a new shell.

It's a good suggestion, but that module is not managed as part of this repo.
The module's page on the PowerShell gallery says its site is
https://github.com/MicrosoftDocs/office-docs-powershell/tree/master/teams

Thank you. I apologise for the inconvenience :-)


New-SSHsession : sequence contains more than one element.

Hello,
I have a server when trying to create a connection I am getting the error : New-SSHsession : sequence contains more than one element.  Till last week I was able to create the connection.
Currently able to connect through putty.
Verbose also not helping. Please advise, what else needs to be checked in order to debug this.
Able to login to the box using putty via ip and fqdn.

Was there any change on the server? What tupe of OS and version of SSH server is the server running ?
Sent from my iPhone

On Oct 16, 2018, at 7:02 AM, azam1980<EMAIL_ADDRESS>wrote:
Hello,
I have a server when trying to create a connection I am getting the error : New-SSHsession : sequence contains more than one element. Till last week I was able to create the connection.
Currently able to connect through putty.
Verbose also not helping. Please advise, what else needs to be checked in order to debug this.
Able to login to the box using putty via ip and fqdn.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub, or mute the thread.

No Change on the servers. OS is RHEL6 & 7 and on 7 - OpenSSH_7.4p1, OpenSSL 1.0.2k-fips
Posh SSH version I am using is 1.7.7

Will setup a lab machine this week to test, are you using username/pass, key, going through proxy?
Sent from my iPhone

On Oct 16, 2018, at 8:11 AM, azam1980<EMAIL_ADDRESS>wrote:
No Change on the servers. OS is RHEL6 & 7 and on 7 - OpenSSH_7.4p1, OpenSSL 1.0.2k-fips
Posh SSH version I am using is 1.7.7
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub, or mute the thread.

yes.  creating a creadential parameter using the username and password and using the parameter acceptkey.
new-sshsession -computername servername -credential $credentials -acceptkey


add note on binbreaking

Good point. I can add a note on that. Major versions = no bincompat. Minor versions = bincompat.
tsec-0.1.0 shouldn't be bincompat with tsec-0.2.0, but tsec-0.1.1 should be with 0.1.2

Isn't 0.0.1 patch, 0.1.0 minor, and 1.0.0 major? I feel I read that 0.1.0 was a major version which was odd to me.

That's semver, yes. I'm not a huge fan of it as it is. I prefer to keep the leftmost digit for redesigns / huge changes or whatever: things that would require more than minor source changes. Having to bump that number just to break bincompat in any way seems unpleasant to me.

All good, just thought if we were not going to keep semver we should note that explicitly, not merely describing how we are going to do it differently.

That's reasonable, although I think the strict definition of "semantic versioning" isn't too common in java/scala circles.

@ChristopherDavenport you have a good point actually...
We should note on this. I have the leftmost digit in my mind as the "deploy the nukes, we have a huge change now" button, rather then semver.

That's because we're scala devs, and scala 3.x.y involves a complete rewrite.
Also scalaz devs, for whom scalaz 8.x.y involves a complete rewrite :P


The automated release is failing 🚨

:rotating_light: The automated release from the 1-integrate-semantic-release branch failed. :rotating_light:
I recommend you give this issue a high priority, so other packages depending on you can benefit from your bug fixes and new features again.
You can find below the list of errors reported by semantic-release. Each one of them has to be resolved in order to automatically publish your package. I’m sure you can fix this 💪.
Errors are usually caused by a misconfiguration or an authentication problem. With each error reported below you will find explanation and guidance to help you to resolve it.
Once all the errors are resolved, semantic-release will release your package the next time you push a commit to the 1-integrate-semantic-release branch. You can also manually restart the failed CI job that runs semantic-release.
If you are not sure how to resolve this, here are some links that can help you:

Usage documentation
Frequently Asked Questions
Support channels

If those don’t help, or if this issue is reporting something you think isn’t right, you can always ask the humans behind semantic-release.

Cannot push to the Git repository.
semantic-release cannot push the version tag to the branch 1-integrate-semantic-release on the remote Git repository with URL<EMAIL_ADDRESS>This can be caused by:

a misconfiguration of the repositoryUrl option
the repository being unavailable
or missing push permission for the user configured via the Git credentials on your CI environment

Good luck with your project ✨
Your semantic-release bot :package::rocket:

:rotating_light: The automated release from the 1-integrate-semantic-release branch failed. :rotating_light:
I recommend you give this issue a high priority, so other packages depending on you can benefit from your bug fixes and new features again.
You can find below the list of errors reported by semantic-release. Each one of them has to be resolved in order to automatically publish your package. I’m sure you can fix this 💪.
Errors are usually caused by a misconfiguration or an authentication problem. With each error reported below you will find explanation and guidance to help you to resolve it.
Once all the errors are resolved, semantic-release will release your package the next time you push a commit to the 1-integrate-semantic-release branch. You can also manually restart the failed CI job that runs semantic-release.
If you are not sure how to resolve this, here are some links that can help you:

Usage documentation
Frequently Asked Questions
Support channels

If those don’t help, or if this issue is reporting something you think isn’t right, you can always ask the humans behind semantic-release.

Cannot push to the Git repository.
semantic-release cannot push the version tag to the branch 1-integrate-semantic-release on the remote Git repository with URL<EMAIL_ADDRESS>This can be caused by:

a misconfiguration of the repositoryUrl option
the repository being unavailable
or missing push permission for the user configured via the Git credentials on your CI environment

Good luck with your project ✨
Your semantic-release bot :package::rocket:


Signing key was not provided and could not be derived on google precondition

This is basically follow up for https://github.com/googleapis/google-cloud-java/issues/739.
I am following code samples here:
https://cloud.google.com/storage/docs/access-control/signing-urls-with-helpers#storage-signed-url-object-java
I am locally authenticating to Google Storage default service account and I can easily read bucket objects as follows:
    private val storage: Storage = StorageOptions
        .newBuilder()
        .setProjectId(projectId)
        .build()
        .service

    fun read() {
        val blob = storage
            .get(BlobId.of(bucket, object))

        println(String(blob.getContent()))
    }

however when I try to generate the signed upload url with:
    fun uploadUrl(objectName: String): String = storage
        .signUrl(
            BlobInfo.newBuilder(BlobId.of(bucketName, objectName)).build(),
            15,
            TimeUnit.MINUTES,
            Storage.SignUrlOption.httpMethod(HttpMethod.PUT),
            Storage.SignUrlOption.withExtHeaders(mapOf("Content-Type" to "application/octet-stream")),
            Storage.SignUrlOption.withV4Signature()
        )
        .toString()

I am getting: signing key not provided.
The tutorial above does not mention anything specific on the permissions: I am authenticated with gcloud auth application-default login with an owner user and I am normally allowed to perform any gcloud tool operations. I am not sure what is the difference here.

@knyttl
Signing a URL requires Credentials which implement ServiceAccountSigner. These can be set explicitly using the Storage.SignUrlOption.signWith(ServiceAccountSigner) option. If you don't, you could also pass a service account signer to StorageOptions, i.e. StorageOptions().newBuilder().setCredentials(ServiceAccountSignerCredentials).
In your example, neither of these options are used, which means your code only works when the credentials are defined via the environment variable GOOGLE_APPLICATION_CREDENTIALS, and those credentials are authorized to sign a URL.
See the java doc for Storage.signUrl for more details.

Does it mean I MUST use a service account? It is not possible to sign it with my user having required permissions?

Yes, as per the current implementation

Thanks for your explanation.


Review Data Persistence techniques in iOS Swift.

Read the article: https://code.tutsplus.com/tutorials/ios-from-scratch-with-swift-data-persistence-and-sandboxing-on-ios--cms-25505
You should review these techniques and decide which one will be useful for storing information about your User.

User default and how to store login data: https://www.youtube.com/watch?v=gjYAIXjpIS8

More: https://www.youtube.com/watch?v=S69sxZELJRQ


Strange Android Crash - DeadSystemException

Description

I've just got this exception from bugsnag itself. When looking at the stack trace, it seems like the crash happened from within bugsnag's code.
Would someone tell me if this is indeed from bugsnag, or something else caused it? Here's the full exception log:
java.lang.RuntimeException android.os.DeadSystemException
    ContextImpl.java:1576 android.app.ContextImpl.registerReceiverInternal
    ContextImpl.java:1528 android.app.ContextImpl.registerReceiver
    ContextImpl.java:1516 android.app.ContextImpl.registerReceiver
    ContextWrapper.java:636 android.content.ContextWrapper.registerReceiver
    Client.java:239 com.bugsnag.android.Client$3.run
    ThreadPoolExecutor.java:1167 java.util.concurrent.ThreadPoolExecutor.runWorker
    ThreadPoolExecutor.java:641 java.util.concurrent.ThreadPoolExecutor$Worker.run
    Thread.java:764 java.lang.Thread.run

Caused by: android.os.DeadSystemException
    ContextImpl.java:1576 android.app.ContextImpl.registerReceiverInternal
    ContextImpl.java:1528 android.app.ContextImpl.registerReceiver
    ContextImpl.java:1516 android.app.ContextImpl.registerReceiver
    ContextWrapper.java:636 android.content.ContextWrapper.registerReceiver
    Client.java:239 com.bugsnag.android.Client$3.run
    ThreadPoolExecutor.java:1167 java.util.concurrent.ThreadPoolExecutor.runWorker
    ThreadPoolExecutor.java:641 java.util.concurrent.ThreadPoolExecutor$Worker.run
    Thread.java:764 java.lang.Thread.run

Issue
Crash
Environment
Library versions:

"bugsnag-react-native": "2.23.2"
"react-native": "0.61.4"

Android version(s): Android 9 - SM-N960U

debug mode or production?: Prod

[X] (Android only) BugsnagReactNative.start(this) is present in the
onCreate method of your MainApplication class?

Hi @cristianoccazinsp
Thanks for the report. The DeadSystemException is documented here:
https://developer.android.com/reference/android/os/DeadSystemException

The core Android system has died and is going through a runtime restart. All running apps will be promptly killed.

So this crash isn't caused by Bugsnag. The Android system has died and the Bugsnag library likely made a call to a system process that no longer exists. The app is already in the process of being terminated by the OS.


Docs: missing description of AuthConfig in API reference

The API reference makes several references to the AuthConfig structure, but it's not described anywhere as far as I can tell.

Hi!
Please read this important information about creating issues.
If you are reporting a new issue, make sure that we do not have any duplicates already open. You can ensure this by searching the issue list for this repository. If there is a duplicate, please close your issue and add a comment to the existing issue instead.
If you suspect your issue is a bug, please edit your issue description to include the BUG REPORT INFORMATION shown below. If you fail to provide this information within 7 days, we cannot debug your issue and will close it. We will, however, reopen it if you later provide the information.
This is an automated, informational response.
Thank you.
For more information about reporting issues, see https://github.com/docker/docker/blob/master/CONTRIBUTING.md#reporting-other-issues

BUG REPORT INFORMATION
Use the commands below to provide key information from your environment:
docker version:
docker info:
uname -a:
Provide additional environment details (AWS, VirtualBox, physical, etc.):
List the steps to reproduce the issue:
1.
2.
3.
Describe the results you received:
Describe the results you expected:
Provide additional info you think is important:
----------END REPORT ---------
#ENEEDMOREINFO

@icecrime did you mean this; https://github.com/docker/docker/pull/15984/files ?

I mean this ;-) Thanks!


Inconsistency between App Store metadata and in-app privacy statement

Describe the bug
In the App Store, the app is classified as 17+ in the Age category. However, the (english) privacy statement says "aimed at people [...] at least 16 years old".
Screenshot App Store:

Screenshot privacy statement

Expected behaviour
But should be consistent.

Steps to reproduce the issue
Compare App Store with the (english) privacy statement in the app.

Technical details

Host Machine OS (Windows/Linux/Mac): n/a

There is simply no 16+ category in the App Store. Please see https://help.apple.com/app-store-connect/#/dev269f11291 for details.
Mit freundlichen Grüßen/Best regards,
SW
Corona Warn-App Open Source Team

Good catch, thank you 👍


autocomplete std macros?

A new feature idea :-)

support for autocompletion of (std) macros http://doc.rust-lang.org/std/#macros
support for find-definition of (std) macros?

I assume this would actually need to happen in the https://github.com/phildawes/racer codebase. We're just a front end for it.

Of course - I mixed it up. ;-)
-> moved to https://github.com/phildawes/racer/issues/310


Fix bookmark

Baby :baby: bookmark fix.

@guardrex  thanks!
The wrong case version still works tho - right?

@Rick-Anderson

The wrong case version still works tho - right?

It might be valid if that's the only anchor with that name on the page. It looks like if there are two with the same name that it's illegal (e.g., we couldn't have "...#Kestrel" and "#kEstrel" on the same page).
Spec:

Anchor names that differ only in case may not appear in the same document.

... and they note that ...

[If] the user agent is not defined ... some user agents may (incorrectly) consider this a match and others may not.

Anywho ... it will stop the CI from throwing a build warning. That's what tipped me off to it.


virtualScroll flashes "wrong" items and in inverse order on init/reinit

Ionic version:  (check one with "x")
[ X] 3.x
I'm submitting a ...  (check one with "x")
[X ] bug report
Current behavior:
A virtualScroll list will flash "wrong" items (and in inverse order) - before showing correctly.
This is especially obvious on (slower) devices (iphone 5s - not really slow but..)
Expected behavior:
The virtualScroll list is not shown until it is in correct order/state, it should not flash "wrong" items, nor in inverse order.
Steps to reproduce:
The plunker is fully up to date. - click the button to empty and refill the list("reinit" the virtualScroll).
the [hidden] does not effect the behaviour nor does the console.log in virtualTrackBy.
use chrome dev tools -> performance -> to throttle cpu 10x-20x and see that the flash is in inverse order.
http://plnkr.co/edit/VyvZo6iJgYK6exyrPcAe?p=preview
Other information:
This is a blocking bug for using virtualScroll.
Also notice that the virtualTrackBy is called 3 separate times (times items.length), might not be optimal.

Thanks for using Ionic, we will look into this.

fwiw, I was able to use https://github.com/rintoj/angular2-virtual-scroll - add to app module and then do the following, appears to work great, and as expected..
<ion-content no-bounce>
  <virtual-scroll
    [hidden]="items.length"
    [items]="items"
    (update)="scrollItems = $event"
    style="height: 100%; display: block"  ion-fixed>
    <div *ngFor="let item of scrollItems;trackBy: item?.id;let i = index" style="width: 100%">
      <ion-item style="width: 100%" (click)="clickItem(item, i)" >
        {{item.id}}
      </ion-item>
    </div>
  </virtual-scroll>
</ion-content>

+1

@petermm I tried using angular2-virtual-scroll but couldn't get it to work with Ionic. It seems like the list that it sends from the update function doesn't contain any items. Which version are you using?

+1


command not found

/usr/local/bin/cmdjudge: line 86: timeout: command not found
i dont know how to fix this, can you help me

i think this will solve your problem:
http://superuser.com/questions/748183/get-timeout-command-from-gnu-coreutils-8-on-rhel-5-7-machine


DBZ-8055 Cap LogMiner range to min open redo thread last scn

https://issues.redhat.com/browse/DBZ-8055
Premise
We have identified cases where on Oracle RAC, due to each node flushing LGWR buffers independently, we cannot rely precisely on the existing LogMiner SCN range calculation.  In short, the range's endScn needs to be capped based on the minimum LAST_REDO_CHANGE# in V$THREAD, which represents the data of the last redo thread flush.  This cap is only applied to open redo threads, any threads that are disabled or closed will not factor into the upper bound cap.

/packit test --labels oracle

/packit test --labels oracle

The Oracle 19 XStream failure has been recorded in DBZ-8102, and a PR has already been opened to make it retriable.

@Naros Applied, thanks! This part of the code becomes a small expert system :-).


ListView Scene behind drawer is clicked by touch. I want it blocked.

I'm using the option, type="overlay". When a drawer is opened and I clicked the offset area, the scene behind drawer is clicked by touch. I want it to be blocked.
What option dose make it possible?
Most applications close the drawer when the offset area(drawer outside) is touched.
That's what I want. Please give me any advice.
For your reference, the scene contains  component, and  contains   component.

short term solution: set captureGestures=true and make sure the panCloseMask is sufficiently large.
that said I will be cutting a release with a overlay this week that will effectively accomplish this out of the box.

Thank you, @rt2zz. Your temporary suggestion does work well to block clicking out of box.
I appreciate for your next release in advance.

@changminc just released v1.12.0 which has an overlay that catches taps when type='overlay'


[Release] Release Request for v0.0.1

Release Version
0.0.1
Target Environment
development

/approve


we need more demo for doc,docx

Why is there no problem with PDF, but doc and docx report errors?
this is my code :demo.tsx
We need more code demo supporting more file types(doc,docx).

@chinyifei unfortunately, doc and docx rendering work only for public URLs as a file sources.


enhance test container entrypoint script behavior

Adds cli parsing to the entrypoint script allowing for:

Custom path to micro-osd script
This allows easier testing of the script itself or quick
and dirty customization for special one-off tests
Selecting tests to run (or "ALL" or "NONE")
This allows the user to run only a subset of the tests as needed.
Option to pause indefinitely after tests complete
This keeps the containerized environment alive after tests
have run for debugging or just for creating a quick and dirty
ceph environment for playing around.

PR updated. PTAL


add var to get the correct NodeGroup

Issue #, if available:
None
Description of changes:
Fails when populating the PROFILE var if the NodeGroup is not 0. Adding the correct nodegroup id (last found by eksctl) in order to populate the PROFILE var
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

reverts in e7b89db


Implement partial proofs

Builds on #144 : Only the latest commit is releveant.  Will rebase the diff once that PR is merged.
What was wrong?
#144 implements a ProofTree with a minimal implementation of the actual Proof object.  We however need a full featured Proof object that can be used to perform high level operations on proofs.
How was it fixed?
This PR fills out the Proof object API and adds tests for it.  Proof objects are the core "unit" that we use to work with proofs, with high level APIs for validating proofs, constructing partial proofs, and reading data from proofs.
Cute Animal Picture

@marcgarreau this has been rebased.

whoops, missed this one before reviewing a couple that came after. will start here this morning.

@marcgarreau and @lithp this now builds from #175
The partial proof construction is now different enough that it deserves a second pass.  I was able to greatly reduce the size of Proof.to_partial by building on the various APIs from #175

This has been rebased and is ready for :eyes:

@lithp lemme know if you find anything you'd like me to cleanup post-merge.


Volume creation code is probably racy

Description
While poking // tests, I did hit this:
level=fatal msg="mkdir /home/dmp.linux/.local/share/nerdctl/1935db59/volumes/nerdctl-test/nerdctl-testvolumeremove: file exists"

We should have a look at volume creation code and figure out if it is locking properly or if the test for existence can be raced out.
Steps to reproduce the issue
na
Describe the results you received and expected
na
What version of nerdctl are you using?
1.7.6
Are you using a variant of nerdctl? (e.g., Rancher Desktop)
None
Host information
No response

Also differences with Docker
docker volume create foo
docker volume create foo
nerdctl volume create foo
nerdctl volume create foo

Create is just fine - code looks good. The error is misleading though, and also the behavior is not the same as Docker.
Will fix.


ERR_CANNOT_CONNECT - fresh addon setup

Is there some configuration that needs to be done that I am not aware of?
Installed the addon and with the latest version it now loads the UI but in the bottom left I get the error above.
I do see in the AddOn logs these messages:
Starting Fusion...
listening on port 5050
env HASS_URL undefined
env ADDON true
No existing file found for ./data/configuration.yaml
No existing file found for ./data/dashboard.yaml

I am getting the same in the logs. There is no new folder created in the custom_components. I have installed using the addon method.

I get this message on my desktop (Mac Studio) with Firefox.
On my Macbook with also Firefox it works however.
I deleted cookies on the desktop but to no avail.

I need more info

I just updated to the latest and it's working now (no error and my devices are showing).
Thanks!

Don't think this should've been closed... I'm trying to set up ha-fusion with Docker and keep getting this same error:

Here are my docker logs:
2024-01-16T16:34:02.989324248Z [HPM] Proxy created: /  -> http://<IP_ADDRESS>:8123
2024-01-16T16:34:02.994219570Z listening on port 5050
2024-01-16T16:34:02.994369608Z env HASS_URL http://<IP_ADDRESS>:8123
2024-01-16T16:34:02.994478011Z env ADDON false
2024-01-16T16:34:40.380370860Z No existing file found for ./data/configuration.yaml
2024-01-16T16:34:40.380565268Z No existing file found for ./data/dashboard.yaml
2024-01-16T16:34:46.329289740Z No existing file found for ./data/configuration.yaml
2024-01-16T16:34:46.329467309Z No existing file found for ./data/dashboard.yaml

Here's my docker-compose file:
  ha-fusion:
    container_name: ha-fusion
    image: ghcr.io/matt8707/ha-fusion
    volumes:
      - ${DOCKER_PATH}/ha-fusion:/app/data
    network_mode: bridge
    ports:
      - 5050:5050
    environment:
      TZ: ${TZ}
      HASS_URL: http://<IP_ADDRESS>:8123
    restart: always

I checked the permissions for the app directory on my host, and made sure that I have read-write access.
josh@myserver:~/docker/ha-fusion$ ll
total 8
drwxr-xr-x  2 josh josh 4096 Jan 16 10:33 ./
drwxrwxrwx 48 josh josh 4096 Jan 16 10:22 ../

Am I missing anything? I tried manually copying the example configuration and dashboard files from the GitHub repo into that app directory and restarting the container, but I still get a ERR_CANNOT_CONNECT error. (Logging in to the Home Assistant login screen on app startup worked without an issue.)

https://github.com/matt8707/ha-fusion/issues/180


WIP 🌱 pkg/virtual/apiexport: add permission claims authorizer

Summary
This adds a permission claims authorizer which respects a permission claim's resource selector.
It partially backports functionality from https://github.com/kcp-dev/kcp/pull/2089 (reverse permission claims)
/cc @nrb
TODOS:

[ ] Add e2e tests

Related issue(s)
https://github.com/kcp-dev/kcp/pull/2456

/close

this has to be done using admission


Organization notification API

API to support the following: an organization owner can push notifications to organization users so that the notifications appear in the notification icon in the main dashboard for organization users.
This is the UI that must be supported:

Tentative API:

[ ] Creation: POST /api/v2/organization/:id_or_name/notifications. The notification has:

a body with 140 characters. Markdown characters should not count for this.
a builders/viewers/all picker.

[ ] Retrieval: GET /api/v2/organization/:id_or_name/notifications. The UI will always request the 12 last ones, and backend should default to page 1, size 12, but we can build a normal paged endpoint, as it's straighforward.
[ ] Deletion: DELETE /api/v2/organization/:id_or_name/notifications/:id. It removes the notification both from admin panel and user notifications.

PUT is not needed, as notifications can't be edited. After talking to @asimcox about the "resend" feature, it looks like it just loads the notification body at the input text, to ease make sending the same text again.
Note: I suggest that the retrieval URL is only based on the user because, although the feature is currently at the scope of the organization, there's no need for an organization-based URL.

@asimcox I just updated this. @xavijam , you'll probably want to have a quick look at it.


Remove MeshCat and fix test/optim/test_lazy_ellipsoids_abstraction.jl

@blegat @JulienCalbert New bug with the Symbolic expression of test/optim/test_lazy_ellipsoids_abstraction.jl
I've implemented an easy fix (see 01abba3408e3d56ae30fdc7989ebd0fc692cc29f) but maybe we should fix the problem at the origin
cc @DeligneS

Resolving package versions... is running indefinitely
Any idea why? I've observed similar behaviours on my machine...
cc @blegat @JulienCalbert

On my machine sometimes it's also running during a very long time (like 30-60min), and if it happens, it always end up with an error about package compatibilities.

@DeligneS Already 5 hours haha

Let's remove the parts of BipedRobot/Project.toml that makes Pkg trip so that we can merge the rest of the PR and move that part to a separate PR. I created a branch https://github.com/dionysos-dev/Dionysos.jl/tree/pkg_stuck so that the Pkg bug is easy to reproduce independently of this branch

@blegat Now BipedRobot tests passed when removing ModelingToolkit from Dionysos.jl/Project.toml


onRegionChangeComplete() callback is called infinitely

https://github.com/react-native-maps/react-native-maps/issues/846#issuecomment-1210079461

Does any other open PR do the same thing?

(please answer here)
No
What issue is this PR fixing?
(please link the issue here)
https://github.com/react-native-maps/react-native-maps/issues/846
How did you test this PR?

(please answer here)
PR is for adding more info on the readme file.

Is it worth noting that this fix only works if you're using Google Maps? isGesture isn't available on Apple Maps.

Is it worth noting that this fix only works if you're using Google Maps? isGesture isn't available on Apple Maps.

You are right. I added this note. Thanks.


Network Interfaces - Get Effective Route Table is misleading

By looking at the docs here, it seems that this api should return a route table, but instead it returns a list of EffectiveRoutes. I think the functionality is fine, but to me the naming is misleading. The documentation even furthers me thinking that it should be returning a route table (which is an existing, valid Azure construct): "Gets all route tables applied to a network interface."
Maybe something like "Get Effective Routes" instead?
I've opened this issue in the python api.

Thanks for this feedback - the SDN team will take a look!


Redstone Flux Crash

The log says you are using the old API, I'm using 1.12.1 and the latest version of forge.

https://pastebin.com/h01y0AcG
Here is the crash report I got for this error. Recommend requiring the API as a dependency rather than packing it like the days of old now that they're updating again.


Poor results obtained for evaluation on Medical Segmentation Decathlon Task07_Pancreas dataset

Hello,
Thank you for your interesting work.
I'm following the instructions you provided for evaluating your checkpoint on the Medical Segmentation Decathlon Task07_Pancreas dataset.
However, the mean Dice I obtained for the pancreas and tumor segmentation are 35.8 and 35.6 respectively, which is quite low.
Are these results within the expected range? Do you have any suggestions on how to use your provided checkpoint and code to get better results?
Thank you.

Hi, harpergith
Thank you for the feedback!
We have not yet conducted an evaluation on MSD07, but this result is also lower than I expected.
One suggestion is that our evaluation setup is at 1.5mm spacing, which is somewhat different from MSD07. Resampling MSD's data to (1.5, 1.5, 1.5) is expected to improve performance.
Looking forward to you sharing more details of the evaluation. We will also test on MSD07 when we have free time, and we can discuss it in the future.


moveit_servo package executabe name has changed

The moveit_servo package executable 'servo_node_main' was renamed to 'servo_node'.

@fdurchdewald could we also add the change for the ee_frame parameter?
diff --git a/ur_moveit_config/config/ur_servo.yaml b/ur_moveit_config/config/ur_servo.yaml
index 9802702..767481a 100644
--- a/ur_moveit_config/config/ur_servo.yaml
+++ b/ur_moveit_config/config/ur_servo.yaml
@@ -40,7 +40,7 @@ move_group_name:  ur_manipulator  # Often 'manipulator' or 'arm'
 planning_frame: base_link  # The MoveIt planning frame. Often 'base_link' or 'world'

 ## Other frames
-ee_frame_name: tool0  # The name of the end effector link, used to return the EE pose
+ee_frame: tool0  # The name of the end effector link, used to return the EE pose
 robot_link_command_frame:  tool0  # commands must be given in the frame of a robot link. Usually either the base or end effector

Otherwise the moveit_servo node also doesn't start.

@Mergifyio backport Iron

@Mergifyio backport iron


fix(shared-data): Update tip overlap

All the flex multi pipettes have different tip overlaps than we thought, so update them and also add entries for filter tipracks since they would otherwise use the default of 10.5mm which is definitely wrong.


Overworked Computer?

Does anyone else's computer seem like it's working REALLY hard when you're using atom? My computer rarely turns it's fan on when I'm working on a hard surface unless I'm doing something that takes a ton of power like editing video or something. My battery settings say "atom is using a significant amount of energy" and I don't care, my computer is plugged in, I just don't want it to like.. explode haha.

@zkoostra
Do you have many tabs open within Atom, by any chance? Sometimes I get a buildup of tabs from assignment to assignment myself

@zkoostra I can tell atom works my computer intensely, but it seems to be managing it well. I'd recommend keeping any other programs closed when you've got atom open. Your computer won't explode, but you might experience lag if you've got a lot of other programs running simultaniously.

Another thing to try is an alternate text editor. I have been playing around with Microsoft’s Visual Studio Code, and it seems as though it may be less processor intensive than Atom. Consider giving it a try.

Also try Command+option+escape to clear up other background programs.

Minimizing the strain on your processor by only using the programs you need up at the moment is always a good option. When working for the class i only have Chrome, Atom, and GitHub open to minimize the workload for my PC.


fix #464 by removing code to filter duplicates

This PR removes the code previously responsible for filtering out duplicate album names, which had caused #464.
I have decided to not any additional text to signal the difference between the albums with the same name as:

That might cause confusion as the the actual name of the album
There is in my opinion no problem with having 2 entries with the same name
The official Spotify client also handles this situation in this manner

This PR however doesn't entirely fix the bug described in the issue, as it seems that this is an upstream issue. The Spotify API doesn't return both copies of the album "Revelations" by "Audioslave" and accordingly the Spotify Client has the exact same behavior as spotfiy_player. But now the two versions of Original Fire show up as they should.
Spotify response sample:
jq '.items[].name' response.json
"Revelations"
"Out of Exile"
"Audioslave"
"Original Fire"
"Original Fire"
"Doesn't Remind Me"
"Sessions @AOL Music - EP (Live)"
"Your Time Has Come"
"Like A Stone (Live from New York City)"
"Be Yourself"
"Audioslave Live on Hollywood Blvd."
"Show Me How To Live"
"Cochise"
"Chris Cornell (Deluxe Edition)"
"Live 8 (Live, July 2005)"
"Rock Clássico"
"90s Grunge"
"Canciones De Hace Un Tiempo - Rock"
"20 Top Rock Hits"
"Summer Rocks Best Buy Version"
"Miami Ibiza St. Tropez, Selected Housetunes Vol. 1"

I have decided to not any additional text to signal the difference between the albums with the same name as:
That might cause confusion as the the actual name of the album
There is in my opinion no problem with having 2 entries with the same name
The official Spotify client also handles this situation in this manner

Maybe you can add album_type and render it in UI to better differentiate between albums of the same name. Official Spotify app does this.
Another approach would be updating the UI to render the full date instead of just year

Here are the two options you suggested:

This one doesn't fix the issue of ambiguity you addressed.

This is, in my opinion, too much detail / text if you are just looking for an album.
+ two other options

This doesn't directly show the exact distinction between the two albums, but does at least show that there is some kind of difference.

Another possible solution would be to display the number of songs in an album. This is probably the most preferable solution, as it would in most cases signal a clear difference between the different versions without adding too much info or text that might not be useful to the end user just looking for an album.
I'll work on implementing option 4, unless you raise any objections with it.

Here are the two options you suggested:

This one doesn't fix the issue of ambiguity you addressed.

This is, in my opinion, too much detail / text if you are just looking for an album.

two other options

This doesn't directly show the exact distinction between the two albums, but does at least show that there is some kind of difference.
Another possible solution would be to display the number of songs in an album. This is probably the most preferable solution, as it would in most cases signal a clear difference between the different versions without adding too much info or text that might not be useful to the end user just looking for an album.
The only problem I have with 4 is that SimplifiedAlbum doesn't contain tracks.

Thanks for the investigation. Really helpful. I would probably go with 2 as it might also help with filtering in case you want to find specific album.


Improve User validation error messages

This PR moves the remaining validation error strings from the User class into the en.yml locale file. This should make it easier to see at a glance the differences in style between the User validation error messages (i.e. some are of the form "Enter an email" and others are of the form "Email can't contain non-ascii characters").
Summary of commits:

Use assert_includes when checking validation error messages in users_test.rb
Correct a mistake in the validation error when User#reason_for_2sv_exemption is present but shouldn't be
Add tests for 3 validation error messages that weren't tested
Move remaining validation error messages from strings in User to the en.yml locale file

I'm probably being dim, but I don't understand how Rails is distinguishing between validation messages of the form "Enter an email" and those of the form "Email can't contain non-ascii characters". In the former case, it doesn't need to prefix the message with the humanized attribute name, but in the latter it does. How does it tell the difference?

Rails isn't distinguishing. We have to do that in the templates where these messages are being displayed, e.g. 1eb45b8ce50223816ff7ebbe77d997310a1b2576. I mentioned in standup today that we're now in a bit of an inconsistent state with error messages because some will require us to use full_message (i.e. attribute + message) and others will require us to use message. Joseph suggested that we might want to change them all in one go so that they're in the new format but that will have to wait. Having typed that response I realise that I don't understand where this new format has come from and whether it justifies the work required to make everything consistent.

I'm probably being dim, but I don't understand how Rails is distinguishing between validation messages of the form "Enter an email" and those of the form "Email can't contain non-ascii characters". In the former case, it doesn't need to prefix the message with the humanized attribute name, but in the latter it does. How does it tell the difference?

Rails isn't distinguishing. We have to do that in the templates where these messages are being displayed, e.g. 1eb45b8. I mentioned in standup today that we're now in a bit of an inconsistent state with error messages because some will require us to use full_message (i.e. attribute + message) and others will require us to use message. Joseph suggested that we might want to change them all in one go so that they're in the new format but that will have to wait.

Thanks for explaining - that makes more sense now.

Having typed that response I realise that I don't understand where this new format has come from and whether it justifies the work required to make everything consistent.

I suspect it comes from this documenation from the GOV.UK Design System.

Hi @floehopper - Is there anything else you think I should do before this can be approved/merged?

Hi @floehopper - Is there anything else you think I should do before this can be approved/merged?

I don't think so - I'll mark it as approved now.

This PR was an attempt to fix the inconsistencies introduced by changing some of the User validation error messages in 1eb45b8ce50223816ff7ebbe77d997310a1b2576.
The intention was to:

move all User validation error messages to the en.yml locale file
make each message self-contained (so that it didn't require the attribute to be prefixed to the error for it to make sense)
change the format of full_message so that it didn't include the attribute prefix
update the templates so that they used full_message everywhere

I think it's probably fairly close to being complete but I've run out of time so have instead ended up reverting the change that introduced the problem (see PR #2736).
Closing.


rpmsg_sockif: add rpmsg_socket_ept_release for free conn

Note: Please adhere to Contributing Guidelines.
Summary
Fix possible use-after-free issues when closing
#0 0x6f9bf9 in rpmsg_socket_ns_unbind rpmsg/rpmsg_sockif.c:445
#1 0x645660 in rpmsg_virtio_ns_callback open-amp/lib/rpmsg/rpmsg_virtio.c:674
#2 0x6449e0 in rpmsg_virtio_rx_callback open-amp/lib/rpmsg/rpmsg_virtio.c:601
#3 0x64fa61 in virtqueue_notification open-amp/lib/virtio/virtqueue.c:711
#4 0x63bd4b in rproc_virtio_notified open-amp/lib/remoteproc/remoteproc_virtio.c:433
#5 0x637205 in remoteproc_get_notification open-amp/lib/remoteproc/remoteproc.c:1006
#6 0x55940c in rptun_worker rptun/rptun.c:329
#7 0x55982d in rptun_thread rptun/rptun.c:370
#8 0x52ba44 in nxtask_start task/task_start.c:129
#9 0x59e8b4 in pre_start sim/sim_initialstate.c:52

0xf3f03ca4 is located 100 bytes inside of 308-byte region [0xf3f03c40,0xf3f03d74)
freed by thread T0 here:
#0 0xf7aa6414 in __interceptor_free ../../../../../src/libsanitizer/asan/asan_malloc_linux.cc:122
#1 0x732f9c in host_free sim/posix/sim_hostmemory.c:197
#2 0x5a0bc7 in mm_free sim/sim_heap.c:230
#3 0x59b708 in free umm_heap/umm_free.c:49
#4 0x6f7243 in rpmsg_socket_free rpmsg/rpmsg_sockif.c:266
#5 0x7025c8 in rpmsg_socket_close rpmsg/rpmsg_sockif.c:1360
#6 0x6f0eb8 in psock_close socket/net_close.c:102
#7 0x6036eb in sock_file_close socket/socket.c:115
#8 0x5ef1d6 in file_close vfs/fs_close.c:74
#9 0x5ecccd in nx_close_from_tcb inode/fs_files.c:670
#10 0x5ecd65 in nx_close inode/fs_files.c:697
#11 0x5ecd90 in close inode/fs_files.c:735
Impact
none
Testing
sim/qemu

[Experimental Bot, please feedback here]
NuttX PR Requirements Review
This PR does not meet the NuttX requirements. Here's why:

Summary:

The summary is too brief. It only mentions the problem ("use-after-free issues when closing") without explaining the cause or the solution.
It lacks details about the functional part of the code being changed.
It doesn't mention related issues or PRs.

Impact:

While stating "none", it's unlikely a bug fix has zero impact.  Be specific about what aspects remain unaffected.

Testing:

Insufficient Detail:  Simply stating "sim/qemu" isn't descriptive enough. Specify:

Host OS and versions used
QEMU version
Specific NuttX configuration for the simulation

Missing Logs:  The "Testing logs before change" and "Testing logs after change" sections are empty.  Provide concrete evidence of the issue before the fix and the successful resolution after.

To improve this PR:

Expand the Summary:

Clearly state the root cause of the use-after-free issue.
Describe the specific code changes made to address the problem.
Link any related NuttX issues this PR addresses.

Elaborate on Impact:

Even if no user-facing changes occur, explain why.
If internal code structure changed, mention potential future impacts.

Provide Detailed Testing Information:

List all host and target environments used for testing.
Include relevant snippets from logs demonstrating the issue and its resolution.

By providing a comprehensive summary, outlining the impact, and including detailed testing information, you'll make it easier for reviewers to understand and approve your PR.


Setup default build settings based on detected vespa build platform using make.

@geirst : please review
@aressem : FYI

I think it is better for @aressem to review this


Re-Think Development containers concept.

Hi, Thanks for this project first of all. Is collaboration features of Cloud9 is still available in the latest taisun? Because as I can see in wiki there will be a set user prompt will be displayed while launching Cloud9 but on the latest build it just directly get's into the IDE
How can I enable collaboration features in Cloud9?

No I killed that off and use a c9 core built specifically for Alpine linux with muslc.
The modfications I made to the previous version can be seen here:
https://github.com/thelamer/core/commit/ab79cfb769249d19e6d0443226a814a49dfabb36
This project has been solely developed by me as a side project, I am not sure there will ever be groups of developers working on it.

Ok, I'll try to build my own c9sdk image with your commit later.

What about VS Code server?

Strange that its not part of the 'Stack' or something, since you do have your own Docker image already: https://hub.docker.com/r/linuxserver/code-server

I think I misunderstand the issue ,  I thought we were talking about the cloud9 container this repo builds for developing Taisun (what I personally use)
As far as the development containers launched by this template https://github.com/Taisun-Docker/taisun/blob/master/public/taisuntemplates/taisundeveloper.yml this concept existed before I ported the cloud9 concept into a Linuxserver.io container. (Even used to have a Desktop container option with a bunch of IDEs and programming envs installed but it was super heavy)
A key thing to keep in mind is I made this application before becoming a member of the Linuxserver.io team and many of the helper containers that came as Taisun features have since been fully ported to Linuxserver.io.
This template can absolutely be modified to launch Code server, or even another recent port Pylon https://github.com/linuxserver/docker-pylon . But optimally the sub menu would just be ripped out and people would upload stacks for all this stuff (code server cloud9 etc etc) to https://stacks.taisun.io as this dedicated menu does not offer the advantages it once did now that it is using.
I am going to re-title this issue and think about a more universal solution for people looking to launch dev stuff in Docker.

I am gonna add the templates and launching for  code-server pylon and cloud9 also update the existing template to support auth variables.
Sorry for the delay, focus was elsewhere for a bit, should be in the next release this week.

Looking forward towards the next update.

Pylon and Code-Server have been added to v0.5.0 just released.


Update interpolation.py

Quick plug in for interpolating lower bound, fixes #971
Please ensure your pull request adheres to the following guidelines:

[ ] Tests for new functionality/models or Tests to reproduce the bug-fix in code.
[ ] Updated documentation of features that add new functionality.
[ ] Update CHANGELOG.md with major/minor changes.

Codecov Report

Merging #972 (0ac63d2) into master (449f40f) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master     #972   +/-   ##
=======================================
  Coverage   71.84%   71.85%
=======================================
  Files          62       62
  Lines        9204     9205    +1
=======================================
+ Hits         6613     6614    +1
  Misses       2591     2591

Impacted Files
Coverage Δ

HARK/interpolation.py
41.09% <100.00%> (+0.03%)
:arrow_up:

Continue to review full report at Codecov.

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


Swagger adds to a spec not exported parameters fields

Problem statement
Swagger adds not exported parameters fields to spec, I believe that should not happen.
Swagger specification
2.0
Steps to reproduce
Example structure with not exported fields which are later added to spec
// Example
// swagger:parameters example
type Example struct {
	// in: query
	Foo  string
	bar   string
}

as a result in a spec, you will find a bar param which should not be there.
Environment
swagger version: master branch
go version: 1.13.5
OS: Fedora 31

/cc @casualjim


Remove committed temporary files

These files seem to have been accidentally committed. They were presumably created at some point as part of a deployment to PyPI.

Thanks @meshy!

My pleasure! Thanks for the merge :)