Add Homebrew test-bot workflow

Tests to see if the subo Formula will run. Will install and run tests for the binary.

@javorszky I think that's because I'm using the tool in a way that doesn't jive normally with their stuff? It will change with the next subo release tho so I'll merge it for now and keep an eye on it.


[idea] copy path to clipboard

It would be a useful feature to copy the path of a node to the clipboard. For instance I'm looking for a value, I find its node and I want to paste the path of the node to an application.

I cam to suggest the same thing! A use case for me is in helping to craft jq queries


Video Course

This is not an issue per se but wanted to know where can I access the video materials for this course?

You can do the complete course on Udemy - https://www.udemy.com/course/kubernetes-crash-course-for-java-developers/


Remove bracket from current activity

Currently a bracket shows at the end of the current activity "}"

This should be a quick fix, just have to find where the extra bracket is coming from

Hello, Can I be assigned to this ?

Hey, i can work on this if you will assign me this.

Hello, Can I be assigned to this ?

Ok - I've assigned you


Issue extending web-recommended and configuring http-compressions

Original report by @jgarber623:
This configuration works, but doesn't include any rule customization:
{
  "extends": [
    "web-recommended"
  ],
  "formatters": [
    "codeframe"
  ],
  "ignoredUrls": [
    {
      "domain": ".*\\.googleapis\\.com/.*",
      "rules": ["*"]
    }
  ]
}

This configuration does not work:
{
  "extends": [
    "web-recommended"
  ],
  "formatters": [
    "codeframe"
  ],
  "rules": {
    "http-compression": ["warning", {
      "html": {
        "brotli": false
      }
    }]
  },
  "ignoredUrls": [
    {
      "domain": ".*\\.googleapis\\.com/.*",
      "rules": ["*"]
    }
  ]
}

The error message shown is "Invalid rule configuration in .sonarwhalrc: http-compression."
My goal is to silence errors/warnings about using Brotli and Zopfli compression.

The following works:
  "extends": [
    "web-recommended"
  ],
  "formatters": [
    "codeframe"
  ],
  "rules": {
    "http-compression": "off"
  },
  "ignoredUrls": [
    {
      "domain": ".*\\.googleapis\\.com/.*",
      "rules": ["*"]
    }
  ]
}

so it looks like a problem when merging the rule configurations options with the extends part.

I've created #907 to fix this issue. If you don't want to wait until things are merged and published on npm you can change html with target in your configuration and it should work.

@jgarber623 We published a new version of the http-compression rule. So, the easiest way to get it is to just do a npm i @sonarwhal/configuration-web-recommended.
Thanks again for taking the time to let us know!

Thanks, @molant and @alrra! Those updates did the trick.
For future travelers, my updated config works great and now looks like:
{
  "extends": [
    "web-recommended"
  ],
  "formatters": [
    "codeframe"
  ],
  "rules": {
    "http-compression": ["warning", {
      "html": {
        "brotli": false,
        "zopfli": false
      },
      "resource": {
        "brotli": false,
        "zopfli": false
      }
    }]
  },
  "ignoredUrls": [
    {
      "domain": ".*\\.googleapis\\.com/.*",
      "rules": ["*"]
    }
  ]
}


[nginx] Ignore Ingress Annotation broken

On version 0.8.3, the Ingress annotation ignore option (added with this pr) doesn't work.
Setting kubernetes.io/ingress.class to something other than nginx for my ingress does not stop it from being accessible through the load balancer.
I0308 20:40:39.378971       1 controller.go:157] Ignoring add for ingress hastebin based on annotation kubernetes.io/ingress.class
I0308 20:40:42.377270       1 controller.go:420] updating service hastebin with new named port mappings
I0308 20:40:52.403862       1 command.go:76] change in configuration detected.

@InAnimaTe please update to the latest version https://github.com/kubernetes/ingress/releases/tag/nginx-0.9.0-beta.2

Ahh thanks. In the process of moving. I'll close this for now and if I have any more issues in the future, open a fresh bug.


Unable to play out muxjs transmuxed HLS with latest Shaka player with timestampOffset.

What version of Shaka Player are you using*
Latest from master.
Can you reproduce the issue with our latest release version:
Latest release does not support muxjs transmuxing.
Can you reproduce the issue with the latest code from master:
Yes
Are you using the demo app or your own custom app:
Both.
If custom app, can you reproduce the issue using our demo app:
Yes
What browser and OS are you using:
OSX/Chrome
What are the manifest and license server URIs:
(you can send the URIs to<EMAIL_ADDRESS>instead, but please use GitHub and the template for the rest)
This probably isn't the sample I analyzed below, but it illustrates the same issue.
https://shakarepro.s3-us-west-1.amazonaws.com/repro/mbr/manifest.m3u8
What did you do?
Attempted to play the above HLS stream.
What did you expect to happen?
For it to play.
What actually happened?
I am seeing this issue where the timestamps output by muxjs are different than my TS segments when the presentationTimeOffset isn't 0.
Basically my first MPEGTS segment starts at 133500:
        {
            "media_type": "video",
            "stream_index": 0,
            "key_frame": 1,
            "pkt_pts": 133500,
            "pkt_pts_time": "1.483333",
            "pkt_dts": 133500,
            "pkt_dts_time": "1.483333",
            "best_effort_timestamp": 133500,
            "best_effort_timestamp_time": "1.483333",
            "pkt_duration": 3750,
            "pkt_duration_time": "0.041667",
            "pkt_pos": "564",
            "pkt_size": "10638",
            "width": 320,
            "height": 240,
            "pix_fmt": "yuv420p",
            "sample_aspect_ratio": "1:1",
            "pict_type": "I",
            "coded_picture_number": 0,
            "display_picture_number": 0,
            "interlaced_frame": 0,
            "top_field_first": 0,
            "repeat_pict": 0
        }

In the logs I see:
First segment 750k.ts starts at 1.4833333333333334

Which matches the first PTS (Although there is an earlier PTS for the first audio packet in the stream which appears later).
The presentationTimeOffset is set to 1.4833333333333334, but the code to adjust the PTS via the transmuxer sets it to adjust by 0:
lib/media/transmuxer.js:
this.muxTransmuxer_.setBaseMediaDecodeTime(startTime * timescale);

^ Tracing in the debugger 'startTime' is 0.
As a result the transmuxed mp4 seems to adjust all of the timestamps:
        {
            "media_type": "video",
            "stream_index": 0,
            "key_frame": 1,
            "pkt_pts": 7500,
            "pkt_pts_time": "0.083333",
            "pkt_dts": 7500,
            "pkt_dts_time": "0.083333",
            "best_effort_timestamp": 7500,
            "best_effort_timestamp_time": "0.083333",
            "pkt_pos": "2877",
            "pkt_size": "28480",
            "width": 640,
            "height": 480,
            "pix_fmt": "yuv420p",
            "sample_aspect_ratio": "1:1",
            "pict_type": "I",
            "coded_picture_number": 0,
            "display_picture_number": 0,
            "interlaced_frame": 0,
            "top_field_first": 0,
            "repeat_pict": 0
        },

And MediaSourceExtensions croaks with the following error:
video frame with PTS 141667us has negative DTS -66667us after applying timestampOffset, handling any discontinuity, and filtering against append window

Also, I wonder if getStartTimeFromTsSegment_ in lib/hls/hls_parser.js needs to be adjusted to search for a minimum timestamp closer to the beginning of the TS stream, rather than just reading the PTS out of the first TS packet it finds that has one.

@michellezhuogg could you take a look at this issue as you are more familiar with the trans-muxing code?

The problem is this is using multiplexed content (a stream with both audio and video).  Also, the audio and video streams don't start at the same time.  We use the first packet to get the time, but the other stream (which appears later) starts before that packet.  Some browsers don't support multiplexed content and require the streams to be separate; we usually only support separate streams, but don't explicitly forbid multiplexed streams.
It may be possible for this to happen for non-multiplexed streams as well.  A media segment doesn't have to start with a key frame.  If it starts with a B-frame, then the PTS times of the frames will be out of order.  We should parse the first few frames until we hit a key frame to avoid this.
But that won't help your stream, since the problem is that there are multiple streams within the TS packet stream.  If we see that it is a multiplexed stream (which I think we will always know), then we can probe further to find the second stream, but it may require a second download probe to get the timestamps.


Started superagent integration

Closed in error; reopening.

We decided to use axios instead of superagent. I'm closing this PR.


Improve transfer type descriptions

The Chore
Transfer descriptions could do a bit more to help people understand what they're looking at
New descriptions
(TRANSFER_TYPE_LOSS) = Final settlement loss: Funds deducted after final settlement loss
(TRANSFER_TYPE_WIN) = Final settlement gain: Funds added to your general account after final settlement gain
(TRANSFER_TYPE_MTM_LOSS) = Mark to market loss: Funds deducted from your margin account after mark to market loss
(TRANSFER_TYPE_MTM_WIN) = Mark to market gain: Funds added to your margin account after mark to market gain
(TRANSFER_TYPE_MARGIN_LOW) =  Margin topped up: Funds deducted from your general account to meet margin requirement
(TRANSFER_TYPE_MARGIN_HIGH) = Margin returned: Excess margin amount returned to your general account
(TRANSFER_TYPE_MARGIN_CONFISCATED) = Margin confiscated: Margin confiscated from your margin account to fulfil closeout
(TRANSFER_TYPE_MAKER_FEE_PAY) = Maker fee paid: Maker fee paid from your general account when your aggressive trade was filled
(TRANSFER_TYPE_MAKER_FEE_RECEIVE) = Maker fee received: Maker fee received into your general account when your passive trade was filled
(TRANSFER_TYPE_INFRASTRUCTURE_FEE_PAY) = Infrastructure fee paid: Infrastructure fee paid from your general account when your trade was filled
(TRANSFER_TYPE_INFRASTRUCTURE_FEE_DISTRIBUTE) = Infrastructure fee received: Infrastructure fee, paid by traders, received into your general account
(TRANSFER_TYPE_LIQUIDITY_FEE_PAY) = Liquidity fee paid: Liquidity fee paid from your general account to market's liquidity providers
(TRANSFER_TYPE_LIQUIDITY_FEE_DISTRIBUTE) = Liquidity fee received: Liquidity fee received into your general account from traders
(TRANSFER_TYPE_BOND_LOW) = Bond account funded: Funds deducted from your general account to meet your required liquidity bond amount
(TRANSFER_TYPE_BOND_HIGH)  = Bond returned: Bond returned to your general account after your liquidity commitment was reduced
(TRANSFER_TYPE_WITHDRAW) = Withdrawal: Funds withdrawn from your general account
(TRANSFER_TYPE_DEPOSIT) = Deposit: Funds deposited to your general account
(TRANSFER_TYPE_BOND_SLASHING)  = Bond slashed: Bond account penalised when liquidity commitment not met
(TRANSFER_TYPE_STAKE_REWARD) = Staking reward received: Rewards for staking received into your general account
(TRANSFER_TYPE_TRANSFER_FUNDS_SEND) = Transfer sent: Funds deducted from your general account to fulfil a transfer
(TRANSFER_TYPE_TRANSFER_FUNDS_DISTRIBUTE) = Transfer received: Funds added to your general account to fulfil a transfer
(TRANSFER_TYPE_CLEAR_ACCOUNT) = Market accounts cleared: Market-related accounts emptied, and balances moved, because the market has closed
(TRANSFER_TYPE_CHECKPOINT_BALANCE_RESTORE) = Balances restored: Your balances were restored after a network restart
Tasks

