Images in reports

Images do not appear to be loading in reports - either as a file in the project, as a static resource served by the project, or an external image from a URL.  See https://github.com/nderwin-forks/quarkus-jasperreports/tree/feature/images-in-reports for examples.
The unit test is currently failing because the host URL needs to be dynamically passed to the report instead of it being hard-coded as http://localhost:8080.

can you submit the PR as draft so I can look?

Sure;  in the 6.x version, I had solved this with changes to the repository service;  the current one works much the same as mine did for external images, but that didn't seem to be working either.
If you fire up the integration-tests project in dev mode, then it'll produce the report, but all of the img tags have an empty string for the src attribute.

Thanks i will take a look.

I had just started looking at this, so not sure what the PDF renderer does.  I'll also eventually need the XLSX output to work.

ok i am looking at the samples. And they use SimpleHtmlExporter to export to a jrpint file in their the exporter it does this
private void setFileHandlers(File destFile)
	{
		File resourcesDir = new File(destFile.getParent(), destFile.getName() + "_files");
		String pathPattern = resourcesDir.getName() + "/{0}";
		resourceHandler = new FileHtmlResourceHandler(resourcesDir, pathPattern);
		imageHandler = resourceHandler;
		fontHandler = resourceHandler;
	}

So its registering a default File handler for images?

Also when I step through the debugger i see it loading the images properly in JRImageLoader

Good news - PDF and XLSX output works fine as-is, but that makes sense because the images are embedded.  HTML output will need some more work to provide an endpoint that serves the generated image files (like img_0_0_2.svg or img_0_0_4.gif).

2 things.

You should add one of these as a Hyperlink like... so we can kill two birds with one stone and test Hyperlink as well.

https://github.com/TIBCOSoftware/jasperreports/blob/a8ec0e6cbbf04d36878c8f783f10c2ff88afeff5/demo/samples/hyperlink/reports/HyperlinkReport.jrxml#L27C4-L31C14

If you want to Add a WEBP file you have to add this dependency to runtime module.

<dependency>
			<groupId>com.twelvemonkeys.imageio</groupId>
			<artifactId>imageio-webp</artifactId>
			<version>3.9.4</version>
</dependency>

https://github.com/TIBCOSoftware/jasperreports/blob/a8ec0e6cbbf04d36878c8f783f10c2ff88afeff5/demo/samples/images/reports/ImagesReport.jrxml#L210-L229

Figured out how to use the `FileHtmlResourceHandler` to dump the images for the HTML output into a temp folder, then provided an endpoint to serve them.  WEBP can be another iteration.

Closing this we don't need to test WEBP if that JAR  or any other TwelveMonkeys JAR is on the classpath it just registers with javax.imageio


[FIX] scroll during drag of first item

Fix list drag and scroll does weird things when the first item moves.

Hm, this seems to help in some cases. There is still a case where scroll moves because of first visible item, not necessarily first item overall. This at least helps in some of the cases.

fixed in f87906c598edf3ff0d89acff7dbf1914c94be669. It should apply to all cases where toPosition == 0.


[Bug] Missing random.choice in rotate

