Failed assertion: line 2666 pos 12: '!_debugDisposed': is not true.

I met the following error while I was using flutter.
======== Exception caught by foundation library ====================================================
The following assertion was thrown while dispatching notifications for FixedExtentScrollController:
'package:flutter/src/rendering/object.dart': Failed assertion: line 2666 pos 12: '!_debugDisposed': is not true.
Either the assertion indicates an error in the framework itself, or we should provide substantially more information in this error message to help you determine and fix the underlying cause.
In either case, please report this assertion by filing a bug on GitHub:
https://github.com/flutter/flutter/issues/new?template=2_bug.md
When the exception was thrown, this was the stack:
#2      RenderObject.markNeedsSemanticsUpdate (package:flutter/src/rendering/object.dart:2666:12)
#3      _RenderCupertinoPickerSemantics._handleScrollUpdate (package:flutter/src/cupertino/picker.dart:473:5)
#4      ChangeNotifier.notifyListeners (package:flutter/src/foundation/change_notifier.dart:308:24)
#5      ChangeNotifier.notifyListeners (package:flutter/src/foundation/change_notifier.dart:308:24)
#6      ScrollPosition.notifyListeners (package:flutter/src/widgets/scroll_position.dart:968:11)
#7      ScrollPosition.setPixels (package:flutter/src/widgets/scroll_position.dart:278:9)
#8      ScrollPositionWithSingleContext.setPixels (package:flutter/src/widgets/scroll_position_with_single_context.dart:82:18)
#9      ScrollPositionWithSingleContext.applyUserOffset (package:flutter/src/widgets/scroll_position_with_single_context.dart:124:5)
#10     ScrollDragController.update (package:flutter/src/widgets/scroll_activity.dart:387:14)
#11     ScrollableState._handleDragUpdate (package:flutter/src/widgets/scrollable.dart:647:12)
#12     DragGestureRecognizer._checkUpdate. (package:flutter/src/gestures/monodrag.dart:449:55)
#13     GestureRecognizer.invokeCallback (package:flutter/src/gestures/recognizer.dart:193:24)
#14     DragGestureRecognizer._checkUpdate (package:flutter/src/gestures/monodrag.dart:449:7)
#15     DragGestureRecognizer.handleEvent (package:flutter/src/gestures/monodrag.dart:298:9)
#16     PointerRouter._dispatch (package:flutter/src/gestures/pointer_router.dart:94:12)
#17     PointerRouter._dispatchEventToRoutes. (package:flutter/src/gestures/pointer_router.dart:139:9)
#18     _LinkedHashMapMixin.forEach (dart:collection-patch/compact_hash.dart:400:8)
#19     PointerRouter._dispatchEventToRoutes (package:flutter/src/gestures/pointer_router.dart:137:18)
#20     PointerRouter.route (package:flutter/src/gestures/pointer_router.dart:123:7)
#21     GestureBinding.handleEvent (package:flutter/src/gestures/binding.dart:440:19)
#22     GestureBinding.dispatchEvent (package:flutter/src/gestures/binding.dart:420:22)
#23     RendererBinding.dispatchEvent (package:flutter/src/rendering/binding.dart:278:11)
#24     GestureBinding._handlePointerEventImmediately (package:flutter/src/gestures/binding.dart:374:7)
#25     GestureBinding.handlePointerEvent (package:flutter/src/gestures/binding.dart:338:5)
#26     GestureBinding._flushPointerEventQueue (package:flutter/src/gestures/binding.dart:296:7)
#27     GestureBinding._handlePointerDataPacket (package:flutter/src/gestures/binding.dart:279:7)
#31     _invoke1 (dart:ui/hooks.dart:185:10)
#32     PlatformDispatcher._dispatchPointerDataPacket (dart:ui/platform_dispatcher.dart:293:7)
#33     _dispatchPointerDataPacket (dart:ui/hooks.dart:98:31)
(elided 5 frames from class _AssertionError and dart:async)
The FixedExtentScrollController sending notification was: FixedExtentScrollController#62e0c(one client, offset 33.2)

Steps to Reproduce

Execute flutter run on the code sample
...
...

Expected results:
Actual results:

Code sample

Logs

Hi @peopleadam ,
When opening an issue, please try to provide all or some of the information indicated in the template.

Issue Template
If you are looking for support, please check out our documentation
or consider asking a question on Stack Overflow:
- https://flutter.dev/
- https://api.flutter.dev/
- https://stackoverflow.com/questions/tagged/flutter?sort=frequent

If you have found a bug or if our documentation doesn't have an answer
to what you're looking for, then fill our the template below. Please read
our guide to filing a bug first: https://flutter.dev/docs/resources/bug-reports

Steps to Reproduce

Please tell us exactly how to reproduce the problem you are running into.

Please attach a small application (ideally just one main.dart file) that
reproduces the problem. You could use https://gist.github.com/ for this.

If the problem is with your application's rendering, then please attach
a screenshot and explain what the problem is.

1. ...
2. ...
3. ...

Please tell us which target platform(s) the problem occurs (Android / iOS / Web / macOS / Linux / Windows)
Which target OS version, for Web, browser, is the test system running?
Does the problem occur on emulator/simulator as well as on physical devices?

Target Platform:
Target OS version/browser:
Devices:

Run your application with `flutter run --verbose` and attach all the
log output below between the lines with the backticks. If there is an
exception, please see if the error message includes enough information
to explain how to solve the issue.

Run `flutter analyze` and attach any output of that command below.
If there are any analysis errors, try resolving them before filing this issue.

Finally, paste the output of running `flutter doctor -v` here.

Closing this issue, since no information has been provided.
Thank you.


change module procedure

Thanks! I have fixed it (pure elemental module), and found other bugs, empty submodule, I also fixed it, I suggest you can pull it from this library again.

https://blog.csdn.net/wzy4072/article/details/79628659


[Docs] Rename remaining Swift Flow refs. Announce merger.

Current coverage is 100.00%

Merging #49 into master will not affect coverage as of 9aad963

@@            master     #49   diff @@
======================================
  Files            6       6
  Stmts          100      90    -10
  Branches         0       0
  Methods
======================================
- Hit            100      90    -10
  Partial          0       0
  Missed           0       0

Review entire Coverage Diff as of 9aad963

Powered by Codecov. Updated on successful CI builds.

I’m not sure why the travis if failing. It’s only docs changes.

Oh. It's just a travis timeout. Should be fixed once it reruns.

@agentk Will need a rebase before we can merge :smile:

Rebase done. I haven't accounted for SwiftFlow/CoreTypes/StoreType.swift from the rebase. That might be best for a artefact cleanup PR.

Merci! :ship:


<module_path_to_serializer_class>.RelatedSerializer?

Can you elaborate on the usage of this expression?

My problem has been solved


Have an autoflush option for files

To prevent files from taking up too much memory, an optional autoflush argument for file objects would be handy. Basically, a number of objects would be specified and every time an object is added, the number of objects in storelist would be compared to this specified limit. If it reaches said limit, the file automatically calls its write method.

This would only work if the file has a filepath associated with it.

Implemented as buffersize in f298976e5f40bba7f8c5811eb4d9b1c4040160ab.


Wrong length of the token

Hello there,
I'm Seyedhamid hosseini from Qotto and I'm starting this issue to discuss a problem we encounter from time to time.
We're using the following parameters to generate an activation token:

a key
a counter
a starting code
the duration

As it is mentioned in the documentation, the activation token is supposed to contain 9 digits, however, from time to time and for specific combinations of the above-mentioned parameters; an 8 digits token is generated.
That being said, this wrong format of the token causes the process to fail and the client has to start again.

Hi!
In this type of cases you just need to pad the token with leading 0s and it should work as expected.
You can use the following example code to convert your token integer into a properly padded string:
'{:09d}'.format(token)

Thanks a lot for your response, we implemented the suggested method and tested it recently.
The issue seems to be fixed for us.

@

Hello there, I'm Seyedhamid hosseini from Qotto and I'm starting this issue to discuss a problem we encounter from time to time. We're using the following parameters to generate an activation token:

a key
a counter
a starting code
the duration

