How to run Emulator

I'm trying to configure the following trick to launch HTML file in a separate Firefox instance

by creating an emulator definition for Firefox associated with HTML files. The emulator definition creates a unique Firefox profile per game and has Firefox launch a new process. This lets Playnite's playtime tracking work perfectly. And creating a new Firefox profile directory inside the game installation directory makes it easy to backup a game save.

Firefox emulator definition:

Executable: C:...\firefox.exe
Arguments: -new-instance -profile "{InstallDir}/firefox-profile" "{ImagePath}"
Supported Filetypes: html,htm

For the game I have a single ROM set pointing to a HTML file
After hitting play:

Based on the screenshot, this issue is unrelated to LaunchMate. The error received is from Playnite itself and seems as though there is something wrong with the way you have the emulator launch action set up.

Based on the screenshot, this issue is unrelated to LaunchMate. The error received is from Playnite itself and seems as though there is something wrong with the way you have the emulator launch action set up.

I thought LaunchMate can be used to configure emulator action like this:

In theory I guess it could, however that error message is not being sent by LaunchMate. The error message you're receiving is from Playnite itself about your launch configuration for this game entry.
If you could explain a bit more what you're trying to accomplish, I may be able to help.

I try to run an HTML page using Firefox that uses new instance and a separate profile
The game:

Has no actions
Has index.html at the {InstallDir}
Has platform "PC (Windows)"

Other parameters are:

App Type: Launch an App
App Path: C:\Program Files\Mozilla Firefox\firefox.exe
App Parameters: -new-instance -profile "{InstallDir}/firefox-profile" "{InstallDir}/index.html"
Launch Group Enabled: True
Auto-Close: True

The game is among "Matched Games"
There are several problems:

Playnite requires at least one Action to perform "Play". It means LaunchMate won't get control if the game has no actions
LaunchMate doesn't replace {InstallDir} by install directory full path inside "App Parameters"

Is it possible

To expand Playnite varibales inside "App Parameters" fields
To ignore Playnite restriction regarding "Play" Action

For the problems:

This is intended behavior. LaunchMate will not take action if no game can be launched
At this time, functionality for expanding Playnite variables has not been implemented

For the questions:

I will look into implementing this in the next version
This I will not implement as it does not fit with the intended behavior of LaunchMate or Playnite. If Playnite considers there to be an error, LaunchMate will not act. LaunchMate is not meant as a replacement for Playnite's game launching, it is meant to perform actions alongside those launches that would typically need to be accomplished by writing a script.

Given all that, it seems your best course of action here would be to stick with launching your firefox instance using Playnite's default emulator launch configuration.
I sincerely appreciate you taking the time to run through this with me, though. I hope you are able to find some other use for LaunchMate in the future.


uncaught exceptions are not getting captured in the console logs

What versions are you running?

$ go list -m github.com/chromedp/chromedp
github.com/chromedp/chromedp v0.3.0
$ google-chrome --version
Google Chrome 72.0.3626.109
$ go version
go version go1.12.4 linux/amd64

What did you do? Include clear steps.

I have an html file which tries to call an undefined function.
I am serving the file through a local web server.

What did you expect to see?
When I load the file using chromedp, I do not see the error captured in the code.
What did you see instead?
No log in the output
But if I just add a console.error("something") above the line, that shows up fine.
<!doctype html>
<html>

<head>
	<meta charset="utf-8">
	<title>Go wasm</title>
</head>

<body>
	<script>
			// console.error("got msg"); If you uncomment this line, "got msg" clearly shows up.
			hello();
	</script>
</body>
</html>

The Go code:
package main

import (
	"context"
	"fmt"
	"log"
	"os"
	"strconv"
	"time"

	cdpruntime "github.com/chromedp/cdproto/runtime"
	"github.com/chromedp/chromedp"
)

func main() {
	logger := log.New(os.Stderr, "", log.LstdFlags|log.Lshortfile)

	// create chrome instance
	timeout, cancel := context.WithTimeout(context.Background(), 5*time.Second)
	defer cancel()
	ctx, cancel := chromedp.NewContext(
		timeout,
	)
	defer cancel()

	chromedp.ListenTarget(ctx, func(ev interface{}) {
		switch ev := ev.(type) {
		case *cdpruntime.EventConsoleAPICalled:
			for _, arg := range ev.Args {
				s, err := strconv.Unquote(string(arg.Value))
				if err != nil {
					logger.Println(err)
					continue
				}
				fmt.Printf("%s\n", s)
			}
		}
	})

	err := chromedp.Run(ctx,
		chromedp.Navigate(`http://localhost:8080`),
	)
	if err != nil {
		logger.Fatal(err)
	}
}

Basically, any uncaught exception does not show up. This came up when I was using this library to make wasmbrowsertest. But CORS issues were not getting logged. I had to manually run the code in the browser to see that it was a CORS issue.
Is there any technical limitation due to which this is happening ?
From the documentation - I see https://chromedevtools.github.io/devtools-protocol/tot/Runtime#event-exceptionThrown to be there which should give you the details. But I don't see anything like that in the code.

Thanks for following the issue template - nice for a change :)
Your ListenTarget only handles console calls, such as console.log. If you also made it handle Runtime.exceptionThrown, you could make that do whatever you want. So that's working as intended, as far as I can see.
On the other hand, you could argue that chromedp should log exceptions by default, since they would help users spot bugs quicker. That isn't happening right now. Is this what you mean by this issue?

Ah my bad for not reading the docs well enough. The reason I thought cdpruntime.EventConsoleAPICalled would capture it because console.errors were getting captured with it, and not just console.log . So I thought something must be missing for exceptions, because the browser logs it as a console error.

On the other hand, you could argue that chromedp should log exceptions by default, since they would help users spot bugs quicker.

I think you could add exception handling here in this example - https://godoc.org/github.com/chromedp/chromedp#example-ListenTarget--ConsoleLog
That should handle the most common cases and would make it clear to the user that there is a separate event for exception.
Thanks for the help :)

What about mixed content warnings in the chrome browser? How can I pick those up in a similar fashion?


timout with null

some jobs is marked as failed when they are succ due null/timeout:
Plugin version: 1.0.13 as 1.0.22 is not accepted by jenkins due cross dep - checking
Jenkins:

Openshift:

trying to upgrade to newer jenkins version

update to bleeding edge sorted this one. sorry. we can close.


Circular require warning

Caused by geocoder/models/mongo_base.rb, require 'geocoder'
Full warning:
C:/Ruby200/lib/ruby/gems/2.0.0/gems/geocoder-1.2.7/lib/geocoder/models/mongo_base.rb:1: warning: loading in progress, circular require considered harmful - C:/Ruby200/lib/ruby/gems
/2.0.0/gems/geocoder-1.2.7/lib/geocoder.rb

Fixed--thanks!


Warning message from 1.8.0-rc3

Hi, I’m getting
WARNING: could not import Base._rangestyle into ArrayLayouts

when I run CI for my package  SIAMFANLEquations.jl . This seems to come from BandedMatrices' dependence on ArrayLayouts. This is new for 1.8.0-rc3.
Is this serious?
See

Updating packages solved the problem.


Command card "Son of Skywalker" missing?

I can't find this: http://cards.boardwars.eu/albums/Core-Box/Command Cards/Son of Skywalker.jpg

Yeap, that is not included in the datased actually. thanks for spotting this.
When I scrapped the albums I neglected to follow up for additional pages in the command cards sub album......
Which is all really unfortunate because I though that I was done with the initial data load.
I will have to load it up in the next few weeks.
I missed scrapping at least this page
I may have missed others, but I will double check.
Sorry for the inconvenience. :(

I asked the guy of the website to provide rest API, so you may automate the process of checking what's missing. In case you haven't already done that, I did it time ago to produce a spreadsheet that I then use for data mining when I build up skirmish lists:
https://docs.google.com/spreadsheets/d/12FeBA_aKu2rM90jLk-IrrWTeqpt3mJZ7TmhevNMwT5c
You can copy and paste the full list of card names and then cross check with what you currently have in JSON. You'll see duplicates in my list because some cards are actually duplicated across boxes and packs. Feel free to copy the spreadsheet if you want to do filtering and any other sort of mining (it's protected from editing).

I counted 212 different command cards. You have 189 in your json.  However I counted duplicates too.

I have found the error the crawler I coded up for http://cards.boardwars.eu
I found 17 additional command cards that I neglected.

Fixed in release v1.0.4


More image extensions

pdf
tiff
ico

Does this work for .ico?
def self.ico?(file)
  ["\000\000\001\000", "\000\000\002\000"].include?(IO.read(file, 4, nil, :encoding => 'binary'))
end

I've added a check for .ico in 1.3.5. I'm not going to treat PDF's as an image because I don't see a consensus that PDF's should be treated as such. Anyway, thanks for the suggestions!


Start publishing docs for libkvikio / kvikio

This PR sets up Doxygen docs for libkvikio and Sphinx docs for kvikio.
Currently, this is only barebone API docs, but should provide a decent structure to iterate on moving forward.

@madsbk not yet - still want to sync with ops to make sure that the required changes to gpuCI are made and get their approval here

Thanks for working on this Charles! Also thanks for helping us get things setup here AJ 😀


Android Support

At some point between the NDI v3 and v4 SDKs, builds for the Android platform were added to the SDK distribution under $NDILIB_REDIST_FOLDER/Android. As far as I can tell, this isn't documented, but it should be possible to include these builds and make Devolay compatible with android environments. This will require Gradle multi-platform builds and a rewrite of how devolay-java loads devolay-native artifacts.

hi,
Libs failed to load when integrated into anroid environment. The ide used is android studio

hi
java.lang.UnsatisfiedLinkError: dlopen failed: "/data/data/com.mtool.ndidemo2/cache/devolay-natives9096383232614741647.so" .dynamic section header was not found

I will be able to take a closer look at this in the next few days. For anyone seeing this thread and not in the loop, since v2.0.0, Devolay has been published with experimental Android binaries. I'll close this issue when they are confirmed working (right now it seems like they aren't).

Okay, thank you very much and look forward to it

Hello, is there any progress on the problem, sorry to disturb you, but I am more anxious to use your library here

