Cannot import CSS from dependency module (importing local CSS is fine)

Have you read the Contributing Guidelines on issues?

[X] I have read the Contributing Guidelines on issues.

Prerequisites

[X] I'm using the latest version of Docusaurus.
[X] I have tried the npm run clear or yarn clear command.
[X] I have tried rm -rf node_modules yarn.lock package-lock.json and re-installing packages.
[X] I have tried creating a repro with https://new.docusaurus.io.
[X] I have read the console error message carefully (if applicable).

Description
Imported CSS from a dependency module has no effect, as if the import statement does not exist.
import 'yet-another-react-lightbox/styles.css';

However, if I manually copy the css file to /src/css/ directory and import it from here, it works.
import '../../css/yarl.css';

I have created a reproducible demo in StackBlitz(link). The relevant code is /src/pages/index.js:L7-L10.
If you wish to try out a larger demo, you can also observe the effect in my own repository (https://github.com/yechs/website/blob/gallery/src/components/Gallery/index.tsx#L6-L8)
Reproducible demo
https://stackblitz.com/edit/github-dyi68j?file=src%2Fpages%2Findex.js
Steps to reproduce

Run the code as is (importing CSS from the module)
Open the page, click on any of the images. Observe that the created div with className yarl__portal yarl__portal_open has no CSS corresponded to them
Uncomment the line import '../css/yarl.css', which imports the CSS file from a local copy
Rerun the code
Open the page, click on any of the images. Observe that the lightbox now functions correctly with all the CSS styles.

Expected behavior
All the CSS are correctly imported from the module.
Actual behavior
No CSS is imported from the module, as if the import statement does not exist.
A hacky work-around is to manually copy the CSS to a local file and import from local file.
Your environment

Public source code:

(see stackblitz) https://stackblitz.com/edit/github-dyi68j?file=src%2Fpages%2Findex.js
OR
https://github.com/yechs/website/blob/gallery/src/components/Gallery/index.tsx#L6-L8

Public site URL:

(see stackblitz)
OR
https://deploy-preview-71--yechs.netlify.app/gallery/

Docusaurus version used:

2.0.0-beta.22, and can also be observed in 2.0.0-beta.21

Environment name and version (e.g. Chrome 89, Node.js 16.4):

Firefox 101 and Node v16.15.1
Firefox 101 and v16.14.2

Operating system and version (e.g. Ubuntu 20.04.2 LTS):

Self-service

[ ] I'd be willing to fix this bug myself.

Note: seems fixed with USE_SIMPLE_CSS_MINIFIER=true

Updated the StackBlitz repro with the environment variable. Seems like it is still not working

I'm pretty convinced this is because they have "sideEffects": false in their package.json which causes the CSS side-effect-import to be tree-shaken:
https://github.com/igordanchenko/yet-another-react-lightbox/blob/50d21a70a769c5a8859effecd0e222bdc9ddadca/package.json#L62
Doesn't seem like a Docusaurus defect, so closing

Oops, you are perfectly right. I'll bring this over to the package maintainer of yet-another-react-lightbox. Really appreciate your help in pinpointing the problem!

I'm pretty convinced this is because they have "sideEffects": false in their package.json which causes the CSS side-effect-import to be tree-shaken:
https://github.com/igordanchenko/yet-another-react-lightbox/blob/50d21a70a769c5a8859effecd0e222bdc9ddadca/package.json#L62
Doesn't seem like a Docusaurus defect, so closing

CSS is always a side effect, seems a shame to possibly reduce the tree shake-ability of a package just because it uses css?


Several Jupyter Notebook command-mode shortcuts not available in Jupyter Keymap

Testing #131209
While in command mode, these Jupyter shortcuts do not work.

Ctrl-Shift-F and P (in command mode) is supposed to open Jupyter Notebook command palette. Ctrl+Shift+F is probably safe to keep as Find in Files for us, maybe even desirable. However, P should open the command palette.
R should change cell to Raw, which I believe our extension supports.
1-6 should change cell to markdown headings at the appropriate level.
Shift-K and Shift-J should work like Shift-Up and Shift-Down
Shift-V should paste cells above
Shift-M should merge selected cells. If only one is selected then it just merges the next cell
O doesn't work at all. Error: command 'notebook.cell.toggleOutputs' not found
H doesn't show keyboard shortcuts in any form. Should it? Related to another bug I'm entering soon.
I I should interrupt kernel
0 0 should restart the kernel with dialog.
Space and Shift+Space should scroll notebook up and down.

O doesn't work at all. Error: command 'notebook.cell.toggleOutputs' not found

@greazer Are you on the latest VS Code Insiders (not Jupyter extension Insiders)?

Support for the rest of these operations needs to be implemented in VS Code.


AADSTS65001: The user or administrator has not consented to use the application with ID

I build a React SPA and a C# WebAPI, both added MSAL library. The frontend works fine, I am able to log in and out. When a call to the API is done I get a 400 (BadRequest) from the AAD with the following trace:
FetchClient.js:78 POST https://login.microsoftonline.com/XXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXX/oauth2/v2.0/token 400 (Bad Request)
Uncaught (in promise) InteractionRequiredAuthError: invalid_grant: AADSTS65001: The user or administrator has not consented to use the application with ID 'XXXXXXX-XXXX-XXXX-XXXX-XXXXXXXXXX' named 'XXXXXXX'. Send an interactive authorization request for this user and resource.
Trace ID: a1d60f68-2c82-475a-8d98-290fdec8a400
Correlation ID: 03324b45-b584-4df7-a41d-e46e2f4f8d12
Timestamp: 2021-07-02 05:50:12Z
at InteractionRequiredAuthError.AuthError [as constructor] (AuthError.js:27)
at InteractionRequiredAuthError.ServerError [as constructor] (ServerError.js:16)
at new InteractionRequiredAuthError (InteractionRequiredAuthError.js:31)
at ResponseHandler.validateTokenResponse (ResponseHandler.js:71)
at RefreshTokenClient. (RefreshTokenClient.js:41)
at step (_tslib.js:75)
at Object.next (_tslib.js:56)
at fulfilled (_tslib.js:46)
I added the access_as_user permission as the tutorial describes, it has been granted by the admin as well:

I tried to create a new app and add the permissions, it doesn't require the admin consent at all...

I found this settings:

Did not find any info on the web, setting it made my Apps working.


Resource type in resource definition should accept parameter or variable

Is your feature request related to a problem? Please describe.
When I define new resource I must use static string for resource type. E.g.
resource storageAccount 'Microsoft.Storage/storageAccounts@2019-06-01' existing = {
  name: resourceName
}

Describe the solution you'd like
Instead I'd like to do something like this
resource storageAccount storageType existing = {
  name: resourceName
}

where storageType is parameter or variable.

@sjuratov - mind sharing the use case you have in mind for this? Some notes to be aware of: https://github.com/Azure/bicep/issues/1002#issuecomment-733850619
We could certainly support referencing a constant string with something like:
var storageType = 'Microsoft.Storage/storageAccounts@2019-06-01'

// storageType is known at build time, so we can definitively understand what type of resource we're working with
// and provide editor/compile-time validation
resource storageAccount storageType existing = {
  name: resourceName
}

Referencing a runtime variable or parameter is more problematic, as it removes the ability for Bicep to do any validation on the type of resource that is being deployed.

Hi @anthony-c-martin , here is what I have in mind.
I would like to create set of Bicep files/modules that will build into an ARM template to allow following:

Apply RBAC at subscription scope
Apply RBAC at RG scope
Apply RBAC at resource scope

For all three there would be parameter (yes/no) if RBAC should be applied at specific scope. I would also be able to say what security principals (SP) would be given specific RBAC at specific scope.
E.g.

Subscription scope - YES apply RBAC

SP1 as Reader

RG scope - DON'T apply RBAC
Resource scope - YES apply RBAC

SP1 as a Contributor on storage account XYZ

So it would be "one stop" ARM template, that is flexible enough to add RBACs on any scope for any existing resource.
What I'd like to have:

For #3 (resource scope), I want to also pass on parameters about resource type and name (e.g. it could be Key Vaul, storage account, VM etc)
I haven't tried, but can I pass on multiple SPs instead of just a single one?

Just an update. @anthony-c-martin has a proposal on #2245 that would allow you to set a resource type as an input parameter:
param genericResource resource
...

I think it might solve some of your use cases, but am not sure if it covers everything above. It'd be great if you could take a look.

@sjuratov Got into a similar case today and despite not being exactly the same, this is a limited workaround for the multi-scope support in one template: https://github.com/matsest/az-bicep-roleassignment-multi-scope
Blocked by #2245 to be Bicep only (one instance of an ARM template to parametrize a resource type-agnostic input for the scope function).


Restart Rabbit on cluster join error, resolves #344

This change uses a custom exception handler to throw errors in join_cluster().  A exception handler in action :join calls Chef::Application.fatal! to keep the same error functionality as before but an ensure block ensures Rabbit is always restarted.  Resolves #344
I have not updated metadata as I'm assuming you want to manage that.

+1 this patch fix problem with ':join', @jjasghar are you panning to merge it?
But for now i will use fork from @CVTJNII to deploy rabbitmq.

Make it so.


Update pbt

Description

Related Issue

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[ ] Chore (non-breaking change that does not add functionality or fix an issue)

Checklist:

[ ] I have read the Code of Conduct
[ ] I have updated the documentation accordingly.
[ ] All commits are GPG signed

Hello @2lambda123! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

In the file xt/algorithm/alg_utils.py:

Line 36:80: E501 line too long (93 > 79 characters)


Multitap not working in games that check for it after boot

tonyhax version: 1.4.5
Installation method: mc annihilator + ulaunchelf on usb, on a ps2
Entry point game: no entry point game. just a memory card
Console model: SHCP-102
Integrity check: yes
BIOS version: v4.5 05/25/00 E
Target game: Crash Team Racing (SCES-02105)
Bug explanation:
Games that ask the user to plug in the multitap after being booted don't work with tonyhax (tested with Crash Team Racing, but also reproducible with Crash Bash).
When the multitap is connected, the console recognises the first controller plugged in the multitap. Connecting more than one controller crashes and reboots the console.

Does this happen with v1.5.2?
Does this happen with the original tonyhax? https://github.com/socram8888/tonyhax/releases

@alex-free yes, it was happening with original tonyhax 1.4.5. I've updated the memory card to tonyhax international 1.5.2 and same thing happens (Crash Bash reboots the console as soon as the multitap is connected).


Update customer-database-tutorial.md

Brackets around If Statement from {} to ()

@mcleanbyron  : The PR passed the acceptance checklist. Please evaluate further and approve/complete the PR if appropriate. Thanks!

Thanks for submitting this fix!


ComfyUI in img2img inverts the colors

This only happens in img2img tab, not in txt2img tab:

p_img2img_init() in lib_comfyui.webui.workflow_patcher is responsible. Will push a fix when I find the cause.

Yep, it's a byte overflow. Apparently * 255 on a byte will invert the bits.


error symbol not found when executing command 'host' or 'dig'

Hi,
Since the PR #24, when executing dig or host, I have this error

Error relocating /usr/lib/libisc.so.1200: explicit_bzero: symbol not found.

@lcaille please try nicolaka/netshoot:edge , it's using the alpine:edge which addresses the isssue.

It works with the edge version, thank you!


[ci:component:github.com/gardener/vpn2:0.13.0->0.14.0]

Release Notes:
Set primary device for bond device to avoid partitioning of VPN clients into two groups

Update builder image to `golang:1.19.5`

/test pull-gardener-integration

/assign @MartinWeindel

/lgtm

/kind enhancement
/approve

/cherry-pick release-v1.62


More macro fixes

Tested with Hunter and works on many more platforms after this fix.

Released: https://github.com/hunter-packages/leveldb/releases/tag/v1.20-p3

Thanks!


zenpower: 0.1.5 -> 0.1.10

Motivation for this change
Things done

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

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

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

It is v0.1.10 atm

Updated to 0.1.10.

@GrahamcOfBorg build linuxPackages.zenpower

Needs to either be fixed or marked as broken or unsupported on aarch64.

this is kernel hwmon mod for AMD processor so should mark this for x86 only

Apologies for the delay. I've added the suggested platform constraint.

@GrahamcOfBorg build zenpower

@GrahamcOfBorg build linuxPackages.zenpower

Commit message would ideally say “linuxPackages.zenpower: …” but I won’t block on that


Update some PopperJS related tests to get rid of RTL act warnings

PopperJS RTL tests are causing act warnings downstream.

This issue is being put on hold in favour of https://github.com/defencedigital/mod-uk-design-system/issues/2955 which will resolve the issue as side effect.
The crux of the problem comes from an upstream library and I don't want to write hacky workarounds in all our test suites.
Marking this as On Hold for now.

This is still an issue - waiting for a fix upstream.

Closing this because there does not appear to be any change in position - gridlock upstream.


[C+ Checklist Needs Completion] [$250] Fix styling of Google Sign In button on Safari

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Action Performed:
Break down in numbered steps

Open the main page on desktop Safari, with Apple and Google sign in links

Expected Result:
Describe what you think should've happened

Google button should appear in full, and stay static once loaded

Actual Result:
Describe what actually happened

The G part of the button looks fine, but right after it loads the white background shifts one pixel right and is cut off by that amount.

Workaround:
Can the user still use Expensify without this being fixed? Have you informed them of the workaround?
Yes
Platforms:

Which of our officially supported platforms is this issue occurring on?

[ ] Android / native
[ ] Android / Chrome
[ ] iOS / native
[ ] iOS / Safari
[X] MacOS / Safari
[ ] MacOS / Desktop

Version Number: 1.3.57-1
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: https://expensify.slack.com/archives/C01GTK53T8Q/p1692805800476329?thread_ts=1692802374.824749&cid=C01GTK53T8Q
https://github.com/Expensify/App/assets/5487802/178f0126-d890-47c0-949c-5cd91376f14c
Expensify/Expensify Issue URL:
Issue reported by:
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1692805800476329?thread_ts=1692802374.824749&cid=C01GTK53T8Q
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~0131c050705af6ac9f
Upwork Job ID:<PHONE_NUMBER>733020160
Last Price Increase: 2023-08-25
Automatic offers:

Pujan92 | Contributor | 26508718

Proposal
Please re-state the problem that we are trying to solve in this issue.
Google sign in button cuts from right in safari
What is the root cause of that problem?
Seems after adding overflow hidden started the problem. When parent width and overflow hidden gets given it seems to create the problem for safari.
https://stackoverflow.com/questions/75306089/safari-when-using-border-radius-and-overflow-hidden-to-parent-and-the-child-th
What changes do you think we should make in order to solve the problem?
We can use styles.willChangeTransform(will-change: transform) for the layout glitch for the safari as suggested in the comment and I think we are already using the same at other places.
OR
We can increase the width of the container and apply some padding so it cut part can be visible on the padding area.

Setting this to $250 as a minor adjustment

not overdue - reviewing / waiting for other proposals

@Pujan92 Could you please be more specific in your proposal on how you want to use the styles.willChangeTransform styles? Thanks

@burczu updated the proposal.

Hello there, can I be assigned to solve this bug?
I will provide solution to it.

@jopaleti Please read the contributing guide and just post your proposal here using the proposal template.

I've reviewed the @Pujan92's proposal and I think we can go with it - it fixes the issue.
🎀 👀 🎀 C+ reviewed

There's two proposed solutions there. Which one do we think is best? @burczu @Pujan92.
Can you explain how will-change: transform fixes the issue? Is there actually a transform being applied to that container?

@arosiclair I was referring to the first solution from @Pujan92's proposal

Thoughts on the above @arosiclair?

Bump @arosiclair <3

@Pujan92 please add comments explaining why will-change: transform is needed

Just to inform: I'll be OOO on Thursday and Friday (7-8 September).

@burczu PR is ready for review!

Sorry for the delay, I've been out with an illness. Processing this today.

@burczu does not require payment (Contractor)
@Pujan92 requires payment offer (Contributor)

Paid. @burczu can you take care of the checklist?

@greg-schroeder is this eligible for urgency bonus as @burczu wasn't available for 2 days based on this comment?

BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:

[x]  [@burczu] The PR that introduced the bug has been identified. Link to the PR: It's not a regression - it was just hard to be noticed during initial testing, I guess.
[x]  [@burczu] The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake. Link to comment: n/a
[x]  [@burczu] A discussion in #expensify-bugs has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner. Link to discussion: n/a
[x]  [@burczu] Determine if we should create a regression test for this bug.
[x]  [@burczu] If we decide to create a regression test for the bug, please propose the regression test steps to ensure the same bug will not reach production again.
[ ]  [@greg-schroeder] Link the GH issue for creating/updating the regression test once above steps have been agreed upon:


MongoDB failure on app that doesn't use Mongo

Hey Pythagora. I love what you're aiming for with the project. I have a simple Express app that runs on a postgresql backend, and I created Pythagora tests on a few simple paths. When I went to run them, I encountered a Mongo error I wasn't expecting:
Running tests in 3 seconds...
Listening on port 5001
Starting tests on endpoints:
/
/v1/status

(node:9065) [MONGOOSE] DeprecationWarning: Mongoose: the `strictQuery` option will be switched back to `false` by default in Mongoose 7. Use `mongoose.set('strictQuery', false);` if you want to prepare for this change. Or use `mongoose.set('strictQuery', true);` to suppress this warning.
(Use `node --trace-deprecation ...` to show where the warning was created)
The app has crashed!
This is likely not related to Pythagora, but the app itself.
MongoRuntimeError: Unable to parse undefined:undefined with URL
    at new HostAddress (/Users/chris/Workspace/my-project/node_modules/mongodb/lib/utils.js:888:34)
    at fromString (/Users/chris/Workspace/my-project/node_modules/mongodb/lib/utils.js:930:16)
    ... 4 lines matching cause stack trace ...
    at new Promise (<anonymous>)
    at NativeConnection.Connection.openUri (/Users/chris/Workspace/my-project/node_modules/mongoose/lib/connection.js:799:19)
    at /Users/chris/Workspace/my-project/node_modules/mongoose/lib/index.js:409:10
    at /Users/chris/Workspace/my-project/node_modules/mongoose/lib/helpers/promiseOrCallback.js:41:5 {
  cause: TypeError [ERR_INVALID_URL]: Invalid URL
      at new NodeError (node:internal/errors:372:5)
      at URL.onParseError (node:internal/url:553:9)
      at new URL (node:internal/url:629:5)
      at new HostAddress (/Users/chris/Workspace/my-project/node_modules/mongodb/lib/utils.js:885:19)
      at fromString (/Users/chris/Workspace/my-project/node_modules/mongodb/lib/utils.js:930:16)
      at Array.map (<anonymous>)
      at parseOptions (/Users/chris/Workspace/my-project/node_modules/mongodb/lib/connection_string.js:210:45)
      at new MongoClient (/Users/chris/Workspace/my-project/node_modules/mongodb/lib/mongo_client.js:46:63)
      at /Users/chris/Workspace/my-project/node_modules/mongoose/lib/connection.js:802:16
      at new Promise (<anonymous>) {
    input: 'iLoveJS://undefined:undefined',
    code: 'ERR_INVALID_URL'
  },
  [Symbol(errorLabels)]: Set(0) {}
}

Hi @chrismclennon ! Thank you so much for trying out Pythagora, really appreciate it.
Looking at this trace I can’t really tell what happened and where the issue is. It looks like you are using mongoose (since you have it in node_modules) but you also mentioned you have postgresql. How come you are using both? From trace it seems like you are trying to connect with mongoose but you are sending 'ilovejs://undefined:undefined' as URL.
In general it is ok to use Pythagora with project that doesn’t have mongoose and it should all run without crashing. Issue on projects without mongoose is that Pythagora will not be able to capture database queries so endpoints that do interact with postgresql will probably fail in testing phase until we start supporting postgresql. We are still not sure at what point we will support it but we have it planned for future.


test1

test
AB#3552

Hi @mahendar1122 is this a test issue or you are going to add anything later to it ? Let us know before we move it to close.
Thanks


Implement projects page

Implements the projects page, uses live data from the GitHub API - look into caching.

Current dependencies on/for this PR:

main

PR #1   👈

This comment was auto-generated by Graphite.


regex does not compile on latest nightly

┌ anrussell@anrussell-mba ~/repos/regex (master)
└╌╌┄┄ ❯❯ rustup run nightly cargo test
error: failed to parse manifest at `/Users/anrussell/repos/regex/Cargo.toml`

Caused by:
  duplicate build target found: `/Users/anrussell/repos/regex/src/lib.rs`

Whoops, looks like this is fixed in master!


"create an account" should be removed from open-source edition

"Create an account" link points to a signup page in the SaaS, it should be removed or dealt differently in open-source edition.

The signup page link ('http://cartodb.com/signup') is in the following files:

app/views/admin/shared/_new_public_header.erb
app/views/admin/shared/_public_footer.html.erb
app/views/admin/shared/_public_header.html.erb
app/views/admin/visualizations/public/_public_map_footer.html.erb
app/views/oauth/ (authorize.html || authorize_failure.html.erb || authorize_success.html.erb)
app/views/sessions/new.html.erb PR#2478

@javisantana ,  should I hide all that links if ( Cartodb.config[:account_host] == "cartodb.com") ??

Closing with this PR#2646 -> Merging on next 9thMonday


feat: group event settings

Resolves #190
This PR group the event settings.
Moved events to the "Events" tab

Also moved Startup event, and Unit testing from "Main" tab

This PR should be on top of https://github.com/zd-zero/waifu-motivator-plugin/pull/195 for this also updated the settings page.

It's hard to fix the conflicts with the *.form file. I'll just redo this 😂


#2088 - New command for copying the page/template spo page copy

This PR contains the spo page copy command for #2088

Coverage remained the same at 100.0% when pulling d7e89609ca7c82cbb461ed408b5bf770321fae48 on estruyf:issue/2088 into d86c41723999b21ea6222f9dc24b43f81e2255cb on pnp:master.

Coverage remained the same at 100.0% when pulling d7e89609ca7c82cbb461ed408b5bf770321fae48 on estruyf:issue/2088 into d86c41723999b21ea6222f9dc24b43f81e2255cb on pnp:master.

Woohoo! That was quick! We'll go through it asap 👍

Woohoo! That was quick! We'll go through it asap 👍

Seems it had to do with the additional PRs that were merged and the change from master to main.

Seems it had to do with the additional PRs that were merged and the change from master to main.

According to GitHub, renaming the branch shouldn't affect any open PRs but maybe we hit an edge case. Anyway, is the PR ready to be checked?

According to GitHub, renaming the branch shouldn't affect any open PRs but maybe we hit an edge case. Anyway, is the PR ready to be checked?

Looks like it is ready, I didn't get any issues during the merge and test as such were still working fine.

Looks like it is ready, I didn't get any issues during the merge and test as such were still working fine.

Unfortunately, the issues are still there when you try to rebase changes onto main. The fork seems to be going all the way back to Nov last year with a number of merges in between, but when trying to rebase, we'd be getting rid of all of them and that's where we're getting conflicts.

Unfortunately, the issues are still there when you try to rebase changes onto main. The fork seems to be going all the way back to Nov last year with a number of merges in between, but when trying to rebase, we'd be getting rid of all of them and that's where we're getting conflicts.

I'll do a new PR starting from the new main branch.

I'll do a new PR starting from the new main branch.


postcss-load-config to postcss org

@michael-ciniawsky @ai
I think we should move postcss-load-config under postcss org to better maintenance, anyone can be absent for some reason, i think better move this repo under postcss org, then update deps and fix problem.
Related issues: https://github.com/postcss/postcss-loader/issues/358, maybe https://github.com/postcss/postcss-loader/issues/307

Or even move it to the core 😏

@ai I think moving it to the core can take more time (triangle/discussion/other meta stuff). I agree will be great have this in core. Just to eliminate current issues, it would be enough to transfer this to repo and upgrade deps.

/cc @michael-ciniawsky can we update deps for postcss-load-config? cosmiconfig have better perf on 4 version

@evilebottnawi do we have breaking changing in that release?

I don’t have npm access to help here with a quick release. Try to write an email.

@ai let's wait @michael-ciniawsky if no answer, we can recreate new repo :disappointed:

Forking is not an option. Especially, just to update dependencies.

We can of course move it under postcss and everyone who needs it can have any access to, since it is meant to be the common postcss config and not just some opinionated module of mine. @evilebottnawi It works without any issues atm, so their is really no need to rush anything here :). Any kind of  'perf' improvement is claiming until it is actually benchmarked... There are a few things outstanding for postcss-load-config but all changes need to be tested across the different runners and I just haven't found the time to do so yet. Global requires for the CLI and better {Error}s are needed, but I was personally waiting for a few things while there isn't any urgency hoping that

