Skype Calling bot gives Internal Server Error

I'm using the project template from https://docs.botframework.com/en-us/csharp/builder/sdkreference/calling.html for building a skype calling bot. I published the bot as an azure site and registered it in the bot framework developer portal and then added it type skype for windows desktop 8.1 (v 7.26).
When I type the echobot works correctly. When I start a call however, the bot replies with 'You can't talk to this Bot just yet. But we are working on it'. When debugging the azure site I get a response:
{StatusCode: 500, ReasonPhrase: 'Internal Server Error', Version: 1.1, Content: System.Net.Http.ObjectContent`1[[System.Web.Http.HttpError, System.Web.Http, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=31bf3856ad364e35]], Headers:
{
Content-Type: application/json; charset=utf-8
}}
in ProcessCallingEventAsync()
Is there something I'm missing or doing wrong or is the Calling bot sample not ready to be demoed yet? We want to show this to a client this week, will this/my issue be fixed shortly?

Could you check if your calling webhook is configured correctly on the portal? Could you also share the bot Id please?

My BotId is 5CACallingBotDemo and my callback hook is https://5cacallingbotdemo.azurewebsites.net/api/calling/call

It is working now. I reset the callback hook several hours ago, maybe that was the solution.

Great. Thanks for the update!

I am facing a similar issue. Does it take time for the calling feature to come up ?

No it should not take more than a couple of minutes to propagate. Is your call back even getting hit?

Hi ,
Thanks for your help. It is working now . I had not specified the call back
in my web.config.
Thanks
Jignesh
On Tue, Nov 22, 2016 at 8:21 PM, KrishnanA<EMAIL_ADDRESS>wrote:

No it should not take more than a couple of minutes to propagate. Is your
call back even getting hit?
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Microsoft/BotBuilder/issues/940#issuecomment-262260654,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AWj6MAg7SmdAD1QVpNNprBOQ7Woi1RLtks5rAwF8gaJpZM4Je_CE
.

Great. Thanks for confirming.

Hello, I am having the same issue, and I am just using the template. My current setup is as follows (I am currently using ngrok to expose my locally running Bot trough the Internet):

I have the CallbackUrl specified on the web.config:

<add key="Microsoft.Bot.Builder.Calling.CallbackUrl" value="http://MY-NGROK-DOMAIN/api/calling/callback" />

In CallingController.cs I have:
```
 [Route("callback")]
 public async Task<HttpResponseMessage> ProcessCallingEventAsync()
 {
     return await CallingConversation.SendAsync(Request, CallRequestType.CallingEvent);
 }
 [Route("call")]
 public async Task<HttpResponseMessage> ProcessIncomingCallAsync()
 {
     return await CallingConversation.SendAsync(Request, CallRequestType.IncomingCall);
 }

And on top of the class definition:

`[RoutePrefix("api/calling")]`

3) In the Skype Channel configuration, I have the following callback hook:

<img width="691" alt="screen shot 2016-11-24 at 19 28 34" src="https://cloud.githubusercontent.com/assets/16963208/20610726/7f270bb4-b27c-11e6-8ffd-e200f3e38765.png">

4) I have ngrok running on port 12345 and as well I have configured my project to run on that port, as well as to accept (via applicationhost.config), extern connections to that port. Firewall is accepting connections on that port as well.

My issue: same as the guys above, whenever I call the bot from Skype, the voice says 'You can't talk to this Bot just yet. But we are working on it', and on ngrok I receive a 500 Internal Server Error when it tries to make a POST to /api/calling/callback (and if I debug it dies inside the ProcessCallingEventAsync after re-entering a couple of times).

Any help?

It is weird because both the call and callback endpoints are being called. First the api/calling/call one, and then the api/calling/callback, which is called three times until it fails and the voice speaks the message described above.
I was thinking that it could be related to something with http and https, (call endpoint is configured as https in Bot Framework and is hosted as http locally)

Can you try running it on Azure instead of ngrok? Does that resolve the issue?
Have you tried posting to the webhook that you have configured on Bot Framework to see if your web service gets invoked?
Thanks
Krishnan

Could you also ensure that the call back URL in web.config is the appropriate https URL?
Thanks
Krishnan

Yes, it was that. Callback in webconfig was in http instead of https!
Now I have something like:
<add key="Microsoft.Bot.Builder.Calling.CallbackUrl" value="**https**://6719abe7.ngrok.io/api/calling/callback" /> and it works.
Thanks :)

Same issue here. Works neither with NGrok nor on Azure. All settings are fine (MicrosoftAppId/MicrosoftAppPassword set, Microsoft.Bot.Builder.Calling.CallbackUrl set to HTTPS et al).
Webhook URLs do not get hit at all (both NGrok and Azure). I tried manually hitting the NGrok Url (xyz.io/api/calling/call) thru Fiddler and it gets hit. But when I make a call on Skype, it does not get hit.
Also, the documentation here seems to be incorrect. It says
the new URL should look like this: https://78191649.ngrok.io/v1/call
Shouldn't it be _https://78191649.ngrok.io/*api/calling*/call_ as per the example given?


Rename hallog to logger

In GitLab by @arisilon on Oct 29, 2019, 15:58

In GitLab by @arisilon on Oct 30, 2019, 13:27
closed

In GitLab by @arisilon on Oct 29, 2019, 16:43
mentioned in commit c3855d1c5bd9ad6e669e8b92575136c32ffd2e09


add function in rwops to read bitrmap from memory and an example to test

Firstly, thanks for sdl2-fortran repo. I see that a lot of work has gone into it. I was missing functions to load bitmaps from memory. I've attached one in this pull request, which proved more difficult that I initially thought (I'm new to fortran). I don't expect you to merge this but it seemed the best way to share my idea and discuss.
In order to embed a bitmap I used objdump to create an object file from fortran.bmp in the examples/alpha folder, which is then linked to the executable as described in alpha1.f90.  I'm unsure if there is a cleaner/better way to do this: f95 fortran.o alpha1.f90
Testing proved positive but fails if the memory is messed up by, for example, running alpha without the fortran.bmp present and then running alpha1 immediately after. Not sure why.

I haven’t tested it yet, but a quick look revealed that one dummy argument of your interface is wrong:
        ! SDL_RWops *SDL_RWFromMem(const void *mem, int size)
        function sdl_rw_from_const_mem(mem, siz) bind(c, name='SDL_RWFromConstMem')
            import :: c_char, c_ptr, c_int
            character(kind=c_char), target, intent(in) :: mem
            integer(kind=c_int),            intent(in) :: siz
            type(c_ptr)                                :: sdl_rw_from_const_mem
        end function sdl_rw_from_const_mem

The dummy argument size needs the value attribute:
        ! SDL_RWops *SDL_RWFromMem(const void *mem, int size)
        function sdl_rw_from_const_mem(mem, siz) bind(c, name='SDL_RWFromConstMem')
            import :: c_char, c_ptr, c_int
            character(kind=c_char), target, intent(in) :: mem
            integer(kind=c_int),            intent(in), value :: siz
            type(c_ptr)                                :: sdl_rw_from_const_mem
        end function sdl_rw_from_const_mem

Furthermore, the target attribute makes no sense in this context. A correct interface would be:
        ! SDL_RWops *SDL_RWFromMem(const void *mem, int size)
        function sdl_rw_from_const_mem(mem, size) bind(c, name='SDL_RWFromConstMem')
            import :: c_ptr, c_int
            type(c_ptr),         intent(in), value :: mem
            integer(kind=c_int), intent(in), value :: size
            type(c_ptr)                            :: sdl_rw_from_const_mem
        end function sdl_rw_from_const_mem

You can pass arbitrary data by using the ISO C binding routine c_loc(), for instance:
res = sdl_rw_from_const_mem(c_loc(data), size)

(The variable data needs the target attribute.)

Thanks. I'm closing this.


css: Make single-line codeblocks inline.

GIFs or Screenshots:
Before:

After:

@zulipbot label "bug" "area: message view"

Can you test what this looks like with very long lines inside an inline code block?  Just want to make sure those look correct.

Yes, it works as expected. I've tried it on Firefox 62.
With display:inline-block:

Without display: inline-block:

OK.  I guess in that specific case, I think the inline-block version is worse.  Maybe worth looking at whether there's a word-wrap setting or something we should be toggling instead to fix the first problem without creating the second?
(@rishig FYI)

On second thought, yes, this looks more like a codeblock now. However, word wrap would just apply to the text section, but the :before and ::after elements used to construct the rounded look aren't affected by it.

OK, sounds like we don't have an improvement to the styling we can make here?

Closing this, since I think the best fix we've been able to find causes worse problems than the original bug.
@synicalsyntax if you want to play around with CSS options for this, it could be a good puzzle.


Cannot run on EC2 server

I am trying to run the program on a Ubuntu EC2 server, and I have set it up, google-chrome is also installed, but when I am getting an error Failed to launch browser process, and when i try to launch chrome i get another error sayin that i cannot run it without sandbox
The errors in question:
[2022-12-18 16:58:20] [error] Error: Failed to launch the browser process! spawn /usr/bin/google-chrome /usr/share/man/man1/google-chrome.1.gz ENOENT
TROUBLESHOOTING: https://github.com/puppeteer/puppeteer/blob/main/docs/troubleshooting.md
at onClose (/home/ubuntu/botviewer/Twitch-Drops-Bot/node_modules/puppeteer-core/lib/cjs/puppeteer/node/BrowserRunner.js:299:20)
at ChildProcess.<anonymous> (/home/ubuntu/botviewer/Twitch-Drops-Bot/node_modules/puppeteer-core/lib/cjs/puppeteer/node/BrowserRunner.js:293:24)
at ChildProcess.emit (node:events:513:28)
at ChildProcess._handle.onexit (node:internal/child_process:291:12)
at onErrorNT (node:internal/child_process:483:16)
at process.processTicksAndRejections (node:internal/process/task_queues:82:21)

And
google-chrome --no-sandbox
[71752:71752:1218/170205.539633:ERROR:ozone_platform_x11.cc(238)] Missing X server or $DISPLAY
[71752:71752:1218/170205.539902:ERROR:env.cc(255)] The platform failed to initialize.  Exiting.
[1218/170205.564135:ERROR:nacl_helper_linux.cc(355)] NaCl helper process running without a sandbox!
Most likely you need to configure your SUID sandbox correctly

You'll need to be running it from the desktop, it won't work in an SSH terminal.
https://ubuntu.com/tutorials/ubuntu-desktop-aws


Support code-blocks in unordered lists

This time I really don't find this 😆😐👀, Can you provide the page link...!.

any update


Update PostgreSQL

Do we want to use PostgreSQL 10?
Let's discuss

As we discussed in our meeting, I think it's a good idea. There are some nice features related to full text search and JSONB that I think we will eventually find useful.
I'm going to do a dry run migration to PostgreSQL 10 in staging. If it's easy (and it should be), we can move production over early next week.

PostgreSQL 10.3 is in production


feat: add newest and oldest properties to releases

This is a lingering branch I had on my mac that I seemingly never pushed. Wanted to get it up before I tried testing other PRs.

going to close this as I'm not sure if it's needed anymore.


Native offset

Everything other is fine with me.
Just let me know - and I'll polish those simple concerns and merge!

@artembilan I commented on the constructor. Agreed on all the accounts, otherwise. I'll try to push some fixes tomorrow morning, but feel free to make the changes on merge.

@mbogoevici , since you haven't done anything here since the last our discussion, so I go ahead with our conclusion and fixing those concerns on merge.
Let me know if that should be some other way.


Using Moor in an e2e test

Is it possible to use Moor in unit tests? For example, wanting to run business logic tests in Circle CI?

Yes. moor_ffi can run outside of Flutter which makes it suitable for unit tests.


tests: add rule to check for tcp/ack - v4

Test for rule type for tcp-ack keyword.
Related to
Issue: 6354
Redmine ticket: https://redmine.openinfosecfoundation.org/issues/6354
Suricata PR: https://github.com/OISF/suricata/pull/
Previous PR: https://github.com/OISF/suricata-verify/pull/1432

Is a README not needed for these tests?

Actually, I don't know.
I did not include it for others and they were approved.

Actually, I don't know. I did not include it for others and they were approved.

I see, thanks. I'll let Juliana decide on that.
We'll need a min-version check of 8 not 7 in test.yaml. This feature isn't and won't be available on 7, I think. I'm unsure though why haven't we flagged that in the previous versions of this and other PRs. 🤔

Actually, I don't know. I did not include it for others and they were approved.

I see, thanks. I'll let Juliana decide on that. We'll need a min-version check of 8 not 7 in test.yaml. This feature isn't and won't be available on 7, I think. I'm unsure though why haven't we flagged that in the previous versions of this and other PRs. 🤔

Ah, I see.
So, I'll have to wait for comments on this then...

Actually, I don't know. I did not include it for others and they were approved.

I see, thanks. I'll let Juliana decide on that. We'll need a min-version check of 8 not 7 in test.yaml. This feature isn't and won't be available on 7, I think. I'm unsure though why haven't we flagged that in the previous versions of this and other PRs. 🤔

Indeed, I don't know, myself. 🤔 Could it be because previous versions were before we opened the 8.0.x branch? (didn't check to confirm)

Is a README not needed for these tests?

So far, none of the tests created for engine-analysis have them... I've assumed that as they're a different category of tests, we didn't see README files as needed, here. Should we change that approach?

Is a README not needed for these tests?

So far, none of the tests created for engine-analysis have them... I've assumed that as they're a different category of tests, we didn't see README files as needed, here. Should we change that approach?

Well, it could be necessary, so the README file carries a description of what we are testing for, especially for persons new to suricata?

Is a README not needed for these tests?

So far, none of the tests created for engine-analysis have them... I've assumed that as they're a different category of tests, we didn't see README files as needed, here. Should we change that approach?

Well, it could be necessary, so the README file carries a description of what we are testing for, especially for persons new to suricata?

I guess it can't hurt to add one, if you're willing to add a good description :)

Is a README not needed for these tests?

So far, none of the tests created for engine-analysis have them... I've assumed that as they're a different category of tests, we didn't see README files as needed, here. Should we change that approach?

Well, it could be necessary, so the README file carries a description of what we are testing for, especially for persons new to suricata?

I guess it can't hurt to add one, if you're willing to add a good description :)

OK.
Will do that.
Done for the other PR on tcp-seq.


Completion for Fish-Shell

Hello,
it would be nice to have autocompletion for fish-shell.

Hello. I've been using fish shell for over a year now I am interested in implementing this feature for Hub. As @narqo said, the problem lies in the fact that Hub's man pages does not list all it's options (assuming that it is aliased as git).
I know it a longer route, but would reformatting the Hub man pages help?

@hasit Yes, we need to improve our man pages (e.g. break the big man page that we have into individual ones), and it would be great if the same pages could be used by fish to extract autocomplete information.

Started working on some initial script
https://github.com/lukaszlenart/hub-fish-completion


Dependencies

Tried running but dockerizing did not work. Said that it did not find the dockerfile.

It's fixed now. If the error persists, copy the Dockerfile to one directory above. It will work


Add support for GitHub Copilot

Adds support to interact with OpenAi GPT-4 Models using Copilot.

something like this:
diff --git a/config_template.yml b/config_template.yml
index 432a580..d806436 100644
--- a/config_template.yml
+++ b/config_template.yml
@@ -92,6 +92,12 @@ apis:
         fallback:
   copilot:
     base-url: https://api.githubcopilot.com
+    models:
+      gpt-4o-2024-05-13:
+        max-input-chars: 392000
+        fallback: gpt-4
+      gpt-4:
+        max-input-chars: 392000
   anthropic:
     base-url: https://api.anthropic.com/v1
     api-key:
diff --git a/mods.go b/mods.go
index 0f2ea63..2017dc2 100644
--- a/mods.go
+++ b/mods.go
@@ -3,6 +3,7 @@ package main
 import (
 	"bufio"
 	"context"
+	"encoding/json"
 	"errors"
 	"fmt"
 	"io"
@@ -351,34 +352,25 @@ func (m *Mods) startCompletionCmd(content string) tea.Cmd {
 			if mod.API == "azure-ad" {
 				ccfg.APIType = openai.APITypeAzureAD
 			}
+		case "copilot":
+			filePath := os.Getenv("HOME") + "/.config/github-copilot/hosts.json"
+			bts, err := os.ReadFile(filePath)
+			if err != nil {
+				return err
+			}
+			hosts := map[string]map[string]string{}
+			if err := json.Unmarshal(bts, &hosts); err != nil {
+				return err
+			}
+			authToken := hosts["github.com"]["oauth_token"]
+			ccfg = openai.DefaultConfig(authToken)
+			ccfg.BaseURL = api.BaseURL
 		default:
 			var key string
-			if cfg.API == "copilot" {
-				filePath := os.Getenv("HOME") + "/.config/github-copilot/hosts.json"
-				data, err := os.ReadFile(filePath)
-				if err != nil {
-					return err
-				}
-				fileContent := string(data)
-				re := regexp.MustCompile(`.*oauth_token...|\".*`)
-				key = re.ReplaceAllString(fileContent, "")
-				if key == "" {
-					return fmt.Errorf("No Copilot OAuth token found in %s", filePath)
-				}
-				key = strings.TrimSpace(key)
-				ccfg = openai.DefaultConfig(key)
-				for _, a := range cfg.APIs {
-					if "copilot" == a.Name {
-						api = a
-						break
-					}
-				}
-			} else {
-				var err error
-				key, err = m.ensureKey(api, "OPENAI_API_KEY", "https://platform.openai.com/account/api-keys")
-				if err != nil {
-					return err
-				}
+			var err error
+			key, err = m.ensureKey(api, "OPENAI_API_KEY", "https://platform.openai.com/account/api-keys")
+			if err != nil {
+				return err
 			}
 			ccfg = openai.DefaultConfig(key)
 			if api.BaseURL != "" {
@@ -447,7 +439,7 @@ func (m *Mods) handleRequestError(err error, mod Model, content string) tea.Msg
 	if errors.As(err, &ae) {
 		return m.handleAPIError(ae, mod, content)
 	}
-	return modsError{ae, fmt.Sprintf(
+	return modsError{err, fmt.Sprintf(
 		"There was a problem with the %s API request.",
 		mod.API,
 	)}

although, testing this, it doesn't seem to work...

Yes, that doesn't work because of the way we select the value of mod.API. At that point in the code, the value is openai, thus it never falls into that case statement.
That's why I added the check inside the default case.

tried to get it to work in #345, but I always get error responses from copilot... I think we're missing something, not sure what though

Mmm... I have it working on my end, albeit with my crude implementation:

I'll check that PR and see if I can make it work

can you run something like mods --api copilot "who are you?"? Just to make sure it is really copilot replying

Since they are both using the same model, they tend to answer the same thing, so I added back the logs:

yeah, for me both branches give the same error

Mmm...
      },
      {
        "role": "user",
        "content": "Who won the world series in 2020?"
      },
      {
        "role": "assistant",
        "content": "The Los Angeles Dodgers won the World Series in 2020."
      },
      {
        "role": "user",
        "content": "Where was it played?"
      }
    ]
  }'

I get an "unknown integration" error

That command work for me, I wonder what's the difference.

That command work for me, I wonder what's the difference.

same... no idea, it works inside neovim apparently... dunno what's the difference 🤔


Change "Ø Rating" to "Avg. Rating"

Change "Ø Rating" to "Avg. Rating" on the game list because this symbol is not commonly known to mean "Average" in some English-speaking countries. Should probably change in other localizations as well but I don't know how other languages would commonly abbreviate or otherwise represent "Average" so I don't want to change it myself and make it worse.

Next time please reference the issue by writing "Fixes #number" in your commit

localization is not done here but in poeditor


Update test fixtures

This isn't passing, but this is the branch we've been releasing from as a workaround for a few days now. I'm working on the fix, however, in the meantime I'm going to bypass the failing checks (since the reason for the failure is known and acceptable ) in order for the daily DB builds to continue to work. I'll follow up with a fix next week.


fix: compose error due to leftover config

thank you

you have to pull before push to make this request


Indium 1.0.4+mc1.19  java.lang.IllegalArgumentException: wrong number of arguments

mods loaded (with PolyMC 1.3.1): Fabric API 0.55.2+1.19, Sodium 0.4.2+build.16, Indium 1.0.4+mc1.19
launching minecraft works, but this error shows up
java.lang.IllegalArgumentException: wrong number of arguments
at jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance0(Native Method) ~[?:?]
at jdk.internal.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:77) ~[?:?]
at jdk.internal.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45) ~[?:?]
at java.lang.reflect.Constructor.newInstanceWithCaller(Constructor.java:499) ~[?:?]
at java.lang.reflect.Constructor.newInstance(Constructor.java:480) ~[?:?]
at link.infra.indium.renderer.aocalc.VanillaAoHelper$1.get(VanillaAoHelper.java:55) [indium-1.0.4+mc1.19.jar:?]
at link.infra.indium.renderer.aocalc.VanillaAoHelper$1.get(VanillaAoHelper.java:51) [indium-1.0.4+mc1.19.jar:?]
at link.infra.indium.renderer.aocalc.VanillaAoHelper.initialize(VanillaAoHelper.java:66) [indium-1.0.4+mc1.19.jar:?]
at net.minecraft.class_778.handler$zii000$onInit(class_778.java:566) [client-intermediary.jar:?]
at net.minecraft.class_778.(class_778.java:40) [client-intermediary.jar:?]
at net.minecraft.class_776.(class_776.java:39) [client-intermediary.jar:?]
at net.minecraft.class_310.(class_310.java:557) [client-intermediary.jar:?]
at net.minecraft.client.main.Main.main(Main.java:198) [client-intermediary.jar:?]
at net.fabricmc.loader.impl.game.minecraft.MinecraftGameProvider.launch(MinecraftGameProvider.java:461) [fabric-loader-0.14.7.jar:?]
at net.fabricmc.loader.impl.launch.knot.Knot.launch(Knot.java:74) [fabric-loader-0.14.7.jar:?]
at net.fabricmc.loader.impl.launch.knot.KnotClient.main(KnotClient.java:23) [fabric-loader-0.14.7.jar:?]
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke0(Native Method) ~[?:?]
at jdk.internal.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:77) ~[?:?]
at jdk.internal.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:43) ~[?:?]
at java.lang.reflect.Method.invoke(Method.java:568) ~[?:?]
at org.multimc.impl.OneSixLauncher.invokeMain(OneSixLauncher.java:104) [NewLaunch.jar:?]
at org.multimc.impl.OneSixLauncher.launchWithMainClass(OneSixLauncher.java:175) [NewLaunch.jar:?]
at org.multimc.impl.OneSixLauncher.launch(OneSixLauncher.java:185) [NewLaunch.jar:?]
at org.multimc.EntryPoint.listen(EntryPoint.java:127) [NewLaunch.jar:?]
at org.multimc.EntryPoint.main(EntryPoint.java:57) [NewLaunch.jar:?]
Full log uploaded:
https://mclo.gs/5IKebix

Hmm, looks like this was fixed in Indigo with https://github.com/FabricMC/fabric/commit/56447d9b95f88da22dab4904831ed013e55ee390; I'll see if I can port over the change... not sure if the change to AWs was for performance or just to reduce the amount of code?

I'm not a modder and therefore no idea what's the question. Just wanted to share this and seems like you already know what's the cause ;) do I need to do anything on my side or simply wait for an update?

I've managed to replicate the issue, thanks for reporting it!

still haven't figured out what might be not working with this error ... what does it mean?

It's an issue with Indium itself that requires an update.


Received linker warning LNK4221

I received numerous linker warnings:
warning LNK4221: This object file does not define any previously undefined public symbols, so it will not be used by any link operation that consumes this library
Do not know if this is normal.

Thanks for your report.

Do not know if this is normal.

Since these are warnings specific to Qt, I am closing the issue.
I suggest you ignore these.
Ideally, these warnings should be addressed upstream in the Qt project itself.
Considering qt accepts only critical patches for 4.8.x maintenance branch, if the warnings are also reported when building Qt5, it may make sense to report upstream.


Separate instance for different terminal emulators?

Hey there!
I'm currently running with iTerm as my terminal emulator and ZSH as my shell. I'm testing out hyper as a replacement. Hyper has an option shell, which is described as "A path to a custom shell to run when Hyper starts a new session".
I'd love to be able to use a separate instance of ZSH and Oh My Zsh for hyper. Is this functionality implemented and if so how can I use it? I've been looking, but I can't find anything about this.

You would have to make a custom setup for this, either using a separate user or loading a separate zshrc file based on how hyper calls the shell.
Maybe you can run zsh with a custom parameter in Hyper and detect that parameter? Or you could use a symlinked zsh and detect that with $0. I don't actually know any solid solutions.


API-1835: Don't use Update in removestaleconditionscontroller

Proof: https://github.com/openshift/cluster-authentication-operator/pull/729
Manual testing done:

Introduced the removestaleconditions controller in https://github.com/openshift/gcp-pd-csi-driver-operator and marked the ManagementStateDegraded condition to be removed.
Removed the managementstate controller from the operator.
Watched the removestaleconditions controller remove the condition correctly.
Added the managementstate controller back to the operator and watched both controller fighting.

/hold cancel

/lgtm


Can you consider using spring-boot?

Pinpoint server is using war file.
Can you consider using spring-boot?
Zipkin support spring-boot, so we can deploy easy.
Thanks you.

Good point, this will make running the collector and web a lot easier.
Will definitely keep this in mind.

Low priority.


openedge-plugin

Description
Add OpenEdge plugin, only one committer for now.
Repository URL: https://github.com/jenkinsci/openedge-plugin
HOSTING issue: https://issues.jenkins-ci.org/browse/HOSTING-776
Submitter checklist for changing permissions

Always

[x] Add link to plugin/component Git repository in description above

For a newly hosted plugin only

[x] Add link to resolved HOSTING issue in description above

For a new permissions file only

[x] Make sure the file is created in permissions/ directory
[x] artifactId (pom.xml) is used for name (permissions YAML file).
[x] groupId / artifactId (pom.xml) are correctly represented in path (permissions YAML file)
[x] Check that the file is named plugin-${artifactId}.yml for plugins

When adding new uploaders (this includes newly created permissions files)

[ ] Make sure to @mention an existing maintainer to confirm the permissions request, if applicable
[x] Use the Jenkins community (LDAP) account name in the YAML file, not the GitHub account name
[x] All newly added users have logged in to Artifactory at least once

FYI, this is not for commit access, but access to do releases of the plugin.


Build WASM target

How would I go about building a WASM target?

.github/workflow/release.yml has a WASM step, and the WASM build artifact can be found at https://www.npmjs.com/package/@tlaplus/tree-sitter-tlaplus?activeTab=code

Consider pointing to https://www.npmjs.com/package/@tlaplus/tree-sitter-tlaplus from the Github release page.

I updated the README with more prominent instructions on how to build the WASM target, and also noted in the summary that it is published in the NPM package.


Improve Pagespeed Score

This PR:

Move css and js from head to body
Move critical css to style element in head

I use VSCode so some stuff was auto formatted. The score went from 71 to 91 with these improvements.
Further improvements can be to add a minifier for JS.
Before:

After:

@tyricec LGTM. I'm busy at the moment, but I will try to review it soon.

🎉 It works:

✅ Fixes #9.


Download button layout on release notes

The "source" button wraps even though there's plenty of room (and when it does wrap the styling is odd since there's no vertical margin between "source" and "windows"). My windows are half the width of my external monitor; window.innerWidth is 1251.

Addressed in #3404.


Not playing track in FireFox

Used to work fine. I was recently made aware that it doesn't play anymore. No error message in console. Album art loads, which means API call to SoundCloud is working ok. Track data seems to buffer judging by visual cue on the player control, but playback doesn't start, even when pressing the play button.

This one's been stumping me too. Any progress on this?

Awesome visualiser by the way. After reading your tutorial I'm attempting one now combined with three.js to get create some 3d visuals.

Hi - thanks! No, I've not done anything on this issue, although I read somewhere recently that Firefox does not allow remote sources for Web Audio, so this may well be the issue. More info here: http://stackoverflow.com/questions/28984883/cannot-analyse-soundclouds-streaming-audio-because-of-the-lack-of-cors-policy

Yes, the solution is:
audioElement.crossOrigin = "anonymous";

You can find more informations about it on MDN - CORS settings attributes.
This is also a "problem" in Chrome >= 42.

Thanks @TimPietrusky. Just noticed that Chrome had updated to v42 on my machine and the visualization stopped working there too. Adding the line you suggest fixes it, but not for every track.
Needs more investigation...

@michaelbromley can you give me a track URL that's not working? Because I'm using the fix in my project and it's working for every track I try.

Yeah try this: https://soundcloud.com/muse/undisclosed-desires

I debugged this again and I found out that some of the SoundCloud streaming servers are not sending the CORS-header.
The CORS-problem was reported to SoundCloud one month ago.


Travis fix

I've tried to fix the Travis builds:

Multi-platform nightwatch configuration
Fixed gulp so that nightwatch actually exits after finishing
Using headless chromium

Unfortunately it still doesn't work. gulp test works fine for me locally (Linux) and I have no idea what the error message in Travis means. Any ideas?

hi @GenaBitu sorry for the delay in answering. As for your work: Dont know why that still thrwos error. But to be honest: I am not very satisfied with (gulp-)nightwatch. Maybe you know some other easy to implement test-solution for those few test cases we have?

Don't worry... I have absolutely no idea, I didn't even know that you could do something like this before looking into this issue.

I've recently read the 2018 State of JS and it seems most people are using Jest and (probably more importantly) are very happy with it...
I've never used it (or any other JS testing tool)...

Jest is more for unit testing, if I see it correctly (or for React tests)...

Fixed testing with https://github.com/rejas/imagelightbox/pull/189


Cannot create subscription inside topic in Azure Service Bus (JavaScript)

Package Name: @azure/service-bus
Package Version: ^7.9.5
Operating system: macOS 14.6.1 (but tested on Windows 11 - not working as well)
[ ] nodejs

version: v20.10.0

[ ] browser

name/version: Mozilla Firefox 130 but also 115

[ ] typescript

version: ^5.0.4

Is the bug related to documentation in

[ ] README.md
[ ] source code documentation
[ ] SDK API docs on https://learn.microsoft.com

Describe the bug
The bug is happening on Mozilla Firefox browser only (so far).
It seems that while executing subscription creation, there is a problem with xml payload created only on Mozilla Firefox browser.
To Reproduce
Steps to reproduce the behavior:

Create subscription like:

this._sbAdminClient.createSubscription(
                        `myApp-${user}`,
                        azureSubscriptionId,
                        {
                            autoDeleteOnIdle: 'PT1H',
                        }
                    )
// {user} variable here is just a simple string and it does contain only simple latin characters

Expected behavior
Fix ASAP Mozilla Firefox integration with your library or provide a workaround.
Screenshots
n/a
Additional context
XML faulty payload generated by Firefox:
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<entry>
    <updated>2024-09-03T07:41:08.210Z</updated>
    <content type="application/xml">
        <SubscriptionDescription
            xmlns:i="http://www.w3.org/2001/XMLSchema-instance">
            <AutoDeleteOnIdle>PT1H</AutoDeleteOnIdle>
        </SubscriptionDescription>
    </content>
</entry>

Response:
400 BAD_REQUEST
Body:
<Error>
400
<Detail>The specified resource description is invalid.
 TrackingId: <id>,
 SystemTracker:*name of my service bus*:Topic:*name of my topic*,
 Timestamp:2024-08-16T08:05:54
</Detail>
</Error>

@CezaryBD thanks for reporting the issue!  It may be related to this issue https://github.com/Azure/azure-sdk-for-js/issues/4983
However, even if we had correct XML payload, this scenario wouldn't work in a secure environment either due to Azure Service Bus service not supporting CORS for the ATOM management API in browsers: https://github.com/Azure/azure-sdk-for-js/issues/4983

Hi @jeremymeng, thanks for quick response. What is the workaround or proposed solution? Our project's functionality relies on this API's method.

@CezaryBD there's an untested and unsupported way to switch to use fast-xml-parser.  It will increase the bundle size of your app.
cat<EMAIL_ADDRESS>diff --git<EMAIL_ADDRESS><EMAIL_ADDRESS>index e3caeef..595f7fb 100644
---<EMAIL_ADDRESS>+++<EMAIL_ADDRESS>@@ -1,5 +1,5 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT license.
-export { stringifyXML, parseXML } from "./xml.js";
+export { stringifyXML, parseXML } from "../esm/xml.js";
 export { XML_ATTRKEY, XML_CHARKEY } from "./xml.common.js";
 //# sourceMappingURL=index.js.map
\ No newline at end of file```

I currently have no CORS issues whatsoever, but I do have issues that e.g. in the Microsoft Edge the payload generated looks like this:
<?xml version="1.0" encoding="UTF-8" standalone="yes"?>
<entry
    xmlns="http://www.w3.org/2005/Atom">
    <updated>2024-09-03T06:39:22.891Z</updated>
    <content type="application/xml">
        <SubscriptionDescription
            xmlns="http://schemas.microsoft.com/netservices/2010/10/servicebus/connect"
            xmlns:i="http://www.w3.org/2001/XMLSchema-instance">
            <AutoDeleteOnIdle>PT1H</AutoDeleteOnIdle>
        </SubscriptionDescription>
    </content>
</entry>

whereas generated for Firefox looks like in the original ticket.
Will there be a tested and supported way or do you completely stop supporting browsers?
Yes, I need to run Service Bus subscription creation code on our browser side, this is a critical part of our project's architecture.

If I move it to our backend, what's the point of existence for this library?

This package supports both browsers and NodeJS.
I'd thought I linked to this above but apparently, I linked to the same issue twice...  Yes, this is a bug on only FireFox.  We will look into it. https://github.com/Azure/azure-sdk-for-js/issues/11655

This has been fixed in @azure/core-xml 1.4.4


ERROR: Archive file or some parts of its content invalid: file.jar

Could you discribe the problom in detail, and you can send the jar file to me at<EMAIL_ADDRESS>And I will check that if there is something wrong with it.
Thank you again for reporting issues.

And it works well on my PC as the image below

Therefore, please send me your jar file at sunyukai584595410@126.com,and I will check that.

Or you can open your regedit.exe and delete the "FernflowerUI_MFC" item as the image below.

Maybe this will work

@6168218c well, I will send all the information to you on the email

@lokha Now the issue is fixed in version 3.4.1,which may be released in August.(I'm working on other features in version 3.4.1)In version 3.4.1 and later , the program will use the fernflower made by @JetBrains,and it can decompile your jar file correctly.

@6168218c it is great

@lokha Thank you again for reporting the issue,and thank you so much for your support:smile:.


Extension Onepace Source is broken

Source information and language
Onepace 13.8
Steps to reproduce
Open the  Extension
Open in webview
Expected behavior
Open the source https://www.zippyshare.com/rest/public/getTree?user=onepace&ident=kbvatgfc&id=%23
Actual behavior
403 Forbidden
Aniyomi version
<IP_ADDRESS> and r5237
Android version
Android 12
Other details
No response
Acknowledgements

[X] I have searched the existing issues and this is a new ticket, NOT a duplicate or related to another open issue.
[X] I have written a short but informative title.
[X] I have updated the app to version 0.13.5.
[X] I have updated all installed extensions.
[X] I have tried the troubleshooting guide.
[X] If this is an issue with the app itself, I should be opening an issue in the app repository.
[X] I will fill out all of the requested information in this form.

I suggest to use the URL onepace.watch Instead of the broken one


channel_analysis: ReferenceError: diff is not defined

Channel analysis page stuck at "Starting wireless scan...".
Caught in Firefox Browser Console:
Uncaught (in promise) ReferenceError: diff is not defined
    handleScanRefresh http://<IP_ADDRESS>/luci-static/resources/view/status/channel_analysis.js?v=git-22.351.50506-a27d2db:234
    handleScanRefresh http://<IP_ADDRESS>/luci-static/resources/view/status/channel_analysis.js?v=git-22.351.50506-a27d2db:232
    promise callback*handleScanRefresh http://<IP_ADDRESS>/luci-static/resources/view/status/channel_analysis.js?v=git-22.351.50506-a27d2db:180
    step http://<IP_ADDRESS>/luci-static/resources/luci.js?v=git-22.351.50506-a27d2db:1189
    setInterval handler*start http://<IP_ADDRESS>/luci-static/resources/luci.js?v=git-22.351.50506-a27d2db:1149
    add http://<IP_ADDRESS>/luci-static/resources/luci.js?v=git-22.351.50506-a27d2db:1095
    render http://<IP_ADDRESS>/luci-static/resources/view/status/channel_analysis.js?v=git-22.351.50506-a27d2db:424
    promise callback*__init__ http://<IP_ADDRESS>/luci-static/resources/luci.js?v=git-22.351.50506-a27d2db:1945
    super http://<IP_ADDRESS>/luci-static/resources/luci.js?v=git-22.351.50506-a27d2db:299
    ClassConstructor http://<IP_ADDRESS>/luci-static/resources/luci.js?v=git-22.351.50506-a27d2db:107
    compileClass http://<IP_ADDRESS>/luci-static/resources/luci.js?v=git-22.351.50506-a27d2db:2526
    promise callback*compileClass http://<IP_ADDRESS>/luci-static/resources/luci.js?v=git-22.351.50506-a27d2db:2502
    promise callback*require http://<IP_ADDRESS>/luci-static/resources/luci.js?v=git-22.351.50506-a27d2db:2541
    instantiateView http://<IP_ADDRESS>/luci-static/resources/ui.js?v=git-22.351.50506-a27d2db:4873
    <anonymous> http://<IP_ADDRESS>/cgi-bin/luci/admin/status/channel_analysis:42
    promise callback* http://<IP_ADDRESS>/cgi-bin/luci/admin/status/channel_analysis:41
luci.js:234:6

DISTRIB_ID='OpenWrt'
DISTRIB_RELEASE='SNAPSHOT'
DISTRIB_REVISION='r21559-1e375c8fbd'
DISTRIB_TARGET='ramips/mt7621'
DISTRIB_ARCH='mipsel_24kc'
DISTRIB_DESCRIPTION='OpenWrt SNAPSHOT r21559-1e375c8fbd'
DISTRIB_TAINTS='no-all'

This is a bug introduced with 75dcb097547257d654e9b7f7d58578b9210968e5, so cc @Ansuel
Possible fix:

diff --git a/modules/luci-mod-status/htdocs/luci-static/resources/view/status/channel_analysis.js b/modules/luci-mod-status/htdocs/luci-static/resources/view/status/channel_analysis.js
index 9f266ebef3..a24742890d 100644
--- a/modules/luci-mod-status/htdocs/luci-static/resources/view/status/channel_analysis.js
+++ b/modules/luci-mod-status/htdocs/luci-static/resources/view/status/channel_analysis.js
@@ -230,7 +230,8 @@ return view.extend({
 					results.push(scanCache[k].data);

 			results.sort(function(a, b) {
-				if (a.channel - b.channel)
+				var diff = a.channel - b.channel;
+				if (diff)
 					return diff;

 				if (a.ssid < b.ssid)

It doesn't work directly for me.

root@archer_c60:~# cat /etc/openwrt_release
DISTRIB_ID='OpenWrt'
DISTRIB_RELEASE='SNAPSHOT'
DISTRIB_REVISION='r21569-9b6dd57a33'
DISTRIB_TARGET='ath79/generic'
DISTRIB_ARCH='mips_24kc'
DISTRIB_DESCRIPTION='OpenWrt SNAPSHOT r21569-9b6dd57a33'
DISTRIB_TAINTS='no-all'

Channel analysis page stuck at "Starting wireless scan...".

Possible fix:

The "possible fix" by @dhewg works for me.
It makes the scan to work again, instead of being stuck.
(Note that after editing the js file on the live router, you likely need to clear your browser cache before you can see the new code in use)
@ansuel , could you please look into this?
Ps.
Curiously, in some conditions the "local interface" is left undrawn?
Note the missing pillar here...

I noticed that too, I tried to look into it but the sauce is just a mess and I keep walking away from it :P
But I think it's related to the channel width "logic". HE just gets ignored, even some HT/VHT nets get the wrong width. And 6Ghz isn't supported at all.
If we get #6104 there're some new ubus values which can be used to clean up some of this mess I guess.

Hi sorry I will look into this ASAP.
Il Ven 23 Dic 2022, 09:15 dhewg @.***> ha scritto:

I noticed that too, I tried to look into it but the sauce is just a mess
and I keep walking away from it :P
But I think it's related to the channel width "logic". HE just gets
ignored, even some HT/VHT nets get the wrong width. And 6Ghz isn't
supported at all.
If we get #6104 https://github.com/openwrt/luci/pull/6104 there're some
new ubus values which can be used to clean up some of this mess I guess.
—
Reply to this email directly, view it on GitHub
https://github.com/openwrt/luci/issues/6167#issuecomment-1363723135, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AE2ZMQXBDVYHX7J2F4IPBMDWOVNT5ANCNFSM6AAAAAATGPW4EY
.
You are receiving this because you were assigned.Message ID:
@.***>

@hnyman well the problem is just that the thing doesn't account for HE width channel... easy fix. Also I have the fix for this error... @dhewg for wrong width it's strange everything is from ubus and parsed. Would be good to have some sample data to repro the problem!

@dhewg for wrong width it's strange everything is from ubus and parsed. Would be good to have some sample data to repro the problem!

Sure, I'll collect some data and post it here, but you're probably right and the iwinfo/ubus data is off.
One thing I noticed is that one ac/ax/b/g/n radio only has one net on the 6ghz band, channel 13, which shows up on the 2.4 tab, spanning from 0 to 14 :D
But I guess it makes sense to wait for https://github.com/openwrt/openwrt/pull/11280. That gives "band", with which this should be easy to fix. And add support for the 6ghz band. And maybe even cache scan results for one radio for all bands, so one doesn't have to rescan when switching between bands on the same radio

Poked around a little, the channel widths are off if wpa_supplicant is used for scanning, because its cli/socket interface doesn't provide that info :\
So that doesn't work if there's an client or mesh connection active on that radio...
If I comment the wpa_supplicant scan in iwinfo, it works, but probably only because I have additional non client/mesh nets on that radio?
Without code changes:
iwinfo radio1 scan: no [V]HT properties are set, hence everything looks as 20MHz.
But the props are present on iwinfo phy2 scan (where iwinfo nl80211 phyname radio1 = phy2`) because that doesn't use wpa_supplicant for me.
But that may not work everywhere too, because using phyX gets one net on that phy for scanning, which in return gets the last one returned by readdir(), so it's not guaranteed to get one non client/mesh one /o\
This is all so fragile it's frustrating...

I added some iwinfo patches to https://github.com/openwrt/openwrt/pull/11280 to improve this situation. Now it prefers non-supplicant-based devices, so we get e.g. HT/VHT operation infos.
This doesn't fix the bug altogether, if there's just a single supplicant based device configured for one radio, HT/VHT is of course still missing. But if there're multiple radios you get that info at least, which fixes that situation for me.


android: egui editor

next time you make progress here can you add a gitignore and squash your branch,

these files are slowing down a number of operations that rely on git


Default readiness timeouts

Is your feature request related to a problem? Please describe.
Setting a default readiness timeout that is long enough (but still reasonable; example 3m) would be useful. This will eliminate the need to specify timeout values (at least in tutorials, if not in prod).

this seems to be done. a default exists. closing.


Add new terminal failure binding condition

(#1051)
This adds a failure condition for bindings and adds a check to not retry once a binding is in this failed condition status.

I'd prefer that any lacking documentation in the tests be done in a follow up as part of issue #1077.

LGTM, thanks for the patch.

@jpeeler I am going to merge and I'm ok with us doing the docs changes in a follow-up


Feat: Create a checkout page that opens on clicking buy now option in home page

Describe the feature
When buy now is clicked, this page will open
Add ScreenShots
Will add once done
Record

[X] I agree to follow this project's Code of Conduct
[X] I'm a GSSOC'24 contributor
[ ] I want to work on this issue

Assigned!
Make sure to attach before & after screenshots in your PR and include the issue number.
Also, please star the repo before working on it. Happy open-source contributing!


Create bundle of developer toolkit

Currently we can't bundle the developer toolkit and for most of the command it doen't generate report.

#6


Text formatting Waypoint: Get JSON with the jQuery getJSON Method

The requirement text has an extra \ as if it was trying to be escaped. Don\'t forget to make the .html change the contents of the message box so that it contains the result of the getJSON.

Looks like this issue was fixed but didn't close correctly. I guess Github is a little picky on how to close issues :stuck_out_tongue:

@ltegman maybe close this? :-)


Are there any discussion forums / mailing lists the maintainers should know about? Please let us know!

(title)

I'm currently sitting in the #cli channel in the gophers slack https://invite.slack.golangbridge.org/
I would love to be on whatever mailing list this https://blog.golang.org/v2-go-modules was posted to???

closing this now b/c it's falling off my radar, but please do add replies if you have any links for us 🙏


Is it mandatory to use MQTT over Websocket on Charge point device or Just Websocket is sufficient?

Dear @galinsky ,
Thanks to confirm if it is mandatory to use MQTT over Websockets on Charge point device?
OR
The communication between Charge point device and OCPP gateway can be based on Websockets and OCPP gateway internally uses MQTT communication with its internal services.
Regards,
Bhuvi-Naveen

If I understood your question, it’s the latter: charge point device to the gateway uses the OCPP protocol over WebSockets, while the gateway to the backend service(s) uses MQTT.

Hi @galinsky,
Thank you for your response!
I have one more question, could you please let me know if OCPP Gateway code used in the proposed solution is developed by Amazon or it is third party stack? Also is it possible to host our own OCPP gateway code in the proposed solution instead of AWS provided OCPP Gateway? please advise.
Regards,
Bhuvi-Naveen

@Bhuvi-Naveen the OCPP Gateway code included here was developed by Amazon. It takes dependencies on third-party libraries, specifically the ocpp python library used in message_processor.py. If your question is around using your own gateway solution instead of the ocpp-gateway-container code, that would surely be something you are free to do. You can also freely modify the gateway.py code to your own business needs.

Thanks Garry for the confirmation.
Regards,
Bhuvi-Naveen


Stage pod unable to validate against any SCC while deleting annotations

From PR #227. It appears something with the DeleteAnnotations function is causing this error to populate. It does not seem related to the changes we are merging in.

I am seeing a log statement after the stage restore runs when deleting annotations on the stage resources:
{"level":"error","ts":1564518878.9188626,"logger":"migration|t89hk","msg":"","error":"pods \"mariadb-1-jc48w-stage\" is forbidden: unable to validate against any security context constraint: []","stacktrace":"github.com/fusor/mig-controller/vendor/github.com/go-logr/zapr.(*zapLogger).Error\n\t/go/src/github.com/fusor/mig-controller/vendor/github.com/go-logr/zapr/zapr.go:128\ngithub.com/fusor/mig-controller/pkg/logging.Logger.Error\n\t/go/src/github.com/fusor/mig-controller/pkg/logging/logger.go:59\ngithub.com/fusor/mig-controller/pkg/logging.Logger.Trace\n\t/go/src/github.com/fusor/mig-controller/pkg/logging/logger.go:65\ngithub.com/fusor/mig-controller/pkg/controller/migmigration.(*Task).deletePodAnnotations\n\t/go/src/github.com/fusor/mig-controller/pkg/controller/migmigration/annotation.go:341\ngithub.com/fusor/mig-controller/pkg/controller/migmigration.(*Task).deleteAnnotations\n\t/go/src/github.com/fusor/mig-controller/pkg/controller/migmigration/annotation.go:297\ngithub.com/fusor/mig-controller/pkg/controller/migmigration.(*Task).Run\n\t/go/src/github.com/fusor/mig-controller/pkg/controller/migmigration/task.go:367\ngithub.com/fusor/mig-controller/pkg/controller/migmigration.(*ReconcileMigMigration).migrate\n\t/go/src/github.com/fusor/mig-controller/pkg/controller/migmigration/migrate.go:77\ngithub.com/fusor/mig-controller/pkg/controller/migmigration.(*ReconcileMigMigration).Reconcile\n\t/go/src/github.com/fusor/mig-controller/pkg/controller/migmigration/migmigration_controller.go:168\ngithub.com/fusor/mig-controller/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/go/src/github.com/fusor/mig-controller/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:215\ngithub.com/fusor/mig-controller/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func1\n\t/go/src/github.com/fusor/mig-controller/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:158\ngithub.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil.func1\n\t/go/src/github.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:133\ngithub.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil\n\t/go/src/github.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:134\ngithub.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait.Until\n\t/go/src/github.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:88"}

It's something I want to investigate but nothing is broken. The stage pod still ran successfully and restic successfully performs a podvolumerestore. I think it has to do with deleteAnnotations or the stage cleanup process.

This came from an investigation into: https://github.com/fusor/mig-controller/issues/191

Assigning this as a beta-blocker as I fear the messages will mislead folks testing, if it's possible to find the issue and fix would make a better beta experience.

@dymurray @jwmatthews Assuming the stage pod was already deleted, we don't need to touch it's annotations. The annotation change in kubernetes 1.12+ causes pod to be reconciled, and the cluster tries to validate the SCC on the annotations of the pod, which were removed at this point. I noticed that the error happens right after the log message about stage pod is removal. What is the course of action here? Skipping deleted pods might be enough?
Here is a snip from the logs:
{"level":"info","ts":1565007430.6335168,"logger":"migration|qmz6q","msg":"Pod annotations/labels added.","ns":"mysql-persistent-test-move","name":"mysql-1-cmsqs"}
{"level":"info","ts":1565007431.0338788,"logger":"migration|lw5pz","msg":"Stage pod created.","ns":"mysql-persistent-test-move","name":"mysql-1-cmsqs-stage"}
{"level":"info","ts":1565007546.1937892,"logger":"migration|wz5bl","msg":"Stage pod deleted.","ns":"mysql-persistent-test-move","name":"mysql-1-cmsqs-stage"}
{"level":"info","ts":1565007546.2349448,"logger":"migration|wz5bl","msg":"Stage pod deleted.","ns":"mysql-persistent-test-move","name":"mysql-1-cmsqs-stage"}
{"level":"info","ts":1565007546.8337853,"logger":"migration|qtbgl","msg":"Pod annotations/labels removed.","ns":"mysql-persistent-test-move","name":"mysql-1-cmsqs-stage"}
{"level":"error","ts":1565007546.8559673,"logger":"migration|qtbgl","msg":"","error":"pods \"mysql-1-cmsqs-stage\" is forbidden: unable to validate against any security context constraint: []","stacktrace":"github.com/fusor/mig-controller/vendor/github.com/go-logr/zapr.(*zapLogger).Error\n\t/go/src/github.com/fusor/mig-controller/vendor/github.com/go-logr/zapr/zapr.go:128\ngithub.com/fusor/mig-controller/pkg/logging.Logger.Error\n\t/go/src/github.com/fusor/mig-controller/pkg/logging/logger.go:59\ngithub.com/fusor/mig-controller/pkg/logging.Logger.Trace\n\t/go/src/github.com/fusor/mig-controller/pkg/logging/logger.go:65\ngithub.com/fusor/mig-controller/pkg/controller/migmigration.(*Task).deletePodAnnotations\n\t/go/src/github.com/fusor/mig-controller/pkg/controller/migmigration/annotation.go:349\ngithub.com/fusor/mig-controller/pkg/controller/migmigration.(*Task).deleteAnnotations\n\t/go/src/github.com/fusor/mig-controller/pkg/controller/migmigration/annotation.go:301\ngithub.com/fusor/mig-controller/pkg/controller/migmigration.(*Task).Run\n\t/go/src/github.com/fusor/mig-controller/pkg/controller/migmigration/task.go:367\ngithub.com/fusor/mig-controller/pkg/controller/migmigration.(*ReconcileMigMigration).migrate\n\t/go/src/github.com/fusor/mig-controller/pkg/controller/migmigration/migrate.go:77\ngithub.com/fusor/mig-controller/pkg/controller/migmigration.(*ReconcileMigMigration).Reconcile\n\t/go/src/github.com/fusor/mig-controller/pkg/controller/migmigration/migmigration_controller.go:168\ngithub.com/fusor/mig-controller/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).processNextWorkItem\n\t/go/src/github.com/fusor/mig-controller/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:215\ngithub.com/fusor/mig-controller/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller.(*Controller).Start.func1\n\t/go/src/github.com/fusor/mig-controller/vendor/sigs.k8s.io/controller-runtime/pkg/internal/controller/controller.go:158\ngithub.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil.func1\n\t/go/src/github.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:133\ngithub.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait.JitterUntil\n\t/go/src/github.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:134\ngithub.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait.Until\n\t/go/src/github.com/fusor/mig-controller/vendor/k8s.io/apimachinery/pkg/util/wait/wait.go:88"}

@Danil-Grigorev Thanks for that write-up, yeah I believe you are correct. I will look into this today and hopefully find an easy fix.

Reopening as it appears this is still a reported issue, even sometimes during EnsureStageRestore phase as I observed on @jwmatthews environment. I also hit this with the latest controller image. Will speak with Danil to get a fix.

Danil mentioned we shouldn't be deleting annotations of stage pods just to delete them, and he is correct. So the solution is easy to just delete stage pods before removing all the annotations/labels on a failure cleanup.


Any chance you can provide an example file?

Sorry, it's been a couple of years and I no longer have the hardware, so I don't want to put something untested up. If you write one, I'd be happy to add it to the repo.
My recollection was that it was fairly easy to use. Construct the ADC_ads8689 object with a buffer size and specify the CS pin.
Then transmit commands and read the results off of the buffer.
It's really just to make the code a bit more readable and reduce the need to reference the datasheet as much for the commands.
If I remember correctly, when you transmit anything other than a read command, the ADC returns the last converted value. The CS pin is shared with the one that controls conversion, so it should always have a recent reading.
Sections 7.5 and 7.6 of the datasheet give detailed descriptions of the commands and registers.

Wow thanks for responding!
This is more or less my setup. I'm not sure what I should be putting in for data on READ calls so it's 0x00. My hardware is at my work so I'll have to try again tomorrow. The values returned just don't really add up.
#include <ADC_ads8689.h>

const int CS_PIN = 10;

ADC_ads8689 adc = ADC_ads8689(32,CS_PIN);

void setup(){
  pinMode(CS_PIN, OUTPUT);
  Serial.begin(9600);

  adc.transmit(ADS8689_READ, ADS8689_RANGE_SEL_REG, 0x00);

  int results = readBuffer();
  Serial.println(results, BIN);
}

void loop(){

}
```

Hello, I'm looking for an exemple also because I can't get it to work (I don't have a lot of experience with SPI).  Here's a test I made(I also tried to clear the Buffer at each readings).
`#include <ADC_ads8689.h>
const int CS = 10;
ADC_ads8689 ain = ADC_ads8689(32, CS);
uint16_t rgconf = 0b0000000000001001;
uint32_t ea_1;
void setup() {
// put your setup code here, to run once:
Serial.begin(9600);
ain.transmit(ADS8689_WRITE_FULL, ADS8689_RANGE_SEL_REG, rgconf);
//pinMode(CS, OUTPUT);
}
void loop() {
if (ain.inputAvailable() > 0) {
ea_1 = ain.readBuffer();
Serial.print("ads8689: ");
Serial.println(ea_1);
}
delay(250);
ain.transmit(0, 0, 0);
}`

I would not mind an example of how to simply read the last conversion result.
The data sheet does not seem to show how, all the register address's are to do with setting up the chip, how do you read the data?
Thanks

@bmentink If you look at section 7.5.3 of the datasheet describing the output word, all commands that do not explicitly request the contents of a register implicitly request the last conversion result.

@Helmuthn  Thanks, does that mean that you send a dummy register reference (like 0)? and what commend ADC8689_NULL?
Cheers

My memory is a bit fuzzy at this point, but I believe just sending a NOP (all zeros) still requests the last conversion. I'd personally choose all zeros for the dummy register reference, but it does say that any unrecognized command will be interpreted as a NOP.
If you get a minimal example working, feel free to submit a pull request.
Like I mentioned at the beginning of this issue, I no longer have access to the hardware and I don't want to put something up entirely untested.

@Dressner95  , @bmentink or @pagravel  did any of you get it working?


Inbox Scripts

It would be great to ship with a base set of tools most users would find helpful. These can be new things or tools from the community that are listed on the PS Gallery.
These can be added to the Scripts XML and Categories XML as needed.

This sounds like something I can help with in my spare time. Do you have any sample list of tools that would need to be added.

This sounds like something I can help with in my spare time. Do you have any sample list of tools that would need to be added.

Totally up to you, things that would be helpful for IT Admins from Help Desk up to more engineering type roles. Ideally you'd want them to accept a -Machine parameter so that the UI can pass input without having to prompt the user again

Hi nkasco, I was able to build the project and test the application. Amazing framework, I see lot of potential and use case for this tool.
Quick question though,
How do you pass machine name parameter to PowerShell script? This is what I have tried.

Place script in script folder (right next to sample script that you have kept)
Append scripts.xml with new script info.
It does show up in the tool  and works as expected.

But  how do I pass the machine information (right top of the tool) info to PowerShell script. I logged all the args that are being sent to PowerShell script and it was empty, so its not sending that info at all?

Hi nkasco, I was able to build the project and test the application. Amazing framework, I see lot of potential and use case for this tool.
Quick question though, How do you pass machine name parameter to PowerShell script? This is what I have tried.

Place script in script folder (right next to sample script that you have kept)
Append scripts.xml with new script info.
It does show up in the tool  and works as expected.

But how do I pass the machine information (right top of the tool) info to PowerShell script. I logged all the args that are being sent to PowerShell script and it was empty, so its not sending that info at all?

scratch that, I was able to get the machine info after entered the details again.
For any future reference, its just passed as unnamed parameter.  Its comma separated single argument when its multi device entry.

Hi nkasco, I was able to build the project and test the application. Amazing framework, I see lot of potential and use case for this tool.
Quick question though, How do you pass machine name parameter to PowerShell script? This is what I have tried.

Place script in script folder (right next to sample script that you have kept)
Append scripts.xml with new script info.
It does show up in the tool  and works as expected.

But how do I pass the machine information (right top of the tool) info to PowerShell script. I logged all the args that are being sent to PowerShell script and it was empty, so its not sending that info at all?

scratch that, I was able to get the machine info after entering the details again.
For any future reference, its just passed as unnamed parameter. Its comma separated single argument when its multi device entry.

Yep exactly, it doesn't have to be complex to be functional :)


Cache Analysis Bazel Workspace File Systems

The Bazel invoker created from #81 will help us understand how our client's Bazel projects are laid out. This is great, but the Bazel invoker takes a long time to complete. We need to cache our client's file system so we don't hang for so long.
Use the Bazel invoker to construct a WorkspaceTree, which is created from in #91. Cache the workspace tree after each Bazel query to save on performance.

[x] Use the bazel invoker to interpret a client's bazel file system.
[x] Initialize the analyzer on extension start. This should cache a representation of our user's file system as a WorkspaceTree
[ ] Whenever appropriate, or possibly on didUpdate from the BazelServices, update the cached workspace stored in the analyzer. Only update what needs to be updated.

Unit tests

[x] Create unit tests to test this behavior

More context found in #82

Whenever appropriate, or possibly on didUpdate from the BazelServices, update the cached workspace stored in the analyzer. Only update what needs to be updated.
Not sure how to do this one quite yet. Still working on it.


Allow explicitely setting the adapter in mix config

This makes it possible to set a different adapter for different environments.
This commit also includes the fix proposed in #38.
Does anyone know what the best practices are for using the application environment in tests? Is there a way to sandbox them?
It's dangerous to have to clean up what I changed in the environment in every test, and I suspect it could cause issues when running tests in parallel.

Thanks!


[WIP][SPARK-40540][SQL] Migrate compilation errors onto error classes: _LEGACY_ERROR_TEMP_1200-1299

What changes were proposed in this pull request?
In the PR, I propose to migrate 100 compilation errors onto temporary error classes with the prefix _LEGACY_ERROR_TEMP_12xx. The error message will not include the error classes, so, in this way we will preserve the existing behaviour.
Why are the changes needed?
The migration on temporary error classes allows to gather statistics about errors and detect most popular error classes. After that we could prioritise the work on migration.
The new error class name prefix _LEGACY_ERROR_TEMP_ proposed here kind of marks the error as developer-facing, not user-facing. Developers can still get the error class programmatically via the SparkThrowable interface, so that they can build error infra with it. End users won't see the error class in the message. This allows us to do the error migration very quickly, and we can refine the error classes and mark them as user-facing later (naming them properly, adding tests, etc.).
Does this PR introduce any user-facing change?
No. The error messages should be almost the same by default.
How was this patch tested?
By running the modified test suites:
$ build/sbt "core/testOnly *SparkThrowableSuite"
$ build/sbt "sql/testOnly org.apache.spark.sql.SQLQueryTestSuite"
$ build/sbt "test:testOnly *SQLQuerySuite"

@itholic @srielau @cloud-fan @anchovYu Please, review this PR.

@HyukjinKwon Could you approve the PR, please.

Merging to master. Thank you, @gatorsmile.


Issues with script,

When running script i get
/conf/PIAWireguard.py debug
searchServer request failed non 200 status code - listing wireguard instances
Permissions are set correctly,
running OPNSense 20.7.4
Wireguard plugin installed,
correct API keys in place.

This appears to have been my own issue, missing a " at the start of the API key, however now i'm getting
wireguardserver addKey request failed non 200 status code - Trying to add instance public key to server in exchnage for connection information

Check your OPNsense API key and the user's permissions (screenshot below) also the user variable opnsenseURL  have you amended that to use the port that your OPNsense instance uses? Default install would be 'https://<IP_ADDRESS>:443' (you can omit the port if its 443 of course.)
By default OPNsense uses port 443 for the web interface, but I always change it to 8443, so that's why its 8443 in the script.

Thank you for your response, i managed to work all that out, but i'm now getting the above error.  I've double confirmed all the permissions and the correct API key in place.
But i cannot see the cause of the
wireguardserver addKey request failed non 200 status code - Trying to add instance public key to server in exchnage for connection information

Please ignore me, i've managed to work it out successfullly ran the script

What was the issue?
Incase others come across this issue on github

I simply removed all of the ports from the url
So just put
https://<IP_ADDRESS>
it didn't seem to work when i put https://<IP_ADDRESS>:443 i have no idea why. I then confirmed that everything else was correct and ran the script and it worked.

Glad to hear it worked, any more questions or issues just say.
I shall close this issue.


fix: update packages

:tada: This PR is included in version 1.0.0-alpha.12 :tada:
The release is available on:

npm package (@release-v1 dist-tag)
GitHub release

Your semantic-release bot :package::rocket:


Implement TokenScript NFT footer

For NFT detail page with TokenScript, any Token View should not replace the default view, it should be added at the bottom (aka Dynamic Attributes).

This has been implemented in a more organic way - the dynamic attributes are added in-line with the static token attributes.


fix(core): Switch location to additional-location

In boot2, location overrides rather than adds to the default config locations.

Caught by our integration tests; similar to what we did in other services, ex spinnaker/echo#536


  This is per the advice of The Pull Request Hack.

Thanks!


Arguments other than parameter 1

according to this documentation, there are not any other arguments other than parameter 1, however in effect they exist. i think you should take a look to Geoff Chappell article about this bugcheck https://www.geoffchappell.com/studies/windows/km/bugchecks/6b.htm and update it because it would be important for many users to be properly documented about this bugcheck, and change redaction style, since there is not a "missing file" from the boot partition, is exclusively ntdll.dll who causes this bugcheck, when this file is missing (i've tested on VM and causes this bugcheck with 2 as its 2nd parameter) or when it cannot open some of its exports due to other causes. Please update the article, thanks.

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

ID: 76528be9-25fa-14da-0f12-98e650278efa
Version Independent ID: 22ae5b42-6beb-81b5-7652-1085a3bb92e6
Content: Bug Check 0x6B PROCESS1_INITIALIZATION_FAILED - Windows drivers
Content Source: windows-driver-docs-pr/debugger/bug-check-0x6b--process1-initialization-failed.md
Product: windows-hardware
Technology: windows-devices
GitHub Login: @DOMARS
Microsoft Alias: domars

@RubberWarrior1999 -  Thanks for the input that we should add additional information on Bug Check 0x6B. We use customer input like this as well as data on how often the bug checks occur to determine how we allocate our documentation team resources. In general in current day Windows, the 1st parameter is the most common, and will be the most actionable.  We will use your input and other data to determine if we can provide additional information on this stop code that is broadly applicable. Again, thanks for the input, and good luck with your stop code troubleshooting.

@RubberWarrior1999 -  Thanks for the input that we should add additional information on Bug Check 0x6B. We use customer input like this as well as data on how often the bug checks occur to determine how we allocate our documentation team resources. In general in current day Windows, the 1st parameter is the most common, and will be the most actionable.  We will use your input and other data to determine if we can provide additional information on this stop code that is broadly applicable. Again, thanks for the input, and good luck with your stop code troubleshooting.


Simulations and CLI tests implementation

Description
This PR implements simulation tests for both the posts and the magpie modules.
Closes #89.
TODO

[x] Add tx posts create CLI tests
[x] Add tx posts edit CLI tests
[x] Add tx posts add-reaction CLI tests
[x] Add tx posts remove-reaction CLI tests
[x] Add query post CLI tests
[x] Add query posts CLI tests
[x] Add tx posts create CLI tests w/ medias
[x] Add tx posts create CLI tests w/ poll
[x] Add tx posts answer-poll CLI tests

Checklist

[x] Targeted PR against correct branch.
[x] Linked to Github issue with discussion and accepted design OR link to spec that describes this work.
[x] Wrote tests.
[ ] Added an entry to the CHANGELOG.md file.
[x] Re-reviewed Files changed in the Github PR explorer.

I think this PR is ready to be reviewed. The current coverage is up to 81% (+6% over master): https://codecov.io/gh/desmos-labs/desmos/branch/riccardo%2Fsimulations-tests

@bragaz Thanks for the check! I've fixed the minor errors and answered your comment. Should all be good to go. Waiting for @kwunyeung now! :rocket:


[minor] formatting fixes

spotted while browsing through the docs.
some evidence:
1

2

Thanks @sappelhoff


Allow stacks to expand independently by axis

Hi there!
I was checking out this library and it's awesome! Thank you so much for making it 😄
There are a couple things that I wished I could do but couldn't. This one in particular seemed straightforward enough to implement, so here's a PR!

This adds support for passing vexpand/hexpand to *Stacks and *Grids to control the 'expansion' on each axis individually. The main use case for me was having HStacks in a VLayout that expand horizontally but not vertically so their height is always the same.
The existing behavior with expand/height/width remains the same, so this change is backwards compatible. But if expand=True, then the new parameters can be used.
Let me know if there's anything I missed 😊

Thanks! Glad you found this useful. I didn't know anyone else was actually using this code so I'm glad it was useful! This is a great idea. I wonder if it would make sense to just replace expand with vexpand and hexpand (with both True by default)? This might make the case of someone accidentally using expand=True, vexpand=False, hexpand=False less confusing. On the other hand, it would make the default case for most containers except grid different as you'd had to remember to use vexpand or hexpand as appropriate. I think your implementation is probably good, just thinking out loud.

@all-contributors please add @kevinrpb for code

I didn't know anyone else was actually using this code so I'm glad it was useful!

@RhetTbull Yeah! Honestly I just found out the package a couple days ago, but it is super useful so far for some small utilities I'm working on!

I wonder if it would make sense to just replace expand with vexpand and hexpand.

Ah, that's a good thought. I'm honestly not sure. Since you are way more familiar with the package I think it makes sense for you to take that decision, both alternatives seem sensible to me 🙂
Just let me know and I'll update the PR as needed 👌🏽

@kevinrpb after thinking about this some more I think it makes most sense to just remove the expand argument. With both vexpand and hexpand as default True, the effect is the same in the default case and your addition allows the expansion to be controlled independently in each direction.  Would you update the PR to remove the expand argument?

Yeah, I can update the PR. Will do that tomorrow 👍🏽

@RhetTbull PR should be good now.
With the change I think the logic in _Stack:_create_widget could be simplified. Please, take a look to verify it if you have the change 🙂.
(PS: Also, feel free to squash the commits when merging. I pushed it into two different ones so I could install the package locally and test the last change.)

Actually, I just realized I need to update some of the example/test files to remove uses of expand. Will do that and get back to you.

@RhetTbull updated it 👍🏽

Thanks! I've published the new release.


Updates for compatibility with async-http

These changes make async-http-faraday compatible with async-http v0.36.2, which is required by falcon. I'm not sure they are the best fixes, but it works for me locally.

Pull Request Test Coverage Report for Build 4

0 of 3 (0.0%)  changed or added relevant lines in 1 file are covered.
6 unchanged lines in 1 file lost coverage.
Overall coverage increased (+9.09%) to 100.0%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

lib/async/http/faraday/adapter.rb
0
3
0.0%

Files with Coverage Reduction
New Missed Lines
%

lib/async/http/faraday/adapter.rb
6
45.83%

Totals

Change from base Build 2:
9.09%

Covered Lines:
13

Relevant Lines:
13

💛  - Coveralls

Great work! Thanks so much.

Released v0.2.0 :)


How to setup default time in Big Calendar

Want initial time to 9 AM when the page first renders, but later if user scroll to say 4 PM and when user changes the date, prevent scrolling again to 9 AM.
code for big calendar reside inside render function
is there any way to set up default time like default date

try scrollToTime: http://intljusticemission.github.io/react-big-calendar/examples/index.html#prop-scrollToTime
I think that's the only thing that might com close. thanks!

@jquense Hi,
I have implemented using scrollToTime, but as I change date, it scroll back up to 9AM

Code is
       let initTime = calendarDate.clone().set({hour: 9, minute: 0, second: 0});       <BigCalendar           events={this.getBookings()}           defaultDate={calendarDate.toDate()}           date={calendarDate.toDate()}           onNavigate={() => {}}           view={this.props.calendarType || CalendarTypes.DAY}           onView={()=> {}}           toolbar={false}           selectable={true}           popup={true}           scrollToTime={initTime}           formats={{dateFormat: 'D', weekdayFormat: 'dddd'}}         />

yeah it looks like it applies the scrollToTime when you navigate as well...that seems wrong, maybe a bug

@jquense
I don't think it's a bug, coz I am not using toolbar provided by component and hence onNavigate method is not firing, I am passing date from outer component and which re-renders Big Calendar on date change
is there way to prevent this behaviour

no, the scrollToTime code specifically runs again when the view navigates back and forward. Your custom toolbar doesn't affect that here, the same behavior you are seeing is present with the included toolbar


Enhance Sign Up Component with Improved UI and Functionality

Title
Enhance Sign Up Component with Improved UI and Functionality.
Enhancement Aim
The aim of this enhancement is to improve the user interface (UI) of the Sign Up component, making it more user-friendly and visually appealing. Additionally, I have implemented better validation mechanisms for the input fields to enhance user experience and prevent errors.
Changes

UI Improvements:

Updated styling for input fields, buttons, and overall layout for a cleaner look.
Added visual feedback for password strength and validation errors.
Enhanced the layout to make it more responsive and accessible.

Functionality Enhancements:

Implemented password strength validation to provide real-time feedback to users.
Improved error handling to inform users of required fields and incorrect input formats.
Added functionality to remember user preferences regarding marketing updates.

Screenshots 📷
N/A
Full Name
Debarati Seal
Participant Role
GSSOC Contributor
Guidelines

[X] I have read the guidelines
[ ] I have the link to my latest merged PR

merge #345 and #344 into a single issue


Make sure colon in the https:// scheme is not replaced

fixes #16

Unfortunately your solution doesn't seem to work on my system with neither GNU nor BSD sed (I expect that the : should be turned to /):
bash-3.2$ echo git@github.com:ruanyl/vim-gh-line.git | gsed 's/(https)\@<!:/\//'
git@github.com:ruanyl/vim-gh-line.git
bash-3.2$ echo git@github.com:ruanyl/vim-gh-line.git | sed 's/(https)\@<!:/\//'
git@github.com:ruanyl/vim-gh-line.git

bash-3.2$ echo git@github.com:ruanyl/vim-gh-line.git | gsed 's/\(https\)\@<!:/\//'
git@github.com:ruanyl/vim-gh-line.git
bash-3.2$ echo git@github.com:ruanyl/vim-gh-line.git | sed 's/\(https\)\@<!:/\//'
git@github.com:ruanyl/vim-gh-line.git

Maybe I'm doing the shell/sed escapes wrong? Could you give any reference to the syntax of the negative lookbehind expression for sed that you're using? I couldn't find any.
However, it's possible to avoid lookbehinds completely for this problem by combining 1st and 2nd substitutions:
bash-3.2$ echo git@github.com:ruanyl/vim-gh-line.git | gsed 's/^[^@:]*@\([^:]*\):/https:\/\/\1\//'
https://github.com/ruanyl/vim-gh-line.git
bash-3.2$ echo git@github.com:ruanyl/vim-gh-line.git | sed 's/^[^@:]*@\([^:]*\):/https:\/\/\1\//'
https://github.com/ruanyl/vim-gh-line.git

bash-3.2$ echo https://github.com/ruanyl/vim-gh-line.git | sed 's/^[^@:]*@\([^:]*\):/https:\/\/\1\//'
https://github.com/ruanyl/vim-gh-line.git
bash-3.2$ echo https://github.com/ruanyl/vim-gh-line.git | gsed 's/^[^@:]*@\([^:]*\):/https:\/\/\1\//'
https://github.com/ruanyl/vim-gh-line.git

PS: It would be nice to set up regression tests and CI for this project (obviously not in this PR, but in general).

Thank you @KostyaEsmukov  for your input.
I have used the regular expressions you suggested. The need for forward-backward slash escaping is rather unfortunate but I think this should work. It did in my setup.

@ruanyl could you please review this PR ? It is a bugfix and it would be good to merge it sooner than later.

thank you very much for the pr, works good with Github/Bitbucket https and ssh remote url.


Numbers and symbols not shown correctly in a certain document

Attach (recommended) or Link to PDF file here:
1Password for Teams White Paper.pdf (original)
Configuration:

Web browser and its version: Firefox 65.0 (64-bit)
Operating system and its version: Windows 10 Pro Version 1809 Build 17763.253
PDF.js version: 2.1.97
Is a browser extension: No

Steps to reproduce the problem:

Go to page 10 of the PDF
Look after the sentence "A Master Password with four such words will be one of" (this is one of several number and symbols that have the problem in this document).

What is the expected behavior? (add screenshot)
The number "180004" is shown correctly.
Using SumatraPDF v3.1.2:

What went wrong? (add screenshot)
The number is unreadable.

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):
Console log:
PDF f22bb023e2e92110a02d2bd381ef5d99 [1.5 XeTeX 0.99999 / LaTeX with hyperref] (PDF.js: 2.1.97) viewer.js:1179:7
Warning: Failed to load font "g_d0_f8": SyntaxError: An invalid or illegal string was specified pdf.js:525:5

Adobe Acrobat Reader DC 2019.010.20069 shows numbers and symbols correctly but displays a warning message. The message pops up after interacting with the document (a few text selections or going to a page number). I'm not exactly sure what triggers it. The message is:

"Cannot extract the embedded font 'VENLVK+FiraMath-Regular-Identity-H'. Some characters may no display or print correctly."

Duplicate of https://bugzilla.mozilla.org/show_bug.cgi?id=1524888 (there's no need to track it twice).


Keyboard theme

Hey all,
I created a configuration for the online Keyboard Layout Editor which you can load to create Dracula themed keycaps.

The configuration to load is available here
I'm not sure where else to mention this, so hope it's alright creating an issue here. I hope you approve :smile:

I think I will replicate the layout on my Chroma, see what it looks like 😸

Hey @karlbateman, that's great!
I'll go ahead and great a repository to host that configuration file. I'll also invite you to participate on the Dracula organization, so you can make any changes you want to that theme.
What do you think?

@zenorocha That's great, thank you.

Hey @karlbateman, I still didn't have a chance to create the repository. Would you mind create one based on the template we created? I can move to the organization right after.

Awesome ;D


Exception error please help!!

Sync.1D39B1D2F24E3A2.crash.zip

Install .NET Framework 4.5
https://www.microsoft.com/en-us/download/details.aspx?id=42642

thank you!

it's nothing


[CFE] Pattern temporary variables are incorrectly typed

Part of https://github.com/dart-lang/sdk/issues/49749.
Example:
String test(List list) {
  return switch (list) {
    <num>[3, >0] => "relational",
    [4, var c as num] => "cast",
    _ => "default",
  };
}

main() {
  test([4, "42"]);
}

Kernel AST before VM-specific transformations (before Target.performModularTransformationsOnLibraries):
  static method test(core::List<dynamic> list) → core::String {
    return block {
      core::String #t1;
      core::int #t2 = -1;
      core::num c;
      final core::List<dynamic> #0#0 = list;
      late final core::bool #0#3 = #0#0{core::List<core::num>}.{core::List::length}{core::int} =={core::num::==}{(core::Object) → core::bool} #C1;
      late final core::num #0#6 = #0#0{core::List<core::num>}.{core::List::[]}(0){(core::int) → core::num};
      late final core::num #0#9 = #0#0{core::List<core::num>}.{core::List::[]}(1){(core::int) → core::num};
      if(#0#0 is{ForNonNullableByDefault} core::List<core::num> && #0#3 && #C2 =={core::num::==}{(core::Object) → core::bool} #0#6 && (let final dynamic #t3 = #0#9 in #0#9.{core::num::>}(#C3){(core::num) → core::bool})) {
        #t2 = 0;
      }
      else
        if(#0#3 && #C4 =={core::num::==}{(core::Object) → core::bool} #0#6 && (let final dynamic #t4 = #0#9 as{ForNonNullableByDefault} core::num in let final dynamic #t5 = c = #0#9 as{ForNonNullableByDefault} core::num in true)) {
          #t2 = 1;
        }
        else
...

The problem is that for the pattern [4, var c as num] we check that length of the list is 2 (in #0#3), and then we access list elements via late variables #0#6 and #0#9 which have type num, but the original list is List<dynamic> and it can have non-num elements. As a result, values of #0#6 and #0#9 may violate their static type. Also, cast #0#9 as{ForNonNullableByDefault} core::num is eliminated as redundant because #0#9 has type num already.
Test co19/LanguageFeatures/Patterns/list_A01_t01 currently fails with RuntimeError due to this bug.
@chloestefantsova @johnniwinther

The test co19/LanguageFeatures/Patterns/list_A01_t02 also fails because of this bug.

I'm working on this.


No module named 'azure.keyvault.v7_0' after CLI Installation

Describe the bug
After installing the Azure CLI on Debian 11 using either the "one step" install or the multi-step install az keyvault commands fail with a missing Python library error.
For example, the command:
az keyvault secret show --vault-name somer-vault -n SOME-KEY

fails with the following traceback:
CLIInternalError: The command failed with an unexpected error. Here is the traceback:
No module named 'azure.keyvault.v7_0'
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/lib/python3/dist-packages/azure/cli/core/commands/__init__.py", line 558, in execute
    self.commands_loader.load_arguments(command)
  File "/usr/lib/python3/dist-packages/azure/cli/core/__init__.py", line 480, in load_arguments
    self.command_table[command].load_arguments()  # this loads the arguments via reflection
  File "/usr/lib/python3/dist-packages/azure/cli/core/commands/__init__.py", line 315, in load_arguments
    super(AzCliCommand, self).load_arguments()
  File "/usr/lib/python3/dist-packages/knack/commands.py", line 106, in load_arguments
    cmd_args = self.arguments_loader()
  File "/usr/lib/python3/dist-packages/azure/cli/command_modules/keyvault/_command_type.py", line 75, in keyvault_arguments_loader
    op = get_op_handler()
  File "/usr/lib/python3/dist-packages/azure/cli/command_modules/keyvault/_command_type.py", line 72, in get_op_handler
    return self.command_loader.get_op_handler(operations_tmpl.format(method_name))
  File "/usr/lib/python3/dist-packages/azure/cli/core/__init__.py", line 829, in get_op_handler
    op = import_module(mod_to_import)
  File "/usr/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 972, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 984, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'azure.keyvault.v7_0'

pip reports that the keyvault packages are installed:
root@5d506032b00b:/# pip3 list | grep keyvault
azure-keyvault                                   4.1.0
azure-keyvault-administration                    4.0.0b3
azure-keyvault-certificates                      4.2.2
azure-keyvault-keys                              4.3.2
azure-keyvault-secrets                           4.2.1
azure-mgmt-keyvault                              8.0.0

Downgrading to azure-keyvault==1.1.0 resolves the issue. This seems like the version requirements of the CLI are incorrect.
I also tried upgrading the CLI which failed.
Related command
az keyvault secret show --vault-name somer-vault -n SOME-KEY
Errors
CLIInternalError: The command failed with an unexpected error. Here is the traceback:
No module named 'azure.keyvault.v7_0'
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/lib/python3/dist-packages/azure/cli/core/commands/__init__.py", line 558, in execute
    self.commands_loader.load_arguments(command)
  File "/usr/lib/python3/dist-packages/azure/cli/core/__init__.py", line 480, in load_arguments
    self.command_table[command].load_arguments()  # this loads the arguments via reflection
  File "/usr/lib/python3/dist-packages/azure/cli/core/commands/__init__.py", line 315, in load_arguments
    super(AzCliCommand, self).load_arguments()
  File "/usr/lib/python3/dist-packages/knack/commands.py", line 106, in load_arguments
    cmd_args = self.arguments_loader()
  File "/usr/lib/python3/dist-packages/azure/cli/command_modules/keyvault/_command_type.py", line 75, in keyvault_arguments_loader
    op = get_op_handler()
  File "/usr/lib/python3/dist-packages/azure/cli/command_modules/keyvault/_command_type.py", line 72, in get_op_handler
    return self.command_loader.get_op_handler(operations_tmpl.format(method_name))
  File "/usr/lib/python3/dist-packages/azure/cli/core/__init__.py", line 829, in get_op_handler
    op = import_module(mod_to_import)
  File "/usr/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 972, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 984, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'azure.keyvault.v7_0'

Issue script & Debug output
CLIInternalError: The command failed with an unexpected error. Here is the traceback:
No module named 'azure.keyvault.v7_0'
Traceback (most recent call last):
  File "/usr/lib/python3/dist-packages/knack/cli.py", line 233, in invoke
    cmd_result = self.invocation.execute(args)
  File "/usr/lib/python3/dist-packages/azure/cli/core/commands/__init__.py", line 558, in execute
    self.commands_loader.load_arguments(command)
  File "/usr/lib/python3/dist-packages/azure/cli/core/__init__.py", line 480, in load_arguments
    self.command_table[command].load_arguments()  # this loads the arguments via reflection
  File "/usr/lib/python3/dist-packages/azure/cli/core/commands/__init__.py", line 315, in load_arguments
    super(AzCliCommand, self).load_arguments()
  File "/usr/lib/python3/dist-packages/knack/commands.py", line 106, in load_arguments
    cmd_args = self.arguments_loader()
  File "/usr/lib/python3/dist-packages/azure/cli/command_modules/keyvault/_command_type.py", line 75, in keyvault_arguments_loader
    op = get_op_handler()
  File "/usr/lib/python3/dist-packages/azure/cli/command_modules/keyvault/_command_type.py", line 72, in get_op_handler
    return self.command_loader.get_op_handler(operations_tmpl.format(method_name))
  File "/usr/lib/python3/dist-packages/azure/cli/core/__init__.py", line 829, in get_op_handler
    op = import_module(mod_to_import)
  File "/usr/lib/python3.9/importlib/__init__.py", line 127, in import_module
    return _bootstrap._gcd_import(name[level:], package, level)
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 972, in _find_and_load_unlocked
  File "<frozen importlib._bootstrap>", line 228, in _call_with_frames_removed
  File "<frozen importlib._bootstrap>", line 1030, in _gcd_import
  File "<frozen importlib._bootstrap>", line 1007, in _find_and_load
  File "<frozen importlib._bootstrap>", line 984, in _find_and_load_unlocked
ModuleNotFoundError: No module named 'azure.keyvault.v7_0'

Expected behavior

I would expect that azure-keyvault==1.1.0 is pinned in the requirements of the CLI so that az keyvault commands succeed.

Environment Summary
root@5d506032b00b:/# az --version
azure-cli                         2.18.0 *

core                              2.18.0 *
telemetry                          1.0.6

Extensions:
azure-devops                      0.19.0

Python location '/usr/bin/python3'
Extensions directory '/root/.azure/cliextensions'
Extensions system directory '/usr/lib/python3/dist-packages/azure-cli-extensions'

Python (Linux) 3.9.2 (default, Feb 28 2021, 17:03:44)
[GCC 10.2.1 20210110]

Legal docs and information: aka.ms/AzureCliLegal

You have 2 updates available. Consider updating your CLI installation with 'az upgrade'

Please let us know how we are doing: https://aka.ms/azureclihats
and let us know if you're interested in trying out our newest features: https://aka.ms/CLIUXstudy

Additional context
No response

Thank you for opening this issue, we will look into it.

FYI this is not actually closed/completed since the fix hasn't been made in the "one liner" installation or in the multi-step installation steps.

Same problem on my side when using the official installation documentation for OpenSuse. I am using OpenSuse Tumbleweed.

@attrobit If the azure-cli package is provided by OpenSuse community, you should report this issue to them.


docs(table-container): add documentation for the TableContainer component

Closes #447
📝 Description
Provide missing documentation for the TableContainer component.
📝 Additional Information
Waiting on response from Issue #160 on how to apply the TableContainer component for both this page and other pages throughout the Doc Site.
Need to wait for PR #452 to be merged before updating search-meta.json Unless PR #452 is merged, will manually update get-search-meta.ts to generate the search meta properly.

@TylerAPfledderer we merged #452. Is this ready for review now?

@TylerAPfledderer we merged #452. Is this ready for review now?

@nikolovlazar Was waiting on a response from you in this issue before I went further. 😄


Backend returns current balance

Fixes https://github.com/trustlines-protocol/merkle-drop/issues/36
Notes:

Rounding mode of decay division (l.70) set to up, couldn't find in solidity doc what is done in the current version :-/
Not sure if a documentation is needed for starting the server or the config params!?

Codecov Report

Merging #38 into master will decrease coverage by 11.22%.
The diff coverage is 0%.

@@            Coverage Diff             @@
##           master     #38       +/-   ##
==========================================
- Coverage   89.62%   78.4%   -11.23%
==========================================
  Files           6       6
  Lines         241     250        +9
==========================================
- Hits          216     196       -20
- Misses         25      54       +29

Impacted Files
Coverage Δ

merkle_drop/cli.py
93.42% <ø> (+1.75%)
:arrow_up:

merkle_drop/server.py
0% <0%> (-50%)
:arrow_down:

Continue to review full report at Codecov.

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