Bug: Cannot read property 'date' of undefined

OS: Windows
Node Ver: 10.x
Console outputs:
C:\Users\Luke Shiels\gh-contributions>npm start

><EMAIL_ADDRESS>start C:\Users\Luke Shiels\gh-contributions
> node server.js

Listening on 9000
/css/gh-style.css - Not Modified
/js/jquery.min.js - Not Modified
/bootstrap/js/bootstrap.min.js - Not Modified
/js/index.js - Not Modified
/bootstrap/css/bootstrap.min.css - Not Modified
/css/github.css - Not Modified

C:\Users\Luke Shiels\gh-contributions\contributions.js:60
            options.dates.push(year[p.x - 1][p.y - 1].date);
                                                     ^
TypeError: Cannot read property 'date' of undefined
    at Object.module.exports.getRepo (C:\Users\Luke Shiels\gh-contributions\co
ributions.js:60:54)
    at C:\Users\Luke Shiels\gh-contributions\server.js:53:27
    at IncomingMessage.<anonymous> (C:\Users\Luke Shiels\gh-contributions\serv
.js:107:13)
    at IncomingMessage.EventEmitter.emit (events.js:92:17)
    at _stream_readable.js:920:16
    at process._tickCallback (node.js:415:13)

npm ERR<EMAIL_ADDRESS>start: `node server.js`
npm ERR! Exit status 8
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>start script.
npm ERR! This is most likely a problem with the gh-contributions package,
npm ERR! not with npm itself.
npm ERR! Tell the author that this fails on your system:
npm ERR!     node server.js
npm ERR! You can get their info via:
npm ERR!     npm owner ls gh-contributions
npm ERR! There is likely additional logging output above.
npm ERR! System Windows_NT 6.1.7600
npm ERR! command "C:\\Program Files\\nodejs\\\\node.exe" "C:\\Program Files\\n
ejs\\node_modules\\npm\\bin\\npm-cli.js" "start"
npm ERR! cwd C:\Users\Luke Shiels\gh-contributions
npm ERR! node -v v0.10.26
npm ERR! npm -v 1.4.3
npm ERR! code ELIFECYCLE
npm ERR!
npm ERR! Additional logging details can be found in:
npm ERR!     C:\Users\Luke Shiels\gh-contributions\npm-debug.log
npm ERR! not ok code 0

C:\Users\Luke Shiels\gh-contributions>

The client on the browser just says: Error: undefined.

@IonicaBizau, I can start working for Windows support if you assign this issue to me.

@silviubogan This should be working on Windows.


Make some buttons in editor use the lighter version of the TabMenu icon

Consistency with the rest of the editor.

Thanks!


changesetがない場合にdeployがエラーになる問題に対応

Codecov Report

Merging #20 (73281c6) into master (15964b5) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #20   +/-   ##
=======================================
  Coverage   98.28%   98.28%
=======================================
  Files           7        7
  Lines         175      175
  Branches       23       23
=======================================
  Hits          172      172
  Partials        3        3

Continue to review full report at Codecov.

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


Some Product advice

One suggestion, when withdrawing, I would like to be able to customize the withdrawal amount instead of the default address balance. A major reason for this is that I am concerned that someone will presume my associated address based on the amount transferred, so I will need to adjust the amount when I have to withdraw money.
The second suggestion is that for transfers to other addresses, is there a history provided? I think this is important for the user who wants to know which addresses I have made transfers to.

Your second suggestion is covered by #263 , this is in our backlog.
Your first suggestion is not really possible because of how Umbra works, but we appreciate your input!


Question regarding MultiProducerSequencer

I use the LMAX disruptor 3.3.2 as part of Log4j2 async logging, lately my application started to behave strangely, logging stopped at some point and the application got unusable. After some test runs and investigation taking a flight recording, I found out that we eventually hit the following code block in the MultiProducerSequencer, which leads to the case that threads are getting parked.
MultiProducerSequencer::next(1) has the following loop. Could someone of you explain under what circumstances I hit this loop and cannot leave it? Thank you very much!
do
{
            current = cursor.get();
            next = current + n;

            long wrapPoint = next - bufferSize;
            long cachedGatingSequence = gatingSequenceCache.get();

            if (wrapPoint > cachedGatingSequence || cachedGatingSequence > current)
            {
                long gatingSequence = Util.getMinimumSequence(gatingSequences, current);

                if (wrapPoint > gatingSequence)
                {
                    LockSupport.parkNanos(1); // TODO, should we spin based on the wait strategy?
                    continue;
                }

                gatingSequenceCache.set(gatingSequence);
            }
            else if (cursor.compareAndSet(current, next))
            {
                break;
            }
}
while (true);

If the consumer has stopped or become slow and the buffer fills up.  If there is no space available in the buffer then Sequencer.next() (both single and multi) will block until space becomes available.  We use the Disruptor in our own logger implementation.  We instead use Sequencer.tryNext() which will return immediately if there is no space available (throws an InsufficientCapacityException in that case).  We write a notification to the console (System.err) if we are unable to put log messages into the Disruptor.
We prefer to lose log messages than stall the application.


Fix dump and restore respository

As title.

Codecov Report

Merging #16698 (9e9cf28) into main (d17f555) will increase coverage by 0.00%.
The diff coverage is 0.00%.

@@           Coverage Diff           @@
##             main   #16698   +/-   ##
=======================================
  Coverage   45.37%   45.38%
=======================================
  Files         758      758
  Lines       85306    85311    +5
=======================================
+ Hits        38711    38718    +7
  Misses      40313    40313
+ Partials     6282     6280    -2

Impacted Files
Coverage Δ

modules/migrations/dump.go
0.00% <0.00%> (ø)

modules/private/restore_repo.go
0.00% <0.00%> (ø)

modules/notification/mail/mail.go
35.29% <0.00%> (-2.95%)
:arrow_down:

models/unit.go
41.09% <0.00%> (-2.74%)
:arrow_down:

modules/notification/ui/ui.go
60.86% <0.00%> (-1.45%)
:arrow_down:

modules/queue/unique_queue_disk_channel.go
47.26% <0.00%> (-1.37%)
:arrow_down:

models/notification.go
65.13% <0.00%> (-0.88%)
:arrow_down:

routers/web/repo/view.go
40.95% <0.00%> (-0.58%)
:arrow_down:

services/pull/pull.go
41.78% <0.00%> (-0.41%)
:arrow_down:

models/issue_comment.go
52.05% <0.00%> (+0.29%)
:arrow_up:

... and 9 more

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 f5158b4...9e9cf28. Read the comment docs.


CFP: Implement Egress IP feature

Cilium Feature Proposal
Is your proposed feature related to a problem?
As a customer i need to be able to ensure that the traffic from one or more pods in one or more namespaces has a consistent source IP address for services outside the cluster network.
I need to be able to do that for plenty of different pods or namespaces assigning specific IP each time
Egress GW don't allow to solve this issue
Describe the feature you'd like
An egress IP address is implemented as an additional IP address on the primary network interface of a node and must be in the same subnet as the primary IP address of the node. The additional IP address must not be assigned to any other node in the cluster.
It can be assigned to a namespace or a pods
similar feature are existing in other CNI like kube-ovn

Hi @tuxtof , there is significant overlap between this proposal and the Egress Gateway feature. Could you explain a bit more about how this proposal would be different and maybe why the existing feature doesn't cover that use case?

Hello @joestringer let me try to give more details
let's take the more simple example where we set egressGateway interface and we get dynamically the egress IP
some issues are:

if we use dynamic k8s node pool where node IP change during pool upgrade, egress IP will change breaking your external firewall rules
if you want to manage a big number of namespace/tenant each using their own egress IP, you need to have a lot of gateway (one for each tenant/ns)

we can also select a specific ip on the egressgateway but there is no way to set it dynamically and no failover mechanism
this is clearly the kind of problem we need to fix here with egress IP feature, or egressgateway improvement
hope this is more clear

if we use dynamic k8s node pool where node IP change during pool upgrade, egress IP will change breaking your external firewall rules

That is not a good use case for egress gateway; if you rely on static IPs, you will need a way to provide static IPs to the egress gateways. A core set of non-dynamic nodes, with ENIs (or equivalent) would be a better architecture.

if you want to manage a big number of namespace/tenant each using their own egress IP, you need to have a lot of gateway (one for each tenant/ns)

AIUI Egress Gateway is not namespace-specific the policies are non-namespaced. So you are free to mix and match namespaces.
What sort of failover are you envisioning?


Add Pidgin translation of English-language blog welcoming Seun

Same blog as #850, but in Pidgin.

[x] Text translated
[ ] Old post links to this one...
[ ] This post links to old post...

@NPDebs - could you cross-link the two posts as well? e.g. "This post is also available in [English](linkie here)"

@yochannah - Ready at #863.
This was a really good request. Thanks! 🙏🏽

Fixed with #863


See #131.

@bkutil

+1


First cut at noms migrate

This iterates over all the values of the old version and creates new
values of the new version.
Closes #2428
Fixes #2272

Hold off reviewing on this. I need to add a test for the cli too.

OK. PTAL

Back to you

Do you still need me to review this?

still lgtm


protoc-gen-go: generate MarshalJSON/UnmarshalJSON methods for messages

Per https://github.com/golang/protobuf/issues/183:

There's several other parts of the proto3 JSON spec that make it impossible to use the standard encoding/json package, like WKTs.

I don't want to have generated protos grow any extra dependencies, so depending on the jsonpb package isn't appropriate.

