Check for new subtitles, don't overwrite when downloaded

Checklist

[X] I'm asking a question and not reporting a bug/feature request
[X] I've looked through the README
[X] I've read the guidelines for opening an issue
[X] I've searched the bugtracker for similar questions including closed ones

Question
I have a question about checking for new subtitles - apologies is this is already stated in the README, but I couldn't find it.
I would like to download a video as soon as possible, whether or not subtitles have been added yet. When subtitles are added, I would like to downloading them as well, but have the same output name. The output name has %(upload_date)s, which I would like to be the same for the video and subtitles, even if they aren't uploaded on the same date. Is there any way to accomplish this?
Caveat: Once the video/subtitles are downloaded, I don't want them to be changed (even it they're updated from the uploader)
Here's the my current config.txt:
-i ...
-f 'bestvideo+bestaudio'
--merge-output-format mp4
--output "%(upload_date)s - %(uploader)s - %(title)s"

--add-metadata
--embed-thumbnail

--write-sub
--sub-lang en,en-us,en_us,en-US,en_US
--convert-subs vtt

Thank you for your time.

The output name has %(upload_date)s, which I would like to be the same for the video and subtitles, even if they aren't uploaded on the same date. Is there any way to accomplish this?

This is already the case. upload_date always points to the video's upload date, not the subtitle's

Caveat: Once the video/subtitles are downloaded, I don't want them to be changed (even it they're updated from the uploader)

--no-overwrite

Please note, that the already downloaded video file will still be changed despite of --no-overwrite when --embed-* is used.

What's the difference between --no-overwrite and --download-archive? Does --no-overwrite apply to all files, and --download-archive to only videos?

They are totally different things
    -w, --no-overwrites              Do not overwrite any files
    --force-overwrites               Overwrite all video and metadata files.
                                     This option includes --no-continue
    --no-force-overwrites            Do not overwrite the video, but overwrite
                                     related files (default)

    --download-archive FILE          Download only videos not listed in the
                                     archive file. Record the IDs of all
                                     downloaded videos in it

Download archive doesn't process the same videos again AT ALL. Doesn't matter if the file still exists or not. the overwrite options only looks at whether files with the same name already exists

@csadam Will this be changed anytime soon? If not, is there an alternative I can look into (like a python script to embed images into videos with the same name)?

It seems that when using this config metadata is still being added when its been added before:
--merge-output mp4

--add-metadata
--write-thumbnail

--write-sub
--sub-lang en,en-us,en_us,en-US,en_US
--convert-subs vtt

--no-overwrite

Yes, see #1467

So I would have to use --download-archive + --embed-thumbnail in a single script, then use another script to search for subs? Something like this I think? :
--skip-download
--write-sub
--sub-lang en,en-us,en_us,en-US,en_US
--convert-subs vtt

until I figure out a solution for #1467, yes.
If you are dealing with large channels, you will get better performance if you write the channel infojson first and load it back for subsequent runs. But this is something that would take a bit of scripting to achieve

I see. Thank you for letting me know. Will keep track of that issue and hoping it'll be fixed.

@pukkandan An issue I found with using 2 separate script for this case is that:

The script using --download-archive records the video title
Then running the script for downloading subtitles with --skip-download will download the subs, but if the uploader changes the video title, the video & subs downloaded will have different titles

Just something I thought you should be aware of. Hopefully it helps you out a bit.

but if the uploader changes the video title, the video & subs downloaded will have different titles

There is nothing we can do about that

@pukkandan As a follow up, is it possible to use something like --download-archive with --skip-download and --write-sub so that subs are only downloaded once? I tried it and it doesn't seem to work.

--force-write-archive

It seems that this is writing the video ID even if subs weren't downloaded (aka as long as it checks the url). I want to only archive the video IDs where subs have been downloaded.

I want to only archive the video IDs where subs have been downloaded. For the videos where subs haven't been downloaded, I want to be able to check again when running the script again.

That's not possible


BUG: Specify pyolog version

Seems like there are issues with 4.2.0 so we will fix 4.1.0 for now.

Closing as we'll just change the version of the pyolog package in the '-tag' channel.


[WIP] Array object

Deferred array object.

What should be done with this PR, now that the array expressions rewrite pass satisfies a large subset of the optimizations which this PR hoped to perform?

I think we should retire the PR, since substantial changes would be needed to make it work, and I think I see more of that deferred execution work in the Dask/Blaze space now (see "dask.array", for example).  We should keep the branch around for historical reference, though.


⚠ Bofrost.be has degraded performance

