Add a container workload

Now that the world has gone "container" crazy and most platforms have support for it, it makes sense to have some comparative workloads with and without containers.
For fun...
http://www.networkworld.com/article/2888180/cloud-computing/everyones-going-container-crazy.html
Input on what to do here is very welcome.

I'm not volunteering for this right now,  but perhaps we could build a primitive that allows to run any of the existing benchmarks within a container, and it can support several container engines (docker, lxc, rocket, lmctfy, etc).
My concern is how do we handle all the configuration options, which are crucial for performance, like not containerizing the containers network stack (uses --net=host), different choice of filesystems (AUFS vs others), container-to-container networking, etc.
The other approach would be to run a workload that is already containerized and treat it as another benchmark, then all it would require is to install the container engine, and pull the image from a registry (like docker), and run it.  The downside with this approach is that it would require to containerize multiple workloads.
One interesting workload that has been containerized is the Acme Air benchmark from IBM. I believe it simulates a 'cloudy' service for an airline, where you book seats for flights, and it even includes a JMeter script to generate load against it.
Here's the uncontainerized version https://github.com/acmeair/acmeair,
and the docker version: https://github.com/aspyker/acmeair-netflixoss-dockerlocal

At this point we have docker workloads, kubernetes, and meso all part of PKB.
@meteorfox - Think is this addressed?


jwks has 0 rows after creating user and signing in

Describe the bug
I've added the jwt plugin and tried a couple different configurations, but it seems no rows are created in jwks and if i hit
http://localhost:1421/api/auth/jwks?currentURL=http%3A%2F%2Flocalhost%3A1421%2F
or query the DB jwks table both return empty.
To Reproduce
It's a bit involved because you'd need to add github keys to the .env.example and then copy it to .env...
git checkout natechat2
yarn
cd apps/onechat

And then run:
yarn docker up

In one terminal and:
yarn dev

Go to http://localhost:1421
Hit the user icon top right to login with github
It will log the jwks fetch for you in console once logged in

Expected behavior
I think it should make a key on successful user create/login/jwks fetch, maybe I'm missing something.
Screenshots

See me logging the user and endpoint fetch:

Desktop (please complete the following information):

OS: Mac
Browser All

Ah nevermind - you have to hit authClient.token() to generate one first. Misread the docs.
Weirdly the authClient.token isn't in the types on mine.


Update service-bus-automation-manage.md

Updating the doc with some examples of PowerShell cmdlets that call the REST API to manage Service Bus.

Hi @csand-msft, I'm your friendly neighborhood Azure Pull Request Bot (You can call me AZPRBOT). Thanks for your contribution!

It looks like you're working at Microsoft (csand). If you're full-time, we DON'T require a contribution license agreement.

If you are a vendor, DO please sign the electronic contribution license agreement. It will take 2 minutes and there's no faxing! https://cla.azure.com.

TTYL, AZPRBOT;

Please see mail - these updates need to be validated and staged in the private repo.


Implement conversations methods

Added support for Conversations methods. Pagination is supported on some methods by passing in the next_cursor from a response to the cursor parameter.
Note that I've only tested some of the methods due to lack of permissions, so results may vary.

superfluous as far as I can see, cause it is already part of https://github.com/os/slacker/releases/tag/v0.13.0

OK, then close if the functionality is the same/better.

Closing this as it's already implemented. Thanks.


COMP: Fix unused variable warning

Fix unused variable warning.
Fixes:
Modules/Remote/Montage/include/itkMaxPhaseCorrelationOptimizer.hxx:305:10:
warning: unused variable 'confidenceFactor' [-Wunused-variable]

raised at:
http://testing.cdash.org/viewBuildError.php?type=1&buildid=5914663

Preferred to comment the line rather than to remove it since further down there is a related commented line and did not study the reason why both could have been used at some point/could be used at a later time.

@ joeykleingers Reviews are welcome. Thanks.


creating "install" from docker container

Hello.
I did:
docker run --runtime=nvidia -it floopcz/tensorflow_cc:ubuntu-shared-cuda /bin/bash

and see that the (pre)built binaries seem to be here:

  /usr/local/lib/tensorflow_cc/
       libprotobuf.a  libtensorflow_cc.so

and the header files here:
 /usr/local/include/tensorflow/tensorflow/
      core/public/version.h

So lib/tensorflow_cc and include/tensorflow/tensorflow should be copied (out of the docker container), and are all thats needed to create an "install" (dir) for any c++ projects that want to use tensorflow? Is there an installation script (even one that only operates inside the docker container) that automatically pulls out just what clients need and leave behind the rest,  that automates this (and will be updated automatically should the paths/contents etc change in future)?
Best Wishes and
Thanks in advance.

This is basically what make install does - it installs the required headers and libraries to the OS. There is a bit more that needs to be copied though, see here.


Generated C++ parser code does not compile for 32-bits build

When trying to compile the generated parser code as 32 bits (with the flag -m32), the following assertion fails:
daedalus/rts-c/ddl/cast.h:44:3: error: static_assert failed due to requirement 'sizeof(unsigned long) * 8 >= 64' "Unsupported cast: uint_to_integer"
   static_assert(sizeof(unsigned long) * 8 >= in,
   ^             ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~

The 32 bits build is desired because address sanitation is enabled and a 64 bits build in that case will allocate too much memory.

Good point, I was kind of assume a 64-bit architecture, I'll have to go through the RTS to see if I can make it work for 32 bits.   I don't have a lot of time to work on it right now though, is this a blocker, or more of a "nice to have"?

Without Address Sanitation, the memory errors that I am supposed to find will not be detectable. I have not been able to run this with Address Sanitation in 64-bits mode, due to the large amount of memory it requires. So for me, this is a blocker indeed, sorry about that :-)

OK, I'll have a look in the next few days.  I think the main challenge I can think if is figure how to interface with libgmp

Thanks. In the past I was able to build generated C++ parser code with -m32 and at that time I already ran into a dependency on libgmp. It did not seem to be a problem: using Ubuntu, I just had to install the package libgmp-dev:i386. I never saw that static_assert fail though (but I was using a different DDL).

Yeah, I think it should build just fine.  The main issue I need to work around is how to convert from a libgmp big integer to a uint 64.  libgmp has a function to give you an unsigned long, and I was assuming that this would be 64 bits, but I can add some more smarts to do something different depending on sizeof(unsigned long).

@reiniert while I am working on 32-bit support, does using Valgrind help with looking for memory errors, or does it also fall over on the examples you you using?

I have not tried valgrind, I do not think it plays well with the fuzzer (AFL++) which I use to find the problems. It may be good for triaging the crashes once found though.

@reiniert sorry for the delay, I just came back from vacation.  This is certainly on my TODO list, and I am hoping to have something working fairly soon, and I'll update the ticket when I do.

@yav I finally got to try this and my code seems to compile with -m32 now, thank you. I wrote "seems to" because I have not actually seen the build step complete because my machine is running out of memory. This may be due to the generated C++ parser code being a little under 30K lines long and the fuzzer version of the compiler is ram-greedy. I will explore further.


Fix appveyour

Pull Request Test Coverage Report for Build #53

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

Totals

Change from base Build #45:
0.0%

Covered Lines:
593

Relevant Lines:
903

💛  - Coveralls


Installed databroker and its dependencies #220

The problem with this was the use of an old pipenv, which was unable to lock. I used a new version of pipenv installed locally to lock. The default dls version will work for installing and setting up the environment so long as the new Pipefile.lock is present.

I've addressed these suggestions in this latest commit.

@DominicOram  For me your test code above failed because the environment lacked a packages dask, bson, cachetools, pymongo - required for databroker.temp . I've added it and it now works.
I have no idea why these dependencies aren't included as requirements in the traditional way - perhaps because they're optional?

@DominicOram  For me your test code above failed because the environment lacked a packages dask, bson, cachetools, pymongo, cachey, fastapi, jmespath, jose - required for databroker.temp. With these additions you'd think it would work, however there is an error in the version of jose imported with a print statement not closing off a bracket :thinking:
I have no idea why these dependencies aren't included as requirements in the traditional way - perhaps because they're optional?

After including many more optional packages, the test code still fails with the package mongo-suitcase when it tries  to import suitcase.mongo_normalized, which should work as the repo shows.
It might be because of another dependency which is also called suitcase.

@DominicOram Your test code now works, should I add it as a unit test?

I think given the amount of dependencies this adds it's not worth installing the databroker as we probably won't need it for now


Display aberrant ability source location in button prompt

Fixes #1154.

I'm fine with this but sounded like @cryogen had some objections from the last PR.


Doesn't work

I paste the whole folder in the "Packages" of my Sublime Text3 but there's no that submenu, I wonder what's wrong, thank you. It's a very cool function

The best method is to navigate using Sublime Text menu -> Preferences -> Browser Packages.
There you can paste this plugin folder.


adding anantshri to maintainers list

Please confirm this pull request meets the following requirements:

[X] I followed the contributing guidelines: https://github.com/github/india/blob/main/CONTRIBUTING.md.

Which change are you proposing?

[ ] Adding/updating open source project/s
[X] Adding/updating maintainer/s

Adding/updating open source project/s

[ ] The project was built in or receives significant contributions from India
[ ] The project has at least 100 stargazers

Why should this open source project be featured?

Please replace this line with an explanation of why you think these changes should be made.

Adding/updating maintainer/s

[X] Maintainer's location on their GitHub profile indicates as India
[X] Maintainer has approved this PR to be included on the website

What open source project/s does the individual maintain?
https://github.com/anantshri/wp-security
https://github.com/anantshri/svn-extractor
https://github.com/anantshri/Android_Security

Please replace this line with GitHub URLs to projects that the individual/s maintains

Something that does not neatly fit into the binary options above

[ ] My suggested edits are not about an existing page, or at least not a single one

Please replace this line with an explanation of your proposed changes.

Please note: we will close your PR without comment if you do not check the boxes above and provide ALL requested information.

hey @mvkaran seems like last build was "Sun, 25 Sep 2022 14:21:56 GMT" can you rebuild the site so new added maintainers can be listed on website.


npm install on Angular 11 - peer dependencies and not working

I'm submitting a ...
[x] bug report => search github for a similar issue or PR before submitting
[ ] feature request
[ ] support request => Please do not submit support request here, post on Stackoverflow or Gitter

Current behavior
I am attempting to install ngx-datatables in a project running Angular 11.
npm install @swimlane/ngx-datatable

After install I get the following warnings:
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.
npm WARN<EMAIL_ADDRESS>requires a peer of<EMAIL_ADDRESS>but none is installed. You must install peer dependencies yourself.

My packages.json has the following:
"@angular/common": "~11.2.6",
"@angular/core": "~11.2.6",
"@angular/platform-browser": "~11.2.6",
"rxjs": "~6.6.0",

