Added support for directives

Example:
'use stricts'
const GraphQLError = use('Adonis/Addons/GraphQLError')

module.exports = {
  async isAuthenticated (next, src, args, { auth }) {
    try {
      await auth.check()
      return next()
    } catch (error) {
      throw new GraphQLError('User has to be authenticated')
    }
  }
}

There can be issues I started to learn GraphQL yesterday.

I don't like much of the idea to create another Directives folder but it's better than nothing.

Hey @alanaasmaa! 👋
Thanks for this contribution! One thing, it seems that you got an issue with your history.
Could you try to fix it by "rebasing" and hard pushing?

Hey @RomainLanz,
I assumed you are gonna ask this 😅
I'm not very comfort by rebasing but I have to learn that skill at some point.
I appreciate very much what you have done to make AdonisJS a great framework.

I think this pull request also need one change.
Currently, I assume it's not backward compatible. Those who update via npm/yarn will get errors when they do not create Directives folder. Should we make the folder optional?

Hey is there any progress on this or a way I can help with the testing?


Fix license badge

The link is outdated. I just updated the link to license in main branch.

Closing in favor of #2358; thanks for your attention to detail!

Yep. I didn't notice there's already one.


Update markdown

Hello,
I remove the Markdown class which was used to convert markdown to html.
I used the markdown_to_html filter instead.
I run the tests after and hey still works.

I like this! Thanks @abdounikarim.
However, we need to revert the changes related to tgalopin/html-sanitizer-bundle because we want to keep that bundle and its features. Thanks!

I see now why you removed the HTML sanitizer bundle ... but instead of that ... we may add a new Twig filter to do this -> ... | sanitize_html   I don't know which could be the best option here ... maybe @tgalopin can suggest us something. Thanks!

After talking with Titouan, he told me that the Sanitizer bundle already provides a sanitize_html Twig filter ... so we may try to do the following:

Reinstall and reenable the HtmlSanitizer bundle as before
Tweak the needed Twig templates to add the |sanitize_html filter after the Markdown filter

After talking with Titouan, he told me that the Sanitizer bundle already provides a sanitize_html Twig filter ... so we may try to do the following:

Reinstall and reenable the HtmlSanitizer bundle as before
Tweak the needed Twig templates to add the |sanitize_html filter after the Markdown filter

@javiereguiluz done

Thanks for this contribution Karim!


Build failure with "CGO_ENABLED=0" in the Makefile

Is this a BUG REPORT or FEATURE REQUEST?:
BUG
What happened:
Build is failing with CGO_ENABLE=0
What you expected to happen:
You will see the following error message during the build:
CGO_ENABLED=0 go run vendor/github.com/gobuffalo/packr/packr/main.go build -v -i -ldflags '-X github.com/argoproj/argo.version=2.1.0 -X github.com/argoproj/argo.buildDate=2018-04-02T20:15:32Z -X github.com/argoproj/argo.gitCommit=fe23c2f651a61a2d7aa877a86edff9802d7b5b47 -X github.com/argoproj/argo.gitTreeState=dirty -X github.com/argoproj/argo/cmd/argo/commands.imageTag=latest' -o /Users/hwang8/dev/src/github.com/argoproj/argo/dist/argo ./cmd/argo
net
os/user
go install os/user: open /usr/local/go/pkg/darwin_amd64/os/user.a: permission denied
go install net: open /usr/local/go/pkg/darwin_amd64/net.a: permission denied
Error: exit status 1
Usage:
  packr build [flags]

Flags:
  -h, --help   help for build

exit status 255
make: *** [cli] Error 1
**How to reproduce it (as minimally and precisely as possible)**:
I just installed the golang 1.9.5 and ran the build, not sure if this only applies to the new dev environment.
**Anything else we need to know?**:

**Environment**:
- Argo version:
``` N/A
$ argo version
``` N/A
- Kubernetes version :
``` N/A
$ kubectl version -o yaml
``` N/A

**Other debugging information (if applicable)**:
- workflow result:
``` N/A
$ argo get <workflowname>
```N/A
- executor logs:
```N/A
$ kubectl logs <failedpodname> -c init
$ kubectl logs <failedpodname> -c wait

workflow-controller logs:

$ kubectl logs -n kube-system $(kubectl get pods -l app=workflow-controller -n kube-system -o name)

The workaround is to delete the CGO_ENABLED=0 in the make file.
There is a golang issue that is discussing the similar issue: https://github.com/golang/go/issues/18981

The root cause is still from golang/go#18981, the following works for me:
Install go via brew:
brew install go
export PATH=$PATH:/usr/local/opt/go/libexec/bin

Instead of installing golang via downloading the package from golang official website, brew is installing the go at the /usr/local/opt/go/ with permission set differently.

Official GO:
MTVL1679decd5:argo hwang8$ ls -l /usr/local/go/pkg/darwin_amd64/os/user.a
-rw-r--r--  1 root  wheel  100768 Mar 28 21:37 /usr/local/go/pkg/darwin_amd64/os/user.a

Brew GO:
MTVL1679decd5:argo hwang8$ ls -l /usr/local/Cellar/go/1.10.1/libexec/pkg/darwin_amd64/os/user.a
-rw-r--r--  1 hwang8  admin  101548 Mar 28 21:28 /usr/local/Cellar/go/1.10.1/libexec/pkg/darwin_amd64/os/user.a

It is usr and group permission issue.


On local terminal reload, scrollback is not restored

Steps to Reproduce:

run tree or something with output longer than the size of the terminal window
reload the window
try to scroll

resolved by Daniel during PR cleanup


HA 2023.5.0 breaks this integration

Upgrading to HA 2023.5.0 breaks the integration.
I don't have log files handy because I downgraded, but if necessary, I'll upgrade again to provide them.

I had the same problem, I paste the contents of the log file. I hope this helps .
Thank you
2023-05-05 12:57:10.695 WARNING (SyncWorker_2) [homeassistant.loader] We found a custom integration brink_home which has not been tested by Home Assistant. This component might cause stability problems, be sure to disable it if you experience issues with Home Assistant
2023-05-05 12:57:20.187 ERROR (MainThread) [homeassistant.config_entries] Error setting up entry (myuser) for brink_home
File "/config/custom_components/brink_home/init.py", line 88, in async_setup_entry

Sorry, I'll add this too
This error originated from a custom integration.
Logger: homeassistant.config_entries
Source: custom_components/brink_home/init.py:88
Integration: Brink-home Venitlation (documentation, issues)
First occurred: 12:57:20 (1 occurrences)
Last logged: 12:57:20
Error setting up entry (myuser) for brink_home
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/config_entries.py", line 387, in async_setup
result = await component.async_setup_entry(hass, self)
File "/config/custom_components/brink_home/init.py", line 88, in async_setup_entry
hass.config_entries.async_setup_platforms(entry, PLATFORMS)
AttributeError: 'ConfigEntries' object has no attribute 'async_setup_platforms'

I will try to fix it, it should be very easy fix

The fix should be released with 1.1.0 version


proposal for fabric-secure-connector

Background
One question that every person designing a HL fabric based solution ask is : What the hack should I do with private key and certificates of clients? . Although fabric-sdk provide two different kind of approaches.

storing client's private key and certificate together on some database (which implements wallet interface). For this approach a rough flow goes something like this

client send request to org's application server to invoke or query chaincode using a give KEY.
org's application upon receiving the request , it fetches private key and certificate corresponding to KEY.
uses the fetched private key to sign fabric message in order to query or invoke chaincode. NOTE : developer of the application has to implement authentication for client's KEY

store private key in HSM , but in this approach application and HSM have to be present on same physical machine (i.e HSM should directly be connected with machine running the application). This approach is mostly used for developing client's desktop application, which will be responsible for communicating with fabric peers.

Major Advantage in First approach : clients won't be responsible for communicating with fabric network
Major Disadvantage in First approach : all private key are stored in single database which leads to money pot for hackers to exploit .
Major Advantage in Second approach : client's private key are never exposed to org's application.
Major Disadvantage in Second approach : clients are responsible for communicating with fabric network

Now comes the fabric-secure-connector which only brings the advantages from both the approaches. Secure Fabric connector provides a solution to the fabric organization for managing their client's private key such that the client's private key is never brought to Node Server for singing. Currently fabric-secure-connector provide a option of keeping client's private key into Vault Transit Engine or in browser Extension (TODO).

Broadly this package will support two kind of identity

Key stored with vault transit engine like server. (which provide cryptography as service)
key stored with client's extension and signing will be done via various transport connection (ws , gRPC)

Another component , is browser extension for

Storing private key (elliptical curve key of size 256 and 384)
singing message digest sent to it via transport layer.

NOTE : design of extension is TBD
FAQ :

Can this not be implemented in cactus-plugin-ledger-connector-fabric ?Ans : Can be , but will lead to lot of breaking changes. developer can choose cactus-plugin-ledger-connector-fabric for the first approach and fabric-secure-connector to support multiple type of identity.

Is their any use case of it ?Ans: currently we are trying to implement vault and ws identity  for blockchain-carbon-accounting project (under hyperledger labs)

Any Blockage that can be faced during the implement fabric-secure-connector?
Ans : currently fabric-sdk-node support sync singing of message , here

export interface ICryptoSuite {
         //........
	sign(key: ICryptoKey, digest: Buffer): Buffer;
}

 but since for Identity supported by this package will require sign method to be as async , so current fabric-sdk-node cannot be directly used. To solve this I have already opened a PR to fabric-node-sdk here . In meanwhile they merge the PR and release version 2.2, I have pushed a npm package of my fork.  PR merged

Where can we find an implementation of it?Ans: https://github.com/Zzocker/blockchain-carbon-accounting/tree/secureFabric/secure-fabric . this support vault identity for now.

Thank you
@petermetz @sichen1234 @brioux @knagware9 @udosson
Below are technical details of how I am planning to implement
Top Level exported class and interface
export enum IdentityProvidersType {
     // identity by default provided by fabric-sdk-node wherein private key and certificate
     // stored together (First approach)
     Default = 'Default-X.509'

     // vault identity provider wherein private key are store with vault transit engine
     // and certificate are store in certData store
     Vault = 'Vault-X.509',

     // [ TRANSPORT ] WebSocket identity provider wherein private key are store with client inside the extension
     // signing of digest are done by sending of data as websocket message through webSocket connection between
     // server and client extension
     // certificate are store in certData store
     WebSocket = 'WS-X.509',

     // [ TRANSPORT ] gRPC identity provider wherein private key are store with client inside the extension
     // signing of digest are done by sending of data as gRPC streaming via gRPC bi-directional connection
     // between server and client extension
     // certificate are store in certData store
     gRPC = 'gRPC-X.509',
}

// every identity  type and certificate data will extends this interface
interface IIdentity{
     // key used for storing identity data with cert datastore
     key:string
     type:IdentityProvidersType
}

// IVaultIdentity represents a vault client.
export interface IVaultIdentity extends IIdentity{
     keyName:string
     token:string
}

// IWebsocketIdentity represents a websocket client.
// message signing will be done by back and forth message between
// server and client over weConn
export interface IWebsocketIdentity extends IIdentity{
     wsConn:connection
}

// IGRPCIdentity represents a bi-directional gRPC client.
// message signing will be done by back and forth message between
// server and client over biConn
export interface IGRPCIdentity extends IIdentity{
     biConn:connection
}

// IIdentityData : data that will be stored with cert datastore
// with key as client's commonName (from X509 certificate) and value as following field
interface IIdentityData extends IIdentity{
     credentials: {
          certificate: string;
          // if identity type is IdentityProvidersType.Default
          privateKey?:string;
     };
     mspId: string;
}

export interface ISecureFabricConnector{
     // array of identity types that application is going to support
     // eg [IdentityProvidersType.Vault , IdentityProvidersType.Default IdentityProvidersType.WebSocket]
     // this will accept client request with vault , default or websocket for signing fabric messages
     supportedIdentity:IdentityProvidersType[]

     // vault server config if Vault identity is support
     vaultOptions?:{endpoint:string,transitEngineMountPath:string}

     // for registering client : NOTE /register endpoint should not be exposed to client
     // rather this endpoint is for org's admin
     registrar : {
          certificate:string
          mspId:string
          // if privateKey is provided , this private will be used for signing
          privateKey?:string

          // if provided , application will use private key of registerer stored with vault
          // to register and revoke client
          vaultKey?:{token:string,keyName:string}
     }
     // usual field required in fabric-sdk-node's GatewayOptions
    connectionProfile:object
    tlsInfo?: {
        certificate: string;
        key: string;
    };
    discovery?: DiscoveryOptions;
    eventHandlerOptions?: DefaultEventHandlerOptions;
    queryHandlerOptions?: DefaultQueryHandlerOptions;
    'connection-options'?: any;
}

export class ISecureFabricConnector{
     constructor(ISecureFabricConnector)

     /**
      * @method transact invoke/query fabric chaincode
      * @param caller client identity
      * @param channel on which chaincode is committed
      * @param ccName : fabric chaincode name
      * @param method supported by chaincode
      * @param args , method specific arguments
      */
     transact(type:'query'|'invoke',caller:IIdentity,channel:string,ccName:string,method:string,...args:string[]):Promise<Buffer>

     /**
      * @method enroll a already registered client
      * @param caller client identity
      * @param request for enrollment
      */
     enroll(caller:IIdentity,request:{enrollmentID:string,enrollmentSecret:string}):Promise<void>

     /**
      * @method rotateKey will rotate client key and store newly enrolled certificate in cert datastore
      *
      */
     rotateKey(caller:IIdentity):Promise<void>
}

@Zzocker This looks awesome! The only thing I'd change is to make this part of the current Fabric connector by way of introducing it as a signing credential type and it's own endpoint. That way we don't need to duplicate a lot of code between the current Fabric connector and the one being proposed here. I shall help out with the code changes!

@petermetz , yes you are right about creating a another plug-in for this will lead to duplication of code. But I will say, that if we implement it in existence connector, will lead to breaking changes and package version might have to upgraded to 1.x.y.

@Zzocker Pre-1.0 breaking changes can be made between minor releases as well. In general, anything that's 0.x.x as a version is considered unstable (that's why I keep telling everyone that Cactus is not ready yet for production use). TLDR: I'm okay with breaking changes as long as we can complete them before the 1.0 release.


[1.x] Replace custom phpdoc_align rule with php-cs-fixer option

Replaces custom phpdoc_align rule with php-cs-fixer option
This also adds an additional test case
Fixes tests, the output for check style is now different:

<checkstyle version="PHP CS Fixer 3.47.0 Big Changes by Fabien Potencier, Dariusz Ruminski and contributors.">
  <file name="/Users/kiekbjul/Home/opensource/laravel-pint/tests/Fixtures/with-fixable-issues/file.php">
    <error severity="warning" source="PHP-CS-Fixer.new_with_parentheses" message="Found violation(s) of type: new_with_parentheses"/>
  </file>
</checkstyle>

format code with the new pint version.

php-cs-fixer phpdoc_align docs: https://cs.symfony.com/doc/rules/phpdoc/phpdoc_align.html

@eusonlito What exactly is your comment about?

After updating to the latest release I got an error The rules contain unknown fixers: "Laravel/laravel_phpdoc_aligment".
I had this rule as false so I leave the link to the php-cs-fixer rule as reference if anyone wants information on how to customize it.

Well, since updating from 1.13.8 to 1.13.9 I've been having issues with pint no being able to fix my issues.
sail pint app/Models/ -v

.....!.............!..!.........!.......

  ! app/Models/A.php                                                                      syntax error, unexpected token ">"
  ! app/Models/B.php                                                                       syntax error, unexpected token ">"
  ! app/Models/C.php                                                                      syntax error, unexpected token ">"
  ! app/Models/D.php                                                 syntax error, unexpected token "=", expecting "=>"

An error from one of the files are:
 ! app/Models/A.php                                                                      syntax error, unexpected token ">"
     85▕             get: function () {
     86▕                 $disk = Storage::disk('images');
     87▕
     88▕                 return [
  ➜  89▕                     'logo'     => isset($this->images['logo']) ? $disk->url($this->images['logo']) : null,

Only change I did to my files was to remove the "Laravel/laravel_phpdoc_alignment": false rule from pint.json, since after this PR it no longer exists.
Other who has issues with this?

@phh probably something with another php-cs-fixer rule. I will send another PR, which bumps the php-cs-fixer version to the patch which was released.

@phh can you send a code snippet, so that I can confirm if the new patch version would work?

@Jubeki Could reproduce on a fresh laravel app with:
pint.json:
{
  "preset": "laravel",
  "rules": {
    "binary_operator_spaces": {
      "default": "align_single_space_minimal"
    }
  }
}

app/Models/TestModel.php
<?php

namespace App\Models;

use Illuminate\Database\Eloquent\Model;
use Illuminate\Support\Facades\Storage;

class TestModel extends Model
{
    public function imageUrls()
    {
        $disk = Storage::disk('test');

        return [
            'a' => $disk->url($this->images['a']),
            'aa' => $disk->url($this->images['aa']),
        ];
    }

    protected function someMethod($items = []): void
    {
        if (count($items) > 0) {}
    }
}


Token filters - word_delimiter [DOC]

What do you want to do?

[ ] Request a change to existing documentation
[x] Add new documentation
[ ] Report a technical problem with the documentation
[ ] Other

Create a separate page for Token filter: word_delimiter
Part of META issue: https://github.com/opensearch-project/documentation-website/issues/1483
*Version: all

I can take this one

This token filter uses deprecated WordDelimiterFilter
Should we remove this entirely from the docs not to add confusion, or mark it as deprecated?


Ability to use Master Key

Is there a capability to use a FileVault Master Password with the Crypt client?
For reference: https://support.apple.com/en-us/HT202385

No, as I consider this to be insecure.
Sent from my iPhone
On Wed, Jan 13, 2016 at 10:55 AM -0800, "Al Berman"<EMAIL_ADDRESS>wrote:
Is there a capability to use a FileVault Master Password with the Crypt client?
For reference: https://support.apple.com/en-us/HT202385
—
Reply to this email directly or view it on GitHub.

Hey Mr. Berman, there was an issue previously that made it you could not reliably rotate the master/'institutional' key without decrypting/re-encrypting/touching every machine. https://plus.google.com/+ClayCaviness/posts/3uu9h3sRpNd
Could you fork/modify the code to do distinct escrow of each master password? Maybe. You could also use something like https://github.com/taniguti/FileVault_Rescue, but in any case we'd want to convince you to use the individual/personal key and Crypt2 while you're at it...


[DRAFT] 2.4 Windows Github Action

Basic setup for creating a GitHub Action to run test on push, merge, etc. for Windows.
This is a draft PR and there are several things about it that require further work. Namely:

It is not clear how the installation files are fetched when Docker is not used (without Docker being used, cannot call Docker hub)
The plugin installation process is not successful and the current version makes use of the broken Windows core version where security does not function.
The final integration tests are built with Docker in mind so this requires migration.

Testing
So far the following functionality has been manually checked on a Windows instance:

Enable longer filenames
Gradle assemble security
Hardcoded 2.3.0 Snapshot pull (command prompt)
Zip extraction and old zip deletion --> The extracted file organization is that of a snapshot so is built for Docker
Starting .\bin\opensearch.bat (powershell) --> This currently fails due to bad security plugin install
Curl get https://localhost:9200 -u 'admin:admin' -k -v (command prompt)
The intermediate steps have not been confirmed to work yet.

Check List

[ ] ~New functionality includes testing~
[X] New functionality has been documented
[X] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

@scrawfor99 It looks like the DCO wasn't signed on one of those commits, could you amend the commit and force push to your branch?
Note; it might be a good time to rebase while you are at it.

Starting .\bin\opensearch.bat (powershell) --> This currently fails due to bad security plugin install

What error message are you receiving? The available windows distribution does not currently contain the .bat file we're seeking to add with this PR and associated PR that adds the bat file. For development of this action you will need to add the file manually in plugins\opensearch-security\tools so its available to be executed.

@scrawfor99 It looks like the DCO wasn't signed on one of those commits, could you amend the commit and force push to your branch?
Note; it might be a good time to rebase while you are at it.
Took care of this. I must think I accidentally mixed something up before.

Starting .\bin\opensearch.bat (powershell) --> This currently fails due to bad security plugin install

What error message are you receiving? The available windows distribution does not currently contain the .bat file we're seeking to add with this PR and associated PR that adds the bat file. For development of this action you will need to add the file manually in plugins\opensearch-security\tools so its available to be executed.

Starting .\bin\opensearch.bat (powershell) --> This currently fails due to bad security plugin install

What error message are you receiving? The available windows distribution does not currently contain the .bat file we're seeking to add with this PR and associated PR that adds the bat file. For development of this action you will need to add the file manually in plugins\opensearch-security\tools so its available to be executed.

So the original action involves building with gradle but the snapshot that is downloaded from the Windows-team link comes prebuilt so I think the solution is going to be building the security plugin and then move it into the core build instead of trying to build both. I am not sure if this is what you meant but it seems like the best option.


Issue with jupyter

After installing nppas (open3d-vedo-conversion branch) into a clean devbio-napari 0.8.0 installation, I receive an error in Jupyter when attempting to save notebooks:
File Save Error for demo.ipynb
Unexpected error while saving file: docs/demo.ipynb 'coroutine' object is not subscriptable

We should make sure that our installation instructions are not causing this,

The error might be related to:
https://github.com/jupyter-server/jupyter_server/issues/1127

Workaround:
mamba install jupyter_server=1.23.3 -c conda-forge

closed in: https://github.com/haesleinhuepf/devbio-napari/pull/33


[Function Request] Peer setup -> Bind to IP

Hey there :-)
For running several applications on my home Server - it is needed to have 2 IP addresses.
Because some apps need exactly "that one port" and you have more than one app also want to have exactly this port.
So it would be really great to bind the peer to an IP and not having em on <IP_ADDRESS> because that leads currently to two instances where you can update and restart - but the communication is only working really good with one of them.

Those are exactly the same instance but it is connected to two seperate ips.
:-)
If that would not be that big thing -> It would be very nice!
Thanks

Hi,
can you please try to run fhempy with -i192.168.0.x (the IP address you want to be used)? This option defines the IP address which will be used in zeroconf to advertise the fhempy service.

Hey ;-)
Thanks for the help.
I run it as a services.
Can I set it in the service file or in a config file?

If you run it as a service you just need to add the option in the fhempy.service file.

Hey there :-)
tried:
`# Source: https://github.com/fhempy/fhempy
ATTENTION
This is an example, please verify your directories and user

