Pose landmarker running on IOS WKWebview returns an error: Creating a context with WebGL 2 failed: UNKNOWN: emscripten_webgl_create_context() returned error 0 (vision_wasm_internal.js, line 1087)

Have I written custom code (as opposed to using a stock example script provided in MediaPipe)
No
OS Platform and Distribution
iOS 16.4.1
Mobile device if the issue happens on mobile device
Iphone
Browser and version if the issue happens on browser
No response
Programming Language and version
Javascript
MediaPipe version
No response
Bazel version
No response
Solution
Pose
Android Studio, NDK, SDK versions (if issue is related to building in Android environment)
No response
Xcode & Tulsi version (if issue is related to building for iOS)
No response
Describe the actual behavior
FilesetResolver.forVisionTasks(
      "https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>    ).then((vision) => {

      poseLandmarker =  PoseLandmarker.createFromOptions(vision, {
        baseOptions: {
          modelAssetPath: `https://storage.googleapis.com/mediapipe-models/pose_landmarker/pose_landmarker_lite/float16/1/pose_landmarker_lite.task`,
          delegate: "CPU"
        },
        runningMode: "VIDEO",
        numPoses: 1
      }).then((poseLandmarker) => {

});

The code when running on WKWebview inside an iOS app returns:

[Warning] I0000 00:00:1685603814.329000       1 gl_context_webgl.cc:81] Couldn't create webGL 2 context. (vision_wasm_internal.js, line 1087)
[Warning] W0000 00:00:1685603814.337000       1 gl_context_webgl.cc:106] Creating a context with WebGL 2 failed: UNKNOWN: emscripten_webgl_create_context() returned error 0 (vision_wasm_internal.js, line 1087)
[Warning] === Source Location Trace: ===  (vision_wasm_internal.js, line 1087)
[Warning] third_party/mediapipe/gpu/gl_context_webgl.cc:82 (vision_wasm_internal.js, line 1087)
[Warning] W0000 00:00:1685603814.340000       1 gl_context_webgl.cc:107] Fall back on WebGL 1. (vision_wasm_internal.js, line 1087)
[Warning] I0000 00:00:1685603814.340000       1 gl_context_webgl.cc:81] Couldn't create webGL 1 context. (vision_wasm_internal.js, line 1087)
[Warning] W0000 00:00:1685603814.340000       1 gl_context.cc:1000] OpenGL error checking is disabled (vision_wasm_internal.js, line 1087)
[Warning] E0000 00:00:1685603814.340000       1 gl_graph_runner_internal.cc:252] StartGraph failed: INTERNAL: Service "kGpuService", required by node mediapipe_tasks_vision_pose_landmarker_poselandmarkergraph__mediapipe_tasks_vision_pose_detector_posedetectorgraph__mediapipe_tasks_components_processors_imagepreprocessinggraph__ImageCloneCalculator, was not provided and cannot be created: emscripten_webgl_create_context() returned error 0; StartRun failed (vision_wasm_internal.js, line 1087)
[Warning] === Source Location Trace: ===  (vision_wasm_internal.js, line 1087)
[Warning] third_party/mediapipe/framework/calculator_graph.cc:651 (vision_wasm_internal.js, line 1087)
[Warning] third_party/mediapipe/framework/calculator_graph.cc:682 (vision_wasm_internal.js, line 1087)
[Warning] third_party/mediapipe/framework/calculator_graph.cc:551 (vision_wasm_internal.js, line 1087)
[Warning] research/drishti/app/pursuit/wasm/graph_utils.cc:87 (vision_wasm_internal.js, line 1087)

Describe the expected behaviour
FilesetResolver.forVisionTasks(
      "https://cdn.jsdelivr.net/npm/<EMAIL_ADDRESS>    ).then((vision) => {

      poseLandmarker =  PoseLandmarker.createFromOptions(vision, {
        baseOptions: {
          modelAssetPath: `https://storage.googleapis.com/mediapipe-models/pose_landmarker/pose_landmarker_lite/float16/1/pose_landmarker_lite.task`,
          delegate: "CPU"
        },
        runningMode: "VIDEO",
        numPoses: 1
      }).then((poseLandmarker) => {

});

Should return poselandmarker object successfully

Standalone code/steps you may have used to try to get what you need
Try to run poselandmarker on WKWebview inside iOS app based on mediapipe codepen:
https://codepen.io/mediapipe-preview/pen/abRLMxN

Other info / Complete Logs
No response

Adding an empty canvas to the html and referencing it in poselandmarker init solved the bug, don't really understand why (it's not written in the documentation as far as I know):
  poseLandmarker =  PoseLandmarker.createFromOptions(vision, {
    baseOptions: {
      modelAssetPath: `https://storage.googleapis.com/mediapipe-models/pose_landmarker/pose_landmarker_lite/float16/1/pose_landmarker_lite.task`,
      delegate: "GPU"
    },
    canvas: document.getElementById('canvas-test2'),
    runningMode: "VIDEO",
    numPoses: 1
  }).then((poseLandmarker) => {});

@galharth,
Thanks for the confirmation, We are forwarding this issue to understand better and let you know the update on it.

Hi @galharth,
Could you please inform us if the issue still persists on your end or if it has been resolved?
Thank you!!


[portal]: division description on the map should contain legal entity name

Currently, when checking divisions on the map - only division name is displayed, which is not informative.
We should show a MSP name + division name
Screenshot attached
cc: @lymychp @kmakarenko

done on prod - division_name(Legal_entity_name)


libreoffice starts with warnings

Issue description
Within a few 100ms, I see the following:
LC_ALL=C libreoffice
mkdir: cannot create directory '/run/user/1000/libreoffice-dbus': File exists
Unknown username "pulse" in message bus configuration file
Failed to drop supplementary groups: Operation not permitted
Failed to start message bus: Failed to set GID to 4: Operation not permitted

Expected output when running libreoffice, no warnings.
NixOS 18.03 (release-channel)

Does it still happen on master, by the way? I think I have made a change that might have fixed it…

No idea. How can I test it without building it from source?

It is actually in unstable channel already.
nix-build -I nixpkgs=https://d3g5gsiof5omrk.cloudfront.net/nixpkgs/nixpkgs-18.09pre139198.9d0b6b9dfc9/nixexprs.tar.xz '' -A libreoffice

@7c6f434c  You fixed this issue. However, when I run it without specifying LC_ALL=C I get the following.
(soffice:31960): Gtk-WARNING **:<time>.198: Locale not supported by C library.
        Using the fallback 'C' locale.

It should output which locale is not supported and it should also output which ones are to make it easy to resolve the problem.

It should output which locale is not supported and it should also output which ones are to make it easy to resolve the problem.

Sorry, what you say should not be fixed in a wrapper. GTK does that, patching GTK error reporting is not in scope.
More to the point: do you have $LOCALE_ARCHIVE in your environment? It sounds like you use Nix on non-NixOS Linux, and do not use the default nix.sh profile snippet.
Unlike DBus and Java paths etc. where LibreOffice has some requirements and some other software doesn't, $LOCALE_ARCHIVE — to the best of my knowledge — is a universal thing for everything from Nixpkgs.

@7c6f434c I was not saying that it was in scope of this particular issue. I am happy that you made libreoffice an improvement for NixOS, as well as that you made it easy for me to test things. I also did not say it had to be fixed in the wrapper.
I am using NixOS (as indicated in many issues on GitHub).
env| grep LOCALE
LOCALE_ARCHIVE=/run/current-system/sw/lib/locale/locale-archive

system: "x86_64-linux"
host os: Linux 4.15.18, NixOS, 18.03.git.45cd609 (Impala)
multi-user?: yes
sandbox: no
version: nix-env (Nix) 2.0.1
channels(root): "nixos-17.09.2182.7f6f0c49f0"
channels(coretemp): "nixos-18.03.132192.ce0d9d638de"
nixpkgs: /nix/var/nix/profiles/per-user/root/channels/nixos/nixpkgs

Sorry for assumptions — well I do expect people to run different things in different environments, so I never try to look up what people used last time.
I was a bit brief with my scoping claim — sorry if it sounded rude, I didn't want to say you were saying or doing something wrong, just that some of your points probably need an additional issue with a different scope for discussing tradeoffs.
I hope that my wrapper is either an improvement or a no-op for every case; I think some cases it helps with are more likely outside NixOS, but no idea. It is cheap to rebuild, so anything that seems related to LibreOffice can be merged into it without too much risk.
As a quick test: if you run perl -e 'print "1\n"', what happens? Best case: it prints 1 and nothing else. Worst case (what I get with LOCALE_ARCHIVE= ): ir complains about locales.
Does you NixOS configuration contain anything about locale? Like supportedLocales, which seem to default to "all" but may be restricted in configuration?

It prints 1 and nothing else.
i18n.defaultLocale = "de_DE.UTF-8"; # only thing related to locales.

What you did is a net improvement. Just for form I will close this issue.

If Perl doesn't complain, I might have (previously) broken something in the wrapper…
If you run libreoffice.libreoffice, does it complain the same way about locale?

I don't know what libreoffice.libreoffice is, but running plain libreoffice (the one in 18.03 stable) does not print the locale error.

In the master branch and in the unstable channels unwrapped Libreoffice is available as libreoffice.libreoffice package.
env -i PATH=$PATH LOCALE_ARCHIVE=$LOCALE_ARCHIVE DISPLAY=$DISPLAY HOME=$HOME LC_ALL=de_DE.UTF-8 libreoffice doesn't complain about locales, so I am trying to narrow down what is going on…

Using libreoffice.libreoffice from unstable as specified above.
/nix/store/yjcya0dni8k63asghwk2crfkllxsy5j5-libreoffice-<IP_ADDRESS>/bin $ env -i PATH=$PATH LOCALE_ARCHIVE=$LOCALE_ARCHIVE DISPLAY=$DISPLAY HOME=$HOME LC_ALL=de_DE.UTF-8 ./libreoffice /nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/bash: warning: setlocale: LC_ALL: cannot change locale (de_DE.UTF-8)
/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/bash: warning: setlocale: LC_ALL: cannot change locale (de_DE.UTF-8)
/nix/store/xn5gv3lpfy91yvfy9b0i7klfcxh9xskz-bash-4.4-p19/bin/sh: warning: setlocale: LC_ALL: cannot change locale (de_DE.UTF-8)
/nix/store/yjcya0dni8k63asghwk2crfkllxsy5j5-libreoffice-<IP_ADDRESS>/bin/.soffice-wrapped: line 160: warning: setlocale: LC_ALL: cannot change locale (de_DE.UTF-8)

Hm. And if you wrap perl in the same way, it doesn't complain, right?