node's decision on how ES Modules will work in the future and what this may mean for postcss-load-config and global/local plugin loading (unresolved). This is important since loading local plugins from a global config requires to hook into module (CJS) which may not be possible with import() and may cause trouble to migrate later on.
Breaking Changes of cosmiconfig v5.0.0 (which was released ~2 hours ago) (resolved). We may also consider just resolving the config on our own since cosmiconfig includes extras we don't really need. Supporting postcss.config.js,.postcssrc.js && pkg.postcss with ctx should be enough. The search algo is trivial here and doesn't really require and an additional module. Still ensuring searching works the same for existing postcss.config.js etc would be needed then
plugins: <Array> won't work with 'global' requires, so this means that it's not just a choice of 'style', but can also result in different config behavior then (bad)

Also for a new major version we might in general triage what we can do better beforehand, what breaking changes (should be avoided at all cost) this would cause etc etc..
Making it just one module would also be a good idea, since splitting it into 3 was a very bad idea of mine back then

@michael-ciniawsky glad you are back 😸. Yeap, no rush.
Could you give me (iskin at npm) npm access just in case of security or other critical issue?
I am still thinking what we will get from adding library as dependency to PostCSS core. Seems like right now all PostCSS runners anyway use it.

It could also be included in core directly instead of being a separate module or core ships with the module included by default (with a condition to exclude it for other targets then node) trying to load a config via options.from || process.cwd(). Since it will always require access to the File System and therefore 'only' really work with node (which is the main use case), the question is if this unnecessarily complicates bundling postcss for other targets (e.g browser). Still something to explore since the implementation is small and trivial atm

