Secondary colour overriden on child elements if using global styles

What
The secondary colour used on hint text is not applied to elements within the div is you are using $govuk-global-styles. The styles for the hint colour need to override things like paragraphs and lists.

The screenshot above shows the paragraph tag overriding the secondary colour.

Hi @chrisadesign 👋 Thanks for raising this issue. I'm going to raise this with the other developers in the team to get their thoughts as I think there's probably a bit of a discussion to be had here about what the recommended use of govuk-global-styles is and whether there's work we need to do to handle those overriding styles in our component styles. I'll add any updates on here once we've discussed it 🙂

@chrisadesign just wondering - are you using govuk-global-styles here within a prototype or in production?

I'm using it in a prototype, it's enabled by default in the kit.


Use the latest version of joffrey/docker-py for testing

The joffrey/docker-py docker image is used to test the proxy. The latest version of the image requires 1.24 Docker API which is implemented by 1.12.x Docker. Unfortunately, this version of Docker is very unreliable because of random hangs which happen during execution of the smoke tests.
By not using the latest version of the image we are risking to miss some breaks which might happen due to changes to the API.
On occasion (e.g. after a new Docker release), try to upgrade CI VM images and bump the  joffrey/docker-py vsn.

Relevant issue: https://github.com/docker/docker-py/issues/1183


Form validation is_unique fails on update

When updating an entry, having is_unique rule for a field that was not changed will make the validation fail.

you can simply do this
is_unique[tablename.username , id , {id}]
'username' => 'required|min_length[2]|max_length[10]|is_unique[users.username ,id,{id}]'

you can simply do this
is_unique[tablename.username , id , {id}]
'username' => 'required|min_length[2]|max_length[10]|is_unique[users.username ,id,{id}]'

That's what the documentation says I can't see it working on implemented anywhere.


Add app screenshot to README

Multiple people have been confused by the app not looking like the Playtime homepage. Until we have a production app that we can link publicly, a screenshot might help clarify things.
The task is to take a nice-looking app screenshot, make it a reasonable size, and put it somewhere logical near the beginning of the README. Bonus points if you make yourself a site manager/admin and add a few more wishlist items!
Let us know if you have any questions!

Can I take this up?

I had started working on this, but I would be more than happy to help you get started.
@tulikavijay Let me know if you are still interested in doing this :) Feel free to ask any queries.

@JacksonIsaac if you have started working on it then I would look for some new issue. Thanks.😊

@tulikavijay It's great to see all the interest on this issue! 🙂
If you're looking for another issue, #140 also deals with documentation/configuration. The test reporter setup processes has good instructions; alternatively, adding the test coverage badge (found here) to the README could be a stand-alone PR.


v0.1.0-rc0 published to hex

Opening this issue as a way for everyone to stay on the same page.
Have published brod_gssapi v0.1.0-rc0 to hex, HCA will begin vetting this today or tomorrow.  Has there been other vetting?
A PR #16 has been opened to switch to using to git for fetching the version to use when publishing for tags. I'm ok with hard code too (normally what I do tbh).  If we do stick with git vsn via tags, then our tags must conform to semantic versioning (note how the current tags do not).
@kjellwinblad  or @zmstone  since I do not have access to create tags (I believe), can you create one an v0.1.0-rc0 tag?
Finally, I'll add you two and Vikas as owners on hex for this package right now.

Hi.
I merged #16 and tagged (pushed) v1.0.0.
Have not yet published it to hex though,
Please let me know if you feel comfortable (or not) publishing v1.0.0 at the current state.
If yes, I'll publish it to hex
If no, I'll delete this tag and use v0.1.0-rc0 for the git tag.

should probably finalise the 2.0 release here ?
https://github.com/kafka4beam/brod_gssapi/blob/master/rebar.config#L18

@zmstone Yes, precisely when we get done vetting at our company, we will bump sasl auth to 2.0.0.
We had defer testing until next week. Will report back ASAP.

Just remembered there was a bug fix in sasl_auth, which resulted in a wrong version bump to v2.0.1, should have been v2.0.0-rc3 instead.
Deleted the v1.0.0 tag for brod_gssapi. and pushed a v0.1.0-rc0 tag to keep in sync with hex.pm
also sent a PR to pin the correct version of sasl_auth: https://github.com/kafka4beam/brod_gssapi/pull/18

@zmstone I have published brod_gssapi v0.1.0 to hexpm. Could you merge https://github.com/kafka4beam/brod_gssapi/pull/20 and tag that master as v0.1.0 after it is merged?

n/m, see if I had rights to do this. Tag created and closing this now :)


Add paper

周小晖,王意洁,徐鸿祚,刘铭宇.基于融合学习的无监督多维时间序列异常检测[J/OL].计算机研究与发展:1-14.
I am interested with this work; will you add this code in the future?

Thanks for your interest!
The first author is working on releasing the source code of this paper. If you are looking for an anomaly detection model for time-series data, you can find a contemporary work at https://github.com/xuhongzuo/couta.
Cheers!

Thanks for your interest!
The first author of this paper is working on releasing the source code. If you are looking for a time-series anomaly detection model, I may suggest a contemporary work at https://github.com/xuhongzuo/couta.
Cheers!

@xuhongzuo. Thank you for your recommendation. I am reading this article, and if I have any questions, I will ask them there.
Cheers!


Update Composer dependencies (2021-11-30-00-04)

Visual regression test passed!
View the visual regression test report


PowerToys stuck in the Settings menu

ℹ Computer information

PowerToys version: v0.33.1
PowerToys Utility: PowerToys Settings
Running PowerToys as Admin: No
Windows build number: Version 20H2 (OS Build 19042.857)
Note: I'm running windows on a mac

📝 Provide detailed reproduction steps (if any)
(This is what I think caused it)

I open PowerToys
Go to the settings menu
Quit PowerToys
Open PowerToys
Can't get to the main menu again

✔ Expected result
The expected result would be to be able to see the main menu
❌ Actual result
I can't see the main menu anymore, and am stuck in the settings menu
📷 Screenshots and Videos
Video of PowerToys.zip

@DeveloperWOW64
the video you posted doesn't show any problem.
Can you please explain what do you mean with "main menu"?

@DeveloperWOW64
It sounds like you are referring to the Getting Started screen. I don't believe that is supposed to be a main menu, rather a kind of introduction for new installs or maybe with updates too. Otherwise, the Setting panel is the main view at the moment.
You can always open that Welcome screen again by selecting PowerToys Tour in the General section of the Settings page.

Ah ok


Only push browser test images to docker hub once in github actions

Description
Change so that pushing the browser test images to docker hub only happens in the push workflow and not the other two workflows that run the bin/build-browser-tests script
Without this fix I was getting Error: Cannot perform an interactive login from a non TTY device from the Build browser testing container workflow, which happens in the docker login line

Thank you, @syntapy !


Adds consoledot pages best practices section

jira: https://issues.redhat.com/browse/RHCLOUD-26876
Rest of the content will follow. This is a base setup.

Don't merge this PR. It may break local development if your setup does not contain correct binaries. A docker image for development is incoming.

/retest

/retest


#377 SnackBar additions

Description of Change

Bugs Fixed

Fixes #377

API Changes
Added:
public class SnackBarOptions : ActionOptions
	{
		public IEnumerable<SnackBarActionOptions> Actions { get; }
	}

public class SnackBarActionOptions
	{
		public Func<Task> Action { get; set; }
		public string Text { get; set; }
		public double FontSize { get; set; } = 14;
		public string FontFamily { get; set; } = "Arial";
		public Color BackgroundColor { get; set; } = Color.White;
		public Color ForegroundColor { get; set; } = Color.Black;
	}

public class ActionOptions
	{
		public MessageOptions MessageOptions { get; }
		public Color BackgroundColor { get; }
		public bool IsRtl { get; }
		public TaskCompletionSource<bool> Result { get; }
		public int Duration { get; }
		public void SetResult(bool result) => Result.TrySetResult(result);
	}

public class MessageOptions
	{
		public string Message { get; set; }
		public double FontSize { get; set; } = 14;
		public string FontFamily { get; set; } = "Arial";
		public Color Color { get; set; } = Color.Default;
	}

Changed:
public static class PageExtension
	{
		public static Task<bool> DisplayToastAsync(this Page page, string message, int duration = 3000)
		public static Task<bool> DisplaySnackBarAsync(this Page page, string message, string actionButtonText, Func<Task> action, int duration = 3000)
		public static Task<bool> DisplaySnackBarAsync(this Page page, SnackBarOptions arguments)
}

Behavioral Changes

PR Checklist

[x] Has tests (if omitted, state reason in description)
[x] Has samples (if omitted, state reason in description)
[x] Rebased on top of main at time of PR
[x] Changes adhere to coding standard

UWP

Android

WPF

@VladislavAntonyuk could you pls resolve merge conflicts

Just one question I'd like to be clear on. Other than that, seems good

Just one question I'd like to be clear on. Other than that, seems good

Added UTs and added new extension method for toasts with ActionOptions()

it is also partially fix https://github.com/xamarin/XamarinCommunityToolkit/issues/260


Support for gs:// and s3:// directories

Would it be possible to add a feature when gs:// or s3:// log_dir can be specified for Tensorboard to start? Assuming the environment is already authenticated to the GCP or AWS.
I'm running job in Google Cloud ML Engine and logs are in a GCS bucket. Also using JupyterLab inside a container on the cloud instances where this can be added too if supported by this package. https://github.com/chaoleili/jupyterlab_tensorboard/issues/9#issuecomment-463513899
cc @chaoleili

While this is still not working, you can start tensorboard from the Cloud Shell, and preview on port 6006 (or whatever port you use for running tensorboard).
However, it'd be nice to open tensorboard from the jupyterlab without this workaround.

+1


useWallets did not return the expected on Server Render

useWallets did not return the expected on Server Render.
stateKey is missing.
const wallets = useWallets();

// output
{
    "wallets": [],
    "walletModules": [],
    "chains": [],
    ...
}

https://github.com/blocknative/web3-onboard/blob/2c8647c28426a7086c1988a1b32c2aa4cd6d5d52/packages/react/src/hooks/useAppState.ts#L31

Hey @lomocc could you provide some more details on what the issue is that you are seeing?

https://codesandbox.io/p/sandbox/nostalgic-mirzakhani-e4x1pd?file=README.md

Hey @lomocc could you provide some more details on what the issue is that you are seeing?

getServerSnapshot should return by stateKey

Ah I see what you mean. Thanks for reporting, we'll get this fixed.

@lomocc this has been fixed with the latest version of the react package v2.8.0. Thank you for reporting!


Support refs for all components

What is it?

[ ] Feature / enhancement
[ ] Bug
[ ] Docs / tests
[ ] Other

Why is it needed?

Checklist:

[ ] My code follows the developer guidelines of this project
[ ] I have performed a self-review of my own code
[ ] I have ran pnpm change and documented my changes
[ ] I have add necessary docs (if needed)
[ ] Added new tests to cover the fix / functionality

could you add some context pls? 🙏  just so we remember why it's needed.


Ability to import dashboards without deleting old versions?

Sorry if this is obvious and I missed it in the docs somewhere, but lets say we've imported all our existing dashboards via wizzy import dashboards, and then someone updates a few dashboards via Grafana's gui, is there a proper way to grab those specific updates via wizzy?
It looks like re-running wizzy import dashboards again results in the existing dashboard json files being completely over-written, which kind of seems like it'd defeat the purpose of having them in git history :)