In 2921058, Bofrost.be (https://www.bofrost.be) experienced degraded performance:

HTTP code: 200
Response time: 843 ms

Resolved: Bofrost.be performance has improved in af57c2a after 11 minutes.


ADC fluctuation

Hi, for a project I use now the built in ESP ADC. But I'm disappointed about its stability. Even with 1000 multisamples and dismissing 80% of extreme values, it is unstable as hell!
=> Seems that the reference voltage is not stable at all and flickers heavily depending on CPU load. A 100nF capacity from ADC Pin to Ground did not solve problem, so it must be therefore the reference voltage.
Can you give me hints where I can solder add'l 10uF and 100nF to stabilize ref voltage? Any schematics to offer??
Thanks a lot!

What value of resistors are you using for the voltage divider?
As I'm sure you've heard...the ESP32 ADC is atrocious.
If ADC is critical to your design might I suggest a discrete ADC?
Like the Adafruit ADS1115 16-Bit ADC that communicates over i2c.

Dont bother with the ESP built in ADC. Its one of the worst out of all MCU's (NRF,STM,Atmegas).
You should not expect better accuracy than +-50mV.
If you need high accuracy, use external ADC chip

I tried working with the reference voltage on the LILYGO T-Display-S3 LCD along with the LM4040.  I tried using a diode in both directions.  I came to the conclusion that the external reference pin was always connected somehow, and I could measure current flow going in both directions.  I gave up using an external reference, and instead used a hard coded value (const float  referenceVoltageConst = 3.2804;).  Using my own voltage divider (20K & 5K resistors), I could measure somewhat decent voltage values of hybrid battery modules (up to 9.0 volts, etc.).  Also, I don't think the LM4040 was providing a voltage of 4.096 like it should have either.

What pin is the reference voltage?
I am interested in what you are trying to do!

Oops... I was referring to the AREF pin on the Arduino boards.  The LILYGO T-Display-S3 does not have an AREF pin.  I worked on this 14 months ago and forgot that I switched to using the LILYGO T-Display-3 LCD.  Back then, I was using the Arduino NANO, which today I confirmed it only supports an internal reference voltage, which explains what I experienced with the NANO board.  https://support.arduino.cc/hc/en-us/articles/360018922239-About-the-AREF-pin  Other Arduino boards do support an external AREF.  Still, on the T-Display-3 board, I find it beneficial to hard code the reference voltage to something like 3.2804.  My project so far tracks the voltage and amps of a single Panasonic Hybrid car module with a potential voltage of 9.6 volts or so.  I use a voltage divider of 20K & 4K resistors which lets me track the voltage up to 13.2 volts (3.3 volts * 4).  I use an ACS712 5A Hall Effect Current Sensor Modules to track up to 5 amps.  I supply the ACS712 with 5v and measure the 3.3v analog value (using the hard coded 3.2804v) and 12 bits of ADC accuracy (4,096 values). The ACS712 module provides a voltage which you then convert to amps using a formula.  I find that I need to keep this sensor away from other wires to get consistent values.  I use it to captures amps, but use my voltage divider (20K & 5K) to capture voltage.  I am thinking of using your suggestion to use the ADS1115 16-Bit ADC to capture more accurate ADC values.  Here is my code for capturing ACS712 amp values. I floats and doubles in the calculations, which returns more refined results.
const float  AnalogToDigitalResolution = 4096.0; // 10-Bits = 1024,  12-Bits = 4096
const float  referenceVoltageConst = 3.2804; // 3.2804 3.28 3.38555 3.3888
const float  ACS712_scale_factor = 0.181; // ACS712 5A current sensor 0.185
double ACS712_zeroPoint = referenceVoltageConst / 2.0;  // The sensor is being fed 5 Volts, the ADC is 3.28 volts
double resADC = referenceVoltageConst / AnalogToDigitalResolution;
double readAmperage(int analogPin)
{
int sampleSize = 100;
long analogValue = 0;
double avgAnalogValue = 0.0;
double ACS712_Voltage = 0.0;
double ACS712_Current = 0.0;
float voltageDifference = 0.0;
for (int i=0; i < sampleSize; i++)
{
analogValue += analogRead(analogPin);
delay(1);
}
avgAnalogValue = analogValue / float(sampleSize);  // calculate avg analog value
ACS712_Voltage = avgAnalogValue * resADC;  // convert analog value to voltage
voltageDifference = ACS712_Voltage - ACS712_zeroPoint;
// report both negative amps and positive amps as positive values
if (voltageDifference < 0.0)
voltageDifference = voltageDifference * -1.0;  // convert negative amp flow to positive amp flow
// Convert ACS712_Voltage into Current using Scale Factor
ACS712_Current = voltageDifference / ACS712_scale_factor * 2.0;
ACS712_Current -= 0.25;  // fudge to zero when no current
if (ACS712_Current < 0.0)
{
ACS712_Current = 0.0;
}
return (ACS712_Current);
}

Thanks for getting back. Cool project, the ADS1115 is well respected.
-Terry


Implement employers model

Yes,we are 💯
On Sat, Oct 7, 2023, 23:02 Jojo Thomas @.***> wrote:

@.**** approved this pull request.
Good work, we're making progress
—
Reply to this email directly, view it on GitHub
https://github.com/Ayobami6/MeetDevs/pull/14#pullrequestreview-1663112579,
or unsubscribe
https://github.com/notifications/unsubscribe-auth/AV25KBPW6N3M7EIKZGPAOBDX6HGQFAVCNFSM6AAAAAA5XBZXU6VHI2DSMVQWIX3LMV43YUDVNRWFEZLROVSXG5CSMV3GSZLXHMYTMNRTGEYTENJXHE
.
You are receiving this because you authored the thread.Message ID:
@.***>


MyLocationNewOverlay doesn't stop following user location immediately

Issue Type
[ ] Question
[x] Bug
[ ] Improvement
[ ] Build system related
[ ] Performance
[ ] Documentation
Description and/or steps/code to reproduce the problem
When MyLocationNewOverlay is set to follow the users location with enableFollowLocation(), it doesn't stop doing so immediately after calling disableFollowLocation(). If I try to move the map after calling disableFollowLocation(), the map stays centered on the users location. When debugging, MyLocationNewOverlay.setLocation() is called multiple times after disabling the user location following.
A small snippet demonstrating the problem:
// Disable following user location to enable the map to move
if(mLocationOverlay != null){
    if(mLocationOverlay.isFollowLocationEnabled()){
        mLocationOverlay.disableFollowLocation();
    }
}

// Try to move the map to location.
IMapController controller = mMap.getController();
controller.animateTo(location); // Doesn't actually move the map

Putting a ~second delay to moving the map works however:
// Disable following user location to enable the map to move
if(mLocationOverlay != null){
    if(mLocationOverlay.isFollowLocationEnabled()){
        mLocationOverlay.disableFollowLocation();
    }
}

// Move the map to location after a second.
// This works as MyLocationNewOverlay stops updating user location after a delay.
new Handler().postDelayed(new Runnable() {
    @Override
    public void run() {
        IMapController controller = mMap.getController();
        controller.animateTo(location);
    }
}, 1000);

If it's a bug, version(s) of android this affects:
Tested on a virtual device running Android 8.0.0
Version of osmdroid the issue relates to:
6.0.2

Hmm, I have not observed this issue so far. AnimateTo always did an animateTo.

It seems I was too quick to report this, looking further into the problem it seemed to be my code that was causing the behaviour. My function that was disabling the user location following is called immediately after returning from a activity to another. The MapView resides inside a fragment in the activity that was returned to. The fragments onResume(), which calls the MapView onResume(), was called after calling the disabling function, causing the MyLocationNewOverlay onResume() to re-enable the user location following. I'm not sure if this is desired functionality but I'm not sure if it's a bug either.

I'm experiencing the same problems. I'm in the same situation where I am calling disableFollowLocation() and then animateTo() within onActivityResult(). map.onResume() is called within my map activity's onResume() override which is called after onActivityResult(). When map.onResume() is called it's implicitly re-enabling follow location after I explicitly disabled it. I tracked down this behavior to HERE. Not sure how necessary this check is, is there an instance where the state may be lost making this necessary?
I found a workaround but it's far from ideal and introduces usability issues. I use a "center" button to allow the user to re-enable follow location if they decide to scroll around the map. I call disableFollowLocation() and enable the "center" button before calling startActivityForResult(). The downside of this is no matter the result from the activity I cannot re-enable follow location automatically and must put that onto the user to push the "center" button because onActivityResult() is called before onResume().
If anyone has a better idea of how to handle this I would greatly appreciate it. Otherwise hopefully this helps someone with the same problem.


bump sigstore/sigstore from 1.3.1 to 1.4.0

Signed-off-by: Kenny Leung<EMAIL_ADDRESS>
Summary

bump sigstore/sigstore from 1.3.1 to 1.4.0
Release Note

NONE
Documentation

To fix the test, update https://github.com/sigstore/fulcio/blob/main/pkg/server/grpc_server_test.go#L959 and https://github.com/sigstore/fulcio/blob/main/pkg/server/grpc_server_test.go#L1366 to be PublicKeyPEMType.

Codecov Report

Merging #745 (0ccad6c) into main (ab74398) will decrease coverage by 0.13%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #745      +/-   ##
==========================================
- Coverage   54.18%   54.05%   -0.14%
==========================================
  Files          36       36
  Lines        2257     2257
==========================================
- Hits         1223     1220       -3
- Misses        942      944       +2
- Partials       92       93       +1

Impacted Files
Coverage Δ

pkg/ca/fileca/load.go
58.62% <0.00%> (-10.35%)
:arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

To fix the test, update https://github.com/sigstore/fulcio/blob/main/pkg/server/grpc_server_test.go#L959 and https://github.com/sigstore/fulcio/blob/main/pkg/server/grpc_server_test.go#L1366 to be PublicKeyPEMType.

thanks for pointing me in the right direction. fixed.


fix(common): compact logging of unrecognized fields

I hereby agree to the terms of the RisingWave Labs, Inc. Contributor License Agreement.
What's changed and what's your intention?
We always print the config on startup. After #9156, the derived implementation of Unrecognized seems too verbose. Let's only print the inner map instead.
Checklist For Contributors

[x] I have written necessary rustdoc comments
[x] I have added necessary unit tests and integration tests
[x] I have added fuzzing tests or opened an issue to track them. (Optional, recommended for new SQL features #7934).
[x] I have demonstrated that backward compatibility is not broken by breaking changes and created issues to track deprecated features to be removed in the future. (Please refer to the issue)
[x] All checks passed in ./risedev check (or alias, ./risedev c)

Checklist For Reviewers

[ ] I have requested macro/micro-benchmarks as this PR can affect performance substantially, and the results are shown.

Documentation

[x] My PR DOES NOT contain user-facing changes.

Click here for Documentation
Types of user-facing changes
Please keep the types that apply to your changes, and remove the others.

Installation and deployment
Connector (sources & sinks)
SQL commands, functions, and operators
RisingWave cluster configuration changes
Other (please specify in the release note below)

Release note


Typo - HttpClient Guide (json results section)

https://angular.io/docs/ts/latest/guide/server-communication.html
Can put a PR in tomorrow just putting the issue out there.

JSON results
As with getHeroes(), we extract the data from the response using the extractData() helper.
Back in the HeroListComponent, we see that its addHero() method subscribes to the observable returned by the service's addHero() method. When the data, arrive it pushes the new hero object into its heroes array for presentation to the user.                ^^^^^^^^^^^^^^^^^

Yes, sounds weird. Please open a PR.


Godot 3.1 Beta 4-6 Parse Error caught only at release build

Godot version:
3.1-Beta-4 and Beta-6 (These are the only I tested)
OS/device including version:
Windows 10
Issue description:
Parse Error caught only at release build, caught by Log
**SCRIPT ERROR**: Parse Error: Cannot assign a new value to a constant.
Steps to reproduce:
This particular case occured by doing something like this:
for numerical_string in ["0","1","2"]:
 numerical_string=int(numerical_string)

Duplicate of #25334.


[BEAM-2855] implement query 10

implemented query 10, now query 10 still does not do anything because it seems like Beam team hasn't made up mind about what to do with this query. (here is the comment in java query 10)

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

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

See the Contributor Guide for more tips on how to make review process smoother.

GitHub Actions Tests Status (on master branch)

See CI.md for more information about GitHub Actions CI.

R: @y1chi
R: @pabloem

Run PythonLint PreCommit

@pabloem Pablo could you help merge this?


Make is easy to configure Selenide Proxy before it's started

The problem
Hi, I am facing one issue with Selenide proxy, I am trying to disable MITM.
Proxy is not configurable, as it is created and started on time to open a browser using open(). MITM may be only disabled before Proxy is started.
Details
I have Configuration.proxyEnabled = true;
Here's my code after opening a browser with open():
proxy = WebDriverRunner.getSelenideProxy().getProxy();
proxy.setMitmDisabled(true);

It returns a IllegalStateException with message "Cannot disable MITM after the proxy has been started".
Expected

Would be nice to have Proxy easily configurable

Environment

Selenide Version: 5.24.2

@IlyaHarlamov Do you use WebDriverFactory or WebDriverProvider in your project?

@IlyaHarlamov Do you use WebDriverFactory or WebDriverProvider in your project?

As discussed in private, none of these

Let's discuss what would be the best API to configure the proxy.
I can suggest few options.
1. New configuration property
Configuration.proxyCustomizerClass = "com.my.FooProxyCustomizer";

2. New static configuration
Similar to Configuration:
  ProxyConfiguration.mitmEnabled = false;
  ProxyConfiguration.host = “localhost”;
  ProxyConfiguration.port = 9999;

3. WebDriverFactory
Add method like createProxy to method WebDriverFactory and/or WebDriverProvider.
Default implementation will just return new BrowserUpProxyServer().

Pros: it's powerful, no static configs
Cons: not everybody uses WebDriverFactory or WebDriverProvider. Many users just call open(url).

4. Lambda
When opening a new webdriver, pass a lambda to customize the proxy:
open(“https://google.com”, (proxy) -> {
  proxy.setMitmEnabled(false);
  proxy.setTrustAllServers(true);
});

Pros: it's powerful, no static configs
Cons: it's not clear if we should shutdown the previous proxy in open method (was it started with the same or different lambda?)

@BorisOsipov @rosolko @dbudim @dstekanov

@asolntsev

Plugin via service loader.

I prefer 1 or 5

maybe...

@asolntsev, it is not only about MITM, there might be more configurations which are not possible to change right now, I know at least one more, the trust servers

@IlyaHarlamov Selenide already does proxy.setTrustAllServers(true); by default.


target cannot reset and run after drag-n-drop program (EUB-34)

Hi, just notice that after drag-n-drop program, the target chip cannot reset and run automatically. can we optimize it? thanks.
connect wires are [GND TX RX BOOT RST 3V3], the target chip tested is ESP32-C3

Hi @wuxx. I'm sorry for the late reply. Do you mean that this is just an ESP32-C3 issue? Have you tried another chip or capture the signals on BOOT and RST with a logic analyzer?
The target chip should reset as per https://github.com/espressif/esp-usb-bridge/blob/master/main/msc.c#L424 and https://github.com/espressif/esp-serial-flasher/blob/0b5ca69c99f6607512dc2914351e89bc968dd723/include/esp_loader.h#L156. If it doesn't then it can be an issue with esp-serial-flasher.
It could be also a HW issue. Do you have a pull-up resistor and capacitor on the boot signal for the target MCU? See M_EN at
https://github.com/espressif/esp-usb-bridge/blob/master/images/schematics.pdf for an example.

hello, @dobairoland got it, I will do more test and check the source code


Improvement: Add toStartOfInterval() function

Hello!
ClickHouse has a good kind of functions like toStartOf*(time) which round down a time to the specified part (minute, hour, day and etc.). It's quite enough but it would be good if there were an universal function which could accept an interval (in seconds, for example).
toStartOfInterval(time, interval [, units])

where time - a DateTime column, interval - an integer time interval for rounding, units - an optional time unit (by default seconds, possible values: seconds, minutes, hours, days and etc.).
For example, the function toStartOfFiveMinute(time) could be represented as toStartOfInterval(time, 5, 'minutes'). And consequently we could group our data by user defined time interval.
I think it will be powerful and useful feature to analyze time-series data.

Hello again! :)
I would like to add one more example which illustrates what I want to see in the nearest future (I hope). TimeScale database offers time_bucket() function which works as described above. Using this function you can aggregate data for example for every 5 minutes:
SELECT
    time_bucket('5 minutes', time) AS period,
    avg(value) AS avgValue
FROM history
GROUP BY period

Finally, I expect behaviour like in the example below.
Table
create table history
(
    sensorId String,
    date Date default toDate(time),
    time DateTime,
    value Float64
)
ENGINE = MergeTree(date, (sensorId, date, time), 8192)

Request
SELECT
    toStartOfInterval(time, 5, 'minutes') AS period,
    avg(value) AS avgValue
FROM history
GROUP BY period
ORDER BY period DESC
LIMIT 10

Result
         period      | avgValue
---------------------+----------
 2018-05-01 12:00:00 |   72202
 2018-05-01 12:05:00 |  837725
 2018-05-01 12:10:00 |  412237
 2018-05-01 12:15:00 | 1173393
 2018-05-01 12:20:00 |   90104
 2018-05-01 12:25:00 |  784596
 2018-05-01 12:30:00 |  574134
 2018-05-01 12:35:00 |  960404
 2018-05-01 12:40:00 |  963104
 2018-05-01 12:45:00 |  160104

P.s. Now I need to do an aggregation for every 1, 5, 10, and etc. minutes and think about some workaround to do that. Maybe you have some ideas how to do it now? Also I'm interesting is it real to implement described functionality? If 'yes' then when we can expect this?

It's very easy to workaround.
Use that as a template
toDateTime(intDiv(toUnixTimestamp(timestamp), 300) * 300)

replace 300 with your time interval (in seconds).
But that question is quite popular, may be the shortcut function should be added.

Maybe the function like roundTime(  time, '5m' ) , I think it's much better than  toStartOfInterval(time, 5, 'minutes')

@sundy-li I don't mind. I just used the existing 'naming convention'.