Could you give me (iskin at npm) npm access just in case of security or other critical issue?

Done

Yep, something like that :)

@michael-ciniawsky thanks for access, now we can solve security problems and big bug  :+1:

😮 I just added @ai on npm atm. Who and what access does anyone envolved need (github/npm) ? Should we move it to postcss or leave it and discuss core integration first (the module is then deprecated and obsolete soon anyways) ?

I think my contact will be enough.
Yeap. Let's create a issue in postcss/postcss

https://github.com/postcss/postcss/issues/1147


feat(fips): utils sha256 hash function (fips compliant)

This PR is a prerequisite for:

https://github.com/Kong/kong-ee/pull/3845

FT-2418

closing - continuing in: https://github.com/Kong/kong/pull/9533


[QTCS] QTCS namespace of dynamic args

Make dynamic_args of QTCS to have a namespace. Easiest resolve, to prefix them with qtcs_, qtcbs_, etc as needed.

I didn't think of that. Well spotted. Will fix.

Thanks. Will do the same for MOS as soon as I can.

Once I fix it for qtc I can fix it for mos as well if you like.

As you wish. Can you have the dict key as qtcs though?

Or qtcbs, qtcs, qtcbcs if they are specific.

Of course.

Great thanks.


Resolve isAlive deprecation warning

Prevent deprecating warnings about threading.Thread.isAlive() by renaming to is_alive().

/morph test

/morph test


Bitcode on iOS should be enabled by default in our templates

Once we have enough feedback on how this is working out, we should consider enabling bitcode by default in our templates and migrate the rest of our applications that are eligible for bitcode to bitcode.
As of this writing, Flutter Gallery is not elligible because of the Connectivity library, which is not build with bitcode.

From https://github.com/flutter/flutter/issues/48092#issuecomment-577345215:

When I run flutter build ios --release in the module, the host app can then archive successfully. When I run flutter build ios --debug in the module, we see the reported error on archive.
The "Run Flutter Build Script" host app script build phase should be swapping the right version of the engine based on the host app build configuration (Debug, Release), but:

The plugin target (shared_preferences) is being built before the host app, so before that build phase can run.

The plugin frameworks get built before the app does.  On archive, if they have bitcode enabled, all frameworks they link on must have bitcode enabled.  So if flutter build --debug was last run, it will fail because the plugins are linking against the Debug version of Flutter.framework, which doesn't have bitcode enabled.  The app build phase script would have copied the Release version of Flutter.framework to the right spot, but it runs after the plugins are built, so the build fails before it even gets that far...

I think as of https://github.com/flutter/flutter/pull/70224 we should try this.

The only problem I can think of is if there are plugins out there (either many or popular ones) that are not compatible with Bitcode.

It's should be up to the plugin and pod authors to set ENABLE_BITCODE=NO themselves in their podspec.  All the dependencies of those specs shouldn't need to also set it.

Update: Per comment in #73607 we shouldn't do this until Apple requires it, to avoid breaking anyone who depends on the current behavior.

Closing this, the only bitcode work we're going to do at this point is remove it https://github.com/flutter/flutter/issues/107887.


[Feature] - R.method or R.propEval

Imagine this use case
function Person (first, last){
  this.firstName = first;
  this.lastName = last;
}

Person.prototype.getFullName = function (){
  return this.firstName + ' ' + this.lastName;
}

var arr = [new Person('Francis', 'Bacon'), new Person('Kevin', 'Bacon')]

We can easily map for first or last name
var firstNames = R.map(R.prop('firstName'), arr);
var lastNames= R.map(R.prop('lastName'), arr);

but you cannot map over full name var fullNames = R.map(R.prop('getFullName'), arr);
I propose R.method or R.propEval which would have the following signature
src/propEval.js
var _curry2 = require('./internal/_curry2');

/**
 * Returns a function that when supplied an object returns the indicated evaluated method of that object, if it exists.
 *
 * @func
 * @memberOf R
 * @since v0.18.0
 * @category Object
 * @sig s -> {s: a} -> a | Undefined
 * @param {String} p The method name
 * @param {Object} obj The object to query
 * @return {*} The value at `obj.p()`.
 * @example
 *
 *      R.propEval('valueOf', new Date()); //=> 100
 *      R.propEval('x', {}); //=> undefined
 */
module.exports = _curry2(function propEval(p, obj) { return obj[p] && obj[p](); });

Additional Thoughts
As it is written, this would evaluate the method with no parameters. It would be possible to edit this or create another method to evaluate the object's method with a given array of arguments.

I do realize that in this PARTICULAR example, I could solve the problem with getters
function Person (first, last){
  this.firstName = first;
  this.lastName = last;
}

Object.defineProperty(Person.prototype, 'fullName', {
  get: function (){return this.firstName + ' ' + this.lastName;}
})

var arr = [new Person('Francis', 'Bacon'), new Person('Kevin', 'Bacon')]

R.map(R.prop('fullName'), arr);

But it could still be useful to evaluate methods on objects.

why not:
var fullNames = R.map(R.invoker(0, 'getFullName'), arr);

Yes, it looks as though invoker already covers this, and also handles the cases of higher arity.

That's entirely correct!
R.map(R.invoker(0, 'getFullName'), arr)

Thank you very much! Didn't catch that one in the documentation


Add build_and_deploy workflow

This is to be used by device types to automatically build a deploy
releases on repository tag.
Change-type: patch
Signed-off-by: Alex Gonzalez<EMAIL_ADDRESS>

@balena-ci I self-certify!


improving perfromance while using ble & wifi (IDFGH-8964)

Answers checklist.

[X] I have read the documentation ESP-IDF Programming Guide and the issue is not addressed there.
[X] I have updated my IDF branch (master or release) to the latest version and checked that the issue is present there.
[X] I have searched the issue tracker for a similar issue and not found a similar issue.

