node 21 - No inline sourcemap produced

Acknowledgements

[X] I searched existing issues before opening this one to avoid duplicates
[X] I understand this is not a place for seek help, but to report a bug
[X] I understand that the bug must be proven first with a minimal reproduction
[X] I will be polite, respectful, and considerate of people's time and effort

Minimal reproduction URL
https://stackblitz.com/edit/node-huzszn?file=index.ts
Version
v4.7.1
Node.js version
v21.6.1
Package manager
npm
Operating system
Linux
Problem & Expected behavior
Hi,
When using node v21.6.1 it seems that no //# sourceMappingURL=... is added at the end of produced files.
Whereas it works with v20.11.0.
For example using npx tsx --inspect-brk example.ts and the source view in chrome://inspect I can see the //# sourceMappingURL=... line.
I have just tried a few times to switch between the 2 versions (using nvm) and observed this.
In my tsconfig.json, I'm using:
  "compilerOptions": {
    "strict": true,
    "module": "ESNext",
    "target": "ES2020",
    "outDir": "dist",
    "moduleResolution": "Node",
    "lib": ["ESNext"],
    "allowJs": true,
    "skipLibCheck": true,
    "esModuleInterop": true,
    "allowSyntheticDefaultImports": true,
    "inlineSourceMap": true,
  },
...

Related: https://github.com/privatenumber/tsx/issues/260
Cheers,
Pierre
Contributions

[ ] I plan to open a pull request for this issue
[ ] I plan to make a financial contribution to this project

I have the same problem with node 21 and vscode debugging.
I tested with a simple project. Only with an index.ts file with a few lines, and then started it with
tsx --inspect-brk index.ts
There is no tsconfig or other files.
I downgraded to brew node@20 and the debugging is now working in vscode.


Change the testuser password into a strong password in aggregation_view test

Description
Change the testuser password into a strong password in aggregation_view test
Issues Resolved

Relate https://github.com/opensearch-project/security-dashboards-plugin/issues/1429
Relate https://github.com/opensearch-project/security/pull/2557

Check List

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

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

Can someone also help me to backport this fix to 2.x and 2.7? Thanks!


Add magic constant "::class" like in PHP 5.5

Php 5.5 example:
namespace Name\Space;
class ClassName {}

echo ClassName::class;

echo "\n";

Output:
Name\Space\ClassName

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/11184781-add-magic-constant-class-like-in-php-5-5?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F280146&utm_medium=issues&utm_source=github).

Suggest to use only uppercase CLASS

Never mind.

:+1:

:+1:


datahub-frontend/bin/playBinary: no such file or directory: unknowng

Describe the bug
when i run docker/dev.sh
To Reproduce
Steps to reproduce the behavior:
Expected behavior
A clear and concise description of what you expected to happen.
Screenshots

Desktop (please complete the following information):
ubuntu20.04
Additional context
ERROR: for datahub-frontend-react  Cannot start service datahub-frontend-react: failed to create shim: OCI runtime create failed: container_linux.go:380: starting container process caused: exec: "datahub-frontend/bin/playBinary": stat datahub-frontend/bin/playBinary: no such file or directory: unknown

Hey. To run dev mode, you need to first build the artifacts before running.
Can you run
./gradlew :datahub-frontend:build
and
./gradlew :metadata-service:war:build
And then start running?
Basically dev.sh and dev-without-neo4j.sh was made for fast development, so we don't have to rebuild docker images every time we make a local code change. It was designed to pick up local build artifacts and run with it.

 If you still run into the error, let us know. Otherwise, we will close out this issue.

Closing due to inactivity.


Small save dialog on 4K

VSCode Version: 1.41.1
OS Version: Windows_NT x64 10.0.18363

Steps to Reproduce:

Have a second monitor with 4k res. Windows scaling is set to 150%
Open VSCode there.
Save a file.
--> The dialog will be tiny (as if it doesn't adhere to the 150% setting but sticks to 100%)

Does this issue occur when all extensions are disabled?: Yes

/duplicate https://github.com/microsoft/vscode/issues/85898


VEGA-1613 enable dark mode on non-embedded pages

This is required as the new search results page is not embedded, and instead redirects the user to the lpa frontend url. I have applied styling to achieve desired dark mode appearance on search results page

Codecov Report
Base: 73.40% // Head: 73.40% // No change to project coverage :thumbsup:

Coverage data is based on head (da9cad8) compared to base (06aa00c).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #238   +/-   ##
=======================================
  Coverage   73.40%   73.40%
=======================================
  Files          83       83
  Lines        3012     3012
=======================================
  Hits         2211     2211
  Misses        576      576
  Partials      225      225

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Bump to Kotlin 1.4.32 + Gradle bump

Closing after pulling changes into https://github.com/touchlab/Kermit/pull/75


SAMZA-2340: Introduce ContainerManager for handling validation for failures & starts of active & standby containers

Summary

Introduce a ContainerManager that can act a single entity maintaining state for validation, starts & failures of any requests relating to active & standby containers
The ContainerProcessManager & ContainerAllocator should call the ContaineProcessManager for any state validation & for fetching the next set of actions for successful & failed start & stop of active & standby cotntainer
ContainerManager should encapsulate the related state & logic behind StandBy & Active Container control actions

What does this PR ADD?

Introducer a new class ContainerManager
Removes StandByContainerManager from ContainerProcessManager & ContainerAllocator. Encapsulates StandByContainerManager inside ContainerManager
Moves the logic to determine the next set of actions on container stop failures & container launch failures from ContainerProcessManager to ContainerManager
Move the logic of validating allocated resources & expired requests from ContainerAllocator to ContainerManager

Rational

Centralizing the logic and state maintenance for active & standby containers helps to add & test new features faster and improve code readability

Note: This PR does not add any new behaviors in the AppMaster ecosystem.

@rmatharu please have a look

@rmatharu Let's hold off on merging this until we create the branch for the next release. Also @Sanil15, please confirm you've tested all the combinations Ray mentioned above before merge.

@rmatharu @prateekm Please see the doc: https://docs.google.com/spreadsheets/d/1v-fw0pHxKRobGkALDCno4FuPCsBhdepQ86vIGLHWu54/edit#gid=0
Covers all the test cases for testing


[9.x] SesTransport: use correct Tags argument

Illuminate\Mail\Transport\SesTransport uses SES v1's ses:SendRawEmail API but passes v2's EmailTags argument.
The call doesn't fail but the tags are not attached to the sent email.
(ConfigurationSet events don't include specified tags)
This PR fixes the parameter name to correct Tags.
Note: I didn't migrate to SesV2Client for backward compatibility.
Reference:
https://docs.aws.amazon.com/ja_jp/aws-sdk-php/v3/api/api-email-2010-12-01.html#sendrawemail
AWS SDK for PHP 3.x: SesClient->sendRawEmail()
I confirmed fired ConfigurationSet events include tags specified in Mailable's metadata() method.
class TestMail extends Mailable
{
    public function build()
    {
        return $this->metadata('env', config('app.env'))->text('emails.test');
    }
}

Mail::to(...)->send(new TestMail());

Before:
{
    "eventType": "Delivery",
    "mail": {
        ...
        "tags": {
            "ses:operation": ["SendRawEmail"],
            ...
        }
    },
    "delivery": ...
}

After:
{
    "eventType": "Delivery",
    "mail": {
        ...
        "tags": {
            "ses:operation": ["SendRawEmail"],
            ...
            "env": ["local"] // <- HERE
        }
    },
    "delivery": ...
}

Ping @kbond does this look good to you?

Good catch @Tietew!

does this look good to you?

I believe so, yes. I didn't realize there are multiple api versions. Symfony uses the AsyncAws client so it wasn't a straight copy in #41422.


Added version

<!-- Make sure you have read http://napalm.readthedocs.io/en/latest/contributing/index.html --!>

Hey @lubon can you rebase to solve the conflicts?


Qwik adaptor

What kind of change does this PR introduce?
This PR adds an adaptor for Supabase auth in Qwik. This makes it possible to use Supabase Auth in Qwik on the Client and in Server Side Rendering (SSR).
Usage instructions and examples are available in the README: https://github.com/TheDeveloper/supabase-auth-helpers/tree/qwik-adaptor/packages/qwik

The Qwik team has created a their own version which they will maintain. https://github.com/BuilderIO/qwik/tree/main/packages/supabase-auth-helpers-qwik


iris.analysis.AreaWeighted regrid speedup

🚀 Pull Request
Description

Aiming to speed up the regridding of chunked data with iris.analysis.AreaWeighted regridder

Moving more weight construction and construction of indices to be applied to the regridded cube all into the prepare step of the regridder, rather than the perform step (which gets repeated on chunked data)
See #4280 for context
To do:

[x] Check benchmark performance
[x] Consider additional testing and benchmarking
[x] Explain here which lines have just had indentation changes rather than being rewritten
[ ] Add to whatsnew

Consult Iris pull request check list

More details on the changes in lib/iris/experimental/regrid.py
_regrid_area_weighted_array
The nested for loop in _regrid_area_weighted_array is no longer present - instead the arguments of this function now include index_info which allows the src_data to be indexed to generate the src_area_datas (with an additional "masking" process to account for some values needing to be 0).
The logic around src_masked is broadly the same as before, but now much more condensed rather than spread over the for loops.
_regrid_area_weighted_rectilinear_src_and_grid__prepare
In _regrid_area_weighted_rectilinear_src_and_grid__prepare we now, in addition to forming the weights, form the indices that will be passed to _regrid_area_weighted_array. This includes a nested for loop (as I can't see a straightforward way to get rid of it) but this now only happens once and is only doing the quick building of indices rather than any indexing.
I have also removed the dynamic definition of a function to do some of the work within this function as it seemed pointless, though I can put it back to reduce the diff if that's helpful.

From the benchmarking CI:
All benchmarks:

       before           after         ratio
     [2dda3a93]       [cfce454f]
     <_base>
              ...
        10.2±0.1s          619±8ms    ~0.06  regridding.HorizontalChunkedRegridding.time_regrid_area_w

😍

Am I reading this right?  GloSea regridding will potentially run ~20 times faster?

When they pass in lazy data, yes @rcomer (I believe they're currently doing similar regrids by first realising the data as a workaround for this having previously been so slow)

That's awesome, thanks @wjbenfold!

Well done @wjbenfold and @stephenworsley, really great to see this massive improvement in performance 💨

On behalf of Team GloSea, thanks for all your work on this @wjbenfold @stephenworsley .


[Analyzer]: Unhandled enum el

How to handle?

@dorian3343 likes it Rust-way
Me:

I'm thinking about some cases where you have a lot of enum members and you really care about just a few of them
U would have to use them all in your code
Also have to update your code each time of them removed or renamed


Strings Have A 2 (Or More) Byte Type Overhead

https://data-oriented-house.github.io/Squash/docs/binary#strings
I realize the code is a simple #string + 2, though the initial TYPE_OVERHEAD variable made it seem like every type only has 1 byte of overhead. Also much like with remote functions, the number of size bytes changes when the value reaches 0x80 (128 characters) because it uses a Variable Length Quantity (VLQ) encoding scheme

If I'm understanding correctly, the#String + 2 should instead be #String + math.ceil(#String / 128) (type overhead being added afterwards; I'll update GetDataByteSize to append the type overhead also) - would this be the intended behavior? I'll take a further look tomorrow.

I believe the calculation would be math.ceil(math.log(#string + 1, 128)).

The above calculation wouldn't work when #string == 0, as the output would be 0; which I assume isn't the correct behavior. I've implemented the following function:
local function GetVLQSize(InitialSize: number, Length: number)
	return math.max(math.ceil(math.log(Length + InitialSize, 128)), InitialSize)
end


Update index.js

This change will allow users to disable this module via environment variables like so:
modules: [
    ['@nuxtjs/localtunnel', { disabled: process.env.DISABLE_LOCALTUNNEL === 1 }],
]

@IgnisDa This is an issue that users will face for any module (see this comment). Could I suggest instead:
modules: [
  process.env.DISABLE_LOCALTUNNEL !== 1 && '@nuxtjs/localtunnel',
].filter(Boolean)

Yes, this is a nice solution. Closing issue.


Fix call state

Fixes #2865
Adds a test that captures the error before the fix is introduced.

The failing tests seem to be flakes and unrelated to objc (
tests.bins/tsan/h2_sockpair_1byte_test invoke_large_request
tests.bins/tsan/h2_sockpair_1byte_nosec_test invoke_large_request
)
LGTM.


Expected metadata resolve root to be scarb.

Problem
Git clone this repo and run cargo build
You will get the following error. Now I'm not sure why this comes, but it expects the root of the package to start with scarb
Is there some issue on my end or something else?

OS: Ubuntu 22.04
Other relevant details:
scarb 2.6.3 (e6f921dfd 2024-03-13)
cairo: 2.6.3 (https://crates.io/crates/cairo-lang-compiler/2.6.3)
sierra: 1.5.0
cargo 1.77.1 (e52e36006 2024-03-26)

Steps

Git clone https://github.com/shramee/starklings-cairo1
run cargo build

Possible Solution(s)
No response
Notes
I'm not sure why this comes, but it expects the root of the package to start with scarb
Version
scarb 2.6.3 (e6f921dfd 2024-03-13)
cairo: 2.6.3 (https://crates.io/crates/cairo-lang-compiler/2.6.3)
sierra: 1.5.0

Hey! Downgrade Rust to 1.76. 1.77 broke our builds and a fix for this will be included in next Scarb release.


Html comments are shown in PR and issue descriptions

Description
Since v1.16.5 Html comments are shown in PR and issue descriptions, they should be hidden as before.
Reproducer: https://try.gitea.io/viceice/test/issues/1
Gitea Version
1.16.5
Can you reproduce the bug on the Gitea demo site?
Yes
Log Gist
No response
Screenshots

Git Version
No response
Operating System
No response
How are you running Gitea?
Official helm chart with slim image
Database
PostgreSQL

It seems < is encoded as \x3C 🤔

As discussed in the chat:
gitea on  HEAD (e6d46ee) (BISECTING) [$] via 🐹 v1.16.15 via  v16.14.0 took 8s
❯ git bisect good
f9ea4ab69ae397be4791c2e2ebf59f6cd9e5a6f8 is the first bad commit
commit f9ea4ab69ae397be4791c2e2ebf59f6cd9e5a6f8
Author: techknowlogick<EMAIL_ADDRESS>Date:   Sat Mar 19 13:46:47 2022 -0400

maybe the github.com/yuin/goldmark v1.4.0 to v1.4.8 update?

I think this is fixed in a later version

https://github.com/yuin/goldmark/issues/285
https://github.com/yuin/goldmark/commit/e64a68fc130e0c74dd2cbec59e6737a2e2bb7398

Should be a quick fix in this case.

When can i expect a fixed gitea version? If it needs some days i would create a small jquery script on page load to fix it temporary

Don't know, one of the maintainers could give us a rough timeline.

Just ask if it's worth to creaste the quickfix 🙃

Can confirm that updating goldmark fixes the issue:
❯ docker run -v $(pwd):/drone/src --workdir /drone/src -it golang:1.18 bash
root@c5219c2fabbe:/drone/src# go get -u github.com/yuin/goldmark
go: downloading github.com/yuin/goldmark v1.4.11
go: upgraded github.com/yuin/goldmark v1.4.8 => v1.4.11
root@c5219c2fabbe:/drone/src# TAGS="bindata sqlite sqlite_unlock_notify" make build

So you could build it with the fixed version from the release/v1.16 branch on your own.

OK, thanks. Will do a jQuery quickfix for now. It's easier, as i already have a custom js file embedded. 😉
$('.task-list-item').contents().each((i, e) => {
  if (e.nodeName === '#text') {
    e.textContent = e.textContent.replaceAll(/\x3C!--.*?-->/g, '')
  }
})


MSID: 94900 Version: 1 DOI: 10.1101/2<IP_ADDRESS>4539

MSID: 94900
Version: 1
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>4539
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=94900
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>4539

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/94900v1
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>4539
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/94900/v1/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!


MoreFilter window (the magnifying glass thing): Type tab: invert and clear buttons: styles vary

In the first group the buttons are slim, in the second they are fat.
They are generated by the exact same function with the exact same class.
And yet, one is cut. Is this a margin issue?
Something bad cascades upon us?
Can't find the culprit. This SHOULD be easy.

If .marginTopNarrow is readjusted to margin-top: 0
then this problem goes away.
Need to remove/override that class.

Flexbox makes the items the same height.
So, if a button is on the same line with checkboxes, it gets their height.
If the buttons, on the other hand, are alone on a row, their height will not be affected - hence smaller.

flexbox (row) needs height: max-content;
and flexbox button needs height: 100%


Current period is (still) broken

The problem:

The pronotepy.Client.current_period property does not return the proper period when a period is not marked as ended on the server.
More detailed explanation:
While the pronotepy.Client.current_period property might work on the demo server provided by index-education which is well configured, in real case situations, periods can be misconfigured by schools and tho a period can have been forgotten to be marked as "ended". This problem is of course not the fault of pronotepy but it can be solved pretty easily in the library.
A proposition to solve the problem:
With the help of the datetime library we could check if datetime.date.today is included between the start date and the end date of the fetched "current period" from Pronote, if it isn't we could take a slower approach.
We could check if datetime.date.today is included between the start and end date of each period, and return the right one.
Drawbacks/Conclusion:
All of this would be a little slower to execute but would avoid a lot of headaches not understanding why does pronotepy don't return the rights elements from current_period

Hi
Pronotepy is a wrapper so it only returns information provided by Pronote.  There is therefore no question of manually calculating the current period.  If you want to find the right period, you can implement this function yourself in your code.

Hello, yes I agree with @Bapt5. Just to be sure, the web client also fails to select the current period, right? If not, then I guess we should do something about it.

Yes for me sometime between two periods the website fail to select the good current period.


Make statuses configurable in TaskChecker

Please describe the feature you would like to see implemented.
Often we move a scheduled task that has an console command equivalent to crontab of the server. Since crontab is easier to monitor and more reliable than the Shopware scheduled tasks with the scheduled_task table.
If we do so we assign the scheduled_task the status 'crontab' to indicate the process runs from crontab.
It would be nice if the statuses you want to ignore are configurable.
Pseudo code:
$data = $this->connection->createQueryBuilder()
      ->select('s.scheduled_task_class', 's.next_execution_time')
      ->from('scheduled_task', 's')
      ->where('s.status NOT IN(:status)')
      ->setParameter('status', $this->configService->getArray('FroshTools.config.monitorTaskList'), ArrayParameterType::STRING)
      ->fetchAllAssociative();

Why don't set the status to inactive + change the next execution time to something in the far future when you use to console command?
inactive is supported by both core and this plugin

See melvin answer


[Sql Database] : Creating sql database in creation mode secondary does not obey the zone_redundant property

When creating a sql database , set the create mode to Secondary  and  zoneRedundant  to true.
Actual: The zoneRedundant is set to false.
Expected:The zoneRedundant is set to true.
Note: The zoneRedundant  could be set to true by updating.

@sinbai Apologies for the late reply. Thanks for reporting this issue. I see that the Service team has already been tagged on this issue.
@azureSQLGitHub Could you please look into this and provide an update once you get a chance. Awaiting you reply.


Add Status and ExecuteType class unit tests #1465

What is the purpose of the pull request

add Status class unit test
add ExecuteType class unit test

Brief change log

Add StatusTest.java
Add ExecuteType.java
Optimize Status
Add UT path to root pom.xml

Verify this pull request
This change added tests and can be verified as follows:

Added ExecuteType to verify the change.
Added StatusTest to verify the change.
Manually verified the change by testing locally.

Codecov Report

Merging #1536 into dev will increase coverage by 0.02%.
The diff coverage is n/a.

@@           Coverage Diff            @@
##             dev   #1536      +/-   ##
========================================
+ Coverage   7.42%   7.45%   +0.02%
========================================
  Files        272     272
  Lines      13634   13634
  Branches    2169    2169
========================================
+ Hits        1012    1016       +4
+ Misses     12555   12550       -5
- Partials      67      68       +1

Impacted Files
Coverage Δ

.../org/apache/dolphinscheduler/api/enums/Status.java
100% <ø> (ø)
:arrow_up:

...apache/dolphinscheduler/api/enums/ExecuteType.java
66.66% <0%> (+66.66%)
:arrow_up:

Continue to review full report at Codecov.

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

o.o okay


GitHub Metadata: Error processing value 'owner':

I have followed the installation guideline step by step. Everything was going well until I run
bundle exec jekyll serve
The outcome is:
Configuration file: /Users/shirinvafaei/Documents/Sites/personal-website/_config.yml
            Source: /Users/shirinvafaei/Documents/Sites/personal-website
       Destination: /Users/shirinvafaei/Documents/Sites/personal-website/_site
 Incremental build: disabled. Enable with --incremental
      Generating...
   GitHub Metadata: No GitHub API authentication could be found. Some fields may be missing or have incorrect data.
   GitHub Metadata: Error processing value 'owner':
  Liquid Exception: uninitialized constant Faraday::Error::ConnectionFailed Did you mean? Faraday::ConnectionFailed in /_layouts/home.html
jekyll 3.8.5 | Error:  uninitialized constant Faraday::Error::ConnectionFailed
Did you mean?  Faraday::ConnectionFailed

I am a beginner in this area. I appreciate any feedback so I can understand or try to figure out how to solve this issue!
Thanks in advance

I think this is related https://github.com/github/personal-website/issues/166

I can confirm adding this in Gemfile fixed this issue
diff --git i/Gemfile w/Gemfile
index b224e17..28f0c04 100644
--- i/Gemfile
+++ w/Gemfile
@@ -3,3 +3,5 @@ gem 'github-pages', group: :jekyll_plugins
 gem "jekyll-github-metadata"
 gem "jekyll-octicons"
 gem "jemoji"
+# Neccessary to prevent Jekyll errors. See https://github.com/github/personal-website/issues/166
+gem 'faraday', '0.17.3'

It's here 😅 https://github.com/github/personal-website/pull/215

I can confirm adding this in Gemfile fixed this issue
diff --git i/Gemfile w/Gemfile
index b224e17..28f0c04 100644
--- i/Gemfile
+++ w/Gemfile
@@ -3,3 +3,5 @@ gem 'github-pages', group: :jekyll_plugins
 gem "jekyll-github-metadata"
 gem "jekyll-octicons"
 gem "jemoji"
+# Neccessary to prevent Jekyll errors. See https://github.com/github/personal-website/issues/166
+gem 'faraday', '0.17.3'

Thank you! This works perfectly!

Running bundle update faraday fixed it for me


Do you have the code for a single CPU?

Thank you so much for your great work!
I am very interested in your work and am trying your code.
However, I only have one gpu. After modifying the code and training the model, it is found that the baseline performance has dropped significantly.
Do you have the code for a single CPU?
I would be very grateful if you could provide it to me.

The smallest model we tested is ShuffleNet. Our training script requires 16 batch size over 2 GPUs (8batch X 2GPUs). So you should modify the script as follows: --nproc_per_node=1 / --bs_mult 16
If the requirements of this training script exceed the memory of your GPU, there is no other way than to use a model smaller than ShuffleNet. Reducing the batch size will cause performance degradation.

Dear author, thank you so much for your effort!
Best regards,


alpha DNA polymerase:primase complex - DNA polymerase and primase question

alpha DNA polymerase:primase complex
is defined as "A complex of four polypeptides, comprising large and small DNA polymerase alpha subunits and two primase subunits, which catalyzes the synthesis of an RNA primer on the lagging strand of replicating DNA; the smaller of the two primase subunits alone can catalyze oligoribonucleotide synthesis. PMID:11395402"
This is DNA polymerase alpha (?): which makes the RNA primer, and then the DNA pol subunit does some limited DNA synthesis.
is_a nuclear DNA-directed RNA polymerase complex
part_of nuclear replisome (which is not classed as a DNA polymerase complex in GO)
This seems to have some history....so perhaps this is a product of change, but I would expect it to relate to both DNA-directed RNA polymerase complex and DNA polymerase complex.
https://www.ncbi.nlm.nih.gov/pmc/articles/PMC4817131/

tagging @sjm41

Why should it be a 'DNA-directed RNA polymerase complex', because it makes the RNA primer ?
Relevant tickets:
#11942
#11165
#12575

It first makes the RNA primer then extends it - two subunits do the priming and one does the DNA elongation

This is an interesting issue that points out the distinction of how complexes are represented in GO. If complexes are represented by membership, then this complex would be a DNA polymerase complex because it contains a DNA polymerase subunit. If complexes are represented as functional units, this complex would not be a DNA polymerase complex because functionally the DNA polymerase complex would synthesize DNA. Currently we have the functional distinction in GO:
This complex through inheritance is: capable of' some 'DNA-directed 5'-3' RNA polymerase activity'
If we were to add the DNA polymerase complex parent, this would also inherit  this: capable of' some 'DNA polymerase activity'
I think this would be incorrect since the primase complex synthesizes RNA. If we want to add the DNA polymerase complex parent to the term because it contains a DNA polymerase subunit, we need to remove the functional assertions about the parents of these terms. Given the two options, I think it is richer for GO to have the functional relationships than the membership relationships. @pgaudet, do you think this is consistent with the current thinking?

I am sorry, I am confused. Both the DNA and RNA polymerase subunits can function independently in vitro. In vivo, the complex is the unit that had both activties - why can't it both be capable of' some 'DNA-directed 5'-3' RNA polymerase activity' and capable of' some 'DNA polymerase activity'?
On a separate point:
why: 'DNA-directed 5'-3' RNA polymerase activity'
when we have
GO:0003896 | DNA primase activity would be the more correct

Then we would need to modify the def to include the DNA polymerase activity. Right now it specifies only the ability to make the primer. I can modify the def to include the DNA polymerase activity. I will check the ref to make sure that this activity is included.
I think your correct about the primase activity parent.

If you look at the ref, I think this term was only meant to describe the complex with respect to the primase activity. From the paper, it looks like it always copurifies with DNA pol alpha. See the section on eukaryotic primases. Complexes complexes complexes oh my.
Should we try to find a separate ref that describes all of the enzymatic activities of the complex? Of course down the road this might lead to issues teasing out which gene product do what. No worse than we are now though.

Jim Borowiec at NYU is an expert on the enzymology of DNA replication who has collborated extensively on the Reactome annotations in this area. I can provide his e-mail address if someone would like to try to recruit him for GO.

I think this reference supports the tweak of the definition and additional parent. PMID:26975377

This is good: https://www.ncbi.nlm.nih.gov/pmc/articles/PMC4817131/  (PMID26975377)
Ultimately.... it would just seem strange to a user to do a query with 'DNA polymerase complex' and not get any subunits of the DNA polymerase alpha. The activity you see with the complex, just depends on the assay.

I think this term was only meant to describe the complex with respect to the primase
activity.

What?!? Are we talking about GO:0005658 (that's the ID for the name "alpha DNA polymerase:primase complex")? If so, that term most assuredly was and is meant to describe the tetrameric complex that includes DNA polymerase and DNA primase subunits, and, as Helen has noted above, executes BOTH functions during DNA replication. (The DNA polymerase only adds a short stretch, ~20 nt, to the primer, but it does indeed act.)
It would be fine to edit the definition to explicitly mention DNA polymerase activity, and to give it 'capable of' links to both GO:0003896 DNA primase activity and GO:0003887 DNA-directed DNA polymerase activity.
Don't be misled by the fact that the text definition cites a review entitled "DNA primases". In it, the passage on eukaryotic primases describes the complex composition, including the DNA polymerase subunits. That sait, it would also be OK to add the reference Helen found.

This reveiw PMID:26109351 appears to have a good summary of primases across species that we should probably look at...just in case there are primases that just do the RNA pol bit.

just in case there are primases that just do the RNA pol bit.

oh, there are ... but they wouldn't use the GO:0005658 CC term anyway

OK. I will modify the definition and add the additional parent. I will also add the additional references.

Hi David,
Something here may have got lost in translation:
I see changes to def and additions of
relationship: capable_of GO:0003887 ! DNA-directed DNA polymerase activity
relationship: capable_of GO:0003896 ! DNA primase activity
but no
is_a
GO:0042575 | DNA polymerase complex

Thanks @hattrill. I will double check. I thought it was inferred, but maybe that was before I added the primase activity. If it's not, I will assert it.

I'm seeing it as being inferred in Protege running ELK. @balhoff?

Looks like the release hasn't happened.  @kltm

@hattrill, where do you see the changes that you note above?

I see the superclass in today's snapshot release.


SuppressMessageAttribute not working for ScriptBlock params

Before submitting a bug report:

Make sure you are able to repro it on the latest released version
Perform a quick search for existing issues to check if this bug has already been reported

Steps to reproduce
Register-ArgumentCompleter -Native -CommandName dotnet -ScriptBlock {
	[Diagnostics.CodeAnalysis.SuppressMessageAttribute('PSReviewUnusedParameter', 'wordToComplete', Justification = 'Reason for suppressing')]
	param(
		$wordToComplete,
		$commandAst,
		$cursorPosition)

	dotnet complete --position $cursorPosition "$commandAst" | ForEach-Object {
		[System.Management.Automation.CompletionResult]::new($_, $_, 'ParameterValue', $_)
	}
}

Expected behavior
Rule `PSReviewUnusedParameter` doesn't trigger for parameter `$wordToComplete`.

Actual behavior
Rule `PSReviewUnusedParameter` triggers for parameter `$wordToComplete`.

If an unexpected error was thrown then please report the full error details using e.g. $error[0] | Select-Object *
Environment data

> $PSVersionTable
Name                           Value
----                           -----
PSVersion                      7.3.0
PSEdition                      Core
GitCommitId                    7.3.0
OS                             Microsoft Windows 10.0.22000
Platform                       Win32NT
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

> (Get-Module -ListAvailable PSScriptAnalyzer).Version | ForEach-Object { $_.ToString() }
Using built-in version of VSCode Powershell extension (v2022.10.0)

The documentation here says: https://learn.microsoft.com/en-us/powershell/utility-modules/psscriptanalyzer/using-scriptanalyzer?view=ps-modules#suppressing-rules

Within the scope of the script, function, or parameter that you decorated, all rule violations are suppressed.

Therefore, this is currently by design.

Thanks for the reply! Putting the param() block in the script file with all the argument completers works nicely and is appropriate since all of them have those three params.
Could we use this issue as a feature request for adding it to script blocks or should I create a separate feature request?


Add ability to customize minikube start command

The minikube start command is currently hardcoded, being able to override it would help in cases where VirtualBox is not installed or users would prefer to customize the minikube installation (such as using an alternative container runtime).

cc @brendandburns

I found that this limitation can be worked around using the minikube config, running minikube config set vm-driver kvm2 allows this extension to be able to create a minikube cluster.

Fixed by #330.


GSD Request

--- GSD JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from v5.14 to before v5.15.6",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=f1a0a376ca0c4ef1fc3d24e3e502acbb5b795674",
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=229c555260cb9c1ccdab861e16f0410f1718f302"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "f1a0a376ca0c4ef1fc3d24e3e502acbb5b795674",
      "note": "introduced"
    },
    {
      "type": "commit",
      "value": "229c555260cb9c1ccdab861e16f0410f1718f302",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "sched/scs: Reset task stack state in bringup_cpu()\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.15.6 by commit 229c555260cb9c1ccdab861e16f0410f1718f302, it was introduced in version v5.14 by commit f1a0a376ca0c4ef1fc3d24e3e502acbb5b795674. For more details please see the references link."
}
--- GSD JSON ---

This issue has been assigned GSD-2021-1002547


CSS: Styles are huge, generally

The non-doc CSS styles (header/footer stuff) seems very large and looks best zoomed out to about 75% in the browser.
The docs CSS styles are also big but not quite as dire, IMO; that said, they look good at 75% too.

CC @CreaturesInUnitards

ok, trimmed back quite a bit on header/footer

@johndmulhausen  LGTM, but I'm re-assigning to you because:

You still might have specific areas that you want ensmallened; and
I can tell that you're not busy. At all.


Add end-to-end tests

This just adds one test:

running cs251tk

Hopefully these integration tests will prevent us (read: me) from breaking the software at a completely unusable level.

Coverage increased (+8.5%) to 65.712% when pulling 9edf14e07ca47fc5a17bd423d357dc82c31b95a8 on end-to-end into 365b47dfdc288b780d5cd47cd7dff8a5373337cf on master.


Google Test Fix for "Go to test" and "Debug Test"

This change addresses items #3280 and #3362
The following changes are proposed:

Get the location of the test definition from CTest's "DEF_SOURCE_LINE" property
Fallback to previous behaviour if DEF_SOURCE_LINE does not exist
Route debug test request to correct CTestDriver based on folder and test URI

Other Notes/Information
Currently, the CTest executable does not output custom properties (defined using CMake's define_property function).
Hence, I have made a  pull request there to address that: https://gitlab.kitware.com/cmake/cmake/-/merge_requests/9305
Note that it falls back to the old behaviour without the CMake pull request merged (which will be CMake 3.29 and above if so).
fixes #3362
Edit:
During testing with my projects, I discovered that the root cause of #3280 is how multi-root workspaces are handled. Fixing now...
Edit 2:
I have fixed #3280 by routing a debug test request to the right CTestDriver using the URI of the test and project.

@microsoft-github-policy-service agree

I've chatted offline with @xisui-MSFT and we think this PR looks good! I've also tested some with some simple gtest projects and everything seems to be working as expected.
Approved!


Replace < and > with the corresponding HTML entities in the HTML input fields

It's better not to implement this, as the html tags would be lost too.


Redirecting everything to user

Trying to make GET, PUT, POST work within user.serivce.ts i.e. for example student.service.ts will be more of an interface...
This doesn't work

type Without<T, U> = { [P in Exclude<keyof T, keyof U>]?: never };
type XOR<T, U> = T | U extends object
  ? (Without<T, U> & U) | (Without<U, T> & T)
  : T | U;

type Child = ZTeacher | ZStudent;
// type ChildModels = XOR<typeof Student, typeof Teacher>;

const createUser = async (
  zUser: ZUser,
  zChild: Child,
  ChildModel: XOR<Teacher, Student>
) => {
  // Not exactly a new user anymore?
  // eslint-disable-next-line @typescript-eslint/no-unsafe-assignment, @typescript-eslint/no-unsafe-call
  const newUser = await ChildModel.create(
    {
      ...zChild,
      user: { ...zUser },
    },
    {
      include: User,
    }
  );

  return newUser;
};

This fails because of some type related problem, the other way around iirc works i.e. User.create()...

Solved in #43


Add set method for config black list in broker/namesrv/controller config.

Add set method for config black list in broker/namesrv/controller config.

Codecov Report
Attention: 11 lines in your changes are missing coverage. Please review.

Comparison is base (e955e43) 43.17% compared to head (9ba633f) 43.16%.
Report is 1 commits behind head on develop.

Files
Patch %
Lines

...a/org/apache/rocketmq/common/ControllerConfig.java
0.00%
4 Missing :warning:

.../apache/rocketmq/common/namesrv/NamesrvConfig.java
0.00%
4 Missing :warning:

.../java/org/apache/rocketmq/common/BrokerConfig.java
25.00%
3 Missing :warning:

Additional details and impacted files
@@              Coverage Diff              @@
##             develop    #7586      +/-   ##
=============================================
- Coverage      43.17%   43.16%   -0.01%
- Complexity      9780     9782       +2
=============================================
  Files           1162     1162
  Lines          84303    84355      +52
  Branches       10949    10952       +3
=============================================
+ Hits           36396    36412      +16
- Misses         43386    43417      +31
- Partials        4521     4526       +5

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Material-UI doesn't work with Gatsby since 1.0.0-beta.13

Since upgrading from<EMAIL_ADDRESS>to >=1.0.0-beta.13, my application cannot even boot up, it crashes straight with an error message.

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Current Behavior

The following error message gets thrown in the form of a "red screen of death":
Invariant Violation: Unknown(...): A valid React element (or null) must be returned. You may have returned undefined, an array or some other invalid object.
invariant
webpack:///~/fbjs/lib/invariant.js:42
ReactCompositeComponentWrapper.mountComponent
webpack:///~/react-dom/lib/ReactCompositeComponent.js:192
Object.mountComponent
webpack:///~/react-dom/lib/ReactReconciler.js:43
ReactDOMComponent.mountChildren
webpack:///~/react-dom/lib/ReactMultiChild.js:234
ReactDOMComponent._createInitialChildren
webpack:///~/react-dom/lib/ReactDOMComponent.js:701
ReactDOMComponent.mountComponent
webpack:///~/react-dom/lib/ReactDOMComponent.js:520
Object.mountComponent
webpack:///~/react-dom/lib/ReactReconciler.js:43
ReactCompositeComponentWrapper.performInitialMount
webpack:///~/react-dom/lib/ReactCompositeComponent.js:368
ReactCompositeComponentWrapper.mountComponent
webpack:///~/react-dom/lib/ReactCompositeComponent.js:255
Object.mountComponent
webpack:///~/react-dom/lib/ReactReconciler.js:43

Steps to Reproduce (for bugs)

Clone https://github.com/simonyiszk/mvk-web
Upgrade material-ui to the latest version (>=1.0.0-beta.13)
Launch the application using yarn develop or npm run develop
Open a web browser at the location shown inside the console

Your Environment

Tech
Version

Material-UI
1.0.0-beta.16

React
15.6.2

Browser
Google Chrome 61.0.3163.100 x64

Gatsby
1.9.55

You have missed one part from the breaking changes section https://github.com/callemall/material-ui/releases/tag/v1.0.0-beta.13.
https://github.com/simonyiszk/mvk-web/blob/3734dd75601fc219660de5e877cfcede0c37753f/src/components/responsive-app-bar.jsx#L167

Thank you very much, I appreciate your kind assistance! Not sure about how I could miss that breaking change...


ng-change does not work for user select fields

The user select fields (chosen) in item-detail and assignment-detail does not fire a ng-change to add selected speaker or candidate... It was introduces with switching to chosen (with 2.0b5). Maybe a bug in chosen.
So currently you can't add other users as speakers/candidates. It's a blocker for 2.0.

I found the problem which was not easy to catch!
Solution: jquery hat to load first before angular is loaded! So we have to change the order in bower.json.
See https://github.com/OpenSlides/OpenSlides/pull/2072


Adding the API into a java project

I am having trouble adding this API into my java project, do you have a detailed tutorial on how to add this API into a new java project? I am using eclipse by the way.

I suppose you aren't using Maven, in which case the easiest way to use this library would be to directly download the JAR file of the library, at the link https://repo1.maven.org/maven2/fr/delthas/javaskype//javaskype-.jar, e.g. https://repo1.maven.org/maven2/fr/delthas/javaskype/1.0.18/javaskype-1.0.18.jar (use the latest version).
The truth is most projects these days use a tool that manages the dependencies of their project. This is called a dependency management tool. There are also tools that can automatically build a project, and they're called build tools. In Java, there's a tool that does both of that and is very used, called Maven. Most projects are "distributed" as Maven dependencies, which means you can very simply add them to your project if your project uses Maven by simply adding a few lines in a project-specific Maven configuration that's called pom.xml.
In your case, if you really want to get involved in real-world Java programming, I suggest you learn a few tutorials about Maven to understand how to use it. There is a very good book (online, free) online that you could read: http://books.sonatype.com/mvnref-book/reference/ that will let you understand what Maven is very quickly. Once you're done with that, maybe you'll need to browse stackoverflow for specific questions such as "how do I create a runnable jar file with maven?" and you'll be able to add my project as a Maven dependency of yours.


Podcast streams failing during playback

Prerequisites

[X] Put an X between the brackets on this line if you have done all of the following:

Checked that your issue isn't already filed: https://github.com/GetStream/Winds/issues

Description
Podcasts hang during playback and do not continue, even after scrubbing backwards, or pausing and unpausing.
Steps to Reproduce

Just play a podcast.

Expected behavior: Podcast plays all the way through.
Actual behavior: Podcast pauses at some point and does not continue.
Reproduces how often: I have tried to play ~3 podcasts, and the issue has occurred during each.
Versions
About Winds fails to produce dialog box.
Additional Information
Any additional information, configuration or data that might be necessary to reproduce the issue.

Thanks for reporting this issue. Which version of Winds and what platform are you using?

Unable to reproduce. Likely a network issue. Will re-open once we know the version. Thank you!


Dumping to DBC compatible with Vector

Added support for multiple message senders when dumping DBC
Added escaping of double quotes in comments when dumping

Coverage increased (+0.1%) to 96.958% when pulling 6fae75f0ad12215fab286a200140cca18907ad87 on sbrw:master into 0ce0eeafb4cfb16faf93116a9934ac05ed5ef527 on eerimoq:master.


architecture diagram

          1 Warning

:warning:
Please start subject with capital letter.
c9bf6ffa7bb4e67d7853448b04648143afe2181c

  Generated by :no_entry_sign: Danger


[bug] Increase timeout in nginx(?)

When converting images, a timeout may occur on the initial image load due to the slow conversion to too many formats.
Need to find out where the timeout comes from (probably nginx) and increase the time it waits.
djangosocial-django    | [2023-11-09 21:42:10 +0000] [10] [CRITICAL] WORKER TIMEOUT (pid:11)
djangosocial-nginx     | 2023/11/09 21:42:10 [error] 29#29: *63 upstream prematurely closed connection while reading response header from upstream, client: <IP_ADDRESS>, server: sociald.theepic.dev, request: "GET / HTTP/1.1", upstream: "http://<IP_ADDRESS>:8000/", host: "sociald.theepic.dev", referrer: "https://sociald.theepic.dev/about/"

Turned out I had to update the gunicorn timeout. Not sure whether the nginx config also needs updating, but increasing proxy_read_timeout was probably a good idea regardless.


Verbose dependency resolution option

When running a paket update , it is problematic on a big solution if one project within that solution references another project dependency that no longer exists, paket displays the following:
Unable to parse C:\Users\admin\Documents\Visual Studio 2015\Projects\ext\src\External\FSharp.Data\src\FSharp.Data.fsproj:
     Could not find a part of the path 'C:\Users\admin\Documents\Visual Studio 2015\Projects\ext\src\External\FSharp.Data\src\FSharp.Data.fsproj'.

It would be useful to have a verbose option that tells you where this .fsproj orignated (its another project referencing the now moved FSharp.Data), otherwise one has to go through Visual Studio and attempt to build all the projects to work out where the issue exists. I see issue https://github.com/fsprojects/Paket/issues/503 asked for a verbose option to help debugging various things (different issue though).

We have -v option.
But can you create a repro so that I can improve the error message?

Sorry, I don't see -v in my paket help, although it does work, perhaps the packet.exe /? should display this verbose option?. Now I get the following, which makes it easy to debug the issue:
Installing to C:\Users\admin\Documents\Visual Studio 2015\Projects\\tests\TestData\Tests.fsproj
Unable to parse C:\Users\admin\Documents\Visual Studio 2015\Projects\\tests\.Agent.Delivery\Delivery.fsproj:
      Could not find a part of the path 'C:\Users\admin\Documents\Visual Studio 2015\Projects\\tests\.Agent.Delivery\Delivery.fsproj'.


Invoke -SSH Command Returns Nothing

#https://kb.vmware.com/s/article/59470
$GethostdValue ="vsish -e get /sched/groups/$grpID/memAllocationInMB"
$SethostdValue ="localcli --plugin-dir /usr/lib/vmware/esxcli/int sched group setmemconfig --group-path /host/vim/vmvisor/hostd --min 490 --max 490 --units mb"
When I run below command, the Output is shown as in code.
$GetResult = (Invoke-SSHCommand -Index 0 -Command $GethostdValue).Output
Write-Host
$GetResult.Trim()

memsched-allocation {
min:8148
max:8148
shares:2147483647
minLimit:8148
units:units: 3 -> mb
}

However when I Run, there is no output. it's unclear if it's failing on on. [Please note when this command as in $SethostdValue is run in putty session, it works fine]
$SetResult = (Invoke-SSHCommand -Index 0 -Command $SethostdValue).Output
Write-Host
$SetResult.Trim()

User Error


feat: encourage user to login

Fix #321
Fix #405

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

GitHub release
v6.23.0

Your semantic-release bot :package::rocket:


Vetala Options  Number in the Name of a Title causes it to become a Group

It ended up being an issue when . is in the name.  . are used to define grouping and are no longer aloud in option names as of this commit.
https://github.com/louisVottero/vtool/commit/8d3843de1c34099ab011dee132224dd383a01cee


AD Users invalid after moved to another OU

We are currently restructuring our AD and all Users will be moved to a new OU.
Now no user got a valid user object in Rancher anymore, as the user CN is changed.
The principal id of the user is still the old one. And I don't have the possibility to change this for the users.

What kind of request is this (question/bug/enhancement/feature request):
Bug
Steps to reproduce (least amount of steps as possible):

Login in Rancher with a Active Directory User
Move the Active Directory User to another OU
The activedirectory_user principalIds is not updated and the user information could not be fetched

Result:

Other details that may be helpful:
Environment information

Rancher version (rancher/rancher/rancher/server image tag or shown bottom left in the UI): 2.2.4
Installation option (single install/HA): Single

I would appreciate any help!

We have the same issue each time we move a user in ActiveDirctory.
The user should be stored in Rancher using Active Directory userPrincipalName (user@exemple.com), which is unique and stable, as opposed to DistinguishedNAme. (see https://serverfault.com/q/567776/238995 )


Cookies are missing on Server

Hi,
If I try to access the cookies on the h3Event they are empty
export const createContext = (event: H3Event) => {
  const cookies = parseCookies(event); // cookies = {}

  ...
}

I think it's a ssr problem like written here: https://trpc.io/docs/ssr#q-why-do-i-need-to-forward-the-clients-headers-to-the-server-manually-why-doesnt-trpc-automatically-do-that-for-me
Is it possible to access the context and set the headers?
It was working with 0.3.* (trpc 9) am I missing something else?

Hey, thanks for noticing that! Updated docs
https://trpc-nuxt.vercel.app/get-started/tips/headers

Thanks, works perfectly


Add-ons for MessageEditor

Hello, jaqobb!
Hope this finds you well. Just a quick note; thank you for this amazing plugin. You've really outdone it with this one, and I hope you bring more updates to the plugin.
The following are add-ons/features that the MessageEditor (ME) users would love to see in the plugin:

Simplification of JSON: I tested ME to see if it can support JSON messages, and it does! Although, the JSON format is a bit complicated, especially for non-developers. I suggested simplifying the JSON format to a format similar or exact to when executing a /tellraw command. A website that I use (and I hope finds this useful for you) to create JSON messages is: minecraftjson.com. Of course, the versions at the bottom of the page must be considered for full compatibility with other Minecraft versions.
BungeeCord implementation: I tested to see if there was a chance to edit BungeeCord messages/plugins, but no good there. As the purpose of this issue, I suggest adding a BungeeCord implementation to the plugin. For example, if I wish to edit a Bungee LuckPerms message, I would have the ability (with the rest of the other amazing, simplified features of ME) to edit those Bungee messages to what I want to replace it. Another example is the typical "No permissions" message that BungeeCord gives to players with no permissions; this implementation would grant me the ability to edit that "No permissions" message that BungeeCord default gives out to players.
Sound and Effects: As the point suggest, adding sounds and particle effects to the ME system would probably freak out (in a good way) the ME community. Particle effects may be difficult, but adding the possibility to have sounds along the replaced message would be great.

For now, that would be all. If I get any ideas on the long run, I will let you know.
Once again, thank you for developing such an useful plugin.

Hey. Thank you for all the suggestions and while I appreciate all of them, I unfortunately will not be able to implement them. I have retired from Minecraft and everything that is Minecraft-related so I do not plan on updating this plugin or any other plugin I have created in the past anytime soon. That being said, I am still willing to merge pull requests if there are any and make an official resource update on the SpigotMC forum.


Warn if publishing code unusable to other Rust code

Publishers are likely to have an expectation that their crate is easy to use :: by other Rust authors. However publishing the cdylib crate type does not publish crates that are ready to be easily used by cargo without considerable more fuss than other crate types.
It is unlikely this is intentional. If people do not understand what they are doing, they should be prevented from doing this in their cargo publish command and offered some kind of option to --override if they do indeed have some sort of legitimate purpose.
This recently became visible in an attempt to use https://github.com/emgyrz/cubic_spline/pull/2 so it is not an idle hypothesis that this is a problem.
I am willing to attempt drafting this feature if it is considered desirable.

There are entirely legitimate reasons to publish a cdylib, though, such as providing a library with a C-compatible interface.
Perhaps we could detect if you're building a cdylib but you don't appear to export any C functions (e.g. non-mangled symbol names)? There are still legitimate reasons for such a cdylib, but that's rare enough that it wouldn't be unreasonable to warn people by default.

That is why I did not recommend banning publishing cdylibs altogether. I should note that the crate that raised this question appears to have worked fine for its purposes (a wasm demo) but also appears to not export C functions.
And I had read the existing documentation about cdylibs before but had not come to the conclusion that it would be impossible to use such a crate in my own Rust applications (e.g. if it had some public function that remained useful purely in a Rust context), as it was not obvious to me that the compilation model for cdylibs would work the way it did for this person. I do not believe I would have made different conclusions.
So I do acknowledge there are reasons to do something, but I do not believe proper usage and understanding is discoverable at the current moment, so I believe discovery should be introduced as close as humanly possible to the actual publishing step.

I have answered this exact same question again for someone else. This time, they were the author of both the using and original crate, so it was an easy fix, and I had experience with the problem, so it was quick, but I expect at least 20 other people have had this problem since it's really common for people to try to use Rust for wasm (for good reasons!).
I do not believe "not exporting any functions" is going to be terribly useful as a heuristic since if I understand correctly, you DO export functions via wasm-bindgen. However, I think I got overly fixated on a single solution and provided an incomplete image of the problem. Allow me to attempt to respecify the problem:

Someone writes a crate, and then they annotate it with wasm_bindgen, to make it usable from wasm. These directions also prompt them to mark it as a cdylib. They may or may not understand the ramifications of doing so, even after reading the documentation so far on cdylibs, but they're told it has to be in order to be for it to be wasm'd, so they do it. They publish the crate or privately reuse it.
Someone, possibly including that same author, attempts to add the cdylib to their deps and then use the library in an ordinary Rusty fashion.
They run a cargo {build,check,test}.
Their build reports it successfully compiled the crate.
Yet their build fails, apparently mysteriously, with error[E0432]: unresolved import {crate}, even though so far they've done everything right to use a crate from Rust.

match get_help() {
   Some(HelpfulWasmer) => deps.patch["crate-type"] += ["rlib"],
   Some(HelpfulRustacean) => unimplemented!("oh, WASM? I don't know..."),
   _ => panic!("oh noooooooooooooooo"),
}

s--crate-type=cdylib, #[crate_type = "cdylib"] - A dynamic system library will be produced. This is used when compiling a dynamic library to be loaded from another language. This output type will create *.so files on Linux, *.dylib files on macOS, and *.dll files on Windows.

This is somewhat opaque if dll is just a file type that breaks your software if it isn't there, and even if you know what a C ABI is, that doesn't mean you'll put together "so Rust doesn't have access to an rlib anymore" yourself, which is what you're used to since that is the default behavior in the absence of a crate-type key.


Holoparasite QoL

Changes:
Holoparasites can now walk a 15 tile radius around their host, previously it was 5.
Holoparasites can now bump into doors to open them.

This isn’t even a QoL aside from maybe the door opening, I’d recommend bumping this down to 10 tiles, which is already a fairly large range, otherwise this is just a straight buff for IPM.

Seems fine.

This isn’t even a QoL aside from maybe the door opening, I’d recommend bumping this down to 10 tiles, which is already a fairly large range, otherwise this is just a straight buff for IPM.

The qol is not having to call your holoparasite back again every 3 seconds

This isn’t even a QoL aside from maybe the door opening, I’d recommend bumping this down to 10 tiles, which is already a fairly large range, otherwise this is just a straight buff for IPM.

The qol is not having to call your holoparasite back again every 3 seconds

the "QoL" is that this only affects the one entire ship, for the one faction, for the one place this spawns in, which would be the IPM hulk, maybe if these were more readily available i wouldn't care, but there doesn't need to be some mad holoparasite going around beating people up, i don't think they're 10 tiles on ss13, let alone 14.

This isn’t even a QoL aside from maybe the door opening, I’d recommend bumping this down to 10 tiles, which is already a fairly large range, otherwise this is just a straight buff for IPM.

The qol is not having to call your holoparasite back again every 3 seconds

the "QoL" is that this only affects the one entire ship, for the one faction, for the one place this spawns in, which would be the IPM hulk, maybe if these were more readily available i wouldn't care, but there doesn't need to be some mad holoparasite going around beating people up, i don't think they're 10 tiles on ss13, let alone 14.

Marge, the PR's merged.

NOOO


The app crashed when I tried to open the hentai after removing it from the favorite list.

Describe the bug
When I tried to open hentai after removing it from the favorite list, then the app crashed.
To Reproduce
Steps to reproduce the behavior:

Open the app.
Click on the love icon to add it as your favorite.
Now, swipe to the right to see your favorite list.
Then, press the love button again, the app will ask your permission to remove it. Press yes.
At last click on that hentai to open it and notice the crash.

Expected behavior
I expected that I could open the hentai after removing it from my favorite list or the hentai would disappear from the favorite list after removing.
Screenshots
https://www.youtube.com/watch?v=TXvJYyflFLo
Smartphone (please complete the following information):

Device: Samsung A5
OS: Android 6.0.1
App Version: 1.0.2

Logcat
E AndroidRuntime: FATAL EXCEPTION: main

07-19 11:46:38.275 10688 10688 E AndroidRuntime: Process: personal.ttd.nhviewer, PID: 10688

07-19 11:46:38.275 10688 10688 E AndroidRuntime: java.lang.IndexOutOfBoundsException: Invalid index 0, size is 0

07-19 11:46:38.275 10688 10688 E AndroidRuntime: 	at java.util.ArrayList.throwIndexOutOfBoundsException(ArrayList.java:255)

07-19 11:46:38.275 10688 10688 E AndroidRuntime: 	at java.util.ArrayList.get(ArrayList.java:308)

07-19 11:46:38.275 10688 10688 E AndroidRuntime: 	at personal.ttd.nhviewer.Controller.fragment.base.ComicListFragment$ComicListAdapter.onListItemClick(ComicListFragment.java:182)

07-19 11:46:38.275 10688 10688 E AndroidRuntime: 	at personal.ttd.nhviewer.Controller.fragment.base.BaseListFragment$BaseListAdapter$1.onClick(BaseListFragment.java:353)

07-19 11:46:38.275 10688 10688 E AndroidRuntime: 	at android.view.View.performClick(View.java:5714)

07-19 11:46:38.275 10688 10688 E AndroidRuntime: 	at android.view.View$PerformClick.run(View.java:22589)

07-19 11:46:38.275 10688 10688 E AndroidRuntime: 	at android.os.Handler.handleCallback(Handler.java:739)

07-19 11:46:38.275 10688 10688 E AndroidRuntime: 	at android.os.Handler.dispatchMessage(Handler.java:95)

07-19 11:46:38.275 10688 10688 E AndroidRuntime: 	at android.os.Looper.loop(Looper.java:148)

07-19 11:46:38.275 10688 10688 E AndroidRuntime: 	at android.app.ActivityThread.main(ActivityThread.java:7325)

07-19 11:46:38.275 10688 10688 E AndroidRuntime: 	at java.lang.reflect.Method.invoke(Native Method)

07-19 11:46:38.275 10688 10688 E AndroidRuntime: 	at com.android.internal.os.ZygoteInit$MethodAndArgsCaller.run(ZygoteInit.java:1230)

07-19 11:46:38.275 10688 10688 E AndroidRuntime: 	at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1120)

Thank you for your report, I will upload a new version later.

The new version is committed and uploaded.

Thanks for fixing the issue.


Q.all() never fulfilled immediately

Is it intentional, that Q.isFulfilled(Q.all([2,2])) === false?
I would have thought that as long as the input of Q.all() contains no promises
or only fulfilled promises, the result of Q.all() should also be fulfilled.
I'm having some problems that originate from always being put into the next eventloop-cycle when doing Q.all()

@nknapp We did intentionally omit synchronous state queries from Q for a long while and left them only for perf optimizations and debugging reasons. We do intentionally call handlers asynchronously, regardless of the state of the promise. If you need an order guarantee, then is there to ensure it.
We don’t intentionally return false for this specific case, but it is an emergent behavior. Creating a special case would probably not be worthwhile, but I would certainly consider a change if it is simple.

Thanks for the quick reply.
It would have been helpful for me if Q.all([2,2]).then(fn) would have called fn immediately, because I would then have been able to use temporary data-structures that are cleaned at the end of the calling function.
But I have now found the part in the docs now and in the Promise A+ spec that explicitly prohibit this behaviour and I can see why. So I'm closing this issue.


Bundle id Error

Describe the bug
Report from Apple:
ITMS-90806: CFBundleIdentifier collision - Each bundle must have a unique bundle identifier. The bundle identifier 'com.el-eleven.Swinject' is used in the bundles '[Swinject.framework, Swinject.framework]'
App have iOS and watchOS target.
To Reproduce

make app with multiple platforms
upload it to AppStore

Expected behavior
No error from Apple
Environment:

Xcode version: Xcode 10.3
Swift version: 5.0.1
OS running SWXMLHash: MacOS 10.14.6 running Xcode

Additional context
Add .${PLATFORM_NAME} to end of bundle id

Resolved by #433, released in 2.7.0


Spell Check

Having a spell checker would be a "nice to have"

Relevant links:
https://github.com/atom/atom-shell/blob/master/docs/api/web-frame.md#webframesetspellcheckproviderlanguage-autocorrectword-provider
https://github.com/atom/node-spellchecker


Enable OpenSSF Scorecard

Description:

Describe the desired behavior, what scenario it enables and how it
would be used.

Monitor security best practices using https://github.com/ossf/scorecard
[optional Relevant Links:]

Any extra documentation required to understand the issue.

ptal @envoyproxy/gateway-maintainers

Should we also consider openssf best practices?

@zhaohuabing I believe scorecard evaluates based on best practices implemented, so implementing them will be a by product of the introducing the scorecard


create event on past day

System Information:

Version: 2.5
Language: en-US
Timezone Offset: -05:00
User Agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/47.0.2526.80 Safari/537.36

What steps will reproduce the problem?

I create an event using the following expression "something on Monday at 1 pm"

What is the expected result?
create the event on next Monday
What happens instead?
creates the event on last Monday
Screenshot URL:
Logs:

ops, looks like you already got enough reports related to it. any clue why they refuse to fix it?

¯_(ツ)_/¯


fix(tooltip): show tooltip on hover

Closes #233

@pimenovoleg this one is based on a branch of that PR https://github.com/radix-ng/primitives/pull/232 so it must wait until #232 gets marged


menu Debug->Refresh App

Debug->Refresh App is  bad designed here .
I worked 3 hours to tag the pictures. and  some object is very near by the menu Debug.
I was wrong to click to the Debug->Refresh App. then everything was over.
in the past 3 hours. I was at least wrong click to Debug  and one time click to Refresh App. because it is so closed to the object.
I think the Refresh App should have a pop up dialog ta ask if given up the previous work....
thanks god, I open it again, all thing are still there!
the menu is so close to the work space. hope can improve in next version

I will take a look glad you were able to recover your tags.


Table of Contents takes too much space

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to  muc.sre.xyz
Github Repository: https://github.com/SRE-MUC/sremuc

Expected behavior
Table of contents should not take so much space.
Screenshots

Hugo & Blowfish versions
HUGO_VERSION = "0.126.2"
Blowfish version = https://github.com/nunocoracao/blowfish/tree/c6c2689195eb47241c613a1e4b8d0828ebf3c6e4

I have disabled the TOC for articles. This helped a little, but blowfish still wastes lots of space. Can I fix this somehow?

Regarding the 1 point I can take a look and create a max size for the TOC should help... can you provide the example for that site so that I can try to reproduce it?
@shibumi to the second point this is using the logic provided by tailwind in order to optimize the number of characters per line that and increase readability for the end-user.

@nunocoracao what's optimization is this in Tailwind? Maybe it we could introduce the ability to make the content larger via a configuration setting, what do you think?

@nunocoracao  The site is here: content/events/sre-munich-summer-event-2024.md


Fix tarball ownership and containerd binary path for containerd.

@mikebrow @dmcgowan Fix the tarball permission and containerd binary path problem.
I've uploaded newest release tarballs.
Signed-off-by: Lantao Liu<EMAIL_ADDRESS>

Let's cherry-pick this into release/1.0 branch, so that it's easier to cut 1.0.x branch in the future.

LGTM


Async scheduler integration?

I am using async scheduler that put instantiate on the separate thread other than main thread. However I am having trouble injecting the instantiated object with Zenject factory or memory pool. Any way to inject a object instantiated on the separate thread over several frames?

Zenject is compatible with multiple threads, however you need to add the define ZEN_MULTITHREADING for this to work properly.  The only catch is that it won't be able to catch circular dependencies with that flag enabled.  One day hopefully we'll change the circular dependency logic to be thread safe.
I don't understand the problem you're having though.  When injecting on a different thread, it should work the same.  Can you elaborate on what doesn't work about that?


Can't target framework 2.1.3 after being upgraded to 2.1.4

I started developing a new website prior to .net core framework 2.1.4 was released, we got our production environment all set up with "Coming Soon" pages, etc.
As this server hosts other sites, we do not want to restart for an upgrade to 2.1.4, so I updated the csproj file with
<RuntimeFrameworkVersion>2.1.3</RuntimeFrameworkVersion>

I also checked the NuGet packages for Microsoft.NETCore.App and it is set to 2.1.3
I even went so far as to uninstall Microsoft .NET Core Runtime 2.1.402 from my dev machine
Yet, when I publish the site and copy it to the folder on the host, it still logs

It was not possible to find any compatible framework version The specified framework 'Microsoft.NETCore.App', version '2.1.4' was not found.

Check application dependencies and target a framework version installed at: "PATH"
Installing .NET Core prerequisites might help resolve this problem: http://go.microsoft.com/fwlink/?LinkID=798306&clcid=0x409
The .NET Core framework and SDK can be installed from: https://aka.ms/dotnet-download
The following versions are installed:
1.0.4 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
1.1.1 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
2.1.2 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]
2.1.3 at [C:\Program Files\dotnet\shared\Microsoft.NETCore.App]

I'm not sure if this is the right place to report this, but I am out of options at this point... So sorry if I'm at the wrong place.

/cc @livarcocc @dsplaisted
It sounds like 2.1.4 is getting burned into your deps file. Can you search for it in there in your published application? Published applications that run on the shared framework are supposed to specify 2.0.0 as the min version. I wonder if this is a regression in the tooling due to recent changes to try to keep more people up to date by default.

Issue moved to dotnet/cli #10037 via ZenHub


Update "text-inputs" page

Automatically generated by Netlify CMS

@sonnakim
I would like to update this page to accommodate the different states for Success, Warning, and Error. I think it would be helpful for users building forms to see live code visuals of these states.
I made some changes to the page structure to accommodate this. I think it could still use some tweaks but I wanted to share it with you to see what you think. Any feedback/suggestions are appreciated. Thanks!
https://deploy-preview-1880--cfpb-design-system.netlify.app/design-system/components/text-inputs

@natalia-fitzgerald I think this is fine! So the only change is to include and explicitly demo the states for success, warning, and error, is that right?

@sonnakim
After going back and forth a bit I decided to simplify the presentation on this page. I changed the Status styling section from "Status" to "Validation status" and I will only show the default states for these. A user can interact with the live code to trigger the Hover and Focus. This seems consistent with what we do for other DS components.
https://deploy-preview-1880--cfpb-design-system.netlify.app/design-system/components/text-inputs


Auth and API calls for multiple users in a single session

Hello!
We would like to be able to authorize multiple accounts (for example, a user's home account and work account) and make API calls for both accounts in one session (for example, get a list of all contacts for both home and work). Is this supported? If so, what is the best way to accomplish this?
Thanks in advance!

Hi @cgilboy, this is not currently supported. You can potentially authorize several accounts, by using gapi.auth2.authorize, store locally the different access_tokens received, and use gapi.client.setToken before each API call to set the specific user's access_token. Would that work?
However, it would not allow to easily fetch a fresh access_token when it expires. We are working on a solution to alleviate this issue though.

Hi @TMSCH, thanks for the quick response!
Your proposed solution (gapi.auth2.authorize multiple users and make requests w/ the resulting access_token) sounds like it would indeed work for our purposes. When the token expires, would we call gapi.auth2.authorize again with prompt: none? Are there other difficulties with refreshing the token?
Thanks!

Looking more closely, there does not appear to be a way to specify which access token/user to refresh with gapi.auth2.authorize. Is there another way to refresh a specific user/token?

@cgilboy you're right, this is not currently possible to refresh a specific token (but you're idea of calling authorize with prompt: 'none' is correct). We're already working on a solution to be able to do so, I will update this thread when it's deployed.

Great, thank you! In the meantime, can a refresh be accomplished via a direct HTTP request?

Without refresh token, unfortunately not.

Interesting! How does gapi.auth2.authorize accomplish refreshing without a refresh token?

The IFrame loaded by gapi.auth2.authorize is responsible for managing the token refresh, and does not provide the required data or mechanism to the client application, to avoid security issues.

Thanks for helping me understand this. If we want to make progress on supporting multiple users right now, is there a way we can use our server to refresh access_tokens on behalf of the browser clients?

For now, if the token expires, you will have to use gapi.auth2.authorize with prompt: 'select_account' so the user can select the right account to get a token from. This is unfortunately not a good UX... It depends how long your users's sessions are. The token expires in 1 hour.

Thanks again. I see that it is possible for installed apps to refresh their access tokens:
https://developers.google.com/identity/protocols/OAuth2InstalledApp#offline
I realize that we would not want to let our browser client have access to our refresh token or client secret, but would it be possible to use our server as a proxy to make these refresh requests?

You can use the server side flow, i.e. Authentication Code exchange to get a refresh token in the server and use it to get fresh access tokens: https://developers.google.com/identity/sign-in/web/server-side-flow

Can the fresh access tokens that the server retrieves be sent to and used by the client to make calls to Google APIs? From the docs, I could not tell if those access tokens are only be meant to be used by the server.

Yes, they can.

Great. Thanks again for bearing with me!

Hi again,
We have implemented the server-side flow as described in the above document. We got an auth code by calling grantOfflineAccess on the client, but when the server calls oauth2Client.getToken (from the nodejs lib) with the auth code or when we manually building a curl request to https://www.googleapis.com/oauth2/v4/token, we receive an access_token but not a refresh_token in return. Do you know if there’s currently any issue with the grantOfflineAccess call, or is there some other way to get a code that would be exchangeable for a refresh_token?
Thanks!

@cgilboy there should not be as far as I know. Could you paste:
window.open = (function() {
    var open = window.open;
	return function(url) {
      console.log(url);
      return open.apply(window, arguments);
    };
})();

In the console, and start the sign-in flow (with the popup opening) and let me know the URL logged in the console?

@TMSCH My mistake! We did successfully receive the refresh token once when we initially ran the authorization flow. Once the app was already authorized, we only received an access token. I believe this behavior is consistent with the documentation. Thanks again for your help.

Great! Closing the issue.

Is there a timeline of when an official solution will be available?

@sodacode could you be more specific? Are you looking for a solution to refresh a specific access_token without UI? (i.e. https://github.com/google/google-api-javascript-client/issues/310#issuecomment-309517122)

@TMSCH Is there an update for this statement? I would like to support multiple concurrent google accounts in a single session without having to use a server flow to grab the refresh token or requiring further user interactions after initially authorizing with the popup to confirm permissions. It appears you were working on a solution for this and were going to update this thread when it became available? Thanks you.

@cgilboy you're right, this is not currently possible to refresh a specific token (but you're idea of calling authorize with prompt: 'none' is correct). We're already working on a solution to be able to do so, I will update this thread when it's deployed.

Again the original request was to: "authorize multiple accounts (for example, a user's home account and work account) and make API calls for both accounts in one session" without requiring an outside workflow.


Any way to figure out code coverage or lines not covered?

This is probably a stretch, but is there a way to get code coverage reports? (percentage of lines covered, lines not covered, etc.)

That's surely beyond what Acutest can reasonably do on its own. With such a low-level programming language as C, you generally cannot do that without compiler support to generate such reports (and possibly other tools to visualize the results in a human readable form).
In a nutshell, build with gcc's options -g --coverage, then run the tests, and finally you may use gcov or lcov or other similar tools to process the collected stats into some human-readable form. I guess MSVC or other compilers can provide similar tools.
If it helps, I'm doing exactly that in CI builds of some of my other projects, so look at their CI build scripts (here the collected stats are uploaded to https://codecov.io/ which is able to visualize the reports nicely):

MD4C (and the result)
Centijson (and the result)


Allow order fields to be subqueries with order

Fixes: https://github.com/ManageIQ/manageiq-ui-classic/issues/5374
Overview
Virtual fields that reference a has_many relationship, often need to have a LIMIT and ORDER in the field definition.
When rails works with these fields in an ORDER BY clause, they sometimes add that field to the SELECT clause for Sql reasons. In that process, they munge the field and break things.
Before
We see this issue on the Vm explorer page. If we sort by compliance (i.e. last_compliance_status) it blows up:
VmOrTemplate.includes(:hardware => :disks).references('x').order(:last_compliance_status).first

ActiveRecord::StatementInvalid (PG::InvalidColumnReference: ERROR:  for SELECT DISTINCT, ORDER BY expressions must appear in select list)
LINE 1: ..."disks"."hardware_id" = "hardwares"."id" ORDER BY (SELECT  "...

After
I put in a Rails PR that avoids string munging and leverages Arel. In the mean time, this PR is a monkey patch to our codebase with the same code that fixes the problem in the short term.

re robocops - I copy and pasted these bits from the rails code.
I am aiming to keep it as close to rails as possible.

cc @jrafanie

@kbrock Can you make the first commit the current code in rails (include which rails version you're pulling from... I have no idea if it's changed in 5.1 -> 5.2 -> 6), and then apply your patch in the second commit... It should make this easier to dissect.   I'm pretty lost.

@jrafanie ok, updated. is this easier to figure out?

Note, ignoring cops because it's rails code... we're trying to avoid making unrelated changes to the core rails code.

@kbrock did you test the UI suite with this PR?

@jrafanie ui classic passed

@miq-bot add_label rails5.2


[Autocomplete] Fix autoHighlight behavior

[x] I have followed (at least) the PR section of the contributing guide.

Closes #20594

Can we merge this since it's all green ?

Actually, your fix is different than mine, and I don't know what the correct behaviour should be.
The docs says : If true, the first option is automatically highlighted.
However, your fix allows the first selected option to be automatically highlighted, while mine was correctly highlighting the first option

Nevermind, I'm gonna rollback that last commit

@qkdreyer Cool, I could notice the same issue without freeSolo set. Let's merge :)


Evaluate relationship blocks in Resource context

Consider a serializer:
class SerializableEmployee < JSONAPI::Serializable::Resource
  attribute :title do
    primary_office.title
  end

  has_one :primary_office do
    data { primary_office }
  end

  private

  def primary_office
    @object.offices.sort_by(&:rank).first
  end
end

The above code will not work, because the has_one data block will eval in the context of a Relationship instance (ie self is_a JSONAPI::Serializable::Relationship). This means I cannot access other methods within the serializable (ie primary_office).
It's confusing, as we would have access to @model and other blocks (like the attribute block above), do eval in the context of the serializable.
I'd prefer this logic live in the serializable, not the model, as it's serialization-specific (For instance, a primary_office method on the resource may fire a query instead of sorting already-loaded objects).

You are right. The way things work currently is that relationship blocks are evald in a specific context in which exposures are forwarded.
A quick workaround for your situation would be to move the method definition inside the has_one block.
I am not sure what the best tradeoff is here.

@beauby the method needs to be shared with the rest of the serializable, though, so I'm not sure that's a DRY solution. Is there a downside to evaling these blocks in the serializable context?

IMHO if it's eval'd in the resource context there needs to be another variable to access the relationship object

So one possible solution to your problem @richmolj would be to forward to base object upon missing method. What do you think?

Sounds like a performance issue?

I doubt it would be a bottleneck but I'll try to hack something together and we'll benchmark it.

I'm experiencing this same problem with a serializer I'm trying to define. I'm not too familiar with the jsonapi-rb internals, but I have a couple of ideas for tackling this.
If method_missing is too much of a concern, one idea could be to provide a way of accessing the resource object from within the relationship context.
attribute :title do
  primary_office.title
end

has_one :primary_office do
  # `resource` could instead be `@resource`, but that would conflict with exposures
  data { resource.primary_office }
end

private

def primary_office
  @object.offices.sort_by(&:rank).first
end

Another idea might be to fully scope helper methods within a defined abstraction. That would force attribute and relationships to use the same interface:
attribute :title do
  helpers.primary_office.title
end

has_one :primary_office do
  data { helpers.primary_office }
end

helpers do
  def primary_office
    @object.offices.sort_by(&:rank).first
  end
end

Maybe an arbitrary def foo outside of helpers (at the same level as attribute do ... end) is unavailable within blocks in this case?
Depending on how complicated this gets with all the overlapping contexts and generic instance variables, methods defined in helpers could be required to take parameters to enforce structure:
attribute :title do
  helpers.primary_office(@object)
end

helpers do
  def primary_office(employee)
    employee.offices.sort_by(&:rank).first
  end
end

Related to this problem: link has the same issue:
link :related do
  # unable to reference methods defined at the serializer level
end

I'm not sure there's a work-around either. def foo won't quite work the same way because there's no nested DSL form for link.


Update and publish docs for 1.0

Update documentation in the README, review all API and Marginalia documentation and generally clean up in preparation for 1.0.

🤷♂


Add basic zsh module

This module is based on programs/bash.nix, it has only basic functionality and should be extended in the future (e.g. add oh-my-zsh support).

Thanks for this module! You should be aware of the #28 PR, which will deprecate the "bash" option in home.sessionVariableSetter and instead only have "shell".
Is zsh compatible with Bash w.r.t to the export command? I think it is but I'm not certain. If it is then this module would just have to source the shell script generated by the module in that PR…
In any case, if you fix the two issues I mentioned I'll merge. Also please change the commit message to read zsh: add module.

@adisbladis Another module for your nice envvars stuff :-)

Thank you for feedback. I fixed two issues you mentioned.
As for compatibility of the export command, yes, as far as I know, it should be fine to use the same script for both bash and zsh.

Looks good! When you are ready please squash together and let me know and I'll merge.

I also added two more options: enableCompletion and enableAutosuggestions, they were mirrored from zsh.nix NixOs module.

Done

Awesome! Rebased into master in cde8e02bf268f86c5ec894a088b2bb5f03b9f59d. Thanks :-)


Super sample blended images

This improves the issue described here: https://github.com/mwydmuch/ZoomVideoComposer/issues/16
It is currently solved by the suggested method of scaling the images before creating the frames. It seems to fix the issue quite well. The improvement is noticeable at scale factor 2 and I couldn't notice any jitter at all anymore at factor 4. It looks like the render time more or less doubles with each super sample factor step.
Test render times
20s clip at 60fps

supersample 1 (no effect):
94s
https://youtu.be/IGAOZLIjaXY

supersample 2:
158s
https://youtu.be/p6GayUAnBZ4

supersample 3:
279s
https://youtu.be/tqZPl2PTa7Y

supersample 4:
560s
https://youtu.be/ekc9XuGE8bM

Thank you for the @sambernaerdt PR. That's a good solution, for sure. However, the current code doesn't work due to a few issues I pointed out in other comments. Are you sure you pushed your latest version?

Hm. I'm not sure what happened... I've pushed the latest changes and it should work now.
I will work my way through the other comments in the near future...

Thank you for the PR. Now, it works great. I've made small changes and merged this.


init testing

And experiment with prow

/ok-to-test

/verify-owners

Issues:

Immediately added the approved label as soon as PR was created.
Isn't recognizing that I'm indeed a member of the org and not throwing error.

/lgtm

/verify-owners

/label mylabel


Gemini Pro and Flash 002

Is it possible to access those models?

You can now pass custom model names. Solved in #53


fix: useFeedbackMessage methods returns a promise

Description

[useFeedbackMessage]
Updated the component to have the methods (info, loading, success, warning, error) to have return a Promise (of type boolean).
Issue ticket number and link

The issue has been notified in #331 .
Checklist

[X] added the link to the Jira task
[X] commit message and branch name follow conventions
[X] tests are included
[X] changes are accessible and documented from components stories
[X] typings are updated or integrated accordingly with your changes
[X] all added components are exported from index file (if necessary)
[X] all added files include Apache 2.0 license
[X] you are not committing extraneous files or sensible data
[X] the browser console does not have any logged errors
[X] necessary labels have been applied to this pull request (enhancement, bug, ecc.)

conflicts on useFeedbackMessage.types.tsx; apart from that lgtm


i18n(zh-cn): Update builderio.mdx

Description (required)
Update builderio.mdx
Related issues & labels (optional)

#9611
Suggested label: i18n

Lunaria Status Overview
🌕 This pull request will trigger status changes.

Learn more
By default, every PR changing files present in the Lunaria configuration's files property will be considered and trigger status changes accordingly.
You can change this by adding one of the keywords present in the ignoreKeywords property in your Lunaria configuration file in the PR's title (ignoring all files) or by including a tracker directive in the merged commit's description.

Tracked Files

Locale
File
Note

zh-cn
guides/cms/builderio.mdx
Localization changed, will be marked as complete.

Warnings reference

Icon
Description

🔄
The source for this localization has been updated since the creation of this pull request, make sure all changes in the source have been applied.


(LOCAL FOLDER): UTF-8: surrogates not allowed

What version of Music Assistant has the issue?
2.3.3
What version of the Home Assistant Integration have you got installed?
2024
Have you tried everything in the Troubleshooting FAQ and reviewed the Open and Closed Issues and Discussions to resolve this yourself?

[X] Yes

The problem
When I open Browse and select my filesystem, I get the error:
str is not valid UTF-8: surrogates not allowed
On the console, I see:
2024-11-22 18:28:49.950 ERROR (MainThread) [music_assistant.webserver] Error handling message: music/browse: str is not valid UTF-8: surrogates not allowed
I tried to run the linux command detox and deleted my docker data for music assistant, but the issue is still there
How to reproduce
Open browse and select my filesystem
Music Providers
Filesystem local disk
Player Providers
Home Assistant
Full log output
No response
Additional information
No response
What version of Home Assistant Core are your running
2024.10.4
What type of installation are you running?
Home Assistant Container
On what type of hardware are you running?
Linux

My fault. Sorry detox did not run through :(


Push fails for LFS-enabled repositories

Gogs version (or commit ref): <IP_ADDRESS>2
Git version:

server: 2.11.0
client: 2.16.2

Git LFS version:

server: n/a
client: git-lfs/2.3.4 (GitHub; linux amd64; go 1.9.2; git 7f83b43)

Operating system:

server: Debian 9.4
client: Arch Linux

Database (use [x]):

[ ] PostgreSQL
[ ] MySQL
[ ] MSSQL
[x] SQLite

Can you reproduce the bug at https://try.gogs.io:

[x] Yes (provide example URL) - Can only provide empty git repository as the issue prevents me from pushing (see further details[1] below)
[ ] No
[ ] Not relevant

Log gist (usually found in log/gogs.log):

[1]: Steps to repro: Initialize new repo on a client that has LFS enabled; add an arbitrary binary file and track it with LFS. Commit and push to this repo. It will fail with (the try.gogs.io server should probably have a FATAL log entry in its serv.log (see below)):

Git LFS: (0 of 1 files) 0 B / 512 B
batch response: Repository or object not found: https://try.gogs.io/bossi/foo.git/info/lfs/objects/batch
Check that it exists and that you have proper access to it
error: failed to push some refs to 'https://try.gogs.io/bossi/foo.git'

Description
Pushing to a Gogs repository seems to cause the server to fail with the following message getting logged in <installRoot>/log/serv.log:

[FATAL] [...its/gogs/cmd/serv.go:166 runServ()] Repository does not exist: /.git upload

This is most likely specific to the following configuration:

repository has more than one remote configured, with the "primary" (tracking) having LFS support and gogs being one of the secondary (precedence irrelevant though)
git lfs url is not explicitly specified in the repository's .git/conf

This causes git lfs to try and push to the gogs repository as well on push and cause the above failure.
Temporary Solution
Setting git config --add lfs.url <httpLfsUrl.git/info/lfs> explicitly solves the problem, getting git to  always push LFS objects to that (non-gogs) LFS endpoint, even when pushing to gogs.

Hi, Git LFS is not currently supported, please see https://github.com/gogits/gogs/issues/1322.

Thanks @Unknwon - I was aware of the lack of active support but thought this edgecase might still be worth mentioning as it comes across a tad like buggy behavior still, even knowing there is no active LTS support on Gogs at the moment - seems to sit somewhere in between :thinking:


Typescript errors when building my project

I am attempting to build my Node.js and typescript project, and am running into a couple of issues.
node_modules/gaxios/build/src/index.d.ts(22,12): error TS1005: '>' expected.
node_modules/gaxios/build/src/index.d.ts(22,49): error TS1109: Expression expected.
gaxios is being used as a dependency of Google-Auth-Library and it's using version 1.2.1.
Is there a fix I can make myself for this?

Greetings!  What version of TypeScript and Node.js are you using?

I hit the same issue using TypeScript 2.8.1 and Node.js 8.11.3

Thanks!  Could you try the latest version of TypeScript?  This is really only compiled on the latest version.

There is no issue on the latest version TypeScript version ^3.3.0. However, I was hoping there would an alternative solution to upgrading the TypeScript version, since that requires quite some work in our current project.

Yeah, apologies - that's just not something we're going to be able to support :/  We're pretty aggressive on staying with the latest version, because you can run into issues in the other direction where people are on latest and having issues with previous versions.  In the general sense - I'd really reccomend staying on top of TypeScript releases with a tool like renovate.  Hope this helps!


FEATURE REQUEST: Add Email Link Auth

Is your feature request related to a problem? Please describe.
Add Email link Auth
Describe the solution you'd like
Firebase Authentication makes it easy to sign users in. Email link auth will be nice, since we can verify users while signing them up and it's also secure.

@darrt2008 Will be requested a review for the PR.

People, please look alive. I need you to contribute to the project.


add adding app inside init_app

according to documentation I need to run
db = SQLAlchemy()
def create_app():
    app = Flask(__name__)
    db.init_app(app)
    return app

to initialize app. there's need to set
self.app = app

insite init_app, because there's error if you don't set self.app.

that goes explicitly against using the db with more than one app,
whats the actual error?

Duplicates #279 (and probably many others).

Closin'


labels: Increase specificity of label selector

Fixes https://github.com/openownership/visualisation-tool/issues/134

@radix0000 Whoops, misread a couple of things there. Please see the update which should sort things out.

Looks good to me also. Thank you all for making this change.