As it is mentioned in the documentation, the activation token is supposed to contain 9 digits, however, from time to time and for specific combinations of the above-mentioned parameters; an 8 digits token is generated. That being said, this wrong format of the token causes the process to fail and the client has to start again.


Added Where-Object

It was bothering me that Where-Object wasn't red like the other cmdlets such as Foreach-Object etc...

Wow, i never noticed! Thanks for the report and fix :+1:
It looks like this is a problem with the grammar i depend on as well (which also highlights stuff on GitHub.com). You can see it here at the moment:
Where-Object
Select-Object

The related issue on SublimeText/Powershell is https://github.com/SublimeText/PowerShell/pull/141. i wonder if this work might be helpful for them too, mind taking a look at that to see if there's a way this work can help them as well?
i'll merge this in while we wait for things to settle out upstream, thanks again!

published to apm thanks!


[publish][m #15]: Publish End-to-End Test

End to End test for Publish Flow

@subhankarb @zelima Please review code for Publish flow Test.

Coverage remained the same at 82.846% when pulling 8f28798c99267f1ec9000416cba9f61a923bd606 on kittdotech:master into 4c043571204dd76b118bd8ce2088a939124e257c on frictionlessdata:master.


GRPCClient for invoking gRPC server

Hello, I am using lens as a Go package to build an indexer, and I have a requirement to use the gRPC server over the RPC server. I see you may have started to add gRPC client support in some areas, but when I try to use the GRPCAddr in ChainClientConfig and call Invoke one of the gRPC methods, I believe it is still trying to reach out to the RPC server instead because it gives me an
rpc error: code = Unknown desc = unknown query path: unknown request
error.
Is gRPC querying to the gRPC server not supported? And if not, is this something that will be available in the future? Thanks!

The idea is to support gRPC as default, but some refactoring needs to happen first. The gRPC support would be through some logic integration dynamic Cosmos I believe


Error Code 1011

This error happens whenever I connect to the lavalink nodes. I don't really know what happens, but I used the same code as u did in the example.

Can you show your code and error with stack trace? Without I don't anyone will help

Yes I'm on my PC now lemme take a screenshot

It's just closing, then connecting again, then closing and etc

and my code is just the one from the README

Are you using shoukaku 1.5.x?

If version is below 1.5.x update it

Fixed in 1.6.5


RPC Commands

With added RPC support from the Lethal Company Mod Library, update commands to allow a player as a parameter.

Support for this has been added in #34


Add api to create new RSASecurityKey

For POP scenarios it is common to require a RSA key.
This API would abstract the underlying crypto operators that change for different OS's and runtimes (Core vs. Desktop)

@brentschmaltz - By looking at the description, I believe that this issue has already been resolved.

yep


Where is bot token

Were do i need to imput the bot token

@itsnotlucasme, this package does not handle the discord connection for you, it simply provides some utility functions that make use of the discord.js api. You will still need to setup a discord bot using discord.js which uses a login function where you can pass the bot token.


Add Steps component to github.mdx

Add Steps component togithub.mdx
Adds the Step component where needed.

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

en
guides/deploy/github.mdx
Source changed, localizations will be marked as outdated.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.

!coauthor


Trying to set exported parameter of inherited scene fails

Godot version
v4.0.3.stable.official [5222a99f5]
System information
Windows 10, Compatability
Issue description
Encountering a strange issue where I'm setting a parameter of a class and it just returns null whenever I try to access it.  This parameter is an exported parameter of a class that is the class that is the root class of an inherited scene.  I'm not sure if that is related to this issue or not.  Anyway, I've tried setting this parameter in the editor and while the editor lets me set it, when I try to read it later on it returns null.  Even if I assign these values in the _ready() method, they are still null when I try to read them later.
Steps to reproduce
Open the attached project in the editor.  The class having the problem is res://scene/column_down_stack.tscn.  It has a property called @export var up_stack:NodePath.  There are several instances of this on the scene that provides the card table: res://scene/card_table.tscn.  Notice that all the col_down_* classes have their Up Stack value set.  I've also made doubly sure by setting this explicitly in the _ready method.  However, whenever ColumnDownStack._on_selected() is called (this happens if the user clicks on a face down pile in the columns area)), the up_stack parameter is null.  There is nothing else that sets this parameter, so I don't know why this is happening.  My only guess is that it has something to do with Godot getting mixed up due to the scene being inherited from the more general card stack scene.
Minimal reproduction project
godotSolitaire.zip

That worked.  I guess it was my fault after all.  Still, it would be helpful if the editor provided a warning when the user does this.

Still, it would be helpful if the editor provided a warning when the user does this.

There is a warning, but it's not very noticeable and for the compiler it is a local variable:

After #73427, there will be an error:

Looks like the issue is resolved, so I'm closing this.

The bug with incorrect/non-obvious variable shadowing is tracked by #54944.
The proper error message will be implemented by #73427.
Or alternatively, if we reject the PR, this code will work in an intuitive way after another PR.


Ignore hits on transparent parts of the image

It seems the entire rectangle of the RenderVectorGraphic absorbs hits (for example when a user taps the screen).
Would it be possible to adjust the hitTestSelf to only return true if the hit is on a non-transparent part of the SVG?

That would be expensive and probably not quite the desired behavior in a lot of cases.
Imagine, for example, you have an icon you're painting on a button - you don't want to do expensive hit testing logic to see if the hit falls within some complicated path describing the outline of the icon just to end up getting a hit on the button "under" it...

Oh absolutely, it's not desirable for every use case. Maybe I should have framed it as "Could you make it optional" to ignore hits in the transparent parts of the SVG.
In my use case, I have a circular button drawn as a SVG, and with the current hit test you can hit outside of the bounds of what's visible, which is undesirable.
I suspect the work around, is for me calculating a mask and implementing my own "hit test" on top of the SVG.


Fix issue 251

added more detailed check of type parameters when comparing two reference types
added test case for this issue

Good work, as always. I am sorry I did not manage to review it earlier


Remove extra parentheses in Show Time/Duration

>>> 1.5 :: Duration
(3/2)
-- Want: 3/2

-- For cases like
>>> (mcatMaybes $ (mconcat [rest, c', ab, db', e^*2, f, g, mconcat [ab^*3, bb]^/2, c'^*2])^/4) :: Voice Int
[((1/4),12)^.note,((1/4),8)^.note,((1/4),13)^.note,((1/2),4)^.note,((1/4),5)^.note,((1/4),7)^.note,((3/8),8)^.note,((1/8),10)^.note,((1/2),12)^.note]^.voice

Fix by implementing showsPrec.

This repo has been merged into: https://github.com/music-suite/music-suite
Closing :)


Polar Area gridline axes inclination issue

Hi!
It seemed gridlines in polar area chart are inclined in 1 deg or something. Check screenshots

@egodive what version are you using?

I use bundled 2.9.3 version via CDN

Ah, I believe we've fixed this in v3 already. https://www.chartjs.org/samples/master/charts/polar-area.html appears to be correct

@etimberg it's great, thank you!
your charts are the best charts on whole web! :)


Add Configurable Post Test Batch Delay

Introduced a new <PostTestDelay> configuration option which allows users to specify a delay time in milliseconds which is applied at the end of each test batch execution (before result parsing).
Closes #176

@netspiri please document also in wiki


Fix sample SFService yaml, case issue with redirectURI

CRD defines the field as redirectUri.
Trying to define a SFService with provided sample fails with .spec.dashboardClient.redirectURI: field not declared in schema

Pull Request Test Coverage Report for Build 5763

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 88.471%

Totals

Change from base Build 5762:
0.0%

Covered Lines:
9723

Relevant Lines:
10606

💛  - Coveralls


rivinecg: default generated client does not compile

ob@MacBook-Pro-783 rchain % dep init
  Locking in master (eeba5f6) for transitive dep golang.org/x/sys
  Locking in master (19c3af6) for transitive dep github.com/rivine/bbolt
  Locking in master (6fb6489) for transitive dep github.com/NebulousLabs/fastrand
  Locking in master (08d5d54) for transitive dep github.com/NebulousLabs/merkletree
  Locking in master (e7e4b65) for transitive dep golang.org/x/net
  Locking in master (b32978b) for transitive dep github.com/NebulousLabs/go-upnp
  Locking in master (c55f737) for transitive dep github.com/tyler-smith/go-bip39
  Locking in v1.0.7 (e28719e) for transitive dep github.com/rivine/smux
  Locking in v0.3.2 (342b2e1) for transitive dep golang.org/x/text
  Locking in v0.8.1 (ba968bf) for transitive dep github.com/pkg/errors
  Using master as constraint for direct dep github.com/bgentry/speakeasy
  Locking in master (4aabc24) for direct dep github.com/bgentry/speakeasy
  Locking in v1.0 (76626ae) for transitive dep github.com/inconshreveable/mousetrap
  Locking in master (9f787ce) for transitive dep github.com/NebulousLabs/errors
  Using ^1.2.0 as constraint for direct dep github.com/threefoldtech/rivine
  Locking in v1.2.0 (fc2dfad) for direct dep github.com/threefoldtech/rivine
  Locking in master (7a9e348) for transitive dep github.com/NebulousLabs/demotemutex
  Locking in master (e9b2fee) for transitive dep golang.org/x/crypto
  Locking in master (3a71999) for transitive dep gitlab.com/NebulousLabs/go-upnp
  Locking in master (603482d) for transitive dep gitlab.com/NebulousLabs/fastrand
  Using ^0.0.5 as constraint for direct dep github.com/spf13/cobra
  Locking in v0.0.5 (f2b07da) for direct dep github.com/spf13/cobra
  Locking in v1.0.5 (2e9d26c) for transitive dep github.com/spf13/pflag
  Using ^1.3.0 as constraint for direct dep github.com/julienschmidt/httprouter
  Locking in v1.3.0 (4eec211) for direct dep github.com/julienschmidt/httprouter
rob@MacBook-Pro-783 rchain % go build ./cmd/rchaind
rob@MacBook-Pro-783 rchain % go build ./cmd/rchainc
# github.com/robvanmieghem/rchain/cmd/rchainc
cmd/rchainc/main.go:26:37: "github.com/robvanmieghem/rchain/vendor/github.com/threefoldtech/rivine/extensions/minting/client".CreateConsensusCmd(cliClient.CommandLineClient) used as value
cmd/rchainc/main.go:28:35: "github.com/robvanmieghem/rchain/vendor/github.com/threefoldtech/rivine/extensions/minting/client".CreateExploreCmd(cliClient.CommandLineClient) used as value
cmd/rchainc/main.go:32:35: "github.com/robvanmieghem/rchain/vendor/github.com/threefoldtech/rivine/extensions/minting/client".CreateWalletCmds(cliClient.CommandLineClient, "github.com/robvanmieghem/rchain/pkg/types".TransactionVersionMinterDefinition, "github.com/robvanmieghem/rchain/pkg/types".TransactionVersionCoinCreation, &"github.com/robvanmieghem/rchain/vendor/github.com/threefoldtech/rivine/extensions/minting/client".WalletCmdsOpts literal) used as value
cmd/rchainc/main.go:52:14: undefined: "github.com/robvanmieghem/rchain/vendor/github.com/threefoldtech/rivine/pkg/client".NewLazyBaseClientFromCommandLineClient
cmd/rchainc/transactions.go:26:51: cannot use bc (type "github.com/robvanmieghem/rchain/vendor/github.com/threefoldtech/rivine/pkg/client".BaseClient) as type *"github.com/robvanmieghem/rchain/vendor/github.com/threefoldtech/rivine/pkg/client".BaseClient in argument to "github.com/robvanmieghem/rchain/vendor/github.com/threefoldtech/rivine/extensions/minting/client".NewPluginConsensusClient

gopkg.toml declares

[[constraint]]
  name = "github.com/threefoldtech/rivine"
  version = "1.2.0"

while the generated code is already for the future 1.2.0 release

that's just how dep init works I guess, it takes the highest version of the dependency

will be fixed for now if the rivine repo is tagged with a higher version, in the future rivine should be supported with go modules.


Build and deploy demo site via GH Action workflow

Summary
Use source files in branch demo-site to build demo site and deploy via GitHub Pages.
A separate branch allows customizing source dir contents without affecting theme.
Though this does include a small amount of duplication (w.r.t Markdown source files), the changes to them will probably be so small that maintainers can manually cherry-pick to the demo-site branch.
Context
https://github.com/jekyll/minima/pull/601#issuecomment-1002753354
Closes #601

@mattr- Can I have your thoughts on this? Thanks. :)

@jekyllbot: merge +dev

Thanks @mattr- :smile:


React Native and snack: undefined is not an object (evaluating ‘r.fetch.bind’) imported from null

On the lines of #8, we're seeing an issue with a library we are building @appbaseio/reactivesearch-native that has a dependency that uses fetch-everywhere.
Exact error:

Device: (1069:6932) Failed to install module ‘@appbaseio/reactivesearch-native’: undefined is not an object (evaluating ‘r.fetch.bind’) imported from null

Do you have any pointers on how we could resolve it?
This is an online environment where the error can be seen: https://snack.expo.io/@siddharthlatest/reactivesearch-snack

I think this is the likely root cause:
https://github.com/lucasfeliciano/fetch-everywhere/blob/master/fetch-npm-browserify.js#L6
On react-native environment, L7 somehow evals to this.fetch.bind(this) which causes the error. See this snack snippet that demonstrates this - https://snack.expo.io/B1gJ6nx8G.
Error log on this snippet:

The solution to this is posted here. I found it after a lot of searching. I'll fix it and put in the pull request in a few. Until it's merged you can monkey patch it in your app yourself or use my fork. Although I do realize this is a very old issue.

Fixed issue in pull request #11


One crate follow-up issue

Issue summary

[ ] TODOs in code
[ ] Rename forestd to forest-daemon
[ ] Refactor CI jobs
[ ] Macro publicity
[ ] Vendored docs

Other information and links

#2957

Rename forestd to forest-daemon

Curious, what's wrong with forestd?

Rename forestd to forest-daemon

Curious, what's wrong with forestd?

I'm afraid people will mistake it for forest when running commands from our documentation. I can easily imagine someone accidentally running forest --import-snapshot ... instead of forestd --import-snapshot. It would be nice if forestd never had to be used directly, but we're not quite there yet.

@aatifsyed What items are still actionable?

All action items have their own issue


Replace some manual from_str impls with strum

This PR replaces some manual from_str implementations with a derive of strum_macros::EnumString.

Thanks for keep on contributing and improving the internals! The change looks good, but now that I merged your other branch #137 there are couple small conflicts. As soon as they are fixed, we should be good to merge.


External CSS files are ignored in HTML preview

Linked CSS files have no effect when previewing an HTML file. I have tried it with selecting htmlp, with Atom 0.176.0 under GNU/Linux.

Did you maintain the base directory for the folder, which has these html/css files. To maintain the base directory type preview-plus:base in the command window. See if it that will be the directory that need to go before your linked css files. Please confirm if it works by updating the base directory for HTMLP.

Yes, the project folder containing the files is set correctly as the HTML base.

can you paste the code/screen shot of the screen.

Here is a screenshot:

And [here] (http://www.mediafire.com/download/4ymp08y0f1bxvcn/myWebPage.zip) are the project files.

Can you try putting a slash at the end of the HTML Base and try it out

can you add under the  tag
<base href='/media/.../webPage/'></base>

with href pointing to the HTML Base and save it in desktop and open in chrome and see if it picks up the css.

It is still the same in both Chromium and Firefox, with everything appearing correctly, but within Atom and htmlp, it looks the same as in the screenshot.

so you saved the html file to desktop with the base tag added as I mentioned and opened the html file in desktop in chromium

Yes.

can you try going to config for preview plus and switching off the webview(it is switched on by default)

If I turn it off, this web page looks the same in the preview. However, it solves another issue in another project, namely UTF-8 character coding.

By the way, Atom says preview-plus makes 4 deprecated calls. I'm not sure if they are related to this problem.

I don't have the problem with windows..I don't have a way to debug it/look at what is missing. I am going to wait for some more feedbacks.

I went ahead and put a patch so you should be see the preview for html with the linked css. To enable this go to the setting for preview-plus package and switch on html url (click the checkbox). and reload.
The only issue would be that inline javascript will not execute with this approach. But you would still be able to use the selected text and preview it using the older approach..
Please check and close the issue if it works.

It seems to work now, thank you very much!


feat(website): initial version of module federation website

Preview: https://wondrous-mooncake-6a9ff7.netlify.app/

LGTM overall, just some minor things related to configuration


04/24/2018 21:24 : Create Pull Request for branch master -> prTesting1_2018042421

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

split/updateTestRef1.md
:white_check_mark:Succeeded

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


info button added in trailing is not accessible by Voice Over

After checking with @pya35, this criterion is correct.


[REFACTOR] Use DI to inject worker using factory

NOT WORKING ❌
error: Dagger does not support injecting @AssistedInject type, dev.hossain.weatheralert.work.WeatherCheckWorker. Did you mean to inject its assisted factory type instead?
dev.hossain.weatheralert.work.WeatherCheckWorker real);

These resources looks promising

https://github.com/illarionov/pixnews-anvil-codegen
https://rbusarow.github.io/Tangle/docs/next/workManager/


Services: first build: abs added

{package} {version} {:snowflake:}
Destination channel: {Snowflake | defaults}
Links

{ticket_number}
Upstream repository
Upstream changelog/diff
Relevant dependency PRs:

...

Explanation of changes:

...

Linter check found the following problems:
ERROR conda.cli.main_run:execute(124): `conda run conda-lint /tmp/abs_7esfpkceq9/clone` failed. (See above for error)
The following problems have been found:
===== WARNINGS =====

clone/recipe/meta.yaml:30: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
clone/recipe/meta.yaml:31: host_section_needs_exact_pinnings: Linked libraries host should have exact version pinnings.
clone/recipe/meta.yaml:44: missing_description: The recipe is missing a description

===== ERRORS =====

clone/recipe/meta.yaml:44: missing_dev_url: The recipe is missing a dev_url
clone/recipe/meta.yaml:44: missing_license_family: The recipe is missing the about/license_family key.
clone/recipe/meta.yaml:44: missing_documentation: The recipe is missing a doc_url or doc_source_url
===== Final Report: =====
3 Errors and 3 Warnings were found


Separate cuda build from non cuda machine

After this commit, by default make won't build cuda, so there is no cuda dependency by default. In machine with cuda library, need to do make cuda_all, (or cuda_debug, cuda_release etc) to build cuda.

is it any possible to build without cuda at all? see #1109
It's a deal breaker to require the dependencies where the cuda support is not needed.

This is the change to build without cuda by default. It's the solution to
address the issue 1109.
Thanks,
Hong
On Fri, Mar 22, 2024 at 2:02 AM Tianhao Wang @.***>
wrote:

is it any possible to build without cuda at all? see #1109
https://github.com/QuarkContainer/Quark/issues/1109
It's a deal breaker to require the dependencies where the cuda support is
not needed.
—
Reply to this email directly, view it on GitHub
https://github.com/QuarkContainer/Quark/pull/1164#issuecomment-2014648506,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ACYKYXTYEXAZDKWEE2TXUOLYZPXQTAVCNFSM6AAAAABFCYMUI2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDAMJUGY2DQNJQGY
.
You are receiving this because you authored the thread.Message ID:
@.***>

On Fri Mar 22, 2024 at 5:48 PM CET, changhong94 wrote:

This is the change to build without cuda by default. It's the solution to
address the issue 1109.
Thanks,
Hong

I misread your PR in the first place. That's why I deleted that comment from
github. Sorry for the noise / thanks for the patch.


DESK-5751: Update minimist

See https://github.com/superhuman/website/security/dependabot/125
Ticket: https://linear.app/superhuman/issue/DESK-5776/minimist

@Hehk how do you feel about replacing this PR with https://github.com/rogueg/zen/pull/27?


octant: 0.16.0 -> 0.17.0

Motivation for this change
Either depends on or replaces #111654
You can either merge that one first then I rebase this branch or just merge this one.

octant: 0.16.0 -> 0.17.0
octant: add update script
octant: refactor

Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[X] Ensured that relevant documentation is up to date
[X] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 113379 run on x86_64-linux 1

1 package built:

octant

Result of nixpkgs-review pr 113379 at b488bfa9 run on aarch64-linux 1

1 package built:

octant

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/applications/networking/cluster/octant/default.nix:29:3:
   |
29 |   installPhase = ''
   |   ^

warning: missing-phase-hooks

Resolved


gitea: 1.9.1 -> 1.9.2

Motivation for this change
Follow-up from #67269
Things done

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

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

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Notify maintainers
cc @

@kolaente It would be great if you could cherry-pick with -x to get the reference to the original commit.

@GrahamcOfBorg build nixosTests.gitea
Regardless of the cherry picking flag we can change the actual changes in the meanwhile.

@etu, done.

@kolaente We usually do a cherry pick of the commit implementing the change rather than the merge commit.
That would be git cherry-pick -x 3ddb1a09fc7c8ba6472814d18822eb3c7604a778 -- that commit 3ddb1a09fc7c8ba6472814d18822eb3c7604a778 is your original change.

@GrahamcOfBorg test gitea

@etu Done. I didn't know that, thanks for clarifying. Still new to nix :sweat_smile:

@GrahamcOfBorg test gitea


update readme

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@0799b91). Click here to learn what that means.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #122   +/-   ##
=======================================
  Coverage        ?   23.07%