General issue report
hello,
how do i improve ble & wifi perfromance much more in wrover 16mb.. right i have been advertising more than 200 beacons per second but only been getting 100 beacons on scan result when i use both wifi and ble.. is there any way to improve this perfromance so that i can get as much data as possibe?
Development Kit: custom board
Module or chip used: ESP32-WROVER
IDF version :4.3
Build System:Make
Compiler version :8.4
Operating System: Windows
(Windows only) environment type: MSYS2 mingw32.
Using an IDE?Yes eclipse IDE
thanks and regards
manilandan v

Hi, @manivalaguru
This issue was created a long time ago, does the issue still exist and need to be solved?

Weird question, given that it's a timeless issue, and it's also phrased that way.
How to improve BLE performance?
Seems like a cue for a new coex and/or BLE section under Performance in the docs.

Weird question, given that it's a timeless issue, and it's also phrased that way. How to improve BLE performance? Seems like a cue for a new coex and/or BLE section under Performance in the docs.

Yes, this is a contradictory issue. WIFI and BLE is sharing a single RF module with time-division multiplexing method, it's normal for BLE performance to degrade when Wi-Fi is enabled.
We cannot adjust the time ratio arbitrarily between WIFI and BLE, because we need to ensure that both WIFI and BLE have enough time slices to maintain the connection (this is done in coexist module automatically). All we can do is prevent abnormal performance degradation in BLE.

Hi, @manivalaguru
Due to no reply for a long time we will close this issue.
Thanks for you report, please feel free to reopen it if necessary.

Thanks for reporting, will close due to short of feedback, feel free to reopen with more updates.


Can support persian calendar ?

Hi , first thanks for awesome packages.
I see material-ui in React.js that fully support Persian calendar.

Is there a way to implement the Persian calendar for angular ?
Thanks for your attentions.

Take a look at this plunker: http://plnkr.co/edit/uH6C87?p=preview

Closing this issue as it does not follow the issue template. Please file a new issue that follows the template, which gives the team the information needed to investigate.


Youth risk assessment updated

@teneightfive - trying to find your suggestion of "Enter the date of the person's first time in custody" - basically yes, nice change


feat: add json format output for http interface

I hereby agree to the terms of the GreptimeDB CLA.
Refer to a related PR or issue link (optional)
Closes #4764.
What's changed and what's your intention?

Some BI or visualization tools like PowerBI accepts json data from custom web apis.

As explained in #4764, it's better if the http service return a json format that is already supported by other products.

Added JsonResponse.
Placed relevant tests.

Checklist

[ ] I have written the necessary rustdoc comments.
[ ] I have added the necessary unit tests and integration tests.
[ ] This PR requires documentation updates.

The branch is including my last PR. Not sure what the cause is.

I think it's because you created this branch from your last feature branch. If your previous branch has been merged, you can rebase to main to fix that

I think it's because you created this branch from your last feature branch. If your previous branch has been merged, you can rebase to main to fix that

I thought so too. thx

Almost LGTM. It would be nice if we can add a test case in tests-integration/tests/http.rs

Added a test case. Let me know if anything else is needed.


Could you provide any installer package, not raw binary files?

Could you provide any installer like .msi, .deb, etc? This may speed up my installation progress. Setting up PATH always take lots of time.
Thanks a lot for your great work.

不用安装，直接就能打开

I think it's fairly easy to write setup scripts by yourself. For example, VBScript is a convenient option on Windows, and POSIX shell script is the most natural choice on Linux. The scripts can include a downloading process or just detect downloaded binaries, and they should handle the rest in a consistent manner.

@iBug Writing a setup script is a great idea, but I'm too lazy to do anything after installation _ (:з」∠) _

An easy solution for Unix-like systems is to put the binary under /usr/local/bin which is almost always in your $PATH.
For Windows, you can always launch a Command Prompt in the directory the executable is in by right-clicking the directory background while holding down [Shift] key.

@iBug That's a solution, thank you.


Make react-click-outside event listener configurable so that it can be attached to event bubbling phase rather than capturing phase.

In most of the scenarios listening to event capturing does make sense. But in certain complex components, it would be cool if it could rely on event bubbling phase rather than capturing phase.
Let me know if you need specific use case. I will send a PR if this is sounds reasonable.

For me, I have a dropdown rendered in an iframe. Clicking outside the iframe triggers the handleClickOutside method, but clicking anywhere in the iframe does not. I don't suppose there's currently a way around this?

I have an interesting one where a Popover is wrapped in withClickOutside to close it, but there are some components that, when clicked, should not close Popover. I’d like to do that by stopping propagation on those components, but it’s tough if the click outside event handler is using capturing.

@gggritso I would suggest not using stopPropagation to stop, since the effect is very implicit. Maybe something like an explicit exclusion would work better, see: https://github.com/kentor/react-click-outside/issues/15


Gist: GitHub returned error 404: Not Found

Mac OS 10.10.5
ST3 -- latest

It looks like it was some problems with github api during that time. You can try again and it should work fine.


skkDictionaries: split up skk-dicts

Description of changes
skk-dicts contain dictionary files that aid writing Japanese text with SKK-style input method editors. It offers multiple different dictionary files that users are supposed to combine based on their style of writing. Exactly which comibination of those dictionaries a user should install is purely based on personal preference. However, the current skk-dicts package lumps the dictionaries into a single package and is opionated in which dictionaries it includes and which of them it doesn't.
This PR offers users the required flexibility by splitting skk-dicts into multiple packages under the skkDictionaries attribute.

Things done

Built on platform(s)

[ ] x86_64-linux
[ ] aarch64-linux
[X] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[X] Tested basic functionality of all binary files (usually in ./result/bin/)
23.11 Release Notes (or backporting 23.05 Release notes)

[X] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[X] Fits CONTRIBUTING.md.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-ready-for-review/3032/2929

I moved the release notes following the release of NixOS 23.11.

@midchildan How is this PR going? It would be helpful if you could rebase it and request another review on Discourse.

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/prs-already-reviewed/2617/1990

Sorry for the delay. I fixed the merge conflict and posted another link in the Discourse thread.

Thank you for the great work :rocket:


Simulation freezes from time to time

I am using the Official IBM Operational Decision Manager for Developers image version 8.10. Simulation is freezing from time to time.

In your docker setting could you increase your memory
In Mac : Docker->Preference->Advanced Put 4Gb

Hi Laurent, I did it and it works fine. By the way I am using the very same options as for version <IP_ADDRESS>: docker run -e LICENSE=accept -p 9060:9060 -p 9443:9443 -v $PWD:/config/dbdata/ -e SAMPLE=false ibmcom/odm:8.10
Thanks again,
Francis


Installation Issue

The following error keeps occuring when running
pip install -e .
 note: This error originates from a subprocess, and is likely not a problem with pip.   ERROR: Failed building wheel for egl_probe   Running setup.py clean for egl_probe Failed to build egl_probe ERROR: Could not build wheels for egl_probe, which is required to install pyproject.toml-based projects
and this error even occurs when trying to do pip install egl_probe . How do i fix this, also are there any independent scripts available for postprocessing the hd5 file collected in "Robosuite" and to extract image observation from it?
Thanks

what if you tried pip install egl-probe==1.0.2?

Still the same issue.

This error has occured for me as well

sudo apt install build-essential can solve my problem.
ref: https://github.com/huggingface/lerobot/issues/105#issuecomment-2096007109


[FEATURE REQUEST] Hide ignored packages

Please confirm these before moving forward.

[X] I have searched for my feature proposal and have not found a work-in-progress/duplicate/resolved/discarded issue.
[X] This proposal is a completely new feature. If you want to suggest an improvement or an enhancement, please use this template.

Describe the new feature
Add a filter "Hide ignored packages" for installed packages so the list is not cluttered with these ignored packages.
Describe how this new feature could help users
It would really help me to overview the relevant ones with over 880 installed packages from all sources (mostly plugins for DAWs and graphic/video editing).

i still get notifications that its trying to update ignored packages too


Readme.md : image name is not available without login

Thanks for this exporter :)
i had to build image locally because nexus-exporter image required authentication to be pulled as mentioned in README.md
docker run [OTHER-DOCKER-RUN-OPTIONS] -p 9184:9184 -e NEXUS_HOST=http://localhost:8081 -e NEXUS_USERNAME=admin -e NEXUS_ADMIN_PASSWORD=admin123  nexus-exporter

looks like using  ocadotechnology/nexus-exporter is better. It is the image I have seen it in the opened issue. Maybe it could be interesting to change REAMDE.md :)

cool, thanks. Have created an mr to fix


feat: implement fd_pread (similar to fd_read)

Fixes #43

Thanks for the PR. Just as a heads up: I'm rather busy right now, so it may take a couple of days before I get around reviewing this.


install R dependencies automatically

This can be called from Python using subprocess.Popen() from run_sb_pipe.py in order to check that all the dependencies are loaded.

packages to install from apt-get / rpm
python-dev

Don't merge this because tests fail after reinstalling the dependencies and there are also errors when these are installing.

The problem is that R packages cannot be installed in /usr/local . When install.packages() is executed on an interactive console, it suggests the user to install the locally. This interactive process is skipped in my R script because it is executed via Python subprocess.
The easiest and simplest solution is to manage the dependencies for Python and R separately.
install_sb_pipe_deps.py => inst_sbpipe_pydeps.py
install_dependencies.r => inst_sbpipe_rdeps.r
Clean the code and move them in sb_pipe/sb_pipe .

Bug solved. Dependency separation from source code.


#19 Show reference names if possible

Have done some super basic testing on this, seems to hold up.
Shows the name of the reference in the One of < block.


language RTL switching

Hi All, After updating to the latest I have noticed that the right to left switching for Farsi is not working. Is there anyone facing same issue and what is the workaround.

Hi Parbil,
I think you can add, in the file Initialization/UserCultureProvider.cs, to the property "Dictionary<string, string> TwoLetterToFourLetter = new Dictionary<string, string>",
the following line
{ "fa", "fa-IR" }
please check https://github.com/volkanceylan/Serene/pull/62/commits/75dbd002bf2ae7dac9eac4860ce3cc8745d8e091#diff-278788fb8a8b7d35d92bc6549b2de2d8
thank you


dataset format

Thanks for your excellent work. I feel confused when I try to read the code about datasets.py. Can you introduce some details about scannet_*.json? How to understand the 'segmentation', 'bbox', 'area'?

I arrange the data annotation the same way as COCO data format for segmentation and detection, (https://www.immersivelimit.com/tutorials/create-coco-annotations-from-scratch). Only add an extra annotation for plane parameters.
These terms you mentioned are:
segmentation: the encoder contour of the instance mask,
bbox: the bounding box of an instance (a plane region),
area: the area of an instance (the number of pixels of a plane region).

Thank you.
Another question is about plane parameters. As the paper said, the prn does not explicitly predict plane parameters. Why the plane parameters are added into the train dataset?

Hi, that because we still want to use the normal vector of plane regions to
supervise the depth estimation, so that the predicted depth is somehow
"flat" or at least not "rough" in these regions. The part about "plane
surface normal loss" explained the idea.
Am Fr., 25. März 2022 um 10:36 Uhr schrieb RenQJ @.***>:

Thank you.
Another question is about plane parameters. As the paper said, the prn
does not explicitly predict plane parameters. Why the plane parameters are
added into the train dataset?
—
Reply to this email directly, view it on GitHub
https://github.com/EryiXie/PlaneRecNet/issues/7#issuecomment-1078828653,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AG7TXSYURF3JTMNYOPMHN4DVBWCIRANCNFSM5RTQNCUQ
.
You are receiving this because you commented.Message ID:
@.***>


Allow Select in Portal deployment SKU Gen1 VPN Gateway Zone redundant options

For Portal option deployment, the only options Gen2AZ. However, there are options for G1 that has Zone redundant. In the case customer design to use that it must be created manually. The request is able to select the Gen1AZ options as well through the portal.

Hey @anlucen,
Thanks for the feature request. We are leading with Gen 2 GWs as they offer better scale support without the need for re-deployment and reconfiguration of a VPN GW.
Is there any reason that the Gen2 lowest SKU wasn't suitable. E.g. was there a hard requirement that it must be Gen 1?
Let us know 👍👍

My main concern would be that VpnGw2 is over twice the price of VpnGw1
$183.9787/month v $474.4712/month

We are discussing this internally and will report back soon

@jtracey93 after a few discussion with the customer finally was ok to implement the VPNGw AZ as best practice. However, they were asking why can't pick that in the portal automation if in the other tools is possible. Let me know any thoughts

Thanks for letting me know. This is because in the ALZ custom portal template, we do not allow the deployment of Gen 1 SKUs as we want customers to deploy Gen 2 and be setup for long term success.
Closing issue


Dont allow serialization of firestore settings

When logging any firestore object like WriteBatch,Transaction,etc the settings object also gets logged / exposed
This can be seen by running JSON.stringify on any firestore object even a document reference
Many developers log firestore objects to help them debug testing/prod issues, this leaking of entire firestore key via this._settings is a bad practice as per me
We can also use Object.defineProperty to make it non-enumerable or any other technique that you like
Thank you for opening a Pull Request! Before submitting your PR, there are a few things you can do to make sure it goes smoothly:

[ ] Make sure to open an issue as a bug/issue before writing your code!  That way we can discuss the change, evaluate designs, and agree on the general idea
[ ] Ensure the tests and linter pass
[ ] Code coverage does not decrease (if any source code was changed)
[ ] Appropriate docs were updated (if necessary)

Fixes #<issue_number_goes_here> 🦕

Thanks for your contribution @abhishekwebcode . If I understand correctly, you are concerned about users seeing the name of your Firestore host from the Settings object, is that correct?

@abhishekwebcode Can you please sign the CLA?
https://github.com/googleapis/nodejs-firestore/blob/main/CONTRIBUTING.md#contributor-license-agreements

hey @ehsannas i have signed the agreement now

There could be scenarios where the serialized settings object is useful. Could we a) make this behavior optional, b) redact only the key, or c) make the key non-enumerable?

Thanks for the suggestion. I have updated the code to only redact the credentials part of the settings object. PTAL @MarkDuckworth


set max request headers timeout

Currently we do not set this and should be set at gateways to prevent from clients creating idle streams without headers
If it is incorrect to reuse the same timeout - we can pick some other value.

[ ] Ambient
[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[ ] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure

I'm thinking whether this should actually be controlled/configured by the users in the gateway configuration rather than hardcoding some arbitrary value.
Which will also allow disabling the timer (as it is today) for users who actually need it (can't think of a scenario though).

I'm thinking whether this should actually be controlled/configured by the users in the gateway configuration rather than hardcoding some arbitrary value.

This is not arbitrary value.  This is the default request timeout we set for all routes (including gateways) if it is not specified  in virtual services. It matches the current default behaviour.

Doesn't sidecar have the same issue?

It typically does not. This mainly needed for untrusted downstreams.

Wondeing can streamIdleTimeout be used to prevent idle stream leak? It has a default value 5m

If it is completely idle - stream idle timeout would detect. But if a rogue client sends partial headers once in a while (breaking stream idleness), the connection would be blocked forever. That is the reason this timeout has been added in Envoy

It typically does not. This mainly needed for untrusted downstreams.

I don't think there is an assumption that we trust downstreams traffic even in sidecars. The only thing we trust is their identity, but we don't have an knowledge of whether those identities are trusted to only send well-formed traffic ("we" here is istio project, in a specific users specific deployment they may have made that decision).

I do not know if it is really needed for sidecars where clients are "known" by their identities.

Istio is to resolve zero-trust, think about the case there is no auth because client has no sidecar.

+1 for sidecar
Looks good to me, normally requestTimeout > requestHeaderTimeout, we set  a loose timeout for header looks good

Added for sidecar inbound also

Thinking about this more - we disable all timeouts at sidecar inbound. Not sure if it is a good idea to add this. So put on hold

After a second thought, I think it is not safe to add it for sidecars as we disable all timeouts. Only added for gateways.
@howardjohn @hzxuzhonghu PTAL and see what you think?

After a second thought, I think it is not safe to add it for sidecars as we disable all timeouts.

From security view, i think the risk is same for both.

Closing this as it is getting difficult to get consensus on this


Clicking config opens a empty terminal

I'm relatively new to Linux and need access to onedrive however I'm not sure how to install RClone browser as I've followed instructions but when I hit the bit where it says to press the config button nothing shows when I click the config button. It loads a empty terminal which you can see in the screengrab and from the instructions I've seen and screenshots it looks like there should be text in the terminal but I'm not seeing anything.

What Linux distro your are on?

On 4 Jul 2021, at 18:54, FC360D @.***> wrote:
I'm relatively new to Linux and need access to onedrive however I'm not sure how to install RClone browser as I've followed instructions but when I hit the bit where it says to press the config button nothing shows when I click the config button. It loads a empty terminal which you can see in the screengrab and from the instructions I've seen and screenshots it looks like there should be text in the terminal but I'm not seeing anything.
https://user-images.githubusercontent.com/43715897/124394765-c3ce2800-dcf8-11eb-95f7-13cf71ce0a07.jpg
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/kapitainsky/RcloneBrowser/issues/172, or unsubscribe https://github.com/notifications/unsubscribe-auth/ADXJHOV2MKIUGAHRT3MUQETTWCN37ANCNFSM47ZMWN5A.

I'm on Lubuntu 20.4 which I think is the same as Ubuntu 20.4 LTS distro.


require-default-props: Configure for rule is invalid

Hi there. I am trying to use react/require-default-props to ensure that non-required fields have defaults set as argument defaults, like this:
interface ComponentProps {
    prop1: number;
    prop2: string;
    prop3?: string;
}
function Component({ prop1, prop2, prop3 = 'default-value' }) {}

The documentation rule options state that I can do that like this:
{
    rules: {
        'react/require-default-props': [ 'error', { functions: 'defaultArguments' } ]
    }
}

However, eslint says this is invalid:
Error: .eslintrc.js:
        Configuration for rule "react/require-default-props" is invalid:
        Value {"functions":"defaultArguments"} should NOT have additional properties.

Is the documentation out of date or am I just interpreting how to use it incorrectly? Thanks!

This was just released in v7.30.0. What version are you using?
Always check the docs on the tag matching the version you're using, not on master - everywhere on github.

Yep, I was a bit behind on my version. Sorry about that. I just set the project up this week, so I didn't have it in mind that I would end up needing to update for this feature that just released 20 hours ago at the time of this comment. Cheers!


add pr

Ⅰ. Describe what this PR does
Ⅱ. Does this pull request fix one issue?

fixes #XXXX
Ⅲ. List the added test cases (unit test/integration test) if any, please explain if no tests are needed.
Ⅳ. Describe how to verify it
Ⅴ. Special notes for reviews

@zysbksn please add description of your pr. And I'm wondering why  there are so many file changes.


Sidebar Example

Hi,
I really like Stipple and I want to use it with StippleUI to create an app with a sidebar which includes different dashboards, but I am not sure which elements to use for a sidebar. I have tried to use drawer, but nothing showed up.
Which elements should I use to create a sidebar?
Thanks!

Thank you - I'm not sure a sidebar would need to be a component, but rather just a section on the page (a div). What would it be its behaviour?
StippleUI uses Quasar - any of these components would match the sidebar? https://quasar.dev/vue-components

Would q-drawer come close to what you want?

I need something like this, where I can choose different contents on the left like in a sidebar. I tried drawer but could not receive a running example.

Hi :)
I have managed to create an example with drawer. I made an example for larger applications with hiding elements.
I added some elements like avatar, scrollarea, toolbar and tabs.
https://user-images.githubusercontent.com/5750890/113455847-ef94f580-940b-11eb-8d64-a4aa741dc37e.mp4
You find the code here: https://github.com/JulesBlubb/DrawerDemo/blob/main/PR_example.jl

Hi @hhaensel :)
I have managed to create an example with drawer. I made an example for larger applications with hiding elements.
I added some elements like avatar, scrollarea, toolbar and tabs.
https://user-images.githubusercontent.com/5750890/113562096-d22b7b80-9605-11eb-96bd-22e0b171f483.mp4
You find the code here: https://github.com/JulesBlubb/DrawerDemo

@JulesBlubb have you figured out how to implement the sidebar? I'm looking for this too.

This should do it:
using Stipple, StippleUI, StippleCharts

using DataFrames

data = DataFrame(Costs = [44, 55, 13, 43, 22])

@reactive! mutable struct Example <: ReactiveModel
    plot_options::R{PlotOptions} = PlotOptions(
        chart_type = :bar,
        labels = ["Team A", "Team B", "Team C", "Team D", "Team E"],
        yaxis_max = 60,
        yaxis_min = 0,
        ), READONLY
    barchart::R{Vector{PlotSeries}} = [PlotSeries("name", PlotData(data.Costs))], READONLY

    drawer::R{Bool} = false
end

Stipple.register_components(Example, StippleCharts.COMPONENTS)
model = Stipple.init(Example)

function handlers(model)
    # onbutton(model.drawer) do
    #     model.drawer[] = ! model.drawer[]
    # end
    on(model.isready) do ready
        ready || return
        notify(model.plot_options)
    end
    model
end

function ui(model)
    dashboard(model, [
        StippleUI.Layouts.layout([
            header([
                    btn("",icon="menu", @click("drawer = ! drawer"))
                    title("Example App")
            ])

            drawer(side="left", v__model="drawer", [
                list([
                    btn("", style = "width: 30px;",icon="menu", @click("drawer = ! drawer"))
                    item([
                        item_section(icon("inbox"), :avatar)
                        item_section("Inbox")
                    ], :clickable, :v__ripple)
                    item([
                        item_section(icon("outbox"), :avatar)
                        item_section("Outbox")
                    ])
                ])
            ])
            h3("Example Plot")
            row(StippleCharts.plot(:barchart, options = :plot_options))
        ])
    ])
end

route("/") do
    model |> handlers |> ui |> html
end

up(8000, open_browser = true)

Thanks for the fast response @hhaensel ! I'll give it a try as soon as I finish some pending issues I have


misc: Refactor weekly-tests.yaml

This adds a matrix to the weekly tests in order to make the file cleaner.
Change-Id: I830a0bf8b7d0406e9c377fedf2a7edfa5beabf40

Note: This can only be merged after the refactoring to the testing directory goes through, as it uses the updated names for the directories.

blocked by #156


fix(type): add NULL type and fix type analysis

This PR adds a NULL variant to the DataTypeKind enum, as the type of NULL value. And fixes the remaining failed tests in type analysis.

In general NULL should be part of the type itself (e.g., int null, varchar null) instead of a separate data type kind? Though I'm okay with this change.

In general NULL should be part of the type itself (e.g., int null, varchar null) instead of a separate data type kind?

Do you mean "nullable"? (e.g. int nullable, varchar not nullable)
The null type in this PR means an "unknown" type for a single NULL value. Previously we represent this by a None in Option<DataType>, and now it will be DataType { kind: Null, nullable: true }.

Oh that seems fine!


Add warning status in the output schema completedWith enum

Fixes FLPATH-1846
Add warning status in the output schema completedWith enum and update workflows accordingly

@masayag No I thought it would be a quick fix but...no

@masayag FYI I removed the changes made on Konflux (moved to #501 and #502 )


Decode URL params

We can discuss it here.

@ivan-tymoshenko the current state is:
        const router = new ProstoRouter()
        const handler = () => {}
        router.get('/🍌/:id', handler)

        console.log(router.toTree())

        expect((router.find('GET', `/%F0%9F%8D%8C/${singleEncode('#')}`)).ctx.params['id']).toEqual('#')
        expect((router.find('GET', `/%F0%9F%8D%8C/${doubleEncode('#')}`)).ctx.params['id']).toEqual(singleEncode('#'))

Both expectations are met.
Do you see any further issues with the way it handles paths?

const router = new ProstoRouter()
const handler = () => {}
router.get('/:id/🍌', handler)

console.log(router.toTree())

expect((router.find('GET', `/${singleEncode('#')}/%F0%9F%8D%8C`)).ctx.params['id']).toEqual('#')
expect((router.find('GET', `/${doubleEncode('#')}/%F0%9F%8D%8C`)).ctx.params['id']).toEqual(singleEncode('#'))

Does it also work? I'm sorry, can't test by myself. I can't even install deps.

@ivan-tymoshenko yes it does. Added this to test file as well.

@ivan-tymoshenko published v0.0.9

const { ProstoRouter } = require('@prostojs/router')

const router = new ProstoRouter()

router.on('GET', '/🍌/:param', () => 'ok')
console.log(router.lookup('GET', '/🍌/test')) // undefined

@ivan-tymoshenko according to rfc3986 the URI contains only ASCII characters. All the rest is encoded in %-notation. So you must call "lookup" method with proper URI. In this case it must be '/%F0%9F%8D%8C/test'.
I don't find it necessary to cover up such synthetic cases because the main purpose for this lib is to resolve URI transferred over HTTP which is impossible to be in UTF-8.

ok
const { ProstoRouter } = require('@prostojs/router')

const router = new ProstoRouter()

router.on('GET', '/~/:param', () => 'ok')
console.log(router.lookup('GET', '/%7E/test')) // undefined

@ivan-tymoshenko alright, that one (~) is more tricky. encodeURI('~') returns '~'. So the browser shouldn't encode that character. Although I assume we must take '/~' and '/%7E as the same URI. I'll see what I can do about it.

It's an example from your issue, where you said that according to the RFC '~' === ''%7e'. And where you said that we never should use the encodeURI function when setting a route. And technically you did it right now in your fix.
https://github.com/delvedor/find-my-way/issues/206#issue-1016799776

@ivan-tymoshenko nice one. Looks like I'm running in circle.

I spent some time trying to deal with it. What I can tell you is there is no good way to do this. What I suggested in the PR I tagged you is to add the encode option. When it equals true, we will encode static parts of the route before inserting it (just like in your fix). And there also '~' not equal to '%7e'. If you want to match '/%7e', you need to set the route manually '/%7e'.
path-to-regexp works in the same way.
If you find out a better solution, please ping me here. If not please come back to the PR (https://github.com/delvedor/find-my-way/pull/253) and write a short conclusion about it. It would be really important because not everyone has time to go deep into this question and your opinion would be important, bc you are the one who found the problem and raise the issue (https://github.com/delvedor/find-my-way/issues/206).
Thanks for your help.

@ivan-tymoshenko yes I can understand the depth of the problem now.
And I think I agree with my 206 comment that we shouldn't ever encodeURI when setting a route because by doing so we bring so much of inconsistency to the URI resolving procedure. I will try to find a better solution.
Thank you for this challenge.

@ivan-tymoshenko I think I found a solution.
Initially everything worked fine except double decoding part. So I reverted everything to how it was before.
Then, as long as the issue is only with double decoding, which can only happen when you use %25 in URI, I added one more thing to URI normalization process.
I replace %25 with some unique characters sequence in order to avoid it to be decoded if first place.
When setting up a route if it contains % I replace it as well (in order to keep it to matching to normalized URIs).
And when I do decode params I first replace unique characters sequence back to %25 and then do decodeURIComponent - so the decode for that particular one happens only once.
All the previous test cases pass.
I deployed v0.0.10

What if I set up to routes like this?
router.on('GET', '/‣PRC⁑', () => 'ok')
router.on('GET', '/%', () => 'ok')

This is also works but it shouldn't
router.on('GET', '/%', () => 'ok')
router.lookup('GET', '/‣PRC⁑')

@ivan-tymoshenko of course this is the downside.

/‣PRC⁑ and /% are identical due to implementation (I might find some better characters combination in future)
The combination used for replacement should be never used (I might deprecate it on lib level)
router.lookup('GET', '/‣PRC⁑') - here you use incorrect URI, as I said earlier URI can only contain ASCII symbols. All the rest characters are to be encoded with %-notation. So there is no way to send UTF-8 chars in HTTP request header .

It's a vulnerability right now. I can set router.on('GET', '/%', () => 'ok') and can get it by router.lookup('GET', '/‣PRC⁑'). It can help to get through some security filters.

@ivan-tymoshenko of course this is the downside.

/‣PRC⁑ and /% are identical due to implementation (I might find some better characters combination in future)
The combination used for replacement should be never used (I might deprecate it on lib level)
router.lookup('GET', '/‣PRC⁑') - here you use incorrect URI, as I said earlier URI can only contain ASCII symbols. All the rest characters are to be encoded with %-notation. So there is no way to send UTF-8 chars in HTTP request header .

And you think that it's a good solution? Restrict some path combinations?

Where do you get '/‣PRC⁑' from as an input? It can not be http request header. What's the use-case?

@ivan-tymoshenko of course this is the downside.

/‣PRC⁑ and /% are identical due to implementation (I might find some better characters combination in future)
The combination used for replacement should be never used (I might deprecate it on lib level)
router.lookup('GET', '/‣PRC⁑') - here you use incorrect URI, as I said earlier URI can only contain ASCII symbols. All the rest characters are to be encoded with %-notation. So there is no way to send UTF-8 chars in HTTP request header .

And you think that it's a good solution? Restrict some path combinations?

Yes I think it's the best I could find. Do you have any better solution?

First of all, it doesn't matter. If you don't use it, it doesn't mean that someone can't. If we will follow this logic, I can ask you why someone will use '%7E' instead of '~'.

Secondly, as I told you I can put this combination manually instead of "%" to get through some security filters.

@ivan-tymoshenko I follow the RFC 3986 and the purpose of the lib is to provide a solid, fast and robust way of routing URIs to proper handlers. The input for lookup method is intended to be a HTTP request header. I can't get what kind of use-cases you're trying to cover.

Yes you can send ~ as well as %7E using browser string or whatever lib like axios. You can even send '%31%32%33' which is the same as '123'. This all should work fine. But you can not send request with URI like ‣PRC⁑ - simply no possible via HTTP request.

const http = require('http')
const { ProstoRouter } = require('@prostojs/router')

const router = new ProstoRouter()

router.on('GET', '/%', () => 'ok' )

const server = http.createServer((req) => {
    console.log(router.lookup(req.method, req.url))
})

server.listen(3000, err => {
    http.get('http://localhost:3000' + '/‣PRC⁑', (res) => {})
})

const http = require('http')
const { ProstoRouter } = require('@prostojs/router')

const router = new ProstoRouter()

router.on('GET', '/%', () => 'ok' )

const server = http.createServer((req) => {
    console.log(router.lookup(req.method, req.url))
})

server.listen(3000, err => {
    http.get('http://localhost:3000' + '/‣PRC⁑', (res) => {})
})

So? It was transformed to %E2%80%A3PRC%E2%81%91 first before request was sent. Then during handling it was decoded once and interpreted as % as I mentioned that in this particular implementation % and ‣PRC⁑ are synonyms.
No double decoding possible. I find this as an expected behavior.
The only thing I would do is to try to think of better combination for ‣PRC⁑ in order to prevent potential collisions with some real routs. or it can be even one character like ‱. And this will be the limitation of this lib: the % and the ‱ are synonyms.

Secondly, as I told you I can put this combination manually instead of "%" to get through some security filters.

.

'use strict'

const http = require('http')
const express = require('express')
const app = express()

app.use((req, res, next) => {
  if (req.url === '/only-admin-path/%') {
      console.log('Only admin access!!!')
  }
  next()
})

app.get('/only-admin-path/%')

app.listen(3000)

http.get('http://localhost:3000' + '/only-admin-path/%', (res) => {})
http.get('http://localhost:3000' + '/only-admin-path/‣PRC⁑', (res) => {})

This is an example of what I mean.
'use strict'

const http = require('http')
const express = require('express')
const app = express()

app.use((req, res, next) => {
  if (req.url === '/only-admin-path/%') {
      console.log('Only admin access!!!')
  }
  next()
})

app.get('/only-admin-path/%', () => {
    console.log('Hello admin!')
})

app.listen(3000)

http.get('http://localhost:3000' + '/only-admin-path/%', (res) => {})
http.get('http://localhost:3000' + '/only-admin-path/‣PRC⁑', (res) => {})

@ivan-tymoshenko  I agree the solution I provided is not perfect and I agree it has the downside you described.
This kept me busy trying to again bring a better solution.
Now:

when registering route I replace % with %25
when normalizing URI on lookup I replace %25 with %2525
everything works fine

This time I can't think of any downsides here.
Please use your critical thinking to find any issues with this approach.
Deployed as v0.0.11

Actually, I like this one. I will think about it. Thanks for your help.
Maybe it will make sense to implement a custom decodeURI function that will decode everything except "%25" instead of encoding + decoding '%25'. It can be faster, but I'm not sure.

AFAK the decodeURI is already pretty fast. I doubt you can rewrite it in JS in any better way. Please keep me posted if you find more optimal way.


python3Packages.browser-cookie3: 0.6.4 -> 0.7.6

Motivation for this change
Version update
Things done

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

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

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

Notify maintainers
cc @

nix-review builds, macos 10.13.

@GrahamcOfBorg build python3Packages.browser-cookie3


Server side validation

I opted to remove permissions for client side inserts/updates using the gql api (for the most part) - reason being that otherwise you could just curl malicious data anyway,
forms are submitted using form actions
because of the above, we're using the service role for inserts/updates. Apart from adding service role permissions for corresponding crud operations using the gql api, I also added checks to prevent crud operations on disallowed rows e.g. user can only modify apps they own - web/lib/permissions/

There are still forms that need to be moved to server side validation, opted not do add them now bc the PR grew too big anyway

@andy-t-wang Can you review?

yes I will look today. Lmk if this seems inline with security recs too

This is actually a huge improvement to our overall dev portal structure!

Do not merge till @ottodimi is done with his proxy stuff. We need to heavy QA this in staging


Add oscillating pattern progress indicator

Codecov Report

Merging #346 (a42fcd9) into main (94ec59c) will increase coverage by 0.09%.
The diff coverage is 74.03%.

@@            Coverage Diff             @@
##             main     #346      +/-   ##
==========================================
+ Coverage   74.98%   75.08%   +0.09%
==========================================
  Files          78       79       +1
  Lines        3958     4037      +79
  Branches      677      688      +11
==========================================
+ Hits         2968     3031      +63
- Misses        588      595       +7
- Partials      402      411       +9

Impacted Files
Coverage Δ

...com/alexvanyo/composelife/util/BoundsExtensions.kt
82.45% <0.00%> (-9.71%)
:arrow_down:

...fe/ui/action/settings/AlgorithmImplementationUi.kt
61.79% <0.00%> (ø)

...omposelife/ui/action/settings/DarkThemeConfigUi.kt
59.57% <0.00%> (ø)

...ypoints/algorithm/GameOfLifeAlgorithmEntryPoint.kt
100.00% <ø> (+14.28%)
:arrow_up:

...ts/dispatchers/ComposeLifeDispatchersEntryPoint.kt
100.00% <ø> (+16.66%)
:arrow_up:

...ts/preferences/ComposeLifePreferencesEntryPoint.kt
100.00% <ø> (ø)

...otlin/com/alexvanyo/composelife/model/CellState.kt
68.18% <25.00%> (-6.82%)
:arrow_down:

...selife/ui/component/GameOfLifeProgressIndicator.kt
73.46% <73.46%> (ø)

...exvanyo/composelife/patterns/OscillatorPatterns.kt
100.00% <100.00%> (ø)

...lexvanyo/composelife/ui/InteractiveCellUniverse.kt
74.07% <100.00%> (ø)

... and 6 more

:mega: Codecov can now indicate which changes are the most critical in Pull Requests. Learn more


New FE behaviour for the require and facets functionality within AOTF

Once this backend issue is completed, the AOTF UI should be adapted to new behaviour for the require functionality.
This ticket will follow up FE progress.

[x] Change 'Require' to 'Include'
[x] Add a tooltip: This results in an 'OR' filtering between the data sources
[x] Adding a (beta) in the Search disease/target specific filter heading (facets heading)
[x] Change a column in the facets category to Target ID (ENSG)

Hey @chinmehta , could you please modify the facets tooltip to

[ ] in the target page

Please search by disease or filter by disease category. Example: Measurement
AND filter across categories; OR filter within a category

[ ] in the disease page

Please search by target or filter by target category. Example: Enzyme
AND filter across categories; OR filter within a category

Adding tooltip on input label is not recommended ui, as discussed we are adding an info button next to facets.

modified to include all the text in the Tip


[Bug]: SSL not working with CloudFlare

Error Message and Logs
Hey guys,
I'm not sure if that's a bug or I'm just missing something, but I'm desperate to make it work (that's why I just added a bounty too).
Steps:

installed coolify
created a CF tunnel as here, except that I used http://coolify-proxy:80 instead of http://localhost:80, as I've seen here; PS: also switched to 'Full' encryption mode & manually created the DNS CNAME for the wildcard
created a cloudflared container with the token from the previous step:

services:
  cloudflared:
    container_name: cloudflare-tunnel
    image: 'cloudflare/cloudflared:latest'
    restart: unless-stopped
    command: 'tunnel --no-autoupdate run'
    environment:
      - 'TUNNEL_TOKEN=${CLOUDFLARE_TUNNEL_TOKEN}'
    networks:
      - coolify
networks:
  coolify:
    external: true

Result: If I create a web app with the domains http://web.app.com,https://web.app.com, the http URL will work, but the https one won't.
I tried modifying the default Traefik configuration, an I also tried the Caddy proxy, but I couldn't make it work whatever I tried.
Am I missing something?
Steps to Reproduce

Mentioned above

Example Repository URL
No response
Coolify Version
v4.0.0-beta.376
Are you using Coolify Cloud?
No (self-hosted)
Operating System and Version (self-hosted)
Ubuntu 24.04
Additional Information
No response

/attempt #4552

Hey! @johnykes
It sounds like you're on the right track, but here are a couple of things to check:

Traefik Entry Points: Make sure your Traefik config has entry points set up for both HTTP and HTTPS. Sometimes, it just needs that little nudge to handle secure traffic.

Here is an example:
entryPoints:
web:
address: ":80"
websecure:
address: ":443"

Cloudflare Settings:  Since you're on 'Full' mode, ensure that your backend (Coolify) has a valid SSL certificate. If it doesn't, you might want to try switching to 'Flexible' temporarily to see if that helps.

3.Logs: Check the logs for Traefik and Cloudflare for any error messages. They can often point you in the right direction.

Hey @aybanda,
Thank your your reply, but

Already tried this. Result: http won't work anymore; https still won't work
'Flexible' mode is not an option, because the origin-cloudflare transfers won't be secure
Checked logs, no errors

check these few more in case:
Enable "Always Use HTTPS" in SSL/TLS settings
SSL/TLS > Edge Certificates:

Always Use HTTPS: ON

Automatic HTTPS Rewrites: ON
labels:

"traefik.enable=true"
"traefik.http.routers.myapp.rule=Host(web.app.com)"
"traefik.http.routers.myapp.entrypoints=web,websecure"
"traefik.http.routers.myapp.tls=true"

Ensure your DNS records in Cloudflare are properly configured with proxy enabled

These labels to be added to proxy or to each app?
"traefik.enable=true"
"traefik.http.routers.myapp.rule=Host(web.app.com)"
"traefik.http.routers.myapp.entrypoints=web,websecure"
"traefik.http.routers.myapp.tls=true"

/attempt https://github.com/coollabsio/coolify/issues/4552
This issue is solved on Support channel of Coolify community server (link to the solution - https://discord.com/channels/459365938081431553/1316486632714539061 )

/attempt #4552
This issue is solved on Support channel of Coolify community server (link to the solution - https://discord.com/channels/459365938081431553/1316486632714539061 )

Issue solve thanks to ShadowArcanist
Had to enable
Always Use HTTPS
Opportunistic Encryption
Automatic HTTPS Rewrites

change the browser for some reason (something cached, even if I use incognito

Thank you!


there should be a kv feature to

read and write the value to a file. To restore or load a file like .aws/cretentials

is implemented via kv "KEY@file[tag]" arguments


Fixing scroll event rate

Opening a new PR with a proper branch, some improvements and squashed commits.
OSX generates many events when scrolling with the touch pad. This is
also the case for mouses that have a continuous wheel. Lwjgl3 reports
small movements with a scroll value lower than 1. The Lwjgl3 backend
rounds these values to 1 or -1. If there are many such "small" events
the scrolling becomes ultra fast.
The current fix accumulates the scrolling values reported by Lwjgl3
and sends a Gdx scroll event when the summed value is greater than

This is done until the accumulated value is lower than 1. This
approach also supports accelerations which are reported as scrolling
value greater than 1. These values become a stream of 1 or -1 scroll
events.

There is also an event which is triggered when the user starts acting
on the wheel or when the user changes the scrolling direction. This
gives a better reactivity, you don't have to scroll until you reach 1
for the scrolling to start.
Cheers,
Julien

Why are scrollYRemainder and lastScrollEventTime class level fields when they are only used by GLFWScrollCallback?

For no reason. I'll move them inside GLFWScrollCallback.
Thanks.
Julien

We are currently testing your PR with our editor (on OSX and Windows), and it improves mouse wheel scroll behavior quite well - so I think we can merge it.

Great ! 👍


fixsupport access glue iceberg with credential list

Proposed changes
from https://github.com/apache/doris/pull/30292

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

run buildall


Seach feature is not highlighting the exact page

When I search anything, the reader does not navigate me to the actual result sub page, it navigates me to the start of the page.

sorry, please wait!

Search is not working @taku33 , can you please fix that?


[Bug] Issue Field History Incremental Load Missing Rows

Is there an existing issue for this?

[X] I have searched the existing issues

Describe the issue
The incremental load for the issue field history tables seems to be missing history for less than 1% of Jira issues.  The missing records begin on the date the Jira issues was created.  Updates that occur to fields that we are tracking do cause the Jira issue to then show up in the jira__daily_issue_field_history table, but there will only be records for the days after the update was made.
Note - A full refresh always resolves this problem.
Example for a Jira issue created on March 25th, and updated on March 27th. Our pipeline ran every day during this period, and there were no failures.
jira__daily_issue_field_history:

All of the intermediate tables only contain data from March 27th onward. I won't post all of them for brevity, but here is int_jira_combine_field_histories:

All of the records are contained in our data lake source that is being queried, and the correct result (all history) appears when I run the compiled query for int_jira_combine_field_histories.
Relevant error log or model output
No response
Expected behavior
It is expected that there would be a row present for each day the issue has been open.
dbt Project configurations
jira:
+schema: jira
intermediate:
+schema: jira
int_jira__issue_epic:
+enabled: false
jira__daily_issue_field_history:
+enabled: false
jira_source:
+schema: jira
stg_jira__project:
+enabled: false
stg_jira__issue:
+enabled: false
jira_issue_history_buffer: 1200
issue_field_history_columns:
- Parent Link
- Resolution
- Reporter
- Assignee
- Project
- Creator
- Priority
- Team
- Epic Link
- Description
- Summary
- Story Points
- Original Estimate
- Σ Original Estimate
- Remaining Estimate
- Σ Remaining Estimate
- Time Spent
- Σ Time Spent
- Work Ratio
- Due date
- Labels
- Bug Severity
- Production Bug?
- Escaped Bug?
- Bug Origin
- Bug Scope
- Bug Priority
- Last Viewed
- Resolved
- Updated
- Status Category Changed
- Created
- Epic Category
- Theme
- Scope Change
- Quarter
- Original Target Date
- Latest Target Date
- DevOps Story Points
- T-Shirt Size
- UX Required?
- Request Origin
- X-Delivery Teams
- Cross Teams
- Security Review Required
- CCB Category Level
jira_using_sprints: false
Package versions

package: fivetran/jira
version: [">=0.15.0", "<0.16.0"]

What database are you using dbt with?
snowflake
dbt Version
1.7
Additional Context
No response
Are you willing to open a PR to help address this issue?

[X] Yes.
[x] Yes, but I will need assistance and will schedule time during our office hours for guidance
[ ] No.

Hello @eli-reber ! Thank you for raising this issue with us, we were able to reproduce a version of this locally so it is definitely something we will need to account for in our models.
After some investigation, we believe that a feature that will introduce lookback windows for incremental models could potentially solve this issue. We are working on that issue this sprint, and once we have a branch in a ready state we would love for you to test it out!
We will keep you in the loop for any future developments. Let us know if you have any questions or thoughts!

Thank you for the update!  I'd be happy to help test your branch when you're ready, just let me know.

Hi @eli-reber I have a working test branch that incorporates the lookback window @fivetran-avinash mentioned, which you can install using the branch below. I'll also note that the main purpose of my updates is to help performance in addition to data quality, so I'd be interested in any feedback you have from that perspective as well!
  - git: https://github.com/fivetran/dbt_jira.git
    revision: feature/performance-enhancement
    warn-unpinned: false

Hi @eli-reber this update has been released in the latest version, which can be installed with the below snipped.
packages:
  - package: fivetran/jira
    version: [">=0.17.0", "<0.18.0"]

I am closing out this issue, but feel free to ping us in this thread with any additional comments or questions!


"Google:HTML Not all asset views lie inside the native ad view" message

Hi there,
Here is the way I call native admob:
...
                    Container(
                      height: 300,
                      child: NativeAdmob(
                        adUnitID: adMobRepository.nativeAdId,
                        loading: ProgressIndicatorWidget(),
                        type: NativeAdmobType.full,
                        //controller: _nativeAdController,
                      ),
                    ),
...

What I have just noticed - right after the ad is successfully loaded, I have this message received in the console:

I/Ads     ( 9832): Received log message: Google:HTML Not all asset views lie inside the native ad view. This indicates an integration problem. Such implementations will not be supported in the future. Please make sure that all native ad assets are rendered inside the native ad view.

Please note, the app runs on an emulator but with real ad unit id code.

@angel1st
I think the container height is not enough for all ad asset views. You can increase the container height to check if still have log ?

@phungrk - the message remains despite of container's height.

The same message appears for me as well despite of the container's height(IOS Simulator).

If this help :
https://github.com/sakebook/flutter_native_ads/commit/675bf466d87ba7e35ac25babc7921819ffeeeb10#diff-06572a96a58dc510037d5efa622f9bec8519bc1beab13c9f251e97e657a9d4ed

If this help :
https://github.com/sakebook/flutter_native_ads/commit/675bf466d87ba7e35ac25babc7921819ffeeeb10#diff-06572a96a58dc510037d5efa622f9bec8519bc1beab13c9f251e97e657a9d4ed


Azure CLI error while connecting azure

Describe the bug
(ResourceNotFound) The Resource 'Microsoft.DBforMySQL/servers/testingmysql' under resource group 'ISOLATE-ENV' was not found. For more details please go to https://aka.ms/ARMResourceNotFoundFix
Code: ResourceNotFound
Message: The Resource 'Microsoft.DBforMySQL/servers/testingmysql' under resource group 'ISOLATE-ENV' was not found. For more details please go to https://aka.ms/ARMResourceNotFoundFix

I tried different commend to check if the issue with command or CLI but i see the same error

Related command
az mysql server show --resource-group Databases-Resources --name geethatest.mysql.database.azure.com
az mysql flexible-server firewall-rule list --resource-group Databases-Resources --name geethatest.mysql.database.azure.com
Errors

Issue script & Debug output
az mysql server show --resource-group Databases-Resources --name geethatest.mysql.database.azure.com
az mysql flexible-server firewall-rule list --resource-group Databases-Resources --name geethatest.mysql.database.azure.com
Expected behavior
Should list the server details
Environment Summary
Prod
Additional context
No response

Thank you for opening this issue, we will look into it.

any update on this


Twitter Stream Agents - Filter & Language

Hi,
thanks for a great piece of software, but i have some trouble with my twitter agents. I'm using the twitterstreamagent and twittersearchagent and only want to get german tweets which got retweeted:
{
"filters": [
#christmas
],
"lang": "de",
"retweeted": true,
"generate": "events",
"expected_update_period_in_days": "2"
}
But im getting results in english and not retweeted. Is there something i missed?
I wish you nice holidays!
Regards,
Stefan

Hey @upperhead, welcome!  I don't think we support the lang option right now.  That probably wouldn't be too hard to add if you wanted to give it a try, but right now we don't pass that on to Twitter.

Hi @cantino, thx for your reply. Could you please give me a hint how to add the properties to my agent? I found the ruby file for my agent, but adding lang, retweet_count etc. to opt is far to easy i think :).

You could try changing line 187 of twitter_stream_agent.rb to be
          :path    => "/1/statuses/#{(filters && filters.length > 0) ? 'filter' : 'sample'}.json?track=#{filters.map {|f| CGI::escape(f) }.join(",")}&language=en,fr",

(for example)
if that works, we can wire it through from the Agent.  It wasn't obvious to me if the Twitter API accepts language on the filter endpoint.  It probably does?  https://dev.twitter.com/streaming/overview/request-parameters#track

Did you try that code @upperhead?


Visual Studio reports DEP1700 while trying to deploy the app

Describe the bug

The build will succeed and the deployment will fail with the message above, this happens on both .NET 5 and 6, however, it seems that it only happens on my machine, not on others, (while we are testing on the same code) I've tried to rebuild the solution, restart vs, reboot computer, reinstall vs but none of them works. My development is suspended due to this issue and is impossible to proceed.
Steps to reproduce the bug

Create a Win UI 3 packaged app in Visual Studio
Build and run

Expected behavior
The program will build and run successfully
Screenshots
No response
NuGet package version
1.0.0
Windows app type

[ ] UWP
[X] Win32

Device form factor
Desktop
Windows version
Insider Build (xxxxx)
Additional context
Visual Studio version: VS 2022 17.0.4
.NET version: .NET 6.0.100
Windows version: 22000.376
it seems that this problem happens only on my computer, I'm looking forward to find a way to reset my development environment, I've tried several methods mentioned above and none of them work

It's due to the corrupted unget cache, clear cache solves the problem


Add missing strings function and their tests

Codecov Report

Merging #7 into master will increase coverage by 1.21%.
The diff coverage is 99.21%.

@@             Coverage Diff              @@
##             master       #7      +/-   ##
============================================
+ Coverage     94.61%   95.83%   +1.21%
- Complexity      170      204      +34
============================================
  Files             6        6
  Lines           353      480     +127
============================================
+ Hits            334      460     +126
- Misses           19       20       +1

Impacted Files
Coverage Δ
Complexity Δ

src/Traits/Strings.php
99.22% <99.21%> (-0.78%)
35 <34> (+34)

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 1c3a002...37a5526. Read the comment docs.


In the Dark Mode, Reviews and Ratings are not visible

I can fix this. Please assign me this issue under gssoc 24.

Proceed with the issue @aditi9402

Hi @abhay-raj19  can you please assign this issue to me?

@Hitesh4278 @abhay-raj19  I've already completed it and raised a PR as well.

Okay.


Update Dockerfile to support Python 3.10

Fixes #11
Challenge here was getting pip installed correctly. Counterintuitively
the answer was installing python3.10-venv, which includes ensurepip
which can be used to install the latest pip.

With this setup pip installs to /usr/lib/python3.10/site-packages, but Python itself is looking in /usr/lib/python3.10/dist-packages, so it can never see the packages.


API to build non-sequential models

Hi there, and thanks for flax.
Flax's quite intuitive, but I noticed that it does not provide an API to handle branching in the model, allowing only sequential models.
I always find myself writing small patches, and I put together a quick PR as a proof of concept to test the idea. You can find it here: https://github.com/google/flax/pull/3243. Do you think it is something that you would consider adding in? Let me know if you like the idea, I can polish the PR.
Example usage below or in a colab:
module = nn.Sequential(
    [
        nn.Dense(4),
        Split(2),
        Parallel((nn.Dense(2), nn.Dense(2))),
        Merge(jnp.sum)
    ]
)

Thanks!

Hey @epignatelli, thanks for taking interest! Historically Flax has tried to reduce the use of combinators, even Sequential was added very late. Not sure this proposed direction aligns with the Flax Philosophy.

Thansk @cgarciae!
What pattern that aligns with the flax philosophy would you use now to do the same thing? Maybe I am missing some pieces.


Messages

:eyeglasses: @ajacksified
goes with https://github.com/reddit/reddit-mobile/pull/275

:fish:

safe to deploy on its own with https://github.com/reddit/reddit-mobile/pull/364


[#275] Seed DB with test data

Purpose
@elliotbonneville has requested some example data with which to test the Fidesops Admin UI.
Changes

Adds a data migration to seed some example policies and privacy requests
This migration adds a PrivacyRequest in every permutation of status and policy

Note
As this is done via a migration, it will be automatically run every time the migrations are run. As such, we shouldn't merge this to main. @elliotbonneville the best course of action here is to merge this branch into your fork, and include the migration in your local stack, then before merging the Admin UI back into main from your fork, we remove this migration.
To execute the migration:

SSH into the app server shell: make server-shell
Navigate to the Alembic source cd src/
Apply the migration: alembic upgrade head
Unapply with migration: alembic downgrade 906d7198df28

Checklist

[ ] Applicable documentation updated (guides, quickstart, postman collections, tutorial, fidesdemo, database diagram)
[ ] Good unit test/integration test coverage
[ ] This PR contains a DB migration. If checked, the reviewer should confirm with the author that the down_revision correctly references the previous migration before merging
[ ] The Run Unsafe PR Checks label has been applied, and checks have passed, if this PR touches any external services

Ticket
Fixes #275

@elliotbonneville has what he needs from this, so I'm going to close this now.