mvn gluonfx:run with -f parameter fails

Using -f parameter with  mvn gluonfx:run results in a failure where its unable to find JavaFX plugin:
[INFO] --- gluonfx-maven-plugin:1.0.4:run (default-cli) @ App ---
[ERROR] Failed to execute goal com.gluonhq:gluonfx-maven-plugin:1.0.4:run (default-cli) on project App: Error reading pom: No JavaFX plugin found -> [Help 1]

Duplicated of #351

A very important issue since I filed it twice 😂


Broaden the catched exception for skipPattern calculation

Relates to  #1679

Because BeanCreationException is too generic, it is dangerous to catch it and continue. I think it would be better always trace the exception (DEBUG could be an adequate level), and only proceed if the exception is a wrapper for BeanCurrentlyInCreationException or directly a BeanCurrentlyInCreationException., throwing exception in other case.


Fix freeform object schema rendering for OpenAPI 2.0

Fix rendering of type: 'object' schema fields missing properties specification. (They were basically ignored.)

Basically solves the same problem as #52 but for OpenAPI 2.0.

Hey @tuffnatty, thank you for the patch. Can you please also add a test to cover this? You can grab one from #52 but add to openapi2 test suite.

@ikalnytskyi Sure the adding test is not a problem, but having looked at the output I started wondering if it would be better to always output a fieldname (object) - description entry for an object-typed node, and then output its properties as usual (if any). Otherwise the description is lost when properties are defined. What do you think?

I re-based and added a test (https://github.com/sphinx-contrib/openapi/pull/77). Authorship is preserved. Thanks!


Missing reference on the Android project

Xamarin.Forms - 3.6
Error on every version above 2.0.4. Only on the Android project.

Hi IeuanWalker
Same problem here.  Did you manage to fix it?

@snice1981 nope and it is also causing us issues with VS2019 for some reason.
We are planning to update to XF 4.1 and take advantage of some of the new features which means we can remove this NuGet.


ExpansionPanel in content projection in modal closes after first open

Reproduction
Use StackBlitz to reproduce your issue: https://stackblitz.com/edit/angular-pjv1un
Steps to reproduce:

Open modal
Close modal
Open modal

Expected Behavior
Expansion panel is opened
Actual Behavior
Expansion panel is closed(it has correct class, but style visibility: hidden, height: 0)
Environment

Angular: ~8.2.0
CDK/Material: ^8.2.0
Browser(s): Chrome
Operating System (e.g. Windows, macOS, Ubuntu): Ubuntu, Windows 10

Same Problem here.


checkcommits: update vendoring

Update vendoring, which includes:

move to pinned version of golang/dep
set packages to track 'master' branches
update, to pick up BurntSushi/toml license file changes

Fixes: #202
Signed-off-by: Graham Whaley<EMAIL_ADDRESS>

@devimc @sboeuf I've put you as reviewers as I'd like your input on if this was the right approach to force the update by pinning a commit version in the Gopkg.toml etc.

I would force the constraint to master instead. This does not mean that it will move automatically, but at least it show our wish to follow the most recent updates.

And I think we should revendor the entire repo properly because it has been done with a version different from the one I mentioned in the email on the mailing list, causing some warning when running dep.

Right. That was my other option (and I see that is what the cc/runtime repo does). I guess it is a balance of risk/reward eh?
Re: version. I must have missed that - are we vendoring with a fixed version of dep? If so, that is important to know indeed - and we need to get that in the docs that I've added all the Issues for.
So, please put that data either here or on the 'document vendoring process' Issues :-)

@grahamwhaley can you give me the link to this document vendoring process issue please ? I will add the version of dep (which is github.com/golang/dep commit 24c37f5f336ec108d44e88a6f36125ab93c79951).

Sure. The one for this repo is here:
https://github.com/clearcontainers/tests/issues/200
but you will also find similar Issues for all the repos, like:
https://github.com/clearcontainers/runtime/issues/332
coz, it's sort of important (as we just found ;-) )

lgtm
I guess I should update localCI too

@grahamwhaley let me know if you want to do the new vendoring for this repo. I can do it ;)

@devimc I already opened and assigned you an Issue for localCI update ;-) https://github.com/clearcontainers/tests/issues/203
@sboeuf I'll sort out the vendoring. We need multiple people knowing how to do this.

@sboeuf OK, I took the plunge and did a whole update of the tool, the .toml config and then the BurntSushi/toml license file update (by implication).
I didn't split the patch down to those three actions, as that would have just been plain hard work beating on dep.
Let me know if the updates work for you - and if you still see incompatibility issues with the version of dep you are using (should now be the same), then provide details of the commands run and the errors seen so I can reproduce and track down.

@grahamwhaley thanks

@grahamwhaley you did not modify the checkcommits README and pullapprove is waiting for the review of the documentation team, I think you should add "cmd/checkcommits/vendor/*" in the pullapprove file
https://github.com/clearcontainers/tests/blob/master/.pullapprove.yml#L52

Ah, yes, I did wonder if that would happen @devimc :-) I wondered if we had a /vendor/ in the yaml or not - I guess not ;-) I'll contemplate if I can add a generic single regex, and if not, then will add individual repos.
thx

LGTM

I just noticed the commit subject is wrong - should be checkmetrics not checkcommits. I may fix that ...

@grahamwhaley should be "vendor:"

@grahamwhaley ohh ya, yes is checkmetrics

@grahamwhaley please update the commit message

why checkmetrics ??

@sboeuf - https://github.com/clearcontainers/tests/tree/master/cmd.

@sboeuf He modified cmd/checkmetrics

Oh my bad, I thought that was the vendoring for the entire repo, I didn't realize it was for the tool only.

@sboeuf , what "should be "vendor:"" - subject on commit for this PR, or for #210 or? Sorry, not clear to me...

@grahamwhaley no nothing, I thought it was the vendoring for clearcontainers/tests, but it is the vendoring for checkmetrics tool. Maybe you can name it "checkmetrics: vendor: update vendoring", up to you !

Updated commit subject and PR subject to checkmetrics (coz I had a brainfart obviously before :-) )

and I hit the Travis she issue - I'll re-push ...

@iphutch could you approve this PR?

re-based and pushed - let's see if that fixes Travis

lgtm


Don't double TLS connections between LXD and agent

For some reason our TLS over vsock logic was incredibly broken and we were effectively doing TLS over TLS.
Not only is that a big waste of CPU time, it's also causing some issues when we need to extract client certificates as is the case in #10610.
Unfortunately I couldn't find a way to gracefully fix this, so as soon as a system will receive this fix, the guests will need a reboot to receive a new agent with the corrected logic.

@tomponline good to go

@monstermunchkin I've already rebased your branch on this one, so you should be unblocked.

This should help speed VM operations up too :)


GetBlockVerbose is broken for bitcoin core

GetBlockVerbose takes an argument verboseTx bool, and make the rpc call
{"jsonrpc":"1.0","method":"getblock","params":["0000000000000000023ce2ceba757ccca3d3ee4938608f37f1e73cd4ec3d6d59",true,verboseTx],"id":1}

However bitcoin core's getblock only takes one argument, so this gives an error

Fixed by #94.


New documentation on the upcoming GitBook

Next week, we'll introduced a new version of GitBook (https://beta.gitbook.com/) 🎉 . This version brings a better reading experience (mobile and desktop), awesome search (powered by Algolia) and some cool new features.
I've run an initial import of the Redux docs in the new GitBook, you can checkout the output at:
https://redux.gitbook.io/docs/
It'd be awesome to get your feedback on this new version, and see if you're interested in switching the Redux documentation to this!

This version also includes some advanced features to analyse the quality of your documentation: a search dashboard and insights about the quality of your docs:

Neat!
Since the CLI tool goes away, do the docs get built on your servers instead? We haven't set it up, but we've always wanted to have automatic docs builds in Travis. If we can make that happen, great!

