How to pass dataset kwargs through config file

I noticed that several dataset classes have kwargs in addition to the root='datasets' that's provided in the ImageDataset base class' __init__ signature. For instance

Market1501

market_1501_500k=False

CUHK03

split_id=0
cuhk03_labeled=True
cuhk03_classic_split=False

VehicleID

test_list=''

VeRiWild

query_list=''
gallery_list=''

I do not see any documentation on how the YAML configuration file can be used to run train_net.py with something other than the above default values. In the training script procedure, I noticed the dataset gets built from the registry in data/build.py:build_reid_train_loader() which only passes the combineall kwarg the the dataset class initializer
dataset = DATASET_REGISTRY.get(d)(root=_root, combineall=cfg.DATASETS.COMBINEALL)

In order to support the other kwargs above, it seems the above line would have to be modified in the source code. For instance, to support Market1501's kwarg
other_kwargs = {
    'market_1501_500k': cfg.DATASETS.get('MARKET_1501_500k', False)
}
dataset = DATASET_REGISTRY.get(d)(root=_root, combineall=cfg.DATASETS.COMBINEALL, **other_kwargs)

However, this would get rather cumbersome adding support for all the various kwargs used by different datasets. Do you have a recommended approach to handle this situation? Perhaps we could create other_kwargs such that it contains all config keys under DATASETS which are not commonly shared
known_keys = ['NAMES', 'TESTS', 'COMBINEALL']
other_kwargs = {k.lower(): v for k, v in cfg.DATASETS.items() if k not in known_keys}

Good question, let me think about it.

@addisonklinke Hi, I just pushed a commit that solved this problem, you can pass other kwargs following the way below
class Trainer(DefaultTrainer):
    @classmethod
    def build_train_loader(cls, cfg):
		other_kwargs = {
				'market_1501_500k': cfg.DATASETS.get('MARKET_1501_500k', False)}
        return build_reid_train_loader(cfg, **other_kwargs)


from typing import List (one line I had to add to python demo example in README.md)  Also ct format doable?

my own TODO
Great work! I may try to tweak to get output in .vienna format which has the mfe appened on same line as dot-brackets. I want to count bulges, and one way to do so involves converting .vienna dot-bracket to .ct (connect) format. Seems doable ;-)
Single-char Ttpo in README.md where I think seqfold -v -l output  should have  ddg not dg that is:   i    j     ddg  description not    i    j     dg  description
BTW what does ddg stand for?  May also want to mention version(s) of python 3.x required. I think typing did not start showing up until 3.5 or 3.6, right? And it works differently in 3.9 per https://stackoverflow.com/questions/57505071/nameerror-name-list-is-not-defined/
Also, one line I needed in python demo
Note I had to add one extra line to the python demo in README.md
from typing import List
Adding above line got rid of NameError... as shown below. I just pasted python demo from README.md into fileseqfold-demo.py file.
$ head -4  seqfold-demo.py. # after adding `from typing import List` line
from seqfold import dg, dg_cache, fold, Struct
from typing import List
# just returns minimum free energy

$ python. seqfold-demo.py

-15.200000000000001
   0   48   -2.1  STACK:GG/CC
   1   47   -2.1  STACK:GG/CC
   2   46   -1.3  STACK:GA/CT
   3   45   -1.3  STACK:AG/TC
   4   44   -2.1  STACK:GG/CC
   5   43   -1.5  STACK:GT/CA
   6   42   -1.3  STACK:TC/AG
   7   41   -0.2  BIFURCATION:4n/3h
   9   22   -1.0  STACK:TT/AA
  10   21   -0.9  STACK:TA/AT
  11   20   -1.5  STACK:AC/TG
  12   19    3.1  HAIRPIN:CA/GG
  25   39   -2.1  STACK:CC/GG
  26   38   -2.2  STACK:CG/GC
  27   37   -2.1  STACK:GG/CC
  28   36    3.4  HAIRPIN:GT/CT

Without that one extra line I get NameError...
$ head -3  seqfold-demo.py # without `from typing import List` line
from seqfold import dg, dg_cache, fold, Struct

# just returns minimum free energy

$ python seqfold-demo.py
Traceback (most recent call last):
  File "seqfold-demo.py", line 7, in <module>
    structs: List[Struct] = fold("GGGAGGTCGTTACATCTGGGTAACACCGGTACTGATCCGGTGACCTCCC")
NameError: name 'List' is not defined
$ python --version
Python 3.8.8

Thank you for reaching out and taking interest in seqfold! I'm very interested in the Vienna output, please let me know how it goes. Will you CR? Seems like an interesting feature either way
Thank you for the README catches, updating ddg and the use of the Typing list now. I just ran the below (copy-paste from README) and confirmed that it worked:
>>> # `fold` returns a list of `seqfold.Struct` from the minimum free energy structure
>>> structs = fold("GGGAGGTCGTTACATCTGGGTAACACCGGTACTGATCCGGTGACCTCCC")
>>> print(sum(s.e for s in structs))  # -12.94, same as dg()
-13.4
>>> for struct in structs:
...     print(struct) # prints the i, j, ddg, and description of each structure
...
   0   48   -1.8  STACK:GG/CC
   1   47   -1.8  STACK:GG/CC
   2   46   -1.3  STACK:GA/CT
   3   45   -1.3  STACK:AG/TC
   4   44   -1.8  STACK:GG/CC
   5   43   -1.5  STACK:GT/CA
   6   42   -1.3  STACK:TC/AG
   7   41   -0.2  BIFURCATION:4n/3h
   9   22   -1.0  STACK:TT/AA
  10   21   -0.6  STACK:TA/AT
  11   20   -1.5  STACK:AC/TG
  12   19    3.1  HAIRPIN:CA/GG
  25   39   -1.8  STACK:CC/GG
  26   38   -2.2  STACK:CG/GC
  27   37   -1.8  STACK:GG/CC
  28   36    3.4  HAIRPIN:GT/CT
>>> # `dg_cache` returns a 2D array where each (i,j) combination returns the MFE from i to j inclusive
>>> cache = dg_cache("GGGAGGTCGTTACATCTGGGTAACACCGGTACTGATCCGGTGACCTCCC")

I bumped the Python requirement to 3.5
DDG

DDG, often referred to as ΔΔG, is the change in the change in Gibbs free energy (double changes intended). DDG is a measure of the change in energy between the folded and unfolded states (ΔGfolding) and the change in ΔGfolding when a point mutation is present.
https://cyrusbio.com/wp-content/uploads/Rosetta-Cartesian-DDG-2019.pdf

README should be fixed now, thanks again


[yourporn] fix extraction and rename to sxyprn

Please follow the guide below

You will be asked some questions, please read them carefully and answer honestly
Put an x into all the boxes [ ] relevant to your pull request (like that [x])
Use Preview tab to see how your pull request will actually look like

Before submitting a pull request make sure you have:

[x] At least skimmed through adding new extractor tutorial and youtube-dl coding conventions sections
[x] Searched the bugtracker for similar pull requests
[x] Checked the code with flake8

In order to be accepted and merged into youtube-dl each piece of code must be in public domain or released under Unlicense. Check one of the following options:

[x] I am the original author of this code and I am willing to release it under Unlicense
[ ] I am not the original author of this code but it is in public domain or released under Unlicense (provide reliable evidence)

What is the purpose of your pull request?

[x] Bug fix
[ ] Improvement
[ ] New extractor
[ ] New feature

Description of your pull request and other information
Fixes the extractor of https://sxyprn.com/ and renames it to SxyPrn. Support for https://yourporn.sex/ is droped, since that's a different website now.

Duplicate of #22255


feat: getTextContents

getTextContents can be used to extract plaintext from the editors value
this mr also includes some docs updates

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

v4.2.0
GitHub release

Your semantic-release bot :package::rocket:


Kai Gidwani's Bug Fix

@kaigidwani hmm ok, your images looked ok when you first posted them, but the links to them seem to have expired. Can you please upload them to the git repo and link to them directly so I can grade your blog post?

on the bright side, shoutout for the kickass alt-test on your images that keeps things understandable anyway


install path /opt/homebrew/Homebrew is unsupported

It seems Homebrew is installing packages from source if the install path is not in an expected location:
https://github.com/orgs/Homebrew/discussions/272
the role installs it to /opt/homebrew/Homebrew instead of /opt/homebrew.
and I wondered for a long time why homebrew is so slow for me.

i just saw there's PR #89 for it.


Can use "useCallback" for optimized child components.

Hi there!
A suggestion is to use useCallback to preserve the identity of the newSetState function, so that newSetState can be used in optimized child components that may choose not to re-render based on prop changes.
This will work as setState itself keeps its identity (see Note), and as long as isComponentMounted remains a ref the function doesn't need to change, as the function will always have access to the ref's current value.
import { useState, useCallback } from 'react';
import useIsComponentMounted from './useIsComponentMounted';

export default function useStateIfMounted(initialValue) {

  const isComponentMounted = useIsComponentMounted();
  const [state, setState] = useState(initialValue);

  const newSetState = useCallback((value) => {
    if (isComponentMounted.current) {
      setState(value);
    }
  });

  return [state, newSetState];
}

@Colkadome You forgot the useCallback dependencies:
export default function useStateIfMounted(initialValue) {
  const isComponentMounted = useIsComponentMounted();
  const [state, setState] = useState(initialValue);

  const newSetState = useCallback((value) => {
    if (isComponentMounted.current) {
      setState(value);
    }
  }, [isComponentMounted]);

  return [state, newSetState];
}


adding special-pages subdirectory

https://app.asana.com/0/0/1204205851573888/f
This PR adds NPM workspace support for packages/special-pages
There's an example page in there, with a playwright test

Current dependencies on/for this PR:

main

PR #325   👈

This comment was auto-generated by Graphite.

@jonathanKingston I updated this PR to use devDependencies


Simple OSIAM Docker Image

I created a simple OSIAM Docker image, that just includes the auth-server and the resource-server. It uses an external postgresql server (docker).
https://hub.docker.com/r/burberius/osiam-simple/
Source:
https://github.com/burberius/osiam-docker-simple
Perhaps it's useful for other users and you could link to it.

I'm sorry, but this one got lost due to the release of OSIAM 3.0. Also, OSIAM 2.x has been discontinued with the release of 3.0. If you like, you can participate in building the new OSIAM 3.0 Docker image. We're currently planning the details and there will be an issue for this doing soon.


Request: Add option for post view to show full image

Hello!
Could we add an option to have the view of a post show the full-size image without clicking through? My rationale is this: If you're on the compact subreddit view and you click on an image post, you've already seen a thumbnail of the image, so the expectation is that what is displayed is the full image.
Here's an example:

The image shown in the screenshot above is a small portion of the full image:

I think it would be better to show the full image from the outset to reduce the amount of clicking through that needs to be done to see the full image, since I see myself needing to do it for quite a few posts.

Also, this is a setting on Android

Done!


Optimize iPad layout

Hello! Thanks for making a great iOS app. I am using it on iPad and it seems like the space is not well used. Like it's the phone app blown up. So I took some screen shots to show the issues I see and the possible fix I have imagined so far. Please take this as constructive suggestions I don't mean any offense. you have done a great job :)
Issue 1 Printer status is hidden

Issue 2: Scrolling down to see printer status looses file name and Pause / Cancel button

Lots of empty space that can be used to show things.

Here is a idea how to make it better off the top of my head. I'm not sure how the bottom half of the screen space could be used better yet. Would need to think about it more.
But the top half can show the temp graph and that video feed. It also can move up a little to not loose information off the screen edge.

Hi @kenthinson
Thanks so much for your UI feedback and proposals. You are correct and no offense has been taken. :) Anything to improve the app is more than welcome. You are also correct in that the iPad is the same iPhone UI but in a bigger screen. As you noticed this prevents us from making better usage of the space as you highlighted.
I will see what I can do to improve UX/UI and yet make it easy to maintain iphone and ipad codebases.
Thanks,
Gaston

This other ticket might be related to this one #318. Both are about improving iPad layout.


Freeze Strings

At work, we've been using this gem at scale (serving ~ 20-100M req/month), and detected this gem to pollute memory by allocating way too many strings.
We've been running this adjustment of freezing multiple strings for six months now without issues (and measurably lower memory footprint).
Sadly, I couldn't find our exact memory profiling results anymore (we did extensive before/after tests using the memory profiler gem and observing memory footprint on the servers).
I remember, that strings for country codes got allocated multiple times before, and only once after freeze strings.
Thank you for maintaining open source :-)

Good one 👍, thanks for the PR


Added new method 'secure_random_string'

Added new method secure_random_string that uses openssl_random_pseudo_bytes to generate best possible random string of given length. If openssl_random_pseudo_bytes not avaible then script will use default random generator.
Also added default length of 16 to both random_string and secure_random_string

Coverage decreased (-0.79%) to 75.87% when pulling a6fac8aa2f8971eee51d1c22f86837d839bc85cb on abhimanyusharma003:patch-2 into 74d79a58e70d3e11026f92a7c2ac488de60f83cd on brandonwamboldt:master.

Coverage decreased (-0.79%) to 75.87% when pulling ff97c011bf3a23fd1494e04a3ad30859dd147b8a on abhimanyusharma003:patch-2 into 74d79a58e70d3e11026f92a7c2ac488de60f83cd on brandonwamboldt:master.

Coverage decreased (-0.79%) to 75.87% when pulling abeb630ae6ea8f9a1d0c5cfb615b4b4ac0032a08 on abhimanyusharma003:patch-2 into 74d79a58e70d3e11026f92a7c2ac488de60f83cd on brandonwamboldt:master.

Sorry, I can't merge this until you revert the changes that aren't compatible with PHP 5.3 (array brackets, the [ and ] style notation for building arrays is only available in PHP 5.4). While I know PHP 5.3 is old, I've still chosen to support it.

Ah my bad, you already reverted that change

Thanks for the contribution :)


Handle outstanding MAC requests when interface is disabled.

Current coverage is 70.18% (diff: 100%)

Merging #689 into master will decrease coverage by <.01%

@@             master       #689   diff @@
==========================================
  Files            97         97
  Lines         13679      13681     +2
  Methods        1883       1883
  Messages          0          0
  Branches       1606       1605     -1
==========================================
+ Hits           9601       9602     +1
  Misses         3590       3590
- Partials        488        489     +1

Powered by Codecov. Last update 6abc980...b1d8795


add codegen for TinyGo actors

Signed-off-by: stevelr<EMAIL_ADDRESS>Updates dependency on weld-codegen to 0.4.4 which is in PR https://github.com/wasmCloud/weld/pull/100

Will review this when the relative path dependency is gone

Build checks won't pass until  https://github.com/wasmCloud/weld/pull/100 is merged. Then, we can remove the relative path and re-trigger the build checks.


Lesson page crashes with unknown bug

Lesson: https://frontend.ro/html/liste#tipuri-de-liste
Browser: Firefox Nightly
Doesn't reproduce every time 🤷♂

i really don't have this error @iampava !

It doesn't happen every time 🤷♂


move shims (foreign items and intrinsics) into submodule

This is in preparation for dlsym support, where there will b a third "kind" of shim.
And maybe some day we find a good way to split those long files...

@bors r+

:pushpin: Commit b8879f3d8cd8401dea369d2ada3c266b2d8c913c has been approved by RalfJung

:hourglass: Testing commit b8879f3d8cd8401dea369d2ada3c266b2d8c913c with merge 048b00d20ffd87854550962ab16a397a1fb77a3f...

:sunny: Test successful - checks-travis, status-appveyor
Approved by: RalfJung
Pushing 048b00d20ffd87854550962ab16a397a1fb77a3f to master...


⚠ GitLab Server has degraded performance

