Audio Category always set to 6 (AudioCategory_GameEffects) when using AudioGraph class

Hi,
I was looking into AudioCreation and AudioCategory samples and I saw a different behavior of both classes (AudioGraph and MediaElement) when trying to set AudioCategory properties of the stream played.
In fact when using ETW on Microsoft_Windows_Audio provider, you can check which 'category' audio engine is using:
GetMixFormat starting on endpoint [{<IP_ADDRESS>00000}.{1764586d-288d-48ae-96bd-6929ae98598d}] (category=6, raw=False, matchformat=False, connector=0)
When using AudioCreation->File Playback, we can see in code that 'Media' category is used:
AudioGraphSettings settings = new AudioGraphSettings(AudioRenderCategory.Media);
But in ETW log you see that audio engine selects 'category = 6' which means AudioCategory_GameEffects (from https://msdn.microsoft.com/en-us/library/windows/desktop/hh404178(v=vs.85).aspx). I tried to change audiocategory to any of possible value, I always see in ETW value 6.
When using AudioCategory sample, you can see that 'category' field in ETW log is following the audiocategory set in the Application (I verified each value, it always match)
Is there any parameter missing in AudioGraph use to let Audio Engine to select correct category ?

Thank you for reporting this. You've just found a platform bug. The value isn't being passed down correctly from AudioGraph into XAudio. We're opening an internal bug to track, and as this isn't an issue with the sample itself, I'll close this report. Unfortunately, there's not a good workaround I can offer at this time, but thank you for letting us know so we can get it addressed.

I understand, do you have an idea when this will get fixed ? Or any tracking number that allows me to know when this will be fixed ?

I currently expect this to be fixed for the Anniversary Update.


Update embedded shariff version

Hi,
Shariff has moved on and provides several new services.
Please update the embedded copy to get access to them.
Thanks,
James

Hi,
I updated it yesterday in the evening and created a corresponding pull request just minutes ago.

Hi!
First of all, thanks for your interest in this theme! It's with heavy heart that I have decided that I can no longer maintain this theme due to lack of time and changing interests. To makes sure the theme lives on however and not goes to waste, I've donated it to the Pelican community. It can now be found in the official pelican-themes repository. I'd suggest you re-submit your issue  there.
Sorry for the inconvenience!


CUDA version

Hi, can I ask what is the cuda version for this repo?

IIRC the experiments were run with CUDA 8, PyTorch 0.4.1.


Updates v1.28 hugo.toml for 1.31 release

Updating configuration files (hugo.toml) for 1.31 release.
/hold for v1.31 release day.
cc: @drewhagen @chanieljdan @salaxander

I believe the params.versions section for v1.26 should be removed since this is for the 1.31 release

/approve

/unhold

Merging for the 1.31 release


[SPIKE/RESEARCH] Async NamedHivePartitionSensor

Acceptance Criteria:
Document possible options and selection reasons for a particular library in this GitHub issue via a Summary comment.
Find a Python library that supports async implementation for NamedHivePartitionSensor if the official library does not support it.
Ensure that connection is set up and working

The implementation pattern for this sensor is very similar to HivePartitionSensor
Tested the sync version example
We have decided to use impala


add FetchRelatedNotes argument to ListVersionedResourceOccurrences RPC

Codecov Report

Merging #90 (5f83fc6) into main (64600a3) will increase coverage by 0.29%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #90      +/-   ##
==========================================
+ Coverage   87.09%   87.39%   +0.29%
==========================================
  Files           8        8
  Lines        1015     1039      +24
==========================================
+ Hits          884      908      +24
  Misses         91       91
  Partials       40       40

Impacted Files
Coverage Δ

server/server.go
85.36% <100.00%> (+0.59%)
:arrow_up:

Continue to review full report at Codecov.

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


refactor(settings): move settings button to navbar & make floating menu

Closed because in v2.0 there will be a new settings menu popup


map about multi-gpu and single-gpu

i use retinanet_r50.
i use my own dataset.
when i train it on single 1080,map(iou=0.5) is 0.32.(lr=0.00125)
but when i train it on 2*2070,map(iou=0.5) is 0.24.(lr = 0.0025)
is it normal?
can you give me some suggest？
thank you.

may be you can check your bn setting

Please follow the Error report issue template to provide further information.


Use spawner with --params-file argument instead of cli verbs

This repo is now in line with the ros2_control_demo repo, especially https://github.com/ros-controls/ros2_control_demos/pull/502
Fixes #395
TBD: xml launch files are still not working

@mergifyio backport jazzy


shields.io status page not working

--> https://status.shields.io/

Hello @tooomm ! 👋🏻
Thanks for raising. I'm aware (https://github.com/badges/shields/pull/6327#issuecomment-838065081), it's on my list of things to fix. 😉

UptimeRobot no longer supports custom domains as part of its free plan. It's probably not worth upgrading to Pro just for that feature, let's use the standard URL instead: https://stats.uptimerobot.com/PjXogHB5p


new stable version of iwg-vpn

A major update? Any breaking change a user should be informed about or such?

No breaking changes, nothing a user has to do. The set of features and stability have reached the grade of the v1. That's the reason.


fix: resolution of references to .js files in initial index.d.ts

Fix #32
@lukasbach

Thanks for the contribution!


I am facing issue while running my test cafe script

I am able to run my test cafe script locally, if I want to test on other environments like dev or stage then it takes URL after loading the URL it won't return any result it will be just blank

Thank you for your inquiry. This issue looks like a question that would be best asked on StackOverflow - an amazing platform for users to ask and answer questions.
We try to keep the GitHub issues tracker for bugs and feature requests only (see also: Contributing). If you think that this issue is a bug, feel free to open a new issue, but please make sure to follow the bug template. Thank you in advance.


Add blockquote style

Blockquotes are currently unstyled, you can't tell which are which in e.g. here:

Random Codecov fail? Rerunning ...


Explore distribution of SICs

Find the distributions of SICs as well as we can.
This may inform a classifier approach where we train a classifier to predict the most common SICs.

There are 2,566,102 job adverts which aren't of type "Recruitment consultancy" in the full OJO dataset
Of these, there are 31,127 unique company names
When you apply the company name cleaning function, this gives 29,854 unique company names
We query these cleaned company names in the cleaned Companies House dataset, and there is exactly 1 match for 15,184 of these companies. We don't want to use the times when there are >1 match since it's harder to know we've been correct.
These companies are from 409,310 job adverts
These companies are associated with 610 unique SICs

The following plots are using those 610 SICs and proportions are out of the 409,310 job adverts.
The proportion of job adverts with each SIC has a long tail and 90% of the job adverts are from 91 SICs.

The 47 SICs which account for 80% of the job adverts are shown here:

The data behind this plot is (SIC: prop job adverts):
{'70100 - Activities of head offices': 0.12711392343211747,
 '78109 - Other activities of employment placement agencies': 0.07677799223082749,
 '73110 - Advertising agencies': 0.05648286140089419,
 '74990 - Non-trading company': 0.05536146197258801,
 '82990 - Other business support service activities n.e.c.': 0.05345092961325157,
 '78200 - Temporary employment agency activities': 0.03531308787960226,
 '86101 - Hospital activities': 0.03395226112237668,
 '86900 - Other human health activities': 0.030654027509711464,
 '41100 - Development of building projects': 0.029845349490606143,
 '49410 - Freight transport by road': 0.020329334734064647,
 '70229 - Management consultancy activities other than financial management': 0.0191908333536928,
 '77110 - Renting and leasing of cars and light motor vehicles': 0.016681732672057855,
 '64205 - Activities of financial services holding companies': 0.014639270968214801,
 '62012 - Business and domestic software development': 0.013598495028218222,
 '85590 - Other education n.e.c.': 0.012814248369206714,
 '96090 - Other service activities n.e.c.': 0.012301189807236568,
 '47300 - Retail sale of automotive fuel in specialised stores': 0.01222789572695512,
 '64209 - Activities of other holding companies n.e.c.': 0.010744912169260462,
 '88100 - Social work activities without accommodation for the elderly and disabled': 0.010251398695365371,
 '47770 - Retail sale of watches and jewellery in specialised stores': 0.010207422247196502,
 '61900 - Other telecommunications activities': 0.009249712931518897,
 '74909 - Other professional, scientific and technical activities n.e.c.': 0.009078693410862183,
 '71129 - Other engineering activities': 0.009051818914758985,
 '87300 - Residential care activities for the elderly and disabled': 0.009010285602599496,
 '62090 - Other information technology service activities': 0.007146172827441304,
 '47190 - Other retail sale in non-specialised stores': 0.007028902298990985,
 '68320 - Management of real estate on a fee or contract basis': 0.006889643546456231,
 '43999 - Other specialised construction activities n.e.c.': 0.006850553370306125,
 '64191 - Banks': 0.006738169113874569,
 '68310 - Real estate agencies': 0.005993012631013169,
 '45111 - Sale of new cars and light motor vehicles': 0.005636314773643449,
 '87900 - Other residential care activities n.e.c.': 0.005531259925240038,
 '85530 - Driving school activities': 0.0048178642105005985,
 '55100 - Hotels and similar accommodation': 0.004812977938481835,
 '56102 - Unlicensed restaurants and cafes': 0.0048032053944443084,
 '88990 - Other social work activities without accommodation n.e.c.': 0.004681048593975227,
 '62020 - Information technology consultancy activities': 0.004649287825853265,
 '85600 - Educational support services': 0.004480711441205932,
 '64999 - Financial intermediation not elsewhere classified': 0.004346338960689942,
 '52103 - Operation of warehousing and storage facilities for land transport activities': 0.004124013583836212,
 '87100 - Residential nursing care facilities': 0.004018958735432802,
 '56101 - Licensed restaurants': 0.003921233295057536,
 '45310 - Wholesale trade of motor vehicle parts and accessories': 0.003845496078766705,
 '47710 - Retail sale of clothing in specialised stores': 0.0037551000464195843,
 '66220 - Activities of insurance agents and brokers': 0.003664704014072463,
 '68209 - Other letting and operating of own or leased real estate': 0.0034985707654345117,
 '81100 - Combined facilities support activities': 0.0032909042046370724}

Dataset outputs/data/green_industries/all_ojo_most_common_sic.csv

35 SICs account for 75% of the job adverts
I get the job advert ids for those job adverts which come from companies linked to these SICs
These job adverts ids are linked to 308,856 job adverts in the company info dataset
These job adverts ids are linked to 261,354 job adverts with descriptions
The following is outputted:


[Feature Request]  Hiding Enchantment Glint unless hurt

Show Enchantment Glow when hurt (basically the same as the armor setting) this would allow people to simulate a magical aura protecting them in combat but not active at all times.
Thank you for your time

This would be an interesting feature for sure, but because of how the ui is structured right now, it'd be difficult to implement in an intuitive way.
I probably wont add this.


Update language around new file

@digitarald @lychung7
@lramos15 @JacksonKearl I am adding an action "select a different editor" in the untitled hint text. its meant to replace the untitled text file with a "rich editor"... The available command is a picker but only creates a new editor. Do either of you have guidance on how to properly "transform" the existing editor. Do I need to just delete the tab and recreate it?

If I misunderstand the question let me know.
This sounds like what we do with the binary editor detection case which is we show the quickpick to choose editor type then it gets replaced. This is done via the editor service by passing in an override of PICK. I can grab you a code pointer if you want to go that route. You will definitely need to replace the current editor if picking a different editor type but that shouldn't result in data loss since once you start typing the hint text goes away.

@lramos15 yea that sounds right, a code pointer would be helpful

Code pointer: https://github.com/microsoft/vscode/blob/4cd32948551642e6898057f4fc8a4590cf876f43/src/vs/workbench/contrib/files/browser/editors/binaryFileEditor.ts#L48-L98


pull request for e2e test at 11/19/2016 11:26:06

:white_check_mark: Validation status: passed

File
Status
Preview URL
Details

optest/docs/updated_at.md
:white_check_mark:Succeeded

For more details, please refer to the build report.
Note: If you changed an existing file name or deleted a file, broken links in other files to the deleted or renamed file are listed only in the full build report.


Integrate distilling

@preethamgali wrote a model distilling framework here which we should aim to integrate into GPT-NeoX

@preethamgali and I discussed this on discord. We do want to use the GPT-NeoX modeling framework and to capture as much of the optimizations that our code provides.
What I had in mind was to make the student model the last stage(s) in a pipeline, so that instead of having T1 -> T2 -> T3 -> T4 and S1 -> S2 you have a single model T1 -> T2 -> T3 -> T4 -> S1 -> S2. Then when you do backprop you just stop after finishing the student model. The teacher and the student models likely will have different widths though, and I’m not sure if that’ll do anything wonky.
@sdtblck suggested running something like
    teacher = GPT2ModelPipe(**kwargs)
    student = GPT2ModelPipe(**student_kwargs)
    ...

    teacher, optimizer, _, lr_scheduler = deepspeed.initialize(
            model=teacher,
            optimizer=optimizer,
            ...)
    student, _, _, _, = deepspeed.initialize(
            model=student,
            optimizer=optimizer,
            ...)

However I worry that DS will not play nicely with multiple models. This is purely conjecture though, and Sid’s suggestion is absolutely worth trying. Or more realistically, asking DS people about.

The cross_entropy loss function used in the framework is mpu.vocab_parallel_cross_entropy, which has implemented to work on 3D parallelism. But for distillation, we need KLDivLoss, MSELoss, CosineEmbedding loss functions. So we also need to implement the same for these losses as well. Please raise the feature request.

We are abandoning this effort as unsuccessful


Update Metadata + Screenshots of Activities- GCI task

@llaske Can you review the changes as well.

I have done the changes as you said and I run the activities in sugar before taking screenshots

Thanks for the updates.
I'm worried.  Sugarizer doesn't need these changes.  These changes are for Sugar.  We've not heard from @llaske yet on this pull request, so I'm not sure if this effort is useful.
Reviewed.  Comments;

[ ] the files are named .PNG instead of .png, please rename them, git mv,
[ ] you've linked the screenshots metadata to your own repository; this will break once you delete your repository, and it would enable your control of the screenshots without review,
[ ] the screenshots and description metadata keys must have a corresponding metadata license key, see https://github.com/sugarlabs/sugar-toolkit-gtk3/blob/master/src/sugar3/bundle/init.py
[ ] you made many separate commits instead of one, causing many GitHub mail notifications to be sent to everyone subscribed to notifications; probably because you used the GitHub editor, which is best used with private projects; please learn to use git command line tools.

Hope that helps.

Sigh.

I'm agree with @quozl , this update don't make sense for Sugarizer.

Thanks.
@Gabbar8055, @vipulgupta2048, should you wish to maintain these activities for Sugar, please take on the Sugar activity maintainer responsibility; i.e. for each activity, (a) create a repository in sugarlabs/, (b) add the files copied from Sugarizer, (c) add necessary metadata, (d) test, stabilise, and release.

@Gabbar8055 How did you run the Foodchain activity in sugar?


Add support for Okta SSO

Feature Suggestion
We want to be able to use Okta with Backstage since that's the identity provider backing our SSO.
Possible Implementation
Looks like there are possible modules available that might facilitate this integration:
https://github.com/techstars/passport-okta-oauth
https://github.com/jaredhanson/passport-openid

I can work on this if it's still available

Thanks @nspirrello -- assigned you! 💯

#1455 PR for this feature

Fixed by #1455!


Error in working with local proxy

⚠ Please verify that this question has NOT been raised before.

[X] I checked and didn't find similar issue

🛡 Security Policy

[X] I agree to have read this project Security Policy

📝 Describe your problem
After updating to version 1.23.12, an error was found (Cannot set properties of undefined (setting 'valid') when working with local proxies
Rolling back to version 1.23.11 solved this problem.
I'm launching it in the docker.
📝 Error Message(s) or Log
No response
🐻 Uptime-Kuma Version
1.23.12
💻 Operating System and Arch
Debian (docker)
🌐 Browser
Chrome
🖥 Deployment Environment

Runtime: Docker 26.0.1
Database:
Filesystem used to store the database on:
number of monitors:

Had the same problem.

I am also experiencing this issue.

Same problem here.

Add me to the list... very annoying, since I have to use the proxy in order to make IPv6 funtion at all under Docker.

What do you mean concretely when you say "when working with local proxy"?

As for me, I monitor a https URL which resolves to an IPv6 address and thus I use a "local" (as in: my own RFC1918 network) squid proxy which is reachable via IPv4.

Not what I was asking: where does the error occur?

On the dashboard log and in the container log as well:
2024-04-19T12:55:17+02:00 [MONITOR] WARN: Monitor #28 'Internet (IPv6)': Failing: Cannot set properties of undefined (setting 'valid') | Interval: 60 seconds | Type: http | Down Count: 0 | Resend Interval: 0 2024-04-19T12:55:19+02:00 [MONITOR] WARN: Monitor #32 'Internet': Failing: Child inaccessible | Interval: 60 seconds | Type: group | Down Count: 0 | Resend Interval: 0 2024-04-19T12:56:17+02:00 [MONITOR] WARN: Monitor #28 'Internet (IPv6)': Failing: Cannot set properties of undefined (setting 'valid') | Interval: 60 seconds | Type: http | Down Count: 0 | Resend Interval: 0 2024-04-19T12:56:19+02:00 [MONITOR] WARN: Monitor #32 'Internet': Failing: Child inaccessible | Interval: 60 seconds | Type: group | Down Count: 0 | Resend Interval: 0

Using gluetun, I make myself a local proxy server.

Что конкретно вы имеете в виду, когда говорите "при работе с локальным прокси"?

What do you mean concretely when you say "when working with local proxy"?

Using gluetun, I make myself a local proxy server.

@CommanderStorm : You were right about line #536 in 'server/model/monitor.js'. When I comment that out, everything works again. I just added a comment to your review...

I think I understand how the error can occur, but strangely I can't reproduce this using a squid proxy connecting over HTTP. What is the protocol used connecting to the proxy when the error occurs?

I think I understand how the error can occur, but strangely I can't reproduce this using a squid proxy connecting over HTTP. What is the protocol used connecting to the proxy when the error occurs?

HTTP

I use http to connect to the Squid proxy and the monitored URL is https. I think it is exactly this which causes the problem, as I wrote here: https://github.com/louislam/uptime-kuma/pull/4630#discussion_r1572640951

Okay, I'll implement the fix later today.


fps decrease when loaded in players

Fedoraware Version Date:
current one
Config used:
https://files.catbox.moe/z095uq.fed
Description:
please help i get a major fps decrease when loading in a player model with my config on even when i turn off the esp chams and glow it still lags compared to the default config and not loaded in player models
my config on without loaded player models

default config with loaded player models

my config on with loaded player models

i dunno if this happens to others cus i have patata pc

i also just realized that when holding a melee lowers it down more

the melee weapon thing idk, maybe swing pred is running all the time even if aimkey isnt pressed and that is lowering fps? idk
for the players being rendered in, this will always drop fps. how much you drop is relative to how good the code is behind chams(DME), esp etc. and we all know fedhak >>>>>>>> so no more complain pls

our chams arent in dme sadly.
also aimbot is always doing something regardless pf whether you are holding the aimkey down or not


Develop prototype of automated Software Onboarding

[x] Submit on SABR Grassroots program
[ ] Automate dependencies vulnerability scan
[ ] Automate licence compliance
[ ] Obtain Office 365 accounts and assess capabilities (@gabrielcossette)

When working on this need to use CATS 13671 code
Note: Accessibility out of scope for now
Link with issues #126 and #218
Onboarding process described here. Look at the 19 policies the process looks at complying.
Grassroot idea submission here

Curent wireframes for software onboarding here:
https://github.com/sara-sabr/ITStrategy/blob/master/drawio/en/SoftwareOnboarding.drawio

Presentation at ARC: Late February
Presentation at OCMC: TBD

The prototype can be found here PR: SW-O prototype #610


Use body caching

@Raynos

lgtm.

What does body caching do?

allows multiple callers of body() to parse the request. if the body of the message has previously been parsed, it'll use that copy.

@mranney

The http sharding logic needs to peak into the body.
Ringpop needs to parse the body to proxy
Sirvice needs to parse the body to do json validation and stuff.

because multiple seperate modules all need the body and its a single stream we are using body caching so that body(req, res, { cache: true }, cb) works if called multiple times.
Caveat it will only work if called sequentially, we dont enable parallel caching.


Outlook add-ins: Is there a limit for the the user list?

How many users can we have in the -UserList since we cannot add distribution groups?
Document Details
⚠ Do not edit this section. It is required for docs.microsoft.com ➟ GitHub issue linking.

ID: c23be67f-a704-0f14-40df-6c174c8914e8
Version Independent ID: ac2fc769-a7db-01e0-2590-68af5cfbbadd
Content: New-App (ExchangePowerShell)
Content Source: exchange/exchange-ps/exchange/New-App.md
Service: exchange-powershell
GitHub Login: @chrisda
Microsoft Alias: chrisda

@yobougaz @dariomws, in looking at the UserList parameter description in Set-App, the limit is 1000 users.
I'll add this information to the UserList parameter description in New-App.
Thanks for the feedback.
Closing

@yobougaz @dariomws, in looking at the UserList parameter description in Set-App, the limit is 1000 users.
I'll add this information to the UserList parameter description in New-App.
Thanks for the feedback.
Closing


Fix typescript definition file

Is it a bugfix ?

Yes

Is it a new feature ?

No

Describe what you've done:
Correction and description of type definitions for use with TypeScript
How to test it ?
Yes, in my App  with TS version

I'm not sure whether the API has changed.
Some of the type definition of this PR doesn't match the @1.5.13


[FR] Parameter Choices 5000 char limit is a problem

Please verify that this feature request has NOT been suggested before.

[X] I checked and didn't find a similar feature request

Problem statement
I'd like to have the choice list exceed the current 5000 character limit to allow for an arbitrary length list of options.
I have a use case for the selection of over 1000 options.
Suggested solution
Parameter choices could be saved to another table instead of a comma separated string.
Drop down should be searchable by the user.
Using something like: https://www.w3schools.com/howto/howto_js_filter_dropdown.asp
Describe alternatives you've considered
Can't find and alternative within the current framework
Examples of other systems
No response
Do you want to develop this?

[ ] I want to develop this.

@reid-p with that many options, mapping it to another table makes sense. Would you be willing to either implement this, or fund development?

Removing the limit is not an option from a security standpoint.

@SchrodingersGat
I could possibly implement this, but it'd probably take me a while to get up to speed with the code base.
Funding the development is also a possibility.
@matmair
What's the security implication with having a longer list?
Isn't it the same as an array of parts or any other item?

Funding the development is also a possibility.

@reid-p I'd be happy to take a look at this, the "fund this issue" button at the top of this issue would be the best way to support it!

Arbitrary length text fields that get parsed (processing of the choices) can be used easily for resource exhaustion or as part of a denial of service attack

This has now been addressed with the addition of the "Selection List" concept for parameters values


Need some clarification regarding Reference encoder architecture

@yanggeng1995 On paper author mentioned on ReferenceEncoder (section 2.2 ) the output of the GRU layers passed through two separate Fully connected layers, but in this implementation, last GRU state passed to two separate FC layer
def ReferenceEncoder(inputs, input_lengths, filters, kernel_size, strides, is_training, scope='reference_encoder'):
    with tf.variable_scope(scope):
        reference_output = tf.expand_dims(inputs, axis=-1)
        for i, channel in enumerate(filters):
            reference_output = conv2d(reference_output, channel, kernel_size,
                      strides, tf.nn.relu, is_training, 'conv2d_{}'.format(i))

        shape = shape_list(reference_output)
        reference_output = tf.reshape(reference_output, shape[:-2] + [shape[2] * shape[3]])

        #GRU
        encoder_outputs, encoder_state = tf.nn.dynamic_rnn(
           cell=GRUCell(128),
           inputs=reference_output,
           sequence_length=input_lengths,
           dtype=tf.float32
        )
        return encoder_state

As you see encoder_state return instead of encoder_output .
On the other author mentioned in the same section that they used same reference_encoder as used in GST tacotron , and I go through the best gst-tacotron implementation on github i.e. https://github.com/syang1993/gst-tacotron
here also reference encoder returned encoder_output and it working fine.
def reference_encoder(inputs, filters, kernel_size, strides, encoder_cell, is_training, scope='ref_encoder'):
  with tf.variable_scope(scope):
    ref_outputs = tf.expand_dims(inputs,axis=-1)
    # CNN stack
    for i, channel in enumerate(filters):
      ref_outputs = conv2d(ref_outputs, channel, kernel_size, strides, tf.nn.relu, is_training, 'conv2d_%d' % i)

    shapes = shape_list(ref_outputs)
    ref_outputs = tf.reshape(
      ref_outputs,
      shapes[:-2] + [shapes[2] * shapes[3]])
    # RNN
    encoder_outputs, encoder_state = tf.nn.dynamic_rnn(
      encoder_cell,
      ref_outputs,
      dtype=tf.float32)

    reference_state = tf.layers.dense(encoder_outputs[:,-1,:], 128, activation=tf.nn.tanh) # [N, 128]
    return reference_state

Interesting thing is that on GST tacotron paper author mentioned to use last GRU state as the reference embedding.
Please take note and clarify weather to take encoder_output or encoder_state as the output of reference_encoder.
Thanks

Sorry, I am on the Spring Festival holiday, and it is a little late to see the news， I use encoder_state as the output of reference_encoder,  but the paper did not specify whether to use state or output, this need to be verified by experiments,  and "On paper author mentioned on ReferenceEncoder (section 2.2 ) the output of the GRU layers passed through two separate Fully connected layers," you can find implementation on here:
def VAE(inputs, input_lengths, filters, kernel_size, strides, num_units, is_training, scope):
with tf.variable_scope(scope):
outputs = ReferenceEncoder(
inputs=inputs,
input_lengths=input_lengths,
filters=filters,
kernel_size=kernel_size,
strides=strides,
is_training=is_training)
    mu = tf.layers.dense(outputs, num_units, name='mean')
    log_var = tf.layers.dense(outputs, num_units, name='vari')
    std = tf.exp(log_var * 0.5)
    z = tf.random_normal(shape=[tf.shape(mu)[0], num_units], mean=0.0, stddev=1.0)
    output = mu + z * std
    return output, mu, log_var

Sorry, I am on the Spring Festival holiday, and it is a little late to see the news， I use encoder_state as the output of reference_encoder,  but the paper did not specify whether to use state or output, this need to be verified by experiments,  and "On paper author mentioned on ReferenceEncoder (section 2.2 ) the output of the GRU layers passed through two separate Fully connected layers," you can find implementation on here:
https://github.com/yanggeng1995/vae_tacotron/blob/b0288f1caa776a98195dd94d1e8ea7ca6ec05f57/models/modules.py#L5

I feel should be use state, I am more confused is where style embedding should be added, from the perspective of the figure of papers, style embedding probably should add in the text embedding? I am doing the experiment recently, and the good result will be sent out


Interfaces refactor & fix Solidity function name conventions

Following the Solidity best practices, I refactored the interfaces as separate .sol files and implemented the mixedCase naming for functions (https://docs.soliditylang.org/en/v0.8.7/style-guide.html#function-names). I've also updated the libkeccak.js test file accordingly.

I don't think we are ready for this yet, but at some point I'll match naming conventions. Also, it adds 50 lines. I understand interfaces like that might be standard, but I think they are so verbose and ugly.


It seems some strings are still missing in the extracted globals.ini

First at all, nice idea to create a repo for this.
And! Nice work from @ROBdk97 with the german translation so far! (Some string may need a better one, but damn... everything seems to be translated so far as possible).
What I saw in a short test is, that some string seems to be missed at all (I didnt find them in the file at all):

FYI: The missing namens in the Main menu friendlist is no bug, I removed them for privacy reasons with AI :) .
May will be a good idea to keep a track in the repo of missing strings, then we may can add them.

Thank you for bringing this to my attention. I'm currently working on addressing the issue. I've also noticed the problem and have made corrections to some translation errors. Additionally, I've identified and fixed issues with entries like ~mission(SOMETHING).

In the file, the German version has 68952 lines, while the English version has 68949 lines. I'm unsure about the discrepancy of the extra 3 lines in the German version.

In this goal, I'm working on a "Status Checks" on PR to check for any missing translations

I think we still miss the Release for my changes in Pull Request #5

Indeed, released! 🚀
(Also looking for a better way to automate releases)

@syntafin Would you like to Close the Issue or is there anything else?

For the moment we can close this one :)


⚠ Bofrost.at has degraded performance

In 7b3ac11, Bofrost.at (https://www.bofrost.at) experienced degraded performance:

HTTP code: 200
Response time: 1637 ms

Resolved: Bofrost.at performance has improved in b5a66f5.


Ensure ETL::Master only runs once per date

In order to ensure that we do not have duplicate
Dimensions::Date objects in our database.

@pmanrubia Thanks for the review Pablo, I have addressed the issues!


code cleanup of .java files in entire codebase and inclusion of .xml based on Google style standards

Uses .xml file based on Google Java Style standards to standardize the formatting for .java files in Lucille. XML is included as part of repo now so that new developers of Lucille will automatically have the style as part of their IntelliJ project.
Tickets involved:
LC-308
LC-312

Javadoc is currently being formatted by the xml, looking into possible solutions for how to get the xml to ignore javadoc / leave it as is

Javadoc is currently being formatted by the xml, looking into possible solutions for how to get the xml to ignore javadoc / leave it as is

Found solution for this, there exists an Enable_javadoc_formatting boolean that can be set to false. Will need to redo the entire formatting with this option in the xml and also check in a new xml with this option

Changed made to accommodate for this, todo task might be to look into how to standardize javadoc across the repo into html tags so that it can easily be formatted


Make initialisation of Gibbs sampler standalone

We've agreed that we can have a specific sampler who's responsible for sampling from prior.

[ ] fix this inside HMC step


[RFE] make control plane key configurable

Currently, via https://github.com/kubeflow/kfctl/pull/123 , a control-plane label is applied to namespaces where kubeflow is running. While this is well-intentioned (to prevent the application of kubeflow webhooks), it unfortunately causes collisions with other solutions.
This issue is a request for enhancement to make the key (control-plane) configurable.
An alternate suggestion would be to make the fixed key prefixed in some way. eg: kf-control-plane.
At this time, the control-plane label is causing an issue with knative serving deployments (no issue has been filed for that yet).

Upvoting it.
In Azure that label also blocks Istio Injection for Kubeflow namespace because of https://github.com/Azure/AKS/issues/1771
Whenever I use kfctl to apply some changes I need to remove the control-plane label from Kubeflow namespace and rollout restart all the deployments in the kubeflow ns.

FYI @kubeflow/wg-manifests-leads


draft: Add tooltips for shortcuts

This currently only adds tooltips for zooming, because I wasn't sure what's the best way to do this, given the different localizations. I don't think the shortcut names need to be localized, it should be added somewhere separate. Not sure if the current solution works for different languages.
Fixes: #4279

Unfortunately there's a few problems with doing something like:

This will be immediately overridden when localization runs, since you're only updating the default fallback values.
Doing this "properly" will invalidate lots of l10n-strings, causing a huge amount of re-translation.
It'll not correctly take different platforms into account, since in general many Firefox shortcuts differ between platforms when it comes to the modifier keys used (note e.g. the differences between Windows and MacOS).

Hence hard-coding shortcuts this way simply doesn't work unfortunately, and assuming that we actually want to try and fix that old issue a more "dynamic" solution would probably be required here; hence closing this since I don't believe that'll work well for the reasons outlined above.

I know this isn't the right way to do this, but I made this PR to get some feedback and discussion on how to do this :)
I don't think it's good to change the l10n-strings, because it takes a lot of re-translation, as you said. I did not consider the differences between shortcuts on different platforms, but I have an idea how to solve this. What if we made files similar to the l10n-strings, but for shortcuts on different platforms.
So for example:

shortcuts/Windows/viewer.properties
shortcuts/Mac/viewer.properties
shortcuts/Linux/viewer.properties

These files would contain something like this:
shortcuts/Windows/viewer.properties
zoom-in-shortcut="Ctrl++"
zoom-out-shortcut="Ctrl+-"
...

shortcuts/Mac/viewer.properties
zoom-in-shortcut="⌘++"
zoom-out-shortcut="⌘+-"
...

I don't know how the l10n strings are implemented, so I don't know if this would be feasible.

I don't think it's good to change the l10n-strings, because it takes a lot of re-translation, as you said.

Unfortunately there's, generally speaking, no "proper" way to implement something like this without some invalidating of existing l10n-strings. Note that it's likely necessary for localizers to be able affect how the shortcut is displayed, but not the actual shortcut itself, as part of the translation.

What if we made files similar to the l10n-strings, but for shortcuts on different platforms.

I'm not sure how well (or even if) the webL10n-framework that we use would handle loading multiple files like that, and the way that the Firefox PDF Viewer handles localization definitely wouldn't be able to deal with that without (potentially) a bunch of work.
Furthermore having separate files per platform just increases the risk of them diverging over time (since generally only the modifier keys vary between platforms), and placing the shortcut definitions "outside" of the viewer-code could also increase the risk that things aren't kept up-to-date properly.

Please see the proof-of-concept linked from https://github.com/mozilla/pdf.js/issues/4279#issuecomment-1262448565, which is probably the smallest/simplest way that this could be addressed (assuming that's still wanted) without too many downsides or too much complexity.


MultiBarChart: Stacked by default & single x, y value

Hi i am using the stacked multibar chart with a single x, y value. the chart renders but i notice that the width of the bar is huge where the width is stretched from beginning of x axis to the point of 'x' instead having a single bar the point (x,y)
COnsole shows the following error :  attribute transform: Expected number, "translate(undefined,0)". in d3.min.js

BTW my xaxis is  datetime Axis &is formatted as follows    tickFormat: function (d) { return d3.time.format('%m/%d %H:%M')(new Date(d)); },
Does anyone know how to fix this?

Figured this out - the chart options i originally had xDomain defined from a start to end time which was causing the error. Also the width of the bars is controlled by the # of data points in my case i had only a single data point which was causing the width to be huge.
http://plnkr.co/edit/NS9IRaEoNN5Xwwoc9VMN?p=preview


Incorrect  switch statement execution in go

package main

func main() {
	s := "f64"
	switch s {
	case "i32":
	case "i64":
	case "f32":
	case "f64":
	default:
		panic(s)
	}
}

go version go1.11.1 darwin/amd64
The default branch is triggered, then panic.
It's ok in browser.


Not all dimensions shown

The extension shows "only" 12 dimensions, can this be bumped up to more?

try this bro

This is not possible - This graph is based on the Google timeline chart, so the functions are limited to that library.


BUG - Cannot drag and drop files

Bug
NPE Version 4.4.53+g671b4f96f5
Installed From: Windows Store
When adding/updating files to the Package Contents pane, you cannot drag files off of a Windows Explorer folder. This was something you could do in earlier releases and was very, very convenient.

I just tried this and it works for me. Does this happen for all packages, or just a specific one?

Okay, just retested and it works. It had been broken for quite a while. Disregard issue.


Test suite now uses socket pairs instead of memory streams

This PR contains 2 changes:
Warnings when using undefined signal constants in a PHPUnit data provider are avoided. Missing constants for signals are defined as needed in the abstract test case.
Streams used by test cases are created as connected socket pairs. Most event loop extension do not support arbitrary file descriptors but they all support socket pairs. The created socket pairs use unix domain sockets if they are available. The fallback to INET sockets is needed in order to support Windows.

There is one failing test (testIgnoreRemovedCallback) remaining with lib ev loop. I think that this test is problematic because it assumes that the loop backend triggers events in the same order as writes within the test code.

The latest commit fixed the ordering problem with libev. The build failed due to a timing issue with the HHVM build, can't do anything about this...
@WyriHaximus this PR should fix some tests failing due to invalid file descriptors in reactphp-async-interop-loop
 as well.

The build failed due to a timing issue with the HHVM build, can't do anything about this...

I've restarted the HHVM build and this seems to have fixed this for now :+1:

Splitting it up sounds good to me.
Should i modify this PR to just address the socket pair stuff and create a new PR for the signal handling later? You could squash to commits during the merge to get rid of the signal stuff.

I removed the signal-related part of this PR. Using real sockets in tests enables IO testing using the abstract test class in #25 and #12.

I fixed the code formatting. Unfortunately the HHVM build failed again due to a timing problem...

@martinschroeder You should probably rebase/merge the master branch to get b946301bfc0f3996855c5f1f8298c04d2701349c in which should fix the HHVM tests.

I merged the master branch, this fixed the HHVM test case. 👍

Ping @martinschroeder, what do you think about the above suggestion?
Also, may I ask you to rebase this on the new "0.4" branch so we get this in for the upcoming v0.4.3 release? :shipit:

I rebased the PR on the 0.4 branch and removed the obsolete / deprecated methods createStream() and writeToStream(). I squashed all my commits into a single commit to avoid work-in-progress commits in the history.

Awesome, much appreciated! :+1:

Looking good, thanks 👍


Theme is getting switched on switching profiles

From @amunger

Have a profile with a custom theme. Eg: GitHub Light Default theme from GitHub.github-vscode-theme extension
Switch to default profile and set a different theme - builtin theme is enough
Make the theme setting apply to all profiles
Switch to the first profile

🐛 Theme is getting reset to default dark

@aeschli This is another example of the theme component resetting the theme when extension is removed.


internal/runner: add gRPC service

This PR implements another service called "runner". It is similar to the "kernel" service. The difference is that "kernel" tries to detect when each command starts and ends and based on that allows submission of new commands. "runner" does not that, it does not try to be smart. It still allows to run a shell with TTY (which is a long running program that needs some input to terminate) but relies on a caller to provide commands and input data when needed.
Note: "runner" does not implement the Connect protocol. It's only a plain gRPC service.
Check out internal/runner/service_test.go to learn how to use "runner"'s RPC methods. internal/runner/command_test.go can also be useful.

Alright, finally got to reviewing this.  I made some notes on some potential concerns - let me know what you think there.
Environment diffing looks great by the way, really love the way you tackled that problem! Great work overall 🎉 !


Reset to the original codebase

Depends

[x] #204

Warnings

I changed the connection state to the original version. In this situation, we need to connect to the wallet again when we reload the page.
I try to persist the wallet when we reload the page by using the localStorage. However, when I stringify the wallet in the initialState and include this in localStorage, and after that, I get it from the localStorage and parse it but I can't use it in TezosToolKit.

JSON.parse(localStorage.getItem('wallet'))

It shows errors when I try to swap tokens. I think the reason is that the parsed data is different from the original data when considering types.

If you have any ideas, please ping me, @glottologist.


Converting to v2.1

With v2.1 being released into alpha I've been brainstorming ideas on how to implement this

New branch, develop concurrently until v2.1 is released fully (May cause many merge conflicts)
Option in settings to convert to v2.1? (Lot of work)
Start converting scripts straight to v2.1 now (Probably not as most people would still want v1.1 to v2)
New repo (would require people downloading both to go from v1.1 to v2.1)

Personally thinking about option 1, but want to get others ideas on it, and any other ideas people might have

Are there going to be many breaking changes?
Option 1 is probably best I think
On Mon, Mar 25, 2024, 6:31 AM Banaanae @.***> wrote:

With v2.1 being released into alpha I've been brainstorming ideas on how
to implement this

New branch, develop concurrently until v2.1 is released fully (May
cause many merge conflicts)
Option in settings to convert to v2.1? (Lot of work)
Start converting scripts straight to v2.1 now (Probably not as most
people would still want v1.1 to v2)
New repo (would require people downloading both to go from v1.1 to
v2.1)

Personally thinking about option 1, but want to get others ideas on it,
and any other ideas people might have
—
Reply to this email directly, view it on GitHub
https://github.com/mmikeww/AHK-v2-script-converter/issues/154, or
unsubscribe
https://github.com/notifications/unsubscribe-auth/AAOQBDB6XEIPEIYYHQAUCB3YZ74GPAVCNFSM6AAAAABFGX2RV6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGIYDKNBQGUZDMOI
.
You are receiving this because you are subscribed to this thread.Message
ID: @.***>


Review the automated and manual migration processes

What
As a team, review the automated and manual migration processes
Why

So we have a shared understanding
Make any changes we agree on
Understand what needs to be documented
Help users in support

Who needs to work on this

developers
interaction designer
content designer

Who needs to review this

developers
interaction designer
content designer

Done when

[x] We have reviewed the manual process
[ ] We have reviewed the automated process
[ ] We agree on any changes
[ ] We know what needs to be documented

Any changes will go to the separate card

Prototype test cases @joelanman to link them to the ticket

here's some, will add more when I find them

https://github.com/alphagov/digital-identity-prototype
https://github.com/alphagov/forms-prototypes
https://github.com/alphagov/col-prototypes
https://github.com/ministryofjustice/hmpps-activity-management-prototype
https://github.com/ministryofjustice/approved-premises-prototype
https://github.com/DFE-Digital/fh-family-experience-prototype

A good way to find them is to go to dept github accounts and search 'prototype' in their repos

In terms of reviewing the automated process, we're part way there. @BenSurgisonGDS and I understand what it is supposed to be doing, and we've shared what we've learnt with @joelanman. Whats outstanding is: @BenSurgisonGDS is continuing to add tests to make sure it is actually doing what it is supposed to; I would like if we had a shared understanding and agreement among the team of what the automated process does and doesn't do.

We want to get some quantitive data before we make decisions on further changes to the automated process (see ticket #1720), so I'm putting this ticket in blocked until that work has been completed.

After looking at the automated migration script I have some proposals for changes:

https://github.com/alphagov/govuk-prototype-kit/issues/1728
https://github.com/alphagov/govuk-prototype-kit/issues/1729

We might make more as a result of reviewing the output of #1720.

We've agreed to move the above tickets https://github.com/alphagov/govuk-prototype-kit/issues/1634#issuecomment-1294863003 into the sprint backlog, but that we don't think further changes are probably needed right now.


adapter not available in init()

i would like to do some common init, like escaping fields and table names, so up and down could reuse them. protected function init() {} seemed right place to do that.
however, $this->getAdapter() returns null at the time init() is invoked:
class SuperCoolMigration extends AbstractMigration
{
    public function init()
    {
        $this->email_table = $this->getAdapter()->quoteColumnName('support_email');
        $this->body_table = $this->getAdapter()->quoteColumnName('support_email_body');
    }

Any update from the team here?


Should some sort of descriptor abstraction live in this crate?

Seems like a question that will have been discussed before.
In corepc I'd like to provide a strong type for
"descriptor" : "desc",         (string) The descriptor in canonical form, without private keys

And I don't really want to pull in a miniscript dependency just for this. Would it be a good idea to have some sort of basic descriptor abstraction here in bitcoin?

Yes!
I am missing this.
To be honest miniscript is great for advanced stuff like spending policies, but falls short when I just want to import/export descriptors from wallets.
Especially when I need to interface with Xpriv and friends.
Something quite trivial like importing/exporting and to_xpriv/to_xpub would be already amazing.
Also some proper type checking that the descriptor string is valid when importing to Descriptor would be nice. Checking fingerprints, derivation paths, and checksums etc.

We have a bip32 module in this crate already.

Also some proper type checking that the descriptor string is valid when importing to Descriptor would be nice. Checking fingerprints, derivation paths, and checksums etc.

This is really heavy and definitely doesn't belong here.
I would maybe be open to parsing and serializing some very simple descriptor templates, though I don't want to think about parsing keys out of them and don't want to think about the checksum and don't want to think about xprivs...so if you want all that I think you're still going to be stuck with miniscript.

What would this abstraction support? Just verifying the checksum? Should it verify that the strings are well-formed (already a complicated task)? That only miniscript fragments appear? In valid locations?

I think that definitely verifying checksums.
Also a method like from_xpriv would be nice (tricky with all the tr wpkh etc).
The others are nice-to-have.

It is a lot of work, but conceptually it should be possible to split rust-miniscript into descriptors and miniscript. For people who want simple descriptors without miniscript can then use it.
I am not recommending this solution, just pointing out that something like could be possible.
@storopoli , yeah miniscript library has first class support public key descriptors. There are APIs to import private key descriptors that work by converting them to public descriptors and a map from private to public keys.

It is a lot of work, but conceptually it should be possible to split rust-miniscript into descriptors and miniscript. For people who want simple descriptors without miniscript can then use it.

That would be the perfectly solution.
Then descriptor could be moved to a standalone crate.
Should we move this issue to rust-miniscript?

@storopoli , yeah miniscript library has first class support public key descriptors. There is a parse API to import private key descriptors that work by converting them to public descriptors and a map from private to public keys.

Thanks, but what about exporting private key descriptors with the prefix, e.g. tr(xpriv.../*)#checksum.?

Should we move this issue to rust-miniscript?

Let's close this one and open a new one in rust-miniscript, since "I want descriptors in rust-bitcoin" is a fairly common request (which I don't think is manageable for the above reasons) and it'd be good to leave a record here.
Vs "let's split rust-miniscript" into two crates, which I think is a great idea, though it will be a lot of work, and we'll need to spend a bunch of time bikeshedding which parts go into which pieces, and we'll also want to think a bit about how the new descriptors crate would relate to bitcoin-addresses. (Maybe miniscript should depend on addresses which depends on descriptors/bech32/base58/etc?)

Moved to https://github.com/rust-bitcoin/rust-miniscript/issues/783. (can't close the issue, not a maintainer)


Add availability gating for async function and actor type definitions.

Explanation: Require actors and async functions to have an availability of macOS 12.0/iOS 15.0/etc. This was already a requirement, but would only fail at link or load time. This escalates it to a semantic check to make it more obvious.
Scope: Affects new code using the Swift Concurrency model.
Radar/SR Issue:  rdar://80098709
Risk: Low.
Testing: PR testing and CI on main.
Original PR:  https://github.com/apple/swift/pull/38510
Related LLDB PR: https://github.com/apple/llvm-project/pull/3116 (test-only update)

@swift-ci please test
https://github.com/apple/llvm-project/pull/3116

@swift-ci please nominate

@swift-ci please test
apple/llvm-project#3116

@swift-ci please test
apple/llvm-project#3116

https://github.com/apple/llvm-project/pull/3116
@swift-ci please test

https://github.com/apple/llvm-project/pull/3116
@swift-ci please test

Build failed
Swift Test OS X Platform
Git Sha - 330d82481e92d7072c420c31ebc5ffc707885c76

@swift-ci please test

Build failed
Swift Test Linux Platform
Git Sha - 4684a7ee126c74d99b2a093344ac147a0422502d

@swift-ci please test Linux


[playwright-ct-angular] Support Angular 18

At the moment, playwright-ct-angular only support Angular 16 and 17. The current version of Angular is 18.x
  "peerDependencies": {
    "@angular/compiler": "^16.0.0 || ^17.0.0",
    "@angular/core": "^16.0.0 || ^17.0.0",
    "@angular/platform-browser": "^16.0.0 || ^17.0.0",
    "@angular/platform-browser-dynamic": "^16.0.0 || ^17.0.0",
    "rxjs": "^7.0.0",
    "zone.js": "*"
  },

Is there any limitation or issues with Angular 18.x ? Or is it just maintenance (i.e. it was never changed)

We use Angular 18 in the repo so it's a maintenance issue. Thanks for reporting!


Fun: Python script to send URLs from streamtheworld.com (iheartradio) to esp-radio player.

Send a radio station to your esp-radio player: GIST
Simple script, at the top of the script, is the usage example, basically, find a radio station on iheartradio:
iheartradio station list USA
iheartradio
iheartradio Canada
Look at the source code of the iheartradio page (click on song history will help a bit) you will find things like this in it:
view-source:http://www.iheartradio.ca/all-stations/7.13793734?mode=history

                        <button class="stream-link stream-radio-station" data-ajax="true"
                                                        data-station-callsign="IHR_TRAN"
                                data-ad-unit="/5479/iheartradio.en/allstations"
                                data-ad-ss="radioplayer"
                                data-station-id="1.1870491"
                        >

                        </button>

This is what we want:
data-station-callsign="IHR_TRAN"

IHR_TRAN is the callsign for the station you want to play. (Trancid)
Another stations callsign CIMXFM etc.
This is the radio station we are grabbing below from iheartradio 89x "CIMXFM"
Usage: python2 streamtheworldespradio.py CIMXFM <IP_ADDRESS>
                                   ^Radio callsign  ^IP address of esp-radio Device

Make sure you have python2 and curl installed on the system you are sending the station from, download the script to a file on that device, then type the command, I named the script streamtheworldespradio.py, so type on the command line:
python2 streamtheworldespradio.py (The radio call sign + FM/AM on the end) (IP Address of your esp-radio Player)

First, test with the call sign above/below make sure it is all in CAPS as I know that one works (may not work outside of Canada), the hard part would be finding how they named the call sign for the station you want, and some may be location locked so be aware. Also add FM/AM on the end of the call sign or the script will complain "Callsign is missing FM, AM".
The script then tells the website to give us the ShoutCast stream URL for audio stream compatibility purposes with _SC.
python2 streamtheworldespradio.py CIMXFM <IP_ADDRESS>

The script is grabbing the stream URL that is a randomly encoded string of characters for that station you want, then send the URL to the ESP8266.
The script requests a URL like this:
http://playerservices.streamtheworld.com/api/livestream?version=1.5&mount=CIMXFM&lang=en
Then it sends back an XML which we extract the values from:
<live_stream_config xmlns="http://provisioning.streamtheworld.com/player/livestream-1.5" version="1.5">


RTSP or MJPG inference possible?

@robmarkcole
Would you be so kind to guide me how to input source of web cam live stream inference on the example of your restapy.py?
I desperately need to run inference on a live video stream using the API. Unfortunately, it's impossible to realize with pure "python detect.py" from the remote host to the server in headless mode. Thus your API example would be of much help.
Thank you!

Yolov5 can process a video stream, checkout their examples. You would not use an API like this for video rate processing, you could sample at say 1/2 frames sec and process, but then there is nothing special about it.

API may be called easier. The problem with online video stream processing is that it needs GUI and cannot work in headless mode (I need to call the function on server  from a client). Here I described my problematics, that's why I decided to opt for other solutions like yours:
https://stackoverflow.com/questions/74306074/opencv-running-ip-cam-inference-script-remotely-via-ssh-results-in-could-not-c

My other project might be suitable with some updates https://github.com/robmarkcole/mqtt-camera-streamer

Yes, could be an option, but my task is simpler.
I can sacrifice realtime RTSP processing in favor of processing JPG every 5 seconds.
My camera JPG URL can be analyzed during a set period of time.
The script should be executed as long as camera motion detection is running.
I still think API variant is more suitable.
Could you advise me what to add in restapi.py code to analyze a particular JPG in cycle each 5 seconds?
Also I need to envisage script work termination when camera stops sending motion detection signal.
I'm actually integrating all this into Home Assistant, so HA has sensor that alerts about motion, I can take information from there.
Thanks!

It is not too complicated what you are asking but it is out of scope of this project. I am sure if you search GitHub someone has done it already


Implement Experimental GraalJsEngine

Proposed Changes
Enables opting in to use GraalJS instead of Rhino JS for javascript evaluation. GraalJS is fully ECMAScript 2022 compliant while Rhino JS only supports ES5.
This feature is experimental for now, but we intend to make GraalJS the default if all goes well.
Enable GraalJS via Flow config
appId: com.example.app
jsEngine: graaljs
---
# Note: The ?? operator is an example of an ES2020 feature and is not supported by Rhino JS
- inputText: ${null ?? 'foo'}

Enable GraalJS via environment variable

Warning
This env var will have no effect when running on Maestro Cloud. Use the Flow config above instead to opt into GraalJS on Maestro Cloud.

export MAESTRO_USE_GRAALJS=true
maestro test my-flow.yaml

GraalJS behavior differences
There are some differences between the new GraalJsEngine and the current RhinoJsEngine implementation that are worth noting. All of the differences below and some others are documented and tested by the GraalJsEngineTest and RhinoJsEngineTest tests.
tldr; The variable scoping with the GraalJS implementation is more consistent and understandable

Rhino JSGraalJS

Can not reuse variable names across scripts within the same Flow
Can reuse variable names if redeclaration lives in a subflow
Can access variables declared earlier in the Flow in a separate script
output variable is shared across all scripts

Variables are local to each script
output variable is shared across all scripts

Some examples

ExampleRhino JSGraalJS

Redeclaring variables across scripts

appId: com.example
---
- evalScript: ${const foo = null}
- evalScript: ${const foo = null}

❌ Variable redeclarations throw an error
✅ Variable names can be reused across scripts

Accessing variables across scripts

appId: com.example
---
- evalScript: ${const name = 'joe'}
- assertTrue: ${name === 'joe'}

✅ Variables are accessible across scripts
❌ Variables can't be accessed across scripts

Special character handling

appId: com.example.app
env:
  FOO: \
---
- inputText: ${FOO}

❌ Single backslash causes an exception
✅ Single backslash and all other special chars are handled correctly

Testing
Ran integration tests with MAESTRO_USE_GRAALJS=true. 2 tests failed due to known differences between GraalJS and RhinoJS described above. All other tests passed.

Addresses #1081

Should the env var MAESTRO_USE_GRAALJS even exists? Given that it only works on Maestro (not on Cloud) and there is already a place to set the JsEngine (MaestroConfig)?

Hm I was was thinking the env var would be useful for testing an entire test suite with graaljs without editing the flow file. This may come in handy when testing against existing test suites (which I think is useful for both Maestro users and for us). I agree that it has the potential to cause confusion but on the other hand it will only exist while graaljs is experimental. But I'm not married to it so we can remove if it seems too problematic

@igorsmotto I was thinking about that but I wanted to also support this case: maestro test myflow.yaml which doesn't use a config file

@igorsmotto I couldn't find a better alternative that doesn't require a lot more rearchitecting, and given that this is a temporary config while graaljs is still experimental, I've landed on the following compromise:

Both the env var and flow are available to enable graaljs
The env var only works locally
The flow config only has an effect if present on the top-level flow
The flow config does not have an effect if present in a subflow


mmwatch - fix issue #28 (#28)

Signed-off-by: Jianwen Ji<EMAIL_ADDRESS>

@majek , please help review this patch.


Adding feature for banks to broadcast announcements to partners issue: 3390

Resolves #3390
Description
New feature added to allow banks to broadcast announcements to the partners.
For the banks there's a link on their side bar that goes to a table of the current announcements where they edit or delete them. There's a button at the top for creating new announcements (screen shots below).
Each announcement consists of a message, with a 500 character limit, an optional link for more info, and an optional expiration date. The creating user and organization are also stored so this table can be used for admin to bank announcements.
Expired announcements persist on the bank side with a red 'expired' tag until deleted but are not shown to partners.
The partner dash board has a card at the top that displays the announcements, or no card if there's no announcement

Type of change

New feature (non-breaking change which adds functionality)
This change requires a documentation update

How Has This Been Tested?
There are tests for the model and how it validates entries and CRUD tests for the controller
All tests pass, new and old

Screenshots
Main bank page, I highlighted the sidebar menu change

New announcement bank page, edit looks the same except for the title and breadcrumbs

Partner dashboard, the card is only shown if there's an announcement and it only shows bank announcements that haven't expired
One announcement:

Multiple announcements:

I'm new, so hopefully I didn't do anything too weird. I'm happy to change anything and greatly appreciate any feedback

@dorner Thank you so much for taking the time to go through my code a provide feedback

Excited to see this one in progress, @cotabas!  The essentials banks are going to be pretty happy to get this.

Hey @cielf, thanks for reviewing, I've enjoyed working on this feature and I'm happy to hear it will be helpful.
I finished the changes @dorner requested.
I've also made all the changes you've requested, I did go the fancy route and have it only displaying the full date if it's from a previous year.
Please let me know if there's anything else I can change

I think we can balance this by adding a method on the announcement class that returns the list of announcements to display given an organization ID - we can unit test that method and ensure that it's the one called in the request test.

I added the announcement method as suggested but I'm having trouble testing it. When I try to create an announcement in the spec with a different organization_id, or no organization_id, it fails with Validation failed: Organization must exist.
I removed the not null on organization_id as suggested earlier so I'm not sure what's going on.
I've updated my test database since that migration
ActiveRecord::RecordInvalid:
       Validation failed: Organization must exist

Thank you @dorner apparently I've never used optional references
Let me know if I need to change anything else

@cotabas Just released this one too!


REST API for accessing Azure Pricing details

Currently, there is no option to access Azure Pricing information programmatically. Definitely, the Pricing Calculator UI is user friendly and is easy to use. However, if one needs to access the same information for creating custom reports or doing planning, then need to switch to the Pricing Calculator, do the math and update it manually in the in house built tools/reports. A best example is the AWS' Price List API. Could we get a similar API for Azure Pricing?

@hellosnow Can you take a look at this suggestion?

@martinmthomas Thanks for your suggestion. 😄
https://github.com/Azure/azure-rest-api-specs is our Azure rest api swagger files, as far as I know there's no overall Azure pricing REST API, but I need to confirm with Azure team on this.

@hellosnow, thanks for looking in to this.

@martinmthomas I'll pass this info to Azure REST API team to see whether we have the on-going plan, thanks.

@hellosnow Is there any update from Azure Rest API team.

@dend please help to follow, thanks 😄

@dend, @powerhelmsman, @hellosnow
I'm also interested on this, my use case is:

I'm already using azure migrate for assessment, but azure migrate is only available for VMWare.

I also need to assess on premisses: bare metal and OpenStack for migration to cloud. I already have ways to get CPU, memory, disc, IO, and now I need an API to get the price for those.
Any help would be appreciated,
Thank you,

Any update on this?

We have a current use case for such an API for Azure. It will really help us deploy self-service options.

Hey guys any update on this ?


[SPARK-22615][SQL] Handle more cases in PropagateEmptyRelation

What changes were proposed in this pull request?
Currently, in the optimize rule PropagateEmptyRelation, the following cases is not handled:

empty relation as right child in left outer join
empty relation as left child in right outer join
empty relation as right child  in left semi join
empty relation as right child  in left anti join

case 1 and 2 can be treated as Cartesian product and cause exception. See the test case in JoinSuite.
How was this patch tested?
Unit test

Test build #84216 has started for PR 19825 at commit cde04c8.

Test build #84217 has started for PR 19825 at commit d6ffcdf.

Test build #84216 has finished for PR 19825 at commit cde04c8.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Test build #84217 has finished for PR 19825 at commit d6ffcdf.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84216/
Test PASSed.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84217/
Test PASSed.

Test build #84284 has started for PR 19825 at commit c359fe3.

Hi @gatorsmile , I have just fixed full outer join and add more test cases.
It is ready for review.

Test build #84288 has started for PR 19825 at commit 189d590.

Test build #84284 has finished for PR 19825 at commit c359fe3.

This patch fails due to an unknown error code, -9.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test FAILed.

Test FAILed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84284/
Test FAILed.

Test build #84288 has finished for PR 19825 at commit 189d590.

This patch passes all tests.
This patch merges cleanly.
This patch adds no public classes.

Merged build finished. Test PASSed.

Test PASSed.
Refer to this link for build results (access rights to CI server needed):
https://amplab.cs.berkeley.edu/jenkins//job/SparkPullRequestBuilder/84288/
Test PASSed.

LGTM

Thanks! Merged to master.


Input type being stripped on nuxt generate

After running nuxt generate an <input type="text" id="name"> becomes <input id="name">.
I made a demo repo here --> https://github.com/stursby/nuxt-tags-bug
For example, if I'm styling using the following, it's no longer applied after generating.
input[type=text] {
  background: lightgray;
  outline: none;
  border: none;
}

Here's what it looks like when running nuxt

And then after running nuxt generate

nuxt generate use the html-minifier lib to minify the *.vue pages ( cf. generator.js script)
It's not an issue, but a compression feature call removeRedundantAttributes.
type=text is remove because it's already the default value of type, so it's like redundant.
A workaround is to disable the minify of html:
https://nuxtjs.org/api/configuration-generate#minify
// nuxt.config.js

module.exports = {
    generate: {
        minify: {
            removeRedundantAttributes: false
        }
    }
}

But, if you want keep the original minify, you have to use another css rule (eg. without type or with a custom class)

@NicoPennec Ok, good to know, thanks! Then yes, I would agree, this is not an issue as I wasn't aware it was behaving this way by default. Thank you for the clarification.


Issue/43

https://github.com/decentralized-identity/element/issues/43

Codecov Report

Merging #51 into development will decrease coverage by 13.47%.
The diff coverage is 87.5%.

@@               Coverage Diff                @@
##           development      #51       +/-   ##
================================================
- Coverage        96.66%   83.18%   -13.48%
================================================
  Files                3       78       +75
  Lines               60     1255     +1195
  Branches             3      113      +110
================================================
+ Hits                58     1044      +986
- Misses               2      198      +196
- Partials             0       13       +13

Impacted Files
Coverage Δ

...ackages/element-lib/src/adapters/database/index.js
100% <100%> (ø)

...lib/src/__tests__/__fixtures__/getLocalSidetree.js
100% <100%> (ø)

packages/element-lib/src/sidetree/resolve.js
84.09% <100%> (ø)

...nt-lib/src/adapters/database/ElementRXDBAdapter.js
86.2% <86.2%> (ø)

packages/element-lib/src/adapters/storage/index.js
100% <0%> (ø)

.../bytes32EnodedMultihashToBase58EncodedMultihash.js
100% <0%> (ø)

packages/element-lib/index.js
100% <0%> (ø)

...ages/element-lib/src/func/batchFileToOperations.js
100% <0%> (ø)

packages/element-lib/src/sidetree/getAnchorFile.js
94.44% <0%> (ø)

...ment-lib/src/adapters/blockchain/ethereum/index.js
82.71% <0%> (ø)

... and 69 more

Continue to review full report at Codecov.

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


Terminal crashes in WSL2 when typing exit directly after a sudo statement in second tab

Environment
Windows build number: Version 10.0.19033.1
Windows Terminal version (if applicable): 0.7.3291.0

Any other software?
Ubuntu 18.04.3 LTS running in WSL2

Steps to reproduce
I have the Ubuntu installation in WSL2 set up as the default profile

open terminal (or have it running)
open a second tab
execute any command as sudo, e.g. sudo hwclock -s for synchronizing the clock
enter the sudo password
type exit

Expected behavior
the tab closes, the other sessions stay open.

Actual behavior
the terminal crashes.
https://i.imgur.com/nbg76pn.gifv

I believe this is the same as #3801 ...
but will you send us some /feedback

I guess, the issue is the same as in #3801 Here's my feedback https://aka.ms/AA6r4za

I guess, the issue is the same as in #3801 Here's my feedback https://aka.ms/AA6r4za

Linking to 3rd party feedback seems to be a bad practice as not everyone can read the response there:

That for example with a company account.

@krisavi thanks for the info. This feedback workflow does suffice for our purposes as Microsoft, but it can be somewhat annoying for others :smile:

Confirmed as a /dupe of #3801. Thanks for the feedback item!


IoT Edge create deployment return success when IoT Hub quota exceeded

Required Information
Entering this information will route you directly to the right team and expedite traction.
Question, Bug, or Feature?
Type: Bug
Enter Task Name: IoT Edge
Environment
Azure DevOps.
Issue Description
Create deployment task.
While IoT Hub deployment quota exceeded, the task still returns success.

https://github.com/Azure/azure-iot-cli-extension/issues/72

This is currently blocked by Azure/azure-iot-cli-extension#72. No changes required from Azure Pipelines task side.


Endpoint validation - why hostname validation is needed and how to cope with changes in endpoints hosts?

Hi,
In the DiscoveryDocumentResponse->ValidateEndpoints, what is the purpose of the hostname validation for all endpoints?
In several cases, the endpoints hosts are different from the discovery URL host. In these cases, the discovery will not pass the endpoint validation. If the "allowed hosts" will be configured on the DiscoveryPolicy, it still could fail in case that the identity provider will add a new endpoint.
For example, in google, the discovery URL is "https://accounts.google.com/.well-known/openid-configuration" but a lot of the endpoints configured with different hosts like for example:
"token_endpoint": "https://oauth2.googleapis.com/token" and  "userinfo_endpoint" : "https://openidconnect.googleapis.com/v1/userinfo".
In this example, Google may also add new additional endpoints with different hostnames and cause the discovery endpoint validation to fail.
My questions are:

What are the best ways to cope with that issue without damaging security?
What is the reason for using it?  Why the AuthorityValidationStrategy is not enough?

Thanks for your support,
Shai

wrong repo. Please use IdentityModel.


Add .env.example [#65]

The current way using a shared .env means that the local .env gets overwritten anytime you checkout another branch. Since .env is in .gitignore anyways it should be removed from the repo and be replaced with an example file.

Amazing! We really needed such a file. LGTM!


portusctl: show example of Registry's configuration

Show a working example of Registry's configuration when the Registry is
not running on the same host of Portus.

LGTM

Rubocop is not liking this PR. Could you fix that ? :)


Where is the Default Template in Sage9 ?

Submit a feature request or bug report

[x] I've read the guidelines for Contributing to Roots Projects
[ ] This is a feature request
[ ] This is a bug report
[x] This request isn't a duplicate of an existing issue
[x] I've read the docs and NPM Debugging Guidelines post and followed them (if applicable)
[x] This is not a personal support request that should be posted on the Roots Discourse forums

I maybe need some glasses but I can't find the Default Template in Sage9. Anywhere.
I suppose it is on any blade files but I can't find any. with this markdown :
{{--
Template Name: Default Template
--}}
Do you have any idea ?
Thanks for your informations, for new glasses maybe ?

This seems like a personal support request, not a bug report, so you might want to try asking over in the Discourse forum: https://discourse.roots.io/c/sage


[FEA] Support null characters in regular expressions in RLIKE

Is your feature request related to a problem? Please describe.
Given this input dataset:

ab
a\u0000b

The behavior or RLIKE differs between CPU and GPU when the pattern contains a null character:

Pattern
Spark on CPU
Spark on GPU

a\\0b
Illegal octal escape sequence near index 3
Correct result

a\0b
Correct result
Incorrect result

a\u0000b
Correct result
Incorrect result

a\\x00b
Correct result
Correct result

Describe the solution you'd like
Ideally, we should have the same behavior as Spark but if this is not possible then we could consider falling back to CPU and maybe having a separate config to enable/disable supporting null characters in regex.
Describe alternatives you've considered
None
Additional context
This is closely related to #3876

This was implemented in https://github.com/NVIDIA/spark-rapids/pull/4044


Added bisection script. Lower case with underscores

Replaced numerical method with ~1000-odd passes with a bisection method that should converge in ~10 passes (at similar tolerance). Added documentation on the Clopper-Pearson confidence interval itself. Moved to lower case with underscores in place of CamelCase for function calls.
Potential test method: export an upscaled disconnection summary with previous and proposed version of the confidence interval script. Each version of upper and lower bounds should be equal within numerical tolerance. It is also possible to black-box test the confidence method methods (i.e. for 0%, 100% observed successes).

@amakarowsky did you try runnning the tool with and without this to see if there are significant changes in results?

Thanks @exfletch . I did export a disconnection summary with previous and proposed versions of the script, the results agreed within numerical tolerance. Probably worthwhile someone repeating this separately however as I am quite new to the tool etc.


error handling

I am using httpimg.Register method to register a url of image.
Like this httpimg.Register(pdf, image_link, "")
I have a json with different image url's, when I loop on all the urls and map them and register using httpimg.Register the url's and then embed the image in my pdf I came across an issue. The problem is there is possibility that url is broken (i.e image doesn't doesn;t exist in some urls ) and because of this issue my program halts the pdf generation and at the end pdf doesn;t generates. I want to handle this error i.e if url is broken then i want to add any empty image etc.
I am confused how to handle this error, please review my code and guide me.
x := 0.0
y := 0.0
col := 0
var img_len int = len(img.Images.Photos)
for i :=0; i<img_len; i++ {
image_link := b[i].Images
pic_link := b[i].Link
println("image link",image_link)
httpimg.Register(pdf, image_link, "")
if col == 5{
x = 0.0
y += 40.0
col = 0
pdf.Ln(3)
}
pdf.Image(image_link, 20+x, 30+y, 30, 30, false, "", 0, pic_link)
pdf.Cell(10, 0, "")
col += 1
x += 35.0

You will want to use pdf.Error(), pdf.ClearError() and friends. For example,
httpimg.Register(pdf, image_link, "")
if pdf.Err() {
  log.Printf("error registering image %s: %s", image_link, pdf.Error())
  pdf.ClearError()
}

Thankyou very much for your response. It really helped me alot 👍

Glad that worked.


Does not build with ghc-9.6

When building this with ghc-9.6 (as a dependency of another package) it fails with:
src/Data/Functor/Of.hs:81:3: warning: [-Wnoncanonical-monad-instances]
    Noncanonical ‘(>>)’ definition detected
    in the instance declaration for ‘Monad (Of a)’.
    ‘(>>)’ will eventually be removed in favour of ‘(*>)’
    Either remove definition for ‘(>>)’ (recommended) or define as ‘(>>) = (*>)’
    See also: https://gitlab.haskell.org/ghc/ghc/-/wikis/proposal/monad-of-no-return
   |
81 |   (m :> _) >> (m' :> y) = mappend m m' :> y
   |   ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
[2 of 4] Compiling Streaming.Internal ( src/Streaming/Internal.hs, dist/build/Streaming/Internal.o, dist/build/Streaming/Internal.dyn_o )

src/Streaming/Internal.hs:209:10: error: [GHC-39999]
    • Could not deduce ‘Show (f (Stream f m a))’
        arising from the head of a quantified constraint
        arising from the superclasses of an instance declaration
      from the context: (Monad m, Functor f, Show (m ShowSWrapper),
                         Show (f ShowSWrapper))
        bound by the instance declaration
        at src/Streaming/Internal.hs:(209,10)-(210,30)
      or from: Show a
        bound by a quantified context
        at src/Streaming/Internal.hs:(209,10)-(210,30)
    • In the instance declaration for ‘Show1 (Stream f m)’
    |
209 | instance (Monad m, Functor f, Show (m ShowSWrapper), Show (f ShowSWrapper))
    |          ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^...

I see HEAD is fixed. I will use a source-repository-package stanza in my cabal.project file for now.

Me too.
Made a respective revision on hackage: https://hackage.haskell.org/package/streaming-<IP_ADDRESS>/revisions/

streaming-<IP_ADDRESS> is now on Hackage which should build on GHC 9.6


Error when Install Playwright command is executed.

Error when Install Playwright command is executed from command palette
Occurs in Windows and Mac OS

This is fixed, closing.


Error when resizing window

Opened a file, dragged out the window to expand it, and BOOM:

Pressing OK resulted in the same error again, then once that was acknowledged, the window was resized but the content wasn't.

I am getting this error on every resize.

Also get the error on every resize, windows 8.1

I get this error with existing csv files using version 0.4.8


Query annotations by area and zoom level

The GET annotation/{id} should either be extended or a new endpoint be created that will return annotations that are within a specified rectangle and some minimum size for a particular zoom level.
This will require a spatial index.  We will probably want to precompute extents of annotations.  Annotations smaller than some size (1 pixel, for instance) at a zoom level should be able to be excluded -- I think the 'minor radius' of the annotation is the relevant metric for this.  We will want any annotation that overlaps the specified region, and it will be better to err on giving some annotations that are not strictly necessary than on being too exclusive (for instance, a polyline that surrounds the region but doesn't enter it can be returned).

A limit for the total number of annotations returned would be useful, too.
Querying by a list of types of annotations may also be desirable.

Simplistically, each annotation has a bounding box and a minimum size.  The minimum size can crudely be the minimum of the width and height of the bounding box.  It would be used as the threshold of whether to show the annotation at a particular zoom level.
Points, however, have zero-area bounding box, , so we may wish to artificially include them at some other resolution.  If we have a line (not a polygon), then pedantically, it has the same problem as a point, in that it has no breadth.  If we are using the bounding box for the size threshold, then this is only a problem if it is vertical or horizontal.
I can see it to be useful to also include:

limiting the number of annotations returned by either a fixed number or by a number relative to their complexity (so fewer complex polygons would be returned than points).  A complexity based threshold would ensure that the viewer always has enough resources to show what we send it, even if it has to only show some of the annotations because of it
filling the limit request with elements outside of the strict bounding box so that (slow) panning doesn't have perceptible lag in showing annotations.  This could just be accomplished by asking for a larger region in the client (which would be easier).

I think we'll need to precompute the bounding boxes as we save the annotation elements, as it will otherwise be prohibitively slow to make our queries.

I think minimum zoom level at which an annotation appears should be an out-of-band piece of metadata -- I could see it being useful to show very small annotations even at low magnification (perhaps rendered as something other than a rectangle), just so a user knows on which section they should zoom in.

I was thinking that if each feature had a size-like parameter, then it can be used as an optional query, so that a client can choose what threshold should be used for that.
If we have a limit to the query, then the sort order becomes important on what gets returned when the limit is reached.  We can expose a sort parameter.  Or we can pick something to start with and expand to a sort parameter later.  I see using distance from center of view and size as useful sorts.

Resolved by PR #180.


passing the correct user to the card footer component

this PR fixes this issue

also Added a button to scroll to the top of the page when scrolling too deep into the RFC argument component.


restrict google login by white-listed domains

not sure if this is the best way to do this, but I need to lock-down my app to my company's google app's domain, and this works...

Thanks for the tips, mlavin.  How about now?

Seems like it should work as intended. dsa_view wrapper should catch this exception and kick the user to the login error page. Some documentation on how to configure this properly probably wouldn't hurt. WHITE_LISTED_DOMAINS = ['foo.com', ] would not work as likely expected since this should be a list of regular expressions. For instance with this setting, an email from footcom.net would be allowed.

Actually, that loop was garbage (would have failed on the first invalid domain).  Made some changes to simply check if the users email domain string is in the list.  Added some documentation.

@yekibud, could you add a prefix to the setting name, something like GOOGLE_ since it's google specific?


Fix clean_e2e_db not running in CI when e2e fails

Which issue this PR addresses:
clean_e2e_db is responsible for cleaning up the cluster database from e2e-eastus, but because it was only being executed in cases of succeeded e2e, it was causing failed e2e runs to leak databases. This eventually resulted in a full CosmosDB quota which was fixed by manually deleting cluster databases using az.

What this PR does / why we need it:
Ensures that clean_e2e_db always runs.

Test plan for issue:
e2e passes, cluster database is deleted. e2e fails, cluster database is also deleted.

Is there any documentation that needs to be updated for this PR?

Marking ready-for-review since this is working as intended, and the Cleanup (Azure) pipeline step is executed even when e2e fails: https://msazure.visualstudio.com/AzureRedHatOpenShift/_build/results?buildId=71332267&view=logs&j=3965c7ea-79de-5618-9d9a-42b627bfb354&t=d9d16192-1f8f-5a67-50e7-e33d65609d41
The e2e itself is still having issues - will open up a separate PR for debugging further.

What about timeouts?  If the e2e takes too long, the pipeline times out.  Will the always() step still run?

@bennerv yes, because when the e2e suite itself times out, it's a timeout of that particular pipeline step, not a timeout of the whole pipeline.

/azp run e2e


Clarify license

Since it's public repository the licensing needs some clarification. What kind of license do you use?

We have been using MIT License from now.


Could the original artical be open accessed?

Like arxiv or somewhere else to get open access?

You may access the article here at the corresponding author's homepage.

Thx~


Testing

To ensure that the results we are displaying are correctly we should implement some testing. People will rely on our chart and perhaps use them in publications, hence, we need to ensure they are 100% accurate. Consequently, we need to ensure they are accurate and validated.
Testing could be done on different levels, more than on the chart itself, which is just a representation of the underlying data we would test the source code and that the interface is rendering well. One option is that we could test your code and make sure we get the expected results. For example, we could import the data for a few test locations, perform the calculations and ensure the final output is correct (e.g. that the average wind speed for Rome at 45°C is XX m/s, or that the CCD in May is YYY). Since the chart is just a representation of these data, it should be correct.
In pythermalcomfort was easier to implement it since I am checking that for example the pmv with a predefined set of inputs always returns the same output.

This is another useful resource: https://dash.plotly.com/testing


Refactor codegen

Refactors codegen pretty much entirely. We now have a set of builders for the various types we generate during codegen. These are:

TypeBuilder: Base type for all builders which build types. Ensures the type has a name, a Hack type, a GQL type, and a superclass.

InputTypeBuilder: Base type for all builders which build input types. Ensures the type has a TCoerced type constant.

InputObjectBuilder: Builds a GQL Input Object from a Hack shape.
InputEnumBuilder: Builds an enum which can be used as input to a GQL field.

OutputTypeBuilder: Base type for all builders which build output types. Ensures the type has a THackType type constant.

ObjectBuilder: Builds GQL interface and objects from Hack interfaces, objects and shapes.
InputEnumBuilder: Builds an enum which can be used as output from a GQL field.

FieldBuilder: Base type for builders which build GQL fields.

MethodFieldBuilder: Builds a GQL field from a Hack method.
StaticFieldBuilder: Builds a GQL field from a Hack static method.
ShapeFieldBuilder: Builds a GQL field from a Hack shape field.

All in all I'm pretty happy with this refactor given it removes code while breaking up a large file into many smaller files that each have their own boilerplate use... statements.

Oops another comment race condition...I guess it's a good sign that we both said the same things though :)

This looks great, thanks!
I'm not sure if this applies anywhere right now, but if you notice any place where you needed to add multiple special cases for different types, sometimes it might be easier to fix the base classes instead of adding more special cases to the codegen -- e.g. I see we have all these type constants with different names (THackType, TCoerced, TExpected, TResolved) and some of these are really the same thing, so it might be easier in some cases to rename the constant, instead of adding a special case to the codegen to generate constants with different names, etc.

Yeah, good call out - I think an input type’s TCoerced and an output type’s THackType are really the same thing - the hack type from which the GQL type is generated. I’ll rename TCoerced to THackType in a subsequent PR! I still think it could be nice to have different base classes for input and output types just to make the type hierarchy explicit.

@mwildehahn I’m gonna merge this down so you can rebase against it. I’ll address feedback in a follow-up.

I think an input type’s TCoerced and an output type’s THackType are really the same thing - the hack type from which the GQL type is generated. I’ll rename TCoerced to THackType in a subsequent PR!

They're not exactly the same thing, but THackType seems like a reasonable description of both so that makes sense.

values of input types are "coerced" at some point, which is a conversion from mixed -> TCoerced (could be called THackType)
values of output types are "resolved" at some point, which is a conversion from TExpected -> TResolved (TExpected is also called THackType in some places)
for types that are both input and output types, TCoerced = TExpected so I guess it makes sense to call these the same thing (e.g. THackType like you suggested)

They're not exactly the same thing, but THackType seems like a reasonable description of both so that makes sense.

values of input types are "coerced" at some point, which is a conversion from mixed -> TCoerced (could be called THackType)
values of output types are "resolved" at some point, which is a conversion from TExpected -> TResolved (TExpected is also called THackType in some places)
for types that are both input and output types, TCoerced = TExpected so I guess it makes sense to call these the same thing (e.g. THackType like you suggested)

Good point — I was playing a bit fast and loose with terminology. The third bullet point (combined with the fact that THackType = TExpected for output types) is what I was alluding to and why I think this rename makes sense.


UC labels and cards are not rendered for the admin role ROLE_SUPER_DUPER_USER

It appears that /app/projects endpoint takes a different path for ROLE_SUPER_DUPER_USER and doesn't return userCommunity attribute. Looks like skills.services.admin.ProjAdminService#loadProjectsForRoot needs to be updated to populate userCommunity value. The ticket can be tested via a service test(s).

:+1:


Instead of showing spinner, show useful install information

Right now pipx hides install information, instead just showing a spinner. It would be nice if that information was instead visible, I don't really see the appeal of hiding it.

Add install info alongside the spinner or make the output using the ‘—verbose’ flag the default? One of the goals of pipx is to work at the right level of abstraction. When installing a package pipx generally avoids output spewing to the terminal when unnecessary.

“Useful information” is an extremely fine line to walk. pip (which is on the extreme other side of the spectrum) is critisised by many to be showing too much output, causing information fatigue. I would prefer pipx to keep its current approach, since it is much more difficult to reduce output than add it.

Verbose definitely outputs a lot more information than it needs to, I think. Showing the output from pipx > (run_subprocess:112): running /home/traverseda/.local/pipx/venvs/ytmdl/bin/python -c #!/usr/bin/env python3 is probably too much.

When installing a package pipx generally avoids output spewing to the terminal when unnecessary.

I don't know, showing information for information's sake seems bad, but so does hiding all the nuts-and-bolts behind a facade, you know? Software is messy, and I still see anything command-line based as firmly in the tools.
Using cached tensorflow-2.2.0-cp38-cp38-manylinux2010_x86_64.whl (516.3 MB)

If relevant information I think. Potentially very relevant if you're on a metered connection.

is critisised by many to be showing too much output, causing information fatigue.

I really don't get that line of thought. You can always ignore output if you want. I see the spinner as automating the job of ignoring output? I just really don't see that perspective. If it was somehow filtering or better displaying the relevant output that would be a good thing, like if you could condense the data to show only the more relevant parts that would be great.
Just automating the task of ignoring output on the users behalf? I don't get it.

@uranusjr I honestly haven't seen any of that criticism. Can you point me in that general direction?

Most of the problems are identified during the recently  conducted user experience research project. As you say, you can always ignore output if you want. Information fatigue is the phenomenon that when you show too much to users, they tend to start glossing over what they think is not important, and missing things that are actually relevant when they appear.
One pattern identified from the interviews by our UX researchers is that people would give feedback like “I think pip should tell me XYZ” only to later be informed those are already in the output message. What pip is currently trying to identify is what exactly are the most common cases when people would need output, and what the most helpful output in those cases.
So what I’d suggest here is for you (and other users) to instead identify concrete cases when the lack of output is actually harmful to the experience, and what output would helpwith the situation.  So pipx should provide output when this happens, i.e. when the directory pipx installs commands in is not in the user’s PATH.
An inverse example would be the “Using cached” message you mentioned. Do most people actively care whether a package is downloaded of installed from cache? According to our interviews, very few ever do. So that would be a message that likely shouldn’t have been added in the first place, and is a candidate for removal now.