=======================================
  Files           ?       10
  Lines           ?      325
  Branches        ?        0
=======================================
  Hits            ?       75
  Misses          ?      247
  Partials        ?        3

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 0799b91...0758c7f. Read the comment docs.


ADD the basic interface of explainer

Signed-off-by: mapleFU<EMAIL_ADDRESS>What problem does this PR solve?
What is changed and how does it work?
Check List
Tests

Unit test
E2E test
Manual test (add detailed scripts or steps below)
No code

Code changes

Has Go code change
Has CI related scripts change
Has Terraform scripts change

Side effects

Breaking backward compatibility

Related changes

Need to update the documentation

Does this PR introduce a user-facing change?:

NONE

The combine logic and check is still unimplemented now.

{:type :info, :f :leave, :value "n5", :process :nemesis, :time<PHONE_NUMBER>0, :index 20112}

Maybe we cannot handle this.

{:type :info, :f :leave, :value "n5", :process :nemesis, :time<PHONE_NUMBER>0, :index 20112}

Maybe we cannot handle this.

But nemesis record won't effect the correct of logic


Serialization format

Currently flatbuffers but there might be reasons to reevaluate.
A new interesting contender - https://chronoxor.github.io/FastBinaryEncoding/ and see https://github.com/mapbox/tippecanoe/issues/777 for evidence that flatbuffers isn't optimal in fx. Go.