In e13bfde, GitLab Server (https://gitlab01.its-telekom.eu) experienced degraded performance:

HTTP code: 200
Response time: 6176 ms

Resolved: GitLab Server performance has improved in c06c1f4.


Release v3

Can we get a push?

Was waiting on https://github.com/orta/ORStackView/issues/10

Gotcha :+1:

That could be a patch release, though?

It would be a minor, but we can wait a bit more if #10 is gonna get done soonish.

Better to presume OSS timelines, pushed 3.0.0.

You know that's right :P


Processes as entities

Closes #1.
This PR explores modeling processes as entities.
We use two components:

LocalCommand, which contains the std::process::Command to run. You spawn this component to run the command.
Process, which contains the running process, the output buffer, and the task which reads the output.

To spawn a new process, you don't use events anymore, but Bevy's Commands:
fn startup(mut commands: Commands) {
    let mut cmd = std::process::Command::new("sh");
    cmd.args(["-c", "echo Sleeping for 1s && sleep 1 && echo Done"]);

    let id = commands.spawn(LocalCommand::new(cmd)).id();
    println!("Spawned the command as entity {id:?}");
}

I debated also adding an API to use an event to spawn the process, but that's actually equally or even more verbose than just using Commands.
The events now only store the Entity of the corresponding Process.
To obtain the PID, Command that was run, and other information, you can simply query for Command and/or Process and search for the entity.
This should be a lot more performant than constantly cloning strings and such.
We also don't need the ActiveProcessMap anymore, we just use the ECS as a collection of our processes.
Using the ECS directly also makes it easier to manage larger collection of processes.
You can simply add marker components or other metadata to the corresponding entities and then filter for that in your queries.
Open Questions

Getting other information (PID, command string representation) for events is now a bit more annoying, as you have to run a query as well. On the other hand, it should scale a lot better and be more performant. Is this an acceptable trade-off? Of course, we could still add more info to the events, at the cost of performance.
In theory you could despawn the entity without killing the process first. I wonder if that could "leak" the processes and leave them running, or if dropping the Process will automatically kill them...

I think that this model also provides a nice future path for this crate, as we can develop "add-ons" to further modify the behavior of command execution.
For example, we can have a LocalCommandChain component which executes multiple commands after one another.
Then we can add a Retry component which controls what happens when the command fails and can simply retry a couple of times or use an exponential backoff strategy, etc.
Or maybe a Repeater component which executes the command in an infinite loop...
I think having a modular setup like this is a great path forward as we can allow the user to customize how they want their commands to be executed without having a giant monolithic setup

Nice! I really like this. I've been sick, but will review and test over the
weekend. Thanks 🙏
On Fri, Jan 5, 2024, 05:31 TimJentzsch @.***> wrote:

I think that this model also provides a nice future path for this crate,
as we can develop "add-ons" to further modify the behavior of command
execution.
For example, we can have a LocalCommandChain component which executes
multiple commands after one another.
Then we can add a Retry component which controls what happens when the
command fails and can simply retry a couple of times or use an exponential
backoff strategy, etc.
Or maybe a Repeater component which executes the command in an infinite
loop...
I think having a modular setup like this is a great path forward as we can
allow the user to customize how they want their commands to be executed
without having a giant monolithic setup
—
Reply to this email directly, view it on GitHub
https://github.com/edouardpoitras/bevy_local_commands/pull/11#issuecomment-1878452663,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/ADLMOANO5CHEPLIXTKBWU6TYM7I7VAVCNFSM6AAAAABBNRQZOWVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQNZYGQ2TENRWGM
.
You are receiving this because you were assigned.Message ID:
@.***>

Looks pretty solid to me. I much prefer this new approach. Looking forward to adding some of those "add-ons" you mentioned.
Does it make sense to just remove the KillProcess event now and have the user perform a query to fetch the process and kill it themselves? If they need the entity ID anyways, the user will probably end up querying it regardless?
If you get a chance, updating the examples in the README would save me the trouble.
Also, you can go ahead and bump up the version to 0.2.5.
Thanks!

Does it make sense to just remove the KillProcess event now and have the user perform a query to fetch the process and kill it themselves? If they need the entity ID anyways, the user will probably end up querying it regardless?

Good point, I'll remove the kill event

Also, you can go ahead and bump up the version to 0.2.5.

I believe we'll have to bump to 0.3.0 as this is a breaking change.
Otherwise, cargo might automatically update the dependency of projects and it will break their build.
See https://doc.rust-lang.org/cargo/reference/manifest.html#the-version-field for reference

Looks good to me!

FYI - regarding your 2nd open question, I did some testing and this is what happens on my Linux box if you despawn the entity before killing the process:

Process continues in the background
No more ProcessOutputEvent
No more ProcessCompletedEvent
Process is killed if Bevy is stopped

FYI - regarding your 2nd open question, I did some testing and this is what happens on my Linux box if you despawn the entity before killing the process:
* Process continues in the background

* No more ProcessOutputEvent

* No more ProcessCompletedEvent

* Process is killed if Bevy is stopped

Thanks for investigating this!
This seems like a potential footgun, I wonder if we can prevent this somehow.
Otherwise, we should probably document this behavior.
Maybe something for a future PR though.


[Bug]: An error is not thrown for an invalid annotation in function pointers

Description
$title.
Steps to Reproduce
Run the following Ballerina code.
import ballerina/io;

var fnPointer = function() {
    io:println("Hello World");
};

public function main() {
    @UndefinedAnnotation
    fnPointer();
}

Affected Version(s)
Ballerina 2201.8.0 (Swan Lake Update 8)
OS, DB, other environment details and versions
OS: macOS 14.1.1 23B81
JDK: openjdk 17.0.8 2023-07-18
Related area
-> Compilation
Related issue(s) (optional)
#41787
Suggested label(s) (optional)
No response
Suggested assignee(s) (optional)
No response

Annotations are not allowed on (call) statements. Seems like it gets dropped by the parser without logging an error.

function value() {
    @val _ = 3;
}

Here also, annotations are not allowed on wild card binding pattern.  It gets dropped by the parser without logging an error.


Audio mix source

Hi @pedroSG94 ,
Do you have any plans to improve the sound mixing quality when using both the microphone and in-app sound?
https://github.com/pedroSG94/RootEncoder/pull/1571
Currently, if both are used, the sound is not very clear. It seems like the microphone is picking up sound from both the speaker and microphone, making the live stream sound noisy and unpleasant.
If there is a way to mix the app sound and microphone audio more transparently, it would be great (I have already enabled echo cancellation and noise cancellation). I tried using the Larix app, and there is a noticeable difference in the sound quality.
Best regards,

Hello,
What do you means with noisy and unpleasant?
Do you have audio pops? echo?

Yes, that's correct. I have tested both the RootEncoder and Larix apps, and below are the test links. The RootEncoder does not perform as well as Larix in terms of sound quality.
Larix: https://youtube.com/live/9tu5ujjUPxk
RootEncoder: https://youtube.com/live/ReBu0ijdaVc

I see, I will check it.
For now, try add this line after create the stream class:
setTimestampMode(TimestampMode.BUFFER, TimestampMode.BUFFER)

This is only available in the master branch so you can use the app example with screen example and add that line here:
https://github.com/pedroSG94/RootEncoder/blob/master/app/src/main/java/com/pedro/streamer/screen/ScreenService.kt#L89

I was doing tests using MixAudioSource and the audio sometimes produce noise but in general the quality is fine in my case. I tested using Pixel 6a and Xioami A3 (Android 14 and Android 10).
Also, after use TimestampMode.BUFFER, the quality is better and then I can't reproduce audio noise so this seem to solve the problem.
Let me know if you was able to fix the audio using that.

Hi,
We will run a test and provide you with the results. Thank you for your help!

Hi,
Thank you for your help. It seems to be working better than before.

Hi @pedroSG94 ,
We've encountered an issue with screen broadcasting that occurs after 20 minutes. At that point, an audio delay appears; for example, gunshots sound before they are visible in the video. You can see an example here: Delay.
Additionally, there's noticeable noise when the microphone is on, even though Noise Cancellation is enabled. Here’s an example: Noise.
Do you have any suggestions on how to address these issues?

Hello,
Ok, I will check it.
Thank you for the report

Hello,
Sorry for no report back. I was testing with 4 differents devices but I'm not able to reproduce it.
The sync and audio is the same that at the start doing tests of 40min+ (quality is good)
Tested devices: Pixel 7, Pixel 6a, Xiaomi A3, Samsung A50
I was using the app example by default and adding this line:
setTimestampMode(TimestampMode.BUFFER, TimestampMode.BUFFER)

For now, I'm testing in a local server
Did you tried with a local server instead YouTube to discard that the problem only happens in YouTube? (I will check this now too)
Can you tell me your video and audio configuration to do the test with the same values?
Did you modify the volume of the microphone or internal?

Hi @pedroSG94
I just tested to the latest version and noticed an improvement in the delayed sound compared to the previous version. However, the sound still becomes distorted when the microphone is turned on.
You can listen to an example of the issue in this live stream: [YouTube Link]
I set this line on my code:
setTimestampMode(TimestampMode.BUFFER, TimestampMode.BUFFER)
Audio: 48Khz/128kps
Best regards,

Hello,
Thank you for the info. I will check it using that audio config.
If you are using this to increase the volume of the microphone try to remove the line:
microphoneVolume = 2f

If the device audio is fine but only fail the microphone audio using MixAudioSource and it is working using MicrophoneSource, the problem could be related with the audio volumen filter or the audio mixer.

Hello,
Finally, I was able to reproduce it. I'm working on it but it seems that the problem is related with the way audio is mixed

Hi,
Thank you. I will also review it on my end, and if I discover anything, I will be sure to inform you.
Best regards,


Complete Assignment 3.2

Question

Complete Task in the following file Assignment
The Name of Program must be Assignment 32.ipynb

Task Completed
Go to Assingment


install ruby failed if default gem install failed

Describe the bug
I have some gems set in .default-gems, and  the gem install fails if the new version of some gems are not compatible with the current ruby version, and also cause the ruby install failed
To Reproduce

set following code in .default-gems
rails

run mise install<EMAIL_ADDRESS>

Expected behavior
install default gems failed should not block the ruby install, I can install a older version gem by myself after install ruby
mise doctor output
version: 2024.5.3 macos-arm64 (2024-05-07)
activated: yes
shims_on_path: no

build_info:
  Target: aarch64-apple-darwin
  Features: DEFAULT, NATIVE_TLS
  Built: Tue, 7 May 2024 00:35:06 +0000
  Rust Version: rustc 1.78.0 (9b00956e5 2024-04-29) (Homebrew)
  Profile: release

shell:
  /bin/zsh
  zsh 5.8.1 (x86_64-apple-darwin21.0)

dirs:
  data: ~/.local/share/mise
  config: ~/.config/mise
  cache: ~/Library/Caches/mise
  state: ~/.local/state/mise
  shims: ~/.local/share/mise/shims

config_files:
  ~/.tool-versions
  ~/.config/mise/config.toml

backends:
  cargo
  go
  npm
  pipx
  ubi

plugins:
  bun     (core)
  deno    (core)
  go      (core)
  java    (core)
  node    (core)
  python  (core)
  ruby    (core)
  usage   https://github.com/jdx/mise-usage.git#fe3888a

toolset:
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>
<EMAIL_ADDRESS>

env_vars:
  MISE_RUBY_DEFAULT_PACKAGES_FILE=/Users/jiz4oh/.rbenv/default-gems
  MISE_USE_TOML=1
  MISE_NOT_FOUND_AUTO_INSTALL=false
  MISE_SHELL=zsh

settings:
  activate_aggressive = false
  all_compile = false
  always_keep_download = false
  always_keep_install = false
  asdf_compat = false
  cargo_binstall = true
  color = true
  disable_default_shorthands = false
  disable_tools = []
  experimental = false
  go_default_packages_file = "~/.default-go-packages"
  go_download_mirror = "https://dl.google.com/go"
  go_repo = "https://github.com/golang/go"
  go_set_gobin = true
  go_set_gopath = false
  go_set_goroot = true
  go_skip_checksum = false
  jobs = 4
  legacy_version_file = true
  legacy_version_file_disable_tools = []
  node_compile = false
  not_found_auto_install = false
  paranoid = false
  plugin_autoupdate_last_check_duration = "7d"
  python_compile = false
  python_default_packages_file = "/Users/jiz4oh/.default-python-packages"
  python_pyenv_repo = "https://github.com/pyenv/pyenv.git"
  raw = false
  trusted_config_paths = []
  quiet = false
  verbose = false
  yes = false
  ci = false
  debug = false
  trace = false
  log_level = "info"
  python_venv_auto_create = false

  [status]
  missing_tools = "if_other_versions_installed"
  show_env = false
  show_tools = false

No warnings found
No problems found

Additional context
Add any other context about the problem here.

I'd accept a PR

😥I love mise but I don't know how to use rust. Hope someone can help, it makes mise more smooth. I have to comment the default gems before installation and uncomment later

the change is not hard, just change this to a warning (there is an example of that in the same method):
https://github.com/jdx/mise/blob/1e61d239e09a55464b312c3fd5fb7c609776cf9e/src/plugins/core/ruby.rs#L373


merge develop to master

now that master has been merged to develop, merge develop back to master

https://github.com/opendifferentialprivacy/smartnoise-core/tree/326_merge_develop_to_master


wasix build not working on macos aarch64 after install

Hi,
I just installed cargo-wasix with cargo install cargo-wasix but after writing a simple fibonacci application I cant seem to get it building or running with wasix.
❯ cargo wasix run
error: couldn't read +wasix: No such file or directory (os error 2)

error: aborting due to previous error

error: Could not execute rustc

Caused by:
    failed to execute "rustc" "+wasix" "--print" "sysroot"
        status: exit status: 1

This is happening on macOS Ventura 13.5.1 on an M1 Max

I just ran into this as well.
It seems like the cargo wasix self clean command doesn't remove everything, so rustup gets left in an inconsistent state.
Here is what I was seeing:
$ cargo wasix build
error: toolchain 'wasix' is not installed
error: Could not execute rustc

Caused by:
    failed to execute "rustc" "+wasix" "--print" "sysroot"
        status: exit status: 1

$ rustc +wasix --print sysroot
error: toolchain 'wasix' is not installed

But if we check rustup, we'll see the wasix toolchain is still "installed".
$ rustup show
Default host: aarch64-apple-darwin
rustup home:  /Users/work/.rustup

installed toolchains
--------------------

stable-aarch64-apple-darwin
nightly-2023-05-25-aarch64-apple-darwin
nightly-2023-08-07-aarch64-apple-darwin
nightly-aarch64-apple-darwin (default)
1.56-aarch64-apple-darwin
1.63-aarch64-apple-darwin
1.64-aarch64-apple-darwin
1.65-aarch64-apple-darwin
1.66-aarch64-apple-darwin
1.67-aarch64-apple-darwin
1.69-aarch64-apple-darwin
1.70-aarch64-apple-darwin
1.71-aarch64-apple-darwin
wasix

installed targets for active toolchain
--------------------------------------

aarch64-apple-darwin
riscv64gc-unknown-linux-gnu
wasm32-unknown-unknown
wasm32-wasi

active toolchain
----------------

nightly-aarch64-apple-darwin (default)
rustc 1.75.0-nightly (75b064d26 2023-11-01)

You need to tell rustup to remove the toolchain before cargo wasix will download it again.
$ rustup toolchain remove wasix
info: uninstalling toolchain 'wasix'
info: toolchain 'wasix' uninstalled

$ cargo wasix build
Finding latest release... (https://api.github.com/repos/wasix-org/rust/releases/latest)...
Downloading sysroot from url 'https://github.com/wasix-org/rust/releases/download/v2023-11-01.1/wasix-libc.tar.gz'...
...

This was fixed in the latest release: v2023-11-01.1

This was fixed in the latest release: v2023-11-01.1


[Bug] Memory leak caused by fetch proxy

https://github.com/Yidadaa/ChatGPT-Next-Web/blob/38bffd423c7e2fd0210ecb768a33785c97ab2961/app/api/common.ts#L28-L38
We should use AbortController to destroy this fetch response after 600 seconds, or it will cause memory leak.

Here are some hints:
// setup AbortController
const controller = new AbortController();
// signal to pass to fetch
const signal = controller.signal;

setTimeout(() => controller.abort(), 60 * 1000);

// pass signal to fetch init options

ref: https://stackoverflow.com/questions/31061838/how-do-i-cancel-an-http-fetch-request

fixed by #1604


cube.aggregated_by produces wrong bounds in some cases

Try the following:
import iris
import iris.cube
import numpy as np

cube = iris.cube.Cube(np.zeros((12, 2, 2)), standard_name='wind_speed', units='m s-1')
co_y = iris.coords.AuxCoord([1996, 1997, 1997, 1996, 1997, 1997, 1996, 1997, 1997, 1996, 1997, 1997], long_name='year')
co_m = iris.coords.AuxCoord(['Dec', 'Jan', 'Feb']*4, long_name='month')
co_r = iris.coords.AuxCoord([2, 2, 2, 3, 3, 3, 4, 4, 4, 1, 1, 1], standard_name=u'realization')
cube.add_aux_coord(co_y, 0)
cube.add_aux_coord(co_m, 0)
cube.add_aux_coord(co_r, 0)

print(cube)

means = cube.aggregated_by(['month','year'],iris.analysis.MEAN)
print('means.coord(\'realization\'):\n', means.coord('realization'))

Result is:
>>> print cube
wind_speed / (m s-1)                (-- : 12; -- : 2; -- : 2)
     Auxiliary coordinates:
          month                         x        -       -
          realization                   x        -       -
          year                          x        -       -
>>>
>>> means = cube.aggregated_by(['month','year'],iris.analysis.MEAN)
>>> print 'means.coord(\'realization\'):\n', means.coord('realization')
means.coord('realization'):
AuxCoord(array([ 1.5,  1.5,  1.5]), bounds=array([[2, 1],
       [2, 1],
       [2, 1]]), standard_name=u'realization', units=Unit('1'))
>>>

But we expected

all points == 2 (instead of 1.5)
all bounds == [1, 4] (instead of [2, 1])

Note:
Trying subsets of the data suggests that it is the last 3 realization values being out-of-order which triggers the problem...
>>> means = cube[:9].aggregated_by(['month','year'],iris.analysis.MEAN)
>>> print means.coord('realization')
AuxCoord(array([ 3.,  3.,  3.]), bounds=array([[2, 4],
       [2, 4],
       [2, 4]]), standard_name=u'realization', units=Unit('1'))

>>> means = cube[6:].aggregated_by(['month','year'],iris.analysis.MEAN)
>>> print means.coord('realization')
AuxCoord(array([ 2.5,  2.5,  2.5]), bounds=array([[4, 1],
       [4, 1],
       [4, 1]]), standard_name=u'realization', units=Unit('1'))

#4315 fixes the bounds problem.  The OP also notes that the points are wrong, and I have opened #4455 to more specifically cover that problem.  So I think this can be closed.


Fix PR template.

Fixes PR template by removing 4 empty spaces.

Thank you for your contribution! Follow this checklist to help us incorporate your contribution quickly and easily:

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] If this contribution is large, please file an Apache Individual Contributor License Agreement.

R: @pabloem or @udim


Bug: It is not calling Token end point.

If base url of my application is http://localhost:3000/login and I try to start the authorisation process, it doesn't hit token endpoint and then i am stuck on login screen.
It works fine if my login page is at http://localhost:3000/
Steps To Reproduce

change login page url form http://localhost:3000 to http://localhost:3000/login
Click on login button.

Sharing screenshot for the number calls happening at http://localhost:3000/login

Hi, thank you for reporting this.
I will have a look at replicating this as soon as I can :slightly_smiling_face:

While @soofstad  is attempting to replicate, would you mind sharing your AuthConfig object, @farhanmalhi-dbank?
And maybe some details like whether you change the AuthConfig when changing the login-location from / to /login etc.
Any details might be helpful. Thanks!

@sebastianvitterso this is how my config looks like
const authConfig: TAuthConfig = {
  clientId: '9tVbRm5vKNGmaiVT4mMCJ_3aMWsa',
  authorizationEndpoint: 'https://wso2-idp.local/t/din.global/oauth2/authorize',
  tokenEndpoint: 'https://wso2-idp.local/t/din.global/oauth2/token',
  redirectUri: window.location.origin,
  onRefreshTokenExpire: (event) => window.confirm('Tokens have expired. Refresh page to continue using the site?') && event.login(),
  scope: 'email internal_login openid profile',
  autoLogin: false
};

we are using WSO2 identity server, the urls are not exact but if you want to try i wil create new url for it.

@soofstad  So this is how my file app.tsx and router file looks like and my routes are wrapped with AuthProvider
import React from 'react';
import { Provider } from 'react-redux';
import { PersistGate } from 'redux-persist/integration/react';
import Store, { PersistedStore } from './store/store';
import { QueryClient, QueryClientProvider } from '@tanstack/react-query';
import { AuthProvider, TAuthConfig } from 'react-oauth2-code-pkce';

import { Router } from './router';
const queryClient = new QueryClient();

const authConfig: TAuthConfig = {
  clientId: '9tVbRm5vKNGmaiVT4mMCJ_3aMWsa',
  authorizationEndpoint: 'https://wso2-idp.local/t/din.global/oauth2/authorize',
  tokenEndpoint: 'https://wso2-idp.local/t/din.global/oauth2/token',
  redirectUri: window.location.origin,
  onRefreshTokenExpire: (event) => window.confirm('Tokens have expired. Refresh page to continue using the site?') && event.login(),
  scope: 'email internal_login openid profile',
  autoLogin: false
};

function App() {
  return (
    <div className="App">
      <QueryClientProvider client={queryClient}>
        <Provider store={Store}>
          <PersistGate loading={null} persistor={PersistedStore}>
            <AuthProvider authConfig={authConfig}>
              <Router />
            </AuthProvider>
          </PersistGate>
        </Provider>
      </QueryClientProvider>
    </div>
  );
}

export default App;

this is how my Router file looks Like
export function Router() {
  return (
    <BrowserRouter>
      <Routes>
        <Route path="/" element={<PrivateRoute />}>
          <Route path="/" element={<LayoutComponent />}>
            <Route
              path="/"
              element={
                <ErrorBoundary FallbackComponent={FallBackComponent}>
                  <Dashboard />
                </ErrorBoundary>
              }
            />

          </Route>
        </Route>
        <Route
          element={
            <ErrorBoundary FallbackComponent={FallBackComponent}>
              <Login />
            </ErrorBoundary>
          }
          path="/login"
        />
      </Routes>
    </BrowserRouter>
  );
}

Okey, so it's not what I suggested.
I don't know those <QueryClientprovider> or <PersistGate> components, but they seem to do something with fetches.
Could it be that they intercept/block etc. the token request? Since it's cross-origin.
Could you try moving the <AuthProvider> to be the outermost provider?

@soofstad
Thanks for the help and prompt response, i was able to fix it by making it the outer most component.
Please mention it in your docs as well.
Just one more thing,
So if the refresh token call fails, it should either re-login via sending token call with "authorization key" or should send me to login page.
and i also enabled
onRefreshTokenExpire: (event) => window.confirm('Tokens have expired. Refresh page to continue using the site?') && event.login(),
butt it didn't prompted me before sending refresh token call.
just to test (what happen if refresh token call fails) what i did was
access token expiry is 20sec
refresh token expiry is 5 sec

@soofstad  Atleast it should show the prompt saying "Your refresh token is expired please refresh you page or you will be logged out" or it should redirect to login page.

I won't respond to your entire comment, I'll leave most of it up to @soofstad. But: I'd like to explain how onRefreshTokenExpire works:
The README says the following about onRefreshTokenExpire:

Optional callback function for the 'refreshTokenExpired' event. You likely want to display a message saying the user need to login again. A page refresh is enough.

This function/callback is only called if the session somehow expires while the web page is open. This can happen e.g. if the client loses its web-connection until the refresh-token expires, or in a couple of other cases.
The onRefreshTokenExpire-callback is not called in these cases:

You're opening the page without having an active refresh token.
You've refreshed the page when the refresh token is expired.
The access token has expired, but the refresh token has not. In this case, the library simply calls the api to get a new access token (and possibly renews the refresh token, depending on the auth server's configuration.

Just explaining the exact issue, you can see my token call got error 400, while i was just sitting idle, neither i got any
nor i was sent to login page to re-login.

Thanks, think I understand your issue now.
So the handling of failed refresh requests are not that robust, and might cause this issue in some scenarios.
It's not always known if the request failed because of an expired refresh_token, or something else, as there is no standard error response for authentication servers if you tried to refresh token with an expired_refresh token.
What you will get tho, is an updated error state. You could check that if it corresponds to a known error, and handle login/logout/refresh/whatever yourself.
Just for reference, could you post the error response you get from wso-2?
If this answers you question, feel free to close this issue

@soofstad  if is like this
error: "invalid_grant", error_description: "Expired or Revoked authorization code received from token request"

Retrying with the same expired token will only return the same 400 response.
But I think we can improve upon how these failed refresh requests are handled.
I will create a new issue for that.
Thank you for your feedback and involvement. You can see I updated the README with a "Knows issues" section, and will try and improve the handling for cases like this.
closing the issue :slightly_smiling_face:


Find the accessory view even if it is not contained in an RCTRootView

I've confirmed that this is actually working around a bug (shortcoming?) in React Native 0.59.5.
It has been fixed in React Native 0.60.4.


[roles-0.1.0] Create/edit new hats tree structures

Looking for a function which takes an array of data representing different Hats, and for each Hat an instruction of whether to Create a new hat with that data or Edit an existing hat with that data.
We also want to enforce some business logic (like the tree structure) near this top level input.

[ ] IN CODE ONLY, NO UI YET
[ ] functions which take the minimally necessary amount of data needed to craft a proposal (or, the data to be fed into a proposal), for

[ ] 1: creating a brand new full hat tree, given a Safe address (top hat plus any layers of children). See below for more description
[ ] 2. creating just a top hat, given a Safe address
[ ] 3: creating a tree structure underneath a top hat, given a top hat

Each hat should have

id
details
1 wearer address
Admin

This is complete


MSID: 99833 Version: 2 DOI: 10.1101/2<IP_ADDRESS>8983

MSID: 99833
Version: 2
Preprint DOI: https://doi.org/10.1101/2<IP_ADDRESS>8983
Step 1. Awaiting reviews
Editorial to post reviews via hypothesis
Useful links:

DocMap: https://data-hub-api.elifesciences.org/enhanced-preprints/docmaps/v2/by-publisher/elife/get-by-manuscript-id?manuscript_id=99833
New model tracking: https://docs.google.com/spreadsheets/d/1_fHaoOy7hjyocptKtVJRijeNpUY4hBS7Ck_aVmx6ZJk/
Reviews on sciety: https://sciety.org/articles/activity/10.1101/2<IP_ADDRESS>8983

For trouble shooting (e.g. no Docmaps available):

DocMap issue addressing: https://miro.com/app/board/uXjVNCwK6EI=/
Explore DataHub DocMaps API: https://lookerstudio.google.com/reporting/4c2f0368-babb-4beb-b5b3-497e7e7b0f08/page/ejphD
Unmatched submissions and preprints: https://lookerstudio.google.com/u/0/reporting/9f86204f-3bf7-477c-9b18-5c5ef141bf69/page/p_gxi57ha93c
Unmatched manuscripts spreadsheet: https://docs.google.com/spreadsheets/d/15QcK8w-ssB7109RQEDtFpJPZ0J5HTGxoHa_2TtpMBbg/edit#gid=1336081641

Step 2. Preview reviewed preprint
Production QC content ahead of publication
Instructions:

QC preview: https://prod--epp.elifesciences.org/previews/99833v2
Update ticket with any problems (add blocked label)
When QC OK, add QC OK label to ticket and add publication date and time to https://docs.google.com/spreadsheets/d/1amAlKvdLcaDp5W8Z8g77NmkwbMF5n_u89ArSqPMO8jg
Move card to next column
(At end of the day post link in #enhanced-preprint and ask for PDF to be generated)

Useful links:

Preprint DOI :  https://doi.org/10.1101/2<IP_ADDRESS>8983
Confirm reviews returned by EPP: https://prod--epp.elifesciences.org/api/reviewed-preprints/99833/v2/reviews
To update the MECA path in the docmap: https://docs.google.com/spreadsheets/d/1mctCQuNFBjSn97Lihy7_vBO6z7-N-oqyLv4clyi6zHg

Step 3: Awaiting search reindex
This step adds the reviewed preprint to the homepage: https://elifesciences.org
The search reindex is triggered once an hour. We need the reviewed preprint to be indexed as the search application serves the journal homepage.
Useful links:

Jenkins pipeline to reindex search can be triggered sooner or monitored here: https://alfred.elifesciences.org/job/process/job/process-reindex-reviewed-preprints/

Step 4: Published! PDF requested
Waiting for PDF to be generated
Useful links:

PDF tracking: https://docs.google.com/spreadsheets/d/106_XeDjmuBae7gexOTNzg60lapeqjl2aRn9DzupGyS8/

Step 5: Introduce PDF to data folder and git repo
Upload PDF to relevent folder in git repo https://github.com/elifesciences/enhanced-preprints-data/
Step 6: Done!

[ ] Kettle is on!

Have fixed this title in Kotahi - need to wait for the change to effectuate.

Hi @acollings, the BRE for this one doesn't have an affiliation. Please could you add it in eJP?

Thanks @fred-atherden -- should be sorted

Thanks!


https://googleapis.github.io/google-api-php-client/main/ is 404

Link to reference documentation is wrong.

Thank you for reporting this!
Looks like our docs generation script was not failing when there was an error, and as a result, an empty gh-pages was being published
I've at least fixed the issue so that it won't overwrite our docs site when there's an error (see https://github.com/googleapis/google-api-php-client/commit/7bac7a5b8710962e05c376a40b8216393ce2fe6d), which worked when it was published (see https://github.com/googleapis/google-api-php-client/actions/runs/4495597156/jobs/7909380673)
Now we just need to fix the error itself.
As I've reverted the publishing error in https://github.com/googleapis/google-api-php-client/commit/87daf6dc1b92b4f78d428247b8c200a4c427def1, and ensured it won't happen again in 7bac7a5b8710962e05c376a40b8216393ce2fe6d, this is now fixed.


[Ruby 3.0] Apply auto-corrections made by rubocop for minor issues

Checklist

[ ] I've run bundle exec rspec from the root directory to see all new and existing tests pass
[ ] I've followed the fastlane code style and run bundle exec rubocop -a to ensure the code style is valid
[ ] I've read the Contribution Guidelines
[ ] I've updated the documentation if necessary.

Motivation and Context

Description

Testing Steps

          1 Warning

:warning:
Big PR

  Generated by :no_entry_sign: Danger


Release a version for new download features

At this time, the version on npm is 1.0.0, without new download features. I'm tried use the github version but it seams build fail. Can you release a version with these cool new features?

Only @johanneslumpe can do this at the moment, speaking of which, can I be added as a collaborator on NPM?
In the meantime @demiurga you can directly reference the GitHub repo in your package.json for the latest version.

@cjdell are we set for a 1.1? Is there anything left to test? @demiurga Your build failed with The current version on github?
@cjdell I'll check out how to do that on Npm :) But we should still have some kind of milestone set up and not wildly publish to npm :D

@cjdell @johanneslumpe Sorry, that's my fail. The current version on GitHub works fine :)

@johanneslumpe I think a version 1.1 release is in order. I'm unlikely to be adding any new stuff over the next few days. I might get some time over Christmas to add some requested features for a version 1.2...

@cjdell I will publish that now but I also added you on npm!

@demiurga published as 1.1.0

@johanneslumpe Sorry to reopen this but just noticed the readme on NPM is still the old one. Close this as soon as you've updated. Thanks

@johanneslumpe Showing up now, closing the issue :-)


hotfix to unblock hparams and OmniConf - removes auto_register_init_args by default

@Borda
Let's go back and add the auto_register to whatever demos/examples we need to.
Right now we need this hot fix.
@Darktex @tullie

Hello @williamFalcon! Thanks for updating this PR.

In the file pytorch_lightning/core/lightning.py:

Line 1722:111: E501 line too long (112 > 110 characters)

In the file tests/models/test_hparams.py:

Line 31:5: E306 expected 1 blank line before a nested definition, found 0

@ashwinb @tullie @darkTex you should be unblocked now.
Let us know what are additional tests we can add as we finalize the init api.
I'd like to not have to call auto_register_module_args manually, and instead do it for the user, but let's put the tests cases here to make sure it doesn't break anything.


[SPARK-39981][SQL] Throw the exception QueryExecutionErrors.castingCauseOverflowErrorInTableInsert in Cast

What changes were proposed in this pull request?
This PR is a followup of https://github.com/apache/spark/pull/37283. It missed throw keyword in the interpreted path.
Why are the changes needed?
To throw an exception as intended instead of returning an exception itself.
Does this PR introduce any user-facing change?
Yes, it will throw an exception as expected in the interpreted path.
How was this patch tested?
Haven't tested because it's too much straightforward.

cc @cloud-fan @gengliangwang

Thanks for catching it, @HyukjinKwon !

BTW, it would be great if we could have a test case to capture the non-codegen code patch.

Merged to master and branch-3.3.

Test conflicts. I just removed the test part in branch-3.3.


The datetime format passed to the gateway is not in a standard format

The datetime format passed to the gateway gives things like "Mar 24, 2018 19:32:53". It would be helpful if it were in a standard format, like ISO_8601.

One can use
Gson gson = new GsonBuilder().setDateFormat("yyyy-MM-dd'T'HH:mm:ss.SSS'Z'").create();

to create a gson object and
 Ion.getDefault(this).configure().setGson()

to have ion use it.

Good catch. Will definitely fix this 👍

@io53 while there, please convert the time into UTC timezone or include timezone information.
For server receiving the JSON it is otherwise impossible (or difficult) to decipher what timezone the sender is.

This is fixed and will be included in the next release.


Eliminate excess clone()s of TcpStream

Before a full Session can be created, the server must receive HeartBeat preferences from the server via a CONNECTED frame. To achieve this, the client manually writes out a CONNECT frame and reads in CONNECTED before instantiating the session. In this process, it clone()s the TcpStream to satisfy the compiler. This is unnecessary and kludge-y.

This has largely been OBE due to the migration to mio. The initial connection process is now the only synchronous send/receive in the library and only does a single clone() on the tcp_stream. There may be an argument for making it async and adding a 'ClientState' enum to the mix to manage it, but that's not a priority.


Its working fine if I copy from html but not with docs.

This is my summernote-clearner code
cleaner:{
          notTime: 2400, // Time to display Notifications.
          action: 'both', // both|button|paste 'button' only cleans via toolbar button, 'paste' only clean when pasting content, both does both options.
          newline: '
', // Summernote's default is to use '

'
          notStyle: 'position:absolute;top:0;left:0;right:0', // Position of Notification
          icon: '[Your Button]',
          keepHtml: true, // Remove all Html formats
          keepOnlyTags: ['
', '
', '
', '
', '', '','', ''], // If keepHtml is true, remove all tags except these
          keepClasses: false, // Remove Classes
          badTags: ['style', 'script', 'applet', 'embed', 'noframes', 'noscript', 'html'], // Remove full tags with contents
          badAttributes: ['style', 'start'] // Remove attributes from remaining tags
    	}

This work fine when i copy from html and paste it to summernote editor. But if i copy from doc it does not event paste anythong but shows msg 'text is cleaned'.
If i remove summernote-cleaner than i can copy paste from doc but I want it form both doc and html.
Any help would be appreciated. Thanks

Try it without adding the options when initializing the plugin. If your using the option icon: with the setting as you've shown above, the plugin could be not working correctly.


[CARBONDATA-3188] Create carbon table as hive understandable metastore table needed by Presto and Hive

Problem:
Current carbon table created in spark creates the hive table internally but it does not have much information like schema, input/output format and location details. So other execution engines like Presto and Hive cannot read the table.
Reason:
Spark always checks in HiveSerde static map whether it is a hive supported table or not, since carbon is not registered to that map it cannot create hive understandable table. It justs creates a table without schema and location and adds its own schema as part of properties.
Solution:
Add the carbon details also to HiveSerde static map so that it can create Hive understandable table.
Be sure to do all of the following checklist to help us incorporate
your contribution quickly and easily:

[ ] Any interfaces changed?

[ ] Any backward compatibility impacted?

[ ] Document update required?

[ ] Testing done
Please provide details on
- Whether new unit test cases have been added or why no new tests are required?
- How it is tested? Please attach test report.
- Is it a performance related change? Please attach the performance test report.
- Any additional information to help reviewers in testing this change.

[ ] For large changes, please consider breaking it into sub-tasks under an umbrella JIRA.

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1854/

Build Failed  with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1855/

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1856/

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/10111/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2064/

@ravipesala This pull request doesn't work in #3012 .HiveExample run exception through your changes.But this update does work in beeline.Therefore will you check the cause of HiveExample exception with me？

@SteNicholas This PR does not work directly on the Hive integration as we need to set the right serde, inputformat and output format. We are planning to refactor the current Hive integration code to unify the inpuformat and outputformat with the hadoop modules input/outformats.  This PR is basic step for both Hive and Presto integration.

@ravipesala Could I refactor the current Hive integration code with you?

Build Success with Spark 2.1.0, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder2.1/1961/

@jackylk OK.I'd like to refactory on carbondata-hive module based on @ravipesala update.

Build Success with Spark 2.3.2, Please check CI http://<IP_ADDRESS>:8080/job/carbondataprbuilder2.3/10213/

Build Success with Spark 2.2.1, Please check CI http://<IP_ADDRESS>:8080/job/ApacheCarbonPRBuilder1/2234/

LGTM


React Helmet

Is your feature request related to a problem? Please describe.
N/A
Describe the solution you'd like
I would like to add React Helmet to dynamically change the title and metadata of each Route.
Describe alternatives you've considered
A simple document.title = 'New title'; placed in the componentDidMount() method of each component that is routed can change the title, but the metadata is a little more complex to change.
Additional context
N/A

Cleaned up old issues


Performance Monitoring & Log Aggregator collectors in Hygieia Dashboard

@satishc1 @MarkRx @tabladrum
Hi Team,
Hygieaia documentation mentions about Performance Monitoring integration as well.
Where can I find that collector and which Performance monitoring tools are being integrated with the same. (For e..g. NewRelice/ Dynatrace etc)
Also, can I have any reference to have any log aggregation integration with Hygieia (for E,g Splunk)

Any thought on this ?
@satishc1 @MarkRx @tabladrum

There are none. @tabladrum was looking at an AppD integration but I don't know what became of that.

i am also looking for these collectors with hygieia. please update if its available for any monitoring solution now.

i need it too! anyone found a solution?

Performance metrics were implemented for Appdynamics (hygieia-perf-appdynamics-collector) and Log aggregation was implemented for AWS cloudwatch (hygieia-cloud-cloudwatch-collector) . For other tools we are open to contributions by taking the mentioned repositories as reference.


Training data and testing data for Caffe Model

Greetings Authors,
I have a few questions on the training data set for the Caffe Model. After going over your code, it seems like all modified coordinates from LSMS were used in creating the downloaded_locs.txt, and downloaded_locs.txt is used by extract_features.py and thus becomes the test set before it could be used regression. To simplify, I am concerned if training set and testing set were mutually exclusive? If yes is it possible for you to share the training coordinates for each country?
Thanks,
Vinit

Hi Vinit,
The Caffe model was trained on a separate dataset of satellite images and corresponding nighttime light labels - this training dataset was sampled from all over Africa near DHS survey locations and is described in this paper.
Hope that is helpful!

Hi Neal,
Thank you for replying. I read through the paper and from what I understand, the  330,000 images training images came by randomly sampling near DHS survey locations. And to illustrate this you mark these locations as white. However, due to an imbalance in the dataset w.r.t night light intensity, you create a 3-class balanced dataset consists of 150,000 training images ( 50k per class) and 8,000
validation images.
And from the LSMS the survey, for a given group, you sample approximately 100 1km×1km images tiling a 10km × 10km area centered at the average household location as input.
So, to summarize,
Training set :
Size: {Xi, Yi} x 150,000
Xi : is the daytime image taken near a DHS survey location
Yi :  night light intensity bin corresponding to the same location as Xi
** Validation set**
Size: {Xi, Yi} x 8,000
Xi : is the daytime image taken near a DHS survey location
Yi :  night light intensity bin corresponding to the same location as Xi
**Test set for **
Size: {Xi} x (Number of [image_lat] and [image_lon] coordinate in downloaded_locs.txt genrated from LSMS for a given country
Xi : is the daytime image taken at [image_lat] and [image_lon] at zoom level 16
Yi: Nightlight bin predicted by the network
Is my understanding here correct? And if yes, while I understand it might be reasonable to assume that locations of the DHS training images and LSMS test images are not the overlapping much since they are taken from two different sources, what I cannot understand is how is the location for calculating DHS asset score are different from the training set. And what happens to remaining 172000 (  330,000 - 150,000 - 8000) images from your training set. Do you simply discard these images?
Again, if possible and available it would be extremely helpful if you could share the training locations since I am trying to replicate your experiments with a different network and I don't want different locations to be a nuisance factor while comparing my result with yours.
Thank you,
Vinit

@sangmichaelxie do you have the training locations for the nightlights task?
@vinitveersingh I think there is a slight misunderstanding for the test set. After we train the CNN to predict nighttime lights, we use it as a fixed feature extractor for the poverty prediction task. What that means is that for each of the LSMS locations, we sample ~100 images from the 10x10 square and then run those through the CNN to get the 100 corresponding feature vectors. We then average those feature vectors to get one feature vector for each LSMS (or DHS) cluster, and use those features for the ridge regression model.

Hi Neal,
Thanks for the correction. And thanks for asking @sangmichaelxie for the training locations.

Good Afternoon Neal,
Were you or Michael able to find the file containing the locations you used for training your network? Will it be possible to share it?
Thank you,
Vinit

I don't have the file

Hi Vinit,
We don't prefer to share the training locations. We've actually tried many different architectures and different samples of training sets - the method is robust to this. Note that the Google Static Map images have also updated since our 2016 paper, so even if you use the same locations, the images will not be exactly the same. So the original locations wouldn't result in the same dataset.

Neal and Michael thanks for answering my questions.
Sincerely,
Vinit


Added bitbucket api package

The package does not fulfill all the requirements mentioned here: https://github.com/chiraggude/awesome-laravel/blob/master/CONTRIBUTING.md


[TextField] type=number missing min, max, step Typescript types

[x] I have searched the issues of this repository and believe that this is not a duplicate.

Expected Behavior

<TextField type="number" min="0" max="10" step="1" /> should compile with Typescript
Current Behavior

Receive following Typescript error for min, similar for max and step: [ts] Property 'min' does not exist on type 'IntrinsicAttributes & TextFieldProps & { children?: ReactNode; }'.
Your Environment

Tech
Version

Material-UI
1.0.0-beta.21

React
16.0.0

Typescript
2.6.1

Does this work?
<TextField type="number" inputProps={{ min: "0", max: "10", step: "1" }} />

Yes that works. inputProps accepts an object with anything in it, though, so I'm effectively opting out of the type system by doing it this way.

Yes that works. inputProps accepts an object with anything in it, though, so I'm effectively opting out of the type system by doing it this way.

Good point, I've opened #9321 to fix the typing of inputProps.

inputProps is intended to allow for anything goes on an InputComponent.  This allows for open integration with external components, for which unfortunately we may not know or have types.
Example:
import * as MaskedInput from 'react-text-mask'

      <TextField
        InputProps={{
          inputComponent: MaskedInput,
        }}
        inputProps={{
          guide: false,
          mask,
          placeholderChar: '\u2000',
        }}
        type="tel"
        value={value}
      />

So the alternative to avoiding strongly typed inputProps in the same scenario (thanks @pelotom) is
import { InputProps as MuiInputProps } from 'material-ui/Input'

      <TextField
        InputProps={{
          ...InputProps,
          inputComponent: (inputProps: MuiInputProps) => (
            <MaskedInput {...inputProps} guide={false} mask={mask} placeholderChar="\u2000" />
          ),
        }}
        type="tel"
        value={value}
      />

@pelotom is filing a new proposal/issue to revisit this API.

This is a really bizarre change! In version 22 and before, to apply a "step" property to an input we needed to add this prop:
inputProps: { step: '0.5' }
I think that was already confusing because it was not clear why some of native properties were directly available on TextField (like name and type) and some where not (like step, min, max). If I wanted to make things more intuitive/less confusing, I'd make TextField act more like a native element, so users can intuitively get things to work. Something as simple as:
step={0.5}
However, in the new release beta.23, the props should be set like this:
InputProps={{ inputProps: { step: '0.005' } }}
Based on what I read the reasoning behind this change is to eliminate confusion. IMHO this is several times more confusing, and in reality users should keep digging documentation and copy/paste this every time thy need it. Of course I'm a single user and have one vote! Others may think differently. I appreciate your great work on this amazing library regardless.

@rosskevin
MaskedInput is not defined,
mask is not defined
:(

This seems like an overly complicated workaround

It would be better to define these at the TextField level, so that validation can be performed without a bunch of repeated logic everywhere.  I'd hate to have to extend TextField every time I want to use it for a number field with min/max/step requirements.

@pelotom , @oliviertassinari , @rosskevin :
If the numbers are entered as input as shown in the image it is easy to bypass the control.
Is there a solution?
Link: codesandbox

@Angelk90 Maybe you can control the value prop and apply the limit.
Related to #19154

this doesn't work when you type something.

How to write float value in TextField as input in material ui component for multiple input field ??

@Himanshudabhi34  Can you Please provide more details? What do you mean by float value in textfield?


backgroundColor attribute from mat-tab-group tag disable default ripple effect of mat-buttons inside mat-tabs tags

What is the expected behavior?
Ripple effect is activated on button click
What is the current behavior?
Rippe effect is not activated on button click
What are the steps to reproduce?
StackBlitz starter: https://stackblitz.com/edit/angular-ytvijf
Which versions of Angular, Material, OS, TypeScript, browsers are affected?

Angular: 7.0.2
Angular Material: 7.3.1
OS: Ubuntu 18.04
Typescript: 3.1.1
Browsers: Chrome 71

Is there anything else we should know?
When open stackblitz, find tab-group-basic-example.html file on /app directory

Closing as a duplicate of https://github.com/angular/material2/issues/14819.


datasource for series are not updating

What TMM version are you using?
tmm_3.0-SNAPSHOT_4fd66bc_win.zip
release, pre-release, nightly, or directly from GitHub/branch?
nightly
What is the actual behaviour?
After add a new datasource folder for series the filter datasources don't update their available datasources.
What is the expected behaviour?
Update datasource folders in filter for series without restart tmm to force an update.
Steps to reproduce:

add a new datasource folder for series
update sources
add a second new datasource folder for series
update sources
in filter -> datasources are not present

Additional
Your Operating system? (win/mac/linux? version?)
Windows 10 x64

should be working again in the next build


A subtle typo ..

[Enter feedback here]

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

ID: 38e8321f-5516-e451-5429-750d49f0ebed
Version Independent ID: 1fd093e9-f47c-ffdd-91e9-5039c3ccde61
Content: Publish events that conform to .NET Guidelines - C# Programming Guide
Content Source: docs/csharp/programming-guide/events/how-to-publish-events-that-conform-to-net-framework-guidelines.md
Product: dotnet-csharp
Technology: csharp-fundamentals
GitHub Login: @BillWagner
Microsoft Alias: wiwagn

Hello @siddiq-habib! Can you specify where exactly do you see the typo, or at least in which section/part of this page?


How do I view training metric values for each iteration?

Describe the bug
Trying to build a ranking model using LightGBMRanker in pyspark but can't see any training metrics. I.e. even if I set isProvideTrainingMetric to True and verbosity to 2, I can't see the map or ndcg on the training or validation sets. If I calculate those metrics by hand before and after training, the reranking after training causes a drop in the training metrics.
If I have early stopping rounds set to 5, why is the model building 70 trees if map or ndcg are not going up?
** Details **
python3.6.9
spark 3.0.0
Building MML spark from source

I am having the same problem with mmlspark_2.11-1.0.0-rc3 running on spark 2.4.7 and python 3.6

I am having the same problem with mmlspark_2.11-1.0.0-rc3 running on spark 2.4.7 and python 3.6

@cadama turned out the metrics are found in executor's stderr. I was using Yarn.

@cadama turned out the metrics are found in executor's stderr. I was using Yarn.

Thanks @OscarDPan. I still think is a bit unpractical to fetch them one by one from the executors. Maybe some logs aggregation would be nice here.

Thanks @OscarDPan. I still think is a bit unpractical to fetch them one by one from the executors. Maybe some logs aggregation would be nice here.


Can't drag to reorder tracks in track group

Only tracks in a pinned group or the scrolling group can be reordered by dragging and dropping. Any thread that is in a collapsible or expandable track group cannot be dragged for reordering.

While investigating https://github.com/flutter/devtools/issues/5242, I discovered that this issue is not specific to the Flutter DevTools embedding, but actually is a bug with the Perfetto web app itself.
When themoveTrack action is triggered, it is only applied to pinned tracks and scrolling tracks. I think this action needs handling to apply the move function to tracks within a track group as well. @chromy @LalitMaganti
https://github.com/google/perfetto/blob/master/ui/src/common/actions.ts#L424-L425

@stevegolton can we mark this as wontfix: I think our attitude here is that we no longer want to support any reordering at all in workspaces, is that correct?

Closing: I'm pretty sure we don't want to support this. We only want to support reordering within pinned tracks.


Sroll and Swipe commands not working on RN app version 0.71.7

Slack channel concersation link : https://mobile-dev-inc.slack.com/archives/C041FU72T54/p1689073684767359
This is currently a blocker for us to run the tests on the pipeline.

Hey, @kavithamahesh can you provide more details:

A sample app/flow reproducing the issue
What exactly is happening when you say not working?
Can you share it separately for android / iOS

Also following environment information:

Maestro version [e.g. v1.2.3]
Platform: [e.g. iOS 16.4.1, Android 11.0, Chrome]
Device model and OS version: [e.g. iPhone6]
Simulator, emulator or physical device
Host [e.g. Mac M1, Windows, Linux, etc]

Closing because of lack of response. Please feel free to open a new issue with all the required info filled out.


JSON.stringify vs immutable data structure

I see the library is using JSON.stringify which is probably a bit slow : https://github.com/FormidableLabs/urql/blob/3f859ed42d7a2ef2bff74c03e5245444425bd429/src/modules/client.ts#L116
Can't you do something like that:
const memoize = (fn) => {
 let prevArgs;
 let prevResult;
 return function(args) {
    if(args === prevArgs)
       return prevResult;
    prevArgs = arg;
    return prevResult = fn.call(this, arg);
 }
};

Instead of
const memoize = (fn) => {
 let cache = {};
 return function(args) {
 let hash = JSON.stringify(args); /* slow */
 if(hash in cache)
   return cache[hash];
 return cache[hash] = fn.call(this, arg);
 };
};

So you mean memoize the stringification?

I mean replacing the JSON.stringify and hash is in cache by a reference comparison using immutable data structures. https://youtu.be/rtcn9I9sB5M

What does this mean for external cache storage?

So the previous points remain unaddressed, the most important one being: the query is a string and variables are rarely huge, so the result is that the number of fields and paths are low. JSON.stringify is also by no means slow but an efficient O(n) which is even preferred in this case.
This can potentially be enhanced by caching a hash for the query and only doing JSON.stringify on the variables or doing a custom recursive & partial combined murmurhash.
Either way, I'll tag this to be addressed in the future

I'm a fan of stringify(), just .02. I find it helps keep functions and undefined properties out, and has the replacer function that comes in handy sometimes. I had to use it recently to clean some __typename stuff out of a mutation when using apollo client. I've been working with cache/localStorage also as mentioned above, so I've been using JSON more.

We likely won't use stringify in the future, but we've now released v1 with the same caching strategy, which is just the default one. The keys are also still the same.
That being said, there won't be a data structure to preserve queries, but I'm thinking more of hashing queries directly, still stringifying variables, and just setting IDs on queries when they're parsed DocumentNodes.
But for now, I'll close this since it's not quite up-to-date with v1.


Create Pricing Card 0 Blue

One need to make this component using HTML and Tailwind CSS. I would suggest to use Tailwind Playgrounds to make things faster and quicker.
Here is a reference to the component.

Follow figma design and to make the button use Button Design System
Make the component responsive too.
After building the component please raise a PR with a screenshot of the component and add the component in Pricing Components/src/pricing-card-0-blue.html
If you need to use any icon please use it from Hero Icons
Good luck.

I would like to contribute.

JWOC

I want to contribute on this issue please assign it to me


Add simple adopters pages

For Flux v1 and Flux v2, driven by yaml files.
Fixes: #129
Needs #176 to be landed first.

Needs https://mikefarah.github.io/yq/ ...

This is not very sophisticated yet, but see https://github.com/dholbach/website/blob/fix-129/adopters/flux-v2.yaml from which https://609019180b1d6800085b92f0--fluxcd.netlify.app/adopters/flux-v2/ is generated. It uses a modified form of https://www.docsy.dev/docs/adding-content/shortcodes/#card-panes which we get through #176 .
It'd be nice to simplify https://github.com/dholbach/website/blob/fix-129/Makefile#L49-L51 - it's required for the Netlify build.

How do you feel about https://github.com/dholbach/website/tree/fix-129/adopters#readme - is this too complicated?

I feel like this is getting somewhere. Things to be figured out still:

[ ] Page description not working yet
[ ] Size of logos
[ ] No checking of paths
[ ] carousel on main page

Do we already want to advertise the pages even if we haven't added companies yet?

Let's move the main-page carousel to a separate PR.

From now onwards, I'll need help making this pretty. :grin: :art: :sparkles:

Cards are the same size, I need help centering the logos now. :grin:

Using card-deck with 5 entries per row. Fixes same size problem. Maybe good enough for now?

Minus adding logos for everyone, the adopters page looks good in general!
I do see something strange on a small viewport – on iPhone in portrait mode, cards are in a single line. In landscape mode, they alternate between rows of 3 and rows of 2. This could be iterated on though, as IMO it looks pretty good to start.
One thing – do we want to add some direct link to this adopters page from the main fluxcd.io landing page as part of this PR?

Thanks for the feedback!

Minus adding logos for everyone, the adopters page looks good in general!

Once it's up, we'll ask folks to add their logos.

I do see something strange on a small viewport – on iPhone in portrait mode, cards are in a single line. In landscape mode, they alternate between rows of 3 and rows of 2. This could be iterated on though, as IMO it looks pretty good to start.

Yes, agreed - this is a known issue I'll need help with. Getting this into the shape it is was already a quite a bit of work (for me :innocent: )...

One thing – do we want to add some direct link to this adopters page from the main fluxcd.io landing page as part of this PR?

Yes, if we feel it's good enough, I'm happy to add it. In the Project dropdown?

I added it to the navbar under Project.


Provide an icon config when compile exe

Thank you for taking the time to file an issue!

Dart SDK Version (dart --version) Dart SDK version: 2.12.1 (stable) (Wed Mar 10 10:18:47 2021 +0100) on "windows_x64"
Whether you are using Windows, MacOSX, or Linux (if applicable) Windows
Whether you are using Chrome, Safari, Firefox, Edge (if applicable)

I want to add an icon to a dart console app, but I tried using rcedit, resourcehacker, none of them works, see https://github.com/electron/rcedit/issues/91#issuecomment-802449008 for more details. Maybe the exe compiled from dart is not an ordinary exe like msvc, rust build, I tried to add icon to exe build from rust, it is ok.
I hope the dart compile could support this feature built-in.

It seems fixed in 2.17.0-246.0.dev.


Chore: migrate repack native modules to turbo modules

Summary
This PR aims to enable newArch and make re.pack native modules to turbo modules while ensuring newArch is implemented with backward compatibility.
re.pack current implementation directly uses RN native modules, but with this change, there is a SPEC implementation that allows for more efficient communication between native and JavaScript. This change allows for both old and new architecture.

Test plan

Make sure you are currently using the newArch and the setup conforms to React Native Doc directives which also means you're on RN>=0.68.0

NOTE: TesterApp currently meets this requirement if you're using repack TesterApp.
For Android

You will only need to update your android/gradle.properties file as follows:
newArchEnabled=true

and then yarn TesterApp:start

For iOS

Run pod install with the flags:
RCT_NEW_ARCH_ENABLED=1 pod install

and then  yarn TesterApp:start

Screen Records

newArch - iOS

https://user-images.githubusercontent.com/22779249/232512294-4ada8342-949c-41ab-811f-6fc7a1b2fe52.mov

oldArch - iOS

https://user-images.githubusercontent.com/22779249/232516328-1b7c5a0e-82ba-44d7-94e6-7367e206e8a4.mov

newArch - Android

https://user-images.githubusercontent.com/22779249/232515374-b566a38e-538b-482d-bb94-35fc09483043.mov

oldArch - android

https://user-images.githubusercontent.com/22779249/232516407-e830662d-b5d2-46e6-a2b3-a2f604618f0e.mov

Overall it looks good 🎉  I have added a few comments
cc @thymikee @jbinda

I don't have much experience when it comes to New Architecture so if @troZee says it's looking good from the configuration perspective I trust his judgment :)
However, one thing that I don't like is that the native implementation of the prefetchScript, loadScript, and invalidateScripts methods is duplicated, thus making the maintenance cost higher in the future. Is this expected when writing backward-compatible modules? That would be a weird design TBH.

Yes, that's the implementation for backward compatibility. There's a common module with the actual implementation of prefetchScript, loadScript, and invalidateScripts within the ChunkManagerModuleImpl.kt file. The only difference between the old and new arch is how these methods are exposed, where the new arch uses override for methods, the old arch uses the @ReactMethod. And that's the only time we use the old or new arch files.

Hi, any ETA on this improvement? Thanks!

Closing in favour of https://github.com/callstack/repack/pull/495

Thank you @teneeto for your all work on this PR 🙏


Minio Chart

A basic Minio chart based off the quickstart guide

A single export backed by a configurable PV
Default Access Key and Secret that can be overridden

@k8s-bot ok to test

Thanks @hunter!! Worked like a charm this time around.


Use JetBrains markdown instead of commonmark?

Looks like the binary size of JetBrains markdown is smaller. Would be nice if we can port that to ij-parsing-core

https://github.com/JetBrains/markdown/blob/master/src/commonMain/kotlin/org/intellij/markdown/ast/ASTNode.kt

What markdown parser does fleet use?

They don't have a front matter parser


Set default JVM memory settings for the build

Will be picked up when building a sub-module
Doesn't work with --file option (MNG-5889)

Rat again: [ERROR] Failed to execute goal org.apache.rat:apache-rat-plugin:0.11:check (default) on project brooklyn-library: Too many files with unapproved license: 1 See RAT report in: /home/jenkins/jenkins-slave/workspace/brooklyn-library-pull-requests/target/rat.txt -> [Help 1]

Thanks @sjcorbett. Fixed in https://github.com/apache/brooklyn-server/pull/436.


NSLGameScanner.py fails to add game shortcuts when using separate prefix for each launcher

After starting fresh, and selecting to use separate prefixes, the script is failing to add game shortcuts into steam.
It was working before when I did not have separate prefixes. The launcher shortcut itself was added and is working.
Here are the logs for the script:

['/home/deck/.config/systemd/user/Modules', '/home/deck/.config/systemd/user', '/usr/lib/python311.zip', '/usr/lib/python3.11', '/usr/lib/python3.11/lib-dynload', '/usr/lib/python3.11/site-packages']
['/home/deck/.config/systemd/user/Modules', '/home/deck/.config/systemd/user/Modules', '/home/deck/.config/systemd/user', '/usr/lib/python311.zip', '/usr/lib/python3.11', '/usr/lib/python3.11/lib-dynload', '/usr/lib/python3.11/site-packages']
Service started.
['/home/deck/Downloads/NonSteamLaunchersInstallation/lib/python3.11/site-packages', '/home/deck/.config/systemd/user/Modules', '/home/deck/.config/systemd/user/Modules', '/home/deck/.config/systemd/user', '/usr/lib/python311.zip', '/usr/lib/python3.11', '/usr/lib/python3.11/lib-dynload', '/usr/lib/python3.11/site-packages']
Epic Games is not installed. Skipping.
Existing shortcut found based on shortcut ID for game Gog Galaxy. Skipping creation.
Updated CompatToolMapping entry for appid:<PHONE_NUMBER>
Ubisoft Connect is not installed. Skipping.
Battle.net is not installed. Skipping.
EA App is not installed. Skipping.
Amazon Games is not installed. Skipping.
itch.io is not installed. Skipping.
Legacy Games is not installed. Skipping.
Humble Bundle is not installed. Skipping.
IndieGala Client is not installed. Skipping.
Rockstar Games Launcher is not installed. Skipping.
Glyph is not installed. Skipping.
Minecraft: Java Edition is not installed. Skipping.
Playstation Plus is not installed. Skipping.
VK Play is not installed. Skipping.
Xbox Game Pass is not installed. Skipping.
GeForce Now is not installed. Skipping.
Netflix is not installed. Skipping.
Hulu is not installed. Skipping.
Disney+ is not installed. Skipping.
Amazon Prime Video is not installed. Skipping.
Youtube is not installed. Skipping.
Amazon Luna is not installed. Skipping.
Twitch is not installed. Skipping.
movie-web is not installed. Skipping.
Fortnite is not installed. Skipping.
The app ID for Gog Galaxy is<PHONE_NUMBER>
Epic Games: EpicGamesLauncher does not exist
Gog Galaxy: GogGalaxyLauncher exists
Traceback (most recent call last):
File "/home/deck/.config/systemd/user/NSLGameScanner.py", line 525, in
os.rename(current_path, new_path)
IsADirectoryError: [Errno 21] Is a directory: '/home/deck/.local/share/Steam/steamapps/compatdata/GogGalaxyLauncher' -> '/home/deck/.local/share/Steam/steamapps/compatdata/3216372511'

@PseudoAnonMS this still needs to be fixed I believe

I also used the prefix thing to install Battle.net. It is failing to detect diablo 4 as a shortcut which unfortunately is a bummer

@Neoxon4865 yeah that scanner doesn't have full SD card support yet...meaning it will detect your games if you DONT use the separate prefixes still working on the scanners slowly but surely. It should pick it up if you choose one prefix.

@Neoxon4865 yeah that scanner doesn't have full SD card support yet...meaning it will detect your games if you DONT use the separate prefixes still working on the scanners slowly but surely. It should pick it up if you choose one prefix.

Yeah I was having issue trouble getting steam deck to recognize the battle net launcher on steam OS (steam launcher in window recognized it well). So I tried to reinstalled it using the separate app id check box and it finally recognize it under steam OS under non steam games. So maybe if I reinstall battle net as non-separate app id, albeit it hasn’t work. But I am hoping this will get resolve soon but so far I only use battle net for diablo 4 so it doesn’t matter much.

@Neoxon4865 when did you try this installing battle net? as of today the launchers seem to all be installing and putting shortcuts in steam, are you using the Desktop Version or the Decky Plugin version? as far as it finding games...it should find the games if you have installed them in the default location of Battlenet, i havnt made it to look for other areas in the system yet.

@moraroy So I got my steam deck today (yay!) and I did start installing Battle.net thru your program in desktop mode. It wasn’t appearing at all as a desktop shortcut until I did the seperate app ID trick I found in another issues. What I do is launch battle.net thru steam in Game Mode, then I pressed to install diablo 4. Then it still doesn’t show up after launching the game and restarting the steam multiple time, and also launching non steam launcher to restart the auto finding service. I could try the decky version but tbh downloading 50gbs of diablo 4 3 times today sucked so I will hear your input on what I could do better or did wrong.

@Neoxon4865 you didn't do anything wrong brother I don't have that game specifically..but I do have overwatch I'll re install and see what's going on, in the mean time , on this github repo look at the NSLGameScanner.py file. It will have the battle net scanner in there.

@Neoxon4865 you also don't need to reinstall anything to use the plugin! They work togather!


them-[red,blue,green].css have a small issue

Kindly pointing to:
https://github.com/matcornic/hugo-theme-learn/issues/111
for details.

@vjeantet was this fixed in docdock? There's so many commits in the referred issue....


QUESTION: Alternatives for anti aliasing effect using WebGL and shapes

We are struggeling getting good visual results with PIXI.Graphics using the WebGL renderer and we are writing for some guidance/tips/ideas on how we can improve the visual result.
Our use case requires

visually appealing edges of shapes and lines
beeing able to render the shapes and lines on screen
beeing able to extract the resulting stage of shapes and lines to an image (with visual quality maintained)
animating the scale, position and rotation of individual shapes and lines

Using CanvasRenderer the graphics are antialiased by default with good enough visual result. With the WebGL Renderer, however, the resulting graphics have jagged edges in some cases.
After reading up on the issue I believe I understand that there are quite some aspects to consider regarding anti-alias and WebGL rendering in PIXI:

anti-aliasing can be enabled on render level at a cost (the antialias option passed to the Renderer constructor)
the PIXI WebGL implementation of Graphics uses the stencil buffer which does not support anti-aliasing
applying filters to a Graphics object cancels the render level anti-alias since secondary frame buffers do not support anti-aliasing
extracting a display object or the whole stage using PIXI.extract does not apply the render level anti-alias
there are several alternative "work-arounds" for getting "smoother" edges without the renderer level anti-alias

FXAA filter on renderer level or on individual display objects
Blur filter on individual display objects
Rendering Graphics objects using canvas renderer and using it as a texture
Rendering Graphics objects at a higher resolution and downscaling it using linear scaling
Rendering the whole stage at a higher resolution and downscaling it using linear scaling

I made an example in PIXI Playground (https://www.pixiplayground.com/#/edit/WMr27ZIwI3RWRotaap00L) where I have experimented with different ways of drawing the graphics, different ways of extracting a frame from the rendered view and also tested different settings for forceCanvas, antialias and forceFXAA. Hopefully these examples can serve as help for others and also serve as basis for further discussion.
Based on the experiments and the fact that we also require anti-alias on graphics that are animated (scale and rotation) we have come to the conclusion that we will pay the performance price of using render level antialias, not allow the application of filters on Graphics objects and use HTMLCanvasElement.toBlob() to extract a particular frame from the animation.
All that said, we would appreciate some input from someone with more experience in WebGL and PIXI to give us a hint on if any of the statements above are incorrect, if we are on the right track with our conclusions or if we have missed or misjudged some alternative altogether.
We are also very curious on if there are any plans of adapting the implementation of PIXI.Graphics to WebGL2 where I understand it could be made to support antialias on Graphics display object level and on render level together with filters. Or if there are any other plans of improving the visual results of using PIXI Graphics with the WebGL renderer?
Environment

pixi.js version: 4.8.7 and 5.0.3
Browser & Version: Chrome 74.0.3729.157
OS & Version: Windows 10
Running Example: https://www.pixiplayground.com/#/edit/WMr27ZIwI3RWRotaap00L

Need MSAAx4 ,.its available in weblgl2 but pixiv5 does not support multisample framebuffers yet. That way we'll have the same quality as Flash.
For downscaled graphics to look the best need a least x3 (means x9 area) , but then it requires advanced downscaling because just linear filtering works only with scale less than 2.
Bear in mind that canvas2d antialias is worse than msaa x4 because its alplied PER PATH, it has transparent exges between two touching paths! Even svg in browser is bad, look at google doodle for US veterans day. Somehow nvidia pascal videos show different result on svg, something about directx enabling msaa automatically. Its fun that we need Pascal to get quality that already existed in Flash for many years.
My personal choice is using 2d for non animated shapes to generate mips and changing sprite texture to certain mip level based on scale log2. If my algo detects that shape is basic enough, it uses plain pixi graphics. There are also special cases for holes gradients and masks. Cant explain it in a few words, im on vacation, but my take on vector graphics with full support of SWF files will be published in autumn.

I can make a guess that you are making something serious. In this case its good if you contact me in t.me/hackerham and we can see whether i can give you some of my experimental code that wasnt published yet.

Thanks for your response @ivanpopelyshev, it is very apreciated.
I interpret it as that as long as PIXI does not support MSAAx4 it is hard to get better results (visual and performance) than our suggested solution.
Looking forward to seeing MSAAx4 in PIXI in the future and also to your work on vector graphics. The "full support of SWF files" (including animation?) is a work without MSAAx4 or is it something that requires MSAAx4? Please let me know how I can find that work when it is finished!

hmm, rust
https://nical.github.io/posts/a-look-at-pathfinder.html

Closing the issue...


Chartpy - Empty chart

Hi there,
Firstly, thank you for three excellent projects. Only recently have I tried plotting a chart using Chartpy. And this has been unsuccessful. I am struggling to figure out why the chart output is not being rendered. I get a PNG and matplotlib display that is blank. The logs are:
2019-08-10 17:38:28,860 - findatapy.market.datavendorweb - INFO - Request AlphaVantage data
2019-08-10 17:38:30,496 - findatapy.market.datavendorweb - INFO - Completed request from Alpha Vantage for ['VUSA.open', 'VUSA.high', 'VUSA.low', 'VUSA.close', 'VUSA.volume']
2019-08-10 17:38:32,513 - findatapy.market.ioengine - WARNING - Couldn't push MarketDataRequest_527__abstract_curve_key-None__alpha_vantage_api_key-x__category-None__category_key-backtest_default-cat_alphavantage_intraday_NYC_VUSA__cut-NYC__data_source-alphavantage__environment-backtest__expiry_date-NaT__fields-open_high_low_close_volume__finish_date-2019-08-10 16:38:27_377636__fred_api_key-x__freq-intraday__freq_mult-1__gran_freq-None__quandl_api_key-x__resample-None__resample_how-last__start_date-2019-08-03 16:38:28_406628__tickers-VUSA__trade_side-trade__vendor_fields-open_high_low_close_volume__vendor_tickers-VUSA_L to Redis: Error 10061 connecting to <IP_ADDRESS>:6379. No connection could be made because the target machine actively refused it.
2019-08-10 17:38:32,749 - matplotlib.legend - WARNING - No handles with labels found to put in legend.
I would really appreciate any thoughts or suggestions.
Thanks
Fig

@lefig have you checked the dataframe is not empty? Eg. you can print it beforehand

Hi @saeedamen ,
Thank you for coming back to me. I am still investigating but am at present unable to download any data. I get a strange error...
File "c:\work\quant\python\data\findatapy\findatapy\market\datavendorweb.py", line 496, in load_ticker
data_frame = data_frame.to_frame().unstack()
File "C:\work\python\3.6.7\python-3.6.7.amd64\lib\site-packages\pandas\core\generic.py", line 5180, in getattr
return object.getattribute(self, name)
AttributeError: 'DataFrame' object has no attribute 'to_frame'
And the code responsible:
if run_example == 1 or run_example == 0:
# downloaded S&P500
md_request = MarketDataRequest(
    start_date="decade",            # start date
    data_source='yahoo',            # use Bloomberg as data source
    tickers=['Apple'], # ticker (findatapy)
    fields=['close', 'open'],       # which fields to download
    vendor_tickers=['aapl'],   # ticker (Yahoo)
    vendor_fields=['Close', 'Open'])# which Bloomberg fields to download)

df = market.fetch_market(md_request)

Yahoo changed their API, so pandas_datareader, which broke findatapy's Yahoo downloader. I've rewritten the Yahoo downloader in findatapy to use yfinance (pip install yfinance). If you remove findatapy and install the latest version (with yfinance too), it should work.


API Deprecate GraphQL

Issue https://github.com/silverstripe/silverstripe-admin/issues/1799

Everything be removed was only intended for the admin schema

unit test failures should be resolved by https://github.com/silverstripe/silverstripe-versioned/pull/464


Apps: add thumbnail page that has all apps together

Currently we have apps and their vizs being saved, and links back to them. We'd like a page that holds all of the apps and has all of their vizs in thumbnail view.
Mockup

s
The above mockups assume we will have a compute service which would supply info like "Status", "Started at", etc, so we don't have to worry about those yet.
Major pieces to get functional:

[ ] Table of contents/List of apps on the left.
[ ] New app/module button. Goes to add-new-app page
[ ] App instance section that includes app name, icons for copy/configure/delete, a New Visualization button, and app instance's thumbnails.

Not worried about styling in this ticket, just function :)

Right now i'm thinking these would be the routes:

/visualizations/: shows thumbnail page that has all apps and vizs. Also has + new app button.
/visualizations/new/: shows list of possible apps to add. The view we get when clicking on the Visualize tab in the feature site currently.
/visualizations/new/{app.name}/ : app configuration. Nothing to change here
/visualizations/computationId/new: add a new visualization to that app. Should land on the page that shows viz options, so for example "Scatterplot" and "Boxplot" options (NewVisualizationPicker). Not the page that has the + new app button. Nothing to change here.
/visualizations/computationId/visualizationId: the full page visualization view (FullScreenVisualization). Nothing to change here.

Not set in stone! Let me know if there is an issue with the above!

This all sounds good to me!

Deployed to mbio feature site so moving to Ready For Testing!


⚠ Samsung has degraded performance

In 7bd112b, Samsung (https://samsung.com) experienced degraded performance:

HTTP code: 200
Response time: 2849 ms

Resolved: Samsung performance has improved in 2eae69e.


fix(deps): Update dependencies

:tada: This PR is included in version 4.0.1 :tada:
The release is available on GitHub release
Your semantic-release bot :package::rocket: