[BUG] The action fails with a `cp: -r not specified; omitting directory

Is there an existing issue for this?

[X] I have searched the existing issues

Does this issue exist in the latest version?

[X] I'm using the latest release

Describe the bug?
The action fails with a cp: -r not specified; omitting directory '/home/runner/work/zone/zone/'
To Reproduce
workflow yaml
      # Puppeteer test
      - name: Checkout
        uses: ./.github/actions/puppeteer

      - name: Test action
        uses: ./
        id: puppeteer-test
        with:
          files: client/src/api/session-pdf-generator.ts # just replace with your test.js

      - name: Copy test output
        uses: tj-actions/docker-cp@v2
        with:
          container: ${{ steps.puppeteer-test.outputs.container-id }}
          source: ${{ steps.puppeteer-test.outputs.home }}/test.pdf
          destination: test.pdf

      - name: Display contents of test.pdf
        run: cat test.pdf

The composite action is not in root. Instead it lives in .github/actions/puppeteer/action.yml
action.yml
name: puppeteer-chrome
description: Run Test through Puppeteer, the headless chrome Node API
author: tj-actions
inputs:
  files:
    description: 'Files to run'
    required: true

outputs:
  container-id:
    description: 'Container ID'
    value: ${{ steps.puppeteer-run.outputs.container-id }}
  home:
    description: 'Home directory'
    value: /home/pptruser

runs:
  using: 'composite'
  steps:
    - name: Run puppeteer
      id: puppeteer-run
      uses: tj-actions/docker-run@v2
      with:
        image: ghcr.io/puppeteer/puppeteer:19.4.1
        name: puppeteer-chrome
        options: '-i --init --cap-add=SYS_ADMIN -v ${{ github.workspace }}:${{ github.workspace }}'
        args: |
          bash -c 'while read -r file; do echo "Running $file..." && mkdir -p $(dirname /home/pptruser/$file) && cp ${{ github.workspace }}/$file /home/pptruser/$file && node $file && echo "Completed executing $file"; done <<< "$(echo "${{ inputs.files }}" | xargs -n1)"'
branding:
  icon: camera-off
  color: white

What OS are you seeing the problem on?
ubuntu-latest or ubuntu-20.04
Expected behavior?
No error is thrown
Relevant log output
Prepare all required actions
Getting action download info
Download action repository 'tj-actions/docker-run@v[2](https://github.com/xxx/xxx/actions/runs/4182473339/jobs/7245604564#step:12:2)' (SHA:957ff81e54cb1f1[3](https://github.com/xxx/xxx/actions/runs/4182473339/jobs/7245604564#step:12:3)3ad1[4](https://github.com/xxx/xxx/actions/runs/4182473339/jobs/7245604564#step:12:4)4523bfa65cd14ae67d6)
Run ./.github/actions/puppeteer
Run tj-actions/docker-run@v2
Run docker run -v "/var/run/docker.sock":"/var/run/docker.sock" --name puppeteer-chrome -i --init --cap-add=SYS_ADMIN -v /home/runner/work/zone/zone:/home/runner/work/zone/zone ghcr.io/puppeteer/puppeteer:1[9](https://github.com/xxx/zone/actions/runs/4182473339/jobs/7245604564#step:12:10).4.1 bash -c 'while read -r file; do echo "Running $file..." && mkdir -p $(dirname /home/pptruser/$file) && cp /home/runner/work/zone/zone/$file /home/pptruser/$file && node $file && echo "Completed executing $file"; done <<< "$(echo "" | xargs -n1)"'
  docker run -v "/var/run/docker.sock":"/var/run/docker.sock" --name puppeteer-chrome -i --init --cap-add=SYS_ADMIN -v /home/runner/work/zone/zone:/home/runner/work/zone/zone ghcr.io/puppeteer/puppeteer:[19](https://github.com/xxx/xxx/actions/runs/4182473339/jobs/7245604564#step:12:21).4.1 bash -c 'while read -r file; do echo "Running $file..."

### Anything else?

_No response_

### Code of Conduct

- [X] I agree to follow this project's Code of Conduct

Hi @benblub i believe the error is due to using the same code in the test.yml file

      - name: Checkout
        uses: ./.github/actions/puppeteer

      - name: Test action
        uses: ./
        id: puppeteer-test
        with:
          files: client/src/api/session-pdf-generator.ts # just replace with your test.js

should replaced with

      - name: Checkout
        uses: actions/checkout@v3

      - name: Test action
        uses: tj-actions/puppeteer@v4
        id: puppeteer-test
        with:
          files: client/src/api/session-pdf-generator.ts

Also note that except your test is generating a test.pdf file there’s no need to have that code.


Proposal to Introduce Ray SQL into DataFusion Python

Is your feature request related to a problem or challenge? Please describe what you are trying to do.
In light of the complexities associated with maintaining and debugging Ballista, the community would like to propose exploring the adoption of Ray SQL within the DataFusion Python project.
Ray SQL, with only 1.7k lines of Rust, is significantly simpler compared to Ballista’s 27k lines. Despite its smaller codebase, Ray SQL has demonstrated the ability to run all TPC-H queries, showcasing its robustness and simplicity. This reduction in complexity could ease maintenance, lower the learning curve for contributors, and provide a functional distributed SQL solution that is much simpler than Ballista because Ray provides so much – there is no need for us to build scheduler and executor processes – we can simply execute Python tasks in the Ray cluster.
If there is enough interest and support, we could start working on bringing the Ray SQL code into the DataFusion Python project.
Describe the solution you'd like

Following options are suggested by @andygrove , all feedbacks are welcome!

Bringing the Ray SQL prototype into the DataFusion Python project
Describe alternatives you've considered
Building a new version inspired by the Ray SQL code
Additional context
We might need to go through the Apache IP clearance process for importing the external Ray SQL codebase in datafusion-contrib which is not part of Apache.

Ray SQL project: https://github.com/datafusion-contrib/ray-sql

@franklsf95 fyi

Things I'm thinking of that should be showcased or discussed:

Showcase complexity/simplicity

Setup and testing of ballista on tpc-h
Setup and testing of raysql on tpc-h

Benchmark differences ballista/raysql
What do we gain/lose by moving towards adopting raysql
What does raysql provide that Ballista currently does, what would need to be added/maintained by the integration

Is the complexity of Ballista coming from cluster management or query planning?
If Ray SQL can achieve similar or better performance than Ballista with only 1.7k loc by delegating cluster management to Ray and only keeping the distributed query planning responsibility, that's pretty amazing !

Is the complexity of Ballista coming from cluster management or query planning?

Ballista and Ray SQL use the same query planning code. The complexity comes from cluster management/communication.

Setup and testing of ballista on tpc-h

Since upgrading to a more recent DataFusion version, Ballista is no longer capable of running the TPC-H suite. It works fine in the 0.12.0 release.

I am curious about the right boundary here. If we go through with this, should the code become a part of datafusion-python, or should the former stay as a single node thing focusing on excellent bindings to core, and we should have a "datafusion-ray" that uses datafusion-python as a library?

Thanks @ozankabak I'm curious on the boundary, too.
One of the goals in Ray SQL is "Drive requirements for DataFusion's Python bindings". So my guess is that if we wanna use datafusion-python as a library for separate datafusion-ray, we might need to ensure APIs of datafusion-ray is a subset of datafusion-python APIs. Otherwise it'll not be able to easily expose complete bindings to python side via PyO3.

Indeed. Do you have an idea of how the subset would look like?

To expand on this, there are several options for adopting the Ray SQL code as part of the DataFusion project.

Create new repo datafusion-ray-sql (possibly renamed to datafusion-ray or something else)
Replace the current Ballista code with Ray SQL (Ballista is already known as the distributed version of DataFusion)
Add as an optional module of the DataFusion Python bindings

100% yes. If Ray SQL can handle all TPC-H queries with just 1.7k lines of code, it’s sounds like a no-brainer. This actually makes a production-ready distributed DataFusion sound achievable and it should attract more contributors.
Can't think of a single reason not to go this route.

One question I have is who would maintain this new code? Ballista I think suffers from a slow decline due to lack of active maintenance and community. We should try to avoid the same thing happening to Ray

Docs/Readme.md probably needs a quick update for anyone joining the catchup. Distributed shuffle using rays object store was implemented in #33.
Get Outlook for Androidhttps://aka.ms/AAb9ysg

From: Andrew Lamb @.>
Sent: Monday, September 16, 2024 6:32:22 PM
To: apache/datafusion-python @.>
Cc: Jordan Fox @.>; Comment @.>
Subject: Re: [apache/datafusion-python] Proposal to Introduce Ray SQL into DataFusion Python (Issue #872)
Caution: This email originated from outside of the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe.
One question I have is who would maintain this new code? Ballista I think suffers from a slow decline due to lack of active maintenance and community. We should try to avoid the same thing happening to Ray
—
Reply to this email directly, view it on GitHubhttps://github.com/apache/datafusion-python/issues/872#issuecomment-2354266142, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AHXVCFKTRUINKJ527GAANVDZW52BNAVCNFSM6AAAAABOIBWHDCVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDGNJUGI3DMMJUGI.
You are receiving this because you commented.Message ID: @.***>

One question I have is who would maintain this new code? Ballista I think suffers from a slow decline due to lack of active maintenance and community. We should try to avoid the same thing happening to Ray

That's a great point. If the project were to become part of the Apache DataFusion project then I would certainly put time into maintaining it and helping build community around the project. I am not able to contribute in its current location.
I have recently been attempting to maintain Ballista by upgrading to more recent versions of DataFusion, but the project is large and complex and the original contributors of much of this code are no longer available to help, so it is challenging.
I believe that DataFusion + Ray is an opportunity to start fresh on a solution for distributed DataFusion as a much lighter weight alternative to Ballista and the project is small enough (~40 commits) that it will be easier for new contributors to follow along.
These are the initial tasks that I would plan on working on (with the community, hopefully) if we were to move forward with this proposal.

Make sure that the current code still runs with recent versions of Python + Ray
Update README to reflect that distributed execution is now supported
Run fresh benchmarks and compare to Ballista
Fix the outstanding bug https://github.com/datafusion-contrib/ray-sql/issues/44
Upgrade to more recent versions of DataFusion (one PR per version)

Another possibility is that interested contributors could start maintaining the project in its current location, but I am not sure who would be able to approve the PRs.

Given that I am +1 on this proposal 🚀

Another possibility is that interested contributors could start maintaining the project in its current location, but I am not sure who would be able to approve the PRs.

I think I have admin rights to the  https://github.com/datafusion-contrib org and so can add / remove people from the project as necessary. Just let me know

Really excited to see this happening! I contributed some code to Ray SQL last year (most notably, making distributed shuffle work using Ray's distributed object store), and can help answer any question regarding Ray (I work with people who build Ray, Ray Data, etc.)
On a high level, by building on top of Ray, you get a distributed execution substrate for free. Ray handles managing the cluster, scheduling tasks, managing distributed memory, fault tolerance, to name a few. This would mean basically to use DataFusion as a single-node query execution engine and build all the distributed stuff in Python on top of Ray. If this is in line with the goal of this project (or the DataFusion project), then I think it would be a good way to go.

I've been thinking some more about where this code should live.
I am now leaning towards putting the code into a new standalone datafusion-ray repository.
My reasoning for this:

We have only just gotten to the point where were have multiple people regularly maintaining the datafusion-python project and I don't think that we should put an additional burden on these maintainers
As @ozankabak mentioned, perhaps it is better to keep datafusion-python as a single node thing focusing on excellent bindings to core and then have datafusion-ray depend on datafusion-python. This will be an excellent way to ensure that datafusion-python has sufficient extension points to enable distributed use cases.
If the project fails to gain traction, then we haven't polluted the datafusion-python repo with unmaintained code

I have been thinking a lot about this and how it would fit into or use datafusion-python. This is the third case I've come across of needing some kind of way to extend datafusion-python. The other two are delta-rs and a personal project where I could really benefit from adding a custom ExecutionPlan. I keep going back and forth between two ideas

Make some kind of requirement that the version dependency is exactly the same and compiler versions are identical. I think we can do this for datafusion-ray but other projects like delta-rs don't want such a requirement.
Create a FFI interface for extending datafusion so we can use that as a safe intermediary between the projects. I have a couple of minimal examples of doing this in a branch, but as you start getting into anything that touches the SessionContext the requirements for what is included in the FFI can explode.

Regarding the question of where it should sit, I would recommend a separate repo datafusion-ray which will force the issue on making datafusion-python more extensible.

Based on @andygrove's reasoning, sounds a good idea starting a separate datafusion-ray project under Apache DataFusion, aimed at replacing Ballista. This will keep things clean and avoid mixing datafusion-python with datafusion-ray, preventing a messy, hodgepodge integration.

It does sound like we have some concensus. The next steps are to start the IP clearance process. I will make a start on that this weekend. It basically involves filling out a form and starting a vote. Once the vote passes, I can create the new datafusion-ray repo and we can push the code there and archive the original repo.
https://incubator.apache.org/ip-clearance/

@franklsf95 Would you be able to file an ICLA (unless you already have one)? The instructions are at https://www.apache.org/licenses/contributor-agreements.html
@austin362667 Before we can start the IP clearance process, we need to update the existing source files to use the ASF header. Is that something you would be able to take on?

@andygrove  Thanks for updating the title and yes, I can help.
I'll sent PR to https://github.com/datafusion-contrib/ray-sql that adds:
# Licensed to the Apache Software Foundation (ASF) under one
# or more contributor license agreements.  See the NOTICE file
# distributed with this work for additional information
# regarding copyright ownership.  The ASF licenses this file
# to you under the Apache License, Version 2.0 (the
# "License"); you may not use this file except in compliance
# with the License.  You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing,
# software distributed under the License is distributed on an
# "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
# KIND, either express or implied.  See the License for the
# specific language governing permissions and limitations
# under the License.

The PR Add ASF license header

@franklsf95 Would you be able to file an ICLA (unless you already have one)? The instructions are at https://www.apache.org/licenses/contributor-agreements.html
@austin362667 Before we can start the IP clearance process, we need to update the existing source files to use the ASF header. Is that something you would be able to take on?

Just submitted one to<EMAIL_ADDRESS>

@austin362667 I created the new repository: https://github.com/apache/datafusion-ray
Could you open a draft PR against this repository to add the code including ASF headers?
Once this is done, the next steps will be:

Hold a vote on the DataFusion mailing list for accepting the donation based on this PR
Once that vote passes, start the IP clearance vote
Once that vote passes, merge the PR

Thanks.

I ran some benchmarks from @timsaucer's repo where he has upgraded to DataFusion 41.
Performance is looking good.

Thanks, @andygrove  The result looks super promising!
And here is the draft PR  https://github.com/apache/datafusion-ray/pull/1 for the Datafusion Ray donation.

I created an issue in the new repo to track the donation process and for the work that I think needs to happen after that so that we can get to a 0.1.0 release.
https://github.com/apache/datafusion-ray/issues/2

As a formality, as part of the ASF IP clearance process, I must remind active committers that they are responsible for ensuring that a Corporate CLA is recorded if such is required to authorize their contributions under their individual CLA.
cc @franklsf95

The donation has now been accepted, so I will close this issue. Thank you to everyone who participated in this discussion.


Wrong results when using OAuthPrompt from group or channel

When using OAuth2 via OAuthPrompt the user authentication flow does not work as expected. It seems some states are being shared between the turns that prevent the authentication to complete.
As race conditions might happen the scenarios may vary. The bot was built based on the sample.
Is that a bug or the OAuthPrompt is expected to only work with private chats? If that's the case how should one proceed to authenticate a conversation started from within a group or channel (Microsoft Teams).

Are you still experiencing this problem?


http headers: use and adjust vnd.inveniordm.v1+json http accept header

closes https://github.com/zenodo/rdm-project/issues/598

The CI build should be fixed by merging https://github.com/inveniosoftware/invenio-rdm-records/pull/1649


Preparation for 0.9.0

Remove the legacy storage migration config
Remove code that run migration prior to migration v6
Add env vars for connection pool settings

Codecov Report

Merging #2 (eea5e44) into master (e762781) will decrease coverage by 0.05%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master       #2      +/-   ##
==========================================
- Coverage   94.62%   94.56%   -0.06%
==========================================
  Files          37       37
  Lines         744      736       -8
  Branches       19       21       +2
==========================================
- Hits          704      696       -8
  Misses         40       40

Impacted Files
Coverage Δ

...m/github/chiefofstate/ServiceMigrationRunner.scala
77.77% <100.00%> (-6.84%)
:arrow_down:

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 e762781...eea5e44. Read the comment docs.


Multiple file upload not working correctly for old android and chrome browser versions

Detailed description of the problem
When using Lychee's 'Upload Photo' functionality on an older smartphone with chrome, it is not possible to upload multiple photos at once - even if multiple photos could be selected.
Using the Samsung Samsung S4 Mini with Android 4.4.2 (Chrome browser 62.0.3202.84) and S5 Mini with Android 6.0.1 (Chrome browser 62.0.3202.84), the shown android dialog after selecting 'Upload Photos' only shows the default gallery and the camera. Using the gallery, it is not possible to select more than one picture at once (I know that this is the app's behavior and not Lychee's fault).
Using the Nexus 5 with android 6.0.1 and Chrome browser 62.0.3202.84, I'm able to select multiple files within the default gallery app, but only the latest one is uploaded. Looking at the server side, I only see one HTTP POST request to index.php which indicates that it may be again a problem of the android app.
In all three cases a user is not able to upload multiple files at once from the respective mobile phone which results in a very annoying amount of work ;)
Looking at the code, it turns out that the following HTML code in index.html is responsible for the upload dialog and gives Android an indication which apps the user is able to choose from:
<input id="upload_files" type="file" name="fileElem[]" multiple accept="image/*">
And this correctly addresses all image media files according to HTML5. But it seams to be the case, that the referenced android versions (and respective versions of the used chrome browser) does not handle the accept="image/*" correctly.
The value of the accept is specified as follows:

The value must be a comma-separated list of unique content type specifiers

Therefore, it is a workaround for me to use the following value of the accept attribute:
accept=".jpg,.jpeg,image/*"
Resulting in the following input element:
<input id="upload_files" type="file" name="fileElem[]" multiple accept=".jpg,.jpeg,image/*">
This results on the Samsung S4 and S5 mini phone in showing "Documents" or "My Pictures" in the presented android dialog after selecting 'Upload Photos". Using the Nexus 5, the android dialog is still the same, but the multiple upload works now correctly.
Steps to reproduce the issue
Using one of the following mobile phones and try to upload multiple files at once using the "Upload Photo" button:

Samsung Samsung S4 Mini with Android 4.4.2 and Chrome browser 62.0.3202.84
S5 Mini with Android 6.0.1 and Chrome browser 62.0.3202.84
Nexus 5 with android 6.0.1 and Chrome browser 62.0.3202.84

Output of the diagnostics (Settings => Diagnostics)
Diagnostics:
Warning: Dropbox import not working. No property for dropboxKey.
Warning: Pictures that are rotated lose their metadata! Please install Imagick to avoid that.
No critical problems found. Lychee should work without problems!
System Information:
Lychee Version:  3.1.6
DB Version:      030102
System:          Linux
PHP Version:     7
MySQL Version:   50505
Imagick:         -
Imagick Active:  1
Imagick Version: -
GD Version:      2.2.4
Plugins:
Browser and system
Multiple mobile phones with different android versions, see above

This issue will be closed because I'm not actively working on Lychee anymore. The repository of Lychee will stay available, but the main development continues under the Lychee Organisation. A fork maintained by the community. Important issues have been migrated. Feel free to re-add your issue(s) when necessary.


Replace uses of assert false with throw new AssertionError(). 264

Fixes #264
Please review it and let me know if it looks good or any more changes has to be made.

@nsaiisasidhar a quick comment: this issue does not cover JUnit asserting such as assertFalse(...). It only covers Java assertions.

@nsaiisasidhar Please follow the workflow documented in https://github.com/oss-generic/process/blob/master/docs/QuestionsIssuesPrs.md#submitting-a-pr
In particular, our project requires all PRs to follow the multi-step PR format. Please update your PR to follow that format and send a review request via CanIHasReview.

Closing due to inactivity. @nsaiisasidhar feel free to reopen when you are ready to resume.


request for paper of this code

sir please send me the full text of this paper

sir please send me the full text of this paper


#855: Pkg - Alerts

#885

@xak
We can right align sure. I don't think those messages should be paragraphs either way. Might be important restrict the amount of type you allow in that alert.

@LeoT7508 I strongly feel that white on orange works better here than black on orange. Should we stick with white?

@saadmhybris One change re: the logic for the close button.
For future reference, these may be helpful ...
To control the white space ... which unfortunately has to be done manually.
http://mozilla.github.io/nunjucks/templating.html#whitespace-control
There is also an indent filter to will indent all lines ... 4 spaces is the default
{{ button({ label: "Click me"}) | indent(8) }}

Also, thinking about what we discussed yesterday and I see that injecting additional classes into a component — like with your action and button — will be necessary as we start to build complex patterns. While having more containers to separate layout from content is usually preferred it does add complexity to the markup.
For example, what do we gain by having an __action container around the button? It allows for additional buttons or content down the line (maybe a help button next to close) but does that possibility warrant more markup and complexity. It's always about finding the right balance between flexibility and simplicity. It is often very difficult to keep things simple.
Let me consider how to best do this and we can circle back if need be.

@saadmhybris
White text on orange doesn't pass color contrast accessibility, so we are going with dark text.

@LeoT7508 - I have changed the color to black, but still don't feel right to me as a person of poor eyesight. maybe we can test this at some point later. for now, it's ready for design review.
@xak - your requested changes are in. please have a look.

@saadmhybris
What branch can I view this in?

@LeoT7508 i just restored feature/885 for your review. The branch was deleted as it was reviewed and merged.


Added very simple example of dataset map and reduce methods

Added very simple example of dataset map and reduce methods

Looks good as a starting point. We make all examples runnable, so we should figure out a way to make some of this code copy-pastable to local or Apify, and it should produce some results.
I think that we should reduce the size of individual dataset items, add an example dataset and then in the final example call Apify.pushData in the beginning with the seed data, and then run the map/reduce.
Also, in the reduce we should use the second parameter memo to inject initial state instead of having an if( index === 0) check there.

@jbartadev This is in examples repo right, so we can close it?

@jbartadev This is in examples repo right, so we can close it?

Yes. These are not valid changes anymore.


Invoke-RestMethod: When manually passing a Content-Length header on a POST, PS7.2.1 adds to it, breaking the headers

Prerequisites

[X] Write a descriptive title.
[X] Make sure you are able to repro it on the latest released version
[X] Search the existing issues.
[X] Refer to the FAQ.
[X] Refer to Differences between Windows PowerShell 5.1 and PowerShell.

Steps to reproduce
Build out a POST request, I was working on a multipart upload for Google Drive using this code as a starting point.
Make sure to specify the 'Content-Length' header
Here is a rough scaffold:
$body = @{
    Name = 'Value'
    Name2 = 'Value2'
} | ConvertTo-Json

$headers = @{
    'Content-Length' = $body.Length
}

$splat = @{
    Uri = 'https://domain.com/api/resource'
    Method = 'Post'
    Body = $body
    Headers = $headers
    'Content-Type' = 'application/json'
}

Invoke-RestMethod @splat

In my experience, this will throw an error complaining about the Content-Type parameter not being valid, which is weird, but since the Content-Length header ends up looking like: Content-Length: 10, 10 I'm assuming that the parser is assuming that is part of the following header or something.
This problem is not present in Windows PowerShell 5.1.
Expected behavior
If I manually specify the `Content-Length` header, I would expect `Invoke-RestMethod` to either replace it  entirely or leave it alone.

Actual behavior
`Invoke-RestMethod` is appending to the `Content-Length` header.

Error details
No response
Environment data
Name                           Value
----                           -----
PSVersion                      7.2.1
PSEdition                      Core
GitCommitId                    7.2.1
OS                             Microsoft Windows 10.0.19044
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0ΓÇª}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Visuals

Is this going to be looked at at any point?  It's giving me huge issues.  If I'm required to include content-length and content-type in the headers, there's just no way around it that I can find.

Tests:
(iwr "https://httpbin.org/post" -method post -headers @{'Content-Length'=2}).baseresponse.RequestMessage.Content.headers

-> Invoke-WebRequest: Sent 0 request content bytes, but Content-Length promised 2.
(iwr "https://httpbin.org/post" -method post -body "a" -headers @{'Content-Length'=1}).baseresponse.RequestMessage.Content.headers

-> Invoke-WebRequest: The cmdlet cannot run because the -ContentType parameter is not a valid Content-Type header. Specify a valid Content-Type for -ContentType, then retry. To suppress header validation, supply the -SkipHeaderValidation parameter.
(iwr "https://httpbin.org/post" -method post -body "a" -headers @{'Content-Length'=1} -SkipHeaderValidation).baseresponse.RequestMessage.Content.headers

-> Invoke-WebRequest: 400 Bad Request
(iwr "https://httpbin.org/post" -method post -body "a" -headers @{'Content-Length'=1} -SkipHeaderValidation -SkipHttpErrorCheck).baseresponse.RequestMessage.Content.headers

->Content-Length {1, 1}
Content-Type   {application/x-www-form-urlencoded}
`