Copy this file to /etc/systemd/system/fhempy.service
Start service with "sudo systemd start fhempy"

[Unit]
Description=fhempy
Wants=network.target
After=network.target
[Service]
User=jorin
Group=jorin
WorkingDirectory=/home/jorin/
ExecStart=/home/jorin/.local/bin/fhempy -i<IP_ADDRESS>
Restart=always
[Install]
WantedBy=multi-user.target
`
get:
/etc/systemd/system$ sudo netstat -tulpen | grep 15733 tcp        0      0 <IP_ADDRESS>:15733           <IP_ADDRESS>:*               LISTEN      1000       80237984   1892016/python3

Sorry, I got it wrong.
I added a new parameter -b where you can specify the IP address to bind to.
Just update to the latest version and use -b instead of -i. The updated version is going to be available in 15min.

Hey there, cool thanks!
So like this?
Hey there :-)
tried:

# ATTENTION
# This is an example, please verify your directories and user
#
# Copy this file to /etc/systemd/system/fhempy.service
# Start service with "sudo systemd start fhempy"
#

[Unit]
Description=fhempy
Wants=network.target
After=network.target

[Service]
User=jorin
Group=jorin
WorkingDirectory=/home/jorin/
ExecStart=/home/jorin/.local/bin/fhempy -b<IP_ADDRESS>
Restart=always

[Install]
WantedBy=multi-user.target

Thanks for testing! Fix will be available in a few minutes.

--bind <IP_ADDRESS> works
-b <IP_ADDRESS> seems not.
For me that is okay ^^  :-)
Thanks

It only works if you remove the blank between b and 1. I don't know why 😕
Nevertheless, great to see that it works! Thank you for your feedback!

Thanks for fixing :-)
Anytime agin.


Adding Notification/Geofence Processing

Status
READY
JIRA Ticket
SPACON-315
Description
Geofencing checks and notification sending both have an input channel that components can put! messages onto.
Todos

[ ] Tests
[ ] Documentation
[ ] License

Steps to Test or Reproduce
Outline the steps to test or reproduce the PR here.
git fetch --all
git checkout <feature_branch>
npm test

@boundlessgeo/spatial-connect

👍


Batching/delaying claim transactions

This issue is about RBF batching all refund and claim transactions that are still in the mempool

Since most users won't have too many claim transactions to batch at a time, we can implement deferring claim transaction broadcast for a certain number of blocks or a specific block height in the future and e.g. only claim once very 24h.

For our BTCPay plugin this actually becomes relevant


[Feature]: Addition of a response for support related issues

I have seen several issues raising support related requests at github, wouldn't it be cool if a command is added to K8s-ci-robot/prow that will ping the author of issue with a message, whenever we put kind/support label to an issue:-
Input command:
/kind support
Output generated by bot:
@Author_of_issue, GitHub is not the right place for support or troubleshooting related requests. If you're looking for help, you could ask on [Server Fault](https://serverfault.com/questions/tagged/kubernetes). You can also post your question on the [Kubernetes Slack](http://slack.k8s.io/) or the [Discuss Kubernetes](https://discuss.kubernetes.io/) forum.

/kind feature

/remove-lifecycle rotten

/remove-lifecycle rotten

It's been a year and nobody seems interested in this feature request.

/close


Json file format additions

Description
built on @InnocentBug work, same logic just with full docs and better variable names
Changes

wrote full docs with doctest code example
added type hinting to the function
added example JSON
changed the name get_self_contained_json because it is not very self documenting and does not fully convey the function's functionality to get_expanded_json.
renamed test function
renamed variables within the test function
added docstrings to test function

Screenshots

Known Issues
Notes

the docstrings are a bit long, but I think it is completly worth it because it becomes super clear as to what this code is doing.

Checklist

[ ] My name is on the list of contributors (CONTRIBUTORS.md) in the pull request source branch.
[ ] I have updated the documentation to reflect my changes.

thanks for making my mess nice and beautiful.

of course, I'm glad I could help! Beautiful code is happy code! 😄


After hover, and mouseoff, "Cannot read property 'removeAttribute' of null

After mouseoff of the tooltip, this error is thrown.

bundle.nocsf.js?browserId=other&minifierType=js&languageId=en_US&b=7010&t=249:1059 Uncaught TypeError: Cannot read property 'removeAttribute' of null
at Tooltip.syncAlignElement (bundle.nocsf.js?browserId=other&minifierType=js&languageId=en_US&b=7010&t=249:1059)
at Tooltip.fireStateKeyChange_ (bundle.nocsf.js?browserId=other&minifierType=js&languageId=en_US&b=7010&t=249:251)
at Tooltip.syncStateFromChanges_ (bundle.nocsf.js?browserId=other&minifierType=js&languageId=en_US&b=7010&t=249:254)
at Tooltip.handleStateChanged_ (bundle.nocsf.js?browserId=other&minifierType=js&languageId=en_US&b=7010&t=249:251)
at Tooltip.emit (bundle.nocsf.js?browserId=other&minifierType=js&languageId=en_US&b=7010&t=249:185)
at State.emitBatchEvent_ (bundle.nocsf.js?browserId=other&minifierType=js&languageId=en_US&b=7010&t=249:284)
at run (bundle.nocsf.js?browserId=other&minifierType=js&languageId=en_US&b=7010&t=249:121)
at runIfPresent (bundle.nocsf.js?browserId=other&minifierType=js&languageId=en_US&b=7010&t=249:122)
at onGlobalMessage (bundle.nocsf.js?browserId=other&minifierType=js&languageId=en_US&b=7010&t=249:124)

@pragmaticivan Is that the same error you had on dashboard?

@eduardolundgren nop. it was related with the style property.

@ssmith353 can you post the version of metal-tooltip and metaljs you're using in your project?

"metal-tooltip" : "2.0.1",
"metal": "2.5.2"

@ssmith353 can you post here how you're creating the instance of this component?

/cc @fernandosouza Could you take a look on this one?

Within the render method of our form component we have:
			<div class="tooltip-wrapper">
				<Tooltip
					delay="[300, 150]"
					elementClasses={'fade'}
					position={Tooltip.Align.Right}
					selector={'[data-title]'}
					visible={false}
				/>
			</div>

with certain form fields containing this

sure @eduardolundgren. I will take a look on it urgently.

Hello, @ssmith353. Are you using Metal Router in the project?

@fernandosouza see our attached package.json dependencies.
		"immutable": "3.8.1",
		"metal": "2.5.2",
		"metal-ajax": "2.0.3",
		"metal-alert": "2.0.1",
		"metal-component": "2.5.5",
		"metal-debounce": "2.0.0",
		"metal-dom": "2.5.3",
		"metal-events": "2.5.3",
		"metal-incremental-dom": "2.5.7",
		"metal-jsx": "2.5.8",
		"metal-path-parser": "1.0.3",
		"metal-position": "2.0.0",
		"metal-promise": "2.0.1",
		"metal-react": "2.0.0",
		"metal-redux": "2.0.0",
		"metal-router": "3.0.8",
		"metal-soy": "2.5.7",
		"metal-state": "2.5.3",
		"metal-structs": "1.0.1",
		"metal-tooltip" : "2.0.1",
		"metal-uri": "2.0.2",
		"metal-useragent": "2.1.1",
		"react": "15.4.0",
		"react-dom": "15.4.0",
		"react-dropzone": "3.6.0",
		"react-select": "^1.0.0-rc.2",
		"redux": "3.6.0",
		"redux-immutable": "3.0.8",
		"redux-thunk": "2.1.0",
		"senna": "2.0.4",
		"string-sub": "0.0.1"

I've asked you about the metal-router because we're trying to figure out if there's something related to the error. I couldn't reproduce it by using only the component.
I've noticed you aren't using the latest version of the modules. Could you update and check if the error is still there, please?
"metal": "2.5.19",
"metal-ajax": "2.0.3",
"metal-alert": "2.0.1",
"metal-component": "2.5.18",
"metal-debounce": "2.0.0",
"metal-dom": "2.5.18",
"metal-events": "2.5.18",
"metal-incremental-dom": "2.5.19",
"metal-jsx": "2.5.19",
"metal-path-parser": "1.0.3",
"metal-position": "2.0.0",
"metal-promise": "2.0.1",
"metal-react": "2.0.0",
"metal-redux": "2.0.1",
"metal-router": "3.1.0",
"metal-soy": "2.5.19",
"metal-state": "2.5.18",
"metal-structs": "1.0.1",
"metal-tooltip" : "2.0.2",
"metal-uri": "2.0.2",
"metal-useragent": "2.1.2",

The issue here seems likely because the new lifecyle. When a parent component of a tooltip gets rerendered its child also gets rerendered and the tooltip has asynchronous behavior that were clearTimeout on dispose only and need to be cleared on rerendered as well. This might fix it.

@ssmith353 we are going to apply the fix on tooltip still

Thanks for your feedback, @ssmith353.
That seems to occur in certain circumstances because I could not reproduce this error even rendering the tooltip as a subcomponent.

@fernandosouza It happens if the parent component re-render while the tooltip is visible.

hey guys, just implemented the new version in our project, and is fixed! Thanks!


In UK, we call it 'maths' not 'math'

UK teachers and children are finding the 'math' palette confusing. Teachers report to us that children are saying 'they spelt maths wrong' or 'what does math mean'.
How could we deal with the country specific terminology differences like this in the PXT editor?

You could localize that category for "en-uk" locales (and other discrepancies).
Get Outlook for iOShttps://aka.ms/o0ukef

From: DavidWhaleMEF<EMAIL_ADDRESS>Sent: Saturday, April 22, 2017 2:35:10 PM
To: Microsoft/pxt
Cc: Subscribed
Subject: [Microsoft/pxt] In UK, we call it 'maths' not 'math' (#1950)
UK teachers and children are finding the 'math' palette confusing. Teachers report to us that children are saying 'they spelt maths wrong' or 'what does math mean'.
How could we deal with the country specific terminology differences like this in the PXT editor?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fissues%2F1950&data=02|01|jhalleux%40microsoft.com|e41629183aed4fcdc8fa08d489c774d4|72f988bf86f141af91ab2d7cd011db47|1|0|636284937130481376&sdata=%2B9Gw2PGa2tdTmE0Lk9xyZsUig4yg3qGGA6bzxAetDbA%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KXZ3OXmRHJluQsMHXmNFOfONgsHBks5rynKOgaJpZM4NFOOZ&data=02|01|jhalleux%40microsoft.com|e41629183aed4fcdc8fa08d489c774d4|72f988bf86f141af91ab2d7cd011db47|1|0|636284937130481376&sdata=mal4TW9JmB7GdFmDjIEvfjOvCazvbwKMEXC4E%2BXUyVI%3D&reserved=0.

Note that in JavaScript, Math is singular so it is probably best to keep it singular.
Get Outlook for iOShttps://aka.ms/o0ukef

From: Peli de Halleux
Sent: Saturday, April 22, 2017 2:37:22 PM
To: Microsoft/pxt; Microsoft/pxt
Cc: Subscribed
Subject: Re: [Microsoft/pxt] In UK, we call it 'maths' not 'math' (#1950)
You could localize that category for "en-uk" locales (and other discrepancies).
Get Outlook for iOShttps://aka.ms/o0ukef

From: DavidWhaleMEF<EMAIL_ADDRESS>Sent: Saturday, April 22, 2017 2:35:10 PM
To: Microsoft/pxt
Cc: Subscribed
Subject: [Microsoft/pxt] In UK, we call it 'maths' not 'math' (#1950)
UK teachers and children are finding the 'math' palette confusing. Teachers report to us that children are saying 'they spelt maths wrong' or 'what does math mean'.
How could we deal with the country specific terminology differences like this in the PXT editor?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHubhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2FMicrosoft%2Fpxt%2Fissues%2F1950&data=02|01|jhalleux%40microsoft.com|e41629183aed4fcdc8fa08d489c774d4|72f988bf86f141af91ab2d7cd011db47|1|0|636284937130481376&sdata=%2B9Gw2PGa2tdTmE0Lk9xyZsUig4yg3qGGA6bzxAetDbA%3D&reserved=0, or mute the threadhttps://na01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FAD-4KXZ3OXmRHJluQsMHXmNFOfONgsHBks5rynKOgaJpZM4NFOOZ&data=02|01|jhalleux%40microsoft.com|e41629183aed4fcdc8fa08d489c774d4|72f988bf86f141af91ab2d7cd011db47|1|0|636284937130481376&sdata=mal4TW9JmB7GdFmDjIEvfjOvCazvbwKMEXC4E%2BXUyVI%3D&reserved=0.

Ah yes, I see your point...

So, the same reason that if an API has a setColor() method, it should not be renamed to setColour() as it would break the API. Point taken.

For completeness, the same issue appears on other visual API's so I think you are right, we'll have to live with it.

(although e.g. for the German translation, the german word for Math will appear in the palette, but the word Math will appear in the generated javascript code).
It's a tricky balance. Treat it as general user feedback, but I'm not sure we can do anything about this.


Default Export - What is it?

During the last meeting I asked about the "instance" or "default" use of our modules having extra logic directly in the module instead of this "extra logic" living in workbox-sw. My hope with this is that we can sanitize our explanation of how workbox works (i.e. they'll be exposed to a module name for a particular task rather than something that covers over our module structure and use).
This raised the question, if a developer imports one of our modules what should they get?
import workboxRouting from `workbox-routing`;
import workboxPrecaching from `workbox-precaching`;
import workboxStrategies from `workbox-strategies`;

When I was prototyping build and usage of ES2015 modules, the mental model I took was that importing the module should result in something easy to use that does what I'd expect/
workbox-routing => Something that does routing
workbox-precaching => Something that does precaching
workbox-strategies => Something that has strategies
etc etc....
Talking informally to a few people, this is referred to as the "default export" or it's just "using the module". No one suggested they expected a Class or an object of classes, it was more along the lines of expecting what makes sense for that module.
I'd argue that the result should be the most common use case. ES2015 modules afford the ability to cleanly supply public classes in case our default isn't what someone wants as well as handling more intricate uses of our modules.
Anyway just wanted to open this for discussion since it appears to have slipped in the V3 doc cc @addyosmani @jeffposnick @philipwalton @prateekbh @mikefowler @deanhume @goldhand

After thinking about this a bit, I'm going to come down on the side of not exposing a default export for the ES2015 modules, pretty much anywhere. I like our model of using classes throughout the libraries, and configuring instances of those classes via their constructors, and don't feel like we need to shy away from requiring that.
I think we're still okay with moving some of the "magic" out of workbox-sw and into helper methods on the various classes used under the hood.

Hi @gauntface - this looks great!
Just thinking of this from the end users point of view (and apologies if this comes across as a silly question!) - as @jeffposnick mentioned, will users still be able to load in the additional modules as they need them using importScripts using the example above? For example:
importScripts('workbox-sw.prod.vX.X.X.js');
importScripts('workbox-routing');
And so on. Hope that makes sense!

@deanhume not a silly question at all.
Yes then will be able to. All our modules will come with a browser bundle as well (which will be available via a CDN as well).
So they can do the following:
importScripts('workbox-routing');

// This is the default export for the module
const routing = goog.workbox.routing.default;

// This would be a public class:
const Router = goog.workbox.routing.Router;

I've added some high-level "Default export should be " to v3 tracker just to ensure this isn't completely overlooked.


Make AndroidView take gesture recognizer factories.

Before this PR AndroidView's gestureRecognizers field was a list of
gesture recognizers. This was problematic as when the widget was rebuilt
with the same gesture recognizer instances we would try to re-join the
recognizers to a gesture arena team and crash (as a OneSeqeunceGestureRecognizer
team can only be set once).
With this change, we instead take a set of factories.
This allows AndroidView to create the gesture recognizers just before
adding them to the team, and thus be sure that they are only added once to a
team.
The factories are identified by the type of the object they create, this
allows AndroidView to know when it is given an equivalent set of gesture
recognizer factories, and do nothing in that case.
Fixes #21514.

Matching changes to maps and webview in flutter/plugins#778

Following offline discussion, removed the type based equality for factories, and added logic to compare the set of gesture recognizer types in the render object.
PTAL

 modulo changes


[New Feature]: a better loader

🎮 feature Request
a better relevant preloader can be set to the webpage
Point down the features
current pre loader:
https://github.com/ChromeGaming/Community-Page/assets/152693538/1ef92fa5-b8e5-41e9-a553-4e51f4e461a5
pre loaders suggested:

or
https://github.com/ChromeGaming/Community-Page/assets/152693538/296fabd0-3033-464e-a391-c62214c25fd8
i would like to work on this, please assign me this under gssoc and give appropriate labels
thankyou
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

the first suggested one will look great.

@Manaregr8

sorry,
unfortunately i got know that this animations resolution is bad on increasing the size which can disturb the webpage look.
so i came up with some more suggestions(these loaders aren't animations , they are made using html and css , so no resolution problem)

https://github.com/ChromeGaming/Community-Page/assets/152693538/71d8c148-f35a-45fc-b64d-eaa39da50b0b

https://github.com/ChromeGaming/Community-Page/assets/152693538/c8334761-0358-4653-abc3-1ab006b66957

https://github.com/ChromeGaming/Community-Page/assets/152693538/f0119467-5492-4218-9cfa-031ec58ef824

https://github.com/ChromeGaming/Community-Page/assets/152693538/35dcf68f-57ef-443c-89b5-9f427df68ed8
Could you please choose one of these , which better suits the website.
Thankyou
Note: the actual size of these preloaders may vary and colors will also be changed according to the website theme :)

@Manaregr8 plz choose one of the preloaders mentioned above, so I can make a PR .
Thankyou.


Build a cross-chain model

Build a cross-chain model using Hephaestus and the exported transactional data. This includes refactoring/using the Python code from Hephaestus. This integrate Hepheastus with SATP. SATP should create an instance of Hepheastus to create the model and do the monitoring in a later issue.
Requirements:

Obtain transactional data from the created cross-chain model endpoint.
Create a pipeline (Python, TS) that calls that endpoint and builds a cross-chain model from the provided data
Create a pipeline that updates the cross-chain model with the data from that endpoint from time to time
Create support scripts to automate the pipelines

I will do this task.


Sprint jumping on straight paths

To increase speed, the bot should have an option to sprint-jump on straight path. Needs careful tuning for overshoots and tights spaces.

Agreed. Manually setting the control state of the bot to sprint jump when traveling in a straight path results in the pathfinder being much less efficient.

Agreed. Manually setting the control state of the bot to sprint jump when traveling in a straight path results in the pathfinder being much less efficient.

e


kpt fn sink and kpt fn run format files differently

Expected behavior
kpt fn source | kpt fn run | kpt fn sink and kpt fn run $DIR have equivalent behavior
Actual behavior
Comments at the top of files are preserved with kpt fn run $DIR, not with the flow using kpt fn sink
Information

v0 of Kpt

@aw185176 Can you please post the example yaml content as well.

We will be eventually retiring support for v0. If you haven't already, please take a look at kpt v1(https://kpt.dev/) and guide to migrate existing v0 kpt packages to v1(https://kpt.dev/installation/migration).

v1 doesn't work for us, see #2302.  Design process was behind Google auth instead of out in the open on GitHub...
Example YAML is literally any YAML with a leading comment.  Should take a maintainer 30 seconds to put together

@aw185176 About losing comments when using functions in a unix pipe chain, look at the issue https://github.com/GoogleContainerTools/kpt/issues/2069


Add syncer page to docs

We already have similar docs at: https://casdoor.org/docs/user/manipulation but it's too simple and low-quality.
We want:

See: https://door.casdoor.com/syncers

Fixed by: https://github.com/casdoor/casdoor-website/pull/171


Visualisation with assign query + lineage fit

Needs the dense matrix to draw the tree. Decide whether to:
a) Recalculate each time
b) Save the dense matrix along with the sparse matrix

Or, I think the best choice:
c) When a visualisation is made create the full matrix and save it. Subsequent visualisations can then load this file rather than recalculating

Or
d) Do b) by default, but add an option to turn off if desired


build-fhs-userenv: change to using bubblewrap over chrootenv

Motivation for this change
this makes FHS envs use bubblewrap over Nix's custom chrootenv.c which might fix issues such as #33106 but more generally gives us the ability to add many new configuration options with bwrap's added power. Maybe using bwrap will be more stable as well since it seems widely used.
I have tested this by building Yocto images by running bitbake in an FHSUserEnv but I have yet to run steam which is likely the most complicated package that utilizes this.
This comes with fixes from a lot of experience using FHS env for Yocto and attempting to run it in various environments. Namely:

/host/etc is now /host-etc because systemd symlinks /etc/resolv.conf to ../run/systemd/resolve/resolv.conf so keeping /host-etc on the same level as /etc helps with that.
/etc/pki is now included so fedora users can get SSL within the env
Removing the mount of /host-etc (/host/etc) and directly bind-mounting all etc files and dirs, then allowing directories of etc from the generated rootfs to override the one from the host.

Applications that work:

[x] applications/blockchains/mist
[x] applications/editors/android-studio
[x] applications/science/electronics/bitscope
[ ] applications/misc/houdini
[x] applications/misc/lutris
[x] applications/misc/sidequest
[ ] applications/networking/dropbox
[ ] applications/video/lightworks
[x] development/arduino/platformio
[x] games/steam
[x] servers/plex
[ ] build-support/appimage
[ ] tools/package-management/appimage-run
[ ] tools/package-management/conda

Things done

[x] Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
Built on platform(s)

[x] NixOS
[ ] macOS
[x] other Linux distributions

[x] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[x] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[x] Assured whether relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

The build error is from this line:
https://github.com/NixOS/nixpkgs/pull/55973/files#diff-0e0af17bff7614ce6822d64c6863bb1fR36
but I'm not sure what the fix is, it seems that I can't readDir "${fhs-env}" during build because that path doesn't exist yet? it's weird because this works on my test fhs-env. Do i do it at runtime or can i force this directory to exist?

ofborg only evaluates things and it's not able to evaluate the buildFHSUserEnv without building due to readDir. The easiest fix might be calculate mounts, ro_mounts and blacklist in bwrap_cmd instead.

did some more tests, most things didn't run because my host system's /etc/fonts/fonts.conf could not be read by these packages for some reason. Nothing that seems to be wrong with the FHS env itself.
also updated the docs.

@grahamcofborg build houdini lightworks conda

@hedning what do I need to merge this?

It would be good to get an OK from @yegortimoshenko or @abbradar first.

@yegortimoshenko if I wanted to edit the bwrap options for my own uses, how should I implement this to make that possible? Maybe I should include it as an arg in the derivation and use overrideAttrs? Or maybe I should just make my own custom FHSUserEnv, but I want to re-use as much upstream code as I can.

It's just that I worry that we might need to switch back at some point, if we ever need anything bubblewrap doesn't implement. If we were to expose bubblewrap options, we will never be able to change underlying implementation without breaking reverse compat.

Yeah, that's what I fear too so my current idea is to proceed with caution and 1) don't expose the implementation 2)  look at bubblewrap's code to see if it's hackable enough.
On May 28, 2019 12:13:58 PM GMT+03:00, Yegor Timoshenko<EMAIL_ADDRESS>wrote:

It's just that I worry that we might need to switch back at some point,
if we ever need anything bubblewrap doesn't implement. If we were to
expose bubblewrap options, we will never be able to change underlying
implementation without breaking reverse compat.
--
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub:
https://github.com/NixOS/nixpkgs/pull/55973#issuecomment-496433818

--
Nikolay.

I didn't yet review the PR itself yet, only bubblewrap -- it's quite more complicated but we should definitely try using it; the code is readable too.
One concern of mine is if nested bubblewrap would work okay, especially if it would blow /proc/mounts. This needs to be tested.
Another thing -- we explicitly don't want any kind of isolation that bubblewrap does, including no automount, any other namespaces etc, and we also don't want to expose it as an implementation. I think we could hit both birds by adding buildBwrapUserEnv upon which old buildFHSUserEnv resides. The former would have all the bubblewrap options exposed and the latter would enforce the old behaviour.

Wrt naming: User seems to be redundant, maybe buildBwrapEnv?

Maybe; don't have big preferences here. @illegalprime can choose the naming ~_^

For reference, the relocation error messages from above are claimed to be gone since glibc 2.30, which has a WIP PR.

I'm back!
@yegortimoshenko I updated this branch to not introduce any new options and I also changed /host/etc to /host-etc because /etc/resolv.conf is symlinked to ../run/ in systemd. Fedora also stores its SSL certificates as a symlink to somewhere not included in the current etc package so that should be investigated too.

OK Fedora stores its SSL certs in /etc/pki so I've added that to the environment.

I added a commit which removes /host/etc (or /host-etc) and bind-mounts specific files or directories from the host's /etc to the chroot. It then will look inside the generated root filesystem's /etc and bind mount subdirectories of it. because this is all subdirectories of etc the generated etc and the host etc are "merged" somewhat at runtime.
This helps avoid weird symlink bugs when nesting chroots like in this commit 06f27dc2e932778cf1d2ce6b0bf30b32575dd16a and in general simplifies things (IMO)

:+1: really cool work. Do you need help to finish this PR?

@illegalprime Are you still working on this. Looks very promising

Rebased on top of nixos-unstable here: https://github.com/Atemu/nixpkgs/tree/chrootenv2bubblewrap

Since the compatibility is not 100% yet, perhaps we should limit bubblewrap use to the programs which are known to work first and move over the rest in later PRs.
Making all of these work feels like too large of a scope to me.

We could call it v2 and migrate everything to it and at some point delete the old version.

I did some rebase magic to do just that in a transparent manner https://github.com/Atemu/nixpkgs/tree/buildFHSUserEnvBw
Should I open a new PR?
Once we've ported all packages over, we can simply rename the function to the old name. This name should be temporary.

I did some rebase magic to do just that in a transparent manner Atemu/nixpkgs@buildFHSUserEnvBw
I also ported three of the FHS packages that I know work fine.
Should I open a new PR?
Once we've ported all packages over, we can simply rename the function to the old name. This name should be temporary.

bring it on.

With #94442 merged, should this be closed now?

#94442 seems to subsume this PR, closing.


Fix cumsum for empty partitions

[x] Closes #7224
[x] Tests

@dask/maintenance I am planning on merging this today unless there is any feedback

+1 on the code


Set runtime scope for Dataflow runner dep on runners-core

Be sure to do all of the following to help us incorporate your contribution
quickly and easily:

[x] Make sure the PR title is formatted like:
[BEAM-<Jira issue #>] Description of pull request
[x] Make sure tests pass via mvn clean verify. (Even better, enable
Travis-CI on your fork and ensure the whole test matrix passes).
[x] Replace <Jira issue #> in the title with the actual Jira issue
number, if there is one.
[x] If this contribution is large, please file an Apache
Individual Contributor License Agreement.

R: @davorbonaci or @dhalperi

Already caught a suspicious depenendecy:
runners/google-cloud-dataflow-java/src/main/java/org/apache/beam/sdk/util/GroupAlsoByWindowViaWindowSetDoFn.java:[33,40] package org.apache.beam.runners.core does not exist

Believe that file can be removed now.

I think putting it back in the old namespace is simplest.

Once the fragility of the service to this namespacing is fixed, we can just do the move along with all the rest of runners-core.

False: it is the new name that must remain. I've put it back.

LGTM on the current version (runtime scope + removal of org.apache.beam.sdk.util.GroupAlsoByWindowViaWindowSetDoFn).
Please squash / self-merge, once test pass. Manual streaming test would be good too.


append data functionality

Append data functionality to the table component.
The Sql-Assessment extension performs long-running server assessment and displays results as it gets returned from STS one-by-one during the execution process. This allows a user to see results right away and do not wait until the full assessment is completed.
Existing table component methods allows to completely replace data source only. This leads to scrolling being re-set to the top of the table every time a new data chunk arrives.
A new proposed function appends new data to the existing data source and doesn't force the table to re-set its data completely.

Pull Request Test Coverage Report for Build 334574481

4 of 10 (40.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.01%) to 39.643%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/sql/workbench/api/common/extHostModelView.ts
0
1
0.0%

src/sql/workbench/browser/modelComponents/table.component.ts
0
5
0.0%

Totals

Change from base Build 334519912:
0.01%

Covered Lines:
20632

Relevant Lines:
47807

💛  - Coveralls


added resources

Issue #149
Changes made:
Added 1 programming cometition website Code jam.
Added 2 JavaScript and 1 TypeScript free resource.
Added 1 java resource
Added 1 resource
Resource links:
The odin project -> https://www.theodinproject.com
30 Day Vanilla JS Coding Challenge - https://javascript30.com
JavaTpoint - https://www.javatpoint.com/java-tutorial
Codeinwp -> https://www.codeinwp.com/blog/webpack-tutorial-for-beginners
Resource description:
The odin project -> The Odin Project provides a free open source coding curriculum on web development that can be taken entirely online.
30 Day Vanilla JS Coding Challenge -> It's a 30-day vanilla JS coding challenge. During these 30 days you'll build 30 different apps using fun and easy video tutorials.
JavaTpoint -> It is a Java programming platform. It includes Java programming APIs such as java.lang, etc. It includes core topics like OOPs, String, Regex, Exception, Inner classes, Multithreading, I/O Stream, Networking, AWT, Swing, Reflection, Collection, etc.
Codeinwp -> CodeinWP is a blog for anyone who wants to learn how to build a website all on their own.

Please make sure the table of contents remain ideal with all headings.

@debrajhyper conflict resolved! Have a look now 👀

Thanks for your contributions. ❤


Correlated layers control item for #61

That gives a working interface. It's very rudimentary, and looks pretty awful. But everything works!

Re-opening to modify this to make it some kind of separate floating panel, activated by a button. In current state, it's unworkably ugly.


question about PMU dataset

hello!
I noticed that three databases are compared in the results of this article. Could you please provide the PMU data?Thanks!

Hi, thank you so much for your interest. Unfortunately, we cannot release the private data due to the policy. Sorry for that.


remove sdk calls altogether for now

old sdk is cached, leading to errors when viewing docs site

🚀 Deployed on https://64b5bc6bf6fdda0d2fff562b--cozy-fox-0defec.netlify.app


Fixes #1298

Moves the background from.nav-wrapper to .nav-wrapper .inner where it will always be visible. Removes the unnecessary/repetitive background color from .nav-main .footer.

Thanks for the PR, @alexgleason. From what I can see this causes problems at mobile viewing sizes:

Perhaps the background just needs to be on both .nav-wrapper and .nav-wrapper .inner ?

@davecranwell Finally tested this, you were right on both those selectors needing the BG color. Will make sure to also check mobile in the future before sending a PR.

bump
Happy to merge this, @davecranwell?

:+1:

As far as I can tell this work is also present in #1746, but yes, go ahead.


[✨] Remove Event Listener after event is invoked once

Is your feature request related to a problem?
The problem I am having is not being able to remove an event listener after it is invoked once, either by setting it on the element itself, or by creating an event using the useOn hook.
Describe the solution you'd like
The ability to set any of the event listener options, such as { once: true }. The event listener would automatically be removed after being invoked once.
Describe alternatives you've considered
As an example, I am currently using state to handle a window onScroll event that I only want triggered once based on the scroll position.
Additional context
Another option is using an intersection observer. I understand there is useVisibleTask?(), which can use an observer under the hood, but there is no way to set any of the Intersection Observer Options, or see the properties on the entry itself, such as the intersectionRatio. If the component/element is above the fold on page load, that Javascript gets loaded right away, which I want to avoid.

you can add an attribute to the element and check the value

@theonlychase is it solved?


Game Script: Baldr Sky Dive1 “Lost Memory” (+raw scripts dir)

Added a new game script: Baldr Sky Dive1 “Lost Memory”
Not to be confused with the version on steam; this is from the original first game from 2009, which doesn't include Dive2. (The steam version doesn't have Japanese support for some reason.) I'm not sure how useful this will be since baldr has a functional text-hook, but in any case this is meant more as an experiment with how well this system could work with a VN.
Extracting this game was a bit of a nightmare. I ended up having to modify an older program to spit out text files in order to make this work; upside is it won't take as long to do again for the second game. (Which I won't be doing until I'm close to finishing this VN most likely.)
Added a new directory for raw/unoptimized-for-g2t scripts, and included my recent raws in it. If you still have any that would be a good fit for that, feel free to add them. (possibly even a sub-dir with alternates, like P4 with the names kept in?) Also tweaked the readme some, by linking the main repo of g2t and adding the info for baldr.

Appreciate the effort, but not sure why we would want to spent time extracting text from visual novels that can be hooked, apart from the cross-platform ones like Danganronpa that can't be hooked on MacOS or Linux.
At the same time, I know people who keep the backlog while they play a VN so they can directly get the full dialog text after their playthrough without all the junk.
I could definitely add some of the raw scripts in case anyone wanted to do their own parsing.

Yeah, this was meant as more of a proof-of-concept, rather than something I plan to do frequently. Some VNs can be kinda picky with the texthook, like after scene transitions having to go find a new hook, which I recall happening with me on baldr. (also did it with the combat/gameplay aspect of this one, which got annoying. likely just a bug with that hook/version, but still.) If the script-match works more consistently in these cases, then I would consider this experiment a success.
And yep, that's why I like having access to the raw scripts, sometimes I like to try my hand at parsing them on my own. (can catch edge cases easier like that sometimes)

Added another new game script: Persona 5 (ps3, not royal)
This is the original JP-region PS3 Persona 5 script. Unfortunately I don't have access to P5R, or I would have gone with that version.
This should include everything, but in the event my parsing broke anything, I have included my compiled text file.


required function is not working in "to-words.js" file

ERROR : - Error: Unknown Locale "en-IN"
Referenced link : https://github.com/facebook/react-native/issues/6391

Hi @Rahul2pancholi, I have moved to static require using switch case for importing locale.
It's available in branch named static-require.
Please help test it if possible.

I have also used the same switch case as a workaround
Any reason why haven't you have pushed it in the master branch..?

No apparent reason. I preferred the dynamic import way, however, it seems that may be problem with React.
I will merge it in main branch and do a release today.


fixImprove analyze timeout. (#33836)

backport https://github.com/apache/doris/pull/33836

Further comments
If this is a relatively large or complex change, kick off the discussion at<EMAIL_ADDRESS>by explaining why you chose the solution you did and what alternatives you considered, etc...

run buildall


Move noUpdateLSD in method options to ThreadsAPI class constructor

@Digital39999: Also noUpdateLSD is in method options, not in class constructor.

Oops, my bad. Should've fixed this.
Originally posted by @junhoyeo in https://github.com/junhoyeo/threads-api/issues/26#issuecomment-1626079968

Done in #35.

@Digital39999 That was fast! Thanks :)

@junhoyeo Np, glad to help!


Flake8

Partly addresses #12
Note: I can't run the codegen-script since I don't have a binary yet. I tried adjusting it to produce noqa comments but I can't verify if it works properly. It probably fails to do so because it is used in conjunction with black, so we can't tell on which lines the violations will end up
Either way, this PR:

Configures flake8 to be compatible with black (according to their docs)
Fixes flake8 violations (a few)
Adds specific error ignores to lines I couldn't/didn't know how to fix (a lot)
Runs flake8 in CI
Also runs black on python files in the root of the repo on CI
Removed existing # noqa that didn't trigger violations

I'm unsure of the following changes:
from py2spirv import python2spirv, i32, vec2, vec3, vec4

Is that how it's supposed to be done?

I can't run the codegen-script since I don't have a binary yet.

It does not load the binary :)  Also, the files it generates are blackified before writing, so I'm curious what noqa's were needed.
Will have a proper look at this after xmas :)

It does not load the binary :)

Really! Doh. I should have just tried it.

Also, the files it generates are blackified before writing, so I'm curious what noqa's were needed.

Black doesn't fix comments that exceed max line length, for example. Also, the undefined type hints.

Black doesn't fix comments that exceed max line length, for example. Also, the undefined type hints.

Maybe we should simply disable line length checking on flake8.

I would be ok with disabling that check!


can't set the expiration time for set / list / Zset / hash yet?

Now, can't set the expiration time for set / list / Zset / hash yet?

Yes, we didn`t add this command right now, maybe in the near future.
If you want to use rosedb in production, I can take the time to add it, or you are just interested in how to implement it, you can contribute to rosedb, see https://github.com/flower-corp/rosedb/issues/103

