Merge files to simplify codebase and not repeat things

resolves alexiusacademia/unitconvert#10
All unit and unit type definitions are now located in the unit.go file. A function for generic conversion and functions calling this are implemented in the convert.go file. This means this refactor will not be breaking the suggested api. Test functions are also compiled into a single test file convert_test.go.
    uc.Convert(<value>, <from>, <to>) # generic convert function
    uc.Length(<value>, <from>, <to>) # will check first if unit types are Length Units

As shown, this will not break the current form of the api.

updated the code commenting out the other functions in convert.go since these functions won't really be used in the future anymore.


fix NavigationPaneSize #867

Fixed the problem described in issue #867.

That look good to me, but the formatting is not correct.
Please fix the format (dart format .)
The job Flutter Analysis must pass.


Cleanup: Remove executable bit from files which don't need it

Drop unused xpmfix.sh script.

Cherry-picked for 3.2.4.

Cherry-picked for 3.2.4.


Support countLostReplicas in failover hooks

{countLostReplicas} is now reported to PostFailoverProcesses and PostUnsuccessfulFailoverProcesses

works as expected

Under what conditions will Lost replicas？Now I will also lose replicas when failing over.

Under what conditions will Lost replicas？Now I will also lose replicas when failing over.

Explained in https://github.com/github/orchestrator/blob/master/docs/topology-recovery.md#discussion-recovering-a-dead-master

Hi. Command line text replacement its it countLostReplicas. Which Environment variable? ORC_COUNT_LOST_REPLICAS?


Add GitHub PR Template

:pushpin: References

Issue: None

:tophat: What is the goal?
I've just noticed this repository has no PR template and I've added the same one we use in other projects 😃
How is it being implemented?
Just creating the file you can see below 👇
How can it be tested?
You'll have to create another PR after merging this branch to check it out 😉

Emojis updated!


Handle frozen url params properties

Ember will freeze concatenated properties so they cannot be mutated once applied. While that makes sense, torii's QueryString does not currently work when passed frozen requiredParams or optionalParams as frozen arrays cannot be passed to Ember.A (see discussion here: emberjs/ember.js#15020). This fixes that by simply cloning those properties before passing them to Ember.A.
Closes #337

This was handled by #344. Thanks for the fix, @marcoow !

released (via #344) in v0.8.2 — @marcoow Is your issue fixed when using Torii @ 0.8.2 ?

Yep, works with 0.8.2!


Fix build error.

Update versions of gatsby-plugin-offline and gatsby.
Change options causing conflicts at build time on gatsby-config.js file.
Signed-off-by: Tomás Castillo<EMAIL_ADDRESS>

This should address the build conflicts from https://github.com/magma/magma-website/pull/32.


fix: Rename jq repository url

JQ repository url has been moved to https://github.com/jqlang/jq
This PR renames the references to the new url

Closed because it is duplicated (#9)


Django ORM 2006 mysql gone away

The default timeout on mysql is 28800 seconds (8 hours).
After that, all requests fail till the server is restarted.
We need to find a good solution to this problem.
In the interim I have found a simple hack which relies on  the request_started signal django uses internally.
from functools import wraps

from django.core.signals import request_started

def django_wsgi_hack(func):
    @wraps(func)
    def wrapper(environ, *args, **kwargs):
        request_started.send(sender=func.__class__, environ=environ)
        return func(environ, *args, **kwargs)
    return wrapper

app = App(...)
app.wsgi = django_wsgi_hack(app.wsgi)

Django's default CONN_MAX_AGE is 0 (zero) which closes the connection at the end of each request:  https://docs.djangoproject.com/en/1.11/ref/settings/#conn-max-age
Which specific MySQL connection timeout setting(s) are involved?

@ssteinerx wait_timeout was the one I believe I ran up against.
Django's WSGIHandler uses the request_started signal to close a connection based off the CONN_MAX_AGE setting you're talking about.

So, are you suggesting that DjangoBackend should use CONN_MAX_AGE (or some other, similar setting) to manage the connection to the database?

I am bringing up a problem I faced and have provided a solution that I am using right now.
I don't know how APIStar wishes to solve the problem.
For example,
One could just close db connections by hand in DjangoBackend's build.
Or call the func this signal calls.
Or one could override a connection's execute method and handle similar db errors there.
Or use the hack I'm using here to let other listeners of the signal do their work. Eg: reset query cache.
I don't have an answer, definitely not one that suits the larger community, especially when half the community doesn't even use Django's ORM.
But I'd really like for the conversation to happen. 😊

0.2 is going to bring in support for components as context managers, which will make things easier to resolve as we'll have enter/exit on the ORM component. (Also in due course we'll likely be working towards deployment options for running API Star within either of Flask or Django completely)

This oughta be resolved now, using Session in 0.2 onwards.


Simulate key presses

 Issue by SirCmpwn
Saturday Sep 26, 2015 at 16:35 GMT
Originally opened as https://github.com/Cloudef/wlc/issues/70

I would like to be able to send simulated keypresses to wayland clients.

 Comment by Cloudef
Saturday Sep 26, 2015 at 17:08 GMT

A) wlc would need to support this. B) We could open wayland internals more, we already pretty much expose libinput and xkb to keep simple.

 Comment by SirCmpwn
Saturday Sep 26, 2015 at 17:10 GMT

The use-case for this is an on-screen keyboard, fwiw.

 Comment by Cloudef
Saturday Sep 26, 2015 at 17:11 GMT

Weston implements on-screen keyboards too. It might be useful to see how they solved it.

 Comment by Cloudef
Wednesday Sep 30, 2015 at 19:12 GMT

#71 somewhat related.

 Comment by Cloudef
Saturday Sep 26, 2015 at 17:22 GMT

For approach B), https://github.com/Cloudef/wlc/blob/feature/wayland-extensions/include/wlc/wlc-wayland.h could include function that returns input resources for view. From there on you can do anything wayland-server.h allows.


Send last statsd data on console.terminate

When a console command is run, there is no kernel.terminate event sent, only console.terminate. Unfortunately we need to catch these kind of events in order to flush remaining statsd data.
Just added corresponding listener.

▶ 2.14.2


CI for Node

Contains the compiled JavaScript, Initializes with two players, Starts the game with a random player, Sets the current player to be whoever it is not


Create rust-toolchain.toml

Just to make it easier to run the repo, I added a toolchain file to setup using nightly and rust-src, so people don't have to worry about doing it manually.

Hmm, makes the tests fail since it prevents installing the stable channel. Is there a point to installing stable if the project needs nightly to work?

@JonahPlusPlus We've switched to using stable Rust for compiling the xtask task runner in CI in https://github.com/rust-osdev/uefi-rs/pull/399. The reason was that the nightly toolchain had a bug with compiling fatfs, a dependency of xtask.
That being said, xtask should automatically switch to the nightly toolchain when compiling the crate. Doesn't this eliminate the need for a rust-toolchain.toml file?

@GabrielMajeri Hmm, I don't think so, I tried using xtask to build and it failed (complained about missing stuff), I had to override to nightly and add the component manually.

Does sound like a possible xtask bug. @JonahPlusPlus could you post the error you get (with the nightly override removed)?

Hmmm, actually, I guess the problem was with me running the cargo build command instead of cargo xtask (in respect to why I needed the rust-toolchain.toml), since that creates an error when I run it without any overrides. I must have conflated the two. Still, the xtask command fails, whether I am using nightly or stable:
error[E0433]: failed to resolve: could not find `sys` in `nix`
 --> xtask\src\qemu.rs:8:10
  |
8 | use nix::sys::stat::Mode;
  |          ^^^ could not find `sys` in `nix`

error[E0432]: unresolved import `nix::unistd`
 --> xtask\src\qemu.rs:9:10
  |
9 | use nix::unistd::mkfifo;
  |          ^^^^^^ could not find `unistd` in `nix`

I guess you guys use Linux? I am running Windows with WSL, but since you can build UEFI apps without xtask (cargo +nightly build --target x86_64-unknown-uefi), I have just been using Windows for programming my UEFI app.

Interesting! I am indeed on Linux, and not very familiar with WSL. The error messages are surprising to me though; I would have assumed nix would more or less work without issue under WSL. And I'm especially confused that the entire sys and unistd modules would be missing, because as far as I can tell those modules are not conditional in any way. Many of the nix modules are gated behind a feature flag or target triple, but it doesn't look like sys or unistd are gated:

https://github.com/nix-rust/nix/blob/v0.24.1/src/lib.rs#L141
https://github.com/nix-rust/nix/blob/v0.24.1/src/lib.rs#L158

@nicholasbishop I think I've figured out why nix cannot be used on Windows. The root lib.rs file for the crate has a #![cfg(unix)] line right at the top, which would make the crate appear completely empty and devoid of members on non-Unix platforms.

@nicholasbishop I guess I should clarify: I have WSL 2, but I was building on Windows, not WSL. If I run xtask from Debian on WSL, it works fine. You just can't use xtask at all on Windows itself.

Edit: I only brought up WSL since I used to use it, but since you can go without xtask to build UEFI apps on Windows, I've just been using Windows lately. It's been a while since I last programmed UEFI, but this must have been the reason I stopped using xtask and just used cargo build.

Thanks, makes sense. Assuming that only the qemu part of xtask is causing problems, perhaps we can use some conditional compilation there to make the rest of xtask work on Windows.

On that note however, why are you guys using xtask to build when cargo provides targets for the same architectures?

The background there is that we need tools to run a variety of commands, not just the build command. Running the qemu tests is the most notable one; it takes a fair amount of code to set that all that up. Originally this code was written in Python, but I rewrote it in Rust (to get all the usual nice checks that Rust provides). Since that xtask code needs to run under the host's target, rather than one of the UEFI targets, we can't have a single cargo.toml config for everything. And in fact even without the xtask code, we need to build for multiple targets since there are three UEFI targets we support (ia32, x86_64, and aarch64).

The background there is that we need tools to run a variety of commands, not just the build command. Running the qemu tests is the most notable one; it takes a fair amount of code to set that all that up. Originally this code was written in Python, but I rewrote it in Rust (to get all the usual nice checks that Rust provides).

Ah, makes sense. What I did for my project was use cargo make, using duckscript to create the output folder and then just run qemu as a command. Granted my bootloader doesn't work (I can't find any resources for how the boot process works in UEFI without some legacy BIOS code getting in the way), but this structure is nicely cross platform.
I'm not sure how cargo make handles arguments, but you probably could get it to compile to multiple platforms by passing a target argument or by making specific commands for each platform.

I've put up a PR that hopefully addresses the Windows compilation issue.


silence spotbugs warnings

i tried to silence a spotbugs warning with:
import edu.umd.cs.findbugs.annotations.SuppressFBWarnings;

        @SuppressFBWarnings
        private T t;

but the dependency is not known. can you pleaes include it into the plugin so one does not need it extra in the project?

The dependency needs to be available at compile time, otherwise the code won't compile. Spotbugs brings in its own dependencies which are needed for the analysis, i.e. at runtime of the scan. These are two very different scopes and should not be mixed.
Furthermore, handling some project's (compile) dependencies can be very involved and needs to be handled with care. I personally don't want some "random" plugin mess with my project's dependencies, just so that some optional feature works. This plugin's purpose is to make "spotbugs" run. Running spotbugs and tweaking your code to please spotbugs (or make spotbugs ignore certain issues) are two different things, which is why I think this is out of scope.
I suggest a) that you, @soloturn, add the dependency yourself, b) to close the issue ("won't do").

