Unsupported Architectures

When I try to submit the bundle to the TestFlight I get this error.
[Transporter Error Output]: ERROR ITMS-90087: "Unsupported Architectures. The executable for NativeRTC.app/Frameworks/WebRTC.framework contains unsupported architectures '[x86_64, i386]'."
[13:44:54]: [Transporter Error Output]: ERROR ITMS-90209: "Invalid Segment Alignment. The app binary at 'NativeRTC.app/Frameworks/WebRTC.framework/WebRTC' does not have proper segment alignment. Try rebuilding the app with the latest Xcode version."
[13:44:54]: [Transporter Error Output]: ERROR ITMS-90125: "The binary is invalid. The encryption info in the LC_ENCRYPTION_INFO load command is either missing or invalid, or the binary is already encrypted. This binary does not seem to have been built with Apple's linker."
[13:44:54]: [Transporter Warning Output]: WARNING ITMS-90080: "The executable 'Payload/NativeRTC.app/Frameworks/WebRTC.framework' is not a Position Independent Executable. Please ensure that your build settings are configured to create PIE executables. For more information refer to Technical Q&A QA1788 - Building a Position Independent Executable in the iOS Developer Library."
[13:44:55]: Transporter transfer failed.
[13:44:55]: WARNING ITMS-90080: "The executable 'Payload/NativeRTC.app/Frameworks/WebRTC.framework' is not a Position Independent Executable. Please ensure that your build settings are configured to create PIE executables. For more information refer to Technical Q&A QA1788 - Building a Position Independent Executable in the iOS Developer Library."

I'm looking at this stackoverflow question. Any help much appreciated :)
Edit 1:
As far as I can see x86_64 architecture is for iPad simulators while i386 is for iPhone simulators, both are 32 and 64bit compatible.
Edit 2:
Here they talk about Archive validation error: Invalid Segment Alignment and one of the answers refers to:

In my case I was submitting an app that include a framework I had built that was a universal framework - created using lipo (for use with both device and simulator).
After replacing the universal compiled framework with device compiled framework, validation was successful.

Edit 3:
Using lipo --info, you can see WebRTC.framework/WebRTC have multiple architectures indeed i386 x86_64 armv7 arm64.
> lipo -info NativeRTC.xcarchive/Products/Applications/NativeRTC.app/Frameworks/WebRTC.framework/WebRTC
Architectures in the fat file: NativeRTC.xcarchive/Products/Applications/NativeRTC.app/Frameworks/WebRTC.framework/WebRTC are: i386 x86_64 armv7 arm64

Edit 4.
From this article looks like https://github.com/oney/react-native-webrtc/blob/master/ios/WebRTC.framework/WebRTC has been complied with i386 x86_64 armv7 arm64 architectures. You can see it by cloning it and running lipo -detailed_info ios/WebRTC.framework/WebRTC.
I'm gonna try to remove i386 x86_64.
Edit 5.
Running:
cd node_modules/react-native-webrtc/ios/WebRTC.framework/
lipo -remove i386 WebRTC -o WebRTC && lipo -remove x86_64 WebRTC -o WebRTC

Solves the problem. I'm gonna test however if it still works on simulators.
Edit 6.
Nope! I get the following error:
Undefined symbols for architecture i386:
  "_OBJC_CLASS_$_RTCConfiguration", referenced from:
      objc-class-ref in libRCTWebRTC.a(RCTConvert+WebRTC.o)
blabla

Edit 7.
So clearly there some issues, at least on my side. Is anyone experiencing the same problems?
Edit 8.
Apparently is an app store submission bug http://www.openradar.me/radar?id=6409498411401216
Edit 9.
I created this little script which helps me jump between architectures during dev and app store release.
'use strict';

const fs = require('fs');
const exec = require('child_process').execSync;

const WEBRTC_BIN_PATH = `${__dirname}/node_modules/react-native-webrtc/ios/WebRTC.framework`;
const ARCH_TYPES = ['i386','x86_64','armv7','arm64'];

if(process.argv[2] === '--extract' || process.argv[2] === '-e'){
  console.log(`Extracting...`);
  ARCH_TYPES.forEach(elm => {
    exec(`lipo -extract ${elm} WebRTC -o WebRTC-${elm}`,{cwd:WEBRTC_BIN_PATH});
  });
  exec('cp WebRTC WebRTC-all',{cwd:WEBRTC_BIN_PATH});
  console.log(exec('ls -ahl | grep WebRTC-',{cwd:WEBRTC_BIN_PATH}).toString().trim());
  console.log('Done!');
}

if(process.argv[2] === '--simulator' || process.argv[2] === '-s'){
  console.log(`Compiling simulator...`);
  exec(`lipo -o WebRTC -create WebRTC-x86_64 WebRTC-i386`,{cwd:WEBRTC_BIN_PATH});
  console.log(exec('ls -ahl | grep WebRTC',{cwd:WEBRTC_BIN_PATH}).toString().trim());
  console.log('Done!');
}

if(process.argv[2] === '--device' || process.argv[2] === '-d'){
  console.log(`Compiling device...`);
  exec(`lipo -o WebRTC -create WebRTC-armv7 WebRTC-arm64`,{cwd:WEBRTC_BIN_PATH});
  console.log(exec('ls -ahl | grep WebRTC',{cwd:WEBRTC_BIN_PATH}).toString().trim());
  console.log('Done!');
}

Did you follow this guide to a T?
I was having similar issues, and I think setting Enable Bitcode to NO will help you.

@aleclarson thanks for you reply. Yes I did.
I set Dead Code Stripping to No`` and Enable Bitcodeto No, both on my main xcode project as well as inRCTWebRTC.xcodeproj`. Still same problem.
However to avoid possible caching problem I will start a new brand project and let you know.

The stackoverflow question you linked to solved it for me. I just followed the first answer.

make documents clearer in #174

I also had to use the stackoverflow script. Please add instructions to the readme!

@8enmann
added in fcfa9d1bd3cc3178a88ec4e8588d8f900654704c
thanks

Thanks!
On Sun, Nov 27, 2016, 8:18 AM zxcpoiu<EMAIL_ADDRESS>wrote:

@8enmann https://github.com/8enmann
added in fcfa9d1
https://github.com/oney/react-native-webrtc/commit/fcfa9d1bd3cc3178a88ec4e8588d8f900654704c
thanks
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/oney/react-native-webrtc/issues/141#issuecomment-263121770,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA-UsPNjiTAo9tvzcjyJozffmOXoFPtPks5rCYM1gaJpZM4KHoXm
.

Even after doing lipo -remove i386 WebRTC -o WebRTC && lipo -remove x86_64 WebRTC -o WebRTC, my build is disappearing from TestFlight without any error or warning. Do you have a solution?

I was having this same issue, even after trying the script included here. (Thanks for that, though!) The tiny script in this post fixed the arch issue reliably without having to run anything manual, and resolved all of the other validation issues I was having (!!!): http://ikennd.ac/blog/2015/02/stripping-unwanted-architectures-from-dynamic-libraries-in-xcode/
Thanks for this library, by the way!

I have a question?
Where do i have to put your script?
Have problem with iOS 10.2

ld: warning: object file (/Users/anhtuank7c/Desktop/Live/ios/build/Build/Products/Debug-iphonesimulator/libRCTWebRTC.a(RTCVideoViewManager.o)) was built for newer iOS version (9.2) than being linked (8.0)
ld: warning: object file (/Users/anhtuank7c/Desktop/Live/ios/build/Build/Products/Debug-iphonesimulator/libRCTWebRTC.a(WebRTCModule.o)) was built for newer iOS version (9.2) than being linked (8.0)
ld: warning: object file (/Users/anhtuank7c/Desktop/Live/ios/build/Build/Products/Debug-iphonesimulator/libRCTWebRTC.a(WebRTCModule+RTCPeerConnection.o)) was built for newer iOS version (9.2) than being linked (8.0)
ld: warning: object file (/Users/anhtuank7c/Desktop/Live/ios/build/Build/Products/Debug-iphonesimulator/libRCTWebRTC.a(WebRTCModule+RTCMediaStream.o)) was built for newer iOS version (9.2) than being linked (8.0)
ld: warning: object file (/Users/anhtuank7c/Desktop/Live/ios/build/Build/Products/Debug-iphonesimulator/libRCTWebRTC.a(RCTConvert+WebRTC.o)) was built for newer iOS version (9.2) than being linked (8.0)
ld: warning: object file (/Users/anhtuank7c/Desktop/Live/ios/build/Build/Products/Debug-iphonesimulator/libRCTWebRTC.a(WebRTCModule+RTCDataChannel.o)) was built for newer iOS version (9.2) than being linked (8.0)
ld: framework not found WebRTC for architecture x86_64
clang: error: linker command failed with exit code 1 (use -v to see invocation)

Here is what I think the correct execution order to get archive right:

run the script with -e
run the script with -d
remove all generated bundle other than WebRTC under node_modules/react-native-webrtc/ios WebRTC.framework/
do  the archiving.
validate or upload.

For myself it just do fine.

@besarthoxhaj
After executing your script with the order -e and -d, and creating the .ipa file, when i try to upload to app store, other than my app these are also listed.

When i upload i get errors similar to

"nvalid Bundle Structure - The binary file 'MyApp.app/Frameworks/WebRTC.framework/WebRTC-i386' is not permitted. Your app can't contain standalone executables or libraries, other than the CFBundleExecutable of supported Bundles.

@siemya
remove all generated bundle other than WebRTC under node_modules/react-native-webrtc/ios WebRTC.framework/

@liudongpu That solved it thanks!
However i have a new problem #223

In terminal: cd react-native-webrtc/tools/ && ./downloadBitcode.sh
It helped me.

Anyone ran the script in a Microsoft App Center build?


VS Code freezes

I open a c# file and VS freezes, on notepad, visual studio community works.
The file have 204KB, aprox. 4102 lines.
I tried reopen without extensions and doesnt work.

System config:

@UndPat Can you open the file if you rename it to .txt ? I believe you might be hitting an issue with a specific code construct that you have in your file that is causing our C# tokenization grammar to take a very very long time to match regular expressions against that particular code pattern when tokenizing the C# file.
Is it possible for you to share the file or a similar file that reproduces the problem?

Renaming the file to .txt works.
_wTeste.txt
(Uploaded in .txt mode, .cs can't)

Narrowed it down to this:
public override String getDynamicStatement(int cursor, Object[] dynConstraints)
{
      return conditional_W04RM2( (int)dynConstraints[0] , (short)dynConstraints[1] , (short)dynConstraints[2] , (int)dynConstraints[3] , (int)dynConstraints[4] , (int)dynConstraints[5] , (short)dynConstraints[6] , (short)dynConstraints[7] , (short)dynConstraints[8] , (short)dynConstraints[9] , (String)dynConstraints[10] , (DateTime)dynConstraints[11] , (DateTime)dynConstraints[12] , (short)dynConstraints[13] , (int)dynConstraints[14] , (short)dynConstraints[15] , (int)dynConstraints[16] , (int)dynConstraints[17] , (int)dynConstraints[18] , (short)dynConstraints[19] , (short)dynConstraints[20] , (short)dynConstraints[21] , (short)dynConstraints[22] , (short)dynConstraints[23] , (short)dynConstraints[24] , (String)dynConstraints[25] , (String)dynConstraints[26] , (DateTime)dynConstraints[27] , (short)dynConstraints[28] , (DateTime)dynConstraints[29] , (DateTime)dynConstraints[30] , (DateTime)dynConstraints[31] , (String)dynConstraints[32] , (String)dynConstraints[33] );
}

Moved to owning repo: https://github.com/dotnet/csharp-tmLanguage/issues/169


output directory

Hi. As I read in Readme file, I created an empty directory named output, and run this code:
python main.py hotpot_train_v1.1.json
but I get this Error:
'CUDA_VISIBLE_DEVICES' is not recognized as an internal or external command,
operable program or batch file.
Traceback (most recent call last):
  File "prepare_pred_gold.py", line 34, in <module>
    with open(preds_file) as reader:
FileNotFoundError: [Errno 2] No such file or directory: 'output/pred_gold_idx.json'
'CUDA_VISIBLE_DEVICES' is not recognized as an internal or external command,
operable program or batch file.
Traceback (most recent call last):
  File "combine_pred.py", line 46, in <module>
    main(span_pred_file, sp_pred_file)
  File "combine_pred.py", line 17, in main
    with open(span_pred_file) as fid:
FileNotFoundError: [Errno 2] No such file or directory: 'output/predictions_ans.json'

I even tried to create two empty json files in output directory with those names that are in error message but it still doesn't work.
could you please help me with this?

I have also encountered this problem. Is there any solution?

ok, thank you. 董磊 @.*** 重庆邮电大学 学生
…
---Original--- From: @.> Date: Wed, Oct 20, 2021 17:13 PM To: @.>; Cc<EMAIL_ADDRESS>Subject: Re: [JD-AI-Research-Silicon-Valley/SAE] output directory (#3) You're probably on windows, just replace CUDA_VISIBLE_DEVICES=0 by set CUDA_VISIBLE_DEVICES=0 & in main.py Hi can you find train.py in run_hotpotqa_roberta.py? I can't find it — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.
Hi ！ can you find train.py in run_hotpotqa_roberta.py? I can't find it. would you provide it for me ,i will  appreciate it .

ok, thank you. 董磊 @.*** 重庆邮电大学 学生
…
---Original--- From: @.> Date: Wed, Oct 20, 2021 17:13 PM To: @.>; Cc<EMAIL_ADDRESS>Subject: Re: [JD-AI-Research-Silicon-Valley/SAE] output directory (#3) You're probably on windows, just replace CUDA_VISIBLE_DEVICES=0 by set CUDA_VISIBLE_DEVICES=0 & in main.py Hi can you find train.py in run_hotpotqa_roberta.py? I can't find it — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.

ok, thank you. 董磊 @.*** 重庆邮电大学 学生
…
---Original--- From: @.> Date: Wed, Oct 20, 2021 17:13 PM To<EMAIL_ADDRESS>Cc: _<EMAIL_ADDRESS>Subject: Re: [JD-AI-Research-Silicon-Valley/SAE] output directory (#3) You're probably on windows, just replace CUDA_VISIBLE_DEVICES=0 by set CUDA_VISIBLE_DEVICES=0 & in main.py Hi can you find train.py in run_hotpotqa_roberta.py? I can't find it — You are receiving this because you commented. Reply to this email directly, view it on GitHub, or unsubscribe. Triage notifications on the go with GitHub Mobile for iOS or Android.
Hi ！ can you find train.py in run_hotpotqa_roberta.py? I can't find it. would you provide it for me ,i will  appreciate it .

Sorry, I haven't dealt with this for a long time, and I can't find it.
这个模型，你以前实现过吗？

请问你们解决了吗，可一起交流下吗？

我在https://github.com/JD-AI-Research-Silicon-Valley/SAE/blob/master/run_hotpotqa_roberta.py 里面没有找到train.py文件，根据论文中的描述，有一个选择文档的过程，使用pairWise排序算法，在代码中没有看到是如何实现的，就是这个train_dataset是怎么来的？看了一周的代码，还是很疑惑，希望能不吝赐教，感谢

请问你们解决了吗，可一起交流下吗？

我在https://github.com/JD-AI-Research-Silicon-Valley/SAE/blob/master/run_hotpotqa_roberta.py 里面没有找到train.py文件，根据论文中的描述，有一个选择文档的过程，使用pairWise排序算法，在代码中没有看到是如何实现的，就是这个train_dataset是怎么来的？看了一周的代码，还是很疑惑，希望能不吝赐教，感谢

可以加一下联系方式，一起交流一下嘛

请问你们解决了吗，可一起交流下吗？

还没解决，最近没看这个代码。

wx:13273814025


Fixing debugger issue in Scenarios with dynamic expression

Description
Thanks for contributing this Pull Request. Make sure that you submit this Pull Request against the develop branch of this repository, add a brief description, and tag the relevant issue(s) and PR(s) below.

Relevant Issues : https://github.com/intuit/karate/issues/1452
Relevant PRs : (optional)
Type of change :

[ ] New feature
[x] Bug fix for existing feature
[ ] Code quality improvement
[ ] Addition or Improvement of tests
[ ] Addition or Improvement of documentation

testing not complete but fyi @ivangsa

testing not complete but fyi @ivangsa

Ready for review - faced a few issues with scenario outlines with dynamic expressions as I posted on the related issue https://github.com/intuit/karate/issues/1452 but it doesn't seem to be related to this PR.

Ready for review - faced a few issues with scenario outlines with dynamic expressions as I posted on the related issue https://github.com/intuit/karate/issues/1452 but it doesn't seem to be related to this PR.


Switch active/inactive state

Hey :)
Try to create a switch and play with interactable/isOn.
If you have interactable unchecked, and toggle isOn (true to false, then false to true etc...) you'll see that the switch changes color instead of using the disabled color.

I'll work on that.

Fixed :)


download jenkins-cli.jar instead of extracting it from jenkins.war

… the Jenkins-cli.
https://github.com/voxpupuli/puppet-jenkins/issues/998

Pull Request (PR) description

This Pull Request (PR) fixes the following issues

This has a merge conflict. Could you make sure it cleanly applies?

This has a merge conflict. Could you make sure it cleanly applies?

The conflict had been fixed

The conflict had been fixed

For reference, the docs for this are here: https://www.jenkins.io/doc/book/managing/cli/#downloading-the-client

For reference, the docs for this are here: https://www.jenkins.io/doc/book/managing/cli/#downloading-the-client

Do we need more changes or it could be merged?

Do we need more changes or it could be merged?

We need the tests to pass. I rebased on latest master and squashed the commits.

The current solution does work and there hasn't been activity for a year. Closing, but feel free to open a new PR.


PHP8.1 compatibility

close #1245
This will probably not be merged as it contains breaking changes.
phpbrew will not work correctly in environments where only PHP8.1 is installed.
This is because phpbrew has not been able to adapt to the changes in PHP 8.1.
Various dependent packages have been updated to apply these changes.
https://github.com/c9s/CLIFramework/pull/121
https://github.com/silexphp/Pimple/pull/266
However, a side effect of this is that I now need PHP 7.2.5 or higher, because pimple requires 7.2.5 or higher.
I don't know phpbrew's support policy, so I don't know if it is correct to break environment compatibility by merging this PullRequest.
If all is well, merge the Pull Request, otherwise close it. However, you will not be able to use phpbrew with PHP 8.1 or later if you don't.

please review
@morozov @c9s @Gasol @jhdxr @marcioAlmada @shinnya @GM-Alex

I have bumpped the version of CLIFramework, here
https://github.com/c9s/CLIFramework/releases/tag/4.1.2

update 4.1.2

@dmnlk I guess we can mark this change for 2.0, so maybe:
phpbrew 2.0 supports php7 and php8
phpbrew 1.x supports php5~php7.2

@c9s this is good idea.　If phpbrew itself doesn't work with the latest version of phpbrew, that's a problem, but if you can support it with 2.0, I don't think it's a problem.

Is there a need to bump to 2.x at the moment?
PHP 8.1 is already out of the door for a while and soon PHP 8.2 will be out. Besides, semver is also fine with bumping dependencies & language constraints in minor version.
So wouldn't it be fine to:

allow PHP 8.1+ in 1.x
bump the min PHP version to use phpbrew to a much more recent PHP version in 1.x
keep compatibility to install older PHP versions

?
I think this would a really nice improvement and also could help to move forward the codebase & contributions. And as this goes forward, it can also be clearer what part of the application need to change for 2.0 and could be deprecated in 1.x more easily, facilitating the upgrade.

https://github.com/phpbrew/phpbrew/pull/1264#issuecomment-1139307895
@theofidry, I think the above solution can be satisfied.

@peter279k it sure can, it just feels a bit more complicated and although means 2.x is keeping PHP7 around which I also don't get why this is desired; But honestly I'm happy with whatever makes things move forward

I also agree with @theofidry because we can still support 1.x version and it can be easy to be maintained.
And we can let dependencies upgrading work be minimum.

Just notice that the 1.28.0 version is released.
After discussing with the @c9s, we decide to let the 1.28.0 be the final 1.x version.
That is, it's the final version to support < 7.2 version.
The next released version is 2.x version and it's for running PHPBrew building tool will support the 7.2 version at least.
Why? The current reason is some dependencies cannot support these too old < 7.2 versions and it's time to drop these PHP versions.

I'm going to close the PR and create new one to include this PR changes :).


[context] Simplify usage of reified Context#jsonAsType by utilizing generic type in parameter

#1779

Thanks !


Most (not all) Click Event not working on https://material-ui.com website on firefox browser

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

Tech
Version

Browser
Firefox Quantum 63.0.1 (64-bit)

etc.
OS Ubuntu 18.10

What version do you have here?

I have the same problem and I can't even tell what version it is because I can't open the side drawer.

Something interesting discovery
Starting from any link in the component api manual or components demo manual say
https://material-ui.com/demos/app-bar/ or https://material-ui.com/api/app-bar/ should bootstrap the router used on the site.
If you lunch the site from a deep link like i now do, check the top left sidebar/drawer and make sure you are on v3.4.0 else click the release version to change the release docs, as i noticed the default docs are for release v3.3.2 which has the routing issues
For anyone facing these problems like i did
Clear all the cached files and all local site data  on https://material-ui.com/ especially the files cached by workbox all have to be purged and the docs defaults to v3.4.0 :100:
Other
I have been running manual click and touch test on my apps and sites built with v3.x.x to see if the bug bled into the distro source. and so far every thing seems fine but the test isn't over so fingers crossed
@oliviertassinari changing the caching rules for workbox or disabling caching complete should fix the site and guard from future breaks on new releases. what do you think?

especially the files cached by workbox all have to be purged and the docs defaults to v3.4.0 💯

Note for self, the next time we have a service worker issue, the best approach is to release an empty implementation for a week, then move to a working version.


Vector DB Sink configuration is not documented and validated

There's only the datasource config property
https://docs.langstream.ai/building-applications/api-reference/agents#vector-db-sink
we should add it for each datasource type

Fixed by #583


chore(.github/workflows/nightly_detect_failure.yml): fix the sha hash

The sha hash was only printed to the github env if the git tag
"nightly-testing-${version}" did not yet exist.
We move this out of the if clause, so that it is always printed.

Thanks!
bors r+


Awesome Plug In! But...

I love this plug in as it has some handy dandy things in it. However I recommend removing any open device settings functions as it includes the use of App-prefs:root APIs to open the settings in which Apple now rejects any apps that use this.
It's no big deal, but I do always have to go into the node modules folder and remove this function before sending to Apple for Approval, just something to think about!

Our app was rejected by Apple review due to the App-prefs:root call as well.  We were only using the moment library, so we removed nativescript-toolbox plugin and installed moment directly through npm as a workaround.  Just wanted to confirm this will be a problem for anyone else submitting to the iOS store using this plugin.


Update the Repository and Policy object in Harvard's asset image to reflect the text with paragraph breaks and hyper links

See this doc for details https://docs.google.com/document/d/1nepZM2rC6FbkH7r3JDkgsIg27_4ca6s9YQyu9of-YNg/edit
Please be sure to capture the hyper links as well as paragraph breaks as described.

this is complete


Make rubocop configurable

I like the idea to have all the configurations on a central repository, but the problem with that is that some people might use some configurations differently. For instance, I personally think that --lint is a bit too restrictive.
I was using these configurations manually on my neovim. But I think that having the power to configure these settings can be helpful for more people.
With the proposed code, one can do:
local languages = {
  -- ...

  ruby = {
    require('efmls-configs.linters.rubocop')({
      args = '--format emacs --stdin "${INPUT}"',
      override = {
        lintCategoryMap = {
          A = 'H', -- autocorrect
          C = 'I', -- convention
          E = 'E', -- error
          F = 'E', -- fatal
          I = 'I', -- info
          R = 'H', -- refactor
          W = 'W', -- warning
        },
      },
    }),
  },

  -- ...
}

What do you think?

This is just complicating the process of extensibility of the configuration, which need to be introduced one level above the configuration (like the fs module).
But if you do want to configure a specific linter/formatter with your own custom

This is just complicating the process of extensibility of the configuration, which needs to be introduced one level above the configuration (like the fs module).
For now I just want to keep it simple and let the user handle their usecase.
But we can achieve this without a builder pattern by just using vim.tbl_extend():
local fs = require('efmls-configs.fs')

local ruby = require('efmls-configs.linters.rubocop')
ruby = vim.tbl_extend('force', ruby, {
  lintCommand = string.format('%s --format emacs --stdin "${INPUT}"', fs.executable('rubocop', fs.Scope.BUNDLE)),
  lintCategoryMap = {
    A = 'H', -- autocorrect
    C = 'I', -- convention
    E = 'E', -- error
    F = 'E', -- fatal
    I = 'I', -- info
    R = 'H', -- refactor
    W = 'W', -- warning
  },
})

This was partially documented under Default Configurations (last paragraph).
I think it's better to document this clearly.

I really like the suggestion! Thanks!


Schema caching

It looks like the schema caching was removed in 3e30d8287ddab91a5b9b3b5709d4ec8ce9a3ee3c (this removed the lru_cache decorator that was added in #41).  Would there be support for adding caching back in?
For recursive validating with many items, it would be nice to avoid fetching the item schema for each item.

Hi. This definitely sounds like a good idea.


Build/go fast

Time to go faster - https://docusaurus.io/blog/releases/3.6#adoption-strategy

Summary by CodeRabbit

New Features

Introduced an experimental configuration option for enhanced performance in the Docusaurus application.
Added a new dependency to improve build efficiency.

Chores

Removed outdated Babel configuration file, streamlining project setup.

Dependencies added
@docusaurus/faster

DescriptionDocusaurus experimental package exposing new modern dependencies to make the build faster.
LicenseMIT

LGTM

Nice!


concurrency issue detected with go1.19

I ran into this issue using go version 1.19. Not sure if this is tested but reading and writing to a map can lead to this I believe
fatal error: concurrent map iteration and map write
goroutine 235 [running]:
github.com/itchyny/gojq.normalizeNumbers({0x22ed940?, 0xc0005e4360?})
/Users/henderiw/go/pkg/mod/github.com/itchyny/gojq@v0.12.10/normalize.go:77 +0x105
github.com/itchyny/gojq.normalizeNumbers({0x22ed940?, 0xc0005e4330?})
/Users/henderiw/go/pkg/mod/github.com/itchyny/gojq@v0.12.10/normalize.go:78 +0x6f1
github.com/itchyny/gojq.normalizeNumbers({0x22369c0?, 0xc00053e270?})
/Users/henderiw/go/pkg/mod/github.com/itchyny/gojq@v0.12.10/normalize.go:73 +0x7d4
github.com/itchyny/gojq.(*Code).RunWithContext(0xc00010e500, {0x2773d50?, 0xc000042078}, {0x0, 0x0}, {0xc00054f030, 0x1, 0x1})
/Users/henderiw/go/pkg/mod/github.com/itchyny/gojq@v0.12.10/compiler.go:50 +0x3b1
github.com/itchyny/gojq.(*Code).Run(...)
/Users/henderiw/go/pkg/mod/github.com/itchyny/gojq@v0.12.10/compiler.go:39

Hi, similar issue as in #188?

Right, this looks like the same issue. For argument of Run, do not use values sharing same data between goroutines.


StringUtilsTest Part 1

Following up on a suggestion elsewhere to  write tests for StringUtilities.  I should be able to get coverage to 100% but just felt like taking a break and getting some questions answered about the intent of basicTextWrap.
Draft for now.  Will see how this goes before deciding to do the rest as a new PR or just add to this.

I think we may as well get these tests in incrementally, instead of waiting for the end.
I've added a couple of tests to StringUtilities as part of another bit of work, and you might have conflicts. No harm in getting it in earlier, I think?

OK.  Real Life has been getting in the way.  I have pushed what I have.  We can see whether you merge into this, we merge this and then you open a PR or what.  I can be flexible :-)
A lot of my previous comments have been OBE.  I will review.


[Bug]: barman not being able to cleanup based on retentionPolicy with Minio S3

Is there an existing issue already for this bug?

[X] I have searched for an existing issue, and could not find anything. I believe this is a new bug.

I have read the troubleshooting guide

[X] I have read the troubleshooting guide and I think this is a new bug.

I am running a supported version of CloudNativePG

[X] I have read the troubleshooting guide and I think this is a new bug.

Contact Details
<EMAIL_ADDRESS>Version
1.23.2
What version of Kubernetes are you using?
1.27
What is your Kubernetes environment?
Self-managed: RKE
How did you install the operator?
Helm
What happened?
CNPG/Barman is not able to do cleanup on Minio S3, causing the S3 (self-hosted) to grow in size with no clean up policy. See the log bellow. Unfortunately I was unable to pin-point the cause of the issue.
Currently we are using Minio policy to delete the data, which is not safe if database is down for long period of time, because it is not using a retention window. The Minio policy is deleting objects when they expire after given time.
*We are running bare metal Kubernetes cluster with RKE2.
Cluster resource
apiVersion: postgresql.cnpg.io/v1
kind: Cluster
metadata:
  name: converge43db
  namespace: cybroslabs-c43
  annotations:
    argocd.argoproj.io/sync-wave: "10"
spec:
  imageName: ghcr.io/cloudnative-pg/postgresql:15.4
  instances: 2
  primaryUpdateStrategy: unsupervised
  primaryUpdateMethod: switchover
  superuserSecret:
    name: superuser-credentials
  storage:
    storageClass: longhorn
    size: 20Gi
  resources:
    limits:
      cpu: "1"
      memory: 2Gi
    requests:
      cpu: 500m
      memory: 2Gi
  bootstrap:
    initdb:
      database: converge
      owner: c4core
      secret:
        name: c4core-credentials
      postInitSQL:
        - ALTER ROLE c4core WITH LOGIN;
  ## Postgres configuration ##
  # Enable 'postgres' superuser
  enableSuperuserAccess: true
  # Postgres instance parameters
  postgresql:
    parameters:
      max_connections: "1000"
  # High Availability configuration
  minSyncReplicas: 1
  maxSyncReplicas: 1
  # Enable replication slots for HA in the cluster
  replicationSlots:
    highAvailability:
      enabled: true
  monitoring:
    enablePodMonitor: true
  backup:
    retentionPolicy: "3d"
    barmanObjectStore:
      tags:
        backupRetentionPolicy: "expire"
      historyTags:
        backupRetentionPolicy: "keep"
      destinationPath: "s3://cybroslabs-backups/databases/converge43_v1"
      endpointURL: "http://<IP_ADDRESS>:9000"
      # endpointCA:
      #   name: cybroslabs-converge-backup-ca
      #   key: ca.crt
      wal:
        compression: bzip2
      data:
        compression: bzip2
      s3Credentials:
        accessKeyId:
          name: cybroslabs-converge-backup-credentials
          key: s3AccessKey
        secretAccessKey:
          name: cybroslabs-converge-backup-credentials
          key: s3SecretKey

Relevant log output
{"level":"error","ts":"2024-06-11T01:09:42Z","logger":"barman","msg":"Error invoking barman-cloud-backup-delete","logging_pod":"converge43db-1","options":["--endpoint-url","http://<IP_ADDRESS>:9000","--cloud-provider","aws-s3","--retention-policy","RECOVERY WINDOW OF 3 DAYS","s3://cybroslabs-backups/databases/converge43_v1/","converge43db"],"stdout":"","stderr":"2024-06-11 01:09:42,251 [535399] ERROR: Barman cloud backup delete exception: An error occurred (XMinioInvalidObjectName) when calling the ListObjectsV2 operation: Object name contains unsupported characters.\n","error":"exit status 4","stacktrace":"github.com/cloudnative-pg/cloudnative-pg/pkg/management/log.(*logger).Error\n\tpkg/management/log/log.go:125\ngithub.com/cloudnative-pg/cloudnative-pg/pkg/management/barman.DeleteBackupsByPolicy\n\tpkg/management/barman/backupdelete.go:89\ngithub.com/cloudnative-pg/cloudnative-pg/pkg/management/postgres.(*BackupCommand).backupMaintenance\n\tpkg/management/postgres/backup.go:393\ngithub.com/cloudnative-pg/cloudnative-pg/pkg/management/postgres.(*BackupCommand).run\n\tpkg/management/postgres/backup.go:291"}

Code of Conduct

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

It looks like this is not CNPG problem. From the log output it looks like barman-cloud problem or even file name problem.
I'd check WAL file names or WAL folders on S3 for unwanted files (with unsupported characters).
You could also update postgres image to 15.7 to test latest barman version.

I kinda thought so, I just wanted to be sure. I will report this to barman then.
Unfortunately I can't downgrade Postgres cluster from 16.x to 15.7 (if I understand it correctly, what are you asking me to do).

Are U sure, You're using PGSQL 16.x ? From the manifest You've presented:
imageName: ghcr.io/cloudnative-pg/postgresql:15.4

Sorry, you are right. I maintain a couple CNPG clusters and some are on 15.4 and some on 16.2...
I will upgrade to 15.7 and get back to this issue.

I've upgraded the Postgres to 15.7, no luck, still getting the same error.
Also, I checked with my other databases and I am having the same issue on 16.2.
I will raise this issue with Barman as well, since this is most likely not CNPG fault.

Hello
Closing this sine is more a Barman issue than a CNPG issue
Cheers!

Hello, I've manage to fix the issue. I will post it here for future reference.
There is a difference for the destination path... it can't contain a trailing slash, see:
- destinationPath: "s3://cybroslabs-backups/databases/converge43_v1/"
+ destinationPath: "s3://cybroslabs-backups/databases/converge43_v1"

That is a tiny change and used in many examples in CNPG documentation, but with Minio this does not work.
I tested it with multiple Postgres cluster on different versions (15.7 and 16.2). I suggest adding a note to documentation, since this can be quite troublesome, yet examples in documentation use trailing slash.

@sxd seems it was not a barman issue ;)
My suggestion is to follow what @vojtechmares mentioned regarding adding a note to the docs, or removing the trailing slash, if that works with other S3 compatible buckets


"Import could not be resolved" warning when publishing an icons.figma.tsx file and external icon lib

I've created an icons.figma.tsx file like the docs suggest using the sample script, and everything is working great as far as Figma goes. The generated code is exactly what I want, ie:
import { Button } from '@acme/ui';
import { IconName } from '@acme/icons';

<Button>
  <IconName />
  Button Text
</Button>

Where @acme/ui are all the design system React components, and @acme/icons are icon React components specifically. These two are separate packages that live in two separate repos.
I'm trying to add the Figma files to the source repo of the React UI lib, so component imports are local files, but the icon imports come from the separate icons package (ie. they're in the node_modules).
The generated code is correct. However, I'm seeing this warning in the output when publishing:
Import for IconName could not be resolved, make sure that your `include` globs in `figma.config.json` matches the component source file (in addition to the Code Connect file). If you're using path aliases, make sure to include the same aliases in `figma.config.json` with the `paths` option.

Where IconName is the name of my React component icon, which comes from a separate library:
// icons.figma.tsx
import figma from '@figma/code-connect';
import { IconName} from '@acme/icons';

figma.connect(IconName, ...);

My figma.config.json looks like this:
{
  "codeConnect": {
    "include": ["src/figma/**"],
    "exclude": [],
    "importPaths": {
      "*": "@acme/ui",
      "@acme/icons": "@acme/icons"
    }
  }
}

And the component Figma files (ie. Button.figma.tsx look like this:
// src/figma/Button.figma.tsx
import figma from '@figma/code-connect';

import { Button } from '../components';

figma.connect(Button, ...);

What am I missing?

Got it, thank you @karlpetersson! That makes sense. I will say that the usage of paths vs importPaths isn't very clear in the current docs, so that definitely helps.

Sadly that didn't seem to do anything. Updated the config to this:
{
  "codeConnect": {
    "include": ["src/figma/**"],
    "exclude": [],
    "importPaths": {
      "*": "@acme/ui"
    },
    "paths": {
      "@acme/icons": [
<EMAIL_ADDRESS>      ]
    }
  }
}

Tried both with the index file specified and without (just the @acme/icons folder) but that didn't make a difference.

I am also getting this same error with a similar setup.


The latest TXF standard seems to have additional taxrefs

Specifically, 682 for wash sales isn't handled.
I see the references for code A, B, and C, but I don't see it for D, E, and F. It seems there are codes that can be put into the file, but the create script doesn't handle but A through F. I see references to X and BC in the process script, but no corresponding value in the box_dict.
I'm not sure that there is a direct mapping of Form 8949 column f codes to the codes used for the tax ref. Definitely a 'W' goes into column F for a wash sale and the tax ref is 682.

Do you have an example for a Charles Schwab or TD Ameritrade Form 1099-B that reports a wash sale?  According to the instructions from Schwab, those would show a W in Box 1f and the amount of nondeductible loss in Box 1g.  I don't have an example so that I can't make sure that it will get parsed correctly.  I'll be happy to add support for wash sales once I have a sample.
                                                                         1b-Date
                                                                        acquired                 1d-Proceeds
1a-Description of property                                               1c-Date            6-Reported to IRS:
(Example 100 sh. XYZ Co.)                                                                     Gross Proceeds                            1g-Adjustments
                                                                          sold or               (except where            1e-Cost or                                  Realized      4-Federal Income
CUSIP Number / Symbol                                         **        disposed                                         other basis     1f-Code, if any        Gain or (Loss)          tax withheld
                                                                                indicated)
1S CALL SYNAPTICS INC              $80           EXP 1 BC             10/13/15 $                       300.27    $         730.72                    -- $           (430.45) $                 0.00
SYNA 10/16/2015 80.00 C                                               10/13/15

The references to X and BC refer to activity codes in column **.  BC is "buy to close" and X is "expiration". With respect to populating Form 8949 with the short sale of an option, they are treated the same, the latter with a cost of zero.
While the TXF specification doesn't refer to Boxes D, E, F, it contains the following.
11/30/2011 - added seven new taxrefs to supplement existing taxrefs (321,323,673 and 682) used for reporting sales of securities.
  Sales get reported on Copy A, B or C of the new Form 8949, and then flow to the Sch D.
  These new taxrefs will allow you to indicate which copy of Form 8949 a sale belongs on:
  Form 8949 Copy A : (you repored cost basis for this sale to the IRS using Form 1099B Box 3)
      321 (Short term holding period); 323 (long term holding period); 673 (you don't know the holding period); 682 (wash);

  Form 8949 Copy B : (you provide cost basis to customer, but you do NOT report it to the IRS using Form 1099B Box 3)
      711 (short term, Copy B); 713 (long term, Copy B); 715 (unknown holding period, Copy B); 718 (wash, Copy B)

  Form 8949 Copy C : (no 1099B issued customer or IRS)
      712 (short term, Copy C); 714 (long term, Copy C); 716 (unknown holding period, Copy C)

Boxes D, E, F are in Part II of Form 8949 for transactions involving capital assets you held more than 1 year (long-term). I verified that importing taxref 713 into TurboTax populates Part II of Form 8949 with Box E checked.

I don’t… I have an ETRADE report and was trying to map the codes you were using. There are no codes in the ETRADE report at all, just a separate long/short and a deferred earnings amount. It’s actually even more complicated than that…
I just posted my code in my branch… it isn’t tested right, I ran into problems with TurboTax. I have to write some way to collapse the trades down and the E*TRADE report is a bit funky in its formatting.
Thanks for you code, it helped me tremendously.
..Bill Chin

On Apr 13, 2016, at 11:45 AM, Andreas Girgensohn<EMAIL_ADDRESS>wrote:
Do you have an example for a Charles Schwab or TD Ameritrade Form 1099-B that reports a wash sale? According to the instructions from Schwab, those would show a W in Box 1f and the amount of nondeductible loss in Box 1g. I don't have an example so that I can't make sure that it will get parsed correctly. I'll be happy to add support for wash sales once I have a sample.
                                                                     1b-Date
                                                                    acquired                 1d-Proceeds

1a-Description of property                                               1c-Date            6-Reported to IRS:
(Example 100 sh. XYZ Co.)                                                                     Gross Proceeds                            1g-Adjustments
sold or               (except where            1e-Cost or                                  Realized      4-Federal Income
CUSIP Number / Symbol                                         **        disposed                                         other basis     1f-Code, if any        Gain or (Loss)          tax withheld
indicated)
1S CALL SYNAPTICS INC              $80           EXP 1 BC             10/13/15 $                       300.27    $         730.72                    -- $           (430.45) $                 0.00
SYNA 10/16/2015 80.00 C                                               10/13/15
The references to X and BC refer to activity codes in column **. BC is "buy to close" and X is "expiration". With respect to populating Form 8949 with the short sale of an option, they are treated the same, the latter with a cost of zero.
While the TXF specification https://turbotax.intuit.com/txf/TXF042.jsp doesn't refer to Boxes D, E, F, it contains the following.
11/30/2011 - added seven new taxrefs to supplement existing taxrefs (321,323,673 and 682) used for reporting sales of securities.
Sales get reported on Copy A, B or C of the new Form 8949, and then flow to the Sch D.
These new taxrefs will allow you to indicate which copy of Form 8949 a sale belongs on:
Form 8949 Copy A : (you repored cost basis for this sale to the IRS using Form 1099B Box 3)
321 (Short term holding period); 323 (long term holding period); 673 (you don't know the holding period); 682 (wash);
Form 8949 Copy B : (you provide cost basis to customer, but you do NOT report it to the IRS using Form 1099B Box 3)
711 (short term, Copy B); 713 (long term, Copy B); 715 (unknown holding period, Copy B); 718 (wash, Copy B)
Form 8949 Copy C : (no 1099B issued customer or IRS)
712 (short term, Copy C); 714 (long term, Copy C); 716 (unknown holding period, Copy C)
Boxes D, E, F are in Part II of Form 8949 for transactions involving capital assets you held more than 1 year (long-term). I verified that importing taxref 713 into TurboTax populates Part II of Form 8949 with Box E checked.
—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub https://github.com/andreasg123/stock-gain-tax-import/issues/1#issuecomment-209518447

I added as much support for wash sales as I could without having a sample.  I changed my CSV format to add an extra column for the wash sale code (W).  If that code is present, I use different taxrefs in TXF.  Also, I add the disallowed amount as another field in TXF.

I am looking for the specification for txf files.  It was at https://turbotax.intuit.com/txf but it has moved.  Anyone have a copy?

A year ago, I changed the link in the README to the internet archive: https://web.archive.org/web/20121103033816/https://turbotax.intuit.com/txf/TXF042.jsp


Config Lab Not Supporting Some Commands That Required Restart

This is a situation that is making the solution not as cloud-native as it should be. If I want to run some configuration commands such as changing the User Authentication system to LDAP for example, that requires a restart of the Spotfire Server.
As the currently Job that is doing the config and the pod for spotfire-server is launched at the same time, you are forced to restart the pod for spotfire-server manually to load the new configuration. This is not a very cloud-native approach and we should find a better way to do so.

The intention is that you should not be required to manually restart the servers when making configuration changes. When the spotfire server detects configuration changes it will shut down when using the default Kubernetes configuration file:
set-config-prop --name=lifecycle.changes-monitoring.configuration-change-action --value=GRACEFUL_SHUTDOWN --configuration="${CONFIGURATION_FILE}" --bootstrap-config="${BOOTSTRAP_FILE}"
See
https://github.com/TIBCOSoftware/spotfire-cloud-deployment-kit/blob/main/helm/charts/spotfire-server/config-job-scripts/default-kubernetes-config.txt#L22
The new configuration will be active after a while when the server starts up again. Is this now working for you?

Thanks @mjohanss-tibco I was able to validate what you commented. Spotfire-Server is restarted automatically. Sorry for bring up a topic that was not 100% accurate. Closing the issue.


don't move files out of examples-config ?

I recommend removing this line from the release-management instructions:
mv asset/wis2box-x.y.z/examples/config/synop-bufr-mappings.yml asset/wis2box-x.y.z/examples/config/synop-bin-mappings.yml asset/wis2box-x.y.z/examples/config/synop-synop-mappings.yml asset/wis2box-x.y.z/examples/config/synop-csv-mappings.yml asset/wis2box-x.y.z/examples/config/station_list.csv asset/wis2box-x.y.z

Done/removed.  Will also require docs update (which @maaikelimper is working on).

Done in #470


demo not running

Just trying things out...
v4.1.2.stable.official [399c9dc39]
as per :https://github.com/ashtonmeuser/godot-wasm/wiki/Examples#wasm-visualizations
I installed the go-dot wasm and then opened the wasm-visualizations from https://github.com/ashtonmeuser/godot-wasm/tree/master/examples/wasm-visualizations

`
Godot Engine v4.1.2.stable.official.399c9dc39 - https://godotengine.org
Vulkan API 1.2.231 - Forward+ - Using Vulkan Device #0: Apple - Apple M2 Pro

  Can't open dynamic library: /Users/apple/workspace/go/src/junk/godot-wasm/examples/wasm-visualizations/addons/godot-wasm/bin/macos/libgodot-wasm.dylib. Error: dlopen(/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib, 0x0002): tried: '/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib' (no such file), '/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib' (no such file).
  core/extension/gdextension.cpp:455 - GDExtension dynamic library not found: /Users/apple/workspace/go/src/junk/godot-wasm/examples/wasm-visualizations/addons/godot-wasm/bin/macos/libgodot-wasm.dylib
  Failed loading resource: res://addons/godot-wasm/godot-wasm.gdextension. Make sure resources have been imported by opening the project in the editor at least once.
  Can't open dynamic library: /Users/apple/workspace/go/src/junk/godot-wasm/examples/wasm-visualizations/addons/godot-wasm/bin/macos/libgodot-wasm.dylib. Error: dlopen(/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib, 0x0002): tried: '/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib' (no such file), '/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib' (no such file).
  core/extension/gdextension.cpp:455 - GDExtension dynamic library not found: /Users/apple/workspace/go/src/junk/godot-wasm/examples/wasm-visualizations/addons/godot-wasm/bin/macos/libgodot-wasm.dylib
  Failed loading resource: res://addons/godot-wasm/godot-wasm.gdextension. Make sure resources have been imported by opening the project in the editor at least once.
  Can't open dynamic library: /Users/apple/workspace/go/src/junk/godot-wasm/examples/wasm-visualizations/addons/godot-wasm/bin/macos/libgodot-wasm.dylib. Error: dlopen(/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib, 0x0002): tried: '/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib' (no such file), '/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib' (no such file).
  core/extension/gdextension.cpp:455 - GDExtension dynamic library not found: /Users/apple/workspace/go/src/junk/godot-wasm/examples/wasm-visualizations/addons/godot-wasm/bin/macos/libgodot-wasm.dylib
  Failed loading resource: res://addons/godot-wasm/godot-wasm.gdextension. Make sure resources have been imported by opening the project in the editor at least once.
  Attempting to parent and popup a dialog that already has a parent.
--- Debugging process stopped ---
  Can't open dynamic library: /Users/apple/workspace/go/src/junk/godot-wasm/examples/wasm-visualizations/addons/godot-wasm/bin/macos/libgodot-wasm.dylib. Error: dlopen(/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib, 0x0002): tried: '/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib' (no such file), '/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib' (no such file).
  core/extension/gdextension.cpp:455 - GDExtension dynamic library not found: /Users/apple/workspace/go/src/junk/godot-wasm/examples/wasm-visualizations/addons/godot-wasm/bin/macos/libgodot-wasm.dylib
  Failed loading resource: res://addons/godot-wasm/godot-wasm.gdextension. Make sure resources have been imported by opening the project in the editor at least once.
  Can't open dynamic library: /Users/apple/workspace/go/src/junk/godot-wasm/examples/wasm-visualizations/addons/godot-wasm/bin/macos/libgodot-wasm.dylib. Error: dlopen(/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib, 0x0002): tried: '/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib' (no such file), '/System/Volumes/Preboot/Cryptexes/OS/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib' (no such file), '/Applications/Godot.app/Contents/MacOS/../Frameworks/libgodot-wasm.dylib' (no such file).
  core/extension/gdextension.cpp:455 - GDExtension dynamic library not found: /Users/apple/workspace/go/src/junk/godot-wasm/examples/wasm-visualizations/addons/godot-wasm/bin/macos/libgodot-wasm.dylib
  Failed loading resource: res://addons/godot-wasm/godot-wasm.gdextension. Make sure resources have been imported by opening the project in the editor at least once.
  editor/editor_node.cpp:1225 - Condition "!res.is_valid()" is true. Returning: ERR_CANT_OPEN

Fixed
Have to open the example and THEN install the admin.

Thanks for documenting the fix, @gedw99. I'll add some more context for anyone stumbling on similar issues. Once you've downloaded the repo which includes examples, you'll have to either 1) compile the addon (see Build Godot Wasm) or 2) install the addon e.g. downloading it from the Godot asset library (see Installation).

thanks @ashtonmeuser
Yeah I did not realise that the add on is local to the project in the Godot IDE. So the pattern is load project and then load add on.
I originally thought add-on are global, but they are not in Godot.
If you need help with docs or a beta tester hit me up :)


Get TagLib 2.0.1 working

This pulls together various commits into something reasonably clean that:

uses TagLib 2.0.1
passes our test suite

@robinst what do you think about merging main into taglib-2? That would shrink this PR to about 4-5 commits.

This isn't quite there yet; we need to fix _wrap_picture_type_to_string so that it doesn't return TagLib::String but a Ruby string instead (and vice versa).

This isn't quite there yet; we need to fix _wrap_picture_type_to_string so that it doesn't return TagLib::String but a Ruby string instead (and vice versa).

I think I fixed this now by replacing a lot of #include <taglib/tpicturetype.h> lines with a few targeted %include <taglib/tpicturetype.h>.

Still not sure if I'm handling TagLib::offset_t correctly. TagLib uses an ifdef to define it at compile time (its definition depends on whether we're on WIN32).  But Swig runs before compile time so what do we tell it?
For now I have told Swig to use long long.

@robinst what do you think about merging main into taglib-2? That would shrink this PR to about 4-5 commits.

Sounds good. Also fine with rebasing if that's easier.

@ufleisch I have turned your patch from https://github.com/robinst/taglib-ruby/pull/129#issuecomment-2010558576 into a commit in this PR, I hope that is OK.
@robinst I think this PR is ready for review.

@ufleisch I have turned your patch from #129 (comment) into a commit in this PR, I hope that is OK. ee55a43

I am not really a great help concerning Ruby and SWIG, so I am just trying to comment about what was discussed concerning TagLib:

The rationale behind offset_t is that we need 64-bit integers to work with large files. These offsets were long in TagLib 1.x, which was not a problem on 64-bit Linux because long has 64 bits there. However, on Windows, long is only 32 bits, also on 64-bit architectures, therefore, it was not possible to work with large files on Windows. off_t would be a good solution on POSIX, but it does not exist on Windows, therefore offset_t was introduced. If you use long long instead, this will be correct on 64-bit architectures.

It would be probably nice to support properties and complexProperties with taglib-ruby, these should be enough for most use cases, so users do not have to deal with the format specific classes.

@ufleisch thanks for taking a look!
I didn't mean to complain about offset_t, I am just clueless about SWIG and I was trying to convince myself and @robinst that the current code correctly handles the ifdef. (It's too late now but I wonder if int64_t would have done what you wanted without needing an ifdef?)
Regarding the properties interfaces: I agree it would be good to support them for the reason you state. We never did until now because we wanted to remain compatible with older taglib 1.x versions that didn't have properties. The reason I'm letting SWIG ignore them in this PR is that I don't want to do too many things in one PR.
My main question was if you are OK with the fact that I made a commit with your name in it (by feeding the patch you posted to git am).

My main question was if you are OK with the fact that I made a commit with your name in it (by feeding the patch you posted to git am).

Of course this is OK, thanks for all your work!


"Iterative mode does not support scenarios" error

I try create 'Stateful contracts' (https://cloud.spring.io/spring-cloud-static/spring-cloud-contract/2.2.1.RELEASE/reference/html/project-features.html#contract-stateful-contracts) but the next error appears:
**Failed to execute goal org.springframework.cloud:spring-cloud-contract-maven-plugin:2.1.1.RELEASE:convert (default-convert) on project ...: Execution default-convert of goal org.springframework.cloud:spring-cloud-contract-maven-plugin:2.1.1.RELEASE:convert failed: java.lang.IllegalStateException: Iterative mode does not support scenarios. Use -DContractFileScanner.useRecursive=true**
<spring-cloud-contract.version>2.1.1.RELEASE</spring-cloud-contract.version>
Contracts structure:
producer_name\
  src\
   main\
        resources\
             contracts\
                consumer_name\
                  rest\
                     1_login.groovy
                     2_logout.groovy

You're using spring cloud contract in version 2.2.x which is not supported anymore (check this page for more information https://spring.io/projects/spring-cloud-contract#support ). Please upgrade to the latest version.


basic unit tests on messenger and observer

Set up basic structures of unit testing using jest, however, we should further abstract some of the components.
the docker compose stuff still doesn't work yet:/, only yarn test

LGTM


Got error while reloading and stuck on 97/97

Error:

This line in script

This is not an error with the userscript itself; this is an error with the game and happens without a userscript.

May we reload page again if this happined?

Will be fixed by #140


Replication lag measurement

Nice work @ngourley . Thank you. I will merge it back if you don't mind.


error getting migrations for current schema (clean-interval): could not find current db schema

bee start error:
error getting migrations for current schema (clean-interval): could not find current db schema
how to fix it?

It sounds like you are trying to run the latest master.  0.6.0 is still under development and not ready for the world to try to run yet...

how to down to 0.53 ?


[Bug]

Before Reporting

[X] I found no existing issues matching my bug
[X] My issue is not caused by a theme
[X] My issue is not caused by a plugin

Describe the Bug

Expected Behavior
еще при загрузке темы вечная загрузка и не устанавливается тема
Steps To Reproduce

Operating System
Windows
Anything else?
No response

fixed!


Getting Error at time Checkout

Please provide answers to all the required item options
(After Complete All Fields.Why this is coming)

This message is given when an item has options and those options are not provided.
What merchant id are you working with?
What item are you attempting to checkout with that is causing this error?


fix(ci): make it work again

Thank you so much for all the PRs, every time. I've been meaning to fix this but couldn't dive in properly due to my time constraints. I really appreciate you taking the time to fix this issue!


[BEAM-12898] Disable Flink Load tests which are leading Dataproc clus…

…ters/GCE VMs
Copy of #15519 for testing.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] Update CHANGES.md with noteworthy changes.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

run seed job


Check behavior of QC report generation with gaps in value range

The value range column in the DQO accuracy file indicates a range of measurements within which DQOs apply for each parameter.  It was assumed that users would supply value ranges that cover all possible outcomes for a parameter.  This may not be true and the behavior of the QC functions with missing value ranges is unknown.  This needs to be tested.

The behavior of the QC functions is good for tabMWRacc(), but not for tabMWRfre().  If there is a missing value range, then it is best to not report any QC checks for the missing range (as if there were an "na" in the corresponding range).  Then the user will not report better results than reality.  This works correctly for tabMWRacc(), which does not include records in the missing range in Number of QC Checks or Number of Misses.  It correctly does the same for a missing range or "na".  However, tabMWRfre() misleadingly includes these records in Number of Dups/Blanks/Spikes.  It should return the same number as the Number of QC Checks in tabMWRacc().

No, the completeness table is unaffected by the DQO objectives.  Qualified records are only defined by the Qualification flag, and in both Completeness and Frequency the Number of Data Records should be the total records regardless of DQO objectives.

Closing this since the warning will be addressed in #58

@ben-wetherill this should be addressed now.  The tabMWRfre() function now requires the DQO accuracy file as input and removes any entries in the results file that are outside of the value ranges.  It seems to be working on my end, but you should test it as well.

Getting closer.  The tabMWRfre() Number of Dups/Blanks/Spikes is working as I would expect.  Number of D/B/S should equal Number of QC Checks in tabMWRacc(), and it does.  However, in tabMWRfre() the Number of Data Records should not be affected by the DQO Accuracy file.  Number of Data Records should be complete regardless of whether a DQO is established.  Number of Data Records should match between tabMWRfre() and tabMWRcom().  In the attached files, the number of data records for TP should be 48, but tabMWRfre() shows 39.
Sample_DQOAccuracy_testing.xlsx
Sample_Results_7-9-23.xlsx

@ben-wetherill ok, makes sense. Should be fixed now.

The Number of Data Records looks accurate now, but I have run into some other issues:

In the attached files, tabMWRfre() Number of Dups/Blanks/Spikes does not match tabMWRacc() Number of QC Checks for TP Field Duplicates.  tabMWRacc() is correct (should be 4), because it is evaluating the average of the result and reference value (0.04 and 0.02) instead of just the result value.  That's what we agreed a while ago, right?
Why is the parameter order different in tabMWRfre() and tabMWRacc()?  It should always be alphabetical, right?  tabMWRfre() doesn't seem to be following that.
Field Blanks are not being handled correctly in tabMWRfre().  The attached files should return 0 Dups/Blanks/Spikes or QC Checks for everything.  tabMWRacc() is reporting correctly.  The same issue happens for all of the other QC types if I adjust the DQO Accuracy file accordingly.

Sample_DQOAccuracy_testing.xlsx
Sample_Results_7-9-23.xlsx
Sample_Sites_5-19-23.xlsx
Sample_WQXMeta_5-19-23.xlsx
Sample_DQOFreqComp_5-19-23.xlsx

@ben-wetherill, these should be fixed now.  Please verify when able.
For number 2, this was again caused by an update to dplyr that changed the sorting behavior that handles upper/lower case words differently (presumably to improve performance, https://dplyr.tidyverse.org/news/index.html#breaking-changes-1-1-0).  This should be fixed now.

This looks good now for gaps in the value range.


Possibility to add OpenNIC Public Servers

Hello,
thank you for your nice program. I'd like to have the possibility to add OpenNIC Public Servers, but I can't find the the server-URL. Do you know, if there is the possibility to find them or would it be possible to add an option in Intra to directly set a server with IPv4 IPv6 parameters?

The DNS Privacy Test servers list shows some resolvers that support both DoH and OpenNIC.  You can get the configuration URL here and enter it in the settings.


☂ Adapt to new secrets manager [GEP-18]

In Gardener v1.45, the GEP-18 has been implemented for the extension-controllers.
The idea is that users can rotate all secrets and certificates managed by Gardener for their control plane through shoot annotations in two phases (start and complete).
The umbrella issue from Gardener to implement this functionality can be found here: https://github.com/gardener/gardener/issues/3292
These are the parts where we have to adapt to the new secrets manager and drop the static kubeconfig generation from earlier Gardener versions:

[x] Firewall

[x] Implement new firewall-controller-manager
[x] Adapt firewall-controller to new controller manager (https://github.com/metal-stack/firewall-controller/pull/131)

[ ] ~Build package that can generate independent certificates for trusted controller communication (https://github.com/metal-stack/metal-lib/pull/64)~
[x] Auditing

[x] Audit Forwarder (https://github.com/metal-stack/gardener-extension-provider-metal/pull/303)

[x] Make communication between forwarder and apiserver work through static-token mounts (https://github.com/metal-stack/gardener-extension-provider-metal/pull/303)

[x] Audit Tailer

[x] Establish trusted connection, use secretsmanager (https://github.com/metal-stack/gardener-extension-provider-metal/pull/304)

[x] Auth Webhook (https://github.com/fi-ts/gardener-extension-authn)

[ ] Establish trusted connection through our cert package (see above)
[x] Try no certificate as it's only control plane traffic inside shoot namespace

[x] droptailer

[x] Establish trusted connection, use secretsmanager (https://github.com/metal-stack/gardener-extension-provider-metal/pull/304)

[ ] ~Integration into #274~
[x] duros-controller (https://github.com/metal-stack/gardener-extension-provider-metal/pull/300)

no problem with normal Gardener approach

[x] metal-ccm (https://github.com/metal-stack/gardener-extension-provider-metal/pull/298)

no problem with normal Gardener approach

[x] group-rolebinding-controller

no problem with normal Gardener approach

[x] machine-controller-manager (https://github.com/metal-stack/gardener-extension-provider-metal/pull/302)

no problem with normal Gardener approach

[x] accounting-exporter (https://github.com/metal-stack/gardener-extension-provider-metal/pull/297)

no problem with normal Gardener approach

Firewall Implementation Details

Give the firewall-controller two kubeconfigs (similar to duros-controller design approach), one for the control plane and one for the shoot
Create an own controller firewall-controller-manager (move the entire firewall reconcile into this controller) and deploy in similar fashion to e.g. duros-controller
Solves the problem of the signature when API changes
Solves the problem of users being able to mutate the firewall resource (because they are cluster-admins)
Must implement token refresh of kubeconfig, should implement ssh key rotation

Outlook
We are aiming to turn some integrations into gardener extensions in the long run:

[ ] gardener-extension-audit (can maybe be donated to community)
[x] gardener-extension-fits-auth
[x] gardener-extension-fits-accounting

Here is the original planning document: https://hackmd.io/@apQaWHASTpeSi3CE79hWWA/r1-Ae1YUs

I think this is done by now.


Theming with CSS3 vars

Great project! I Would you be open to receive a pull request for an idea I had, to use CSS3 vars to theme tweakpane? I often want to adapt the panes in by projects.

Before implementing the feature, I think we need to categorize component parts, organize them, and figure out a minimum required color palette.
I also think it would be great if users can create a color theme or apply the existing ones (e.g. Solarized, Gruvbox, Monokai, Onedark, Iceberg, ... etc.) to the pane easily.
Based on these, I made a proposal of themable color palette:

Base background

Pane
Picker

Color

Point2D

Folder foreground

Folder title
Folder screw

Folder background (active, focus, hover)

Field label

Input view foreground

Text input
Checkbox mark

Point2D

Input view guide foreround

Point2D axis

Input view background (active, focus, hover)

Text input
Checkbox box

Point2D pad

Slider track

Monitor view foreground

Monitor

Text

Graph

Monitor area background

Monitor

Text

Graph

Button-like control background (active, focus, hover)

Button
Slider knob

List

Button-like control foreground (active, focus, hover)

Button title

List title
List arrow

Separator

--

$color-base-bg
$color-folder-fg
$color-folder-bg
$color-folder-bg-active
$color-folder-bg-focus
$color-folder-bg-hover
$color-input-bg
$color-input-bg-active
$color-input-bg-focus
$color-input-bg-hover
$color-input-fg
$color-input-guide-fg
$color-monitor-fg
$color-monitor-bg
$color-button-bg
$color-button-bg-active
$color-button-bg-focus
$color-button-bg-hover
$color-separator

Released in 1.3.0.

Is it possible to make one of the labels a different color? For example - in red to emphasize that the data here is incorrect. I didn't find that...

I am wondering the same thing as above

Added element property to blades in #392 and you'll be able to modify elements from the next version.


Define some basic naming conventions.

Currently we have names like first and last but also have names like getLeft and getHeight. I'm not sure which convention I'd like to use just yet, but having both rubs me the wrong way.

Why not follow the convention of the SPL here? Once it's not working, you can start with prefixes (get, is, has), most likely for reserved keywords (IIRC that's the first case of "not working"). Because once you start with prefixes you can't stop. But getters is more for field-access and here are mostly interfaces or abstract classes. Whether or not it will represent a field is an implementation detail and most likely not necessary to know for the interface.
And the counter-argument:
getters signal state. data-structures normally represent state. sure those are bulky and we do the mistakes of Java however in SPL we have current(), key(), count() etc. .
I know that w/o get prefix you run into problems with reserved names more quickly in PHP. So more often in the last times when I needed to decide on that and if I'm lazy I decide for the getters. It's nothing I'm proud of but it's working quite well. It's also with is and has prefixes.
Perhaps keep the "core" methods from Iterator and Countable and for the own use getter-style.
Conclusion:
I personally prefer the non-getter style, but I know I run into problems with that time-to-time and getters is often accepted but also the lazy decision. Type less, do more, so if you can keep getters out (as long as possible), the interface is less bulky and more precise. It could also allow concrete classes implementing an interface to add getters as they wish. So yes, I'm leaning towards to not introduce get prefixes II think.
In the spirit of good code, keep waste out as much as possible.

I'd keep SPL interfaces' methods aliases of the properly-named ones. SPL is not very consistent.
Also, "has" and "is" aren't good prefix either.
getIsEmpty()
setIsEmpty($bool)
i know this doesn't exist, but explains why one should have get/set also for booleans
you can't take java as example because java has method overloading, while php doesn't - and really hope it will never do
prefixes should be get(getHas*, getIs*) add, set, remove, replace(or swap, it's shorter)
getHasItem($item)
getHasItemsAll(Set $items)
getHasItemsNone(Set $items)
getIsEmpty()

suffixes should be All, None, or maybe
getHasAllOfItems(Set $items)
getHasNoneOfItems(Set $items)

no prefix at all would be confusing. what does ->empty() do? does it clear the collection or tells if the collection is empty?
HTH

It's also worth mentioning that the problem with reserved words in classes is being limited in PHP7 (not removed entirely, as some words like public, protected and private have to be reserved, but fewer words will cause conflicts).  See [this RFC] (https://wiki.php.net/rfc/context_sensitive_lexer).

I am still not sure what to do here. Leaving open.

I'd like to brieflyThis post has a tl;dr enumerate some of my thoughts on this topic...:

Other languages tend to use single-word methods for things such as height, size, peeking, pushing, popping...:

Queues: C#, Java, C++.
Stacks: C#, Java, C++.

Method names should be as succinct & intuitive as possible. Ambiguous names should be changed.
Getters (get*) should just retrieve the corresponding instance variable (getX should return $this->x). If the operation is more complex, or has nuances not-sufficiently described by just "get*", then the name could probably be better. Additionally, as @hakre said, we're dealing with interfaces/abstractions -- whether or not a method will represent an instance variable is an implementation detail.

For what it's worth, I would suggest:

Queue::first → Queue::peek: peek is used across multiple languages and is clearer than both first and getFirst.
Stack::last → Stack::peek: again, peek is a widely-used term for safely viewing the next item out of a one-way structure; it is clearer than both last and getLast.
BinaryTree::left → BinaryTree::getLeft: whilst both are very clear, I would opt for getLeft because...:

It is a valid getter (it retrieves the instance variable $this->left).
Object methods should read as queries or commands, and it should be clear which are which:

"Does this BTree have a left node?" → $tree->hasLeft().
"Add the number 5 to this AVLTree!" → $tree->add(5).
"Give me the left node of this tree!" → $tree->getLeft().

I believe it implies O(1) complexity (which is the case).

BinaryTree::getHeight → BinaryTree::height: whilst the intent of both is fairly clear, getHeight suggests to me a simple operation, when, really, height calculation can be expensive.

With regards to some of the other issues addressed:

Reserved words: As mentioned by @darkphoenixff4, this will become less of an issue.
Mixing get* with one-worders: Curiously, no reasons have been provided for as to why this is bad. I believe the main goal in choosing an identifier is to improve readability as much as possible, and -- while consistency helps (f.e., using mostly get* or one-worders) -- it does not always guarantee the best readability.

Hope these ramblings help.
TL;DR: I suggest not to use get* unless:

It's an expected O(1) operation and you can't think of a better name, or...
It refers to an instance variable.

I can't do Queue::peek and Stack::peek because then you can't implement Queue and Stack in the same structure (like a Deque).

I don't think queues and stacks should be implemented within a single structure because they have kinda conflicting definitions: Queue::enqueue & Stack::push (or Queue::dequeue & Stack::pop, depending on which way you look at it) do exactly the same thing (try merging these two diagrams: 1 2); unless you're thinking that enqueue and dequeue both function on the same end of the structure, but then you get stuff like this:
$struct->enqueue("First in");
$struct->enqueue("Last in");
echo $struct->dequeue(); // outputs: "Last in"

If you wish to merge them "nicely", you'd be forced to consider @WesNetmo's solution; however, I'm not too fond of this because the Stack, as a concept, doesn't really have a front/back/first/last.
With regards again to peek, if you should adopt this, I would suggest refactoring it to its own interface (since the behaviour applies to multiple structures).


German translation

Changes to localization references in XAML files:

MainWindow: Moved all localization templates which were in the "Header" attribute of a MenuItem to a separate "MenuItem.Header" element. Otherwise the gettext-plugin won't create an entry in the localization .pot-file. (Well, this was the case for me, I hope this wasn't just a configuration issue)

MetroMessageBoxWindow: Moved the declaration of the button's content-text (Yes, No, Ok, Cancel) to a nested l:Catalog so that an entry in the .pot-file can be created.

DownloadStashWindow and CraftWindow: Moved the window title to a l:Catalog in an "MetroWindow.Title" element so that an entry in the .pot-file can be created.

The layout of the settings menu is now a grid with two columns. The width of the first column is set to "auto" to prevent issues with long localized labels.

Added german translation

Thanks for the translation!


⚠ Bofrost.de has degraded performance

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

HTTP code: 200
Response time: 1899 ms

Resolved: Bofrost.de performance has improved in 9bb9eff after 9 minutes.


Jhonny/pro 245 update sdks to retry loading ca on

Why
Validating CA expire date
How
every http request validates if it has to update the certificate.

Closing in favour of #82 to get the desired bump from semantic release


Incompatibility regarding Mariums Soulslike Weaponry in conjunction with other mods

Mod Name
Mariums Soulslike Weaponry/Iron's Spells 'n Spellbooks
Mod Homepage
https://www.curseforge.com/minecraft/mc-mods/irons-spells-n-spellbooks /  https://www.curseforge.com/minecraft/mc-mods/mariums-soulslike-weaponry
Describe the bug
Using the forementioned mods together, which both depend on GeckoLib, triggers a crash while loading. I think there was a similar issue, which had been fixed before.
Steps to reproduce

Download both Mods and their dependencies
Launch the game
Crash

Logs
https://gist.github.com/tharrmeehan/23506f0ec2c19c03313e0d8befba243b
Additional context
No response

The error is caused by Geckolib Forge being initialized earlier than it expects by one of the Fabric mods. Can you try using the Fabric version of Geckolib instead?

Using Geckolib fabric causes a similar error.
https://gist.github.com/tharrmeehan/33155cd106c7bd269169ec195fc3adee
I have to add that isolating the mods does make it work (with iron spells for example), but it seems that a specific mod (which i am trying to find out which one) is causing this crash to happen without being mentioned in the logs.

I did actually update to the latest Connector and Fabric API, but to no avail. I'm not getting smarter by skimming through the logs either, but I will try to remove mods one by one and see if that solves the problem.

and the game crashes due to incompatibility with a iron's spells.
https://gist.github.com/tharrmeehan/f233a1b2c4a54355fa9c64cf88232505

Sounds like an incompatibility between Forge and Fabric geckolib. Unfortunately, there isn't much that can be done about it as the two versions contain different code. And two mods with identical mod IDs can't be run together.

Alright, thanks for the reply. I'm closing this issue.


how to hear the callback onComplete in vue?

how to hear the callback onComplete in vue? would it be with watch? I need when I click the done button, it shows me that it is complete, but where is it in the vue? I'm kind of lost.

Resolved


FIX: apt-get update + install libssl-dev for kernel build

Few "fixes" for build script.

Thanks!


Split mappings to storage and network modules

Split mappings to storage and network modules
Issue:
StorageMaps and NetworkMaps are two different modules sharing code, they are now in one directory.
Fix:
Split the module specific code to two different directories

@yaacov
the idea makes sense to me. One comment: we need a place to share code between modules. utils and @kubev2v/common  seems  not appropriate. Do we need src/shared/Mapping directory?

utils and @kubev2v/common seems not appropriate. Do we need src/shared/Mapping directory?

I agree, does src/components/Mappings makes sense ?
@sgratch @sgratch @rszwajko what do you prefere ?
a. src/shared/{subject}/components
b. src/shared/components/{subject}
c src/components/{subject}
d. ?

@yaacov

c. src/components/{subject}

let's keep it simple:)

@rszwajko @sgratch please re-review


CrossPost - Cemu emulator issue with "Corrupt pipeline cache when when booting game a 2nd time"

https://github.com/cemu-project/Cemu/issues/396

[mvk-error] VK_ERROR_INITIALIZATION_FAILED: Render pipeline compile failed (Error code 3):
output of type float4 is not compatible with a MTLPixelFormatRGBA16Uint color attachement..
-[MTLVertexDescriptorInternal newSerializedDescriptor]:726: failed assertion `Serialized Descriptor Creation
None of the attributes set bufferIndex to 29, but MTLVertexDescriptor set buffer layout[29].stride(16).
'

[mvk-error] VK_ERROR_INITIALIZATION_FAILED: Render pipeline compile failed (Error code 3):
output of type float4 is not compatible with a MTLPixelFormatRGBA16Uint color attachement..

What happens when you run your emulator with the Vulkan validation layer enabled? (You don't even have to do this on a Mac; you can use Linux or Windows for this as well.)

@cdavis5e happy to test this out for you and provide the output.. but can anyone first tell me how do I run Cemu with the Vulkan validation enabled please? I don't see that as an option in the UI so I guess I have to run the command with a special switch first?

@vesper8

Appears that option is greyed out on Mac

Unsure if the message has changed but this is the output as of the latest experimental release

[mvk-info] MoltenVK version 1.2.0, supporting Vulkan version 1.2.231.

Error: signal 6:
0   Cemu                                0x000000010525d9dd _Z20handlerDumpingSignali + 93
1   libsystem_platform.dylib            0x00007ff81a6a1c1d _sigtramp + 29
2   ???                                 0x0000000000000000 0x0 + 0
3   libsystem_c.dylib                   0x00007ff81a5c0ca5 abort + 123
4   libc++abi.dylib                     0x00007ff81a631082 abort_message + 241
5   libc++abi.dylib                     0x00007ff81a622225 _ZL28demangling_terminate_handlerv + 242
6   libobjc.A.dylib                     0x00007ff81a321258 _ZL15_objc_terminatev + 104
7   libc++abi.dylib                     0x00007ff81a6304a5 _ZSt11__terminatePFvvE + 8
8   libc++abi.dylib                     0x00007ff81a632d05 __cxa_get_exception_ptr + 0
9   libc++abi.dylib                     0x00007ff81a632ccc _ZN10__cxxabiv1L22exception_cleanup_funcE19_Unwind_Reason_CodeP17_Unwind_Exception + 0
10  Cemu                                0x0000000104fa2f96 _ZN14VulkanRenderer10InitializeEv + 1814
11  Cemu                                0x0000000104f072ab _Z17Latte_ThreadEntryv + 443
12  Cemu                                0x0000000104e729ec _ZNSt3__114__thread_proxyINS_5tupleIJNS_10unique_ptrINS_15__thread_structENS_14default_deleteIS3_EEEEPFivEEEEEEPvSA_ + 44
13  libsystem_pthread.dylib             0x00007ff81a677259 _pthread_start + 125
14  libsystem_pthread.dylib             0x00007ff81a672c7b thread_start + 15

The generated shaders are using too many samplers--on Metal without argument buffers, you're limited to 16 samplers. Try enabling argument buffers--the limit on samplers is lifted in this case. (The easiest way to do this is to set MVK_CONFIG_USE_METAL_ARGUMENT_BUFFERS=1 in the program environment.)
@nastys's comment suggests it might be because the stack is too small. If you're doing this from a thread other than the main thread, increasing the stack size in the call to pthread_attr_setstacksize() might help.

I thought the issue was caused by this .
Some vertex stride in Breath of the Wild are equal to 1 but metal requires multiples of 4.

Setting  MVK_CONFIG_USE_METAL_ARGUMENT_BUFFERS=1 seems to work fine.
Without: [mvk-error] VK_ERROR_INITIALIZATION_FAILED: Shader library compile failed (Error code 3):
program_source:261:1888: error: 'sampler' attribute parameter is out of bounds: must be between 0 and 15
With: [mvk-error] VK_ERROR_INITIALIZATION_FAILED: Render pipeline compile failed (Error code 3):
output of type float4 is not compatible with a MTLPixelFormatRGBA16Uint color attachement..
and it loads ok.

How can I set this Environment Variable when using the Cemu App on Mac?

How can I set this Environment Variable when using the Cemu App on Mac?

You'll need to start it from Terminal. You can set it when you run open(1) to open the app:
env MVK_CONFIG_USE_METAL_ARGUMENT_BUFFERS=1 open /path/to/Cemu.app

you can also build the moltenvk on github by yourselves by using make macos MVK_CONFIG_USE_METAL_ARGUMENT_BUFFERS=1 and then change the dylib in the cemu.app

Using this method to start, the game membership card cannot enter the file loading page

在 Mac 上使用 Cemu 应用程序时如何设置此环境变量？

您需要从终端启动它。您可以在运行open(1)打开应用程序时设置它：
env MVK_CONFIG_USE_METAL_ARGUMENT_BUFFERS=1 open /path/to/Cemu.app

Using this method to start, the game membership card cannot enter the file loading page

Any new ideas here?

try with 2.0-36


Fix isDateInRange

When the range is selected first day of next month doesn't get .calendar_days_in-range class

I still see the exact same issue appearing. Any fix coming soon?

can anyone please merge @AbdelkaderBah  fix to master?


Debug this plugin

I want to debug the plug-in itself and press F5 to start debugging. However, when setting breakpoints for code-debug, only the breakpoints in extension.ts are active, while the breakpoints in other files are gray and cannot be paused.
Another debugging method is to use console. log () for output. Only the console. log function in extension. ts has output.
Why is this and what are the solutions?

Have you had a look at https://github.com/WebFreak001/code-debug/blob/master/HACKING.md#runningdebugging? Please outline where it is misleading or misses information.

Sorry, I may not have made it clear. We wanted to make some changes to code-debug, so we wanted to debug code-debug, but some breakpoints did not work. As shown in the figure below, the breakpoints in mibase are gray. As can be seen from the code, the handleBreakpoint function in mibase.ts has been executed, but it has not been paused.

As far as I see it you want to debug both the frontend and backend, and the docs I've linked explicit writes how to do that (you need to run two launch configurations at once.


Update app-governance-visibility-insights-overview.md

@WendyLiuC : Thanks for your contribution! The author(s) have been notified to review your proposed change.

Hi @WendyLiuC - This pull request was opened in the public repo. PMs should work in the private repo, per the Microsoft Docs contributor guide. Would you make this content update in the private repo instead? Thank you! We'll close this PR.

#label:"aq-pr-triaged"


Create several example applications

I'd like to demonstrate several ways to use Cassette. What I have in mind at the moment is:

[ ] A playlist-oriented music library application (like Spotify or iTunes)
[ ] A playlist-oriented video library application (like Netflix or YouTube)
[ ] A social media ish page with multiple concurrent videos
[ ] A live media streaming application (like Twitch or iHeartRadio)

Anyone who wants to help me with this is more than welcome! :smiley:

For media content we can use this code to get JSON search data from archive.org:
function getArchiveSearchUrl(mediatype, { search, fields, page, pageSize }) {
  return (
	`https://archive.org/advancedsearch.php?q=` +
    `${search}+AND+mediatype:${mediatype}&fl[]=${fields}` +
    `&rows=${pageSize}&page=${page}&output=json`
  );
}

const search = 'Computer games';
const fields = '*'; // (can be like 'name,genre')
const page = 1;
const pageSize = 15;

// search for video
fetch(getArchiveSearchUrl('movies', { search, fields, page, pageSize }))
  .then(res => res.json()).then(res => console.log(res.response.docs));

// search for audio
fetch(getArchiveSearchUrl('audio', { search, fields, page, pageSize }))
  .then(res => res.json()).then(res => console.log(res.response.docs));

Unfortunately that API doesn't yet have adequate CORS headers for access from a browser so we may need to run a proxy server.


RestfulAttribute case string exception

https://github.com/apereo/cas/blob/3c23af44fcb69e4918fcea8da8f86738acd75702/core/cas-server-core-util/src/main/java/org/apereo/cas/util/HttpUtils.java#L81

service.getId() is long type.
There will be case string exception, there require string type.
It is require string before append params to url.

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.You have signed the CLA already but the status is still pending? Let us recheck it.

Coverage increased (+0.02%) to 45.646% when pulling 3d4f7115b9ac52e81e25287da6c2c36406185a2b on kawhii:5.2.x into 3c23af44fcb69e4918fcea8da8f86738acd75702 on apereo:5.2.x.

Coverage increased (+0.006%) to 45.63% when pulling f30320bae176da83fb01842ff5180f2de906f863 on kawhii:5.2.x into 3c23af44fcb69e4918fcea8da8f86738acd75702 on apereo:5.2.x.


Make rivet.game url dynamic

Fixes CLI-29

Current dependencies on/for this PR:

main

PR #165   👈

This stack of pull requests is managed by Graphite.

Merge activity

Dec 24, 4:15 AM: @@NathanFlurry started a stack merge that includes this pull request via Graphite.


Snippets for modes that don't exist are created

some snippets eg the ones added by this commit
are placed in folders for modes like javascript-d3-mode which don't really exist.
I can take a look at this over the weekend.

Maybe we could "merge" these snippet definitions into the js-mode snippets (or whichever mode that have this issue too), or generate a .yas-setup.el file that activates these extra modes when the major mode is activated, for example when js-mode is activated, it could activate javascript-d3-mode.
What do you think?


Can't start my app!!! djcelery import error

By mistake I've upgraded djcelery, and now I have this import issue that make our production app to be stopped!!! Please help!!!
import djcelery
Traceback (most recent call last):
File "", line 1, in
File "/home/ubuntu/www/local/lib/python2.7/site-packages/djcelery/init.py", line 25, in
from celery import current_app as celery  # noqa
File "/home/ubuntu/www/local/lib/python2.7/site-packages/celery/compat.py", line 141, in getattr
module = import(self._object_origins[name], None, None, [name])
File "/home/ubuntu/www/local/lib/python2.7/site-packages/celery/_state.py", line 20, in
from celery.utils.threads import LocalStack
File "/home/ubuntu/www/local/lib/python2.7/site-packages/celery/utils/init.py", line 28, in
from .functional import noop
File "/home/ubuntu/www/local/lib/python2.7/site-packages/celery/utils/functional.py", line 19, in
from kombu.utils.compat import OrderedDict, next
ImportError: cannot import name next

Continued with @karamanolev comment https://github.com/celery/django-celery/issues/276#issuecomment-52148622
To list versions of kombu
pip install kombu==

ref. http://stackoverflow.com/a/26664162/248616


Select correct parent node after orch. template removal

The UI should select either the "CloudFormation Templates" or
"Heat Templates" node after orch. template removal.

@chessbyte Fixed!

Verified fix.  @mzazrivec Please rebase, then can be merged.

@dclarizio Rebased & modified the logic slightly.


Use npm for build on windows

Bun still isn't fully supported on windows so build.fsx is failing on windows right now. This updates to use npm if the OS is windows.

This probably resolves itself next Tuesday, https://x.com/bunjavascript/status/1717497691893285232?s=20
Also: https://x.com/JuavaScript/status/1701702174810747346?s=20

bun


UI for editing YouTube content

@nikooo777 how feasible would it be to allow @robvsmith and other growth team members to edit YouTube publishes and channels? Ideally via the same UI in app.

I discussed this briefly with Grin today. If I understand it right, what we want to have is a way to add tags or other properties and mark a video as "dirty" so that ytsync picks it up and "updates" it.
It would take changes in both ytsync and internal-apis as right now channels are processed by channel changes, not video changes (as in, the channel is market as "dirty" and ytsync goes through the whole channel).
I don't think having it done through the app would make much sense as we'd technically want to have the claims visible in the app and thus be able to edit the claim straight away from there, not through ytsync.
However an additional entry in api forms to handle tags (and not sure what else) could be doable.


Unsubscribe/subscribe performance

Hello
Got more of a question, rather than an issue report, but I feel that it is unsuitable for StackOverflow or other places as it is really eventemitter3 specific. And also it would be perfect to hear some thoughts from people who actively develop this library.
Basically my issue is that for some use cases I might be triggering from 100 to 1000 unsubscribe/subscribe cycles per second.
Is that an issue for the library?
I could of course measure this, but my question is kinda more from a design and core ideas perspective.
If it sounds like smth that won't work well (as it was never designed to handle such workloads) then I will look into optimizing my approach.
Thank you in advance.

You should measure, but 100-1000 subs/second should not be a problem.  If N is very high, there may be better approaches - the collection of callables is an Array (see addListener), not some kind of Set/Map, so removeListener() will be linear in the number of listeners per event instead of O(1).

Yep, exactly what bothered me - callbacks stored in array.
For my purposes I decided to store ref enclosing frequently changing dependency of a listener and only subscribe once.
But anyways, thanks for your answer :)


msal:loginSucces not broadcast unless handleRedirectCallback() is called

Library

<EMAIL_ADDRESS><EMAIL_ADDRESS>
Framework

Angular 9.1.11

Description
The BroadcastService doesn't trigger an msal:loginSuccess event after successful login via loginRedirect() unless this.msal.handleRedirectCallback() has been registered.
Error Message
N/A
Security

Not security related

Regression

Unknown

MSAL Configuration
function MSALConfigFactory(): Configuration {
  return {
    auth: {
      clientId: environment.msal.clientId,
      authority: 'https://login.microsoftonline.com/common/',
      validateAuthority: true,
      redirectUri: environment.msal.redirectUri,
      postLogoutRedirectUri: environment.msal.postLogoutRedirectUri,
      navigateToLoginRequestUrl: true,
    },
    cache: {
      cacheLocation: 'localStorage',
      storeAuthStateInCookie: false,
    }
  };
}

function MSALAngularConfigFactory(): MsalAngularConfiguration {
  return {
    popUp: false,
    consentScopes: environment.msal.consentScopes,
    protectedResourceMap: environment.msal.protectedResourceMap as [string, string[]][]
  };
}

Reproduction steps
Here I'd expect the BroadcastService to push a msal:loginSuccess event after a successful login completes.
ngOnInit() {
  this.broadcast.subscribe('msal:loginSuccess', () => {
    console.log('msal:loginSuccess');
  });
}

login(): void {
  this.msal.loginRedirect();
}

However, the event isn't pushed unless I also register handleRedirectCallback(), then it works fine.
ngOnInit() {
  this.msal.handleRedirectCallback(() => {});
  this.broadcast.subscribe('msal:loginSuccess', () => {
    console.log('msal:loginSuccess');
  });
}

Expected behavior
This is ok, but the 1.0.0 changelog is perhaps misleading

Applications using Redirect methods can optionally implement the handleRedirectCallback method

so ideally the handleRedirectCallback() step wouldn't be necessary to enable msal:loginSuccess for loginRedirect(), or the docs could highlight the prerequisite.
Browsers/Environment

Chrome

Thanks for the heads up here, I was able to reproduce this. Moving over to be fixed. For now, just implementing a simple handleRedirectCallback or using loginPopup can serve as a workaround, as you mentioned.

"ClientAuthError: Invalid state" unable to get catch this error in 'handleRedirectCallback' in angular 9 . Please assist me, our project must support password reset feature also.


Feature: RC Immix

paper (PDF)
paper code (java)
this paper's packed full of goodies. in particular,, RC-Immix and RC-Immix (No Proactive Copying) may be of interest, but i also saw your ticket about a conservative collector, so you might enjoy RC-Immix(cons) too.
I'm tempted to implement one or two of these for my toy lisp. If I get around to it, would you be interested in a PR?

@jjl hello! Comet is not maintained now. But I would like to see your implementation of RCImmix in Rust. :)


Captions API Refactoring

This PR makes some slight changes around how captions are stopped as well as being more consistent with captionsId and not captionId

@manchuck, the sample has been updated that allows the multi-session approach for the captions. This requires the client to pass the session ID in the request, but that should not be an issue. please re-review this PR when you have a chance, thanks 🙏


Create guidance on deploying to AWS

Note: this is blocked on issues #62 and #63
Overview
Amazon Web Services (AWS) offers a suite of services that can be used to host and manage a OneBusAway (OBA) deployment. Now that we have containerized the OBA, we need to create documentation (and any necessary accompanying code changes) that will allow a reasonably technical IT worker to set up and deploy OBA on AWS.
Outcome and Deliverables
The end result of this should be a guide written in Markdown that a reasonably technical person, but not necessarily a software developer (think senior IT at a transit agency) can use to deploy OneBusAway in a scalable, secure way on AWS. Despite the fact that this task appears in the onebusaway-docker issue tracker, the end product should be added to https://github.com/oneBusAway/onebusaway-docs
Look at the guides published on https://developer.onebusaway.org, like this Quickstart guide for a sense of tone, length, etc.
Thoughts about the guide

The guide should start at the point of creating an AWS account. It doesn't need to lay out how to do this, but it should include a link to the AWS help docs that explain how to do this.
The guide will then explain how to apply our IaC artifacts created in #62 for setting up OBA on AWS.

Hii  @aaronbrethorst  i am working on this issue please assign this to me!!

complete!


Prevent copying logs

How to prevent copying SQL query logs?. I need to prevent bulk copying of logs by users with access rights

@akse2023 -- thank you for your question.  Are you following a particular learn.microsoft.com article?  Meanwhile, consider these resources:

SQL Server on Microsoft Q&A
DBA Stack Exchange
Stack Overflow

I am closing this issue now.  You are welcome to @ mention me for any followup.  We hope to hear from you again.