Also https://github.com/mapbox/protozero maybe a interested contender because flatgeobuf is going to be a stable wireformat

While it's tempting to make the decision to constrain to an encoding that cannot change for a bit of extra potential performance I'm not convinced yet.

Very intresting thoughts at https://github.com/google/flatbuffers/issues/5875


feat: upgrade AWS sdk to v3

rename class from SqsHandler to SQSHandler
rename types Simple* to Parsed* (e.g. ParsedMessage)
BREAKING CHANGE: SQSHandler constructor takes a SQSClient from AWS sdk v3 now

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


aws-ecr: repositoryName of imported repository may be incorrect

Repro:
const repo = Repository.import(this, 'MyRepo', {
  repositoryArn: 'arn:aws:ecr:us-east-1:585695036304:repository/foo/bar/foo/fooo'
});

Now, we expect repo.repositoryName to resolve to foo/bar/foo/fooo but it will resolve to foo.
This is because ImportedRepository uses CFN's split/select to extract the first component after "/".

If the user provides a concrete (non-token) string, we can just use synth-time string manipulation to extract the full repository name. If repositoryArn is a token, we will have to require that repositoryName will also be passed.


Pull Mirrors: Option to disable force-push

Feature Description
I use pull mirrors for backup repos that have risks of removal. One way to remove a repo is to force push one commit and in this case my backup will be deleted as well.
There is no document about if gitea does or does not force push on pull mirrors. I did an experiment and found out that it did force push.
Screenshots
No response