@iSazonov @ThePoShWolf @FordycetheSecond It looks like it was fixed in master, can you please check? (I don't know exactly when it was fixed but it doesn't work in PS 7.3.2)
Test:
$body = @{
    Name = 'Value'
    Name2 = 'Value2'
} | ConvertTo-Json

$headers = @{
    'Content-Length' = $body.Length
}

$splat = @{
    Uri = 'https://httpbin.org/post'
    Method = 'Post'
    Body = $body
    Headers = $headers
    ContentType = 'application/json'
}

Invoke-WebRequest @splat
Invoke-RestMethod @splat

Perhaps it was fixed in .Net 8 Preview1.

@iSazonov I confirm it works correctly in PS 7.4.0 preview2


✨ Support custom strategies

closes #2
What
The goal of this PR was to close #2 and allow users to specify custom storage strategies.
It also significantly refactors the flow of config values through the module and provides some better documentation.
// old api
const {withWebhook, withShop, shopifyRouter} = require('@shopify/shopify-express')
app.use(shopifyRouter(config))
app.use(someRoute, withShop(), someHandler)
app.use(someRoute, withWebhook, someHandler)

// new api
const shopifyExpress = require('@shopify/shopify-express')
const {router, middleware: {withShop, withWebhook}} = shopifyExpress(config)
app.use(router)
app.use(someRoute, withShop, someHandler)
app.use(someRoute, withWebhook, someHandler)