i want to use rosedb in production.

Which data structure is most urgently needed, or all of them?


GAE Calendar (Java, Spring MVC, JPA, jQuery, Datastore vs Cloud SQL)

Hello!
I'd like to submit a calendar app written in Java and demonstrating the usage of GAE. The technology stack is: Java, Spring MVC, JPA, jQuery and both Google Cloud Datastore and Google Cloud SQL.
Code for the project is here:

Cloud SQL version: https://github.com/invaa/CloudCalendar/tree/spring-cloud-sql
Datastore version: https://github.com/invaa/CloudCalendar/tree/spring-data-v2

Demo is running here: http://cloudcalendarservice.appspot.com/
I'm looking forward to your review. Thanks in advance.

Good job, I do however have a question. Is it possible to create a custom repository 9Spring Data) for DataStore whitch can be used within an application outside Gae?
Thjanks in advance

Hello, schabiyo! Thanks for the question.
Using DataNucleus library (just as in the provided sources) you can create custom repositories as well.
Here is the info on custom repos:
http://docs.spring.io/spring-data/data-commons/docs/1.6.1.RELEASE/reference/html/repositories.html
Although, some JPA limitations do exist, they are not related to Spring Data custom repositories. Here is more info on JPA limitations:
https://cloud.google.com/appengine/docs/java/datastore/jpa/overview-dn2