[x] Add a hover/tool tip feature so the transfer types can have a short title and a longer description
[x] Update fields in libs/types/src/global-types-mappings.ts

@candida-d  could you provide two more descriptions for the following?
TRANSFER_TYPE_CLOSE = 'Close',
TRANSFER_TYPE_WITHDRAW_LOCK = 'Lock amount for withdraw',

review types

only suggested change is: Trade -> order in

(TRANSFER_TYPE_MAKER_FEE_PAY) = Maker fee paid: Maker fee paid from your general account when your aggressive trade was filled
(TRANSFER_TYPE_MAKER_FEE_RECEIVE) = Maker fee received: Maker fee received into your general account when your passive trade was filled

But only on @candida-d's approval

rationale: when an order is filled is it because a "trade" happened. Trades are just trades. Orders are aggressive or passive IMO

@campbellssource Makes sense, I'll update the issue body with that change.


How to perform filter operations on a CSV/Parquet file

Hello, I can't seem to find an example that reads from a CSV file to perform a filter operation like scan or extract. Does QPL support this? I know IAA works with in-memory databases, so does one have to convert to some format first supported by IAA?
Thanks.

Hi @Johnnyjax , QPL doesn't support reading specific input formats, but rather reading a raw output, you'll need some sort of a wrapper function to parse CSV or any other format and input this into QPL.
You can check out our current examples for scan to see how to use QPL APIs at https://github.com/intel/qpl/blob/develop/examples/low-level-api/scan_for_specific_value_example.cpp.

Closing as no reply in a long time, please feel free to re-open if still relevant.


await response errors

It may be the case that the default error middleware could be rebuilt to support promises too. Don't have time for that right now though

Thinking a bit more about it, is this fix actually needed? 🤔
Feels like this should be equivalent to await Promise.reject(Promise.resolve('nope')).catch(e => e), or for a more precise example:
p = await (async () => { throw Promise.resolve('nope') })().catch(e => e)

I mean the end game is that we give synchronous values to the constructor of the error so they can be read synchronously by consumers. https://github.com/backstage/backstage/blob/7c116bcac7fcb93f86c381ac744fe694a81db03b/packages/errors/src/errors/ResponseError.ts#L82


Google storage component doesn't work

I can't get the simple Google Storage route below to work in Camel K.
<routes id="google-storage-routes" xmlns="http://camel.apache.org/schema/spring">
    <route id="google-cloud-storage-route">
        <from uri="direct:google-cloud-storage-queue" />
        <convertBodyTo type="java.lang.String" />
        <to uri="google-storage:xxxxx" />
    </route>
</routes>

