Efficient Polynomials For Sumcheck

Motivation
We currently store too many preprocessed polynomials in the proving key, like ID_1, ID_2, ID_3, and L_FIRST and L_LAST.
All of these can be efficiently evaluatable by the verifier, so they do not need to be committed. Moreover, it may be more efficient to derive their "edge extension" directly by exploiting their structure.

ID Polynomials
Start by noting that $\mathsf{ID}(X_0,\ldots,X_{d-1}) = X_0 + X_1 \cdot 2 + \cdots +  X_{d-1} \cdot 2^{d-1}$, and that $\mathsf{ID}_j(\vec{X}) = (j-1)n + \mathsf{ID}(\vec{X})$. (As part of indexing from 0, I suggest we relabel these with $j=0,1,2$ so that $\mathsf{ID}_j(\vec{X}) = j \cdot n + \mathsf{ID}(\vec{X})$).
The state required to define $\mathsf{ID}(X_0,\ldots,X_{d-1})$ is only $d$, and possibly $j$ in order to define $\mathsf{ID}j(X_0,\ldots,X{d-1})$ (since $n = 2^d$).
At round $\ell$ of Sumcheck, we will have had to partially evaluate it in $u_0,\ldots, u_{\ell-1}$ which are the verifier Sumcheck challenges. We define this partially evaluated polynomial as $\mathsf{ID}^{(\ell)}j( X\ell, \ldots, X_{d-1})$:
$$
\begin{align}
\mathsf{ID}^{(\ell)}j( X\ell, \ldots, X_{d-1})
&= \mathsf{ID}j(u_0,\ldots,u{\ell-1}, X_\ell, \ldots, X_{d-1}) \
&= \underbrace{\left( j \cdot n + \sum_{k=0}^{\ell-1} u_k \cdot 2^k \right)}{c\ell} + X_\ell \cdot 2^\ell + \left( \sum_{k=\ell+1}^{d-1} X_k \cdot 2^k \right) \
&= c_\ell + X_\ell \cdot 2^\ell + \left( \sum_{k=\ell+1}^{d-1} X_k \cdot 2^k \right)
\end{align}
$$
Going from $\mathsf{ID}^{(\ell)}j$ to $\mathsf{ID}^{(\ell+1)}j$ by evaluating $X\ell$ in $u\ell$ means we update the constant $c_{\ell+1} \gets c_\ell + u_\ell \cdot 2^\ell$ and increment $\ell$ to $\ell+1$. This constant is part of the polynomial's state, and we initialize $c_0 = j\cdot n$.
When we are iterating through the edge $(i_{\ell+1}, \ldots, i_{d-1}) \in {0,1}^{d-\ell-1}$, we let $i = i_{\ell+1} + i_{\ell+2}\cdot 2 + \cdots i_{d-1} \cdot 2^{d-\ell-2}$.
$$
\begin{align}
\mathsf{ID}^{(\ell)}j( X\ell, i_{\ell+1} \ldots, i_{d-1})
&= c_\ell + X_\ell \cdot 2^\ell + \left( \sum_{k=\ell+1}^{d-1} i_k \cdot 2^k \right) \
&= c_\ell + X_\ell \cdot 2^\ell + 2^{\ell+1}\cdot \left( \sum_{k=\ell+1}^{d-1} i_k \cdot 2^{k-(\ell+1)} \right) \
&= c_\ell + X_\ell \cdot 2^\ell + 2^{\ell+1}\cdot \left( \sum_{k=0}^{d-\ell-2} i_{\ell+1+k} \cdot 2^{k} \right) \
&= c_\ell + X_\ell \cdot 2^\ell + 2^{\ell+1}\cdot i \
\end{align}
$$
The first edge univariate is $U_0(X_\ell) := c_\ell + X_\ell \cdot 2^\ell$, and if we have the edge univariate $U_{i}(X_\ell)$, we obtain $U_{i+1}(X_\ell) = U_{i}(X_\ell) + 2^{\ell+1}$.
template <typename FF>
struct IDUnivariate {
	FF       constant{0};  // c_l
	FF       two_pow_l{1}; // 2^l

	size_t   l{0};         // Round number

	// Equal to ID_0
	IDUnivariate() = default;

	// Initialize to ID_j
	IDUnivariate(size_t d, size_t j)
	: constant{(1 << d) * j} {}

	// Partially evaluate in X_l = u_l
	void fold(const FF challenge /*u_l*/) {
		l++;
		constant += challenge * two_pow_l;
		two_pow_l.self_dbl();
	}

	// Returns the (not-extended) univariate edge for the edge at index i
	Univariate<FF, 2> current_edge(size_t i) const {
		// Compute (c_l + 2^l * (2i)) + 2^l * X_l
		return {constant + two_pow_l * i, two_pow_l};
	}
}

Note that in sumcheck_round.hpp, we have i equal to $2 \cdot i$ (the index is incremented by 2 at every loop), so we can simplify the computation of the edge.
We also realize that we don't need to store the number of variables $d$.
During the Sumcheck, both parties must run fold with the new challenge. After the last round, the veriifer can get the result of the evaluation by querying constant.

Implementation & Performance considerations
The main challenge with implementing this issue is not about the polynomials themselves, but rather which component of the proof system should be responsible for them.
For example, the $ID$ polynomial seems like it would only really be used by the GrandProductRelation, so it might make sense to let this component store the polynomial. At the moment, our relations are stateless though it's not clear why this needs to be the case. If we allow them to hold state, we could do away with the RelationParameters and instead initialize each relation with the data it needs.  For example, we could store the beta, gamma, public_input_delta directly in the GrandProductRelation, and it would also take care of initializing its own LAGRANGE_FIRST selector.
For the ID polynomials, we need to consider the upgrade to UltraPlonk where the IDs are actually committed as part of the verification key. We will also have to consider the case where we are combining it with the lookup argument.
Lagrange polynomials as "selectors"
As discussed in Mexico, there are overwhelming benefits to handling "selectors" separately from other "fixed" preprocessed polynomials. By "selector", we refer to polynomials whose evaluations over the hypercube are boolean, and stored in blocks (preferably whose sizes are powers of 2). In this case, the univariate edges for these selectors will be identically 0, which means that any thing they multiply does not need to be evaluated. A Lagrange polynomial is just a selector that

is efficiently evaluatable by the verifier
activates a block of size 1 = 2^0.

It may be useful to use std::optional (or some other way of flagging the 0 polynomial) that would indicate whether the edge actually needs extending, and avoiding any expensive multiplication.

The name ****Univariate really isn't great, I just copied it from when I had created PowUnivariate. It might make sense to call it ****MultiLinearExtension and add methods to return the full Polynomial for testing purposes. At the same time, it could unify the two implementations of pow.


JIT: Add type annotation in _compile.py

Blocked by #6844

LGTM!

/test mini

/test mini

@kmaehashi Failed to backport automatically.

Not cleanly cherry-picked

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?

PR #6837 should be also backported?

@kmaehashi This pull-request is marked as to-be-backported, but the corresponding backport PR could not be found. Could you check?


[new page] migrating from a prior version

Originally: https://github.com/influxdb/influxdb.com/issues/349
We need to duplicate the 0.7 to 0.8 upgrade page with a doc on how to migrate from 0.8 to 0.9. Link to the import/export README, don't re-write it.

What's the migration? Maybe using a Docker command I can run to do that?

@marcellodesales 0.8 to 0.9 is a highly manual migration. There is no way to fully automate it unless you have a trivial 0.8 schema.

Fixed by https://github.com/influxdata/docs.influxdata.com/pull/135. Duplicate of https://github.com/influxdata/docs.influxdata.com/issues/133


msvcrt_windows-x86.sig contains overly broad signatures

Found at least one eggregious function signature in msvcrt_windows-x86.sig that will cause it to match any 32bit x86 function with:

save frame pointer prologue
call of a function with a single 32bit argument pushed to the stack
standard epilogue/return

Here's a screenshot of assembly from a simple hello world app that matches:

Here's the relevant section of the signature library:

We likely need to increase the minimum size function that signatures are attempted to match against and re-generate the libraries. (Because this particular signature contains an epilogue it's likely possible to simply increase the minimum match size but this may have other effects)

I also experienced a couple of cases where this particular signature file was being over aggressive.

Also happens frequently with msvcrt_windows-x86_64.sig. https://dogbolt.org/?id=1701d225-02d4-4210-8f99-9d230c9e0418#Hex-Rays=185&BinaryNinja=54
In this case because of the similarities between c++ vfuncs and constructors they often will match against each other.


style breaks

styling break label doesnt animate after html is loaded from a template using AngularJS UI-Router.

Please send us a link with the problem so we can check. Also, let us know what browser/OS you use and we will reproduce it on the same environment.
Best,
Alex


getting a syntax error on 2.1.0 with python 2.7.12

django is failing to work - throwing a Syntax Error on
raise ValidationError([_(msg) for msg in [*warnings, *suggestions]], code=self.code, params={})
  File "/path/to/local/lib/python2.7/site-packages/django/urls/resolvers.py", line 364, in resolve
    for pattern in self.url_patterns:
  File "/path/to/local/lib/python2.7/site-packages/django/utils/functional.py", line 35, in __get__
    res = instance.__dict__[self.name] = self.func(instance)
  File "/path/to/local/lib/python2.7/site-packages/django/urls/resolvers.py", line 407, in url_patterns
    patterns = getattr(self.urlconf_module, "urlpatterns", self.urlconf_module)
  File "/path/to/local/lib/python2.7/site-packages/django/utils/functional.py", line 35, in __get__
    res = instance.__dict__[self.name] = self.func(instance)
  File "/path/to/local/lib/python2.7/site-packages/django/urls/resolvers.py", line 400, in urlconf_module
    return import_module(self.urlconf_name)
  File "/usr/lib/python2.7/importlib/__init__.py", line 37, in import_module
    __import__(name)
  File "./portal/urls.py", line 8, in <module>
    from users.views import logout_page
  File "./users/views.py", line 16, in <module>
    from users.forms import RegistrationForm
  File "./users/forms.py", line 4, in <module>
    from fields import PortalPasswordField, PortalPasswordConfirmationField
  File "./users/fields.py", line 1, in <module>
    from zxcvbn_password.fields import PasswordField, PasswordConfirmationField
  File "/path/to/local/lib/python2.7/site-packages/zxcvbn_password/__init__.py", line 13, in <module>
    from zxcvbn_password.fields import PasswordConfirmationField, PasswordField
  File "/path/to/local/lib/python2.7/site-packages/zxcvbn_password/fields.py", line 14, in <module>
    from zxcvbn_password.widgets import (
  File "/path/to/local/lib/python2.7/site-packages/zxcvbn_password/widgets.py", line 11, in <module>
    from .utils import zxcvbn_min_score
  File "/path/to/local/lib/python2.7/site-packages/zxcvbn_password/utils.py", line 3, in <module>
    from .validators import DEFAULT_MIN_SCORE
  File "/path/to/local/lib/python2.7/site-packages/zxcvbn_password/validators.py", line 60
    raise ValidationError([_(msg) for msg in [*warnings, *suggestions]], code=self.code, params={})
                                              ^
SyntaxError: invalid syntax

Hmmm, this is due to #96.
I don't want to personally support Python 2, for which official support ends in two weeks. If you want to use django-zxcvbn-password with Python 2, you can:

use version 2.0.3 (<2.1.0)
send PRs to fix the code for it to work on Python 2 and review future PRs to make sure they don't break the app for Python 2

Sorry for the inconvenient!


Error on Facebook Login

Nice repo.
I want to ask about this :
How to remove this error?

What should I add for domain?

Thanks

You need to add your domain/hostname to your facebook app settings.

I already add http://localhost:8080/ but error still show.
Any other advice?
Thanks

Are you using website as platform?

Yes, I am.


Catching instances of owl:Nothing

from owlapy.owl_reasoner import SyncReasoner
from owlapy.class_expression import OWLClass
from owlapy.owl_individual import OWLNamedIndividual
from owlapy.iri import IRI

ontology_path = "KGs/Family/father.owl"
hermit  = SyncReasoner(ontology=ontology_path, reasoner = "HermiT")

thing=OWLClass(IRI('http://www.w3.org/2002/07/owl#', 'Thing'))
nothing=OWLClass(IRI('http://www.w3.org/2002/07/owl#', 'Nothing'))
print(nothing.iri.as_str())
print(hermit.instances(nothing))

Throws the following error
Traceback (most recent call last):
File "ManchesterOWLSyntaxClassExpressionParser.java", line 53, in org.semanticweb.owlapi.manchestersyntax.parser.ManchesterOWLSyntaxClassExpressionParser.parse
Exception: Java Exception
The above exception was the direct cause of the following exception:
Traceback (most recent call last):
  File "/home/cdemir/Desktop/Softwares/owlapy/temp.py", line 12, in <module>
    print(hermit.instances(nothing))
  File "/home/cdemir/Desktop/Softwares/owlapy/owlapy/owl_reasoner.py", line 1233, in instances
    mapped_ce=self.mapper.map_(ce)
  File "/home/cdemir/anaconda3/envs/temp_owlapy/lib/python3.10/functools.py", line 926, in _method
    return method.__get__(obj, cls)(*args, **kwargs)
  File "/home/cdemir/Desktop/Softwares/owlapy/owlapy/owlapi_mapper.py", line 154, in _
    return self.parser.parse(owl_expression_to_manchester(e))
org.semanticweb.owlapi.manchestersyntax.renderer.org.semanticweb.owlapi.manchestersyntax.renderer.ParserException: org.semanticweb.owlapi.manchestersyntax.renderer.ParserException: Encountered Nothing at line 1 column 1. Expected one of:
	Class name
	Object property name
	Data property name
	inverse
	not
	(
	{

Using nothing doesn't seem to work
# hasChild a nothing.
assert hermit.instances(OWLObjectSomeValuesFrom(property=hasChild, filler=nothing))==eval("{OWLNamedIndividual(IRI('http://example.com/father#', 'michelle')), OWLNamedIndividual(IRI('http://example.com/father#', 'heinz')))}")

This got fixed. There was a Manchester rendering & parsing issue for owl:Nothing.
Btw you can use OWLNothing and OWLThing instead of declaring them from scratch.
from owlapy.class_expression import OWLNothing, OWLThing

Great thank you!

hermit.instances(OWLNothing.get_object_complement_of()) doesnt seem to work :(

This is now fixed fixed! PR: #73


Document that tests and tested assembly must be separate (no single assemblies)

Document that tests and tested assembly must be separate (no single assemblies)

This has been added to the README


float()-ing a numeral string returns a factually wrong value.

Godot version:
v3.2.2.stable.official
OS/device including version:
Xubuntu 18.4
Issue description:
I wanted to turn a string containing numbers into a float
Steps to reproduce:
Literally write:
print(float("12314.123123")). it can be any number
Note the second number being factually wrong

Minimal reproduction project:
Not worth the effort as it can be reproduced in 20 characters

I think this is expected, as the numbers you've specified can't accurately be represented in a 32-bit floating-point number.

defining REAL_T_IS_DOUBLE will give more precision, but why godot is going with 32-bit floats?
// math_defs.h
#ifdef REAL_T_IS_DOUBLE
typedef double real_t;
#else
typedef float real_t;
#endif

32-bit float : 122.54 =<PHONE_NUMBER>1552734375
64-bit float : 122.54 =<PHONE_NUMBER>00000006252776074689

@ThakeeNathees 32-bit floats are faster compared to 64-bit floats. There are plans to fix REAL_T_IS_DOUBLE so it can reliably be used as a build-time option in 4.0, but it won't be the default option for performance reasons (especially on mobile/Web platforms).

This isn't related to REAL_T_IS_DOUBLE because this isn't about vectors or physics. Variant floats and ints are supposed to be always 64-bit. This wasn't the case for strings, so I fixed it here: #40678.


Versioning strategies

Quoting from https://github.com/cdevents/spec/issues/40 (thank you @xibz):
How do you version and support interoperability between all the possibilities?
For instance, some service A could be on version 1, and some service B could be on version 2. Are major versions always going to be backwards compatible, so sending requests between these services should always work?
Further, types have to have a version as well. The com.github.pull_request.opened type may have a completely different payload years later. That adds a whole layer of complexity... In addition, adding a new type to the spec would have to be a major version bump of the spec, since anyone using version 1.x and 1.y adds a new type, then 1.x will have no idea on how to deal with that.
How does major versioning look like with interoperability? Does EVERY service need to update to the new major version? How do we plan on handling that???

We need to have an initial statement on this topic for v0.1 so that it's clear that "v0.1" means

This is probably a duplicate of #32 :

Follow up to the discussion in https://github.com/cdevents/spec/pull/30#discussion_r817818324
We should clarify whether to we want to version events individually or not and what does that mean in terms of compliance to the spec for producers and consumers.

Since this one is included in the roadmap already I would close #32 in favour of this.

Quoting @m-linner-ericsson comment in #32 so we don't miss it
https://github.com/cdevents/spec/issues/32#issuecomment-1059115457

We might want to include something like following text as requirement for the version.

MUST be the version of the event/spec used for producing the event

When it comes to versioning of the events themselves please also note https://github.com/cloudevents/spec/blob/main/cloudevents/primer.md#versioning-of-cloudevents

We need at least a basic versioning for the spec. We have https://cdevents.dev/docs/spec/#version for now.
We need a statement about when we need a new version of the spec. This is also needed for the SDK as a reference.

The suggested version for the event, is that intended to match the complete protocol version or is it a specific version for the event type itself? I.e. for CDEvents v0.1 do we intend to give dev.cdevents.artifact.packaged.v0.1 or still dev.cdevents.artifact.packaged.v1, where "v1" represents some specific event type version?
When it comes to whether the version should be part of the type property or not:
When filtering on events it would be easier if there was a separate version field instead of having it as part of the event type. Then you could use filters like:
version = 1.0.1
version > 1.0.0
select all version | uniq

If it should be part of the type, should it be the whole semantic version of the protocol (or individual event type) or should it just be part of it, i.e,:
type = dev.cdevents.artifact.packaged.v0
or type = dev.cdevents.artifact.packaged.v0.1
or type = dev.cdevents.artifact.packaged.v0.1.2

The suggested version for the event, is that intended to match the complete protocol version or is it a specific version for the event type itself? I.e. for CDEvents v0.1 do we intend to give dev.cdevents.artifact.packaged.v0.1 or still dev.cdevents.artifact.packaged.v1, where "v1" represents some specific event type version?

It is not intended to match the version of the spec.
If it did, we could simply have the spec version and would not need a version on the events.
If we do use event specific versions, I think that we could have the following situation:

spec v0.1 (initial state)

eventA.v1
eventB.v1
eventC.V1

spec v0.2, eventC in changed, eventD is added

eventA.v1
eventB.v1
eventC.V2
eventD.v1

spec v0.3, format of the context is changed, eventE is added

eventA.v2
eventB.v2
eventC.V3
eventD.v2
eventE.v1

A consequence of this approach is more granularity in versions i.e. if a new spec version introduces no changes in the subset of events that are used by a tool, that tool is not forced to update the SDK to work with producers/consumers that adopted the new spec version.
An alternative could be to have stage level versions, so that for instance new incident events are added, that has no impact on tools that do not deal if incidents at all, but if a change to any of the event in that stage is made, the entire group of events is updated to a new version. This would reduce the number of versions to maintain.

When it comes to whether the version should be part of the type property or not: When filtering on events it would be easier if there was a separate version field instead of having it as part of the event type. Then you could use filters like:
version = 1.0.1
version > 1.0.0
select all version | uniq

I think there are pros and cons in both cases.
Having all in one field allow for filtering based a one field, but it also require the filter engine to support regex/startWith or use multiple rules to accept multiple versions of the same event.

If it should be part of the type, should it be the whole semantic version of the protocol (or individual event type) or should it just be part of it, i.e,:
type = dev.cdevents.artifact.packaged.v0
or type = dev.cdevents.artifact.packaged.v0.1
or type = dev.cdevents.artifact.packaged.v0.1.2

If we wanted to use semantic versioning for event versions, what would major and minor versions mean?
We could use minor version for backward compatible changes (optional fields added), and major versions for backward incompatible changes, like fields removed, renamed, new mandatory fields added or structural changes in the context or subject.

Ok. Then I get it. Yes, I'm pro having individual event versions. One reason being the one you mention about being able to notify what events within a new spec version that are actually changed, to make the life easier for the consumers.
I believe we should have semantic versions for the individual events. You state it very well, that a non-backwards compatible change would step the major version and that could be removed or renamed properties.
And the spec version will then more or less just be a grouping of individual event versions, right? What would major.minor.patch on the spec reflect then? Major for removed or renamed events for example? And minor for added events?
If we would have "stage level version" those would behave in a similar way I believe, but I don't think we need to define "stage level" (or "bucket specific"?) versions just yet.
We already have the "version" field in the spec: https://cdevents.dev/docs/spec/#version. The documentation currently says that that is the spec version, but that should probably reflect the individual event version instead then? Otherwise when we step the spec version without changing an individual event, that event's schema won't be correct anymore (if the schema has that field hard coded).

Ok. Then I get it. Yes, I'm pro having individual event versions. One reason being the one you mention about being able to notify what events within a new spec version that are actually changed, to make the life easier for the consumers.
I believe we should have semantic versions for the individual events. You state it very well, that a non-backwards compatible change would step the major version and that could be removed or renamed properties.
💯

And the spec version will then more or less just be a grouping of individual event versions, right? What would major.minor.patch on the spec reflect then? Major for removed or renamed events for example? And minor for added events?

👍

If we would have "stage level version" those would behave in a similar way I believe, but I don't think we need to define "stage level" (or "bucket specific"?) versions just yet.

👍

We already have the "version" field in the spec: https://cdevents.dev/docs/spec/#version. The documentation currently says that that is the spec version, but that should probably reflect the individual event version instead then? Otherwise when we step the spec version without changing an individual event, that event's schema won't be correct anymore (if the schema has that field hard coded).

I think we should still have the spec version in the context.
When tag release v0.1, we can set the spec version to v0.2-wip on main, so that any WIP change to event will temporarily belong to that version. As part of the v0.2 release, we can change the release version in the schema to v0.2.
For events that do not change across releases, we could use the spec version enum to list all valid spec version, and use the latest one as default.
From the example above, in spec v0.2, eventB.v1 would have [v0.1, v0.2] in the version enum, with v0.2 as default.
In spec v0.3, eventB.v2 would have [v0.3] in the version enum.

Adding a few comments from the somewhat related PR (#61) about json schemas, to not miss these here:

We should consider hand-editing the schemas, or scripting some adjustments of them, for easier read. For example the "type" field should be listed as the first field within each object for easier read.
The same way as done on the version field, we could hard code the context.type to its explicit event type within the schema

From the working group:

Proposed starting approach:
   - Version in the context for the spec, free text
   - Semantic version for events in the event type string

https://github.com/cdevents/spec/pull/67


⚠ Bofrost.at has degraded performance

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

HTTP code: 200
Response time: 1021 ms

Resolved: Bofrost.at performance has improved in fbb7547.


Fix deadlink

Purpose of this pull request
refer to https://github.com/apache/seatunnel/issues/6279

Does this PR introduce any user-facing change?

How was this patch tested?

Check list

[ ] If any new Jar binary package adding in your PR, please add License Notice according
New License Guide
[ ] If necessary, please update the documentation to describe the new feature. https://github.com/apache/seatunnel/tree/dev/docs
[ ] If you are contributing the connector code, please check that the following files are updated:

Update change log that in connector document. For more details you can refer to connector-v2
Update plugin-mapping.properties and add new connector information in it
Update the pom file of seatunnel-dist

[ ] Update the release-note.

https://github.com/debezium/debezium.github.io/commit/a8542d1508c836332e26aa2144f99e7ea2867c81
1.6 has been hidden.

So sorry, just found this. I created a PR fixed this.


New metric: Share of Voice

Adds a metric based on past experience as a community manager
Design is based on current metric template

Signed-off-by: Matthew Broberg<EMAIL_ADDRESS>

I am not sure this is metric. This feels more like describing methods of comparison.

Feedback from @germonprez during Value WG:

"It feels like a metric to me." Bar chart is giving transparency. (@GeorgLink making a point that it's a composite metric rather than raw data.)
Would like more notes on "why do we even need this metric?"
Before I invest in a tool, I may want to look into its growth or shrinking share of voice to see if I want to invest in it.
To assess my own project, I want to see that we are reaching in adoption.

Add these notes to the objective.

I didn't see the bar chart when I first looked at it. Ratio of conversation does seem like it could be a metric however it wasn't clear what "value" we were comparing from the introductory text or if ratio of conversation was just one example of the value we could compare. What is the difference between ratio of conversation and shared voice? Are they the same thing?

Closing until I update this metric based on new data and structures.


Values aren't retained with WizardForm when destroyOnUnmount and initialValues are used in conjunction

If I set initialValues on a form that uses destroyOnUnmount, when I return to that form the values are reset.

This is a duplicate of https://github.com/erikras/redux-form/issues/1503

Closing for dupe reason.


highlight abstracts for terms

e.g. genes, species, ages (young adult)

Done in commit https://github.com/mcdougallab/pipeline/commit/feea5805eed64d69c1ca0a364d8998e1664f8c3f


accounts.deploy : error when the gas limit is not set

brownie Version: v1.0.0b5
solc Version: 0.5.7
Python Version: 3.6.3
OS: linux

What was wrong?
I encountered a bug when deploying a contract when gas limits are not configured.
$ brownie console

Using solc version v0.5.7
Brownie v1.0.0b5 - Python development framework for Ethereum

Using solc version v0.5.7
Brownie environment is ready.

>>> a[0].deploy(SafeMath)

  File "brownie/cli/console.py", line 112, in _run
    exec(cmd, self.__dict__, local_)
  File "<string>", line 1, in <module>
  File "brownie/network/account.py", line 215, in deploy
    txid = _raise_or_return_tx(e)
  File "brownie/network/account.py", line 332, in _raise_or_return_tx
    data = eval(str(exc))
SyntaxError: invalid syntax (<string>, line 1)

# I think the bug come from here :
# brownie/network/account.py:deploy:

try:
     txid = self._transact({
     'from': self.address,
     'value': wei(amount),
     'gasPrice': wei(gas_price) or self._gas_price(),

     # This line, and specifically the self._gas_limit function call
     'gas': wei(gas_limit) or self._gas_limit(amount, data),
     'data': HexBytes(data)
})
except ValueError as e:
      txid = _raise_or_return_tx(e)

I think ths arguments are out of place. The function _gas_limit is called with (amount, data) arguments, but _gas_limit function is defined by _gas_limit(self, to, amount, data=""). So, 'amount' refers to 'to', and 'data' refers to 'amount'.
I solved my problem by manually configuring gas_limit to avoid the _gas_limit function and thus bypassing the crashing branch :
a[0].deploy(SafeMath, gas_limit=500000)
It works, but ofc the main problem is still here.

Sorry...  this is a result of me moving too quickly and breaking things.  You're right about the cause of the issue.  I have a fix implemented in a branch I'm about to push, and I'll try to get another release out before the end of the day.
Version 1.0.0b5 was all about code refactoring prior to writing unit tests. My next focus is writing those tests so that this doesn't happen anymore :)

Fix has been merged and a new version pushed to Pypi


Add codecov action to tests workflow

Codecov Report

:exclamation: No coverage uploaded for pull request base (main@c40ba73). Click here to learn what that means.
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff            @@
##             main       #10   +/-   ##
========================================
  Coverage        ?   100.00%
========================================
  Files           ?        25
  Lines           ?       810
  Branches        ?         0
========================================
  Hits            ?       810
  Misses          ?         0
  Partials        ?         0

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Error while compiling

Please provide a detailed description of your question or the information you seek.
After I did git clone, I tried to compile. I type this command：./build/astra_ns3/build.sh -c . Then it prompt me with this error below: GOOGLE_LONGLONG was not declare. And I failed to compile.

I recommend pulling the latest version of the repository and trying to compile again. I recently compiled it without encountering any issues.


Should Distribution::incorporate() take a weight?

(This is a discussion issue rather than an action issue!
The weight would have a default of 1.0, so that all present users wouldn't have to change anything.
But this would be useful in at least three different ways:

Sometimes we get fractional evidence that a data point belongs to a distribution, and this lets us update the distribution on that.  For example, in the SimpleStringEmission class, we track the probabilities of insertions, deletions and substitutions via BetaBernoulli models.  But we can only guess at those given our actual inputs, which are clean and dirty strings -- is (clean="hello", dirty="hellp") a substitution or an insertion plus a deletion?  The "right" thing to do would be to assign probabilities to both of those possibilities, and to call the corresponding BetaBernoulli with fractional weights.

Sometimes we get bulk evidence, and it is slightly inefficient to have to call .incorporate() in for loop rather than with an integer weight > 1.  To use SimpleStringEmission as an example again, (clean="hello", dirty="") is evidence of five deletions, and it would be nice to just call deletion.incorporate(true, 5.0);

We could implement .unincorporate(x) in base.hh::Distribution as incorporate(x, -1.0) and then child Distributions wouldn't have to worry about it.

The biggest downside is

It might be hard or impossible to implement this for some distributions.  I don't think this is true for any of existing distributions, or any others that I can think of off the top of my head, but please correct me if I'm wrong!

This makes sense to me!

Implemented in https://github.com/probcomp/hierarchical-irm/pull/76
Another use for this just came up:  when generalizing the Sometimes Emission adapter to handle BaseEmissors that can output dirty == clean with non-zero probability.  When incorporating a (clean, clean) pair in such a situation, the right thing to do turns out to be to pass fractional weights to the incorporate methods of the contained BetaBernoulli and BaseEmissor classes.


Create Acknowledgements page

Let's create a page where we clearly indicate who contributed to statcheck.

email contributors to ask permission


Rename fleet summary component

@Shavindra Rename FleetSummary component to ResourcesSummary

@aalves08 Think this would have been a good candidate to git fetch upstream && git reset --soft upstream/master and then recreate the commit. There's a lot of merges in here instead of rebases


Remove legacy routing code

Fixes initialization issues in the latest chrome release.
More details in slack conversation
@himdel @ZitaNemeckova commit 185e202bd2a45e2754b2e55c615bf03ca256fc5a is the one that fixes the issue and should be cherry-picked all the way to prod-stable build. The other one is just some clean-up.

(cypress failure unrelated, pr-checks is failing on src/loaders/insights/Routes.js vs prettier)

Cc @ZitaNemeckova any chance your insights mode is working?
I think this PR is ready, would approve and merge :+1: ... but when testing, qa.foo, ci.foo and stage.foo come completely blank, while prod.foo has the menu and top nav while only the automation hub parts are blank :) .. and I get the same for master as well.

@himdel Insights mode gives me 403 on every page :/

Yeah, agreed, we'll need to fix the local setup, but not blocking this PR for it :) We can test on stage.


TabContent

Hi,
This is only a question/suggestion. In TabContent source code I did not find any option to disable rendering when content is not visible. In my app I have quite a lot of tabs and every content gets interpreted event if if doesn't need to. I don't need any animations when switching tabs either.
I would suggest to introduce new prop called forceRenderTabPanel (on Tabs component) with default value of true to keep the backward compatibility  just like in https://github.com/reactjs/react-tabs. Do you think it is a good idea for this project ? If yes I would try to provide pull request for this feature.
Thanks

Actually that's a good idea!
I would only name the prop more like renderActiveTabContentOnly and default it to false (to not break v1 code). But I think defaulting to true probably makes more sense so we can do a v2 release (with the changed default) after shipping it to v1.
I'll be waiting for the PR if you are interested!

Ok deal. I'll provide pull request myself in a couple of days

Hey @char0n, have you made any progress on this?

Ah, It has been already 21 days. I am sorry. I have been caught up in my job a little bit. I will try to produce pull request tomorow. I have some time in the evening.

No worries, just wanted to know what's the status, if you still want to do it. I'll leave this open and will wait for the PR then.

Fixed by https://github.com/patrik-piskay/react-tabs-redux/pull/8


Support running multiple scripts, both in parallel or sequential

I don't know if this is out-of-scope for dum, but I find that it could be a perfect (and faster) alternative for npm-run-all (that, at this point, I don't even think it's maintained anymore).
I'd love it if dum could run multiple scripts in one command, e.g. dum clean build, and, with a flag, run them in parallel, so something like dum --parallel build:ts build:dts.
Another impressive feature of npm-run-all is its glob-like script-name resolution (e.g. build:*). It would be chef's kiss if this could be implemented and translated to "run every script that matches this pattern in parallel." This would make it very easy to have one script, dum clean build:*, that cleans the output directory, and, after that, runs both build:ts and build:dts in parallel.

I would love to implement this feature, probably through a syntax like dum  "foo bar", without quotes bar will be passed to the foo script as cli args.


AngularDamping seems unable to stop a rotating Sphere

As title, it seems that a RigidBody with only a ShapeSphere will never stop spinning due to only AngularDamping. The damping is working, in that it slows fast-spinning spheres, but only to a certain point. This is difficult to explain, but you can see the effects by placing some spheres with 0 friction on to a large surface with also 0 friction and moving them. They will move until they gradually stop due to VelocityDamping, but they will never stop spinning. In fact, they always seem to settle on specific spinning speed unrelated to what you actually have AngularDamping set to.
This issue can also be seen when you do have friction on, where spheres will instead roll forever at the same constant speed. Am I missing something, or is this a bug / not yet implemented feature?

Thanks a lot. I will check this.

In the version 0.6.0 of ReactPhysics3D, support of rolling resistance has been added. It can be used to stop rolling shapes on a flat surface.


this is a test - no changes

FYI You don't need to open a PR. I believe it is sufficient to open a branch to run CI

Does that mean you don't want us opening DNM PRs? (I find PR simpler because it links to the CI runs)

Does that mean you don't want us opening DNM PRs? (I find PR simpler because it links to the CI runs)

I don't really care. Just wanted to point out this isn't necessary. I agree, it's easier to track (and ideally a bot would just post the results to the PR)


Update README.md

little

I'm seeing some design flaw here.

Remove some - and add a # will do:
### Training Without UI/Gradio, Prevent Banning
#### Download Traning Assets
- [![Open In Colab](https://colab.research.google.com/assets/colab-badge.svg)](https://colab.research.google.com/github/ardha27/AI-Song-Cover-RVC/blob/main/Download_Training_Assets.ipynb)
#### Training Without UI/Gradio, Prevent Banning
- [![Open In Colab](https://colab.research.google.com/assets/colab-badge.svg)](https://colab.research.google.com/github/ardha27/AI-Song-Cover-RVC/blob/main/TrainingV2_NoUI.ipynb)

Preview:
Training Without UI/Gradio, Prevent Banning
Download Traning Assets

Training Without UI/Gradio, Prevent Banning


[install]
tidelift: rubygems/nokogiri

Have you read and followed the installation tutorial at http://www.nokogiri.org/tutorials/installing_nokogiri.html?

[< ] Yes!

.......
What is the complete output of gem install or bundle install?

the output goes here

If installation completed but is broken, what is the complete output from nokogiri -v?

the output of "nokogiri -v" goes here

If installation failed during compilation, what are the complete contents of the mkmf.log file generated during the failed installation?

the mkmf.log file contents go here

Tell us about your system!
What is the output from ruby -v?
What is the output from gem -v?
What is the output from gem env?
the output of "gem env" output goes here

If you're using Bundler:

what is the output from bundle version?
what is the output from bundle config? (Take care to redact any credentials)

the output of "bundle config" goes here

If you're on MacOS, please note:

the version of XCode you have installed (if you know)
the output of gcc -v or clang -v

If Linux or a BSD variant, please note:

the distro you're using
the output of uname -a
the contents of /etc/lsb-release.

If Windows, please note:

whether you're installing the precompiled gems, or compiling yourself with DevKit
the version of RubyInstaller you've installed
or if you're not using RubyInstaller, how did you install Ruby?

@Bigcannin87 If you need help, please fill out the template with the requested information.


Spark lineage not recorded in DataHub from AWS Glue

Describe the bug
I am evaluating DataHub as a data catalog and lineage tool for the Data Platform and using this article - https://aws.amazon.com/blogs/big-data/part-2-deploy-datahub-using-aws-managed-services-and-ingest-metadata-from-aws-glue-and-amazon-redshift/ as an example. After running Glue job, I can see Spark DataPipeline and DataTask created in DataHub however the lineage doesn't exist. Gms logs look ok without errors.
Identical issue has been raised previously https://github.com/datahub-project/datahub/issues/8997 but then closed without any comments.
Another related issue closed without resolution or explanation - https://github.com/datahub-project/datahub/issues/5724
To Reproduce
Steps to reproduce the behavior:

Follow instructions from Capture data lineage in the web-page mentioned above
Run glue job
Open datahub UI and go to Platform->Spark.
Go to DataTask created after the glue job run and open Lineage section

Expected behavior
Can see upstream and downstream components in the lineage for the DataTask
Screenshots
If applicable, add screenshots to help explain your problem.

Desktop (please complete the following information):

OS: [e.g. iOS]
Browser [e.g. chrome, safari]
Version [e.g. 22]

Additional context
Add any other context about the problem here.

Is there any update on this issue?

any update on this issue ?

This still hasn't been answered so can't really close it

@denystyshetskyy @vinothdataeng Our latest Spark plugin (0.2.8 or 0.2.9) supports Glue, and please give it a try:

On Glue please set:
The spark.datahub.stage_metadata_coalescing config parameter and Glue don't send an explicit application end event.
You can check other config parameters here:

Hi @treff7es for your response.
Can you please clarify what do you mean by latest spark plugin (0.2.8 or 0.2.9)? For the integration with datahub I am using datahub-spark-lineage.jar which is in the version 0.13+
Do you know of any relevant documentation you can share for this use case?

@denystyshetskyy:

Spark plugin, by default, only emits the upstream lineage edge and not the datasets and the schema. If you want the plugin. Ideally, you should capture lineage with the specific DataHub source (for example using MySql source to capture datasets and schema). If you want to use the spark lineage plugin you can enable it with the following config parameters:

--conf "spark.datahub.metadata.dataset.materialize=true" to materialize datasets and --conf "spark.datahub.metadata.dataset.experimental_include_schema_metadata=true" to capture schema from the Spark Plugin

To capture glue tables as glue and not as hive you should use set the config property:
--conf "spark.datahub.metadata.dataset.hivePlatformAlias=glue"


Fix for: Pyjks doesn't convert alias to lowercase (#38)

Alias will now be converted to lowercase when creating certificate/private key entries. Added some tests also.

Seems like a good idea to keep compatibility with the primary JKS tools. I'm going to merge under the assumption that if this breaks any current pyjks users they'll come in here and enlighten us as to the format's true nature :)


CODE RUB: AIFiles Exception Classes Update v2.10.1

Updated All Exceptions associated to the AIFiles directory to adhere to Anemic Data Models as well as updated the service layer to include the exception messaging.

Hello @glhays Greg,
Thank you so much for the information as it is much appreciated. I will follow your advice and will study up on the best practices and guidelines to ensure I do not add unnecessary stress or work to the team. I will make sure I update the titles of my other pull requests after I reed the guidelines. Thanks again and I look forward to working with you.

@Lezune are you on the discord channel?

@glhays I am on te discord channel. I changed my username to Lezune to make it easier to spot me. Also, thank you for the feedback.

If my comments are fixed, please mark the comment resolved in the git IDE.
On Thu, May 30, 2024, 9:13 AM Lezune @.***> wrote:

@.**** commented on this pull request.
In
Standard.AI.OpenAI/Models/Services/Foundations/AIFiles/Exceptions/FailedAIFileServiceException.cs
https://github.com/hassanhabib/Standard.AI.OpenAI/pull/436#discussion_r1621034924
:

@@ -9,11 +9,6 @@ namespace Standard.AI.OpenAI.Models.Services.Foundations.AIFiles.Exceptions
{
public class FailedAIFileServiceException : Xeption
{

   public FailedAIFileServiceException(Exception innerException)

       : base(

           message: "Failed AI file service error occurred, contact support.",

               innerException: innerException)

   { }

This has been completed. Sorry for the wait.
—
Reply to this email directly, view it on GitHub
https://github.com/hassanhabib/Standard.AI.OpenAI/pull/436#discussion_r1621034924,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AKD4KOYQZ2ILD26OCCVAWJTZE5F3NAVCNFSM6AAAAABINXEGYOVHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMZDAOBYG43DSNBUHA
.
You are receiving this because you were mentioned.Message ID:
@.***>

Thank you for all of the help. All conversations have been marked as resolved.

Updated and refactored code to reduce redundancy by adding methods in the AIFilesService class and calling the methods in the AIFilesService.Exception and AIFilesService.Validation classes as suggested by @glhays.
Please ignore AIModel classes as they should be processes in the following Pull Requests:
#507
#508
#509
#510
#511
#512
#513
#514
#515
#516
#517

@Lezune I suggest that you do a similar merge with this PR, your not in an appropriate branch like
user/lezune/coderub-aifiles-upgarde-v2101 but your going from from your forks main to main.
Don't worry about the comments you can always open the closed PR to review if needed.

Thank you for the advice, @glhays. I will follow your suggestion and merge this into a new branch later today.

Closing Pull Request. Resolved in PR #558.


Hard to Identify what is Clickable on the Badge

When clicking on the icon, it is hard to identify what is clickable on the badge since the default cursor pointer is the same.
A change of the cursor would make it more intuitive and interactive .
I suggest adding this for better UX
span.t-Badge-icon.margin-left-sm {
    cursor: pointer;
}

Great idea; I appreciate the feedback. I added a new class, cn-info-badge-icon, and added the cursor: pointer to that class. This is updated in the demo app. I will release a new version of the Plugin with this change in a few days. Thank you again for the great suggestion.


Zoom Camera in on Game Over Screen

Describe your problem here.
I'd like to zoom in on the Players Death Animation when the player dies, but I can't seem to get it correct. This is what it looks like right now, the default stage zoom is 0.4, and no, I do not want to change the zoom setting, I set it to 0.4 because the stage would need that space to show everything on the screen. I don't know if there's a way to zoom in on death using .Lua. So if anyone could help it's appreciated!

Are you modding a build from source or with Lua?
Lua
What is your build target?
Windows x64
Did you edit anything in this build? If so, mention or summarize your changes.
No.

function onUpdate()
     if inGameOver then
     setProperty('defaultCamZoom', 0.7)
     end
end

This Seems Like It Would Work, I Can't Test Though

function onGameOverStart()
     setPropertyFromClass('flixel.FlxG', 'camera.zoom', value)
end


Add modal component

Changes:

Adds a modal component.
Adds a modal container component that wraps the modal component.
Adds tests for modal.
Creates a new module for utility functions.
Adds a function that adds or removes classnames from a classname string.
Adds tests for toggle classname function.

I have:

[x] Set target branch to the correct branch, usually dev.
[x] Ensured code is formatted, linted.
[x] Cleaned up git history.
[x] Ensured commit messages describe what was changed, and why.
[x] Ran tests locally, checked output.

Requires more testing:

It may cause issues with components that have their own callbacks. This is because modals typically don't exist on the page before they're shown and dash callbacks don't like that.
Exporting the module in components/__init__.py causes errors on pages that do not have any modals, once again due to the aforementioned requirements of callbacks.

Potential improvements:

Instead of letting the container create the modal it could instead be passed in. This may also solve some undefined behavior when creating multiple modals on the same page.


Running in non-root repo and specifying path arguments fails

This is because I use a function withCurrentDirectory without updating the path arguments.
#32 will fix this but there needs a patch for the current releases.

Fixed in v<IP_ADDRESS>, v<IP_ADDRESS> and v<IP_ADDRESS>.


How to add commit trailer based on commit type selected

Hello. I use Gitlab built-in feature (glab) to generate Changelogs for my projects based on Commit trailer messages as described here:
https://docs.gitlab.com/ee/user/project/changelogs.html
In order for glab to detect commits, each commit needs to have a commit trailer.  For example if I make typo when writing a commit trailer, the glab will not detect the commits.
I was hoping to use better-commits to automate this process and ensure each commit is tagged with appropriate commit trailer based on selected commit type.
For example, I have a list of commit types:
    "commit_type": {
        "enable": true,
        "initial_value": "feat",
        "infer_type_from_branch": true,
        "append_emoji_to_label": false,
        "append_emoji_to_commit": false,
        "options": [
            {
                "value": "feat",
                "label": "feat",
                "hint": "A new feature",
                "emoji": "✨"
            },
            {
                "value": "fix",
                "label": "fix",
                "hint": "A bug fix",
                "emoji": "🐛"
            },
            {
                "value": "docs",
                "label": "docs",
                "hint": "Documentation only changes",
                "emoji": "📚"
            },
            {
                "value": "test",
                "label": "test",
                "hint": "Adding missing tests or correcting existing tests",
                "emoji": "🚨"
            },
            {
                "value": "ci",
                "label": "ci",
                "hint": "Changes to our CI configuration files and scripts",
                "emoji": "🤖"
            },
            {
                "value": "",
                "label": "none"
            }
        ]
    },

Is there a possibility to append a trailer to commit message based on selected commit type? For example, if I select feat , I want to automatically append Changelog: added as my Commit trailer.
If I select fix, I want to append Changelog: fixed and etc...
Thanks in advance!

Hey thanks for writing up this issue! - I've seen trailers on Github PRs like "Co-Authored by X" but had no idea they were their own flag on git commit, pretty cool!
Yeah I think this makes sense and is easy to add to the project.


Adding imagerunner services

Proposed changes
Add imagerunner service container feature. This is similar to gitlab-ci service container, lets customers run side containers with their tests.
Types of changes

[ ] Bugfix (non-breaking change which fixes an issue)
[x] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] Documentation Update (if none of the other choices apply)

Checklist

[x] Lint and unit tests pass locally with my changes
[x] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[x] I have updated the json schema (if appropriate)

Further comments
This is for devx to take ownership, feel free to change the saucectl input.

Looks like a bug in backend mounting files on service container, let me fix first.

Backebd bug is fixed.


Build would not run - gevent imports - gevent.__hub_local.py - pyinstaller

Hi -
I recently cloned this repo and attempted to build as-is as I have a Flask app that I'd like to operate as a standalone app. Here are the issues I ran into.

Could not run __main__.py due to import errors
Once built, the *.exe would report import errors relating to gevent. (could not import gevent.__hub_local.py, gevent.__hub_primitives, etc)
cefpython would report errors that some data files could not be found.

Here are the fixes

Changed from gevent.wsgi import WSGIServer to from gevent.pywsgi import WSGIServer
Recent versions of gevent cause problems with pyinstaller. I specified the version 1.2.2 in requirements and this eliminated this problem. I'm sure there is another way to use > 1.2.2 but I do not understand how to implement compiled modules with pyinstaller.
This is something with trying to find files with frozen installs. I changed the following section in def main(): in __main__.py:

def main():
    check_versions()
    sys.excepthook = cef.ExceptHook  # To shutdown all CEF processes on error
    settings = {}
    if getattr(sys, 'frozen', False):
        settings = {
            'resources_dir_path': sys._MEIPASS,
            'locales_dir_path': sys._MEIPASS + os.sep + 'locales',
            'browser_subprocess_path': sys._MEIPASS + os.sep + 'subprocess.exe',
        }

Once I applied this fixes, I was able to build and run the exe file without issue. I was also able to edit the build.spec file to build into a single file.
I only use git for personal use as I'm completely an amateur so I'm not sure how to issue a pull request - if that's the correct thing to do.
This repo helped me accomplish what I needed - thanks!

Hey @jonbluhm I'm glad this project helped!
v2.0.1 should fix the import errors 1 and 2 you had.
Let me know if no. 3 is still an issue for you. I do not have that issue.


build: Use Node 20 as default for github actions workflows

On Jenkins with manually modified Jenkinsfile: https://studio-jenkins-csb-codeready.apps.ocp-c1.prod.psi.redhat.com/job/Fuse/job/VSCode/job/vscode-camel-dap-release/645/


Made "currentDay" glow based on which bar is lit

Description
This is a continuation of PR #337 where I added in a slight glow to the "currentDay" as suggested by @bporch84 by added some styles to the .currentDay class as listed on this Trello card to emphasize where the child user is along their mission/journey.

 add "currentDay" glow

Great Job! Perhaps, we can keep both the progress div & day div wrapped in flex div. Then have a vertical progress bar that lines up with each day instead.


Distinguish OpenSearch vs OpenSearchDashboards manifest file during build and assemble workflow

Is your feature request related to a problem? Please describe
Currently both manifest files share the same name inside builds and dist folders. Thus they cannot co-exist. We want to have both together. This is motivated by #604
Describe the solution you'd like
OpenSearch and OpenSearchDashboards manifest files could be distinguished and thus co-exist.
Describe alternatives you've considered
No response
Additional context
No response

I'd rather not mix outputs, so either we add ./build.sh --output-dir <path> or default to builds/opensearch vs. builds/opensearch-dashboards?

I will follow "default to builds/opensearch vs. builds/opensearch-dashboards"


Attribute error on the scraper

AttributeError
'tuple' object has no attribute 'get_font_size_list'
See: https://sentry.io/wellcome-data-labs/web-scraper/issues/863364118/

 Fix Attribute Error in the web-scraper


MGMT-12054: Support discovery kernel arguments for full-iso and minimal-iso

When streaming the discovery ISO file, the kernel arguments are included as part of some files: grub.cfg and isolinux.cfg
Each of these files contains an area where the kernel arguments can be embedded. This are can be identified by the regex "(\n#+) COREOS_KARG_EMBED_AREA". The arguments should overwrite this are from the beginning and should not pass the last # (hash sign). To do that, an overlay was created.
In new Openshift versions, there is a file that contains the file names, while in old versions the file does not exist. If the file exists, the files to embed the kernel arguments are taken from this config file. Otherwise, the default files [/EFI/redhat/grub.cfg, /isolinux/isolinux.cfg] are used.
Description

How was this code tested?

Assignees

/cc @carbonin
/cc @filanov
Links

Checklist

[ ] Title and description added to both, commit and PR
[ ] Relevant issues have been associated
[ ] Reviewers have been listed
[ ] This change does not require a documentation update (docstring, docs, README, etc)
[ ] Does this change include unit tests (note that code changes require unit tests)

/assign @avishayt

Ahhh wait, is there a reason this code isn't using coreos-installer?  I am not totally sure our existing kargs area is stable public API.

@cgwalters I think there are still a few gaps.
The last feature gap IIRC was that the minimal ISO generated by coreos-installer doesn't let us set up our static networking (posisibly for multiple interfaces) and proxy information before pulling the rootfs.
I recall talking to @jlebon about this a while back

But it may very well be time to meet up again to discuss if everything is covered now.
I definitely wouldn't mind moving to coreos-installer if it covered all our use cases.

hmmm also even if coreos-installer supports streaming the output iso to stdout we'd still be missing range request support I think.
I think it would be good to move this discussion out of this PR though. I'll open an issue for us to consider coreos-installer.

Yeah, we discussed this in the past. The use of COREOS_KARG_EMBED_AREA here is not ideal, and we're hoping that coreos-installer can satisfy AI use cases in the future. We have support for network configuration now (see https://coreos.github.io/coreos-installer/cmd/iso/#coreos-installer-iso-network-embed), but the proxy bit is still missing (tracked at https://github.com/coreos/coreos-installer/issues/794). Would be good to discuss the range request bit in more details somewhere.

Opened https://issues.redhat.com/browse/MGMT-12340 to track this.
I tried to add a much as I could think of to that. Let's move the discussion there

In new Openshift versions, there is a file that contains the file names, while in old versions the file does not exist. If the file exists, the files to embed the kernel arguments are taken from this config file.

As a separate enhancement we should probably expand our use of this file to use it to determine where to add the rootfs parameter when creating the minimal iso.
This would probably solve some problems (or at least minimize the amount we would need to fix) for the IBM folks doing power and Z integration.
cc @mtarsel @jacobemery

/hold

/unhold


Bad, sad error when attempting to push a package built without using -c

Description:
$title.
Steps to reproduce:

create a simple package
build without using -c (i.e., bal build)
attempt pushing it - I tried to push it to the local repo (bal push --repository=local)

This results in a bad, sad error with the following in the ballerina-internal.log
[2021-03-17 16:06:42,552] SEVERE {b7a.log.crash} - cannot find bala file for the package: proj. Run 'bal build' to compile and generate the bala.
io.ballerina.projects.ProjectException: cannot find bala file for the package: proj. Run 'bal build' to compile and generate the bala.
	at io.ballerina.cli.cmd.PushCommand.validateBalaFile(PushCommand.java:221)
	at io.ballerina.cli.cmd.PushCommand.pushPackage(PushCommand.java:177)
	at io.ballerina.cli.cmd.PushCommand.execute(PushCommand.java:135)
	at java.base/java.util.Optional.ifPresent(Optional.java:183)
	at io.ballerina.cli.launcher.Main.main(Main.java:58)

Even this error is not useful since I did use bal build.
Affected Versions:
slalpah3-SNAPSHOT

@azinneera, with the fix for this, there is no bad, sad error but the error I get is still
ballerina: cannot find bala file for the package: proj. Run 'bal build' to compile and generate the bala.

Instead, can't/shouldn't we say
ballerina: cannot find bala file for the package: proj. Run 'bal build -c' to compile and generate the bala.

@azinneera, with the fix for this, there is no bad, sad error but the error I get is still
ballerina: cannot find bala file for the package: proj. Run 'bal build' to compile and generate the bala.

Instead, can't/shouldn't we say
ballerina: cannot find bala file for the package: proj. Run 'bal build -c' to compile and generate the bala.

Just noticed that this has been fixed with https://github.com/ballerina-platform/ballerina-lang/pull/29349.


Create a maintained mapping of <string,sf::Keyboard::Key> for configuration files.

I've been using SFML to create a project and upon creating custom configurations (using JSON), I discovered the amount of difficulty in extracting an enum value from a string without having a  unordered_map of "string, EnumType".
I initially decided to use integers in the configuration but this is a problem as the sf::Keyboard::Key could change which would require a re-mapping and could cause odd behaviour in applications due to key codes changing position.
I'm confused in how this would be implemented without having a maintainable map of keys which would have a helper function such as "sf::Keyboard::keyWithIdentifier("Up")" or something along those lines. It would add a lot of simplicity to the library when it comes to flexibility and protects the programmers from any problems further down the line.
My current implementation is a class with a map of the key names and the keys. For example:
keyMap["A"] = sf::Keyboard::Key::A
keyMap["B"] = sf::Keyboard::Key::B
keyMap["C"] = ...
This seems future-proof enough but I feel like this should be in the library, not in my own code-base. C++17 will hopefully be introducing a way to extract Enum Values via a string (which is exactly what my requirement would be) using reflection.
Unfortunately, this isn't something that's in C++11 and should have a simple method.
That's my thoughts from using the library for a while. I'm really enjoying it so far, good job!

I initially decided to use integers in the configuration but this is a problem as the sf::Keyboard::Key could change which would require a re-mapping and could cause odd behaviour in applications due to key codes changing position.

And if any name in the SFML implementation changes, you've got the very same problem. :)
I don't think providing a mapping is the right way to go here (should still happen in "library user land", but I think there should be a function to retrieve key names (there are typically platform specific API calls for that).

[...]  I discovered the amount of difficulty in extracting an enum value from a string [...]

You should have researched a bit, you're not the first one with this problem. This functionality has been in Thor for quite a while now :wink:

but I think there should be a function to retrieve key names (there are typically platform specific API calls for that).

Why do you need platform-specific key names in portable library like SFML? You can't use them for serialization...

Because this didn't attract much attention and is still flagged as undecided, I'm closing this. But feel free to further make your case (preferably on the forum). Besides, @MarioLiebisch referred to #7, which is more fundamental.


Added configuration file that enables correct commenting using keyboa…

…rd shortcut.

I merged this into language-gnuplot-5


clawsMail: 3.13.0 -> 3.13.1 (CVE-2015-8614)

Fixes a remotely triggerable buffer overflow.
http://www.thewildbeast.co.uk/claws-mail/bugzilla/show_bug.cgi?id=3557

15.09 has version 3.11.1. Do you know if that one's vulnerable as well?

@vcunat After looking at http://git.claws-mail.org/?p=claws.git;a=blob;f=NEWS;h=f717c2abe3fa6f7ec6dbdf9ec5ffcf95b8dabb42;hb=HEAD I think we should cherry-pick this (and the rest of the fixes to this expression) into 15.09. In 3.12.0 there was already a fix for CVE-2010-5109 and the commit history for the affected file src/codeconv.c shows that 3.11.1 is indeed also vulnerable to CVE-2015-8614.

:+1: should be cherry-picked to 15.09

Done.

Thanks!

:-)


US 01.02.01 Viewing QR Codes in the Player's Account

As a player, I want to be able to see what QR codes I have added to my account so that I do not accidentally add the same QR code again.
Priority: Medium
Story Point: 3
Risk Level: Medium

Acceptance Criteria:

player will have a feature to keep track of all QR code they have scan.
the system will also have an feature automatically help players to eliminate duplicate QR code in their account.
player scanned an QR code more than once the system will alert the player that the "QR code has been scanned Try different one "

Duplicate feature with #34


Feat: Add Copyrights Section at footer

Describe the feature
I want to add copyrights section at the bottom of the page in Resource page. There is no copyrights section in resource page.
Add ScreenShots

Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[X] I want to work on this issue

@MuraliDharan7 Thanks for your issues but it is already in our web try to work on different issues.


backgroundColor fix

Fixed BackgroundColor not changing by replacing DecoratedBox to Container

I have found the problem: I only take the width, but not the height of the widget.
I don't want to use Container at this point because SizedBox is more efficient.
Thank you very much for your help. If you like, I can include your GitHub tag in the next changelog.


<EMAIL_ADDRESS>decompress error

Signature detected: Inno Setup Setup Data (6.3.0)
This is not directly supported, but i'll try to unpack it as version 5602
; Version detected: 6300
Critical error: The setup files are corrupted. Please obtain a new copy of the program.

Error persists even if I redownload installer.

can be fixed by https://github.com/ScoopInstaller/Extras/issues/13547#issuecomment-2210208796

Duplicates #13709.


Include version in purl (database)

Hi,
Today, the database only have one purl matching one cpe with all its different versions. So for exemple pkg:github/wp-plugins/simple-banner will have the following cpe:

cpe:2.3:a:simple_banner_project:simple_banner:1.0.1:*:*:*:*:wordpress:*:*
cpe:2.3:a:simple_banner_project:simple_banner:1.0.2:*:*:*:*:wordpress:*:*
cpe:2.3:a:simple_banner_project:simple_banner:1.0.3:*:*:*:*:wordpress:*:*
[...]
cpe:2.3:a:simple_banner_project:simple_banner:2.9.4:*:*:*:*:wordpress:*:*
cpe:2.3:a:simple_banner_project:simple_banner:-:*:*:*:*:wordpress:*:*

Don't you think it would make sense to add the version in the purl field ?
For with my previous example purl field would becomes:

<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>[...]
<EMAIL_ADDRESS>pkg:github/wp-plugins/simple-banner

Is it something that would make sense for you ?
Thank you

Sorry duplicate of this https://github.com/scanoss/purl2cpe/issues/11. But I'm not sure to understand why it's not possible to extract version from cpe and add it to purl ?

The CPE version cannot be automatically added at the end of the PURL because in various cases the 2 of them are different strings. Take for example "cpe:2.3:a:openssl:openssl:1.1.1r:::::::*". The GitHub PURL with version included, for openssl 1.1.1r, is "pkg:github/openssl/openssl@OpenSSL_1_1_1r", since that's how the openssl developers have declared the version(tag) in GitHub. And, according to the PURL specifications, the version for a GitHub purl is the tag name.


fix julia README

The repo links of julia-mode and julia-repl were swapped in /modules/lang/julia/README.org.  This corrects the repo links.

Thanks!


AQAvit Meeting: Dec 14, 2022

Join via https://bluejeans.com/917304424
This meeting is part of the public Adoptium community calendar, if you wish to import the calendar use this link
Proposed Agenda:

New TRSS features (rerun links) - @momanmalik
Last meeting of 2022 / Some highlights of the year - All

We will cancel this call and fold in the 2022 highlights into the Adoptium community call that happens earlier in the day!


feat: add width, height, type and name to images in metadata

Codecov Report

Merging #438 (c764e86) into main (5315c52) will decrease coverage by 0.61%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #438      +/-   ##
==========================================
- Coverage   55.53%   54.93%   -0.61%
==========================================
  Files         121      104      -17
  Lines        3450     3264     -186
  Branches      573      567       -6
==========================================
- Hits         1916     1793     -123
+ Misses       1534     1471      -63

Flag
Coverage Δ

common
84.61% <ø> (ø)

core-sdk
52.33% <ø> (ø)

eth-connect-sdk
17.33% <ø> (ø)

ethers-sdk
?

ipfs-storage
94.02% <ø> (ø)

metadata
97.56% <ø> (ø)

unittests
54.93% <ø> (-0.61%)
:arrow_down:

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

Impacted Files
Coverage Δ

packages/metadata/src/product-v1/index.ts
96.61% <ø> (ø)

...c/contracts/factories/ERC721Enumerable__factory.ts

...ntracts/factories/IBosonDisputeHandler__factory.ts

...ts/factories/MockNativeMetaTransaction__factory.ts

...contracts/factories/IBosonFundsHandler__factory.ts

packages/ethers-sdk/src/ethers-adapter.ts

packages/ethers-sdk/src/contracts/index.ts

...s/factories/IBosonOrchestrationHandler__factory.ts

...hers-sdk/src/contracts/factories/ERC20__factory.ts

...contracts/factories/IBosonOfferHandler__factory.ts

... and 8 more

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


mediation clean up

this PR contains code merged from main with fixes to demonstrate mediation with WebSocket transports.

@burdettadam very nice work on the e2e test! I think this will allow to finally remove the dependency on running separate scripts for mediators.
I hope everything works fine when merging with the latest changes I introduced at mediation-rebased branch. I've fixed a few bugs I had on my side regarding the promiseWithTimeout and KeylistUpdated event (not sure if it's the best way to fix them, as I feel that internal routing event naming is not clear enough).
For the rest of the changes (such as the autoAcceptMediationRequests flag and cleaning up of mediation-client and other classes) I don't think there will be any issue.

@genaris, Thanks. I did fix some small things when I cherry-picked your contributions, I think also needed to change something for the keylistUpdated flow. Is there more code I can cherry-pick? I am open to suggestions if you see improvements I overlooked.
I think we will be able to condense a lot of code as things become more stable with mediation. @JamesKEbert was able to fix some frustrating issues I had with mocking transports, things are much better now in e2e.
I do realize there is some funny naming, suggestions are welcome. :)

@burdettadam, yeah your cherry-pick with fixes was almost perfect! After your PR including it, I committed some minor fixes (you can check it here). It seems that this PR can be merged without any major problem.
The only conflicting point could be the KeyListUpdated routing event type I added to distinguish the one generated in mediator service for sending the key list update response (MediationKeylistUpdated) from the one generated for updating keys in a mediation record (used in both mediator and recipient modules). I was a bit lost out there (though I managed to make it work fine in the flow I implemented in e2e.http.test.ts). I'm sure that naming can be improved!

@burdettadam quite some changes in the main branch. Let me know if I can help with merging / rebasing


An error occurs when you save an encrypted Excel document and open it again.

When I opened my encrypted Excel for the first time, everything went as expected. But when I open it again in the same way,error occurred.In short, as long as you don't call write, you should have no problem reopening,   Is there something wrong with the way I keep it?,Can someone help me? thanks!
the following error occurs:
NPOI.POIFS.FileSystem.OfficeXmlFileException:“The supplied data appears to be in the Office 2007+ XML. You are calling the part of POI that deals with OLE2 Office Documents. You need to call a different part of POI to process this data (eg XSSF instead of HSSF)”
and this is my code:
   `public void Test()
    {
        IWorkbook workbook = null;
        string filePath = @"C:\Users\14028\Desktop\test.xlsx", password = "123";
        using (FileStream fileStream = new FileStream(filePath, FileMode.Open, FileAccess.ReadWrite))
        {
            Stream stream = null;
            try
            {
                dynamic fs = new POIFSFileSystem(fileStream);
                EncryptionInfo info = new EncryptionInfo(fs);
                Decryptor decryptor = Decryptor.GetInstance(info);
                bool isPasswordCorrect = decryptor.VerifyPassword(password);
                stream = decryptor.GetDataStream(fs);

                workbook = new XSSFWorkbook(stream);
            }
            catch (Exception ex)
            {
                Console.WriteLine(ex.Message);
                return;
            }
            finally
            {
                if (stream != null)
                {
                    stream.Close();
                }
            }
        }

        using (FileStream file = new FileStream(filePath, FileMode.Create, FileAccess.Write, FileShare.ReadWrite))
        {
            workbook.Write(file);
        }
    }`

Encrypted Excel is not supported to be saved by NPOI so far.


Get cloudwatch metrics in another AWS account?

Is it possible to use this adapter to pull in cloudwatch metrics from another AWS account?

Never mind, figured out you can easily do this by passing env vars to the Deployment with an access key/secret key. Leaving this in case it's helpful for someone else in the future.
spec:
  template:
    spec:
      containers:
      - name: k8s-cloudwatch-adapter
        ...
        env:
        - name: AWS_ACCESS_KEY_ID
          value: <the-key>
        - name: AWS_SECRET_ACCESS_KEY
          value: <the-secret>


: uif-label is not rendered

Expected Behavior
uif-label is rendered
Compare to: http://dev.office.com/fabric/components/dropdown
Actual Behavior
uif-label is not rendered

Labels are not automatically added to components, this is up to the developer to add a label using our uif-label directive.

Thank you for the fast response. Other components like uif-textfield do offer contain the uif-label. it would be great to have these consistent between typical form components.

Thanks for bringing this up... we'll make sure to standardize across all directives.


Design logo for application

Design logo for application

@avinashupadhya99 pls assign me this issue.

Sure, please go ahead. You can find the Dictu logo here and you can add playground to it.

Make a PR with the logo in the public directory under frontend


Build failure: python3.11-mkdocs-simple-hooks

Steps To Reproduce
Steps to reproduce the behavior:

Enable services.ntfy-sh (e.g. use sth like https://codeberg.org/artemislena/nixos-server-configs/src/branch/main/http/apps/ntfy.nix) (tried on both unstable n unstable-small)
nixos-rebuild switch

Build log
https://pad.artemislena.eu/code/#/2/code/view/nhDlUGw3cgwA7VxnPhAzbXEoWx0qHrKNl-B225ydTyk/present/
Additional context
Seems somewhat similara https://github.com/NixOS/nixpkgs/issues/286480
Notify maintainers
@arjan-s
Metadata
root@rp64-alu:~/ > nix-shell -p nix-info --run "nix-info -m"
 - system: `"aarch64-linux"`
 - host os: `Linux 6.1.81-hardened1, NixOS, 24.05 (Uakari), 24.05pre612742.66adc1e47f87`
 - multi-user?: `no`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.18.2`
 - channels(root): `"nixos"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Add a :+1: reaction to issues you find important.

Perhaps we can just remove this package? It's only required by ntfy-sh currently, which doesn't use it anymore since 2.9.0. Also, according to those release notes, hooks are a native part of mkdocs since 1.4, so I want to propose to just remove this.


make sure phone only have + and number

phone number form when logging in case have non number characters like ",". the query is still send to the backend but is being rejected. there should be a check before sending the query that the phone number has the same regex check than the backend has.

PR 76

Looks like this was fixed in #76.


Allow specifying a custom initialTime for ReactiveDatePicker

I have a use case where the firstDate depends on the current date. The property is stored in a database, but is editable. This means that I end up with a case where a user selected a date that was valid at the time they selected it. However, if the then selected date has passed, the user is no longer able to edit the date, to set a new, valid date.
The specific error is something like:
'package:flutter/src/material/date_picker.dart': Failed assertion: line 167 pos 5: '!initialDate.isBefore(firstDate)': initialDate 2022-01-26 00:00:00.000 must be on or after firstDate 2022-01-27 00:00:00.000.
The issue is obvious, the date is set to a past date, which is no longer valid. But the user should still be able to edit it. Currently, the only solution is to modify the Form data in order to update the value to a valid date, before opening the picker. However this is causes a whole range of other issues and problems.
It seems that an obvious and simple fix would be to let us specify a custom nullable initialDate, which would take precedence over any other date when provided.

Another similar case that would be solved by the same fix is when the firstDate > now, but the form value is null. initialDate will then default to now, which fails validation and the same error occurs. And I don't want to modify the form data unless the user actually selects a value. It should remain null if the user doesn't touch the field.

Another, and perhaps better solution, would be to consider the firstDate in the _getInitialTime() function, and default to firstDate if the fieldValue is null and firstDate is specified. And also default to firstDate if fieldValue < firstdate

Hi @ezet
As you posted in your first message - the issue comes from “ package:flutter/src/material/date_picker.dart” which is behind our control as far as we just proxy values to the component.
Is you have any custom logic that does not fit into standard flutter control you can create your own component referring to exiting date picker as an example.

@vasilich6107 Yes, the error is thrown from a framework package, but the problem arises because this library passes an invalid argument to the framework "date_picker", without giving us a way to change that behavior. The framework picker accepts an argument initialDate, which is invalid in this case. ReactiveDatePicker sets that value through it's own calculation, which has buggy behavior, as I described in detail above. There are two solutions I can think of, as mentioned:

Add a new argument on ReactiveDatePicker, that lets me override the initialDate
Fix the bugged calculation, so that it doesn't provide an invalid value to the framework picker. https://github.com/joanpablo/reactive_forms/issues/264#issuecomment-1022579701 describes a possible fix.

If required, I can provide a PR, but I'd first need to know whether this is something you'd accept, and which solution is preferred.

@ezet I suppose that giving an ability to override _getInitialTime would be a universal solution

@ezet i'm maintaining the package reactive_datetime_picker
https://github.com/artflutter/reactive_forms_widgets/blob/master/packages/reactive_date_time_picker/lib/reactive_date_time_picker.dart
I will try to add an ability to override _getInitialTime which could be helpfull
It will be the shortest path cause the releases of reacitve_forms are unpredictable)
There are 7 PR from myself waiting to be merged for month already

@vasilich6107 Thank you, this should suffice for now! I realize this should've been reported as an issue in your repo, not here. But thanks for assisting anyways! You can consider also applying the logic correction I proposed in my comment, while also allowing callers to provide a custom value. I don't think my proposal would be breaking, as it would only use a different initialDate in cases that would otherwise result in an exception.

Technically it was easier to me to give an ability to override the initial date getter)

@ezet I will try to find time to apply the same improvement here...
But only the gods know when it will be merged

@ezet i'm also working on generator plugin for reactive forms
https://github.com/artflutter/reactive_forms_generator
It would be great if you could try it

Hi @ezet,
Thanks for the issue, I guess the solution is to fix the "bug" in the initial date calculation because you already set the value of the date in the Control.
Or maybe both, if you provide the initialDate to the Widget and the value of Control is Null then the control may assume the value of the Widget. (But I will work on fix the calculation first)
I will see what is the error and will try to release the fix asap.
@vasilich6107 sorry for the delay on PR. I know they are almost all related to FocusNode, I haven't prioritized them yet since they are not a critical issues, but you are right about the delay.

Hi again @ezet,
Do you think the argument of initialDate should be a method or just another optional property like firstDate, and lastDate? If we add an extra argument to the ReactiveDatePicker would that be enough for you? (Of course I will reimplement the calculation of the _getInitialDate as a separated fix)

Hi @ezet and @vasilich6107,
this is the New implementation of the _getInitialDate
static DateTime _getInitialDate(DateTime firstDate, DateTime lastDate, DateTime? defaultInitDate) {
    var initialDate = defaultInitDate ?? DateTime.now();

    if (initialDate.compareTo(firstDate) < 0) {
      initialDate = firstDate;
    } else if (initialDate.compareTo(lastDate) > 0) {
      initialDate = lastDate;
    }

    return initialDate;
  }

this is how I call it
initialDate: _getInitialDate(firstDate, lastDate, initialDate ?? field.value)

As you can see, now the ReactiveDatePicker will have an optional new argument: initialDate, this one will have more priority than the field.value. If both values are null then the DateTime.now() will be used. And inside the method I make sure that the value is inside the range of firstDate and lastDate.
That should fix the issue.
I will add some tests and let you know when it is released.

Hi @ezet and @vasilich6107,
I have released a new version v11.0.1 of Reactive Forms which not only includes an optional argument initialDate, but also fix the initialDate calculation.
Please if you still have issues, don't hesitate to reopen the issue or create a new one.

Do you think the argument of initialDate should be a method or just another optional property like firstDate, and lastDate? If we add an extra argument to the ReactiveDatePicker would that be enough for you? (Of course I will reimplement the calculation of the _getInitialDate as a separated fix)


Add Observability sample data to Dashboards

Currently we have this page

Offering sample data to play with.
Can we add an observability data set, along with the associated saved objects (if we need any apart from what the O11y plugin provides out-of-the-box). That will make it easy for users to experiment with the plugin and its features.

Thanks for the issue!
We already have sample data for observability available in the logs tab. It uses the flight and web log sample data sets for various features. Can you help us understand more about what you feel is missing?

I'm looking for data to use with the Observability plugin, including service map, traces, and log patterns.


Can not convert #STRING to a NodeList!

Throws this error when i trying to parse the xml file
org.apache.hadoop.hive.serde2.SerDeException: java.lang.RuntimeException: javax.xml.xpath.XPathExpressionException: com.sun.org.apache.xpath.internal.XPathException: Can not convert #STRING to a NodeList!

How does your XML and CREATE TABLE statement look like?

I did not receive requested information from the submitter. Closing this issue as "Cannot replicate".


fix: appbar submenu, added new test cases

fix: appbar submenu
added test cases for docker, dropdown, ellipsis, fab, hscroll

@ryantanrk can you run npm run format before committing?

LGTM


Update login.tsx

Adding a ??missing?? link to forgot password page

@allanice001 We already have the link in the login page
https://github.com/boxyhq/saas-starter-kit/blob/main/pages/auth/login.tsx#L89-L93


User can copy and paste location names in the search bar

Description:
Locations can be added to the search bar by copying and pasting from another page

Preconditions:
The website is up and running.

Step 1
Copy a location name from another website (e.g LEGE nr. 2 din 16 februarie 1968

Expected results
The location name was copied

Step 2
Paste the location name in the search bar

Expected results
The location name appears in the search bar.


[Marketing] Mailchimp Registration Emails

Summary:
This issue is dedicated to generating the mailing list for the current registered users and assigning them the proper tags.
Drive Location:
https://drive.google.com/drive/u/0/folders/11TuqI6s_mp6y21B02n9uGgjCxiTaqHKr
Tasks:

[x] Export Data from Mailchimp and merge with Drive (Data/Registrations)
[x] Add tags to the users

This is now fixed


Not able to install RDS CALs Per User License in Azure Windows 2019 VM

[Enter feedback here]
I am stuck with installing RDS CALs per user license. I am in a windows server 2019 VM and allowed 3 users to login and access the concurrently, this is happening but when I am trying to activate the server and install the license it is not happening. I also made this VM as a DC but still getting the same error. I am following all the steps which is Microsoft documentation but why I am still not able to achieve this? am I wrong or the documentation is missing steps? Please I need help I have to deliver this to the customer. Please provide me a solution.

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

ID: 5a02d662-56db-4efc-52d4-61689541f632
Version Independent ID: f8692043-d888-09d3-1bf3-bff51ae20e34
Content: Install RDS client access licenses
Content Source: WindowsServerDocs/remote/remote-desktop-services/rds-install-cals.md
Product: windows-server
Technology: remote-desktop-services
GitHub Login: @lizap
Microsoft Alias: elizapo

Hello @AgassiJoeLVedala89. GitHub issues for this repo are for reporting problems with our documentation.  Please open a support ticket or post your question on Microsoft Q&A.


[Event Request] Table 39 Purchase Line - trigger OnDelete - OnDeleteOnAfterPurchLineDeleteAll

Please add an Event OnDeleteOnAfterPurchLineDeleteAll in trigger OnDelete in Table 39 Purchase Line
[Event Request] Table 39 - Purchase Line - trigger OnDelete
trigger OnDelete()
var
    PurchCommentLine: Record "Purch. Comment Line";
    SalesOrderLine: Record "Sales Line";
    IsHandled: Boolean;
begin
    IsHandled := false;
    OnDeleteOnBeforeTestStatusOpen(Rec, IsHandled);
    if not IsHandled then
        TestStatusOpen();
    ...
    if "Line No." <> 0 then begin
        PurchLine2.Reset();
        PurchLine2.SetRange("Document Type", "Document Type");
        PurchLine2.SetRange("Document No.", "Document No.");
        PurchLine2.SetRange("Attached to Line No.", "Line No.");
        PurchLine2.SetFilter("Line No.", '<>%1', "Line No.");
        OnDeleteOnBeforePurchLineDeleteAll(PurchLine2);
        PurchLine2.DeleteAll(true);

        OnDeleteOnAfterPurchLineDeleteAll(Rec, PurchLine2); // <-- NEW EVENT
    end;
    ...

[IntegrationEvent(false, false)]
local procedure OnDeleteOnAfterPurchLineDeleteAll(var PurchaseLine: Record "Purchase Line"; var PurchLine2: Record "Purchase Line")
begin
end;

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: 46009.


the same promblem , how to fix please?

I use window 10 , python3.8。
$ source activate py3env
(py3env) preks ~
$ flask fab create-admin
Username [admin]: admin
User first name [admin]: admin
User last name [user]: admin
Email<EMAIL_ADDRESS><EMAIL_ADDRESS>Password: admin
admin

I have installed apache superset in my virtual env. I ran the command 'superset db upgrade'
And set the flask app using 'export FLASK_APP=superset'
But when I try to create admin user, I'm not asked to repeat for confirmation, instead it waits for input and when I re-enter the password and hit enter, nothing happens. It again waits for input.
How can I fix it?
Originally posted by @ericfonsi in https://github.com/apache/superset/issues/9143#issuecomment-721595344

Unfortunately Superset doesn't officially support Windows, so closing this. However, please feel free to ask for help on GitHub Discussions on on the Superset Slack. Also, please always use a descriptive title + fill out the standard template when opening new issues.


a small optimization

https://github.com/Miko008/SDAonCUDA/blob/457e5c998406faca512d2353917d3a6cc61fad22/SDAonCUDA/test.cu#L54
The line above is equivalent to:
if (condtition_1 && condtition_2 && condtition_3)
Here, condition_1 is the most CPU-consuming. On the other hand, it evaluates to False in approx. 20%, while condition_2 and condition_3 evaluate to True much more often.
Anyway, I think that the it would be better to move condition_3 to the last position.

Also moved multiply operation to caller in argument passing.


Update JDA to 5.0.0-beta.3 (and change command privileges)

Overview
This pull request updates the JDA dependency to 5.0.0-beta.3. It also removes command privileges, and adds in command permissions (as a replacement).
Command privileges can now only be created by a moderator of a Discord server. Command permissions do similar functions, denying certain users from running commands, and can be changed by a guild moderator.
Changes

@Privilege() and any code that registers that annotation has been removed.
net.dv8tion:JDA has been updated to 5.0.0-beta.3
@Permissions() has been added as a suitable replacement for @Privilege()
The @Command().enabled property has been removed as it is not referenced in any code.

The jda-kotlinx PR has been merged and the dependency needs to be updated to fc536ec

Hey @ruffrick! I've updated jda-kotlinx to the latest and applied your suggestion.


Add spec for parallel execution semantics

I think that recent PRs with specs for collective ops (e.g. #503) would benefit from a more formal definition of parallel execution semantics of StableHLO programs.
That will provide additional clarity which is needed at the moment (see discussions in the PRs) and reduce the duplication between the specs of individual collective ops.
The description of execution semantics isn't yet as formal as I'd like, but it can already benefit specs for collective ops. I'll leave the rest to follow-up work within #484.

Merging given an approval for this PR and the absence of requests for changes. If I misread the room, and some of this feedback was blocking, please let me know, and I'll address in a follow-up PR.


Expense attachments icons are not respecting their container size on Firefox mobile

On mobile:

See https://opencollective.com/captainfact_io/expenses/35329

It looks like it's fixed.

@snitin315 You're viewing an expense where you don't have permissions over the attachments. The bug only seems to affect the preview icon as seen in the screenshot above. I just tested on Firefox Mobile + Android and it's still there.

@Betree : Quick fix for this done by making sure the attachment holder stays the same size. Hope that looks good now. Let me know. 😄

Facing something similar on Safari:

This is still happening. Context:

Latest Firefox mobile on Android
https://opencollective.com/captainfact_io/expenses/64909
Logged in as a collective admin

Still seeing this issue, I'm adding a bounty on it

:trophy: This issue has been completed by @snitin315
To claim the bounty, you can either:

Submit an invoice to https://opencollective.com/engineering/expenses/new (please mention the issue number)
Or ask for an Open Collective gift card of the same value (to donate to other collectives)


Console button changes to Suspend or Shutdown

When I want to see a console of running VM for the first time (no console in use dialog), the button changes to suspend or shutdown button for a moment and then changes back to console button. Console runs without problems.
Tested in: ovirt-web-ui-<IP_ADDRESS>.20180814git2ec0a1e-1.fc28.noarch.rpm, Mozilla Firefox for Fedora
(Sorry for low quality pic)

@sjd78 @bond95 thoughts on this one?

@gregsheremeta @sjd78 problem is in action filtering. Because disabled actions are filtered. And console action is disabled for a moment.
actionDisabled: isPool || !canConsole(status) || vm.getIn(['actionInProgress', 'getConsole']),
As you can see console action is disabled when actionInProgress, this was done to prevent more clicking on button.

Actually, problem was in filtering and now it fixed. In any case, with console page patch, first button will be Suspend. So I think this issue is not relevant and can be closed.
WDYT? @gregsheremeta @sjd78

Actually, problem was in filtering and now it fixed. In any case, with console page patch, first button will be Suspend. So I think this issue is not relevant and can be closed.
WDYT? @gregsheremeta @sjd78

works for me, closing. thanks!


Change hover color of navbar items same as Join Waitlist

I want to change hover color of navbar items to blue same as Join Waitlist

Kindly assign me this issue.

@singodiyashubham87  great

@alonkeyval I've been searching a few times now but didn't find the code for the navbar, Can you please guide me on where should I make my styling changes?

you can find it in the "Header" folder @singodiyashubham87

@alonkeyval Not able to navigate, please tell me the exact path.

/components/Header
@singodiyashubham87

I would love to work on it

@Vijaykv5 I have gone through the conversation, and i find that the code related to this issue is not in this repo, but in the other repo of the same organisation which is, landing-page repo.

You can find the code here

I'm working on it @alonkeyval


Unable to reproduce results in the experiment 2

Hi,
Thanks for sharing the code. However I'm not able to reproduce the paper results in experiment 2 particularly. I downloaded the code and ran bash replicate_experiment_2.sh as per the instructions. It seems like Stable SGD always returns Avg accuracy less than or approximately equal to the Naive SGD for all three datasets. I also tried varying hyper-parameters such as gamma, lr and dropout etc. but the Stable SGD results did not come near to ones given in the Paper.
Is there really a bug in replicate_experiment_2.sh or it is my misunderstanding? I was wondering if you could provide any hints for reproducing the results.
Thanks in advance.

Hi Pranshu,
There were minor issues in the earlier version of the code but after the last merge-request, it is almost solved. I'm currently working on the camera-ready version of the paper and I'll update the repository in ~2 weeks and by then I will make sure that everything works smoothly. For now, please use the following commands to replicate results.
To replicate experiment 2 (Rotated MNIST), use the following params use:
python -m stable_sgd.main --dataset rot-mnist --tasks 20 --epochs-per-task 1 --lr 0.1 --gamma 0.55 --hiddens 256 --batch-size 64 --dropout 0.25 --seed 2345
To replicate experiment 2 (CIFAR-100), use the following params use:
python -m stable_sgd.main --dataset cifar100 --tasks 20 --epochs-per-task 1 --lr 0.15 --gamma 0.85 --batch-size 10 --dropout 0.1 --seed 1234
Also, we have just published a new method [MC-SGD] that outperforms other methods (including stable-SGD) by a large margin. For that work, we have posted reproducible Codalab for rotation MNIST with 20 tasks (we'll add these links for other experiments later). You can find those links here: https://github.com/imirzadeh/MC-SGD where you can reproduce the rotated MNIST experiment with 20 tasks using those links very simply and get ~83% accuracy.

Hi Iman,
Thanks for a fast reply. The above set of hyper-parameters combination worked fine for both Rotated-MNIST and CIFAR-100. Although the combination for Rotated-MNIST also results in a better performance on Permute-MNIST, its avg accuracy is not the same as given in the paper.
I also looked at the MC-SGD method and the results are pretty interesting. I wanted to discuss about the approach and performance on other benchmark datasets. I'm closing the issue here, but can we take the discussion to e-mail?

Sure. Please feel free to send an email to me. You can find email from my papers.


[network] added chain-id to differentiate chains

The chain-id is to differentiate between different blockchains, and is
enforced at the handshake level similar to network-id.

I can't seem to find @aching's comment, but the whole config is logged during startup.
https://github.com/libra/libra/blob/master/libra-node/src/main.rs#L36

@bors-libra r=davidiw

:pushpin: Commit aa5f374 has been approved by davidiw

:hourglass: Testing commit aa5f374d43ba52401518a99b916af2f420bf9634 with merge 96168d83e2cf2599acd67b83c2c023569f5bd324...

:sunny: Test successful - checks-actions_land_blocking_test, checks-circle_commit_workflow
Approved by: davidiw
Pushing 96168d83e2cf2599acd67b83c2c023569f5bd324 to master...