Issue #24

BQ tables will now have descriptions matching the kafka schema documentation.
Added verification test.

I think @criccomini was also hoping that each description() field of each of the resulting BigQuery Schema's Fields could be populated by the doc() field of each of the corresponding Kafka Connect Schemas (since each Kafka Connect Field comes with its own Schema).
In order to accomplish that, it'd probably make more sense to make this change in the BigQuerySchemaConverter class instead of the SchemaManager, with the top-level Kafka Connect Struct's doc() field being used for the resulting BigQuery Table's description field.

ack on the descriptions of the individual fields.
As for the description of the BigQuery tables, those aren't part of the BQ schemas, so I can't use the schema converters to get the kafka docs into the BQ table descriptions.

Brief thought about the general unpleasantness of having the table documentation conversion happen in SchemaManager rather than SchemaConverter:
One way to get around this would be to have our SchemaConvert actually convert from Kafka Schemas to BQ TableInfos.  This wouldn't have to change the code that much because converting from BQ Schema to TableInfo is just a couple of lines that we can just tack onto the end of the Schema Conversion. I think this would make the code more confusing, though, and I'm not sure the trade-off is worth it.  I'm open to opinions though.

Did some thinking about it and I like your idea of converting from KC Schema to BQ TableInfo. I think if changes have to be made in the future, it's more likely they'll occur with the SchemaManager and less likely (like a lot less likely) with the SchemaConverter, so it would behoove us to keep from complicating the SchemaManager too much.
I also noticed that you have checks for null Field descriptions in the SchemaConverter but not in the SchemaManager. Have you tested what happens when a null description is set for a TableInfoBuilder? And either way, just to be safe, would you consider adding a unit test as a sanity check just in case Google breaks things in a future release?

Okay, after messing with this a bit, my idea of having the SchemaConverts convert to TableInfos doesn't quite work as well as I had hoped.
TableInfos have their BQ TableId in them, so to convert to a TableInfo you would need more than just the kafka schema: you'd also need to know where in BQ you wanted this schema to end up, which is awkward.
I thought maybe we could get around this by passing around TableInfo.Builders instead, containing schema and table documentation but no TableId, that you would later pass a TableId to before building into a full TableInfo, but you can't do that either. TableInfo.Builders require a TableId.  You have to pass one when creating the builder and google's code will verify that it's not null when that happens. Personally I think the idea that you can't put together a builder piecemeal is a poor design decision on google's part, but that's what we are working with.
I have the much smaller change of only setting the table documentation if it's not null though, that I will push here. The rest of the changes that I made will just remain on my box for now, until we make a decision about what we want to do.
I agree that setting the tableDocumentation in SchemaManager is messy, but after all this work for nothing I'm not convinced that alternatives will be better.

Yeah, I definitely wouldn't spend any more time investigating how to put the changes solely into the SchemaConverter. The complexity involved in that is a pretty convincing argument that it's just simpler to allow things to happen in the SchemaManager.
Looks good to me, merging.


templates: improve randInt error when start >= stop

A rather cryptic error pops up when you run randInt a b where a >= b: invalid argument to Int63n. Improve it a bit so that the error is easier to understand without needing to look through Go docs for what Int63n is.

Rebased.


Azure File copy WinRM cannot complete the operation.

Hi,
Not sure what I am doing wrong...
2019-08-13T14:47:17.3599547Z ##[debug]System.Management.Automation.RuntimeException: The remote session query failed for xxx.xxx.xxx.xxx with the following error message: WinRM cannot complete the operation. Verify that the specified computer name is valid, that the computer is accessible over the network, and that a firewall exception for the WinRM service is enabled and allows access from this computer. By default, the WinRM firewall exception for public profiles limits access to remote computers within the same local subnet.  ---> System.InvalidOperationException: <f:WSManFault xmlns:f="http://schemas.microsoft.com/wbem/wsman/1/wsmanfault" Code="2150859046" Machine="fv-az130"><f:Message>WinRM cannot complete the operation. Verify that the specified computer name is valid, that the computer is accessible over the network, and that a firewall exception for the WinRM service is enabled and allows access from this computer. By default, the WinRM firewall exception for public profiles limits access to remote computers within the same local subnet. </f:Message></f:WSManFault>
2019-08-13T14:47:17.3613064Z ##[debug]   at System.Management.Automation.MshCommandRuntime.ThrowTerminatingError(ErrorRecord errorRecord)
2019-08-13T14:47:17.3625879Z ##[debug]   --- End of inner exception stack trace ---
2019-08-13T14:47:17.3680330Z ##[error]The remote session query failed for xxx.xxxx.xxxx with the following error message: WinRM cannot complete the operation. Verify that the specified computer name is valid, that the computer is accessible over the network, and that a firewall exception for the WinRM service is enabled and allows access from this computer. By default, the WinRM firewall exception for public profiles limits access to remote computers within the same local subnet.
2

@greengumby could see if you select the option of "Enable Copy Prerequisites" option in task and re-run it

Hi,
Yes, Enable Copy Prerequisites is already selected.
Thanks
<EMAIL_ADDRESS>Jeremy Martin | Software Development Manager
T +61 8 9211 8675 | M +61 448 811 535
Level 6, 130 Stirling Street, Perth WA 6000 | www.snowdengroup.com
FOLLOW SNOWDEN ON LINKEDINhttps://www.linkedin.com/company/snowden?trk=nmp_rec_act_company_photo
From: Chaitanya Shrikhande<EMAIL_ADDRESS>Sent: Wednesday, 21 August 2019 11:56 AM
To: microsoft/azure-pipelines-tasks<EMAIL_ADDRESS>Cc: Jeremy Martin<EMAIL_ADDRESS>Mention<EMAIL_ADDRESS>Subject: Re: [microsoft/azure-pipelines-tasks] Azure File copy WinRM cannot complete the operation. (#11158)
@greengumbyhttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fgreengumby&data=02|01|Jeremy.Martin%40snowdengroup.com|cd5b9b8d820948de04c408d725eb6ec6|fa4bc6e2c0ed43ec9d6d145e7e36a1a5|0|0|637019565425025189&sdata=XuO6hnbMjjsp6tva%2B3wZyLbEZmbgzdUAN7Zv%2FhrNLGc%3D&reserved=0 could see if you select the option of "Enable Copy Prerequisites" option in task and re-run it
[image]https://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fuser-images.githubusercontent.com%2F11972852%2F63401612-94102280-c3f5-11e9-9312-f39cc36b2313.png&data=02|01|Jeremy.Martin%40snowdengroup.com|cd5b9b8d820948de04c408d725eb6ec6|fa4bc6e2c0ed43ec9d6d145e7e36a1a5|0|0|637019565425035178&sdata=bMNJtHCYu03Wt7m379v%2BJ%2BwIzYQuBHC3izlPNlUpG6I%3D&reserved=0
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHubhttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fmicrosoft%2Fazure-pipelines-tasks%2Fissues%2F11158%3Femail_source%3Dnotifications%26email_token%3DACQGVCQOEH7PPZ3KLZGK4KDQFS4DNA5CNFSM4INYZCL2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD4YLECI%23issuecomment-523285001&data=02|01|Jeremy.Martin%40snowdengroup.com|cd5b9b8d820948de04c408d725eb6ec6|fa4bc6e2c0ed43ec9d6d145e7e36a1a5|0|0|637019565425035178&sdata=%2BJ4B7zkAgXVesAcG%2B8HEkEHiMm67Y%2Fuwr43xZizpxyE%3D&reserved=0, or mute the threadhttps://aus01.safelinks.protection.outlook.com/?url=https%3A%2F%2Fgithub.com%2Fnotifications%2Funsubscribe-auth%2FACQGVCRIJMVYSHVUMOHT5TLQFS4DNANCNFSM4INYZCLQ&data=02|01|Jeremy.Martin%40snowdengroup.com|cd5b9b8d820948de04c408d725eb6ec6|fa4bc6e2c0ed43ec9d6d145e7e36a1a5|0|0|637019565425045177&sdata=lCqqMiK2gFTY4E9Vf3hY5oKiO9bTF1njOgthA2tM4%2Bg%3D&reserved=0.

Scanned for Threats by Messagelabs: http://www.messagelabs.com/email

Any more information on this or alternatives? Currently I cannot copy my artifacts to my Azure VM's for testing.
Thanks

I have the same issue and here's my yaml snippet:
    - task: AzureFileCopy@3
      inputs:
        SourcePath: '$(Pipeline.Workspace)/SomeArtifact'
        azureSubscription: $(Subscription)
        resourceGroup: $(ResourceGroup)
        storage: $(StorageAccount)
        Destination: 'AzureVMs'
        ResourceFilteringMethod: machineNames
        MachineNames: $(TargetHosts)
        vmsAdminUserName: $(TargetHosts.AdminUserName)
        vmsAdminPassword: $(TargetHosts.AdminPassword)
        TargetPath: $(TargetHosts.Path)
        CleanTargetBeforeCopy: true
        enableCopyPrerequisites: true
      displayName: 'Deploy to $(TargetHosts)'

And here's the error for that:
The remote session query failed for xxx.xxx.xxx.xxx with the following error message: WinRM cannot complete the operation. Verify that the specified computer name is valid, that the computer is accessible over the network, and that a firewall exception for the WinRM service is enabled and allows access from this computer. By default, the WinRM firewall exception for public profiles limits access to remote computers within the same local subnet.

Please help!

@coin8086 The lack of support for these pipeline components is really concerning. So much so I am considering jumping off the Azure train.

The lack of support for these pipeline components is really concerning. So much so I am considering jumping off the Azure train.

I agree. Sometimes I have to pull a local copy of the source code of the Pipeline tasks and enable diagnostic info from the Pipeline to figure out why an error comes out, and sometimes I do this just because the document is rather confused or even misleading! That's really annoying!

@greengumby can you try running a powershell task and try to run the invoke-command cmdlet and see if you can make a connection to the target machine.

@greengumby i am closing this issue. Feel free to reopen if you are still facing this issue.

I don't know why this is closed as the issue still exists.

@coin8086 Did you solve/workaround this issue?

@coin8086 Did you solve/workaround this issue?

No, I suspend my deployment with it. I have a lot of work to do and can't afford days trying and diagnosing for their problem. But it seems they don't work on it.

Try running WinRM quickconfig in cmd and check if WinRM service is running and enabled for remote connection. Enjoy !

The same Problem i faced for Azure File Copy, Windows File Copy Tasks.
The following steps has resolved the above issue.

Make a deployment group under the Pipeline --> New Deployment Group.
A Powershell Script will be generated . Make sure to check "Use a personal access token in the script for authentication"
Execute that script using PowerShell in Administrator Privilege in the VM you want to make File Copy Tasks or the Task that uses WinRM Protocol.
The VM will get added to the deployment Group.
Now the tweak here is when selecting the Empty Job in Azure Release Pipeline by default it will add a "Agent Job". But this will not work.
Select the Stage and Click "Add Phase Options" ( three dots ...) to choose the "Add Deployment Group Job"
This will have the option to link the VM you have already added to Deployment Group in Step 4.
Now when you add a New Azure File Copy or Power shell script tasks or any thing that uses the WinRM will work.

This resolved the issue for me. Hope this helps.
My another suggestion would be to add a Step in Azure Release Pipeline to check for the Deployment Group Job be mandatory while choosing the Tasks that uses the WinRM. Or this step should be documents in the Tasks Documentation as Note.

This issue needs to be reopened. These "work arounds" either don't work or are unacceptable.

@coin8086 Did you solve/workaround this issue?

No, I suspend my deployment with it. I have a lot of work to do and can't afford days trying and diagnosing for their problem. But it seems they are not working on it. @arjgupta

Unbelievable. This is unacceptable. Question. Did you open a support case with Microsoft? I'm opening one, RIGHT NOW. I've been using Azure File Copies in my build pipeline (though it is still a huge pain), but can't get them working in the release.
I've confirmed that WinRM is enabled, and my SP has the correct permissions. I'm not going through my release to put everything in a deplpoyment group. This is NOT what deployment groups should be used for.


fixes some issues with argonaut sprint 6 and adds a sprint 7, also ad…

…ds unit tests around sprints 6 & 7

General question: does it make sense to generalize ArgonautSprint6Test, and then subclass it for ArgonautSprint7Test, just changing the LOINC codes/etc.?

There are some additional requirements for sprint 7 where it needs to be able to handle composite labs as well... so it is not as simple as replacing the loinc codes.  There is some code that could be extracted out, but probably not enough to justify creating a parent-child hierarchy


Allow module to be directly required

Allow module to be directly required without needing a replace directive.
This directly affects cli/cli#1389.
For reference, it might be good to consider pushing this type of thing upstream, e.g., shurcooL/graphql#53.

@mislav I have a strange hackish work around if you want to fix cli/cli#1389 while getting out of the forking dependency business and still supporting the likes of cli/cli#997.
Did you know you can cherry pick PR commits as they do go into the repo (but are not associated with any branch or tag)? The key here is that these are the same:

a4a48d3af0f4d1e937f58787378e657fd3bff388
shurcooL/graphql@a4a48d3af0f4d1e937f58787378e657fd3bff388

Here are the steps I believe will resolve the issue and more importantly allow you to get out of the forking dependency business:

First update the cli/cli#1335 code to support your changes from shurcooL/graphql#57 (moving from 0f7232a2bf7e6d8f393025a8f5cafbd219a7ebeb to a4a48d3af0f4d1e937f58787378e657fd3bff388). Presumably you know how to do that or you would have not have submitted it as such. This will likely need to be done sooner or later anyway as it seems you are headed that way.
Fix cli/cli#1389 by removing the replace directive and stop referring to this forked repo (keep it for your PR work though) in go.mod by changing the require to refer to: github.com/shurcooL/graphql v0.0.0-20200728183420-a4a48d3af0f4 instead of github.com/shurcooL/graphql v0.0.0-20181231061246-d48a9a75455f.
Maintain shurcooL/graphql#57 since you opened the PR, you can submit updated commits. If you want to pull in something adding on the master branch just rebase your changes and submit the new commit to the PR and then update cli/cli go.mod require to the rebased commit in the PR again.

For example:  shurcooL/graphql@a4a48d3af0f4d1e937f58787378e657fd3bff388 has shurcooL/graphql@d48a9a75455f6af30244670bc0c9d0e38e7392b5 from Dec 30, 2018 as a parent but you could rebase your PR changes off of shurcooL/graphql@18c5c3165e3af7c70ca625593f8066462ad684cc and update/push back to the PR to pull in the new changes on master through Sep 27, 2020 (in this case those changes are not essential to cli/cli but you get the idea).
This lets you maintain sort of an illicit pseudo-branch in the target repo even without commit permission, etc.

Going to close this as https://github.com/cli/shurcooL-graphql/pull/3 will address this and allow the module to be required.


Compressing videos

Hi!
I'm trying to compress videos with your library, and at first the app was crashing, but after I read through some of the closed issues it's not crashing anymore, but it's also not working to compress the video.
That's how I'm trying to do it:
Controller.getInstance().run(MediaUtils.getCompressCommand(path, activity, newFileName));
public static String[] getCompressCommand(String path, Context context, String newFileName) {         return new String[]{"-y", "-i", path, "-s", "640x480",                 "-b:v", MediaUtils.getBitrateFromVideoResolution(context, path),                 "-strict", "-2", "-c:v", "libx264", "-preset", "ultrafast", "-crf", "18",                 "-c:a", "aac", "-threads", String.valueOf(SystemUtils.getNumberOfCores()),                 "-ac", "2", "-ar", "44100", "-b:a", "128k",                 "-f", "mp4", newFileName};     }

I am not able to reproduce this. Can you please tell me the values of MediaUtils.getBitrateFromVideoResolution(context, path)
SystemUtils.getNumberOfCores()
And possibly the video you are actually trying to compress?
Do you see any error message in logcat?

Thanks for the quick response!
The bitrate used is 1M and the number of cores is 2.
The video I tested:
test_compress.mp4.zip
I don't see any error messages in the logcat

@madhavanmalolan : Getting the same issue, I started the video compressing but nothing is happening.
The video was about 3.5mb.
This is the command i used,
val complexCommand = arrayOf<String>("ffmpeg", "-y", "-i", filePath!!, "-strict", "experimental", "-vcodec", "libx264", "-preset", "ultrafast", "-crf", "24", "-acodec", "aac", "-ar", "44100", "-ac", "2", "-b:a", "96k", "-s", "320x240", "-aspect", "4:3", outputFile)

Controller.getInstance().run(complexCommand)

Can you please check?

Have either of you @FranciscoNin @HardikChauhanSA found a solution to this?
I have not been able to debug this issue :disappointed:

I used to compress the video process with the below comments, it works for up to Nougat 7.0 (API level 24), Latest Android versions stopped working. (Versions are
Oreo | 8.1.0 | API level 27
Oreo | 8.0.0 | API level 26
Nougat | 7.1 | API level 25 )
Package Name: Xamarin.Android.FFmpeg
Version: 1.0.32
string[] cmds = new string[] {
"-y",
"-i",
inputFilepath,
"-strict", "experimental",
"-vcodec", "libx264",
"-preset", "ultrafast",
"-crf","30", "-acodec","aac", "-ar", "44100" ,
"-q:v", "20",
"-vf",
"mp=eq2=1:1.68:0.3:1.25:1:0.96:1",
ouputFile,
};
string cmdParams = string.Join(" ", cmds);
FFMpeg.Xamarin.FFMpegLibrary.Run(Xamarin.Forms.Forms.Context, cmdParams, loggerAction);
Can you please help me on this issue?

@madhavanmalolan I gave up and used the ffmpeg4android library, but even by tweaking every possible command combination I wasn't able to compress in less than 30 seconds for a 3 seconds video.
What I made is to make my own camera in app so I would recordo only videos in a lower resolution with a max size of 22mb (in a 1min video), so I wouldn't need to compress it before sending to the server.

@FranciscoNin Having the same issue. Thanks for the sharing your notes. Is your custom camera support to latest Android OS 9.0 Pie? Do you have sample code on GitHub as open source? can you share me that link that would be helpful?

@khetiyachintan sure, I did that recently on an issue from the ffmpeg library for Android.
There I explained what I tried to achieve with that library and what I ended up doing with a custom camera view.
Here:
https://github.com/WritingMinds/ffmpeg-android-java/issues/265#issuecomment-441251261
Hope it helps. If there's anything else I can do to help, please let me know.
Cheers


fix(default): Now default scripts will be a fallback only

What:
This adds logic to check first for non-default scripts, then fallback on
default scripts if none are found that satisfies the input
Why:
Because #25
How:
Create a clone of the config object and remove all default scripts,
check the prefix with that, if none is found attempt again with the
original config object.
Closes #25
BREAKING CHANGE: You're unlikely to have this problem, but if you had a
script that matched the prefix def for example, the default script
would be run. Now, your specific script will be run instead.

LGMT

LGTM

[Current coverage][cc-pull] is 100% (diff: 100%)

Merging [#39][cc-pull] into [next][cc-base-branch] will not change coverage

@@           next   #39   diff @@
=================================
  Files         7     7
  Lines       137   148    +11
  Methods       0     0
  Messages      0     0
  Branches      0     0
=================================
+ Hits        137   148    +11
  Misses        0     0
  Partials      0     0

Powered by Codecov. Last update [a9a6a62...d092ca2][cc-compare]
[cc-base-branch]: https://codecov.io/gh/kentcdodds/p-s/branch/next?src=pr
[cc-compare]: https://codecov.io/gh/kentcdodds/p-s/compare/a9a6a6231f340ad77744ab8976fc2809a706dd2a...d092ca28490eb5e603f72f40beab372f137da06e?src=pr
[cc-pull]: https://codecov.io/gh/kentcdodds/p-s/pull/39?src=pr


Disabling stash option?

Hello!
I have seen new awesome feature of stashing and unstashing changes in version 8+
But is there any option for disabling such behaviour?) We all are used to old one(and stashing seems slow)
I couldn't find a flag to disable it

No there is no such option and I don’t plan to support one. You can just stick to v7 if you don’t like it.

It’s the way it’s implemented in git since we don’t use native stash.

There is a PR that uses native stash. You could try it out and see if that works better and leave your feedback there.


download

brother disney will behead you

dont know why i opened this as a issue sorry.
might bite you in the arse later though.


NAS-130585 / 25.04 / Exclude nvidia-related changes from /usr daatset diff

This PR has been merged and conversations have been locked.
If you would like to discuss more about this issue please use our forums or raise a Jira ticket.


GPMDP makes wingpanel unresponsible and sometimes crashes

OS:
elementaryOS Loki 64 bit (beta)
Issue Descriptions:
When opening your app the top bar indicators stop working until I close it. Terminal doesn't give any output besides Gtk-Message: Failed to load module "pantheon-filechooser-module"  which I think is unrelated since it's a usual warning with some other apps.
Changing settings inside the app, such as "use system window borders" or "minimize to tray" doesn't solve the problem.
Everything worked fine back in elementaryOS Freya.
Steps to Reproduce:
Install elementary OS Loki and install GPMDP. Wingpanel stops working and crashes after a while. App continues working as expected.

Hmmmm, this sounds like either an Electron problem or a Loki problem.
Can you try running any other electron application that use tray icons and see if they also cause the crash
/cc Linux dude @jostrander

Can you give me an example of those apps? I've tried with Brave  and Mancy but I don't think those use tray icons.
By the way, if this is a tray icon issue it may be because the elementary made some changes and dropped support for Ayatana. From their blog :

No More Ayatana
The brand new set of wingpanel indicators has landed and Ayatana is no more. If your app is making use of the Ayatana AppIndicator API, now is the time to stop. The FreeDesktop Actions spec is the proper way to have quick application actions. These appear in the dock, when right-clicking apps in the applications menu, and also in the applications menu search. If users want your app to be quickly accessible, they can pin it to the dock where you can use the LauncherEntry API to show badges and progressbars. Notifications from your app will now persist in the notifications indicator, and as always music players making use of MPRIS will show in the sound indicator. The focus here is on empowering users to choose which apps are allowed to be omnipresent on their desktops and keeping the indicator area useful and clutter-free. There is a Wingpanel API, but this API should be used only by permanently-running, system-level utilities (ie redshift); not applications (ie skype).

@Maccer1 Try installing the desktop client for Slack
That has a tray icon

No problem with Slack.

Can you give a good explanation of what wingpanel actually does.  I think I have a misunderstanding of it's purpose

Wingpanel is the top bar of elementaryOS. From Loki beta they decided that developers shouldn't use it except for system-level apps.
I'm not sure how this app crashes with Wingpanel since there wasn't any icon in the top bar. I don't remember having any icon back in Freya. My guess it's that it has something to do with the way the app interacts with the media controls that are inside sound-indicator.
Does GPMDP use MPRIS ? If not, that may be the issue I think.
If you tell me how, I can give you some logs about what is happening.

Yes we follow the MPRIS spec
It works correctly with literally every other linux OS, so Loki must have a bad implementation of the MPRIS spec.  Are there any like crash logs or something?

I'm not familiar with elementaryOS, however I believe it's a modified debian distribution. Maybe check /var/log/syslog for any errors related to the issue at hand?

As a follow up to this, I tried getting a copy of the Loki beta, but my google-fu must be pretty bad because I couldn't find a download. Need some kind of log on this otherwise we can close it as distro specific upstream bug.

Thanks I didn't know where to find that log. I've been asking on their subreddit but I didn't got any response. Anyway here's the output related to the crash:
 Jun 25 18:56:03 Maccer-PC org.gnome.gedit[1253]: (gedit:10587): GVFS-WARNING **: can't init metadata tree /home/nicolas/.local/share/gvfs-metadata/root: open: Permission denied Jun 25 18:56:12 Maccer-PC org.gnome.gedit[1253]: message repeated 2 times: [ (gedit:10587): GVFS-WARNING **: can't init metadata tree /home/nicolas/.local/share/gvfs-metadata/root: open: Permission denied] Jun 25 18:56:12 Maccer-PC org.gnome.gedit[1253]: ** (gedit:10587): WARNING **: Set document metadata failed: Error setting file metadata: can't open metadata tree Jun 25 18:56:17 Maccer-PC gnome-session[1172]: Gtk-Message: Failed to load module "pantheon-filechooser-module" Jun 25 18:56:20 Maccer-PC gnome-session[1172]: Window manager warning: Buggy client sent a _NET_ACTIVE_WINDOW message with a timestamp of 0 for 0x3a00001 (Google Pla) Jun 25 18:56:37 Maccer-PC kernel: [ 7339.623540] show_signal_msg: 31 callbacks suppressed Jun 25 18:56:37 Maccer-PC kernel: [ 7339.623547] wingpanel[1509]: segfault at 0 ip 00007f03eda8a770 sp 00007ffcc8ebf3d8 error 4 in libglib-2.0.so.0.4800.1[7f03eda51000+10f000] Jun 25 18:56:38 Maccer-PC gnome-session[1172]: ** Message: ProcessWrapper.vala:99: Process 'wingpanel' watch exit Jun 25 18:56:38 Maccer-PC gnome-session[1172]: ** Message: ProcessWrapper.vala:111: Process wingpanel is being terminated Jun 25 18:56:38 Maccer-PC gnome-session[1172]: ** Message: ProcessWrapper.vala:123: ET = 7333.377349 secs#011Min allowed time = 3.500000 Jun 25 18:56:38 Maccer-PC gnome-session[1172]: ** Message: ProcessWrapper.vala:46: Crash count of 'wingpanel' has been reset Jun 25 18:56:38 Maccer-PC NetworkManager[778]: <info>  [1466873798.6453] audit: op="device-autoconnect" arg="autoconnect:1" pid=10712 uid=1000 result="success" Jun 25 18:56:39 Maccer-PC gnome-session[1172]: ** (process:10777): CRITICAL **: string_contains: assertion 'self != NULL' failed Jun 25 18:56:39 Maccer-PC gnome-session[1172]: ** (process:10777): CRITICAL **: string_contains: assertion 'self != NULL' failed Jun 25 18:56:40 Maccer-PC org.gnome.gedit[1253]: (gedit:10784): GVFS-WARNING **: can't init metadata tree /home/nicolas/.local/share/gvfs-metadata/root: open: Permission denied
The elementaryOS beta is here

And, I think the problem is in this part:

Jun 25 18:56:20 Maccer-PC gnome-session[1172]: Window manager warning: Buggy client sent a _NET_ACTIVE_WINDOW message with a timestamp of 0 for 0x3a00001 (Google Pla)
Jun 25 18:56:37 Maccer-PC kernel: [ 7339.623540] show_signal_msg: 31 callbacks suppressed
Jun 25 18:56:37 Maccer-PC kernel: [ 7339.623547] wingpanel[1509]: segfault at 0 ip 00007f03eda8a770 sp 00007ffcc8ebf3d8 error 4 in libglib-2.0.so.0.4800.1[7f03eda51000+10f000]
Jun 25 18:56:38 Maccer-PC gnome-session[1172]: * Message: ProcessWrapper.vala:99: Process 'wingpanel' watch exit

I think the "Buggy client" thing is the real issue, however this seems to be out of our control. Can you raise in issue in electron? The window manager libraries would be something on their end I believe.

Done

If you happen to have this same bug someday, this may be interesting:

The "Window manager warning: Buggy client" warning comes from gnome-session and it happens for lots of popular apps. The real problem is wingpanel crashed, and I think you should report the bug to wingpanel, even though it may be caused by Electron sending wrong messages.

So it seems like this is something that just is with gnome-session. If you do submit this over to wingpanel add a link back to here so people that might stumble on this in the future might be able to follow it up. But seeing as other OS's with gnome-session work fine, I'd consider this to be an issue with loki specifically.

@Maccer1 @jostrander I'm going to close this as an upstream issue that we have zero control over.  If you could update this issue if / when you get a response from the Loki team

Well I got some news. This issue is fixed on Loki beta 2. I didn't get any response on my bug posted in the elementary bug tracker but anyway its fine now.


Ambiguous description of ambiguous syntax

This page notes that because casting uses an identifier surrounded by parentheses, some expressions may be ambiguous. It's not clear, though, what implication ambiguous syntax has for a programmer. Is there a possibility of the compiler interpreting it one way, but later being updated to interpret it another way? Should the developer take care when using third-party compilers, such as that for Mono? Or is the interpretation well-specified for any ECMA-334 compliant compiler?
For the example given, (x)-y, it looks like the current behavior (VS 2017 15.5.5, C# 7.2) actually does disambiguate the cases:

If identifiers x and y both denote variables, there is no compiler error or warning, and the expression is treated as x - y (y subtracted from x).
If identifier x denotes a type and y denotes a variable, the compiler emits error CS0075 (even if an explicit conversion exists). This error cites the language specification section on cast expressions, which repeats this example and notes that there is a disambiguation rule that the compiler is following.

Given that there is a disambiguation rule in the spec, should this page even mention that there could be an ambiguity? It seems like "ambiguous grammar" is an implementation detail resolved by an additional rule on the compiler. Or is there another reason to caution developers when using the cast operator?

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

ID: 2fe11a11-230c-eebb-9a64-e22d5d73094d
Version Independent ID: 05d2b6e1-d471-7baa-ed9e-79d72d99bd4a
Content: () Operator (C# Reference)
Content Source: docs/csharp/language-reference/operators/invocation-operator.md
Service: unspecified
Product: dotnet-csharp
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Thanks for pointing this out. I'm doing some research to determine exactly what the fix should be.
In the meantime, the lesson is to pick different names for variables and types. 🤷♂

We should remove this sentence. You have to write some very contrived code to demonstrate this issue:
public class C
{
}

public static class Program
{
    public static void Main()
    {
        var C = 5;
        var y = 8;
        var z = (C)-y; // Hmm. Is 'C' the type, or the variable?
    }
}

The compiler decides that C is the variable, but it could be parsed either way.


Google_Service_Analytics::management_accounts::listManagementAccounts() result is not right

When calling
 $accounts = $analytics->management_accounts->listManagementAccounts();

as found in documentation, $accounts contains all accounts which contain the app's client_email as an entry in their User Management section of Google Analytics Admin. In it's current state, $accounts is not limited to just the accounts that belong to the authenticated user, but rather all accounts which this app has been given access to.
Before the recent upgrade, this function returned back all accounts which belong to the authenticated user

As stated in the documentation [Accounts: list] (https://developers.google.com/analytics/devguides/config/mgmt/v3/mgmtReference/management/accounts/list). Lists all accounts to which the user has access.  This is the current authenticated user.  Each request you send against the google api is against the authenticated user.   It has nothing to do with the authenticated applications access.
Even the old legacy v2  Account Feed The account feed is the entry point to all management information for the authenticated user.
This is something ingrained in OAuth and not related to the PHP client library.  If anything it would be related to the Google Analytics API.
The results returned by the API are correct.

Your reply makes sense. When I downgrade to v1, it works as expected for me without changing any code, surprisingly. When calling the service from the OAuth2 playground it also works as expected.
That's why I suspect it must be internal code at fault. I will look more into why this is happening within the scope of this project and return.

Tip:  Account Summaries: list   will get back all the Accounts -> web properties -> and views that a user has access to in one call :)

It looks like a member of our great community answered your questions. Thanks Community! Please feel free to reopen if you have any more questions.


Unwind support on ARM architecture

WIP support for ARM, can do backtrace on basic functions. Eventually, this PR will solve https://github.com/nbdd0121/unwinding/issues/3.
For a working (backtrace) example you can check https://github.com/arturkow2000/qemu-example/pull/1.
You need to
export DEFMT_LOG=trace
export RUSTFLAGS=-Cforce-unwind-tables=on

before building app. For details see README in the repo.

Was there a particular reason that this was closed?


style: 💄 update SQFormAutocomplete styles

Updated sizing and weight for label and value text. Switched success and warning colors in validation to new 'textWarningGYellow' and 'textSuccessGreen' styles.
Update the current implementation of the Autocomplete with added specifications. Here is the current implementation of this SQForm: Webpack App (this is an example link)
Design reference: SC3 DESIGN LIBRARY
Typography: SC3 DESIGN LIBRARY
Color: SC3 DESIGN LIBRARY
Default:
Label: Text Style: Label 12 px Weight: 400
Value: Text Style: Value 14px Weight: 600 Semi-bold
Focused:
Label: Text Style: Label 12 px Weight: 400
Value: Text Style: Value 14px Weight: 600 Semi-bold
With Validation:
Label: Text Style: Label 12 px Weight: 400
Value: Text Style: Value 14px Weight: 600 Semi-bold
Line: #C67000 Warning
Helper Text/icon: #C67000 Warning #3C7107 Text/Success
Icon Size: 12 x 11 Warning 9 x 11 Success badge
✅ Closes: #836
Loom Demo: here

So do we plan on making updates to other components? I'm just a little concerned about changing CSS for some fields and not others. If we eventually want changes for all components we could update the theme.
I guess I'm basically just asking what the main driver behind this ticket is,.

I guess I'm basically just asking what the main driver behind this ticket is,.

This and most of the SSR tickets right now are to get all the components in line with the SC3 Design Library. Carlos and I have been splitting it out into components to make sure all the aspects are checked. A lot of the colors would be the same across components so looking into updating the theme is probably a good idea.

A lot of the colors would be the same across components so looking into updating the theme is probably a good idea.

Would it be better to pass theme elements instead of making reference to the root.css file variables, and then hold off on merging this until the theme in scplus-shared-components is updated overall?  Once we can test and confirm that the styles have applied correctly, we could then merge?


%%Javascript IPython.notebook.save_notebook() executing successfully but notebook is not saved

Applies To

[X] Notebooks (.ipynb files)
[ ] Interactive Window and/or Cell Scripts (.py files with #%% markers)

What happened?
Running following cell, I expect the notebook to save
IPython.notebook.save_notebook()

The cell is executing without error but notebook is not saved.
VS Code Version
1.92.2
Jupyter Extension Version
v2024.7.0
Jupyter logs
No response
Coding Language and Runtime Version
Python 3.12
Language Extension Version (if applicable)
No response
Anaconda Version (if applicable)
No response
Running Jupyter locally or remotely?
Local

Thank you for coming this issue
Unfortunately we do not support access to the ipnb notebook via the python code in this manner


Christoffel symbols of the first kind ill calculated - Calculation of the Christoffel symbols of the second kind returns an error

Dear All,
Galgebra does not seem to obtain the correct Christoffel symbols of the first kind; at least the values printed out indicate a skew symmetry, which is strange.
Starting with the metric tensor relating to a two dimensional manifold,  the unit sphere, viz.,

galgebra yields the following components for the Christoffel symbols of the first kind,

when, in fact, the correct values are

As is well known, these symbols are symmetrical in the indices alpha and beta.
As for the Christoffel symbols of the second kind they are not calculated, an error is returned instead:
“
C:\Users\Freddy\Desktop\Issues\Christoffel>C:\Anaconda3\python "Christoffel_symbols_checking.py"
Traceback (most recent call last):
File "Christoffel_symbols_checking.py", line 47, in
Cf2 = sp2.Christoffel_symbols(mode=2)
File "C:\Anaconda3\lib\site-packages\galgebra\metric.py", line 490, in Christoffel_symbols
Gamma_ijk += Gijl * self.g(l,)
TypeError: 'MutableDenseMatrix' object is not callable
C:\Users\Freddy\Desktop\Issues\Christoffel>pause
Press any key to continue . . .
“
It is not at all clear how the differential operators can be properly computed  (see zip file enclosed)
when it is explicitly stated in Section 2.3.1 of Galgebra: a Geometric Algebra Module for Sympy (October 8, 2016) that the Christoffel symbols of the first kind are made use of to perform a number of calculations, notably to obtain the derivatives of the basis vectors (footnote on page 17). Unless the Christoffel symbols are not actually used at all! This would explain that, but I can be totally wrong on this.
Look forward to receiving any views on all this.
Kind regards,
FB
<EMAIL_ADDRESS>Christoffel.zip

It is not at all clear how the differential operators can be properly computed (see zip file enclosed)
when it is explicitly stated in Section 2.3.1 of Galgebra: a Geometric Algebra Module for Sympy (October 8, 2016) that the Christoffel symbols of the first kind are made use of to perform a number of calculations, notably to obtain the derivatives of the basis vectors (footnote on page 17). Unless the Christoffel symbols are not actually used at all! This would explain that, but I can be totally wrong on this.

Historically, derivatives_of_basis() is implemented before Christoffel_symbols() and the latter is currently not covered by any examples or tests, see https://codecov.io/gh/pygae/galgebra/src/15-print-pow/galgebra/metric.py#L426 .
Christoffel_symbols() was implemented in commit https://github.com/rschwiebert/galgebra/commit/e9177cc128db2eb17a1fe79fb7cdb2d6285c9cf3 and not changed afterwards.
And there's duplicate code in derivatives_of_basis() for calculating Christoffel symbols of the first kind.
This might partially answers your doubt.

@FreddyBaudine According to the implementation, indices need to be interpreted as  Christoffel_symbols()[i, j, k] = \Gamma_{kij}. In this interpretation, the result is correct and identical to the result given by Maple.

On the other hand the Christoffel symbols of the second kind are not implemented correctly, working on a fix.

@FreddyBaudine I've just recreate the same form in your Maple file, by:
Cf1 = sp2.Christoffel_symbols(mode=1)
Cf1 = permutedims(Array(Cf1), (2, 0, 1))
print(r'\text{Christoffel symbols of the first kind: }')
print(r'\Gamma_{1, \alpha, \beta} = ', latex(Cf1[0, :, :]), r'\quad', r'\Gamma_{2, \alpha, \beta} = ', latex(Cf1[1, :, :]))

The result looks like this:

Dear Utensil,
Many thanks for all this. Your doing a fantastic job.
I tried your solution with Python 2.7, 3.4 and 3.7 together with version 1.3 of Sympy. Everything turned out alright.
Kind regards,
FB

Dear Freddy,
I'm glad it checks out and it's a pleasure to be helpful to your work. Also thanks for your verification on multiple platforms which are not verified on CI yet (only 2.7 and 3.6 for now).
P.S. Did you notice my invite for you to join our discussion on https://clifforddev.slack.com/ ? I sent the invitation to your email address mentioned in the main thread of this issue.
Regards,
Utensil

Dear @FreddyBaudine ,
The calculation of Christoffel symbols of the second kind has been implemented in https://github.com/pygae/galgebra/blob/15-print-pow/galgebra/metric.py#L467 .
And a equivalent result of your script has been implemented in https://github.com/pygae/galgebra/blob/15-print-pow/examples/LaTeX/christoffel_symbols.py#L48 and it looks like this:

It's slightly different from your Maple file, because it simplified cos(theta)/sin(theta) to 1/tan(theta).
In the process, I've refactored it quite a bit, so now derivatives_of_basis() actually use Christoffel_symbols() instead of duplicated but slightly different code.
Also I would like the code to look like the original math formulas because reading the original code style can't help me eyeball the correctness of the implementation which I believe it's much improved now. Please take a look, comments welcome!
Regards,
Utensil

Dear Utensil,
As before, I tried your solution with Python 2.7, 3.4 and 3.7 together with version 1.3 of Sympy. Everything turned out alright.
Kind regards,
Freddy

Dear @FreddyBaudine,
Thanks for testing the solutions. Please note that galgebra currently have issues with SymPy 1.4, see #31 .
Have you verified the calculation of Christoffel symbols (both for the first and second kind) for other metrics? Although I'm sure the implementation is correct, but it's better to know that it works for more scenarios.
There's one more thing I need to ask you: During the fix of this issue, I added your code to the test cases and examples of glgebra, it's fine for an unmerged PR #17, but it might have licensing issue if they become part of galgebra. What do you think? Do we have your permission to add them into galgebra test cases and examples?

Dear Utensil,
I checked your corrections with higher dimension metrics and, unfortunately, the Christoffel symbols of the second kind are not computed correctly in some cases.
As far as I can judge, this mishap happens only if the matrix of the metric possesses off-diagonal elements. That’s why I tried two such metrics, viz., the rotating frame metric and the Kerr-Debney metric for they are not too complicated. I will test the Kerr metric eventually.
It appears that, in all cases envisaged, the Christoffel symbols of the first kind are computed correctly.
I did all this making use of Python 3.7 and Sympy version 1.3
Kind regards,
Freddy
PS: A priori, no license issue as this work is strictly personal. Hence, you can add my code in galgebra test cases and examples.
37.zip

Dear @FreddyBaudine,
Great catch!
Currently I have no idea where went wrong, because both the metric inverse code and the Christoffel symbols of the second kind code seem ok to me. The metric inverse code is possibly the primary suspect.
I'm cross-checking the implemtation and test cases in galgebra and sympy.diffgeom.

A priori, no license issue as this work is strictly personal. Hence, you can add my code in galgebra test cases and examples.

That would be greate, thanks a lot!
Regards,
Utensil Song

Also I intend to verify the implementation against some cases in https://github.com/spacetimeengineer/spacetimeengine/blob/master/spacetimeengine/src/solutions.py .

I've checked the Kerr-Debney metric in your attached script, it turns out the the Christoffel symbols of the second kind is correctly calculated. They don't look identical to the Maple one because of two reasons:
First, we can't use 1/2 to express half in SymPy but should use Rational(1, 2) instead to avoid being treated as 0.5 and also the bug that galgebra will translate it to something like 0 \cdot 5 for all decimal numbers.
You can do
from galgebra.mv import ONE, ZERO, HALF

and write the metric as:
g = [[0, 0, -exp(-z), 0],
     [0, HALF*u**2*exp(4*z), 0, 0],
     [-exp(-z), 0, 12*exp(-2*z), u*exp(-z)],
     [0, 0, u*exp(-z), HALF*u**2]]

Now it will be much easier to see the next reason.
Secondly, if we write:
Cf2 = Cf2.normalized()

Then the result is identical to the Maple one:

Dear Utensil,
Very many thanks for your input.
I tried to reproduce your solution but I got the following error with Cf2.normalized() :
“
C:\Users\Freddy\Desktop\Issues\Christoffel\Higher dim\37>C:\Python37\python "Christoffel_kerr_debney_metric37.py"
Traceback (most recent call last):
File "Christoffel_kerr_debney_metric37.py", line 48, in
Cf2 = Cf2.normalized()
AttributeError: 'list' object has no attribute 'normalized'
C:\Users\Freddy\Desktop\Issues\Christoffel\Higher dim\37>pause
Press any key to continue . . .
“
How come I got this error when you did not?
For your convenience I have enclosed the modification in a zip file as I may have made some mistake.  If this is the case, will you be so kind as to forgive me to have wasted your time.
You will have guessed that I am not quite competent with Python. I am much more familiar with Maple, Mathematica and a little bit of maxima.
Thanks again to take the time to answer my queries.
Kind regards,
FB
Christoffel_kerr_debney_metric37.zip

@FreddyBaudine Oh I forgot to mention thatnormalized() should be called after permutedims() call because after that it will be an array and it will have such method.
Sent with GitHawk

Dear Utensil,
Many thanks for your quick reply.
I placed the instruction Cf2 = Cf2.normalized() after  the instruction
Cf2 = permutedims(Array(Cf2), (2, 0, 1)) as you suggested.
Now I get the following error :
“
C:\Users\Freddy\Desktop\Issues\Christoffel\Higher dim\37>C:\Python37\python "Christoffel_kerr_debney_metric37.py"
Traceback (most recent call last):
File "Christoffel_kerr_debney_metric37.py", line 49, in
Cf2 = Cf2.normalized()
AttributeError: 'ImmutableDenseNDimArray' object has no attribute 'normalized'
C:\Users\Freddy\Desktop\Issues\Christoffel\Higher dim\37>pause
Press any key to continue . . .
“
I trust you will find what’s wrong.
Kind regards,
FB
Christoffel_kerr_debney_metric37_Corrected.zip

Sorry, my mistake.
At first, I noticed galgebra's output is different by a factor of * (-4 * exp(-2 * z) * u**(-4)), so I used
Cf2 = Cf2 * (-4 * exp(-2 * z) * u**(-4))

to achieve the expected result. But I think this is caused by lack of normalization so I added the normalized() call but it's actually incorrect. And unfortunately I ran the script by repeating the last command which actually didn't run the script and only open the last pdf file generated by  Cf2 = Cf2 * (-4 * exp(-2 * z) * u**(-4)).
Sorry for the inconvenience caused and I was on other computer and didn't have access to the script and gave the wrong guide based on impression.
So, I need to find the actual fix.

I believe we should use norm=True when initializing the GA object, but it will raise an error:
  File "~/Christoffel_kerr_debney_metric37.py", line 23, in <module>
    g4 = Ga('eu ex ey ez', g = g, coords = g4coords, norm=True) # Create g4
  File "~\projects\pygae-galgebra\galgebra\ga.py", line 325, in __init__
    metric.Metric.__init__(self, bases, **kwargs)
  File "~\projects\pygae-galgebra\galgebra\metric.py", line 700, in __init__
    self.e_norm.append(square_root_of_expr(self.g[i, i]))
  File "~\projects\pygae-galgebra\galgebra\metric.py", line 108, in square_root_of_expr
    (coef, pow_lst) = sqf_list(expr)
  File "~\Miniconda3\envs\py3\lib\site-packages\sympy\polys\polytools.py", line 6216, in sqf_list
    return _generic_factor_list(f, gens, args, method='sqf')
  File "~\Miniconda3\envs\py3\lib\site-packages\sympy\polys\polytools.py", line 5988, in _generic_factor_list
    raise PolynomialError("a polynomial expected, got %s" % expr)
sympy.polys.polyerrors.PolynomialError: a polynomial expected, got 12*exp(-2*z)

I read the code a little bit, it seems to be a bug and it seems that it will take quite some time to fix this bug.
Please simply times det(g)^-1 like this for now:
Cf2 = g4.Christoffel_symbols(mode=2)
Cf2 = permutedims(Array(Cf2), (2, 0, 1))
Cf2 = Cf2 * (g4.g.det() ** -1)

Dear Utensil,
Thank you for the workaround.
I trust you won’t be surprised of the following. As the Christoffel symbols of the second kind are calculated correctly when the matrix of the metric is diagonal, the workaround should not work in that situation. Sure enough, this is the case; all the components get divided by the determinant. I’ve tested this with the Schwarzschild metric.
Kind regards,
FB

The direct cause is found. The Christoffel symbols of the second kind relies only on the Christoffel symbols of the first kind and the inverse of metric. The former is correctly calculated, so I initially suspected the inverse of metric, but when looking at your pdf, the inverse of metric seems to be correct. Unfortunately, it's correct simply because yours is calculated by simply g.inv() and galgebra did something completely different.
At first I thought the problem is in Mv.inverse_metric() but it looks logical:
    def inverse_metric(self):

        if self.g_inv is not None:
            return

        if self.is_ortho:  # Orthogonal metric
            self.g_inv = eye(self.n)
            for i in range(self.n):
                self.g_inv[i,i] = S(1)/self.g(i,i)
        else:
            if self.gsym is None:
                self.g_inv = simplify(self.g.inv())
            else:
                self.detg = Function('|' +self.gsym +'|',real=True)(*self.coords)
                self.g_adj = simplify(self.g.adjugate())
                self.g_inv = self.g_adj/self.detg
        return

And I checked the conditions, it should execute the line self.g_inv = simplify(self.g.inv()), but simplify(self.g.inv()) actually gives the correct inverse.
After some debugging and code searching, I finally found the inverse is actually calculated by Ga.build_reciprocal_basis():
        g_inv = eye(self.n)
        self.dot_mode = '|'

        # Calculate inverse of metric tensor, g^{ij}

        for i in self.n_range:
            rx_i = self.r_symbols[i]
            for j in self.n_range:
                rx_j = self.r_symbols[j]
                if j >= i:
                    g_inv[i, j] = self.dot(self.r_basis_dict[rx_i], self.r_basis_dict[rx_j])
                    if not self.is_ortho:
                        g_inv[i, j] /= self.e_sq
                else:
                    g_inv[i, j] = g_inv[j, i]

        self.g_inv = g_inv

The self.e_sq in the executed line g_inv[i, j] /= self.e_sq is just (eu^ex^ey^ez) * (eu^ex^ey^ez) and it's equal to det(g). So it should have already been divided once, why isn't it enough?
Still investigating. The GA way of doing this is quite different from ordinary GR textbooks.

Dear @FreddyBaudine ,
I finally figured out that it's caused by #32, please see https://nbviewer.jupyter.org/github/utensil/julia-playground/blob/master/py/galgebra_reciprocal_basis.ipynb for step-by-step investigation.
Simply put, the inverse metric calculation in Ga is wrong for non-orthogonal basis.
This is because the calculation of reciprocal basis should use E_n^1 but choose to use E_n instead so all calculation that involves reciprocal basis need to remember divide the factor E_n^2. In the case of inverse metric, it's the dot product of two reciprocal basis vector, so the factor should be E_n^4 but implemented as E_n^2 possibly due to oversight.
So the line g_inv[i, j] /= self.e_sq should be g_inv[i, j] /= self.e_sq**2, see 08f33f9 . (The commit passes all test cases but one, I'll fix the failed case next week).

Dear Utensil,
Thank you for your involvement in all this, this is most appreciated.
I downloaded your corrected version of ga.py and run my little codes all over again, this times round obtaining the correct results.
While I was casting an eye on the Jupyter nbviewer, I noticed that you designed to verify the logic step by step using “Rotating frame spacetime manifold (Kerr-Debney Metric)”. I am afraid I made a mistake in as much as the Kerr-Debney metric has got nothing to do with the rotating frame metric. Nothing very unfortunate,  I simply forgot to delete the two print instructions in the file I originally posted. “Christoffel_kerr_debney_metric37.py” that were relating to the rotating frame metric.  Sorry about that, corrected file enclosed.
In your message, you’re explaining that the commit passed all tests but one. May I ask you which one that is?
Kind regards,
FB
ga_corrected.zip

Dear @FreddyBaudine,
The Kerr-Debney Metric typo is fixed, thanks for the heads up.

In your message, you’re explaining that the commit passed all tests but one. May I ask you which one that is?

examples/LaTeX/lin_tran_check.py failed and it's now fixed (in the sense of the mismatched output, not-simplified g_inv and poorly printed matrices) at https://github.com/pygae/galgebra/pull/17/commits/84c06ed24676feb0a9db6abfaa6535d17d2bcf66 . But it always had issues: it's not properly simplified using GA identities so the formulas are very loooooong.
I also tried some other metrics in https://github.com/spacetimeengineer/spacetimeengine/blob/master/spacetimeengine/src/solutions.py  at https://nbviewer.jupyter.org/github/utensil/julia-playground/blob/master/py/galgebra_gr_metrics.ipynb , they seem to have correct results. ( BTW the project is incomplete and has quite some typos.)
I have tried the Kerr metric, but GAlgebra has serious performance issue and it takes more than 20 minutes to initialize GA (I finally gave up trying for now until I get to fix #23 which I have no clue yet besides delay the simplification to the printing stage ).
In the process I also find some annoying bugs in how GAlgebra handles printing, particularly for lists, Arrays and Matrices, I'll try to fix them some time later because they likely require substantial structure refactor.
P.S. May I ask what's the background of your work on metrics with Maple and GAlgebra? More precisely, I wonder what more do you need from GAlgebra? Because in the process of fixing these issues, I realize that GAlgebra is far from complete for treating GR problems:

it doesn't calculate Ricci tensor or Riemann curvature tensor etc.
it uses semi-traditional semi-GA method to do certain calculations
it doesn't make use of full power of GA method (i.e. the various identities to facilitate simplifying or calculation)
And I would like to make GAlgebra (by modifying it or implement something based on it) more suitable for your use case (if feasible).

Regards,
Utensil Song

Dear Utensil,
Sorry for my late reply but I had been taken ill.
In one of your messages you had requested that I verified the calculation of the Christoffel symbols for other metrics, possibly on higher dimensional manifolds. I simply found it easy to use some GR metrics that were easily accessible.
I understand that the task ahead is huge if Galgebra is ever to be used to solve advanced problems.
Kind regards,
Freddy

Dear FreddyBaudine,
Sorry for my late reply, I was under the wrong impression that I've replied.

I had been taken ill

I'm sorry to hear that. Have you become better now?
Although I haven't been working on galgebra directly for a while, mostly because of busy work, I'm actively learning and preparing for further work on galgebra. Also, I've written a Julia wrapper of galgebra at https://github.com/pygae/GAlgebra.jl with fancier syntax using Unicode symbols as infix, prefix and postfix operators.
We had a lot of discussions in https://clifforddev.slack.com/ , and it would be great to hear from you on the discussions.
Regards,
Utensil Song


NPE in AstNodeExtension.nodeCovering.containsOffset()

Spotted this in my logs while setting up the server on Windows:
#0      AstNodeExtension.nodeCovering.containsOffset (package:analyzer/src/utilities/extensions/ast.dart:29:63)
#1      AstNodeExtension.nodeCovering.childContainingRange (package:analyzer/src/utilities/extensions/ast.dart:41:34)
#2      AstNodeExtension.nodeCovering (package:analyzer/src/utilities/extensions/ast.dart:59:21)
#3      CompilationUnitExtension.select (package:analysis_server/src/utilities/selection.dart:449:24)
#4      RefactoringContext.selection (package:analysis_server/src/services/refactoring/framework/refactoring_context.dart:42:8)
#5      RefactoringContext.selection (package:analysis_server/src/services/refactoring/framework/refactoring_context.dart)
#6      RefactoringProducer.selection (package:analysis_server/src/services/refactoring/framework/refactoring_producer.dart:51:40)
#7      MoveTopLevelToFile._selectedNodes (package:analysis_server/src/services/refactoring/move_top_level_to_file.dart:310:26)
#8      MoveTopLevelToFile._membersToMove (package:analysis_server/src/services/refactoring/move_top_level_to_file.dart:214:25)
#9      MoveTopLevelToFile.isAvailable (package:analysis_server/src/services/refactoring/move_top_level_to_file.dart:171:21)
#10     RefactoringProcessor.compute (package:analysis_server/src/services/refactoring/framework/refactoring_processor.dart:36:21)
#11     DartCodeActionsProducer.getRefactorActions (package:analysis_server/src/lsp/handlers/code_actions/dart.dart:203:39)
#12     CodeActionHandler.handle.<anonymous closure> (package:analysis_server/src/lsp/handlers/handler_code_actions.dart:216:31)
#13     OperationPerformanceImpl.runAsync (package:analyzer/src/util/performance/operation_performance.dart:172:29)
#14     CodeActionHandler.handle (package:analysis_server/src/lsp/handlers/handler_code_actions.dart:215:32)
<asynchronous suspension>
#15     ServerStateMessageHandler.handleMessage (package:analysis_server/src/lsp/handlers/handlers.dart:274:22)
<asynchronous suspension>
#16     LspAnalysisServer._handleRequestMessage (package:analysis_server/src/lsp/lsp_analysis_server.dart:987:20)
<asynchronous suspension>
#17     LspAnalysisServer.handleMessage.<anonymous closure>.<anonymous closure> (package:analysis_server/src/lsp/lsp_analysis_server.dart:464:15)
<asynchronous suspension>
#18     OperationPerformanceImpl.runAsync (package:analyzer/src/util/performance/operation_performance.dart:172:14)
<asynchronous suspension>
#19     LspAnalysisServer.handleMessage.<anonymous closure> (package:analysis_server/src/lsp/lsp_analysis_server.dart:447:11)
<asynchronous suspension>

1685823954881:Req:{"jsonrpc"::"2.0","id"::43,"method"::"textDocument/codeAction","params"::{"textDocument"::{"uri"::"file::///C%3A/Dev/Google/dart-sdk/sdk/pkg/analysis_server/lib/src/lsp/client_capabilities.dart"},"range"::{"start"::{"line"::35,"character"::61},"end"::{"line"::35,"character"::61}},"context"::{"diagnostics"::[],"triggerKind"::2}},"clientRequestTime"::1685823954882}
1685823954881:Res:{"id"::43,"jsonrpc"::"2.0","error"::{"code"::-32001,"message"::"An error occurred while handling textDocument/codeAction request"}}

It looks very similar to https://github.com/dart-lang/sdk/issues/52362, but it seems to be a few lines lower than where the fix for that was made.

@scheglov (for the similarity to another issue that you previously fixed)

I was not able to reproduce this, and don't want to fix without a test.
It might be something wrong happened before, e.g. during parsing, and we did not set tokens property, so that there is no next (normally I expect it only for comments).

I can reproduce it, thank you.

https://dart-review.googlesource.com/c/sdk/+/307504


Problems observed with 0.5

Problems with the 0.5 release.

There is no button or menu choice to re-generate the display after changing preferences.  If you change grid color, or grid spacing, there's no way to invoke those changes.
Data Management / Plot Stars always asks for you to choose a data-set rather than re-plotting what you chose before.  Shouldn't Choose data set be seperate from Plot this data set?
Display / Toggle Side Pane appears to do nothing. I can drag the side pane open, or drag it down to zero width, but the toggle choice doesn't ]
Nothing displays the version I'm running.  Help / About needs to display the version.
After loading the 250 LY ChView data set, the program claims to have plotted 5200 objects, but no where NEAR that many are displayed. Only the central 20 LY are displayed.  The program used to ask me what radius to display.  Now it doesn't.
Querying the 250 LY data set, even if I run the slider all the way to the right and choose to get 150 LY worth of data, it only displays 20 LY worth of stars.
Show Query Dialog maxes distance at 150LY, even if you're loading the 250 LY data set.
Show Query Dialog always default's to the top dataset in the list rather than defaulting to the data set identified in the status bar at the bottom.  Far too easy to make a mistake and query the wrong data set.
Still no way to drag display after zoom.
Can not find a place to change grid spacing any more.
Can not find a place to change the overall radius to be displayed.
The datasets list in the sidebar is inactive. They don't do anything, it's just a list.  We should either kill it, or make clicking one the same as picking one in Dataset Management / Plot
Settings button in toolbar toggles sidebar on and off instead of bringing up the settings dialog.  Change the text to "Sidebar"
Labels button in toolbar does nothing so far as I can tell.
When displaying stellar properties in the sidebar, spacing can be "weird"

comments:

Preferences isn't completely hooked in yet. There were 4 parts to changing preferences. setting up the UI (took a lot of time), building an appropriate schema for the DB, storing, and retrieving. Plus all the wierd little ways this can be called. The last part was to get the preferences changed in the actual display.
This last part needs to be finished and everything else is there.
I will reopen that ticket.

For Data Management / Plot Stars always asks for you to choose a data-set rather than re-plotting what you chose before. Shouldn't Choose data set be separate from Plot this data set?
Ans:
that was how I did it but I can change how it works.
Initially, all the actions were completely independent of each other. When I first started, there wasn't a context as everything was in the same db. There there was the idea of a dataset, etc.
I'll open up a feature request since it works as designed :)

For Data Management / Plot Stars always asks for you to choose a data-set rather than re-plotting what you chose before. Shouldn't Choose data set be seperate from Plot this data set?
Ans: Ah, I didn't retest that and I guess, the code got removed. Easy fix.
I will create an issue for it.

for Nothing displays the version I'm running. Help / About needs to display the version.
I haven't checked that code in a long while and it appears to have stopped working.
Will issue a ticket for it

I grouped all these together because they are related.
After loading the 250 LY ChView data set, the program claims to have plotted 5200 objects, but no where NEAR that many are displayed. Only the central 20 LY are displayed. The program used to ask me what radius to display. Now it doesn't.
Querying the 250 LY data set, even if I run the slider all the way to the right and choose to get 150 LY worth of data, it only displays 20 LY worth of stars.
Show Query Dialog maxes distance at 150LY, even if you're loading the 250 LY data set.
I'll take a look

for:
Show Query Dialog always default's to the top dataset in the list rather than defaulting to the data set identified in the status bar at the bottom. Far too easy to make a mistake and query the wrong data set.
I created an issue about establishing a context so I will add this there.

for:
Still no way to drag display after zoom.
Ans: I still haven't added that. It isn't straight forward. That is still open as #20

For:
Can not find a place to change grid spacing any more.
I reopened issue #18

for
Can not find a place to change the overall radius to be displayed.
Ans: I have the UI  under application preferences and everything is there except the display isn't using it currently

For:
The datasets list in the sidebar is inactive. They don't do anything, it's just a list. We should either kill it, or make clicking one the same as picking one in Dataset Management / Plot
Ans:
That actually works as a context menu if you click on the dataset

for:
Settings button in toolbar toggles sidebar on and off instead of bringing up the settings dialog. Change the text to "Sidebar"
Ans: easy enough

For:
Labels button in toolbar does nothing so far as I can tell.
Ans: I am still working on that. I thought I had an easy fix for that and treat it like the grid and extensions/stems, but the mechanism I had doesn't seem to work.  Something more complicated is going on there

For:
When displaying stellar properties in the sidebar, spacing can be "weird"
Ans: I am completing rewriting how that works. Mostly because I wanted to add an edit star function and the record that I store at each star is not complete.
Expect the amount of data to be different and better formatted

Re: for
Can not find a place to change the overall radius to be displayed.
Ans: I have the UI under application preferences and everything is there except the display isn't using it currentl

No, no, I mean, no matter what data I load or query, I always get 20 LY centered on Sol.  I can't get a 100 LY wide display.  Nothing to do with the radius of the STARS.
There used to be a place to enter the VIEW RADIUS.

Ok, that is fixed.I refactored the whole dataset context and made even listeners to propagate the changes throughout the software. It also exposed some bugs with query and I fixed those as well.
Aside from the gird initially displaying and then coming back (mystifying), its pretty much done and will be in v0.5.2


Splitting fixtures into separate files gives an error

Lets say we have cms suite, and configs placed at 2 files:
# config/fixtures/cms/frontpage/frontpage_about_us.yaml
sylius_fixtures:
    suites:
        cms:
            fixtures:
                block:
                    options:
                        custom:
                            frontpage_about_us:
                                ...

config/fixtures/cms/frontpage/frontpage_product_slider.yaml
sylius_fixtures:
    suites:
        cms:
            fixtures:
                block:
                    options:
                        custom:
                            frontpage_product_slider:
                                ...

When we try to load this suite - we will get exception:
Argument 1 passed to BitBag\SyliusCmsPlugin\Fixture\Factory\BlockFixtureFactory::createBlock() must be of the type string, int given, called in /vendor/bitbag
  /cms-plugin/src/Fixture/Factory/BlockFixtureFactory.php on line 100

Debugging will show that we have first block's key as expected (frontpage_about_us), when second block's (frontpage_product_slider) key is 0 for some reason (and 1, 2, etc if we have more files - all keys except the first one rewritten).
Things were working before splitting into separate files.
So I guess its something related to configuration merging.
Looks like somehow related to #249

Workaround is you want to have cms fixtures splitted into separate files
You should set unique keys for fixtures, like this:
sylius_fixtures:
    suites:
        cms:
            fixtures:
                cms_block_frontpage_about_us: # <---------------------
                    name: block # <---------------------
                    options:
                        custom:
                            frontpage_about_us:
                                channels:

sylius_fixtures:
    suites:
        cms:
            fixtures:
                cms_block_frontpage_product_slider: # <---------------------
                    name: block # <---------------------
                    options:
                        custom:
                            frontpage_product_slider:

This seems to be how https://github.com/Sylius/SyliusFixturesBundle works not CMS plugin issue, so I am closing this


More stylcop warnings fixes on library project

Mostly documentaiton and formatting. I recommend reviewer while hiding whitespace only changes :smile:

Summary

Good to merge once build is finished


Building OpenEXR

When building OpenEXR I had to first build Zlib and then add  -DZLIB_ROOT="C:/zlib" to the second project in order for it to build successfully.
Not a big problem, but though I comment on it just in case anybody else has the same issue.

When you say the second project, do you mean openexr.cmd needs to be amended to refer to zlib?

Yes, cmake could not build without the reference.
cmake -G "Visual Studio 14 2015 Win64" -DILMBASE_PACKAGE_PREFIX="%current%\local"  -DZLIB_ROOT="C:/zlib" -DCMAKE_INSTALL_PREFIX="%current%\local" ..\..\openexr\OpenEXR
It also did not work with a prebuilt version of Zlib. It needed the built version. Thankfully you guys have a zlib.cmd that built it and all I had to do is add that DZLIB_ROOT argument.

If you are using zlib.cmd, then -DZLIB_ROOT would be %current%\local. Could you try the latest and reopen this issue if the problem persists? 1318dc7c77be10fcec4e046ecb0534c2a2c8a9c6


The default path for x2goclient

the default path for x2goclient for MAC OS should be:
/Applications/x2goclient.app

Failed in Master.

The path on Mac should be /Applications/x2goclient.app which works for me but I have to reset it in settings manually with each time I upgrade.

Both of them are correct, just make sure desktop ssh is working on master

Done in Master.

Done in Prod.


esbuild: 0.24.0 -> 0.24.2

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/evanw/esbuild/releases.
meta.description for esbuild is: Extremely fast JavaScript bundler
meta.homepage for esbuild is: https://esbuild.github.io
meta.changelog for esbuild is: https://github.com/evanw/esbuild/blob/v0.24.1/CHANGELOG.md
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.24.1 with grep in /nix/store/gbj45djsh7frbc8bp560zinm7ykxsyy2-esbuild-0.24.1
found 0.24.1 in filename of file in /nix/store/gbj45djsh7frbc8bp560zinm7ykxsyy2-esbuild-0.24.1

Rebuild report (if merged into master) (click to expand)

13 total rebuild path(s)

13 package rebuild(s)

First fifty rebuilds by attrpath

esbuild
libsForQt5.bismuth
opcua-commander
plasma5Packages.bismuth
plausible
quarto
quartoMinimal
rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper
vscode-extensions.rust-lang.rust-analyzer

Instructions to test this update (click to expand)

Either download from the cache:
nix-store -r /nix/store/gbj45djsh7frbc8bp560zinm7ykxsyy2-esbuild-0.24.1 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The nixpkgs-update cache is only trusted for this store-path realization.)
For the cached download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A esbuild https://github.com/r-ryantm/nixpkgs/archive/6742cfa2ca712094bcc646f9f1beee3b9ddbe19e.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/6742cfa2ca712094bcc646f9f1beee3b9ddbe19e#esbuild

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/gbj45djsh7frbc8bp560zinm7ykxsyy2-esbuild-0.24.1
ls -la /nix/store/gbj45djsh7frbc8bp560zinm7ykxsyy2-esbuild-0.24.1/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 11 packages built:

esbuild
libsForQt5.bismuth
opcua-commander
plausible
quarto
quartoMinimal
rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper
vscode-extensions.rust-lang.rust-analyzer

Maintainer pings
cc @Lucus16 @undefined-moe @ivan for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.

Automatic update generated by nixpkgs-update tools. This update was made based on information from https://github.com/evanw/esbuild/releases.
meta.description for esbuild is: Extremely fast JavaScript bundler
meta.homepage for esbuild is: https://esbuild.github.io
meta.changelog for esbuild is: https://github.com/evanw/esbuild/blob/v0.24.2/CHANGELOG.md
Updates performed

Golang update

To inspect upstream changes

Release on GitHub

Compare changes on GitHub

Impact
Checks done

built on NixOS
The tests defined in passthru.tests, if any, passed
found 0.24.2 with grep in /nix/store/m19swpxan43m9h4why5542lgygaxlr60-esbuild-0.24.2
found 0.24.2 in filename of file in /nix/store/m19swpxan43m9h4why5542lgygaxlr60-esbuild-0.24.2

Rebuild report (if merged into master) (click to expand)

13 total rebuild path(s)

13 package rebuild(s)

First fifty rebuilds by attrpath

esbuild
libsForQt5.bismuth
opcua-commander
plasma5Packages.bismuth
plausible
quarto
quartoMinimal
rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper
vscode-extensions.rust-lang.rust-analyzer

Instructions to test this update (click to expand)

Either download from the cache:
nix-store -r /nix/store/m19swpxan43m9h4why5542lgygaxlr60-esbuild-0.24.2 \
  --option binary-caches 'https://cache.nixos.org/ https://nixpkgs-update-cache.nix-community.org/' \
  --option trusted-public-keys '
  nixpkgs-update-cache.nix-community.org-1:U8d6wiQecHUPJFSqHN9GSSmNkmdiFW7GW7WNAnHW0SM=
  cache.nixos.org-1:6NCHdD59X431o0gWypbMrAURkbJ16ZPMQFGspcDShjY=
  '

(The nixpkgs-update cache is only trusted for this store-path realization.)
For the cached download to work, your user must be in the trusted-users list or you can use sudo since root is effectively trusted.
Or, build yourself:
nix-build -A esbuild https://github.com/r-ryantm/nixpkgs/archive/5580564c40aadd497b84cb9c6ea7985b60e11cc1.tar.gz

Or:
nix build github:r-ryantm/nixpkgs/5580564c40aadd497b84cb9c6ea7985b60e11cc1#esbuild

After you've downloaded or built it, look at the files and if there are any, run the binaries:
ls -la /nix/store/m19swpxan43m9h4why5542lgygaxlr60-esbuild-0.24.2
ls -la /nix/store/m19swpxan43m9h4why5542lgygaxlr60-esbuild-0.24.2/bin

Pre-merge build results
We have automatically built all packages that will get rebuilt due to
this change.
This gives evidence on whether the upgrade will break dependent packages.
Note sometimes packages show up as failed to build independent of the
change, simply because they are already broken on the target branch.
nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review

x86_64-linux

:white_check_mark: 11 packages built:

esbuild
libsForQt5.bismuth
opcua-commander
plausible
quarto
quartoMinimal
rstudio
rstudio-server
rstudioServerWrapper
rstudioWrapper
vscode-extensions.rust-lang.rust-analyzer

Maintainer pings
cc @Lucus16 @undefined-moe @ivan for testing.

[!TIP]
As a maintainer, if your package is located under pkgs/by-name/*, you can comment @NixOS/nixpkgs-merge-bot merge to automatically merge this update using the nixpkgs-merge-bot.

Add a :+1: reaction to pull requests you find important.


KAFKA-9355: Fix bug that removed RocksDB metrics after failure in EOS

Added init() method to RocksDBMetricsRecorder
Added call to init() of RocksDBMetricsRecorder to init() of RocksDB store
Added call to init() of RocksDBMetricsRecorder to openExisting() of segmented state stores
Adapted unit tests
Added integration test that reproduces the situation in which the bug occurred

Committer Checklist (excluded from commit message)

[ ] Verify design and implementation
[ ] Verify test coverage and CI build status
[ ] Verify documentation (including upgrade notes)

LGTM! Merging to trunk and cherry-picking to 2.5

Cherry-picked to 2.4


Image File is not uploading to the server.

I am trying to upload my file to server , but the file is not sending to controller method.
Here is my code:
$(document).ready(function() {
$('.editable').mediumInsert({
            editor: editor, // (MediumEditor) Instance of MediumEditor
            enabled: true,
            addons: {
                images: {
                    fileUploadOptions: {

                    	 type: "POST",
                    	 paramName:"files[]",
                    	 multipart: true,
                        url: '${pageContext.request.contextPath}/a/{storeurl}/upload/image.json',
                        acceptFileTypes: /(\.|\/)(gif|jpe?g|png)$/i

                    }
                }

            }

        });
    });

Not enough info. Can you repro on jsfiddle/codepen/our demo page?


Bugfix strtolower template name

Hi,
When i install Zend Framework 3 on ubuntu or ubuntu based operating systems, an exception new pages, "cannot resolve Application/Index/index" template. I found this bug's fix. I think, this problem occured so UTF-8.

@kaansef there are two problems with this patch:

[ ] missing tests
[ ] this patch does not have a direct dependency on ext-mbstring, while your code assumes that it is available

Hi again,
I'm Turkish, and my English very bad. So sorry. Sorry for my pull request's doesn't use format. I think this bug only some Linux distros (UTF-8 used). Thank for reply.

@kaansef can you add a test to demonstrate the problem?

Closing due to inactivity. Missing test case.


can not see sub theme icon

because any path to sub theme are 500

#5675

fixed in 1,10

see my comment in #5675 about multiple web.config in subfolder Scripts, Styles and Content and <add name="StaticFile" which is causing the 500 error. Adding <remove name="StaticFile"/> above each fixes this issue, but themes like TheThemeMachine are not affected and I don't know why.


esp_http_client exception on 401 without Www-Authenticate header

Environment

Development Kit:      [none]
Kit version (for WroverKit/PicoKit/DevKitC): -
Core (if using chip or module): [ESP-WROOM3]
IDF version: be81d2c16d7f4caeea9ceb29fece01510664caf3
Development Env:		[Eclipse]
Operating System:		[Ubuntu 16.04]
Power Supply:         [external 3.3V]

Problem Description
When the server gives back a 401 response and the "Www-Authenticate" header is missing, a null pointer Exception occurs in the function "http_utils_trim_whitespace()".
On line 70  in "/components/esp_http_client/lib/http_utils.c" a NULL pointer may be passed in, when header is missing:
...
char *end;
    char *start = *str;

    // Trim leading space
    while (isspace((unsigned char)*start)) start ++;
...

Expected Behavior
Maybe issue warning?
Debug Logs
I (12025) HTTP_CLIENT: Posting to https://<IP_ADDRESS>/
Guru Meditation Error: Core  1 panic'ed (LoadProhibited). Exception was unhandled.
Core 1 register dump:
PC      : 0x4010d4dc  PS      : 0x00060930  A0      : 0x8010c01f  A1      : 0x3ffcabe0
0x4010d4dc: http_utils_trim_whitespace at /home/peter/esp/esp-idf-master/components/esp_http_client/lib/http_utils.c:74 (discriminator 1)
A2      : 0x3ffcac00  A3      : 0x00000000  A4      : 0x3ff96354  A5      : 0x0000ca11
A6      : 0x00000191  A7      : 0x00000000  A8      : 0x00000000  A9      : 0x00000000
A10     : 0x00000000  A11     : 0x3ffd4b04  A12     : 0x00000010  A13     : 0x00000000
A14     : 0x3ffcb230  A15     : 0xff000000  SAR     : 0x00000010  EXCCAUSE: 0x0000001c
EXCVADDR: 0x00000000  LBEG    : 0x4000c2e0  LEND    : 0x4000c2f6  LCOUNT  : 0x00000000
Backtrace: 0x4010d4dc:0x3ffcabe0 0x4010c01c:0x3ffcac00 0x4010c392:0x3ffcac30 0x400d6aef:0x3ffcac50 0x400d6b87:0x3ffcace0
0x4010d4dc: http_utils_trim_whitespace at /home/peter/esp/esp-idf-master/components/esp_http_client/lib/http_utils.c:74 (discriminator 1)
0x4010c01c: esp_http_check_response at /home/peter/esp/esp-idf-master/components/esp_http_client/esp_http_client.c:981
0x4010c392: esp_http_client_perform at /home/peter/esp/esp-idf-master/components/esp_http_client/esp_http_client.c:981
0x400d6aef: http_rest() at /home/peter/esp/WTW_controller/main/API/http_client.cpp:109
0x400d6b87: http_test_task(void*) at /home/peter/esp/WTW_controller/main/API/http_client.cpp:498 (discriminator 8)

Thanks for discovering this issue, the fix is pending internal review and will be updated shortly


Startup Error

Microsoft PowerToys version
0.62.0
Running as admin

[ ] Yes

Area(s) with issue?
General
Steps to reproduce
On startup.
✔ Expected Behavior
Normal startup.
❌ Actual Behavior
2022-09-18.txt
Other Software
No response

Can you please upgrade to latest 0.73 and verify if the issue still occur?
/needinfo


Definition of U gate is not valid

Hi, I tried implementing U gate via the decomposition of it given on qiskit docs (https://qiskit.org/documentation/stubs/qiskit.circuit.library.UGate.html) . However the results from the decomposition and U gate are not concurrent.
u3(pi/2,pi/3,pi/4) q[0] gives:  [ 0.707+0j, 0.354+0.612j ]
while
rz(pi/3) q[0];
rx(-pi/2) q[0];
rz(pi/2) q[0];
rx(pi/2) q[0];
rz(pi/4) q[0];
gives: [ 0.43-0.561j, -0.701+0.092j ]
Is it an issue or am I doing something wrong?

Yes, I guess you are right. I just run a simulation and got different results. But chances are that I may be wrong too. I would love to have the mistake pointed out (if there is one).

Did you account for the global phase factor into the implementation? I rapidly computed the unitary matrix for this sequence of gates on paper, I think you can easily check that RZ(phi-pi/2)*RX(pi/2)*RZ(pi-theta)*RX(pi/2)*RZ(lambda-pi/2) = exp(-j(phi+lambda)/2)U3(theta,phi,lambda).
If you account for this phase factor in your example, which will be exp(-7pi/24j), I believe you should get the desired result.


Merge main4 to sync stack

API changes have been detected in com.azure:azure-core. You can review API changes here

API changes have been detected in com.azure.resourcemanager:azure-resourcemanager. You can review API changes here

API changes have been detected in com.azure.resourcemanager:azure-resourcemanager-containerservice. You can review API changes here

API changes have been detected in com.azure.resourcemanager:azure-resourcemanager-keyvault. You can review API changes here


Set correct cursor on hover for datepicker pick date icon

Which package is this issue related to?
ffe-datepicker
Describe your issue (screenshots welcome!)

What is the expected behavior?
When I hover the mouse over the icon the cursor should indicate that it is clickable
What is the actual behavior?
The cursor has the caret cursor
Steps to reproduce

Go to https://design.sparebank1.no/styleguidist/index.html#datepicker
Hover the calendar icon

On closer inspection the datepicker pops up as soon as the input gets focus and clicking the icon is not needed so in this case I am thinking it's okay for it to not have the mouse pointer on hover,
Closing this, unless @adidick wants to discuss with UU/UX if the icon should still have a pointer icon on hover or if we should consider it as being strictly decorative.


Implement clone for web runtime

this allows for the pattern
let net = NetworkResource::new_client(tokio_rt.clone(), task_pool.0.clone());

for both native and web.
Not sure about not having Option on the parameter though.

Thank you for the PR! I fully agree that removing the Option is probably a good idea, since we already insert the fake Runtime for web builds anyway.


Add more Functionalities

Please add basic edit functionalities like brightness, contrast, flip and croping

I will keep it in mind for another versions, but isnt priority

Add Multiple Request on TakePhoto Option


USelectMenu can't accept searchable and multiple at the same time

Environment

Operating System: Darwin
Node Version:     v20.11.1
Nuxt Version:     3.11.2
CLI Version:      3.11.1
Nitro Version:    2.9.6
Package Manager<EMAIL_ADDRESS>Builder:          -
User Config:      ssr, devtools, app, modules, typescript, colorMode, components, plugins, css, runtimeConfig, routeRules
Runtime Modules<EMAIL_ADDRESS><EMAIL_ADDRESS>Build Modules:    -

Version
v2.16.0
Reproduction
https://stackblitz.com/edit/nuxt-ui-gvsbl5?file=app.vue
Description
Using searchable and multiple prop at the same time the component become unusable
Additional context

Logs
No response

In your reproduction, you didn't pass an array as modelValue. You need one when using the multiple prop.


Current extraction logic produces duplicates

Currently the jobs that extract the data from Elastic run every 10 minutes. By default, the query asks for all data from the last 11 minutes. This produces an overlap of 1 minute. So each entry from Elastic that has a timestamp between h:59:00 and h+ 1:00:00 will potentially be extracted twice, e.g. both the celery job starting at 8:00:00 and the one starting at 8:10:00 will extract entries from 7:59:30.
One possible solutions would be to asks for the data from the last 10 minutes. But depending on how accurately celery times the job execution, some data might not be extracted at all.
Another idea is to use kind of a sliding window that always extracts the entries from the last completed 10 minute period, e.g. the celery job starting at 8:10:00 will extract every entry with a timestamp t for which 8:00:00 <= t < 8:10:00 applies.

yep, last option would have probably be the best way to solve it

Fixed.


RuntimeError: Response has status code 404 not (200,)

I just installed the osf client, but I always got this error when I tried basic commands, like this one:
(base) D:\>osf -p xxx<projectid> ls
Traceback (most recent call last):
File "D:\Applications\Anaconda\Scripts\osf-script.py", line 9, in <module>
   sys.exit(main())
File "D:\Applications\Anaconda\lib\site-packages\osfclient\__main__.py", line 104, in main
   exit_code = args.func(args)
File "D:\Applications\Anaconda\lib\site-packages\osfclient\cli.py", line 91, in wrapper
   return_value = f(cli_args)
File "D:\Applications\Anaconda\lib\site-packages\osfclient\cli.py", line 218, in list_
   project = osf.project(args.project)
File "D:\Applications\Anaconda\lib\site-packages\osfclient\api.py", line 24, in project
   return Project(self._json(self._get(url), 200), self.session)
File "D:\Applications\Anaconda\lib\site-packages\osfclient\models\core.py", line 60, in _json
   status_code))
RuntimeError: Response has status code 404 not (200,)

I installed and run it within an Anaconda enviroment, not sure if that's the reason?

I have the same exact error! I'm also running in an Anaconda environment

I hit the error even outside of the Anaconda environment

FSL:~ $ osf ls
Please input your password:
Traceback (most recent call last):
File "/fslhome/nmella/.local/bin/osf", line 10, in
sys.exit(main())
File "/fslhome/nmella/.local/lib/python3.6/site-packages/osfclient/main.py", line 104, in main
exit_code = args.func(args)
File "/fslhome/nmella/.local/lib/python3.6/site-packages/osfclient/cli.py", line 91, in wrapper
return_value = f(cli_args)
File "/fslhome/nmella/.local/lib/python3.6/site-packages/osfclient/cli.py", line 218, in list_
project = osf.project(args.project)
File "/fslhome/nmella/.local/lib/python3.6/site-packages/osfclient/api.py", line 24, in project
return Project(self._json(self._get(url), 200), self.session)
File "/fslhome/nmella/.local/lib/python3.6/site-packages/osfclient/models/core.py", line 60, in _json
status_code))
RuntimeError: Response has status code 404 not (200,)

I just found out that I hit this error because I wasn't using the correct project name. For example, look at the screen shot below. I originally was using the name of the project (TCGA Data Types Benchmark), but in reality I needed to use "3snep" which you can see in the url https://osf.io/3snep/


[Crane] Set the onClick label for DayContainer

This specifies the onClick label to use for the individual calendar
days. I also stopped passing the selected boolean to Surface. When it is
passed, that uses the selectable version of Surface which announces the
composable as a Tab and automatically states "selected" at the start of
the announcement when selected==true but does not announce anything at
the start when not selected.
Instead, it's now using the clickable version of Surface, so our
announcements are like:
"{Selected/not selected}, January 1st 2020, Button, Double-tap to
select"
This seems reasonable, since the other Roles are Checkbox, Switch,
RadioButton, and Image, all of which seem less correct than Button.

This fixes https://github.com/android/compose-samples/issues/766


Complete assignment-04 (everything but reverse proxy)

Rubric for Assignment 4: Networking
Submission
Canvas
1: The link submitted on Canvas is for the pull request containing the assignment submission.
0: The link submitted on Canvas is not for the pull request containing the assignment submission.
Git Branch
1: The branch includes only changes for this assignment.
0: The branch includes commits or changes outside this assignment.
Git Commit Message
1: All commit messages for the branch are reasonably clear.
0: One or more commit messages for the branch are reasonably clear.
GitHub Pull Request
1: The pull request is made to the main branch of the student's fork of the course repository.
0: The pull request is made to a branch other than main or to a repo that is not the student's fork of the course repository.
Points
Submission Score: 3/4

Contains pulls/pushes from other assignments/merges (-1)

Correctness
HTTP
Unit tests are provided for this section; for each of the three tests:
2: The test passes
1: The test fails due to a minor error
0: The test fails due to a major error
0/6 - provided unit tests failing (-2 each)
Origin Server
For each of the following API routes

GET Request to /orders
GET Request to /orders/{id}
POST Request to /orders
DELETE Request to /orders
DELETE Request to /orders/{id}
Home page (nothing or /)

2: The server responds correctly to this command with the intended behavior as documented in README.md
1: The server responds to this command, but there is a minor error in the behavior
0: There is a major error in the implementation of this pathway
Additionally, the following error types must be handled appropriately in an idiomatic, non-fatal manner and return an appropriate HTTP response (1 pt each)

Invalid JSON for an OrderRequest in a POST request
Invalid HTTP Request
Attempting to access a path that does not exist
Attempting to call an unsupported method for a path

Reverse Proxy
5: Reverse proxy correctly forwards requests towards origin server and sends response to client under a manual test
3: Reverse proxy server does not correctly forward requests and responses due to a minor error
1: Reverse proxy server does not correctly forward requests and responses due to a major error
0: Reverse proxy server was not attempted
Points
Correctness Score: 0/21
Code Quality
Design
Tests
For each of the 6 API Routes:
2: This functionality is appropriately tested with passing unit test cases, including error cases
1: This functionality has unit tests, but they don't adequately cover code path/reasonable input cases
0: This functionality has no unit tests
General Cleanliness
4: The code is effectively documented, especially for sources of user-facing errors. Additionally, function signatures are clear and the code is overall very readable.
2: Attempts at verbose documentation were made, but certain sources of error are unclear OR poor function signatures are used and readers would have slight difficulties while scanning the code.
0: Documentation is severely lacking in several places and the code is not written in a readable way.
4/4
Lint/Format
RustFmt
2: RustFmt produces no warnings when run on the changed files
1: RustFmt produces a few minor warnings when run on the changed files
0: RustFmt produces many warnings when run on the changed files
2/2
Clippy
2: Clippy produces no warnings when run on the changed files
1: Clippy produces a few minor warnings when run on the changed files
0: Clippy produces many warnings when run on the changed files
Points
Code Quality Score: 6/20
Summary
Total Score: 9/45

Unable to compile http.rs due to various errors & as a result, cannot test any of the other functions
Seems to be an issue with how you have handled the AspirinEatsError in lines 24 & 27
origin.rs appears to be missing a lot of sections that were originally given, including the route & handle orders functions

Are there any commits that you forgot to push by any chance?


ChainState: Move blocks from immutable DB to mutable DB on corruption

Blocks are only moved from the volatile DB to the immutable DB when they come, well, immutable. This means that unless the immutable DB is empty, the volatile store must always contain a chain fragment of at least length k. However, this invariant gets broken when the volatile store in some way gets corrupted.
The way we have been talking about data corruption so far is that corruption leads to truncation of the chain: corruption is easy to deal with, as we simple end up with a shorter chain. What we have not been very explicit yet though is what consequences this has (if any) for how far we can rollback.

Suppose our tip was as T before the data corruption, and at T' < T after. If data model is truncation, then by rights this means that we should now be able to roll back to T' - k < T - k.
One might argue that this is morally incorrect: after all, we moved stuff to the immutable DB because we regarded them as immutable; the fact that we had some data loss should not mean that we can suddenly start to consider them volatile after all.
However, that view leads to having to keep two stateful values: the current tip, and separately the "how far can we roll back point". While this is not disaster, the assumption that we can roll back k might be implicit in many places in the code base and it's hard to audit for this.
Moreover, note that data loss can already mean that we roll back further than k anyway: if the immutable DB gets truncated, by definition we are effectively rolling back past k.

Having considered the alternatives, we prefer to keep this problem localized to the storage layer, so that as far as the rest of the system goes, data loss really is truncation. What this means is that if the volatile store gets corrupted and does not have a chain fragment of length k anymore, we need to move some blocks from the immutable DB back to the volatile store and truncate the immutable DB.

We will not do this after all; the ChainDB is now fully in charge of rollback. See the semi-formal spec of the ChainDB for detailed discussion.


Bug fix: Handled cases where we have only one line address.

Fixed issue when we have only one line address.
The android.location.Address.getMaxAddressLineIndex() method returns 0 if we have only one line address.

Could you show me the example query address?

This happens to any address when we use Geocode API on android 7.0 with latest play services

In order to test your code, please give me the input data, and output data.

Request
"position" -> "{"lat":41.458544,"lng":-81.626817}"
Response on Android 7.0
Address[
addressLines=[0:"8400-8474 Aetna Rd, Cleveland, OH 44105, USA"],
feature=8400-8474,admin=Ohio,sub-admin=Cuyahoga County,locality=Cleveland,thoroughfare=Aetna Road,postalCode=44105,countryCode=US,countryName=United States,hasLatitude=true,latitude=41.458311699999996,hasLongitude=true,longitude=-81.62689639999999,phone=null,url=null,extras=null]
Response on Android 5.0
Address[
addressLines=[0:"8400-8474 Aetna Road",1:"Cleveland, OH 44105",2:"USA"],
feature=8400-8474,admin=Ohio,sub-admin=null,locality=Cleveland,thoroughfare=Aetna Road,postalCode=44105,countryCode=US,countryName=United States,hasLatitude=true,latitude= 41.458311699999996,hasLongitude=true,longitude=-81.62689639999999,phone=null,url=null,extras=null]

Found the document.
Returns the largest index currently in use to specify an address line. If no address lines are specified, -1 is returned.
https://developer.android.com/reference/android/location/Address.html#getMaxAddressLineIndex()
In that case, my code is wrong. Thank you for figured out this small bug.


Craft CMS 3 support?

Any idea if this plugin works well with Craft 3?

I suppose not - we have never tested it tho.

Are there plans to upgrade to Craft 3?

I think so :-)

Is there a timeline for Craft 3 support?

No - basically whenever we feel like it. With RC1 out, there's more chance we'll be using that for our projects soon. And with that, there's also more chance we will port Schematic ;-)