How about this if we want to track the changes?
wizzy import dashboards
git init
git add dashboards
git commit -m "Added dashboards to git repo"

Now lets say the someone made some changes in the GUI and you pull the changes again.
wizzy import dashboards

You are correct that this will overwrite the existing dashboards in the dashboards directory. But then you can run:
git diff

to find what changed in the dashboards.
Does this resolves your question?

Well, unless I'm mistaken or missing something, if I follow those steps the files are still completely over-written (I'd previously misspoken) deleted and replaced with the newly imported files, and so doing git diff does not result in actually being able to see any diffs via git diff for me.

git diff might not be helpful for you I guess. Also, you are right that whenever you import dashboards, the dashboards are overwritten. Anyways, we can do the following to know what changed in the dashboards.

Hopefully now you should be able to see the changes made on GUI with what you have in your Git master branch.
Let me know if this still does not solve the issue. I would be happy to answer as I might be missing something too. :)

It might be helpful if possible to drop certain fields like the dashboard id

@PaulCapestany I think now I got your point. We should have this functionality to import old dashboards too. It is a valid feature request.

@PaulCapestany is the main change you are seeing is every dashboard has it's id updated?
This should fix that issue - https://github.com/utkarshcmu/wizzy/commit/c91e17f988603f65338cbd8ae354c41d946b957c

Let me know if that did not accomplish what you wanted.

@sstarcher appreciate the heads-up. Grabbed your commit (as well as other recent commits) and ran npm install && npm link, and then the following:
grafana-entities ▸ wizzy import dashboards
Importing 350 dashboards:

Unfortunately, after wizzy is done running for some reason it seems that now no files are updated or imported at all (and when I delete previously downloaded dashboards, no new files appear in this case either).

@PaulCapestany try installing the latest release 0.5.9.
I'm currently running it without any issues.

@sstarcher unless I'm missing something, there isn't a 0.5.9 release, and the latest release is 0.5.8. I tried running that, as well as the most recent commit @ e3795f25bd79756f148ac6e81946ffc751d0d9ea, and had the same issue of wizzy seemingly not downloading anything at all 😕

@PaulCapestany You are correct, there is no 0.5.9 release yet. Try running 0.5.7 version. That should get your dashboards. I am also seeing some issues with 0.5.8 currently. Will fix it asap.


Fixes for the vscode language server extension

Fix URL to the OpenCL.DebugInfo.100 grammar file, which moved from SPIRV-Tools to SPIRV-Headers.
Add a go.mod and go.sum file.
Go has now fully migrated to using modules, and relative imports no longer work.
Fixup all the relative imports to use a fully qualified path based on the module name.
Fix the .exe executable extension on windows.
Update the generated grammar files.
Update the README.md with more information.

I was able to build the vscode extension following these steps.  This is a great tool - thanks!  👍


Allow expressions in the xlab

I would like to label the x-axis in ssdplots with “Dissolved copper (ug/L)” where the u is actually a mu.
Usually I do this with expression in xlab(), but the way that the plot ssd_plot function works it doesn’t update if I use + xlab(expression("Copper EC10 value ("mu"g/L)")); and if I use an expression within ssd_plot (xlab = "xxxx") that doesn’t work either. Is there a way to allow x-labels to be added that allows for expressions and symbols?
Thanks

@jenni-gnz - this code does what you want
library(ssdtools)
library(ggplot2)
library(latex2exp)

gp <- ssd_plot(ssddata::ccme_boron, boron_pred, label = "Species", shape = "Group")
gp + scale_x_continuous(TeX("Concentration ($\\mu g$/L)"))
#> Scale for x is already present.
#> Adding another scale for x, which will replace the existing scale.

Created on 2024-09-18 with reprex v2.1.1

Thanks. I also had to add into scale_x_continuous(..., breaks = c(1, 10, 100, 1000, 10000))  to stop the scale labels changing to the squished up look.
For those that want a subscript in there, here is a solution:
scale_x_continuous(TeX("Copper $EC10_{my subscript}\phantom{x}(\mu g$/L)$"),
breaks = c(1, 10, 100, 1000, 10000))
(this took ages to figure out as extra spaces were added with various other options, so adding here in case anyone else wants it, including future me)

Thanks @jenni-gnz - we'll add it to the FAQs

done in dev branch


animateOnDataChange causes chart not to render

When using animateOnDataChange it causes the chart lines not to render at all. The axes are visible.
Minimal code example:
      <View style={{ backgroundColor: "white" }}>
        <LineChart
          isAnimated
          data={data}
          // animateOnDataChange
          animationDuration={1000}
          onDataChangeAnimationDuration={300}
        />
      </View>

I tried this on both POCO F3 Android phone and on iPhone 13, on both it happens the same.
I tried this on 1.4.7 and 1.4.10 versions of this library.
    "react-native": "0.73.6",
     "react-native-gesture-handler": "2.16.0",
     "react-native-gifted-charts": "1.4.7",
     "react-native-linear-gradient": "2.8.3",
     "react-native-reanimated": "3.8.1",
     "react-native-svg": "15.1.0",
    "react-native-svg-icon": "0.10.0",

Same issue, any solutions?

Hi @KraljSamo @piotr-weareconflict
This has been fixed from versions 1.4.7 onwards 🎉
Please use the latest version of the library.


C exports for fully functional binding library

Adding APIs and interfaces for:

[x] wallet creation
[x] account creation
[x] address creation
[x] address validation
[ ] address structure
[ ] utxo representation
[ ] signing transaction
[ ] transaction querying
[ ] transaction preparation interface
[ ] fee calculation

Hi @vincenthz ,
Appreciate your update. We are trying to support our Cardano from our mobile wallet (built on React Native for both iOS and Android); so going to use cardano-c repository. Any idea when will the rest of the functionality be ready?
Regards,

@vincenthz @NicolasDP how is it going? Any update for the progress of C binding library?

@Captainkoko we've been busy and focus with another aspect of the project, and hopefully we'll get to the binding probably this month. lots of tasks should be simple enough, that anyone requiring certain bindings could add it themselves. Anyone is more than welcome to start chipping away at the bindings


typo in docs

[Enter feedback here]
Typo in docs:
"Visualizations on dashboards can't be copied and pasted into Power BI reports or other dashboards.
Open a report that has at least one visualization.
Select the visualization and use Ctrl +C to copy, and Ctrl +V to paste."
"Can't" in this paragraph should be "can"
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: 00479245-9654-ad47-b2d5-12f1e36ccdc9
Version Independent ID: b46f2147-f0e8-cad0-7240-28b7231ae70d
Content: Copy and paste a visualization in Power BI - Power BI
Content Source: powerbi-docs/visuals/power-bi-visualization-copy-paste.md
Service: powerbi
Sub-service: pbi-visuals
GitHub Login: @mihart
Microsoft Alias: mihart

Hi @FelixPeckitt ,
Thank you for your feedback. While you can copy/paste a visual from a report, you can't do the same for a visual on a dashboard.
All the best,
The DOCS team


Using widgets to modify currentDrivers and currentClients. Schedule() uses the dataframe which doesn't reduce based on widget selection.

Schedule() was removing clients in the process of making the schedule. To fix I gave Schedule() a copy of the clients list instead so it can destroy that instead of the original.


🐛 BUG: confusing/misrepresented HTTP status output in CLI

What version of Wrangler are you using?
2.0.0.
What operating system are you using?
MacOS 12.3.1
Describe the Bug
When using wrangler CLI, there is a disconnect between what the wrangler connection receives as the HTTP status and what the Worker returns. The distinction is important yet confusing to most. Here is a simple reproduction demonstrating the behaviour.
$ echo "export default { fetch() { return new Response('hello world', { status: 419 }) } }" > index.js

$ npx wrangler dev index.js

⬣ Listening at http://localhost:8787

Now, make a request to the endpoint using curl (in another terminal) or a browser.
$ curl -so /dev/null http://localhost:8787 --write-out '%{http_code}'
419

Flip back to the wrangler CLI, and you'll see the wrangler output says the HTTP status was  HTTP 200.
9:49:52 am GET / 200
Script modified; context reset.

Now, this isn't really a bug but a misrepresentation of what is actually happening (and to be clear, the issue is present in the UI but shown differently so slightly less confusing but still not great).
Under the covers, the log entry isn't actually getting HTTP 200; it is getting {"outcome":"ok", ...} which is being displayed as the HTTP 200. What I'd be proposing here is updating the log line to something like this:
9:49:52 am GET / 419 [outcome=ok]

I'm not married to the syntax so happy to yield to whatever however, the intent would be that the CLI makes the subtle differences a little clearer to avoid people debugging HTTP status confusion.

Good catch, yeah we need to fix this, it's clearly just wrong.

nice! you beat me to it. thanks!


[4.1] Fix Maui Rendering on iOS 17.5 (Disable GPU Rendering)

Fixes this https://github.com/Mapsui/Mapsui/issues/2669

Was surprised, now GPU support is just totally cut up for MAUI in this library, while Android had its MAUI renderer/handler long enough for SKGLView and SkiaSharp 2.88.8 shipped it's own Apple Metal handler.
What is currently possible:

Was surprised, now GPU support is just totally cut up for MAUI in this library

Same here. And unfortunately this can have a rather big impact on performance. Complex maps with multiple layers get pretty slow without GPU acceleration.
In any case, Mapsui 4.1.6 seemed to work well on physical iOS 17.5 devices for me, and #2669 seems to indicate that the crashes only occur in the simulator.
@pauldendulk @inforithmics  Maybe it would be better to disable UseGPU only when running in a simulator, but not on real devices?

Btw, I also tried to enable UseGPU on Android (with a .NET MAUI 8 app), and this seems to work well initially, but after I navigate away from the map and then back, the map is all white (no content drawn any more). Would be great if this could be fixed as well (Android also profits greatly from GPU acceleration).
Is there an upstream SkiaSharp bug concerning GPU on Android, or is this a Mapsui-specific problem?

Is there an upstream SkiaSharp bug concerning GPU on Android, or is this a Mapsui-specific problem?

I found #1896 (fixed via #1934), but it does not seem to answer this question.

Isn't it possible to set UseGPU before the first use in any case you are sure, that the control doesn't crash? Inforithmics made a default setting, which works for each platform, so that new users have a satisfying experience. If you want to change something and you are sure, that this platform is capable to use GPU, then set MapControl.UseGPU to true before you instantiate the MapControl the first time.

Btw, I also tried to enable UseGPU on Android (with a .NET MAUI 8 app), and this seems to work well initially, but after I navigate away from the map and then back, the map is all white (no content drawn any more). Would be great if this could be fixed as well (Android also profits greatly from GPU acceleration).
Is there an upstream SkiaSharp bug concerning GPU on Android, or is this a Mapsui-specific problem?

SkiaSharp is not to blame here, quick guess is either GRContext changed, either the view Handler was somehow disposed when you navigated away.. Are you using Maui Shell?

Isn't it possible to set UseGPU before the first use in any case you are sure, that the control doesn't crash? Inforithmics made a default setting, which works for each platform, so that new users have a satisfying experience.

Yes, of course one can always override this default, and I do that for now to work around it.
But my point is that the default should be reasonable. So if there is no indication that GPU rendering fails on physical iOS devices (but only on the simulator) then it should be enabled for that case by default. This is also in line with the code snippet posted by @taublast above (which is from SkiaSharp, I assume?).

