Split polygon interiors into their own layer

It is hard to make polygons for entire countries or oceans with good detail on the coasts (to z13 or so) without also requiring the interiors to be represented by thousands of identical tiles that do nothing but cover the entire tile with a square.
I think the way to handle this is to provide a way to automatically split polygon layers into a detail layer and a fill layer.
At some particular zoom layer, say, 8, all the polygons whose interior contains one or more entire z8 tiles would have those interior tiles cut out into a separate layer. The remainder of the polygons would  continue to be tiled down to the maxzoom, to provide coastal detail. The interiors would exist only at that one zoom level and would be overzoomed to fill in the polygon interiors at higher zooms. The division between the two layers would be in the tile buffer so the edge wouldn't be visible.
(When should the cutover be? The longer you put it off, the more area you cut out, reducing the ultimate number of detail tiles, but the more fill tiles you need to produce. It should be possible to calculate the optimal zoom level to switch over that will produce the smallest final tileset size.)
To be verified: can these be separate layers within the same tileset, or must they be entirely separate tilesets?

For Natural Earth countries:

zoom
total tiles
exact squares
savings at zooms z+1 through 12

0
1

1
4

2
16

3
61
4
1398096

4
216
30
2621400

5
694
179
3910076

6
2311
924
5045040

7
8031
4472
6099808

8
29206
20340
6915600

9
109923
88730
7453320

10
423413
374339
7486780

11
1656278
1545837
6183348

12
6540964
6296817

Which suggests that z10 is the right place to output the filler squares, to save 7.486M total tiles at z11 and z12. For other maxzooms, it looks like the best cutover is:

maxzoom
cutover

5
4

6
5

7
6

8
6

9
7

10
8

11
9

so always something like 80% of the maxzoom. There is enough edge detail required that getting that 4x or 20x multiplier for one or two zooms is always better than cutting somewhere deeper.

Finally a workable implementation of this in https://github.com/mapbox/tippecanoe/pull/808

I assume that a command line such as tippecanoe -pF -f -o out_file geo.json will trigger this behavior from a testing standpoint?

Yes, that's how it works in the implementation I've been trying out.


A better name for the openpubkey/oidc package

There are too many oidc packages and similar dependencies: core-os/oidc, zitadel/oidc, core/go-oidc
What should we rename it to? For a while we were calling it simpleoidc, but that was too long as we reference the openpubkey/oidc package in so many places.
If someone wants to try their hand at the arcane art of discovering the true name of this package, I'm open to suggests. To avoid merge conflicts suggest the name here before creating the PR because I have a soon to be merged PR.

Hello there! I am interested in this issue, how about mykey/my-oidc as a simpler name?

Hi @RishavKumarSinha thanks for taking a look at this.

We need to keep the module name as openpubkey, so it would be openpubkey/my-oidc.
"my" isn't sufficiently meaningful. Who is the subject that "my" refers to? Maybe opk-oidc (opkoidc) or our-oidc?

I've been thinking about "simple" synonyms that are shorter bare-oidc? Or maybe we rethink using oidc, maybe jwc (JSON Web Cryptography), jwsigs (JSON Web Sigs)?
What other name ideas you got?

Oh, thanks a lot for clarifying the requirement. I think the names "bare-oidc" or "jwc" suggested by you are already pretty great. The only other name, I can come up with at the moment is "jwauth" from JSON Web and Authorization.

Riffing off your "sigweb" idea, jwsig?

Indeed a good name combination, I like it.

@RishavKumarSinha Want to put a PR together that renames openpubkey/oidc to openpubkey/jwsig?
Make sure to get rid of the simpleoidc package alias in https://github.com/openpubkey/openpubkey/blob/eba749e7d12b83798f2ab29ed684ba3bafed5e6f/providers/google.go#L32

Yes, of course, I would however need specific instructions to complete the PR request. How do I get started?

Use refactoring tools to rename the package and all references to the package in OpenPubkey.
Ensure nothing is breaking after this change using go ./... and running some of the examples.
Git push to your branch
Open a PR into the openpubkey repo

@EthanHeilman, I have successfully managed to refactor the name. However, I am having difficulties with removing the alias 'simpleoidc'. I would appreciate it if you could help me out.

I see you have a fork.

create a new branch git branch <branchname>
checkout that branch git checkout <branchname>
use git add to add the files you want to commit to that branch git add <file1> <file2> ...
commit those changes to that branch git commit -m "your commit message here"
push those changes to the remote git push

Then follow these instructions to create a PR.
https://docs.github.com/en/pull-requests/collaborating-with-pull-requests/proposing-changes-to-your-work-with-pull-requests/creating-a-pull-request-from-a-fork

@EthanHeilman, I have successfully created the pull request, thanks a lot for taking the time to guide me through the process.
PR - https://github.com/openpubkey/openpubkey/pull/194


Is wind 3 components in m/s?

Very cool paper and thanks for releasing it as open source. I've got some questions about the wind input.

Is the wind in m/s?
Does the wind have a z component? The example provides an array of 3 doubles but the code seems to only use the first 2.

Cheers

Hi @martinbaer ,
Thanks for the interest in our work!

Yes the wind is in m/s.
The wind does have a z component, but it isn't used within the trochoid calculation itself which just focused on 2D. The z values of the final path are merely linearly interpolated. We point this out in the last paragraph of the methodology section of our paper, and future work could include solving trochoidal paths in 3D.


