fix(general): fixed unused deps and updated them

Only updated package.jsons dependencies, ran depcheck to delete unused packages and tests
Resolves #541
Resolves #542
Resolves #543
Resolves #559
Resolves #560
Resolves #561
Signed-off-by: Enrique Alcázar<EMAIL_ADDRESS>

Latest version of  fabric  SDK (1.4.17) breaks compatibility for fabric 1.4.x and 2.x due to a hubs timeout error.
packages/cactus-plugin-ledger-connector-fabric/src/test/typescript/integration/fabric-v1-4-x/run-transaction-endpoint-v1.test.ts 1> 2021-02-15T22:41:35.825Z - warn: [TransactionEventHandler]: _strategyFail: strategy fail for transaction "3fcba282f505c36728e693b36940c2d7397fcddb598cf0fa130f12647d5ed849": TimeoutError: Event strategy not satisfied within the timeout period. No response received from event hubs: peer0.org1.example.com, peer1.org1.example.com:8051, peer0.org2.example.com:9051, peer1.org2.example.com:10051
packages/cactus-plugin-ledger-connector-fabric/src/test/typescript/integration/fabric-v1-4-x/run-transaction-endpoint-v1.test.ts 1>     at TransactionEventHandler._timeoutFail (/mnt/c/Users/enriq/Accenture/Cactus/packages/cactus-plugin-ledger-connector-fabric/node_modules/fabric-network/lib/impl/event/transactioneventhandler.js:101:17)
packages/cactus-plugin-ledger-connector-fabric/src/test/typescript/integration/fabric-v1-4-x/run-transaction-endpoint-v1.test.ts 1>     at Timeout._onTimeout (/mnt/c/Users/enriq/Accenture/Cactus/packages/cactus-plugin-ledger-connector-fabric/node_modules/fabric-network/lib/impl/event/transactioneventhandler.js:78:15)
packages/cactus-plugin-ledger-connector-fabric/src/test/typescript/integration/fabric-v1-4-x/run-transaction-endpoint-v1.test.ts 1>     at listOnTimeout (internal/timers.js:554:17)
packages/cactus-plugin-ledger-connector-fabric/src/test/typescript/integration/fabric-v1-4-x/run-transaction-endpoint-v1.test.ts 1>     at processTimers (internal/timers.js:497:7) {
packages/cactus-plugin-ledger-connector-fabric/src/test/typescript/integration/fabric-v1-4-x/run-transaction-endpoint-v1.test.ts 1>   transactionId: '3fcba282f505c36728e693b36940c2d7397fcddb598cf0fa130f12647d5ed849'
packages/cactus-plugin-ledger-connector-fabric/src/test/typescript/integration/fabric-v1-4-x/run-transaction-endpoint-v1.test.ts 1> }
 FAIL  packages/cactus-plugin-ledger-connector-fabric/src/test/typescript/integration/fabric-v1-4-x/run-transaction-endpoint-v1.test.ts
 ✖ Error: Request failed with status code 500

Moved everything to the latest version we used 1.4.11

Related issue is the one where we'd start executing the full CI script in the GH actions (if that issue was already resolved then the CI would've caught this in the PR automatically as well)
https://github.com/hyperledger/cactus/issues/497


Crash when in the url there is no streaming content

Describe the bug
The app crashes when i put an url that has no streaming content (like www.google.com) in IOS on release. If i use an invalid url like "hello" it doesnt happen
To Reproduce
Use the code and run the app on release mode on IOS
Environment (please complete the following information):

System:
OS: macOS 10.15.3
CPU: (8) x64 Intel(R) Core(TM) i7-4870HQ CPU @ 2.50GHz
Memory: 786.83 MB / 16.00 GB
Shell: 5.7.1 - /bin/zsh
Binaries:
Node: 13.10.1 - ~/.nvm/versions/node/v13.10.1/bin/node
Yarn: 1.22.1 - ~/.nvm/versions/node/v13.10.1/bin/yarn
npm: 6.13.7 - ~/.nvm/versions/node/v13.10.1/bin/npm
Watchman: 4.9.0 - /usr/local/bin/watchman
Managers:
CocoaPods: 1.9.1 - /usr/local/bin/pod
SDKs:
iOS SDK:
Platforms: iOS 13.4, DriverKit 19.0, macOS 10.15, tvOS 13.4, watchOS 6.2
Android SDK:
Android NDK: 21.1.6352462
IDEs:
Android Studio: 3.6 AI-192.7<IP_ADDRESS>92135
Xcode: 11.4.1/11E503a - /usr/bin/xcodebuild
Languages:
Java: javac 12 - /usr/bin/javac
Python: 2.7.16 - /usr/bin/python
npmPackages:
@react-native-community/cli: Not Found
react: 16.11.0 => 16.11.0
react-native: 0.62.2 => 0.62.2
npmGlobalPackages:
react-native: Not Found

What react-native-track-player version are you using?
Im using the 1.2.3 version
Are you testing on a real device or in the simulator? Which OS version are you running?
It happens on real device and in the simulator
Code
const prueba = {
      id: 1,
      artwork: 'https://softfabrics.com.mx/images/galleria/factura/01.jpg',
      title: 'prueba',
      artist: 'artist',
      url: 'www.google.com'
    };
    await TrackPlayer.add(prueba);
    await TrackPlayer.play();

@fcaride , can you test the same on Android?
You really shouldn't be providing URL's that are not audio content, but it would be nice to protect the app from crashing in this case.

In android it doesnt crashes

ok, nice. Good to know. Thanks.

My bad its happening in Android release also

Looking at crashalytics im getting this error message.
Fatal Exception: com.facebook.react.common.d
ReferenceError: Property 'Event' doesn't exist, js engine: hermes, stack: anonymous@1:962306 anonymous@1:511350 value@1:62459 value@1:57754 anonymous@1:56286 value@1:57352 value@1:56244

In the previous version it didnt happen (1.2.2)

Can you post the playback-error callback you are using?

I found what was happening!. I was using the class Event of the v2.0 and for one reason it only was crashing on release. Thank you so much. Sorry for your trouble!!


feat(js): scroll to top when query changes

Description
This scrolls to the top of the panel when the query changes (i.e., when new results come in) so that users don't have to.
How
We introduce a new CSS utility class in the theme called .aa-Panel--Scrollable which hints Autocomplete that this is the container responsible for displaying the list of items.
By default, this CSS class is attached to .aa-PanelLayout, but when we introduce the Preview Panel plugin, it will be attached to the left child of .aa-PanelLayout.
Whenever the query changes, we scroll to the top of this scrollable container.
Preview
Before

After

Could it make sense to expose this as a boolean (e.g. scrollToTopOnQueryUpdate)? I feel like this could be something that the user want to toggle in some special cases

@shortcuts When would you want to not have this behavior?

@francoischalifour in some e-commerce cases I guess. I could imagine some side-by-side sections (like https://www.ldlc.com/) where you'd probably not want to scroll all of them but handle it separately.

@shortcuts That's exactly why we have this .aa-Panel--Scrollable CSS class, you can add it to the containers you need 🙂
That makes me think that we should use document.querySelectorAll to scroll to the top of all the containers marked with this CSS class.


Use secure https in Location URLs for redirects

This change makes https://httpstat.us (secure HTTPS) the Location URL for all
https://httpstat.us/3xx responses, rather than http://httpstat.us/ (insecure
HTTP). Rationale: For any requests made to https://httpstat.us/3xx URLs from
frontend JavaScript running in a browser, the browser will block the request and
emit a “Mixed Content” error message. For a test case, see the last part of the
Stack Overflow answer at https://stackoverflow.com/a/46031800/441757

Hi @aaronpowell—Much thanks for merging this so fast! (And come back to visit Tokyo again soon)

Thanks! Deployed :)

w00t—you do quick work, chief

One of these days @sideshowbarker, one of these days 😉


hyperstart should not end when rescan scsi fails

With this patch an error will be reported when re-scan
scsi fails, allowing hyperstart continue with its work
Signed-off-by: Julio Montes<EMAIL_ADDRESS>

hyperstart fails in many cases, some are non-important or have workaround.
we will cleanup them.

Hi,
it seems you don't need scsi block device as container root device?

Hi @gao-feng
that's right

@gao-feng please take a look to this new change

lgtm


[Gally]: master <- dev

Automatically created by Git-Ally

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

npm package (@latest dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


lint: limit max return values from a function to 4

Background
See an example of where this has grown out of control and was later fixed:
https://github.com/osmosis-labs/osmosis/pull/5983
To prevent so many return values causing decrease in readability, we should add a linter
Acceptance Criteria

add linter
document style guidelines in CONTRIBUTING.md
Use https://github.com/osmosis-labs/osmosis/pull/5983 as an example

These examples need fixing prior to adding the linter:
x/twap/client/cli/query.go:152:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 5 (revive)
func twapQueryParseArgs(args []string) (poolId uint64, baseDenom string, startTime time.Time, endTime time.Time, err error) {
^
simulation/executor/legacyconfig.go:118:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 5 (revive)
func SetupSimulation(dirPrefix, dbName string) (cfg Config, db dbm.DB, logger log.Logger, cleanup func(), err error) {
^
x/concentrated-liquidity/math/tick.go:16:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 5 (revive)
func TicksToSqrtPrice(lowerTick, upperTick int64) (sdk.Dec, sdk.Dec, sdk.Dec, sdk.Dec, error) {
^
x/concentrated-liquidity/position.go:396:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 5 (revive)
func (k Keeper) CreateFullRangePosition(ctx sdk.Context, poolId uint64, owner sdk.AccAddress, coins sdk.Coins) (positionId uint64, amount0, amount1 sdk.Int, liquidity sdk.Dec, err error) {
^
x/concentrated-liquidity/position.go:427:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 6 (revive)
func (k Keeper) CreateFullRangePositionLocked(ctx sdk.Context, clPoolId uint64, owner sdk.AccAddress, coins sdk.Coins, remainingLockDuration time.Duration) (positionId uint64, amount0, amount1 sdk.Int, liquidity sdk.Dec, concentratedLockID uint64, err error) {
^
x/concentrated-liquidity/position.go:447:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 6 (revive)
func (k Keeper) CreateFullRangePositionUnlocking(ctx sdk.Context, clPoolId uint64, owner sdk.AccAddress, coins sdk.Coins, remainingLockDuration time.Duration) (positionId uint64, amount0, amount1 sdk.Int, liquidity sdk.Dec, concentratedLockID uint64, err error) {
^
x/concentrated-liquidity/swaps.go:339:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 5 (revive)
func (k Keeper) computeOutAmtGivenIn(
        ctx sdk.Context,
        poolId uint64,
        tokenInMin sdk.Coin,
        tokenOutDenom string,
        spreadFactor sdk.Dec,
        priceLimit sdk.Dec,
) (tokenIn, tokenOut sdk.Coin, poolUpdates PoolUpdates, totalSpreadFactors sdk.Dec, err error) {
x/concentrated-liquidity/lp.go:35:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 7 (revive)
func (k Keeper) CreatePosition(ctx sdk.Context, poolId uint64, owner sdk.AccAddress, tokensProvided sdk.Coins, amount0Min, amount1Min sdk.Int, lowerTick, upperTick int64) (positionId uint64, actualAmount0 sdk.Int, actualAmount1 sdk.Int, liquidityDelta sdk.Dec, lowerTickResult int64, upperTickResult int64, err error) {
^
x/concentrated-liquidity/swaps.go:468:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 5 (revive)
func (k Keeper) computeInAmtGivenOut(
        ctx sdk.Context,
        desiredTokenOut sdk.Coin,
        tokenInDenom string,
        spreadFactor sdk.Dec,
        priceLimit sdk.Dec,
        poolId uint64,
) (tokenIn, tokenOut sdk.Coin, poolUpdates PoolUpdates, totalSpreadFactors sdk.Dec, err error) {
x/concentrated-liquidity/position.go:663:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 5 (revive)
func (k Keeper) validatePositionsAndGetTotalLiquidity(ctx sdk.Context, owner sdk.AccAddress, positionIds []uint64, fullyChargedDuration time.Duration) (uint64, int64, int64, sdk.Dec, error) {
^
x/concentrated-liquidity/lp.go:407:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 5 (revive)
func (k Keeper) UpdatePosition(ctx sdk.Context, poolId uint64, owner sdk.AccAddress, lowerTick, upperTick int64, liquidityDelta sdk.Dec, joinTime time.Time, positionId uint64) (sdk.Int, sdk.Int, bool, bool, error) {
^
x/concentrated-liquidity/simulation/sim_msgs.go:325:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 6 (revive)
func RandomPreparePoolFunc(sim *osmosimtypes.SimCtx, ctx sdk.Context, k clkeeper.Keeper) (sdk.AccAddress, sdk.Coin, sdk.Coin, uint64, sdk.Dec, error) {
^
x/concentrated-liquidity/simulation/sim_msgs.go:360:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 5 (revive)
func RandomPrepareCreatePositionFunc(sim *osmosimtypes.SimCtx, ctx sdk.Context, clPool cltypes.ConcentratedPoolExtension, poolDenoms []string) (sdk.AccAddress, sdk.Coins, int64, int64, error) {
^
x/gamm/keeper/migrate.go:20:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 7 (revive)
func (k Keeper) MigrateUnlockedPositionFromBalancerToConcentrated(ctx sdk.Context,
        sender sdk.AccAddress, sharesToMigrate sdk.Coin,
        tokenOutMins sdk.Coins,
) (positionId uint64, amount0, amount1 sdk.Int, liquidity sdk.Dec, poolIdLeaving, poolIdEntering uint64, err error) {
x/gamm/simulation/sim_msgs.go:417:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 7 (revive)
func getRandPool(k keeper.Keeper, sim *simtypes.SimCtx, ctx sdk.Context) (uint64, types.CFMMPoolI, sdk.Coin, sdk.Coin, []string, string, error) {
^
x/superfluid/keeper/migrate.go:49:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 8 (revive)
func (k Keeper) RouteLockedBalancerToConcentratedMigration(ctx sdk.Context, sender sdk.AccAddress, providedLockId int64, sharesToMigrate sdk.Coin, tokenOutMins sdk.Coins) (positionId uint64, amount0, amount1 sdk.Int, liquidity sdk.Dec, poolIdLeaving, poolIdEntering, concentratedLockId uint64, err error) {
^
x/superfluid/keeper/migrate.go:79:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 8 (revive)
func (k Keeper) migrateSuperfluidBondedBalancerToConcentrated(ctx sdk.Context,
        sender sdk.AccAddress,
        originalLockId uint64,
        sharesToMigrate sdk.Coin,
        synthDenomBeforeMigration string,
        tokenOutMins sdk.Coins,
) (positionId uint64, amount0, amount1 sdk.Int, liquidity sdk.Dec, concentratedLockId, poolIdLeaving, poolIdEntering uint64, err error) {
x/superfluid/keeper/migrate.go:134:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 8 (revive)
func (k Keeper) migrateSuperfluidUnbondingBalancerToConcentrated(ctx sdk.Context,
        sender sdk.AccAddress,
        lockId uint64,
        sharesToMigrate sdk.Coin,
        synthDenomBeforeMigration string,
        tokenOutMins sdk.Coins,
) (positionId uint64, amount0, amount1 sdk.Int, liquidity sdk.Dec, concentratedLockId, poolIdLeaving, poolIdEntering uint64, err error) {
x/superfluid/keeper/concentrated_liquidity.go:32:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 6 (revive)
func (k Keeper) addToConcentratedLiquiditySuperfluidPosition(ctx sdk.Context, sender sdk.AccAddress, positionId uint64, amount0ToAdd, amount1ToAdd sdk.Int) (uint64, sdk.Int, sdk.Int, sdk.Dec, uint64, error) {
^
x/superfluid/keeper/migrate.go:190:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 8 (revive)
func (k Keeper) migrateNonSuperfluidLockBalancerToConcentrated(ctx sdk.Context,
        sender sdk.AccAddress,
        lockId uint64,
        sharesToMigrate sdk.Coin,
        tokenOutMins sdk.Coins,
) (positionId uint64, amount0, amount1 sdk.Int, liquidity sdk.Dec, concentratedLockId, poolIdLeaving, poolIdEntering uint64, err error) {
x/superfluid/keeper/migrate.go:263:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 5 (revive)
func (k Keeper) validateMigration(ctx sdk.Context, sender sdk.AccAddress, lockId uint64, sharesToMigrate sdk.Coin) (poolIdLeaving, poolIdEntering uint64, preMigrationLock *lockuptypes.PeriodLock, remainingLockTime time.Duration, err error) {
^
tests/e2e/configurer/chain/queries.go:446:1: function-result-limit: maximum number of return results per function exceeded; max 4 but got 5 (revive)
func (n *NodeConfig) QueryPropTally(proposalNumber int) (sdk.Int, sdk.Int, sdk.Int, sdk.Int, error) {

Started adding linter on this branch: https://github.com/osmosis-labs/osmosis/tree/roman/max-return-linter

Can revive automatically fix these out of curiosity?

I was unable to find such an option. I think it would require some more complex edge case handling, something AST-like.
From doing this manually, I think it's hard to construct a deterministic set of rules that would apply to every usage of the function being refactored.


StringNonEmpty validator

hmm you did changes to just to include this new validator but there is a diff for the whole file, this makes the review harder. Can you investigate why the whole file is being changed? I thought it was a conversion from spaces to tabs but doesn't look like


New App: Tidbyt App Track

Description
This app tracks and displays apps available for the Tidbyt, as well as any new apps that come in.
Copilot

🤖[deprecated] Generated by Copilot at f680bdd
Summary
🚀📱🐍

This pull request adds a new applet called Tidbyt App Track, which shows new apps on the Tidbyt app page. It includes a manifest file apps/tidbytapptrack/manifest.yaml and a script file apps/tidbytapptrack/tidbyt_app_track.star that implement the applet functionality and configuration. The script file uses Starlark and the Tidbyt API.

New applet for Tidbyt
manifest and script files
Autumn of listings

Walkthrough

Create and implement a new applet for tracking the apps on the Tidbyt app page (link, link)

Add the manifest file manifest.yaml with the applet metadata (link)
Add the main script file tidbyt_app_track.star with the applet logic and schema (link)

Preview:


Switch PhEDEx injector tests to mock for SiteDB and DBS

Don't merge yet

Ok @ticoann this can be merged too, it looks like. Aside from the big one, which is WorkQueue, this completes the SiteDB portion of this. @alexanderrichards is working on WorkQueue right?

Yes Alex is working on WorkQueue. Thanks,


JSON to Code

Is your feature request related to a problem? Please describe.
As a web developer, it common to have to convert a large JSON to C# classes in order to consume certain web API.
Describe the solution you'd like
Add a JSON to C#, TypeScript and Java tool.

Could I work on this?

With pleasure!
Some guidance:

Feel free to make your own solution for the conversion, or take example on existing implementation on GitHub. Just be mindful of the license. I'd like to keep DevToys under MIT license, so I'd avoid anything that has a license requiring us to publish DevToys under the same than their.
Keep things simple, for now. I'd prefer shipping a converter with a minimal set of languages and options for it than delivering a lot options right at the beginning. For example, for C#, I think the following options to start should be enough:

Should array be treated as [] or List
Should properties support nullable values or not

Follow the coding style guidance here: https://github.com/veler/DevToys/blob/main/CONTRIBUTING.md#coding
Let's start with JSON to C# and TypeScript first. Let's see in a future iteration for other languages like Java, Python or C++
Let's support Smart Detection feature. Here is an example: https://github.com/veler/DevToys/blob/d6cf9414110d9fecf4c2c44af8df74074ea1def4/src/dev/impl/DevToys/ViewModels/Tools/JsonFormatter/JsonFormatterToolProvider.cs#L29-L32
When designing the UI

Consider using the monaco editor like in Json Formatter or Json <> Yaml tool.
If you need more than 3 options in the UI, consider an expandable one, like in Regex Tester tool
Make sure to test the UI in compact mode, maximized window and small (width and height) window side.

Don't hesitate to ask questions here or ping me on Discord :)

Visual Studio already has an option for this under the Edit menu.

@Jaiganeshkumaran , thank you :)
What you're saying is true, but it lacks a few options to define, for example, whether we want PascalCase, camelCase, and more options that we can find on websites online.
We're trying to bring the power of these online service into an offline app, not to limit ourselves to what Visual Studio already does.

Hi guys! Is there any chance to see Class to JSON converter? I'd love to use it for API testing

By Class to Json, to you mean generating mocked JSON value from a class definition?

Exactly. I see this as a input window for my C# class and (maybe) a JSON constructor where I can pick values I need in my JSON.
I mentioned that because I often test my APIs by Postman and I always struggle with converting my API's input model into JSON especially a large ones.
That initial conversion can be done by Swagger, but Swagger generates model as full as it can be, so I have to manually clean it and that can be very time consuming

I kinda implemented it with NJsonSchema on a sample console app but the output is messy, still have to port it to the app.

[December 2022 update]
This could be a potential paid feature for DevToys 2.0.
UX could be as simple as:
Input:
JSON
Output:
TypeScript, Python, Java, C#, PHP, C++, Ruby, Go, Dart, Rust, Swift
Settings:
They would be specific to each output language selected. For example, for C#, we could allow to chose whether a [JSONProperty] attribute should be added.

support added with Devtoys.JsonToCSharp extension for DevToys 2.0.1


TargetedPatchingOptOut attributes are irrelevant for the documentation

The documentation should not include TargetedPatchingOptOut attributes on API signatures. These attributes were internal implementation plumbing for .NET Framework. They do not exist in .NET Core at all.

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

ID: eb7b63ce-50ba-fdb1-a002-58e127ecf4e1
Version Independent ID: 38a2e573-13d6-f730-a8d5-93b711b1715b
Content: UriBuilder.Path Property (System)
Content Source: xml/System/UriBuilder.xml
Product: dotnet-api
GitHub Login: @dotnet-bot
Microsoft Alias: dotnetcontent

Thanks @jkotas. We have a PR already trying to remove that: https://github.com/dotnet/dotnet-api-docs/pull/439

Also, showing the right attributes for the different implementations is on the pipeline of the engineering team. That will happen in a few months.


Feat(database)

Description

Validation

Related Issues

Check List

[ ] I have formatted using npx turbo format
[ ] I have checked the types using npx turbo type-check
[ ] I have tested the build using npx turbo build
[ ] I have covered the code with tests if necessary

@AugustinMauroy Could you check the source code out. I am not really familiar with Next. It seems like it is importing the files again instead of just referencing the current objects. Every request takes currently around 5 seconds.


Create types instead of using tuples for index

For instance, change NoteCollectionIndex and others like it to a struct instead of a tuple of values.

I'd be happy to help with some contributions and if possible I'd like to join the Slack channel as massimobio gmail.
These simpler issues are a good way to try and get familiar with the code base.
I just pushed a PR #123 that converts the NoteCollectionIndex from tuple to Struct.
Any other tuples come to mind that I could tackle?

Closed by #123


DESY parser does not strip auther id value

For DESY workflows, author ORCID ids are incorrect.
https://orcidorg/0000-0001-7325-2274 should be 0000-0001-7325-2274.
https://inspirehep.net/holdingpen/3174764
ValidationError: {u'value': u'https://orcidorg/0000-0001-7325-2274', u'schema': u'ORCID'} is not valid under any of the given schemas

@oguzdemirbasci there's nothing we can do here. All we're doing on our side is converting the metadata from MARCXML to JSON. This needs to be handled by @fschwenn.


Update to NUnit3.5

Updated nuget package "NUnit" to 3.5.

ExpectedExceptionAttribute into Assert.Throws<T>
Added arguments to IgnoreAttribute
Installed nuget package Microsoft.AspNet.Providers.Core in order to add reference to the missing System.Web.Providers.dll
Changed the target framework version of some projects to 4.5.2
SetUpAttribute and TearDownAttribute into OneTimeSetUpAttribute and OneTimeTearDownAttribute
Small refactorings etc.

Thanks for the pull request but there are a few issues that make me not able to accept it. First its to large. I would need a pull request for each section. For example the the nunit upgrade and Assert.Throws in a single pull request. Second, not sure why the Providers.Core is needed. Third raising the target version is a big deal when hundreds of people work with the source code so has to be taken slowly and Isolated.


mattersip not working

When I compiling and installiing this plagin trough web-console, I'm getting an error.
"unable to start plugin: must at least have a web app or server component"

Sorry, at this plugin has not been worked for 5 years, we ourselves do not use an asterisk system anymore.


hist: PR followups

Fixes up some follow-on comments from 0.
Also fixes a compile time warning:
[25/85] Building CXX object src/CMakeFiles/runtime.dir/output.cpp.o In file included from /usr/include/c++/13.2.1/cassert:44,
                 from /home/dxu/dev/bpftrace/src/log.h:3,
                 from /home/dxu/dev/bpftrace/src/output.cpp:3:
/home/dxu/dev/bpftrace/src/output.cpp: In static member function ‘static std::string bpftrace::TextOutput::hist_index_label(int, int)’: /home/dxu/dev/bpftrace/src/output.cpp:53:16: warning: comparison of integer expressions of different signedness: ‘int’ and ‘const uint32_t’ {aka ‘const unsigned int’} [-Wsign-compare]
   53 |   assert(index >= n); // Smaller indexes are converted directly.
      |          ~~~~~~^~~~

Checklist

[ ] Language changes are updated in man/adoc/bpftrace.adoc and if needed in docs/reference_guide.md
[ ] User-visible and non-trivial changes updated in CHANGELOG.md
[ ] The new behaviour is covered by tests

@ajor yeah, makes sense. I pushed up a change to do that


Segmentation fault (core dumped) in tf.raw_ops.FractionalMaxPoolGrad

Issue type
Bug
Have you reproduced the bug with TensorFlow Nightly?
Yes
Source
source
TensorFlow version
tf 2.16.1
Custom code
Yes
OS platform and distribution
Ubuntu 20.04
Mobile device
No response
Python version
3.11
Bazel version
No response
GCC/compiler version
No response
CUDA/cuDNN version
No response
GPU model and memory
No response
Current behavior?
If row_pooling_sequence is a minimal negative number, tf.raw_ops.FractionalMaxPoolGrad encounters "Segmentation fault (core dumped)".
Standalone code to reproduce the issue
# FractionalMaxPoolGradOp

import tensorflow as tf

overlapping = True
orig_input = tf.constant(100, shape=[1,4,7,1], dtype=tf.float32)
orig_output = tf.constant(.453409232, shape=[1,7,13,1], dtype=tf.float32)
out_backprop = tf.constant(.453409232, shape=[1,7,13,1], dtype=tf.float32)
row_pooling_sequence = tf.constant(-1250999896764, shape=[5], dtype=tf.int64)
col_pooling_sequence = tf.constant(0, shape=[5], dtype=tf.int64)
tf.raw_ops.FractionalMaxPoolGrad(orig_input=orig_input, orig_output=orig_output, out_backprop=out_backprop, row_pooling_sequence=row_pooling_sequence, col_pooling_sequence=col_pooling_sequence, overlapping=overlapping)

Relevant log output
ASAN Report:
'''
AddressSanitizer:DEADLYSIGNAL
=================================================================
==870==ERROR: AddressSanitizer: SEGV on unknown address 0x412468dbc9b0 (pc 0x7fe87681d217 bp 0x7ffc4041d560 sp 0x7ffc4041c680 T0)
==870==The signal is caused by a READ memory access.
    #0 0x7fe87681d217 in tensorflow::FractionalMaxPoolGradOp<float>::Compute(tensorflow::OpKernelContext*) (/tensorflow_src/venv/tensorflow-2.16.1-asan/lib/python3.11/site-packages/tensorflow/python/platform/../../libtensorflow_cc.so.2+0x4139e217)
    #1 0x7fe89f4e714a in tensorflow::ThreadPoolDevice::Compute(tensorflow::OpKernel*, tensorflow::OpKernelContext*) (/tensorflow_src/venv/tensorflow-2.16.1-asan/lib/python3.11/site-packages/tensorflow/python/platform/../../libtensorflow_framework.so.2+0x1cdc14a)
    #2 0x7fe89f06dfe6 in tensorflow::(anonymous namespace)::SingleThreadedExecutorImpl::Run(tensorflow::Executor::Args const&) (/tensorflow_src/venv/tensorflow-2.16.1-asan/lib/python3.11/site-packages/tensorflow/python/platform/../../libtensorflow_framework.so.2+0x1862fe6)
    #3 0x7fe89ef4a306 in tensorflow::FunctionLibraryRuntimeImpl::RunSync(tensorflow::FunctionLibraryRuntime::Options, unsigned long, absl::lts_20230802::Span<tensorflow::Tensor const>, std::vector<tensorflow::Tensor, std::allocator<tensorflow::Tensor> >*) (/tensorflow_src/venv/tensorflow-2.16.1-asan/lib/python3.11/site-packages/tensorflow/python/platform/../../libtensorflow_framework.so.2+0x173f306)
    #4 0x7fe89efa7f74 in tensorflow::ProcessFunctionLibraryRuntime::RunMultiDeviceSync(tensorflow::FunctionLibraryRuntime::Options const&, unsigned long, std::vector<std::variant<tensorflow::Tensor, tensorflow::TensorShape>, std::allocator<std::variant<tensorflow::Tensor, tensorflow::TensorShape> > >*, std::function<absl::lts_20230802::Status (tensorflow::ProcessFunctionLibraryRuntime::ComponentFunctionData const&, tensorflow::ProcessFunctionLibraryRuntime::InternalArgs*)>) const (/tensorflow_src/venv/tensorflow-2.16.1-asan/lib/python3.11/site-packages/tensorflow/python/platform/../../libtensorflow_framework.so.2+0x179cf74)
    #5 0x7fe89efb58b1 in tensorflow::ProcessFunctionLibraryRuntime::RunSync(tensorflow::FunctionLibraryRuntime::Options const&, unsigned long, absl::lts_20230802::Span<tensorflow::Tensor const>, std::vector<tensorflow::Tensor, std::allocator<tensorflow::Tensor> >*) const (/tensorflow_src/venv/tensorflow-2.16.1-asan/lib/python3.11/site-packages/tensorflow/python/platform/../../libtensorflow_framework.so.2+0x17aa8b1)
    #6 0x7fe8670d72eb in tensorflow::KernelAndDeviceFunc::Run(tensorflow::ScopedStepContainer*, tensorflow::EagerKernelArgs const&, std::vector<std::variant<tensorflow::Tensor, tensorflow::TensorShape>, std::allocator<std::variant<tensorflow::Tensor, tensorflow::TensorShape> > >*, tsl::CancellationManager*, std::optional<tensorflow::EagerFunctionParams> const&, std::optional<tensorflow::ManagedStackTrace> const&, tsl::CoordinationServiceAgent*) (/tensorflow_src/venv/tensorflow-2.16.1-asan/lib/python3.11/site-packages/tensorflow/python/platform/../../libtensorflow_cc.so.2+0x31c582eb)
AddressSanitizer:DEADLYSIGNAL
AddressSanitizer: nested bug in the same thread, aborting.
'''

@LongZE666,
Thank you for reporting the issue. I was able to reproduce the issue on tensorflow v2.15, v2.16 and tf-nightly. Kindly find the gist of it here and the screenshot of the check fail for the reference.

@LongZE666,
I tried to execute the mentioned code on the Tensorflow v2.16 in the alternative approach and it was failing with the error which was intended. Kindly find the gist of it here.
row_pooling_sequence--A Tensor of type int64. row pooling sequence, form pooling region with col_pooling_sequence
Thank you!


[AKS] Refactor ACS tests to use kwargs style instead of .format()

Makes the ACS scenario tests more readable. See also #5392.

View a preview at https://prompt.ws/r/Azure/azure-cli/5478
This is an experimental preview for @microsoft users.


Add linux musl target, add aarch64 for all major platforms

Checklist

[x] Updated CHANGELOG.md describing pertinent changes.
[ ] Updated README.md with pertinent info (may not always apply).
[ ] Updated site content with pertinent info (may not always apply).
[x] Squash down commits to one or two logical commits which clearly describe the work you've done. If you don't, then Dodd will 🤓.

Closes #552

The issue I responded to only asked for x64 musl, but since Apple has moved to full-time aarch64 machine production and ARM based Windows machines are also on the rise (around 10% of new Windows machines sold now), I think it's good sense to support it in future releases. So I added x64 musl, aarch64 musl, gnu linux versions, aarch64 macos and windows.

I have no confidence that this will actually work. To prove that this will work empirically, we need to add these targets to .github/workflows/ci.yaml so that it is clear that real and functioning binaries are being built and are useable on our example projects.


[CHANGE]: Change update methods in vuex states

Desired Change
Change the update methods in the vuex states to retrieve changed entities form the backend after successful change, if possible.
Reason/Motivation for Change
Good for keeping the state between the front and backend synchronized.
Impact of Change
No response

changed in 1d6703a1e146fe78da9b3aa64fcda255d3b1abce.


added a short wait in process_input and added parameter values

Added a 1 s wait in the process_inputs function to see if this affects the "TooMAnyRequests" error we are getting from the  Pangaea server
https://github.com/pangeo-forge/paleo-pism-feedstock/issues/2
Also adds parameter values to the zarr.

@cisaacstern, when will the recipe run, https://pangeo-forge.org/dashboard/recipe-run/1048, start? Currently it says it is still queued.

@jkingslake thanks for checking in! These recipe runs require a project admin to manually run them, using a chatops command here on the discussion thread. I will do that now.

/run paleo-pism-stacked


import { RegisterComponent } should explicitly be mentioned

Category: enhancement
Severity: Low
Description of issue / expected enhacement /Comments
In https://github.com/devonfw/devonfw-tutorial-sources/wiki/build-devon4ng-application#creating-components it is said:

We need to inject an instance of Router object and declare it into the name router in order to use it into the code, as we did on onRegisterClick(), using the navigate function and redirecting to the next view, in our case, using the route we are going to define in app.routing.module.ts: [...]

const appRoutes: Routes = [
  { path: 'FormLogin', component: FormLoginComponent},               // Redirect if url path is /FormLogin.
  { path: 'Register', component: RegisterComponent},               // Redirect if url path is /Register.
  { path: '**', redirectTo: '/FormLogin', pathMatch: 'full' }  // Redirect if url path do not match with any other route.
];

I suggest to add import { RegisterComponent } from './register/register.component'; to the code snippet.
It may not be neccessary to explicitly instruct on adding this line because Visual Studio Code analyzes the { path: 'Register', component: RegisterComponent},  code fragment and indicates the missing import.
Your environment - windows/Linux, Devonfw version , component
irrelevant

hello @Jan-PeterRichterCG
This issue has been solved by explicitly adding the import into the snippet:
....
import { RegisterComponent } from './register/register.component';
....
const appRoutes: Routes = [
  { path: 'FormLogin', component: FormLoginComponent},               // Redirect if url path is /FormLogin.
  { path: 'Register', component: RegisterComponent},               // Redirect if url path is /Register.
  { path: '**', redirectTo: '/FormLogin', pathMatch: 'full' }  // Redirect if url path do not match with any other route.
];
....

i am gonna close this issue since it has been solved,
Thanks for the feedback and regards,
trivesc


feat: Add security scanning with FOSSA and Snyk

cc @ALRubinger

@alecthomas @worstell any ideas of how to solve this error?

[ERROR] Failed to execute goal org.apache.maven.plugins:maven-dependency-plugin:3.2.0:tree (default-cli) on project ftl: Cannot build project dependency graph: Could not resolve nor collect following dependencies: [xyz.block:ftl-runtime:jar:1.0-SNAPSHOT (compile)]: Could not resolve dependencies for project ftl:ftl:pom:1.0-SNAPSHOT: The following artifacts could not be resolved: xyz.block:ftl-runtime:jar:1.0-SNAPSHOT (absent): Could not find artifact xyz.block:ftl-runtime:jar:1.0-SNAPSHOT -> [Help 1]

The examples won't compile without having first installed the FTL libraries. Running mvn install from the root should do it.

Does executing mvn install from untrusted forked PRs can execute any malicious code? Or it will always install benevolent packages?
Eg: say something add some malicious dependency package, is it possible it execute any malicious binary code? Or it's a simple pull of maven jars? (@ALRubinger maybe you know the answer here?)
Asking because this workflow trigger event pull_request_target has access to our exposed secrets, in this case SNYK_TOKEN and FOSSA_API_KEY. (Security reference here)

Actions from untrusted forked PRs aren't executed by default. An admin has to approve them.

Yep @leordev exactly as @alecthomas says. Relies on the project leads to review the deps being brought in (and the code in the PRs!) to ensure there's no env var sniffing there which would expose our secrets. Good on ya @leordev for flagging so the FTL team is aware.

I'm a bit curious why a security scan needs to build anything though, to be honest. That seems like a very odd thing to be doing during a scan.

I'm a bit curious why a security scan needs to build anything though, to be honest. That seems like a very odd thing to be doing during a scan.

Same here... We can observe from the CI logs that FOSSA didnt need to install anything, whereas Snyk is complaining about it... I tried to run the checks locally without a java/go environment and confirmed above too... FOSSA is able to scan... and Snyk complains about mvn and go cmds not found....
I'm thinking about adding the mvn install in the CI step to unblock this and trust that external PRs will going to be reviewed before approving their workflow runs then. WDYT?

I guessing it's because the mvn build is what runs the dependency resolution step, leading to how we get the full dependency tree in the Maven Effective POM. The full dep tree isn't available from the pom.xml alone.

I'm going to close this. Feel free to reopen once resuming work.


Handler resolver with message in context

I have not created a separate issue, let me know if you want one.
This is related to #257
The problem: today HandlerResolver doesn't have the context of the incoming message it only has the context of Queue. The context of the message is handy for providing extras to the IoC container. For example, it would be useful to provide the Tenant context (or any other message metadata) to the IoC container so that other components like Repositories can use correct connections strings etc. At the moment this has to be achieved by passing properties from handler down the call chain all the way to Repositories. I know the example is JE specific but the concept is general.
For example, my NinjectResolver could look like this:
    public class NinjectHandlerResolver : IHandlerResolver
    {
        private readonly IResolutionRoot _resolutionRoot;

        public NinjectHandlerResolver(IResolutionRoot resolutionRoot)
        {
            _resolutionRoot = resolutionRoot;
        }

        IHandlerAsync<T> ResolveHandler<T>(HandlerResolutionContextWithMessage context)
        {
            return _resolutionRoot.GetAll<IHandlerAsync<T>>(new ConstructorArgument("tenantContext", new TenantContext(context.Message.Tenant), true));
        }
    }

This could be further simplified by not having a check here  But instead performing this check just in time. Then there would be no need for:
HandlerResolutionContextWithMessage , IHandlerAndMetadataResolver and HandlerResolverAdapter
Alternative suggestions welcome.

@Sergej-Popov apologies for not looking at this sooner. This will need rebasing, as we've made changes to the resolver so that it happens per message, that likely overlaps with what's done here, or possibly simplifies it after some re-work in this PR.
I could see how this would be useful specifically for some JE scenarios.
I feel that we should keep JustSaying as simple as possible in terms of responsibility, this functionality could be very simply done inside of the Handler, you handler becomes your entry point, which could be a simple interceptor that fans out the Message to other handlers.
That said, I'm conflicted as I can see it would be useful. So it's a case that more is less, but more is also more 😆
Does anyone else have thoughts around this features?
@AnthonySteele @martincostello

@stuart-lang thanks for finding time for this =)
Originally this PR was to help with multitenancy by allowing for tenant context to be available at handler resolution time.

could be very simply done inside of the Handler

only if you have a handful of handlers yes. If like us you have hundreds then it becomes very unwieldy. We had to do some code generation to overcome this issue. We want to get rid of this code generation, hence this PR is still open.
If I remember correctly there were talks about adding a headers dictionary for storing meta-data like Tenant and idea of moving away from Tenant property (and some other properties) on the message itself. It would be optimal to have such headers available at resolution time rather than the message.
I would rather everyone come to an agreement on how that should look like before re-basing (or rather re-implementing) this. Personally, I would much rather see the headers + resolver supplying headers combination than what was in this PR. Let me know.

Closing as this is almost 1 year old. I don't think it is realistic that it will be merged. Might revisit the problem once other features like message envelope/meta data headers will be implemented as it might open alternative ways of solving this issue.


Yarn publish nightlies and verdaccio

Closes
✅ Pull Request Checklist:

[ ] Included link to corresponding React Spectrum GitHub Issue.
[ ] Added/updated unit tests and storybook for this change (for new code or code which already has tests).
[ ] Filled out test instructions.
[ ] Updated documentation (if it already exists for this component).
[ ] Looked at the Accessibility Practices for this feature - Aria Practices

📝 Test Instructions:

🧢 Your Project:

Rob-bot
Verdaccio builds:
CRA Test App
NextJS Test App
RAC Tailwind Example
RAC Spectrum + Tailwind Example
CRA Test App Size
NextJS App Size
Publish stats
Size diff since last release
Docs
Storybook
S2 Storybook

Tested nightly script by running
Terminal 1
yarn verdaccio --listen 4000

Terminal 2
make build
yarn npm-cli-login -u abc -p abc -e<EMAIL_ADDRESS>-r http://localhost:4000
yarn config set npmPublishRegistry http://localhost:4000
yarn config set npmRegistryServer http://localhost:4000
yarn config set unsafeHttpWhitelist localhost
yarn config set npmAlwaysAuth true
yarn npm login // login with u/p from first step
yarn version:nightly
yarn publish:nightly

dry run looks a little different i think? https://github.com/yarnpkg/berry/issues/4918

We're manually trying this out, if it doesn't work, we'll revert it


Serve app over HTTPS

Do all work for this task in a branch called issue-32
Branch issue-32 from main, test HTTPS connection.

Edited mup.js to ensure HTTPS is used. Deployed from this branch, DON'T FORGET TO MERGE ISSUE-32 INTO MAIN LATER


Add initial test suite of end to end tests.

This adds an initial test suite of end to end tests that can verify the correctness of a Rockstar implementation. These are Rockstar programs, input files (if applicable), and expected output.
Some conventions

Programs are named with the .rock extension
Expected output matches the program name but with the .rock.out extension
Input, if given, matches the program name but with the .rock.in' extension
Files in the correct folder are expected to match the output of the corresponding output file
Files in the parse-errors folder are expected to give an error when being parsed
Files in the runtime-errors folder are expected to give an error when run, but should parse correctly

These tests may need some adjustments but should be a good start! They're based on the spec and some of cwfitzgerald's extension PRs. Also, most variables are not named particularly idiomatically, but these could be changed as desired.

Looks good to me, thanks for all the great tests, will make our lives easier.


Publish 10/05/2021, 10:30 AM

Docs Build status updates of commit 6cc2140:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

docset/winserver2019-ps/defender/Set-MpPreference.md
:white_check_mark:Succeeded
View (WindowsServer2019-ps)

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


build: disable failing int tests until they are fixed

PR summary
Disables the integration tests for Case Mgmt, Enterprise Mgmt, and Resource Mgr services until they can be properly fixed.
PR Checklist
Please make sure that your PR fulfills the following requirements:

[ ] The commit message follows the Angular Commit Message Guidelines.
[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type

[ ] Bugfix
[ ] Feature
[ ] Code style update (formatting, local variables)
[ ] Refactoring (no functional changes, no api changes)
[ ] New tests
[x] Build/CI related changes
[ ] Documentation content changes
[ ] Other (please describe)

What is the current behavior?

What is the new behavior?

Does this PR introduce a breaking change?

[ ] Yes
[x] No

Other information

:tada: This PR is included in version 0.7.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


404 on Appsource links

Both app source links (top and bottom) return a 404 not found error.

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

ID: f2d7de2a-88ce-c549-b28c-5c035a904b44
Version Independent ID: 12426cca-3145-8431-9bdb-81b20f5489d9
Content: Dynamics 365 education accelerator higher education component - Common Data Model
Content Source: common-data-model/hied-accelerator.md
Service: common-data-model
GitHub Login: @meenoorami
Microsoft Alias: merami

@correnasergent Thanks for reporting this issue. We've fixed the app source link. The other link will be fixed shortly.


Improve VS Marketplace searching

@Scottj1s  should we add "cppwinrt" to this list in response to the Q&A question on the marketplace site?
https://github.com/microsoft/cppwinrt/blob/eb2ad4af75dd971e14e15d867a9cc8d814eb69bb/vsix/source.extension.vsixmanifest#L14

yes, we should


ES6-BOOK

I used Gitflow in branching model and workflow for version control in software development projects
I used Eslint linter tool for JavaScript to help enforce consistent coding practices and identify potential issues in code.
I Added Javascript ,index.html and CSS Navbar.

I have deleted and reinstalled it, but I'm still experiencing the same issues. I went ahead and deleted all directories to start again, but I encountered the same issues.


Add displacement logging on ifc calculated using ASE

Add log when calculating second and third order forceconstants using ASE.

Added this minor fix in the CNT example 2.0 pull request.

Added this minor fix in the CNT example 2.0 pull request.

Added require feature and merged to master.

Added require feature and merged to master.


ID Claim 075: Text Commands Wheel

Mod ID: 075
Mod Name: Text Commands Wheel
Mod Author: Darth Pointer
That is it, I assume it is my last legacy-BPMM using mod.
Still not sure if I will transfer existing ones to new system any soon.

Added.
There's no pressure to migrate your old mods to the native spawning system, it's totally up to you.


Colour inversion for terminal cursor

Abstract

When cursor shape is block and cursor is moved over to some character then cursor completely hides the character making it unreadable.
Currently this happens in contour

and this is in alacritty

Motivation

Whenever cursor is moved to some character, the background cell color and text color can be inverted. Alacritty also does this by allowing to set
  cursor:
    text: CellBackground
    cursor: CellForeground

in config file
Specification

Oh yes, let me fix that ASAP. thanks for pushing me. :-)

Thanks :-)

  cursor:
    text: CellBackground
    cursor: CellForeground

I really struggled in accepting the naming scheme of Alacritty, as much as I tried to be in line with their naming scheme to not confuse users, but cursor.cursor doesn't sound quite right. I'm not saying I found the best solution though, but here we go. (PR should be merged soon).

@Utkarsh-khambra this is how you can configure it:
https://github.com/contour-terminal/contour/blob/a008f1383e1c32dffbce7c8e3f504136bf682286/src/contour/contour.yml#L277-L291
While now being merged to master, CI should have created binary packages already also. Otherwise, I think it's time to make another point release. Enjoy.

Thanks it works great now :)


Workbox landing page revamp

R: @petele @philipwalton
This is a small set of changes to https://developers.google.com/web/tools/workbox/
I think there's a decent amount of additional work to be done, but I hit this low-hanging fruit:

No longer generate a "Contributors" section by pulling in profile images from the GitHub API. While we still ❤ our contributors, a cleaner and more up-to-date approach is just to link to the corresponding GitHub Contributors page. This also lets us drop the workbox-generate-contributors.js gulp task.

Remove the various logos and the "Who's using Workbox?" section, some of which were out of date, and none of which provided any context. Instead, I've added a "Case Studies" section with a link to a few representative examples of successful Workbox deployments.

And then some miscellaneous cleanup as well.

Whoops!
There were 9 warnings that will prevent this PR from being merged. Please take a look, and either fix, or provide a justification for why they can't be fixed.
WARNINGS
gulp-tasks/workbox/workbox-generate-contributors.js - Unable to read file, was it deleted?
src/content/en/tools/workbox/_shared/contributors.html - Unable to read file, was it deleted?
src/content/en/tools/workbox/images/third_party/logos/bbva.png - Unable to read file stats: ENOENT: no such file or directory, stat 'src/content/en/tools/workbox/images/third_party/logos/bbva.png'
src/content/en/tools/workbox/images/third_party/logos/forbes.png - Unable to read file stats: ENOENT: no such file or directory, stat 'src/content/en/tools/workbox/images/third_party/logos/forbes.png'
src/content/en/tools/workbox/images/third_party/logos/iheartradio.png - Unable to read file stats: ENOENT: no such file or directory, stat 'src/content/en/tools/workbox/images/third_party/logos/iheartradio.png'
src/content/en/tools/workbox/images/third_party/logos/pinterest.png - Unable to read file stats: ENOENT: no such file or directory, stat 'src/content/en/tools/workbox/images/third_party/logos/pinterest.png'
src/content/en/tools/workbox/images/third_party/logos/starbucks.png - Unable to read file stats: ENOENT: no such file or directory, stat 'src/content/en/tools/workbox/images/third_party/logos/starbucks.png'
src/content/en/tools/workbox/images/third_party/logos/tinder.png - Unable to read file stats: ENOENT: no such file or directory, stat 'src/content/en/tools/workbox/images/third_party/logos/tinder.png'
src/content/en/tools/workbox/images/third_party/logos/wired.png - Unable to read file stats: ENOENT: no such file or directory, stat 'src/content/en/tools/workbox/images/third_party/logos/wired.png'


[SPARK-43872][PS] Support (DataFrame|Series).plot with pandas 2.0.0 and above.

What changes were proposed in this pull request?
This PR proposes to remove parameter sort_columns from (DataFrame|Series).plot to support pandas 2.0.0.
Also enabling the multiple plot tests:

test_area_plot
test_area_plot_stacked_false
test_area_plot_y
test_bar_plot
test_bar_with_x_y
test_barh_plot_with_x_y
test_barh_plot
test_line_plot
test_pie_plot
test_scatter_plot
test_hist_plot
test_kde_plot

Why are the changes needed?
To support pandas 2.0.0 & match the behavior.
Does this PR introduce any user-facing change?
sort_columns will no longer available.
How was this patch tested?

Merged to master.


Migrate to CircleCI 2.0

Summary

CircleCI runs speed up.

before 3:43
after 2:01 https://circleci.com/gh/serima/c3/34

But only one test has failed.

Test
Only one test has failed.
Is there anyone who knows something about this problem?
I tried passing the --no-sandbox flag to chrome in karma.conf.js, but the result was the same.
like this:
    browsers: ['ChromeNoSandbox'],
    // ...
    customLaunchers: {
      ChromeNoSandbox: {
        base: 'Chrome',
        flags: ['--no-sandbox']
      }
    },

Chrome 61.0.3163 (Linux 0.0.0): Executed 297 of 297 (1 FAILED) (47.826 secs / 47.487 secs)
TOTAL: 1 FAILED, 296 SUCCESS

1) should split tick text properly
     c3 chart axis axis.x.tick.width category axis not rotated
     Expected 'tick text on' to be 'tick text on category'.
    at SVGTSpanElement.<anonymous> (/tmp/spec/axis-spec.js:417:45 <- /tmp/fb3886d1049e972d58c80727300a4e30.browserify:11406:46)
    at /tmp/node_modules/d3/d3.js:962:0 <- /tmp/fb3886d1049e972d58c80727300a4e30.browserify:963:16
    at d3_selection_each (/tmp/node_modules/d3/d3.js:968:0 <- /tmp/fb3886d1049e972d58c80727300a4e30.browserify:969:30)
    at Array.d3_selectionPrototype.each (/tmp/node_modules/d3/d3.js:961:0 <- /tmp/fb3886d1049e972d58c80727300a4e30.browserify:962:12)
     Expected 'category axis' to be 'axis'.
    at SVGTSpanElement.<anonymous> (/tmp/spec/axis-spec.js:417:45 <- /tmp/fb3886d1049e972d58c80727300a4e30.browserify:11406:46)
    at /tmp/node_modules/d3/d3.js:962:0 <- /tmp/fb3886d1049e972d58c80727300a4e30.browserify:963:16
    at d3_selection_each (/tmp/node_modules/d3/d3.js:968:0 <- /tmp/fb3886d1049e972d58c80727300a4e30.browserify:969:30)
    at Array.d3_selectionPrototype.each (/tmp/node_modules/d3/d3.js:961:0 <- /tmp/fb3886d1049e972d58c80727300a4e30.browserify:962:12)

Codecov Report

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

@@           Coverage Diff           @@
##           master    #2170   +/-   ##
=======================================
  Coverage   73.98%   73.98%
=======================================
  Files          51       51
  Lines        4178     4178
=======================================
  Hits         3091     3091
  Misses       1087     1087

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 69f975c...1ac6aa5. Read the comment docs.

Thanks for the contribution!
I like the idea to migrate to circle ci 2.0. Thanks for creating the configuration!
The only problem here is the test case above. That test case depends on the font of the environment and should be fixed somehow systematically. We are going to fix the test case later. So please skip it for now.
like:
xit('should split tick text properly', function () {

Hi @kt3k, thank you. skipped a test!

@serima
Thanks! Merging.


release 0.6.5 breaks buttons

I think after update to 0.6.5 tool bar buttons are broken, at least for my .jl scripts.  Was working great on 0.6.4, even though the html button was invisible.  0.6.5 brings back the html button, but no functionality for either button.

now the command is not defined for the julia scope, since you only need to use that for the weave scope.

I'm sorry but I'm confused.  If I have a .jl file open in the editor, how
do I issue a weave command in the weave scope? I tried calling 'using
Weave' on the REPL command line, and then using the bottom menu to change
to the Weave scope, but still the buttons don't function. It seems to me
that the most sensible use case would be that I could just open a .jl file,
and then push the weave button. Can I do something similar to retain that
functionality?
I hope I'm not overly critical; I think Weave is awesome, and anticipate
lots of value from using it.
Thanks for your help.
BM
On Sat, Oct 12, 2019, 4:16 AM Shuhei Kadowaki<EMAIL_ADDRESS>wrote:

now the command is not defined for the julia scope, since you only need to
use that for the weave scope.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JunoLab/language-weave/issues/35?email_source=notifications&email_token=AF7MJJUGBDOYTVYH6VP324TQOGBUPA5CNFSM4JABLEZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBBZKEQ#issuecomment-541299986,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF7MJJS4IQGFPU3EZXM7ISTQOGBUPANCNFSM4JABLEZA
.

Well, .jl technically aren't weavable -- you should be in a .jmd or similar file (or set the editor's grammar to one of the other Weave ones).
That said, I'm not sure if we need to be so strict here -- I don't think there's any real downside to allowing editors with an arbitrary grammar to be weavable.

Ok thanks.  Just to be clear, I'm not implying I'll want to weave any
arbitrary .jl file, but my use case is taking something very similar to the
FIR_design.jl example given in the weave package, and making it into a pdf.
Ideally, I could do that with one (or a few?) clicks.
On Sat, Oct 12, 2019, 12:16 PM Sebastian Pfitzner<EMAIL_ADDRESS>wrote:

Reopened #35 https://github.com/JunoLab/language-weave/issues/35.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JunoLab/language-weave/issues/35?email_source=notifications&email_token=AF7MJJWXABHZPWZHXXKTYFDQOHZ6VA5CNFSM4JABLEZKYY3PNVWWK3TUL52HS4DFWZEXG43VMVCXMZLOORHG65DJMZUWGYLUNFXW5KTDN5WW2ZLOORPWSZGOUFVWCZI#event-2708169061,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF7MJJQWXWJOICTHQG7RLTDQOHZ6VANCNFSM4JABLEZA
.

That said, I'm not sure if we need to be so strict here

yep, the only reason I made those changes is that not to allow users to dispatch the command on scopes that the commands won't work.

but my use case is taking something very similar to the FIR_design.jl example given in the weave package

so is there any case when you want to use the command for .jl files ?
I'm really okay to reregister the command to the workspace, then.

Yes, please. If you could re-register, I would be very happy about that.  I
prefer that my scripts maintain the .jl extension, and the ability to run
as regular Julia codes, or be weaved to pdf with the comments included.
Thanks for the super prompt response!
BM
On Sat, Oct 12, 2019, 12:47 PM Shuhei Kadowaki<EMAIL_ADDRESS>wrote:

That said, I'm not sure if we need to be so strict here
yep, the only reason I made those changes is that not to allow users to
dispatch the command on scopes that the commands won't work.
but my use case is taking something very similar to the FIR_design.jl
example given in the weave package
so is there any case when you want to use the command for .jl files ?
I'm really okay to reregister the command to the workspace, then.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/JunoLab/language-weave/issues/35?email_source=notifications&email_token=AF7MJJWEL4TB3N5NYIWYUQLQOH5RXA5CNFSM4JABLEZKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBCDRPA#issuecomment-541341884,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AF7MJJVKXEK46HXHFYACGITQOH5RXANCNFSM4JABLEZA
.


ci: test ci

Signed-off-by: wllenyj<EMAIL_ADDRESS>

Codecov Report

Merging #5 (5ff2c52) into main (a8df593) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main       #5   +/-   ##
=======================================
  Coverage   90.13%   90.13%
=======================================
  Files          19       19
  Lines        2970     2970
=======================================
  Hits         2677     2677
  Misses        293      293

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 a8df593...5ff2c52. Read the comment docs.


 I can probably handle this in BukkitEnchantments.vanillaEnchantment().

This has been fixed in Custom Items 11.6


CassetteSettings.SourceDirectory should be public?

Is CassetteSettings.SourceDirectory deliberately declared internal?
To me it would make sense to allow users to provide their own implementations of IDirectory and IFile where desired.  For example, this could be a first step towards fixing #59.
The IDirectory and IFile interfaces are already public, so everything else is in place to allow this, except that there is no public integration point. Just changing CassetteSettings.SourceDirectory to be public should allow users to choose an IDirectory implementation in their implementation of ICassetteConfiguration.Configure.
Does this sound reasonable or have I missed something?

I've been very strict about keeping most parts of Cassette internal to make being SemVer compliant easier. However, I don't mind making things public when there is a good reason. It seems fair for SourceDirectory to public. I'll try to get this into the next release.

(I suppose you could make the same argument about CassetteSettings.CacheDirectory, in case someone wants to provide their own storage solution for caching).

Cool, thanks!


ES6 import fails as function name mismatch on Chartist.plugins.tooltip

The return and declared functions name are mismatching.. Es6 imports fails because of that. Change it to the same name in both places.
Declared as:
Chartist.plugins.tooltip
Returns:
return Chartist.plugins.tooltips;

This has been fixed c476a2dd255134241e4238f562ac3cb8b617bc79


Warn user if target process has high privileges (window messages will not be handled)

ctypes.windll.shell32.IsUserAnAdmin() works to check Python process administrator privileges. For target app process the following resources may help:

https://www.codeproject.com/Questions/544981/IsplusProcessplusRunningplusasplusAdministrator
http://docs.activestate.com/activepython/2.6/pywin32/win32security.html
https://msdn.microsoft.com/en-us/library/windows/desktop/aa376389(v=vs.85).aspx
https://stackoverflow.com/questions/15773868/check-if-an-arbitrary-user-is-in-administrator-group-with-python

So warn user if target process is admin, but Python one is not. I think this is the most useful scenario. Not sure about uiAccess=true privilege, it would be more complicated to check.

Implemented it another way (got back to old "method of probes and errors" for OpenProcess, but added errors for some window messages that won't work). See commit https://github.com/vasily-v-ryabov/pywinauto/commit/7aab3d97d6db8ed4350ae29a1e532d434fd528ca.

Fixed in #499.


Allow OIDC prompt query param

The OpenID Connect Core spec defines an extension to the authorization code grant type that allows that grant type to take an optional prompt query parameter that allows the client to have more control over how the auth server handles an authorization code request in scenarios where the user is not signed in or has not consented to all the scopes.
I think the handling of the prompt query argument is the responsibility of whomever is using this library, but I wonder if you'd be open to adding some support for the prompt query argument to this library. I tried to handle prompt without getting this library involved, but starlette's QueryParams class is immutable, so I don't have a good means of removing the prompt query argument from the request before passing the request on to this library.
This PR currently reflects the bare minimum needed to make this library play nicely with the prompt query parameter, but I'd be happy to add some validation of the prompt query parameter to AuthorizationCodeGrantType.validate_request if you are open to aioauth including some basic support for the prompt query param.
Here's what the OIDC Core spec has to say about prompt:
prompt
  OPTIONAL. Space delimited, case sensitive list of ASCII string values that
  specifies whether the Authorization Server prompts the End-User for
  reauthentication and consent. The defined values are:
    none
      The Authorization Server MUST NOT display any authentication or consent
      user interface pages. An error is returned if an End-User is not already
      authenticated or the Client does not have pre-configured consent for the
      requested Claims or does not fulfill other conditions for processing the
      request. The error code will typically be login_required,
      interaction_required, or another code defined in Section <IP_ADDRESS>. This
      can be used as a method to check for existing authentication and/or
      consent.
    login
      The Authorization Server SHOULD prompt the End-User for reauthentication.
      If it cannot reauthenticate the End-User, it MUST return an error,
      typically login_required.
    consent
      The Authorization Server SHOULD prompt the End-User for consent before
      returning information to the Client. If it cannot obtain consent, it MUST
      return an error, typically consent_required.
    select_account
      The Authorization Server SHOULD prompt the End-User to select a user
      account. This enables an End-User who has multiple accounts at the
      Authorization Server to select amongst the multiple accounts that they
      might have current sessions for. If it cannot obtain an account selection
      choice made by the End-User, it MUST return an error, typically
      account_selection_required.
  The prompt parameter can be used by the Client to make sure that the End-User
  is still present for the current session or to bring attention to the
  request. If this parameter contains none with any other value, an error is
  returned.

Codecov Report

Merging #75 (871f762) into master (48896f2) will increase coverage by 0.00%.
The diff coverage is 100.00%.

:mega: This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more
@@           Coverage Diff           @@
##           master      #75   +/-   ##
=======================================
  Coverage   98.78%   98.78%
=======================================
  Files          14       14
  Lines         658      659    +1
  Branches       99       99
=======================================
+ Hits          650      651    +1
  Misses          4        4
  Partials        4        4

Impacted Files
Coverage Δ

aioauth/requests.py
100.00% <100.00%> (ø)

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

This PR currently reflects the bare minimum needed to make this library play nicely with the prompt query parameter, but I'd be happy to add some validation of the prompt query parameter to AuthorizationCodeGrantType.validate_request if you are open to aioauth including some basic support for the prompt query param.

sure, approved. you can add the implementation within this or next PR.

I'm still evaluating how I want to extend this. I'm fine to keep this PR open if it's all the same to you.

I've been thinking this over a bit today and I'm inclined to say that the way that authlib and oauthlib handle the different standards they support is what I'd recommend for this library as well: handle concerns specific to one standard in a standard specific class. A feature flag for OIDC begets a feature flag for RFC6750 and so on. We'd be falling into the trap that this refactoring aims to remedy: Replace conditional with polymorphism. It also would make tests more annoying by increasing the number of standard specific configuration standards that need to be tested.
So, though what I've suggested in this PR is simple, I'd argue that it moves this library a small step in the wrong direction. What I would instead propose is beginning to migrate the OIDC specific behaviors into OIDC specific subclasses. I get the impression that creating a new subclass of Query and a new subclass of BaseRequest would do the trick, however it runs into a problem at the level of aioauth-fastapi which is fixed on using the standard Request, Query, and Post classes. But that's a problem for that repo.

@aliev, I've updated this PR in such a way as to begin the road toward handling standard specific behaviors in standard specific subclasses for various components. Let me know what you think. If you're happy with what's here, I'd request that you merge this PR and I can use it as a base for migrating some of the id_token work I have open in #76.
I'm still trying to figure out what needs to happen in aioauth-fastapi to make it better able to work with standard specific subclasses, but I don't think that's something that needs to block this PR.

@aliev , I think this is ready for your review whenever you have the opportunity.

@aliev, I don't mean to rush you on this, but I want to make sure we're aligned on what's possible here.
I know that you granted me some kind of permissions for this repo, but I'm not able to move this code forward without a review from a code owner.
If you intend to get to this PR and haven't had a chance, that is totally cool, but I just want to make sure you're not expecting me to take care of this on my own.

@aliev, I don't mean to rush you on this, but I want to make sure we're aligned on what's possible here.
I know that you granted me some kind of permissions for this repo, but I'm not able to move this code forward without a review from a code owner.

I have added you as a collaborator so that the pipelines can run automatically without my approval.

If you intend to get to this PR and haven't had a chance, that is totally cool, but I just want to make sure you're not expecting me to take care of this on my own.

Sorry for the delay. Please give me some time, I will double-check the entire changes.

@aliev , thanks for clarifying! There's really no rush, I just wanted to make sure you didn't expect me to take action on my own.

I approved this PR and will merge it. Overall, everything looks good. The only thing I would suggest is to add a docstring to the prompt field and the Query class in the aioauth/oidc/core/requests.py file, with a reference to the specification.
However, I think this can be done in your next PR, so as not to delay your progress.


Look deeper in failing LASYOU

the fragmentation works, but we do not get a rcsr code

If we do not work on the primitive, we have all kinds of issues with images

I guess this is the reason i saw dependence on the file


Manual C2-C3 identification : out of bound error

p.56 ->
When navigating the slices (left/right arrows) to identify manually C2-C3, the viewer crashes with an out of bound error :
sct_label_utils -i t2.nii.gz -create-viewer 3 -o label_c2c3.nii.gz -msg "Click at the posterior tip of C2/C3 inter-vertebral disc"

--
Spinal Cord Toolbox (git-master-846d5d1bf50fe37d602b99b142a0c3032492e566)

Traceback (most recent call last):
  File "/home/olix86/sct/spinalcordtoolbox/gui/sagittal.py", line 89, in increment_vertical_nav
    self.sagittal.refresh()
  File "/home/olix86/sct/spinalcordtoolbox/gui/widgets.py", line 296, in refresh
    data = self._image.data[:, :, self._z]
IndexError: index 64 is out of bounds for axis 2 with size 64
Aborted (core dumped)

I think it comes from the fact that you've reached the edge of the 3D volume.
One fix, if possible, would be to show which slice you are seeing and the max number you can go up to.

Yep the ideal would be to not be able to go out of bounds, but if that's not possible due to the viewer, displaying x out of y slices would be an alternative

the ideal would be to not be able to go out of bounds

that's definitely the way to go. Fairly easy to implement: just add a case here. Anyone willing to take on that case is welcome to :-)


Play Store Description: link to this project is broken (5.30.0)

Description
This is an easy one! In the Google Play store, at the end of the description for this app, the text reads

This app is open source at https://github.com/duckduckgo/Android.

The link is including the period at the end of that sentence - it should be removed. (I suspect the hyperlink is generated automatically, so there should either be a space before the period, or it should just be taken out entirely).
Expected Behavior
Link in Google Play Store brings you to The duckduckgo/Android project
Actual Behavior
Link in Google Play Store brings you to a 404 page, due to a trailing period in the URL.

After a quick search, I don't think the Google Play Store dsscription exists as part of this repository, unlike the changelog etc. This will need to be changed via the Google developer / play store web interface. I think that's accessible here: https://play.google.com/apps/publish/
See: https://stackoverflow.com/questions/26436865/app-description-update-on-play-store

Good spot! Fixed it now; Google Play should show the change within the next few hours.
Thank you!


For interaction token request change /token call to interaction_code

When doing a social login, after the social IdP redirects back to Okta and Okta redirects the browser back to the mobile app with a valid code, the SDK makes the /token call with the below parameters,
{
  client_id=0oa3...
  code=A6DFLjjlDPbXmL7jmbfiaLlxnay2bFcfKzlUHHm674
  code_verifier=rhUEmyr3T7SYlV7CbGGKwM18jl_CU6qhffaeOKhxfpk
  grant_type=interaction_code
}

which returns,
{
  "error": "invalid_grant",
  "error_description": "The interaction code is invalid or has expired."
}

Code change was to change code=A6DFLjjlDPbXmL7jmbfiaLlxnay2bFcfKzlUHHm674 to interaction_code=A6DFLjjlDPbXmL7jmbfiaLlxnay2bFcfKzlUHHm674

@emanor-okta Can you rebase from master? I just merged a change that fixes the build issue this change is facing.


Sublime crash on closing the first panel

Reproduce the behavior :

CTRL - SHIFT - N (open a new window for the test)
CTRL - K , CTRL - RIGHT (create a panel at right)
CTRL - K , CTRL - DOWN (create a panel at bottom)
CTRL - K , DOWN (go to the bottom panel)
CTRL - K , MAJ + UP (delete the top-left panel)

( On Sublime 3083 - Windows 7 64b )
<3

Cannot repro. It works fine for me (I'm on Mac OS though).

confirm on windows SO it crash closing the last tab in a panel with "origami_auto_close_empty_panes" at true

I just experienced this issue on SublimeText 3 on Windows 7.
Like eparisio says, 'origami_auto_close_empty_panes' must be set to true.
Setting this to false, 'solved' the issue for me.

This appears to be a bug in Sublime Text. I don't think I can solve it.

You can prevent it by opening a new empty tab if theres no tabs in the window.

This is related to Sublime Text's close_windows_when_empty preference (thanks @titoBouzout). If you set this to false you should not see this problem anymore.


Returning a reference of a sent message

So in the pre-v2 days you could do something like this:
let sentMessage = channel.send(`Hello!`);

setTimeout(() => {
  sentMessage.updateMessage('I will be deleted in 2 seconds.');
}, 1000);

setTimeout(() => {
  sentMessage.deleteMessage();
}, 3000);

Nowadays the callback for rtm.sendMessage only returns an error object and no other arguments.
I had a look through the source and found that although you can update messages via channel.updateMessage, you need a reference to the message you sent, in particular its timestamp value. Plus it seems that you can only retrieve messages by using channel.getMessageByTs, again requiring that pesky timestamp.
Is there a way in the current system to emulate the above behavior? If not is it perhaps an idea to add a reference to the sent message in rtm.sendMessage's optional callback like so:
rtm.sendMessage(`Hello!`, channel.id, (err, msg) => {
  channel.updateMessage('Updated!', msg.id);
});

Happy to implement this myself is someone can point me in the right direction.

So, I did a little research into it, and i found some good information that would help with this: messages always have unique time stamps (versus ids, which I don't think they have at all in the same sense that a channel has an id), and that time stamp is sent back via RTM api.
The issue with this is that you have to wait for the RTM client to receive the ok from the server to know what that timestamp is. The callback (and message id) should be held until that ok comes back.
Also, I found, in my search, that there was a TODO for this exact functionality. @l12s the slack api doesn't indicate that a type is given in responses to RTM messages, so is that a bug in this module or is it an oversight in the api docs?
As for now, it seems that the RTM client will emit a CLIENT_EVENTS.RTM.MESSAGE event whenever a message that is sent is recieved. Listen to that event and check for a "reply_to" field. That'll be the id that the client generated to be assigned to that message. How you'll make all of this knowledge work together, I don't know.

Looking at this now.

Ok, thanks for that context @Everesty - I think I understand what the issue is.
So, two possible scenarios you'll want to have a message callback trigger on:
Fail
the websocket send fails, callback is called with the error object and a null response
Success
the message succeeds and the websocket (quickly, but async from the send acknowledgement on the client socket) replies with a message that looks like:
{
	"reply_to": 1,
	"type": "message",
	"channel": "C0CHZA86Q",
	"user": "U0CJ5PC7L",
	"text": "test",
	"ts": "1457305861.000002"
}

The failure case is currently handled, with the optCb function from sendMessage passed through. However, it's also treating the successful send of the message as equivalent to the success case.
So, I'm going to do two things:

add promise support to sendMessage + send, as a I forgot to when I added it to the web API
track any callback that's passed through and call it with the message object from the success case when that returns

:+1: Exactly what I was hoping for, thanks!

Ok, the above PR should fix this for you. I'd appreciate some :eyes: on it if you have a moment to check it out.
I'll add a couple tests for the new behavior too.


Forbidden Lands - sheetworker action button fix

Although there doesn't seem to be any ill effect with the sheet using action button names with "", I've renamed these buttons and sheetworkers to replace "" with "-" as per the wiki's warning.

Submission Checklist
Required

[ ] The pull request title clearly contains the name of the sheet I am editing.
[ ] The pull request title clearly states the type of change I am submitting (New Sheet/New Feature/Bugfix/etc.).
[ ] The pull request makes changes to files in only one sub-folder.
[ ] The pull request does not contain changes to any json files in the translations folder (translation.json is permitted)

New Sheet Details

The name of this game is: <   >  (i.e. Dungeons & Dragons 5th Edition, The Dresden Files RPG)

The publisher of this game is: <   > (i.e. Wizards of the Coast, Evil Hat)

The name of this game system/family is: <   > (i.e. Dungeons & Dragons, FATE)

[ ] I have followed the Character Sheets Standards when building this sheet.

[ ] This sheet has been made on behalf of, or by, the game's publisher.
[ ] This game is not a traditionally published game, but a copy of the game rules can be purchased/downloaded/found at: <   >
[ ] This sheet is for an unofficial fan game, modification to an existing game, or a homebrew system.

Changes / Description

Character Sheet Info Roll20 Internal Use only.


Help debugging why black server doesn't start

Hi there,
I've just tried to install black in VSCode but am having some issues.  However, it's quite possible this is proxy related as I'm running inside a compute cluster with minimal internet access.  We recently opened up the firewall using the following as a guide:  https://code.visualstudio.com/docs/setup/network
I've set up my settings.json as directed in the setup guide and it currently looks like:
{
    "python.formatting.provider": "none",
    "[python]": {
        "diffEditor.ignoreTrimWhitespace": false,
        "editor.defaultFormatter": "ms-python.black-formatter",
    },
    "editor.formatOnSave": true,
    "black-formatter.trace.server": "verbose",
}

My Output tab looks like:
Name: Black Formatter
Module: black
Python extension loading
Waiting for interpreter from python extension.
Python extension loaded
[Error - 1:29:13 PM] Starting client failed
Launching server using command /.venv/bin/python failed.

I found another thread where someone was asked if they could run the server directly and in my installation I seem to be able to successfully run it:
python ~/.vscode/extensions/ms-python.black-formatter-2022.7.13271013/bundled/tool/server.py
Ctrl-C

I'm on Python 3.8.5 with VSCode 1.75.0.
Edit: and I'm obviously, from that output log, running in a venv.  I had assumed that the printed path must have been some sort of bug in the logger but could it really be trying to look for my venv in root rather than the working directory?

@shareefj It looks like the path to python interpreter might be getting messed up. It should not be starting with /. Can you try setting "black-formatter.interpreter": ["${workspaceFolder}/.venv/bin/python"] ? make sure you have the latest version of black formatter extension 2022.6.0.

@karthiknadig Indeed, that has fixed it and the server starts up correctly now.
Any ideas what could have caused this?  The interpreter was correctly recognised by VSCode (bottom right) and I went through the process of trying to force that a few times to check.
I don't particulary want to have to check in a hard coded path to a venv name.

This is likely a bug in the API for python extension moving it there for further investigation.


237/delete env

Remove .env file from repository. Keeping that around can mess with different deployment environments in hard to predict ways.
NOTE: ACTION MUST BE TAKEN TO MAINTAIN CURRENT BEHAVIOR

developers will need to add a .env file to their root directory. They can copy the dev.env file found in the env_templates directory to do so
The environment variables VITE_SOLID_IDENTITY_PROVIDER_PRODUCTION and VITE_SOLID_IDENTITY_PROVIDER_DEV have been condensed into VITE_SOLID_IDENTITY_PROVIDER. Environments (code for PDX pages) need to be updated accordingly.

I would love it if it was set to "solidcommunity.net" so I don't have to copy that into the login component anymore. 🤣

I'm a little confused on what we are doing here...
Is the goal just to remove the .env file from GitHub?    Seemingly so, since directions are for devs to create a .env file.
Couldn't we just delete the file here and add .env to the .gitignore ?    Is that what this PR effectively is doing?

Couldn't we just delete the file here and add .env to the .gitignore ? Is that what this PR effectively is doing?

It's because we have an .env on GitHub, GitHub continues to track it for changes. So having this branch remove the .env and having developers copy a new one locally will let .gitignore kick in.

I would love it if it was set to "solidcommunity.net" so I don't have to copy that into the login component anymore. rofl

This will happen after merge if you copy templates/dev.env to .env. Part of the reason I made this fix is because you were having that issue :)


AdvancedJetEngine should depend on SolverEngines

But I think that's only in the next release? @NathanKell should know and I think might have even told me but I've forgotten.

Yep.

Closing as implemented.


python38Packages.importlib-resources: 3.3.1 -> 5.1.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from passthru.updateScript.
meta.description for python38Packages.importlib-resources is: "Read resources from Python packages"
meta.homepage for python38Packages.importlib-resources is: "https://importlib-resources.readthedocs.io/"
meta.changelog for python38Packages.importlib-resources is: ""
Updates performed

Ran passthru.UpdateScript

To inspect upstream changes
Impact

Checks done (click to expand)

built on NixOS

Warning: a test defined in passthru.tests did not pass

0 of 0 passed binary check by having a zero exit code.

0 of 0 passed binary check by having the new version present in output.

found 5.1.2 with grep in /nix/store/q29cl3rrzpfnxazd8z81x13611jbp3iy-python3.8-importlib_resources-5.1.2

directory tree listing: https://gist.github.com/8bae7cd771686c8a443aca3577a1b8c4

du listing: https://gist.github.com/735ed5e6dc254e2c74de56af2d602910

Rebuild report (if merged into master) (click to expand)

87 total rebuild path(s)

22 package rebuild(s)

22 x86_64-linux rebuild(s)
22 i686-linux rebuild(s)
21 x86_64-darwin rebuild(s)
22 aarch64-linux rebuild(s)

First fifty rebuilds by attrpath
glasgow
maestral
maestral-gui
mailman
matrix-synapse
mycli
python38Packages.desktop-notifier
python38Packages.glasgow
python38Packages.importlib-resources
python38Packages.maestral
python38Packages.mailman
python38Packages.mailman-hyperkitty
python38Packages.nmigen
python38Packages.nmigen-boards
python38Packages.nmigen-soc
python38Packages.pysaml2
python39Packages.importlib-resources
python39Packages.mailman
python39Packages.mailman-hyperkitty
python39Packages.pysaml2
pythonIRClib
qutebrowser

Instructions to test this update (click to expand)

Either download from Cachix:
nix-store -r /nix/store/q29cl3rrzpfnxazd8z81x13611jbp3iy-python3.8-importlib_resources-5.1.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nix-community.cachix.org/' \
  --option trusted-public-keys '
  nix-community.cachix.org-1:mB9FSh9qf2dCimDSUo8Zy7bkq5CX+/rkCWyvRCYg3Fs=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The Cachix cache is only trusted for this store-path realization.)
For the Cachix download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A python38Packages.importlib-resources https://github.com/r-ryantm/nixpkgs/archive/29a65852f17978cf23253cda7a8c5398b0ffe713.tar.gz

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/q29cl3rrzpfnxazd8z81x13611jbp3iy-python3.8-importlib_resources-5.1.2
ls -la /nix/store/q29cl3rrzpfnxazd8z81x13611jbp3iy-python3.8-importlib_resources-5.1.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
Result of nixpkgs-review run on x86_64-linux 1

1 package failed to build:

pythonIRClib

19 packages built:

glasgow (python38Packages.glasgow)
maestral (python38Packages.maestral)
maestral-gui
mailman
matrix-synapse
mycli
python38Packages.desktop-notifier
python38Packages.importlib-resources
python38Packages.mailman
python38Packages.mailman-hyperkitty
python38Packages.nmigen
python38Packages.nmigen-boards
python38Packages.nmigen-soc
python38Packages.pysaml2
python39Packages.importlib-resources
python39Packages.mailman
python39Packages.mailman-hyperkitty
python39Packages.pysaml2
qutebrowser

Maintainer pings

Result of nixpkgs-review pr 116090 at 29a65852 run on aarch64-linux 1

5 packages failed to build (new failures):

glasgow (python38Packages.glasgow)
python38Packages.nmigen
python38Packages.nmigen-boards
python38Packages.nmigen-soc
pythonIRClib

15 packages built successfully:

maestral (python38Packages.maestral)
maestral-gui
mailman
matrix-synapse
mycli
python38Packages.desktop-notifier
python38Packages.importlib-resources
python38Packages.mailman
python38Packages.mailman-hyperkitty
python38Packages.pysaml2
python39Packages.importlib-resources
python39Packages.mailman
python39Packages.mailman-hyperkitty
python39Packages.pysaml2
qutebrowser

1 suggestion:

warning: maintainers-missing
Package does not have a maintainer. Consider adding yourself?
Near pkgs/development/python-modules/importlib-resources/default.nix:33:3:
   |
33 |   meta = with lib; {
   |   ^

It seems like this update might break compatibility with python2.7, which then causes problems for some packages that unfortunately still depend on python2.


⚠ Bofrost.lu has degraded performance

In d2264d1, Bofrost.lu (https://www.bofrost.lu) experienced degraded performance:

HTTP code: 200
Response time: 1262 ms

Resolved: Bofrost.lu performance has improved in 73d771f after 34 minutes.


Add docker-compose.yml to start databases in Docker containers

I've added a docker-compose.yml configuration file to start the following databases: mysql, postgres and mongo.
Test are still failing because some of the connection are hardcoded. I tried to have a look but I could not understand the logic so I didn't touch anything.

I will have a look later thank you for your contribution

This will need to have an example docker config to be be included though.
The question is, to have a consistent way to address the containers, we will need to add a node js execution image with the dbs linked.

Actually forgot the questions: The question is, to have a consistent way to address the containers, we will need to add a node js execution image with the dbs linked, so how do we make the usage more convinient again.
Working from inside the docker container is most of the time not a good option for the developer.

I was thinking that the db.config.example.json in the test folder could be updated with the configuration that is in the docker-compose.yml however, we need to fix every hardcoded configuration in the tests first. Here is few examples:

https://github.com/db-migrate/node-db-migrate/blob/master/test/driver/pg_schema_test.js#L9
https://github.com/db-migrate/node-db-migrate/blob/master/test/driver/pg_schema_test.js#L23

I still don't really understand what these tests are here for? What are they meant to tests exactly?

@Maxwell2022 This is true for your your examples, but the config from test folder is not being updated. I want a consistent setup for docker container if we introduce them. Thus also a config dedicated to this containers that do not change, this might be either the default config of the project itself or part of the entrypoint of the docker container hosting the node app where the data container are linked to.

@Maxwell2022 Well that is history: https://github.com/db-migrate/node-db-migrate/commits/master/test/driver/pg_schema_test.js
Actually it was introduced to test the functionality of changing the schema passed as parameter to the driver.


TransactionBatch Node SDK

Hi Is there going to be any upcoming support for TransactionBatch in the Node SDK?

Hey @wpitallo
Can you provide some more details as to which Azure service are you working with and any docs around the expected feature?

Hey @wpitallo
Can you provide some more details as to which Azure service are you working with and any docs around the expected feature?

Hi @ramya-rao-a sure, I am using @azure/cosmos npm package for node. What I would like to do with transactions in node is what is described here: https://devblogs.microsoft.com/cosmosdb/introducing-transactionalbatch-in-the-net-sdk/.  I have had a look in the @azure/cosmos package but it seems its currently only supported in .NET SQL SDK.

Hi @ramya-rao-a sure, I am using @azure/cosmos npm package for node. What I would like to do with transactions in node is what is described here: https://devblogs.microsoft.com/cosmosdb/introducing-transactionalbatch-in-the-net-sdk/.  I have had a look in the @azure/cosmos package but it seems its currently only supported in .NET SQL SDK.

Thanks for the details! Looping in @zfoster who should be able to help

Thanks for the details! Looping in @zfoster who should be able to help

Hi @zfoster any updates on this?

Hi @zfoster any updates on this?

Hey @wpitallo we do plan to add it but there are other feature priorities in line before it. We support bulk operations at the moment and batch is the next feature in that family. I unfortunately can’t give an ETA, but we do plan to support it in the SDK.

Hey @wpitallo we do plan to add it but there are other feature priorities in line before it. We support bulk operations at the moment and batch is the next feature in that family. I unfortunately can’t give an ETA, but we do plan to support it in the SDK.

@zfoster In terms of bulk operation, if one item operation failed, does it will rollback for the whole bulk?

no @Zachery2008


open-in-browser-noninteractively causes newsboat's terminal to stop rendering

Version: 2.23.0
Config:
browser xdg-open %u
confirm-exit yes
text-width 80

bind-key j down feedlist
bind-key k up feedlist
bind-key j next articlelist
bind-key k prev articlelist
bind-key J next-feed articlelist
bind-key K prev-feed articlelist
bind-key j down article
bind-key k up article
bind-key g home
bind-key G end
bind-key o open-in-browser-noninteractively
bind-key ESC quit

Steps to reproduce the issue:

Enter any article.

Type "o" to trigger open-in-browser-noninteractively.

Focus newsboat/its terminal again and run some commands, for example "q q q y" to quit. It will quit, but you won't see that rendered until the browser process has ended. You can continue to enter arbitary input to the underlying terminal, for example "notify-send x", and this will all be processed once the browser process has ended and newsboat has quit.

I've tried a few variations on the browser config value, including appending & and & disown, removing %u, and directly opening different browsers (vimb, firefox-developer-edition).
This is on Arch Linux with everything up to date. I can reproduce this behaviour in both Alacritty and xterm and each of fish and bash.

I didn't try to reproduce, but I think I see the problem already: this line
browser xdg-open %u

tells Newsboat that the "browser" is "xdg-open". The "%u" is ignored. I guess that's what tripped you up when you added & and other stuff — all of that was ignored.
The correct incantation is browser "xdg-open %u &" — note the double quotes around the setting's value. Can you please try that and confirm that it does work?
Of course, the proper fix on Newsboat's side is to add a check for this situation (settings stored in the ConfigContainer have exactly one value, so it should fail if the config provides more than one value.)

Success! Cheers :slightly_smiling_face:

Excellent! I submitted a pull request that adds an error for such cases: #1571. You would be the last person to fall into this pitfall, hopefully ^_^


Unable to download model even without modification

This problem can be reproduced with any model，like resnet18

@111qqz Thanks for the feedback! This was really a bug and has been fixed.


vs2010 deps is out of sync w.r.t. vs2008 deps.

There are three commits in vs2008 deps not in vs2010 deps:
http://code.google.com/p/realxtend-tundra-deps/source/detail?r=f57fe31eef242b75c9bcfb34c7ecac064cfdb925&name=prebuild-vs2008
http://code.google.com/p/realxtend-tundra-deps/source/detail?r=c9937f0d986afaba5368f2423750a20dd8efcb73&name=prebuild-vs2008
http://code.google.com/p/realxtend-tundra-deps/source/detail?r=c869f5e9ff63e69546b3ccadc4e5ec3c82ce661a&name=prebuild-vs2008
gfxguru reported skyx not building for vs2010 currently.

Yes they have been lacking for a while as I don't use them with everyday work. Ill try to update libmumble and skyx at some point.

libmumbleclient has been removed, boban updates skyx and ali updated knet. Speex is afaik the only one missing, I'll try to push that at some point.
Though seems that we are doing to change how we aquire build deps so maybe I'll wait a bit to see how it goes.


Bridges map not resetting

Expected behavior
Hello the maps of bridge doesn't get reset
Observed/Actual behavior
So, when i play bridge there is already block placed and premade bridges when I checked the arena it looked fine
How to Recreate (if known)
No response
Stack trace
Paste your stack trace or a https://pastebin.com/ link here!
Plugin list

Frost version
12.2
Confirmations

[X] I have searched for and ensured there isn't already an open issue regarding this.

Other
No response

The blocks might be there if you didn't shutdown your server properly or if it crashed in the past.
You can check each arena copy with /arena manage, and also update or reset the arena copy chunks with /chunk.

Issue fixed


feat: logo component

Hi @AdeboyeDN, kindly review.
Light 👇🏼

Dark 👇🏼

Yes this is great

So here is the logo I recommend you use for the dark mode.

Thanks!  I'll make the change.

Hi @AdeboyeDN,
I realized that my former approach was too complex, and it didn't even work as it should. One thing that sorta hindered me also was the png images. I don't know why one of them was not being rendered at all, so I switched to the SVG format.
Below are the results.

Light theme
Dark theme

This looks great,
Thanks, man.

🚀
Please if any issue comes up in the future, I'd love to take it on.


show a warning message if country code was not added when mobile number is activated

@miltontony @nathanbegbie @Ntu2koTrevor @codiebeulaine ready for review :)

@saeedmb8 👍


Would need to update Drizzle DB generate command

Config File
{
"hasSrc": false,
"packages": [
"shadcn-ui",
"drizzle",
"lucia"
],
"preferredPackageManager": "npm",
"t3": false,
"alias": "@",
"analytics": true,
"rootPath": "",
"componentLib": "shadcn-ui",
"driver": "sqlite",
"provider": "better-sqlite3",
"orm": "drizzle",
"auth": "lucia"
}
Description of the bug

While running "npm run db:generate" I get the error " This command is deprecated, please use updated 'generate' command (see https://orm.drizzle.team/kit-docs/upgrade-21#how-to-migrate-to-0210)"
Here are logs:

<EMAIL_ADDRESS>db:generate
drizzle-kit generate:sqlite

To Reproduce
Steps to reproduce the behavior:

Go via kirimase config route
Try generating first instance of db
See error

Expected behavior
Drizzle db should be generated at this particular step
Desktop (please complete the following information):

OS: Windows
Browser: Chrome
Version: Chrome 125 on Windows 11

Resolved with #194