inject context kwargs to share/log values across LitAPI methods

When trying to share request context across decode, predict and encode method, the current way to do this is not clean and breaks batching.  This PR injects a context that lives across decode, predict, and encode which can be used to log important details that can be passed on from one method to another.
User impact: context_kwargs is an optional argument and users won't be forced to include it while defining a custom LitAPI.
Example, we can track the number of tokens for token usage schema of OpenAISpec:
import time
from typing import Optional

from transformers import pipeline
import litserve as ls
import logging

class GPT2LitAPI(ls.LitAPI):
    def setup(self, device):
        pass

    def predict(self, x, context_kwargs):
        context_kwargs["num_tokens"] = 0
        for text in "this is a generated text".split():
            context_kwargs["num_tokens"] += 1
            yield text + " "

    def encode_response(self, output, context_kwargs):
        for e in super().encode_response(output):
           if context_kwargs["num_tokens"] > num_max_tokens:
                context_kwargs["finish_reason"] = "length"
                return
           usage = {"num_tokens": context_kwargs["num_tokens"]}
           # we can encode and return usage here
            yield e

if __name__ == '__main__':
    api = GPT2LitAPI()
    server = ls.LitServer(api, accelerator='auto', spec=ls.OpenAISpec())
    server.run(port=8000)

@aniketmaurya start the PR with a clear user story and why this is needed (now).
"When trying to do XYZ, this is not possible/had issue/whatever. this thing we are adding will enable XYZ and it's important because ABC."

However the caveat is with unhappy paths: can you show what happens when a user passes two arguments to predict?

class SomeAPI(LitAPI):
    def predict(x, y):
        ...

Right now, we pass the decoded output to predict as it is, we don't differentiate if decode returns list or dict.

I'm wondering, can't we just have any kwargs after the first positional argument be context, instead of injecting?

If we unroll the kwargs, the objects like integer would be passed as a value instead of reference and any change in one method would not reflect in the other.
will add tests soon, wanted to get first round of feedback first :D !

Right now, we pass the decoded output to predict as it is, we don't differentiate if decode returns list or dict.

Right, I'm referring to the case where we pass 2 arguments to predict, not a list. Try to do that on this PR and look at the stack trace and error message. Is it still clear or has it been affected by the injection magic?

@lantiga added a test to make sure that correct error is shown to users in case of unhappy path -
class SomeAPI(LitAPI):
    def predict(x, y):
        ...


Remove Balsamiq.Wireframes version 4.6.1

URL is 404
Microsoft Reviewers: Open in CodeFlow

Old url that 404'ed was: https://builds.balsamiq.com/bwd/Balsamiq_Wireframes_4.6.1_x64_Setup.exe
Url for new version: https://builds.balsamiq.com/bwd/Balsamiq_Wireframes_4.7.2_x64_Setup.exe
Isn't it a bit drastic to blindly(?) remove the whole app from winget-pkgs just because the url 404'ed?


Add an shortcut information for recognize the contents existed or not on the list

Each Characters meaning like below,
E-Exporter
I-Exporter(IaC by Helm chart)
A-Alert-rule
G-Grafana dashboard

I think actually E-Exporter is no need anymore, because it is default contents :)
Please check this out @Joylish
Thanks :)


Bump oidc authenticator version

Purpose
$subject

PR builder started
Link: https://github.com/wso2/product-is/actions/runs/5876686105

PR builder completed
Link: https://github.com/wso2/product-is/actions/runs/5876686105
Status: success


Clear Dates Button doesn't clear partially entered or erroneous date

If you type in the same date or a partial date the clear dates button will not clear the data from the field.

Also it doesn't matter if it is start date or end date.

Seems like the solution would be to lift this.state.dateString out of <DateInput/> and into <DateRangePickerInputController/> and <SingleDatePicker/> and instead pass it down as a prop, and clear out that value when clearDates() or clearDate() is called. This issue is that there's no way for <DateInput/> to know that a "clear values" action just occurred. @majapw WDYT?

Any update on this issue?


filetype: pacman related tools' config files are not recognized

paru.conf use same format of pacman.conf
makepkg use bash syntax, https://man.archlinux.org/man/makepkg.conf.5.en Although man makepkg.conf doesn't mention it, e.g.,
PACKAGER="$DEBFULLNAME <$EMAIL>"
OPTIONS+=(strip zipman purge)

can work, add space before = will not.

Included to #14505


Error making a socket connection when I try with ipv6.

Is Libgdx support IPV6 when making a socket connection? I got an error when i do. Please give me a hand.
com.badlogic.gdx.utils.GdxRuntimeException: Error making a socket connection to my_version_6_ip:my_port
	at com.badlogic.gdx.net.NetJavaSocketImpl.<init>(NetJavaSocketImpl.java:51)
	at com.badlogic.gdx.backends.iosrobovm.IOSNet.newClientSocket(IOSNet.java:63)
	at vn.me.engine.network.MobileClient.doConnect(MobileClient.java:155)
	at vn.me.engine.network.MobileClient.access$3(MobileClient.java:153)
	at vn.me.engine.network.MobileClient$1.run(MobileClient.java:122)
	at java.lang.Thread.run(Thread.java:837)
Caused by: java.net.UnknownHostException: Host is unresolved: my_version_6_ip
	at java.net.Socket.connect(Socket.java:826)
	at java.net.Socket.connect(Socket.java:786)
	at com.badlogic.gdx.net.NetJavaSocketImpl.<init>(NetJavaSocketImpl.java:48)

I got this on:

[x] iOS (robovm)
[x] Windows
[x] MacOS

It seems you don't provide a correct IPv6 Address and the port should be an int, too.

Please use the issue template. Feel free to reopen when you have submitted a code example that reproduces your issue.


Support zero queue consumer for partitioned topic.

Is your feature request related to a problem? Please describe.
Currently, the zero queue consumer just can subscribe to a non-partitioned topic. In some case, we need to use zero queue consumer to subscribe to a partitioned topic.

It’s not enabled because we cannot know which partition the next message will be coming from. Any suggestion on how to achieve that?

Hello, I am planning to work on this issue. I am planning to read how zero queue consumer is subscribed to a nonpartitioned topic and proceed from there by getting some kind of input for which partition this consumer should be subscribed to and so on. Can you tell me where to find the code related to this?

@sundar-10 You can check the ZeroQueueConsumerImpl.

@merlimat How about syncing the backlog between the broker and client periodically. Treat topics with more backlogs as a high priority. Since we don't know if a topic will there be any messages written, to avoid a consumer block to one topic(no more data are written), we can reset the permits in a sync cycle if there are topics that have more backlog, looks like

reset permits to 0
re-order the consumers
trigger read data from the high priority consumer

Users can achieve the desired purpose by setting the frequency of synchronization.
We can optimize the sync mechanism, e.g. carry the backlog when dispatching messages. Only if the message is not dispatched for a period of time, perform a separate sync command.


test: set e2e config correctly

ssia

Current dependencies on/for this PR:

main

PR #173

PR #174   (this PR)

PR #171

This comment was auto-generated by Graphite.


Proposal to import dc:publisher

To give a sound based example, BioAcosutica include recordings made by an individual who is the legal owner, but BioAcoustica is the primary source by which that recording is made available to the public. In this case dc:publisher seems to be something worth adding explicitly to AudubonCore.

An analogous situation involving images might be images submitted to iNaturalist where the submitter is an individual with no web presence and iNaturalist is likely to be the only place where the images would be maintained for access.

We publish a lot of third-party images for the first time in VicFlora as well.
We should have a think – and add some notes – about the difference between this and ac:provider.

To take a couple of real examples from bio.acousti.ca:
In the first we have copies of recordings that were originally published alongside a book on the publisher's website. In this case that publisher is dc:publisher and bio.acousti.ca is ac:provider?
In the second bio.acousti.ca has recordings of mine that I have not otherwise published. In this case bio.acousti.ca is both dc:publisher and dc:provider?

At the Maintenence Group meeting on 22/08/2024 it was decided to work through this and related issues by developing some 'recipes for using rights' in the document here: https://docs.google.com/document/d/1Nv0rNWioJIX-DJRxQExmd22BmaU48qdRYQSzRzdyMQU/edit?usp=sharing.
Please feel free to add usage examples to the document for review at the next meeting.


Docker-compose ERROR

Dear Sir
I hope you are fine.
During development process with docke-compose dev file an ERROR exist as below:
ready - started server on <IP_ADDRESS>:3000, url: http://localhost:3000
2022-09-05T06:56:48.788707413Z npm ERR! path /src
2022-09-05T06:56:48.790430807Z npm ERR! command failed
2022-09-05T06:56:48.790841437Z npm ERR! signal SIGBUS
2022-09-05T06:56:48.790961420Z npm ERR! command sh -c -- next dev

Can you specify what command you ran? Also can you check if you built the images before running the containers?
I am not able to replicate this issue


Adds job satisfiability according to data obtained from cluster monitoring services

Includes tests of monitoring functionality and accounts for dependencies and job state when evaluating job satisfiability.

Prefer pure to return.

@futufeld I've rebased this onto master and added a quality commit over here. Do you want to have a quick look?

Merged after rebasing.


Disable two lint rules which are being removed.

Each of these are going to be removed soon.

avoid_null_checks_in_equality_operators
unsafe_html

Work towards https://github.com/dart-lang/linter/issues/5063 and https://github.com/dart-lang/linter/issues/5001

@Goddchen please review. This PR is blocking the removal of these lint rules in the Dart SDK, as flutter-rfb is in Flutter's customer test registry.


Add Windows Server 2019/RS5 to hello-world

library/hello-world currently support Windows Server 2016, 1709, and 1803.
Windows Server 2019 aka RS5 was (re-)released in November. The build number to reference is 10.0.17763.1
The same build number will enable support for the semi-annual channel Windows Server 1809 release.

See also my recent comment on https://github.com/docker-library/golang/pull/257#issuecomment-449772799. :disappointed:

Manually verified on Server 2019 10.0.17763.107


Wrong public from private key extraction logic

Description
Address generated from given private key is not correct!
Steps to reproduce
Generate a new account and try to import generated private key in NanoWallet:
> nem --chain testnet account vanity --no-estimate TABC TACB TBAC TBCA TCAB TCBA
Address: TABCSO-OLWKHI-VJNZAN-TAFLN7-TM4RNG-MPLBKJ-JAM3
Public key: ab0a9835f6b01cb6cbce6ea1b1c3c54a4bf6e5b9684d40c1f3dc6e329b82f3a6
Private key: c06faacfd991564a5f2e1910dffbdeb464b9fa06d83eec5a96c9708e93d7aab9

Expected behavior
Receive the similar address in NanoWallet - TABCSO-OLWKHI-VJNZAN-TAFLN7-TM4RNG-MPLBKJ-JAM3
Actual behavior

Library version
v0.2
Go version (e.g. go version)
go version go1.9 windows/amd64
OS version (e.g. uname -a)
MINGW64_NT-6.1 dulanov-PC 2.9.0(0.318/5/3) 2017-09-13 23:16 x86_64 Msys