How
We now return a single master function shopifyExpress, that takes a shopifyConfig object. This lets us build all of our middlewares out by injecting the config dependency into them as we go.
Why
I'll annotate the PR with comments to give more specifics, but in general I took this approach for a few reasons

@shopify/shopify-express should not have secret environment variables it cares about now that it's a standalone package
we can't let users pass in their own strategy if they're hardcoded, and if they are going to be passed in they need to be available to other modules inside the project somehow
explicit over implicit dependency passing makes it easy to follow how the code flows through the app

Example usage
this branch of shopify-node-app
Trying it out
Assuming you already have an installed app on a store running a local shopify-node-app

cd to your local shopify-express clone
checkout refactor/custom-strategies-and-cleaner-api
yarn install
yarn link
cd to your local shopify-node-app clone
yarn install
checkout experiment/shopify-express-refactor
yarn link @shopify/shopify-express

PS: Sorry for the giant PR

Versioning question:
Since this is a breaking change, should we bump the version in package.json, publish to NPM, and add an entry to the CHANGELOG? I think it might make sense for the change to not just magically happen in Git and nowhere else.

Versioning question:
Since this is a breaking change, should we bump the version in package.json, publish to NPM, and add an entry to the CHANGELOG? I think it might make sense for the change to not just magically happen in Git and nowhere else.

Versioning question:
Since this is a breaking change, should we bump the version in package.json, publish to NPM, and add an entry to the CHANGELOG? I think it might make sense for the change to not just magically happen in Git and nowhere else.

Versioning question:
Since this is a breaking change, should we bump the version in package.json, publish to NPM, and add an entry to the CHANGELOG? I think it might make sense for the change to not just magically happen in Git and nowhere else.

Just did a 🎩  on shopify-node-app and it worked like a charm. You'll need to rebase that branch and probably bump the shopify-express version in the example app.


Invalid hunt query

This is not a valid hunt query.
DeviceAlertEvents|where Title == "Tamper Protection bypass"

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

ID: f31e7b39-b7c3-188a-0b2a-02ff10a5be6d
Version Independent ID: 44940a13-0a2d-e281-560b-cb4d8a5d1ccf
Content: Protect security settings with tamper protection
Content Source: microsoft-365/security/defender-endpoint/prevent-changes-to-security-settings-with-tamper-protection.md
Product: m365-security
Technology: mde
GitHub Login: @denisebmsft
Microsoft Alias: deniseb

@sslawter Thank you for your feedback. I'm currently testing this and will update the document if need be.
Thnaks
Sri

@sslawter Thank you for your feedback. It should be AlertInfo|where Title == "Tamper Protection bypass". The changes are updated in the document.
Hope this helps!
Thanks
Sri

@msbemba Thank you very much for the contribution and sharing this explanation. @sslawter Hope this comment is helpful for you. If you see a documentation update is required, please feel free to open an issue for the same. We proceed here to close it. Thanks for taking out some time to open the issue. Appreciate and encourage you to do the same in future also.


Help including IREE as a thirdparty to target RISCV

I'm trying to build a version of iree-run-module-main.cc (https://github.com/google/iree/blob/main/iree/tools/iree-run-module-main.cc) and run it on RISCV QEMU (https://google.github.io/iree/get-started/getting-started-riscv-cmake).
As a first step, I wanted to try and just build iree-run-module-main-v2.cc and the supported RISCV IREE tools on x86. I passed the following flags to build IREE as cross-compiling IREE compiler isn't supported:
      -DIREE_BUILD_BINDINGS_TFLITE:BOOL=OFF
      -DIREE_ENABLE_TENSORFLOW:BOOL=OFF
      -DIREE_BUILD_COMPILER:BOOL=OFF
      -DIREE_ENABLE_MLIR:BOOL=OFF
      -DIREE_BUILD_SAMPLES:BOOL=OFF

When I try to build the above project with dependency on IREE, I get the following linker error:
/usr/bin/ld: iree-run-module-v2/CMakeFiles/iree-run-module-v2.dir/iree-run-module-v2.cc.o: in function `iree::Run()':
nnc-iree-run-module.cc:(.text+0x7c): undefined reference to `iree::CreateDevice(std::basic_string_view<char, std::char_traits<char> >, iree_hal_device_s**)'
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Observation/things that I've tried:

The library libiree_tools_utils_vm_util.a that contains iree::CreateDevice() signature is generated. I tried explicitly linking the library but with no success. Changing the order of linking didn't help too.
Building IREE with IREE_BUILD_SAMPLES=ON successfully builds but this flag in turn turns on the compiler that cannot be cross compiled.
In the function call to iree::CreateDevice(), I tried using std::string as well as absl::string_view so I don't think it's a signature matching issue.

The same error comes when building with RISCV toolchain. The reason for building for x86 arch was just to make debugging easier.
Please let me know if you need any other information, any help would be much appreciated. Thank you!

Just a note: cross compiling the compiler isn't supported, but I did succeed at it recently (through Emscripten) with some effort. See this gist for details.

