kubectl docs: explanation json patch, strategic merge $patch directives

Description
When learning k8s I found myself confused by the difference between json and merge patch types, usage of $patch strategic merge directives.
It's very confusing for new (and not so new) users that merge is the name for the RFC7386 JSON merge patch ("dumb" structural merge), json is the name for the RFC6902 json patch patch (array of operations), and strategic is the name for the strategic merge patch. And all of them can be in yaml...
Then there's server-side apply, which may act like a patch rather than replace in the presence of multiple resource managers.
Draft to  expand the kubectl patch (and soon, apply) guidance a bit to:

Link to the specification of strategic merge patches (obsoleted by server-side apply?)
mention $patch directives for overriding the merge strategy in strategic merge patches
More clearly explain the difference between a json (json patch) and merge (json merge) patch
add an example of a json patch, including selection of a container in an array and a test for the container name
add an example of using a json patch to append to volumes and volumemounts

This is draft, because I'm still not 100% sure myself of the rules surrounding server-side vs client-side patching and strategic merge patches so I have some more tests to write before proposing for merge. The current documentation is very confusing - server-side-apply is actually used for both patch and apply --server-side operations, but patch can accept json / merge / strategic formats, etc. I will seek to further refine my changes until they clearly explain how exactly k8s applies changes in different modes, and update further.
TODO

clarify if $patch directives delete, replace, merge, $retainKeys directive, $deleteFromPrimitiveList directive, $setElementOrder directive still relevant for kubectl patch --type strategic and if not, are there equivalent ways to specify with the input patch
clarify role of https://sigs.k8s.io/structured-merge-diff in kubectl patch and kubectl apply
determine kustomize use of strategic merge directives, see:

https://github.com/kubernetes-sigs/kustomize/issues/3694
https://github.com/kubernetes-sigs/kustomize/issues/306

Issue

You'll need to sign the CLA before we can review your changes, but you might like to check out https://kubernetes.io/docs/reference/using-api/api-concepts/#patch-and-apply and see if you want to also update that page, or maybe to direct readers to look there.

@sftim Thanks very much. The more I got into this the worse it got - I don't think I can do justice to untangling all the confusion and outdated or incomplete info in the relevant docs regarding the patch types, server-side apply, etc.
I can't clearly figure out for example if strategic merge patch directives are still supported with --server-side apply, and I don't expect to have time to do sufficiently in-depth study and test writing for it. I'm happy to do some further docs work if I can get some guidance understanding the problem space better, but I realised when I started looking into server-side apply that it's way more different than I thought.
I'll look at the ref you provided, and see if I can tidy this docs change up to be a minimal update that doesn't go too deep into the weeds.
I'm currently stuck waiting for my org to eventually appoint a CLA manager so I can sign the CLA, which is taking forever.

abandoning


[MNT] Handle updates to pre-commit linters

Reference Issues/PRs
Fixes error in #119.
What does this implement/fix? Explain your changes.

There were changes in a few of the linters used in skbase's continuous integration process.

pydocstyle moved to the use of tomli or tomllib for parsing pyproject.toml files (we are going to use tomli since tomllib is not available in older Python versions)

flake8-bugbear validation B028 was catching cases of manual quotes in string formatting. It caused #119 to fail. flake8-bugbear has since moved this to an optional validation. This enables that validation, along with two others. And fixes the resulting items flagged by the CI.

Note that B028 was changed to B907. And optional validations B03 and B904 were also turned on.  flake8-bugbear readme has a description. This PR includes fixes issues identified by B904.

Codecov Report

Merging #120 (ca26233) into main (929314d) will decrease coverage by 0.11%.
The diff coverage is 14.28%.

@@            Coverage Diff             @@
##             main     #120      +/-   ##
==========================================
- Coverage   77.93%   77.82%   -0.11%
==========================================
  Files          23       23
  Lines        1867     1867
==========================================
- Hits         1455     1453       -2
- Misses        412      414       +2

Impacted Files
Coverage Δ

skbase/base/_base.py
82.54% <0.00%> (ø)

skbase/base/_meta.py
15.27% <0.00%> (ø)

skbase/lookup/_lookup.py
95.13% <0.00%> (-0.35%)
:arrow_down:

skbase/testing/utils/deep_equals.py
63.88% <0.00%> (ø)

skbase/testing/utils/_dependencies.py
56.43% <50.00%> (-1.00%)
:arrow_down:

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


Change linting from flake8 to ruff

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

Coverage data is based on head (1032923) compared to base (1ebf1b4).
Patch has no changes to coverable lines.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #321   +/-   ##
=========================================
  Coverage   100.00%   100.00%
=========================================
  Files           10        10
  Lines          196       196
=========================================
  Hits           196       196

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

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


Using a method instead of a template

This is really just for discussion.  In the CMS for https://www.raise.me I actually don't use templates at all with the Shortcode gem.  I set the template for every shortcode to '<%= @content %> and generate a presenter that returns the content I want.
I actually have a presenter factory class that take in a name and a block and uses that block to generate the content.
Here is that class: https://gist.github.com/toddnestor/d6c94655e98ce5fbe0e65843262b0b58
I want to discuss if there is a good way to incorporate something like this as a feature in the shortcode gem.
I'd guess there are other people who need to be able to basically tie a method to a shortcode in this way like I do, but maybe not.  I have no problem if this doesn't become a feature, just want to open up a discussion about it.
So any thoughts about something like this that allows just tying a block of code to a shortcode instead of using a template?

Obviously my presenter factory class was a workaround, it seems to me that within this plugin there could probably be a more direct way to tie a shortcode to a method if we wanted to incorporate that feature.
Perhaps the shortcode arrays could be refactored to be hashes with the names as the keys and the value being a hash of options, one of those options could be to use a template or a method, or something along those lines.
I'll have to think about it some more.


added GMAO model support and updated documentation

@dbekaert @jlmaurer For the GMAO model, I removed the hashed out lines as suggested. For the documentation, I replaced the actual API key with a symbol.

@leiyangleon I suggest to run the following from the base RAiDER repo and then commit the updates:
autopep8 tools/RAiDER/ --recursive --select E231,E225,E251,E111,E128,E127,E265,E303,E261,E301 --in-place

To fix formatting/whitespace issues.

@jlmaurer could you add a template for PR with some guidance like this?
see under setting then scroll down to templates.

@dbekaert @jlmaurer all of the suggested changes by @Askaholic have been implemented except the one on "_uniform_z", which also calls the interp_along_axis function. However, there are some specific use in "_uniform_z", such as

self._e is used instead of self._p in all weather model readers

the output of self._uniform_z will enforce self._xs, self._ys and self._zs to be 1-D vectors instead of 3-D cubes, which will cause the program to fail for all weather model readers in the current stage. This has been reported to @jlmaurer (here)however without reply yet.

@dbekaert @jlmaurer @Askaholic I installed autopep8 and ran the command as suggested by @jlmaurer without any messages printed out. So I assume it is all set.

@dbekaert @jlmaurer the change on pydap version in environment.yml has been excluded in this PR, otherwise it keeps issuing conflicts to be resolved. So once this PR is merged, I will issue another one just to update pydap version.

Did it modify the files? You'll need to commit the changes, as the version currently pushed to github does not have the fixed formatting.
You'll also need to merge in the latest changes to the dev branch. To do this you'll first need to add this repo as a remote:
git remote add upstream https://github.com/dbekaert/RAiDER.git

and then run git pull upstream dev. This will pause on the conflicting file asking you to resolve the conflicts. Once they are resolved you can run git add environment.yml and git commit (just use the default message). Then you can push the merged changes to your fork.


Flatpak install, nothing happens when launched / cant find config.yaml

What type of installation are you running?
Desktop
Which Linux OS are you using?
aurora-dx-stable
Which version of LNXLink has the issue?
https://flathub.org/apps/io.github.bkbilly.lnxlink
Describe the problem
I have tried installing lnxlink via the flatpak, as using docker I couldn't get it working for the mic and webcam used status.
The install seems to go through, but there is no action when I click launch and no detailed instructions - I'd assume I would still need to do the config, but I can't find a config.yaml created by this install.
Additional context
No response
Configuration
No response
Logs
No response

The flatpak is very new addition, so I didn't have the time to write documentation for it.
You will need to run it with the -ic option and the config path should be under the documents directory.
-ic ~/Documents/config.yaml
Keep in mind that it won't create a systemd service, so you might want to create one manually.

Cheers, I'll give it a try tomorrow and report back.

Thanks, this worked and I managed to get it working
I created a systemd service manually to make sure it starts on boot.

That's great!
I am currently trying to release a new version that runs without any arguments and sets the default path to ~/Documents/lnxlink.yaml

That's great! I am currently trying to release a new version that runs without any arguments and sets the default path to ~/Documents/lnxlink.yaml

Could I suggest using a LNXlink folder?
~/Documents/LNXlink/config.yaml or so? Just neater in my opinion.
Happy to test any new release and give feedback.
Thanks.

That's a good idea.
I am still figuring out how to make a flatpak release update, so I don't know when I'll have it done.

I updated the flatpak package with the latest updates and it works with a default config file you suggested.
I've also added support for docker and systemd, but I am not sure if the systemd works on all systems.
There are more modules that still don't work with flatpak, but fortunately the most important ones work as expected.

I will give it a try next week.
Thanks

Hi,
Just tried this, and checked the new documentation for flatpak install to make sure I wasn't missing anything.
I uninstalled the previous version first (and disabled the systemd service I had created before etc to avoid any weirdness).
Whenever I try to run I get an error:

flatpak run io.github.bkbilly.lnxlink
bwrap: Can't make symlink at /var/run: File exists

Thanks

I can't think of any reason why you get this error.
The /var/run/ directory is only used for the files /var/run/reboot-required and /var/run/docker.sock, but even if these files don't exist, I don't get any errors.

I have a /var/run/docker.sock but no /var/run/reboot-required.
I did not have any error with the previous version

You could try installing the latest version to check if you still got this issue.

I'm away for a bit but I'll give this a try next week and report back.
Thanks

You could try installing the latest version to check if you still got this issue.

With the latest flatpack I am getting the following error:

❯ flatpak run io.github.bkbilly.lnxlink
bwrap: Can't make symlink at /var/run: existing destination is ../run


Thoughts on motors api standardization

I've read many messages related to motors or PTZ implementation from openIPC members in telegram chats and collected thoughts on api standardization in one place.
Also, my view on abstraction layers is represented in the architecture diagram.  If everything is right, I'll create a common motor-wrapper.h (or different name) with standardized function-signatures which will be implemented in each wrapper (i2c-motor, xm-kmotor etc.).
Any suggestions or corrections are welcome!

Thank you !


Youtube playlist stopped