Duplicate of #14076


Discrepancy between unicode-7.0.0\bidi character classes and respective Unicode date from ftp://ftp.unicode.org/Public/UNIDATA/UnicodeData.txt

After comparing the character classes found in unicode-7.0.0\bidi\index.js and corresponding classification under Unicode official site ftp://ftp.unicode.org/Public/UNIDATA/UnicodeData.txt
I found multiple differences, like at points 13,44,46,48,134,174,666,667 etc.
What is the coorect data to rely upon?

ftp://ftp.unicode.org/Public/UNIDATA/UnicodeData.txt is the latest available Unicode version, which is 8.0.0 at the moment, not 7.0.0.
See https://github.com/mathiasbynens/node-unicode-data/blob/2ead4522aa1564e8c569189e7fe8ceaf3d7b99d7/data/resources.js#L165-L175 for which resources are used.

Thanks a lot, my apologies


firefox, nss: enable SSLKEYLOGFILE option

Motivation for this change
See https://developer.mozilla.org/en-US/docs/Mozilla/Projects/NSS/Key_Log_Format:

Note: starting with NSS 3.24 (used by Firefox 48 and 49 only), the
SSLKEYLOGFILE approach is disabled by default for optimized builds
using the Makefile (those using gyp via build.sh are not affected).
Distributors can re-enable it at compile time though (using the
NSS_ALLOW_SSLKEYLOGFILE=1 make variable) which is done for the
official Firefox binaries. (See bug 1188657.) Notably, Debian does not
have this option enabled, see Debian bug 842292.

This patch fixes that, and allows other programs to also use this
options as they desire, with the allowSSLKeylogFile on the nss
package.
(NOTE: this option is NOT enabled by default on Firefox, the user
specifically has to set the SSLKEYLOGFILE environment variable!)
Things done

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

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

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

Success on x86_64-linux (full log)
Attempted: nss
Partial log (click to expand)
/nix/store/ail05likjg5m2qws9rvjd3qpkqxi6l7a-nss-3.38

Success on aarch64-linux (full log)
Attempted: nss
Partial log (click to expand)
these paths will be fetched (1.13 MiB download, 4.48 MiB unpacked):
  /nix/store/5wq31rcya980b059ria4kdwm09qdy5wa-nss-3.38
copying path '/nix/store/5wq31rcya980b059ria4kdwm09qdy5wa-nss-3.38' from 'https://cache.nixos.org'...
/nix/store/5wq31rcya980b059ria4kdwm09qdy5wa-nss-3.38

@xeji @fpletz It would allow for users to capture their own SSL/TLS encrypted traffic with tools like Wireshark.
I personally wanted to check how a website was getting its updates, so I opened up Firefox's element inspector (tab Network). The updates weren't showing up in there, so I was looking for a way to allow Wireshark traffic capture through SSL. Most of the suggestions were "Oh, just set this env variable, and you'll be able to decrypt the traffic". I tried this, and it didn't work. It wasn't immediately clear, until I stumbled upon the link mentioned in the PR. I assumed that what was the case for Debian would also be the case for NixOS.
On the security risk side: there is a potential attack, but to exploit it, an attacker has to be able to:

Set arbitrary enviornment variables
Read a file the firefox process can write
Intercept traffic

It's possible, but if those conditions are met, you are already screwed I'd say. Furthermore, the official Firefox binaries also include this option.
If this is still not wanted, I also see another course of action: give a warning when starting Firefox with the SSLKEYLOGFILE variable set, and have another Firefox package (nixpkgs.firefoxSslKeyLog) that has this option enabled.

Thanks for explaining.
I don't think we should enable this feature by default. It does increase the attack surface. I agree it's not a straightforward attack, but this might for instance be used by an attacker who has short-term access to your computer to intercept traffic later...
For debugging TLS connections  there are better options like installing a man-in-the middle proxy (like mitmproxy): it must only be installed when you really use it, plus it's generic and doesn't require modifying/rebuilding applications.

I assumed that what was the case for Debian would also be the case for NixOS.

Not necessarily. Each distro makes their own tradeoffs. Arch, for instance, does not enable this.
That said, I'm fine with adding the option but not with enabling it for firefox by default. A user who really wants it should then override firefox locally.

have another Firefox package (nixpkgs.firefoxSslKeyLog) that has this option enabled.

Definitely not. Since there are good alternatives for debugging, this feature isn't important enough to duplicate firefox.

That said, I'm fine with adding the option but not with enabling it for firefox by default. A user who really wants it should then override firefox locally.

I'm really not a fan of having users override Firefox locally, since it takes forever to build.

I understand. But we also struggle with firefox build timeouts on Hydra, so adding another firefox build doesn't sound great either.

not rebuilding firefox

Can't you just put it into LD_LIBRARY_PATH or something?
Btw, MITMproxy works for all SSL things, not just for NSS.

@xeji I don't think having this environment variable is more dangerous then allowing LD_PRELOAD where an attacker can do the same shenanigans.

So the alternative to this pull request would be documentation how nss can be overriden in combination with LD_LIBRARY_PATH to inject it into firefox.

Any updates on this pull request, please?


ReturnValue field in documentation is wrong.

[Enter feedback here]
Suggested the article says Return value is None, which is not correct. Remarks sesion and signature clearly say that the return value is NTSTATUS

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

ID: 93b50201-ea38-2862-1ae4-b72dddf23d4c
Version Independent ID: 72388b48-638a-6729-87cf-a35b2bdb303c
Content: IoAcquireRemoveLock macro (wdm.h) - Windows drivers
Content Source: wdk-ddi-src/content/wdm/nf-wdm-ioacquireremovelock.md
Product: windows-hardware
Technology: wdk-api-reference
GitHub Login: @barrygolden
Microsoft Alias: bagold

@pankajbharti, thanks for pointing out the missing return value information. The topics has been updated.


Allow editing of summary blurb on home page

Update the summary (and/or make this editable by an admin) on the home page to reflect what we actually look for in a session and messaging to reiterate compliance with the code of conduct is required of all presenters and their materials.
Check with @jennapederson for context and exact messaging.
Ideally this is easily editable by an admin as it seems to be dynamic.

@jennapederson could you post a screenshot of the summary blurb to help clear things up? I'm not sure that I'm looking at the right thing.

Main goal here is easy updating of this blurb. It's so out of date right now, but I can see it being slightly more dynamic than changing once every 10 years 😂


improve the wires facing direction, add option to clear the diagram

I make the wires to always face the same direction, i.e. inputs facing to the left and outputs to the right, although it might need some refinement. The effect is most noticeable in the SR Latch demo. Please let me know if you need me to change something.


Feature Request: Add Semantic clue to functions that return Maybe

There no way of knowing if a function like List.tail returns a Maybe a just from looking at the function name. I think there is an opportunity to add clues to the return value on the function name, adding meaning (semantics) to the function names.
In Ruby there is a "!" at the end of a method that alters the original element and a "?" to ones that return a boolean. Now I don't think neither the "!" or the "?" would work for the normal function name, but since I found myself using the Maybe.withDefault construct a lot, it could be built in.
For example:
List.tail list -> Maybe a
and
List.tail! default list -> List
by doing a
tail! default list =
  case List.tail list of
    Nothing -> default
    Just x -> x

They you'd know any method that has a "!" (bang) alias will return a Maybe without the  bang.

In functional languages, one typically has the type signature at hand, but, beyond that, Maybe can be easily inferred by the purpose of a function: if there is a possibility that it will not be able to return an element, as is the case e.g. with List.head and List.tail, the result must be wrapped in a type that can represent both a value and no value. Maybe is the obvious choice here, though it's trivial (and oftentimes necessary) to make your own algebraic data type.
I don't think it's a good idea to introduce special syntax or even a naming convention for this. Type signatures are your friends!