Depending on when you branched from main, I think you'll need to incorporate the changes to iree-run-module.cc from https://github.com/google/iree/pull/5602 to your iree-run-module-main-v2.cc. Specifically, ABSL_FLAG(std::string, was replaced with IREE_FLAG(string,, along with some other details.
From your linker error, I think there was an implicit cast here that no longer exists:
- IREE_RETURN_IF_ERROR(CreateDevice(absl::GetFlag(FLAGS_driver), &device));
+ IREE_RETURN_IF_ERROR(CreateDevice(std::string(FLAG_driver), &device));

That CreateDevice function accepts absl::string_view:
https://github.com/google/iree/blob/e749314a05d7d0ff8bc174096ff546d6fc65ec1c/iree/tools/utils/vm_util.h#L79

(you can also bring your own flags impl, including Abseil's, or avoid flags entirely, but maybe a different cast is needed to match that string_view parameter?)

I removed the flag in my program and had explicitly casted the driver name to both std::string and absl::string_view but gives the above linker error

As a more general note, if you want to use IREE in a thirdparty project, https://github.com/iml130/iree-template-cpp might be worth to look at.

Thank you @marbre and @ScottTodd for your inputs. I tried again with the above template and was able to sort the issue. The linker error was introduced due to a cmake file (which I thought to be innocuous!) that I was including in my project.


Partly revert removal of docs and coverage CI

Motivation
Research and fix the cause of failing CI docs and coverage.
Solution
Revert https://github.com/ZcashFoundation/zebra/pull/1688 fully but use beta as the toolchain instead of nightly in the docs. For the coverage we need to stay in nightly as coverage uses a compiler option -Z only available there.
More info about docs failure at https://github.com/Alexhuszagh/rust-lexical/issues/55
We used nightly because of some nightly-only docs feature that are available in beta now.
Review
@dconnolly after CI pass.
Related Issues
Closes https://github.com/ZcashFoundation/zebra/pull/1688

This is a partial revert, because some of the reverted config has been changed to beta rust


Adding REST API for delete user from groups

To support some use case scenarios that at times customer want to have a clean start by ignoring all previous/existing user-group mappings, so that they can make sure no unwanted messages got sent.
request to implement the REST API with DELETE /v1/hubs/<hub_name>/users/<user_id>
behavior will be remove all existing user-group map for user_id.
e.g. the sequence
PUT /v1/hubs/h1/groups/g1/users/u1
PUT /v1/hubs/h1/groups/g2/users/u1
PUT /v1/hubs/h1/groups/g1/users/u2
DELETE /v1/hubs/h1/users/u1
PUT /v1/hubs/h1/groups/g3/users/u1
after this sequence, u1-g1 and u1-g2 will both be removed, so after all these, if send message to g1, only u2 will receive it.
This is a feature request from two large MS internal customers.

Are there updates on timeline?

Hi @joeuy We added an new REST API called remove-user-from-all-groups
With this api, you can remove all previous/existing user-group mappings of the user.


docs: update Breadcrumb widget example

Before

After

That's indeed way better than an empty box 😅


[Component] Buttons

Buttons
Tracking the design and development lifecycle of the button component.
Related links:

https://cfpb.github.io/design-system/components/buttons
https://cfpb.github.io/design-system/development/variables#buttons
https://cfpb.github.io/design-stories/?path=/docs/components-button--docs

Effort level: Large
(Levels: Small, Medium, Large, Extra Large)
Definition of ready
Checklist

[X] The component has an identified use case (one of the 3 options below)
[X] The current state of the component has been identified (one of the 3 options below)
[X] The task is broken into steps

Definition of ready questionnaire:
Identify the use case:

[X] Small business lending filing platform
[ ] Metro 2 tool
[ ] No identified project need but does exist in the CFPB Design System

Identify the current state:

[X] The component already exist in the CFPB Design System

Build an exact React translation of the CFPB Design System component
Build the component in Figma (if it is not already included in the CFPB Design System template)

[ ] The component does not exist in the CFPB Design System but does exist on cf.gov

Conduct a design audit of any component variations that exist on our website (if applicable)
Determine final design
Build the component in Figma (if it is not already included in the CFPB Design System template)
Add the component to UI component library
Incorporate the component into the CFPB Design System or alert the CFPB Design System maintainers that this component will need to be added.

[ ] The component does not exist in the CFPB Design System or on cf.gov (but has an identified use case and is being tracked as part of the SBL or M2 work)

Design the component as a part of the SBL or Metro 2 project work
Build the component in Figma (if it is not already included in the CFPB Design System template)
Add the component to UI component library
Incorporate the component into the CFPB Design System or alert the CFPB Design System maintainers that this component will need to be added.

Indicate which types we will include in the React component library.
Types

[X] Primary
[X] Secondary
[X] Disabled
[X] Destructive
[X] Printed

Sizes

[X] Super
[X] Full-width

Groups

[X] Group

Buttons with icons

[X] Static icon (may be covered by the pagination component)
[X] Animated icon (confirm use case for SBL)

Definition of done
Component is fully functional

[ ] Visual regression testing has been conducted
[ ] All interactive states are accounted for and incorporated
[ ] The component is successfully integrated into the Small business lending filing platform (or prototype) or other CFPB  web app

Accessibility checks have been completed

[ ] Component is keyboard-friendly (navigable with tab, space, enter, arrow keys, etc.)
[ ] Component is screen reader friendly
[ ] Component does not introduce new errors or warnings in WAVE

For existing components:

[ ] React component visually and functionally matches the CFPB Design System component
[ ] Visual regression testing has been conducted and component has passed all tests
[ ] Manual visual review has been conducted and component has passed this review
[ ] Storybook page includes all types and variations of the component that are included in the CFPB Design System

For new components:

[ ] The CFPB Design System maintainers have been alerted that there is a new component that must be added to the system OR
[ ] We have added the new components to the CFPB Design System
[ ] Documentation has been written and is included in Storybook and/or the CFPB Design System

Final sign off

[ ] The component has been reviewed and approved by UI, UX, and FEWD

@meissadia
Here's a checklist of Design System button components that have been brought into the component library:
Types

[X] Primary
[X] Secondary
[X] Disabled
[X] (Design System calls this a destructive button and in our library it's a warning button. The code calls it a warning button. Check whether it should be renamed in DS or here)
[ ] Printed

Sizes

[X] Super
[X] Full-width

Groups

[ ] Group

Buttons with icons

[ ] Static icon (may be covered by the pagination component)
[ ] Animated icon (confirm use case for SBL)

Button as link and button as destructive link may be more appropriate in the links section. In the button section the link is shown in a section called "Destructive actions" and includes the button and the link. The destructive link on it's own does also appear in the Links section of the Design System.

Other points of emphasis

H2 for Types/Groups/Sizes
Apply sentence case to all labels (in stories)
Match DS language directly (labels, etc)

@natalia-fitzgerald Re: Destructive link/button
They are different HTML elements but use the same CSS classes to style them.
Based on the elemental difference I'd say that they would be separate components.
I've gone ahead an implemented the Destructive action variant as a button.


Samba addon crashes if samba location is offline

I've noticed that Samba Addon is exiting the container if the destination is unavailable. Would be possible to introduce a setting for which the addon only tries to reach out the samba destination when the cron should start?
I often shutdown my nas and periodically turn on (just before the cron schedule).

duplicate of #148


[log4cxx] Update to 0.12.0

Update to 0.12.0 and re-generate patches.
Fixes #19914 maybe fix #19730.

Hi @SergeiDelov, does this update fix your issue?

/azp run

@strega-nil-ms Do you know why the gz file extract failed?
  1 CMake Error: Problem with archive_read_next_header(): Pathname can't be converted from UTF-8 to current locale.
  2 CMake Error: Problem extracting tar: /Users/vcpkg/Documents/vcpkg/downloads/apache-log4cxx-0.12.0.tar.gz

@JackBoosY I have no idea... I guess it's because the tarball has a non-ASCII pathname, but that should be supported...

If the problem is with files under src/test/resources/output, those can be ignored - that folder was incorrectly added to the distribution.  Those files are output data from the unit tests.

Use command tar xzvf result: pass.

Maybe related to these file:
src\test\resources\output\dom³
src\test\resources\output\dom㆕
src\test\resources\output\PaxHeader\dom³
src\test\resources\output\PaxHeader\dom㆕

@JackBoosY yeah, the issue is that the locale is set to "C", which is apparently not treated as utf-8; the fix here is to fix the CI, since we can't in general set the locale to a utf-8 locale.


fix: aria-label of font_size

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

PR Type
What kind of change does this PR introduce?

[X] Bugfix
[ ] Feature
[ ] Code style update (formatting)
[ ] Refactoring (no functional changes)
[ ] Documentation content changes

Other information

Issue Number: N/A

Oopsie, thanks!


Getting this issue while train model with object of TrainValidationSplit with param grid builder

[LightGBM] [Info] Number of positive: 52, number of negative: 34
[LightGBM] [Info] Total Bins 80
[LightGBM] [Info] Number of data: 86, number of used features: 4
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
[LightGBM] [Warning] No further splits with positive gain, best gain: -inf
Traceback (most recent call last):
File "prac4.py", line 92, in
model = tvs.fit(train)
File "/home/xpms/workspace/spark-env/lib/python3.6/site-packages/pyspark/ml/base.py", line 132, in fit
return self._fit(dataset)
File "/home/xpms/workspace/spark-env/lib/python3.6/site-packages/pyspark/ml/tuning.py", line 545, in _fit
for j, metric in pool.imap_unordered(lambda f: f(), tasks):
File "/home/xpms/anaconda/lib/python3.6/multiprocessing/pool.py", line 735, in next
raise value
File "/home/xpms/anaconda/lib/python3.6/multiprocessing/pool.py", line 119, in worker
result = (True, func(*args, **kwds))
File "/home/xpms/workspace/spark-env/lib/python3.6/site-packages/pyspark/ml/tuning.py", line 545, in
for j, metric in pool.imap_unordered(lambda f: f(), tasks):
File "/home/xpms/workspace/spark-env/lib/python3.6/site-packages/pyspark/ml/tuning.py", line 50, in singleTask
metric = eva.evaluate(model.transform(validation, epm[index]))
File "/home/xpms/workspace/spark-env/lib/python3.6/site-packages/pyspark/ml/base.py", line 171, in transform
return self.copy(params)._transform(dataset)
File "/home/xpms/workspace/spark-env/lib/python3.6/site-packages/pyspark/ml/wrapper.py", line 254, in copy
that._java_obj = self._java_obj.copy(self._empty_java_param_map())
File "/home/xpms/workspace/spark-env/lib/python3.6/site-packages/py4j/java_gateway.py", line 1257, in call
answer, self.gateway_client, self.target_id, self.name)
File "/home/xpms/workspace/spark-env/lib/python3.6/site-packages/pyspark/sql/utils.py", line 63, in deco
return f(*a, **kw)
File "/home/xpms/workspace/spark-env/lib/python3.6/site-packages/py4j/protocol.py", line 328, in get_return_value
format(target_id, ".", name), value)
py4j.protocol.Py4JJavaError: An error occurred while calling o282.copy.
: java.lang.NoSuchMethodException: com.microsoft.ml.spark.LightGBMClassificationModel.(java.lang.String)
at java.lang.Class.getConstructor0(Class.java:3082)
at java.lang.Class.getConstructor(Class.java:1825)
at org.apache.spark.ml.param.Params$class.defaultCopy(params.scala:846)
at org.apache.spark.ml.PipelineStage.defaultCopy(Pipeline.scala:42)
at com.microsoft.ml.spark.LightGBMClassificationModel.copy(LightGBMClassifier.scala:95)
at com.microsoft.ml.spark.LightGBMClassificationModel.copy(LightGBMClassifier.scala:59)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:62)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43)
at java.lang.reflect.Method.invoke(Method.java:498)
at py4j.reflection.MethodInvoker.invoke(MethodInvoker.java:244)
at py4j.reflection.ReflectionEngine.invoke(ReflectionEngine.java:357)
at py4j.Gateway.invoke(Gateway.java:282)
at py4j.commands.AbstractCommand.invokeMethod(AbstractCommand.java:132)
at py4j.commands.CallCommand.execute(CallCommand.java:79)
at py4j.GatewayConnection.run(GatewayConnection.java:238)
at java.lang.Thread.run(Thread.java:748)
2018-06-25 15:58:36 WARN  CacheManager:66 - Asked to cache already cached data.
(spark-env) xpms@xpms-Latitude-E6440:~/spark_project$ 2018-06-25 15:58:36 ERROR DAGScheduler:91 - Failed to update accumulators for task 0
java.net.SocketException: Connection reset
at java.net.SocketInputStream.read(SocketInputStream.java:210)
at java.net.SocketInputStream.read(SocketInputStream.java:141)
at java.net.SocketInputStream.read(SocketInputStream.java:224)
at org.apache.spark.api.python.PythonAccumulatorV2.merge(PythonRDD.scala:632)
at org.apache.spark.scheduler.DAGScheduler$$anonfun$updateAccumulators$1.apply(DAGScheduler.scala:1131)
at org.apache.spark.scheduler.DAGScheduler$$anonfun$updateAccumulators$1.apply(DAGScheduler.scala:1123)
at scala.collection.mutable.ResizableArray$class.foreach(ResizableArray.scala:59)
at scala.collection.mutable.ArrayBuffer.foreach(ArrayBuffer.scala:48)
at org.apache.spark.scheduler.DAGScheduler.updateAccumulators(DAGScheduler.scala:1123)
at org.apache.spark.scheduler.DAGScheduler.handleTaskCompletion(DAGScheduler.scala:1206)
at org.apache.spark.scheduler.DAGSchedulerEventProcessLoop.doOnReceive(DAGScheduler.scala:1820)
at org.apache.spark.scheduler.DAGSchedulerEventProcessLoop.onReceive(DAGScheduler.scala:1772)
at org.apache.spark.scheduler.DAGSchedulerEventProcessLoop.onReceive(DAGScheduler.scala:1761)
at org.apache.spark.util.EventLoop$$anon$1.run(EventLoop.scala:48)