Sorry for not getting back to you faster. I am having a few issues that seem to only pop up using the Android Studio emulator. I just published my latest code to the integrated_builds branch (https://github.com/WalkerKnapp/devolay/tree/integrated_builds), and it seems to crash from a JNI error on the emulator. However, it works fine on the two physical devices I tested it on.
If you can, it would be great if you could download the devolay-2.0.1-integrated.jar jar file from the github action and test it in your workflow. Let me know the details of any crashes, if they pop up.
To note, you'll be the first person using integrated builds, so please make sure to check the Licensing Considerations section of the README before you publish anything.

I can only replicate these issues now on x86_64 Android Emulators. I assume this is either an issue with NDI's binary for that platform, or that it's an emulation issue. Either way, I am almost certain that it isn't an error in Devolay (or at most, it is an error in Devolay's buildscript).
I am going to go ahead and close this issue with release v2.0.2, since it appears to support all physical devices and non-x86_64 emulators. I opened a separate issue for x86_64 emulators (#19). If anyone experiences more issues with pure Android support, feel free to either comment on this issue or open a new issue.


Add default focus to dialog

Define an attribute that should be used as a first focus.
Component should remember old focused element and focus it back when closed.

We might also want to trap focus inside of the modal -> https://hiddedevries.nl/en/blog/2017-01-29-using-javascript-to-trap-focus-in-an-element


how to change UI language

Actions before raising this issue

[X] I searched the existing issues and did not find anything similar.
[X] I read/searched the docs

Is your feature request related to a problem? Please describe.
No response
Describe the solution you'd like
No response
Describe alternatives you've considered
No response
Additional context
No response

I want to change the web UI language to Chinese, how to  achieve it . thanks

We do not support Chinese language. There is existing issue about localization support https://github.com/cvat-ai/cvat/issues/374


empty 404.html file is generated

🐞 Bug report
Description
When rendering Scully, it generates a 404.html file at the root of the output directory with the following content
<!DOCTYPE html><html lang="en-US" dir="ltr"><head>
  <meta charset="utf-8">
  <title>Title</title>
  <base href="/">
  <meta name="viewport" content="width=device-width, initial-scale=1">
  <link rel="icon" type="image/x-icon" href="favicon.ico">
</head>
<body>

<app-root></app-root>

<script src="runtime.6f0cf46042137677.js" type="module"></script><script src="polyfills.25496f96c6795738.js" type="module"></script><script src="main.a6b398da8515d7cc.js" type="module"></script>

</body></html>

The content of the page is empty, so there is no use for this 404 file.
I created a custom 404 file and add route to the angular router
{
  path: '404.html',
  component: NotFoundComponent
}

But on rendering, it gives an error
  x Error during file write Error: ENOTDIR: not a directory, open '/.../project/dist/static/en-US/404.html/index.html'

🔬 Minimal Reproduction
💻Your Environment
Angular Version:

     _                      _                 ____ _     ___
    / \   _ __   __ _ _   _| | __ _ _ __     / ___| |   |_ _|
   / △ \ | '_ \ / _` | | | | |/ _` | '__|   | |   | |    | |
  / ___ \| | | | (_| | |_| | | (_| | |      | |___| |___ | |
 /_/   \_\_| |_|\__, |\__,_|_|\__,_|_|       \____|_____|___|
                |___/

Angular CLI: 14.1.3
Node: 16.16.0
Package Manager: npm 8.11.0
OS: darwin arm64

Angular: 14.1.3
... animations, cli, common, compiler, compiler-cli, core, forms
... localize, platform-browser, platform-browser-dynamic
... platform-server, router

Package                         Version
---------------------------------------------------------
@angular-devkit/architect       0.1401.3
@angular-devkit/build-angular   14.1.3
@angular-devkit/core            14.1.3
@angular-devkit/schematics      14.1.3
@nguniversal/builders           14.1.0
@nguniversal/express-engine     14.1.0
@schematics/angular             14.1.3
rxjs                            7.5.6
typescript                      4.7.4

Scully Version:

2.1.36

🔥 Exception or Error

  x Error during file write Error: ENOTDIR: not a directory, open '/.../project/dist/static/en-US/404.html/index.html'

Did you try using this? https://scully.io/docs/Reference/plugins/community-plugins/http404/


The "missing port in address" error when the port is not required

Welcome!

[X] Yes, I've searched similar issues on GitHub and didn't find any.
[X] Yes, I've searched similar issues on the Traefik community forum and didn't find any.

 Fallback to request host. error="address example.com: missing port in address" providerName=acme
2024-09-27T22:32:33Z DBG pkg/provider/acme/challenge_http.go:104 > Retrieving the ACME challenge for example.com (token "kwuJCKyII22J2FkgrRa-mEWQvejMf5lFEQO5alUybgU")... providerName=acme
2024-09-27T22:32:34Z DBG pkg/provider/acme/challenge_http.go:84 > Unable to split host and port. Fallback to request host. error="address example.com: missing port in address" providerName=acme
2024-09-27T22:32:34Z DBG pkg/provider/acme/challenge_http.go:104 > Retrieving the ACME challenge for example.com (token "kwuJCKyII22J2FkgrRa-mEWQvejMf5lFEQO5alUybgU")... providerName=acme
2024-09-27T22:32:34Z DBG pkg/provider/acme/challenge_http.go:84 > Unable to split host and port. Fallback to request host. error="address example.com: missing port in address" providerName=acme
2024-09-27T22:32:34Z DBG pkg/provider/acme/challenge_http.go:104 > Retrieving the ACME challenge for example.com (token "kwuJCKyII22J2FkgrRa-mEWQvejMf5lFEQO5alUybgU")... providerName=acme
2024-09-27T22:32:34Z DBG pkg/provider/acme/challenge_http.go:84 > Unable to split host and port. Fallback to request host. error="address example.com: missing port in address" providerName=acme
2024-09-27T22:32:34Z DBG pkg/provider/acme/challenge_http.go:104 > Retrieving the ACME challenge for example.com (token "kwuJCKyII22J2FkgrRa-mEWQvejMf5lFEQO5alUybgU")... providerName=acme
2024-09-27T22:32:34Z DBG pkg/provider/acme/challenge_http.go:84 > Unable to split host and port. Fallback to request host. error="address example.com: missing port in address" providerName=acme
2024-09-27T22:32:34Z DBG pkg/provider/acme/challenge_http.go:104 > Retrieving the ACME challenge for example.com (token "kwuJCKyII22J2FkgrRa-mEWQvejMf5lFEQO5alUybgU")... providerName=acme

What did you see instead?
The "missing port in address" errors. To me it was not clear if that was my fault or something else. After running traefik with delve I've found out that the address is the value of the Host header of the request coming from CA/Let's Encrypt. As such the error message makes no sense (the port is not required) and only confuses  users.
What version of Traefik are you using?

Debian 10
If applicable, please paste the log output in DEBUG level
No response

Hello @x-yuri,
The logs you are seeing are debug logs (DBG) describing a decision being taken by Traefik.
If something was wrong on your configuration on anything else it would have been logged as an error.


Product.availability and ProductVariant.availability have incorrect deprecation message

The message tells the user to use is_available on the parent type but the actual name is isAvailable as GraphQL uses camel case.

Hello, can I help out on this ticket?

@MatthewBosek Sure, feel free to pick it up! AFAIK nobody in our team is working on that.

@maarcingebala Excellent! I will get on it in the evening when I return from work.

@maarcingebala I keep receiving this error when setting up the db:
raise ImproperlyConfigured("The SECRET_KEY setting must not be empty.")
django.core.exceptions.ImproperlyConfigured: The SECRET_KEY setting must not be empty.
I have gone into the settings.py file and set a value to the SECRET_KEY.  How should I proceed? Thanks,

Fixed in #4707


libssl bindings fall back to ancient library version

The bindings for libssl try to figure out the installed version of the library via pkg-config. If that does not work out (pkg-config missing or the specific configuration for libssl is missing), it falls back to a default version.
This default version is currently 0.0.0, i.e. the lowest version number. That means all version comparisons regarding more recent versions will report the version to be lower. Newer versions of libssl have a number of breaking ABI changes compared to ancient versions which are still supported in the bindings.
As as result, if pkg-config fails, the bindings will be tailored for an old version and most likely won't work with whatever reasonable version is available on the system.
I think this is a very poor fall back behaviour. Of course, ideally, pkg-config should be available and provide the necessary information. But that's not always the case. I think a more reasonable fall back would be to a healthily more modern version. This should allow most parts of the bindings to work by default with more modern version of the library.
I think a good choice would probably be 3.0.0, or maybe 1.1.0 although the former has spread pretty wide by now.

Related: #7244, #11069

I still believe #7245 was the correct solution: get the information from C headers when pkg-config is missing. It would detect the actual version of libressl or openssl and fail when the headers are missing.
We can't guess what version or which of libressl or openssl is installed, and we shouldn't have to.

@ysbaddaden This could be helpful, but not a proper solution. The whole problem is more complex and improving autodiscovery only get's us so far.
This issue is just about the current fallback solution if autodiscovery of the library version doesn't work. #7245 wouldn't solve this problem because C headers could be missing as well. So we still need a default behaviour when autodiscovery doesn't work.

I'd like to keep this issue focused narrowly on the fallback version in the current state of things. It could be relatively easy to gain a small improvement by updating the fallback to a more recent version. That's certainly not a wholesome solution to the overall problem, but it's managable and has a direct benefit.
We should have another discussion about the whole topic of library information. I'm collecting some talking points for that and will open a thread on the forum.

There is no fallback in #7245: compilation will fail if the C headers aren't installed (or reachable).

Coming back to this: the openssl situation can't be solved like the other libraries.
There are two conflicting forks, each with their own set of features, but the same set of header files (sic) and lib names (sic), the pkg-config can be missing or report an older version (sic) and we can't tell libressl vs openssl anyway.
We try to guess and we fail in the worst possible manner: by happily building an invalid executable that will crash or raise exceptions without any sort of explanation :sob:
The only way to tell OpenSSL from LibreSSL and which version they are is to look at OPENSSL_VERSION_NUMBER and LIBRESSL_VERSION_NUMBER from the C headers. We MUST require the C headers to be present and we MUST use the above constants to determine their version.
I should rework #7245: nobody came up with anything better :shrug:
I prefer to fail compilation with an explicit and nice:

please install openssl or libressl development files (you can specify OPENSSL_DIR to target a specific installation).

Might not help much, but it should be possible to tell LibreSSL vs. OpenSSL form pkg-config data. Just not directly with the pkg-config tool alone.
Usually the package config file contains additional metadata such as Name and Description. And I presume they should mention whether the package is libre or open. Might not be guaranteed, though. And these fields are not required as far as I know, so the information could just be missing.
The metadata is not available via the pkg-config CLI. But it can tell the location of the discovered pc file via the variable pcfiledir.
This lets us construct the full path as $(pkg-config --variable=pcfiledir $modname)/$modname.pc. We can read the file and check if it contains LibreSSL.
But yeah, the entire situation is pretty messed up.

Sadly it doesn't :sob:
[vagrant@openbsd7 lib]$ cat /usr/lib/pkgconfig/libssl.pc
prefix=/usr
exec_prefix=${prefix}
libdir=${exec_prefix}/lib
includedir=${prefix}/include

Name: OpenSSL-libssl
Description: Secure Sockets Layer and cryptography libraries
Version: 2.0.0
Requires.private: libcrypto
Libs: -L${libdir} -lssl
Cflags: -I${includedir}

[vagrant@openbsd7 lib]$ grep VERSION_NUMBER /usr/include/openssl/opensslv.h
#define LIBRESSL_VERSION_NUMBER 0x4000000fL
#define OPENSSL_VERSION_NUMBER  0x20000000L
#define SHLIB_VERSION_NUMBER "1.0.0"

So it's technically not lying: the libressl pc file reports the "openssl" version (that never existed) :shrug:

Maybe the shell script from #7245 then we rebuild a simplified semantic version string from the version number:
0x10200000L => 0x1_02_00_00_f => "1.0.2"
0x30202000L => 0x3_02_02_00_f => "3.2.2"
0x4000000fL => 0x4_00_00_00_f => "4.0.0"

The script would allow to autodetect OpenSSL vs LibreSSL from installed headers, but also allow to override the version string with environment variables, for example OPENSSL_VERSION=1.1.1 and LIBRESSL_VERSION=4.0.0 to override the behavior.


Skeleton object css is not grouped properly

currently being displayed in "General" Group.

closed by https://github.com/whitesmith/qnorr-styles/commit/aac6c6c7fc5d2438810f0a307672b666daf15d87


Update vaccines page copy and edit batches

Update the design of the vaccines list page to (better) match the prototype and edit functionality:

With more user-friendly date-specific validations (that have also been added to the new route):

"Archive" functionality will be added in a future PR.
I did have some troubles matching the styling of the text below the vaccine brand to the prototype, note the spacing and the font used below:
Prototype

Build

Put this PR in the camp of "review apps were useful to have here and saved time" 👍


Dashboard - Feed: The user should be able to see the "delete card" hint animation only once in each session

The user should be able to see the "delete card" hint animation only once in each session
and not every time the feed is being loaded.

Checked on Dev env (V 0.19.0 / V 0.19.2)
The "delete card" hint animation is displayed only once in each session for user
The device used: iPhone 8 Plus // iOS 13.1.3
Desktop// Windows 10 x64 // Google Chrome 81.0.4044.122
Video: https://www.screencast.com/t/yOWb22JQ


track_caller on custom assert functions

This makes test helper functions point to the failing test, rather than internals of a function that did some checking.

Good call.
@bors r+

:pushpin: Commit 3f7f0942cd641b5959b7d21a8909eb039eec2de0 has been approved by Eh2406

:hourglass: Testing commit 3f7f0942cd641b5959b7d21a8909eb039eec2de0 with merge 3886aaa6e2f00021ec3c5af70ba6cd464cf6eee7...

:broken_heart: Test failed - checks-actions

I don't get why the "homu" test failed. The log only shows:
Run exit 1
  exit 1
  shell: /usr/bin/bash --noprofile --norc -e -o pipefail {0}
Error: Process completed with exit code 1.

@bors retry

:hourglass: Testing commit 3f7f0942cd641b5959b7d21a8909eb039eec2de0 with merge 61a31bc5ff290322d5c025bf750c56501999235d...

:sunny: Test successful - checks-actions
Approved by: Eh2406
Pushing 61a31bc5ff290322d5c025bf750c56501999235d to master...


Add test git-dep-with-submodule

A reproduction test and working on a fix for #110.
Reproduce with
nix build -f test.nix git-dep-with-submodule

So I included a ... partial fix? I'm not sure. I don't think moving to pkgs.fetchgit is better. Unfortunately submodules = true; isn't supported on fetchGit... Do you have any advice?

I don't have a working solution to this so I'll close and leave the report open. :)

Hey! I couldn't give this more attention, I have a lot on my plate at the moment. I'll try to look https://github.com/nmattia/naersk/issues/110 as soon as possible. In the meantime, thanks for giving this a shot!

I think we all do right now. :) Keep up your really amazing work @nmattia ! Thanks for looking at this!


adding a domain drop down for okta workbook

Fixes #
Proposed Changes
-Adding Domain as a Drop Down to the OKTA workbook to allow for organizations that have multiple okta environments.

Hi @v-jayakal - Please review and accordingly approve / merge this PR - that way this can get deployed as well. Thanks.


Create systemd unit file

Current deployment options that are available to be used by the users are Docker, Fly.io, and Heroku. What if they want to run it on their VM without any of that with minimal resource? They can use systemd.
The task is very simple, just provide a systemd unit file with environment variables already loaded (and give notice for them to change it). Please take into account of the restart policy and it needs network connection.

Hey @aldy505 - I can add a systemd service file. Please assign this to me.


JSS helper components not null when receiving empty layout service field

TL;DR
Layout Service/JSS helper components aren't smart enough to be null when not in EE or field value is empty.
Potential solutions:

JSS Helper Components should be Null or a React Component instead of rendering as null when field is null or value is empty (and not in EE)
Layout service does EE check

Is your feature request related to a problem? Please describe.
Common issue is that we are passing JSS React components down into vanilla components.
Say I have a vanilla Card component that conditionally renders body if it's passed in
const Card = ({ body }) => (

Hello World
 {body &&
{body}
}

);

I might use them like this and in all of these cases, the div wrapping body will render or not render appropriately:
const MyCardSection = () => {
  return (

      <Card body={"hi mom"} />
      <Card body={null} />
      <Card />

  );
};

Now I want to use my Card in JSS Sitecore so I create a JSS wrapper component that takes in layout service data and passes in a RichText to handle EE. However, when we're working with the layout API and using JSS Components, we lose the ability to know if a value being passed into the card is null because we're passing in a JSS Component:
const JSSCard = ({ fields }) => {
  const body = <RichText field={fields.body} />;
  return <Card body={body} />;
};

Above, RichText is we're still passing a valid react component into my Card, even if it will at some point render null. So when using Card in JSS, even if fields.body is {value: ""}, the div tag wrapping my body prop inside my card will always render because RichText is not null, it's a react component that will render as null. Note that this is behavior that we only want in EE.
Describe the solution you'd like
My current thinking is that ideally the EE check would happen in the layout service so we do something like this with confidence:
field.body.value == "" ? <RichText field={fields.body} /> : null

Or another potential solution would be to update the JSS React Helper components to be wrapped by a function that either returns null or a react component instead of returning a react component that will render as null if field is null or it's value is "".
Describe alternatives you've considered
A solution to this is that every time I want to use a field from the layout service in a JSS helper component like Text, I have to essentially do something like
(fields.body && fields.body.value !== "") || sitecoreContext.pageEditing ? (
  <RichText field={fields.body} />
) : null;

That's repetitive and annoying. One little util we came up with to solve this problem is something like this
import { isExperienceEditorActive } from "@sitecore-jss/sitecore-jss";

const nullifyFieldsIfNoValueAndNotEditing = fields => {
  return isExperienceEditorActive() ? fields : Object.keys(fields).reduce((newFields, fieldName)=> {
    const field = fields[fieldName];
    const fieldHasValue = field && field.value;
    newFields[fieldName] = fieldHasValue ? field : null
    return newFields
  }, {})
};

And just wrap every fields in it in our components:
const {
  body,
} = nullifyFieldsIfNoValueAndNotEditing(fields);

So we can safely just do
fields.body ? <RichText field={fields.body} /> : null;
Obviously still not the best, but a util like this could be useful to include in JSS (or here for reference) if better solutions aren't immediately feasible.

@jplwood You can safely use getFieldValue which you can import from sitecore-jss-react.
Look at this function here
Example:
import { getFieldValue } from '@sitecore-jss/sitecore-jss-react';

const props = {
  "fields": {
    "heading": {
      "value": ""
    },
    "sample": {
      "value": "111."
    },
    "sample2": {
      "value": "222."
    }
  }
}

console.log('Value: ', getFieldValue(props.fields, 'heading', null))

@jplwood Feel free to reopen this issue, if you have same problem


Compose box background blur doesn’t extend to notch area

I’d expect the send button to be as far away from the display edge as it is at the moment, but the blurred background should extend all the way to the edge of the display (both left and right).

Fixed in 1.3.1


Issue #1: First

Link	first
Location	Menu navbar, first link after the logo
Expected result	Bob Monkhouse's quote (he is the first of the six)
Routing Strategy	Normal routing
Required	No routing rule
How to Fix It	A suitable new controller in controllers/
Hint	application/controllers/First::index()

fixed by @New3


I have made a new file

Please share you email id then we can mail you the participation certificate


fix: changed hello world message to display it in console

Updated Pull Request #30

@rodrigomata Thanks for your contribution. The test result is published and it is inline with go http.


Readme sample does not work

Are you requesting a feature or reporting a bug?
What is the current behavior?
What is the expected behavior?
How would you reproduce the current behavior (if this is a bug)?
Provide the test code and the tested page URL (if applicable)
Tested page URL:
Test code
import { Selector } from 'testcafe';

fixture `Example page`
    .page `http://devexpress.github.io/testcafe/example/`;

test('Check property of element', async t => {
    const developerNameInput = Selector('#developer-name');

    await t
        .expect(developerNameInput.value).eql('', 'input is empty')
        .typeText(developerNameInput, 'Peter Parker')
        .expect(developerNameInput.value).constains('Peter', 'input contains text "Peter"');
});

Result:
> testcafe chrome test.js

 Running tests in:
 - Chrome 55.0.2883 / Windows 10 0.0.0

 Example page
 × Check property of element

   1) TypeError: t.expect(...).eql(...).typeText(...).expect(...).constains is not a function

      Browser: Chrome 55.0.2883 / Windows 10 0.0.0

          7 |    const developerNameInput = Selector('#developer-name');
          8 |
          9 |    await t
         10 |        .expect(developerNameInput.value).eql('', 'input is empty')
         11 |        .typeText(developerNameInput, 'Peter Parker')
       > 12 |        .expect(developerNameInput.value).constains('Peter', 'input contains text "Peter"');
         13 |});

         at <anonymous> (E:\Repositorys\testcafe\test.js:12:43)
         at step (E:\Repositorys\testcafe\test.js:1:545)
         at <anonymous> (E:\Repositorys\testcafe\test.js:1:785)
         at <anonymous> (E:\Repositorys\testcafe\test.js:1:446)
         at test (E:\Repositorys\testcafe\test.js:6:1)

 1/1 failed (0s)