Not sure whether you meant this, but:
env -i PATH=$PATH LOCALE_ARCHIVE=$LOCALE_ARCHIVE DISPLAY=$DISPLAY HOME=$HOME LC_ALL=de_DE.UTF-8 perl -e 'print "1\n"'
1

Thanks!
So there is some LibreOffice-specific problem, I cannot reproduce it locally, and I have no idea what it is… annoying.

One way to fix this is to write a NixOS module, which supports less environment variables. For example, only LC_ALL, PAPER_SIZE and LANGUAGE. This way you don't have to support all possible combinations of environment variables users might throw at your wrapped version.
I would try to fix the error message upstream. I don't expect you to do this. I think libreoffice already works better on NixOS with your fixes than it does on some other Linux operating systems.

Well, I try to fix in the wrapper things I can reproduce easily. And I don't want to complain to upstream about things I cannot reproduce, either… (also: I don't use NixOS mainline; and I think manipulating the environment that is needed only for LibreOffice could be reasonably done on the level of the wrapper — maybe optionally).
For example: apparently, even with the same restricted environment LibreOffice complains about locale and Perl doesn't. If I could reproduce that behaviour locally, I would have an opinion what is the saner way to debug and triage and maybe fix it…
If you are willing to do some more random experiments, maybe creating a completely fresh/empty user account and comparing the behaviour there could help localizing the difference…


Course Live Update

We will need to be able to update a course config without restarting the server (especially given #6).
Most of this is not too hard, watch the course.json file for a change in modtime.
However, this may also require a course-level lock when an update is in-progress.

Implemented in becb2867206cc91a7c9b3a7ddf48ee2d5b68f075.


Strange bug, moving from DOWN to UP

Hi,
First, I would like to thanks you about your job and yours tutorials about Q learning method with tensorflow. I learn lots of things thanks to you !
I would like to report a bug. I prepared a little code to show you what happening. Maybe I did something wrong, but according to me, this is a normal behavior which should work.
`class PongPlayer(PyGamePlayer):
def init(self, force_game_fps=60, run_real_time=False):
"""
Example class for playing Pong
"""
super(PongPlayer, self).init(force_game_fps=force_game_fps, run_real_time=run_real_time)
self.last_bar1_score = 0.0
self.last_bar2_score = 0.0
self.it = 0
def get_keys_pressed(self, screen_array, feedback, terminal):
    # TODO: put an actual learning agent here
    self.it += 1
    if self.it < 100:
        print "K_DWON"
        return [K_DOWN]
    else:
        print "K_UP"
        return [K_UP]`

When I execute this code, the agent going down, but when a new action is returned (K_UP), the agent stops working and don't move anymore. Even If continue to return K_UP.
Here, I made a simple example, but this cause lot of problems when I try to launch my real code.
Here, a gif showing the problem.

Thanks you for your help !

Thanks for fixing @thibo73800


Fix typos in Task.get_param_names: include_significant -> include_insignificant

Fix typo include_significant  -> include_insignificant in Task.get_param_names

Description

Fix typos  include_significant  -> include_insignificant in Task.get_param_names so the names match the actual behavior.
Motivation and Context

No context.
The typos are inconsistent with the actual behavior.
Have you tested this? If so, how?

Not tested.
But there will be no side effects as just fixing typos and include_significant only appears in the fixed method.

This problem is also mentioned in #3167 suggesting:

So, I'd suggest to rename the parameter to include_insignificant (insignificant), or to change the implementation that on default it only returns the insignificant parameters, which would be weird though.

This request is equivalent to the former suggestion of renaming in #3167.
I see the include_significant is just a typo and expected behavior is to return only significant parameters on default, as the my code in this request does.
I also add the later suggestion of changing default behavior in #3167 is a major change that can cause some problems in existing codes.


Enable session_cookie_http_only by default

There is no real good reason why the session cookie should be accessible by javascript by default; if someone really, really wants to do this, they will know what to configure.

thanks for a patch


[Bug]: Export result set without logs

ODC version
odc423
OB version

What happened?

What did you expect to happen?
Export result set with logs
How can we reproduce it (as minimally and precisely as possible)?
Export result set, view logs
Anything else we need to know?
No response
Cloud
No response

pass


JwksValidationHandler + requestAccessToken: false = Error validating tokens

Hi,
In my use case, we don't use the access_token, we only use the id_token. So I've set requestAccessToken to false, but I still need to validate the id_token.
I'm validating the token using this.oauthService.tokenValidationHandler = new JwksValidationHandler();, which throw an error after the login:
angular-oauth2-oidc.js:1183 Error validating tokens
angular-oauth2-oidc.js:1184 TypeError: Cannot read property 'words' of undefined
core.js:15714 ERROR Error: Uncaught (in promise): TypeError: Cannot read property 'words' of undefined

I think that that the validateAtHash method in the ValidationHandler should not be called if requestAccessToken is set to false.
For now I simply extended the JwksValidationHandler and override validateAtHash:
import {JwksValidationHandler, ValidationParams} from 'angular-oauth2-oidc';

/**
 * Override the validateAtHash method as we don't request/use the access_token
 */
export class NoAccessTokenValidationHandler extends JwksValidationHandler {
    validateAtHash(validationParams: ValidationParams): Promise<boolean> {
        return Promise.resolve(true);
    }
}

Are there any news on this issue?

Ok, I found my issue... I was calling the hasValidAccessToken method later in the code to initialize the implicit flow. After replacing it with hasValidIdTokeneverything works fine.

I am running into the same issue with a bare bones implementation of the oAuthService (the code below is everything I have)
auth.config.ts
export const authConfig: AuthConfig = {
  issuer: environment.idProviderUrl,
  redirectUri: window.location.origin + '/login',
  clientId: environment.idProviderClientId,
  scope: 'openid profile email',
  oidc: true,
  requestAccessToken: false,
};

app.component.ts
  private configure() {
    this.oauthService.configure(authConfig);
    this.oauthService.tokenValidationHandler = new JwksValidationHandler();
    this.oauthService.loadDiscoveryDocumentAndTryLogin().then(() => {
      if (!this.oauthService.hasValidIdToken()) {
        this.oauthService.initImplicitFlow();
      }
    });
  }
}

I was able to resolve it by not using the tokenValidationHandler at all
  private configure() {
    this.oAuthService.configure(authConfig);
    this.oAuthService.loadDiscoveryDocumentAndTryLogin().then(() => {
      if (!this.oAuthService.hasValidIdToken()) {
        this.oAuthService.initImplicitFlow();
      }
    });
  }

Doing it this way works for me and I am able to get my id_token so it seems to me that you aren't meant to use a tokenValidationHandler when using requestAccessToken = false. Can anyone confirm if that is correct or if there is an actual bug here?

Sorry for answering this old thread, it's still open. Maybe I can clarify things:
@orditeck I just ran into the same issue. Same configuration, no access_token.
TypeError: Cannot read property 'words' of undefined.
Thoroughly debugged @manfredsteyer 's code; was wondering why the JwksValidationHandler is missing the jwks object though it has been loaded.
Facepalm: I initialized the JwksValidationHandler BEFORE the DiscoveryDocument has been loaded. This solved my issue:
this.oauthService.loadDiscoveryDocument(myfunnyurl).then(() => {
  this.oauthService.tokenValidationHandler = new JwksValidationHandler();
});

Adding error handling for this would be nice @manfredsteyer ;)
@coltonneil The NullValidationHandler is not a solution; you're just skipping validation, right..?

@ddhille
We actually switched to using access tokens so this no longer effects us, but yes in the implementation I laid out above, we were simply skipping it because I believed that the JwksValidationHandler was only validating access tokens and we were using id tokens.
New implementation:
auth.config.ts
export const authConfig: AuthConfig = {
  // Url of the Identity Provider
  issuer: environment.provider,
  // URL of the SPA to redirect the user to after login
  redirectUri: window.location.origin + '/auth',
  // The SPA's id. The SPA is registered with this id at the auth-server
  clientId: environment.clientId,
  // set the scope for the permissions the client should request
  scope: 'openid profile email',
  // set response type, use code for Code Flow + PKCE
  responseType: 'code',
  // request an id_token, must be set to false
  oidc: false,
  // percent (95%) of token lifetime before token is refreshed
  timeoutFactor: 0.95
};

app.component.ts
...
export class AppComponent implements OnInit, AfterViewInit {

...
  ngOnInit() {
    this.authService.onInit();
  }

...

}

auth.service.ts
import {Injectable, EventEmitter} from '@angular/core';
import {OAuthService} from 'angular-oauth2-oidc';
import {authConfig} from '../model/auth.config';

@Injectable({
  providedIn: 'root'
})
export class AuthService {
...
  /**
   * configures OAuthService and triggers authentication process
   */
  onInit(): void {
    this.subscribeToEvents();
    this.oAuthService.configure(authConfig);
    this.oAuthService.setStorage(localStorage); // this is deprecated but the documented method was not working
    this.oAuthService.loadDiscoveryDocumentAndTryLogin().then(() => {
      if (!this.hasValidAccessToken()) {
        this.oAuthService.initCodeFlow();
      }
    });
    this.oAuthService.setupAutomaticSilentRefresh();
  }

...
}

We aren't using a validation handler because we are doing token validation in our API via the issuing auth server.

Forgot to close this. We've added this check a while ago.


Possible recommenders

Greedy uncertainty
Point that minimises sum of variances of test samples
Point that minimises sum of squared errors of test samples (assuming y=μ)
Most change of the function (assuming y=μ)

@chengsoonong, anything to add?

Here's a potentially useful paper that summarises a few approaches, but only focuses on one for the rest.
https://arxiv.org/pdf/1206.2944.pdf
Also don't forget the four frameworks for active learning in the acton repo, issue 93-96
https://github.com/chengsoonong/acton/issues

Cool, I’ll add Hyperopt to the list above

My modification of UCB appears to work reasonably well, depending on the choice of the parameter k. Lower k values improve learning at the start, but work poorly towards the end of the learning curve. Higher k values learn poorly at the start, but very well towards the end. Note that uncertainty recommendation is the limiting case as _k_⟶∞.

^ This is slightly better interpolation, based on the tan function.

One way to choose the schedule for changing k is to use sqrt(n).
See:
https://arxiv.org/pdf/0912.3995.pdf

I need to write this up in thesis.


Raspberry Pi installation failed，No arm version installed

OS：Raspberry Pi OS
release:5.10.17-v7l+
machine:rmv7l
I executed gvm install go1.4 -B and then executed gvm install go1.5, an error was reported: cannot execute binary file: Exec format error
I guess version 1.4 is installed into the wrong version, not the arm version, what should I do?