Hello invaa,
Thanks for the quickl reply. I will give it a try. One last question, to witch JPA limitation were you refering to? Is it something than can prevent me from reaching my target?
Thanks again,

No it shouldn't. But I mentioned it just to let you know that DataNuleus JPA implementation for Google Cloud Datastore has some limitations and it's important to keep it in mind.


improvement request for dockerfile

when I try to install Dockerfile.vim in Docker environment. need to perform the following instructions:
WORKDIR /tmp
RUN git clone https://github.com/ekalinin/Dockerfile.vim.git && \
         cd Dockerfile.vim && make install && cd .. &&  rm -rf Dockerfile.vim

which required make package, while make package is not necessary for docker image. May I suggest the following improvement ?
# Install Dockerfile.vim
RUN git clone https://github.com/ekalinin/Dockerfile.vim.git  ~/.vim/pack/plugins/start/dockerfile && \
          vim +"helptags ~/.vim/pack/plugins/start/dockerfile/doc" +qall

by using vim-plug, it's not a issue anymore


Using input with ng-if

Hi Carlos!
I'm trying to use this directive in an input that also uses ng-if. (And I want to use ng-if because I want to reuse the input as a directive itself.)
I've spent a lot of time trying to figure out why my ng-model wasn't binding, but after a lot of trial and error, it looks like ngMask thinks that no ng-model or mask is available, because I'm using ng-if...and the element isn't on the DOM yet. Tested this by using ng-show/ng-hide and ngMask works amazingly, but was wondering if you can help me figure out how to use ngMask on ng-if elements?
Thanks a lot!