@timdorr Yes, you'll no longer have to run the CLI, the new GitBook will keep the website synced with your GitHub repository!
I can send you an invite (I'll need your email or ping me at samy@gitbook.com).
The next steps if you want to switch are:

Merge this PR
Use the invite to join the "Redux" organization on beta.gitbook.com
Setup the GitBook project to use this repository as a GitHub source
Add the redux.js.org domain to the GitBook project and update the redux.js.org DNS

It's<EMAIL_ADDRESS>Thanks!

Submitted a request to update the DNS: https://github.com/js-org/dns.js.org/pull/1774
Looks good to me. Let's do this!

Welcome back https://redux.js.org
(and sorry for the delay)

Not sure I realized we'd actually pulled the trigger and switched.  At first glance, I see two things that are broken:

All of our pages have had URLs ending with .html so far.  , like http://redux.js.org/docs/FAQ.html .  The new site doesn't have .html at all, so every external Redux docs link is now presumably a 404.  That's not good, and we need some kind of redirect setup to handle this.
I had hand-written short anchor tags and TOCs for the FAQ pages.  Those no longer appear to work.  I see that the new site has an auto-generated TOC on the right, but I wanted those short anchors to make it easier to write the URLs.  (For example, handwritten https://redux.js.org/faq/react-redux#react-multiple-components vs auto-generated https://redux.js.org/faq/react-redux#should-i-only-connect-my-top-component-or-can-i-connect-multiple-components-in-my-tree ).

I'm not happy with these differences, and we need to figure out a way to resolve them ASAP.

@markerikson

All of our pages have had URLs ending with .html so far.

GitBook automatically setups redirects for .html to the new urls, for example https://redux.js.org/faq/immutable-data.html redirects to https://redux.js.org/faq/immutable-data.
The issue come from the docs/ prefix in previous url. I've missed it in my previous PR. I've opened a PR to fix it: #2843
I'm really sorry about this :(

I had hand-written short anchor tags and TOCs for the FAQ pages.

We are going to release a fix for this, no action is needed on your side. I'll keep you updated ASAP.


Animation type request: by path length

I imagine an option that delays the drawing of small path at the end of the animation. In this case, the details of the drawing are added in the end only. For example, in the case of a face, it can be nice to simulate a drawer’s process and trace eyes details in the end, after the main paths of the silhouette.
I can imagine that the oneByOne type simply respect the initial order of the paths in the SVG, but I think designers shouldn’t have to care about the animation while they design a shape.
Maybe this can be done simply by ordering paths drawing by length. What do you thin?

Hi,
I completely understand that designer shouldn't care about the order of their layers. But using the layer order was the most natural default behavior to implement.
In your case you want some control over the drawing scenario, this is why I recommend you check the type scenario.
It give you the possibility to define the start and duration of each path element. In this case, the layer order doesn't matter. But it might take you some time to script your SVG.
Check the readme.md file and come back to me if you have any questions.

Thank you, I am aware about the type scenario I actually think about an automated order able to treat images with a lot of paths.

Thanks for making this fantastic js file but I can't get it to work via external svg file. I have scanned this thread and looked at the methods to construct it with the object tag and went through the patch test "showMySVGInsideThisElement" (inserting svg object into the span) but it comes up with this error:
'Uncaught ReferenceError: Pathformer is not defined' which seems to stem from the vivus.js file (line 426: Vivus.init.
Can you show a working example in JsFiddle/JsBin?

Sorry for the late reply but yes, an example on JSfiddle would be amazing. But create a new issue for it :)

Le 17 Mar 2015 à 00:50, SLQ Web Team<EMAIL_ADDRESS>a écrit :
Thanks for making this fantastic js file but I can't get it to work via external svg file. I have scanned this thread and looked at the methods to construct it with the object tag and went through the patch test "showMySVGInsideThisElement" (inserting svg object into the span) but it comes up with this error:
'Uncaught ReferenceError: Pathformer is not defined' which seems to stem from the vivus.js file (line 426: Vivus.init.
Can you show a working example in JsFiddle/JsBin?
—
Reply to this email directly or view it on GitHub.

At the moment I don't plan to implement this kind of features. But I found a way to hook this feature.
new Vivus('yourSvgId', {
    type: 'oneByOne',
    onReady: function (obj) {
      var i,
        counter = 0,
        newMap = obj.map.sort(function(a,b) {
          return b.duration - a.duration
        });

      for (i = 0; i < newMap.length; i++) {
        newMap[i].startAt = counter;
        counter += newMap[i].duration;
      }

      obj.map = newMap;
    }
  });

This trick wait the SVG to be ready to reformat the map in the order of path length. All other parameters are the same. Feel free to put this callback into an independent function to use it many times. You will need the v0.2.0 to make it work.


Fix the testnet ION denomination icon

The testnet ION denomination icon was referring to tmION rather than tION.

[ ]   General Comment
 I am ok, lets just discuss why tmION? as well as why tION?
t: for test network
m: ?mode?
Visually, I somehow like tmION more. tION would probably be technically
 :shipit:
 @cevap good pickup, I have now added unit_tion.xcf.
Description of denominations:
tION: Testnet
mION: Milli-ION (1 thousandth of an ION)
uION: MicroION (1 millionth of an ION)

This pull request is now managed with CodeReviewHub. Refresh.

I am ok, lets just discuss why tmION? as well as why tION?

t: for test network
m: ?mode?

Visually, I somehow like tmION more. tION would probably be technically

:shipit:

@cevap good pickup, I have now added unit_tion.xcf.
Description of denominations:
tION: Testnet
mION: Milli-ION (1 thousandth of an ION)
uION: MicroION (1 millionth of an ION)

mtION stays for - Milli-Testnet-ION.
We should not remove it at all. I will close and comment this pull as it actually makes no sense, those visual icons should just tell as addition that you are on testnet, not more.
I would not mind to get new icons, but we will keep t for testnet, we might add r for regtest (is obsolete unless we readd miner part), and unittest uses main's settings, but for visual warning, we could add it to ensure that user does see additionaly that unittest is launched.

@cevap apologies, I might not have made a clear description to what I am doing here. I am fixing the tION icon, not removing anything.
If you open the Qt wallet on testnet, look at the bottom right and select tION you will see that the denomination icon is incorrectly showing tmION rather than tION.
Demo:

Because this is a bugfix, could we please reopen and merge?

@cevap apologies, I might not have made a clear description to what I am doing here. I am fixing the tION icon, not removing anything.
If you open the Qt wallet on testnet, look at the bottom right and select tION you will see that the denomination icon is incorrectly showing tmION rather than tION.
Demo:

Because this is a bugfix, could we please reopen and merge?


[terra-dropdown-button] Fixed Focus issue for mouse and keyboard interactions.

Summary
Fixes #2632
Additional Information

Clicking a DropdownButton/SplitButton should open the menu but not auto-highlight the first option.

we are using initialFocus prop of Focus-Trap to prevent focus on first option when dropdown opened via mouse click.
<FocusTrap focusTrapOptions={{ returnFocusOnDeactivate: true, initialFocus: openedViaKeyboard ? '' : buttonRef, clickOutsideDeactivates: true }}>

initialFocus prop will be set to buttonRef which will be holding ref of dropdownbutton when opened via mouse click. openedViaKeyboard boolean is used to differentiate between keyboard event  and mouse event.
initialFocus will be set to empty to set focus on first option when dropdown is opened via keyboard ( openedViaKeyboard is true ). Focus trap will set to first accessible element when initialFocus prop is set to '' empty.
This introduced new issue by making dropdown list inaccessible in VO/JAWS mode when opened via mouse click. this was due to holding focus back on dropdown button on open ( when opened via mouse click ) instead of moving focus to first list item.
Since dropdown list is rendered in portal dom order will be broken as it renders outside of root div this makes focus to shift from button to other elements in page instead of list items on navigation up and down key press.
To fix this issue I used keydown event handler of DropdownButton and SplitButton. Added check for up and down arrow key so that we can set focus to first list item on down arrow key and on last list item on up arrow key press.
if (event.keyCode === KeyCode.KEY_DOWN && this.state.isOpen && !this.state.openedViaKeyboard) {
      // puts focus on first list element on down arrow key press when dropdown is opened by mouse click
      const firstOption = this.dropdownList.querySelector('[data-terra-dropdown-first-list-item = "true"]');
      firstOption.focus();
} else if (event.keyCode === KeyCode.KEY_UP && this.state.isOpen && !this.state.openedViaKeyboard) {
      // puts focus on last list element on up arrow key press when dropdown is opened by mouse click
      const lastOption = this.dropdownList.querySelector('[data-terra-dropdown-last-list-item = "true"]');
      lastOption.focus();
}

openedViaKeyboard boolean is used to ensure that focus sets to list items on navigation key press only when dropdown is opened via mouse click.
setting focus on list option in keydown event resolved VO and JAWS issue of accessing list items through key board.

Focus issue of dropdown (focus not going into dropdown on key down press) has been addressed with commit : e6707e13790226d459fbd937a35b0d1744a4eff6
Made changes for keyboard navigation when drop-down is opened by mouse click with reference to : https://www.w3.org/TR/wai-aria-practices-1.1/examples/menubar/menubar-2/menubar-2.html

Would you mind adding the specific scenarios these code changes are intended to alter to the PR summary? Such as the expected outcome from clicking the dropdown or opening it with the keyboard and focus interaction states. Occasionally issues have lengthly threads on the originally logged issue to track down the expected changes. A brief recap is really helpful to provide with the PR.

Would you mind adding the specific scenarios these code changes are intended to alter to the PR summary? Such as the expected outcome from clicking the dropdown or opening it with the keyboard and focus interaction states. Occasionally issues have lengthly threads on the original logged issue to track down the expected changes. A brief recap is really helpful to provide with the PR.

Sure, I have updated the PR description with details of code changes that I have made let me know if I have missed something.

The deployment seem to be outdated, recent changes need to be re-deployed. I pulled down the branch and noticed that the focus does not go to caret button upon dismissing the dropdown on Safari.

The deployment seem to be outdated, recent changes need to be re-deployed. I pulled down the branch and noticed that the focus does not go to caret button upon dismissing the dropdown on Safari.

Fixed it same as primary button : here

Functional verification complete.  No issues identified.


[improve][io] Add ElasticsSearch BatchSource

Fixes #20890
Motivation

Large datasets can accumulate in ElasticSearch, either through intentional design or organic growth.  Migrating those datasets out of ElasticSearch can be difficult, as can simply re-indexing the data from within ElasticSearch in order to change index mappings or other characteristics.  Pulsar, with its highly scalable parallel function execution as well as strong schema support and tiered storage into low-cost Object storage (S3, GCS), can be a helpful companion when migrating or managing very large indexes in ElasticSearch.
Pulsar has a number of advantages as a long-term data store, including strong Schema support (for reduced record size compared to JSON), and easy off-loading to Bucket storage for cost efficient long term storage.  In combination with the ElasticSearchSink, an ElasticSearch BatchSource would allow a user to read a large dataset from ElasticSearch, perform complex processing in Pulsar, and write the results back to ElasticSearch or other systems.
Modifications
Added ElasticSearchBatchSource
Added SlicedSearchProvider abstract class to separate the sliced search implementations from the existing indexing code.
modified the OpenSearchRestHighLevelClient to use version 2.7.0.
modified the OpenSearchRestHighLevelClient to be compatible with OpenSearch 2.X.
deprecated the use of type when interacting with OpenSearch clusters.
Verifying this change

[ ] Make sure that the change passes the CI checks.

This change added tests and can be verified as follows:

Added integration tests for ElasticSearchSink and ElasticSearchBatchSource for OpenSearch 2.X
*Added integration tests for ElasticSearchBatchSource on ElasticSearch 7 and 8 and Opensearch 1 and 2.

Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ X] Dependencies (add or upgrade a dependency): Updates OpenSearch rest client from 1.2.4 to 2.7.0
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[ ] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ X ] doc-required
[ ] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/Q6Cyber/pulsar/pull/2

While working on the writeup, I realized I need to move the creation of the PIT / Scroll to happen during discover from the BatchSource. I still need work on:

[ ] Move PIT / Scroll creation to discover phase
[ ] Build key from configured fields in ES record
[ ] Schema Handling
[ ] Other identified issues / shortcomings


[BUG] - Line numbers do not line up with code

Describe the bug
Line numbers do not line up with code.
Screenshots

Desktop (please complete the following information):

OS: macOS 12.1
Xcode: 13.3

Additional context
Note that in Xcode the line numbers are smaller than the font size of the code. I'd imagine both the line number and the code both use the same line height to ensure that always line up.

A possible idea for a fix might be to increase the spacing between the line gutters

Ok, while this is absolutely not a fix, I'm messing with the CodeEditor struct in the CodeEditModules, and changing the font size of the LineGutter within scrollview.verticalRulerView seems to help reduce the visibility. However that seems to create it's own problems as well with numbers cutting each other off. My thought is that the LineGutter and the CodeEditor views need to be more connected in some way.
Something that might help with this is having Xcode style line highlighting, but I have no idea how to implement that.

Ok, while this is absolutely not a fix, I'm messing with the CodeEditor struct in the CodeEditModules, and changing the font size of the LineGutter within scrollview.verticalRulerView seems to help reduce the visibility. However that seems to create it's own problems as well with numbers cutting each other off. My thought is that the LineGutter and the CodeEditor views need to be more connected in some way.
Something that might help with this is having Xcode style line highlighting, but I have no idea how to implement that.

Or the CodeEditor telling the LineGutter how man lines one item is. Also the Editor font is bigger than that of the LineGutter

@austincondiff is this issue still there? See also https://github.com/CodeEditApp/CodeEdit/pull/409


Problem with texturepack support

@Uraxys
About 4 days ago I decided to create the first full(currently supported) texture pack based on Minecraft textures.
Everything worked perfectly until the first 100 or so textures were introduced, the first symptoms of problems were difficulty starting the game, a moment after showing the start-screen of Unity the game closed itself, sometimes it was related to the program hanging, sometimes not. The game started after about 3/4 of starting.
After introducing about 600 textures the game stopped launching completely regardless of the number of attempts. Deleting the texturepack from the "texturepack" folder solves the problem and the game launches normally.
BepInEx console shows nothing, last note before crashing the game:

[Message:     Unity] CustomSteamManager init

Content of  ErrorLog.log is as follows:

Setting breakpad minidump AppID = 2103530
SteamInternal_SetMinidumpSteamID:  Caching Steam ID:<PHONE_NUMBER>7553602 [API loaded no]
Fatal error. 0xC0000005
at Il2CppInterop.Runtime.IL2CPP.il2cpp_runtime_invoke(IntPtr, IntPtr, Void**, IntPtr ByRef)
at UnityEngine.ImageConversion.LoadImage(UnityEngine.Texture2D, Il2CppInterop.Runtime.InteropTypes.Arrays.Il2CppStructArray`1)
at IdlePlus.TexturePack.TexturePack.CreateAtlas(System.String)
at IdlePlus.TexturePack.TexturePack.Load()
at IdlePlus.TexturePack.TexturePackManager+<>c__DisplayClass8_0.b__1()
at IdlePlus.Utilities.IdleTasks+<>c__DisplayClass2_0.b__0(IdleTask)
at IdlePlus.Utilities.IdleTasks+IdleTask.Run()
at IdlePlus.Utilities.IdleTasks+IdleTask.Tick()
at IdlePlus.Utilities.IdleTasks.Tick()
at DynamicClass.Trampoline_VoidThisIdlePlus.IdlePlusBehaviourUpdate(IntPtr, Il2CppInterop.Runtime.Runtime.Il2CppMethodInfo*)
at DynamicClass.Invoker_VoidThis(IntPtr, Il2CppInterop.Runtime.Runtime.Il2CppMethodInfo*, IntPtr, IntPtr*, IntPtr*)

So, after analyzing the code, if I'm not mistaken, all textures are loaded at the same time in CreateAtlas() method, instead of being loaded asynchronously. I'll try to modify the code as soon as I manage to finish fighting with VS, I've never worked with projects of this type in C#

Are you able to upload the pack which crashes the game?

Are you able to upload the pack which crashes the game?

MC Pack.zip

I've released version 1.3.1, which should hopefully resolve the issue. If you could test it and let me know whether the fix works, I'd really appreciate it!

Problem solved, good job!


Responsive Cinematic Play View

Features

Adds nine slice dialog bubble. (Lets us remove the svg canvas completely)
Cinematic responds to all screen sizes.

Speech bubbles stay in place
At dimensions larger than >1366px the cinematic is centered.
At dimensions smaller than 1366px the cinematic shrinks (keeping aspect ratio).

Cinematic background added
Font size and line height set

Manually test by

Run your local environment with npm run dev and npm run proxy.
Log in as admin
Navigate to http://localhost:3000/cinematic/1fhl1c5 and enjoy a responsive cinematic.

Screenshots
Full screen

Tall screen

@shubhi1092 Yes. Removing the svg is an amazing simplification. This means the speech bubble is controlled entirely by a 9-slice speech bubble. And therefore also responsive to the canvas via CSS.


storage blocks not formatted very well

Example:
storage {
    map1: StorageMap<u64,
    bool>, map2: StorageMap<u64,
    u8>, map3: StorageMap<u64,
    u16>, map4: StorageMap<u64,
    u32>, map5: StorageMap<u64,
    u64>, map6: StorageMap<u64,
    (b256, u8, bool) >, map7: StorageMap<u64,
    Struct>, map8: StorageMap<u64,
    Enum>, map9: StorageMap<u64,
    str[33]>, map10: StorageMap<u64,
    [b256;
    3]>, map11: StorageMap<bool,
    u64>, map12: StorageMap<u8,
    u64>, map13: StorageMap<u16,
    u64>, map14: StorageMap<u32,
    u64>, map15: StorageMap<(b256,
    u8, bool), u64 >, map16: StorageMap<Struct,
    u64>, map17: StorageMap<Enum,
    u64>, map18: StorageMap<str[33],
    u64>, map19: StorageMap<[b256;
    3],
    u64>,
}

From https://github.com/FuelLabs/sway/pull/1754/files. I expect each StorageMap to get its own line.

@eureka-cpu I'm not sure if there's already a formatter issue open related to the same problem. If there is, please feel free to merge the two issues.

Will test, but I believe this is actually already solved


Move useful capabilities of Firebase pod into the FirebaseCore pod

We want to eventually remove the need for an umbrella Firebase pod and enable developers to directly request the particular Firebase{component} pods that they need.
The first step is moving the Firebase module functionality to Firebase core so that import Firebase and #import <Firebase/Firebase.h> still work.
If this can be done in a non-breaking way, it can happen before Firebase 6.

I did some investigation of splitting the Firebase module capability out of the Firebase pod yesterday.  The conclusion is that there is no great solution without some level of a breaking API change so I'm proposing that we defer this to Firebase 6 along with potentially addressing other questions about the utility of the Firebase umbrella pod.
Investigation details in the next comment.

Here are some approaches to separating the Firebase module from the Firebase pod:

I first tried to move the current Firebase pod preserve_path implementation of the Firebase module directly to the FirebaseCore pod.  This works great for deployed pods, but doesn't work for Development pods. See https://stackoverflow.com/questions/53399463/accessing-preserve-path-files-in-a-development-pod. This would be a development issue and an issue for deploying pre-releases.
I next tried making a CocoaPod with a vendored_library specification to a binary version of the Firebase framework with the header and module map. I also had to add a dummy binary framework since CocoaPods tries to force a linkage to all vendored_frameworks. Once I added the framework, this solution works fine.  However, I'm not sure this hack is compelling enough to add an extra dynamic framework to all Firebase apps.
Finally I made a source FirebaseModule CocoaPod that included Firebase.h, the module map and a dummy .m file. This worked in all scenarios except the static library Podfile scenario.  In a static library configuration modules are generated and included for vendored_libraries but not for source pods.

Implementation details at this branch.

Recommendation for breaking change implementation if we decide to eliminate the umbrella pod functionality:
Merge Firebase module capability into FirebaseCore and rename FirebaseCore to Firebase.
This would break the presumably small number of developers who reference the FirebaseCore module. It also make the Firebase module behavior consistent with other CocoaPods pods, but require static library developers to import the Firebase.h header instead of the module.

The value of this is not worth the potential disruption.  A more likely future path is removing the Firebase pod all together, but that is at least two major release away.


Support args in contributed keybindings

If you insert the following Codeblock to keybindings.json everything works fine:
            {
                "key": "ctrl+oem_2",
                "command": "editor.action.insertSnippet",
                "args": { "name": "scriptbrackets" }
            }

But if you try to insert this block to "keybindings-section" in a package.json of an extension it always calls the snippet selector dropdown (at the top of the window) and is NOT calling the snippet refered by the snippet name.
  "contributes": {
        "languages": [{
            "id": "f-script",
            "aliases": ["F-Script", "f-script"],
            "extensions": [".ff",".fs",".fss",".fx",".lb",".lbd",".lbf",".lbe",".lbi",".lbx",".config"],
            "configuration": "./language-configuration.json"
        }],
        "keybindings" : [
            {
                "key": "ctrl+oem_2",
                "command": "editor.action.insertSnippet",
                "args": { "name": "scriptbrackets" }
            }
        ]
	}

VSCode Version: 1.17.0
OS Version: WIN 10 PRO

Steps to Reproduce:

define a snippet
create a keybinding in keybinding.json
execute the keybinding in a file
try to move the keybinding to package.json.

Hope you can enable this feature also on language-extensions.
Raydir

I know @bpasero added supported for args in keybindings customisation, tho unsure if it is @bpasero or @alexandrudima

We just did not wire this in to the keybindingsRegistry and keybindingsService.
I see no reason why we could not or would not support this but I let @alexandrudima comment on that.
Maybe also an area for a first contributor PR.

https://github.com/Microsoft/vscode/issues/31372


Add device manufacturer specific startup scripts

Some WiFi cards have difficulty with the standard sudo airmon-ng start command and need a bit more help.
I want to add a folder with specific scripts per manufacturer or even groups of devices.

Added some useful generic scripts that can be used

Added generic ones that should be sufficient imho


Bug fix: Array key not exists for $api_key variable

cool thanks for ur bugs fixing

thanks for ur bugs fixing my friend


Swagger UI renders double brackets when array is used in request body of a post method

Please find the attached screenshots for the code. As mentioned, swagger UI generates two brackets for the example array.

as this is related to swagger-ui, please move this ticket there


Prepare for 0.10.0

I wanna hold off on the big 2.0 release for a few days, should just be these two issues https://github.com/danger/danger/milestone/1 - pretty trivial to do, but I wanna time it with the website being ready.
Updates the Dangerfile because now it's shown on Danger.systems

          1 Warning

:warning:
Tests were not updated

  Generated by :no_entry_sign: danger


Newsletter208: add Chinese translation

A few suggestions for ZH links that are available instead of EN

I see. We will notice this next time! Thanks for the suggestion!


Training stopped without notification

I used nvidia-docker to run the code, and this is the output.
02-17 23:40:30 Epoch[0] Batch [9540]    Speed: 18.53 samples/sec        Train-RpnAcc=0.958210,  RpnL1=0.420432, RcnnAcc=0.871656,       RcnnL1=1.303369,
02-17 23:40:42 Epoch[0] Batch [9550]    Speed: 13.73 samples/sec        Train-RpnAcc=0.958218,  RpnL1=0.420353, RcnnAcc=0.871662,       RcnnL1=1.303065,
02-17 23:41:00 Epoch[0] Batch [9560]    Speed: 8.70 samples/sec Train-RpnAcc=0.958227,  RpnL1=0.420287, RcnnAcc=0.871672,       RcnnL1=1.302830,
02-17 23:41:09 Epoch[0] Batch [9570]    Speed: 17.01 samples/sec        Train-RpnAcc=0.958238,  RpnL1=0.420242, RcnnAcc=0.871690,       RcnnL1=1.302588,
root@1e2726dafa6a /o/i/u/d/simpledet#

My training used config tridentnet_r50v2c4_c5_1x which was set to train for 6 epochs with coco2014. But it stopped at Epoch[0] and didn't print any information.
Additionally, I found  in ./deteciton_train.py: 213, in mod.fit func, the param num_epoch=end_epoch, is it configured right?

All the config files should work well. Can you reproduce this issue, and provide more information? Make sure your program is not conflict with others.

OK, I try to train for a second time.

@xchani It seems the problem has disappeared, it successfully run through the whole epoch and saved the weight.

@DayChan I meet the same problem,could you tell me how to solve it ? Thank you !


Proposition: switch to branch releases

Right now, we don't have a great way of handling breaking changes:

we don't have release notes or any communication channel to warn users
we don't have any communication channel to instruct users how to "upgrade"
we don't have any "stable releases"

To fix all three, I propose that we generally follow the practices of nixpkgs:

have a master branch that has no stability guarantees
have one branch per NixOS release, for example

a nixos-23.05 branch
a nixos-23.11 branch
a nixos-24.05 branch

each of these "release branch" is guaranteed to not contain breaking changes
on those "release branches", EPNix will depend on the corresponding nixpkgs branch

Since NixOS/nixpkgs itself has breaking changes, there's a good chance that we'll be affected by them (for example, a major upgrade of a dependency), which is why I think we should have the same release schedule.
Right now, I don't have a specific plan on which of those branches we'll keep supporting. Since the life of an accelerator is much greater than 6 months, there's a good chance that we'll have to maintain those branches longer than nixpkgs. I think for now it's enough to say "best effort".
cc @agaget @Rider128 @stephane-cea @Synthetica9 for your opinions

I agree with your proposition. We will have to update all our EPNix IOCs in order to not use the master branch anymore, this will represent quite some work since we already have many EPNix IOCs, but the sooner the better!

I agree with the proposition. I think a branch "unstable" will be great, for push easily in master.

Seems sensible, wouldn't really impact us as we're not running EPNix in production (yet)

Note from experience in nixpkgs, I think I'm going to disable creating any branch in the main EPNix repository, so that there's only "official branches" there. People would have to contribute through forking.

Branch releases are now in place, with the help of a backport mechanism (the same as in nixpkgs). We now have three main branches:

main
nixos-23.05
nixos-23.11

No new branches can be created on the main repository (except for backports and dependabot).


Consider explicit deletion of stream objects on exit

Depends on the balance between the number of times stdio_mgr is used, the frequency of calls, and the frequency of suitable garbage collection.
Prompted by this SO answer. That example uses a class for the context manager, though, with a recommendation to retain the instance for storing the captured stream contents.  With stdio_mgr being implemented as a function, especially one where the streams are closed on context exit, there may not be as big of a problem with memory consumption.

https://github.com/bskinn/stdio-mgr/issues/3 is sort of related, and on its way to being solved.
I think .close() would have similar effect on memory consumption, as all buffers are released, but worth checking.

In most modern contexts, for anything but the most ridiculous of large streams, I can't imagine this Issue is likely to be a big issue. Seems very YAGNI.
Especially with the changes from #25 and #26, though -- as you noted at the bottom of #26, the mocked stream objects are now storing internal references to the underlying buffers in some situations, and that could lead to memory accumulation if enough things stay in scope/have references.

Although -- both RandomTextIO and TeeStdin store a reference to the buffer in ._buf... as long as the (i, o, e) objects remain in scope, I would think they will retain references to their ._bufs the whole time, even after .close() or .detach(), right?
I guess .close() frees the actual buffer memory, so in that case it is just ._closed_buf to worry about in terms of memory consumption.
But for .detach(), I'd think it would still matter -- though, if someone is .detach()-ing, they likely still want to work with the stream data, and so it would probably be a total footgun to wipe the buffer after just a .detach().
So, yeah: just ._closed_buf to worry about, then.

With #58 switching the persisted-content mechanism from ._closed_buf to a callback, this issue should be moot.

I think we at least need tests for this, and it turns out to be very important to delete objects when using TemporaryFile and when using the real sys handles.


Crash on Login on Server with Quark

this particular crash is wierdly complex because it involves the enchanted books in the hands of minecolonies mobs.
But looks to us like an intersection of this with quark.
https://pastebin.com/ApYtfGbh

I believe the pull request that I submitted should fix this but it's up to the author of the mod to accept it and release an updated version.


Do we have something like custom footer options to show sum/totals for each column ?

Please use the issues section for bugs / feature requests.
I answered your question here: http://stackoverflow.com/questions/41049448/w2ui-grid-custom-footer-display/41055253#41055253

Thanks mpf82. Yes I see you answered it on stack overflow. I'll keep in mind to mention only issues related topics here.


bug: Failed to parameterize error while running move2kube transform on docker-compose file

Describe the bug
I am getting an error failed to parameterize YAML files in the source directory while running move2kube transform on the below docker-compose file
To Reproduce
Steps to reproduce the behavior:

Input docker-compose.yml

services:
  hello_worldDLR_BR_OP_TAG_BR_CL:
    command:
    - /bin/echo
    - Hello world
    deploy:
      placement:
        constraints:
        - node.a.b.c == true
    environment:
    - env:${CODE}
    image: ubuntu:${TAG}
    links:
    - redis${TAG}:redis${TAG}
    ports:
    - ${UIPORT}:8443
version: "3"

Run move2kube transform over the source folder dc-input.zip which contains the above docker-compose file.

Choose the cluster type as IBM Openshift or Openshift. (The error doesn't appear if I select Kubernetes as the cluster type.)

? Choose the cluster type:
ID: move2kube.target."default".clustertype
Hints:
- Choose the cluster type you would like to target

  [Use arrows to move, type to filter]
  AWS-EKS
  Azure-AKS
  GCP-GKE
  IBM-IKS
> IBM-Openshift
  Kubernetes
  Openshift

It throws an error towards end of transformation.

? Provide the ingress host domain
ID: move2kube.target."default".ingress.host
Hints:
- Ingress host domain is part of service URL

 t14.com
INFO[0041] Transformer Kubernetes Done
INFO[0041] Transformer ClusterSelector processing 2 artifacts
INFO[0041] Transformer ClusterSelector Done
INFO[0041] Transformer Tekton processing 2 artifacts
INFO[0041] Transformer Tekton Done
INFO[0041] Created 30 pathMappings and 10 artifacts. Total Path Mappings : 30. Total Artifacts : 3.
INFO[0041] Iteration 4 - 10 artifacts to process
INFO[0041] Transformer Parameterizer processing 6 artifacts
WARN[0041] Error in walking through files due to : "stat /var/folders/sw/_kjrz98j3ws1qj7zz6chzhrr0000gn/T/move2kube1729845829/environment-Parameterizer-3160189886/2622198968/deploy/cicd/buildconfig: no such file or directory"
ERRO[0041] failed to parameterize the YAML files in the source directory /var/folders/sw/_kjrz98j3ws1qj7zz6chzhrr0000gn/T/move2kube1729845829/environment-Parameterizer-3160189886/2622198968/deploy/cicd/buildconfig and write to output directory /var/folders/sw/_kjrz98j3ws1qj7zz6chzhrr0000gn/T/move2kube1729845829/environment-Parameterizer-3160189886/2995956675/buildconfig-parameterized . Error: "stat /var/folders/sw/_kjrz98j3ws1qj7zz6chzhrr0000gn/T/move2kube1729845829/environment-Parameterizer-3160189886/2622198968/deploy/cicd/buildconfig: no such file or directory"

Expected behavior
The above error should not show up while running move2kube transform
Screenshots

Desktop (please complete the following information):
move2kube version -l
version: v0.3.0+unreleased
gitCommit: 0d8f71ad8264029e04b5d919ca7f5ccb9c1d8c7d
gitTreeState: clean
goVersion: go1.18.1
platform: darwin/amd64

Seems the ir.ContainerImages is empty

I just checked that the same issue can also be reproduced with the move2kube-demos docker-compose sample https://github.com/konveyor/move2kube-demos/blob/main/samples/docker-compose/multiple-services/docker-compose.yaml if I comment-out/remove lines 6 and 17, i.e., if the docker-compose file doesn’t have the build key for any service.

Tried adding the build key to the Dockerfile in the original post:
services:
  hello_worldDLR_BR_OP_TAG_BR_CL:
    build: .
    command:
    - /bin/echo
    - Hello world
    deploy:
      placement:
        constraints:
        - node.a.b.c == true
    environment:
    - env:${CODE}
    image: ubuntu:${TAG}
    links:
    - redis${TAG}:redis${TAG}
    ports:
    - ${UIPORT}:8443
version: "3"

This works without error, produces ContainerImages and BuildConfig

fixed by adding a check before creating the artifact in BuildConfig transformer https://github.com/konveyor/move2kube/pull/963/files#diff-2a1ff621d16165f6fe85b3c4599804d42481d872fbebab5c38136dce75c83e01R152


NDJSON test data doesn't contain variable names

The NDJSON data doesn't contain variable names in each row, only values.
For example:
With variables names:
{"name": "Leandro","lastName": "Shokida"} {"name": "Mariano","lastName": "De Achaval"}
Without variable names:
{"Leandro", "Shokida"} {"Mariano", "De Achaval"}
From what I can tell we can:

Update the NDJSON test data so that each row contains variable names.  We can then stream this directly to a data frame.
If this is intentional, we can read this in as a list of lists, bind the rows and convert to a data frame.

@mstackhouse Are you aware or able to check with Sam or Lex to confirm the test data for ndjson is valid?

@nicholas-masel are you talking about the row-level data itself? So for the data records, or for the variable level metadata? Because this is the same case for the non-NDJSON data too:
From here
{
  "datasetJSONCreationDateTime": "2023-06-28T15:38:43",
  "datasetJSONVersion": "1.1.0",
  "fileOID": "www.sponsor.xyz.org.project123.final",
  "dbLastModifiedDateTime": "2023-05-31T00:00:00",
  "originator": "Sponsor XYZ",
  "sourceSystem": {
      "name": "Software ABC",
      "version": "1.0.0"
  },
  "studyOID": "cdisc.com.CDISCPILOT01",
  "metaDataVersionOID": "MDV.MSGv2.0.SDTMIG.3.3.SDTM.1.7",
  "metaDataRef": "https://metadata.location.org/CDISCPILOT01/define.xml",
  "itemGroupOID": "IG.DM",
  "isReferenceData": false,
  "records": 18,
  "name": "DM",
  "label": "Demographics",
  "columns": [
      {"itemOID": "ITEMGROUPDATASEQ", "name": "ITEMGROUPDATASEQ", "label": "Record Identifier", "dataType": "integer"},
      {"itemOID": "IT.STUDYID", "name": "STUDYID", "label": "Study Identifier", "dataType": "string", "length": 12, "keySequence": 1},
      {"itemOID": "IT.DOMAIN", "name": "DOMAIN", "label": "Domain Abbreviation", "dataType": "string", "length": 2},
      {"itemOID": "IT.USUBJID", "name": "USUBJID", "label": "Unique Subject Identifier", "dataType": "string", "length": 8, "keySequence": 2},
      {"itemOID": "IT.AGE", "name": "AGE", "label": "Age", "dataType": "integer"}
  ],
  "rows": [
      [1, "CDISCPILOT01", "DM", "CDISC001", 84],
      [2, "CDISCPILOT01", "DM", "CDISC002", 76],
      [3, "CDISCPILOT01", "DM", "CDISC003", 61],
      ...
  ]
}

The only change for NDJSON is that the rows elements are instead there own lines of the file:
{
  "datasetJSONCreationDateTime": "2023-06-28T15:38:43",
  "datasetJSONVersion": "1.1.0",
  "fileOID": "www.sponsor.xyz.org.project123.final",
  "dbLastModifiedDateTime": "2023-05-31T00:00:00",
  "originator": "Sponsor XYZ",
  "sourceSystem": {
      "name": "Software ABC",
      "version": "1.0.0"
  },
  "studyOID": "cdisc.com.CDISCPILOT01",
  "metaDataVersionOID": "MDV.MSGv2.0.SDTMIG.3.3.SDTM.1.7",
  "metaDataRef": "https://metadata.location.org/CDISCPILOT01/define.xml",
  "itemGroupOID": "IG.DM",
  "isReferenceData": false,
  "records": 18,
  "name": "DM",
  "label": "Demographics",
  "columns": [
      {"itemOID": "ITEMGROUPDATASEQ", "name": "ITEMGROUPDATASEQ", "label": "Record Identifier", "dataType": "integer"},
      {"itemOID": "IT.STUDYID", "name": "STUDYID", "label": "Study Identifier", "dataType": "string", "length": 12, "keySequence": 1},
      {"itemOID": "IT.DOMAIN", "name": "DOMAIN", "label": "Domain Abbreviation", "dataType": "string", "length": 2},
      {"itemOID": "IT.USUBJID", "name": "USUBJID", "label": "Unique Subject Identifier", "dataType": "string", "length": 8, "keySequence": 2},
      {"itemOID": "IT.AGE", "name": "AGE", "label": "Age", "dataType": "integer"}
  ]
}
[1, "CDISCPILOT01", "DM", "CDISC001", 84]
[2, "CDISCPILOT01", "DM", "CDISC002", 76]
[3, "CDISCPILOT01", "DM", "CDISC003", 61]
...

Yeah, I was talking about variable names on the row-level data.  I reached out to Sam and he confirmed this was not included due to file size.
I am trying out reading as a list instead of a df, and it seems to work, but is causing some other type issues downstream that didn't appear when reading this directly to a df.
yyjsonr::read_ndjson_str(
      file,
      type = "list",
      nskip = 1,
      opts = json_opts
    )


String Handling in Noir

Problem
I want to implement hashmap, random and decimals in Noir but these would require the presence of a good String type that has the required handling functions. Iterating over chars and changing them should be possible with mutable strings.
Proposed solution
A String/Text datatype in the base library that can do this. A detailed documentation would also be nice.
Alternatives considered
None. Pretty sure no language has ever had no string handling since the days of assembly language.
Additional context
No response
Submission Checklist

[ ] Once I hit submit, I will assign this issue to the Project Board with the appropriate tags.

I'm curious what you exactly need strings for. Can't you do a hash map of just string hashes? Decimal arithmetic might be doable with strings, but I think bytes arrays are more fitting


Adapted to swift 4

Thanks for moving this forward! Can you update the .travis.yml file to what I have in https://github.com/CrossWaterBridge/Attributed/blob/2b101cd6961ee9f8df2512cc81c705d1ae4e1487/.travis.yml, and run the unit tests locally to see if anything needs fixing there?

Hi @hiltonc,
I will close this PR and create a separate one, as I don't have access to the repository I have created PR from.


can't pipe through multiple commands

I can't pipe through multiple commands I'm thinking it's getting caught in buffering and not flushing output.
but this isn't working
chokidar **/*.scala | awk -F: '{print $2}' | awk -F/ '{print $1}'
isn't working at all

This is an issue for me as well. I see no output when piping commands, e.g.
chokidar src | cat


Add event listener to appended element on Media Control

Hello, everyone.
I was writting this plugin, where I append an element to the Media Control. I added a click listener to the element, but it wouldn't fire.
I don't know if there's a div above the controls, or a ponter-event: none on some of the parents. Anyways...
I could work a way around though, but it ain't pretty. I added a class to the element, then I listened to clicks anywhere on the player, and I check if the target of the click contains the class I put on it.
// ...
          this.core.$el.on('click', this.onMediaControlButtonClick.bind(this));
}

onMediaControlButtonClick : function(mouseEvent) {
            if(!mouseEvent.target.classList.contains('media-control-subtitle-toggler'))
                return;
          // ... handle it here
/// ...

Is there an official way to add a listener to an appended element? Thank you

Hi @thalleshmm, I would recommend you to do something like we do on some control plugins like clappr-chromecast-plugin. Just create a UICorePlugin with a click listener bound directly on its properties:
class MyPlugin extends UICorePlugin {
  //...
  get events() {
    return {
      'click': 'click'
    }
  }
  function click() {
    // ... handle it here
  }
}

Please let me know if this works for you.

Thank you @towerz. =)
It didn't work for me though. I'm not writing in ES2015. I must have done something wrong. It should be something like the code below, shouldn't it?
Clappr.CorePlugin.extend({
        events :  {
            click : 'click',
        },
        click : function() {
            alert('yay, you clicked');
        },
})

@thalleshmm you can do the same without ES2015. The attribute notation is from ES5 standard. The code would look like this:
Clappr.CorePlugin.extend({
  get events() {
    return { click: 'click' }
  },
  click: function() {
    alert('yay, you clicked!')
  }
})

Didn't know the "get" thingy would work. Throught it was part of the ES2015 specs. I'll take a look at that and write u back. Thank you!


sy-index in WHILE

Hello
sy-index for WHILE seems to be broken

CLASS zcl_secret_handshake DEFINITION
  PUBLIC
  FINAL
  CREATE PUBLIC .

  PUBLIC SECTION.
    METHODS get_commands
      IMPORTING
        code            TYPE i
      RETURNING
        VALUE(commands) TYPE string_table .
  PROTECTED SECTION.
  PRIVATE SECTION.
ENDCLASS.

CLASS zcl_secret_handshake IMPLEMENTATION.

  METHOD get_commands.
    DATA(lt_handshake) = VALUE string_table( ( |wink| )
                                             ( |double blink| )
                                             ( |close your eyes| )
                                             ( |jump| )
                                             ( |REVERSE| ) ).
    DATA(l_code) = code.
    DATA(lv_index) = 0.
    WRITE / |------------------------|.
    WHILE l_code > 0.
      lv_index += 1.

      WRITE / |{ lv_index } - { sy-index }|.

      IF l_code MOD 2 = 1.
        DATA(l_command) = lt_handshake[ lv_index ].
        IF l_command <> |REVERSE|.
          APPEND l_command TO commands.
        ELSE.
          DATA(lt_copy) = VALUE string_table( ).
          LOOP AT commands INTO l_command.
            INSERT l_command INTO lt_copy INDEX 1.
          ENDLOOP.
          commands = lt_copy.
        ENDIF.
      ENDIF.

      l_code = l_code DIV 2.
    ENDWHILE.
  ENDMETHOD.

ENDCLASS.

thanks, reproduced as
DATA i TYPE i.
WHILE i < 10.
  WRITE / sy-index.
  i = i + 1.
ENDWHILE.

exercism update = https://github.com/exercism/abap-test-runner/pull/154

👍


Test xml metadata

Added test to evaluate xml metadata creation for stitching.

@cbrrrry @jtyoung84 Is this still being worked on?

No I think this is a stale branch now. We are no longer parsing these large log files, and thus mocking them is unnecessary. I will close this PR.

Can we delete the branch

No I think this is a stale branch now. We are no longer parsing these large log files, and thus mocking them is unnecessary. I will close this PR.

Can the branch be deleted?

I think so. The relevant changes are being worked on in other branches/refactoring efforts.


High CPU usage on 5.2

Describe the bug
After upgrade to recent version lens started to use cpu very intense. It takes about one full core and quickly makes fan loud. I can't see that cpu usage is even higher when I'm connected to some cluster and running some consoles there. It's about 40% of cpu. When it's completely idle (disconnected from everything on welcome screen) it's about 20%.
To Reproduce
Nothing special - just running lens on windows 10.
Expected behavior
It should not eat so much CPU for sure.
Screenshots
This is when it's completely idle on start screen:

When it's connected and used normally it's much worse:

Environment (please complete the following information):

Lens Version: latest - 5.2
Windows 10
Installed via installer from homepage

Additional context
This happened soon after update, I see it's using GPU but same thing happens in chrome where hardware accelleration is disabled. I don't know how to open "chrome://gpu" panel information, but this may be good point to start.
Please take a look at performane console with lens running and closed (last few seconds)

Hi, i had the same problem and succeeded in reducing the high CPU consumption by disabling these options.

@jmcarranza I already disabled first two options, but this is not that
issue. After lens restart I see that it's working ok without high cpu
usage. Later it eat more and more CPU resulting in something like in
screens. It may be somehow related with computer sleep mode (I'm using that
frequently).
On Tue, Sep 14, 2021 at 3:45 PM jmcarranza @.***> wrote:

Hi, i had the same problem and succeeded in reducing the high CPU
consumption by disabling these options.
[image: image]
https://user-images.githubusercontent.com/9244484/133268911-0bdea3a7-264a-4c1a-9981-b59fc0784c6c.png
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/lensapp/lens/issues/3777#issuecomment-919166221, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/ACSQNBKKQLLJ4GWYOYNQSKDUB5GYFANCNFSM5D2BIJXQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Same here @dominch but I only have between 2-8 processes.
But high CPU usage starts immediately after starting Lens and doing nothing. Just watching the Pod list with three items.
And it does not matter if the UI is in the background or active usage.

I'm also on version 5.2 and have the same issue.

To be honest, I've noticed this with several versions of Lens going back to 4.x. Is this to be expected? It'd be nice to get an official response on if this. Not sure if it matters, but I have ~10 clusters added.

To be honest, I've noticed this with several versions of Lens going back to 4.x. Is this to be expected? It'd be nice to get an official response on if this. Not sure if it matters, but I have ~10 clusters added.

I don't think that this is a problem if the number of clusters. I have added between one and two clusters and use sometimes both in parallel.
But the problem happens even if I only use one.

I can reproduce this on Windows 10 and it's most likely something to do with new node-fetch based api communication backend.

Good day!
I have the same problem on Windows 10.

Version:

It seems that latest update fixes issue:

It came today and clearly it don't drain CPU anymore. I'll play more with that version and report if anything is wrong, but as far as I can see after few minutes problem is probably fixed.

I agree with @dominch. Looks like the last update

fix issue with CPU's perfomance.

Yes, I can also confirm that the latest release is much better.
But when I open a terminal or the logs overview my CPU is used around 10-20%.
Maybe that is normal but the CPU cooler is pretty busy 😅😏.

hi, i update to the same verison and the problem persists

Confirmed. 5.2.3 does not resolve the issue.
5.1.3 is the last one that works properly.

Today I was able to reproduce issue with current version,

Interestingly - here it's graph with cpu usage and lens restarting and yet again don't drain cpu:

(one peak is lens starting, next one is connecting to cluster)
On older version there was always about 20-30% of cpu, now soon after restart it's ok - less than 4% on idle (one cluster connected). As you can see sometimes things goes wrong and yet again cpu usage is higher, I had today constant 15-20% and soon later 40%. For now I need to restart lens when this happens.
For now let's reopen issue and try to find cause. It does not happen always but sure there is something more about cpu usage.

Yesterday I had no problem with high cpu usage, today after few restarts I noticed that it starts with good cpu consuption <3%, after one cluster is connected and listing few pages (just clicking on menu positions) it starts to use 15-20% of cpu. Turning on debug console doubles cpu usage to 35-45%, everything on idle.

When problem is visible restarting lens helps, any reload has no effect I need to shut down it, wait about 10s and start it again.

After tonight upgrade Yet again seems to be ok.

I've been using it for about 2-3 hours and it did not reached more than 1% of CPU now.

Let's hope that issue is solved :) I will test it today and report back at the end of day.

I've been trying to isolate the cause for this issue.  It happened today after I restarted kubelet on the control plane.
It appears if there is a connectivity issue with the cluster, even a transient one, cpu suddently spikes.   The connection does get restored, but Lens had to be restarted to get CPU usage back to normal.  (Closing the GUI was not enough.  Killing was task manager was required in one instance when I could not interact with the system tray icon.)

It looks like this fixed it for me. Thanks for the quick turnaround!

Indeed, it looks like it is resolved. I have been running it the whole day connected to four GKE clusters simultaneously. Lens stays below 1% utilization, briefly jumping to 3-4% from time to time. Thank you

Yes, yet again it's ok :) After four days it stay <%1 almost all the time
I'm marking it as resolved again


Order prop not  working for me

Issue description
Order prop not  working for me, all the notations show up at once
Here  my code
    <RoughNotationGroup show={true}>
      <RoughNotation type="highlight" order={1} show={true}   >
        some text
      </RoughNotation>
      <RoughNotation type="underline" order={2} show={true}  >
        some other text
      </RoughNotation>
    </RoughNotationGroup>

I'm using
    "react": "^18",
    "next": "^14.2.17",

Hello guys, Not sure if it is an issue or i'm doing something wrong ?
Can you help me with it.
Thank you!

By removing show={true}  it works


Latest sass 1.69.6 doesnt work - Cannot read properties of undefined

Environment

Operating System: Darwin
Node Version:     v21.0.0
Nuxt Version:     3.9.0
CLI Version:      3.10.0
Nitro Version:    2.8.1
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      devtools, css
Runtime Modules:  -
Build Modules:    -

Reproduction
It does NOT happen on Stackblitz for some reason:
https://stackblitz.com/edit/github-qarseh
But I confirmed by creating a new project on my machine:

nuxi init sass-test
cd sass-test
yarn add sass
Put this in nuxt.config.ts: css: ['~/assets/styles/main.scss']
Create file: assets/styles/main.scss
yarn dev

Describe the bug
Using the latest version 1.69.6 of sass which has been released 5 days ago throws an error when starting with nuxt dev:

[5:54:18 PM]  ERROR  Pre-transform error: Preprocessor dependency "sass" failed to load:
Cannot read properties of undefined (reading 'indexOf')

And then when visiting localhost:

500
[vite-node] [plugin:vite:css] [VITE_ERROR] /assets/styles/main.scss

Downgrading to 1.69.5 works
Additional context
No response
Logs
No response

This is likely a sass or vite issue rather than a Nuxt one. Would you see if you can reproduce without Nuxt?
https://stackblitz.com/github/nuxt-contrib/vue3-ssr-starter


Update README.md

Other table column titles match their respective bullet point title. It would be nice for phonetic to read phonetic in the bullet point and not matcher

This is actually the other way around in the code. So the header in the table is incorrect, not the text under it.

To be fair, I think it would be better if the field was named phoneticMatcher but that would be a breaking change.

Oooh
Okay, noted.

Do I revert changes and close this or do I make the change to the incorrect one?

Pushing for coherency

Redis is pretty cool as well
Learning it cos I have a project where its use seems reasonable.
Using it together with MongoDB

Just go ahead and update the PR and then I can merge it in.

I'm away from my laptop
But I would have loved to share my planning document so you see.
It's an app that'll have contests between users.
So there's a need to show in relatime contests on both devices in sync and send and receive events as fast as possible at a reasonable cost

I've made the change also

I'm away from my laptop But I would have loved to share my planning document so you see.
It's an app that'll have contests between users. So there's a need to show in relatime contests on both devices in sync and send and receive events as fast as possible at a reasonable cost

Sounds like a good fit for Redis, probably including things beyond what Redis OM provides. Unfortunately, I get a lot of requests like this and don't have a lot of time to look at them. However, you might find some folks on our Discord server that would be interested in your project. It's at discord.gg/redis.

I thought I responded to this, but I didn't.
Apologies.
Thank you Guy.
Will look into the Discord server.
On Fri, Jun 7, 2024, 18:06 Guy Royse @.***> wrote:

I'm away from my laptop But I would have loved to share my planning
document so you see.
It's an app that'll have contests between users. So there's a need to show
in relatime contests on both devices in sync and send and receive events as
fast as possible at a reasonable cost
Sounds like a good fit for Redis, probably including things beyond what
Redis OM provides. Unfortunately, I get a lot of requests like this and
don't have a lot of time to look at them. However, you might find some
folks on our Discord server that would be interested in your project. It's
at discord.gg/redis.
—
Reply to this email directly, view it on GitHub
https://github.com/redis/redis-om-node/pull/238#issuecomment-2155214850,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEB3T7GCIR47R35M7PUZ7H3ZGHSAJAVCNFSM6AAAAABIV5U4Q2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDCNJVGIYTIOBVGA
.
You are receiving this because you authored the thread.Message ID:
@.***>


fix(jellyfish-api-core): Account history missing some fields for pool rewards

What happened:
The txn and txid fields are missing in an AccountHistory instance returned from pagination for pool rewards (options.no_rewards = false) of listAccountHistory RPC
What you expected to happen:
The txn and txid fields should be included in the AccountHistory
How to reproduce it (as minimally and precisely as possible):
Reproduce by adding a test into [packages/jellyfish-api-core/tests/category/account/listAccountHistory.test.ts]
Anything else we need to know?:
https://github.com/DeFiCh/ain/blob/acd6ce34181f54ab97a45733a6b830961ee1450f/src/masternodes/rpc_accounts.cpp#L65 According to this rewardhistoryToJSON method implementation in Ain, The txn and txid fields are not included

@chanakasameera AFAIK, these AccountHistory are legitimately have neither txid nor txn.
Jellyfish-api-core listAccountHistory is forced to have no_rewards true as well thus result have non undefined values for these two. Maybe we should close this?


Problems with "?" in robots.txt

In https://www.welt.de/robots.txt there are ? containing entries like Disallow: /*?config. Hence https://www.welt.de/test?config should be allowed but it is not. Whereas entries like Disallow: /*.xmli work properly and disallow https://www.welt.de/test.xmli. After my investigation I figured out that ? is the problematic character.
I use RobotstxtServer#allow("https://www.welt.de/test?config") for testing.

Hi @s17t - I'm interested in why having the sitemap parsing code in the jar is an issue for you. Does it add a significant amount of code, and/or pull in a lot of other jars as transitive dependencies? Thanks!

... Disallow: /*?config. Hence https://www.welt.de/test?config should be allowed but it is not.

According to Google's spec this URL is not allowed (* is a glob character, while ? is not). The RFC draft does not support any glob characters in allow/disallow statements (see robotstxt.org):

Note also that globbing and regular expression are not supported ... Specifically, you cannot have lines like ... "Disallow: *.gif".

crawler-commons follows here the Google spec which is widely used - I suppose that even the webmaster of welt.de wants to disallow /test?config.

According to https://en.wikipedia.org/wiki/Glob_(programming) both * and ? are glob characters. RFC drafts are nice but ultimately Google sets the standards (even if they are not RFC-compliant). And according to Google * is supported as stated in Google's spec.

Agreed, Google sets the standard and this means ? has no glob meaning in the Disallow: statement which " support a limited form of "wildcards" for path values".

@loxal, or did I misunderstand you: Shall /*?config match /test?configor not? If yes, it's disallowed, right?

@sebastian-nagel whatever Google considers correct. But ? does not need any glob meaning to support
Disallow: /*?config and disallow /test?config? In this case * is a glob character and is supported, whereas ? is just a regular character.
I think the current implementation considers also ? as a glob character and therefore has problems to evaluate *?.

Ok, we agree. I haven't tested crawler4j but just wanted to make sure that crawler-commons behaves as expected: /test?config is forbidden but eg. /test/config is allowed. Thanks!

I just checked with https://www.google.com/webmasters/tools/robots-testing-tool
No matter what Google's documentation says, the actual implementation speaks another language:
Entries like Disallow: /*?config result in /test?config being ignored. One of those -  not too rare - case where documentation is misleading.

Entries like Disallow: /*?config result in /test?config being ignored.

What does "being ignored" mean? Is it crawled or not?

If it means that Googlebot does not visit /test?config this is in accordance with the documentation.

The crawler ignores it. /test?config is not crawled. This is what Google's robots Tool says.

Yes, but then the behavior and documentation agree: the wildcard * matches test and all remaining characters (including ?) are matched literally. It's a disallow rule, so bots shall not crawl everything matching  /*?config, right?

Yes.

I'll keep this on the radar, and will add a unit test to crawler-commons' robots.txt parser, just to make sure that it continues to work. Thanks!

Thank you.
On Thu, Mar 22, 2018 at 11:27 AM, Sebastian Nagel<EMAIL_ADDRESS>wrote:

I'll keep this on the radar, and will add a unit test to crawler-commons'
robots.txt parser, just to make sure that it continues to work. Thanks!
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/yasserg/crawler4j/issues/304#issuecomment-375231908,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABrbW1ALA3rpmrjB8xS0nJo6K1-FNmSFks5tg26egaJpZM4StjHr
.


distribute() command in multi-channel pipette and WellSeries

Protocol:
from opentrons import containers, instruments, robot

robot.reset()
plate = containers.load('384-plate', 'A1')

platerows = []
platerows.append(plate.rows(0).wells('A', length=8, step=4))
platerows.append(plate.rows(1).wells('A', length=8, step=4))

p50multi = instruments.Pipette(axis='a', channels=8)
matcell = platerows[0]

p50m.distribute(
    10,
    platerows[0],
    platerows,
    new_tip='once'
)

robot.commands()

Results in 20 aspirate/dispense pairs. Expected: 2 aspirate/dispense pairs.
Expected behavior can be achieved through:
p50multi.pick_up_tip()
for platerow in platerows:
    p50multi.aspirate(1.0, matcell)
    p50multi.dispense(1.0, platerow.bottom())
p50multi.drop_tip()

@andySigler

By default max_volume is set as min_volume + 1, thus in the case above we have max_volume set to 1, and min_volume value is 0. That leads the robot in situation to be able to transfer at max only 1ml of the liquid on each run.
Python 3.5.3
[GCC 4.2.1 Compatible Apple LLVM 7.3.0 (clang-703.0.31)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from opentrons import containers, instruments, robot
>>>
>>> robot.reset()
<opentrons.robot.robot.Robot object at 0x10e159f28>
>>> plate = containers.load('384-plate', 'A1')
>>>
>>> platerows = []
>>> platerows.append(plate.rows(0).wells('A', length=8, step=4))
>>> platerows.append(plate.rows(1).wells('A', length=8, step=4))
>>>
>>> p50multi = instruments.Pipette(axis='a', channels=8)
>>> matcell = platerows[0]
>>>
>>> p50multi.distribute(
...     10.0,
...     platerows[0],
...     platerows,
...     new_tip='once'
... )
<opentrons.instruments.pipette.Pipette object at 0x10ef0fcc0>
>>>
>>> robot.commands()
['Aspirating 1.0 at <WellSeries: <Well A1><Well E1><Well I1><Well M1><Well A1><Well E1><Well I1><Well M1>>',
'Dispensing 1.0 at <WellSeries: <Well A1><Well E1><Well I1><Well M1><Well A1><Well E1><Well I1><Well M1>>',
'Aspirating 1.0 at <WellSeries: <Well A1><Well E1><Well I1><Well M1><Well A1><Well E1><Well I1><Well M1>>',
'Dispensing 1.0 at <WellSeries: <Well A1><Well E1><Well I1><Well M1><Well A1><Well E1><Well I1><Well M1>>',
'Aspirating 1.0 at <WellSeries: <Well A1><Well E1><Well I1><Well M1><Well A1><Well E1><Well I1><Well M1>>',
'Dispensing 1.0 at <WellSeries: <Well A1><Well E1><Well I1><Well M1><Well A1><Well E1><Well I1><Well M1>>',
'Aspirating 1.0 at <WellSeries: <Well A1><Well E1><Well I1><Well M1><Well A1><Well E1><Well I1><Well M1>>',
'Dispensing 1.0 at ...
 'Dispensing 1.0 at <WellSeries: <Well A2><Well E2><Well I2><Well M2><Well A2><Well E2><Well I2><Well M2>>', 'Returning tip', 'Drop_tip ']

as you can see in example above we transfer bunch of times. But if we set amount of volume transferred to 1.0 (probably dot was missing):
Python 3.5.3
[GCC 4.2.1 Compatible Apple LLVM 7.3.0 (clang-703.0.31)] on darwin
Type "help", "copyright", "credits" or "license" for more information.
>>> from opentrons import containers, instruments, robot
>>>
>>> robot.reset()
<opentrons.robot.robot.Robot object at 0x10b5c6f28>
>>> plate = containers.load('384-plate', 'A1')
>>>
>>> platerows = []
>>> platerows.append(plate.rows(0).wells('A', length=8, step=4))
>>> platerows.append(plate.rows(1).wells('A', length=8, step=4))
>>>
>>> p50multi = instruments.Pipette(axis='a', channels=8)
>>> matcell = platerows[0]
>>>
>>> p50multi.distribute(
...     1.0,
...     platerows[0],
...     platerows,
...     new_tip='once'
... )
<opentrons.instruments.pipette.Pipette object at 0x10c386358>
>>>
>>> robot.commands()
['Aspirating 1.0 at
<WellSeries: <Well A1><Well E1><Well I1><Well M1><Well A1><Well E1><Well I1><Well M1>>',
'Dispensing 1.0 at <WellSeries: <Well A1><Well E1><Well I1><Well M1><Well A1><Well E1><Well I1><Well M1>>',
'Aspirating 1.0 at <WellSeries: <Well A1><Well E1><Well I1><Well M1><Well A1><Well E1><Well I1><Well M1>>',
'Dispensing 1.0 at <WellSeries: <Well A2><Well E2><Well I2><Well M2><Well A2><Well E2><Well I2><Well M2>>', 'Returning tip', 'Drop_tip ']

Closing because this ticket is over 2 years old, is for PAPIv1, and I can't actually tell if there's a bug here


fix useWindowDimensions hook

Summary
I found useWindowDimensions hook, and thought that it would solve many issues for me, but implementation looked interesting. So I created a new project, copy pasted the code and console.log dimentions returned from the hook, then rotated iOS simulator and it's printing dimentions non-stop.
Below is my fix for useWindowDimensions, and it works for me.
Changelog
[GENERAL] [CHANGED] - fix useWindowDimensions hook
Test Plan
Copy paste the code to RN project and console.log the return value, and rotate screen it'll log new dimensions.

please review @brunolemos

Yes it looks more similar to what I would write myself. But the hook was written by @sahrens so I’ll let this to him.

I don't think that objects will be the same, even if dimensions are same.

Ah yes, the issue is that when we added the bind to forceUpdate, now the dependencies for the useEffect change on every render, which means it also does the comparison with initialDims, which means we always re-render if you change the dims once.
Your implementation here is the simple obvious one, but risks missing an event and never rendering with the correct dims. I'm pretty sure if we just pass an updater function when calling forceUpdate rather than binding it on every render, everything should work.

I'm going to import this but with some changes to handle lost updates.

I'm not getting your concerns about missing events, thus re-renders. This
hook sets initial value and handles events as they occur, and components
will re-render itself if value changes.

@grabbou Can you confirm that mate? :) Thanks

I also ran into this. Would be great if it would be included in the next react-native release.

@jfrolich This change is indeed included in the 0.62 stable branch, so it should be included :)
https://github.com/facebook/react-native/blob/0.62-stable/Libraries/Utilities/useWindowDimensions.js


Godot GDScript Lacking Syntax Highlighting

Would it be possible to add GDScript support for syntax highlighting? You can see super() and methods calls aren't coloured.

If there's any information required let me know I'm not sure what you'd need.

There's a grammer reference here. Maybe it'll be useful.

The issue here seems to be more with the extension, which needs to be installed, which doesn't recognize some special keywords.
For example with the super() calls, it recognizes them as function.any-method

which in my opinion clearly isn't. It's not a function/method like any others.
About method calls, they shouldn't be colored according to the syntax highlighting of the website:

Ah ok. Thanks for getting back to me. This doesn't seem to be an issue with the theme then. I'll close this now.


[ISSUE #6881] Fix scheduled messages are replayed bug

Which Issue(s) This PR Fixes
https://github.com/apache/rocketmq/issues/6881
Fixes #issue_id
Brief Description
scheduled messages are replayed bug
How Did You Test This Change?

Codecov Report

Merging #6882 (d9d2f65) into develop (4d82b30) will decrease coverage by 0.01%.
The diff coverage is 100.00%.

@@              Coverage Diff              @@
##             develop    #6882      +/-   ##
=============================================
- Coverage      42.45%   42.45%   -0.01%
  Complexity      9104     9104
=============================================
  Files           1125     1125
  Lines          80255    80248       -7
  Branches       10453    10453
=============================================
- Hits           34072    34067       -5
- Misses         41884    41895      +11
+ Partials        4299     4286      -13

Impacted Files
Coverage Δ

...cketmq/broker/schedule/ScheduleMessageService.java
58.57% <100.00%> (+0.02%)
:arrow_up:

... and 26 files with indirect coverage changes
:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more


Invalid timestamp seems incorrect

Have just been sending a lot of messages testing the PoW stuff and noticed that 406 errors are happening when they shouldn't be
The problem seems to happen if my client hits a break point before sending a message and doesn't send it for a bunch of seconds
Some logic in the validateTimestamp or whatever is prob busted

Closing until happens again


Window Background Not Following The Theme

[ ] What you've tried (include code snippets, or whole files as necessary):
I'm extending AppIntro/AppIntro2 and using my own custom fragment (like Sample Slide). In the manifest I have set the theme for my AppIntro activity to Theme.AppCompat.Light.DarkActionBar (I do want the action bar).
[ ] What you thought should happen:
I expect the window background of my slide fragment to be the default light colour because of the theme I have set.
[ ] What actually happened:
Window background of the slide fragment is dark.
What am I missing or doing wrong?

Can you post the xml for the fragment layout, along with your app theme xml (or style if you have it on there).
And finally, can you post your implementation of your Fragment.

Fragment layout
<RelativeLayout
    xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent">

    <TextView
        android:text="hello world"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
</RelativeLayout>

AppTheme xml
<style name="AppTheme" parent="Theme.AppCompat.Light.DarkActionBar">
    <!-- Customize your theme here. -->
    <item name="colorPrimary">@color/colorPrimary</item>
    <item name="colorPrimaryDark">@color/colorPrimaryDark</item>
    <item name="colorAccent">@color/colorAccent</item>
</style>

Setting theme in Manifest
....
<activity
    android:name=".MyAppIntro"
    android:theme="@style/AppTheme" />
....

Fragment is just the one I get from the wizard in Android studio. The code is very similar to the SampleSlide code.

Anyone else got a similar issue?

I'm having related to this issue. I've black color in all my slides. I added "android.R.color.holo_red_dark" to newInstance method of last parameter(background_color) but I've only the black color. Even I tried with my own fragments too that one also having the same black color background. Anyone help me please... Thanks in advance
 addSlide(AppIntroFragment.newInstance("Octocat","Meow.... It is your friend",R.drawable.octocat,android.R.color.holo_red_dark));
AppTheme
<style name="AppTheme.NoActionBar" parent="Theme.AppCompat.Light.DarkActionBar">         <item name="windowActionBar">false</item>         <item name="windowNoTitle">true</item>         <item name="android:windowFullscreen">true</item>     </style>

@janakagamini, @vinothvino42
 android:background="@color/yourcolor" in your fragment's root layout.

I had the similar issue. Then I passed the background color argument like
addSlide(AppIntroFragment.newInstance("Join with Music party", "Easy way to find local music parties", R.drawable.join_party, Color.parseColor("#000000")));
This worked.

@KChernenko Yeah, I know this is what I am doing now. I'm just pointing out that it doesn't follow the theme. Or perhaps it doesn't follow the theme by design?

@janakagamini After further digging and research of stack issues -- I think this is the behavior the library should keep.
If a library defines styles that conflict on app level -- then you'll encounter merger issues that not many know how to fix on their own -- so instead, allow library users to implement their own styles.

Hi @avluis thanks for replying! I have since worked around this problem by setting the colour I want as the background of my root layout.

Good to know, if you feel this issue has been addressed -- go ahead and close it for now.


Integrate cargo-hack in CI

The way cargo resolves dependencies prevents us to run cargo check with --no-default-features for all crates. To circumvent this, we need an alternative. See https://github.com/rust-lang/cargo/issues/12470 for more details.
Definition of Done
Integrate cargo-hack into the CI and resolve potential compiler errors

Done in https://github.com/NomicFoundation/hardhat/pull/4380


update titles for multiple published posts

@rominirani thanks for all of your contributions. Could you please sign the Google CLA? Use the same gmail address you use with GitHub

Already updated.


[CodeCamp #1493] Add CocoPanopticMetric to MMEval

Motivation
Add the CocoPanopticMetric from mmdet to MMEval.
Modification

No need to import mmcv.
The parameters of the metric include panoptic image arrays and json files.
Support json file in the form of list, dictionary and path to it.

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

Hi, Thanks for your PR, do you have any plan to create a PR in mmdet?

Hi, Thanks for your PR, do you have any plan to create a PR in mmdet?

Yes. I've written a part of it and I'm testing it.

Hi @Even-ok ！We are grateful for your efforts in helping improve this open-source project during your personal time.
Welcome to join OpenMMLab Special Interest Group (SIG) private channel on Discord, where you can share your experiences, ideas, and build connections with like-minded peers. To join the SIG channel, simply message moderator— OpenMMLab on Discord or briefly share your open-source contributions in the #introductions channel and we will assist you. Look forward to seeing you there! Join us ：https://discord.gg/UjgXkPWNqA
If you have a WeChat account，welcome to join our community on WeChat. You can add our assistant ：openmmlabwx. Please add "mmsig + Github ID" as a remark when adding friends：）
Thank you again for your contribution❤


OneDrive on the roadmap?

From @SQL-MisterMagoo on Tuesday, October 13, 2015 2:54:28 AM
Hi, Just wondering if anyone is planning on implementing OneDrive webhooks into this project?
Thanks
Copied from original issue: aspnet/WebHooks#15

From @garora on Saturday, October 17, 2015 12:21:59 PM
Earlier, I supposed to give a try with reference to onedrive. But, due to other priorities, I wouldn't able to start. I can give a try but not sure when. @HenrikFrystykNielsen - do you have any plans? I would love to take this initiative. Unfortunately, currently I have other priorities.

From @HenrikFrystykNielsen on Tuesday, October 27, 2015 7:47:18 AM
Does OneDrive support WebHooks?

From @garora on Tuesday, October 27, 2015 7:55:35 AM
After visiting documentation of OneDrive, did not find relevant information.

From @SQL-MisterMagoo on Tuesday, October 27, 2015 8:45:20 AM
It's in preview still, but the details are available here : Service-to-service notifications (preview)
I was just wondering if it would be on the roadmap, as I would like to try it out, and this seems like the right tool for the job.
If it's not on the roadmap, that's all I need to know.

From @HenrikFrystykNielsen on Saturday, October 31, 2015 8:34:32 AM
We'd definitely be happy to have support for it! if you'd like to help submitting a PR then it will likely go faster ;)

From @garora on Friday, June 3, 2016 2:15:00 AM
Documentation is moved here WebHooks.
@SQL-MisterMagoo , @HenrikFrystykNielsen  - is anyone working on this?

From @HenrikFrystykNielsen on Saturday, June 4, 2016 12:46:45 PM
Not at the moment...

From @garora on Saturday, June 4, 2016 11:14:01 PM
I'd try it after completing one, I am working on.

From @HenrikFrystykNielsen on Tuesday, August 2, 2016 2:00:38 PM
awesome!

From @garora on Monday, January 16, 2017 1:42:07 AM
Putting it on hold due to Issue#92

From @garora on Monday, August 21, 2017 4:44:22 AM
Still waiting for a solution for #92

Oops, bot misbehaving again... Ignore this issue.


Feature request deepRequired

Hello,
I am at a point where I need to generate sample data from a schema and for that I need to have all possible values of the schema tree.
I notice there is a .deepPartial method but I need the opposite so the .deepRequired method.
Any reason this method does not exist ?
Thanks

Up

Last up ?

+1 for this feature. I need to define two schemas, one is for optional input, and the other for the resolved result. (you can think of this case of request-response DTO definition)

Would also be interested in this feature. We have some pretty complex zod schemas where some deep properties are optional. Ran into a use case where I'd like to make those deep properties required so that we can pick from them (can't pick from ZodOptional), without changing the original schema. Side note: would also really like to see deepPick implemented.

I took the deepPartial code and changed it around to function as deepRequired instead. Here's a Gist with the code, feel free to use it https://gist.github.com/dimfeld/02cd41dad63c095291c15619484ab170
If I find some time next week I'll can try opening a PR for real.


Quick fix for IPython, can not work on class object.

Thank you for creating this wonderful package.
This make it run on IPython environment except class object.
Unfortunately, it seems there is an issue to get object source https://github.com/ipython/ipython/issues/11249
Feel free to ignore this.

Thanks for the pull request, I'll take a look :-)


Fixes #141 - make log level verbose for 'Create' logs

We should also add a total message like other plugins do, i.e "X files created" in the end

Could this be reverted and replaced with a config so we can either enable log or not since using verbose causes us to see all our other scripts to show there full content.


Logitech MX Master 3 - Can't assign Macro

When using Logitech MX Master 3 I can't map to macro, the default map for the "thumb" button is marked as UNKNOWN (it seems to be SUPER+TAB).
I would love to map this button to take screenshot as I do on Mac OS X.
I tried to map to other custom buttons, like (button 6) for example but it doesn't work also.
All times it gave an error on piper the message I receive from ratbagd --verbose is:
ratbag error:     HID++ error from the device (3): ERR_INVALID_ARGUMENT (02)
ratbag error: Error while writing profile: 'Protocol error' (-71)
ratbag error: hidpp20: failed to update button (-71)
ratbagd error: error committing device (-1000)

Anyway, it worked to map the "thumb" button to back, left, right and middle click.
Built from git source:
libratbag: 8c295479d887d379865fd0596a9d8fc5745f5ed8
piper: 4fab9ccdccbac296e866f9af1dd46a8979f192d1
Ubuntu 20.04

I just updated from git master yesterday, head up on  8c29547.
Just to update this:
ratbagctl chanting-squirrel button 7 action set button 7

or any other button number, doesn't give an error but also does nothing (the button keeps mapped from what I mapped previously on Piper).

That is weird, can you start ratbagd with verbosity and paste the logs?
ratbagd --verbose=raw

With piper
Works:
Left mouse button
Right mouse button
Middle mouse button
Backward
Forward
Ratchet Mode
Fails:
Button 5 click Log: https://pastebin.com/54P0aDJ2
Button 6 click
Button 7 click
Double click
Wheel left
Wheel right
Wheel up
Wheel down
Cycle resolution up
Cycle resolution down
Resolution up
Resolution down
Resolution switch
Default Resolution
Cycle Profile up
Cycle Profile down
Profile up
Profile down
Second Mode
Battery Level
Send keystroke
With ratbagctl
The ones that fails with piper gives no error on ratbagctl screen, but
prints similar errors on ratbagd output
The ones that works with piper dgives no error at all, but doesn't works
Mapping to button:
builddir/ratbagctl list
chanting-squirrel:   Logitech MX Master 3

$ builddir/ratbagctl chanting-squirrel button 7 action get
Button: 7 is mapped to UNKNOWN
$ builddir/ratbagctl chanting-squirrel button 7 action set button 1

$ builddir/ratbagctl chanting-squirrel button 7 action get
Button: 7 is mapped to 'button 1'

but doesn't work - LOG: https://pastebin.com/78Nf0z99
I think it might be two different issues.
If you need any other logs, just tell me.

same problem for Logitech MX Master 2S with v0.13.
ratbagctl cheering-viscacha button 5 action set macro KEY_A
Error: assigning a macro is not supported on this device

It's not the same problem. Mapping the key to A id not supported:

Just as information for anyone that want to achieve this in the meantime: I've been using this other project:  https://github.com/PixlOne/logiops to achieve custom mapping on my MX Master 3 thumb button. Is not as nice and user friendly as libratbag + piper, but I could fully configure all my mouse buttons the way I wanted.

@FFY00 But what are the supported macros then? Mapping it to a simple keystroke seems like one of the easiest macros I could make...

That is weird, can you start ratbagd with verbosity and paste the logs?
ratbagd --verbose=raw

I did not see an actual log attached with the daemon startup. So here it is from my machine https://pastebin.com/sDeLJXcR
Running Linux 5.7.7 with Ubuntu 20.04
libratbag/ratbagd (git)
Piper (git)
Facing the same issue reported here (just received the mouse today):  can not map a macro to the "thumb" on the Logitech MX Master 3.
Beyond the "thumb" button, it seems I can not assign a keystroke to any of these buttons: button 3, 4 or 5.
Looked for kernel messages but it does not appear hidraw/others put anything useful there by default.
The only visible errors are outputted  by ratbag itself:
ratbag error: hidpp20: failed to update report rate (-95)
ratbagd error: error committing device (-1000)
@FFY00 what would be required to troubleshoot this further please?

@BjoernDaase sorry for the delay. Unfortunately I don't know, and I don't have documentation here. I know how this works in the gaming devices, but the productivity line does it in a different way.
@glunardi that is good. It is not the same issue, in your case the report rate is failing to be set. Can you open a different issue so that we can debug there?
I am pretty sure I fixed this, can go through the recent releases and see if any of them work?

Same problem here, updated libratbag to 0.15 from master but still can't change the thumb button on my MX Master 3 :(

Seems like I have the same issue but with a MX Anywhere 2. Some macros work, the same that the author of this issue. I use libratbag 0.15-1 from Arch repo, 5.9.8-arch1-1 kernel. Using Unifying receiver.
Do I need to open a different issue ? Here is my logs :

Same using libratbag 0.16 on a MX Master 3.
Piper reports "Something went wrong. The devices has been reset to the previous state" (but without resetting the setting on the UI, I guess I got to report a bug there too). This happens as soon as I click Apply in the Macro selection screen.

same here with MX Master 3 and ratbag 0.16 on Arch

Same here. I'm on Fedora 36, with ratbagd 0.16 and using piper as a flatpak

Same here with libratbag 0.16 MX ergo trackball. Piper throws Something went wrong. The devices has been reset to the previous state.

same here with MX Master 3 and ratbag 0.16 on Arch

same MX Master3 and ratbag 0.16 on Xubuntu

One more for MX Master3. Digging around for solutions as it's the last thing keeping me dual booting right now lol

Having the same issue on ratbagd version 0.20 (API version 2)

Linux mint, official repo, Logitech MX Master 3S - not working.


Version 2.1.3 opens settings instead of the app

Hi, thank you for the great app.
Unfortunately since the last update instead of seeing the app I see "app info" screen for the app. I know it sounds a little silly but I have no idea what could be causing it. I tried few different ways to open it: home screen, app list, via F-Droid, all the same. Tried force-stopping it too. I see how for a split second a normal activity opens but then it opens settings instead.
Is there any advice you can give?
version 2.1.3 from F-Droid
Android 11, Fairphone 3

The exact same thing happens to me with the latest version, and I can't reinstall the previous version as it says "not supported".
Nor can I reinstall it since that would imply losing my notes.
I have a Xiaomi POCO X3 Pro with Android 12 (MIUI 13)

Do you mean the app settings, or system settings?

Have you downloaded the app from Google Play Store or GitHub?

This is what I see
I downloaded from F-Droid

This issue is known, and an update has been published two days ago. Please update your app.

2.1.3 is the latest release and that's what I'm using?

The fix is published with the same release version. Play Store should let you update the app. However, if you have downloaded the app from F-Droid, and F-Droid isn't showing the update, then you can download the update from the Releases page on GitHub.

Hi, I'm afraid this won't work, F-Droid builds and signs the app on their own.
It came to my attention that I got it not from an official F-Droid repo but from IzzyOnDroid one. I am not sure how it is set up to update but it might be by tag. I will ask the owner of the repo about the update. Would you be open to making a release tag with a patch version?

Yeah, I haven't published the app to F-Droid myself. I can make a new tag if you want, no problem at all!

It will probably also require version code bump, I'll try to figure it out, thank you for the responses!

Have you tried downloading the updated APK?

I asked the author and here's the response:
https://mastodon.technology/@IzzyOnDroid/108723826744379377
Turns out this repo doesn't build the app, just fetches the APK so I could just update from the releases page which I did and it works. Sorry for the noise!

No problem! I apologize for the inconvenience.
@extropista Could you please update the app from the Releases page, and let me know if it works for you?

Hello @alialbaali thanks for your reply, ..after following the thread of the conversation, I have reinstalled the app by downloading the file in the github releases, which has worked for me too.
(I already had the same version installed, 2.1.3, but I guess it happened when updating from Droid-ify with the Izzyondroid repository)
Greetings and thanks for everything!

Glad it is working for you!

It will probably also require version code bump

Without that, Android won't recognize it as update, right. But at least as long as the versionCode was not decreased, it still allows a manually forced update (e.g. adb install -r app.apk).
As pointed out at Mastodon, my updater looks for new tags – in your case: the tag name should indicate a higher versionName and should match that of the attached APK, then it will be picked up within 24h.


[BENCH] DFS traversal

Benchmark Depth First Search Traversal
The goal of this issues is to add a benchmark for the DFS traveral algorithm. The algorithm implementation is located under include/graaflib/algorithm/graph_traversal/depth_first_search.h. Benchmarks are vital to our library and allow us to measure the impact of future performance improvements.
We use the Google benchmark framework. For inspiration, please take a look at the existing benchmarks in /perf.
The benchmark should be added under /perf in a directory which resembles the file structure of the original algorithm. i.e. if the algorithm is implemented in include/graaflib/algorithm/coloring/greedy_graph_coloring.h then the benchmark should be added to perf/graaflib/algorithm/coloring/greedy_graph_coloring_benchmark.cpp.
The benchmark should measure the runtime performance of the algorithm for increasing input sizes.
Running Benchmarks
If you IDE has the necessary integrations for it, all benchmarks can be run in the IDE from the perf/graaflib/benchmark.cpp file.
Otherwise, we can run the benchmarks from the command line:
# run all benchmarks
cd build/perf && ./Graaf_perf

To run an individual benchmark:
./Graaf_perf --benchmark_filter=YOUR_BENCHMARK_NAME

For more options, pass the --help flag.
Definition of Done

[ ] A benchmark is added for the algorithm
[ ] Benchmark results (copy past of the output is fine) is added to the PR

Hey @bobluppes, assign to me please! I wanna work on this after the clique PR.

@bobluppes see PR #164