dockerfile code is not working

plz check dockerfile code universal-resolver doesn't have implementation folder now

I've tried updating the docker file to match the structure of the universal-resolver repo:
RUN cd /opt/universal-resolver && mvn clean
RUN cd /opt/universal-resolver && mvn install -N -DskipTests
RUN cd /opt/universal-resolver/uni-resolver-core && mvn install -N -DskipTests
RUN cd /opt/universal-resolver/uni-resolver-client && mvn install -N -DskipTests

However the build is now failing with:
#12 37.03 Usage: javac <options> <source files>
#12 37.03 use -help for a list of possible options
#12 37.03
#12 37.03 [INFO] 1 error
#12 37.03 [INFO] -------------------------------------------------------------
#12 37.03 [INFO] ------------------------------------------------------------------------
#12 37.03 [INFO] BUILD FAILURE
#12 37.03 [INFO] ------------------------------------------------------------------------
#12 37.03 [INFO] Total time: 35.719 s
#12 37.03 [INFO] Finished at: 2022-01-24T17:13:16+00:00
#12 37.10 [INFO] Final Memory: 17M/294M
#12 37.10 [INFO] ------------------------------------------------------------------------
#12 37.11 [ERROR] Failed to execute goal org.apache.maven.plugins:maven-compiler-plugin:3.8.1:compile (default-compile) on project uni-resolver-core: Compilation failure
#12 37.11 [ERROR] javac: invalid flag: --release
#12 37.11 [ERROR] Usage: javac <options> <source files>
#12 37.11 [ERROR] use -help for a list of possible options
#12 37.11 [ERROR] -> [Help 1]
#12 37.11 [ERROR]
#12 37.11 [ERROR] To see the full stack trace of the errors, re-run Maven with the -e switch.
#12 37.11 [ERROR] Re-run Maven using the -X switch to enable full debug logging.
#12 37.11 [ERROR]
#12 37.11 [ERROR] For more information about the errors and possible solutions, please read the following articles:
#12 37.11 [ERROR] [Help 1] http://cwiki.apache.org/confluence/display/MAVEN/MojoFailureException
------
executor failed running [/bin/sh -c cd /opt/universal-resolver/uni-resolver-core && mvn install -N -DskipTests]: exit code: 1

@peacekeeper, Any hints?  I have not dug into it yet.


[Issue] URGENT: CompareExchangeHashes.ps1 does not detect hidden system files

I manually found a VERY MALICIOUS file here on a hafnium exploited exchange 2019 server:
C:\Program Files\Microsoft\Exchange Server\V15\FrontEnd\HttpProxy\owa\auth\OutlookEN.aspx
This file has this attribute list: RHSA  so is a HIDDEN SYSTEM file. The CompareExchangeHashes.ps1 script does NOT SEE this file because of the SYSTEM attribute set on the file.
Expected behavior
CompareExchangeHashes.ps1 should detect and report on extra or modified files with the "System" attribute.
Script Output
The CompareExchangeHashes.ps1 does NOT report that this EXTRA file is present. This is bad, and leads to a false sense of security after the CompareExchangeHashes.ps1 script has been run and shows "no extra or modified files detected".
Additional context
This is a very urgent issue with the CompareExchangeHashes.ps1 script. It is NOT fully working to actually detect malicious files present in the exchange installation folder.

This script is deprecated. Please use EOMT.ps1 script.
Please read the readme file here: https://github.com/microsoft/CSS-Exchange/tree/main/Security

tagging @dbacon247 to help with the file found.

EOMT does not check for additional files nor modified files unless the malware definitions have been updated with all known variations of the malware, which this is WAY to early to do that. Like in this case, EOMT.ps1 does NOT DETECT THIS MALICOUS FILE that I found in the exchange install directory.
You should not deprecate CompareExchangeHashes.ps1 this early, you are leaving many people vulnerable, and with active infections of 0-day malware. And your giving a false sense of security from the EOMT.ps1 script.


Expose option to set User Agent Header

I'd like to request that opendal expose an option to set a User Agent header from calling libraries.
This request is related to https://github.com/mozilla/sccache/issues/2136, which explains the desired use case.

thanks! I will close this issue.


753 - Samples table: Change 'Other Modalities' to 'Modalities'

Issue Number
Closing #753
Purpose/Implementation Notes
The following changes are included in this PR:

Changed the name of the sample colum from "Other Modalities" to "Modalities"
Prefixed the above column values with "Single-cell" and removed the default "NA" value

@dvenprasad Please see the latest UI here.
Types of changes

Refactor (addresses code organization and design mentioned in corresponding issue)

Functional tests
N/A
Checklist

[x] Lint and unit tests pass locally with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)
[ ] Any dependent changes have been merged and published in downstream modules

Screenshots
N/A

Single-cell and Bulk RNA_seq are two different modalities. There should be a comma between them.

Hey Deepa, I've removed the extra prefix, pls take a look here.
About multiplexed, are we going to remove the "Multiplexed" here?


Reload function for HoLC

Resolves #1280
Main change:

Reload current file function

Additional changes:

Added a menubar
Added a "close" menu action

I want to wait with this one until after my mega PR that adds component library exorts. it changes some things in holc that will likely conflict slightly with this code. And thus one will be easier to fix if necessary.


daily view not properly matching 'days of week' and 'day of month numbers'

When Monday is set as the first day of the week, then everything is good when I toggle between 'monthly' and 'daily' view.  However, when Sunday is set as the first day of the week, the 'daily' view is incorrect.  In 'daily' view, the days-of-the-week do not properly match the days-of-the-month.  They are shifted by one day.

Thanks for reporting, I'll take a look

Hi, I think it is fixed now. Thanks again for reporting.


Unexpected driver error occurred while connecting to the database      java.lang.NullPointerException

System information:
Windows 10 (64)
DBeaver CE version
21.3.2
Java Version 8 Update 321
Connection specification:
Database name and version:
Snowflake
Driver name:
net.snowflake:snowflake-jdbc:RELEASE
mssql-jdbc-9.4.1.jre16.jar
Do you use tunnels or proxies (SSH, SOCKS, etc)?
No, I don't use it
I'm trying to connect to the Snowflake database. DBeaver offers to download and update the driver. I do it. Then he offers to do it again. By default, it offers to install the file version 3.13.6, but I tried to install both it and the latest version. I also tried to download the drivers manually and add them, but the error repeats. The Java version is the latest. I tried reinstalling Java, but the error persists.

mssql-jdbc-9.4.1.jre16.jar

This driver is for use with Java 16.
Could you try again with mssql-jdbc-9.4.1.jre8.jar?

mssql-jdbc-9.4.1.jre16.jar

That's the driver for Java 16 runtime.
DBeaver Community Edition 21.3.2 (https://dbeaver.io/files/21.3.2/dbeaver-ce-21.3.2-win32.win32.x86_64.zip) is bundled with Java 11 runtime.
Might be worth to try again with mssql-jdbc-9.4.1.jre8.jar.

I've tried with mssql-jdbc-9.4.1.jre8.jar . The error is the same. Also I tried other versions of dbeaver, but the error is the same.

Hello @elinazelentsova
The whole error log will be beneficial.
And what is the purpose of mssql-jdbc driver? This is the driver for SQL Server.
It's ok to use only the net.snowflake:snowflake-jdbc:RELEASE driver for the Snowflake connection. Unnecessary libraries in the list of drivers libraries can lead to errors in connection.

There has been no update on the issue for a long time. if it is still actual - feel free to reopen it

This happened to me due to the wrong host URL. Dbeaver expects the host to be in the following format:
accountId.region.cloud.snowflakecomputing.com

Example: accId.west-us-2.azure.snowflakecomputing.com

Not sure about the error.
I used something like
accountId.region.cloud.snowflakecomputing.com/
today, and I had another issue like "incorrect URL", but not like the main topic of this issue.


sunodo shell should provide an option to define the shell to use

📚 Context
The shell that's called by sunodo shell is hardcoded to /bin/bash.
It should be possible to define another shell.
✔ Solution
Default to /bin/bash and add a --shell argument to define a different one.
sundo shell  --shell=/bin/sh

It should be possible to define another shell.

Can you describe why it is needed ?
as far as I know this sub-command starts a shell in cartesi machine of application that is a docker container that has bash. So, why should we consider other shells ?

I'm working on a small image with busybox, and face the problem of not having /bin/bash in place.
I could fix it easily by creating a symlink, but It felt weird having to do this.
We could have a default and allow to change for whatever shell (busybox sh, /bin/sh, '/bin/zsh...) or at least document somewhere that the image needs a /bin/bash in place.
Also, my suggestion looks weird reading this afterward, this shell --shell repetition is odd.

Also, my suggestion looks weird reading this afterward, this shell --shell repetition is odd.

could use --shell-path flag.
I'll send a PR.

Why don't we provide an option to exec something in the cartesi machine, instead of having a specific command for shell?
Something like sunodo exec instead of sunodo shell.
sunodo exec can be confused with sunodo run, but I think we can think about it.
An interactive flag may also be necessary.

interactive

U mean Something like this ?
sunodo exec --command ls -la

I mean sunodo exec /bin/sh

I mean sunodo exec /bin/sh

My worry with exec is the confusion with the docker exec, where it's expected that the command is going to be executed in an already running container.

I mean sunodo exec /bin/sh

My worry with exec is the confusion with the docker exec, where it's expected that the command is going to be executed in an already running container.

Yeah, but sunodo run is already taken for the node execution.

I mean sunodo exec /bin/sh

I'm going to send a PR for it.

Is this customization really necessary?
Can't we have a shell that we know will always be there?
I want to minimize as much as possible new options and new commands. To keep sunodo as simple as possible for users.

We could have this as a requirement somewhere. After we start using the cruntime (#455) we won't need this and we can control what's inside the main rootfs.

So I guess we can wait until cruntime is ready, and not implement this.

We need a better development workflow written down somewhere so that collaborators don't start working on things that were not discussed and got a tag or milestone that defines a issue as good-to-go.

A simple label can take care of that.

A simple label can take care of that.

triage is popular one. Meaning: "people, we need to discuss this further, before implementing it"


ci: update GH actions

updates all GH actions/* to non-deprecated/unmaintained/outdated up-to-date versions
Also moving to dtolnay/rust-toolchain from actions-rs/toolchain since it is pretty much abandoned.

In my fork, all test passes and emits no CI warnings from deprecated stuff

@TonyGiorgio what am I missing? Is this PR rejected because the codebase is only supposed to be touched by MutinyWallet org members? Or are there technical issues with the contribution?

Is this PR rejected because the codebase is only supposed to be touched by MutinyWallet org members

Contributions are welcomed but not random unknown CI changes from random first time contributors that aren't actually fixing anything. No offense.


⚠ Fortnite has degraded performance

In ac468e8, Fortnite (https://fortnite.com) experienced degraded performance:

HTTP code: 200
Response time: 1463 ms

Resolved: Fortnite performance has improved in cc3f8a8.


doom-dashboard not loaded on start

Description
When I start Emacs the doom-dashboard is not loaded initially.
This is my Emacs after the initial start:

After opening Emacs messages, it seems that the dashboard has been reloaded correctly:

Any ideas why I run into this problem?
Log output:
Loading /nix/store/a94rvycbpi6lhshrhl45r6dban6x4jpb-emacs-with-packages-27.2/share/emacs/site-lisp/site-start.el (source)...done
Loading /nix/store/9hawkcaxsvqj98w70l4haasfip8rkaq7-emacs-packages-deps/share/emacs/site-lisp/default.el (source)...
doom-emacs is not placed in ‘doom-private-dir’,
loading from ‘site-lisp’
Loading /nix/store/pa2lan4qrxsnkd0p8xv1a3d0n7dxcsp3-doom-emacs/early-init.el (source)...done
Loading /nix/store/pa2lan4qrxsnkd0p8xv1a3d0n7dxcsp3-doom-emacs/init.el (source)...
DOOM Initializing Doom
Debug mode on
DOOM New variables available: (use-package-verbose)
Debug mode off
Debug mode on
DOOM Initializing core modules
DOOM Initializing user config
Loading /nix/store/r6aig18jvyqpj034592c4pp54ykw5z66-doom.d/config.el (source)...done
Loading /nix/store/691xda5bkx31c6jqqlsxw4phwscrcp14-doom-private/config.extra.el (source)...done
Loading /nix/store/pa2lan4qrxsnkd0p8xv1a3d0n7dxcsp3-doom-emacs/init.el (source)...done
Loading /nix/store/9hawkcaxsvqj98w70l4haasfip8rkaq7-emacs-packages-deps/share/emacs/site-lisp/default.el (source)...done
Doom loaded 146 packages across 30 modules in 0.608s
DOOM Loading deferred package server from doom--after-call-server-h
DOOM Incrementally loading tree-widget
DOOM Incrementally loading recentf
DOOM Incrementally loading savehist
DOOM Incrementally loading yasnippet
DOOM [editor/evil] Initialized evil-collection-elisp-mode
DOOM [editor/evil] Initialized evil-collection-comint
DOOM [editor/evil] Initialized evil-collection-custom
[yas] Check your `yas-snippet-dirs': /nix/store/691xda5bkx31c6jqqlsxw4phwscrcp14-doom-private/snippets/ is not a directory
[yas] Prepared just-in-time loading of snippets with some errors.  Check *Messages*.
DOOM Incrementally loading with-editor
DOOM Incrementally loading git-commit
DOOM [editor/evil] Initialized evil-collection-epa
DOOM [editor/evil] Initialized evil-collection-dired
DOOM [editor/evil] Initialized evil-collection-log-edit
DOOM Incrementally loading package
DOOM [editor/evil] Initialized evil-collection-package-menu
DOOM Incrementally loading magit
DOOM [editor/evil] Initialized evil-collection-diff-mode
DOOM [editor/evil] Initialized evil-collection-imenu
Loading /nix/store/3lpkncgg0cicmxzd3jfxg52piki7s3cp-straight-emacs-env/straight/repos/magit/magit-version.el (source)...done
DOOM [editor/evil] Initialized evil-collection-magit
DOOM [editor/evil] Initialized evil-collection-calendar  [2 times]
DOOM [editor/evil] Initialized evil-collection-org
DOOM [editor/evil] Initialized evil-collection-compile
DOOM [editor/evil] Initialized evil-collection-grep
DOOM New variables available: (async-debug)
Debug mode off
Debug mode on
DOOM [editor/evil] Initialized evil-collection-magit-todos
DOOM Incrementally loading org-agenda
DOOM Incrementally loading org-capture
DOOM [editor/evil] Loading deferred package evil-easymotion from doom--after-call-evil-easymotion-h
DOOM [completion/vertico] Loading deferred package orderless from doom--after-call-orderless-h
DOOM [editor/evil] Initialized evil-collection-vertico
DOOM [editor/evil] Initialized evil-collection-which-key
DOOM [ui/doom-dashboard] Reloading dashboard at 09:43:30
DOOM [ui/doom-dashboard] Changed dashboard’s PWD to ~/

Configuration
flake.nix
(https://github.com/stefanDeveloper/nixos-lenovo-config/blob/master/flake.nix)
emacs.nix
{ pkgs, inputs, config, ... } :

{
  home-manager.users.stefan = {

    programs.emacs = {
      enable = true;
    };

    imports = [ inputs.nix-doom-emacs.hmModule ];
    programs.doom-emacs = {
      enable = true;
      doomPrivateDir = "${config.users.users.stefan.home}/doom.d";
    };
  };
}

Metadata
Please run nix-shell -p nix-info --run "nix-info -m" and paste the result.
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.10.105, NixOS, 21.11 (Porcupine)`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.5.0pre20211206_d1aaa7e`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

Missed this in the GH notification sea. Does this happen with Doom's normal installation method? (git-clone to .emacs.d)
I use Emacs in daemon mode which doesn't seem to have this problem but this is probably just caused by us not respecting Doom's version pins which makes some things break sometimes.

I can confirm that the Doom's normal installation method works just fine with my NixOS setup.

Alright well, if you need help debugging this I'm @ckie:ckie.dev on Matrix and in the NixOS space. Otherwise I don't have much to say since I'm not currently motivated to work on this.

I am also running into this, I get the scratch layer loaded on startup. Hitting : then ESC reloads the doom dashboard.
This is just a guess, but I think this line might be the issue:
`doom-emacs is not placed in ‘doom-private-dir’.

@Pacman99 thanks for your input.
Soon I'll be working on it and try to fix it. @ckiee I'll contact you, thanks for offering your support. :)

I am also running into this, I get the scratch layer loaded on startup. Hitting : then ESC reloads the doom dashboard.
This is just a guess, but I think this line might be the issue: doom-emacs is not placed in ‘doom-private-dir.
My config can be found here: https://gitlab.com/coffeetables/lower/-/tree/master/home/profiles/apps/emacs

Same issue here.
After some git bisecting, I found a good revision to run, but this line showed as well. I think this line may be irrelevant.
Here is git bisect log:
git bisect start
# bad: [8e818ce70f399fcb11211296cf5bcb332203d7f2] flake.lock: Updating 'flake-utils (numtide/flake-utils)' - 3cecb5b0 -> a4b154eb (#110)
git bisect bad 8e818ce70f399fcb11211296cf5bcb332203d7f2
# bad: [5cf6eb2bfe12d0d6e6b42c897f4878efa9dba8c1] flake.lock: Updating 'org (emacs-straight/org-mode)' - 22e6ed6b -> e7ea951a (#51)
git bisect bad 5cf6eb2bfe12d0d6e6b42c897f4878efa9dba8c1
# good: [fee14d217b7a911aad507679dafbeaa8c1ebf5ff] home-manager: set `services.emacs.package` only if it is defined
git bisect good fee14d217b7a911aad507679dafbeaa8c1ebf5ff
# bad: [19b494b62dc756a8446b5476a79bf23c3d0d8041] flake.nix: run nixfmt
git bisect bad 19b494b62dc756a8446b5476a79bf23c3d0d8041
# bad: [e3517b0e001f55ab6fc42f3f44f291661c2edc10] last commit again
git bisect bad e3517b0e001f55ab6fc42f3f44f291661c2edc10
# bad: [0cb69d4c2098028bc6a77c56f8f64802bb5df44c] update references: vlaci → nix-community
git bisect bad 0cb69d4c2098028bc6a77c56f8f64802bb5df44c
# good: [6b3bfba4c8bd5bdbca7783275a45e9a14d1dda62] advice: disable doom-cli--ci-deploy-hooks
git bisect good 6b3bfba4c8bd5bdbca7783275a45e9a14d1dda62
# bad: [29688ccca571d28db89456da0bc109f55a50e9fc] nix-integration: update patch
git bisect bad 29688ccca571d28db89456da0bc109f55a50e9fc
# bad: [258b70a44105a1abf40a03e04b4be1fe0c745e7b] flake.lock: update
git bisect bad 258b70a44105a1abf40a03e04b4be1fe0c745e7b
# first bad commit: [258b70a44105a1abf40a03e04b4be1fe0c745e7b] flake.lock: update

This commit https://github.com/hlissner/doom-emacs/commit/114841bfd211ce90fb22eda31508a602db8bdf1a delay the call of doom-init-ui-h until the hook 'window-buffer-change-functions which seems not being triggered during the init process. And that's why 'Hitting : then ESC reloads the doom dashboard.': the buffer changed.
According to the author's comment
;; Initialize UI as late as possible. `window-buffer-change-functions' runs
;; once, when the scratch/dashboard buffer is first displayed.
(add-hook 'window-buffer-change-functions #'doom-init-ui-h -100)

The hook 'window-buffer-change-functions should be triggered once the scratch buffer is first diaplayed, but it did not.
A temporary fix would be adding this to your config.el
(add-hook! 'emacs-startup-hook #'doom-init-ui-h)

Are you on  darwin? I've observed this issue myself but only on my darwin machines, on NixOS I can't reproduce it.

I have this issue on NixOS

A temporary fix would be adding this to your config.el
(add-hook! 'emacs-startup-hook #'doom-init-ui-h)

Just tested this and it fixes the issue, thanks!

This PR seems to fix the issue if you're running Emacs 29+ according to my tests: https://github.com/nix-community/nix-doom-emacs/pull/191
The only reason I can think is that maybe doom-dashboard depends on early-init.el to be loaded at, well, during the early init phase, however we used to load it from site-lisp/default.el before.
Going to say though, this was completely accidental and it was not the objective of the PR (I don't even use doom-dashboard myself :sweat_smile: ).

Closing this since my tests on https://github.com/nix-community/nix-doom-emacs/pull/191 shows that this issue is fixed.
If it is not, please answer this thread and I will reopen it. Keep in mind that the fix in #191 only works in Emacs 29+, so if you're still having this issue check if you're using emacsGcc or something similar first.

Getting this as well with the instructions specified on the README.
I'm on NixOS WSL


Cannot see cursor on white background - should have an invert option

Environment
Windows build number: Microsoft Windows [Version 10.0.19041.388]
Windows Terminal version (if applicable): Version: 1.1.2021.0

Steps to reproduce
I have read this post:
https://github.com/microsoft/terminal/issues/1203#issuecomment-632888374
Apparently I am not smart enough to figure this out.
What I don't understand is how to set my cursor to just simply invert the FG and BG?
To me it is silly to set one of them to a fixed colour, and set the other to a foreground of background, cause at some point you will end up with the same colour FG and BG, and then cant read the text.
I use VIM, and I am tired of when the cursor is on white text, I have to guess where I am, type a few chars, delete them, move a little, try again.
I can't find where in the docs does cursorTextColor go (I think its not in the docs)? Under profiles? Under schemes, so I added it to both.
E.g. I have tried various combinations of cursorColor, cursorTextColor set to textForeground, textBackground, and null
Actual behavior
White text on white background
Expected behavior
It seems impossible to have the following (should be default scenario):
Case 1: White text on Black background -> Cursor is Black text on White background
Case 2: Black text on White background -> Cursor is White text on Black background
I must be missing the obvious here?

Alas! You've unfortunately stumbled into one of our known-bad rendering issues. Sorry about that.
I just had a trip over the repo, and it seems like we did lose track of the request that the cursor simply invert. More info in the spec #6151.

I'm totally set back by this unreadability of cursor text.

@bohrshaw while it's not as flexible as an inverted cursor, you could always set the cursorColor to something other than white

while it's not as flexible as an inverted cursor, you could always set the cursorColor to something other than white

@zadjii-msft But as long as there is no cursorTextColor (#6161) setting you will still have unreadable chars.

That's correct. Like I said, it's not as flexible, but it's certainly a helpful reminder for people who like a white background for their terminal.

That's correct. Like I said, it's not as flexible, but it's certainly a helpful reminder for people who like a white background for their terminal.

The thing is, when there are popups, or modals, the colors are often inverted to the normal working colour. I prefer darktheme like most terminal uses. But there are times when the background becomes white beyond my control.

That's correct. Like I said, it's not as flexible, but it's certainly a helpful reminder for people who like a white background for their terminal.

In practice it's still a deal breaker for anyone using terminal Vim because every color is going to clash with how your Vim theme colors up keywords. Even with a dark theme you still run into scenarios where the cursor hides the character under it.
The only workaround I've seen so far is to use the vintage cursor but this has a lot of limitations around dynamically setting the cursor depending on which Vim mode you're in because a thick underline isn't a standard cursor but a block cursor is.

@DHowett Hey there, found this issue since I'm still encountering it. What is the most up to date ticket on this?

I think not being able to see the text should be prioritised over fancy effects. Personally I have changed to linux since opening this issue. It was a hard transition having only ever used windows before, but I have never looked back was I perfected my set up.

I think not being able to see the text should be prioritised over fancy effects. Personally I have changed to linux since opening this issue. It was a hard transition having only ever used windows before, but I have never looked back was I perfected my set up.

I've been considering it, but my biggest thing has been that I have lots of windows only software (including games). I may end up just having to get another drive just for Linux dual booting.
Either way, would love to see this fixed!

FYI you'll find that the new Windows Terminal Preview 1.18 will have practically fixed this issue, even if it doesn't implement inverted cursors just yet. I intend to add an invert cursor option in 1.19.
(Note however that 1.18 has a known issue with a small number of fonts. If you're affected by this, I've put a temporary workaround here: https://github.com/microsoft/terminal/issues/15199#issuecomment-1562785741.)

I just realized that I once again mixed up invert and reverse. Sheesh. Invert is 1-color whereas reverse is swap(foreground, background) and I mix them up every single time.
Windows Terminal Preview 1.18 has an invert option: Just set the cursor color to #ffffffff. The text renderer will ensure that the cursor text will always remain visible. I'll use this issue to track invert and explicit cursor text colors, but technically, at least going by the title, we can close it.

3yrs later, I still can't see the cursor. I switched to Linux, but have recently returned to using Windows due to work laptop. The version in the app store in v.117 and one still can't see the cursor. Amazing.

@mangelozzi The comment right above yours:

The version in the app store in v.117 and one still can't see the cursor

I'm not sure if you're intentionally ignoring parts of what I wrote so I'll mark your comments and my responses as off-topic, because at this point I'm starting to be a little bit annoyed by this.
Just to be entirely clear: Right now the feature is available in "Windows Terminal Preview". Not "Windows Terminal". You will find it in the "Microsoft Store" (= app store) and it will be version 1.18.
In a few days 1.18 will be available as non-Preview and you'll be able to use the new AtlasEngine with the fixed cursors there if you manually opt into it (it's off by default in non-Preview and enabled by default in Preview).

Since we now have cursor inversion and a forced contrast distance for solid cursor colors, I'm going to change the priority on this bug to 2 and move it to the 1.23 milestone.


[Bug] [Metadata] PostgreSql get DDL script error

Search before asking

[X] I had searched in the issues and found no similar issues.

What happened
获取 pgsql 的 ddl 时，生成的 ddl 语句和源数据库的 ddl 不一致，如下图

问题如下：

not null 属性错位了
no primary key
create table 语句末尾截取异常

When getting the DDL of pgsql, the generated DDL statements are not consistent with the DDL of the source database, as shown in the following figure:

The questions are as follows:

The not null property is misplaced
no primary key
The exception is truncated at the end of the create table statement

What you expected to happen
Generate the correct pgsql DDL
How to reproduce
Just get the DDL for pgsql
Anything else
No response
Version
dev
Are you willing to submit PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

https://github.com/DataLinkDC/dinky/blob/dev/dinky-metadata/dinky-metadata-postgresql/src/main/java/org/dinky/metadata/query/PostgreSqlQuery.java  看下这里的代码   重新打包

dev or 0.7.2 ?

dev,我这边已经修复好了，这两天会把修复好的PR上去

我这边刚拉取了下最新的代码，还是存在问题

我后续还在这个 issues 下面提交PR？

我这边刚拉取了下最新的代码，还是存在问题
我后续还在这个 issues 下面提交PR？

我把修复代码提交的 0.7 分支了


Uncaught Error: Module name "uniwheel" has not been loaded yet for context

Uncaught Error: Module name "uniwheel" has not been loaded yet for context: _. Use require([])
http://requirejs.org/docs/errors.html#notloaded
makeError @ require.js:166
localRequire @ require.js:1407
requirejs @ require.js:1752
(anonymous function) @ svg-pan-zoom.js:1

The limitation relates to the simplified CommonJS syntax vs. the normal callback syntax:
http://requirejs.org/docs/whyamd.html#commonjscompat
https://github.com/jrburke/requirejs/wiki/Differences-between-the-simplified-CommonJS-wrapper-and-standard-AMD-define
Loading a module is inherently an asynchronous process due to the unknown timing of downloading it. However, RequireJS in emulation of the server-side CommonJS spec tries to give you a simplified syntax. When you do something like this:
var foomodule = require('foo');
// do something with fooModule

What's happening behind the scenes is that RequireJS is looking at the body of your function code and parsing out that you need 'foo' and loading it prior to your function execution. However, when a variable or anything other than a simple string, such as your example...
var module = require(path); // Call RequireJS require
...
then Require is unable to parse this out and automatically convert it. The solution is to convert to the callback syntax;
var moduleName = 'foo';
require([moduleName], function(fooModule){
    // do something with fooModule
})

Given the above hopefully you can solve the issue.

When using Require.js you should load the compiled library (and not the source files).
Check Require.js demo and the source file.


[fix] Fix tag of default unpack image

The default tag for unpack image is currently "latest". Based on the recently built images in quay:
https://quay.io/repository/operator-framework/rukpak?tab=tags it looks like the "latest" tag does not exist. Instead "main" is used. This PR fixes the same.

cc: @joelanford

I think the idea we're actually going for is "refer to a local-only development tag in all the places" because that's actually what we build and use almost exclusively when interacting with rukpak via this source code repository. We never actually push it this local image anywhere, so the intent is to avoid using a name that we push (or that sounds like something that would be pushed).
So I'd suggest:

Changing latest to devel
Except in the default unpack image, where I think it's okay to leave main.

For 2, I'm thinking of doing a follow-up to further change that to look for and use a git tag, and if not set, then fall back to main. IIRC, we do something similar in operator-sdk when we scaffold Ansible and Helm project Dockerfiles.


Add note box to dontaion page

Request to add a note to the donation.

Note added to donation line on process payment, this works for all donations.


Compile to assembly on linux dotnet core?

Hi,
How would I go about compiling a python source to a dotnet core assembly, using IronPython 2.7.8 on linux? I can't find any pyc.exe or pyc.py in the zipped release.

At this time there is no support for compiling assemblies using .NET Core. There are some APIs relating to saving of assemblies that are missing from .NET Core.
See: https://github.com/dotnet/corefx/issues/4491

Ok, thanks.
Can an assembly be built from Windows, targetting .NET standard 2.0, and
then used on linux dotnet core? How would one build such an assembly using
the Windows tooling?
Thanks.
On Fri, Mar 9, 2018 at 9:02 AM slozier<EMAIL_ADDRESS>wrote:

At this time there is no support for compiling assemblies using .NET Core.
There are some APIs relating to saving of assemblies that are missing from
.NET Core.
See: dotnet/corefx#4491 https://github.com/dotnet/corefx/issues/4491
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/IronLanguages/ironpython2/issues/351#issuecomment-371820084,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AA-GPKrZOKZ3BV9rPdD4pPr6VURrRcuNks5tcotjgaJpZM4Sjtf4
.

--
Simon Labrecque

I'm not sure if this works, but you could try using clr.CompileModules to generate the assembly on Windows and then try to execute it with the .NET Core build of IronPython. See https://blogs.msdn.microsoft.com/srivatsn/2008/08/06/static-compilation-of-ironpython-scripts/ for info on CompileModules.

Ok, I'll try that eventually, and report back my findings. Thanks!

Reviving this... I've compiled a set of modules in IronPython hosted with net462, then I try to load them in a .NET Core 3.1 hosted IronPython, but no beans:
Could not load type 'System.Runtime.CompilerServices.Closure' from assembly 'System.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089'. (System.TypeLoadException)
Traceback (most recent call last):
  File "c:\code\HWBuild\support\hwlibs\Questa.hwlib.py", line 55, in final
SystemError: Could not load type 'System.Runtime.CompilerServices.Closure' from assembly 'System.Core, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=b77a5c561934e089'.```

Unless I'm doing something wrong...?

Unfortunately the System.Core facade of .NET Core does not type-forward System.Runtime.CompilerServices.Closure.

Looks like there's some movement in the ability to save assemblies in .NET 8 https://github.com/dotnet/runtime/issues/62956
In the meantime, is there any way one would be able to compile using NET4x but then run in .NET Core?  Perhaps removing the use of types like System.Runtime.CompilerServices.Closure may be able to get it to run.
This is a pretty big blocker for us to run precompiled ironpython modules and would appreciate any help here, or if someone could point us in the right direction where we might be able to make those changes.

In the meantime, is there any way one would be able to compile using NET4x but then run in .NET Core? Perhaps removing the use of types like System.Runtime.CompilerServices.Closure may be able to get it to run.

Unfortunately the Closure stuff is done under the hood so outside of somehow re-writing the DLL to pick up the type from another assembly I don't have any great ideas.

Hey @slozier, thanks so much for the feedback.  I'll explore the option of rewriting the assembly.
When you say "under the hood", do you mean something in .NET Framework?  What exactly is writing/generating that code?  I wonder how that code would look if it was generated in .net core. 🤔

@cwensley Yeah, as far as I know, the Closure type isn't used anywhere in IronPython or the DLR so it's not something we can just remove. I believe the .NET Framework is adding it when it's compiling the lambda expressions...
Anyway, I did try some DLL hackery on a very simple example and managed to get something to run on .NET 6:

I added the System.Runtime.CompilerServices.Closure type to the .NET version IronPython.dll.
Compiled the python code using clr.CompileModules.
Using dnSpy, I updated the ResolutionScope of the TypeRef of Closure to point to the IronPython and then saved the DLL. Changing the DLL so Closure comes from IronPython instead of System.Core is basically just changing one byte so probably don't even need dnSpy to do it.
Loaded up the assembly in .NET and it worked.

Thanks @slozier, that sounds very promising! At the very least we should be able to load precompiled python scripts when running in .NET Core by using something like Mono.Cecil to replace that type.  Hopefully it's the only one.


Added DPD Class

FYI @mdwhitten

Hey @cindyxie; I realize you probably weren't around for the meetings, but R&D has requested we develop in LabVIEW 2016 since they have to support back to 2016. Could you save these for LabVIEW 2016 instead?

@mdwhitten  Just converted them to LabVIEW 2016.

I suggest we review one pull request or the other first. I imagine any changes coming from one will apply to the other.

@cindyxie Let's go ahead and migrate all of the C# changes to LabVIEW. I think we've gotten it to a stable enough point to do so. Thanks!

@mdwhitten C# changes are migrated.

@mdwhitten , where do you install the Quick Drop Shortcut?

@mdwhitten , where do you install the Quick Drop Shortcut?

C:\Program Files\National Instruments\LabVIEW 2016\resource\dialog\QuickDrop\plugins
or for 32-bit
C:\Program Files (x86)\National Instruments\LabVIEW 2016\resource\dialog\QuickDrop\plugins

@mdwhitten , changes are made in this commit https://github.com/NISystemsEngineering/rfic-reference-design-libraries-labview/pull/30/commits/a11b872be1487a5dffd5272f5b589bd085ecdf5d

@cindyxie I noticed that your current DPD renewed branch does not have the commits that I made at the end of this pull request in it. I think that you did not sync to the latest before continuing your development. We need to merge these changes into your branch first.
@smooresni could you help her with this?

@cindyxie I noticed that your current DPD renewed branch does not have the commits that I made at the end of this pull request in it. I think that you did not sync to the latest before continuing your development. We need to merge these changes into your branch first.
@smooresni could you help her with this?

@mdwhitten sure thing. @cindyxie i attempted to merge master into your branch but there are several merge conflicts. please get with me sometime today so we can resolve them.

@mdwhitten @smooresni , thanks. I believe the merge conflicts are resolved.


Paths fail to plot when there is only 1 histogram bin

Very nice package! I'm learning stochastic calculus and this is a really cool tool!
I found a small bug:
The paths plotted by SPExplicit.draw() are colored to match the histogram bin that their final point falls into.
If we plot N=3 or fewer paths, then we set n_bins = int(np.sqrt(N)) <= 1, and then this logic which helps set the path colors:
bin_centers = 0.5 * (bins[:-1] + bins[1:])
col = bin_centers - min(bin_centers)  # scale values to interval [0,1]
col /= max(col)

results in a divide by zero
/Users/alexrubin/code/aleatory/aleatory/utils/utils.py:76: RuntimeWarning: invalid value encountered in divide
    col /= max(col)

because col == [0.].

It seems like the 3 lines of logic above could be replaced by col = np.linspace(0, 1, n_bins, endpoint=True).

Many thanks for spotting this! Will include this in the next release :-)

Change has been released in v.0.1.3 🎉


How to reproduce the ppl behaviours?

the figures show that the longer input the lower ppl it get.

but the table show that the longer input the larger ppl it get.

Hi @66RING! Thank you for your thoughtful concern.
The decrease in perplexity (ppl) is actually the expected behavior, as it indicates that the model is effectively utilizing the long context to improve language modeling. You can see this decreasing trend in both Table 1 and Figure 4 of our paper.
As for Table 2, it serves as an extreme case to demonstrate the potential of our method to extend context to a very large scale. The 96k and 128k models were trained using 2k context for 1k steps, similar to previous models. There is still significant room for improvement, such as increasing training steps or training context, to optimize performance. Our main point here is to show that the ppl is still very low, indicating that the 128k model can function normally.
Thank you for your attention to our work. I hope this explanation addresses your concern :-)


Allow deploying custom CNO image with openshift_cno_image

Like we do for OVN-Kube images.

@venkataanil just below this though, the relay code uses the same pattern though:
  - name: Wait for OVNK new master images to roll out
    command: oc rollout status -n {{ ovn_namespace }} ds/ovnkube-master

  - name: OVN SBDB block
    block:
    - name: Deploy ovnkube-sbdb-relay
      script: sbdb-relay.sh deploy

    - name: Wait for OVNK sbdb-relay images to roll out
      command: oc rollout status -n {{ ovn_namespace }} deployment.apps/ovnkube-sbdb-relay

    - name: Verify ovnkube-sbdb-relay connectivity
      script: sbdb-relay.sh verify
    when: openshift_toggle_ovn_relay|bool

I was only working on patching ovn and sno images. So I didn't touch openshift_toggle_ovn_relay part. I will report an issue in scale-ci-deploy to fix the tasks in "openshift_toggle_ovn_relay" block.

lgtm, I will close my PR.


Script for CI release flow to update Widgets version

Jira issue:
https://glia.atlassian.net/browse/MOB-1564
Additional info:
Screenshots:

!merge


Cut a new release support text-2 and servant-0.20

@rashadg1030 ping

@ocharles I too would like to see this. I fail to understand why cabal cannot resolve the dependencies with current required bounds though: is it simply due to the missing version restrictions on servant and text? lucid does not seem to be the problem as far as I can tell, but the dependency resolution log does not make any sense to me

@JonathanLorimer cheers! 🙌 Am using nix too so that works great!

@JonathanLorimer can't find your fork, will it be back or should i create one?

@sebastiant it's htmx-lucid on Hackage, I believe

@ocharles ahh, so the repository is now at https://github.com/JonathanLorimer/htmx
thanks :pray:


LiveChannelMonitor Service

This service creates the event requested in #144 , and also attaches a OnChannelOffline event.
However, the constructor for the service is fairly awkward in that it requires a List<JoinedChannel> as a list of channels to monitor. As such, I'll try to fix this prior to merging this PR. Just putting this up for discussion :smile:

This was the post I made over on the issue discussing this.
Looks good, just a couple recommendations I'd make:

Modify the namespace in each of the events from TwitchLib.Events.Services.FollowerService to TwitchLib.Events.Services.LiveChannelMonitor
Kind of nitpicky, but I would change the name from LiveChannelMonitor to LiveStreamMonitor, since the Twitch API differentiates the two (channel objects always exist (unless user is banned or something) and stream objects only exist while the stream is active.
I think the ClientId value should be accessible from TwitchAPI, that way we're always sourcing this value from the same place. (it looks like this needs to be applied to FollowerService as well).
As you mentioned, we probably want to pass in a list of strings that represent the channels/streams to watch.
Additionally, we also might want to accept a list of strings representing channel/user IDs as an overloaded constructor so that we account for the move from usernames -> user id's.
Another nitpicky thing, perhaps move the #Events region to at the top so that someone viewing the code would immediately know the events to subscribe to.

Other than that, looks good. Thanks a ton :)

I can add those fixes if you want, or you are welcome to take a stab. Let me know.

Ah oops, guess you can tell I copypasted a lot from the FollowerService lol.
Alright sure, I'll be trying to add those fixes tonight.
Thanks!

No proble, I do the same thing haha. All about consistency 👍

Alright, all of the namespace/typo stuff should be fixed now.
As far as the ClientID value goes, since it isn't used anywhere in the service itself, shouldn't we be ok internalizing it within TwitchAPI? I thought the only purpose of ClientID in the services was to override the existing ClientID if necessary.
Channels are also stored as strings now, which should make everything better lol.
Does BroadcasterOnlineAsync handle parsing userids, or do we need to validate/call a different function if userids are passed in?
Thanks!

Actually you're right. I think we'd want to default to the TwitchApi Client ID, but if give the user the ability to pass in a client ID that the service would use instead of the TwitchApi client ID.
This presents an interesting problem. Because both usernames and user ids are strings (per Twitch API specification), there's no clear way to differentiate between a user id and a username that happens to look like a user id. Indeed, a username could match a user id exactly. Ideally, we'd do what Twitch would want us to do and only accept user ids, but I'm guessing the majority of users of the library would want the ability to specify a username. I'm actually not quite sure what we should do.

This looks like an inconsistency on Twitch's end. The Get Channel Editors endpoint returns an array of users that have a numeric ID, not a string. Channel followers uses a string ID, channel subscribers is numeric. I'll make a post on the dev forum there asking why they aren't all numeric.

I spoke to a Twitch dev, and he said that all endpoints should be string (they're working on changing them), and that any implementation of their API should treat user ids as strings because other products may introduce user ids that are strings, and to be globally safe, they should all be strings. This obviously complicates making the username and user id differentiation.
I think Twitch would say that we should treat all inputs as user ids, period. But, obviously that's not very convienent for 95% of the users of the library.

Yeah, I agree, using IDs would be great, just not feasible with the info the API provides IMO. My bot has a crappy little points system, it has to use username because there's no way to get the ID of a mentioned user from the message itself. (Discord has a separate property in the response with the user object of everyone who was mentioned in the message, it's really handy.)
As much as I hate the idea, I don't know if there's a way other than multiple functions with names signifying whether the string parameter is an ID or a username.

Alright, are there any changes with the username/userid distinction we can address within the scope of this service, or should this be implemented elsewhere in the library? Unfortunately I'm familiar at all with the specifics of the API, so I don't really have anything to add here :(
Thanks!

So here is what I recommend we do:

Constructor of LiveStreamMonitor accepts simply checkIntervalSeconds and clientId.
We have two public methods, one called SetStreamsByUsername(List<string> usernames) and SetStreamsByUserId(List<string> userids)
If the user attempts to start the service without username(s) or userid(s) set, then we need to throw an exception and inform them, and to do this we need:
An exception designated for this purpose
In addition to the the methods listed above, we also need to distinguish between the two api endpoints that we want to hit (v5 for userids, v3 for usernames). To do this I recommend we make an enum as such:

enum StreamIdentifierType {
      Usernames,
      UserIds
}

In SetStreamsByUsernames, we'd set the usernames to a private list instance variable and we'd set a private instance variable to the enum value Username.
In SetStreamsByUserIds, we'd set the userids to a private list instance variable and we'd set a private instance variable tot he enum value UserIds.
Finally, when we go to check the actual stream status, we refer to the enum value we set to know which endpoint we should hit with the stream value (either username or userid).

This is a bit of work, but it should make it ultra clear which endpoint we should be querying so the user of the library doesnt screw it up.

Oops, looks like my previous comment didn't come through.
Sure, that all sounds good!
I'll take a crack at trying to implement some of this as well; not sure how far I'll be able to get though haha.
Should we be looking at a new PR for the v5 stuff, or should we just lump it all in with this PR?

This looks good and almost ready to merge. I'd just ask that you rebase and fix the merge conflicts that Github is saying exist and then we should be good.
Thanks again :)

Alright great! I think I've rebased and fixed all the merge conflicts already, so we hopefully should be good on that front. I'll try to polish up the exception handling/XML in the next couple days. However, I haven't actually implemented the v5 API stuff yet, since I'm not super familiar with the API itself; would you mind looking into that?
Also, no problem! I'm super happy to be able to contribute to such a great library :)


Unable to calculate from Japan to Canada

I tried calculating from Imabari port ( lon / lat = 132.9972 , 34.0667) to Vancouver ( lon / lat = -123.1207 , 49.2827)
I keep gettin "Invalid longitude and/or latitude." error
There are other tools that have similar issues:
https://www.distance.to/Vancouver,BC,CAN/Imabari,Ehime,JPN
http://ports.com/sea-route/#/?a=4758&b=295&c=Port of Imabari, Japan&d=Port of Metro Vancouver, Canada
anyone has any suggestion on how to fix this package, or am I doing something wrong?

Hi,
Are you using the latest version 1.4.1 ?

Yes, that's the version I was using

I just tied and I get this result :
>>> import searoute as sr
>>> origin = 132.9972 , 34.0667
>>> dest = -123.1207 , 49.2827
>>> sr.searoute(origin, dest)
{"geometry": {"coordinates": [[133.085632, 34.070862], [133.634949, 34.332096], [134.005737, 34.418239], [134.393005, 34.388779], [134.859924, 34.098159], [135.1, 33.2], [136.598133, 33.846445], [137.197266, 34.325292], [137.857074, 34.385927], [138.139109, 34.436354], [138.421143, 34.486752], [138.60434, 34.44684], [138.787537, 34.40691], [139.191284, 34.655804], [139.3, 35], [139.9, 34.8], [141.3, 35.9], [145.5292, 38.0301], [150, 40], [164.183239, 44.859607], [180, 50], [180, 50], [183.3, 51.6], [188.444846, 51.295103], [191.7944, 51.0966], [191.91903, 51.079944], [192.52837, 51.486083], [194.825116, 51.59311], [196.74513, 51.682581], [198.159485, 51.74849], [199.9108, 51.8301], [201.694499, 51.77577], [205.2721, 51.6668], [207.156897, 51.496653], [209.691945, 51.267807], [210.110557, 51.230017], [215.9508, 50.7028], [220, 50], [222.458389, 50], [224.943217, 50], [230, 50], [234.4984, 48.5683], [235.2708, 48.4862], [236.0268, 48.2659], [236.5202, 48.2308], [236.5723, 48.1629], [236.6086, 48.1644], [236.7578, 48.2173], [236.8895, 48.2079], [236.8902, 48.274], [237.0255, 48.3305], [237.2085, 48.4075], [237.2568, 48.463], [237.252, 48.5269], [237.2709, 48.6586], [237.2236, 48.7537], [236.949, 48.8551], [236.6205, 49.0699]], "type": "LineString"}, "properties": {"duration_hours":<PHONE_NUMBER>7992775, "length": 8757.555063024629, "units": "km"}, "type": "Feature"}

To support, could you share your code how you call searoute function?


fix: changed height calc for embeded app page

Description

changed height calc for embedded app page

Fixes #8573 , #8393
Type of change

Bugfix (non-breaking change which fixes an issue)

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my own code
[x] I have commented on my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[x] New and existing unit tests pass locally with my changes

Test coverage results :test_tube:
:green_circle: Total coverage has increased
// Code coverage diff between base branch:release and head branch: bug/whitespace-embeded-page-bottom

Status
File
% Stmts
% Branch
% Funcs
% Lines

:green_circle:
total
54.96 (0)
36.65 (0.01)
34.68 (0)
55.47 (0)

:green_circle:
app/client/src/utils/autocomplete/TernServer.ts
52.94 (0.23)
41.67 (0.84)
36.21 (0)
56.99 (0.25)

/ok-to-test sha=c974751

@techbhavin Verified this and it is working as expected

With respect to online JS environment we cannot test deploy previews due to authorization will not work in DP's is that okay? @techbhavin @dilippitchika

/ok-to-test sha=07aebc7

With respect to online JS environment we cannot test deploy previews due to authorization will not work in DP's is that okay? @techbhavin @dilippitchika

@YogeshJayaseelan having a hard time understanding this, can you please elaborate?

/ok-to-test sha=07aebc7


Add testing of l1<>l2 messages for messaging module

Supposedly, there is a plugin for devnet which integrates with hardhat.
It's important, a minimal thing would be to run it on demand, or per-deploy, so we are sure that thing is working.
I know that YAGNI, but we have the feature, and if it's there - it should be working (or be removed).

Connected with #579

We decided to entirely remove this module


Upgrade to Razzle 3.1.1

It has some nice things, upgrade direct deps, and others like the client only build.
@terapyon did you asked for it in the past, right?
I haven't the use case yet, but who knows!
We will have to add the documentation, but now it's possible:
yarn start-spa
yarn build-spa
You have to rename public/index.html.spa to public/index.html... works like a charm 😊

@tiberiuichim indeed, it's the remainings from the original plugin fork. I did the replacement quick to make it work.

Awesome! 🎉
The pedantic part of me would say that removing support for CSS Modules should be referred to as a breaking change.
On the other hand, not sure if anybody uses those for Volto projects. It would be interesting to know that.

The pedantic part of me would say that removing support for CSS Modules should be referred to as a breaking change.

It will for sure! I also prepared the upgrade docs.

@pnicolli reenabled! https://github.com/plone/volto/pull/1495


[Improvement][k8s] No configuration provided for loading MySQL JDBC driver during Helm deployment to Kubernetes.

Search before asking

[X] I had searched in the issues and found no similar feature requirement.

Description
When creating a new MySQL data source, I need to copy the MySQL data source JAR file to the following directories: api-server/libs, alert-server/libs, master-server/libs, worker-server/libs, and tools/libs. Currently, I can only achieve this by modifying the YAML files in the helm/templates folder.
    command:
            - sh
            - -c
            - |
              sed -i 's|$DOLPHINSCHEDULER_HOME/libs/\*|$DOLPHINSCHEDULER_HOME/libs/\*":"$CLASSPATH_API|g' /opt/dolphinscheduler/bin/start.sh;
              exec /opt/dolphinscheduler/bin/start.sh

Additionally, after replacing the code as described, I need to configure the classpath_api environment variable externally to load the MySQL driver from common.sharedStoragePersistence. This method is extremely cumbersome.
It would be better to provide a Persistent Volume Claim (PVC) to load external JAR files. Ideally, there should be a separate external JAR loading path for each component: api-server, alert-server, master-server, worker-server, and tools. This would allow external users to easily provide these JAR files to DolphinScheduler.
Are you willing to submit a PR?

[X] Yes I am willing to submit a PR!

Code of Conduct

[X] I agree to follow this project's Code of Conduct

For missing JARs, such as MySQL or Oracle, rebuilding docker images directly may be more convenient in many cases. However, shared storage is also a suitable solution.

Another solution is to modify the java startup command in /opt/dolphinscheduler/bin/start.sh by adding the CLASSPATH environment variable to the -cp parameter. Then, in Helm, provide separate environment variable configurations for the api-server/libs, alert-server/libs, master-server/libs, worker-server/libs, and tools/libs components. We need to discuss the best solution.


--get-url option not work in aws ec2 environment from a week ago

I run two linux environments. then, One is on VMware and another one is on AWS.

  CL of 2NE1) M_V-FrG4TEcSuRg.mp4
[download] 100% of 74.17MiB in 00:00

Basically, everything looks fine. Even downloading the video completed successfully both.
But i can only see video in internet browser with url that generated on my VMware environment.
I mean that video url is not valid only on my AWS environment while downloading video works fine.
It had been worked till a week ago. but now it hasn't been working.
Anyone have any idea for this issue?
Thanks
Regards

See this FAQ entry. In short: URLs extracted from -g is not guaranteed to be usable on other machines.


update rule for libjpeg

See:

http://packages.ubuntu.com/yakkety/libjpeg-dev
http://packages.ubuntu.com/zesty/libjpeg-dev

@wjwwood could you add debian rules too please?

I guess that's the right libjpeg rule for stretch?

But that's the same thing we're doing in Ubuntu:
http://packages.ubuntu.com/yakkety/libjpeg-dev
I don't know which we should prefer, because you could also do libjpeg8-dev.

hmm good question. I guess it comes down to: do we prefer to have keys that are the same across platforms or call out versions.
I'm guessing based on the other rules that you're right we should use libjpeg-dev for both. Is the consensus to use the "generic" name as long as it doesn't result in ambiguity to which package provides it? (like it would be the case for libtlfk-dev for example provided by both  libfltk1.3-dev and libfltk1.1-dev )

Yeah, but in the case of libfltk-dev we explicitly want the non-default version (1.3 rather than 1.1). I don't care what we do here, but I'd want to make it consistent between ubuntu and debian.

We decided to leave it as proposed here.


No sensor appears after configuration

After configuring the extension no sensor appears, it remains empty

Same here.

Latest version fixes it for me.


fix: use global-bin-dir config instead of PNPM_HOME environment variable

When the global-bin-dir config is set, the environment variable npm_config_global_bin_dir is added.
This did not exist when using only the PNPM_HOME environment variable.
We need to debug both cases of using the global-bin-dir config and using the PNPM_HOME environment variable.


Feature 1.5.0

Implement Liquid templating syntax

Codecov Report

Merging #14 into master will increase coverage by 0.37%.
The diff coverage is 81.03%.

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
+ Coverage   80.09%   80.46%   +0.37%
==========================================
  Files          17       17
  Lines         849      896      +47
  Branches      185      196      +11
==========================================
+ Hits          680      721      +41
- Misses        107      110       +3
- Partials       62       65       +3

Impacted Files
Coverage Δ

src/X12Transaction.ts
86.48% <0.00%> (ø)

src/X12TransactionMap.ts
74.86% <82.14%> (+4.18%)
:arrow_up:

src/X12SerializationOptions.ts
100.00% <100.00%> (ø)

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 a269b77...478f55b. Read the comment docs.


Accept rawChannel or url for VncScreen component

Is there an existing issue for this?

[X] I have searched existing issues, it hasn't been reported yet

Use case description
Basically we have our own required implementation of creating a ws connection. We are not relying on third-part libraries to do it for us. It covers for us security and auth etc.
noVnc package has support for accepting either url or already created ws channel to work with.
export default class RFB extends EventTargetMixin {
    constructor(target, urlOrChannel, options)

So i am offering to accept not only url as string but alrerady created data channel and pass it to a rfb constructor.
Wath do you think ?
Thank you!
Proposed solution (optional)
No response

Seems fair, if noVNC supports it, this library should as well. Right now, if type-safety isn't a concern, I think you can still pass a raw channell object to the url field in VncScreen and it'll still work (as is evident from the following line(s) of code).
https://github.com/roerohan/react-vnc/blob/main/src/lib/VncScreen.tsx#L231
I could make a change wherein if the url is absent but a channel is present, I pass that to the RFB constructor. If both are present however, it would accept the url field. (P.S. I don't want to make it urlOrChannel since I want it to be typed distinctly). Does this solution sound good? What do you think?

@sklymoshenko could you share your thoughts on this? I'll update the source accordingly.


Update get-started.md

corrected a typo:
was: Are you going to use EF to map your objects ton an existing database...
now: Are you going to use EF to map your objects to an existing database...

Thanks @hugotoledo!


Fix file path not working problem in Windows operating system

In order for it to work in the windows operating system, I took the hrefs of the file paths with the url package in nodejs and added them to the routes. I was able to test it in windows. Technically it should work in other operating systems. You can still check it well.
Related to  #1

Just checked, and seems fine, will release it promptly

Released in v0.0.6, thanks for your contribution ;)


Add option to disable gerrit integration for a project

When working in a mixed environment, e.g., gerrit and github, it can be quite frustrating if you push to refs/for/... on github because you forgot to uncheck Push to Gerrit and then wonder where your branch ended up.
So it would be great if the integration could be enabled/disabled on a per project level.

This depends on #159.


[FLINK-18141][doc][parquet] Add documentation for Parquet format

What is the purpose of the change
Add documentation for Parquet format for table file system connector.
Verifying this change

Thanks @danny0405 for review, updated.


Square up group icon layout on Collaborate page

Make groups icon layout equal height/width.

@mauriel133 Hope this is an easy fix.

@dtarb, @pkdash  Could you elaborate on this one?

This had to do with the group layout looking irregular.  It is working nicely now on www, so was evidently fixed at some point and I will close.


Bank Transfer within IBAN zone

Thanks for your work on the font - I love it!
I have a request for a Bank Transfer within the IBAN zone icon

+1


Added Feedback Button

I added the Feed Back Button.

Why is this action towards the master branch?
We are trying to make this master branch obsolete as soon as possible. If there need to be any new changes - please make sure you do that in the development branch


fix(deisctl/units): stop k8s services without errors

The various kube-* binaries all exit with a return code of 2 on CTRL+C / SIGQUIT|TERM|KILL. This change considers 2 to be a successful k8s service exit as well, to avoid systemd failures on deisctl stop k8s.
See http://www.freedesktop.org/software/systemd/man/systemd.service.html#SuccessExitStatus= for background docs.
Closes #4213.

Code LGTM.

Another one of those "I didn't know you can do that" moments with systemd.
Code LGTM.


Upgrade Operator-sdk from v1.25.0 to v1.26.0

When running bundle validation with #64 it doesn't add any warnings.
From the Operator-sdk release notes it seems like it got fully supported from Operator-sdk v1.25.1, thus we can even upgrade it to the latest version (v1.26.0).

/lgtm


Fix for log_sum_exp

log_sum_exp is erroneously marked as non-increasing and positive.
This seems to have been introduced in
https://github.com/cvxgrp/cvxpy/commit/9366ccb54440ae0354f52cfbd14f30cba371b4db#diff-e6d9effeb355db619cf5bf6d59da6685

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

Coverage remained the same at 82.824% when pulling 57f7533baaaf66c46d83bcca72bb89c9c2b98a40 on aseyboldt:fix-log-sum-exp into 7327123c2a35f7deb5f4931cf8da8c2ea1e0749e on cvxgrp:1.0.

Woops good catch. Can you please sign the CLA?

Done.
But don't you think you are maybe overthinking that copyright stuff a bit? :-)


tsml_react_config is not defined

After out meeting on Saturday, I tried the new tsml_ui (react) using my district's feed. When I first did that, when I followed the instructions under "Install on my website", I didn't see any meetings. After a little troubleshooting, I found the error "tsml_react_config is not defined" in the Web Console of my browser.
I then had to add the following code to my web page:
<script>     var tsml_react_config = {         }; </script>
Of course I could add options to this code if I wanted to.
I'm seeing the same thing when I click on the second "Here's a demo" link (https://react.meetingguide.org/demo.html), in that it give the same tsml_react_config is not defined error.
Brian W.

Ah, hmmm... it looks like we might need to initialize this as an empty struct if it isn't passed. Thanks for the heads up, @brianw-area24.

This problem is here: https://github.com/code4recovery/tsml-ui/blob/master/src/helpers/settings.js#L38
flags: tsml_react_config?.flags ? [] : ['Men', 'Women'],

A straightforward solution would seem to be doing this before the settings struct:
// Initialize tsml_react_config if it isn't defined in HTML
if (typeof tsml_react_config === 'undefined') {
  var tsml_react_config = {};
}

However, this overwrite the tsml_react_config that is sent in from HTML. Will have to investigate further. I have a solid repro, at least.

This problem is here: https://github.com/code4recovery/tsml-ui/blob/master/src/helpers/settings.js#L38
flags: tsml_react_config?.flags ? [] : ['Men', 'Women'],

A straightforward solution would seem to be doing this before the settings struct:
// Initialize tsml_react_config if it isn't defined in HTML
if (typeof tsml_react_config === 'undefined') {
  var tsml_react_config = {};
}

However, this overwrite the tsml_react_config that is sent in from HTML. Will have to investigate further. I have a solid repro, at least.

Hi FlipperPA - where would I put this code exactly?   Thank you...

@FlipperPA yeah this is a head scratcher. how do you check for the existence of this variable without overwriting it locally? i tried your idea in settings.js and it did just that.
If fixed the demo so at last that should be working. The only people encountering this should be the people who don't follow the instructions and choose not to define that variable. Anyone using WordPress should have that defined with the time zone at least.

@joshreisner Yeah, maybe we just make the variable mandatory. I'm okay with that, at least for now!


Lagoon

Pre-submission checklist:
Please check each of these after submitting your pull request:

[x] Are you only including a repo_url if your project is 100% open source? If so, you need to pick the single best GitHub repository for your project, not a GitHub organization.
[x] Is your project closed source or, if it is open source, does your project have at least 300 GitHub stars?
[x] Have you picked the single best (existing) category for your project?
[x] Does it follow the other guidelines from the new entries section?
[x] Have you included a URL for your SVG or added it to hosted_logos and referenced it there?
[x] Does your logo clearly state the name of the project/product and follow the other logo guidelines?
[x] Does your project/product name match the text on the logo?
[x] Have you verified that the Crunchbase data for your organization is correct (including headquarters and LinkedIn)?
[ ] ~5 minutes after opening the pull request, the CNCF-Bot will post the URL for your staging server. Have you confirmed that it looks good to you and then added a comment to the PR saying "LGTM"?

Build failed because of:
No city for a crunchbase entry for amazee-io at https://www.crunchbase.com/organization/amazee-io

Deploy preview for landscape failed.
Built with commit 81a7e72d14aaae783a55568f780f73539f1cdd1f
https://app.netlify.com/sites/landscape/deploys/5e861ea70ddf61000767a46e

Build failed because of:
No city for a crunchbase entry for amazee-io at https://www.crunchbase.com/organization/amazee-io

Deploy preview for landscape failed.
Built with commit 918871c55601d746cd8e0d54a5a9bcfddcd14564
https://app.netlify.com/sites/landscape/deploys/5e861ee1deeb9f00076e14b4

LGTM


chore: simp lemmas for HashMap Insert/Singleton

No point having this separate, I'll roll these changes into #5590.


Dropdown: close menu when focusing outside element

Currently the behavior of dropdown menu is inconsistent for mouse and keyboard users:

when menu is open, clicking outside, close it
when menu is open, focusing outside (tab key), doesn't close it

The changes in this PR close the dropdown menu when focusing on an element outside the dropdown menu.
It's handled in a similar fashion it's currently done for click events (except that focusin events inside the menu don't close it contrary to click events).

Hi @vanduynslagerp!
Thanks for wanting to contribute to Bootstrap by sending this pull request!
Unfortunately, your pull request seems to have some problems:

Changes must be made to the original Sass or Less source code file(s), not just the compiled CSS file(s).
Changes must be made to the original JS source code file(s), not just the generated concatenated JS file(s).

You'll need to fix these mistakes and revise your pull request before we can proceed further.
Once you've fixed these problems, you can either ask the maintainers to re-open this pull request, or you can create a new pull request.
Thanks!
(Please note that this is a fully automated comment.)


Gatsby/fix/mobile menu

Codecov Report

Merging #167 into master will decrease coverage by 0.24%.
The diff coverage is 0.00%.

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


A2C/A3C: don't they use Q-learning?

On this page: https://spinningup.openai.com/en/latest/spinningup/rl_intro2.html
More specifically in this diagram: https://spinningup.openai.com/en/latest/_images/rl_algorithms_9_15.svg
I am surprised that the "A2C/A3C" box doesn't make use of Q-learning. They are, by their names, Actor-Critic methods. The critic uses Q-learning to learn the value of the states. As such, shouldn't they be connected to the "Q-learning" branch?

Hi @MasterScrat!
Actor-critic is a generic term, like an adjective, that describes a wide range of RL algorithms. An actor-critic algorithm has both

an actor (a policy with some learnable parameters, where at least some of those parameters only belong to the actor)

and a critic: a value function approximator with learnable parameters, where at least some of those parameters only belong to the critic).

The critic in an actor-critic algorithm can be any kind of value function: either an on-policy value function (V^pi (s)), an optimal value function (V* (s)), an on-policy action-value function (Q^pi (s,a)), or an optimal action-value function (Q^* (s,a)). It doesn't just refer to Q-functions.
A2C/A3C learns an on-policy value function that only takes state as an argument (V^pi (s)), not a Q-function. As such, they should not be connected to the Q-learning branch.
Critics that approximate value functions (as opposed to action-value functions, AKA Q-functions) show up in every modern policy optimization algorithm (eg VPG, TRPO, PPO, A2C/A3C, and others). As a result, while "actor-critic" was historically a useful term for distinguishing between some different kinds of algorithms (eg distinguishing policy optimization algorithms that used Monte Carlo return estimates, which were actor-only methods, from policy optimization algorithms that used critic-based advantage estimation, which were actor-critic methods), it is no longer a specific-enough characteristic to carry useful info.
Hopefully this is helpful!


Confirm that nmslib always returns K neighbors

Some queries seem to return < K lookup results.

It may be possible to return variable length Lookup sets. To address this, we unpack the list of Lookup sets into a RaggedTensor, notify the user via a printed warning, and then impute reasonable defaults to produce a full tensor. See 56ba48bbc51d9ef2576d32150ecad940f15f9053


[WIP] Check to see if C++2a is supported

This is just a test-PR that checks to see if C++2a is supported across all targets

 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.You have signed the CLA already but the status is still pending? Let us recheck it.

I believe we may not exercise all build platforms in our CI - or at least when we switched from 11 to 14 we did a search for embedded compiler support rather than just validating our CI.


QuantumProcessorSampler

fixes #5360
QuantumProcessorSampler shims AbstractProcessor to implement the cirq.Sampler interface and works with local-simulated or "real" processors. This is in contrast to QuantumEngineSampler which uses run_xxx methods that are not part of the AbstractEngine interface and can therefore only work with "real" engine connections.
Backwards incompatible changes:

the return type of AbstractProcessor.get_sampler() has changed. The behavior of the resulting object has not changed for EngineProcessors (unless you're doing something bad, see below), but has changed for SimulatedLocalProcessors. This fixes the issues like #5360
ProcessorSampler doesn't handle program argument types of EngineProgram. We'd need to teach AbstractProgram some run methods. cc https://github.com/quantumlib/Cirq/issues/1958 for context
Previously, you could have done something weird where you use the sampler from EngineProcessor.get_sampler() to run on processors other than the one pointed to by EngineProcessor. This is no longer possible.

@dstrain115 @verult

Thanks for the preliminary check. I'll flesh this out

@dstrain115 @wcourtney PTAL now that this is complete

any custom code that currently lives in QuantumEngineSampler methods be moved into EngineProcessor methods that the sampler calls.

We can't do this because QuantumEngineSampler can dispatch to multiple processors.

The other glue code relates to repetitions: cirq.Sampler accepts a list of values but AbstractProcessor accepts only one. Lots of discussion about this here: https://github.com/quantumlib/Cirq/pull/3265

We can't do this because QuantumEngineSampler can dispatch to multiple processors.

Do we ever use this? To me it makes more sense to just create separate sampler instances for each processor you want to talk to (just like you'd create separate AbstractProcessor instances for each of them). I know we thought early-on that it might make sense to allow sending a program and letting the service decide which of a set of processors to dispatch to, but given NISQ realities and how much device-specific calibration is needed to make this run well this seems unlikely to be useful.

The other glue code relates to repetitions: cirq.Sampler accepts a list of values but AbstractProcessor accepts only one.

True, but the code to handle this is almost identical in both QuantumEngineSampler and QuantumProcessorSampler, so this doesn't seem like a good argument for having separate classes.

I agree that QuantumEngineSampler has annoying semantics and should be deprecated. This would have to be done with some care.
Is your only remaining concern the duplicate code that exists in QuantumEngineSampler?
One cannot refactor QuantumEngineSampler to use ProcessorSampler because of the multi-processor_id and engineprogram support.

I think I'm going to keep it as ProcessorSampler to highlight the link to AbstractProcessor (which happens to return EngineResults). I suspect most folks will not notice or care that it returns EngineResults, which are a subclass of cirq.Result and the cirq.Sampler interface only guarantees the methods and properties on cirq.Result.

opened https://github.com/quantumlib/Cirq/issues/5371 for follow-on work


Docker healthcheck fails in 8.3.0

The container remains in status starting for some time then turns into unhealthy. This seems to be related to endpoint /iot/about being protected by access control, when probably it shouldn't. The service is available nevertheless, but this is quite misleading and worth fixing, in my opinion.
Start the container (requires an additional IDM service and env vars for the service to start successfully):
docker run --rm -d --name wilma -e PEP_PROXY_PORT=1027 fiware/pep-proxy:8.3.0

Login and run healtcheck script:
docker exec -it -u root wilma bash
node bin/healthcheck

The result is Performed health check, result 401.
The problem can be confirmed by installing curl in the container (apt update && apt install -y curl) and issuing a request:
curl http://localhost:1027/iot/about

Result:
{
    "type": "urn:dx:as:MissingAuthenticationToken",
    "title": "Unauthorized",
    "detail": "Auth-token not found in request header"
  }

/iot/about is just a fallback if no PEP_PROXY_PUBLIC_PATHS have been defined. The default config.js just sets an empty array, so the health check needs to attempt to contact somewhere:
https://github.com/ging/fiware-pep-proxy/blob/master/bin/healthcheck.js#L18
Since the PEP Proxy is not tightly bound to any particular web service, the details of your preferred public path will vary depending upon which service you want to protect.
Set something like  PEP_PROXY_PUBLIC_PATHS=version if you want to use the PEP Proxy with the Orion Context Broker.


Highlights doc for Artsy Engineering

I'd like to do something similar to what I do for Peril  , but for Artsy.
So if you have ideas for things, either let me know in an upcoming slack thread - or push some changes to this branch

OK, merge on green

Ah, added it too late. Doing merge on green should also merge if it's green 🍡


Simulated states from hidden Markov model should be sampled from {1, 2,...,N} (rather than 0,1,2,...,N-1)

Issue originally posted on stan-dev/stan#2957.
Summary + Description:
For the new hidden Markov model functionality, hmm_latent_rng() uses the foward-filtering backward-sampling algorithm to simulate from the joint posterior distribution of the underlying state process, implemented in the generated quantities block below. Given an N-state HMM, the underlying states should be drawn from {1, 2, ..., N} at each point in time, rather than {0, 1, ..., N-1} as is implemented now. This facilitates using the states as indices when generating draws from the posterior predictive distribution of the observations.
Reproducible Steps:
Stan code for an HMM using the functionality in CmdStan.
data {
int<lower = 0> N; // number of states
int<lower = 1> T; // number of observations
real y[T];
}

parameters {
simplex[N] theta[N]; // N x N tp
ordered[N] mu; // state-dependent parameters
vector<lower=0>[N] sigma;
simplex[N] init_dist;
}

transformed parameters {

matrix[N, T] log_omega;
matrix[N, N] Gamma;

// build log_omega
for (t in 1:T)
for (n in 1:N) log_omega[n, t] = normal_lpdf(y[t] | mu[n], sigma[n]);

// build Gamma
for (n in 1:N) Gamma[n, ] = theta[n]';

}

model {

// priors
mu ~ normal(2, 3);
sigma ~ normal(0.5, 1);

target += hmm_marginal(log_omega, Gamma, init_dist);

}

generated quantities{

matrix[N, T] states_prob = hmm_hidden_state_prob(log_omega, Gamma, init_dist);
int states_pred[T] = hmm_latent_rng(log_omega, Gamma, init_dist);

vector[T] y_pred;
for (i in 1:T)
y_pred[i] = normal_rng(mu[states_pred[i] + 1], sigma[states_pred[i] + 1]);

}

Current Output:
Currently, my defined states_pred object in generated quantities stores a sequence of underlying states that could include 0. For instance, for a 3-state HMM, we could have the underlying posterior draw:

states_pred = c(0, 0, 1, 1, 2, 0, 1, 2)

However, this means that I need to add a 1 if I would like to use the predicted states as indices, as shown below:
for (i in 1:T) y_pred[i] = normal_rng(mu[states_pred[i] + 1], sigma[states_pred[i] + 1]);

Desired Output:
It would be great to not have to add a 1, so rather than the code above, I could write:

for (i in 1:T) y_pred[i] = normal_rng(mu[states_pred[i] ], sigma[states_pred[i]]);

Current Version:
v2.24.0

The doc is not consistent with the current implementation, but with the proposed fix.

Closed by #2113

(@charlesm93 release is today and docs say rngs return [1, K] which matches the latest implementation, so we're good)


DXCDT-309: Add summary to job

🔧 Changes
When a job completes the job status response will include totals of successful, failed, inserted, and updated records (https://auth0.com/docs/manage-users/user-migration/bulk-user-imports#check-job-status).
The goal of this PR is to provide the summary on the jobs when reading a job’s details through https://auth0.com/docs/api/management/v2#!/Jobs/get_jobs_by_id.

📚 References

🔬 Testing

📝 Checklist

[x] All new/changed/fixed functionality is covered by tests (or N/A)
[x] I have added documentation for all new/changed functionality (or N/A)

Codecov Report
Base: 94.84% // Head: 94.86% // Increases project coverage by +0.02% :tada:

Coverage data is based on head (87a571a) compared to base (773df61).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@                        Coverage Diff                         @@
##           patch/issue-134-fix-job-import     #140      +/-   ##
==================================================================
+ Coverage                           94.84%   94.86%   +0.02%
==================================================================
  Files                                  38       38
  Lines                                6444     6471      +27
==================================================================
+ Hits                                 6112     6139      +27
  Misses                                264      264
  Partials                               68       68

Impacted Files
Coverage Δ

management/job.go
43.93% <100.00%> (ø)

management/management.gen.go
100.00% <100.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Feature : Dark Mode

Current
No dark theme available
Change
I can apply dark theme for better UI
Work Sample (its just a sample i will make it more better)
https://github.com/codervivek5/VigyBag/assets/101205226/241f640c-19f1-44b0-99f7-3afc3935f8a1
I want to work on this issue please assign me this

Hii @codervivek5 Please review my issue once

heyy @parasss19 I would Like To work on this issue ?

Can i see your changes

yes sure


Performance optimization of LayerComposition Update

Performance optimization of LayerComposition._update function, which executes when something in the scene changes (mesh / light / camera added / removed from layer, material blend mode changes ...)
Main reason for low performance - merging / searching arrays using array.IndexOf lookup.
Solution - using Set / Map as needed, also reorganized code to optimize algorithms
Test case (on Mac)

Hierarchy engine example, with one mesh being added / removed from hierarchy each frame.

test with 5000 mesh, performance improvement from 70ms to 2ms per frame
test with 1300 meshes, performance improvement from 5ms to 0.1ms per frame

Expected impact:

in applications with higher number of meshes in the scene, regularly changing the scene should be a lot faster.

Just to ensure: IE11 has limited support for features of Set and Map, does this PR only relies on supported features of IE11?

Just to ensure: IE11 has limited support for features of Set and Map, does this PR only relies on supported features of IE11?

yes I only used few functions supported everywhere.

Did you consider wrapping the array+set combo into a reusable data structure, say UniqueArray?
Yes it crossed my mind but then decided against it. Changing (private) interface, extra layer to access things.


Jio Tv not working

Your Environment

Add-on name: [e.g. Jiotv]
Add-on version: [e.g. 2.0.0]
Kodi version: [e.g. Kodi 19]
Operating system version/name: [e.g. Windows 10, LibreElec 9.0, etc...]
Using from country: [eg. India, USA, etc...]

Describe the bug
A clear and concise description of what the bug is.
To Reproduce
Steps to reproduce the behavior:

Go to '...'
Click on '....'
Scroll down to '....'
See an error

Expected behavior
A clear and concise description of what you expected to happen.
Screenshots
If applicable, add screenshots to help explain your problem.
Additional context
Add any other context about the problem here.
Debug log

The debug log can be found from this link:

I am also facing the same issue

bro just reinstaal jio tv addon and login again and setup then u good to go

I did that but still no success

bro just reinstaal jio tv addon and login again and setup then u good to go

Btw is it working for you

@0harshkumar0 @iMro0t , Device: Android, tried reinstalling kodi 19.3, Jio v2.2.2, tried connection with vpn (foreign), vpn india, without VPN, it throws play back failed, one or more items failed to play.  on back logs I could see plugin.jio.tv url failing http 406
`2021-12-14 18: 45: 04.253 T: 29218 INFO
files/.kodi/addons/plugin.video. jiotv/ : CPython Invoker (2, /storage/ emulated/0/Android/data/org.xbmc.kodi/ files/.kodi/addons/plugin.video.jiotv/ addon.py): script successfully run : VideoPlayer::Open File: plugin:// plugin.video.jiotv/resources/lib/main/play/ ?pickle-80049529000000000000007d94288c075 f7469746c655f948c07536f6e79204844948c0a6368 616e6e656c5f6964944d2301752e 2021-12-14 18:45:05.511 T:29313 : Creating InputStream 2021-12-14 18:45:06.412 T:29313  CCurlFile::FillBuffer HTTP returned error 406 <general: CCurlFile::FillBuffer - Reconnect, (re)try 1 Failed:
INFO
ERROR
2021-12-14 18:45:06.413 T:29313 WARNING
2021-12-14 18:45:06.757 T:29313 ERROR
<general»: CCurlFile::FillBuffer- Failed: HTTP returned error 406
2021-12-14 18:45:06.758 T:29313 ERROR
<general»: CCurlFile:: Open failed with code 406 for https://jiotv.live.cdn.jio.com /packagerx_mpd3/Sony_HD_HLS/Sony_HD.m3u8 ?jct=Ukwn22UP0dBsAF6By3rrqw&pxe=1639520823& st=9p-0_v1qIyd6E-rf8_gEOQ:`
Similar to :https://github.com/botallen/repository.botallen/issues/105

got it fixed by following below step
Since URL was working fine I needed no vpn case may different for other
Before starting ensure that you have updated your official jio tv app and have logged in with your sim on your phone
Now in your tv, follow steps and install this repo. Do login and try once to start channel
If doesn't then follow below....
Goto add on-- pvr client--- download IPTV simple client and once installation done goto add on jiotv long press on it goto Setup and choose simple IPTV option. It will ask you to restart kodi... Exit the app from power button on top left and sometimes it hangs wait for 20 sec if it doesn't go then kill the app...
Open and test
It worked for my case

I tried everything but Reinstall is only option it worked for me .

Thanks working now.

Update to v2.2.2 and re-login should fix the problem

Starts working after reinstalling the app
On Wed, Dec 15, 2021, 1:39 PM abhi @.***> wrote:

Update to v2.2.2 and re-login should fix the problem
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/botallen/repository.botallen/issues/242#issuecomment-994474224,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AVV4GCWVYP6ZQBVRRWICLDTURBEJXANCNFSM5KAXONDQ
.
Triage notifications on the go with GitHub Mobile for iOS
https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675
or Android
https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

Here's what worked for me:

Reinstall PVR IPTV Simple Client by going to Add-ons -> My add-ons -> PVR IPTV Simple Client -> Versions
Login to JioTV addon again from Video add-ons, select and long-press JioTV -> Settings -> Account -> Login
Comeback to JioTV Settings, select Setup -> Setup Simple IPTV PVR. A notification will appear stating 'Playlist updated, Restart to apply the changes.
Exit Kodi or remove it from memory
Launch again

For me, I unchecked the option Extra HD channels in Extra settings for the add on.
That seems to work for me

Earlier i could watch the past tv programs by going to the past program and clicking on play channel . By one fine morning , live tv I can watch not the past channels. I installed kodi, upgraded to 19.5 version several times . But I failed and gave up. I am really sad and frustrated . I have tried all the suggestions given by you friends, but couldn't succeed .


[5.4] Add assertJsonFragment()

This methods asserts that the response contains the given JSON fragment.
In assertJson you must provide full pass when testing response with nested array keys, however in this method you don't have to.

Awesome!


Latest Release?

I'm using BoxBilling 4.20. Update checker says that it's the latest release. But in your github, latest release is 4.21. If so, then why I can't update from BoxBilling itself? What are the changelogs?

I just manually updated it now it says 0.0.1 version... I am so confused.

Ok guys don't click the download button as it says on README.md. Use clone or download.

Any news?

Update it manually. Before update change "VERSION" to 4.21 in "/bb-library/Box/Version.php"

Download latest BoxBilling from here.
Extract files at your computer
Upload (overwrite) extracted files via FTP to /home/xecreate/clientsarea.xecreators.pk
When upload is complete, execute https://yourdomain.com/bb-update.php in your browser
Your BoxBilling is now updated to latest version.


Feature request: populate defaults for TokenValidityUnits when unspecified by the end-user

Hey @jagregory
The TokenValidityUnits can be specified by the end-user in the inputs to the CreateUserPoolClient API request, and in this scenario it seems to me on a surface level that everything in cognito-local functions as expected.
In the scenario where the end-user does not specify TokenValidityUnits however, I think I am finding a mismatch in behavior. The cognito-local container leaves the units unset whereas production AWS cognito will set them to default values. It seems that units for AccessToken and IdToken defaults to "hours" whereas RefreshToken defaults to "days"
[TokenValidityUnits](https://docs.aws.amazon.com/cognito-user-identity-pools/latest/APIReference/API_CreateUserPoolClient.html#API_CreateUserPoolClient_RequestSyntax)

    The units in which the validity times are represented. The default unit for RefreshToken is days, and default for ID and access tokens are hours.

    Type: [TokenValidityUnitsType](https://docs.aws.amazon.com/cognito-user-identity-pools/latest/APIReference/API_TokenValidityUnitsType.html) object

    Required: No

Not too terribly difficult to work around this by explicitly setting the values.
That being said, in the spirit of making the simulation match the behavior of production cognito, this would be a nice add!
Reference: https://docs.aws.amazon.com/cognito-user-identity-pools/latest/APIReference/API_CreateUserPoolClient.html
Thanks for considering!

Hey @RobHoman, I've published a fix for this. New clients now default to the unit values in the docs, and I also fixed the token generation so it actually uses the configured validity times too instead of just my hard-coded defaults.

Right on! Thanks.


Test ParseQuery

Starting small.
The PR adds tests to ParseQuery function. It also changes the way arg is split. Instead of single space split, it uses regex to split by variable spaces between words.

Nice!


New test branch

Integrated google analytics

Next time please name the merge request properly (i.e. in this case add Google Analytics)


Improve check for paths as Hashes in CaptureOptions

Hi,
Thanks for the awesome project!
This is my first PR and my Ruby is a little bit rusty, so please excuse any idiosyncrasies! I'm also running windows locally so I couldn't get the whole dev environment up and running to test everything before pushing, but I'll fix up whatever Travis finds.
Without knowing a whole lot about how this was supposed to be working, hopefully I've captured the intent of the code a bit better and fixed the issue reported in #536 at the same time.
I went for options.fetch so that it raised a KeyError, because I figured it'd make it more obvious that you're supposed to provide a path key if your path is a hash, rather than silently returning nil.

I just noticed that a related bug was present in the GalleryGenerator too, so I've added a fix for that in 0c39b03.
The code duplication suggests that the GalleryGenerator should use the CaptureOptions to parse the options, but I didn't want to make such a big change here just to fix this small bug.


Plans to have document updated for SBC's internal interface

Hi,
Today SBC vendors has started to support a scenario where other Teams user have access to SBC's internal interface only. Will you update the documentation to support that scenario?

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

ID: 10b198f4-9576-a14d-4359-1e8cc0319cda
Version Independent ID: ba7c3252-21d4-9be1-2a84-de2d51646f7b
Content: Plan for media bypass with Direct Routing - Microsoft Teams
Content Source: Teams/direct-routing-plan-media-bypass.md
Service: msteams
GitHub Login: @CarolynRowe
Microsoft Alias: crowe

@Petri-X Thank you for submitting feedback and contributing to the docs. We are currently investigating this.

@CarolynRowe is this in the pipeline and should I add a note to the document?

@Petri-X thank you for reaching out. I found this recent article https://docs.microsoft.com/MicrosoftTeams/direct-routing-plan. Does this relate to your query?

@Petri-X  We appreciate any feedback that improves the content in the Microsoft docs but we haven't had any response from you so we are going to close this issue.
Please feel free to re-open this issue if there is a specific area of the docs that we can improve or make better. Thank you.


Update web-sites-staged-publishing.md

Changed staging environments to the actual feature name: deployment slots

This seems like a small (but important) contribution, so no Contribution License Agreement is required at this point. We will now review your pull request.
Thanks,
Microsoft Pull Request Bot

@bmaluijb : Thanks for your contribution to the Azure documentation! The author, @cephalin, has been notified to review your proposed change.

@bmaluijb Thank you for your contribution. The current wording in this article is intentional. We love the fact that our customers have a deep understanding of our Services and the naming of our features However, we're trying to ensure that we are inclusive to potential or new customers who aren't as intimately familiar with our Services. In this case, and in many other places around our documentation, you will note that we are using industry standard terminology instead of product names. I appreciate the time you took to review our documentation and to make this contribution, but unfortunately, we want to keep the industry standard term in this case. Looking forward to seeing future contributions from you, Cheers!
#please-close

Thanks for the feedback, I didnt know that the naming was intentional.


test1

Codecov Report

Merging #2 into master will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master       #2   +/-   ##
=========================================
  Coverage     28.48%   28.48%
  Complexity       90       90
=========================================
  Files           130      130
  Lines          1678     1678
  Branches         74       74
=========================================
  Hits            478      478
  Misses         1184     1184
  Partials         16       16

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 1059523...43d9e74. Read the comment docs.


Grunt mods to support real time dev testing and minification

Modify grunt so that it automatically runs tests and minification as you change specified js files during development.

Excellent idea. We should address Issue #18 first and then add this on to that functionality.

Pull Request #29 addressed this


AC no longer works after the last update

After installing the update and restarting homebridge, the plugin is no longer able to control the AC. Shortly after, the AC disappears from Homekit. I wanted to save the logs, but Homebridge showed an error in config.json and restarted immediately. Had to revert to v1.0.1 to get everything working again.

Managed to save logs this time. Homebridge web interface log returns:

[7/14/2022, 12:05:08 PM] [TuyaIR] Client ID is not configured. Please check your config.json

Checkd config.json, Client ID is actually configured. Let me know how I can be of any further assistence.

I think i've managed to resolve the case. Swithed tuya ir (v.1.0.3) to Child bridge mode. Turned on auto fetching. restarted child process several times. Then added TuyaIR as a bridge in HomeKit and added devices to rooms. Now do not breath)

I refilled the Client ID and Client Secret after updating, and it works now.

Great. Closing the issue then.


Implement scaled units in elapsed time readout

Seeing e.g. 1907 seconds is not helpful, implement auto scaling of time units, investigate D:H:M:S.ss or decimal of largest appropriate unit.

Closed by commit https://github.com/ezocher/Gravity-Toy/commit/6558b1d382d27c1baff2cca38e248deb0ace0e93

Ended up doing decimal of largest appropriate unit