You should use controller as syntax to avoid this sort of issue. I guess there is no dot in your ng-model

@ericodes Wow, a year later I face the exact same problem. Do we have to use ng-show? In my case ng-if is better.
@maxisam I don't think that's the case. In my ng-model there is a dot notation (i.e. ng-model="sendToPostcodeObj.zipCodeFrom")
I also tried creating the scope object in the controller first, so that it won't be undefined. Didn't work. Any ideas?

Hey @tahaerden, can't speak to what's happened since, but I looked at my code from last summer and it looks like I went with ng-show. Sorry I can't be of more help!

@ericodes No problem! Actually I think I just fixed it by adding restrict="reject" and it worked in the ng-if :)


Removed useAccountBalance

Description

Removed useAccountBalance hook.
Updated Balance.tsx to replace useAccountBalance by grabbing price from global state. Grabs the balance using useBalance passing in only address property and setting watch to true. Additionally, assigns a variable to the formatted balance and converts the result to a Number, then uses the variable during rendering.
Updated FaucetButton.tsx to replace useAccountBalance by grabbing the balance using useBalance passing in the address obtained from useAccount.
Updated ~~/hooks/scaffold-eth/index.ts to remove reference to useAccountBalance.

Additional Information

[x] I have read the contributing docs (if this is your first contribution)
[x] This is not a duplicate of any existing pull request

Related Issues
Closes #687
Your ENS/address: JacobHomanics.eth/0xc689c800a7121b186208ea3b182fAb2671B337E7

Not sure if we need to force passing in the chainId via targetNetwork.id when calling useBalance. I've omitted it for now.

Thanks @Hotmanics !! Looking great !!


issue with loading Fluidsynth on Woof using DOOM Mod Loader v2.5

OS: Windows
DML Version: v2.5 (Stable)
Sourceport: Woof!

From user report:

I came across an error where I cannot use the Fluidsynth sound backend for playing midi when using the loader. And yes I can boot it just fine outside the program when using Woof as a standalone

WORKAROUND:

Open woof! Doesn't matter if through DML or directly.
Change from "native" to "Fluidsinth".  I will tell you to close it on order to apply changes. Close it.

Copy the "soundfonts" folder in the DML 2.5 folder. Now you should have 2 soundfonts folder, the original in the woof folder and one next to the DML 2.5 exe.
Open woof through DML v2.5. It will now start without any error and using fluidsinth!

Woof folder:

DML 2.5 folder:

You may have to repeat this process with any new woof! release, depending if anything inside that folder changes between releases.

Updated issue with workaround. Please note that woof! It's not an officially support sourceport for v2.5


Install script unification

In our project, there is target/ directory and install-deps script for each target.
(install-deps-tegraTX1.sh, install-deps-raspberry-pi2_3.sh)
In many open source projects, they use one script file and use argument to support several target platforms.
Therefore, i propose make the install-deps script one using argument with the target name.
Our install.sh script is already done with the argument of target board name.
(./install.sh --target=raspberry_pi2_3)
Which do you think it's better? unify the install-deps.sh script or leave it as it is now?