I think json usage is pervasive enough that we should revisit this decision.  Message types whose JSON encoding diverges from the standard encoding/json package should provide MarshalJSON and UnmarshalJSON methods.  Then users could use the standard package without the need to explicitly interact with jsonpb.
Yes, it's possible that would add a few more dependencies for programs that don't strictly need them.  To the extent that that's a problem, it's a problem that should be fixed in the compiler and linker: the marginal increase in build and link times due to the extra dependencies seems like it would be miniscule compared to the complexity of interacting with two separate and incompatible packages for JSON encoding (e.g. https://github.com/golang/protobuf/issues/248).

Ok so what would the generated MarshalJSON method do.
Would it call jsonpb (which calls encoding/json again) or would it be actual generated code that marshals json faster than with reflect and gives packages like ffjson a run for their money?

@awalterschulze: This proposal is about the API, not the implementation.
Initially it would probably just call jsonpb.  If we find it's worth the extra overhead to generate code (or use some other optimization technique to avoid reflection), it would be easy enough to change the implementation while preserving the same API.

Ok so encoding/json is going to call the generated MarshalJSON method which is going to call jsonpb which sometimes also calls encoding/json.  Some psuedo circular dependency.  We hope it doesn't become a loop.  Sounds like any easy bug to make.
Maybe we can list the differences between encoding/json en jsonpb?  Even if its just for the record.
I know one reason was the fact that non string keys could not be encoded by encoding/json.  But the go1.7 release notes now state the following:
In earlier versions of Go, this package only supported encoding and decoding maps using keys with string types. Go 1.7 adds support for maps using keys with integer types: the encoding uses a quoted decimal representation as the JSON key. Go 1.7 also adds support for encoding maps using non-string keys that implement the MarshalText (see encoding.TextMarshaler) method, as well as support for decoding maps using non-string keys that implement the UnmarshalText (see encoding.TextUnmarshaler) method. These methods are ignored for keys with string types in order to preserve the encoding and decoding used in earlier versions of Go.

Which means this problem might not be a problem anymore?
What are the other differences between encoding/json and jsonpb?

Maybe we can list the differences between encoding/json en jsonpb? Even if its just for the record.

Differences I'm aware of:

Field names.  (Easy enough to fix with a proto compiler change, but it's a breaking change for current users of the json package with generated message types.)
The default behavior for json is to include default ("empty") values; the default behavior for jsonpb is to omit them.
jsonpb encodes enums by name rather than as integers.
jsonpb recognizes protobuf Well-Known Types and parses them accordingly.
Maybe some differences involving numbers encoded as strings vs. numbers encoded as JS number literals?
Maybe handling of NaNs and Infinity?

Some of these things are solvable:

Field names: maybe the new json_name feature can be used to have something consistent here?
"empty": simply generate structures with omitempty?
enums: We can generate MarshalJSON and UnmarshalJSON methods for enums, like was done in the past?
WKTs: We can write MarshalJSON and UnmarshalJSON for those types.
numbers vs strings literals: This might be a problem?
NaNs, Infinity, I also don't know about this.

So what if we start by moving the jsonpb implementation to be closer to the encoding/json implementation.
Maybe we can start by generating omitempty for proto3 and then simply copying the behaviour of encoding/json for this case?
Then we could start handling well known types in a less specific way in the jsonpb package and rather use MarshalJSON and UnmarshalJSON implementations.
Then maybe we can get to the point where the proto json marshaler is simply the encoding/json Marshaler with a few custom options?
Then our generated types don't need to generate MarshalJSON and UnmarshalJSON methods?

Then maybe we can get to the point where the proto json marshaler is simply the encoding/json Marshaler with a few custom options?

I agree that that's a nice goal in principle, and I've retitled the proposal accordingly.  I'm not sure whether it's feasible.

Yes this title is something I can get behind.

Maybe we can start by generating omitempty for proto3 and then simply copying the behaviour of encoding/json for this case?

Yes please, this would solve so many problems that are common: people deserializing with jsonpb and serializing with json

Ok, so to sum up this discussion Step 1 would be:

general MarshalJSON and UnmarshalJSON for message structs that simply delegate to jsonpb
remove the json tag on the struct to not confuse people
add a flag to the protoc-gen-go compiler that allows you fallback to the previous behaviour (no MarshalJSON UnmarshalJSON generation json tag is there)
document it in a the release notes protoc-gen-go

I like this approach, because it gets us 90% there (you can use encoding/json like normal humans), there is a fallback for prior users of json, and the jsonpb deserializes the "whole tree" (without sub message caling) so we de-risk any edge cases.
Step 2 would be  to consider code-generating the MarshalJSON/UnmarshalJSON functions that work in a tree-like fashion. I.e. if a message has a nested message, the relevant MarshalJSON/UnmarshalJSON of the nested message is called. This may be tricky with:

numbers vs strings literals: This might be a problem?
NaNs, Infinity, I also don't know about this.
But I'm sure we can have helper functions in jsonpb library as a stepping stone that handles the field values inside the codegenned methods.

Note that MarshalJSON and UnmarshalJSON would have to be opinionated on which options (fields in jsonpb.Marshaler and jsonpb.Unmarshaler) are used.  It may be fine to have this as a default, but I don't think we can remove the jsonpb package.

@zombiezen True, but I think the protobuf JSON spec and the Go encoding/json package provide pretty clear defaults.
The Marshaler options:

EnumsAsInts is not allowed by the spec: it requires "[t]he name of the enum value as specified in proto".
EmitDefaults is explicitly allowed and must default to false.
Indent is allowed (the spec is silent about whitespace), so it should default to the same formatting produced by encoding/json.
OrigName is not allowed by the spec: it requires that "names are mapped to lowerCamelCase".

The Unmarshaler options:

AllowUnknownFields is not addressed in the spec.  We should probably get some clarification on that, and/or check the published conformance-test suite.

SGTM. Just wanted to make sure we thought about it.

I just wanted to add a note: the json StructTag value of "-" isn't checked.
I found out about this in a new project, where I added json:"-" for a password field (just to be sure) - it didn't work.
So it would seem, there is currently no way to ignore a value with jsonpb.

@morphar Curious, how did you set the JSON struct tag value of "-" on a generated type field?  If this is for a dynamic message type, you can implement JSONPBMarshaler/JSONPBUnmarshaler.  See https://github.com/golang/protobuf/blob/master/jsonpb/jsonpb_test.go#L767.

any work on this? need help?

In case anyone is interested, I implemented protoc-gen-go-jsonpb to generate MarshalJSON/UnmarshalJSON functions for messages.

Would love MarshalJSON/UnmarshalJSON methods to be generated for Message types. Would make it a lot easier to deal with map/array types as well, such as the following:
message A {
  string name = 1;
  map<string, B> b_field = 2;
}

message B {
  float64 C = 1;
}

In this case, say we have a backend service that needs to deserialise only A.b_field:
func (h *Handler) storeBFields(a *mypackage.A) error {
  b := a.GetBField()

  jsonpb.Marshal(b)  // <--- Wrong: map[string]*mypackage.B does not implement proto.Message

}

By having MarshalJSON/UnmarshalJSON methods, it would allow users to use only stdlib encoding/json's Marshal/Unmarshal functions to unmarshal protobuf to it's exact JSON mapping, without using hacky workarounds.

Perhaps related, at least as far as decoding enums is concerned: https://github.com/golang/protobuf/issues/866

Whoops I missed this dup when I reported my issue (#926).
My stab is similar to @tnarg above, I didn't see this thread and googling didn't bring anything up. We've now started using this and so far has been working great: https://github.com/mitchellh/protoc-gen-go-json
It'd be great to have this sort of functionality built-in.

Is this waiting on the refactor mentioned in #266 for v2  of this pkg?
Anything members of the community can help out with to move this particular set of issues through? Very much would like to see this land in the core pkg.

Huh, this isn’t such a bad idea. Since we can override JSON marshaling/unmarshalling per message to call into jsonpb instead, we can avoid incidents of people confusing which one they are supposed to be using. And meanwhile, standard encoding/json does not have to support any of the unique issues of protobuf JSON encoding…

Thanks @mitchellh, that package works perfectly! Would love to see this built in as well.

Is there a status on this? I recently came across this issue when trying to wrap a protobuf generated struct inside another struct that contained other non protobuf generate structs. Now I can't simply use json.Unmarshal/Marshal because the protobuf related struct requires special handling related to the enum type fields. I'd be super happy to have these protobuf structs implement the MarshalJSON and UnmarshalJSON functions with some sane defaults.
Please let me know if this is ready to be worked on.

@jbgrunewald , use https://github.com/mitchellh/protoc-gen-go-json

Thanks, this looks very promising.
James Grunewald

On Dec 4, 2020, at 8:25 PM, Tamal Saha<EMAIL_ADDRESS>wrote:

@jbgrunewald , use https://github.com/mitchellh/protoc-gen-go-json
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub, or unsubscribe.

So what's going on now? It's 2024.

https://github.com/mfridman/protoc-gen-go-json did the trick very well for me. If found this through the one linked by @tamalsaha , but that turned out to be a fork of @mfridman's work and had been archived in the meantime. The original one appears to be better maintained, and was able to handle optional fields in proto messages. So worth to check out.


Problems with synchronization on Amazon server

Hi,
When I synchronized to December 15, 2015, the nodes stopped synchronizing, and there were many server denial problems in the debug. log file. I tried to restart the node service, but it began to resynchronize in March 2014. I also seem to have found exception information in the debug. log file, specifically EXCEPTION: St9bad_alloc std:: bad_alloc bitcoin in Process Messages (). What causes this? This happened two times.

A lot of servers refuse to connect now.

@dexX7


Wiki of Merge Cells Via Cell Renderers

@joneit  Says that the grouped column header is already an example that could be used. Worth confirming


Case customization possible?

Similar to ansible, would it be possible to support modifications to {{container_name}} such as {{container_name|upper}} or {{container_name|title}}?
Eg: for container named portainer:
{{container_name}} : portainer
{{container_name|title}} : Portainer
{{container_name|upper}} : PORTAINER
Thank you. I wasn't sure if this was something you were leveraging from either the compose specification or it was written into Autokuma. I am not a developer. Sorry if this is a dumb question.

This is currently not possible as I'm doing a simple search and replace, I've though about switching this out with a fully fledged template engine, but this would end up being a breaking change so I've been hesitant for now.

This is now available on master, please check the CHANGELOG.md as wells as Tera Documentation


Error validating sh:maxCount on nested objects in the live demo

Steps to reproduce
I'm using the following shape:
@prefix sh: <http://www.w3.org/ns/shacl#> .
@prefix mud: <https://raw.githubusercontent.com/Multi-User-Domain/vocab/main/mud.ttl#> .
@prefix mudchar: <https://raw.githubusercontent.com/Multi-User-Domain/vocab/main/mudchar.ttl#> .
@prefix mudcharshapes: <https://raw.githubusercontent.com/Multi-User-Domain/vocab/main/shapes/core/mudchar.ttl#> .

mudcharshapes:Character a sh:NodeShape ;
    sh:targetClass mudchar:Character ;
    sh:property mudcharshapes:SpeciesProperty .

mudcharshapes:SpeciesProperty
    sh:path mud:species ;
    sh:name "Species" ;
    sh:description "A Character is not obliged to have a species" ;
    sh:maxCount 1 ;
    sh:class mud:Species .

mudcharshapes:Species a sh:NodeShape ;
    sh:targetClass mud:Species ;
    sh:property [
        sh:name "@id" ;
        sh:description "Enter the urlid of the desired species" ;
        sh:defaultValue "https://raw.githubusercontent.com/Multi-User-Domain/vocab/main/mud.ttl#Human" ;
        sh:minCount 1 ;
        sh:maxCount 1 ;
    ] .

The live demo renders the form for Species correctly. When I select to add a species, it's rendered correctly (the sub-form is shown and the user is only able to add one urlid)
During validation I receive the following error in the console, and the graph isn't validated:

Uncaught (in promise) Error: Cannot find validator for constraint component http://www.w3.org/ns/shacl#MaxCountConstraintComponent

When I remove sh:maxCount from the mudcharshapes:Species NodeShape the error is removed and I can continue (but the user is able to add multiple urlid fields)

This seemingly is an issue with the validator I use: https://www.npmjs.com/package/rdf-validate-shacl
If you add an sh:path to your property, it shall work as expected.


TCVP-2481 Ticket Validation status filter & Add status column in Decision Validation tab

Description
This PR includes the following proposed change(s):

TCVP-2481

Type of change

[ ] Bug fix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Refactoring / Documentation
[ ] Version change

if your change is a breaking change, please add breaking change label to this PR
How Has This Been Tested?
Please describe the tests that you ran to verify your changes.
Does the change impact or break the Docker build?

[ ] Yes
[x] No

If Yes: Has Docker been updated accordingly?

[ ] Yes
[x] No

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have made corresponding changes to the documentation
[x] New and existing unit tests pass locally with my changes

@KDNeufeld I just changed inside the searchFilter


Throttle breaks the observable chain while running on wasm in .net 8

Bug

Which library version?
6.0.0 and 5.0.0 has the issue

What are the platform(s), environment(s) and related component version(s)?
.net 8, avalonia 11.0.9 using wasm and DynamicData 7.9.5 (but I tried 8.3.27 and same result)

What is the use case or problem?
If I call Throttle while this avalonia app is running under wasm in .net 8, it breaks the observable chain and no data can be seen while it shows on desktop or if I comment out the Throttle call or if I downgrade to .net 7.

What is the expected outcome?
To match the behavior on the desktop version and .net 7

What is the actual outcome?
No data can be seen on the wasm version of this avalonia project while they are seen on the desktop version. Data can also be seen on both versions if I downgrade to .net 7.

What is the stacktrace of the exception(s) if any?
None, no exception seems to be thrown.

Do you have a code snippet or project that reproduces the problem?
Yes, this issue contains a zip with a minimal Avalonia project using Rx.net via DynamicData.

AvaloniaApplication2.zip

I'm not confident I know what this means:

it breaks the observable chain

Looking at your code, it appears that this:
private readonly List<Stuff> _backingList= new(Enumerable.Repeat(new Stuff(), 10).ToList());

produces a fixed-size (10 entry) list of Stuff instances, each of which looks like this:
public partial class Stuff : ObservableObject

    [ObservableProperty]
    private int _thing = Random.Shared.Next();
}

which appears to pick a random number during construction.
None of these things appears to change, so I'm not sure what dynamic observable behaviour you're expecting. If there are no changes, what is there to observe?
I tried running your app. The desktop one shows the same number 10 times over in a list, which wasn't quite what I expected. And the Browser one was unable to get as far as showing anything at all. It just sits there with a browser page showing about:blank, and then after a while shows this:
Failed to launch debug adapter.  Additional information may be available in the output window.

Unable to launch browser: "Could not open wss://localhost:5001/debug?browser=wsAFF<IP_ADDRESS>A49289FdevtoolsFbrowserFf1eb29a5-bd1d-4a43-941b-eebd23f7b450"

I got that error both with and without the call to Throttle.
So I'm going to have to guess what problem you're seeing since the browser app in your repro won't start on my machine, and I don't understand what "breaks the observable chain" means.
I'm going to guess that you mean that the observable stops working—that you don't get data appearing in the display.
It's possible that this is connected to #2061 which caused by the fact that the default schedulers don't currently work very well on Blazor Wasm because there are many situations in which they attempt to create threads, which you're not allowed to do in that runtime environment.
Currently we have no way to run tests in Blazor Wasm because there seems to be no support for doing that - https://github.com/microsoft/testfx/issues/2196
We intend to improve support for wasm in the browser as soon as possible, but the inability to run automated tests makes it tricky for now.

I'm not confident I know what this means:

it breaks the observable chain

Looking at your code, it appears that this:
private readonly List<Stuff> _backingList= new(Enumerable.Repeat(new Stuff(), 10).ToList());

produces a fixed-size (10 entry) list of Stuff instances, each of which looks like this:
public partial class Stuff : ObservableObject

    [ObservableProperty]
    private int _thing = Random.Shared.Next();
}

which appears to pick a random number during construction.
None of these things appears to change, so I'm not sure what dynamic observable behaviour you're expecting. If there are no changes, what is there to observe?
I tried running your app. The desktop one shows the same number 10 times over in a list, which wasn't quite what I expected. And the Browser one was unable to get as far as showing anything at all. It just sits there with a browser page showing about:blank, and then after a while shows this:
Failed to launch debug adapter.  Additional information may be available in the output window.

Unable to launch browser: "Could not open wss://localhost:5001/debug?browser=wsAFF<IP_ADDRESS>A49289FdevtoolsFbrowserFf1eb29a5-bd1d-4a43-941b-eebd23f7b450"

I got that error both with and without the call to Throttle.
So I'm going to have to guess what problem you're seeing since the browser app in your repro won't start on my machine, and I don't understand what "breaks the observable chain" means.
I'm going to guess that you mean that the observable stops working—that you don't get data appearing in the display.
It's possible that this is connected to #2061 which caused by the fact that the default schedulers don't currently work very well on Blazor Wasm because there are many situations in which they attempt to create threads, which you're not allowed to do in that runtime environment.
Currently we have no way to run tests in Blazor Wasm because there seems to be no support for doing that - microsoft/testfx#2196
We intend to improve support for wasm in the browser as soon as possible, but the inability to run automated tests makes it tricky for now.

Sorry for the ambiguity, let me clarify.
What I mean by it breaks the chain is indeed that no data appears in the display. They exist, removing the Throttle call shows them on wasm, but it completely wipes them out if I leave throttle in.
As for why you cannot launch the debugger, I don't know. I am using Rider and f5 debugging appears to work with chrome, but it is limited. You can repro the issue with a simple run without debugging, but I am not sure why it didn't launch on your end.
As for the setup to repro the issue, it's mostly a minimal example to have some kind of data to isolate the problem that it's throttle that causes the issue. I normally got this from a much more complex project, but I thought that it was too complex  for an issue report so I made the most basic version I could think of that repro the issue in a new project.
There is one thing I am perplexed on: this doesn't reproduce if I downgrade the project to .net 7. It means somehow, .net 8 specifically is involved, but it's not clear why that would be. Is there anything that comes to mind on what could have changed?

Maybe you need ToObservableChangeSet instead of AsObservableChangeSet?
Source: https://stackoverflow.com/a/72542353/61158

Maybe you need ToObservableChangeSet instead of AsObservableChangeSet?
Source: https://stackoverflow.com/a/72542353/61158

This doesn't behave the same and it doesn't acomplish the same thing.
Besides, I don't know why it would have to do with this issue because other methods from DynamicData or Rx works just fine (I could even get a text filter to work).

I've now tried it in Rider, and am able to run the Blazor app and repro the problem. However, Rider seems entirely unable to step into Rx source code when debugging a Blazor app.
I've enabled external source debugging, and it works fine if I run the desktop app in your repro—with that, I can set a breakpoint inside Rx.NET's DefaultScheduler and watch what how Throttle operator is using that. But when I switch to the Blazor app, this all stops working.
I've found that Visual Studio also seems to have a bunch of similar problems when debugging Blazor apps, so I guess there are debug limitations with .NET on Browser WASM.
So far I have very little experience with .NET Browser WASM development. Perhaps there's something simple I'm missing, because as far as I can tell, I can't to set breakpoints inside NuGet libraries for a Blazor app. I can't even get the debugger to break in any time an exception is thrown (something that appears not to work for me on Blazor apps in general, not just with this app).  (As far as I can see, Blazor apps don't get the normal messages that the Debug Output window would typically show each time an exception is thrown.) If someone can point me at anything that would help with any of these blockers, that would be really handy, because right now, the debugger is making it more or less impossible for me to look into this in any meaningful way.
But failing that, I think what I'll probably need to do is open up the main Rx.NET repo and add a Blazor project to that, in the hope that it will then let me single-step through the Rx source code (because I won't be using it via NuGet in that case - it'll be a project reference). Right now I've got some other work in progress for Rx, so it might be a while before I can get to this.
Unfortunately, until such time as it's possible for us to run unit tests in Browser Wasm, we can't meaningfully offer support for that environment, so although I'm going to try to work out what's happening here, we just have to say that we don't properly support Blazor today.

if I downgrade the project to .net 7. It means somehow, .net 8 specifically is involved, but it's not clear why that would be. Is there anything that comes to mind on what could have changed?

.NET's WASM support is evolving quite quickly at the moment, so a lot of things will have changed from .NET 7 to .NET 8, making it very hard to guess which particular change might be at the root of this.


⚠ Bofrost.at has degraded performance

In cf62f7e, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1434 ms

Resolved: Bofrost.at performance has improved in ca9b99f.


UIDrawerController: Property automaticallyAddMenuButtonItems don't work as suggestion

First of all great work on this project! I really like to learn how it's easy to write a drawer controller based on your code. Thank you.

Setting automaticallyAddMenuButtonItems to false still adding the menu buttons.
This is the code from Example project:
    func application(_ application: UIApplication, didFinishLaunchingWithOptions launchOptions: [UIApplicationLaunchOptionsKey: Any]?) -> Bool {

        window = UIWindow(frame: UIScreen.main.bounds)

        let centerVC = UIViewController()
        centerVC.view.backgroundColor = .white

        let leftVC = UIViewController()
        leftVC.view.backgroundColor = .red

        let rightVC = UIViewController()
        rightVC.view.backgroundColor = .blue

        let drawer = UIDrawerController(centerViewController: UINavigationController(rootViewController: centerVC), leftViewController: leftVC, rightViewController: rightVC)
        drawer.automaticallyAddMenuButtonItems = false
        window?.rootViewController = drawer
        window?.makeKeyAndVisible()
        return true
    }

However, the setting automaticallyAddMenuButtonItems to false  actually not working at the time of initialization of UIDrawerController, it's working if I set it within the code it self.

Thanks ill look at these when I have a moment! In the meantime try out some of my other controllers in https://github.com/nathantannar4/NTComponents
They are more refined

@nathantannar4 Thanks! Yes, sure I will try other controllers too. Would you please check out the other open issues as well?

Fixed now


Update ACK MemoryDB controller tutorial

Issue #, if available: Update ACK MemoryDB controller tutorial
Description of changes: Updated ACK MemoryDB controller tutorial
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


Intl.plural: first argument as num

Fix error "The argument type 'dynamic' can't be assigned to the parameter type 'num'. dart(argument_type_not_assignable)" in generated code for Intl.plural calls
static String m0(count) =>       "${Intl.plural(count, one:

Somehow, I missed this PR and didn't check it in a decent time. Sorry for that.
After reviewing it, here are some conclusions:

Some tests are failing.
I've noticed that this change will introduce an unnecessary cast in the l10n.dart file for compound messages that include ICU Plural (e.g., "John has {count, plural, one {{count} apple} other {{count} apples}}."). This may annoy users who utilize this package.
Other ICU messages, like ICU Gender and ICU Select, will still encounter a similar issue (e.g., "The argument type 'dynamic' can't be assigned to the parameter type ..."). Hence, this PR only addresses the issues with plural messages and not the broader challenges with stricter code analysis.
The added check to determine if it's a plural message, followed by inserting a cast, does resolve the problem. However, it also complicates an already intricate code, potentially making future maintenance more challenging. If we were to adopt the same approach for other ICU messages facing similar stricter type check issues, we'd end up adding substantial code merely to patch things. I would prefer to rewrite the code to better accommodate stricter type checks rather than adding more conditional statements, if such a change is really needed.
Lastly, developers don't have control over generated code in the same way they do with their code. For that reason, they could exclude generated files from code analysis. That would allow them to ensure their code complies with stricter code analysis without concerning themselves with generated boilerplate code.


docs: add guide for using custom sources

Checklist

[x] The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
[ ] Links to related issues (if applicable)
[ ] Tests for the changes have been added/updated (if applicable)
[x] Docs related to the changes have been added/updated (if applicable)

This is a great start!
I think we also need to edit this doc : https://datahubproject.io/docs/metadata-ingestion/adding-source to say that there are two ways to add a source.

Contribute a source to DataHub, add it to the project
Write a custom source for yourself that you don't intend to contribute to the project (yet).


Add LazyTask loader

It is most popular solution for splitting monolithic gulpfile and lazy loading of modules.
I think all developers made it by themselves
I didn't cover tests this function because there is some problem with loading files by require function and test it. But if it has important for this project I can create in test-runtime jsfile for test loading

Coverage decreased (-11.8%) to 86.525% when pulling e1eca19051f6a1f9f4570f29da404a1a849a58ef on aliaksandr-pasynkau:master into b74a5ff121471ed00b84fb1e73a0e75488d33ccd on gulpjs:master.

gulp-util is deprecated so we shouldn't add more stuff to it.


Create transactions and redirect to /sign in the NEAR Wallet without logging in first

Problem
Currently, apps must always have their users "login" via the NEAR Wallet by adding an access key, before they can sign any transaction that includes a NEAR deposit. This is because the entire transaction is serialized before redirecting to the wallet, and the account name must be known ahead of time so it can be included.
Solution
Allow devs to send an unserialized TX to the wallet (or serialized without the account name and recovered in the wallet), so they do not need to implement "login" just to get the account name.

Linked Wallet issue: https://github.com/near/near-wallet/issues/1148

Is this still the case? I believe I've seen the behavior working, or what I'm interpreting this behavior to be. Maybe this is slightly different than what I'm picturing in my head.
Before, if a user was redirected to Wallet and didn't have an account, they'd have to sign up, then essentially follow the link again in order to confirm a transaction. It seems like that was fixed and pretty adjacent to what we're discussing here.

Hey everyone, I'm currently conducting a clean-up on the issues of NAJ. Please let me know if this isuse is still relevant; otherwise, I'll proceed to close the issue in 5 days.


Implement mode in Statistics calculation

Another nice statistics to have handy. Imo could be very useful
for memory measuremnts as I hope that memory usage will often be
very stable so getting the mode should be close to the actual
memory usage.
We also get the mode_count for "free` not sure if we should
include it or not (I have never seen it used), but it seems sort
of useful (like for memory usages we could say/include stuff like
60% of all samples were this that helps)

Well well, color me surprised once again. My performance concerns about doing mode "correctly" were unfounded. Tried it out in my elixir play ground and it's sometimes faster (probably also due to my more specific code/tail call optimized variant). I'll push it soon:
Operating System: Linux
CPU Information: Intel(R) Core(TM) i5-7200U CPU @ 2.50GHz
Number of Available Cores: 4
Available memory: 8.053432 GB
Elixir 1.5.0
Erlang 20.0
Benchmark suite executing with the following configuration:
warmup: 2.00 s
time: 5.00 s
parallel: 1
inputs: 100k, 5M, thousand
Estimated total run time: 42.00 s

Benchmarking with input 100k:
Benchmarking advanced mode...
Benchmarking basic mode...

Benchmarking with input 5M:
Benchmarking advanced mode...
Benchmarking basic mode...

Benchmarking with input thousand:
Benchmarking advanced mode...
Benchmarking basic mode...

##### With input 100k #####
Name                    ips        average  deviation         median
basic mode            30.62       32.66 ms    ±19.44%       31.35 ms
advanced mode         30.13       33.19 ms    ±23.28%       31.57 ms

Comparison:
basic mode            30.62
advanced mode         30.13 - 1.02x slower

##### With input 5M #####
Name                    ips        average  deviation         median
advanced mode          0.24         4.20 s     ±0.04%         4.20 s
basic mode             0.24         4.23 s     ±0.59%         4.23 s

Comparison:
advanced mode          0.24
basic mode             0.24 - 1.01x slower

##### With input thousand #####
Name                    ips        average  deviation         median
advanced mode        3.55 K      281.97 μs    ±35.78%      256.00 μs
basic mode           3.42 K      292.75 μs    ±36.86%      264.00 μs

Comparison:
advanced mode        3.55 K
basic mode           3.42 K - 1.04x slower


Rename Git repository to node-knockknock

We want to rename the Git repository from https://github.com/Skelp/knockknock.git to https://github.com/Skelp/node-knockknock.git.
We will need to make sure that we update all relevant links as well as uses of the old repository name and/or URL.
This does not affect the name of the package, just the repository.

This may also affect other things like webhooks (Coveralls, Travis, README badges).


In RTL mode align-right is not set correctly

When you generate a customized css in RTL mode the value for class ".align-right" is not set correctly:
Currently:
.align-right {      -ms-flex-pack: end;       justify-content: flex-end; }
and It should be:
.align-right {      -ms-flex-pack: start;       justify-content: flex-start; }
Because the content should be started from right and be stick to the right side of the container ;)

cc @abdullahsalem

Thanks @mozaffar for reporting this issue, and @IamManchanda for the mention.

Closing in favor of #10676


EditorFeatures enum is incorrectly ordered after new 'Game View' feature has been added

Tested versions

Reproducible in: 4.4.dev4.mono

System information
Godot v4.4.dev4.mono - Windows 10 - Vulkan (Forward+)
Issue description
In Editor -> Manage Editor Features..., the addition of the new 'Game' main feature has caused each feature's displayed name to be mismatched with the actual connected feature.
Relevant issue: #97257
Relevant changes seem to be with changes to the enums at:

editor/editor_feature_profile.cpp:46
editor/editor_feature_profile.cpp:58
editor/editor_feature_profile.cpp:70
editor/editor_feature_profile.cpp:313

Steps to reproduce

Open Editor -> Manage Editor Features... to the Manage Editor Feature Profiles window.
Disable 'Game View' feature, and note that the 'Asset Library' feature is disabled instead.
Disable 'Asset Library' feature, and note that the 'Scene Tree Editing' feature is disabled instead.

Minimal reproduction project (MRP)
N/A

reproduce


Moved Python microservices to new API

Changes proposed ✍

Moved Merge suggestions to new API

Refactored how the service works, such that instead of having a coordinator and checking merge suggestions periodically, we call the service every time a member is added to do a check against the whole database.
This is possible since usernames are static, we did not actually need a coordinator
If a username changes, we are adding a new member anyway, so we are not loosing that edge case

Moved engagement score to new API
Removed some unnecessary modules in Python
Fixed a typo in workerTypes

Checklist ✅

[x] Label appropriately with type:feature 🚀, type:enhancement ✨, type:bug 🐞, or type:documentation 📜.
[ ] Tests are passing.
[ ] New backend functionality has been unit-tested.
[ ] Environment variables have been updated

[ ] Front-end: frontend/.env.dist
[ ] Backend: backend/.env.dist, backend/.env.dist.staging, backend/.env.dist.staging.
[ ] Configuration docs have been updated.
[ ] Team members only: update environment variables in Password manager and update the team

[x] API documentation has been updated (if necessary) (see docs on API documentation).
[x] Quality standards are met.
[x] All changes have been tested in a staging site.
[x] All changes are working locally running crowd.dev's Docker local environment.

@joanreyero also tests are failing.


networking.extraHosts doesn't work

Bug description
Just using networking.extraHosts will not change the /etc/hosts file since that one is by default managed by WSL.
To Reproduce
set
config.networking.extraHosts = 'test <IP_ADDRESS>'
Expected behavior
I think either there should be an assertion that makes sure that wsl.wslConf.network.generateHosts and networking.extraHosts aren't set together, or even more ideally would be ofc a solution where we could still append to the wsl hosts config ^^

We already have https://github.com/nix-community/NixOS-WSL/blob/c6f663e8583912059605b55c07583b6e96b5e690/modules/wsl-distro.nix#L78-L85 . Not to sure how good the UX would be with an additional assert.

There should probably be a warning, when you try to use extraHosts while wsl.wslConf.network.generateHosts is true


Preview mode for captured HTML

Can we also have a preview option for the scraped HTML?

like capture the site preview as an image?

Like show the captured HTML rendered

please use HTML node for HTML preview


basex: fix Icon= store path to point to nix store.

Noticed the problem on never disappearing diff reported by:
$ ./maintainers/scripts/rebuild-amount.sh --print HEAD
Estimating rebuild amount by counting changed Hydra jobs.
      1 x86_64-linux

basex.x86_64-linux /nix/store/5ng...-basex-9.4.3

Before the change local non-store path was used:
$ nix-build -A basex; fgrep -R Icon result/
result/share/applications/basex.desktop:Icon=/home/.../tools/text/xml/basex/basex.svg

After the change the file got into store as expected:
$ nix-build -A basex; fgrep -R Icon result/
result/share/applications/basex.desktop:Icon=/nix/store/...-basex.svg

Things done

Built on platform(s)

[x] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
21.11 Release Notes (or backporting 21.05 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 137699 at b2500edc run on x86_64-linux 1

1 package built successfully:

basex

1 suggestion:

warning: missing-phase-hooks
installPhase should probably contain runHook preInstall and runHook postInstall.
Near pkgs/tools/text/xml/basex/default.nix:28:3:
   |
28 |   installPhase = ''
   |   ^


Add map_some and map_none methods

It's extremely common to have a pattern in the combinators where we directly want to map a T -> Option<T> like the below example:
comment().map(|_| None))
    .or(|| instruction().map(|i| Some(i)))

Thus a simple convenience method to encapsulate this pattern would be a nice space saver. Like the following:
comment().map_none()
    .or(|| instruction().map_some())

Won't do


fix: fixed operator 404 issue while using operator pool volume encryp…

Made sure that operator is created after k8s cluster so that the following required policy is in place before operator is created
Allow service blockstorage to use keys in compartment id ${var.compartment_id} where target.key.id = '${var.node_pool_volume_kms_key_id}'
Signed-off-by: Deepak Koli<EMAIL_ADDRESS>

As discussed, the operator module is independent and it should not depend on the oke module. As @KSN2510 is the original contributor to the operator volume encryption feature, can you please discuss with him to find why this is happening?

Closes #488


systemd service support

@Michael-F-Bryan asked about systemd support in #42 and you mentioned opening an issue if anything was needed. Systemd supports socket activated services, services that are lazily started when a client connects to a desired UNIX/TCP/fifo endpoint. libsystemd provides methods for systemd enabled services to receive the activated socket file descriptor such as sd_listen_fds. Rust socket activated daemons would need to call the C API to receive a systemd file descriptor. I wrote a tiny crate that provides that functionality in pure rust, would you be interested in either using it in spirit or merging that crate into this project? I wouldn't mind "donating" it, it's quite simple and it's only purpose is to make writing rust daemons easier (just like spirit :) )

Hello
Before jumping to conclusions, I'd like to know how that would fit into spirit. It would certainly not be part of the base spirit, it would have to be some kind of spirit-systemd-sockets or such. But still, how do you propose this would get integrated? How do these sockets get updated if the configuration changes? Etc… so, do you have a proposal for how the interface would look like?
Is that functionality actually used in practice, or is it one of these clever ideas that never got any traction? What's the advantage over just having a daemon listen to the sockets itself?
Can you link your crate you mentioned? I'd probably not want to merge it, as it could be useful separately and also I try to not inflate spirit itself too much if possible.

How do these sockets get updated if the configuration changes?
If the service's systemd unit file is updated, then systemd handles restarting the service.

Is that functionality actually used in practice, or is it one of these clever ideas that never got any traction?
Definitely. avahi, cups, and docker all use socket activation. This article: http://0pointer.de/blog/projects/socket-activation.html covers the benfits of socket activation: easier service parallelization, service persistence, easier service upgrades, etc.

Can you link your crate you mentioned?
Sure, here it is: https://github.com/jabedude/sd-daemon-rs/. It's very simple, but its necessary for systemd daemons who want to go the socket activation route.

I'm reading through the article and through your responses. I'm still not sure how this should interact with spirit, if at all. I mean, spirit is mainly about reading arguments and configuration and being able to adapt to the configuration at runtime.
The sockets from systemd would not be coming from configuration or command line, this seems to be somewhat orthogonal to spirit. I mean, you can have a service that uses both spirit and your library, or just one of them.
But maybe I'm still not seeing something. Could you maybe provide a snippet/example service and show how that would tie together? And what the advantage over using both spirit and your library but without any direct interaction would be?

If the service's systemd unit file is updated, then systemd handles restarting the service.

That's actually quite the opposite to what spirit tries to do. Let me explain with an example. If you have a database or a DNS resolver, it takes quite some time to start or to get to its full performance, because it fills caches (either from the disk or by asking authoritative DNS servers as it processes requests). So one wants to keep it running and not restart it when the configuration needs to be changed. Spirit allows, in some cases and with some assistance, to be able to change the configuration at runtime ‒ including things like sockets it listens on.
And I don't quite buy the things with seamless upgrades the article describes. If I'm a database server and I get restarted, it doesn't really help that the listening socket got preserved and that no new connection got lost during the time. The real trouble is killing all the long-term connections with the old instance ‒ which systemd doesn't seem to help in any way.
Nevertheless, I'm not against adding any meaningful support. After all, not all services are database servers. But I still don't understand how it would fit with spirit.

I was imagining having an extension for the spirit builder to maybe have the daemon configuration also receive activated sockets. A service should be able to use either or both spirit and my crate, so I guess adding this functionality isn't necessary.
Thank you, I have some uncommitted changes and then I will publish it :)