https://www.youtube.com/playlist?list=PLb_y3RjS2-KC4tdxgeihcFVc7gKJBd_at
Everytime i'm tried, it always stopped the playlist queue once error appear
[youtube] V1VFkR0aU3s: Downloading thumbnail ...
[youtube] V1VFkR0aU3s: Writing thumbnail to: D:\youtube-dl\youtube\green screen effect blue auraa super saiyan.jpg
[download] D:\youtube-dl\youtube\green screen effect blue auraa super saiyan.mp4 has already been downloaded and merged
[download] Downloading video 84 of 160
[youtube] njEcGE6-s0Q: Downloading webpage
[youtube] njEcGE6-s0Q: Downloading video info webpage
ERROR: This video is not available.
Traceback (most recent call last):
  File "c:\users\gen32uc\appdata\local\programs\python\python37\lib\site-packages\youtube_dl\YoutubeDL.py", line 794, in extract_info
    ie_result = ie.extract(url)
  File "c:\users\gen32uc\appdata\local\programs\python\python37\lib\site-packages\youtube_dl\extractor\common.py", line 529, in extract
    ie_result = self._real_extract(url)
  File "c:\users\gen32uc\appdata\local\programs\python\python37\lib\site-packages\youtube_dl\extractor\youtube.py", line 1982, in _real_extract
    raise ExtractorError(error_message, expected=True)
youtube_dl.utils.ExtractorError: This video is not available.

Stopping on errors by default is expected behavior.

@wankio
    -i, --ignore-errors              Continue on download errors, for example to
                                     skip unavailable videos in a playlist

oh nice thank,


Publish in npm

Congratulations great job! Can you please publish the project in npm and make it compatible for loading it with webpack?
Thank you!

Hi @aldebaran798, sorry for the delay, the lib is now available on NPM as a module supporting ES6 import syntax.
See more on https://www.npmjs.com/package/jk-router


Application not launched after installation (windows msi installer)

After installing (windows, msi installer) nothing happens. User expectation is usually that the app is launched or at the very least have an app shortcut available somewhere (meny and or desktop).
Now you have to open an file explorer, go to c:\program files\jdk mon and launch it yourself. No launcher shortcut in the menu or desktop.

Need to check if there is the ability to add that behavior when creating the msi package but it might not be possible

I'm afraid this is not possible with the current solution, at least I did not found a way to configure it, sorry.

Semms to be an option to jpackage for linux and windows:
https://docs.oracle.com/en/java/javase/16/jpackage/manage-installation.html#GUID-E0E57A9E-CE36-48CE-8882-8AFB912B0E25

To have a shortcut created when users install your application, use the --linux-shortcut or --win-shortcut option when you package your application. To show a custom icon for your application, use the --icon option.

Shortcuts are supported for Linux and Windows platforms. If you don't provide an icon, a default icon is used. If you provide a custom icon on Linux, a shortcut is automatically created and the --linux-shortcut option is not needed. Custom icons must be in a format that meets the requirements of the platform.
The following command creates a


I create a new model in your repo and I get 72.95%mIoU but the picture it predict is very bad

You said aux_head improve STDC，but in models/init ,STDC  just support detail_head but doesn't support aux_head .


Broken Page

I installed Cocorico with Docker (the newest version) but i got a broken page. There is only HTML on my page.  -> http://<IP_ADDRESS>/en/
How can i change it?

Hi @latcher1
You need to modify app/config/parameters.yml and add your url to cocorico.assets_base_urls.
Hope that helps.

Hello @latcher1 @dazrour,
In fact, the parameter should be in your case:
cocorico.assets_base_urls: 'http://<IP_ADDRESS>'


fix: cascade delete service users on org delete

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 42.124%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/serve.go
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
4295

Relevant Lines:
10196

💛  - Coveralls


#274 Split project into Core and Maven Plugin

This is a first shot at splitting the project into a "core" project for code generation and a "maven" project for the actual plugin.
These two should be deployed as maven artifacts seperately :)
The core part can later be picked up by a custom Gradle plugin.
Also it contains a temporary fix for https://github.com/raml-org/raml-java-parser/issues/561

Talking with @stojsavljevic we agreed on opening another PR (#295 ) to go through this step by step.


Add SearchBar in the contributors page

Is your feature request related to a problem? Please describe.
Add a searchbar in the contributors page
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Are you participating under any open-source program?
( fill NA if none )

I want to work on it

@AtriSukul1508 Assigning you this issue . Happy coding !


Codegen don't creates Themes/Modules in right Solution directory

Branch master, version 1.10.0
When using codegen with /IncludeInSolution:true to create theme/module, it's created at the right place on file system, but it appears at the root of the solution in Visual Studio.

I experimented this in Dev branch also.
And the created subfolders is not optimal (no Controler, Drivers, Handlers but lot of assets often useless)+ the property file is marked as content.

I identified the problem, a new section was appended to the end of the solution file :
GlobalSection(ExtensibilityGlobals) = postSolution

and project attachment records are created in it instead of in :
GlobalSection(NestedProjects) = preSolution


Option (or secondary key binding) to open containing folder

View In Browser works great but I can't find a way to change the code to open the containing folder instead of the open file. I'm using it for local server testing and I have folder structures like these:
project/path/to/index.php
               /file.md

So, let's say I'm editing project/path/to/file.md. I'd like to open localhost/path/to/ (which will trigger the index.php) instead of localhost/path/to/file.md (which will just reveal the .md file in Finder). What better way to preview my Markdown files than to use my original website style, instead of having to install a separate Markdown preview plugin?

Honestly using a Markdown plugin for previewing Markdown files is probably best. There are several decent options out there. I use https://github.com/revolunet/sublimetext-markdown-preview


feat: batch operation support

This PR was generated using Autosynth. :rainbow:
Here's the log from Synthtool:
synthtool > Executing /tmpfs/src/git/autosynth/working_repo/synth.py.
synthtool > Ensuring dependencies.
synthtool > Pulling artman image.
latest: Pulling from googleapis/artman
Digest: sha256:ef1a98ab1e2b8f05f4d9a56f27d63347aefe14020e5f2d585172b14ca76f1d90
Status: Image is up to date for googleapis/artman:latest
synthtool > Cloning googleapis.
synthtool > Running generator for google/cloud/talent/artman_talent_v4beta1.yaml.
synthtool > Generated code into /home/kbuilder/.cache/synthtool/googleapis/artman-genfiles/js/talent-v4beta1.
.eslintignore
.eslintrc.yml
.github/ISSUE_TEMPLATE/bug_report.md
.github/ISSUE_TEMPLATE/feature_request.md
.github/ISSUE_TEMPLATE/support_request.md
.jsdoc.js
.kokoro/common.cfg
.kokoro/continuous/node10/common.cfg
.kokoro/continuous/node10/docs.cfg
.kokoro/continuous/node10/lint.cfg
.kokoro/continuous/node10/samples-test.cfg
.kokoro/continuous/node10/system-test.cfg
.kokoro/continuous/node10/test.cfg
.kokoro/continuous/node11/common.cfg
.kokoro/continuous/node11/test.cfg
.kokoro/continuous/node8/common.cfg
.kokoro/continuous/node8/test.cfg
.kokoro/docs.sh
.kokoro/lint.sh
.kokoro/presubmit/node10/common.cfg
.kokoro/presubmit/node10/docs.cfg
.kokoro/presubmit/node10/lint.cfg
.kokoro/presubmit/node10/samples-test.cfg
.kokoro/presubmit/node10/system-test.cfg
.kokoro/presubmit/node10/test.cfg
.kokoro/presubmit/node11/common.cfg
.kokoro/presubmit/node11/test.cfg
.kokoro/presubmit/node8/common.cfg
.kokoro/presubmit/node8/test.cfg
.kokoro/presubmit/windows/common.cfg
.kokoro/presubmit/windows/test.cfg
.kokoro/publish.sh
.kokoro/release/docs.cfg
.kokoro/release/docs.sh
.kokoro/release/publish.cfg
.kokoro/samples-test.sh
.kokoro/system-test.sh
.kokoro/test.bat
.kokoro/test.sh
.kokoro/trampoline.sh
.nycrc
.prettierignore
.prettierrc
CODE_OF_CONDUCT.md
CONTRIBUTING.md
LICENSE
README.md
codecov.yaml
renovate.json
Skipping: samples/README.md
synthtool > Replaced 'https:\\/\\/cloud\\.google\\.com[\\s\\*]*http:\\/\\/(.*)[\\s\\*]*\\)' in src/v4beta1/doc/google/protobuf/doc_timestamp.js.
synthtool > No replacements made in **/doc/google/protobuf/doc_timestamp.js for pattern toISOString\], maybe replacement is not longer needed?
synthtool > Replaced '"https:\\/\\/www\\.google\\.com"' in src/v4beta1/doc/google/cloud/talent/v4beta1/doc_company.js.

><EMAIL_ADDRESS>postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/core-js
> node scripts/postinstall || echo "ignore"