When I add the import to app.module.ts and run my app, I get the following errors:
[error] Error: The target entry-point "@swimlane/ngx-datatable" has missing dependencies:
 - @angular/core
 - @angular/common
 - rxjs
 - rxjs/operators

So why am I having an issue?
Expected behavior
No warnings, and the project runs
Reproduction of the problem
What is the motivation / use case for changing the behavior?
Would like to have a working program.
Please tell us about your environment:
Windows 10, Visual Studio 2019, Angular 11
Angular CLI: 11.2.6
Node: 14.16.0
OS: win32 x64
Angular: 11.2.7
... animations, common, compiler, compiler-cli, core, forms
... localize, platform-browser, platform-browser-dynamic, router
Ivy Workspace: Yes
Package                         Version
@angular-devkit/architect       0.1102.6
@angular-devkit/build-angular   0.1102.6
@angular-devkit/core            11.2.6
@angular-devkit/schematics      11.2.6
@angular/cli                    11.2.6
@schematics/angular             11.2.6

Browser Chrome

Language: all

I updated rxjs to 6.6.7, and so that warning went away.  Still left with the other 3.

Are you able to provide an exampkel repo?

@Hypercubed not sure what exactly you are asking for?

From the description it should work.  You have the peers installed.  Do you have a full repo I can look at?

I updated rxjs to 6.6.7, and so that warning went away. Still left with the other 3.

thanks you, i fixed issue it


Fix allergies UUID

The CI didn't trigger for some reason so I missed this issue in my earlier PR.

Looks like the issue was that I had two robot-simulator exercise entries from the earlier resorting perhaps?


Updated index

Description
Home page fixed #176  and added the #165 issue stated features. Because #172 PR was aborted and there is no such feature till now in the project.
Type of change

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

Explain the Testing instructions
Test Configuration:

Operating system:
Version:
Text-editors used:

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented my code, particularly in hard-to-understand areas

ATTACH SCREEN-SHOTS / DEPLOYMENT LINK
Refer to PR #172

@Ayan-thecodeking What is the problem with the PR. I am not understanding. Can't it be merged?

@Ayan-thecodeking What is the problem with the PR. I am not understanding. Can't it be merged?

no issues are there @sr-sweta ..just ur PR is in hold ....last time ur PR has created some issues in the home page thats why i have to revert that PR.....
I'll add this PR

@Ayan-thecodeking What is the problem with the PR. I am not understanding. Can't it be merged?

no issues are there @sr-sweta ..just ur PR is in hold ....last time ur PR has created some issues in the home page thats why i have to revert that PR.....
I'll add this PR

Ok, Thankyou


Running prisma generate with missing generator dependency installed leads to CLI crash

C:\Users\Jan\Documents\throwaway\erd-49>npx prisma generate
Environment variables loaded from .env
Prisma schema loaded from prisma\schema.prisma
Error: spawn prisma-erd-generator ENOENT

same problem. did you solved?


Tidy up the rheology code

On first looking at the rheology code, it looks like an horrendous mess.
However, I suggest that by a few simple typedefs, the code can be made to look more like physics and less like C++.
For example, here is some of the core calculation of the function StressUpdateHighOrder() in BBM.hpp:
            const Eigen::Matrix<double, 1, NGP* NGP> powalphaexpC
                = (d_gauss.array() * expC.array()).pow(params.exponent_relaxation_sigma - 1);
            const Eigen::Matrix<double, 1, NGP* NGP> time_viscous
                = params.undamaged_time_relaxation_sigma * powalphaexpC;

            //! BBM  Computing tildeP according to (Eqn. 7b and Eqn. 8)
            // (Eqn. 8)
            const Eigen::Matrix<double, 1, NGP* NGP> Pmax
                = params.P0 * h_gauss.array().pow(params.exponent_compression_factor) * expC.array();

I would argue that adding a typedef Eigen::Matrix<double, 1, NGP * NGP> GaussVar; makes the physical equations that are being evaluated much clearer:
        // Eqn. 25
        const GaussVar powalphaexpC = (d_gauss.array() * expC.array()).pow(
                params.exponent_relaxation_sigma - 1);
        const GaussVar time_viscous = params.undamaged_time_relaxation_sigma * powalphaexpC;

        //! BBM  Computing tildeP according to (Eqn. 7b and Eqn. 8)
        // (Eqn. 8)
        const GaussVar Pmax = params.P0 * h_gauss.array().pow(params.exponent_compression_factor)
                * expC.array();

because the code is no longer dominated by the types and template arguments. To me, even as someone experienced with C++, this is easier to read.
There may be other renamings and rearrangings that are possible to make the code in these functions even clearer.

The variable name GaussVar could of course itself be changed to something more accurately descriptive.

I'm especially interested in comments from @winzerle on this.

Using a typedef is no problem and of coarse, will help the readability. This type is always a vector of values in the Gauss points within the element. So GaussVar is ok, or maybe GaussVector.
In different parts of the code such a vector appears for values in the elements and for values on the edges. Once it is NGP*NGP as template argument, one just NGP.
There is another thing: to avoid linking problems the value of NGP always has to match. I see that this function still locally uses
#define NGP ( ((DGstress == 8) || (DGstress == 6) ) ? 3 : (DGstress == 3 ? 2 : -1))
I would prefer that we always use
#define GAUSSPOINTS(Q) (( (Q == 8) || (Q==6) ) ? 9 : (Q == 3) ? 4   : (Q==1) ? 1 : -1)
#define GAUSSPOINTS1D(Q) (( (Q == 8) || (Q==6) ) ? 3 : (Q == 3) ? 2  : (Q==1) ? 1 : -1)
from include/NextsimDynamics.hpp
It could then be
typedef Eigen::Matrix<double, 1, GAUSSPOINTS > GaussVector
typedef Eigen::Matrix<double, 1, GAUSSPOINTS1d > GaussVectorEdge
In cleaning up the code there is just one important issue: we have to avoid using the keyword "auto" with Eigen. This will kill the efficiency.

we have to avoid using the keyword "auto" with Eigen.

Good to know!

Tidied up as part of PR #477.


[stable/docker-registry] Add affinity for pod assignment

What this PR does / why we need it:
This PR adds affinity for for pod assignment for [stable/docker-registry]
Special notes for your reviewer:
No breaking changes, this is PR just exposes std kubernetes pod/node scheduling mechanism. The chart already has toleration and node selectors, and this PR extends the pod assignment with affinity.
Checklist

[x] DCO signed
[x] Chart Version bumped
[x] Variables are documented in the README.md

/assign @jpds

/ok-to-test

/assign @rendhalver

@redbaron  @jpds any comments on this?


cleanup: fix tests

These should be fixed as a part of https://github.com/angular/material2/commit/3c6f7a276e58cc52ffa3f1b43d1828db49be9ce4.


UwU @albertonl mentioned you in a pull request

uwu @MatejMecka idk

UwU this is a bit weird isn't it?

UwU shut up

pls forgive my notifs uwu

uwu we're making of the world a better place thanks to our open source so no one shuts up uwu

's/tehc/upen sores'

uwu

can we please stop now uwu

UwU you have a new notification

uwu no one stops uwu

uwu?

bye.

uwu rip @shreyasminocha


Make index files look like a datatype of index1

Done in https://github.com/m-lab/jostler/pull/28.


Hash value collision occurs in processing in PropertyInfoRegistry.

@Override
public int hashCode() {
  int result = 31 + initialType.hashCode();
  result = 31 * result + propertyName.hashCode();
  result = 31 * result + configuration.hashCode();
  return result;
}

It seems that the return value of hashCode () may be the same in different class and method.
e.g.
hashCode of Class A is 736630696.
hashCode of Method A in Class A is -353765065.
hashCode of Class B is -68707585.
hashCode of Method B in Class B is -1158082130.
hashCode of configuration is 1.
If the hash value is the above combination, the calculation result will be as follows.
hashCode of ClassA#MethodA is 31 * (31 * (31 + 736630696) - 353765065) + 1 =<PHONE_NUMBER>
hashCode of ClassB#MethodB is 31 * (31 * (31 - 68707585) -<PHONE_NUMBER>) + 1 =<PHONE_NUMBER> (same!!)
ClassA a1 = someMethod();
ClassA a2 = new ClassA();
ModelMapper mapper = new ModelMapper();
mapper.map(a1, a2);

=> org.modelmapper.MappingException is occur.
What should I do in such a case?
For example, how about executing the hash value calculation as follows?
String key = initialType.getTypeName() + "#" + propertyName + "#" + Integer.toString(configuration.hashCode());
return key.hashCode();

I think the org.modelmapper.MappingException is not related to the hash code collision. Hash code collision only affects the performance when we lookup a cached value from the hash map because we always check the equality.
Can you provide more information?


fix: change leetcode domain

change leetcode domain from "leetcode-cn.com" to "leetcode.cn".
btw, maybe we can add support for "leetcode-us"?

lgtm


Card image overflow

When I click to view a card, it looks like this...

I thought I had this fixed, but apparently regressed when I fixed a different, worse graphical bug. Should be an easy fix.


Can't specify interface/IP for controller, leading to invalid apiserver certificate

Describe the bug
I have 3 network interfaces. k3s server should be made reachable on the IP of eth2. k3s selects the IP of eth0 for use in the apiserver certificate though, making the certificate invalid to be used for connecting to the IP on eth2.
To Reproduce

Configure multiple interfaces
run k3s server
Connect to k3s via the IP of the second interface

Expected behavior
I should be able to configure the apiserver certificates IP addresses or the interface to use.

I believe this duplicates issue #72

Depends on the resolution, but yeah if one could specify interface it would also solve #72. Leaving it to @ibuildthecloud to close this one.

This should be addressed by the --tls-san flag from PR rancher/k3s/pull/239 and https://github.com/rancher/k3s/issues/214#issuecomment-478003830, if not please re-open.

That works, thanks!

@discordianfish can you explain how you were able to get it to work? which command did you use to install?


Blueprint circuit for the circuit library

Summary
Add a blueprint circuit base class to the mutable circuit library objects. Fixes performance issues in spirit of #4169 and provides a safe wrapping of the circuit data in a central place for all library object where this is necessary.
Details and comments
Since the attributes of some circuit library objects can be subject to change these circuits are constructed on-demand, i.e. when internal data such as .data is asked for, and marked for re-build when an attribute changes.
There are plenty of situations in algorithms where this mutability is required.

Variational Quantum Eigensolver (Chemistry)
class VariationalQuantumEigensolver:
    def __init__(self, guess_wavefunction: QuantumCircuit):
        # the guess wavefunction is an Ansatz for the ground state in form of a parameterized circuit
        # there are several structures that have proven useful (which are indep. of the num. qubits)

    def compute_minimum_eigenvalue(self, operator):
        # adapt Ansatz circuit to right size
        self.guess_wavefunction.num_qubits = operator.num_qubits
        # run algo and optimize over the parameters

