[Bug]: The source map is not working on Sentry

System Info
System:
OS: macOS 15.0
CPU: (10) arm64 Apple M1 Max
Memory: 2.03 GB / 64.00 GB
Shell: 5.9 - /bin/zsh
Binaries:
Node: 22.8.0 - ~/.nvm/versions/node/v22.8.0/bin/node
npm: 10.8.3 - ~/.nvm/versions/node/v22.8.0/bin/npm
bun: 1.1.27 - /usr/local/bin/bun
Browsers:
Chrome: 128.0.6613.138
Safari: 18.0
Details
The source maps stopped working on Sentry. After discussing with their team, we identified that the issue was caused by an invalid source map generated by Rspack.
When testing the source map with source-map-explorer or other tools, the following error appeared:
'Your source map refers to generated column 53099 on line 39, but the source only contains 53098 columns on that line. Please verify that you are using the correct source map.'
Approximately 70% of the source map files we tested are working, while 30% have failed.
Reproduce link
No response
Reproduce Steps
https://github.com/cjnoname/rspack-reproduce
npm run s

please provide rspack version

please provide rspack version

1.0.5 the latest

please provide rspack version

I could send you the buggy soucemap files and mjs files via email or whatever tools but I don't want to share them with public.

I think this is the same issue as https://github.com/web-infra-dev/rspack/pull/7647. we found that source map stopped generated properly since 1.0.1 version. @cjnoname, you can try the 1.0.0 version of rspack to identify when it was broken.

I think this is the same issue as https://github.com/web-infra-dev/rsbuild/issues/3433 we found that source map stopped generated properly since 1.0.1 version. @cjnoname, you can try the 1.0.0 version of rspack to identify when it was broken.

@hardfist
This is a significant issue since Sentry is the primary monitoring tool for many companies, and I hope someone can assist in looking into it.
Verifying the issue is straightforward - a single command line should work:
npx source-map-explorer index.mjs index.mjs.map

@hardfist
This is a significant issue since Sentry is the primary monitoring tool for many companies, and I hope someone can assist in looking into it.
Verifying the issue is straightforward - a single command line should work:
npx source-map-explorer index.mjs index.mjs.map

I can't investigate more without any reproductions

@hardfist
This is a significant issue since Sentry is the primary monitoring tool for many companies, and I hope someone can assist in looking into it.
Verifying the issue is straightforward - a single command line should work:
npx source-map-explorer index.mjs index.mjs.map

@cjnoname I can't investigate more without any reproductions, maybe you should try upgrade to 1.0.5 to see whether the sourcemap problem is solved

https://github.com/cjnoname/rspack-reproduce
You can find the issue with a simple command: "npm run s".
I am not sure if this is the only issue causing the sourcemap to break, or if there might be other issues as well.

maybe you should try upgrade to 1.0.5 to see whether the sourcemap problem is solved

Hey hi, guys 👋
Would like to confirm that the latest v1.0.5 has a broken source map.
We upgraded from 0.7.5 to 1.0.3 and noticed an issue with the broken source-map - there is a big % of [no source] code based on source-map-explorer package analyse.
(NOTE: you need to run the package with --no-border-checks key to process the broken source maps completely, e.g: npx source-map-explorer --no-border-checks ./dist)
So we downgraded all @rspack package to stable v1.0.0

@yuzhva @cjnoname sorry for the inconvenience and reproduction, confirmed it's a bug we are gonna fix it ASAP.

@yuzhva @cjnoname sorry for the inconvenience and reproduction, confirmed it's a bug we are gonna fix it ASAP.

I’ve put together a minimal reproduction repository last night. While I haven't done something exactly like this before, it should accurately represent the issue.
https://github.com/cjnoname/rspack-reproduce
Not sure if it's the only issue on sourcemap or not.

I am currently investigating this issue.

@cjnoname Currently, it looks like the issue with the source map is caused by the swc minifier. Please use the Terser Webpack Plugin to handle code minification first.
If the issue persists, there may be other factors causing the source map generation error. Please provide additional repos to reproduce the problem.

In this repository, you can observe that the source maps generated by SWC cause errors in source-map-explorer.
https://github.com/SyMind/swc-minifier-source-map-bug

In this repository, you can observe that the source maps generated by SWC cause errors in source-map-explorer. https://github.com/SyMind/swc-minifier-source-map-bug

Thank you for investigating this issue. I understand that the SWC minifier is also maintained by your team, correct?
Could you let us know if there are any plans to address this issue in the near future? We are willing to wait a few days if necessary.
We are experiencing poor performance with the Terser Webpack Plugin, which we would prefer not to revert to.
Do you have an estimated timeline for a fix?
Thanks

@cjnoname I'm not certain if the issue is actually caused by the SWC minifier. Could you please try using the Terser Webpack Plugin and see if the problem persists?

@yuzhva Please provide the a demo to reproduce the issue.

@cjnoname I'm not certain if the issue is actually caused by the SWC minifier. Could you please try using the Terser Webpack Plugin and see if the problem persists?

The issue persists even after I turned off the minimizer and disabled output: module. It still occurs. You can easily check it in my repo by adjusting the rspack.config.js file.

@cjnoname You can see the code in source-map-explorer here: https://github.com/danvk/source-map-explorer/blob/master/src/lib/explore.ts#L208

@cjnoname You can see the code in source-map-explorer here: https://github.com/danvk/source-map-explorer/blob/master/src/lib/explore.ts#L208

It works, but it's running much slower than before. Could you help me report this issue to SWC?

@cjnoname Yes. If you can confirm that the issue is caused by the SWC minifier, I will continue to work on it.

@cjnoname Yes. If you can confirm that the issue is caused by the SWC minifier, I will continue to work on it.

Yeah... Terser Webpack Plugin works fine for now. Please go ahead.

Here is a fork with an example for source-map "investigation":
https://github.com/yuzhva/rspack-repro
I just forked the original repo https://github.com/web-infra-dev/rspack-repro and enabled the source map for rsbuild.config.ts and rspack.config.mjs configs:
Results of Source Map investigation just open:
For v1.0.5:

webpack: ./webpack-dist-v105/webpack-dist_main.js-Source_Map_Explorer.html
rspack: ./rspack-dist-v105/rspack-dist-v105_main.js-Source_Map_Explorer.html
rsbuil: ./rsbuild-dist-v105/rsbuild-dist-v105_static_js_index.1a9c415f.js-Source_Map_Explorer.html

@SyMind
cc: @cjnoname
I can confirm that the root cause is in swc minifiyer.
Source Map works with one of the next configs:

disable modifications:

optimization: {
  minimize: false,
},

pass empty array as minimizer key for config:

optimization: {
  minimize: true,
  minimizer: [],
},

Or use the Terser Webpack Plugin:

const TerserPlugin = require("terser-webpack-plugin");

optimization: {
  minimize: true,
  minimizer: [new TerserPlugin()],
},

The Terser - is extremely slow.

Our current configuration right now (which does not work):
const { SwcJsMinimizerRspackPlugin } = require('@rspack/core');

optimization: {
  minimize: true,
  minimizer: [new SwcJsMinimizerRspackPlugin()],
},

same here

Looks like SWC fixed the issue, thanks @SyMind for posting it there! Out of curiosity, how does the rsbuild release process look like for downstream dependency updates (such as SWC)?

Looks like SWC fixed the issue, thanks @SyMind for posting it there! Out of curiosity, how does the rsbuild release process look like for downstream dependency updates (such as SWC)?

normally we bump swc version in minor release, but if it contains hotfix bug, we may release early

Looks like SWC fixed the issue, thanks @SyMind for posting it there! Out of curiosity, how does the rsbuild release process look like for downstream dependency updates (such as SWC)?

normally we bump swc version in minor release, but if it contains hotfix bug, we may release early

Any plan to release a new version soon?

@cjnoname it's already available in 1.0.7

Nope

Ouch. You're right. My B.

Seems like this SWC version is buggy
https://github.com/swc-project/swc/issues/9592

Seems like this SWC version is buggy

@cjnoname  Thanks for your reminder 🌹  . Rspack currently does not use the functions related to this issue. If there are any problems in the future, we will follow up in time.
And it's already available in 1.0.8.

Seems like this SWC version is buggy

@cjnoname Thanks for your reminder 🌹 . Rspack currently does not use the functions related to this issue. If there are any problems in the future, we will follow up in time.
And it's already available in 1.0.8.

This is awesome. Much appreciated!

@cjnoname Yes. If you can confirm that the issue is caused by the SWC minifier, I will continue to work on it.

Hi mate, it's still not working... due to a different issue. Please refer to the same repo to reproduce.
https://github.com/cjnoname/rspack-reproduce
Thanks.

 1.0.8

devtool: 'source-map'

I can confirm the issue is still there in 1.0.10. Just like before, source-map is broken, cheap-module-source-map works fine. I see the issue is being discussed in other threads (e.g., https://github.com/web-infra-dev/rspack/issues/8021), is there an official issue tracking this bug and do we know if it's SWC or rspack/rsbuild-related? Is there any workaround besides downgrading to Rspack 1.0.0?

This is actually a serious bug that prevents people from using Rspack normally. I hope someone can fix it soon so Rspack can return to normal use.

Today, I checked and found that the issue with SWC still exists. Previously, I upgraded SWC and did some local testing. It might be because I modified the source-map-explorer code, which led me to mistakenly believe that the issue with SWC had been fixed.
I will continue working to resolve this issue.

Today, I checked and found that the issue with SWC still exists. Previously, I upgraded SWC and did some local testing. It might be because I modified the source-map-explorer code, which led me to mistakenly believe that the issue with SWC had been fixed.
I will continue working to resolve this issue.

Thanks for fixing the previous issue with Sentry—it worked as expected. However, there's a different issue with some other packages that I'm quite certain about. I'm also a bit concerned there might be additional issues now that the original one has been resolved.

@cjnoname I've released a canary version of Rspack 1.0.9-canary-5f4b3e7d-20241010033454. Could you please verify if it works correctly? Thank you!

@cjnoname I've released a canary version of Rspack 1.0.9-canary-5f4b3e7d-20241010033454. Could you please verify if it works correctly? Thank you!

Hey mate, I could not find your version

@cjnoname Please use the following package to replace @rspack/core<EMAIL_ADDRESS>Update your dependencies as shown below:
"@rspack/core": "npm<EMAIL_ADDRESS>

@cjnoname Please use the following package to replace @rspack/core<EMAIL_ADDRESS>Update your dependencies as shown below:
"@rspack/core": "npm<EMAIL_ADDRESS>
It works mate, thank you

@SyMind how about us folks using Rsbuild 👼 ?

This change is included in<EMAIL_ADDRESS>right? (I see it includes rspack@1.0.9). For what is worth, it's still not working for us 😿
Ton of sourcemaps area are unreachable and can't be debugger. Downgrading Rsbuild (a lot) fixes it. I'll try to investigate a bit more, but  I'm a bit lost at the moment.

@SyMind what about us folks using Rsbuild 👼 ? Will this be included in the next release?

<EMAIL_ADDRESS>does not include this change, but it should be included in the next version.

Can we release it as soon as possible? You wouldn't want to see a bundling tool that doesn't support source maps, right? :)