Thank you for using core-js ( https://github.com/zloirock/core-js ) for polyfilling JavaScript standard library!

The project needs your help! Please consider supporting of core-js on Open Collective or Patreon:
> https://opencollective.com/core-js
> https://www.patreon.com/zloirock

Also, the author of core-js ( https://github.com/zloirock ) is looking for a good job -)

><EMAIL_ADDRESS>postinstall /tmpfs/src/git/autosynth/working_repo/node_modules/protobufjs
> node scripts/postinstall

npm notice created a lockfile as package-lock.json. You should commit this file.
added 619 packages from 867 contributors and audited 1659 packages in 12.438s
found 0 vulnerabilities

><EMAIL_ADDRESS>fix /tmpfs/src/git/autosynth/working_repo
> eslint --fix '**/*.js'

/tmpfs/src/git/autosynth/working_repo/src/v4beta1/job_service_client.js
  20:26  error  "protobufjs" is extraneous  node/no-extraneous-require

✖ 1 problem (1 error, 0 warnings)

npm ERR! code ELIFECYCLE
npm ERR! errno 1
npm ERR<EMAIL_ADDRESS>fix: `eslint --fix '**/*.js'`
npm ERR! Exit status 1
npm ERR!
npm ERR! Failed at the<EMAIL_ADDRESS>fix script.
npm ERR! This is probably not a problem with npm. There is likely additional logging output above.

npm ERR! A complete log of this run can be found in:
npm ERR!     /home/kbuilder/.npm/_logs/2019-06-14T11_23_08_233Z-debug.log
synthtool > Cleaned up 2 temporary directories.
synthtool > Wrote metadata to synth.metadata.

@alexander-fenster looks like the generated code is pulling in protobufjs unnecessarily, not really a big deal but it breaks the build.

/tmpfs/src/github/nodejs-talent/src/v4beta1/job_service_client.js
  20:26  error  "protobufjs" is extraneous  node/no-extraneous-require

Getting close :)
 Scanning http://localhost:5873/google.cloud.talent.v4beta1.html
  [404] http://localhost:5873/google.type.html


No Pyroland Addon doesn't work

Describe the bug
The addon "No Pyroland" doesn't work. It conflicts with the already existent mtp.cfg in any mastercomfig presets. The addon is useless and should be descarted.
How you encountered the issue
I encountered the issue by exploring the files in the .vpk files.
Desktop (please complete the following information):

Config version: don't say latest, say the actual version, found by running version_comfig in console: mastercomfig version: 8.2.2 | November 23 2019
Config preset and addon(s): medium-low, no extra models, no tutorial and no pyroland for the test
Custom values (if any): This is my modules.cfg:
download=all
lod=high
lighting=medium_high
shadows=off
effects=low
water=ultra
particles=low
post_processing=off
pyrovision=low
motion_blur=off
aa=off
aa_msaa=off
texture_filter=trilinear
characters=ultra
decals=low
decals_models=off
decals_art=on
sprays=on
gibs=off
lowviolence=off
props=medium
ragdolls=off
3dsky=on
jigglebones=off
sheens_speed=low
sheens_tint=high
textures=very_high
texture_blending=off
bumpmap=on
specular=on
ropes=ultra
hud_player_model=on
hud_panels=on
messages=enable
killfeed=on
killstreaks=high
outlines=high
dynamic_background=off
sound=ultra
DX level: 95
Launch options: -novid -nojoy -nosteamcontroller -softparticlesdefaultoff -nohltv -particles 1 -primarysound -snoforceformat
Operating system (OS): Windows 10 Pro 64-bits
Graphics card (GPU): NVIDIA GeForce GTX 1050 Ti
Graphics driver version: 441.66
Processor (CPU): AMD FX(tm)-6300 Six Core Processor
TF2 on HDD or SSD: HDD
Memory (RAM) size in MB or GB: 8 GB

This can be fixed by renaming the file. I will be changing all addons to be of the format mastercomfig-addon-name and presets to be mastercomfig-preset-name. Because you are using medium-low (mastercomfig-medium-low-preset), m comes before the alphabet than n in no-pyroland (mastercomfig-no-pyroland-addon).


Corporation search in structures failed.

Hi. Trying to enter the corporation for a structure results in the following message every time.
"
Search failed
GuzzleHttp\Exception\ClientException, HTTP
404, CLient error: GET http://esi.evetech.net/v2/search/? categories=corporation&search=artificial&strict  resulted in a 404 Not Found` response:
{"error":"Not found"}
deleted
"
I am a new user and probably it is my mistake, but I can't find a way to fix it.

CCP recently-ish moved the /search endpoint behind an ESI scope.
https://esi.evetech.net/ui/#/Search

I have also been having this issue since I started to use pathfinder.

If anyone knows where this needs to be changed, please let me know so I can change my fork. I have been unable to find it.


[kong 2.1.0] Error: "No LuaRocks module found for resty.worker.events"

Summary
Build kong from source code success but get error No LuaRocks module found for resty.worker.events from error.log after get started
Steps To Reproduce

Follow the developer guide to build from source code
Build success with output:
kong 2.1.0-0 is now installed in /root/project/kong-sources/kong-build-tools/openresty-build-tools/build/luarocks (license: Apache 2.0)
verify kong version by kong version --vv:
2020/07/28 19:14:26 [verbose] Kong: 2.1.0 2020/07/28 19:14:26 [debug] ngx_lua: 10015 2020/07/28 19:14:26 [debug] nginx: 1015008 2020/07/28 19:14:26 [debug] Lua: LuaJIT 2.1.0-beta3 2.1.0
setup environment variable and run kong start -c myconfig with output:
kong started

Additional Details & Logs

Kong version: branch: 2.1.0

Kong debug-level startup logs: start_log.txt

Error log: error.log

Kong configuration: should be issue unrelated

Operating system: Centos 7

I set my $LUA_PATH by eval $(luarocks path) which is:
/root/project/kong-sources/kong-build-tools/openresty-build-tools/build/luarocks/share/lua/5.1/?.lua;./?.lua;/root/project/kong-sources/kong-build-tools/openresty-build-tools/build/openresty/luajit/share/luajit-2.1.0-beta3/?.lua;/usr/local/share/lua/5.1/?.lua;/usr/local/share/lua/5.1/?/init.lua;/root/project/kong-sources/kong-build-tools/openresty-build-tools/build/openresty/luajit/share/lua/5.1/?.lua;/root/project/kong-sources/kong-build-tools/openresty-build-tools/build/openresty/luajit/share/lua/5.1/?/init.lua;/root/project/kong-sources/kong-build-tools/openresty-build-tools/build/luarocks/share/lua/5.1/?/?.lua;;/root/.luarocks/share/lua/5.1/?.lua;/root/.luarocks/share/lua/5.1/?/init.lua;/root/project/kong-sources/kong-build-tools/openresty-build-tools/build/luarocks/share/lua/5.1/?/init.lua
My building folder: /root/project/kong-sources/kong-build-tools/openresty-build-tools/build
I can find events.lua at
/root/project/kong-sources/kong-build-tools/openresty-build-tools/build/luarocks/share/lua/5.1/resty/worker/events.lua
but error log show
no file '/root/project/kong-sources/kong-build-tools/openresty-build-tools/build/luarocks/share/lua/5.1/resty/worker/events/resty/worker/events.lua'
Anyone could help on this? Thanks!

Looks like the OS related error. When I change to use Ubuntu 18.04 instead of CentOS Linux release 7.7.1908 build the source with the same steps, it doesn't report the error.


Vectored IO for sockets.

Change sockets to use vectored IO and use an SDU length based on the local lazy bytestring.

Newer versions of the network package already use vectored IO. https://hackage.haskell.org/package/network-<IP_ADDRESS>/docs/src/Network.Socket.ByteString.Lazy.Posix.html#send


Implement leader-for-life on apicast operator

testing branch
Don't merge

Test upgrade

won't do


Add module_depends_on resource to private_service_access

private_service_access module depends on Service Networking API and vpc_network, so it would be very helpful to have module_depends_on in private_service_access.
Also description for private_service_access should be updated, it useful not only for MySQL but also for Postgres databases.

Thanks for the suggestion, these both sound like reasonable additions/fixes.


MudDateRangePicker: Add missing properties: PlaceholderStart/End and SeparatorIcon

Description

Fixes: https://github.com/MudBlazor/MudBlazor/issues/5324
Added PlaceholderStart, PlaceholderEnd and SeparatorIcon properties to MudDateRangePicker.
This properties are used by MudRangeInput so even without this change it is possible to set them.
For example this code works just fine: https://try.mudblazor.com/snippet/wEQnEnkJxCIfABKW
But they were not present on the API docs and are not visible to user (I mean as autocompletion in IDE).
You could only deduce that they are present by looking into RangeInput source code).
How Has This Been Tested?

unit
Types of changes

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)

Checklist:

[x] The PR is submitted to the correct branch (dev).
[x] My code follows the code style of this project.
[x] I've added relevant tests.

@henon I think we should merge this
I just found our that our usage example
https://github.com/MudBlazor/MudBlazor/blob/b13a71ec64bc1396ba7fe6d33afe056f14237a42/src/MudBlazor.Docs/Pages/Components/DatePicker/Examples/DateRangePickerUsageExample.razor#L6
sets the PlaceholderStart/End, but IDE shows them they are missing, I don't know why the compiler doesn't give an error.

It doesn't give an error because they are then treated as normal html arguments

Thanks @gaplin and sorry that your PR was overlooked for almost a year


add tokenrequest permission for import controller

Signed-off-by: Wei Liu<EMAIL_ADDRESS>

/assign @JakobGray

refer to refer to https://github.com/stolostron/backlog/issues/24450


Adding a first tag

Hello Florent,
Do you think it would be possible to create a first tag like a 1.0.0 version ?
thanks for you work !
Pierre

Hello Florent, did you had time to have a look for this issue ? :)

🙈 that totally slipped my mind, thank you for reminding me!
Tag created: https://github.com/Pluies/config-reloader-sidecar/releases/tag/v0.0.1


Complete MongoDB error handling

https://prisma.io/docs/concepts/components/prisma-client/handling-exceptions-and-errors currently says:

Error handling incomplete in MongoDB
Error handling for the MongoDB connector is incomplete.

We should fix that and remove this note when done.

Do we know what this means specifically? Off the top of my head, I don't know what this means.

Possibly these: https://github.com/prisma/prisma/issues?q=is%3Aopen+label%3A"topic%3A+mongodb"+label%3A"topic%3A+error"+
Definitely https://github.com/prisma/prisma/issues/11853 as that might be a common occurrence with existing MongoDB databases with questionable schemas.

Some minor work has been done wrt. this issue: https://github.com/prisma/prisma-engines/pull/2782


Failures on Expose CommentToken as analyzer public API...[analyzer] Prioritize errors in analysis_options/pubspec errors when running "dart analyze"

There are new test failures on Expose CommentToken as analyzer public API...[analyzer] Prioritize errors in analysis_options/pubspec errors when running "dart analyze".
The tests
pkg/dartdev/test/commands/analyze_test RuntimeError (expected Pass)

are failing on configurations
unittest-asserts-release-win-arm64

For example, https://ci.chromium.org/ui/p/dart/builders/ci.sandbox/pkg-win-release-arm64/1860/overview
00:11 +35 ~1 -1: analyze display mode default does not prioritize warnings in analysis_options.yaml [E]

  Expected: 'Analyzing myapp...\n'
              '\n'
              '  error - lib\\main.dart:1:16 - A value of type \'Null\' can\'t be returned from the function \'foo\' because it has a return type of \'int\'. - return_of_invalid_type\n'
              'warning - analysis_options.yaml:1:10 - The include file \'package:lints/recommended.yaml\' in \'c:\\opt\\s\\w\\ir\\x\\t\\dartdev7ceedc5f\\myapp\\analysis_options.yaml\' can\'t be found when analyzing \'c:\\opt\\s\\w\\ir\\x\\t\\dartdev7ceedc5f\\myapp\'. - include_file_not_found\n'
              '\n'
              '2 issues found.'
    Actual: 'Analyzing myapp...\n'
              '\n'
              '  error - lib\\main.dart:1:16 - A value of type \'Null\' can\'t be returned from the function \'foo\' because it has a return type of \'int\'. - return_of_invalid_type\n'
              'warning - analysis_options.yaml:1:10 - The include file \'package:lints/recommended.yaml\' in \'C:\\opt\\s\\w\\ir\\x\\t\\dartdev7ceedc5f\\myapp\\analysis_options.yaml\' can\'t be found when analyzing \'C:\\opt\\s\\w\\ir\\x\\t\\dartdev7ceedc5f\\myapp\'. - include_file_not_found\n'
              '\n'
              '2 issues found.'
     Which: is different.
            Expected: ... yaml' in 'c:\\opt\\s ...
              Actual: ... yaml' in 'C:\\opt\\s ...
                                    ^
             Differ at offset 277

  package:matcher/src/expect/expect.dart 149:31      fail
  package:matcher/src/expect/expect.dart 144:3       _expect
  package:matcher/src/expect/expect.dart 56:3        expect
  pkg\dartdev\test\commands\analyze_test.dart 664:9  defineAnalyze.<fn>.<fn>.<fn>
  ===== asynchronous gap ===========================
  package:test_api/src/backend/declarer.dart 215:9   Declarer.test.<fn>.<fn>
  ===== asynchronous gap ===========================
  package:test_api/src/backend/declarer.dart 213:7   Declarer.test.<fn>
  ===== asynchronous gap ===========================
  package:test_api/src/backend/invoker.dart 258:9    Invoker._waitForOutstandingCallbacks.<fn>

cc @DanTup

Thanks - I noticed this the other day and opened https://dart-review.googlesource.com/c/sdk/+/372300 which should fix it.


can support kcp-go?