After a quick research I found that a problem with ed25519 go library, because the returned public key is wrong:
prv, _ := hex.DecodeString("2c52aee96f0e30f21c86b3fab7a18e927f579618818e8148e7ded1e01875ef0b")
pub, _, _ := ed25519.GenerateKey(bytes.NewReader(prv))
exp, _ := hex.DecodeString("9d1e9d01ab916dbdde0e76ba43df2246575d637db0bca090f46c1abce19a43e3")
assert.Equal(t, pub, KeyPair{Public: exp})

Fixed in #86ada5ae92697ede13fc2f622a63cbfe366b6f5d


Redesign Supercharge screen

Description

Other changes
Removed some unused feature flags.
Tested
Manually and with unit tests.
How others should test

Open the screen while not verified, check that there's a bullet point prompting you to verify.
Open the screen with no balance, check that there's a bullet point prompting you to add a balance.
Open the screen with a verified number and more than 10 cUSD, check that you see the 'Supercharging' text.

Related issues

Part of #2017

Backwards compatibility
N/A

One question about releasing this, I see it already mentions the claim part, but that part is still being worked on.
When do we plan to release this change? Given next cut is Wednesday 23rd.

Working on it, should be able to open a PR today but we are indeed very close to the deadline

Hey @MuckT, We have verified the above task on Latest Android Internal Release Build & Test flight Release Build V1.27.0 and updated status in Test rails.
Test rail links:
Android 10: https://valoraapp.testrail.io/index.php?/tests/view/12925
iOS 14: https://valoraapp.testrail.io/index.php?/tests/view/12987
iOS 15: https://valoraapp.testrail.io/index.php?/tests/view/13018
Devices: iPhone 13 mini(15.1.1), iPhone 12 (14.7.1), Samsung Galaxy M40 (10.0)
cc: @gnardini
Thanks..!!


Added direct lvm recommendation

What are your changes?
Added direct lvm recommendation
What type of changes does your doc introduce?

[ ] Bug fix
[x] New feature

What is the urgency level of this change?

[ ] Blocker
[ ] High
[x] Medium

I have completed these items:

[x] I have tested all commands and code snippets.
[ ] I have built locally and tested for broken links and formatting.
[x] I have made this change for all affected versions (e.g. 1.7, 1.8, and 1.9).
[x] My doc follows the style guide: https://github.com/dcos/dcos-docs#contributing.

Ping @emanic @sascala @joel-hamill for reviewing pull request changes.
If you included a comment with your commit, it appears here:

@cmaloney @karlkfi Okay, if we don't want to change that's fine. I was working off this internal JIRA from support: https://mesosphere.atlassian.net/browse/DOCS-1052.

@joel-hamill: changing the recommendation in not something that should be taken lightly. We need buy-in from engineering that they are testing this in CI and support that they are willing to support it going forward. Just because Docker recommends it, doesn't mean we need to blindly recommend the same. And if we do want to recommend it then we need proof (manual and automated testing) that it passes all known use cases.
I'm not saying we don't want to change it. I'm saying there is a cost to changing it and we haven't paid it yet or even promised to pay it.
I would like to see buy-in from @spahl & @graham-murphy before we change the recommendations. And when we do, we need to change it everywhere.


[FEATURE] Add checkbox to indicate an event is free.

Is your feature request related to a problem? Please describe.
WebTools recently added a boolean field to indicate if an event is free. We need to add a similar checkbox to User Events.
Describe the solution you'd like
Please add a check box called Free Event to the new/edit event user interface. If selected, this will set a database entry called isEventFree to true. This data item should be sent to Events Buiding Block when creating/updating an event. Please update the UI label Cost to Cost Description.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Epic: #639

@clav1d, Could you please create a follow-up issue to move the "Free Event" label and checkbox above the Cost Description section? I think that may be a more logical position for that control. Thanks.
CC: @bingzhang

@clav1d, Could you please create a follow-up issue to move the "Free Event" label and checkbox above the Cost Description section? I think that may be a more logical position for that control. Thanks.
CC: @bingzhang

Will do.


Add support for Jinja templates

Thanks for your contribution @ashleyblackmore. I've recently been added as a maintainer on this project and am working through the backlog. I just cherry picked the one commit out of this set that actually made a change and pushed it to master. Just a small note: next time please consider rebasing to clean up your branches  when submitting—it makes consideration for inclusion upstream a lot easier.

Will do, thanks.


Zhangya

Description