@swaritras thanks for finding this, it is a bug in the copy method:
https://github.com/Azure/mmlspark/blob/dca1cc7de0dadcd5ea134bf8800190f45557e868/src/lightgbm/src/main/scala/LightGBMClassifier.scala#L107
we should instead be returning LightGBMClassificationModel()
I will send out a PR

@swaritras I've created PR #330 to fix this issue - I was able to repro and verify the fix with a test case using TrainValidationSplit.

@swaritras the PR has been merged in, I'm closing the issue - this build contains the fix:
--packages com.microsoft.ml.spark:mmlspark_2.11:0.12.dev24+1.gc5b6cb9
--repositories https://mmlspark.azureedge.net/maven


Fixed user_vm_view to only display keypairs belonging to the account.

The user_vm_view displayes the keypair information by joining vm_details with ssh_keypairs on the key value exclusively.
We found a scenario in which this can cause information leakage. If there are two accounts using the same key, but create a different key name for it, and then a vm is created using one of the keys, the view will list both keypairs as belonging to the vm, which can in turn cause confusion to the users who see a keypair name which they did not create.
The fix simply limits the view to displaying keypairs which belong to vm's account.
I added it to the latest schema migration only; should I also include it in the previous ones?

Thanks for the fix @nnesic ! Pinging @DaanHoogland @bhaisaab to review.

Looks good to me, though I've not tested it. Since the key is the same (content wise), it should not cause any security issue other than some naming confusion.

Manual Testing:
Before:
ssh_keypairs
mysql> select * from ssh_keypairs;
+----+------------+-----------+--------------+-------------------------------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
| id | account_id | domain_id | keypair_name | fingerprint                                     | public_key                                                                                |
+----+------------+-----------+--------------+-------------------------------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+
|  1 |         60 |         3 | rajani       | 60:f3:cb:9b:c4:b8:c1:cd:36:13:3a:28:17:26:01:b2 | ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAAAgQDo6DcY9r8Br56JUD2Vv33KNiieElPQV0+Qepz+8lGHlIlYn8I2ubXcVOrGBtET6JGlHqVlSmjKEjeHXcvILPZsxxMjGX4skPQtBVOaH1kQcYU2YpYxxm5piQPpOo263AkWmQrfBb/W0n8ChWfOuBSYvbpzdqATsQUGnNCQuthBKQ== |
|  3 |         62 |         3 | bharat       | 69:72:47:63:10:aa:76:37:d9:fc:c2:51:66:1e:b9:93 | ssh-rsa AAAAB3NzaC1yc2EAAAADAQABAAAAgQDo6DcY9r8Br56JUD2Vv33KNiieElPQV0+Qepz+8lGHlIlYn8I2ubXcVOrGBtET6JGlHqVlSmjKEjeHXcvILPZsxxMjGX4skPQtBVOaH1kQcYU2YpYxxm5piQPpOo263AkWmQrfBb/W0n8ChWfOuBSYvbpzdqATsQUGnNCQuthBKQ== |
+----+------------+-----------+--------------+-------------------------------------------------+----------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------+

user_vm_view;
mysql> select id,name,account_name,keypair_name from user_vm_view;
+----+------+--------------+--------------+
| id | name | account_name | keypair_name |
+----+------+--------------+--------------+
| 67 | rvm1 | rajanik      | rajani       |
| 68 | bvm1 | bharatk      | rajani       |
| 67 | rvm1 | rajanik      | bharat       |
| 68 | bvm1 | bharatk      | bharat       |
+----+------+--------------+--------------+

Ui vm screenshot:

checked out the PR
did mvn clean install -Pdeveloper,systemvm on an existing setup
manually ran the drop view and create view query
start jetty mvn  -pl client jetty:run -o

After the change.
user_vm_view
mysql> select id,name,account_name,keypair_name from user_vm_view;
+----+------+--------------+--------------+
| id | name | account_name | keypair_name |
+----+------+--------------+--------------+
| 67 | rvm1 | rajanik      | rajani       |
| 68 | bvm1 | bharatk      | bharat       |
+----+------+--------------+--------------+
2 rows in set (0.00 sec)

UI Vm screenshot

:+1:
Can you create an issue for this in jira and add it to the commit message please?


You've got the why and how, but not the "what"

It is obvious how you did made this and why, but there's no indication as to what this is.

Thank you! I updated the README in https://github.com/PokerFacowaty/isfri/commit/e411cee5439b4da9215a01ab3d5c3313e8701a6b


Adds both list and single step spec Gallina signalling counter

Here I've added simpler single step semantics, and two Gallina specifications for the signalling counter. They all agree on the test inputs, but I have not seriously tried the Admitted proofs of implementation meets spec.

Moving to draft- Now we have monad semantics/example I will update this with to match. #375


#1172 extend ketcher api add possibility to run server functions

The current solution is not suitable, should implement changes according to discussion


Setting User-Agent?

How much control do we want to give users over the user-agent header? Is appending lib_name and lib_version enough? Or should they have full control over the header?
See googleapis/google-cloud-common#265

Users can now set metadata in the configuration, and can use this to set a "user-agent" header. Is this sufficient? Or should users have a specific configuration option for providing the header value?

The config has metadata, and users can set a user agent header in it.


Fix calculation of receiver id

I used 79f as sender id, but the received frames were dropped during the check in Elster.listener. The reason was the incorrect calculation of the receiver id. This is fixed by this pull request.
I've used KCanFrame::GetReceiverId() in NCanUtils.cpp as reference.

@fknipp Thank you!


Bug / Feature request: do not load tiles on hidden layer

I have a dozen of WMS layers in a leaflet map. Within the layer control I can turn visibility of that layer on and off.
So far so good.. but with quite a few layers the darn thing gets quite slow.. always loading tiles I don't want to see!
I've been fumbling around with adding and removing layers, but when playing around with turning on and off layers the control is lost rather fast (wrong layers becoming visible and unable to turn on/off layers).
Also I've been trying to get this to work within WMS layers, but no luck:
https://stackoverflow.com/questions/28551376/avoid-loading-of-invisible-tiles-when-using-multiple-tile-layers-in-leaflet
Wouldn't it be great for overall performance to simply not load invisible tiles?
And only start loading them upon a visibility level of more the 0 ?

Sorry, made an error!
detection of which layer was on/off was wrong in my application


[EXPLOIT] Not every block deserve to be placed

Floating CactoCUBE

Triplechest
https://i.gyazo.com/fec182312d5f34b24c0e8dbc8a37eccb.gif

And yes, i used blueprint block to place them.

Noted lol.


Remove travis-ci build

just remove the .travis.yml?

And go into Travis and remove it there

Turned off build in Travis as well


[Tracking] RFC-002: Sync Buffer Path Changes from Host To Guest

We intend to implement RFC-002 through 2 pull requests which need to be implemented simultaneously in teletype and teletype-client
Refs: Original Issue #147

Closing atom/teletype-client#55 in favor of the implementation in atom/teletype-client#57
Refactored #353 to reflect this change.

@jasonrudolph Thanks so much for working with us for this RFC! I believe that we are now all set, and this tracking issue can be closed!


"Cannot read property 'config' of null" while running "ng build"