Is there an upstream SkiaSharp bug concerning GPU on Android, or is this a Mapsui-specific problem?

SkiaSharp is not to blame here, quick guess is either GRContext changed, either the view Handler was somehow disposed when you navigated away.. Are you using Maui Shell?

Yes, Maui Shell with TabBar navigation. One of the tabs has a MapControl, and I'm calling MapControl.StopUpdates/StartUpdates when navigating to/from a different tab.

Is there an upstream SkiaSharp bug concerning GPU on Android, or is this a Mapsui-specific problem?

SkiaSharp is not to blame here, quick guess is either GRContext changed, either the view Handler was somehow disposed when you navigated away.. Are you using Maui Shell?

Yes, Maui Shell with TabBar navigation. One of the tabs has a MapControl, and I'm calling MapControl.StopUpdates/StartUpdates when navigating to/from a different tab.

I tried removing the StopUpdates/StartUpdates call, but the problem persists.
Note that if I navigate to a different page via Shell.Current.CurrentPage.Navigation.PushAsync(new FancyPage()), then the map keeps working upon return. But if I navigate to a different tab and back, the map is all white.

Is there an upstream SkiaSharp bug concerning GPU on Android, or is this a Mapsui-specific problem?

SkiaSharp is not to blame here, quick guess is either GRContext changed, either the view Handler was somehow disposed when you navigated away..

@taublast Any idea how I can check which of the two is the problem? What can I do to debug this?
I already tried looking at the Mapsui log, but it doesn't seem to show any errors:
            Mapsui.Logging.Logger.LogDelegate += (level, message, ex) =>
            {
                Debug.WriteLine($"Mapsui: {level}: {message} - {ex}");
            };

@janusw Did you test this behavior with only SkiaSharp itself? I would assume, that it is a SkiaSharp problem and not a Mapsui one.
Replace the MapControl in your app with a native SkiaSharp control and add a line draw or anything else to it. Then try you usecase.

@janusw Did you test this behavior with only SkiaSharp itself?

An SKCanvasView still renders correctly after switching tabs, but I guess one needs to try an SKGLView (will try to look into that soon) ...

I would assume, that it is a SkiaSharp problem and not a Mapsui one.

That's not clear, but it would be good to know. I rather suspect the opposite ;)

@janusw Did you test this behavior with only SkiaSharp itself?

An SKCanvasView still renders correctly after switching tabs, but I guess one needs to try an SKGLView (will try to look into that soon) ...

Actually a plain SKGLView shows the same problem on Android: Works the first time, but does not render any more after switching tabs.
Seems to be this bug: https://github.com/mono/SkiaSharp/issues/2550

Did you checked the hack in the thread? To set MapControl.IsVisible = true when coming back to the tab containing Mapsui's MapControl?

Did you checked the hack in the thread? To set MapControl.IsVisible = true when coming back to the tab containing Mapsui's MapControl?

Yeah, that workaround actually works well (both for a plain SKGLView, as well as a MapControl). But note that setting IsVisible = true is not enough. You have to toggle it (set it to false and then true again).


Support for Statamic 5 and Laravel 11

Description
I think the title is self-explanatory. In any case, it would be great if the addon is updated soon.
Steps to reproduce
See above
Environment
See above

v3.3.0+ supports both - its just been released.

Statamic 5 compatibility added in v3.3.0.


Bug: Wrong running pod details on new deployment

📜 Description
We were deploying a new release on an env with a blue-green strategy, so when a new pod was created it was showing one pod with container creating state and an old pod with 0 counts, But after refreshing the app details page it was showing the old pod counts 1 and new pod getting created.
👟 Reproduction steps

Create a new app with a blue-green strategy.
Deploy the application and check the app details pages without refreshing it.

👍 Expected behavior
It should show the pod count in the old pod column.
👎 Actual Behavior
Write now it just removes it from the UI and after refreshing the page it shows the correct details.
☸ Kubernetes version
EKS 1.23
Cloud provider

🌍 Browser
Chrome
🧱 Your Environment
No response
✅ Proposed Solution
No response
👀 Have you spent some time to check if this issue has been raised before?

[X] I checked and didn't find any similar issue

🏢 Have you read the Code of Conduct?

[X] I have read the Code of Conduct

Not reproducible now closing the issue.


consistency and syntax questions

[x] https://github.com/ZerataX/yacx/blob/master/include/yacx/KernelArgs.hpp#L70 should probably be m_voArgs?
[x] https://github.com/ZerataX/yacx/blob/master/include/yacx/Options.hpp#L54 should be content?
[ ] https://github.com/ZerataX/yacx/blob/master/include/yacx/Options.hpp#L51 use fold expression instead?
[ ] https://github.com/ZerataX/yacx/blob/master/include/yacx/Program.hpp#L47 see above
[x] it says executor instead of executer in multiple places, also yacx - yet another cudaexecuter ist very long and makes the docs look bad on mobile
[x] https://github.com/ZerataX/yacx/blob/master/include/yacx/Headers.hpp#L3 util is imported twice
[x] executerDiagramm should probably be just called classDiagram b7ada6e22778974285ed0f18c21921f9c167a068

see branch https://github.com/ZerataX/yacx/tree/consistency


google_container_engine_versions asking for project variable

Wanted to let you know that I see the same behaviour as in #4.
Terraform Version
Terraform v0.11.8

provider.google v1.17.1
google-terraform-modules/kubernetes-engine/google 1.15.0

Adding 'project' fixes this.
As this is rather an issue with google provider, I have filled #1945

same with google_container_engine_versions

Hi @dmitrijsf thanks for your feedback.
project is optional yes ...
https://github.com/terraform-providers/terraform-provider-google/blob/f0d6b4d4f44b86e0594b30293478f4bf7a4f071c/google/data_source_container_registry_repository.go#L19
I will follow the open issue on the provider for the next release.
Thanks !


The Deployed App Insights Resource does not work

Followed the documentation and created a new Azure Resource Group which produced an Application Insights resource that is not connected/used and the dependencies are reversed. The website should depend on Application Insights and it should include an appsettings resource which specifies the instrumentation key.

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

ID: ea439429-69f0-2878-495f-92cbaf61c789
Version Independent ID: d9e2e3f9-8387-cde6-2042-93bae86645f0
Content: Create & deploy Visual Studio resource group projects - Azure Resource Manager
Content Source: articles/azure-resource-manager/vs-azure-tools-resource-groups-deployment-projects-create-deploy.md
Service: azure-resource-manager
GitHub Login: @tfitzmac
Microsoft Alias: tomfitz

Thanks for your comment. We are actively investigating and will get back to you shortly. Thanks for your patience.

@femsulu - can this issue be forward to the Visual Studio team? It is not really a documentation issue. The Visual Studio team would need to decide whether to make a change to the default template that is created when selecting web app. There is no update that can be made to the documentation.

adding @bmoore-msft , any idea who from VS can help review this?

pinged @bmoore-msft offline for an update.

assigning to Paul Yuk.

@KSchlobohm  - here's a workaround you can try using ARM.
The key line (578) goes into the appsettings sub-resource of the website and setting the appinsights ikey.  Note: pass the desired name of the app insights resource name using a parameter, and the template will dynamically insert the key using the resource it just created.
`
"properties":
{
| "APPINSIGHTS_INSTRUMENTATIONKEY": "[reference(concat('microsoft.insights/components/',   parameters('coreimagegalleryappinsightsName'))).InstrumentationKey]",
  | "AzureStorageConnection": "[Concat('DefaultEndpointsProtocol=https;AccountName=',variables('coreimagegallerystorageName'),';AccountKey=',listKeys(resourceId('Microsoft.Storage/storageAccounts',   variables('coreimagegallerystorageName')), providers('Microsoft.Storage',   'storageAccounts').apiVersions[0]).keys[0].value)]",
  | "ApplicationInsightsAgent_EXTENSION_VERSION": "~2",
  | "XDT_MicrosoftApplicationInsights_Mode": "recommended",
  | "APPINSIGHTS_PROFILERFEATURE_VERSION": "1.0.0",
  | "DiagnosticServices_EXTENSION_VERSION": "~3",
  | "APPINSIGHTS_SNAPSHOTFEATURE_VERSION": "1.0.0",
  | "SnapshotDebugger_EXTENSION_VERSION": "~1",
  | "InstrumentationEngine_EXTENSION_VERSION": "~1",
  | "XDT_MicrosoftApplicationInsights_BaseExtensions": "disabled"
  | }
  |
`
Also note:, dependsOn isn't set for the website and appsettings to depend on app insights but the template does ensure the app insights is created first in the JSON document.
Do let us know if this helps.
Cheers,

Thanks @femsulu, I was able to deploy a working version but wanted to surface this so that the default project template for Azure Resource Groups would also be updated to reflect the appropriate change.
Also, I'm now seeing that the default website template includes 4 separate Microsoft.Insights/alertrules which started throwing an error during deployment. I just used VS2019 16.5 preview 1 to verify these four rules are still included by default.

CPUHigh
LongHttpQueue
ForbiddenRequests
ServerErrors

##[error]10:25:21 PM - Resource Microsoft.Insights/alertrules 'ForbiddenRequests AzureResourceGroupTest' failed with message '{
  "code": "BadRequest",
  "message": "You cannot create or modify classic metric alerts for this subscription as this subscription 02d31b67-afd1-4bac-81bc-5ca6a7cd37af is being migrated or has been migrated to use new metric alerts. Learn more - aka.ms/alertclassicretirement"
}'

@KSchlobohm,   we have a workitem to fix the template in the VS tooling and plan to get to it soon.  Thanks!

Thank you Devin!

Will now close this thread since we have an internal workitem tracking this effort. Will make a note to update the thread once the templates are updated. Dont hesitate to ping if you have any followup questions.

I've published an update which removes the offending alert rules (app insights fragment, webapp project and webapp+sql projects).

@femsulu Can you provide me with an updated template or know who I can work with get them properly updated?  I notice there's a similar quickstart here: https://github.com/Azure/azure-quickstart-templates/blob/master/101-webapp-windows-ASPNET/azuredeploy.json.
Here are the current templates: WebSiteTemplates.zip.
Thanks!
(Note: I'll be OOF starting tomorrow until January)

@femsulu ping thanks!


publish-package-to-npm

General Checklist:

[ ] Tests added for this feature/bug

[ ] All tests are passed

[ ] No lint error

Review

[ ] Has been reviewed

[ ] Conform by the [merge request performance guides]

[ ] Conform by the [style guides]

[ ] Squashed related commits together

Tasks Checklist

[ ] Study requirement
[ ] Todo something 2
[ ] Todo something 3

What are the relevant issue numbers?
Closes #24

Pull Request Test Coverage Report for Build 204

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 83.678%

Totals

Change from base Build 202:
-0.2%

Covered Lines:
271

Relevant Lines:
311

💛  - Coveralls

Pull Request Test Coverage Report for Build 204

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.2%) to 83.678%

Totals

Change from base Build 202:
-0.2%

Covered Lines:
271

Relevant Lines:
311

💛  - Coveralls


refactor: use new input component to support dual currency on edit budget screen

Describe the changes you have made in this PR
On the Edit Allowance screen I removed the TextField component in favor of the new DualCurrencyField component to support fiat values on this input element.
Link this PR to an issue
#1016
Type of change (Remove other not matching type)

feat: New feature (non-breaking change which adds functionality)

Screenshots of the changes (If any)
BEFORE:

AFTER:

How has this been tested?
Ran locally and tested setting a new budget, checked console for any errors and verified new budget was set.
Checklist

[x] My code follows the style guidelines of this project and performed a self-review of my own code
[ ] New and existing tests pass locally with my changes
[x] I checked if I need to make corresponding changes to the documentation (and made those changes if needed)

NOTE: I also added a new prop to the DualCurrencyField component to support adding a hint text which will appear below the input element (it is optional). The TextField component had this functionality so I thought we should also have it in this new component.

Would be nice if the hint has the same style as the hint from the textfield-input

I pushed a WIP fix for the unit test FAIL that is happening, it looks like the currency setting is undefined in this unit test environment. Which is weird because this setting has a default of USD. So I am trying to set this setting before the test runs but it is still not working. Any ideas here?

Would be nice if the hint has the same style as the hint from the textfield-input

Good call, I updated this to be the case.

Merge conflicts resolved, unit test fixed and this is ready for final review and merging now.


fix: better error message for compute_note_hash_and_nullifier.

Closes #3012
Checklist:
Remove the checklist to signal you've completed it. Enable auto-merge if the PR is ready to merge.

[ ] If the pull request requires a cryptography review (e.g. cryptographic algorithm implementations) I have added the 'crypto' tag.
[ ] I have reviewed my diff in github, line by line and removed unexpected formatting changes, testing logs, or commented-out code.
[ ] Every change is related to the PR description.
[ ] I have linked this pull request to relevant issues (if any exist).

Benchmark results
No metrics with a significant change found.

Detailed results
All benchmarks are run on txs on the Benchmarking contract on the repository. Each tx consists of a batch call  to create_note and increment_balance, which guarantees that each tx has a private call, a nested private call, a public call, and a nested public call, as well as an emitted private note, an unencrypted log, and public storage read and write.
This benchmark source data is available in JSON format on S3 here.
Values are compared against data from master at commit 524cecf4 and shown if the difference exceeds 1%.
L2 block published to L1
Each column represents the number of txs on an L2 block published to L1.

Metric
8 txs
32 txs
128 txs

l1_rollup_calldata_size_in_bytes
45,444
179,588
716,132

l1_rollup_calldata_gas
222,828
867,776
3,449,648

l1_rollup_execution_gas
841,915
3,594,884
22,205,017

l2_block_processing_time_in_ms
1,967 (-2%)
7,555
29,754

note_successful_decrypting_time_in_ms
292 (-2%)
858 (-2%)
3,135

note_trial_decrypting_time_in_ms
94.0 (-4%)
65.0 (-19%)
134 (-1%)

l2_block_building_time_in_ms
13,166 (-1%)
52,226 (-1%)
208,786 (-1%)

l2_block_rollup_simulation_time_in_ms
11,853 (-1%)
47,007 (-1%)
187,737 (-1%)

l2_block_public_tx_process_time_in_ms
1,272 (-1%)
5,084
20,570

L2 chain processing
Each column represents the number of blocks on the L2 chain where each block has 16 txs.

Metric
5 blocks
10 blocks

node_history_sync_time_in_ms
21,758 (+2%)
42,774 (+2%)

note_history_successful_decrypting_time_in_ms
2,023 (+1%)
3,963 (+1%)

note_history_trial_decrypting_time_in_ms
121
144

node_database_size_in_bytes
1,630,220
1,095,466

pxe_database_size_in_bytes
27,188
54,187

Circuits stats
Stats on running time and I/O sizes collected for every circuit run across all benchmarks.

Circuit
circuit_simulation_time_in_ms
circuit_input_size_in_bytes
circuit_output_size_in_bytes

private-kernel-init
72.7 (+1%)
61,697
18,841

private-kernel-ordering
43.6
24,233
8,089

base-rollup
1,782 (+1%)
656,309
811

root-rollup
76.2 (-7%)
4,072
1,097

private-kernel-inner
53.3 (+1%)
81,504
18,841

public-kernel-private-input
41.8 (+1%)
41,455
18,841

public-kernel-non-first-iteration
26.6 (+1%)
41,497
18,841

merge-rollup
0.920 (+2%)
2,592
873

Miscellaneous
Transaction sizes based on how many contracts are deployed in the tx.

Metric
0 deployed contracts
1 deployed contracts

tx_size_in_bytes
8,723
27,240


Implement custom bot

Description
Allow adding of custom bot detection
What do you expect
Add config to specify bots.yml path or a method which allows adding of custom bots

You can do this today by settings something like Browser::Bot.bots["some string the user agent must have"] = "Readable name for the bot". If you need to completely erase the bot definition, go with Browser::Bot.bots.delete_if { true } and then Browser::Bot.bots.merge!(new_hash_with_bots_definition).
But I agree we should have some public API for this.


Time Series labelling window disappears

Describe the bug
Time Series labelling window gets blank when I scroll the time series window to the rightmost border. See gif attached.
To Reproduce

Open time series labelling task.
Scroll the time series window to the right edge. See dig

Expected behavior
The task window shows rightmost section.
Screenshots

Environment (please complete the following information):

OS: Windows
Chrome 92.0.4515.159
Label Studio Version 1.2

Additional context

@alexandrnikitin Thank you very much for the report, it's bug and I've created a ticket about it for our frontend team.

Any updates on this?

Turns out that I can't get to labeling after that issue happens. See this recording

Hey Alex 👋 . What version of LabelStudio are you currently running? I would try 1.5 or higher if you can, as this may have been fixed. If not, we will generate ticket to get this resolved.

The last comment and recording was made using the latest 1.5.0.post0.

Would you be able to post the output of the browser console when this occurs? Also, if you are able to (@makseq might be able to aid in this) try a direct build from the develop branch. There is a possibility this may have been fixed and not yet tagged in a release.


[$250] Expense - Merchant error message is not cleared after completing the field properly

If you haven’t already, check out our contributing guidelines for onboarding and email<EMAIL_ADDRESS>to request to join our Slack channel!

Version Number: 9.0.75-0
Reproducible in staging?: Yes
Reproducible in production?: No
Email or phone of affected tester (no customers<EMAIL_ADDRESS>Issue reported by: Applause Internal Team
Device used: Motorola MotoG60 - Android 12 - Chrome / Windows 10 - Chrome
App Component: Money Requests
Action Performed:

Open the staging.new.expensify.com website.
Tap on the FAB and select "Create Expense"
Complete with any amount and tap on "Next"
Select any workspace and try to submit the expense without any merchant.
When the error message is displayed, complete the merchant field with valid information.
Try to submit expense again.

Expected Result:
User should be able to submit the expense after completing the merchant field and fixing the error.
Actual Result:
After the user tries to submit an expense to a workspace without merchant, the displayed error message remains despite the field being completed. User is not able to submit the expense after the error is displayed.
Workaround:
Unknown
Platforms:

[ ] Android: Standalone
[ ] Android: HybridApp
[x] Android: mWeb Chrome
[ ] iOS: Standalone
[ ] iOS: HybridApp
[ ] iOS: mWeb Safari
[x] MacOS: Chrome / Safari
[x] MacOS: Desktop

Screenshots/Videos
https://github.com/user-attachments/assets/79bd685d-3ec3-49e4-9b4c-7cbba1cb9f4e
View all open jobs on GitHub
Upwork Automation - Do Not Edit

Upwork Job URL: https://www.upwork.com/jobs/~021867142072270891331
Upwork Job ID:<PHONE_NUMBER>270891331
Last Price Increase: 2024-12-12

Issue OwnerCurrent Issue Owner: @rojiphil

I can raise the quick PR if my proposal is correct

@dominictb proposal's RCA and solution makes sense to me. I think we can go ahead with the PR.
@Gonals @joekaufmanexpensify what do you think?

Sounds good to me too. We are going with @paultsimura's solution, then, @rojiphil?

Sounds good to me to have @Pujan92 review if this is a follow up from another issue

Fixed in staging

PR is merged

PR is out on prod. There isn't any payment to be issued here. This issue was a regression from https://github.com/Expensify/App/pull/51517 , which originally came from https://github.com/Expensify/App/issues/46897. All payment will be handled here.
Going to close this for now, but LMK if anyone has questions on that!


memcached rejects meta-get commands with no flags

Describe the bug
A meta-get command without any flags is rejected (probably by the check at https://github.com/memcached/memcached/blob/046c4bb5d8498420c13e5357c8299b60952b2595/proto_text.c#L1067).
To Reproduce
Send a meta-get without any flags. E.g., mg foo.
Detail (please include!)
A meta-get command without any flags is rejected (probably by the check at https://github.com/memcached/memcached/blob/046c4bb5d8498420c13e5357c8299b60952b2595/proto_text.c#L1067) which checks to see if at least three tokens are in the command. This does not work when the command looks like mg foo which only has two tokens. The docs seem to suggest this is valid.
The context is that I am implementing an existence check for keys and am not passing the v flag to this mg command (nor any other flag for that matter). Thus, I will either get an HD or EN response from which I can determine existence of a value for the key.
I have worked around the problem by passing an unused O flag, but I would have expected this to work given the documentation.

Hey, thanks!
I remember this being a deliberate decision, but I can't remember offhand anymore why I had done that. So I'll review the situation.

https://github.com/memcached/memcached/pull/938 for a fix

released in 1.6.18


Unity Editor : walls overlapping error

When loading the main scene with oculus link in the editor I get the following error:
It appears some of your walls overlap, are the walls intersecting? Please recreate your room in Settings. (in headset)
Error parsing walls, are the walls intersecting? Index was outside the the bounds of the array.
It seems that it can't access the fourth corner information and it throws an exception.
I am on quest 2, Unity 2020.3.18f1.
The official app release has no problems and I have tried redoing the set up.

Hello, is this error still occurring? Does "Clear Room Setup" fix it? (not just redoing Room Setup)


[feat] add transfer to .cfx domain owner (similar to .eth domain transfers in metamask)

Problem:
I want do be able to send tokens to the owner of a .cfx domain, similar how metamask users can tranfer token to .eth domain owners.
Therefore no need to copy or type complex non-human readable addresses but simple domain names for transactions.
Solution:
Instead of showing "Recipient address is invalid" when typing e.g.  "conflux.cfx", query the conflux domain name service nft contract to resolve the owner of a .cfx domain, show this owner in the portal and continue the tranaction

this should be tackled after CIP-46 is implemented


update config file for the openAI base url

Describe the bug
OpenAI base url is not correct.
How To Reproduce
Steps to reproduce the behavior (example):

use openai and obeserve error "raise self._make_status_error_from_response(err.response) from None
openai.NotFoundError: Error code: 404 - {'error': {'type': 'invalid_request_error', 'code': 'unknown_url', 'message': 'Unknown request URL: POST /chat/completions. Please check the URL for typos, or see the docs at https://platform.openai.com/docs/api-reference/.', 'param': None}}"

I changed config.toml file like this -
from
OPENAI = "https://api.openai.com"

to
OPENAI = "https://api.openai.com/v1"

its fixed just now. Pull the latest commits


Bump version to v0.5.0

Codecov Report

Merging #119 into master will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##           master     #119   +/-   ##
=======================================
  Coverage   27.65%   27.65%
=======================================
  Files          14       14
  Lines         188      188
  Branches       36       36
=======================================
  Hits           52       52
  Misses        106      106
  Partials       30       30

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 ca7dd89...4f229d1. Read the comment docs.


[sysrst_ctrl] Fix H2L met pulse detection

Fix #9660
Signed-off-by: Michael Schaffner<EMAIL_ADDRESS>

(Feel free to merge this once CI completes)


Add credo with a custom config

@keathley figured I'd start by bringing in the full generated config and we can whittle away the stuff you're not a fan of in this pr <3

So, I read through all of the credo checks to figure out what they do. Based on that here's the .credo.exs file with the stuff that I think makes the most sense to keep enabled: https://gist.github.com/keathley/acf81b3e6033361f6e72531194121465

@joeyrosztoczy I've ci to norm so feel free to add to rebase this branch against master and add the credo checks to the build process as well.

@keathley since we now have 2 - 3 checks with compile + test + credo I added caching to the build step and re-use the build in test and credo.
LMK if that's overkill and I can revert and we could rebuild in credo or we could just run them sequentially in one step. Just thought I'd suggest it but no worries if not
This would also ask for a CIRCLE_CI_CACHE_VERSION env var set in circleci (which could be used to bust the cache quickly and easily without a code change) which I could also remove.
I don't see the check's integrated with the project yet just FYI - there might be something we need to configure on the Github <-> CircleCI configuration to ensure build pass on the PR

@joeyrosztoczy sorry for the delay. This looks great to me. Would you mind rebasing your branch against master? It should run against CI now.

@joeyrosztoczy sorry for the delay. This looks great to me. Would you mind rebasing your branch against master? It should run against CI now.

You bet! Going to close and reopen got my remote out of sync


Points do not coincide with the grid for the x axis

Please check attached screenshot and you'll notice the problem.
As you can see from the first point of the chart there is discrepancy between grid for the X axis and the point. First value on the X axis is '1' and the first point has this value but when you look closer you can see that point is drawn after the line presenting '1' value on the X axis. Next example - the next line of the grid is '3', but the point is after it. All points are shifted to the right side. I have something wrong in the configuration? I've tried to use 'beginAtZero' option but it doesn't help in this situation.

Options for the chart:
		var chartOptions = function() {
			var title = {};
			title.display = true;

			var xLabel = {};
			xLabel.display = true;
			xLabel.labelString = "Maturity";

			var xTicks = {};
			xTicks.maxTicksLimit = 30;

			var xAxes = {};
			xAxes.stacked = false;
			xAxes.scaleLabel = xLabel;
			xAxes.ticks = xTicks;
			var xAxesArr = [];
			xAxesArr.push(xAxes);

			var yLabel = {};
			yLabel.display = true;
			yLabel.labelString = "Interest Rate (%)";

			var yTicks = {};
			yTicks.suggestedMin = 0;

			var yAxes = {};
			yAxes.stacked = false;
			yAxes.scaleLabel = yLabel;
			yAxes.ticks = yTicks;
			var yAxesArr = [];
			yAxesArr.push(yAxes);

			var scales = {};
			scales.xAxes = xAxesArr;
			scales.yAxes = yAxesArr;

			var hover = {};
			hover.mode = "dataset";

			var tooltips = {};
			tooltips.mode = "x-axis";

			var options = {};
			options.spanGaps = true;
			options.title = title;
			options.scales = scales;
			options.hover = hover;
			options.tooltips = tooltips;
			options.maintainAspectRatio = false;
			options.responsive = true;

			return options;
	        };

Environment

Chart.js version: 2.7.2

This can happen when the offsetGridLines option for the X-axis is set to true. If this is true, the element for a particular data point fall between the grid lines. This is set to true for a category scale in a 'bar' chart type by default.
You can fix this problem by setting offsetGridLines to false or using 'line' chart type with the default options.

Closing as resolved


Datepicker UI becomes ambiguous when changing the date programmatically

If you change the bound date programmatically, the UI is very ambiguous about which date is selected.

In this screenshot, the 19th is the selected date, the 20th is the last date the user has clicked on with the mouse, the 18th is the current date, and the month has been changed using the left and right arrows a few time such that the 1st appears 'pressed'. The currently selected date should be made visually distinct from all the other dates in some way, preferably with a css class that could be overwritten by apps using the datepicker component.
Example code to reproduce:
import { Component, OnInit } from '@angular/core';
import { FORM_DIRECTIVES } from '@angular/forms';
import { DATEPICKER_DIRECTIVES } from 'ng2-bootstrap/ng2-bootstrap';

@Component({
  selector: 'cui-example-broken-class',
  template: `
    <datepicker
          [(ngModel)]="exampleDate"
          [initDate]="exampleDate"
          [showWeeks]="false"
          minMode="day"
          maxMode="day"
          datepickerMode="day">
    </datepicker>
    <div>
    {{ exampleDate }}
    </div>
  `,
  directives: [DATEPICKER_DIRECTIVES, FORM_DIRECTIVES]
})
export class BrokenCssClassComponent implements OnInit {
  exampleDate: Date = new Date(2016, 4, 5);

  ngOnInit(): void {
    setTimeout(
        () => {
          this.exampleDate = new Date(2016, 4, 10);
        },
        5000);
  }
}

Remember to change the month a few times or you will not see the behavior I have described.

+1 Got the same problem in my component.

+1


Idea - Event hooks for various dashboard operations

Hello,
Can we have various events that can be triggered on different dashboard level operations such as whenever a sheet is removed from the dashboard, or the current extension is closed/deleted? My use case is, the extension that I am trying to build is linked to a Sheet of the user's choice and if the sheet is removed from the Dashboard or if the fields used in that sheet changed, I should get a callback to ensure I can reflect the changes without taking the overhead to check for changes manually.
The biggest requirement would be to be able to access the state of the extension's own state, like when it is removed from the Dashboard. I tried to use the DOM's unload and beforeunload event listeners on the window and the parent of the window (Since the extension is embedded in an iframe) but none get triggered. This would help us make a clean-up API call to update something like let's say usage.
Which brings me to the next and final requirement, can we have some way to identify the user via some ID or session ID? I need to license my plugin to ensure it complies to our FUP (Fair Usage Policy) but I am unable to do that since I cannot who is viewing the extension right now. Since the dashboards are shared instances across users, I cannot correctly track how many users are viewing my extension.
If there is any way to do this right now, please let me know or are these feasible or if this is intentionally left out, can you brief about that?
Thanks in advance.

Parts of the original request here have been included as part of Extensions API Release 1.7 where several new event types are now available.


UTM keeps crashing when quitting SuperTuxtKart on Debian

Describe the issue
Start SuperTuxtKart and it freezes and on quitting it crashes UTM
Configuration

UTM Version: 2.4.1
OS Version: 12.1
Apple Silicon

Crash log
QEMULauncher-2022-01-05-205349.ips.txt
QEMULauncher-2022-01-05-191629.ips.txt
crash.txt
** log**
debug.log.txt
Upload VM
config.plist.txt

Duplicate of #3053


Global Placement Crash

Describe the bug
Crash when calling global_placement. Crash is avoided by passing -skip_nesterov_place.
Expected Behavior
Graceful exit or no crash
Environment
[WARNING] Your current OpenROAD version is outdated.
It is recommened to pull the latest changes.
If problem persists, file a github issue with the re-producible test case.
kernel: Linux 6.8.10-zabbly+
os: Ubuntu 22.04.3 LTS (Jammy Jellyfish)
cmake version 3.25.1
-- The CXX compiler identification is GNU 11.4.0
-- Detecting CXX compiler ABI info
-- Detecting CXX compiler ABI info - done
-- Check for working CXX compiler: /usr/bin/c++ - skipped
-- Detecting CXX compile features
-- Detecting CXX compile features - done
-- OpenROAD version: v2.0-13997-gad54bbe88
-- System name: Linux
-- Compiler: GNU 11.4.0
-- Build type: RELEASE
-- Install prefix: /usr/local
-- C++ Standard: 17
-- C++ Standard Required: ON
-- C++ Extensions: OFF
-- The C compiler identification is GNU 11.4.0
-- Detecting C compiler ABI info
-- Detecting C compiler ABI info - done
-- Check for working C compiler: /usr/bin/cc - skipped
-- Detecting C compile features
-- Detecting C compile features - done
-- Found Python: /usr/bin/python3.10 (found version "3.10.12") found components: Interpreter
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD
-- Performing Test CMAKE_HAVE_LIBC_PTHREAD - Success
-- Found Threads: TRUE
-- Performing Test C_COMPILER_SUPPORTS__-Wall
-- Performing Test C_COMPILER_SUPPORTS__-Wall - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wall
-- Performing Test CXX_COMPILER_SUPPORTS__-Wall - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-array-bounds
-- Performing Test C_COMPILER_SUPPORTS__-Wno-array-bounds - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-array-bounds
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-array-bounds - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-nonnull
-- Performing Test C_COMPILER_SUPPORTS__-Wno-nonnull - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-nonnull
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-nonnull - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-maybe-uninitialized
-- Performing Test C_COMPILER_SUPPORTS__-Wno-maybe-uninitialized - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-maybe-uninitialized
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-maybe-uninitialized - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-format-overflow
-- Performing Test C_COMPILER_SUPPORTS__-Wno-format-overflow - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-format-overflow
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-format-overflow - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-variable
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-variable - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-variable
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-variable - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-function
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-function - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-function
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-function - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-write-strings
-- Performing Test C_COMPILER_SUPPORTS__-Wno-write-strings - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-write-strings
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-write-strings - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-sign-compare
-- Performing Test C_COMPILER_SUPPORTS__-Wno-sign-compare - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-sign-compare
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-sign-compare - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-deprecated
-- Performing Test C_COMPILER_SUPPORTS__-Wno-deprecated - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-deprecated
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-deprecated - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-c++11-narrowing
-- Performing Test C_COMPILER_SUPPORTS__-Wno-c++11-narrowing - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-c++11-narrowing
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-c++11-narrowing - Failed
-- Performing Test C_COMPILER_SUPPORTS__-Wno-register
-- Performing Test C_COMPILER_SUPPORTS__-Wno-register - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-register
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-register - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-format
-- Performing Test C_COMPILER_SUPPORTS__-Wno-format -CMake Warning at src/CMakeLists.txt:245 (message):
  spdlog: SPDLOG_FMT_EXTERNAL=ON

CMake Warning (dev) at src/sta/CMakeLists.txt:33 (option):
  Policy CMP0077 is not set: option() honors normal variables.  Run "cmake
  --help-policy CMP0077" for policy details.  Use the cmake_policy command to
  set the policy and suppress this warning.

  For compatibility with older versions of CMake, option is clearing the
  normal variable 'USE_TCL_READLINE'.
This warning is for project developers.  Use -Wno-dev to suppress it.

 Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-format
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-format - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-reserved-user-defined-literal
-- Performing Test C_COMPILER_SUPPORTS__-Wno-reserved-user-defined-literal - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-reserved-user-defined-literal
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-reserved-user-defined-literal - Failed
-- Performing Test C_COMPILER_SUPPORTS__-fpermissive
-- Performing Test C_COMPILER_SUPPORTS__-fpermissive - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-fpermissive
-- Performing Test CXX_COMPILER_SUPPORTS__-fpermissive - Success
-- Performing Test C_COMPILER_SUPPORTS__-x
-- Performing Test C_COMPILER_SUPPORTS__-x - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-x
-- Performing Test CXX_COMPILER_SUPPORTS__-x - Failed
-- Performing Test C_COMPILER_SUPPORTS__c++
-- Performing Test C_COMPILER_SUPPORTS__c++ - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__c++
-- Performing Test CXX_COMPILER_SUPPORTS__c++ - Failed
-- Performing Test C_COMPILER_SUPPORTS__-std=c++17
-- Performing Test C_COMPILER_SUPPORTS__-std=c++17 - Failed
-- Performing Test CXX_COMPILER_SUPPORTS__-std=c++17
-- Performing Test CXX_COMPILER_SUPPORTS__-std=c++17 - Success
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-but-set-variable
-- Performing Test C_COMPILER_SUPPORTS__-Wno-unused-but-set-variable - Success
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-but-set-variable
-- Performing Test CXX_COMPILER_SUPPORTS__-Wno-unused-but-set-variable - Success
-- TCL library: /usr/lib/x86_64-linux-gnu/libtcl.so
-- TCL header: /usr/include/tcl/tcl.h
-- TCL readline library: /usr/lib/x86_64-linux-gnu/libtclreadline.so
-- TCL readline header: /usr/include/x86_64-linux-gnu
-- Found SWIG: /usr/bin/swig4.0 (found suitable version "4.0.2", minimum required is "4.0")
-- Found Boost: /usr/local/lib/cmake/Boost-1.81.0/BoostConfig.cmake (found version "1.81.0")
-- boost: 1.81.0
-- Found Python3: /usr/include/python3.10 (found version "3.10.12") found components: Development Development.Module Development.Embed
-- Found ZLIB: /usr/lib/x86_64-linux-gnu/libz.so (found version "1.2.11")
-- spdlog: 1.9.2
-- Found BISON: /usr/bin/bison (found version "3.8.2")
-- Could NOT find Doxygen (missing: DOXYGEN_EXECUTABLE)
-- STA version: 2.5.0
-- STA git sha: e01d3f163f483f233db00410b6515a767a6ca03b
-- System name: Linux
-- Compiler: GNU 11.4.0
-- Build type: RELEASE
-- Build CXX_FLAGS: -O3 -DNDEBUG
-- Install prefix: /usr/local
-- Found FLEX: /usr/bin/flex (found version "2.6.4")
-- TCL library: /usr/lib/x86_64-linux-gnu/libtcl.so
-- TCL header: /usr/include/tcl/tcl.h
-- SSTA: 0
-- Found SWIG: /usr/bin/swig4.0 (found suitable version "4.0.2", minimum required is "3.0")
-- STA executable: /home/karim/work/OpenROAD/src/sta/app/sta
-- Found re2: /home/karim/Downloads/or-tools_x86_64_Ubuntu-22.10_cpp_v9.5.2237/lib/cmake/re2/re2Config.cmake (found version "9.0.0")
-- Found Clp: /home/karim/Downloads/or-tools_x86_64_Ubuntu-22.10_cpp_v9.5.2237/lib/cmake/Clp/ClpConfig.cmake (found version "1.17.7")
-- Found Cbc: /home/karim/Downloads/or-tools_x86_64_Ubuntu-22.10_cpp_v9.5.2237/lib/cmake/Cbc/CbcConfig.cmake (found version "2.10.7")
-- Found SCIP: /home/karim/Downloads/or-tools_x86_64_Ubuntu-22.10_cpp_v9.5.2237/lib/cmake/scip/scip-config.cmake (found version "8.0.1")
-- Found OpenMP_CXX: -fopenmp (found version "4.5")
-- Found OpenMP: TRUE (found version "4.5")
-- GPU is not enabled
-- TCL library: /usr/lib/x86_64-linux-gnu/libtcl.so
-- TCL header: /usr/include/tcl/tcl.h
-- GUI is enabled
-- Charts widget is not enabled
-- Found Boost: /usr/local/lib/cmake/Boost-1.81.0/BoostConfig.cmake (found version "1.81.0") found components: serialization
-- Could NOT find VTune (missing: VTune_LIBRARIES VTune_INCLUDE_DIRS)
-- Found Boost: /usr/local/lib/cmake/Boost-1.81.0/BoostConfig.cmake (found suitable version "1.81.0", minimum required is "1.78")
-- TCL library: /usr/lib/x86_64-linux-gnu/libtcl.so
-- TCL header: /usr/include/Number of processor cores: 20
tcl/tcl.h
-- Found Boost: /usr/local/lib/cmake/Boost-1.81.0/BoostConfig.cmake (found version "1.81.0") found components: serialization system thread
-- Found Boost: /usr/local/lib/cmake/Boost-1.81.0/BoostConfig.cmake (found version "1.81.0")
-- Found Eigen3: /usr/share/eigen3/cmake/Eigen3Config.cmake (found version "3.4.0")
-- TCL readline enabled
-- Tcl Extended disabled
-- Python3 enabled
-- Configuring done
-- Generating done
-- Build files have been written to: /tmp/tmp.Es870W5hWq

To Reproduce
Extract the following zip file. cd into the extracted folder and run openroad script.tcl
multimacro.zip
Relevant log output
[InitialPlace]  Iter: 5 CG residual: 0.00000008 HPWL: 6487761
[INFO GPL-0031] FillerInit:NumGCells:         0
[INFO GPL-0032] FillerInit:NumGNets:        667
[INFO GPL-0033] FillerInit:NumGPins:       2061
Signal 8 received
Stack trace:
 0# 0x00005578D632FB7B in openroad
 1# 0x00007FCBD1842520 in /lib/x86_64-linux-gnu/libc.so.6
 2# gpl::BinGrid::initBins() in openroad
 3# gpl::NesterovBase::NesterovBase(gpl::NesterovBaseVars, std::shared_ptr<gpl::PlacerBase>, std::shared_ptr<gpl::NesterovBaseCommon>, utl::Logger*) in openroad
 4# gpl::Replace::initNesterovPlace(int) in openroad
 5# gpl::Replace::doNesterovPlace(int, int) in openroad
 6# 0x00005578D63F5D7E in openroad
 7# TclNRRunCallbacks in /lib/x86_64-linux-gnu/libtcl8.6.so
 8# 0x00007FCBD5D37028 in /lib/x86_64-linux-gnu/libtcl8.6.so
 9# Tcl_EvalEx in /lib/x86_64-linux-gnu/libtcl8.6.so
10# Tcl_Eval in /lib/x86_64-linux-gnu/libtcl8.6.so
11# sta::sourceTclFile(char const*, bool, bool, Tcl_Interp*) in openroad
12# ord::tclAppInit(Tcl_Interp*) in openroad
13# Tcl_MainEx in /lib/x86_64-linux-gnu/libtcl8.6.so
14# main in openroad
15# 0x00007FCBD1829D90 in /lib/x86_64-linux-gnu/libc.so.6
16# __libc_start_main in /lib/x86_64-linux-gnu/libc.so.6
17# _start in openroad

Screenshots
No response
Additional Context
Design has multiple regions.

Hi @kareefardi, thanks for your report. Right out of the bat I see a 0 for the number of GCells on the log you sent.
Were you able to run your scripts on a previous version of OR before?

I created a PR to avoid the crash. Although, I did not yet discovered the root cause of the issue. With the following fix we get another error moving forward in gpl.

@gudeh Thanks for the update. I didn't run this specific script on previous version.
I tried the branch in the PR. Indeed I ran into error requesting a higher density. However after setting a higher density I am running into
[ERROR GPL-0304] RePlAce diverged at initial iteration with steplength being -nan. Re-run with a smaller init_density_penalty value.

I set the smallest possible init_density_penalty but I am still getting the same error. Do you have hints on how to proceed?

Are you sure the floorplan is correct?

The design has two regions. All the cells are divided into these regions. There are no cells in the area outside these regions. This is my first time dealing with regions, but it seems unlikely that there something illegal about have the design's cells belong only to regions.

Hi @kareefardi. It seems GPL support to regions is broken. We do not use set_domain_area anywhere on ORFS scripts, so this has not come to light previously, thanks for sharing the error! I will try to fix this.

Hello @gudeh. Thanks for the reply. Since this is untapped territory, there is not telling whether this is something simple to fix or might take a while. Is that correct?

Indeed, here is what happens. I have been recently assigned to work on gpl. Although, this is the first time I dive deeper into the initial placer step, and also with regions and groups. This issue is good, because it is an opportunity for me to be more proficient with the algorithm, and OFC solve an actual problem.
The Replace::doIinitialPlace() instantiates some important objects for gpl, including a PlacerBase object for each existing group, and one for instances without any groups. The test case in tools/OpenROAD/test/upf_test also uses two regions, although we have some cells without any group. On your test case all cells are assigned to at least one group. We end up having one PlacerBase object without any cells (it is curious that it assigned some "dummy instances", which I do not fully understand yet), consequently further in the algorithm some calculations end up having NANs, because there are no instances in this PlacerBase object.
By erasing the PlacerBase object without cells the gpl converges until the end:

I am still a little worried, because I would expect the cells to be initially placed on the center of each region. Since, when we don't have regions, they usually start at the center of the core area.

It looks like initial placement doesn't consider the regions and that gets fixed by nesterov.  It would be good if initial placement were are as well.

I'm a bit more concerned that at the end of gpl there are objects outside their regions.  dpl is fixing that but it seems wrong.  Do you understand why this happens?

I think your idea to remove the empty placer is correct.

I think your idea to remove the empty placer is correct.

I ran a secure-CI and the failures it got are the same happening on master. I will update the PR for this crash.

It looks like initial placement doesn't consider the regions and that gets fixed by nesterov. It would be good if initial placement were are as well.

I'm a bit more concerned that at the end of gpl there are objects outside their regions. dpl is fixing that but it seems wrong. Do you understand why this happens?

All of the instances completely outside the regions are clock buffers: sky130_fd_sc_hd__clkbuf_1 and sky130_fd_sc_hd__clkbuf_2. Is it common that they have a special treatment? I see we have a MBFF class on gpl, maybe it is related?

mbff is for flop clustering and is likely unrelated.  Perhaps those buffers are being added by the resizer.  If you look at the names of the instances it should give a clue.  If they are really in the no region then their placement is fine.

I forgot to mention @kareefardi, the prints I sent here were with a -density 0.7, I was not able to run with your preconfigured 0.1, although 0.15 did finish with my fix.

This issue raised another problem also, with the TCL interpreter. We have an exit in the middle of the script, before detail_placement, if we leave it there and run the script. It exits after running all of the commands in the script, as if the exit was in the end of the document, not the middle.
Should I create another issue for that?

mbff is for flop clustering and is likely unrelated. Perhaps those buffers are being added by the resizer. If you look at the names of the instances it should give a clue. If they are really in the no region then their placement is fine.

some of their names are:

spm_inst_0/input23
spm_inst_0/input27
spm_inst_0/input31
spm_inst_0/input30
spm_inst_1/input1
spm_inst_1/input30

Those are input buffers so they wouldn't be in the incoming netlist.  Its fine for them to be outside the regions afaik.

Should I create another issue for that?

Yes

I am closing this issue. Please, feel free to confirm if your design is working as expected @kareefardi.

@gudeh Thanks for your effort. The design does finish now.
However following up on @maliberty comment with respect to these kind of instances spm_inst_0/input23, they do belong to the region. The netlist created for spm_inst_0 used resizer to insert these input buffers. But in the top level design that instantiates the macros, no input buffers are inserted by resizer. Here is a snippet of the generated def file:
GROUPS 2 ;
    - spm_inst_0_domain spm_inst_0/output35_y_iso_y_iso_o_isolation spm_inst_0/output35 spm_inst_0/input9
         spm_inst_0/input8 spm_inst_0/input7 spm_inst_0/input6 spm_inst_0/input5
         spm_inst_0/input4 spm_inst_0/input34 spm_inst_0/input33 spm_inst_0/input32
         spm_inst_0/input31 spm_inst_0/input30 spm_inst_0/input3 spm_inst_0/input29
         spm_inst_0/input28 spm_inst_0/input27 spm_inst_0/input26 spm_inst_0/input25
         spm_inst_0/input24 spm_inst_0/input23 spm_inst_0/input22 spm_inst_0/input21


fix: safe option in binary to term

binary_to_term has a safe switch for when you deserialize untrusted inputs (which is not the case in the places we're using it) but just so no one stores atoms in the DB ... here it is!
Overview
Added a :safe argument to binary_to_term/2
Changes

http://erlang.org/doc/man/erlang.html#binary_to_term-2

Testing
/

Coverage remained the same at ?% when pulling f75b7ad02303120f126408b4555b248b9849baf4 on sobelow2 into 98bb24cdf3e700da8b6ee4101cec49fef784a310 on master.

Why is that? I presume we're doing what sobelow wants us to do and would emit warnings for this?

We want to suppress the warnings, since we have added the safe flag and we are going to review the input filtering so we can be sure that only sane input reaches the functions. Sobelow showing us no more errors allows us to define as part of the CI and fail the build if it finds sth new.


chore: bump to vue v3.2.32

CI is failing because v3.2.32 changed how the root component is used in createApp https://github.com/vuejs/core/commit/d121a9bc7e7af59adb2d2803954cfeee95b35270; The root component is now copied before used.
This broke how we stub components, as we used to add the root component to the list of component to not stub.
As the component is now no longer the same, all tests using shallowMount are failing.
The fix is to register the copy of the component created by Vue in createApp (which is available as app._component)


Add shortname for unset-env command

Does this PR modify CLI v6 or v7?
Both

What Need Does It Address?
Provides a consistent system of aliases for commonly used command groups that set and unset via the CLI

Who Is The Functionality For?
Any user that uses environment variables with cloud foundry

How Often Will This Functionality Be Used?
Hopefully in place of cf unset-env APP_NAME ENV_NAME

Possible Drawbacks
None

Why Should This Be In Core?
No new functionality was added, just a short name alias

Description of the Change
Adds the ability to use cf ue instead of cf unset-env

Alternate Designs
Considered cf use APP_NAME ENV_NAME but "use" could conflict with future commands and from a language standpoint could be confusing as an alias for unsetting an environment variable

Applicable Issues
None

How Urgent Is The Change?
Not at all

Other Relevant Parties
N/A

Hi @andrebrowne thanks; sorry about the delay. We want to make sure we have a convention for this for unset commands before we merge this in. Thanks for your patience on this.

Making the recommended edits. Had to put the cf cli environment on a new machine.

@Ema93sh All requested changes completed. Please squash and merge at your convenience.

We ran into some trouble when trying to squash the commits together and found it simpler to create a patch from the diff and apply the patch to master and commit it as you. Also removed the integration test from global as it was duplicating the functionality of the shared tests and then fixed a minor assertion error in one of the tests.
Closed by: 6f0ffb1fb7f6c0ae035841532038038397a909b7

Thanks for your contribution! The changes have been pulled into master, so I'll close this pr.


$PSUICulture Variable Empty in PS7 on Ubuntu 18.04

Steps to reproduce
Log into PS 7.0 on an Ubuntu 18.04 machine:
$PSUICulture

Expected behavior
en-US

Actual behavior
Returns empty.

Environment data
Name                           Value
----                           -----
PSVersion                      7.0.0
PSEdition                      Core
GitCommitId                    7.0.0
OS                             Linux 5.0.0-1032-azure #34-Ubuntu SMP Mon Feb 10 19:37:25 UTC 2020
Platform                       Unix
PSCompatibleVersions           {1.0, 2.0, 3.0, 4.0…}
PSRemotingProtocolVersion      2.3
SerializationVersion           <IP_ADDRESS>
WSManStackVersion              3.0

Notes

$PSUICulture returns correct value on PS Core 6 and PS 7 on Ubuntu 16.04.
Seems like an issue specific to PS 7 on Ubuntu 18.04.

As a data point: In my local Ubuntu 18.04.4 LTS VM, with the latest updates applied, I see a different kernel, and I also don't see the problem you describe.
Here's the - differing - kernel info from my VM, extracted from $PSVersionTable's outptu:
OS    Linux 4.18.0-041800-generic #201808122131 SMP Sun Aug 12 21:33:20 UTC

@mklement0 - interesting.
I was testing using the standard Azure Ubuntu 18.04 marketplace image and had not applied any updates to it.
I first detected the issue occurring in Azure DevOps Pipelines ubuntu-18.04 pipeline images:
https://dev.azure.com/dscottraynsford/GitHub/_build/results?buildId=1431&view=logs&j=1b4c76ef-e880-5241-0155-88566c4dcdab&t=c3b097e0-8d2d-5cef-f943-5ce972a892ac&l=64

The variable returns System.Threading.Thread.CurrentThread.CurrentUICulture.Name.
Docs says:

by default is the same as the operating system culture

So I believe the Ubuntu image have probably an invariant culture like "C".

Good catch, @iSazonov.
The invariant culture CultureInfo object indeed has the empty string as its .Name property, which is what $PSUICulture (and $PSCulture) reports when the C or POSIX locale is in effect in the calling POSIX-like shell (run locale to check).
# From Bash:

$ LC_ALL=en-US pwsh -noprofile -c '$PSUICulture'
en-US # as expected

$ LC_ALL=C pwsh -noprofile -c '$PSUICulture'
 # empty string

$ LC_ALL=POSIX pwsh -noprofile -c '$PSUICulture'
 # empty string

The invariant culture's CultureInfo object (examine its properties with [cultureinfo] '' | Format-List)

Use [cultureinfo]::InvariantCulture for that (edit or react to this and I'll mark off topic)

Thanks, @SeeminglyScience , I've made the edit, but I'm also curious (happy to mark this as off-topic later):
Did you  suggest [cultureinfo]::InvariantCulture solely for conceptual clarity, given that [cultureinfo]  '' works too?
The only difference - which I just now realized - is that [cultureinfo] '' creates a read-write copy of the invariant culture. Was that the reason?

Yeah mainly conceptual clarity.  It does also create an extra object that isn't needed, and also won't have reference equality with the static instance (neither of these are likely to ever matter though).


Why use new() constraint

The implementaion is a little weird:

There's no public paramterless constructor, so how does the new() constraint work?
I don't see any necessary to new T() if just to get a static value. Why not use info.GetValue(null) directly?
The List method of the CarType doesn’t scale well, what if we need to add a new option?

Close it as not link to the doc


compiler bind hooks

import stdio.h;

C Compiler should be a mixin library i.e. monkey patch.

(register_parser( '.h_s ansi-c-parse-file ))


Scheduler doesn't accept queue names as symbols

Currently if a queue is specified by symbol, such as: sidekiq_options queue: :production, schedules are seen as disabled. The current code seems to expect strings.
Quick fix would be to add a .to_s to
https://github.com/moove-it/sidekiq-scheduler/blob/c8ebc9d803955ee6cdf86a3414fe595f125720ad/lib/sidekiq/scheduler.rb#L68
Sidekiq itself is fine with queues being named by symbols or strings

PR #186

Released v2.1.5


does tesseract support intel mkl?

I heard that intel mkl could speed up matrix computation, so does anyone know if it could be introduced to tesseract? thanks!

Please respect guidelines for posting issue: use tesseract user forum for asking questions/support.

sorry for my mistake, never happen again.


elevation not loaded with WPF

Hi there,
I have a problem, and I don't found any simple issue:
When I want to load elevation source, with "ArcGISTiledElevationSource", it returns its loaded status was "Not Loaded".
And I also check my browser can connect to "https://elevation3d.arcgis.com/arcgis/rest/services/WorldElevation3D/Terrain3D/ImageServer".
my environment:
OS : WIN11
IDE : VS2022
Programming Language : C# WPF
framework : .net6.0
these are my code, please help me to fix it.
Scene myScene = new Scene(BasemapStyle.ArcGISImagery);
private readonly Uri _elevationUri = new Uri("https://elevation3d.arcgis.com/arcgis/rest/services/WorldElevation3D/Terrain3D/ImageServer");
// Create a surface for elevation data.
Surface surface = new Surface();
surface.ElevationSources.Add(new ArcGISTiledElevationSource(_elevationUri));
// Add the surface to the scene.
MySceneView.Scene.BaseSurface = surface;
// Create a graphics overlay for the scene.
GraphicsOverlay sceneGraphicsOverlay = new GraphicsOverlay()
{
SceneProperties = new LayerSceneProperties(SurfacePlacement.Absolute)
};
MySceneView.GraphicsOverlays.Add(sceneGraphicsOverlay);

Sorry about that, I miss to verify API key.

Sorry about that, I miss to verify API key.


Hidden cursor becomes visible on resize

Windows Terminal version
n/a
Windows build number
10.0.19041.1348
Other Software
No response
Steps to reproduce

Open a WSL bash shell in conhost.
Hide the cursor with printf "\e[?25l"
Resize the window.

Expected Behavior
The cursor should remain hidden.
Actual Behavior
The cursor becomes visible after the window is resized.
The source of the problem is the SCREEN_INFORMATION::_AdjustScreenBuffer method, where the cursor is temporarily hidden while the buffer is resizing. The problem is that it doesn't check whether the cursor was already hidden to start with, so it mistakenly sets it visible when the resize is finished.
https://github.com/microsoft/terminal/blob/bb71179a24ff186eab999676732afff8a345cc12/src/host/screenInfo.cpp#L1023-L1031

Great catch! Sorry, may take a bit to get to your PR :)

@j4james after hiding the cursor, resizing vertically only (grab top or bottom edge) keeps the cursor hidden. Horizontal resize unhides the cursor. Any pointers on why the vertical resize keeps the cursor hidden? Thx!

@elsaco I didn't notice that, but that does make sense. Resizing the window vertically wouldn't usually lead to the buffer being resized (unless the buffer was the exact height of the window). And if the buffer isn't being resized, then this code path is skipped. The test is a few lines up from the initial code I quoted:
https://github.com/microsoft/terminal/blob/bb71179a24ff186eab999676732afff8a345cc12/src/host/screenInfo.cpp#L1014-L1016


Nested validation using Data Annotations

Hello!
It's possible to do nested object validation using Data Annotations along with the Validate method from Nancy?
I'm basically got two scenarios:

A complex object

public class Person : IValidatableObject {

 // I want to validate this object as well.
 public Address Address { get; set; }

}

A list

public class Person : IValidatableObject {

 // I want to validate every item from this list.
 public List<Job> Jobs { get; set; }

}

I see it's possible to call the Validate method for every object separately but is there a better solution?

If your object implements IValidatableObject and the nested objects does the same, why don't you just call the nested object's Validate method as part of the top-level one?

It's a very good option but this can't be done automatically, right?
And changing the subject a little, for my list property: how I'm going to know what list item trigger a validation error? I will need to do a for and return the current index?

Nope. This is a limitation the data annotations validation and not Nancy per se. There are ways to do recursive validation using data annotations, but I don't think that's something we'll add support for in Nancy's validation abstraction. It basically boils down to what the underlying validation framework supports. Maybe FluentValidation supports it?


Changing math font for ranges breaks redering of URLs with Hyperref

Description
If I change the math font for a range of characters, these characters break in URLs set by hyperref.
Check/indicate

[✓] Relevant for XeTeX
[?] Relevant for LuaTeX
[✓] Issue tracker has been searched for similar issues?

Minimal example demonstrating the issue
\documentclass{article}
\usepackage{unicode-math,hyperref}
\setmathfont[range=up/{num}->sf]{Asana Math}
\begin{document}
\url{a1b2c3d4e}
\end{document}

In the output the numbers are replaced by spaces.
Further details
Issue #228 is similar

I think unicode-math should not only add \__um_switchto_up: to \Url@FormatString  but also switch the digits back to the symoperator family: (I'm using here \Url@MathSetup for the patch as it is executed behind the \__um_switchto_up:):
\documentclass{article}
\usepackage{unicode-math}
\usepackage{url}
\setmathfont{Asana Math}
\setmathfont[range=up/{num}->sf]{Asana Math}

\usepackage{xpatch}
\makeatletter
\preto\Url@MathSetup
 {%
  \Umathcode 48=7\symoperators 48
  \Umathcode 49=7\symoperators 49
  \Umathcode 50=7\symoperators 50
  \Umathcode 51=7\symoperators 51
  \Umathcode 52=7\symoperators 52
  \Umathcode 53=7\symoperators 53
  \Umathcode 54=7\symoperators 54
  \Umathcode 55=7\symoperators 55
  \Umathcode 56=7\symoperators 56
  \Umathcode 57=7\symoperators 57
 }
\makeatother

\begin{document}

$1=a$

\url{a1b2/~#c3d4e56789}

\urlstyle{sf}
\url{a1b2/~#c3d4e56789}

\urlstyle{rm}
\url{a1b2/~#c3d4e56789}

\end{document}


Error message description cleanup.

Example

In quite a view cases the error messages are too verbose. This is often when the error originates in the scope but still get prepended by the typechecker. I think we should only generate the error in the typechecker as it has more information (access to the AST).
I don't think we need the message from the scope as it will only fail if the function cannot be found.

All really bad errors are already fixed


create manpage for commands

Generate manpage for cobra commands using cobra's build in tool or ronin as described in Praqma/the-phlow#24

I tried to generate one man page for the entire CLI because I though that was standard procedure. That turned out to be difficult because that is not the standard. Cobra does support generating one man page for the entire command set, it does however support generation a manage for each and every command. Further documentation can be found here https://github.com/spf13/cobra/issues/400


Create issue template - #797

KK, addressed your feedback. I will squash the commits when rebasing. Oh and I myself used a template for this template, thanks for extra checking :)


For a selected item trigger in SharePoint- Run automatically

Hi,
Is it possible to automate the trigger/run 'For a Selected Item' trigger in SharePoint?
A little background:
We are using power automate web API (https://docs.microsoft.com/en-us/power-automate/web-api) to create an approval flow ( called master flow). We programmatically create dynamic flows based on this master template client/flow definition whenever a site gets created in SharePoint Online. These flows appear in the SharePoint site's ellipsis menu and the users can trigger it on demand  by selecting a document. Now, Is it possible to automatically trigger this workflow without the user clicking on 'Run' given that we know the site URL and document ID?
Any help is greatly appreciated. Thank you in advance.

@hamenon-ms Hi Harysh, do you have any thoughts on this?

I assume you want to run it when a new document is added? If so you can use the "When a file is created" for libraries or "When an item is created" for lists and run similar set of automations. We do have existing templates for that scenario like this one: https://flow.microsoft.com/en-us/galleries/public/templates/c641196e198b4598b10aabe61500dfae/start-an-approval-when-a-file-is-added-to-sharepoint/

Hi @hamenon-ms , Thank you for your response. I could use one of them - 'When a file is created' or 'When an item is created' but users need to trigger the workflow for a specific document and not all. The trigger for this approval flow is SharePoint: 'For a Selected File'. So, I am looking to automate the 'Selected File' trigger.
The overall idea is to develop a User Interface where the user can select a SharePoint site from the drop down list which will then pop up another drop down list displaying all documents from the site's document's library. The user will then select a document and click on 'Run' to run the workflow. So I was wondering that since we know the site URL and document ID at this point, if it is possible to run the workflow automatically i.e. Instead of users going to the site and selecting 'Automate -> Approval workflow' from the ellipsis menu? Please let me know how can I accomplish this.
Appreciate your time and help.


Corrected usedGdBlocks calculation

This is a tiny change meant to fix incorrect behavior encountered when doing tar2vhd conversion of a file with an output size of 5gb+.
To be more specific there is bug when writing out a file larger than<PHONE_NUMBER> bytes within compact.go (At least with the default 16gb max file size). In particular when doing the calculation for how many blocks have to be allocated for the group descriptors the calculation can exceed reality leading to  throwing an exceededMaxSizeError{w.maxDiskSize}. The bug probably stems from someone using the wrong variable name but here is the context and justification nonetheless because why not:
Blocks are size 4096 bytes and the information for a group descriptor can be held in 32 bytes of data (that's what groupDescriptorSize is) . This means we can hold 128 group descriptors in a single block which is enough to store the information for 16gbs worth of data or up to 128 groups. However the current calculation indicates different numbers:
32 groups  (4 GiB)
usedGdBlocks := (32-1)/32+ 1 = 1

64 groups (8 GiB)
usedGdBlocks := (64-1)/32+ 1 = 2

128 groups (16GiB)
usedGdBlocks := (128-1)/32+ 1 = 4

129 groups (16.1 GiB):
usedGdBlocks := (129-1)/32+ 1 = 5

If the default max size of 16GiB is used we expect to be able to output 128 groups into a single  group descriptor block so on the later check we would error with anything larger than 4GIB. With the changem things now look like:
32 groups  (4 GiB)
usedGdBlocks := (32-1)/128+ 1 = 1

64 groups (8 GiB)
usedGdBlocks := (64-1)/128+ 1 = 1

128 groups (16GiB)
usedGdBlocks := (128-1)/128+ 1 = 1

129 groups (16.1 GiB):
usedGdBlocks := (129-1)/128+ 1 = 2

Signed-off-by: Esteban Rey<EMAIL_ADDRESS>

@estebanreyl We just turned on some branch protection policies recently so you'll just need to rebase on master again and we should be all good to check in

Awesome, just rebased


Parameter files

Could you please provide the model parameter files after pre-training? I want to use this model for some downstream tasks, but I don’t have enough conditions to pre-train.

Thank you for your interest in our paper.
The pretrained model checkpoint you require has been uploaded and is available for download at Google Drive.
Please cite our article in your work. Thank you for your support.
@inproceedings{zhang2023text,
title={Text-visual prompting for efficient 2d temporal video grounding},
author={Zhang, Yimeng and Chen, Xin and Jia, Jinghan and Liu, Sijia and Ding, Ke},
booktitle={Proceedings of the IEEE/CVF Conference on Computer Vision and Pattern Recognition},
pages={14794--14804},
year={2023}
}

Hi，have you reproduced this paper？


Have readinessProbe call /api/public/ready

Why

Readiness-endpoint has been introduced in the PRs mentioned below.

Implementation: https://github.com/langfuse/langfuse/pull/3222
Documentation update: https://github.com/langfuse/langfuse-docs/pull/788

With this change, the health-endpoint no longer contains the isSigtermReceived check. This might result in the health-endpoint returning 200 OK, even when langfuse is being shut down. However, if there are multiple replicas, terminating langfuse-pods should no longer receive any traffic. Therefore new /api/public/ready-endpoit should be used for readinessProbe

What

Have readinessProbe call new endpoint /api/public/ready

Discussion points

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


re-merge latest master to react-native

@nicastelo i've reset react-native to pre your merge
things were merged incorrectly
please review this with @MathiasMoralesDixtra


Add an option in the Admin page, which let user set their default notebook home dir.

My problem may be a duplication of this issue, but the solution to it is not user-friendly. So I hope we could add an option in the Admin Page. User can set their own notebook home directory easily. How do you think?

It doesn't make sense to put this on the admin page because only admins can see the admin page. This wouldn't allow users to set their own home directory.
If users want to set their own notebook directory, that's actually part of the single-user server, and not part of JupyterHub at all. Setting NotebookApp.notebook_dir in ~/.ipython/ipython_notebook_config.py should do it.

Thanks for your reply.
Actually, when I was saying that put an option on the admin page, I was going to say, maybe, user's config page.
The scenario of needing an option on the page is that, when I was doing python training at client's, I hope to use Jupyter to start an central server, the students could connect to it to do some exercises without installing anything of configuring anything, because they may not have experience of IPython. So making some customisation on the page, instead of modifying some file on remote server, should help them focus on there exercise.

I totally agree with tongda here. This is not the sort of config task you want to go through when you are a starting Python programmer (or any other language). And Jupyter is a great tool for this kind of audience, so it should be considered. Plus, it is a basic must-have configuration that is to be expected of such a tool. Otherwise great work on the project.

Closing this issue since it is resolved through user configuration. Please feel free to leave a new issue at jupyter/help or leave additional questions here. Thanks to all!


[HOLD][$250] Migrate Checkbox.js to function component

Class Component Migration
Filenames

[ ] src/components/Checkbox.js

type: React.Component
has state values: true
has refs: true
has context: false
lifecycle methods: none

Task

We currently have some class components in our codebase that we would like to refactor to a function component.
Here's a link with some general advice on how to refactor a class component to a function component: https://react.dev/reference/react/Component#alternatives
If you need additional guidance, please ask in #expensify-open-source
Test for any regressions and verify that there are no breaking changes

Proposal
Please re-state the problem that we are trying to solve in this issue.
-> Class Component Migration for Checkbox Component
What is the root cause of that problem?
-> class components are more verbose and harder to reason about than functional components.
What changes do you think we should make in order to solve the problem?
-> In the function component I would  use the useState hook to create a state value isFocused.
-> And two functions onFocus and onBlur to update the state value.
-> And also refactor two callback functions handleSpaceKey and firePressHandlerOnClick to handle keydown and click events respectively.
-> Finally Modify the firePressHandlerOnClick function to use the useState hook to update the isFocused state value.
What alternative solutions did you explore? (Optional)

@kuluruvineeth Please stop spamming the issues in this repo. You are free to comment on things that have the Help Wanted label on them.

@marcaaron my apologies, i was not aware that this issues were internal i learnt the same just a while ago. I shall make sure it won't happen again thanks.

Yeah, there's not really any good reason to spam a bunch of issues with similar comments in any case. If you are not sure about something ask in Slack and we are happy to provide assistance there, thanks!

Heads up, I'm taking this issue off HOLD and making it a Daily. If you are unable to work on it please remove your assignment and add the External + Help Wanted labels as per this update.

Nice, I'll push up a PR for this first thing monday

PR is up, working on testing

I love to work on this issue

oh hmm looks like we need a BZ member to issue payment to @aimane-chnaif for reviewing https://github.com/Expensify/App/pull/20193 - adding the appropriate label now

@greg-schroeder kindly volunteered to help out here 🙇

created a job https://www.upwork.com/jobs/~014cc74cd0980ecb9b
sent you an offer for reviewing this @aimane-chnaif

Pending acceptance and then we can close