Can't open existing file...

This might just be user error but when I tried to open existing Untitled.tex (in svn_docs/bg/pvlv/bvpvlv sub direction) file by clicking OK as in the picture below - but nothing happened.

can you say what command you used to open the file?

I mean, how did you get the file chooser in the first place?  what command were you using within the gide app?

File -> Open file… then I clicked thru to the subdirectory and file I wanted to open, selected it, then clicked the Ok button at the bottom left.
--
Thomas Emil Hazy MD
thazy<EMAIL_ADDRESS>303.883.1009
On Oct 19, 2018, at 10:27 PM, rcoreilly <notifications<EMAIL_ADDRESS>wrote:
I mean, how did you get the file chooser in the first place? what command were you using within the gide app?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/goki/gi/issues/161#issuecomment-431547622, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AcotAgLHfRAzK7y3eIP-s2F_9pK_4qyqks5umqZFgaJpZM4Xxm_P.

ok, probably the subdirectory was the issue.  how many subdir levels btw?  I'll take a look.

~[home]/svn_docs/bg/pvlv/bvpvlv/ …
--
Thomas Emil Hazy MD
thazy<EMAIL_ADDRESS>303.883.1009
On Oct 19, 2018, at 10:31 PM, rcoreilly <notifications<EMAIL_ADDRESS>wrote:
ok, probably the subdirectory was the issue. how many subdir levels btw? I'll take a look.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/goki/gi/issues/161#issuecomment-431547793, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AcotAu4RqRy884ZT-jsQCWEeKf5uLwi6ks5umqcjgaJpZM4Xxm_P.

aha -- there is a key misunderstanding here: gide has the notion of a "project" which is the proximal directory containing the files you want to work on.  opening a file way down in a subdir, starting from home, is not the intended use.  you should do "Open Path.." and get to bvpvlv as your project,  and THEN open specific files from there..

Okidoke!  Thanks!!
--
Thomas Emil Hazy MD
thazy<EMAIL_ADDRESS>303.883.1009
On Oct 19, 2018, at 10:39 PM, rcoreilly <notifications<EMAIL_ADDRESS>wrote:
aha -- there is a key misunderstanding here: gide has the notion of a "project" which is the proximal directory containing the files you want to work on. opening a file way down in a subdir, starting from home, is not the intended use. you should do "Open Path.." and get to bvpvlv as your project, and THEN open specific files from there..
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHubhttps://github.com/goki/gi/issues/161#issuecomment-431548138, or mute the threadhttps://github.com/notifications/unsubscribe-auth/AcotAnz8an48cItySGm3EG7SkXP9zSTeks5umqkRgaJpZM4Xxm_P.

ps. wrote an intro: https://github.com/goki/gide/wiki/Intro


Support readline from MacPorts

Support for MacPorts has been added via the path mentioned in this PR in commit 02a67c8


Fix format documentation

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint.


merge from arlac77/template-github-action,arlac77/template-kronos-component

package.json

chore(deps): add ^7.3.1 remove ^7.3.0 (devDependencies.c8)

Codecov Report

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

@@           Coverage Diff           @@
##           master     #831   +/-   ##
=======================================
  Coverage   28.94%   28.94%
=======================================
  Files           1        1
  Lines          76       76
  Branches        1        1
=======================================
  Hits           22       22
  Misses         54       54

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 1c1ec40...aa5dd53. Read the comment docs.


Windows install directions need update for correct username

The windows install directions at https://github.com/dularion/streama/wiki/Getting-Started-(Windows) say

head to http://localhost:8080/, sign in with this account: username & password: admin, and you're set for your free self-hosted Netflix!

But this is not the account. The linux instructions have:

Step 4 - Login
the first thing you will see when opening the URL is a login screen. Use username admin and password admin to log in.

And this combination works on a Windows server.

It's not wrong, username & password: admin; i.e. both username and password is admin. But I agree that it's not as clear as the linux instructions, so I've updated it to match.


Add extra parameters to Resolve-DependenciesFromAzureFeed

Using the Resolve-DependenciesFromAzureFeed in our Azure DevOps pipelines I would like to expand this function with two extra parameters:

ignoreAppVersion
ignoredPublishers

ignoreAppVersion
Would like to retrieve the most up-to-date version from Azure Feed and ignore the specific version from the app.json
ignoredPublishers
Some Apps aren't in our Azure Feed. Based on the Publisher would like to exclude these from trying to download.

@Arthurvdv looks good. I never thought about ignoring publisher but thats a very good idea.

In my latest PR I used a diffrent technique for filtering dependencies. That might be a better solution here as well. I pass a filter thats then used by Where-Object what why you can basicly filter on any attribute thats in json.


Test fails with failed to load init class runtime error when using interop

Description:
A Runtime exception occurs when running a particular Ballerina project using interop. It is a unit test that makes use of a particular java class to test dependently typed variables. Class VariableReturnType resides in testerina-integration-test and the interop paths to the build/libs/integration-test.jar
The project can be found here.
Test fails with :
Compiling source
        intg_tests/object_mocking:0.0.0

Running Tests with Coverage

        object_mocking
error: failed to load init class :intg_tests.object_mocking.0_0_0.$_init

Generated module jar and Testable jar do not contain any classes. Only the manifest.mf file.
Steps to reproduce:
**Affected Versions:**beta 1
OS, DB, other environment details and versions:
Related Issues (optional):

Suggested Labels (optional):

Suggested Assignees (optional):

This happens due to the java dependency not being added in the Ballerina.toml. This is supposed to be a compilation failure since the InteropValidator should complain about the missing interop classes and functions.But due to aa bug in the diagnostic reporting in InteropValidator these errors are not captured during compilation. As the fix, the module that the diagnostic is related to should be set when reporting the diagnostic.


Documentation missing for important functions

Thx for the great package.
Some "public" functions do not have proper documentation yet.
e.g. utils.fetch_dataset
Relatedly, documentation should follow numpy standards.

@bthirion thanks for the feedback!
I have documented some more public API and move non-public API to sub-packages.
https://ancpbids.readthedocs.io/en/latest/api.html

Public interfaces should now all be covered by (numpy style) docstrings.


Replace client-side highlight.js by server-side syntect highlighting

This is setup to only support the same languages we targeted before: Rust, Markdown and TOML.
fixes #1652
TODO in this PR:

[x] some screenshots/comparisons with current highlighting
[x] is there a better alternative than submodules for getting the syntax files

Future extensions:

we could load more syntax definitions, either from the main Packages subrepository, or there's a collection of third-party syntaxes at https://github.com/sharkdp/bat/tree/master/assets/syntaxes/02_Extra
the mapping from syntect classes to colors could be extended, I just added a minimal set of rules to get some samples rendering well, but I almost certainly missed some common ones

Screenshots (original on left, updated on right):
TOML and Rust rendered as part of a readme, https://docs.rs/crate/bpaf/0.6.1:

The same section of readme in markdown source, https://docs.rs/crate/bpaf/0.6.1/source/README.md :

Unsupported filetype, https://docs.rs/crate/bpaf/0.6.1/source/.gitignore :

Wouldn't it be better to store the rendered content directly?

I couldn't find a better alternative than submodules for the assets.

I'm not sure if you're answering my question. In case you are, I'll try to reword it better:
Instead of highlighting every time the page is requested, why not store the rendered page to not recompute it every time?

Wouldn't it be better to store the rendered content directly?

That would mean that any improvements to the highlighting wouldn't apply to old files. Highlighting is very cheap, and these pages aren't requested that often compared to rustdoc pages.

I see, makes sense. Thanks for the explanation! :+1:

Fixed the readme block, .rustfmt.toml, Cargo.lock and Cargo.toml.orig. I hadn't realised the highlight.min.js actually contains some default languages, including bash used for the test file (I guess via first-line detection) and less used for the ci.yml (not sure why it detected this instead of the builtin yaml syntax it also supports). I'll take a look at what it supports vs. what's in the default sublime package and add some extras to our set.

I'll take a look at what it supports vs. what's in the default sublime package and add some extras to our set.

Feel free to ping me / request a review when I should have another look.
Thank you!

Extracted the full list of languages from the version of highlight.js we had: apache, bash, c, coffeescript, cpp, csharp, css, diff, go, http, ini, java, javascript, json, kotlin, less, lua, makefile, markdown, nginx, objectivec, perl, php, php-template, plaintext, properties, python, python-repl, r, ruby, rust, scss, shell, sql, swift, typescript, vbnet, xml, yaml
The languages included in the base sublime syntax packages: ActionScript, AppleScript, ASP, Batch, File, C#, C++, Clojure, CSS, D, Diff, Erlang, Git, Formats, Go, Graphviz, Groovy, Haskell, HTML, Java, JavaScript, JSON, LaTeX, LICENSE, Lisp, Lua, Makefile, Markdown, Matlab, Objective-C, OCaml, Pascal, Perl, PHP, Python, R, Rails, Regular Expressions, RestructuredText, Ruby, Rust, Scala, ShellScript, SQL, TCL, Text, Textile, XML, YAML
The major missing languages I can see are: ini, kotlin, swift, typescript, but these are all very niche and could be left for a future PR.

I don't know how we are even finding that readme to render, it's not set via package.readme in the Cargo.toml and crates.io doesn't see it https://crates.io/crates/itertools/0.10.1, the builder code is only supposed to look for README.md:
https://github.com/rust-lang/docs.rs/blob/52932dbcf0d6f326259c6b69f27f806e9378852f/src/db/add_package.rs#L265
highlight.js is detecting that codeblock as apache for some reason. There is no first_line_match defined for the Rust syntax file so syntect can't ever detect an unannotated block as it, in general it seems it's much less aggressive about detecting every codeblock as something and only detects relatively unambiguous things like shebang lines or DOCTYPE.

now that's a neat edge-case :D

We should probably stop doing that, it doesn't make sense when those docs are being rendered as part of the documentation anyway...

I agree, IMO we don't have to show more than crates.io here.

The largest file I could find was tests/crates_regex.rs at 124K, that took 251ms to serve the response on my machine. A more "normal" sort of large file like the 3.4K tests/bytes.rs is significantly faster at only 18ms.
Searching a few more crates that I recall having large files I found a 447K file in web-sys. That took 939ms to serve the response.
If there are issues from this even with caching, one option could be to disable syntax highlighting if the file exceeds some threshold.

thanks for the investigation @Nemo157  :)
I'm fine with the PR as-is, the only open question I see is around the submodule by @jyn514

First thought I had was: how do you distinguish between an outdated submodule and a wip update of a submodule?

We can only clone it if the directory doesn't exist yet.
Anyway, if you don't like the idea I don't feel strongly.

I really hate manually managing submodules; can we clone them in the build script instead of requiring people to deal with it themselves?

I doubt these will get more than 1 update per year, the overhead of working out how to manage them seems higher than the overhead of running git submodule update --init once per clone. (First thought I had was: how do you distinguish between an outdated submodule and a wip update of a submodule?).

@Nemo157 one thought about this: if these will really get only that little updates, why not just copy them over? Similar to what we do with font-awesome?

@Nemo157 I saw you rebased / added a new commit, feel free to ping me when this is ready again

We can only clone it if the directory doesn't exist yet.

Finally got round to implementing this and rebasing to resolve the workflow conflicts. (Then removed it again after the following changes 😁)

@Nemo157 one thought about this: if these will really get only that little updates, why not just copy them over? Similar to what we do with font-awesome?

It's ~1.3k files/2.8MB, which I guess is actually smaller than the font-awesome files. I've switched to using git subtree --squash to vendor them (I didn't realize it supported --squash before, otherwise I probably would have done this from the start).


Calculate Fieldtype

what is the best way for calculate diferent values from fields in a collection and punt into result field.
For example for multiply quantity and price in a collection row.
It's posible to make a new Fieldtype called calculate and define som rules in options, to choose diferent fields and apply aritmethic rules.

I think this doesn't exist yet, but could be implemented.
Maybe specifying a function name in a Calculate field by calling a specific php function that takes the form data in a file in the config directory. Something like functions.php.
I might even try to make this myself.

@mpartipilo no. in that case it won't be available in the admin backend, but you get it in your query result-set. you can also use the field to query, sort etc via the api.

@aheinze i try your code but nothing happens.

I create a collection named calculation
and create 3 fields

price

quantity

calculated

this is my config/bootstrap.php
<?php
$collectionName = "calculation";

$app->on('collections.save.before.{$collectionName}', function($name, $entry, $isUpdate) {

   if (isset($entry['price'], $entry['quantity'])) {
        $entry['calculated'] =  $entry['price'] * $entry['quantity'];
   }
});

$app->on('collections.save.before.calculation', function($name, &$entry, $isUpdate) {

   if (isset($entry['price'], $entry['quantity'])) {
        $fichero =  __DIR__ . '/debugcalculator.txt';

		$entry['calculated'] =  $entry['price'] * $entry['quantity'];
		file_put_contents($fichero, $entry['calculated']);
   }
});

&$entry does the trick


Realtime Database Android APK Deployment Issue

Step 1: Describe your environment

Android device: Android Studio Emulator
Android OS version: Pixel 4A - 11.0 / API 30
Google Play Services version: n/a
Firebase/Play Services SDK version: most recent

Step 2: Describe the problem:
Steps to reproduce:

Build the app via the Quick Start Code with the corresponding google-service.json file and export APK via "release" format
Push apk via adb and install
Attempt to register user

Observed Results:

What happened?
The APK installed correctly, but appears to stall while "registering" the user in the android application. I set the realtime database Rules per the guide. I set the Authentication Email/Password combo but that appears to not help the issue.

Expected Results:

What did you expect to happen?
The user to register and login.

Relevant Code:
// TODO(you): currently not any.

@n0psn0ps when you say "appears to stall while registering" do you mean it crashes?
I successfully reproduced this issue and the app only crashes when running the Kotlin quickstart in release build; the Java one works just fine.
I was able to track it down to some Kotlin model classes being removed from the APK by proguard. In our proguard-rules.pro file there's a rule to keep the Java models, but we forgot to add one for Kotlin:
https://github.com/firebase/quickstart-android/blob/320f5fb45f155de3daf8b997c3788a4a187a024d/database/app/proguard-rules.pro#L28-L30
I've opened a PR (#1356) to fix this, but in the meantime you can try adding that rule yourself and see if it works.

Awesome thank you @thatfiredev i will give that a try now and see if I can get it to deploy.

Looks like a may be missing something in my setup process. I can get the app to deploy into my android emulator no problem, but when I attempt to sign-up and login I am met with a failure in both regard. Screenshot below.

I have set the correct permissions for the Realtime Database:
{
  "rules": {
    // User profiles are only readable/writable by the user who owns it
    "users": {
      "$UID": {
        ".read": "auth.uid == $UID",
        ".write": "auth.uid == $UID"
      }
    },

    // Posts can be read by anyone but only written by logged-in users.
    "posts": {
      ".read": true,
      ".write": "auth.uid != null",

      "$POSTID": {
        // UID must match logged in user and is fixed once set
        "uid": {
          ".validate": "(data.exists() && data.val() == newData.val()) || newData.val() == auth.uid"
        },

        // User can only update own stars
        "stars": {
          "$UID": {
              ".validate": "auth.uid == $UID"
          }
        }
      }
    },

    // User posts can be read by anyone but only written by the user that owns it,
    // and with a matching UID
    "user-posts": {
      ".read": true,

      "$UID": {
        "$POSTID": {
          ".write": "auth.uid == $UID",
        	".validate": "data.exists() || newData.child('uid').val() == auth.uid"
        }
      }
    },

    // Comments can be read by anyone but only written by a logged in user
    "post-comments": {
      ".read": true,
      ".write": "auth.uid != null",

      "$POSTID": {
        "$COMMENTID": {
          // UID must match logged in user and is fixed once set
          "uid": {
              ".validate": "(data.exists() && data.val() == newData.val()) || newData.val() == auth.uid"
          }
        }
      }
    }
  }
}

Enabled the login via email and password.

Do I also need to set my root nodes via the Console? This is a quick demo of what it looks like on my end for the users nodes.

Thanks in advance.

@n0psn0ps Sorry for the delay here...
I tried to reproduce this with the security rules that you provided and it worked just fine, I had no errors.
I see from your database screenshot that you have a password field on the database that is not supposed to be there. I'm guessing that could be causing the error.
Can you please delete all the users that you have on your Database Console? And only add users through the sign up button on the app.


Use DoctrineMigration 3

Sylius 1.8 bring the great feature to delegate the migrations to the bundles instead of copy the migration in our apps, it will be nice if this bundle could benefit from it :)
Here the documentation from the upgrade note of Sylius.
And an example of the update of the Refund plugin in order to be compliant with the new DoctrineMigrations.
Note that it's a breaking change since it's need to use Sylius 1.8.

Hi @moafred, thanks for Your message.
When we support plugin on sylius version 1.8 and higher then we change the migration.
But many thanks for Your suggestion.


include requirements.txt in MANIFEST.in

Without including the requirements.txt in the MANIFEST.in the installation via pip fails.

Dear @jan-janssen, thank you for your interest in our package! I will definitely include your pull request in the next version of the library. However, I would like to clarify the essence of the problem. I have not encountered this before, and I'm interested to understand when it occurs. I ran two tests, in both cases everything worked successfully:
Test 1. In the "fresh" colab notebook I run !pip install ttopt==0.6.1 and the output contains the lines:
Collecting ttopt==0.6.1
  Downloading ttopt-0.6.1-py3-none-any.whl (15 kB)
Requirement already satisfied: numpy>=1.22 in /usr/local/lib/python3.10/dist-packages (from ttopt==0.6.1) (1.22.4)
Requirement already satisfied: scipy>=1.9 in /usr/local/lib/python3.10/dist-packages (from ttopt==0.6.1) (1.10.1)
Installing collected packages: ttopt
Successfully installed ttopt-0.6.1

that is, in this case, the file with the requirements was successfully read.
Test 2. On my macbook I tried the following code in a folder that do not (!) contain the ttopt code:
conda create --name test1 python=3.8 -y && conda activate test1
!pip install --no-cache-dir ttopt==0.6.1

and the output contains the lines:
Collecting numpy<1.25,>=1.22 (from ttopt==0.6.1)
...
Collecting scipy<1.11,>=1.9 (from ttopt==0.6.1)

That is, it seems that the file with the requirements is successfully read in both cases. If I then run the demo script (like demo/base.py), it works successfully.

Dear @jan-janssen, please also note the following lines in the installation script:
https://github.com/AndreiChertkov/ttopt/blob/8261db306483d7ccc182e85f7638c1889ada6c27/setup.py#L28
https://github.com/AndreiChertkov/ttopt/blob/8261db306483d7ccc182e85f7638c1889ada6c27/setup.py#L69
The file with the requirements is loaded automatically and it is obvious that everything will be correctly taken into account during the subsequent installation. It seems to me that the edit you propose is still not necessary?

Hi @AndreiChertkov , the difference is that you install the precompiled binary - wheel *.whl - it fails when you try to install from source.
pip install --no-binary :all: ttopt==0.6.1

Dear @jan-janssen, thank you. I have never used this option, but if it is in demand, then this edit is really needed. I have added the relevant line to the manifest file. The update is available in the new version 0.6.2.


slight improvements: comma-like, underscore, a, f, Yogh

a: improve shape
comma, semicolon, psili, dasia: more recognisable tadpole shape
underscore: slightly more space between two underscores
f: sharper curve if possible
Yogh: lessen top bowl

Fixed in https://github.com/agarick/agave/releases/tag/v27

a: rounded bottom-right corner (like "g", "p", "q")
comma-like: tadpole tail curved rather than blocky
underscore: 32 units on either side so a single underscore within a series is more distinguishable
f: sharper curve to make space between the two bars more visible, making the glyph appear more defined at smaller sizes
Yogh: slightly smaller top bowl


Dash Missing in Network Config

Hey @joshackland , I think it's actually missing some indentation, not the extra dash.
https://docs.docker.com/compose/compose-file/compose-file-v2/#ipam
So it should be
networks:
  default:
    driver: bridge
    ipam:
      config:
        - subnet: <IP_ADDRESS>/24
          gateway: <IP_ADDRESS>

@klutchell
Yes that version works for me but also the original one is now working.
When I first tried running, I received the error:
ERROR: The Compose file './docker-compose.yml' is invalid because: networks.dns_net.ipam value 'gateway' does not match any of the regexes: '^x-'
Adding the dash in this PR is what fixed it but seems to not be an issue anymore.


Support rxjs 6

it's currently not possible to use this component in Angular 6 since Angular is on RxJS 6

Is there any progress on this?  I too am now stuck as my project needs to upgrade but I use your component.

Hi,
Sorry for delay, but I have health problem this year :( And so have also lot of work on my job too.
You have try with rx-compat ?
If you have time, PR are welcome, but I think angular and rx 6 have better handling if project is migrated under angular cli / ng packager.

I also came across this issue using Angular 6.1 (import of rxjs/Subject failed) and after yarn add rxjs-compat I could run the component.


Not obvious which greevil is talking in Chapter 1

[ ]  Highlight sunsfans greevil in some way, health bar is not visible enough
[ ]  Too much going on in his greevil portrait to make an immediate connection to the ingame model

Could use a different gesture for the dialog, right now it's the attack animation but maybe we could use a channeling one. Could also put some kind of highlight on the talking one.


Question: Why can I not see shortcuts for SplitHeader menus?

I'm running Chatterino 2.1.7 (commit 2e39dd4d9) on Linux, with i3 as my window manager and compton as my compositor. What could be the cause of this behavior?

Do you mean shortcuts or mnemonics? Either way, none of them exist.

For example in https://user-images.githubusercontent.com/61775978/75855862-2a6bca80-5dc1-11ea-9278-97575395764c.png this image you can see that Ctrl+R is a shortcut for Change channel
I would like to see these.

There is a keybindings tab in the settings

I updated my system and they weirdly started showing, closing the issue.


There are some bugs in the display results of the (mne.viz.plot_alignment())  function

READ THIS FIRST: If you are having trouble getting MNE-Python to work with
your own data, you should ask for help on the
MNE Forum.
Our GitHub issue tracker is only used to report bugs and suggest improvements
to MNE-Python. For any other questions, please use the forum.
Usage questions that are posted as GitHub issues are usually closed without
being answered. See
the FAQ entry on filing bug reports
for more guidance. If you're pretty sure your problem is a bug in MNE-Python,
please delete this section and fill in the headings below, replacing the
placeholder text with your own responses. Surround any code samples with triple
backticks above and below the code block (see
the GitHub guide to markdown
for help with issue formatting). Alternatively, you can put your MWE in a
public gist and link to it in this issue.
Describe the bug
*To use the MNE-Python with 3D visualization capabilities, I create the environment with the code:  conda create --name=mne --channel=conda-forge mne.  but when I run the 'Source alignment and coordinate frames' Part, the display result doesn't look right
*
Steps to reproduce
Replace this text with a code snippet or minimal working example MWE to
replicate your problem, using one of the built-in datasets, preferably the
one called sample. If you can't replicate on a built-in dataset, provide also
a link to a small, anonymized portion of your data that does yield the error.
Expected results

*

Actual results

Additional information
Replace this text with the output of mne.sys_info().

Hello @LiFeng-SECUC,
please provide all the information that is requested when you open an issue.


Make FontSystem not self-referencing and update fontdb and rustybuzz

This makes the std implementation of FontSystem not self-referencing, which in turn allows defining a db_mut method for mutable access to the underlying fontdb::Database (requested in #75).
The high level changes are:

Adds a new FontKey struct that stores the fontdb::ID and (if the swash feature is enabled) the swash key data (u32, swash::CacheKey).

A FontKey can be obtained via the new FontSystem::get_font_key(id).
FontKeys are cached instead of caching Fonts.

FontSystem::get_font now takes a FontKey instead of fontdb::ID and creates a Font on the fly, reusing the key data.
Removes FontMatches.

FontSystem::get_font_matches now returns a Arc<Vec<FontKey>> instead of Arc<FontMatches<'a>>.
FontFallbackIter borrows fontdb::Database and creates the font on the fly using Font::from_key(db, key).

Just a heads up: before version 0.13.0, lookups in fontdb take linear time. As part of this PR, if you're making breaking changes anyways, I would bump fontdb to 0.13.0 so that lookups take O(1) time.

I  fixed the duplicate ttf-parser entries by bumping rustybuzz to version 0.7. Not sure I can do something about the failed license requirement.

I've added Zlib to the list of allowed licenses. I think there is no harm in that.

@geieredgar I merged this since CI tests were passing but it appears to have drastically reduced the performance of the editor-test example, so I reverted. Please see if you can replicate this issue.

Yes, I could replicate this issue. The reason is that rustybuzz::Face::from_slice takes a lot of time. I wrongly assumed it would behave similar to ttf_parser::Face::parse in terms of performance, which is intended to be cheap and therefore can be executed on the fly, but that isn't the case. I will look into trying to make rustybuzz::Face::from_slice not as expensive and then compare the performance of my approach with the old approach. Sorry for wasting your time!


handle .install file correctly

In our opam file, I've now added "cp" "opam/files/oloop.install" "./" to the install step. But is it necessary, or does the command "ocaml" "setup.ml" "-install" already do that anyway? Or is there some other better solution?
More generally, we used oasis2opam to generate the opam/ directory, but then modified it manually. That seems wrong. If the opam/ directory is auto-generated, maybe we shouldn't include it in the repo. But that wouldn't allow us to do an opam pin, so that also seems wrong. Should we just document that the opam/ directory is auto-generated and should not be manually modified? Is that correct?

See https://github.com/ocaml/opam-repository/pull/4343#issuecomment-117686590 for the discussion.

I do not remember whether "ocaml" "setup.ml" "-install" install the binaries (it may well) but "ocamlfind" "remove" will not remove them (oasis could but it would need the data of the previous compilation).  This is why oasis2opam genrates this .install file.
Note that since https://github.com/ocaml/oasis2opam/commit/99bee4f8e0d52d1e5091991bea623fc64c36d937 oasis2opam generates the install file at the root of the directory.

More generally, we used oasis2opam to generate the opam/ directory, but then modified it manually. That seems wrong.

This is no longer required with the latest oasis2opam.

But that wouldn't allow us to do an opam pin, so that also seems wrong.

Indeed.  If you want to be able to "Git pin", then this is needed.

Should we just document that the opam/ directory is auto-generated and should not be manually modified? Is that correct?

Maybe it is enough to add a make target that updates OPAM metadata when _oasis changes?

Maybe it is enough to add a make target that updates OPAM metadata when _oasis changes?

That sounds right. When auto-generated files do need to be checked in for some reason, I usually do:

Fully automate their generation.
Document clearly they are auto-generated and shouldn't be modified manually.


Outdated cask: glyphs

Outdated cask: glyphs.
Info:

version: 2.2.2-827.
appcast url: https://updates.glyphsapp.com/appcast2.xml.

2.2.2-827

Updated in #18390.


[Settings] - Can start multiple settings dialogs

double click the systray icon a bunch :)

double click systray while settings in focus, you get a weird white box that doesn't dismiss until settings is closed

Had created the white box issue earlier today #2330 :D

@laviusmotileng-ms is this done?

Yes it's done. I closed the earlier issue linked to this #2330 but forgot to close this one.


Context argument is not set in createRouter function for the path "/"

Context argument is not set in createRouter function for the path "/"
import Vue from 'vue'
import Router from 'vue-router'
import axios from 'axios'

Vue.use(Router)

const createRouter = (context) => {
  console.log(context)
  const router = new Router({
    mode: 'history',
    routes: [{
      path: '/',
      component: () => import('./Home.vue')
    }, {
      path: '/about',
      component: () => import('./About.vue')
    }]
  })

  if (process.browser) {
    const nprogress = require('nprogress')
    require('nprogress/nprogress.css')

    router.beforeEach((from, to, next) => {
      nprogress.start()
      next()
    })
    router.afterEach(() => {
      nprogress.done()
    })
  }

  return router
}

export default { createRouter }

context is only available on server-side.

I am logging context on the server side but Context argument is undefined in createRouter function for the path "/" on server-side and context is properly set for other routes.

I guess we need to implement a server-side redirect method in context, using the native next function in beforeEach((to, from, next) => {}) would not work.

Hmm, didn't understand you correctly, do you mean something like this?
router.beforeEach((to, from, next) => {
  if (to.path === '/private') {
    if (process.server && !ctx.req.session) {
      return next('/')
    }
    if (process.browser && !localStorage.getItem('api-key')) {
      return next('/')
    }
  }
  next()
})

Yes I am trying to do just that with some modifications but I am unable to use custom express server with ream and backpack module. Nuxt works fine with custom express server and backpack https://github.com/nuxt/express. Is ream compatible with backpack? Ream is much better than Nuxt If can solve few issues.

So is this issue solved already? I can see context:

Is ream compatible with backpack?

What issues do you run into while using Ream with Backpack? It's just a cli tool to run your node app, so I think it will work fine with Ream. Keep in mind that while using custom express server it's just a normal node app.

Yes after updating to rc.5 context is available in createRouter function.
I am using code from template-ream with my own server code
const dev = process.env.NODE_ENV !== 'production'
const client = ream({dev: dev, entry: 'src/client/index.js'})

client.prepare()
  .then(() => {
    app.get('*', client.getRequestHandler())
this.instance = app.listen(process.env.PORT || 5000,
 (err) => {
if(err) {
   Log.error(`Unable to start API server ${process.pid}.`);
   Event.emit('shutdown')
} else {
          Log.info(`API server ${process.pid} started...`);
    }
});
  })

client.on('ready', () => {
  ream.displayStats(client.stats)
  })

Pacjage.json
 "start": "backpack build && node ./build/main.js"

Build is successful and server is started. Here is the output

It seems that babel-preset-env module is defining node version number as a string and not as a number. Both backback and Neutrino reporting same error. @egoist Could you please suggest a solution.

that's weird, we didn't define the node version in babel-preset-env

@egoist My project is hosted on glitch.com. If you can share your email then I can send you an invite on glitch.com

yup, you can find my email on my profile page

@egoist I just sent you an invitation of my project on glitch.com

It's working now thanks for the help.


Feature Request: ability to get the current scan line index

This request came from some of the users on the Discord server. I had always responded that VERA was out of register space. Recently someone suggested that there is almost never a software use case for reading the line IRQ value, but many software use cases for polling/waiting for a specific scan line; it would be better to have the line IRQ return the current scanline when read.
I have made this change and one of the community members wrote a test case to demo/exercise it. The results looked great. If there is no objection to adding this feature, I will prep a PR in the next day or two with the two-line change that enables it.

I have no objection to this. So go ahead 😄

SCANLINE[8] overlapping IRQLINE[8] at IEN[7] is generating more unexpected behavior bugs by software devs than I had anticipated. Primarily due to enabling and disabling interrupts by reading IEN, setting or clearing the desired interrupt bits, then writing it back out. The read-modify-write cycle only preserves IRQLINE[8] if the SCANLINE value when the code is executed happens to be in the same half of the display as IRQLINE.
Weighing the options: nobody wants to lose IRQLINE; moving SCANLINE[8] to IEN[6] is almost as simple and allows read-modify-write cycles to work as developers are expecting. Since SCANLINE is brand new, and the emulator does not have support for it yet, no software other than the test programs for the SCANLINE feature make use of it; this change does not impact any existing software.
I will put together another PR that moves SCANLINE[8] from IEN[7] to IEN[6] during reads and restores IRQLINE[8] to IEN[7] during reads so that RMW cycles work again.


Update swagger-parser to 2.0.31

Updates io.swagger.parser.v3:swagger-parser from 2.0.28 to 2.0.31.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "io.swagger.parser.v3", artifactId = "swagger-parser" } ]

labels: library-update, early-semver-patch, semver-spec-patch, commit-count:1

Superseded by #152.


fatal error: 'boost/lockfree/detail/branch_hints.hpp' file not found

At the very least, make should not fail due to missing dependencies.  CMake should test those and abort at the configuration stage when dependencies are missing/broken.
+make -j2
Scanning dependencies of target hpx_init
Scanning dependencies of target hpx
[  0%] Building CXX object src/CMakeFiles/hpx_init.dir/hpx_main.cpp.o
[  0%] Building CXX object src/CMakeFiles/hpx_init.dir/hpx_main_argc_argv.cpp.o
[  0%] Building CXX object src/CMakeFiles/hpx.dir/exception.cpp.o
[  0%] Building CXX object src/CMakeFiles/hpx_init.dir/hpx_main_variables_map.cpp.o
[  0%] Building CXX object src/CMakeFiles/hpx.dir/exception_list.cpp.o
[  0%] Building CXX object src/CMakeFiles/hpx_init.dir/hpx_user_main.cpp.o
[  0%] Building CXX object src/CMakeFiles/hpx.dir/hpx_init.cpp.o
[  0%] Building CXX object src/CMakeFiles/hpx_init.dir/hpx_user_main_argc_argv.cpp.o
In file included from /Users/travis/build/jeffhammond/PRK/PRK-deps/hpx_0.9.11/src/hpx_init.cpp:8:
In file included from /Users/travis/build/jeffhammond/PRK/PRK-deps/hpx_0.9.11/hpx/hpx.hpp:10:
In file included from /Users/travis/build/jeffhammond/PRK/PRK-deps/hpx_0.9.11/hpx/include/runtime.hpp:13:
In file included from /Users/travis/build/jeffhammond/PRK/PRK-deps/hpx_0.9.11/hpx/include/threadmanager.hpp:14:
/Users/travis/build/jeffhammond/PRK/PRK-deps/hpx_0.9.11/hpx/runtime/threads/thread_data.hpp:31:10: fatal error: 'boost/lockfree/detail/branch_hints.hpp' file not found
#include <boost/lockfree/detail/branch_hints.hpp>
         ^
[  3%] Building CXX object src/CMakeFiles/hpx_init.dir/util/lightweight_test.cpp.o
[  3%] Building CXX object src/CMakeFiles/hpx_init.dir/main.cpp.o
1 error generated.
make[2]: *** [src/CMakeFiles/hpx.dir/hpx_init.cpp.o] Error 1
make[1]: *** [src/CMakeFiles/hpx.dir/all] Error 2
make[1]: *** Waiting for unfinished jobs....
[  3%] Linking CXX static library ../lib/libhpx_init.a
[  3%] Built target hpx_init
make: *** [all] Error 2

Full details shown on https://travis-ci.org/jeffhammond/PRK/jobs/103648565.

From the logs I gather that you're compiling against Boost 1.55.0. This version of boost however contains this file (see: here). Could you please verify that the file is available?

My scripting wasn't quite right, but I believe Boost 1.60.0 was used, per the following excerpt from the log.
+brew upgrade boost
==> Upgrading 1 outdated package, with result:
boost 1.60.0_1
==> Upgrading boost
==> Downloading https://homebrew.bintray.com/bottles/boost-1.60.0_1.mavericks.bo
==> Pouring boost-1.60.0_1.mavericks.bottle.tar.gz
🍺  /usr/local/Cellar/boost/1.60.0_1: 11,139 files, 415.4M

@jeffhammond Yes, please use an older version. Boost 1.60 was not available at the point in time when HPX 0.9.11 was released. The master branch should have this problem fixed, however.

Thanks.  I will switch to your Git master shortly.

Let us know how it goes. For now I shall close this ticket, please feel free to reopen, if necessary.

Thanks.  Indeed it should be closed.  Using the correct version of Boost solved the problem (and revealed another, but that's a new issue waiting to be created).


Added red color for error code 1 else yellow  blue for usage.

Well I added some color, just to the error echo and usage. red only when the exit code is 1 all other the color will be yellow. blue for the usage. This is just a simple way to add some color. To be more flexible we should probably write a little echo function. See #1664

Line comments added and adjusted the $ecode check.

Considering #1753 (and the fact that echo "\escapes" isn't portable), I think it'd be worth it to make either a bunch of global constants like $RED and $BLACK, or to put something like the following function (that I used once in a random project) into installer/functions (@drinkcat for style suggestions):
# Set a console color (shorthand)
# usage: c [l][color]
# Specifying "l" uses the light version of the color.
# Color can be black, green, cyan, red, purple, or orange.
# Color can be specified with just the first letter.
# example: echo "$(c lr)light red $(c g)green $(c)normal color"
c() {
    printf "\033["
    if [ -z "${1:-}" ]; then
        printf "0m"
        return
    fi
    local c="$1"
    if [ "${c#l}" != "$c" ]; then
        printf "1;"
        c="${c#l}"
    else
        printf "0;"
    fi
    case "$c" in
        b*) printf "34m";;
        g*) printf "32m";;
        c*) printf "36m";;
        r*) printf "31m";;
        p*) printf "35m";;
        o*) printf "33m";;
         *) printf "30m";;
    esac
}

I immediately realized after opening #1753 that we need some sort of 'Universal Coloring Engine' so that everyone is outputting in the same format and color.
We can create a function like crouton_echo to replace the default echo usage so we can do crouton_echo blue "Hello World" .
I think the global variables may not seem so smart.

That has potential too...you could always mix-and-match colors by using -n as well.  "crouton_echo" isn't very descriptive though; how about echo_color?
I'm ambivalent whether we take the echo_color approach or the $(c) approach described above.

I prefer the $(c) approach. This has the advantage when only needing to highlight one word it is possible without splitting up the echo lines in the code.

Sure,  the name of the function I gave isn't the most suitable one.  I just randomly came up with one for expressing the idea.

I prefer the $(c) approach. This has the advantage when only needing to highlight one word it is possible without splitting up the echo lines in the code.

Is it something we plan to do? I'm not sure ,-) Also, c makes it easier to forget to reset the color at the end of the line... Acually, I wouldn't necessarily mind having both echo_color and c functions (with echo_color as a simple wrapper). We could even have a cat_color, if that's useful in some contexts.
Finally, we want to check that the output is a tty before outputing colors, so we don't introduce binary characters when stdout/err is redirected to a file. (that's what ls does, for example)

@drinkcat You are right we don't want to make the terminal like a candy shop :) I created an echo_color function now.

Actually, I think USAGE is one such example where we wouldn't want the entire echo to be a single block of color...I'm not sure it should be any color at all, since it's the only text being spit out.

@dnschneid Yep, I know USAGE is indeed something I am also not happy with. It should be no color like you said. Let me think about it.

So in conclusion: echo_color shouldn't have any redirects inside the function, but usages of echo_color should as appropriate (basically, as they did before with echo).

This also means you can technically error 1 "$(echo_color r "ERROR")"...

This also means you can technically error 1 "$(echo_color r "ERROR")"...

Is this something we want? Or do we leave color red by default in the error function for exit code 1. I like removing the default yellow color for other exit codes > 1. So we can also display the usage message with no color and add the echo_color y to where it is needed.

Is this something we want? Or do we leave color red by default in the error function for exit code 1. I like removing the default yellow color for other exit codes > 1. So we can also display the usage message with no color and add the echo_color y to where it is needed.

Sorry for being impatient, but any feedback on this?

Sorry, red by default for exit code 1, and no colors otherwise makes sense.  Our usage of error codes appears to be as follows:
0: success (duh)
1: processing error
2: user error (bad parameter to script)
3: coding error (bad parameter to function)
99: target doesn't work on platform (specially handled by tests)
99 should probably be a color too (orange?)...

We only seem to have the following colors available (missing orange ;)) See http://ascii-table.com/ansi-escape-sequences.php
Foreground colors
30	Black
31	Red
32	Green
33	Yellow
34	Blue
35	Magenta
36	Cyan
37	White

Background colors
40	Black
41	Red
42	Green
43	Yellow
44	Blue
45	Magenta
46	Cyan
47	White

I also see that I should change the comments in the code, because 1; turns on bold 0; regular text
See the following screenshot for an example:

for i in 30m 31m 32m 33m 34m 35m 36m 37m 40m 41m 42m 43m 44m 45m 46m 47m; do printf "\033[1;$i"; echo color; printf "\033[0;0m"; done

Whoops.  Yellow works too then :)

Al ready added some yellow color to the error 3 in functions. All the other error calls needs still to be done in files where they are present. I thought of creating a different pull request for that. To keep this a bit clean.
Color codes the way I see it:

Red for all exit 1 errors.
Yellow exit 2, 3, 99 will depend on the message
Blue usage messages and prompts.

There are also some WARNING messages, I thought of giving them Magenta color.
Feel free to add any input about color usage... Also should we support background colors in echo_color, or just stick to bold/regular text colors?

Also should we support background colors in echo_color, or just stick to bold/regular text colors?

I think only colored text would be efficient. The mixed usage of text color and background color may make the interface looks too colorful and confuse the users.

@tonyxue yep I agree, that is why I left them out.
BTW to clarify with usage messages for color blue I meant for example $TIPS. So not the $USAGE they should be just normal color.

But if all parts of the $TIPS is in the same color, how could we make message like this stands out? Or do we need to make it especially stand out?
As https://github.com/dnschneid/crouton/issues/1804#issuecomment-107759896 that the user obviously didn't see that. And if it still blends with other messages, the users will still not see that.
So the point is we add color to make some part of the output stand out, so we should not employ colors in a large scale. But I am not sure whether $TIPS is large scale. People will still ignore the download link if they choose not to read $TIPS at all. We should only use a small amount of colors precisely.

@divx118 And maybe we can modify the title of this PR a bit? This PR is not as simple as the title is describing now. :P

@tonyxue It should be possible to highlight some special messages like the one you described with $(echo_color "m" "You must install the Chromium OS extension for integration with cro..."). I still think $TIPS should overall have a color, or at least start with a color so you can see the script has ended and you should start reading from the first color line.

How about this, we only highlight "Here's some tips:" blue then we highlight only messages in $TIPS blue when they need further user action like the example you posted.

How about this, we only highlight "Here's some tips:" blue then we highlight only messages in $TIPS blue when they need further user action like the example you posted.

That sounds nice. +1

Any update on this?
And any suggestion on adding color for this ?
Although not many people care about this, it is still essential to know.

We could colorize it, I suppose, as long as it's not obnoxious.

Ping!
Are we adding more colors or colored texts? It seems good to merge now.
We could colorize the rest of the texts later.

Seems we have now everything to merge. Or are there still any other thoughts ...

One additional thought/suggestion: Could we add a color, possibly white, to any 'WARNING: ..." messages?
(I.E.) echo "WARNING: $RELEASE is an unsupported release.
They get spit out early in the install and I think they are often overlooked.

@DennisLfromGA That is indeed possible, but those messages can be changed in different pull request(s).
After this merge, something like the following will highlight WARNING: in yellow color.
echo "$(echo_color "y" "WARNING:") " $TARBALL is an old-style bootstrap or backup." 1>&2

@DennisLfromGA Yep sorry for the confusion. The yellow was just a quick example. I indeed think magenta is a good color for those and only highlight the "WARNING:" to prevent the output becoming like a rainbow.
To summarize coloring can be as follows:

Red for all exit 1 errors.
Yellow exit 2, 3, 99 will depend on the message
Blue usage messages and prompts. Also highlight messages in $TIPS that require further user action like installing the chromeos extension.
Magenta warning messages just the text "WARNING:"

@DennisLfromGA Yep sorry for the confusion. The yellow was just a quick example. I indeed think magenta is a good color for those and only highlight the "WARNING:" to prevent the output becoming like a rainbow.
To summarize coloring can be as follows:

Red for all exit 1 errors.
Yellow exit 2, 3, 99 will depend on the message
Blue usage messages and prompts. Also highlight messages in $TIPS that require further user action like installing the chromeos extension.
Magenta warning messages just the text "WARNING:"

@divx118,
I guess without the 'c' function mentioned in https://github.com/dnschneid/crouton/pull/1710#issuecomment-102679054, we could still do something like this -
(if I understand the 'echo_color' function correctly to add '-n'):
echo "$(echo_color "y" "-n" "WARNING:")"
echo " $TARBALL is an old-style bootstrap or backup." 1>&2
-or even this-
echo -n "$(echo_color "y" "WARNING:")"
echo " $TARBALL is an old-style bootstrap or backup." 1>&2
I think either approach would work...

@divx118,
I guess without the 'c' function mentioned in https://github.com/dnschneid/crouton/pull/1710#issuecomment-102679054, we could still do something like this -
(if I understand the 'echo_color' function correctly to add '-n'):
echo "$(echo_color "y" "-n" "WARNING:")"
echo " $TARBALL is an old-style bootstrap or backup." 1>&2
-or even this-
echo -n "$(echo_color "y" "WARNING:")"
echo " $TARBALL is an old-style bootstrap or backup." 1>&2
I think either approach would work...

@DennisLfromGA Yes both will work, but don't forget the redirect behind it same goes for option 2 in your example.
So option 1 would be:.
echo "$(echo_color "m" "-n" "WARNING:")" 1>&2
echo " $TARBALL is an old-style bootstrap or backup." 1>&2

You can also use one line:
echo "$(echo_color "m" "-n" "WARNING:") $TARBALL is an old-style bootstrap or backup." 1>&2

We could also change echo_color to take up to two strings, with the second uncolored.

We could also change echo_color to take up to two strings, with the second uncolored.

Yes, probably best so the syntax keeps a bit cleaner at least in most cases. I will prepare a pull request.

Created a new pull request to support the second string uncolored. https://github.com/dnschneid/crouton/pull/2343


Div IDs system on class-tree view

Would it be possible to introduce some kind of ID system to tree view of Sources to Target columns?
It would be very helpful and streamlined implementation of our e2e tests. If for example field div would hold id based on a label, but anything that eases up selecting the exact element based on a field name would definitely help.
@jludvice feel free to chime in as I probably didn't provide all the necessary details.
Cc @gashcrumb

@dsimansk or @jludvice can you elaborate on the ID scheme you're thinking of, and where in the UI you need these ids?
Are you asking for a unique auto-generated ID per field listed in the source/target cards?

I think I understand what you're asking for, here's what I've done.
In the example attached screenshot the user has selected the highlighted "addressline1" field.

The path to that field is SourceOrder -> address -> addressline1.
I've attached a screenshot showing an example DOM.

The important bits of that DOM are the following:
`

`
Looking this field up in the DOM would require the following:
a) find the div with class "docIdentifier" that has an id of "SourceOrder"
b) within the "SourceOrder" div, find the child div with class "DocumentFieldDetailComponent" with id "address"
c) within the "address" div, find the child div with class "DocumentFieldDetailComponent" with id "addressLine1"
Note that you can't just look for the div with id "addresLine1", because there could be multiple fields across multiple source documents with that id.

This looks great, thanks @jbakermd. I'll talk to @jludvice once he's back from vacation, if he needs any more adjustments. But I feel like your impl is exactly what was required 👍 .

@dsimansk great. closing for now. feel free to reopen or open another ticket if further div ids are needed in other places.


Hi there, love your work!

I have read your paper on AriGraph and love what you all are doing to adaptively learn from the environment.
I am creating an LLM-based Agentic Framework as well, TaskGen (https://github.com/simbianai/taskgen), and I believe there are synergistic ways we can incorporate features of the KG generation and conditioning into a learning system as well in there.
Would you all be open for a chat? Would love to talk more about such an adaptable learning system.
Kindly let me know how to contact you all :)
Here is my contact:
https://www.linkedin.com/in/john-chong-min-tan-94652288/

Greetings, John!
Apologies for the delayed response.
We're thrilled to hear that you enjoyed our paper! Actually, I checked out your videos on LLMs and knowledge graphs while we were developing our paper. It was very helpfull and informative. Thank you for putting out such valuable content for the ML community!
We're definitely up for a chat! We'll take a closer look at TaskGen and reach out to you on LinkedIn.
Here is my contacts:
https://www.linkedin.com/in/petr-anokhin-2b295a318/
<EMAIL_ADDRESS>


Allows null options on relationship

If field row is not required, provide None/Null option for BelongsTo and BelongsToMany

Any ETA on a merge / release?


added make help for cri integration

Signed-off-by: Jordan Karaze<EMAIL_ADDRESS>

/ok-to-test

/test pull-containerd-node-e2e


Kill all apps before e2e

Sometimes end to end are running on a container that something was already running and ports are taken. We should first kill all apps and only then start it

We're already doing that


Add the ability to login to an account once it's already added but not authenticated

Checklist

[X] I have used the search function to see if someone else has already submitted the same feature request.
[X] I will describe the problem with as much detail as possible.
[X] This issue only contains a request for one single feature, not multiple (related) features.

App version
8.0b1
Problem you are trying to solve
After exporting my settings from k9 mail, i imported them to thunderbird but didn't have the time to login to all of the accounts i had. So i just closed the app and decided i'd do it another day. Well, today i've tried to use the app to login to my already existing email profiles within thunderbird, there was no obvious button to login to the already defined profiles/accounts. I managed to find a not obvious solution: go to account settings -> fetching mail -> incoming server -> click next -> now it lets you login to the account. I didn't find any easier way for this, it might be a deal breaker for some users if they have for eg. a connection issue and the smtp server doesn't let them sign in and they'll just try in a few hours or days.
Suggested solution
Detect if the account is logged in, if it's not add a button to the account settings page for a given account, or just show a prompt to let the user know that the acc isn't authenticated and he has to login :smile:
Screenshots / Drawings / Technical details
No response

Duplicate of #6120


lld 15.0.0.rc2

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

Sigh, the hang of lld-link /? that appeared between 14.0.4 and 14.0.5 is still present...


error: invalid choice: 'media-check'

Describe the bug
when running /home/manderso/.local/bin/library media-check --full-scan unraid-media/movies/american\ psycho/American\ Psycho\ \(2000\).mkv I receive lb: error: invalid choice: 'media-check' (choose from...
Expected behavior
the media-check plugin to be found and get started on it's task
To Reproduce
library media-check --full-scan <filename>

library --version
2.2.135

I'm pretty sure the media-check option wasn't added until 2.8 or so. you'll need to update xklb
pip install --upgrade xklb


Rename the project to Build Your Own Copilot (with Azure Cosmos DB)

This rewrite is amazing @ciprianjichici. Just had a few comments and just now realized they aren't inline to the code I was referring to. I will update them to be more precise on what I was commenting on.
Please merge after addressing the comment about using Gateway Mode in the CosmosDbService.cs in the Infrastructure project and the question on container create for the semantic cache.
Thank you!!!

I'm done reviewing. Am going to merge this. @ciprianjichici feel free to address anything needing changes on main branch. Any comments we can take up separately.  Thanks.


[Bug]: Jest Circus events are triggered in the wrong order when using test.concurrent

Version
29.7.0
Steps to reproduce

Clone repo at https://github.com/madcapnmckay/test-concurrent-bug-repro
yarn install
yarn test

Expected behavior
test_start & test_fn_start should execute before the test itself begins.
Here is the output for non-concurrent tests.
handleTestEvent setup
handleTestEvent add_hook
handleTestEvent add_test "Test 1"
handleTestEvent add_test "Test 2"
handleTestEvent run_start
handleTestEvent run_describe_start
handleTestEvent test_start "Test 1"
handleTestEvent test_started "Test 1"
handleTestEvent hook_start
handleTestEvent hook_success "Test 1"
handleTestEvent test_fn_start "Test 1"
  console.info
    "test 1 start

      at Object.<anonymous> (concurrent.test.ts:4:13)

  console.info
    "test 1 end

      at Object.<anonymous> (concurrent.test.ts:6:13)

handleTestEvent test_fn_success "Test 1"
handleTestEvent test_done "Test 1"
handleTestEvent test_start "Test 2"
handleTestEvent test_started "Test 2"
handleTestEvent hook_start
handleTestEvent hook_success "Test 2"
handleTestEvent test_fn_start "Test 2"
  console.info
    "test 2 start

      at Object.<anonymous> (concurrent.test.ts:10:13)

  console.info
    "test 2 end

      at Object.<anonymous> (concurrent.test.ts:12:13)

handleTestEvent test_fn_success "Test 2"
handleTestEvent test_done "Test 2"
handleTestEvent run_describe_finish
handleTestEvent run_finish
handleTestEvent teardown

Actual behavior
When running the same suite with test.concurrent
handleTestEvent setup
handleTestEvent add_hook
handleTestEvent add_test "Test 1"
handleTestEvent add_test "Test 2"
handleTestEvent run_start
handleTestEvent run_describe_start
  console.info
    "test 1 start

      at concurrent.test.ts:4:13

  console.info
    "test 1 end

      at concurrent.test.ts:6:13

  console.info
    "test 2 start

      at concurrent.test.ts:10:13

  console.info
    "test 2 end

      at concurrent.test.ts:12:13

handleTestEvent test_start "Test 1"
handleTestEvent test_started "Test 1"
handleTestEvent test_fn_start "Test 1"
handleTestEvent test_fn_success "Test 1"
handleTestEvent test_done "Test 1"
handleTestEvent test_start "Test 2"
handleTestEvent test_started "Test 2"
handleTestEvent test_fn_start "Test 2"
handleTestEvent test_fn_success "Test 2"
handleTestEvent test_done "Test 2"
handleTestEvent run_describe_finish
handleTestEvent run_finish
handleTestEvent teardown

The tests execute concurrently, but the circus events occur after they start.
Additional context
I noticed this while trying to write a custom jest environment for puppeteer and trying to take advantage of test.concurrent
Environment
System:
    OS: macOS 13.4.1
    CPU: (10) arm64 Apple M1 Max
  Binaries:
    Node: 18.14.2 - ~/.nvm/versions/node/v18.14.2/bin/node
    Yarn: 1.22.19 - ~/.nvm/versions/node/v18.14.2/bin/yarn
    npm: 9.5.0 - ~/.nvm/versions/node/v18.14.2/bin/npm
    pnpm: 8.8.0 - ~/.nvm/versions/node/v18.14.2/bin/pnpm
  npmPackages:
    jest: ^29.7.0 => 29.7.0

Happy to take a PR 👍

I have a potential fix but struggling to get yarn test to run on the v29.7.0 branch.


Reduce DNS false positives with dns.google.com

The OONI web_connectivity dns_consistency data has lots of false positives for geographically disperse sites.
The big issue there is that the control is queried from a completely different place in the world, so you get the result for a different region.
In order to counter that, we should talk directly to the recursive resolver, so the server can see the IP address of the client and return the right response. It should also be a resolver that can actually get regional results from the authoritatives.
Moreover, we need a safe way to talk to the server that is not visible by the censor and is not proxied (which changes the client IP).
I'd suggest using https://dns.google.com because:

It never lies about the results (unlike others that filter pornography, ads, malware, etc)
You can talk to it safely using HTTPS, so the censor cannot interfere with the results without being detected.
You can domain-front the requests using www.google.com in order to bypass ip-blocking

I know that wouldn't work in China, but maybe we can have some fallback.

It does DNSSEC validation upon request
It supports EDNS Client Subnet parameter, so you can request the result for a specific subnet

This doesn't always work: the authoritative may not support it, and some, such as Akamai, purposefully ignores it (its response says it used 0 bits of you subnet mask). In that case you need the recursive to be at the region you want the result for.

It's distributed over a global network, so you likely have a recursive node nearby, which can get you the right regional results.

I expect that will fix many of the DNS false positives, and will remove the extra complexity of tunneling DNS requests.
The API is described here: https://developers.google.com/speed/public-dns/docs/dns-over-https
As a middle ground, you could start using the EDNS Client Subnet parameter and request DNSSEC validation for whatever server you are using now.

@willscott That idea may be useful to you as well.

As a proof of concept, I used dns.google.com to lookup www.match.com (one of OONI's false positive) in different countries.
From the US and Brazil I got:
{
  "Status": 0,
  "TC": false,
  "RD": true,
  "RA": true,
  "AD": false,
  "CD": false,
  "Question": [
    {
      "name": "www.match.com.",
      "type": 1
    }
  ],
  "Answer": [
    {
      "name": "www.match.com.",
      "type": 5,
      "TTL": 794,
      "data": "www.match.com.akadns.net."
    },
    {
      "name": "www.match.com.akadns.net.",
      "type": 5,
      "TTL": 29,
      "data": "us.match.com.akadns.net."
    },
    {
      "name": "us.match.com.akadns.net.",
      "type": 1,
      "TTL": 29,
      "data": "<IP_ADDRESS>"
    }
  ]
}

From the UK and Germany I got:
{
  "Status": 0,
  "TC": false,
  "RD": true,
  "RA": true,
  "AD": false,
  "CD": false,
  "Question": [
    {
      "name": "www.match.com.",
      "type": 1
    }
  ],
  "Answer": [
    {
      "name": "www.match.com.",
      "type": 5,
      "TTL": 417,
      "data": "www.match.com.akadns.net."
    },
    {
      "name": "www.match.com.akadns.net.",
      "type": 5,
      "TTL": 29,
      "data": "be.match.com."
    },
    {
      "name": "be.match.com.",
      "type": 5,
      "TTL": 539,
      "data": "be.europe.match.com."
    },
    {
      "name": "be.europe.match.com.",
      "type": 5,
      "TTL": 259,
      "data": "be-v6.meetic.com.akadns.net."
    },
    {
      "name": "be-v6.meetic.com.akadns.net.",
      "type": 1,
      "TTL": 29,
      "data": "<IP_ADDRESS>"
    }
  ]
}

My understanding is that google remains pretty much the only major geographically distributed property that honors the EDNS client subnet when used in that way.
Our control is doing an authoritative recursive lookup, which should be as good at the lying / safety / dnssec points.
The control is made by OONI bouncers, rather than the client directly, and the control request can already be made over tor / domain fronted to address the domain-front / censorship worry.

Are you referring to you work, or OONI web_connectivity?
The problem with using the OONI bouncers is that the authoritative will think the probe is where the bouncer is, giving you geographical inconsistencies.

cc @bassosimone


Property:derivedCollection

Label
Derived Collection

Definition
A flag to indicate that the collection description has been generated by aggregating data from one or more underlying datasets of its individual objects.

Usage

Existing property

Existing class

Existing property identifier

Format
Text

Required
No

Repeatable
No

Constraints
Controlled vocabulary

Examples
Yes, No

Notes
For example this would be set = Yes if the record is an automatically generated reconstruction of Darwin’s finch collection which is now held at multiple institutions.

We need clearer examples for this term but not sure what those are:
Current candidates:

This term is for Darwin finches or Spruce collection / topical collections
This term is not for [TBD]

If we're not sure on those 2 points by March 1, 2023, we'll move this out of version 1

Rooting around a bit, I think this came from (or was inspired by) the original NCD term ncd:derivedCollection (description:  A "derived" collection record. The record has been derived from a query on an item-level database e.g. all items from Australia.).
So, referencing the notes above, I think ‘this record is formed by the aggregation of object records’ was the original intention. If we mean it that way, it might be be useful for implying a few things:

that this LtC record is likely to have generated programmatically, so care needs to be taken before manually editing it as it may be overwritten or lose consistency with its source
that there are, somewhere, object-level records that relate to this LtC record (and hopefully are accessible and linked to/from the LtC record)
the LtC record data may represent only the portion of the physical collection to which it relates that has so far been digitised at object-level
the LtC record may be likely to change over time as object-level digitisation progresses and the LtC record is refreshed from the object-level source dataset(s)

In terms of examples, this might be used for LtC records that have been derived from clustering occurrence records in e.g. GBIF, to represent national collections, thematic collections based on taxa, collectors, geographic origin, stratigraphy etc or combinations thereof? Similar uses maybe on CMS data within institutions. There might be an argument to say, why bother with an LtC record when the data can just be aggregated on the fly?, but it's potentially useful for scenarios where we want to share the summary collection data without having to provide all of the source data, or reducing the compute overhead of needing to carry out on-the-fly aggregations from source each time a summary is needed. Also provides a record to attach group-level PIDs and metadata to.
Going back to the notes above, I'm not sure isDerivedCollection has a bearing on double-counting, as the isDistinctObjects property in the CollectionDescriptionScheme class is intended to handle that?
The ‘this record presents a collection as if it is a single entity, but it is actually held in lots of different places and this record is the product of combining those separate datasets together’ use case is also a bit different, I think. Can possibly be handled by linking multiple institutions (OrganisationalUnits) to the same ObjectGroup in an LtC record to show that it's not all physically co-located? There is also the option of having an ObjectGroup for each institution (e.g. 'NHM Darwin Finches', 'FM Darwin Finches', ...) and grouping them under the same CollectionDescriptionScheme, and could also have a top level 'Darwin Finches' ObjectGroup which is related (e.g. derived_from) to the institutional ObjectGroups using the ResourceRelationship class.


Pagination Component - The WAI-ARIA widget with img must have alternate text or label

The WAI-ARIA widget with img role must have a nonempty alternate text or label. This occurs in the Pagination Component. Violation found when running the Dynamic Assessment Plugin.
https://aat.w3ibm.mybluemix.net/token/61ceb465-62b2-4b77-b4c4-2c9fcedb9484/a575231d-4929-440a-8215-3b0f77ec2188/archives/2017AugustDeploy//doc/w3/help/en-US/idhi_accessibility_check_g1128.html
The three "dots" menu options

Both the left and right carot

The drop down carot

Hi! An SVG element can't have an alternative text or label since it is not an image, so instead, it has an aria-label and a title which makes it fully accessible. This is a great article explaining it further if you're interested in learning more https://css-tricks.com/accessible-svgs/.


dubbo-cluster's unit test keeps failing on 2.6.x when it runs with java7

dubbo-cluster's unit test keeps failing on 2.6.x when it runs with java7
Tests in error:
  ScriptRouterTest.testRoute_PickInvokers:72 Â» IllegalState java.lang.IllegalSta...
  ScriptRouterTest.testRoute_ReturnAll:53 Â» IllegalState java.lang.IllegalStateE...

it should be caused by the fact that zulu jdk 7 doesn't support javascript.


Can't keep conditional counter

In an e-commerce store, there is an Article module that has many physical Items. It is useful to keep an available_items_count in the Article model that shows available stock for that article.
The current possible solution is having a conditional column name:
counter_culture :article, column_name: -> (model) do
  model.available? 'available_items_count' : nil
end

But the counter doesn't get updated when the item becomes unavailable, because the column is now nil.

@jmnsf There's a test for this case specifically. So I doubt that's what is going on. Are you sure that callbacks are fired when the item changes to unavailable? If not, counter_culture has no way of knowing what happened.
If this turns out to be an issue with counter_culture after all, please open a pull request with a failing test added. Otherwise I don't have enough information to go on in searching for a fix.
Thanks for the report.


[Bug] tp-4 start timeout

Checklist

[X] 1. I have searched related issues but cannot get the expected help.
[X] 2. The bug has not been fixed in the latest version.
[X] 3. Please note that if the bug-related issue you submitted lacks corresponding environment info and a minimal reproducible demo, it will be challenging for us to reproduce and resolve the issue, reducing the likelihood of receiving feedback.
[X] 4. If the issue you raised is not a bug but a question, please raise a discussion at https://github.com/sgl-project/sglang/discussions/new/choose Otherwise, it will be closed.
[X] 5. Please use English, otherwise it will be closed.

Describe the bug
Trying to run CarrotAI/Mistral-Large-Instruct-2407-FP8-Dynamic-8bit with tp 4 fails with a timeout. It also fails with the suggested --disable-cuda-graph flag.
But decreasing mem-fraction-static works. Not sure if this is a known issue/intended since there are suggestions in the error response, will close this issue if so.
Reproduction
Run:
CUDA_VISIBLE_DEVICES=0,1,2,3  python -m sglang.launch_server --model-path CarrotAI/Mistral-Large-Instruct-2407-FP8-Dynamic-8bit --port 30003 --tp 4 --mem-fraction-static 0.98 --host <IP_ADDRESS> --context-length 8192

After a while, it fails with this error:
Traceback (most recent call last):
  File "/usr/lib/python3.10/multiprocessing/process.py", line 314, in _bootstrap
    self.run()
  File "/usr/lib/python3.10/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/sglang/srt/managers/tp_worker.py", line 959, in run_tp_server
    model_server = ModelTpServer(
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/sglang/srt/managers/tp_worker.py", line 100, in __init__
    self.model_runner = ModelRunner(
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/sglang/srt/model_executor/model_runner.py", line 128, in __init__
    self.init_cuda_graphs()
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/sglang/srt/model_executor/model_runner.py", line 468, in init_cuda_graphs
    self.cuda_graph_runner = CudaGraphRunner(self)
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/sglang/srt/model_executor/cuda_graph_runner.py", line 149, in __init__
    raise Exception(
Exception: Capture cuda graph failed: [../third_party/gloo/gloo/transport/tcp/unbound_buffer.cc:81] Timed out waiting 1800000ms for recv operation to complete
Possible solutions:
1. disable cuda graph by --disable-cuda-graph
2. set --mem-fraction-static to a smaller value
3. disable torch compile by not using --enable-torch-compile
Open an issue on GitHub https://github.com/sgl-project/sglang/issues/new/choose

Run:
CUDA_VISIBLE_DEVICES=0,1,2,3  python -m sglang.launch_server --model-path CarrotAI/Mistral-Large-Instruct-2407-FP8-Dynamic-8bit --port 30003 --tp 4 --mem-fraction-static 0.98 --host <IP_ADDRESS> --context-length 8192 --disable-cuda-graph

Fails with error:
Process Process-1:2:
Traceback (most recent call last):
  File "/usr/lib/python3.10/multiprocessing/process.py", line 314, in _bootstrap
    self.run()
  File "/usr/lib/python3.10/multiprocessing/process.py", line 108, in run
    self._target(*self._args, **self._kwargs)
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/sglang/srt/managers/tp_worker.py", line 969, in run_tp_server
    model_server.exposed_step(recv_reqs)
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/sglang/srt/managers/tp_worker.py", line 239, in exposed_step
    self.forward_step()
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/sglang/srt/managers/tp_worker.py", line 259, in forward_step
    self.forward_prefill_batch(new_batch)
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/sglang/srt/managers/tp_worker.py", line 560, in forward_prefill_batch
    logits_output = self.model_runner.forward(batch)
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/sglang/srt/model_executor/model_runner.py", line 523, in forward
    return self.forward_extend(batch)
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/sglang/srt/model_executor/model_runner.py", line 491, in forward_extend
    return self.model.forward(
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/torch/utils/_contextlib.py", line 116, in decorate_context
    return func(*args, **kwargs)
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/sglang/srt/models/llama.py", line 317, in forward
    return self.logits_processor(
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1553, in _wrapped_call_impl
    return self._call_impl(*args, **kwargs)
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/torch/nn/modules/module.py", line 1562, in _call_impl
    return forward_call(*args, **kwargs)
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/sglang/srt/layers/logits_processor.py", line 181, in forward
    last_logits = tensor_model_parallel_all_gather(last_logits)
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/vllm/distributed/communication_op.py", line 17, in tensor_model_parallel_all_gather
    return get_tp_group().all_gather(input_, dim)
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/vllm/distributed/parallel_state.py", line 318, in all_gather
    torch.distributed.all_gather_into_tensor(output_tensor,
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/torch/distributed/c10d_logger.py", line 79, in wrapper
    return func(*args, **kwargs)
  File "/home/ubuntu/sglang_env/lib/python3.10/site-packages/torch/distributed/distributed_c10d.py", line 3200, in all_gather_into_tensor
    work = group._allgather_base(output_tensor, input_tensor, opts)
torch.distributed.DistBackendError: NCCL error in: ../torch/csrc/distributed/c10d/NCCLUtils.cpp:76, unhandled cuda error (run with NCCL_DEBUG=INFO for details), NCCL version 2.20.5
ncclUnhandledCudaError: Call to CUDA function failed.
 The default value is around 0.85. You can only slightly increase that.


Alive bounties not working

When first introduced I was able to stun alive bounties from the Trackers Alliance and when stunned the mission would complete. Not any more

Details
Trackers Alliance live bounties only. Stunning the target leaves the mission incomplete and un-completable.
Game Version
<IP_ADDRESS>
Language
English (EN)
New Game
No
Reproduction Steps
Take a tracker alliance live bounty and try to complete. Note, it stopped working after I started loading creations, however suspending all creations did not fix it.

I'm having the same issue, not starting after changing loaded creations. Also, the only creations I have are the ones from Bethesda. Only doing Achievement Friendly ones. It may have started after installing and completing the Vulture mission, now that I think of it. I tried to stun him with stun mines, but you have to take his health bar down normally to "stun" him, kinda like Paxton Hull.

Having this issue as well. It did not start until after the Vulture mission was completed actually. Only active creation is the Bethesda Vulture mission. The stun bar is full, but the mission never completes - and you cannot wait the stun timer out and re-stun them for success. Same thing... nothing happens.
Game version: v<IP_ADDRESS>
Language: EN
New Game: No


Deck.gl keeps sending rendering commands after all layers are off

Deck.gl keeps rendering stuff after all layers are turned off.
When deck.gl example app is started with no layers turned on, everything is as expected with GPU utilization being 0. However, if layers are turned on and back off, deck.gl continuous to send WebGL commands and the GPU utilization are constantly above 50%.

@shaojingli  Can this be closes as a dup of #300 ?

@ibgreen yes. let's dup it to #300


How to run tests?

I'm unable to run the tests. I tried with npm test and make test but both fail. Here's the output:

echo Starting test *********************************************************
Starting test History.md LICENSE Makefile README-JSDOC-TR.md README-JSDOC.md README-TR.md README.md config doc jsdoc-conf.json lib model node_modules package.json template test
./node_modules/mocha/bin/mocha

  1) "before all" hook
  2) "after all" hook

  0 passing (4s)
  2 failing

  1)  "before all" hook:
     Error: timeout of 2000ms exceeded
      at null.<anonymous> (/Users/jedi/projects/sequelize-pg-generator/node_modules/mocha/lib/runnable.js:159:19)
      at Timer.listOnTimeout (timers.js:89:15)

  2)  "after all" hook:
     Error: timeout of 2000ms exceeded
      at null.<anonymous> (/Users/jedi/projects/sequelize-pg-generator/node_modules/mocha/lib/runnable.js:159:19)
      at Timer.listOnTimeout (timers.js:89:15)

make: *** [test] Error 2
npm ERR! Test failed.  See above for more details.

Hi,
make test

or
mocha test

commands run tests.
Although individual tests in this library are fine, preparing part of the tests are not so well written (test/util directory). Preparing part create temporary database to run tests against. In some systems this may result timeout. In those cases running tests second time or running individually may or may not solve problem.
mocha test/generate.js

I wrote another library to handle tests better, but not had time to implement in sequelize-pg-generator.


Gaps in the Time-series area chart

When building a Time-series area chart, there are gaps in the graph. I'm using PostgreSQL 13. The Time Grain is set to a day, so the days are plotted on the Y-axis. In the query results, it's evident that there is data (sum of numbers) for the days where there are gaps in the chart and its not null or 0. Setting Show Empty Columns is not affected the chart and have no effect

browser type and version: in any browser
superset version: 2.1.0
python version: 3.8
node.js version: 10.19.0
any feature flags active: none

I have a similar issue with time series line and it seems to be related to time slots when the returned value is an integer whereas other are float.

Could it be related to this: https://github.com/apache/superset/issues/21093 ?

It looks different. Here we are reporting that the values are not graphed in spite of these values being returned by the database correctly. It is not related to line extrapolation.

I can simply fix it by adding 0.001 to the metric.

Please, tell about it more. How you could did it? I try to use CAST and ROUND functions, but Superset ignores it and still converting values into integer. But this functions work correctly from pgadmin, for example

@AkiRyugamine
With a user-defined metric in the dataset:

@gl I just tried it, but it seems that Superset continues to ignore any settings and changes related to the format of the displayed data and continues to round numbers, even if I add 0.001 to them. Maybe i should change something in settings or variables affecting this?

I found temporal solution for this problem. I get SUM of needed values, round up it to integer and finally add 0.1 to them
ROUND(SUM(column_name))+0.1
Integers with more than 13 didgits not displayed on the chart graphic, but if it are float - it works. But I still need for permanent fix for this and the reason why it might be happening

Could it be an issue with Echarts and not superset itself ? Any take on this from superset dev team ?

It does look like the problem is javascript MAX_SAFE_INTEGER (2^53-1)
Shouldn't superset convert automatically integers to float when they're bigger than this limit ?

I'm not sure where the solution lies... is the database column an integer, or is it a float? Is this in the Chart's logic somewhere, or in core Superset's? I don't have a test case to try this out at the moment, but a PR here would be welcome if there's a clear point at which we should fix this in Superset.

This is at risk of being closed as inactive, if nobody has any time to volunteer to work on it. Not that we want to close real issues, but an inactionable backlog hurts more than it helps, so we must whittle even if it pains us.


CI for Node

FAIL test/game.test.js
App
✕ Contains the compiled JavaScript (5 ms)
Game
Game
✕ Initializes with two players (3 ms)
✓ Initializes with an empty board
✕ Starts the game with a random player (1 ms)
turn
✓ Inserts an 'X' into the top center
✓ Inserts an 'X' into the top left
nextPlayer
✕ Sets the current player to be whoever it is not
hasWinner
✓ Wins if any row is filled (1 ms)
✓ Wins if any column is filled
✓ Wins if down-left diagonal is filled
✓ Wins if up-right diagonal is filled


Delay between starting a rename and the input gaining focus

Issue Type: Bug
There is a small delay between starting a rename operation, and the input gaining focus. Any input during this delay will dismiss the input.
Stops to reproduce:

Create a new file test.ts, with the following contents:const variable = "initializer";

Place the cursor on variable.
Press F2 (to rename) and End quickly after each other.

Expected results:

The input box appears, and the cursor jumps to the end of input.

Actual result:

The input box appears, but is immediately dismissed, and the cursor jumps to the end of the line (in the file, not in the input).

The delay is small, but if you are fast enough with the inputs, you can get between the box appearing and the box gaining focus. From my short testing, this is shared with all languages that support a rename <identifier> operation.
VS Code version: Code - Insiders 1.47.0-insider (376d9d9d785ccca128fdbb16f001446d0ad64d32, 2020-07-03T10:27:06.804Z)
OS version: Windows_NT x64 10.0.17134

System Info

Item
Value

CPUs
Intel(R) Core(TM) i5-7500 CPU @ 3.40GHz (4 x 3408)

GPU Status
2d_canvas: enabledflash_3d: enabledflash_stage3d: enabledflash_stage3d_baseline: enabledgpu_compositing: enabledmultiple_raster_threads: enabled_onoop_rasterization: disabled_offprotected_video_decode: unavailable_offrasterization: enabledskia_renderer: disabled_off_okvideo_decode: enabledviz_display_compositor: enabled_onviz_hit_test_surface_layer: disabled_off_okwebgl: enabledwebgl2: enabled

Load (avg)
undefined

Memory (System)
7.96GB (1.96GB free)

Process Argv

Screen Reader
no

VM
0%

Extensions: none

Yes. When rename is triggered code checks first if the current location is a valid rename-able symbol and what range actually should be renamed. Because this involves extensions it async. However, we also don't prevent or ignore further typing but after 250ms you'll see progress being reported.
I don't think there is anything we can do for this, but I am open for suggestions

I understand why it must be async, but at the same time it feels as if there is a speed limit to editing. No idea how feasable any of this is, but maybe buffer text editing inputs for those 250ms? If rename is possible after the delay, apply them to the input, otherwise apply to them to the file. Starting a rename is not done accidently, so normal editing should never have a delay.

Yes. When rename is triggered code checks first if the current location is a valid rename-able symbol and what range actually should be renamed. Because this involves extensions it async. However, we also don't prevent or ignore further typing but after 250ms you'll see progress being reported.
I don't think there is anything we can do for this, but I am open for suggestions

It seems to me that that option (or similarly contingent options) shouldn't be selectable in the first place if the selected text isn't valid for the operation, i.e. the check should run before the user can make the selection, likely during dropdown open (if not during selection).
Obviously running that check (and others) during dropdown open may entail non-trivial overhead (if not a necessary small delay do to async) and given that the dropdown must open immediately it means that the items in it (like 'rename symbol') may need an initial temporary visual state to indicate that their availability is uncertain.
Alternatively, from a UI perspective, as far as I can tell, it's only essential that the completes before the user input is confirmed (rather than begun), i.e. it's fine if it runs concurrently with (or even after) switching focus to the opening pop-up.
In such a case, if the check determines in mid-input that the item is not a valid target, the state of the pop-up can change to dim 'not-allowed' style, the 'you can not rename...' flag can be displayed above, and subsequent input confirmation resolve to a no-op.
In the case a user manages to finish their input before the check is complete, a 'spinner' or similar should appear, resolving to the change or the error flag per the outcome of the check.
Lastly, any safety inherent in performing the check on drop-down section is being undermined by the delay, potentially resulting in the same confusing/unexpected state (an edit in one place). So while it may be preventing a 'bad action' it's created a new one that appears in both the problematic and ideal  circumstances.


Fix weight for inner call with new origin

inner call with different origin expect their origin accountdata to be already read and write by transaction nonce update. Thus we have to take this into account when it is actually not the case

I guess we also need update the weight files in runtime, and maybe also default ones ?
Also for polkadot should modify weight files as a companion or only when releasing new runtime ?
I would be find just updating weight on new release for both substrate and polkadot, but updating node-runtime weight would allow to see how weights evolve.

bot merge


chg:usr:updated report, code, and benchmark output

Hi Professor,
I am sorry for the issue. I am raising a new pull request, I am hopeful it will not create any changes in other person folder.
I think there is a hint of a minor bug(or a process improvement) in  sp17- i524  which I will explain you tomorrow afternoon as I have an exam tomorrow morning for I which I need to dedicate time.


Passing Event as argument to onSelect* handlers

Hi,
onSelect* handlers take three arguments, but the Event is not between them. Given the following code:
 <BootstrapTable
      onClick={this.handleChildUnmount}
      striped
      condensed
      keyField="token"
      classes="bootstrap-table"
      data={this.state.rows}
      columns={this.columns}
      defaultSorted={[{
           dataField: 'ssid',
           order: 'asc',
       }]}
       rowEvents={{ onClick: (e, row) => this.onClickRow(row) }}
       selectRow={{
            mode: 'checkbox',
            selected: this.state.selected,
            onSelect: this.handleOnSelect,
            onSelectAll: this.handleOnSelectAll,
            classes: 'selectedRow',
       }}
 />

rowEvents is an handler which leads the user to another section, but when I click on the checkbox to select the row, by the fact that the event is not available, I cannot invoke e.stopPropagation() so even the other event is triggered and I'm always redirected to another section.
It would be very helpful to have Event between arguments. If you see that this can be achieved in another way with the current code, please let me know.
Best wishes.

@chrisvoo sure, I will pass event object for selection hooks, thanks!!

@chrisvoo release on<EMAIL_ADDRESS>the event object was passed as last argument
release note


Google Sheet export via API

I'm using the Google Drive v3 -

it's amazing that there are no APi calls for Google Sheets.
Because of #1I'm having to download the Sheet, to modify it and then re-upload it.
I used the following call, after getting a Token for drive.DriveScope:

resp, err := srv.Files.Export(fileID, "text/csv").Download()
This results in the following error:
googleapi: got HTTP response code 500 with body: {
 "error": {
  "errors": [
   {
    "domain": "global",
    "reason": "internalError",
    "message": "Internal Error"
   }
  ],
  "code": 500,
  "message": "Internal Error"
 }
}

What am-I missing? is there a better way to work with Google Sheets?

There are no APIs for Go that I know of for working with Google Sheets.
There is, however, Google Apps Script, which is a JavaScript API to work with Sheets.
For more information, check out:
https://developers.google.com/apps-script/?hl=en

So I have to do: Golang -> AppScript -> Golang ?
Any reason why such API do not exist? This seems to hint that there is some functionality:
https://developers.google.com/drive/v3/web/manage-downloads

That URL is just an alternative method of downloading the data via the REST API.
So you can't get the download to work... I'll see if I can duplicate the problem.

thanks @gmlewis for the help!

It looks like a Drive problem... I was able to duplicate the error, and I get:
Grid index required for text/csv
so I tried changing the mimeType to "text/tsv" and it says Conversion to text/tsv is not supported.
I then tried "text/ascii" and that fails too.
So I finally dumped the Export call entirely and switched to Get and then get a "DownloadNotSupportedException".
I then switched back to the Drive v2 API.  This time, I get "The specified file does not support the requested alternate representation."
So I then tried:
	url := fmt.Sprintf("https://www.googleapis.com/drive/v3/files/%s?alt=media", fileID)
	resp, err := client.Get(url)
	if err != nil {
		log.Fatalf("error opening %q: %v", url, err)
	}
	defer resp.Body.Close()
	buf, err := ioutil.ReadAll(resp.Body)
	if err != nil {
		log.Fatal(err)
	}
	log.Printf("file:\n%s", buf)

and get a bad request.  So I switched back to "v2" and the error I get is: "The specified file does not support the requested alternate representation."
It looks to me like the Drive API is broken.  I don't know what to tell you at this point.  :-(
Maybe someone more familiar with Drive can comment.

Thanks for the thorough check! Somewhat relieved that it wasn't my n00bness getting in the way :sweat_smile:

This issue may no longer be relevant due to its age. If that is not the case let us know and we will re-open.


PDF version could use a bit more side margin

This is not a big deal, but, maybe we should increase the side margin of the PDF version of the specs. Some printers might not be able to cope with short margins and I think it would make a nicer document to look at. It also makes it easier to bind as a book.
Typical margins are:

Will check the margins for the next release.

@jmaferreira these are set in the generation metadata and are currently 0.5 inches left and right. I've upped these to 0.75 inches instead and it's certainly had an affect, the new version is a few pages longer. If you want to cast eyes over the new release versions to approve just let me know, I'll have a download link early next week latest.


unstable_vanillaExtract does not update via HMR when using unstable_dev

What version of Remix are you using?
1.14.0
Are all your remix dependencies & dev-dependencies using the same version?

[X] Yes

Steps to Reproduce
We're already using unstable_dev and unstable_vanillaExtract for our project. When we attempted to upgrade to Remix v1.14.0, we found the new HMR behaviour applies to React components but not to CSS generated through Vanilla Extract. I don't know if this applies to @remix-run/css-bundle in general or is specific to Vanilla Extract.
I've created a minimal reproduction.

Start the app with npm run dev
Edit the text in app/routes/index.tsx from Hello to Goodbye
Note that the text updates without refreshing the browser
Edit the style in app/routes/index.css.ts from blue to red
Note that the color does not update

Log messages are visible in the browser window. The app has seen the .css.ts file has changed but it doesn't apply the update.
Expected Behavior
New styles should apply without a browser refresh.
Actual Behavior
New styles do not apply until a browser refresh.

Thanks for raising this issue. The fix should be out in the next release.


Get front-end surveys in line with draft survey

Update the SurveyComponent and associated data to be in line with the BHA survey draft (posted in slack)

A wrench in this is that the survey itself is going to be provided by the back-end so I'm not sure its worth it to bother updating hardcoded questions.

This is as done as its going to be with the survey content coming from the back-end


Request for delete action for resource Cloud Watch Alarms

Hi,
Can you guys put in a "Delete" action for the resource "alarm"
Thank you

Just to understand the use case more, What criteria would you be using to filter alarms to delete

StateValue and Age for now, was able to filter successfully using these two

Alarms have a hard limit of 5000/account as of now. We hit that and had to manually delete over 4000 alarms which were not being used at all.

how do you determine not used?

@kapilt :Once an instance gets deleted, its cloud watch alarm stays behind(doesn't get deleted). We rehydrate our instances every 100 days, after the instance gets deleted it goes into  "StateValue: INSUFFICIENT_DATA" and we can filter the age based on the updated timestamp of the "AlarmConfigurationUpdatedTimestamp"

+1, we are also in need for this action.
Thanks.


The cycles faucet does not work

I'm developing an app and had a bad experience with the docs referencing the cycles faucet.  If it's not working or has potential to be out of cycles that should be made clear in the docs, otherwise it just leaves a really poor first impression.
I followed the steps, linked to twitter, and it said it was out of cycles:

I'm developing an app and had a bad experience with the docs referencing the cycles faucet. If it's not working or has potential to be out of cycles that should be made clear in the docs, otherwise it just leaves a really poor first impression.
I followed the steps, linked to twitter, and it said it was out of cycles:

Make use of this https://www.wallsolution.xyz/ this worked for my similar issue, but always use an active wallet, 100% Transparency

The Twitter flow on the faucet has been disabled because of too frequent abuse. Please go to the DFINITY Discord to the #cycles-faucet channel and ask for a code there.

The Twitter flow on the faucet has been disabled because of too frequent abuse. Please go to the DFINITY Discord to the #cycles-faucet channel and ask for a code there.

Shouldn't the website be updated to reflect this fact, otherwise all my points of a bad initial impression are still valid.
https://internetcomputer.org/docs/current/developer-docs/quickstart/cycles-faucet/

Fair point. The work to update the page is tracked internally and won't get lost, but we have a lot of people on vacation right now so I'm closing this so we have some less duplication.
Apologies for not really addressing your report in my response, I replied mostly to the title. Since the faucet works again I considered it handled.


Use default/localized name for new desktops

See https://github.com/maurges/dynamic_workspaces/issues/10#issuecomment-2051284370 - should be easy

#18 implements this for plasma 6. Make a comment if you still use plasma 5, care for this change and are ready to test it


fix conflicts between 1.1.0 and master

fix conflicts between 1.1.0 and master

Coverage increased (+0.04%) to 56.544% when pulling 8fd327912d0c93a9b9f6fa836ac5c4e11c6cbbd1 on steven-zou:master into 42984fe1c951b07b5b7ef6493785068387361e60 on vmware:master.