match recommenders/docs/source/*.rst files with code
Related Issues

Checklist:

[ ] I have followed the contribution guidelines and code style for this project.
[ ] I have added tests covering my contributions.
[ ] I have updated the documentation accordingly.
[ ] This PR is being made to staging branch and not to main branch.

@YanZhangADS  just a small nitpick, sometimes the branch names get confusing so it's better if you can put more descriptive naming in the branch
e.g. zhangya -> zhangya/doc_fix or something like that

@YanZhangADS I think another thing it would be good to update is the readme https://github.com/microsoft/recommenders/tree/main/docs
if you have already reviewed the html files, let me know, and I'll do another pass

@YanZhangADS I think another thing it would be good to update is the readme https://github.com/microsoft/recommenders/tree/main/docs
if you have already reviewed the html files, let me know, and I'll do another pass

Should I check-in all the html files in "build" folder to my branch?
How would you suggest to update the doc? It looks good to me.

@YanZhangADS please let me know when the code is ready to review again, I'll compile and do another pass

also @YanZhangADS , there is a conflict with wikidata, you probably want to merge sooner than later to avoid future conflicts. Let me know if you need my help with this


Attempt at responsive table for entry list using approach outlined

here: https://techblog.livingsocial.com/blog/2015/04/06/responsive-tables-in-pure-css

At widths under 1100px, this hides the <thead> and introduces per-field labels using css to generate content from the data-label attribute from the field. This has the disadvantage that the field labels in the full-size tabular view are links, but in the narrow list view the labels are not links.
Could this be done instead by having a <span> containing per-field labels marked up appropriately, and having them hidden by default but appear when the viewport is narrower than 1100px?

@philandstuff ready for close ups


Attention timer not updating with ball of wool

For the "Needs Attention", the timer does not update if I use a ball of wool on the cat. Stroking the kitten correctly updates the timer though. Thank you for the time format change though! Very nice to save screen space.
I'm on version bca0d078

I think it's due to a typo here in DIALOG_CAT_BALL_OF_WOOL. There should be a space between it and is.
Current: "That kitten loves to play with that ball of wool. I think itis its favourite."
Fixed: "That kitten loves to play with that ball of wool. I think it is its favourite."


Show the dedicated node discount

Description
Make the dedicated node discount clearer for the user
Changes
add icon with warning color
Related Issues
#1892
Documentation PR
https://github.com/threefoldtech/tfgrid-sdk-ts/assets/10484295/91b29ece-a58d-476c-8375-787fad236199
Checklist

[x] Tests included
[ ] Build pass
[ ] Documentation
[ ] Code format and docstrings
[x] Screenshots/Video attached (needed for UI changes)

The discount icon appears on shared node.

Screencast.from.05-21-2<IP_ADDRESS>.PM.webm

fixed @AlaaElattar please check


Magritte: use a AsyncFutureExecutionValueGroup with Magritte value

We have all Magritte related Future Workers in one place (the first pane) with a better option to observe to which fields (widgets) they belong.


vs.nim

For your list of made with Futhark.
https://github.com/mantielero/vs.nim
This is a VapourSynth wrapper made with Futhark for video editing.

Thanks for the contribution, added in: https://github.com/PMunch/futhark/commit/c10283315f99bc7fe276469050ada9dccb340575


Fix typo in mpu.exe parameters binding

Hi!
Here is simple typo and also i found one more mpu.exe bug:
I get 'Process is terminated due to StackOverflowException.' when trying to prepare serializer for something like:
   class Example
   {
       public Example[] Childs;
   }

I know what it is in this function:
https://github.com/msgpack/msgpack-cli/blob/20ae0c36d292c3f6f51248872e362170fcaf36d4/src/MsgPack/Serialization/SerializerGenerator.cs#L498
But it is so tangled.. i gave up trying to refactor it. I would completely remove it and write again.. But i need more time to understand the logic. You possibly see less radical solution.

I suppose build was broken before my changes.

I agree the build failure was caused before your commit. I'll fix it soon. Thank you!


refix calculate offset

#568

r? @adamgreig
(rust-highfive has picked a reviewer for you, use r? to override)


Scrapy outputs blank history.csv file

When running the command scrapy crawl yth_spider -o history.csv -L ERROR the history.csv file outputted is blank.
Cookie was set properly; and unfortunately no error logs are printed to the console.

Bump ^

I am also having this issue, i tried running python3 -m scrapy crawl yth_spider  and the output was pretty normal except for this: 2020-07-29 15:40:05 [scrapy.core.engine] INFO: Spider opened 2020-07-29 15:40:05 [scrapy.extensions.logstats] INFO: Crawled 0 pages (at 0 pages/min), scraped 0 items (at 0 items/min) 2020-07-29 15:40:05 [scrapy.extensions.telnet] INFO: Telnet console listening on <IP_ADDRESS>:6023
I inputted the cookie by just copy-pasting the entirety from the request header, and if I did not do this it would give me a "not logged in on first page" error but when the cookie was correct (I think) it just didn't scrape anything.

I also have the same issue as dwhickox

Hey everyone, seems like there maybe was a change to the page structure or to how the code is handled on Youtube's side.
I will try to give a look at this later this week and see if I can fix this either by simply fixing the changes or by rewriting part of the scraper.

That would be great! I built a visualizer off of this, and just as I was about to release it, this (an obvious key component) broke. No worries though, apis change etc etc, but I would still love to release it sometime soon!
Keep us posted on your progress!

@derectoverso have you got a chance to look into this?
It'd be greatly appreciated if you could fix the changes.

+++

I am also having the same issue. Any update on this?

I have same problem as this project used to solve. And I found simple solution in this article.
As I see from Acrhival Notice YouTube API V3 gives watch history. Also you could just download it using Google Takeout. It's intuitively and there is simple instruction in the article, but I just want to say, that you could choose JSON format for YouTube Watch History and not to parse HTML manually.
Hope it will help someone.

Infortunatly, google takeout shows only history of half of year((. Maybe there is count limit of videos (I watch a few tens of them every day)

I have same problem as this project used to solve. And I found simple solution in this article.
As I see from Acrhival Notice YouTube API V3 gives watch history. Also you could just download it using Google Takeout. It's intuitively and there is simple instruction in the article, but I just want to say, that you could choose JSON format for YouTube Watch History and not to parse HTML manually.
Hope it will help someone.

Infortunatly, google takeout shows only history of half of year((. Maybe there is count limit of videos (I watch a few tens of them every day)

It sounds interesting. For one of my accounts it shows full history, for the secong it shows searches from 2013 and video history from 2016


Non-updating of RM 206/238 weight mean diagram

When I have "auto reject"  and "plot rejected" selected then toggle between 204 and 207 index isotope, or toggle between systems 206Pb/238U and 208Pb/232Th ,the diagram isn't updating with each toggle. The list of points change every toggle, but not the diagram. (see figures below).
Is this error trapping, as there is one analysis with No Age Calculated?
(1) 206Pb/238U weighted mean diagram with "auto reject" , prior to selecting "plot rejected"

(2) 206Pb/238U weighted mean diagram after "plot rejected" selected

(3) 208Pb/232Th weighted mean diagram with both auto reject and plot reject selected

(4) toggle to 206Pb/238U weighted mean diagram with both auto reject and plot reject selected
...no update as we still have the still have 208Pb/232Th

Allanite C36 A46.squid.zip

@AllenKennedy - on first pass, if I remove the offending spot TARA.4.1, all seems well.  So we need to decide what to do in the presence of no age - currently Squid3 is choking.  One solution is to skip the offender or plot a blue dotted line?  Ideas welcome here.

I think a blue dotted line would make the user aware that something is wrong in the data. Maybe others will have an opinion.

@AllenKennedy - Please do for now ...


Support lists when building k8s manifests

This commit modifies the generation of kubernetes manifests so that we
can support lists of resources. As an example, where before the
following jsonnet would fail:
{
  resources: [
    {
      apiVersion: "v1",
      kind: "ConfigMap",
      metadata: {
        name: "config",
      },
    }
  ],
}

We will now return a valid configmap resource.
In addition to this change, we have also made the following adjustments:

Any output in the jsonnet that doesn't look like a Kubernetes object
is ignored, rather than tanka throwing an error
Kubernetes resources are taken to be dictionaries that contain the
keys apiVersion, kind, and metadata.name. This is stricter than we
were before, as we now enforce apiVersion, but this is more in keeping
with the kubernetes API conventions

Whilst making these changes, we've also tidied up the tests for building
manifests. This included extracting common fixtures into a testdata
directory, where they can be represented as much more readable json. We
have also renamed the Reconcile function to be Build, as the term
reconcile is overloaded in the kubernetes context (think controller
reconcile loops) and have simplified the filtering logic.
Depends on #111

Actually, having thought about this, I think it's a really bad idea for us to silently fail if bad manifests are in the jsonnet output. I'm going to adjust walkJSON to fail if this is the case, as it did before.
Other than that, I think the rest of this change is good.

am afraid that most of this will become redundant once #111 is merged

Awesome! Didn't catch this PR before, it's only good news if the work is already done :tada:
Fwiw, I caught a comment in a separate PR about supporting nested lists. This would be pretty important for us, and feels like a surprise if you support lists at the top level. For constructing large kubernetes projects generated from many sparate packages, it's likely to trip them people up if they need use lists anywhere but at the top level.

Fwiw, I caught a comment in a separate PR about supporting nested lists. This would be pretty important for us, and feels like a surprise if you support lists at the top level. For constructing large kubernetes projects generated from many sparate packages, it's likely to trip them people up if they need use lists anywhere but at the top level.
I agree, though this is not straightforward to implement in go, because of the extensive type checking required (we need to switch to []interface{}, do type switching and ensure we catch all edge cases). If you like, you can work on this, I just didn't felt it was worth to do in the initial iteration :D

@sh0rez I've adjusted the walkJSON to support recursive lists, hopefully it doesn't look too bad: https://github.com/grafana/tanka/blob/3356ab29918e3065de6d76a360515b6c6f41b6ea/pkg/kubernetes/walk_json.go
I've also had a peek at the refactor PR. Looking like some good changes in there! I'm happy to wait for that to go in, then we can have a look at how to apply some of these changes on top.

#111 is merged!
I'd suggest closing this one but introduce what is still left in separate PR's:

[ ] Test fixtures as files
[ ] better object recognition in Reconcile (#113)
[ ] rename Reconcile to Extract or ExtractManifests (probably together with  #113)
[ ] add support for nested arrays

Closing this to separate into other PRs.


CPython2.Exe  failed to install

This issue has been moved from a ticket on Developer Community.

[severity:It bothers me. A fix would be nice]
Again, you reported the update error: I have no idea why it was even trying to update Python-yadda-yadda.  Nor do I much care.  I just clicked on "report" or whatever.  Next time I know better.

Original Comments
Feedback Bot on 1/14/2021, 05:26 PM:
We have directed your feedback to the appropriate engineering team for further evaluation. The team will review the feedback and notify you about the next steps.

Original Solutions
(no solutions)

Package 'CPython2.Exe.x64,version=2.7.18,chip=x64' failed to install.
Search URL
https://aka.ms/VSSetupErrorReports?q=PackageId=CPython2.Exe.x64;PackageAction=Install;ReturnCode=1
Details
Command executed: "c:\windows\syswow64\windowspowershell\v1.0\powershell.exe" -NoLogo -NoProfile -Noninteractive -ExecutionPolicy Unrestricted -InputFormat None -Command "& Get-Acl -Path ([Environment]::GetEnvironmentVariable("""ProgramFiles""")) | Set-Acl -Path ([Environment]::GetEnvironmentVariable("""SystemDrive""") + """\Python27amd64""")"
Return code: 1
Return code details: Incorrect function.
Log
C:\Users\rfleischer\AppData\Local\Temp\dd_setup_20210114110818_003_CPython2.Exe.x64.log
Impacted workloads
Python development (Microsoft.VisualStudio.Workload.Python,version=16.0.28621.142)
Impacted components
Python 2 64-bit (2.7.18) (Component.CPython2.x64,version=2.7.18)

Package 'CPython2.Exe.x64,version=2.7.18,chip=x64' failed to install.
Search URL
https://aka.ms/VSSetupErrorReports?q=PackageId=CPython2.Exe.x64;PackageAction=Install;ReturnCode=1
Details
Command executed: "c:\windows\syswow64\windowspowershell\v1.0\powershell.exe" -NoLogo -NoProfile -Noninteractive -ExecutionPolicy Unrestricted -InputFormat None -Command "& Get-Acl -Path ([Environment]::GetEnvironmentVariable("""ProgramFiles""")) | Set-Acl -Path ([Environment]::GetEnvironmentVariable("""SystemDrive""") + """\Python27amd64""")"
Return code: 1
Return code details: Incorrect function.
Log
C:\Users\rfleischer\AppData\Local\Temp\dd_setup_20210114110818_003_CPython2.Exe.x64.log
Impacted workloads
Python development (Microsoft.VisualStudio.Workload.Python,version=16.0.28621.142)
Impacted components
Python 2 64-bit (2.7.18) (Component.CPython2.x64,version=2.7.18)

Error: Set-Acl : Cannot find path 'C:\Python27amd64' because it does not exist.


Idea

We can load childProcess like this
import createBenchEvent from "./event";
import { isESM, isNode, taskIdFromEnv } from "./utils";
import Bench from "./bench";
import Task from "./task";

let childProcess = null;

export async function runner (this: Bench) {
    this.dispatchEvent(createBenchEvent('start'));

    const taskId = taskIdFromEnv();
    if (taskId !== -1) {
        const task = this.getTask(taskId);
        if (task) {
            await task.run();
            return [task];
        }
        return [];
    }

    const values: Task[] = [];
    for (const task of [...this._tasks.values()]) {
        if (isNode) {
            console.log(isESM);
            // @ts-ignore
            childProcess ??= isESM ? await import('node:child_process') : require('node:child_process');
            const result = childProcess.execSync(`TINYBENCH_TASK_ID=${task.id} ${process.execPath}`, { env: process.env });
            console.log(result.toString())
        }

        if (this.signal?.aborted) values.push(task);
        else values.push(await task.run());
    }
    this.dispatchEvent(createBenchEvent('complete'));
    return values;
}

I cant get it to work properly. :D
Maybe somebody else has to take over.

Do you consider the possibility that tasks can be added in a different order at each launch?
so the id will be different on each run

Any summary on this PR?

@Uzlopak @sirenkovladd: I do not see the added value of having a todo primitive. I will remove it.

See #120


feat: Implement using Playwright for taking screenshots in reports

SUMMARY
Implements SIP-98.
If a feature flag PLAYWRIGHT_REPORTS_AND_THUMBNAILS is on, use Playwright instead of Selenium to take screenshots of dashboards and charts when preparing reports and thumbnails.
BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF
The change should be visible when running a report on a dashboard that contains deck.gl charts.
Before:

After:

TESTING INSTRUCTIONS

Enable PLAYWRIGHT_REPORTS_AND_THUMBNAILS ff
Setup reports (https://superset.apache.org/docs/installation/alerts-reports/)
Create a report and verify that the screenshot was taken correctly. If possible, also verify using deck.gl charts
Generate thumbnails (https://superset.apache.org/docs/installation/cache/#caching-thumbnails)
Verify that thumbnails screenshots were taken correctly (including deck.gl charts)
If the feature flag is disabled, the screenshots should be taken using Selenium - no changes.

ADDITIONAL INFORMATION

[x] Has associated issue: https://github.com/apache/superset/issues/24948
[x] Required feature flags: PLAYWRIGHT_REPORTS_AND_THUMBNAILS
[ ] Changes UI
[ ] Includes DB Migration (follow approval process in SIP-59)

[ ] Migration is atomic, supports rollback & is backwards-compatible
[ ] Confirm DB migration upgrade and downgrade tested
[ ] Runtime estimates and downtime expectations provided

[ ] Introduces new feature or API
[ ] Removes existing feature or API

/testenv up FEATURE_PLAYWRIGHT_REPORTS_AND_THUMBNAILS=true

Thank you @villebro @eschutho for your comments!

@kgabryje I left a comment on a part that may break existing functionality. I'd suggest let's see if we can fix forward.


tests: Add pytest and nushell based tests

tests: Prune testcloud cache on each run
Because it's not a cache, it's a copy (there's no cache invalidation,
GC, etc.)
Signed-off-by: Colin Walters<EMAIL_ADDRESS>
dockerignore: Exclude more things
This avoids doing a container rebuild when touching the tmt
bits for example.
Signed-off-by: Colin Walters<EMAIL_ADDRESS>
tests: Add pytest and nushell based tests
I've been trying to keep this project in "one" programming
language by writing even tests in Rust...but specifically
for our integration tests it's pretty painful not just to
compile them but have to deal with baking them into the base image.
The tmt framework is very GHA like in that it scrapes the
git source tree and copies it into the target environment, which
works really well with scripts.
Now, if you know me you know I am not a fan of dynamic programming
languages like bash and Python. I'm one of those folks that actually
tries to use Rust for things that feel like "scripts" i.e. they're
mostly about forking external processes (see the xtask/
crate which uses "xshell").
Some of our testing code is in Rust too. However...there's a giant
tension here because:

Iteration speed is very important for tests and scripts
The artifact being an architecture-dependent binary pushes us
to inject it into container images; having the binary part
of the bootc image under test conceptually forces us to reprovision
for each test change, which is super expensive

Most other people when faced with the testing challenge would
just write shell scripts (or Python); that's definitely what tmt
expects people to do.
The podman project has a mix of a "bats" suite which is all
bash based, and a Go-based framework.
The thing is: bash is easy to mess up and has very little ability
to do static analysis. Go (and Python) are very verbose for forking external
processes.
I've been using https://www.nushell.sh/ for my interactive shell
for quite a while; I know just enough to get by day to day
(but honestly sometimes I still type "bash" and run a few things there
that I know how to express in bash but not nu)
Anyways though, nushell has a lot of desirable properties for
tests (which are basically scripts):

Architecture independent
Running an external process requires zero ceremony; it's the
default!
But it is easy to e.g. scrape JSON from an external binary
into a rich data structure
A decently rich standard library

The downside is, it's a new language. And in the end, I'm
not going to say it's the only way to write tests...maybe we
do end up with some more bash. It wouldn't be the end of the world.
But...after playing with this, I definitely like the result.
OK, and after some debate we decided to add Python too, so this
demos a pytest test.
Signed-off-by: Colin Walters<EMAIL_ADDRESS>
Closes: https://github.com/containers/bootc/issues/543

OK I added infrastructure sufficient to run pytest too...not yet dropping the nushell stuff since I'm in a bit of a "sunk cost fallacy mode" 😄 and don't feel like rewriting the larger test I did in the other PR for now...but, we'll see how things go. Maybe we can drop it if it isn't holding its weight later.

Thank you and indeed, please do not drop nushell just on my feedback, I would love to hear more voices here and I personally would not mind trying something new! nushell does sound really interesting! So maybe the dual approach is fine for now (but maybe now I'm also overcompensating for pushing back earlier…).

I like the nushell so much as I'm a big fan of shell. nutshell was one of my RHEL for Edge test tool candidates. But I finally choose the ansible to run interactive tests because I need to run test out of VM and tests need to be run in RHEL, CS, and Fedora.

I like the nushell so much as I'm a big fan of shell. nutshell was one of my RHEL for Edge test tool candidates.

Ah, interesting to hear!

But I finally choose the ansible to run interactive tests because I need to run test out of VM and tests need to be run in RHEL, CS, and Fedora.

To be clear, Ansible I think should definitely continue to play a role I think. What we're doing with it in provisioning is OK; but for the common use case of "mostly run arbitrary commands, potentially scraping output into JSON" it's also very verbose and in a lot of those cases is really arbitrary code masquerading as declarative YAML.
I would say for sure though we should continue to have Ansible be a gating test on the Fedora-bootc derivative integration testing and releasing.


Check the guide document for bad hyperlinks or references to obsolete documents

Although a general revision of the AC documents is not within the scope of bringing them into conformance with the Standards Documentation Specification, I noticed some places in the guide document where there were references to obsolete documents.  For example, see the first paragraph of Section 9 which refers to the RDF document that is no longer relevant.

OK, I have made a pass at this.  We may discover additional problems in future proofreading, but it should be good enough to allow for merging the documentation-conversion branch


5.0 breaking changes

Summary
Ongoing 5.0 branch that contains breaking changes. This should be merged after the coming release so we have a whole release cycle to track down any possible bugs
Contains:

[x] #2995
[x] #3039
[x] #2928

Add any PRs to the above that should be in the 5.0 release
Release notes
Updates Stan math to 5.0
Checklist

[x] Copyright holder: Simon's Foundation, Daniel Lee, Sean Pinkney
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)

[x] the basic tests are passing

unit tests pass (to run, use: ./runTests.py test/unit)
header checks pass, (make test-headers)
dependencies checks pass, (make test-math-dependencies)
docs build, (make doxygen)
code passes the built in C++ standards checks (make cpplint)

[x] the code is written in idiomatic C++ and changes are documented in the doxygen

[x] the new changes are tested

@syclik should we also remove the cdf_log etc functions?

@syclik should we also remove the cdf_log etc functions?

Definitely! Let me see what's left on the 5.0 branch.

(I think the answer is all of them are still on the branch.)

@syclik the release is finished. Should we merge this once all the tests pass?

Yes! We should!

Cool! Since I opened the PR can you approve?

@SteveBronder, I'm going to merge the PR now just so we can get it flowing through the automatic testing on develop.


Change modules visibility

The goal of this pull request is to hide some modules in the library. To do so, the module test_utilities has been moved to a separate crate (because it should not be public, but the replicator and the printer use it).
Due to the saving progress interface introduced in #69, the cdc_types module should be made public. I am going to add documentation to this module in #90.

@piodul done, I think it's ready to merge.


⚠ SceneTime has degraded performance

In 061bb85, SceneTime ($ST) experienced degraded performance:

HTTP code: 200
Response time: 1296 ms

Resolved: SceneTime performance has improved in 2d5e033 after 6 minutes.


Taskbar shows . dot on XFCE

Suggest taskbar item in XFCE shows title of application eg. SystemD Manager
Currently is shows just a dot.

The window name is also shown as . when using i3.

If anyone knows what causes this, that would be great. The title of the application should be set in the glade file, but I'm not able to get glade to fix this problem.


Include the line number of the class declaration in binding keys

Previously, we were using only the class name and filename as the binding key, which caused collisions if the same name was used to declare more than one (local) class in the same file (as in test_registry_attrs.py).
In the process, this simplifies the process of getting the stack frames: we had been retrieving the same frames (redundantly) in two different ways in the process of defining a field, but the potential sharing was somewhat obscured because the code was split into very short functions.
Fixes #43.

I think we should just fix that test - it seems like a bug anyways to have two classes with the same name.

Having two classes with the same name isn't a bug: each class is defined within the scope of a function, so they really are distinct classes (with different class IDs), and they don't collide.
This also applies to nested classes, in which the inner class might have the same reported function name as the outer class (TestClass vs. TestClass.TestClass). I've added a regression test to illustrate that case.

I'm not sure we need to include the line number, but I suppose it doesn't hurt.

If we could assume that every class with @inject_field annotations is correctly paired with an @inject_define decorator, and that inner classes always have names that are distinct from outer ones, then I think it would suffice to index by class name. But confirming that those assumptions hold (so that we can report violations) seems even more complex than avoiding them in the first place. 🤷♂

Added a minor version bump and a make clean target to clear caches and venv.


skyBox, skyAtmosphere, globe

Three new/updated options for CesiumWidget & Viewer.  skyBox was already an option, but it was impossible to specify the absence of a skyBox with false, so that has been added.  skyAtmosphere is a new option, for those that don't want the Earth's glow.   globe is a new option for those who are currently specifying imageryProvider=false, to disable rendering of the globe.  This last one allows users to place the camera inside of the Earth, looking out, with code like this:
var viewer = new Cesium.Viewer('cesiumContainer', {
    globe: false,
    imageryProvider: false,
    baseLayerPicker: false,
    skyAtmosphere: false,
    mapProjection : new Cesium.GeographicProjection(new Cesium.Ellipsoid(1,1,1))
});
viewer.dataSources.add(Cesium.CzmlDataSource.load('../../SampleData/simple.czml'));

I haven't written tests for this branch yet as there was some debate over its usefulness.  I'll concede that this is not the "right" way for a particular use-case I had in mind, however, I still think there are other cases where these options are needed.  Several users have asked on the mailing list how to turn off skyBox and the answer is to destroy the default one after construction.  Same answer for skyAtmosphere.  Turning off the globe is a valid case for those wishing to place the camera inside (for example, the projection dome that we currently power with a special build of STK, might be replaced by this someday).
Let me know if this might get merged, I'll write the tests.

    skyAtmosphere: false,
    skyBox: false

These are good with me.
    globe: false,

I'm fine with this too, but agree it is not ideal.
I'd write tests and merge if @mramato it is OK with it.

What is the plan here?  If we want this for 1.8, please merge this week.

I'll write the tests.  If I don't get to it in time, it can go in 1.9.

This is finally ready!

Thanks @emackey

As we discussed offline, I was originally against this change mainly because I think it will be really hard to maintain (anywhere that uses scene.globe now has to verify that the globe exists and that's an easy mistake to make).  Is there a reason we chose to not create the globe rather than simply hide it with globe.show = false?  You aren't really losing out on anything memory wise and that function already exists.  It also removes my main issue (that the making scene.globe unavailable).
I also think that our rats nests of incompatible Viewer options arguments needs to be dealt with, this isn't this PRs fault, but I think the better solution is to allow people to specify whatever values they want and we only actually throw if there's a direct incompability.  For instances, BaseLayerPicker without a globe should be no problem if we go with my suggestion above.    This will frustrate users as well.
Thoughts?

I agree the options are getting a bit out of control.  I do want the user to have the option of supplying a non-Earth globe, if they desire.  Also, Scene is not guaranteed to have a globe, and most of its unit tests take place without a globe, so no code should assume that scene.globe is defined.  The ellipsoid can be had from scene.mapProjection.ellipsoid regardless if a globe is available.  I believe it saves some ellipsoid terrain generation work when the globe is off.
I do like the idea of auto-setting the options that need to be set false.  I'll open a separate PR.

Whatever you guys want to do is fine, but if something is being added to or removed from 1.8, please do it ASAP.

I'm still not sold on the idea of the globe going away, but since @pjcozzi was okay with it, I'll deal.  I just think Viewer has way too many constructor options at this point.

I think one solution (definitely not for tomorrow) is to deprecate a bunch of the top-level options and make them sub-objects, I'm not sure why we didn't do that before.  I don't want to start making changes until we are sure though because this is a high-level area of the code and I'd like to avoid churn.


Bump engine version for new insiders in jupyter insiders extension

Last step in the Jupyter release plan. Bump up the engine version in the new insiders releases.

Codecov Report

Merging #11300 (839f86e) into main (587cb9e) will decrease coverage by 0%.
The diff coverage is n/a.

@@          Coverage Diff           @@
##            main   #11300   +/-   ##
======================================
- Coverage     63%      63%   -1%
======================================
  Files        477      477
  Lines      33878    33878
  Branches    5490     5490
======================================
- Hits       21512    21504    -8
- Misses     10311    10318    +7
- Partials    2055     2056    +1

Impacted Files
Coverage Δ

src/kernels/execution/notebookUpdater.ts
88% <0%> (-8%)
:arrow_down:

src/kernels/raw/session/rawJupyterSession.node.ts
79% <0%> (-3%)
:arrow_down:

src/kernels/jupyter/baseKernelConnectionWrapper.ts
35% <0%> (-3%)
:arrow_down:

src/standalone/api/kernelApi.ts
73% <0%> (-2%)
:arrow_down:

src/kernels/kernelProvider.base.ts
95% <0%> (-1%)
:arrow_down:

src/kernels/common/baseJupyterSession.ts
80% <0%> (-1%)
:arrow_down:

...c/kernels/execution/cellExecutionMessageHandler.ts
77% <0%> (-1%)
:arrow_down:

src/platform/common/utils/localize.ts
76% <0%> (+<1%)
:arrow_up:

src/kernels/raw/launcher/kernelProcess.node.ts
73% <0%> (+<1%)
:arrow_up:

src/notebooks/outputs/tracebackFormatter.ts
100% <0%> (+6%)
:arrow_up:


Unable to log in to Gmail or Outlook with avast

This feature was added in Nylas 2.0.18 to fix this issue but currently I don't see it in this fork's 2.1.:

Allow bypassing of invalid certificates during authentication

And because of the lack of it I get: Please contact<EMAIL_ADDRESS>An unforeseen error has occurred.
Are there any related issues?
No
What operating system are you using?
Windows 10
What version of Nylas Mail are you using?
2.1.0
--
Bug? Yes
Do you have any third-party plugins installed? If so, which ones?
None (also because of a bug)
Is the issue related to a specific email provider (Gmail, Exchange, etc.)?
Gmail and outlook
Is the issue reproducible with a particular attachment, message, signature, etc?
Just install avast free and enable mail filter

If this was in a pre existing build, you should see the fix/work around in the upcoming release.

The latest release should hopefully fix this. Let us know if it works https://github.com/nylas-mail-lives/nylas-mail/releases/tag/2.2.0

Yes, this update/rebase fixed it.


Non animated models get "None" appended to their name.

If you try exporting a model without bones, that model will be exported with the name "None" appended to it. This would make sense if there's bone groups, but for models without any it's kinda unecessary.

Fixed in 18f2fb7


Error when using FileSystemWritableFileStreamTarget

The following error:

Occurs when trying to use FileSystemWritableFileStreamTarget with fileWriter.
This write function (in the build code) is expecting a blob not an object

For now I've worked around the issue by wrapping the fileWriter
   const wrapFileWriter = (fileWriter) => {
        const originalWrite = fileWriter.write;
        const modifiedWrite = (payload) => {
            const blob = new Blob([payload.data]);
            originalWrite.call(fileWriter, blob);
        }
        fileWriter.write = modifiedWrite;
        return fileWriter;
    };
    let muxer = new Mp4Muxer.Muxer({
	target: new Mp4Muxer.FileSystemWritableFileStreamTarget(wrapFileWriter(fileWriter)),
        ...
    }

The FileSystemWritableFileStreamTarget expects a FileSystemWritableFileStream as a parameter! Its write method also accepts ArrayBuffers as input.
It seems like you are passing it something else. How are you getting your fileWriter?

@Vanilagy
Aah right, I see! Indeed I'm passing a different object. I'm passing it a FileWriter which is returned from a fileEntry.createWriter directly.
I'm obtaining this fileEntry from the following code
export default function requestTempFile(name, sizeInMb) {
    return new Promise((resolve) => {
        const fileSystem = (window.requestFileSystem || window.webkitRequestFileSystem);
        fileSystem(window.TEMPORARY, sizeInMb * 1024 * 1024, (fs) => {
            fs.root.getFile(name, { create: true }, (fileEntry) => {
                resolve(fileEntry);
            });
        });
    });
}

Which is a non-standard, chromium-specific way of creating a sandboxed file.
It's a slightly different use case, it can likely just be handled by the StreamTarget directly

Yes exactly, these custom cases are supposed to be handled by StreamTarget. It'll be very easy and quick to use! In fact, FileSystemWritableFileStreamTarget is nothing but a very thin wrapper around StreamTarget.

Has your issue been resolved?

I'll close this issue for now as it seems stale, so I assume your issue has been resolved. If you still need help with something, feel free to reopen it :)


[Relay][Frontend][Onnx] Refactor where importer to support dynamic shapes.

The current implementation of Where in our importer uses static shape analysis to do broadcasting. Unfortunately, this can break when inputs have dynamic shapes. This PR refactors the analysis to instead support dynamism.

@soiferj can you take a look at this PR?

I would definitely like to but wasn't sure what the right way to force dynamic shapes in a model is. I think we would need to compose a composite graph of some dynamic op followed by where, which might be a very ugly test. Do you know a better way to do that?

My goto has been do throw in a shape_of op and do some computations on it to get dynamic inputs, even if it's just op(x) -> op(reshape(x, shape_of(x))

Thanks @jwfromm


Absolute paths?

Is your feature request related to a problem? Please describe.
I don't like paths like this:
import { appRouter } from "../../../server/router";
import { createContext } from "../../../server/router/context";

because I have to open my editor's file browser to figure out where appRouter and createContext come from.
Describe the solution you'd like
We can set "baseUrl": "." in tsconfig.json so we can use absolute paths (relative paths still work):
import { appRouter } from "src/server/router";
import { createContext } from "src/server/router/context";

That's much clearer.

Please search for existing issues before making one. Closing this as it has been discussed in #85.


change redis-client library to be cluster compatible

use of vertx-redis-client- 4.0.3 instead of JRedisJSON client library (as JRedisJSON doesn't support clustering).
define and use of JSON.GET  using Command.create

Support for ReJSON in vertx-redis-client
Vertx Redis Client API Docs
Vertx Redis client Clustering mode,
Command Interface
 Redis Command documentation - Do  COMMAND INFO JSON.GET in redis-rejson server to find out the appropriate flags and parameters needed for command.create in vertx-redis-client

Some advantages of Redis clustering:

High performance and linear scalability up to 1000 nodes. The ability to automatically split your dataset among multiple nodes.
Acceptable degree of write safety: the system tries (in a best-effort way) to retain all the writes originating from clients connected with the majority of the master nodes. Usually there are small windows where acknowledged writes can be lost. Windows to lose acknowledged writes are larger when clients are in a minority partition.
Availability: Redis Cluster is able to survive partitions where the majority of the master nodes are reachable and there is at least one reachable slave for every master node that is no longer reachable.

Some Limitations of Redis clustering:

Multi-key operations like Set type unions or intersections or MGET or JSON.MGET are implemented as long as the keys all hash to the same slot. Hashing to same slot can be achieved through hash tags.
Multi-database not supported.

See more at Redis cluster spec
Hopefully vertx-redis-client should be sufficient for querying using Rejson commands and for clustering.
This code was tested against Redis standalone (using redis-rejosn docker ) and cluster mode (in K8s, redis-cluster bitnami chart with Rejson module loaded). In both modes, the latest api call for secure and non-secure resource works.
Please review,  I'm not so good at vertx.

Also Following  Redis commands are not supported in cluster mode, by vertx-redis-client
 ASKING, AUTH, BGREWRITEAOF, BGSAVE, CLIENT, COMMAND, CONFIG,DEBUG, DISCARD, HOST, INFO, LASTSAVE, LATENCY, LOLWUT, MEMORY, MODULE, MONITOR, PFDEBUG, PFSELFTEST, PING, READONLY, READWRITE, REPLCONF, REPLICAOF, ROLE, SAVE, SCAN, SCRIPT, SELECT, SHUTDOWN, SLAVEOF, SLOWLOG, SWAPDB,SYNC, SENTINEL, FLUSHALL


imagemin-pngquant installing problem

When executing npm install --save imagemin-pngquant in powershell of windows 10,it turn up untar error C:\Users\Administrator\AppData\Roaming\npm-cache\imagemin-pngquant\4.2.0\package.tgz,how can i solve?

https://github.com/npm/npm/issues


Specify prescribing month for runner

This addresses #524 .
I don't think it's quite right, but it's a good starting point for discussion.  Specifically, the year and month that we're now passing around everywhere in runner.py refer only to the year and month of the prescribing data.

Code Climate issues can be ignored.


Python imports in baseline container

You've been more than helpful already, so I am sorry to keep bothering, but I have come across one more bug that I just cannot figure out. It only manifested this afternoon -- prior to that, everything was going pretty well, even past this step. The problem is with a python module import when running one of your scripts in the corresponding container. To reproduce it, try this:
$ singularity shell docker://scrnaseqbenchmark/baseline:latest
Singularity> python3
>>> from sklearn import linear_model

I get this error.
/usr/local/lib/python3.5/dist-packages/scipy/__init__.py:115: UserWarning: Numpy 1.13.3 or above is required for this version of scipy (detected version 1.10.1)
  UserWarning)
RuntimeError: module compiled against API version 0xb but this version of numpy is 0xa
Traceback (most recent call last):
  File "rank_gene_dropouts.py", line 8, in <module>
    from sklearn import linear_model
  File "/usr/local/lib/python3.5/dist-packages/sklearn/__init__.py", line 76, in <module>
    from .base import clone
  File "/usr/local/lib/python3.5/dist-packages/sklearn/base.py", line 16, in <module>
    from .utils import _IS_32BIT
  File "/usr/local/lib/python3.5/dist-packages/sklearn/utils/__init__.py", line 13, in <module>
    from scipy.sparse import issparse
  File "/usr/local/lib/python3.5/dist-packages/scipy/sparse/__init__.py", line 230, in <module>
    from .csr import *
  File "/usr/local/lib/python3.5/dist-packages/scipy/sparse/csr.py", line 13, in <module>
    from ._sparsetools import (csr_tocsc, csr_tobsr, csr_count_blocks,
ImportError: numpy.core.multiarray failed to import

I am completely stumped -- running an import command inside of a container should be the definition of reproducible; what could either of use possibly do differently? If you have any ideas, I am happy to send any info you need.

Could you try running this again but using this command to enter the container instead:
singularity shell --containall docker://scrnaseqbenchmark/baseline:latest

Singularity by default mounts some directories of the host machine. This might be the cause of the problem, though I can't say for sure.  Using the --containall flag will disable this behaviour and the container should become fully self-contained (just like it would be if run with docker).

That works! Thank you! When invoking snakemake, I will add that to --singularity-args.


When using doSisuAuth cached token is never returned.

Detailed description of a problem
Currently when using doSisuAuth the verifyTokens function in XboxTokenManager will always return false thus forcing a token refresh on every call, this is because of the follwing snippet of code
const ut = await this.getCachedUserToken()
const xt = await this.getCachedXstsToken()
if (!ut || !xt || this.forceRefresh) {
  return false
}

Sisu auth flow doesn't generate a userToken because userToken, deviceToken and titleToken are all returned in the sisu response however we only cache the xsts token.
Expected behavior
When using doSisuAuth return the cached XSTS token instead of refreshing.
Additional context
Ideally, we need to cache the returned tokens from sisu and have a solution that uses all available tokens in the cache to generate an xsts token after the current one expires as well as change the verifyTokens function to check all tokens

Is this still an issue? I think you mentioned caching more data at some point.

Still an issue, I'll open a pr to fix just this


If a json property starts with a capital letter, it is not included in the deserialized object

Describe the bug
For example, if the price on the offer is "Price", it will not be picked up.
Steps to reproduce
This unit test reproduces the issue:
[Test]
public void GetSchemaOrgOffers_GetOfferInProduct_GetPrice()
{
    // Arrange
    var pageContent = File.ReadAllText("C:/SchemaOrgTestPages/SchemaNetProductExample.json");

    // Act
    var product = SchemaSerializer.DeserializeObject<Product>(pageContent);

    // Assert
    Assert.IsTrue(product.Offers.Value2.Any(o => o.Price.HasValue1 || o.Price.HasValue2));
}

The contents of the SchemaNetProductExample file is:
{
	"@context": "http://schema.org/",
	"@type": "Product",
	"name": "Cucumber Mint Lip Balm",
	"title": "Cucumber Mint Lip Balm",
	"image": "https://www.burtsbees.com/dw/image/v2/AAPA_PRD/on/demandware.static/-/Sites-burtsbees-master-catalog/default/dw19603097/images/large/NI-44070_BBd_LIP_CB_4Pk_CucumberMint_Frnt_13-07-18-1300.jpg?sw=380&sh=380&sm=fit",
	"description": "",
	"brand": "Burt's Bees",
	"sku": "792850905337",
	"gtin": "792850900936",
	"size": "4 Pack",
	"color": "",
	"offers": {
		"@type": "Offer",
		"url": "https://www.burtsbees.com/product/cucumber-mint-lip-balm/VM-792850900936.html",
		"priceCurrency": "USD",
		"Price": "11.99",
		"availability": "http://schema.org/OutOfStock",
		"itemCondition": "http://schema.org/NewCondition"
	}
}

It works fine when the price is lower case and Assert.IsTrue passes. However when price starts with a captial P, the Assert.IsTrue fails as both HasValue1 and HasValue2 are false on the offer.
Expected behaviour
I think it should pick up the price irrelevant of the case, as the following website says it is valid schema.org: https://validator.schema.org/
Schema objects
https://schema.org/Offer

This one I'm more on the fence about - on one hand, you're right, the validator says it is fine. On the other, should we be potentially looking at more broad case insensitive parsing? I mean, while your example is the first letter, maybe we should be every letter of a property. Do we then look at enum values too and parse them case insensitively?
Schema.org doesn't specify any case sensitivity requirements that I could find so maybe the answer is we shouldn't be case sensitive either.
@RehanSaeed - I'm curious what you think when you're available to look at this.

We should make what we accept more open and accept case insensitive JSON as you suggest. We'd happily accept a PR to that effect.


Does this work with FLUX models?

?

no

Is it possible to make it work with it, or does that need to rewrite the code from scratch? @cubiq . I mean you encourage meto go ask for it in the official intant id repo?

it needs the model to be retrained for flux

OK thanks, and do you know other similar techniques for face swapping with FLUX? Per chance


[Package Request]: Password Safe

How can we help?
I would like someone else to build the manifest.
Please read and ensure the following

[X] The installer meets the above requirements

Please provide the following information
Download Page Url: https://sourceforge.net/projects/passwordsafe/files/latest/download – BUT serves 32-bit by default
Publisher: https://pwsafe.org/
Package Name: Password Safe / pwsafe
Description: password manager
Package Version: current V3.64
Installer URL: https://sourceforge.net/projects/passwordsafe/files/Windows/3.64.0/pwsafe64-3.64.0.msi/download
There is a GitHub but it's non-Windows versions only:  https://github.com/pwsafe/pwsafe/releases/latest
Note was in list at https://github.com/microsoft/winget-pkgs/issues/906

This package identifier is already present as RonyShapiro.PasswordSafe, stored in manifests/r/RonyShapiro/PasswordSafe. Current latest version of this package in winget is v3.62.0. I can make help updating the package.
PS> winget search RonyShapiro.PasswordSafe
名称          ID                       版本   源
-----------------------------------------------------
Password Safe RonyShapiro.PasswordSafe 3.62.0 winget


NIFI-9969: Ensured that callback to RemoteQueuePartition do not attem…

…pt to obtain the Partition Read Lock. The Read Lock is not necessary as long as the 'partitioner' is volatile, because it doesn't matter whether or not the actual partitions themselves change, since the only partition that would be touched is the Rebalancing Partition,on, which is fixed. Obtaining the partition read lock can lead to a deadlock as outlined in the Jira description.
Thank you for submitting a contribution to Apache NiFi.
Please provide a short description of the PR here:
Description of PR
Enables X functionality; fixes bug NIFI-YYYY.
In order to streamline the review of the contribution we ask you
to ensure the following steps have been taken:
For all changes:

[ ] Is there a JIRA ticket associated with this PR? Is it referenced
in the commit message?

[ ] Does your PR title start with NIFI-XXXX where XXXX is the JIRA number you are trying to resolve? Pay particular attention to the hyphen "-" character.

[ ] Has your PR been rebased against the latest commit within the target branch (typically master)?

[ ] Is your initial contribution a single, squashed commit? Additional commits in response to PR reviewer feedback should be made on this branch and pushed to allow change tracking. Do not squash or use --force when pushing to allow for clean monitoring of changes.

For code changes:

[ ] Have you ensured that the full suite of tests is executed via mvn -Pcontrib-check clean install at the root nifi folder?
[ ] Have you written or updated unit tests to verify your changes?
[ ] Have you verified that the full build is successful on both JDK 8 and JDK 11?
[ ] If adding new dependencies to the code, are these dependencies licensed in a way that is compatible for inclusion under ASF 2.0?
[ ] If applicable, have you updated the LICENSE file, including the main LICENSE file under nifi-assembly?
[ ] If applicable, have you updated the NOTICE file, including the main NOTICE file found under nifi-assembly?
[ ] If adding new Properties, have you added .displayName in addition to .name (programmatic access) for each of the new properties?

For documentation related changes:

[ ] Have you ensured that format looks appropriate for the output in which it is rendered?

Note:
Please ensure that once the PR is submitted, you check travis-ci for build issues and submit an update to your PR as soon as possible.

Will review


Sensors unavailble Cannot subscribe to topic 'frigate/Garaget/person', MQTT is not enabled

Version of the custom_component

Configuration
mqtt:
host: <IP_ADDRESS>
topic_prefix: frigate
client_id: frigate
user: XXXX
password: xxxxx
port: 1883
stats_interval: 60
cameras:
Garaget:
ffmpeg:
inputs:
- path: rtsp://xxxx@<IP_ADDRESS>:554/stream1
roles:
- detect
record:
enabled: false
detect:
enabled: false
width: 1920
height: 1088
fps: 5
go2rtc:
streams:
back:
#Garaget
- rtsp://xxx@<IP_ADDRESS>:554/stream1
detectors:
coral:
type: edgetpu
device: usb
ffmpeg:
hwaccel_args: preset-vaapi
Add your logs here.
Error adding entity binary_sensor.garaget_person_occupancy for domain binary_sensor with platform frigate
Error adding entity binary_sensor.garaget_all_occupancy for domain binary_sensor with platform frigate
Error adding entity binary_sensor.garaget_motion for domain binary_sensor with platform frigate
Traceback (most recent call last):
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 580, in _async_add_entities
await coro
File "/usr/src/homeassistant/homeassistant/helpers/entity_platform.py", line 890, in _async_add_entity
await entity.add_to_platform_finish()
File "/usr/src/homeassistant/homeassistant/helpers/entity.py", line 1333, in add_to_platform_finish
await self.async_added_to_hass()
File "/config/custom_components/frigate/init.py", line 465, in async_added_to_hass
self._sub_state = await async_subscribe_topics(self.hass, state)
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/mqtt/subscription.py", line 133, in async_subscribe_topics
await sub.subscribe()
File "/usr/src/homeassistant/homeassistant/components/mqtt/subscription.py", line 62, in subscribe
self.unsubscribe_callback = await self.subscribe_task
^^^^^^^^^^^^^^^^^^^^^^^^^
File "/usr/src/homeassistant/homeassistant/components/mqtt/client.py", line 174, in async_subscribe
raise HomeAssistantError(
homeassistant.exceptions.HomeAssistantError: Cannot subscribe to topic 'frigate/Garaget/person', MQTT is not enabled

## Describe the bug
Sensors are unavalible i Home Assistant
I have tried to reinstall the Frigate NVR Custom Component, Frigate and MQTT. Frigate is working. Via MQTT Explorer I can see the there is activity. The only trace I can find is in the log as above. I have a Home Asistant test system with the same set-up and there is everything working fine. I have five cameras. All five works OK in the testsystem. There is no difference if I have one or five cameras in the productionsystem activated.

Best Regards Toni

## Debug log

<!-- To enable debug logs check this https://www.home-assistant.io/components/logger/ -->

```text

Add your logs here.

What are the Frigate logs

Hi, Here the are. I can't find any error related to the integration. The last log is after I have reloaded the integration.
 But the MQTT is up and running and Frigate is working. I can also see activity in MQTT Exploer. So where to enable MQTT??
Toni

inside home assistant

Pls, Where?
Regatds Toni

Found it. Thank you.


error parsing log with pgbadger v11, but works with pgbadger v10.2

Attached is a very small log file that shows pgbadger V11 cannot parse it correctly, but it is parsed correctly with version 10.2.
/apps/opt/postgres/pgbadger-11.1/pgbadger -p '%t:%r:%u@%d:[%p]:' -f stderr /apps/opt/postgres/mikev/pgbadger_test.log -o /apps/opt/postgres/mikev/badger11_t3.html
Unmatched ( in regex; marked by <-- HERE in m/^(?:\d+-\d+-\d+T\d+:\d+:\d+\.\d+Z)?\s*(\d+)-(\d+)-(\d+) (\d+):(\d+):(\d+)\s*[^:]*:( <-- HERE [^\(:]+\(\d+\):([^\@]+)\@([^:]+):\[(\d+)\]:(\d{4}-\d{2}-\d{2} \d{2}:\d{2}:\d{2})(?: [A-Z\+\-\d]{3,6})?:([a-zA-Z0-9\-\.]+|\[local\]|\d{1,3}\.\d{1,3}\.\d{1,3}\.\d{1,3}|[0-9a-fA-F:]+)?[\(\d\)]*:([0-9a-zA-Z\_\[\]\-\.]*)@([0-9a-zA-Z\_\[\]\-\.]*):\[(\d+)\]:(LOG|WARNING|ERROR|FATAL|PANIC|DETAIL|STATEMENT|HINT|CONTEXT|LOCATION):\s+(.*)/ at /apps/opt/postgres/pgbadger-11.1/pgbadger line 2430.

pgbadger_test.log

Hi Michael,
I guess that this issue have been fixed in last development code, at least I can not reproduce it:
$ perl ./pgbadger -p '%t:%r:%u@%d:[%p]:' -f stderr pgbadger_test.log
[========================>] Parsed 8127 bytes of 8127 (100.00%), queries: 2, events: 0
LOG: Ok, generating html report...

Yes, I downloaded the latest version and reran it with no problems.  Cannot account for why the previous downloaded version for 11.1 didn't work since I did not install that version.


Update slider.js

Added string conversion to ternary operator for bubble display.  This fixes the issue of the lower handle value not being displayed, as the comparison was looking at numerical zero vs an empty string, which both evaluate to false, resulting in the second value being returned.  By converting to string, when the value is '0' this is selected over the empty string.

Fix for Issue #23.


Can't wrap variadic tuples into other generic types

TypeScript Version:  4.1.0-dev.20200813
Code
interface Decoder<T> {
    // ...
}

export function decodeTuple<T extends unknown[]>(
    decoders: [...Decoder<T>] // ERROR
): Decoder<[...T]> {
    // ...
}

Expected behavior: This code should work fine
Actual behavior: The decoders argument's type causes an error: A rest element type must be an array type.ts(2574)
Playground Link: https://www.typescriptlang.org/play?ts=4.0.0-beta#code/JYOwLgpgTgZghgYwgAgCIQQewCbQDwAqAfMgN4BQyVyA9DcgHRPkC+55EAHgA6ZRjIYAVxAIwwTCGS4suAkO4AbCIWRdIIbAGdkIgNYhMAdxABtALpEAFJWoyc0LQC5kppg3Sz8xc7XoBRACVAgHlA8gBKF08HKDw3JgJLMlsqOkZmFiA

This is working as intended. A rest element must have an array type, and Decoder<T> isn't. It works if you have Decoder<T> extend Array<T>:
interface Decoder<T> extends Array<T> {
    // ...
}

Indeed, but what I want is to have an array of Decoder<T>, without Decoder<T> being an array itself.


Is there a option to add border to line chart

@gionkunz

Just add a border to your container:
.ct-chart {
  border: 1px solid black;
}

Hi there. I'll close this issue. If you still have more questions, please feel free to comment further.

thanks for ur help
On Wed, Jun 10, 2015 at 12:36 PM, Gion Kunz<EMAIL_ADDRESS>wrote:

Hi there. I'll close this issue. If you still have more questions, please
feel free to comment further.
—
Reply to this email directly or view it on GitHub
https://github.com/gionkunz/chartist-js/issues/339#issuecomment-110624313
.


VS Code snap broken on Ubuntu 24.04

Does this issue occur when all extensions are disabled?: Yes/No

VS Code Version: 1.91.1
OS Version: Ubuntu 24.04

Steps to Reproduce:

Install VS Code as a snap instead of an official .deb
Observe issues running it: https://github.com/microsoft/vscode/issues/212430

All of the suggestions in the above issue point to the snap installation being broken. Since the snap is maintained in this repository, I figured it would be right to have an issue about this.

/duplicate https://github.com/microsoft/vscode/issues/202072


No IMS registration

Hello @herlesupreeth. Hope you doing great
Actually i'm new to the setup using SDR to build a network. So i decided to use the Kamailo (the same link on your github)to build the network and perform a VoLTE call (which is not working yet)and the following have already been compeleted:

i was able to do a successful attach and i'll add the PCAP Here.
To perform the IMS, i downloaded the CoIMS app in the case of samsung and i had to add certificate to the sim so it could be accessed. Initially when i put the SIM, the app was saying "carrier priviledges not allowed" but when certificates was added it got solved. but volte provisionned was still disabled
I also saw one of your comments on a post to use IPHONE because it will not use the coIMS app. i used the iphone and VoLTE was autolmatically activated on the phone but could not see anything in the logs.

Please can you help guide me on this?
Thanks
5_27_2024.zip

Hey @kamgaarnold I see that you had wrong APN configured first and then changed to "F2G" and UE was able to connect to the 4G network, but I dont see UE requesting a PDN connection towards "ims' APN i.e. there should be another PDN connectivity request after the highlighted packet in below screenshot

May I know which UE it is in the trace? iphone or samsung phone?

Hi @herlesupreeth. Thanks for the quick reply. Yes I changed the apn from internet to F2G in both the Phyhss and open5gs
The pcap attach in my first comment is for the samsung s10.
When using apple I had an issue because the pcap log was complain about "unknown PLMN". But the Volte was enable. Besides I was using an iphone8 and couldn't add an APN.
Yes and as you said there is no PDN connectivity for the ims which is where I'm stucked for now.
I can see that all the core éléments are initialized but no PDNConnectivity is coming in for qci-5.
Thankw

Hey @kamgaarnold I see that you had wrong APN configured first and then changed to "F2G" and UE was able to connect to the 4G network, but I dont see UE requesting a PDN connection towards "ims' APN i.e. there should be another PDN connectivity request after the highlighted packet in below screenshot

May I know which UE it is in the trace? iphone or samsung phone?
Hi @herlesupreeth. Thanks for the quick reply. It is a samsung phone. Photo is attached

The pcap attach in my first comment is for the samsung s10.

CoIMS method sadly doesnt work on Samsung device running Android 10 and above. The reason you dont see PDN connectivity request for ims APN is because VoLTE is not enabled on that device.

The pcap attach in my first comment is for the samsung s10.

CoIMS method sadly doesnt work on Samsung device running Android 10 and above. The reason you dont see PDN connectivity request for ims APN is because VoLTE is not enabled on that device.

@herlesupreeth  thanks for letting me know that Android does not support CoIMS and it makes sense also because the VoLTE is not enable as shown on the photo.
Please do you advice any other UE i can use? I tried using a oneplus but it does not recognise my sim.

Hello @herlesupreeth.
Please in order to have the PDN, do we need to masquerade the Sgi interface in the case of docker_5gs?
By the way what UE do you recommend for this setup please.
Thanks

Please do you advice any other UE i can use? I tried using a oneplus but it does not recognise my sim.

You can use any other phone apart from Samsung devices. Just keep note that its better to go with a phone which has Qualcomm chipset in it.

Please in order to have the PDN, do we need to masquerade the Sgi interface in the case of docker_5gs?

I dont think those two are related

Hello @herlesupreeth. please i've been stocked on this IMS issue. Actually i tried today and i see "extended service request" followed by CSFB.
I used the kamailo setup. so i don't know if there is an issue or i'm missing something?
The attach is ok with QCI-9 but for ims i get nothing.
6_19_2024.zip

The attach is ok with QCI-9 with both APN added on the UE but i don't get any PDN for ims hence no sip registration.

sadly its a UE issue and nothing can be done. I would suggest to try with any other phone than Samsung

suggest

Alright. I tried with an iphone 13, but i still get CSFB 🥺

The attach is ok with QCI-9 with both APN added on the UE but i don't get any PDN for ims hence no sip registration.

sadly its a UE issue and nothing can be done. I would suggest to try with any other phone than Samsung

@herlesupreeth I tried with an iphone13 but it still does CSFB 🥺

I tried with an iphone13 but it still does CSFB 🥺 eventhough VoLTE is enabled

please restart the phone and try to attach with iPhone 13 and send me the pcap

Alright. let me try and attach the PCAP.
Thanks

Hurlee.zip
Hello @herlesupreeth. i attached 3 PCAP in the folder.
Thanks. i'll keep checking but still facing CSFB.
I'm not sure if something is missen in my config. If you need something more, i can share with you

I think your APN configuration in open5gs is wrong. You should have internet APN first and then ims APN and i think you have the other way around

@herlesupreeth if you go through the zip file, the 3rd pcap has internet as default. The other 2 you can disregard because I set ims as défault when doing a scénario to see if the UE still attach.
Thanks

hmm.. then I didnt see UE attempting a PDN connection ims APN. Can you reprogram the SIM to 00101 and set MNC and MCC accordingly in env file and give it a try ?

@herlesupreeth ok i'll do that. but the quick question i have is if all my scenario keep doing CSFB, then it probably means that the IMS is not up and then the UE prefers to fall to 3G. Is there something i need to check in any config file concerning the ims?

Aha, I now see ims PDN connectivity request from UE and thats good news. Can you please take the pcap on the entire machine running docker_open5gs when the UE attempts to connect to network. The pcap you took has only S1AP packet and no GTP packets

But there is no dedicated bearer activated so no call will go through.

Dedicated bearer is created only during a VoLTE call and not before

@herlesupreeth I don't get it. It's true we can't see gtp packet. I'm not sure on how to do what you're suggesting. Can you guide a little.
In the enb.conf what line do I need to uncomment to see gtp packet?

Use tcpdump to take trace on the host machine where you are running docker_open5gs (please look up on internet how to do so)

In the enb.conf what line do I need to uncomment to see gtp packet?

Its not a setting in eNB.

Hello @herlesupreeth . i attach the pcap here. i succeeded to see the SIP. but now when i use 2 Devices with 2 IMSI'S i can't perform a call. Is there any work around to perform this?
Thanks
6_28_2024.zip

You have incorrectly provisioned the SCSCF URI in the HSS. Its should be sip:scscf.ims.mnc001.mcc001.3gppnetwork.org:6060
and NOT sip:scscf.ims.mnc01.mcc001.3gppnetwork.org:6060 (Notice the extra leading zero after mnc)

@herlesupreeth that got updated already. When I analysed the logs I did correct it.
But when making a complete call I need to provision another sim and try? Because I've been trying that but facing some difficulties

But when making a complete call I need to provision another sim and try?

yes, without that you will receive 404 from IMS as it doesnt know the user you are trying to call

It does not matter, the other user must be registered at the IMS  in order for call to be successful. Please read refer to IMS material about how IMS works and how VoLTE calling works

@herlesupreeth ok thanks.
I'll do that ans keep you posted.

/home/volte1/Logs/01_07_2024_sipokfor113145/callfrom1003to113145_successful.pcapng
@herlesupreeth  some good new here. It is successful

@herlesupreeth having some good news.
Thanks


HIE assumes extensions are enabled when they are not.

It seems that HIE assumes that PatternSynonym and Arrows are enabled by default and recognizes proc and pattern as keyword when it should not.
Probably related to #1374

nvm this is a but of haskell-mode, sorry


Creating a Tamil Localization

Localization to Tamil Language.
https://en.wikipedia.org/wiki/Tamil_language
https://www.loc.gov/standards/iso639-2/php/langcodes_name.php?iso_639_1=ta

Issue: #

Requirements

Filling out the template is required. Any pull request that does not include enough information to be reviewed in a timely manner may be closed at the maintainers' discretion.

[ ]  issue was opened to discuss proposed changes before starting implementation.
[ ]  during development, node version specified in package.json was used (ie using nvm).
[x]  package versions and package-lock.json were not changed (npm install --no-save).
[x]  app version number was not changed.
[x]  all new code has tests to ensure against regressions.
[ ] npm run lint reports no offenses.
[ ] npm run test is error-free.
[ ]  README and CHANGELOG were updated accordingly.

Description of the Change

Verification Process

Other information

closing as after redesign many strings has changed


Very long swipe will not call onSlideChangeStart or onSlideChangeEnd

Hi!
If I hold down and swipe all the way to a new slide and then release (so that it is already at the new position on release), none of my callbacks is called. Only onTransitionStart which is called at the end.

Need live example, jsfiddle or video

No problem, give me some minutes and I will fix that

Will this do? http://tinypic.com/r/28jgv48/8

Ok, first of all, it is super rare edge case, which is 99% impossible in real world for user to swipe perfectly pixel to pixel on a width of swiper. Second, when you set it in such position, which is snap position of second slide, so there will no be any transition, and transitionEnd/slideChangeEnd event can't be triggered.
I will look on way to fix it, but as for now, you shouldn't worry much about it

Yes, I was just noticing this issue today, so I decided to use onTransitionStart and onTransitionEnd instead. But, unfortunately, I've just found out another issue: if I set autoplay and fade effect, when the slider reaches the end those callbacks like onSlideChangeStart / onSlideChangeEnd / transitionStart / transitionEnd don't work anymore. They work, instead, with any other effect (slide, cube, coverflow).
Here's a working example (slide effect): http://codepen.io/pasqualevitiello/pen/zxbNaN
and here's the issue (fade effect): http://codepen.io/pasqualevitiello/pen/azMpLp

@pasqualevitiello try with latest version, there were some fixes for fade effect

Superfast answer! But that does not fix the issue.

I use onTransitionEnd to instead, expect author to find a way to fix it. thanks

@pasqualevitiello fixed issue with autoplays in fade effect

That's OK, it works fine now! Are you going to merge this fix on next official release?

Yes, it will be in next official release


Fix incorrect detection of exhaustive matches for structural equality comparisons on some primitives.

for primitives that have custom 'eq' methods or implement Equatable[T] where T is a subtype of the primitive.
The type was inferred by looking at the type of the full case AST which was inferred
from the first argument of the eq method. Now the exhaustive match logic is
considering the actual type of the case expression.
This should fix #2327

It was failing on OSX for the stdlib tests compiled in release mode:
**** FAILED: process/STDERR
dispose: child exit code: 1
dispose: stdout: 0 bytes
dispose: stderr: cat: file_does_not_exist: No such file or directory
dispose: total data process_time:<PHONE_NUMBER>09 ns
dispose: ProcessNotify lifetime: 	51128992 ns
ProcessError: KillError

restarting the job.

@jemc added that test

Great work!

It was a pleasure, an honour!


Migrating packages to Maven Central

Hello. Due to JCenter deprecation and end of service are you planning to start migrating your packages to a new host, such as Maven Central?

Yes. But not sure about the timeline. Would you like to contribute for to this migration ?

@burhanrashid52 unfortunately, I'm new in library publishing. But if I'll have the needed changes I'll create a new pull request. Anyway, we have time up to February 2022

https://search.maven.org/artifact/com.burhanrashid52/photoeditor/1.1.1/aar


Styled components

What
Support generating React DOM code using styled-components for styling. In theory RN and Sketch should work too, though I haven't tested.
This can be configured with the --styleFramework=styledcomponents CLI flag.
A couple things to note:

Dynamic styles (set via logic) are still applied using the style jsx attribute of the element. I'm not sure if styled-components prefixes these -- but all of the styles we support setting dynamically should work on every browser without vendor prefixing, except maybe shadow.
I temporarily removed an svg optimization where I reused the same svg component if the svg is used twice within the same file. I'm not sure what's the best way to support this using styled-components.
A left a couple theme-related things commented since I don't want to forget what we had planned.
For RN/Sketch there will be naming collisions if you leave your layers as default names like "View", but we can deal with that if we ever end up using styled-components with those.

Testing Plan

[X] Tested this change locally
[X] Checked that existing features work

@outdooricon

WOOOOOO


Adds action to run ESLint on commit

This PR:

adds npm run lint to auto-run eslint on our source file(s)
adds npm run lint-fix to auto-fix all autofixable errors
creates a github action, lint, to check if eslint succeeds
changes the eslint env to node (from browser, which was incorrect)

As a note, there are currently some auto-fixable errors; I have not fixed these yet (since they may cause a merge conflict), but I can fix all errors with npm run lint-fix.
Debating on whether or not we should merge this first or #28 - @uclaacm/discord-bot-team what do y'all think?
(note: we'll squash this before merging)
part of #29

My 2 cents is we should wait for Brett's PR to merge in first (despite it not being linted) then merge this one and follow-up with a sanitation PR where we run this new lint-workflow on all files.

Seems reasonable to me!

Just to comment here for posterity - is there a reason why setMsg in the msg.js file is never used? Is this a mistake from the refactor, or just an incomplete function from earlier? The other error (dotenv not being used) isn't as bad
cc: @entrohpy @Bwoltz

@mattxwang I believe that's an issue from the refactor. setMsg should be called in main.js. I might not have hooked it up correctly.
https://github.com/uclaacm/ACM-at-UCLA-Discord-Bot/blob/4bdf86925b63137bb0afaf25981d9d2ad083a632/main.js#L314

@Bwoltz @mattxwang sorry I did not catch that in the review! I've exported it and pushed the change.


Autorotation for warrior question

Does auto rotation only work in duties where the map is supported, so only ultimate,  ex, savage basically

Nah it works everywhere to my knowlage. I tested it BRD and WHM in Summerford farm next to Limsa


How do you install this?

Hi, I have python 3.10.10 running.
I've downloaded and unzipped the zip. Then ran
[noel@family-desktop bark-main]$ python ./setup.py
/usr/lib/python3.10/site-packages/setuptools/config/pyprojecttoml.py:114: _BetaConfiguration: Support for `[tool.setuptools]` in `pyproject.toml` is still *beta*.
  warnings.warn(msg, _BetaConfiguration)
validate_pyproject.api.load_builtin_plugin defines `tool.distutils` schema
validate_pyproject.api.load_builtin_plugin defines `tool.setuptools` schema
usage: setup.py [global_opts] cmd1 [cmd1_opts] [cmd2 [cmd2_opts] ...]
   or: setup.py --help [cmd1 cmd2 ...]
   or: setup.py --help-commands
   or: setup.py cmd --help

error: no commands supplied

What commands should I supply, or am I doing it wrong?
Thanks.

The readme gives a sample script.

yup, either install directly from git or clone repo and use pip install .

yup, either install directly from git or clone repo and use pip install .

Ok thanks. I'm used to seeing such instructions earlier on in the readme.

@klmitch Hi, sorry for the cold mention. We are having a bunch of people running into issues doing pip install bark and getting the wrong package. I saw your bark project isn't really active anymore. Any chance you'd be willing to part with the 'bark' name on pypi? please shoot us a quick email at<EMAIL_ADDRESS>Thanks in advance, much appreciated!

Very much appreciated! My pypi username is gkucsko associated with email gkucsko at gmail.com

I just deleted the sole release associated with that project.  Pypi now complains about not finding that project if you click the project in the results list for "bark".  Best guess is that they'll regenerate the results list either in a few hours or sometime overnight, and then "bark" will be free for the taking.

And then I discovered that the project itself could be deleted, which I have done.  "bark" is completely free for the taking.  Enjoy.

Many thanks! We appreciate it. Put a staging build up of the package to claim it.

Thanks a bunch! If you have a ‘buy me a coffee’ link we’d be happy to contribute something to your next coding streak! - Best, Suno Team


Error "Expected a value of type XXX but received" on emun

Hi,
I have an issue with the Enum type.
We are developing a web app and sharing the same develop database during development.
The issue is, if one of us added some testing data or new data into the enum field in database, then others will get the error "Expected a value of type XXX but received" when doing a full query on table for example. Because the testing data will be read by everyone using this database and those whose branch does not have this new data in Enum will get this error.
It's an annoying issue that we try to avoid, and it is getting worse and worse while the product is growing.
Is there a way to solve it or to hide this kind of error?
Thanks
CJ

Enum means only a certain range of value is valid. So if you can have any string in DB, you should mark the type as string, not an enum.

Enum means only a certain range of values is valid. So if you can have any string in DB, you should mark the type as string, not an enum.

Sorry mate, maybe my description was not clear.
I'll give you a simple example of this issue.
A and B are two developers, and they are using a string enum to store the access permission.
enum is:
enum Access = {
All = "All",
None = "None
}
All the users has a database filed, which is a string array, to store their own access.
A added a new option in the enum, which is "Some"
The enum become
{
All = "All",
Some = "Some",
None = "None
}
Then A created a user which has the new field "Some"
MongoDB field is:
access: ["All", "Some", "None"]
B is actually working in another branch and he does not have "Some" in his Access Enum.
When B queries all items in users table, he can pickup data from DB without any issue.
But when he tried to pass the value back to frontend through type-graphql, he will got this error until he merge A's code in his branch.
In real world situation, we may have many developers working with the same database but different branches of code and working on tons of enums at the same time.  Then this issue become annoying.
Currently if it happens, we have to either merge this bit of enum code to everyone's branches, or we have to clean up the database or force guys to work on their local db which is not ideal...

There's nothing I could do about it. The Expected a value of type error comes from underlaying graphql-js that checks if the returned data match the GraphQL schema contract.
In your case you should just filter the data in the repository/DB layer, so it will convert the additional values to the supported one.
Different branches of code and global shared database means only one: 💣 💥

There's nothing I could do about it. The Expected a value of type error comes from underlaying graphql-js that checks if the returned data match the GraphQL schema contract.
In your case you should just filter the data in the repository/DB layer, so it will convert the additional values to the supported one.
Different branches of code and global shared database means only one: 💣 💥

Thanks for your advise. Close the issue 👍


config from_* methods should document their return type

They do return something, but it's not obvious what it is, so having a :return: whatever they return in the docstring would be nice to have.

Can I get some info about how to get started with this issue? I would like to contribute

Can I get some info about how to get started with this issue?

@starlightknown Basicially, you will need to find these from_* methods in src/flask/config.py, then add a :return: REPLACE THIS WITH ACTUAL RETURN TYPE statement in the docstring for each method (example).

Got it, can I get assigned to it or start working on it? @greyli

Yes, you're welcome to submit a pr, no need to ask.


Formatting for build check

Codecov Report

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

@@           Coverage Diff           @@
##           master     #258   +/-   ##
=======================================
  Coverage   62.80%   62.80%
=======================================
  Files          48       48
  Lines        3828     3828
=======================================
  Hits         2404     2404
  Misses       1024     1024
  Partials      400      400

Continue to review full report at Codecov.

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


fix: Fix Hidden not setting unstable_animating to false while visible

Closes #407

Codecov Report

Merging #410 into master will increase coverage by 0.63%.
The diff coverage is 50%.

@@            Coverage Diff             @@
##           master     #410      +/-   ##
==========================================
+ Coverage   94.13%   94.76%   +0.63%
==========================================
  Files          89       89
  Lines        1398     1395       -3
  Branches      442      439       -3
==========================================
+ Hits         1316     1322       +6
+ Misses         82       73       -9

Impacted Files
Coverage Δ

packages/reakit/src/Hidden/Hidden.ts
91.3% <0%> (+10.53%)
:arrow_up:

packages/reakit/src/Hidden/HiddenState.ts
97.43% <100%> (+15.38%)
:arrow_up:

Continue to review full report at Codecov.

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


Native Testing does not work (Ruby)

I tried to turn on testExplorer.useNativeTesting = true, but this does not work.
GUI shows the progress x of 170 tests passed...suddenly changing to 16/16 tests passed and stops.
However - when I monitor the Ruby Test Explorer logfile, the tests are indeed running in the background... but the GUI does not update.
When I click a single tests, they seem to work, but when I try to run multiple, it fails.
Also when trying only system tests its says x of 40 tests passed..suddenly changing to 20/20 passed.
The logfile seem to indicate that the tests pass.
So logfile and test process seems right. GUI seems lost.
Everything works fine when Native Testing = false

Please open this issue in https://github.com/microsoft/vscode-test-adapter-converter.


[bct] move changelog delimiters to changelog tracker

Only print the changelog delimiters around the changelog

@msyyc I'm assuming these delimiters arent necessary around the breaking changes reported? Only needed around the changelog output to capture the result?

@msyyc I'm assuming these delimiters arent necessary around the breaking changes reported? Only needed around the changelog output to capture the result?

Yeah, here is the logic I capture the changelog from print output: https://github.com/Azure/azure-sdk-for-python/blob/625e0fd737fd3f0f9289b906fd58806e0f128789/tools/azure-sdk-tools/packaging_tools/package_utils.py#L33-L35. I am fine with PR as long as the delimiters could be kept so that I could extract the changelog concisely.

/check-enforcer evaluate