How to encrypt and decrypt data using PHP programming language

We ask that you please be patient as our team works through approving and publishing all articles/tutorials in a timely manner. Allow 1-3 days for a topic to be reviewed and/or approved - allow 3-7 days for an articles to be reviewed and/or published.
Be sure to visit our Resources Page for tools, resources, and example articles to go over.
NOTE: (Please ensure that you have only one open issue + linked pull request at a time. This will ensure that we complete the article in a timely manner from inception to publishing.)
NOTE: We tend to stray away or tend not to publish reviews/comparisons of commercial product offerings.
We're looking for the first 2-3 paragraphs of the article that appropriately summarize what your article will be about.
Proposed title of article
Your title should be descriptive of the article/tutorial. Be Specific. Use keyword research to gain your article higher ranking.
How to encrypt and decrypt data using PHP programming language
Introduction paragraph (2-3 paragraphs):
Please write the Introductory paragraph(s), that would be included in your article. We will use this writing snippet to help us assess overall quality before approval. We're looking for the first 2-3 paragraphs of the article that appropriately summarize what your article will be about.
Take your time and write the content as you would intend to get it published.
during data transmission information is much prone to attack and access from unauthorised personnel whom can use it for malicious purposes.
as we all know encryption is vital process to secure sensitive data in any software application.
encryption is the process of encoding information.
this process converts the original representation of the information known as plaintext into alternative form known as ciphertext.
only authorized parties can decipher a ciphertext back to plaintext and access the original information.
through encryption we protect data in transit and stored in database from unauthorised access.
it also maintains the confidentiality of data during transmision between sender and recipient.
decryption is a process of converting encoded data into a form that is readble and understood by human and computers through the use of a secret key.
works by applying the opposite conversion algorithm used to encrypt data.
through PHP encryption and decryption data is well protected from interception by hackers.
the following feautures of PHP will be used for encryption and decryption:
openssl encrypt
openssl decrypt
in this article will be using PHP programming language  for data encryption and decryption.
will also use different PHP encryption algorithm available in market.
Key takeaways:
What are the 3-5 most important things the reader should understand or be able to do after reading this article? Use this area to get your ideas down on the bulk of your article or tutorial.
1.why we need to encrypt  data during transmission and need for data decryption.
2.how to use PHP simple encryption and decryption class.
3.PHP uses of openssl_encrypt for encryption techniques.
4.PHP use of openssl_decrypt for decryption technique.
5.use of different PHP encryption algorithm such as hashing, secret key encryption e.t.c.
6.PHP code snippets of encryption algorithm used and its output.
7.decryption of encrypted data using PHP
References:
Please list links to any published content/research that you intend to use to support/guide this article.
N/A
Templates to use as guides

How To Guide Template
Software Review Template
Tutorial Template

Good afternoon and thank you for submitting your topic suggestion. @Wakanyaben254
Your topic form has been entered into our queue and should be reviewed (for approval) as soon as a content moderator is finished reviewing the ones in the queue before it.


Toggl Button Challenge Slack

This is my submission to the Toggl Button Challenge with slack integration. This shows the button in each channels header, but not in the individual chat's. If a project exist with the same name as the team name in slack, that project will be the selected project.

Just for information. The pull request will be evaluated and merged after the end of the toggl developer challenge (https://toggl.com/toggl-developer-challenge) on February 14th. Thank you for your contribution.

Merged with rebase. Thank you for your contribution.


show -dimm columns

ipmctl show -dimm shows these columns:
DimmID  Capacity        HealthState     ActionRequired  LockState       FWVersion
0x0011  502.5 GiB       Healthy         0               Disabled        <IP_ADDRESS>0

This command should show the most important values.
Suggestions:

Drop LockState (the security frozen status). That's not important enough.
Drop FWVersion. That's important, but more of a maintenance item.
Shorten ActionRequired column header to Events (saving space), change the values from "1" and "0" to "yes" and "no".  The recommended action is to fetch the events; this column name suggests that better.
Add MemoryCapacity, the amount of volatile memory being provided. Very important; Confirms that a goal configuration was really applied.
Add AppDirectCapacity, the amount of persistent memory being provided. Very important. Confirms that a goal configuration was really applied.
Add DimmUID, the name of the device. Used to identify the device everywhere, no matter where it happens to be (doesn't vary based on the DIMM slot or system, and is printed on the label)
The DimmID (the NFIT Device Handle) is a 32-bit field, so should show all 8 hex digits (not just 0x0001 like now).  Those will be non-zero in systems using node ID bits.  If printed with %04x, that'll mess up the column formatting, since there are only 2 spaces before the next column.

This could look like:
DimmID      DimmUID                Capacity   Volatile   Persistent  HealthState  Events
0x00000011  1234-56-1800-abcdabcd  502.5 GiB  500.0 GiB  0.0 GiB     Healthy      yes

Robert, this tool is not a part of PCM.


Question: How can I use Marker Image

How can I use icon from

URL ?
local resource using require('../images/pin.png')

I have tried these without any luck
<Marker
    ...
    icon={{
        path: 'https://example.com/mypin.png',
    }}

<Marker
    ...
    icon={{
        path: require('../mypin.png'),
    }}

Reference Document
https://github.com/HMS-Core/hms-react-native-plugin/blob/master/react-native-hms-map/README.md#bitmapdescriptor

Hi,
path: is for images that are in your android storage. Your app runs on your android device not on the PC so it doesn't recognize your development environement's resources.
Examples:
icon={{ path: "/sdcard/Download/icon.png",  }}

icon={{ path: "/storage/emulated/0/Download/icon.png",  }}

The best way to use custom images is that you add them in your android/app/src/main/assets folder.

You can call them via icon={{ asset: "ic_launcher.png", }}  // under assets folder.
Also, downloading images from the internet is not supported in native SDK right now.

Thanks for your response.
How about rendering Component as icon?

If you are asking doing this:
<Marker>
   <CustomIconView />
</Marker>

Unfortunately, it's not possible for now. Marker gets only InfoWindow as a child.

Got it!
I can't wait to see this new feature soon!


chore(release): Automated agent plugin release notes (v0.10.58)

Automated PR

PR made for testing release-notes pipeline


Added IMP graph algos

BFS
Cycle detection in undirected graph
Dijstras algo for single source shortest path
Travelling Salesman Problem

Tested & Working code !!


PHOENIX-2952 fix array_length return nagetive value

https://issues.apache.org/jira/browse/PHOENIX-2952

Already merged.


Add timestamp to authority signatures

We will eventually need to display timestamp on explorer, and it needs to support transactions that haven't made into checkpoints yet. I think the best way to do this is to add timestamp to authority signatures (at the time of signing), and then when there is a certificate, make the time median time of all authority signatures.
https://github.com/MystenLabs/sui/pull/1686 introduced AuthorityQuorumSignInfo that would make this much easier to do.
Next we just need to add a timestamp field to both authority sign info.
What do people think about this? @gdanezis @huitseeker @kchalkias

There are many ways to introduce timestamps to our protocol, all with their pros and cons.
a) user sets a timestamp or range as a tx input (or better sign it as metadata), and if validators receive it inside some threshold they also sign. Obviously this provides very relaxed guarantees.
b) validators add it to their fastX signatures (thus certified tx carries many timestamps, as each validator might pick a slightly different one). Cons: as quorum signatures are malleable and a cert can be of any combination of authsigs, again you cannot guarantee a single accurate  timestamp, thus we should play with median etc.
c) we add a single timestamp to each consensus block; thus, indirectly applied to all txs in the block. Obviously this timestamp introduces some latency compared to fastX certificates. This is what most BFT blockchains do.

What timestamp do you think we should display in an explorer for each transaction?

Some explorers also add "time received" from their backing full node. That's the easiest atm for UI purposes.

There is another issue with this solution: now each validator signs a different message and if we go with BLS aggregation, there is extra cost to aggregated-sig verification.

I was thinking that we shouldn't sign the timestamp, as it comes with the signature which is malleable.

The issue with introducing a timestamp, particularly within the signature is that inevitably people will interpret it as having high integrity. I do wonder if for this use case a local timestamp, at the time of reception of the tx or checkpoint is not what we need.
The bigger issue of a timestamp we can use within move is much more tricky.

The issue with introducing a timestamp, particularly within the signature is that inevitably people will interpret it as having high integrity.

Would it work if we name the field with something very explicit, like median_sign_time?

The issue with introducing a timestamp, particularly within the signature is that inevitably people will interpret it as having high integrity

I'm not so sure.
The drift tolerance for the block timestamp in Ethereum is 900 seconds, and in practice, discussions on the subject show people correctly interpret this value as malleable. Other docs clearly label their time as imprecise, and so far I haven't heard of this being a problem.

Should this be merged with https://github.com/MystenLabs/sui/issues/226?

Let me rename this issue. I think they are different. This issue is less about on-chain time, but more about what time an explorer can use for display.

Current approach: add timestamp on fullnode, the moment we see a tx from validator.

@longbowlu --> feel free to place the text for the tooltip in this thread and I'll create a PR implementing it.


[material-ui-icons] Add named export warning to README, fix migration…

… path.

[ ] PR has tests / docs demo, and is linted.
[ ] Commit and PR titles begin with [ComponentName], and are in imperative form: "[Component] Fix leaky abstraction".
[ ] Description explains the issue / use-case resolved, and auto-closes the related issue(s) (http://tr.im/vFqem).

@oliviertassinari Added this before I saw your changes. It's going to break what that's merged, so do you just want to add these changes to your PR?

I will resolve the conflicts


ocw-2022-session63

ID#11238 - Leverage Deep Learning Based Image Analysis at Scale
ID#11207 - Leveraging Data Science for Predictive Modeling
General requirements

Information in the workshop is adequate and updated
Code is correct and working
Links are correct
Make sure you enter the help email link in your manifest.json
Please make sure WMS URLs are updated as needed after your PR is approved

Checklist - Refer to the QA document for the complete list
Please confirm that the following is completed before submitting your PR

[x] All filenames are lower case (including folders, images, files, etc.)
[x] Filenames are descriptive
[x] Your workshop folder structure should be similar to the one used in the sample workshop (https://github.com/oracle-livelabs/common/tree/main/sample-livelabs-templates/sample-workshop)
[x] Are you using multiple versions (desktop/, sandbox/, tenancy/)? Make sure that each of them contains a manifest.json and an index.html
[x] Image references in markdown contain an alternate text

Just added both labs WMS ID in the PR title


best model not saving

train.py does not create a best model file in work directory. Can you please help me, or send me your best model file?
Thanks

thanks Mortezagolzan/Face-Anti-Spoofing for your response, this is my
official email, please send the model here.
typing my email here as well for your convenience.
@.***
Thanks again.
On Mon, Jan 17, 2022 at 1:19 PM Morteza Golzan @.***>
wrote:

train.py does not create a best model file in work directory. Can you
please help me, or send me your best model file? Thanks
Hi,
The work_dir in checkpoint_cfg property of the train.py file is the
where your best model save. As it is obvious here, the model is saves if it
gets better.
Anyway, please tell me your email address to send the best model file.
—
Reply to this email directly, view it on GitHub
https://github.com/mortezagolzan/Face-Anti-Spoofing/issues/2#issuecomment-1014251071,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AOC4XGU2GP5VEHX62QULTQDUWPGIZANCNFSM5MD6WQ3A
.
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.
You are receiving this because you authored the thread.Message ID:
@.***>

--
Tauseef Ahmed Memon

Unfortunately, your email is not readable in your comment. send me an email to<EMAIL_ADDRESS>


download file

As a user
I want to download my track,
So that I can listen it on my devices.

Acceptance criteria (ac)

[ ] there is a button
[ ] when the file button os clicked, the recording will be saved to the device
[ ]  the button is left from the progress bar

tasks

[ ]  create a button
[ ]  create a icon
[ ]  position the button on the page
[ ]  add event listener to to save the file on the device

Hey hey.
Maybe it would make more sense to have a list of my tracks and download them from there?
Do you save them locally first as it sounded in issue 6 with the save button you wanted to add there? I can't see a button like that in this drawing. Also, how would a user know if a track has already been saved, when the view is still the same as on the recording page?
In the ac, you wrote that the button is left of the progress bar, but in the picture it's on the right.
(sorry for all these questions, I hope it's not too much!)

Hi :) Thanks for your reply. Ah thats a good idea with the list. I will definitely change that and try to implement it.
In fact, I was wrong about the button. I'll adjust that too.

One thing I just thought of: Maybe you could also give each recorded trach an indivudual title, so that the user knows which one is which. Otherwise it looks very nice!


Use jku of token to retrieve keys

Use jku of token to rtetrieve token keys instead of building
url.
Aligns this library with other client libs.

Confilicts are solved, code is formatted.


Inverted condition.

Code is not using defined location (if any) for web socket.

Thanks again!

I released<EMAIL_ADDRESS>with your fixes.


Does CacheFactory Supports  Angular 1?

Does CacheFactory supports AngularJS application as well?  I am using both AngularJS and Angular2+ in my  project. Does this library will give leverage so that if I load data in cache by angular 2, will it be able to use it in angular 1 also?

This library has nothing to do with angular of any version (including angularjs). It is a generic javascript caching solution. There exists a project called $cacheFactory for angularjs but this project is not it.
You can share data between angular, angularjs, react, global functions on the window object, or whatever else you want; but it will be up to you to manage the sharing of the cache objects.
That said this project looks dead so you might want something else regardless.


Create / Edit Page - Unsecure Phone number field allows alpha characters

Secure Phone will not allow you to enter alpha characters or invalid characters but Unsecure Phone number field will accept alpha characters if Secure Phone has a valid number.  The same validation on Secure Phone should pertain to Unsecure Phone field too.

We have tried Chrome, FF, and IE and every strategy we can think of to get this thing to take alphabet characters and we can't reproduce this bug. Any other details you can give us?

Make sure you have a valid number in Secure Phone number field.
Both numbers are displayed with the red bar when you first come in (that is
a problem).  I enter a valid number in Secure Phone number field and that
turns off the red highlights for both fields.  I than enter "as" and no
error, no red highlights nothing happens.
I can reproduce it now and if you have access to the high side I can email
it to you.
On Tue, Apr 21, 2015 at 10:14 AM, Sarah Deighan<EMAIL_ADDRESS>wrote:

We have tried Chrome, FF, and IE and every strategy we can think of to get
this thing to take alphabet characters and we can't reproduce this bug. Any
other details you can give us?
—
Reply to this email directly or view it on GitHub
https://github.com/ozone-development/ozp-center/issues/206#issuecomment-94810537
.

I'm unable to reproduce this, It may have been a browser specific issue. @clarkjefcoat if you could test before calling it a non issue just to be sure.

@wski - I agree with you.  It should be marked resolved because it can't be reproduced.  (Chome, IE10, FF)

I also tried to reproduce this issue in production.  I can't duplicate the error.


Setup default Xcode.app

With multiple Xcode installations xcode files are open with one of the Xcode installed (the last one installed). Would be nice if xcode-install allow tweak Finder setup to set current Xcode.app as default Xcode for the system. This would be companion to xcode-select where path to the active developer directory is updated.

I'm not really sure how to actually do this, do you have any pointers?

$ mdls /Applications/Xcode-beta.app
(...)
kMDItemContentTypeTree         = (
    "com.apple.application-bundle",
    "com.apple.application",
    "public.executable",
    "com.apple.localizable-name-bundle",
    "com.apple.bundle",
    "public.directory",
    "public.item",
    "com.apple.package"
(...)

then maybe something with com.apple.LaunchServices ref: http://stackoverflow.com/a/9173115/699944 Haven't tried yet.

http://duti.org/ may be of use.


Add default check connection interval to be 10 seconds

The SDK didn't actually set the heartbeatIntervalMs for the gRPC check connection interval. This creates a default value of 10 seconds.

I'm not sure if this is a backoff situation -- this isn't retrying to establish a connection that might not exist, but rather the interval for checking that the connection does exist (which I think should be at a set interval rather than backing off). Unless I'm missing something?

I'm not sure if this is a backoff situation -- this isn't retrying to establish a connection that might not exist, but rather the interval for checking that the connection does exist (which I think should be at a set interval rather than backing off). Unless I'm missing something?

Yeah it's totally unrelated, sorry about that.


Added Linter and codespell github actions

Codespell and linter are now the only things that will be added by this pull request.


Invalidate loop

A first shoot to use an invalidate loop to redraw MapControl. For this, more things are moved into the common MapControl. The interval of the loop could be changed and the loop could be started and stopped, if the control vanishes from screen.
RefreshGraphics() now only set a flag.
Should replace #1187

Thanks for the update! Seems the CI is not quite happy with it, though:
Error: MapControl.cs(59,13): error CS0103: The name 'InternalInitialize' does not exist in the current context [/Users/runner/work/Mapsui/Mapsui/Mapsui.UI.Android/Mapsui.UI.Android.csproj]
Error: MapControl.cs(66,13): error CS0103: The name 'InternalInitialize' does not exist in the current context [/Users/runner/work/Mapsui/Mapsui/Mapsui.UI.Android/Mapsui.UI.Android.csproj]
Error: MapControl.cs(159,13): error CS0103: The name 'CommonPaintControl' does not exist in the current context [/Users/runner/work/Mapsui/Mapsui/Mapsui.UI.Android/Mapsui.UI.Android.csproj]

And also:
Mapsui.UI.Shared\MapControl.cs(278,21): error CS0111: Type 'MapControl' already defines a member called 'RefreshGraphics' with the same parameter types [D:\a\Mapsui\Mapsui\Mapsui.UI.Uwp\Mapsui.UI.Uwp.csproj]

So, I hope now, that the basic problems are solved.
Should it be possible in the XAML renderer of WPF to check for the PixelDensity, then some more code could be moved to CommonDrawControl().
In the current code, we don't distinguish between rendering and drawing. I think, that these are two different processes. But this is another issue.

I measured the Map Display time
From

To

so roughly a 400% performance improvement

Is it possible to see the column headers? Would be nice. Thank you.

Is it possible to get the number of calls for Mapsui.UI.Forms.MapControl.PaintSurface()? In the first image it is around 4919.

I updated the Performance comment

Thank you for the update.
That is strange. PaintSurface is called 13 times, while VisibleFeatureIterator is called 17 times. It seems, that the later one is also called from outside of the draw process. Have to look into this.

The additional Renderings come from a RenderToBitmap for Rasterizing.

For UpdateInterval, I'm missing a corresponding property in the MapView, otherwise it cannot be set when using a MapView (same for StartUpdates and StopUpdates).

I added these in my fork: https://github.com/janusw/Mapsui/commits/InvalidateLoop
Feel free to include it into this PR if you agree!

This is the answer on the last comment of @pauldendulk on #1092, which is closed.
Yes, we have to do this and that. And yes, there are hundreds of ways to improve the performance. But all this isn‘t discussed here.
This discussion is about reducing the redraws of the screen. It is about the fact, that in the moment each change of anything triggers a redraw of the screen. To get rid of this, we have to collect such redraws. But what is the correct way to do this? This approach waits a given amount of milliseconds and than, if there is something to draw, updates the screen.
So the question is:

Do we accept the need of such a collection?
Do we think, this approach could work?
Do we have a better approach for this?

This discussion is about reducing the redraws of the screen. It is about the fact, that in the moment each change of anything triggers a redraw of the screen. To get rid of this, we have to collect such redraws. But what is the correct way to do this? This approach waits a given amount of milliseconds and than, if there is something to draw, updates the screen.
So the question is:
1. Do we accept the need of such a collection?

From my point of view: Clearly YES!
The current scheme of redrawing the map everytime something changes simply does not scale well to complex scenarios. I can show you some where the changes here make a hell of a difference.
As an example, think of a map with a large number of pins (order of magnitude: hundreds or thousands), each of which is updated regularly. The current Mapsui fails badly on such a scenario (completely unusable, in particular on iOS), and this branch gives phenomenal improvements.
It seems @inforithmics has more relevant use cases.

2. Do we think, this approach could work?

It sounds very reasonable to me (in theory), and also in practice I can confirm that it greatly improves the responsiveness of the map for scenarios like the one described above. So yes, it works!

3. Do we have a better approach for this?

I don't have a much better idea right now. Having a configurable update interval is a good idea, and also the methods StartUpdates and StopUpdates are pretty useful, since they avoid wasting time on complex rendering if the map is not being displayed at all.
IMHO we should clearly go ahead with this. Further performance improvements can certainly be added later, but should not be subject of this PR.

BTW I assume that this PR nothing change at the FPS. It stops the rate at around 60 FPS (1000 ms/16 ms). But in the moment the FPS is limited by the time CommonDrawControl() needs. So, if we want to increase the FPS, we have to improve this part. This PR is only for reactiveness of the control.

And if I interpret the performance diagram of @inforithmics correct, then we could draw the map 13 times in a second, which leads to 13 FPS, correct?
Opposite to the last proposal, this hasn't a calculation for FPS. This should go in the next step into CommonDrawControl(). Should be the next PR.

Perhaps a ForceUpdate() could be helpful in scenarios, where you want to have a static map that should be redrawn after the initial configuration. It invalidates the screen once.

Perhaps a ForceUpdate() could be helpful in scenarios, where you want to have a static map that should be redrawn after the initial configuration. It invalidates the screen once.

Might possibly be useful and is trivial to implement. (I personally don't have a use case for this, though.)
@charlenni Could you please push your MapView changes (or merge mine from https://github.com/janusw/Mapsui/commit/bfe05ed93848d28947c34fdb3aec099dacd3578f), please?

@janusw Sorry for the delay. I'm still on vacation, and it isn't so easy to find a WiFi to upload the changes ;-)

@inforithmics If you now compare the performance of Mapsui, you should post the values for CommonDrawControl(). That are the times, that the control needs to draw its content. The Timeand Calls. First divided by the second will give a mean drawing time. Correct?

@janusw Sorry for the delay. I'm still on vacation, and it isn't so easy to find a WiFi to upload the changes ;-)

Nevermind (also: thanks!). You were posting all days, so I thought you were available. Enjoy your vacation!

All possible with the smartphone, but no PR changes.

 All this wrappers. Perhaps I should make the buttons as widgets.

 All this wrappers.

Certainly not very elegant. Wasn't there the idea to make MapView inherit from MapControl? Is that feasible? At least it would remove the need for these kinds of wrappers ...

Alright, we have two approvals here. @pauldendulk Do you have any further comments, or can I merge this?

Alright, we have two approvals here. @pauldendulk Do you have any further comments, or can I merge this?

I'll would like to have a look at it later this week.

I'll would like to have a look at it later this week.

Sure, take your time! (I have to admit that this issue is kind of urgent on my side, but of course I understand that you want to have a proper look at it.)
I guess this PR would be a good topic for a meeting on Friday (if we manage to have one this week) ...

Should it be possible in the XAML renderer of WPF to check for the PixelDensity, then some more code could be moved to CommonDrawControl().

There is a method to get the PixelDensity for WPF, so I think this should be possible. Or are you saying there is problem with this code? (see Mapsui.UI.Wpf.MapControl.GetPixelDensity())
It would be nice if it could be moved but perhaps an exception would occur on canvas.Scale(PixelDensity, PixelDensity); if it is zero.

Is there a general way in which this could be tested? Which project did you run? Is there a sample for this? Of not there should be (not part of this PR). Perhaps based on @bertt's sample.

How should this be made testable? BTW did we tested the way before?

To be correct: it is a visible control like a button. So how to test it? There is a tool from Xamarin, where you could test such things in many, many devices. But it costs.

I was merely asking which project I should run in order to see the performance problem so I could see how it is affected by the patch.
But on the topic. It is possible to unit test the individual parts. A invalidate loop could be in een separate class that could be tested. This is hard because it is a process over time, but it still possible. That fact that we did not have much tests in the past does not mean we should not add it in the future.

I will merge this PR.
I had some hesitation because the underlying problem is in the platform and was thinking about a solution that is closer to the platform.
All these platforms have a method called Invalidate or SetNeedsDisplay. These methods have names like these because they should not do a direct call to the OnPaint method. They should set a flag to like the '_refresh' flag in this PR. And ideally the platform implementation should also have a way to cap the framerate. Apparently this is not what is happening.
A solution I was thinking of was to fix this in the OnPaint method. Here we could use a Thread.Sleep to cap the framerate. In that way it is also easier to avoid overlap in calls (if the paint itself takes more than 16 ms).  A problem with Thread.Sleep is however that it locks the thread, which could be problematic as well. Not something I want to investigate now.
This PR solves a problem which probably hits many users. Thanks a lot for this contribution @charlenni!

You can use any sample that uses a MapControl on any platform. Because it is the same for all platforms. It seems that the most problems are on iOS. Use a sample that changes the map often.

If we make a solution for each platform, then the code is getting complicated. I thought about a method, so we could handle this problem on all platforms in the same way.
We could have made it on Android totally different. It seems, that Android could handle this by itself. But than we had to do it for the Forms or Maui platform also separate for each platform, what needs many #if in the code.
So I chooses a way, which is the same on all platforms, make it better on some of them and doesn't make it worth on others.
If we find better solution in the future, it isn't a problem to change this.
BTW it is problematic in the OnPaint event, because you don't know, if the redraw is called because something on the map has changed which could wait or if the control reappears on the screen and needs an immediate redraw.

If we make a solution for each platform, then the code is getting complicated. I thought about a method, so we could handle this problem on all platforms in the same way.

I imagined a generic solution with code placed in a shared class which is called from all implementations like the CommonDrawControl. But perhaps you are right and we should take platform differences into account. With that in mind I really would like to have more statistics.


tutlebot_gazebo building error

OS: Ubuntu 20.04 Xfce
ROS-Version: ROS2 Foxy
Hello,
when trying to set up the turtlebot3 simulation following the e-manual I keep running into the same problem over and over again:

--- stderr: turtlebot3_gazebo
CMake Error in CMakeLists.txt:
Imported target "dart" includes non-existent path
"/include"
in its INTERFACE_INCLUDE_DIRECTORIES.  Possible reasons include:
The path was deleted, renamed, or moved to another location.
An install or uninstall procedure did not complete successfully.
The installation package was faulty and references files it does not
provide.

Installing the turtlebot3_simulations package as a source file also didn't work for me.
Does anyone have an idea how to fix this issue?
Thank you.
Sincerly Nik

Hi @nikboy111.
Thank you for your questions.
Your installation package seems to be faulty
Please refer to PC Setup in the Quick Start Guide and reinstall.
Thank you.

Hi @ashekim,
thank you for your answer. I resinstalled ROS as well as all the packages mentioned in the Guide and still get the same error.

Hello. @nikboy111.
The problem seems to be caused by the dart package.
I am attaching an issue with symptoms similar to yours.
https://github.com/dartsim/dart/issues/1428
thank you.

Same issue as @nikboy111 here on Ubuntu 20.04. Any help welcome

Hi @shrinivas96 @p79281 @nikboy111 , does anyone of you managed to solve the problem?


2 README file

Why we need README.md and  README.rst?

The rst version shows code examples in reStructuredText!


Datagrid bug

[Title]: Datagrid import bug?
What package(s) are you using?
"@carbon/charts": "1.6.1",
"@carbon/charts-react": "1.6.1",
"@carbon/ibm-products": "2.0.0-rc.10",
"@carbon/react": "1.19.0",

[X] Carbon for IBM Products (or Carbon for IBM Cloud & Cognitive)
[ ] Legacy/CDAI
[ ] Legacy/Security

Detailed description

Describe in detail the issue you're having.

Trying to use a Datagrid component in my app and it breaks on runtime.
I created a fresh react app with these dependencies:
"dependencies": {
    "@carbon/charts": "1.6.1",
    "@carbon/ibm-products": "2.0.0-rc.10",
    "@carbon/react": "1.19.0",
    "react": "17.0.2",
    "react-dom": "17.0.2",
    "react-scripts": "5.0.1",
}

that builds with react-scripts and Datagrid also fails there with the same error.
I would expect Datagrid to work with the default webpack configuration, like the rest of  ibm-product components. Is there a special configuration required in order to make this work?

Is this issue related to a specific component?

Yes Datagrid

What did you expect to happen? What happened instead? What would you like to
see changed?

We were able to find what causes the issue, it is the way DataTable is imported.
For example, at DatagridContent.js file this is how DataTable is de-structured:
import { DataTable} from '@carbon/react';

const { TableContainer, Table } = DataTable; <--- TableContainer and Table are undefined

Making this change at all the places where DataTable is imported solves the issue.
import { TableContainer, Table} from '@carbon/react';

What browser are you working in?

Google chrome

What offering/product do you work on? Any pressing ship or release dates we
should be aware of?

I am working on Turbonomic ux-app project
Steps to reproduce the issue

Try to use a Datagrid component with any configuration with  react-scripts

Please create a reduced test case in CodeSandbox
https://codesandbox.io/s/github/carbon-design-system/ibm-cloud-cognitive/tree/main/examples/cloud-cognitive/codesandbox

Additional information
Console errors

Code example

Not the actual code but this code also throws the same error

Possible fix

Cloned the repo and replaced all the places where Datagrid is imported like this

Can anyone please take this at priority and give us a fix

Any target version of due date of when this might be fixed.  Currently my team is having to build a custom carbon-ibm-product library with proposed fix in order to be able to work with datagrid.

@vnikiten01 We are looking into a fix now.

@vnikiten01 @Billzg13 @rakeshmusturi
Upgrade your Carbon form IBM Products to  "@carbon/ibm-products": "^2.0.0-rc.17",.
I'm seeing the fixes on my end 🙂


Alphanumeric, number first issue

I've found another issue with sorting alphanumeric numbers with the number first. Add this to the unit-test.html file:
wrapTest(
	['5D','1A','2D','33A','5E','33K','33D','5S','2C','5C','5F','1D','2M'],
	['1A','1D','2C','2D','2M','5C','5D','5E','5F','5S','33A','33D','33K'],
	'alphanumeric - number first'
);

And the resulting sort will look like this:
['1D','1A','2D','2C','2M','5F','5D','5E','5C','5S','33D','33A','33K']

fixed as of @Mottie 's 0ea168cbe9d861266c7483a742d9b46ae202674a commit


chore(main): release 1.2.5

:robot: I have created a release beep boop
1.2.5 (2024-09-18)
Dependencies

update dependencies (#14) (499320b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/aimeerivers/watchthis-home-service/releases/tag/v1.2.5 :sunflower:


Draft: failing suspense test

Failing test case due to never resolving suspense boundary when using async atoms within molecules.
I added the failing test which is currently being discussed in #69 , currently this is just a draft pull request.
Type of change

[ ] Bug fix (non-breaking change that fixes an issue)
[ ] New feature (non-breaking change that adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[x] Documentation or Development tools (readme, specs, tests, code formatting)

Thanks for the PR @lazakrisz I'll take a look

Thanks for the PR @lazakrisz I'll take a look

No problem, I was also thinking whether this is an issue on bunshi's side or my app's side. I did a little bit more thorough explanation on the original issue: https://github.com/saasquatch/bunshi/issues/69#issuecomment-2155890100
Since then I tried finding another library / reference which would show some more integration with React's Suspense, but I was unable to find any pointers.
let me know if there's anything else I can help with 🙏  I might also try some further debugging


Experiment: Rotation constraint on mobile

My idea for #1351

@mapbakery Hi, you can try this and see how you feel.

That works for boxes and for text (I think) but for lines it doesn't work, there is no center to go to (and there is no rotate handler on lines)

Interesting approach.. It should work only on mobile that way.. right?

Yes, mobile only.

For lines, we'll need to implement rotation handle, anyway. So what about showing 2 handles on mobile?

Not sure about the lock icon.

Too many handles.. especially on mobile you want to have less..

That's true. Bot for lines this PR's solution wouldn't work :(.

Two handles would be fine for me, I don't have objections against the additional handle. Perhaps it's good to  use the handle with the lock icon for bigger rotation steps like 22.5 degrees

For @mapbakery use case, rotating lines won't help, I presume.

Yes, it would help

Is it hard to implement this solution: start rotating, and then place 2nd finger anywhere on the screen and rotate with 2 fingers to apply step-wise rotation. (would need a hint, for sure).

I'm positive to have all resize handles + rotation handle for 2-points lines/arrows. I know it's redundant, but it seems consistent.

Here's the thing: The original issue was about 2-point lines/arrow, which this PR doesn't solve. However, this PR approach works for all other shapes: rectangles, diamonds, ellipses, curved lines and texts. This feature is almost harmless, because you can not precisely rotate around the center anyway. If users are used to this feature, they can rotate widely and then narrows to the center to lock the angle. The center area could be a little bigger, but the current one is acceptable (and harmless). How do you guys think?

I agree, and it's useful for me!

@dwelle pls check this, whenever you get time

I'm not a fan of this. We shouldn't arbitrarily restrict what we can do on mobile. I would try to find a way to emulate the shift behavior that makes sense with mobile-friendly controls.

I think if one wants to have the angle constraint, they always want it. If that's the case, it would be better to provide a global config. Same would apply for grid constraint.

My original request was some kind of config setting. In my case, for transit map sketches) I will always use the same (45 degrees) angle constraints for everything. So I'm also happy with the 22.5 degrees steps, to keep it consistent with how it works when using the SHIFT key normal devices.

Let me close this. I agree to look for more explicit ways.


[#3211] fix bug

Codecov Report

Merging #3348 into master will increase coverage by 0.01%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master    #3348      +/-   ##
==========================================
+ Coverage   40.35%   40.36%   +0.01%
==========================================
  Files        2132     2132
  Lines       70025    70025
  Branches     9684     9684
==========================================
+ Hits        28259    28267       +8
+ Misses      39049    39039      -10
- Partials     2717     2719       +2

Impacted Files
Coverage Δ

...m/navercorp/pinpoint/plugin/netty/NettyPlugin.java
0% <0%> (ø)
:arrow_up:

...vercorp/pinpoint/rpc/packet/ServerClosePacket.java
52% <0%> (-16%)
:arrow_down:

...rp/pinpoint/rpc/server/PinpointServerAcceptor.java
79.86% <0%> (-4.87%)
:arrow_down:

...tor/metric/datasource/DefaultDataSourceMetric.java
75.75% <0%> (-3.04%)
:arrow_down:

...navercorp/pinpoint/rpc/common/SocketStateCode.java
82.92% <0%> (-2.44%)
:arrow_down:

...p/pinpoint/collector/receiver/udp/UDPReceiver.java
77.58% <0%> (-0.87%)
:arrow_down:

...m/navercorp/pinpoint/test/OrderedSpanRecorder.java
72.11% <0%> (ø)
:arrow_up:

...orp/pinpoint/rpc/server/DefaultPinpointServer.java
80.06% <0%> (+1.04%)
:arrow_up:

.../pinpoint/rpc/client/PinpointClientHandshaker.java
67.36% <0%> (+6.25%)
:arrow_up:

...pinpoint/profiler/instrument/AccessorAnalyzer.java
65.71% <0%> (+8.57%)
:arrow_up:

... and 1 more

Continue to review full report at Codecov.

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


debug: DEBUGGING FLAKE, DO NOT MERGE

Signed-off-by: Tom Hadlaw<EMAIL_ADDRESS>Please ensure your pull request adheres to the following guidelines:

[ ] For first time contributors, read Submitting a pull request
[ ] All code is covered by unit and/or runtime tests where feasible.
[ ] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[ ] All commits are signed off. See the section Developer’s Certificate of Origin
[ ] Provide a title or release-note blurb suitable for the release notes.
[ ] Thanks for contributing!

Fixes: #issue-number
<!-- Enter the release note text here if needed or remove this section! -->

/ci-aks

/ci-aks

/ci-aks

/test

/ci-aks

/ci-aks

/ci-aks


springfox-data-rest: Inconsistent model returned between findAll and search methods

When using the latest snapshot (2.8.1-SNAPSHOT) of springfox-data-rest, I found an inconsistent behavior between findAll and search methods. Both methods return List<Entity> and produce the same JSON HAL similar to this:
{   "_embedded" : {     "someResourceRel" : [ {entities}]   },   "_links" : {...
but springfox reports different models used.
For findAll method a Resources<<Entity>> is reported while for search method a Resources<<List<<Entity>>>> is reported.
In the contract for declaration-spring-data-rest.json it is also defined a similar example with ResourcesOfPerson and ResourcesOfListOfPerson while it should probably produce the same result.
Resources is already a collection wrapper... I doubt it wraps a collection of additional collections.
I would gladly contribute a PR if this proves to be a bug indeed. Any thoughts?

Seems like a duplicate of #1980

Not quite... #1879,  #1927 and #1980 report the different output format, i.e. the serialization of Resources which can almost be corrected using the Spring HATEOS object mapper (_halObjectMapper bean with Jackson2HalModule) but that is a different subject.
What this bug report is trying to address is the wrong content type inside the Resources instance.
The EntityFindAllExtractor @ line 71 resolves the return type using the RepositoryMetadata.getReturnedDomainClass method which extracts the domain type from the given collection.
resolver.resolve(Resources.class, repository.getReturnedDomainClass(handler.getMethod())));
On the other hand, EntitySearchExtractor @ line 82 resolves the return type directly from the method signature, thus presenting the return type as Resources<Collection<Entity>> instead of just Resources<Entity>
ResolvedType returnType = methodResolver.methodReturnType(handler);
if (Collections.isContainerType(returnType)) {
  return resolver.resolve(Resources.class, returnType);
}

@iles-2e you are right thats a good catch, it should be
ResolvedType returnType = methodResolver.methodReturnType(handler);
if (Collections.isContainerType(returnType)) {
      return resolver.resolve(Resources.class, Collections.collectionElementType(returnType));
}

Yeah PR's are most welcome!


onNextMonth and onPrevMonth are required

Sounds like there's an obvious default action for onNextMonth and onPrevMonth: to show the next or previous month respectively.
Would be great if they were not required.

Great idea. I added an onChangeMonth for this in v1.2.0.

@Hanse Here, this would make all the props optional. So e.g., just <Calendar /> should work for the common use case. By default, the month buttons would scroll calendar to previous and next month.
Would you accept it?


Vendoring libnetwork to fix #17526 & #17527

This PR fixes #17526

Ungraceful docker daemon restart results in container with multiple networks losing connectivity
if a container is connected to multiple networks, the config is not saved to the disk immediately. It was handled when the container is brought down (either during stop or daemon is gracefully brought down).
When the daemon is ungracefully restarted, the configs goes missing. The fix is to write the config to the disk when the configuration succeeds. Also fixed a corresponding issue in libnetwork where during ungraceful restarts, the newly connected endpoints using network connect is not honored (because the sandbox was updated later to the controller DB).

Also fixes #17527

Fixed a race in host sandbox creation

LGTM

LGTM

it needs a rebase :pensive:

dammit needs a rebase

Ahh needs a rebase somehow @mavenugo sorry :(

@tiborvass @abronan @calavera rebased and updated to the latest libnetwork that pulls in the fix for #17527

Thanks @madhu. Still LGTM.

LGTM, let's wait until those builds are :green_heart: before merging

I'm pretty sure these changes only affect linux, mostly because the changes are in container_unix.go.
It looks like the windows build is stuck for an unrelated reason, so I'm going to merge this PR.


added Unenroll button

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

:exclamation: No coverage uploaded for pull request base (main@a613769). Click here to learn what that means.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #83   +/-   ##
=======================================
  Coverage        ?   68.62%
=======================================
  Files           ?        2
  Lines           ?      306
  Branches        ?        0
=======================================
  Hits            ?      210
  Misses          ?       96
  Partials        ?        0

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


Crash with Requiem installed

NPE in ModelOverride code: crash-2021-05-17_18.27.53-client.txt
I believe it may be related to this mixin, which dynamically adds a new override to enchanted books: ModelLoaderMixin.java
Not sure what it is exactly though, there is nothing obvious in Chime's code that could trip.

@Pyrofab I ran in to this issue too. It was causing quite the headache. I now have to either remove requiem or search through a bunch of mods to find which one has chime bundled...

That's rather unexpected now, as requiem depends on Chime. Could you share your crash report ?

That's rather unexpected now, as requiem depends on Chime. Could you share your crash report ?

I might have an older version of Requiem... I’m going to update and see if it still crahses. The crash was an NPE when loading the models of some items in the inventory.

Yeah, this is Requiem's fault. The latest version fixes it (and also bundles chime)!

@Pyrofab after I updated, is fixed the issue. I think it’s time to close this one.

Right, I guess it's not necessary to fix this issue anymore. Would still have been nice to know what went wrong, but there's a bunch of more exciting things to do with our time.


Fix the spacing in align_equispaced when there are multiple channels

Summary
Fixes #5413
Details and comments
align_equispaced relocated the sub-schedules with incorrect spacing when the sub-schedules were across multiple channels. This PR changes the calculation so that align_equispaced uses the total duration of all the sub-schedules rather than the duration of the schedule when determining the spacing between the sub-schedule.
If the total duration is longer than the given duration, then align_equispaced will return the schedule unchanged.

 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.msuwama seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


Possible tag conflict with Farmer's Respite

Using CoffeeDelight-Fabric-1.19.2-1.3Fix2 and Farmersrespite-Fabric-2.2.4 together and just found a possible conflict with the cutting board use. Farmer's Respite has coffee berries that you're supposed to cut with a knife and cutting board as well. The Coffee Delight version of coffee berries has the correct tag in the coding but in game, there is no item you can use to cut the berries into raw beans.

Thanks for report, I will fix it!


Update dependencies

This repo seems to be having trouble installing Microsoft.NETCore.App, so I've updated the BuildTools to see if that fixes it.

It doesn't, closing.


Binary Tree

I added Binary Tree implementation. It uses Generics and can be used right-away as key-value data structure.
Example:
//                   key, value ( payload )

var mbt = BinaryTree<Int,String>()

for i in [10,8,9,24,22,26]{
    mbt[i] = "\(i)_test"
}

mbt.delete(10)
print("\(mbt.root!.rightchild!.rightchild!.key) -> \(mbt.root!.rightchild!.rightchild!.value)")
mbt.delete(26)
print(mbt.root!.rightchild!.rightchild?.value)

Thanks! Someone else is already providing a binary search tree but their implementation uses an enum and doesn't have all your functionality.
Once their version is added to the repo, could you add your code in that same folder? And also write an explanation of how your version works? That way readers can compare the different approaches, that would be great.

Sure, I will do that. Should i close this pull request ?

Yeah, I'll close it.

Please provide some additional information about the folder structure ? How should i put it there ? Should i add another readme ?

Ideally, I'd like to have all the explanations in a single README file. The existing version uses an enum. So you'd add a new section to the README explaining how your version is different, and explain any additional functionality that you added.
There is a small problem: the other binary search tree README isn't finished yet. So it's probably a bad idea to start messing with those files right now.
Here's an idea: Create a folder "Binary Search Tree 2" (in the top-level directory) and add your stuff in there. Then at some point I will merge the READMEs.
How does that sound?

Thanks a lot for the info. That sounds great, but i just rewrote it as an AVL tree. You can have look at it (here)[https://github.com/mitghi/swift-algorithm-club/tree/master/AVL Tree]. I am completing the README file and as soon as its finished, i make a pull request.
Thankd


Implement supportsDataTestProperties to disable tagless component assertion

Fixes #290

@bendemboski @Turbo87 Any update on this would be released?

@bendemboski @Turbo87 Thanks! I tried using this PR locally however it throws an error:
You cannot use `attributeBindings` on a tag-less component

I tried to fix it with https://github.com/simplabs/ember-test-selectors/pull/323 which essentially extends this PR, please have a look.
This is a blocker for native classes migration as it has babel 7 support as a prerequisite. I am currently working on ES6 classes codemods with @pzuraq
Your feedback is much appreciated!!
@scalvert @tomdale @rwjblue FYI

thanks for testing and coming up with a fix @ssutar, I'll review in details once more as soon as I find a bit of free time

closing in favor of #323


Wait for Istio proxies after Reset

See also #294

/lgtm
/approve

/retest

Let's merge it with the existing code duplication and clean it up aftwards in #369


How about a 'space out' or 'nice' mode

Currently if I call a limited function continuously, it will execute as much as fast as you want until the limit is hit, and then stop (or exception). Does it make sense to have a 'nice' option for the sleeping, whereby each call will have a pause, so the calls are forced to be equally spaced out during the time period?
It just seems nice on the service you're using (I'm using flickr, not twitter) to space out requests, and I think it's nicer to see continuous slower progress, rather than a burst of activity and then a long time doing nothing.

Closing as I think just setting e.g. max 1 request per 1 second achieves what I'm looking for.


Statusbar and Navigation Bar are not correct

I've got an application with a customized navigation bar, style and tint have been updated.
The "menu" pushes to the stack the MWPhotoBrower view. When it is popped off, the status bar and navigation bar of the view that pushed the MWPhotoBrowser is not restored to its original state.
The tableview on the calling view is now starting at 0,0 instead of after the statusbar + navigation bar.

Closed, found it in the closed issues already.
Should add functionality to remember the nav bar settings and restore them when leaving from MWPhotoBrowser

I'm hoping to implement that sometime soon.


TechDocs: As a consumer of docs in Backstage, I want to see breadcrumbs of the current documentation page I am looking at

TechDocs > MkDocs

These breadcrumps are hyperlinks to the page.
Possible Implementation
Can use this UI component https://material-ui.com/components/breadcrumbs/
On individual TechDocs page, we have access to path. We can use the path to show the breadcrumbs.

@katz95 do we have any design ready for this that we can attach to this issue?

Hi, I'd like to work on this.

@mahmoodhosseini - Awesome! Assigning this to you.


Handle double-writing the same manifest in the storage service

This is the cause of yesterday's failure: https://wellcome-ingest-inspector.glitch.me/ingests/35a445ac-4c36-438d-a01e-d4c44e44e7d9
As best I can tell, here's what happened:

The bag register wrote the manifest digitised/b17735270/v2, but for some reason didn't delete the message from the SQS queue or mark the ingest as succeeded. We see the following log messages in the bag register:
14:46:18.942 INFO w.s.b.client.AkkaBagTrackerClient - CREATED for POST to /bags with digitised/b17735270/v2
14:46:18.943 INFO w.s.b.client.AkkaBagTrackerClient - CREATED for POST to /bags with digitised/b17735798/v2
14:46:18.960 INFO w.s.s.m.IngestStepWorker$$anon$1 - Successful: location=PrimaryS3ReplicaLocation(s3://wellcomecollection-storage/digitised/b1773521x/v2), space=digitised, externalIdentifier=b1773521x, version=v2, ingestId=fd5a6b1f-721c-42b1-8593-5acbc93e2b2b, duration=PT2.203149S, durationSeconds=2
14:46:19.045 INFO w.s.s.m.IngestStepWorker$$anon$1 - Successful: location=PrimaryS3ReplicaLocation(s3://wellcomecollection-storage/digitised/b17735798/v2), space=digitised, externalIdentifier=b17735798, version=v2, ingestId=b0fa3bb0-da07-43f1-8167-3cdfd51a466d, duration=PT2.480333S, durationSeconds=2

but then nothing else from that task. It was doing a lot of work, and the bag register metrics show spikes in CPU around this time:

I wonder if the task crashed?

SQS waits for the queue visibility timeout, then makes the message visible again. The bag register picks it up, and discovers there's already a storage manifest with this version. It fails the bag.

Proposed changes:

[ ] Increase the CPU available to the bag register; currently it only has 0.5 vCPUs.
[ ] Improve the error logging from the storage libraries; seeing java.lang.Error: null is seriously unhelpful. You have to poke through a stack trace to find the actual error.
[ ] Support idempotent writes in VersionedStore (which is what the bag register uses). If I try to write the same object twice to the same version, that should be okay – writing different objects is the issue.
[ ] Add a test that the bag register is idempotent.

I think making idempotent writes acceptable is a good plan. 👍

Done.


[RFR][Feature][OSF-598]Utilize linkifyjs for URLifying comments

Purpose:
OSF-598
Update comments to automatically recognize URLs
Changes:
Update package.json to require linkifyjs
Update website/static/js/comment.js update var CommentModel to utilize linkifyjs for URL recognition.
Side effects
None
[#OSF-598]

LGTM. Dropping this in RTM 🚲

This is good to merge. Will merge for next improvements release.


update soak test onboarding links

Issue #, if available: N/A
Description of changes:

update doc links and pre-reqs with latest

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

/lgtm


Help getting text from images

I need help getting the text on the left hand side of these images. The blurred background changes colours
I already have a function that get the bitmap area with each of the five text line entries
here is my vb function (just the ocr part)
Function GetWordsFromBitmap(bm As Bitmap) As String
Using img As Pix = PixConverter.ToPix(bm)
Using scaledimg As Pix = img.Scale(8, 8)
Using p As Page = m_engine.Process(scaledimg, PageSegMode.Auto)
Return p.GetText()
End Using
End Using
End Using
End Function

dont understand what text you need recognize. red or blue area?

I need the text in the red area abd centre top

not a document image, you'd better do some preprocess before OCR like binarisation, crop etc
ImproveQuality

I figured some pre-processing was needed. What I want is for someone to possibly supply some sample code using the leptonica/tesseract implementation to get the result. I've tried the following tool (http://capture2text.sourceforge.net/) that uses tesseract and it gets results 98% of the time accurately. Can someone please provide a .net pre-processing sample?

+1 I'd also like to see some C# preprocessing examples. Right now I'm trying out https://stackoverflow.com/questions/30727343/fast-converting-bitmap-to-bitmapsource-wpf to do the cropping because I think some handles are missing in the c# bindings for leptonica like pixClipRectangle https://stackoverflow.com/questions/8605513/crop-pictures-with-leptonica-api-or-which-image-processing-lib-to-use


Add Carbon-Components

I would be sooooo grateful if someone could add IBMs  carbon components to this :)

Some helpful links:
https://github.com/carbon-design-system/carbon
and
https://github.com/carbon-design-system/carbon-components-svelte

It's unclear to me that there's enough here worth having an adder. The setup isn't that involved. But the setup instructions are very long and confusing. I think it'd be better just to simplify the setup instructions of that package instead. I sent https://github.com/carbon-design-system/carbon-components-svelte/pull/1989 as a start


Test org.gradle.api.reporting.components.ComponentReportIntegrationTest Fails When There Is No Installed Toolchain

In org.gradle.api.reporting.components.ComponentReportIntegrationTest tests informs the user when project has no components defined and shows details of multiple components fail when there is no installed toolchain.  Annotated those tests with @RequiresInstalledToolChain so that they are ignored instead of failing when there is no installed toolchain.

Thanks @LarryNorth


no package 'dropout_layer_norm'

in ops/layer_norm.py
import torch
from torch.nn import init

import dropout_layer_norm

I can't find any dropout_layer_norm package. is this an experimental feature?

cd csrc/layer_norm && pip install .
worked

Why isn't this installed by pip install . ?


Tell user if override is set in the environment after rustup default

Resolves: #1648
Given: an environment with stable, beta, and nightly installed, and an override for beta set on the current working directory.
Previous Behavior:
canis@latrans:~/rustup.rs$ rustup default nightly
info: using existing install for 'nightly-x86_64-unknown-linux-gnu'
info: default toolchain set to 'nightly-x86_64-unknown-linux-gnu'

  nightly-x86_64-unknown-linux-gnu unchanged - rustc 1.34.0-nightly (57d7cfc3c 2019-02-11)

New behavior:
canis@latrans:~/rustup.rs$ rustup default nightly
info: using existing install for 'nightly-x86_64-unknown-linux-gnu'
info: default toolchain set to 'nightly-x86_64-unknown-linux-gnu'

  nightly-x86_64-unknown-linux-gnu unchanged - rustc 1.34.0-nightly (57d7cfc3c 2019-02-11)

info: using override set by current environment
info: beta-x86_64-unknown-linux-gnu (directory override for '/home/canis/rustup.rs')

@kinnison No tests were affected, it seems, so I'll work on adding tests to cover this.

@kinnison Added a test.

@dwijnand Warning message simplified, commit squashed.


Angular 1.5 compatibility

I want to use your module with angular 1.5, does it work fine with this version ? If yes could your update your bower dependency ?

I'm using it successfully with 1.5.8

@phrough how to use this with 1.5.8, i try webpack ways or script tag but all failed.It always says "webBench.js:73509 Error: [$injector:unpr] Unknown provider: treeControlProvider <- treeControl
http://errors.angularjs.org/1.4.14/$injector/unpr?p0=treeControlProvider <- treeControl".plz help

I'm using it with 1.6.3, and it works!


Add Location Search to Services Page

A user shall be able to search a service with an address keyword such as country, city or district name.

Functionality has been added. A user can search for a service through the search bar with keywords related to the address field content.


jdec module

Hello,
I coded this module with helper functions for json marshal/unmarshal. To do things I could not with current json support.
Thanks!

Why not improve the json module instead? :)


⚠ Samsung has degraded performance

In 484b6b3, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2205 ms

Resolved: Samsung performance has improved in b44733a.


how to save graph

Export is currently not supported. Exporting SVG with external CSS seems a little bit tricky, but I will work on it in the upcoming version.

yes please add this functionality , as for large call graph I am not able to see the functions , so may be after saving i can zoom into it , i suppose

Version 0.2.0 is just released. You can now export call graphs by right-clicking on them and select "Export Call Graph" in the context menu.


STV "stats" field requires double form submit to filter

For some reason the state does not properly update on the first form submission of this field.
The row value is a Record<string, object>, where the key is a string steam64 and used for the filtering.
Terrible temporary workaround: #506
This does not occur on any other fields in the app...?

Fixed in #546


Document how to set up a fork for mdx-deck patching / development

I'm struggling to figure out how to successfully use a fork of mdx-deck. I realize this is due to my being a node.js newbie, but having spent about a day struggling to figure out whether to use npm or yarn, where in the source tree to do a yarn link (I'm guessing in packages/mdx-deck), why moew is missing, what it means that I have more than one workspace with name "@mdx-deck/webpack-html-plugin", etc., what I wish for is a HOWTO that documents the steps needed to use a fork of mdx-deck.
mdx-deck is awesome enough to get me to learn a new dev environment, and I hope to contribute something useful back once I get a working dev environment setup.

FWIW, my current status is that I believe I've gotten everything cleanly setup, but when I try yarn start it is either stuck, or is taking a really long time. Since there isn't much CPU or network activity, I'm assuming it's stuck.
$ npm start

><EMAIL_ADDRESS>start /Users/david/stuff/mdx-experiment
> mdx-deck deck.mdx

[mdx-deck] starting dev server

● [mdx-deck] █████████████████████████ building (20%) 90/91 modules 1 active
 babel-loader › deck.mdx

Possibly related to https://github.com/yarnpkg/yarn/issues/6801

Hi! Sounds like you've gotten part of the way there. I apologize that the CONTRIBUTING.md doc isn't fully up-to-date.
After cloning, you should be able to run yarn in the root directory. You'll need to install the Yarn CLI if you don't have it already. The repo uses Yarn workspaces, which should link the local packages together for local development.

To start the demo in docs/demo.mdx, run yarn start
To run the test suite, yarn test

From there, hopefully you can poke around the source code to see how things fit together, but feel free to ask any other questions, or let me know if you're still stuck with the dev setup

And be sure you're on Node version 8, 10, or 12 – sometimes that can be an issue with development setups like this one

Thanks! Yes, I have the demo and tests running. The pointer to yarn workspaces was helpful. But I'm still stuck.
What I haven't been able to figure out is how to use my local fork of mdx-deck in own my presentation. I've tried npm link and yarn link and yalc and some others things I've found, but keep having problems. Either I end up with build problems (that generally revolve around @mdx-deck/webpack-html-plugin), or I get an runtime error about hooks and multiple copies of react.
What's the recommended workflow here?

I don’t know what the best approach for what you’re doing is, but one common way of using a forked package is to rename whatever packages you changed with your username scope on npm and publish it; e.g. for myself, it could be renamed @jxnblk/mdx-deck
If you’re planning on opening a PR with changes, that can also be useful for testing things out outside of the repo and not being blocked by the PR

Closing due to inactivity, but feel free to open a new issue if you're still encountering issues


ListContainer deserialization error: podman returns null for Size

Crate version: 0.4.0
OS: AlmaLinux 8/9
Output of running podman version on the command line:

Client:       Podman Engine
Version:      4.2.0
API Version:  4.2.0
Go Version:   go1.17.12
Built:        Wed Aug 10 18:19:17 2022
OS/Arch:      linux/amd64

Podman swagger doesn't specify that "Size" can be null, but returns it.
Example output from curl:
[{"AutoRemove":false,"Command":["bash"],"Created":"2022-08-10T18:57:42.243852139-07:00","CreatedAt":"","Exited":false,"ExitedAt":-62135596800,"ExitCode":0,"Id":"ed289912f4fa9c2bdf1599d80e614397b05d947c1c7d0a55d6f261d31e512ddf","Image":"docker.io/library/ubuntu:latest","ImageID":"df5de72bdb3b711aba4eca685b1f42c722cc8a1837ed3fbd548a9282af2d836d","IsInfra":false,"Labels":null,"Mounts":[],"Names":["bash"],"Namespaces":{},"Networks":["podman"],"Pid":0,"Pod":"","PodName":"","Ports":null,"Size":null,"StartedAt":-62135596800,"State":"created","Status":""}]
Example rust code:
    let api = Podman::unix("/var/run/podman/podman.sock");

    let data = api.containers().list(
        &ContainerListOpts::builder()
            .all(true)
            .build(),
        ).await?;

Works if .size(true) used, fails with Error: invalid type: null, expected struct ContainerSize at line 1 column 497 otherwise.

There was a slight problem with generated models in 0.4.0 that made most struct fields required (non-optional) but it should be resolved as of c01d4b5db1302ca7163876debad180b9ec1954b7 . There is a new version 0.5.0 published that should contain those changes.


please tag releases

This library seems to be carefully versioned (great!) yet very few releases have been tagged.
Please tag releases.

Thanks for your interest in Marpa.
There is no public versioning, so what you see is for my own use, and I'm glad to hear it seems careful.  I did have in mind that I'd need to go public someday, which explains some of the care I took.
Do you have recommendations for tags, etc?  One reason I didn't create tags was that, thinking of the day I created a public versioning, I didn't want to paint myself into a corner.

Not sure if you are asking generally about versioning scheme or something more specific...
I would suggest to prefix version tags with "v" and use semantic versioning - i.e. always use 3 parts (major, minor. revision), bumping revision for each release with no ABI change, or bump minor and reset revision for backwards-compatible ABI changes like feature additions, or bump major and reset minor + revision for backwards-incompatible changes.
I would at first cheat a bit and use major version 0 as being "any ABI" - i.e. instead of not making formal tagged releases as all I would generously tag but indicate that there is no promise about ABI breakage by staying below major release 1.
That way the progress of the codebase gets tracked while it matures, and when you realize at some point that it has matured you can simply re-release as version 1.0.0.  Or if you never stabilize the ABI then just stay below 1.0.0 forever.

Since you are already track library API, I suggest using major version of API as minor version of library releases - i.e. now that code is at API 8.6.2 I suggest tagging that as v0.8.0 and move on from there using semantic versioning methodology for the release versions.

Oh - I just realized that you have already made 3 tags using 7 as major version.
If you want casual versioning at first, then I suggest simply ignoring those 3 tags and do as suggested above: make a v0.8.0 release and move on from there. If you then at some point was to treat your library as stable then you can bump straight to major version 8 (or whatever major version your API has become).

I was asking the question in both senses.  I was asking for general guidelines, but I also planned to ask you what you thought about specific version tags.
Speaking of which, I think it may be time for v1.0.0.  Libmarpa has been stable for some time and an essentially identical version has seen widespread use as the basis of Marpa::R2.  This suggests to me I must at this point decide to be one of those shy folks who never does a v1.0.0, or else take the plunge.
Also, I note you have been associated with Debian.  Is your interest in Marpa at this point at all related to this?
Back to v1.0.0.  Either way, I will want to think this over.  It's a major plunge on a project that is well over a decade old.  This means that, while in one sense it is a very simple change, I will not resolve this issue quickly.

I've been away from this repo for a while, and have started looking into it, making sure I can build it on my environment, etc.
One solution is really simple.  Create a v8.6.2 tag, and announce that v8.6.2 is the first pubic release.  @jonassmedegaard -- any comment?

Releasing/tagging what you got now as v8.6.2 was what I had in mind when I filed this issue.  That's the simplest sensible action I can think of doing now - but does not take into account shyness...

Also, I note you have been associated with Debian. Is your interest in Marpa at this point at all related to this?

Indeed, I am a Debian developer.  In 2014 I packaged Marpa::R2, then a dependency of Catmandu - but by the time I succeeded getting Marpa::R2 into Debian, Catmandu changed to use Parser::MGC instead and I lost interest in Marpa::R2.
Fast-forward 7 years, and my work authoring App::licensecheck has got me curious about faster-than-regex parsing and I rediscovered Marpa and wanted to at least modernize the packaging in Debian, even if possibly I fail at wrapping my head around using it myself.
In Debian shared libraries are preferably built separately from wrappers, and also we prefer tracking the true source - that's the reasons I ended at this git repo - hoping to be able to tie this codebase and the Marpa::R2 perl codebase together.

I'm doing a full test with Marpa::R2, prior to creating the tag.  This will involve running Marpa::R2 using a libmarpa library, a process which I documented in https://github.com/jeffreykegler/Marpa--R2/blob/master/cpan/engine/STATIC and https://github.com/jeffreykegler/Marpa--R2/blob/master/cpan/engine/SHARED .  This sounds like it's of interest to you, and in the process I will wind up testing and fixing typos (I hope no worse) in these documents.  I'm rusty at all this, so this will take a few days.
I think you understand, because I expect folks who want something in Debian just say, "Why not copy the repo and be done with it?" and if one is careful, things are not that simple.

I do understand, and I appreciiate your work.
Please do tell if you want me to test or proof-read something.  Or even better: Simply release as soon as you think it is ready yourself - and then be prepared to make another bugfix release in case I manage to spot something too flawed that it cannot collect dust until next time you get around to make a feature release :-)

I completed the tests, and have added the tag https://github.com/jeffreykegler/libmarpa/releases/tag/v8.6.2 .  This is the first official release.
@jonassmedegaard If all looks good to you, let me know and I will close this issue.
For followup discussion, you might consider our IRC channel: #marpa on<EMAIL_ADDRESS>.
If it's an issue, of course, the best place for it is as another Github issue.  Thanks!

Since I haven't heard any negative feedback since my last comment, I am closing this issue for the reasons given in that comment.  Thanks @jonassmedegaard for bringing it to my attention.


Use getacversion(_cn) and getacbuildtype(_cn) to detect eggcube users

Add these functions to onPlayerConnect.
Maybe for the start just add these values to the log:
print("\n\nAC VERSION: " .. getacversion(_cn) .. "\nAC BUILD TYPE: " .. getacbuildtype(_cn) .. "\n\n");

Eggcube seems to be dead, so no need to detect cheaters who use it.

Eggcube seems to be dead, so no need to detect cheaters who use it.


submit transaction response error, but infact the transaction has been successfully submitted. version: 1.7.0

My request
{
    "network_identifier": {
        "blockchain": "cardano",
        "network": "testnet"
    },
    "signed_transaction": "8279018a38346134303038313832353832303564636532623139663133363932323131313565653132633134616633356235346639363535326239333535343732346562646336336233616631373137373130303031383138323538316436306464613634313865373634616337373032343464616439373636656138383936646434646563343931616562316435316363316132643037316130356566633638303032316130303033306434303033316130336566313438306131303038313832353832306166386330623861623034336136633565356161313436363665313933333439623335306538313435326462343738333034346437303635326132343538363135383430333936323230396635663334316361613661333434663431343339633037343336353039336435366162363930636636636134646364373136643963666331393363376632663363323838636433653965613361313666646136393762613336626366323332373763626430316264366162363964613331623534303636306566356636a1696f7065726174696f6e60"
}

cardano-rosetta response
{
    "code": 5006,
    "message": "Error when sending the transaction",
    "retriable": true,
    "details": {
        "message": "Parse signed transaction error"
    }
}

but infact, the transaction has been submit to testnet. https://testnet.cardanoscan.io/transaction/607452ed1beaafd69df00cb3667dd393cea655a58686442c2d74914066486a20
when I checked the log of rosetta, I found this:

[5ea2eecd:cardano.node.Mempool:Info:25853] [2022-03-29 09:45:07.26 UTC] fromList [("tx",Object (fromList [("txid",String "607452ed")])),("kind",String "TraceMempoolAddedTx"),("mempoolSize",Object (fromList [("bytes",Number 200.0),("numTxs",Number 1.0)]))]
{"level":30,"time":1648547107270,"pid":104,"hostname":"5ea2eecd794e","reqId":101996,"msg":"[submitTransaction] transaction successfully sent Transaction successfully submitted.","v":1}
{"level":30,"time":1648547107270,"pid":104,"hostname":"5ea2eecd794e","reqId":101996,"msg":"[constructionHash] About to get hash of signed transaction","v":1}
{"level":30,"time":1648547107270,"pid":104,"hostname":"5ea2eecd794e","reqId":101996,"msg":"[getHashOfSignedTransaction] About to hash signed transaction 84a400818258205dce2b19f1369221115ee12c14af35b54f96552b93554724ebdc63b3af17177100018182581d60dda6418e764ac770244dad9766ea8896dd4dec491aeb1d51cc1a2d071a05efc680021a00030d40031a03ef1480a10081825820af8c0b8ab043a6c5e5aa14666e193349b350e81452db4783044d70652a24586158403962209f5f341caa6a344f41439c074365093d56ab690cf6ca4dcd716d9cfc193c7f2f3c288cd3e9ea3a16fda697ba36bcf23277cbd01bd6ab69da31b540660ef5f6","v":1}
{"level":30,"time":1648547107270,"pid":104,"hostname":"5ea2eecd794e","reqId":101996,"msg":"[getHashOfSignedTransaction] About to parse transaction from signed transaction bytes","v":1}
{"level":50,"time":1648547107271,"pid":104,"hostname":"5ea2eecd794e","reqId":101996,"error":"Deserialization failed in Transaction.auxiliary_data because: Expected null, found other type","msg":"[getHashOfSignedTransaction] There was an error parsing signed transaction","v":1}
{"level":50,"time":1648547107271,"pid":104,"hostname":"5ea2eecd794e","reqId":101996,"code":5003,"retriable":false,"msg":"Parse signed transaction error","stack":"Error: Parse signed transaction error\n    at Object.exports.buildApiError (/cardano-rosetta-server/dist/src/server/utils/errors.js:118:56)\n    at Object.parseSignedTransactionError (/cardano-rosetta-server/dist/src/server/utils/errors.js:144:51)\n    at Object.getHashOfSignedTransaction (/cardano-rosetta-server/dist/src/server/services/cardano-services.js:233:41)\n    at /cardano-rosetta-server/dist/src/server/controllers/construction-controller.js:179:56\n    at Generator.next (<anonymous>)\n    at fulfilled (/cardano-rosetta-server/dist/src/server/controllers/construction-controller.js:5:58)","type":"Error","v":1}
{"level":50,"time":1648547107272,"pid":104,"hostname":"5ea2eecd794e","reqId":101996,"code":5006,"retriable":true,"details":{"message":"Parse signed transaction error"},"msg":"[errorHandler] An error ocurred and will be sent as response","stack":"Error: Error when sending the transaction\n    at Object.exports.buildApiError (/cardano-rosetta-server/dist/src/server/utils/errors.js:118:56)\n    at Object.sendTransactionError (/cardano-rosetta-server/dist/src/server/utils/errors.js:153:51)\n    at /cardano-rosetta-server/dist/src/server/controllers/construction-controller.js:187:56","type":"Error","v":1}

Hi @somereason thanks for raising. Is this reproducible? It would be useful to have the steps you took to construct the transaction.

   //cardano client lib
    @Test
    void calcTxHash() throws Exception {
        BigInteger fee = BigInteger.valueOf(200000);
        BigInteger inputAmount = BigInteger.valueOf(99600000);
        var bodyBuilder = new TransactionBody.TransactionBodyBuilder()
                .ttl(66000000)
                .inputs(List.of(new TransactionInput.TransactionInputBuilder()
                        .transactionId("607452ed1beaafd69df00cb3667dd393cea655a58686442c2d74914066486a20")
                        .index(0)
                        .build()))
                .outputs(List.of(new TransactionOutput.TransactionOutputBuilder()
                        .address(address)
                        .value(Value.builder()
                                .coin(inputAmount.subtract(fee))
                                .build())
                        .build()))
                .fee(fee);
        Transaction tx = Transaction.builder()
                .body(bodyBuilder.build())
                .auxiliaryData(AuxiliaryData.builder()
                        .metadata(new CBORMetadata().put(BigInteger.valueOf(1924), "hello world"))
                        .build())
                .build();
        tx = account.sign(tx);
        System.out.println(tx.getBody().getHash());
        Assertions.assertTrue(tx.getBody().getHash().equals("ed274c6e445d3b173956d788bd33acfe1fcafa18d7a2b9c0d89c2ffd465888b6"));
        System.out.println(HexUtil.encodeHexString(getRosettaBytes(tx.serializeToHex())));
    }

    public static byte[] getRosettaBytes(String signedTxn) throws CborException {
        ByteArrayOutputStream baos = new ByteArrayOutputStream();
        new CborEncoder(baos).encode(new CborBuilder()
                .addArray()
                .add(signedTxn)
                .addMap()
                .addKey("operation")
                .value("")
                .end()
                .end()
                .build());
        return baos.toByteArray();
    }

note: getRosettaBytes works fine under cardano-clinet-lib version 0.1.5 and cardano-rosetta 1.5.0, but now I'm using cardano-clinet-lib version 0.2.0-beta2

I couldn't say exactly the cause here because Rosetta is intended to be used with the construction workflow, see Flow of Operations. Perhaps this issue is related to deserialising a transaction with custom metadata.

I couldn't say exactly the cause here because Rosetta is intended to be used with the construction workflow, see Flow of Operations. Perhaps this issue is related to deserialising a transaction with custom metadata.

but this transaction has broadcasted. so it seems the hex is legal.


fix/focus_web

Description

[webos, tizen] Template starter focus issues on Home screen

Related issues

https://github.com/flexn-io/renative/issues/1579

Npm releases
n/a

@pauliusguzas I'll fix  app-harness focus  in the next commit.

@ElenaDiachenko one thing noticed while testing app-harness though not sure if it can be considered bug, that once user navigates to lower buttons he can't go all the way up to see "hermes".


Minimum Required Columns For Read 'MAF'

Could read.maf's documentation specify what are the minimal columns to create a valid MAF class object? This information would be useful to know if you don't really have a MAF, but want to create an object you could easily and quickly draw an oncoplot with. I was looking at IGV's Issue Tracker and noticed there are multiple varieties of MAF and it made me wonder if MAFtools supports an input table that only has a few important columns and what the column names of those columns would be.

Hi,
This has been mentioned in the vignette.
You need following mandatory columns to read and generate an MAF object.
Hugo_Symbol, Chromosome, Start_Position, End_Position, Reference_Allele, Tumor_Seq_Allele2, Variant_Classification, Variant_Type and Tumor_Sample_Barcode.

Even if you dont really have above basic columns (such as ref-allele, tum-seq allele, chr) - for onocplot purpose you can create a dummy columns with the above names and fill them with NAs. It should just work fine. For onocplot you will only need Tumor_Sample_Barcode, Hugo_Symbol, and Variant_Classification

The vignette answers my question.


Delete unnecessary screenshots from application navigation with eligibility conditions test

Description
The tests already call applicantQuestions.seeEligibilityTag(...), which makes the screenshots redundant.
Release notes
The title of the pull request will be used as the default release notes description. If more detail is needed to communicate to partners the scope of the PR's changes, use this release notes section.
Checklist
General
Read the full guidelines for PRs here

[ ] Added the correct label: < feature | enhancement | bug | under-development | dependencies | infrastructure | ignore-for-release | database >
[ ] Assigned to a specific person, civiform/developers, or a more specific round-robin list
[ ] Added an additional reviewer from outside your organization as FYI (if the primary reviewer is in the same organization as you)
[ ] Created unit and/or browser tests which fail without the change (if possible)
[ ] Performed manual testing (Chrome and Firefox if it includes front-end changes)
[ ] Extended the README / documentation, if necessary

Database evolutions
Read the guidelines here

[ ] Assigned two reviewers
[ ] Guarded against already existing resources using IF NOT EXISTS and IF EXISTS
[ ] Downs created to undo changes in Ups

User visible changes

[ ] Followed steps to internationalize any new strings

[ ] Added context strings to new messages
[ ] Didn't use a message in applicant facing code that isn't translated yet (unless behind a flag)

[ ] Wrote browser tests using the validateAccessibility method
[ ] Tested on mobile view. See mobile device mode
[ ] Manually tested at 200% size
[ ] Manually evaluated tab order

New Features

[ ] Add new FeatureFlag env vars to server/conf/helper/feature-flags.conf
[ ] Conditioned new functionality on a FeatureFlag
[ ] Wrote browser tests with the feature flag off and on, etc.

Instructions for manual testing
If instructions are needed for manual testing by reviewers, include them here.
Issue(s) this completes
Fixes #<issue_number>; Fixes #<issue_number>...

#8182  👈
main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @ssandbekkhaug and the rest of your teammates on  Graphite


Should IEndpointConventionBuilder-derived types live in M.A.Builder?

See: https://github.com/aspnet/AspNetCore/blob/a9a9298d4f2ae06b373090a0a0dacb1f0911fe93/src/SignalR/server/SignalR/src/HubEndpointConventionBuilder.cs#L7
In MVC we put these in the Microsoft.AspNetCore.Builder namespace. It probably doesn't super matter because you wouldn't usually refer to one of these by type, but we should try to follow a convention.
I came across this during API review for components, which isn't in the M.A.Builder namespace.

/cc @davidfowl @JamesNK @anurse @BrennanConroy

To be clear, it's the interface and the class that need to be moved, right? Do we even need the class to be public?
Anyway, let's do the needful.

Even though this was added in 3.0, we should do a quick announcement with the breaking change label.

#Beans

yes, these types are supposed to be public sealed types.

we should try to follow a convention

@rynowak Do you think Microsoft.AspNetCore.Builder is a convention that third-party frameworks should follow? e.g. gRPC

https://www.youtube.com/watch?v=GxBSyx85Kp8


Feature Request: Explorer File Search

VSCode Version: 1.20
OS Version: Win10

Actually I can search for files with strg + p for some files and open them in the editor. But I think it would be better if I can type the filename in a Searchbar in the explorer. Because of the following tree:
api
- load
--- data.js
- add
--- data.js

Id I search for my data.js I got two files and with strg + p its complicated to understand wich is the wrong file. If I can search in the Workspace folder and see the files in the directorys I see wich one is the right one. In Visual Studio 2017 it's implemented that I can search in my wokrspace for files.

@bpasero can you reference the duplicated issue please?

ok no reference for us... sad life


Sync should verify content hash if file already exists

It doesn't appear that the current sync implementation checks whether a file that already exists in Dropbox is indeed the same local file, only that it exists, in which case it skips the put operation.
To ensure correct syncing, including possibly adding an option to delete remote files, the file content hash should be compared (see https://www.dropbox.com/developers/reference/content-hash).

Done, released v0.0.5 and updated example notebook to demonstrate. Let me know if this works for you.
https://gist.github.com/shadiakiki1986/7c478d451a4221d464d7bcfd5fc6a914


BrowserFS Deprecated

Hi there, I'm the maintainer of BrowserFS. I've been going through the dependents of browserfs.
Please see https://github.com/browser-fs/NOTICE
Would you be interested in updating your project to use the updated fork, ZenFS?

I see. Thanks. I will find time to work on this migration.


shotcut: 17.11 -> 18.03.06

Motivation for this change
update
Things done

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

[X] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[ ] Tested compilation of all pkgs that depend on this change using nix-shell -p nox --run "nox-review wip"
[X] Tested execution of all binary files (usually in ./result/bin/)
[X] Fits CONTRIBUTING.md.

Can you add yourself to borg? https://github.com/NixOS/ofborg/pull/151


Feature/case sensitivity#9

Codecov Report

Merging #11 into master will decrease coverage by <.01%.
The diff coverage is 97.22%.

@@            Coverage Diff             @@
##           master      #11      +/-   ##
==========================================
- Coverage   97.73%   97.73%   -0.01%
==========================================
  Files          15       18       +3
  Lines         398      442      +44
==========================================
+ Hits          389      432      +43
- Misses          9       10       +1

Impacted Files
Coverage Δ

...deMissingFramework/Steps/StepPipelineContext.swift
100% <ø> (ø)
:arrow_up:

...issingFramework/Steps/UnusedFileAnalyzerStep.swift
100% <100%> (ø)
:arrow_up:

Sources/XcodeMissingFramework/Model/File.swift
100% <100%> (ø)

...eworkTests/Steps/UnusedFileAnalyzerStepTests.swift
100% <100%> (ø)
:arrow_up:

...singFrameworkTests/Steps/DeleteFileStepTests.swift
96.29% <100%> (ø)
:arrow_up:

...singFramework/Steps/XcodeReferenceGatherStep.swift
100% <100%> (+5.26%)
:arrow_up:

...singFrameworkTests/Steps/FileGatherStepTests.swift
92.15% <100%> (ø)
:arrow_up:

Sources/XcodeMissingFramework/Model/Files.swift
100% <100%> (ø)

...s/XcodeMissingFramework/Steps/FileGatherStep.swift
100% <100%> (ø)
:arrow_up:

...s/XcodeMissingFramework/Steps/DeleteFileStep.swift
100% <100%> (ø)
:arrow_up:

... and 7 more

Continue to review full report at Codecov.

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


[Question]: is playwright.ps1 install needed if we have <PlaywrightPlatform>?

Your question
Question:
Do I need to run playwright.ps1 install if I have this in my csproj?
<PropertyGroup>
  <PlaywrightPlatform>linux</PlaywrightPlatform>
</PropertyGroup>

I do a dotnet publish.
If not, then what's the benefit of <PlaywrightPlatform>?
(related: #2098, #2086)

Here is how I currently think Playwright works:
  flowchart TD
      subgraph language [Language Process]
      .NET
      Java
      Python
      end

      subgraph nodeprocess [Node.js Process]
      driver[Playwright Driver]
      linux -.- driver
      osx -.- driver
      win -.- driver
      end

      subgraph browserprocess [Browser Process]
      Chromium
      Firefox
      WebKit
      others[Chrome / Edge / others via WS / CDP]
      end

      language-- Playwright Protocol ---nodeprocess
      nodeprocess-- DevTools Protocol ----browserprocess

      style others stroke-dasharray: 5 5

Playwright Protocol: https://github.com/microsoft/playwright/blob/main/packages/playwright-core/src/protocol/protocol.yml
DevTools Protocol: https://chromedevtools.github.io/devtools-protocol/
As you can see there are three different parts

The Playwright language specific code
The Playwright Node.js driver and Node.js runtimes for each platform
The Browsers

The Microsoft.Playwright NuGet package contains the Playwright .NET code (Microsoft.Playwright.dll) as well as the Node.js driver and a bundled Node.js runtimes for each platform. And because these Node.js runtimes are quite big (https://github.com/microsoft/playwright-dotnet/issues/1845) and you most likely only need one of them, we added a PlaywrightPlatform property (https://github.com/microsoft/playwright-dotnet/pull/1955) where you can specify which Node.js runtime you want.
The browsers on the other hand are not included in the NuGet package so you need to run playwright.ps1 install or Microsoft.Playwright.Program.Main(new []{ "install" }) to download them. This isn't done automatically because they are even bigger and there are usecases where you don't need them like

https://playwright.dev/dotnet/docs/browsers#google-chrome--microsoft-edge
https://playwright.dev/dotnet/docs/api/class-browsertype#browser-type-connect
https://playwright.dev/dotnet/docs/api/class-browsertype#browser-type-connect-over-cdp

or you only need one of them like playwright.ps1 install chromium / Microsoft.Playwright.Program.Main(new []{ "install", "chromium" }).

Excellent! Thanks!

WOW... thanks


Input/Output error when accessing public/private folder

Hi,
I want to access my public and/or private folder, but I get an error message stated "Input/Output error".
Tried to uninstall and reinstall keybase, but no luck.
$ cd /keybase/public/rgruyters
$ ls
ls: .: Input/output error
$
$ echo "This is a test" > test.txt
-bash: test.txt: Input/output error
$
$ ls
ls: .: Input/output error
$

my log id: 2a8f1e00402d1708a8245f1c

@rgruyters Hi, this was an outage at our end, apologies.  It's fixed now!

Yes it is working again. thanks.


Folder cell height should match the height of file cell

The height for the folder icon and folder name cell is slightly shorter than the one for files.  Kindly match height for consistency.

@ahmelsayed Fixed it!


Fix scrutinizer ci and remove code coverage github actions.

Q
A

Is bugfix?
✔

New feature?
❌

Breaks BC?
❌

https://scrutinizer-ci.com/g/yiisoft/yii2/inspections/bf428b4b-ae88-40ba-a025-82983e87314b fixed,

👍


Agg: replace the usage of ThreadPool by ThreadPoolManager.

Signed-off-by: fuzhe1989<EMAIL_ADDRESS>What problem does this PR solve?
Issue Number: close #4581
Problem Summary:
What is changed and how it works?
Check List
Tests

[ ] Unit test
[ ] Integration test
[ ] Manual test (add detailed scripts or steps below)
[ ] No code

Side effects

[ ] Performance regression: Consumes more CPU
[ ] Performance regression: Consumes more Memory
[ ] Breaking backward compatibility

Documentation

[ ] Affects user behaviors
[ ] Contains syntax changes
[ ] Contains variable changes
[ ] Contains experimental features
[ ] Changes MySQL compatibility

Release note

None

/run-all-tests

/merge


iOS SDK

URL to download iOS SDK is 2 years old version. Is this the latest one?

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

ID: 9bf40ff9-0c42-088c-ade2-4c991019db1e
Version Independent ID: 56531b33-1ca6-8726-bcfd-77e880ecc03c
Content: Use the iOS SDK - Azure Mobile Apps
Content Source: articles/app-service-mobile/app-service-mobile-ios-how-to-use-client-library.md
Service: app-service-mobile
GitHub Login: @elamalani
Microsoft Alias: emalani

@suvisudhakar Yes, that is correct version and URL.
If you're looking for an approach that has been recently updated, you can look into Azure App Center.
https://docs.microsoft.com/en-us/appcenter/
https://docs.microsoft.com/en-us/appcenter/sdk/getting-started/ios

@suvisudhakar We will now proceed to close this thread. If there are further questions regarding this matter, please tag me in your reply. We will gladly continue the discussion and we will reopen the issue.


get french cloud providers metrics

In order to better estimate energy consumption, it would be interesting to have access to dynamic data.
Two metrics have been identified and could help the estimation, if dynamically available (or computable):

Power Usage Efficiency, which could be aggregated with a compute time and any specific hardware configuration.

Data Center Energy Management, a fairly new KPI which is being discussed, technical specification can be found here (but I still had not seen anything particular to ML)

It would be interesting to have those values (or data to compute it) for cloud providers, starting with French ones as some data is available.
Model that integrates PUE metric :

Source: Assessing the suitability of the Greenhouse Gas Protocol for calculation of emissions from public cloud computing workloads
Another (secondary) goal for Code Carbon, instead of measuring the whole impact of a ML algorithm in the chain, could be to highlight its measure in the Life cycle assessment, to let users know which is the context of the measure.

Along their paper published on carbon emissions on deep learning algorithms, Google has published some GCP data based on carbon emissions (could / should we really trust their Grid carbon intensity values ?)
Their hourly mix seems to also be computed using ElectricityMap data.

https://github.com/mlco2/codecarbon/pull/398 Added the option to manually input the PUE


cilium-operator: don't reconcile non-GAMMA xRoutes without a Cilium-managed Gateway

Please ensure your pull request adheres to the following guidelines:

[x] For first time contributors, read Submitting a pull request
[x] All code is covered by unit and/or runtime tests where feasible.
[x] All commits contain a well written commit description including a title,
description and a Fixes: #XXX line if the commit addresses a particular
GitHub issue.
[x] If your commit description contains a Fixes: <commit-id> tag, then
please add the commit author[s] as reviewer[s] to this issue.
[x] All commits are signed off. See the section Developer’s Certificate of Origin
[x] Provide a title or release-note blurb suitable for the release notes.
[x] Are you a user of Cilium? Please add yourself to the Users doc
[x] Thanks for contributing!

This is a revival of https://github.com/cilium/cilium/pull/34753.
As of 1.16.3, the Cilium Gateway API operator currently makes several assumptions around it being the only Gateway API reconciler in a given cluster; deploying the Cilium operator alongside another Gateway API operator causes an infinite back-and-forth loop of reconciliation.
This PR makes the following changes:

adds logic to the (non-GAMMA) xRoute reconcilers to check for the existence of the Cilium gateway operator class
adds a test to ensure that the HTTPRoute reconciler does not update HTTPRoutes that do not have a cilium-managed Gateway in its parentRefs
updates the non-existent gateway test to ensure that the operator correctly errors without updating the status when a malformed HTTPRoute is encountered

Fixes: https://github.com/cilium/cilium/issues/34754
operator: don't reconcile non-GAMMA xRoutes without a Cilium-managed Gateway

/test

/test

/test

/ci-gateway-api

Not stale!

future TODO: ensure we only clobber status entries from our matching controller name (in case someone's using the same route with two Gateways/GatewayClasses; I'm not doing this so I'm punting on it for now)