I try to use this main.cpp to connect kcp-go(golang) server,But failed
kcp-go can't accept any data from client
and kcp-go send data to client,but client can't get any data
main.cpp:
`KCPNetClient lKcpClient;
KCPSettings lSettingsClient;
if(lKcpClient.configureKCP(lSettingsClient,
                           gotDataClient,
                           noConnectionClient,
                           "<IP_ADDRESS>",
                           12345,
                           10,
                           nullptr)) {
    std::cout << "Failed to configure the KCP Client" << std::endl;
}

lKcpClient.sendData((const char*)lData.data(), 4000);`
main.go:
`package main
import (
"log"
"github.com/xtaci/kcp-go/v5"

)
func main() {
if listener, err := kcp.ListenWithOptions("<IP_ADDRESS>:12345", nil, 10, 3); err == nil {
    // spin-up the client
    for {
        s, err := listener.AcceptKCP()
        s.SetNoDelay(1, 10, 2, 1)
        if err != nil {
            log.Fatal(err)
        }
        go handleEcho(s)
    }
} else {
    log.Fatal(err)
}

}
// handleEcho send back everything it received
func handleEcho(conn *kcp.UDPSession) {
buf := make([]byte, 4096)
for {
n, err := conn.Read(buf)
if err != nil {
log.Println(err)
return
}
println("get:" + string(buf[:n]))
n, err = conn.Write(buf[:n])
if err != nil {
log.Println(err)
return
}
}
}
`

Hi.
Just out of curiosity. Guess you found the error since you closed the ticket. Was there something we should do or know about?
Else thanks for contacting us.
/A


Fixed runtime panic when creating roles, added docs

What does this PR (Pull Request) do?

[x] This PR fixes #3891 and partially addresses #3443

Fixes runtime panics caused by requests with invalid payloads - the endpoint handler required a capabilities array but didn't check for it until after a role had already been created/replaced, thus returning an error message while still seemingly completing the operation successfully. To solve this, the capabilities array was made optional in both POST (where its absence will result in simply no capabilities added to the Role) and PUT requests (where its absence will result in capabilities not being updated on the Role - NOT deleting all existing capabilities, which is what would happen if an empty array was submitted).
As the POST, PUT and DELETE methods of the endpoint were undocumented, I also took the opportunity to fill those in - thereby documenting the new behavior. I also added some missing query parameters to the documentation for the GET method.
Which Traffic Control components are affected by this PR?

Documentation
Traffic Ops

What is the best way to verify this PR?
Verify existing tests pass. Build and run Traffic Ops/DB/Portal (CDN-in-a-Box recommended) and verify that admin users can create roles of privilege levels less than or equal to 30 (admin's privilege level) successfully, and no erroneous error message is reported. Build and read the documentation to ensure that it is accurate, comprehensible, and free of guideline violations and/or spelling/grammar mistakes.
If this is a bug fix, what versions of Traffic Control are affected?

master (c18d535)
3.0.1
3.0.2 (RC2)
3.1.0 (pre-release candidate)

The following criteria are ALL met by this PR

[x] This PR adds no additional tests, but tests for the endpoint already exist
[x] This PR includes documentation
[x] An update to CHANGELOG.md is not necessary
[x] This PR includes any and all required license headers
[x] This PR does not include a database migration
[x] This PR DOES NOT FIX A SERIOUS SECURITY VULNERABILITY

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4201/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4202/
Test PASSed.

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4456/

retest this please

Refer to this link for build results (access rights to CI server needed):
https://builds.apache.org/job/trafficcontrol-PR/4474/


fixed template path for issue #12

There was a path mismatch in $Template.Basepath and paths in
/pyhelm/chartbuilder.py:153
#12

@flaper87 udp


Add initial support for iOS, tvOS and watchOS

I've only tested this on a 64 bit iPhone running iOS 12. 32 bit is obsolete and I don't have access to the other platforms.

std.net.curl is disabled, libcurl is not shipped on these platforms. This should possible to fix by static linking libcurl and shipping libcurl together with the D compilers. But for that, some extra work are required due to dlopen not being allowed.

Most of the functionality in std.process is disabled. Creating new processes are not allowed.

Some of the unit tests in std.experimental.allocator fail. Perhaps the https://github.com/dlang/phobos/commit/537cef3c792f797f68bf75db12709d9ca87387d2 commit should be removed since it doesn't fix the unit test anyway

Some math related unit tests fail

One test in std.socket fail, but that is allowed to fail

The rest of the unit tests pass.
I recommend reviewing the commits one by one. The individual commit messages contain information of why something was changed. The diff for std.process is very big, but the only thing I did was to move environment, thisThreadID and thisProcessID to the top of the module. This allows to easily version out the rest.
druntime PR: https://github.com/dlang/druntime/pull/2910.
LDC PR: https://github.com/ldc-developers/ldc/pull/3288.

Anyone wants to have a look at this PR?

std/file.d(139): Error: `std.process.thisProcessID` called with argument types `()` matches both:
std/process.d(195):     `std.process.thisProcessID()`
and:
std/process.d(678):     `std.process.thisProcessID()`

Are these security limitations non-negotiable, e.g. would they still hold on a jailbroken device? If not, perhaps it would make sense to leave the code but only disable the tests.

I don't know. I don't plan to jailbreak my phone to find out. The APIs are available on iOS but not on watchOS or tvOS.

The std.process diff is unfortunately not very readable. I'm guessing things were moved in order to section off a part of definitions with version(...): or equivalent.

Yes, it's in the PR description.

Why does this fail: https://circleci.com/gh/dlang/phobos/11297?

I can't see the CircleCi failure at the moment, but buildkite fails because the public tests must be runnable outside of the module.

I can't see the CircleCi failure at the moment, but buildkite fails because the public tests must be runnable outside of the module.

Why? Isn't half of the point of having the unit tests inside the same module as the implementation that it can access non-public symbols?

Why? Isn't half of the point of having the unit tests inside the same module as the implementation that it can access non-public symbols?

Public unit tests are displayed in the documentation. Users must be able to execute the examples.

Public unit tests are displayed in the documentation

What do you suggest then? Using __traits to bypass the protection or duplicate the unit test for iOS and and remove ///? None of those alternatives seem ideal.

What about making the symbol public?

What about making the symbol public?

That's possible, but it doesn't seem ideal either.
Perhaps come up with a test that doesn't depend on these platform specific details. But on the other hand, AscendingPageAllocator seems to be very platform specific.

pageSize and minimumPageSize have been made available in druntime. Using those now.

This commit fixes the socket tests on iOS
https://github.com/etcimon/phobos/commit/984dbd18e75eebeccb7fb369861f554f98279d92

This commit fixes the socket tests on iOS

Awesome, thanks 👍.

The reason for failing was that the number of sockets monitored by select exceeded the FD_MAXSET limit on iOS of 1024

Hmm, ok. How is that possible? What other sockets were open? Something in Phobos that is not closed ?

Hmm, ok. How is that possible? What other sockets were open? Something in Phobos that is not closed ?

The unittest tries to open PAIRS*2 file descriptors to run a .select on them, which basically waits for one of those sockets to trigger an event. PAIRS*2 was 1533 or so, but on iOS, there's a limit of 1024 open file descriptors, thus the INVALID_ARGUMENT error that was returned.

dlang.phobos fails with:
/d/a/1/dmd/.azure-pipelines/lib.sh: line 37: HOST_DMD_VERSION: unbound variable

@jacob-carlborg : Should be fixed now, just rebase on master

dlang.phobos Windows_VisualD win32-ldc is still failing.

You didn't rebase (or didn't push, at least).

You didn't rebase (or didn't push, at least).

Ah, I missed the second part of your message.
Done. Let's see what happens now.

dlp 0.1.0: building configuration "dlp-test-application"...
--
  | Linking...
  | Running ./dlp-test-application
  | vendor/dmd/src/dmd/gluelayer.d(47): [unittest] Assertion failure
  | core.exception.AssertError@vendor/dmd/src/dmd/gluelayer.d(47): Assertion failure

That failure looks legit. Everything else is green or should be.

That failure looks legit. Everything else is green or should be.

I don't know why it would start to fail now. But I updated to a later version of the DMD frontend (which doesn't have this assert 😃). It passes now.
But now vibe.d was failing. Might be due to out of memory. I'm retrying that test now.

Everything is green now. Let's get this merged sooner rather than later. The changes in std.process are quite prone to cause merge conflicts.

@Geod24 ping.

I have no power here. Let's summon @thewilsonator .

Well I can't be arsed reviewing that, fortunately it seems @burner already has.

Congrats @jacob-carlborg !

Congrats

Thanks 😃.


BufferedPull pullArray

I believe https://github.com/zio/zio/blob/master/streams/shared/src/main/scala/zio/stream/ZStream.scala#L4002 contains a bug, what is basically going on is when we call update.as(chunk.drop(idx)) for the last time the update will fail (signaling end) de-facto losing the last element. In pullElement calling update is delegated to the subsequent call

Confirmed: https://github.com/zio/zio/pull/4164


[SPARK-19831][CORE] Reuse the existing cleanupThreadExecutor to clean up the directories of finished applications to avoid the block

Cleaning the application may cost much time at worker, then it will block that the worker send heartbeats master because the worker is extend ThreadSafeRpcEndpoint. If the heartbeat from a worker is blocked by the message ApplicationFinished, master will think the worker is dead. If the worker has a driver, the driver will be scheduled by master again.
It had better reuse the existing cleanupThreadExecutor to clean up the directories of finished applications to avoid the block.

Can one of the admins verify this patch?

I think you need to call cleanupApplicationThreadExecutor.shutdownNow() when worker is stopped.

ok to test

Test build #74220 has started for PR 17189 at commit a353262.

Test build #74220 has finished for PR 17189 at commit a353262.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74220/
Test PASSed.

Merged build finished. Test PASSed.

Test build #74266 has started for PR 17189 at commit cbf14b2.

Test build #74266 has finished for PR 17189 at commit cbf14b2.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74266/
Test PASSed.

@hustfxj Looks pretty good. Could you update the PR title and description to reflect the latest changes?

Test build #74397 has started for PR 17189 at commit 8e44aab.

@zsxwing Of course, Thank you for your reminding

Test build #74397 has finished for PR 17189 at commit 8e44aab.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/74397/
Test PASSed.

LGTM. Merging to master. Thanks!


Extend plugins injections

Plugins can inject to onCreate/applicationDidBecomeActive
Non native plugins can skip linking via skipLinking option
Fixed missing placeholders (Wear)
Added example of plugin config
Plugins - use of MainApplicationMethods: "" is DEPRECATED, use { "mainApplication": { "methods": []}} instead

Codecov Report

Merging #290 into develop will decrease coverage by 0.07%.
The diff coverage is 0%.

@@             Coverage Diff             @@
##           develop     #290      +/-   ##
===========================================
- Coverage    29.79%   29.71%   -0.08%
===========================================
  Files           54       54
  Lines         5723     5738      +15
  Branches      1253     1259       +6
===========================================
  Hits          1705     1705
- Misses        2913     2922       +9
- Partials      1105     1111       +6

Flag
Coverage Δ

#rnvunittests
29.71% <0%> (-0.08%)
:arrow_down:

Impacted Files
Coverage Δ

packages/rnv/src/platformTools/android/index.js
16.15% <ø> (ø)
:arrow_up:

packages/rnv/src/platformTools/apple/index.js
13.54% <ø> (ø)
:arrow_up:

...ackages/rnv/src/platformTools/apple/swiftParser.js
11.68% <0%> (-0.16%)
:arrow_down:

...ages/rnv/src/platformTools/android/gradleParser.js
16.91% <0%> (-0.26%)
:arrow_down:

...ages/rnv/src/platformTools/android/kotlinParser.js
25% <0%> (-3.95%)
:arrow_down:

packages/rnv/src/platformTools/runner.js
28.94% <0%> (ø)
:arrow_up:

Continue to review full report at Codecov.

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


can't run adjoint example --tutorial_global_oce_optim

Note: for general support questions, please use the mailing list<EMAIL_ADDRESS>This repository's issues are intended for feature requests and bug reports.

after i make adall

Hey there, and welcome to the MITgcm+adjoint world! I see these warnings regularly, but I think they are benign. Does the compilation stop there? Are you trying to use TAF or OpenAD? Can you give some specs about the system your using and the genmake arguments you're passing?
Thanks!

Hey there, and welcome to the MITgcm+adjoint world! I see these warnings regularly, but I think they are benign. Does the compilation stop there? Are you trying to use TAF or OpenAD? Can you give some specs about the system your using and the genmake arguments you're passing?
Thanks!

thank you very very much. I am a beginner.the compilation is stop.i am try to use taf.

Hmm. Well it seems to be finding your MITgcm checkout along with the AD_OPTFILE. What is the final output before exiting? Did you run
genmake <... + other arguments...>
make depend
make adall

The warnings from your first comment show up after make depend is done, but since those are usually benign, I'm still not sure what's causing the compilation to fail. Could you show the very last output after running make adall which might give us a clue what the error is?
Also, if you intend to run with MPI you'll want to add a -mpi flag to the genmake command. See this in the documentation here if that pertains to you.

Hmm. Well it seems to be finding your MITgcm checkout along with the AD_OPTFILE. What is the final output before exiting? Did you run
genmake <... + other arguments...>
make depend
make adall

The warnings from your first comment show up after make depend is done, but since those are usually benign, I'm still not sure what's causing the compilation to fail. Could you show the very last output after running make adall which might give us a clue what the error is?
Also, if you intend to run with MPI you'll want to add a -mpi flag to the genmake command. See this in the documentation here if that pertains to you.

Thank you for your reply.You are very nice!! This is my make adall errors.

Hey no problem - we're getting somewhere now! At this point the build process needs to send your code to the company fastopt in order to write the adjoint mode corresponding to your forward model code. This is enabled through the proprietary software as part of TAF and you will need a license to use it. With the license you are given the binary file staf which is used to communicate between your machine and the TAF servers. Make sure that your computer can see the staf binary. A useful way to do this is to put staf in a safe spot (for instance I put it in a directory like ~/local/bin or ~/.local/bin). Then, add this directory to your PATH environment variable in e.g. your ~/.bash_profile or ~/.bashrc file as follows:
export PATH=$PATH:/path/to/staf

Note that you'll need to either source the file where you add that or open a new terminal window for this to take effect.

Hey no problem - we're getting somewhere now! At this point the build process needs to send your code to the company fastopt in order to write the adjoint mode corresponding to your forward model code. This is enabled through the proprietary software as part of TAF and you will need a license to use it. With the license you are given the binary file staf which is used to communicate between your machine and the TAF servers. Make sure that your computer can see the staf binary. A useful way to do this is to put staf in a safe spot (for instance I put it in a directory like ~/local/bin or ~/.local/bin). Then, add this directory to your PATH environment variable in e.g. your ~/.bash_profile or ~/.bashrc file as follows:
export PATH=$PATH:/path/to/staf

Note that you'll need to either source the file where you add that or open a new terminal window for this to take effect.

you mean i should buy a taf license?it is not free?

Yeah this is a tough reality unfortunately. However, OpenAD is an open source option which should serve most of your adjoint needs. You can read a little bit about OpenAD in the MITgcm manual here.
I should say that I am not well-versed with using OpenAD, and unfortunately it looks like the documentation is a little thin... If you would like help with using OpenAD I would suggest emailing the MITgcm support list<EMAIL_ADDRESS>as this is the preferred avenue for getting help. There are a few folks who use OpenAD who could probably help get you setup. If and when you email the list, please consider giving a couple of details regarding your end goals (i.e. what kind of domain you would like to simulate, purpose, etc), because not all MITgcm functionality works with OpenAD and we could get that figured out from the beginning.

Yeah this is a tough reality unfortunately. However, OpenAD is an open source option which should serve most of your adjoint needs. You can read a little bit about OpenAD in the MITgcm manual here.
I should say that I am not well-versed with using OpenAD, and unfortunately it looks like the documentation is a little thin... If you would like help with using OpenAD I would suggest emailing the MITgcm support list<EMAIL_ADDRESS>as this is the preferred avenue for getting help. There are a few folks who use OpenAD who could probably help get you setup. If and when you email the list, please consider giving a couple of details regarding your end goals (i.e. what kind of domain you would like to simulate, purpose, etc), because not all MITgcm functionality works with OpenAD and we could get that figured out from the beginning.

thank you for your advise~~

I'm going to close this issue now. Future support questions should be directed to the support mailing list.


Unable to play Honkai : Star Rail crashes immediately after launching the game with GPTK2b3 (Crossover D3DMetal)

Describe the bug
When I click on "Launch Game" in Yaagl HSR OS, a black window opens before crashing
Screenshots

System Information (please complete the following information):

macOS: Sequoia 15.1 (24B2083)
Device: MacbookPro 14" 2024
CPU: Apple M4
RAM: 16G
Wine Distribution: [Crossover D3DMetal with GPTK 2.0 beta3]

Logs
Couldn't get first exception for process 00d8 Z:\Users\elrohir\Yaagl OS\HSR\StarRail.exe.


Add an OOP article for high-scores

In response to discord message https://discord.com/channels/854117591135027261/1157359032760287302/1258802989985173546
The concept's about.md and introduction.md have the same content.
The content is largely taken from my personal mentoring notes.

This looks great! How is this exposed via the Exercism UI? Does this automatically get linked from any exercise tagged with oop?

This looks great! How is this exposed via the Exercism UI? Does this automatically get linked from any exercise tagged with oop?

Hmm I don't know: http://forum.exercism.org/t/how-does-dig-deeper-work/12016

I felt the tone of the tutorial is not really concept-y which is why I moved it to an article. What would you prefer?

I'm good with an article


Wrong object exported (classroom.blend)

Using Blender 2.8
classroom.blend (the example file come with Blender)
Exported as gltf 2.0
Check with any gltf viewer app, no chairs in scene, but lots of door handles
Check the gltf file manually
All the object chair and its instances become object Box.024(the door handle)
Something wrong with the exporter

This is a duplicate of #619 : 3 scenes are exported, but the active scene is the last exported.
But:: Something is wrong when animation is exported. Created #639 for this issue.

I checked all 3 scenes, none of them contain object chair but door handles
If I select the chair and enable export selected object option, I got 3 empty scenes

The scene is using lots of particles and dupli-instances. This is a know limitation, will be manage in #500

Seems to be fixed in #1378


Theme customisation on variant level

Duplicates

[X] I have searched the existing issues

Latest version

[X] I have tested the latest version