that would be then a problem of all other software as well where warnings can be silenced, like pmd, checkstyle, sonar, .... what makes spotbugs special?

There is nothing special. SpotBugs provides own annotations so if you want to annotate your code with it, you need to add the annotation artifact via compileOnly dependency.
And actually java.lang.SuppressWarnings also works even for SpotBugs. SpotBugs has a longer history than java.lang.SuppressWarnings so it has its own annotation, but it does not mean that SpotBugs users are forced to use SuppressFBWarnings.
Anyway your doubt is unrelated to this Gradle plugin so that I will close this issue. If you still have doubts to ask, open a discussion at spotbugs/spotbugs project.


Updated Basket Reducer

Description
Initially clicking "add to cart" creates a new instance of the item in the cart so added logic to increment the number of the item in cart as well as include the new item if "add to cart"  is clicked for a new item
This PR fixes #
Notes for Reviewers

@Omafovbe does this also solve the dependency conflicts ? https://github.com/legendarykamal/Lets-go-shop/issues/24

I tried but it didn't work but on my local machine it did
On Sun, 21 Aug 2022, 17:31 Kamal Nayan, @.***> wrote:

@Omafovbe https://github.com/Omafovbe does this also solve the
dependency conflicts ? #24
https://github.com/legendarykamal/Lets-go-shop/issues/24
—
Reply to this email directly, view it on GitHub
https://github.com/legendarykamal/Lets-go-shop/pull/30#issuecomment-1221578846,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AEXP2H53QJX62ONFE2CHU6DV2JKX3ANCNFSM57FGHKGA
.
You are receiving this because you were mentioned.Message ID:
@.***>


Allow multiple desc elements

We allow multiple title elements in info, using the type and role attributes to ascertain when to use which. But we always use the same desc element. When it's useful to provide alternate titles, it's sometimes useful to provide alternate descs. Allow multiple desc elements with the type and role attributes, using the same rules as title for when each is used.

http://projectmallard.org/mep/mep0008.html

Bumping to 1.2

I've revised the MEP to use the new proposal for common lookup rules for type-and-role data:
http://projectmallard.org/mep/mep0018


Add an input for passing additional arguments to cargo audit call

As mentioned in #132 it would be useful to be able to pass extra command-line arguments to the cargo audit invocation.
I tried to model the changes on the way that the args input is handled in actions-rs/cargo.  This is my first time using typescript however so please let me know if I'm doing anything suboptimal.
Happy to make any changes requested.
Fixes #132.

Hey @svartalf could you take a look at this when you get a chance? I'd love to be able to use this feature in the audit check for tokio.


[add-resource] New Plugin/Theme

I would like to add my new plugin and theme to the Grav Repository.
Here are the project details: https://github.com/fyr77/grav-theme-masonry and https://github.com/fyr77/grav-auto-author/
These are forks of an existing plugin and theme, slightly modified to display the author differently, therefore making it better suitable for a multi-author site.

The reason I didn’t create a pull request was because for a single author site, my versions are completely unnecessary.
But thinking of it, I should just add a flag in the config for „single-author“ vs „multi-author“ and just submit a pull request.
Sorry for not thinking that through.


Parental controls matter before displaying channel

Seems like there's an issue with accessing the TvInputManager so this may not be difficult ATM.

Okay I figured out how to access it through system services and implement in 0.2.3's SimpleSession.


Removing extraneous methods from and adding missing functionality to ConfigurationClient

Changes the surface area of ConfigurationClient

Removes the extra list methods we had
Adds in the add/set/update methods missing
No longer exposes "If-Match" and "If-Not-Match". Rather, uses the value of etag to determine which header is correct.

Fixes #3111 and #3124

Can one of the admins verify this patch?


How to show current file relative path in status bar?

Version confirmation

[ ] Confirm

Neovim version
NVIM v0.8.2 Build type: Release LuaJIT 2.1.0-beta3 Compiled by<EMAIL_ADDRESS>Features: +acl +iconv +tui See ":help feature-compile"     system vimrc file: "$VIM/sysinit.vim"   fall-back for $VIM: "/opt/homebrew/Cellar/neovim/0.8.2/share/nvim"  Run :checkhealth for more info
Operating system/version
macos 13.1
Terminal name/version
Item2
$TERM environment variable
No response
Branch info
main (Default/Latest)
Fetch Preferences
SSH (use_ssh = true)
How to reproduce the issue
Is there any way to show current file's relative path in status bar?

Expected behavior
display file relative path
Actual behavior
No response
Additional information
No response

Here's the section u config lualine in this config:
https://github.com/ayamir/nvimdots/blob/c761c1a7cfdbfbef6ec30a0fad15a1df083834f7/lua/modules/ui/config.lua#L436-L599
Here's some documentation for u to reference how to config lualine:
https://github.com/nvim-lualine/lualine.nvim#default-configuration
https://github.com/nvim-lualine/lualine.nvim/wiki

Thanks.

@leowucn Just a reminder. You don't seem to have the required font installed correctly. Please download your desired font here or via Homebrew Cask Fonts, and then select the font ending with Nerd Font Mono (e.g., JetBrainsMono Nerd Font Mono) in your iTerm2's custom profile.


Update install.md with GPG key addition via trusted.gpg.d and add instructions to install Gazebo CMake from source

🦟 Bug fix
Fixes #
Summary
Update install.md with GPG key addition via trusted.gpg.d and add instructions to install Gazebo CMake from source
Checklist

[x] Updated documentation (as needed)

Note to maintainers: Remember to use Squash-Merge and edit the commit message to match the pull request summary while retaining Signed-off-by messages.

@j-rivero mind taking a look. Does this line up with the latest recommended installation instructions elsewhere?


Prefix TFE vars with TFE_VAR_ to silence "unused variable" warning in Terraform Cloud

Warning from Terraform Cloud:
To silence these warnings, use TF_VAR_... environment variables to provide certain "global" settings to all configurations in your organization. To reduce the verbosity of these warnings, use the -compact-warnings option.

This was implemented in PR #24. They were all set to category=env.


Could really use help with some nested arrays/objects