Implemented syntax toStartOfInterval(time_or_data, INTERVAL x unit [, time_zone]) (see https://github.com/yandex/ClickHouse/pull/4304#issue-251152989 for examples).


NGA NSG PROFILE

Has geopackage-js and leaflet plugin been updated to support NGA NSG TILING SCHEMA/profile?

It would be great to have support the NGA's NSG GeoPackage Tiling Profile Specification?  (NSG) GeoPackage Encoding Standard 2.0
Interoperability Standard
(2016-09-14)

Please see ngageoint/GeoPackage#4


chore: github workflow rebase staging(dev) with main on release

What would you like to be added?
When merging from staging to main, there is a extra commit in main that staging branch don't have. we have agreed that we should have a automated workflow to sync staging to the main branch.
Anything else you would like to add?
No response
Are you willing to submit PRs to contribute to this feature?

[ ] Yes, I am willing to implement it.

discussed offline, it is ok for main to contain extra merge commits as long as content is the same as dev branch. Will reopen this issue if there is a need to sync dev to main


delete hook integration tests

resolves #

Description
Happy to be wrong here but it doesn't look like we need to test this here.

Checklist

[ ] I have read the contributing guide and understand what's expected of me
[ ] I have signed the CLA
[ ] I have run this code in development and it appears to resolve the stated issue
[ ] This PR includes tests, or tests are not required/relevant for this PR
[ ] I have opened an issue to add/update docs, or docs changes are not required/relevant for this PR
[ ] I have run changie new to create a changelog entry

Jerco, thanks for trawling through the history here. I did a quick scan of the repo and this seems to be. If we want to adopt a testing philosophy where we create adapter-repo specific tests to create coverage for macro overrides (seems reasonable), I think we should not remove this test, but perhaps have it pull from an adapter zone variant or have a trivial test in this dbt-bigquery repo with a comment explaining its presence.
I've advised a similar policy on other adapter tests (or been encouraged to do the same).


improve sourceOverrides

currently sourceOverrides look like this:
sourceOverrides = oldSources: {
  bar."13.2.0" = builtins.fetchTarball {url = ""; sha256 = "";};
  baz."1.0.0" = builtins.fetchTarball {url = ""; sha256 = "";};
};

The problem is, that the version needs to be specified. This makes the override harder to write and likely to brake once the dependencies change in the future.

Perhaps something more similar to overlays could be used (seems to work well in poetry2nix. If there was an attribute set without the version and perhaps a list of conditions when the override should be applied that would work. Or have a function that will receive the requested version. I wonder if sometimes the condition will be connected to other things than the version itself, like language runtime version, system etc. could potentially be good to support that as well:
sourceOverrides = oldSources: {
  bar = {version, ...}:
    if lib.versionAtLeast version "13.2.0"
    then builtins.fetchTarball {url = ""; sha256 = "";};
  }
    else oldSources.bar;

  baz = {version, buildSystemVersion, ...}:
    if buildSystemVersion == 3 && lib.versionOlder version "1.0.0"
    then builtins.fetchTarball {url = ""; sha256 = "";};
    else oldSources.baz;
};

sourceOverrides = oldSources: {
  bar = [{
    condition = version: lib.versionAtLeast version "13.2.0";
    src = builtins.fetchTarball {url = ""; sha256 = "";};
  }];
  baz = [{
    condition = version: lib.versionOlder version "1.0.0";
    src = builtins.fetchTarball {url = ""; sha256 = "";};
  }];
};


Hello JK. Another question for you so I might replace ryu with your algorithm since your table looks smaller?

Do you have an efficient string to floating-point algorithm based on Grisu-Exact? It looks like the process could be reversed as long as the precision is not beyond the range of floating-point.

Short answer: No.
Long answer:
I think it will be possible to invert what Grisu-Exact is doing by again using the Grisu-like idea with the Ryu's magic for mathematically proving sufficiency of certain precision. I was actually thinking of it this January, but never really worked out.
As far as I know, Ulf already did something similar and implemented a string-to-float routine, and actually what he was considering is something even more general.
I think when I was thinking of the decimal-to-binary conversion (which is the core of string-to-float) I had been thinking that it should be simpler than binary-to-decimal for some reason, but now I can't recall the exact reason.
My current guess is that if I really work out, I might be able to develop a new algorithm for decimal-to-binary conversion, but it will take a considerable amount of effort (if not impossible) to beat what' currently offered by Ryu, in terms of performance.
Right now, I have no plan for actually doing it. But if you are really interested in it, maybe you can read the Grisu-Exact paper and discuss it with me. By doing so, you might get the idea of how to do the reverse. As far as I remember, the core idea was not that difficult to come up with once you understand Grisu-Exact.


Add MQTT

https://github.com/hobbyquaker/awesome-mqtt

Looks good! :)

Would be awesome if you could add the awesome badge to your readme :D


UAT| etreasury_payment issue.

https://jam.dev/c/690db11d-2362-40ba-84b9-18087ce18fd1

response:
{"timestamp":"2024-10-04T06:52:51.119+00:00","status":500,"error":"Internal Server Error","path":"/etreasury/payment/v1/_processChallan"}
curl:
curl 'https://dristi-kerala-uat.pucar.org/etreasury/payment/v1/processChallan?=1728024770693'
-H 'accept: application/json, text/plain, /'
-H 'accept-language: en-GB,en;q=0.9'
-H 'content-type: application/json;charset=UTF-8'
-H 'origin: https://dristi-kerala-uat.pucar.org'
-H 'priority: u=1, i'
-H 'referer: https://dristi-kerala-uat.pucar.org/digit-ui/citizen/dristi/home/file-case/e-filing-payment?caseId=e7314fbf-6054-4fc0-97dc-feced45048f2'
-H 'sec-ch-ua: "Chromium";v="129", "Not=A?Brand";v="8"'
-H 'sec-ch-ua-mobile: ?0'
-H 'sec-ch-ua-platform: "Linux"'
-H 'sec-fetch-dest: empty'
-H 'sec-fetch-mode: cors'
-H 'sec-fetch-site: same-origin'
-H 'user-agent: Mozilla/5.0 (X11; Linux x86_64) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/<IP_ADDRESS> Safari/537.36'
--data-raw '{"ChallanData":{"ChallanDetails":{"FROM_DATE":"26/02/2020","TO_DATE":"26/02/2020","PAYMENT_MODE":"E","NO_OF_HEADS":"1","HEADS_DET":[{"AMOUNT":"4","HEADID":"00374"}],"CHALLAN_AMOUNT":"4","PARTY_NAME":"Kashish lit","DEPARTMENT_ID":"4ac7566f-5e63-4258-a9b2-0e25bd398a78","TSB_RECEIPTS":"N"},"billId":"504375d1-8f51-4f1e-b475-f8ffb24dcaeb","serviceNumber":"KL-000127-2024_CASE_FILING","businessService":"case-default","totalDue":"878788","mobileNumber":"9033333333","paidBy":"Kashish lit","tenantId":"kl"},"RequestInfo":{"apiId":"Rainmaker","authToken":"6955f912-0db4-49c9-a90d-c86d0f809253","userInfo":{"id":711,"uuid":"a9accc75-ae08-4753-956c-7597946c186b","userName":"9033333333","name":"Kashish lit","mobileNumber":"9033333333","emailId":null,"locale":null,"type":"CITIZEN","roles":[{"name":"HEARING_VIEWER","code":"HEARING_VIEWER","tenantId":"kl"},{"name":"CASE_VIEWER","code":"CASE_VIEWER","tenantId":"kl"},{"name":"DEPOSITION_VIEWER","code":"DEPOSITION_VIEWER","tenantId":"kl"},{"name":"Citizen","code":"CITIZEN","tenantId":"kl"},{"name":"APPLICATION_CREATOR","code":"APPLICATION_CREATOR","tenantId":"kl"},{"name":"SUBMISSION_DELETE","code":"SUBMISSION_DELETE","tenantId":"kl"},{"name":"HEARING_ACCEPTOR","code":"HEARING_ACCEPTOR","tenantId":"kl"},{"name":"CASE_RESPONDER","code":"CASE_RESPONDER","tenantId":"kl"},{"name":"DEPOSITION_CREATOR","code":"DEPOSITION_CREATOR","tenantId":"kl"},{"name":"SUBMISSION_RESPONDER","code":"SUBMISSION_RESPONDER","tenantId":"kl"},{"name":"ORDER_VIEWER","code":"ORDER_VIEWER","tenantId":"kl"},{"name":"CASE_EDITOR","code":"CASE_EDITOR","tenantId":"kl"},{"name":"SUBMISSION_CREATOR","code":"SUBMISSION_CREATOR","tenantId":"kl"},{"name":"APPLICATION_VIEWER","code":"APPLICATION_VIEWER","tenantId":"kl"},{"name":"TASK_VIEWER","code":"TASK_VIEWER","tenantId":"kl"},{"name":"CASE_CREATOR","code":"CASE_CREATOR","tenantId":"kl"}],"active":true,"tenantId":"kl","permanentCity":null},"msgId":"1728024770693|en_IN","plainAccessRequest":{}}}'

The cheque amount is entering from UI is 12 characters but from backend it is allowing only 10 characters


Draw model for the windmill

As title said.
Art work kills those programmers who are not good at art

As the windmill is removed... this issue shall be closed.


[BUG] FCMCustomLuaWindow.lua lint error

There is a lint error being reported on line 129 of FCCustomLuaWindow.lua. It appears to be a typo in that that the variable f is not defined or referenced anywhere else, so the error message will either fail on the concat or always say "not found in 'nil'".
@ThistleSifter

Thanks, fixing it now.


Update README with Mapping module

When I created the PR for Mapping I forgot to update this part of the README

Nice


Users can get a maximum of 30 points per month #162

Description
points validation and tests
Fixes #162
Developer Checklist
You can mark items off by replacing [ ] with [x] or in the rendered
markdown, you can simply check the checkbox.

[x] I have read and followed the contribution guidelines.
[x] All specs and linters pass (including Code Climate)
[x] I have followed the style of nearby code
[x] The build on Semaphore is successful
[x] I have added tests for my changes.
[x] I have gone through my changes by hand, either in the browser or terminal,
which ever makes sense
[x] I have internationalized my
changes

Hi @IlinDmitry! Thanks for your work on this 🙂 In the future, if you could read the comments on the issue you're starting that would be much appreciated! Just want to make sure that we don't have two people working on the same thing.
@pierce-m is going to review this and work with you to make some changes before merging it 🙂

Hey Dmitry, thanks for your contribution. We've updated the description of issue #162 to be more specific about the user-experience we're hoping to achieve with this fix. Would you mind reading it over and making some changes to fit the new criteria? Thanks.

Hi all.
Is something wrong with this pull request? Should i fix something?

Hello @crawfoal.
I refactored my code.

Thanks for making those changes! I apologize again for switching things up on you multiple times 😕
The UI still needs a little work. With just the icon, it isn't clear that the "button" is disabled, so people might not think to hover over it. When you use a disabled button, typically it kind of grays it out to give a visual hint that it's disabled. Also, on mobile, they won't be able to hover so we need something more. Would you like to fix these things in this PR or create an issue so that these things can be addressed in a separate PR?

I'll fix it in this PR

Hello @crawfoal. I fixed the button