So, do you think this can be closed as there's nothing to be done on spirit's side?

Yes, if I think of any other systemd compatibility issues I'll open another issue!


css variables

I think the current css variables already offer a decent level of customization. I will reject this pr for the sake of a more lightweight plugin.


Unification of injected overloaded constructors

There is a missing unification case for two injected, overloaded constructors:
\inj{Sub1, Super}(Overloaded1) ∧ \inj{Sub2, Super}(Overloaded2)

where there is an overloading symbol Overloading in a sort Mid <= Super.
We do not know how to handle this case.
See also: #1269

I've discussed this briefly with @virgil-serbanuta and we think this should work:

If Sub1 and Sub2 have no common  subsorts, this fails, being already handled by unificationInj
Otherwise, look for an Overloaded' which overloads both Overloaded1 and Overloaded2 (and, therefore, is on a common supersort Super', subsort of Super)

If found, then transform \inj{Sub1, Super}(Overloaded1(child1)) ∧ \inj{Sub2, Super}(Overloaded2(child2)) into \inj{Sub1, Super}(Overloaded1(child1)) ∧ \inj{Sub2, Super}(Overloaded2(child2)) \inj{Sub1,Super'}(Overloaded'(\inj{SC1,SC1'}(child1) ∧ inj{SC2, SC2'}(child2)`

if not found, unification is not possible, return \bottom.

Some discussion items from the daily meeting, and unresolved questions:

We need to attempt resolving overloading when unifying injections for wasm-semantics.
When the children are overloaded into a common sort that is a subsort of Super, we can lift the children into the overloading sort and continue unification there.

What if there are multiple suitable overloading symbols?
It might be helpful to make an OverloadGraph similar to SortGraph.
It seems like overloads and injections are intertwined; it would be a good idea to think about how (or if) we will separate these responsibilities.

When the children are overloaded into a sort that is a supersort of Super, we should throw an error. We do not understand how this could occur in a correct definition or how to handle it safely.


Allow for passing a filter as a class

Jersey does not behave exactly the same when registering
object and class during a DynamicFeature configuration.
This commit allows for both behaviour to be exploited in
AuthDynamicFeature.
This fixes #1630.
@evnm @psamsotha please review and tell me if we need more doc that what I added :)

Coverage decreased (-0.06%) to 82.298% when pulling f275ac372e27a195dc6d7af6d9a056c39e0c03af on victornoel:issue-1630 into ea6656b19e8931a1d12969264eb68e8d7275e739 on dropwizard:master.

Change looks good! Just a couple documentation nit-picks.

Should be alright now :)

Coverage decreased (-0.06%) to 82.298% when pulling adb35d0ea122e84d196ff114ab74115bdf578e14 on victornoel:issue-1630 into ea6656b19e8931a1d12969264eb68e8d7275e739 on dropwizard:master.

Thanks for taking care of this, @victornoel.

Ok
Le lun. 16 sept. 2019 à 12:23, Victor Noël<EMAIL_ADDRESS>a
écrit :

@victornoel commented on this pull request.
In
dropwizard-auth/src/main/java/io/dropwizard/auth/AuthDynamicFeature.java
https://github.com/dropwizard/dropwizard/pull/1715#discussion_r324621792
:

@@ -21,13 +21,26 @@

If authorization is not a concern, then {@link RolesAllowedDynamicFeature}

could be omitted. But to enable authentication, the {@link PermitAll} annotation
should be placed on the corresponding resource methods.

Note that registration of the filter will be done following the semantics of

{@link FeatureContext#register(Class)} and {@link FeatureContext#register(Object)}:

passing the filter as a {@link Class} to the {@link #AuthDynamicFeature(Class)}

constructor will do dependency injection while passing the filter as an object to

the {@link #AuthDynamicFeature(ContainerRequestFilter)} constructor won't.

@meilleureVie https://github.com/meilleureVie please open a ticket or
ask on the google group for help, this is a merged PR.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/dropwizard/dropwizard/pull/1715?email_source=notifications&email_token=AF4CQ25IWMA3PRN7UJYCNQLQJ5UCRA5CNFSM4COLIKE2YY3PNVWWK3TUL52HS4DFWFIHK3DMKJSXC5LFON2FEZLWNFSXPKTDN5WW2ZLOORPWSZGOCEZQ73A#discussion_r324621792,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AF4CQ27CXXHHC553QDIOCXLQJ5UCRANCNFSM4COLIKEQ
.


Error when attaching discount to a customerGroup in the Admin Hub

Lunar version: 1.0.0-alpha.7
Laravel Version: 10.48.3
PHP Version: 8.1
Database Driver & Version: MySQL 8.2

Expected Behaviour:
The Discount should be attached to a CustomerGroup .
Actual Behaviour:
an error is prompted (refer image).

There is the customerGroups relationship in the Lunar\Models\Discount, however it is calling the relationship from Lunar\Models\CustomerGroup instead.
Steps To Reproduce:
In Admin Hub, create a discount and attach it to a customerGroup.

A similar error happens with scheduling channel availability

Can be closed now https://github.com/lunarphp/lunar/pull/1658 has merged


Fix react state issue by removing mutation

Just cleans up the weird react state trouble with some help from a friend of mine who knows loads about these things.

Think we still have some stale client stuff elsewhere but that's a level of abstraction issue caused by react expecting certain things at component level only AFAICT. Will resolve once moneta is out. Thx for the patch :)


Weird diagram rendering of estimator

https://inria.github.io/scikit-learn-mooc/python_scripts/parameter_tuning_sol_01.html

maybe some CSS issue?

Yep I assume this is a CSS issue. It works in notebooks and VSCode.

CSS problem => @GaelVaroquaux in my mind :wink:

CSS problem => @GaelVaroquaux in my mind 😉

It would be worth investigating this indeed, and fixing it in scikit-learn.
However, I have in mind that we fixed a few things regarding CSS in
scikit-learn a little while ago. Did you check that the problem occurs
with scikit-learn master?

I would rather think this is an issue scikit-learn with jupyter-book.
Everything is fine in the scikit-learn master documentation.
On Wed, 16 Dec 2020 at 11:08, Gael Varoquaux<EMAIL_ADDRESS>wrote:

CSS problem => @GaelVaroquaux in my mind 😉

It would be worth investigating this indeed, and fixing it in scikit-learn.
However, I have in mind that we fixed a few things regarding CSS in
scikit-learn a little while ago. Did you check that the problem occurs
with scikit-learn master?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/INRIA/scikit-learn-mooc/issues/146#issuecomment-746026767,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABY32PYXOCWLXG4L6N5ADFLSVCBIPANCNFSM4U4OA7JA
.

--
Guillaume Lemaitre
Scikit-learn @ Inria Foundation
https://glemaitre.github.io/

I would rather think this is an issue scikit-learn with jupyter-book.

I agree.

Everything is fine in the scikit-learn master documentation.

My question was: does this happen if we generate the jupyter-book with
scikit-learn master?
My hunch is that we should hard-code some aspects of the CSS (eg borders)
so that they are not overrided by the outer CSS.

This has been fixed in the scikt-learn development version: https://github.com/scikit-learn/scikit-learn/pull/19417


Create one PyxisGQL instance per thread

GQL's sync transport (RequestsHTTPTransport) can't be shared among threads, create one PyxisGQL instance per thread to run Pyxis queries in parallel.
JIRA: CWFHEALTH-1352

We can use gql's async transport, but it requires much more refactoring work, so at this moment, just create one PyxisGQL instance per thread, in our phase 2 of refactoring work, we can try to use gql's async transport.

Please ignore the CI failures, they have been fixed in the main branch.

Forgot to commit a local change to tests/test_image.py, sorry, PR is amended and ready for review.

All comments have been addressed, I'm going to merge this since there is one approval already :)


GH-334: remove duplicate "--proto_path" logs

issue - duplicate "/src/main/protobuf" in argline
fix - there was a redundant setting of importPaths that overwrote the previous logic which filtered duplicates

Just fixed the pipeline rules as they were not quite right on my side, so updated your branch to allow it to run.

Thanks!


Remove active health check on target group

We have implemented sophisticated KEA perfdhcp health checks and don't
use this health check as validation that the node is healthy.
This NLB health check takes around a minute to finish and even though
ECS reports that the server is up, it still needs to wait for this
health checks to complete before being added to the load balancer.
By removing this health check, we still get a required passive health check
before target is considered healthy.

Does this mean we can remove nginx on the dhcp server docker container? IT seems that dependency is now redundant and we should probably also lock down service ports too.


Wands and Spells mod

Howdy, (I hope it's the correct GitHub) I have played with your amazing magic mod, it's quite good, but I have few minor complains, first of all bolt spell type doesn't seem to work at all, it shoots projectile yes, but the projectile itself doesn't really deal any damage, if we talk about the projectiles then maybe consider improving them, add some particles, maybe work on how they look in general (optionally do the same to wands), because for now they look kinda boring and magic isn't boring, a lot of spells work but in very bad way, water dome works just fine, but it's just annoying on it's own, earth removal targets only dirt and doesn't give a shit about grass or stone, pillar and circle stone spells have somewhat same problem (or they just don't work at all, hard to tell), lastly mana system, it's rather pointless I would say given you can have only single spell on wand and you can't really spam them nor spells are very powerful, I recommend removing it and instead just extend recharge of wand (more mana spell would take = more time you will have to wait to cast it again) it seems better , it's just annoying and if you play with many magic mods then it's just another mana you have to keep up with, other than that it's great, simple yet fun, will be great for early game spell system, just a proposition but maybe add magic armour sets and definitely more spells plus maybe make spell books of specific type (fire, air, earth, etc.) look different so you can tell by just looking at the book what spell type it is, anyway that's all I wanted to say, again great mod, have a good one buddy and take care of yourself.    :)

This is a lot, let me take it bit by bit

but the projectile itself doesn't really deal any damage

Intended. You can change the damage via configs or game rules

add some particles, maybe work on how they look in general

If you can help with these render issues, you are welcome to PR

earth removal targets only dirt and doesn't give a shit about grass or stone

Not working on grass is intended. It should work on stone though. Just tested and it works fine. If you are sure it doesn't work on Stone or Cobblestone, open an issue for that alone please. It is also not supposed to work on e.g. Stonebricks. All blocks it should work on are in magic/tags/block/earth.json. In a modpack you can add other appropiate blocks.

pillar and circle stone spells have somewhat same problem

same thing, same tag. Shoot pillar on Stone or Netherrack or Endstone. Earth Circle can only raise earth/stone when there is some nearby, else it doesn't work of course.

lastly mana system, it's rather pointless

I disagree.

you can't really spam them

Wand cooldown is another gamerule you can change

it's just annoying and if you play with many magic mods then it's just another mana you have to keep up with

I'm open for compat issues :)

just a proposition but maybe add magic armour sets

If I had a pixel artist...

definitely more spells

Planned, suggestions are welcome

plus maybe make spell books of specific type (fire, air, earth, etc.) look different

I use vanilla mechanics for the enchanted books, so this is not possible rn/only with a lot of effort I'm not ready to put in.

again great mod

Thanks for your feedback :)

Well I actually know a guy who could work as an artist, not sure how good he is in terms of armour but I strongly believe in him, https://www.curseforge.com/members/count_sjorn_paradox/projects
I have noticed a LOT of features in your mod are configurable so maybe let's go for compromise, make mana system a optional feature, both sides are happy this way,
You could add summon spell type, allowing you to summon allies, mounts, n such, but I'm not sure if it would fit this mod as it's very vanilla, maybe self fling, similar to levitate, but faster, wardens echo attack would be kinda cool, charge forward spell is another nice one, maybe like a option to make ennemie fight for you, stealth one just because it's a popular spell in a lot of mods and lastly wither skull and dragons breath ball, oh also could you make 1.19.2 version, a lot of mods are still for this version of the game, thanks and have a good one.  ^^


CRI-Plugin crashes when defining other runtimes in imported config-file

Description
Hi,
I have recently worked on integrating WebAssembly runtimes in Kubernetes.
Part of that was to configure additional runtimes in containerd.
Lets assume the following config inside /etc/containerd/config.toml:
version = 2
root = "/var/lib/containerd"
state = "/run/containerd"
imports = ["/etc/containerd/config.d/*.toml"]

[grpc]
address = "/run/containerd/containerd.sock"

[plugins."io.containerd.grpc.v1.cri".containerd]
default_runtime_name = "runc"
discard_unpacked_layers = true

[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.runc]
runtime_type = "io.containerd.runc.v2"

[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.runc.options]
SystemdCgroup = true

[plugins."io.containerd.grpc.v1.cri".cni]
bin_dir = "/opt/cni/bin"
conf_dir = "/etc/cni/net.d"

[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.spin]
runtime_type = "io.containerd.spin.v2"
[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.spin.options]
BinaryName = "/bin/containerd-spin-shim-v2"

The important part are the last four lines:
[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.spin]
runtime_type = "io.containerd.spin.v2"
[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.spin.options]
BinaryName = "/bin/containerd-spin-shim-v2"

Initially I wanted to place those in a separate file like /etc/containerd/config.d/webassembly.toml
However, if I do that the CRI plugin simply will not start and I could not find out why.
At first I thought I am doing things wrong in setting this up, but one day I was too lazy to put my config in the separate file and that is when things suddenly worked.
I went back and move those lines to the imported config again and CRI was crashing again.
I tried it with other settings like discard_unpacked_layers = true, which are working from imported configs.
But those regarding the runtimes are not.
Seems like a bug to me.
Steps to reproduce the issue

Setup containerd with a config similar to the above and put the runtimes parameters for WebAssembly into a config that gets imported.
Restart containerd
Run ctr plugin ls to see that the CRI plugin has crashed

Describe the results you received and expected
I was expecting that it does not make any difference in which config file I put my settings and containerd just keeps on working.
What version of containerd are you using?
containerd github.com/containerd/containerd 1.7.11 64b8a811b07ba6288238eefc14d898ee0b5b99ba
Any other relevant information
runc version 1.1.11
Show configuration if it is related to CRI plugin.
version = 2
root = "/var/lib/containerd"
state = "/run/containerd"
imports = ["/etc/containerd/config.d/*.toml"]

[grpc]
address = "/run/containerd/containerd.sock"

[plugins."io.containerd.grpc.v1.cri".containerd]
default_runtime_name = "runc"
discard_unpacked_layers = true

[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.runc]
runtime_type = "io.containerd.runc.v2"

[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.runc.options]
SystemdCgroup = true

[plugins."io.containerd.grpc.v1.cri".cni]
bin_dir = "/opt/cni/bin"
conf_dir = "/etc/cni/net.d"

[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.spin]
runtime_type = "io.containerd.spin.v2"
[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.spin.options]
BinaryName = "/bin/containerd-spin-shim-v2"

You can use follow config to set spin runtime, and you can use containerd config default command to generate default config. hope can help you.
    [plugins.'io.containerd.cri.v1.runtime'.containerd]
      default_runtime_name = 'runc'
      ignore_blockio_not_enabled_errors = false
      ignore_rdt_not_enabled_errors = false

      [plugins.'io.containerd.cri.v1.runtime'.containerd.runtimes]
        [plugins.'io.containerd.cri.v1.runtime'.containerd.runtimes.spin]
          runtime_type = "io.containerd.spin.v2"
        [plugins.'io.containerd.cri.v1.runtime'.containerd.runtimes.spin.options]
          BinaryName = "/bin/containerd-spin-shim-v2"

Hi @lengrongfu ,
thank you for your comment.
The problem is however not to get a working config. I have that.
I am just reporting that if you set certain parameters in a config file that gets imported, the CRI plugin crashes, which seems like a bug to me.
I do not have a problem currently, this just seems faulty.

The error is in the containerd stdout/stderr when starting with your included config:
WARN[2024-04-04T18:30:37.688903174Z] failed to load plugin io.containerd.grpc.v1.cri  error="invalid plugin config: no corresponding runtime configured in `containerd.runtimes` for `containerd` `default_runtime_name = \"runc\""

This error is due to the current merging strategy of imported config sections. It definitely had some drawbacks, and specifically, a contributor has been working on an improved strategy in a PR I'll link below. In the current implementation, your second file with definitions for a new runtime entry is effectively replacing, not adding to, the definitions for runtimes in the main file. If you move everything about CRI to the second file (the whole subtree of the config) then the CRI plugin works properly.
More details in this PR: #9982

Hi @estesp ,
thank you for looking into this.
I can confirm that if I build my imported config like this it works:
[plugins."io.containerd.grpc.v1.cri".containerd]
default_runtime_name = "runc"
discard_unpacked_layers = true
[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.runc]
runtime_type = "io.containerd.runc.v2"
[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.runc.options]
SystemdCgroup = true
[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.wasmedge]
runtime_type = "io.containerd.wasmedge.v1"
[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.wasmedge.options]
BinaryName = "/bin/containerd-shim-wasmedge-v1"
[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.spin]
runtime_type = "io.containerd.spin.v2"
[plugins."io.containerd.grpc.v1.cri".containerd.runtimes.spin.options]
BinaryName = "/bin/containerd-shim-spin-v2"

Containerd will restart with no errors and the CRI will be okay.
I must have overlooked the warning in the stdout/stderr of containerd initially. When I looked for it today, I found it of course.
I am curious to see how the PR works out in the end.


[docs] bad links in README and project pages

The README page has a link to a 404 page:

The project page links to a site that does not exist:

Happy to help out with a PR, but I'm not sure where these should link to...

oh, the problem is I change the DNS hoster from google dns to cloudflare

I will fix this soon. Sorry for that

Fixed

please wait for a while or clear your browser cookie


[support]: spack load ncl results in an error about ninja being required

Name of Software
ncl
URL for Software
No response
Contact Details
<EMAIL_ADDRESS>HPC System
NERSC
Request Description
When trying to load ncl in e4s-22.11, I get the following error about ninja missing, however it is there. It seems a dependency of ncl requires ninja, but it is not seeing it?
We do not specifically list ninja as an external in our configuration. Would that be the case?
There is a similar issue in the spack repo, which I will link here: https://github.com/spack/spack/issues/37704
Thanks again for your hard work and any help you can provide.
Relevant log output
$ spack load ncl
==> Error: spack requires 'ninja'.  Error occurs here. `ncl` is not loaded

Here is some output from spack --debug load ncl
==> [2023-06-09-14:59:05.674956] CommandNotFoundError: spack requires 'ninja'. Make sure it is in your path.
==> [2023-06-09-14:59:05.675008] Error: spack requires 'ninja'. Make sure it is in your path.
Traceback (most recent call last):
  File "/global/common/software/spackecp/perlmutter/e4s-22.11/83104/spack/lib/spack/spack/main.py", line 1003, in main
    return _main(argv)
  File "/global/common/software/spackecp/perlmutter/e4s-22.11/83104/spack/lib/spack/spack/main.py", line 958, in _main
    return finish_parse_and_run(parser, cmd_name, env_format_error)
  File "/global/common/software/spackecp/perlmutter/e4s-22.11/83104/spack/lib/spack/spack/main.py", line 986, in finish_parse_and_run
    return _invoke_command(command, parser, args, unknown)
  File "/global/common/software/spackecp/perlmutter/e4s-22.11/83104/spack/lib/spack/spack/main.py", line 635, in _invoke_command
    return_val = command(parser, args)
  File "/global/common/software/spackecp/perlmutter/e4s-22.11/83104/spack/lib/spack/spack/cmd/load.py", line 115, in load
    env_mod.extend(uenv.environment_modifications_for_spec(spec))
  File "/global/common/software/spackecp/perlmutter/e4s-22.11/83104/spack/lib/spack/spack/user_environment.py", line 92, in environment_modifications_for_spec
    spec, context="run", set_package_py_globals=set_package_py_globals
  File "/global/common/software/spackecp/perlmutter/e4s-22.11/83104/spack/lib/spack/spack/build_environment.py", line 1017, in modifications_from_dependencies
    add_modifications_for_dep(dspec)
  File "/global/common/software/spackecp/perlmutter/e4s-22.11/83104/spack/lib/spack/spack/build_environment.py", line 993, in add_modifications_for_dep
    dpkg.setup_dependent_package(changes, spec)
  File "/global/common/software/spackecp/perlmutter/e4s-22.11/83104/spack/var/spack/repos/builtin/packages/ninja/package.py", line 81, in setup_dependent_package
    determine_number_of_jobs(parallel=self.parallel),
  File "/global/common/software/spackecp/perlmutter/e4s-22.11/83104/spack/lib/spack/spack/util/path.py", line 160, in path_filter_caller
    return func(*args, **kwargs)
  File "/global/common/software/spackecp/perlmutter/e4s-22.11/83104/spack/lib/spack/spack/util/executable.py", line 315, in which_string
    raise CommandNotFoundError("spack requires '%s'. Make sure it is in your path." % args[0])
spack.util.executable.CommandNotFoundError: spack requires 'ninja'. Make sure it is in your path.

I acknowledge this issue. We will investigate. Thanks.

There is a similar issue in the spack repo, which I will link here: spack/spack#37704

Hi Justin, thanks for bringing this to our attention. Yes, we believe the issue you describe here is the same as https://github.com/spack/spack/issues/37704. We will coordinate with Spack team on a resolution and close the ticket here when it is resolved.

I confirmed with Spack team that unfortunately this bug remains unresolved. It has been added to a list of bugs to fix but is not complete yet. We will keep this issue open until we can report it has been resolved upstream.

A fix has been merged upstream! This is now resolved.

https://github.com/spack/spack/pull/35737


FB Bots error help!!

Here i get error when i start my bots and its says
And i get default bots without 132 mass

who help i give all my accounts

Link to the script you use?
Sendt fra min iPhone

Den 28. mar. 2016 kl. 07.25 skrev sejo153<EMAIL_ADDRESS>who help i give all my accounts
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub

find on youtbe break agar and check latest video. that script

same


Set the auto_registration.entities.repository.enabled option to true when calling repo register --all

Summary
When registering all repositories, what the user really means is
"register all future and current repositories. The current invocation of
the call only registered all current ones. Let's also flip the option to
register all future repositories at the same time.
Fixes: #3817
Change Type

[ ] Bug fix (resolves an issue without affecting existing features)
[x] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing

minder provider get should either show the auto registration is set to false or not set at all
minder repo register --provider=$NAME --all
minder provider get again should show the auto registration is seat to true

Note: minder repo register --all currently requires the provider name to be passed. We should fix the command to iterate over providers that implement the RepoLister trait instead.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[ ] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[ ] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

@blkt 👀

coverage: 52.906% (-0.1%) from 53.048%
when pulling 3855a0d23ec580c6ad09315e503ffddbdaf4b9f5 on jhrozek:register_all_on_provider_update
into 41e48333e3c329cc02a6eb688bf0192dcbd77493 on stacklok:main.


Release v0.0.19

:white_check_mark: odo build 1548 completed and artifacts can be found here (for commit https://github.com/redhat-developer/odo/commit/b66bc6f210 by @surajnarwade)

You can ignore CircleCi for now


feat(hax-lib): intro hax-lib

This commit adds a crate named `hax-lib`.

This crate contains hax-specific helpers for Rust programs. Those
helpers are usually no-ops when compiled normally but meaningful when
compiled under hax.

For now, essentially, this adds an assert and an assume macro.

Thanks for the review :smiley:
I've fixed all the points you mentioned (I think!).
If those changes are OK for you, I'll rebase things, the CI is broken on the PR for the other branch type-closure-fstar :(

CI is going through in the other PR :partying_face:


Executor

This is a first draft of the executor integration. I implemented a noop executor that is used when pro-feature is inactive.
There is a sample application in the "plots" endpoint.

 Executor that decouples processing from web requests

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

544 of 633 (85.94%)  changed or added relevant lines in 11 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.03%) to 85.358%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

operators/src/error.rs
0
1
0.0%

services/src/error.rs
0
1
0.0%

services/src/pro/contexts/in_memory.rs
1
3
33.33%

services/src/pro/contexts/postgres.rs
1
4
25.0%

operators/src/executor/error.rs
0
8
0.0%

operators/src/executor/mod.rs
0
11
0.0%

services/src/handlers/plots.rs
18
29
62.07%

operators/src/pro/executor/replay.rs
326
347
93.95%

operators/src/pro/executor/mod.rs
191
222
86.04%

Totals

Change from base Build<PHONE_NUMBER>:
0.03%

Covered Lines:
27995

Relevant Lines:
32797

💛  - Coveralls


Error: Type 'TableMetadata' not found | FAILURE: Build failed with an exception

I am new to flutter and I am facing this issue that after cleaning the build (flutter clean) my code stopped working and it is stuck in build failed..
I tried deleting the flutter SDK and doing that process again but still I am getting the same error messages.
I was trying to fix the Firebase login so after restarting my laptop and then replaced the 'google-services.json' file and then after these commands flutter [clean build, pub get and pub upgrade] the code is stuck in build.
(this is the error message and the screenshot)
`Running Gradle task 'assembleDebug'...
../flutter/.pub-cache/hosted/pub.dartlang.org/flutter_widget_from_html-0.5.1+5/lib/src/widget_factory.dart:60:57: Error: Type 'TableMetadata' not found.
  Widget buildTable(BuildMetadata m, TextStyleHtml tsh, TableMetadata data) =>
                                                        ^^^^^^^^^^^^^
../flutter/.pub-cache/hosted/pub.dartlang.org/flutter_widget_from_html-0.5.1+5/lib/src/internal/layout_grid.dart:7:24: Error: Type 'TableMetadata' not found.
    TextStyleHtml tsh, TableMetadata data) {
                       ^^^^^^^^^^^^^
../flutter/.pub-cache/hosted/pub.dartlang.org/flutter_widget_from_html-0.5.1+5/lib/src/widget_factory.dart:60:57: Error: 'TableMetadata' isn't a type.
  Widget buildTable(BuildMetadata m, TextStyleHtml tsh, TableMetadata data) =>
                                                        ^^^^^^^^^^^^^
../flutter/.pub-cache/hosted/pub.dartlang.org/flutter_widget_from_html-0.5.1+5/lib/src/internal/layout_grid.dart:7:24: Error: 'TableMetadata' isn't a type.
    TextStyleHtml tsh, TableMetadata data) {
                       ^^^^^^^^^^^^^

FAILURE: Build failed with an exception.

* Where:
Script 'D:\Flutter\flutter\packages\flutter_tools\gradle\flutter.gradle' line: 904

* What went wrong:
Execution failed for task ':app:compileFlutterBuildDebug'.
> Process 'command 'D:\Flutter\flutter\bin\flutter.bat'' finished with non-zero exit value 1

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

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

BUILD FAILED in 6m 10s
Running Gradle task 'assembleDebug'...
Running Gradle task 'assembleDebug'... Done                       376.2s (!)
Exception: Gradle task assembleDebug failed with exit code 1`

tried to search for this build failed issue but I can't get to fix it.

@yoush97
Looking at the error log, it is pointing to 3rd party plugin flutter_widget_from_html. Please open this issue in dedicated repo: https://github.com/daohoangson/flutter_widget_from_html/issues
Closing from here. If you disagree, write in comments and I'll reopen it.


"&" in text stops further display of annotated text; also stops processing

This a result of #12


Display the icon in Linux

I confirmed it works in the development mode, the production mode, and also in the packaged binary. No change is required to the build script.
Before:

After:

ss.png is resized to 128x128 because this post recommends to do so:

... the icon will display in ubuntu if:
...
2. The image is 128x128 or less (Sometimes 256x256 works, but in most cases it doesn't)

It turned out that the word "icon" in https://github.com/mazun/ASlack-Stream/commit/08d5bd8621dee9f9cd8f498e000ee6809ddc79a0 and https://github.com/mazun/ASlack-Stream/commit/dd2f04331b34557b3ee607638b53236a16acc7c6 refers the one shown in the filer (explorer.exe in Windows and finder in Mac), but not the one treated in this PR.
I will close this PR and re-open a new one with the clarification about this thing.


Allow PACKET key in MultiSelectionPrompt

When use mobile RD Client, can not toggle items through <SPACE>, it actually <PACKET> key. (I tested iOS, not sure
about other platforms)
Here is my test:

To toggle items, must use Screen Keyboard...

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Sorry, i closed this PR due to a mistake...


Make symlink targets writeable, so we can write altnames

Fixes #9667.
I was also thinking if I should first read the existing set of permissions, change it, run xattr and then change it back as it was, so that nobody knows we were here :smile_cat:
but since we're already changing the content of that app my writing the alt name, I don't think it's necessary.

@rolandwalker will you review this, please?

Any news on getting this PR assigned/merged - issue with permissions seems to be same on Mavericks? Thanks

This looks fine to me. @vitorgalvao, are you comfortable with merging this?

Yes.


Validations that are based on external data

Hello. I have a validation that relies on some external data. Imagine we're validating the creation of a meeting room booking, where every user has a maximum duration they are allowed.
I want to check the provided user_id belongs to someone who exists, and then that the duration is below their configured maximum. For example:
Dry::Validation.Form do
  required(:user_id).filled
  required(:duration).filled(:int?)

  validate(valid_user: %[user_id]) do |user_id|
    !User.find(user_id).nil?
  end

  validate(valid_duration: %[user_id duration]) do |user_id, duration|
    user = User.find(user_id)
    duration < user.max_duration
  end
end

Is there a good way to avoid calling User.find twice? This is actually a call to an HTTP API, so it would be great if I could reduce this to just one call. Thanks!

Hi @samstarling — we encourage you to use https://discourse.dry-rb.org/ for advice and support requests. Please go ahead and make your posts there and we'll see if someone can get back to you!


Fix README.md for autosuggestions and history-substring-search

Fixes #1441 and #1444
Proposed Changes

Fixed documentation regression for autosuggestions module
Fixed the incorrect use of 'conjuncture' by replacing it with 'conjunction'.

Regarding changes in autosuggestions module refer to my comments in #1444.
TL;DR: "it should be loaded after it" meant "this module should be loaded after XYZ module"

@belak could you take a look?

@belak , done

Looks good, thanks!


I think i found a typo but i'm not sure it might have been intended.

Bootstrap documentation main page:

Intended to be as-is. Thanks though!


LogReader missing files counter

#32797

can you implement it in the error message? You can also avoid adding too many lines by incorporating a check to ensure that missing files == 0 or similar

I hope that i understood properly what were you looking for...

By checking the code, i figured out that there is some possible redundancy, in way that there is needlessly checking of qlogs. It probably could be optimized.

Only way to do that things is to count rlogs and qlogs separately. With that we can use more intuitive if statement, and incorporate right values for missing logs.

If it's possible, this should be a couple line change. We don't want to add this many lines for such a feature

Then we can just count missing rlogs, cause you expecting all qlogs to be there, and does not returning them anywhere.


Conversational form show nothing

l created a project with a conversational form and it worked prefectly.
Then l created addtional one in exiting project and the browser and the console show
no errors but the page was blank - with no conversation.

Remove


[Documentation Update]: Add hyperlinks to README.md

Is there an existing issue for this?

[X] I have searched the existing issues

Issue Description
There aren't any links to the technologies used in the README MD file.

Suggested Change
Add links to the official websites of the technologies used.

Rationale
This will help visitors easily navigate to the official documentation of the technologies used in the project and thus, they will be understand how the project works quicker.
Urgency
Low
Record

[X] I have read the Contributing Guidelines
[X] I have starred the repository

Hey @dmotts I can do this task please can you assign it to me.
Thanks!

Sure thing, thanks for your help!


deployment.environment for RUM dashboards

:thread:[deployment.environment] for RUM dashboards
Slack Message

moving to Asana


Openchannel fails - rpc error: code = Code(189) desc = You gave bad parameters:Unknown chain-hash

Background
I can't open a channel with node 02dd4cef0192611bc34cd1c3a0a7eb0f381e7229aa3309ae961a7fc0076b4d2bb6@<IP_ADDRESS>:9735.
I tried other node and could open a channel.
Your environment

version of lnd : "0.4.2-beta commit=b5a228808b0446d6758a23e24d6c686aab9936fc"
which operating system : Linux Celeri 4.15.0-23-generic #25-Ubuntu SMP Wed May 23 18:02:16 UTC 2018 x86_64 x86_64 x86_64 GNU/Linux
version of bitcoind : "version": 160000,
"subversion": "/Satoshi:0.16.0/",
"protocolversion": 70015,

Steps to reproduce
Bitcoind is synced and running. Lnd idem.
Connect 02dd4cef0192611bc34cd1c3a0a7eb0f381e7229aa3309ae961a7fc0076b4d2bb6@<IP_ADDRESS>:9735 ==> OK
openchannel 02dd4cef0192611bc34cd1c3a0a7eb0f381e7229aa3309ae961a7fc0076b4d2bb6 50000 ==> failure
Expected behaviour
Opening a channel with the said node and local amount of 50000 sat.
Actual behaviour
Got this message : [lncli] rpc error: code = Code(189) desc = You gave bad parameters:Unknown chain-hash<PHONE_NUMBER>19d6689c085ae165831e934ff763ae46a2a6c172b3f1b60a8ce26f

That means you're on say testnet, but trying to connect to mainnet. Make sure you're on the same chain as the node you're trying to connect to.


Cross compare functionalities with iridium-browser

Hi,
What are the things that could be ported over to this project from the iridium browser, if any?

I've ported over pretty much everthing except the redundant patches (that were already in other patchsets) and those that redirected traffic to Iridium servers. I did mention in the README that I used Iridium patches, but I was not very clear as to what I included.
Does this answer your question?

Nothing in particular, this browser has gotten some exposure recently, I wanted to make sure we're not missing anything. If that's the case, then I'm closing this.


Unable to use v2.0.1 in go.mod

Added to go.mod:
github.com/sendinblue/APIv3-go-library/v2 v2.0.1

Getting this error with go mod download:
go<EMAIL_ADDRESS>go.mod has non-.../v2 module path "github.com/sendinblue/APIv3-go-library" (and .../v2/go.mod does not exist) at revision v2.0.1

Hi @kbokarius
Here, you can follow the given steps:

Make sure you have set GO111MODULES, if by mistake it is off then please make it enable.
go env -w GO111MODULE=on
go to your work directory in terminal and run:

go mod init <filename/module name>
go mod tidy
eg, suppose your directory name is "TestProject" and inside this having a module directory as myProject having go files.
Structure would be like:
--TestProject
--myProject
- *.go
- *.go
Then command would be like go mod init myProject and then go mod tidy

It seems to be working now. Thanks @jain-raunak. Love the customer support you guys provide!


Wrong dot position in floating action button.

The dot is in wrong position with com.google.android.material.floatingactionbutton.FloatingActionButton

Hi,
Can you please share a sample snippet of layout xml and java/kotlin to reproduce the same?


[UI] - Failed to deploy app when a namespace already exists

The installation process fails telling us that the workspace namespace already exists, it looks like it tries to create it again.

Previous screen is

Versions used:
Latest dev dashboard, backend-ui and epinio.
Spotted by the new CI (polishing still in progress): https://github.com/epinio/epinio-end-to-end-tests/runs/6917078783?check_suite_focus=true

Validated in the latest build, thx for the fix.


Assigning subsample_name causes pep.subsample_table to return a list rather than the DataFrame object

Snakemake version
5.23.0
Describe the bug
When I specify a subsample_name column in the subsample_table.csv the pep.subsample_table function returns a list of length 1 containing the pandas DataFrame. Ideally, it should just return the DataFrame itself.
Logs
This tends to break rules where I am getting a subsample_name for a given sample_name:
Error:
  AttributeError: 'list' object has no attribute 'loc'

Minimal example
I've created a reproducible example from a minimal config/samples/Snakefile directory: https://www.dropbox.com/s/feuj1dqtkqmfuip/example.tar.gz?dl=0

@jma1991, this was as temporary, inadvertent API change in peppy, the PEP processing package.
pep.subsample_table returned a list of pandas.DataFrame objects since multiple subsample tables can be defined for a project. So in case there is just one, you can just select the first list element.
In peppy 0.30.3 (just released) we do this by default, so this issue will go away.


LVM support

As #356 was closed, and there is no replies to it, i'm opening another issue.
#356 says LVM support is in 0.4.0+, but I can not find any documentation for this, nor can I find the lvm commands.
Was this not added? If it was added, where is the documentation?
Thanks

I'm wondering about this too.
I have a server where I use mdadm and LVM to manage my storage configuration.
If LVM actually works, even through an addon Docker, I want to migrate my server to run RancherOS.

weird , nooooo answers

I made some attempts, as follows:
root@ip-172-31-19-80:# ros -v
ros version v1.1.2-rc2

root@ip-172-31-19-80:# ros console switch ubuntu
root@ip-172-31-19-80:# apt-get install lvm2

# prepare another volume, then check it
root@ip-172-31-19-80:~# fdisk -l
....
....
Disk /dev/xvdf: 10 GiB,<PHONE_NUMBER>0 bytes, 20971520 sectors
Units: sectors of 1 * 512 = 512 bytes
Sector size (logical/physical): 512 bytes / 512 bytes
I/O size (minimum/optimal): 512 bytes / 512 bytes

# edit /etc/lvm/lvm.conf and change use_lvmetad=1 to use_lvmetad=0
# then reboot

Now you can use LVM, I created one pv and one vg.
root@ip-172-31-19-80:/home/rancher# pvscan
  PV /dev/xvdf1   VG niusmallnan     lvm2 [10.00 GiB / 10.00 GiB free]
  Total: 1 [10.00 GiB] / in use: 1 [10.00 GiB] / in no VG: 0 [0   ]
root@ip-172-31-19-80:/home/rancher# vgscan
  Reading all physical volumes.  This may take a while...
  Found volume group "niusmallnan" using metadata type lvm2