Any update?

The same problem here!
golang@mini:~ $ lsb_release -a
No LSB modules are available.
Distributor ID:	Raspbian
Description:	Raspbian GNU/Linux 10 (buster)
Release:	10
Codename:	buster

golang@mini:~ $ gvm version
Go Version Manager v1.0.22 installed at /home/golang/.gvm

golang@mini:~ $ gvm install go1.4 -B
Installing go1.4 from binary source

golang@mini:~ $ gvm install go1.5
Downloading Go source...
Installing go1.5...
 * Compiling...
/home/golang/.gvm/scripts/install: line 84: /home/golang/.gvm/gos/go1.16.6/bin/go: unable to execute binary file: Error in exec format
ERROR: Failed to compile. Check the logs at /home/golang/.gvm/logs/go-go1.5-compile.log
ERROR: Failed to use installed version

golang@mini:~ $ cat /home/golang/.gvm/logs/go-go1.5-compile.log
##### Building Go bootstrap tool.
cmd/dist
ERROR: Cannot find /home/golang/go1.4/bin/go.
Set $GOROOT_BOOTSTRAP to a working Go tree >= Go 1.4.
./make.bash: line 121: /home/golang/go1.4/bin/go: non-existent file or directory

My workaround: use the option -B.
golang@mini:~ $ gvm install go1.5 -B
Installing go1.5 from binary source

 @OCDpunk


How do i copy excel table to word using python with win32com

like below, copy excel table to a word and how to keep the format of tables using win32com?
It's really hard to read the docs
sheet1.Name
sheet1
sheet2.Name
sheet2
Can somebody try to help me!

There are many answers and examples on stackoverflow  addressing these methods. If you have an issue with a specific document that can be improved, please link to it here and someone will take a look.


rustic-cargo-run: minibuffer and history

This reads arguments for the cargo run command from the minibuffer and
saves the history.
I was missing this feature when switching form the default compile or
projectile-run-project.

Thanks! Do you think it's important that cargo run gets its own history or would it be ok to use compile-history ?

Thanks! Do you think it's important that cargo run gets its own history or would it be ok to use compile-history ?

You are right it should be compile-history, I just wasn't aware of it.

Can you please use rustic-cargo-bin instead of 'cargo'. I have to finish the PR that takes care of the remaining commands.

Ah sorry didn't notice you already updated.


Update helm chart version to 0.4.3

Description
Update helm chart to latest released version
Why is this needed
Updated docs
Fixes: #
How Has This Been Tested?

How are existing users impacted? What migration steps/scripts do we need?

Checklist:
I have:

[ ] updated the documentation and/or roadmap (if required)
[ ] added unit or e2e tests
[ ] provided instructions on how to upgrade

Hey @DMajrekar , thanks for this! Mind updating your commit to have a DCO signoff? ref: https://github.com/src-d/guide/blob/master/developer-community/fix-DCO.md


Copy as PNG and SVG with transparent background

When you copy single elements, shapes etc. as PNG or SVG it always have a white background. It would be great if it would be possible to copy drawings with transparent background.
I know there is an option in settings to export image with background that can be turned off but it's for saving whole drawing and not single selected element.

Hi. Changing the settings in the export dialog extends to the clipboard export actions, too!
Also, when you select an element, the export dialog will contain an option to toggle exporting just the selected elements.

Hi. Changing the settings in the export dialog extends to the clipboard export actions, too!
Also, when you select an element, the export dialog will contain an option to toggle exporting just the selected elements.

When i right click element and copy it to clipboard as PNG or SVG it always has a white background, even though i have turned off exporting background for images in general settings and added command line in frontmatter.
When i use ctrl + click on the object it exports whole canvas as i would save it without selecting any element.
And i have never seen any dialog when saving/coping drawings or their parst.

Same here, would be good to export it with transparent background. Atm it's a missing feature.

When i right click element and copy it to clipboard as PNG or SVG it always has a white background, even though i have turned off exporting background for images in general settings and added command line in frontmatter.

When i use ctrl + click on the object it exports whole canvas as i would save it without selecting any element.
And i have never seen any dialog when saving/coping drawings or their parts. Am I doing something wrong?
@Gentleman03 main menu (top-left) -> export image -> toggle the background off

This setting extends into the copy-to-clipboard-as-png setting.
@ngzero can you confirm whether this fixes the issue for you as well? 🙏

@dwelle I can confirm that changing the background toggle via main menu > export image also changes the behavior of the export as png functionality via right click on any shape/object.
I'm able to export with transparent background as png via right click.


environment error

Hi, thank you for such an excellent work.
When I try to install the environment, it gives an error like ResolvePackageNotFound or Found conflicts!
How can I solve these problems or is there other ways to install the environment?

Hi,
thanks a lot for your interest in our work!
We ran the released code on many devices with different GPUs across institutions and did not encounter the same issue.  The conda environment config file is sufficient. I wonder if you are using the Ubuntu system and installed the environment using the conda config file.
Good luck!

Thank you very much for your reply!
I used the window system before and there would be errors. When I switch to the Ubuntu system, there is no problem.


Dalton fails to build

From our Hydra:
exporting https://gitlab.com/dalton/dalton.git (rev 9d7c5e435b75a9695d5ac8714121d12e6486149f) into /nix/store/kbga2mxlp180w21gck2rkd0nm22ygrpd-dalton-9d7c5e4
Initialized empty Git repository in /nix/store/kbga2mxlp180w21gck2rkd0nm22ygrpd-dalton-9d7c5e4/.git/
remote: Enumerating objects: 4666, done.
remote: Counting objects: 100% (4666/4666), done.
remote: Compressing objects: 100% (3460/3460), done.
remote: Total 4666 (delta 1539), reused 2980 (delta 1114), pack-reused 0
Receiving objects: 100% (4666/4666), 26.87 MiB | 8.48 MiB/s, done.
Resolving deltas: 100% (1539/1539), done.
From https://gitlab.com/dalton/dalton
 * branch            9d7c5e435b75a9695d5ac8714121d12e6486149f -> FETCH_HEAD