Quadratic Binary Unconstrained Optimization (Optimization)
Equivalent situation to above, but now the entire minimization algorithm is passed and must run on an operator of unknown size.
class QUBOSolver:
    def __init__(self, minimum_eigensolver):  # such as the VariationalQuantumEigensolver
         # the minimum eigensolver is used as subroutine in the solve method

    def solve(self, operator):  # the operator represents the problem to solve
        # the eigensolver must work for any operator, therefore it must be able to
        # resize the guess wavefunction
        self.minimum_eigensolver.compute_minimum_eigenvalue(operator)

Quantum Amplitude Estimation (Standard algorithm)
class AmplitudeEstimation:
    def __init__(self, num_evaluation_qubits: int, qft: QuantumCircuit):
        # different flavors of the normal QFT exist
        # the number of evaluation qubits determines the accuracy of the estimation

    def set_accuracy(num_evaluation_qubits):
        # change the accuracy without re-instantiating a new object

    def run(self):
        # uses QFT of size num_evaluation_qubits

To indicate that a circuit has not been constructed yet, the _data attribute is set to None. If the circuit is constructed, the internal data is populated. This can lead to problems if the _data member is accessed without building the circuit first, e.g.
circuit = LinearPauliRotations(num_qubits=4, slope=2)
circuit.count_ops()  # counts ops in _data, which is still None

In principle it would be sufficient to replace all _data occurences with data and ensure that the circuit is build in the data getter. However the getter contains extra validation and returns a QuantumCircuitData object, which is much slower than referring to the list of _data. To maintain the performance of the quantum circuit, we should not change the circuit class to use data but create a new intermediate circuit object that ensures the circuit has been built before it is used.
Since these circuit structures contain information on how they are constructed, like a blueprint, BlueprintCircuit seems like a suitable name. (LazyCircuit would also be descriptive of the "laziness" only, not the structure-manual and, I think, sounds less appealing.) But that can of course be discussed!
It is also open for discussion if this laziness is the correct way to go in the future. Maybe the circuits should be immediately rebuilt on change. Or maybe the circuit object itself should become "lazy" in the same sense? Clear is, we need this functionality and the circuit library and the next release of Aqua are coming very soon, therefore this provides a suitable solution.

#4169 made some changes to QFT.append() and QFT.qasm(), do they have to be reverted here?

Yep that can be reverted!


perf(consensus): add simplistic block validation cache

Closes #2854
Follow up from #2960
This PR adds a simplistic 1-element block validation cache in BlockExecutor. This addresses a performance bottleneck raised as part of #2854

PR checklist

[ ] Tests written/updated
[ ] Changelog entry added in .changelog (we use unclog to manage our changelog)
[ ] Updated relevant documentation (docs/ or spec/) and code comments
[ ] Title follows the Conventional Commits spec

Do we want to backport this?

@melekes AFAIU, @ValarDragon and team need this, but they are cherry-picking to their fork (they just want to know we're up-streaming these changes, so they'll eventually disappear from their fork; @ValarDragon please chime in if that's not the case). So I see no point in backporting to v0.37. or v0.38.x. As for v1.x we are about to cut RC1 (we are long past "code freeze", which we reached when we carried out QA), so backports to v1.x should be done under the same conditions as the older branches at this point.
Changes to consensus a delicate, I'd prefer to backport only if needed (bug, security issue, etc) or if totally harmless (doc, comments, logs, etc).

I'd like to get this backported to v0.38.x if folks are fine with this. Our standing plan was to do an SDK 50 upgrade and be pretty on par with v0.38.x . (I am not caught up on whats the latency / breaking behavior of v1.x vs v0.38.x)

@ValarDragon For backporting changes -- that are not bugs, or security related -- to release branches, our policy is to backport to the corresponding *-experimental branch. In the case of v0.38.x, the branch is v0.38.x-experimental. The difference is that we do not guarantee that the code changes landing on the *-experimental branch underwent any quality assurance (IOW, the *-experimental branches are not considered code-frozen). The advantage is that it's us who carry out the backport (solving conflicts, adapting where necessary) rather than the users.
Is backporting this to v0.38.x-experimental good enough for you?

Yes!

In a patch release and working great!


3-0-stable version

Are you guys planning on 3-0-stable version which points to spree 3-0 branch.
Current spree_reviews master points to 3.1.0.beta and there's no 3-0-stable branch. We're planning on upgrading to spree 3-0-stable branch and we're stuck here.

Hi @jdatti, a 3-0-stable branch now created. Happy coding!

@futhr Thank You. You guys rock.


Full screen bug on macos

Does not "Fullscreen"

This is generally something that happens when loading minecraft, it should fix itself once you get to the main menu


the future of this repo: updates, maintenance, and ownership

I want to spend a bit of time thinking about the future of this repo, especially since issues like #36 are being raised.
I think this has been a very influential project:

it was ACM's first public-facing internal training, and led to participants like @matthewcn56 joining us!
it motivated future projects like centralized-intern-training, teach-la-dev-training, and tla-dev-intern-training-w21
this model of write + record became very helpful for accessibility concerns

It's also still the only no-experience full training plan we have, and one of the few fully-recorded training sessions.
However, it's showing its age, and there is no active maintenance. Some things I want to specifically call out:

we should be teaching React hooks-first at this point
we need to update how create-react-app works - both in the template code it generates and eschewing the CLI for npx
we should be updating syntax for major version bumps for things like Jest, Express, Firebase (this is a big one)
we probably need to stop recommending enzyme given its lack of activity, and switch to something like testing-library
we should probably spend a bit of time talking about "new" ecosystem developments: CJS versus ESM, yarn (and pnpm?), deno/bun, Next, etc.
I'm still greatly unhappy with how I taught lesson 3, especially since we now don't need to teach classes (really) and can probably make a more concise curriculum set

I think this presents several problems.

I don't like just archiving this project, since:

even if we replace it with something else, we lose a lot of history
we currently don't have a drop-in replacement (in my personal opinion); in particular, I think there's significant utility from having one coherent track instead of making people do multiple different external courses (which often don't overlap well)
I'm still quite proud of our impact, and it's used by others (especially due to the high star count)

However, nobody maintains this.

I'm too far removed from ACM at this point to be the point person, though I'm happy to help out.
Internally, ACM is split across several repos: the ones I linked above, + Hack's stuff, + whatever external recommendations we make.
Obviously, ACM's dev team + Teach LA's dev team aren't in the same place they were 2 years ago, and each have many more obligations.

But, if we do nothing, we're actively creating bad learning experiences.

Information is inaccurate/frustrating,
and we're splitting attention

So, what do we do? Obviously the best case is we somehow do a stellar job maintaining this without dropping any other ACM commitments, but that seems rather unlikely and unfair towards the current ACM people. It would be great if I could get some current officers to chime in.
(thanks for bearing with this rant)!
cc: @matthewcn56 @doubleiis02 @jainsujay02

My thinking is that we should focus on maintaining repos that we actively use as part of our workshops.

I want to focus on projects that don’t have free/easily accessible alternatives. The topics covered in this particular Repo can mostly be learned from Coursera and other MOOCs. (I'll admit that I couldn’t find anything as comprehensive as this repo quickly)
Second, I’m reluctant to request people to maintain learning repos that we are not actively using because they will miss out on the connections formed from in-person activities/teaching.

Nevertheless, I recognize the value of this particular repo so I don’t mind if someone from ACM/dev team wants to work on it but I am content to let it be archived.

I really love how the learning-lab-crash-course helped offer me an introduction to web development and am grateful for it :). Similarly to what should be done for the QWERHacks Firebase Workshops I believe that updating long-term learning tools is the better alternative to creating new ones (something I am guilty of unfortunately.)
With regards to how we can go about updating it without placing too much extra commitment on people's shoulders, I think that we can take a similar approach to how we have updated legacy projects in the past for TeachLA.

As committees like TeachLA, Hack, and the Dev Team teach their members with new training, a common problem is that there aren't that many hands on ways that people can apply what they've learnt. What we've done for TeachLA is that common exercises for people after training is giving them issues to update old projects. Specifically, I want to call out the great work that's been done on the TeachLA editor's front-end with refactoring and turning older Class-based style of react components into the more modern functional hooked versions.

I believe this approach of offering open-source issues for people to update curriculum for tools such as React and Firebase are a great hands on way for people to practice and not put too much work on officers who already have roles and responsibilities.

For updating non-React parts like revamping how JS is taught, new ecosystem developments and any curriculum in general, a lot of this content is also covered by individual committees.

I think a good way to provide intern projects this year for TeachLA's training director interns (and possibly the dev team training interns as I am thinking of creating a training director role for the Dev Team as well :eyes: ) could work on revamping sections that we believe need tuning up and covering them as workshops for their respective committees.

Our centralized intern training plan for web development draws a lot of inspiration from the learning lab crash course for the introductory HTML/CSS/JS sections, and in large part that is because I haven't found the need to change how things are taught for the introductory sections. I echo your sentiment on changing how JS was taught and found myself not going over everything from the JS/React section and only teaching what was core to web development principles and react in the live version.
I feel like the web development centralized training repository, the yearly hackschool repos, TeachLA dev training and this crash course have a lot in common in terms of providing a one stop shop for web development. TeachLA now just pulls from the TeachLA Dev Training Repo every year  and I'm guessing the Dev Team and all the interns interested in web development just learn from the centralized-intern-webdev-training repository (ultimately up to the intern training director but I'll def give my opinion next year as well!) which is great for centralization on the committee/intern scale, but now the larger problem is what should we push towards for people to learn web development from ACM as a whole?
I feel that we should decide one of these tools that we would want to push as the de-facto content to represent ACM and spend time touching that one up and updating it as a living document. This repo offers a lot of great content from the HTML/CSS/JS perspective, TeachLA's dev training repo covers a lot of extra backend guides and advanced react guides, and  JSChats (and each individual guest training sessions which I loved like Omar's TS session) offers a lot of great lessons in the latest advancements of web devlopment, while tools like HackSchool and the centralized-intern-training plan offer concise ways to cover the absolute necessities of all of these tools. Now the question lies in how do we combine all these great resources together!

After reading @matthewcn56's thinking, I'd like to propose that we just maintain one comprehensive web-dev training repo that

covers all the common concepts, frameworks, and tools that we (TLA Dev Team, ACM Dev Team, Hackschool, and others) teach
is not geared towards any one part of ACM but focuses on teaching the content thoroughly
is used as a foundation course with independent modules that are forked by different committees/training directors to cover the concepts that are directly relevant to their current workshop and they can then add lessons (if necessary) that are only relevant to their workshop (I imagine there won't be too many).

I also like Matt's idea to have interns revamp parts of the repo as their intern project. I have always found that teaching someone else a concept reinforces my own learning so it should pan out!
Of course, since this learning lab repo has the most detailed content, it would make sense to use this as our base and update/add/re-write lessons where necessary.
Needless to say, for my idea to work, there will have to be buy-in from all of the teaching/dev team directors, but I believe this approach should improve the quality of our training while reducing the burden of maintaining legacy repos.
What do you think @matthewcn56 @doubleiis02 @mattxwang?

Just with a month left in the summer - want to give a chance for @doubleiis02 to chime in first! I do have some thoughts but I'll save them for a bit 😊

Thank you everyone for providing your valuable opinions and being patient
for my response.
Like Matt Nieva, I also used this repo to get started with web development
and found it very helpful in getting me started on projects :) That said, I
definitely agree with the issues Matt Wang brought up, not only the fact
that it's outdated but also that some of the tools that the workshop series
teaches is not as relevant to Teach LA's projects (or in general, ACM's
projects) anymore. I'm pretty sure I did not end up watching all of the
videos, probably only the first ones about html/css/js and react, but I
still found that enough to get started.
In addition, our current training directors last spring mostly relied on
the content from the teach-la-dev-training
https://github.com/uclaacm/teach-la-dev-training repo instead of this
one. I feel that this repo has a good amount of useful content, and out of
all the options I would vote to continue using this repo over the summer
2020 repo. I really like Matt's suggestion to have training interns work on
updating the repo!!
As for Sujay's suggestion to create a new generalized repo for not only
Teach LA's dev team but also for the ACM dev team, Hackschool, etc; I like
the idea and there definitely is a lot of overlap. It would definitely be
useful to have the resources stored in one place rather than having to
repeat information in different locations. The only thing is that the three
teams have unique purposes for their training. I imagine that even if we
end up creating one centralized repo, Hackschool may still create their own
repos since their training is more of a workshop track for a large audience
and they need a place to store recordings, slides, etc. However if we do
decide to work on this, it would make the ACM-wide intern training
workshops much easier to plan for and develop!
Given all of this, here is what I think:

option 1: update teach-la-dev-training
https://github.com/uclaacm/teach-la-dev-training

add new workshops that the Teach LA dev team creates to this repo
migrate and organize existing workshops that the Teach LA dev team
has created to this repo
*pros: *most efficient method / less work for our training
directors, repo remains catered to the Teach LA dev team's
training style,
more likely to be maintained since Teach LA training directors will have
ownership
*con: *we will have to collaborate with the other teams for the
general ACM intern training anyways later on

option 2: create centralized repo for ACM's webdev training

do what option 1 does but also gather and organize resources from
other places
*pros: *make centralized ACM intern training an easier task, have a
single repository for all of ACM's webdev content, ideally very robust
cons: more effort and communication required, some teams may work
on this repo and then also end up creating their own, more dispersed
ownership and less likely to be maintained, repo may get quite big or
harder to organize

For both of these options, we would have the training directors + interns +
anyone else who creates a workshop participate!
On Fri, Aug 19, 2022 at 3:23 PM Matt Wang @.***> wrote:

Just with a month left in the summer - want to give a chance for
@doubleiis02 https://github.com/doubleiis02 to chime in first! I do
have some thoughts but I'll save them for a bit 😊
—
Reply to this email directly, view it on GitHub
https://github.com/uclaacm/learning-lab-crash-course-su20/issues/37#issuecomment-1221134279,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AMC7L2M6A7UGVTJF2VLARDLV2ACPVANCNFSM53NAGG2A
.
You are receiving this because you were mentioned.Message ID:
@.***>


Default GestureDetector Hit Test Behavior of deferToChild Is Counterintuitive

Steps to reproduce

Create a text widget with some padding around it and wrap it in a gesture detector
Tap on the visible text and note that the on tap is fired
Tap on the padded space immediately next to the text

Expected results
The on tap is fired in step 3
Actual results
The on tap is not fired in step 3
Why this is a problem
This is "working as intended" because the default hit test behavior for GestureDetector is deferToChild and Padding does not consider the padded space for hit testing. This is reasonable if not desirable behavior in many circumstances, but it is not intuitive enough to be a sane default. Developers who are unaware of this nuanced behavior introduce usability issues everywhere in their app by creating text buttons with much smaller tappable regions than intended. Ideally, developers would just use InkWell or TextButton, but in practice the vast majority of major companies do not want to be that tied to Material and choose to write their own custom button wrappers as a part of their company-specific design system.
Proposal
Make HitTestBehavior behavior a required argument with no default value. This will be slightly more annoying for developers, but it will force them to consciously choose the desired hit test behavior. IMO the increase in verbosity is more than justified by the reduction in developer confusion and bugs-explicitness is pretty reasonable here given that GestureDetector is a comparatively low level widget and if you want ease-of-use you should probably be using the more ready-made equivalents such as InkWell, etc.
Code sample
Code sample
return MaterialApp(
  title: 'Flutter Demo',
  home: Scaffold(
    body: Column(
      mainAxisAlignment: MainAxisAlignment.center,
      children: [
        // The padding around the button does not detect hit events because the
        // default hit test behavior is `deferToChild` and `Padding` doesn't
        // count padded space for hit testing.
        GestureDetector(
          onTap: () {
            print('Button A pressed!');
          },
          child: const Padding(
            padding: EdgeInsets.all(40),
            child: Text('Button A'),
          ),
        ),
        const Divider(),
        // Setting `behavior` to translucent fixes this.
        GestureDetector(
          behavior: HitTestBehavior.translucent,
          onTap: () {
            print('Button B pressed!');
          },
          child: const Padding(
            padding: EdgeInsets.all(40),
            child: Text('Button B'),
          ),
        ),
      ],
    ),
  ),
);

Screenshots or Video
No response
Logs
No response
Flutter Doctor output
Not relevant.

This is probably a big enough breaking change that it would have to be something like:

https://github.com/flutter/flutter/issues/24722

For the near term maybe we should explain this in the docs, though. A lot of people will probably still get tripped up but maybe it's better than nothing.


proxy status: add more mapping to proxystatus

DurationTimeout -> ConnectionTimeout
UnauthorizedExternalService -> ConnectionRefused
DownstreamConnectionTermination -> ConnectionTerminated
OverloadManager -> ConnectionLimitReached
DropOverLoad -> ConnectionLimitReached
FaultInjected -> HttpRequestError
Commit Message:
Additional Description:
Risk Level: low
Testing:
Docs Changes:
Release Notes:
Platform Specific Features:
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/assign @yanavlasov

/retest

/retest


[Doc Feature] [Update channel meetings]

Docs Build status updates of commit 51b973d:
:warning: Validation status: warnings

File
Status
Preview URL
Details

msteams-platform/TOC.yml
:warning:Warning
View
Details

msteams-platform/concepts/app-fundamentals-overview.md
:warning:Warning
View
Details

msteams-platform/apps-in-teams-meetings/teams-apps-in-meetings.md
:warning:Warning
View
Details

msteams-platform/whats-new.md
:warning:Warning
View
Details

msteams-platform/get-started/glossary.md
:warning:Warning
View
Details

msteams-platform/apps-in-teams-meetings/meeting-appextensibility.md
:white_check_mark:Succeeded
View

msteams-platform/assets/images/apps-in-meetings/PostMeeting.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/apps-in-meetings/PreMeeting.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/apps-in-meetings/in-meeting-dialog.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/apps-in-meetings/mobilepremeeting.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/apps-in-meetings/permission-for-add-remove-apps-in-meeting.png
:white_check_mark:Succeeded
View

msteams-platform/assets/images/apps-in-meetings/permissions.png
:white_check_mark:Succeeded
View

msteams-platform/TOC.yml

Line 285, Column 11: [Warning: file-not-found - See documentation] Invalid file link: 'apps-in-teams-meetings/meeting-app-extensibility.md'.

msteams-platform/concepts/app-fundamentals-overview.md

Line 86, Column 80: [Warning: file-not-found - See documentation] Invalid file link: '../apps-in-teams-meetings/meeting-app-extensibility.md'.

msteams-platform/apps-in-teams-meetings/teams-apps-in-meetings.md

Line 36, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'meeting-app-extensibility.md'.
Line 45, Column 3: [Warning: file-not-found - See documentation] Invalid file link: 'meeting-app-extensibility.md'.

msteams-platform/whats-new.md

Line 114, Column 94: [Warning: file-not-found - See documentation] Invalid file link: 'apps-in-teams-meetings/meeting-app-extensibility.md'.

msteams-platform/get-started/glossary.md

Line 18, Column 3: [Warning: file-not-found - See documentation] Invalid file link: '../apps-in-teams-meetings/meeting-app-extensibility.md'.
Line 84, Column 3: [Warning: file-not-found - See documentation] Invalid file link: '../apps-in-teams-meetings/meeting-app-extensibility.md'.
Line 99, Column 3: [Warning: file-not-found - See documentation] Invalid file link: '../apps-in-teams-meetings/meeting-app-extensibility.md'.
Line 117, Column 3: [Warning: file-not-found - See documentation] Invalid file link: '../apps-in-teams-meetings/meeting-app-extensibility.md'.

For more details, please refer to the build report.
If you see build warnings/errors with permission issues, it might be due to single sign-on (SSO) enabled on Microsoft's GitHub organizations. Please follow instructions here to re-authorize your GitHub account to Docs Build.
Note: Broken links written as relative paths are included in the above build report. For broken links written as absolute paths or external URLs, see the broken link report.
Note: Your PR may contain errors or warnings unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them.
For any questions, please:Try searching the docs.microsoft.com contributor guidesPost your question in the Docs support channel

Acrolinx Scorecards
We currently enforce a minimum score of 80.
Click the scorecard links for each article to review the Acrolinx feedback on grammar, spelling, punctuation, writing style, and terminology:

Article
Score
Issues
Scorecard
Processed

msteams-platform/apps-in-teams-meetings/meeting-appextensibility.md
93
18
link
:white_check_mark:

More info about Acrolinx
Use the Acrolinx extension, or sidebar, in Visual Studio Code to check spelling, grammar, style, tone, clarity, and key terminology when you're creating or updating content. For more information, see Use the Visual Studio Code extension to run Acrolinx locally.


Fail to renew certificate when connecting via duckdns with the duckdns addon

I'm using the duckdns addon. Whenever I connect to the site via the duckdns site, it tells me that the certificate is expired (it worked correctly for at least a year before this).
When I restart the addon, it states that the certificate is still valid until December. What could cause a discrepancy between this?

Maybe a better question would be where I can find the certificate that is checked. Removing the pems in ssl/ does not trigger a regenerate.


0.4.15 requires Node 10+, which seems like a breaking change from 0.4.14

I'm upgrading an old project from<EMAIL_ADDRESS>When I hit the step to upgrade to 0.4.*, I went for 0.4.15 to see what worked. I then saw that this dependency exists:
<EMAIL_ADDRESS>-> ember-auto-import -> jsdom -><EMAIL_ADDRESS>Unfortunately<EMAIL_ADDRESS>uses optional catch binding, which means it requires Node 10+. It seems that adding ember-auto-import is the source of this issue.
It looks like at the time of publication, ember-cli-mirage0.4.15 may have been compatible with Node 8. But now it is not.
I don't see any mention of explicit Node version support in ember-cli-mirage. However, going from 0.4.14 to 0.4.15 means that the supported Node version does change.
I don't know if you want to fix this. Feel free to close if not. But if you do, maybe a 0.4.16 release that locks that dependency down would be in order.
Regardless, can you add a breaking change note to 0.4.15 saying this is the case?

Updated the release notes, sorry you ran into that!
Happy to merge a PR that locks that dep + publish as 0.4.16, if someone gets around to it 👍


Add seller parameters

Issue: #1
Changes:

Added seller property to user database model
Modified /register resource to handle seller option
Modified /store POST and DELETE methods to check if user is a seller. 403 status is returned if the user is not a seller.

Comments:

Not sure how to approach github registration regarding this feature

Thanks for the contributions , merged .
If you like the project please do star it , I will be working on this more and also might put up further issues that I wish to implement if you want to check those out.


update dcrlibwallet version, go fmt

Recent change to the dcrlibwallet-wip branch of dcrlibwallet led to build issues on godcr. This PR fixes that

I am unable to build
app/walletmediums/dcrlibwallet/dcrlibwallet.go:25:10: assignment mismatch: 2 variables but 1 values


doc: function and file descriptions

Issue being fixed or feature implemented

Adding descriptions to functions, structs, traits, and files as well as licensing.
What was done?

Adding descriptions to functions, structs, traits, and files as well as licensing.
How Has This Been Tested?

n/a
Breaking Changes

n/a
Checklist:

[ x] I have performed a self-review of my own code
[ x] I have commented my code, particularly in hard-to-understand areas
[ x] I have added or updated relevant unit/integration/functional/e2e tests
[ x] I have made corresponding changes to the documentation

For repository code-owners and collaborators only

[ ] I have assigned this pull request to a milestone

Good job!


ds4 controller triangle and square button not recognized

i have 2 ds4 contorllers, the controller that comes with the ps4 for some reason dont recognice the triangle and the square button, the rest of the buttons works fine. the new controller works fine, seems to be the version of the controller is not the same and led for some issue.... any idea??

#620
you are probably trying to use the obsolete version if you are downloading from this Jays2Kings/DS4Windows site. Read the linked post and follow the link to up-to-date version of DS4Windows app.


[hotfix][python] Change "flink-python-" to "flink-python" for the change  of artifact of flink-python module

What is the purpose of the change
*Because the artifact of flink-python has changed, the code in PackagedProgram can't find the flink-python_*jar. *
Brief change log

Changes "flink-python-" to "flink-python" in PackagedProgram

Verifying this change
This change is already covered by existing tests
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

@flinkbot approve all


STORM-2810: Fix resource leaks in storm-hdfs tests

https://issues.apache.org/jira/browse/STORM-2810
The tests are leaking at least threads and HDFS file locks. Instead of cleaning up everything manually, I've made sure threads are cleaned up, and have switched the tests that use locks to start a new DFS cluster per test instead of per test class.
I ran the tests on Travis 3 times without failure, and 100 times on a local Ubuntu VM.

Reviewers might want to add ?w=1 to the diff url to disable whitespace diffing, some of the modified files were autoformatted from 2-space indentation to 4-space.


chore: release 0.3.0

:robot: I have created a release *beep* *boop*
0.3.0 (2022-09-25)
Features

close notification automatically (#116) (0b0a1fc)
update continuous integration (#117) (2675d37)
update docs (#120) (be07421)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/kucera-lukas/rusty-battery/releases/tag/v0.3.0 :sunflower:


cannot make progress using cci+tcp

Using cci+tcp, HG_Hl_addr_lookup_wait() and the na_addr_lookup_wait() wrapper does not return. Further investigation shows that my code is hanging inside NA_Progress(). Have you seen this issue? Do you have a working example using CCI + TCP? Thanks!

Hmmm are you doing blocking progress?

Blocking progress? I only found one function to make progress: NA_Progress(). I didn't create a separate thread to call NA_Progress() though.

Yes. I tried MAX_IDLE_TIME which is 3600*1000.  I also tried 500 and 1000, HG_Hl_addr_lookup_wait() and NA_Progress() return in this case but no progress is made.

That seems weird, why are you calling both both NA_Progress and HG_Hl_addr_lookup_wait() ?

I am not calling them at the same time. I tried na_addr_lookup_wait() and HG_hl_addr_lookup_wait() seperately.

Could you please send me an example of what you're doing? Does it happen only with CCI?

yes it only happens with cci+tcp. bmi+tcp works fine. let me see if I can get you an example.

One difference with BMI is that CCI will try to connect whereas BMI only establishes connection at the time of communication. Is it possible that you're trying to connect to a target that is not reachable? Which string are you passing to lookup_wait? This may be related to the problem that @carns described in the pull request #73


got error (void-function string-prefix-p) at emacs 23.1 on centos 6.0

Installing emacs 23.3 on centos 6.0 is troublesome, I tried.
Here is the patch,
'''
commit ad2c6a95c3281c0e74bd2c5fba3e459053723de3
Author: chen bin<EMAIL_ADDRESS>Date:   Thu Dec 1 13:20:46 2011 +0800
add missing internal api for emacs 23.1

diff --git a/init.el b/init.el
index 2feabb8..f54c5c6 100644
--- a/init.el
+++ b/init.el
@@ -1,3 +1,13 @@
+;make .emacs.d support emacs 23.1 and above
+
+;only emacs 23.3 and above has string-prefix-p defined
+(if (not (fboundp 'string-prefix-p) )

(defun string-prefix-p (shorter longer)
(let ((n (mismatch shorter longer))

    (l (length shorter)))

 (if (or (not n) (= n l)) l nil)))

)

;; -- coding: utf-8 --
(add-to-list 'load-path (expand-file-name "~/.emacs.d"))
'''

BTW, how to send you pull request for only one commit on my branch

Thanks for you tips.
BTW,
As I can see from your branch, you does lots of web development, right?
If the answer is YES, could you publish your normal
workflow/toolchain/framework at your blog. People like me will
definitely learn much.
Best Regards,
Chen Bin
On Fri, Dec 2, 2011 at 3:59 PM, Steve Purcell
<EMAIL_ADDRESS>wrote:

Thanks - I'd been aware of this incompatibility for a few weeks, but I rarely use Emacs 23. I eventually placed the patched function in a different file.

--
help me, help you.

That's right. My workflow isn't anything unusual, though; these days I work mostly with Rails and Clojure, so the main tools are just rinari and slime. I'll write it up sometime, though.
Cheers,
-Steve


SrtSharp for .net7 and Android?

I am planning to use SRT and your wrapper would be very useful, the problem is that's just compiled for .net standard 2.0. Could you please compile it for .net7 and .net7-android?
If you've plans on opensourcing the wrapper I am more than happy to contribute.

Oh, I was thinking that I needed to create a .net7 compiled and other for android to provide the .so and .dll on each plafform. But if that's not needed then sorry I'll have a look :)

@BlackGad I am trying to use SrtSharp wrapper with .NET MAUI, in Windows it works well so far but then compiling for Android but it has a crash on any method call. I suppose as it couldn't found the .so library.

Having a look on SrtSharp package it looks like it has runtime just for linux-x64 and win-x64 but not for android (armv7 and x86). It is enough if If add these .so by hand in my project ? If so, from where should I compile srt to have it compatible with your wrapper?
Once again, happy on contributing with you if you've plans on opensourcing the wrapper :)

I see. First of all, .NET Standard is indeed compatible with .NET 7 Android. However, SrtSharp is a facade for the original SRT library created by SWIG, and it relies on runtime for specific platforms (Linux, Windows). All the source code is public and available; we even created a pull request to the official repo Haivision/srt#2577.
Currently, we are building this package internally using our CI pipelines. We are not actively developing for the Android platform at the moment. I will look into the feasibility of adding support for another platform, but the probability is quite low at this time.


[cloud_firestore_platform_interface] controller != true

Describe the bug
After updating flutterfire plugins I am getting this error occasionally
[ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: 'package:cloud_firestore_platform_interface/src/method_channel/method_channel_firestore.dart': Failed assertion: line 159 pos 12: 'controller != null': is not true.
To Reproduce
I am getting this error usually when I query a collection
Flutter doctor
Doctor summary (to see all details, run flutter doctor -v):
[√] Flutter (Channel stable, v1.17.5, on Microsoft Windows [Version 10.0.19041.450], locale en-US)
[√] Android toolchain - develop for Android devices (Android SDK version 29.0.0)
[√] Android Studio (version 3.6)
[!] IntelliJ IDEA Community Edition (version 2019.2)
X Flutter plugin not installed; this adds Flutter specific functionality.
[√] Connected device (1 available)
! Doctor found issues in 1 category.
 [ERROR:flutter/lib/ui/ui_dart_state.cc(157)] Unhandled Exception: 'package:cloud_firestore_platform_interface/src/method_channel/method_channel_firestore.dart': Failed assertion: line 159 pos 12: 'controller != null': is not true.
E/flutter ( 9482): #0      _AssertionError._doThrowNew  (dart:core-patch/errors_patch.dart:42:39)
E/flutter ( 9482): #1      _AssertionError._throwNew  (dart:core-patch/errors_patch.dart:38:5)
E/flutter ( 9482): #2      MethodChannelFirebaseFirestore._handleError
E/flutter ( 9482): #3      MethodChannelFirebaseFirestore._handleDocumentSnapshotError
package:cloud_firestore_platform_interface/…/method_channel/method_channel_firestore.dart:113
E/flutter ( 9482): #4      MethodChannelFirebaseFirestore._handleDocumentSnapshotEvent
package:cloud_firestore_platform_interface/…/method_channel/method_channel_firestore.dart:103
E/flutter ( 9482): #5      new MethodChannelFirebaseFirestore.<anonymous closure>
package:cloud_firestore_platform_interface/…/method_channel/method_channel_firestore.dart:38
E/flutter ( 9482): #6      MethodChannel._handleAsMethodCall
package:flutter/…/services/platform_channel.dart:409
E/flutter ( 9482): #7      MethodChannel.setMethodCallHandler.<anonymous closure>
package:flutter/…/services/platform_channel.dart:377
E/flutter ( 9482): #8      _DefaultBinaryMessenger.handlePlatformMessage
package:flutter/…/services/binding.dart:199
E/flutter ( 9482): #9      _invoke3.<anonymous closure>  (dart:ui/hooks.dart:290:15)
E/flutter ( 9482): #10     _rootRun  (dart:async/zone.dart:1184:13)
E/flutter ( 9482): #11     _CustomZone.run  (dart:async/zone.dart:1077:19)
E/flutter ( 9482): #12     _CustomZone.runGuarded  (dart:async/zone.dart:979:7)
E/flutter ( 9482): #13     _invoke3  (dart:ui/hooks.dart:289:10)
E/flutter ( 9482): #14     _dispatchPlatformMessage  (dart:ui/hooks.dart:164:5)
E/flutter ( 9482):

I had the same issue, after upgrading from 0.13 to 0.14
I fixed it by bumping the version of analytics in build.gradle from
implementation 'com.google.firebase:firebase-analytics-ktx:17.4.3'
implementation 'com.google.firebase:firebase-analytics:17.4.3'

to
implementation 'com.google.firebase:firebase-analytics-ktx:17.5.0'
implementation 'com.google.firebase:firebase-analytics:17.5.0'

Tried to reproduce several times, no such errorr
  firebase_core: ^0.5.0
  cloud_firestore: ^0.14.0

code sample
import 'package:cloud_firestore/cloud_firestore.dart';
import 'package:firebase_core/firebase_core.dart';
import 'package:flutter/material.dart';

void main() async {
  WidgetsFlutterBinding.ensureInitialized();
  FirebaseApp app = await Firebase.initializeApp();
  assert(app != null);
  print('Initialized default app $app');
  runApp(MyApp());
}

class MyApp extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return MaterialApp(
      debugShowCheckedModeBanner: false,
      title: 'Material App',
      theme: ThemeData.dark(),
      home: Home(),
    );
  }
}

class Home extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return Scaffold(
      appBar: AppBar(
        title: Text('Firestore Example'),
      ),
      body: Center(
        child: BookList(),
      ),
    );
  }
}

class BookList extends StatelessWidget {
  @override
  Widget build(BuildContext context) {
    return StreamBuilder<QuerySnapshot>(
      stream: FirebaseFirestore.instance.collection('books').snapshots(),
      builder: (BuildContext context, AsyncSnapshot<QuerySnapshot> snapshot) {
        if (snapshot.hasError) return new Text('Error: ${snapshot.error}');
        switch (snapshot.connectionState) {
          case ConnectionState.waiting:
            return new Text('Loading...');
          default:
            return new ListView(
              children: snapshot.data.docs.map((DocumentSnapshot document) {
                return new ListTile(
                  title: new Text(document.data()['title']),
                  subtitle: new Text(document.data()['author']),
                );
              }).toList(),
            );
        }
      },
    );
  }
}

flutter doctor -v
[✓] Flutter (Channel stable, 1.20.2, on Microsoft Windows [Version 10.0.19041.450], locale en-US)
    • Flutter version 1.20.2 at C:\Code\flutter_stable
    • Framework revision bbfbf1770c (6 days ago), 2020-08-13 08:33:09 -0700
    • Engine revision 9d5b21729f
    • Dart version 2.9.1

[✓] Android toolchain - develop for Android devices (Android SDK version 30.0.1)
    • Android SDK at C:\Code\sdk
    • Platform android-30, build-tools 30.0.1
    • ANDROID_HOME = C:\Code\sdk
    • Java binary at: C:\Code\android-studio\jre\bin\java
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)
    • All Android licenses accepted.

[✓] Android Studio (version 4.0)
    • Android Studio at C:\Code\android-studio
    • Flutter plugin version 48.1.2
    • Dart plugin version 193.7361
    • Java version OpenJDK Runtime Environment (build 1.8.0_242-release-1644-b01)

[✓] VS Code (version 1.48.0)
    • VS Code at C:\Users\Taha\AppData\Local\Programs\Microsoft VS Code
    • Flutter extension version 3.13.2

[✓] Connected device (1 available)
    • Android SDK built for x86 (mobile) • emulator-5554 • android-x86 • Android 10 (API 29) (emulator)

• No issues found!

Please add these in pubspec.yaml and try again
  firebase_core: ^0.5.0
  cloud_firestore: ^0.14.0

if the problem persist, Can you please provide your flutter run --verbose and a complete reproducible minimal code sample
Thank you

As I said it's occasional it happens if I query collections again rapidly according to me. I am not able to reproduce it on demand

@TahaTesser #3237

Okay so digging into this one, this assertion is thrown when a snapshot handler throws a native error, and for some reason the StreamController for that subscription no longer exists.
I manage to get this myself but couldn't replicate consistently.

Found the issue! PR linked.

Fix merged. Release will be out in the next hour or so.

Release with a fix is now published. Latest versions can be seen here.
Thanks again for the bug report :)

@Salakar  Thanks for fixing it quickly

Already fixed! Impressive, thanks for your hard work guys.

I just had this 'controller != null': is not true error. I'm on 0.14.0+2. Is it supposed to be fixed in that version?

Yeah it should be. If you still havre the issue can you reproduce it consistently?

I have only seen it once (emulator), and not sure when/where it happened. I'll let you know if it happens again.

I just had it again. Could have been after adding a document to Firestore (using .set), or in relation to a StreamBuilder using Firestore.
E/flutter ( 5308): [ERROR:flutter/lib/ui/ui_dart_state.cc(166)] Unhandled Exception: 'package:cloud_firestore_platform_interface/src/method_channel/method_channel_firestore.dart': Failed assertion: line 159 pos 12: 'controller != null': is not true.
E/flutter ( 5308): #0      _AssertionError._doThrowNew (dart:core-patch/errors_patch.dart:46:39)
E/flutter ( 5308): #1      _AssertionError._throwNew (dart:core-patch/errors_patch.dart:36:5)
E/flutter ( 5308): #2      MethodChannelFirebaseFirestore._handleError (package:cloud_firestore_platform_interface/src/method_channel/method_channel_firestore.dart:159:12)
E/flutter ( 5308): #3      MethodChannelFirebaseFirestore._handleQuerySnapshotError (package:cloud_firestore_platform_interface/src/method_channel/method_channel_firestore.dart:84:5)
E/flutter ( 5308): #4      MethodChannelFirebaseFirestore._handleQuerySnapshotEvent (package:cloud_firestore_platform_interface/src/method_channel/method_channel_firestore.dart:74:7)
E/flutter ( 5308): #5      new MethodChannelFirebaseFirestore.<anonymous closure> (package:cloud_firestore_platform_interface/src/method_channel/method_channel_firestore.dart:32:18)
E/flutter ( 5308): #6      MethodChannel._handleAsMethodCall (package:flutter/src/services/platform_channel.dart:432:55)
E/flutter ( 5308): #7      MethodChannel.setMethodCallHandler.<anonymous closure> (package:flutter/src/services/platform_channel.dart:385:33)
E/flutter ( 5308): #8      _DefaultBinaryMessenger.handlePlatformMessage (package:flutter/src/services/binding.dart:267:33)
E/flutter ( 5308): #9      _invoke3.<anonymous closure> (dart:ui/hooks.dart:282:15)
E/flutter ( 5308): #10     _rootRun (dart:async/zone.dart:1190:13)
E/flutter ( 5308): #11     _CustomZone.run (dart:async/zone.dart:1093:19)
E/flutter ( 5308): #12     _CustomZone.runGuarded (dart:async/zone.dart:997:7)
E/flutter ( 5308): #13     _invoke3 (dart:ui/hooks.dart:281:10)
E/flutter ( 5308): #14     _dispatchPlatformMessage (dart:ui/hooks.dart:156:5)
E/flutter ( 5308):

@joknjokn It'll be the StreamBuilder - could you share the code which you're using to stream the data?

It happens to me too. When I start the App from the VSCode-Debugger OR on the device (using the build apk).
But it works fine when using the Terminal and flutter run --verbose. Hope this helps somehow.

I had the same problem. This usually occurs when many items are fetched into the stream.
This makes it impossible to make most of the Flutter apps that use Firebase, so I think it is a problem that needs to be solved quickly.

package
firebase_auth: ^0.18.0+1
cloud_firestore: ^0.14.0+2
firebase_storage: ^4.0.0
firebase_admob: ^0.10.0-dev.1
firebase_messaging: ^7.0.0
cloud_functions: ^0.6.0
firebase_core : ^0.5.0
...

code
_streamSubscription = stream.listen((item) async {});
Here stream is collection stream.

error log
E/flutter (24517): [ERROR:flutter/lib/ui/ui_dart_state.cc(166)] Unhandled Exception: 'package:cloud_firestore_platform_interface/src/method_channel/method_channel_firestore.dart': Failed assertion: line 159 pos 12: 'controller != null': is not true.
E/flutter (24517): #0      _AssertionError._doThrowNew (dart:core-patch/errors_patch.dart:46:39)
E/flutter (24517): #1      _AssertionError._throwNew (dart:core-patch/errors_patch.dart:36:5)
E/flutter (24517): #2      MethodChannelFirebaseFirestore._handleError (package:cloud_firestore_platform_interface/src/method_channel/method_channel_firestore.dart:159:12)
E/flutter (24517): #3      MethodChannelFirebaseFirestore._handleQuerySnapshotError (package:cloud_firestore_platform_interface/src/method_channel/method_channel_firestore.dart:84:5)
E/flutter (24517): #4      MethodChannelFirebaseFirestore._handleQuerySnapshotEvent (package:cloud_firestore_platform_interface/src/method_channel/method_channel_firestore.dart:74:7)
E/flutter (24517): #5      new MethodChannelFirebaseFirestore. (package:cloud_firestore_platform_interface/src/method_channel/method_channel_firestore.dart:32:18)
E/flutter (24517): #6      MethodChannel._handleAsMethodCall (package:flutter/src/services/platform_channel.dart:432:55)
E/flutter (24517): #7      MethodChannel.setMethodCallHandler. (package:flutter/src/services/platform_channel.dart:385:33)
E/flutter (24517): #8      _DefaultBinaryMessenger.handlePlatformMessage (package:flutter/src/services/binding.dart:267:33)
E/flutter (24517): #9      _invoke3. (dart:ui/hooks.dart:282:15)
E/flutter (24517): #10     _rootRun (dart:async/zone.dart:1190:13)
E/flutter (24517): #11     _CustomZone.run (dart:async/zone.dart:1093:19)
E/flutter (24517): #12     _CustomZone.runGuarded (dart:async/zone.dart:997:7)
E/flutter (24517): #13     _invoke3 (dart:ui/hooks.dart:281:10)
E/flutter (24517): #14     _dispatchPlatformMessage (dart:ui/hooks.dart:156:5)

I've added a PR for this which protects against the crash. It's not a full solution but should help against these unwanted errors.

@Ehesp Please have a look at #3354


Update zio-interop-cats to 3.3.0-RC4

Updates dev.zio:zio-interop-cats from 3.3.0-RC1 to 3.3.0-RC4.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "dev.zio", artifactId = "zio-interop-cats" } ]

labels: library-update, early-semver-pre-release, semver-spec-pre-release, commit-count:1

Superseded by #55.


Adding Unique Constraints for Events