[4.0.0] Add back ability to specify random order of flags on create

Allow the options/flags to be specified in any order on the $ phonegap create --name --id rather than assume the order is always --id and --name.
This is not supported in the cordova CLI, but one would assume it would work that way since you're specifying the actual flag before the value you want it to have and it shouldn't matter which order it's found.  I believe this is also supported in the recent version of the PG CLI currently available as well.

Closing. This was added in the 4.0.0 tag.


Dynamic mounts

Sometimes a client app needs to mount its nodes after the initialization.
We can have two types of dynamic mounts:

Nodes that are mounted once at some point and then their structure persist during the application run
Nodes that implement interface to some dynamic data, so the subtree structure may differ with each access

The methods description has to be provided for each path. Handlers will provide its own dir and ls implementations.
Provide two variants of the mount method, or parametrize with an enum.
Consider allowance of 'nested' mounts on nodes that already have (or can have) its own properties (#2) .

[x] Static, dynamic mounts
[x] Dynamic mounts requests processing either in the client task or a separate task
Mount command to add mounts during the Client execution - not implemented for no use case


Add a substrate-peerset crate

Add a new crate :drum: : the substrate-peerset.
The substrate-peerset crate is currently implemented as a dummy, but once it is implemented correctly its responsibility will be to track the reputation of nodes, in other words how much they are useful for us, and tell the networking code which nodes we should be connected to/stay connected to, and which nodes which should disconnect from.
Keep in mind that the words "be connected" and "disconnect from" can be misleading. This corresponds to the Substrate overlay network on top of libp2p. Libp2p itself also compromises Kademlia, Identify, and maybe future protocols. Libp2p automatically drop connections that aren't used by Substrate, but it's not instantaneous and not guaranteed. In general, the set of nodes to which we are actually connected to is a superset of what the peerset wants.
Here are the basic high-level changes:

Creating a NetworkService now returns a (NetworkService, Arc<Peerset>). This is not currently the case, but the Arc<Peerset> will be shared amongst the code base and will be used to report nodes which are useful or not to us through the report_peer method.
The other part of the peerset is the PeersetMut, which is owned by the network. This PeersetMut communicates with the network via a channel and sends messages to it, most notably Connect(PeerId) and Drop(PeerId) which instruct the network to respectively establish or remove a link with the given peer.
Removes all serialization/deserialization on the network code. The nodes.json file is now unused.
Totally revamped the logging of sub-libp2p. It should now be much easier to read in order to understand what is wrong.
The NetTopology type is removed (closes #1403). This puts more emphasis on Kademlia, as it is now Kademlia that is responsible for storing/loading the addresses of the nodes of the network.
Slightly modifies the output of the system_networkState RPC query, and what is sent to the topology (pinging @gterzian @maciejhirsz as usual).

The reason for this design (a separate crate) is mostly to make the API of the peerset as simple as possible so that its implementation doesn't have to track superfluous things.
This PR is not ready because we need some changes in Kademlia in libp2p beforehand.

Still needs to be done:

https://github.com/libp2p/rust-libp2p/pull/996
https://github.com/libp2p/rust-libp2p/pull/1012
https://github.com/libp2p/rust-libp2p/issues/1003
Add a back-off mechanism; when a node misbehaves and the PSM asks to connect again to the same node, we add an artificial delay before actually trying.

Ready for review.
I'm using a separate branch from my libp2p fork while waiting for reviews there. We will have to wait for them in order to merge.

looks about right.

This also seems to help with #1777 as a side effect.
Will unignore the test in a future PR in order to not have two changes at once.

if nobody else will review by this evening then you can go ahead and merge.

Pushed another commit that changes the discovered peers from a HashSet to a Vec, so that we have a round-robin system for the nodes to try.

test chain_spec::tests::test_connectivity ... test chain_spec::tests::test_connectivity has been running for over 60 seconds

Oh that sucks.

Fixed that.


Highlight learning requirements.

This is my final suggestion for an improvement that promotes our recommendations for learning the "other" packages.
As it happens, it came out really small 😉
With this done, I for one am ready to merge back !

@pp-mo Looks good !


[SIP] Add Atgenomix SeqsLab data engine to Superset supported database.

[SIP] Proposal for adding Atgenomix SeqsLab data engine to Superset supported database
Motivation
Atgenomix SeqsLab allows its users to use Superset to on-demand access and visualize their omics databases stored in data lake.
Proposed Change
Add documentation in the list of supported databases and dependencies.
https://superset.apache.org/docs/databases/installing-database-drivers/

New or Changed Public Interfaces
SeqsLab db engine spec
New dependencies
The SeqsLab Connector for Python based on pyhive
Migration Plan and Compatibility
N/A
Rejected Alternatives
N/A

This is nice! In my opinion we don't need a SIP for this, since it doesn't affect any public interfaces or major features. And even though it introduces a new dependency, DB engine spec dependencies are optional, so it should be fine. You can go ahead an PR it and add me as a reviewer, I'll be happy to take a look.
A few things of notice:

I wrote a tool to test DB engine spec implementations (and SQLAlchemy dialects as well, to make sure they implement everything Superset needs). You can read more about it in https://github.com/apache/superset/pull/24918.
You have the option of having the DB engine spec be part of Superset, or you can distribute it in a separate package (see an example here and here). Distributing it in a separate package means you can release new versions independently of Superset, which allows people to adopt features more quickly. But the downside is that the DB engine spec interface is not public, so it might change between major versions and break existing 3rd party DB engine specs.

@betodealmeida #25176 please can you review it.

Looks like we can close this issue/SIP in favor of the attached PR, but feel free to comment if you think we need to reopen.


Refine GPR annotation for Glycolysis pathway

Description of the issue:

Enzyme
rxn
GPT result
GPT reason
Action

ENSG00000198099
MAR03905: ethanol[c] + NAD+[c] = acetaldehyde[c] + H+[c] + NADH[c]
No
The substrates are all-trans-retinol, 9-cis-retinol, long chain omega-hydroxy fatty acids, 20-HETE, 20-oxoarachidonate, and (5Z,8Z,11Z,14Z)-eicosatetraenedioate.
Rhea:21284;42052;39803;39799;60660

ENSG00000184254
MAR04283: acetaldehyde[c] + H2O[c] + NADP+[c] = acetate[c] + 2 H+[c] + NADPH[c]
No
Catalyzing the formation of retinoic acid from all-trans retinal
Rhea:42080

ENSG00000147576
MAR03905: ethanol[c] + NAD+[c] = acetaldehyde[c] + H+[c] + NADH[c]
No
The substrates are gamma-hydroxybutyrate (GHB), succinic semialdehyde (SSA), 2-ketoglutarate (2-KG), D-2-hydroxyglutarate (D-2-HG), D,L-3-hydroxyisobutyrate and L-3-hydroxybutyrate (L-3-OHB).
Rhea:23048

ENSG00000143149
MAR08357: acetaldehyde[m] + H2O[m] + NAD+[m] = acetate[m] + 2 H+[m] + NADH[m]
Yes
Yes, the substrates and products of this enzyme cover acetaldehyde, acetate, H+, H2O, NAD+, NADH. This enzyme catalyzes the irreversible oxidation of a broad range of aldehydes to the corresponding acids in an NAD-dependent reaction, which includes acetaldehyde being converted into acetate, and NAD+ being reduced to NADH.
Rhea:17985;19105

ENSG00000132746
MAR04283: acetaldehyde[c] + H2O[c] + NADP+[c] = acetate[c] + 2 H+[c] + NADPH[c]
No
No, the substrates and products of this enzyme do not cover the given metabolites. This enzyme specifically oxidizes medium and long chain aldehydes into non-toxic fatty acids, so the only metabolites it covers are aldehydes.
remove the gene

ENSG00000180011
MAR03905: ethanol[c] + NAD+[c] = acetaldehyde[c] + H+[c] + NADH[c]
No
No, the substrates and products of this enzyme do not cover the metabolites given (acetaldehyde, ethanol, H+, NAD+, NADH). This enzyme functions as a 15-oxo-prostaglandin 13-reductase and acts on 15-keto-PGE1, 15-keto-PGE2, 15-keto-PGE1-alpha and 15-keto-PGE2-alpha with highest efficiency towards 15-keto-PGE2-alpha, but not on acetaldehyde, ethanol, H+, NAD+, NADH.
Rhea:11912

ENSG00000180011
MAR04360: H2O[c] + methylglyoxal[c] + NADP+[c] = 2 H+[c] + NADPH[c] + pyruvate[c]
No
No, the substrates and products of this enzyme do not cover metabolites given: H+, H2O, methylglyoxal, NADP+, NADPH, pyruvate. The enzyme functions as 15-oxo-prostaglandin 13-reductase, with substrates and products being 15-keto-PGE1, 15-keto-PGE2, 15-keto-PGE1-alpha, 15-keto-PGE2-alpha, and NADPH.
Rhea:11912

ENSG00000131482
MAR08652: carbamoyl-phosphate[r] + glucose[r] = CO2[r] + glucose-6-phosphate[r] + NH3[r]
No
No, the substrates and products of this enzyme do not cover all of the metabolites given. The enzyme hydrolyzes glucose-6-phosphate to glucose, meaning the substrates and products of the enzyme are glucose-6-phosphate and glucose, not carbamoyl-phosphate, CO2, glucose, glucose-6-phosphate, and NH3.
Rhea:16689

ENSG00000130649
MAR08757: ethanol[c] + H+[c] + NADPH[c] + O2[c] = acetaldehyde[c] + 2 H2O[c] + NADP+[c]
No
No, the substrates and products of this enzyme do not cover the metabolites given: acetaldehyde, ethanol, H+, H2O, NADP+, NADPH, O2. This enzyme only catalyzes the hydroxylation of carbon-hydrogen bonds in fatty acids, and its substrates and products are fatty acids and oxygen, respectively.
Rhea:50076;39787;50088;39751;50096;26205

ENSG00000143891
MAR07745: beta-D-glucose[c] <= glucose[c]
No
No, the substrates and products of this enzyme do not cover metabolites beta-D-glucose and glucose, as the enzyme only catalyzes the interconversion of beta-D-galactose and alpha-D-galactose.
Rhea:28675;10264

ENSG00000141349
MAR08652: carbamoyl-phosphate[r] + glucose[r] = CO2[r] + glucose-6-phosphate[r] + NH3[r]
No
No, the substrates and products of this enzyme do not cover the given metabolites. The enzyme hydrolyzes glucose-6-phosphate to glucose, so the substrates and products are glucose-6-phosphate and glucose, not carbamoyl-phosphate, CO2, glucose, glucose-6-phosphate, and NH3.
Rhea:16689

ENSG00000006534
MAR04283: acetaldehyde[c] + H2O[c] + NADP+[c] = acetate[c] + 2 H+[c] + NADPH[c]
No
No, the substrates and products of this enzyme do not cover the metabolites given. This enzyme oxidizes medium and long chain saturated and unsaturated aldehydes, benzaldehyde, and can use both NADP(+) and NAD(+) as electron acceptor. It has low activity towards acetaldehyde and 3,4-dihydroxyphenylacetaldehyde, and may not metabolize short chain aldehydes. The metabolites given are acetaldehyde, acetate, H+, H2O, NADP+, NADPH, which are not covered by this enzyme.
Rhea: 33739;44100;59904; 21660;11840;59908;59912;59916;59920

ENSG00000197894
MAR03905: ethanol[c] + NAD+[c] = acetaldehyde[c] + H+[c] + NADH[c]
No
No, the substrates and products of this enzyme do not cover acetaldehyde, ethanol, H+, NAD+, NADH. This enzyme catalyzes the oxidation of long-chain primary alcohols and S-(hydroxymethyl) glutathione, as well as the oxidation of long-chain omega-hydroxy fatty acids, such as 20-HETE, producing both an intermediate aldehyde and a dicarboxylic acid. It does not catalyze the oxidation of ethanol and does not involve acetaldehyde, H+, NAD+, or NADH.
Rhea:19981;19985;39803;39799

ENSG00000072210
MAR08357: acetaldehyde[m] + H2O[m] + NAD+[m] = acetate[m] + 2 H+[m] + NADH[m]
No
No, the substrates and products of this enzyme do not cover acetaldehyde, acetate, H+, H2O, NAD+, NADH. This enzyme catalyzes the oxidation of medium and long chain aliphatic aldehydes to fatty acids, not acetaldehyde, acetate, H+, H2O, NAD+, NADH.
Rhea:36135;33739;39015;44016;44020;44168;44104;44172;44100;44108;24216

ENSG00000149925
MAR04355: DHAP[c] + erythrose-4-phosphate[c] = sedoheptulose-1,7-bisphosphate[c]
No
No, the substrates and products of this enzyme do not cover DHAP, erythrose-4-phosphate, and sedoheptulose-1,7-bisphosphate. This enzyme catalyzes the reversible conversion of beta-D-fructose 1,6-bisphosphate (FBP) into two triose phosphates, which does not include DHAP, erythrose-4-phosphate, and sedoheptulose-1,7-bisphosphate as metabolites.
remove the gene

ENSG00000154305
MAR04368: 1,3-bisphospho-D-glycerate[c] + ADP[c] <= 3-phospho-D-glycerate[c] + ATP[c]
No
No, the substrates and products of this enzyme do not cover the metabolites 1,3-bisphospho-D-glycerate, 3-phospho-D-glycerate, ADP, and ATP. This enzyme is responsible for transporting large cargos like collagen VII, lipoproteins, and COPII coat components, but does not directly interact with the above metabolites.
remove the gene

ENSG00000166800
MAR04281: H+[x] + NADH[x] + pyruvate[x] <= L-lactate[x] + NAD+[x]
Yes
Yes, the substrates and products of this enzyme cover the given metabolites: H+, L-lactate, NAD+, NADH, pyruvate. This is because the enzyme catalyzes the interconversion of L-lactate and pyruvate with NAD+ as a coenzyme. Therefore, these metabolites are all part of the reaction catalyzed by the enzyme.
remove the gene, not x annotation

ENSG00000121691
MAR08360: ethanol[x] + H2O2[x] = acetaldehyde[x] + 2 H2O[x]
No
No, the substrates and products of this enzyme do not cover metabolites given: acetaldehyde, ethanol, H2O, H2O2. This enzyme catalyzes the degradation of hydrogen peroxide (H2O2) to water and oxygen, and does not involve acetaldehyde, ethanol, H2O.
Rhea:20309

ENSG00000196616
MAR03905: ethanol[c] + NAD+[c] = acetaldehyde[c] + H+[c] + NADH[c]
No
No, the substrates and products of this enzyme do not cover the metabolites acetaldehyde, ethanol, H+, NAD+, NADH. This enzyme catalyzes the NAD-dependent oxidation of all-trans-retinol and its derivatives as well as the NADH-dependent reduction of all-trans-retinal and its derivatives.
Rhea:21284;55936; 60632

I hereby confirm that I have:

[X] Done this analysis in the main branch of the repository
[X] Checked that a similar issue does not exist already

@feiranl maybe clarify a bit the action column.
are those Rhea ids the reactions to be added?

We will firstly fix the remove gene part. reaction addition part will be fixed later

fixed in #517 and #535


[bug]: Is there a way to automatically close a Popover if it is not in viewport?

Describe the bug
I am attaching a Popover to a button on my menu bar. I want the Popover to be hidden when it is no longer in the Viewport when scrolled up, but it is scrolled down as well. I have tried using z-indexes on other elements overlaying it but it doesn't work.
Affected component/components
Popover
How to reproduce

Create button in navigation
Click button to show Popover
When scroll, Popover is not in Viewport but it always show

Codesandbox/StackBlitz link
No response
Logs
No response
System Info
Google

Before submitting

[X] I've made research efforts and searched the documentation
[X] I've searched for existing issues

you have open from the popover, and openOnChange
setting open to false will automatically close it too , throw current Y position and dimensions in an effect and de-render when popover dimensiosn - viewport


[enhancement] Wifi's channel

It's possible to get the Wifi channel?
Thanks;

@Jesus32cm the only thing I could implement easily would be the ScanResults' channelWidth. If that would be what you would need, let me know and I can implement and publish it.

Yes, it could be. Thank you very much for your attention.

@Jesus32cm I hope this is what you're looking for (v1.0.3).


[2.0]: Hover state for right click of token menu not working as expected

Hover states on right click menu for tokens is not working as expected - see recording
https://hyma-team.slack.com/archives/C06RHB9RT2B/p1712625403060619

I can't replicate, moving to needs more info

Closing as not planned due to missing info


Should __array__ method always return array of vanilla numpy? (instead of experimental Modin wrapper)

In some cases inside Modin, the series and dataframe objects are converted to a numpy array as follows: numpy.array(value). However, the functions further do not expect that there can be an distributed object after this call.
@RehanSD what do you think about?

NumPy expects array to return a NumPy object - if we modify it I think NumPy will complain, so to me, it makes sense to have array default to NumPy. If a user wants to convert to a Modin NumPy array, I think it’s far more likely they’ll use .to_numpy or Modin.numpy.array(object) - so we don’t need to worry about users getting tripped up by this!

NumPy expects array to return a NumPy object - if we modify it I think NumPy will complain, so to me, it makes sense to have array default to NumPy. If a user wants to convert to a Modin NumPy array, I think it’s far more likely they’ll use .to_numpy or Modin.numpy.array(object) - so we don’t need to worry about users getting tripped up by this!

@RehanSD I prepared a pull request with the appropriate changes :) #6300


chore(main): release 1.0.2

:robot: I have created a release beep boop
1.0.2 (2022-11-02)
Bug Fixes

utilize PAT to trigger other workflows (#137) (85d6180)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/whispir/openapi/releases/tag/v1.0.2 :sunflower:


When will support Java 21 be added?

Environment:

Jib version: 3.4.0
Build tool:  Gradle
OS: Mac

Description of the issue:
When will support for building Java 21 be available?

FWIW, I managed to get it working by specifying an image name to use as a base like so
jib {
    from {
        image = "eclipse-temurin:21-jre"
    }
}

FWIW, I managed to get it working by explicitly specifying a jre image name to use as a base
jib {
    from {
        image = "eclipse-temurin:21-jre"
    }
}

Are your containers running with root after using from ?
I also had the same problem and when I specified the images explicitly, the jib containers are running with root user.


Fixed slider bar overflow issue

When a slider’s value exceeded its max value that bar overflows outside
of the slider. This occurs when the value is programmatically set by
listen().

Thanks! This has been fixed in e129e08b9d4ea7023873528ca61f49f2ef897b4b by setting "max-width: 100%;" for the bar.


getFields stores Image Field type as empty [] in the data field

When saving a custom Image field, the image gets stored as an empty array json instead of the uploaded file.

 I'm not too sure if we'll ever get it working with nova-menu-builder. For now, you could try any of the media libraries available that upload the image asynchronously (ie optimistdigital/nova-media-field).

@Tarpsvo Doing this will create an array with the media ID in the data attributes. How do you retrieve the actual url, path or filename? I can't find docs about this in optimistdigital/nova-media-field

The images are stored in a separate table. You can just query them using Media::find($mediaId) or Media::findMany($mediaIds). When you get the media model, you can just gets its url like so: $media->url.


ROX-24784: Migrate workspaces to OCI artifacts in Konflux builds

Description
Pipeline adapted based on https://github.com/konflux-ci/build-definitions/blob/main/pipelines/docker-build-oci-ta/patch.yaml.
This pipeline is used by central-db, operator, roxctl, scanner-v4-db - other builds still use the PVC workspace until migrated.
Notes

show-summary removed per https://github.com/konflux-ci/build-definitions/blob/main/pipelines/docker-build-oci-ta/patch.yaml#L120-L126
added a second custom task that supports OCI TA
the .git & .prefetch images will be placed in the same Quay repository as the image (e.g. quay.io/rhacs-eng/central-db) and expire at the same time, controlled by params.image-expires-after

Checklist

[x] Investigated and inspected CI test results (Konflux)
~[ ] Unit test and regression tests added~
~[ ] Evaluated and added CHANGELOG entry if required~
~[ ] Determined and documented upgrade steps~
~[ ] Documented user facing changes (create PR based on openshift/openshift-docs and merge into rhacs-docs)~

If any of these don't apply, please comment below.
Testing Performed
Passing Konflux CI is sufficient.

/retest


where is the water animation?

?????
Better make it clear, mate.
This makes no sense.

 Great for emphasis, home pages, sliders, and general just-add-water-awesomeness.

So, I wonder, where is the water effect? There is no documentation about it. I've also searched on the code in github, and couldn't find anything about water-effect but the documentation.

You may be taking it too literally.. Just-add-water is being used as a literary device.
Though a water animation would be rather nice to see.

Ohh, OK then :) I didn't know that, sorries.

@deeperx, could you please close this issue as it's solved?


edited

thanks

@Viratsinha  thnx for contributing , try this year aswell with updated repo and make sure to follow me and star this repo so that you don't miss updates regarding the new changes and issues during Hacktoberfest 2020.


add support for record_property

Note: As my knowledge of pytest-internals is very limited (to basically zero), I can't say much more than "works on my machine".
The code is inspired from what pytest itself does in the junitxml reporter. The relevant code seems to be (from src/_pytest/junitxml.py in function pytest_runtest_logreport):
for propname, propvalue in report.user_properties:
    reporter.add_property(propname, str(propvalue))

Thanks! I just added a few safeguards. (Closing #57)

Thanks – just updated to the latest version and using it in my code now.


counter in home tab

Description
Hi ,
is there a possibility to create an incremental  counter at the home tab like a timer
01:00 ,01:01,01:02,01:03 ..etc without updating or re- publishing the whole home view of-course .
if not possible , do you had any suggestions to do such a functionality using your slack api in other way than home tab
Regards,
Marwa
What type of issue is this? (place an x in one of the [ ])

[ ] bug
[x] enhancement (feature request)
[x] question
[ ] documentation related
[ ] testing related
[x] discussion

Requirements (place an x in each of the [ ])

[ ] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[ ] I've read and agree to the Code of Conduct.
[x] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
package version:
node version:
OS version(s):
Steps to reproduce:

Expected result:
What you expected to happen
Actual result:
What actually happened
Attachments:
Logs, screenshots, screencast, sample project, funny gif, etc.

without updating or re- publishing the whole home view of-course .

Unfortunately, it's not possible to implement what you want only with App Home's UI features. The only way to update Home tab is  calling views.publish API: https://api.slack.com/methods/views.publish

HI @seratch ,
Thanks a lot for your quick reply
Regards,
Marwa


Create a Babel preset for peer deps

Now that https://github.com/kaicataldo/eleventy-plugin-react/pull/15 has been merged and I've decided to keep the peer deps, I'd like to create a separate preset package that can be installed to set up all the peer deps and configuration for you. This seems like the best balance of "sane defaults" while allow complete configuration for the end user.

I wonder maintaining a separate package is actually necessary? Maybe we suggest folks install this package using install-peerdeps
npx install-peerdeps eleventy-plugin-react

And also note that with NPM 7 peer dependencies will automatically be installed.

I had originally thought that this would be a nice way to allow folks to extend off the base configuration for the user, but now that we're exposing the base configuration maybe we should just let npm 7 do its thing?

I had originally thought that this would be a nice way to allow folks to extend off the base configuration for the user, but now that we're exposing the base configuration maybe we should just let npm 7 do its thing?

Going to close this for now - we can always revisit this in the future.

Going to close this for now - we can always revisit this in the future.


Is it possible to use cookies until response is Cloudflare challange?

Hello!
This isn't a really issue but more of a question and an example of it would be nice to see.
My question is that is it possible to run a session where we call the cloudscraper once and use those cookies into the pages we use for requests and to refresh cloudscraper cookies until the requests returns a reponse of 502 (Which I believe means cloudflare challange)?
Would something like this be possible to do?

yup...Hey @BarryThrill , refer to https://github.com/VeNoMouS/cloudscraper#retrieving-a-cookie-dict-through-a-proxy

Hmm alright! So I tried to use your example and also the retrieving-a-cookie-string method. What I tried first is that I tried to use ur code:
scraper = cloudscraper.create_create()
scraper.get('https://someite')

headers = scraper.headers
requests.get('https://somesite', headers=headers)

This just returned me a 503 response asking for Cloudflare challange.
I tried also to apply __cfduid and cf_clearance= in a cookie headers which also returned a response of 503 (Cloudflare challange).
This is my code:

def get_scraper_cookies():

	url = 'https://.....'
    tokens, user_agent = cloudscraper.get_tokens(url)
    return tokens

def main():
    connected = False
    while not connected:

        try:

            grab_cookies = get_scraper_cookies(thread)

            url = 'https://.....'

            headers = {
                'Connection': 'keep-alive',
                'Cache-Control': 'no-cache',
                'dnt': '1',
                'Upgrade-Insecure-Requests': '1',
                'User-Agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/70.0.3538.110 Safari/537.36',
                'Accept-Language': 'sv-SE,sv;q=0.9,en-US;q=0.8,en;q=0.7,de;q=0.6',
                'cookie': '__cfduid=' + headers_done['__cfduid'] + '; cf_clearance=' + headers_done['cf_clearance']
            }

            proxies = get_random_proxy()
            r = requests.get(url, headers=headers, proxies=proxies, timeout=5)
	    print(r.status_code)


pull request for e2e test at 01/22/2017 16:15:17

:white_check_mark: Validation status: passed
For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Rails 4 - scss warnings in server log

Warning. Error encountered while saving cache c0654524501a4d3513caee58728144b72b841d40/*.scssc: can't dump anonymous class #<Class:0x00000005c247a0>

This warnings appear when parsing scss, it seems like a bug in sprockets.
https://github.com/sass/sass/issues/1144
https://github.com/sass/sass/issues/1028

I'm not able to reproduce this issue. The first issue has a solution to update sass and sass-rails gems.  I've looked in Rails 4 Gemfile and the versions that are included are newer than what is mentioned in the issues linked.
Consider closing? @david-yz-liu

I just tried this out on the rails4 branch, and got the same error. sass and sprockets versions are what's in the Gemfile (newest, I presume). We can talk about this one more in person.

Issue was discussed in person. I'm using Ruby 2.1.3 (as opposed to Ruby 2.1.2) and not receiving any errors in the server console. Perhaps it has to do with this?


LanguageSelectorV3 (Refactoring old LanguageSelector)

LanguageSelectorV3
What does this PR do?

Resolve #442

I copied old LanguageSelector component, fixed it, and adjusted the design to unify it across all three applications.

@AngeloCG97 I copied old LanguageSelector component, made some fixes, and adjusted the design. I tested it via link to engage app. Please review.


Maintain repository ?

Hello,
Are you going to maintain MaterializeCSS ?
Thanks in advance for your reply

Yes we are. Curing the RC builds, we don't merge non-essential fixes. Since we have now released 1.0. We will merge PR's that are are clear in what they fix and also contain a codepen as an example of the current bug.


Support option to include/exclude all tests in a given test-suite

This adds support to provide just the testsuite name for the include or exclude option in order to include all test cases in the suite, or to exclude all test cases in that suite.
Addresses Issue #138

@oliverhu ready for review and if LGTM, merge.
BTW, is there a "group" I should tag for review and merges?

BTW, the tests passed in this job: https://travis-ci.org/linkedin/bluepill/jobs/232892274

@oliverhu addressed your code comments.

@oliverhu comments addressed!

@oliverhu comments addressed 🎉

method comment added to renamed expandTests method
expandTests moved under normalizeBPConfiguration
space after * removed

Ready for glory!

Added two more minor comments, approved, please address and merge it

@oliverhu changed from to with. Ready for your merge!

💯 🎆 📯 🕺


Updating digi and mixing scripts

Updates about database purposes and versions. Also fixed documentation.

Hi @soleti,
You have proposed changes to files in these packages:

Scripts

which require these tests: build.
@Mu2e/write, @Mu2e/fnalbuild-users have access to CI actions on main.
:hourglass: The following tests have been triggered for 1c63cca331d6a3c9f581db8f85818095ea039969: build  (Build queue has 1 jobs)
About FNALbuild. Code review on Mu2e/Offline.

:sunny: The build tests passed at 1c63cca331d6a3c9f581db8f85818095ea039969.

Test
Result
Details

test with
:white_check_mark:
Command did not list any other PRs to include

merge
:white_check_mark:
Merged 1c63cca331d6a3c9f581db8f85818095ea039969 at a3195de84ea665910b3b10fefc7a569156c758fb

build (prof)
:white_check_mark:
Log file. Build time: 12 min 05 sec

ceSimReco
:white_check_mark:
Log file.

g4test_03MT
:white_check_mark:
Log file.

transportOnly
:white_check_mark:
Log file.

POT
:white_check_mark:
Log file.

g4study
:white_check_mark:
Log file.

cosmicSimReco
:white_check_mark:
Log file.

cosmicOffSpill
:white_check_mark:
Log file.

ceSteps
:white_check_mark:
Log file.

ceDigi
:white_check_mark:
Log file.

muDauSteps
:white_check_mark:
Log file.

ceMix
:white_check_mark:
Log file.

rootOverlaps
:white_check_mark:
Log file.

g4surfaceCheck
:white_check_mark:
Log file.

FIXME, TODO
:white_check_mark:
TODO (0) FIXME (0) in 0 files

clang-tidy
:white_check_mark:
0 errors 0 warnings

N.B. These results were obtained from a build of this Pull Request at 1c63cca331d6a3c9f581db8f85818095ea039969 after being merged into the base branch at a3195de84ea665910b3b10fefc7a569156c758fb.
For more information, please check the job page here.
Build artifacts are deleted after 5 days. If this is not desired, select Keep this build forever on the job page.


Support unique keys in organization & application membership.changeRole() & remove()

Change-type: minor
Signed-off-by: Thodoris Greasidis<EMAIL_ADDRESS>
Contributor checklist

[ ] Includes tests
[ ] Includes typings
[ ] Includes updated documentation
[ ] Includes updated build output

it ends up with a strange logic to support it.

To which logic you refer? B/c supporting this didn't add any new code in the implementation, it only added tests.
Actually users could already do that if they didn't use/care about TS errors, b/c this PR is simply re-exposing what pine client already supports.
Imo it's more convenient than having to find the membership id and it also allows us to drop one extra custom method call from the UI.


Issue when using ttc/otc file as custom font

I try to let my egui app to use system font.
I can use some font query/loading library to find the font location and read it as binary data, however, it's a ttc file(for example, Microsoft YaHei), so egui report the data a invalid when parsing.
It seems ab_glyph do support collection, by using FontRef::try_from_slice_and_index, so any idea how can I use ttc/otc ffont file as custom font in egui?
update
Maybe there is some bug in the font loading library I used.
I changed from font-loader to fontdb, and now it's working, no invalid font report.
But, IIRC,  font loading library gives whole file content, and because I can't pass the font face index to egui, current it will always using the font face at index 0, which means when using ttc/otf, the render result may not be as expected.

For example, Helvetica Neue(Normal style) is in index 10

If you could specify the face index in epaint::FontDefinitions (together with the FontData), would that solve the problem for you?

Yes, I think add a font_face_index: HashMap<String, usize> field in FontDefinitions (with the same key in font_data, values is the index) is a feasible solution.
Considering compatibility, a default value of 0 can be used when the index cannot be found in this field.

It seems ab_glyph::FontArc lacks the convenience of try_from_vec_and_index and try_from_slice_and_index. If you make a PR to ab_glyph to add those I can hook them up in egui later!

FontArc does implement From<FontRef> though. That should be enough?

Good point @parasyte - I made a PR; please try it out @7sDream !

Thank you! I can try this pr tonight, after work


Update deye_4mppt.yaml

Some small improvements.

Sorry to bother you..
but within this patch only PV1 current got added the missing
state_class: "measurement"

PV3 and PV4 current are still missing this addition

Sorry to bother you.. but within this patch only PV1 current got added the missing
state_class: "measurement"

PV3 and PV4 current are still missing this addition

Sorry you are right, fixed it.


Use the official Postgres image

This would avoid you to maintain the postgres part. The Postgres Docker image has evolved a lot in a year, and today the current stable is 9.5.
This Docker image looks awesome, but I don't want to downgrade my DB.

This repo description is about Postgres 9.3, but the first line from the README.md file is about Postgres 9.5. In my opinion, the Postgres version shouldn't be specified.
A tag of this repo docker image should match with each tags from the offical postgres image like the following:

Official Postgres image
Docker Wall-e

9.3.11
9.3.11

9.4.6
9.4.6

9.5.1
9.5.1

Great idea! I've kinda let this image decay since I don't use it much anymore. Would like to replace supervisord with a better init too.

I've kinda let this image decay since I don't use it much anymore.

I guess it's still the case so I'm closing this issue.


Store sensitive information on Android

I'm trying to migrate an ObjC app to RN in order to add support for Android, and one of my concerns is that in ObjC I'm using Keychain to store OAuth Login information, and I don't know if there's a way to replicate this kind of secure storage on Android with RN.
I know how to create a module in ObjC and then export it to RN in order to add the functionality, and there's also this library that allows me to work with Keychain in iOS, but I couldn't find anything about storing sensitive data in a secure way on Android.
Any ideas that doesn't involve me learning Java :P?
Thanks!

AFAIK there isn't anything out there yet. As a meta point though, I highly recommend being familiar with Java+Android and Obj-C+Cocoa if you're using React Native. It's ~so~ much better when you or someone you're working with writes a native module to meet your needs.

@ide Thanks for the quick answer, and yes, I code in ObjC so I can create the modules I'm missing for iOS, but I didn't want to learn Java :stuck_out_tongue_closed_eyes: .
Thanks again!


[Jenkinsfile] MLIR is not tested on gfx908.

This issue is originated from #147, please see https://github.com/ROCmSoftwarePlatform/MIOpen/pull/147#pullrequestreview-702561380 for details.

via #1027


Fix Issues with Account Creation Flow from QA Session

Related Issue or Background Info
Fixes multiple issues from different QA sessions. (Apologies for another mega-PR!)
The biggest issues addressed are:

page back throws a 404
error messages not being correctly passed from the backend, resulting in a vague "API error" message
Okta rejected phone numbers with dashes
numerous small copy improvements
addresses some feedback from PR/1817, in particular changing some endpoints to return JSON instead of strings and using the @SessionAttribute tag

The full list of issues is in this document.
Changes Proposed
New getUserStatus Endpoint
In order to handle the back case properly, we added an endpoint to fetch the user's status. This endpoint is called on initial page load, as well as anytime the user tries to go back.
This change is also why this PR is so large - we needed to add status tests to LiveOktaAuthenticationTest, which requires regenerating all the Okta WireMock mappings. Any changes under /mappings can probably be safely ignored, since they just reflect the new request/responses from Okta. (Many of those changes are also very inconsequential since just the user id or activation id changed, but that changes the filename too.)
Changes to Initialization
As part of the changes for getStatus, we had to change the way initialization is done. Now user initialization happens immediately on page load, and is a separate endpoint from setPassword.
Properly Displays Errors
The frontend now correctly displays API error messages. The copy of some messages was also changed.
Other

Removes "Return to previous page" links
Adds some validation to the recovery questions page
stopped using redux store since backend can provide user state

Screenshots / Demos

Checklist for Author and Reviewer
Deploy to test, create a new user in Okta, visit https://test.simplereport.gov/app/uac/?activationToken=token w/ token from email, make sure flow issues are fixed
UI

[ ] Any changes to the UI/UX are approved by design
[ ] Any new or updated content (e.g. error messages) are approved by design

Testing

[x] Includes a summary of what a code reviewer should verify

Changes are Backwards Compatible

[x] Database changes are submitted as a separate PR

[x] Any new tables that do not contain PII are accompanied by a GRANT SELECT to the no-PHI user
[x] Any changes to tables that have custom no-PHI views are accompanied by changes to those views
(including re-granting permission to the no-PHI user if need be)
[x] Liquibase rollback has been tested locally using ./gradlew liquibaseRollbackSQL or liquibaseRollback

[x] GraphQL schema changes are backward compatible with older version of the front-end

Security

[ ] Changes with security implications have been approved by a security engineer (changes to  authentication, encryption, handling of PII, etc.)
[ ] Any dependencies introduced have been vetted and discussed

Cloud

[ ] DevOps team has been notified if PR requires ops support
[ ] If there are changes that cannot be tested locally, this has been deployed to our Azure test environment for verification

SonarCloud is complaining about potential security vulnerabilities (because a URL is constructed from session info) but there's no real way around that as far as I understand it.

@nullflux sorry, just saw your question up top about QA testing! We've been doing a ton of QA-ing for this PR, had one session last week and one this morning. As of now I think it's in as good a state as we can hope for before deploying to prod 🤞