Updating files: 100% (4425/4425), done.
Switched to a new branch 'fetchgit'
Submodule 'external/gen1int' (https://gitlab.com/bingao/gen1int.git) registered for path 'external/gen1int'
Submodule 'external/pelib' (https://gitlab.com/pe-software/pelib.git) registered for path 'external/pelib'
Submodule 'external/qfitlib' (https://github.com/cstein/qfitlib.git) registered for path 'external/qfitlib'
Cloning into '/nix/store/kbga2mxlp180w21gck2rkd0nm22ygrpd-dalton-9d7c5e4/external/gen1int'...
Cloning into '/nix/store/kbga2mxlp180w21gck2rkd0nm22ygrpd-dalton-9d7c5e4/external/qfitlib'...
Cloning into '/nix/store/kbga2mxlp180w21gck2rkd0nm22ygrpd-dalton-9d7c5e4/external/pelib'...
Submodule path 'external/gen1int': checked out '1e4148ecd676761b3399801acba443925a1fee6b'
Submodule path 'external/pelib': checked out '19dd0e91afbd18b7c9f2611b9978b1aeaa1c19f9'
Submodule path 'external/qfitlib': checked out '1acdc9863fdeae2cdbc7f5a599413257a095b8ad'
fatal: not a git repository (or any parent up to mount point /)
Stopping at filesystem boundary (GIT_DISCOVERY_ACROSS_FILESYSTEM not set).

Can be reproduced locally with the Flake.

Is that an "impurity" in one of the git submodules (e.g. refering to master)?

Yup, saw it passing in your Hydra. I'm not sure about the git submodules, as it is actually checking out commits (from git submodule status in the dalton repo). The fetchgit also succeeds in intialising them. It fails after it has initialised the submodules. I've also tried leaveDotGit in fetchgit but it does not help. :disappointed:

.gitsubmodules does not specify a specific revision. That is probably the source of the issue.


Testing, please ignore. (pddtex)

This is a bug bounty test.. Please DO NOT APPROVE THIS! (pddtex)

 Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.pddtex fxlib seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.


feat/fennel language server

feat: add gospel
feat: add gospel
feat: add fennel-language-server

@mason-org/triage notify(Reason::NewPullRequest)
This comment ensures a notification is delivered to maintainers.
 🤖 This is an automated comment.    📖 Source code


VS only able to sign when runing as administrator

"To create and sign an assembly with a strong name by using Visual Studio"
Step 1 should be:
Open Visual Studio as Administrator

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

ID: 2cadf5e8-f790-3892-d187-6fafd9a5647b
Version Independent ID: 8993ead4-6e89-f6fe-9b39-730194d59f7f
Content: How to: Sign an Assembly with a Strong Name
Content Source: docs/framework/app-domains/how-to-sign-an-assembly-with-a-strong-name.md
Product: dotnet-framework
GitHub Login: @rpetrusha
Microsoft Alias: ronpet

SO https://stackoverflow.com/a/31542468/1586498


Adds Status Page and Status Page Incidents resources

I noticed the API for incidents doesn't allow you get get a specific one by ID, the code currently allows it (just returns null) in case this is intended to be added.
Let me know if I need to change this 👍

Nice, I'll leave this open for now so I can add the endpoint to get a single incident upcoming week.


Giveaway popup always enabled entering on a giveaway

Hi,
as said in the title it seems that there is a bug that prevent me to disable the giveaway popups (the one shown in the picture below) when I enter a giveaway through the giveaway button in the main giveaway list.

It used to work correctly before but after a certain update (I don't remember exactly the version but the one of around one month ago) it stopped to work.
This is my configuration related to the giveaway button (maybe I'm missing something or there is something else to configure?)

Firefox 67.0 (64-bit) on Windows 10
ESGST 8.3.15
If you need more info don't hesitate to ask.
Thanks!

That's not a bug because the change was intended. ESGST changelog for 8.3.8:

1326 Enter / Leave Giveaway Button: remove option to show description and make it mandatory

There was some discussion about that change in the SG thread starting around here.


Define an Enum as part of a Union Type

Is your feature request related to a problem?
There are quite a few parameters, that accept an enum OR some other type of value (like int or float, but also stuff like lists, dictionaries, or lists of dictionaries).
Desired solution
There needs to be a way to deal with these types of parameters. Seems to be a bit out of scope to think of a solution right now.
Possible alternatives (optional)
No response
Screenshots (optional)
No response
Additional Context (optional)
Examples:

#/sklearn/sklearn.metrics._classification/f1_score/average

Duplicate of https://github.com/lars-reimann/api-editor/issues/762.


Dragging external items

What do you think about dragging external items onto the ngGrid?
Maybe we could support this by marking outside draggable objects as ngGridDraggable (new component) which will carry some data (an object). We will then use this object to create a new ngGridItem on the grid.
Will this fit the purpose of this library? I would gladly do a pull request if so.

At the moment, it's not possible to handle within the grid due to it being a directive and the item list being populated outside. If there were a component version of the grid with two-way binding on the item list it could be done, but with it using directives, in order to add an item, you have to actually add it to the item source that is bound to the for loop.
I guess, it would be possible for a new directive "ngGridDraggable" to trigger the grids dragging property and set itself as the dragging item, but on drag stop the grid would then try to add the item to it's internal representation of the grid state, which would result in there being a "ghost item". Well, we could add a case for that, but then the most we could do is trigger an event and the user will have to handle adding it to the item set. Either way, not ideal. Thoughts?

What I did in my fork: https://github.com/catalysts/angular2-grid branch demo-eocs was exactly what you said here.
I added an event which would be triggered on drop and in my app I listened for that event and added the item to the items list (which added it to the grid). Even so, I don't really like this solution because it seems to be a hack.
What do you think about making the directive a component with an internal list of items? Then you could do something like:
<ngGrid [items]="items" ></ngGrid>

If we do it like this then there is no need for a for-loop inside the ngGrid component. The config per item could be specified in the items list.
What do you think? is this to big of a change? maybe we can do this and still keep the current format so we don't break apps using the current implementation.

One more usecase for this: dragging from one grid to another. So we could have 2 grid side by side (or one inside another) and we could drag from the first to the second

Yes an ngGrid component would work. I would probably add it as NgGridComponent that utilises the directives rather than replacing the current implementation. However, that would then require being able to specify the item layout within that, say by doing:
<ngGrid [items]="items">
    <ngGridItem #item>

{{item.title}}

{{item.content}}

    </ngGridItem>
</ngGrid>

I'll have to look into how Angular 2 handles that kind of thing, and should probably do #83 first.

I've created a new ticket for creating the component #86, so we can move this discussion there and come back to this after that's done

So I played around to the component and ended up with a template like this (inside it):

and the template of a gridItem is given by another component (and added dynamically):
    private addItems(items: any[]): void {
        items.forEach((item, index) => this.createItem(item.component, index));
    }

    private createItem(component, index) {
        this.cmpResolver.resolveComponent(component)
            .then((factory:ComponentFactory) => factory.create(this.viewContainer.injector, undefined, `#gridItem_${index}`))
    }

This way we can specify the component and config of our item inside an object with this structure:
        {
            config: {
                col: 0,
                row: 12,
                sizex: 16,
                sizey: 2,
            },
            component: TestComponent,
        },

What do you think of this approach? it is working pretty well for me

Is there a "built-in" way to use ngGrid to create a toolbox and a canvas so we can drag/drop items from the toolbox to the canvas. The items wouldn't be movable/resizable within the toolbox but would be on the canvas. Finally, when selecting an item on the toolbox, we would "copy" the item before dragging it to the canvas so it remains on the toolbox. I guess you see the big picture! Thanks in advance!

Is there a "built-in" way to use ngGrid to create a toolbox and a canvas so we can drag/drop items from the toolbox to the canvas. The items wouldn't be movable/resizable within the toolbox but would be on the canvas. Finally, when selecting an item on the toolbox, we would "copy" the item before dragging it to the canvas so it remains on the toolbox. I guess you see the big picture! Thanks in advance!


New Chat Message Creator modules.

Also fix blue textbox bug.

Three tasks:

Make the util.KEY_CREATOR_FUNCTION return an object with keys
Add a header to all the default MessageCreatorModules (we're probably gonna be adding a bunch more hooks. think on click, etc)
Headers for all the default MessageCreatorModules.

Good once we have that.


Adding Pub/Sub docs and JS to dedent for the section header.

Also removing tabs from docs/_static/js/main.js
@tseaver Do we want to leave the doc you wrote in place? Certain things in there like # API request need to be cleaned up if it's going to be permanent.

Coverage remained the same at 100.0% when pulling 24ab93173ba158fcd1cb0e091c58eec0ed441c39 on dhermes:add-pubsub-docs into 65341aebbfe8f03ce0b907061bce796ebbfb7f00 on GoogleCloudPlatform:master.

Maybe the document should be renamed to docs/pubsub-surface.rst (following your lead in
I think highlighting exactly where an API request occurs is useful, in the spirit of #545 / #632.

@tseaver I rebased over the original with the docs/pubsub-usage.rst -> docs/pubsub-surface.rst rename.
As for the docs/pubsub-surface.rst doc itself, I haven't done a full pass. I seem to remember during #691 or #742 that we discussed the doc wasn't necessarily intended to be published, just for discussion. (I can't find those comments just now, but only quickly scanned the PRs.)
Anyhow, we can punt on that outside of this PR, but I need to do a second pass of docs/pubsub-surface.rst and I think you should too.

Coverage remained the same at 100.0% when pulling 53bf519a7e404e93fb99fc77dc6f2022eb78d8e4 on dhermes:add-pubsub-docs into 24ba02bf15edb42ca00b4a3e3956d24b59f00f09 on GoogleCloudPlatform:master.

@tseaver Any remaining issues?

LGTM


Improve handling of GPS data

Goal is to make it easy to translate GPS readings onto a map.
Currently, GPS is stored as lat/long subchannels (with optional time and speed values). Original plan was to store raw NMEA data, and we could go back to that if it improves things.
See https://midetech.atlassian.net/wiki/spaces/EC/pages/227606551/Basic+GPS+Recording for a discussion.

Looks like there are already enough issues for this in Lab


Proposal: Add some feedback info when user run daemon with --selinux-enabled option

The docs describes that "SELinux does not presently support the BTRFS storage driver".
When i run Docker Daemon with command as follow, Docker Daemon is running successfuly,
docker -d -s btrfs --selinux-enabled=true
I found the reason is that i have not config selinux, and function selinuxEnabled in sourcecode return false, so the error as follow was not implemented, the Docker Daemon goes on running.
if selinuxEnabled() && config.EnableSelinuxSupport && driver.String() == "btrfs" {
return nil, fmt.Errorf("SELinux is not supported with the BTRFS graph driver!")
}
As above, in my opinion, maybe we can add some logs about selinuxEnabled return result to advoid  user misunderstand it.
Thanks.
[Docker info]
Containers: 0
Images: 108
Storage Driver: btrfs
Execution Driver: native-0.2
Kernel Version: 3.13.0-24-generic
Operating System: Ubuntu 14.04 LTS
CPUs: 4
Total Memory: 3.666 GiB
Name: mabin-PC
ID: JRLZ:LAQV:CT2Z:WLEF:CCEB:J24M:HY75:GBOK:Y3A5:6Y2B:OLM3:CKXZ
No Proxy: localhost,<IP_ADDRESS>/8,::1

I think PR https://github.com/docker/docker/pull/11274 is created for this?


SAML controller needs functional tests

The SAML controller recently had functional tests added verifying user info that's passed over it


mark-merge-commits-in-list broken

Description

How to replicate the issue + URL
See https://github.com/typed-ember/ember-cli-typescript/commits/master?after=5ff0c078a4274aeccaf83382c0d6b46323f57397+174
Related

https://github.com/refined-github/refined-github/issues/7572

Extension version
24.6.14, 24.7.22+
Browser(s) used
Safari, Chrome

@fregante  The issue has been fixed by #7593


Alter Admin initialization to admit plugin-defined handlers

Problem
There's no mechanism for plugins to install endpoints onto the admin service.
There should be.
Solution
Introduce the Admin.WithHandlers trait, which allows admin initialization to
detect modules that expose handlers.
Refactor Admin initialization to compose handlers during initialization.
Previously, admin classes were subtyped to extend functionality. Now, admin
endpoints are concatenated during initialization so that an arbitrary number of
endpoints may be installed without comabting the type system.
Furthermore, the Namerd and Linker types now have
configured-but-not-yet-initialized Admin values (rather than AdminConfig),
so that Admin initialization is congruent with other subsystems.
Finally, remove unused/unwanted parts of admin interface to fix #98.

:star: 🉐

no js changes, rolling forward with this


[Docs] summary of complement and related terms

What we need to document?

summary of JHU data complement
related terms including "recovery period"

This is related to #514.
Where to document

example/usage_dataset.ipynb, https://lisphilar.github.io/covid19-sir/usage_dataset.html#The-number-of-cases-(JHU-style)
docs/markdown/TERM.md, https://lisphilar.github.io/covid19-sir/TERM.html

Created pull request #541 for this.
Added "recovery period" term and described .show_complement() usage.

Created pull request #541 for this.
Added "recovery period" term and described .show_complement() usage.


Fix #70

Fixes the null exception in #70 as well as adding a flag to ignore websocket messages while the websocket is being closed.

Awesome, thanks @doronrosenberg!

I released 0.4.1 with the fix.


Small improvement to the GPG command

Description of the change
It doesn't work on my machine without this option. I thought I'll add it.
Checklist
Please make sure that the proposed change checks all the boxes below before requesting a review:

[x] I have reviewed the guidelines for contributing to this repository.
[ ] The preview looks fine.
[ ] The tests pass.
[ ] The commit history is clean and meaningful.
[ ] The pull request is opened against the main branch.
[ ] The pull request is no longer marked as a draft.
[ ] You agree to license your contribution under the MIT license to Spacelift (not required for Spacelift employees).
You have updated the navigation files correctly:

[x] No new pages have been added, or;
[ ] Only nav.yaml has been updated because the changes only apply to SaaS, or;
[ ] Only nav.self-hosted.yaml has been updated because the changes only apply to Self-Hosted, or;
[ ] Both nav.yaml and nav.self-hosted.yaml have been updated.

If the proposed change is ready to be merged, please request a review from @spacelift-io/solutions-engineering. Someone will review and merge the pull request.
Spacelift employees should request reviews from the relevant engineers and are allowed to merge pull requests after they got at least one approval.
Thank you for your contribution! 🙇

@eliecharra fair. I was just confused because the command literally didn't work on my machine 😅

alright


Changing CharData::m_mapName to EntityData::m_current_map

I didn't expect to make so many changes... To summarize, this is about removing m_mapName from CharData and creating m_current_map in EntityData. I might be making too much changes at one go here.
This is compilable but untested, I will be awaiting your inputs before going in further.

Updated serializers for CharData and EntityData
Incremented class versions for affected structs
Character::from/toActualCharacter now takes in EntityData as parameter, where it is going to be serialized to/from QString
Character::reset and getCurrentCostume no longer deals with map names
Created uniquePtr for EntityData in Entity
GameAccountResponseCharacterData now has m_serialized_entity_data, which will be used by GameHandler
GameHandler::on_map_req now gets the map name from EntityData.m_current_map
DataHelpers::getMapName and setMapName now takes in Entity as parameter

Some things to do off the top of my head:

Test da ting (I feel a little scared to test this now tbh, with all the changes made :P)
Fill up EntityData::reset and dump functions with something
DataHelpers::getMapName is now unused, call it from somewhere else

Closes #302 .

So I was like, well let's just expect the unexpected with testing and jumped in.
This is what happens when on a newly created account (minus the created admin character):
blob:https://imgur.com/5113b582-75bd-4da7-8798-ca8dce56ea93
Every character slot (except the ones created by the player) are now treated as bad data O_O. You can try to enter the game with them, but MapInstance.cpp line 660 will give out an assertion failure. The created admin character can go into the game without a hitch.

I will pull this down for testing after the next round of changes.

To add to my commit message, I also removed MapName enum in GameHandler, as those are now stuff handled in DataHelpers' function (For GameHandler's part, specifically the new getMapPath() function).
But here's the bad thing - all of a sudden I'm reaping stale links again when trying to enter the map. I have no idea why since I didn't do a test run after merging with upstream/develop (my bad). Can you help check for me?

Will pull this down for testing tonight

@eos428 everything looks good to me. Those two comments are my only real input.
Friends online status doesn't work, but that's not because of your code, it's because we never update the online status or mapname of players in your friendslist. Truth is, we should probably look those values up realtime instead of saving them to the dbase. That's another issue altogether.

The Enum and helper functions are a fine solution, but spread the relevant data across the code base. What about using a vector of a Map data struct, and storing those values like an array?
This would be similar to how we store SlashCommands.
std::vector<int m_map_idx, MapData m_map_data> or something.
Would make it easier to add new maps and update existing with changes, and we can reference maps by their map idx without entering them sequentially

Overall, this looks good and makes things more robust.
What is interesting to me, is the reason for moving map name to EntityData, are we planning on allowing NPC to roam between maps ? :wink:

The move to entity data was at my direction. Of course we don't anticipate NPCs to roam between maps, but it makes more sense to me because that's where we store positional data? All Entities will have position and be assigned to a map, but not all entities will have character data?

Admit it, you just want the mob trains to follow some poor soul through the whole world :D

CHOOO CHOOO! 🚂 🚋 🚋 🚋

NPCs that roam between maps actually sound like a fun concept :D Think of a bounty hunter or bodyguard system, for example. If LUA scripting can implement that, then I'd like to try that!

Testing this...
Currently it's not showing the correct map_display_name on CharSel screen or in FriendsList (or TeamsList). It's still showing the full map path. Perhaps we should save IDX to the database, instead of name, and reference your g_defined_map_data where applicable?
I don't want to expand the scope more than necessary,  as there are valuable changes here, but as a baseline I think the CharSel screen should show the proper mapname prior to merge, as this was the original focus of #302
As an aside, I'll open a new issue for implementing this array in JSON per Nem's recommendation.

Oops, my mistake. I returned m_map_name instead of m_display_map_name. We do have MapIdx value in EntityData and we could use that for reference instead if you like.

I guess ultimately it doesn't matter if we use IDX or mapname, as long as the server knows what's what

7ec5734 works great!


Chrome extension Windows 10 Accounts not needed anymore?

I think the Chrome extension  "Windows 10 Accounts" isn't needed anymore because Chrome natively supports CloudAP.
https://chromeenterprise.google/policies/?policy=CloudAPAuthEnabled

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

ID: 3d3bcf99-60c3-876c-c421-3351e507a81b
Version Independent ID: c906978e-acf2-462c-d7ff-20221872a793
Content: Primary Refresh Token (PRT) and Microsoft Entra ID - Microsoft Entra ID
Content Source: docs/identity/devices/concept-primary-refresh-token.md
Service: active-directory
Sub-service: devices
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

#remove-label:"cxp"
#label:"assigned-to-author"
#reassign:@MicrosoftGuyJFlo

#please-close


Look into using our VMs for flutter/plugins

Per discussion today, we should evaluate whether we can use our own VMs for the flutter/plugins Cirrus jobs.
(@godofredoc Just after our conversation I realized that this may be problematic as it may run into the same security policy change issues that cause us to use LUCI for Windows flutter/plugin tests rather than Cirrus. I'm not familiar with the details of what's involved here though, so maybe not.)

We already have a service account being used in the engine repo and got confirmation that if we use our own VMs we don't need to use compute credits.

We already have a service account being used in the engine repo

I just resurrected an email thread for you; if I'm understanding what the proposal is here, re-creating the existing flutter/engine setup in flutter/plugins hit roadblocks.

Update: it is possible to use our own gke instances to scale up cirrus and run linux and windows tasks without restrictions with no impact to Flutter's  budget.
Also from the security point of view several changes will be required to the flutter-cirrus project and we can register gcp credentials for flutter/plugins as part of that process.
I'll create and share a one pager to describe the entire process.


doc:fix characteristics of wrongly written words

fix characteristics of wrongly written words in architecture-design-cn.md

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.苏帅 seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.You have signed the CLA already but the status is still pending? Let us recheck it.

@sushuai please sign the CLA.

@membphis  CLA signed

many thx.


Add a way to disable background shadow per provider

Is your feature request related to a problem? Please describe.
I have a provider to cycle through theme variations similar to the built-in . I like the background shadow, but it gets in the way of this provider.
Describe the solution you'd like
I'd like the ability to set this option for a specific provider, or at least a hook that can be used to run general configuration before the windows are drawn. If there is a way, I must have overlooked it.
Describe alternatives you've considered
I tried setting g:clap_enable_background_shadow = v:false in an init() method but that didn't seem to work.

So it's a provider in your personal config?
If so, wouldn't something like this work for your use-case?
com! ClapColors call ClapColors()
fu! ClapColors()
  let g:clap_enable_background_shadow = v:false
  Clap custom_colors
endfu
" and reset g:clap_enable_background_shadow in your provider's exit

Yes, that does work. I was just wondering if there was something like the inverse of on_exit that could be called to set variables before running the provider, without wrapping in a function.
It works doing what you suggested.

I don't see any hook for that so it doesn't seem to be possible to run a function before the init function.


IP geo locate service maxmind is not publicly available anymore

The geo IP DB from maxmind is not available publicly anymore according to this blog post:
https://blog.maxmind.com/2019/12/18/significant-changes-to-accessing-and-using-geolite2-databases/
So, starting the docker image of openvpn-status fails with this error:
Error: getaddrinfo ENOTFOUND geolite.maxmind.com geolite.maxmind.com:443
Since the docker image available on the docker registry is not quite up to date it does not include this fix to allow startup without the geo IP DB:
https://github.com/AuspeXeu/openvpn-status/commit/b01889fdbab7a1f46abb51e2d8f58e7a2c807a6e

Relinked to this repo so the docker image should be up to date again in a few minutes.

On a fresh install, node server.js yields this error:
[Error: ENOENT: no such file or directory, open `./GeoLite2-City.mmdb`]
{
   errno: -2,
   code: 'ENOENT',
   syscall: 'open',
   path: './GeoLite2-City.mmdb'
}

Maybe this can be used to fix this issue https://github.com/maxmind/GeoIP2-node

I suppose.  Perhaps Maxmind GeoLite accountID and licenseKey could be added to cfg.json.  I was just able to sign up for an account here: https://www.maxmind.com/en/geolite2/signup.  We can use their free databases if you really want to keep the map.

Were you able to use the package I posted previously?

I'm sorry.  I have not tried.  All of my experience is in Angular, I have none in Vue.  Personally, I would like to create my own monitor in Angular.


share Graphix-t5+picard predicted SQL file

Hello, may I ask if you can share the predicted SQL that you finally ran on the validation set? Replicating your work is too time-consuming for me as my server does not have a GPU.

https://github.com/AlibabaResearch/DAMO-ConvAI/tree/main/graphix/dev_result
Just finished pushing, enjoy it!


Pod controller refactoring

Styling is now consistent throughout the whole file
Usermessages replaced with net messages
HasPod() and HasPly() have been removed, with their functionality being merged into GetPod() and GetPly(), respectively. These functions were only used in this file and have had all uses replaced
Instead of constantly triggering updates to the pod's many outputs, they are now cached on the pod itself and only updated when they change, which is significantly faster overall
Reduced many __index calls and other miscellaneous optimizations

I forgot about these PRs for a couple weeks lol
I've made the recacheOutput method check for self.Outputs.(name).Value instead as requested. I also moved the output trigger for the player's team into PlayerEntered in order to save a bit more performance (do players really need to change teams and have that actually impact other Wire entities without having to get out of the vehicle that often?)


Add support for Couchbase relations

Please make sure the below checklist is followed for Pull Requests.

[ ] Travis tests are green
[ ] Tests are added where necessary
[ ] Documentation is added/updated where necessary
[ ] Coding Rules & Commit Guidelines as per our CONTRIBUTING.md document are followed

@jdubois any idea why this fails? this seems related to the sonar-scanner recently added

@MathieuAA I don't get why the "sonar-scanner" command is not found, I restarted the build, let's see

Still failing.... that's probably because that only works on master -> you can ignore it, and I'm going to try to disable it if it's not on master

Good point, thanks :)


Remove 7 year old unimplemented fields

These fields have been sitting here for seven years. There is no guidance on implementation of these fields https://github.com/dart-lang/intl/issues/52. I suggest that we remove them completely if they are not to be implemented.


bug(recipe): should we not transform the last step in recipe during recipe.fit()

should we not transform the last step during fit(): https://github.com/ibis-project/ibis-ml/blob/main/ibis_ml/core.py#L512
    def _fit_table(
        self, table: ir.Table, targets: tuple[str, ...] = (), index: str | None = None
    ) -> None:
        metadata = Metadata(targets=targets)

        if index is not None:
            table = table.drop(index)

        for step in self.steps:
            step.fit_table(table, metadata)
            table = step.transform_table(table)

        self.metadata_ = metadata

if it used in sklearn or other pipeline, I think it is needed. But it looks weird, if we use recipe independently...


Update tutorial.asciidoc to note curl discrepancy on Windows

Apparently you need to escape the double quotes in the json body of the request on Windows.

Indeed haven't tried this on Windows, so thanks for spotting and fixing, @DaveSlinn.


Break up quickstart example projects and introduce JUnit 5 reference

This change adds a quickstart reference for using TestContainers with JUnit 5. Still uses the "old" Gradle configurations to ensure uniformity across the whole documentation.

Required breaking up the quickstart example projects and making them subprojects
Separates Redis code examples from generic code references and changed references to it
Contains some text duplication but maybe this can be addressed separately
Adds similar issue as described in https://github.com/testcontainers/testcontainers-java/issues/1415

Thanks, this looks great!
Just one thing, we kind of decided to go on moving all code examples (also the doc ones) to ./examples in the project root (@bsideup @rnorth please correct me if I'm wrong).
Would you be okay with doing it for the examples that correspond to this PR?

Would you be okay with doing it for the examples that correspond to this PR?

I think it would be more straight forward to keep the structure you have right now. How about merging this PR first? I'd be happy to follow up with another PR shortly thereafter that moves the examples in general. This approach would lead to less, better reviewable changes per PR as all references to example would have to be changed.

@kiview Any more thoughts on this? Is there anything else I can do to get the changes merged?

Is this still of interest? Otherwise I might as well close it.

Sorry @bmuschko this is still of interest - we've just all been quite snowed under for several weeks. I'll try and review/merge this week.
I'm very sorry for the delay.

@rnorth I volunteer to deliver this PR.
Could you please re-review (since @bmuschko applied some changes)?

@bsideup please could you resolve the merge conflicts, then I think it's basically good to go.
It would be good to have another go at making the docs codeinclude the dependencies directly from the relevant build.gradle file, so that the docs effectively get updated by dependabot when new versions of JUnit 5 are released.
We could do that later though; it'd be good to get this merged even without that.

@rnorth resolved

Awesome, thanks @bsideup.
@bmuschko thank you so much for this PR, and sorry that it's taken such a long time!
Merging 😄


Update with instructions for FreeBSD

Found this well hidden secret while trying to the exact same work dch already so masterly performed. =)
Could perhaps be nice for future FreeBSD users to find this information in the readme and potentially save them some time.

:green_heart: :blue_heart: :purple_heart: :yellow_heart: :heart:


Conversion for api name should be from uppercamel not lowercamel

This makes a difference since acronym replacing only occurs on upper camel causing toolkit to not WAI.  An baseline test case needs to be made to cover acronym replacements.

Could you do new baseline test case as part of this PR? You can just change the API name on the existing no_path_templates test(s).

Will do

Current coverage is 78.01% (diff: 100%)

Merging #506 into master will not change coverage

@@             master       #506   diff @@
==========================================
  Files           222        222
  Lines          8437       8437
  Methods           0          0
  Messages          0          0
  Branches       1131       1131
==========================================
  Hits           6582       6582
  Misses         1556       1556
  Partials        299        299

Powered by Codecov. Last update 119f6c5...332c57d

Ready for another review. Thanks for pointing out no path as a place to put the baseline test. I was a little curious as to the best way to add an acronym to a service name.

Fixed.

Thanks!


feat: add nodeSelector, pod tolerations, node affinity configuration

It would be good to filter pod schedule on some nodes, so I added this feature.

@eshepelyuk Any updates on this?

Sorry, I didnt have time to review this. Hope will be doing a review this week.

Hello @dmakeroam
I am going to introduce breaking changes into a project, so I would like to understand the state of this PR, before I do it.
My intention is to wait as much as possible until you complete this contribution, so I can roll out a new release without breaking changes.
And then, make my breaking changes.
Please, describe your plans and intentions, so I can schedule my work on the project.
Thanx !

nodeSelector,  pod tolerations, node affinity configuration might be good features for the project, but right now I haven't much time to figure out.
To not block your changes, I would try to finish it by tomorrow.

Look @dmakeroam I can pick up this PR and finish myself.
Let's do it like this ?

You can sir, if you can't wait more

You can sir, if you can't wait more

Well, the PR is almost done, so only cosmetic changes are left. Dont worry I will complete PR by myself.


Update rotation filter

Codecov Report

Merging #180 into master will decrease coverage by 0.22%.
The diff coverage is 11.11%.

@@            Coverage Diff             @@
##           master     #180      +/-   ##
==========================================
- Coverage   58.25%   58.02%   -0.23%
==========================================
  Files          19       19
  Lines        1284     1289       +5
==========================================
  Hits          748      748
- Misses        536      541       +5

Impacted Files
Coverage Δ

dpemu/filters/image.py
66.76% <11.11%> (-0.97%)
:arrow_down:

Continue to review full report at Codecov.

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


Update MS1 DDA Search tutorial

Fixed link to tutorial PDF, and updated some text and a screenshot.

/rebase


Expand Task Param fields

Expected Behavior
Tasks should be reusable. When you look at a Task, it should be clear how to use it, specifically how to provide the right parameters and what they are. There should also be defaults for parameters, i.e. each Param should look like this:
  - name: DOCKERFILE_NAME
    description: The name of the Dockerfile
    default: Dockerfile

Description should be required
Default should be optional

Please also update examples.
Actual Behavior
Currently Task Params have only one field: name.
Additional Info
See knative build template docs.
Question: perhaps we want this functionality for resources as well?

@bobcatfish I'll tackle this one.

Thanks @tanner-bruce !
And like you were saying in slack, I wonder if we should add a type field back - it had seemed like we only needed string, but now it's looking like we could do with some kind of array/list type as well, e.g. for expressing multiple arguments.

@bobcatfish want to do that as a separate issue? Seems like it warrants a discussion about syntax/implementation.

Thanks again @tanner-bruce ! ❤


Create setup.py

The current installation method (pip installation from github) fails due to the missing setup.py.
 pip install git+https://github.com/citation-file-format/github2cff
Collecting git+https://github.com/citation-file-format/github2cff
  Cloning https://github.com/citation-file-format/github2cff to /private/var/folders/qq/fxkml5kx4pq9jqxt09967pyc0000gn/T/pip-req-build-7igmln_b
    Complete output from command python setup.py egg_info:
    Traceback (most recent call last):
      File "<string>", line 1, in <module>
      File "/usr/local/miniconda3/envs/science3/lib/python3.7/tokenize.py", line 447, in open
        buffer = _builtin_open(filename, 'rb')
    FileNotFoundError: [Errno 2] No such file or directory: '/private/var/folders/qq/fxkml5kx4pq9jqxt09967pyc0000gn/T/pip-req-build-7igmln_b/setup.py'

    ----------------------------------------
Command "python setup.py egg_info" failed with error code 1 in /private/var/folders/qq/fxkml5kx4pq9jqxt09967pyc0000gn/T/pip-req-build-7igmln_b/

Could you add a setup.py please?

Hi @sbliven, this project has been started at Collaborations Workshop 2018 and we haven't had the time to keep working on it.
I personally think that it would be useful to have this as infrastructure once we have stabilized CFF to 1.1.0 (we're currently working towards it), but I don't have time to work on github2cff any time soon.
The good news is: it's up for grabs, and we'd be really thankful if someone took to working on it :).


[SPARK-10671] [SQL] Throws an analysis exception if we cannot find Hive UDFs

https://issues.apache.org/jira/browse/SPARK-10671

Merged build triggered.

Merged build started.

Test build #42616 has started for   PR 8800 at commit 604dc33.

Test build #42616 has finished for   PR 8800 at commit 604dc33.

This patch fails Scala style tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42616/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #42617 has started for   PR 8800 at commit 5874854.

Currently saveAsTable tries its best to save the table as Hive compatible format, and falls back to Spark SQL specific format if it fails. I wonder whether it's too confusing for users to reason about behavior of saveAsTable. Would it be better to have a separate SQL option to force using Hive compatible format? With this option on, users can be quite confident that if a saveAsTable call succeeds, the saved table must be Hive compatible.

The impact of saving the metadata in a hive compatible way or not will appear when a user want to read the data. I am not sure they can remember how the metadata is stored. It is also very possible that the data producer and consumer are different persons/different teams.

Test build #42617 has finished for   PR 8800 at commit 5874854.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42617/
Test FAILed.

Merged build finished. Test FAILed.

Test build #1771 has started for   PR 8800 at commit 5874854.

Test build #1771 has finished for   PR 8800 at commit 5874854.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build triggered.

Merged build started.

Test build #42631 has started for   PR 8800 at commit 693939d.

Test build #42631 has finished for   PR 8800 at commit 693939d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42631/
Test FAILed.

Merged build finished. Test FAILed.

Merged build triggered.

Merged build started.

Test build #42672 has started for   PR 8800 at commit e133bee.

Will split this PR to two PRs.

Test build #42672 has finished for   PR 8800 at commit e133bee.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42672/
Test PASSed.

Merged build triggered.

Merged build started.

Test build #42679 has started for   PR 8800 at commit e0e1a4d.

Test build #42679 has finished for   PR 8800 at commit e0e1a4d.

This patch fails Spark unit tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42679/
Test FAILed.

Merged build triggered.

Merged build started.

Test build #42692 has started for   PR 8800 at commit 4746b82.

Test build #42692 has finished for   PR 8800 at commit 4746b82.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins/job/SparkPullRequestBuilder/42692/
Test PASSed.

LGTM


Create account on AWS

Create a new account to leverage the free-tier offered by AWS.

<EMAIL_ADDRESS>created on AWS as well as 2 users. Details sent by email.


Incompatability with Ubuntu 22.04 OpenSSL Heroku Stack 22

Deployed on Heroku stack 20, although all builds use the same Gemfile with gem "openssl", '~> 3.0.1'. The OpenSSL PKCS#12 parser doesn't work with both self-built P12 files and real ones.

Labeling as invalid since this is an issue with Heroku stack 22 not this repository, but leaving the notice here in case other's run into this bug with certain patches of OpenSSL or Ubuntu 22.04 on PKCS#12 parsing.


Add constant for MIME encoding to PGPLiteralData class

This format is introduced in rfc4880-bis10
See https://tools.ietf.org/id/draft-ietf-openpgp-rfc4880bis-10.html#name-literal-data-packet-tag-11

Thanks for the patch! Merged. I've got a feeling this is going to be interesting.... text should have been enough, I wonder what they've got in mind, I guess we'll find out soon enough.

This change, as well as the earlier ones is now up in https://www.bouncycastle.org/betas 171 b04 or later.


Allow for disabling of nbviewer

Currently, all notebooks from Github show a preview using nbviewer.
In some situations this causes a "404: Not Found" message, even though everythign is correct (e.g. when trying to launch a file that is wrapped inside a Docker image).
For example: https://mybinder.org/v2/gh/crestdsl/crestdsl-demo-binder/master?filepath=GettingStarted.ipynb launches a notebook that is not in the repo, but in the Docker image that is launched by the repo.
I believe that providing an url-argument e.g. &preview=false could be useful in these situations to disable the preview (so users aren't confused).
The concerned code places (as far as I see) are the ParameterizedMainHandler in main.py, where it would be necessary to extract the argument and pass it to the loading.html template. Code to add should be something like preview=self.get_argument('preview', True).
The other part is in the template itself (e.g. near Line 27 ) where a {% if preview %}
can be added.  (Line 30 already contains a condition that could also be easily extended...)
This issue is to allow feedback / opposition before starting implementing. Thus:

What do you think?
Is my current though process correct? or is my "very simple" solution not as simple after all?

PS: lacking time right now, so PR will come in a few weeks earliest.

I think there are two ways this could go:

we add a URL parameter like this (see #712 for naming)
or we make our preview smarter so that it hides itself if it can't show something useful

I don't know how much harder the second one is but it would be cool as it doesn't rely on the creator of the link knowing what they are doing and keeps the URLs shorter.
Is it feasible for BinderHub to make a HEAD request to nbviewer to find out if we would show a 404 or other error page and then decide on hiding the preview?

I think there are two ways this could go:

we add a URL parameter like this (see #712 for naming)
or we make our preview smarter so that it hides itself if it can't show something useful

Does one exclude the other? An URL param would certainly allow people to control the preview (i.e. even if there was something to useful to show?)
It appears to me that the former is the MVP solution. The more sophisticated one is the "nice-to-have".
As for

as it doesn't rely on the creator of the link knowing what they are doing
I guess the default creator won't be using/needing it (default works fine), while "power-users" get more control.

Adding a URL parameter is probably less coding work than making a request to nbviewer.
However once we introduce the URL parameter we are stuck with it "for life". We can't take it away again or change what preview=.. means/does. So as with all these things coding isn't the bottleneck but getting us to some kind of consensus is :)
What about using ..&nopreview without the need of =False as URL parameter?
I like the simplicity of the URL parameter approach. We can always add the "let's try and guess from the nbviewer response" approach later.

What about using ..&nopreview without the need of =False as URL parameter?

My reasoning behind preview=False was that it would prepare the URL for further (heavy) customization of the preview.
Notebook-branding (preview=http://mybusiness.com/myLogo.jpg), and the needs of the felinophile population (preview=funnyCats.gif) could then be implemented reusing the same, already existing parameter. (How about being shown cat-videos while waiting for the notebook to launch?)
PS: This is not a strong opposition to &nopreview, just asking whether a &preview=XYZ could make sense at some point in the future.

Notebook-branding (preview=http://mybusiness.com/myLogo.jpg), and the needs of the felinophile population (preview=funnyCats.gif)

My initial reaction to that is "what? no!". Maybe someone else can weigh in as well on what they think of the current and future use cases.

Yeah, I don't think allowing users to specify a different preview is something we should do at any point. That should be up to the operator, not the user. Disabling is fine, though. If this is really just preventing the display of 404s on nbviewer, it does seem to me that gating the preview on a successful fetch is the 'right' solution, though user opt-out is a simple band-aid.

Being able to gate the preview is a nice feature. The 404 may cause a client to quit too soon.
I would like to see the nbviewer service URL to be parametrizable. The advantage is that installations inside firewalls that are retrieving notebooks in GHE would have a mechanism for rendering. The current renderer can't reach the notebook, even though the internal binderhub can. This could be fixed if would could configure our own nbviewer service - something that already exists in GHE. At the moment, I don't yet know how GHE does it.

What is GHE?
Making the URL used for the iframe configurable sounds like a "no brainer" aka we should definitely do that. A good first PR? It might take a bit more Python/Jinja templating foo to determine if HEAD <nbviewerURL/path/to/notebook> returns a status 200 and only display the preview if so.

What is GHE?
Sorry, GitHub Enterprise.  GitHub, but set up by enterprises, ordinarily not available from the internet.

The same case as the OP happens in scikit-learn:

the notebook is in the docker image not in the repo
we use urlpath in the binder URL to link to a specific notebook

It would be great indeed to be able to disable the preview to prevent user confusion.
More details about the scikit-learn context: https://github.com/scikit-learn/scikit-learn/pull/11221#issuecomment-522472749

This commit in my branch (or see diff below)  seem to do what I want, i.e. disable the preview if the nbviewer URL can not be accessed. I tested it locally following https://github.com/jupyterhub/binderhub/blob/master/CONTRIBUTING.md#user-interface-changes.
Diff:
diff --git a/binderhub/main.py b/binderhub/main.py
index 3333108..add51da 100644
--- a/binderhub/main.py
+++ b/binderhub/main.py
@@ -1,6 +1,9 @@
 """
 Main handler classes for requests
 """
+import urllib.request
+import urllib.error
+
 from tornado.web import HTTPError, authenticated
 from tornado.httputil import url_concat
 from tornado.log import app_log
@@ -69,6 +72,12 @@ class ParameterizedMainHandler(BaseHandler):

             blob_or_tree = 'blob' if filepath else 'tree'
             nbviewer_url = f'{nbviewer_url}/{org}/{repo_name}/{blob_or_tree}/{ref}/{filepath}'
+
+            try:
+                urllib.request.urlopen(nbviewer_url)
+            except urllib.error.HTTPError:
+                nbviewer_url = None
+
         self.render_template(
             "loading.html",
             base_url=self.settings['base_url'],

This is a first naive attempt (full disclosure: I don't know that much about Tornado), let me know if the approach seems reasonable enough to move to a PR. In particular I am wondering about:

is it fine to do a blocking call like this in .get.
is there a better way to do this with Tornado rather than urllib

With #934 we now don't show the preview if it would contain an error so I'll close this issue.


Angular 2

Hi Nick, work in this project!
I want to use this in my project in angular 2 but is not that straightforward,  is it possible to have an implementation of this in angular 2 or more detail explanation using systemjs and esriSystem??

Hi @Basicside,
No worries - there's a few good resources to help you out.
You should probably just get an angular app up and running with the arcgis js api first. Here's some repositories that show how to get started:
https://github.com/nickcam/angular2-esri-netcore
https://github.com/jwasilgeo/angular2-esri-playground
Once you're up and running you can just take the flareClusterLayer.ts file and include it in your project somewhere with your other typescript files.
You'll need to add the dojo typings through npm and also copy over the typings/custom-typings.d.ts.
Then just import and use the FlareClusterLayer class like any other class in your project.
Hope it helps,
Nick

Hi, Thanks for the reply :)
having the arcgis api running on angular 2 is kind of tricky but I already manage that, my problem is with the flareCluster it self.
first of all, I have the FlareClusterLayer on my application inside the fcl folder, and on my index.html the configurations for the dojo to import the file.
but then I have to add to the custom-typings  something like that`declare module
 fcl/flareClusterLayer'{
    import flareClusterlayer = __esriExtend.flareClusterlayer;
    export = flareClusterlayer;
}

so angular can know what is the import FlareClusterLayer from fcl/flareClusterLayer
and I have to remove the "dojo/gfx from the custom-typings because it is in conflict with the dojo-typings
but then I'm having a error in the Flare file on line 644 surface.containerGroup = surface.createGroup();
so I comment that line and the three next lines, so I can go forward and see this working.
but know that this should be working when I run my application I have this error on the console
Uncaught ReferenceError: exports is not defined     at flareClusterLayer.ts:1
TypeError: flareClusterLayer_1.default is not a constructor

Hey man,
Sounds like you may be trying to use the javascript version and then reference it in typescript. Easiest thing is to just copy the typescript/flareClusterLayer.ts file to somewhere along side the rest or your typescript files.
Here's some quick steps that may help:

Copy typescript/flareClusterLayer.ts alongside your other ts files.
Add the typings: dojo-typings through npm, and index.d.ts and custom-typings.d.ts to somewhere in your project, ie ./typings/
Add a reference to index.d.ts to the top of your app.module.ts file (or whatever your main module is called) - ie /// <reference path="../typings/index.d.ts" />
Add the required esri and dojo module references to load.ts where you use esri-system-js.
ie, in the  esriSystem.register([ part. You can see it in this repo - https://github.com/nickcam/angular2-esri-netcore
Update the imports in flareClusterLayer.ts from this format:
import * as GraphicsLayer from "esri/layers/GraphicsLayer"; to this format -
import GraphicsLayer from "esri/layers/GraphicsLayer";
Then just import flareClusterLayer into any other typescript class you want.
for example -  import { FlareClusterLayer } from ' .\flareClusterLayer\flareClusterLayer';

If you create a plunkr or some other demo site I can have a look and edit it if you like.
Good luck,
Nick

Hello Nick,
Sorry for not saying anything, I'm up to my hair with work, thanks to you I
was able to get the FlareCluster to work on the angular2, I will create a
plunk with my solution, and added to the discussion because somebody else
may needed.
Thanks for your help
Ruben
Em sáb, 1 de abr de 2017 às 12:37, Nick Cameron<EMAIL_ADDRESS>escreveu:

Hey man,
Sounds like you may be trying to use the javascript version and then
reference it in typescript. Easiest thing is to just copy the
typescript/flareClusterLayer.ts file to somewhere along side the rest or
your typescript files.
Here's some quick steps that may help:

Copy typescript/flareClusterLayer.ts alongside your other ts files.
Add the typings: dojo-typings through npm, and index.d.ts and
custom-typings.d.ts to somewhere in your project, ie ./typings/
Add a reference to index.d.ts to the top of your app.module.ts file
(or whatever your main module is called) - ie ///
Add the required esri and dojo module references to load.ts where
you use esri-system-js.
ie, in the esriSystem.register([ part. You can see it in this repo -
https://github.com/nickcam/angular2-esri-netcore
Update the imports in flareClusterLayer.ts from this format:
import * as GraphicsLayer from "esri/layers/GraphicsLayer"; to this
format -
import GraphicsLayer from "esri/layers/GraphicsLayer";
Then just import flareClusterLayer into any other typescript class
you want.
for example - import { FlareClusterLayer } from '
.\flareClusterLayer\flareClusterLayer';

If you create a plunkr or some other demo site I can have a look and edit
it if you like.
Good luck,
Nick
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/nickcam/FlareClusterLayer/issues/22#issuecomment-290910431,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AEsbvVfQ4CoswomC0PQuyGcGnmASzqRWks5rriEogaJpZM4MuW_R
.

--
Melhores cumprimentos,
Ruben Costa

Good stuff! Glad you got it going...closing this issue :).

@Basicside can you please all the plnkr for this one as you got it to working .

Hey @dealsmyself - this advice would be a bit outdated now. There's now esri-loader instead of esri-system-js. Just raise a new issue outlining your problems in detail if you can't get it going.

Hi @nickcam quick question
in

FlareClusterLayer_v4.ts

exporting interface works , i.e
interface FlareClusterLayerProperties extends __esri.GraphicsLayerProperties {} is good
but when exporting class
export class FlareClusterLayer extends __esri.GraphicsLayer
got a error saying " __esri is not defined"
Please suggest.
Thank you

Hi @dealsmyself - yeah you can't subclass an interface - __esri.GraphicsLayer is an interface in the typings.
Where are you getting that code from? The repository has the class signature like this -

import * as asd from "esri/core/accessorSupport/decorators";
import * as GraphicsLayer from "esri/layers/GraphicsLayer";
... other imports as well

@asd.subclass("FlareClusterLayer")
export class FlareClusterLayer extends asd.declared(GraphicsLayer)

Hi @nickcam , in another issue i saw you gave a code for esri.service.ts  in
https://gist.github.com/nickcam/162f3a375206737e5b5e42fc638c0fd8
there you referenced a  BaseLoaderService
import { BaseLoaderService } from '../shared/base-loader.service';
could you please provide the baseloader service , that will be helpful for me.
Thank you.

Ok, yeah you should be using esri-loader.
If you just want to include FlareClusterLayer and not compile it as part of your project, just see this documentation from the esri loader readme. https://github.com/Esri/esri-loader#configuring-dojo
As for the BaseLoaderService, just ignore it. It was an application specific service for my app I forgot to take out of the gist, I've removed it now.

Thank you very much for your quick response @nickcam , let me give it a try and  i will update you.
Thank you

when i do this https://github.com/Esri/esri-loader#configuring-dojo
i dont need to npm install dojo typings right?
somehow its giving me
 http://localhost:4200/fcl/FlareClusterLayer_v3.js net::ERR_ABORTED 404 (Not Found)
thank you


Update Version 25.31.4, Add Maintainer

Fix rust version

Checklist

[x] Used a personal fork of the feedstock to propose changes
[x] Bumped the build number (if the version is unchanged)
[x] Reset the build number to 0 (if the version changed)
[x] Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
[x] Ensured the license file is being packaged.

Hi! This is the friendly automated conda-forge-linting service.
I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

@thewchan, @tobymao
Ready for review and merge - Thank you and regards.
Will merge https://github.com/conda-forge/sqlmesh-feedstock/pull/9 when this dependency is ready.

merged as part of https://github.com/conda-forge/sqlglot-feedstock/pull/482 -
Thank You @thewchan


Provide action object as second parameter of handle handlers

According to issue https://github.com/lelandrichardson/redux-pack/issues/27

I agree, this is a great idea! Thanks!


fix: url override

fixes #7

:tada: This PR is included in version 1.11.2 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket:


[RESTEASY-1974] Checkstyle - final fixes

Fixes according to rules in checkstyle.xml, mostly:

trailing spaces
modifier order
final arguments

Update checkstyle.xml

allow System.out/err, printStackTrace in comments
allow to suppress checkstyle using 'CHECKSTYLE.OFF' and 'CHECKSTYLE.ON'
ignore .properties files

Update pom.xml

run checkstyle check by default

Jira: https://issues.jboss.org/browse/RESTEASY-1974
Master PR: https://github.com/resteasy/Resteasy/pull/1763

This is not ready to merge yet.

@tterem please post a comment when this is ready for review/merge, I'm setting the "Do not merge" label for now.

@asoldano you can review/merge now.


Sonoff B1: Dimmable E27 LED Lamp RGB Color Light Bulb_Problem

We can't control Sonoff B1 Light Bulb,
Will be cool to switch it on and off at least !

Hi @HBrunet11 , as written in the README, this plugin only works for Sonoff basic relay (only 1 on/off switch). As I do not have other ewelink device except the basic switch one, it is very hard (if not impossible) for me to implement support for those devices.
It seems ByronTudhope have a fork with dimmer support, you can find his fork here: https://github.com/ByronTudhope/homebridge-ewelink-max

Hi, if I'll donate you and you will buy your own Sonoff B1 Light Bulb - will you make update including full support it? Very interested in this, hope that a lot lf people too... :)
If you interested - contact me plz via<EMAIL_ADDRESS>Sincerely your's, Victor.


Admin area url submission: un-modifiable

IN YOURLS core, there is the shunt_html_addnew filter to enable manipulation of the admin new url submission form, which I used with the Expiry plugin. This behavior seems to be bypassed in Sleeky, and the submission form does not include the option to expire the link.

@Flynntes Any update on this? I'd love to be able to give users the option to set an expiration time for links on the public frontend.

A fix for this would be awesome indeed! :)


@subtitle?

It's quite impossible to define a larger guide structure.
It would have been great if we could have a parser for at least '@subtitle' if not for more heading types.

I agree. It would be nice to be able to have a "collection" fall under a @title, and then have @subtitle for individual components (e.g. someone using Atomic design could do @title Atoms and @subtitle Buttons).
Not sure if this is what you were referring to, or not.


BUG 2142780: osd: remove stale dm device during osd-prepare-job

During osd-prepare-job,
The encrypted device is found to be closed in some cases when the OSD deployment has been removed manually accompanied by any of following cases:

node reboot
csi managed PVC being unmounted etc

while re-opening the block to re-hydrate the OSDInfo, the dm device <pvc-name>-block-dmcrypt clashes with the one used by OSD pod which is stale by now.
This commit adds cmd to remove this stale dm device. Error with output "No such device" is ignored.
Signed-off-by: Rakshith R<EMAIL_ADDRESS>(cherry picked from commit 24fa9ed935b3287acaf2867c58d015f5addd1b43)

Description of your changes:
Which issue is resolved by this Pull Request:
Resolves #
Checklist:

[ ] Commit Message Formatting: Commit titles and messages follow guidelines in the developer guide).
[ ] Skip Tests for Docs: If this is only a documentation change, add the label skip-ci on the PR.
[ ] Reviewed the developer guide on Submitting a Pull Request
[ ] Pending release notes updated with breaking and/or notable changes for the next minor release.
[ ] Documentation has been updated, if necessary.
[ ] Unit tests have been added, if necessary.
[ ] Integration tests have been added, if necessary.

/cc @travisn @BlaineEXE @agarwal-mudit

/lgtm
/approve

/cheery-pick release-4.11

/cherry-pick release-4.11


Voice input and output

enhancement
i can talk to  Bard really well with a good cardioid mic , endless long prompts, 99% almost.  even about whatever.
But i would like to command query and complete against a certain Domain specific language in context .
Microsoft is doing a Windows Copilot but i have my doubts..   I'll just leave this idea here in case they don't do it , its been suggested by people they should do this..
this might be out of scope but its seems this  can act as a plugin to a tool ur control apps supporting Automation/ plugins, and for OS, you can look in the folders, scrape apis with Silk or however the do it..
The best one is https://github.com/omlins/JustSayIt.jl    because its low latency.. you should be able to use any UI hands free.
https://github.com/omlins/JustSayItRecipes.jl/blob/main/recipes/ubuntu_max.jl   i think you can run that from c# but deploying it .. i donno, might be difficult.
I don't know how Whisper is but its doesnt   seems Realtime  or fast enough from the docs.
If you can limit the domain say vs an API or App or Language automation or object model,  or act as a tool plugin it can better do completion and voice recognitions as its doing that together.   but context should be able  provided by something other than prior words, allowing to scrape user commands, apis, and apply  synonyms and user defined as a last resort.
Simplex voice and no look operation like Query . ..command, and interrupt , repeat, copy that.  do you copy? "Over, or Send ", means send it.   visual studio has that nice code feature search, it could start with that...
there are a lot of plugins for tools but using Automation / com /c# or reflection, can help allot in accuracy.
Microsoft Voice Access leaves a lot to be desired and if done properly it could be voice and hands free for at least debugging and operating UI and menus and command could be if coding its too much..   But they are taking the approach of running the mouse and its pretty bad.
a Mic could the start menu for any OS and any App.  using the context given by a hotword...     "Windows"  <-hot,  Launch Visual Stu   "Alpha"    <-- choice tagged on auto-completion
Voice Access is using 1, 2, 3.. etc..   but its too easy to confuse with Class1, etc..
choose among sets of hotwords:
define custom hot words , visual markers Tokens like  Alpha , Bravo , Charlie, for choices, and  say" Over" for "Send", or use  Radio Coms protocols
this is on the wishlist  of the gpt4all

Plugin support for langchain other developer tools
Text to speech - have the AI response with voice
Speech to text - give the prompt with your voice

If you provide C# .NET audio input solution that works cross-platform (Windows, macOS, Linux, Android, iOS and wasm) and speech to text that's also has good api for C# .NET and works cross-platform (Windows, macOS, Linux, Android, iOS and wasm) I might consider, otherwise it's huge work and would need to be heavily sponsored or submitted via code contribution. Also note someone has to maintain and update it, otherwise it's huge maintenance burden.

well keep an eye out .. MS voice input Voice Access ,  is getting  pretty good, maybe its learning and its real time.  . but well see if they scape APIs for context, fix Start and do all this .. they really should.. Julia one is very linux oriented.. it only workd on android if you run linux on it.  but the voice input module in JustSayit.il  is separate.. Key is a 100 $ sm-58 rockstar mic, a sv 1000 or cardioid or studio mic with preamp helps alot to filter out fans, noises , a near field mic  is super important .  Bard is best at it because the prediction / typo fixing is aligned with it voice to text typo fix AI + context.    If they get completion right the , mix witht the code/feature search they go from the worst design by commit mess UI to the best and simplest assistant.

helpful generalized spell checker in one prompt, also a way to get choice from the voice access at least on windows inot a postprocess and a choice:     still needs a collaboration.  the OCR part @killian i think did some..,   but on widows could be useful , i my try to combite their work and chain some agents but someone will do it..
pico voice said my idea was SPAM and they have a useless slow laggy product and dont know have is a dsp or FPGA.  or how to do hotwords fast enough.   but this helps.  at least with universal spell check.  menu context completing
so having tried the they cause million of hours of duplicate work.
but the key is a good dynamic mic.
https://twitter.com/PAF_Kontrol/status/1758368591911530924
You are a helpful assistant named TypoFixerDictionationOrMenuPicker.  for a long input,  just fix typos in my text, make no major grammar or wording changes or reinterpretations.  on a second pass, if it appears  that  there is  the wrong word or acronym that's out of  context you can switch the word,
Output the fixed text, then
give a numbered list of the changed typos, then the changed words on the second pass.    For acronyms if they don't fix in context include those..    contains  a  list like     [ Open File, File Close, Open Last Workspace],   just choose the closest match    for the word or words following the list.  then output that as a "Change Context and Command is"   , then give the likely match