I pulled the latest version of the project and issued the following commands:

cd quickstart
npm install
npm install --save-dev @angular/cli
npm install --save-dev @angular/compiler-cli

Then I tried to run "ng build --verbose" and got the following weird error:
Cannot read property 'config' of null
TypeError: Cannot read property 'config' of null
at Class.run (C:\tmp\ang\quickstart\node_modules@angular\cli\tasks\build.js:15:56)
at Class.run (C:\tmp\ang\quickstart\node_modules@angular\cli\commands\build.js:149:26)
at Class.Command.validateAndRun (C:\tmp\ang\quickstart\node_modules@angular\cli\ember-cli\lib\models\command.js:128:15)
at C:\tmp\ang\quickstart\node_modules@angular\cli\ember-cli\lib\cli\cli.js:92:22
at tryCatch (C:\tmp\ang\quickstart\node_modules\rsvp\dist\rsvp.js:539:12)
at invokeCallback (C:\tmp\ang\quickstart\node_modules\rsvp\dist\rsvp.js:554:13)
at C:\tmp\ang\quickstart\node_modules\rsvp\dist\rsvp.js:629:16
at flush (C:\tmp\ang\quickstart\node_modules\rsvp\dist\rsvp.js:2414:5)
at _combinedTickCallback (internal/process/next_tick.js:67:7)
at process._tickCallback (internal/process/next_tick.js:98:9)
What I'm doing wrong?

Same issue was discussed here: https://github.com/angular/angular-cli/issues/4736
I'm also struggling to make it work..

Same issue when I try to 'ng build' this Visual Studio Template.

You can't do this stuff here. The quickstart is a systemjs project. You can't convert it to CLI just by installing two packages.

Same of my how to resolve this issue?

I got the similar issue , but resolved it by adding .angular-cli.json file at root level (As I creted a diffrent repo by just copying the existing angular project)

I got similar issue and resolved it by adding .angular-cli.json file.

@ppgowda4 I got the same issue and I didn't solve it. Can you tell me where did you create that json file? I mean what's the exact path of it? And what's the content of it?

Same problem.
Solution copy .angular-cli.json from you angular folder
and change root path to be adequate.

Why are you people trying to do "ng build" here? This is not a cli project nor is meant to be

Well in my case i was Trying to do a ng serve

nor ng serve. This is a completely different thing

I get the same error while running "ng serve" where to copy .angular-cli.json file?

Don't do it.

What can be the possible solution?

to what problem?

when i enter 'ng serve'
Cannot read property 'config' of null TypeError: Cannot read property 'config' of null     at Class.run (C:\Users\Shubham Kamath\Desktop\Projects\web-chat\node_modules\@angular\cli\tasks\serve.js:23:63)     at check_port_1.checkPort.then.port (C:\Users\Shubham Kamath\Desktop\Projects\web-chat\node_modules\@angular\cli\com mands\serve.js:114:26)     at <anonymous>     at process._tickCallback (internal/process/next_tick.js:188:7)

And why are you running ng serve?

"Ng serve" is to start the agunlar application. I dont understand your question Foxandxss. Thats when people are experiencing issues.
So if you dont use ng serve what do you use?

No, ng serve is a command that the Angular CLI applications have. This is not an Angular CLI application so that command doesn't exist.
It is like trying to use a plane like if it were a car. They both travels, but they are different things.

I'm learning Angular and was going through a tutorial. This is Angular-Firebase application. I figured out that .angular-cli.json file was moved. Now pasted it back and it's working. Thanks!

I feel like I am invisible. Don't use this project like a CLI application, I created this repo, I know what I am talking about.
It is up to you.
What is that tutorial @shubhaem ?

Thanks Sir Foxandxss for npm start

@Foxandxss omg ty... yesterday was everything fine, but today i totally forgot, that I started the Project with "npm start" and tried "ng serve"... shame on me

@Foxandxss i understand what you mean, so what we do to deploy application?

you need .angular-cli.json to run "ng serve""
like this, save as .angular-cli.json
{
"$schema"<EMAIL_ADDRESS>"project": {
"name": "win"
},
"apps": [
{
"root": "src",
"outDir": "dist",
"assets": [
"assets",
"favicon.ico"
],
"index": "index.html",
"main": "main.ts",
"polyfills": "polyfills.ts",
"test": "test.ts",
"tsconfig": "tsconfig.app.json",
"testTsconfig": "tsconfig.spec.json",
"prefix": "app",
"styles": [
"styles.scss"
],
"scripts": [],
"environmentSource": "environments/environment.ts",
"environments": {
"dev": "environments/environment.ts",
"prod": "environments/environment.prod.ts"
}
}
],
"e2e": {
"protractor": {
"config": "./protractor.conf.js"
}
},
"lint": [
{
"project": "src/tsconfig.app.json"
},
{
"project": "src/tsconfig.spec.json"
},
{
"project": "e2e/tsconfig.e2e.json"
}
],
"test": {
"karma": {
"config": "./karma.conf.js"
}
},
"defaults": {
"styleExt": "scss",
"component": {}
}
}

@marslan2037 This project is not optimized for deployments, it says so in the readme. It is a demo project for the docs, not to create your own applications.
@adamnurdin01 I said it already, but don't do that.

As a forewarning, we are moving the CLI npm package to "@angular/cli" with the next release,
which will only support Node 6.9 and greater. This package will be officially deprecated
shortly after.
To disable this warning use "ng set --global warnings.packageDeprecation=false".
Parsing angular-cli.json failed. Please make sure your angular-cli.json is valid JSON. Error:
SyntaxError: Unexpected token  in JSON at position 0
npm ERR! Test failed.  See above for more details.

npm start doesnt solve the problem.. can anyone help with this?

Fixed this issue in my case...If you are moving your project through zip file then   1.move the zip file to the wanted folder 2. unzip  the project 3.run npm install 4. run npm start ..
hope this helps

OK, since I found this page, like apparently many others looking for "ng serve/build" with null config error, I will add my 2 cents here too so it will be seen...
When I saw the bit about the .angular-cli.json file being missing, I realized
My problem was I had  (unusual for me) copied one project folder to another via the Mac OSX Finder,
WHICH DOESNT COPY THE ".whatever" (i.e. hidden) files!! (arrg).
(Normally, I do that sort of thing via Eclipse which does do the copy.)

Its because it's searching for the angular-cli.json and it can't find it.
adding it back should work.

In project folder, renaming the file _angular-cli.json to .angular-cli.json solved this issue.

I got the similar issue , but resolved it by adding .angular-cli.json file at root level (As I creted a diffrent repo by just copying the existing angular project) .it was missing in my case and I didnt created project using cli

The solution for me was this one. If you are creating the image with Docker, make sure .angular-cli.json is not in your .dockerignore file


Always On via Task

I'd like to see an example of how to set "Always On" via the task. Is that something I need to put in appSettings? in configurationStrings? if so, what is the string? -alwaysOn ? --always-on? AlwaysOn?

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

ID: f82ad1fa-cee5-c5c6-7050-21aaec6b714a
Version Independent ID: 617316ff-2dab-d021-880f-c1c79313c320
Content: Azure Function App task - Azure Pipelines
Content Source: docs/pipelines/tasks/deploy/azure-function-app.md
Product: devops
Technology: devops-cicd-tasks
GitHub Login: @azooinmyluggage
Microsoft Alias: atulmal

Thanks for your questions. It looks like you're working through an issue with your scenario or implementation, rather than an issue with the documentation. Here are a couple of options where you might consider asking your questions:

Azure DevOps Support Bot
Azure DevOps on Stack Overflow

cc @azooinmyluggage for awareness

Thanks for the links. The Docs demonstrate how to set Always On via the portal, but not how to set them via this task. That's my reason for opening this discussion here.

@bennypowers -- Benny, thank you for clarifying.  My apologies for the delay in responding.
@azooinmyluggage -- Atul, please look into this issue.


Release/1.1.0

Implements CORS configuration is not available in this release. When it will be available?

Implements CORS configuration is not available in this release. When it will be available?
It will be contained in the next release (somewhere in July) and is already available in the development branch: https://github.com/eclipse-basyx/basyx-java-sdk/pull/83

The integration in our components is currently being worked on and will be made available shorty.