@IlinDmitry I'm psyched that you're interested in our project and I hope to work with you in the future on it. That said, I'm unfortunately going to have to close this PR because:
(a) I don't like the way that the user experience is evolving for this feature as a whole. That is partly due to the way the issue was updated and I'm sorry about that.
(b) There are still issues with the user experience regarding the button (e.g. they are probably wondering why they can't click it).
(c) There are still places in the code that would need to be refactored. I happy to work with you on this type of thing in a future PR, but because my first point (a) will require a complete re-write and I need to think through the details a little more I'm closing this.


Add lang/expr and lang/clss groups for tck18

This resolves backlog/issues/647
Signed-off-by: Mesbah-Alam<EMAIL_ADDRESS>

Test : Grinder/19533


Use guard let instead of nil checking

Please try with the even simpler code, and don't forget to run tests (utils/build-script -RT).

@lgerbarg thanks! But I got an error, I don't think it's because the patch
Exit Code: 1
Command Output (stderr):
--
/Users/arsen/code/apple/swift/test/Driver/Dependencies/bindings-build-record.swift:17:13: error: expected string not found in input
// NO-EXEC: inputs: ["./main.swift"], output: {{[{].*[}]}}, condition: check-dependencies
            ^
<stdin>:1:1: note: scanning from here
# "x86_64-apple-macosx10.9" - "swift", inputs: ["./main.swift"], output: {object: "./main.o", swift-dependencies: "./main.swiftdeps"}, condition: run-without-cascading
^
<stdin>:1:34: note: possible intended match here
# "x86_64-apple-macosx10.9" - "swift", inputs: ["./main.swift"], output: {object: "./main.o", swift-dependencies: "./main.swiftdeps"}, condition: run-without-cascading
                                 ^

--

********************
Testing Time: 768.08s
********************
Failing Tests (1):
    Swift :: Driver/Dependencies/bindings-build-record.swift

  Expected Passes    : 7247
  Expected Failures  : 8
  Unsupported Tests  : 33
  Unexpected Failures: 1
*** Failed while running tests for swift (check-swift-all-macosx-x86_64)
utils/build-script: command terminated with a non-zero exit status 1, aborting


fix image

This is for practice, there really is no image to fix

We hopped this would close automatically fixed fixed #1 c2f147addba1a6296f72c83db791aab8cca27c7e

We hoped this would fix it automatically fixed issue #1


Close connection to Redis after watcher execution

I ran into Redis' max client connection limit after multiple iterations of the watcher not closing its connections properly.

Fixed.

Thank you @Mitch528, will upload a new package to the NuGet later on!


Spelling error

[Enter feedback here]
the word corporate is misspelled below.
deviceOwnership | Equals, NotEquals | Supported values are "Personal" for bring your own devices and "Company" for corprate owned devices | (device.deviceOwnership -eq "Company")

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

ID: fcc79933-58e1-66d1-81ae-f43802d41223
Version Independent ID: 5be1fefc-356e-d1eb-ab97-8cbe5654ff43
Content: Filter for devices as a condition in Conditional Access policy - Azure Active Directory - Microsoft Entra
Content Source: articles/active-directory/conditional-access/concept-condition-filters-for-devices.md
Service: active-directory
Sub-service: conditional-access
GitHub Login: @MicrosoftGuyJFlo
Microsoft Alias: joflore

@TheDeuceYouSay
Thanks for your feedback! We will investigate and update as appropriate.

@TheDeuceYouSay
Thanks for pointing this out and contributing towards enhancement of the document.
Required changes have been performed, it will reflect in the document itself once approve by the author.


feature: add /create_channel endpoint for exposer

Closes: #396

Also closes: #399

Closes: #398

Also closes: #396


Example of K8s conf

Hi,
If you're interested, here is an example Kubernetes configuration for traefik-forward-auth using AUTH_HOST:
##
# Secrets to store Google's client secret and the app's secret
##
kind: Secret
apiVersion: v1
metadata:
  name: traefik-forward-auth-secrets
  namespace: kube-system
  labels:
    name: traefik
type: Opaque
data:
  CLIENT_SECRET: --> GOOGLE_CLIENT_SECRET_BASE64_ENCODED
  SECRET: --> A_RANDOM_SECRET_BASE64_ENCODED
---

##
# Main deployment
##
apiVersion: apps/v1
kind: Deployment
metadata:
  labels:
    app: traefik-forward-auth
  name: traefik-forward-auth
  namespace: kube-system
spec:
  replicas: 1
  selector:
    matchLabels:
      app: traefik-forward-auth
  template:
    metadata:
      labels:
        app: traefik-forward-auth
    spec:
      containers:
        - name: traefik-forward-auth
          image: thomseddon/traefik-forward-auth
          ports:
            - containerPort: 4181
              protocol: TCP
          env:
            - name: CLIENT_ID
              value: XXXXXX.apps.googleusercontent.com
            - name: CLIENT_SECRET
              valueFrom:
                secretKeyRef:
                  name: traefik-forward-auth-secrets
                  key: CLIENT_SECRET
            - name: SECRET
              valueFrom:
                secretKeyRef:
                  name: traefik-forward-auth-secrets
                  key: SECRET
            - name: COOKIE_SECURE
              value: 'true'
            - name: COOKIE_DOMAINS
              value: example.com,example.org
            - name: DOMAINS
              value: example.com,example.org
            - name: AUTH_HOST
              value: auth.example.com
          livenessProbe:
            tcpSocket:
              port: 4181
            initialDelaySeconds: 20
            failureThreshold: 3
            successThreshold: 1
            periodSeconds: 10
            timeoutSeconds: 2
---

##
# Related service
##
kind: Service
apiVersion: v1
metadata:
  name: traefik-forward-auth
  namespace: kube-system
spec:
  selector:
    app: traefik-forward-auth
  ports:
    - port: 80
      targetPort: 4181
      protocol: TCP
---

##
# Ingress for the auth host
##
kind: Ingress
apiVersion: extensions/v1beta1
metadata:
  name: traefik-forward-auth-ingress
  namespace: kube-system
  annotations:
    kubernetes.io/ingress.class: traefik
    ingress.kubernetes.io/auth-type: forward
    ingress.kubernetes.io/auth-url: http://traefik-forward-auth
    ingress.kubernetes.io/auth-response-headers: X-Forwarded-User
spec:
  rules:
    - host: auth.example.com
      http:
        paths:
          - backend:
              serviceName: traefik-forward-auth
              servicePort: 80

And without the AUTH_HOST:
##
# Secrets to store Google's client secret and the app's secret
##
kind: Secret
apiVersion: v1
metadata:
  name: traefik-forward-auth-secrets
  namespace: kube-system
  labels:
    name: traefik
type: Opaque
data:
  CLIENT_SECRET: --> GOOGLE_CLIENT_SECRET_BASE64_ENCODED
  SECRET: --> A_RANDOM_SECRET_BASE64_ENCODED
---

##
# Main deployment
##
apiVersion: apps/v1
kind: Deployment
metadata:
  labels:
    app: traefik-forward-auth
  name: traefik-forward-auth
  namespace: kube-system
spec:
  replicas: 1
  selector:
    matchLabels:
      app: traefik-forward-auth
  template:
    metadata:
      labels:
        app: traefik-forward-auth
    spec:
      containers:
        - name: traefik-forward-auth
          image: thomseddon/traefik-forward-auth
          ports:
            - containerPort: 4181
              protocol: TCP
          env:
            - name: CLIENT_ID
              value: XXXXXX.apps.googleusercontent.com
            - name: CLIENT_SECRET
              valueFrom:
                secretKeyRef:
                  name: traefik-forward-auth-secrets
                  key: CLIENT_SECRET
            - name: SECRET
              valueFrom:
                secretKeyRef:
                  name: traefik-forward-auth-secrets
                  key: SECRET
            - name: COOKIE_SECURE
              value: 'true'
            - name: DOMAINS
              value: example.com,example.org
          livenessProbe:
            tcpSocket:
              port: 4181
            initialDelaySeconds: 20
            failureThreshold: 3
            successThreshold: 1
            periodSeconds: 10
            timeoutSeconds: 2
---

##
# Related service
##
kind: Service
apiVersion: v1
metadata:
  name: traefik-forward-auth
  namespace: kube-system
spec:
  selector:
    app: traefik-forward-auth
  ports:
    - port: 80
      targetPort: 4181
      protocol: TCP

And here is how to configure an authenticated ingress using annotations (equivalent to Compose / Swarm labels):
kind: Ingress
apiVersion: extensions/v1beta1
metadata:
  name: my-app
  namespace: my-app-namespace
  annotations:
    kubernetes.io/ingress.class: traefik
    ingress.kubernetes.io/auth-type: forward
    ingress.kubernetes.io/auth-url: http://traefik-forward-auth.kube-system.svc.cluster.local
    ingress.kubernetes.io/auth-response-headers: X-Forwarded-User
spec:
  rules:
    - host: my-app.com
      http:
        paths:
          - backend:
              serviceName: my-app
              servicePort: 80

Also, here the command to encode the secrets in base64:
echo -n 'my-great-secret' | base64

Best

sorry that i am a newbie. How can i access the traefik-dashboard with this example? Do i need to add another service to serve this?

I don't think so, you can expose the dashboard by creating a service and ingress:
---
#
# Dash Service
#
apiVersion: v1
kind: Service
metadata:
  name: traefik-dashboard
  labels:
    app: traefik
spec:
  type: ClusterIP
  selector:
    app: traefik
  ports:
  - name: dashboard-http
    port: 8080
    targetPort: 8080
---
#
# Dash Ingress
#
apiVersion: extensions/v1beta1
kind: Ingress
metadata:
  name: traefik-dashboard
  labels:
    app: traefik
spec:
  rules:
  - host: traefik.ourdomain.com
    http:
      paths:
      - backend:
          serviceName: traefik-dashboard
          servicePort: dashboard-http

I'm working on making an example helm chart to simplify the deployment a little. It's my first one however, and I'm having a little trouble. Can anyone point out what I'm doing wrong. Here are my computed service, deployment and secret:
`---
Source: traefik-forward-auth/templates/secret.yml
kind: Secret
apiVersion: v1
metadata:
name: traefik-forward-auth-secrets
namespace: default
labels:
name: traefik
type: Opaque
data:
CLIENT_SECRET: b64sec1
SECRET: b64sec2
Source: traefik-forward-auth/templates/service.yml
kind: Service
apiVersion: v1
metadata:
name: traefik-forward-auth
namespace: default
labels:
annotations:
{}
spec:
selector:
app: traefik-forward-auth
ports:
- port: 80
targetPort: 4181
protocol: TCP
Source: traefik-forward-auth/templates/deployment.yml
apiVersion: apps/v1
kind: Deployment
metadata:
labels:
app: traefik-forward-auth
annotations:
{}
name: traefik-forward-auth
namespace: default
spec:
replicas: 1
selector:
matchLabels:
app: traefik-forward-auth
template:
metadata:
labels:
app: traefik-forward-auth
spec:
containers:
- name: traefik-forward-auth
image: thomseddon/traefik-forward-auth
ports:
- containerPort: 4181
protocol: TCP
env:
- DEFAULT_PROVIDER=oidc
- PROVIDERS_OIDC_ISSUER_URL=https://oidc.my-domain.com/auth/realms/master
- name: PROVIDERS_OIDC_CLIENT_ID
value: traefik-forward-auth
- name: PROVIDERS_OIDC_CLIENT_SECRET
valueFrom:
secretKeyRef:
name: traefik-forward-auth-secrets
key: CLIENT_SECRET
- name: SECRET
valueFrom:
secretKeyRef:
name: traefik-forward-auth-secrets
key: SECRET
- name: INSECURE_COOKIE
value: true
- name: COOKIE_DOMAIN
value: my-domain.com
- name: DOMAIN
value: my-domain.com
- name: AUTH_HOST
value: auth.my-domain.com
livenessProbe:
tcpSocket:
port: 4181
initialDelaySeconds: 20
failureThreshold: 3
successThreshold: 1
periodSeconds: 10
timeoutSeconds: 2`
and here is the error I'm getting:
Error: release traefik-forward-auth failed: Deployment in version "v1" cannot be handled as a Deployment: v1.Deployment.Spec: v1.DeploymentSpec.Template: v1.PodTemplateSpec.Spec: v1.PodSpec.Containers: []v1.Container: v1.Container.Env: []v1.EnvVar: readObjectStart: expect { or n, but found ", error found in #10 byte of ...|[{"env":["DEFAULT_PR|..., bigger context ...|ik-forward-auth"}},"spec":{"containers":[{"env":["DEFAULT_PROVIDER=oidc","PROVIDERS_OIDC_ISSUER_URL=|...

Sorry I was able to figure out I had bad formatting for DEFAULT_PROVIDER, PROVIDERS_OIDC_ISSUER_URL, and the value for INSECURE_COOKIE.

When I login with my organization account, I get this error: time="2020-02-26T08:18:46Z" level=warning msg="Missing csrf cookie" source_ip=<IP_ADDRESS>
Not sure what is this.


not finding robot_interface

everything compiled okay and i have the robot_interface.so file however, the skripts (a1_robot and test_robot_interface) cannot resolve it

everything compiled okay and i have the robot_interface.so file however, the skripts (a1_robot and test_robot_interface) cannot resolve it

Have you ever solved this problem?


2.5 is here :)

Just a question does the use of https://github.com/ARM-software/ComputeLibrary provide any advantages with TF on a Pi?
Do you have to compile specifically unlike Intel optimisation which is now just a environ variable?
Thought I would ask and if you had ever tried?

PS already seen it cheers Pinto :)


Geocoder::Calculations.distance_between returns distance in km.

The documentation for Geocoder::Calculations.distance_between method mentions that the distance is returned in miles ->
"# distance (in miles) between Eiffel Tower and Empire State Building
Geocoder::Calculations.distance_between([47.858205,2.294359], [40.748433,-73.985655])
=> 3619.77359999382"
I tried the method to calculate distance between two points as follows -
Geocoder::Calculations.distance_between([1.3075366324333333,103.76218967612083],[1.306306,103.76228100000003])
=> 0.13721615685642177
Calculated same distance in movable-type.co.uk and following were the results
Distance: 	0.1372 km (to 4 SF*)
Initial bearing: 	175°45′25″
Final bearing: 	175°45′25″
Midpoint: 	01°18′25″N, 103°45′44″E
Notice that the distance of 0.1372 is in km not in miles. The distance in miles using the haversine formula is 0.0852145259814396.
I think similar issue exists in the near method perhaps because same distance calculation procedure is followed there.
Please give it a look.
Thanks!

The method will actually use whatever units you have set in Geocoder.configure(units: ...). Documentation might need to be updated...


documentation extention attempt 1

see #28

Going to close in in favor of of https://github.com/aio-libs/async_lru/pull/33 , feel free to create new PR, reopen this if needed


Adjust for New Core Routes

Based on this https://github.com/kanboard/kanboard/pull/5228/files for v1.2.28

It appears that ALL the routes (in the plugin) for the core have been merged into the Kanboard core.


Setup_bam_logging.sh fails silently on re-install

Please review and merge
This PR contains the fixes for
PAAS-140- Setup_bam_logging.sh fails silently on re-install

Gayan: This pull request has become obsolete now as we have moved master branch to ppaas 4.1.0 development. Can you please confirm whether we have already applied this fix to Stratos 4.1.0?
Thanks

Thanks Imesh for checking this.
Seems this is not available in the code. This is a small fix. Added -o option for unzip .This is to  overwrite files without prompting when unzip the distribution. As we have moved the master branch to ppaas 4.1.0 , we can add this change manually to ppaas 4.1.0. So we can close this PR.


Implement immediate take-control

closes #14
Set is_our_control flag to make sure that this instance of mavlink_vehicle will
be in charge of autorotate.
Signed-off-by: Guilherme Campos Camargo<EMAIL_ADDRESS>

I still don;t like the way the control is handled. The patch looks good, but I don't think it closes #14 . Merge it and update the issue leaving just the part explaining the current situation and what the problem is.

#14 has been updated


Cannot connect with other nodes

Hi! I ran into an error that I'm not able to connect with other nodes.

n8n version: 1.3.1
Can you help me? :)

fixed in 0.1.6

a bug was made in 0.1.6 and 0.1.7, please use 0.1.8 now

Yep. 0.1.7 had the same issue. The 0.1.8 works well! Thanks!


[Feature] Added docker-compose.yml file for local development to compose with MongoDB cluster

Fixes issue #199

Added a docker-compose.yml file for local development to compose with MongoDB replica set cluster

This PR is on top of PR #202
Default secrets are in place to facilitate development and to also to make an easier experience for new contributors, added notes highlighting this point not to use these secrets or this compose out of the box in prod. Same approach that other official projects, like MongoDB docker hub page, that provide a default docker compose with values that should work out of the box.
To compose up:
docker-compose up -d
And then if you run kytosd -f -E --database mongodb with mongodb backend:
2022-04-08 18:56:55,325 - INFO [kytos.core.db] (MainThread) Trying to run 'hello' command on MongoDB...
2022-04-08 18:56:55,339 - INFO [kytos.core.db] (MainThread) Ran 'hello' command on MongoDB successfully. It's ready!
...
2022-04-08 18:56:55,339 - INFO [kytos.core.atcp_server] (MainThread) Kytos listening at <IP_ADDRESS>:6653
...
2022-04-08 18:56:55,700 - INFO [kytos.napps.kytos/topology] (MainThread) Created DB index (interfaces.id, 1), collection: switches)
2022-04-08 18:56:55,755 - INFO [kytos.napps.kytos/topology] (MainThread) Created DB index (endpoints.id, 1), collection: links)

Very good @viniarck, this will make much easier to use Mongo during development.

Thanks, Antonio. Appreciated your review.


[docs] R docs cleanup

Result: https://lightgbm.readthedocs.io/en/r_examples/R/reference/index.html

Fixed duplicated output from examples.

Before:

After:

Unified rendering of lgb.Dataset and lgb.Booster names.

Unfortunately, I have no R installed on my machine, so someone should update https://github.com/microsoft/LightGBM/tree/master/R-package/man folder before merging this PR.

I can regenerate the man/ folder right now

@jameslamb Thank you very much!

A few other small things in there that seem unrelated to your changes, seems they were not regenerated in past PRs.

Yeah, changes from #2176 were not regenerated.


Fixing invalid shader bytecode errors for OpenGL ES and Metal.

Overview
When compiling the MRTK/Standard shader for Android (OpenGL ES 1/2/3.x) you encounter a few compilation errors following this general pattern:
Shader error in 'Mixed Reality Toolkit/Standard': Program 'frag', error X8000: D3D11 Internal Compiler Error: Invalid Bytecode: Incompatible min precision type for operand #1 of opcode #29 (counts are 1-based). Expected int or uint. (on gles)
This cryptic error was due to invalid cast conversion occurring when returning the result from RoundCorners and RoundCornersSmooth. The fix was to change the return value from a low precision float to a full precision float.
After making that change all errors go away but one error. Except the following:
Shader error in 'Mixed Reality Toolkit/Standard': Non system-generated input signature parameter (BLENDWEIGHT) cannot appear after a system generated value. at line 775 (on gles3)
I tried a few things, but I don't believe I can fix this issue. I opened a Unity bug for this awhile back, it is marked as fixed but doesn't appear to be fixed in the 2019 LTS build. So I'll reach out to some folks to see if we can reopen it.
Note, building for Android a second time results in the error going away and a successful build. (In case anyone is blocked by this.)
Lastly I took this opprinunity to remove the IF macro which made code a bit hard to read/understand and doesn't provide any performance benefits.
Changes

Fixes: #6359
Fixes: #9851

Verification

Please try building MRTK for the Android platform to see if all but one of the compilation errors go away. I would also love if someone could test for iOS (since I don't have a way to build for that at the moment).
As a reviewer, it is possible to check out this change locally by using the following
commands (substituting {PR_ID} with the ID of this pull request):
git fetch origin pull/{PR_ID}/head:name_of_local_branch
git checkout name_of_local_branch

/azp run

/azp run


Backup files should only be created if the file changed

It may be intentional the way it is, if so please let me know why and close this. Otherwise, I think that there should only be *.bk files for changed files. Right now it is kind of annoying to always have to cleanup a pile of backup files even when nothing changed.

I believe this was fixed by https://github.com/rust-lang-nursery/rustfmt/pull/740. Have we regressed here?

Yes there has been a regression, it seems .bk files are being generated even if the files do not change.

I can't reproduce this with current rustfmt.
@skylerberg, @Amanieu are you able to repro with a current master? Could you post exactly how (flags, non-default config options, project structure)? Thanks!

The easiest why that I can repro this is using this repo: https://github.com/redox-os/ion. I go to the src folder and run rustfmt to_num.rs. Nothing changes, but a file to_num.rs.bk is produced. I just downloaded the and built from the latest master, so rustfmt should be up to date. I did nothing to configure it.
I am running on Ubuntu 14.04.

I tried again to repro this and could not, so closing for now. Happy to re-open if we can find a way to repro.


Relax constraints on sortedcontainers for python3.8

As in version 2.0 the method was changed from iteritems to items, it seemed easiest to connect the two in __getattr__ as _six.items doesn't exist.
The reason to upgrade sortedcontainers is to support python3.8: in that version, the path that it imports ABCs from will no longer exist:
/srv/venvs/service/trusty/service_venv_python3.7/lib/python3.7/site-packages/sortedcontainers/sortedlist.py:9
  /srv/venvs/service/trusty/service_venv_python3.7/lib/python3.7/site-packages/sortedcontainers/sortedlist.py:9: DeprecationWarning: Using or importing the ABCs from 'collections' instead of from 'collections.abc' is deprecated, and in 3.8 it will stop working
    from collections import Sequence, MutableSequence

In addition, super(ParameterMapper, self).__getattr_ doesn't exist.

No problem


Deposit: Creator search should not allow creating an entry if it's already created

Search for a user that returns no results.
Click "create a new entry"
Manually add data for the author, save and then try to search again.

Expected behaviour
If the full form with all fields are already displayed, then the text: "couldn't find ...." since it doesn't makes sense.

Addressed in #1348. The "new entry" block appears only when search_only is set in config, and not after it has been clicked the first time


Add codecov

66% not so good.

Can be merged to develop, master will have to wait for a bit. Badge images are not always rendering, but this is a known problem with GitHub (various threads about it) .


proposal: math: overflow detecting arithmetic

Currently there is no standard, efficient way of detecting overflow/underflow in integer arithmetic.
Everyone has to devise their own functions and algorithms for checking if an overflow/underflow happened, which is tedious and likely to be inefficient.
I propose a new suite of functions in the math package.
Each one would return the result together with a boolean flag set to true when an overflow/underflow took place.
func AddOverflow[T int | uint | int64 | uint64 | int32 | uint32 | int16 | uint16 | int8 | uint8](i T) (T, bool)
Add two integers and detect overflow.
func SubOverflow[T int | uint | int64 | uint64 | int32 | uint32 | int16 | uint16 | int8 | uint8](i T) (T, bool)
Subtract two integers and detect overflow.
func MulOverflow[T int | uint | int64 | uint64 | int32 | uint32 | int16 | uint16 | int8 | uint8](i T) (T, bool)
Multiply two integers and detect overflow.
func DivOverflow[T int | uint | int64 | uint64 | int32 | uint32 | int16 | uint16 | int8 | uint8](i T) (T, bool)
Divide two integers and detect overflow.
Example usage:
import "math"

var a int8 = 127
var b int8 = 1
c, ok := math.AddOverflow(a, b)
//  c = -128
// ok = false

a = 25
b = 4
c, ok = math.AddOverflow(a, b)
//  c = 29
// ok = true

Another option would be to enhance the arithmetic operators to optionally allow two return values.
This however would constitute a language change.
var a int8 = 127
var b int8 = 1
c, ok := a + b
//  c = -128
// ok = false

Integer division cannot overflow.

@icholy Consider the following example
var a int8 = -128
var b int8 = -1
fmt.Println(a / b)
# -128

Closing as dup of #24853, and related to #30209 and #32063. Some of these functions are easily simulateable with math/bits operations.
Integer division can overflow, in the very special case of math.MinInt / -1.
And arguably, /0 also. Presumably DivOverflow would return false and not panic if dividing by 0.


Fix resigning for AdHoc and Enterprise environments

Fix #159 resign for adhoc deployments include beta-reports-active
Fix #158 resign for enterprise fails to sign when beta-reports-active
doesn't exist in plist

Looks good to me. :+1:
Cc @KrauseFx

Thanks for your contribution @chrisvire and thanks @kommen vor verifying the patch :+1:


Battery threshold for service messages

Recently I experienced the following behaviour according to the battery service messages:
The battery threshold value, which can be configured by the user in the configuration of the thermostat is set to 2.1 V -- a few days ago the value dropped to 2.3 V and the Homematic Web UI showed me a service message that the battery is empty.
In my honest opinion the configured threshold value should be used to define when it's time to generate the service message.
Firmware: <IP_ADDRESS>80715
best regards and thanks for the good work

Sorry, but this is no RaspberryMatic specific issue but rather a potential issue of the general CCU firmware. So please discuss that in the homematic forum and report that issue to the vendor (eq3) instead.


feat: split the links found in front-end category to CSS and JS categories

What type of PR is this? (check all applicable)

[x] Added new link to category
[ ] New category added
[ ] Fixed broken link
[ ] Typo fix
[ ] Readme update
[ ] Optimization
[x] Documentation update
[ ] Frontend update

Description

Splitted the links found in front-end category to CSS and JS categories.

close #53


docs: Update version to 0.4.6 and enhance documentation

Add new command line flag for full export browsing data
Refactor README files for clarity and updated information

Proposed changes

Checklist

[ ] Pull request is created against the dev branch
[ ] All checks passed (lint, unit, build tests etc.) with my changes
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] I have added necessary documentation (if appropriate)