Hello,
We're working to allow for "UPSERTS" in the events tables in postgres, and need the following in order to run the upsert commands:
ALTER TABLE media_events ADD CONSTRAINT media_unique_constraint UNIQUE (action, event_time, object_id, actor_id, edapp_id) ;
ALTER TABLE assessment_events ADD CONSTRAINT assessment_unique_constraint UNIQUE (action, event_time, object_id, actor_id, edapp_id) ;
ALTER TABLE assessment_item_events ADD CONSTRAINT assessment_item_unique_constraint UNIQUE (action, event_time, object_id, actor_id, edapp_id) ;
ALTER TABLE grade_events ADD CONSTRAINT grade_unique_constraint UNIQUE (action, event_time, object_id, actor_id, edapp_id) ;
Do you think this is something that could be added to the initialization of rails when tables are created? Or do you see any issues with these constraints for the upsert logic within these postgres tables?

closed due to duplicate of issue 26


[DSA]: Introduction to Tries and Its Implementation in Cpp, Java or Python

Description
Introduction to Tries and Its Implementation in Cpp, Java or Python
Domain
DSA
Type of Contribution
Documentation
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of this project.

/assign

@nimisha-yadav Please assign this issue to me

Go ahead @priyanshu688

@nimisha-yadav Kindly review and merge this PR - https://github.com/priyanshu688/winter-of-contributing/pull/33

The deadline to create new PRs has passed. Thank you for all your contributions.


Difference function issse

Hi, I think the difference function is not working correctly.  It was reported earlier as well. Basically i would expect to see this result:
select difference([1,2,3],[1,2])
output: []

select difference([1,2],[1,2,3])
output: [3]

But in both 2.2 Alpha and 2.1.5, it just returns as if it performed "unionAll()" operation. The last known version it worked correctly was 2.0.8.
Regards

@luigidellaquila Any thoughts on this issue? Is it a valid bug? If yes when can we expect it to be fixed properly?
Similar issues were opened and closed with comments "Fixed" however the issue persists.

@lvca  How much longer should we wait for this issue to be resolved? It has already been more than a month.

@luigidellaquila Thanks for the work around. It works after we rewrite the query. Will wait for the final fix