Hi,
Thank you for the wonderful work!
Just wanna share a tiny concern. It seems to me that here the magitude should be magnitude * random.choice([-1, 1], otherwise the rotate will always be in one direction.
https://github.com/DeepVoltaire/AutoAugment/blob/17d718251f25c0d9413bf30f91b523907924f33a/autoaugment.py#L208
Thank you.

Thank you for reporting the bug!


Incorrect constant names or values defined for time conversion

LeapHandController defines constants for time conversion:
const float 	NS_TO_S = 1e-6f
const float 	S_TO_NS = 1e6f

Be that as it may, a nanosecond is 1e-9 seconds not 1e-6 seconds. Either the constant name is wrong or the value is wrong.

Note, the constant is USED to convert microseconds to seconds, so it is the name, not the value that is wrong.


Upgrade to PostCSS 6.x

This requires a major version bump.

Hey! Published new version on npm. Thank U!


cc-wrapper: ensure existence of some $out/nix-support/*

Done because of f3f7612a409.  Now it's possible to simply use
$(< ${stdenv.cc}/nix-support/libcxx-cxxflags)

Motivation for this change
Things done

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

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

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

This sounds quite safe to me, so it seems more a question of which approach to choose exactly.  I'm now running rebuilds with it anyway.

Sorry! I made the branch with the fix and ran the tests, and then forgot to open the PR. Opened now with https://github.com/NixOS/nixpkgs/pull/92089

OK, better use that, I expect.  I've given it almost no thought.


InvalidArgumentException is thrown with ->fixPermissionsWithAcl()

Getting exception with acl:
 [InvalidArgumentException]
  The "setfacl" option is not defined.

https://github.com/EasyCorp/easy-deploy-bundle/blob/master/src/Deployer/DefaultDeployer.php#L46
I suppose that $this->getConfig('setfacl'), should be replaced with 'setfacl'

got the same error

There are two PR's with a fix. #28 and #45

Closing as resolved in #28


False-positive F401 when typing is used in comment

from typing import List, Callable, Tuple

keys = [
        ('IP', 'ip', str),
        ('Domain', 'domain', str)
]  # type: List[Tuple[str, str, Callable]]

ruff --isolated file.py (0.0.285) gives

file.py:1:20: F401 [*] typing.Callable imported but unused
file.py:1:30: F401 [*] typing.Tuple imported but unused
file.py:1:37: F401 [*] typing.List imported but unused
Found 3 errors.
[*] 3 potentially fixable with the --fix option.

But if i remove the imports and run mypy file.py (1.5.1) i get the following error:

file.py:1: error: Name "List" is not defined  [name-defined]
file.py:1: note: Did you forget to import it from "typing"? (Suggestion: "from typing import List")
Found 1 error in 1 file (checked 1 source file)

I believe this is the same as: https://github.com/astral-sh/ruff/issues/1619.


Feature Request: Return text instead of URI

Hi!
Could you please add a possibility to get text from QR code instead of URI to netbox?
Thank you very much!
meego

Hi @meegoSVK ! What text do you want to encode in a QR code image?


Replace deprecated unzipper with jszip

Unzipper uses deprecated module fstream (https://www.npmjs.com/package/fstream https://github.com/ZJONSSON/node-unzipper/issues/261), therefore we should replace it with jszip.
I will create a pull request to fix this deprecated dependency.

Unzipper uses deprecated module fstream

What is the impact for users here?
I am hesitating a bit to switch to jszip as it hasn't seen any contributions for 2 years while unzipper still seems to be actively maintained.

What is the impact for users here?

There are already several documented vulnerabilities in the transitive dependencies of fstream: https://github.com/ZJONSSON/node-unzipper/issues/261
The fstream package is officially marked as deprecated, so there is no maintenance ongoing anymore.
We use npm audit in our deployment process, so that's why we came across this. Another way would be to replace fstream with something else in unzipper, but it seemed easier to me to use another zip tool in the first place.
Actually you're right, jszip is probably not maintained much better. At least there are no known CVE or deprecations in its current dependency tree.
Do you have another suggestion how to deal with the deprecation of fstream?

https://gildas-lormeau.github.io/zip.js/ might be a good option. I could provide another MR using that too.

gildas-lormeau.github.io/zip.js might be a good option. I could provide another MR using that too.

I think this is a great option. Can we migrate to this package?

Yes, i will do that in the next few days.


Swift version ?

Are you planning to translate your code in swift? Thx

This has been asked before.  I don't have any plans at the moment, but I might in the future.  Sorry about that!


View model loses family when switching between tabs

With family open, switch to map and then go back to family page

Found that Google's documentation of LiveData had a big flaw.. It showed observers being added in a fragments onCreate, but because fragments are retained between attaching/detatching, only onCreateView and onDestroyView are called and the view will never be updated.
Solution is:
setup observers in onCreateView (or OnViewCreated)
mFamilyInformationViewModel.getCurrentFamily().observe(this, this);
REMOVE observers in onDestroyView (otherwise, adding observers won't trigger a callback)
mFamilyInformationViewModel.getCurrentFamily().removeObserver(this);

see: https://github.com/googlesamples/android-architecture-components/issues/47


Character Class should inherit from Entity Class?

Maybe I'm misunderstanding the intended design, but it seems like the Character class should inherit from the Entity class, no? This would allow us to reuse variables that either class may want to see. Such as m_character_description or m_origin etc.
Further, a Character is always an Entity, but an Entity may be many different things.
My intended use, for example: in Character.cpp the method void Character::sendDescription(BitStream &bs) const should reuse the m_character_description and m_battle_cry variables from Entity. I don't think we should make these variables static, so if not through inheritance, what's the best way to go about this?

Second issue first: Since cars or doors or npc do not have battle cries, nor character descriptions ( I think ? ) we could just move those 2 fields to Character ?

Hrm.. Doing that, and then converting uses from src.m_battle_cry to src.m_char.battle_cry causes the client to crash when more than one entity enters the world.

Can entities have titles? I moved those variables out into Character class as well. Maybe that's why I'm crashing?
Below makes it seem like Entity needs to check for titles. Not sure why that would give the Network Error though.
void sendTitles(const Entity &src,BitStream &bs)
{
    Character c = src.m_char;
    bs.StoreBits(1,c.m_has_titles); // Does entity have titles?
    if(!c.m_has_titles)
        return;
    if(src.m_type==Entity::ENT_PLAYER)
    {
        bs.StoreString(c.getName());
        bs.StoreBits(1,c.m_has_the_prefix);       // likely an index to a title prefix ( 0 - None; 1 - The )
        storeStringConditional(bs,c.m_titles[0]); // Title 1 - generic title (first)
        storeStringConditional(bs,c.m_titles[1]); // Title 2 - origin title (second)
        storeStringConditional(bs,c.m_titles[2]); // Title 3 - yellow title (special)
    }
    else // unused
    {
        bs.StoreString("");
        bs.StoreBits(1,0);
        storeStringConditional(bs,"");
        storeStringConditional(bs,"");
        storeStringConditional(bs,"");
    }
}

My working branch:
7adf86016d72880cf59833ec1f4d0c31216d02a1

I appear to have fixed it by changing StoreString here to storeStringConditional
void Character::sendTitles(BitStream &bs) const
{
    bs.StoreBits(1,m_has_the_prefix);   // likely an index to a title prefix ( 0 - None; 1 - The )
    storeStringConditional(bs, m_titles[0]);        // Title 1 - generic title (first)
    storeStringConditional(bs, m_titles[1]);        // Title 2 - origin title (second)
    storeStringConditional(bs, m_titles[2]);        // Title 3 - yellow title (special)
}


fix e2e watch mode

Signed-off-by: Ke Xu<EMAIL_ADDRESS>
Description

Checklist

[ ] I have added screenshots above for all UI changes.
[ ] I have added e2e tests for all UI changes.
[ ] Documentation was updated if it was needed.

Codecov Report

Merging #1740 (e82f338) into main (46bf1ff) will increase coverage by 8.17%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main    #1740      +/-   ##
==========================================
+ Coverage   88.51%   96.68%   +8.17%
==========================================
  Files          97       10      -87
  Lines        5344      151    -5193
==========================================
- Hits         4730      146    -4584
+ Misses        614        5     -609

Flag
Coverage Δ

unittests
96.68% <ø> (+8.17%)
:arrow_up:

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files
Coverage Δ

...sponsibleai/responsibleai/managers/base_manager.py

...ponsibleai/responsibleai/_tools/shared/versions.py

...ponsibleai/databalanceanalysis/balance_measures.py

responsibleai/responsibleai/_data_validations.py

erroranalysis/erroranalysis/_internal/__init__.py

...i/responsibleai/managers/counterfactual_manager.py

raiwidgets/raiwidgets/error_analysis_dashboard.py

...nalysis/erroranalysis/_internal/version_checker.py

erroranalysis/erroranalysis/_internal/constants.py

...ibleai/responsibleai/managers/explainer_manager.py

... and 77 more

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


Transform http url to https, except localhost urls

This PR follows the conversation with @leonardehrenfried : https://github.com/mediagis/nominatim-docker/issues/195#issuecomment-818929515
It changes all URL to https except for localhost.
It also fix a missing slash in localhost urls in READMEs

Thanks @Bakaniko!
@all-contributors please add @Bakaniko for docs and code.


Does not work with higher values for char codes

Some people use higher values for char codes in custom iconsets, for example, we have a icon font using block from \uf2001 to \uff223. But String.fromCharCode() does not work with them, change the method to String.fromCodePoint() can solve this problem.
https://github.com/oblador/react-native-vector-icons/blob/14be3c6f0570d607b57cc51d6bdb8c784f1cecc3/lib/create-icon-set.js#L63-L65
Convert all the decimal codes to chars before call createIconSet()  works well, too.

+1

This was eventually implemented


JFYI  a similar approach by Rancher with elemental-toolkit

Just for your information Suse/Rancher activly works on a similar idea in a gitops approach.
See elemental-toolkit

https://github.com/rancher/elemental-toolkit
https://rancher.github.io/elemental-toolkit/docs/

Could be an interesting source of inspiration for your project.
Regards.

Thanks @j-landru. I'm not sure it is that similar. It seems to me to be way more complex and the dependency on cloud-init is a no go as it requires python to be installed.


Issue installing qhull

CMake Error at scripts/cmake/vcpkg_build_cmake.cmake:175 (message):
Command failed: C:/Program Files/CMake/bin/cmake.exe;--build;.;--config;Release;--target;install;--;-v;-j1
Working Directory: I:/VCPKG/vcpkg/buildtrees/qhull/x64-windows-static-rel
See logs for more information:
I:\VCPKG\vcpkg\buildtrees\qhull\install-x64-windows-static-rel-out.log
Call Stack (most recent call first):
scripts/cmake/vcpkg_install_cmake.cmake:24 (vcpkg_build_cmake)
ports/qhull/portfile.cmake:24 (vcpkg_install_cmake)
scripts/ports.cmake:71 (include)
Error: Building package qhull:x64-windows-static failed with: BUILD_FAILED
Please ensure you're using the latest portfiles with .\vcpkg update, then
submit an issue at https://github.com/Microsoft/vcpkg/issues including:
Package: qhull:x64-windows-static
Vcpkg version: 0.0.113-nohash
vcpkg update says everythin gis up to date.

install-x64-windows-static-rel-out.log
my rel-out.log

I guess i solved it
Deleted the tmp QHull#####.rar file found in the vcpkg/Downloads/ area and rebuilt, seems to work now.

For reference, I had the same problem today while building qhull:x64-windows-static and deleting the qhull archive from vcpkg/Downloads fixed the issue for me as well.


Let /sensors index endpoint limit search to an asset (and sub-assets)

PR #1191 made the sensors API index endpoint more powerful, which we need for editing sensors_to_show, flex-context and flex-model.
We forgot to add the capability to search by asset (and sub-assets) which those features actually need.
I propose we add an asset parameter as well. Here is the new docstring (the beginning):
This endpoint returns all accessible sensors.

By default, "accessible sensors" means all sensors in the same account as the current user (if they have read permission to the account).

You can also specify any other `account` (an ID parameter), if the user has read-access to that.
Alternatively to filtering by account, we can filter by asset tree. If you send the `asset` parameter (an ID), then all sensors on that asset or its sub-assets are looked up (if the user has read access to that asset).

In any of the two custom filtering cases above, you can
- not filter by both account and asset, as that is not useful - if the asset is not part of the specified account, a 422 error is raised.
- add the parameter flag `include_public_assets`, which adds sensors under public assets, as well.  The default value is `false`.

Finally, you can simply use the powerful `all_accessible` parameter to list sensors from all assets that the `current_user` has read access to, (across accounts and assets), as well as all public assets. The default value is `false`.

(then we go on talking about pagination - actually the docstring can explain that a bit better, and also mention the filter and unit parameters)

Perhaps a search-child-assets flag (default true) should govern whether we also search for sensors on child assets of the asset that matches the search filter.

We do have an endpoint on assets/ID/sensors if that is all you need. Then we don't need to complicate this one any further.


devicesChangedEventStream does not broadcast AudioDevice Details on iOS 15

Flutter doctor: (I don't use android on Mac)
[✓] Flutter (Channel stable, 3.0.2, on macOS 12.4 21F79 darwin-arm, locale
    en-US)
[!] Android toolchain - develop for Android devices (Android SDK version
    32.1.0-rc1)
    ✗ cmdline-tools component is missing
      Run `path/to/sdkmanager --install "cmdline-tools;latest"`
      See https://developer.android.com/studio/command-line for more details.
    ✗ Android license status unknown.
      Run `flutter doctor --android-licenses` to accept the SDK licenses.
      See https://flutter.dev/docs/get-started/install/macos#android-setup for
      more details.
[✓] Xcode - develop for iOS and macOS (Xcode 13.4.1)
[✓] Chrome - develop for the web
[✓] Android Studio (version 2021.1)
[✓] VS Code (version 1.68.0)
[✓] Connected device (3 available)
[✓] HTTP Host Availability

Testing on a physical iPhone X running iOS 15.2 with generic wireless bluetooth earbuds
To replicate, active audio_session and listen to devicesChangedEvent
  void listen() {
    _audioSession.devicesChangedEventStream.listen((devicesChangedEvent) {
      print('AudioSession: Listener called with event. Added: ${devicesChangedEvent.devicesAdded}. Removed: ${devicesChangedEvent.devicesRemoved}');
    });
  }

The headphones are successfully detected by session.getDevices
When headphones are connected or disconnected, the stream is called. However, no devices are listed.

Would you be able to include the output for each event?

Would you be able to include the output for each event?

Hi,
The listener is called when a device is added or removed, but both devicesAdded and devicesRemoved return empty.
If you call session.getDevices after the listener is called, the updated device list appears successfully.
I'll do more testing tomorrow.
Thank you

Thanks for reporting the problem and confirming it.
I run into the same problem.
However, listeners are not called. (Even after adding Bluetooth, the listener is not called, so the reply is not void either)
I would like to know if this problem can be solved or if there is another solution.
I get the following output only when I start playback with just_audio (^0.9.29).
flutter: devices: {AudioDevice(id:Built-In Microphone,name:iPhone microphone,isInput:true,isOutput:false,type:AudioDeviceType.builtInMic), AudioDevice(id:Built-In Receiver,name:receiver,isInput:false,isOutput:true,type:AudioDeviceType.builtInEarpiece)}

Hi,
core.dart#96 should be declared outside the stream I think. I tried this and it worked, gives you the list of added/removed devices.

Oh you're right, that line is clearly a mistake. I will take a careful look tonight.

OK, so that line was intended to be moved down one line:
      final currentRoute = await _avAudioSession!.currentRoute;
      final previousRoute = _previousAVAudioSessionRoute ?? currentRoute;
      _previousAVAudioSessionRoute = currentRoute;

Can you test the above and let me know if it works for you?

Or easier, I have just pushed this commit to master. You can simply clone the latest code and see how it goes.
If all is well, I'll publish the next release.

Sure. I'm testing it out as we speak. Let me get back to you.

@ryanheise Works as expected! Thanks.

This has now been released to pub.dev as version 0.1.11. Thanks @sgowda97 for pointing out the line where I had made the mistake, and thanks @Colton127 for originally reporting the issue.

tested at version 0.1.13
empty set no device details.
[log] AudioSessionManagerImpl.Devices added: {}
[log] AudioSessionManagerImpl.Devices removed: {}


Fixing typo 'dipsatch'.

After fixing the merge conflict, the PR no longer show the typo. Closing.


Request run api with tgt_type error

When I called  'localhost:8000/run'  with the tgt_type param, I got the type error below.
File "/usr/lib/python2.6/site-packages/salt/client/init.py", line 535, in cmd\n    **kwargs):\nTypeError: get_cli_event_returns() got multiple values for keyword argument 'tgt_type'\n
I don't know if the tgt_type is supported, but I hope to call run with other target method.

The targeting argument name is expr_form. Try that instead. I know
tgt_type is still used in a few other places in Salt which is unfortunate
and confusing.
http://docs.saltstack.com/en/latest/ref/clients/index.html#salt.client.LocalClient.cmd
On Wed Jan 21 2015 at 12:22:09 AM Bing<EMAIL_ADDRESS>wrote:

When I called 'localhost:8000/run' with the tgt_type param, I got the
typeerror. I don't know if the tgt_type is supported, but I hope to call
run with other target method.
—
Reply to this email directly or view it on GitHub
https://github.com/saltstack/salt-api/issues/164.

Thank you very much, it worked.


Deeplink FAQ in the Settings to alphawallet.com/faq

https://github.com/AlphaWallet/alpha-wallet-ios/issues/2583

Second task for @mpaschenko @AW-STJ

@colourfreak @JamesSmartCell should I use internal browser like for dapps or external would be fine?

@colourfreak @JamesSmartCell should I use an internal browser like for dapps or external would be fine?

Please check out how this is implemented on iOS. @hboon


MTD, YTD, etc logic problem / question

Hello, it seems like the logic for doing any of the time based calculations for MTD, QTD, or YTD, relay on a hidden measure called "_ShowValuesForDates".  That makes sense in order to limit the dates that are using the in the formulas.  However, it seems like the logic for that hidden measure is flawed, or at least I don't understand the correct way to use it.  It seems that it always returns true, no matter what the date, since the "__FirstDateVisible" is always less than the last date with data.  I was able to work around this by adding another calculated column to the dates table that only returns true if that row's date is less than the max date in my data table and then adding that column as a filter on my page, but it seems like that should be built into the generated dates table somehow.  If there is another way I should be using this, please let me know.

The measure _ShowValuesForDates is created dynamically based on the Date columns selected in Bravo to generate the Date table. If you set a fixed range of years, that probably doesn't work automatically, and you should adjust that manually. Can you provide the _ShowValuesForDates definition you have in the model so we can validate whether this is your case?

Here is the code.  SubmittalBatch is my table and SubmittalDate is the date column I selected when generating the dates table.
_ShowValueForDates =  VAR __LastDateWithData =     CALCULATE (         MAXX ( { MAX ( 'SubmittalBatch'[SubmittalDate] ) }, ''[Value] ),         REMOVEFILTERS ()     ) VAR __FirstDateVisible =     MIN ( 'Date'[Date] ) VAR __Result =     __FirstDateVisible <= __LastDateWithData RETURN     __Result


chore: release  code.pianist 1.8.0

:robot: I have created a release
1.8.0 (2024-08-17)
Features

321321321 (e82735d)

This PR was generated with Release Please. See documentation.

:robot: Created releases:

code.pianist-v1.8.0
:sunflower:


Update model_endpoints_metadata.json

Add swin model to model endpoints metadata.

[ ] need to validate training
[ ] validate job's name

after discussion with @RussellJimmies and @MaxenceGui , we'll use the script instead of hand editing this.


Package resource name is mismatched.

Because of a ros recommendation, the last commit which I requested changed the package resource name as "hector_slam_example" from "hector-slam-example".
However, this causes a problem to build now.
Could you return the name to "hector-slam-example" at https://github.com/DaikiMaekawa/hector-slam-example/blob/master/package.xml#L3 and https://github.com/DaikiMaekawa/hector-slam-example/blob/master/CMakeLists.txt#L2 , or change the project name to "hector_slam_example" on GitHub?

:+1: I agree with your opinion. BTW,  did not care naming rule because this repository is sandbox.

Thank you so much!


Proper Age Validation Added in Sign-up form additional Details

Description
Proper Age Validation Added in Sign-up form additional Details
Fixes #126
Replace #126 with the issue number which is fixed in this PR
Have you read the Contributing Guidelines on Pull Requests?

[x] Yes
[ ] No

Type of change
Please delete options that are not relevant.

[x] Bug fix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the style guidelines(Clean Code) of this project
[x] I have performed a self-review of my own code
[x] My changes generate no new warnings
[x] I have added tests/screenshots(if any) that prove my fix is effective or that my feature works.

I am also a GSSOC'21 participant....Please review it and merge it with add the required labels associated with that program and related isuue(#126)
Hope you review it and merge it soon @HimeshNayak @avinashkranjan

Hope you review it and merge it soon @HimeshNayak @avinashkranjan


Fixed syntax errors and import

Table of Contents

Description
Motivation and Context
How Has This Been Tested?
Screenshots (if appropriate):
Types of changes
Checklist:

Description

Motivation and Context

How Has This Been Tested?

Screenshots (if appropriate):
Types of changes

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

Checklist:

[ ] My code follows the code style of this project.
[ ] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.
[ ] I have added tests to cover my changes.
[ ] All new and existing tests passed.

Hi Idan, thank you for submitting this PR.
I think it's a dupe of https://github.com/marvelapp/react-ab-test/pull/14, the README should be fixed now:
https://github.com/marvelapp/react-ab-test#standalone-component


fix deepkeys to work with empty arrays and objects

fix #104

CI is failing

You need to update the docs.

You also need to update index.d.ts docs.


Promote architecture call on 2019-03-26

Can someone with access please tweet about upcoming architecture call so it can get a little more visibility please? All details are in the MicroProfile calendar. Thanks!

@rsjames can we prioritize this to go out this week and 1 day before the call. The very first call for this project is next week. Thanks!

@xstefank Martin, thanks for prompt issue submission to enable internal prioritizing! :)

minimum requirements for the scheduled tweet that will be constant:

call takes place the 4th Tuesday in 2019

11am PDT

Minutes doc: https://docs.google.com/document/d/1uAKRymdAL5Wj1KADzrYKZzSnAFsq74lBfj8lxKQnI6I/edit?usp=sharing

video call:  https://bluejeans.com/3958254622

@rsjames designs approved, excellent and simple :) perfect!
description of the tweet, please schedule it twice a month, Thrusday before the call and Monday. Lets schedule it for 2019 covering all months up until Nov 2019!  Thanks
#Join #contribute to the @MicroProfileIO 💛architecture call that happens the 4th Tuesday of each month https://bluejeans.com/3958254622. # Proactively help open & close PRs during each meeting, it is a #funprocess. Add what matters to you to the agenda via minutes https://bit.ly/2UMWIBx

Is there something I need to do for this, or it's more an FYI?

@xstefank @kenfinnigan tweets are set for the rest of 2019 starting today.
Note that I love emojis, bias, the call must survive with 1 heart 💯

@kenfinnigan you can review the description and kill it adjust it before it goes live :) this is a fluid process yet we cannot wait for proactive feedback! own it!

I think it's fine. Hard to have anything more specific for something that covers a wide range of topics

yes, i notice that too :) thanks for 3rd eyes man!
@rstjames execute, lets do this/

Final Tweet:
Join & #contribute to the @MicroProfileIO 💛architecture call that happens the 4th Tuesday of each month https://bit.ly/2JE74Cm #Proactively help open & close PRs during each meeting. Add what matters to you to the agenda via minutes https://bit.ly/2UMWIBx
See you there! #OSS

looks good. Ryan, lets make it automatic for  twice a month up until the end of Nov 2019?
Please confirm that is possible via buffer, issues-- shoot :) thanks!
lets close this ticket today.

https://twitter.com/MicroProfileIO/status/1108784492955750401

Thank you @xstefank for creating the ticket and being explicit during the Community call about this call. :)
closing issue.

@aeiras @rstjames thank you! I wasn't expecting it to have such an impact :) Well done!

@xstefank by now you should know that we, you included, do not BS or waist time when enough beef is added to make something better. This is part of the fun and it wouldn't be possible without you owning it first! :) keep healthy and thanks!


RichTextLabel BB Code fill problem

Good afternoon,
I'm a Godot user for some time now (currently developing a GLES2 2D game for Android).
I'm using Godot Version: 3.2.3 stable with the following Devices/OS for developing:
a) Windows 10
AMD Ryzen 5 2400G
AMD Radeon RX Vega 11
b) Linux Mint 20.1
AMD Athlon 64 2X 3400+
AMD Radeon HD 5770
c) Windows 10
Intel Mobile Core 2 Duo T9300
NVIDIA GeForce G 105M
Effect identification and test were done in GLES2. The described effect is also present in Android devices (OS ranging from 5 to 8).
I use  RichTextLabel nodes to present information to players (help, copyright, statistics). Texts are quite long (scroll is activated) and are formatted using BB Code (alignment, images, ...). Every time I open my info scene, text formatted with fill BB Code presents some lines not correctly formatted (not aligned to the edges of RichTextLabel node). But as soon as I click on a icon in the scene to present again the same text (there are 3 icons to choose the option of text to be presented - help, copyright, statistics) the same text becomes correctly formatted.
Images ilustrating the problematic effect:
Fill format KO

Fill format OK (after click in icon to present again same text).

I have developed a small project (annexed TextTests.zip) to illustrate the problem. The first time the project is run the text is not fill formatted but as soon as you click the screen (note: needs mouse touch screen emulation on) the same text becomes fill formatted.
The same effect is present with or without custom fonts and at different font sizes. One interesting fact is that if you change the RichTextLabel width you can eliminate the problem (try making the RichTextLabel node larger in the annexed project - example 290).
If more information or action are necessary, please ask.
Best regards,
Paulo Reis
TextTests.zip

Good afternoon,
I have verified Godot version 3.2.4 Beta6 (using a Linux Mint 20.1 @ Athlon 64 2X 5600+ with GeForce GT 720 video card - my Godot non stable versions test system) and the reported effect also appears.
a) BB Code fill text KO b) BB Code fill text OK (after click in icon to present again same text)

Also: TextTests project give the same results in Godot version 3.2.4 Beta6.
Best regards.
Paulo Reis

Good afternoon,
I have verified Godot version 3.2.4 Beta6 (using a Linux Mint 20.1 @ Athlon 64 2X 5600+ with GeForce GT 720 video card - my Godot non stable versions test system) and the reported effect also appears.
a) BB Code fill text KO b) BB Code fill text OK (after click in icon to present again same text)

Also: TextTests project give the same results in Godot version 3.2.4 Beta6.
Best regards.
Paulo Reis

Good afternoon,
Godot 3.2.4 RC1 shows the same effect (texted using a Linux Mint 20.1 @ Athlon 64 2X 5600+ with GeForce GT 720 video card).
a) BB Code fill text KO b) BB Code fill text OK (after click in icon to present again same text)

Best regards.
Paulo Reis

Good afternoon,
Godot 3.2.4 RC1 shows the same effect (texted using a Linux Mint 20.1 @ Athlon 64 2X 5600+ with GeForce GT 720 video card).
a) BB Code fill text KO b) BB Code fill text OK (after click in icon to present again same text)

Best regards.
Paulo Reis

Good morning,
Godot 3.2.4 RC2 shows the same effect (tested using a Linux Mint 20.1 @ Athlon 64 2X 5600+ with GeForce GT 720 video card).
Best regards.
Paulo Reis

@lone72 Please stop bumping the issue to confirm on every RC. There haven't been code changes to RichTextLabel in the 3.2 branch for a while now.

As I tested, this issue is 3.2-specific and doesn't occur in 4.0. The problem is with the fill tag itself.
The bbcode_text I used for test:

[fill]This is a very long sentence which I made randomly and repeated several times. This is a very long sentence which I made randomly and repeated several times. This is a very long sentence which I made randomly and repeated several times.[/fill]

master

3.2.4.rc2

The fill formatting can be corrected by calling set_bbcode() again after entering the tree in 3.2.4.


bootstrap_jquery_url template tag calls slim version of jQuery

The template tag {% bootstrap_jquery_url %} currently calls the slim version of jQuery, which doesn't support quite a few features of the full jQuery version. See here:
https://github.com/zostera/django-bootstrap4/blob/master/bootstrap4/bootstrap.py#L19
I'm happy to issue a P.R. to include the full version of jQuery, but it raises the question of which direction to head:
Option (1): make the existing template tag point to the full version.
Option (2): make the existing template tag point to the full version, and add another template tag called {% bootstrap_jquery_slim_url %}
Option (3): add a template tag called {% bootstrap_jquery_full_url %} that points to the full version.
django-bootstrap3 currently points to the full version of jQuery.
Thoughts?

Option (4): Add jquery=falsy|slim|full (default=full) to the tag
falsy = no jquery
slim = slim jquery
full = full jquery (default)
This should also be passed on from the bootstrap_js template tag

Op 31 okt. 2017, om 20:53 heeft Timothy Allen<EMAIL_ADDRESS>het volgende geschreven:
The template tag {% bootstrap_jquery_url %} currently calls the slim version of jQuery, which doesn't support quite a few features of the full jQuery version. See here:
https://github.com/zostera/django-bootstrap4/blob/master/bootstrap4/bootstrap.py#L19 https://github.com/zostera/django-bootstrap4/blob/master/bootstrap4/bootstrap.py#L19
I'm happy to issue a P.R. to include the full version of jQuery, but it raises the question of which direction to head:
Option (1): make the existing template tag point to the full version.
Option (2): make the existing template tag point to the full version, and add another template tag called {% bootstrap_jquery_slim_url %}
Option (3): add a template tag called {% bootstrap_jquery_full_url %} that points to the full version.
django-bootstrap3 currently points to the full version of jQuery.
Thoughts?
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub https://github.com/zostera/django-bootstrap4/issues/43, or mute the thread https://github.com/notifications/unsubscribe-auth/AAFBi6BogyJL2ei1yXvXrqtrjZkjaXEYks5sx3qmgaJpZM4QNVFf.

P.R. issued: https://github.com/zostera/django-bootstrap4/pull/44
Thanks, as alawys, @dyve!

...and I just noticed that there was a final line to your message that wasn't on my secondary monitor! Another P.R. will be coming soon for the bootstrap_javascript tag.

This has been address with https://github.com/zostera/django-bootstrap4/pull/44
Thank you!


Create an Architectural Decision Record document to explain the migration from Wrangler to Controller Runtime

This card is about creating a simple document to explain and showcase the motivations of moving from Wrangler framework to Controller Runtime framework in elemental-operator.
The idea is to mimic the procedure described in https://github.com/rancher/rke2/tree/master/docs/adrs#architectural-decision-records-adrs.
For that I'd suggest to create a new discussion in https://github.com/rancher/elemental/discussions trying to briefly cover three aspects:

The motivation (upstream community, maintainability, testing, etc.)
Rough steps on the process to achieve it (swap controllers one by one in a parallel branch, prepare current ci pipes on main for the new branch, links to the draft PRs, etc.)
Few thoughts about the impact of the change over the product (is that implying changes on the API, backward compatibility, etc.)

https://github.com/rancher/elemental/discussions/409

Closing, there is already a ADR posted as a discussion within the repository


Fix custom verb issue

Signed-off-by: Wayne Zhang<EMAIL_ADDRESS>To fix:  https://github.com/grpc-ecosystem/grpc-httpjson-transcoding/issues/49
The root problem is:  current implementation is treating custom verb as last segment; replace ":" with "/".  It has limitation.
Changes:

treat custom verb as part of http_method; append custom verb to http_method for register and lookup,
not to append verb as last segment.
variable.end_segment needs to be updated too.

@nareddyt could you help to review it too?  Somehow I could not add you.


MySIMBL install on Sierra  - Mail Glitch

I installed MySIMBL and while it works, now my Apple Mail is failing to open file/save file.  I went through this once before, and the issue was the MySIMBL - I reinstalled it today hoping the bug was fixed, but no luck - back to same issues.

Hmm. It's probably due to a specific plugin but I don't know what. Does this happen with no plugins enabled? Maybe just blacklist Mail in the SIMBL tab.


[BUG] Transform job failing if indices are deleted while executing the search phase

What is the bug?
While the search phase for the transform job is executing, if any of the indices that are part of the source index are deleted then the transform job fails with the following error - Failed to search data in source indices and gets disabled.
How can one reproduce the bug?
Steps to reproduce the behavior:

Create a transform job with source index as a pattern.
Before the transform job is about to get executed (we can check when it was last run and based on the frequency we can get when it will be executed next) start deleting a few indices of the source index pattern
Once the transform job executes, it fails with the error as described in the above section.

What is the expected behavior?
Ignore the search failure that had occurred for the deleted index and continue with the remaining results.
What is your host/environment?
Tested in version 1.3
Do you have any screenshots?
N/A
Do you have any additional context?
Stack trace of the error
[2024-04-25T10:49:28,130][WARN ][o.o.i.t.TransformSearchService] [62ad7c2d69ee53adcacae4d2dd88ed7d] Operation failed. Retrying in 1s.
Failed to execute phase [query], Partial shards failure (1 shards unavailable)
        at org.opensearch.action.search.AbstractSearchAsyncAction.onPhaseFailure(AbstractSearchAsyncAction.java:667)
        at org.opensearch.action.search.AbstractSearchAsyncAction.executeNextPhase(AbstractSearchAsyncAction.java:404)
        at org.opensearch.action.search.AbstractSearchAsyncAction.onPhaseDone(AbstractSearchAsyncAction.java:702)
        at org.opensearch.action.search.AbstractSearchAsyncAction.successfulShardExecution(AbstractSearchAsyncAction.java:587)
        at org.opensearch.action.search.AbstractSearchAsyncAction.onShardResultConsumed(AbstractSearchAsyncAction.java:575)
        at org.opensearch.action.search.AbstractSearchAsyncAction.lambda$onShardResult$9(AbstractSearchAsyncAction.java:558)
        at org.opensearch.action.search.QueryPhaseResultConsumer$PendingMerges.consume(QueryPhaseResultConsumer.java:366)
        at org.opensearch.action.search.QueryPhaseResultConsumer.consumeResult(QueryPhaseResultConsumer.java:130)
        at org.opensearch.action.search.AbstractSearchAsyncAction.onShardResult(AbstractSearchAsyncAction.java:558)
        at org.opensearch.action.search.SearchQueryThenFetchAsyncAction.onShardResult(SearchQueryThenFetchAsyncAction.java:150)
        at org.opensearch.action.search.AbstractSearchAsyncAction$1.innerOnResponse(AbstractSearchAsyncAction.java:286)
        at org.opensearch.action.search.SearchActionListener.onResponse(SearchActionListener.java:57)
        at org.opensearch.action.search.SearchActionListener.onResponse(SearchActionListener.java:42)
        at org.opensearch.action.search.SearchExecutionStatsCollector.onResponse(SearchExecutionStatsCollector.java:79)
        at org.opensearch.action.search.SearchExecutionStatsCollector.onResponse(SearchExecutionStatsCollector.java:49)
        at org.opensearch.action.ActionListenerResponseHandler.handleResponse(ActionListenerResponseHandler.java:67)
        at org.opensearch.action.search.SearchTransportService$ConnectionCountingHandler.handleResponse(SearchTransportService.java:584)
        at org.opensearch.transport.TransportService$6.handleResponse(TransportService.java:736)
        at org.opensearch.security.transport.SecurityInterceptor$RestoringTransportResponseHandler.handleResponse(SecurityInterceptor.java:306)
        at org.opensearch.transport.TransportService$ContextRestoreResponseHandler.handleResponse(TransportService.java:1347)
        at org.opensearch.transport.TransportService$DirectResponseChannel.processResponse(TransportService.java:1425)
        at org.opensearch.transport.TransportService$DirectResponseChannel.sendResponse(TransportService.java:1405)
        at org.opensearch.transport.TaskTransportChannel.sendResponse(TaskTransportChannel.java:65)
        at org.opensearch.action.support.ChannelActionListener.onResponse(ChannelActionListener.java:57)
        at org.opensearch.action.support.ChannelActionListener.onResponse(ChannelActionListener.java:40)
        at org.opensearch.action.ActionRunnable.lambda$supply$0(ActionRunnable.java:71)
        at org.opensearch.action.ActionRunnable$2.doRun(ActionRunnable.java:86)
        at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:50)
        at org.opensearch.threadpool.TaskAwareRunnable.doRun(TaskAwareRunnable.java:78)
        at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:50)
        at org.opensearch.common.util.concurrent.TimedRunnable.doRun(TimedRunnable.java:57)
        at org.opensearch.common.util.concurrent.ThreadContext$ContextPreservingAbstractRunnable.doRun(ThreadContext.java:809)
        at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:50)
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
        at java.base/java.lang.Thread.run(Thread.java:829)

[2024-04-25T10:50:10,913][DEBUG][o.o.a.s.TransportSearchAction] [62ad7c2d69ee53adcacae4d2dd88ed7d] [q_TaXY7OQBKUZ1s0YJhIYw][metrics-2024-04-05][4]: Failed to execute [SearchRequest{searchType=QUERY_THEN_FETCH, indices=[metrics-*], indicesOptions=IndicesOptions[ignore_unavailable=false, allow_no_indices=true, expand_wildcards_open=true, expand_wildcards_closed=false, expand_wildcards_hidden=false, allow_aliases_to_multiple_indices=true, forbid_closed_indices=true, ignore_aliases=false, ignore_throttled=true], types=[], routing='null', preference='null', requestCache=null, scroll=null, maxConcurrentShardRequests=0, batchedReduceSize=512, preFilterShardSize=null, allowPartialSearchResults=false, localClusterAlias=null, getOrCreateAbsoluteStartMillis=-1, ccsMinimizeRoundtrips=true, source={"size":0,"query":{"bool":{"must":[{"match_all":{"boost":1.0}}],"should":[{"bool":{"filter":[{"range":{"timestamp":{"from":1714039200000,"to":1714042800000,"include_lower":true,"include_upper":false,"time_zone":"UTC","format":"epoch_millis","boost":1.0}}},{"terms":{"metricname.keyword":["Speed_Out"],"boost":1.0}},{"terms":{"unit.keyword":["southern"],"boost":1.0}},{"terms":{"namespace.keyword":["attorney"],"boost":1.0}},{"terms":{"dim_instance.keyword":["Republican"],"boost":1.0}},{"terms":{"dim_instance_name.keyword":["national"],"boost":1.0}},{"terms":{"dim_account.keyword":["apply"],"boost":1.0}},{"terms":{"dim_account_id":[95750.0625],"boost":1.0}},{"terms":{"dim_name.keyword":["Christine Fox"],"boost":1.0}}],"adjust_pure_negative":true,"boost":1.0}},{"bool":{"filter":[{"range":{"timestamp":{"from":1714039200000,"to":1714042800000,"include_lower":true,"include_upper":false,"time_zone":"UTC","format":"epoch_millis","boost":1.0}}},{"terms":{"metricname.keyword":["Speed_Out"],"boost":1.0}},{"terms":{"unit.keyword":["street"],"boost":1.0}},{"terms":{"namespace.keyword":["think"],"boost":1.0}},{"terms":{"dim_instance.keyword":["foreign"],"boost":1.0}},{"terms":{"dim_instance_name.keyword":["thing"],"boost":1.0}},{"terms":{"dim_account.keyword":["give"],"boost":1.0}},{"terms":{"dim_account_id":[73027.78125],"boost":1.0}},{"terms":{"dim_name.keyword":["Andrea Jones"],"boost":1.0}}],"adjust_pure_negative":true,"boost":1.0}},{"bool":{"filter":[{"range":{"timestamp":{"from":1714039200000,"to":1714042800000,"include_lower":true,"include_upper":false,"time_zone":"UTC","format":"epoch_millis","boost":1.0}}},{"terms":{"metricname.keyword":["Speed_Out"],"boost":1.0}},{"terms":{"unit.keyword":["stop"],"boost":1.0}},{"terms":{"namespace.keyword":["others"],"boost":1.0}},{"terms":{"dim_instance.keyword":["address"],"boost":1.0}},{"terms":{"dim_instance_name.keyword":["toward"],"boost":1.0}},{"terms":{"dim_account.keyword":["wall"],"boost":1.0}},{"terms":{"dim_account_id":[5028.21240234375],"boost":1.0}},{"terms":{"dim_name.keyword":["Reginald Steele"],"boost":1.0}}],"adjust_pure_negative":true,"boost":1.0}},{"bool":{"filter":[{"range":{"timestamp":{"from":1714039200000,"to":1714042800000,"include_lower":true,"include_upper":false,"time_zone":"UTC","format":"epoch_millis","boost":1.0}}},{"terms":{"metricname.keyword":["Speed_Out"],"boost":1.0}},{"terms":{"unit.keyword":["special"],"boost":1.0}},{"terms":{"namespace.keyword":["service"],"boost":1.0}},{"terms":{"dim_instance.keyword":["research"],"boost":1.0}},{"terms":{"dim_instance_name.keyword":["former"],"boost":1.0}},{"terms":{"dim_account.keyword":["out"],"boost":1.0}},{"terms":{"dim_account_id":[7327.4951171875],"boost":1.0}},{"terms":{"dim_name.keyword":["Amy Roach"],"boost":1.0}}],"adjust_pure_negative":true,"boost":1.0}},{"bool":{"filter":[{"range":{"timestamp":{"from":1714039200000,"to":1714042800000,"include_lower":true,"include_upper":false,"time_zone":"UTC","format":"epoch_millis","boost":1.0}}},{"terms":{"metricname.keyword":["Speed_Out"],"boost":1.0}},{"terms":{"unit.keyword":["speech"],"boost":1.0}},{"terms":{"namespace.keyword":["response"],"boost":1.0}},{"terms":{"dim_instance.keyword":["government"],"boost":1.0}},{"terms":{"dim_instance_name.keyword":["produce"],"boost":1.0}},{"terms":{"dim_account.keyword":["street"],"boost":1.0}},{"terms":{"dim_account_id":[62391.12890625],"boost":1.0}},{"terms":{"dim_name.keyword":["Jerry Nunez"],"boost":1.0}}],"adjust_pure_negative":true,"boost":1.0}},{"bool":{"filter":[{"range":{"timestamp":{"from":1714039200000,"to":1714042800000,"include_lower":true,"include_upper":false,"time_zone":"UTC","format":"epoch_millis","boost":1.0}}},{"terms":{"metricname.keyword":["Speed_Out"],"boost":1.0}},{"terms":{"unit.keyword":["sport"],"boost":1.0}},{"terms":{"namespace.keyword":["imagine"],"boost":1.0}},{"terms":{"dim_instance.keyword":["country"],"boost":1.0}},{"terms":{"dim_instance_name.keyword":["into"],"boost":1.0}},{"terms":{"dim_account.keyword":["air"],"boost":1.0}},{"terms":{"dim_account_id":[58682.67578125],"boost":1.0}},{"terms":{"dim_name.keyword":["Paul Martin"],"boost":1.0}}],"adjust_pure_negative":true,"boost":1.0}},{"bool":{"filter":[{"range":{"timestamp":{"from":1714039200000,"to":1714042800000,"include_lower":true,"include_upper":false,"time_zone":"UTC","format":"epoch_millis","boost":1.0}}},{"terms":{"metricname.keyword":["Speed_Out"],"boost":1.0}},{"terms":{"unit.keyword":["standard"],"boost":1.0}},{"terms":{"namespace.keyword":["realize"],"boost":1.0}},{"terms":{"dim_instance.keyword":["party"],"boost":1.0}},{"terms":{"dim_instance_name.keyword":["attack"],"boost":1.0}},{"terms":{"dim_account.keyword":["international"],"boost":1.0}},{"terms":{"dim_account_id":[11613.3173828125],"boost":1.0}},{"terms":{"dim_name.keyword":["Andrea Thomas"],"boost":1.0}}],"adjust_pure_negative":true,"boost":1.0}},{"bool":{"filter":[{"range":{"timestamp":{"from":1714039200000,"to":1714042800000,"include_lower":true,"include_upper":false,"time_zone":"UTC","format":"epoch_millis","boost":1.0}}},{"terms":{"metricname.keyword":["Speed_Out"],"boost":1.0}},{"terms":{"unit.keyword":["speech"],"boost":1.0}},{"terms":{"namespace.keyword":["energy"],"boost":1.0}},{"terms":{"dim_instance.keyword":["well"],"boost":1.0}},{"terms":{"dim_instance_name.keyword":["officer"],"boost":1.0}},{"terms":{"dim_account.keyword":["south"],"boost":1.0}},{"terms":{"dim_account_id":[53121.55078125],"boost":1.0}},{"terms":{"dim_name.keyword":["Daniel Phillips"],"boost":1.0}}],"adjust_pure_negative":true,"boost":1.0}},{"bool":{"filter":[{"range":{"timestamp":{"from":1714039200000,"to":1714042800000,"include_lower":true,"include_upper":false,"time_zone":"UTC","format":"epoch_millis","boost":1.0}}},{"terms":{"metricname.keyword":["Speed_Out"],"boost":1.0}},{"terms":{"unit.keyword":["speak"],"boost":1.0}},{"terms":{"namespace.keyword":["expect"],"boost":1.0}},{"terms":{"dim_instance.keyword":["great"],"boost":1.0}},{"terms":{"dim_instance_name.keyword":["loss"],"boost":1.0}},{"terms":{"dim_account.keyword":["professional"],"boost":1.0}},{"terms":{"dim_account_id":[84024.0078125],"boost":1.0}},{"terms":{"dim_name.keyword":["Raymond Spencer"],"boost":1.0}}],"adjust_pure_negative":true,"boost":1.0}},{"bool":{"filter":[{"range":{"timestamp":{"from":1714039200000,"to":1714042800000,"include_lower":true,"include_upper":false,"time_zone":"UTC","format":"epoch_millis","boost":1.0}}},{"terms":{"metricname.keyword":["Speed_Out"],"boost":1.0}},{"terms":{"unit.keyword":["speak"],"boost":1.0}},{"terms":{"namespace.keyword":["item"],"boost":1.0}},{"terms":{"dim_instance.keyword":["light"],"boost":1.0}},{"terms":{"dim_instance_name.keyword":["account"],"boost":1.0}},{"terms":{"dim_account.keyword":["popular"],"boost":1.0}},{"terms":{"dim_account_id":[48242.03515625],"boost":1.0}},{"terms":{"dim_name.keyword":["Leslie Moore"],"boost":1.0}}],"adjust_pure_negative":true,"boost":1.0}}],"adjust_pure_negative":true,"minimum_should_match":"1","boost":1.0}},"track_total_hits":-1,"aggregations":{"sample_transform":{"composite":{"size":10,"sources":[{"timestamp":{"date_histogram":{"field":"timestamp","missing_bucket":true,"order":"asc","calendar_interval":"1h","time_zone":"UTC"}}},{"metricname":{"terms":{"field":"metricname.keyword","missing_bucket":true,"order":"asc"}}},{"unit":{"terms":{"field":"unit.keyword","missing_bucket":true,"order":"asc"}}},{"namespace":{"terms":{"field":"namespace.keyword","missing_bucket":true,"order":"asc"}}},{"dim_instance":{"terms":{"field":"dim_instance.keyword","missing_bucket":true,"order":"asc"}}},{"dim_instance_name":{"terms":{"field":"dim_instance_name.keyword","missing_bucket":true,"order":"asc"}}},{"dim_account":{"terms":{"field":"dim_account.keyword","missing_bucket":true,"order":"asc"}}},{"dim_account_id":{"terms":{"field":"dim_account_id","missing_bucket":true,"order":"asc"}}},{"dim_name":{"terms":{"field":"dim_name.keyword","missing_bucket":true,"order":"asc"}}}]},"aggregations":{"sum":{"sum":{"field":"value"}},"avg":{"avg":{"field":"value"}},"max":{"max":{"field":"value"}},"min":{"min":{"field":"value"}},"count":{"value_count":{"field":"value"}}}}}}, cancelAfterTimeInterval=null}] lastShard [true]

[2024-04-25T10:50:10,976][DEBUG][o.o.a.s.TransportSearchAction] [62ad7c2d69ee53adcacae4d2dd88ed7d] #[null,java.lang.NullPointerException]#4 shards failed for phase: [query]
[clusterState.metadata.index(index) must not be null]; nested: NullPointerException[clusterState.metadata.index(index) must not be null];
        at org.opensearch.OpenSearchException.guessRootCauses(OpenSearchException.java:695)
        at org.opensearch.action.search.AbstractSearchAsyncAction.executeNextPhase(AbstractSearchAsyncAction.java:383)
        at org.opensearch.action.search.AbstractSearchAsyncAction.onPhaseDone(AbstractSearchAsyncAction.java:702)
        at org.opensearch.action.search.AbstractSearchAsyncAction.successfulShardExecution(AbstractSearchAsyncAction.java:587)
        at org.opensearch.action.search.AbstractSearchAsyncAction.onShardResultConsumed(AbstractSearchAsyncAction.java:575)
        at org.opensearch.action.search.AbstractSearchAsyncAction.lambda$onShardResult$9(AbstractSearchAsyncAction.java:558)
        at org.opensearch.action.search.QueryPhaseResultConsumer$PendingMerges.consume(QueryPhaseResultConsumer.java:366)
        at org.opensearch.action.search.QueryPhaseResultConsumer.consumeResult(QueryPhaseResultConsumer.java:130)
        at org.opensearch.action.search.AbstractSearchAsyncAction.onShardResult(AbstractSearchAsyncAction.java:558)
        at org.opensearch.action.search.SearchQueryThenFetchAsyncAction.onShardResult(SearchQueryThenFetchAsyncAction.java:150)
        at org.opensearch.action.search.AbstractSearchAsyncAction$1.innerOnResponse(AbstractSearchAsyncAction.java:286)
        at org.opensearch.action.search.SearchActionListener.onResponse(SearchActionListener.java:57)
        at org.opensearch.action.search.SearchActionListener.onResponse(SearchActionListener.java:42)
        at org.opensearch.action.search.SearchExecutionStatsCollector.onResponse(SearchExecutionStatsCollector.java:79)
        at org.opensearch.action.search.SearchExecutionStatsCollector.onResponse(SearchExecutionStatsCollector.java:49)
        at org.opensearch.action.ActionListenerResponseHandler.handleResponse(ActionListenerResponseHandler.java:67)
        at org.opensearch.action.search.SearchTransportService$ConnectionCountingHandler.handleResponse(SearchTransportService.java:584)
        at org.opensearch.transport.TransportService$6.handleResponse(TransportService.java:736)
        at org.opensearch.security.transport.SecurityInterceptor$RestoringTransportResponseHandler.handleResponse(SecurityInterceptor.java:306)
        at org.opensearch.transport.TransportService$ContextRestoreResponseHandler.handleResponse(TransportService.java:1347)
        at org.opensearch.transport.TransportService$DirectResponseChannel.processResponse(TransportService.java:1425)
        at org.opensearch.transport.TransportService$DirectResponseChannel.sendResponse(TransportService.java:1405)
        at org.opensearch.transport.TaskTransportChannel.sendResponse(TaskTransportChannel.java:65)
        at org.opensearch.action.support.ChannelActionListener.onResponse(ChannelActionListener.java:57)
        at org.opensearch.action.support.ChannelActionListener.onResponse(ChannelActionListener.java:40)
        at org.opensearch.action.ActionRunnable.lambda$supply$0(ActionRunnable.java:71)
        at org.opensearch.action.ActionRunnable$2.doRun(ActionRunnable.java:86)
        at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:50)
        at org.opensearch.threadpool.TaskAwareRunnable.doRun(TaskAwareRunnable.java:78)
        at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:50)
        at org.opensearch.common.util.concurrent.TimedRunnable.doRun(TimedRunnable.java:57)
        at org.opensearch.common.util.concurrent.ThreadContext$ContextPreservingAbstractRunnable.doRun(ThreadContext.java:809)
        at org.opensearch.common.util.concurrent.AbstractRunnable.run(AbstractRunnable.java:50)
        at java.base/java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1128)
        at java.base/java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:628)
        at java.base/java.lang.Thread.run(Thread.java:829)
Caused by: java.lang.NullPointerException: clusterState.metadata.index(index) must not be null
        at org.opensearch.indexmanagement.rollup.util.RollupUtilsKt.isRollupIndex(RollupUtils.kt:69)
        at org.opensearch.indexmanagement.rollup.interceptor.RollupInterceptor$interceptHandler$1.messageReceived(RollupInterceptor.kt:81)
        at org.opensearch.performanceanalyzer.transport.PerformanceAnalyzerTransportRequestHandler.messageReceived(PerformanceAnalyzerTransportRequestHandler.java:64)
        at org.opensearch.transport.RequestHandlerRegistry.processMessageReceived(RequestHandlerRegistry.java:94)
        at org.opensearch.transport.TransportService.sendLocalRequest(TransportService.java:938)
        at org.opensearch.transport.TransportService.access$100(TransportService.java:92)
        at org.opensearch.transport.TransportService$3.sendRequest(TransportService.java:151)
        at org.opensearch.transport.TransportService.sendRequestInternal(TransportService.java:876)
        at org.opensearch.security.transport.SecurityInterceptor.sendRequestDecorate(SecurityInterceptor.java:209)
        at org.opensearch.security.OpenSearchSecurityPlugin$7$2.sendRequest(OpenSearchSecurityPlugin.java:665)
        at com.amazonaws.elasticsearch.iam.IamTransportRequestSender.sendRequest(IamTransportRequestSender.java:94)
        at com.amazonaws.elasticsearch.ccs.CrossClusterRequestInterceptor$AddHeaderSender.sendRequest(CrossClusterRequestInterceptor.java:132)
        at org.opensearch.transport.TransportService.sendRequest(TransportService.java:763)
        at org.opensearch.transport.TransportService.sendChildRequest(TransportService.java:838)
        at org.opensearch.transport.TransportService.sendChildRequest(TransportService.java:826)
        at org.opensearch.action.search.SearchTransportService.sendExecuteQuery(SearchTransportService.java:196)
        at org.opensearch.action.search.SearchQueryThenFetchAsyncAction.executePhaseOnShard(SearchQueryThenFetchAsyncAction.java:124)

Thanks @rayshrey for opening this.

While the search phase for the transform job is executing, if any of the indices that are part of the source index are deleted

I think you mean when the source index is provided as a pattern, and then any of the resolved indices are deleted. Although, I suspect it would fail even if the only provided source index is deleted as well.

I think you mean when the source index is provided as a pattern, and then any of the resolved indices are deleted.

Yes, exactly

Although, I suspect it would fail even if the only provided source index is deleted as well.

Yes will need to check this as well.

Any idea on which releases this affect?
@rayshrey Can you please add the version you are testing

@karthikeyan21 I tested in version 1.3 (Updated the issue details as well)

Catch All Triage - 1 2 3 4 5 6


Sort available attributes.

Change availableAttributes from Set to List, so that the list of the available
attributes is/can be sorted.

@mmoayyed @tsschmidt Any comment?

This is fine with me, thank you; I don't know if Travis plans to cut another 5.3.x release though.

Thank you. I will do a PR for 6.0.x and master tree also.


Comments in response file

I've added this feature where you can have comments delimited by # and newlines in the response file (dmd @cmdfile). It works on separate lines, and when whitespace separates it from an argument.

Also needs a pull for the documentation to describe this.

Is there any further documentation I should add?


Bidirectional slash in templates

Often we may want to use bidirectional slash, A|B, which means both A\B and A/B, as syntactic sugar in templates, for, usually, the direction of slash does not matter when writing templates. E.g., the entries for (S\NP)/NP and (S\NP)\NP etc can be replaced by the single entry for (S|NP)|NP, which is more general and applicable to both English and Japanese.

Thank you for the feature suggestion! I will try to implement it this week.

@pasmargo I think it is ready to merge this branch to master. Now the templates (en/semantic_templates_en_event.yaml) are changed to a vertical version, which works better. For Japanese, the templates are the same as before.

Thank you for checking the branch!
I have merged it.


Request: Provide TF 2.3 in ml container / Provide Jupyter in TF / Torch containers

Hi, thanks for the excellent work here, but might I suggest you provide the ml-container with the more current tensorflow 2.3 (possibly as an option) rather that the (outdated) tensorflow 1.15?
Furthermore, I believe it would be fantastic if you could provide the additional ml packages, especially Jupyter, also in the tensorflow respectively pytorch pacakges.
Best regards,
Reinhold

Hi @dspyrhsu , thanks for the feedback!   We will have to look at the number of users on TF 1.x vs TF 2.x and determine when to switch over the l4t-ml container.
Regarding l4t-pytorch and l4t-tensorflow, we do try to keep those containers smaller (whereas there is not as big an emphasis on that in the l4t-ml container).
All of the dockerfiles and build scripts for these containers is public though, so feel free to modify them and build your own containers:   https://github.com/dusty-nv/jetson-containers

I have a fork of this repo that builds two ML images, one tagged with each version of TensorFlow. It seems like you could push both to NGC and let people download either or both. There's a distressing amount of software out there that only works with 1.15, so it needs to be "always" available.

Hi, thanks for getting back to me. I actually just built a version of l4t-ml with tf2.3 instead of tf1.15 by modifying the last line of the build script for all. In order for this to work with sklearn, I have to pass an environment variable when running the container (see here for the arising issue and solution: https://github.com/keras-team/keras-tuner/issues/317). Otherwise, I was able to do a transfer learning with densenet 121 on the Nano (with a very small data set, but still ...) - so, I am pretty impressed and thanks again.
Also, I will definitely have a look at your fork, @znmeb and I also thought about the miniforge approach. However, as of now, conda still delivers tf2.2, which is a complete disaster. So I will try that only after they ship tf2.3 or higher.

@dspyrhsu My conda-forge scripts are installing the NVIDIA TensorFlow and PyTorch, not the conda-forge ones. I need to do some testing to verify that they actually execute, but they do install with pip.  The fork has moved: it's now at
https://github.com/edgyR/jetson-containers.
If @dusty-nv wants a pull request I'd be happy to send it along. The resulting images are pushed to
https://hub.docker.com/u/edgyr
That organization also has my images with R, OpenCPU and RStudio Server, but I'm still cleaning up the documentation for them.

I am using a jetson nano with the nvcr.io/nvidia/l4t-tensorflow   r32.4.3-tf2.2-py3 image
because it is the only image I found with tensorflow 2 at
https://ngc.nvidia.com/catalog/containers?orderBy=scoreDESC&pageNumber=0&query=jetson&quickFilter=containers&filters=
but unfortunately it does not have jupyterlab So I would also like to have jupyterlab with the tensorflow Dockerfile

@japrogramer Try docker pull edgyr/l4t-ml:r32.4.4-tf2.3-py3 and if you find bugs open an issue at https://github.com/edgyR/jetson-containers/issues/new

@znmeb I noticed that the only change that will conflict is the change to where the built imaged is pushed.
Any chance we can get a pull request?
having tf2 in the ml images would make that set of images consistent with how the tensorflow images are tagged  at https://ngc.nvidia.com/catalog/containers/nvidia:l4t-tensorflow

the push change is a separate branch - the main branch has NVIDIA's push script


hu: Word does not match

I have been attempting to convert these dictionaries to qtwebengine format using qt's qwebengine_convert_dict tool.
I was unable to convert the file hu/index.bdic due to the following error:
Word does not match! - Index:    35768 - Expected: gĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłl - Actual:   gĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂłĂl - ERROR converting, the dictionary does not check out OK.
Most other dictionaries did build using the tool so it leads me to believe that the fault may be with hu/index.bdic but as someone who is unfamiliar with hunspell, I am unable to tell if it should be the Expected string.
If so, it seems more useful to report it here than just fix it at my end.

I’m having an issue with magyarispell, as raised here: https://github.com/laszlonemeth/magyarispell/issues/9. May be related to that.

My guess was character encoding but it's a wild one and I'm completely unfamiliar with hunspell.
Hopefully it's an easy fix.


Fix error when changing Protobuf item values in Visual Studio

Potential fix for bug detailed here https://github.com/dotnet/project-system/issues/5573
Note: I don't have end-to-end build of C-core setup on my computer, including Grpc.Tools. I'm not able to test this. Should double check before merge that it fixes the issue and doesn't regress anything else.
@jtattermusch

Adhoc test run: https://sponge.corp.google.com/invocation?id=d9e99abf-69af-4ec5-96e2-7f42494e438b&searchFor= (I will paste the generated Grpc.Tools nuget once that finishes).

Boop. NuGet package to test?

Sorry for delay:
https://storage.googleapis.com/grpc-testing-kokoro-prod/test_result_public/prod/grpc/core/master/windows/grpc_build_packages/8826/20191018-071740/github/grpc/artifacts/csharp_nugets_windows_dotnetcli.zip

I tested with the new packages. Bug is fixed. I didn't notice any regression. All grpc-dotnet tests continue to pass.

Known failures:
https://github.com/grpc/grpc/issues/20519
https://github.com/grpc/grpc/issues/20502


Changed one line

Updated the copyright year from 2016 to 2017.

Thanks.
I'll accept it if you also change the year number from README.md.

And please use a better commit message the next time. If you don't look at the detailed message you only see "Changed one line", which is horrible and doesn't tell you anything.


Refactor/prepare ents

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

33 of 45 (73.33%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.8%) to 88.507%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/types.ts
0
12
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.8%

Covered Lines:
941

Relevant Lines:
1056

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>

33 of 45 (73.33%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.5%) to 88.828%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/types.ts
0
12
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-0.5%

Covered Lines:
941

Relevant Lines:
1056

💛  - Coveralls

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Warning: This coverage report may be inaccurate.
This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

For more information on this, see Tracking coverage changes with pull request builds.
To avoid this issue with future PRs, see these Recommended CI Configurations.
For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

33 of 45 (73.33%)  changed or added relevant lines in 2 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-1.0%) to 88.282%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/types.ts
0
12
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
-1.0%

Covered Lines:
941

Relevant Lines:
1056

💛  - Coveralls


Admin

Gonna do an incremental merge on this to get it into staging, then will make more fixes and prob PR again.


Webifiers and Scrams/Disruptors Should perhaps be seperated

Took me two minutes of looking for web scramblers which were on the same row as webifiers with the web icon.

Sorry to hear you had a negative experience with the site. I've fixed this in 8068e0.


Make simple pipeline ClientService public

When creating a simple pipeline client, the response type from
TcpClient::connect() was private. Documentation was also added to the
ClientService to pass the linting.
Fixes #81

Thanks!


Not able to list audio devices and how exactly do you choose your indexed test audio streaming

Enter your choice: 1
An unexpected error occurred while listing audio devices: COM is not supported
======SIM7600G-H Modem Tool======

this is happening on a raspberry pi 5

The code in the AudioDeviceManager.cs class uses the NAudio library which is not compatible with Linux systems since it relies on COM calls. I plan on adding the code to make this function work on Linux soon, hopefully by the end of this week.

i don't know if you have a raspberry pi 5 but that is what i am testing on

I've got a Raspberry Pi 4 that I've been testing on as well.


Independent Install for FancyZones

Description of the new feature / enhancement
Independent Install for FancyZones (and/or other individual utilities)
Scenario when this would be used?
Many utilities packaged within PowerToys are security risks for various business environments, but the fact remains, some of the non-security related utilities would be extremely helpful to have. Without the ability to install different utilities independently, some of them can't be used because we lack the approval to install all of them. I would like to be able to install FancyZones independently of other PowerToys utilities either through independent installers or a utility install selection menu.
Supporting information
The Mouse without Boarders utility has an independent option here: https://www.microsoft.com/en-us/download/details.aspx?id=35460

/dup #21260


Nodeshift-config - Specify the location of the project

The nodeshift-config module is responsible for loading the configuration, setting up the openshift rest client and locating the users projects package.json
this code currently assumes the users project is the current directory the plugin is being run in.  This should also be configurable.
would be nice for testing too, since we can tell the test to look in some directory for a package.json.  src files will then assumed to be relative to that

i believe it landed here: 520ba7e6810e3cc02e5016606fd4253934a27e52


KeyClu crashes on editing "My Shortcuts" + Unable to Remove Shortcuts

Newest version from 2 weeks ago. 00.15 I think. I am able to select a category (i.e., Global or a specific application) without trouble. Adding a shortcut in "My Shortcuts" appears to not be an issue, although I'm not sure what the purpose of the section is, as it doesn't seem to do anything, at least the way I tried to use it. However, when trying to remove a shortcut (from the visible list), there doesn't appear to be a way to do this. Also, when I try to change the hotkey, it appears to allow that, but then the app crashes and quits when confirming. It does appear to hold the changed value, however.
Also, I can't successfully find a way to remove a shortcut while still leaving the item on the list. In many apps, there is an ✗ you can click, or you can highlight the shortcut as if changing it and then click backspace. I can't find any way here.
Note, please do not implement the backspace method if it would conflict with setting a hotkey using backspace. If both are possible, no problem.

Hey natejohnathan
Thanks for bringing these issues to my attention.
Firstly, I'm aware of the problem that causes a crash when adding a new shortcut due to simultaneous editing of two values. This issue has already been resolved and will be included in the next release.
Regarding the deletion of custom shortcuts from the list, when you select a row in the table, a delete button (represented by a minus icon) appears at the bottom right corner of the table. I've taken steps to make it more user-friendly, by making it more visible and accessible.
As for the option to reset a shortcut to be empty, this is not currently in plans. However, you can easily remove an unwanted item from the list if it's no longer needed.
If you have any further questions or concerns, please don't hesitate to ask.

Absolutely, both of the features you mentioned will be implemented.


Is it possible to  starttime, inpoint, outpoint?

I not sure if it possible but i would like to have some picture and video and some audio files and i want to add them to layers and the set the inpoint and outpoint of the pic lay to match the audio file length and set the start time of the videos to before the images and after.

you can check out the boilerplate project:
https://github.com/inlife/nexrender-boilerplate
especially look on how project file is structured, and how expressions influence it's behavior


[graphiql] upgrade graphiql deps

This is a rebase of #3193

@bgentry Hi! sorry for the very late response. Since I don't have the rights to force push on your fork, I had to create a new branche here. If you want, you can reset your branch on mine, so that you appear as the PR author :-)

Thank you, I just wasn’t sure how to get this resolved so I appreciate it. No need to change anything, seems I’m already listed as co author on the commits anyway 🙏🏻

@bgentry Ok cool! I've updated the integration tests, it should be good to merge now :-) Just waiting for the CI
Thank you again for the work! Very appreciated!


Fix theta uncertainty bug

Before you submit this PR: make sure to put all operations-related information in a wiki-note, a PR should be about code and is publicly accessible
What does the code in this PR do / what does it improve?
This fixes a bug found by @sebvetter (figure below) in how theta uncertainties were handled due to contours that passed over the -pi and pi boundary. These contours would have an uncertainty that is an order of magnitude smaller than the contours that do not cross this boundary.

Can you briefly describe how it works?
The theta uncertainty is calculated as the difference between the min and max $\theta$ of a contour. The correction is to rotate the contour around (0,0) by the average $\theta$ within the contour so that the contour is now around $\theta = 0$ to avoid the seam, then add $\pi$ and modulo $2\pi$ to handle large angular contours.
# Correction for circular nature of angle
avg_theta = np.arctan2(
    np.mean(events[f"{type_}_position_contour_cnf_naive"][..., 1], axis=1),
    np.mean(events[f"{type_}_position_contour_cnf_naive"][..., 0], axis=1),
)

avg_theta = np.reshape(avg_theta, (avg_theta.shape[0],1))
theta_array_shift = (np.subtract(theta_array, avg_theta) + np.pi) % (2 * np.pi)
theta_min = np.min(theta_array_shift, axis=1)
theta_max = np.max(theta_array_shift, axis=1)

theta_diff = theta_max - theta_min

Can you give a minimal working example (or illustrate with a figure)?
Given a generated fake contour, we can show that this method works for the three cases:

edge case described above at the $-\pi$ and $+\pi$ boundary
contour that overlaps (0,0)
small angle contour with $\theta < \pi$

Below is a quick minimal modified example of the code above on a fake generated contour located at the seam we want to test this method on:
# Generate fake contour located on -pi and pi seam
angles = np.linspace(0, 2 * np.pi, 16, endpoint=False)
radius_uncertainty = 2 # width of contour
xdist_from_origin = 30 # x dist of contour from (0,0)

# Calculate x and y coordinates for each angle
circle_contour = np.array([
    [radius_uncertainty * np.cos(angle), radius_uncertainty * np.sin(angle)] for angle in angles])

circle_contour = np.vstack([circle_contour, circle_contour[0]])

# Calculate expected theta diff at theta = pi/2 to calculate
check_contour = circle_contour.copy()
circle_contour[:,1] += np.abs(xdist_from_origin) # shift up y-axis
test_theta_array = np.arctan2(
    circle_contour[..., 1],
    circle_contour[..., 0],
)
theta_min = np.min(test_theta_array)
theta_max = np.max(test_theta_array)

theta_diff = theta_max - theta_min

expected_theta_diff = theta_diff/2

print(f'Theta Diff: {expected_theta_diff:.12f}')

which prints:

Theta Diff: 0.066568163776

After generating the above contour, the contour can be moved to show that this still returns the expected values at angles on and off of the $-\pi$ and $\pi$ seam.
# Now shift to -pi and pi seam for testing correction
# Define cases
test_case1 = check_contour.copy() # over -pi and pi seam
test_case2 = check_contour.copy() # at theta = pi/2

test_case1[:,0] -= xdist_from_origin # shift left on x-axis
test_case2[:,1] += np.abs(xdist_from_origin) # shift up on y-axis

# Calculate if theta_diff matches expected theta_diff
for contour in [test_case1, test_case2]:
    test_theta_array = np.arctan2(
        contour[..., 1],
        contour[..., 0],
    )

    avg_theta = np.arctan2(
        np.mean(contour[..., 1]),
        np.mean(contour[..., 0]),
    )

    # Correction for circular nature of angle
    theta_array_shift = (np.subtract(test_theta_array, avg_theta) + np.pi) % (2 * np.pi)
    theta_min = np.min(theta_array_shift)
    theta_max = np.max(theta_array_shift)

    theta_diff = theta_max - theta_min
    print(f'Theta Diff: {expected_theta_diff:.12f}')

which also prints:

Theta Diff: 0.066568163776
Theta Diff: 0.066568163776

Because the contour is not a perfect circle in the above example (only 17 points are selected, as per the CNF contour array shape), rotating the contour will cause slight differences in theta_diff depending on the orientation of this contour.
To check that this works on data, I have loaded a set of events from a Kr83m run to test. The above figure was reproduced and after rerunning with the code change, the bug is no longer visible.
For event contours:

For peak contours:
(The sharp dip in the center is a result of very small radii close to 0.)

coverage: 90.156% (-0.04%) from 90.199%
when pulling 1f7b889b916f4e2c6ebce968ccfcca8e24a1928a on theta_uncertainty_fix
into f1bb2293e593bd6e2fa7eaf9c994c0a4533a8ad0 on master.

In addition to the specific comments, the shared code should be refractored into a static method as @dachengx suggested, and the inconsistency in naming should be fixed.

@juehang is referring to position_contours_cnf -> position_contour_cnf (slack thread)


how do you build this stuff?

Before the projects were combined, there were instructions/makefiles for building the individual projects.  How does one build the components for the new combined project?  I'm specifically interested in building on Linux.
Thanks.

Binary build and RPM build instructions have been added.


Not an issue but a question

what is the number 16 used for in the below code?
https://github.com/songtaohe/Sat2Graph/blob/93ee6037312179acba720d65e7a5853444cccccb/model/dataloader.py#L189-L190

what is the number 16 used for in the below code?
https://github.com/songtaohe/Sat2Graph/blob/93ee6037312179acba720d65e7a5853444cccccb/model/dataloader.py#L189-L190

The goal of this line (and similar lines in other places) is to remove vertices near the boundary of the big tile (2048 by 2048). If I remember correctly, it may not need to be 16, any number greater than 2 should work here.


Output of linting child processes truncated

lint-action uses execSync to run child processes for each linter. However, the default maxBuffer is 1MB. This is an issue for larger/legacy codebases that may have many violations not part of the diff, the effect being that the JSON is not formed properly and can't be parsed by parseOutput:
Error: Error parsing ESLint JSON output: Unexpected end of JSON input. Output: <1 MB dump>

Increasing the maxBuffer on that line (and/or exposing that option) is a potential fix, though there may be another problem innate to the use of child_process and the non-blocking nature of console.log (which might be an ESLint issue, I didn't dive that deep).

Hi Robert, thanks for the report. I'm assuming that you run into this problem? What size would have worked for your case?

After running some benchmarks against my codebase, I think a good number with breathing room is 10MB.

Longer term it should be possible to architect the github action so you only perform linting against files changed rather than reducing from an entire repository.


Improve the image view docs on QGIS

https://twitter.com/Terranigmus/status/1491018115538644993

related to https://github.com/MerginMaps/docs/issues/65


this.server.currentLayer.layer is null on Jensens Range.

https://github.com/Thomas-Smyth/SquadJS/blob/f825b4a65b486fd1cd738480e732afe74eb13aec/squad-server/layers/layers.js#L43
Jensens Range has 3 entries in
https://raw.githubusercontent.com/Squad-Wiki-Editorial/squad-wiki-pipeline-map-data/master/completed_output/_Current Version/finished.json
With Identical Names, as returned by RCON.
In the case of Jensens Range, we end up with matches having 3 entries, failing the equality check against .length === 1
The length check match means we pass null when the map is ambigious, this breaks server status and the currentlayer/layerhistory in the server object.
           "Name": "Jensen's Training Range",
            "rawName": "CAF_JensensRange_CAF-RUS",
            "levelName": "CAF_Jensens_Range_v4",

            "Name": "Jensen's Training Range",
            "rawName": "JensensRange_GB-MIL",
            "levelName": "Jensens_Range_v1",

            "Name": "Jensen's Training Range",
            "rawName": "JensensRange_US-RUS",
            "levelName": "Jensens_Range_v2",

            "Name": "Jensen's Training Range",
            "rawName": "JensensRange_MEA-INS",
            "levelName": "Jensens_Range_v3",

Without more information, I'm not sure we can disambiguate the current map, perhaps a hook into the log file/new game event?
It appears we do this with layer classname on new game, however the intervaled update at
https://github.com/Thomas-Smyth/SquadJS/blob/f825b4a65b486fd1cd738480e732afe74eb13aec/squad-server/index.js#L400
Is where we fail to update the layer information correctly, overwriting the current map information with "null"

This is excluded in the new layers.json as per the Wiki guidance. So it will not show in the bot anymore.

This is excluded in the new layers.json as per the Wiki guidance. So it will not show in the bot anymore.

What exactly do you mean, all Jensens layers are removed from the output JSON?

Jensens Range not in Squad Wiki
This is the link currently used in SquadJS (contains Jensens):
https://raw.githubusercontent.com/Squad-Wiki/squad-wiki-pipeline-map-data/master/completed_output/_Current Version/finished.json
This link does not contain Jensens (posted above):
https://raw.githubusercontent.com/Squad-Wiki-Editorial/squad-wiki-pipeline-map-data/master/completed_output/_Current Version/finished.json
@werewolfboy13 You are involved in the Squad Wiki project, correct? Is this behavior intentional or does the first file contain Jensens by accident / is obsolete?
Ambiguous layer names
We could use the team information we get from A2S_INFO as a fallback.
The team info look like this:
{"TeamOne_s":"Kohat_RAAS_v08_AUS","TeamTwo_s":"Kohat_RAAS_v08_RUS"}
{"TeamOne_s":"GooseBay_RAAS_v3_USMC","TeamTwo_s":"GooseBay_RAAS_v3_RUS"}

We could just take the overlap of TeamOne_s and TeamTwo_s and try to look up the layer with that.

Jensens Range not in Squad Wiki
This is the link currently used in SquadJS (contains Jensens): https://raw.githubusercontent.com/Squad-Wiki/squad-wiki-pipeline-map-data/master/completed_output/_Current Version/finished.json
This link does not contain Jensens (posted above): https://raw.githubusercontent.com/Squad-Wiki-Editorial/squad-wiki-pipeline-map-data/master/completed_output/_Current Version/finished.json
@werewolfboy13 You are involved in the Squad Wiki project, correct? Is this behavior intentional or does the first file contain Jensens by accident / is obsolete?
Ambiguous layer names
We could use the team information we get from A2S_INFO as a fallback.
The team info looks like this:
{"TeamOne_s":"Kohat_RAAS_v08_AUS","TeamTwo_s":"Kohat_RAAS_v08_RUS"}
{"TeamOne_s":"GooseBay_RAAS_v3_USMC","TeamTwo_s":"GooseBay_RAAS_v3_RUS"}

We could just take the overlap of TeamOne_s and TeamTwo_s and try to look up the layer with that.

Do not use the old link, we have moved over to a different org. I don't provide support on the old resource. I did a PR fixing that.
Actually looking at this I am going to close this out as wont fix. As this is an extremely outdated ticket. I recommend that you update your squad instance to the current version

If you are still having any other issues make a ticket regarding those. The issue in this ticket is stale.


server: fix hash stability issues with maps

@hanamantagoudvk @nehaLohia27
This should fix the issue while keeping PortOverrides (I hope!).

changes look good @mcluseau .

/lgtm

/approve


Remove duplicate test cases

I'm a newbie to functional programming and recently I notice something weird to me when looking into Ramda source code. Some methods of Ramda seem not favour pointfree style, like gt, substract and concat. The argument order of concat I expected is concat(what, data), but Ramda's implementation is more likely to be translated as "a concat b". When using Ramda's concat in partially application or function pipeline, we write code like:
const concatRamda = flip(concat('ramda'))
compose(toUpperCase, concatRamda)('enter ') //=> 'ENTER RAMDA'

In my opinion concat(what, data) is more nature and intutive. We does not need to introduce extra flip and it's more consistent with the remaining codebase. Does Ramda design these methods for a reason?

I can't tell whether you think these tests should be removed because you think the API should change or if you see real duplication in the tests as your issue title implies.  If the latter, what do you see duplicated?
As to this:

Does Ramda design these methods for a reason?

I'm just going to repost part of my answer from https://github.com/ramda/ramda/issues/2895#issuecomment-536237789:

BTW it's strange to see that R.gt and R.lt using first argument as value.

You're asking about things that have a long history of controversy in Ramda.  I'm afraid I don't have the strength to look up the references, as I will start re-reading all our old arguments and tedious discussions about them.  But if you search for "operator", you will find at least a dozen issues related to this.
Briefly, there is a serious tension between, for instance,
R.subtract(10, 3) //=> clearly should be 7

and
const less3 = R.subtract(3)

less3(10) //=> we want this to be 7, but it's -7.

This is true for gt as well, and for any binary operator where op(a, b) !== op(b, a).  We have never really found a solution we liked for this.

Sorry I didn't provide enough imformation about this PR. There is real duplication in the test:
  it('adds combines the elements of the two lists', function() {
    eq(R.concat(['a', 'b'], ['c', 'd']), ['a', 'b', 'c', 'd']);
    eq(R.concat([], ['c', 'd']), ['c', 'd']);
  });

  var z1 = {
    x: 'z1',
    concat: function(that) { return this.x + ' ' + that.x; }
  };
  var z2 = {
    x: 'z2'
  };

  it('adds combines the elements of the two lists', function() {
    eq(R.concat(['a', 'b'], ['c', 'd']), ['a', 'b', 'c', 'd']);
    eq(R.concat([], ['c', 'd']), ['c', 'd']);
  });

Now it's obvious I supposed.

Weird, I swear I looked at the file and didn't see it.  I must have looked at the one from your commit...
Thank you.  This is clearly needed.


[FR]: Create switches to enable/disable VPN servers

Describe the new feature or enhancement idea
For both Openvpn and Wiregaurd. REST APIs exist.
https://docs.opnsense.org/development/api/core/openvpn.html
https://docs.opnsense.org/development/api/core/wireguard.html
Will you be building this?

[X] I intend to submit a PR to implement this

Additional Details
No response

Please consider also adding status entities for VPN servers to make this feature more complete


Add logic operation examples

This change is

Thanks for these awesome contributions! Unfortunately we're on a small code freeze until Friday, but as soon as that's over I'll merge these in :)

Thanks to you as well. I've been really enjoying the library! First time working with ML and the documentation(especially the examples) has been really useful.

Review status: 0 of 1 files reviewed at latest revision, 7 unresolved discussions, some commit checks failed.

src/ops/logical_ops.ts, line 31 at r2 (raw file):

   *
   * ```js
   * const a = tf.tensor1d([0, 1], 'bool');

you can use true and false as values to the bool, it might make this clearer

src/ops/logical_ops.ts, line 33 at r2 (raw file):

   * const a = tf.tensor1d([0, 1], 'bool');
   *
   * a.logicalNot().print();

it looks like we don't actually have this on the chain API, this is a mistake!
If you're up for it, could you add a method like this to tensor.ts?
https://github.com/tensorflow/tfjs-core/blob/master/src/tensor.ts#L666

Comments from Reviewable

@nsthorat I can write the method in a bit. Having local dev environment issues at the moment so I'll get to it some point this week.

Review status: 0 of 2 files reviewed at latest revision, 9 unresolved discussions, some commit checks failed.

src/ops/logical_ops.ts, line 31 at r2 (raw file):
Previously, nsthorat (Nikhil Thorat) wrote…
you can use true and false as values to the bool, it might make this clearer

I think the only comment left is to replace 0 and 1 with false and true to be clearer. Let me know when you do that and we'll merge this in. Thank you!

Comments from Reviewable

@dsmilkov Should be good to go.

Reviewed 1 of 2 files at r4.
Review status: all files reviewed at latest revision, 9 unresolved discussions.

Comments from Reviewable

Thank you!!


feat(plugins): use display rules in cc-messages plugin, closes #102

:rocket: PR was released in v0.7.0 :rocket:


Improve semantic release test 3

Description
copilot:summary
Related Issue

Explanation of Changes
copilot:walkthrough
Checklist before requesting a review

[ ] I have performed a self-review of my code.
[ ] Implemented tests for new features.
[ ] I tested the features already implemented.
[ ] Added usage examples in the readme.

:tada: This PR is included in version 1.9.0 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


Cannot unzip 4.1.1 and 4.1.0 in Ubuntu 20.04 LTS

Unzipping TerasologyLauncher-linux64.zip (both version 4.1.0 and 4.1.1) in Ubuntu 20.04 LTS fails with this message:
error: invalid zip file with overlapped components (possible zip bomb)
The 4.0.0 zip can be extracted without this error message.
unzip has version "6.0-25ubuntu1".
...
inflating: TerasologyLauncher-linux64-4.1.1/jre/conf/security/policy/unlimited/default_local.policy
inflating: TerasologyLauncher-linux64-4.1.1/jre/conf/security/policy/unlimited/default_US_export.policy
inflating: TerasologyLauncher-linux64-4.1.1/jre/conf/security/java.security
inflating: TerasologyLauncher-linux64-4.1.1/jre/conf/security/java.policy
error: invalid zip file with overlapped components (possible zip bomb)
sha256 checksums of the 4.1.0 and 4.1.1 (renamed):
c98d8ca791acb7d75284d9379ae57fc4673350ce8b930663499cf3376bb4a581  TerasologyLauncher-linux64-4.1.0.zip
1a86e41b22e909008d762f7b8b4dd6d44c310666ce564f67c0c4ecfd6dc79aec  TerasologyLauncher-linux64-4.1.1.zip

Thanks for the report @tvierb 💚
This might be related to #581 and we "just"™ need to make sure to not include this file (or any symlinks) leading to the possible zip bomb.

I expect these two issues are different presentations of the same bug, yes.
I remember wondering if the latest code in git still has this bug, because of changes in how we build things, but off the top of my head I'm not sure which changes I was thinking of or if they've been merged.

I'm not sure any of https://github.com/movingblocks/terasologylauncher/compare/v4.1.1...master really changes anything in that regard. Maybe one of the open PRs has some improvements for this...

Fixed by #599 and available as https://github.com/MovingBlocks/TerasologyLauncher/releases/tag/v4.1.2

@tvierb Feel free to test the new release ;)


fixed #354, list existed DJ channels

This is still a WIP

Will provide another patch for this, working on it.


Homepage contact form

Adds form validation to support browsers that do not implement HTML5 form validations (Safari / iOS Safari)

:+1:
I'm only wondering if the acknowledge text is bib/accented enough? Maybe we can use $purple...? :)


fix(@angular/cli): keep polyfills required for IE

Since IE9+ is officially supported by Angular, the CLI should have the polyfills for it enabled by default, as developers would expect that to work by default as the CLI takes care of polyfills.
The edge case or additional scenarios like SVG, animations, and i18n are still commented by default.

Hi @Meligy,
These defaults were discussed with our team, and it was decided that the default should be Evergreen. You're right that we support IE9/10/11, and if you need those you should uncomment the lines. By default, in your project, we should only target evergreen browsers though, as it results in smaller builds and most people would like to target those.
Cheers.


Is there a way to remove Sensors from manual_control.py ? Eg. Semantic Segmentation, LiDAR, etc.. If yes, is there a specific set of steps?

I want to see if I can increase the FPS by removing some sensors which I dont need for the work Im doing, I want to achieve this but am unable to figure out where to begin.

As far as I know, sensors are defined in a list(self.sensors) in CameraManager in manual_control.py, and modifying that should work . I have tried that for comparison between segmentation and RGB imgs.

But according to the pipeline for carla, the client also has cpp code for it.. Is that also to be removed?

@TheMaveric If you do not care about image quality, you may as well try to set server quality as low.


gnome: Default font isn't changing to Inter

Testing changes from #132 in VMs
KDE Plasma changes worked just fine
In GNOME, though, the default font (tested by running gsettings reset org.gnome.desktop.interface font-name) is still Cantarell 11, despite the changes in dconf

...Except after a reboot on my main system, i.e. bare metal, the default font did change over to Inter as expected. No idea what could be going wrong in the VM, then


docs: improvements in the handling of transformations

The transformations are performed with the change-case and pluralize libraries, as well as allowing to combine them with each other. See the documentation. Also, I consider it a major update as it changes and/or removes some transformations (names, behavior).

Moin @mrelemerson, thanks for the PR!
Overall I really like your changes. However if possible I would like to not introduce another breaking change so soon again.
Would it be alright with you to handle the old transformer names and translate them into the new ones?
e.g. lowercase should still work for the enduser but internally it will be translated to lowerCase  to work with the change-case library.
That way we won't break the extension for every single user out there but can still take advantage of the change-case library.

Hi @Huuums, yes I will change it to maintain compatibility.

ready


Add morguefile.com

:) https://www.morguefile.com/

Will add this in a bit. Thanks for the suggestions.

Thanks !


resistances are inversed between P and AP state !

*** The resistance should be high at anti-parallel (AP) state and low at parallel state (P) ***

The pinned layer magnetization vector is initialized UP ( p_theta_pl = 0 and pl_z = 1)
The parallel state requires the magnetization vector of the free layer to be UP also (m_theta= 0 and m_z = 1) The formula R = f(mz) should be corrected as follows in order to get low resistance when V(mz) = 1 : // simple TMR based conduction
r_mtj_module (fl pl) bsource r=p_r_p*(1 + p_tmr/(p_tmr + 2))/(1 + p_tmr/(p_tmr + 2)*V(mz))

the formula from https://doi.org/10.1109/TMAG.2017.2788010 was adopted whereas the MTJ pinned layer configuration is different.


Add link to imgproxy package in AUR for Arch Linux users

Fun fact: actually AUR doesn't contain packages and software is being build every time from the sources on end user's machine. So it is long and loud process. But it is as simple as package manager – you just check imgproxy and it will be installed for you along with libvips.

Thanks! Looks like it's time to upgrade the package to v2.3 ;)

Looks like it's time to upgrade the package to v2.3 ;)

Oops, I missed the release notification from GitHub. Done!


Look into integrating Browserify

Use your favorite NPM modules on the client with Browserify.
https://github.com/gulpjs/gulp/blob/master/docs/recipes/browserify-with-globs.md

Closing after https://github.com/OraOpenSource/apex-frontend-boost/issues/179