I've been tearing my hair out for a couple hours now. Cannot figure out how to get a big csv with prop1, prop2, prop3, prop4 as my headers. I just can't quite get the syntax here. Any hints/help?
{
 "notes": [
  {
   "properties": [
    {
     "prop1": "awefweaweefw",
     "prop2": "awefewfaaewfefw",
     "prop3": "wewafeefwefw",
     "prop4": "waefaewfewf"
    },
    {
    "prop1": "awefweaweefw",
     "prop2": "awefewfaaewfefw",
     "prop3": "wewafeefwefw",
     "prop4": "waefaewfewf"
    },
    {
    "prop1": "awefweaweefw",
     "prop2": "awefewfaaewfefw",
     "prop3": "wewafeefwefw",
     "prop4": "waefaewfewf"
    },

The challenge is that it goes more like

{
 "notes": [
  {
   "properties3": [
    {
     "prop1": "awefweaweefw",
     "prop2": "awefewfaaewfefw",
     "prop3": "wewafeefwefw",
     "prop4": "waefaewfewf"
    },
    {
    "prop1": "awefweaweefw",
     "prop2": "awefewfaaewfefw",
     "prop3": "wewafeefwefw",
     "prop4": "waefaewfewf"
    },
    {
    "prop1": "awefweaweefw",
     "prop2": "awefewfaaewfefw",
     "prop3": "wewafeefwefw",
     "prop4": "waefaewfewf"
    }],
"properties3": [
    {
     "prop1": "awefweaweefw",
     "prop2": "awefewfaaewfefw",
     "prop3": "wewafeefwefw",
     "prop4": "waefaewfewf"
    },
    {
    "prop1": "awefweaweefw",
     "prop2": "awefewfaaewfefw",
     "prop3": "wewafeefwefw",
     "prop4": "waefaewfewf"
    },
    {
    "prop1": "awefweaweefw",
     "prop2": "awefewfaaewfefw",
     "prop3": "wewafeefwefw",
     "prop4": "waefaewfewf"
    }]

You need to combine the data into one array, so something like: https://runkit.com/knownasilya/5a7b14fbde8a180012049fbf

Hmm, this helpful. Is doing it on the command line going to be pretty hard then?

You could write a script that takes what I did above and uses node's fs.* methods to create a new json file, or you can use something like https://github.com/antonmedv/fx to transform the json in the CLI.


Failed to withdraw GAS, and the transfer also failed. The 3.0 master version is used

Failed to withdraw GAS, and the transfer also failed. The 3.0 master version is used

Now it work well, If there are still erros, please raise issues here https://github.com/neo-ngd/neo3-gui

Now it work well, If there are still erros, please raise issues here https://github.com/neo-ngd/neo3-gui


Format pretty terminal output

sickkk!!!

Thanks man 🙌
Can you also approve, merge and release a new version as well?

Replaced with https://github.com/paradime-io/paradime-python-sdk/pull/12


Tooltips are not working for

I refer to the example page:
https://github.com/extnet/examples.ext.net/blob/3d91a9ceb02164adcff2ff9ded69c6c1ed09a9e0/src/Pages/samples/layout/formlayout/overview/index.cshtml#L52-L54
Live:
https://examples.ext.net/#/layout/formlayout/overview
In the form the expected tooltips are not showing up.
Can you provide a fix or workaround, please?

Ext.NET 7's support for the datatip plugin was not implemented. The code including the plugin to the sample is commented out because it couldn't make it in time of the release.
Unfortunately at this time there's no fix for that other than actually porting the plugin implementation from Ext.NET 5.
We have created issue extnet/Ext.NET#1867 to track the implementation of this plug in, so you may want to subscribe to it to get an update when the support is implemented.


 in unclear to me

Hi,
for me it is not clear what <YOUR-IP-ADDRESS> means on the below page.
Some more details would be fine.
I guess Your-IP-Address means my IP, so that I can connect via psql. This would mean it is optional. If it is optional,
then maybe don't provide it by default. Because I first thought you need to insert the IP of the pod.

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

ID: 894542f3-4a24-3523-a8af-cd69bb738ea4
Version Independent ID: d3c30283-8d4a-0b91-f3d5-46859338e9b7
Content: Tutorial: Deploy Django on AKS cluster with PostgreSQL Flexible Server by using Azure CLI
Content Source: articles/postgresql/flexible-server/tutorial-django-aks-database.md
Service: postgresql
GitHub Login: @mksuni
Microsoft Alias: sumuth

@guettli Please let us know if you need any help.

@guettli We are going ahead and closing this feedback. Please get back to us if you need any further information.


Rename repository and demo to mashup-demo

The name should be hyphenated according to the other repositories.

Implemented on master (v1.1.0)


chore: fix some tests and make test harness more robust to individual failures

I noticed that the tests are failing both locally and in GitHub actions. They're "succeeding" because of the continue-on-error block in test.yml
The first issue (that this PR fixes) is that if calling Neogit in the git_harness setup fails, then the working directory wasn't getting reset. So I changed that to a pcall() and called the cleanup if it failed.
You can see this currently manifests as all tests requiring the git repo harness after a failing test will fail with a "can't find directory" error:
Fail	||	status buffer staging files - s can stage a subsequent hunk under the cursor of a tracked file
            ./tests/git_harness.lua:21: Vim(cd):E344: Can't find directory "/tmp/neogit_test_koufu" in cdpath

            stack traceback:
            	./tests/git_harness.lua:21: in function 'prepare_repository'
            	./tests/git_harness.lua:33: in function <./tests/git_harness.lua

This is because the cp tests/.repo command (that creates the temp directory) is failing, because the working directory wasn't being reset.
The second issue I found is that some requires() were failing -- neogit.popups.diff and neogit.popups.branch were not present in popups.lua. This seems to only be an issue in tests, but it seems harmless to add these.
Example of the current master build failing on this error: https://github.com/TimUntersberger/neogit/runs/2869021175#step:4:39
This doesn't entirely fix the tests, however. I spent some time trying to debug why status_buf_spec.lua was failing. It seems like maybe (?) the <tab> isn't getting fed properly to expand the staged file. I'm not too familiar with the neovim APIs so I didn't spend too much time on it.
If you have any thoughts on why this may be happening, I'm happy to continue investigating, and then we can turn off the continue-on-error flag, assuming you want test failures to show up as Red on GitHub actions.

Hey, thanks for cleaning my mess :)
I'll try to have a look this weekend, but the failing tests are most likely timing issues - The invoked actions are asynchronous, but the test has no good way of waiting for them. I've built some crutches to facilitate that, but as we continue to better structure the code, those crutches routinely break (also because nobody notices the tests starting to fail :sweat_smile:)

All the tests are now working in #381. Is this PR still relevant?

Sounds like not :)
Closing in favor of #381


Hotfix/queryfail

Some protections against very long running filters in the streaming filter module.

There is a SET STATEMENT max_statement_time=10 FOR prequel in run_active_queries, to limit each to 10 seconds. Most take less than 2 seconds. Also sends email if the timeout is breached.
When saving a query it is run to pre-inject the kafka with 10 results, and this also has a timeout of 10 seconds.
The word SLEEP is specifically excluded from the SELECT statements

Thank you Gareth
Of course I have tested in the full system with lasair-dev. I don't think a unit test would work, since we are catching exceptions from the database. Do we have any automated tests that talk to a database? If you can figure out how to do that, I would be happy to make a test.
Yes, I will put some notes in the dox about the 10-second timeout for streaming queries.
Roy

Note that this fix sends an email to the user if their query fails for any reason, not just timeout.


Continue On... accessible through the Remote indicator

I observe myself clicking the remote indicator to get to "Continue On" and it's not there. Rather than a generic "Continue On" entry it might make sense to have for each remote group a separate one. Like "Continue on in a Codespace" in the second group.

It looks like you are possibly in an empty workspace here (since you're in https://insiders.vscode.dev and there is no Continue On from Remote Repositories). Continue On is currently disabled in this scenario because I don't think we have any Continue On options which allow you to start in an empty workspace--currently they all assume you have a git or remote GitHub repository. Is this an empty workspace or a local folder here? What would you expect to be able to do in this scenario?

I also think the remote menu is a good place for "Continue on". However I also think that "Edit Sessions" and "Continue on" should be closer together and not disconnected. I find that "Edit Sessions" is the technical requirement to get "Continue on" working, but it is not so obvious.
I wonder if we should rather hide the "Edit Sessions" entries and make this all acessible via the concept of "Continue On".

I explored three options for surfacing Continue On through the remote indicator in desktop, since it's currently not there at all (and these proposals would also apply to the remote indicator in web). I think we would still want to retain the Continue Working On command which presents all options in a single list when entering the feature via the terminal and debug welcome views, so this mostly relates to how Continue On options are presented in the remote indicator.
Single top-level Continue On entry
This is the same as what we have in VS Code for the Web today, i.e. a single Continue Working On entry in the remote indicator that would now be present in web, desktop, and remote windows. It's compact but requires two clicks to get to the list.
https://user-images.githubusercontent.com/30305945/201412683-a2910cb4-89f8-4e3b-ab31-69e2d0b280a0.mp4
Separate Continue On category
This flattens the list of Continue On items into the top-level remote indicator, removing the need for two clicks, but it means that extensions which have their own remote group and a continue on contribution will appear in two different groups in this list, which can be confusing.
https://user-images.githubusercontent.com/30305945/201412682-d2bf8786-3740-426c-8b6d-5493f6e1467e.mp4
Continue On integrated into each remote group
This integrates the continue on options into each extension's remote group. (NB there's an adoption cost here since the contributions would have different display names depending if they're shown in the Continue Working On... picker or inline with the extension's remote group)
https://user-images.githubusercontent.com/30305945/201413183-e2cc1421-a802-432b-98ce-b3c0b44bb120.mp4
cc for feedback and suggestions @aeschli @daviddossett

At a glance, I prefer option 1 despite the extra step since it solidifies the concept of "Continue Working On" as a top-level task. I don't feel like I have to think when I see that.
2 somewhat buries that value props and the options feel a bit like we cherrypicked items from the list below. I'm also not sure most users would read the section label since it's aligned to the right. So we might miss out on users making that connection to "Continue Working On" when they see it elsewhere.
3 is nice but, to me, too closely couples to specific extensions with the concept. They will also potentially be lost in this long list if a user is just scanning through.

To me, the remote menu is more for connecting to another machine. Many actions open a new window and that window is a remote window.
Continue On is more like an Open, or even more an Open Recent. I would therefore add something in the File menu, right after the Other Opens, or inside Open Recent submenu.

It's true that Continue On from web RemoteHub to desktop RemoteHub / desktop git clone or from desktop RemoteHub to a desktop git clone don't relate to connecting to another machine per se, so I can see how it may make sense to instead move it to the File menu:

It doesn't feel like a fit for Open Recent to me though, since Continue On transfers your current working context to a new development environment, and 'recent' implies a window I have already visited before.

Can we UX discuss this Wednesday before making a decision?

Yes definitely, I intentionally made the impl a draft PR.

@brettcannon had related feedback on wanting a Codespaces: Continue On in a Codespace command palette action in slack:

I was viewing the action as, "I want to go to this place," not, "I want to take my edits somewhere." So the destination was what I had in mind, not the fact that I wanted to generically move my (future) edits to another place as I had not done anything (yet).

I have added support for one-step actions like Codespaces: Continue On in a Codespace in the command palette (requires a new release of the Codespaces extension and latest VS Code Insiders.
@aeschli since some but not all Continue On options deal with opening a remote, what would you think of surfacing the one-step actions which relate to remotes in the remote indicator in their specific categories? i.e. Dev containers and Codespaces would have actions in the Dev containers and Codespaces categories which say 'Continue on in a (Codespace | Dev Container)'.

Codespaces: Continue On in a Codespace fits very well in the remote menu.
In fact now I relaize that it's about the reopening the current folder in a remote location. The same what already have for Dev containers (Reopen in Container) and WSL (Reopen Folder in WSL).
So I would even suggest to consider using Reopen: instead of Continue On: Reopen in Codespace


FIX: Plot with new ReportPlotter

Codecov Report
All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 88.65%. Comparing base (5602cb1) to head (1e6489d).

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #295   +/-   ##
=======================================
  Coverage   88.65%   88.65%
=======================================
  Files          12       12
  Lines        3262     3262
=======================================
  Hits         2892     2892
  Misses        370      370

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


Launch workers alongside leader with toil launch-cluster (resolves #1383)

This allows worker nodes to be launched along side leader by adding -w option to toil launch-cluster. To spin up a cluster with 4 workers (not including the leader) run:
toil launch-cluster CLUSTER_NAME --nodeType=t2.micro --keyPairName<EMAIL_ADDRESS>-w 4

Jenkins test this please

Jenkins skip this please

Jenkins test this please.

Jenkins, test this please


Dialog with asynchronously loaded content is removed during page load

Inside my .html.twig file I create a new dialog box that shows a toolbar.
The toolbar has items in to be used on this page, so should be loaded as soon as possible (rather than opening on a button press, which might be a more typical use case).
Here is the code:
require(['oro/dialog-widget', 'routing'], function(Dialog, routing){
   var dialog = new Dialog({
       url: routing.generate('defacto_toolbar'),
       title: ('Content Toolbar - stand alone'),
       dialogOptions: {
           width: 260,
           height: 500
       },
       stateEnabled: false
   });
    dialog.render();
});

The content for my dialog is loaded asynchronously from the server (because the contents are dynamic). When my page loads I get the following error:

Uncaught TypeError: Cannot assign to read only property 'loading' of [object Object]

I've traced this down to the Widget-Manager class removing widgets without a Dom element. I believe this was added to help fix memory leaks:
/**
 * Reset manager to initial state.
 */
resetWidgets: function() {
    _.each(this.widgets, function (widget) {
        // if widget is detached from DOM, remove this widget
        if (!$.contains(document.documentElement, widget.el)) {
            widget.remove();
        }
    });
},

Unfortunately it seems my widget is removed from the page, because it does not have a widget.el at this point (the contents are still being loaded via Ajax). When the widget is removed it is disposed, which causes all of it's properties to be set to read-only.
Is there a suggested work around to this? Am I forced to load my widget at a later point, or is my use-case valid and you will implement a fix? I am guessing that other widget types could experience the same issue, if they load their content asynchronously at the start of the request.

For my use-case, I can simply not allow the removal of the dialog widget if the content is still loading.
I did this with a local override, as it only affects my dialogs.
This feels like a simple, but not perfect fix. Please let me know your thoughts.
define(['oro/dialog-widget'], function (DialogWidgetAbstract) {
    'use strict';

    /**
     * Do not remove the widget if it is still loading it's contents
     *
     * @see https://github.com/orocrm/platform/issues/240
     **/
    var DialogWidget = DialogWidgetAbstract.extend({
        remove: function(){
            if (! this.loading){
                DialogWidget.__super__.remove.call(this);
            }
        }
    });

    return DialogWidget;
});

Core fix for this could be:
        /**
         * Removes dialog widget
         */
        remove: function() {
            if (this.loading) {
                return;
            }
            if (this.widget) {
                // There's widget, close it before remove.
                // Close handler will invoke dispose method,
                // where remove method will be called again
                this.widget.dialog('close');
            } else {
                DialogWidget.__super__.remove.call(this);
            }
        },

Yes, this is expected behavior. If a widget's element is detached from DOM, widget manager destroys this widget on pageAfterChange event. If to add exception for loading widget (as you proposed) what would be a natural way to dispose this widget?
For your case, I would recommend you to add the dialog to DOM manually.
require(['jquery', 'oro/dialog-widget', 'routing'], function($, Dialog, routing){
    var dialog = new Dialog({
        url: routing.generate('oro_entityconfig_widget_info', {id: 1}),
        title: ('Content Toolbar - stand alone'),
        dialogOptions: {
            width: 260,
            height: 500
        },
        stateEnabled: false
    });
    $(document.body).append(dialog.$el);
    dialog.render();
});

Hi @Grygir - Thanks for the feedback and work around, this does work for me.
To answer your question - I could not think of a time that I would like to remove a dialog widget whilst it was still loading - can you?
It is slightly hidden that you need to add Dialog.$el to the DOM manually, but only if you load dialog with URL before pageAfterChange event?
What are your thoughts on the manager checking this state? That way dispose would still work but I would not need to add Dialog to the DOM manually for Ajax Dialogs?
Any scenarios I have missed here?
/**
 * Reset manager to initial state.
 */
resetWidgets: function() {
    _.each(this.widgets, function (widget) {
        // if widget is detached from DOM and not loading, remove this widget
        if (!$.contains(document.documentElement, widget.el) && !widget.loading) {
            widget.remove();
        }
    });
},

Hi @MikeParkin
Loading is not sufficient sign that widget is still actual. There might some error have happened and flag loading is not removed, we have to be able to clean up such widget on page change action. I see the problem that pointing out. Think we will add a solution in to coming release.
Thanks for your question.

Fixed in 1.7


Use CentOS Stream9 as a base image

celery & flexmock are not packaged for CentOS Stream 9 yet, but they can be easily installed from PyPI (we've actually already been using celery from PyPI in worker)
Related to packit/deployment#410

I hope there's no issue here because of some newer Python features, cause running the base image locally yields:
Python 3.9.16 (main, Dec  8 2022, 00:00:00)

There's also python3.11 already installed (pulled in by ansible), so switching to it hopefully shouldn't be that hard if really needed.
$ podman run -ti --rm quay.io/packit/base:c9s rpm -q python3.11
python3.11-3.11.2-2.el9.x86_64

recheck


IE 8 : 'originalEvent' is null or not an object

Hi, I have an error on IE 8, line 691 : 'originalEvent' is null or not an object
getRelativePosition = helpers.getRelativePosition = function(evt){
			var mouseX, mouseY;
			var e = evt.originalEvent || evt,

Is the library still compatible with IE 8 ?
Thanks by advance !

That should just be a warning. The way that line is written, it is supposed to fall back to the event itself if the event.originalEvent object is not defined (which it won't be, since it's not defined in IE8's DOM). Are you getting an exception in the JavaScript console?

It is a warning, but each time this warning is coming, IE is becoming very very slow... and my donught is becoming unusable.

IE8 itself has a rather slow JavaScript engine and uses VML, so you're not going to get optimal performance when using that browser. Have you followed the guidelines in http://www.chartjs.org/docs/#notes-browser-support, and can you create a test case via JSBin or JSFiddle to demonstrate what you're seeing?

need a test case to get anywhere with this issue, closing unresponsive


Mending only repairing items actively used

Timings link
https://timings.pl3x.net/?id=d67f5b1dd5324afba9a339bfd6d94e99
Expected behavior
When having an item with the mending enchantment in my inventory it should be repaired on receiving EXP.
Observed/Actual behavior
Instead I get the EXP added to my levels and nothing gets repaired unless I'm holding the item in my hand or left hand or have it equipped as armor. The problem is occuring both either with better-mending turned off or on
Steps/models to reproduce

Enchant one or multiple items with mending.
Earn some EXP either with farming or EXP bottles

Purpur version
git-Purpur-1327
Agreements

[X] I am running the latest version of Purpur available from https://purpur.pl3x.net/downloads.
[X] I have searched for and ensured there isn't already an open issue regarding this.
[X] My version of Minecraft is supported by Purpur.

Other
No response

Any items with Mending in the player's main hand, offhand, or armor slots that are not at maximum durability are repaired by experience orbs collected from any source. The rate of repair is two durability per point of experience contained in the orb.

Source : https://minecraft.fandom.com/wiki/Mending


vmware workstation / player font issue (square characters instead of text)

Describe the bug
Fonts are missing, square characters are displayed

Steps To Reproduce
Steps to reproduce the behavior:

Install vmware-workstation
Start vmware workstation or player

Additional context
I started a discussion on https://discourse.nixos.org/t/font-issue-with-vmware-workstation/19669
Notify maintainers

/cc @deinferno
Metadata
[user@system:~]$ nix-shell -p nix-info --run "nix-info -m"
 - system: `"x86_64-linux"`
 - host os: `Linux 5.15.43, NixOS, 22.05 (Quokka), 22.05.714.e5556c75ac0`
 - multi-user?: `yes`
 - sandbox: `yes`
 - version: `nix-env (Nix) 2.8.1`
 - channels(root): `"nixos-22.05"`
 - nixpkgs: `/nix/var/nix/profiles/per-user/root/channels/nixos`

I can reproduce this, probably related to some missing font in freshly installed systems

I launched vmware with FC_DEBUG=8, but I'm not able to interpret these errors messages, hope this can help
FC_DEBUG=8
adding fonts from /usr/share/fonts
cache scan dir /usr/share/fonts
adding fonts from /home/pascal/.local/share/fonts
cache scan dir /home/pascal/.local/share/fonts
adding fonts from /home/pascal/.fonts
cache scan dir /home/pascal/.fonts
FontSet 0 of 0
/nix/store/v5yhx9hp13dy9hzil83xwgbrzp7c8gni-dconf-0.40.0-lib/lib/gio/modules/libdconfsettings.so: undefined symbol: g_log_structured_standard
Failed to load module: /nix/store/v5yhx9hp13dy9hzil83xwgbrzp7c8gni-dconf-0.40.0-lib/lib/gio/modules/libdconfsettings.so
'g_io_module_load': /nix/store/z4yprly9sv4gp672nxizig0x5dn5bd8c-glib-networking-2.72.0/lib/gio/modules/libgioenvironmentproxy.so: undefined symbol: g_io_module_load
Failed to load module: /nix/store/z4yprly9sv4gp672nxizig0x5dn5bd8c-glib-networking-2.72.0/lib/gio/modules/libgioenvironmentproxy.so
/nix/store/z4yprly9sv4gp672nxizig0x5dn5bd8c-glib-networking-2.72.0/lib/gio/modules/libgiognomeproxy.so: undefined symbol: g_task_set_name
Failed to load module: /nix/store/z4yprly9sv4gp672nxizig0x5dn5bd8c-glib-networking-2.72.0/lib/gio/modules/libgiognomeproxy.so
/nix/store/z4yprly9sv4gp672nxizig0x5dn5bd8c-glib-networking-2.72.0/lib/gio/modules/libgiognutls.so: undefined symbol: g_byte_array_steal
Failed to load module: /nix/store/z4yprly9sv4gp672nxizig0x5dn5bd8c-glib-networking-2.72.0/lib/gio/modules/libgiognutls.so
/nix/store/jang67xg1fidp5zpd146f3v14978hhwr-vmware-workstation-16.2.3/lib/vmware/lib/libstdc++.so.6/libstdc++.so.6: version `GLIBCXX_3.4.29' not found (required by /nix/store/p9p9wbgfjc6v23q9lrp5p4201h4pzcgr-libproxy-0.4.17/lib/libproxy.so.1)
Failed to load module: /nix/store/z4yprly9sv4gp672nxizig0x5dn5bd8c-glib-networking-2.72.0/lib/gio/modules/libgiolibproxy.so
/nix/store/dbqgkps4h076lkavi46ms3zzwl8q2wnv-gvfs-1.50.2/lib/gvfs/libgvfscommon.so: undefined symbol: g_task_set_name
Failed to load module: /nix/store/dbqgkps4h076lkavi46ms3zzwl8q2wnv-gvfs-1.50.2/lib/gio/modules/libgioremote-volume-monitor.so
/nix/store/dbqgkps4h076lkavi46ms3zzwl8q2wnv-gvfs-1.50.2/lib/gvfs/libgvfscommon.so: undefined symbol: g_task_set_name
Failed to load module: /nix/store/dbqgkps4h076lkavi46ms3zzwl8q2wnv-gvfs-1.50.2/lib/gio/modules/libgvfsdbus.so

This issue has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/font-issue-with-vmware-workstation/19669/6

I found reason for missing fonts, it just loads fonts.conf from lib/vmware/libconf/etc/fonts/fonts.conf  instead of system path and vmware's config points to wrong FHS paths, removing file fixes it, i will make a PR with fix, thanks.

I confirm that deleting lib/vmware/libconf/etc/fonts/fonts.conf fixes the issue.
Thanks a lot !


Cow: Watch megatransfers.com

Cow requests the watch of the watch_keyword megatransfers\.com. See the MS search here and the Stack Exchange search in text, in URLs, and in code.

megatransfers\.com has been seen in 1 true positive, 0 false positives, and 0 NAAs.

Approved by Dragonrage in Charcoal HQ


inter: 3.11 -> 3.12

Motivation for this change
https://github.com/rsms/inter/releases/tag/v3.12
Things done

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

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

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

Thanks for catching this! Looks like they have a few variants to choose from-- Hinted ("for Windows") variants, ... err, here's the listing:
https://gist.github.com/dtzWill/e956d2f197404acb5059e43c1ead3533
Not sure why Mac's unzip is behaving different (!) but glad it did because we should make an intentional choice here :).
Hinted version seems the way to go? But upstream's instructions for Linux are to use unhinted-- I'm thinking user's font settings can choose to ignore hinting information and font size doesn't seem a significant concern (for Desktop use).
Thoughts/input welcome, may want to be more particular than just this choice...

Pantheon wants to use this font and they noticed kerning differences between variants https://github.com/elementary/default-settings/issues/152#issuecomment-577280887. I don't really know much about fonts in macos though.

Thanks for catching this! Looks like they have a few variants to choose from-- Hinted ("for Windows") variants, ... err, here's the listing:
https://gist.github.com/dtzWill/e956d2f197404acb5059e43c1ead3533
Not sure why Mac's unzip is behaving different (!) but glad it did because we should make an intentional choice here :).
Hinted version seems the way to go? But upstream's instructions for Linux are to use unhinted-- I'm thinking user's font settings can choose to ignore hinting information and font size doesn't seem a significant concern (for Desktop use).
Thoughts/input welcome, may want to be more particular than just this choice...

I'd think we'd want to distribute all variants? They could even be in different outputs if desired.

The hinted TTFs are just run through the FreeType2 ttfautohint, rather than being manually hinted.  For Linux I would tend to prefer letting FreeType2 do its autohinting instead, both so that you get the choice yourself, and as you might be running a newer version with improved autohinting.
We could certainly distribute the other variants as well however, since people might indeed have uses for the other variants.

Confirmed darwin failure is a regression https://hydra.nixos.org/job/nixpkgs/nixpkgs-20.03-darwin/inter.x86_64-darwin

Actually I think the conflict is due to case-sensitivity (or insensitivity on Darwin)-- looks like the conflict is between Inter-Italic.otf and the variable font file inter-italic.otf.
The hinted files are ttf's so I was too hasty before in suggesting the cause of the problem, sorry :).
Not seeing a clearly best answer, anyone?
Personally I'd prefer to avoid splitting into multiple outputs because that sounds like great way to ensure they aren't used ;).

There's not really any other option though.  If someone wants the variable version they need to explicitly choose which one, as otherwise the different fonts would override eachother.
F.ex. Inter (Variable)/Inter.otf,  Inter (Variable, single axis)/Inter-italic.otf, and Inter (Variable, single axis)/Inter-roman.otf just have Inter as the font fullname, meaning that you would not be able to choose between them if all of them would be in the same output.  At least Inter V (Variable)/Inter-V.otf has the distinct fullname Inter V though.
Theoretically it might be nice to just have Inter (Variable)/Inter.otf as it should cover everything in Inter/, but there are still widely used software that does not support them well.  E.g. chromium does not: https://rsms.me/inter/var-test.html.
It also seems like the multi-axis variable version doesn't work in all applications due to the metadata being wrong as well?
https://github.com/rsms/inter/issues/142
So Inter V could probably be included in the regular inter output without causing any problems, which would let people use the variable one if they wish.  But it would also make the inter output bigger for everyone who doesn't want to use it.

I'm still of the opinion that the unhinted version should be the default, with the others being available in separate outputs, as that is the arrangement that will work for the mast usecases.


TypeMismatch: "Null is not a Int"

I just updated Argo to 1.0.0 and I'm seeing a regression:
I have a model with some optional values and when I try to parse a JSON containing a null value (an NSNull), I'm getting a TypeMismatch: "Null is not a Int" in runtime.
Here's the model:
public struct Identifiers {
    public let trakt: Int
    public let tvdb: Int?
    public let imdb: String?
    public let tmdb: Int?
    public let tvrage: Int?
    public let slug: String?
}

extension Identifiers: Decodable {
    static func create(trakt: Int)(tvdb: Int?)(imdb: String?)(tmdb: Int?)(tvrage: Int?)(slug: String?) -> Identifiers {
        return Identifiers(trakt: trakt, tvdb: tvdb, imdb: imdb, tmdb: tmdb, tvrage: tvrage, slug: slug)
    }

    public static func decode(j: JSON) -> Decoded<Identifiers> {
        return Identifiers.create
            <^> j <| "trakt"
            <*> j <|? "tvdb"
            <*> j <|? "imdb"
            <*> j <|? "tmdb"
            <*> j <|? "tvrage"
            <*> j <|? "slug"
    }
}

And the parse itself:
let json: NSDictionary = [
            "trakt": 36440,
            "tvdb": 3254641,
            "imdb": "tt1480055",
            "tmdb": 63056,
            "tvrage": NSNull()
        ]

let value = JSON.parse(json)
 let id = Identifiers.decode(value)

Any thoughts? I could remove the keys with NSNull values from the dictionary, but I think Argo should handle it (as it was before 1.0.0).

@marcelofabri Thanks for submitting the issue. This is a regression that we weren't aware of. When we switched to the Decoded type we treated .Null cases as .TypeMismatch which will throw the errors. We need to change that to return .MissingKey and it will work. I'll work on this soon and get back to you.

We're working on a fix here: #117

@marcelofabri This issues is fixed. Use version 1.0.1 and let us know if you have any more issues.

Thanks! It works now. Could you please push the Podspec to trunk too?

@marcelofabri Should be all set now!


429 too many requests response from mapzen

I get 429 too many requests responses from the Mapzen service. Is there some kind of throttling that Mapzen expects?
cc @rmglennon
example request:
def geocode_mapzen(address, maps_api_key):
	return(geocoder.mapzen(address, key=maps_api_key))

mapzen seems to see the requests in my quota but they are well below the 25k.

Not too sure how to fix this one, sounds like a Rate limiting issue with Mapzen, try doing an HTTP request to their API directly and see if it's the same behavior.

With Mapzen APIs, 429 errors typically occur when you exceed:

monthly free rate limits on a particular service and do not have a payment method on file
your monthly spending limit
per-second rate limits.

By default, geocoding and autocomplete are currently set at 6 requests per second.

Thanks @rmglennon for the quick Mapzen reply, much appreciated! ❤

closing the issue since we have the answer :alarm_clock:


iptables: fsync and close temp file before rename

Closing.
There is another problem of renaming between different partitions too.


Wrong classname (with namespaces) in SparkPost

There are few wrong classes and namespaces like Ivory\HttpAdapter instead of \Ivory\HttpAdapter\HttpAdapterInterface in SparkPost::__construct.
It's a bit distracting, because IDE shows warnings e.g. when I'm creating a SparkPost instance with provided HttpAdapter
May I do a PR to fix these issues?

Please feel free to submit a PR. That would be awesome. :)

Check PR #43 please.


modify jsdoc comments

Coverage increased (+18.6%) to 95.349% when pulling 75c8ce843f7c916182d0ca9a4f7dcec0301fa968 on staging into feb7be030edf855cfa459367608d191784c998fe on master.

Coverage increased (+18.6%) to 95.349% when pulling cdd3a3f9871ad21e77bb971c5414c3a27e79f053 on staging into feb7be030edf855cfa459367608d191784c998fe on master.

Coverage increased (+18.6%) to 95.349% when pulling cdd3a3f9871ad21e77bb971c5414c3a27e79f053 on staging into feb7be030edf855cfa459367608d191784c998fe on master.


[ECO-1965] fix build process

This PR fixes the build process that is currently failing on main.
When fallback is set to true, the page has to handle the case where props are undefined. This PR adds that edge case handling.

Noting that the PR works, but markets visited before their registration (let's say someone tries to go to market X before market X is registered), will cache the Not Found page for a certain amount of time, so if the market is registered just afterward, a delay will be present until the market can actually be viewed.


Added Bird-View Computer Vision Script

Fix to  Issue #50
===============================
This program will require four points to form a shape and will automatically convert that shape into an equivalent rectangle showing bird view.
===============================
Checklist

[x] I read the contributing guidelines
[x] I structured my project as stated in the contributing guidelines
[x] My project uses dependencies
[x] I starred the repo
[x] I had fun contributing!

@dellucifer
Thanks mate.


Extended asset information from stellar.toml and SEP-enabled services

Gather and index extended information from stellar.toml and other services defined in SEPs (for example, SEP-6 /info endpoint). Change storage structure to support API queries by this info.

ref #136

Implemented in v0.11.6


Failed to load correctly

Having this issue on trying to load the mod, anyone know a solution for this?

could you open the crash report there and post the contents of that, please? unfortunately, i can't do much with just this error screen :<


Support for multiple simultaneous metadata keys

Currently the Adaptor internally assumes a single metadata key (i.e. "traffic-profile"), the internal structures and API (e.g. PUT /mappings) should be updated to support multiple different keys existing at the same time.

This seems like a significant/useful enhancement target


WebStorm / intelliJ files to ignore

Greetings,
According to https://github.com/github/gitignore/blob/master/Global/JetBrains.gitignore
you should ignore:
User-specific stuff:
.idea/workspace.xml
.idea/tasks.xml
.idea/dictionaries

@blakemcbride What do you mean ? It is commented by default.


fix(Axis): Correct time axis bandwidth calculation

Brief Information
This pull request is in the type of:

[x] bug fixing
[ ] new feature
[ ] others

What does this PR do?

Added a method to calculate the bandwidth of the time axis, and fixed the problem of not displaying the width when the axisPointer type is shadow.
添加时间坐标轴关于bandwidth的计算方法，修复axisPointer type为shadow时，不显示宽度的问题
Fixed issues

Details
Before: What was the problem?

The intervals between timestamp data may be uneven, so the difference in data length cannot accurately reflect the number of data points, resulting in the inability to calculate the correct bandwidth value for the time type axis. When the axisPointer type is shadow, only a line close to 0 is displayed.
时间戳数据之间的间隔可能不均匀，因此数据长度的差值无法准确反映数据点数量，导致时间类型的轴无法计算得到正确的bandwidth值，当axispointer的type为shadow的时候，只显示一条接近于0的线

After: How does it behave after the fixing?

Added a method to calculate data length. When the axis type is time, the _approxInterval attribute is used to represent the average interval of the time scale. By dividing by this interval and rounding up, the number of data points can be estimated more accurately.
添加数据长度的计算方法，当轴的类型为时间时，利用_approxInterval 属性代表时间刻度的平均间隔，通过除以这个间隔并向上取整，可以更准确地估算出数据点的数量，从而得到有宽度的矩形高亮区

Document Info
One of the following should be checked.

[x] This PR doesn't relate to document changes
[ ] The document should be updated later
[ ] The document changes have been made in apache/echarts-doc#xxx

Misc
ZRender Changes

[ ] This PR depends on ZRender changes (ecomfe/zrender#xxx).

Related test cases or examples to use the new APIs
N.A.
Others
Merging options

[ ] Please squash the commits into a single one when merging.

Other information

After the modification, the width of the shadow does not specifically indicate any meaning; and if it is for an uneven axis, this shadow will also cause problems.
在修改后，shadow的宽度并不能具体表明任何含义；而且如果对于不均匀的轴来说，这个shadow也会出问题
Logically, the shadow should express the range corresponding to the highlighted data, but for numerical types such as value and time, the axis is continuous and there is no concept of range.
从逻辑上讲，shadow 应该表达的是高亮数据对应的范围，但是 value、time 这种数值型的，轴是连续的，没有范围的概念
If it is for visual effects, you can use type as line and adjust the width through lineStyle to interact
如果是为了视觉效果，可以采用type为line，通过lineStyle调整线宽来进行交互


Add split support in from folders

🚀 Feature

From ImageClassificationData.from_filepaths allows me to define splits for validation but from ImageClassificationData.from_folders does not this is inconsistent and should be a standard feature.

@aribornstein this come after #141 . See: https://github.com/PyTorchLightning/lightning-flash/blob/44d70e16ae74f890376a2a52175ef9c43e2b0291/flash/data/data_module.py#L379


remove unused test function

Codecov Report

:exclamation: No coverage uploaded for pull request base (master@8951f63). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             master     #469   +/-   ##
=========================================
  Coverage          ?   42.08%
=========================================
  Files             ?      119
  Lines             ?    10248
  Branches          ?        0
=========================================
  Hits              ?     4313
  Misses            ?     5447
  Partials          ?      488

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 8951f63...a65b442. Read the comment docs.


Update changelog for 1.2.6

Just needs a new release, so you can update the date ;)

Thanks for the work! Might be putting more into this release than what you've outlined, so I'm going to close it and create the CHANGELOG.md once we've settled on the contents of 1.2.6.

No need to tag directly , you can still merge and update with new releases, or change the version to -dev or something..


Css variables arent removed when switching stories

When switching between stories, css variables are not removed, so the changes applied in one story impact the others.
This may be similar to #14 but thats not clear to me.
Ive also noticed that, unless the addon is the first in the list of addons in main.js, the css property values are not applied - if needed, I can create a separate issue for this.
Love the addon!! This is huge!
Version: 2.3.1
Node: 14.18.0
Storybook version: 6.4.20

That worked! The variables are now scoped to the story so that if they are used elsewhere, they won't be overridden elsewhere. Thanks!


Fix session can't save to store

In SessionMiddleware when sid is None, execute new() will return a session instance, but session.should_save is False. so session will never be saved and can not set the cookie. SessionMiddleware is not works

I don't see the point in saving an empty session. Usually you immediately write to the session after creating it, and then it will be marked as modified.

Also please file issues first before submitting PRs, this breaks a testcase which actually asserts what you call a bug.

I'm going to close this, please file an issue if you really want to propose this behavior change.

@untitaker  there is a case:
from flask import Flask
from werkzeug.contrib.sessions import (
    SessionMiddleware, FilesystemSessionStore)

app = Flask(__name__)
app.wsgi_app = SessionMiddleware(app.wsgi_app, FilesystemSessionStore())

@app.route('/')
def hello():
    return 'Hello'

if __name__ == '__main__':
    app.run(host='<IP_ADDRESS>', port=9000)

but thie never save session file to /tmp

@dongweiming by design sessions are only stored if they get modified, empty session do never get stored
please modify the session if you want to see it used


Checkbox/Radio pointer cursor hover issue

Looks like the 'cursor: pointer' is only firing for the very top and right sections of the checkboxes and radio buttons.  This can be observed in the UI-kit demo.

Doesn't seem to have cursor: pointer on the input itself?

hmm...I don't see the issue or possibly misunderstanding it :neckbeard: can you drop a screenshot?

both should have pointer, but first one (which is more inside the box) is not

hmm...same behavior with http://getbootstrap.com/css/#checkboxes-and-radios.  Seems expected behavior from browsers + Bootstrap.
Thoughts?

Ok, probably not worth dealing with on it on our side.  I'll just make QA aware this is a Bootstrap issue.  Close this issue?

yes please :smile_cat:


Could not pass event PlayerMoveEvent to Sentinel

The output of command /version on my server is:
[Fri, 29. Mar 2019 04:37:15 UTC INFO] This server is running CraftBukkit version git-Spigot-518206a-a5b9c7b (MC: 1.13.2) (Implementing API version 1.13.2-R0.1-SNAPSHOT)
[Fri, 29. Mar 2019 04:37:15 UTC INFO] Checking version, please wait...
[Fri, 29. Mar 2019 04:37:16 UTC INFO] You are 89 version(s) behind
The output of command /version citizens on my server is:
[Fri, 29. Mar 2019 04:37:49 UTC INFO] Citizens version 2.0.24-SNAPSHOT (build 1604)
[Fri, 29. Mar 2019 04:37:49 UTC INFO] Website: http://www.citizensnpcs.co/
[Fri, 29. Mar 2019 04:37:49 UTC INFO] Authors: aPunch and fullwall
The output of command /version sentinel on my server is:
[Fri, 29. Mar 2019 04:38:19 UTC INFO] Sentinel version 1.8.5 (build 238)
[Fri, 29. Mar 2019 04:38:19 UTC INFO] Author: mcmonkey
A pastebin.com link to my Sentinel/config.yml file is: https://pastebin.com/NyYT06Hf
Every once in a while, the message [ERROR] Could not pass event PlayerMoveEvent to Sentinel v1.8.5 (build 238) is spammed in the server console many times per second, and slows down the server.
The message does not stop unless the server is restarted or if there are no players on the server. I encountered the same problem without the "workaround" options enabled in the config, and enabled them hoping it would solve the problem.
It seems to be an issue with respawning the Sentinel, as it seems to start after one of them dies, and if the respawn time is set using Citizens, (not Sentinels) then the errors only occur until the NPC is respawned.

Please update Sentinel, Citizens, and Spigot. All 3 are outdated.


Make TaskExecutor cx public and use TaskExecutor for concurrent HNSW graph build

Description
This PR proposes to :

Make the TaskExecutor public which is currently pkg-private - As we are making more use to concurrency and not restrictive at searching time eg:  now at indexing time we concurrently create the hnsw graph (#12660). We could use the TaskExecutor implementation to do this for us.
Use TaskExecutor#invokeAll in HnswConcurrentMergeBuilder#build to run the workers concurrently.

@javanna I have added the CHANGES entry and addressed the comment. Seems the precommit fails on to :lucene:documentation:markdownToHtml task which looks unrelated? Not sure.

@javanna I have addressed the latest comments now. Thanks!

I left one last small comment, sorry I could have made my earlier comment clearer and avoid more back and forth. Thanks for your patience.

No problem. I have addressed your comment to hold the TE in the classes instead of ES in the latest revision. Thanks for reviewing @javanna !

Could you also address the merge conflict in the changes file please?

Fixed

Thanks @shubhamvishu !


FamaMacBeth Std. Errors / T-Stats

I have encountered an issue pertaining to the computation of standard errors and measures dependent on them such as t-stats and p-values in the FamaMacBeth function. On some occasions the function will produce a parameter estimate, but no error statistics. More specifically, I receive a runtime warning of the following sort:
..../linearmodels/panel/results.py:70: RuntimeWarning: invalid value encountered in sqrt
Any suggestions are very welcome :)

Can you produce an example using simulated data that makes this happen?  A small self-contained example makes it much easier to understand what is specifically happening.
If not, could you describe the size of the data you are using, and provide the function call the produces issues?  Are you rolling through the data and it occasionally occurs when rolling?
Broadly something in the covariance matrix estimation is not positive definite.  This is producing nan values which then propagate.

Unfortunately, I have not been able to produce the error with simulated data. Thus, to create a minimal example, I have disguised my data and made it accessible via this dropbox link:
https://www.dropbox.com/s/d7apc41c0pc86x3/FM_MinExampleData.pkl?dl=0
The data consists of roughly 30,000 observations, 700 entities and 50 times. The underlying panel is unbalanced. The function call is static, i.e. non-rolling.
The function call is as follows:
import pandas as pd
import statsmodels.api as sm
from linearmodels import FamaMacBeth

#Load Sample
temp = pd.read_pickle('FM_MinExampleData.pkl')
temp.set_index(['Entity','Time'],inplace=True)

#FM Regression
fmbobj = FamaMacBeth(temp['Y'],sm.add_constant(temp[['X1','X2','X3','X4','X5','X6']]))
FMBres = fmbobj.fit(cov_type='kernel', kernel='parzen')

print(FMBres)

Thanks - I found the issue, a bug (wrong place for parentheses)

Fabulous! Many thanks


source_gen ^2.0.0

source_gen ^2.0.0

FYI, it requires Dart 3.6.0.  So if applied to this package, the range of supported Dart / Flutter versions would decrease significantly.  Might be worth waiting a bit since not everyone updates Flutter immediately, or at least deploy it as a new "breaking change" major version.

First, thanks for your work and time
I see, maybe you can release it as a Prerelease version?

I'm not a maintainer, so my ability to deploy anything doesn't exist.
However, if you need this now, I saw that dependabot created a PR to do it and you can use a git ref against it:
dependencies:
  envied:
    git:
      url: https://github.com/petercinibulk/envied
      ref: dependabot/pub/packages/envied_generator/main/source_gen-2.0.0

thanks


Illegal keyword: continue

with this code， jscs will throw en Error ： Illegal keyword: continue
for (var i = 0; i < 1; i++) {
  if (bool) {
    continue;
  }
}

but this block is working fine, is this a bug ?

---
Want to back this issue? **[Post a bounty on it!](https://www.bountysource.com/issues/11515713-illegal-keyword-continue?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github)** We accept bounties via [Bountysource](https://www.bountysource.com/?utm_campaign=plugin&utm_content=tracker%2F281640&utm_medium=issues&utm_source=github).

Looks like. Thanks for a report but I can't repeat it.
Can you provide your .jscsrc or find a rule that throw it?

You have enabled disallowKeywords rule, probably through crockford preset.
You can null it out, or redefine it -
{
	"preset": "crockford",
	"disallowKeywords": [
		"with"
	],

	// Or
	// "disallowKeywords": null
}

If there problem is somewhere else, please let us know, and i will re-open it


Svelte auth helpers have no provider_token strategy

I see no way to fetch the provider_token using the new Svelte helper code and example application. Can this functionality be added please.

This is saved in a cookie when using the auth-helpers-sveltekit library and can be retrieved from the cookies. You can take a look in the example code to see how it is done https://github.com/supabase-community/auth-helpers/blob/develop/examples/sveltekit/src/routes/github-provider-token.ts#L12-L13

I've added a getProviderToken method to the 0.6.6 release which only requires the request object and it will return the token once you are logged in.

Thanks @silentworks


doc: /usr/local/go/bin not added to PATH with Mac package

Downloaded Go 1.4 .pkg from https://golang.org/dl/ for Mac OS X 10.10.1
Clicked on the package to go through installation process
Restarted all the Terminal sessions

Go was installed successfully in /usr/local/go, but /usr/local/go/bin was not added to PATH. This should have happened as mentioned at https://golang.org/doc/install#osx.

After installation, is there a file /etc/paths.d/go in your file system?
The content should be "/usr/local/go/bin\n"

Yes, the file exist and has the expected content.

And if there exists a /etc/paths.d/go, then what's the output of
/usr/libexec/path_helper -s
?

PATH="/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/usr/local/git/bin:/usr/local/go/bin:/usr/local/MacGPG2/bin:/Users/arungupta/.rvm/gems/ruby-2.1.0/bin:/Users/arungupta/.rvm/gems/ruby-2.1.0@global/bin:/Users/arungupta/.rvm/rubies/ruby-2.1.0/bin:/Users/arungupta/tools/apache-maven-3.2.3/bin:/Users/arungupta/.rvm/bin"; export PATH;

There is /usr/local/go/bin in the PATH shown, could you check your actual
$PATH
against the output from path_helper -s?
Try source /etc/profile and then check $PATH again, /etc/profile should
contain
the following lines:
if [ -x /usr/libexec/path_helper ]; then
eval /usr/libexec/path_helper -s
fi
I suspect somehow your local setting doesn't source /etc/profile.

$PATH is exactly same as shown above.
/etc/profile has:
if [ -x /usr/libexec/path_helper ]; then
eval /usr/libexec/path_helper -s
fi
if [ "${BASH-no}" != "no" ]; then
[ -r /etc/bashrc ] && . /etc/bashrc
fi
source /etc/profile does not seem to update PATH. It shows:
echo $PATH
/usr/local/bin:/usr/bin:/bin:/usr/sbin:/sbin:/opt/X11/bin:/usr/local/git/bin:/usr/local/go/bin:/usr/local/MacGPG2/bin:/Users/arungupta/.rvm/gems/ruby-2.1.0/bin:/Users/arungupta/.rvm/gems/ruby-2.1.0@global/bin:/Users/arungupta/.rvm/rubies/ruby-2.1.0/bin:/Users/arungupta/tools/apache-maven-3.2.3/bin:/Users/arungupta/.rvm/bin

but then /usr/local/go/bin is right there in the $PATH.
so you still can't execute the go command?

I manually edited .bash_profile to include it in there. Will try it on a clean machine and report back.

Timed out.


Relay connect failure in beta releases. Missing Logger on stable releases.

Describe the bug
Hello. I get relay connection error on the newer beta releases:
[RelayClient], _connect error: SocketException: Failed host lookup: 'relay.walletconnect.org' (OS Error: No address associated with hostname, errno = 7)
And I cannot use stable cause there I get compile error on missing Logger class
To Reproduce
Try to connect a wallet on android. (Pixel 8, latest public android version)

Hello @haykgalstyan! SocketException: Failed host lookup: 'relay.walletconnect.org' (OS Error: No address associated with hostname, errno = 7) is usually related to not having connectivity in the device. In fact that error is coming from http package and not from AppKit. Can you please confirm you have Internet access in the device?


fix setting #EXT-X-MEDIA-SEQUENCE

rfc8216:

6.2.2.  Live Playlists
The server MAY limit the availability of Media Segments by removing
Media Segments from the Playlist file (Section 6.2.1).  If Media
Segments are to be removed, the Playlist file MUST contain an EXT-X-
MEDIA-SEQUENCE tag.  Its value MUST be incremented by 1 for every
Media Segment that is removed from the Playlist file; it MUST NOT
decrease or wrap.  Clients can malfunction if each Media Segment does
not have a consistent, unique Media Sequence Number.

The incrementation in our case only took place when persist?: true were set.
It caused live streams to loop or freeze.

new fixture file generated by changing: @create_fixtures flag to true


Object installation fails with error in 'obj_install' (local package install method)

Describe the bug
I'm trying to deploy Spectrum Scale enabling Object protocols. The installation fails with the following error message:
TASK [ibm.spectrum_scale.obj_install : install | Configure ZIMon YUM repository] *******************************************************************************************
fatal: [p1326-kvm1]: FAILED! => {"msg": "The conditional check 'scale_install_repository_url != 'existing'' failed. The error was: error while evaluating conditional (scale_install_repository_url != 'existing'): 'scale_install_repository_url' is undefined\n\nThe error appears to be in '.../roles/obj_install/tasks/install_pmswift.yml': line 15, column 7, but may\nbe elsewhere in the file depending on the exact syntax problem.\n\nThe offending line appears to be:\n\n\n    - name: install | Configure ZIMon YUM repository\n      ^ here\n"}

This seems to be related to an incorrect condition here. Replacing this line with the following causes the playbook to suceed...
when:
  - ...
  - scale_install_repository_url is defined

To Reproduce
Here are the roles I'm running:
- ibm.spectrum_scale.core_prepare
- ibm.spectrum_scale.core_install
- ibm.spectrum_scale.core_configure
- ibm.spectrum_scale.core_verify
- ibm.spectrum_scale.gui_prepare
- ibm.spectrum_scale.gui_install
- ibm.spectrum_scale.gui_configure
- ibm.spectrum_scale.gui_verify
- ibm.spectrum_scale.obj_prepare
- ibm.spectrum_scale.obj_install
- ibm.spectrum_scale.obj_configure
- ibm.spectrum_scale.obj_verify

Here are (the relevant) variables I'm defining:
scale_install_localpkg_path=.../Spectrum_Scale_Data_Management-<IP_ADDRESS>-ppc64LE-Linux-install
scale_protocols:
  smb: true
  nfs: true
  object: true

Expected behavior
Installing & enabling Object protocols should succeed using the local package install method
Environment
ansible 2.9.27
python 3.9.7
GPFS <IP_ADDRESS> (ppc64LE)

Will submit a PR once I can get to it...

#587 should fix this.

Closing this as #587 was merged.


Should not show 404 on loading bounty page

Describe the bug
When we are viewing a bounty page, we see a 404 page for a split second
To Reproduce
Steps to reproduce the behaviour:

Go to https://bountyboard.bankless.community/61b21cf5e68d920049e55485
First it shows skeleton loading
Then it shows 404
Bounty loads

Expected behaviour
404 screen should not appear
Screenshots

Start by looking here: https://github.com/BanklessDAO/bounty-board/blob/main/packages/react-app/src/pages/[id]/index.tsx


This is a Private bot

I get "This is a Private bot" messenge every time. Bot scripts not working for me.
Openwrt 22.03.

Only the Onwner of the chat started with the bot can send message to him.
This limitation is in here:
https://github.com/alexwbaule/telegramopenwrt/blob/aa57c1c7e3a648c598a77fdb5ccbb105ea29e3c9/sbin/telegram_bot#L54
If you want to use another chat, that don't belongs to you, you can change here and compare with a list of allowed users.

I still owner of my bot.

Only the Onwner of the chat started with the bot can send message to him. This limitation is in here:
https://github.com/alexwbaule/telegramopenwrt/blob/aa57c1c7e3a648c598a77fdb5ccbb105ea29e3c9/sbin/telegram_bot#L54
If you want to use another chat, that don't belongs to you, you can change here and compare with a list of allowed users.

config telegramopenwrt 'global'
	option key **'5837****37:[********]-8iJl0ryNxjVF_BpEH64'**
	option url 'https://api.telegram.org/bot'
	option my_chat_id **'5837****37'**
	option timeout '60'
	option ignored_macaddrs_file '/etc/telegramopenwrt/macaddr.ignore'

config telegramopenwrt 'ipcam'
	option header_auth '[HEADER AUTH]'
	option ipaddr '[CAMERA IP OR HOSTNAME]'
	option user '[CAMERA USER]'
	option passwd '[CAMERA PASSWD]'

config telegramopenwrt 'smtp'
	option from '[MAIL FROM]'
	option to '[MAIL TO]'

config telegramopenwrt 'television'
	option ipaddr '[TV IP]'
	option sender_name '[Sender Name]'
	option sender_number '[Sender Number]'
	option receiver_name '[Receiver Name]'
	option receiver_number '[Receiver Number]'

config telegramopenwrt 'proxy'
	option http false
	option https false

this is my config. i still cant use my bot.


How do you close an open database or table

How do you close an open database? I looked in Index.md but didn't find it.
How do you close a table?
Thank you

See Connection::deinit.

@Using Microsoft .MDB you can close a table so I thought there is an equivalent in SQLite.
In ViewController I have this code:
  import UIKit
  import SQLite

  class ViewController: UIViewController {

@IBOutlet weak var cmdSave: UIButton!
@IBOutlet weak var cmdClose: UIButton!

private var dbs: Connection?
let users = Table("users")
let id = Expression<Int64>("id")
let email = Expression<String>("email")
let name = Expression<String?>("name")  //allow null

//--------------------------
@IBAction func cmdRead(_ sender: AnyObject) {
    if sqlite3_close(dbs) != SQLITE_OK {
        print("error closing database")
    }

    dbs  = nil
}

//--------------------------
@IBAction func cmdClose(_ sender: AnyObject) {
    if sqlite3_close(dbs) != SQLITE_OK {
        print("error closing database")
    }

    dbs  = nil
}

I have successfully installed Sqlite.swift from GitHub but ...
I have been able to successfu
I am getting 2 errors:
unresolved identifier on 'SQLITE_OK'
unresolved identifier on 'sqlite3_close'
So I must not be importing a module correctly, how do I fix this?

sqlite3_close_v2  is ok, but sqlite3_close does not return SQLITE_OK
also, I feeling it can not free the connection


Incorrect signing assigned for multi dimensional flavor configurations

When there are more than one flavor dimension, the assignment of the signature is not properly done

Already done!


Schematic input does not validate against the Schema: {"name":"myApp"}

I'm submitting a...

[ x ] Bug report

In package

[x] @ng-toolkit/init
[ ] @ng-toolkit/serverless
[ ] @ng-toolkit/universal
[ ] @ng-toolkit/pwa
[ ] @ng-toolkit/firebug

Current behavior

Error on ng new :

Schematic input does not validate against the Schema: {"name":"myApp"}
Errors:

  Data path "" should have required property 'version'.

Minimal reproduction of the problem with instructions

npm i -g @ng-toolkit/init
ng new --collection @ng-toolkit/init myApp
Schematic input does not validate against the Schema: {"name":"myApp"}
Errors:

  Data path "" should have required property 'version'.

Environment

Angular CLI: 7.0.2
Node: 8.9.4
OS: linux x64
Angular:
...

Package                      Version
------------------------------------------------------
@angular-devkit/architect    0.10.2
@angular-devkit/core         7.0.2
@angular-devkit/schematics   7.0.2
@schematics/angular          7.0.2
@schematics/update           0.10.2
rxjs                         6.3.3
typescript                   3.1.3

im stuck with the same behavior how to solve it please,
Schematic input does not validate against the Schema: {"name":"myApp"}
Errors:
Data path "" should have required property 'version'.

I tried ng new xxx-app --version 7 instead of ng new xxx_app worked for me hope no further bugs in there


Mobility data stopped on April 5th

Latest entries in mobility data are April 5th.  Reports on the google mobility website show data up to April 11th.

Google's mobility website was updated only a few hours ago, the PDF reports need to be converted to CSV and then the data can be ingested

Reopening this issue, since the data is still stale in the upstream source

I've tried to get the pastelsky code working again.  Forked it and with minimal changes have the April 11 reports pulling in and processing.
If you change your fetch_mobility url to something like this:
https://ssugar.github.io/covid-19-mobility-tracker/output/VN/mobility.json
Repo with my latest changes is here: https://github.com/ssugar/covid-19-mobility-tracker
then we can have data up to April 11th.  I've done a few basic spot checks and it looks ok.
I should be able to keep the data pulls happening if pastelsky is less responsive.  FYI- I have submitted my minimal code changes back to pastelsky as a pull request.

@owahltinez - looks like Google's added a global csv file to their site, so all the reverse-engineering of the pdfs is no longer necessary:
https://www.gstatic.com/covid19/mobility/Global_Mobility_Report.csv

Thank you for the PR! I merged it for now. I will look into integrating the proper CSV file tomorrow / later this week.

Sounds good.  Thanks for all the great work on this :)

The official data source is now in the repo, although it appears to be updated sporadically and is always 1-2 weeks behind...


chore(release): v1.2.0 🎉

:robot: I have created a release beep boop
1.2.0 (2024-06-18)
Features

options: expose the .cause native error option (cadbf3b)

This PR was generated with Release Please. See documentation.

:robot: Release is at https://github.com/ehmpathy/error-fns/releases/tag/v1.2.0 :sunflower:


Cluster operator UI broken on OpenShift 4.2 - "Invariant Violation"

We are testing brand new, vanilla StroageOS deployment on OpenShift 4. 2 - when browsing cluster operator UI status URL:
https://cluster-console.app.local/k8s/ns/storageos-namespace/clusterserviceversions/storageosoperator.v1.5.1/storageos.com~v1~StorageOSCluster/example-storageos
We are getting the following error:
Oh no! Something went wrong.

Invariant Violation
Description:
Minified React error #31; visit https://reactjs.org/docs/error-decoder.html?invariant=31&args[]=object%20with%20keys%20%7BdirectfsInitiator%2C%20director%2C%20kv%2C%20kvWrite%2C%20nats%2C%20presentation%2C%20rdb%7D&args[]= for the full message or use the non-minified dev environment for full errors and additional helpful warnings.

Stack Trace:
Invariant Violation: Minified React error #31; visit https://reactjs.org/docs/error-decoder.html?invariant=31&args[]=object%20with%20keys%20%7BdirectfsInitiator%2C%20director%2C%20kv%2C%20kvWrite%2C%20nats%2C%20presentation%2C%20rdb%7D&args[]= for the full message or use the non-minified dev environment for full errors and additional helpful warnings.
    at https://cluster-console.app.local/static/vendors~main-chunk-67899ee09ff4a8f7b1bd.min.js:104:425
    at a (https://cluster-console.app.local/static/vendors~main-chunk-67899ee09ff4a8f7b1bd.min.js:104:528)
    at po (https://cluster-console.app.local/static/vendors~main-chunk-67899ee09ff4a8f7b1bd.min.js:104:47246)
    at p (https://cluster-console.app.local/static/vendors~main-chunk-67899ee09ff4a8f7b1bd.min.js:104:48881)
    at m (https://cluster-console.app.local/static/vendors~main-chunk-67899ee09ff4a8f7b1bd.min.js:104:49983)
    at https://cluster-console.app.local/static/vendors~main-chunk-67899ee09ff4a8f7b1bd.min.js:104:51959
    at wi (https://cluster-console.app.local/static/vendors~main-chunk-67899ee09ff4a8f7b1bd.min.js:104:58902)
    at Mi (https://cluster-console.app.local/static/vendors~main-chunk-67899ee09ff4a8f7b1bd.min.js:104:67287)
    at Va (https://cluster-console.app.local/static/vendors~main-chunk-67899ee09ff4a8f7b1bd.min.js:104:90685)
    at qa (https://cluster-console.app.local/static/vendors~main-chunk-67899ee09ff4a8f7b1bd.min.js:104:91069)

The cluster seems to work fine though and we can provision volumes - so it looks like an UI glitch (although it makes it rather hard to manage / monitor state...)

@nktl How are you accessing the StorageOS UI?

Just though default openshift web console:

Clicking it results with the following:

@nktl I think that may be the issue then. The example-storageos resource is a CustomResource called StorageOSCluster that the StorageOS operator acts on to create a StorageOS cluster.
The StorageOS UI is available on any node running StorageOS on port 5705. You can access the StorageOS UI through the StorageOS service, using port-forwarding or perhaps it works via the Openshift UI.  Alternatively, you can access your Nodes directly you can try :5705 to get to the browser.

Thanks, the funny thing is that the link/view in question actually works when cluster is in provisioning state, then it breaks after deployment is completed - so it looks like some value from the completed deployment is bugging it out - so might be worth investigating still. You are correct however that I misnamed this thing 'UI', it seems to be just a cluster status page.
On the actual UI front, it looks to be operational - but I am not exactly sure what the creds are, the ones from the documentation (storageos/storageos) do not seem to work, neither the api creds set during the provisioning process.

Thanks, the funny thing is that the link/view in question actually works when cluster is in provisioning state, then it breaks after deployment is completed

That's very useful to know - thank you!
The credentials for the UI are what you created in step 5 if you followed the Openshift 4.1 docs. If not then it's the apiUsername and apiPassword that are defined in the secret that you created as part of the installation process.

Hi @nktl, thanks for raising this.  We're going to try and reproduce this - it looks like an Openshift error but we might be passing it data in the wrong format.  We made some changes to the status format recently (to be more compliant with Openshift!)


Create helloyusron.cpp

What the program do ?

In what programming language it is written?

This pull request do not contains files in specific folders. Please fix it and reopen


Drop easyjson

in favor of standard library. It does double encoding both ways.

did not check the diff yet but I love the PR description :)


HTTPS does not work on non-www domain

Visiting https://thisworddoesnotexist.com/ results in a HTTPS error, as the certificate is only valid for www.thisworddoesnotexist.com and not thisworddoesnotexist.com.

Thanks for pointing that out! I'm using Google Cloud certificates and am not sure how to get the basename automatically verified. Let me drop them a line.

It took a kubernetes master upgrade, but it works now! Thanks for pointing that out


Move last element to front of a given Linked List

Description
Write a function that moves the last element to the front in a given Singly Linked List.
For example:
If the given Linked List is 1->2->3->4->5, then the function should change the list to 5->1->2->3->4.
If the given Linked List is 16->28->36->42->53, then the function should change the list to 53->16->28->36->42.
If the given Linked List is 10->20->30->40->90, then the function should change the list to 90->10->20->30->40.
Domain
Data Structure
Type of Contribution
Addition
Code of Conduct

[X] I follow Contributing Guidelines & Code of conduct of LearnCPP.

/assign


[Kahi_minciencias_opendata_affiliations] key error postcode

Plugin minciencias_opendata_affiliations failed
Traceback (most recent call last):
File "/home/ozapatam/.local/bin/kahi_run", line 37, in
kahi.run()
File "/home/ozapatam/.local/lib/python3.11/site-packages/kahi/Kahi.py", line 144, in run
status = run()
^^^^^
File "/home/ozapatam/.local/lib/python3.11/site-packages/kahi_minciencias_opendata_affiliations/Kahi_minciencias_opendata_affiliations.py", line 223, in run
self.process_openadata()
File "/home/ozapatam/.local/lib/python3.11/site-packages/kahi_minciencias_opendata_affiliations/Kahi_minciencias_opendata_affiliations.py", line 189, in process_openadata
"postcode": institution["addresses"][0]["postcode"],
~~~~~~~~~~~~~~~~~~~~~~~~~~~^^^^^^^^^^^^
KeyError: 'postcode'

closed via #224


Trying to apply a big array of actions, results in a 500 error from the backend

We are integrating part of our data cleaning process into OpenRefine. We have a ~7MB/2500 events file to record it all. When we try to apply it, the backend responds with 500 and nothing on the command line/logs. I think that should be another bug, since 500 errors, also the "please wait" loading screen should error out when the back end respond with an error.

To Reproduce
Steps to reproduce the behavior:

Grab a file
Do tons of edits, over 2400 to be sure
Extract
Revert all of the actions
Try to apply the extracted

Current Results
500 error and infinite loading screen
Expected Behavior
Either apply or error out with debugging information
Versions

Operating System: Windows or Linux
Browser Version: Firefox, Chrome and Edge
JRE or JDK Version: 11.0.14 or whatever comes with Windows installation
OpenRefine: 3.5.4 and 3.5.2

Datasets
We tested with 1 record document, dataset seems irrelevant

Hello @wetneb  Am an outreachy applicant , Love working on this issue, Kindly assigne it to me. Thank you.

Hello @braiam , am trying to reproduce this issue. on the first procedure, Instead of grab a file, do you mind sharing which file extension should i use.

Hello @wetneb  is there any specific file that i can use to reproduce this issue.

So if you are not encountering a 500 error that means you have not reproduced the issue yet :)

I think so , one issue is the file am to import and do editing.
On Wed, Apr 13, 2022, 5:03 PM Antonin Delpeuch @.***>
wrote:

So if you are not encountering a 500 error that means you have not
reproduced the issue yet :)
—
Reply to this email directly, view it on GitHub
https://github.com/OpenRefine/OpenRefine/issues/4721#issuecomment-1098091419,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ALOKS6F3PWRMNRKQOTZAPZLVE3H3RANCNFSM5S5KAXUQ
.
You are receiving this because you were assigned.Message ID:
@.***>

The 500 error occurs when trying to apply the changes, not on import.

Hello @antoine2711  Will pause this since its giving me hard time to reproduce such that i can work on it sometimes due to time available.

Hello @antoine2711 Will pause this since its giving me hard time to reproduce such that i can work on it sometimes due to time available. Otherwise i will be glad for guidance on reproducing it.

Just remove yourself from the « Assigned » position in the upper right.

Regards, Antoine

Sure thanks alot @antoine2711 .

Reviewed the json object (history extract) that the OP supplied in a comment earlier above.
It seems to be 80% of the operations are mass edits such as:
   {
        "op": "core/mass-edit",
        "engineConfig": {
            "facets": [],
            "mode": "row-based"
        },
        "columnName": "new_centro_salud",
        "expression": "value",
        "edits": [
            {
                "from": [
                    "SEMMA"
                ],
                "fromBlank": false,
                "fromError": false,
                "to": "HOSPITAL DOCENTE SEMMA SANTO DOMINGO"
            }
        ],
        "description": "Mass edit cells in column new_centro_salud"
    }

and 20% or less with text transform mass edit such as:
 {
        "op": "core/text-transform",
        "engineConfig": {
            "facets": [],
            "mode": "row-based"
        },
        "columnName": "new_CENTRO_PRIVADO",
        "expression": "value.replace(\"CEBTRO \",\"CENTRO \")",
        "onError": "keep-original",
        "repeat": false,
        "repeatCount": 10,
        "description": "Text transform on cells in column new_CENTRO_PRIVADO using expression value.replace(\"CEBTRO \",\"CENTRO \")"
    }

I see that both https://github.com/OpenRefine/OpenRefine/blob/master/main/src/com/google/refine/operations/cell/MassEditOperation.java  and https://github.com/OpenRefine/OpenRefine/blob/master/main/src/com/google/refine/operations/cell/TextTransformOperation.java
are setup to throw exceptions, but both are EngineConfig dependent as well as rely on efficient serialization and Jackson JSON handling, which could be were the real issue lays when massive amounts of either of those operations are applied (although OpenRefine currently applies them sequentially and not in parallel.)

@braiam how long was the time between "When we try to apply it," --<amount of time?>-- "the backend responds with 500"


Support for Cultivars and ICNCP naming

Hi,
Cultivars, and I suppose the ICNCP in general, appear not to be recognized by the parser. Is that something you might consider adding support for?

Support for cultivars and viruses in on a priority list, so it should happen eventually

Ok, good news that you have it on the road map at least. Thanks for the update.

Closing, because of all the great job done by @tobymarsden introduced a decent support for cultivars


WIP: Support AppImage builds

Work-in-progress, creating this PR early to test changes to the GitHub Actions workflow
TODO:

[x] The GUI needs to use a different config file path when running out of an AppImage because the current working directory is a read-only path inside the AppImage package

The config will be stored at ~/.config/jgenesis/jgenesis-config.toml for AppImage builds, using the directories crate to determine the correct base config directory

[ ] Test to make sure the builds work in different distros

Tested on a few different distros running in VMs:

OS
Status
Comment

Pop!_OS 22.04
✅
Works out-of-the-box

Ubuntu 22.04
✅
Works after installing libfuse2 (required to run any AppImage in Ubuntu)

Zorin OS 17.1
✅
Works out-of-the-box

Fedora Workstation 39
✅
Works out-of-the-box

Manjaro Linux 23.1
~
Works out-of-the-box but only with the OpenGL wgpu backend

OpenSUSE Leap 15.5
❌
glibc version is too old

OpenSUSE Tumbleweed
❌
Fails to initialize wgpu with both Vulkan and OpenGL backends

The Manjaro and OpenSUSE Tumbleweed problems seemed like graphics driver issues, possibly related to running in a VM. OpenSUSE Leap is the only one that seemed like it didn't work because of how the package was built.


Investigate / improve streaming speed

Streaming speed seems to be noticeably slower than regular completion.

Just my two cents here: I prefer streaming over regular inference with llama-cpp-python, because somehow it works "smoother". When I ask the model for a reply using normal inference, my graphics card's fans immediately go into overdrive (while they are at rest when idle), and my PC sounds like it is going to fly away. When using streaming on the other hand, the fans only smoothly start, when it takes a bit longer, but the performance is nearly the same.


support cross-namespace konk usage

Currently, to use konk, the service must be in the same namespace as konk and mount the konk-kubeconfig secret. Secrets cannot be mounted across namespaces, so that means the services must be in the same namespace.
Find some way to eliminate this requirement.

duplicate #85


Fix file browser segfault by backporting shared-mime-info upgrade to nixos-23.11

Description of changes
After a recent update, I experienced crashes that occurred any time a file chooser or pcmanfm attempted to display a directory containing .js files.  Based on strace output, the crash occurred immediately after loading /run/current-system/sw/share/mime/mime.cache.  Cherry-picking the below patch fixes the issue.
Things done

Built on platform(s)

[X] x86_64-linux
[ ] aarch64-linux
[ ] x86_64-darwin
[ ] aarch64-darwin

For non-Linux: Is sandboxing enabled in nix.conf? (See Nix manual)

[ ] sandbox = relaxed
[X] sandbox = true

[ ] Tested, as applicable:

NixOS test(s) (look inside nixos/tests)
and/or package tests
or, for functions and "core" functionality, tests in lib/tests or pkgs/test
made sure NixOS tests are linked to the relevant packages

[ ] Tested compilation of all packages that depend on this change using nix-shell -p nixpkgs-review --run "nixpkgs-review rev HEAD". Note: all changes have to be committed, also see nixpkgs-review usage
[ ] Tested basic functionality of all binary files (usually in ./result/bin/)
24.05 Release Notes (or backporting 23.05 and 23.11 Release notes)

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module

[ ] Fits CONTRIBUTING.md.

Add a :+1: reaction to pull requests you find important.

https://github.com/NixOS/nixpkgs/pull/278999 this formal backport replaces this.