Actually, install-deps-tegraTX1.sh and ...-raspberry-pi2_3.sh is very similar.
If you unify install-deps-*.sh, where is this file located?

Unifying install-deps-*.sh files is a good idea. As @LeeHayun said, they behavior in very similar manner.

I expect that the install-deps.sh should be located in tools/ dir.
Or it's also okay in script/ dir.

@sinban04
For the integrity, moving all the scripts in scripts directory (including install.sh) to tools would be good.
Can you integrate tools/install-deps-rpi.sh for Travis CI docker with target/raspberry-pi2_3/install-deps-raspberry-pi2_3.sh for real device?

I agree all the script files should be located tools/ dir.
The integration of two install-deps script (raspberry-pi2_3 and tegraTX1) should be done first,
and travis can use that script as it is.

I unified install script in PR #127 to use cmake and kconfig.
Therefore I close this issue.


feat(CL): add swap logic

Closes: #XXX
What is the purpose of the change

Add a description of the overall background and high level changes that this PR introduces

(E.g.: This pull request improves documation of area A by adding ....
Brief Changelog
(for example:)

The metadata is stored in the blob store on job creation time as a persistent artifact
Deployments RPC transmits only the blob storage reference
Daemons retrieve the RPC data from the blob cache

Testing and Verifying
(Please pick one of the following options)
This change is a trivial rework / code cleanup without any test coverage.
(or)
This change is already covered by existing tests, such as (please describe tests).
(or)
This change added tests and can be verified as follows:
(example:)

Added unit test that validates ...
Added integration tests for end-to-end deployment with ...
Extended integration test for ...
Manually verified the change by ...

Documentation and Release Note

Does this pull request introduce a new feature or user-facing behavior changes? (yes / no)
Is a relevant changelog entry added to the Unreleased section in CHANGELOG.md? (yes / no)
How is the feature or change documented? (not applicable   /   specification (x/<module>/spec/)  /  Osmosis docs repo   /   not documented)

Is this PR needed still given that we also have https://github.com/osmosis-labs/osmosis/pull/3497?

No, I was referencing a couple of things but should be good now, closing


Cluster view does not show the correct status after a Failover

In case a HANA failover occurs, the former secondary becomes the new master and the nodes switches roles. The new Cluster view is not presenting the correct information after such event:
The HANA secondary sync state presents an empty green box (while the SR is not enabled):

Also, both nodes presents itself as primary (when only HANA 2 is the real primary)

The Virtual IP also gets moved to the secondary node, so the vIP information should be shown correctly:

Closing, split in #153 and #154


Artifacts in firework example

Crate versions
bevy version: 0.14.2
bevy_hanabi version: main (1ad6298). It works fine on 0.12.2.
AdapterInfo { name: "AMD Radeon RX 570 Series (RADV POLARIS10)", vendor: 4098, device: 26591, device_type: DiscreteGpu, driver: "radv", driver_info: "Mesa 24.2.2-arch1.1", backend: Vulkan }
Describe the bug
The firework example has artifacts. It's drawing ribbons between seemingly random particles.
Expected behavior
No artifacts
To Reproduce
cargo r --example firework --no-default-features --features="bevy/bevy_winit bevy/bevy_pbr 3d"
Screenshots
https://github.com/user-attachments/assets/56f9e75e-075e-40a5-b995-01ebc521e89b

Thanks @Seldom-SE I'll keep an eye out, but unfortunately I've been using that example to work on a new feature and I've not seen this locally, so it's going to be hard to diagnose until I can repro.

I can confirm that I am also seeing this bug with my own modified version of the fireworks example (see attached files in zip).
This is with bevy 0.14.0 and bevy_hanabi 0.12.2.
Moreover, I'm seeing the following errors being output in the console:
2024-09-22T14:39:54.533651Z ERROR bevy_hanabi::render: Allocator didn't allocate sequential indices (expected 3, got 5). Expect trouble!
2024-09-22T14:39:54.533817Z ERROR bevy_hanabi::render: Allocator didn't allocate sequential indices (expected 3, got 5). Expect trouble!
2024-09-22T14:39:55.068075Z ERROR bevy_hanabi::render: Allocator didn't allocate sequential indices (expected 4, got 7). Expect trouble!
2024-09-22T14:39:55.068198Z ERROR bevy_hanabi::render: Allocator didn't allocate sequential indices (expected 4, got 7). Expect trouble!
2024-09-22T14:39:55.835265Z ERROR bevy_hanabi::render: Allocator didn't allocate sequential indices (expected 5, got 10). Expect trouble!
<and many more of these!>
Search for "BUG???" in the firework.rs file to see what (I think) might be the responsible line of code in my example (it's during despawning)
Also, I've noticed that the first firework doesn't visibly burst, so you'll find a weird hack in my code (search for "Weird hack") where I force a firework burst way offscreen.  No idea if this is related, but I've thrown it in here in case it helps track down the issue.
Disclaimer: I'm pretty new to Rust / Bevy / Bevy_hanabi, and this code is just my way of messing around to understand how things work, so apologies if it's horrible to read.
hanabi_test.zip

Re:

Also, I've noticed that the first firework doesn't visibly burst

I also noticed on my side that the first burst of my effect is different from this that follow.

The first burst issue is due to Bevy's first frame having a delta time of zero I believe.

I can repro trivially now, not sure why. I'll investigate.

FYI - @pcwalton found the likely cause and has a fix getting ready.


DOCS-Fix-TOC-entry

Align left nav title with card title.
Product Version(s):
CE 3.17, 3.18, next
Issue:

Link to docs preview:

SME review:

[ ] An SME has approved this change.

DOCS review:

[ ] A member of the docs team has approved this change.

Additional information:

Merge checklist:

[ ] Deploy preview inspected wherever changes were made
[ ] Build completed successfully
[ ] Test have passed

@pasanw I noticed that the card says, "Multi-cluster management and federation" but the left nav did not. I'm not sure when "federation" was added. I did not make this update for CC because mcm docs were taken out and we'll need to


Add contact section

Contact section

Contact section

hello @arpitghura can you please assign this issue to me  I want to work on it under kwocc2022

Hey @Riya14-dangra I think, there is no need of contact form on this website.

Hey @Riya14-dangra I think, there is no need of contact form on this website.

Then how we getting the blogs for testimonials?


Introduce Storybook

Overview
We can see components in developing on browser using Storybook.
So I thought Storybook would help newcomers with #452 and added it.
If you agree with my opinion, please consider to introduce Storybook.
Changes

Add Storybook settings corresponding to tsx
Use :6001 for Storybook
Start with docker-compose -f docker-compose.development.yml exec crowi npm run storybook

Screenshot
This is client/components/Common/Modal on Storybook.
I feel it doesn't has its own styles. Is it displayed correctly?

@saitoeku3 Thank you for your great contribution.
That's awesome!
Can you rebase latest master first?


Add memory limiter extension

I don't know if we need to remove the ballast  extension also (this PR removes it), is deprecated and cannot be used with this extension
https://github.com/open-telemetry/opentelemetry-collector/blob/main/extension/memorylimiterextension/README.md

Warning
The memory_limiter extension cannot be used if the deprecated memory_ballast extension is enabled.

is deprecated and cannot be used with this extension
https://github.com/open-telemetry/opentelemetry-collector/blob/main/extension/memorylimiterextension/README.md

Warning
The memory_limiter extension cannot be used if the deprecated memory_ballast extension is enabled.

Both cannot be used in the same OTEL Collector but you should be able to add them to the same build.
I think we should keep both during some releases to not break customers using the ballast extension.

is deprecated and cannot be used with this extension
https://github.com/open-telemetry/opentelemetry-collector/blob/main/extension/memorylimiterextension/README.md

Warning
The memory_limiter extension cannot be used if the deprecated memory_ballast extension is enabled.

Both cannot be used in the same OTEL Collector but you should be able to add them to the same build.
I think we should keep both during some releases to not break customers using the ballast extension.

Make sense. I revert that change.


x/website: Date/Time in Playground set to fixed value

What version of Go are you using (go version)?
n/a
Does this issue reproduce with the latest release?
yes
What operating system and processor architecture are you using (go env)?
n/a
What did you do?
https://play.golang.org/p/rlm4zFwKDjE
(Output current time in Playground)
What did you expect to see?
A date in 2020.
What did you see instead?
2009-11-10 23:00:00 +0000 UTC m=+0.000000001

@doomguy yes, time is fixed in the playground to permit caching of build results. see the about button in the top right hand corner.


Revert backend paths to opendistro

Signed-off by: David Cui<EMAIL_ADDRESS>Description
Revert backend paths from using opensearch to opendistro to keep cluster permissions same with ODFE. For example:
cluster:admin/opensearch/reports/definition/create vs cluster:admin/opendistro/reports/definition/create
Issues Resolved
[List any issues this PR will resolve]
Check List

[ ] New functionality includes testing.

[x] All tests pass, including unit test, integration test and doctest

[ ] New functionality has been documented.

[ ] New functionality has javadoc added
[ ] New functionality has user manual doc added

[x] Commits are signed per the DCO using --signoff

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Codecov Report

Merging #218 (304f8de) into main (860e89c) will increase coverage by 0.14%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##               main     #218      +/-   ##
============================================
+ Coverage     63.57%   63.71%   +0.14%
  Complexity      296      296
============================================
  Files            99       99
  Lines          4115     4115
  Branches        628      628
============================================
+ Hits           2616     2622       +6
+ Misses         1357     1351       -6
  Partials        142      142

Flag
Coverage Δ

dashboards-reports
75.16% <ø> (+0.30%)
:arrow_up:

reports-scheduler
53.01% <ø> (ø)

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

Impacted Files
Coverage Δ

...tsscheduler/action/CreateReportDefinitionAction.kt
66.66% <ø> (ø)

...sscheduler/action/GetAllReportDefinitionsAction.kt
66.66% <ø> (ø)

...rtsscheduler/action/GetAllReportInstancesAction.kt
66.66% <ø> (ø)

...portsscheduler/action/GetReportDefinitionAction.kt
66.66% <ø> (ø)

...reportsscheduler/action/GetReportInstanceAction.kt
66.66% <ø> (ø)

...rtsscheduler/action/InContextReportCreateAction.kt
66.66% <ø> (ø)

...ortsscheduler/action/OnDemandReportCreateAction.kt
66.66% <ø> (ø)

...tsscheduler/action/UpdateReportDefinitionAction.kt
66.66% <ø> (ø)

...heduler/action/UpdateReportInstanceStatusAction.kt
66.66% <ø> (ø)

dashboards-reports/public/components/main/main.tsx
63.72% <0.00%> (+5.88%)
:arrow_up:

Continue to review full report at Codecov.

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


Enable blocking all downloads via managed policy

In a nutshell, this commit allows users to set the DownloadRestrictions managed policy key to either 0 or 3 to respectively permit or block all downloads.
The key values are as follows.....
0 = No special restrictions. Default.
1 = Block malicious downloads and dangerous file types.
2 = Block malicious downloads, uncommon or unwanted downloads and dangerous file types.
3 = Block all downloads.
4 = Block malicious downloads. Recommended.
In this commit, DownloadRestrictions values of 1, 2 and 4 are effectively neutered by disentangling them from safebrowsing and treating them as invalid. None of the reinstated code appears to be connected to safebrowsing.
The relevant section of chrome/browser/download/chrome_download_manager_delegate.cc will end up looking like this.....
bool ChromeDownloadManagerDelegate::ShouldBlockFile(
    download::DownloadItem* item,
    download::DownloadDangerType danger_type) const {
  DownloadPrefs::DownloadRestriction download_restriction =
      download_prefs_->download_restriction();

  switch (download_restriction) {
    case (DownloadPrefs::DownloadRestriction::NONE):
      return false;

    case (DownloadPrefs::DownloadRestriction::ALL_FILES):
      return true;

    // DownloadRestrictions policy key values 1, 2 and 4 treated as invalid
    case (DownloadPrefs::DownloadRestriction::POTENTIALLY_DANGEROUS_FILES):
    case (DownloadPrefs::DownloadRestriction::DANGEROUS_FILES):
    case (DownloadPrefs::DownloadRestriction::MALICIOUS_FILES):
    default:
      LOG(ERROR) << "Invalid download restriction value: "
                 << static_cast<int>(download_restriction);
  }

  return false;
}

I have tested DownloadRestrictions values of 0-5, with 0 and 3 behaving as expected. Running without a policy file also behaves as expected, same as 0 (ie there is no intrinsic change in behaviour from this commit).
Values 1, 2 and 4 behave the same as 5, emitting a non-fatal "Invalid download restriction value" error and allowing the download to proceed.
This PR will fix issue #2974.

So it is either allow downloads (by default), or block all of them?

So it is either allow downloads (by default), or block all of them?

That's correct.


Nuxt 3 Support

Plans to create Nuxt module?

I was reading through the migration guide and it seems it may work with Nuxt 3 already?


[py2py3] pre-production test of ReqMgr2 with the py3 environment

Impact of the new feature
This should involve mainly ReqMgr2 and its dependencies.
Is your feature request related to a problem? Please describe.
After making sure that the whole WMCore passes unit tests in py3, we should run some pre-productions tests to check if ReqMgr2 is compatible with py3 in a production-like environment.
Describe the solution you'd like
We should deploy ReqMgr2 to testbed, run some workflows and see how it behaves. (please feel free to correct me or expand this section of the issue with more details!)
Describe alternatives you've considered
There is not much else that we can do.
Additional context
I would tackle this after we have completed

[ ] #10422

Python3 ReqMgr2 seems to be fully functional starting with WMCore 1.5.0.pre6, which is currently available in cmsweb-testbed and will be deployed to production tomorrow.
Closing this ticket.


[SPARK-35606][PYTHON][INFRA] List Python 3.9 installed libraries in build_and_test workflow

What changes were proposed in this pull request?

In the build_and_test workflow, tests are run against both Python 3.6 and Python 3.9. However, only libraries installed in Python 3.6 are listed. We should list Python 3.9's installed libraries as well.
Why are the changes needed?

Listing Python 3.9's installed libraries is helpful for debugging.
Does this PR introduce any user-facing change?

No.
How was this patch tested?

Manual check.

cc @dongjoon-hyun FYI. The purpose of this PR is to list up the installed depenedncies of Python 3.9 in CI builds.

Test build #139188 has started for PR 32737 at commit 703d903.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43708/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43708/

Yep. Thank you for pinging me, @HyukjinKwon .

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43708/

Test build #139194 has started for PR 32737 at commit c77a184.

Test build #139194 has finished for PR 32737 at commit c77a184.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139194/

Test build #139188 has finished for PR 32737 at commit 703d903.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43716/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139188/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43716/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43716/

BTW, @xinrong-databricks and @HyukjinKwon .
Although I'm adding pip explicitly, as a side note, we don't need to check this inside GitHub Action. It's a docker image.
$ docker run -it --rm dongjoon/apache-spark-github-action-image:20210530 python3.8 -c 'import numpy'
$ docker run -it --rm dongjoon/apache-spark-github-action-image:20210530 python3.8 -c 'import plotly'
Traceback (most recent call last):
  File "<string>", line 1, in <module>
ModuleNotFoundError: No module named 'plotly'

The other packages are installed correctly. Only pip seems to be screwed for some reason during pypy3 pip installation.

The image is ready. I'll make a PR to update it and ping you.
$ docker run -it --rm dongjoon/apache-spark-github-action-image:20210602 python3.8 -m pip list
Package             Version
------------------- --------------------
certifi             2019.11.28
chardet             3.0.4
dbus-python         1.2.16
idna                2.8
numpy               1.20.3
pandas              1.2.4
pip                 21.1.2
plotly              4.14.3
pyarrow             2.0.0
PyGObject           3.36.0
python-apt          2.0.0+ubuntu<IP_ADDRESS>
python-dateutil     2.8.1
pytz                2021.1
requests            2.22.0
requests-unixsocket 0.2.0
retrying            1.3.3
scipy               1.6.3
setuptools          45.2.0
six                 1.14.0
urllib3             1.25.8
wheel               0.34.2
xmlrunner           1.7.7

Here is the PR, @xinrong-databricks and @HyukjinKwon

https://github.com/apache/spark/pull/32755

Test build #139264 has started for PR 32737 at commit 5ba0b32.

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43788/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43788/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43788/

Test build #139264 has finished for PR 32737 at commit 5ba0b32.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139264/

Test build #139296 has started for PR 32737 at commit d77b6be.

Test build #139296 has finished for PR 32737 at commit d77b6be.

This patch fails to build.
This patch merges cleanly.
This patch adds no public classes.

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/139296/

Kubernetes integration test starting
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43819/

Kubernetes integration test status success
URL: https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder-K8s/43819/

Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder-K8s/43819/

I verified the GitHub Action result on PR manually.

Merged to master. Thank you, @xinrong-databricks and @HyukjinKwon .

Thanks @dongjoon-hyun !


Delete cookie seems to not work in routes custom folder

Hello,
I have created a mini project to show you my problem: https://github.com/gmunumel/fresh-cookie-issue
I have a delete cookie feature that is working if I run it from routes/ but doesn't work from routes/user:

delete.tsx:
import { deleteCookie } from "$std/http/cookie.ts";

export function handler(req: Request): Response {
  console.log(req.url);
  console.log(new URL(req.url).origin);
  const headers = new Headers({
    location: new URL(req.url).origin,
  });
  deleteCookie(headers, "test_cookie");
  return new Response(null, {
    status: 302,
    headers,
  });
}

index.tsx
/** @jsx h */
import { h } from "preact";
import { tw } from "@twind";

export default function Home() {
  return (
    <div class={tw`p-4 mx-auto max-w-screen-md`}>
      <a class={tw`my-6`} href="/create">
        Create Cookie

      <a class={tw`my-6`} href="/user/delete">
        Delete Cookie (not work)

      <a class={tw`my-6`} href="/delete">
        Delete Cookie (ok)

  );
}

That worked. Thanks


Authentication CLI refactoring

Refactoring of the authentication CLI from pixi to make it usable across all the sub-projects (e.g., rattler-build) .

Closed by #537


[Bug report] 图片瀑布，在iOS safari上 快速滑动并加载了很多图片后，浏览器会出现 "此网页出现问题，已重新载入"

Version
9.0.7
Platforms
iOS, Web
Device Model
iOS 17.7
flutter info
[✓] Flutter (Channel stable, 3.24.5, on macOS 14.1.1 23B81 darwin-arm64, locale zh-Hans-JP)
    • Flutter version 3.24.5 on channel stable at /Users/xiao/fvm/versions/stable
    • Upstream repository https://github.com/flutter/flutter.git
    • Framework revision dec2ee5c1f (4 weeks ago), 2024-11-13 11:13:06 -0800
    • Engine revision a18df97ca5
    • Dart version 3.5.4
    • DevTools version 2.37.3

[✓] Android toolchain - develop for Android devices (Android SDK version 34.0.0)
    • Android SDK at /Users/xiao/Library/Android/sdk
    • Platform android-34, build-tools 34.0.0
    • Java binary at: /Applications/Android Studio.app/Contents/jbr/Contents/Home/bin/java
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)
    • All Android licenses accepted.

[✓] Xcode - develop for iOS and macOS (Xcode 15.0.1)
    • Xcode at /Applications/Xcode.app/Contents/Developer
    • Build 15A507
    • CocoaPods version 1.15.2

[✓] Chrome - develop for the web
    • Chrome at /Applications/Google Chrome.app/Contents/MacOS/Google Chrome

[✓] Android Studio (version 2022.3)
    • Android Studio at /Applications/Android Studio.app/Contents
    • Flutter plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/9212-flutter
    • Dart plugin can be installed from:
      🔨 https://plugins.jetbrains.com/plugin/6351-dart
    • Java version OpenJDK Runtime Environment (build 17.0.6+0-17.0.6b829.9-10027231)

[✓] VS Code (version 1.92.0)
    • VS Code at /Applications/Visual Studio Code.app/Contents
    • Flutter extension version 3.102.0

[✓] Connected device (6 available)
    • iPhonemm (mobile)               • 00008020-0001345A3A79002E            • ios            • iOS 17.7 21H16
    • iPhone 15 Pro (mobile)          • E8D40FB9-CFDD-42FD-B884-A8C6D83F737D • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-17-0 (simulator)
    • iPhone 15 Pro Max (mobile)      • 69D82AB1-9610-4E2D-8437-DFA036E86962 • ios            •
      com.apple.CoreSimulator.SimRuntime.iOS-17-0 (simulator)
    • macOS (desktop)                 • macos                                • darwin-arm64   • macOS 14.1.1 23B81 darwin-arm64
    • Mac Designed for iPad (desktop) • mac-designed-for-ipad                • darwin         • macOS 14.1.1 23B81 darwin-arm64
    • Chrome (web)                    • chrome                               • web-javascript • Google Chrome 131.0.6778.109

[✓] Network resources
    • All expected network resources are available.

• No issues found!

How to reproduce?
快速滑动，加载更多的数据；
https://fluttercandies.github.io/extended_image/
WaterfallFlowDemo, 一样可以复现
Logs
No response
Example code (optional)
No response
Contact
No response

真机上才会出现

你可以自己debug 跑下最新的代码，然后看看哪里报错吗？


Get version of actions/checkout in consistency check job form used-actions.yaml

Needed so that bumping the version e.g. by some dependency updating bot in used-actions.yaml results in changing version of actions/checkout in all parts of the workflows.
Impl plan: when generating the bindings, store a simple text file in generated dir with ref name e.g. v4, which will later be read by the DSL library.

Closing in favor of server-side action bindings modeled as Maven artifacts (#1318).