Pull Request Test Coverage Report for Build<PHONE_NUMBER>
Details

0 of 1 (0.0%)  changed or added relevant line in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage remained the same at 17.034%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

cmd/hack-browser-data/main.go
0
1
0.0%

Totals

Change from base Build<PHONE_NUMBER>:
0.0%

Covered Lines:
340

Relevant Lines:
1996

💛  - Coveralls


Integration test for settings method

Test Scenario 1: Successfully launch settings screen
Steps to reproduce:

Initialize Tiki SDK with valid parameters.
Create and accept an offer.
Call the settings method with a valid context.
Verify that the settings screen is launched successfully.

Expected Result: The settings screen should be launched successfully, with all the necessary UI elements displayed.
Test Scenario 2: SDK not initialized
Steps to reproduce:

Do not initialize Tiki SDK.
Create and accept an offer.
Call the settings method with a valid context.

Expected Result: An IllegalStateException should be thrown with a message indicating that the SDK is not initialized.
Test Scenario 3: No offers created
Steps to reproduce:

Initialize Tiki SDK with valid parameters.
Do not create any offers.
Call the settings method with a valid context.

Expected Result: An IllegalStateException should be thrown with a message indicating that no offers have been created.

outdated


platform dependent modules

closes #7

Looks good, thanks! I'll DRY up the require line and merge.

That's good, that looks good.
BTW, keep in mind, that line
https://github.com/mceachen/exiftool-vendored.js/blob/3b0a2c0eb2ec22caaac9be051a8ea48e59e7c666/src/ExifTool.ts#L19
is unreachable, because previous require will throw an error if module not exist.

The continuation of the idea how to solve problem with exiftool path.
Tests will not pass until exiftool-vendored.exe and exiftool-vendored.pl not published.
I've tested it locally on ubuntu and windows 10, temporary replacing optionalDependencies with
 "optionalDependencies": {
    "exiftool-vendored.exe": "https://github.com/amokrushin/exiftool-vendored.exe#exiftool-path",
    "exiftool-vendored.pl": "https://github.com/amokrushin/exiftool-vendored.pl#exiftool-path"
  }

all tests excluding version check passed.

Anyway, thank you for great module!

p.s. Previous idea with "exiftool": "./bin/exiftool" do not looks suitable. Because perl version of exiftool relies on cwd to find ifself lib, strange behavior, but it is what it is. On the other side, windows version of npm do not creates symlink, but *.cmd wrapper, and most of the exiftool tests are failed in this case.

Turns out the postinstall target is abysmal for this use--there isn't a way to discriminate between local development (where we'd need to compile) and normal tarball consumption of the module (where the compiled source is in the tarball already). I've dropped the postinstall validation script.
Thanks again for your help and ideas.


[New Member]: Christina Kirschbaum

Name
Christina Kirschbaum
URL name
christina-kirschbaum
ORCID identifier
https://orcid.org/0009-0000-1608-9431
Aliases
No response
Bio
Christina is a PhD student in the Bioinformatics & Translational Research Team. She studied Bioinformatics at Free University of Berlin.
Her project focuses on pangenomics, aiming to compare existing typing schemes for Bacteria and to combine pangenome graphs and cgMLST to improve the strain typing. Before, she worked on a project about variant detection and risk assessment for SARS-CoV-2 and Influenza viruses.
Socials
No response
Agreement

[ ] Yes, I have signed the document and sent it to Martin


Text-input not working properly with time-picker-inline option

Describe the bug
When using text-input option together with time-picker-inline it is not possible to write date to input field. First click opens the picker but input immediately loses the focus. The second click closes the picker and cursor is now present in the input but when I try to write something, it opens the picker and focus is lost again.
<template>
    <VueDatePicker v-model="date" placeholder="Start Typing ..." text-input time-picker-inline />
</template>

<script setup>
import { ref } from 'vue';

const date = ref();
</script>

To Reproduce
Steps to reproduce the behavior:

Use example above
Click on the input
Try to write something
See error

Expected behavior
It should work exactly same way like it works without time-picker-inline option.
Screenshots
I think this bug is pretty straightforward.
Desktop & mobile:

Chrome version 119.0.6045.159
Library version 7.2.2
MacOS Sonoma 14.1.1

Not sure what you mean, the text input works as intended.
https://github.com/Vuepic/vue-datepicker/assets/23171105/c17c89ea-26d6-4d1f-8494-a20b84db7546


ftfy bin folder conflicts if packed with other libraries.

Thanks for all your hardwork on ftfy library.
I am facing an issue while installing it using pip. I need to pack this library with several other libraries but ftfy creates a bin folder while installing it into a targeted folder. Since bin folder already exists there (created by another library), the command asks me to use forece to replace it. I checked ftfy bin folder has a file called "ftfy" and I am not sure if it really needed by the library. I just want to confirm if the code needs this binary file to work?

The binary is a command line script that runs ftfy on a file. It's created by the usual setuptools process of making an "entry point". It's not necessary for using ftfy as a library.


Replace deprecated Lucene's RegexQuery.

This is a followup of commit: 60533a3ee41e7c990bbf55351f768c7018c2e476

Current coverage is 81.55% (diff: 88.23%)

Merging #4659 into master will decrease coverage by 0.12%

@@             master      #4659   diff @@
==========================================
  Files          1307       1307
  Lines         44949      44956     +7
  Methods           0          0
  Messages          0          0
  Branches       7070       7072     +2
==========================================
- Hits          36715      36664    -51
- Misses         5559       5598    +39
- Partials       2675       2694    +19

Powered by Codecov. Last update c6f5a89...585fd04


nvm install VERSION with yarn defined in default-packages installs an old version of yarn

Operating system and version:
Ubuntu 22.04
nvm debug output:

nvm --version: v0.39.1
$SHELL: /usr/bin/zsh
$SHLVL: 1
whoami: 'me'
${HOME}: /home/me
${NVM_DIR}: '${HOME}/.nvm'
${PATH}: ${NVM_DIR}/versions/node/v16.4.0/bin:${HOME}/Android/Sdk/:${HOME}/Android/Sdk//platform-tools/:${HOME}/.local/bin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin
$PREFIX: ''
${NPM_CONFIG_PREFIX}: ''
$NVM_NODEJS_ORG_MIRROR: ''
$NVM_IOJS_ORG_MIRROR: ''
shell version: 'zsh 5.8.1 (x86_64-ubuntu-linux-gnu)'
uname -a: 'Linux 5.15.0-46-generic #49-Ubuntu SMP Thu Aug 4 18:03:25 UTC 2022 x86_64 x86_64 x86_64 GNU/Linux'
checksum binary: 'sha256sum'
OS version: Ubuntu 22.04.1 LTS
curl: /usr/bin/curl, curl 7.81.0 (x86_64-pc-linux-gnu) libcurl/7.81.0 OpenSSL/3.0.2 zlib/1.2.11 brotli/1.0.9 zstd/1.4.5 libidn2/2.3.2 libpsl/0.21.0 (+libidn2/2.3.2) libssh/0.9.6/openssl/zlib nghttp2/1.43.0 librtmp/2.3 OpenLDAP/2.5.12
wget: /usr/bin/wget, GNU Wget 1.21.2 built on linux-gnu.
git: /usr/bin/git, git version 2.34.1
ls: cannot access 'grep:': No such file or directory
grep: grep: aliased to grep --color=auto --exclude-dir={.bzr,CVS,.git,.hg,.svn,.idea,.tox} (grep --color=auto --exclude-dir={.bzr,CVS,.git,.hg,.svn,.idea,.tox}), grep (GNU grep) 3.7
awk: /usr/bin/awk, awk: not an option: --version
sed: /usr/bin/sed, sed (GNU sed) 4.8
cut: /usr/bin/cut, cut (GNU coreutils) 8.32
basename: /usr/bin/basename, basename (GNU coreutils) 8.32
rm: /usr/bin/rm, rm (GNU coreutils) 8.32
mkdir: /usr/bin/mkdir, mkdir (GNU coreutils) 8.32
xargs: /usr/bin/xargs, xargs (GNU findutils) 4.8.0
nvm current: v16.4.0
which node: ${NVM_DIR}/versions/node/v16.4.0/bin/node
which iojs: iojs not found
which npm: ${NVM_DIR}/versions/node/v16.4.0/bin/npm
npm config get prefix: ${NVM_DIR}/versions/node/v16.4.0
npm root -g: ${NVM_DIR}/versions/node/v16.4.0/lib/node_modules

nvm ls output:

❯ nvm ls
       v14.17.6
->      v16.4.0
       v16.13.0
default -> 16.4.0 (-> v16.4.0)
iojs -> N/A (default)
unstable -> N/A (default)
node -> stable (-> v16.13.0) (default)
stable -> 16.13 (-> v16.13.0) (default)
lts/* -> lts/gallium (-> N/A)
lts/argon -> v4.9.1 (-> N/A)
lts/boron -> v6.17.1 (-> N/A)
lts/carbon -> v8.17.0 (-> N/A)
lts/dubnium -> v10.24.1 (-> N/A)
lts/erbium -> v12.22.12 (-> N/A)
lts/fermium -> v14.20.0 (-> N/A)
lts/gallium -> v16.17.0 (-> N/A)

How did you install nvm?
Install script
What steps did you perform?
nvm install 16.4.0 after adding yarn to default-packages
What happened?
Yarn is at 1.22.16
What did you expect to happen?
Yarn should be at 1.22.19
Is there anything in any of your profile files that modifies the PATH?
Yes, adds ANDROID_HOME

If you are having installation issues, or getting "N/A", what does curl -I --compressed -v https://nodejs.org/dist/ print out?

❯ curl -I --compressed -v https://nodejs.org/dist/
*   Trying <IP_ADDRESS>:443...
* Connected to nodejs.org (<IP_ADDRESS>) port 443 (#0)
* ALPN, offering h2
* ALPN, offering http/1.1
*  CAfile: /etc/ssl/certs/ca-certificates.crt
*  CApath: /etc/ssl/certs
* TLSv1.0 (OUT), TLS header, Certificate Status (22):
* TLSv1.3 (OUT), TLS handshake, Client hello (1):
* TLSv1.2 (IN), TLS header, Certificate Status (22):
* TLSv1.3 (IN), TLS handshake, Server hello (2):
* TLSv1.2 (IN), TLS header, Finished (20):
* TLSv1.2 (IN), TLS header, Supplemental data (23):
* TLSv1.3 (IN), TLS handshake, Encrypted Extensions (8):
* TLSv1.3 (IN), TLS handshake, Certificate (11):
* TLSv1.3 (IN), TLS handshake, CERT verify (15):
* TLSv1.3 (IN), TLS handshake, Finished (20):
* TLSv1.2 (OUT), TLS header, Finished (20):
* TLSv1.3 (OUT), TLS change cipher, Change cipher spec (1):
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
* TLSv1.3 (OUT), TLS handshake, Finished (20):
* SSL connection using TLSv1.3 / TLS_AES_256_GCM_SHA384
* ALPN, server accepted to use h2
* Server certificate:
*  subject: CN=*.nodejs.org
*  start date: Jan 11 00:00:00 2022 GMT
*  expire date: Feb 11 23:59:59 2023 GMT
*  subjectAltName: host "nodejs.org" matched cert's "nodejs.org"
*  issuer: C=GB; ST=Greater Manchester; L=Salford; O=Sectigo Limited; CN=Sectigo RSA Domain Validation Secure Server CA
*  SSL certificate verify ok.
* Using HTTP2, server supports multiplexing
* Connection state changed (HTTP/2 confirmed)
* Copying HTTP/2 data in stream buffer to connection buffer after upgrade: len=0
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
* Using Stream ID: 1 (easy handle 0x55dd4bbc3e80)
* TLSv1.2 (OUT), TLS header, Supplemental data (23):
> HEAD /dist/ HTTP/2
> Host: nodejs.org
> user-agent: curl/7.81.0
> accept: */*
> accept-encoding: deflate, gzip, br, zstd
>
* TLSv1.2 (IN), TLS header, Supplemental data (23):
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
* TLSv1.3 (IN), TLS handshake, Newsession Ticket (4):
* old SSL session ID is stale, removing
* TLSv1.2 (IN), TLS header, Supplemental data (23):
* Connection state changed (MAX_CONCURRENT_STREAMS == 256)!
 Really confusing.

How are you "showing" a yarn version? Are you using corepack at all?

Not consciously using corepack, but maybe? I'm not that familiar.
I did yarn -v and then a which yarn to confirm it was pointing to where I expected.

Are you using an internal registry that might not have had .19?

Nope.
We did have a .yarnrc.yaml file or whatever it is that lets you set a specific Yarn version for the local project (removed just now for unrelated reasons) but it defined 1.22.19 so I'm really not sure what could have caused the issue.
We probably previously had 1.22.16 installed some months back, but otherwise I have no theories about possible causes.

Since you presumably can’t reproduce the .16 issue, and nvm just calls npm install with whatever’s in default_packages, I’m not sure if there’s anything that can be done here :-/

Since it's worked itself out and we've worked the possible causes through, closed it is. Thank you very much for your attention as always.


Mock implementation for unit testing

One thing I found very helpful in Angular 1 was the $httpBackend  service. This provides a fake HTTP backend to verify requests made by the unit under test. If, for instance, the code to test does this:
// The controller code
function MyController($scope, $http) {
  var authToken;

  $http.get('/auth.py').then(function(response) {
    authToken = response.headers('A-Token');
    $scope.user = response.data;
  });
}

then the test could be:
// testing controller
describe('MyController', function() {
   beforeEach(inject(function($httpBackend) {
     ...
     authRequestHandler = $httpBackend.when('GET', '/auth.py')
                            .respond({userId: 'userX'}, {'A-Token': 'xxx'});
   }));

   afterEach(function() {
     $httpBackend.verifyNoOutstandingExpectation();
     $httpBackend.verifyNoOutstandingRequest();
   });

   it('should fetch authentication token', function() {
     $httpBackend.expectGET('/auth.py');
     var controller = createController();
     $httpBackend.flush();
   });
});

It would be very helpful to have something similar for the fetch-client.

Thanks for the suggestion, but I don't think we'd gain anything by adding an API for this. For unit tests I would recommend mocking HttpClient. If you want to run HttpClient code in your test then you can either mock window.fetch, or add a request interceptor that returns a Response, which will bypass the call to window.fetch.

It's not just the mocking itself, it's the administration of (un)met expectancies. I think this is a pretty common concern for projects that make requests to a backend. It would be nice to have it in this project, so people don't have to re-invent the wheel.
On the other hand, if you hook into window.fetch, you can do the same without relying on the HttpClient API. So perhaps this belongs in the testing framework itself, or a plugin, or a separate project.
Thanks for your time.


Feature: do not filter testdata in results once we have raw result support is landed

See https://github.com/ossf/scorecard/issues/1256#issuecomment-1000618647
The testdata folders should be filtered out thru the policy

This one could be good to revisit again, either for maintainer annotations or policy per ecosystem?


disable in app purchases

I think we can disable in app purchases @DavidHallgren:
https://github.com/euclidthegame/euclidthegame.iOS/blob/b76dd3f7d6b5fd321a8dbec4097e7ae0f1c65c3e/Euclid.xcodeproj/project.pbxproj#L612

done in this commit:
https://github.com/euclidthegame/euclidthegame.iOS/commit/034c5d41709d8069068ae7675e202ec88cf62d4a


Add a bat file to do the same thing as the shell script

This is still lacking some basic checks that you are typing the the correct name of a Service.
I didn't manage to get the postgres to work properly either. Rabbit seems to work.

Well, I don't, but since there is a Git CMD as well, and some people don't understand the glory of bash, it might help some poor windows user... :-)
But mostly I liked your implementation and wondered if I could pull it of with a bat file as well.
Regarding postgres, it couldn't start due to the user had no rights to the mapped volume. (I think) I need to test it some more, but right now I did an update of Docker and it will not start again.
If you don't what this cmd based script, feel free to discard it and just see it as a token of appreciation :-)

I've pushed a couple of commits, take a pull and give it a new try in Git Bash :-)

This is absolutely great work, but I'm closing this PR since I don't want the maintenance burden of a special Windows version for now. I've added a note to the readme on how to run it under Windows using Git Bash.


Viewport issue

While developing we faced with viewport issue.

You can see on the gif that the lower one carousel is disappearing.

It seems like the lower grid is indeed disappearing but the title is still visible. Can you provided us a part of the source so we can investigate?

Problem was in the definition of the boundsMargin in the listComponent, by either removing the boundMargin or specifying a number larger then 0 for the top we managed to fix the problem.
Closing the ticket for now.


Merge AADProfile during update

What this PR does / why we need it:
This PR merges old and new AADProfile API models for the update case as all properties are not required during update and ACS Engine merge API models for the caller.

/approve
/lgtm

LGTM

Thanks @jackfrancis for the quick review!


Does not work (list index out of range)

I cant get it to work:
Loading eulerian_source_videos/baby.mp4
Traceback (most recent call last):
  File "replicate_study.py", line 84, in <module>
    replicate_study()
  File "replicate_study.py", line 75, in replicate_study
    pyramid_levels=pyramid_levels
  File "/home/calle/eulerian-magnification/eulerian_magnification/base.py", line 27, in eulerian_magnification
    vid_data = collapse_laplacian_video_pyramid(vid_pyramid)
  File "/home/calle/eulerian-magnification/eulerian_magnification/pyramid.py", line 68, in collapse_laplacian_video_pyramid
    return pyramid[0]
IndexError: list index out of range

Here is the exact way i tried to use it from a fresh ubuntu 1604 vm.
    6  sudo apt install python3-pip
    7  sudo apt install git
   12  git clone https://github.com/brycedrennan/eulerian-magnification
   13  cd eulerian-magnification/
   16  pip3 install -r requirements.txt
   19  pip3 install opencv
   22  pip3 install matplotlib
   25  sudo apt-get install python3-tk
   28  sudo python3 setup.py install
   31  pip3 install numpy scipy matplotlib
   32  python3 replicate_study.py
   33  history

Yes I haven't done significant work on this for a very long time. Does not surprise me that it's broken. If you figure out the issue I welcome a pull request.

I've added a dockerfile which should allow you to play with this without worrying about installing dependencies correctly


luci-mod-admin-full: Allow setting cache size and flag to query all available upstream servers to admin_network dhcp page

This allows setting cache size (defaults to 150) and flag to query all available upstream servers to admin_network dhcp page.
Original patch found at https://github.com/gwlim/mips24k-ar71xx-lede-patch/blob/lede-17.01/patch/063-add-DNS-cachesize-option.patch
Signed-off-by: Daniel Petre<EMAIL_ADDRESS>

Just a formal feedback:

Please "sign-off" every commit, no only the PR.
Don't mix different core changes in one PR.

This can't be applied in this form.


Exceptions caused when Eggdrop/Bmotion bots wave around their special characters.

<Ashley> Good.. bad.. I'm the bot with the gun%percent^$£*!%percent$^%percent!!£~£~#.
<Justice> An exception has occured and has been logged: https://gist.github.com/c80e29ca1a778c152c15 | 'utf-8' codec can't decode byte 0xa3 in position 96: invalid start byte

https://gist.github.com/c80e29ca1a778c152c15

Error reproducable from within Bmotion itself, with offending code:
https://github.com/jamesoff/bmotion/blob/master/modules/extra/randomword.tcl#L80 (more permanent link)
Still don't know how to fix this, though. Adding 'help wanted' tag.

Something like this
+encodings = ("utf-8", "iso-8859-1")
+	for encoding in encodings: # This should probably not be a permanent solution.
+		try:
+			return buff.decode(encoding)
+		except ValueError:
+			pass
+	raise ValueError("Could not decode whois response from {server}".format(server=server))

will probably fix it, but considering it's frequency, I don't think it's very much of an issue.


include pdf.js with url parameters cause pdfjsDistBuildPdfWorker is undefined

Configuration:

Web browser and its version: Chrome 58.0.3029.110 (64-bit) / Firefox 53.0.3 (32 bit)
Operating system and its version: win 10 - v.1703
PDF.js version: 1.7.225

Steps to reproduce the problem:

include pdf.js in html page with an url param -> <script src=".../pdf.js?param=value" type="text/javascript"></script>
pdfjsDistBuildPdfWorker is not defined

What is the expected behavior?
window.pdfjsDistBuildPdfWorker is defined at line 8129
What went wrong?
window.pdfjsDistBuildPdfWorker is undefined
Solution:
Change function getWorkerSrc() at line 8116, in thi mode:
return pdfjsFilePath.replace(/\.js$/i, '.worker.js'); -> return pdfjsFilePath.replace(/\.js(\?.*)?$/i, '.worker.js');

Closing as duplicate of issue #5710, which was fixed by PR #8309.


Error entries in system Logfile

Home Assistant release with the issue:

Last working Home Assistant release (if known):
Operating environment (Hass.io/Docker/Windows/etc.):

Component/platform:

Description of problem:
Following errors appears time by time:
Problem-relevant configuration.yaml entries and (fill out even if it seems unimportant):
netatmo:
  api_key: "mykey"
  secret_key: "mysecretkey"
  username: "myuser"
  password: "mypassword"

sensor:
  # Weather prediction
 - platform: netatmo
    station: IMEERBUS121
    modules:
      Garten:
        - temperature
        - humidity
        - min_temp
        - max_temp
      RS Garten:
        - rain
        - sum_rain_1
        - sum_rain_24
      Wind Garten:
        - windstrength
        - guststrength

Traceback (if applicable):
Thu Mar 28 2019 08:05:12 GMT+0100 (Mitteleuropäische Normalzeit)
Error doing job: Task exception was never retrieved
Traceback (most recent call last):
  File "/usr/local/lib/python3.7/site-packages/homeassistant/helpers/entity_platform.py", line 344, in _async_add_entity
    raise HomeAssistantError(msg)
homeassistant.exceptions.HomeAssistantError: Entity id already exists: sensor.netatmo_wind_garten_strength. Platform netatmo does not generate unique IDs

Additional information:

Closing in favor of duplicate #21885 which references possible duplicates #18113 #20375 #20383 #21031. Please use GitHub search next time before opening an issue 😸

sorry :-)

No problem!


Provide trail from user story to concrete tools

To make onboarding easier there should be documentation which describes the trail coming from user stories (what do I want/need to do) to capabilities of the different functional building blocks of the big picture (e.g. License & copyright scanner) and to a concrete instances of tools which implement the capabilities (this will also be a good base to identify needed glue code and/or APIs to be implemented in the concrete tools)

Fully agree that we should have this tracing top-down, which also helps in shaping the requirements for new tools to be added, extensions/adaptations of existing tools to be made and/or the necessary interfaces.
Based on an action from the previous F2F meeting we are currently working on part of this topic in a smaller group, i.e. setting up the capabilities (glossary) and the mapping of architectural components in the unanimous understanding back to the capabilities (reverse mapping). Further, we would like to harmonize the terms and semantics also with regards to the "big picture" that you have created, ideally on the level of capabilities. The next step would then be to derive the needs and structure for necessary interfaces between components (finally mapping to APIs used by tools), which should ideally make use of the data types defined in the data model, thus closing the loop.
As there are still quite some needs for discussions we have been keeping the updates in the separate branch defining-capability-map for the moment. Any comments/suggestions are welcome.

Great that there is already work available. I had a look at:
https://github.com/Open-Source-Compliance/Sharing-creates-value/blob/defining-capability-map/Tooling-Landscape/Unanimous-Understanding/OC-Reference-Tooling-Capabilities-Glossary.md
What we still have to add is the user story part to the capabilities and the capabilities -> concrete tools.


Remove debug prints.

The debug prints like the one on EPA should be removed to avoid altering the console output of applications using ncollide.

Is this ticket still open? Would love to start contributing from here.


Echo participant undiscovery + Discovery callbacks update

Echo participant only advised participant discovery.
With this PR it also advises participant undiscovery when participant has been updated and it is not active.
In addition, participant discovery callback is now also implemented in DdsCommonParticipant, and already implemented discovery callbacks (both for RTPS and DDS) updated to adapt to changes introduced in Fast-DDS middleware (e.g. ignore participant/endpoint feature).

TODO:
Review all possible DISCOVERY_STATUS
enum DISCOVERY_STATUS
{
    DISCOVERED_PARTICIPANT,
    CHANGED_QOS_PARTICIPANT,
    REMOVED_PARTICIPANT,
    DROPPED_PARTICIPANT,
    IGNORED_PARTICIPANT       <----
};

Codecov Report
Attention: Patch coverage is 9.30233% with 39 lines in your changes are missing coverage. Please review.

Project coverage is 34.07%. Comparing base (07dc292) to head (3bac881).

Files
Patch %
Lines

...ants/src/cpp/participant/dds/CommonParticipant.cpp
10.71%
15 Missing and 10 partials :warning:

...s/src/cpp/participant/auxiliar/EchoParticipant.cpp
12.50%
7 Missing :warning:

...nts/src/cpp/participant/rtps/CommonParticipant.cpp
0.00%
7 Missing :warning:

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #97      +/-   ##
==========================================
- Coverage   34.19%   34.07%   -0.13%
==========================================
  Files         154      154
  Lines        7781     7818      +37
  Branches     3465     3494      +29
==========================================
+ Hits         2661     2664       +3
- Misses       3230     3253      +23
- Partials     1890     1901      +11

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


md-input-container with md-prefix produces unexpected DOM

Bug,
<md-input-container>       <span md-prefix>           <md-icon>home</md-icon>       </span>       <input mdInput name="givenName" readonly disabled>   </md-input-container>
gives wrong DOM, div.mat-input-prefix is empty while the span.md-prefix is generated inside div.mat-input-infix
could be seen on the following plnkr:
http://plnkr.co/edit/WRtJEaWx0ilHPBeGLSM5?p=preview
which is regression from beta.6 on 4.1.3:
http://plnkr.co/edit/dGxYTBaObsjDs557jM4k?p=preview

mdPrefix works instead of md-prefix though I don't totally understand it


ERROR: Trying to select an annotation which has not been added

Error occurred when selecting the button added on the MyCustomCallout then it logs the error on the console and the button action callback doesn't call.
This issue is random but observer frequently.
When user Zoom-In and zoom-out on the map and then open the callout view and selects the button then its works fine.

I don't understand the problem you are describing. Can you please provide a sample project that demonstrates this issue.

I assume you have solved your problem because I haven't heard back for a while. Please reopen if needed.

Also seeing this on my end, working on repro steps

Trying to select an annotation which has not been added ... sometimes This error when i am click anotationView..i am using delegate method  func mapView(mapView: MKMapView, didSelectAnnotationView view: MKAnnotationView)

Please provide a sample project so I can debug this problem

Trying to select an annotation which has not been added ... sometimes This error when i am click anotationView..i am using delegate method func mapView(mapView: MKMapView, didSelectAnnotationView view: MKAnnotationView)

Hi,
I am also facing the same issue.
Any updates on this?
Thanks.


feat(publisher-s3): allow ACL omission

This allows the caller to omit the ACL from the upload request, per Amazon's recommendation of using bucket owner-enforced permissions.

[x] I have read the contribution documentation for this project.
[x] I agree to follow the code of conduct that this project follows, as appropriate.
[x] The changes are appropriately documented (if applicable).
[x] The changes have sufficient test coverage (if applicable).
[x] The testsuite passes successfully on my local machine (if applicable).

Summarize your changes:
AWS recommends disabling the use of ACLs and instead using bucket owner-enforced permissions (see https://docs.aws.amazon.com/AmazonS3/latest/userguide/ensure-object-ownership.html). The current configuration and implementation of the S3 publisher always applies an ACL, which simply doesn't work with buckets configured this way. This PR adds an omitAcl option to the config that simply omits the ACL from the upload command.
I considered making this config look more like the Google Cloud Storage publisher (which uses essentially applies some of its options, such as predefinedAcl, public, and private, directly to its upload object); however, that would be a breaking change, and I opted for keeping this non-breaking. I'm happy to refactor if you think that is preferred.

Added your comment, @felixrieseberg. Thank you!


Task failed: Restore node_modules cache

I wonder if this task would be a candidate for automated retry:
https://dev.azure.com/monacotools/Monaco/_build/results?buildId=152867&view=logs&j=672276a2-8d3a-5fab-615d-090c51352f92&s=27eddb93-7805-576c-c80f-37b2176e40f7&t=ea87c51f-82a8-56c2-f294-bc5abf1e656f&l=1
Starting: Restore node_modules cache
==============================================================================
Task         : Cache
Description  : Cache files between runs
Version      : 2.198.0
Author       : Microsoft Corporation
Help         : https://aka.ms/pipeline-caching-docs
==============================================================================
Resolving key:
 - nodeModules                                       [string]
 - Windows_NT                                        [string]
 - .build/arch, .build/terrapin, .build/yarnlockhash [file pattern; matches: 3]
   - s\.build\arch         --> 5609F728403E197BB255EF50C62AEABB1F93B09F7B7C379903440B65CD4319CB
   - s\.build\terrapin     --> 3CBC87C7681F34DB4617FEAA2C8801931BC5E42D8D0F560E756DD4CD92885F18
   - s\.build\yarnlockhash --> 961BA8D1CA23FB7CE762807FACEDD0354A70FAF0679CBF660F525F01A184C368
Resolved to: nodeModules|Windows_NT|PiwMiHOtDOo+eFSXXrBVw8HDsjh+OKhdrZmngKSZ8o0=
Using default max parallelism.
Max dedup parallelism: 192
ApplicationInsightsTelemetrySender will correlate events with X-TFS-Session 680621af-b1a2-4934-aa02-e4c091c517bd
Getting a pipeline cache artifact with one of the following fingerprints:
Fingerprint: `nodeModules|Windows_NT|PiwMiHOtDOo+eFSXXrBVw8HDsjh+OKhdrZmngKSZ8o0=`
ApplicationInsightsTelemetrySender correlated 2 events with X-TFS-Session 680621af-b1a2-4934-aa02-e4c091c517bd
##[error]The HTTP request timed out after 00:01:40.
Finishing: Restore node_modules cache

Unfortunately:

Issue in DevOps: https://github.com/microsoft/azure-pipelines-tasks/issues/11864
Retrying doesn't solve it: https://github.com/microsoft/azure-pipelines-tasks/issues/15501


Refractor SOMAExperimentAxisQuery obs() var() and X() to allow for iteration.

Issue and/or context:
Completes #1285
Changes:

SOMAExperimentAxisQuery$obs() returns a TableReadIter
SOMAExperimentAxisQuery$var() returns a TableReadIter
SOMAExperimentAxisQuery$X() returns a SOMASparseNDArrayRead
Updates internal uses of the above
Updates tests
Add tests
Updates vignette.

library(tiledbsoma)

ctx <- tiledbsoma::SOMATileDBContext$new()
config <- as.character(tiledb::config(ctx$context()))

human_experiment = load_dataset("soma-exp-pbmc-small", tiledbsoma_ctx = ctx)

query <- SOMAExperimentAxisQuery$new(human_experiment, "RNA")

query$obs()
#<TableReadIter>
#  Inherits from: <ReadIter>
#  Public:
#    clone: function (deep = FALSE)
#    concat: function ()
#    initialize: function (sr)
#    read_complete: function ()
#    read_next: function ()
#  Private:
#    soma_reader_pointer: externalptr
#    soma_reader_transform: function (x)

query$obs()$read_next()
#Table
#80 rows x 9 columns
#$soma_joinid <int64 not null>
#$orig.ident <large_string>
#$nCount_RNA <double>
#$nFeature_RNA <int32>
#$RNA_snn_res.0.8 <large_string>
#$letter.idents <large_string>
#$groups <large_string>
#$RNA_snn_res.1 <large_string>
#$obs_id <large_string>

query$X("counts")
#<SOMASparseNDArrayRead>
#  Public:
#    clone: function (deep = FALSE)
#    initialize: function (sr, shape)
#    sparse_matrix: function (zero_based = FALSE)
#    tables: function ()
#  Private:
#    shape: 80 230
#    sr: externalptr

query$X("counts")$tables()$read_next()
#Table
#4456 rows x 3 columns
#$soma_dim_0 <int64 not null>
#$soma_dim_1 <int64 not null>
#$soma_data <double not null>

str(query$X("counts")$sparse_matrix()$read_next())
#Formal class 'dgTMatrix' [package "Matrix"] with 6 slots
#  ..@ i       : int [1:4456] 0 0 0 0 0 0 0 0 0 0 ...
#  ..@ j       : int [1:4456] 1 5 8 11 22 30 33 34 36 38 ...
#  ..@ Dim     : int [1:2] 80 230
#  ..@ Dimnames:List of 2
#  .. ..$ : NULL
#  .. ..$ : NULL
#  ..@ x       : num [1:4456] 1 1 3 1 1 4 1 5 1 1 ...
#  ..@ factors : list()

Codecov Report
Patch coverage has no change and project coverage change: -11.47 :warning:

Comparison is base (43d09be) 64.45% compared to head (e4c5079) 52.98%.

:exclamation: Current head e4c5079 differs from pull request most recent head 14669ba. Consider uploading reports for the commit 14669ba to get more accurate results

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.
Additional details and impacted files
@@             Coverage Diff             @@
##             main    #1505       +/-   ##
===========================================
- Coverage   64.45%   52.98%   -11.47%
===========================================
  Files         102       72       -30
  Lines        8324     5822     -2502
===========================================
- Hits         5365     3085     -2280
+ Misses       2959     2737      -222

Flag
Coverage Δ

python
?

r
52.98% <ø> (-0.03%)
:arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.
see 31 files with indirect coverage changes

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.


Add support for 1.8+

Kraken only supports 1.7 clients, players on 1.8+ will receive a blank tab list upon joining, players that join after will populate the tab list.

@alexandeh Players on 1.8 for me receive a regular tab list, with player names.

Edited, thank you @lolitsalex.

The tab doesnt update

@codinglib Create a runnable? That is what I did, worked fine for me.
NOTE: Run it async if you are going to do this.
ANOTHER NOTE: This is not the way @alexandeh probabily did it, so I would suggest you go to the wiki or ask him.

@codinglib Anixie because you don't know how to code anything without JD-GUI

@yaboicriminal  omfg that was fucking funny and realistic xD

Great API

The 1.8 tablist handles a lot more players, so you should try to go with a 4x20 tablist [80 entires, not 60],
can be made with
((CraftPlayer)p).getHandle().playerConnection.networkManager.getVersion();

All tab entries are attempting to be added to my scoreboard (on the right of screen).
Any ideas? https://gyazo.com/2f39241ecbed7835056568e22971ff50