Made the table conforming to rst syntax (should resolve #1185)

Hmm. On MacOS, sphinx-doc generates multiple errors because it thinks that test files cannot see modules they use, e.g.:
WARNING: [autosummary] failed to import 'cirq.AmplitudeDampingChannel': no module named cirq.AmplitudeDampingChannel
WARNING: [autosummary] failed to import 'cirq.AsymmetricDepolarizingChannel': no module named cirq.AsymmetricDepolarizingChannel
...

We could omit the test files. But I'm very surprised this detail differs between linux and mac... are you running in a python 3 environment?

Yep, that might be the case. I have a bit funky setup new (new machine), which means Python 2 by default, and Python3 installed with brew.

Better, got different errors now :D
reading sources... [100%] tutorial
/usr/local/google/home/ashalynd/cirq/docs/api.rst:434: WARNING: failed to import contrib.tpu
/usr/local/google/home/ashalynd/cirq/docs/api.rst:434: WARNING: toctree references unknown document 'generated/contrib.tpu'
/usr/local/google/home/ashalynd/cirq/cirq/google/sim/xmon_simulator.py:docstring of cirq.google.XmonStepResult.sample:5: WARNING: Inline substitution_reference start-string without end-string.
looking for now-outdated files... none found
pickling environment... done
checking consistency... done
preparing documents... done
writing output... [100%] tutorial
/usr/local/google/home/ashalynd/cirq/docs/api.rst:442:<autosummary>:: WARNING: toctree contains reference to nonexisting document 'generated/contrib.tpu'
b'<partial node>':: WARNING: toctree contains reference to nonexisting document 'generated/contrib.tpu'
b'<partial node>':: WARNING: toctree contains reference to nonexisting document 'generated/contrib.tpu'
(wall of similar errors)

Missing files somewhere? I had a fresh checkout.

There's an open issue to figure out the tpu problem.

Yep, verified that it's working.

It's not yet rendering super nicely (not sure why) but at least it's rendered as a table.

Could you include a screenshot?


Add missing pulse-related deprecation

This commit deprecates inst_map in PassManagerConfig.__init__. It should have been part of the pulse deprecation PR #13164 but I somehow missed to cover it. We're already in our feature freeze period for version 1.3 but since this is not a new feature but rather an addendum to the pulse deprecation PR  we should squeeze this in as well.

Pull Request Test Coverage Report for Build<PHONE_NUMBER>5
Details

17 of 17 (100.0%)  changed or added relevant lines in 2 files are covered.
11 unchanged lines in 3 files lost coverage.
Overall coverage increased (+0.01%) to 88.726%

Files with Coverage Reduction
New Missed Lines
%

crates/accelerate/src/unitary_synthesis.rs
1
92.2%

crates/qasm2/src/lex.rs
4
91.73%

crates/qasm2/src/parse.rs
6
96.69%

Totals

Change from base Build<PHONE_NUMBER>7:
0.01%

Covered Lines:
76426

Relevant Lines:
86137

💛  - Coveralls


[SMALLFIX]add static imports for class BlockWorkerTest.java

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16192/
Test FAILed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16191/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/Alluxio-Pull-Request-Builder/16193/
Test PASSed.


how to enable opentracing feature and combined with popular opentracing tools in skywalking

Please answer these questions before submitting your issue.

Why do you submit this issue?
[ ] Question or discussion
[ ] Bug
[ ] Requirement
[ ] Feature or performance improvement

Question

What do you want to know?

Bug

Which version of SkyWalking, OS and JRE?

Which company or project?

What happened?
If possible, provide a way to reproduce the error. e.g. demo application, component version.

Requirement or improvement

Please describe your requirements or improvement suggestions.

What do you mean popular opentracing tool in SkyWalking? We don't have concepts like that.

sorry my bad, i mean since skywalking claims support for open tracing , is there any integration example for skywalking agent plus opentracing server except skywalking collector like jaeger or other stuff?

Simple answer, we don't. We don't say we support ot without condition. If you want Jaeger ecosystem, go to their repo please. It doesn't make sense to keep asking here about another project. It is their responsibility, not ours.

sorry and thanks for your quick feedback!
personally I like skywalking very much but needs some comparison about other system, for now I am using 6.6 since 8.x not working on my env complaining collector not up and try figuring out can skywalking support over hundred microservices with heavy payload.
The reason I am asking jaeger is what they claimed good support about k8s in the doc but I never succeeded following it.
Btw is the java agent universal for all ( zipkin etc, collector)

2 things, one, 6 is not recommended. And most users have upgraded to 8 as it has much better performance and more maintainability. Then, Jeager and Zipkim have their communities and ecosystems, both of them are not SkyWalking responsibility anyway, no matter tech itself. The open source projects are driven by their contributors, currently, we don't have people working on that.Also, we can't tell others performance, as they are not experts about them.

thanks and I found this https://github.com/SkyAPMTest/Agent-Benchmarks, hope it helps!
also I am thinking about since open tracing is just a spec that means technically everybody following same format should be able to communicate with each other

No, it isn't a spec. It never is. I am a member of open tracing, trust me, it is far from a spec. OpenTracing actually is being merged into OpenTelemetry, very little people use it than you thought. Please don't read a doc and believe it. Do some research before have conclusion.
More importantly, OpenTracing doesn't have format, never, it is just an API layer. Read the thing and understand the thing you bring up.

We wouldn't and shouldn't continue on talking a thing doesn't relate and matter to SkyWalking much.


Add a succinct way to lazily check for the presence or absence of a provider

Expected Behavior
There should exist a more succinct way to lazily check for the presence or absence of a provider. Something like:
val providerIsPresent: Provider<Boolean> = someProvider.isPresentProvider()
val providerIsNotPresent: Provider<Boolean> = someProvider.isNotPresentProvider()

Alternatively, the return value of the existing Provider#isPresent() should be a Provider<Boolean> instead of a primitive boolean and a corresponding Provider<Boolean> returning Provider#isNotPresent() is added.
val providerIsPresent: Provider<Boolean> = someProvider.isPresent
val providerIsNotPresent: Provider<Boolean> = someProvider.isNotPresent

Current Behavior (optional)
Currently, the existing Provider#isPresent() returns a primitive boolean, thus eagerly resolving the provider.
Context
It is common to want to configure something based on the presence or absence of some environment variable, system property, etc.
For example, the new Develocity Gradle plugin has adopted the Provider API and it is common to want to configure some settings differently when the build is running on CI:
val isCI = System.getenv("CI") != null
develocity.buildScan.uploadInBackground = !isCI

The example above shows how it is most commonly done now, but it reads the environment variable at initialization time. Lazy alternatives exist, but they are rather verbose:
providers.provider { System.getenv("CI") != null }
providers.provider { providers.environmentVariable("CI").isPresent }
providers.environmentVariable("CI").map { true }.orElse(false)

Internal Slack discussion: https://gradle.slack.com/archives/CDYR952SH/p1712605750090979

It should be noted that the last two lazy alternatives listed aren't equivalent to the first, as they don't check if the environment variable is empty. After accounting for this, the verbosity of map no longer looks that uncalled for: providers.environmentVariable("CI").map { !it.isEmpty() }.orElse(false).

Thanks. Good catch. I've updated the description accordingly.

This feature request is in the backlog of the relevant team and is prioritized by them.


How to call initModals from JS?

In docs you mention
https://flowbite.com/docs/getting-started/introduction/#components
The initFlowbite function sets up all of the init functions for dropdowns, modals, navbars, tooltips and so on to hook onto the data attributes. Alternatively, you can also initialise each component category class separately with initDropdowns or initModals.
but there is no example how to use that in plain JS, not components. I am dynamically adding buttons that needs have tags to toggle modal (that is present in the code already) and need flowbite to re-detect these buttons and make the toggles work. I expect there is a simple function like the one mentioned initModals() that can do that, but I cannot seem to find it anywhere once the flowbite js code is loaded. Maybe its hidden in some namespace?

I have encountered the same problem, wherein I have a page that contains multiple comments, each with a 'report' button that links to a singular report modal. Whenever I dynamically add new comments (using HTMX), these newly added report buttons do not have a listener to the report modal. Using the original initModals alone does not solve the issue as it also adds event listeners to buttons that already have existing listeners, causing event stacking (and massive performance issues). To address this issue, I have created my own initModals in JavaScript that adds event listeners to new content only. Additionally, I modified the source code to access the original Modal instance (that is created when the page loads), allowing me to tag the new comment's respective report button to the original report modal!
I would like to hear a devs opinion to address this type of issue.

as I purchased the PRO version of this code, I'd love to hear some feedback from the developer, because in current situation this is rather useless. Its not only problem of modals, its any case of "data-dropdown-toggle" or any other component bound like that. Bootstrap has this sorted out very nicely and I don't see a reason why this should not be working the same way.

Hey @RadekHavelka,
Can you please give me an example of how you're trying to use it?
I'll need a reproduction to be able to provide a patch or feature update for this.
Cheers,
Zoltan

I have HTML page that uses flowbite. After it loads, when user does a certain interaction, new HTML code is loaded via javascript fetch, and injected into page.  If this code contains any "interactive" flowbite things like dropdowns, modals etc, they are NOT initialised and not working as expected, because (I believe) flowbite initialisation runs only once and I am not aware of any function to reinit it again (see my original post here about missing info in doc).
In bootstrap you would simply call something to do the init the newly added items, create listeners etc.
Would be enough to have such a call available in plain javascript (ie window.flowbite.init() or whatever), without the problems mentioned above (that it adds multiple listeners in case the elements were already initialised).
I suggest to add some data attribute or class to initialised elements and not reinit them again, this might do the magic.
I am aware I can do all the interactivity manually using javascript, but that looses advantage of this framework.

Hey @RadekHavelka,
Thanks for the explanation - this should be an easy fix.
All we need to do is attach the initFlowbite() function to the window object for the general file.
You could technically also import flowbite via WebPacks (ESM or CJS) and then import initFlowbite from 'flowbite' and you could initialise the components programatically at any point in time based on your own codebase.
I'll look into this for sure for a new minor version.
Cheers,
Zoltan

What I used to use in JQuery was that .live(click, ...) that used to work on any matching elements inserted now or in future, that was pretty handy.
https://api.jquery.com/live/
its deprecated now, but might be useful for ideas

Is there any updates on this issue?

I have tried to implement a basic webpack setup for the assets in my Django project and replaced django-compressor.
In the main script I simply tested to initialize flowbite when an htmx request loads.
// Initialize Flowbite
import { initFlowbite } from 'flowbite';

htmx.onLoad(function() {
    initFlowbite();
});

This works, but is extremely slow and buggy when I scroll the page. (Not if I comment out the initFlowbite).
Also, this initialization creates multiple modal-backdrops at the end of the body tag.  This results in an ugly and unperformant UI.
Any recommendations or solutions?

Hey everyone,
With the new release of v1.6.6 you should now be able to access all init functions including initFlowbite via the window object - I believe a part of the import problems should be addressed.
Regarding more advanced ways of initialising the event listeners we'll have to make a more comprehensive update that uses advanced verifications so we only init the newly added items to the DOM.
Cheers,
Zoltan

I am having a similar issue and it may just boil down to me not knowing how to implement it.
I have a modal (with an id of "pro") that I would like to pop up after a certain amount of time and won't have a button on the page for it. How would I go about doing this?
Thanks!

Hey everyone!
Flowbite v2.0 fixes this issue by bringing a brand new Instances Manager.
Please read more about it here:
https://flowbite.com/docs/getting-started/javascript/
https://github.com/themesberg/flowbite/releases/tag/v2.0.0
Cheers,
Zoltan

I'm struggling frustratedly to see how this issue is "completed". It's all well to say you have solved the issue with "Instances Manager".
Please, how difficult is it to present a simple example of how to initialize the interactivity of Modal, Dropdown, Drawer, etc whenever a new element bearing the flowbite data attributes is DYNAMICALLY ADDED to the page in vanilla JS?
I'm really confused since I'd assume that this is a very basic scenario that ought to be covered in a framework like this. Any suggestion for me to implement the details myself defeats the whole purpose of using a framework such as this in the first place!

Same question, how do I use Instances Manager to make fix the problem with subsequence calls to initFlowbite creating additional bindings for existing already bound components?


Fatal Error Upon Start

I try to use the following projects (https://github.com/dnaextrim/php_zklib, https://github.com/kamshory/ZKLibrary) that relate to ZKTeco and I use some cool stuffs like getting attendance, users, and system info. But I am looking for some advance stuffs like enrolling fingerprint, so I try to use these project but when I start I got already an error so I cannot advanced.
I  am using ZKTeco TX628.
I follow the instruction at README.md. And use the following codes:
<?php
require 'lib/TADFactory.php';
require 'lib/TAD.php';
require 'lib/TADResponse.php';
require 'lib/Providers/TADSoap.php';
require 'lib/Providers/TADZKLib.php';
require 'lib/Exceptions/ConnectionError.php';
require 'lib/Exceptions/FilterArgumentError.php';
require 'lib/Exceptions/UnrecognizedArgument.php';
require 'lib/Exceptions/UnrecognizedCommand.php';

 $tad_factory = new TADPHP\TADFactory();

 use TADPHP\TADFactory;
 use TADPHP\TAD;

 $comands = TAD::commands_available();

 $b1 = (new TADFactory(['ip'=>'<IP_ADDRESS>']))->get_instance();

 $dt = $b1->get_date(); // method executed via TAD class.
echo $dt;

I've got the following error:
Fatal error: Uncaught TADPHP\Exceptions\ConnectionError: Imposible iniciar conexión con dispositivo <IP_ADDRESS> in C:\xampp\htdocs\tad\lib\TAD.php:408 Stack trace: #0 C:\xampp\htdocs\tad\lib\TAD.php(271): TADPHP\TAD->check_for_connection() #1 C:\xampp\htdocs\tad\index.php(24): TADPHP\TAD->__call('get_date', Array) #2 {main} thrown in C:\xampp\htdocs\tad\lib\TAD.php on line 408
Help please!!!

Hi @aice09 did you solved this?

how did you solve the problem?


Eternl wallet: respect UTxO locking #1301

Closes #1301  and #1311
Pre-review checklist

[x] All code has been formatted using our config (make format)
[ ] Any new API features or modification of existing behavior is covered as defined in our test plan
[x] The changelog has been updated under the ## Unreleased header, using the appropriate sub-headings (### Added, ### Removed, ### Fixed), and the links to the appropriate issues/PRs have been included

I tested it by hand in the e2e browser since the npm run e2e-test command keep failing all the test. All the test passed in the browser. What remains is to refactor the code as right now is a mess.

Current state:

Most test now shows the same error for eternl e2e-test as in develop.
OneShotMinting and ECDSA examples have it's assertions broken.

I will try to fix the problem with eternl test in this pr, but it seems that the main changes for #1301 are done and working, so opening for reviews again.

Status:
I put two loops waiting until the utxos and address are updated and it seems to work:

In the browser we still have some errors happening but none of them related to the previous problem, and most of the tests are successful now.
in the terminal, we get a timeout, so we need to augment the timeout for eternl wallet, I don't know how to do that, I would figure it out tomorrow. @klntsky Can you give me some pointers on how todo it please?

The only error that I think I can't solve is
  ✗ nami:http://localhost:4008/?nami:SignMultiple:

  Error: Error: "Insufficient Utxos for 2 transactions"

but it seems to be a common problem in all wallets.

after runing the tests with differents timeouts, the only persisting problem is the following error :
  ✗ eternl:http://localhost:4008/?eternl:OneShotMinting:

  Error: Error: `submit` call failed. Error from Ogmios: [{"missingVkWitnesses":["4d149ae923cdce83dad20c83c1bb82b5da3d0940e057dd05fe1a5438"]}]

it's happening on 3 examples but the rest of them are working right now with eternl!

after runing the tests with differents timeouts, the only persisting problem is the following error :
  ✗ eternl:http://localhost:4008/?eternl:OneShotMinting:

  Error: Error: `submit` call failed. Error from Ogmios: [{"missingVkWitnesses":["4d149ae923cdce83dad20c83c1bb82b5da3d0940e057dd05fe1a5438"]}]

it's happening on 3 examples but the rest of them are working right now with eternl!

I had the same error when tried to run contract while my CTL runtime had not been completely synchronized yet.
Maybe the reason of your error is somewhat similar 🤷.

We don't need this PR anymore after https://github.com/Plutonomicon/cardano-transaction-lib/pull/1440 - the problem should be fixed


Fix prod css/sass sourcemaps (#1793)

Sass sourcemaps fail to generate in production builds due to cssnano
combined with OptimizeCSSAssetsPlugin.  Moving cssnano into
postcss-loader resolves issue.
More detail https://www.webfoobar.com/node/109
Note eval-source-maps still will not work at all for CSS, so
implementers must use Mix.sourceMaps(true, 'source-map') to allow
development builds to successfully generate sourcemaps.

Note all tests are passing when run locally. I notice last few commits do not pass appveyor ie: d432101c23198252e3952d8d89ee246d209265a8

I can confirm this PR solved the problem for me.
In my case, I was compiling LESS, SCSS, and JS files with Mix. I was trying to compile app.js, app.less, error.less, landing.less, landing.js, and bootstrap.scss.
After dumping the Webpack configuration to the console before Mix ran, it was mis-calculating the entry points as follows:
entry:       {
  '/js/app':     [
    '/Users/Me/Sites/MySite/resources/js/app/app.js',
    '/Users/Me/Sites/MySite/resources/less/app/app.less',
    '/Users/Me/Sites/MySite/resources/less/app/error.less',
    '/Users/Me/Sites/MySite/resources/less/landing/landing.less',
    '/Users/Me/Sites/MySite/resources/scss/bootstrap.scss',
  ],
  '/js/landing': ['/Users/Me/Sites/MySite/resources/js/landing/landing.js'],
}

This was causing only two sourcemaps to be generated, for /js/app and /js/landing, but none corresponding to the CSS files.
Manually patching this PR causes the entry points to calculate correctly — one for each file — and 6 corresponding sourcemaps to be generated, rather than just 2.


initial standard library commit

Great, perhaps we should establish some style guides some point since we're starting to write actual code in Alloy for the standard library (what you've written looks nice anyway).
We should try and make a new function that works around printf, and perhaps we could introduce file visibility to certain variables or functions, which means that people can't use the C functions in their code normally?

Yeah, I definitely agree about visibility. That would probably need to be implemented at the same time with modules. And the compiler would need to check functions/variables at compile time to see if they were defined.
For printf, I personally love Go's way of doing it.

Yeah Go does it nicely, I think it would be good if we could infer it too, i.e Rust:
println!("{}", 5);

Will infer it. or instead of {} or %%?

Hmmm, %% actually might be quite nice.

Even then it might be nice to support print and println functions where you can do
print("some" + "value");

Also, we should follow Rust's route of making it a macro, i.e. printf/print/println would be compile-time operations instead of run time?

Thanks
I'm not sure about whether to use macros or not.

We don't have any macro support in Alloy yet, so for now a normal run-time implementation is okay. But when we do have macro support, it would be cool (and an optimization).


Fix check-libs, update libraries and reviewers, bump versions

Proposal

Add missing 'secrets: inherit'.
Update list of default reviwers
Sync names to match common stucture
Update check-lib version
Update libraries for charm and test app

Context
We are fixing CI/CD error on lib-check, due to missing 'secrets: inherit':

charmcraft 2.2.0 from Canonical** installed
/snap/bin/charmcraft fetch-lib
No keyring found to store or retrieve credentials from.
Full execution log: '/home/runner/.local/state/charmcraft/log/charmcraft-20230302-123159.495395.log'

Release Notes
No necessary.
Testing
Can be tested on GitHub only.

I have included Dragomir fixes from #31 here. Tnx!


Allow specifying a default when asking for a route parameter from the request

Right now you can do stuff like $request->route('id') to get a route parameter via the request, but we don't expose the $default argument you get when you call $request->route()->parameter('id', 'default').
This PR just adds another optional argument to allow specifying the default value without chaining the extra method call.

This is already possible using $request->route('id') ?? 'default'

Possible and IMO more obvious

This is already possible using $request->route('id') ?? 'default'

So should we remove every $default parameter in the framework?

So should we remove every $default parameter in the framework?

Wait, are there only two ways to go? We either add it everywhere, or remove it everywhere? Is there no middleground where we keep the existing ones to avoid breaking changes, but avoid introducing new functionality?
While talking about breaking changes, this is a change in a method signature and thus a breaking change.
Am I forced to defend every existing line of code in the framework just because I point out that php's language features already does this?

The ?? approach is not the same behavior. You can't defer execution using a Closure with that approach like you can with the "default" parameters available in Laravel.


Emoticon lag

After this commit emoticons are sent late

@Latios1 what about now? I just updated the repo.

Now they work, thanks


Improve game assets

all images should be Square with transparent background
add canvas/background image
add snake head body tail image
add bugs image
try to animate the snake by flipping the image
same with the bugs
change BLOCK_SIZE to image size

Please finish this first thing today


Update de.json

Translated new strings for V4.4.3

Although the new texts from V4.4.3 are translated into german, and merged, they are neither used nor displayed.
See also ABRP https://abrp.upvoty.com/b/report-a-bug/german-gui-translation-from-v443-is-not-used/


Broken image

Broken image on https://www.oldnyc.org/#703418f-a

This was a newly-rotated image and I hadn't run ./generate_rotated_images.py.


CIImage from a MTIImage ?

 I'm trying to use a skin smoothing filter and our codebase use CIImages.
How would I get a CIImage back from the filter below ?
class MetalPetalSkinSmoothingFilter: Filter {

    var name: String = "MetalSkinSmoothing"
    private let filter = MTIHighPassSkinSmoothingFilter()

    func process(image: CIImage) -> CIImage {
        let mtimage = MTIImage(ciImage: image)
        filter.inputImage = mtimage
        return filter.outputImage! // Get a CIImage?
    }
}

Thanks a ton for pointing me to the right direction :)
Checklist

[x] I've read the README
[x] I've searched for existing GitHub issues

Ok I've worked out this:
class MetalPetalSkinSmoothingFilter: Filter {

    var name: String = "MetalSkinSmoothing"
    private let filter = MTIHighPassSkinSmoothingFilter()
    let imageView = MTIImageView(frame: UIScreen.main.bounds)

    var ctx: MTIContext?

    init(amount: Float) {
        if let device = MTLCreateSystemDefaultDevice() {
            self.ctx = try? MTIContext(device: device)
        }
        filter.amount = amount
    }

    func process(image: CIImage) -> CIImage {
        guard let ctx = ctx else {
            return image
        }
        let mtimage = MTIImage(ciImage: image, isOpaque: true)
        filter.inputImage = mtimage
        let outputMTimage = filter.outputImage!
        do {
           return try ctx.makeCIImage(from: outputMTimage)
        } catch {
            return image
        }
    }
}

Just to be sure, is that the correct way to do this?
Cheers :)

Yes, this should work, though I have some doubt about GPU command synchronization here. Anyway, let me know if you encounter any problem.

Awesome, Thanks  for taking a look :)
My first implementation was using Lady and CPU usage shot up 150%.
Now with MetalPetal it's only 40% and everything is smooth. \o/


Support RedPajama-INCITE-Chat-3B-v1

It works and builds a proper iOS app which runs pretty fast (based on branch redpajama-bak)! Need to validate if it still works after this aggressive rebase.


Great Asset

It is a great project but Please add a detailed readme file

will be uploaded soon

its done