Values and Units of Measurement

Added showValues and unitOfMeasure options:
datasets: [                 {                     data: [{ from: 'TEST1', to: 'TEST2', flow: 15}, {from: 'TEST3', to: 'TEST2', flow: 20}, {from: 'TEST4', to: 'TEST2', flow: 25}, {from: 'TEST2', to: 'TEST6', flow: 60}, {from: 'TEST5', to: 'TEST6', flow: 10}],                     showValues: true,                     unitOfMeasure: "kW",                     colorFrom: (c) => getColor(c.dataset.data[c.dataIndex].from),                     colorTo: (c) => getColor(c.dataset.data[c.dataIndex].to),                     borderWidth: 0,                     borderColor: 'black',                 }             ],

I'm sorry, I didn't understand what you mean.
Searching through the examples uploaded to the repository I found this one, which perhaps could come close to what you wrote in the last comment to this PR.
However, trying to change that callback, it doesn't seem to update in the chart.
Would that be what you meant?

Closing this, I will document the tooltip behavior when I have time to add docs.


Upgrade org.apache.maven.plugins:maven-clean-plugin 3.2.0 -> 3.3.1

This PR contains the following updates:

Package
Type
Update
Change

org.apache.maven.plugins:maven-clean-plugin
build
minor
3.2.0 -> 3.3.1

[ ] If you want to rebase/retry this PR, check this box

Warning
Renovate's suggested commit message is being replaced with improved initial commits to enable automerging. As a side effect, these suggested commit messages might have changed. Consider comparing the initial commit message and this message to determine the most suitable one. Please leave feedback in #sys-renovate.

Suggested commit message:
Upgrade maven-clean-plugin 3.2.0 -> 3.3.1


brew cask uninstall --force should also clean file associated with the given distributions from the download cache

Refs only remaining item in https://github.com/caskroom/homebrew-cask/issues/7571.

I am actually against this behaviour. I can already see myself running into something like this:

uninstall stage fails in brew cask reinstall some-huge-cask
try brew cask uninstall --force some-huge-cask
brew cask install some-huge-cask now re-downloads the whole thing

@reitermarkus Agreed.


fix bug when declaring a cmap and color argument would produce len(df…

…) colors instead of len(lines)
Please confirm that this PR has done the following:

[x] Tests Added
[x] Documentation Added
[x] Description in RELEASE_NOTES.md Added

Adding to RELEASE_NOTES.md (remove section after adding to RELEASE_NOTES.md)
Please add a single line in the release notes similar to the following:
- (#XX)[http://link-to-pr.com] Added feature which does something

Description of PR
Please describe the changes introduced by this PR.

Coverage increased (+0.02%) to 84.205% when pulling 95f7c7c48fb37300bcc44a5982d1be50409d6537 on gidden:cmap into a6ac0c574c3d5363fb8dba7d8b42a845fd3dbe87 on IAMconsortium:master.

Coverage increased (+0.02%) to 84.205% when pulling 95f7c7c48fb37300bcc44a5982d1be50409d6537 on gidden:cmap into a6ac0c574c3d5363fb8dba7d8b42a845fd3dbe87 on IAMconsortium:master.

@gidden, is this good to go? can you do a quick rebase, please?

should be good to go for a review @danielhuppmann

@danielhuppmann I rebased this now for most recent CI updates. Should be good to go.


Server Side Buttons Excel Export very slow

Hi Yajra!
I have the following problem: I'm using a mysql stored procedure to display the results for datatables. Everything works fine, until I hit the Export/Excel button. With 4-5000 records it took 20-25 sec to get the file. But with 14.000 records it does not work, i suffer from php timeot (max execution time is 180 sec)
The Stored procedure itself runs and gives back the results within 1.2 sec...
public function query() {
//.....
//setting up filters...
//.....
$sql = 'CALL `proc_reservation_list`(' .
            $this->language_id . ', \'' . $filter_reservation_date . '\', ' . $filter_room_id . ',
            ' . $filter_payment_id . ', \'' . $filter_contact_name . '\', ' . $filter_reservation_id . ',
            \'' . $filter_contact_email . '\', \'' . $filter_contact_phone . '\', \'' . $order_field . '\', \'' . $order_direction . '\',
            \'"active", "inprogress", "completed"\', ' . $location_id . ')';
        Log::debug($sql);

        $result = collect(\DB::select($sql));
        return $result;
}

public function ajax() {
  $result = $this->datatables
            ->of($this->query());
  return $result->make(true);
}

Operating System: Windows 10
PHP Version: 7.1.5
Laravel Version: 5.3
Laravel-Datatables Version: 6.24.4

You can override the excel method and maybe use chunk when exporting large recordset. Or better ask laravel-excel repo for a better response on how to handle large records? Thanks!


Add generic types to the Paper store

By passing the types available on a schema, there should be accessible types and autocomplete:

[ ] Within Paper.prototype.mutate and operations
[ ] Within Paper.prototype.data

CHANGELOG

package-name
* (feature) Added a new feature

Any news on this?!

Hi @mammadataei, I'm a bit stuck on keeping the context type in tact for the operations when they're destructured. I've moved the work to a typescript playground to iterate more quickly with all the types in one place. Take a look at the playground, let me know if you have any ideas. Essentially, the types get lost in the createBoundOperations function, but in the example boundCreate works when it's done directly on the operation function.

@mammadataei I just realized that this work was about having typing with the GraphQLHandler but I think your main ask had to do with the Paper store which should be much easier to spike on. I will spike on typing the Paper instances with a "Schema Types" similar to the types used in this pull request.
I also wanted to check in since the last we spoke and see if you're still use the graphql-paper and if you have any other feedback on your experience.
Cheers!

@mammadataei Okay, I got it, but it's super messy and I'm not sure which parts are required. I'll have to do a few passes, there's a bit here that's outside my understanding.

@chadian Hi again, sorry I've been busy recently. I've checked your playground, and I have no idea what's going on! I'll try to read the source code to learn how it's working; I hope I can help better. Any suggestion on where should I start?!

Hi @mammadataei, all good! I think I know how it's working in the playground, essentially type parameters need to be destructured next to each other to get the underlying type inference, and by not setting default types or passing any in. I've just updated the PR and I need to add some tests using https://github.com/SamVerschueren/tsd and I think it'll be good to go.

I thought I had it in the playground, and it's pretty close, but it's missing some sort of inference in the transaction and being bound with the operation context. I've whittled it down a more representative example with samples of what should and shouldn't pass in this playground.

Note to self: try bumping typescript and using satisfies.


MiniMap Tutorial

Description
Tutorial for MiniMap Feature
What is the purpose of this pull request?

[ ] Bug fix
[ ] New Feature
[X] Documentation update
[ ] Other

I have a problem with minimap.can you help me?

@shahbaziparisa if you have a problem, you need to open a new issue. We are trying to solve many of them this week, but we are a small team, so please be patient. Thanks!


[improve][broker] Reuse netty timer and eventloop for broker side PulsarAdminImpl

Motivation
Current when PulsarAdmin is create on the broker side. the internal async http client will create its own netty eventloop and netty timer. maybe we can just reuse the broker side resources like PulsarClient.
Modifications

add configuration in PulsarAdminBuilder to support set eventloop and timer.
save those object in ClientConfigurationData just to pass those object to the AsyncHttpClientConnector for easy parameter pass.
set those object when create AsyncHttpClient

Verifying this change
This change is already covered by existing tests, such as (please describe tests).
Does this pull request potentially affect one of the following parts:

If the box was checked, please highlight the changes

[ ] Dependencies (add or upgrade a dependency)
[ ] The public API
[ ] The schema
[ ] The default values of configurations
[x] The threading model
[ ] The binary protocol
[ ] The REST endpoints
[ ] The admin CLI options
[ ] The metrics
[ ] Anything that affects deployment

Documentation

[ ] doc
[ ] doc-required
[x] doc-not-needed
[ ] doc-complete

Matching PR in forked repository
PR in forked repository: https://github.com/lifepuzzlefun/pulsar/pull/19

@lhotari Thank you so much for providing so many background links. After read them carefully, I change to code. If i hide those share resource method in the PulsarAdminBuilderImpl can this method works without break the public api ? hopes for your idea.

all unit test pass in my local repo except coverage upload see https://github.com/lifepuzzlefun/pulsar/pull/19

@lhotari Hi, I have changed the code, PTAL : -)


upd(the-honkers-railway-launcher-bin): 1.7.1 -> 1.8.1

Yep. LGTM.

Thanks for the suggestion, I actually thought these dependencies were there in the pacscript or was preinstalled on ubuntu by default but I guess not, but yeah I added the same dependencies to all of the launchers associated.


Update to Lighthouse 5

The original container uses an ancient version of Lighthouse. Also for the new version to run some changes were required.

@justinribeiro mind to merge and create a new build over at DockerHub?

thanks


Add --input-renaming-map and --input-renaming-map-format options

With these options, it is possible to provide a file
that contain a renaming map in command line.
This feature is useful to build several css files
with the same map (like themes, specific devices, ...).
The generated map contains the content of the input map and
others renaming fields that did not in input map.

Is nobody interested with this feature ? Or everybody is overworked ;)

Hi Sébastien,
We're interested, but I have some restructuring of our internal repository
to do first. I started last week but ran into some problems that I hope to
address this week. (I thought it would be largely mechanical, but it turned
out that there was quite the hairball lurking under the surface. :-)
Ian
On Mar 19, 2015 3:07 PM, "Sébastien Mennetrier"<EMAIL_ADDRESS>wrote:

Is nobody interested with this feature ? Or everybody is overworked ;)
—
Reply to this email directly or view it on GitHub
https://github.com/google/closure-stylesheets/pull/8#issuecomment-83591988
.

We need this feature. Is everything OK?

I rebased the patch and fix the merge conflict. Everything seems OK for me.

Thanks for your work! It's awesome. I've updated infrastructure of our building system.
And we're still waiting for google accept community patch...

Hi Sébastien,
I've (finally!) gotten our internal repository ready to accept contributions. If you could update this pull request so that it is on head, that would be great.
Ian

Hey Sébastien,
If you don't have time for this, I can completely understand. Would you mind if I pulled it in internally, fixed it up and merged it? I'll give you credit in the commit log.
Thanks,
Ian

Hi Ian
Sorry for my silence, I was so busy ;)
I think that I will try to work on this patch this week or next week. I 'll let you know.
If you need urgently, I let you do the merge.
Regards,
Seb

No worries. This is not urgent—I just didn't want your work to be forgotten.
Thanks!
Ian

Hi Ian,
I rebased my patch on your master branch. I made some modifications according your previous comments.
I didn't create tests case. I don't know how test a method that take a PrintWriter as argument. I have 2 solutions:

Modify the method writeRenamingMap in RenamingMap Class in order to use a Writer and not a PrintWriter as argument. If the argument is a Writer, we can use the RenamingMap with a StringWriter in tests.

Create an object that override the PrintWriter object in tests.

