Issue with opening Chinese document properly using pdf.js

Attach (recommended) or Link to PDF file here:
China EE_Handbook_China_JustSomeIndexContent.pdf
Configuration:

Web browser and its version:
Operating system and its version:
PDF.js version:
Is a browser extension:

Steps to reproduce the problem:

Try embedding the pdf.js and start using it for PDF viewing in the application.And try viewing the attached Chinese document, you will see its not getting opened
You will see the following issues while inspecting in Google chrome using the pdf.js chrome extension :

Access to XMLHttpRequest at 'file:///Users/sandeep.suthari/SandeepSuthari/Softwares/pdfjs-2.5.207-es5-dist/web/cmaps/Adobe-GB1-UCS2.bcmap' from origin 'null' has been blocked by CORS policy: Cross origin requests are only supported for protocol schemes: http, data, chrome, chrome-extension, chrome-untrusted, https.
util.js:336 Warning: Error during font loading: Unable to load binary CMap at: ../web/cmaps/Adobe-GB1-UCS2.bcmap
What is the expected behavior? (add screenshot)
The PDF attachment should be rendered properly
What went wrong? (add screenshot)

Link to a viewer (if hosted on a site other than mozilla.github.io/pdf.js or as Firefox/Chrome extension):

Access to XMLHttpRequest at 'file:///Users/sandeep.suthari/SandeepSuthari/Softwares/pdfjs-2.5.207-es5-dist/web/cmaps/Adobe-GB1-UCS2.bcmap' from origin 'null' has been blocked by CORS policy: Cross origin requests are only supported for protocol schemes: http, data, chrome, chrome-extension, chrome-untrusted, https.
util.js:336 Warning: Error during font loading: Unable to load binary CMap at: ../web/cmaps/Adobe-GB1-UCS2.bcmap

The error message clearly points to the browser being responsible for the blocking, and not the PDF.js library, hence this isn't an actionable/valid issue here; see also https://github.com/mozilla/pdf.js/wiki/Frequently-Asked-Questions#faq-xhr
Furthermore, as mentioned in the README, many browsers enforce special restrictions for file:// URLs which is why you should be using a server; see https://github.com/mozilla/pdf.js/#getting-the-code

Hi Snuffleupagus, the error details are from the google chrome inspect option, with the embedded Pdf.js also in our product
we are facing the same issue where the PDF is not being rendered

In our application, following are the error details :
**PDF e74d03f695f8434bb9dc85375982f3c3 [1.5 - / -] (PDF.js: 1.5.188)
Warning: Error during font loading: Cannot read property 'push' of undefined.  (pdf.js:230) **

with the embedded Pdf.js also in our product
we are facing the same issue where the PDF is not being rendered

That doesn't change the fact that this is, as already mentioned, a browser limitation; hence you should actually look into the CORS issues (however that's not something that anyone here can help you fix).

PDF e74d03f695f8434bb9dc85375982f3c3 [1.5 - / -] (PDF.js: 1.5.188)_

That version is several years out-of-date, and is consequently not supported; please find the current releases at https://github.com/mozilla/pdf.js/releases

Warning: Error during font loading: Cannot read property 'push' of undefined_. (pdf.js:230)

That seems totally unrelated to the rest of the issue, and anyway you're using an unsupported version.

Furthermore, when opening an issue you should always provide all of the information requested in https://github.com/mozilla/pdf.js/blob/master/.github/ISSUE_TEMPLATE.md; as-is this issue contains no actionable information.
Also, note https://github.com/mozilla/pdf.js/blob/master/.github/CONTRIBUTING.md (emphasis mine):

If you are developing a custom solution, first check the examples at https://github.com/mozilla/pdf.js#learning and search existing issues. If this does not help, please prepare a short well-documented example that demonstrates the problem and make it accessible online on your website, JS Bin, GitHub, etc. before opening a new issue or contacting us in the Matrix room -- keep in mind that just code snippets won't help us troubleshoot the problem.

Finally, please see http://mozilla.github.io/pdf.js/getting_started/#introduction (emphasis mine):

The viewer is built on the display layer and is the UI for PDF viewer in Firefox and the other browser extensions within the project. It can be a good starting point for building your own viewer. However, we do ask if you plan to embed the viewer in your own site, that it not just be an unmodified version. Please re-skin it or build upon it.


Close caption not working

Reserves staff report that cc is not working for:

http://library.ucsd.edu/dmr/media/1821

Let's check to make sure this isn't a general issue with all videos.
Question: does the cc button only show up once there is background noise or should it appear immediately?

@gamontoya The CC is working fine for other videos - for example - http://library.ucsd.edu/dmr/media/427
CC button show up immediately for safari.  However for Firefox, it show up later.

@VivianChu Okay, but what about the one I reference above?

I see the cc button in Safari for /1821 but no captioning. I wonder if it is the video and not the player?

@gamontoya  CC for the one you reference above is not working.  CC for this matrix video is also working http://library.ucsd.edu/dmr/media/872  .  Do you know how the CC is created for those videos? Do they use the same tool?  Thanks

@gamontoya  http://library.ucsd.edu/dmr/media/1537  CC for that video is working automatically.  Don't need to click on CC button.

Okay, so we are suspecting the same thing. /1821 has faulty cc creation. I'll ask the creator.

@gamontoya Should we leave this ticket open until the creator re-create the video with CC?

@VivianChu We'll leave it open since I'm not sure it's the video yet. Stay tuned.

Closing this ticket for now.


Bodyless PUT

Current webrick is stalling when receiving a PUT request with no body in it.

[ ] check current webrick version
[ ] if latest, create pull request to webrick
[ ] If non latest, upgrade

Related ruby/webrick#67

Happens in 1.7.0
Works on master

Confirmed that works nicely using last version of the branch in the repository.
But it needs to install git to be able to use the gem. Another issue will take care of a multi step build for it.
In this one, must also need a way to update Gemfile.lock, maybe via Makefile ?


"unknown denomination" errors on multiple inscriptions

hey guys.. do you happen to know what this error is.. i ran into this issue when trying to send some inscriptions.  here is one of them.
ord wallet send --fee-rate 3 bc1pcsmusfcyv8al7zycncespvzf26qfqwtjhpg4xupjcs8t66y89a0qrwtfxr 0cd4554ef970e765b3278e3929d2e42258479b514d39bdbd87e6807a6f5f7c0i0
error: Invalid value "0cd4554ef970e765b3278e3929d2e42258479b514d39bdbd87e6807a6f5f7c0i0" for '': unknown denomination: cd4554ef970e765b3278e3929d2e42258479b514d39bdbd87e6807a6f5f7c0i0
thanks!

It doesn't seem like 0cd4554ef970e765b3278e3929d2e42258479b514d39bdbd87e6807a6f5f7c0i0 is a valid inscription id (starting with a 0). So it is interpreting the leading "0" as an amount of btc/sats to send and the remainder of the hash as the unit of currency, as you usually specify 1000sats or 0.0001btc etc. Are you sure that is the correct inscription id?

Where the F?
0cFd4554ef970e765b3278e3929d2e42258479b514d39bdbd87e6807a6f5f7c0i0
Put the missing f after the 0c at the start and it will work.

It doesn't seem like 0cd4554ef970e765b3278e3929d2e42258479b514d39bdbd87e6807a6f5f7c0i0 is a valid inscription id (starting with a 0).

Starting with 0 is fine. It's because it is less than 66 characters long that it is disqualified from being parsed as an inscription id.


Decide how to deal with grants

There is some native support for access control lists in Nextcloud:
https://nextcloud.com/permissions/
https://nextcloud.com/wp-content/themes/next/assets/img/features/ACLs_in_groupfolders_sidebar.png?x53054
But not enough to support Reva's full permissions system.
I don't know how much support for e.g. read-only sharing exists in OCM (I should actually test that in the OCM test suite).
For now I think we may want to leave permissions&grants out of scope entirely for our first version and then see where we go from there

And same for references and arbitrary metadata, probably.


Add Switch and Hr components

Add Switch and Hr components

This update adds two new components to Blue: Switch and Hr.
The Switch component is an alternative component to Checkbox, and provides
a switch-toggle UI.
The Hr component is an enhanced verson of the HTML default hr selector.
The styling for both components are provided by Seed (seed-switch and
seed-hr). These packages were installed as devDependencies, which
cause and update to package-lock.json.
Stories, Tests, and README have been added for both components.
These components are presentional, and don't affect any other components.
cc'ing @ryan-mulrooney 😘
Resolves: https://github.com/helpscout/blue/issues/93

Coverage remained the same at 100.0% when pulling 6db73a3035530328a0f4fa90bfc200a92d50b34f on switch-component into fd4aed80a342c3cd4f86d334637745f7e5e9ee8e on master.


Sink target point inside the ground

@gberaudo looks good but if I am not mistaken, Nils should test this PR and say should we merge or not

I asked them to test.

discussed today with swisstopo, we will not do that for the moment.


adding speakers to events pages without creating a profile

Right now if you add a speaker to an event, you must create a profile in order for the speaker's name to appear on the website. We should be able to create a user record within the event without having to create a profile (esp. for guest speakers). If no profile is filled in, there should be a fallback logic to user name, rather than not appearing at all.

Tested by creating new event; problem resolved.


module java.base does not "opens java.lang.reflect" to module com.jfoenix

Hello everyone,
Before start describing how I'm getting this error, here is some important information:

It is essential the usage of the module-info.java in my project since jpackage won't work without using it.
I'm using SDK 14.0.2 (this is the minimum version that enables the usage of jpackage)
Every comment will be appreciated; although, if you're going to comment something related to using a specific VM argument, I ask you to press ctrl+F to check if I'm already using the argument you're going to suggest -since there's a bunch of VM arguments in my build.gradle

Ok, let's take a look about my issue:
First, focus on the VM argument below:

"--add-opens=java.base/java.lang.reflect=com.jfoenix",

If I don't use this argument, the following error pops up when the program runs:

java.lang.reflect.InaccessibleObjectException: Unable to make boolean java.lang.reflect.AccessibleObject.setAccessible0(boolean) accessible: module java.base does not "opens java.lang.reflect" to module com.jfoenix

IMPORTANT -> This is how my view shows without using the mentioned VM argument  (let's call it image 1):https://snipboard.io/QJ5Fdc.jpg
"Ok, so why don't you just use the VM argument?"
Great question! Alright, let's add it to my VM arguments and run the program once again.
After doing so, this is how my view looks like right now (let's call it image 2): https://snipboard.io/fbhGxw.jpg
Great! This is exactly how my view should be (note that considering it worked as expected, I've got no errors this time).
So, with everything working, I can finally move on and run my jpackage gradle task. After doing so, things stop making sense, since my view looks like the "image 1" view, regardless the fact that my project is working properly when I run it with the "run" gradle task.
### Any thoughts on why is this happening? (My guess is that my module-info.java is the key to solve it, since every time I remove an "opens" statement, e.g.: "opens my.package.name to javafx.fxml", the program gets me almost the equal error).
Let me know whether any code samples will be needed. All assistance will be appreciated. Thanks!
related stack overflow question: https://stackoverflow.com/questions/67302959/module-java-base-does-not-opens-java-lang-reflect-to-module-com-jfoenix

def args = [
        "--add-exports=javafx.graphics/com.sun.javafx.scene=ALL-UNNAMED",
        "--add-exports=javafx.controls/com.sun.javafx.scene.control.behavior=ALL-UNNAMED",
        "--add-exports=javafx.controls/com.sun.javafx.scene.control=ALL-UNNAMED",
        "--illegal-access=warn",
        "--add-opens=javafx.controls/javafx.scene.control.skin=com.jfoenix",
        "--add-exports=javafx.controls/com.sun.javafx.scene.control=com.jfoenix",
        "--add-exports=javafx.base/com.sun.javafx.binding=com.jfoenix",
        //the next line changes everything.
        **"--add-opens=java.base/java.lang.reflect=com.jfoenix",**

        "--add-exports=javafx.controls/javafx.scene.control.skin=com.jfoenix",
        "--add-exports=java.base/java.lang.reflect=ALL-UNNAMED",
        "--add-exports=java.base/java.lang.reflect=com.jfoenix",
        "--add-exports=javafx.controls/com.Sun.javafx.scene.control.behavior=com.jfoenix",
        "--add-exports=javafx.graphics/com.sun.javafx.stage=com.jfoenix",
        "--add-exports=javafx.controls/com.sun.javafx.scene.control.behavior=com.jfoenix",

        "--add-exports=javafx.base/com.sun.javafx.event=ALL-UNNAMED",

        "--add-exports=javafx.graphics/com.sun.javafx.scene=ALL-UNNAMED",
        "--add-exports=javafx.controls/com.sun.javafx.scene.control.behavior" +
                "=ALL-UNNAMED",
        "--add-exports=javafx.controls/com.sun.javafx.scene.control=ALL-UNNAMED",
        "--add-exports=javafx.controls/com.sun.javafx.scene.control.behavior" +
                "=com.jfoenix",
        "--add-exports=javafx.controls/com.sun.javafx.binding=com.jfoenix",
        "--illegal-access=warn",
        "--add-opens=javafx.controls/javafx.scene.control.skin=com.jfoenix",
        "--add-opens=java.base/java.lang.reflect=ALL-UNNAMED",
        "--add-exports=javafx.controls/com.sun.javafx.scene.control.behavior" +
                "=com.jfoenix",
        "--add-exports=javafx.controls/com.sun.javafx.scene.control=com.jfoenix",
        "--add-exports=javafx.base/com.sun.javafx.binding=com.jfoenix",
        "--add-exports=javafx.graphics/com.sun.javafx.stage=com.jfoenix",
        "--add-exports=javafx.base/com.sun.javafx.event=com.jfoenix",

        "--add-exports=javafx.controls/com.sun.javafx.scene.control.behavior=com.jfoenix",
        "--add-exports=javafx.controls/com.sun.javafx.scene.control=com.jfoenix",
        "--add-exports=javafx.base/com.sun.javafx.binding=com.jfoenix",
        "--add-exports=javafx.graphics/com.sun.javafx.stage=com.jfoenix",
        "--add-exports=javafx.base/com.sun.javafx.event=com.jfoenix"
]

run {
    jvmArgs = args
}

@FelipeAumannRS if you're using jdk 14 you can also try my material library which requires JDK11+
Here: MaterialFX

It can be solved without any JVM parameter with Burningwave Core reflection components  that thanks to a special driver work on all JDKs from 8. In this case you can also simply call the method org.burningwave.core.assembler.StaticComponentContainer.Modules.exportAllToAll()

Please take a look at my new article that explain how to export all modules to all modules at runtime in Java 16 and later without using any JVM parameter

Please take a look at my new article that explain how to export all modules to all modules at runtime in Java 16 and later without using any JVM parameter

@FelipeAumannRS I suggest to switch to non-modular project by removing the module-info.java, and use the plugin https://github.com/beryx/badass-runtime-plugin to make the executable.
And for error module java.base does not "opens java.lang.reflect" add this library https://www.burningwave.org, and then make this call inside your init() in your main class:
Modules.create().exportAllToAll();
Hop this will help.

@FelipeAumannRS I suggest to switch to non-modular project by removing the module-info.java, and use the plugin https://github.com/beryx/badass-runtime-plugin to make the executable.
And for error module java.base does not "opens java.lang.reflect" add this library https://www.burningwave.org, and then make this call inside your init() in your main class:
Modules.create().exportAllToAll();
Hop this will help.

Note that in this way the Modules.exportAllToAll() method is called twice because this method is called automatically when the StaticComponentContainer class is initialized and since the Modules class makes use of components that are in the StaticComponentContainer class when you call the Modules.create() you have practically called the Modules.exportAllToAll().
To change this behavior by allowing only the manual invocation of the Modules.exportAllToAll() methods it is necessary to include in the base folder of your class path a file called burningwave.static.properties which contains a property called module.export-all-to-all whose value is false (note that it is not necessary to put in the burningwave.static.properties file all the properties present in the previous link).
Note also that is better to call org.burningwave.core.assembler.StaticComponentContainer.Modules.exportAllToAll() (copy this entire line) instead of creating a new Modules component.

@Roberto-Gentili , yes , I didn't notice that.
After some debugging you need just to use the class StaticComponentContainer in any ways, ex:
org.burningwave.core.assembler.StaticComponentContainer.JVMInfo.getVersion()
and this will automatically create the class and call Modules.exportAllToAll() only one time.

@Roberto-Gentili , yes , I didn't notice that.
After some debugging you need just to use the class StaticComponentContainer in any ways, ex:
org.burningwave.core.assembler.StaticComponentContainer.JVMInfo.getVersion()
and this will automatically create the class and call Modules.exportAllToAll() only one time.

Exactly! and remember that you can change this behavior by allowing only manual calls to the exportAllToAll method by following the instructions in the previous answer


I've added 210k files into the repo.
It failed with message: grpc: received message larger than max (67323059 vs. 20971520)
I've executed pachctl finish-commit <repo> <commit>
it failed with the same message grpc: received message larger than max (67323059 vs. 20971520)
Looks like you need to adjust: "grpc.max_send_message_length" for grpc client.
Here is the same issues that has been solved using this method:
https://github.com/grpc/grpc/issues/8098

@olegserov Can you provide information about your version of Pachyderm and Kubernetes. We recently updated certain commands to streaming to deal with this issue, so it could be a version thing. If not, we should definitely investigate what's happening here. Thanks in advance!

I've used it locally just to upload the data into the repo. The version was the latest at that date. I've just finished the installation. Unfortunately, I don't have that version anymore.

I was able to reproduce this issue. I'm looking into it now.


How to use in my use case

Dont know how to initialize the crawler to handle my situation, I have this:

http://www.nexion.com.ar
http://smarterp.nexion.com.ar
http://pms.nexion.com.ar

The www.nexion.com.ar has some pages linking to the other subdomains (smarterp, and pms)
I'm starting the crawler like this:
  var site_crawler = new Crawler('www.nexion.com.ar', '/', 80);
  site_crawler.scanSubdomains = true;

But the scanSubdomains = true option is not working as the subdomains are not being crawled.
Any advice please?

The problem was a 500 error on the subdomains, but the fetch works fine. Sorry!


Read OIDC config from current context

This PR makes dexter read OIDC configuration from current context.

Thank you so much for your contribution(s) to dexter. I really like the idea of using credentials already present in the kube configuration. I've reviewed your changes and would like to suggest some changes and in fact it's a pretty big change. I'm not sure what's the best way to handle this situation. I really want you to get the credits for this change. I'll put my changes right into this comment. Happy to discuss the details ;-)
func (d *dexterOIDC) createOauth2Config() error {
	// no commandline client credentials supplied
	if d.clientID == "REDACTED" && d.clientSecret == "REDACTED" {
		// no builtin defaults - let's try auto-configuration
		if defaultClientID == "" && defaultClientSecret == "" {
			log.Info("Autopilot mode - no credentials set")
			if err := d.autoConfigureOauth2Config(); err != nil {
				return errors.New(fmt.Sprintf("failed to extract oidc configuration from the kube config: %s", err))
			}
		} else if defaultClientID != "" && defaultClientSecret != "" {
			// use build-time defaults if no clientId & clientSecret was provided
			log.Info("Using builtin credentials - no credentials set")
			d.clientID = defaultClientID
			d.clientSecret = defaultClientSecret
		}
	}

	// setup oidc client context
	oidc.ClientContext(context.Background(), d.httpClient)

	// populate oauth2 config
	d.Oauth2Config.ClientID = oidcData.clientID
	d.Oauth2Config.ClientSecret = oidcData.clientSecret
	d.Oauth2Config.RedirectURL = oidcData.callback

	switch oidcData.endpoint {
	case "azure":
		d.Oauth2Config.Endpoint = microsoft.AzureADEndpoint(oidcData.azureTenant)
		d.Oauth2Config.Scopes = []string{oidc.ScopeOpenID, oidc.ScopeOfflineAccess, "email"}
	case "google":
		d.Oauth2Config.Endpoint = google.Endpoint
		d.Oauth2Config.Scopes = []string{oidc.ScopeOpenID, "profile", "email"}
	default:
		return errors.New(fmt.Sprintf("unsupported endpoint: %s", oidcData.endpoint))
	}

	return nil
}

// populate the Oauth2Config object from the kube config. Return an error when the operation failed
func (d *dexterOIDC) autoConfigureOauth2Config() error {
	// initialize the clientConfig and error variables
	var clientCfg *clientCmdApi.Config
	var err error

	// try to load the credentials from the kubeconfig specified on the commandline
	if d.kubeConfig != "" {
		clientCfg, err = clientcmd.LoadFromFile(d.kubeConfig)

		if err != nil {
			return errors.New(fmt.Sprintf("failed to load kubeconfig from %s: %s", d.kubeConfig, err))
		}
	} else {
		// try to load credentials from CurrentContext
		clientCfg, err = clientcmd.NewDefaultClientConfigLoadingRules().Load()

		if err != nil {
			return errors.New(fmt.Sprintf("failed to load kubeconfig from the default locations: %s", err))
		}
	}

	// loop through all the contexts until we find the current context
	for contextName, context := range clientCfg.Contexts {
		// find the context definition that matches the current active context
		if contextName == clientCfg.CurrentContext {
			// loop through the global authentication definitions
			for authName, authInfo := range clientCfg.AuthInfos {
				// find the authentication definition that is in use in the current context
				if authName == context.AuthInfo {
					// ensure it is oidc based
					if authInfo.AuthProvider != nil && authInfo.AuthProvider.Name == "oidc" {
						// verify that relevant keys exist
						for _, key := range []string{"client-id", "client-secret", "idp-issuer-url"} {
							if _, ok := authInfo.AuthProvider.Config[key]; !ok {
								return errors.New(fmt.Sprintf("%s is missing in kubeconfig: %s", key, err))
							}
						}

						// set client credentials and idp url based on the kubeconfig definition
						d.clientSecret = authInfo.AuthProvider.Config["client-secret"]
						d.clientID = authInfo.AuthProvider.Config["client-id"]
						idp := authInfo.AuthProvider.Config["idp-issuer-url"]

						// set endpoint based on a match on the issuer URL
						if strings.Contains(idp, "google") {
							oidcData.endpoint = "google"

						} else if strings.Contains(idp, "microsoft") {
							oidcData.endpoint = "azure"
							// TODO: tenant variable is not used. useless in regex
							re, err := regexp.Compile(`(?P<tenant>[0-9a-fA-F]{8}\-[0-9a-fA-F]{4}\-[0-9a-fA-F]{4}\-[0-9a-fA-F]{4}\-[0-9a-fA-F]{12})`) //uuid is tenant

							if err != nil {
								return errors.New(fmt.Sprintf("failed to extract the azure tenant from the issuer URL: %s", err))
							}

							res := re.FindStringSubmatch(idp)
							if len(res) == 1 {
								oidcData.azureTenant = res[0] // found tenant
							} else {
								return errors.New(fmt.Sprintf("failed to identify tenant ID. Found %d matches", len(res)))
							}
						}

						return nil
					}
				}
			}
		}
	}

	return errors.New("failed to auto-configure OIDC from kubeconfig")
}

I've also put my changes on your work in the branch autopilot (https://github.com/gini/dexter/compare/autopilot?expand=1) if you want to see the changes directly.

Nice work, and much more resilient code. As I mentioned in previous PR this is my first attempt to do a pull request and also my first Golang lines. Happy to contribute with ideas!
That said, you shouldn't return error on no tenant found. oidcData.azureTenant should set to "common" and continue.
The if match on the uuid regexp will never evaluate to true as I've used parentheses around it (?P....), the regexp will always return two elements on match. Either increase to if len(res) == 2, or better remove the parantheses and  ?P. First element is matching part of regexp, and second element is whats matches inside each parentheses pairs.
What is next, should I update my PR with your suggestion and these two comments?

I'd appreciate if you could add my suggestions and your own fixes into the PR. So we get the best of both worlds ;-)

LGTM. Thank you!


Respect externalIP setting in spark-submit for jar upload to NodePort service

For a use case @mccheah and I are building, I think we'll need to support NodePort's externalIp metadata.  This comes in two places:

when set the spark-submit script will place the given IP as the external IP on the NodePort service created on the driver (for jar uploads, webui, etc)
when uploading local jars to the driver pod for execution, spark-submit will use the external IP if present on the service, rather than randomly selecting a pod IP and using that (otherwise default behavior)

A strawman name for the new flag would be spark.kubernetes.driver.externalIP
https://kubernetes.io/docs/tutorials/stateless-application/expose-external-ip-address/

After chatting with @mccheah, actually creating an Ingress is more valuable for our workflows than the external IP setting is.  So I'll close this issue and open a separate one for Ingress support.
If anyone needs externalIP support in the NodePort proxy, please ping here and let's reopen to discuss.


[DynamicCasts] Fix a bug in dynamic casting from optionals to protocols

Assume we have:
Protocol P
An intenal extension to Optional - conforming to P
Compiling under whole module mode
Use of Optional type T?
Given a dynamic cast from T? to P, that cast might succeed even if T itself does not conform to P
rdar://problem/40500142

@swift-ci Please smoke test and merge


[WIP] Refactoring documentation

Description:
With this PR, I'm proposing the substitution of Fornax by Nacara as the static site generator for the Saturn docs pages (cc @MangelMaxime).
During my tests, I was able to reuse most of the pages already written with not much changes, except for one parameter (the layout).
Some points of consideration related to this proposed change, are:

This tool uses Node.js. ~If we're going to really use it, I can create another PR (or even tackle it in this one), changing the CI job that builds the documentation~ (done).
I'm not sure how straightforward it is to replace the current configuration in the gh-pages branch (supposing the blog is attached to it). But I'll try my best to replicate what we currently have in this branch here.
~Re-write the API Reference content using markdown~ (done).
~Fix the commands broken ("Docs")~ (done).

Other ideas to consider in the future are:

[ ] Use other themes.
[ ] Add more stylish features.

Continuation of #357.
How to test it?
Nacara requirements:

Node.js 12.20, 14.14, or 16.0

cd docs/
npm run watch

# open localhost:8080 in your browser

After testing the project, one can use this section of the Nacara documentation to understand how to build and expose the project: https://mangelmaxime.github.io/Nacara/nacara/guides/deploy-your-site.html.

Each new commit decrease the quantity of changes since I'm removing a bunch of HTML stuff and using MD instead.

Finished the translation of the references.

I think the PR is done, but I'm not sure how to test locally. Using GH pages to host the built code does not look good, at least in my local repository: https://64j0.github.io/Saturn/. But, if I serve it through npm run watch the project looks beautiful.
I added some questions in this topic to the Issue mentioned before in this PR timeline.

Just discovered that Nacara is going to be deprecated in the future in favor of another project. Due to it, I'm going to consider other possibilities:

https://github.com/MangelMaxime/eleventy-layout-nacara
https://github.com/facebook/docusaurus

Update, I was able to make the documentation look beautiful in my project!

I really like the updated look in your link. Will give this a look-over!

That looks great. How are all those .md files generated?

That looks great. How are all those .md files generated?

@Krzysztof-Cieslak in current implementation with Fornax they are generated automatically in the build process from the code/comments, right? I was not able to implement a similar feature, unfortunately. For this PR, the process was manual.
I got the HTML files from the current gh-pages branch, was looking at the information we have in the docs page, and "translating" it to MD syntax. I was not able to replicate everything as it is in the HTML files, due to limitations in markdown (multi-line tables, for example).
In this scenario I used a different approach, avoiding the table and using another structure to present the same information. Something like this:

Type: SomeType
Description: Some multi-line description.
More description line.
Example:
# comment
printfn "Code example"

Tested locally, and it seems to be ready to be reviewed. I removed the [WIP] from the PR title.

In current implementation with Fornax they are generated automatically in the build process from the code/comments, right?

Yes, that is correct, indeed.

I was not able to implement a similar feature, unfortunately.

I am a bit worried about depending on manual updates for updating API reference documentation - this greatly increases the chance of things being out of sync/people forgetting to update docs in both places etc.
Maybe we could generate those .md files using a standalone (i.e. not depending on Fornax) script/tool? The main heavy lifting is done by F#.Formatting dependency. I believe apirefloader.fsx should have code for generating structured data... and generating .md files is just a matter of string concatenation.
Similarly, we could do the same for generating Lunr search index (lunr.fsx in the old implementation) which powered search capability [which seems to be not there in the new implementation].

I am a bit worried about depending on manual updates for updating API reference documentation - this greatly increases the chance of things being out of sync/people forgetting to update docs in both places etc.
Maybe we could generate those .md files using a standalone (i.e. not depending on Fornax) script/tool? The main heavy lifting is done by F#.Formatting dependency. I believe apirefloader.fsx should have code for generating structured data... and generating .md files is just a matter of string concatenation.
Similarly, we could do the same for generating Lunr search index (lunr.fsx in the old implementation) which powered search capability [which seems to be not there in the new implementation].

All right, thanks for pointing it, I'll take a look at those files to understand it better.

I am a bit worried about depending on manual updates for updating API reference documentation - this greatly increases the chance of things being out of sync/people forgetting to update docs in both places etc.
Maybe we could generate those .md files using a standalone (i.e. not depending on Fornax) script/tool? The main heavy lifting is done by F#.Formatting dependency. I believe apirefloader.fsx should have code for generating structured data... and generating .md files is just a matter of string concatenation.

I agree generating it automatically is better.
Nacara has a dotnet tool to generate F# API: https://mangelmaxime.github.io/Nacara/nacara-apigen/introduction.html
However it is in a really early state.
It is used for generating Fable.Promise API doc

Things to know too is that Nacara is being completely rewritten to work on top of Eleventy. This will allow me to focus on F# specific features + just providing the layout stuff. I will not be responsible anymore for providing template support, watch mode, etc.
I already have the port working in a repo and I am currently working on the documentation and migration guide. According to my test the migration should be smooth:

Adapting the layout property of the docs files
Exploding the nacara.config.js file into several Eleventy Data file (more or less copy/paste)

When ready I will revisit this issue/project to help in the migration.

I decided to not keep going with this effort, to focus in other activities. Due to it, I'm closing this PR.


What's Elfie package licence?

https://www.nuget.org/packages/Microsoft.CodeAnalysis.Elfie
We've been using the NuGet package in our project, but since it's been unlisted the licence info is gone and we don't know if we can still use it or not. Can you help?

Elfie is licensed under the MIT License at the root of the elfie-arriba repo. If you feel a copy is also needed in the Elfie folder, let me know and I'd be happy to put another copy there.
Sorry for the long delay in responding. I didn't have notifications correctly configured for issues in this repo. :/


Fails with "Invalid package file"

Running StartOS with the latest system updates.
The StartOS System and all Services are up to date.
Currently running AlbyHub ver 1.5.3 on my StartOS.  I did not try uninstalling this before installing the new package.  Are there any issues with doing this?  Would removing and replacing affect my Albyhub lightning channel I have already established?
When I attempt to Sideload the service by browsing to the file, I get the "invalid package file" error.
https://github.com/horologger/albyhub-startos/actions/runs/10852287383

Yeah this is like what I reported in #8. I'm not sure if alby hub is even checking these issues. I had put mine up 2 weeks ago with no comment.

I am thinking about uninstalling the older version then try this one.

I unzipped it and it took the file that was inside.

Sigh, this worked. I'm closing my issue.

The somewhat stable version starts from 1.8.0 and onwards. I wouldn’t recommend spending too much time on earlier versions. Please try installing fresh from this here and report back. Thanks!

1.8.0 is now merged.  Coming soon to a marketplace near you.

Closing.  Reopen if needed.


Ignore ros-args in parameter bridge

Starting from eloquent, the --ros-args flag becomes mandatory and, according to the design document, even an empty --ros-flags is valid.
When embedding the parameter_bridge in a launch file in eloquent, i.e. in the following way (forwarding ignition clock to ROS2):
    <node pkg="ros_ign_bridge" exec="parameter_bridge"
<EMAIL_ADDRESS>    />

ros_ign_bridge crashes because it doesn't expect the --ros-args flag:
[ERROR] [parameter_bridge-1]: process has died [pid 1398, exit code 255, cmd '/home/luca/demos_ws/install/ros_ign_bridge/lib/ros_ign_bridge/parameter_bridge<EMAIL_ADDRESS>--ros-args'].
This PR ignores every command line argument after the --ros-args (since it's not used anyway in the bridge), fixing the crash.

Technically, user-defined arguments could come after a -- after --ros-args. Should we handle that case? i.e. ignore all args between --ros-args and --?

True, the specific use case was for use in launch files but it makes sense to fix all the cases.
Since it was becoming quite ugly if we kept the same structure (i.e. we would have needed to "change" the iterator in two places or edit the vector while it is being iterated on) I created a new function that to preprocess the arguments and convert them to a vector of strings for easier iteration.
Tested with the following commands:
Original use case:
ros2 run ros_ign_bridge parameter_bridge /clock<EMAIL_ADDRESS>--ros-args`
And recommended use case:
ros2 run ros_ign_bridge parameter_bridge --ros-args test1 -test2 __test3 -- clock<EMAIL_ADDRESS>


Add Cosmos (ATOM) support

Closes #92. Previous discussion on #107 and #105.
Not sure what all these other commits have been added for - doesn't seem to affect the code, though. Any way I can remove them?

Not sure what all these other commits have been added for - doesn't seem to affect the code, though. Any way I can remove them?

Yes, you can squash your commits (https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History)

Yes, you can squash your commits (https://git-scm.com/book/en/v2/Git-Tools-Rewriting-History)

@terorie I did squash, but I believe it made me 'author' a bunch of non-changes as a result to those other 65 commits. I just found out about cherry-pick though, will update...

@jamesscaur Thank you for PR, now Trust users have transactions history

Thanks for the review @kolya182, have been using Trust for the last year, feels great to have contributed


Name collision

Hi!
Unfortunately there is screenshoting tool from KDE with the same name: https://kde.org/applications/utilities/org.kde.spectacle

In the commit a47fd22, I am renaming the target name, such that there won't be any file conflict with other packages.
Since the LV2 plugin URL doesn't change, this will probably not break any sessions.
The official package name becomes spectacle-analyzer. It will be used starting with the next release.

Will you rename this repo too?

I don't consider it necessary, maybe.
This annoys me more, given there exist already a published link out there.
GH is going to create a redirect, but I don't know if it's lasting / how long.

Yep, agreed that it isn't strictly mandatory. My question was more about a consistency approach. Not a big deal.
All good then!


Advanced OpenSSL config (client certs)

I want to use Iron in a context where I'm required to use client certificates.
However, the OpenSSL config is quite limited now, and I'm not able to set any of the advanced configuration options.
I'd like to contribute, but I'm not sure what the best approach would be. I'm considering adding a struct to Protocol::Https, but that would break backwards compatibility with existing code.
I can also add a new enum variant, something like Protocol::AdvancedHttps with an additional struct with advanced OpenSSL options (such as client certs).
What do you think?

Sorry for off-topic. I just feel the SSL thing may be out of the core scope of iron, as a web framework. In python world we saw a clear separation between servers and frameworks. I think it's time for iron developers think about #261 and #295

I do agree, but at the same time I don't really have the time, or a choice, right now :)


Unable to run on a fresh install, "Unable to find an inventory file"

Trying to use battleschool on a brand new install, but I'm getting the error below. Steps I followed to get to this point:

install Xcode command-line tools
curl -Ls https://bootstrap.pypa.io/get-pip.py | sudo python
sudo pip install battleschool

Then (using the example file from the docs):
[mac:~ home]$ battle --config-file https://db.tt/aG2uyydU
Traceback (most recent call last):
  File "/usr/local/bin/battle", line 7, in <module>
    sys.exit(main(sys.argv[1:]))
  File "/Library/Python/2.7/site-packages/battleschool/main.py", line 118, in main
    inventory = ansible.inventory.Inventory(options.inventory)
  File "/Library/Python/2.7/site-packages/ansible/inventory/__init__.py", line 139, in __init__
    raise errors.AnsibleError("Unable to find an inventory file, specify one with -i ?")
ansible.errors.AnsibleError: Unable to find an inventory file, specify one with -i ?

Unable to progress past this point.
/usr/share/battleschool/defaults/hosts doesn't exist, actual location is /Library/Python/2.7/site-packages/usr/share/battleschool/defaults/hosts

@OldhamMade when I install fresh, the file gets put into /usr/share/battleschool/defaults/hosts.  What does the output of env look like?

env says the user is me, and I've also tried installing from a version of Python & pip installed via homebrew, but I'm still getting the same issues.
I've worked round the problem by symlinking /Library/Python/2.7/site-packages/usr/share/battleschool/defaults/hosts to /usr/share/battleschool/defaults/hosts after installing battleschool.

Fresh El Capitan install. Installed Homebrew, installed python, pip installed battleschool.   I get the same as above.

+1, I think this might be related to the SIP https://en.wikipedia.org/wiki/System_Integrity_Protection

@fearoffish I think it's not a big issue. You can fix this on fresh install on El Capitan with installed battle school:
cd ~ && mkdir .battleschool
cat /usr/local/lib/python2.7/site-packages/usr/share/battleschool/defaults/hosts > .battleschool/hosts
battle -i .battleschool/hosts

fixed via ae22fb613d5465973e1a0437bfc484f24cb90e01


Implement on_status parameter for KubernetesDeleteJobOperator

Introduced three new parameters for KubernetesDeleteJobOperator:

on_status = None | "Complete" | "Failed" - remove the job regardless of its status | remove only successful jobs | remove only failed jobs. The default is None.
wait_until_job_complete - Whether to wait for the job to complete. The default value is False.
job_poll_interval - Interval in seconds between polling the job status. Used when the on_status parameter is set. The default value is 10.0.

Default values for new parameters ensure that existing DAGs will work with no changes.

Can you please update PR so it will be only your commits? :)
It will be easier to review :D

Yes, seems that rebase wasn't clean.

It's better now. Thanks for noticing

Thank you


Possible bug in side bar

I'm using this dashboard in my project, so I programming my pages and components. But I think that occurred this bug in side bar component. I think so this problem is scss code, because I removed the 'div' class and the problem has slightly improved. But that did not solve. Can you help me?

I have met same issue.
This worked fine before, but now I met this issue.

@RoraxHan I fixed the bug and create a pull request here.
But you can fix in your project, follow the steps:

go to index.html file
find this line: https://maxcdn.bootstrapcdn.com/bootstrap/latest/css/bootstrap.min.css
change by: https://maxcdn.bootstrapcdn.com/bootstrap/3.3.6/css/bootstrap.min.css
this must be fix bug.


Feature/typescript

Adds typescript support.

@webzwo0i Do you know how the github action logic is working? The rollup config copies with npm run build relevant files to the dist folder.


CUDA out of memory

V100 32G run base model

Emm, you can try to reduce the batch size and reduce learning rate accordingly


NeuQuant seems inappropriate for limited-colour images

Hi.
This is a kind of upstream investigation of https://github.com/38/plotters/issues/273.
It looks like this library always uses NeuQuant to generate palettes, even if the source image has a limited number of colours and could just be converted directly into a paletted image. This is both slower than necessary, and can produce surprisingly sub-optimal palettes (see above bug).
I guess the normal response is "Use from_palette_pixels or from_indexed_pixels for limited-colour images, however people accessing the gif crate through the image crate (https://github.com/image-rs/image/blob/master/README.md) don't seem to have the option to provide an explicit palette.
My knowledge of image-gif and image are very limited, but I think I can see two main options to providing a convenient interface:

image-gif recognises images with < 256 colours and generates palettes directly without NeuQuant.
image, when calling to image-gif, recognises images with < 256 colours, and does the conversion to a palette-based call.

I think the first option is more general, is likely to have minimal performance impact in the high-colour case, and improve both performance and output quality in the low-colour case.
My Rust is somewhat basic, but I would be happy to try to contribute a PR if this solution sounds acceptable.
Thanks,
Simon.

This all sounds reasonable. I'd welcome a PR for image-gif to recognize whether there's already < 256 colors before calling into NeuQuant

Thanks. I'm making a start on it, but my free time is a little unpredictable at the moment. I think I'll also appreciate code review on my style, as my knowledge of idiomatic rust (let alone the specific style for image-gif!) is pretty limited, but I guess we can deal with that once I've got a rough draft.

I've raised https://github.com/image-rs/image-gif/pull/110 for this issue.

Hi. Is there anything I can do to help move this along? I've updated the PR after code review, but I don't know if there's anything else I should be doing. Cheers.


Crash when more than 10 tabs opened

Crash when more than 10 tabs opened
To Reproduce
Steps to reproduce the behavior:

Ctrl + T to open a new tab
Repeat step 1 (at least 10 times)

Desktop (please complete the following information):

OS Version: [e.g. Windows 10 20H2 19042.685]
App version: [master]

Screenshot

Merging with #2145.

Merging with #2145.

This happens even if you open tabs "slowly" unfortunately and 100% of time

This happens even if you open tabs "slowly" unfortunately and 100% of time

@gave92 It only seems to happen if the last tab is selected when opening the new tabs.

@gave92 It only seems to happen if the last tab is selected when opening the new tabs.


feat: add collection feature for cluster

refer to: https://github.com/horizoncd/horizon-web/pull/4
Add Table
create table tb_collection (`id` bigint(20) unsigned NOT NULL AUTO_INCREMENT,
`resource_id` bigint(20) unsigned NOT NULL DEFAULT 0 COMMENT 'refer to resource id',
`resource_type` varchar(128) NOT NULL DEFAULT 'cluster' COMMENT 'resource type, cluster or application',
`user_id` bigint(20) unsigned NOT NULL DEFAULT 0 COMMENT 'refer to user id',
PRIMARY KEY (`id`),
UNIQUE KEY `uk_resource_user` (`resource_id`,`user_id`, `resource_type`));

Need to add open-api doc

openapi has been added, @iamyeka @bysph please take a look


Bluebird promise cancellation for sqlite3

Implement query cancellation for sqlite3 driver according to Bluebird promise cancellation semantics:

Timeout cancellation in sqlite3 driver according to Knex API (available as a separate PR https://github.com/tgriesser/knex/pull/1834 ). E.g. knexQueryPromise.timeout(5000, { cancel: true }) would actually stop the query execution
A fix for Bluebird issue https://github.com/petkaantonov/bluebird/issues/1303 , which requires resource allocator to handle onCancel event if promise cancellation is used. E.g. it ensures knexQueryPromise.cancel() doesn't leak the allocated connection if cancellation happens before the allocator promise returns
The query cancellation itself in sqlite3 driver. E.g. knexQueryPromise.cancel() would stop the query execution.

Note that while mysql driver also provides a cancellation API (3) is implemented for sqlite3 driver only as the custom query promise is created on the driver level - if such functionality is needed for other drivers, it could either be implemented inside the driver (reusing cancelQuery() code) or by wrapping the per-driver query promise in the custom promise in the generic layer, where onCancel is handled by calling cancelQuery() (though i have no idea about performance and memory use implication of such an extra wrapping)

sounds like a good addition, documentation and tests are mandatory though.

Closing older incomplete  PRs, can be reopened if finished.


Not picking up translations used inside filters

It looks like gulp-wp-pot is not picking up translations inside (some?) filters or functions.
I have two local sites. Site 1 is running gulp-wp-pot 2.3.6 and site 2 is 2.4.0. They're using the exact same gulp file, packages, theme and structure except all packages are up to date on site 2.
Site 1, running gulp-wp-pot 2.3.6 , is correctly picking up translations inside this function from mytheme/woocommerce/checkout/form-coupon.php:
<?php wc_print_notice( apply_filters( 'woocommerce_checkout_coupon_message', esc_html__( 'Have a gift card?', 'sg-one' ) . ' ' . esc_html__( 'Click here to use your gift card', 'sg-one' ) . '' ), 'notice' ); ?>
This function is the same as the WooCommerce core function - i just replaced the translatable strings and translation domain.
The exact same translations are however not generated for site 2, running gulp-wp-pot 2.4.0.
This simple string is also in the same template file:

<?php esc_html_e( 'If you have a gift card code, please apply it below.', 'sg-one' ); ?>
.
Both versions generate this string just fine. That's why I suspect that there is some bug with functions/filters.
These are my dev dependencies:
"devDependencies": {
    "browser-sync": "^2.26.7",
    "gulp": "^4.0.2",
    "gulp-autoprefixer": "^7.0.1",
    "gulp-concat": "^2.6.1",
    "gulp-ext-replace": "^0.3.0",
    "gulp-imagemin": "^7.1.0",
    "gulp-sass": "^4.0.2",
    "gulp-sort": "^2.0.0",
    "gulp-sourcemaps": "^2.6.5",
    "gulp-uglify": "^3.0.2",
    "gulp-wp-pot": "^2.4.0",
    "gulp-zip": "^5.0.1",
    "imagemin-mozjpeg": "^8.0.0",
    "imagemin-webp": "^5.1.0",
    "merge-stream": "^2.0.0",
    "pump": "^3.0.0"
}

This is my gulp task:
gulp.task('translate', function() {
	return gulp.src(themeDir + '**/*.php')
		.pipe(sort())
		.pipe(wpPot({
			domain        : 'sg-one',
			package       : 'SG One'
		}))
	.pipe(gulp.dest(themeDir + 'languages/en_GB.pot'))
});

I'm still having the issue without .pipe(sort()).

Thanks for your report, should be fixed in 2.4.1


"is touching" block doesn't work with element groups

Should work now! https://github.com/MaxBittker/sand-blocks/commit/e7d7b48a48bbe599a166a44a26c0d588b1fa2b3a


chore(deps): update common-dev-assets digest to 708b123

This PR contains the following updates:

Package
Update
Change

common-dev-assets
digest
9403075 -> 708b123

Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.

[ ] If you want to rebase/retry this PR, click this checkbox.

This PR has been generated by Renovate Bot.

:tada: This PR is included in version 1.0.3 :tada:
The release is available on:

GitHub release
v1.0.3

Your semantic-release bot :package::rocket:


Adding NetworkGroups to a NetworkGroup isn't supported.

Hi,
Thanks for the useful package helped me get some scripts that I needed to integration do some FMC automation. Anyway I wasn't able to add NetwortGroups into a NetworkGroup.. unless I missed that option. So I made a simple addition to the api_objects.py to allow this.
api_objects.py just after the if action == 'add'
         if action == 'addgroup':
             netg1 = NetworkGroup(fmc=self.fmc)
             response = netg1.get()
             if 'items' in response:
                 new_net = None
                 for item in response['items']:
                     if item['name'] == name:
                         new_net = {'name': item['name'], 'id': item['id'], 'type': item['type']}
                         break
                 if new_net is None:
                     logging.warning('Network "{}" is not found in FMC.  Cannot add to Ne tworkGroup.'.format(name))
                 else:
                     if 'objects' in self.__dict__:
                         duplicate = False
                         for obj in self.objects:
                             if obj['name'] == new_net['name']:
                                 duplicate = True
                                 break
                         if not duplicate:
                             self.objects.append(new_net)
                             logging.info('Adding "{}" to NetworkGroup.'.format(name))
                     else:
                         self.objects = [new_net]
                         logging.info('Adding "{}" to NetworkGroup.'.format(name))

Thanks
Chris

merged into develop branch.


Create migrating.md for SC4 to SC5 breaking changes

Explicit callouts of what's changed from SC4 to SC5
Description
Adding a page to Sauce Connect documentation calling out how to migrate from version 4 to version 5, specifically with what flags have changed or been removed, and what features like Selenium Relay that are no longer supported.
Motivation and Context
Addresses confusion of what has changed between versions.
Types of Changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)
[x] Documentation fix (typos, incorrect content, missing content, etc.)

Should update the sidebar to include SC4 Migration Guide or smth like this, under Install


OCM-4024 | fix: fedramp admin stage urls

Fixes Admin URLs for FedRAMP staging environment

/lgtm

/approve


Profiling not supported for the Anaconda installed by VS

Dev 15
Anaconda 4.1.1 64-bit 3.5
I'm getting this error when I launch profiling:
Traceback (most recent call last):
  File "C:\Program Files (x86)\Microsoft Visual Studio\VS15Preview\Common7\IDE\Extensions\Microsoft\Python\Profiling\proflaun.py", line 35, in <module>
    vspyprof.profile(__file__, globals(), locals(), profdll)
  File "C:\Program Files (x86)\Microsoft Visual Studio\VS15Preview\Common7\IDE\Extensions\Microsoft\Python\Profiling\vspyprof.py", line 74, in profile
    raise NotImplementedError("Profiling is currently not supported for " + sys.version)
NotImplementedError: Profiling is currently not supported for 3.5.2 |Anaconda 4.1.1 (64-bit)| (default, Jul  5 2016, 11:41:13) [MSC v.1900 64 bit (AMD64)]
Press any key to continue . . .

FYI I get the same error for the (C) Python 64-bit 2.7.

In VsPyProf we assume that VsPerf150.dll will be installed into the system directory, but this is likely not the case. We need to properly support resolving this DLL.

Looks like it is currently still the case, but only the 32-bit DLL is being installed. Pinging the team to see what their plans are.

Not our bug - the internal bug number is 246996


abstract colors as variables to be interpolated, disambiguate blues

Rad TMUX experience, thanks for crafting and sharing!
When I was configuring it to use the molokai color palette to mesh with my vim and iTerm, I found it useful to abstract the hex values for the colors into variables.
Here's a branch with those changes, submitted for your perusal.

Helo @krry 👋
I'm not sure I'm going to merge this, but it's good to have the idea around and discuss about it.
If I had to abstract colors, I would use color1, color2, ... because I don't believe it's a good idea to encourage e.g. red='#ffff00' 😉
From there, the upside of using color1, color2 etc would be to avoid having slightly different blues.

Hey @gpakosz 😁
Cool beans. I agree about the color naming. Having them abstracted lets me
switch between light and dark mode with a block comment and a
source-file, though ideally I would be able to send tmux a command to do
this.  Haven't figured out how to do that yet.
Anyway, thanks for your consideration.
On Mon, Mar 18, 2019 at 7:24 AM Gregory Pakosz<EMAIL_ADDRESS>wrote:

Helo @krry https://github.com/krry 👋
I'm not sure I'm going to merge this, but it's good to have the idea
around and discuss about it.
If I had to abstract colors, I would use color1, color2, ... because I
don't believe it's a good idea to encourage e.g. red='#ffff00' 😉
From there, the upside of using color1, color2 etc would be to avoid
having slightly different blues.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/gpakosz/.tmux/pull/224#issuecomment-473888309, or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAa2n3NBTJQmBmn6rGt7vTzx4WKHVbrTks5vX4VigaJpZM4bOSTI
.

--

Kerry Snyder
⊂( ´ ▽ ` )⊃

@gpakosz I agree with @krry that extracting colors to variable names is a good choice for this case. Instead of going through the whole config file, it's much easier to change color values in one place.

Hello @krry 👋
There's no a gh-189-244-255 branch to address this need

Yay! Thanks for including this feature!


Dependencies

Remove sphinx and pytest from the normal dependencies, since they are not necessary to use the library. Especially sphinx takes a long time to install on travis (but necessary for the mpnum/_docs.py).
Suggestion: Move the HTMLTranslator somewhere where it makes more sense or remove it and scale the svg pics appropriately.

Related: The custom HTMLTranslator causes issues with readthedocs. I would strongly prefer the second suggestion above

Done with 70fc25e0697b2c19479f8c0db2c460486ba9bba4
pytest is still in there, but for now that should make any problems


The volar plugin caused the editorOverviewRuler. selectionHighlightForeground property of vscode to become invalid

Type: Bug
I have submitted an issue to the Volar plugin, but they replied that they don't know what's going on. I'm not sure which side is the problem, but I think there must be a problem with the coordination between volar and vscode. So I also raised a question here, hoping to get help.
Firstly, in vscode, set the value of the editorOverviewRuler.selectionHighlightForeground property to a certain color, such as # 1EC163.
"workbench.colorCustomizations": {
        "editorOverviewRuler.selectionHighlightForeground": "#1EC163"
    }

Then click to select a character in the code (. vue file or. js file), such as "console". If there are multiple "console" on the page, the color block that was just set will appear in the corresponding position of the scrollbar on the right side of the editor, making it easy to quickly navigate through the characters on the page that are the same as the selected character.
When I don't start the volar plugin, this function is normal. I can see the color blocks I set, but once the volar plugin is enabled, the colors I set will no longer take effect.
VS Code version: Code 1.79.2 (695af097c7bd098fbf017ce3ac85e09bbc5dda06, 2023-06-14T08:57:04.379Z)
OS version: Windows_NT x64 10.0.19045
Modes:

/duplicate
Continues on #187120

Thanks for creating this issue! We figured it's covering the same as another one we already have. Thus, we closed this one as a duplicate. You can search for similar existing issues. See also our issue reporting guidelines.
Happy Coding!


Create niteen-kumar.md

Notes for Reviewers
This PR fixes #
Signed commits

[ ] Yes, I signed my commits.

🚀 Preview for commit dce5c9d2550c69a47c066fa06bbd17c5d85653e2 at: https://6711115e1a5d342ea99cb7f4--meshery-docs-preview.netlify.app


any plan to keep key working after new deployment?

basically the db file is not kept

I'm not sure what you mean here.
Can't you just... copy the sqlite file to your new deplayment?


Adding option for Radians to trigonometric calculator

Is your feature request related to a problem? Please describe.
There is no option to enter the angle value in radians to calculate trigonometric functions.
Describe the solution you'd like
I'd like to add the feature to enter the angle in radians as well as degrees, along with providing the option to enter it in terms of pi, which is a popular measurement of angles in radians.

Additional context
I am a Contributor at GirlScript Summer of Code 2022. (GSSOC)
I want to work on this issue. Please assign this issue to me.

@Amit366 Can I work on this issue?

@saujanyasood u can work

I want to work on this issue

@riyajaiswal25 u can work

Hello, can I work on this issue under SSOC?

@its-ayush-07 ok


Karyon guice, governator and Auto Binding

Hi! First of all, apologies if the question is not very clear, I'm newbe in the NetflixOSS ecosystem :)
I'm trying to use the Auto Binding from governator in this example of karyon:
https://github.com/Netflix/karyon/blob/master/karyon2-examples/src/main/java/netflix/karyon/examples/hellonoss/server/jersey/JerseyHelloWorldApp.java
Since I understand...using a manual "bind" with guice you can use the annotation @inject in your classes...

protected void configureServer() {
            bind(AuthenticationService.class).to(AuthenticationServiceImpl.class);
            interceptorSupport().forUri("/*").intercept(LoggingInterceptor.class);
            interceptorSupport().forUri("/hello").interceptIn(AuthInterceptor.class);
            server().port(8888).threadPoolSize(100);
        }

@Inject
    public AuthInterceptor(AuthenticationService authService) {
        this.authService = authService;
    }

But Governator gives you a @AutoBindSingleton to avoid this "manual" declarations since I understand...
In the documentation talks about a "usingBasePackages()"
I have found and example using the LifecycleInjector...
final Injector injector;
            try {
                injector = LifecycleInjector.builder()
                        .usingBasePackages("com.company.microservice")
                        .build()
                        .createInjector();
                LifecycleManager manager = injector.getInstance(LifecycleManager.class);
                manager.start();
            } catch (Throwable t) {
                t.getStackTrace();
            }

But I'm a bit confused and I don't know how to handle through Karyon this initial configuration of Governator.
Maybe someone can enlighten me :)
Thanks!

@victuxbb in general AutoBindSingleton is not a recommended approach. Over time using it in applications inside Netflix, we have found out that it is anti-pattern as it relies on class path scanning and an application owner looses control on which objects get initialized in their applications.
Using explicit bindings is the recommended approach with karyon.

looses the control over the objects? Maybe I don't understand the docs but...
in the documentation of Governator explains that only the classes with the annotation
@AutoBindSingleton(BaseClass.class)
public class MyClass implements BaseClass {
   ...
}

If MyClass is in a package specified via usingBasePackages(), Governator will automatically bind it as:
binder.bind(BaseClass.class).to(MyClass.class).asEagerSingleton();

As I see it's change from a manual "bind" to an annotations in classes.
May be I misunderstood something?
Thanks!

Sorry, I should have explained better. The problem comes when libraries (dependencies provided by third-party) used in an application start using AutoBindSingleton. Since, AutoBindSingleton uses classpath scanning to discover classes, any AutoBindSingleton in the classpath starts to get initialized (due to it being an eager singleton). This gets out of control if an application uses a lot of libraries which define a number of AutoBindSingleton classes and hence causes inconvenience for application developers in terms of gaining control of what gets initialized in an application.
So, it is better to define bindings via modules and hence the application owner decides which modules are to be added to the application. This results in a more predictable and easier to maintain/operate environment.

Ok I understand, thanks for the info :+1:


guide on how to setup private npm registry

The documentation for setting up build is lacking documentation on how to setup support for private npm packages.

Thanks for pointing this out @Evanion. You can achieve this using a build script. Take a look at a way someone else using App Center has solved this.
We have folder with build scripts that is community driven if you'd like to contribute there!

You can also do this using an .npmrc file in your project root. It's pretty simple. This is a tutorial explaining the exact process


Bump latest version to 1-26 on EKD_LATEST_RELEASES

The latest field on EKSD_LATEST_RELEASES should ideally be maintained at least on par with the minimum Kubernetes version supported for the release as that will help us in maintaining a reasonably decent version skew with the max K8s version supported in the release. Trying to bump this to a more recent K8s version has resulted in issues in the past as this field essentially decides the kubectl version that get installed on cli-tools and the later recent versions might not be backward compatible with the older versions we support.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

/approve


[Documentation Update]:  Add Javascript Roadmap

Is there an existing issue for this?

[X] I have searched the existing issues

Issue Description
adding roadmap of javascript
Suggested Change
documentation Update
Rationale
No response
Urgency
High
Acknowledgements

[X] I have read the Contributing Guidelines
[X] I'm a GSSOC'24 contributor
[X] I have starred the repository
[X] I have read and followed the Contribution Guidelines
[X] I have followed the code style guidelines of this project
[X] I have checked for any existing open issues that my pull request may address
[X] I have ensured that my changes do not break any existing functionality
[X] Each contributor is allowed to create a maximum of 4 issues per day. This helps us manage and address issues efficiently
[X] I have read the resources for guidance listed below
[X] I have not used AI-generated content (e.g., ChatGPT, other AI tools)
[X] I have not used content from existing sites (e.g., text, images)
[X] I have followed all security rules and only shared trusted resources

@dhairyagothi, checkout some information https://github.com/CodeHarborHub/codeharborhub.github.io/issues/2838#issuecomment-2219382990

@dhairyagothi, checkout some information #2838 (comment)

And If you want to work on it then create a new file like src/database/roadmap/javascript/index.tsx


Azure DevOps Integration no longer working

OS version: Windows 10
Browser version: Opera <IP_ADDRESS>
Extension version: 3.0.7

Relevant integration (if any):
VisualStudio / Azure DevOps - azure.com
🐛 Describe the bug

I use Opera, which uses the Chrome extension marketplace. The Toggl extension has recently stopped working in Azure Devops (the start task button does nothing) and today the button has disappeared entirely. I have already tried re-configuring the site list. And I've also uninstalled and re-installed the extension to no avail.

I have tried a custom URL in conjunction with the pre-loaded template and resetting settings and it did not fix it. The rest of my team are experiencing the same issue. I suspect the HTML structure of the Azure Devops page has changed.

Expected behavior
The Azure DevOps integration should work :)
Steps to reproduce
Steps to reproduce the behavior:

Go to Azure DevOps
Start a Free account
Enable the integration
Notice the button is not visible

Intercom case

case reference

Case


fix(fan): make fan percentage state behave like light brightness

Description
This makes display state for fan speed percentage similar to that of light brightness.
Note the subtle differences between lights and fans:
https://github.com/piitaya/lovelace-mushroom/blob/829ece99388d5474fba895bd58cc2f3007217ac3/src/cards/light-card/light-card.ts#L177-L180
https://github.com/piitaya/lovelace-mushroom/blob/829ece99388d5474fba895bd58cc2f3007217ac3/src/cards/fan-card/fan-card.ts#L122-L125
Related Issue

#765
Motivation and Context

I have groups of fans and lights and right now, the fans stick out:

This PR makes fan speed percentage behave like light brightness:

How Has This Been Tested

I tested fans with and without support for fan speed percentage, in off and on states.
Types of changes

[X] 🐛 Bug fix (non-breaking change which fixes an issue)
[ ] 🚀 New feature (non-breaking change which adds functionality)
[ ] 🌎 Translation (addition or update a translation)
[ ] ⚠ Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[X] My code follows the code style of this project.
[X] My change requires a change to the documentation.
[ ] I have updated the documentation accordingly.

Note
We'd need to update screenshots in the project README and the fan card docs. It's non-trivial for me to reproduce the screenshot in the project README.

[X] I have tested the change locally.

Hi ! Thank you for the contribution 🙂


Is this type supported

the image insert in the text

I didn't implement it temporarily. But if there are just a few specific
embeded images you can use setIcon and setIconStr of Tokenizer.
On Fri, Feb 24, 2017, 2:05 PM Neexz<EMAIL_ADDRESS>wrote:

[image: image]
https://cloud.githubusercontent.com/assets/9331698/23292356/32d97ec0-fa9a-11e6-964d-328841e65e1a.png
the image insert in the text
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/daquexian/FlexibleRichTextView/issues/4, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ALEcnydBWCKSyklPiJi6MCYWx49TQrQTks5rfnMsgaJpZM4MK292
.

I will implement this feature later :)
On Fri, Feb 24, 2017, 2:23 PM quexian da<EMAIL_ADDRESS>wrote:

I didn't implement it temporarily. But if there are just a few specific
embeded images you can use setIcon and setIconStr of Tokenizer.
On Fri, Feb 24, 2017, 2:05 PM Neexz<EMAIL_ADDRESS>wrote:
[image: image]
https://cloud.githubusercontent.com/assets/9331698/23292356/32d97ec0-fa9a-11e6-964d-328841e65e1a.png
the image insert in the text
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/daquexian/FlexibleRichTextView/issues/4, or mute the
thread
https://github.com/notifications/unsubscribe-auth/ALEcnydBWCKSyklPiJi6MCYWx49TQrQTks5rfnMsgaJpZM4MK292
.

Tokenizer.setIcons(R.drawable.ic_answered);
Tokenizer.setIconStrs("answer");
flexibleRichTextView.setText(" answer " + "文件和目录操作最终是与操作系统和文件系统相关的，不同系统的实现是不一样的，但Java中的java.io.File类提供了统一的接口);
the way you recommended can implement this effect， but i can‘t control the icon size， and  the distance bewteen icon and text can't been control yet


Weaviate compile instructions

Is there a URL or reference on compiling weaviate with go?, windows, linux, macos?

I am on intel mac and trying to run it without docker.
Does it need quoted flags like in your docker example ?
Is there an example JSON config file in the repo anywhere ?

mkdir -p ./modules
weaviate-srv --host <IP_ADDRESS> --port 8080 --scheme http
{"action":"config_load","config_file_path":"./weaviate.conf.json","level":"info","msg":"no config file specified, using default or environment based","time":"2022-02-04T07:39:05+01:00"}
{"action":"startup","error":"invalid config: no authentication scheme configured, you must select at least one","level":"error","msg":"could not load config","time":"2022-02-04T07:39:05+01:00"}
make: *** [srv-run] Error 1

@gedw99 You need to specify the required environment variables. For example, see a user-facing docker-based configuration here. You don't need to use Docker, but you need to set the required vars.
Alternatively a typical dev setup here which you can run without docker. This example assumes running a text2vec-contextionary module, but you can also remove all modules if you like.
Please also see the convenience scripts mentioned here. These are commonly used by the Core dev team during development. For running tests, see here.


The error "Failed to update connection settings for resource xxx to Log Analytics" occurs when try connect an azure function to azure log analytics.

I have an azure function named "xxx".
In the azure Log Analytics workspace -> azure resource, I can see azure function "xxx" is there, then I click it, fill in any fields -> then click "save". But I see this error: Failed to update connection settings for resource xxx to Log Analytics.

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

ID: 521c75f5-5d71-10b5-df71-a8a6c0204a2e
Version Independent ID: 4343bd59-d34e-8d99-a723-292fc27b60bd
Content: Log Analytics FAQ
Content Source: articles/azure-monitor/platform/log-faq.md
Service: log-analytics
GitHub Login: @MGoedtel
Microsoft Alias: magoedte

@imeya Thanks for the comment! We will investigate this issue and update you soon.

Hi @imeya
Sorry for the issue you are experiencing. We are happy to help but this channel is reserved for documentation issues/bugs.
For the best possible response, we recommend posting your issue on either our Log Analytics or Azure Function forums where one of our capable engineers will engage asap.
Your other option is to open a service request with our technical support team using these steps.
We will now close this issue on Github but hope to engage with you further via one of the support channels mentioned above.
Cheers.


should svn endpoint with explicit revision really add /trunk to svn path?

Output of bower -v && npm -v && node -v:
1.8.0
3.7.5
v4.3.0

Steps to reproduce the issue:

bower info svn+https://[svnrepo]/[somepath]#r[revnum]

Describe the results you received:
ECMDERR Failed to execute "svn export [...] https://[svnrepo][somepath]/trunk -r[revnum]
Describe the results you expected:
Because svn support appends /trunk it is not possible to get a module out of a svn repo with modules bundled by project:
[project1]
  trunk/
    module1/
    module2/

Additional information:

Bower worked like this for some time, so I doubt that could be changed now as it would break builds.
However this seems like a fairly common scenario, so I just added a workaround to the alternative SVN resolver I wrote a few months ago: https://github.com/Polarisation/svn-resolver
After installing that you should be able to do:
bower info svn+https://[svnrepo]/[somepath]#r[revnum]-no-trunk and it will not append /trunk as usual. I haven't tested it thoroughly, feel free to log issues if you find any.

This behavior won't be changed but you can use pluggable resolver to implement it differently https://bower.io/docs/pluggable-resolvers/


Error on AvailabilityGroup Creation: SqlAG

Details of the scenario you tried and the problem that is occurring
When Trying to create Sql Availability Group I am getting error.
Generated at 2018-06-25 15:42:49 +0000
Chef::Exceptions::PowershellCmdletException: dsc_resource[create_availability_group] (SqlServer::ae-dsc-availability-group line 11) had an error:
Chef::Exceptions::PowershellCmdletException: Powershell Cmdlet failed:
PowerShell DSC resource MSFT_SqlAG  failed to execute Set-TargetResource functionality with error message:
Creating the availability group 'AG_DBTL'.
InnerException: Cannot bind parameter 'InputObject'. Cannot convert the "[DBServer01]" value of type "Microsoft.SqlServer.Management.Smo.Server" to type "Microsoft.SqlServer.Management.Smo.Server".
    + CategoryInfo          : InvalidOperation: (root/Microsoft/...gurationManager:String) [], CimException
    + FullyQualifiedErrorId : ProviderOperationExecutionFailure
    + PSComputerName        : localhost

The DSC configuration that is using the resource (as detailed as possible)
dsc_resource 'create_availability_group' do
  resource :SqlAG
  property :Name, 'AG_DB01'
  property :Ensure, 'Present'
  property :InstanceName, 'MSSQLSERVER'
  property :ServerName, 'localhost'
  property :AutomatedBackupPreference, 'Primary'
  property :AvailabilityMode, 'SynchronousCommit'
  property :ConnectionModeInPrimaryRole, 'AllowAllConnections'
  property :FailoverMode, 'Automatic'
  property :PsDscRunAsCredential, ps_credential((node['sqlserver']['Username']), (node['sqlserver']['Password']))
end

Version of the operating system and PowerShell the target node is running

Caption                                  OSArchitecture Version    MUILanguages $PSVersionTable.PSVersion

Microsoft Windows Server 2016 Datacenter 64-bit         10.0.14393 {en-US}      5.1.14393.2248
SQL Server edition and version the target node is running

Edition  : Enterprise Edition: Core-based Licensing
Version  : 13.2.5026.0
Language : 1033
What SQL Server PowerShell modules, and which version, are present on the target node.

Name       Version Path

xSQLServer <IP_ADDRESS> C:\Program Files\WindowsPowerShell\Modules\xSQLServer\<IP_ADDRESS>\xSQLServer.psd1
SQLPS      1.0     C:\Program Files (x86)\Microsoft SQL Server\130\Tools\PowerShell\Modules\SQLPS\SQLPS.psd1
Version of the DSC module you're using, or write 'dev' if you're using current dev branch

Name         Version  Path

SqlServerDsc <IP_ADDRESS> C:\Program Files\WindowsPowerShell\Modules\SqlServerDsc\<IP_ADDRESS>\SqlServerDsc.psd1

I have removed xSqlServer DSC Resource and now my server has only SqlServerDsc resource only.
see the output of get-Module sql -ListAvailable
PS C:\Users> Get-Module sql -ListAvailable
Directory: C:\Program Files\WindowsPowerShell\Modules

ModuleType Version    Name                                ExportedCommands

Manifest   <IP_ADDRESS>   SqlServerDsc
Directory: C:\Program Files (x86)\Microsoft SQL Server\130\Tools\PowerShell\Modules

ModuleType Version    Name                                ExportedCommands

Manifest   1.0        SQLPS                               {Backup-SqlDatabase, Save-SqlMigrationReport, Add-SqlAvail...

Please see issue #812, I think you experience the same problem.

Yes same problem but only one change. #812 facing issue with xSqlServer DSCResource but in my case I am facing issue with SqlServerDsc DSCResource. Also I dont find any solution provided for #812, so please let me know how do I resolve this issue?
Thanks

@mohamednazar Issue #812 is for SqlServerDsc as well, just that the issue has been open since before the rename.
Do you have SSMS installed on the target node?
Do you have any other instances of SQL server than version 13?
Are you running SQL Server 2016 with Service Pack 2 (just to confirm the version number)?

@johlju
we installed with SSMS 17.7 on target node.
There are only one version (13.0.5149.0) of Sql server is running on target node.
Target node installed with SQL Server 2016 with Service pack 2 and Cumulative Update 1
Microsoft SQL Server 2016 (SP2-CU1) (KB4135048) - 13.0.5149.0 (X64)   May 19 2018 09:41:57   Copyright (c) Microsoft Corporation  Enterprise Edition: Core-based Licensing (64-bit) on Windows Server 2016 Datacenter 10.0  (Build 14393: ) (Hypervisor)
Please let me know if you need any other info?
Thanks,

@johlju  waiting for your response.

I running this in my lab to see if I can reproduce it, but it's gonna take a while since I don't have that much bandwidth for this.

Thanks for the update, please let me know as soon as you find anything...

The problem seems to be when SSMS is installed prior to the configuration is run. SSMS probably adds assemblies that are the wrong version than SQLPS expects.
PowerShell DSC resource MSFT_SqlAG  failed to execute Set-TargetResource functionality with error message: Creating the availability group 'TESTCLU02-AG1'. InnerException: Cannot bind parameter 'InputObject'. Cannot convert the "[sqltest.company.local\SQL2016]" value of type "Microsoft.SqlServer.Management.Smo.Server" to type "Microsoft.SqlServer.Management.Smo.Server".
    + CategoryInfo          : InvalidOperation: (:) [], CimException
    + FullyQualifiedErrorId : ProviderOperationExecutionFailure

When running the configuration without SSMS 17.7 installed the AG is created successfully.

Thanks after I removed SSMS 17.7 I am able to create SQLAG using DSCResource.
I have to install SSMS before SQLAG creation, so shall I install any lower version of SSMS like 14.0.17177.0?

Not sure it helps with a lower version, but please try and let me know the result. An workaround is to install SSMS after AG is created, but then maybe consistency checks fail when SSMS is installed -  I have not tested that. 🤔
The problem I think is that "Import-Module SQLPS" is loading the wrong assemblies when SSMS is installed.

Using SSMS 17.8.1 together with latest SqlServer module seems to work.
Successful

Windows Server 2012 R2
SSMS 17.8.1
Latest SqlServer module
SQL Server 2016 SP2 CU1
Run SqlAG configuration

@mohamednazar can you confirm that the SSMS 17.8.1 and latest SqlServer module works for you as well?

upgrading ssms to 17.8.1, how to I get latest SqlServer module? Shall I uninstall and reinstall SqlServerDsc module? While I reinstalling will it get latest version of module?

SqlServer module and SqlServerDsc module is two different modules. SqlServer is the "new" SQLPS module. https://www.powershellgallery.com/packages/SqlServer

Oh. I am using SqlServerDsc module. Shall I continue to use it or should I change to SqlServer? Which one is the correct one?

Ah sorry, you should have both installed on the target node 🙂 SqlServerDsc have the DSC resources, and SqlServer has all the cmdlets that SQLPS have.

ok I will install both and try with 17.8 version of SSMS. I will update you the status once done.

@mohamednazar I have merged a few PR that will mitigate these issue (loading wrong assembly) in certain scenarios. But you combination of using SQLPS module with SSMS 17.x will never work. Closing this, and assuming you got this resolved when using the necessary versions of each components above.

@johlju  installing SQL AG first then SSMS 17.x is working fine for me. You can close this case.


Function _neat-parse-media finished without @return)

I've tried to set up breakpoints according to the March 27, 2017 post on Neat
My variables look like this (I assume the syntax is correct via the tutorial, please correct me if I'm wrong):
`
$first-breakpoint-value: 768px;
$second-breakpoint-value: 1023px;
$third-breakpoint-value: 1200px;
$fourth-breakpoint-value: 1280px;
$sm-grid: (
columns: 3,
gutter: 20px,
media: (max-width: $first-breakpoint-value),
color: rgba(#00d4ff, 0.25),
direction: ltr,
);
$md-grid: (
columns: 6,
gutter: 20px,
media: (max-width: $second-breakpoint-value),
color: rgba(#00d4ff, 0.25),
direction: ltr,
);
$lg-grid: (
columns: 12,
gutter: 20px,
media: (max-width: $third-breakpoint-value),
color: rgba(#00d4ff, 0.25),
direction: ltr,
);
$xl-grid: (
columns: 12,
gutter: 20px,
media: (max-width: $fourth-breakpoint-value),
color: rgba(#00d4ff, 0.25),
direction: ltr,
);
`
Using Codekit as my compiler I'm receiving the following error:

Compass: Compiling failed with this error:     error /path/_grid-media.scss: Function _neat-parse-media finished without @return)

max width should be:
media: 'screen and (max-width: #{$first-breakpoint-value})' (need to format it as a string, and escape the variable)

@kgcreative I attempted that and it doesn't correct the issue. I tried using pixel widths as well and compiling still fails. Is this a conflict with Compass?

Can you paste the code you tried?
On Wed, Apr 5, 2017 at 11:59 AM Tom Finley<EMAIL_ADDRESS>wrote:

@kgcreative https://github.com/kgcreative I attempted that and it
doesn't correct the issue. I tried using pixel widths as well and compiling
still fails. Is this a conflict with Compass?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/thoughtbot/neat/issues/594#issuecomment-291961877,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ADSJxtE96oEvxBLn80yxEs0Pf4oU4iwPks5rs-SAgaJpZM4M0qVL
.

I missed the single quotes you included in that code. What differentiates my usage vs. the example in the above Thoughtbot post that details media property usage like this:
`$max-width-grid: (
media: (max-width: 1200px),
);
$small-only-grid: (
media: (min-width:10px) and (max-width: 700px),
);
$print-only-grid: (
media: print,
);`

@whmii - see above blog post. Y'all might need to clarify variable and string usage in neat media calls.
Specifically,
$small-only-grid: (
  media: (min-width:10px) and (max-width: 700px),
);

should probably be formatted as
$small-only-grid: (
  media: '(min-width:10px) and (max-width: 700px)',
);

so that:
$small-only-grid: (
  media: '(min-width:#{$sm-min}) and (max-width: #{$sm-max})',
);

will actually compile

@kgcreative good catch. I will issue a correction

The blog post that contained the error has been corrected 🔨


Set-SCPItem for a directory does not support unregular characters

Trying to send a whole directory using Set-SCPItem that contains unregluar characters in its name leads to what seems like a locked terminal and nothing happens. I haven't investigated this deeply (maybe the same goes for regular files as well) but a command like this:
>> Set-SCPItem -ComputerName computername -Path .\Folder(Name)\ -Destination /Upload/ -Verbose

VERBOSE: Using SSH Username and Password authentication for connection.
VERBOSE: ssh-ed25519 Fingerprint for computername: <...mac-adress-or-something>
VERBOSE: Fingerprint matched trusted ssh-ed25519 fingerprint for host computername
VERBOSE: Connection successful
VERBOSE: Uploading: C:\Users\User\Folder(Name)\
VERBOSE: Destination: /MUSIC/Interplay/Folder(Name)

This will just hang forever (at least over night 😅) and cant be interupted with ctrl+c. I'm gussing it has to do with some path-parsing on the destination side as ( needs to be escaped 🤔 ( on a *unix machine
As a sidenote; if one tries to send a directory, and a directory with the same name doesn't exist on the Destination, one would get a
>> Set-SCPItem -ComputerName computername -Path .\SomeDir\ -Destination /Upload/ -Verbose
(...)
Set-SCPItem: scp: /Upload/SomeDir: No such file or directory

Which is a little counter-intuitive when one tries to send a complete directory, but not a huge problem. Just annoyance 😇
 >>  Get-Module Posh-SSH

ModuleType Version    PreRelease Name                                ExportedCommands
---------- -------    ---------- ----                                ----------------
Manifest   3.0.8                 Posh-SSH                            {Get-SCPItem, Get-SFTPItem, Get-SSHHostKey, Get-SSHJsonKnownHost…}`

you may try to use -PathTransformation parameter

For which part? The sending of directory with unregular characters, or the handling of "no such file or directory"?
(Both?)

the single command you use in your example - Set-SCPItem :)

I still have this problem and not fix it yet, and I'm trying tying to use Set-SCPItem with for loop to looping on each file on my folder and then copy it into my server.
$files = Get-ChildItem -Path $MyPath -File
foreach ($file in $files) {
    $src = Join-Path -Path $MyPath -ChildPath $file.Name

    Set-SCPItem -ComputerName $ComputerName -Credential $Credentials -path $src -Destination $Destination -Verbose
}

What special characters are in your case causing it to fail?

If you specify a directory it will copy the directory with its name as is to the specified path. It is behaving as expected. If you want to upload each file in the directory individually then you will need a loop to upload each to the root of the target path. SFTP is a better option since you will mot have to establish a tear down a connection for each file


ACME udp i/o timeout

After trying out for more than 7 hours :
I am getting a udp error.. Not sure what is the reason.. do i need to open more ports ..
note i am running this in docker swarm(exposed ports, hence it was able to connect and request for certs).
Also, not clear from the documentation, where the certificates gets created, is it a new file .pem or within the acmee.json
[entryPoints]
   	[entryPoints.http]
	address = ":80"
   	[entryPoints.https]
   	address = ":443"
     	[entryPoints.https.tls]

defaultEntryPoints = ["http", "https"]
[acme]
entryPoint = "https"
email =<EMAIL_ADDRESS>#delayDontCheckDNS = 0
#acmeLogging = false
onDemand = true
#dnsProvider = "manual"
#storage = "traefik/acme/account"
# https://letsencrypt.status.io/
caServer = "https://acme-v01.api.letsencrypt.org/directory"
storage = "/etc/traefik/acme.json"
# or "traefik/acme/account" if using KV store
[[acme.domains]]
   main = "aaaaaaaa.com"
   sans = ["www.aaaaaaa.com"]

Using traefik:latest
within docker
| 2017/08/06 00:08:06 server.go:2753: http: TLS handshake error from <IP_ADDRESS>:59452: Cannot obtain certificates map[aaaaa.com:Time limit exceeded. Last error: NS ns15.domaincontrol.com. returned NXDOMAIN for _acme-challenge.api.remitgulf.com.]+v
 | 2017/08/06 00:08:10 server.go:2753: http: TLS handshake error from <IP_ADDRESS>:59822: Cannot obtain certificates map[aaaaa.com:Error presenting token: read udp <IP_ADDRESS>:57243-><IP_ADDRESS>:53: i/o timeout]+v

Thanks for your interest in Traefik !
Every question in the issues means features less 😉.
The issue tracker is for reporting bugs and feature requests only.
For end-user related support questions, refer to one of the following:

Stack Overflow using the "traefik" tag
the Traefik community Slack channel


Logto Integration - Accesstoken

Issue and Steps to Reproduce

I am using logto as my Oidc provider and I amhaving issues with accesstoken, When i do the initial login without a resource it sends me an opaque format accesstoken, which i cannot use to protect my api resource but when i do the login with the a resource it returns a jwt accesstoken which can use to protect the api resource but this token isn't going to work to fetch the user.

Is there a way around this, is there a way i can get a new accesstoken when making request to my api's
Can i do this without hooks.

Versions
latest
Additional Info
https://docs.logto.io/docs/references/resources/
https://docs.logto.io/sdk/Kotlin/android/io.logto.sdk.android/-logto-client/get-access-token-constructor/

Installed packages:

hi @mrige, thank you for your issue.
If you want an access-token that work to fetch user you need to require "openid" scope (that may depend of your provider).
It should work with that scope.
If you do not want use hook, you can use the vanilla-oidc work whithout react. The react part is very light.

Hi @mrige , does it solve your problem?

I close the issue @mrige . Feel free to reopen it if you need it.


Use cloud basePath for API clients from configuration passed in

The API clients aren't using the same basePath derived from the cloud passed in via the client configuration. The causes all requests to fail if the same domain isn't being used for auth.
This passes in the basePath for API clients that is derived from the cloud configuration, keeping the auth and request contexts in sync
#191

/cc @jlangdev @jshcodes
without this change my requests always fail

@jlangdev @jshcodes Any chance this could get looked at?

Thank you for your contribution! 😄

Any update on when this will be published?


Update CLI version to 2.1.301

I think this can be merged, now.

I'm assuming I was tagged accidentally? 😄

@adammodlin Yep, sorry. Clumsy clicking!


elasticPoolName is null for the results returned from restorableDroppedDatabases

Imagine a database that was in an elastic pool was deleted.
When issuing a request to perform a PIT restore for the database with an Elastic Pool as a target, it's required to specify the Elastic Pool ID:
{
  "location": "westeurope",
  "properties": {
    "createMode": "Restore",
    "restorableDroppedDatabaseId": "/subscriptions/{subscriptionid}/resourceGroups/{resourcegroup}/providers/Microsoft.Sql/servers/{server}/restorableDroppedDatabases/{restorableDbId}",
    "restorePointInTime": "2020-07-13T14:50:31.503Z",
    "elasticPoolId": "/subscriptions/{subscriptionid}/resourceGroups/{resourcegroup}/providers/Microsoft.Sql/servers/{server}/elasticPools/{poolname}"
  }

However, the results of restorableDroppedDatabases call always have null as the value of elasticPoolName property, even for the databases that were in a pool. The API version is api-version=2014-04-01, which is the latest available.
{
            "id": "/subscriptions/{subscriptionid}/resourceGroups/{resourcegroup}/providers/Microsoft.Sql/servers/{server}/restorableDroppedDatabases/pooldb2,132391256405870000",
            "name": "pooldb2,132391256405870000",
            "type": "Microsoft.Sql/servers/restorableDroppedDatabases",
            "location": "West Europe",
            "properties": {
                "databaseName": "pooldb2",
                "edition": "Basic",
                "maxSizeBytes": "2147483648",
                "serviceLevelObjective": "ElasticPool",
                "elasticPoolName": null,
                "creationDate": "2020-07-13T14:34:04.987Z",
                "deletionDate": "2020-07-13T14:54:00.587Z",
                "earliestRestoreDate": "2020-07-13T14:43:40Z"
            }
        }

I understand that it should be possible to restore the database to a different pool/to standalone db, so it makes sense that this field is null in a case the original pool no longer exists. However, the field is always null, even if the original pool still exists.
If that's the case and the field is always null, maybe it makes sense to remove the field from the model of RestorableDroppedDatabase and update the documentation accordingly? Or am I missing something?

Hi @bosh-ms, Thanks for your feedback!
are you an internal MSFT user? If so, can you raise sev 3 CRI to unblock you from any Azure SQL DB issue? Github issues are not monitored for a long time.
I will close this issue 'cause it is raised 7 months ago.


Disable options

Is it possible to disable the options button (3 vertical dots) or the facebook/twitter sharing options? since they may cause the embedded player to be filled with an unusable login page.

Hey @dkk thanks for the question. To be honest, I'm not sure. YouTube disabled the "showinfo" player param which turned down some of the overlay info/controls. When I tested, it looks like the share buttons properly open a share sheet on iOS, but that probably does happen on every version of iOS.

Closing this for now. Please feel free to comment if this continues to be an issue you think should be resolved by this library. Thank you!


Wrong formatting for day

Describe the bug
Executing this line:
console.log(moment('2018-01-07T19:58:18.26-08:00').format('dddd, MMMM d, h:mm a, YYYY'))

gives me this result:
"Sunday, January 0, 7:58 pm, 2018"

January 0, instead of January 7
To Reproduce
Created this CodePen: https://codepen.io/anon/pen/WmRabe
Expected behavior
"Sunday, January 7, 7:58 pm, 2018"

The time zone setting of the machine the code is running on
Location set to Los Angeles

The time and date at which the code was run
03/06/2019 12:13 PM

Other libraries in use (TypeScript, Immutable.js, etc)
None

Please run the following code in your environment and include the output:
"Wed Mar 06 2019 12:14:32 GMT-0800 (Pacific Standard Time)"
"3/6/2019, 12:14:32 PM"
480
"Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Ubuntu Chromium/71.0.3578.98 Chrome/71.0.3578.98 Safari/537.36"
"2.24.0"

Additional context
None

Okay it seems that format option d needs to be changed to D.