I think this has been considered on the mailing list in the past and been rejected. Even if not (yet), the mailing list would be the best place to have this discussion, since the issue tracker here is mainly for bugs and their fixes. So I'm closing the issue here. Of course, you can still take it to the mailing list (possibly after going through the mailing list's archive to see whether/what the potential previous discussion was like).


sokol_app:window size is incorrect.

My platform is win10.
I create a window with following code:

#define SOKOL_IMPL
#define SOKOL_GLCORE33
#include "sokol_gfx.h"
#include "sokol_app.h"
#include "sokol_glue.h"

#include <iostream>

struct state_t {
    sg_pass_action pass_action;
};

static state_t state;

static void init(void)
{
    sg_desc desc = {};
    desc.context = sapp_sgcontext();
    sg_setup(&desc);
    state.pass_action.colors[0].action = SG_ACTION_CLEAR;
    state.pass_action.colors[0].value = { 0.0f, 0.5f, 0.7f, 1.0f };
}

static void frame(void)
{
    sg_begin_default_pass(&state.pass_action, sapp_width(), sapp_height());
    sg_end_pass();
    sg_commit();
}

static void cleanup(void)
{
    sg_shutdown();
}

static void event(const sapp_event* e)
{
    std::cout << "6666666"
              << " " << sapp_dpi_scale() << std::endl;
    std::cout << sapp_width() << "--" << sapp_height() << std::endl;
    if (e->type == SAPP_EVENTTYPE_KEY_DOWN) {
        if (e->key_code == SAPP_KEYCODE_ESCAPE) {
            sapp_request_quit();
        }
    }
}

sapp_desc sokol_main(int argc, char* argv[])
{
    sapp_desc desc = {};
    desc.init_cb = init;
    desc.frame_cb = frame;
    desc.cleanup_cb = cleanup;
    desc.event_cb = event;
    desc.width = 800;
    desc.height = 600;
    // desc.gl_force_gles2 = true;
    desc.window_title = "6666666";
    desc.win32_console_utf8 = true;
    desc.win32_console_attach = true;
    return desc;
}

But i get a window with size 1400x1050, that is weird.
Any help is welcome.

This can be because of the DPI scaling in Windows. When you go (in Win10) to Settings => System, there's a "Scale and layout" dropdown. If this is not 100% then the sokol_app.h window size will be scaled accordingly. This is to prevent that a window appears extremely tiny on system which have a HighDPI display but use the Windows DPI scale setting to compensate.
If your "Scale and layout" setting is 100% however, please reopen this ticket, because then this needs to be investigated :)

Ok, i see.


Add support for "Optional chaining" in React

Babel and eslint are not set up to use "Optional chaining" in the plugin. Optional chaining makes the code clean and understandable, so it will be good to have that setup.
Some of the dependencies in package.json are also not compatible to use optional chaining, so we need to upgrade those to support optional chaining with eslint.

Closing as already fixed


Content appears behind AppBarLayout

Using version <IP_ADDRESS>, NestedScrollView content appears behind AppBarLayout

<?xml version="1.0" encoding="utf-8"?>
<android.support.design.widget.CoordinatorLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    xmlns:tools="http://schemas.android.com/tools"
    android:id="@+id/main_content"
    android:layout_width="match_parent"
    android:layout_height="match_parent">

    <android.support.v4.widget.NestedScrollView
        android:id="@+id/nested_scroll_view"
        android:layout_width="match_parent"
        android:layout_height="match_parent">

        <LinearLayout...>
    </android.support.v4.widget.NestedScrollView>

    <me.henrytao.smoothappbarlayout.SmoothAppBarLayout
        android:id="@+id/app_bar_layout"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:background="@color/colorPrimary"
        android:minHeight="?attr/actionBarSize">

        <android.support.design.widget.CollapsingToolbarLayout
            android:id="@+id/collapsing_toolbar_layout"
            android:layout_width="match_parent"
            android:layout_height="match_parent"
            app:layout_scrollFlags="scroll|exitUntilCollapsed">

            <android.support.v7.widget.Toolbar
                android:id="@+id/toolbar"
                android:layout_width="match_parent"
                android:layout_height="wrap_content"
                app:layout_collapseMode="pin" />

            <LinearLayout...>
        </android.support.design.widget.CollapsingToolbarLayout>
    </me.henrytao.smoothappbarlayout.SmoothAppBarLayout>
</android.support.design.widget.CoordinatorLayout>