Summary 💡
I am trying to customise MUI V5 theme to define colours of a design system for all components according to variant, color and state(hover).
I tried to follow the approach mentioned in: https://mui.com/blog/callback-support-in-style-overrides/ and it works fine when I tried it for MuiButton as shown below. but to apply the design system I have to write the same conditions for all defined components(Fab, chip, selector,...) with all supported colors and variants :
MuiButton: {
   defaultProps: {
        variant: 'contained',
       },
   styleOverrides:{
      root: ({ ownerState, theme }) => ({
            ...(ownerState.color === 'error' && {
              ...(ownerState.variant === 'contained' &&{
                      '&:hover':{
                          backgroundColor: '#530F00',
                       }
              }),
            ...(ownerState.variant === 'outlined' &&{
               borderColor: '#D48E80',
               '&:hover':{
                   backgroundColor: '#F8EDEB',
                }
           }),
      }),
   },
},

Is there any way to implement a theme customisation on variant level so it would applied for all components that support that variant? Or is there a better way to achieve this?
Examples 🌈
No response
Motivation 🔦
No response

StackOverflow question: https://stackoverflow.com/questions/72201515/material-ui-customise-theme-for-hover-state


Unable to launch any Catalog services

Rancher Versions:
Server: v1.5.5
kubernetes: 1.5.4-rancher1-3
Docker Version:
 Running: 16
 Paused: 0
 Stopped: 1
Images: 14
Server Version: 1.12.6
Storage Driver: btrfs
 Build Version: Btrfs v4.4
 Library Version: 101
Logging Driver: json-file
Cgroup Driver: cgroupfs
Plugins:
 Volume: local
 Network: null overlay host bridge
Swarm: inactive
Runtimes: runc
Default Runtime: runc
Security Options: apparmor seccomp
Kernel Version: 4.4.0-64-generic
Operating System: Ubuntu 16.04.2 LTS
OSType: linux
Architecture: x86_64
CPUs: 1
Total Memory: 1.952 GiB
Name: rancher-public-k8s-a0.dev.dsync.local
ID: FHB6:2PP7:TQN7:ICVW:XEFV:56LG:LQT3:TEKO:QK24:QJM6:WSCN:IBI3
Docker Root Dir: /var/lib/docker
Debug Mode (client): false
Debug Mode (server): false
Registry: https://index.docker.io/v1/
WARNING: No swap limit support
Insecure Registries:
 <IP_ADDRESS>/8

OS and where are the hosts located? (cloud, bare metal, etc):
AWS rancher-server running behing a reverse proxy
Setup Details: (single node rancher vs. HA rancher, internal DB vs. external DB)
Single node with external RD RDS
Environment Type:
Kubernetes
Steps to Reproduce:
Go to the Catalog
Launch anything
Results:
The stack gets created with nil containers.
No useful log:
time="2017-04-12T03:56:51Z" level=info msg="Stack Create Event Done" eventId=985d4a49-7fd6-4289-a0c5-4fa5813a0b08 resourceId=1st45

Screenshot:

When you say nil containers, do you mean it comes up with 0 containers?
Can you share a screen shot of the stacks?
Did your infrastructure stacks come up with any containers?

@deniseschannon yes that's correct. I've noticed this behaviour since v1.5.3 as well.
Screenshot attached to issue.

The catalog is deprecated for Kubernetes in favor of providing Helm as of 1.4. You've found a path to the catalog where templates are still shown, but the backend is not there to deploy them anymore.
There are a couple of ways to "find" your way to the catalog linked in this other issue:
https://github.com/rancher/rancher/issues/8336


Upgrade micrometer to 1.6.6

Motivation: Micrometer 1.7 is out and we shouldn't stay 2 minor versions behind
I refrained from jumping right to 1.7 because

it's still quite recent
it introduces a breaking change by requiring prometheus simpleclient 0.10.0, which has a small change in metrics name convention. https://github.com/prometheus/client_java/releases/tag/parent-0.10.0

LGTM, thanks!


Zip vex and transform zipped

rough initial versions on lattice pricers and simd utils needed

Adding first cut of useful features for zipped iteration, spans and samplers and adding some simple lattice/tree pricing model implementations as examples.   Additionally, non variadic versions of multi reduce and multi transform reduce are added together with first cut at pairwise vectorised reduction ( these have significant performance benefits and greater accuracy too)


Fix possible NPE when using HttpClientCodec

Motivation:
It was possible to produce a NPE when we for examples received more responses as requests as we did not check if the queue did not contain a method before trying to compare method names.
Modifications:

Add extra null check
Add unit tet

Result:
Fixes https://github.com/netty/netty/issues/9459

@netty-bot test this please

@netty-bot test this please


Custom Filter: parameters() method in Swift wont get called

Im making a custom filter subclassing MTIUnaryImageRenderingFilter
But the parameters method is not called like in the Pixellate example.
Im getting:
validateFunctionArguments:3469: failed assertion `Fragment Function(prismoj): missing buffer binding at index 0 for time[0].'
Here's the filter class:
import UIKit
import MetalPetal
class G8PrismOJ_sh: MTIUnaryImageRenderingFilter {
    var time: Float

    override init() {
        time = Float(1.0)
        super.init()
    }

    override class func fragmentFunctionDescriptor() -> MTIFunctionDescriptor {
        let bundle = Bundle(for: G8PrismOJ_sh.self)
        let metalLibraryUrl = bundle.url(forResource: "default", withExtension: "metallib")
        return MTIFunctionDescriptor(name: "prismoj", libraryURL: metalLibraryUrl)
    }

    func parameters() -> [String : Any?] {
        return [
        "time": MTIVector(x: time, y: time)
        ]
    }
}

I actually would like to pass time as a Float but I dont know if it will get accepted by the shader.
But anyways, the parameters() function is not called, maybe Im translating wrong from ObjC to Swift based in the Pixellate example.

The parameters is a "property" not a method.

So right !!! Thank you :)


add tpc recoinit

Please hold off on merging this until https://github.com/sPHENIX-Collaboration/macros/pull/899 is merged and in the nightly build


process crashes if disconnected happens soon after connect

If the socket is suddenly shutdown right after a connect (how unlikely, it can happen), the onReady event does not wrap or handle flush failures.
Check here: https://github.com/JustinTulloss/zeromq.node/blob/1a8977137a51aa2259fdc4b3c1441aca301ac0db/lib/index.js#L297
Test Code:
zmqSocket.connect(url);
zmqSocket.disconnect(url);

Error:
Uncaught Exception: Error: No route to host
    at Error (native)
    at Socket._flushWrite (/Users/almirkadric/Projects/Mage/mage2/core/nodejs/modules/mageCore-messageRelay-zmq/node_modules/zmq/lib/index.js:648:19)
    at Socket._flushWrites (/Users/almirkadric/Projects/Mage/mage2/core/nodejs/modules/mageCore-messageRelay-zmq/node_modules/zmq/lib/index.js:693:19)
    at _zmq.onReady (/Users/almirkadric/Projects/Mage/mage2/core/nodejs/modules/mageCore-messageRelay-zmq/node_modules/zmq/lib/index.js:298:24)

EDIT: This also applies to sending a message to an invalid address right after a connect. The onReady event will try and flush this message and end up excepting crashing the process.

After looking at this with @ronkorving he mentioned that we get the throw only if a callback isn't provided. Which leaves me with the following feedback:

The throw doesn't emit an error on the global emitter
The throw can both happen synchronously and asynchronously,
making it impossible to catch all of them
Due to the above behaviour, the send function should heavily rely on the callback, not doing so makes it extremely convoluted to use.

Actually this still breaks even if I use the callback. The reason behind this is because i use an array for the message to specify router [identity, context, message] however if the first frame fails, the callback isn't set according to this code here: https://github.com/JustinTulloss/zeromq.node/blob/1a8977137a51aa2259fdc4b3c1441aca301ac0db/lib/index.js#L607

To reproduce:
zmqSocket.connect(url);
zmqSocket.disconnect(url);
zmqSocket.send([nodeHostname, context, message], null, function (error) {
  if (error) {
    console.log('should have', error);
  }
});

Instead of the error being logged we get this stack exception:
Uncaught Exception: Error: No route to host
    at Error (native)
    at Socket._flushWrite (./node_modules/zmq/lib/index.js:648:19)
    at Socket._flushWrites (./node_modules/zmq/lib/index.js:693:19)
    at Socket.send (./node_modules/zmq/lib/index.js:615:10)

The callback should be set according to the code, and it should fire. I'll look into this.

What kind of socket have you created, and what kind of socket options have you set?

zmqSocket = zmq.socket('router');
zmqSocket.identity = serviceDiscovery.getNodeHostname();

zmqSocket.setsockopt(zmq.ZMQ_ROUTER_MANDATORY, 1);

You found a massive error on my part, thanks.


Fix suggest on staging

To fix #2245.
The suggest request on Firefox is returning {"err":"Disallowed"} which suggests the problem is CORS-related.
https://github.com/badges/shields/blob/88402dd7a8ed64953937157de866d9a62dab8c48/lib/suggest.js#L112-L120
On staging, unlike production, the hostname is the same, which means browsers might not send CORS headers by default. Maybe there's a difference between how Chrome and Firefox handle that.
Anyway, here's a fix based on that hunch.

Warnings

:warning:

This PR modified helper functions in lib/ but not accompanying tests. That's okay so long as it's refactoring existing code.

Messages

:book:

:sparkles: Thanks for your contribution to Shields, @paulmelnikow!

  Generated by :no_entry_sign: dangerJS


Create 0000-01-02-dev2350.md

created a branch, created a file and made a commit, and opened a pull request


[NCL-5884] Also consider new defaultAlignmentParams in adjust

All Submissions:

[ ] Have you added a note in the CHANGELOG wiki for your change?
[ ] Have you added unit tests for your change?

@thescouser89 Hi Dustin! I was wondering about the ordering of the parameters. As far as I understand, in case there are multiple parameters specified, the rightmost one "wins", right? So, considering the default_alignment_parameters, the user_defined_alignment_parameters and the repour_default_parameters, would the ordering be:
default_alignment_parameters + user_defined_alignment_parameters + repour_default_parameters ?
This way, users can override the default_alignment_parameters, but Repour would takeover any parameter which is not meant to be overridden? Thanks!

@vibe13 The wording in the code is certainly confusing.
It's:
repour_default_parameters + build_config_default_parameters + user_defined_parameters (as provided in `extra_adjust_parameters`)


add error log

Proposed changes
Issue Number: 31441
Solved core dump caused by Log(FATAL)
Further comments
LOG(FATAL) in Doris_main.cpp changed

run buildall

run buildall

run buildall


sidebar not auto hidden in mobile

hello, lovely to have more SSG framework for deno. however, despite #36 being resolved, it seems like sidebars in pyro.deno.dev is not mobile friendly. specifically, it hides half of page.

Thanks for the issue. I will work to resolve this soon.


Rantdocs; Unity - Example not complete

Describe the bug
The Unity usage example assumes that user has Rantionary.rpkg file in assets/resources folder but there is no step by step information on how to import those and from what source.
The Rant3.zip from github doesn't contain said file.
To Reproduce
Steps to reproduce the behavior:

Follow the Running in Unity tutorial
Run the example in Unity
See the error

Expected behavior
error seen in console
NullReferenceException: Object reference not set to an instance of an object
ranter.Start () (at Assets/ranter.cs:24)
refers to
using (var pkgStream = new MemoryStream(pkgAsset.bytes))

@TheBerkin Hey, any update on this? I'm facing the same error. I have a rantionary rantpkg file created using the RCT tool in my Resources folder in Unity. I'm getting the same error as mentioned above.

@TheBerkin Nevermind. I solved it. You need to rename the .rantpkg file into the .bytes file. For example, Rantionary.rantpkg should be renamed to Rantionary.bytes. The reason for this is according to the manual https://docs.unity3d.com/Manual/class-TextAsset.html, there are only a few select formats for textasset files that are supported.


📦 Transformation Blocks Add to Canvas

Addition:

Added TransformationCatalog & TransformationBlock DTO for data handling
Created base node handler
Created add to canvas feature
Created empty modal for the configuration

Disabled configure button
The configure button of a transformation block should be disabled as long as the block isn't connected to preceding blocks (see designs).


[Exposure Request] codeunit 410 "Update Analysis View" - FlushAnalysisViewEntry

Please make function FlushAnalysisViewEntry externally available

Thanks for reporting this. We agree, and we’ll publish a fix asap, either in an update for the current version or in the next major release. Please do not reply to this, as we do not monitor closed issues. If you have follow-up questions or requests, please create a new issue where you reference this one.
Build ID: .


command+F not available in ghost desktop

Not sure if this is an issue or an oversight, but Command+F is not available in ghost desktop to find text.

Likely an oversight, but I think this could be filed as an enhancement.
Electron doesn't support find in page natively, but that doesn't us from making it work via window.find. We'd probably have to be a little smarter about it, since it would likely end up highlighting content outside of the ghost instance (sidebar, preferences pane, etc.).
Thoughts, @felixrieseberg?

Yeah, let's put it on the roadmap. Good news though: While the UI isn't there, the methods (findInPage and stopFindInPage) are: https://github.com/atom/electron/blob/master/docs/api/web-contents.md#webcontentsfindinpagetext-options


Schema examples for the YAML configuration

These should be similar to what is in the design doc, but will be more finalised and critized further.
Multiple examples should be created, similar to here in the armada repo,

We will want to add changing queue and changing priority support to this as well


[Bug]: Web UI large request completion percentage incorrect

What steps will reproduce the bug?
When a request has a large number of tasks and is close to completion the Web UI will incorrectly round up to 100 in the "status" column for the request.  For example if there are 331 tasks out of 333 it will show as 100.  May just need to round down instead of up.
What is the expected behavior?
I would expect it to show 99% instead of 100%.
What do you see instead?
Status shows 100%
Additional information
No response

I don't believe this is a bug in the web ui check out the logic here it does already round and I'm using success+failed tasks / total tasks: https://github.com/google/turbinia/blob/942ec314c190890feb54d40cdf55d718a5a00edd/web/src/components/RequestList.vue#L193

It looks like it is doing a normal math Round, but that means it will round something like 99.7 to 100, so I think we probably actually want to do a Floor calculation instead of Round and show 99 in this case and not 100.
Here is an example output of what turbinia-client status request <req id> shows for a request exhibiting this behavior which is 281+51/333 == 332/333 == ~0.99699, but we see 100% in the UI.
* Failed tasks: 51
* Running tasks: 0
* Successful tasks: 281
* Task Count: 333


[BUG]  org.opensearch.index.shard.RemoteIndexShardTests.testNoFailuresOnFileReads

Describe the bug
It say test passed here - https://build.ci.opensearch.org/job/gradle-check/29071/testReport/junit/org.opensearch.index.shard/RemoteIndexShardTests/testNoFailuresOnFileReads/
But shows up as failed in gradle check
https://build.ci.opensearch.org/job/gradle-check/29071/
Seeing the below exception in Standard output
[2023-10-26T08:50:15,190][ERROR][o.o.i.s.IndexShard       ] [testNoFailuresOnFileReads] [test][0] skip local recovery as no index commit found
[2023-10-26T08:50:15,503][INFO ][o.o.i.r.RecoverySourceHandler] [org.opensearch.index.shard.RemoteIndexShardTests] [test][0][recover to s1] finalizing recovery took [222.3ms]
[2023-10-26T08:50:22,157][INFO ][o.o.i.s.RemoteIndexShardTests] [testNoFailuresOnFileReads] --> Corrupting file _0.cfe
[2023-10-26T08:50:22,159][INFO ][o.o.t.CorruptionUtils    ] [testNoFailuresOnFileReads] Corrupting file --  flipping at position 397 from 0 to 1 file: _0.cfe
[2023-10-26T08:50:28,469][WARN ][o.o.i.r.SegmentReplicationTarget] [org.opensearch.index.shard.RemoteIndexShardTests] [test][0] Error reading name [_0.cfe], length [405], checksum [6e7t5f], writtenBy [9.8.0]
org.apache.lucene.index.CorruptIndexException: Illegal CRC-32 checksum:<PHONE_NUMBER>4603987 (resource=MockIndexInputWrapper(NIOFSIndexInput(path="/var/jenkins/workspace/gradle-check/search/server/build/testrun/test/temp/org.opensearch.index.shard.RemoteIndexShardTests_A71542B40F44C18D-002/tempDir-006/indices/uuid/0/index/_0.cfe")))
	at org.apache.lucene.codecs.CodecUtil.readCRC(CodecUtil.java:631) ~[lucene-core-9.8.0.jar:9.8.0 d914b3722bd5b8ef31ccf7e8ddc638a87fd648db - 2023-09-21 21:57:47]
	at org.apache.lucene.codecs.CodecUtil.retrieveChecksum(CodecUtil.java:535) ~[lucene-core-9.8.0.jar:9.8.0 d914b3722bd5b8ef31ccf7e8ddc638a87fd648db - 2023-09-21 21:57:47]
	at org.opensearch.indices.replication.SegmentReplicationTarget.validateLocalChecksum(SegmentReplicationTarget.java:238) [main/:?]
	at java.base/java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:178) [?:?]
	at java.base/java.util.stream.ReferencePipeline$2$1.accept(ReferencePipeline.java:179) [?:?]
	at java.base/java.util.ArrayList$ArrayListSpliterator.forEachRemaining(ArrayList.java:1625) [?:?]
	at java.base/java.util.stream.AbstractPipeline.copyInto(AbstractPipeline.java:509) [?:?]
	at java.base/java.util.stream.AbstractPipeline.wrapAndCopyInto(AbstractPipeline.java:499) [?:?]
	at java.base/java.util.stream.ReduceOps$ReduceOp.evaluateSequential(ReduceOps.java:921) [?:?]
	at java.base/java.util.stream.AbstractPipeline.evaluate(AbstractPipeline.java:234) [?:?]
	at java.base/java.util.stream.ReferencePipeline.collect(ReferencePipeline.java:682) [?:?]
	at org.opensearch.indices.replication.SegmentReplicationTarget.getFiles(SegmentReplicationTarget.java:200) [main/:?]
	at org.opensearch.indices.replication.SegmentReplicationTarget.lambda$startReplication$2(SegmentReplicationTarget.java:170) [main/:?]
	at org.opensearch.core.action.ActionListener$1.onResponse(ActionListener.java:82) [opensearch-core-2.12.0-SNAPSHOT.jar:2.12.0-SNAPSHOT]
	at org.opensearch.common.util.concurrent.ListenableFuture$1.doRun(ListenableFuture.java:126) [main/:?]
	at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:52) [main/:?]
	at org.opensearch.common.util.concurrent.OpenSearchExecutors$DirectExecutorService.execute(OpenSearchExecutors.java:343) [main/:?]
	at org.opensearch.common.util.concurrent.ListenableFuture.notifyListener(ListenableFuture.java:120) [main/:?]
	at org.opensearch.common.util.concurrent.ListenableFuture.addListener(ListenableFuture.java:82) [main/:?]
	at org.opensearch.action.StepListener.whenComplete(StepListener.java:95) [main/:?]
	at org.opensearch.indices.replication.SegmentReplicationTarget.startReplication(SegmentReplicationTarget.java:169) [main/:?]
	at org.opensearch.indices.replication.SegmentReplicationTargetService.start(SegmentReplicationTargetService.java:515) [main/:?]
	at org.opensearch.indices.replication.SegmentReplicationTargetService$ReplicationRunner.doRun(SegmentReplicationTargetService.java:501) [main/:?]
	at org.opensearch.common.util.concurrent.ThreadContext$ContextPreservingAbstractRunnable.doRun(ThreadContext.java:908) [main/:?]
	at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:52) [main/:?]
	at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1136) [?:?]
	at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:635) [?:?]
	at java.base/java.lang.Thread.run(Thread.java:833) [?:?]
[2023-10-26T08:50:28,834][INFO ][o.o.i.s.RemoteIndexShardTests] [testNoFailuresOnFileReads] after test

@dreamer-89 : Can you please take a look on priority? You newly added this test and it is failing - https://github.com/opensearch-project/OpenSearch/pull/10933

https://github.com/opensearch-project/OpenSearch/pull/10944#issuecomment-1780520777

Looking into it

The test fails while asserting there are at least one file which is corrupted here.

still seeing this test failing as part of this backport PR: https://github.com/opensearch-project/OpenSearch/pull/12193 (org.opensearch.index.shard.RemoteIndexShardTests.testNoFailuresOnFileReads)

[Triage - attendees 1 2 3]
The backport PR with the fix for this issue is stalled and has not been merged: https://github.com/opensearch-project/OpenSearch/pull/11806

Closing in favour of https://github.com/opensearch-project/OpenSearch/pull/11806


BouncyCastle.Crypto.dll is missing after MSI setup when Windows Defender is active

Hello,
I am using the NuGet package BouncyCastle (Version <IP_ADDRESS>) in my project. For installing my software, I am providing a MSI installer created with WiX toolset.
When installing the software when Windows Defender Real-time protection is disabled, the library is copied correctly to the installation directory located in a subfolder in "C:\Program Files". When the installation path is excluded by rule, the file is also copied correctly.
But when Windows Defender is active without exclusion, the file is missing after installing the msi.
When checking the DLL manually with Defender and on virustotal.com, no threat is detected. Have you ever heard of this behavior?

The problem still occurs. But I noted, that it is only happening when using Windows Server 2019 - on Server 2016, the Windows Defender doesn't block the file.
In the Defender's log, nothing is logged.


Separate 32 bit and 64 bit prime tables, plus other minor improvements

In order to reduce memory usage, I've split the primes and modular inverses arrays into 32-bit and 64-bit segments.  This doesn't make a noticeable difference to performance, but reduces memory usage.
Other minor fixes:

Cached the value of sieve.additionalFactorsToEliminateCounts[sieveIteration] in a loop in doSieveTask.  Because it's an atomic it's expensive to access
Switched away from using std::array for the thread_local factorsCache.  This considerably improved performance of the presieve on Windows.
Changed the fix to AVX2 support, allowing the old hand edited assembly code to be used, which is a few % faster than what ISPC produces.

Very nice, AVX2 performance is better now and about 1.6 GB of memory is saved for PTL >= 2^32, which is always welcomed.
Weird, I read a lot of sources stating that Std Arrays are not slower than Raw Arrays and it seemed to be true in my benchmarks... But looks like that is indeed not true on Windows :|...
I did not find nor encounter any issue, thank you for this Pull Request!

I guess the problem is because of the way Windows handles thread_local variables.  In general I think std arrays are OK.
It might be a good idea to refactor to remove the thread local variables and just pass a context struct into the functions that need them.


Bluetooth issue

Hi, i have some problem with my DW1820A, your clover configuration and Catalina OS.
The bluetooth device board is recognized but I cannot connect any external device (keyboard, mouse, etc,.) Do you have any advice on this?
Thanks

The DW1820A cards are problematic so I do not recommend using them. Here's how it works:
Turn off the computer and unplug the power cord, and then wait 5-10 minutes.
Start an installed Windows 10 and install the official DELL driver for it: Dell Wireless 1820A Bluetooth Driver
Pair a Bluetooth device.
You can then reboot your PC and then you can successfully pair your device with macOS.

Thank you so much, your suggestion worked perfectly.


Fix compiler warnings

I went through all the warnings that cargo check produces on my machine, and fixed them.

When I run cargo check --all-targets I get a few more warnings, but I'm not quite sure what to do about them, so I'm ignoring them for now.  An example:
warning: non-local `impl` definition, they should be avoided as they go against expectation
   --> rkyv_test/src/lib.rs:135:27
    |
135 |         #[derive(Archive, Deserialize, Serialize, Debug, PartialEq)]
    |                           ^^^^^^^^^^^
    |
    = help: move this `impl` block outside the of the current function `archive_raw_identifiers`
    = note: an `impl` definition is non-local if it is nested inside an item and neither the type nor the trait are at the same nesting level as the `impl` block
    = note: one exception to the rule are anon-const (`const _: () = { ... }`) at top-level module and anon-const at the same nesting as the trait or type
    = note: this lint may become deny-by-default in the edition 2024 and higher, see the tracking issue <https://github.com/rust-lang/rust/issues/120363>
    = note: the derive macro `Deserialize` may come from an old version of the `rkyv_derive` crate, try updating your dependency with `cargo update -p rkyv_derive`
    = note: this warning originates in the derive macro `Deserialize` (in Nightly builds, run with -Z macro-backtrace for more info)

This is an open issue with the nightly toolchain. Let's wait for this issue to settle before we start addressing the warnings: https://github.com/rust-lang/rust/issues/121708

Waited long enough, sorry for the delay!

No worries. Thanks for merging.


CI can't find sol file , but works locally

Component
Forge
Have you ensured that all of these are up to date?

[X] Foundry
[X] Foundryup

What version of Foundry are you on?
forge 0.2.0 (63ed109 2022-08-09T00:20:50.211015Z)
What command(s) is the bug in?
forge test --gas-report
Operating System
Linux
Describe the bug
This job fails , but works locally. I am not sure why this happens and would love some assistance with it

hmm I wonder if there's something off with the root path, could you try two things:

log the cwd in CI script
add a foundry.toml file forge config --basic > foundry.toml or provide --root .

Thanks for responding so quickly. The repo already has a foundry.toml file. I will explore the other options

Here is the new run. I have provided the --root . argument , and it seems to be in the right directory. However , I still get the same erro.

looks weird, can you log forge config in CI
btw this file https://github.com/realiotech/realioverse-contracts/blob/fix/ci/contracts/src/LandNft.sol is called LandNft
but is used as "../LandNFT.sol"

Thanks I had a dirty git cache. closing this now


livecheck: Add ExtractPlist skip to SkipConditions

[x] Have you followed the guidelines in our Contributing document?
[x] Have you checked to ensure there aren't other open Pull Requests for the same change?
[x] Have you added an explanation of what your changes do and why you'd like us to include them?
[x] Have you written new tests for your changes? Here's an example.

[x] Have you successfully run brew typecheck with your changes locally?
[x] Have you successfully run brew tests with your changes locally?

When the --extract-plist option was added to livecheck (https://github.com/Homebrew/brew/pull/16759), conditions were added in #run_checks to skip casks using ExtractPlist if the --extract-plist isn't used and the run involves multiple formulae/casks. This integrates the skip into the SkipConditions class.
Besides that, this updates the language around the --extract-plist flag (usage description, code comments) to clarify the behavior. From the description, it would seem that the ExtractPlist strategy is only enabled when --extract-plist is used. Instead, livecheck automatically enables the strategy if the command is run on only one cask.

The --extract-plist option is only available for brew livecheck however adding this to SkipConditions will mean that this new skip condition will now affect brew audit and brew bump as well. Is that intentional?
I'm specifically looking at the changes to SkipConditions.referenced_skip_information which is used in those three commands.

The --extract-plist option is only available for brew livecheck however adding this to SkipConditions will mean that this new skip condition will now affect brew audit and brew bump as well. Is that intentional?

It wasn't intentional and I had simply forgotten that bump and cask audits also use SkipConditions, so thanks for the catch. I'll take a look at it tomorrow but, off the top of my head, if we want to maintain the existing behavior (not restricting ExtractPlist), it could be as simple as adding extract_plist: true to the related calls in those areas. Alternatively, we could have extract_plist default to true and specifically pass a false value in Livecheck#run_checks to trigger the skip.

Either of those options makes sense to me.

I updated the #skip_information and #referenced_skip_information methods to use true as the default extract_plist value, which will take care of the aforementioned issue. With this setup, we don't have to update the bump and audit calls (or any future calls outside of livecheck), which maybe seems a little better than the other option (adding an explicit extract_plist: true to those calls).

Thanks @samford!


Set up Customize Powerline in WSL works on Windows, not on Ubuntu 20.04

I follow the instructions but for some reason, in Ubuntu 20.04 it doesn't work.

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

ID: d3cbeb9a-9e7c-f8f4-5c03-5d6262d2b440
Version Independent ID: eb575c79-5b2a-45f6-646f-9d41d5d223e8
Content: Windows Terminal Powerline Setup
Content Source: TerminalDocs/tutorials/powerline-setup.md
Product: dev-environment
Technology: windows-terminal
GitHub Login: @cinnamon-msft
Microsoft Alias: cinnamon

Closing this as we did some major updates to the command prompt customizing tutorial (replacing the powerline tutorial with Oh My Posh): https://docs.microsoft.com/windows/terminal/tutorials/custom-prompt-setup
Please reopen or let me know if you see any issues with this new method of customizing your prompts. Thanks for the contribution!


[Issue #252] feat: sync txs on txs views

Description:
Accessing the button detail view and the dashboard now sync transactions.
Test plan:
Log in, go to the dashboard or the button detail view. Both should work normally.
Then delete all transactions by running:
$ yarn docker db
> DELETE FROM Transaction;

And access one of the pages. The transactions should appear back again.

Hmm the app isnt working for me...getting a 500 error on /api/dashboard. Not sure what is causing it. Any ideas? I have tried deleting and restarting everything...Is there a new .env config or something Im missing?

Tests failing.

Hmm the app isnt working for me...getting a 500 error on /api/dashboard. Not sure what is causing it. Any ideas? I have tried deleting and restarting everything...Is there a new .env config or something Im missing?

Does the error come with any message? It should take a couple of seconds to load, since it will be syncing the transaction again. The env file should have the three URLs, then you can just run make reset-dev in the branch and test it.

Also seeing these
grpc-bchrpc-node: failed network integrity check
error - Error: [DecimalError] Invalid argument: undefined

Working now, but Im not following the test plan...Do I need to install something or something? Whats yd?

Sorry, yd is yarn docker, is just an alias that I use (so much that I forgot it was not the actual name), fixed in the description.

Okay sweet thanks. Yeah looks good to me!
Test plan worked, was able to delete the rows and then they came back on page view.
I think we should definitely add some loading screens though. Not sure if there is already a task for it, but I can add and tackle in another pr if not

We don't have any tasks for loading screens / spinners/ etc but I was just thinking about that last week. Maybe make one for whatever scope you think is reasonable for a single branch and we can start there.

some ideas (in the short term):
on the client-side (dashboards): https://nextjs.org/docs/basic-features/data-fetching/client-side#client-side-data-fetching-with-swr
on the server-side (API Routes): add a redis service, let critical endpoints hit it first, if not, populate it after fetching

I think the performance hit on this one is just too much to be ok with.

yeah, I agree.

Maybe @ScottMcDermid or @Gattaca1 have some ideas for a better solution?
Like if I was to approach this, I might do something like:
* Sync address txs when accessed from the API (every time, before returning results). Remember that this will be protected behind AWS so we can choose how often the queries actually get through to the server.

The endpoint transactions/sync/[address] does that.
I think that the real solution here is using chronik, which will solve this once for all by syncing exactly the necessary & sufficient amount. Until then, I don't think we should spend too much time in another sub-optimal solution, so my suggestion is to just implement some button that does the sync, so that there is no impact on the performance in any views, and if the user feels like something is out of sync, they can confortably hit a button and know that the sync is happening.

I don't like the button idea and also we can't switch BCH to chronik so we do need a better solution anyway.
The solution needs to work for the client too - as in every time the API is hit, that needs to be showing live data (even more important than making sure the server is up to date technically).

I think we can close this, I will reimplement it by creating a task that uses https://www.grpc.cash/docs/bchrpc/#pb.SubscribeTransactionsRequest with the DB addresses, so that we get transactions notifications.

The solution needs to work for the client too - as in every time the API is hit, that needs to be showing live data (even more important than making sure the server is up to date technically).

I think in this case it suffices for the client to use the API endpoint I mentioned above, which will first sync all transactions before sending the data. I will also create a column lastSyncBlockHeight on the Network table so that we don't redundantly resync transactions that were already synced.

Ok. Sounds like a better way that will be closer to what we do with Chronik anyway.


[WireGuard] Multiple peers not showing up in configuration/handshakes

Describe the bug
I have two WireGuard servers I want to add, both are created with Algo and hosted on their own cloud servers. They don't have anything in common, so unique addresses and keys. When I create Local and Endpoint entries for each and enable WireGuard, it only ever lists the first one in List configuration and Handshakes. I expect to see both of them connected with their own interface (i.e. wg0 and wg1).
Screenshots

Environment
os-wireguard 1.7
OPNsense 21.7.1-amd64
FreeBSD 12.1-RELEASE-p19-HBSD
OpenSSL 1.1.1k 25 Mar 2021
Intel(R) Xeon(R) D-2146NT CPU @ 2.30GHz (16 cores)

Good bot.
I also have this bug. Will try to create using one of their templates.

Have you checked the logs? Because I do not think your vpn2 interface has even started; it should listen on a different port than vpn1.

51820
51821


Waypoint placement fix when rotated

Description

This PR fixes #557. That is, the waypoint is placed correctly even if the map is rotated. The PR also did some refactoring to extract the "screen click position to world position"-logic and added new tests for this logic.

Testing

There exist 9 new tests in ScreenToWorldMapperNoRotationTest.java and ScreenToWorldMapperRotationTest.java.
To test if the PR work during gameplay, you can do the following:

Start a new game.
Place a marker on the map and see that is placed correctly.
Go to a planet to introduce rotation of the world.
Place a marker again and see that it is placed correctly.

Notes

OBS The commits have a linked issues/PR. These are from the forked repo but GitHub seems to confuse them with the original repo. These linked issue/PR should be ignored.

Pre Pull Request Checklist:

[x] Code has been scanned with SonarLint
[x] There are no errors present in the project
[x] Code has been formatted and indented
[x] Methods have appropriate Javadoc (How to write Javadoc)

Would you mind checking Allow edits from maintainers on this PR so that I can resolve the conflicts and get this merged? Or if you're able to resolve them yourself it would be appreciated. Thanks :)

@NicholasBatesNZ Happy to hear that! :D Could not find the "Allow edits from maintainers", so instead, we made a merge and fixed the conflicts.

Thanks so much :)


Markdown parser

Implements custom md parser using remark.
Example usage:
// to get list of checks one need to perform
const { tree, currentCheckboxes, defaultCheckboxes } = await parseMd('- [ ] Content of the MD document');

// `defaultCheckboxes` would contain an array of checks one need to perform in this markdown (in their default state)
console.log(defaultCheckboxes)
[
  {
    type: 'listItem',
    spread: false,
    checked: false,
    children: [ [Object] ],
    position: { start: [Object], end: [Object] },
    path: '/children/1/children/0',
    conclusion: undefined
  },
  // ...
]

To flip the checkbox, you only need to set conclusion to one of the following strings:

"correct" – to indicate that the check is correct ✅
"critical" – to indicate that the check is incorrect ❌
"acceptable" – to indicate that the risk is acceptable ⚠
"not applicable" – to indicate that this check is not applicable

For example:
// Set the very first check as correct
defaultCheckboxes[0].conclusion = "correct"

// Render tree back into markdown
const markdown = await renderMd({ tree, checkboxes: defaultCheckboxes });

console.log(markdown)
// '- ✅ Content of the MD document'

I will continue the work in another PR, if needed (extracting extra comments and potentially rendering html?)


Update d3 to version 4.12.0 to remove contextify

Hi nick，I am berlin. I get a error when run npm install , you advised me to upgrading d3 to version 4.12.0 to remove contextify and it work. so I submit a pull request

Coverage remained the same at 62.14% when pulling 20e79658294097cf7e4b164afc617ce846cdd9bd on lbl-dev:upgrade-d3-to-version4.12-to-remove-contextify into a153e9d4c0ffba1c3a603846229d79e7ecad2fe7 on codecombat:master.


Webpack requires wrong version

I have an application which uses a plugin that I authored. In my app I require lodash@4, but my plugin requires lodash@3, both of are correctly specified in the two package.jsons.
When I import something from my plugin like this:
import * as deepAccess from "babel-plugin-check-data-access/lib/deepAccess"

the deepAccess function gets the lodash@4 from my main-app (node_modules/lodash/) instead of the one sitting in it's own folder: node_modules/babel-plugin-check-data-access/node_modules/lodash/.
The plugin with the require("lodash") call is here: https://github.com/fab1an/babel-plugin-check-data-access/blob/master/lib/deepAccess.js
My webpack-config:
const webpack = require('webpack'),
      path = require('path')

const SRC = path.resolve(__dirname, "src"),
      VENDOR = path.resolve(__dirname, "vendor"),
      NODE_MODULES = path.resolve(__dirname, "node_modules")

const babelSettings = {
  presets: ["es2015", "react"],
  plugins: [
    "check-data-access"
  ]
}

module.exports = {
  entry: [
    './src/index.js'
  ],
  module: {
    loaders: [{
      test: /\.jsx?$/,
      include: SRC,
      loader: `babel-loader?${JSON.stringify(babelSettings)}`
    }]
  },
  resolve: {
    root: [NODE_MODULES, SRC],
    extensions: ['', '.js', '.jsx'],
  },
  output: {
    path: path.resolve(__dirname, 'public', 'assets'),
    publicPath: '/assets',
    filename: 'bundle.js'
  },
};

root: [NODE_MODULES, SRC],

You specified that only your apps node_modules should be used. That causes the issue.

Thank you! What would be the correct line?

root: SRC