@mmazzarolo To override @rspack/core, use the following configuration. This will include the next version:
"@rspack/core": "npm<EMAIL_ADDRESS>

@cjnoname I will release a new version today.

@mmazzarolo To override @rspack/core, use the following configuration. This will include the next version:
"@rspack/core": "npm<EMAIL_ADDRESS>
I'm on rsbuild so the process is a bit different but good to know, thanks! Thought it was released because I saw the issue being closed :)

@cjnoname I have released the 1.0.10 version https://github.com/web-infra-dev/rspack/releases/tag/v1.0.10

In 1.0.10 version source maps have been restored for 27.4% for our project - it's still better than having 100% of broken source maps.
It's a bit hard to debug to understand what exactly part of the code does not receive generated source maps: the app code, .mjs modules, or something else 🤷
Will try to debug it more to understand which parts of source maps are missing...

@mmazzarolo can you give me a reproduce.

I've been trying, but I can't produce a "minimal" reproducible example 😔. Out production codebase is pretty big and as @yuzhva mentioned it's hard to determine what's causing the issue and putting it into a minimal reproducible example. I tried using same RSBuild config and some similar npm packages for example, but with no success.
The only thing I can confirm is that our issues have been introduced in https://github.com/web-infra-dev/rspack/releases/tag/v1.0.1 (I assume it's from https://github.com/web-infra-dev/rspack/pull/7647) — if we revert to a version of rsbuild that depends on 1.0.0 everything works fine (we're basically stuck in this state since our app is otherwise unusable without working sourcemaps).

I will release a new version for rspack today to fix this issue.

@mmazzarolo I've released a canary version of Rspack<EMAIL_ADDRESS>Could you please verify if it works correctly? Thank you!
To override @rspack/core, use the following configuration:
"@rspack/core": "npm<EMAIL_ADDRESS>

@SyMind thanks for looking into ti!
I assume you mean to add it as an override/resolution, right?
So adding:
  "resolutions": {
    "@rspack/core": "npm<EMAIL_ADDRESS>  },

If I do that, the install step (using pnpm i) goes fine, but building fails:
  Rsbuild v1.0.14

error   Failed to build.
error   Unmatched version<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>    at checkVersion (/Users/matteo.mazzarolo/workspace/TTVPlatform/node_modules/@rspack/core/dist/index.js:16986:21)
    at #getInstance (/Users/matteo.mazzarolo/workspace/TTVPlatform/node_modules/@rspack/core/dist/index.js:17488:19)
    at #build (/Users/matteo.mazzarolo/workspace/TTVPlatform/node_modules/@rspack/core/dist/index.js:17412:22)
    at /Users/matteo.mazzarolo/workspace/TTVPlatform/node_modules/@rspack/core/dist/index.js:17384:18
    at done (/Users/matteo.mazzarolo/workspace/TTVPlatform/node_modules/@rspack/lite-tapable/dist/index.js:473:13)
    at AsyncSeriesHook.callAsyncStageRange (/Users/matteo.mazzarolo/workspace/TTVPlatform/node_modules/@rspack/lite-tapable/dist/index.js:480:20)
    at AsyncSeriesHook.callAsync (/Users/matteo.mazzarolo/workspace/TTVPlatform/node_modules/@rspack/lite-tapable/dist/index.js:82:21)
    at _Compiler.compile (/Users/matteo.mazzarolo/workspace/TTVPlatform/node_modules/@rspack/core/dist/index.js:17378:30)
    at /Users/matteo.mazzarolo/workspace/TTVPlatform/node_modules/@rspack/core/dist/index.js:17240:16
    at done (/Users/matteo.mazzarolo/workspace/TTVPlatform/node_modules/@rspack/lite-tapable/dist/index.js:473:13)
/Users/matteo.mazzarolo/workspace/TTVPlatform/apps/studio:
 ERR_PNPM_RECURSIVE_RUN_FIRST_FAIL<EMAIL_ADDRESS>build: `./scripts/build-app-with-rsbuild.sh`
Exit status 1
 ELIFECYCLE  Command failed with exit code 1.

Not sure why. Pinning specific versions of @rspack/core has worked for me before, though. Just not this one.

I'm having the same problem with Sentry sourcemaps, and I think this canary build does not work with @rspack/cli. I keep having unmatched version error, and following the dist code I think @rspack/core thinks its on version 1.0.11, while bindings give the canary version.
[rspack-cli] Error: Unmatched version<EMAIL_ADDRESS><EMAIL_ADDRESS><EMAIL_ADDRESS>

@SyMind was the fix shipped with<EMAIL_ADDRESS>Asked because i tried forcing it as a resolution to use it with rsbuild and the issue is still there for us.

Can confirm, issue is not resolved on 1.0.12. I guess we will have to wait until the next release.

Yes! It seems resolved, I can see the sourcemaps on sentry. Thank you for your help.

In 1.0.10 version source maps have been restored for 27.4% for our project - it's still better than having 100% of broken source maps.

In v1.0.13 source maps are resolved for 77.6%
Here are screenshots:
...

In 1.0.10 version source maps have been restored for 27.4% for our project - it's still better than having 100% of broken source maps.

In v1.0.13 source maps are resolved for 77.6%
NOTE: the same amount of source maps was broken in v0.7.5, so it seems to be an old issue, which is not related to the update to v.1
Here are screenshots:
v1.0.10 - 27.2% of source maps are works:
v1.0.13: - 77.6% of source maps are works:
TerserPlugin plugin resolves 100% of source maps:
It seems that:

11.2% of broken source maps are still in node_modules
another 11.2% of broken source maps are still in coudebase

How about the RC version mate, was it 100%?

Thanks @SyMind 🙏 This seems to be fixing the issues so far. But we'll need a bit more time to do a complete check.
@cjnoname my understanding from @yuzhva is that the current version (the one with the fix) has a coverage of 77.6% for him. But the coverage with rspack/swc might have never been 100% (even before 1.0.1, which we suspected being the version that introduced the issue).
TerserPlugin does provide a complete coverage though... it would be interesting to understand why we can't do that with rspack/swc.

Hey guys, I can confirm that the source map is inaccurate, especially as it shows errors on the wrong lines of code.

Thanks @SyMind 🙏 This seems to be fixing the issues so far. But we'll need a bit more time to do a complete check.
@cjnoname my understanding from @yuzhva is that the current version (the one with the fix) has a coverage of 77.6% for him. But the coverage with rspack/swc might have never been 100% (even before 1.0.1, which we suspected being the version that introduced the issue). TerserPlugin does provide a complete coverage though... it would be interesting to understand why we can't do that with rspack/swc.

if terserPlugin provides complete coverage and swc don't provide, then it's normally a swc minifier bug, we may need more information to investigate swc minifier problem

I will continue to make efforts on the SWC minifier, trying to bring it to 100%. However, this process might be quite slow.

@yuzhva can you give me a reproduce repo?


Updated documentation of Tweet class to match code.

README used new Tweet but the Tweet class has a named constructor ::make()

Coverage remained the same at 100.0% when pulling 70292aa0338892199087d536abfc8b18b28476c3 on Brunty:fix-readme into<PHONE_NUMBER>f59333dab534ab7e176d72391a7557 on jedkirby:master.

Coverage remained the same at 100.0% when pulling 70292aa0338892199087d536abfc8b18b28476c3 on Brunty:fix-readme into<PHONE_NUMBER>f59333dab534ab7e176d72391a7557 on jedkirby:master.

Great spot, thanks @Brunty !


Automatically complete statement on semicolon

It works like Visual Studio C# IntelliSense 'Automatically complete statement on semicolon'

The setting is called editor.acceptSuggestionOnCommitCharacter

Hi, jrieken:
Thank you for your reply. I didn’t make it clear enough.
I mean the function that is within parentheses and you type a semicolon , it automatically moves it to the end of the brackets
even no parameter method.


Correct sensors

This PR updates some units and resolutions of sensors. I guess this is a good opportunity to discuss how we should handle database migrations coming from plugins. Just a proposal: add a CLI command in the plugin, one for each migration?

Here is one direction to think towards: Each plugin can use its own alembic migrations. The plugin could configure that its table is {plugin_name}_alembic_version, so its history of migrations is not mixed with the one from FlexMeasures.
Of course, one needs to be careful with changes. If the plugin authors remove a column FlexMeasures works with, for example, things will break. But right now it seems doable.
That would be one step more for plugins to be full projects on their own.

Worthwhile to note: this PR also moves from an hourly to a quarterhourly CO₂ signal.
Migration steps for correcting units and resolutions:
update sensor set (unit, event_resolution) = ('MW', interval '15 minutes') where (name='Scheduled generation' and unit='MWh' and event_resolution=interval '1 hour');
update sensor set unit='MW' where (name='Solar' and unit='MWh');
update sensor set unit='MW' where (name='Onshore wind' and unit='MWh');
update sensor set unit='MW' where (name='Offshore wind' and unit='MWh');
update sensor set event_resolution=interval '15 minutes' where (name='CO2 intensity' and unit='kg/MWh' and event_resolution=interval '1 hour');

Then rerun flexmeasures entsoe import-day-ahead-generation --from-date <however far your history went> to fill in the missing quarterhourly values.
Finally, the migration steps for correcting names:
update sensor set name='Wind Onshore' where (name='Onshore wind');
update sensor set name='Wind Offshore' where (name='Offshore wind');
update sensor set name='CO₂ intensity' where (name='CO2 intensity');


lonely > on a new line when not necessary

Describe the desired feature/functionality
Just a small feature request. I would understand if we choose to mark this as wont-fix.
A few generated code snippet shows the closing > on a new line when no props are present, making my prettier head twitch. This is drawer subheader's playground, with its default configuration:

Suggested implementation details
When no prop is present, the > shall be on the same line as the component name.

Please refer to AppBar playground utility function.
https://brightlayer-ui-components.github.io/react-dev/components/app-bar/playground


🐛 [firebase_ml_vision] Can not build in android

I tried the sample of firebase_ml_vision
But I could not build.

I am using flutter 2.5.3 stable.
How can I fix this issue?

Hi @liqiang1995, please share the output of flutter doctor -v the Android OS version and the version of the firebase_ml_vision for which the app fails to build.
Thanks

according to the flutter dev the firebase_ml_vision currently discounted. I think they will release new update very soon. You may visit https://pub.dev/packages/firebase_ml_vision for more info.

Thanks, @liqiang1995, Closing as firebase_ml_vision is no longer being maintained.


Update polymorphic relationships have no effect

Hello, and thank you for your awesome work.
I tried to update a polymorphic (one to many or many to many with pivot table) relationship but the relationship do not update in the database. I cannot explain why. I had to add a update method to my Adapter which updates the relationship manually.
Here is the data sent by the client :
data: {   id: "1557",    type: "precaution",   attributes: {     ...attributes   },   relationships: {     precaution_cible: {       data: {         type: "principe-actif", id: "163"       }     }   } }
Thank you

Are you using the $fillable property on the adapter, and if so, is the relation listed in it?

My bad, I did not add the relation in my Adapter class.
Sorry for the inconvenience.


Fixes for malformed codes and valuesets

Fixes null pointer exceptions when codes are incompletely specified
Fixes null pointer exceptions when valuesets are incompletely specified
The above is accomplished by ignoring (i.e. filtering out) partially specified codes

resolves #414


Fixed #35732 -- Wrapped Postgresql Concat args in parenthesis to fix operator precedence bug.

Trac ticket number
ticket-35732
Branch description
A change from 5.0.8 to 5.1 raised a test failure in one of our queries that combines Concat with TrigramSimilarity. @emicuencac tracked this down to a recently deployed simplification on how Concat is rendered to sql here https://github.com/django/django/pull/17471 which leads to the pipe operator not being wrapped in parenthesis which were implicit when using CONCAT(...).
Sorry for the draftiness of this PR, we will amend any mistakes you point out. We wrote a proposal to fix it and a test to back it up which fails without the fix.
Checklist

[x] This PR targets the main branch.
[x] The commit message is written in past tense, mentions the ticket number, and ends with a period.
[x] I have checked the "Has patch" ticket flag in the Trac system.
[x] I have added or updated relevant tests.
[x] I have added or updated relevant docs, including release notes if applicable.
[x] I have attached screenshots in both light and dark modes for any UI changes.

Thanks @avilaton for starting this work. Could you please review the PR template and double check that the items that you checked are truly done? For example:

The commit message is written in past tense, mentions the ticket number, and ends with a period.

None of this holds at this time :smiley:

Thanks @avilaton for starting this work. Could you please review the PR template and double check that the items that you checked are truly done? For example:

The commit message is written in past tense, mentions the ticket number, and ends with a period.

None of this holds at this time 😃

Ok I think we are getting somewhere now, any other suggestions for us?

Thanks @avilaton for starting this work. Could you please review the PR template and double check that the items that you checked are truly done? For example:

The commit message is written in past tense, mentions the ticket number, and ends with a period.

None of this holds at this time 😃

Ok I think we are getting somewhere now, any other suggestions for us @nessita ?

Thanks for the continued work on this PR! :star2:
The branch is now listed in the "PRs needing review" in the Django Dashboard, which is sufficient to get a full review soon. One thing that we'll request is a release note added to the docs/releases/5.1.2.txt file since this will be released along with other bugfixes in about a month. Could you give that a try?
Once that's done, please unset the flag "needs documentation" in the Trac Ticket.

LGTM!
FWIW I tried to see if this was a generalized problem that manifested itself on SQLite as well which also make use of || but looks [it wasn't affected][https://www.sqlite.org/lang_expr.html#operators_and_parse_affecting_attributes). Knowing that I think adding a test solely for Postgres Trigram like its the case here makes the most sense.


defaultProps does not exist on Text

I've upgraded to the latest version of NativeBase. I was using Text.defaultProps.allowFontScaling = false to prevent font scaling. I now get a compilation error:  Property 'defaultProps' does not exist on type 'typeof Text'.
react-native, react and native-base version
react-native: 0.55
react: 16.3.1
native-base: 2.5.2
Expected behaviour
Should be able to set the defaultProps.allowFontScaling property.
Actual behaviour
'defaultProps' cannot be accessed on Text class.
Steps to reproduce (code snippet or screenshot)
 constructor(props: any) {             super(props);             Text.defaultProps.allowFontScaling = false;   }

For those who need to get compilation working while globally setting Text.defaultProps, you can do something ugly like this:
interface TextWithDefaultProps extends Text {
  defaultProps?: { allowFontScaling?: boolean };
}

const App = () => {
  ((Text as unknown) as TextWithDefaultProps).defaultProps =
    ((Text as unknown) as TextWithDefaultProps).defaultProps || {};
  ((Text as unknown) as TextWithDefaultProps).defaultProps!.allowFontScaling = false;
// etc.

Hi, set allowFontScaling to false ignoring system font styles but boldfont is not ignoring
how to ignore boldfont from system setting
please help
Thank You


Wrong commands given for AD vs AAD

The commands for set/clear mobile numbers are incorrect. They are referencing the wrong location.
The section that says Microsoft Entra ID are referencing AD and the section that says On-Premises AD are using AAD in the command.
Set MobilePhone and AlternateMobilePhones properties in Microsoft Entra ID:
Set-ADSyncToolsDirSyncOverridesUser<EMAIL_ADDRESS>-MobileInAD '999888777'  -OtherMobileInAD '0987654','1234567'
Set Mobile and otherMobile properties in on-premises Active Directory:
Set-ADSyncToolsDirSyncOverridesUser<EMAIL_ADDRESS>-MobilePhoneInAAD '999888777' -AlternateMobilePhonesInAAD '0987654','1234567'
Clear MobilePhone and AlternateMobilePhones properties in Microsoft Entra ID:
Clear-ADSyncToolsDirSyncOverridesUser<EMAIL_ADDRESS>-MobileInAD-OtherMobileInAD
Clear Mobile and otherMobile properties in on-premises Active Directory:
Clear-ADSyncToolsDirSyncOverridesUser<EMAIL_ADDRESS>-MobilePhoneInAAD -AlternateMobilePhonesInAAD
The commands are backwards.

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

ID: a6cb547b-5c9f-4a5e-87c6-6705a17bb5a2
Version Independent ID: edd3b439-9f25-79a6-6621-d453783828eb
Content: How to use the BypassDirSyncOverrides feature of a Microsoft Entra tenant - Microsoft Entra
Content Source: articles/active-directory/hybrid/connect/how-to-bypassdirsyncoverrides.md
Service: active-directory
Sub-service: hybrid
GitHub Login: @billmath
Microsoft Alias: billmath

@jamestockwell
Thanks for your feedback! We will investigate and update as appropriate.

@jamestockwell
Thanks for reporting this! We have created a PR for this issue and the changes should go live soon.


Issue with playground code editor on bhailang.js.org

Description

Cursor position not proper wrt the word being typed.
Syntax highlighting makes other bigger smaller, i.e until a particular word is not keyword it's font size is incosistent with the font size of the keyword.

Requirements
Need the repository for the website.
Domain
Other
Type of Contribution
Enhancement
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of bhai-lang.

bht mast kaam krta hai bhaii

Resolved. It was due to different font family in editor. Closing this.


Ingestion: pip's dependency conflict

Affected module
Does it impact the UI, backend or Ingestion Framework?
Ingestion
  #42 166.1 ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
  #42 166.1 apache-airflow-providers-google 10.2.0 requires google-cloud-datacatalog>=3.11.1, but you have google-cloud-datacatalog 3.6.2 which is incompatible.

in addition to #12491, we received multiple conflicts:
  2023-07-18T18:15:50.5280479Z #44 34.79 dask 2023.6.1 requires importlib-metadata>=4.13.0, but you have importlib-metadata 4.12.0 which is incompatible.
  2023-07-18T18:15:50.5282507Z #44 34.79 gcloud-aio-auth 4.2.3 requires setuptools<67.0.0,>=66.0.0, but you have setuptools 65.6.3 which is incompatible.
  2023-07-18T18:15:50.5283242Z #44 34.79 gcloud-aio-bigquery 6.3.0 requires setuptools<67.0.0,>=66.0.0, but you have setuptools 65.6.3 which is incompatible.
  2023-07-18T18:15:50.5283962Z #44 34.79 gcloud-aio-storage 8.2.0 requires setuptools<67.0.0,>=66.0.0, but you have setuptools 65.6.3 which is incompatible.
  2023-07-18T18:15:50.5284706Z #44 34.79 google-cloud-spanner 3.36.0 requires sqlparse>=0.4.4, but you have sqlparse 0.4.3 which is incompatible.

@ayush-shah are we handling it in 1.1.1?

#44 37.19 ERROR: pip's dependency resolver does not currently take into account all the packages that are installed. This behaviour is the source of the following dependency conflicts.
#44 37.19 google-cloud-spanner 3.36.0 requires sqlparse>=0.4.4, but you have sqlparse 0.4.3 which is incompatible.

The above error remains after addressing other dependency conflicts, where openmetadata-sqllineage needs to upgrade sqlparse to 0.4.4


[BUG] TH was successfully paired to the DUT with wrong Discriminator

Reproduction steps
When running test MCORE_FS_1_4, we start two Matter devices (Lighting Example and Fabric-Bridge) using the same passcode, and attempt to distinguish them using the discriminator.
According to the logs, it appears that the TH has successfully paired with the Fabric-Bridge despite the discriminator not matching.
It looks something wrong with mDNSResolver, the Lighting server is broadcasting with correct Discriminator
[SERVER] [1725495299.702053][53195:53195] CHIP:SPT: LinuxCommissionableDataProvider didn't get a PASE salt, generating one.
[SERVER] [1725495299.703226][53195:53195] CHIP:DL: Device Configuration:
[SERVER] [1725495299.703233][53195:53195] CHIP:DL:   Serial Number: TEST_SN
[SERVER] [1725495299.703237][53195:53195] CHIP:DL:   Vendor Id: 65521 (0xFFF1)
[SERVER] [1725495299.703242][53195:53195] CHIP:DL:   Product Id: 32769 (0x8001)
[SERVER] [1725495299.703243][53195:53195] CHIP:DL:   Product Name: TEST_PRODUCT
[SERVER] [1725495299.703247][53195:53195] CHIP:DL:   Hardware Version: 0
[SERVER] [1725495299.703249][53195:53195] CHIP:DL:   Setup Pin Code (0 for UNKNOWN/ERROR): 20202021
[SERVER] [1725495299.703251][53195:53195] CHIP:DL:   Setup Discriminator (0xFFFF for UNKNOWN/ERROR): 1596 (0x63C)
[SERVER] [1725495299.703255][53195:53195] CHIP:DL:   Manufacturing Date: (not set)
[SERVER] [1725495299.703256][53195:53195] CHIP:DL:   Device Type: 257 (0x101)

And TH has correctly configured the filter to mDNSResolver
    await self.default_controller.CommissionOnNetwork(
        nodeId=th_server_th_node_id,
        setupPinCode=self.th_server_passcode,
        filterType=ChipDeviceCtrl.DiscoveryFilterType.LONG_DISCRIMINATOR,
        filter=self.th_server_discriminator,
    )

[MatterTest] 09-04 17:14:59.695 INFO: test_TC_MCORE_FS_1_4: 1596
[FS-BRIDGE][35m[1mINF[0m  [1725495299.697907][53141:53141] CHIP:EM: >>> [E:1898r S:45918 M:78020313 (Ack:215262786)] (S) Msg RX from 1:000000000001B669 [7875
[FS-BRIDGE][96m[1mDBG[0m  [1725495299.697950][53141:53141] CHIP:EM: Found matching exchange: 1898r, Delegate: (nil)
[FS-BRIDGE][96m[1mDBG[0m  [1725495299.697977][53141:53141] CHIP:EM: Rxd Ack; Removing MessageCounter:215262786 from Retrans Table on exchange 1898r
[SERVER] [1725495299.698424][53195:53195] CHIP:DL: ChipLinuxStorage::Init: Using KVS config file: /tmp/TC_MCORE_FS_1_4v3rw03o2/kvs-app-b9v080wp
[MatterTest] 09-04 17:14:59.698 INFO :pychip_DeviceController_OnNetworkCommission
[MatterTest] 09-04 17:14:59.698 INFO :filter.code:1596
[SERVER] [1725495299.698653][53195:53195] CHIP:DL: ChipLinuxStorage::Init: Using KVS config file: /tmp/chip_kvs
[SERVER] [1725495299.698658][53195:53195] CHIP:DL: ChipLinuxStorage::Init: Attempt to re-initialize with KVS config file: /tmp/chip_kvs
[MatterTest] 09-04 17:14:59.698 INFO :DiscoverCommissionableNodes:filter.code:1596

Bug prevalence
Whenever
GitHub hash of the SDK that was being used
7fdfc88383676eeecd022e047ae1c5a8ce1b9b47
Platform
raspi
Platform Version(s)
1.3
Anything else?
No response

Uploading pair_to_dut_wrong_discriminator.txt…

pair_to_dut_wrong_discriminator.txt

@yufengwangca Which dns-sd implementation is TH using here?  Minimal or platform?


Expose render_inline parameter to the users

Exposing render_inline gives the user a flexibility whether to have the css style separated or inlined in the html. The default will still be True when exporting, and False when grip is used as a service.

Pull request #165 has been created.

Just published v4.2.0 which contains this.
CLI was renamed option to --no-inline. After playing with it, --inline=True when using the grip server didn't make as much sense. So --no-inline is the new name and it only works with --export.
Thanks for the idea and for diving in, @mrexmelle!


Fix doc typo for Number#divmod

As per https://github.com/crystal-lang/crystal/pull/3426#issuecomment-254293080.

Thank you!!


Avoid double-application of local-time conversion.

If I understand correctly, this should fix https://github.com/fmeringdal/rust_rrule/issues/15

Codecov Report

Merging #16 (972fdc7) into main (8b415c0) will decrease coverage by 0.02%.
The diff coverage is 75.00%.

@@            Coverage Diff             @@
##             main      #16      +/-   ##
==========================================
- Coverage   93.45%   93.42%   -0.03%
==========================================
  Files          18       18
  Lines        7223     7251      +28
  Branches      386      381       -5
==========================================
+ Hits         6750     6774      +24
- Misses        216      218       +2
- Partials      257      259       +2

Impacted Files
Coverage Δ

tests/rrule.rs
99.78% <73.33%> (-0.16%)
:arrow_down:

src/rrule_iter.rs
78.43% <100.00%> (+0.21%)
:arrow_up:

src/iter/poslist.rs
82.14% <0.00%> (-3.58%)
:arrow_down:

src/rrulestr.rs
58.98% <0.00%> (-0.21%)
:arrow_down:

src/iter/iterinfo.rs
87.50% <0.00%> (+1.34%)
:arrow_up:

src/iter/mod.rs
89.17% <0.00%> (+2.46%)
: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 8b415c0...972fdc7. Read the comment docs.

This looks great, thank you very much!
I have released a new version 0.5.6, with this fix.


webhook: add --namespace option to deployment script

closes #78

Codecov Report

Merging #83 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master      #83   +/-   ##
=======================================
  Coverage   77.24%   77.24%
=======================================
  Files          10       10
  Lines        1090     1090
=======================================
  Hits          842      842
  Misses        230      230
  Partials       18       18

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 1f236d5...5de8d18. Read the comment docs.


Allow defining custom functions from outside the less files

Hi,
This is related to https://github.com/less/less.js/issues/2747
I'd like to be able to add custom functions for usage in mixin guards.
The thing is that I'm not really interested declaring these fonctions in Less syntax but just register some custom useful functions in my build process.
Is it possible to do?

Yes, you just write your own plugin. (There're actually two slightly different plugin formats currently - 1, 2).

Closing as out of the plugin scope.


Why is HttpHelper marked "Internal" (in Microsoft.Bot.Builder.Integration.AspNet.Core)

I've had to create my own BotFrameworkAdapter-derived class, but in so doing I need to call HttpHelper.WriteResponse. This is annoying at the moment because the class is marked "Internal", so I've had to create my own copy of it - is there some reason it's not a Public class?

We just made this internal because it was just part of the implementation of the BotFrameworkHttpAdapter. I don't see a reason why we couldn't make this public if it was useful. We were just being a little conservative in our thinking.
However, there is always cost - we generally try to have unit test around the public service area. Obviously its tested to some significant degree because its part of the current implementation. So we could make this public in R7.

Ok great news. I'm super heads down at the moment, but if I get a chance soon I'd be most happy to help on it. If it does become "public", maybe it should change from being a "helper" to something more concrete. Possibly extension methods on HttpRequest & Response, or similar

@HiltonGiesenow We certainly welcome pull requests!

this will be public in the next (minor) release - due this month

This is in master now. It will be in the latest package to be published next week.

I really appreciate your responding to this and sorting it out quickly, but to be honest I've been wondering whether this "helper" itself really should be public... While the methods themselves are useless, I'm wondering if we could put them somewhere more specific than just a general "helper". What do you think? As a possible suggesting, as it's only two main methods (one one static field), perhaps we could:

put the static BotMessageSerializer field into a static class, like a [namespace].Serialization static class
make the remaining methods into Extenstion methods: ReadRequest, which takes in just an HttpRequest, could be made into ReadAsActvivity, for example, and WriteResponse could be WriteInvokeResponse

What do you think? I'm happy to work on this as a PR, either the design proposed or an alternative.


Build error ubuntu 16.04

CXX      script/libbitcoinconsensus_la-bitcoinconsensus.lo
CXX      libbitcoinconsensus_la-uint256.lo
CXX      libbitcoinconsensus_la-utilstrencodings.lo
CXXLD    libbitcoinconsensus.la
/usr/bin/ar: u' modifier ignored since D' is the default (see `U')
make[2]: *** No rule to make target 'DudgXd.cpp', needed by 'DudgXd-DudgXd.o'.  Stop.
make[2]: Leaving directory '/home/wallets/DudgX-1.0/src'
Makefile:7083: recipe for target 'all-recursive' failed
make[1]: *** [all-recursive] Error 1
make[1]: Leaving directory '/home/wallets/DudgX-1.0/src'
Makefile:595: recipe for target 'all-recursive' failed
make: *** [all-recursive] Error 1

Obviously someone doesn't realize that Linux unlike windows is case sensitive.. For now, just rename src/dudgxd.cpp to src/DudgXd.cpp   and run 'make' again..
Dev: this needs to be fixed.


go1.8: merge CL 36417 to Go 1.8 branch

We need to merge CL 36417 to the Go 1.8 branch for #18968.
CC @crawshaw @bradfitz @broady

(ab067cde34b515172cf51b4c562b2b4ef3cbe587)

Since I didn't want to open a new issue for this one
CL 36470 and commit https://github.com/golang/go/commit/3b68a647696ebfb61d199155f5f1faa5740e5c55
Perhaps we can repurpose this bug to track CLs that we need to merge for Go1.8?

/cc @griesemer who fixed it.

I think it's fine to open a new issue for anything that needs merging for 1.8. There shouldn't be many (if any at all).
Closing this (tracked in #18915).

@broady Did you mean to close this issue?  This one was about CL 36417.  Reopening.

@ianlancetaylor Nope, I'm just confused. @odeke-em can you open a separate issue for CL 36470?

https://go-review.googlesource.com/36421  for this issue.

https://go-review.googlesource.com/36422 for CL 36470.

Thanks @broady @rsc @griesemer, no need for me to file the issue,
it's already been handled as @rsc mentioned.


Credentials setup documentation is out-of-date

It seems in v1.23 the method for defining credentials has changed. Please update the documentation regarding setting up credentials for GitHub access.

I will try to get to this today.

I am open to changes to how the readme is done if you want to make a pull request, and I also updated the wiki page.

Fantastic, thanks a lot for the fast turn-around! I'll take a look and see if I have any ideas for improvement.


Inject otel traces to grpc client.

This uses the global trace provider.
If a trace provider is not set, a no-op provider is used.

Interesting errors in CI...

If you rebase on main HEAD you should be able to get a clean CI run now

Have we seen this on Windows on other PR's?

race: limit on 8128 simultaneously alive goroutines is exceeded, dying

I don't think I've ever seen that error

Could there be something in the OpenTelemetry stack which is actually going haywire and creating goroutines? Seems weird that this PR would continue to get that Windows error.

Would be nice to get even a partial stack dump when this error is hit.

I'm tempted to close this one because the traces can be enabled through WithDialOpts.

Added a stack dump to the tests, hopefully it'll dump and show us what's going on with those Windows tests.

This one?
race: limit on 8128 simultaneously alive goroutines is exceeded, dying
exit status 66

https://github.com/golang/go/search?q="simultaneously+alive+goroutines+is+exceeded"&type=issues

Needs rebase, then can see if tests pass


Qt UI on NEOS

For reference, I've run into Error 00000502 after convex fill` like in #2120 .

For reference, I've run into Error 00000502 after convex fill` like in #2120 .


Investigate a new library to substitute Slickgrid

At this moment Slickgrid has many issues:

jquery under the hood, the core of Slickgrid is quite old and hardly developed
tree view representation always sorts fields despite flags
Selection in tree view doesn't implemented
Has issues with re-render data: autosize columns by data value doesn't work correctly (width of grid is always resets to minimum after changing container size); grid size doesn't follow the container resize or has different issue with container resize; etc.
Has it's own copy of data
Hardly adjusted theme
...

Possible libraries:

Fluent UI Tree and DataGrid
https://jspreadsheets.com/

Hello I'm the maintainer of Slickgrid-React and Slickgrid-Universal and I just saw this and thought of commenting on some of these.

jquery under the hood, the core of Slickgrid is quite old and hardly developed

not true, I dropped jQuery for native code in v3.x and you're using v5.x, so jQuery is no longer required (if you have it installed, you surely don't need it for the grid)

tree view representation always sorts fields despite flags

SlickGrid needs an initial sort to be build the Tree from flat to hierarchical, but after that it should be able to sort by any other field. You can log an issue with a repro if that's not the case.

Selection in tree view doesn't implemented

I haven't tried that one, neither provided a demo, but it should work. Open an issue if not

Has issues with re-render data: autosize columns by data value doesn't work correctly (width of grid is always resets to minimum after changing container size); grid size doesn't follow the container resize or has different issue with container resize; etc.

It might be dependent on the data provided, I would need a repro to investigate. Side note, double-clicking on a the column resize (the handle between 2 columns) will resize the current column by its content

Has it's own copy of data

It has an extra copy of data for the hierarchical data (Tree) but it shouldn't duplicate the flat dataset. There are grid options to deep copy the data but it shouldn't be enabled by default (if it is then file an issue). Side note, to avoid deep copy of data and for perf reasons, I mutate the arrays directly (i.e.: by adding props like __treeLevel, ...)

Hardly adjusted theme

I provided ton of SASS/CSS variables and also provide Dark Theme, so this seems hardly true. Reach out if you need help customizing the theme.
I'm open to suggestions/contributions on my end but if you file any issues then make sure to provide repro, otherwise it's hard for me to provide any help. I'm open to feedback, however do know that I'm mainly a single developer on this open source project, so keep that in mind even if I'm very active on the project. Another side note is that I'm not a React developer, questions related to that might be hard for me to answer even if I support the Slickgrid-React project. I'm currently supporting 3 different frameworks and I'm in the work of adding Vue to the mix.
So far all the issues that @tnaum-ms reported were fixed. It would be nice if you keep SlickGrid in the project but if not that's ok too, I won't lose sleep over this 😉 Just go with what you think best fit your project. Cheers


downloadAttachmentContent works only with UserAuth and no with ApiKeyAuth

Hello,
When we use the function
Issue previousIssue = mgr.getIssueManager().getIssueById(Integer.parseInt(id), Include.attachments);
for (Attachment attachment : attachments) {
Attachment finalAttachment = downloadAttachmentContent(attachment);
}

It doesn't works because we need an authentification with login and Password in the java File :
RedmineAuthenticator.java

Linked to https://github.com/taskadapter/redmine-java-api/issues/121

I assume this is a duplicate of issue #121


No log for connection issues? Cannot do anything, but no log entries either.

I seem to have a similar issue to #16 , but for different reasons. I think the issue is that Git cannot connect, but I don't know - there's no popups, no login requests, nothing.
Not even a log file. I wasn't sure if I looked in the right place until I enabled Tracing, which gave me a log file with a single line, namely that tracing is now enabled.
Is this normal? If so, I would suggest adding something to the log, even if it's just "new log file created", so users know they're looking in the right place and there is no log file (yet).

I swear exceptions should be getting logged and I'm baffled why they aren't. Could you check in the editor log? Maybe they're ending up in there for some reason.

For some reason (best guess: the universe likes to troll people), there are logs now. Maybe because I closed unity/hub and started everything anew?
The log doesn't say much to me, however.
I'm not entirely sure if I'm using the correct name in the settings - is it the display name, or the original account name? I'm sure I tried both.
Could the issue be because the repository does not belong to me, but is shared with me? (It's hidden)
Christoph

Von: Andreia Gaita @.>
Gesendet: Donnerstag, 9. Februar 2023 18:12
An: spoiledcat/git-for-unity @.>
Cc: Christoph Pichlmann @.>; Author @.>
Betreff: Re: [spoiledcat/git-for-unity] No log for connection issues? Cannot do anything, but no log entries either. (Issue #17)
I swear exceptions should be getting logged and I'm baffled why they aren't. Could you check in the editor log? Maybe they're ending up in there for some reason.
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fspoiledcat%2Fgit-for-unity%2Fissues%2F17%23issuecomment-1424534252&data=05|01|pichlmann%40fredlarts.at|f6dc5f542b844e080a5108db0ac0d2e1|9099c869e91a4c7696f9953e2cf0dc7b|0|0|638115595588927072|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=dWTPBUPrEtBcdlzev5Cq26ry9qfWBuU%2Fvu1W6ofRHP8%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABBHCQKJKMGVP2K33LITONTWWUQPZANCNFSM6AAAAAAUVZ77CA&data=05|01|pichlmann%40fredlarts.at|f6dc5f542b844e080a5108db0ac0d2e1|9099c869e91a4c7696f9953e2cf0dc7b|0|0|638115595588927072|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=8nCTeJ4sEVIG6%2BQa4EYQVQNQbeZo5vBn6CqRKOL7%2FIs%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>
Public

Are you referring to the name/email fields in the settings? Those are there because some git operations require a name/email to be configured - they can be anything, and are not related to authentication.
This version of Git for Unity doesn't authenticate you to whatever git provider you're using (GitHub/GitLab/etc) - it relies on authentication being already set up before. If the repository that you're using is private, then you need to make sure you can actually pull/push to it. Since you have the project open, I assume you've cloned the repository in some way outside of Git for Unity? To doublecheck that Git in general is working, open a command line (outside of Unity), go to where your project is, and run git fetch to see if git works and is successfully authenticated - this might fix whatever authentication issue you might be having.
I'm working on fixing the logs to actually log whatever the heck is going wrong with these "can't do anything" issues, I'll update as soon as I have something.

I see - that explains a lot, thank you!
Also thank you for creating this - even if it doesn't work (for me) right now, I can see that it's very useful. It works with my friend, after all. So currently I'm using Git for Desktop to upload/get the files. Good enough for now 🙂
Christoph

Von: Andreia Gaita @.>
Gesendet: Freitag, 10. Februar 2023 11:44
An: spoiledcat/git-for-unity @.>
Cc: Christoph Pichlmann @.>; Author @.>
Betreff: Re: [spoiledcat/git-for-unity] No log for connection issues? Cannot do anything, but no log entries either. (Issue #17)
Are you referring to the name/email fields in the settings? Those are there because some git operations require a name/email to be configured - they can be anything, and are not related to authentication.
This version of Git for Unity doesn't authenticate you to whatever git provider you're using (GitHub/GitLab/etc) - it relies on authentication being already set up before. If the repository that you're using is private, then you need to make sure you can actually pull/push to it. Since you have the project open, I assume you've cloned the repository in some way outside of Git for Unity? To doublecheck that Git in general is working, open a command line (outside of Unity), go to where your project is, and run git fetch to see if git works and is successfully authenticated - this might fix whatever authentication issue you might be having.
I'm working on fixing the logs to actually log whatever the heck is going wrong with these "can't do anything" issues, I'll update as soon as I have something.
—
Reply to this email directly, view it on GitHubhttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fspoiledcat%2Fgit-for-unity%2Fissues%2F17%23issuecomment-1425611338&data=05|01|pichlmann%40fredlarts.at|c46caf0071d3424c8eea08db0b53cd94|9099c869e91a4c7696f9953e2cf0dc7b|0|0|638116226819843234|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=6WP2HtKes7W6A3vwiq6zH9GzXws0H43NTaHwGTUb4S8%3D&reserved=0, or unsubscribehttps://nam12.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FABBHCQLIVOJJTVONTF274VTWWYLZHANCNFSM6AAAAAAUVZ77CA&data=05|01|pichlmann%40fredlarts.at|c46caf0071d3424c8eea08db0b53cd94|9099c869e91a4c7696f9953e2cf0dc7b|0|0|638116226819843234|Unknown|TWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D|3000|||&sdata=m90gUc7rK%2BDZTqirRRvio9hRXQh665kKW71sl5DQmmI%3D&reserved=0.
You are receiving this because you authored the thread.Message ID: @.***>
Public


Upgrade from 5.0 to 5.1 leads into VerifyCsrfToken error

Hey guys,
I followed up procedure for upgrade to 5.1.0 version and after that I run into this error
FatalErrorException in VerifyCsrfToken.php line 33: Class 'Symfony\Component\Security\Core\Util\StringUtils' not found
any hint what I'm missing here?

Please upgrade to the latest 5.1.x rather than 5.1.0.

VerifyCsrfToken.php

Is that in laravel/laravel. If so, you need to update it to use the laravel 5.1 version.

@GrahamCampbell yeah my version is 5.1.26

Did you delete the compiled.php file?

I do not have compiled.php at my code base.

This doesn't look like a bug in our code. Looks more like your code is relying on a dependency that we no longer have. As I said, you need to fix your middleware. Do a search of your code for equals(.

Any other questions, please ask on the forums.

gotcha, seems like composer didn't update laravel code base on proper way, I see abstract class BaseEncrypter but that one doesn't exist anymore, etc..


master

I signed it!

I signed it!
On Sep 2, 2015 10:31 AM, "googlebot"<EMAIL_ADDRESS>wrote:

Thanks for your pull request. It looks like this may be your first
contribution to a Google open source project, in which case you'll need to
sign a Contributor License Agreement (CLA).
[image: :memo:] Please visit https://cla.developers.google.com/
https://cla.developers.google.com/ to sign.
Once you've signed, please reply here (e.g. I signed it!) and we'll
verify. Thanks.

If you've already signed a CLA, it's possible we don't have your
GitHub username or you're using a different email address. Check your
existing CLA data https://cla.developers.google.com/clas and verify
that your email is set on your git commits
https://help.github.com/articles/setting-your-email-in-git/.
If you signed the CLA as a corporation, please let us know the
company's name.

—
Reply to this email directly or view it on GitHub
https://github.com/googlesamples/android-RecyclerView/pull/12#issuecomment-137103394
.

Hi Igor. Thanks for the contribution, but we've got a script that centrally manages Gradle versions across all of our samples. (As you can see, we've already updated the Gradle version for this sample.)


fix npm publish .DS_Store

What kind of change does this PR introduce? (check at least one)

[x ] Bugfix
Does this PR introduce a breaking change? (check one)
[ ] Yes
[x ] No

prevent .DS_Store from being publish
see<EMAIL_ADDRESS>Close #10112

https://github.com/vuejs/vue/issues/10112

@posva I think someone should make a new PR, I deleted this forked repo so seems I can't edit this PR anymore

@posva I think someone should make a new PR, I deleted this forked repo so seems I can't edit this PR anymore

Thanks for the initiative though!

Thanks for the initiative though!


ReferenceError: Editor is not defined with CDN

Describe the bug
I followed the tutorial at https://github.com/nhn/tui.editor/blob/master/docs/en/getting-started.md using the CDN for both the JS and CSS libraries. When I opened the HTML file, I was greeted with a blank page, and an error in the browser console saying Uncaught ReferenceError: Editor is not defined
To Reproduce
Steps to reproduce the behavior:

Follow the tutorial at https://github.com/nhn/tui.editor/blob/master/docs/en/getting-started.md using the CDN options
Open the resulting HTML page:

<!DOCTYPE html>

    <meta charset="UTF-8">
    <link rel="stylesheet" href="https://uicdn.toast.com/editor/latest/toastui-editor.min.css" />
    <title>ToastUI Test</title>

I also tried placing the  line in the head tag, and at the beginning of the body area. Placing the editor definition in an onload event did not solve the problem either.
Expected behavior
The editor should have appeared on my page and I should have been able to use the editor. Instead, I got an error in the browser console saying Uncaught ReferenceError: Editor is not defined. The message states the error occurs on line 15 (const editor = new Editor({)
Desktop (please complete the following information):

OS: Windows 10
Browser: Firefox, Google Chrome and Microsoft Edge
Version: Latest version for all three browsers

Additional context
As far as I can tell, I have followed the tutorial exactly. Due to project constraints the only way I can access the library is through the CDN

After some more digging, I came across https://nhn.github.io/tui.editor/latest/tutorial-example01-editor-basic and checked the JavaScript tab. It shows to change the line:
const editor = new Editor({

to:
const editor = new toastui.Editor({

Which solved the problem. It would be very helpful if the documentation reflected this.


Reinitializing with enableReinitialize doesn't update values in nested FieldArrays

Are you submitting a bug report or a feature request?

Bug report
What is the current behavior?

In my form I am passing the initialvalues through the below code.
function mapStateToProps(state) {
  return {
    initialValues: {
      POItems: state.poItems.poItems
    }
  }
}

AddCustomerPOItems = reduxForm({
  form: 'AddCustomerPOItemsForm',
  enableReinitialize: true,
})(AddCustomerPOItems)

AddCustomerPOItems = connect(mapStateToProps, poItems)(AddCustomerPOItems)

export default AddCustomerPOItems

This issue is in nested field array.
When a new item is added reducers will update the state and initilize the fields and works correctly, but if i update a field ,the reducers work fine and update the state correctly but the values not going to initialize.
In first table its a fieldarray and other table (Yellow one) is a nested fieldarray of first table

What is the expected behavior?
Intilize the form when updated happen
Sandbox Link

What's your environment?

Redux form 7.2.0
Other information

Is there any solution to this issue?

I am facing the same problem. Version 8.1.0


Update Vietnamese

Update Vietnamese

Thanks @TanNhatCMS for the Vietnamese translation.
@tabacitu Please merge🎉

Thanks for the PR @TanNhatCMS
It will be available in a few moments in 6.2.1 🙏
Cheers


Add Description Userrole view

Codecov Report

Merging #1489 (ad3bd22) into main (10c54ca) will increase coverage by 19.60%.
The diff coverage is 0.00%.

@@             Coverage Diff             @@
##             main    #1489       +/-   ##
===========================================
+ Coverage   31.80%   51.40%   +19.60%
===========================================
  Files          54       69       +15
  Lines         827     1208      +381
  Branches       47        0       -47
===========================================
+ Hits          263      621      +358
- Misses        554      572       +18
- Partials       10       15        +5

Impacted Files
Coverage Δ

...ntrollers/MarketParticipant/Dto/UserRoleViewDto.cs
0.00% <0.00%> (ø)

...ntrollers/MarketParticipantPermissionController.cs
0.00% <0.00%> (ø)

...Controllers/MarketParticipantUserRoleController.cs
0.00% <0.00%> (ø)

...taHub.WebApi/Controllers/Wholesale/Dto/BatchDto.cs
0.00% <0.00%> (-100.00%)
:arrow_down:

...trollers/MarketParticipant/Dto/FilteredActorDto.cs
16.66% <0.00%> (-83.34%)
:arrow_down:

...gistration/HealthEndpointRegistrationExtensions.cs
45.45% <0.00%> (-36.37%)
:arrow_down:

...WebApi/RequireNonNullablePropertiesSchemaFilter.cs
80.00% <0.00%> (-8.00%)
:arrow_down:

apps/dh/api-dh/source/DataHub.WebApi/Startup.cs
91.66% <0.00%> (-0.93%)
:arrow_down:

...ntegration/Controllers/WholesaleControllerTests.cs
97.75% <0.00%> (ø)

...taHub.WebApi.Tests/Integration/HealthCheckTests.cs
100.00% <0.00%> (ø)

... and 13 more

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

You need to add description to mock data as well

:-)

You need to add description to mock data as well

:-)

ohh, i'll get them added


allow log4j2.xml baseDir to be overridden by system property

This specifies baseDir in log4j2.xml to use a system property (-DbaseDir=/tmp) but if it isn't set then it will fall back to the log42.xml baseDir property that it is using now. This allows for overriding the default config without changing it or using the overlay/initializr config (which may also need this change, if this is accepted).

We might want to change the property name to cas.log.baseDir or cas.log.basedir or something alone those lines. Since this file is inside war and people can't override it from the command line now, we could change it and it shouldn't break anything. If people are using an existing overlay then they already have their own log4j2.xml and have probably changed baseDir. I am interested in overriding it for when running cas in cicd pipeline without modifying the overlay from the initializr.

@apereocas-bot runci

This looks good to me. Thanks much!


Blui 3907 GA4 update

Fixes BLUI-3907 .

Changes proposed in this Pull Request:

Remove GA.js and dev/master id's
Remove analytics from firebase deploy on PR
Set Ci config to export measurement ID

Screenshots / Screen Recording (if applicable)

To Test:

Any specific feedback you are looking for?

Is there any way to test this?

Nothing to test for this PR until it is deployed


mapN is gone

Hi @alexandru
When trying to upgrade I noticed mapN was gone, and I was curious what the reasoning behind this was?
Thanks,
Olly

@OliverJAsh which class they're missing from? Those are still present at least for IO and Future (v7.0.1), dunno about others, didn't use them

https://github.com/funfix/funfix/blob/master/packages/funfix-exec/src/future.ts#L980
https://github.com/funfix/funfix/blob/master/packages/funfix-effect/src/eval.ts#L371

It's probably the Option (v6.2.2: https://github.com/funfix/funfix/blob/v6.2.2/packages/funfix-core/src/disjunctions.ts#L652)

I removed mapN because it's non-standard and it was less useful than I thought it is.
Option also has filter on it, so you can do ... option.filter(_ != null) and I find this one more intuitive.
If you have a good use case for it, I'm willing to add it back.

Actually the use-case would be the one in the sample:
Option.of(user)
  .mapN(_ => _.contacts)
  .mapN(_ => _.length)

But in this case I kind of used Try instead, so Try.of(() => user.contacts.length).
So you know, if you liked mapN, copy/paste it from 6.2.2 and make a PR to re-add it.


Fixed invalid super-call in test code.

This mistake was discovered using mypy in the django-stubs project.

Thanks, this method was unused and we remove it in 6461583b6cc257d25880ef9a9fd7e2125ac53ce1.


Unhandled Error:The type initializer for 'FHIRBulkImport.FHIRUtils' threw an exception.Issue

FHIR Loader tool not able to process files:-

After we upload NDJSON files in bundles folder, we can see them getting split successfully however after couple of minutes they are getting moved into bundleserr folder with the following error message in the response files

Unhandled Error:The type initializer for 'FHIRBulkImport.FHIRUtils' threw an exception.
at FHIRBulkImport.FHIRUtils.CallFHIRServer(String path, String body, HttpMethod method, ILogger log, String customAudience) in C:\home\site\repository\FHIRBulkImport\FHIRUtils.cs:line 62
at FHIRBulkImport.ImportUtils.ImportBundle(String name, ILogger log, TelemetryClient telemetryClient) in C:\home\site\repository\FHIRBulkImport\ImportUtils.cs:line 101
Frankly since this tool is not supported by Azure it being open source, it took couple of weeks for us to get to this point earlier we used to get all kinds of key vault security error and when we managed to resolve all those we are hitting this show stopper.
Please provide help as we are working on a client project that  requires this feature.

@NeerajSAS Thanks for reporting this issue, we will take a look and let you know!

HI @NeerajSAS  - we were not able to repro this issue. Can you make sure that you are uploading the NDJSON file in the NDJSON container, rather than directly in the bundle container? Let us know if that works; if not, we may need to get more detailed steps to try and repro the issue.


How do I set up PhantomJS to work with uncss?

Hey,
I just started checking out uncss today and I was wondering how I can set up PhantomJS on my machine to work with it? I have installed uncss correctly and it runs but I get this error:
Fatal error: PhantomJS: Cannot open about:blank
I don’t think I ever had PhantomJS installed on my machine so I installed it with npm:
npm install -g phantomjs
All good up to that point but then I try running grunt uncss again and I get the same error… so I’m not really sure what I’m supposed to do or how these two tools are supposed to interact with each other. Here is my Gruntfile.js:
module.exports = function(grunt) {

    // Project configuration.
    grunt.initConfig({
        pkg: grunt.file.readJSON('package.json'),

        uncss: {
            dist: {
                files: {
                    'style.css': ['*.php']
                }
            }
        },

        sass: {
            dist: {
                options: {
                    style: 'compressed'
                },
                files: {
                    'style.css': 'scss/style.scss'
                }
            }
        },

        watch: {
            sass: {
                files: ['scss/**/*.scss'],
                tasks: ['sass'],
            },
            livereload: {
              options: { livereload: true },
              files: ['scss/**/*.scss'],
            },
        },

    });

    grunt.loadNpmTasks('grunt-uncss');
    grunt.loadNpmTasks('grunt-contrib-sass');
    grunt.loadNpmTasks('grunt-contrib-watch');

    // Default task(s).
    grunt.registerTask('default', ['watch']);

};

I’m not that experienced with PhantomJS obviously but I’m thinking this is either to do with PhantomJS or maybe because uncss doesn’t take .php files as input?
Thanks for the task!

I have the same issue

Same issue here for me. Even with html files.

Same

I got the same problem, I am using Node.js 4.2.1, npm 2.14.7, grunt 0.4.5.

+1 for me.
No one has solved this yet? I am also on El Capitan.

Hi.
I solved the problem by change the file type. convert (.php) file to (.html) file.
after uncss, delete (.html) file.

+1


[Warn] Non-function value encountered

Vue version
3.4.21
Link to minimal reproduction
CodeSandbox
Steps to reproduce
Importing component and providing it as a prop to another component.
<template>
  <Test :components="components" />
</template>

<script>
import Test from "../assets/test.mdx";
import Link from "./Link.vue";

export default {
  components: {
    Test,
  },

  setup() {
    const components = {
      a: Link,
    };

    return { components };
  },
};
</script>

What is expected?
No warnings
What is actually happening?
[Vue warn]: Non-function value encountered for default slot. Prefer function slots for better performance.
System Info
No response
Any additional comments?
No response

I did just that initially and they claim the issue is with Vue and not related to MDX, asking me to post the issue here instead.

I think this has nothing to do with MDX. It works with many runtimes, and they work. Vue is choosing to warn. Duckduckgo’ing your warning seems to indicate that Vue dislikes certain h calls (https://stackoverflow.com/questions/69875273/non-function-value-encountered-for-default-slot-in-vue-3-composition-api-comp). But we use the automatic runtime. Vue chooses to support it. It should not warn.

https://github.com/mdx-js/mdx/issues/2512#issuecomment-2220985663

I've tried to dig into this a bit further, though I'm very much out of my depth here.
As I understand it, MDX is setting compilerOptions.jsxImportSource to 'vue'. This ultimately leads to this bit of Vue code:
https://github.com/vuejs/core/blob/314ce82e479dbb33a9281ba8c2ebe288536b32df/packages/vue/jsx-runtime/index.mjs#L1-L12
The problem being that h is then called with non-function children when type is a component.
Perhaps the implementation of the jsx function above should be tweaked to automatically wrap children if type is a component?

Sounds like that could possibly be the case.
Maybe @LinusBorg can further clarify as to why this is an issue with @mdx-js/vue and not vue?


[stable/openebs] Update jiva image default to 1.7.1

Signed-off-by: mittachaitu<EMAIL_ADDRESS>
Is this a new chart

NOTE: We're experiencing a high volume of PRs to this repo and reviews will be delayed. Please host your own chart repository and submit your repository to the Helm Hub instead of this repo to make them discoverable to the community. Here is how to submit new chart repositories to the Helm Hub.

No
What this PR does / why we need it:
This PR Updates the Jiva image defaults to 1.7.1 and it changes the following files

Updated README
Updated chart version
Updated values.yaml

Which issue this PR fixes
(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

fixes #

Special notes for your reviewer:
Checklist
[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

[ ] DCO signed
[ ] Chart Version bumped
[ ] Variables are documented in the README.md
[ ] Title of the PR starts with chart name (e.g. [stable/mychartname])

/ok-to-test


Convert components with variants to a common variant=".." API

Is your feature request related to a problem? Please describe.
certain components like button and fab have properties with incompatible variant apis. e.g.
<mwc-button outlined raised>
An outlined button cannot be raised
Describe the solution you'd like
<mwc-button variant="outlined"> and <mwc-button variant="filled-raised">
same goes for fab extended and mini and standard

Is this still the preferred way of implementing variants (value attributes). The main readme.md seems to indicate that the variants will become separate components?

Yeah, we decided to go with separate components due to how google's internal build visibility system works as well as internal metrics and tracking across google teams.


Bitwarden access manager

Description
Add script to retrieve Bitwarden configuration using API and to compare with a configuration file.
The script reuse the logic of github_access_manager script and some parts can be share between the two script. This will be perform in another PR.
Note for the reviewer

member attribute access_all is set to true only when a member has the role type owner
member attribute type refers to the member role: 0 = user, 1 = owner, 2 = admin, 3 = manager, 4 = custom (when writing this, I realize that I forgot to add a logic to make it readable in the toml config. I will add it later)
group attribute access_all is set to true only when Grant access to all current and future collections box is tick.

GroupCollectionAccess read_only attribute is set to true when the permission Can view or Can view, except password are set. This is set to false when the permission Can edit or Can edit, except password are set. The API doesn't do the distinction Can ... and Can ..., except password permissions.
The API doesn't allow to retrieve the collection name, we need to set external ID in the UI to retrieve his name

Example output
The following collections on Bitwarden need to be changed to match organization.toml:

  [[collection]]
  collection_id = 8e69ce49-85ae-4e09-a52c-afaf00a90a3f
  external_id = ""
  member_access = [
    { member_id = 2564c11f-fc1b-4ec7-aa0b-afaf00a9e4a4, member_name = "yan", role = "c6a13b93-edc1-4c3b-9fc5-afaf00a8d33f" },
  ]
  group_access = [
-   { group_id = c6a13b93-edc1-4c3b-9fc5-afaf00a8d33f, group_name = "group1", readOnly = "True" },
+   { group_id = c6a13b93-edc1-4c3b-9fc5-afaf00a8d33f, group_name = "group2", readOnly = "True" },
  ]

The following members on Bitwarden need to be changed to match organization.toml:

  [[member]]
  member_id = 2564c11f-fc1b-4ec7-aa0b-afaf00a9e4a4
  member_name = yan
- email =<EMAIL_ADDRESS>- type = 3
+ email =<EMAIL_ADDRESS>+ type = 2
  access_all = False

The following groups on Bitwarden need to be changed to match organization.toml:

  [[group]]
  group_id = c6a13b93-edc1-4c3b-9fc5-afaf00a8d33f
  group_name = group1
- access_all = False
+ access_all = True

The following members of group 'group1' are specified in organization.toml, but are not present on Bitwarden:

  yunkel

Issue chorus-infrastructure #3941

This is still unresolved.

What is unresolved here ?

This: https://github.com/ChorusOne/github-access-manager/pull/10/files#r1128160013. If you go to the “files changed” tab you can see the full context, from the “conversation” tab sometimes things are missing.

Thanks, I will check with the full context next time !


RPGitems not working on bukkit 1.9.1

I have a bukkit server and i am using this version of the plugin. At first it seems to work fine but any other command except "/rpgitems" doesn't work, it doesn't say anything.
Also it spams the console with error messages.
1461511999072-log.txt
Here is the config
language: en_GB
autoupdate: false
defaults:
hand: One handed
sword: Sword
damage: Damage
armour: Armour
support:
worldguard: false
version: '0.5'

RPGItems is only compatible with spigot
1.7.2-v3.5.106 is for spigot 1.7 (#92)

(Ugh I meant to post this like 2 days ago...)
That 1.7.2-v3.5.106 you see, means it's for Minecraft 1.7, which is very different from 1.9.
I don't know if the current builds have 1.9 support added, but the last one specifically for 1.9 is this build of 105: https://github.com/NyaaCat/RPGitems-reloaded/tree/1.9-v3.5.105


is there a java server sample that can work with this project?

Hi,
since I cannot join the chat.socket.io , I tried to use a simple java server sample  to run this project. But it show these log when I emit login event:
2016-12-12 10:31:47.386 ERROR 582 --- [ntLoopGroup-3-4] c.c.socketio.JsonSupportWrapper          : Can't read value: ["chatevent","{userName:gg}"] for type: class com.corundumstudio.socketio.protocol.Event
And I check the json android client submit , find out the source code wrapped everything with jsonarray as above and it cannot be mapped to a com.corundumstudio.socketio.protocol.Event class. Please help, how can I make it run?

Hi @machinezhou I am having a similar problem. Is it that you can't proceed from the login activity on your app with the default server chat.socket.io ?

@Lazarus118 yes and socket.io website is closed too.

https://github.com/socketio/socket.io/tree/master/examples/chat
I just do as the url said, and it work very well .

https://github.com/socketio/socket.io/tree/master/examples/chat
I just do as the url said, and it work very well .


Support everything new in Tailwind 3.0

https://tailwindcss.com/blog/tailwindcss-v3

Is there a diff can one easily update it?

Hi. Thanks the feedback. Hoping to add version 3 support soon.
@gobijan, PRs are always welcome. If you download the latest main branch, you will find some classes have already been updated for v3.0.0. These are in the "Definitions/3.0.0" folder. I use the Tailwind docs at, https://tailwindcss.com/docs, as closely as possible to define the structure of these files. The latest commits should provide some examples of my current update process. If you wish to contribute and have any questions, please let me know.

I had a look at the tailwindcss-intellisense package for vscode and how they do it and found that they developed an LSP for this. It is currently nested in the project (not a separate repo as one might suggest):
https://github.com/tailwindlabs/tailwindcss-intellisense/tree/master/packages/tailwindcss-language-server
Further they also provide a tailwindcss language service:
https://github.com/tailwindlabs/tailwindcss-intellisense/tree/master/packages/tailwindcss-language-service
It could make sense to evaluate this option so the extension for nova wouldnt need to reimplement the definitions based on the documentation.
Have you seen this? Would it make sense to you?
Cheers

Thanks @gobijan. This is definitely something I have considered and it does make complete sense. I was hoping to get some user input on this along the way. I'm going to create a new issue so that this discussion might get more attention.

This has been added in v4.0.


Fix grammar in docs

name: :memo: Documentation Fix
about: Simple grammatical error fix
Checklist

[ ] Is there an existing issue for this PR?

link issue here

[ ] Have the files been linted and formatted?

What docs page needs to be fixed?

Section: Redux Essentials
Page: Async Logic and Data Fetching

What is the problem?
A sentence is missing the word "of".
What changes does this PR make to fix the problem?
Adds the word "of".

Thanks!


Codegen: Arguments in Result Fields Support

This issue is a continuation of #264
The generation of the graphql files seem to complete now but the validation fails with the error Validation of GraphQL query document failed. I believe the issue is related to the warnings in the original error message about "limit" not being provided (example: .../graphql/queries.graphql: Field "videos" argument "limit" of type "Int!" is required but not provided.).
I've included a snippet of the schema below, it looks like the generation isn't adding the extra arguments (limit and nextToken) for the fields: posts, videos, stories, etc. Are arguments in result fields not supported?
Actual Schema that the mutation returns
type User {
    username: String!
    preferred_username: String!
    picture: String!
    email: String!
    name: String!
    family_name: String!
    posts(limit: Int!, nextToken: String): PostConnection
    videos(limit: Int!, nextToken: String): VideoConnection
    stories(limit: Int!, nextToken: String): StoryConnection
    podcasts(limit: Int!, nextToken: String): PodcastConnection
    notifications: NotificationConnection
    followerCount: Int!
    followingCount: Int!
    followerIds: [String!]!
    followingIds: [String!]!
    followingTags: [String!]
}

What's being generated (this mutation returns a User but the field posts, for example, should have the arguments, limit and nextToken):
mutation UpdateFollowingTags($tags: String!) {
  updateFollowingTags(tags: $tags) {
    username
    preferred_username
    picture
    email
    name
    family_name
    posts {
      items {
        id
        date
        updatedAt
        title
        authorId
        content
        likesCount
        video
        isLiked
      }
      nextToken
    }
    videos {
      items {
        id
        authorId
        date
        viewCount
        category
        upvotesCount
        upvotes
        downvotesCount
        downvotes
        title
        alternativeTitle
        thumbnailUrl
        alternativeThumbnailUrl
        description
        videoUrl
        videoLength
        tags
      }
      nextToken
    }
    stories {
      items {
        id
        date
        authorId
        image
        video
      }
      nextToken
    }
    podcasts {
      items {
        id
        date
        updatedAt
        url
        authorId
        description
        likesCount
        isLiked
      }
      nextToken
    }
    notifications {
      items {
        userId
        notificationId
        date
        likes
        newLikesCount
        followers
        newFollowersCount
      }
    }
    followerCount
    followingCount
    followerIds
    followingIds
    followingTags
  }
}

Thanks for reporting this, in the mean time could you add the a default value for the limit.

@yuth Thanks, I've added default values and got passed that issue but it seems there is an issue with using non-null arrays in arguments as well. The places I have $tags: [String!]! seem to be replaced with a single item instead of an array. I've included an example below ($users should be $users: [String!]!:
...
query GetUsers($users: String!) {
  getUsers(users: $users) {
...


python312Packages.yara-python: 4.5.0 -> 4.5.1

Diff: https://github.com/VirusTotal/yara-python/compare/refs/tags/v4.5.0...v4.5.1
Changelog: https://github.com/VirusTotal/yara-python/releases/tag/v4.5.1
Description of changes

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

Result of nixpkgs-review pr 315275 run on x86_64-linux 1

34 packages built:

acquire (python311Packages.acquire)
acquire.dist (python311Packages.acquire.dist)
apkid
apkid.dist
python311Packages.dissect
python311Packages.dissect-target
python311Packages.dissect-target.dist
python311Packages.dissect.dist
python311Packages.karton-autoit-ripper
python311Packages.karton-autoit-ripper.dist
python311Packages.karton-config-extractor
python311Packages.karton-config-extractor.dist
python311Packages.karton-yaramatcher
python311Packages.karton-yaramatcher.dist
python311Packages.malduck
python311Packages.malduck.dist
python311Packages.yara-python
python311Packages.yara-python.dist
python312Packages.acquire
python312Packages.acquire.dist
python312Packages.dissect
python312Packages.dissect-target
python312Packages.dissect-target.dist
python312Packages.dissect.dist
python312Packages.karton-yaramatcher
python312Packages.karton-yaramatcher.dist
python312Packages.yara-python
python312Packages.yara-python.dist
stacs
stacs.dist
volatility3
volatility3.dist
yaralyzer
yaralyzer.dist


⚠ HopeAlley Ecards has degraded performance

In a2c2ce0, HopeAlley Ecards (https://hopealley.com/ecards) experienced degraded performance:

HTTP code: 200
Response time: 6695 ms

Resolved: HopeAlley Ecards performance has improved in 5075fd9.


installing and running omnimotion with only cpu

Hello! Thank you for sharing this code - omnimotion looks really promising and I would love to test it out with some of my microscope videos. But having tried installing it, it seems that I cannot run CUDA on my computer. Is it possible to run omnimotion with cpu only?

Hi, unfortunately, it would be extremely slow to run on CPU. I suggest you look into this work https://co-tracker.github.io/ for your project.

Thank you for your suggestion! I will check it out :)


Tonyhax update image.

I would like a install disc image for tonyhax and entry-points.
Could be combined with the boot disc and if you don't swap out the disc and press a button to bring up a menu to install the different things on the memory cards.

Honestly that's something I planned but never got around implementing. Maybe some day.

There's apparently an installer here so I'm closing this: http://www.psxdev.net/forum/viewtopic.php?f=66&t=3910


Parallelize test runs

Could we add a feature for the runner to parallelize test execution (using an executor service based on the available core or whatever policy you deem appropriate)?
I've about 1000 parametrized tests in a single module and while intellij is showing me a total runtime of about 4s, in reality it takes more like 40-50s (i suspect the startup/shutdown harness work, class loading, etc, are at play). These are pure unit tests and do not access outside resources so they are definitely CPU bound and CPU utliziation is only about 30% while running. Suggestions very welcome. Thanks.

I found this https://github.com/MichaelTamm/junit-toolbox so now trying to figure out if i can use @Rule so i can make junitparams run with the parallel runner...

Actually I just saw this: https://github.com/Pragmatists/JUnitParams/issues/13 .... not looking promising. :(

Any solutions on how to run parameterized tests in parallel? It takes me around 5 sec to run one iteration of test. Running many iterations is time consuming .

We were trying to implement it but with no luck. To be honest I think we'll not implement it. Any support on it from you, who are interested in solution, will be appreciated. Maybe together we will find some quite simple solution for it.

Hi,
I'm interested in working on this. Perhaps we can discuss if this is possible and what are the options.


Instrumentation using OTEL

This PR adds Instrumentation to the orchestrator codebase.

Adds a file telemetry.rs containing configs and initialisation files for telemetry data.
used tokio tracing for adding tracing instrumentation.
It is to note that although #instrumentation is added to almost all the functions, they are yet commented, and will only be uncommented as per tracing demands.
Tracing Queue size is currently 10000, this can be decreased based on optimal usage.
Metrics are added to 'general_/create/process/verify_job` to let know what block is in which state, more metrics can be added upon requirement.
While tracing it's best to ignore heavy parameters and only keep small//useful params for analytics, you'll notice that config is always skipped.

coverage: 60.746% (-4.1%) from 64.811%
when pulling ef00afe128bb6de5c56523499f73cdf02ff863dd on feature/instrumentation
into ab66b83a49f62f774b9de48d012feab3f4f124d3 on main.

Completed Self Review, resolved this.


Make contributing to docs easier

With this PR readers can suggest changes to the docs with an easy to use WYSIWYG markdown editor.
This is how the editor looks like for this repo.
https://github.com/remorses/docusaurus-example/assets/31321188/43c8bc68-3668-4a25-bf1c-a70eceab7bcb

cool，I will try this feature in the coming days

Let me know if you have any feedback or find any bugs!

no problem，thx a lot.


cri: append envs from image config to empty slice to avoid env lost

Fix #5023
When call CreateContainer for multiple pods with the same image config but different envs in container spec at nearly the same time,  they get image config from imagestore but the Envs slice in the image config is pointing to the same object.  The Envs is json.Unmarshal from the content db, so the cap and len of the Envs slice maybe not equal.
So the operation for appending container env into the image env may cause the first few envs of the first container is overlapped by the second container (maybe from another pods) with some probability if the two container has the same image and created at the very near same time.
To resolve this, we can deepcopy image config directly after get image to avoid env or other field conflict.

Could you please sign off your commits (try git commit -s) in order to pass CI checks?
https://github.com/containerd/project/blob/master/CONTRIBUTING.md#sign-your-work

Could you please sign off your commits (try git commit -s) in order to pass CI checks?
https://github.com/containerd/project/blob/master/CONTRIBUTING.md#sign-your-work

Done, Thanks.

CI failure seems to related to https://github.com/containerd/containerd/issues/4924.
Exception 0xc0000420 0x0 0x0 0x7ff871002e0a
PC=0x7ff871002e0a
signal arrived during external code execution

time="2021-02-18T08:45:54Z" level=info msg="running tests against containerd" revision=66f2b3647c7a3d0618085dc4276a39ab6f426859 runtime=io.containerd.runc.v2 version=66f2b36
time="2021-02-18T08:45:54Z" level=info msg="start to pull seed image" image="docker.io/library/alpine:latest"
--- FAIL: TestContainerPids (0.11s)
    container_test.go:459: expected 2 process but received 1
FAIL

Not sure it is flaky case. rerun the CI.

Hmm...
-- FAIL: TestSnapshotterSuite (0.77s)
    --- FAIL: TestSnapshotterSuite/DevMapperUsage (0.77s)
        snapshotter_test.go:141: assertion failed: expression is false: layer2Usage.Size < sizeBytes+256*dmsetup.SectorSize: 1179648 < 1179648
FAIL
FAIL	github.


Cuda capabilities and cuda versions

Hi!
Seems like an awesome and easy module, and hope i get to use it, just running into a bit of problems when running your examples.
Does this only work with CUDA toolkit 10.0 ? and a GPU with cuda capabilities of 6 ?
I'm running into problems saying my GPU (which have cuda capabilities 5 ) is not high enough.
Is there a way to solve this problem ?
I also have to run on GPU, since my model that's built in python tensorflow requires a gpu to un on.
Appreciate any help!

Hi!
Sorry for the late answer. Which CUDA toolkit and cuDNN version do you have?

@paalpri I've got it running with the Cuda toolkit 10.0. I'm completely new on this domain, so I don't know what do you mean with "cuda capabilities of 6". I've installed Cudnn 7.6.4 from here: https://developer.nvidia.com/rdp/cudnn-archive, debian/buster, Nvidia driver from the apt repos.


error cert

I put the server on google cloud
ubuntu 16
ran the script by setting everything up in stages
domain is configured correctly on cloudflare
I also checked with www.
error in end :
IMPORTANT NOTES:
 - The following errors were reported by the server:
   Domain: axtar.work
   Type:   unauthorized
   Detail: Invalid response from
   https://axtar.work/.well-known/acme-challenge/m3GMqkFTsBKJHHtLNR_mF6yVBWGU-pcQPB7PUi7Lt-E
   [2606:4700:3033::681f:5abe]: "<!DOCTYPE html>\n<!--[if lt IE 7]>
   <html class=\"no-js ie6 oldie\" lang=\"en-US\">
   <![endif]-->\n<!--[if IE 7]>    <html class=\"no-js "
   To fix these errors, please make sure that your domain name was
   entered correctly and the DNS A/AAAA record(s) for that domain
   contain(s) the right IP address.
Failed to get cert.

screens :
1 cloud
screen
2 cloudflare
screen
3 terminal error end
screen

Try to uncheck Proxy status in your Cloudflare DNS settings before run this script.
This script use HTTP-01 challenge to get a certificate from Let’s Encrypt, but your server IP will be hidden by Cloudflare if Proxy status is enabled. Let’s Encrypt can't access to your web server through Cloudflare's IP, because Cloudflare does not allow direct IP access. As a result, the validation checks fail.

shadowsocks android error
unable to reslove host "cp.cloudflare"...


⚠ Bofrost.de has degraded performance

In c645bac, Bofrost.de (https://www.bofrost.de) experienced degraded performance:

HTTP code: 200
Response time: 3442 ms

Resolved: Bofrost.de performance has improved in 85a7151 after 7 minutes.


No basic auth credentials supplied using custom registry

When configuring a custom registry (using Amazon ECR), Basic Auth credentials are stripped out on pulling the docker image from the registry:
docker-compose.yml:
myapp:
  image: 95629****971.dkr.ecr.us-east-1.amazonaws.com/app

Results in:
Waiting for [instance:mystack_myapp_1]. Instance status: 500 Server Error: Internal Server Error ("Get    https://95629****971.dkr.ecr.us-east-1.amazonaws.com/v2/app/manifests/latest: no basic auth credentials")
Though the registry shows active.

Useful
Info

Versions
Rancher v1.0.0 Cattle: v0.159.2 UI: v0.100.3

Access
Disabled

Route
environment.index

Can you confirm what type of environment you have? Cattle/K8S/Swarm?

@deniseschannon I'm using Cattle on CentOS 7.

I re-read the issue and realized that this is a duplicate of https://github.com/rancher/rancher/issues/3126.
We don't have support for Amazon ECR.

Ok I thought this was now supported: http://rancher.com/using-amazon-container-registry-service/

It looks like there is a basic auth issue with custom registries. Please follow: https://github.com/rancher/rancher/issues/3634
Thanks for pointing out the Amazon ECR blog post. :)

I forgot to mention that it will be in our next major release (1.1).


Fix plugin/git ggpull and ggpush backward compatibility

Current ggpush is git push origin instead of git push origin $(current_branch)
Same for ggpull
Closes #4055
Closes #3995
And probably many others.

Closes #4053

Thanks, I was about to do it.


tss2_tpm2_types.h: Fix typo in HMAC_SESSION_LAST and POLICY_SESSION_L…

…AST.
MAX_ACTIVE_SESSIONS1 isn't defined, this should instead be
MAX_ACTIVE_SESSIONS - 1
Signed-off-by: Philip Tricca<EMAIL_ADDRESS>

this resolves #575