Please set header padding for NestedScrollView (same height as SmoothAppBarLayout's height)

The library isn't supposed to do this itself ? I've used it in another project some time ago without having to set this padding manually. (maybe I am wrong)
Thanks for your quick answer

Hi @UNCVR
Due to the approach of SmoothAppBarLayout, you need to handle header view. It's tricky to do it in the library. In original AppBarLayout, you don't have to care about header view but it causes another issues.

I had the same issue, very annoying. Soo glad I found this post.
Here is how you do that:
<android.support.design.widget.CoordinatorLayout

    <android.support.v4.widget.NestedScrollView
        ...
        android:paddingTop="XXXdp"
        ...

    <me.henrytao.smoothappbarlayout.SmoothAppBarLayout
        ...
        android:layout_height="XXXdp"
        ...

</android.support.design.widget.CoordinatorLayout>

Thanks @roman-ku. I am glad that you can make it work.

Maybe add a note to the read me?


Postgresql container swapped for bitnami/postgresql suddenly in stable/postgresql

So I just went to look at stable/postgresql and it suddenly switched from the official container to the bitnami container in this PR:
https://github.com/helm/charts/pull/8004
I was checking out the template vars and noticed replication was added, how great! Oh wait, it was added via a non standard container and would break everything I have running in production if I upgraded
I'm a little bit puzzled how you can suddenly pick an entirely different underlying container that doesn't work with any of the past charts and claim the chart postgresql is a stable chart.
I've definitely lost hope in running Helm in production. Please stick to standard containers and if you're going to use alternative containers, call them stable/bitnami-postgresql and don't just switch stable/postgresql to something completely different.
I do not want to run bitnami containers in production.

Some thoughts on official images vs. Bitnami images in general:
Having a company like Bitnami maintain Docker base images for official Helm charts is perfectly fine in my opinion. The bitnami-docker-* images seem well maintained and are definitely "production grade", and even minideb seems to be a good base image.
However, depending on specific servers in the images is what bothers me a bit. I like the idea of being able to re-create those images, using only official sources (Debian base image or their servers for debootstrap, maybe postgresql.org) in case the maintainer deletes them from the repositories. Bitnami images use some kind of "magic" in their Stacksmith packages installed with "bitnami-pkg" (from downloads.bitnami.com). There is no open documentation of how these files are being generated (or is there?), and how I could replicate this in case Bitnami stops maintaining one of these...

so it looks like bitnami has no interest in supporting business continuity of postgresql users.  sad...

To be honest, this has been discussed for 2 months before being implemented whereas the postgres helm chart was without maintainers for weeks.
I know this is not a proper solution, and yes we had to do the transition to bitnami-based image as well, but at least it is alive.
Maybe the change was a mistake. Maybe not. I strongly encourage you to create your own postgres helm chart based on the last commit using the official image. This is not a joke: as the decentralization of this repository will be happening in a few months, it is good to have both "postgres chart with bitnami-based image" and "postgres chart with docker-backed based image".


'Someone else is editing this field' error doesn't disappear

When the two users get a 'Someone else is editing this field' error, both edits are rejected, but then after that, when the network is back up, it should be possible to get things back on the rails? Maybe if both keep retrying until some edit succeeds, and then remove the error text on both sides?
Or do you think this error should be unrecoverable?

Thanks for giving this a test!
You can see in the source that this warning is shown when the value of the field is an array. According to the intuitive semantics of the application, this is a conflict – though not according to the default semantics of m-ld. So this is an example of the application deciding what to do... in this case, show the other person's value, and wait for one or other of the participants to manually set the value again.
https://github.com/m-ld/m-ld-web-starter/blob/eecf488dfd4fea9f3ae0b3032666a043801744e1/form.js#L299
This is probably not a great user experience, but this demo is intended to throw up this kind of thing for discussion 😀!
A key point is that when the clones are online, the data always converges, guaranteed. What happens to the app's UI is up to the app. In this case, the data can even look like it has diverged – the participant have different values and might be showing a warning.
To solve this specific case we could add a single-valued constraint to the field – I'll add another comment later how to do this.
I also need to add this to the documentation.

OK, so the current behaviour is not really how m-ld is supposed to be used then, right?
But if we apply the single-valued constraint then the merge engine will instead pick a winner?

Actually, it really is how m-ld is meant to be used! There are lots of ways for apps to deal with "conflicts", and a valid one is to just tell the user there's something going on and wait until they deal with it. This app is just very basic.
Constraints are another, and yes, with single-valued the engine would pick a winner (and there are different ways for it to do so, to be explored!).


Map tiles not correctly loaded when setting "center"

Steps to reproduce:

start tabris-maps-example
zoom into the map until you can see the "SYDNEY TOWER" location marker
select the "Go to Blacktown" button
now the map only shows a grid and no tiles are loaded
now you can alternate locations via the "Go to Sydney" and "Go to Blacktown" buttons - Sydney shows map tiles, Blacktown doesn't

Could not reproduce. Closing. Please reopen when the issue persists.


Generate client test vectors

Copying test vectors from debug logs is tedious and error prone.
This PR generates them dynamically and saves them to json files
which are easilly imported by other implementations which seek
interoperability tests.

Codecov Report

Merging #951 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #951   +/-   ##
=======================================
  Coverage   50.33%   50.33%
=======================================
  Files          34       34
  Lines        2573     2573
=======================================
  Hits         1295     1295
  Misses       1040     1040
  Partials      238      238

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 32cc187...1584969. Read the comment docs.

Thank you for pushing back on the initial PR.
Moved test vector generation into a separate cmd/gen-test-vectors program.
Test vectors can now be generated by go generate ./core/testdata
Ready for another look!


NSG attached to NIC

Hello,
We have a customer that has requirement to have NSG attached at NIC level in addition to Subnet.
Customer understands the management complexity and would work on a strategy to move away from NIC level NSG's but to deploy VM's in Azure, currently they are asking for this capability.
[High Priority]

Closing issue as merged into master.


fail to change device mode or toggle

Hi, I am trying to  add fan with 'mode' trait, but I can not change its 'mode' or 'toggle' from the simulator.
For example, when I input "set my living room fan fan speed to level three", the simulator always respond "Sorry, this action is not available in simulation".
when I input "set my living room fan oscillation on", the simulator still respond "Sorry, this action is not available in simulation".
But I can successfully power on/off my fan.
Here is my sync response for a fan device
 {
            "type":"action.devices.types.DRYER",  // There is no devices type fan, so I use DRYER type instead...
            "traits":[
               "action.devices.traits.OnOff",
               "action.devices.traits.Modes",
               "action.devices.traits.Toggles"
            ],
            "name":{
               "defaultNames":[
                  "Dyson Fan"
               ],
               "name":"fan",
               "nicknames":[
                  "living room fan"
               ]
            },
            "willReportState":true,
            "roomHint":"living room",
            "deviceInfo":{
               "manufacturer":"Smart Home Provider",
               "model":"g1338",
               "swVersion":"1.0.31",
               "hwVersion":"1.1"
            },
            "attributes":{
               "availableModes":[
                  {
                     "name":"fan_speed",
                     "name_values":[
                        {
                           "name_synonym":[
                              "speed",
                              "fan speed",
                              "level",
                              "fan speed level"
                           ],
                           "lang":"en"
                        }
                     ],
                     "settings":[
                        {
                           "setting_name":"one",
                           "setting_values":[
                              {
                                 "setting_synonym":[
                                    "one",
                                    "1",
                                    "level one"
                                 ],
                                 "lang":"en"
                              }
                           ]
                        },
                        {
                           "setting_name":"two",
                           "setting_values":[
                              {
                                 "setting_synonym":[
                                    "two",
                                    "2",
                                    "level two"
                                 ],
                                 "lang":"en"
                              }
                           ]
                        },
                        {
                           "setting_name":"three",
                           "setting_values":[
                              {
                                 "setting_synonym":[
                                    "three",
                                    "3",
                                    "level three"
                                 ],
                                 "lang":"en"
                              }
                           ]
                        },
                        {
                           "setting_name":"four",
                           "setting_values":[
                              {
                                 "setting_synonym":[
                                    "four",
                                    "4",
                                    "level four"
                                 ],
                                 "lang":"en"
                              }
                           ]
                        }
                     ],
                     "ordered":true
                  }
               ],
               "availableToggles":[
                  {
                     "name":"oscillate",
                     "name_values":[
                        {
                           "name_synonym":[
                              "oscillate",
                              "oscillation"
                           ],
                           "lang":"en"
                        }
                     ]
                  }
               ]
            },
            "customData":{
               "smartHomeProviderId":"SGs84HiURHSb6ggrGjiv9jJ3t7U4ze"
            },
            "id":"4"
         }

I'll look at this after the break. It might be the way you're giving commands is slightly incorrect.

Same issue here, the payload missing the 'params'
{ uid: '59c7a80d27e03a44449accb2',
  requestId: '2695982391929483863',
  commands:
   [ { devices: [ { id: 'klima' } ],
       execution: [ { command: 'action.devices.commands.SetToggles' } ] } ] }

The "execution" object should have a 'params' according to https://developers.google.com/actions/smarthome/traits/toggles#request but it just dont have it.
(I was unable to give command to control any modesetting btw..)

How are you getting that object above?

sorry, its logged by https://github.com/actions-on-google/actionssdk-smart-home-nodejs/blob/master/smart-home-provider/smart-home-app.js#L437

So it looks like that you are getting something @asturel, in https://github.com/actions-on-google/actionssdk-smart-home-nodejs/blob/master/smart-home-provider/smart-home-app.js#L117 you do get the execute intent. I am unsure what's going on with the payload. Can you log the entire object you receive from line 117 with JSON.stringify so I can check out what's happening?
@JudyHsiao can you try a statement with the same syntax that @asturel used? I want to see if the problem is related to your grammar or a problem on your server.

the entire reqdata (request.body):
{ inputs:
   [ { intent: 'action.devices.EXECUTE',
       payload:
        { commands:
           [ { devices: [ { id: 'klima' } ],
               execution: [ { command: 'action.devices.commands.SetToggles' } ] } ] } } ],
  requestId: '5734843749710602213' }

From the docs:
"The user can set a mode such as temperature explicitly with a command like Set the washer’s temperature to cold."
Does that work?
How many toggles does your device have?

2 toggles (here's the sync output https://pastebin.com/5rr5Efuf ), 'Set the klima's speed to low.' doesn't works, it's just a google search. :(

Hey @asturel I was looking at the docs and unfortunately there's currently not support for custom mode names.
"Currently, you must submit names to us so we can add support for them."
So if we don't have speed or a speed of low those commands won't work.
The same is true for toggles.

What a bummer. So how can we know what names are supported (besides the example)?

Both 'turn on oscillate for living room fan.' and 'Set the living room fan's speed to three.' get this response 'Sorry, this action is not available in simulation'.
How do we  submit setting names to you so you can add support for them?

Yes, settings/setting_name seems to have the same limitation.
I guess the best way is to file a new issue on this project. I'll redirect the request internally for the time being.

Are we eligible to request new 'names' even if the project isn't published (and probably never will? so is it gonna be added because of 1 user?)? Will this change in the future (so we don't have to request new names)?

Is there anywhere that permitted synonyms/names are listed?

The doc listed above will have the ones that definitely should work. I will try to find a more comprehensive list.

So you didnt found a list about the available modes/toggles?

We will not be publishing a full list at this time.


Layout: make Attributes table in PDP bearable (width, some spacing)

added width to table and padding to cells


Avoid crashes from conflicts between BPBReachability and CDVReachability

Build https://github.com/SDA-SE/kundenapp-ui/pull/546 including the reference to this fork for cordova-plugin-network (this is the case in the branches' package.json, so no need to change anything)
with fotoapp as PLUGIN= in your local .env file.
Ensure no crashes occur when running the app and at any point changing from or to airplane mode.


How to send a voice/audio to a user from the app?

How do I send an audio/voice to a user from the app so that it appears like this:

video would be good too.

Hi @master0v, as long as you use common file formats, all you need to do is just to upload your audio/video files using files.upload API!
import logging, os
logging.basicConfig(level=logging.DEBUG)

from slack_sdk.web import WebClient
client = WebClient(token=os.environ["SLACK_BOT_TOKEN"])

channels = ",".join(["#random"])

# Upload an audio file to #random channel
filepath = "./test-recording.m4a"
response = client.files_upload(channels=channels, file=filepath)

# Upload a video file to #random channel
filepath = "./test-video.mov"
response = client.files_upload(channels=channels, file=filepath)

By doing above, you will see the preview as below:

If the automatic file format detection does not work for your files, you can set type etc. in the files.upload` API call. Refer to https://api.slack.com/methods/files.upload for available parameters and more details.
If everything is clear, would you mind closing this issue?

Hi Kazuhiro Sera,
Thank you for the explanation and for the code sample.  My app is going to be sending the same file to thousands of users.  Is there a way to upload it to slack once, and then send it to users in a message by file id?
Thanks,
Alexei

@master0v As Block Kit supports only remote files so far, the only way I can suggest is to have a file's permalink in your message:
from slack_sdk.web import WebClient
client = WebClient(token=os.environ["SLACK_BOT_TOKEN"])

upload_response = client.files_upload(file="./tmp.txt")
file_permalink = upload_response["file"]["permalink"]

message_response = client.chat_postMessage(
    channel="C12345",
    text=f"Here is the uploaded file: {file_permalink}",
)

This should work as you expect. Let me know if you have anything further to ask.

That unfortunately shows the full url of the permalink, which is kind of ugly.  Is there any way to hide it?

You can use a special syntax in message formatting: https://api.slack.com/reference/surfaces/formatting#linking-urls
Doing like f"Here is <{file_permalink}|the uploaded file>" can improve this.

Thanks.  Do you think I can use files_remoteAdd and files_remoteShare to improve the visual presentation?

(i.e. pretend that the permalink is actually a remote file)

First off, the files.remote.* APIs are designed for external (=not slack.com) URLs. For this reason, we don't recommend it.
Also, trying to do so won't improve the presentation as the remote file API allows having only preview image. The audio/video players are not supported, and, as far as I know, there is no plan to add them in the short term.

understood.  Thanks for your help.   Will try to do something with markdown to hide the URL as you suggested.


Update ReadMe And How to build a duck

Changed the way we use the ReadMe and How to build a Duck
With the changes made over the last couple of days, we now have support for PlatformIO.
By adding #30 we now can reference installation instructions to  doc/how to build a duck and clean up the main Readme
Added Installation Section

Moved Arduino Ide Installation to How to build a Duck
Moved PlatformIO  Installation to How to build a Duck
Moved IBM Watson IoT platform  Installation to How to build a Duck
Fix Project OWL Website links

Updated How to build a duck

Added installation for Submodule Libraries
Added PlatformIO section
-Added IBM Watson IoT section
Updated Duck Pictures

Just three general comments:

Currently you have at least 3 different sources for assets: this repo, the repo over from project owl and the project ow website. It would probably make sense to have all assets centralized here in case something is not reachable or has changed

Personally, I don't think it's a good idea to do an API description by hand. There are several tools to do that. I made a suggestion in PR #31 . Doxygen is used. This has several advantages: The user can see the comments in the IDE while developing his ducks, for you as a developer the documentation is created where it takes place, namely directly at the API function. Doxygen also notices when a parameter changes and warns if it has not been documented yet etc.

To keep your repo small you should think about Git LFS (for pictures, PDFs, zips, ...)

@Salzi very good points
I will change the Files and ad LFS , for the second one I will pull in @nfeuer and take a close look at your PR

I had the feeling that the team was not completely convinced of the auto generated solution. Because you have to upload the files to a server by hand. Therefore I played a bit with github action. Here is my solution (would recommend to integrate it here instead of having another PR).
Change the file .github/workflows/documentation.yml to
name: Documentation

on: push

jobs:
  convert_via_pandoc:
    runs-on: ubuntu-18.04
    steps:
      - uses: actions/checkout@v2
      - run: mkdir output
      - uses: Ender-events/pandoc@master
        with:
          args: "--resource-path=doc -f markdown-implicit_figures doc/howToBuildADuck.md --pdf-engine=xelatex  -V \"mainfont:DejaVu Sans\" -V \"geometry: left=2cm,right=2cm,top=3cm,bottom=3cm\" -o output/howToBuildADuck.pdf"
      - uses: docker://michaelperrin/pdfunite:0.48.0
        with:
          args: "doc/assets/pdfs/title.pdf output/howToBuildADuck.pdf output/final.pdf"
      - run: rm output/howToBuildADuck.pdf
      - run: cp output/final.pdf doc/HowToBuildADuck.pdf
      - uses: actions/upload-artifact@master
        with:
          name: output
          path: output
      - run: rm -rf output
      - uses: mikeal/publish-to-github-action@master
        if: github.ref == 'refs/heads/master'
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

The main difference is the last step:
      - uses: mikeal/publish-to-github-action@master
        if: github.ref == 'refs/heads/master'
        env:
          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

This will push the generated pdf back to the repository but only when running on the master branch. So basically you create a branch, edit the how to and push it to the repository. The github action will generate the pdf and stores it as artifact. You can download it and check it. If everything is ok merge it with master and the action will push the pdf to your master branch. You can see the running setup here (check Automated Publisher). So on your websites you can always link to the pdf on your master branch and it would be always up to date.
With this solution it's even more important to use git lfs. So at least at a .gitattributes file with:
*.pdf filter=lfs diff=lfs merge=lfs -text

to your repository. (Better would be to add all other binary formats too. Like: *.png, *.jpg, *.jpeg, *.psd ...)
Also consider using git lfs migrate to clean up your repository.

Hi @Salzi seems like a great solution let me get on it now

[ ] Install and migrate to LFS

[ ] Test and validate Pandoc pdf generator

[ ] Update image sources


Add a "latest" pointer on the CDN to avoid having to update versions manually

Not sure if this is a good or bad idea, but I was thinking it might be useful to just point to a latest version of raven.js instead of having to manually bump versions each time a new version is released?
If this is already available, just ignore me.

So, one reason we've avoided this is because we follow semver, and we've introduced "breaking" API behavior in Raven.js in the past (although very rarely, and typically very conservatively).
It would be nice to allow people to say, /latest/3.x/, /latest/2.x/ or something.

Yeah the latest/major version seems nice.  Basically take the bug fixes as them come, but breaking changes we have to opt into.

Duplicate of https://github.com/getsentry/raven-js/issues/911


Issues with IFC geometry

Geometry imported with the IFC import leads to issues with Base objects with displayValue not being converted.
Since the displayValue prop is a dynamic List not a IEnumerable<Base>.
EDIT: ^ is the correct reason why it's failing? need to double check

Closed as linked pr has been merged.


Make final iteration optional

When one is not interested in actual expression counts, this justs eats up unnecessary CPU hours.

This is already possible, just run igdiscover run nofinal. Should we make this the default?

Yes, make it the default.

I’ve heard different opinions from different users, so I’m not going to change it for now.


[README] state (optional) dependencies

... and which features they leverage.

this should also include info on the git submodules and how to bring those in...


Upgraded reference in sample projects from version 4 to 5

As mentioned in issue #13 the samples have references to version 4. When installing version 5 directly (without version 4) Visual Basic upgrades the OCX from version 4 to version 5 but cannot upgrade the tlb reference, thus the project can't run.

Actually, this doesn't solve issue #13. Doing another pull request to fix that one.

Thank you for you help!


Mark tests that hardlink/copy Swift executables as unsupported when...

…running with "tensorflow" configuration.
A follow-up to https://github.com/apple/swift/commit/ff49d78601cdc274e163a0980f2acca934e6cb35.

Merging without running tests because this test failure occurs only on macOS.