npm ERR! Test failed.  See above for more details.

Specify your

operating system: Windows 10
testcafe version: 0.11.1
node.js version: 7.4.0

Hi @StarpTech,
Thanks for noticing that. It's a typo in the constains word, it should be contains. I've fixed it.

Oh,  thanks it was very late :)
Am 19.01.2017 05:02 schrieb "Alexander Moskovkin"<EMAIL_ADDRESS>
:

Hi @StarpTech https://github.com/StarpTech,
Thanks for noticing that. It's a typo in the constains word, it should be
contains
https://devexpress.github.io/testcafe/documentation/test-api/assertions/assertion-api.html#contains.
I've fixed it.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/DevExpress/testcafe/issues/1149#issuecomment-273676495,
or mute the thread
https://github.com/notifications/unsubscribe-auth/ABrsSPoTWqueH0ew7XQw-aBWn4hRT0bkks5rTuA8gaJpZM4LnhqQ
.


Proposal to update TIKE content

This PR documents a proposal to revamp the markdown content a TIKE user will see once their JupyterHub instance has been started.
New introduction page
Most notably, this PR proposes a short-but-sweet introduction page (Introduction.md) which makes it easier to discover the key information a user needs to get started. This new introduction currently looks as follows:

Further work
I envisage further improvements (either via this PR or new ones) to include the following items:

Adding documentation on the default Python environments (i.e. kernels) provided by TIKE.
Adding tutorials specific to leveraging the TESS data available in S3.  In particular, documenting the use of tesscut in the cloud (cf. https://github.com/spacetelescope/astrocut/pull/44).
Updating the list of pre-installed software packages (cf. software-installed.md).
Reviewing the guidance we want to provide users on how long they may expect a server instance and/or data to persist (cf. caution message at the bottom of Introduction.md).
Figuring out how to best integrate tike_content with the relevant TESS and Kepler science tutorials that we created last summer in spacetelescope/notebooks.  These notebooks do not currently appear to be pulled into TIKE.

Note: this PR adds five SVG icons under images/icons.  These icons were copied from https://github.com/google/material-design-icons under the Apache License 2.0.  I included a file called images/icons/LICENSE to document this.

This is awesome!   One comment:  spacetelescope/notebooks is checked out as spacetelescope-notebooks in the Tike docker image.   I don't recall my rationale for the name change,   possibly just the opinion that "notebooks" is too general,  I'll submit a PR to change it to "notebooks".

@jaytmiller Ooops, my bad, I simply didn't spot spacetelescope-notebooks in the home directory!  I think keeping the name is fine; calling it notebooks does risk being too generic/vague.
I will go ahead and modify this PR to have tike_content take advantage of the local presence of spacetelescope-notebooks.

Really nice work!


Add pipeline bridges API

Add support to get pipeline bridge jobs documented in https://docs.gitlab.com/ee/api/jobs.html#list-pipeline-bridges

Rubocop failure seems unrelated to the changes in this PR.

Thank you.


Document interfaces

There is currently no documentation on interfaces like DataColumn. It makes it hard to add custom implementations for them. For example, it is not clear what ndistinct means.

0.8-rc still has a lot of problems with documentation. For example it is not clear how generic is used here: https://github.com/Kotlin/dataframe/blob/db20e697fa40af068a4cf7155e7086349f12d14a/src/main/kotlin/org/jetbrains/kotlinx/dataframe/DataFrame.kt#L22

@altavir I've added KDocs for major interfaces, please, check if it's clearer now.

A little bit. But not fully. For example, it is not clear what variance of T is allowed in the schema. Should it be variant T or all descendants of T (for example if T is Number).
The same goes for methods. Some methods are self-explanatory, but some are not. Several examples (just in this class, not going anywhere else):

https://github.com/Kotlin/dataframe/blob/50dadec68dec40143508fa46e5f2f4efcffdb2e1/src/main/kotlin/org/jetbrains/kotlinx/dataframe/DataFrame.kt#L47
What does it do?

https://github.com/Kotlin/dataframe/blob/50dadec68dec40143508fa46e5f2f4efcffdb2e1/src/main/kotlin/org/jetbrains/kotlinx/dataframe/DataFrame.kt#L43
what is the iteration order?

Thank you for quick reply. I'm currently writing KDocs and comments for the library. Probably I'll not be able cover the whole codebase, so your requests for clarification of particular parts of code are appreciated.


Adds currency conversion - fixes a few bugs I was having

Was having an issue with Stripe's currency conversion and clients paying in a currency that Blesta isn't setup for so this fixes it so that the non-exchanged payment amount/currency is used for Blesta (that the amount/currency that the invoice was generated in). If I didn't make this change Blesta credits the client with the exchanged currency amount in the exchanged currency and doesn't apply it to invoices.
Default to using the email provided by Stripe (and fallback to the id), so the Blesta client profile transactions table extend off the screen with the long reference ids and that hides the date and edit button in Blesta.
I've also updated the stripe-php dependency to the latest version, I have checked and no breaking code was used between the versions so no code needed to be changed to support the latest version, it is just very good to use the latest version where possible.

I have been using these changes for a week without issue so thought I'd contribute back the changes and share them with everyone.

I am not entirely sure about the ref id overflow, by any chance you have a screenshot of that?
Email is not a unique id to ref an transaction so I am kinda hesitate to do that.

Here's a screenshot of the overflow and what it looks like with my change.
You can also see how the Blesta PayPal gateway does it too, I was using them previously - they use email address.
I understand email not being as unique but the transaction ID is already provided (That's what the "pi_" number is), which is what Stripe use to identify payments, you can see that by going to your Payments view in the Stripe dashboard.
I actually have a better idea:
Use the customer ID that Stripe provides for someone(they're a number that starts with gcus_ or cus_), they're short enough so they don't overflow and it's unique, plus Stripe provides a filter for customer ID in "More filters" where you can enter the ID to see all payments from that person.

At least, I don't think the checkout session ID should be used, Stripe doesn't normally provide that(You have to dig around in the events and logs section to find it), they're very long, and Stripe doesn't provide a filter for it which makes them hard to use.

I actually have a better idea:
Use the customer ID that Stripe provides for someone(they're a number that starts with gcus_ or cus_), they're short enough so they don't overflow and it's unique, plus Stripe provides a filter for customer ID in "More filters" where you can enter the ID to see all payments from that person.

Actually lets separate this into a different PR for now --- The currency fix and the SDK update seems to be fine.
For SDK update I believe the vendor folder also needs to be updated.


JSX & SCSS color adjustments

According to this issue, we can highlight React components in JSX with different colors, like in VS Code. JSX is XML from IDEA's point of view. So I make this small adjustment.
I also fixed SCSS comments color, so it inherits from CSS comments now.
P.S. I took colors from Microsoft's VS Code Toolkit.

PhpStorm before
PhpStorm after
VS Code

Thank you for submitting a PR.
I wonder if it would have any impact on current XML highlighting? Perhaps we should apply this highlighting only if the fileType is JSX? Thoughts.

I think it may affects only "Custom tag" in XML. This comment says: "For JSX you need to tweak XML settings. Unfortunately, historically React uses XML settings, and we cannot change the behavior for the new option, to keep the consistency.".
So, I don't see other way to highlight components in JSX/TSX. And I don't know, how we can enable this XML settings only if fileType is jsx/tsx.

Any updates?

@amstiel - Sorry for the delay. I haven't had a chance to look into this. I want to incorporate this without breaking the XML highlighting. I'll see what I can do over this weekend.

So, what do you think? Can you find a solution without breaking the XML highlighting? (I think, a possible change of "custom tag" highlighting is not an error, though)

Sorry took a while. It's been a crazy town here.
Can you also create the same keys in src/main/resources/themes/vscode_dark_brighter.xml but just have the color# as 47ccb1 rather than 4ec9b0

Jetbrains uses the swing components, for some reason the colors show a pinch duller than VScode. So I have been manually adjusting those colors. In the future, I am planning to automate this process but for the time being, we have to manually keep vscode_dark.xml and vscode_dark_brighter.xml in sync.

Sorry took a while. It's been a crazy town here.
Can you also create the same keys in src/main/resources/themes/vscode_dark_brighter.xml but just have the color# as 47ccb1 rather than 4ec9b0

Jetbrains uses the swing components, for some reason the colors show a pinch duller than VScode. So I have been manually adjusting those colors. In the future, I am planning to automate this process but for the time being, we have to manually keep vscode_dark.xml and vscode_dark_brighter.xml in sync.

Done ✔


[ARRSOF-76] Documentation added

This pull request is being overridden by https://github.com/marinvvasilev/appc.sqlite3/pull/13. Since the latter changes fewer base project files.


ocamlPackages.odate: init at 0.6

Motivation for this change
Simple date and duration manipulation in OCaml: https://github.com/hhugo/odate
Things done

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

[x] NixOS
[ ] macOS
[ ] other Linux distributions

[ ] Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
[x] Tested compilation of all pkgs that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review wip"
[ ] Tested execution of all binary files (usually in ./result/bin/)
[ ] Determined the impact on package closure size (by running nix path-info -S before and after)
[ ] Ensured that relevant documentation is up to date
[x] Fits CONTRIBUTING.md.

Result of nixpkgs-review pr 105373 1

1 package built:

ocamlPackages.odate

Tests are broken: https://github.com/hhugo/odate/issues/7


UI Issue: Profile Icon should be middle of the screen in mobile device

Description
UI Issue: Profile Icon should be on middle of the screen in mobile device...Can you assign this issue to me as GSSOC23 contributor?

Screenshots
No response
Additional information
No response

Could you optimize the mentor form?

yeah, sure I can do... @anmode

great!

you can continue that commit... of reverting that line in this pR.

you can continue that commit... of reverting that line in the pR of this issue!

Are you talking about that line gap:25px in css which I mentioned you on discord?

yes yes!. I'll change the issue to mentor form improvisation.

one thing.. i noticed there is issue. https://github.com/anmode/grabtern-frontend/issues/48.

I'll close this issue! R u willing to contribute to this? I'll assign you then https://github.com/anmode/grabtern-frontend/issues/75

I'll close this issue! R u willing to contribute to this? I'll assign you then #75

ok then, you can close this issue...


Array validation issue in query

Hi,
I ran into an issue upgrading swagger-node-runner to the latest version and found an issue in your params parser (more specially in the convertValue of the helpers file).
I use an API like this:
/api/path?ids=111111111111111111111111

In my swagger definition ids is an array of string (the given id in my test is the simplest mongodb ObjectId possible which is 24 hexadecimal chars)
In your code you are doing this to know if the given parameter is a valid array:
  // Attempt to parse the string as JSON if the type is array or object
  if (['array', 'object'].indexOf(type) > -1) {
    try {
      value = JSON.parse(value);
    } catch (err) {
      // Nothing to do here, just fall through
    }
  }

Which is bad in my case because JSON.parse("111111111111111111111111") does not fail but respond<PHONE_NUMBER>11111111111111
The rest of the code fails because the value has changed from a string to a Number and the API respond a Validation Error despite the fact that the parameter is good.
Best regards,
David

This is already fixed on master: 94ba34fe8008e9febdff2ff1ee48027ac95a881f


KOGITO-4671 : New elements should always be connected by its central magnetic point

Hi @romartin can you review this PR? Artifact can be found here. https://drive.google.com/drive/folders/1tjkhXfTHDic8Ef4wiLLmx3pgzK70RIiQ?usp=sharing

jenkins run fdb

Yea @inodeman please rebase to get the FDB check :)

@domhanak artifacts generated, ready to test once possible :) Thanks in advance!

@inodeman @romartin LGTM, could you please report this issue you mention in comments:
Current implemenetation is set for gateays to have a fixed magenet from source and to target, if changed to center it actually does not look right as there is a space gap, so leaving new nodes to be connected to center, except Gateways.
I think that this is a bug and should be fixed in the future & as this fix is also useful for gateways.

@inodeman The test failed in FDB is unstable currently and we are investigating so good to merge anyway.

@inodeman @romartin https://github.com/kiegroup/kogito-editors-java/pull/18#issuecomment-845147218 see my question please.

Hi @domhanak say your comment, so you mean that in the future all nodes should be connected from the center even gateways after the gap is fixed? Let me know

Hey @domhanak  @inodeman
Not really why you talk about some issue guys sorry, in the comment https://github.com/kiegroup/kogito-editors-java/pull/18#issuecomment-845147218 I see no issue, nothing to be reported. Consider the behavior for magnets in gateways has not being changed, we still have the same behavior as before this work. This work applies to all other nodes, but not for gateways. In fact we are using fixed magnets for gateways intentionally, it was a requirement being asked by Kris long time ago, and this work stays with same approach. Does it makes sense?
Sooo can I go ahead with merging guys?

@inodeman @romartin I am talking about this GAP that is emerging see the picture:

I think that it is a bug, is this reported somewhere? I remember this was discussed some time ago and I can't find any JIRA :(

Hi @domhanak say your comment, so you mean that in the future all nodes should be connected from the center even gateways after the gap is fixed? Let me know

Yes that is what we  eventually want anyway for our editors. I am curious about the GAP issues, is it reported? Could you please confirm?

Found it, I was looking in KOGITO jira not JBPM :confused: GAP issue reported here -  https://issues.redhat.com/browse/KOGITO-5210
Let's merge.

The test failure in FDB of VSCode is unrelated.


Can using different autoencoder architectures help improve results?

If I am not getting the desired results with my dataset, will modifying the architecture of the autoencoder help?   Do I have to keep the embedding layer below a certain size?

@vivektreddy You can choose a better autoencoder structure for your own dataset. The dimension of the embedding layer (10 in this code) is not necessarily the best choice either.


DateTimePicker

Why you release a datePicker, a timePicker but not a dateTimePicker?

:+1: by this idea

Because this makes for more flexible use of the library? It's easy enough to combine both...

Read the guidelines before opening an issue please: https://github.com/amsul/pickadate.js/blob/master/CONTRIBUTING.md
This request has already come up several times.


Special Ops Insignia - Should it hinder people who draw 20-ish cards per turn? (thanks to Discord's Brickman #9899 for pointing this one out)

Options:

Only triggers once per turn; <- Brickman
Option to let it draw or not;

Does someone NEEDS this triggering more than once?

No, it shouldn't. So, the power should work only once per turn.


Create Head component

A Head component that contains all related <head /> content.

https://github.com/zenorocha/react-email/commit/555ccd1f96c2bd8a6b6c84e11caeabeef1874fdd


Managed expression ID in evolution objects

Changes

WolframModelEvolutionObject version is changed 2 -> 3.
It now stores the managed expression ID in the evolution object.
TODO: Changes the documentation to use the new versions for the evolution objects.

Comments

This results in the evolution objects using more memory as the Set instances in libSetReplace are no longer immediately deallocated.
However, it enables developing features which require interaction between the C++ and the WL code such as #436.
It allows for the properties to be implemented in C++, which is especially useful for #443 which requires low-level GPU implementation.

Examples

Check the new values in the evolution objects:

In[] := KeyTake[WolframModel[{{1, 2}} -> {{1, 3}, {1, 3}, {3, 2}}, {{1, 1}}, 4][[1]],
                {"Version", "LibSetReplaceSet"}]
Out[] = <|"Version" -> 3, "LibSetReplaceSet" -> LibSetReplaceSet[1]|>

This change is

Closing this in favor of the future design from Project Yellowstone.

Closing this in favor of the future design from Project Yellowstone.


Add package extension for Makie

This PR creates a package extension for the additional functionality we implemented that requires Makie.jl to be loaded. To remain backwards compatible with Julia v1.8, I've kept the current approach that is based on Requires.jl.
Package extensions were added in Julia v1.9. However, in the original implementation they are not triggered if the dependency is loaded from a different environment than the currently active one. This was remedied only in https://github.com/JuliaLang/julia/pull/49701 (and will end up in Julia v1.9.1), thus I created the version guard such that the referenced PR is included.
When an appropriate, new Julia version is used, the package extension is loaded instead of relying on Requires.jl. The only user-visible difference is that we need to define and export the functions iplot and iplot! already in Trixi.jl, since the package extension cannot export anything itself. This is somewhat against our usual policy and (and against POLA), since it means we have an exported function that has no usable implementation unless Makie.jl is loaded. @kristofferc suggested a potential remedy on Slack:

For the plotting case why not just export whatever name of the plotting function you want in the parent package, put the code for plotting in the extension and then call that code from the parent plotting function based on checking Base.get_extension when it is called. If the extension is not loaded, give some informative error message?

However, for now I do not think this is necessary, as it creates new headaches (e.g., it splits the API definition between two places). Thus I suggest to do it this way for now and gather some real-world experience before using a more complicated implementation.

For reference: We assume the fix for package extensions will be available in Julia v1.9.1 since it's included in the corresponding backports PR https://github.com/JuliaLang/julia/pull/49680

Did you test this with

some older version of Julia using the Requires.jl branch
some development version of Trixi.jl using the package extensions branch
?

Did you test this with
* some older version of Julia using the Requires.jl branch

* some development version of Trixi.jl using the package extensions branch

?

With my original implementation, I tried Julia v1.9.0 and v1.10.0-DEV.1288. They both worked and showed the expected behavior. For v1.8, I thought to let CI sort it out whether it works or not

If Requires is only used for this feature (I didn't check) it could also be conditionally loaded.

If Requires is only used for this feature (I didn't check) it could also be conditionally loaded.

No, it's still used for Plots.jl and some other stuff, but we'll eventually migrate everything. This is just to get our feet wet and to resolve the precompilation issue you mentioned in #ttfx

However, for now I do not think this is necessary, as it creates new headaches (e.g., it splits the API definition between two places).

While true, I think a user is more happy to get an informative error message over
julia> iplot(rand(10))
ERROR: MethodError: no method matching iplot(::Vector{Float64})

While true, I think a user is more happy to get an informative error message

I completely agree. But how would you implement that without having to duplicate the function signature from the extension inside the main package? And adding a catchall iplot(args...; kwargs...) in the main package that throws an error also seems like a non optimal (and potentially confusing) solution.

@ranocha I added the ext directory to the list of folders that get processed for coverage. I didn't find anything else in our lives various workflow files, but could you please have a look as well that I didn't miss anything CI related?

I think it should be fine but we can never be sure as long as the Linux MPI tests keep failing. I created https://github.com/trixi-framework/Trixi.jl/pull/1497 to mitigate this a bit. If you agree, we could first consider merging that PR to see whether everything is fine with coverage etc. here

If you agree, we could first consider merging that PR to see whether everything is fine with coverage etc. here

Sounds good!

Coverage looks good. Thanks, @sloede


accouting subquery's cost into total cost  #72

Thanks, merged!

Thanks for your advises. And I am not very familiar with details of git, so I have made some very low-level mistake.  Thank you again.


Fix scaling overlap + add install script and partial send

closes #1
Install script will

Install BepInEx
Set the correct config values (hide manager)
Download and move KeyboardOSC


Skip docker registry login by default, login only when pulp is deployed

LGTM, but I assume it's untested?

Yes it's untested

Have now tested, it works.


VibrantLinux Packaging Request

Project description
vibrantLinux is a GUI tool to serves the same purpose as VibranceGUI for Windows, which to modify the saturation of any given screen whenever certain programs are launched/are in focus.
Metadata

homepage URL: https://github.com/libvibrant/vibrantLinux
source URL: https://github.com/libvibrant/vibrantLinux
license: MIT
platforms: Linux Only

Bump. I have some low vision issues and "vibrance" makes it much easier to read the screen. Thanks!


New x86_64 GCC shards with fixed AVX512 intrinsics

We backported the patch for a GCC bug:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=87467.  For reference, see also
https://github.com/JuliaPackaging/Yggdrasil/issues/2389.
Companion PR in Yggdrasil: https://github.com/JuliaPackaging/Yggdrasil/pull/2415.  CC: @omus

[ Info: Beginning full shard test... (this can take a while)
 Downloading artifact: GCCBootstrap-x86_64-linux-musl.v5.2.0.x86_64-linux-musl.unpacked
25l
signal (6): Aborted
in expression starting at /home/runner/work/BinaryBuilderBase.jl/BinaryBuilderBase.jl/test/runners.jl:30

very good

[ Info: Beginning full shard test... (this can take a while)
 Downloading artifact: GCCBootstrap-x86_64-linux-musl.v5.2.0.x86_64-linux-musl.unpacked
25l
signal (6): Aborted
in expression starting at /home/runner/work/BinaryBuilderBase.jl/BinaryBuilderBase.jl/test/runners.jl:30

very good

very good

I can't tell if you're being serious 😆

very good

I can't tell if you're being serious 😆

No, I'm joking 😞 the crash doesn't even happen on one of the new compilers...

No, I'm joking 😞 the crash doesn't even happen on one of the new compilers...

Ok, CI is now happy.  I tried this locally
% julia-master -e 'using BinaryBuilderBase; BinaryBuilderBase.runshell(Platform("x86_64", "linux");preferred_gcc_version=v"7", verbose=false)'
sandbox:${WORKSPACE} # qfind /opt/ -name avx512fintrin.h -exec grep -H _mm512_abs_pd '{}' \;
/opt/x86_64-linux-gnu/lib/gcc/x86_64-linux-gnu/7.1.0/include/avx512fintrin.h:_mm512_abs_pd (__m512d __A)
/opt/x86_64-linux-musl/lib/gcc/x86_64-linux-musl/7.1.0/include/avx512fintrin.h:_mm512_abs_pd (__m512d __A)
/opt/x86_64-linux-musl/lib/clang/9.0.1/include/avx512fintrin.h:_mm512_abs_pd(__m512d __A)
sandbox:${WORKSPACE} # qfind /opt/ -name avx512fintrin.h -exec grep -H _mm512_mask_abs_pd '{}' \;
/opt/x86_64-linux-gnu/lib/gcc/x86_64-linux-gnu/7.1.0/include/avx512fintrin.h:_mm512_mask_abs_pd (__m512d __W, __mmask8 __U, __m512d __A)
/opt/x86_64-linux-musl/lib/gcc/x86_64-linux-musl/7.1.0/include/avx512fintrin.h:_mm512_mask_abs_pd (__m512d __W, __mmask8 __U, __m512d __A)
/opt/x86_64-linux-musl/lib/clang/9.0.1/include/avx512fintrin.h:_mm512_mask_abs_pd(__m512d __W, __mmask8 __K, __m512d __A)

The signatures seems correct (all using __m512d and not __m512).  By inference I conclude all other shards are fine as well

Ok, CI is now happy.  I tried this locally
% julia-master -e 'using BinaryBuilderBase; BinaryBuilderBase.runshell(Platform("x86_64", "linux");preferred_gcc_version=v"7", verbose=false)'
sandbox:${WORKSPACE} # qfind /opt/ -name avx512fintrin.h -exec grep -H _mm512_abs_pd '{}' \;
/opt/x86_64-linux-gnu/lib/gcc/x86_64-linux-gnu/7.1.0/include/avx512fintrin.h:_mm512_abs_pd (__m512d __A)
/opt/x86_64-linux-musl/lib/gcc/x86_64-linux-musl/7.1.0/include/avx512fintrin.h:_mm512_abs_pd (__m512d __A)
/opt/x86_64-linux-musl/lib/clang/9.0.1/include/avx512fintrin.h:_mm512_abs_pd(__m512d __A)
sandbox:${WORKSPACE} # qfind /opt/ -name avx512fintrin.h -exec grep -H _mm512_mask_abs_pd '{}' \;
/opt/x86_64-linux-gnu/lib/gcc/x86_64-linux-gnu/7.1.0/include/avx512fintrin.h:_mm512_mask_abs_pd (__m512d __W, __mmask8 __U, __m512d __A)
/opt/x86_64-linux-musl/lib/gcc/x86_64-linux-musl/7.1.0/include/avx512fintrin.h:_mm512_mask_abs_pd (__m512d __W, __mmask8 __U, __m512d __A)
/opt/x86_64-linux-musl/lib/clang/9.0.1/include/avx512fintrin.h:_mm512_mask_abs_pd(__m512d __W, __mmask8 __K, __m512d __A)

The signatures seems correct (all using __m512d and not __m512).  By inference I conclude all other shards are fine as well


Added documentation about testing with the explicit loop

What do these changes do?
Updates on documentation about how to test by passing the loop explicitly and why you should do that.
Related issue number
#1069

Current coverage is 97.71% (diff: 100%)

Merging #1097 into master will not change coverage

@@             master      #1097   diff @@
==========================================
  Files            28         28
  Lines          6434       6434
  Methods           0          0
  Messages          0          0
  Branches       1094       1094
==========================================
  Hits           6287       6287
  Misses           77         77
  Partials         70         70

Powered by Codecov. Last update 4599080...abc767e

Thanks!
Feel free for adding very explained texts about aiohttp problems and best usage practices like this one.

Sure! :)


Dataset for saving a sklearn pipeline with a tensorflow estimator

Description
A scikit learn pipeline where the last step is an instance of KerasRegressor cannot be saved directly as a pickle dataset. To save it using the current datasets would require two datasets: one for the scikit learn pipeline without the tensorflow model, and another for the tensorflow model. However, managing this model in this way would be cumbersome because to make an inference say, I would need to load both the scikit learn pipeline, and the tf model, put them together, and predict.
Context
When working with tensorflow models, users might want to add them to a scikit-learn pipeline.
Possible Implementation
(Optional) Suggest an idea for implementing the addition or change.
Possible Alternatives
(Optional) Describe any alternative solutions or features you've considered.

Hi @juan-carlos-calvo, just wondering if you had any updates on this? Do you think we still need the PR for a new dataset?

I ran a test with the simple example in the docstring of the dataset implementation (using the latest tensorflow) and it throws the same error TypeError: cannot pickle 'weakref' object

Hmm, that's weird - I just tried this out  and it works perfectly for me. Take a look at my environment and the example project in the blah directory. kedro run seems to work ok, as does loading up the datasets in kedro ipython. What am I doing differently here? 🤔

seems you are absolutely right!
not sure if it was because I tried the version just before 2.7 but anyways, it works! thank you very much for taking the time to verify this


ignore/only{Fs} should accept arrays

getWorkspaces({
  cwd,
  ignoreFs: [
    'packages/css-packs/*',
    'build/*',
    'website'
  ]
});

https://github.com/boltpkg/bolt/blob/b2f52a25311df2152794002f1d8792fadebe8360/src/functions/getWorkspaces.js#L7-L10
via @jamiebuilds

Brilliant idea

Can I work on this?

@padmaia @jamiebuilds
Nice Idea, What are the thoughts on it accepting string | Array<string> ?


Link break: enforce the .NET coding conventions on build

There is a link named "enforce the .NET coding conventions on build" in "Enforce code styles on build" section, but after click I got 404 error.

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

ID: faebd126-53cd-1a98-bf75-38225933b0f2
Version Independent ID: 4340fcba-6779-68d2-6c51-2ae37123dc26
Content: Code style options and code cleanup - Visual Studio
Content Source: docs/ide/code-styles-and-code-cleanup.md
Product: visual-studio-windows
Technology: vs-ide-general
GitHub Login: @TerryGLee
Microsoft Alias: tglee

Note that the same broken link is also on the page https://docs.microsoft.com/en-us/visualstudio/ide/editorconfig-code-style-settings-reference?view=vs-2019

Seems like this was fixed by https://github.com/MicrosoftDocs/visualstudio-docs/commit/1ff41bd9513b5d2f3d0a006d95f40a7bfb481757

@cremor -- thank you for the update.
@mavasani -- thank you for fixing this.


Add support for environment map transformation

Objective

Fixes: https://github.com/bevyengine/bevy/issues/14036

Solution

Add a world space transformation for the environment sample direction.

Testing

I have tested the newly added transform field using the newly added rotate_environment_map example.

https://github.com/user-attachments/assets/2de77c65-14bc-48ee-b76a-fb4e9782dbdb
Migration Guide

Since we have added a new filed to the EnvironmentMapLight struct, users will need to include ..default() or some rotation value in their initialization code.

I think the rotation is backward compared to the other skybox rotation PR:
This is what I get when the skybox & env maps share the same rotation (notice we see what's in front of us in the reflection instead of what's behind)
https://github.com/user-attachments/assets/634a2194-f327-4612-b75f-00396be6c4b4

I think the rotation is backward compared to the other skybox rotation PR:
This is what I get when the skybox & env maps share the same rotation (notice we see what's in front of us in the reflection instead of what's behind)
1721060695.mp4

Yes, I have reproduced this bug using a debug texture, which more clearly illustrates this issue:

The result indicates that the skybox and the environment map are rotated in opposite directions:

I found some clues here: "Cube maps are left-handed so we negate the z coordinate" (https://www.khronos.org/opengl/wiki/Cubemap_Texture)
@fragment
fn skybox_fragment(in: VertexOutput) -> @location(0) vec4<f32> {
    let ray_direction = coords_to_ray_direction(in.position.xy, view.viewport);

    // Cube maps are left-handed so we negate the z coordinate.
    let out = textureSample(skybox, skybox_sampler, ray_direction * vec3(1.0, 1.0, -1.0));
    return vec4(out.rgb * uniforms.brightness, out.a);
}

For the skybox cubemap, we rotate the ray in a right-handed view space coordinate, while for the environment map, we rotate it in a right-handed world space coordinate. However, there appears to be a difference between these two types of cubemaps. I will do some research to figure it out.

I've identified the issue: In the environment map, I initially inverted the z before transforming, which placed the transformation in a left-handed coordinate system. I should transform first, like with the skybox, and then invert the z. 😄

Great catch!

There appears to be some binding errors for the alien_cake_addict example, let me check where the problem is.

@alice-i-cecile @Azorlogh I apologize for the mistake that led to the mismatch of the mesh view bindings. I should have conducted more thorough testing before submiting these changes. I have now resolved all the issues and tested numerous examples. Below is a summary of the code changes.

The EnvironmentMapUniform should be configured as a per-view buffer, therefore I have modified the uniform to use a dynamic offset. We also need to include the ViewEnvironmentMapUniformOffset in all the associated ViewNodes and RenderCommands.
uniform_buffer::<EnvironmentMapUniform>(true).visibility(ShaderStages::FRAGMENT)

Some of the camera may not contain a EnvironmentMapLight, so we need to fallback to a default binding for the EnvironmentMapUniform
fn gather_environment_map_uniform(
    view_query: Extract<Query<(Entity, Option<&EnvironmentMapLight>), With<Camera3d>>>,
    mut commands: Commands,
) {
    for (view_entity, environment_map_light) in view_query.iter() {
        let environment_map_uniform = if let Some(environment_map_light) = environment_map_light {
            EnvironmentMapUniform {
                transform: Transform::from_rotation(environment_map_light.rotation)
                    .compute_matrix()
                    .inverse(),
            }
        } else {
            EnvironmentMapUniform::default()
        };
        commands
            .get_or_spawn(view_entity)
            .insert(environment_map_uniform);
    }
}

Changing the binding number in mesh_view_bindings.wgsl and mesh_view_bindings.rs is insufficient; the tone mapping LUT bindings also required adjustment. Since these numbers are injected through shader definitions, they are easily overloooked. This is the root cause of this issue.
#define_import_path bevy_core_pipeline::tonemapping_lut_bindings

@group(0) @binding(#TONEMAPPING_LUT_TEXTURE_BINDING_INDEX) var dt_lut_texture: texture_3d<f32>;
@group(0) @binding(#TONEMAPPING_LUT_SAMPLER_BINDING_INDEX) var dt_lut_sampler: sampler;

diff --git a/crates/bevy_pbr/src/render/mesh.rs b/crates/bevy_pbr/src/render/mesh.rs
index 8432cbd97..282b116e9 100644
--- a/crates/bevy_pbr/src/render/mesh.rs
+++ b/crates/bevy_pbr/src/render/mesh.rs
@@ -1765,11 +1765,11 @@ impl SpecializedMeshPipeline for MeshPipeline {
             shader_defs.push("TONEMAP_IN_SHADER".into());
             shader_defs.push(ShaderDefVal::UInt(
                 "TONEMAPPING_LUT_TEXTURE_BINDING_INDEX".into(),
-                20,
+                21,
             ));
             shader_defs.push(ShaderDefVal::UInt(
                 "TONEMAPPING_LUT_SAMPLER_BINDING_INDEX".into(),
-                21,
+                22,
             ));


Samsung - apps cannot be added to "Apps that won't be put to sleep" list

We've had a user report that some apps cannot be added to the "Apps that won't be put to sleep" list.
Samsung support reportedly said that "the developer of the app should implement Samsung's device care (battery management) features in order to appear among the applications which the user can put on the "Apps that won't put to sleep" list."
I've googled and googled but I couldn't find any relevant information. Does anyone have any idea what would they mean by that?
What I found (and think is relevant but not exactly it):

https://docs.samsungknox.com/devref/knox-sdk/reference/com/samsung/android/knox/application/ApplicationPolicy.html#addPackageToBatteryOptimizationWhiteList(com.samsung.android.knox.AppIdentity)
https://developer.samsungknox.com/forum-topic/app-foreground-service-being-put-sleep-samsung-android-90-device

I've asked on the Samsung developer forum
https://forum.developer.samsung.com/t/apps-that-wont-be-put-to-sleep-list-dont-include-all-apps/7412

Hey @Artaud ,
Did you have any solution for this one?
I also can't see my app in the list.

On my Samsung A33 with Android 12 I can only add apps to this list if battery optimization for this app is switched on (setting “optimized“ or “restricted”). If I set it to “unrestricted“, then it disappears from all “put to sleep“ lists at once.
See also: https://eu.community.samsung.com/t5/galaxy-a-series/samsung-a52-android-12-battery-optimization-settings-sleeping/td-p/4628416


telosome and shelterin: swap main name and synonym

Pubmed search for shelterin = 427 entries
Pubmed search for telosome = 55 entries
Shelterin used in vertebrates
Telosome used in yeast (mainly? others too likely?!)
currently these terms exist on QuickGO
Component 	GO:0070187 	telosome
Process 	GO:0071573 	telosome assembly
Process 	GO:1904790 	regulation of telosome assembly
Process 	GO:1904791 	negative regulation of telosome assembly
Process 	GO:1904792 	positive regulation of telosome assembly

I think I got them all.
[Term]
id: GO:0070187
-name: telosome
+name: shelterin complex
namespace: cellular_component
def: "A nuclear telomere cap complex that is formed by the association of telomeric ssDNA- and dsDNA-binding proteins with telomeric DNA, and is involved in telomere protection and recruitment of telomerase. The complex is known to contain TERF1, TERF2, POT1, RAP1, TINF2 and ACD in mammalian cells, and Pot1, Tpz1, Rap1, Rif1, Rif2 and Taz1 in Schizosaccharomyces. Taz1 and Rap1 (or their mammalian equivalents) form a dsDNA-binding subcomplex, Pot1 and Tpz1 form an ssDNA-binding subcomplex, and the two subcomplexes are bridged by Poz1, which acts as an effector molecule along with Ccq1." [GOC:expert_mf, GOC:mah, GOC:vw, PMID:18828880]
synonym: "Pot1 complex" EXACT [GOC:vw]
synonym: "Pot1-Tpz1 complex" EXACT [GOC:vw]
-synonym: "shelterin complex" EXACT [GOC:mah, GOC:vw]
+synonym: "telosome" EXACT [GOC:mah, GOC:vw]
is_a: GO:0000783 ! nuclear telomere cap complex
[Term]
@@ -423232,15 +423232,16 @@
[Term]
id: GO:0071573
-name: telosome assembly
+name: shelterin complex assembly
namespace: biological_process
-def: "The aggregation, arrangement and bonding together of a set of components to form a telosome. A telosome is a nuclear telomere cap complex that is formed by the association of telomeric ssDNA- and dsDNA-binding proteins with telomeric DNA, and is involved in telomere protection and recruitment of telomerase." [GOC:mah, GOC:vw]
+def: "The aggregation, arrangement and bonding together of a set of components to form a shelterin complex. A shelterin complex is a nuclear telomere cap complex that is formed by the association of telomeric ssDNA- and dsDNA-binding proteins with telomeric DNA, and is involved in telomere protection and recruitment of telomerase."  [GO_REF:0000058, GOC:BHF, GOC:BHF_telomere, GOC:nc, GOC:TermGenie, PMID:24270157]
+def: "Any process that modulates the frequency, rate or extent of shelterin complex assembly." [GO_REF:0000058, GOC:BHF, GOC:BHF_telomere, GOC:nc, GOC:TermGenie, PMID:24270157]
synonym: "regulation of Pot1 complex assembly" EXACT [GOC:TermGenie]
synonym: "regulation of Pot1-Tpz1 complex assembly" EXACT [GOC:TermGenie]
synonym: "regulation of shelterin complex formation" EXACT [GOC:TermGenie]
+synonym: "regulation of telosome assembly" EXACT []
intersection_of: GO:0065007 ! biological regulation
-intersection_of: regulates GO:0071573 ! telosome assembly
+intersection_of: regulates GO:0071573 ! shelterin complex assembly
created_by: nc
creation_date: 2015-11-06T16:13:03Z
[Term]
id: GO:1904791
-name: negative regulation of telosome assembly
+name: negative regulation of shelterin complex assembly
namespace: biological_process
-def: "Any process that stops, prevents or reduces the frequency, rate or extent of telosome assembly." [GO_REF:0000058, GOC:BHF, GOC:BHF_telomere, GOC:nc, GOC:TermGenie, PMID:24270157]
+def: "Any process that stops, prevents or reduces the frequency, rate or extent of shelterin complex assembly." [GO_REF:0000058, GOC:BHF, GOC:BHF_telomere, GOC:nc, GOC:TermGenie, PMID:24270157]
synonym: "down regulation of Pot1 complex assembly" EXACT [GOC:TermGenie]
synonym: "down regulation of Pot1-Tpz1 complex assembly" EXACT [GOC:TermGenie]
synonym: "down regulation of shelterin complex formation" EXACT [GOC:TermGenie]
@@ -561768,16 +561770,17 @@
synonym: "negative regulation of Pot1 complex assembly" EXACT [GOC:TermGenie]
synonym: "negative regulation of Pot1-Tpz1 complex assembly" EXACT [GOC:TermGenie]
synonym: "negative regulation of shelterin complex formation" EXACT [GOC:TermGenie]
+synonym: "negative regulation of telosome assembly" EXACT []
intersection_of: GO:0065007 ! biological regulation
-intersection_of: negatively_regulates GO:0071573 ! telosome assembly
+intersection_of: negatively_regulates GO:0071573 ! shelterin complex assembly
created_by: nc
creation_date: 2015-11-06T16:13:11Z
[Term]
id: GO:1904792
-name: positive regulation of telosome assembly
+name: positive regulation of shelterin complex assembly
namespace: biological_process
-def: "Any process that activates or increases the frequency, rate or extent of telosome assembly." [GO_REF:0000058, GOC:BHF, GOC:BHF_telomere, GOC:nc, GOC:TermGenie, PMID:24270157]
+def: "Any process that activates or increases the frequency, rate or extent of shelterin complex assembly." [GO_REF:0000058, GOC:BHF, GOC:BHF_telomere, GOC:nc, GOC:TermGenie, PMID:24270157]
synonym: "activation of Pot1 complex assembly" NARROW [GOC:TermGenie]
synonym: "activation of Pot1-Tpz1 complex assembly" NARROW [GOC:TermGenie]
synonym: "activation of shelterin complex formation" NARROW [GOC:TermGenie]
@@ -561785,6 +561788,7 @@
synonym: "positive regulation of Pot1 complex assembly" EXACT [GOC:TermGenie]
synonym: "positive regulation of Pot1-Tpz1 complex assembly" EXACT [GOC:TermGenie]
synonym: "positive regulation of shelterin complex formation" EXACT [GOC:TermGenie]
+synonym: "positive regulation of telosome assembly" EXACT []
synonym: "up regulation of Pot1 complex assembly" EXACT [GOC:TermGenie]
synonym: "up regulation of Pot1-Tpz1 complex assembly" EXACT [GOC:TermGenie]
synonym: "up regulation of shelterin complex formation" EXACT [GOC:TermGenie]
@@ -561798,7 +561802,7 @@
synonym: "upregulation of shelterin complex formation" EXACT [GOC:TermGenie]
synonym: "upregulation of telosome assembly" EXACT [GOC:TermGenie]
intersection_of: GO:0065007 ! biological regulation
-intersection_of: positively_regulates GO:0071573 ! telosome assembly
+intersection_of: positively_regulates GO:0071573 ! shelterin complex assembly
created_by: nc
creation_date: 2015-11-06T16:13:18Z


math support

How is math supported? I didn't find a link on your feature page https://sphinxcontrib-confluencebuilder.readthedocs.io/en/stable/features/
Our org has https://docs.adaptavist.com/cfm4cc/latest/macros/latex
is it enough to push formulas to our confluence server?

Math is supported using the math directive when using the sphinx.ext.imgmath extension (which the math extension also requires a LaTeX and dvipng or dvisvgm installation; outlined in its documentation -- this is usually covered by a TeX suite installation on a Linux system or a MiKTeX installation on a Windows system). An example of this can be seen on the Math demonstration page (additional recommendations can be found in the Tips documentation).
If you wish to take advantage of Adaptavist's LaTeX macro, you could try injecting a raw macro of the content you wish to render. An example could be something like the following:
.. raw:: confluence_storage

    <ac:structured-macro ac:name="latex-formatting" ac:schema-version="1">
        <ac:plain-text-body><![CDATA[$ax^2 + bx + c = 0$]]></ac:plain-text-body>
    </ac:structured-macro>

Although, you will have to check if the macro name is correct by inspecting an example output on a test page (i.e. manually creating a page with a LaTeX math example, selecting  "Advanced -> View storage format" of the page and copy/pasting/edit the example into the raw directive block).

Thanks for the prompt reply. Using static images is not really a solution in 2022, and if my system and server both support latex, why cannot the confluence builder support it?
By using the suggested workaround, how can I keep the original functionality of the rst, and make HTML if it is needed? It seems it breaks usability, and the rst will suit only the confluence build. Isn't it possible to define a new role, that is interpreted as math, if the builder is not confluencebuilder, but interpreted as per your suggestion, if the builder is confluencebuilder?

[If] my system and server both support latex, why cannot the confluence builder support it?

The Confluence builder extension aims to provide as much functionality it can with respect to supporting the features provided by Sphinx on a Confluence instance's stock capabilities. Adaptavist's LaTeX macro is a paid extension solution, which fits outside the scope of what the Confluence builder extension tries to support.
That being said, Confluence builder extension tries to be flexible enough to allow a user to handle things like custom macros (either internally build or third-party solutions). Using the raw directive is just one of the means where a user has the freedom to utilize any type of custom markup they wish to apply.
I can agree that it is annoying -- to want to just be able to use an example markup like the following:
.. math::

    E = mc^2

And have it "just work" on all builder types without complication. While it would be nice to use the sphinx.ext.mathjax capabilities which are used in the html builder, there is not a way to push script on a stock Confluence instance to handle this. This is why this extension defaults/forces the use of using the sphinx.ext.imgmath extension since (1) it is a capability provided by Sphinx and (2) Confluence easily supports image content on pages. While SVG generated images can look fine for a user (assuming no browser issues), there is the annoyance of not having capabilities to raw copy text and other manipulations which may be provided by a mathjax solution.

[How] can I keep the original functionality of the rst, and make HTML if it is needed?

There are ways you could make this work in your environment. While you could use both a combination of raw directives targeting Confluence and "not Confluence" format, I think we can all agree that is not an ideal solution. We would want to way where one could just define a math block and have it work on all the builders we use.
Typically, a solution like this could be achieved by just overriding the node types in a project's setup call to override how math nodes are processed (see this comment an example). However, with the existing implementation of the Confluence builder extension, there isn't a way to stop it pre-converting math nodes into image nodes before node translation processing kicks in.
A workaround would be adding the following into your documentation, where we replace math nodes to a custom LaTeX node, and then wrap the contents with an assumed-valid latex-formatting macro. Is it not pretty, but in a project's setup.py, you could add the following:
from docutils import nodes
from sphinxcontrib.confluencebuilder.builder import ConfluenceBuilder
from confluence_support import MathOverride
from confluence_support import override_math_mode

def custom_visit_latex_math(self, node):
    self.body.append('<ac:structured-macro ac:name="latex-formatting" ac:schema-version="1">')
    self.body.append('<ac:plain-text-body><![CDATA[')

def custom_depart_latex_math(self, node):
    self.body.append(']]></ac:plain-text-body>')
    self.body.append('</ac:structured-macro>')

def confluence_builder_inited(app):
    if not isinstance(app.builder, ConfluenceBuilder):
        return

    app.add_transform(MathOverride)

def setup(app):
    app.add_node(override_math_mode,
        confluence=(custom_visit_latex_math, custom_depart_latex_math),
        override=True)

    app.connect('builder-inited', confluence_builder_inited)

And create a confluence_support/__init__.py file alongside your documentation which defines:
from docutils import nodes
from sphinx.transforms import SphinxTransform
from sphinx.util.math import wrap_displaymath
import itertools

class MathOverride(SphinxTransform):
    default_priority = 99

    def apply(self, **kwargs):
        for node in itertools.chain(self.document.traverse(nodes.math),
                self.document.traverse(nodes.math_block)):

            if not isinstance(node, nodes.math):
                if node['nowrap']:
                    latex = node.astext()
                else:
                    latex = wrap_displaymath(node.astext(), None, False)
            else:
                latex = '$' + node.astext() + '$'

            new_node = override_math_mode(latex, latex, **node.attributes)
            node.replace_self(new_node)

class override_math_mode(nodes.TextElement):
    pass

(we have to create a different file primarily to hold the node definition, or Sphinx can fail when attempting to pickle/cache the node type)
While adding the above is not fun, it should be when using a combination of builders like html and confluence (although, limited testing has been done).
In the future, we can change the Confluence builder extension to possibly detect for custom overrides provided by the Sphinx configuration on the math type, and then prevent it from auto translating math blocks into image blocks. This would avoid the need to add this custom node type/transform into a configuration -- allow users to more simply override these node types or enable extensions stacked on top of the Confluence builder extension to manipulate content as they see fit.

Thanks, will try it out!

Another idea is that we could add an advanced option in this extension, which when set, will just forward Math-defined LaTeX to a custom-defined macro for a user. That will allow this extension to properly extract/parse the LaTeX content and leave it up to the user to say what macro will process it ~ then users would be free to inject this content into any type of macro definition which could support LaTeX processing (although I wonder what would be the best way to properly test this in terms of development...).

I am not familiar what a macro means in the rst world, but I am not against installing katex or mathjax into my sphinx environment to get the right HTML or confluence output. I just don't want to maintain 2 versions of my rst files, one for HTML output and one for confluence, yet I want proper math rendering. And within a decade, I'll also want proper equation numbering and references :)

Sorry, for clarification (and if interested), when I talk about macros, I am talking about the features supported by a Confluence instance. Confluence provides a series of "standard" macros, such as the "info" blocks or "code" blocks. These are typically enabled/supported on all Confluence instances, which the Confluence Builder extension for Sphinx will attempt to utilize.
The LaTeX macro you mentioned in your environment is provided by Adaptavist, and is limited to the instances which have purchased/installed the Content Formatting Macros application. The Confluence Builder extension for Sphinx focuses on the stock/default-provided macros -- since we do not want to convert a reStructuredText math directive into a Adaptavist LaTeX macro and have users who do not have this macro seeing a broken math block. For example:

This is why the existing implementation of this extension relies on
sphinx.ext.imgmath -- since Sphinx provides the extension and all Confluence instances default-support image macros.
My recent suggestion above was a possible approach to give flexibility for users who want to use an alternative LaTeX rendering for their math definitions. There looks to be several implementations of LaTeX macros available on the Confluence marketplace. So -- trying to think of a possible way (aside from just having users rely on the raw directive) to easily translate a math block into using a LaTeX rendering of their choice -- and avoiding generating/attaching images (without even getting into Math capabilities which may be supported by a Markdown format). An idea is that we could default to use sphinx.ext.imgmath for math blocks, but provide a way (for example) a user to define confluence_advanced_math_macro = '<ID>'. If this is set, we can have this extension automatically parse the math content into LaTeX (as Sphinx does with any other builder), and forward it to the macro type support on a users Confluence instance.

@danieltuzes, I took some time to try to help improve support for users wanting to integrate with LaTeX macros. Using the most recent development implementation, changes (#620) have been made to support a new configuration option confluence_latex_macro. If set, math (and other LaTeX content) can be rendered on Confluence using a LaTeX macro installed on an instance (if any).
Recently added a new math guide to the documentation. Please take a look, and if there is anything unclear from it, we can try to improve this information before we make a next release.
One final note, which I was not aware was the case with the current release, but SVG images for math will not be rendered unless sphinx.ext.imgmath was explicitly listed in the extension's configuration. Previous to v1.7, we force-loaded this extension to ensure easy SVG support; however, this would cause issues where if the Confluence builder extension was listed, other builders would be forced to use this extension as well. When correcting this in v1.7, the new requirement was not explicitly mentioned anywhere what had to be done. This information should now be reflected in the same guide referenced above.

v1.8 is now available on PyPI -- marking as closed.

Thanks, this will make this plugin more useful for companies paying for Confluence and macros.


Proxy

It lacks the ability to set up a proxy to connect.

Do you mean having a simple HTTP server to test webhook?

I began to test the robot on a computer connected to the Internet through a proxy. And the request
import telepot
bot = telepot.Bot ('***** PUT YOUR TOKEN HERE *****')
bot.getMe ()
I get the appropriate error.
I looked implementation method.
def getMe (self):
r = requests.post (self._methodurl ('getMe'), timeout = self._http_timeout)
return self._parse (r)
The simplest thing you can do is write:
def getMe (self):
r = requests.post (self._methodurl ('getMe'), timeout = self._http_timeout, proxies = urllib.request.getproxies ())
return self._parse (r)
so we work at least take those proxy settings that are registered in the system.

never needed a proxy to it, but have you tested to set your env proxy
settings?
export http_proxy=http://proxy.vort.se:8080
export https_proxy=http://username:password@proxy.incolume.com.br:8080
On Wed, Jan 27, 2016 at 7:11 AM, Garmahis666<EMAIL_ADDRESS>wrote:

I began to test the robot on a computer connected to the Internet through
a proxy. And the request
import telepot
bot = telepot.Bot ('***** PUT YOUR TOKEN HERE *****')
bot.getMe ()
I get the appropriate error.
I looked implementation method.
def getMe (self):
r = requests.post (self._methodurl ('getMe'), timeout =
self._http_timeout)
return self._parse (r)
The simplest thing you can do is write:
def getMe (self):
r = requests.post (self._methodurl ('getMe'), timeout =
self._http_timeout, proxies = urllib.request.getproxies ())
return self._parse (r)
so we work at least take those proxy settings that are registered in the
system.
—
Reply to this email directly or view it on GitHub
https://github.com/nickoala/telepot/issues/21#issuecomment-175499917.

Yes, telepot lacks the ability to set proxies on requests.post().
According to requests' documentations, you may achieve the same effect by:
$ export HTTP_PROXY="http://<IP_ADDRESS>:3128"
$ export HTTPS_PROXY="http://<IP_ADDRESS>:1080"

I may consider adding that configuration to some future release.

After a night sleeping on it, I have decided not to add proxy configuration to telepot. It's rather ad-hoc, and can be gotten around quite trivially.

I figured out the problem was really in my code. Everything works fine. Many thanks!


blns.txt in Pull Requests unexpectedly being changed to binary format w/ no diff

I have no idea why this keeps happening.
I can't auto accept any files that show this because it might corrupt readability on the web. I can try to merge the PRs; worst case, I'll add the strings manually to my own copy and close the PRs.

Ah, it appears to be my fault: I broke it at https://github.com/minimaxir/big-list-of-naughty-strings/commit/b518990120a10f0555fdad23444f1ebecbbd6968
I guess those strings were naughty indeed. I'll see if I can isolate.

Per Stack Overflow, the fix is to use a .gitattributes file. Commiting that now.
Of note:

Git will even determine that it is binary if you have one super-long line in your text file. I made a long String span several source code lines, and suddenly the file went from being 'binary' to a text file that I could see (in SmartGit).

Unfortunately this means I have to invalidate some of the pull-requests made since then.

So the actual cause was the presence of a null character (U+0000), as noted in this Stack Overflow answer:

"Why is Git marking my file as binary?" The answer is because it's seeing a NUL (0) byte somewhere within the first 8000 characters of the file.

This is fixed with https://github.com/minimaxir/big-list-of-naughty-strings/commit/62066882b590d6acdfa1c3695eb38018d900c7b2 which looks like it did the trick.


[DO-NOT-MERGE] Investigate test failures in TestLlapQueryExecutionListener

What changes were proposed in this pull request?
Investigate test failures in TestLlapQueryExecutionListener
How was this patch tested?
N/A

I can't reproduce this in my local. Will investigate this a bit first.

Oh BTW, please squashed merge with copying and pasting the PR description into commit message :-). the commit logs here are messy.

thanks a lot for tracking this down.


Upgrading vertx-utils

@sreejit123 thanks for the PR
Can you please add the changelog between the 2 versions in the PR description as a reference?


Add E2E Tests for OnPrem / Non-Storage scenarios

Once the storage free on prem work is unblocked by https://github.com/Azure/azure-webjobs-sdk/pull/1184, we need to put e2e regression tests in place for this to the e2e test suite.

I'll get this assigned

@paulbatum - something something your own job


Update README.md

small correction on configuration sample on readme

Thanks @vikkio88

was a silly thing, no worries


assess_region() and site_assess_region() functions don't seem to use cache when running a request a second time.

These functions don seem to store or don't access assessment results that have been processed already.
Running these example links a second time doesn't seem to access the results data from the initial run from cache.
<IP_ADDRESS>:8000/suitability/assess/Cairns-Cooktown/slopes?Depth=-4.0:-2.0&Slope=0.0:40.0&Rugosity=0.0:6.0 OR
<IP_ADDRESS>:8000/suitability/site-suitability/Cairns-Cooktown/slopes?Depth=-4.0:-2.0&Slope=0.0:40.0&Rugosity=0.0:6.0&SuitabilityThreshold=95&xdist=450&ydist=50

As noted in discussion, caching was not implemented hence why the results were not being saved from the initial query.
This is now implemented in #32


some problems when run dqn_agent.py

My work env:Python 3.5.2 :: Anaconda custom (64-bit),tensorflow 1.0.1,keras 2.0.4.There are some questions I have encountered.As follows:
1.File "/usr/anaconda3/lib/python3.5/site-packages/tgym/envs/init.py", line 1, in
from trading import SpreadTrading
ImportError: No module named 'trading
=>so I  have to solve it by import sys and append the code sys.path.append('/usr/anaconda3/lib/python3.5/site-packages/tgym/envs/')
=>There are also analogous problems:
File "dqn_agent.py", line 118, in
from tgym.gens.deterministic import WavySignal
File "/usr/anaconda3/lib/python3.5/site-packages/tgym/gens/init.py", line 1, in
from csvstream import *
ImportError: No module named 'csvstream'
2.when I solve the problems about the module,there are also error:
=> File "/usr/anaconda3/lib/python3.5/site-packages/tgym/gens/csvstream.py", line 6
print csv
^
SyntaxError: Missing parentheses in call to 'print'
=>I have saw the codes of dqn_agent.py,it uses pytho 3.x.However,csvstream.py uses pyhton 2.x
===>Therefore ,I want to ask is there a efficient way to run the examples.I do not know how many bugs there are.Thank you!

I also had problems with the init.py files regarding imports. But after fixing them I had this error:
Traceback (most recent call last):
File "D:\FG\Machine Learning\OpenAI\CryptoTrade-Orders\DQN_Trading2.py", line 121, in
generator = WavySignal(period_1=25, period_2=50, epsilon=-0.5)
File "C:\Users\Filipe\Anaconda3\lib\site-packages\tgym\core.py", line 54, in init
self.n_products = len(self.next()) / 2
File "C:\Users\Filipe\Anaconda3\lib\site-packages\tgym\core.py", line 71, in next
return self.generator.next()
AttributeError: 'generator' object has no attribute 'next'
Looking at the code in core.py I couldn't figure out what was going wrong

Hi both. Thanks for reporting these issues.
I suspect the problems come from using python 3.x. We are not yet fully supporting python 3.x so I would recommend running tgym with python 2.x.

/home/dav/anaconda3/lib/python3.6/site-packages/tgym/envs/init.py in ()
----> 1 from trading import SpreadTrading
ModuleNotFoundError: No module named 'trading'

Hi davdigioia
This is caused by Python 3.x. It does not support relative import any more.
you might change it to:
from tgym.envs.trading import SpreadTrading

Thanks. I will try
On 25 Oct 2017 06:55, "paulg"<EMAIL_ADDRESS>wrote:

Hi davdigioia
This is caused by Python 3.x. It does not support relative import any more.
you might change it to:
from tgym.envs.trading import SpreadTrading
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/Prediction-Machines/Trading-Gym/issues/9#issuecomment-339222717,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AUCb4HtjJYMw6qZYSeOS65PcBEGGrtErks5svs06gaJpZM4OtPWm
.

Is this dqn agent in static mode ? cause i run th generator using live feed . it say chunk error.