Implement mocking for call activities

With this PR (which closes #14) we can write the following code:
when(scenario.waitsAtMockedCallActivity("CallActivity")).thenReturn((callActivity)` ->
    callActivity.complete();
);

Scenario.run(scenario)
    .withMockedProcess("Child")
    .startByKey("BoundaryInterruptingTimerTest")
    .execute();

verify(scenario, times(1)).hasStarted("CallActivity");
verify(scenario, times(1)).hasFinished("CallActivity");

Hi @martinschimak . We have big bpmn, with a few sub processes which we are trying to mock out of the parent - so mocking childrens. Ran into an issue, where its complaining about the child hasn't been deployed - but with the above code, I should be able to run the parent scenario without deploying the child.
when(parentScenario.waitsAtMockedCallActivity("childFlow")).thenReturn(ExternalTaskDelegate::complete);
Scenario.run(parentScenario)
.withMockedProcess(CHILD.key)
.startByKey(PARENT.key, variables)
.fromBefore("randomActivity")
.execute();
Error:
org.camunda.bpm.engine.exception.NullValueException: no processes deployed with key = 'childFlow' in deployment = 'd93cce89-fd61-11ee-a7fb-047bcbbda71c': processDefinition is null


[bump_v1.12.0] Additional cherrypicks for 1.12.0 rc5

Cherrypicking:
https://github.com/docker/docker/pull/25047
https://github.com/docker/docker/pull/25042
https://github.com/docker/docker/pull/25043
https://github.com/docker/docker/pull/25038
https://github.com/docker/docker/pull/25041

LGTM

LGTM

LGTM 🐳 👼

fyi; we're merging this so that the piñata team can build/run tests; we'll do another WIP cherry-pick with (e.g.) some completion scripts, and have CI run a full cycle on that one


CLI-2 # Updated to latest version of argv-one

Blocked by https://github.com/jokeyrhyme/argv-one.js/pull/1 and the following npm publish

Done: https://github.com/jokeyrhyme/argv-one.js/releases/tag/1.0.1

Thoughts over in https://github.com/jokeyrhyme/argv-one.js/pull/1#issuecomment-241647130 apply here, too.


Side by Side Embedding Feature

Multimodal exploration of spatial data with side-by-side embeddings
Rationale:
This is foundational work to support data consumers’ ability to generate hypotheses from spatial data. Being able to compare the location of cells in physical space with their position in UMAP or other lower-dimensional “transcriptomic space” enables data consumers to investigate how gene expression profiles change in different cell types and tissue locations.
User stories:
As a user, I want to directly compare cells’ placement in physical space (spatial embedding) with its representation in a low-dimensional embedding (e.g., UMAP or tSNE) so that I can better understand how patterns in gene expression vary across space.
Requirements:

We can show two scatterplots side by side; one or both of these may be a spatial embedding with an image underlay.
It’s very important to keep the state of the data consistent across the entire viewer – that is, the filters, coloring, and which points are in view should be internally consistent with each other.
Hovering over cells in the left sidebar should highlight the same cells in both embeddings.
Hovering over cells in one embedding should highlight the same cells in the other.
Subsetting affects both embeddings. E.g., if the lasso selector is used to subset, the set of cells is displayed in the other embedding (even though they are not contiguous).
Colorby and filtering affects both embeddings.

Design - https://www.figma.com/file/eJ3acPxG04PKVlom6eS8ft/Spatial?type=design&node-id=304%3A195512&mode=dev
@hthomas-czi

@seve I don't remember if the data point limit we currently support is memory bound or CPU/GPU bound, but maybe worth creating a ticket to double check that the side by side mode will work with the max data points we support 😊?
Thank you!


flamegraph of test runs

flamegraph of test runs - which functions time is spent inside
wallclock time vs. cpu time

Part of epic https://github.com/dotmesh-io/dotmesh/issues/296


expose securityContext and default to runAsNonRoot: true

BOT NOTES:
resolves https://github.com/solo-io/skv2/issues/260

This LGTM and will unblock a UI issue, will let Scott approve


Access Package Policies - Ability to rename a policy

Currently if a Access Package Policy is renamed in a json file it doesn't rename it on the Access Package it creates a new policy, keeping the existing AP Policy, would be nice if it could rename the existing access policy.
Below is an example of "Initial Policy" being renamed in the config but a new policy is crated.

.

I've created an internal feature request.

Renaming of accessPackageAssignmentPolicies is possible in the latest version (1.5.493) by adding a string array "oldNames" which holds the oldName(s) of the object.


allow modern enum class/scoped enums with type specifier and assigning values

Scoped enums have been available since C++-11 (13 years ago). Other DDS implementations support this in their IDL.
The advantages are well documented.
Add IDL support for scoped enums a.k.a. "enum classes" with a type specified "enum base" that can be any cstdint and allow specifying a value for each enumeration independently, including negative values, and since scoped allow different enums to reuse label names.

@JSG-0
Thanks for your contribution. According to Fast DDS CONTRIBUTING guidelines I am moving this ticket to the Ideas forum where feature requests are discussed.
On the one hand, enum class support has been introduced in #271 into the development branch feature/xtypes_typeobject. This branch will be merged and released soon.
On the other hand, I am afraid that support for @bit_bound builtin annotation that allows modifying the enum base and the @value builtin annotation that allows setting the value of each enumeration literal, is not in the current roadmap. You might contact eProsima's commercial support team to discuss options to accelerate their development.
Regarding the scope, this is something already supported in the current Fast DDS-Gen. Nevertheless I have issued eProsima/dds-types-test#22 to add this specific test case into Fast DDS-Gen CI. The CI is updated in #292.


supertux-editor: mark as broken

Motivation for this change
SuperTux now uses an in-game editor, so there's no point in trying to build this old C# monstrosity.
I don't think it's actually broken yet, see #122759 for the change that breaks it.
Things done

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

[ ] 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
[x] Fits CONTRIBUTING.md.

Should I merge now or wait until https://github.com/NixOS/nixpkgs/pull/122759 is ready to go?

I think merging it now is fine, if there is someone that's actually using it they'll have a better chance to participate.


Pending members should not be visible to anyone but circle admins

Alongside #232 , we need to make sure pending members can be seen by the admins of the circle that they're applying to

Hi @NolanChan  is this ready for @phil-monroe to review and merge? Thanks!

Merged, closing.


app.message(callbackFn) : callbackFn not called for messages sent by channel owner

Description
When subscribing to messages, the app.message callback function isn't triggered for messages posted by the channel owner. However, it is notified if the messages are posted by anyone else.
What type of issue is this?

[X] bug
[ ] enhancement (feature request)
[ ] question
[ ] documentation related
[ ] example code related
[ ] testing related
[ ] discussion

Requirements

[X] I've read and understood the Contributing guidelines and have done my best effort to follow them.
[X] I've read and agree to the Code of Conduct.
[X] I've searched for any related issues and avoided creating a duplicate issue.

Bug Report
Filling out the following details about bugs will help us solve your issue sooner.
Reproducible in:
package version:
@slack/bolt": 3.8.1
node version:
node v12.14.1
OS version(s):
Ubuntu 18.04 LTS
Steps to reproduce:

Creat a bot app, set everything up in slack web console

Add the relevant events subscriptions:

Add the bot to a channel

code your bot as follows:

import { App, LogLevel } from '@slack/bolt';

const app = new App({
  token: process.env.SLACK_USER_TOKEN,
  socketMode: false,
  signingSecret: process.env.SLACK_SIGNING_SECRET,
  logLevel: LogLevel.DEBUG,
  developerMode: true,
});

const messageHandler: Middleware<SlackEventMiddlewareArgs<'message'>> = async ({ message }) => {

  // The following line is only executed if incoming messages are from non-owners of the channel
  // If the message is from the owner, this code is never run
  console.log('messageHander.message', message);
};

app.message(messageHandler);

(async () => {
  try {
    await app.start({
      port: +process.env.PORT || 3000,
    });
    console.log('⚡ Slack-bot app is running!');
  } catch (error) {
    console.error('Unable to start App', error);
    process.exit(1);
  }
})();

Deploy your app or run it with ngrok
configure callback URLs to your app URL in the slack web console

Expected result:
Once the app is running, any message posted into the channel where the bot is installed should be captured by Bolt and the messageHandler function should be called with the relevant payload
Actual result:
If anyone but the owner of the channel posts a message to the channel, the messageHandler is triggered and everything works fine.
However, if the owner of the channel posts a message, the messageHandler is never triggered.
I'm sure this is a Bolt issue because I checked that the Slack server is actually posting correct payloads to my server for both situations. I've also checked it with the Slack help desk, which has confirmed that indeed, their servers send the appropriate payloads.
To make sure this was not an issue with the app.message() function, I also tried the same process, but subscribe through app.event('message', callback) with exactly the same results.
I attached the 2 payloads received by my app and the diff. You will see that apart from timestamps and Ids, only the user field is different

Attachments:
Logs, screenshots, screencast, sample project, funny gif, etc
triggeringPayload.json.txt
NOTtriggeringPayload.json.txt
.

Hi @pmullot, thanks for writing in!
If the message event is associated with the channel owner, this situation can arise. bolt-js intentionally self events to avoid causing an infinite loop of messaging and self-replying.
To customize this behavior, you can set ignoreSelf: false option when initializing your App. Please refer to https://github.com/slackapi/bolt-js/issues/1113#issuecomment-918424410 for more details. Could you try this option?
I hope this helps.

Works perfectly!
Thanks a lot mate!!


Unit tests

Adding some unit test to the core of the application.

If you need mre information, about it, don't hesitate.


simplify multi cluster delete

Signed-off-by: Rama Chavali<EMAIL_ADDRESS>[ ] Configuration Infrastructure
[ ] Docs
[ ] Installation
[X] Networking
[ ] Performance and Scalability
[ ] Policies and Telemetry
[ ] Security
[ ] Test and Release
[ ] User Experience
[ ] Developer Infrastructure
Pull Request Attributes
Please check any characteristics that apply to this pull request.
[X] Does not have any changes that may affect Istio users.

@ramaraochavali: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
1a14a045893b6d092e669088896b2e718d9281f8
link
/test integ-multicluster-k8s-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

@ramaraochavali: The following tests failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-multicluster-k8s-tests_istio
1a14a045893b6d092e669088896b2e718d9281f8
link
/test integ-multicluster-k8s-tests_istio

integ-pilot-multicluster-tests_istio
1a14a045893b6d092e669088896b2e718d9281f8
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/retest

@ramaraochavali: The following test failed, say /retest to rerun all failed tests:

Test name
Commit
Details
Rerun command

integ-pilot-multicluster-tests_istio
1a14a045893b6d092e669088896b2e718d9281f8
link
/test integ-pilot-multicluster-tests_istio

Instructions for interacting with me using PR comments are available here.  If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here.

/test integ-pilot-multicluster-tests_istio


Remove unused class from jsx if the class is removed from the css module file

Hi, I have another request in order for this library to work a bit better for Gatsby related projects 😄.
Problem
Gatsby will throw errors if you import a style that does not exist in the css module file at build time.
Cannot read properties of undefined (reading 'e')
error Generating JavaScript bundles failed

For example, the jsx is left like so after the conversion: ${styles.navItemLink} text-blue-300 no-underline
But .navItemLink is removed from the css module file because all styles have been transformed to tailwindcss classes.
Gatsby will attempt to import .navItemLink from the file and throw an error because it is no longer present.
Proposed Solution
Remove the styles.className from the className={} if the class no longer exists in the module.

Thank you for the quick response and implementation (again)!
This works really nicely.
There is one more case to solve for–when the *.module.css file is empty, it should be removed,  and the import of that file in the jsx should also be removed. That way a project wouldn't have empty files or unused imports.
I understand if this is somewhat out of scope, but it would be a nice to have.

Hi, you can try version 0.1.9-beta.2, because the import specifier is removed, the corresponding style member also needs to be modified, for example:
/** index.module.css */
.test {
   display: flex;
}

import style from 'index.module.css';

const Component = () => (

);

After removed:
const Component = () => (

);

Deleting import specifier is a dangerous operation, please make sure your code run successfully,  thanks~

I tried 0.1.9-beta.2 and I got the same result  as beta.1 where the *.css.module file is empty after the transfomr, but the import * from styles import specifier is still in the file, so I'm not seeing any change in the output. Is there any other config to tell it to remove the specifier? Maybe if it was opt + a warning when using it instead of default behavior it would be OK that it is a dangerous operation?
Also, please let me know if you accept sponsorships/donations through any platform as I am trying to get some $$ to you on behalf of my employer for all this work, thank you!

@CobyPear Can you try version 0.1.9-beta.4, I found a bug with an unhandled promise rejection in the code.
Thanks for your suggestion, if no one uses it, I don't bother to optimize it.

I am seeing the same result for 0.1.9-beta.4.

Thanks for your suggestion, if no one uses it, I don't bother to optimize it.
You're welcome, thanks for implementing it so quickly!

I need code samples that can reproduce the problem, such as https://codesandbox.io/

I am trying to get together a minimal reproduction for you, but it is a bit difficult since we are using your library via npx as a step in our CLI, and right before we run your library we run postcss on our css variables to transform those into something your library can read first. I can outline the steps to setup our CLI locally but I worry it's a bit of work. I will try to get something less involved on Monday, but if you'd like I'll outline the steps I am using below. Please do not feel pressured to use these steps as I understand it is quite involved.

Clone the following monorepo: git@github.com:pantheon-systems/decoupled-kit-js.git
in the root of the monorepo, pnpm i you may need pnpm 8.1.0 You may see errors of things failing to build, you can ignore those.
On this line, change update the css-modules-to-tailwind command to the @0.1.9-beta.4 https://github.com/pantheon-systems/decoupled-kit-js/blob/canary/packages/create-pantheon-decoupled-kit/src/actions/convertCSSModules.ts#L45
The, to call the CLI the easiest thing to do is to set up a watch script. I will copy one below that you can use. You may want to change the outDir to something else. the important part that will run the convert action is tailwindcss: true
in the terminal, run pnpm watch:cli. You should see the CLI generate a new app then try to convert the CSS modules. The last linting step will fail due to styles being an unused var in the src/pages/404.jsx file. You can see the 404.module.css is empty

Here is the `watch.ts` file you will put at the root of `decoupled-kit-js/packages/create-pantheon-decoupled-kit`
import type { ParsedArgs } from 'minimist';
import path from 'path';
export const watchOptions: ParsedArgs = {
	_: ['gatsby-wp'],
	outDir: path.resolve(process.env.HOME as string, 'test-cli', 'css-mods-repro'),
	appName: 'css-modules-to-tw-repro',
	cmsEndpoint: 'https://any',
	noInstall: false,
	// force overwrite of the target directory.
	// WARNING: this option could overwrite uncommitted work.
	// Choose an empty outDir for best results with this options.
	force: true,
	// silent: true,
	// number of milliseconds to debounce file system watching
	tailwindcss: true,
	debounce: 5000,
};

Nice, LGTM! Thanks again for the quick turnaround on this, it is very much appreciated!

Version 0.1.9, released.

Thanks again, just ran another test and it works great.


CFM-1861 Nifi service data and settings are gone after datahub upgrad…

…ed from 7.2.6 to 7.2.9. Disabled file provider and enabled database provider, because NiFI Registry in CFM-<IP_ADDRESS>, should work with DB, on datahub. For private could it should work with file provider, that is why added changes into CB template.
See detailed description in the commit message.

@doktoric could you please review/merge current changes? Also could you please clarify, is it enough to merge this change only to master?

@doktoric could you please review/merge current changes? Also could you please clarify, is it enough to merge this change only to master?

if you sen it to master then the next release will contains this fix. The branch cut will happen today.

@doktoric could you please review/merge current changes? Also could you please clarify, is it enough to merge this change only to master?

if you sen it to master then the next release will contains this fix. The branch cut will happen today.

@doktoric we need that change in 7.2.9. As i understand it's next release, so that's fine.


start from a new Cassandra

I try to run the examples.
first, I got the newest Cassandra 3.0.3, and I got this Error:
16/03/02 11:29:57 ERROR NIOServerCnxnFactory: Thread Thread[main,5,main] died java.io.IOException: Failed to open native connection to Cassandra at {<IP_ADDRESS>}:9042 	at com.datastax.spark.connector.cql.CassandraConnector$.com$datastax$spark$connector$cql$CassandraConnector$$createSession(CassandraConnector.scala:181) 	at com.datastax.spark.connector.cql.CassandraConnector$$anonfun$2.apply(CassandraConnector.scala:167) 	at com.datastax.spark.connector.cql.CassandraConnector$$anonfun$2.apply(CassandraConnector.scala:167) 	at com.datastax.spark.connector.cql.RefCountedCache.createNewValueAndKeys(RefCountedCache.scala:31) 	at com.datastax.spark.connector.cql.RefCountedCache.acquire(RefCountedCache.scala:56) 	at com.datastax.spark.connector.cql.CassandraConnector.openSession(CassandraConnector.scala:76) 	at com.datastax.spark.connector.cql.CassandraConnector.withSessionDo(CassandraConnector.scala:104)
then I got the Cassandra 2.1, it works.
Actually we use Cassandra 3.0.x in the test environment, how can I do to fix this?
What if I upgrate the other dependencies?
I'm new to spark, tell me step by step please.
Many thanks.

Hi @carreymu, please see the version of cassandra this was last built against (which was a long time ago now) https://github.com/killrweather/killrweather/blob/master/project/Versions.scala#L23 So yes, if you are using a newer version you will get that error ;) If you would like to contribute a PR for the upgrade (with cassandra, the driver and the spark cassandra connector) that is consistent with all binaries and passes IT tests and the runtime demos and samples, that would be greatly appreciated!

Hi @carreymu, even with the latest librairies version I wasn't able to make it work with Cassandra 3.0.3... :-(
See FutureT: @uncheckedVariance #30


When I open a menu and click the mouse button on the other space (not on opening windows), program crash!

I open a menu like this，and I tried to close it by clicking the other no window place, then program crash

I run the Java example in IntelliJ IDEA, here is the report:

Hi @MetaPhyicers,
Thanks for reporting, it should be fixed
Give it a try and then close if confirmed
ciao


No pot recipe in Jei or whatever in 1.20.1

Hello @blueofice641,
Like it is often said, Farmer's Delight fabric version do not integrate JEI, use REI instead.


Quantity and 'worn' state of items in packing list should update correctly after drag and drop

When reordering items in the packlist the items drop at the right spot but the quantity and 'worn' state don't update.
See screencap:
https://github.com/Packstack-Tech/app/assets/1029855/73609fa7-4add-420a-8bc1-d67faa20b9f9
Browser refresh shows the quantity and status in the right place again so it's hopefully just a minor scoping issue. 🙏

I've spotted the problem with the item quantity, but I'm not able to reproduce the worn state issue.


Optimize ConcurrentReadWriteDictionaryTest

Optimize the run time for the tests (locally from 25 sec to 6 sec)
Rename it to MutableDictionaryTest because it does not only test for concurrency but also functionalities.

Runtime reduced from 116s to 16s on Travis

Codecov Report

Merging #1639 into master will increase coverage by 0.39%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1639      +/-   ##
==========================================
+ Coverage   45.85%   46.24%   +0.39%
==========================================
  Files         825      825
  Lines       40128    40128
  Branches     5279     5279
==========================================
+ Hits        18400    18557     +157
+ Misses      20073    19907     -166
- Partials     1655     1664       +9

Impacted Files
Coverage Δ

...r/dociditerators/RangelessBitmapDocIdIterator.java
55.55% <0%> (-11.12%)
:arrow_down:

...e/operator/dociditerators/BitmapDocIdIterator.java
53.57% <0%> (-7.15%)
:arrow_down:

.../predicate/NotEqualsPredicateEvaluatorFactory.java
27.65% <0%> (-5.32%)
:arrow_down:

...not/broker/broker/helix/ClusterChangeMediator.java
62.71% <0%> (-3.39%)
:arrow_down:

.../broker/routing/HelixExternalViewBasedRouting.java
88.5% <0%> (-1.4%)
:arrow_down:

...inot/core/operator/docidsets/AndBlockDocIdSet.java
53.79% <0%> (-1.38%)
:arrow_down:

...manager/realtime/HLRealtimeSegmentDataManager.java
76.24% <0%> (-1.11%)
:arrow_down:

...ntroller/helix/core/PinotHelixResourceManager.java
37.93% <0%> (-0.22%)
:arrow_down:

...ot/broker/requesthandler/BrokerRequestHandler.java
84.14% <0%> (+0.64%)
:arrow_up:

...manager/realtime/LLRealtimeSegmentDataManager.java
53.67% <0%> (+0.79%)
:arrow_up:

... and 18 more

Continue to review full report at Codecov.

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


brightness

Made brightness global across all modes.
Added colorWipe Mode

Had a read through your code, and haven't tested anything, but I think you should split the two features in this branch into separate ones. This will make it easier to track in the long run and easier to understand which commits relate to each feature. Essentially keeping 1 feature to 1 pull. Have a read of the comments. I made a few which need to be tidied up before merging in. But first I would have a read of the below.
I really like the looks of the colour wipe mode, always great to see some new modes contributed. But the global brightness idea has both some positive and negatives for implementation with the way I have written my previous modes.
I have read over your code and mine quite a few times with regards this, and from what I can see it contradicts a couple of things. The first is the way the colour's are set for each mode that uses colour selection (ie colour, clock, and bell curve). In essence, you set each RGB channel which technically handles the brightness component at the same time as colour selection. Since you can set each of the channels to a value of 0 to 255 a dim red light could be CRGB(64, 0, 0) set from the RGB picker of the website for example. This allows the user to use a colour picker like the one I have chosen to also adjust the brightness as well as colour at the same time.
The issue with a global brightness over the top of RGB values, and the way FastLED would handle it, means that an already dim light will be dimmed further. This could lead to confusing or undesirable results when trying to use the light as there are essentially two brightness values you will have to adjust to get the brightness you want.
The second issue would be if someone else were to write a mode which they did not want global dimming to occur, such as my night rider mode. I wrote this to be simple and just do one thing at a set brightness. Decreasing its brightness would affect the way it looks and therefore defeat what I was trying to achieve. Again this might not be the end of the world but it does in the long run restrict the implementation of new modes. Instead I encourage new modes to use their own brightness variables. These will then be saved in flash and reloaded each time a mode is changed allowing for a more dynamic setting.
The global brightness would also affect the way the visualiser brightness is set so I would not want it to be applied during that mode.
I understand why you would want to use a global brightness in modes such as rainbow and new modes such as the ones you have written. It also makes sense from a control point of view by being able to send the brightness as a single message rather than a mode with the correct values. But hopefully from the explanation above you can understand why I have not done so in the first place.
I absolutely encourage you to use my code in any way that you want and to modify and play with it till you are happy, but I think for the meantime I will have to pass on the global brightness setting in the main branch. If you can figure out a decent way of being able to implement it without affecting colour accuracy or mode behavior I am all ears. If in the mean time you need help merging in the new modes into a pull request I can give you a hand.

I understand your concern (I think) and I'll see if I can come up with a solution. Being said, I do like to be able to set the Knight Rider mode a bit more dimmed (doesn't affect the coolness in anyway I think).
I can't comment on the visualiser, since I don't have anything connected to the analogue input yet...

Understandable. Had a good think in the car on the way home. It might all be achievable with a method like maximizeBrightnes from fastLED.

You may close this pull request. At the moment I'm working on a whole new program, combining your work with a few others. I'll let you know when I have a working example.

No worries. Thanks for your efforts. Let us know when you are done to see if there is a chance for merging.


highlight does not move with urwid-1.3.1

I was able to make this work with urwid-2.0.1, so mostly informational.  I had to unmask it in my Gentoo installation.
When I was using the keys to move between rows, the blue highlight stayed at the top.  By hitting enter, or going to the next page, I could finally see it was selecting different tasks but the highlight never moved.  Upgrading urwid solved this.

Thanks for this report. As a remedy going forward, I've forced minimum versions of all required packages, which as of the fix, are:
pytz>=2019.3
tasklib>=1.3.0
tzlocal>=2.0.0
urwid>=2.1.0


Unable to use Action on Self-Hosted Windows Runner

We are trying using the Windows version of action to publish unit tests, but we are getting below error as mentioned in the screen shot.

Below is the task which we have added in the yml file.
- name: Setup Python
  uses: actions/setup-python@f677139bbe7f9c59b41e40162b753c062f5d49a3
  with:
    python-version: '3.10'

- name: Publish unit tests results
  uses: EnricoMi/publish-unit-test-result-action/windows@30eadd5010312f995f0d3b3cff7fe2984f69409e
  if: always() && ${{ (inputs.unittests) || (github.event_name == 'pull_request') }}
  with:
    files: |
      ${{ github.workspace }}/src/TestResults.xml
      ${{ github.workspace }}/src/CoverageReport/Cobertura.xml
      ${{ github.workspace }}/src/OpenCover.xml

Looks like you do not have PowerShell (pwsh) installed.

Thannks for your reply @EnricoMi , I have Powershell installed already in my machine and same is updated at environment variables also.

and most of my steps inside workflow runs using shell: powershell only

Looks like PowerShell v5 provides powershell.exe whereas PowerShell v6 and v7 provide pwsh.exe.

Can you test the following please:
uses: EnricoMi/publish-unit-test-result-action/windows/powershell@windows-powershell

This action is tag is not allowed in my enterprise organization. but let me download the action manually and test once please.

@EnricoMi , I dont find any action/sub folder for powershell under windows folder in the code. Can i still use the action as you mentioned?

uses: EnricoMi/publish-unit-test-result-action/windows/powershell@windows-powershell

Check out branch windows-powershell, there is windows/powershell.

I have tried this - uses: EnricoMi/publish-unit-test-result-action/windows/powershell@windows-powershell
but I am getting this error every time. Module not found for 3.8, 3.10 & 3.12 versions of python.

Looks like powershell has a different syntax than pwsh. I would need to setup a powershell environment to migrate and test the pwsh script.

Thank you @EnricoMi, I have upgraded my powershell to v7 and now i can able to use the action and it seems working fine with v7. But few of the file formats are accepting it seems.
Below is the output summary of  action with TestResults.xml
Is OpenCover.xml & Cobertura.xml are accepted formats in this action?
ERROR LOG:

Sounds like those file contain test coverage information, which are not test result information.

yes @EnricoMi , we have coverage information in it. do we have any plan to create an action to support coverage as well please?

Adding coverage sounds like a natural fit for this action. But I have not looked into this. Happy to review pull requests on this area.

I have created issues #640 and #639 to track these new features. Please reopen if there are any further issues.


Add discard confirmation dialog when pressing back

Show a discard confirmation dialog when the user presses the back button while composing a message and drafts are disabled.

nice work.


with_min_len/with_max_len doesn't work with u64 ranges because they don't implement the indexed trait

This is highly surprising and I think it definitely should not work like this.
My guess is that u64 should be used instead of usize everywhere, since programs can process more than 2^32 iterator items (but not more than 2^64 due to limited CPU speeds and human lifetimes).

Note that the standard library has ExactSizeIterator with the method fn len(&self) -> usize, which is not implemented by Range<u64>.  I think it would be more surprising if Rayon departed from the established pattern of using usize lengths.
That said, I agree it would be nice to improve the support for 64-bit ranges, somehow.

There might be a design mistake in the standard library too.
That said, the standard library mostly/only uses ExactSizeIterator to pre-allocate space for collecting from an iterator, and by definition you can't do that for values greater than usize::max_value().
On the hand, Rayon, unlike the standard library, uses it to split the iterator, and here u64 makes sense, since a 32-bit program can still execute more than 2^32 data items (but probably not more than 2^64, at least with current technology) even though it can't store them in memory.
There could be an argument that it should actually be u128, since if Rayon were to add support for distributing task over a network, it would become possible to run 2^64-2^128 tasks.

You can accomplish any bit-depth by nesting iterators, like:
    let last = (0u32..1_000_000_000)
        .into_par_iter()
        .map(|i| i as u128 * 1_000_000_000)
        .flat_map(
            |i| {
                (0u32..1_000_000_000)
                    .into_par_iter()
                    .map(move |j| i + j as u128)
            }
        )
        .map(|i| i as u128 * 1_000_000_000)
        .flat_map(
            |i| {
                (0u32..1_000_000_000)
                    .into_par_iter()
                    .map(move |j| i + j as u128)
                    .with_min_len(1_000)
                    .with_max_len(1_000_000)
            }
        )
        .find_last(|_| true);
    println!("{:?}", last);

That's iterating 0..10^27, with each sequential job limited between a thousand and a million items, and this correctly reports Some(999999999999999999999999999).  It doesn't even take very long since find_last prunes most of the lesser splits as soon as anything is found.
You can still argue that this is harder to set up than it should be, but it does work.

To what extent is our use of usize something that the "end-user sees" versus an implementation detail?
One thing that maybe comes to mind is that, when we switch to the "sequential iteration" phase, we currently do have some annoyances around ExactSizeIterator, although I think that's kind of orthogonal from this (that's more to do w/ rev() and zip(), I guess). But not wholly independent.
Another question, could we just make u64 ranges work but panic dynamically if they span a gap bigger than usize::MAX and use with_min_len or whatever?

To what extent is our use of usize something that the "end-user sees" versus an implementation detail?

len(), enumerate(), position_any() et al., and indirectly collect_into(), all expose usize.

Another question, could we just make u64 ranges work but panic dynamically if they span a gap bigger than usize::MAX and use with_min_len or whatever?

We could assert in len() -- we already cheat that way with chain().  But that doesn't satisfy @winsut's desire to actually iterate more than usize::MAX items.  Or if you want to actually use u64 lengths internally, then we'd have to assert on all the other usize-ish APIs.

Yes, OK. An annoying problem. I do certainly agree that e.g. enumerate() should match the Iterator signature. It may be worthwhile trying to convert the "internal" APIs to use u64, though. Unclear.

Discussing in the meeting. I think the inclination is not to solve this for 1.0 -- even if we found a satisfactory solution, it wouldn't have much time to "Bake" and it involves tinkering at a fairly low level with the IndexedParallelIterator trait and so forth.
We can mitigate by adding inherent methods to Range<u64> and would consider fixing this either via a 2.0 release someday or if default associated types or some other language feature are added to rust that makes it more plausible.


Support "Multiple Instances"

Project Type
3D
Feature Description
I'm not sure if this is a bug or a feature request. Whenever I start with multiple instances (i.e. under Godot Debug settings), one of the Cameras seems to lose its "Follow Parameters". I haven't looked too deeply into this but can only assume it has something to do with a singleton pattern
https://github.com/ramokz/phantom-camera/assets/13342266/7462fa47-8686-450c-b36a-dc670d2c94a9
Use Cases
As the title and description explain, allows support for Godot multi-instance
Importance
High - there are critical things I can't do without this feature
Usage
All the time - the feature can be very useful for a varity of different projects
(Optional) Proposed Solution
No response

That is very odd.
I tried running 3 instances of the 3DFollowFramedExampleScene.tscn, but each instance didn't seem to replicate the issue with losing their follow parameters when running the scene.
Are you applying any of the properties via code, or is it all done from the inspector?
Sample video
https://github.com/ramokz/phantom-camera/assets/5159399/054001c4-5719-43b6-a582-e099263f4dfe

I just spotted you're using a PCam in a Framed Follow mode, which might be the same issue as in #143.
Try replacing your PhantomCamera3D.gd with the version attached below and see if that solves it:
phantom_camera_3D.gd.zip

Hey there, thanks for the followup. I think this is actually a bug with Framed cameras sometimes not initializing their offset. I've submitted a PR to fix https://github.com/ramokz/phantom-camera/pull/147

Closing as issue has been resolved in #147


How to get code coverage

Hi, The JQF is very beautiful. I integrate the JQF to my project. but i hava some question. How to get the code coverage with ZestGuidance? Do you  hava some example?  I see always 0 when it running.
I hava readed the paper "https://people.eecs.berkeley.edu/~rohanpadhye/files/zest-issta19.pd". I think it must be run a javaagent when i wanted the coverage data. is that right?
so,i add some command when execute test ,such as "-javaagent:/xxx/jqf-instrument-1.4-SNAPSHOT.jar" . javaagent is running ,but it has some failure.

"name == this.className", is alway false. make sure there is don't use "name.equals(this.className)"?
when i rewrite the code to "equals". there is another Exception...

so....how to get code coverage?
thank you for you reading.please help me .

Hi @snakefor100.

I hava readed the paper "https://people.eecs.berkeley.edu/~rohanpadhye/files/zest-issta19.pd". I think it must be run a javaagent when i wanted the coverage data. is that right?

You do not have to specify the javaagent yourself. You can run Zest either via the script bin/jqf-zest or by using the Maven plugin. Both of these handle launching the instrumentation for you. The jqf-zest script will use the appropriate javaagent and the Maven plugin will use a custom instrumenting class-loader. Check out the following tutorials:
https://github.com/rohanpadhye/jqf/wiki/Fuzzing-with-Zest (shows usage of the jqf-zest launcher)
https://github.com/rohanpadhye/jqf/wiki/Fuzzing-a-Compiler (shows usage of the Maven plugin)
The advantage of the Maven plugin is that you do not need to download this repository locally. You can simply add a dependency on JQF via Maven and fuzz your project. The disadvantage is that you cannot instrument JDK classes that are loaded by the bootstrap classloader.
If you want to extend/modify the fuzzing logic, then using bin/jqf-zest is probably the best option. Please follow the tutorial and let me know if you have any issues with getting code coverage.

"name == this.className", is alway false. make sure there is don't use "name.equals(this.className)

This probably looks like a bug. I don't know how this assert never failed until now >_<. Perhaps the name was always the same object? Changing that to equals seems like a good idea.

What Java version have the target classes been compiled with? I just pushed an update that upgrades the dependency on the ASM bytecode toolkit. Can you tell me if you still see the error after these changes?

What Java version have the target classes been compiled with? I just pushed an update that upgrades the dependency on the ASM bytecode toolkit. Can you tell me if you still see the error after these changes?

I retry test case, but still has error. my project use jdk8. i find two error .
1:when the agent runing, it will load many classes, some class is NestHostClass.so it will be run this code:

In the visitNestHostExperimental method, we can see this code：

the api is Opcodes.ASM5,it is fixed.

so ,there is always UnsupportedOperationException when the class is NestHostClass. i don't know  if the error  had any impact, when the error is happened, it  can not  block the agent runing.
2: the second error make me confusion. when the agent runing, it will be has NoClassDefFoundError. it looks like the SingleSnoop class has some error,but i can't find why the error happened.

when the agent load class to enhance, is that contain jqf'class? if it is , can be exclude? because it looks like unnecessary

Hi @snakefor100.

I hava readed the paper "https://people.eecs.berkeley.edu/~rohanpadhye/files/zest-issta19.pd". I think it must be run a javaagent when i wanted the coverage data. is that right?

You do not have to specify the javaagent yourself. You can run Zest either via the script bin/jqf-zest or by using the Maven plugin. Both of these handle launching the instrumentation for you. The jqf-zest script will use the appropriate javaagent and the Maven plugin will use a custom instrumenting class-loader. Check out the following tutorials:
https://github.com/rohanpadhye/jqf/wiki/Fuzzing-with-Zest (shows usage of the jqf-zest launcher)
https://github.com/rohanpadhye/jqf/wiki/Fuzzing-a-Compiler (shows usage of the Maven plugin)
The advantage of the Maven plugin is that you do not need to download this repository locally. You can simply add a dependency on JQF via Maven and fuzz your project. The disadvantage is that you cannot instrument JDK classes that are loaded by the bootstrap classloader.
If you want to extend/modify the fuzzing logic, then using bin/jqf-zest is probably the best option. Please follow the tutorial and let me know if you have any issues with getting code coverage.

"name == this.className", is alway false. make sure there is don't use "name.equals(this.className)

This probably looks like a bug. I don't know how this assert never failed until now >_<. Perhaps the name was always the same object? Changing that to equals seems like a good idea.

It looks like the method visitNestHostExperimental is deprecated in ASM.  Do you (or does anyone else) know what causes it to be invoked during instrumentation? I have never encountered a class with nested hosts in any of the applications that I have fuzzed before.

emm..i add some classes to "janala.excludes",then i solve this problem. thank you for your help.Wish the jqf better and better in the future.
finally,i have one suggest, this code:

it looks like the default value is true is better. because i just follow with interest my code, i don't care the coverage of other jar.if the default is false, the agent will load all classes in my classloader to enhance.

The default value is false because by default we want to instrument as much as possible, unless the user has explicitly requested excluding some classes.
If the default was true for shouldExclude, then first-time users would see nothing being instrumented, and it would be very difficult to manually include every single application class/package.
Anyway, thanks for the follow up and update @snakefor100 !


Is it possible to select white as color?

I have the same issue, because I would like to choose a gray scale. Is it possible?


Add L2A Cloud Optimized GeoTIFFs on AWS S3

Link: https://registry.opendata.aws/sentinel-2-l2a-cogs/

Thank you for adding this @kylebarron - this is the first STAC COG entry into the list, which I am pretty excited about. Let's hope there's more to come.

I guess the others don't have STAC metadata, but it appears that several others are in COG. I think at least portions of AWS's Sentinel 1, 3, and 5P archives are in COG. And there's also a sort of analysis ready data product derived from Sentinel 1 that's in COG: https://registry.opendata.aws/sentinel-1-rtc-indigo/


Which readMode to use?

We have Redis cluster with 6 nodes (3 masters and 3 slaves), and using redisson 3.6.5. In redisson cluster setting, we applied default value (SLAVE ) on readMode.
Here are some questions on this setting:

What will happen if one of slave node fails? Will redisson detect the error and switch to master on Read, or it will try to connect to Slave only and throw exception?

If we use MASTER_SLAVE, read operation will go to master/slave randomly or it has a way to go to slave first, and then master when slave fails?

Redisson uses MASTER if no SLAVES are available.

Load balancer is used defined through loadBalancer setting


Issues with the provided ansible playbook

[Enter feedback here]
There are some issues with the provided ansible script.

Copy past to vscode gives spacing errors.
ansible_system does not exist in windows facts
When a job is is skipped because condition is not met the register will be run. Because windows and linux task are using the samen register names this gives erros with next job.

I have rewrite the playbook so it works for us for linux and windows servers.
While typing this I think you could also use a block for linux and windows. So you have less checking.
---
- name: Onboard Linux and Windows Servers to Azure Arc-enabled servers with public endpoint connectivity
  hosts: all
  # vars:
  #   azure:
  #     service_principal_id: 'INSERT-SERVICE-PRINCIPAL-CLIENT-ID'
  #     service_principal_secret: 'INSERT-SERVICE-PRINCIPAL-SECRET'
  #     resource_group: 'INSERT-RESOURCE-GROUP'
  #     tenant_id: 'INSERT-TENANT-ID'
  #     subscription_id: 'INSERT-SUBSCRIPTION-ID'
  #     location: 'INSERT-LOCATION'
  tasks:
  - name: Check if the Connected Machine Agent has already been downloaded on Linux servers
    stat:
      path: /usr/bin/azcmagent
      get_attributes: False
      get_checksum: False
    register: azcmagent_lnx_downloaded
    when: ansible_system == 'Linux'

  - name: Download the Connected Machine Agent on Linux servers
    become: yes
    get_url:
      url: https://aka.ms/azcmagent
      dest: ~/install_linux_azcmagent.sh
      mode: '700'
    when: (ansible_system == 'Linux') and (azcmagent_lnx_downloaded.stat.exists == false)

  - name: Install the Connected Machine Agent on Linux servers
    become: yes
    shell: bash ~/install_linux_azcmagent.sh
    when: (ansible_system == 'Linux') and (not azcmagent_lnx_downloaded.stat.exists)

  - name: Check if the Connected Machine Agent has already been downloaded on Windows servers
    win_stat:
      path: C:\Program Files\AzureConnectedMachineAgent
    register: azcmagent_win_downloaded
    when: ansible_os_family == 'Windows'

  - name: Download the Connected Machine Agent on Windows servers
    win_get_url:
      url: https://aka.ms/AzureConnectedMachineAgent
      dest: C:\AzureConnectedMachineAgent.msi
    when: (ansible_os_family == 'Windows') and (not azcmagent_win_downloaded.stat.exists)

  - name: Install the Connected Machine Agent on Windows servers
    win_package:
      path: C:\AzureConnectedMachineAgent.msi
    when: (ansible_os_family == 'Windows') and (not azcmagent_win_downloaded.stat.exists)

  - name: Check if the Connected Machine Agent has already been connected
    become: true
    command:
     cmd: azcmagent check
    register: azcmagent_lnx_connected
    ignore_errors: yes
    when: ansible_system == 'Linux'
    failed_when: (azcmagent_lnx_connected.rc not in [ 0, 16 ])

  - name: Check if the Connected Machine Agent has already been connected on windows
    win_command: azcmagent check
    register: azcmagent_win_connected
    when: ansible_os_family == 'Windows'
    ignore_errors: yes
    failed_when: (azcmagent_win_connected.rc not in [ 0, 16 ])

  - name: Connect the Connected Machine Agent on Linux servers to Azure Arc
    become: yes
    shell: azcmagent connect --service-principal-id "{{ azure.service_principal_id }}" --service-principal-secret "{{ azure.service_principal_secret }}" --resource-group "{{ azure.resource_group }}" --tenant-id "{{ azure.tenant_id }}" --location "{{ azure.location }}" --subscription-id "{{ azure.subscription_id }}"
    when:  (ansible_system == 'Linux') and (azcmagent_lnx_connected.rc is defined and azcmagent_lnx_connected.rc != 0)

  - name: Connect the Connected Machine Agent on Windows servers to Azure
    win_shell: '& $env:ProgramFiles\AzureConnectedMachineAgent\azcmagent.exe connect --service-principal-id "{{ azure.service_principal_id }}" --service-principal-secret "{{ azure.service_principal_secret }}" --resource-group "{{ azure.resource_group }}" --tenant-id "{{ azure.tenant_id }}" --location "{{ azure.location }}" --subscription-id "{{ azure.subscription_id }}"'
    when: (ansible_os_family == 'Windows') and (azcmagent_win_connected.rc is defined and azcmagent_win_connected.rc != 0)

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

ID: a88d4cbe-fe02-3ec4-d952-e2288e68658d
Version Independent ID: c2a1531e-24ad-b7f5-6dea-58c26959cda0
Content: Connect machines at scale using Ansible Playbooks - Azure Arc
Content Source: articles/azure-arc/servers/onboard-ansible-playbooks.md
Service: azure-arc
Sub-service: azure-arc-servers
GitHub Login: @johnmarco
Microsoft Alias: johnmarc

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

#sign-off

#please-close


[cherry-pick]enhancement support BE http action: list_cache and clear (#41037)

Proposed changes
Add a http action which is useful when you debug.
API
GET /api/file_cache

request parameter
request parameter1

param
type
desc
require

op
string
the value must be list_cache, other value you can refer to #40831 #37484
yes

value
string
the segment file name
yes

request parameter2

param
type
desc
require

op
string
the value must be clear, other value you can refer to #40831 #37484
yes

value
string
the segment file name
yes

sync
bool
clean local cache in sync
no

response
response1
if success

param
type
desc

array
return the segment file cache in local path

if fail

param
type
desc

array
empty array

response2
if success

param
type
desc

status
string

msg
string

example
case 1
curl  '<IP_ADDRESS>:8040/api/file_cache?op=list_cache&value=0200000000000001bf42c14374fff491ffb7c89a1a65c5bb_0.dat'

return
["/opt/doris/be/file_cache/c6a/c6a599f453f67f0949f80ad9990fa3dd/0"]

case 2
curl '<IP_ADDRESS>:8040/api/file_cache?op=clear&sync=true&value=0200000000000001284b68fea3dcfe8a83e65cd88426b081_0.dat'

return
{
    "status": "OK",
    "msg": "OK"
}

(cherry picked from commit 99d07480d3ae685f57b6cf61866343863b481489)
What problem does this PR solve?

Issue Number: close #xxx

Related PR:  cherry-pick #41037
Problem Summary:
Check List (For Author)

Test

[x] Regression test
[ ] Unit Test
[ ] Manual test (add detailed scripts or steps below)
[ ] No need to test or manual test. Explain why:

[ ] This is a refactor/code format and no logic has been changed.
[ ] Previous test can cover this change.
[ ] No colde files have been changed.
[ ] Other reason

Behavior changed:

[x] No.
[ ] Yes.

Does this need documentation?

[x] No.
[ ] Yes.

Release note

Check List (For Reviewer who merge this PR)

[ ] Confirm the release note
[ ] Confirm test cases
[ ] Confirm document
[ ] Add branch pick label

run p0

run buildall


refactor: eliminate useless '_has_metadata' function

Remove the '_has_metadata' function as it does not serve a purpose. Initially, it was designed to determine whether a local dataset group had a member on GBIF and was used to guide decision logic concerning resetting dataset endpoints and re-uploading metadata in the event of a dataset update. However, it became apparent that this function returned
'True' even if only boilerplate stand-in metadata was posted to GBIF
before the actual metadata was posted during a crawl operation.

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (53219b6) 84.88% compared to head (f34dcff) 84.58%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@               Coverage Diff               @@
##           development      #54      +/-   ##
===============================================
- Coverage        84.88%   84.58%   -0.30%
===============================================
  Files                6        6
  Lines              258      253       -5
===============================================
- Hits               219      214       -5
  Misses              39       39

Files
Coverage Δ

src/gbif_registrar/_utilities.py
88.28% <ø> (-0.45%)
:arrow_down:

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


Issue with invalid use of aria-* attributes

doing some checks related to #49, I noticed that the link test was not returning any instances of invalid use of aria-* attributes, even though no aria-* attributes are allowed on <link>.  Is there a bug here?

Hi @scottaohara I think it's because the links all have invalid roles, so it doesn't get to the aria-* checks.


Prometheus query_range is slow

What did you do?
I am requesting Prometheus's query_range interface between 15 seconds and 25 seconds every minute, and the number of requests is about 200,000 to 300,000 every minute.
What did you expect to see?
Prometheus can return data smoothly.
What did you see instead? Under which circumstances?
The interface of Prometheus sometimes returns data very slowly
Environment
Here is the yaml file of my Prometheus:
apiVersion: apps/v1
kind: Deployment
metadata:
  annotations:
    deployment.kubernetes.io/revision: '29'
    meta.helm.sh/release-name: sunfire-prometheus
    meta.helm.sh/release-namespace: default
  creationTimestamp: '2021-10-27T06:36:49Z'
  generation: 29
  labels:
    app: sunfire-prometheus
    app.kubernetes.io/managed-by: Helm
    chart: sunfire-prometheus-0.1.0
    group: group1
    heritage: Helm
    release: sunfire-prometheus
spec:
  progressDeadlineSeconds: 600
  replicas: 1
  revisionHistoryLimit: 10
  selector:
    matchLabels:
      app: sunfire-prometheus
      group: group1
      release: sunfire-prometheus
  strategy:
    rollingUpdate:
      maxSurge: 25%
      maxUnavailable: 25%
    type: RollingUpdate
  template:
    metadata:
      annotations:
        prometheus.io/path: /metrics
        prometheus.io/port: '9090'
        prometheus.io/scheme: http
        prometheus.io/scrape: 'true'
      labels:
        app: sunfire-prometheus
        chart: sunfire-prometheus-0.1.0
        group: group1
        heritage: Helm
        release: sunfire-prometheus
    spec:
      containers:
        - args:
            - '--config.file=/etc/config/prometheus.yaml'
            - '--storage.tsdb.path=/prometheus-data'
            - '--web.enable-lifecycle'
            - '--storage.tsdb.retention.time=5m'
            - '--storage.tsdb.max-block-duration=5m'
            - '--storage.tsdb.min-block-duration=5m'
            - '--storage.tsdb.retention.size=2GB'
            - '--query.max-concurrency=4000'
            - '--web.max-connections=40000'
          env:
            - name: HOST_IP
              valueFrom:
                fieldRef:
                  apiVersion: v1
                  fieldPath: status.podIP
            - name: PORT
              value: '9090'
            - name: POD_NAME
              valueFrom:
                fieldRef:
                  apiVersion: v1
                  fieldPath: metadata.name
            - name: GROUP
              valueFrom:
                fieldRef:
                  apiVersion: v1
                  fieldPath: 'metadata.labels[''group'']'
            - name: IDC
              value: center-idc
            - name: ROLE
              value: prometheus
            - name: HEALTH_CHECK_URL
              value: 'http://localhost:9090/-/healthy'
            - name: BRAIN_HB_URL
              value: 'http://sunfire-brain:28123/heartbeat'
          image: >-
            xxxxxx
          imagePullPolicy: Always
          name: sunfire-prometheus-group1
          ports:
            - containerPort: 9090
              protocol: TCP
          readinessProbe:
            failureThreshold: 12
            httpGet:
              path: /-/ready
              port: 9090
              scheme: HTTP
            initialDelaySeconds: 30
            periodSeconds: 17
            successThreshold: 8
            timeoutSeconds: 15
          resources:
            limits:
              cpu: '4'
              memory: 8Gi
            requests:
              cpu: '1'
              memory: 2Gi
          terminationMessagePath: /dev/termination-log
          terminationMessagePolicy: File
          volumeMounts:
            - mountPath: /etc/config
              name: config-volume
            - mountPath: /prometheus-data
              name: storage-volume
      dnsPolicy: ClusterFirst
      restartPolicy: Always
      schedulerName: default-scheduler
      securityContext: {}
      serviceAccount: default
      serviceAccountName: default
      terminationGracePeriodSeconds: 5
      volumes:
        - configMap:
            defaultMode: 420
            name: sunfire-prometheus-group1
          name: config-volume
        - emptyDir: {}
          name: storage-volume

System information:

Prometheus version:

Alertmanager version:

Prometheus configuration file:

global:
  scrape_interval: 15s
  scrape_timeout: 15s
  query_log_file: /prometheus/query.log
scrape_configs:
- job_name: pushgateway
  honor_labels: true
  kubernetes_sd_configs:
  - role: pod
  relabel_configs:
  - source_labels: [__meta_kubernetes_pod_annotation_prometheus_io_scrape]
    action: keep
    regex: true
  - source_labels: [__meta_kubernetes_namespace]
    action: keep
    regex: default
  - source_labels: [__meta_kubernetes_pod_label_group]
    action: keep
    regex: group1
  - action: labeldrop
    regex: pushgateway
  - action: labeldrop
    regex: siteName

Alertmanager configuration file:

----

Logs:
Here are some typical query logs:

{"error":"query was canceled in expression evaluation","httpRequest":{"clientIP":"<IP_ADDRESS>","method":"GET","path":"/api/v1/query_range"},"params":{"end":"2022-03-30T09:35:59.000Z","query":"jvm_threads_current{appName=\"sunfire-selfmonitor-reduce.url4625\", }","start":"2022-03-30T09:35:00.000Z","step":60},"stats":{"timings":{"evalTotalTime":5.672762408,"resultSortTime":0,"queryPreparationTime":5.464064693,"innerEvalTime":0,"execQueueTime":0.000004868,"execTotalTime":5.995925647}},"ts":"2022-03-30T09:35:22.686Z"}

{"error":"query was canceled in expression evaluation","httpRequest":{"clientIP":"<IP_ADDRESS>","method":"GET","path":"/api/v1/query_range"},"params":{"end":"2022-03-30T09:38:59.000Z","query":"jvm_memory_pool_bytes_max{appName=\"sunfire-selfmonitor-reduce.url1927\", pool=~\"Code Cache\"}","start":"2022-03-30T09:38:00.000Z","step":60},"stats":{"timings":{"evalTotalTime":3.5984797520000003,"resultSortTime":0,"queryPreparationTime":2.605931976,"innerEvalTime":0,"execQueueTime":0.911022524,"execTotalTime":5.196457487}},"ts":"2022-03-30T09:38:25.080Z"}

{"error":"query was canceled in expression evaluation","httpRequest":{"clientIP":"<IP_ADDRESS>","method":"GET","path":"/api/v1/query_range"},"params":{"end":"2022-03-30T09:38:59.000Z","query":"jvm_buffer_pool_used_bytes{appName=\"sunfire-selfmonitor-reduce.url132\", pool=~\"mapped\"} / jvm_buffer_pool_capacity_bytes{appName=\"sunfire-selfmonitor-reduce.url132\", pool=~\"mapped\"}","start":"2022-03-30T09:38:00.000Z","step":60},"stats":{"timings":{"evalTotalTime":3.5998245989999997,"resultSortTime":0,"queryPreparationTime":2.607460066,"innerEvalTime":0,"execQueueTime":0.910834909,"execTotalTime":5.20090463}},"ts":"2022-03-30T09:38:25.081Z"}

I get the CPU time of the program through pprof:

 - '--storage.tsdb.retention.time=5m'
 - '--storage.tsdb.max-block-duration=5m'
 - '--storage.tsdb.min-block-duration=5m'

The reason I use these three parameters is because I want to reduce the memory usage as much as possible (currently this should keep the memory at 5 minutes of data, right?)
I don't know if it is because of the setting of these parameters that the CPU consumption is too high, because I see that the official does not recommend setting these parameters.

Hello,
We unfortunately do not support that short block durations. A lot of things can happen with such values. It also means that you will always by compacting the TSDB in some way, which can cause quite big CPU consumption.

@roidelapluie OK, thanks. It looks like I still need to use your recommended parameter configuration.

It's not possible, but I think we should have the option to do so (not generate blocks at all, only work from wal).

Is there any proposal or plan about the  idea of supporting the relative short duration?

See https://github.com/prometheus/prometheus/issues/9595


Extension causing issue with Git integration of VS Code

See microsoft/vscode#69334 for details.
There was some bug on VS Code which was fixed but the same bug as described over there still occurs when the Kubernetes extension is active. As soon as I disable it and reload VS Code the issue is gone.
Here's a video of demonstrating this issue: https://youtu.be/-ZTeocap3uw

How weird.  We don't do anything with SCM or with images so I don't know why a deleted image in Git would cause us to blow up.  I'll take a look - thanks for letting us know!

@itowlson Yep. Of course I disabled all SCM related extensions first and was surprised when disabling this extension, fixed the problem.
Let me know if you can reproduce.

Yes, it's reproducing for me (on Windows).  THIS IS SO WEIRD.
But it gets weirder.  The hang happens EVEN IF THE EXTENSION DOESN'T ACTIVATE.
I don't know how this is even possible.
^^ reassuring maintainer comments

Found it, and I'm afraid it's a VS Code bug, which I will raise on their repo.

@levrik Is it okay if I close this on the k8s extension side please?

@itowlson Sure.

@itowlson This issue was fixed on VS Code's side in the latest 1.36.0 insiders version (https://github.com/microsoft/vscode/commit/7bc29bf67c17608b11e23a8497c8f1de5a0922cf, 2019-06-11T08:29:28.099Z)

That's great @levrik - thanks for the followup and for your help in identifying this.


Add Jorstad et al., 2023 as a reference to CL terms for cortical neurons mentioned in the paper

The paper from Jorstad et al., 2023 DOI: 10.1126/science.adf6812  provides an extensive scRNA-seq taxonomy on cortical cell types.
We decided to include it as a database_cross_reference in cortical neuron CL terms.
These are the CL terms where we are including the reference:

L5/6 near-projecting glutamatergic neuron
L6 intratelencephalic projecting Car3 glutamatergic neuron
L5 extratelencephalic projecting glutamatergic cortical neuron
L6b glutamatergic cortical neuron
L6 corticothalamic-projecting glutamatergic cortical neuron
L2/3 intratelencephalic projecting glutamatergic neuron
L4 intratelencephalic projecting glutamatergic neuron
L5 intratelencephalic projecting glutamatergic neuron
chandelier cell
pvalb GABAergic cortical interneuron
sst GABAergic cortical interneuron
sst chodl GABAergic cortical interneuron
lamp5 GABAergic cortical interneuron
sncg GABAergic cortical interneuron
vip GABAergic cortical interneuron


Recalculating excel formulas

Some TIMES models rely on ~FILL tables to retreive the data specified elsewhere in the model files and link those tables through formulas to other input tables. openpyxl does not support recalculation of formulas.

It seems that one could use some JS packages for this purpose, e.g. https://github.com/fabiooshiro/xlsx-calc

On Ubuntu we could probably use libreoffice-calc for this, as already used in CI to convert between xls and xlsx

Or we could also try this python package: https://pypi.org/project/formulas/


Misdescribed bug: "Inline elements are not treated as flex-items"

Bug 12, "Inline elements are not treated as flex-items", seems to be
misdescribed, at least for IE11.  With that browser, on Windows 7, if I open
up the workaround (12.1.b, at http://codepen.io/philipwalton/pen/NqLoNp) and
change display: block to display: inline, the bug doesn't strike and the
page is unchanged.
If I reload the page and add
.FlexContainer::before
{
  display: inline;
}

at the end of the CSS, the bug doesn't strike and the page is unchanged.

Hmmm, it looks like you're right. It seems pseudo-elements don't work when no display value is used, but when a display value of inline is used, they work fine. That makes me wonder what default value IE11 gives them.
Unfortunately I don't think it's possible to tell, since the developer tools don't let me select the pseudo-element.


pynYNAB no longer requires dev version

When running pip install -r requirements.txt it fails on pynYNAB===dev.
I removed ===dev, and updated main.py. The import statement failed
with the pip installed version of pynYNAB.

Thanks a lot for this!


Improve code coverage:  Added testcases for file  JSR310DateConverters, RandomUtil, ProblemDetailWithCause

Description:
Added few test cases for

JSR310DateConverters
RandomUtil
ProblemDetailWithCause file

Impact:
Improved code coverage by 1% overall.

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.0 out of 2 committers have signed the CLA.:x: DISHA ANAND:x: DishaAnandDISHA ANAND seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@DishaAnand can you sign the CLA?

@DishaAnand can you sign the CLA?

Hi @DanielFran
Thank you for informing me,
I have signed the CLA
Kindly have a look at the PR..

Thanks @DishaAnand for the contribution


[BUG] Get-PnPMicrosoft365Group: AADSTS1002012 when using certificate auth

Reporting an Issue or Missing Feature
When using Get-PnPMicrosoft365Group with certificate app-only authentication, you get an error.
Expected behavior
Microsoft 365 groups are returned
Actual behavior
Error: Get-PnPMicrosoft365Group: AADSTS1002012: The provided value for scope Group.ReadWrite.All is not valid. Client credential flows must have a scope value with /.default suffixed to the resource identifier (application ID URI).
Steps to reproduce behavior
Connect-PnPOnline -Url https://domain-admin.sharepoint.com -CertificatePath PnPPowerShell.pfx -ClientID:clientidhere -tenant:domain.com
Get-pnpMicrosoft365Group
What is the version of the Cmdlet module you are running?
2.10.0
Which operating system/environment are you running PnP PowerShell on?

[X] Windows
[ ] Linux
[ ] MacOS
[ ] Azure Cloud Shell
[ ] Azure Functions
[ ] Other : please specify

This does seem to be an issue particular to 2.10.0. I rolled backed to 2.9.0 and the same command works.

@danmyhre - thanks for reporting and apologies for this, its indeed a regression in 2.10 . We have fixed this and the fix is in the nightly builds. Closing this issue. We will be releasing 2.11 shortly to address this .


Date Histogram is not generating date columns in sequence

Date Histogram is not generating date columns in sequence.

On giving histogram interval as DAY, It should have generated dates in sequence, my date format is YYYY-MM-DD, but sorting is not date wise. Am I missing something here.

Hi,
You can have the columns in the right order, by checking "sort split columns" option, in "Options" tab.


Possibly incorrect halo region filling and missing boundary condition

Two issues here:

We are missing a boundary condition: namely, a NoPenetration boundary condition which denotes that the normal component of the velocity field is zero at a boundary. Currently we are using a convention whereby the name of the field (in this case, "v"), along with a no flux boundary condition is used to infer the no penetration condition:

https://github.com/climate-machine/Oceananigans.jl/blob/ac7dc2a917f990f1c709fee597309b8edac01cd7/src/halo_regions.jl#L41

In the case of a NoFlux condition in the y direction on "u", or "w", it appears that a no-flux condition is applied on one side of the field, while a different condition is applied on the other side. Why is that? See here:

https://github.com/climate-machine/Oceananigans.jl/blob/ac7dc2a917f990f1c709fee597309b8edac01cd7/src/halo_regions.jl#L25
for example. Note that this condition is equivalent to no-slip when the halo size is 1, but becomes some other condition (that does not correspond to any physical condition I know of) when the halo size is > 1. The no-slip condition would require filling halos with mirrored and signed-reversed interior values.
See also #307.

I think this line:
https://github.com/climate-machine/Oceananigans.jl/blob/ac7dc2a917f990f1c709fee597309b8edac01cd7/src/halo_regions.jl#L24
is also incorrect for halo sizes > 1. All of the halo points should be copies of the single boundary-adjacent field value, rather than a copy of a chunk of interior values of size H.


Multiple Values with Select Component

feature request

I believe, this would be a feature request. I am currently trying to use a  <select>  component. I am using a .map function to populate  <option>  values. I am using another reducer for props to populate the option values. I am able to pull my values into the dropdown just fine. However, I am using  and name to populate the values. I am able to populate the JSON object with the name from the  list. However, I would like to pass the object back to the database with the ID that ties to the Name from . I currently only able to pass the name.
I thought maybe I would be able to use  to accomplish what I am trying to do with "names", and "values". However, I am still unable to pass two values from the dropdown <select> <option>. In order to get the values in the object I am doing something like the following:

value={${firstname.Name} + ${firstname.Id}}

example of code:
<Field name={`${newName}.Name`} type="text" component="select">
<option />
{this.props.dropdownValues.map(firstname => (
<option
key={firstname.Name + firstname.Id}
value={`${firstname.Name} + ${firstname.Id}`}
>
{firstname.Name}
</option>
  )
 )
}
</Field>

The output of this is the following:

{
Firstname: "name + 1"
}

I would like to be able to pass the following to the values object:

{
Firstname: "name"
Id:  1
}

I might be missing an option in the documentation, but I have looked at multiple options in the  documentation, and multiple stackoverflow. I am unable to find a solution for what I am attempting to accomplish. This would be useful in order to provide the backend developer with the database Id.
If there is an option for this type of output already. I believe, there should be a section for
<select> in the documentation that points passing multiple values from <option>  in the documentation.
Thanks

I still think this would be a nice enhancement. However, I was able to get around this using react-widgets.

import DropdownList from 'react-widgets/lib/DropdownList';

import 'react-widgets/dist/css/react-widgets.css';

const renderDropdownList = ({ input, data, valueField, textField }) => (     <DropdownList       {...input}       data={data}       valueField={valueField}       textField={textField}       onChange={input.onChange}     /> );
http://jquense.github.io/react-widgets/docs/#/?_k=sro1s2


Very large orgs are restricted by API response limits

Xero have implemented size limits (kb) on their invoices endpoint. A user with a significant amount of invoices hit this limit and wasn't able to bulk void.
Maybe I should allow people to enter a custom date/time range for retrieval instead of month by month.
I'm very very busy so probably will get to this in the next month or so.

Resolved during version 4 release - range picker is now available.


PyPI package

Is there a reason to not upload a package to PyPI? The current pip installation method involves cloning the git repository, which takes non-negligible time.

We now have a very clean installation mechanism through conda, and everything is documented. So I'm closing this.


[Suggestion] Remake create.md by adding params, etc.

Add relevant details to user guide pages.

Close via #672


Define evaluation metrics

Add in some basic metrics to track rule/entity evaluations to the engine. Note that the metrics for alerts and remediations are not wired in yet since they depend on some changes which will be added in the next PR.
I have taken a different approach to handling noop metrics than is used in the codebase so far: instead of defining an interface and separate real vs noop implementations, I have decided to make use of the noop metrics handling built into otel: there is now a metrics meter factory with alternative implementation for noop vs "real" metrics. Any part of the code which needs to create metrics can use the factory without caring whether the metrics are actually exported or not.
Relates to: #3556
Summary
Provide a brief overview of the changes and the issue being addressed.
Explain the rationale and any background necessary for understanding the changes.
List dependencies required by this change, if any.
Fixes #(related issue)
Change Type
Mark the type of change your PR introduces:

[ ] Bug fix (resolves an issue without affecting existing features)
[x] Feature (adds new functionality without breaking changes)
[ ] Breaking change (may impact existing functionalities or require documentation updates)
[ ] Documentation (updates or additions to documentation)
[ ] Refactoring or test improvements (no bug fixes or new functionality)

Testing
Outline how the changes were tested, including steps to reproduce and any relevant configurations.
Attach screenshots if helpful.
Review Checklist:

[x] Reviewed my own code for quality and clarity.
[x] Added comments to complex or tricky code sections.
[ ] Updated any affected documentation.
[x] Included tests that validate the fix or feature.
[ ] Checked that related changes are merged.

coverage: 52.777% (+0.01%) from 52.766%
when pulling 1209d3087b0a2d4fc2bf590f970e553d6462bc6e on eval-metrics
into 2c0c6b259f295f5ba48e04565ea651f13d4c20ce on main.

coverage: 52.772% (+0.006%) from 52.766%
when pulling 1209d3087b0a2d4fc2bf590f970e553d6462bc6e on eval-metrics
into 2c0c6b259f295f5ba48e04565ea651f13d4c20ce on main.

coverage: 52.768% (-0.007%) from 52.775%
when pulling 03c021a40409cf534dd306d0297227ff4837bf73 on eval-metrics
into 0287d08c807f4123a7988046fc8531410686138d on main.


monorepo migration

this will allow me to build @nextkit/client easily

https://github.com/preconstruct/preconstruct/issues/443


fix text selection and cursor position in lines with rtl text

assume a line contain both english text and persian (rtl) text, like this
Sample Text متن نمونه sample text

atom always look at first rect size of Text Element and calculate position, this cause cursor to  place in wrong position
in our example Range.getClientRects() return three rects, look at this pen
http://codepen.io/farnabaz/pen/rWxrGW
before:

after:

Hi @farnabaz, can you please add some specs for this?
/cc @as-cii

atom has problem with selection and cursor position in lines contain rtl-language characters, as you see in first picture when a line has persian characters, cursor act strange after this chars and do not point right place,  everything works fine before persian chars  but after them cursor position is wrong and selection does not highlight
you can test this bug with this text, copy this text and try to select word World
Hello سلام دنیا World

this is because atom try to locate cursor column using Range.getClientRects()  in LinesYardstick class.
in  src/lines-yardstick.coffee: 129 text size calculates by position of first rect of its range, this works fine with ltr texts, because they has only one rect in their ranges, but a line with both right-to-left and left-to-right chars has more than one rect, I update this pen so you can see what i'm talking about rect
http://codepen.io/farnabaz/pen/rWxrGW

Sorry, what I meant was adding a test to make sure that this behavior doesn't regress.  They go in the specs folder and you can find more information about them at http://flight-manual.atom.io/hacking-atom/sections/writing-specs/.

my bad, sorry for misunderstanding
I add a test in lines-yardstick-spec

Thanks!  Can I also ask that you use two-space indentation?

Yes, I use two-space indentation in all files

/cc @as-

Thanks for the pull request, @farnabaz! The reason why we used getClientRects is explained in https://github.com/atom/atom/commit/244f117d9504bbac439ce3cc44c475931c595493: when a range has zero-width, getBoundingClientRect will return a wrong result, and that's why we decided to use getClientRects instead, falling back to getBoundingClientRect only in case the client rect list is empty.
I wonder if we might still be able to use getClientRects by inverting the logic to something like the following:
this.rangeForMeasurement.setStart(textNode, startIndex)
this.rangeForMeasurement.setEnd(textNode, endIndex)
const boundingClientRect = this.rangeForMeasurement.getBoundingClientRect()
if (boundingClientRect.width > 0) {
  return boundingClientRect
} else {
  return this.rangeForMeasurement.getClientRects()[0]
}

@nathansobo: thoughts? 💭

Do we know if getBoundingClientRect() still returns incorrect results, or has that been fixed since April?

Great, thanks. https://goo.gl/JBljdR

As Nathan mentioned in that commit, sometimes getClientRects() return empty list, I think it may better to change your solution to something like this to avoid empty list and reach our goal
this.rangeForMeasurement.setStart(textNode, startIndex)
this.rangeForMeasurement.setEnd(textNode, endIndex)
const clientRects = this.rangeForMeasurement.getClientRects()
if (clientRects.length == 1) {
  return clientRects[0]
} else {
  return this.rangeForMeasurement.getBoundingClientRect()
}

@as-cii

@ungb this looks like it could be helpful. Could you test it when you get a chance?

@nathansobo I was able to test this and see that the behavior is better than before for the given scenario, I also went through and spot check a few cases and didn't see any issues with find-and-replace or the overlay for snippets
All are tested with atom built from this PR:
After change(before is describe above):

NOTE: the one thing I noticed is when I want to move between the persian (rtl) text, I have to click left a lot to go to the english text, and once to go right of it, but can never go in the middle
Notice the row and column count as I click right and left and where the cursor is.

Overlay(looks good):

Find and Replace(looks good):

I'm not sure why the build is failing on x64 bit windows, but looks related to the new test you added.
https://ci.appveyor.com/project/Atom/atom/build/4490/job/8xa7kuw9g618p8my#L484.  It passed on x86 and on travisci.

@nathansobo @as-cii I was able to also test the folding behavior.
I didn't see any issues with it

Made some adjustments to the tests and merged master. Waiting on a green build.

@ungb it seems monospace font size is different on Windows x64, this picture shows same line in Mac and Windows x64 (both lines has font-size:14px), font size on Windows is a little smaller than other

@farnabaz Have you tested the same lines in Chrome on Mac and Windows x64 to ensure that it isn't a difference in the Chrome font rendering on the two platforms?

@farnabaz Thanks for taking this on! It will be great to handle RTL languages a bit better, though it seems like we still have a bit of room for improvement based on @ungb's testing. Still, this is way better than it was. ✨

@lee-dohm On mac Chrome act exactly like Atom, but on Windows x64 line is different
white line shows same line in Chrome 55 on Windows x64

Here is my test page: http://codepen.io/farnabaz/pen/dOQEGE

@nathansobo It's a bit difficult to fix cursor position and selection in RTL languages, one of things that need to be fixed is screenPositionForPixelPosition function in LinesYardstick. I'll try to explain how and why this function return wrong value in RTL languages.
Should I need to create an Issue for this? or simply start to change on my fork and then explain it on Pull Request?

@farnabaz Explaining it in a pull request would be great. And yeah, the better you can help us understand what you're learning about this problem the better. Thanks so much for your interest in this!

in new beta version 1.14.0 you released a fix for this issue but the problem still exsists ! the selection result may is different but it is still incorrect
hope this issue will be fixed finally
tnx atom team

Still exists in version 1.15.0 stable

I'm keen to test any improvements for both Hebrew and Syriac — neither script is really functional in the released version at the moment, using either the keyboard or the mouse.

Still it exists in version 1.18.0 :(

also exists in version 1.44.0 😃


The billing adapater is an implementation detail

For the PAYG listing the billing-dapater is an implementation detail tha th customer doesn't need to know about. As such wording as submitted in #14 "The Azure marketplace PAYG offer is tied to a billing adapter AND Neuvector version." is providing information the customer doesn't really know what to do with. Form the customer perspective we provide 1 offer, that is NeuVector PAYG. The billing adapter is part of that offer. NeuVector is a product composed of a number of containers and the billing adapater, in the PAYG offer is part of NeuVector, we do not describe the ui controller, the scanners etc. as separate parts either.
If we feel the need to explain why a special update process is needed then let's explain it without picking appart the application layer.

@yarunachalam I don't have the ability to assign issues in this project, but I believe this is something you can cover in your next doc update... thanks!

it is resolved


Fix incomplete support for exponential operator

While looking through the es2016+ table here I noticed that babel is not marked green for exponentiation (**) operator - basic support. This has to do with the parenthesis that are mandatory for prefixed literals.
So ultimately it should be like this:
5 ** 6; // success
(-5) ** 6; // success
-(5 ** 6); // success
-5 ** 6; // fail
-(5) ** 6; // fail
(-5 ** 6); // fail

But currently it is like this:
5 ** 6; // success
(-5) ** 6; // fail
-(5 ** 6); // success
-5 ** 6; // fail
-(5) ** 6; // success
(-5 ** 6); // fail

I added some currently failng tests.

I found a solution to fix it. Luckily we already have a lot of information about parenthesis in extra.

👍

[Current coverage][cc-pull] is 97.00%

Merging [#75][cc-pull] into [master][cc-base-branch] will not change coverage

@@             master        #75   diff @@
==========================================
  Files            19         19
  Lines          2972       2972
  Methods           0          0
  Messages          0          0
  Branches          0          0
==========================================
  Hits           2883       2883
  Misses           89         89
  Partials          0          0

Powered by Codecov. Last updated by [0b62ece...ffae95b][cc-compare]
[cc-base-branch]: https://codecov.io/gh/babel/babylon/branch/master?src=pr
[cc-compare]: https://codecov.io/gh/babel/babylon/compare/0b62ecee21ab1df7de9ba70abd0eba732eaee4ea...ffae95bf63e1747e43dd1680f72fcd9d383357f2?src=pr
[cc-pull]: https://codecov.io/gh/babel/babylon/pull/75?src=pr


multiplication table

Description
I need to print to the user the multiplication table of a given no.
Domain
CPP
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

/assign

/assign


[KeyVault] updating autorest version and swagger

[ ] The purpose of this PR is explained in this or a referenced issue.
[ ] The PR does not update generated files.

These files are managed by the codegen framework at Azure/autorest.go.

[ ] Tests are included and/or updated for code changes.
[ ] Updates to CHANGELOG.md are included.
[ ] MIT license headers are included in each file.

/azp run go - azkeys

/azp run go - azkeys


blooprifle: report exit code in exception

To help debugging, the failure code is now part of the exception in case the bloop server failed to start.
I've worked on this during debugging #1843. Turns out, knowing the exit code was not really helpful in this case. However, I believe that it is never wrong to load the exception with it.

@Flowdalic, the CI should pass if you rebase your branch with the newest changes from main.
The same applies to #1845

@Flowdalic it seems the following tests are failing andwill need to be adjusted.
integration.test.test 3 tests failed:
  scala.cli.integration.BloopTests.invalid bloop options passed via cli cause bloop start failure
  scala.cli.integration.BloopTests.invalid bloop options passed via global bloop config json file cause bloop start failure
  scala.cli.integration.UpdateTests.updating dummy scala-cli using update command
Error: Process completed with exit code 1.

I suspect scala.cli.integration.UpdateTests.updating dummy scala-cli using update command is an intermittent connection failure, but the other 2 will need code adjustments.
BTW. let me know if you'll have trouble to find the time to fix this, in which case either me or someone else from the maintenance team will have a look.


fix: sign-out link

The sign-out link must of been changed recently.

The update will fix this error:
/node_modules/amazon-order-reports-api/lib/index.js:198
            throw new Error(message ?? `Assertion failed! No element matching selector: ${selector}`);
                  ^

Error: No sign-out link detected, failed to sign in?
    at AmazonOrderReportsApi._assert (/node_modules/amazon-order-reports-api/lib/index.js:198:19)
    at processTicksAndRejections (node:internal/process/task_queues:96:5)
    at async AmazonOrderReportsApi._handleLogin (/node_modules/amazon-order-reports-api/lib/index.js:331:9)
    at async AmazonOrderReportsApi._navigate (/node_modules/amazon-order-reports-api/lib/index.js:360:9)
    at async AmazonOrderReportsApi._getReport (/node_modules/amazon-order-reports-api/lib/index.js:265:9)
    at async AmazonOrderReportsApi.getItems (/node_modules/amazon-order-reports-api/lib/index.js:124:9)
...

Seemed to have been a temporary issue.

Turns out I was triggering the captcha from having too many report requests


⚠ BeyondHD has degraded performance

In d2abae9, BeyondHD ($BHD) experienced degraded performance:

HTTP code: 200
Response time: 4275 ms

Resolved: BeyondHD performance has improved in ec61276 after 6 minutes.


[Feature Request] Add support for alerts

GitHub has supported alerts since the end of 2023; see the community discussion here.
For example:

[!NOTE]
This is a note.

Problem
Running mdformat (with mdformat-gfm installed) changes the correct syntax:
> [!NOTE]
> This is a note.

to
> \[!NOTE\] This is a note.

Which does not correctly render.
Request
Can this feature be supported? If not, is there some way to force the linter to ignore these cases?

I have a package to support GitHub alerts, 'mdformat-gfm-alerts'!

I can't believe I didn't find this earlier. This is amazing, and it works perfectly!


allow multiple hostnames in certificate

That would not only help in not getting rate limited, but also when deploying non-http services that listen on different hosts. They could share a certificate that way.

@tnozicka I could pick this up and work on it. Would you be willing to accept a PR for this?


Review draft of federal sales data blog draft

Federal Sales blog draft

@AlexandracmONRR provided feedback as well. Happy to discuss. Great start!

After I incorporate edits, send tracked changes and clean version to erin

@AlexandracmONRR closing issue and then you can make another issue for cleanup.


Any way to use it with parametrized fixtures?

I use parametrized fixtures to run the same test using different sets of data. Each is a different test case in test rail. Anyway to do use this to do that?

Maybe
@pytest.mark.parametrize('data', [
    pytest.param(1, marks=pytestrail.case('C10')),
    pytest.param(0, marks=pytestrail.case('C11'))
])
def test_three(data):
    assert data

Closing as no response from OP


refact: proposal for a new design arround component tree

This is a refactoring proposal to make the notion of component tree simpler. This refactoring removes the 3 distinct classes that inherit from each other to represent the 3 trees. It removes cascading inheritance and method overloading.
There is a single class ComponentTree class that can combine fragments. One fragment for BMC, one for CMC, one for SMC. A fragment can be frozen to lock its contents. Fragments are an instance of the same ComponentTreeFragment class.
The application contains 2 tree construction methods. build_base_component_tree which builds the tree the application needs upon loading with a fragment for BMCs and CMCs. build_session_component_tree builds a tree dedicated to the user session with 3 fragments, one fragment for SMCs, one for CMCs and a frozen fragment for BMCs. The fragments are copied to separate them from the base representation.
I added in the tests, 2 distinct contexts, one to represent a ui manager when streamsync application is loading and one to represent a ui manager in an event handler.

This is a step in the right direction. However, I'd like to change the nomenclature "smc" as I don't think it's reflective of what we're doing.
Manage should be interpreted in the sense of enabling the user to create, delete, etc. Not from an internal perspective. Two categories emerge:

Builder-managed
Code-managed

The other classification emerges from the specificity of the components:

Shared/base
Session-specific

Therefore, the combinations are:

Builder-managed base components
Code-managed base components
Builder-managed session-specific components
Code-managed session-specific components

There are currently no plans to have builder-managed session-specific components. For simplicity, we can adopt the following:

Builder-managed base components: bmc
Code-managed base components: cmc
Code-managed session-specific components: session-cmc

@mmikita95 fyi, feel free to add to the conversation

This is a step in the right direction. However, I'd like to change the nomenclature "smc" as I don't think it's reflective of what we're doing.

That's sound interesting, I will take a look at the change involved.

@ramedina86 great breakdown, thank you!
One thing I would probably adjust is to shift focus on "session CMCs" as "main CMCs" (i.e., use names like "pre-session CMCs vs session CMCs or in-session CMCs"), because in my opinion, while "pre-session CMCs" are expected to be used more widely, technically the session context is the last point that collects all components into a "render package", and one can view "pre-session CMCs" as "copies made for the session". Therefore, to better reflect this technicality, I'd suggest this naming convention. Sorry if this is turning out to be too confusing 😅

Excellent point @mmikita95 , glad I tagged you.  The pre is slightly confusing to me. How about:

bmc
initial_cmc
session_cmc

in line with state (initial_state, session_state).


scrappy appearance

Hi,
I need some help.
I tried your data table component, but appearing in my environment is scrappy. A used your basic example. Can you tell me, what is the problem?
I use: "angular": "~1.5.2", "angular-material": "~1.0.7",

Second problem is that you replace default angular start and end symbols in your code, but I’m using ‘{$’ and ‘$}’ symbols because of python-django. Can you prepare your code for using interpolateProvider?
Thanks for your help.
Tom

Hi @tonipiano , can you please double check if this is still happening with the newest version?
Thanks!


apt upgrade

Hi there,
is it safe to update raspiblitz with "apt update + apt upgrade" ?
Or should I always wait for official raspiblitz releases ?
I want to have packages with latest security updates.
Thank you

we can't predict if new versions of everything will break things.
If there is anything specific you want to update use
sudo apt install PACKAGE_NAME
If the system breaks the tested release (latest SDcard image) is there as a fallback.

Since the Tor configuration has been refactored apt upgrade is less prone to break and it is the besst practice running linux to keep paackages up to date.
Usually it is not necessary to run this between releases, but can do especially if there is some known vulnerability.


Add example to path param

I have been adding examples to some of my parameters, but up to this point they have not been path params. I tried to do it with the following but it doesn't show up.
route_param :id, desc: "User ID", documentation: { example: User.first.id }  do

I can't even get a description to show up for a route_param. +1 Must fix.

Yeah these aren't first class params in Grape. It needs work.

:+1:

I'm facing this issue as well. I've written two tests that related to documentation & examples for route params.
https://github.com/rjaus/grape-swagger/commit/dd98a217b166ca3799eb4fff8fae335a67202a3a
I'm happy to try and fix it too, if someone will point me in the right direction. @dblock where does this need to be fixed? What needs to be changed?

I'm happy to try and fix it too, if someone will point me in the right direction. @dblock where does this need to be fixed? What needs to be changed?

in this project, but I don't know specifically where

Can someone please verify whether https://github.com/ruby-grape/grape-swagger/pull/943 fixes this?


(GH-2207) Log when default inventory file cannot be loaded

This change updates how Bolt logs which inventory file it loaded.
Previously, if Bolt loaded the default inventory file, it would log that
it loaded the file, even if the file does not exist. Now, if the default
inventory file does not exist, Bolt will instead notify the user that it
tried to load the file, but that it does not exist.
!bug

Log when default inventory file cannot be loaded
(#2207)
Bolt now logs that it tried but failed to load the default inventory
file when the default inventory file does not exist. Previously, Bolt
would log that it loaded the default inventory file, even when it was
unable to do so.

CLA signed by all contributors.


[Relay][Frontend][Onnx] Fix GEMM converter when C is not a parameter.

The change made to the Onnx gemm converter in #7489 assumes that the value of C will always be a parameter. When it is a constant or an expression, the special case handling will fail. This very tiny PR makes the handling of C more general and adds test cases that would otherwise fail.

@mbrookhart @xutianming @masahi can one of you take a look at this PR?


"Move between apps" feature is a set back

For that it has cancelled the navigation menu which is very very important for users..

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

ID: 46216c83-4fb2-1be0-b3ad-a909be419e32
Version Independent ID: c287a530-09df-f62c-c30e-a8ec2e051973
Content: Basic navigation in a model-driven app - Power Apps
Content Source: powerapps-docs/user/navigation.md
Service: powerapps
GitHub Login: @mduelae
Microsoft Alias: mkaur

@YahyaManasrah the menu is still there. When you move between apps in the current release you click the drop-down menu and then choose an app. In Preview, you select app name and then you can choose the app you want to use.
https://docs.microsoft.com/en-us/powerapps/user/navigation#move-between-app
If you product feedback, visit https://ideas.powerapps.com.
Thanks,
Mints


how to use this program without docker??

You can begin to read the readme.
https://github.com/osscameroon/broken_link_checker/blob/main/README.md#running-the-project-locally


[ZEPPELIN-2368]. Option to run all paragraphs sequentially

What is this PR for?
This PR change the "run all paragraphs" to be sequentially running.
What type of PR is it?
[Improvement]
Todos

[ ] - Task

What is the Jira issue?

https://issues.apache.org/jira/browse/ZEPPELIN-2368

How should this be tested?

First time? Setup Travis CI as described on https://zeppelin.apache.org/contribution/contributions.html#continuous-integration
Strongly recommended: add automated unit tests for any new or changed behavior
Outline any manual steps to test the PR here.

Screenshots (if appropriate)

Questions:

Does the licenses files need update? No
Is there breaking changes for older versions? No
Does this needs documentation? No

Thanks @felixcheung  Will add tests for this. Here's some discussion about this issue.
http://apache-zeppelin-users-incubating-mailing-list.75479.x6.nabble.com/Implementing-run-all-paragraphs-sequentially-td6236.html

What happens now if after doing run all , a later paragraph that isn't submitted yet is run explicitly? Will it run twice?

@namanmishra91 Do you mean after after clicking run all, user click one paragraph of this note to run it ?

@zjffdu Yes, that's right.

@namanmishra91 In that case, the paragraph will be execute twice. One improvement thing I think we can do it in frontend is to freeze the frontend to disable user run paragraphes when user click the run all button. Thoughts ?

Will merge it if no more comments.

Apologies for the late comment but at least you agree now that this feature will require frontend changes also. This "UI is frozen" information will also need to be passed to the frontend on notebook load.

@namanmishra91  Right, this could be done in a separate ticket.  Very appreciated if you could help on this, I am not a frontend expert.


⚠ Bofrost.si has degraded performance

In 9276c1b, Bofrost.si (https://www.bofrost.si) experienced degraded performance:

HTTP code: 200
Response time: 2059 ms

Resolved: Bofrost.si performance has improved in fe8ee4b.