I am not a specialist of Java, maybe that others solutions exists. Anyway, I think that the tests can be add in another patch if you are OK.
Let me know if you have others comments on the patch.
Seb

Hi Sébastien,
Thanks a bunch for doing this! I'm about to take a look at your patch. First, though, you can construct a PrintWriter out of any Writer, so it should be possible to do something like:
  StringWriter testWriter = new StringWriter();
  PrintWriter testPrintWriter = new PrintWriter(testWriter);
  ...
  CLOSURE_COMPILED.writeRenamingMap(map, testPrintWriter);
  assertEquals("...some expected string...", testWriter.toString());

Let me know if that works for you.
Ian

Hi Ian,
I think that your idea to construct a Printwriter with a StringWriter in parameter should work.
I will try to write test ASAP with this solution :D

Hi Ian,
I pushed a new patch that contain the class InputMapSubstitutionMap. I can merge it in previous patch if need.
Let me know if it is OK for you. If it is, I will write tests about that in a new patch.
Seb

Hi Sébastien,
Yes, this looks pretty good. I think that you can write the tests now.
 That way I'm sure that I don't break any of our internal clients. So, if you want to just keep adding commits to this branch, that's fine with me. This is the same thing we did with @eunomie's change in #66. You'll still show up as the author for the commit. See 5e681fe82d57942003056786cfb797822ff66502 for an example.
Ian

Hi Ian,
Good :smile:, I will implement tests ASAP and I will push them here.
Seb

Hi Ian,
I'm afraid for my long absence but I was really busy.
I rebased (and so rewrote) my patch in order to add this feature.
I also updated/added some tests.
Let me know if something is missing.
Seb

Hey Sébastien,
Thanks for this! I'm going to pull this internally to check it out later this week. I'll get back to you after that.
Ian

Hey Sébastien,
I looked at the change and have pulled it internally for review. I've made some changes to it so that it fits a bit better with how other things are done.
For example, I changed the input format and reader into a Map that gets read when the JobDescription gets created. That way it's possible to copy the JobDescription in a reasonable way.
You'll still get credit, though. :-)
Ian

Fixed by 13fd65c21baf7c5248570aeb7b22995071de2172.


Add option for transparent sticker click through (PE-214)

Addresses issue PE-214.  Adds an option to be able to click-through views, so stickers that have alpha do not block stickers behind them.
https://github.com/burhanrashid52/PhotoEditor/issues/214

Thanks for the PR. I would suggest extracting the transparent logic part to separate class and inject that class as a dependency in MultiTouchListner. Please let me know your thoughts on this.

Sounds good.  Inject as in use a DI framework or just make a static
instance?  Any thoughts to the transparent class name?
On Sun, Jan 5, 2020, 1:33 AM Burhanuddin Rashid<EMAIL_ADDRESS>wrote:

Thanks for the PR. I would suggest extracting the transparent logic part
to separate class and inject that class as a dependency in
MultiTouchListner. Please let me know your thoughts on this.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
https://github.com/burhanrashid52/PhotoEditor/pull/217?email_source=notifications&email_token=AAH2GCQHHGK5P4FMX4LDPQDQ4GSOLA5CNFSM4KCZSAPKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEIDSH3Y#issuecomment-570893295,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2GCXF6M5AGNQVKK6XA4TQ4GSOLANCNFSM4KCZSAPA
.

No DI framework. Just a manual instance injection. Is this just for sticker ? or for text and emoji also?

No DI framework. Just a manual instance injection.

Apologies but "manual instance injection" without a DI framework is still a bit unclear to me.  Do you mean adding it to the factory, making it a static final, adding it to the constructor, or something else?

Is this just for sticker? or for text and emoji also? If this is just for sticker than StickerTransparecny might be work. It just a suggestion. Any better name is welcome.

Uses View opacity so it should work for anything.  I'll use ViewTransparencyHelper so it's clear that it's not an actual View.

Can you give me some steps to test this?

@lucianocheng Cannot merge because of conflict.

I've changed my approach for this one. I'll close this one and recreate it.
On Sat, Apr 24, 2021, 3:30 AM Burhanuddin Rashid @.***>
wrote:

@lucianocheng https://github.com/lucianocheng Cannot merge because of
conflict.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/burhanrashid52/PhotoEditor/pull/217#issuecomment-826071776,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AAH2GCVWYYGZODGNUVPK2A3TKKMVLANCNFSM4KCZSAPA
.

@lucianocheng Do you have a plan for this to be done?

Yes, I actually have this working but it's a huge change.
Essentially the editor has to keep track of a "mask" of each image that can be used to quickly calculate intersection. This mask must be updated whenever a user resizes an image.
The recalculation of the masks happens on a threadpool, and the storage happens within each MultiTouchListener. It can also sometimes slow down the editor or take up a good amount of memory to store the masks and update them.
I need to clean up the implementation a bit and figure out how to flag it. Also right now I create a threadpool on PhotoEditor and just pass it into MultiTouchListener, which is fine but not great.

Biggest non-obvious problems with trying to solve this:

Calculating intersection without a mask will make the editor lock up. Trying to get it to traverse the tree then "search" through the image to see if it intersects a alpha pixel is very slow and users will complain
Traversing the view stack with the resize is very flaky. You can't trust getWidth() and getHeight() at all since the only thing that resizes is the scaling factor on the outer frame of the sticker. Snapshot is the only way to go that makes sense.
You need a "radius" around each alpha pixel to account for finger width. Basically if a user is trying to select a thin sticker with alpha all around it, hitting it with your finger exactly is very difficult.

@burhanrashid52 lets move further discussion over to the ticket #214


some triviality

[x] hhg overlay on mcorr in power plots.
[x] slope graph with sample mgc
[x] investigate false positives in Eklund

@jovo

I made hhg overlap above all methods except sample MGC, and updated the figure
I realized we only have sample mgc for mcorr, so I cannot draw slopegraphs for sample mcorr / mantel / dcorr....since the sample and oracle have very similar advantages from type 6-19, and I am happy with live with the oracle version for the slopegraphs.
For Eklund, they compared with parametric methods in their paper, which is why they have more false positives than expected.
To reflect this, in the abstract, I added "MGC does not suffer from .... by ""parametric methods"" "; and in real data section, I said "many standard parametric methods for fMRI analysis", and changed false discoveries to false positives, since we are not doing multiple testing here.

what do you think about at least looking at adding a 4th column for sample mgc vs. mcorr?
re:eklund, great!

done!
you can check out the slopegraph on overleaf, and I slightly updated the
caption and its main text as well, as we probably want it there.
let me know if you want other improvement of the slopegraph.
On Mon, Sep 12, 2016 at 8:53 PM, joshua vogelstein<EMAIL_ADDRESS>
wrote:

what do you think about at least looking at adding a 4th column for
sample mgc vs. mcorr?
re:eklund, great!

—
You are receiving this because you modified the open/close state.
Reply to this email directly, view it on GitHub
https://github.com/neurodata/MGC/issues/117#issuecomment-246541254, or mute
the thread
https://github.com/notifications/unsubscribe-auth/ALX0ywfbOjyUthw5Si50RtnBxp-gwczbks5qpfQkgaJpZM4J68Ua
.

i would crop the sides so it fills with width of the screen.
other than that, i think ready to send back to ruth :)

I slightly enlarged the figure. Will adjust it more once I send to Heller first


[Models] Support Azure Service Operator

Current Situation
The Azure Service OperatorAzure Service Operator (ASO) provides a wide variety of Azure Resources via Kubernetes custom resources.
Proposed Change
Create one new model for each Azure Service controlled by the AZO.

https://github.com/Azure/azure-service-operator
https://azure.github.io/azure-service-operator/

General Contributor Guides and Handbook

🙋🏾🙋🏼 Questions: Discussion Forum and Community Slack

Understanding Meshery Models

📚 Models

📚 Components
📚 Relationships

👨💻Models Repository

Contributing to Meshery Models

Contributing Models

Contributing Components
Contributing Relationships

Sir may i know where that Azure service file is located ?

@leecalcote could you assign this issue to me?


#56 - Collect JPA listeners on IdentifiableTypeMetadata

#56 - Collect JPA listeners on IdentifiableTypeMetadata
https://github.com/sebersole/hibernate-models2/issues/56

At the moment, this is just some preliminary work changing how we process hierarchies - now we start at the "root root" and walk down.


Small improvements

Signed-off-by: Marko Strukelj<EMAIL_ADDRESS>

@vsevel Could you merge this PR as well?
Alternatively, you can open your branch for external contributors, and I can add commits directly to your branch, bypassing the PR mechanism here.

Alternatively, you can open your branch for external contributors, and I can add commits directly to your branch, bypassing the PR mechanism here.

I invited you as a contributor on the repo. is this enough?

I guess it should be enough. We'll see when we need it.


Strings are getting escaped

It appears strings are getting escaped.  My unit tests are failing while switching out to strings generated by i18n.t().  Is there a way to prevent this?  I'd like the terminal to handle the newline vs getting an escaped newline.
---
en:
  command:
    failed: |
      Idempotence test failed because of the following tasks:\n%{failed_tasks}

E         - ('Idempotence test failed because of the following tasks:\\n',)
E         ?                                                           -
E         + ('Idempotence test failed because of the following tasks:\n',)

Hi,
I think this is rather related to how yaml handles escaped characters.
You could change our yml file either to
---
en:
  command:
    failed: |
      Idempotence test failed because of the following tasks:
      %{failed_tasks}

or to
---
en:
  command:
    failed: "Idempotence test failed because of the following tasks:\n%{failed_tasks}"

@tuvistavie oh duh, thanks that solved it.


Support stateless for multi container scaling and deployment

Description
I have a custom kubernetes deployment that requires stateless applications which has been a standard for a long time.
Suggested solution
I'd like to be able to deploy the app and any state necessary be offloaded to redis or refactoring to support stateless applications which have been a standard for a long time i.e. 12 factor apps
Alternative
Our internal kubernetes system doesn't support sticky sessions.
Additional context
error received:
  File "/bb/libexec/workflow-metrics-notebooks/python/lib/python3.10/site-packages/marimo/_server/api/deps.py", line 135, in require_current_session
    raise ValueError(f"Invalid session id: {session_id}")
ValueError: Invalid session id: s_p41hf3

import argparse
import logging
from typing import Any

import marimo
from fastapi import FastAPI
import uvicorn

LOG_LEVELS = ["DEBUG", "INFO", "WARNING", "ERROR", "CRITICAL"]

def parse_args(args: list[str] | None = None) -> dict[str, Any]:
    parser = argparse.ArgumentParser()
    parser.add_argument("--log-file", "-l")
    parser.add_argument("--host", default="<IP_ADDRESS>")
    parser.add_argument("--log-level", default="INFO", choices=LOG_LEVELS)
    parser.add_argument("--dir", default="./notebooks")
    parsed_args, unknown = parser.parse_known_args(args=args)
    logging.info(f'unknown args - {unknown}')
    return vars(parsed_args)

def start_server(host: str, dir: str) -> None:

    server = (
        marimo.create_asgi_app()
        .with_app(path="", root=f'{dir}/workflow-metrics.py')
    )

    # Create a FastAPI app
    app = FastAPI(host=host)
    app.mount("/", server.build())

    uvicorn.run(app, host=host, port=8080)

def main() -> None:
    args = parse_args()
    start_server(args.pop("host"), args.pop("dir"))

if __name__ == "__main__":
    main()

Copying a response from Discord, in case others are also interested:
This will be difficult since not everything is serializable so can’t be stateless. These are running programs that inherently have state. Even if we made marimo stateless, your code may not be (e.g threads, db connections, etc)
I think a better request would be a load balancer that can manage multiple instances.
Based on what you know about marimo, if you have suggestions on how marimo might one day support stateless execution, we're open to hearing them.

Thanks for the link. We'll likely look into this one day (perhaps we could patch globals() to hit an external cache) — I appreciate how this would make horizontal scaling very easy — but it's not on our short-term roadmap.


Cannot Install windows update on the server

Hi,
I've tried to install KB4025334 update on windows server 2016 using the independent installer, it is giving me an error "the update is not applicable to your computer". and when I try to run "Install-Module -Name PowerShellGet -Force" in power shell it is showing an error "PackageManagement\Install-Package : No match was found for the specified search criteria and module name
'PowerShellGet'."  I tried on other windows 2016 server has all updates up to date but same error messages.

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

ID: 38213294-2784-2341-ac3e-77d2b017e2a2
Version Independent ID: 8d30b4c5-68df-5fa2-d7e9-ede9b406527f
Content: Set up the Remote Desktop web client for your users
Content Source: WindowsServerDocs/remote/remote-desktop-services/clients/remote-desktop-web-client-admin.md
Product: windows-server-threshold
GitHub Login: @Heidilohr
Microsoft Alias: helohr

I'm facing the same issue on a newly deployed 2016 server.

#please-close


python311Packages.types-urllib3: <IP_ADDRESS> -> <IP_ADDRESS>

Description of changes

Things done

Built on platform(s)

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

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[ ] 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/)
23.05 Release Notes (or backporting 22.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.

Result of nixpkgs-review pr 230206 run on x86_64-linux 1

14 packages built:

gphotos-sync
gphotos-sync.dist
python310Packages.pyarr
python310Packages.pyarr.dist
python310Packages.types-requests
python310Packages.types-requests.dist
python310Packages.types-urllib3
python310Packages.types-urllib3.dist
python311Packages.pyarr
python311Packages.pyarr.dist
python311Packages.types-requests
python311Packages.types-requests.dist
python311Packages.types-urllib3
python311Packages.types-urllib3.dist


package name with capital letters causes error

I'm trying to install polymer and its components using rails-assets. This is part of my Gemfile:
source 'https://rails-assets.org' do
  gem 'rails-assets-PolymerElements--iron-ajax'
end

After bundle install and running rails server it gives me
$ rails server
Could not find rails-assets-polymer--polymer-1.1.3 in any of the sources
Run `bundle install` to install missing gems.

Then I looked up into the Gemfile.lock and noticed following lines:
    rails-assets-Polymer--polymer (1.1.3)
      rails-assets-webcomponentsjs (>= 0.7.2, < 0.8)
    rails-assets-PolymerElements--iron-ajax (1.0.4)
      rails-assets-Polymer--polymer (>= 1.0.0, < 2)
      rails-assets-polymerlabs--promise-polyfill (>= 1.0.0, < 2)
    rails-assets-polymer--polymer (1.1.3)
      rails-assets-webcomponentsjs (>= 0.7.2, < 0.8)
    rails-assets-polymerlabs--promise-polyfill (1.0.0)
      rails-assets-polymer--polymer (>= 1.0.0, < 2)
    rails-assets-webcomponentsjs (0.7.12)

The rails-assets-PolymerElements--iron-ajax package requires rails-assets-Polymer--polymer and the  rails-assets-polymerlabs--promise-polyfill package requires rails-assets-polymer--polymer. These two maps to the same rails-assets package but the actual installed local package name is rails-assets-Polymer--polymer, so rails thinks that this package is not installed.
By now manually replacing the package name to the actual installed name in Gemfile.lock can fix this problem.

@gksu  the problem comes from bower files, since they have different root paths for the polymer dependency.
[Polymer Iron-ajax] (https://github.com/PolymerElements/iron-ajax) has on it's [bower file] (https://github.com/PolymerLabs/promise-polyfill/blob/master/bower.json) the following (upcased Polymer)
"dependencies": {
    "promise-polyfill": "polymerlabs/promise-polyfill#^1.0.0",
    "polymer": "Polymer/polymer#^1.0.0"
}

while [Promise polyfill] (https://github.com/PolymerLabs/promise-polyfill) has this  on it's [bower file] (https://github.com/PolymerLabs/promise-polyfill/blob/master/bower.json) (downcased polymer)
"dependencies": {
    "polymer": "polymer/polymer#^1.0.0"
  }


Can't run textdistance

Both Python 2.7.14 64bit and 2.7.7 32 bit
>>> import textdistance
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "C:\Program Files (x86)\Python27\lib\site-packages\textdistance\__init__.py", line 109
      def w(f, *texts, equality=False):
                            ^
SyntaxError: invalid syntax
>>>

My workaround after testing several libraries looks as follows:
try:
  from textdistance import distance as _distance  # https://github.com/orsinium/textdistance
  def distance(a, b): return _distance('l', a, b)  # h = hamming, l = levenshtein, dl = damerau-levenshtein
  assert distance("abc", "cbe") == 2  # until bug has been fixed
  if verbose: print("Using textdistance library")
except:
  try:
    from edit_distance import SequenceMatcher as _distance # ttps://github.com/belambert/edit-distance
    def distance(a, b): return _distance(a, b).distance()
    assert distance("abc", "cbe") == 2
    if verbose: print("Using edit_distance library")
  except:
    try:
      from editdistance import eval as distance  # https://pypi.python.org/pypi/editdistance/0.2
      assert distance("abc", "cbe") == 2
      if verbose: print("Using editdistance library")
    except:
      def distance(a, b): return 0 if a == b else 1  # simple distance measure fallback
      assert distance("abc", "cbe") == 1
      if verbose: print("Using simple comparison")

textdistance developed for python3, sorry. I think, this error related with combination of starred notation and keyword arguments.
You can use python3, remove equality argument from your own code or make fork & pull request for compatibility with python2.

OK, will give it a shot at making it portable.

GNU license allow you improve and contribute this module. Feel free to contribute :)

@ArneBachmann, now TextDistance big, fast and popular. Thank you for help on start :)


The X-axis labels are overlapping at times

When a sequence number is used for points of the X axis, the labels are printed perfectly.

Sequential output:

Sequential results:

However, scaling values are used for the graphic display a more realistic representation, the labels are stacked.

Scaled output:

Scaled results:

My configuration:

I made some attempts to fix, but it still fails. I could not find the real reason of this problem.

Hi, I think I don't understand:) Could you check if you have pasted correct screenshots. I'm not sure why have you put i outside of the if (diff >= 0.1) in the second code sample.

This is part of my business rule, but I will explain why.
I work with lots of data then to try to decrease I do some "cuts" in my data, before adding to chart.
If the difference between the previous record and the current record is less than 0.1, then I delete this record of my chart.
With that, I create "spaces" within my dataset.
Theoretically no problem, because I'm not only add a record on the chart.
But this way, the Labels are not printed correctly. For the value of i is not sequential.

I have the same issue as well. Is there a way of preventing the x-axis labels from overlapping on the chart i.e. some sort of spacing rule?


Escaping fullscreen doesn't return image position/size

Happens on some devices, not happening consistently.
Black background disappears after clicking to escape fullscreen, but image sticks to the screen taking up full width. Possibly just an issue with the npm package.

Issue occurs consistently when leaving fullscreen using ESC button. Image breaks and only way to fix it is to switch to another year on the timeline or refresh the page

Possible fix on timeline branch, no longer hangs with back (mobile) or ESC (desktop). Goes fullscreen but image still has .image CSS styling when in fullscreen mode.

Screen capture of current state when fullscreened:

Currently trying to find a fix to bypass the image CSS styles when in fullscreen, however, found a bug in mobile, when trying to zoom in using my fingers, it moves to another timeline item when zoomed in enough. Reverses itself when zooming out.

Currently trying to find a fix to bypass the image CSS styles when in fullscreen, however, found a bug in mobile, when trying to zoom in using my fingers, it moves to another timeline item when zoomed in enough. Reverses itself when zooming out.

I see. that's likely an issue caused by the swipe-to-navigate code. I'll take a look and see if theres a way to bypass swipe listening when using two finger pinch.

Combining this with #22

Finger zooming on mobile doesn't work on full screen, check full screen pkg docs for workaround.


Create class to map strings to values

Used to parse colormaps, normalizer types and other future stuff from command line.

Replaced with std::map solution as of 42c47686c6452e1751f7ba8447a04959479b61f4


sdist filename differs from that generated by setuptools

Running python -m build --sdist, I see:
+ meson dist --allow-dirty --no-tests --formats gztar
Created /home/stefan/src/scikit-image/.mesonpy-zxuf91t1/build/meson-dist/scikit-image-0.20.0rc7.dev0.tar.gz
Successfully built scikit_image-0.20.0rc7.dev0.tar.gz

The first filename (with the -) matches what setuptools used to produce. But then it gets renamed to scikit_image-.... I'd like to verify whether this is (a) correct [given our package name is scikit-image] and (b) intended.

According to pypa specifications, meson-python normalizes correctly. It's different from what we had before with setuptools, but correct.


Temporarily skipping failing integration test until it can be investigated

Temporarily skipping failing integration test until it can be investigated.  This was newly added but is failing.
Motivation:
This is blocking PR builds.

@swift-ci please smoke test

Thanks for skipping, not sure why this started failing. Going to have to do something to improve these failure messages, because this is not informative:
22:57:59 Test Case '-[IntegrationTests.XCBuildTests testTestProducts]' started.
22:57:59
/Users/buildnode/jenkins/workspace/swift-package-manager-with-xcode-self-hosted-PR-osx/branch-main/swiftpm/IntegrationTests/Tests/IntegrationTests/XCBuildTests.swift:146: error: -[IntegrationTests.XCBuildTests testTestProducts] : XCTAssertEqual failed: ("terminated(code: 1)") is not equal to ("terminated(code: 0)")
22:57:59 /Users/buildnode/jenkins/workspace/swift-package-manager-with-xcode-self-hosted-PR-osx/branch-main/swiftpm/IntegrationTests/Tests/IntegrationTests/XCBuildTests.swift:184: error: -[IntegrationTests.XCBuildTests testTestProducts] : XCTAssertEqual failed: ("terminated(code: 1)") is not equal to ("terminated(code: 0)")

Looks like the remaining PR builds must pass before I can merge, but the one that previous failed seems to have passed.


Update Rollups example script

Assume we run the cron jobs on an hour basis, the following will happen:

timeline
runs_at
last_rollup_time
curr_rollup_time
what happens

the Nth cron job runs
2021-01-05 20:00:00
2021-01-05 19:00:00
2021-01-05 20:00:00
hours included [20]

new http_requests are created at 2021-01-05 20:30:00, let's say http_requests (A and B)

the (N+1)th cron job
2021-01-05 21:00:00
2021-01-05 20:00:00
2021-01-05 21:00:00
hours included [21]

Events A and B will not be added to the rollup table. To confirm:
> select date_trunc('hour', '2021-01-05 20:30:00'::timestamp) <@ tsrange('2021-01-05 19:00:00'::timestamp, '2021-01-05 20:00:00'::timestamp, '(]');
> true

Events A, and B won't be added here although their ingest_times are within the range, because they are not in the citus DB yet, they are created at 20:30:00 and the job is running at 20:00:00. That's 30 minutes in between.
> select date_trunc('hour', '2021-01-05 20:30:00'::timestamp) <@ tsrange('2021-01-05 20:00:00'::timestamp, '2021-01-05 21:00:00'::timestamp, '(]');
> false

Events A, and B won't be added here because their ingest_times are not within the range.
Which means that http_requests A and B will be lost.
I think the issue can be fixed if we use the timestamp itself without any truncation in the where clause, so instead of using date_trunc('minute', ingest_time) we just use ingest_time.

Hi @M-Sayed, is the method you're proposing preferable to that in #943? I just merged that older PR, but we can switch to use your suggestion instead if you think that's best.


feat: port Data.Nat.PSub

mathlib3 SHA: 62a5626868683c104774de8d85b9855234ac807c
I'm not sure if the file should be called Psub.lean or PSub.lean - mathport chooses the former.

Please rename to PSub.

Thanks for your comments, I think I've made all of the relevant changes.

Great, thanks. It's helpful if in mathlib PR reviews, if the author can click "resolve conversation" on everything they've dealt with. Because we have multiple reviewers dealing with things, it's great if someone coming in can quickly see what is and isn't done.
bors merge

That makes sense, thanks - I'm not very used to collaborating on projects like this just yet.


I always get the error 405

Can someone explain the error?

Could you provide the code that led to this error? Assuming you are referring to HTTP error 405, it means the request method that was used is not allowed on that API endpoint, so this might be a library issue.


perf(hummock manager): apply version delta from version checkpoint

I hereby agree to the terms of the Singularity Data, Inc. Contributor License Agreement.
What's changed and what's your intention?
apply version delta from version checkpoint
this PR is successor of https://github.com/singularity-data/risingwave/pull/3503 and #3529
Checklist
Refer to a related PR or issue link (optional)

See talk in https://github.com/singularity-data/risingwave/pull/3529.
We shall remove HummockVersionDelta when the number of which reaches large.

Btw, can you create a tracking issue for incremental version update support?

Btw, can you create a tracking issue for incremental version update support?

#3597


Feature/wallet context

Hi!
I've been working on adding a Wallet Context for encapsulationg connection & address logic.
The idea is to use:
const { address, onConnect, onReset } = useContext(WalletContext);

on Functional Components to remove the neccesity of passing the address through props on a chain.
I also refactored some components to be Functional Components instead of class Components.
What do you think about this?
Brian

Unfortunately many changes happened with the latest version of the code and now we have significan merge conflicts. I can't merge this as is right now.


Use async event emitters

Documentation: https://docs.openwebui.com/tutorials/plugin/functions/
Note the section on Valves / Event Emitters / Status / Message

This is useful for updating the user with in-between steps, such as the results of a screenpipe search.

Here's a different tool implementation that uses the event emitter:
https://openwebui.com/t/haervwe/arxiv_search_tool


Trace the epoch number and slot in epoch of the current tip

Fixes #1095.

bots r+

bors ping

bors merge


Fix pool dockerfile

During the build of the pool docker image, pip installation fails with the following warnings:
#6 [3/11] RUN pip3 install -U 	pip==9.0.3 	setuptools
#6 1.341 Collecting pip==9.0.3
#6 1.643   Downloading https://files.pythonhosted.org/packages/ac/95/a05b56bb975efa78d3557efa36acaf9cf5d2fd0ee0062060493687432e03/pip-9.0.3-py2.py3-none-any.whl (1.4MB)
#6 2.354 Collecting setuptools
#6 2.673   Downloading https://files.pythonhosted.org/packages/6d/38/c21ef5034684ffc0412deefbb07d66678332290c14bb5269c85145fbd55e/setuptools-50.3.2-py3-none-any.whl (785kB)
#6 2.846 Installing collected packages: pip, setuptools
#6 2.846   Found existing installation: pip 8.1.1
#6 2.849     Not uninstalling pip at /usr/lib/python3/dist-packages, outside environment /usr
#6 3.612   Found existing installation: setuptools 20.7.0
#6 3.614     Not uninstalling setuptools at /usr/lib/python3/dist-packages, outside environment /usr
#6 3.987 Successfully installed pip-9.0.3 setuptools-50.3.2
#6 4.143 You are using pip version 9.0.3, however version 20.2.4 is available.
#6 4.143 You should consider upgrading via the 'pip install --upgrade pip' command.

Related to these (perhaps causally) may be the following error, which fails the build:
#10 [7/11] RUN apt-get update -y && apt-get install -y         python3-pyzmq...
#10 0.597 Get:1 https://repo.sovrin.org/deb xenial InRelease [28.4 kB]
#10 0.610 Hit:2 http://security.ubuntu.com/ubuntu xenial-security InRelease
#10 0.612 Hit:3 http://archive.ubuntu.com/ubuntu xenial InRelease
#10 0.691 Hit:4 http://archive.ubuntu.com/ubuntu xenial-updates InRelease
#10 0.697 Get:5 https://repo.sovrin.org/deb xenial/master amd64 Packages [345 kB]
#10 0.771 Hit:6 http://archive.ubuntu.com/ubuntu xenial-backports InRelease
#10 1.037 Fetched 373 kB in 0s (535 kB/s)
#10 1.037 Reading package lists...
#10 2.041 Reading package lists...
#10 3.065 Building dependency tree...
#10 3.197 Reading state information...
#10 3.242 Some packages could not be installed. This may mean that you have
#10 3.242 requested an impossible situation or if you are using the unstable
#10 3.242 distribution that some required packages have not yet been created
#10 3.242 or been moved out of Incoming.
#10 3.242 The following information may help to resolve the situation:
#10 3.242
#10 3.242 The following packages have unmet dependencies:
#10 3.292  indy-plenum : Depends: python3-orderedset (= 2.0) but 2.0.3 is to be installed
#10 3.292                Depends: python3-psutil (= 5.4.3) but 5.6.6 is to be installed
#10 3.292                Depends: python3-psutil (= 5.4.3) but 5.6.6 is to be installed
#10 3.292                Depends: python3-pympler (= 0.5) but 0.8 is to be installed
#10 3.300 E: Unable to correct problems, you have held broken packages.

This may be related to the recent changes in the way pip resolves dependencies, or the new version of setuptools.
Temporarily, this issue was resolved by fetching a prebuilt pool image uploaded to GitHub Container Registry.

I can acknowledge that while going through Indy's getting started guide. I see the same error when I do

docker-compose up

I see same problem with https://github.com/hyperledger/indy-sdk/blob/master/ci/indy-pool.dockerfile

Hello, how are you?
I have the lastest version from master, but i still get the same bug when trying to build the pool like it says in the Indy's started guide:
...
Hit:1 http://archive.ubuntu.com/ubuntu xenial InRelease
Hit:2 http://archive.ubuntu.com/ubuntu xenial-updates InRelease
Hit:3 http://archive.ubuntu.com/ubuntu xenial-backports InRelease
Hit:4 http://security.ubuntu.com/ubuntu xenial-security InRelease
Get:5 https://repo.sovrin.org/deb xenial InRelease [28.4 kB]
Get:6 https://repo.sovrin.org/deb xenial/master amd64 Packages [345 kB]
Fetched 373 kB in 1s (209 kB/s)
Reading package lists...
Reading package lists...
Building dependency tree...
Reading state information...
Some packages could not be installed. This may mean that you have
requested an impossible situation or if you are using the unstable
distribution that some required packages have not yet been created
or been moved out of Incoming.
The following information may help to resolve the situation:
The following packages have unmet dependencies:
indy-plenum : Depends: python3-orderedset (= 2.0) but 2.0.3 is to be installed
Depends: python3-psutil (= 5.4.3) but 5.6.6 is to be installed
Depends: python3-psutil (= 5.4.3) but 5.6.6 is to be installed
Depends: python3-pympler (= 0.5) but 0.8 is to be installed
E: Unable to correct problems, you have held broken packages.
Any ideas on what's wrong?

Hello, I just worked through the readme and pulled the repo from the master.
I got the exact same issues as GuilhermeMenezes with the current master.
I am running an ubuntu 20.04 machine if that helps in some regard.


Switching frame graphs after resizing breaks.

Describe the bug
On initially changing frame graphs for example deferred -> hybrid, after having changed the resolution of the window. Wisp crashes on memcpy, after the acceleration structures are being uploaded.
To Reproduce
Steps to reproduce the behaviour:

Swap to master branch
Launch default wisp renderer (viknell scene)
Resize window to any resolution
press "F2" to switch from deferred to hybrid
Enjoy the crash.

Expected behaviour
Wisp switches to hybrid at new resolution without crashing.
Desktop (please complete the following information):

GTX 980m
Fallback layer.

Additional context

Tested on RTX, resizing then switching to hybrid showed black pattern artefacts on the final render and then crashed once I tried resizing after

I am unable to reproduce this.

other pr fixed this, closing issue.


Fixes wide images from overflowing in FF and IE

https://github.com/adobe-community/issues/issues/7503
Tested in FF45 and IE11:

approved

we're sure this wasn't inline-block for a real reason?  centering perhaps?  did you test it with an image that is smaller than width of the window?

I'll test with smaller images and see how it works out.

Smaller images work fine, in addition I discovered another bug which is also fixed by this PR
Notice the arrow with small image and display:inline-block

Arrow fixed with display:block

IE11 with small image and display: block (ignore the quirkyness in the browser that's because of dev tools)

FF45

as noted in a conversation with @designjockey, the design dictates that the arrows should be 40px from the edge of the image, not from the edge of the screen

So, this is a known issue in FF44, 45 and has been fixed in FF 46 which is set to release on 4/19. I downloaded the FF Nightly and it works fine in it.
https://bugzilla.mozilla.org/show_bug.cgi?id=932996

Closing: refer to issue https://github.com/adobe-community/issues/issues/7503 for more info.


[Feature request] opkg (OpenWRT package manager) support

Please test it. If there's anything wrong, please upload the file /usr/lib/opkg/status for me to debugging.

It works, thanks!

Is there a terminal on your openwrt (router?) device?

Is there a terminal on your openwrt (router?) device?

No

Can you pull the newest code and try it again with --format json?

 I think it is okey to be detected as a terminal.


Cartesian Plane -> find(Point, const char name) -> function

define the find function for points in Cartesian Plane class

Done


[Refactoring, Documentation] Clarify the entangle function

The entangle function is an operation available only for the Pulser backend which helps generate GHZ states with a specific sequence of pulses. We should:

Give it a less vague and more formal name
Implement it in PyQTorch
Document it properly with the math details on what it is doing

@jpmoutinho Is this still relevant ? Since entangle doesn't exist anymore.

It does still exist in the code base, maybe we can keep this around for the time being


[WIP] Add php-cs-fixer coding standards validation to Travis CI

Alternative PR to #13732
Right now it contains only some basic rules, but I will try to extend it in this week.
Refs: https://github.com/FriendsOfPHP/PHP-CS-Fixer

Omg, Chrome crashed on Files changed tab :fearful:

Basic fixers configuration is ready to review. There are some fixers that could be also enabled, but I prefer discuss this later in separate PRs.
The main idea is to move yii\cs to separate package (like yii2-cs), so everyone could use it as dev dependency to easily adopt Yii 2 coding standards in own projects. Right now package contain only configuration, but in future we could create some custom fixers for rules specific to Yii 2.

I'm not able to properly assess this commit because of it's size, the many changes and the fact that Chrome doesnt load the page properly.

Hmm, works fine for me on Chrome and FF. If anyone have problems try:

Browse raw patch/diff in some local tool or IDE
Browse commits one by one
Browse smaller ranges of commits (example)

how about the 120 characters soft limit from psr2? http://www.php-fig.org/psr/psr-2/ is there a 'softer' limit we can use for 80 characters?

How this "soft limit" fixer should work?

nvm https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/1361 not implemented by php-cs-fixer.

nvm https://github.com/FriendsOfPHP/PHP-CS-Fixer/issues/1361 not implemented by php-cs-fixer.

@rob006 would you please resolve conflicts? I think it's time to merge it.

Merged. Thank you!

hi @samdark Can you make cs avaliable on the packagist, So we can use it now. :smile:

@lichunqiang This is still work in progress - there are rules that I want to add to config and some issues that needs to be resolved in framework core. I want to finish config in main repo and then extract it to separate package.

Basic fixers configuration is ready to review. There are some fixers that could be also enabled, but I prefer discuss this later in separate PRs.
The main idea is to move yii\cs to separate package (like yii2-cs), so everyone could use it as dev dependency to easily adopt Yii 2 coding standards in own projects. Right now package contain only configuration, but in future we could create some custom fixers for rules specific to Yii 2.

This is exactly what I am trying to do? Has this been done?

This is exactly what I am trying to do? Has this been done?

No, not yet. You could probably install yiisoft/yii2-dev package and use the files from that. Or copy the current config from https://github.com/yiisoft/yii2/tree/master/cs


Logging with LDAP:true crashes with TypeError

Hi
When I try to log in with ldap flag set to true app crashes :
const { driver } = require('@rocket.chat/sdk');

  const runbot = async () => {

    const conn = await driver.connect( { host: HOST, useSsl: SSL})

    myuserid = await driver.login({username: USER, password: PASS, ldap:true});

fails with error message :
(node:7576) UnhandledPromiseRejectionWarning: Unhandled promise rejection (rejection id: 1): TypeError: exports.asteroid.loginWithLDAP is not a function

I've just pushed an update that ignores the LDAP settings. This isn't a fix that allows logging in with LDAP, but it at least prevents throwing due to the missing method call.
Here's what need to happen to resolve this.

need a way to test instances with LDAP logins
either change login credentials or add a new method to login with LDAP when ROCKETCHAT_AUTH=ldap
add unit tests for LDAP login, using utils to somehow enable LDAP login in the local instance before + disable after

Would love to do more but my afternoon got chewed up by other bugs thrown from core changes.
@joserenan - can you consider this in your DDP updates. Does your replacement do anything to allow logging in with LDAP?

Update on this issue. The "TypeError" is resolved because the LDAP login logic has been removed for now (it never would have worked anyway).
I tried setting up a local LDAP server with ldapjs, to be used by unit tests attempting to pass whatever attributes are required in the ddp login method for ldap.
Unfortunately I don't know enough about the LDAP login requirements to either A) setup the LDAP server appropriately, B) configure Rocket.Chat to work with it, or C) define the methods to use it in the SDK.
If anyone actually implementing LDAP can find out some or all of those answers, please comment.


Add ability to see all upcoming events of a tag or chapter

handle urls
/:tag -> redirect to /:tag/events
/chapter/:chapterId -> redirect to /chapter/:chapterId/events

Implemented in https://github.com/gdg-x/firefly/pull/102 for the next 100 events of a tag. Still need to make a page for all events of a chapter though.

Next request will be searching for an event in the list of 100 :)


update: change percent text Bot to End

In the status bar, the percentage text displays "Bot," which initially confused me. After moving the cursor around, I realized it stands for "bottom." It might be clearer to change it to "End," which would make the meaning more obvious to users.

Sorry, I'm afraid I can't accept this PR. The use of "Bot" (short for bottom) is intended to pair with "Top", as in bottom / top
And using "End" sounds a bit strange, like end / top — it doesn't quite follow the usual English convention.
Also, "Bot" is borrowed from Vim/Neovim - they also use "Bot" as the short form of "bottom". Since Yazi is a Vim-like file manager, I don't think this should be an issue.


RequestInit::new() is hanging

Hi!
It seems like something's wrong with RequestInit::new(). Here is the simple code, it creates empty RequestInit and then returns empty response. And it stucks.
use worker::*;

mod utils;

fn log_request(req: &Request) {
    console_log!(
        "{} - [{}], located at: {:?}, within: {}",
        Date::now().to_string(),
        req.path(),
        req.cf().coordinates().unwrap_or_default(),
        req.cf().region().unwrap_or("unknown region".into())
    );
}

#[event(fetch)]
pub async fn main(req: Request, env: Env) -> Result<Response> {
    log_request(&req);

    utils::set_panic_hook();

    let _init = RequestInit::new(); // the code stucks here

    Response::empty()
}

And here's the logs from wrangler dev:
👂  Listening on http://<IP_ADDRESS>:8787
Sun Sep 12 2021 12:06:47 GMT+0000 (Coordinated Universal Time) - [/], located at: (59.8983, 30.2618), within: St.-Petersburg
Error: Worker exceeded CPU time limit. at line 0, col -2
{
  "exceptionDetails": {
    "columnNumber": -2,
    "exception": {
      "className": "Error",
      "description": "Error: Worker exceeded CPU time limit.",
      "preview": {
        "description": "Error: Worker exceeded CPU time limit.",
        "entries": null,
        "overflow": false,
        "properties": [],
        "subtype": "error",
        "type": "object"
      },
      "subtype": "error",
      "type": "object",
      "value": null
    },
    "lineNumber": 0,
    "text": "Uncaught (in response)",
    "url": "undefined"
  },
  "timestamp":<PHONE_NUMBER>649
}
[2021-09-12 15:06:46] GET response-bot-dev.idebugger.workers.dev/ HTTP/1.1 503 Service Unavailable

Thank you for reporting this. RequestInit definitely needs some work, but it is interesting that it seems to be consuming so much CPU. Maybe it is recursively calling something. I will take a look.

Thank you for reporting this. RequestInit definitely needs some work, but it is interesting that it seems to be consuming so much CPU. Maybe it is recursively calling something. I will take a look.

It's this line:
https://github.com/cloudflare/workers-rs/blob/00ce8ce9ae0d899d0d5dbc7e037a6ecd94b23c1f/worker/src/request_init.rs#L84-L88
Request_init::default called Request_init::new which called Request_init::default recursively.

I hope this will be fixed soon. AFAIK it's impossible to make POST requests without RequestInit. Is it so?

@OlegWasHere - I'm aiming to get a release out today which fixes this. However, if you don't need any Cloudflare-specific features for your outbound Request, then I'd try to use another HTTP client crate that works with wasm32 target, such as reqwest

@nilslice Oh, thanks for clarifying! I didn't know that it's possible to use other HTTP clients. And thanks for the fix :)


Feature: Zarr output

In order to enable alternative stitching and fusion capsule (i.e. those in use that implement BigStitcher), file out put from this capsule should allow zarr as an option.
I don't think this needs to be OMEZarr yet, but it might -- need to check the relevent capsules. At the very least, it should align with the OMEZarr conventions for folder structure.

@camilolaiton : If I find time to work on this, would you like me to work on a forked repo? It looks like I can't create a branch here

Hi @miketaormina, thanks for following up on this. Just gave you permissions to add your branch. I'll be working on writing zarr for the future release.


Not all candidates gets send to the visualisation

I experienced some problem with the visualisation. It seems to be the case that not all new generations get send any more.
While looking at the output it seems to be the case that a "solution" is found, but no survivor is received in the visualizer

This is a feature


How to find out where the dragged item has ended

Hi,
Not an issue, per se, but I'm trying to do some checking on where an item has been dragged to. Using onDragEnd I have this, which only gives me an object (in bold):
window.addEventListener('load', function() {
let eLLs = document.getElementsByClassName('pdg'); let tOT = 0;
while (tOT<eLLs.length) {
draggable = new PlainDraggable(eLLs[tOT]);
draggable.onDragEnd = function(clientY) {    console.log('Where is this? '+clientY);    }
// Where is this? [object Object]
tOT++;
}
});
How do I access the relevant properties of left and top of the object?
Regards,
Philip Bolt

Hi @WillyWilson1, thank you for the comment.
Replace this line:
draggable.onDragEnd = function(clientY) { console.log('Where is this? '+clientY); }

with:
draggable.onDragEnd = function(clientY) { console.log('Where is this? left:', clientY.left, 'top:', clientY.top); }

Sorry, my English is poor.
Do you mean that you want to get each index number of  the draggable instances?

Hi Anseki,
Thanks for another quick reply.
Yes, that is exactly what I mean - below is an example
This is what I have on the screen

The draggable divs are below in yellow, and I want to move them over the same word above in white/blue, and when I do
I get this in the console - your code

That is exactly right, but I further need to access the innerHTML of the dragged item to check where it has ended its drag. This is what the 2 moved elements look like
This

This

This~0~OvSBg~26.40~20~20~10~10~30~30~FREE~125~42.74~25~100

fullstop

.

.~16~OvSBg~9.32~141.87~45~131.87~35~151.87~55~FREE~100~130.18~25~100

I need to access this dragged item through the div id, for example, move_14_OvSBg and then get the innerHTML - which is another div which has top and left values of the correct dropped area.
Finally, as you can see, the console.log values for top reference the top of the page, not the containing div (blue border), which is why I need to know how to define that div through its id in the code.
I hope this is clear - your English looks fine to me.
Any advice much appreciated.
PB/WW

To get each index number, replace the line with:
draggable.onDragEnd = function(clientY) { console.log('Where is this? left:', clientY.left, 'top:', clientY.top, 'index:', tOT); }

You can get the element and its innerHTML by this:
draggable.onDragEnd = function() { console.log(this.element.innerHTML); }

Hi Anseki,
Thanks for another quick reply.
Yes, that is exactly what I mean - below is an example
This is what I have on the screen

The draggable divs are below in yellow, and I want to move them over the same word above in white/blue, and when I do
I get this in the console - your code

That is exactly right, but I further need to access the innerHTML of the dragged item to check where it has ended its drag. This is what the 2 moved elements look like
This

This

This~0~OvSBg~26.40~20~20~10~10~30~30~FREE~125~42.74~25~100

fullstop

.

.~16~OvSBg~9.32~141.87~45~131.87~35~151.87~55~FREE~100~130.18~25~100

I need to access this dragged item through the div id, for example, move_14_OvSBg and then get the innerHTML - which is another div which has top and left values of the correct dropped area.
Finally, as you can see, the console.log values for top reference the top of the page, not the containing div (blue border), which is why I need to know how to define that div through its id in the code.
I hope this is clear - your English looks fine to me.
Any advice much appreciated.
PB/WW

Hi Anseki,
That's great! All the values I need are now accessible.
The last thing is how to define and access a div as the containment area so that I can use that top to check against the dragged item's top. I have a div with an id and all the styling, but currently am using the default containment area - which is the top of the page, I think.
As before, any advice/example would be appreciated.
I think we must be in different time zones.
PB/WW

Maybe you mistook this duplicated comment (https://github.com/anseki/plain-draggable/issues/102#issuecomment-968062257).
You can access the containment option via containment property:
draggable.onDragEnd = function() { console.log(this.containment); }

Hi Anseki,
Yes, I did. Your code now gets me the right containment item

so how do I get the top and left values here - both 10px?
PB/WW

You can access to CSSStyleDeclaration via HTMLElement.style property.
draggable.onDragEnd = function() {
  this.containment.style.left = '10px'; // SET
  this.containment.style.top = '20px'; // SET
  console.log('left:', this.containment.style.left, 'top:', this.containment.style.top); // GET
}

Hi Anseki,
Yes, that's exactly it - I get confused between element and this.
Many thanks for the code and your patience.
PB/WW

I'm glad if I could help you. :smile:
The this refers to the current PlainDraggable instance.
See document for details.
https://anseki.github.io/plain-draggable/
The document includes description about the this, element, left, top, containment, and more.


activate widget by object

So far it is only possible to activate widgets via the Tab key. All the activation methods from QWidget are private, and the ones from QLayout are protected. Maybe it's possible to inherit from QLayout and hack something together with the protected methods like activateEvent or cycleEvent, however that's quite cumbersome.
Also, I don't like the tab key very much. It might make sense for some applications, but definitely not for all. E.g. in an application that I'm building right now, I have a horizontal layout with a visible and a hidden widget. When I press a special key I want to show the hidden widget and activate it. If I press another key, I want to hide it again, and jump back to the previously active widget.
Additionally at the start of a qui application, no widget is active, and once you iterated over all widgets then for a moment no widget is activated and you have to press Tab again to reactivate the first widget. Both things are also quite annoying for my purpose.

In this pull request I add a new method to activate a widget.
The QTerminal object has now a public bool activateWidget(QWidget target) method.
This method recursively searches the layouts for the widget, and activates it.
I've also added an example in the quidemo application.
The Tab functionallity is still preserved, this pull request simply adds an additional way of activating widgets.

Thanks for contributing. Just a few more changes and this will be in release.


Command line bug after releasing and update

Description
After executing appcenter codepush release-react -a <owner>/<appName>, the command line bugs out. When pressing the arrow up or down, it doesn't show previous executed commands. It shows ^[[A or ^[[B.
Repro Steps
Please list the steps used to reproduce your issue.

Execute appcenter codepush release-react -a <owner>/<appName>
Press arrow up or down or backspace.

Expected behavior
CMD behaving normally.
Details
App center CLI version: 2.10.10
react-native info:
System:
    OS: Windows 10 10.0.19042
    CPU: (16) x64 AMD Ryzen 7 2700X Eight-Core Processor
    Memory: 11.99 GB / 31.93 GB
  Binaries:
    Node: 14.16.0 - C:\Program Files\nodejs\node.EXE
    Yarn: 1.22.10 - ~\AppData\Roaming\npm\yarn.CMD
    npm: 6.14.11 - C:\Program Files\nodejs\npm.CMD
    Watchman: 20210102.202219.0 - C:\Users\roger\watchman\bin\watchman.EXE
  SDKs:
    Android SDK: Not Found
    Windows SDK: Not Found
  IDEs:
    Android Studio: Version  <IP_ADDRESS> AI-201.87<IP_ADDRESS>42882
    Visual Studio: 15.9.28307.1401 (Visual Studio Community 2017)
  Languages:
    Java: 17.0.1
  npmPackages:
    @react-native-community/cli: Not Found
    react: ^17.0.2 => 17.0.2
    react-native: ^0.68.1 => 0.68.1
    react-native-windows: Not Found
  npmGlobalPackages:
    *react-native*: Not Found

Hi @Rc85 , this is a normal behaviour when you perform some long lasting command. It is not related to the appcenter-cli.
I'm closing this issue, but feel free to ask any additional questions.


The entire module stopped working

I made this update progress:
Bump vite-plugin-static-copy from 0.17.1 to 1.0.0
"@vitejs/plugin-react": "4.2.1",
"vite": "5.0.7",
"vite-plugin-electron": "^0.15.4",
"vite-plugin-electron-renderer": "^0.14.5",
"vite-plugin-static-copy": "1.0.0"

Now the module no longer loads on Vite in any way.
vite.config.js:4:31:
      4 │ import { viteStaticCopy } from 'vite-plugin-static-copy';

Error: Build failed with 1 error:
node_modules/esbuild/lib/main.js:1373:27: ERROR: [plugin: externalize-deps] "vite-plugin-static-copy" resolved to an ESM file. ESM file cannot be loaded by `require`. See https://vitejs.dev/guide/troubleshooting.html#this-package-is-esm-only for more details.

When I downgraded, everything went back to normal. So the problem is in the version 1.0.0

1.0.0+ is ESM only. See the link in the error messsage.

But I'm using an ES environment. And yet he thinks I'm not in an ES environment

I renamed the file vite.config.js to vite.config.mjs and this is working now.
Your module was using the file name as a reference to know if it was an ESM or not.


PP-10864: Update go live request ticket (for Worldpay only)

Update go live request ticket with current go live macro when making a service live. Unfortunately solving the ticket using the zendesk client is troublesome at the moment as the custom fields "[Pay] Type of user" and "[Pay] Type of user other (enter detail)" need to be populated first. This is certainly possible but is something we can make happen later.
Video that shows successful updating of a ticket from toolbox.
Screenshot of "Gateway account created" page in my local toolbox:

The changes look good to me. Looks like adding the zendesk library is introducing vulnerable dependency (highlighted by Snyk). Can this be resolved?

I had a look at it before but I don't think so. We're using the latest zendesk node library already. Looking at https://app.snyk.io/org/govuk-pay/project/db2615e3-e5a1-40d4-8b62-65ad04f3e1bc/pr-check/7ec90881-3756-428c-88fe-47fcc6992784, the node-zendesk library is using<EMAIL_ADDRESS>which is deprecated. We're using the same node-zendesk library in selfservice too, I wonder why snyk isn't picking up the vulnerability 🤷

Snyk only looks into PR changes and this PR has added the request library. It doesn't look like the library is officially supported by zendesk https://developer.zendesk.com/documentation/ticketing/api-clients/nodejs/. May be fine for now but we should look into using a different library which is maintained or integrated with zendesk directly.

Snyk only looks into PR changes

Oh didn't know that! It makes snyk look marginally useful then. Or maybe we need to pay them more to scan current dependencies.

we should look into using a different library which is maintained well or integrate with zendesk directly

Looks like the path to go down would be integrating with zendesk directly. Their official docs have NodeJS examples which use axios, for eg
var axios = require('axios');

var config = {
  method: 'GET',
  url: 'https://example.zendesk.com/api/v2/tickets',
  headers: {
	'Content-Type': 'application/json',
	'Authorization': 'Basic <auth-value>', // Base64 encoded "username:password"
  },
  params: {
    'external_id': '',
  },
};

axios(config)
.then(function (response) {
  console.log(JSON.stringify(response.data));
})
.catch(function (error) {
  console.log(error);
});


How to perform onClick

I want to make target view invisible on performing onclick or please suggest any other option to do the same.

Following are few methods you can use for that:
performClick(boolean) - Perform a click on target view
dismissOnTouch(boolean) - Dismiss spotlight on touch outside
enableDismissAfterShown(boolean) - Dismiss spotlight on touch outside after spotlight is completely visible
If this is not you want then please elaborate more.


Support --parallel auto option

Currently, when specifiying  sphinx-performance --parallel auto  (equivalent sphinx-build -j auto) we get the error:
Error: Invalid value for '--parallel': 'auto' is not a valid integer.

Thanks for the reports, but I'm not sure if a performance-report script should provide a non reproducible option, which may interact differently on each machine.
Also "auto" would currently not get resolved in the report and you would have no glue, how many cpu cores got used for a specific report.
For me "auto" makes only sense, if you don't know the available cores on a system. But then I don't know, if a created report is comparable to any other report....

Ok, makes sense, thne maybe just add to the doku, that with --parallel, the auto option is not supported like it is for -j in sphinx-build.
The -j option itself is not very good, since you never know if the number you specified is really supported/used, since sphinx limits itself internally if too big, so you'll never know for sure how many cores were used ;)

Thanks for the hint with -j.
Maybe in this case sphinx-performance should also do a check and write a warning, if not so many cores are available as specified.

Thanks for the hint with -j. Maybe in this case sphinx-performance should also do a check and write a warning, if not so many cores are available as specified.

Then sphinx-performance would be better than sphinx-build, actually ;)
As a side note: -j option doesn't work at all with -j on Windows. It is simply not supported, but sphinx-build doesn't give any info on that ...


database/sql: permit tagged struct in Scanner

Hi all,
First of all, thank you for Go. Its Awesome!
I like the idea how encoding/json Marshals and Unmarshals json objects into structs..
One could define the database model like so:
type UserModel struct {
    Id int `db:"id"`
    Name string `db:"user_name"`
}

And when scanning rows from the database:
for rows.Next() {
    user := UserModel{}
    if err := rows.Scan(&user); err != nil {
        panic(err)
    }
}

What are your thoughts?

3rd-party packages to make nicer scanners seem like the best route here. I don't see a compelling reason to try to standardize this in database/sql itself, and worse I don't see one obvious design. There are many possibilities. Let people use what works for them instead of forcing one.

I'm thinking, time.Time should have simple scanner. Because when driver can't recognize type of the field value, driver must return string or []byte.


feat(cards): added data-point styling within cards

Description

Styles lg-data-point element within lg-card-content

Supports 1 to 3 lg-data-points per card

Adds fr as a supported unit in the CSS

Fixes # (issue)
Requirements
Please briefly outline any requirements for the work which may aid the testing and review process.
Design link: https://www.figma.com/file/Mts4nibXQw6tDW2PTDHlKd/Patterns?node-id=1333-15784&t=cmMhG0Bg3IAbQyKf-0
Screenshot:

test app:

storybook:

Checklist:

[x] The commit messages follow the convention for this project
[ ] I have provided an adequate amount of test coverage
[x] I have added the functionality to the test app
[ ] I have provided a story in storybook to document the changes
[ ] I have added the documentation
[ ] I have added any new public feature modules to public-api.ts

I can't see any similar designs for this but it seems to me that the gap between the two is a bit too small? @alexcanning ?

I've added a 16px gap between the 2 columns


typo mistake resolved

Dear all,
I've noticed a typo mistake in the code, and I was expecting it to be addressed in the recent PRs. As it hasn't been updated, I'm submitting a separate PR to rectify this issue. Thank you!!

Effectively, that's not a typo. 2*sv_area_sz(sp) is reserved for Vmode (Its Hypervisor Mode save area), so using it for Umode will break a few things within trap handler, but there is a bug in GOTO_Lower_Mode macro as it is not going in Umode correctly.

@Ammarkhan561 I have updated the GOTO_Lower_mode macro, so this PR can be closed.


Adding zio-http repo to Scala-Steward, for keeping dependencies updated

For more information:
Using Scala-Steward, for keeping dependencies updated.
https://github.com/scala-steward-org/scala-steward

Added repos dream11/zio-http to scala-steward.
cf. https://github.com/scala-steward-org/repos/pull/620

Thank you @sh0hei


Add guide from nos.pt

#1818

Hello, thks for the work, how put the epg nos.pt in progrmam? Thks

@ajanes3 I could well be wrong, so please do not approve PR if you are not sure it is error-free.

Approved. I'm just going to assume you did this correctly.
@Thermaldude This is still a pull request. You need to wait until it is merged before it will be available.

Is already pass, but i dont find in epg link, you can help me please? Sorry but i really dont understand how this work, thks

Hello @Thermaldude, The change was merged 18 hours. You need to allow from 24 to 48 hours after it has been merged to see the new EPG link.


added option to exclude packages from build

Could you please add a corresponding change to rdmd's documentation?


An apostrophe breaks syntax highlighting

Looks like the grammar thinks it's a Javascript string even when it's in the JSX part of the file:

Looks like there's still an issue. This is 0.12.10.

Still an issue on 1.2.4

also, text inside JSX elements gets keyword highlighting:

@dmnd can you reopen this issue please?

:+1:

I can't find how to reopen, so I just created #124.

This is still an issue. Is anyone working on the problem?

It seems to regress often. Whoever fixes it should try to add a regression
test too.
On Wed, Apr 20, 2016 at 10:33 Jessie Richardson<EMAIL_ADDRESS>wrote:

This is still an issue. Is anyone working on the problem?
[image: screen shot 2016-04-20 at 1 32 21 pm]
https://cloud.githubusercontent.com/assets/9681896/14684142/69cb5666-06fc-11e6-8503-a06bdf8936b4.png
—
You are receiving this because you were mentioned.
Reply to this email directly or view it on GitHub
https://github.com/orktes/atom-react/issues/3#issuecomment-212527752

I've noticed that adding a . on the same line before the apostrophe is sometimes responsible for triggering this.

I also have this problem

I have this problem right now too - despite an escape even

I am having this issue too so I've decided against using apostrophes. I just hope the grammar police don't come calling.

Any update on this?

Running into this issue as well, I am coerced into foregoing apostrophes  :(  Any updates?

The struggle is still real.  Any ideas?

Same issue in my project

if i use this ’, instead of this ' it works. Not sure if it's grammatically right and allowed.

Same issue here!

Same issue for me.
A temporary work around is to use ' instead of '.

I'm having this issue too.
Workaround: put the text in a JavaScript var and insert it into JSX with the {} notation:
  render() {
    const title = "Don't worry, be happy!";
    return (

{title}

    );
  }

This is really bothersome :(

Yes, this is a sad limitation, makes a component almost unusable any time we need someone's stuff to have an apostrophe :-(

decorative apostrophe ftw

Dirty workaround : use  ' entity 👎

Yet another workaround:

  I{`'`}m a workaround. Let{"'"}s get some frosty chocolate milkshakes!

1 for '

I have the same problem, any updates?

I'm having the same issue as @kengres.

Sames. Question mark also causes a problem - doesn't error-highlight subsequent text but stops syntax highlighting

same as @kengres

I have been working around the issue by adding a double apostrophe or double single quotes

Rest of team uses WebStorm and (understandably) don't want workarounds for atom-react in the code. Can we get word on if you guys have seen this issue and if its being prioritised? thanks

@proftom @srinath-ragh @justinzelinsky I've installed a package called language-babel and the issue is gone...

related issue... https://github.com/orktes/atom-react/issues/212
Also looks like language-babel is no longer supported. Wish I had some free time to help with a fix!

I'm hitting this bug as well.  It was first reported two and a half years ago!
Any work arounds?

I'm encountering this as well. I hoped this gets resolved soon. @OriginalPenguin's workaround is ingenious, so thanks for that!

This was a problem for me, too. Then I switched the language to JavaScript (JSX) and it started working. The only remaining problem is that syntax highlighting still breaks when the apostrophe isn't followed by another character. So something like dogs' would not work.

Actually, a co-worker discovered the language-babel plugin handles the trailing apostrophe as well.

Still not working...

Issue is still around in 2017

Switch to language-babel ... much nicer
On Fri, 20 Jan 2017 at 1:13 pm, Charles Moyes<EMAIL_ADDRESS>wrote:

Issue is still around in 2017
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
https://github.com/orktes/atom-react/issues/3#issuecomment-273958233,
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAcs8rHjaklZYH_mYN1ekLmvpSnUdTXfks5rUBgygaJpZM4CH72o
.

@orktes Any updates to this?

language-babel is the way to go, thanks for the recommendation

How is this a thing?

I would suggest installing language-babel package [https://atom.io/packages/language-babel]

Why does this issue still exist...? just ran into it...

Good, it's not just me. Was really annoying. Installing Language-babel solved the issue.

I believe this post on Stack Overflow should help. https://stackoverflow.com/q/32979512
...
return (

{"I've seen the movie."}

 )
...

@orktes I guess you can close this one. As @kengres mentioned long ago there is a solution for this.
Installing language-babel on current 1.18.0 version (and assume that also on others) fixes the issue

Is there a reason to use this project in 2017? All features from atom-react are native in Atom except the code snippets.

@NathanCH Atom doesn't include proper indentation support (nested tags and attributes), highlighting and tag auto completion by default.

language-babel solves this issue

Our workaround is to use '

this code is usefull for me. you can use it as follow :
Let{"'"}s start by getting to know your business...

this code is usefull for me. you can use it as follow :

Let{"'"}s start by getting to know your business...

I installed language-babel and it resolved the issue for me.

use different class like Constant.js
declare your text there:
export const TEXT = 'Didn`t find what you want?';

app.js

import { TEXT } from '../Constants';

<Text>TEXT</Text>

Another workaround:
{'We won't do anything with you email. It's just to prevent spam.'}
A bit cleaner than the others suggested.

Just downloaded Atom yesterday and this is an issue for me as well :(

Follow @nseller's suggestion
https://github.com/orktes/atom-react/issues/3#issuecomment-371200678

Thanks! That worked wonders! :)

Any updates on this?
Same issue here.

language-babel

Additional not breaking style alternative: https://atom.io/packages/react

nseller's comment worked for me but I had to disable atom-react.

nseller's comment worked but I had to uninstall atom-react.

language-babel worked, and I didn't have to uninstall atom-react OR restart Atom. Cool!

I uninstalled atom-react and installed language-babel, and I'm still having this issue with both, even after uninstalling.


ActionView::Template::Error: undefined local variable or method `distinct' for #Follow:0x007ff4bca831e0

View details in Rollbar: https://rollbar.com/trystant/BOW/items/476/

NameError: undefined local variable or method `distinct' for #<Follow:0x007ff4bca831e0>
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/activemodel-4.2.6/lib/active_model/attribute_methods.rb", line 433, in method_missing
  File "/Users/dariusgoore/development/blackops/app/models/follow.rb", line 23, in mom_uniq_followers_growth
  File "/Users/dariusgoore/development/blackops/app/views/analytics/index.html.erb", line 72, in _app_views_analytics_index_html_erb__3780132927396732422_70344499505500
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionview-4.2.6/lib/action_view/template.rb", line 145, in block in render
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/activesupport-4.2.6/lib/active_support/notifications.rb", line 166, in instrument
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionview-4.2.6/lib/action_view/template.rb", line 333, in instrument
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionview-4.2.6/lib/action_view/template.rb", line 143, in render
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionview-4.2.6/lib/action_view/renderer/template_renderer.rb", line 54, in block (2 levels) in render_template
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionview-4.2.6/lib/action_view/renderer/abstract_renderer.rb", line 39, in block in instrument
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/activesupport-4.2.6/lib/active_support/notifications.rb", line 164, in block in instrument
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/activesupport-4.2.6/lib/active_support/notifications/instrumenter.rb", line 20, in instrument
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/activesupport-4.2.6/lib/active_support/notifications.rb", line 164, in instrument
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionview-4.2.6/lib/action_view/renderer/abstract_renderer.rb", line 39, in instrument
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionview-4.2.6/lib/action_view/renderer/template_renderer.rb", line 53, in block in render_template
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionview-4.2.6/lib/action_view/renderer/template_renderer.rb", line 61, in render_with_layout
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionview-4.2.6/lib/action_view/renderer/template_renderer.rb", line 52, in render_template
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionview-4.2.6/lib/action_view/renderer/template_renderer.rb", line 14, in render
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionview-4.2.6/lib/action_view/renderer/renderer.rb", line 46, in render_template
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionview-4.2.6/lib/action_view/renderer/renderer.rb", line 27, in render
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionview-4.2.6/lib/action_view/rendering.rb", line 100, in _render_template
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionpack-4.2.6/lib/action_controller/metal/streaming.rb", line 217, in _render_template
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionview-4.2.6/lib/action_view/rendering.rb", line 83, in render_to_body
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionpack-4.2.6/lib/action_controller/metal/rendering.rb", line 32, in render_to_body
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionpack-4.2.6/lib/action_controller/metal/renderers.rb", line 37, in render_to_body
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionpack-4.2.6/lib/abstract_controller/rendering.rb", line 25, in render
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionpack-4.2.6/lib/action_controller/metal/rendering.rb", line 16, in render
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionpack-4.2.6/lib/action_controller/metal/instrumentation.rb", line 44, in block (2 levels) in render
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/activesupport-4.2.6/lib/active_support/core_ext/benchmark.rb", line 12, in block in ms
  File "/Users/dariusgoore/.rubies/ruby-2.3.0/lib/ruby/2.3.0/benchmark.rb", line 308, in realtime
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/activesupport-4.2.6/lib/active_support/core_ext/benchmark.rb", line 12, in ms
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionpack-4.2.6/lib/action_controller/metal/instrumentation.rb", line 44, in block in render
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionpack-4.2.6/lib/action_controller/metal/instrumentation.rb", line 87, in cleanup_view_runtime
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/searchkick-1.2.1/lib/searchkick/logging.rb", line 158, in cleanup_view_runtime
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/activerecord-4.2.6/lib/active_record/railties/controller_runtime.rb", line 25, in cleanup_view_runtime
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionpack-4.2.6/lib/action_controller/metal/instrumentation.rb", line 43, in render
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/remotipart-1.2.1/lib/remotipart/render_overrides.rb", line 14, in render_with_remotipart
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionpack-4.2.6/lib/action_controller/metal/implicit_render.rb", line 10, in default_render
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionpack-4.2.6/lib/action_controller/metal/implicit_render.rb", line 5, in send_action
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionpack-4.2.6/lib/abstract_controller/base.rb", line 198, in process_action
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionpack-4.2.6/lib/action_controller/metal/rendering.rb", line 10, in process_action
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/actionpack-4.2.6/lib/abstract_controller/callbacks.rb", line 20, in block in process_action
  File "/Users/dariusgoore/development/blackops/.bundle/gems/ruby/2.3.0/gems/activesupport-4.2.6/lib/active_support/callbacks.rb", line 117, in call
  File "/Users/dariusgoore/d

This is development, this can be closed:


Support accent in localSort() for grid

Describe the problem your feature will solve.
Sorting the grid with the sort() or localSort() method doesn't handle the accents, for exemple in French : É is sorted after Z, which is not true.
Describe the solution you'd like
Sorting methods should consider accented characters and non-accented characters as the same.
Describe alternatives you've considered
I bypassed this method and created my own method directly in JS file but I guess it should be included in the source code.

submit a pull request

I think that could easily be done by passing Intl.Collator().compare as sortMode. Docs: https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Intl/Collator/compare
It's supported by all modern browsers: https://caniuse.com/?search=Intl.Collator.compare
We could also add it as an internal feature, as addition to "natural", e.g. by passing international as sortMode.
Comparison:
['1', '10', '2', '20', '3', 'a', 'ä', 'e', 'é', 'i', 'o', 'ö', 'u', 'ú', 'ü',].sort()
['1', '10', '2', '20', '3', 'a', 'e', 'i', 'o', 'u', 'ä', 'é', 'ö', 'ú', 'ü']

['1', '10', '2', '20', '3', 'a', 'ä', 'e', 'é', 'i', 'o', 'ö', 'u', 'ú', 'ü',].sort(w2utils.naturalCompare)
['1', '2', '3', '10', '20', 'a', 'e', 'i', 'o', 'u', 'ä', 'é', 'ö', 'ú', 'ü']

['1', '10', '2', '20', '3', 'a', 'ä', 'e', 'é', 'i', 'o', 'ö', 'u', 'ú', 'ü',].sort(Intl.Collator().compare)
['1', '10', '2', '20', '3', 'a', 'ä', 'e', 'é', 'i', 'o', 'ö', 'u', 'ú', 'ü']

However, combining natural sort and international sort would be a whole different story.

Closing issue: implemented in PR #2111


cherrypickapproved: Add option to approve PRs without lgtm or approved labels

This PR extends the cherrypickapproved plugin with two new configuration options: allow_missing_approved_label and allow_missing_lgtm_label. Those two fields are bools defaulted to false. If the option is set to true, the respective label will not be needed to successfully cherry-pick approve the PR.
This is very useful for projects where approvers are also cherry-pick-approvers. At the moment, this plugin is not much useful to those projects, because you either have to approve the PR two times or manually remove the cherry-pick-not-approved label. This change should make the user experience a bit better for this use case.
/assign @saschagrunert @cpanato @puerco
cc @kubernetes-sigs/release-engineering

/approve
/hold
Holding for the two required_reviewers listed in the OWNERS file:

@MadhavJivrajani
@Priyankasaggu11929

@xmudrii, I have a few questions (please correct me if I'm misunderstanding this plugin):
The cherrypickapproved plugin (before this PR) requires the approved / lgtm labels from code/component OWNERS. Once these labels are in place, and a cherry-pick-approvers approves the PR, the cherry-pick-not-approved label is removed, and cherry-pick-approved is applied.
With the new config options (allow_missing_approved_label & allow_missing_lgtm_label) set to True:

Where is the check to ensure that the person approving the cherry-pick is both (i) listed as a cherry-pick-approvers and (ii) a code OWNERS for the repo/path? Currently, we only check if they are in cherry-pick-approvers (here), not a code/component OWNERS.

Does this mean that with these config options enabled, any cherry-pick-approvers can apply the cherry-pick-approved label (and remove cherry-pick-not-approved) regardless of their OWNERS status?

At the moment, this plugin is not much useful to those projects, because you either have to approve the PR two times or manually remove the cherry-pick-not-approved label.

I'm not sure how easy/ hard this is to implement, but an alternative could be to check if the person approving the cherry-pick is listed both in OWNERS and cherry-pick-approvers. If so, all labels (lgtm, approved, cherry-pick-approved) would be applied (and cherry-pick-unapproved removed) instead of skipping the lgtm and approved labels check.

@Priyankasaggu11929 Thank you for the review! I'll do my best to answer all the concerns.

The cherrypickapproved plugin (before this PR) requires the approved / lgtm labels from code/component OWNERS. Once these labels are in place, and a cherry-pick-approvers approves the PR, the cherry-pick-not-approved label is removed, and cherry-pick-approved is applied.

Yes, that's correct.

With the new config options (allow_missing_approved_label & allow_missing_lgtm_label) set to True:
Where is the check to ensure that the person approving the cherry-pick is both (i) listed as a cherry-pick-approvers and (ii) a code OWNERS for the repo/path? Currently, we only check if they are in cherry-pick-approvers (here), not a code/component OWNERS.
Does this mean that with these config options enabled, any cherry-pick-approvers can apply the cherry-pick-approved label (and remove cherry-pick-not-approved) regardless of their OWNERS status?

Yes, that's correct and I have done it intentionally this way. I've been long thinking if there should be some check of the OWNERS status, but then I realized that it might not fit different situations.
For example, let's say that I'm responsible for approving all cherry-picks, including those cherry-picks for which I'm not code/component OWNER. I could still approve the cherry-pick from the "administrative side" (e.g. it satisfies the cherry-pick criteria), but leave it to the code OWNER to review the PR and approve/lgtm it later. Because of that, even if we would implement check of the OWNERS status, I would still make it optional.
This combination as-is (both options set to True) allows both the case that I described in the PR description and this use case explained in this comment. If this is risky for some organizations, they can always leave allow_missing_approved_label to false until we don't come up with a "better" implementation.
Additionally, it's important to clarify two things. This doesn't have any impact to the Kubernetes projects, as we'll not use this option. This change is being made for other Prow users that have similar processes in place as Kubernetes, but yet a little bit different, making these options (allow_missing_approved_label & allow_missing_lgtm_label) required.
The second thing is even if I'm not a code OWNER, but cherry-pick approve the PR, it's only going to affect the cherry-pick-unapproved/cherry-pick-approved label. The PR is not going to get merged until an OWNER doesn't lgtm/approve the PR.
Finally, I'm not really sure how we could implement this case, it potentially requires a lot of work as I'm not sure if that part of the code is reusable from the approvers plugin or we need to reimplement it from scratch. There are many different cases as well, including handling OWNERS_ALIASES, teams, etc...

Is there an example to see how Prow currently reacts, when both approve & cherrypickapproved plugins are configured for a repo & someone listed as both cherry-pick-approvers and appropriate code OWNERS adds the approve/lgtm labels?

cherrypickapproved plugin does nothing in that case. I don't have an example in the Kubernetes org at the moment, but here are some examples from other org with this setup: https://github.com/kubermatic/kubeone/pull/3326, https://github.com/kubermatic/kubeone/pull/3335, https://github.com/kubermatic/kubeone/pull/3264
In the first two examples, I resorted to applying the label manually. In the last example, I tried approving the PR again which also did the trick.

Unblocking the merge, since the config options are not to be used by Kubernetes project repos and doesn't change default behavior.

Can I remove the hold from this PR or should I leave it for @MadhavJivrajani to take a look?

cherrypickapproved plugin does nothing in that case. I don't have an example in the Kubernetes org at the moment, but here are some examples from other org with this setup: https://github.com/kubermatic/kubeone/pull/3326, https://github.com/kubermatic/kubeone/pull/3335, https://github.com/kubermatic/kubeone/pull/3264
In the first two examples, I resorted to applying the label manually. In the last example, I tried approving the PR again which also did the trick.

Thanks for the example links, @xmudrii.
yea, good to unhold from my end.

Thank you, @Priyankasaggu11929!
/hold cancel

/retest


bottom menu

Hi.
Are there provide bottom menu?
I am going to develop a mobile web app, but i don't familiar with front end.
could you give a simple mobile web front end structure example?

Hi,
You can use our documententation as a starting point.
But for how-to questions and other non-issues, please use StackOverflow instead of Github issues.


Database Routes

write the routes to get, post, put, update and delete data obtained from the API in the database

Works correctly in our http://localhost:3000/ but some git merges (Including Wishlist Routing that was updated in merge #65) are not updating to Railway deployed site, even after redeploying railway site.


how to check if id is valid

is there any function, method w/e to check if the input youtube-id is valid?

When you call getInfo for a video id that is not exists, it will throw an exception:
try {
	$youtube = new YoutubeDownloader('VIDEO-ID-HERE');
	$result = $youtube->getInfo();
} catch (YoutubeException $e) {
	echo $e->getMessage();
}


Improved Testing

Added new tests to cover Vitessce components and configuration.

Great that individual vitessce components are being tested. Currently, the last 2 tests are failing, which I think is due to the fetch function. I'll comment these out temporarily and we can pick this up separately. Just waiting on other checks to pass before merging.


Use object parameters for functions

adapter.getProps & adapter.render changed to object params
rename scene.getLayers to scene.getScene

Pull Request Test Coverage Report for Build 747220191

0 of 0   changed or added relevant lines in 0 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 100.0%

Totals

Change from base Build 747159733:
0.0%

Covered Lines:
2

Relevant Lines:
2

💛  - Coveralls


package updates

[x] bump to ocean.js v2.5.1
[ ] Bump to Next.js v13

issue for that web3.js + Next.js problem:
https://github.com/web3/web3.js/issues/5601


feat: ajout du champ contacts dans organismesReferentiel

fix #2791

:tada: This PR is included in version 3.16.0 :tada:
The release is available on:

GitHub release
v3.16.0

Your semantic-release bot :package::rocket: