How to search by custom id field with select.row?

Thanks for the great project! I can't quite figure out how to use the select.row when the id is custom. My id is _id, how do I modify this setState function to look among the _ids instead of id? I know it says that you can do it in the manual, but I just couldn't figure out how from the code.
This is the sample code in the documentation. My initial guess was to sub in rows[selectedRowIndex]._id but that doesn't seem to work.
    this.setState(
      select.row({
        rows,
        selectedRowId: rows[selectedRowIndex].id
      })
    );

I can "fix" the issue by going into the row.js file and changing to row._id === selectedRowId on line 13 but that seems like a bad idea.

Hi,
I think you need to override isSelected. See the full source for a better idea of what's going on there.

Yes, I have looked at that particular file and that is where I saw on line 3 that it looks for row.id === selectedRowId instead of say row[customKeyName] === selectedRowId with the ability of passing in customKeyName to define the custom key name. I am really sorry but I think I am failing to understand how I can use isSelected to pass in a custom key name. How would I change the select.row call in my first post to pass in the custom key name without resorting to changing the row.js file?  Thanks so much!

No worries. I would try something along this:
select.row({
  rows,
  isSelected = (row, selectedRowId) => row._id === selectedRowId,
  selectedRowId
})

Let me know how it goes.

That's beautiful, it totally worked! Thanks so much, I really appreciate your help!


Partially revert connection timeout behavior

Related to issue #1790.
If there is a connection timeout the next route should
be tried. Timeouts during a request/response should
probably not be retried.
The timeout/interrupt behavior was changed in PR #1257
for issue #1146. This modified both connection and
request/response behavior. The issue reported was actually
about the use of Thread.interrupt(), not timeouts, but the
behavior change modified both behavior for interrupt and
timeout.
PR #1388 split the handling of exceptions so that separate
code now handles connection errors from that which handles
request/response errors. It faithfully kept the behavior
from PR #1257.
The modification of the timeout behavior during connection
caused problems on Android. Now that the two types of error
handling are split it's possible to deal with them
differently and return the connection behavior to trying
the next route. This does not modify behavior during
request/response.
A test has been added for timeouts. It does not seem to be
possible to interrupt a Socket.connect() call with
Thread.interrupt() on either OpenJDK or Android so a
placeholder has been implemented. If it ever is implemented
a Thread.interrupt() during connect will not be retried.

LGTM

Travis suggests the new test is flaky.
Tests run: 4, Failures: 1, Errors: 0, Skipped: 1, Time elapsed: 1.195 sec <<< FAILURE! - in com.squareup.okhttp.internal.http.ThreadInterruptTest
timeoutConnect(com.squareup.okhttp.internal.http.ThreadInterruptTest)  Time elapsed: 0.011 sec  <<< FAILURE!
java.lang.AssertionError: expected:<1> but was:<0>
	at org.junit.Assert.fail(Assert.java:88)
	at org.junit.Assert.failNotEquals(Assert.java:743)
	at org.junit.Assert.assertEquals(Assert.java:118)
	at org.junit.Assert.assertEquals(Assert.java:555)
	at org.junit.Assert.assertEquals(Assert.java:542)
	at com.squareup.okhttp.internal.http.ThreadInterruptTest$TimeoutCountingSocketFactory.assertTimeoutCount(ThreadInterruptTest.java:233)
	at com.squareup.okhttp.internal.http.ThreadInterruptTest.timeoutConnect(ThreadInterruptTest.java:122)

I am not surprised!

I have removed the test.


Getting translation issue in notification while Admin trying to log in the inActive user.

Terms

[X] Before you openning this issue, i have checked if the issue has already been reported.

Bagisto Version(s) affected
master
Issue Description
Getting translation issue in notification while Admin trying to log in the inActive user.
Preconditions

Admin should be logged in.
Admin should have at least one registered customer.

Steps to reproduce

Go to 'Admin panel'
Click on 'Customers/customer'
Click on the login option
See the error

Expected Result
It should show the notification properly.
Actual Result
Getting translation issues in the notification. Here
Screenshots
No response
Additional context
No response

Now, this issue is fixed. so I am closing this issue.
And I have attached a screenshot Here, for future reference.


logs: make log level configurable through meshconfig

New osmLogLevel option has been added under observability meshconfig
spec (seems the most related and relevant to the available categories)
to allow changing the log level of OSM at runtime.
The initial log level flag has been kept at it still mandates boot log
level up to the point configurator and the log level handler are both
up, which then meshconfig value takes over.
The handler can't be easilly moved to the logger package as it
introduces a hard dependency cycle between logger<->configurator, hence
the handler has been left on osm-controller main.
Adds a unit test, and see below a working example on a running deployment.
Fixes #3646
Signed-off-by: Eduard Serra<EMAIL_ADDRESS>
Description:

Affected area:

Functional Area

New Functionality
[ ]

Documentation
[ ]

Install
[ ]

CLI Tool
[ ]

Certificate Management
[ ]

Control Plane
[ ]

Ingress
[ ]

Egress
[ ]

Networking
[ ]

Observability
[ ]

SMI Policy
[ ]

Sidecar Injection
[ ]

Security
[ ]

Upgrade
[ ]

Tests
[ ]

CI System
[ ]

Demo
[ ]

Performance
[ ]

Other
[ ]

Please answer the following questions with yes/no.

Does this change contain code from or inspired by another project?

Did you notify the maintainers and provide attribution?

Is this a breaking change?

...
{"level":"info","component":"envoy/rds","time":"2021-06-23T19:07:17Z","file":"response.go:105","message":"RDS did not fulfill all requested resources (diff: Set{}). Fulfill with empty RouteConfigs."}
{"level":"info","component":"envoy/rds","time":"2021-06-23T19:07:17Z","file":"response.go:105","message":"RDS did not fulfill all requested resources (diff: Set{}). Fulfill with empty RouteConfigs."}
<only info logs til around now>

change to trace through kubectl edit meshconfig
{"level":"info","component":"osm-controller/main","time":"2021-06-23T19:07:58Z","file":"osm-controller.go:371","message":"Global log level changed to: trace"}
{"level":"trace","component":"certificate/CertRotor","time":"2021-06-23T19:08:02Z","file":"rotor.go:51","message":"Cert ads will not be rotated; expires in 87599h58m50.3121094s; renewBeforeCertExpires is 30s"}
{"level":"trace","component":"certificate/CertRotor","time":"2021-06-23T19:08:02Z","file":"rotor.go:51","message":"Cert osm-validator.osm-system.svc will not be rotated; expires in 87599h58m50.429469332s; renewBeforeCertExpires is 30s"}
{"level":"trace","component":"certificate/CertRotor","time":"2021-06-23T19:08:02Z","file":"rotor.go:51","message":"Cert server-0-1.server-0.cluster.local will not be rotated; expires in 23h59m10.113151758s; renewBeforeCertExpires is 30s"}
{"level":"trace","component":"certificate/CertRotor","time":"2021-06-23T19:08:02Z","file":"rotor.go:51","message":"Cert server-0-0.server-0.cluster.local will not be rotated; expires in 23h59m10.272788317s; renewBeforeCertExpires is 30s"}
{"level":"trace","component":"certificate/CertRotor","time":"2021-06-23T19:08:07Z","file":"rotor.go:51","message":"Cert server-0-1.server-0.cluster.local will not be rotated; expires in 23h59m5.112912093s; renewBeforeCertExpires is 30s"}
{"level":"trace","component":"certificate/CertRotor","time":"2021-06-23T19:08:07Z","file":"rotor.go:51","message":"Cert server-0-0.server-0.cluster.local will not be rotated; expires in 23h59m5.272500951s; renewBeforeCertExpires is 30s"}
{"level":"trace","component":"certificate/CertRotor","time":"2021-06-23T19:08:07Z","file":"rotor.go:51","message":"Cert ads will not be rotated; expires in 87599h58m45.311669134s; renewBeforeCertExpires is 30s"}
{"level":"trace","component":"certificate/CertRotor","time":"2021-06-23T19:08:07Z","file":"rotor.go:51","message":"Cert osm-validator.osm-system.svc will not be rotated; expires in 87599h58m45.429139366s; renewBeforeCertExpires is 30s"}
....

reset to info through kubectl edit meshconfig
{"level":"info","component":"osm-controller/main","time":"2021-06-23T19:08:12Z","file":"osm-controller.go:371","message":"Global log level changed to: info"}
<no more trace logs>

Codecov Report

Merging #3650 (88a7cc5) into main (3d2ec16) will increase coverage by 0.14%.
The diff coverage is 67.64%.

@@            Coverage Diff             @@
##             main    #3650      +/-   ##
==========================================
+ Coverage   67.05%   67.20%   +0.14%
==========================================
  Files         179      179
  Lines        8705     8741      +36
==========================================
+ Hits         5837     5874      +37
+ Misses       2837     2834       -3
- Partials       31       33       +2

Flag
Coverage Δ