When run in Camel K using the command below, the deploy succeeds but an error is thrown since the google component isn't available.
kamel run google-storage-routes.xml --dev
...
[1] 2021-08-16 18:54:12,534 ERROR [org.apa.cam.qua.mai.CamelMainRuntime] (main) Failed to start application: org.apache.camel.FailedToCreateRouteException: Failed to create route google-cloud-storage-route at: >>> To[google-storage:xxxxxx] <<< in route: Route(google-cloud-storage-route)[From[direct:google-cloud-s... because of No endpoint could be found for: google-storage://xxxxx, please check your classpath contains the needed Camel component jar.

If I add the dependency, the deploy fails.
google-storage git:(master) ✗ kamel run -d camel-google-storage google-storage-routes.xml --dev
integration "google-storage-routes" created
Condition "IntegrationPlatformAvailable" is "True" for Integration google-storage-routes: default/camel-k
Progress: integration "google-storage-routes" in phase Initialization
Progress: integration "google-storage-routes" in phase Building Kit
Integration google-storage-routes in phase "Initialization"
Integration google-storage-routes in phase "Building Kit"
Condition "IntegrationKitAvailable" is "False" for Integration google-storage-routes: creating a new integration kit
Integration google-storage-routes subresource kit-c4dba6esrkndflf45f8g (Integration Kit) changed phase to "Build Submitted"
Integration google-storage-routes subresource kit-c4dba6esrkndflf45f8g (Build) changed phase to "Scheduling"
Integration google-storage-routes subresource kit-c4dba6esrkndflf45f8g (Build) changed phase to "Pending"
Integration google-storage-routes subresource kit-c4dba6esrkndflf45f8g (Build) changed phase to "Running"
Integration google-storage-routes subresource kit-c4dba6esrkndflf45f8g (Integration Kit) changed phase to "Build Running"
Integration google-storage-routes subresource kit-c4dba6esrkndflf45f8g (Build) changed phase to "Failed"

Why isn't Camel K automatically pulling the google-storage dependency, and why is it failing when the dependency is explicitly added? How should this be updated so that the google-storage component may be used within Camel K?

Hi @jeffgbradley2, I can't reproduce using the master branch. What version of camel-k are you using ? The latest release is v1.5.0, Thanks !

It's main branch now.
Il mar 17 ago 2021, 10:52 John Poth @.***> ha scritto:

Hi @jeffgbradley2 https://github.com/jeffgbradley2, I can't reproduce
using the master branch. What version of camel-k are you using ? The
latest release is v1.5.0
https://github.com/apache/camel-k/releases/tag/v1.5.0, Thanks !
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/apache/camel-k/issues/2576#issuecomment-900116051,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ABG6XV326QGM743U3RGPDQLT5IPOHANCNFSM5CIKEHUQ
.

We've tried with 1.4.0 and 1.5.0, same results.

I've also tested the same with 1.5.0 and it's working as expected. The only thing is that it requires a --build-property quarkus.google.cloud.project-id=test to startup.
The dependency should be detected automatically. In order to troubleshoot it you can execute kamel describe integration route and see if the dependency was indeed included. Also, you may have a look at the camel-k-operator pod log in order to check problems during the assembly phase. Finally, you may check the Build that is in charge to create a Kit via k describe builds kit-c4ec3jk3k6d8tsit3bvg


[Storage] Replace import * as Models with explicit re-export of model types

Blobs: https://github.com/Azure/azure-sdk-for-js/pull/5567
Files: https://github.com/Azure/azure-sdk-for-js/pull/5532
Queues: https://github.com/Azure/azure-sdk-for-js/pull/5534

This task is completed via the above mentioned PRs from @jeremymeng


Fix ja translations of daysOfTheWeek

The order of the days of week was wrong.
So I corrected it.

Thanks!


New React Libs with logic to pick environment specific endpoints

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):
Fix #283
Test URLs:

Before: https://main--sling--aemsites.aem.live/programming/sports
After: https://283-packages--sling--aemsites.aem.live/programming/sports
Before: https://main--sling--aemsites.aem.live/drafts/km/slinghome
After: https://283-packages--sling--aemsites.aem.live/drafts/km/slinghome

It looks good, but wondering if I should not be seeing content differences compared to the live site?

p.s. if I ever get another project with these kinds of complicated React components, I will come-a-calling for you :-D


[wip] flask-restplus upgrade to 0.10

Motivation for changes:
Native JSON schema support, latest stuff...
Currently inflicted by https://github.com/noirbizarre/flask-restplus/issues/194 which causes Swagger page to crash, tests and UI work

The Flask-RESTPlus issue has been closed.

The AttributeError exception is hiding the real error so each time it can be a different issue.
This won't be the case anymore and you will be able to either fix it if this is possible or report the real issue if necessary.

I know, waiting on The 0.11 release


Images gets corrupted if an array of image is uploaded to the server

@danial I am facing some problems while uploading an array of images to the server. As the upload procedure is async all my files becomes  corrupted as angular is sending one after another image to Web API before the first one's upload is completed... But when i upload only one image it works fine. do you have any suggestions?

@khaled4vokalz I believe you meant to tag @danialfarid

Fix your server code or upload the next file inside the success function of the previous file.
something like
function uploadFile(i) {
Upload.upload(...).then(function() {uploadFile(i + 1);});
}
uploadFile(0);


Link to Project site doesn't work

Description
The link to Project site is not clickable on some plugins
Steps/Code to Reproduce
Navigater to this page:
https://www.napari-hub.org/plugins/napari-workshop-browser
Click here:

Expected Results
I go to the Project page
Actual Results
Nothing happens

For all icons, none of the links in the dropdown menus work, including the project site link

@haesleinhuepf thank you for reporting this & @dgmccart thanks for poking at this some more! For me, all the links work, although some require clicking a 2nd time, which is not ideal.
@codemonkey800 could you look into this? If it's a light lift, I think it'd be worth taking on.

For me, all the links work, although some require clicking a 2nd time, which is not ideal.

Maybe this is a browser related issue. I'm working with Chrome on Windows. Clicking on the "Documentation" on this page does not work in any case.

I am using Chrome on a Mac and double-clicking doesn't fix the issue for me.

Thanks all for the info! I'll have the team look into this and hopefully work on a fix.

@codemonkey800 Just a heads up that I'm labeling this a P2/medium priority again given our bug prioritization definition. Even though it's a P2, it seemed worth doing an initial investigation into and whether there might be a quick fix available.  I know you haven't been able to reproduce yet but keep us posted on your findings.

can still extract the URL and manually navigate to it even if it's not clickable

Do you think this is a workaround common end-users are aware of?

can still extract the URL and manually navigate to it even if it's not clickable

Do you think this is a workaround common end-users are aware of?

@haesleinhuepf That's a good point - possibly not, but it is one component of how we prioritize bugs (vs. there being no possible workaround). The exact priority (low or medium) in this case isn't going to change the fact that we are investigating.
Given where we are with resourcing, in general we are currently only able to commit to taking on high or critical bugs that significantly impair site functionality, though that could change in the future and we appreciate the feedback and bug reports!

Hi @codemonkey800 ,
it works now. Awesome.
Thanks!
best,
Robert


SW-16564

Description
Please describe your pull request:

Why is it necessary?
If shop configuration not contains a newline (\n) in hostalias users get an error. For example if backend users try to login into an account from customer backend controller.
What does it improve?
This commit improve the where clause in query builder to get an shop by request.
Does it have side effects?
I think no.

Questions
Answers

BC breaks?
no

Tests pass?
yes

Related tickets?
https://issues.shopware.com/issues/SW-16564

How to test?
Enter an hostalias (which is different from host) without an ending newline (\n)

fix an error if you entered an hostalias which is different from host without an ending newline (\n)

We decided to close this PR because the change is incomplete and does not fix the underlying issue. If this change is still important to you, feel free to create a new pull request or create an issue on https://issues.shopware.com.
For more information about our contribution guidelines, see https://developers.shopware.com/contributing/contribution-guideline/.


[New Feature]: Add undo and restart functions

🎮 feature Request
If a player wants to reverse their position in a game to improve their score, they may need to  undo there position of the game
Point down the features
feature points
Select program in which you are contributing
GSSoC24
Code of Conduct

[X] I follow CONTRIBUTING GUIDELINE of this project.

assign me

@Tarungupta18 Please raise any other issue, i'll assign you
If you have any doubt you can directly send message on our discord (https://discord.gg/rZb46cCMmK), we have created separate channals for all projects...
Thank you :)


Automated cherry pick of #8575

Cherry pick of #8575 on cloud.
/cc  @michelengelen

/release-note-none


New command: add site collection app catalog

Add site collection app catalog in the specified site.

command: spo site appcatalog add -s|--siteUrl <siteUrl> --confirm
siteUrl: absolute URL of the site where the site collection app catalog should be added
confirm: suppress prompting for confirmation
requires connection to a SharePoint site

For more information see the Add-SPOSiteCollectionAppCatalog cmdlet in the November release of the SPO PowerShell.

Anyone working on this one? If not, can assign it to me (as well as the associated remove one one in #64)

They’re yours!

$0.02, suggest we ditch the --confirm prompt. If the MSFT provided POSH cmdlet doesn't require it, we don't need to introduce it.

Agreed @andrewconnell


Timestamp.fromDate using local time not UTC time

Environment details

OS: Any OS
Node.js version: v18.20.2
npm version: 10.5.0
@google-cloud/firestore version: 6.8.0

Steps to reproduce

Create a field with following value Timestamp.fromDate(new Date())
Check saved value in firestore console

Expected
If create a field at 12PM, we should can get 12PM UTC+{local timezone}
What really get
If create a field at 12PM, we get (12PM + {local timezone}) UTC+{local timezone}
How to fix it
Since date using local time and firestore require utc time as input, Remove timezone offset from the time
https://github.com/googleapis/nodejs-firestore/blob/6dbe4b0baac261f03f9032765b375938ce5e46d7/dev/src/timestamp.ts#L96
return Timestamp.fromMillis(date.getTime() + date.getTimezoneOffset() * 60 * 1000);

And it still not fixed in v7.7.0

Thank you for the bug report. I'll take a look into this and get back to you. My immediate thought, though, is that this would be a breaking change and requires a great deal of care. An alternative to modifying the behavior of fromMillis() may be to simply add a new method, something like fromMillisUTC().

Thank you for the bug report. I'll take a look into this and get back to you. My immediate thought, though, is that this would be a breaking change and requires a great deal of care. An alternative to modifying the behavior of fromMillis() may be to simply add a new method, something like fromMillisUTC().

Yes it can be mess up every old codes.
Currently, im making new fromLocalDate(date: Date)` and using it.
I hope there is no bug and it was just my misunderstand of the timestamp calculation.

I'm sorry, My devops team report me that the server was set to UTC not local time.
Should i close or waiting for you?

I can close it. Thanks for letting me know!


[DT-6] Add Local Activity to event history filter

Description & motivation 💭

Adds a Local Activity to

the Workflow Events filter in event History view
the Event Type filter in event Compact view

Clears the filter when switching from History view to Compact view if the filter option does not exist

Screenshots (if applicable) 📸

History view
Compact view

Design Considerations 🎨
Testing 🧪
How was this tested 👻

[x] Manual testing
[ ] E2E tests added
[x] Unit tests ~added~ updated

Steps for others to test: 🚶🏽♂🚶🏽♀

[ ] Verify the Local Activity option* works as expected in the

History view
Compact view

[ ] Verify LocalActivity events also show up when filtering for Marker events in the History view
[ ] Verify when changing from the History to Compact view

If the filter option exists it is persisted
If the filter option does not exist the filter is cleared

*A sample with LocalActivity events can be found here.
Checklists
Draft Checklist
Merge Checklist
Issue(s) closed

DT-6

Docs
Any docs updates needed?

Looks like you gotta update that unit test for those new options.


bus error

on my laptop running this code in IPython terminal or jupyter
from pathlib import Path
import mne

sample_dir = Path(mne.datasets.sample.data_path())
sample_fname = sample_dir / 'MEG' / 'sample' / 'sample_audvis_raw.fif'

raw = mne.io.read_raw_fif(sample_fname, preload=True)
raw.crop(tmax=4)

sfreq = raw.info['sfreq']
first_samp = 0
raw = mne.io.RawArray(raw.get_data(),
                      raw.info,
                      first_samp=first_samp)

raw.plot()

gives a bus error:
In [1]: %run debug_bus_error_qt_browser.py
[1]    14095 bus error  ipython --matplotlib=qt --no-banner

can someone replicate? I don't have any problem without the RawArray creation.

Is this on macOS?

Are you maybe out of memory? Is you disk full?

Lastly, is this a "fresh" conda environment?

no my disk it not full and it's not a fresh conda env. It's my old messy
one :)
but it suggests it's just me then?

Message ID: @.***>

but it suggests it's just me then?

Until now apparently yes. The issue is that the bug is very unspecific and I would suspect it has to do with some binary incompatibilities inside your "messy" environment… maybe. 😅

ok fair enough... i'll just close


Adding a CHANGELOG

Would it be possible to add a simple changelog so it will be easier to know what changed between versions?

Do you prefer a file in the sources called CHANGELOG or would it be enough to have a link in the README to the releases section of the github page?

either way, as long as we can see what has changed
If there is only one maintainer/version bumper, a releases section would suffice


fix(gboost) resolves #234 git check fail due to -v vs --version

Issue #, if available:
https://github.com/awslabs/green-boost/issues/234
Description of changes:
Trivially change git -v to git --version check to work reliably across different versions of git.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Thanks, @firxworx! Could you add a changeset? I'd do it but want to give you credit for the fix. You can run pnpm changeset in the root of the repo and follow the prompts.


[OZ][L-02]Bridge Can Be Reinitialized

Description:
The L1StandardBridge contains a guard condition to prevent it from being reinitialized.
However, it assumes the messenger will be non-zero after initialization, which is not
guaranteed by the initialize function. This means it is possible to initialize the other
variables and later overwrite them. In practice, the contract will be non-functional until the
messenger is set.
Nevertheless, in the interest of predictability, consider ensuring the messenger is non-zero
during initialization.
Recommendation:
Consider add the solidity modifier on messenger check

PR:https://github.com/mantlenetworkio/mantle/pull/1027


Update new endpoints and URL format

Removal of \ symbol on URL endpoints. Adding the new endpoints NPS Extension must have connectivity

@Justinha - can you review the proposed changes?
IMPORTANT: When the changes are ready for publication, add a #sign-off comment to signal that the PR is ready for the review team to merge.
#label:"aq-pr-triaged"
@MicrosoftDocs/public-repo-pr-review-team).

#sign-off


Add functions susceptible to AST stack overflow to calls blacklist

This adds the following calls to the blacklist:

ast.literal_eval
ast.parse
compile
dbm.dumb.open

I did not add eval and exec as they are already covered by B307 and B102, respectively, and I'm not sure if duplicating them would make sense.
References:

https://bugs.python.org/issue32758
https://github.com/python/cpython/pull/5960
https://github.com/python/cpython/pull/6043

I think you must create first the Issue.
However, I'm not sure if a DoS on the code must be a rule here.
Maybe @ericwb or @sigmavirus24 could help us to know if this is good new rule or not.

However, I'm not sure if a DoS on the code must be a rule here.

I'm not sure what you mean by this.

Maybe @ericwb or @sigmavirus24 could help us to know if this is good new rule or not.

I think it's a fair addition to the list of rules. I don't think it will cause too much pain for existing consumers (beyond other PyCQA projects). I do wonder, however, if this is project is starting to be treated less as a list of security vulnerabilities and more as a "Well this could be used in a highly specialized case" stick to hit people with.

@ehooo, created issue.

I do wonder, however, if this is project is starting to be treated less as a list of security vulnerabilities and more as a "Well this could be used in a highly specialized case" stick to hit people with.

@sigmavirus24, thanks for feedback. I can't comment on the intended scope of the project. I can see how this rule can introduce potential false positives, but that can be said about most other rules as well.
My biggest issue is with ast.literal_eval which is recommended as a safe(r) alternative to eval by both the official documentation and bandit itself, so I suspect it is commonly[citation needed] used for parsing untrusted data where some evaluation is needed, instead of using eval, which might cause a false sense of security. My goal was to bring attention to that.

I'm not sure what you mean by this.

@sigmavirus24 I mean that maybe is not importat add rule for denial of services caused by known bug on python

I mean that maybe is not importat add rule for denial of services caused by known bug on python

@ehooo then should we remove the checks for other standard libraries too? I think we do our best to warn folks about things in the language/standard library enough that this makes sense to add. That's just my 2 pence though. I think I definitely want to hear @ericwb's opinion on this as well at least.

@sigmavirus24  @ehooo,
updated PR to lower rule severity to low, added both eval() and exec() to the rule.

I think we also need to update the documentation for the existing eval() and exec() checks to explain that the recommendations are for safer but not safe methods.

Should this be covered in this PR?


Service improvements

Hi, first of all, thanks for creating this module! :) Saved me a bunch of time when integrating psad into my puppet agents.
However, I ran into some issues on Ubuntu (14.04 Server LTS) when requiring a restart of the psad service.
For example, when I manually shut down the psad service, running the puppet agent did not automatically restart it.
Also, when PSAD is already running and a configuration file has changed, thus requiring a restart, puppet would not do this correctly because on my system it uses /usr/sbin/psad restart, which outputs the following [*] /usr/sbin/psad process is already running! Exiting. (exit code 1), and so your config changes are not activated.
Last but not least, the default service psad status or psad -S (identical) is not the correct way to determine if psad is running, and it can take a long time to perform on a system which has been running for a while and as a result, slows down your puppet run times. I changed this by checking to see if the psad process is running or not and using the exit code of the grep command to determine success or failure. This is much quicker and safer way to determine if PSAD is running or not. PSAD's behaviour, by the way, also causes Ubuntu to think it is not running, while in fact it is. Try a service --status-all for example, and then a psad -S, the first will tell you it is not running, the second will tell you it is.
Maybe these issues are only related to Ubuntu, I have only tested this on Ubuntu Server 12.04 & 14.04 LTS (64-bit).
The changes I made to the service class fixed the above issues for me so hopefully you will merge them back into your master! :+1:

Awesome, thanks for the pull request!

The newest version is up on puppet forge. Thanks again for your contribution!

No problem! :)
I'm working on something else as well: installing via the downloadable tarball to circumvent the automatic dependency for postfix on Ubuntu systems. Once I get it working I'll create another pull request!
Regards,
Rinck

Cool, I like that plan. Try to make it optional though for people who still want to use their package managers.


Add Additional Fields (Issue #9)

Added Additional Fields to the final JSON object

This resolves Issue #9


Setting file path

I'm concerned of the proper way to add the image path ! I'm selecting the directory at first but the an error appears.
python3 main.py  Exception in Tkinter callback Traceback (most recent call last):   File "/usr/lib/python3.8/tkinter/__init__.py", line 1892, in __call__     return self.func(*args)   File "main.py", line 135, in scales_onchange     load_image()   File "main.py", line 102, in load_image     file_path = data_path + "/" + files[file_index] IndexError: list index out of range Exception in Tkinter callback Traceback (most recent call last):   File "/usr/lib/python3.8/tkinter/__init__.py", line 1892, in __call__     return self.func(*args)   File "main.py", line 135, in scales_onchange     load_image()   File "main.py", line 102, in load_image     file_path = data_path + "/" + files[file_index] IndexError: list index out of range Exception in Tkinter callback Traceback (most recent call last):   File "/usr/lib/python3.8/tkinter/__init__.py", line 1892, in __call__     return self.func(*args)   File "main.py", line 135, in scales_onchange     load_image()   File "main.py", line 102, in load_image     file_path = data_path + "/" + files[file_index] IndexError: list index out of range Exception in Tkinter callback Traceback (most recent call last):   File "/usr/lib/python3.8/tkinter/__init__.py", line 1892, in __call__     return self.func(*args)   File "main.py", line 135, in scales_onchange     load_image()   File "main.py", line 102, in load_image     file_path = data_path + "/" + files[file_index] IndexError: list index out of range Exception in Tkinter callback Traceback (most recent call last):   File "/usr/lib/python3.8/tkinter/__init__.py", line 1892, in __call__     return self.func(*args)   File "main.py", line 135, in scales_onchange     load_image()   File "main.py", line 102, in load_image     file_path = data_path + "/" + files[file_index] IndexError: list index out of range Loading:  /home/mhalwani/ProjectiveTransformation/image.png Traceback (most recent call last):   File "main.py", line 306, in <module>     load_image()   File "main.py", line 124, in load_image     raise e   File "main.py", line 118, in load_image     show_info(title + " Circles found: " + (0 if circles is None else str(len(circles[0, :])))) TypeError: can only concatenate str (not "int") to str

I made a change in main.py file, Check if its fixed!
This bug occures when selecting an empty folder!

The code is working now, but for some reason I'm not able to detect a single circle ! I attached my image, I want to detect the 8 blue circles in the 4 corners.
Not sure if its a bug

Dear Mohamad. You should configure parameters of Hough algorithm in top of window to be able to detect circles.


Picovoice Console Issue: Can't Sign in with Google

Make sure you have read the documentation, and have put forth a reasonable effort to find an existing answer.
Expected behaviour
Login with Google Credentials
Actual behaviour
Failed to fetch

Steps to reproduce the behaviour
I'm honestly a bit of a novice, so I'm not entirely sure. I was testing out uses of Leopard last night, running transcriptions and then manipulating the data. I was using the same audio file repeatedly, as a test, not sure if that might be related to the issue.
When I try to use it today I get back:
pvleopard.leopard.LeopardActivationError
So I assumed something changed with the activation key. But I'm unable to login and check.
Anyways, thanks for the awesome product! It is really going to speed up my video production speed.
(Include enough details so that the issue can be reproduced independently.)

Thanks for reposting this.
We are aware of the issue and sorry for the inconvenience, but it's out of our control as it is caused by AWS. I'll update you once the problem is completely fixed and Picovoice Console returns to normal.

Got it. Thanks for the update! It's helpful just to know what's going on. Here's hoping it's fixed soon.

Looks like it is fixed. Thanks!

You are welcome; everything seems reasonably stable now.


import.sql handling

fixed grammar and script parsing.  still need to test inclusion in create/drop handling.
marking draft for now

Good point, even though no one cares about Windows :laughing:


retrieve_possible_twitter_handles stopped returning Twitter search results

In this file:
WeVoteServer/import_export_twitter/controllers.py
...in the function retrieve_possible_twitter_handles the array search_results is often empty for some reason, even though we get results when you search on Twitter.com. I think some part of connecting with Twitter stopped working at some point in September 2020.
This is part of the Batch Process system.
You can execute this function from this page: http://localhost:8001/import_export_batches/batch_process_list/
...by clicking the "Start General Maintenance" link with Twitter Processing turned on.

This may be working as it should, but needs some investigation at a later date.


goog.structs.weak.Map and goog.structs.weak.Set do not exist

https://github.com/google/closure-library/blob/master/closure/goog/structs/weak/weak.js#L18-L20
 * The helpers in this file are used for the shim implementations of
 * {@code goog.structs.weak.Map} and {@code goog.structs.weak.Set}, for browsers
 * that do not support ECMAScript 6 native WeakMap and WeakSet.

This suggests that goog.structs.weak.Map and goog.structs.weak.Set exist, but they do not.

yes, this looks like the author lost interest after a rollback.
On Fri, May 27, 2016 at 2:56 PM, Paul Draper<EMAIL_ADDRESS>wrote:

https://github.com/google/closure-library/blob/master/closure/goog/structs/weak/weak.js#L18-L20

The helpers in this file are used for the shim implementations of
{@code goog.structs.weak.Map} and {@code goog.structs.weak.Set}, for browsers
that do not support ECMAScript 6 native WeakMap and WeakSet.

This suggests that goog.structs.weak.Map and goog.structs.weak.Set exist,
but they do not.
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/google/closure-library/issues/712, or mute the thread
https://github.com/notifications/unsubscribe/ABMDKpSVJ4x_LDHlFIg8m5QCe9dNx75dks5qF2iKgaJpZM4Io5OL
.

Closure compiler will be providing polyfills in the near future.  Given that it was never finished, can we just delete this file?

The 'weak' versions do not exist and it seems the Set and Map implementations in Closure are deprecated.
This need for this issue is likely obviated by the deprecation.

This file has been deleted (long ago).


xtensa-lx106-elf-gcc: not found

Hey Charles.
This has got me scratching my head.
I stood up a new ubuntu/trusty64 instance via vagrant, ssh'd in and apt-get installed all the usual build tools/libs and git. So i have a squeaky clean build environment, with everything i need.
From then, i follow your instructions, and everything seems to go well, gcc seems to build forever, and doesn't bomb out.
except when i run make on your project i get
/home/vagrant/esp8266/xtensa-toolchain-build/build-lx106/root/bin/xtensa-lx106-elf-gcc: not found
and indeed gcc is not in there, everything else is though:
vagrant@vagrant-ubuntu-trusty-64:~/esp8266/xtensa-toolchain-build/build-lx106/root/bin$ ll
total 67612
drwxr-xr-x 2 root root    4096 Dec 26 23:05 ./
drwxr-xr-x 5 root root    4096 Dec 26 23:05 ../
-rwxr-xr-x 1 root root 4185842 Dec 26 23:05 xtensa-lx106-elf-addr2line*
-rwxr-xr-x 2 root root 4357881 Dec 26 23:05 xtensa-lx106-elf-ar*
-rwxr-xr-x 2 root root 6012816 Dec 26 23:05 xtensa-lx106-elf-as*
-rwxr-xr-x 1 root root 4139582 Dec 26 23:05 xtensa-lx106-elf-c++filt*
-rwxr-xr-x 1 root root  108494 Dec 26 23:05 xtensa-lx106-elf-elfedit*
-rwxr-xr-x 1 root root 4740248 Dec 26 23:05 xtensa-lx106-elf-gprof*
-rwxr-xr-x 4 root root 5522640 Dec 26 23:05 xtensa-lx106-elf-ld*
-rwxr-xr-x 4 root root 5522640 Dec 26 23:05 xtensa-lx106-elf-ld.bfd*
-rwxr-xr-x 2 root root 4225431 Dec 26 23:05 xtensa-lx106-elf-nm*
-rwxr-xr-x 2 root root 5203846 Dec 26 23:05 xtensa-lx106-elf-objcopy*
-rwxr-xr-x 2 root root 5826235 Dec 26 23:05 xtensa-lx106-elf-objdump*
-rwxr-xr-x 2 root root 4357880 Dec 26 23:05 xtensa-lx106-elf-ranlib*
-rwxr-xr-x 1 root root 1414615 Dec 26 23:05 xtensa-lx106-elf-readelf*
-rwxr-xr-x 1 root root 4202808 Dec 26 23:05 xtensa-lx106-elf-size*
-rwxr-xr-x 1 root root 4173624 Dec 26 23:05 xtensa-lx106-elf-strings*
-rwxr-xr-x 2 root root 5203877 Dec 26 23:05 xtensa-lx106-elf-strip*

i've been through the loop twice on this, and cant see why gcc isn't getting built.
what did i miss??
edit: fixed markdown

I'm facing the same problem presented above.But unable to understand your proposed solution.
Can you please elaborate that?

Would you mind posting the full build (And error) log?

I met the same problem. It should be solved by add your "xtensa-lx106-elf-gcc/bin/" to PATH.
But I solved it like this (assume /usr/games/ in PATH, and if I'm wrong, easy to delete them):
sudo cp -r ~/xtensa-lx106-elf-gcc/bin/* /usr/games/


Performance overhead on HiveConnector::createDataSource if split preloading kicked in

Bug description
It's observed in Gluten's use case, HiveConnector::createDataSource slows down data scan when split preload is turned on.
In the case a hotspot appeared in filter expression compilation (namely SimpleExpressionEvaluator::compile):

In the case the filter expression contains bloom-filters so it took much longer time than usual since bloom-filter's compilation can be slower than other types of expressions.
In the case when split preloading is turned off, the scan time can be shorten by ~6x (~30s vs ~5s). The estimated total split number is ~200K.
Related code:
https://github.com/facebookincubator/velox/blob/3eb9f011f4292f6365203d61b5442d34cab92182/velox/exec/TableScan.cpp#L290-L329
To solve the issue, perhaps split-preloading procedure could adopt some kind of reuse logics to avoid compiling the expressions every time a split is preloaded.
System information
Velox System Info v0.0.2
Commit: b98b20414faf414775f506b6d446f314cb27fe38
CMake Version: 3.29.2
System: Linux-5.4.0-156-generic
Arch: x86_64
C++ Compiler: /usr/bin/c++
C++ Compiler Version: 9.4.0
C Compiler: /usr/bin/cc
C Compiler Version: 12.0.0
CMake Prefix Path: /usr/local;/usr;/;/opt/cmake;/usr/local;/usr/X11R6;/usr/pkg;/opt

cc @oerling @Yuhta @mbasmanova

Can you put the bloom filter in dynamic filters instead?  It does not sound right to put a bloom filter in remaining filter.

It does not sound right to put a bloom filter in remaining filter.

What's the ideal use case for remaining filter? I am not pretty much familiar with this part of design in Velox.

Where is the bloom filter generated?

It's generated by Spark query planner before creating Velox task.

Remaining filter is part of the where clause that cannot be converted to tuple domains (key value filters).  In your case it is probably easier to wrap the bloom filter in a Filter object and pass it using TableScan::addDynamicFilter before you kick off the task.

 Thanks.
template <typename T>
struct BloomFilterMightContainFunction {
  VELOX_DEFINE_FUNCTION_TYPES(T);

  using Allocator = std::allocator<uint64_t>;

  void initialize(
      const std::vector<TypePtr>& /*inputTypes*/,
      const core::QueryConfig&,
      const arg_type<Varbinary>* serialized,
      const arg_type<int64_t>*) {
    if (serialized != nullptr) {
      serialized_ = serialized->str();
    }
  }

  FOLLY_ALWAYS_INLINE void
  call(bool& result, const arg_type<Varbinary>&, const int64_t& input) {
    if (serialized_.has_value()) {
      bloomFilter_.merge(serialized_.value().c_str());
      serialized_ = std::nullopt;
    }
    result = bloomFilter_.isSet()
        ? bloomFilter_.mayContain(folly::hasher<int64_t>()(input))
        : false;
  }

 private:
  BloomFilter<Allocator> bloomFilter_;
  std::optional<std::string> serialized_;
};

If it is the same in all data sources, recompiling them is a waste of CPU

I see, make sense. Then how about store the DataSource created in preload threads in some collection and reuse them?
Thanks.

Reusing them will be tricky.  The most straightforward way would be wrap the bloom filter in filter object and push them into dynamic filters instead of modeling them as expression (it seems more logically right that way as well).  Otherwise keeping them as expression will be a fairly large change on connector to factor out the remaining filter compilation (the connector interface might be hard to change to accommodate this).  We are not seeing compilation cost anywhere else so it's a question whether it is worthing doing it just for spark bloom filter.


Add Code Climate

I see Netlify CMS is using a number of linters. Code Climate is free for open source projects! It's a great way to boost code quality. I've added it to a number of my own open source repos.

Hey, thanks for opening this issue.
Actually, we are not very happy with our current linters, and we'll probably change the way they are setup here at some point. However, I would rather not add another external tool at the moment to control code quality, specially because I know how easy is to ignore them as soon as they don't fit the project's workflow.
We're not going to add Code Climate at the moment but we might consider it in the future again.

David,
I totally understand. I'm happy to help out if/when you decide to hookup
Code Climate. Thanks for all of your open source contributions!
On Fri, Mar 31, 2017 at 4:49 PM, David Calavera<EMAIL_ADDRESS>wrote:

Hey, thanks for opening this issue.
Actually, we are not very happy with our current linters, and we'll
probably change the way they are setup here at some point. However, I would
rather not add another external tool at the moment to control code quality,
specially because I know how easy is to ignore them as soon as they don't
fit the project's workflow.
We're not going to add Code Climate at the moment but we might consider it
in the future again.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/netlify/netlify-cms/issues/335#issuecomment-290826085,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AGJa98UYocv3pME4taJxoO5LxDPa4V6fks5rrWbIgaJpZM4MwGaK
.

--
Kevin McCormack
Full Stack Web Developer
HarlemSquirrel.github.io
917-209-0326


Splitting the command by spaces renders an invalid/broken command

This fails, at least on my windows machine:
var forever = require('forever-monitor');
var child = forever.start('server.js');

the result was:
events.js:72
        throw er; // Unhandled 'error' event
              ^
Error: spawn ENOENT
    at errnoException (child_process.js:980:11)
    at Process.ChildProcess._handle.onexit (child_process.js:771:34)

Debugging the problem yielded the following (excerpt from monitor.js - trySpawn()):
Monitor.prototype.trySpawn = function () {
  var command, cmdarr, stats, args;

  if (/[^\w]node$/.test(this.command) && this.checkFile && !this.childExists) {
    try {
      stats = fs.statSync(this.args[0]);
      this.childExists = true;
    }
    catch (ex) {
      return false;
    }
  }

  this.spawnWith.cwd = this.cwd || this.spawnWith.cwd;
  this.spawnWith.env = this._getEnv();

  if (this.stdio) {
    this.spawnWith.stdio = this.stdio;
  }

_at this point, command === "C:\Program Files\nodejs\node.exe"_
  cmdarr  = this.command.trim().split(/\s+/);
  command = this.command;

_now its "C:\Program"_
  args    = this.args;

  if (cmdarr.length > 1) {
    command = cmdarr[0];
    args = cmdarr.slice(1).concat(this.args);
  }

  if (this.fork) {
    if (!this.stdio) {
      this.spawnWith.stdio = [ 'pipe', 'pipe', 'pipe', 'ipc' ];
    }
    return spawn(command, args, this.spawnWith);
  }

  return spawn(command, args, this.spawnWith);
};

I'm not entirely sure if it is something that needs fixing (would love a comment on that).
In the meanwhile my workaround was to start with the "command" option:
var forever = require('forever-monitor');
var child = forever.start('server.js', { command: 'node' });

Relying on the fact that node is in my environment path

I have the same problem with 2.0.0, why is this issue closed?
I have my node binary installed in /Users/jiangzhu/work/Genie/Local Forms Testing/node-macos/bin
and have a monitor.js script to monitor adapter.js, both in /Users/jiangzhu/work/Genie/Local Forms Testing/node-macos/dist folder.
Current working directory is /Users/jiangzhu/work/Genie/Local Forms Testing/node-macos/dist/adapter, when I run ../bin/node monitor.js, I get the error below which caused by incorrect spawnargs
Starting monitor for file /Users/jiangzhu/work/Genie/Local Forms Testing/node-macos/dist/adapter.js
events.js:288
      throw er; // Unhandled 'error' event
      ^

Error: spawn /Users/jiangzhu/work/Genie/Local ENOENT
    at Process.ChildProcess._handle.onexit (internal/child_process.js:267:19)
    at onErrorNT (internal/child_process.js:469:16)
    at processTicksAndRejections (internal/process/task_queues.js:84:21)
Emitted 'error' event on ChildProcess instance at:
    at Process.ChildProcess._handle.onexit (internal/child_process.js:273:12)
    at onErrorNT (internal/child_process.js:469:16)
    at processTicksAndRejections (internal/process/task_queues.js:84:21) {
  errno: 'ENOENT',
  code: 'ENOENT',
  syscall: 'spawn /Users/jiangzhu/work/Genie/Local',
  path: '/Users/jiangzhu/work/Genie/Local',
  spawnargs: [
    'Forms',
    'Testing/node-macos/bin/node',
    '/Users/jiangzhu/work/Genie/Local Forms Testing/node-macos/dist/adapter.js'
  ]
}


RangeError: Maximum call stack size exceeded when a .env file has environment  variable with default values

Describe the bug
if you have a .env file in your project and this has  environment  variable with default values, you will get this error
⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯ Unhandled Error ⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯⎯
RangeError: Maximum call stack size exceeded
    at RegExp.exec (<anonymous>)
    at file://test-app/node_modules/vite/dist/node/chunks/dep-e8f070e8.js:36830:54
    at Array.reduce (<anonymous>)
    at _interpolate (file://test-app/node_modules/vite/dist/node/chunks/dep-e8f070e8.js:36829:18)
    at file://test-app/node_modules/vite/dist/node/chunks/dep-e8f070e8.js:36863:15
    at Array.reduce (<anonymous>)
    at _interpolate (file://test-app/node_modules/vite/dist/node/chunks/dep-e8f070e8.js:36829:18)
    at file://test-app/node_modules/vite/dist/node/chunks/dep-e8f070e8.js:36863:15
    at Array.reduce (<anonymous>)
    at _interpolate (file://test-app/node_modules/vite/dist/node/chunks/dep-e8f070e8.js:36829:18)

this only happened to me in Linux and my ci with docker, this does not happen on windows (my main machine)
Reproduction

create a empty project npm init -y
install vitest
create a .env file with this content

MY_SECRET=${MY_SECRET:-}

run npx vitest run

System Info
System:
    OS: Linux 5.15 Ubuntu 22.04.2 LTS 22.04.2 LTS (Jammy Jellyfish)
    CPU: (12) x64 AMD Ryzen 5 5600X 6-Core Processor
    Memory: 29.10 GB / 31.31 GB
    Container: Yes
    Shell: 5.1.16 - /bin/bash
  Binaries:
    Node: 18.16.0 - ~/.nvm/versions/node/v18.16.0/bin/node
    Yarn: 1.22.19 - /mnt/d/programas/nodejs/yarn
    npm: 9.5.1 - ~/.nvm/versions/node/v18.16.0/bin/npm
  npmPackages:
    vitest: ^0.31.2 => 0.31.2

Used Package Manager
npm
Validations

[X] Follow our Code of Conduct
[X] Read the Contributing Guidelines.
[X] Read the docs.
[X] Check that there isn't already an issue that reports the same bug to avoid creating a duplicate.
[X] Check that this is a concrete bug. For Q&A open a GitHub Discussion or join our Discord Chat Server.
[X] The provided reproduction is a minimal reproducible example of the bug.

This looks like an issue with Vite. I recommend opening an issue in their repo.

This looks like an issue with Vite. I recommend opening an issue in their repo.

in my particular case, i am not using Vite or installed it, i just installed vitest, is this still applies?

This looks like an issue with Vite. I recommend opening an issue in their repo.

in my particular case, i am not using Vite or installed it, i just installed vitest, is this still applies?

Yes, Vitest uses Vite internally. It's basically a wrapper around Vite.

This looks like an issue with Vite. I recommend opening an issue in their repo.

in my particular case, i am not using Vite or installed it, i just installed vitest, is this still applies?

Yes, Vitest uses Vite internally. It's basically a wrapper around Vite.

thanks, created ticket on : https://github.com/vitejs/vite/issues/13399


Setup/Documentation headache

For a bundle with the name "simplebus" its setup is far from simple.
Having to navigate through 5 different sites of documentation, jumping back to the github repo to find the documentation link each time is a pain.
It also lacks a "general" overview of what steps are required to setup all the different components.
Wouldn't it be better to merge all the documentation together on 1 site?
/edit: Just noticed an issue here with the link to the new documentation. Guess this can be closed.

Hi @samvdb thanks for taking the time to open this issues. We are aware of this and we are working on it but it is taking some time to complete;

https://github.com/SimpleBus/docs


Add SuccessMessage, ValidationFailures, and RequestToken to 'Request' category in template

These variables:

are currently only on the Target in the built in templates that are expecting a form submission such as Change password, etc. However, it would be immensely helpful to have these available in all templates if people are building custom form submissions from their own templates.

Task:

Add these to the Request category also.

Rather than be so rigid, we're providing the ViewData object in general.

Now the developer can do whatever they want.


Testing data

Emil Hvitfeldt as a smol package that has the coordinates of roundabouts across the world. We can use this to query OSM and check how this package works for matching on roundabouts which I suspect is a weak point
https://github.com/EmilHvitfeldt/roundabouts?tab=readme-ov-file

Good! That sounds good to me..


Insecure HTTP download links on OrientDB website

When downloading OrientDB from the following link, the download links are all links to bit.ly but use HTTP instead of HTTPS. This means that, in principle, an attacker could intercept the request to e.g. http://bit.ly/orientdb-ce-2-2-33 and produce a malicious executable instead of the user's desired OrientDB version.
This is made even more problematic by the fact that the link immediately downloads a file, and does not lead to a website where the user can notice the lack of an "HTTPS padlock" in their browser's address bar.
Since bit.ly supports HTTPS, I think all those links can be replaced with corresponding HTTPS links, for example: https://bit.ly/orientdb-ce-2-2-33.

Fixed, thanks!
Luigi


fix: update to eye v10.5.9

fix: update to eye v10.5.9

:tada: This PR is included in version 16.2.10 :tada:
The release is available on:

npm package (@latest dist-tag)
GitHub release
v16.2.10

Your semantic-release bot :package::rocket:


Add and_then, or_else, transform, and transform_error.

https://en.cppreference.com/w/cpp/utility/expected in C++23 will have those four member functions. We should have them in this expected library too.

thanks for the heads-up.

See proposal p2505.

Implemented by @szaszm in #60.


Create Post “2020-05-20-need-a-dynamic-web-app-use-a-static-site-generator-1”

Automatically generated by Netlify CMS

Added content


Issue launching "make test" to run OpenLane

I'm getting the following error message. Do I need to reinstall Docker?

Thanks

@ishfaqahmed29
From 2nd screenshot make pdk not completed. Try with sudo make pdk
Then try sudo make test or an other way try following command:
make mount
./flow.tcl -design spm

Hi @vijayank88
Linux, 20.04.1-Ubuntu: Python 3.8.10: Docker version 20.10.14, build a224086349

I tried sudo make pdk, and then sudo make test

@ishfaqahmed29
You have permission issue.
Just install OpenLane where user have read/write access.
After successfull installation try
make mount
./flow.tcl -design spm


Make smbus_read_i2c_block_data() write into &mut Vec/[u8] rather than returning new Vec?

It may be worthwhile for smbus_read_i2c_block_data() and similar functions to write data into a preallocated buffer (of fixed size &mut [u8], or allowing the function to resize a &mut [u8]), rather than returning a new Vec every time. This would save allocations, but would probably be an insignificant micro-optimization (unless you were running in a no-alloc environment or similar).

I took a look on how to implement this for the i2c_smbus_read_i2c_block_data function in src/ffi.rs and I came up with this:
pub fn i2c_smbus_read_i2c_block_data_inplace(
    fd: RawFd,
    register: u8,
    block: &mut [u8],
) -> Result<usize, I2CError> {
    let mut data = i2c_smbus_data::empty();
    unsafe {
        i2c_smbus_access(
            fd,
            I2CSMBusReadWrite::I2C_SMBUS_READ,
            register,
            I2CSMBusSize::I2C_SMBUS_BLOCK_DATA,
            &mut data,
        )?;
    }

    let bytes_available = data.block[0] as usize;

    let bytes_written = if bytes_available >= block.len() {
        // since this is a &mut [u8], we cannot resize it
        // and we just override the entire container
        block.len()
    } else {
        // otherwise, we write the amount of bytes fetched from the line
        bytes_available
    };

    // zip allows us to be bounded by the shortest iterator,
    // therefore we will never overrun the buffer
    data.block[0..bytes_available]
        .iter()
        .zip(block.into_iter())
        .for_each(|(source, destination)| {
            *destination = *source;
        });

    // similar to the `std::io::Write::read` method
    Ok(bytes_written)
}

If the caller provides a &mut [u8], then it should be assumed that the starting position of writing is at the beginning and no resizing can be done.
This would still allow for a no-alloc environment, as both the i2c_smbus_data and block would have a known size at compile time.


[Feature] Docker Image

As containerization technologies such as docker became very popular in the past few years,  it would be very useful and powerful to provide a docker image.
This would fulfill a lot of use cases. some of them are:

empowering people who are not familiar with Node (or do not have it installed) to run the application support center on a multitude of container orchestration platforms (Kubernetes, Docker Swarm, Hashicorp Nomad, Apache Mezos  to name few)
a docker-compose.yml file can be used to try and test the application for users who do not have Node or Postgres installed
providing a standard isolated environment (same Node version, same OS Distro...)  to deploy the application that is common between most of the users
high availability and horizontally scaling the application support center with replica sets to handle more traffic would become much easier

The Docker image can be hosted on GitHub Container Registry (Or Dockerhub, or both) and the building and testing process can be automated using a CI/CD pipeline with GitHub Actions.
I can work on and implement this feature if that's fine by everyone.

Hi @samerbahri98 - sure, working on this would be great, please feel free to submit a pull request when you ready.


[Checkbox] Modifying the color of the checkboxes per row (including its nested ones)

What package is this for?

React

Summary

I'd like to be able to change the color of each parent row's checkbox (along with its nested rows' checkboxes) to be the same color. Currently this is not possible since

The table does not provide a flag for each row along with its nested rows to indicate which nested row belongs to which parent
the data-parent-row is true on nested child rows as well

Specific timeline issues / requests
It's related to an upcoming release, it's crucial to be implemented soon.
This is the use case I need to implement.

Carbon add-ons for Watson IoT is a collaborative system. We encourage teams to
build components and submit them for integration.

This needs to go through design approval before coming to development. Please reach out to your designer, and they will work with @dianatran18


elasticsearch::instance / ensure => absent will fail in second puppetrun.

If you set a  elasticsearch::instance to ensure => absent, it will remove it correctly but will in the second puppet run complain, that it cannot remove it because it is not there :-/
rgds,
j.

Hi,
Thank you very much for reporting it.
Would you be able to pastebin/gist the output of the second run so i can see which parts of it fail?
Thanks!

Hi,
I've done several tests with the latest release and didnt see this issue.
Could you verify?

Closing due to lack of response. Please open a new issue if this is still a problem. Thank you.


Add input column to define type of nanopore/pacbio sequencing

This is used to set the mode parameter in the FLYE module. Right now it assumes the longread sequences are new high quality sequences.

We have decided to just use the low quality setting for now and perhaps consider OC results when choosing in the future rather than having the user provide that info.


[T4A2][W14-A1]Lim Jie

Ready for Review

Some comments added. Feel free to add more comment if you have any questions. Once you are done, you may close the PR after reading comments.


Graph definition forces input to have batchsize number of images

Suppose you want to restore the trained/saved model and test new inputs on. The way the model is written makes it inflexible to the number of inputs passed in, you must always pass in a number of examples equal to the batchsize the model was trained on. You can't for example run the model on just one test image, which would be nice.

I fixed this by changing the following lines:
        samples = tf.random_normal([tf.shape(self.images)[0], self.n_z], 0, 1, dtype=tf.float32)
        generated_flat = tf.reshape(self.generated_images, [tf.shape(self.images)[0], 28*28])

Fixed this by replacing self.batchsize in the model definition with tf.shape(self.images)[0] and passing an extra argument to conv_transpose():
    def __init__(self):
        self.writer = tf.summary.FileWriter('./log')

        self.mnist = input_data.read_data_sets("MNIST_data/", one_hot=True)
        self.n_samples = self.mnist.train.num_examples

        self.n_hidden = 500
        self.n_z = 20
        self.batchsize = 100

        self.images = tf.placeholder(tf.float32, [None, 784], name='input')
        image_matrix = tf.reshape(self.images,[-1, 28, 28, 1], name='reshaped_input')
        z_mean, z_stddev = self.recognition(image_matrix)

        # Sample from Gaussian distribution
        samples = tf.random_normal([tf.shape(self.images)[0], self.n_z], 0, 1, dtype=tf.float32)
        guessed_z = z_mean + (z_stddev * samples)

        self.generated_images = self.generation(guessed_z)
        generated_flat = tf.reshape(self.generated_images, [tf.shape(self.images)[0], 28*28])

        self.generation_loss = -tf.reduce_sum(self.images * tf.log(1e-6 + generated_flat) + (1-self.images) * tf.log(1e-6 + 1 - generated_flat),1)

        self.latent_loss = 0.5 * tf.reduce_sum(tf.square(z_mean) + tf.square(z_stddev) - tf.log(tf.square(z_stddev)) - 1,1)
        self.cost = tf.reduce_mean(self.generation_loss + self.latent_loss)
        self.optimizer = tf.train.AdamOptimizer(0.001).minimize(self.cost)

    # encoder
    def recognition(self, input_images):
        with tf.variable_scope("recognition"):
            h1 = lrelu(conv2d(input_images, 1, 16, "d_h1")) # 28x28x1 -> 14x14x16
            h2 = lrelu(conv2d(h1, 16, 32, "d_h2")) # 14x14x16 -> 7x7x32
            h2_flat = tf.reshape(h2,[tf.shape(self.images)[0], 7*7*32])

            w_mean = dense(h2_flat, 7*7*32, self.n_z, "w_mean")
            w_stddev = dense(h2_flat, 7*7*32, self.n_z, "w_stddev")

        return w_mean, w_stddev

    # decoder
    def generation(self, z):
        with tf.variable_scope("generation"):
            z_develop = dense(z, self.n_z, 7*7*32, scope='z_matrix')
            z_matrix = tf.nn.relu(tf.reshape(z_develop, [tf.shape(self.images)[0], 7, 7, 32]))
            h1 = tf.nn.relu(conv_transpose(z_matrix, 32, [tf.shape(self.images)[0], 14, 14, 16], name="g_h1"))
            h2 = conv_transpose(h1, 16, [tf.shape(self.images)[0], 28, 28, 1], name="g_h2")
            h2 = tf.nn.sigmoid(h2)

        return h2

def conv_transpose(x, prev_size, outputShape, name):
    with tf.variable_scope(name):
        # h, w, out, in
        w = tf.get_variable("w",[5,5, outputShape[-1], prev_size], initializer=tf.truncated_normal_initializer(stddev=0.02))
        b = tf.get_variable("b",[outputShape[-1]], initializer=tf.constant_initializer(0.0))
        convt = tf.nn.conv2d_transpose(x, w, output_shape=tf.stack(outputShape), strides=[1,2,2,1])
        return convt


aurelia-i18n and checkbox translation not working

I've tried translating checkbox label using what the i18n API suggested:
<md-checkbox t="rememberMe" md-checked.bind="rememberMe">R</md-checkbox>

the string "Remember me" gets translated but the widget losts its input and label elements resulting in such HTML fragment:
<md-checkbox t="rememberMe" md-checked.bind="rememberMe"
             class="au-target" au-target-id="97">Remember me</md-checkbox>

This is because i18n.updateValue(), if attr == 'text', runs through this code:
     node._newChild = newChild;
     while (node.firstChild) {
          node.removeChild(node.firstChild);
     }

That removes from the generated md-checkbox content the input and label elements.
I've tested this solution that however it's probably far from optimal: checkbox.html should use a named slot:
<template>
  <input type="checkbox" id="${controlId}" ref="checkbox" blur.trigger="blur()" />

    <slot name="label"></slot>

</template>

then its usage could be:
<md-checkbox md-checked.bind="rememberMe">RS</md-checkbox>

With my actual knowledge I can't say where the fix should go, in checkbox.html in i18n.js or in both. The problem here is that the actual format for inserting checkbox stops working, so the following results in an empty label:
<md-checkbox md-checked.bind="rememberMe">Remember me</md-checkbox>

while (node.firstChild) {
  node.removeChild(node.firstChild);
}

If node is the md-checkbox element, why would that not also remove the slot?
I'm currently setting up a skeleton with which I can try this. If you already have anything I could clone then this would help. :smiley:

In fact removing the label it removes also the slot it contains and substitutes everything with a text element created here:
newChild = _aureliaPal.DOM.createTextNode(this.tr(key, params));

About skeleton: what do you think if I try to add a i18n section to the component browser? So in the future one can post non working i18n fragment that can be dropped into that section.

About skeleton: what do you think if I try to add a i18n section to the component browser?

I like the idea. :+1:
I just think because it doesn't work it wouldn't help people much. :smiley:
An i18n section would be great, anyway. So, yes.
Also probably worth trying to put i18n into a gist.run..
 Because...
<md-checkbox md-checked.bind="rememberMe">R</md-checkbox>

Seems to work. :smiley:

What do you think about translating the whole catalog? I mean in principle, not that it's necessary you yourself who does it (although, of course, I'd appreciate this). :smiley:
I can provide German translation and I could probably get Dutch. Maybe Russian, too..

Ok the snippet you've posted is working, I haven't tried it. However it's tricky, I mean without translation I can remove the span element with translation I cannot translate the md-checkbox content but add a span and translate it. Should probably ask to the i18n guys if this is an intendent use case.
But ok I'll close this issue, we will speak again about it when a translated section of the component catalog will be in place.

I guess they could have used textContent or innerText but I know that this is much slower (at least innerText on IE).

After a quick look at the component browser I think I could

add the language selector beside the color-switcher
modify app.js and main.js to load i18n and set default language to english
add some working examples for instance label, input and buttons. For each component it's probably advisable to add a section aurelia-i18n integration.
into the aurelia-i18n integration one can writes working and not working examples.
what do you think?

Good idea on the language selector. :+1:
So, this means I can see i18n in action by switching to Italian and in each sample there's a section "i18n" which shows the implementation for the current component?
This is good.
The thing is this (not really a problem, therefore "thing"):

The component browser shows the actual source of the current page (although not all examples are migrated to this format).
That means, i18n is already in each sample.
Which does not mean it's bad. :smiley:

The i18n section could still carry some more information about how to achieve i18n.
I don't want the samples to be too "overwhelming" but when I get this right, actual usage  of i18n is just an additional attribute, so that's easy.
TLDR: Let's try what it looks like.. :smiley:


Snapshot save location as an Environment Variable

I first wanted to say thank you for the amazing library! We love using it in our project 😃
We want to utilize Test Plans to test our app against English and Spanish locales. We have snapshots recorded for the English variant of our app, but were unclear about how we can save Spanish snapshots side-by-side with our existing snapshots.
A solution that came to mind was, much like how we can alter the destination of snapshot diffs via the SNAPSHOT_ARTIFACTS env variable, we can allow a user to define an environment variable for the destination of the recorded snapshots. This would allow us to declare something like
// English Test Plan Configuration
SNAPSHOTS_DIR="__English_Snapshots__"

// Spanish Test Plan Configuration
SNAPSHOTS_DIR="__Spanish_Snapshots__"

and allow us to assert against the appropriate snapshots per test plan configuration.
I was wondering if this has already been discussed and is documented somewhere or if this would be a new feature altogether.

It looks like I may have spoke too soon. There's already a PR open with the aforementioned functionality to support Test Plans

I don't understand the problem, just add the localizations as test plan configurations and the locale identifiers will be added to all filenames automatically...


Improved coverage configuration

Maybe you could explain in the commit message exactly how things are improved? (maybe it's self-explanatory to someone more familiar with coverage)

Done. Maybe somebody else with coverage experience can have a look, too.


Lost HTTP Header Attribute

From @gaborposz on Friday, August 30, 2019 8:02:30 AM
Hello,
During development we have encountered a very strange issue regarding HTTP Headers.
We use WinHttpHandler on the client, and Kester (with self-hosting) on the backend for HTTP/2 communication, and we put some custom attributes into the HTTP header to transfer data together with the HTTP requests.
We noticed that occasionally -after several requests with specific header sizes- our custom header attribute is completely missing on the server side, although it was seemingly sent successfully by the client. It’s still unclear whether the error is on client side (WinHttpHandler) or on server side (Kestrel), but it only happens if HTTP 2.0 protocol is used with secure communication (HTTPS).
I have created a simple project to reproduce this error, the code is pretty straightforward, but let me know if you have any question.
Please have a look, and give us feedback if it can be fixed, or workaround is possible.
MissingCustomHeaderReproduce.zip
Thanks and Best Regards,
Gabor Posz
Copied from original issue: dotnet/corefx#40700

From @davidsh on Friday, August 30, 2019 1:21:19 PM
What version of .NET Core are you using?  Output of "dotnet --info" ?
Why are you using WinHttpHandler?  You should consider using .NET Core.  As of .NET Core 3.0, HttpClientHandler (which uses SocketsHttpHandler) supports HTTP/2 protocol.

From @gaborposz on Friday, August 30, 2019 2:32:11 PM
We are using WinHttpHandler because our client is .NET Framework 4.7 based (it must be, because it is used by other .NET Framework components), and this is the only way to support HTTP/2 with .NET Framework.
Only our backend service is .NET Core based. We use the .NET Core 2.2.3 Runtime (Microsoft.NETCore.App 2.2.3) in production environment, and .NET Core 2.2.105 SDK for development.

From @davidsh on Friday, August 30, 2019 3:56:22 PM

our custom header attribute is completely missing on the server side, although it was seemingly sent successfully by the client.

How do you know this?  Did you do a Wireshark trace? Although with HTTP/2 normally over TLS that would be difficult.

It’s still unclear whether the error is on client side (WinHttpHandler) or on server side (Kestrel)

Can you try another server (such as IIS) to determine whether it is really a client-side or server-side problem?

From @gaborposz on Friday, August 30, 2019 4:20:38 PM
I tried with Wireshark, but with secure communication it is not visible, and with disabled HTTPS the error is not reproducible.
I might try with IIS, but I'm not familiar with that, so it would take more time.
Isn't there an easier way to find out whether it's the client or the server? E.g.: Dump out the raw data by WinHttpHandler or Kester before/after the encryption?

From @gaborposz on Monday, September 2, 2019 2:38:28 PM
Hi,
After some struggle I eventually managed to try it with IIS. I tested it with an ASP.NET (.NET Framework) as well as an ASP.NET Core handler, and both of them was working fine. It indicates that the problem is most probably in the self hosted Kestrel.

From @davidsh on Tuesday, September 3, 2019 4:21:12 PM
Seems like this issue should be moved to one of the ASP.NET repos.
cc: @Tratcher

Can you re-test against 3.0 preivew9 (out soon). This may be the same issue as https://github.com/aspnet/AspNetCore/pull/12782.
Another way to get a trace for this is using Kestrel's connection logger.
https://github.com/aspnet/AspNetCore/blob/2217be28261ee2df07dd5d293d92a05a458f5a6b/src/Servers/Kestrel/samples/SampleApp/Startup.cs#L108-L112
Be sure to turn your log levels up to Trace.

Hi,
I managed to try it with .NET Core 3 preview 9 and the error is gone, so it seems to be indeed the same as #12782.
However, we would need the solution in .NET Core 2.2, because upgrading to 3.0 is not feasible within the available time frame. Can you merge it back to 2.2?

@shirhatti can you check on this 2.2 patch request? I know 2.2 will go out of support by the end of the year. https://dotnet.microsoft.com/platform/support/policy/dotnet-core
A workaround I haven't tested would be to reduce kestrelOptions.Limits.Http2.HeaderTableSize to 1 to effectively disable request header compression.
https://github.com/aspnet/AspNetCore/blob/ac9a6f331e3976e6ea170bec5a6964d080400c16/src/Servers/Kestrel/Core/src/Http2Limits.cs#L47

Hi,
The workaround seems to be working, but we would rather upgrade to new 2.2 patch version if possible.
Could you please give me feedback if the back merge to 2.2 is possible?
Thanks,
Gabor

@gaborposz We'll check, but having a simple workaround usually disqualifies issuing a patch. Even more so when the fix is available in a later release (3.0).

Agreed with @Tratcher here. We won't be backporting this change to 2.2 as it has a viable workaround and is fixed in the (imminent) 3.0 release.

@anurse I'm sad to hear it :(
We will use the workaround then until we can upgrade to 3.0 (or the 3.1 LTS).
@Tratcher: What is the impact of the workaround? Can we expect e.g. decreased performance, or is it negligible?

The client won't be able to use dynamic header compression making requests slightly larger, but you would only see a performance difference at extreme loads.

@gaborposz Sounds good, we're happy to help get the workaround in place. Backporting always comes with a bunch of known risks and some unknown ones too, and at the moment we don't believe the impact to be worth those risks given that it's fixed in 3.0. Thanks for your understanding here! Feel free to let us know if there are issues that block adoption of 3.0 (to get the fix) or implementation of the workaround!
We just released a release candidate of 3.0 which is extremely close to the final shipping version if you'd like to start investigating an upgrade now.


image create and edit pages issues

[x] 1. Users are allowed to create an image without an URL
- the user can still create an image using an empty space value, the UI should add string trim and not empty validation.

[x] 2. Users are not able to add any other labels when editing an image
-  failed to edit the image Error: the server could not find the requested resource, the API URL is incorrect you should get the namespace from the medata.namesapce instead of using fix harvester-system ns.

[x] 3. It's not very comfortable that the image name can be super long, it could make the status icon too close to the Overview button
- set image name length to maximum 63 characters

[x] 4. Annotation format on the edit page, not like the label format

[x] 5. Maybe the image names are duplicate?
- remove the second one, move the message to the end and only show it when it has an error message

[x] 6. On the image edit page, the copy tooltip sometimes overlaps the image name (move the mouse from right to the copy button)

[x] 7. Edit as YAML, what is the generateName for, every image has the same name image- [this is not bug]

[x] 8. Edit as YAML, the background color is necessary for the Unified and split buttons?

Validated


Run BiDi tests from wpt in CI

@sadym-chromium this PR now is now in a state where you could try to add the command line arguments to make ./wpt run use the chromium-bidi implementation. The path to anything in this repo should be ../chromium-bidi/bla when references from the ./wpt run step.


[FLINK-35890][cdc] Select only the increment but still enter the SNAPSHOT lead to ckp failed

ISSUE
Hi, when I was using Flink CDC Postgre, I chose 'debezium.snapshot.mode' = 'never' to skip the snapshot phase, but the code still entered the snapshot branch, and this log was printed, 'Database snapshot phase can 't perform checkpoint, acquired Checkpoint lock.'
(Since our business creates the task first and then writes the data), the task cannot perform ckp, and the status is always stuck in in progress. After the timeout, the task restarts.
Normally, if you only choose incremental data synchronization, you should not enter this branch.
The failure of ckp does not mean that the first one fails. After multiple ckp are successfully performed, it will then get stuck, causing a timeout and restarting the task. This is a problem that must occur. After it appeared in production, it also recurred many times in local testing.
Then we test, if there is data flowing in, the task status will be normal.
sql and conf like this

Why do this？
I think this miss one case，when I  set "snapshot = nerver",  I don't want to enter DB snapshot phase, and it can resolve my problem.

Thanks @leosanqing for the contribution, could you also check other connectors? I think some start up mode of debezium should not pass to underlying debeziunm engine once the Flink CDC incremental snapshot mode enabled.


User Survey plugin test cases

Will review it this week. Sorry, trying to catch up on system console work.

Great work! Lot of great cases bringing clarity to requirements.


Transform UnipipeServiceBroker to Stateless Architecture

Our current Unipipe-service-broker solution needs a local storage to store the temporary git changes and then it merges(pushes) them into the remote repo. This cycle makes our OSB eventually consistent.
This solution does not handle temporary storages well enough and if we want to use Serverless services like (GCP Cloud Run) we should improve how our OSB works on the backend side.
The serverless approaches are so fast to initialize and it is also so cheap because it only costs when it is active.
We should consider a way to use cloud storages or some other solution.

Additional Info: The CloudRun service sends a SIGTERM signal before shutting down the containers. Our application can handle these signals to gracefully shutdown itself.
https://cloud.google.com/run/docs/reference/container-contract#instance-shutdown


Add libstdc++ tests to the testsuite.

https://gcc.gnu.org/onlinedocs/libstdc++/manual/test.html

Can't seem to be able to get this to work... doesn't seem worth the trouble.


JWT not verifying properly

Steps to verify -

clone this repo.
Add env file with variables for passage app
Login

I had to dig pretty deep to figure out what was going on. From my testing what is happening is the login is occurring correctly on the home page and storing the JWT in the cookies. When redirecting to the /dashboard page, in getServerSideProps I'm validating the token and the token is failing. What is happening is the redirect is occuring so quickly that the "nbf" property in the JSON is after the validation function is rendering.
The value was "nbf":<PHONE_NUMBER>, and the time that the token was evaluated was<PHONE_NUMBER>580, or just over 400ms before the token is available to be used.
Maybe the token should have a nbf value of a few seconds before the request to prevent this kind of overlap?

Hello @joshua-richardson-88.  Thank you for reporting this issue.  After cloning your repo, I was able to reproduce the bug you were seeing regarding the JWT "nbf".  We have just released an update to our JWT creation logic to set the "nbf" to 5 seconds before the "iat".  This conforms to the JWT spec and will provide us with some leeway to deal with clock skew.
Please try it out and make sure that you are no longer having any issues authenticating your JWTs.
I will be closing this issue, but if the issue persists for you please comment and reopen this same issue report.

Hi there, I have tried this out several times since this ticket has closed - and I'm still receiving the NotBeforeError on validating the jwt. It seems like 5 seconds was added to the nbf field (5 seconds further into the future) than before. A sample I have from time<PHONE_NUMBER>451 is "nbf":<PHONE_NUMBER>

I'm following the instructions for a next js app but using the new app router instead of the pages router.
Once I call passage.authenticateRequest(req) I get Error: Could not find valid cookie for authentication.
These are the cookies in my browser after successful authentication.

#references https://github.com/passageidentity/supabase-passage-example/issues/5#issue-1751221374


Test UI of storage_alias

The storage_alias_ui tests are currently quite meager.
Should be extended to check all common combinations and types.

Do you have any thing that you are missing?

Mainly some success tests to fix the UI in place, prio to changes like https://github.com/paritytech/substrate/issues/13263 so we dont accidentally break it.
The metadata comparison test from here would also be nice https://github.com/paritytech/substrate/commit/79378d1000f4858aa37c1783e611cca4a29e25a4 for all types.