Delete Translation

Delete Translation system

Issue closed - OTE v1 branch closed.   All new development on OTE v2.


Tabs order don't work correctly with ngFor and static tabs

If write that code (before create tab through ngFor, after static tab)
<tabset>
    <tab  *ngFor="let tab of tabs">
      hello!
    </tab>
    <tab heading="static tab">
       static tab
    </tab>
</tabset>

first will be showed static tab and after them will be tabs from ngFor

This looks to be the same behaviour as in #823

Implemented an index property for tab component. Waiting for the feat-rc5 branch to be merged before creating a pull request. Change can be viewed/tested from here.

duplicate, closing this one


[SDPA-3228] Move timeline paragraph component to grants module

JIRA issue: https://digital-engagement.atlassian.net/browse/SDPA-3228
Changed

Move timeline paragraph component to grants module

Screenshots

This PR is back to discovery now.

Backend team will do the fix, so this PR can be closed now.


HELP - Firebase Timeout no matter what I do

Hey there,
I have the following code. It was a part of a much larger sketch but I have taken out the firebase parts and refined them. No matter what I do, I cannot get firebase to connect. I did a few weeks back but then I moved to other parts of the sketch and when I came back it no longer works. If someone could take a 2 second look. I'm sure they will see what I am missing.
#include <Arduino.h>
#include <FirebaseESP32.h>
#include <FirebaseJson.h>
#include <ArduinoJson.h>

StaticJsonDocument<200> doc;
FirebaseData firebaseData;

int RED = 0;
int BLUE = 0;
int GREEN = 0;

String esid = "Charge House 2.0";
String epass = "fairenough";

String location1 = "/lamps/";
String deviceID = "42272cac-ff77-4e6d-8dc0-320bca2378ef";

void streamCallback(StreamData data) {
  if(data.dataType() == "int") {
    if(data.dataPath() == "/color/r") {
      RED = data.intData();
    } else if (data.dataPath() == "/color/g") {
      GREEN = data.intData();
    } else if (data.dataPath() == "/color/b") {
      BLUE = data.intData();
    }
  } else if (data.dataType() == "json") {
    deserializeJson(doc, data.jsonString());
    RED = doc["color"]["r"];
    GREEN = doc["color"]["g"];
    BLUE = doc["color"]["b"];
  }
  Serial.println(RED);
  Serial.println(GREEN);
  Serial.println(BLUE);
}

void streamTimeoutCallback(bool timeout)
{
  if(timeout){
    Serial.println("Firebase Timeout");
  }
  delay(1000);
}

void setup() {
  Serial.begin(115200);
  WiFi.begin(esid.c_str(), epass.c_str());
  Firebase.begin("xxxxx-e4ff9.firebaseio.com", "xxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxxx");
  Firebase.setStreamCallback(firebaseData, streamCallback, streamTimeoutCallback);
  if (!Firebase.beginStream(firebaseData, "lamps/42272cac-ff77-4e6d-8dc0-320bca2378ef")){
    Serial.println(firebaseData.errorReason());
  }
}

void loop() {

}

The dev key has been removed for security but confirmed to be correct on my side. Here is a quick snapshot of my simple database:

As you can see, this is my serial output

Check your internet connection.
In Arduino IDE, choose the example BearSSL https request and run it. It should ok if your internet is ok.

No need to use delay in the stream timeout callback.
The library try it best to resume by restart the WiFi and resume.
If it fails to resume, your internet connection is the problem.

Thanks for the tips. Turned out, with verbose logging, that it was a memory issue. Couldn't start pls with only 70kb ram. Using the NimBLE library instead of bluedroid solved my issue. I can now use Wi-Fi, bluetooth and firebase while still having heaps of memory.


[7.x] Update items when making calls into the collection of a paginator

Using the paginate method would return the paginator instance.
return User::paginate();
// Illuminate\Pagination\LengthAwarePaginator

Before this PR, when using paginate and then making calls into the collection would result in returning the collection instance and losing the paginator.
return User::paginate()->map(function () {
    // your code here
});
// Illuminate\Database\Eloquent\Collection

After the PR
return User::paginate()->map(function () {
    // your code here
});
// Illuminate\Pagination\LengthAwarePaginator

If you called a method on the collection that does not return a collection like count() or first(), it will return the result of the method called.
return User::paginate()->count();
// The number of items in the collection

This will help when using collection methods to transform items in the collection of the paginator.

Why this got closed :/ ?

I don't think so.


[FLINK-12496][table-planner-blink] Support translation from StreamExecGroupWindowAggregate to StreamTransformation.

What is the purpose of the change
The pull request aims to support translation from StreamExecGroupWindowAggregate to StreamTransformation.
Brief change log

Update translateToPlanInternal in StreamExecGroupWindowAggregate.
Add WindowAggregateITCase.
Delete TimeIndicatorTypeInfo in blink planner module, Use TimeIndicatorTypeInfo in table-common module.
Fix Bug in TypeConverters when convert RowTimeIndicator or PROCTIME_INDICATOR
Fix Bug in StreamExecSink when inputRowType contains RowTimeIndicator field.
Fix Bug in StateListView and StateMapView to using correct namespace when read or write from underlying Statebackend.
Fix Bug in createKeyedState of PerWindowStateDataViewStore.
Fix Bug in TestingUpsertTableSink to set same parallism of Map and sink with input DataStream.
Fix minor bug in DistinctAggCodeGen.
Fix bug when calls equals to ROWTIME_INDICATOR or PROCTIME_INDICATOR in TimeIndicatorTypeInfo.

Verifying this change
ITcase
Does this pull request potentially affect one of the following parts:

Dependencies (does it add or upgrade a dependency): (no)
The public API, i.e., is any changed class annotated with @Public(Evolving): (no)
The serializers: (no)
The runtime per-record code paths (performance sensitive): (no)
Anything that affects deployment or recovery: JobManager (and its components), Checkpointing, Yarn/Mesos, ZooKeeper: (no)
The S3 file system connector: (no)

Documentation

Does this pull request introduce a new feature? (no)
If yes, how is the feature documented? (not applicable)

LGTM


Fix two compiler warnings

warning C4099: 'tensorflow::port::`anonymous-namespace'::CPUIDInfo': type name first seen using 'class' now seen using 'struct'

Can one of the admins verify this patch?

Jenkins, test this please.

Jenkins, test this please (Flaky infra)

The failing test is a known flaky test.
Im going ahead with the merge.


Link to beta offset page

enable the offset button in dapp and link to the beta

No longer needed after #249


Fixes reverse mapped type members limiting constraint

fixes https://github.com/microsoft/TypeScript/issues/56910

@jakebailey would you mind preparing a TS playground for this one? :)

@typescript-bot test top200
@typescript-bot user test this
@typescript-bot run dt
@typescript-bot perf test this
@typescript-bot pack this

Also, why was a test removed? I can't obviously identify the old test in the new test content.

Those tests are not in the new test content but rather in the old content of that test file that I touched. I just rechecked it and both of them are there. They are almost identical - variable names involved don't include _ though.


[LIVY-393] [Minor] Update httpclient to 4.5.3

Currently the httpclient version in the root pom is 4.5.2 and in the client-http module it is 4.5.1
Now both use a common version and it's been updated to the latest version 4.5.3

Codecov Report

Merging #37 into master will increase coverage by 0.13%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master      #37      +/-   ##
============================================
+ Coverage     70.48%   70.61%   +0.13%
  Complexity      777      777
============================================
  Files            97       97
  Lines          5370     5370
  Branches        806      806
============================================
+ Hits           3785     3792       +7
+ Misses         1050     1041       -9
- Partials        535      537       +2

Impacted Files
Coverage Δ
Complexity Δ

...java/org/apache/livy/rsc/rpc/KryoMessageCodec.java
94.64% <0%> (-3.58%)
18% <0%> (-1%)

...main/java/org/apache/livy/rsc/ContextLauncher.java
82.92% <0%> (+2.43%)
18% <0%> (ø)
:arrow_down:

...in/java/org/apache/livy/rsc/rpc/RpcDispatcher.java
68% <0%> (+4%)
20% <0%> (+1%)
: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 ca5b1be...bb6e15c. Read the comment docs.

Looks good to me, merge to master and branch 0.4.


Passing all tests (ruby 2.1.1 / rails 4.2)

Adjust for daylight savings time if from_time and to_time are in different daylight savings states
Preserve days calculated from year_month_days not overridden by build_days
Supply either duration, or (from/to time) in constructor

Not sure if something breaks in previous versions of rails or ruby (currently not much time to check)

I'm happy if the tests are happy and the code looks good :)


Create or use a dependency injection framework

Problem: components blindly access this.gameStarter without any regard for concrete dependencies.
Problem: some components have a ridiculous number of dependencies.
InversifyJS is probably too heavyweight (even at 4KB).
Perhaps each component can mark the component classes it needs (decorator? public readonly Function[]?), and have GameStartr auto-create them as needed? This might do well as a separate container.

Maybe something like:
export class MyGame extends GameStartr {
    @module(this.createCustomModule)
    protected customModule: CustomModule;

    @component(MyComponent)
    protected myComponent: MyComponent<this>;
}

export class MyComponent extends Component {
    @dependency(CustomModule)
    public readonly customModule: CustomModule;
}

resetComponents and resetModules would do the logic of new-ing everything up and setting their members.
resetModules(): void {
    for (const generator of this.moduleGenerators) {
        generator.call(this);
    }
}

resetComponents(): void {
    for (const generator of this.componentGenerators) {
        generator.call(this);
    }

    for (const component of this.components) {
        for (const dependency of component.dependencies) {
            component.setDependency(this.components, dependency);
        }
    }
}

Map works in IE11. This should be compatible.

Relevant: https://gist.github.com/JoshuaKGoldberg/8b94ac3883919f682944328d6bdad9bb


Update node http sample with express-like api

Hi @mamaso, I'm your friendly neighborhood .NET Foundation Pull Request Bot (You can call me DNFBOT). Thanks for your contribution!
You've already signed the contribution license agreement. Thanks!
The agreement was validated by .NET Foundation and real humans are currently evaluating your PR.
TTYL, DNFBOT;

:shipit:


CMOS-78: Add Fluent Bit tooling info

Initial docs to provide Fluent Bit info.
Added action for PR tests but cannot verify until merged - follow on updates for exporter as well to use it.
Resolved image links for GH vs Antora.
@princerachit cannot seem to request you as a reviewer

Took a while but got a working action for a local repo: https://github.com/couchbaselabs/observability/actions/runs/1462920201

Added few comments.
Something maybe urelated: Can logs land on Loki without fluent bit? Or is this only way?

Added few comments.
Something maybe urelated: Can logs land on Loki without fluent bit? Or is this only way?

Something has to push them, PromTail seems to be the other option but Fluent Bit is CNCF and does a lot of other back ends too which is why we initially picked it plus it started life for embedded.


refactor: 1.79 cleanup

What does this PR try to resolve?
Two tiny cleanup for Rust 1.79.0
How should we test and review this PR?
Additional information

Thanks. r=ehuss with the lock file fixed.

:pushpin: Commit f4985827d7d6dbdeccf98a721e7a6b4c2c75565c has been approved by ehuss
It is now in the queue for this repository.

:hourglass: Testing commit f4985827d7d6dbdeccf98a721e7a6b4c2c75565c with merge f911dfdb05b4cf647e0e72c4e8032cb59baf36de...

:sunny: Test successful - checks-actions
Approved by: ehuss
Pushing f911dfdb05b4cf647e0e72c4e8032cb59baf36de to master...


Can I use rethinkdb.ninja domain [Question]

Hi,
I registered a domain called RethinkDB.ninja, going to deploy this https://github.com/axcoto/rethink-weekly
Then I realized that this us very confusing to other people think that rethinkdb.ninja is part of rethinkdb.com and also maybe illegal to use rethinkdb phrase in my domain name with a different tld.
Do you guys think it's ok for me to use this domain? If not, I will go find another domain.
I think I shouldn't use it, but I already bought it so I just asked it here :)

@kureikain We're looking into it and will have a response for you soon.

@kureikain sorry this has fallen through the cracks. Had we actually gotten back to you about this (I feel like probably not)?


Provide context to the pathfinder.

For the moment the pathfinder finds connection between nodes by calling getConnections() on the IndexedNode.
My problem is that there is no context to decide if a connection should exist or not or to provide a different cost for the connection.
Simple example, you could imagine that some units could go over water cells while other cannot. Or that some agent have a different cost for a specific connection. Problem is, there is no way to tell for which entity (i.e. context) the pathfinding is running.

You have to inform the graph before searching the path

That could be a solution but you can only search one path at a time then.

The Indexed A* algorithm imposes this limitation already

@davebaol sending the path request through getCost wouldn't help here? @Nauktis?

Hmmm... PathFinderRequest is used for time sliced pathfinding only.


Include 'display address' in the DAP submission data

What is the context of this PR?
We need to include the display_address in the submission data sent to DAP.
display_address is a schema metadata value that has now been included in converter.py.
How to review
On submission of a Census survey, check that the submission payload now contains the 'display_address' claim.

 Include display_address in DAP data


Coordinates display incorrectly depending on the user's locale

Because some regions use comma for decimal places, and others use periods users in locales where periods are used show incorrect coordinates.

Fixed in https://github.com/milankarman/fast-stronghold/commit/65eaa03a4803e4b8d83448f0ca75336ea9f4ee29


Stdlib defs/NG updates for 1.36.3/1.37

Updated stdlib_defs: conforms to 1.37 spec other than having child output elems for nodedefs, and kept geomattrvalue as well as adding geompropvalue.
Updated stdlib_ng: new transform* nodegraph implementations, minor naming fixes.
Updated stdlib TestSuite files to conform to new stdlib_defs.

Closing so we can instead push to a shared stdlib_updates branch.


Do not log ExecutableInfo races dealing with files

As they are verbose and not actionable

closing in favour of #2039


Add freezing rain plugin and CLI

As a IMPROVER science lead, I want a plugin and CLI to generate probabilities of freezing rain so that it can form part of an IMPROVER suite chain producing a blended level-3 freezing rain product.
Related issues: https://github.com/metoppv/mo-blue-team/issues/225
Only one temperature threshold is required, 273.15K. We want to multiply the probability of being below this threshold (data are above the threshold) by the sum of the rain and sleet probabilities for each rain/sleet threshold.
Acceptance criteria:

CLI accepts inputs in any order of: probability of rain (rate or accumulation), probability of sleet (rate or accumulation), probability of screen temperature (instantaneous or minimum-over-time). We could consider accepting any cube with temperature in the title, in case another developer wants to put in surface temperature.
Plugin rejects inputs that are not temporally consistent (all instantaneous, or all over identical time window)
Plugin rejects inputs that do not have consistent realization coords (but should run with and without this)
Plugin returns a single cube of probability of freezing rain with title probability_of_lwe_freezing_rainfall_rate_above_threshold and threshold coord name lwe_freezing_rainfall_rate (and equivalent for accumulations). The units and threshold values are inherited from the rain and sleet cubes, which must match. The cube passes the inspect-metadata CLI without errors.
Acceptance test uses data for a real freezing rain case. If it helps, widespread freezing rain was forecast for Sunday 14th Feb 2021 (from 10th) and possibly also across northern areas on 15th Jan 2022 (forecast from 13th). Another freezing rain case predicted by UKV/UKPP for Mon 29th Nov 2021 from Thu 25th Nov.

Two cache building suites launched to gather data ready for building acceptance tests:
http://fcm1/rose-bush/cycles/ppdev/?suite=uk_freezing_rain_cache (20210210T1200Z --> 20210210T2300Z)
http://fcm1/rose-bush/cycles/ppdev/?suite=gl_freezing_rain_cache (20210210T1200Z --> 20210210T1800Z)
Intended freezing rain period is 13th/14th February 2021.

First output
The calculation (P(sleet rate) + P(rain rate)) x P(temp < 0C) that we are using for our new diagnostic, using MOGREPS-UK inputs, gives a signal for some freezing rain this afternoon in Scotland. GPP does not give a signal using the same inputs, as can be seen below.
GPP output for 15Z 25/02/2022 at T+12:

New diagnostic output for 15Z 25/02/2022 at T+12:

Issues encountered
Stephen's envisaged plan for the freezing rain diagnostic was as follows:

One significant issue with this plan is a mismatch in the ensemble members. The tempminmem1h step provides 18 members, whilst the precipphase steps provide only 3 members. As such the diagram needs modification to remove the freezing rain blendcycles step, and instead include a time lagged ensemble step for the rain and sleet diagnostics following precipphase but prior to the calculation of the freezing rain diagnostic.
This is conceptually simple enough. The time lagging applied to the precip phases should mirror what has been done to the raw temperature forecasts passed into ECC. As such we should still be matching related ensemble members, though we need to check. The more tricky issue is that this step begins to become very memory hungry. The inputs to the freezing rain CLI will be:
rain / sleet cubes: 18x23x970x1042
temperature cube: 18x970x1042
This is quite a lot of data to handle. In a simple schematic we would create a freezing rain diagnostic with shape 18x23x970x1042 and pass this to realization_collapse. However that involves a lot of data in memory and IO. As such I will redesign the freezing rain plugin to slice over realizations of rain / sleet / temperature, to accumulate the probabilities for each precip threshold, and then average this data by the number of contributing ensemble members. This fixes the various contributing cycles as equally weighted, but it also massively reduces the data in memory and the amount of IO. It removes the realization collapse step from the chain as well.

Acceptance test data
I've generated freezing rain diagnostics using the proposed method over the 13th and 14th February 2021, with the forecasts produced on the 10th. The 14th was proposed in the description, and the 13th also has a record of freezing rain in Roger Brugge's weather diary. I include 3-hourly output over the two days below. This is not a "freezing rain" signal, but rather a precipitation falling through a cold lower atmosphere diagnostic; this has been decided as the right approach for now, though it gives a fairly widespread signal in the plots below; thresholds of 0, 0.1, and 1mm are shown left to right.

Acceptance test data 3-hour period
Below is plotted the acceptance test data produced using ENGLUK over a 3-hour period, with this period spanning 03Z to 06Z on Sunday 14th February 2021.

In the case of the KGO, the columns represent the realizations; I've included more the mogreps-g just to highlight that it is more than a 3-member ensemble.

All plots above have been updated following a bug fix to the plugin; prior to this a smaller number of realizations for any diagnostic was being accommodated by broadcasting. This did not ensure that realizations were matching.

Acceptance test data merged


Redundant label-has-for default?

Is there a reason why label-has-for's required option defaults to "every"?
It seems redundant to require both nesting and the htmlFor  attribute. I was confused why the linting was failing despite whether I nested or used htmlFor .

It's not redundant; some accessibility devices only support nesting; others only support for-ID linking - the only way to cover 100% of devices (an important goal of accessibility is that everyone has access) is to do both.

Some references:
Old WCAG 2.0:
H44: Using label elements to associate text labels with form controls
https://www.w3.org/TR/WCAG20-TECHS/H44
(doesn't mention nesting at all)
A previous version discouraged implicit labelling (nesting)
https://www.w3.org/TR/2008/WD-WCAG20-TECHS-20080430/H44.html

The HTML and XHTML specifications allow both implicit and explicit labels. However, some assistive technologies do not correctly handle implicit labels

Labeling Controls
https://www.w3.org/WAI/tutorials/forms/labels/

A label and a form control should be associated with each other either implicitly or explicitly

Whenever possible, use the label element to associate text with form elements explicitly

Generally, explicit labels are better supported by assistive technology.

A very important post by Steve Faulkner, Paciello Group a11y specialist and HTML 5.1 co-editor among many other things. The post is a bit old (2011) but still valid.
HTML5 Accessibility Chops: form control labeling
https://developer.paciellogroup.com/blog/2011/07/html5-accessibility-chops-form-control-labeling/

Until browser implementers fix their accessibility support, if you want controls to be understandable to AT users label controls using for and id. Do not use the control inside the label method. Do not use a combination of the 2 methods.

Labeled With Love
by Aaron Gustafson
https://www.aaron-gustafson.com/notebook/labeled-with-love/

Dragon’s Naturally Speaking also doesn’t recognize implicit association

Overall, I'd tend to think this issue should be re-opened for better investigation and research.
/Cc @ljharb @aardrian @stevefaulkner

Can you provide an example of something that breaks why both? Using both nesting and linking has been a best practice for over a decade, and I’ve never once heard a report that using both is harmful besides that one article you linked, which i claim is wrong.

@ljharb well it's not only the article... (maybe better to check who wrote it before claiming it's wrong? 🙂 ). Have you checked the W3C tutorial? https://www.w3.org/WAI/tutorials/forms/labels/
There, it's clearly stated "either implicitly or explicitly".
As far as I can remember, using both caused the label to be announced twice in some old browsers / AT combinations. However, I'm not arguing or blaming anyone 🙂 I'm only asking to reopen this issue for better research and investigation. I'm pretty sure @aardrian and @stevefaulkner can answer your question better than me and provide data if necessary.

If someone can provide a concrete browser or device - ideally still in use - where using both is problematic, I’ll be happy to reopen.

There is a table with support data linked in the Steve Faulkner post. Data are from 2012. See http://www.html5accessibility.com/tests/form-labels.html
From there, you can see that all software tested read out correctly the label:

with only label using for/id (except VoiceOver with Opera 11.6 which is a dead browser)
with only label wrapped

So, asking by default to implement both is a bit pointless. As per potential problems caused by using both, I'd defer to someone more expert and updated than me.


Use owo-colors instead of anstyle

Closes #179.
owo-colors was suggested as a replacement for anstyle by @mainrs because it exposes color support determination. After investigating this crate more, it turns out that it does this using the supports-color crate. The way that owo-colors exposes this ability does not work for my performance improvement, so I opted to use the supports-color crate directly.
However, while I was looking around the owo-colors docs, I found it to be much more ergonomic for exposing raw color codes as const. I've therefore replaced anstyle with owo-colors and confirmed that it has the same output.
I've also made color=auto default because supports-color has made it more powerful at determining color support.

Thank you for looking into this!

so I opted to use the supports-color crate directly.

Ok. Do we need the "full" functionality of this crate (checking for truecolor support etc.)? Does it provide anything additional on top of env::var_os("NO_COLOR").is_some()?

I've also made color=auto default because supports-color has made it more powerful at determining color support.

Well the main use-case that I want to support by making always the default (which, admittedly, looks like an error.. but is really on purpose) is piping into a pager. hexyls main selling point is the colorized output. So it would be a real shame to force users to use --color=always in case they want to pipe to less, for example.

Well the main use-case that I want to support by making always the default (which, admittedly, looks like an error.. but is really on purpose) is piping into a pager.

That's a little bit unusual, from my perspective. I didn't actually know that colored output worked when piping; I assumed it didn't because the previous auto check would base coloring on if it was a TTY or not. It makes more sense to me for auto to use terminal support and environment variables than whether it's piped into a pager.
With that in mind, I would construct the auto option as if it either supports colors in general through supports-color, or if it's piped i.e. not a TTY. That might seem a bit backwards but considering that paging colored output is an intended outcome I think it might be best.

I assumed it didn't because the previous auto check would base coloring on if it was a TTY or not. It makes more sense to me for auto to use terminal support and environment variables than whether it's piped into a pager.

That's how --color=auto works for a lot of tools. Try for example grep --color=auto … | cat or ls --color=auto … | cat. Both will not enable colors. This is not just useful for piping into other processes, but also if you want to pipe to a file. You typically don't want ANSI escape codes to end up in there.
hexyl behaves the same way when using --color=auto. It's just that we don't use this by default. For the reason given above.
Now the question is of course what to do with NO_COLOR. This is actually answered on https://no-color.org/, in FAQ item 2. I interpret this as suggesting that --color=… should overwrite NO_COLOR. So if a user has NO_COLOR set, but uses --color=auto or --color=always, hexyl should behave as if NO_COLOR was not present.


Add Rider offline self debug configurations for the backend solution

Same as https://github.com/JetBrains/resharper-fsharp/pull/231 but with --offline option

@DedSec256 Thanks!


No indent lines after 10 levels of indentation

Using Macvim with Vim 8.0. Everything works as expected except that when indentation level is over 10 the indent lines disappeared, as shown in attached pic. What could be the issue or is there any config that I'm missing? Thanks.

·let g:indentLine_indentLevel = 100·


new release

@c42f  should we make a new release ? Probably a breaking one v0.3.0 due to the protocol change.

Go for it! Just bump the Project.toml to 0.3.0 and trigger JuliaRegistrator if you're happy to do that, I think you have permissions?


New feature: Edit todo items

Current code does not have option to edit newly added todo items. Add support for the same.

Fixed


Create better examples and tutorials

When version 2.0 (#44) comes out the "Getting Started" instructions for Konami-JS will have to change. I like that the new version will integrate better with modern toolchains and processes, but  I don't want to lose the ability for anyone to copy-and-paste a couple lines of code to stick it on their blog.
I'd like new examples to link to CDN-hosted versions of Konami-JS at unpkg.com. The the code will need to be written so that a distribution-ready version of konami.js is accessible via npm. That touches on the discussion of build-tools and bundlers (#48) to some degree.
Here are some example ideas listed under "Coming Soon!"  on the current GitHub-hosted site:

How to present different easter eggs for gesture versus keyboard input
How to override the default code for both keyboard and gesture
More complex examples beyond a simple redirect
How to track the Konami Code using Google Analytics

Since Konami it's a small library, I don't think a documentation with Docusaurus or any other tool like this is necessary. Maybe a Wiki should do the trick.

Agreed. A Wiki would do fine, but seeing as lot of beginning developers end up finding and playing with the library I'd like to make more thorough demos for the GitHub page. Copy-and-paste examples for common uses cases and questions I've received over the years. Things like:

How do I make Konami-JS load a different web page?
How do I change the keyboard/gesture sequence?
How do I track Konami-JS "triggers" with Google Analytics?
How do I make a video or sound play?

That's all beyond the realm of basic API documentation, but I like the idea of providing these sort of examples, especially for newcomers.


iPhone XR Notification bug

Not getting notifications while Jailbroken
Device (please complete the following information):

iOS Version: 13.3
iOS Device: iphone xr
unc0ver Version: 4.1.1

Not good at this sorry for a lack of info

Also can confirm notifications not working

Notifications workaround not working ether

Update to 4.2.1 it’s fixed close thread


Upgrade carbon.identity-inbound-auth-oauth.version to 6.4.117

Fixes https://github.com/wso2/product-apim/issues/10115
related carbon-apimgt PR : https://github.com/wso2/carbon-apimgt/pull/9959

@msm1992  We will close this pr since the identity version update was not doing due to update complexity


Failed to run pip3 command

A brief description

Sometimes pip command fails, sometimes does not fail in WSL.
Troublingly this does not always happen...
~# pip3 install -U pip
python3: ../sysdeps/posix/getaddrinfo.c:2603: getaddrinfo: Assertion `IN6_IS_ADDR_V4MAPPED (sin6->sin6_addr.s6_addr32)' failed.
Aborted (core dumped)

Expected results

No core dump

Actual results (with terminal output if applicable)

~# pip3 install -U pip
python3: ../sysdeps/posix/getaddrinfo.c:2603: getaddrinfo: Assertion `IN6_IS_ADDR_V4MAPPED (sin6->sin6_addr.s6_addr32)' failed.
Aborted (core dumped)

My Windows build number

14986.1000

Steps / All commands required to reproduce the error from a brand new installation

sudo su
apt update
apt install python3 wget
wget https://bootstrap.pypa.io/get-pip.py
python3 get-pip.py
pip3 install -U pip

Strace of the failing command

https://gist.github.com/kozo2/5dcb761b828998d4562cba4af8e56ef6

Required packages and commands to install

python3 and pip
sudo su
apt install python3 wget
wget https://bootstrap.pypa.io/get-pip.py
python3 get-pip.py

It seems that this can by occasionally fixed by reboot.

I'm seeing repro as well, exact same error and file/line, this time related to apt-get, not pip. Likewise inconsistent. Will fail to work, but can then start working.
patrickt@DESKTOP-3F5IQKD:~$ sudo apt-get update
[sudo] password for patrickt:
0% [Working]http: ../sysdeps/posix/getaddrinfo.c:2603: getaddrinfo: Assertion `IN6_IS_ADDR_V4MAPPED (sin6->sin6_addr.s6_addr32)' failed.
Reading package lists... Done
E: Method http has died unexpectedly!
E: Sub-process http received signal 6.

One other observation: Once it starts working, the error does not appear to recur for a while (haven't found a good bound on this time). Eg, if run in a loop, once the command succeeds it will reliably succeed after.

This error also occured with
knishida@DESKTOP-ETL8RDS ~/ipynb> nb --no-browser --ip='*'
[W 15:02:36.278 NotebookApp] WARNING: The notebook server is listening on all IP addresses and not using encryption. This is not recommended.
python3: ../sysdeps/posix/getaddrinfo.c:2603: getaddrinfo: Assertion `IN6_IS_ADDR_V4MAPPED (sin6->sin6_addr.s6_addr32)' failed.
knishida@DESKTOP-ETL8RDS ~/ipynb>

Disabling IPv6 as mentioned in #2142 works.

#508 is one of the earlier cites


fix(frontend): fix mobile create table

🍰 Pullrequest
Makes the mobile create button functionality to enter my room work again
Issues

fixes #1353

I fixed the disabled tests and activated them.


[Docs] Add docs for KoP upgrade

Fixes https://github.com/streamnative/kop/issues/879
Add a dependent document to describe the KoP upgrade issue. And fix the wrong links of README in main page. Remove all links for KoP < 2.8.0 and recommend users to use KoP >= 2.8.0

I've addressed the comments and there are two differences from your comments. PTAL again @Huanli-Meng


Implement Subscriptions

Fixes #91

Looks good, Few nitpicks

Add screenshots
Check the feasibility of moving logic out of the view using presenter pattern or decorator pattern
Check if we can simplify the frontend operations with https://stimulusjs.org/

Thanks for the PR. Can you summarize what is the current behavior? I noticed that you generated views from the default commentator plugin, can you summarize the changes.

I have made small changes to views of subscription which differ from default commentator plugin
->There were syntax error in the orignal repo (I raised a issue in their repo as well)
->We may need to customize the mailer views.So having app/views/commontator/subscriptions_mailer/ is be necessary.
->The UI of overall commontator does not match with our UI.so would have to change it at some point or other.
Rest the working as usual

I have made small changes to views of subscription which differ from default commentator plugin
->There were syntax error's in the orignal repo (I raised a issue in their repo as well)
->We may need to customize the mailer views.So having app/views/commontator/subscriptions_mailer/ is necessary.
->The UI of overall commontator does not match with our UI.So would have to change it at some point or other in future.
Rest the working as usual

Thank you, could you also describe how subscriptions happen. Does it happen when the author creates the project, or when someone comments on the project. Or is it a manual subscruption. Also how are the emails sent? Are they sent for every comment? A screenshot for an email will be helpful.

->The email happens when someone else comments on the subscribed project.
->The User can opt to manually opt to subscribe.
->Emails are sent via smtp and would be using the same smtp settings.
->When a new comment is generated the email would be sent to all the subscribed users.it will enqueue the tasks in sidekiq which is then processed via smtp.

->The email happens when someone else comments on the subscribed project.
->The User can opt to manually opt to subscribe.
->Emails are sent via smtp and would be using the same smtp settings.
->When a new comment is generated the email would be sent to all the subscribed users.it will enqueue the tasks in sidekiq which is then processed via smtp.

Can you configure auto subscription? And manual unsubscription? I really doubt users will ever subscribe even if they would be interested.

These were available options , I changed it both automatic and manual.

Is the author of the project automatically subscribed even if he doesn't make a comment?

No,I just checked .Commontator does not provide this functionality.

No,I just checked .Commontator does not provide this functionality.The author would also have to subscribe to comments or automatically when commenting.

I am sure Commontator has some function which we can call to subscribe/unsubscribe? After all, they need to have a way to change it themselves. Simply add a after_create hook?

ill check

ill check

We can resolve this in a different issue.

undefined method `project_url'
This is the error that is coming in production, did you test the mail in development?

ya i tested with gem letter_opener
let me see again.I am in class right now.Will definitely look into it.

  config.comment_url_proc = ->(comment, app_routes) {
    app_routes.polymorphic_url(comment.thread.commontable, anchor: "comment_#{comment.id}_div")
  }

I think the answer is here.


#78 autoupdate to helm's version, not to default "latest" in values.yaml

Closes #78

Builds only on tags
Updating to helm's version, not to "latest", that is in values.yaml


Misc clippy fixes

I went over the crates with Clippy. Some suggested changes are only stylistic, others more optimization oriented. The commit messages should hopefully explain well enough why each change was made. Otherwise see what clippy says on current master :)
I left some warnings, since the suggested changes were not really objectively better, or possibly larger than what I wanted to do right now.

Also pulled in the removal of the autocfg dependency that we stopped using over in #189

This is technically a breaking change, but we're about to make a major version bump anyways...

bors r+

My thought exactly. I'm bumping versions and drafting a changelog at the moment. Will submit soon.


UVI Request

--- UVI JSON ---
{
  "vendor_name": "Linux",
  "product_name": "Kernel",
  "product_version": "versions from  to before v5.10.46",
  "vulnerability_type": "unspecified",
  "affected_component": "unspecified",
  "attack_vector": "unspecified",
  "impact": "unspecified",
  "credit": "",
  "references": [
    "https://git.kernel.org/pub/scm/linux/kernel/git/stable/linux.git/commit/?id=1a1dbc4473974867fe8c5f195c17b341c8e82867"
  ],
  "extended_references": [
    {
      "type": "commit",
      "value": "1a1dbc4473974867fe8c5f195c17b341c8e82867",
      "note": "fixed"
    }
  ],
  "reporter": "joshbressers",
  "reporter_id": 1692786,
  "notes": "",
  "description": "PCI: aardvark: Fix kernel panic during PIO transfer\n\nThis is an automated ID intended to aid in discovery of potential security vulnerabilities. The actual impact and attack plausibility have not yet been proven.\nThis ID is fixed in Linux Kernel version v5.10.46 by commit 1a1dbc4473974867fe8c5f195c17b341c8e82867. For more details please see the references link."
}
--- UVI JSON ---

This issue has been assigned UVI-2021-1000998


How to handle backpress on camera page?

I don't want the user to press back button on the camera page. how to achieve that? Help.

As per my limited knowledge you need to use the fragment scanner in your own fragment activity and handle back key press. But be careful to comply with Android specs.


Code release

Hi, this is a great work!
When do you plan to release the code? Can't wait to try it!

Hello, thank you for your attention, this work is under review, once the paper is accepted we will publish the code in time, I will @ you then!

OK, thanks! May I ask what's the resolution of the generatred avatar?

Hello, Body resolution is 512, clothing and accessories resolution is 256.


Fix network performance issue on arm64 for runv

Network performance in runv is lower significantly
than in virtual machine started by qemu,as vnet_hdr
feature is not enabled in runv when get access to
a tun/tap device. Adding CIFF_VNET_HDR flag to GetTapFd
func will fixed that issue.
Signed-off-by: Jianyong Wu<EMAIL_ADDRESS>Signed-off-by: Wei Chen<EMAIL_ADDRESS>@liwei @gnawux @bergwolf  help check it.

+1
thank you @jongwu , sound great

LGTM


Removing the Bobby Pin

When I first wrote this mod, the Paper Clip hasn't been in the vanilla game. Now that there is a paper clip, maybe we can remove the Bobby Pin?
I am not sure how rare Paper Clips currently are, but we could simply increase their spawn rates a bit.
What do you think?

Hi,
I don’t really know the frequency of paper clips. With the new build 32), i still use the lock picking mod and i found some paper clip (but not yet the screwdriver), maybe if you ask the community on the forum they can help to choose the right decision ?
Best regards.
Cyb

Le 6 juil. 2015 à 22:43, rm-code<EMAIL_ADDRESS>a écrit :
When I first wrote this mod, the Paper Clip hasn't been in the vanilla game. Now that there is a paper clip, maybe we can remove the Bobby Pin?
I am not sure how rare Paper Clips currently are, but we could simply increase their spawn rates a bit.
What do you think?
—
Reply to this email directly or view it on GitHub https://github.com/cyberbobjr/pz-lockpicking-mod/issues/2.

Okay, seems like there still is a reason to keep them in the game.


Release 0.15.2

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

Comparison is base (9c444ed) 33.69% compared to head (f1c5f6c) 33.69%.

:exclamation: Your organization needs to install the Codecov GitHub app to enable full functionality.
Additional details and impacted files
@@           Coverage Diff           @@
##             main      #83   +/-   ##
=======================================
  Coverage   33.69%   33.69%
=======================================
  Files          13       13
  Lines         650      650
=======================================
  Hits          219      219
  Misses        431      431

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


add local install of kubeflow

In order to be able to mess around with kubeflow, let's put that in the installer.

Could you add some context to this? Helpful to have some specific checkboxes too so that external contributors can get an idea of what needs to be done.

Closing due to inactivity.


No matching distribution found for botocore<1.24.0,>=1.23.34

Describe the bug
Boto3 Install fails with botocore dependency unable to match a version.
Specifying boto3==1.20.31 works, which uses botocore==1.23.31
Steps to reproduce
pip install boto3
Expected behavior
package is installed

Hi @matty87a,
Can you provide some more info about the environment you're using like Python version and pip version? We're not able to reproduce the issue with the tests below:
pip install boto3

pip install --upgrade boto3

pip install "botocore<1.24.0,>=1.23.34"

Apologies,  this can be closed. It looks like an outdated version of pip was the culprit.


Create eks-chaos-mesh-stack.ts

That isnt an experiment template but rather a stack to deploy EKS cluster and ChaosMesh setup. While useful, it is not the place for that.


Fix: typo and compilation error

Motivation
Right now, pulsar build is failing due to typo introduced in #7078 . Fixed the typo.

@rdhabalia I've submitted the same fix this morning: #7144


Mobile App Reverse Engineering

I know you said it's secondary, but I did a lot of reverse engineering on their mobile app which relies mostly on Firebase. If you look at my source code https://github.com/dcmeglio/hubitat-petnet you'll see some info that could be useful. Figured I'd share in case it helps

oh wow, that is super helpful! very readable groovy code there. Can you help out? We have a Slack going but there are only a few people :/ need more brains to solve this.

I wouldn't mind helping but I've never worked in Python before, I've mainly been a .NET guy so I'll have somewhat of a learning curve. I was actually considering a different direction, I was going to replace the guts of the device with a rpi to control the sensors but I wouldn't mind trying both techniques out.

I'm using a rpi to hijack the API calls. The hardware seemed a little excessive for a pi (or esp32) brain replacement. It's a good excuse to run a PiHole locally :)
email me; the code shouldn't be too overwhelming.

I set up a fake DNS entry for api.arrowconnect.io and mqtt-a01.arrowconnect.io and it's connecting to both.
It appears it wants to connect to port 8883 for MQTT which it's reading as an HTTP request when I forward it to responderapp.py. Do I need to set up some kind of MQTT to websockets connector to get that part to work?
  -> HTTP protocol error in client request: Bad HTTP request line: b'\x10\xa9\x01\x00\x04MQTT\x04\xc0\x00'

In the meantime, I just setup a mosquitto server to see what it would want to do. I'm not that familiar with the internals of MQTT, but here's what the log output:
1601876314: New client connected from <IP_ADDRESS> as 40285608b03f085d1b4e31ec5f60b3b912f64f2d (p2, c0, k10, u'/pegasus:40285608b03f085d1b4e31ec5f60b3b912f64f2d').
1601876314: No will message specified.
1601876314: Sending CONNACK to 40285608b03f085d1b4e31ec5f60b3b912f64f2d (0, 0)
1601876319: Received SUBSCRIBE from 40285608b03f085d1b4e31ec5f60b3b912f64f2d
1601876319: 	krs/cmd/stg/40285608b03f085d1b4e31ec5f60b3b912f64f2d (QoS 2)
1601876319: 40285608b03f085d1b4e31ec5f60b3b912f64f2d 2 krs/cmd/stg/40285608b03f085d1b4e31ec5f60b3b912f64f2d
1601876319: Sending SUBACK to 40285608b03f085d1b4e31ec5f60b3b912f64f2d
1601876320: Received SUBSCRIBE from 40285608b03f085d1b4e31ec5f60b3b912f64f2d
1601876320: 	krs/api/stg/40285608b03f085d1b4e31ec5f60b3b912f64f2d (QoS 2)
1601876320: 40285608b03f085d1b4e31ec5f60b3b912f64f2d 2 krs/api/stg/40285608b03f085d1b4e31ec5f60b3b912f64f2d
1601876320: Sending SUBACK to 40285608b03f085d1b4e31ec5f60b3b912f64f2d
1601876320: Received PUBLISH from 40285608b03f085d1b4e31ec5f60b3b912f64f2d (d0, q1, r0, m4, 'krs.tel.gts.40285608b03f085d1b4e31ec5f60b3b912f64f2d', ... (209 bytes))
1601876320: Sending PUBACK to 40285608b03f085d1b4e31ec5f60b3b912f64f2d (m4, rc0)
1601876320: Received PUBLISH from 40285608b03f085d1b4e31ec5f60b3b912f64f2d (d0, q1, r0, m5, 'krs.tel.gts.40285608b03f085d1b4e31ec5f60b3b912f64f2d', ... (177 bytes))
1601876320: Sending PUBACK to 40285608b03f085d1b4e31ec5f60b3b912f64f2d (m5, rc0)

Closing this out since PetNet is very dead at this point and I doubt we will get much more from RE'ing the app.


Feature request: validate README.md samples

Motivation
Right now we're not testing these, but they show up as the readme on crates.io
Drawbacks
No response
Rationale and alternatives
No response
Additional context
No response

You can add this to the windows crate's lib.rs file to enable testing:
#[cfg(doctest)]
#[doc = include_str!("../../../../.github/readme.md")]
pub struct TestReadme;

The trouble is that rustdoc doesn't know to enable the necessary crate features to allow the code to build successfully so it fails, and I haven't found a way to tell rustdoc which features to enable.

Hey @jyn514 do you happen to know if there's a simple way to do this? Here's the readme:
https://github.com/microsoft/windows-rs/blob/master/.github/readme.md
As you can see, for each example there are various features that need to be enabled.

I am no longer on the rustdoc team. You can ask on Zulip if you like.

but in this case you probably want cfg_attr(feature = "xxx", include_str!("..."))

Thanks @jyn514 I'll give that a try - sorry to bother you. 😊

Haven't found a way to automate this testing. Will continue to test manually for the time being.


Define attribute packed on enums

Is there a way to define packed_struct on enums? I tried, but it seems to be ignored.
Would appreciate to be able to set that instead of using a global `-fshort-enums' or similar options on the project.

There is packed_enum option: https://github.com/nanopb/nanopb/blob/master/generator/proto/nanopb.proto#L75

Awesome! Maybe we should update the 'Proto file options' section on https://jpa.kapsi.fi/nanopb/docs/reference.html :)


common: refactor exponential backoff calculation

Hi,
this is my first time contributing to this project.
I noticed that the std::min() call in the exponential backoff interval calculation wasn't needed since backoff <= max_interval is already guaranteed, so I removed the call.
Additionally, since max_interval is const,  I added a const member for the "halfway point" when checking whether we can double again or clamp to max. Trying this in godbolt it seemed to get rid of the div/shift. Hope this is okay, tested it locally and UTs where green.
Best,
Frank

Risk Level: Low
Testing: Unit Test (backoff_strategy_test)
Docs Changes: N/A
Release Notes: N/A
Platform Specific Features: N/A
[Optional Runtime guard:]
[Optional Fixes #Issue]
[Optional Fixes commit #PR or SHA]
[Optional Deprecated:]
[Optional API Considerations:]

/assign @envoyproxy/first-pass-reviewers

/assign-from @envoyproxy/first-pass-reviewers


Symbol's function definition is void: tramp-tramp-file-p

I am newer to emacs. My emacs was installed throw Homebrew with version 24.5.1
When I learn you repo, specifically, ./lisp/init-utils.el file. I type M-x browse-current-file.
Something unexpected happened.
Please have a look and help me figure it out.
Thank you in advance.

Thanks, that's fixed in d4d51d3.

@purcell Thank you very much for your help.

Thanks for filing the issue, and for trying out my emacs config.


EFT optimizations

EFT optimizations, "with data" images, remove mounts and volumes

update to with data images for redoer and stream loader

public.ecr.aws/senzing/redoer-with-data:1.3.10
public.ecr.aws/senzing/stream-loader-with-data:1.8.4

remove EFS data mount for redoer and stream loaders

MountPoints: /opt/senzing/data
Volumes: RootDirectory: /data/2.0.0

PR merged


404 error while fetching GDB sources

Change the link in 14 lesson README to http://ftp.rediris.es/mirror/GNU/gdb/gdb-7.8.tar.gz because of 404 error

Thanks to both of you for reporting and fixing the issue!


[Feature]: The noVNC can be accessed by subpath (https://example.com/novnc)

Is your proposal related to a problem?
Currently, the noVNC client assumes it is being accessed directly via http://localhost:6080/. However, when I place it behind a reverse proxy (NGINX), the resources are not accessible.
Additionally, the WebSocket connections are always established using the insecure ws:// protocol, which gets blocked when using HTTPS connections through NGINX.
Describe the solution you'd like.
The noVNC server can be accessed via URL like https://example.com/novnc
Describe alternatives you've considered.
I tried my best to set up Nginx, it cannot work perfectly. The Nginx Config is given below.
location /novnc/ {
      proxy_pass http://<IP_ADDRESS>:8006/;

      sub_filter 'href="/' 'href="/novnc/';
      sub_filter 'src="/' 'src="/novnc/';
      sub_filter_once off;
}

Additional context
No response

There are too many hard-coded paths:
scripts.js
function getInfo() {
    var url = "/msg.html";
    // Should be "msg.html"
    ...

function processInfo() {
       ....
            setInfo("Connecting to VNC", true);
            var webSocket = webSocketFactory.connect("ws://" + window.location.host + "/websockify");
            // Should be:
            // var protocol = window.location.protocol == "https:" ? "was:" : "ws";
            // var wsUrl = protocol  + "//" + window.location.href + "/websockify"
            // var webSocket = webSocketFactory.connect(wsUrl )
      ....


feat: override proctoring requirements email

Pre-Flight checklist

[x] Screenshots and design review for any changes that affect layout or styling

[x] Desktop screenshots
[ ] Mobile width screenshots

[x] Testing

[x] Code is tested
[ ] Changes have been manually tested

[ ] Settings

[ ] New settings have reasonable development defaults, if applicable
[ ] Opened issue in the ol-infrastructure repo for any necessary configuration changes to deployed environments

What are the relevant tickets?
https://github.com/mitodl/hq/issues/725
What's this PR do?

Adds a template override for the proctoring requirements email that's sent when a user enrolls in a course that has proctoring enabled and has proctored exams in it.

How should this be manually tested?
There are two options to test this locally.

Setup Proctoring backend, Configure eCommerce

Follow the instructions on https://edx.readthedocs.io/projects/edx-partner-course-staff/en/latest/proctored_exams/proctored_enabling.html to enable proctored exams
Create a course mode with any of Professional, Masters, Verified
Setup the ecommerce things for the course https://edx.readthedocs.io/projects/edx-installing-configuring-and-running/en/latest/ecommerce/index.html
Configure email to work locally

If you are looking for a simpler way to test the PR (See below)

open openedx/core/djangoapps/enrollments/api.py and at line#591 add CourseMode.HONOR in the appropriate_modes list

(Common Steps)

Setup this theme
Create a course in the studio
Create a course mode Honor (For testing so that we don't have to enable e-commerce functionality)
in Advanced Settings enable Enable Proctored Exams
In the course content, add a section/sub-section/unit and mark it as Proctored through the section grade setting
Now enroll in this course
check /edx/src/ace_messages directory you should see *.datestamp.html files for all the emails being sent.

Where should the reviewer start?
(Optional)
Any background context you want to provide?
(Optional)
Screenshots (if appropriate)
Before

After

I've setup a course https://rc.mitxonline.mit.edu/courses/course-v1:Arbisoft+ARB_TPC_1/  on Rc that can be used to test proctoring email for cross matching.

It seems like having a link to instructions and requirements would be good. Is there a reason why that's omitted in the screenshot?

Oh, nevermind. I see the comment from Shira when I asked for a link:

➤ shira commented:
pdpinch not really. All Proctoring Instructions and Requirements would be different dependent on the course team. I think we should probably just remove that link if we can.

Two more comments:

Will the broken image be fixed when we go to production?

The paragraphs are very close together. Can we add a line break or some other spacin?

The code and test look good. Just need to add some spacing for the  tag per Peter's comment.

Two more comments:

Will the broken image be fixed when we go to production?

Yes, This is a cdn/cloudfront url broken locally, but it should work fine on RC and Prod.

The paragraphs are very close together. Can we add a line break or some other spacin?

Yes, Added them, Updated the screenshot in the PR description. Could you please take a look and let me know if it looks good to you.


Fixed #26430, Refs #24462 -- Added BaseSubquery datastructure.

Added new Join-like class BaseSubquery which is meant to be used in
FROM clauses. Rewrote Query.get_aggregation with it. Deprecated
AggregateQuery and SQLAggregateCompiler as redundant.

Simon, I think it would be really better if you could take ownership of the patch. I would really like to help, but doing thing blindly for which I don't understand implications is not wise.

@gizmondo, do you have any following patches, even POCs, to reach #24462 goals?

@claudep, first of all, thanks for trying to revive this effort!
There were some experimental patches, but unfortunately their history is in a private repo to which I don't have access anymore. Nothing particularly valuable there anyway, afair.

It think this PR is partly obsolete. @charettes, if you think anything here is worth resurrecting, please speak!


Input and output files are not containing any multibyte characters

I would like to recommend taking into consideration, if the benchmark is intended as a real-world representation, to mix in non-ASCII characters.

Thanks for the suggestion but that shipped has sailed. I can't modify the problem statement this late into the competition.

Clarification, does anything in the rules prevent implementers from choosing libraries that only work with ASCII files and strings? This can make things faster and reduce memory footprint.

That's a tricky one, while the problem doesn't specify, as you said...in a real world scenario, it's reasonable to expect that utf8 is supported. I think I will modify the rules as opposed to modifying the problem.

Agreed. And explicit introduction of Unicode support would imply correct and complete implementation including collation, e.g. whether ñ is greater than n and less than o :)
That is indeed a can of worms.


adding int type to argparse arguments nodes and tasks per node

Each of "node" and "tasks_per_node" is parsed as a number (eg %d) in the script to execute the launch, so they should be input as ints. This will close #1

added module functionality to this PR! Will close #3


"Caption" in website

Replace "Caption" on the video with
QuISP in action: setting up connections, then beginning transmission of photons in a network with five quantum routers, five quantum repeaters, and ten quantum end nodes.

The caption is replaced in index.html


Add joypad input map conversion to project converter

Fixes godotengine#77314
Changes

Converts all of the Godot 3 joypad button index mappings to their Godot 4 equivalent mappings. "button_index":10 -> "button_index":4
On Godot 4, the entries for the L2 and R2 buttons have been moved to the axis mappings, so entries for button index mappings for those buttons get converted to axis mappings.
InputEventJoypadButton, "button_index":6,"pressure":0.0,"pressed":false -> InputEventJoypadMotion, "axis":4,"axis_value":1.0
Converts the Godot 3 axis mappings for L2 and R2 to the Godot 4 axis mappings . (These are the only 2 axis mappings that have changed).
"axis":7 -> "axis":5

Testing
Tested by running the converter and the validator with a Godot 3 project that contains a project file with the following entries:

Button mapping that should be converted to its Godot 4 value.
Button mapping that shouldn't be converted (Same mapping in 3&4)
Axis that should be converted. (L2 and R2).
Axis that shouldn't be converted.
Button index that's greater than the ammount of mappings (23).

All of these conversion get performed as expected.
Also 3 validation tests have been added in the code for the most general conversion cases.

Thanks!


middleware: fix issue #4

replace net.SplitHostPort with core.SplitHostPort (available in core 0.6.2+)

v0.0.4 tagged @karasz


(electron) loadUrl is deprecated. Use loadURL instead

(electron) loadUrl is deprecated. Use loadURL instead

I'm aware of these being a non-conflict and should fix that message. Thanks.
ACK.


#18 Add non-tested gdeh116Z91.cpp 11.6 color EPD

This was an epaper that was going to be implemented in issue #18
Sadly no-one had time to test it yet and I don't have this model myself. So it will stay some weeks open and if not we can close this and implement it some time in the future

Won't merge due to lack of interest / Feedback on this epaper


Show number of rows processed

to fix #261, suggestions listed in https://github.com/theodi/csvlint/issues/261#issuecomment-295339728 advised for subsequent fixes

Coverage remained the same at 95.556% when pulling 54d20d161b4f266be8216d850cfb3376c086b943 on show-number-of-rows-processed into c70f80b794bb1f3aa6d92356909da0a1d02f464e on master.


How to use ajax, please, don't make it so complicated

How to use ajax, please, don't make it so complicated

Making AJAX requests is straight forward as it is. @imsheng do you have any specific issue?

I believe we have clarified this on the new documentation site: https://select2.org/data-sources/ajax


Automated release via Github Actions

This PR adds CI to build and release ungoogled chromium for windows automatically. Thank @networkException for providing the multi-step building script.
Fixes #108

Can you rebase this on top of the master branch?

I think it's because you set is_official_build.
https://github.com/ungoogled-software/ungoogled-chromium/issues/1935#issuecomment-1122361329

I intentionally did not set the flag in the last PR as the build process takes longer to finish and requires more HDD and RAM (at least on my machine). That's also mentioned in the issue referenced by @Nifury.
However, I'm curious to see how the compile time is affected in the CI. If it's not notably different, we could set the flag by default.

It's because of the 'is_offical_build' flag. This requires an additional patch for the generate_resource_allowlist.py file to be applied which is normally not needed and therefore not included here.
In my fork of this repo, there is a branch with the flag set. You can find the required patch in this branch.

Thanks! giving it another shot at https://github.com/LABPBS/ungoogled-chromium-windows/actions/runs/2476986726

However, I'm curious to see how the compile time is affected in the CI. If it's not notably different, we could set the flag by default.

24 hours in GHA for the failed one

It's because of the 'is_offical_build' flag. This requires an additional patch for the generate_resource_allowlist.py file to be applied which is normally not needed and therefore not included here.

In my fork of this repo, there is a branch with the flag set. You can find the required patch in this branch.

Thanks! giving it another shot at https://github.com/LABPBS/ungoogled-chromium-windows/actions/runs/2476986726

Did the build succeed and if yes, how long did it take? The link you posted is no longer valid.

Great!
Thanks


docs: add RPM extraction to install guide

Update the install guide with details of how to manually extract the
driver source files from an RPM. Without Debian package support,
some users are forced to use this method on Debian-based systems.

We call cpack -G DEB from the build directory to generate .deb packages. Does that not work?

Haven't tried it..  Song asked me to add this to support some AE's who are currently having to do this to a release RPM (they don't consume the source, so aren't able to generate a .deb).


python312Packages.netcdf4: 1.7.1.post2 -> 1.7.2

Diff: https://github.com/Unidata/netcdf4-python/compare/v1.7.1.post2...v1.7.2rel
Changelog: https://github.com/Unidata/netcdf4-python/raw/v1.7.2rel/Changelog
closes https://github.com/NixOS/nixpkgs/pull/361953

Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[x] Fits CONTRIBUTING.md.

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

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 365242

x86_64-linux

:fast_forward: 6 packages marked as broken and skipped:

python311Packages.rio-tiler
python311Packages.rio-tiler.dist
python311Packages.sunpy
python311Packages.sunpy.dist
python312Packages.rio-tiler
python312Packages.rio-tiler.dist

:x: 28 packages failed to build:

python311Packages.arviz
python311Packages.arviz.dist
python311Packages.bambi
python311Packages.bambi.dist
python311Packages.corner
python311Packages.corner.dist
python311Packages.flowmc
python311Packages.flowmc.dist
python311Packages.nutpie
python311Packages.nutpie.dist
python311Packages.pymc
python311Packages.pymc.dist
python311Packages.wrf-python
python311Packages.wrf-python.dist
python312Packages.arviz
python312Packages.arviz.dist
python312Packages.bambi
python312Packages.bambi.dist
python312Packages.corner
python312Packages.corner.dist
python312Packages.flowmc
python312Packages.flowmc.dist
python312Packages.nutpie
python312Packages.nutpie.dist
python312Packages.pymc
python312Packages.pymc.dist
python312Packages.wrf-python
python312Packages.wrf-python.dist

:white_check_mark: 44 packages built:

python311Packages.boltztrap2
python311Packages.boltztrap2.dist
python311Packages.datashader
python311Packages.datashader.dist
python311Packages.gstools
python311Packages.gstools.dist
python311Packages.h5netcdf
python311Packages.h5netcdf.dist
python311Packages.meshio
python311Packages.meshio.dist
python311Packages.netcdf4
python311Packages.netcdf4.dist
python311Packages.pygmt
python311Packages.pygmt.dist
python311Packages.qcodes
python311Packages.qcodes-contrib-drivers
python311Packages.qcodes-contrib-drivers.dist
python311Packages.qcodes.dist
python311Packages.rioxarray
python311Packages.rioxarray.dist
python311Packages.sfepy
python311Packages.sfepy.dist
python312Packages.boltztrap2
python312Packages.boltztrap2.dist
python312Packages.datashader
python312Packages.datashader.dist
python312Packages.gstools
python312Packages.gstools.dist
python312Packages.h5netcdf
python312Packages.h5netcdf.dist
python312Packages.meshio
python312Packages.meshio.dist
python312Packages.netcdf4
python312Packages.netcdf4.dist
python312Packages.pygmt
python312Packages.pygmt.dist
python312Packages.qcodes
python312Packages.qcodes-contrib-drivers
python312Packages.qcodes-contrib-drivers.dist
python312Packages.qcodes.dist
python312Packages.rioxarray
python312Packages.rioxarray.dist
python312Packages.sfepy
python312Packages.sfepy.dist

nixpkgs-review result
Generated using nixpkgs-review.
Command: nixpkgs-review pr 365242

x86_64-linux

:fast_forward: 10 packages marked as broken and skipped:

python311Packages.rio-tiler
python311Packages.rio-tiler.dist
python311Packages.sunpy
python311Packages.sunpy.dist
python311Packages.wrf-python
python311Packages.wrf-python.dist
python312Packages.rio-tiler
python312Packages.rio-tiler.dist
python312Packages.wrf-python
python312Packages.wrf-python.dist

:x: 24 packages failed to build:

python311Packages.arviz
python311Packages.arviz.dist
python311Packages.bambi
python311Packages.bambi.dist
python311Packages.corner
python311Packages.corner.dist
python311Packages.flowmc
python311Packages.flowmc.dist
python311Packages.nutpie
python311Packages.nutpie.dist
python311Packages.pymc
python311Packages.pymc.dist
python312Packages.arviz
python312Packages.arviz.dist
python312Packages.bambi
python312Packages.bambi.dist
python312Packages.corner
python312Packages.corner.dist
python312Packages.flowmc
python312Packages.flowmc.dist
python312Packages.nutpie
python312Packages.nutpie.dist
python312Packages.pymc
python312Packages.pymc.dist

:white_check_mark: 44 packages built:

python311Packages.boltztrap2
python311Packages.boltztrap2.dist
python311Packages.datashader
python311Packages.datashader.dist
python311Packages.gstools
python311Packages.gstools.dist
python311Packages.h5netcdf
python311Packages.h5netcdf.dist
python311Packages.meshio
python311Packages.meshio.dist
python311Packages.netcdf4
python311Packages.netcdf4.dist
python311Packages.pygmt
python311Packages.pygmt.dist
python311Packages.qcodes
python311Packages.qcodes-contrib-drivers
python311Packages.qcodes-contrib-drivers.dist
python311Packages.qcodes.dist
python311Packages.rioxarray
python311Packages.rioxarray.dist
python311Packages.sfepy
python311Packages.sfepy.dist
python312Packages.boltztrap2
python312Packages.boltztrap2.dist
python312Packages.datashader
python312Packages.datashader.dist
python312Packages.gstools
python312Packages.gstools.dist
python312Packages.h5netcdf
python312Packages.h5netcdf.dist
python312Packages.meshio
python312Packages.meshio.dist
python312Packages.netcdf4
python312Packages.netcdf4.dist
python312Packages.pygmt
python312Packages.pygmt.dist
python312Packages.qcodes
python312Packages.qcodes-contrib-drivers
python312Packages.qcodes-contrib-drivers.dist
python312Packages.qcodes.dist
python312Packages.rioxarray
python312Packages.rioxarray.dist
python312Packages.sfepy
python312Packages.sfepy.dist


Add dual-stack support to HostedCluster

What this PR does / why we need it:
Enables support for IPv6 in HostedCluster API. Deprecates existing
scalar values and replaces them with slices that mirror network
settings in InstallConfig.
Replaces individual fields for networking in the HostedControlPlane spec
with the networking struct of the HostedCluster spec.
These changes enable parity at the API level with standalone OpenShift
Which issue(s) this PR fixes (optional, use fixes #<issue_number>(, fixes #<issue_number>, ...) format, where issue_number might be a GitHub issue, or a Jira story:
Fixes #HOSTEDCP-500
Checklist

[x] Subject and description added to both, commit and PR.
[x] Relevant issues have been referenced.
[x] This change includes unit tests.

Supersedes https://github.com/openshift/hypershift/pull/1177

Thanks @csrwng drop some small comments, looks great!
PTAL @danwinship or please tag some one from networking team.

Thanks, squashed

@enxebre this is squashed (still 2 commits to keep the dependency bump separate), and is passing tests, ptal

/lgtm


Request creation of a tutorial

This is related to an issue posted in drywall, but it was less informative. https://github.com/jedireza/drywall/issues/378
So, the request is this,
Can we hope that there will be any good tutorial or some quick workaround to show people how they can add their own things to frame/aqua/drywall? Its deeply related to any framework.
For example, such situations,

User wants to add a field called favorite food in the framework? So that it can be shown in admin panel.
User wants to show this field favorite food in the home page after login.
User wants to see their favorite food after login instead of the admin panel with clock.

I used favorite food as example, but it can be just anything. No one knows this framework better than you, so I just thought it would be best if you could share such small tutorials with us so if we ever want, we could link others to these tutorials.
And, its an issue with documentation (quick start). If I could be of help, I'd glad to. But there should be a starting point at least.
Thanks.

Thanks for opening an issue.

Can we hope that there will be any good tutorial or some quick workaround to show people how they can add their own things to frame/aqua/drywall? Its deeply related to any framework.

These (Drywall/Frame/Aqua) are boilerplate apps, not frameworks. They're starting points which use other frameworks and modules. Express is a framework, hapi is a framework and the both have great documentation.

And, its an issue with documentation (quick start). If I could be of help, I'd glad to. But there should be a starting point at least.

One should be able to accomplish the "favorite food" examples you mention above by following the documentation of the frameworks and modules we use and following the code paths. A developer using these boilerplates really do need to be comfortable reading the code, debugging and finding relevant documentation to accomplish their goals.
With that said, the wiki is open and I'm happy to review any guides/tutorials you (or others)  create and/or link to blog posts.


Adds CONTRIBUTING info for tests.

Needs work.

Need to add information about the different classes of tests and how to write for each of them as per #223

@hamishwillee I updated this doc with all suites of tests and addressed your comments. Feel free to review again.
These docs will accompany the 1.4.0 release (which is mostly just the addition of the test infrastructure).

Hi @tcr3dr
Look pretty good. I pushed some minor subedits for style structure.  Please check you're happy with those.
Comments:

This branch has no unit tests so I can't assess how useful the unit test advice is
FYI, the theme I'm testing against doesn't differentiate 4th level headings, which is one reason I changed the heading levels (the other being I think it improves the structure).
The web client is not actually part of the Python public API - as you will see in the API reference it is not documented. It would be, but too much was not implemented so we hid it. We might want to remove this altogether from docs until next version of the cloud API.
The integration test advice says ""Avoiding printing any data from your test.". However test_12.py file we supply does have prints. I think this is an old version of test_simple.py and probably should be removed
We advise to use asserts. a) should we state in text that you can pull asserts_equals from testlib? b) That is the only assert I see - do we need to provide (and document) others or is this enough?
Should we be further recommend need for meaningful names for the files. test_simple.py is not as useful for searching as test_vehicle_parameters.py (and I think we should change this name too unless this is just considered a "demo" file - in which case perhaps "test_simple_demo" might be even better.)
Should we also recommend that each file and method added be given brief documentatio? I think this a good idea.
We might want to move the example of setting environment variables (or copy it) up to the integration test section, since here we also state that you need to set them.

@hamishwillee I address most of your concerns.

The branch is now rebased so you can test if unit tests work.
I added a warning to, but didn't remove, the Web API test section (for completeness)
The recommendations for test_simple and test_12 are good and I'll open an issue for those.

If this passes CI and you are happy with it, let me know!

@tcr3dr Really good fix/response! I created #233 (like #232) and added a couple of minor typo fixes to highlighting of assert_equals.
My only concern is that we refer to mock but there is no useful real-world example (e.g. test_api.py) that shows enough to know "correct use".
Consider the example:
import mock
from mock import MagicMock
import droneapi
from droneapi.module.api import APIModule
from nose.tools import assert_equals

def test_mode():
    api = APIModule(MagicMock())
    res = api.get_connection()
    assert_equals(len(res.get_vehicles()), 1)

From a user perspective trying to work out how to use Mock with dronekit I wonder what APIModule is (since it isn't public). Why not use the local_connection?
Reading the mock docs I might assume that here you are providing the connection to use, and this might set up a fake connection definition that guarantees that get_vehicles() returns 1, but I can't see where that might be set?
The use of Mock implies that these tests never run against a real device - correct?
Upshot, a little more explanation of mock or a better example which we link to as an example would help.
I would be happy for what we have here now to go out and this additional material to be provided in a separate issue if you preferred.

@hamishwillee Good points. I'll split that into a separate issue. Mostly because I'm not super familiar with Mock myself yet, though I've seen it used extensively by @eliao to good effect. As the tests improve, so shall the docs.

I'm relatively new to Mock myself, but it seems quite handy.  It's very useful for isolating tests so that they test only exactly what they are meant to.  If the tests include networking, etc, then they can fail/succeed due to things that are outside their control.  Unit testing vs. functional testing.


Save / load nest

Is there any way to save / nest to textile?

Could you please be more specific. What do you mean by textile?

Sorry i mean text file. Csv, xml or like.

There is no such option so far. But I'll try to implement it asap

The function is available in
https://github.com/9swampy/DeepNestSharp
Maybe you can get inspiration there?

Now it is possible to save/load a project to xml file


[Refactor] Use ThemeContext instead of withTheme

Now that there is a way to access a theme through hooks I think it makes sense to replace withTheme function calls with ThemeContext.
See here for more details: https://www.styled-components.com/docs/advanced#via-usecontext-react-hook

@korhaliv what do you think about this. I can do this if ACKed.

I'd like to see this done @bolatovumar . Using the useContext hook to access the current theme would be preferable to using the withTheme HOC and result in slightly cleaner and more easy to read code IMO.


[spk] Introspection dashboard should store filters in URL query string

As a: developer/manager/operator
I want: to run the service introspection dashboard with filters
So that: I can bookmark or share a URL which will run the filters I want to apply
Describe the solution you'd like:
Currently the solution being worked on for filters applies client side filters without updating the URL.

[ ] Update the dashboard to apply query strings for applied filters, and apply filters for query string in the URL

Acceptance Criteria:
Dashboard applies filters based on URL query string
Describe alternatives you've considered:
Considered the possibility of keeping the filters client side but the user may not be able to share/bookmark the URL
Additional context:
Does this require updates to documentation?: No

Opened a bug https://github.com/microsoft/azure-devops-ui/issues/257 since pre-selecting items in the filter bar is not straightforward with the azure-devops-ui library. Would be great to get some response from the azure team, otherwise, it's looking difficult to get around pre-selecting items.

Managed to get around this with the help of https://github.com/microsoft/azure-devops-ui/issues/257#issuecomment-563503939


Move to Keycloak 22.0.4

Updated required libraries
Adjusted JDBC Ping configuration (related to the PR submitted today https://github.com/mbta/terraform-keycloak-sso/pull/16)
Note: Since the base image is now ubi9-micro, there is no package manager at all.

@jurakp at some point can you push the updated mbta-keycloak-browser-authenticator code to the https://github.com/mbta/keycloak-browser-authenticator repo?


Maximum time delay

Check Maximum Time Delay between request and response

Will be fixed in Release 1.1.0


Java bytecode parsing/execution

Execution of arbitrary Java bytecode.
Code examples:
ICONST_2
ICONST_2
IADD
I2D
DRETURN

Adds 2+2, casts to double, and returns.
GETSTATIC "java/lang/System" "out" "Ljava/io/PrintStream;"
LDC "Hello, World!"
INVOKEVIRTUAL "java/io/PrintStream" "println" (Ljava/lang/String;)V
RETURN

Prints "Hello, World!" to the console
ICONST_0
ISTORE 1
:L1
ILOAD 1
ALOAD 0
ARRAYLENGTH
IF_ICMPGE L2
GETSTATIC "java/lang/System" "out"  "Ljava/io/PrintStream;"
ALOAD 0
ILOAD 1
AALOAD
INVOKEVIRTUAL "java/io/PrintStream" "println" "(Ljava/lang/String;)V"
IINC 1 1
GOTO L1
:L2
RETURN

Assuming this is put in the entry method, logs all arguments to the console.
Things that would be good:

automatically adds generic class structure around bytecode (so you don't have to write it yourself, because that's tedious)
detects return type and accommodates for that. Logs return value automatically.
Logs stack (execution stack, not call stack) frames after each instruction
possibly split into the following categories:

mjbytecode: places the provided bytecode in a custom method, with arguments you define. Class definition & main method are automatically generated.
fjbytecode: you provide the full bytecode to be executed. Doesn't add any surrounding class stuff.
jbytecode places the provided bytecode in the main method, and builds a class around it.

This could perhaps be done with Jasmin

Jasmin is out of date. There's an alternative, Krakatau, that's been updated March this year.


Add cron entry for sending admin issue reports

This cron entry is for 8:15 every Monday morning. Not sure if Sun night might be better when (I assume) there is less going on.

@ldodds would you mind also checking the times of this cron entry please? Just want to make sure it doesn't clash with anything else.


Fix #198

This commit introduces a boolean flag that tracks whether we currently execute a transaction related SQL command. We set this flag to true directly before starting the future execution and back to false afterwards.
This enables us to detect the cancellation of such futures while the command is executed. In such cases we consider the connection to be broken as we do not know how much of the command was actually executed.

Hey the PR looks great, exactly to the point. I updated the diesel-async implementation on the reproduction repository and yes, it fixes the issue.
Personally I think this solution is great because, even though cancellation is still bad, it normalizes the behavior, which is not ideal (due to rust limitations) but it's still a step up.

Thanks for testing. I will try to fix the CI next week and afterwords that should be ready to go.


Option to seed vhosts (and users) from deployment manifest

When CF platform users need a RMQ service, they can obtain it in a self-service manner using the marketplace/broker.
But we also would like to use the RMQ service on a infrastructure level - and for this, we would prefer to configure the needed vhosts and users directly in the deployment manifest so that they're automatically provisioned on deployment time of the BOSH release.
I.e. the cf-mysql release (that is not only used as a service, but very often also on infrastructure level, i.e. to back CF components themselves) offers such an option:
https://github.com/cloudfoundry/cf-mysql-release/blob/62b1e35c31ba412c1647e2a7f916c5fa5f89a6dc/jobs/mysql/spec#L127
https://github.com/cloudfoundry/mariadb_ctrl/blob/b2b272a1ef3bca647bd314b8524dcbb1ca3bba8e/mariadb_helper/mariadb_helper.go#L202
Would you welcome such a PR that adds the option to specify such "seeded" vhosts/users to the RMQ release?
How we imagine it:
# add a new configuration option:
  rabbitmq-server.seeded_vhosts:
    description: 'Set of vhosts to seed'
    default: {}
    example: |
      - name: vhost1
        username: user1
        password: pw1
        tags: administrator
        permissions: .* .* .* # maybe also split this up to the individual fields conf/read/write?
      - name: vhost2
        username: user2
        password: pw2
        tags: monitoring
        permissions: conf read write

These vhosts, users and permissions would be created upon startup (effectively overwriting already existing user attributes/permissions, similar to how the admin user provisioning is done).
This could then be implemented in a couple of lines of shell script in rabbitmq-server.init.bash, reusing (and probably refactoring) the methods handling the admin/mgmt user setup.

You want definitions import. I highly recommend not reinventing them on top of BOSH.

Are you talking about Load definitions at startup?
If we can specify this in the deployment manifest - which is not possible currently - this could indeed solve our problem.

@MatthiasWinzeler yes, that section. I'm pretty sure there is a way to specify arbitrary config settings in Ops Manager, so presumably there should be a way to do that with straight BOSH.
Even if somehow that's not possible or very difficult, making that possible should be the goal instead of reinventing definitions import with BOSH and rabbitmqctl.

@michaelklishin You're right, we could set the config entry load_definitions using the  rabbitmq-server.config key.
But: load_definitions needs a JSON file on the server itself - so we would need to add a mechanism to the BOSH release to write this file to the server.
What do you guys think about this:
# Introduce a new configuration item
rabbitmq-server.load_definitions:
  description: definitions (JSON) to be imported on RabbitMQ startup
  default: null
  example: |
    {
	"users": [{
		"name": "defuser",
		"password": "defpasswd",
		"tags": "administrator"
	}],
	"vhosts": [{
		"name": "from-def"
	}],
	"permissions": [{
		"user": "defuser",
		"vhost": "from-def",
		"configure": ".*",
		"write": ".*",
		"read": ".*"
	}]
}

This would then write the contents as a file to the FS and point the RMQ config entry to it.

@MatthiasWinzeler we recently implemented this feature. We expose load_definitions property in the rabbitmq-server.  Differently from your suggestion the property is not a json serialized as string but directly a structure that will be serialized and loaded using http://www.rabbitmq.com/management.html#load-definitions.
The feature is available from release v226.6
rabbitmq-server.load_definitions:
 description: definitions (JSON) to be imported on RabbitMQ startup
 default: null
 example:
   vhosts:
   -  {name: from-def}
   users:
   - name: defuser
     password: pwd
     tags: administrator
   permissions:
   - user: defuser
     vhost: from-def
     configure: '*.'
     write: '.*'
     read: '.*'

Nice to hear - thank you!


Deploying service api failed when using  todo-nodejs-mongo-aks template

Describe the issue:
Failed to deploy service api Whe run  bash ./test-templates.sh -t todo-nodejs-mongo-aks -b staging.
The error message as follows:

Repro steps:

Run command azd auth login.
Run command  bash ./test-templates.sh -t todo-nodejs-mongo-aks -b staging

Environment:
OS: MacOS desktop, Linux desktop,  WSL, DevContainer in VS Code,CodeSpace.
Template: todo-nodejs-mongo-aks
Azd version: 0.8.0-beta.2-daily.2713284  (commit https://github.com/Azure/azure-dev/commit/cb28058af1e7139be4381532f6b1167d9cd948fb)
Expected behavior:
bash ./test-templates.sh -t todo-nodejs-mongo-aks -b staging can pass.
@rajeshkamal5050 for notification.

Not able to reproduce.  The template provisioned & deployed without issue in Windows & WSL.
Note: If running on an MSFT internal azure subscription the tests will fail because of VM based networking restriction put in place.

When not using the MSFTVPN network, the following error occurs：

@rajeshkamal5050 This issue also occurs in the following cases:

OS: WSL, Windows desktop, MacOS desktop, Linux desktop, Devcontainer in VS Code, Codespaces
Template: todo-nodejs-mongo-aks
Branch: Staging
Azd version: 0.8.0-beta.3-daily.2760917  (commit https://github.com/Azure/azure-dev/commit/2201b16963dc43ae045c52374c0dd395152cb309)

@rajeshkamal5050 This issue also occurs in the following cases:

OS: WSL, Windows desktop, MacOS desktop, Linux desktop, Devcontainer in VS Code, Codespaces
Template: todo-nodejs-mongo-aks
Branch: Staging
Azd version: 0.8.0-beta.3-daily.2763383  (commit https://github.com/Azure/azure-dev/commit/3f484a283b309b3cab6f91386a4a7af7f4a082a2)


Option 2: CocoaPods -- Untested Way -- Works

Hello, I've been trying to build stuff without manual intervention.
Using default app.xcodeproj/project.pbxproj file generated by react-native upgrade.
Adjusted Podfile from the AIR example:
# You Podfile should look similar to this file. Whether you use_frameworks! or not, the following configuration should work.
#
# However if you DO NOT use_frameworks! and you prefer to install pods instead of
# dragging the AirMaps directory into your project, refer to the comments below (steps 1~4)

source 'https://github.com/CocoaPods/Specs.git'

platform :ios, '8.0'
# use_frameworks!

# Change 'extraSpecialCoolAppName' to match the target in your Xcode project.
target 'extraSpecialCoolAppName' do
  pod 'React', path: '../node_modules/react-native', :subspecs => [
    'Core',
    'RCTActionSheet',
    'RCTGeolocation',
    'RCTImage',
    'RCTLinkingIOS',
    'RCTNetwork',
    'RCTSettings',
    'RCTText',
    'RCTVibration',
    'RCTWebSocket'
  ]

  # when not using frameworks  we can remove this line (2/4):
  # pod 'GoogleMaps', '~> 2.1.1'  # <~~ remove this line if you do not want to support GoogleMaps on iOS

  # when not using frameworks  we can do this instead of including the source files in our project (1/4):
  pod 'react-native-google-maps', path: '../node_modules/react-native-maps/react-native-google-maps.podspec'  # <~~ if you need GoogleMaps support on iOS
  pod 'react-native-maps', path: '../node_modules/react-native-maps/react-native-maps.podspec'
end

# when not using frameworks this might be necessary, but try without it first and see if `pod install` works (3/4):
# THIS IS ONLY NECESSARY IF YOU NEED GoogleMaps SUPPORT
pre_install do |installer|
  # copied from https://github.com/CocoaPods/CocoaPods/issues/2926
  # workaround for https://github.com/CocoaPods/CocoaPods/issues/3289
  def installer.verify_no_static_framework_transitive_dependencies; end
end

# when not using frameworks (4/4):
# THIS IS ONLY NECESSARY IF YOU NEED GoogleMaps SUPPORT
# #Crud, this shouldn't be necessary, but https://github.com/CocoaPods/CocoaPods/issues/5429
post_install do |installer|
  installer.pods_project.targets.each do |target|
    target.build_configurations.each do |config|
        config.build_settings['CLANG_ENABLE_MODULES'] = 'NO'
    end
  end
end

It's also necessary to change GoogleMaps version in this .podspec to 2.1.1. GMSMapStyle is present in 2.1.x but is used with 2.0.1 in .podspec (build fails).
Then everything will work with no issues.

thanks for the report!


removed unneeded 'self' parameter.

this makes the migrations work.

Thanks!  This was fixed by #88

django-celery 2.4.2 is available at PyPI now.


Adding hbase 1.0 and 1.1 user agents.

Both bigtable-hbase-1.0/pom.xml and bigtable-hbase-1.1/pom.xml have the same artifactId and version. This mean bigtable-hbase-1.0 and bigtable-hbase-1.1 are going to send the same user_agent header. I don't think this is what we wanted.

I updated the code as per most of the comments.
I'm not sure what you meant by the last comment.  bigtable-hbase-1.1/pom.xml has an artifactId of  bigtable-hbase-1.1 and  bigtable-hbase-1.0/pom.xml has an artifactId of  bigtable-hbase-1.0.  What am I missing?

FYI, here's the results...
$ find . -name '*.properties' | xargs grep bigtable  | grep 0.1.6
./bigtable-hbase-1.0/target/classes/com/google/cloud/bigtable/hbase/bigtable-hbase.properties:bigtable.hbase.user_agent=bigtable-hbase-1.0-0.1.6-SNAPSHOT
./bigtable-hbase-1.1/target/classes/com/google/cloud/bigtable/hbase/bigtable-hbase.properties:bigtable.hbase.user_agent=bigtable-hbase-1.1-0.1.6-SNAPSHOT

Updated the method name.  Good catch.

Sorry! I was looking at parent artifactId :P
This means we have following two different artifact IDs for bigtable-hbase:
bigtable-hbase-1.0 and bigtable-hbase-1.1
and with the same version: 0.1.6-SNAPSHOT
I don't like what we have. I thought the artifact ID should stay the same and the versions are different. Two different artifact IDs with the same version is different from same artifact with two different versions. If this is what you have decided to use, I am fine with it.

Other than the different artifact IDs I don't like, I am fine with just PR.  I think it is doable to have different versions of bigtable-hbase in a same github project but not under the same parent project.
Please ask Angus to take a second look.
LGTM

Merging this.  The questions @kevinsi4508 has are about the project structure rather than the user agent header.   If we change the project structure, the proposed code in this PR ought not change.


[DashboardEditor] card jumps when clicked

What package is this for?

[x] React
[ ] Angular

Describe the bug
When a card is clicked to be selected, it is also triggering the drag start effect created through react-grid-layout.

To Reproduce
Steps to reproduce the behavior:

Go to any DashboardEditor story
Add a card
Click the card multiple times
See error

Expected behavior
The card should only move if the user's mouse begins to actually drag / move
Environment/versions:

OS: MacOS
Browser: chrome
carbon-addons-iot-react version: 2.140.0-next.7

Specific timeline issues / requests
Do you want this work within a specific time period? Is it related to an
upcoming release?
No specific timeline

it is also triggering the drag start effect created through react-grid-layout

That might be the cause of the movement too - I think rgl places a class on the element on drag, which could apply styles that cause the movement.
This all might be fixed by stopping the event propogation of a single non-drag click so it doesn't reach rgl.

this doesn't seem to be happening anymore


Update os-lib to 0.8.0

Updates com.lihaoyi:os-lib from 0.7.8 to 0.8.0.
GitHub Release Notes - Version Diff
I'll automatically update this PR to resolve conflicts as long as you don't change it yourself.
If you'd like to skip this version, you can just close this PR. If you have any feedback, just mention me in the comments below.
Configure Scala Steward for your repository with a .scala-steward.conf file.
Have a fantastic day writing Scala!

Ignore future updates
Add this to your .scala-steward.conf file to ignore future updates of this dependency:
updates.ignore = [ { groupId = "com.lihaoyi", artifactId = "os-lib" } ]

labels: library-update, early-semver-major, semver-spec-minor, commit-count:1

Codecov Report

Merging #240 (06c3c41) into main (2d01d66) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #240   +/-   ##
=======================================
  Coverage   88.84%   88.84%
=======================================
  Files          91       91
  Lines        2770     2770
  Branches       67       67
=======================================
  Hits         2461     2461
  Misses        309      309

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 2d01d66...06c3c41. Read the comment docs.


question: how to display pool external incentives

Pool 629 now has external incentives, which added recently. However, I'm unable to locate instructions on displaying them in the Osmosis frontend.

Not yet. Start time is 2023-04-08 05:56:24 . This is transaction hash https://www.mintscan.io/osmosis/txs/7FBB410870D694E9C5376E72A5FB9F7D03B976E9F316AC94E0C40C0163A26B09

Once it's started they will be displayed.


Create npm-publish-github-packages.yml

I don't see package.json anywhere in this repo, so I am not convinced that there is a package here.  What do you hope to publish?


Final project - Jiasheng Yu

Final project for MAE6286.

Jiasheng you have 3 different notebooks. Which ones you want us to read, 1D and 2D?  Why not put all together?  Is there a reason to have them separated?

Now I update my project folder with only one python file. I combined two separated files. Is it OK now? Thank you for helping with git command.


feature request: automatically PR meeting minutes once meeting has concluded

At present there's several manual steps of automatically PR'ing in the meeting minutes. Need to copy from Google Docs, create a new file in the repo with the correct title structure (currently, most repos use the date of the meeting as the filename) in the correct directory, and submit a PR. This is a process that unnecessarily uses human time that could be used elsewhere for more productive things.
It would be excellent if meet could automatically PR any meeting minutes for meetings that have already completed but have not yet been PR'ed.

Maybe this could be a separate Action (separation of concerns)?

It definitely could be! It would potentially be nice to have an all-in-one that reduces the overhead of configuration and setup for folks trying to ramp up to the tooling and just get going, but I could conceive of reasons to have it be separate.

I guess I'd be fine with it being implemented either way :)

I would also be fine with it being implemented either way - if someone's willing to do the work, whatever they're willing/able to do is a +1 from me :)

I'm sorta looking into it, although I'm writing an example using HackMD instead of Google Drive (I suggested we try out HackMD on nodejs/diagnostics). One thing we would need to decide is how to fetch the meeting issue and how to find the meeting link in the issue. We could either use a heuristic approach (search issues by title name and chose the most recently opened meeting + search the issue body for "Minutes: .*"), or we could have explicit identifiers to make things easier (have a label current-meeting or next-meeting, and in the issue we could add the link to the minutes as metadata as a comment <!-- minutes: .* -->). The upside of the latter approach is that if title or the issue text is manually changed, automation will still works).
Another thing we can do as a part of this feature is to add the recording link to the minutes (which has similar challenges as suggested above, plus finding the recording which I think is harder than finding an issue).

I'm in favor of the label approach since it also provides a good way for humans to identify which meeting issue is the one they should be looking at (some repos are slow to close old meeting issues and the title system is not currently very helpful).
@mmarchini could you explain the metadata comment a bit more to me? I don't totally understand.

I think placeholder is a better word than metadata for what I meant. Basically, instead of having:
## Links

* **Recording**:
* **GitHub Issue**: https://github.com/nodejs/diagnostics/issues/397
* **Minutes Google Doc**: https://docs.google.com/document/d/14W5g4rHw38XzX1wGYgCOSXY-v7AW0_jw_b2IcctnGKM/edit

We would have
## Links

* **Recording**: <!-- recording-placeholder -->
* **GitHub Issue**: https://github.com/nodejs/diagnostics/issues/397
* **Minutes Google Doc**: https://docs.google.com/document/d/14W5g4rHw38XzX1wGYgCOSXY-v7AW0_jw_b2IcctnGKM/edit
<!-- minutes:https://docs.google.com/document/d/14W5g4rHw38XzX1wGYgCOSXY-v7AW0_jw_b2IcctnGKM/edit -->

Since <!-- --> are comments, they are not rendered, but are easier to be found by scripts.

Another feature that may be potentially useful and completely solve this: https://hackmd.io/c/tutorials/%2Fs%2Flink-with-github

Right, as long as we don't require PRs for minutes it should work. Reading the comments it I wonder how well it would work with several folks making changes to the minutes and trying to push (because users need to push and pull manually). Also, seems like this feature requires users to manually link the minute every time (unless there's an API for that, which I don't think there is yet).
Worth keeping an eye on this feature, they might add improvements to it in the future :)

cc @joesepi this is the issue for the feature you want, by the way. Would genuinely love to see a PR around this - shouldn't be massively complex ❤


Run tests on Node.js 8+12 instead of 8+10

The deployment is still using 8, but 12 is now LTS and the version
we'd likely upgrade to, making it the more useful version to test.

The same number of tests run (and pass) in 8 and 12:
https://github.com/web-platform-tests/wpt-pr-bot/pull/119/checks?check_run_id=304440466#step:5:546
https://github.com/web-platform-tests/wpt-pr-bot/pull/119/checks?check_run_id=304440481#step:5:547
We could have even higher confidence in this by merging https://github.com/web-platform-tests/wpt-pr-bot/pull/118 first.

Thanks for rebasing @zcorpan! Looks like this works from the logs of the checks.

Yep, thanks for checking! This is now deployed on heroku.