S3 Duplicates

Currently, if a duplicate image is added to the S3 bucket, I believe it simply does not add a new one.. Need to check for existing names..? Or something..? Look into it

Also, if a war is deleted or if a picture is changed, the old photo has to be removed..


Authentication fails when using AT&T TV login (ATTOTT)

Checklist

[ ] I'm reporting a broken site support issue
[X] I've verified that I'm running yt-dlp version 2021.07.24
[X] I've checked that all provided URLs are alive and playable in a browser
[X] I've checked that all URLs and arguments with special characters are properly quoted or escaped
[X] I've searched the bugtracker for similar bug reports including closed ones
[X] I've read bugs section in FAQ

Verbose log
[debug] Command-line config: ['-v', '--ap-mso', 'ATTOTT', '--ap-username', 'PRIVATE', '--ap-password', 'PRIVATE', 'https://disneynow.com/shows/bunkd/season-05/episode-19-a-star-is-torn/vdka23900520']
[debug] Encodings: locale UTF-8, fs utf-8, out utf-8, pref UTF-8
[debug] yt-dlp version 2021.07.24 (zip)
[debug] Python version 3.8.10 (CPython 64bit) - Linux-5.4.0-80-generic-x86_64-with-glibc2.29
[debug] exe versions: ffmpeg N-53917-g5ff2ff6bd9-static, ffprobe N-53917-g5ff2ff6bd9-static
[debug] Proxy map: {}
[debug] [Go] Extracting URL: https://disneynow.com/shows/bunkd/season-05/episode-19-a-star-is-torn/vdka23900520
[Go] vdka23900520: Downloading JSON metadata
[Go] VDKA23900520: Downloading Provider Redirect Page
[Go] VDKA23900520: Downloading Provider Login Page
ERROR: Unable to extract post url; please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; see  https://github.com/yt-dlp/yt-dlp  on how to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.
Traceback (most recent call last):
  File "/usr/local/bin/localyt-dlp/yt_dlp/YoutubeDL.py", line 1152, in wrapper
    return func(self, *args, **kwargs)
  File "/usr/local/bin/localyt-dlp/yt_dlp/YoutubeDL.py", line 1177, in __extract_info
    ie_result = ie.extract(url)
  File "/usr/local/bin/localyt-dlp/yt_dlp/extractor/common.py", line 572, in extract
    ie_result = self._real_extract(url)
  File "/usr/local/bin/localyt-dlp/yt_dlp/extractor/go.py", line 238, in _real_extract
    auth = self._extract_mvpd_auth(
  File "/usr/local/bin/localyt-dlp/yt_dlp/extractor/adobepass.py", line 1580, in _extract_mvpd_auth
    mvpd_confirm_page_res = post_form(provider_login_page_res, 'Logging in', {
  File "/usr/local/bin/localyt-dlp/yt_dlp/extractor/adobepass.py", line 1376, in post_form
    post_url = self._html_search_regex(r'<form[^>]+action=(["\'])(?P<url>.+?)\1', form_page, 'post url', group='url')
  File "/usr/local/bin/localyt-dlp/yt_dlp/extractor/common.py", line 1133, in _html_search_regex
    res = self._search_regex(pattern, string, name, default, fatal, flags, group)
  File "/usr/local/bin/localyt-dlp/yt_dlp/extractor/common.py", line 1124, in _search_regex
    raise RegexNotFoundError('Unable to extract %s' % _name)
yt_dlp.utils.RegexNotFoundError: Unable to extract post url; please report this issue on  https://github.com/yt-dlp/yt-dlp . Make sure you are using the latest version; see  https://github.com/yt-dlp/yt-dlp  on how to update. Be sure to call yt-dlp with the --verbose flag and include its complete output.

Description
This is not an issue with any one specific site, it is instead an authentication issue, so the above is just an example, I could have picked any show on any supported site and would have had the same problem.  From what I have read, what is now called AT&T TV was formerly DIRECTV NOW and was supported using "--ap-mso ATTOTT" option.  However, since they changed to AT&T TV, authentication apparently hasn't worked.
Things that do NOT work (at least not for me):

Using the --cookies option and a valid cookies.txt file.  You get: "ERROR: This video is only available for users of participating TV providers. Use --ap-mso to specify Adobe Pass Multiple-system operator Identifier and --ap-username and --ap-password or --netrc to provide account credentials."

Getting the source code and changing the adobepass.py file (in the extractors directory), specifically these lines near the top of the file:

    'ATTOTT': {
        'name': 'DIRECTV NOW',
        'username_field': 'email',
        'password_field': 'loginpassword',
    },

Perhaps I did not hit the right combination but I did inspect the elements on the login page in my web browser and it appears that 'username_field' should be 'userID' (not 'email') and 'password_field' should be just 'password', not 'loginpassword', but I can't be certain.  I also tried changing the name field to 'AT&T TV'.  I also tried a few variations but nothing seemed to work.  I can see that it is failing at the point of trying to authenticate but I know very little about programming or about effectively using browser tools, so I'm afraid I've reached the limit of my diagnostic abilities.  If you are in the USA you can get a free two week trial account, although you don't need even that to get to the login screen.  Playback works fine in the browser using the same user ID and password, so I would hope this isn't a difficult fix. If you have any suggestions for anything else I can try, I'd appreciate hearing them.

You will need to provide an account if you want this to be fixed @tv21.

You will need to provide an account if you want this to be fixed @tv21.

Unfortunately the problem with that is that it not only gives you open access to the account, but also to all the personal and billing information associated with the account, including payment info.  That is information I would not even give to most members of my family, let alone someone on the Internet.  No offense intended, but other that by your work on this software I don't know anything about you.  You might be the most trustworthy guy in the world (and I hope you are) but I have no way of knowing that.  Again, no offense intended, I would have the same reservations about giving this information to nearly every other person on the Internet, including almost all of the people that I have met personally.
So my question is this, if you went to a provider site (such as the link I tried to use in my original post) and attempted to do a login and specified AT&T TV (or whatever the DirectTV equivalent is if they have changed the name) would you have any way of observing what is being requested and seeing how that differs from what the ATTOTT code is expecting to see?  I have this feeling that it's just a matter of a changed string somewhere, for example maybe instead of "email" or "loginpassword" different strings are being used.  Thing is, if you can even take a educated guess as to what should be changed, I am more than willing to test it to see if it works.  I know how to run the program using the Python source code (even if I don't understand the code itself), so I can easily test any changes to the code if you can just tell me which file to change, and give me some hint as to what to try changing.  That way I would not have to reveal my login information, and I will be sure to let you know if it works.
But also, I had another thought.  Would it be possible to make yt-dlp accept cookies.txt files from more services?  I have no idea why with some sites you can use a cookies files and others you can't - it it just that the various extractors were never coded to accept the cookies files, or is it really that you just absolutely can't use a cookies file with certain  online services that require a provider login?  For example, would it be possible to make the go.py extractor accept a cookies file?  The nice thing about using a cookies files is that (I assume) it contains all the authentication information that the site needs.  What I find interesting is that I can log into the AT&T TV account in a browser window and even if I come back days later and attempt to watch something, it won't ask for my login info again, as long as I haven't cleared the cookies.  So this is why I would assume that a cookies file would include everything you need to do a successful authentication.  But attempting to use a cookies file with yt-dlp for the example link above gets me nowhere, and I don't know why, when that technique works fine on some other services.  Can you please enlighten us on why this is, and could the use of a cookies file possibly be expanded to more extractors and services?

Closing this issue as ATT TV is not ATTOTT, and you have failed to provide the needed information to resolve this.

Um, shirt-dev, who are you?  I thought I was talking to pukkandan!  And yes, AT&T TV is ATTOTT, it was originally DirecTV Now I believe, then it was renamed to AT&T TV and at some point given the label ATTOTT (which I assume stands for AT&T's Over-The-Top service, to distinguish it from AT&T U-Verse and other services that were part of AT&T's wireline or satellite services.  And now it has come full circle and AT&T TV is now DirecTV something or other again, but many sites still refer to it as AT&T TV.
Anyway, I wish you had let pukkandan respond to this, especially since you seem to be under some misconceptions.  And I really would like to know about the cookies file thing.  But I'll create another issue for that.

Um, shirt-dev, who are you? I thought I was talking to pukkandan! And yes, AT&T TV is ATTOTT, it was originally DirecTV Now I believe, then it was renamed to AT&T TV and at some point given the label ATTOTT (which I assume stands for AT&T's Over-The-Top service, to distinguish it from AT&T U-Verse and other services that were part of AT&T's wireline or satellite services. And now it has come full circle and AT&T TV is now DirecTV something or other again, but many sites still refer to it as AT&T TV.
Anyway, I wish you had let pukkandan respond to this, especially since you seem to be under some misconceptions. And I really would like to know about the cookies file thing. But I'll create another issue for that.

I believe I'm the one who initially replied to this issue, pukkandan simply added the account-needed label.
ATT TV differs from ATT TV Now (formerly DirecTV Now), which was discontinued. Their API is different from ATT TV Now. The fact is that credentials are needed to add support for an AdobePass MSO. Once you provide those credentials, then we can look into adding support.
As for the question regarding the cookies, that is due to the nature of how the AdobePass authentication process works.

I think the reason most people would be reluctant to provide their login credentials is that it not only gives open access to their account, but also to all the personal and billing information associated with the account, including payment info.
I had been told in a forum thread that AT&T TV was the same as ATT Now and the former DirecTV Now, but maybe I was given wrong information.  If there is any way at all that you think you might be able to add support just from what you know about the service, I'd be happy to test it and report back the results.  But I cannot give you any credentials because the account is not in my name and the person whose name it is in (another family member) absolutely forbids it.

No, it is not possible to write an extractor without an account

shirt-dev, just FYI I was talking to someone else this afternoon who was looking at the inspector for another video on the same site as the one I mentioned in my original request, and he says that when he selects AT&T TV as the provider he can see ATTOTT in the inspector, and if you click on it, it shows this image:
https://cdn1.edgedatg.com/aws/v2/abc/consolidatedMVPDs/67883574-d995-4ec8-b85a-4c9b0bb058ac/454d1e59105976498d191c73d3b093c4/528x132-Q80_454d1e59105976498d191c73d3b093c4.png
So he thinks you are wrong about AT&T TV not using the ATTOTT MSO designation.  And that brings me back to the thought that if the ATTOTT extractor really is the correct one, then all that needs to be determined is what has changed in the login procedure.  And to me it would seem that this is information that should be available BEFORE a login and password is entered.  And that is why I said that if you have any thoughts at all on what may need to be changed, I'd be happy to try them.
The person I was talking to also said it sounded like you either just don't want to make this work, or that you really want an AT&T TV login for some reason, even though it would provide access to all the information related to the account and would therefore be a huge security failure for the person that provided it.  I do not know if either of those are true, I just know that I cannot give you the login that I use, I've been absolutely forbidden from doing that by the account holder.

I could be wrong about ATTOTT, but that does not change that an account is needed to write an extractor.
Just so you're aware, you're coming across as very rude and confrontational. Do I need to remind you that this is free software that we develop in our spare time?
You're welcome to find someone else to write you  an extractor, but after your behavior, I wouldn't write it even if you provided the required account.

Exactly what makes you think you can demand users provide their private account details and they will not have reservations about doing so?  Something is very fishy about this, and I think what you really don't like is me pointing out that no sane person would just hand over their account details to someone they've never met on the Internet.  You find it "rude and confrontational" that I suggest that is an unreasonable demand.  I just don't know what to say to that, because I guess you must be living in an alternate reality where everyone is trustworthy, but clearly you never had any intention of trying to help unless I could meet that insane demand, so I guess I haven't lost a thing by you being pissed off about it.

Nobody is demanding that you provide account details. We are simply saying that we cannot fix the login unless we have access to an account. Since we dont have an account ourselves, you either need to give account details, or fix it yourself and make a PR


[Swift 1.2] Query.Swift missing argument in parameter #2 in call

In commit 0bb3a86d, the changes to Query.swift for the new style closure is causing an issue with split. reverting the change on that line resolves the compile error on line - missing argument in parameter #2 in call.
let pair = split(each.expression.SQL) { $0 == "." }

to
let pair = split(each.expression.SQL, { $0 == "." })

I don't have the brain capacity today to really look into it, so I thought i would inform instead!

@ssherar Did you download yesterday's newly-released Xcode/Swift beta?
The commit you're referencing requires you're on Xcode 6.3b2 (Swift 1.2b2).

@stephencelis Alas that is the issue once again. How on earth did I miss that update?
Cheers!


Dashboard is a bit messed up in v0.4.0

Describe the bug
When many clusters are defined in configuration, the dashboard looks messed up:

multiple tables instead of 1
cluster ordering is out-of-order (that was fixed earlier for left side list)
columns are not aligned

Set up

Docker-compose with configuration set via environment variables (v0.4.0 tag from Docker Hub)

Steps to Reproduce
Steps to reproduce the behavior:

Add 10+ clusters to the list
Look at the dashboard

Expected behavior
Dashboard looks clean (single table, columns aligned, clusters ordered same as in configuration)
Screenshots

Additional context

Screenshot cannot show ordering issue due to needing to mask cluster names for privacy

PS: thanks for working on this project, it is the best UI for looking at Kafka cluster data I've seen so far.

Hey Alex, thanks for the feedback, glad you like the new release :)
We'll take a look into the problem. We've done a lot of frontend refactoring recently, so this might be the reason.


NeoVim Config: Better git workflow

Decide between neogit and vim fugitive and integrated into config

Added neogit in abf4abb. Don't think it makes a difference.

Removed since 818b351


Make new experiment concerning filtering

Note @yjernite we could also save right before the filtering by size, and then start from there to compute length values to obtain the buckets you requested


No way to resolve dispute if settlement offer is ignored

I disputed a signaling proposal that had been up for a long time with no activity because I felt like it was no longer relevant, proposer did not take action when the proposal was challenged, and I should therefore be able to win the challenge by default but there is not option in the UI to do that:

So it won't raise to Celeste because the Proposal dont did action? It's UI or Gardens vs Celeste design interaction change?


Fix bitbucket templates references

Signed-off-by: Chmouel Boudjnah<EMAIL_ADDRESS>

Codecov Report

Merging #296 (0e3286f) into main (74252f6) will increase coverage by 0.00%.
The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #296   +/-   ##
=======================================
  Coverage   70.60%   70.60%
=======================================
  Files          32       32
  Lines        2109     2116    +7
=======================================
+ Hits         1489     1494    +5
- Misses        483      485    +2
  Partials      137      137

Impacted Files
Coverage Δ

pkg/webvcs/github/acl.go
82.89% <0.00%> (-0.69%)
:arrow_down:

pkg/matcher/annotation_tasks_install.go
89.74% <0.00%> (-0.51%)
:arrow_down:

Continue to review full report at Codecov.

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


Native Image run includes SPI for libraries that are not loaded, leading to crashes

This has been uncovered during work on #11546 where I have tried 'fixing' the issue that Base_Tests used to depend on Standard.Table.
Apparently, when I run Base_Tests under Native Image without the Table import, the FileSystemSPI still sees the registrations for file formats from Standard.Table. However, since the Standard.Table is no longer loaded, trying to instantiate the modules related to these formats fails, causing our tests to fail.
To make the tests pass I've re-introduced the Standard.Table import to Base_Tests, but overall we should get rid of it.
To reproduce the issue, remove the workaround that has been added in #11546 by applying the following patch:
Subject: [PATCH] workaround for https://github.com/enso-org/enso/issues/11707
---
Index: test/Base_Tests/src/System/File_Read_Spec.enso
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/test/Base_Tests/src/System/File_Read_Spec.enso b/test/Base_Tests/src/System/File_Read_Spec.enso
--- a/test/Base_Tests/src/System/File_Read_Spec.enso	(revision a600180b634861d96972366c40c281251ded151b)
+++ b/test/Base_Tests/src/System/File_Read_Spec.enso	(date<PHONE_NUMBER>671)
@@ -106,11 +106,8 @@
             r2.should_be_a Vector
             r2.should_equal [js_as_text.get, "Hello World!"]

-        ## Workaround for https://github.com/enso-org/enso/issues/11707
-           This pending check should be removed once it is fixed.
-        is_table_imported = File_Format.all.map .to_text . contains "Delimited_Format"
-        table_import_pending = if is_table_imported then "Base_Tests should not import Table, but they sometimes do as workaround for #11707. This test can only run if Table is not imported."
-        group_builder.specify "would default to returning as merged Table, but will raise a helpful error if Standard.Table is not loaded" pending=table_import_pending <|
+        group_builder.specify "would default to returning as merged Table, but will raise a helpful error if Standard.Table is not loaded" <|
+            is_table_imported = File_Format.all.map .to_text . contains "Delimited_Format"
             Runtime.assert is_table_imported.not "This test assumes that Base_Tests does not import Standard.Table."
             files = [enso_project.data / "sample.json"]
             r1 = Data.read_many files
Index: test/Base_Tests/src/Main.enso
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/test/Base_Tests/src/Main.enso b/test/Base_Tests/src/Main.enso
--- a/test/Base_Tests/src/Main.enso	(revision a600180b634861d96972366c40c281251ded151b)
+++ b/test/Base_Tests/src/Main.enso	(date<PHONE_NUMBER>460)
@@ -93,10 +93,6 @@

 import project.Random_Spec

-## Workaround for bug https://github.com/enso-org/enso/issues/11707
-   The Standard.Table import should be removed once the bug is fixed.
-import Standard.Table
-

 main filter=Nothing =
     suite = Test.build suite_builder->

Then you need to build the Native Image runner (sbt engine-runner/buildNativeImage) and run it. A small test that you can check out is e.g.:
.\built-distribution\enso-engine-0.0.0-dev-windows-amd64\enso-0.0.0-dev\bin\enso.exe --run .\test\Base_Tests\ -- "JSON Deser"

Note that the issue is further obscured by an UnknownIdentifierException being intercepted by Truffle code and mistakenly wrapping it in another error that suggests that get_module is the missing identifier, whereas in fact the missing identifier is a module that fails to be loaded, e.g. Delimited_Format that comes from Standard.Table.
@JaroslavTulach suggested that we should change the type of exception raised to avoid obscuring the real problem:
Index: engine/runtime/src/main/java/org/enso/interpreter/runtime/scope/TopLevelScope.java
IDEA additional info:
Subsystem: com.intellij.openapi.diff.impl.patch.CharsetEP
<+>UTF-8
===================================================================
diff --git a/engine/runtime/src/main/java/org/enso/interpreter/runtime/scope/TopLevelScope.java b/engine/runtime/src/main/java/org/enso/interpreter/runtime/scope/TopLevelScope.java
--- a/engine/runtime/src/main/java/org/enso/interpreter/runtime/scope/TopLevelScope.java	(revision a600180b634861d96972366c40c281251ded151b)
+++ b/engine/runtime/src/main/java/org/enso/interpreter/runtime/scope/TopLevelScope.java	(date<PHONE_NUMBER>797)
@@ -22,6 +22,7 @@
 import org.enso.interpreter.runtime.builtin.Builtins;
 import org.enso.interpreter.runtime.data.EnsoObject;
 import org.enso.interpreter.runtime.data.vector.ArrayLikeHelpers;
+import org.enso.interpreter.runtime.error.PanicException;
 import org.enso.interpreter.runtime.type.Types;
 import org.enso.pkg.Package;
 import org.enso.pkg.QualifiedName;
@@ -125,12 +126,12 @@
   abstract static class InvokeMember {
     @CompilerDirectives.TruffleBoundary
     private static Module getModule(TopLevelScope scope, Object[] arguments)
-        throws ArityException, UnsupportedTypeException, UnknownIdentifierException {
+        throws ArityException, UnsupportedTypeException {
       String moduleName = Types.extractArguments(arguments, String.class);

       var module = scope.getModule(moduleName);
       if (module.isEmpty()) {
-        throw UnknownIdentifierException.create(moduleName);
+        throw new PanicException(scope.builtins.error().makeModuleDoesNotExistError(moduleName), null);
       }

       return module.get();


build: Upgrade to go-1.23, Linter1.61.0 and Alpine 3.20

PR Checklist
Please check if your PR fulfills the following requirements:

[ ] Tests for the changes have been added (for bug fixes / features)
[ ] Docs have been added / updated (for bug fixes / features)

If your build fails due to your commit message not passing the build checks, please review the guidelines here: https://github.com/edgexfoundry/edgex-examples/blob/master/.github/CONTRIBUTING.md.
What is the current behavior?

Issue Number:
What is the new behavior?
Does this PR introduce a breaking change?

[ ] Yes
[ ] No

New Imports
Are there any new imports or modules? If so, what are they used for and why?
Specific Instructions
Are there any specific instructions or things that should be known prior to reviewing?
Other information

Welcome to Codecov :tada:
Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.
Thanks for integrating Codecov - We've got you covered :open_umbrella:


Corrects spelling of some comments and method names

Corrects spelling of some comments and method names

 Thank you for your submission, we really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.:x: MihaMarkic sign nowYou have signed the CLA already but the status is still pending? Let us recheck it.

MihaMarkic This PR appears to have broken some things. I suspect you fixed a typo in one of the JS interop functions, but didn't correct all of the usages.

@rynowak Yep, my find window had filtering by "*.cs". Found another instance and renamed in UriHelper.ts.

Great! Thanks


Database migrations after the initial migration aren't being generated

The migration files are being generated, but the actual migration code has to be written manually.

Is that why I get PostgreSQLSeverity.fatal 28000: role "dart" does not exist Detail: null Hint: null Table: null Column: null Constraint: null when I do aqueduct db upgrade?
That's when I'm using OSX 10.11.6 and the Postgres.app.
The 00000001_Initial.migration.dart generated by aqueduct db generate is identical to the one generated on another Mac with macOS 10.12.1 where the aqueduct db upgrade works fine.

I don't believe so, it sounds like the dart user hasn't been created in your local database on the other machine.

Thanks, anachlas. The solution was to take the postgres server down, then do aqueduct setup -c and aqueduct db upgrade.

Yes. I reproduced this too.
I created a project with CLI
and aqueduct db generate was ok for the first time
and then i added some field and generate it again
and all i get is...

class Migration2 extends Migration {
  Future upgrade() async {
  }

  Future downgrade() async {
  }
  Future seed() async {
  }
}

This is next on the list. All of the stuff to do it is there, just need to sew it all togethers and write the tests. Estimating it to be about two weeks from now. In the meantime, you can manually write the code in upgrade:
Future upgrade() async {
  database.addColumn("table", new SchemaColumn("fieldName", ManagedPropertyType.String));
}

See the API ref for SchemaBuilder, which is the type of database - a property all Migrations have.
Make sure to run aqueduct db validate. This ensures that all of the migration files will create the schema described by your ManagedObjects; it'll spit out every difference that it finds.

Still waiting for awesome migration feature.

You can get early access to this feature by installing the CLI from the open branch. The branch has a few more tests that need to be written, so it may have errors, but it'd be good to get your feedback regardless.
On your local machine, clone this repository and switch the jc/migrate branch:
git clone https://github.com/stablekernel/aqueduct.git
cd aqueduct
git checkout jc/migrate

Then, while still inside this directory, activate this local copy in pub:
pub global activate -spath .

From there, you should be able to run aqueduct db generate in your project directory.

Work is completed in #234. Waiting for 2.0.3 release.

Completed in 2.0.3.


Add extra options for getting single merge request

Hi!
I just want to add some options to get merge request call.
Its pretty useful when you want to know number of commits behind the target branch.
Tell me what you think about it

Thanks!


[test] Add libVC simd test

Will fix UES-487

Hello @jgphpc, Thank you for submitting the Pull Request!

In the file cscs-checks/microbenchmarks/simd/vc.py:

Line 76:1: W391 blank line at end of file

Do see the ReFrame Coding Style Guide

@jenkins-cscs retest this please
unittests/test_cli.py:156: AssertionError

@jenkins-cscs retry all

Codecov Report

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

@@            Coverage Diff             @@
##           master     #899      +/-   ##
==========================================
+ Coverage   91.71%   91.73%   +0.01%
==========================================
  Files          78       78
  Lines       10339    10339
==========================================
+ Hits         9482     9484       +2
+ Misses        857      855       -2

Impacted Files
Coverage Δ

reframe/core/config.py
85.71% <0%> (+1.78%)
:arrow_up:

Continue to review full report at Codecov.

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

Build "failed" because of unittest failure:
reframe/core/runtime.py:321: in _fn
    ret = fn(*args, **kwargs)
unittests/test_schedulers.py:143: in test_cancel
    self.assertLess(t_job.total_seconds(), 30)
E   AssertionError: 42.954022 not less than 30

@jenkins-cscs retry daint

@jenkins-cscs retry none

@teojgo Can you approve this PR?


Clear input and textarea on setValue

I had similar issues as described in #198, this should fix it and delete every character in the input or textarea.

Send Delete- and Backspace-Keystrokes for every Character
in the Input or Textarea
Previous behaviour which sent all deletions + given value in
one String didn't work

Making several postValue calls (as you did in this PR) would result in several change JS events triggered for that input/textarea. That is what we're trying to avoid by combining input clearing and new value setting into same postValue call.
This would also result in corresponding test, that checks change event being triggered only once, failure.

Previous behaviour which sent all deletions + given value in one String didn't work

Can you please elaborate more on this one? The tests, that checks existing value removal on new value setting pass, so this works as expected (or tests are incomplete).

Can you please elaborate (version of Selenium, Browser, Mink, Driver) more on this one?

Of course:
OS: Mac 10.10.5
Selenium Version: 2.47.1
Browser: Firefox 40.0.3
Mink: 1.7.0
Selenium2-Driver: 1.3.0
I have a form where one Input (type text) is already pre-filled. In my test I am trying to replace the value with a custom one. The driver only seems to send one DELETE-Sequence (or selenium only interprets it as a single one), resulting in the original string being shortened by one from the start. The new value then gets placed beforehand.
Example:
Existing Value: Username150
New Value: foo
Result: foosername150

This would also result in corresponding test, that checks change event being triggered only once, failure.

What's the reason for this expected behavior?

Sounds like a bug in Selenium Server version. Have you tried problem happens also in Chrome? or maybe earlier Firefox version?

Sounds like a bug in Selenium Server version. Have you tried problem happens also in Chrome? or maybe earlier Firefox version?

Okay, thanks for the fast response. I'm heading over to this selenium issue and see if there's more to find out


CoinView: moved evict under lock

Minor perf optimization.
Instead of exiting async lock and then entering it inside EvictAsync I've moved eviction under the lock and changed the method to be EvictLocked.
Also fixed TODO

Looks good but I didn't run tests.


Add an Import Manager

The import manager's job is it to generate the import section automatically.
For example client generator code will call ${importManager.getType("javaslang.collection.List")}, which

returns List and generates an additional import javaslang.collectionList, if no other List is 'registered' yet
return javaslang.collection.List, if another List is registered already (e.g. java.util.List)

Additionally the priority of the current package is higher than others regarding generation of simple names.

TODO:

Use reflection to add java.lang.* to knownSimpleClassNames, in order to prevent classes with same simple names to be imported
Handle static imports, i.e. implement ImportManager.getStatic() to import static methods
Add a wildCardThreshold feature, which defines, when multiple imports from the same package (type) or class (static) are reduced to one import statement using wildcard '*'

The 'Use reflection to add java.lang.* to knownSimpleClassNames, in order to prevent classes with same simple names as java.lang.* to be imported' feature is dropped due to a (cost / benefit analysis](http://stackoverflow.com/questions/520328/can-you-find-all-classes-in-a-package-using-reflection).
The preferred way is to control the import behavior by using the (already existing) knownSimpleClassNames parameter. I.e. set knownSimpleClassNames = List("java.lang.String")if ImportManager.getType("my.package.String") should result in not importing my.package.String and use full qualified references within the code.


SelectField component missing documention for menuItems

This page is missing documentation for menuItems required prop. Not sure how to show text or value.
http://material-ui.com/#/components/select-fields

valueLink and value are also undocumented.

I confirm.
Evrey time I see an issue regaring missing documentation, I just want to right some code to extract it from the source code. I will end up doing it.

The onChange event handler is also documented wrong. It should be handleChange(element, index, object).

So what is the correct way to do it?


cold-induced thermogenesis

This ticket stems from #84 on the https://github.com/geneontology/helpdesk/issues/83#issuecomment-372829184.
The proposal is to create new terms for an annotation project .
The requester suggests that the terms of heat generation and homeothermy do not suffice.
"Cold-induced thermogenesis (CIT) should be a children term of the existing term "adaptive thermogenesis". In GO, adaptive thermogenesis has only one children term "diet-induced thermogenesis". Cold-induced thermogenesis is another important type of adaptive thermogenesis, which is an endothermic thermoregulation process with a basic property of the ability to generate heat by increasing metabolism in response to cold ambient temperatures to maintain a stable core body temperature. ."
Will need to get a decent reference.

@dsptk2017, can you please supply what you would consider a good definition AND reference your your required term. Thanks.

"The process of heat production in an organism in response to cold or low temperature"
?
The process by which exposure to low ambient temperatures, mammals increase oxidative metabolism for the production of heat. PMID: 28988821

@hdrabkin  I think this definition is better . "an endothermic thermoregulation process with a basic property of the ability to generate heat by increasing metabolism in response to cold ambient temperatures to maintain a stable core body temperature." (PMID: 27876809 ).

@dsptk2017
Tweaked a bit to be consistent with other terms
"The process by which heat is generated by increasing metabolism in response to cold ambient temperatures to maintain a stable core body temperature."
PMID: 27876809
is_a adaptive thermogenesis
It also occurs to me: is this process limited to mammals?

synonyms
CIT

@hdrabkin  Yes.  CIT is its synonym. Is adaptive thermogenesis is limited to mammals?

do you mean yes it is limited to mammals (I ask because I may want to put in some taxa restrictions (ie, only in mammals).
GO:0106106
is_a adaptive thermogenesis
also
is_a temperature homeostasis

term added; closed

I mean yes. CIT is its synonym. I not sure whether it is limited to mammals. If its parent term "adaptive thermogenesis" is limited to mammals, CIT should also be limited to mammals.

Ok, I'll leave like it is for now
So you can now annotate to GO:0106106; You may want to contact GOA about uploading/adding your annotations.

This is a deeply naive question, but thermogenesis sounds like it should be restricted by definition to warm-blooded animals, so mammals and birds?

I am not sure. I just know birds have thermogenesis.
On Wed, Mar 21, 2018 at 4:01 PM, deustp01<EMAIL_ADDRESS>wrote:

This is a deeply naive question, but thermogenesis sounds like it should
be restricted by definition to warm-blooded animals, so mammals and birds?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/geneontology/go-ontology/issues/15414#issuecomment-375094119,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Agw7XYSvWmrL1CDczTiH2WlbozCWrXqWks5tgr-fgaJpZM4SqiYm
.

Ok. I will contact GOA. Thank you very much.
On Wed, Mar 21, 2018 at 3:59 PM, Harold Drabkin<EMAIL_ADDRESS>wrote:

Ok, I'll leave like it is for now
So you can now annotate to GO:0106106; You may want to contact GOA about
uploading/adding your annotations.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
https://github.com/geneontology/go-ontology/issues/15414#issuecomment-375093736,
or mute the thread
https://github.com/notifications/unsubscribe-auth/Agw7XVWbOib2HG0XiCMmtBTdLOf1jcvZks5tgr9HgaJpZM4SqiYm
.


Update category.json

Comments out the position field, which was accidentally left enabled.
Harness Developer Pull Request
Thanks for helping us make the Developer Hub better. The PR will be looked at
by the maintainers.
What Type of PR is This?

[ ] Issue
[ ] Feature
[ ] Maintenance/Chore

If tied to an Issue, list the Issue(s) here:

Issue(s)

House Keeping
Some items to keep track of. Screen shots of changes are optional but would help the maintainers review quicker.

[ ] Tested Locally
[ ] Optional Screen Shoot.

Restoring delegate node to previous position and hoping to eliminate a conflict that I'm having trouble finding in so doing.

Preview environment: https://ci.pr.harness.io/pr-771


:wrench: new drop is swirls

Thank you for your contribution to the KodaDot - One Stop Shop for Polkadot NFTs.
👇 __ Let's make a quick check before the contribution.
PR Type

[ ] Bugfix
[x] Feature
[ ] Refactoring

Context
Lauching tomorrow
Did your issue had any of the "$" label on it?
Copilot Summary

🤖[deprecated] Generated by Copilot at 34b2e73
Updated the link to the Swirls collection on the unlockable landing tag component. This is part of a feature branch that adds a new NFT collection to the gallery.

🤖[deprecated] Generated by Copilot at 34b2e73

nuxt-link changes
Swirls replace Pareidoscope
New NFTs for spring

@vikiival ok, i saw that you wrote that is launching tomorrow, but should the generation be working already? because its not
also not on drops page(i know in this pr you just changed the landing link)

but should the generation be working already? because its not

it is disabled by me

because its not
also not on drops page

also will reveal tomorrow


Load only latest 50 messages instead of all

Problem
As this app is growing and is used more, the total number of messages is becoming very large and retrieving them all from firestore will get slow. Also, old messages are not usually relevant, so there's no reason for so much overhead
Solution
Only fetching and showing the latest 50 messages will speed up load times and improve a user's experience, by showing only relevant content

Nice idea but we can fetch the latest 50 messages
But we must also need to find a way to access the previous messages (beyond 50) if user wants
Do try to find a solution for this.

Nice idea but we can fetch the latest 50 messages
But we must also need to find a way to access the previous messages (beyond 50) if user wants
Do try to find a solution for this.

Sure, I'll make a PR for loading latest 50 messages soon, and after that make another PR to add a button at the top of all messages that on being clicked loads older messages


build_meta does not include setup.py

When someone invokes the build operation via the build_meta (via https://github.com/pypa/setuptools/blob/master/setuptools/build_meta.py#L170 - https://www.python.org/dev/peps/pep-0517/#build-sdist) the setup.py is no longer automatically packaged.
In case of python setup.py sdist what ensured that this will happen is https://github.com/python/cpython/blob/master/Lib/distutils/command/sdist.py#L250. Here distribution.script_name is set to setup.py when called via python setup.py sdist. This is not set though when calling it directly via build_meta so packaging setup.py is safely ignored.
Note the files to package is determined while running egg_info, and this is re-used across runs (by reading SOURCES.txt). Therefore this bug is hidden when:

anyone beforehand calls python setup.py sdist,
when there's a distutils extension that automatically pulls in setup.py (such as setuptools-scm),
when some config value pulls in (one in MANIFEST.in, setup.cfg, setup.py).

Working on this based on the PyPA sprint : https://github.com/orgs/pypa/projects/1


[Documentation][Input Validation] Input validation documentation issues

Platform
What platform is your issue or question related to? (Delete other platforms).
Documentation
Details
This issue tracks outward facing documentation for issues related to Input Validation. Specifically, we need to document:

We have introduced a breaking change in which inputs are returned by a submit action in a cards with show cards. Previously all inputs would be returned for all cards, now only those belonging to the card with the submit action and any parent cards will be returned.
Input validation of inputs which can be hidden by an Action.ToggleVisibility is not recommended.
Input validation may not be well supported when the host is using popup show cards rather than the default inline show cards.

@RebeccaAnne assigning this to you for follow up please! thanks


Enable support for ES 5.0.0

This PR enables support for ES 5.0.0
We need to address the following issues:

Percolator is deprecated in 5.0 . Figure out if there are stats for the new "percolate query" to export

getIndexWriterMaxMemoryInBytes not avaivable in 5.0

index. getSuggest() not  avaivable in 5.0

Otherwise everything seems to be working properly

Thank you for your job !

It was you who did the heavy lifting at first place :)


Screen glitches to the right at seemingly random times

Hyprland Version
Hyprland, built from branch  at commit 1b48642fd15c433c53876f1b933dcd46265caf8f dirty (). Tag: v0.30.0  flags: (if any)
Bug or Regression?
Bug
Description
At seemingly random times, the screen glitches for (what I think) 1 frame to the right side, I am attaching a video that I recorded on my phone of the glitch happening.
Also, the glitches always occur only on my laptop screen. If an external screen is connected to the HDMI port, the second screen never glitches.
Laptop model: Lenovo Thinkpad T14s Gen 4 (AMD)
CPU: AMD Ryzen 7 PRO 7840U w/ Radeon 780M Graphics
How to reproduce
For me, the gltches aren't 100% reproducible. However, they occur far more if I am watching something on fullscreen through VLC for example.
Crash reports, logs, images, videos
https://github.com/hyprwm/Hyprland/assets/20685614/680f8cac-e04a-4818-8531-281fdb5c0760

@vaxerski for some reason, the Arch repo has this outdated version... I updated today and the issue persists on hyprland-git. Do you need me to provide specific logs? I can also provide you with a list of exact times (to the second) when the glitch occurred during a session.

I wonder if this isnt a radeon bug. I remember a similiar report from a framework... I think... laptop.

It totally might be...

testing sway-git

did you find a fix?


Log at error if renderer control methods are called before initialise completes

This is a best efforts attempt to warn devs if they are trying to set up a renderer before initialise is called. This is too early and many settings will be lost during initialise.

Actually, there is a better way


Fixes #85, toggling enabled no longer crashes the game

Small fix to the enable toggle crash. Just adds a check for if CustomHud.getActiveProfile() is null, and if it is, returns getGpuUsageAndOtherPerformanceMetrics as false.
Seems to fix the issue.

Closing as I'm applying this fix to all 1.19.2-1.19.4

It has been over a month now, it doesn't seem to be fixed...

@Clozent if the main version is still crashing for you, you can try the version I patched for my modpack. https://github.com/optipack/ported-mods/releases/tag/v1.3.9

I'm sorry about the delay, college and finals were really kicking me.
I'm able to work on the mod again, so there should be a release within the next week


Issue with NcacheQuery in .net core 2.2

Hi folks! NCacheQuery doesn't work correctly in .net core application with NuGet package "Alachisoft.NCache.OpenSource.Linq". If you install this package, Ncache doesn't allow you to use LINQ for queries data in cache because you will get this error:
The type 'Cache' is defined in an assembly that is not referenced. You must add a reference to assembly 'Alachisoft.NCache.Web, Version=<IP_ADDRESS>, Culture=neutral, PublicKeyToken=cff5926ed6a53769'
In a classic .net framework, everything works flawlessly.
The workaround is to add DLL from sample application that going with Ncache: "%NCHOME%\samples\dotnetcore\NCacheLINQ\NCacheLINQ\lib\Alachisoft.NCache.Linq.dll" to project dependency and with that library, everything starts working. Can u fix this, pls?

Test console .net core app:

using System;
using Alachisoft.NCache.Web.Caching;
using Alachisoft.NCache.Linq;
using System.Linq;

namespace ConsoleAppCoreNCache
{
    class Program
    {
        static void Main(string[] args)
        {
            var cache = NCache.InitializeCache("myCache");
            cache.Clear();

            Add(cache);

            var query = new NCacheQuery<TestObject>(cache);

            var results = query.Where(x => x.OddFlag == true);

            foreach(var result in results)
            {
                Console.WriteLine($"Odd number: {result.Number}");
            }

            Console.WriteLine("Press any key...");
            Console.ReadKey();
        }

        static void Add(Cache cache)
        {
            for (int i = 0; i < 20; i++)
            {
                var key = $"{i}";

                cache.Insert(key,
                    new TestObject
                    {
                        Id = key,
                        Number = i,
                        OddFlag = i % 2 == 0 ? false : true
                    });
            }
        }
    }

    [Serializable]
    public class TestObject
    {
        public string Id { get; set; }
        public int Number { get; set; }
        public bool OddFlag { get; set; }
    }
}

config.ncconf:

 <query-class id="ConsoleAppCoreNCache.TestObject" name="ConsoleAppCoreNCache.TestObject">
            <query-attributes id="Id" name="Id" data-type="System.String"/>
            <query-attributes id="Number" name="Number" data-type="System.Int32"/>
            <query-attributes id="OddFlag" name="OddFlag" data-type="System.Boolean"/
</query-class>

Hello Sunsikh,
Thank you for pointing out this issue to us and furnishing your application code so that we can narrow down the root cause, this was very helpful.
I have reproduced the scenario on my end and confirm that there is a version mismatch with NCache LINQ integration assembly that is shipped with NuGet package and this is why it is raising the error. Our engineering has published an updated NuGet Package Alachisoft.NCache.OpenSource.Linq (v <IP_ADDRESS>) with correct version information that should work as expected in your environment.
You can install this Nuget Package from Visual Studio and use it in your application.
Here is the NuGet.org link to the updated NuGet package:
https://www.nuget.org/packages/Alachisoft.NCache.OpenSource.Linq/<IP_ADDRESS>

And it's worked! Thx you very much!

I have noticed that referencing either version of the NCache.Linq packages causes InitializeCache to hang or otherwise fail when attempting to use a "partitioned" cache.  This is reproducible by changing the cache used in the NCacheLINQ sample project from "myCache" to "myPartitionedCache".
I have not found any documentation indicating that NCache Linq can only be used with a "local" cache?
If anyone has a solution for this (using Linq with partitioned cache) or information as to why we cannot use Linq with a partioned cache, kindly let us know.
Thanks.

I have noticed that referencing either version of the NCache.Linq packages causes InitializeCache to hang or otherwise fail when attempting to use a "partitioned" cache. This is reproducible by changing the cache used in the NCacheLINQ sample project from "myCache" to "myPartitionedCache"....

It appears that the NuGet packages for LINQ install local configurations, which was confusing the issue.
So the issue I mention above is probably not an issue.


chore(release): 1.23.0

See CHANGELOG

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject89A8053A-6Jw05QLuWWwe
Commit ID: 338b42ec63e6a6b8a1b1bfe152bfd4d9637d8811
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject89A8053A-6Jw05QLuWWwe
Commit ID: 1a6347bb09f009121e202d7b210431c6703e0dea
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository

AWS CodeBuild CI Report

CodeBuild project: AutoBuildProject89A8053A-6Jw05QLuWWwe
Commit ID: 7ece698a0dcb881f7b5450a90ccc94fbba53efb2
Result: SUCCEEDED
Build Logs (available for 30 days)

Powered by github-codebuild-logs, available on the AWS Serverless Application Repository


[WIP] Tremor unified cli

Fixes #200 and adds a number of test frameworks

[x] exit status (heinz)
[x] generate stdlib docs
[x] updating docker

Updates that could go in 10.1

[ ] generating cli doc (10.1?)
[ ] improve tremor test integration ... so assert pass/fail status is obvious in ci ( no colors means not obvious without text prompt/hint )

Adoption of unified binary and removal of legacy binaries ( CI etc.. )

[x] delete tremor-script binary
[x] delete tremor-query binary
[x] delete tremor-server binary
[x] delete tremor-tool binary
[x] Patch CI to adopt new binary for integration tests
[ ] Patch CI to adopt new binary for api curl based tests
[ ] Patch CI to adopt new binary for api tremor cli based tests
[ ] Extend CI to support tremor stdlib unit style tests
[ ] Extend CI to support tremor cli self hosted tests
[ ] Delete legacy integration tests and replace with new cli based integration tests
[x] Patch cross compiler and packaging for unified cli tools

Code coverage

[ ] Figure out code coverage against unified client

I looked at the heredoc tests and I don't see any good way fo handeling it, if removing identation detection is the cost of adding interpolation lets put in a HUGE note on the release and call it a good?
In the long run we could seperate interpolated and non interpolated strings? something like
"string literal"
`string interpolated`

and pick two of them perhaps? That would also allow us to natively read JSON again since interpolated strings would not be in any json files so "{snot}" read from a JSON file would be preserved.


Support for dynamic revisions different from SBT / ivy

It seems coursier only supports dynamic revisions in major, minor and revision positions of a version, but not anywhere else.
For example 2.5.10-+ is correctly resolvable by vanilla SBT, but coursier spits out:
[error]   com.example:base_2.11:2.5.10-+:
[error]     not found:
[error]       /home/claudio/.ivy2/local/com.example/base_2.11/2.5.10-+/ivys/ivy.xml
[error]       https://jcenter.bintray.com/com/example/base_2.11/2.5.10-+/base_2.11-2.5.10-+.pom
[error]       https://repo1.maven.org/maven2/com/example/base_2.11/2.5.10-+/base_2.11-2.5.10-+.pom
[error]       https://artifactory.redbullmediahouse.com/rb3ea-ivy-virtual/com.example/base_2.11/2.5.10-+/ivys/ivy.xml

For the following minimal build file:
scalaVersion := "2.11.8"
libraryDependencies += "com.example" %% "base" % "2.5.10-+" intransitive()

This is probably related to this, latest.integration (as in addSbtPlugin("organization" % "artifact" % "latest.integration")) doesn't seem to work with coursier.


Payload creation v2

Pull Request Test Coverage Report for Build 906

0 of 0   changed or added relevant lines in 0 files are covered.
81 unchanged lines in 2 files lost coverage.
Overall coverage decreased (-0.1%) to 80.392%

Files with Coverage Reduction
New Missed Lines
%

dist/routes/v2/payloadCreation.js
32
88.79%

dist/routes/v2/address.js
49
75.91%

Totals

Change from base Build 902:
-0.1%

Covered Lines:
2506

Relevant Lines:
2932

💛  - Coveralls


Add "Copy All" menu in programmers mode.

Problem Statement

We need few times select copy if we want copy some values in programmers mode.
Evidence or User Insights

It's hard to copy to clipboard many times when we want copy some values.
Proposal

Add "Copy All" menu that copy all value(hex, dec, oct, bin) to clipboard.
Goals

Copy all values to clipboard in order as follow:

hex
dec
oct
bin

Low-Fidelity Concept

Requested Assignment

If possible, I would like to implement this.

This is your friendly Microsoft Issue Bot. I've seen this issue come in and have gone to tell a human about it.


Erroneous certificate validation error

Terraform is producing errors about certificate validation records which look like:
│ Error: 1 error occurred:
│ 	* missing login.scimma.org DNS validation record: _33408912c503b310e6cf65fded7bff5b.login.scimma.org
│   with module.keycloak.aws_acm_certificate_validation.validation,
│   on .terraform/modules/keycloak/main.tf line 133, in resource "aws_acm_certificate_validation" "validation":
│  133: resource "aws_acm_certificate_validation" "validation" {

even when the record in question exists (and in fact the certificate has already been issued):
$ dig _33408912c503b310e6cf65fded7bff5b.login.scimma.org
_33408912c503b310e6cf65fded7bff5b.login.scimma.org. 3292 IN CNAME _6c67972ded5b4fe51e07595bfdcc29df.gfjzbhpkrq.acm-validations.aws.

This noise makes it hard to use terraform and we should find a way to clean it up.

This problem has a couple of parts: The first is that the cert_validation aws_route53_record is being created for each domain name to be validated but using the same zone_id for all, which will not work when the records need to belong to different zones. This problem should be fixed in this code. Doing so is necessary, but not sufficient to solve the overall problem, however, because in the case where this is coming up, the second zone needed is scimma.org, which is not managed through Route53/terraform.


Is ncm2 compatible with ncm-R?

Look forward to having auto completion by ncm-R and ncm2. Just wondering is there any test done for this?

I took me some time, but I'm working on it :fireworks: . Please give your feedback over at https://github.com/gaalcaras/ncm-R/issues/13

Thanks for your time! Will continue the discussion if necessary at #13.


Add more documentation regarding making custom components

Is your feature request related to a problem? Please describe.
Currently, there is no documentation on how to create a custom processor or exporter. The only way to understand how to do so is by looking a other components' source code.
Describe the solution you'd like
More documentation on custom processors and exporters, most likely in the form of a tutorial like for receivers
Describe alternatives you've considered
Reading source code

@codeboten transfered
@Annosha nobody is working on it right now, so you can give it a try, here are existing documentations for a receiver and a connector that you can use as a starting point:

https://opentelemetry.io/docs/collector/building/receiver/
https://opentelemetry.io/docs/collector/building/connector/


Default styling + bootstrap leaves unselected options invisible

Out of the box, with bootstrap3 and no other styles, the items in the dropdown that are not highlighted are invisible for me.  I'm assuming white-on-white, but I can't really tell because when I'm using the chrome inspector, the act of trying to inspect them makes them disappear from the DOM.  I tried overriding the styles of .-autocomplete-item so the default color is black, but that didn't seem to help.  I feel like I'm missing something and would appreciate any pointers in the right direction.
Would be great if the README had some tips on styling.

Bootstrap's label class, among others, applies this "bold white" text style. You can fix this by looking at your DOM tree and seeing what parent elements have classes that activate this, and taking the autocomplete box out of that element.
You can also debug this by pausing the browser using a keyboard shortcut.

Perfect - thank you! The debug tip is extremely helpful.


Expose Cofacts API on Cofacts LINE bot GraphQL

Related: https://g0v.hackmd.io/BBeVH2VxRNOCq5QB_HC6EQ#Devs---Q2
This enables LIFFs like viewed article list, upvote, downvote and reply request reasons to invoke Cofacts API directly.

Cofacts API graphQL fields are available with CofactsAPI prefix.

executor is designed to read authorization header and passes userId of this LINE bot user to Cofacts API.

add article field userArticleLink object type, which will map to CofactsAPIArticle

schema stitching mechanism will invoke Cofacts API executor and perform GraphQL call and unidirectional type merging

npm run cofactsapi will load SDL from Cofacts API to data/

Pull Request Test Coverage Report for Build 1302

9 of 10 (90.0%)  changed or added relevant lines in 4 files are covered.
No unchanged relevant lines lost coverage.
Overall coverage decreased (-0.1%) to 86.712%

Changes Missing Coverage
Covered Lines
Changed/Added Lines
%

src/graphql/cofactsSchemaExecutor.js
1
2
50.0%

Totals

Change from base Build 1297:
-0.1%

Covered Lines:
934

Relevant Lines:
1062

💛  - Coveralls

Merge as opened for 10+ days


can not get tagslam pose

Hi, i used my owndata(rosbag) to run tagslam,  "the subgraph err :" always cout "0", and can not publish odomtry! Is there anything wrong ?  It is very confused !!!

Can you please provide more information to go by? The beginning of the tagslam console output, your tagslam launch files, your tagslam.yaml, cameras.yaml, camera_poses.yaml, maybe a rosbag info <name_of_rosbag>?

Can you please provide more information to go by? The beginning of the tagslam console output, your tagslam launch files, your tagslam.yaml, cameras.yaml, camera_poses.yaml, maybe a rosbag info <name_of_rosbag>?

The rosbag info is:

The tagslam launch file is :

The cameras.yaml is :

The camera_poses.yaml and the tagslam.yaml is same as example file.
The output is :

i want get camera pose, but it is not optimize and can not publish odomtry!

Can you please provide more information to go by? The beginning of the tagslam console output, your tagslam launch files, your tagslam.yaml, cameras.yaml, camera_poses.yaml, maybe a rosbag info <name_of_rosbag>?

hi , i have find the problem, thank you very much, you can close this issue


Use a single network connection for reading and writing

Might have to change this to no to fix the codecov/patch:
https://github.com/perlin-network/noise/blob/master/.codecov.yml#L25

can't they still do that in the old code? all they need to do is change Sender in the PrepareMessage?

In the old code, the remote identity (at least the remote address) is verified by always actively connecting for the write channel.


Logo and style changes

@MariaSolovyeva, I am merging this and giving you a login to play on the live site.


Rotating images, click to embiggen

The size of the rotating images makes it hard to see some of the detail. We don't necessarily want to make them bigger on the front page, but we want to make them 'clickable' and to open up a bigger version of the image.
We don't simply want to link to a large version of the infographic poster because eventually we will want to add in additional rotating images.

Sometimes we might want to go to a bigger version of the image, but the current ones are really only "decoration"


StackTrace Not Working

Stack trace is not working .
I am using the following code to fetch stack trace:
string trace = engine.GetStackTrace();
It always returns blank  value.
Any help would be higly apppreciated.
Regards

Hi @jaskaran0001,
If you're using JScript or VBScript, GetStackTrace always returns an empty string unless you pass WindowsScriptEngineFlags.EnableDebugging to the engine constructor. This is a limitation of Windows script engines.
Good luck!

i am using the following code to create vbscript engine.
 engine = new VBScriptEngine(WindowsScriptEngineFlags.EnableDebugging);
It is still returning empty string for GetStackTrace. Is there anything else that i need to be careful about ?

Hi @jaskaran0001,
A stack trace only exists when the engine is executing script code, so calling this method only makes sense in a host method invoked by script code.
Here's a complete sample program that demonstrates GetStackTrace usage:
using System;
using Microsoft.ClearScript;
using Microsoft.ClearScript.Windows;
namespace Test {
    public class Host {
        public ScriptEngine Engine;
        public void DoSomething() {
            Console.WriteLine(Engine.GetStackTrace());
        }
    }
    internal static class Program {
        public static void Main() {
            using (var engine = new VBScriptEngine(WindowsScriptEngineFlags.EnableDebugging)) {
                engine.Script.host = new Host { Engine = engine };
                engine.Execute("host.DoSomething");
            }
        }
    }
}

Are you calling GetStackTrace in some other scenario?


bug: positional offsets in some joins are wrong

This is showing up (at least) in TPC-H Query 03 where we are translating a join incorrectly to join on a str to an int32, which is causing an attempted CAST that will definitely not work.
Going to bisect and see if I can find the culprit

Fixed in #316


[BUG] Prometheus Monitoring is showing "Page not found 404 Error" in cluster home

Rancher Server Setup

Rancher version: 2.7.1
Installation option (Docker install/Helm Chart): Helm

If Helm Chart, Kubernetes Cluster and version (RKE1, RKE2, k3s, EKS, etc): RKE1

Proxy/Cert Details:

Information about the Cluster

Kubernetes version: 1.20.15
Cluster Type (Local/Downstream): Downstream

If downstream, what type of cluster? (Custom/Imported or specify provider for Hosted/Infrastructure Provider): Custom

User Information

What is the role of the user logged in? (Admin/Cluster Owner/Cluster Member/Project Owner/Project Member/Custom)

If custom, define the set of permissions: Admin

Describe the bug

Cluster home with Monitoring installed is showing "Page not found 404 Error".
To Reproduce

Installed Prometheus Monitoring app from Apps > Charts > Monitoring. After everything is installed, navigate to the Cluster home. Scroll to the bottom and the Cluster Metrics, Kubernetes Component Metrics and Etcd Metrics should now be displayed. However, it is showing "Page not found 404 Error".
Result
No Grafana graphs are shown, only "Page not found 404 Error".
Expected Result

The respective Grafana graphs should be displayed
Screenshots

Additional context

Same on Rancher 2.6.11, monitoring 100.2.0+up40.1.2

Might be related to https://github.com/rancher/charts/issues/2445

Looks like this PR would fix it for me, https://github.com/rancher/prometheus-federator/pull/65


Cmp instead of idb

This PR aims to fix issue #2064
What I've changed is basically the comparison function, from idb.cmp() to cmp() defined at src/functions/cmp.ts

Thanks for the fix. I think this was the last place where we are using idb.cmp. If you have time, could you check if we could also get rid of the line const idb = this.db._deps.indexedDB;. I believe it's not needed anymore.

I've checked it out and it's not being used anywhere else in table.ts, so removed it


Unknown type error with Interface

Getting below error while implementing interfaces
Unknown type "CarVehicle". Did you mean "Vehicle", "OldVehicle", or "CarData"?
I think the document misses a full example. There is only how to define an interface and how to use interface in types, but how to query using this is not mentioned anywhere in docs. Also resolve functions not used in example.
QueryType
'verticalData' => [
    'type' => InterfaceTypes::vehicleInterface(),
    'description' => 'Return vehicle data based on Reg Number',
    'args' => [
        'id' => Types::id(),
    ],
    'resolve' => function ($root, $args, $c) {
        return $c['Actions']->get('VehicleAction')->getVehicleById($args['id']);
    }
]

VehicleInterfaceType
[
    'name' => 'VehicleInterface',
    'description' => 'Vehicle data',
    'fields' => function () {
        return [
            'id' => Types::id(),
            'make' => Types::string(),
        ];
    },
    'resolveType' => function ($value) {
        if($value->getVehicleVertical() == 'bike') {
            return Types::bikeVehicle();
        }

        return Types::carVehicle();
    }
];

CarVehicleType
[
    'name' => 'CarVehicle',
    'description' => 'Car Information from prefill api data',
    'fields' => function () {
        return [
            'vehicleType' => Types::id(),
            'make' => Types::string(),
       ];
    },
    'interfaces' => [
        InterfaceTypes::vehicleInterface()
    ]
];

I am running below query
query{
  verticalData(id: 199){
    make
    ... on CarVehicle {
      vehicleType
    }
  }
}

CarVehicle type is not recognized
If I remove inline fragment from query like this
query{
  verticalData(id: 199){
    make
  }
}

Than it gives an error
Schema must contain unique named types but contains multiple types named "CarVehicle". Make sure that resolveType function of abstract type "VehicleInterface" returns the same type instance as referenced anywhere else within the schema (see http://webonyx.github.io/graphql-php/type-system/#type-registry).

Type registry class which returns only single instance of CarVehicle
use Type\CarVehicleType;
use Type\VerticalDataType;

class Types
{
    // Object types
    private static $carVehicle;
    private static $verticalData;

     public static function carVehicle(): CarVehicleType
    {
        return self::$carVehicle ?: (self::$carVehicle = new CarVehicleType());
    }
    public static function verticalData(): VerticalDataType
    {
        return self::$verticalData ?: (self::$verticalData = new VerticalDataType());
    }
}

This situation is possible when the object type is never referenced in fields directly but is still a part of a schema because it implements an interface.
You have to pass such "orphan" types via option types when constructing the schema. See details about this option in docs.
Alternative workaround is to add a dummy field of this object type somewhere in the schema.


Could not register accessory, skipping

[8/21/2020, 10:10:41 AM] [Elgato Key Lights] Discovered accessory on network: Elgato Key Light Air ****
...
[8/21/2020, 10:12:53 AM] [Elgato Key Lights] Could not register accessory, skipping Elgato Key Light Air ****

Not sure if this might be because I had another key light plugin, nextcart’s, installed? Uninstalled before adding yours. That one connected and worked somewhat reliably.

I got this same error when setting up my lights. One caveat for my setup is that I have a separate subnet for "smart devices" that are isolated on their own VLAN. I allow them access to the internet, and I allow my private VLAN access to open connections to the IOT VLAN, but not the reverse (so the devices can't see any other devices on the local network).
Anyway, I couldn't connect to them via hostname, but checking the "Use IP" setting in Advanced worked for me. Just in case others have a similar setup.

Hi,
I've just had the same issue, same error. Maybe put power cycle and restart Homebridge as fix on your readme?
FYI: One thing for me is that when I first got the Key Light, I called it Desk Left. When I upgraded the firmware (a few weeks ago), I changed the name to Desk. You still find it as Desk Left and then rename it - which is great. Just thought I'd let you know.
Holly

Yes, unfortunately these lights are sometimes unresponsive for some reason, and the official Elgato App doesn't work either in that case.
Adding a note to the Readme is a good idea, will do.

Hi there,
I have the same behaviour, systematically:
[4/23/2021, 9:50:45 PM] [Elgato Key Lights] Discovered accessory on network: Elgato Key Light Air 2B20
[4/23/2021, 9:50:45 PM] [Elgato Key Lights] Discovered accessory on network: Elgato Key Light Air 2997
[4/23/2021, 9:50:45 PM] [Elgato Key Lights] Could not register accessory, skipping Elgato Key Light Air 2B20
[4/23/2021, 9:50:45 PM] [Elgato Key Lights] Could not register accessory, skipping Elgato Key Light Air 2997
I tried to switch off and on the Key Lights, then restarting Homebridge, or uninstalling and reinstalling the plugin, ticking "Use IP address instead of hostname" in the Advanced setting, or creating a virtual dedicated bridge with this plugin: nothing works.
Any help there?


Using lures?

I am following a tutorial on setting up evilginx2, but it uses the old phishlets get-url method. What would be the equivalent using lures of "phishlets get-url reddit https://reddit.com/"?

@SamGurdus1 lures edit redirect_url <id> <redirect_url>. To find the ID of your lure just type lures. help lures is your friend, although I thank @ewhitman1 for the link, which I never would have found since it's not in the README or wiki. Incidentally, the get-url command is also not in the help lures output, only in the blog post, but that seems to be the only one missing from help (see core/terminal.go)

hi need some help please .. after getting the lures, lures get-url 0 .. the address (URL) generated not displaying any page on browser .. not sure if i have done this correctly?

Please make sure your DNS and nameserver settings are set up correctly. Not enough information.

hi got this to work after editing the etc/hosts file. Web page can now be displayed.. but website/phishlet twitter not displaying correctly like it should.

Hi,
Anyone worked on changing the landing page for 5 seconds and then the main phishlet URL to open.
My phishlet is working fine, but now I want to load two custom images first and then the actual URL.


Redesign custom server page

Description:

Currently we allow to choose multiple targets for relaying data. Let's simplify the custom server page to reflect this. Notice that when Ruuvi Cloud is selected HTTP(S) option will be greyed out.

EN: You're able to relay data to multiple target. If Ruuvi Cloud is selected you won't be able to select HTTP(S) target simultaneously.
FI: Voit lähettää tiedot useampaan kohteeseen. Et voi valita HTTP(S) kohdetta mikäli Ruuvi Cloud on valittuna.

Rename and reorganise settings for statistics. Notice that you can only choose one option using radio buttons.

EN:
Title: Statistics
Radio Buttons: Send statistics to Ruuvi (default), Send to custom server, Don't send statistics
FI:
Title: Statistiikka
Radio buttons: Lähetä statistiikka Ruuville (oletus), Lähetä omalle palvelimelle, Älä lähetä statistiikkaa

Please remove the default server texts from the descriptions and text fields.

Related issue https://github.com/ruuvi/ruuvi.gwui.html/issues/174 closed

Build 771 has typo on EN page.
Should be "targets".

QA passed


[Theme Switching] Re-structure theme switching and styles components

Clean up our folder organization after theme-switching migration

@chrispader

Closing as completed! Reviewed internallt


Remove version info from Werkzeug in the docs

The following version information is coming from the Werkzeug docs: "New in version 0.6: The location can now be a unicode string that is encoded using the iri_to_uri() function." See http://flask.pocoo.org/docs/0.11/api/#flask.redirect.
There might be more of this kind of issue in the documentation. For this particular one, you can trace the code to docs/api.rst,:
.. autofunction:: redirect

I've started investigating this. It's going to be messy, but I think it's possible.
The sphinx.domains.changeset.VersionChange directive class controls the version messages. Unfortunately, it's not designed to be overridden, the sphinx.domains.changeset.versionlabels dict  has strings that expect one %s substitution only, such as 'Changed in version %s'. This substitution is done a few lines in to the VersionChange.run method. We'll have to copy and modify the code for that method.
Once we do that, we can look up VersionChange.source in intersphinx. If it's not there, we can fall back to the source package name instead of the intersphinx title. If there's no intersphinx method for looking it up, we can fall back to scanning VersionChange.env.intersphinx_inventory.
Still need to investigate how we could determine if the source is the current package being documented, or an inherited method. That way we could exclude the package name if it's the current package. I think I'd actually prefer everything to have package names consistently though.


Github Repository for Version 1 of Moose2Model will not be used anymore

The development will move to version 2 of Moose2Model. This version 1 will only be maintained on Smalltalkhub. See wiki how to get the code. Or install into Moose 6.1 using:
Gofer new
smalltalkhubUser: 'RainerWinkler' project: 'RW-Moose-Diagram';
package: 'RWMooseDiagram';
load.

This will not be done. The development will proceed in Version 1 of the repository.


Update Hugo: security fix

See https://twitter.com/GoHugoIO/status/1340339808716075015?s=19
Help wanted (:

I can take this one.


patch pkgconfig files on mac, libtool control files on linux and mac

This resolves #265.

There are additional locations in the package config files that need patching as well.  And the lib/*.la files need patching as well.

Thank you.

v1.2.0b4 with the fix out.


something wrong , setting polygon extrudedHeight and perPositionHeight

var orangePolygon = viewer.entities.add({
name : 'Orange polygon with per-position heights and outline',
polygon : {
hierarchy : Cesium.Cartesian3.fromDegreesArrayHeights([-108.0, 25.0, 300000,
-104.0, 25.0, 200000,
-100.0, 25.0, 300000,
-100.0, 30.0, 300000,
-104.0, 30.0, 200000,
-108.0, 30.0, 300000]),
extrudedHeight: 0,
perPositionHeight : true,
material : Cesium.Color.ORANGE.withAlpha(0.5),
outline : true,
outlineColor : Cesium.Color.BLACK
}
});
Sandcastle example:

Thanks @xxbld, this is a known issue.  I've added a comment to https://github.com/CesiumGS/cesium/issues/2924 so we can notify you when we have a solution.


Can't resolve path contain dot

This line use require('./Root.prod') to load module, linker can't properly resolve module path.

Mhh that's an edge case, because I use path.extname to determine if there is a file extension present. I'll have a look on it later this week. If you are interested in sending a fix, this would be the right place to start debugging it. https://github.com/octo-linker/core/blob/e970d1c655c6bb82d953c6679c6ac316338f72c2/lib/live-resolver/index.js#L87

Fixed by @josephfrazier with https://github.com/OctoLinker/browser-extension/issues/271


Call onload handler once surrogate is called

Task/Issue URL:  https://app.asana.com/0/1200277586140538/1200918044697295/f
Tech Design URL:
CC:
Description:
Moved from: https://github.com/more-duckduckgo-org/macos-browser/pull/316
Depends on: https://github.com/duckduckgo/BrowserServicesKit/pull/36
Steps to test this PR:

Open dwell.com

Page should render without full page DOM blank screen.

Open https://privacy-test-pages.glitch.me/privacy-protections/surrogates/

Notice that main and iframe load tests now work.

Internal references:
Software Engineering Expectations
Technical Design Template

Rebased after merging: https://github.com/duckduckgo/BrowserServicesKit/pull/36


Bug: graphs can only be nested once

Currently, the serialization format supports subgraphs in the form of graphs property. This only works at the depth of one: there is no way to address subgraphs that are nested more deeply, since an absolute id (the hashtag of the URL) is used.

One approach here might be re-bundling: when about to add another layer of subgraphs, flatten all other ones (perhaps with some prefix), so that there's only one layer still.
Feels like a hack


Tooling for normalizing addresses

Hello, OA!
I have use case at the moment (analyzing broadband coverage in the U.S.) where OA addresses would make a great starting point, but I need to get them closer to complete and normalized postal addresses to get much further. It's OK if the normalized form is a best guess and not perfect, but the idea is that the concatenated OA address (say 123 Main Street 92240) should be fleshed out to a more canonical USPS one like 123 MAIN ST, DESERT HOT SPRINGS, CA 92240.
I've been thinking about how to backfill missing fields in my copy of the OA data (I've noticed a number of data sources understandably don't include their own city or state) and some of the challenges/pitfalls there. Before I go much deeper, I though I'd check in here and see if anyone else has taken a stab at this. I've dug through old issues and am not seeing a lot of prior art, but would greatly appreciate a 👍 / 👎 if I'm reinventing the wheel on this.
Thanks everyone!

You might look at what the Pelias project does with OpenAddresses data in https://github.com/pelias/openaddresses.

That's a great pointer, thanks @iandees!


[BEAM-8889] Upgrade GCSIO to 2.1.6

This is required to address null-pointer issue happening with 2.1.5 of gcsio. GCSIO 2.1.6 has a fix.
R: @kennknowles

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

[ ] Choose reviewer(s) and mention them in a comment (R: @username).
[ ] Format the pull request title like [BEAM-XXX] Fixes bug in ApproximateQuantiles, where you replace BEAM-XXX with the appropriate JIRA issue, if applicable. This will automatically link the pull request to the issue.
[ ] 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.

Linkage check is blocked by BEAM-11250.

You can pass -PskipCheckerFramework as a workaround. So far I have not reproduced the problem on the bug.

But I would mention that is a real error. It should be suppressed at the class level until it can be fixed.

I can managed to run the test without errors by adding -PskipCheckerFramework option to beam-linkage-check.sh
$ sdks/java/build-tools/beam-linkage-check.sh beam-sdks-java-io-google-cloud-platform,beam-runners-google-cloud-dataflow-java
No new linkage errors

I fixed it in https://github.com/apache/beam/pull/13342. With those changes I could run:
/bin/bash sdks/java/build-tools/beam-linkage-check.sh github/master github/pr/13311 beam-sdks-java-io-google-cloud-platform

Confirmed that there are no new linkage errors.

Run Java PreCommit

run java postcommit

Is there a way to reproduce the NPE in a Beam test?

@kennknowles I found that issue when running the wordcount example on dataflow with --experiments=use_grpc_for_gcs option.

OK then I am happy to merge. It is an experiment and there are no linkage errors and existing tests pass.


Question about bounding box format

Thanks for the amazing work! However, I got a little confused when going through the code.
In function loss_boxes, you said that the target boxes are expected in format (center_x, center_y, h, w), but in function ConvertCocoPolysToMask the following computation is done on the boxes:
boxes[:, 2:] += boxes[:, :2]  boxes[:, 0::2].clamp_(min=0, max=w)  boxes[:, 1::2].clamp_(min=0, max=h)
This means that target['boxes'] is in format (x1, y1, x2, y2), which is contradicts with (center_x, center_y, h, w).
May I ask in which function are target['boxes'] converted to format (center_x, center_y, h, w) or whether I have missed something?
Thanks a lot!!

Hi @Dou-Yiming
The targets are converted to CxCyWH at the end of the preprocessing transformations, namely here: https://github.com/ashkamath/mdetr/blob/bf09d98b0b41cd615185dcb0082299a5ba24c319/datasets/transforms.py#L267
We first convert to XYXY (as you noted) because the math for the preprocessing (eg cropping) is easier, but then go back to CxCyWH. Sorry for the confusion.
Hope this helps

Hi @Dou-Yiming
The targets are converted to CxCyWH at the end of the preprocessing transformations, namely here:
https://github.com/ashkamath/mdetr/blob/bf09d98b0b41cd615185dcb0082299a5ba24c319/datasets/transforms.py#L267
We first convert to XYXY (as you noted) because the math for the preprocessing (eg cropping) is easier, but then go back to CxCyWH. Sorry for the confusion.
Hope this helps

Thanks for the rapid reply, the question is solved!


can i use two different body parsers based on a route?

For example for route1 I want to use bodyParser() and for route2 I want to use betterBodyparser().
Is that possible?
The reason why is I am working with an API I don't control and I need the rawBody as a buffer on one of my endpoints, and bodyParser isn't working properly, but there are some others that do.

Yes, you can.
If you're using a router such as koa-router you could do something like (off the top of my head):
router1.use(bodyParser())
router1.use(mw)
router2.use(betterBodyParser())
router2.use(mw)

app.use(router1.routes())
app.use(router2.routes())

If not then compose yourself, e.g.:
const router = () => {
  const routes1 = compose([bodyParser(), ...mw])
  const routes2 = compose([betterBodyParser(), ...mw])
  return async (ctx, next) => {
    if (/* some path check */) await routes1(ctx, next)
    else await routes2(ctx, next)
    await next()
  }
}


wrong version from --version

The bundle for the latest version still has the previous version in it.
$ npx --package @sladg/nextjs-lambda cli --version                                                         1640ms
Need to install the following packages:
<EMAIL_ADDRESS>Ok to proceed? (y) y
7.0.3

See:
20657   │ // package.json
20658   │ var package_default = {
20659   │   name: "@sladg/nextjs-lambda",
20660   │   version: "7.0.3",

This is connected with master being released instead of release branch. Let me fix that pipeline.


netbird: 0.8.11 -> 0.9.1

Description of changes
https://github.com/netbirdio/netbird/releases/tag/v0.9.1
https://github.com/netbirdio/netbird/compare/v0.8.11...v0.9.1

Things done

Built on platform(s)

[X] x86_64-linux
[X] aarch64-linux
[X] x86_64-darwin
[X] aarch64-darwin

[ ] For non-Linux: Is sandbox = true set in nix.conf? (See Nix manual)
[X] Tested, as applicable:

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

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

[ ] (Package updates) Added a release notes entry if the change is major or breaking
[ ] (Module updates) Added a release notes entry if the change is significant
[ ] (Module addition) Added a release notes entry if adding a new NixOS module
[ ] (Release notes changes) Ran nixos/doc/manual/md-to-db.sh to update generated release notes

[X] Fits CONTRIBUTING.md.

@ofborg build netbird netbird-ui

@ofborg build netbird netbird-ui


Tests for #577

test long lines in create-archive
Related #577

pycharm is evil, the tests will pass this time


Gas Canisters / Tanks can't be picked from AE2 Terminal

Maybe correlated to the issue where shiftclicking space suits leads to a desync sometimes

This seems like a CCA issue, but I don't know anything else other than that it happens for now.


[DNM] ext: nffs: update for synch witch upstream

Origin: Apache Mynewt NFFS
URL: https://github.com/apache/mynewt-nffs/tree/master
Commit: ?
Maintaned-by: External
This patch synchronize code with upstream.
Upsteram was aligned in order to fit to Mynewt as well,
so this patch doesn't change zephyr functionality.
Signed-off-by: Andrzej Puzdrowski<EMAIL_ADDRESS>

I will take a look at moving those OS specific things to os.h and remove much of this #ifdefery...

@nvlsianpu The relevant PR is here: https://github.com/apache/mynewt-nffs/pull/14


🪡 Various RLOO fixes

What does this PR do?

Remove vf_loss_stats (not used)
-Remove unnecessary global step increment (same as #1950)

Before submitting

[ ] This PR fixes a typo or improves the docs (you can dismiss the other checks if that's the case).
[ ] Did you read the contributor guideline,
Pull Request section?
[ ] Was this discussed/approved via a GitHub issue? Please add a link
to it if that's the case.
[ ] Did you make sure to update the documentation with your changes? Here are the
documentation guidelines.
[ ] Did you write any new necessary tests?

Who can review?
Anyone in the community is free to review the PR once the tests have passed. Feel free to tag
members/contributors who may be interested in your PR.

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.


Apply Xcode recommended settings.

Fixes warnings described in https://github.com/shpakovski/MASShortcut/issues/154#issuecomment-781726519

@shpakovski Hi, can you review?

Hi @nevack! Sorry for the slow reply 🙏


Flyout: shift caret left/right towards flyout to align with rounded corners

Bug: There is a gap between flyout and caret when flyout has one line of text.
Fixed by shifting the caret inwards towards the flyout 2 pixels when caret is on the left/right.
2 callouts about this:

One downside to this solution: When the flyout is white, the stroke cuts into the flyout (fig 3)
The caret is less visible by 2 pixels, if we stick with this solution and this is undesirable, I can make the caret larger.
This issue does not exist if the contained box has paddingY > 3.  However there are many cases on web where paddingY = 3

alternative solutions

Increasing the minimum height of the flyout.  However this would cause any existing one line flyouts on web containing a Box element with a margin of 3 or less to be misaligned vertically.
Make the corners less round.  Corners are more rounded for Lego
Make the caret much smaller or less obtuse

Moved the background color to the element in front of the caret.  The shadow is still layered behind the caret and the flyout is layered over the caret.
Please comment any suggestions on these solutions/drawbacks.

[x] Documentation
[x] Tests
[ ] Experimental evidence (required for Masonry changes)
[x] Accessibility checkup

Updated 5/12:
Before:

After:

@ldubberstein Thank you for this PR and mentioning the trade-offs! The stroke on the caret keeps causing issues (e.g. #837). Could we remove the stroke and use drop-shadow instead of box-shadow?
With these changes, please ensure Annie Teng signs off from the design side.
packages/gestalt/src/Contents.css
(uses @support for to make sure we still have a good fallback in IE11)
+@supports (filter: drop-shadow(0 0 8px rgba(0, 0, 0, 0.1))) {
+  .contents {
+    box-shadow: none;
+    filter: drop-shadow(0 0 8px rgba(0, 0, 0, 0.1));
+  }
+}

packages/gestalt/src/Contents.js
     // Needed to prevent UI thrashing
     const visibility = mainDir === null ? 'hidden' : 'visible';
     const background = `${bgColor}Bg`;
-    const stroke = bgColor === 'white' ? '#efefef' : null;

     return (
-      <div
-        className={styles.container}
-        style={{ stroke, visibility, ...flyoutOffset }}
-      >
+      <div className={styles.container} style={{ visibility, ...flyoutOffset }}>
         <div
           className={classnames(
             border && styles.border,

filter: drop-shadow

box-shadow

@christianvuerings I tried removing the stroke as mentioned, but the shadow would still be layered above the flyout.  Since the shadow would need to be applied on the svg, it does not seem possible to control the caret shadow with css.
Instead I found a fix by bringing the background color to the inner-contents div and layering it over the caret.  The design now looks the same as originally intended without the bug.


An Error ERR_UNHANDLED_REJECTION

API Platform version(2.7) affected: x.y.z
"vuetify": "^2.4.0","vuex-map-fields": "^1.4.1"
generate-api-platform-client 0.7.3
Description

When I run npx @api-platform/client-generator -g vuetify https://demo.api-platform.com src/  ,it can generator part of resoures ,but  also return an error
Additional Context

node:internal/process/promises:246
          triggerUncaughtException(err, true /* fromPromise */);

UnhandledPromiseRejection: This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason "#<Object>".

Have you solved your issue ?

API Platform version(2.7) affected: x.y.z "vuetify": "^2.4.0","vuex-map-fields": "^1.4.1" generate-api-platform-client 0.7.3 Description
When I run npx @api-platform/client-generator -g vuetify https://demo.api-platform.com src/  ,it can generator part of resoures ,but also return an error
Additional Context
node:internal/process/promises:246
          triggerUncaughtException(err, true /* fromPromise */);

UnhandledPromiseRejection: This error originated either by throwing inside of an async function without a catch block, or by rejecting a promise which was not handled with .catch(). The promise rejected with the reason "#<Object>".

I cannot reproduce it in the main branch:


Can we change the password policy in existing deployment?

Hello
As  I know, the  default password policy for sign up is "Minimum 8 characters. At least one lowercase, one uppercase and one symbol".
Can we change that policy on deploying or after deployment?
Thanks in advance.

I believe we have configuration in place for password policy. You can also check standard edx configurations to change password policy. Yes, you can change the policy manually after deployment. I don't believe we made that as a pre-deployment configuration option.


How to change font in DialogList?

I want to change the font on Dialog in DialogList. I am unable to find it in the docs.
Kindly help me implement this.

This is RecycleView - use it docs

@mayuroks Are you able to font family ?? Let me know, I am having the same issue.

@mayuroks Are you able to font family ?? Let me know, I am having the same issue.


Update the default room version to 9

Spec PR for MSC3589
Based on https://github.com/matrix-org/matrix-doc/pull/2803#issue-715183175 I haven't included a changelog but let me know if one is required.

I'm assuming it would be a feature?

Yup, thank you!


feat: connection gater

https://github.com/libp2p/go-libp2p-core/blob/master/connmgr/gater.go

https://github.com/libp2p/js-libp2p/issues/175
https://github.com/libp2p/js-libp2p/issues/744
https://github.com/libp2p/js-libp2p/issues/769

@vasco-santos : is this desirable to land soon, and is this a PR you can look at, or is there someone else we can delegate to?

Too many conflicts from master, may postpone until next month.

Continued in https://github.com/libp2p/js-libp2p/pull/1142 since this PR does not allow maintainer commits.


missing break in switch-case in config.cpp?

Since in the CMakeLists.txt the compiler is set to pedandic and Werror, build fails on rviz_common/config.cpp:145 as there is no break in the case section when using gcc7 (gc 7.2.1 used here):
/home/felix/Downloads/ros2/src/ros2/rviz/rviz_common/src/rviz_common/config.cpp:145:9: error: this statement may fall through [-Werror=implicit-fallthrough=]
         }
         ^
/home/felix/Downloads/ros2/src/ros2/rviz/rviz_common/src/rviz_common/config.cpp:147:5: note: here
     case Value:
     ^~~~
At global scope:
cc1plus: error: unrecognized command line option ‘-Wno-gnu-zero-variadic-macro-arguments’ [-Werror]
cc1plus: all warnings being treated as errors```

I know that gcc7 is not officially supported, but this might still be unintentional? Otherwise a statement suppressing this warning or at least a comment why there is no break there would be nice I guess.

I've encountered that too and I think this actually is a bug inherited from the mainline. GCC7 simply added implicit-fallthrough warning to it's -Wextra.
My understanding is that there should be a break; statement, as falling through to the last case will erroneously set the type of the Config object to Value (within .setValue()).
Another issue is the unrecognized command line option ‘-Wno-gnu-zero-variadic-macro-arguments’: this is solved by explicitly setting the project type in CMakeLists.txt:
project(rviz_common CXX)

(solution found here)

The two issues above are in fact unrelated to each other - I'm not sure whether it would be better to open a PR for both of them at once or separately?

After fixing those issues I've hit another wall:
In file included from /usr/include/c++/7.2.1/ext/string_conversions.h:41:0,
                 from /usr/include/c++/7.2.1/bits/basic_string.h:6349,
                 from /usr/include/c++/7.2.1/string:52,
                 from /home/mjbogusz/documents/ros2/src/ros2/rviz/rviz2/src/main.cpp:31:
/usr/include/c++/7.2.1/cstdlib:75:15: fatal error: stdlib.h: No such file or directory.
 #include_next <stdlib.h>
               ^~~~~~~~~~
compilation terminated.

After setting CMAKE_VERBOSE_MAKEFILE to true and analyzing the compilation command I've noticed this little gem:
-isystem /usr/lib64/cmake/yaml-cpp/../../../include

That effectively makes /usr/include/stdlib.h go up on the list of possible stdlib.h files to include from cstdlib and therefore #include_next has no valid target (this helped me understand that).
I've traced it down to rviz_yaml_cpp_vendor triggering environment hooks and setting additional include directories even if yaml-cpp is found on the system. I've fixed that by wrapping the whole download/build/hook/package block in an if statement:
find_package(yaml-cpp QUIET)
if(NOT yaml-cpp_FOUND)
    (...)
else()
    ament_package()
endif()

To ensure proper linking of yaml-cpp however the libraries for rviz_yaml_cpp_vendor had to be set so I've added another cmake 'extras' file doing just that.
I'll create a PR soon.

Looking forward to your PR :)

The break is needed ?
From https://developers.redhat.com/blog/2017/03/10/wimplicit-fallthrough-in-gcc-7/
I use regex /* FALLTHRU */

I created the following pull requests to fix the issues listed here: #157, #158, #159.

@Theosakamg yes, the break is indeed needed there. It's an upstream bug.
@racko I have a slightly more complete soliution for the yaml-cpp thingy but I want to clean it up and test it first.

Since the originally reported issue has been resolved in #158 I'm going to close this.


Add CSS-style attribute assignment

The goal of this PR is to come up with a nicer API for assigning HTML id and class attributes that clash with Python syntax. One approach is to overload the __getitem__ method to allow for this:
from html5tagger import Document

doc = Document("Test")
doc.div["#some-id.one.two.three[href=/some/path][another='foo bar']"]

print(str(doc))

<!DOCTYPE html>
<meta charset="utf-8">
<title>Test</title>

There are a few considerations:

Whether the value passed to __getitem__ should be a single string, or a tuple of strings: ["#id", "class", ...]
Whether the values (particularly class names) should be dot (.) prepended: .one.two.three v one two three
Whether arbitrary attributes should be supported in this pattern

The implementation I proposed answers these as by opting for a single string that follows CSS-style selector. While potentially we could go with #some-id one two three and just check the first character for a #, in which case we pop off the one value, this pattern was not chosen because it then means that this lib would be introducing a new pattern that is not found in HTML.
Alternatively, we could have multiple values builder["#some-id","one two three"]. While this is valid Python, it is a bit awkward (granted, any overload of __getitem__ is certainly non-standard).
Finally, assuming "#some-id.one.two.three" is preferred over "#some-id one two three", the next logical question is why not arbitrary attribute assignment with [foo=bar] style syntax.
As a side benefit, it would allow a data-foo_bar=thing attribute, if that is desired.

Needs README update I guess, but is there anything else stopping this from merging?


dropbox: add libGL to the FHS environment

Adds libGL.so.1 to the environment, as dropbox v209 has stopped shipping libGL.so which otherwise causes a crash on startup if $DISPLAY is set.
Things done

Built on platform(s)

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

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

[ ] sandbox = relaxed
[ ] sandbox = true

[ ] Tested, as applicable:

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

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

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

[ ] Fits CONTRIBUTING.md.

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

This pull request has been mentioned on NixOS Discourse. There might be relevant details there:
https://discourse.nixos.org/t/dropbox-stopped-working/53942/10


disksort: save IO by putting in-memory shard directly into merger heap [phab:D2100]

Created by schroederc at 2017-12-12 18:42:32:

zarko accepted this revision at 2017-12-12 18:46:41

schroederc closed via commit a6e864b2359cbb34e32f56604c61d99c8e2aa669 at 2017-12-12 18:49:56