unittests
67.20% <67.64%> (+0.14%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

pkg/configurator/mock_client_generated.go
0.00% <0.00%> (ø)

cmd/osm-controller/osm-controller.go
24.41% <81.48%> (+10.03%)
:arrow_up:

pkg/configurator/methods.go
69.44% <100.00%> (+0.43%)
:arrow_up:

pkg/catalog/outbound_traffic_policies.go
86.98% <0.00%> (+0.23%)
:arrow_up:

pkg/kubernetes/announcement_handlers.go
77.14% <0.00%> (+5.71%)
:arrow_up:

pkg/configurator/client.go
84.48% <0.00%> (+8.62%)
:arrow_up:

Continue to review full report at Codecov.

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

@Mergifyio backport release-v0.9


[Do not review] Experiment to remove unknown layers in manifest list

Add an extra build arg to turn off default provenance attestations. Reference: https://docs.docker.com/build/building/env-vars/#buildx_no_default_attestations

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Mention the appropriate issue in your description (for example: addresses #123), if applicable. This will automatically add a link to the pull request in the issue. If you would like the issue to automatically close on merging the pull request, comment fixes #<ISSUE NUMBER> instead.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.
To check the build health, please visit https://github.com/apache/beam/blob/master/.test-infra/BUILD_STATUS.md
GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI or the workflows README to see a list of phrases to trigger workflows.


Verbose console log does not include step parameters

Expected behavior
Verbose console log to include parameters
Actual behavior
parameters are marked as {}
Steps to reproduce

gauge init python
gauge run -v

# Specification Heading
  ## Vowel counts in single word
      * Vowels in English language are {}.	 ...[PASS]
      * The word {} has {} vowels.	 ...[PASS]
  ## Vowel counts in multiple word
      * Vowels in English language are {}.	 ...[PASS]
      * Almost all words have vowels {}	 ...[PASS]

Successfully generated html-report to => /.../sample-python/reports/html-report/index.html
Specifications:	1 executed	1 passed	0 failed	0 skipped
Scenarios:	2 executed	2 passed	0 failed	0 skipped

Gauge version
Gauge version: 1.0.5
Commit Hash: 562f036

Plugins
-------
html-report (4.0.6)
json-report (0.2.2)
python (0.3.5)
screenshot (0.0.1)
xml-report (0.2.2.nightly-2018-12-11)

This issue has been verified and found fixed with the version 1.0.6.nightly-2019-05-27


Rough and ready prototype of custom serialization

This is to demonstrate one possible approach to #2534.

Closing this now as the real implementation is on the way.


HOSTEDCP-1767: feat: install prerequisite CRDs for AROHCP

What this PR does / why we need it:

[!NOTE]
This PR is stacked on top of #4643 . It adds a single commit on top it and shouldn't be merged until #4643 is merged.

When installing the HO operator on an AKS cluster, there are a few CRDs the following CRDs need to be installed first:

podmonitors.monitoring.coreos.com
prometheusrules.monitoring.coreos.com
servicemonitors.monitoring.coreos.com
routes.route.openshift.io

This PR adds the capability to the install command of installing those CRDs when the --managed-service is set to ARO-HCP.
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes HOSTEDCP-1767
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[ ] This change includes docs.
[ ] This change includes unit tests.

/hold waiting for #4643 to be merged first

/retest-required

We make need to hold/rethink this PR a bit in light of RFE-6258. I don't know that we will need the Prometheus CRs now. We will still need the Routes CR though.

No longer needed
/close

Ah I thought we still needed this


Controller/Joystick Button 0 Launches a new Instance of the Game

Godot version: v3.0.6.stable.official.8314054

OS/device including version: Windows 10 Version: 1803 OS build: 17134.191; Fedora 28
It appears to be joystick and computer agnostic.

Issue description: When running the game, the button 0 on my joystick triggers running cmdline: "path/to/executable" "--path" "path/to/game". I was under the impression that this was a setting in the Project Settings > Input Map and in this case "ui_accept" (which maps button 0) but the "Enter" and "Space" keys don't behave this way. I was expecting to be able to map button 0 to perform a particular action but instead it appears to be stuck to launching a new window and starting the game again.

Steps to reproduce:

Launch the Godot Launcher
Select any game
Press run
Press button 0 on joystick 0

Minimal reproduction project:

It appears to me that this issue is project agnostic.

Duplicate of #15199, fixed in the master branch.

@akien-mga Apologies

There's nothing to apologize for, it's ok to file duplicates - it's our job to find and close them ;)


Ensure test runs in the Rolling-CI build that are stable and successful are marked as 'fail on error'.

We have many test phases running in our rolling CI build. We currently run tests for this matrix:

Windows
macOS

unitTest

testSingle

testMulti

testDebugger

Each matrix row/column runs tests under Python 2.7, 3.4, 3.5, 3.6, and 3.7.
Many of the specific runs are successful, but many are not. We've identified some of the major problems causing test failures and added them as issues, but we will necessarily not get to all of them in the near term.
Currently, all tests are all marked as 'fail with warnings' and not 'fail with error'. This failure as warnings is causing us a problem with our quality control - since all of our rolling builds succeed with warnings, we aren't aware of problems that are new.
To fix this, we should add another setting to our Rolling CI build where the existing tests that are succeeding and stable (= 5 or more builds where the tests for that matrix slot are successful) get marked as 'fail as error'. As each test matrix slot becomes successful, they should be moved into the 'fail as error' category.

Closed a while ago, but extra-closed as of #3555


[Suggestions] Will you please add multimedia protocol support？

First , thansk to develop the program.
I've ported it to ArchLinux AUR : https://aur.archlinux.org/packages/flexplayer
Will you please add multimedia protocol support , such as udp/rtsp/rtmp/hls/http/https and so on ?
Thanks a lot.
My English level is very poor, please forgive me a lot.

Thanks for your interest in flexplayer.
Since flexplayer is an electron app, it seems to me that it could easily support media that chrome natively supports. Also, I think hls can be supported using a JavaScript library (hls.js).
I'm not familiar with udp/rtsp/rtmp, so I did a little research, but I didn't think it would be possible to support these.
So, for example, I can probably implement a feature where you press Ctrl-N and type in an mp4 or m3u8 URL and a video will be added to the grid. Does this match your image?
By the way, my level of English is also poor and I am writing this text while translating.

Thanks a lot for your reply.I will try it later.


Added using-alias-directive support.

Seems to work correctly with Roslyn, but on Mono there seem to
be a bug in the Evaluator that throws a NullReferenceException.
Since the exception is so vague, I added a better error message
for the user.
I've filed a bug report with Xamarin for this that can
be found at https://bugzilla.xamarin.com/show_bug.cgi?id=32894#c0
Resolves #328

Merged


Installation breaks project

Adding the dependency to my project causes flutter run to spit out this error.
FAILURE: Build failed with an exception.

* Where:
Script '/home/tristan/Files/ProgramFiles/flutter/packages/flutter_tools/gradle/flutter.gradle' line: 221

* What went wrong:
A problem occurred configuring project ':logger_flutter'.
> Failed to notify project evaluation listener.
   > Could not get unknown property 'android' for project ':logger_flutter' of type org.gradle.api.Project.
   > Could not find method debugProvided() for arguments [file collection] on object of type
   org.gradle.api.internal.artifacts.dsl.dependencies.DefaultDependencyHandler.

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

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

BUILD FAILED in 1s
  Command: /home/tristan/Projects/CyWoodsAppFlutter/android/gradlew app:properties

Please review your Gradle project setup in the android/ folder.

I get the same error

check the logs to see whats wrong then :) Sorry could not help myself.

It looks like this project is missing all required Android files. The pubspec declares:
androidPackage: com.github.leisim.logger_flutter
pluginClass: LoggerFlutterPlugin
However none of these are present. I guess they should be added to Git?

I get the same error

I get the same error, ONLY when i add this dep to my pubspec.yaml, last 2 lines:
logger: ^0.7.0+2
logger_flutter: ^0.7.0
A problem occurred configuring project ':logger_flutter'.

Failed to notify project evaluation listener.
Could not get unknown property 'android' for project ':logger_flutter' of type org.gradle.api.Project.
Could not find method debugProvided() for arguments [file collection] on object of type org.gradle.api.internal.artifacts.dsl.dependencies.DefaultDependencyHandler.

I am facing same issue only on iOS, logger_flutter plugin is added to .flutter-plugins file but cannot be installed because the project does not include ios podspec file. From the git repo, it looks like it also does not include any Android files either (as a previous comment above reports). Is there a command or configuration in flutter to install a package and bypass the attempts to install platform specific code when none exists?

Thanks for reporting and sorry for the late reply. I fixed this in version 0.7.1.


[ refactor ] using a universe to clean up show's constraints

A quick and dirty demonstration. I have left the auxiliary definitions in the middle
of the file where I needed it but it should probably be moved to an auxiliary file.
As I was saying on zulip you could get even more mileage by distinguishing π (for
the natural number stored in a vector's cons) from → (for the A corresponding to
the vector's cons' head). Of course that comes at the price of having more constructors
to deal with when you write your generic operations. :(

Closing this as we ended up being able to resolve constraints using instance search.
However I don't think I ever got to thank you for this so a sincere thank you!
It was a nice trick to discover at the time, and I actually ended up with a similar kind of Helper construction (and your definitions for generic n-ary functions), although we use a functional representation to make constructions simpler.


Server side i18n

@hanzei I still prefer a panic when I'm not correctly generating the translation string (I would expect to catch that during any trivial test, and problably, in the future with unit testing), than a incorrect/hard-to-detect behavior.

The panic can even happen if the go code is correct. Certain languages e.g. require a few and many translation while others not. If it's missing, the plugin will panic. See https://github.com/matterpoll/matterpoll/issues/252 as an example.

Mmm, that is a very good point. I'll change it with the proposed solution.


Add support for Angular 1.5

Allow framework to be compatible with angular 1.5, without dropping support for 1.3 or 1.4

I am working with 1.5 and haven't encountered any issues thus far.

Updating to 1.5 causes the tests to fail: https://travis-ci.org/base-apps/angular-base-apps/builds

I'll take a look at that

Fixed


Fix bug in parsers.py

TextParser and ImageParser both access block_size_target_characters attribute of library when library is None, they should get it from parser.library

Looks good.  Thanks for the Pull Request!


[Entitas:] Format for CodeGeneratorData

Now CodeGeneratorData is a dictionary that maps a string literal constant to an object.
I suggest to replace a dictionary with arbitrary string keys to an ordinary class with fields. It would automatically define a format  for CodeGeneratorData, because now you should investigate CodeGeneratorDataProvider/CodeGenerator to understand what format it excepts.
CodeGeneratorData format also includes PublicMethodInfos that contains Reflection-specific fields which obviously cannot be used by other data providers.  I suggest to replace it to ordinary class with string fields.

The idea was that you can any kind of data to the dictionary. A class would be limiting for customisation. I agree, the PublicMemberInfos should be replaced with sth else in the first place, but the whole idea of the code generator refactoring was that you can add anything at any time.
This means, you can e.g. add a roslyn data provider that adds whatever you need to the dictionary. A custom code generator can read those values and produce any kind of code. In the Entitas Preferences Window you can select your data provider and code generator and everything should work. In this very case I understand it would be nicer if you only have to add a new data provider and use the existing code generator.

You could duplicate the existing component generators and use your data instead of public member infos

the whole idea of the code generator refactoring was that you can add anything at any time

You can have class fields for stable things like component public fields and an auxiliary dictionary for adding new features on the fly.  Anyway you have coupling between DataProvider and CodeGenerator and I think that an explicit declaration of common CodeGenerator requirements is a nicer than implicit.

Btw, I was thinking about roslyn again. It's great that you do this. If there's ever a way we can invoke roslyn from within unity, I'd replace the reflection based one. I think that would be very powerful! I just started migrating a very big project today and we have some custom generators, but they're all based on reflection. Of course, there are lot of compiler errors and I cannot re-generate, but the dsl doesn't help either because those generators are custom. So roslyn would be very beneficial for those cases!

I've made a Roslyn-based data provider, you can check it out https://github.com/vedun-z/Entitas-CSharp/commits/develop . Now I have two separate commits. First commit introduces MemberInfosData for holding field/property info. The second one introduces Roslyn support. I've checked that my provider creates the same data for Tests.Fixtures project.  :-D
I use nuget to install Rolsyn. It would work fine for windows, however, for mac/linux I except some troubles with msbuild. Roslyn uses msbuild to open solution/project files and  as far as I remember We've used custom build of msbuild and 4.6 runtime in mac/linux systems. So, If you have some ideas how to create multisystem Rolsyn distribution, please share them.


Crystal tool context/implementations with ECR

Feature Request
I am developing the vscode extension and it would be useful to improve the experience writing ECR through type information / jump to definition. The context and implementation tools should both be able to accept an .ecr file just the same as it currently does a .cr file, with line/col information, and if there exists a method/variable/etc at the location, it should return relevant information. The only other solution would be a custom ECR parser, but it would be better to have this integrated into the compiler itself. This does not break backwards compatibility.
greeter.ecr
<%
  my_var = name
%>

greeter.cr
require "ecr"

class Greeter
  getter name : String

  def initialize(@name : String)
  end

  ECR.def_to_s "greeter.ecr"
end

Greeter.new("John").to_s # => "Greetings, John!\n"

$ crystal tool context -c greeter.ecr:2:12 greeter.cr

1 possible context found

| Expr | Type   |
| name | String |

$ crystal tool implementations -c greeter.ecr:2:12 greeter.cr

1 implementation found
/path/to/greeter.cr:4:3
 ~> macro getter: expanded macro: macro_12341234:123:12

It would also be useful if crystal tool format worked with ECR's as well, just for verifying the syntax / grammar of the file


Experimental EIP-1559 support

Well here it is, the long awaited support for EIP-1559 (and other new transaction types)
This is a pretty significant rewrite of the EthereumTransaction core to a more modular approach. With this modular approach, adding new transaction types, in the future, will be much easier (if this PR were to be adopted). This PR does contain some breaking changes, out of necessity, in how you interact with an EthereumTransaction object. As a result of the breaking changes, it may make this PR undesirable. I've done my best to add a deprecated shim layer so that legacy code should still compile and work, albeit with deprecated warnings, with little to no modification.
What this PR brings:
Full encode, decode, signing, and verification of Legacy, and EIP-2718 type transaction envelopes.
Currently EIP-2718 defines two transaction types; EIP-2930 and EIP-1559, both are fully supported for encode, decode, signing, and verification.
I've also built on-top of the excellent work already done by Yaroslav and others for EIP-1559 support (now in PR #510) in the Feature/EIP-1559 branch. So included here is the work already done for the new Gas calculations and block management, though that work is still incomplete and not fully integrated. As it stands now EIP-1559 transactions can be created, signed, and sent, but automatic gas calculation is not implemented.
All parts of the library that rely on EthereumTransaction have been updated as necessary to work with the new Transaction Interface. Though any code that works with gas still only understands and works with the legacy values.
Things needed
Testing, Testing... testing!!! While I've added some tests to validate the decode, verification [and by extension encode] more tests are needed. Preferably made by someone other than me.  Having someone else who has their own independent interpretation of the specs would help to find any errors in implementation. [though I have tested this code on Rinkeby with good success]
Documentation! As always we need to document how things work. I've added a bunch of quick-help style comments in the code to aide in this direction, but more work, and help, here is needed.
Automatic Gas calculation. This really needs to be complete before we can claim full support for EIP-1559.
Creating Transactions
For the most part things work as they did before. The notable change is you use the TransactionOptions structure to pass the gas and other transaction specific parameters to the transaction object.
    /// Universal initializer to create a new EthereumTransaction object
    /// - Parameters:
    ///   - type: TransactionType enum for selecting the type of transaction to create (default is .legacy)
    ///   - to: EthereumAddress of the destination for this transaction (required)
    ///   - nonce: nonce for this transaction (default 0)
    ///   - chainID: chainId the transaction belongs to (default: type specific)
    ///   - value: Native value for the transaction (default 0)
    ///   - data: Payload data for the transaction (required)
    ///   - v: signature v parameter (default 1) - will get set properly once signed
    ///   - r: signature r parameter (default 0) - will get set properly once signed
    ///   - s: signature s parameter (default 0) - will get set properly once signed
    ///   - options: TransactionObject containing additional parametrs for the transaction like gas
    public init(type: TransactionType? = nil, to: EthereumAddress, nonce: BigUInt = 0,
                chainID: BigUInt? = nil, value: BigUInt? = nil, data: Data,
                v: BigUInt = 1, r: BigUInt = 0, s: BigUInt = 0, options: TransactionOptions? = nil) {

TransactionType is the enum that controls the type of transaction that will be created
public enum TransactionType {
    case legacy
    case eip2930
    case eip1559
}

When using a higher level API, you can still create new transaction types by setting type within the TransactionOptions object you pass. Alternatively you can use the applyOptions function with type set to migrate the transaction type before signing and transmitting.
Direct access to the type specific parameters is no longer available. getOptions() and applyOptions() can be used  to query or modify any of those parameters. This is how you will need to access all the gasXXX parameters. A given transaction type will ignore any gas parameters it does not support.
That's all for now
I'm sure there will be questions... ask them below. And please test this code, and comment, and help find the bugs.
I'm not going to mark it as a draft PR for the time being, but I consider it still incomplete, though the core functionality is all there. It's a good start, and I wanted to get it out there so people can start working with it to iron out any issues.
Closes #349

Will continue the review later. Not finished yet 🙂

Rebased on the latest push from Yaroslav. Apparently the GasLimit for the block API has been fixed for 1559 now. I'll have to look at how to incorporate that into the code for automatic resolution now.

merged in changes in #510 not in Feature/EIP-1559

now that #510 was merged into develop, we should be in the home stretch of changes for this PR. I have a few potential changes I'm considering, centered around the gas parameters in prep for automatic gas estimation (if selected) to make things operate in a cleaner and more consistent manner across the transaction types. The bulk of the changes for gas will depend on a separate PR to implement the gasOracle that Yaroslav has planned. So automatic gas resolution may end up being in a separate PR, depending on timing, and interdependence between the PR's

Updated the tests and added signing tests
Fixed a bug with signing the new transaction types. The swift wrapper on the SECP256K1 we have is unexpectedly hard-coded to return 27/28 for V instead of the 0/1 parity I was expecting. Compensated for it in the EIP-2718 Envelope extension so the new transaction types automatically inherit the fix.

@mloit I've left a set of comments+suggestions some of which can be ignored as there is nothing major.
A few useful improvements like an option to set an optional message in Web3Error.dataError to understand the context of the issue better are amongst these comments.
In some places just typos or possibilities to shorten the codebase.
Most likely I've left comments on some parts of the codebase that are just reorganized code but since it appears in PR's changes I decided not to skip it.
Thanks for the great work!

@mloit I've left a set of comments+suggestions some of which can be ignored as there is nothing major.

Thanks for the review. You made some good suggestions, many of which I used. There were some that I did not, despite being excellent suggestions. I felt they moved beyond the scope of this PR, but are certainly great candidates for a PR on their own, or part of a more generalized refactor. I'll be pushing shortly incorporating many of your suggested changes.

I have no more comments for this PR.

Reverted some of the changes made previously back to what's in develop in rep for a bit of a refactor on how I handle the transaction parameters.  The codebase still works, you will just get a lot of deprecated warnings, as most of what I backed out were the external changes I made within the library to interact with EtereumTransaction in the new way.

The failure is due to an implementation error in Oracle for legacyGasFees, specifically gasPriceLegacyPercentiles()

Found a workaround/fix. It seems some nodes are reporting a gasPrice value in the JSON for EIP-1159 transactions, even though EIP-1159 transactions have no such field. Oracle appears to depend on this, so the workaround is to store this non-standard value still and report it as gasPrice to satisfy Oracle. This really should be corrected for 3.0, and the workaround removed.


Syntax Highlighting for the preview pane

Does it make sense to support this? since the preview pane only shows the matched lines.
I initially supported syntax highlighting, but it was quite slow.
the highlighting should be:

lazy loaded
https://github.com/extrawurst/gitui for inspiration.

I would recommend using https://github.com/trishume/syntect, which is I believe what the great TUI file manager https://github.com/sxyazi/yazi uses to syntax highlight files incredibly fast!

That's such a great file manager, i will get inspiration of how they do it, thanks for the suggestion 🙏.


Hit And Miss morphological operation

Implementation of Hit And Miss morphological operation

Based on the code provided in this page: http://opencv-code.com/tutorials/hit-or-miss-transform-in-opencv/ . Minor fixes added to prevent incorrect results due to erosion with all-zeros kernel (refer to this Q&A post)

New documentation refers to upper link, but I plan to modify such tutorial to publish in OpenCV site, showing new functionality of morphologyEx function.

ocl version not implemented

If everything is alright and gets merged, I will provide 2.4.x PR

Code sample showing functionality here

:+1: nice one :)

@ilya-lavrenov ok, you can take another look

thanks! :+1:

@LorenaGdL thank you for the algorithm! Could you provide the same for 2.4?

@StevenPuttemans that was already on my to-do list (check above). I'll do it as soon as I have some spare time, and same applies to the tutorial :)

:+1:


Errors with new version of Omnisharp

I updated my omnisharp-server this morning with the newest version via Mason (Neovim).
The server starts, but as soon as I try to use GoToDefinitio or any other method I get an error.
Error when executing textDocument/definition : Internal Error - System.MissingMethodException: Method not found: 'System.Collections.Generic.IEnumerator1<Microsoft.CodeAnalysis.NodeStateEntry1<System.ValueTuple2<System.__Canon,System.__Canon>>> Microsoft.CodeAnalysis.NodeStateTable1.GetEnumerator()'.
09:58:07 AM msg_show    at Microsoft.CodeAnalysis.ExternalAccess.RazorCompiler.HostOutputNode`1.AppendOutputs(IncrementalExecutionContext context, CancellationToken cancellationToken)

@allisonchou knows all about this error and the pain it has caused. I was on holiday. 😛

We need to have O# include and prefer a matched version of the RazorCompiler external access dll (matched meaning the same version as our roslyn version). Sorry I forgot about this, I thought that the razor support was an entirely separate process and didn't need to line up with O# versions at all. See https://github.com/dotnet/roslyn/pull/70176.


Only host can enter interior dungeon

like the name states, only the  host is able to enter the facility/dungeon in a multiplayer lobby, everyone else gets dropped into the void upon entering the doors falling for infinity,

I recant my bug report, turns out to be the dependency for this mod, Lethal Expansion, Keep up the planet making king!

Yeahh LE is a little problematic, looking into the new alternative and will likely switch the moon's dependency.


Data product content Schema

HAving the data product data flow Schema available would make it easier to prepare usage and use the Schema in validation of the data stream. Also providing a sample would be nice.

Duplicate, more accurate issue is #209

Duplicate


Button break style after 4.28

Laravel Nova 4.28

Laravel Nova 4.27

Nova remove the "LoadingButton" from the components and created a seperate js package called "laravel-nova-ui" with a single Button component in it. To fix it, you can simply create a package with "artisan nova:asset" and copy-paste the LoadingButton from a previous version. I made myself a package for that and you are free to use it to fix the problem in the meantime laravel-nova-ui-fix

Heya guys! Sorry for the delay. I fixed the buttons in version 1.7.2 now. ;)


Is it real to run this by 8Gb VRAM ?

I'm trying to run by1070 8Gb, but have only this:
CUDA out of memory. Tried to allocate 16.00 MiB (GPU 0; 8.00 GiB total capacity; 7.33 GiB already allocated; 0 bytes free; 7.36 GiB reserved in total by PyTorch)
Could you explayn is it real to run and what I have to do ?

You can run it. Go to generation.py in the Bark folder and set SUNO_USE_SMALL_MODELS to True. This should let you download smaller models and have no issue with generation with this turned on.

@Acephalia Any idea how does it affect the results?

In my opinion it is currently only usable for a little bit of fun and experimentation.
I don’t know if there is any quality drop between the full model and smaller models the samples provided are cherry picked and you are not going to get that kind of output without a lot of work and generations. Haven’t eeen anyone else say otherwise yet either.
But remember it’s still pretty amazing and barely in its infancy, and if we look at how fast things like stable diffusion etc have developed it shouldn’t be too long before Bark evolves into something pretty substantial.
I foundthis fork it is the best GUI option currently imho. If you are still having trouble with this, give that one a go.

gonna close since a bit off topic for the issue


MudDatePicker don't display selected date when DateMask is set

Bug type
Component
Component name
MudDatePicker
What happened?
For a MudDatePicker with empty value and a DataMask the date is not displayed after a date selection.
When the DateMask is removed from the component, the date selection works as expected.
Expected behavior
The date should be displayed after a date selection.
Reproduction link
https://try.mudblazor.com/snippet/mucGOiOioUakDaOf
Reproduction steps

Click the icon to open the calender and select a date.
The selected date is not displayed.
Clickt into the input field and press space.
The date is displayed.
Click the icon again to open the calender and select an other date.
The displayed date don't update.
Clickt into the input field again and press space.
The displayed date has been updated.

Relevant log output
No response
Version (bug)
6.0.13
Version (working)
No response
What browsers are you seeing the problem on?
Firefox
On what operating system are you experiencing the issue?
Windows
Pull Request

[ ] I would like to do a Pull Request

Code of Conduct

[X] I agree to follow this project's Code of Conduct

I have the exact same issue:
when I add DateMask and Editable to a MudDatePicker, the mask works correctly, but selecting a date from the picker doesn't work. If I remove the mask, selecting a date works again.
Tested on MudBlazor 6.9.0 and net7

I found the problem. You need to reassign the mask property using new DateMask() because if you set the mask to a variable, like DateMask mask = new DateMask(), and then assign @mask to the component, the mask stops working after selecting the date via the picker. Try here selecting the date via picker (first work, second not)
I solved this in my custom component that extends the MudDatePicker by overriding the Submit() method and reassigning the Mask.
I hope this can be useful to someone.


Can i customize the markdown syntax instead of CommonMark syntax ?

Since i am developing an user-generated content project, i would like to display some content with markdown. But i dont want the user to use those markdown.
However, the user content may contains the syntax like **
Solution 1:
I would like to customize the markdown syntax to avoid user using the markdown syntax, such as :
** to [bold][bold]
Solution 2:
Add a specific character such as \ to skip the markdown syntax checking.
Is it possible ? Thanks a lot.

I'd suggest to read docs about customization principles https://github.com/markdown-it/markdown-it/tree/master/docs
The general question is out of this project scope.


Fix UI issues

[x] fonts in expander component
[x] icon alignment in COPY link and Open In Data Viewer link

@katamartin,
i'm trying to make actionable buttons like "Copy" and "Open in Data Viewer" stand out by giving them a distinct style. how does this look? :)
https://leap-data-catalog-ax1j7qgao-carbonplan.vercel.app/?catalog=https://raw.githubusercontent.com/leap-stc/data-management/migrate-pangeo-forge-osn-feedstocks/catalog/output/consolidated-web-catalog.json

Getting used to the colors, but I think I like them! Also as a whole, this is looking a lot cleaner ✨
Some hierarchy issues on large screen sizes:

The store expander font-size is the same as the top-level title (both 24px). The store names should be smaller, as they are on narrower screens.

Now that you're using color to highlight the "Copy" and "Open in data viewer" buttons, I wonder what it would look like to also used color to render the store name? Like the actions that are already rendered in color are relevant to the specific Zarr store, so might be nice to make that connection through repeated use of color.

The font-size of the grey links at the bottom is inconsistent with the other metadata values (18px vs 16px). I'd suggest bumping to 18px across the board at this breakpoint.

A few more points of visual polish:

The badge padding still looks off!

The horizontal spacing between the store name and expander element is too large. Something closer to the proportions here would be good.

It looks like you've already noticed this, but some of the colors are very bright and hard to read against the background (e.g., the colors currently shown for GPCP, the "LEAP Data Library Prototype," and EVOLV2K). If we move forward with this pattern (I think it's pretty nice!), we should make sure all colors are readable. We could achieve this by curating a different (and shorter if needed) set of colors and/or using mix() to tone down bright colors by mixing with the primary color.

@katamartin,
i'm trying to make actionable buttons like "Copy" and "Open in Data Viewer" stand out by giving them a distinct style. how does this look? are the alternating colors a bit much ? :)
https://leap-data-catalog-ax1j7qgao-carbonplan.vercel.app/?catalog=https://raw.githubusercontent.com/leap-stc/data-management/migrate-pangeo-forge-osn-feedstocks/catalog/output/consolidated-web-catalog.json

I quite like the colors, but can we use the 'leap' blue and green maybe? Or do we need more than two?

Thanks for all the work here. Quite excited to get this out!

I quite like the colors, but can we use the 'leap' blue and green maybe? Or do we need more than two?

@jbusecke, i just tried the leap blue and green
const colors = ['rgb(189,214,236)', 'rgb(50,117,105)' // extracted these values from the logo

and here's what they look like:

in my opinion, they don't look very good in this context. curious to hear your take

@katamartin / @norlandrhagen / @jbusecke, when you get a chance, this PR is ready for another round of review :)

I quite like the colors, but can we use the 'leap' blue and green maybe? Or do we need more than two?

@jbusecke, i just tried the leap blue and green
const colors = ['rgb(189,214,236)', 'rgb(50,117,105)' // extracted these values from the logo

and here's what they look like:

in my opinion, they don't look very good in this context. curious to hear your take

Hmmm I don't hate the blue but the green is too dark for sure.

thank you all for the feedback! So glad to see this PR brought to completion


Nativescript 8 support

Hello, I was curious to know if there were plans to provide support for NS8.
Thanks that's all.

https://docs.nativescript.org/plugins/localize.html

I've try to hack Nativescript-localize for VUE project.
Please help me to guide to do it.


Glass.Sitecore.Mapper.SitecoreContext.SitecoreContextQuery.QuerySingleRelative

Current (doesn't use query result, but instead uses current item):
public T QuerySingleRelative(string query, bool isLazy, bool inferType) where T: class
{
Item item = Context.Item;
item.Axes.SelectSingleItem(query);
return base.CreateClass(isLazy, inferType, item);
}
Should be:
public T QuerySingleRelative(string query, bool isLazy, bool inferType) where T: class
{
Item item = Context.Item;
Item result = item.Axes.SelectSingleItem(query);
return base.CreateClass(isLazy, inferType, result);
}

Cool!
PS: Thank you for making an awesome framework :^)

Hi
I have pushed the fix into the repo. I need to make a few other updates as well (mainly adding more commments) so it will be the weekend before I make a release.
You can check the fix here:
https://github.com/Glass-lu/Glass.Sitecore.Mapper/blob/master/Source/Glass.Sitecore.Mapper/SitecoreContext.cs
Also do you have a twitter account?
Cheers

I have pushed a release to Nuget that fixes the problem, it is in release <IP_ADDRESS>

My twitter account is http://www.twitter.com/uliweltersbach


Language match for Irish

Hello all,
Sorry to keep bombarding, but I wonder if Irish English (iris1255) would be a better language match than Irish (iris1253) for the society ‘Irish’ (Cg3, xd543). The focal year for this society is 1930. In the 1926 census only 18% of the population of the Irish Free State reported even being able to speak Irish.
Best wishes,
Oliver Sheehan

Hi @olcsheehan, I understand that Irish English is better in terms of most speakers, but on the other hand, in terms of history and phylogeny, most Irish people were probably descendants of people speaking Irish/Celtic languages not English/Germanic.
I asked Harald Hammarström who was involved in the creation of D-PLACE, and he also thinks that the principle should be the historical language.


Merge in nmdc-schema v10.5.2

closed by https://github.com/microbiomedata/berkeley-schema-fy24/pull/187


/opt/zeek/auxil/libkqueue/src/common/debug.c:1: fatal error: opening dependency file CMakeFiles/objlib.dir/src/common/debug.c.o.d: Permission denied

I am getting this error while installing zeek?
Any lead due to what issues it's giving me error

For Zeek installation issues, I would recommend asking them over at the Zeek GitHub Repository.
https://github.com/zeek/zeek


No update on the changes over New Exchange Admin center

I certainly enjoy the change but few thing they are considering that compliance portal can handle but i guess they are far behind to give that feature available through changes.
One of the important feature i am considering here is Retention tags, and I strongly believe this feature should not be remove.

from https://docs.microsoft.com/en-us/exchange/security-and-compliance/messaging-records-management/default-retention-policy
They say Retention policy from compliance center should take care of it from here. But i strongly disagree. There is no option to create tag on the basis of folders, No labels for moving data into archives after certain period.
MRM is not part of legal compliance, Team has to understand that it is always the part of Mailbox Hygiene and should still be there.
In this specific article they are not even talking on how to handle 3 year, 4 year and 5 year move to archive through new portal updates be it be Exchange admin center or Compliance portal.
Please suggest! how we can achieve?
~Yash from CloudNerdy
yash<EMAIL_ADDRESS>
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 81e9c1f9-d22d-8c83-25bb-637f3121ef33
Version Independent ID: b6c1f57d-978f-c959-ef0a-00ef1b9d9cab
Content: Set up an archive and deletion policy for mailboxes in your organization - Microsoft 365 Compliance
Content Source: microsoft-365/compliance/set-up-an-archive-and-deletion-policy-for-mailboxes.md
Service: o365-seccomp
GitHub Login: @markjjo
Microsoft Alias: markjjo

@yashbhilware Thank you for your feedback.

You should continue using MRM to move messages to archive mailboxes as the feature is not available yet in the New exchange admin center or compliance center. Once the feature if available, you will see that added to the document.

Retention labels can be applied on folder level in outlook -https://docs.microsoft.com/en-us/microsoft-365/compliance/create-apply-retention-labels?view=o365-worldwide#manually-apply-retention-labels

The recommendation is to use retention policy and retention labels from the compliance center for proactive deletion or retention as this is a single mechanism to control the content across Microsoft 365. Also, note that MRM will continue side by side with the retention policy and labels as mentioned in the document.
Hope this helps!
Thanks
Sri


Unhandled exception – provide the application access to the exception details.

Hi Martin
If I understood the code correctly, the library is catching unhandled exceptions and then calls Win32ServiceHost.ReportServiceStatus without throwing the exception?
Is there a way to notify the application about such an exception?
…So that the application for example can write a last “emergency” log statement bevor it exits.
Maybe an event similar to AppDomain.CurrentDomain.UnhandledException?
Btw.
Really cool library, thanks for sharing it.

I've already heard feedback about not really being able to debug exceptions. The problem with throwing is that it breaks the connection to windows' service management system.
I'm open to suggestions for how to handle this best.
The netfx ServiceBase reports to the event log, which also isn't available in .net core yet, it will be part of the windows compatibility pack in the .net core 2.1 timeframe. This also contains ServiceBase for .net core which kind of obsoletes part of this library (except for service creation/update/delete).
Two options:

Wait until .NET Core 2.1 release and write to the event log on windows (the compatibility pack should be available for netstandard2.0 but released alongside .NET Core 2.1). I don't want to create another PInvoke wrapper over the event log APIs, esp. since there is one coming in the compatibility pack.
Provide a hook on the service host like events, IExceptionReporter or similar.

Having an entry in the event log (with .Net Core 2.1) would work for me.


How could I use this in developer visual mode? Thank you

Tried to use this in developer visual mode but it doesnt work

I use 'pbiviz start' but it doesn't work

These visuals have been built outside of the pbiviz-cli ecosystem, so they don't have a developer mode built in.  You'll need to generate a fresh build whenever you update though.

The command to build has been changed to npm test, the README hasn't been updated since we are in the process of transitioning to a new build system.


Feat: Validate that referenced columns exist upstream

Fixes #482

Closing this because there are some unresolved issues I'm currently working on, may reopen later.


AB#169308: Update error-run-copy-activity-azure.md

Add a note informing the requirement of a JAVA_HOME environment variable for Adoptium OpenJDK installations.

Learn Build status updates of commit 6b0e111:
:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

support/azure/general/error-run-copy-activity-azure.md
:white_check_mark:Succeeded

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 learn.microsoft.com contributor guidesPost your question in the Learn support channel

@lestermarch Thanks for your contribution. We will send the changes to the triage team for review. You can check the work item AB#169308 for the latest status.


text cut off at halt when appendRecords with multi-byte encoding

https://github.com/yortus/DBFFile/blob/5b82aaed1f8d0440f6540d53b7ec4bc3ee0d9d52/src/dbf-file.ts#L365
this line should be
let byte = k < b.length ? b[k] : 0x20;

instead of value.length

Thanks @mmagi! Should be fixed in v1.3.0.


⚠ Bofrost.de has degraded performance

In bbe503a, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 1331 ms

Resolved: Bofrost.de performance has improved in 63089ea after 6 minutes.


⚠ Mastodon server has degraded performance

In 6ca2ab6, Mastodon server (https://so.fyle.uk/) experienced degraded performance:

HTTP code: 200
Response time: 12605 ms

Resolved: Mastodon server performance has improved in 9abcd02.


Clarification about reporting in Non-Screenplay scenarios

Hi @jan-molak ,
It is really sad that we can't take the screenshot of each step in a Non-Screenplay pattern. Can we put that for enhancement?

Come back to the actual issue. Can I configure a custom message in Check.whether? "Inspector ensures that 'failed' does equal 'failed'" may not be an appropriate message to show.

Not only that, if we set the stage as suggested in the description of this issue, we get a message like:

serenity.callToStageFor(...) has been deprecated. Please use `actorCalled(name)` and `actorInTheSpotlight()` functions from @serenity-js/core to access the actors instead.

Screenshots in each step are highly required.  It was working fine in Serenity 1.x in Non-Screenplay scenarios. Is there any workaround to get it solved?

My terminal is not showing the passed tick marks on passed scenarios.

Hey @abhinaba1080 - it looks like 4 different issues in one ticket, could we tackle them separately and close this one, please?

Closing as we have separate tickets to track those requests:

see #362 for reporting for non-screenplay scenarios
see #524 for better support of Windows terminals


TST: fix test failure due to changes in numpy scalar behavior.

Closes gh-16174. Note that this will likely be fixed in NumPy, but the test is fine written this way and it fixes an annoying CI failure, so we can leave the test like this.
[skip azurepipelines]

This fixes the failure (CI is not reporting status, but the job that was failing before completes now). This makes CI all green again, so let's merge this straight away.


Prognostic run tests fail via some entrypoints lacking FSSPEC_GS_REQUESTER_PAYS

There are several ways to get into the prognostic run container to run the tests (make enter_prognostic_run && pytest, test_prognostic_run, image_test_prognostic_run, cd workflow/prognostic_c48_run && make test, etc. Running the tests requires that FSSPEC_GS_REQUESTER_PAYS=vcm-ml be set, but currently only image_test_prognostic_run sets this variable upon entering the container, so the others fail. We should simplify the number of entrypoints and ensure that the variables needed for the tests always get set.

Resolved by #1837.


Customize Template/Markup

We would like to be able to customize the classes used in the markup of the colorpicker.
To achieve that we would like to make the used classes and/or the markup configurable and would provide a PR for that.
Do you want to go into that direction?
cc @Xiphe

Hi, are you speaking specifically about overriding this markup? https://github.com/bgrins/spectrum/blob/61ad86cbccf9e494a83eb8cd5ccd8eef068e7501/spectrum.js#L93-L129.  My main concern would be that if some selectors changed then behavior wouldn't get bound to the elements, like here: https://github.com/bgrins/spectrum/blob/61ad86cbccf9e494a83eb8cd5ccd8eef068e7501/spectrum.js#L214-L228.  Or are you talking about options for changing the names of all of those classes?

Hey

Hi, are you speaking specifically about overriding this markup?

Yes, we would like to be able to pass our own markup as an option.

Or are you talking about options for changing the names of all of those classes?

Yes, we would then pass all varying selectors/classes as options too.

What's your desired markup look like?

I think this is quite similar to, maybe less so, but anyway: I'm looking for a Spectrum where both the OK and Cancel would be HTML button's (Ie. a button element).
Now currently the Cancel is a "a" (anchor element) essentially a different kind of HTML element. It's small step, but haven't got into doing it - since I'd like to still use the original as much as possible, not directly kniving the source. The package comes via build system so changes on that level would get overwritten, if I understood correctly.

Now currently the Cancel is a "a" (anchor element) essentially a different kind of HTML element. It's small step, but haven't got into doing it - since I'd like to still use the original as much as possible, not directly kniving the source. The package comes via build system so changes on that level would get overwritten, if I understood correctly.

I'm curious, what's the use-case for having it be a <button> - for styling purposes?  You could always have extra css rules on the page to modify the built in styles without needing to make changes to the lib.  And if we do allow arbitrary elements to replace the built-ins it's likely you'd need to bundle that change with style changes anyway, since the default theme is built assuming certain elements are there (that could probably be partially mitigated by changing the theme, although it would be some work).


Fix the profiles test which is trying to install wego into a test-namespace

https://github.com/weaveworks/weave-gitops/blob/main/test/acceptance/test/profiles_test.go#L32
		namespace        = "test-namespace"

Ah, so a week ago, this restriction was introduced. But because the profiles test is still pended ( until the acceptance test run is figured out ) it didn't surface yet. Until now. :)

Oh, I see. The install namespace needs to be the same as the WegoNamespace is configured in. This was added in https://github.com/weaveworks/weave-gitops/commit/3af4189b6e78b9896428558da73b9d2dda7a8f77,

This is fixed now.


Deriving Hash, PartialEq, Eq on With

I have a use case which involves using With wrappers for both an IndexMap and its constituent key/value pairs (also leveraging rkyv-with since the original type is not in my crate). The existing implementation of ArchivedIndexMap.serialize_from_iter_index() however requires the keys to implement Hash + Eq, which With does not.
I therefore propose deriving Hash, PartialEq and Eq on With.

Can you provide an example of how you're using With? With is really more of an implementation detail than a struct that should be used standalone.

I tried to put together a working example, see below. To my mind it really comes down to satisfying what ArchivedIndexMap::serialize_from_iter_index expects using a With-Wrapper.

External Crate
use indexmap::IndexMap;

#[derive(PartialEq, Eq, Hash)]
struct UnarchivableKey {
    idx: u32,
}

struct UnarchivableValue {
    val: String,
}

struct UnarchivableStruct {
    map: IndexMap<UnarchivableKey, UnarchivableValue>,
}

My Crate
use indexmap::IndexMap;
use rkyv::{Archive, with::{ArchiveWith}, Deserialize, Serialize};
use rkyv_with::{ArchiveWith};

use crate::wrapper::AsKvMapped;

#[derive(PartialEq, Eq, Hash, Archive, ArchiveWith, Serialize, Deserialize)]
#[archive_with(from(UnarchivableKey))]
struct ArchivableKey {
  idx: u32,
}

#[derive(Archive, ArchiveWith, Serialize, Deserialize)]
#[archive_with(from(UnarchivableValue))]
struct ArchivableValue {
  val: String,
}

#[derive(Archive, ArchiveWith, Serialize, Deserialize)]
#[archive_with(from(UnarchivableStruct))]
struct ArchivableStruct {
  #[with(AsKvMapped<ArchivableKey,ArchivableValue>)]
  #[archive_with(from(IndexMap<UnarchivableKey, UnarchivableValue>), via(AsKvMapped<ArchivableKey,ArchivableValue>))]
  map: IndexMap<UnarchivableKey, UnarchivableValue>,
}

With-Wrapper for indexmap
use std::marker::PhantomData;
use indexmap::IndexMap;
use rkyv::{with::{ArchiveWith, SerializeWith, With}, collections::{ArchivedIndexMap, index_map::IndexMapResolver}, ser::{Serializer, ScratchSpace}};
use core::hash::Hash;

/// Rkyv With-Wrapper for archiving an IndexMap<OK,OV> as an IndexMap<K,V>
pub struct AsKvMapped<K, V> {
    _k: PhantomData<K>,
    _v: PhantomData<V>,
}
impl<OK, OV, K, V> ArchiveWith<IndexMap<OK, OV>> for AsKvMapped<K, V>
where
    K: ArchiveWith<OK>,
    V: ArchiveWith<OV>,
{
    type Archived =
        ArchivedIndexMap<<K as ArchiveWith<OK>>::Archived, <V as ArchiveWith<OV>>::Archived>;
    type Resolver = IndexMapResolver;

    unsafe fn resolve_with(
        field: &IndexMap<OK, OV>,
        pos: usize,
        resolver: Self::Resolver,
        out: *mut Self::Archived,
    ) {
        ArchivedIndexMap::resolve_from_len(field.len(), pos, resolver, out)
    }
}
impl<OK, OV, K, V, S> SerializeWith<IndexMap<OK, OV>, S> for AsKvMapped<K, V>
where
    K: ArchiveWith<OK> + SerializeWith<OK, S> + Hash + Eq,
    V: ArchiveWith<OV> + SerializeWith<OV, S>,
    S: ScratchSpace + Serializer + ?Sized,
    OK: Hash + Eq,
{
    fn serialize_with(
        field: &IndexMap<OK, OV>,
        serializer: &mut S,
    ) -> Result<Self::Resolver, S::Error> {
        let mapper = field
            .iter()
            .map(|(k, v)| (With::<OK, K>::cast(k), With::<OV, V>::cast(v)));
        unsafe {
            ArchivedIndexMap::serialize_from_iter_index(
                mapper,
                |k| field.get_index_of(k.as_ref()).unwrap(),
                serializer,
            )
        }
    }
}

That said, I'm not particularly confident that this is the right way of going about it and would love to learn about alternative approaches for achieving the same goal (idiomatically archiving an IndexMap<OK,OV> as an ArchivedIndexMap<K,V>, with OK, OV being in an external crate).

Sorry, rkyv_with is someone else's crate for working with rkyv. You probably meant to file against https://github.com/MaxOhn/rkyv-with.


Ignore hidden files in opa test

We can't have opa test load files from .github directory, the YAML
files in there will not be of the same schema, so we'll end up with:
$ opa test .
1 error occurred during loading: .github/workflows/pre-merge-ci.yaml: merge error

This adds --ignore to ignore all hidden directories.

Would opa test ./policy be a solution? There is some rego in ./checks though..

Either way, lgtm.


[WIP]Avoid knife ssh freeze on windows

Signed-off-by: dheerajd-msys<EMAIL_ADDRESS>
Description

The knife ssh subcommand “freezes” after printing the name of a remote Windows 2016 host.
Adding --ssh-pty flag will ensure to use a check for PTY enabled/disabled host.

Related Issue

Fixes: https://github.com/chef/chef/issues/9447
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[x] I have read the CONTRIBUTING document.
[ ] I have run the pre-merge tests locally and they pass.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.
[x] All commits have been signed-off for the Developer Certificate of Origin.

Test case addition is remaining.

Testing done on Ubuntu 18.04
a) With requiretty

=======================
sudo cat /etc/sudoers
#
# This file MUST be edited with the 'visudo' command as root.
#
# Please consider adding local content in /etc/sudoers.d/ instead of
# directly modifying this file.
#
# See the man page for details on how to write a sudoers file.
#
Defaults        requiretty

=======================

bundle exec knife ssh 'public_ipv4:<IP_ADDRESS>' 'ls /etc/chef' -x ubuntu -i ~/Downloads/ddubey-msys-chef.pem -C 1 -c /home/dheerajdubey/Downloads/chef-repo/.chef/knife.rb

ec2-3-128-179-75.us-east-2.compute.amazonaws.com accepted_licenses
ec2-3-128-179-75.us-east-2.compute.amazonaws.com chef_guid
ec2-3-128-179-75.us-east-2.compute.amazonaws.com client.pem
ec2-3-128-179-75.us-east-2.compute.amazonaws.com client.rb
ec2-3-128-179-75.us-east-2.compute.amazonaws.com first-boot.json
dheerajdubey@PUN-LAP-DHEERAJ:~/Chef_Repo/chef$

a) Without requiretty
=============================
sudo cat /etc/sudoers
#
# This file MUST be edited with the 'visudo' command as root.
#
# Please consider adding local content in /etc/sudoers.d/ instead of
# directly modifying this file.
#
# See the man page for details on how to write a sudoers file.


ci: Add actionlint job.

Fixes #1916.

mmm, I see it's failing on https://github.com/inspektor-gadget/inspektor-gadget/actions/runs/5808843413/job/15746472851?pr=1947, so perhaps we need to fix those issues before merging this, right?


Update dependencies

Codecov Report

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

@@            Coverage Diff            @@
##             master      #19   +/-   ##
=========================================
  Coverage          ?   89.58%
=========================================
  Files             ?        4
  Lines             ?      240
  Branches          ?        0
=========================================
  Hits              ?      215
  Misses            ?       25
  Partials          ?        0

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 d56c986...44accf6. Read the comment docs.


Is it not possible to setup new device without accessing other devices?

I signed up for keybase using my home computer a few days ago and added Github handle to the keybase account.  After that I did not provision any other device to my keybase account. Last night I wiped the Home Computer OS drive and did a fresh install of the operating system. I realized later that I had lost access to the only device that was linked to my keybase account.
On launching the keybase app on this fresh install of OS, I was prompted to enter my keybase username. I remember my keybase username and entered it successfully. Next I was asked to choose a device to connect with, showing me the Home Computer I originally linked with my keybase account.  This is where I get stuck because this Home Computer does not exist now. I have attached a screenshot of this window.

Please advise what I should do to link my fresh installation to keybase account.

login via web UI and revoke all keys.
 Would revoking keys mean I'd lose access to that team-id as well?

I don't know.
But it doesn't matter, since you don't have the key and can't recover the key, so even if you decided "I don't want to revoke the key because it would leave me locked out of my team" you still don't have the key, so you can't access your team now, anyways.

When I signed up I was given a passphrase, I have that with me. That doesn't help?

nope

I have to say, docs are really bad. I am very confused, this is very unlike other identity services. There is always some way to gain access to one's account.

I have to say, docs are really bad.

I agree

I am very confused, this is very unlike other identity services.

That is true. The goal is to do something no other identity service does. Keybase is completely trustless.

There is always some way to gain access to one's account.

Which means that someone is always holding a master key... Keybase does not require trust because they have no master key.

Keybase is completely trustless.

sounds heartless too.  I appreciate the sincerity of developers to keep Keybase incorruptible but I'd appreciate them even more if there was some warning on the homepage for all the ways a user could lose access to their account.


Adding Vanadium to 1.20.1-2 for Custom Color support

As Colormatic doesn't receive any new updates, i think it would be great to add Vanadium as a replacement for those versions, as it just got its first stable release.
Here's the link to the mod:
https://modrinth.com/mod/vanadium

Polytone has been added, so this issue is no longer relevant


Snippet not working

Prerequisites

[X ] Put an X between the brackets on this line if you have done all of the following:

Reproduced the problem in Safe Mode: http://flight-manual.atom.io/hacking-atom/sections/debugging/#using-safe-mode
Followed all applicable steps in the debugging guide: http://flight-manual.atom.io/hacking-atom/sections/debugging/
Checked the FAQs on the message board for common solutions: https://discuss.atom.io/c/faq
Checked that your issue isn't already filed: https://github.com/issues?utf8=✓&q=is%3Aissue+user%3Aatom
Checked that there is not already an Atom package that provides the described functionality: https://atom.io/packages

Description
I try to write a snippet for a TeX file.When I start to write$in_R the following snippet is not working:
'.text.tex.latex .string.other.math.tex':
    'in_R':
      'prefix': 'in_R'
      'body': "\\\\in \\\\mathbb{R}"

On the other hand, if I am not in math-mode in LaTeX everything works fine with this code:
'.text.tex.latex':
    'in_R':
      'prefix': 'in_R'
      'body': "\\\\in \\\\mathbb{R}"

Versions
You can get this information from copy and pasting the output of atom --version and apm --version from the command line. Also, please include the OS and what version of the OS you're running.
Atom    : 1.12.9
Electron: 1.3.13
Chrome  : 52.0.2743.82
Node    : 6.5.0
OS is Ubuntu 16.04 LTS
Additional Information
Any additional information, configuration or data that might be necessary to reproduce the issue.

Thanks for contributing!
This doesn't appear to be a bug but more of a question about how to get a particular snippet to work in a specific way. Unfortunately, I don't use TeX or the language-latex package (I assume that's the package you're referring to here), so I have no way of reproducing this problem. Because we treat our issues list as the Atom team's backlog, we close issues that are questions since they don't represent a task needing to be completed. For most questions about Atom there are a lot of options, check out:

The Atom Flight Manual
The Atom FAQ on Discuss
Discuss, the official Atom and Electron message board
Join the Atom Slack team

On Discuss and in the Atom Slack team, there are a bunch of helpful community members (including ones that use TeX regularly) that should be willing to point you in the right direction.


Enter beta mode

Is it worth trying to push this to beta rather than master? I'm not sure how gracefully we can exit pre mode.

I can probably just run the snapshot on this branch?


v1.3.1 tubetool patch

v1.3.1 release patch. Released to ContentDB.
Consider changing dependencies instead, it might be best to have pipeworks as hard dependency for tubetool.
If that is possible then replace this patch with mod.conf update for new versions / 2.x series.

Would like to add support for tubelib, that wont play well with having pipeworks as hard dependency so forget about hard dependencies. Basically whole mod should have only optional dependencies, tools that are not usable / needed can be disabled anyway.


Removed double suggestions when using vscode workspaces plugin

Summary of the Pull Request
When using VSCode Workspaces plugins, I always stumble upon double the amount of results of tips when searching. Made a fix to prevent this from happening

@microsoft-github-policy-service agree

@coldfeudal
Can you please

creat an issue where we can track the issue and reference it in the PR description. (If one already exists please reference that.)
describe what the cause of the duplicated paths is. I am wondering why a workspace can exist multiple times? 🤔
Add screenshot how it looked before and after. I saw that you changed something on the title and description. (As I don't have any workspaces it is hard to review without screenshots.)
Describe what changes you implement for title and description and why.

Ping @coldfeudal

@coldfeudal


Formatting typo in README.md

Looks like:
pip-faster [![Build
Status](https://travis-ci.org/Yelp/pip-faster.svg?branch=master)](https://travis-ci.org/Yelp/pip-faster) Coverage
Status

to me.  Looks like that's supposed to be a hyperlink.

#92


Feedback: Great work with this!

Tried out this instead of the code-push lib that I could not get to restart after applying an update.  Worked at once for both platforms!
Looking forward to future updates.
One thing to put on your radar, don't know how feasible it is - but since there is no manifest file logic going on here I just skipped the assets part to keep my .zip size down for smoother updates.  But that came with the downside that the assets was not available over doing an OTA.   Is there some way that it can just build up if there are new ones?  Essentially a flow something like

Copy original assets + bundle available from the first native release to somewhere you have control over
When you get an OTA - just replace or expand the assets in the same folder (so it will grow if needed)
This way  you have a native release - and then just NEW assets (if any) with OTA updates.  If that makes sense :)

Thanks again!

Thank for your feedback, about the update flow of your ideal, its very good way to reduce the bundle size when we release OTA, I will think about it, but it not easy to manage the assets, I think need a mapping file for assets and control from native size and even control when export the bundle, btw still need to detect what we need to export from CLI, then control it from native size, seem a bit related to backend size as well to know how is the previous  assets version to decide how we export the data of bundle assets 👯

Yes agreed that this job is usually made at the backend side - to make sure only new assets are in the .zip, but would be nice to have a way to rule out the backend completely and only have static .zip's if possible.
Yeah guess the logic is something like (client code)

At first startup copy (or move) all assets and current bundle used to somewhere you have control over
When a new OTA i applied, just overwrite that path with everything included in the OTA zip

make a rollback option for the js-part, so you have a change to just reset the js-file

Downside is that this will grow and grow, but if you as a developer knows about the flow you can just release native-updates from time to time to get rid of now not used assets anymore.

let me think about it later on, but now I am thinking about git version control on this library, we can pull the change of commit instead of download whole bundle file with assets, maybe just around 2-50 KB if less changing, it will save more time when do OTA, I am trying to integrate git engine into this library, and we can use git repository like GitHub / bitbucket... to store the bundle, no need backend anymore

let me think about it later on, but now I am thinking about git version control on this library, we can pull the change of commit instead of download whole bundle file with assets, maybe just around 2-50 KB if less changing, it will save more time when do OTA, I am trying to integrate git engine into this library, and we can use git repository like GitHub / bitbucket... to store the bundle, no need backend anymore

Sounds great! Looking forward to this in the future!

@vantuan88291 I just wanted to give a big round of applause for this! I was working on it and had already completed the iOS part when I stumbled upon your repo. You’re a lifesaver!

let me think about it later on, but now I am thinking about git version control on this library, we can pull the change of commit instead of download whole bundle file with assets, maybe just around 2-50 KB if less changing, it will save more time when do OTA, I am trying to integrate git engine into this library, and we can use git repository like GitHub / bitbucket... to store the bundle, no need backend anymore

Sounds great! Looking forward to this in the future!

Just finished that, please feel free to review this PR: https://github.com/vantuan88291/react-native-ota-hot-update/pull/12


Elapsed time display is incorrect with verbose option

When building with "ninja -v" the elapsed time displayed by %es is often (always?) incorrect. It appears that the start time is not initialized in this case. Note the different times for running the same build steps with and without -v:
C:\src\chromium4\src>ninja -v -C out\gn_release_official chrome.exe
ninja: Entering directory `out\gn_release_official'
[0 processes, 1/3 @ 0.0/s : 955990.631s ] C:/src/depot_tools/python276_bin/python.exe gyp-win-tool link-wrapper environment.x64 False link.exe /nologo /OUT:initialexe/chrome.exe /PDB:initialexe/chrome.exe.pdb @initialexe/chrome.exe.rsp
[0 processes, 2/3 @ 0.0/s : 955990.685s ] C:/src/depot_tools/python276_bin/python.exe gyp-win-tool stamp obj/chrome/reorder_imports.inputdeps.stamp
[0 processes, 3/3 @ 0.0/s : 955990.834s ] C:/src/depot_tools/python276_bin/python.exe ../../build/win/reorder-imports.py -i initialexe -o ../gn_release_official -a x64

ninja -C out\gn_release_official chrome.exe
ninja: Entering directory `out\gn_release_official'
[0 processes, 3/3 @ 0.0/s : 60.164s ] ACTION //chrome:reorder_imports(//build/toolchain/win:x64)

set ninja
NINJA_STATUS=[%r processes, %f/%t @ %o/s : %es ]

ninja --version
1.7.1

Looks like maybe Stopwatch needs to have Restart() called to start it
https://github.com/ninja-build/ninja/blob/b231274de33a6ebaa5ed284eb233fa360d84ad8f/src/metrics.h#L76
and RateInfo's ctor doesn't call it
https://github.com/ninja-build/ninja/blob/5739c144354c826b57042ad0968680c272a95856/src/build.h#L245
and .Restart() is only called when the build completes.  So it's not clear
to me when this was supposed to be started.
On Tue, Jul 12, 2016 at 1:49 PM, Bruce Dawson<EMAIL_ADDRESS>wrote:

When building with "ninja -v" the elapsed time displayed by %es is often
(always?) incorrect. It appears that the start time is not initialized in
this case. Note the different times for running the same build steps with
and without -v:
C:\src\chromium4\src>ninja -v -C out\gn_release_official chrome.exe
ninja: Entering directory `out\gn_release_official'
[0 processes, 1/3 @ 0.0/s : 955990.631s ]
C:/src/depot_tools/python276_bin/python.exe gyp-win-tool link-wrapper
environment.x64 False link.exe /nologo /OUT:initialexe/chrome.exe
/PDB:initialexe/chrome.exe.pdb @initialexe/chrome.exe.rsp
[0 processes, 2/3 @ 0.0/s : 955990.685s ]
C:/src/depot_tools/python276_bin/python.exe gyp-win-tool stamp
obj/chrome/reorder_imports.inputdeps.stamp
[0 processes, 3/3 @ 0.0/s : 955990.834s ]
C:/src/depot_tools/python276_bin/python.exe
../../build/win/reorder-imports.py -i initialexe -o ../gn_release_official
-a x64
ninja -C out\gn_release_official chrome.exe
ninja: Entering directory `out\gn_release_official'
[0 processes, 3/3 @ 0.0/s : 60.164s ] ACTION
//chrome:reorder_imports(//build/toolchain/win:x64)
set ninja
NINJA_STATUS=[%r processes, %f/%t @ %o/s : %es ]
ninja --version
1.7.1
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/ninja-build/ninja/issues/1172, or mute the thread
https://github.com/notifications/unsubscribe/AAAPB5dMhPDKO7eCFclIts4ZUZb__TSoks5qU_3xgaJpZM4JKz7S
.

BTW, I first noticed/mentioned this a few months (???) ago when there was a release to Chrome of a significant new version of ninja - I can't remember what version it was.

dupe of #1146. fixed on trunk, still need to make a 1.7.2 :-/


Extending the documentation

Hello,
I wanted to support your project and thought of extending your documentation!
I first started to progress on the operators subpage.
I then thought about creating an extra subpage where smalldragon could present and explain some examples (blog-like), and I want to ask you if this would be okay for you. I would also write these explanations for chosen examples!

I would like to work on your website in the future :)
I require feedback for the changes and want to know if this kind of help is required

@MarcoSteinke work on the website is much appreciated :)
I would like to keep it static handwritten html/css + bootstrap css (without any js)
and without much copy-pasting of headers/footers.
Recently, the "chapter" numbers have been added to allow for reading the docs sequentially, like you
would read a book.
So each concept should be introduced on its own and without requiring anything from the previous chapters.


Replace function relative to actual test

Hey I was trying to use this package to rewrite some paths used in while running tests, but unfortunately the replaceFunc is determined from the file babel is processing instead of from where the node process originated.
A fix would be to use replace process.cwd() with process.env.PWD in https://github.com/liady/babel-plugin-module-rewrite/blob/master/src/index.js#L4
Would that make sense or would that be breaking? If so maybe make it configurable which to use?
Thanks

@aertmann thank you for your feedback.
I published a new version that supports 'resolveFrom' parameter, that is later going to be evaled.
It defaults to process.cwd().
For example:
{
  "plugins": [
    ["module-rewrite", {
        "replaceFunc": "./utils/replace-module-paths.js",
        "resolveFrom": "process.env.PWD" // this is going to go through eval(), defaults to "process.cwd()"
    }]
  ]
}

Let me know if this works for you.

hey @liady thank you so much, really appreciate it! super quick response and works great 😃


switch to using semver to compare matching versions

@lukebatchelor if you could have a look at this, that would be great.

@Noviny I didnt want to push to your branch, but I've finished these changes in #55 if you wanna take a look.

Closing as resolved in #55


AWS Secrets Manager: ValidationException: Invalid namedev and prod deployments differences.

Hello everyone 🙂 I am trying to do npx sst build --stage prod or npx sst deploy --stage prod and getting
Collecting page data  ...Error retrieving secret from AWS Secrets Manager: ValidationException: Invalid name. Must be a valid name containing alphanumeric characters, or any of the following: -/_+=.@!
Error retrieving secret from AWS Secrets Manager: ValidationException: Invalid name. Must be a valid name containing alphanumeric characters, or any of the following: -/_+=.@!
ValidationException: Invalid name. Must be a valid name containing alphanumeric characters, or any of the following: -/_+=.@!
 at X (/Users/belarusrulez/IdeaProjects/gray-area-us/packages/web/.next/server/chunks/3287.js:10:24415)
 at /Users/belarusrulez/IdeaProjects/gray-area-us/packages/web/.next/server/chunks/3287.js:10:24546
 at v (/Users/belarusrulez/IdeaProjects/gray-area-us/packages/web/.next/server/chunks/3287.js:1:127979)
 at process.processTicksAndRejections (node:internal/process/task_queues:95:5)
 at async /Users/belarusrulez/IdeaProjects/gray-area-us/packages/web/.next/server/chunks/3287.js:1:268781
 at async /Users/belarusrulez/IdeaProjects/gray-area-us/packages/web/.next/server/chunks/3287.js:1:252751
 at async /Users/belarusrulez/IdeaProjects/gray-area-us/packages/web/.next/server/chunks/3287.js:1:267524
 at async /Users/belarusrulez/IdeaProjects/gray-area-us/packages/web/.next/server/chunks/3287.js:1:217309
 at async c (/Users/belarusrulez/IdeaProjects/gray-area-us/packages/web/.next/server/chunks/9595.js:7:8832)
 at async p (/Users/belarusrulez/IdeaProjects/gray-area-us/packages/web/.next/server/chunks/9595.js:7:9055) {
'$fault': 'client',
'$metadata': {
 httpStatusCode: 400,
 requestId: '30f185da-bb30-4f9b-9826-9f06e2aa7330',
 extendedRequestId: undefined,
 cfId: undefined,
 attempts: 1,
 totalRetryDelay: 0
},
__type: 'ValidationException'
}

in dev everything works
I tried to deploy infrastructure first and set all secrets using
npx sst secrets load.
still getting the same issue
If I do
npx sst dev --stage dev
npx sst deploy --stage dev
I don't get the error and it says it's deployed successfully.

We are transition this repo to https://github.com/sst/v2 and are closing this issue for now. If it is still relevant please reopen it there - sorry for the trouble


4.0 Alpha 5 regression: AnimationTree does not respect AnimationPlayer values. It resets properties values to default.

Godot version
4.0 Alpha 5
System information
Windows 10
Issue description
When using an Animation Tree  it changes properties to default values after an animation is ended. As an example, in the minimal project attached:

If we use the AnimationPlayer directly, it works as expected (transparency to 0) (first video).
If we use the AnimationTree, it resets ColorRect to a default white (1,1,1,1) color value (second video).

This occurs for any property on any node. The problem is new, Alpha 4 did not preset it. Imported animations from Blender seems not to be affected.
https://user-images.githubusercontent.com/8985409/159957515-f3aa843f-b6b6-4c6c-a5cf-8dc2b060bbe5.mp4
https://user-images.githubusercontent.com/8985409/159957539-43f6a082-1c78-4fb2-894f-587d72a52659.mp4
Steps to reproduce
In the attached minimal project, enable or disable "Use animation tree" option and run the project to see the behavior as in the videos.
Minimal reproduction project
BugReport02_4alpha5.zip

cc @TokageItLab

just a question, the animationplayer in question was made local?

just a question, the animationplayer in question was made local?

Yes, animations were made inside Godot. Those are which have the problem.

It is a simple animatio

just a question, the animationplayer in question was made local?

Yes, animations were made inside Godot. Those are which have the problem.

It is a simple Animation Player node in main scene which changes the color of a child node, the color rectangle.

I think this is a bug caused by Variant::zero() actually using an initial value instead of zero: we want Color(0,0,0,0) but it return Color(1,1,1,1), maybe we should create separate functions like Variant::zero() and Variant::init().

At least this seems to be a special case bug for the alpha value of Color. I have sent a fix so that Variant::zero() explicitly returns Color(0,0,0,0) for blending as #59490.

This is not exclusive of color. Animated collision shapes also go back to its default (0,0,0) position in a similar behavior.

For now, this seems to be a special case bug for the alpha value of Color. I have sent a fix so that Variant::zero() explicitly returns Color(0,0,0,0) for blending as #59490.

@jcarlosrc Can you test #59490?

Note: It may be that it should be mentioned in the documentation, but if one of the two animations to be blended is missing a track, it will be blended with the zero value.

If a track is missing in the other animation it should not be blended. This is the behavior of all versions previous to 4.0 alpha 5. Otherwise, we will need to insert a track for every property ever keyframed in any animation to avoid a bad mixing. This does not make sense.

It creates an incongruity that the blended result will not always be the same, and we decided in discussions with reduz that it is an edge case.


Add pyproject.toml for Custom Node Registry

We are building a global registry for custom nodes (similar to PyPI). The registry makes using custom nodes more reliable by reserving globally unique names for each custom node, and supporting semantic versioning.
Here’s some more information on the registry.
Action Required:

[ ] Go to the registry. Login and create a publisher id. Comment it here and we will update the PR (or just merge it and change it yourself).
[ ] Write a short the description.
[ ] Merge the separate Github Actions PR and run the workflow.
If you want to publish the node manually, install the cli and run comfy node publish

Please message me on Discord if you have any questions!

hey @aegis72, wanted to send a small nudge and ask if you had any questions!

@aegis72, wondering if there's any way we can support you on this PR!


refactor(main): delete node for check

🤖 Generated by Copilot at e6f3a96
Summary
:wastebasket::truck::hammer_and_wrench:

This pull request removes the PreProcess functions from the delete and scale processor pipelines, as they are redundant after refactoring the DeleteNodes and ScaleNodes functions in pkg/apply/delete.go and pkg/apply/scale.go. This simplifies the processor pipelines and avoids unnecessary checks and actions.

PreProcess gone
Logic moved to other funcs
Pipeline simplified

Walkthrough

Remove PreProcess functions from delete and scale processor pipelines (link, link)

/cherry-pick release-v4.2


[Feature Request] RSP and CPC commands

Overview
Add commands to tell users more about Ravi's Study Program (RSP) and Competitive Programming Club (CPC).
Details

/RSP - Returns info about RSP
/CPC - Returns info about CPC

Why would this feature be useful?
Would let users easily know more about RSP and CPC.
Additional Notes
No response

Completed in #19.


Fix cards grid custom fitted displays

Should the container query exist on the Card Container?
Before

After

Resizing shud work

I think this makes sense. the fitted view should exist within a block element that fills the entire space allotted to it, 100% width and height will do that correctly. then the fitted view will measure the space yielded to it ans size accordingly.


Update twelve-days to match specification

Closes #858.

Awesome, thanks!
Do you want to also have a go at regenerating the tests?

@Insti Done. Incidentally before I saw that you updated your comment I tried regenerating the tests and it looks like several of them changed. I could put that into a new pull request if you like.

I could put that into a new pull request if you like.

That would be great thanks!

Thanks @PatrickMcSweeny :heart:


Confusing wording

Challenge Waypoint: Target the same element with multiple jQuery Selectors has an issue.

Add the shake class to all the buttons with class .btn.

implied I should try to use jQuery to select elements with type button AND which have class .btn. Of course, only buttons have class .btn. "button"'s redundancy created confusion, suggest change to "elements".

@willnwhite I don't think it's confusing at all. It clearly says: "Add the shake class to all the buttons with class .btn". It says nothing about targeting elements with the "type" button.

@AkiraLaine is right. It clearly states to target .btn class not all the button elements.
Happy Coding


net: ipv6: Do not pass ICMPv6 packets to net_conn_input

ICMPv6 network packets are processed and consumed (if not dropped) within net_icmpv6_input function. Therefore they shoud not be passed to the net_conn_input function.
This fixes the assertion I've mentioned in #13301. If ICMPv6 packet was not intended for drop, it was processed after net_icmpv6_input, even though it was already freed in the respective ICMPv6 handler. This could lead to a system crash (or assert if enabled) when packet was reallocated in a meantime.
And another side effect is that packet was marked for drop in net_conn_input, which led to double-release of this packet (Zephyr seems to be protected against it though, hence it was not the source of crash):
[00:00:24.143,933] <err> net_conn: No suitable protocol handler configured
[00:00:24.145,424] <err> net_pkt: *** ERROR *** pkt 0x0041f80c is freed already by handle_na_input():1689 (processing_data():151)

Signed-off-by: Robert Lubos<EMAIL_ADDRESS>

Codecov Report

Merging #13568 into master will increase coverage by <.01%.
The diff coverage is 100%.

@@            Coverage Diff             @@
##           master   #13568      +/-   ##
==========================================
+ Coverage   52.45%   52.45%   +<.01%
==========================================
  Files         322      322
  Lines       46592    46592
  Branches    10768    10768
==========================================
+ Hits        24441    24442       +1
- Misses      17246    17247       +1
+ Partials     4905     4903       -2

Impacted Files
Coverage Δ

subsys/net/ip/ipv6.c
58.72% <100%> (+0.85%)
:arrow_up:

subsys/net/ip/connection.c
72.98% <0%> (-0.28%)
:arrow_down:

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 2072fbd...8731feb. Read the comment docs.


CDI events

Demo the possibility to receive UI init event and/or session init events.

More info here: https://quarkus.io/guides/lifecycle and https://quarkus.io/guides/cdi-reference
Implement a demo bean which @Observes various events
Document the approach
Verify what kind of events the Quarkus Extension CDI code is firing.

Blocked by #6

The CDI extension fires the following events:

UIInitEvent
SessionInitEvent
SessionDestroyEvent
ServiceDestroyEvent
AfterNavigationEvent
BeforeEnterEvent
BeforeLeaveEvent
PollEvent

I'll create an application-scoped bean which will keep log of these events, and a route which will then show the events in a grid.

The CDI extension fires the following events:

UIInitEvent
SessionInitEvent
SessionDestroyEvent
ServiceDestroyEvent
AfterNavigationEvent
BeforeEnterEvent
BeforeLeaveEvent
PollEvent

I'll create an application-scoped bean which will keep log of these events, and a route which will then show the events in a grid.


issue #5216 "infinite loop from returning function in groovy"

this is revert for issue #5147 "lazySeq"

just disabling tests that fail is not correct.

This testd was added with change, which I deleted. I can delete this rests
too
20 апр. 2017 г. 18:09 пользователь "Scott Draves"<EMAIL_ADDRESS>написал:

just disabling tests that fail is not correct.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/twosigma/beakerx/pull/5226#issuecomment-295771757,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AKH3afehdSnVOAon5XY56Ujyp4ViL0KNks5rx3UxgaJpZM4NDAvw
.


Unexpected behavior from IScheduler.Schedule for iterator overload

      return Observable.Create<byte[]>(o =>
      {
          source.Seek(offset, SeekOrigin.Begin);
          var buffer = new byte[bufferSize];
          var onGoingReadTask = new SerialDisposable();
          var scheduledTask = new SerialDisposable();
          Action<Action> iterator = (self) =>
              onGoingReadTask.Disposable = >Observable.FromAsync((CancellationToken token) => source.ReadAsync(buffer, 0, bufferSize, token))
                        .Subscribe(count =>
                        {
                            Console.WriteLine("from inside:" + count);
                            if (count > 0)
                            {
                                o.OnNext(buffer.Take(count).ToArray());
                              self();
                            }
                            else
                                o.OnCompleted();
                        }, o.OnError);
          scheduledTask.Disposable = Scheduler.CurrentThread.Schedule(iterator);
          return new CompositeDisposable(scheduledTask, onGoingReadTask);
      });

consider the observable sequence. if a subscriber dispose the subscription, it also disposes the disposable returned by scheduler. but the iterators still keeps running. which is not expected. if add the following code my problem gets resolved. but it is really unexpected behavior and i had spend few hours on a bug because of this!

                                if (scheduledTask.IsDisposed == false)
                                    self();

Are you able to isolate this in a unit test so I can reproduce this on my end?

Can't reproduce with preview 4 and the following setup:
var source = Observable.Create<int>(observer =>
{
    var count = 0;
    var onGoingReadTask = new SerialDisposable();
    var scheduledTask = new SerialDisposable();
    Action<Action> iterator = (self) =>
        onGoingReadTask.Disposable = Observable.FromAsync(
            token => Task.Delay(100, token).ContinueWith(t => ++count, token)
        ).Subscribe(value => {
            Console.WriteLine("from inside:" + count);

            self();
        })
        ;

    scheduledTask.Disposable = Scheduler.CurrentThread.Schedule(iterator);

    return new CompositeDisposable(onGoingReadTask, scheduledTask);
});

Console.WriteLine(">> Starting");

var d = source.Subscribe();

Thread.Sleep(3000);

d.Dispose();

Console.WriteLine(">> Disposed, waiting a bit more");
Console.WriteLine(">> should not print many more 'from inside:'");

Thread.Sleep(3000);

Note though that Task.Delay.ContinueWith is not an ReadAsync but I'd guess ReadAsync is tested well enough so that it responds to the cancellation token properly.

@shovonnn Can you retry/confirm ?

Closing as resolved with 4.x for some time now.


https://wallet.lum.network/ returns TX hash w/ lowercase letters

If you complete a transaction using the lum wallet the resulting tx hash has lowercase letters:

Searching the explorer for the tx like this doesn't find a result:
https://explorer.lum.network/search/ccb59630a4311eae6ae8e656ff65d398fa8d2fd2b692b670c2b0510a0b89c75c
I can go to Mintscan and find that tx:
https://www.mintscan.io/lum/txs/ccb59630a4311eae6ae8e656ff65d398fa8d2fd2b692b670c2b0510a0b89c75c
If I take their capitalized version of the tx and search the explorer again, I can find the tx:
https://explorer.lum.network/txs/CCB59630A4311EAE6AE8E656FF65D398FA8D2FD2B692B670C2B0510A0B89C75C

Thanks for the feedback, team has been notified, closing the issue since it is not in the appropriate repository