I have two sets of records as follows:
select @rid from Consumer where deleted=false
result: 165102 records
select distinct(consId) as consId from (
              select expand($c) as consId
              let
              $a=(select inE('isClientOf').outV()[deleted=false] as consId from #73:3 unwind consId),
              $b=(select distinct(out) as consId from installs where active=true and out.deleted=false),
              $c=unionall($a, $b)
          )

result: 1070 records
So I need to obtain those records from Consumer (1st query) that are not in the 2nd query result:
  select expand(difference($cons, $cust))
  let
  $cons=(select @rid as consId from Consumer where deleted=false),
  $cust=(select distinct(consId) as consId from (
              select expand($c) as consId
              let
              $a=(select inE('isClientOf').outV()[deleted=false] as consId from #37:3 unwind consId),
              $b=(select distinct(out) as consId from installs where active=true and out.deleted=false),
              $c=unionall($a, $b)
          )
  )

result: 165102 records (wrong result, the expected value is 165102 - 1070 = 164032
Maybe my understanding of how difference() works is wrong, can you provide any tips?
Thanks

Any updates and/or workarounds on this?
All I have been able to do to make it work is the query below, but it's really slow:
select count(*) from (
  select expand($diff)
  let
  $cons = (select @rid as consId from Consumer where deleted=false),
  $cust = (select distinct(consId) as consId from (
              select expand($c) as consId
              let
              $a = (select inE('isClientOf').outV()[deleted=false] as consId from #37:3 unwind consId ),
              $b = (select distinct(out) as consId from installs where active=true and out.deleted=false),
              $c = unionall($a, $b)
          )
        ),
  $diff = (select from $cons where consId not in (select from $cust))
)

Hi @mindnuts
I just fixed it in develop branch, the fix will be available in v 2.2.1.
@vlasni could you please test it on your dataset and see if it fixes your problem as well? In case could you please file a separate issue?
Thanks
Luigi


fix: don't leave scale undefined

Follow up to #244 in which I fixed one bug, but introduced another

rebased on https://github.com/pymmcore-plus/napari-micromanager/pull/260 to hopefully fix the tests


Add support for SameSite=None

Fixes https://github.com/twitter/secure_headers/issues/412
For sites that will break when samesite lax becomes default, an opt out is needed. Supplying none is the opt out.
All PRs:

[x] Has tests
[x] Documentation updated todo

cc @lgarron

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

Lol
On Tue, Jan 7, 2020 at 12:12 PM CLA assistant<EMAIL_ADDRESS>wrote:

[image: CLA assistant check]
https://cla-assistant.io/twitter/secure_headers?pullRequest=414
Thank you for your submission, we really appreciate it. Like many open
source projects, we ask that you sign our Contributor License Agreement
https://cla-assistant.io/twitter/secure_headers?pullRequest=414 before
we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us
recheck
https://cla-assistant.io/check/twitter/secure_headers?pullRequest=414
it.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/twitter/secure_headers/pull/414?email_source=notifications&email_token=AADNQBH5B6NGRWT2RZPVKKTQ4T43ZA5CNFSM4KD7ZN3KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIKO6EY#issuecomment-571797267,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AADNQBCG5UTSRXFWKCUYVU3Q4T43ZANCNFSM4KD7ZN3A
.

--
Neil

Hi, do you know of any links to read about this samesite lax change and what the impact will be? The new release of this gem was the first I heard about it.

Check out https://www.troyhunt.com/promiscuous-cookies-and-their-impending-death-via-the-samesite-policy/ and https://web.dev/samesite-cookies-explained/.

Thank you @jacobbednarz!


Fix: Hiding required validation messages for tracks-hidden pages

Description
A bug was reported to me directly on Slack that seemed to cause hidden required fields to produce 'empty field' validation messages - if the pages they were in were hidden using the legacy 'tracks' functionality. I haven't bisected to verify, but I would think this might have been introduced with the recent validation refactor. There was no test for this to catch the problem when it appeared.
This should fix the issue by properly returning true in node.isHidden() for these components - although that does somewhat break the expectations for expressions - where it is expected that component lookups in tracks-hidden pages still work (i.e. does not always return null). For this reason, I'm opting to turn this behaviour off everywhere besides the validation logic.
Related Issue(s)

closes #{issue number}

Verification/QA

Manual functionality testing

[ ] I have tested these changes manually
[ ] Creator of the original issue (or service owner) has been contacted for manual testing (or will be contacted when released in alpha)
[ ] No testing done/necessary

Automated tests

[ ] Unit test(s) have been added/updated
[x] Cypress E2E test(s) have been added/updated
[ ] No automatic tests are needed here (no functional changes/additions)
[ ] I want someone to help me make some tests

UU/WCAG (follow these guidelines until we have our own)

[ ] I have tested with a screen reader/keyboard navigation/automated wcag validator
[x] No testing done/necessary (no DOM/visual changes)
[ ] I want someone to help me perform accessibility testing

User documentation @ altinn-studio-docs

[ ] Has been added/updated

[x] No functionality has been changed/added, so no documentation is needed
[ ] I will do that later/have created an issue

Changes/additions to component properties

[ ] Changes are reflected in both src/layout/layout.d.ts and layout.schema.v1.json, and these are all backwards-compatible
[x] No changes made

Support in Altinn Studio

[ ] Issue(s) created for support in Studio

[x] This change/feature does not require any changes to Altinn Studio

Sprint board

[x] The original issue (or this PR itself) has been added to the Team Apps project and to the current sprint board
[ ] I don't have permissions to do that, please help me out

Labels

[x] I have added a kind/* label to this PR for proper release notes grouping
[ ] I don't have permissions to add labels, please help me out

I think the same issue is present in backend validation, but expression based required validation it manually activated, so hopefully only for those who doesn’t use tracks.


We should strip out the test runner output for failed tests.

It's kind of confusing when a test fails to see:
maelstrom-worker dispatcher::tests::available_job_ord............................................................FAIL   0.015s
maelstrom-worker dispatcher::tests::cancel_unknown.................................................................OK   0.015s

running 1 test
test dispatcher::tests::available_job_ord ... FAILED

failures:

failures:
    dispatcher::tests::available_job_ord

test result: FAILED. 0 passed; 1 failed; 0 ignored; 0 measured; 105 filtered out; finished in 0.00s

stderr: thread 'dispatcher::tests::available_job_ord' panicked at crates/maelstrom-worker/src/dispatcher.rs:760:9:
stderr: assertion `left == right` failed
stderr:   left: Less
stderr:  right: Equal
stderr: note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

All that stuff to stdout is confusing. One has to know that cargo maelstrom is invoking a test binary that has its own output, and we're showing that output there.

lol, this didn't close because I wrote fix; instead of fix: in the commit message


train error

When I started training, I reported the following error. Do you know why and what version of Pytorch you are using? Thanks.

The same problem.


update GMC e2e and Doc

Description

fix minor errors
sync to Gaudi scripts
add doc

Issues
n/a.
Type of change
List the type of change like below. Please delete options that are not relevant.

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds new functionality)
[ ] Breaking change (fix or feature that would break existing design and interface)

Dependencies
n/a.
Tests
CI/CD will cover the tests

some nit, otherwise LGTM. Thank you!


feat: Add eslint & flow.

I'm no expert on the subject but I remember an effort to remove flow from our projects in favor of typescript? I guess this project is somewhat isolated from those other areas but I just wanted to make sure we made an attempt to be consistent with where we wanted to go in the future?

I can't seem to approve the PR so I'm not sure if it's a repo setting missing or something wrong with my perms? Anyways, I approve but would like more info on why we use flow here for my education.

@jmacelroy I'm not aware of this effort. I've heard some suggestions about it. Do we actually agree on that? Do we have  any projects that uses typescript?
Note I'm not against moving to typescript!
I'm not worried for consistency because as far as I know we don't have typescript projects (or at least I'm not aware that we have :) ). Also jitsi-meet, lib-jitsi-meet, jitsi-meet-electron, jitsi-meet-electron-utils and many many more don't have any typescript in them. So I'm not so worried about consistency. Of course if I missed something and this the direction we want to go probably we should work on that at some point.

@hristoterezov @jmacelroy there was no decision, but yes, there is an ongoing effort to move away from Flow and jump into TS, especially for new projects (we're not sure if meet will ever be converted to TS, it's just too much of a hassle).
If you're interested, here are a few repos that are already done with TS: https://github.com/jitsi?q=&type=&language=typescript
Most of them were done by me, but i'm no Ts expert by any means :)


iOS Adapter for SDK 7

There's currently no adapter for the latest version of the SDK.
Any sight on when there will be one?

A newer version of the adapter is being worked on. No exact ETA yet, but it should be out soon.

For now it appears to be wording with the v6 adapter, but an official release would be nice

There was an update a while back, but I forgot to update this thread. Closing. Thanks.


Rollup dependency cleanup

Its a followup for #203

There are still rollup dependencies leftovers in the package.json. It should be decided if they should be removed or kept and being used for something.
It can still be valuable to leave it to:

provide flat bundle (single file bundle - keeping internals hidden)
build UMD version of the lib, so its easy to include on the webpage and experiment

See #230


Added "Administrator: " prefix to window title

Posh-Git was not including this prefix when it set the title, and I keep forgetting which window was launched with "Run As Administrator" and which was not.

This is a great addition - :disappointed: that I let it sit for so long.

Thanks!


support provisioning tencent cloud resources

Signed-off-by: royyuewang<EMAIL_ADDRESS>

Hi, would you like to finish the job? I think it would be very exciting if we have tencent cloud resources here.


Added the ability to toggle the theme of the website

This pull request fixes #555

Amazing work!

Thank you @AM1CODES!! 🙂


gltfpack: Improve handling of meshes with very large UV bounds

gltfpack version: 0.14 and 0.15
I convert my gltf model to gltfpack format by -tc, it will generate a new model correct. And then use KTX2Loader and GltfLoader to load the target model in Threejs(v0.122.0).
But some textures are displayed as stretch(LEFT: before convert, RIGHT: after convert),  I don't know what's the reason for this result? what should I do to resolve it?

Please attach a model that would allow me to reproduce this.

test07.zip
I put the model into the zip file in attachment, thank you

I looked into why this happens; this is due to some of the textures being mapped on geometry with very large UV range, e.g. one of the walls has a texture stretched because the ground mesh has some geometry that is mapped to the same texture with UV range min -376010.687500 0.709212 max<PHONE_NUMBER>00 0.921277.
For now I'd recommend using -noq for this mesh. There are some changes planned to gltfpack that can improve the behavior for cases like these, so I'm relabeling this as "enhancement".

OK, thank you

I have had similar issue that models from different sources had very different UV scale/vertex scale, affecting the quantization. I had to change source models to be in a similar range to fix this. I think if meshoptimizer could handle this in itself (e.g. by normalizing outlying values to the 90-percentile scale), this could be a non-issue.

I have had similar issue that models from different sources had very different UV scale/vertex scale, affecting the quantization. I had to change source models to be in a similar range to fix this. I think if meshoptimizer could handle this in itself (e.g. by normalizing outlying values to the 90-percentile scale), this could be a non-issue.


ABOUT page blue box still looks strange

What should happen:
The blue box on the ABOUT page http://<IP_ADDRESS>:5000/about still doesn't look great. So, please...

Shorten the box vertically, so the bottom of the page mostly loads (otherwise it's unclear that there's text below the box)
Make the text in the box a few point sizes smaller
Edit the text so it just says, "Our collaborative Atlas combines existing databases into one open-source wiki, and shares data across other websites. Please join us!"

What it looks like now:

pushing changes marking as fixed.. pls reopen if you dont find changes on checking

Looks excellent, thanks!!


Bug 259: Error on unsafe signed/unsigned comparisons.

Fixes http://d.puremagic.com/issues/show_bug.cgi?id=259 in a way that uses the range propagation of expressions. It is also still backwards compatible with C code, though some C code may now produce an error. It still breaks lots of existing code though.
Examples:
uint a = 3;
if (a > 4) {} // ok

uint b = 3;
if (b > -2) {} // now an error

byte c = -67;
uint u;
if (u < c + 0x100) {} // ok

int[] a;
for (int i = 0; i < a.length; ++i) {} // error, ugh

This is clever and I like it. But it does break a lot of code left and right, and perhaps that code is broken. I'm going to leave it in #ifdef'd out for the moment.


Fixed typo ruby_ls -> ruby_lsp in server_configurations.md

Fixed a tiny typo.

it's generate from file name.


Call reusable delete-preview workflow

Another step toward https://github.com/ProjectPythiaCookbooks/cookbook-template/issues/61

I'll have to merge this to see if the preview delete action actually works.

Yup, worked. Preview is gone.


define call should include empty dependencies list

see: https://github.com/umdjs/umd/pull/34

Without this, the factory may be scanned for additional dependencies matching the literal string require("module-id"), which is not correct. See https://github.com/amdjs/amdjs-api/wiki/AMD#simplified-commonjs-wrapping-.

This PR should also include this fix https://github.com/umdjs/umd/commit/6e693154138b111e17b264e4e27f76e6116f1aba

And this https://github.com/umdjs/umd/pull/22

Thanks. Do you want to do another PR for the other fixes?

Yep. Will do.


fix: ensure tenant_id matches the tenant a client is registered with

Description
While debugging 5XX errors, I noticed there was a potential to send a notification to a client from another tenant. This shouldn't have been as issue as the push_message has signature validation and the relay behaviour should have been within specification.
How Has This Been Tested?
Builds locally
Due Diligence

[ ] Breaking change
[ ] Requires a documentation update
[ ] Requires a e2e/integration test update

I noticed something while making this, we were not settings the tenant_id for stored tenants, so for now we need to store for all existing and not change behaviour for past. When we re-build for Push Service, client's won't be able to be migrated so this can be solved there.


Add new methods to avoid showing rationale

Hello,
I added 2 public methods. The idea of that pull request is to provide a way to not show any rationale if we do not want any. In my case, in my app, I do not feel the need to have that message showing up. Any questions or comments?
Thank you,
Carl

I signed it!

@carlphilipp thanks for this contribution.  I understand your motivation here, but the goal of this library is to make it very simple for developers to follow the Android M permission guidelines.  Avoiding permission rational is not in the spirit of the guidelines, so I think adding this boolean option could negatively affect some user experiences.
I am going to close this PR.  In the future if you want to make a change like this it's always safer to discuss it in an Issue first so you don't have to go through all the work of writing the code before getting feedback.

@samtstern, it's ok I understand, I will ask next time before doing any change.
Also note that the boolean I added is in a private method, so it's not exposed to the developer. The two new methods exposed do not contain any rationale String, that's the only diff to the developer.
Thank you for taking the time to review it!

Hi,
I need these methods too， but i didn't saw it in version 0.4.
Is there any methods replaceable?


v0.4.0 Add v4 base url

closes #1928

Coverage remained the same at 80.241% when pulling 5277c9ba26abe3825dcbeeb9b58f91a80e29c3e7 on mparke:docs-v4-url into 96b8bd32b07321b5e5c73724c9f9f32ecb306962 on linode:master.

Coverage remained the same at 80.241% when pulling 1c330395eb8a4212b695331ef7bb8f066dee7de1 on mparke:docs-v4-url into 96b8bd32b07321b5e5c73724c9f9f32ecb306962 on linode:master.

Coverage increased (+0.06%) to 80.297% when pulling d36fd5acc7ce47b19f140f1e36c6bbfa2864002a on mparke:docs-v4-url into fd81b0c1cd26eda480e2a42993693db5ec43830c on linode:master.


Override existing menu entry

Select your issue type: (check at least one)

[ ] Bug
[ ] Question
[x] Suggestion
[ ] Other (please describe):

Describe your issue:
Some applications already have a "Check for updates" menu entry, e.g., here:
https://github.com/Subsurface-divelog/subsurface/blob/30f2d16dabd49ec0cb871f18ec1bb69f21478020/desktop-widgets/mainwindow.ui#L118-L126
It would be awesome if we could override the existing one (or attach our functionality to it), not just add another menu entry.

Okay :+1: But please open issues in relevant repo. This will be transferred to updatedeployqt.

I never know what is difference between AppImageUpdaterBridge, AppImageUpdaterBridgePlugin, updatedeployqt, and so on - wouldn't it be much easier to have everything in one repository?

I never know what is difference between AppImageUpdaterBridge, AppImageUpdaterBridgePlugin, updatedeployqt, and so on - wouldn't it be much easier to have everything in one repository?

Well I deleted ,  AppImageUpdaterBridgePlugin in favor of the QtUpdateBridges repo.
updatedeployqt , does all the injection stuff.
AppImageUpdaterBridge , is just a Qt C++ library.

Wouldn't it still be much easier to have everything in one repository? The advantage would be that if you make a change in any of those parts, everything would be rebuilt on Travis CI automatically. Same for pull requests. Much less work!

Wouldn't it still be much easier to have everything in one repository?

No, I think everything is better this way. This library has no connection to updatedeployqt stuff or any other stuff which uses this. Some people or most of them use this as a standalone library. Projects like qTox always compiles libraries from source and never use prebuilt stuff. They even build appimagetool from source.

I fail to see why using source rather than compiled stuff has something to do with how many source code repos to use...

In simple terms, I like everything decentralized. It just works this way. I don't see any benefits from combining everything under a single repo rather than more work.


change standalone.yml

That platform is not really need,
I get an error if I run with this:
ERROR: The Compose file './docker-compose.standalone.yml' is invalid because:
Unsupported config option for services.mysql: 'platform'
So I think need remove that

Tks for your contribution.


up_squared: use HPET timer and fixes some tests

tests: workq/user_work: change how resource pool is exhausted
tests: c_lib/thrd: more stack for up_squared board
boards: x86/up_squared: use HPET timer

Fixes #68616

Suggest to use overlay file in test app instead of changed default APIC TSC timer to HPET in board configuration.

is this fixing 68616, if so, added "Fixes #68616" to the body

Suggest to use overlay file in test app instead of changing default APIC TSC timer to HPET in board configuration.

The issue is that UP Squared boards can be ordered with different CPUs running at different speed. Using HPET is the only way to target all these SKUs with correct real world timing.

The issue is that UP Squared boards can be ordered with different CPUs running at different speed. Using HPET is the only way to target all these SKUs with correct real world timing.

Sorry, for clarity: you're saying that different devices are sold with different clocking, but we need to use them all with the same binary Zephyr build and can't discriminate with kconfig/DTS, right?  Not that different cores on the same device are showing skew between their TSC values (which would be a hardware bug with the invariant TSC).

The issue is that UP Squared boards can be ordered with different CPUs running at different speed. Using HPET is the only way to target all these SKUs with correct real world timing.

Sorry, for clarity: you're saying that different devices are sold with different clocking, but we need to use them all with the same binary Zephyr build and can't discriminate with kconfig/DTS, right? Not that different cores on the same device are showing skew between their TSC values (which would be a hardware bug with the invariant TSC).

Not necessarily the same binary. The default clock rate (in board config) is 1.6GHz with APIC timer, for Atom. So if someone buys a board with Pentium or Celeron, which runs at 1.1GHz, they will be very confused on why the board would sleep ~15 seconds instead of the commanded 10 seconds if they use the board config as-is. Therefore, since the board config is "generic" to all variants of UP Squared (i.e. applies to all SKUs), we should use HPET as the base clock rate is the same on all of them. Also note that the DTS specifies 2GB as main memory size even though you can get boards with 8GB memory.